Closed Bug 1253382 Opened 8 years ago Closed 8 years ago

Add a new test that measures how long it takes to paint the content for newly opened tabs

Categories

(Testing :: Talos, defect)

defect
Not set
normal

Tracking

(firefox47 fixed)

RESOLVED FIXED
mozilla47
Tracking Status
firefox47 --- fixed

People

(Reporter: mconley, Unassigned)

References

Details

Attachments

(3 files)

There are two cases to consider with this test:

1) The tab has been opened from the parent
2) The tab has been opened from content via a target="_blank" link, for example

This was spun out of the work done in bug 1251732, as we suspect there's a slow path here for e10s that we're not currently measuring.
Blocks: 1251547, 1174770
This is necessary in order to be able to open new tabs from the initial
browser window, since the initial browser window needs to have its
BrowserDOMWindow attribute set when it goes remote - and if it fires
before the onLoad handler in browser.js, the BrowserDOMWindow attribute
that's set on the XUL window (which gets copied over to the remote browser),
hasn't been set yet.

Review commit: https://reviewboard.mozilla.org/r/38013/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/38013/
tabpaint measures how long it takes to open a new tab and show its content.

This is a mostly done work-in-progress and isn't yet ready for review.

Review commit: https://reviewboard.mozilla.org/r/38015/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/38015/
Attachment #8726458 - Attachment description: MozReview Request: Bug 1253382 - [WIP] Add a new tabpaint talos test. r?jmaher → MozReview Request: Bug 1253382 - Add a new tabpaint talos test. r?jmaher
Attachment #8726458 - Flags: review?(jmaher)
Comment on attachment 8726458 [details]
MozReview Request: Bug 1253382 - Add a new tabpaint talos test. r?jmaher

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/38015/diff/1-2/
Comment on attachment 8726458 [details]
MozReview Request: Bug 1253382 - Add a new tabpaint talos test. r?jmaher

https://reviewboard.mozilla.org/r/38015/#review34679

keep in mind that we need to sign addons now, so any new addons or changes to existing ones need a signed addon.  If you update the source, you can see it done for all other addons.  If you look in test.py you will see references to the .xpi instead of the source.

Some tips for signing addons:
https://wiki.mozilla.org/EngineeringProductivity/HowTo/SignExtensions

::: testing/talos/talos/test.py:254
(Diff revision 2)
> +        'browser.link.open_newwindow.restriction': 2,

can you provide a brief comment about what these values of 3 and 2 mean?
Attachment #8726458 - Flags: review?(jmaher)
Comment on attachment 8726457 [details]
MozReview Request: Bug 1253382 - Force the initial browser remoteness only after the browser's onLoad has fired. r?jmaher

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/38013/diff/1-2/
Attachment #8726457 - Flags: review?(jmaher)
Comment on attachment 8726458 [details]
MozReview Request: Bug 1253382 - Add a new tabpaint talos test. r?jmaher

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/38015/diff/2-3/
Attachment #8726458 - Flags: review?(jmaher)
Comment on attachment 8726457 [details]
MozReview Request: Bug 1253382 - Force the initial browser remoteness only after the browser's onLoad has fired. r?jmaher

https://reviewboard.mozilla.org/r/38013/#review34755

thanks, this looks nice and simple.
Attachment #8726457 - Flags: review?(jmaher) → review+
Comment on attachment 8726802 [details]
MozReview Request: Bug 1253382 - Sign the most recent version of TalosPowers. r?jmaher

https://reviewboard.mozilla.org/r/38241/#review34759

ship it real good
Attachment #8726802 - Flags: review?(jmaher) → review+
Comment on attachment 8726458 [details]
MozReview Request: Bug 1253382 - Add a new tabpaint talos test. r?jmaher

https://reviewboard.mozilla.org/r/38015/#review34761

thanks for the signed addon, talos.json changes and the comments in test.py.

Please add an entry here:
https://wiki.mozilla.org/Buildbot/Talos/Tests
Attachment #8726458 - Flags: review?(jmaher) → review+
this is for winxp e10s (regular and pgo).  I don't think it is expected, most likely the cause of the pageloader addon change:
https://hg.mozilla.org/integration/mozilla-inbound/rev/64db01968f1d
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: