Firefox's awesomebar and context menus don't work when a webpage is opened from a lens

Bug #1050687 reported by Mark Tully
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Unity
Fix Released
High
Neil J. Patel
unity (Ubuntu)
Fix Released
High
Unassigned
Quantal
Fix Released
High
Unassigned

Bug Description

When a result in a lens is a webpage which opens using Firefox, the webpage will open in a new tab in Firefox, but Firefox's context menu will stop working corectly (it will open once, but only once) Furthermore, the location bar dropdown will also no longer open until Firefox is restarted and suggestions will not be offered when filling in form fields.

I have tested this using the askubuntu scope (unity-scope-askubuntu) of the help lens (unity-lens-help) and also using the reddit lens from my own PPA. It has also been reoprted on askubuntu as an issue in Ubuntu 12.04.1 using unity-lens-askubuntu from the software centre: http://askubuntu.com/questions/184437/bug-involving-the-launch-of-firefox-from-askubuntu-lens

A workaround for this appears to be using a specific command to open the webbrowser in 'activate_uri' rather than letting unity automatically open the result.

eg: in the askubuntu scope, connecting the 'activate_uri' signal to the scope and making it run the following:
def activate_uri(self, scope, uri):
        webbrowser.open(uri)
        return Unity.ActivationResponse(handled=Unity.HandledType.HIDE_DASH, goto_uri='')

Revision history for this message
Chris Coulson (chrisccoulson) wrote :

This is a Unity bug (it's another case of it launching applications with a completely bogus timestamp)

affects: firefox (Ubuntu) → unity (Ubuntu)
Revision history for this message
Chris Coulson (chrisccoulson) wrote :

Just here, in fact: http://bazaar.launchpad.net/~unity-team/unity/trunk/view/head:/dash/DashView.cpp#L724

Unity should not be passing the system time, it should be passing the X timestamp from the event that generated the activation. GTK updates the last event timestamp in Firefox with the broken timestamp from Unity when this happens, and because it is so far ahead of the X server timestamp it then breaks everything that depends on it (eg, mouse grabs). This is probably the cause of the random menu popup issues we continue to see on Precise, which only occur under Unity.

affects: unity (Ubuntu) → firefox (Ubuntu)
affects: firefox (Ubuntu Quantal) → unity (Ubuntu Quantal)
Changed in unity (Ubuntu Quantal):
importance: Undecided → High
status: New → Triaged
Neil J. Patel (njpatel)
Changed in unity (Ubuntu Quantal):
milestone: none → ubuntu-12.10-beta-2
Changed in unity:
milestone: none → 6.6
Changed in unity (Ubuntu Quantal):
milestone: ubuntu-12.10-beta-2 → ubuntu-12.10
Changed in unity:
milestone: 6.6 → 7.0
milestone: 7.0 → 6.6
Neil J. Patel (njpatel)
Changed in unity (Ubuntu Quantal):
milestone: ubuntu-12.10 → ubuntu-12.10-beta-2
Neil J. Patel (njpatel)
Changed in unity:
assignee: nobody → Neil J. Patel (njpatel)
importance: Undecided → High
status: New → Confirmed
Omer Akram (om26er)
Changed in unity:
status: Confirmed → Fix Committed
Changed in unity (Ubuntu Quantal):
status: Triaged → Fix Committed
Changed in unity:
milestone: 6.6 → 7.0
Omer Akram (om26er)
no longer affects: unity/6.0
Changed in unity:
milestone: 7.0 → 6.6
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (9.2 KiB)

This bug was fixed in the package unity - 6.6.0-0ubuntu1

---------------
unity (6.6.0-0ubuntu1) quantal-proposed; urgency=low

  * New upstream release.
    - Fixes non-escaped character sequences in dash previews (LP: #1039020)
    - Updated background layer for preview cover-art and details panels
      to be 10% low-light
    - Expand a PlacesGroup if it is the only category that contains results
      (LP: #950710)
    - Update unity autopilot tests to match autopilot API
    - Updated the convert files to fix some typos in the key names
    - Add gmodule dependency
    - Activate proper result if the categories aren't displayed in-order
      (LP: #1040101)
    - Refactor device launcher icons (LP: #713423)
    - LauncherController: make the controller enable the launcher struts,
      based on hide-mode option (LP: #1044005)
    - Launcher: make always possible to drag an icon to the bottom or top
      of its sub list (LP: #1043968)
    - Don't desat bfb/hud icon in DNDReset (LP: #1043963)
    - Progressively adjust the speed of the minimize animation. First
      time it is used is slower, then speeds up the more it is used.
      (LP: #1017510)
    - Implement new ordering of categories for home lens. (LP: #1043915)
    - UnityWindow now implements ScaleWindowInterface (LP: #876017)
    - Launcher: restore an icon position after that the dragging has been
      cancelled (LP: #955561)
    - LauncherDragWindow: cancel drag on window mapped/unmapped
      (LP: #1044723)
    - Now there is a check of an override color in RefreshColor (which is
      called when a PropertyNotify event happens). Also added a check in
      FullySaturateColor to a void division by zero. (LP: #975350)
    - Queue redraw after cover-art texture is updated from a url/file source.
      (LP: #1043947)
    - Fixed ability to delete glib::Source wrapper during its callback
      (LP: #1044823)
    - Close preview when dash is hidden. (LP: #1045298)
    - LauncherModel: rewrite the Reordering functions to keep the icon
      priority deltas (LP: #761155)
    - Make sure we can pass extra hints when activating preview actions.
      (LP: #1046352)
    - UnityWindow: scale window code improved (LP: #1033935)
    - The mouse will now cause the HUD buttons to change selection
      (LP: #1042692)
    - "Alt+Space" shortcut to reveal the window menu is not hardcoded, but a
       Compiz key option. " (Hold)" should also be translated. Made all
       Compiz plug-in names and all Compiz plug-in option names in
       unityshell.cpp static constants.
    - remove unity --reset, it's not anymore really needed now that we are
      in stable days of unity and we moved to gsettings
    - Removed the variables 'oldPrev' and 'oldNext' which got assigned the
      value NULL, but then were never used
    - Fixed the size of the previews to 770x380 pixels. (LP: #1045243)
    - UnityWindow: use smart pointers, use static close_icon (with dynamic
      state) and PanelStyle context (LP: #1033935) (LP: #1045127)
      (LP: #1046124) (LP: #1046126)
    - Remove everything in the #ifndef USE_MODERN_COMPIZ_GL ifdefs and remove
      the ifdefs alltogether. unity now requires compiz...

Read more...

Changed in unity (Ubuntu Quantal):
status: Fix Committed → Fix Released
Changed in unity:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.