After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 642976 - blocked on select file to send via bluetooth
blocked on select file to send via bluetooth
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2011-02-22 17:37 UTC by sathyz
Modified: 2011-03-02 15:31 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
choose file to send (8.55 KB, text/plain)
2011-02-22 17:37 UTC, sathyz
  Details
PopupMenuManager: don't throw exceptions if don't have focus (1.08 KB, patch)
2011-02-22 19:58 UTC, Giovanni Campagna
committed Details | Review

Description sathyz 2011-02-22 17:37:12 UTC
Created attachment 181606 [details]
choose file to send

I was trying to send a file using bluetooth to a mobile phone device on gnome-shell and the window hanged on "choose file to send".

Steps:
1. click on "Bluetooth" icon on system status area
2. Click on My Mobile device (micromax q7 in my case) -> Send files
3. A modal window appeared for "choose files to send" and hanged. It listed the files/folders in my home directory but unable to select.

I was able to come out on clicking Bluetooth -> bluetooth settings.

Attached is the error message shown on terminal.
Comment 1 Giovanni Campagna 2011-02-22 19:58:10 UTC
Created attachment 181628 [details] [review]
PopupMenuManager: don't throw exceptions if don't have focus

stage.key_focus can be null, but passing null to _activeMenuContains
throws (from gjs marshalling). In that case, just assume it returned
false.
Comment 2 Dan Winship 2011-02-23 19:16:25 UTC
Comment on attachment 181628 [details] [review]
PopupMenuManager: don't throw exceptions if don't have focus

>+            let fromActive = false;
>+            if (focus)
>+                this._activeMenuContains(focus);

ok, but I'd just say

    let fromActive = focus && this._activeMenuContains(focus);
Comment 3 Giovanni Campagna 2011-03-02 15:31:09 UTC
Attachment 181628 [details] pushed as b9cb37d - PopupMenuManager: don't throw exceptions if don't have focus