Bug 105431 (deCOM)

DeCOMtamination tasks

NEW
Unassigned

Status

()

P5
normal
17 years ago
7 days ago

People

(Reporter: waterson, Unassigned)

Tracking

(Depends on: 17 bugs, Blocks: 1 bug, {memory-footprint, meta, perf})

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

17 years ago
Meta-bug to track removal of unneeded interface layers.
(Reporter)

Updated

17 years ago
Status: NEW → ASSIGNED
Depends on: 92144, 96108, 104336, 105430
Keywords: footprint, meta, perf

Updated

17 years ago
Blocks: 71668, 92580

Updated

17 years ago
Depends on: 92141

Updated

17 years ago
Depends on: 75620

Updated

17 years ago
Depends on: 106152

Updated

17 years ago
Summary: DeCOMtaminiation tasks → DeCOMtamination tasks
(Reporter)

Updated

17 years ago
Depends on: 116551

Updated

17 years ago
Depends on: 120818
We need to deCOMtaminate most of the content and layout DLL internals.  Bryner's
keen to do it and just the man for the job!  I'll let him file more specific
bugs tracked here.

/be
Assignee: waterson → bryner
Status: ASSIGNED → NEW

Comment 2

16 years ago
Here is our recent reportcard form performance analysis that we periodicially
get from In*&?...

-Great job with DeCOMtamination efforts
-Unnecessary virtual functions can cause a great deal of overhead –  a cache
line for each vtable & active object, indirection to call a function that isn’t
easily predicted and inefficiencies from get/setters and other small virtuals
that can’t be inlined due to calls through an interface
-Great job on reducing total instruction count per page load – probably 25%
improvement or more from 6.0 (but keep it up  - still ~1.5x vs IE)

Keep going on this!  We need the decomtamination of layout to happen as soon as
possible...
For anyone doing this kind of work --- after you've come up with a cleaned-up
interface, please consider appealing to drivers and super-reviewers for a
"review plus superreview in advance" blank cheque to go ahead and update all the
uses of that interface without having to get specific module owner review and
superreview for the changes.

Updated

16 years ago
Blocks: 203448
Assignee: bryner → nobody
QA Contact: scc → xpcom
There are currently quite a few open bugs relating to deCOM; some of them have not seen attention in years.  Here is a search that will find them (as long as the string "decom" appears in the summary ;-) -- the boolean chart is necessary to avoid a bunch of false hits.

https://bugzilla.mozilla.org/buglist.cgi?short_desc=decom;resolution=---;negate0=1;field0-0-0=short_desc;type0-0-0=substring;value0-0-0=decomp;field0-0-1=short_desc;type0-0-1=substring;value0-0-1=decomm

I'm going to go through the list and ping all the old bugs, and I've marked all of the currently open ones as dependencies of this one, but let this also serve as a heads-up to reviewers and committers.

Updated

9 years ago
Alias: deCOM

Updated

9 years ago
Depends on: 566808
Depends on: 576794

Updated

8 years ago
Depends on: 575901

Updated

8 years ago
Depends on: 576831

Updated

8 years ago
Depends on: 576877

Updated

8 years ago
Blocks: 577002

Updated

8 years ago
No longer blocks: 577002
Depends on: 577002

Updated

8 years ago
Blocks: 577974

Updated

8 years ago
No longer blocks: 577974
Depends on: 577974

Updated

8 years ago
Depends on: 577976
Depends on: 585137

Updated

8 years ago
Depends on: 590390

Updated

8 years ago
Depends on: 595103
Depends on: 610305
Depends on: 619996

Updated

8 years ago
Depends on: 631464

Updated

8 years ago
Depends on: 635170

Updated

8 years ago
Depends on: 637555

Updated

8 years ago
Depends on: 174055

Updated

8 years ago
Depends on: 648925

Updated

8 years ago
Depends on: 655655
Depends on: 656996
Depends on: 657160
Depends on: 657210
Depends on: 660604
Depends on: 677085
Depends on: 678813
Depends on: 680417
Depends on: 682611
Depends on: 684821

Updated

7 years ago
Depends on: 697032
Depends on: 697475

Updated

7 years ago
Depends on: 734008
Depends on: 737976

Updated

6 years ago
Priority: -- → P5
Depends on: 864165
Depends on: 868726
Depends on: 956015
Depends on: 969035
Depends on: 969331
Depends on: 1017758
Depends on: 1293596
Depends on: 1296993
Depends on: 1299335
Depends on: 1325234
Depends on: 1007090
Depends on: 1336437
Depends on: 1378060
Depends on: 1392883
Depends on: 1176342
Depends on: 1477576
You need to log in before you can comment on or make changes to this bug.