Closed
Bug 993689
Opened 11 years ago
Closed 11 years ago
Provide native js shell to tarako b2g builds
Categories
(Release Engineering :: General, defect)
Tracking
(blocking-b2g:1.3T+, b2g-v1.3T fixed)
People
(Reporter: catlee, Assigned: catlee)
Details
Attachments
(1 file)
1.33 KB,
patch
|
fabrice
:
review+
catlee
:
checked-in+
|
Details | Diff | Splinter Review |
If we have a native js shell, then the build system minifies the js sources, resulting in a smaller omni.ja
Attachment #8403566 -
Flags: review?(fabrice)
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → catlee
Updated•11 years ago
|
Attachment #8403566 -
Flags: review?(fabrice) → review+
Assignee | ||
Comment 1•11 years ago
|
||
Comment on attachment 8403566 [details] [diff] [review]
js-mozilla-b2g28_v1_3t.patch.0
https://hg.mozilla.org/releases/mozilla-b2g28_v1_3t/rev/a5e49b9b5c9f
Attachment #8403566 -
Flags: checked-in+
Updated•11 years ago
|
blocking-b2g: --- → 1.3T+
Assignee | ||
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Comment 2•11 years ago
|
||
Can you give me the memory shrink data ?
I found homescreen memory usage is less than before.
Comment 3•11 years ago
|
||
(In reply to James Zhang from comment #2)
> Can you give me the memory shrink data ?
> I found homescreen memory usage is less than before.
I didn't run memory usage numbers recently but that used to be pretty significant: https://bugzilla.mozilla.org/show_bug.cgi?id=903149#c0
omni.ja is about 1MB smaller as far as ROM size is concerned.
Updated•11 years ago
|
status-b2g-v1.3T:
--- → fixed
Assignee | ||
Comment 4•11 years ago
|
||
Fabrice, is this worthwhile doing on other devices as well?
Flags: needinfo?(fabrice)
Comment 5•11 years ago
|
||
(In reply to Chris AtLee [:catlee] from comment #4)
> Fabrice, is this worthwhile doing on other devices as well?
I tend to think so, but only for user builds. It's kind of hard to debug minified chrome js and we have no sourcemaps yet.
Flags: needinfo?(fabrice)
Updated•7 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•