624017 - Move more JavaScript into delay-loaded files

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Matt Brubeck (:mbrubeck)]

Attachment: patch

This moves several JavaScript objects from common-ui.js into individual files, reducing the size of common-ui.js by 800 lines.

This patch only affects objects that are loaded after startup.  The remaining objects in common-ui.js are all loaded at startup (during UIReadyDelayed), so there's no performance gain from moving them to separate files.

The attached patch is pure copy/paste, with no modifications in the JavaScript files themselves.

Comment 1

[Matt Brubeck (:mbrubeck)]

Attachment: patch v2

Forgot to update one line when I changed the file naming.

Comment 2

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

Comment on attachment 502107 [details] [diff] [review]
patch v2

>+    {
>+      name: "Google Reader",
>+      callback: function callback(aURL, aTitle) {
>+        let url = "http://www.google.com/reader/link?url=" + encodeURIComponent(aURL) +
>+                  "&title=" + encodeURIComponent(aTitle);
>+        BrowserUI.addTab(url, Browser.selectedTab);
>+      }

Change to BrowserUI.newTab(url, Browser.selectedTab);

(I think this bug was filed somewhere)

I'm not the biggest fan of separate files, but it might be better for performance. I guess we'll find out.

We can always re-org later as needed.

Comment 3

[Matt Brubeck (:mbrubeck)]

http://hg.mozilla.org/mobile-browser/rev/f8add7971e4b

Comment 4

[Martijn Wargers (dead)]

I could only find this data about ts, to see if this patch could have influenced ts:
http://graphs.mozilla.org/graph.html#tests=[[16,11,621]]&sel=1294157766,1294675722
Perhaps it did, but I have no idea, really.

Comment 5

[Martijn Wargers (dead)]

Is there any other way to verify this bug other than code inspection?

Comment 6

[Matt Brubeck (:mbrubeck)]

(In reply to comment #5)
> Is there any other way to verify this bug other than code inspection?

No, this change should be transparent to users (except for possible Ts improvement).

Comment 7

[Andreea Pod]

Then can somebody please verify this?