533017 - Bring Fennec to the top when clicking on it from the hildon desktop

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Stuart Parmenter]

Because we have the X-Osso-Service in our .desktop file, rather than our exec being called our application receives a top_application dbus message, which we don't currently handle.  I believe we we need to simply setup osso_application_set_top_cb per http://maemo.org/api_refs/5.0/5.0-final/libosso/ docs, and run some code similar to http://mxr.mozilla.org/mobile-browser/source/components/BrowserCLH.js#55 in our callback

Comment 1

[Mark Finkle (:mfinkle) (use needinfo?)]

Attachment: WIP 1

I think this is what stuart has in mind. I need to test this to make sure the commandline is passed and the BrowserCLH.js code is executed correctly.

Comment 2

[Mark Finkle (:mfinkle) (use needinfo?)]

Attachment: patch

This patch implements the dbus rpc callback on maemo. The patch handles "top_application" which occurs when a shortcut is used to start an app and it handles "launch_url" and "open_new_window" which pass arguments.

Arguments are converted to nsCommandLine-friendly types. BrowserCLH.js handles both cases (arguments or no arguments) and will focus the running window, optionally opening new tabs based on the any arguments.

This is a Maemo only change

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

You can test "load_url" using this command line:

dbus-send --system --type=method_call --dest="mozilla.fennec" --print-reply /mozilla/fennec/request mozilla.fennec.load_url string:"http://ebay.com"

Comment 4

[Mark Finkle (:mfinkle) (use needinfo?)]

Attachment: patch 2

While doing more testing, I found that handling "top_application" in commandline handlers had the bad side effect of opening two (2) windows when launching from desktop. Launching from commandline (symlink or real binary) did not open two windows. The "top_application" method is fired by dbus on startup and was causing nsDefaultCLH to open a second Fennec window.

To work around this, the new patch treats the "top_application" method internally - it does not pass it off to the commandline handlers. We just focus the topmost Fennec window, which brings the application to the front.

Other methods are still passed off to the commandline handlers, since we have some use cases for supporting other methods.

Comment 5

[Mark Finkle (:mfinkle) (use needinfo?)]

pushed m-c:
http://hg.mozilla.org/mozilla-central/rev/c3a97b86e763

Comment 6

[Mark Finkle (:mfinkle) (use needinfo?)]

Attachment: pushed patch

Pushed the patch with an addition which was IRL r+ by DougT

It added a check for the "quit" method

Comment 7

[Mark Finkle (:mfinkle) (use needinfo?)]

pushed m-192:
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/8df91edf9ecd