Closed
Bug 944155
Opened 11 years ago
Closed 6 years ago
Build b2g with --disable-xul
Categories
(Firefox OS Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: ehsan.akhgari, Assigned: fabrice)
Details
Attachments
(4 files)
77.62 KB,
patch
|
Details | Diff | Splinter Review | |
41.06 KB,
patch
|
Details | Diff | Splinter Review | |
685.55 KB,
text/plain
|
Details | |
668.89 KB,
text/plain
|
Details |
Technically, we should not have any dependencies on XUL. It would be nice if we could avoid building and shipping this code. Michael, any idea who the right person to work on this would be? (And what the right component for this bug would be?)
Flags: needinfo?(mwu)
Comment 1•11 years ago
|
||
I could try to build it tonight and see what problem we run into.
Reporter | ||
Comment 2•11 years ago
|
||
(In reply to comment #1) > I could try to build it tonight and see what problem we run into. That would be great, thanks!
Assignee | ||
Comment 3•11 years ago
|
||
I started on that when working on bug 894927. I guess I could rebase my patch... Things that I'm sure are broken when removing xul support: - scrollbars - html <menu> (it uses <xul:menu>) - video and audio controls
Reporter | ||
Comment 4•11 years ago
|
||
(In reply to Fabrice Desré [:fabrice] from comment #3) > I started on that when working on bug 894927. I guess I could rebase my > patch... > > Things that I'm sure are broken when removing xul support: > - scrollbars > - html <menu> (it uses <xul:menu>) Sigh :( > - video and audio controls I thought they just use XBL?
Assignee | ||
Comment 5•11 years ago
|
||
(In reply to :Ehsan Akhgari (needinfo? me!) from comment #4) > > - video and audio controls > > I thought they just use XBL? They use xbl, but the controls themselves are xul elements. I'm fairly sure we could move that to html since we now have a nice flex model available here also.
Reporter | ||
Comment 6•11 years ago
|
||
(In reply to comment #5) > (In reply to :Ehsan Akhgari (needinfo? me!) from comment #4) > > > > - video and audio controls > > > > I thought they just use XBL? > > They use xbl, but the controls themselves are xul elements. Yes, you're right. > I'm fairly sure we > could move that to html since we now have a nice flex model available here > also. Yeah, I think we should be able to do that too.
Comment 7•11 years ago
|
||
The scrollbars I see in gaia are done in JS, are you sure they need XUL? Sounds like it's not much to fix. How much code would that save us? It's particularly bad if we -barely- use it because we have to map these pages into memory but could avoid using it with little effort.
Assignee | ||
Comment 8•11 years ago
|
||
Not sure what you mean by "done in js". They are "standard" scrollbars that use xul:thumb and xul:scrollbar. We style them in https://mxr.mozilla.org/mozilla-central/source/b2g/chrome/content/content.css#13 Once we'll apzc everywhere I think the responsibility of drawing scrollbars should be at the layout/gfx level. So no big deal for this bug indeed. I'll post what I have later today once I get everything building again.
Assignee: nobody → fabrice
OS: Mac OS X → Gonk (Firefox OS)
Hardware: x86 → All
Comment 10•11 years ago
|
||
From Bug 894927 comment 26, I think we can save at least 1~2 MB on chrome process.
Comment 11•11 years ago
|
||
(In reply to Benoit Girard (:BenWa) from comment #7) > The scrollbars I see in gaia are done in JS, are you sure they need XUL? AFAIK Gaia has nothing to do with scrollbar.
Comment 12•11 years ago
|
||
(In reply to Alive Kuo [:alive][NEEDINFO] from comment #11) > (In reply to Benoit Girard (:BenWa) from comment #7) > > The scrollbars I see in gaia are done in JS, are you sure they need XUL? > > AFAIK Gaia has nothing to do with scrollbar. Right that was a mistake.
(In reply to Kai-Zhen Li from comment #10) > From Bug 894927 comment 26, I think we can save at least 1~2 MB on chrome > process. Why do you believe that that comment is relevant to this?
Also, even if it is, decreasing VSS is not particularly useful. Decreasing RSS is what matters. The phone is limited by physical memory, not virtual address space, so if binary code gets loaded into VSS and discarded because it's not used it doesn't affect us much.
Comment 15•11 years ago
|
||
(In reply to Kyle Huey [:khuey] (khuey@mozilla.com) from comment #14) > Also, even if it is, decreasing VSS is not particularly useful. Decreasing > RSS is what matters. The phone is limited by physical memory, not virtual > address space, so if binary code gets loaded into VSS and discarded because > it's not used it doesn't affect us much. Kyle, Thanks! Yes, we should focus on RSS.
Comment 16•11 years ago
|
||
(In reply to Kyle Huey [:khuey] (khuey@mozilla.com) from comment #14) > Also, even if it is, decreasing VSS is not particularly useful. Decreasing > RSS is what matters. The phone is limited by physical memory, not virtual > address space, so if binary code gets loaded into VSS and discarded because > it's not used it doesn't affect us much. From my understanding when you touch a code page it never gets paged out on mobile. So if we execute a large code surface of xul to provide the features mentioned above we're paying the cost permanently. Right?
(In reply to Benoit Girard (:BenWa) from comment #16) > (In reply to Kyle Huey [:khuey] (khuey@mozilla.com) from comment #14) > > Also, even if it is, decreasing VSS is not particularly useful. Decreasing > > RSS is what matters. The phone is limited by physical memory, not virtual > > address space, so if binary code gets loaded into VSS and discarded because > > it's not used it doesn't affect us much. > > From my understanding when you touch a code page it never gets paged out on > mobile. So if we execute a large code surface of xul to provide the features > mentioned above we're paying the cost permanently. Right? If that's true it would be very unfortunate.
Reporter | ||
Comment 18•11 years ago
|
||
(In reply to comment #17) > (In reply to Benoit Girard (:BenWa) from comment #16) > > (In reply to Kyle Huey [:khuey] (khuey@mozilla.com) from comment #14) > > > Also, even if it is, decreasing VSS is not particularly useful. Decreasing > > > RSS is what matters. The phone is limited by physical memory, not virtual > > > address space, so if binary code gets loaded into VSS and discarded because > > > it's not used it doesn't affect us much. > > > > From my understanding when you touch a code page it never gets paged out on > > mobile. So if we execute a large code surface of xul to provide the features > > mentioned above we're paying the cost permanently. Right? > > If that's true it would be very unfortunate. If we're using glandium's on the fly decompressor linker, then that would be true, since we don't have swap space to back those pages. But also note that currently the XUL binary code should be fairly hot given the scrollbar implementation.
Assignee | ||
Comment 19•11 years ago
|
||
That's my current wip rebased. It's not actually removing *all* of xul, for instance everything from under /dom/interfaces/xul is still there. I had to move some the nsIController(s) and the nsXULController implementation - they are actually not bound to any xul markup, and not building them breaks all the input fields. I don't have yet memory comparison numbers available.
Assignee | ||
Comment 20•11 years ago
|
||
This patch make us not build dom/interfaces/xul
Assignee | ||
Comment 21•11 years ago
|
||
Assignee | ||
Comment 22•11 years ago
|
||
Comment 23•11 years ago
|
||
I am implementing scrollbars in the compositor. We won't need any of the widget/xul code.
Updated•11 years ago
|
blocking-b2g: --- → 1.3?
Whiteboard: [tarako]
Comment 24•11 years ago
|
||
triage: remove dependency for tarako project as this should not have obvious gains for tarako
Comment 25•11 years ago
|
||
(given that this is assigned, assuming the right person for this has already been found)
Flags: needinfo?(mwu)
Comment 26•6 years ago
|
||
Firefox OS is not being worked on
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•