Closed Bug 1090995 Opened 10 years ago Closed 9 years ago

Investigate 2-3% regression on Session Restore Test caused by e10s printing work

Categories

(Core :: Printing: Output, defect)

Product:
Core
Component:
Printing: Output
Version:
32 Branch
Platform:
x86
All
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED WONTFIX
Tracking Flags:
Tracking Status
e10s + ---

People

(Reporter: mconley, Assigned: mconley)

References

Details

This happy note just landed in my inbox:

Regression: Fx-Team-Non-PGO - Session Restore Test - Ubuntu HW 12.04 - 2.19% increase
-------------------------------------------------------------------------------------
    Previous: avg 1656.621 stddev 8.695 of 12 runs up to revision 8546940128b3
    New     : avg 1692.825 stddev 9.202 of 12 runs since revision a9530d7f005a
    Change  : +36.204 (2.19% / z=4.164)
    Graph   : http://mzl.la/1wLFchW

Changeset range: http://hg.mozilla.org/integration/fx-team/pushloghtml?fromchange=8546940128b3&tochange=a9530d7f005a

Changesets:
  * http://hg.mozilla.org/integration/fx-team/rev/24a95294f600
    : Mike Conley <mconley@mozilla.com> - Bug 1082575 - Make PrintUtils and printPreviewBindings.xml more e10s friendly. r=Mossop.
    : http://bugzilla.mozilla.org/show_bug.cgi?id=1082575

  * http://hg.mozilla.org/integration/fx-team/rev/a9530d7f005a
    : Mike Conley <mconley@mozilla.com> - Bug 1082579 - Introduce PPrinting.ipdl and proxies for opening printing UI. r=blassey.
    : http://bugzilla.mozilla.org/show_bug.cgi?id=1082579

Bugs:
  * http://bugzilla.mozilla.org/show_bug.cgi?id=1082579 - Introduce PPrinting.ipdl and proxies for opening printing UI
  * http://bugzilla.mozilla.org/show_bug.cgi?id=1082575 - Make PrintUtils and printPreviewBindings.xml e10s friendly
tracking-e10s: --- → ?
Some try pushes to try to determine which patch introduced the regression:

Baseline (no PrintUtils or PPrinting.ipdl patch): https://tbpl.mozilla.org/?tree=Try&rev=b6c4bb73d4bd
Just PrintUtils patch: https://tbpl.mozilla.org/?tree=Try&rev=434ebf7a5751
Compare talos suggests that it was the PrintUtils patch that introduced this regression:

http://compare-talos.mattn.ca/?oldRevs=b6c4bb73d4bd&newRev=434ebf7a5751&server=graphs.mozilla.org&submit=true
Blocks: 1082575
Assignee: nobody → mconley
tracking-e10s: ? → +
any issues with me closing this?
Flags: needinfo?(mconley)
(In reply to Joel Maher (:jmaher) from comment #3)
> any issues with me closing this?

None.
Flags: needinfo?(mconley)
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.