956982 - Clean up omni.ja content

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Runtime, defect)

Description

[[:fabrice] Fabrice Desré]

The omni.ja we ship in b2g has many useless files. Let's do some cleanup, especially since we have size limitations on some platforms.

Comment 1

[[:fabrice] Fabrice Desré]

Attachment: wip cleanup-omnijar.patch

My current wip, with still a lot to be done. Currently this patch shrinks omni.ja from 5235483 bytes to 4943272 bytes.

Comment 2

[[:fabrice] Fabrice Desré]

Attachment: wip cleanup-omnijar.patch v2

With this patch, the one in bug 957241 and applying chrome js minification (bug 903149) I end up with an omni.ja that is 3424073 bytes, which is 1.8MB less than without any patch.

Comment 3

[James Zhang (Spreadtrum)]

Cheng, can you test this patch on tarako build?

Comment 4

[cheng.luo]

I suggest this patch should be landed to v1.3 as soon as possible.
I have to merge the patch manually one by one.
the result data about omni.ja on tarako as follow:
before  5305815
after   4783942

Comment 5

[[:fabrice] Fabrice Desré]

We can't take that on 1.3 either, it's fairly risky. I also need to clean it up since it's breaking b2g desktop builds.

Comment 6

[Jason Smith [:jsmith]]

marking blocking- as such with povb to indicate that this lands only in tarako

Comment 7

[Joe Cheng [:jcheng] (please needinfo)]

triage: 1.3T+ for tarako branch only

Comment 8

[Gregor Wagner [:gwagner]]

(In reply to Fabrice Desré [:fabrice] from comment #5)
> We can't take that on 1.3 either, it's fairly risky. I also need to clean it
> up since it's breaking b2g desktop builds.

Blocking and fairly risky is a bad combination. I think it should either land asap or we shouldn't block on it.

Comment 9

[Joe Cheng [:jcheng] (please needinfo)]

Fabrice, wonder if this is still being worked on? or do we still need this in Tarako? Thanks

Comment 10

[[:fabrice] Fabrice Desré]

I'd like to finish that but it's a bit risky now.

Comment 11

[[:fabrice] Fabrice Desré]

This patch is very likely to be the cause of bug 982592 - I'm clearing the 1.3t nomination since it's really too risky at this point.

Comment 12

[BMO Automation]

Firefox OS is not being worked on