Closed
Bug 1508399
Opened 6 years ago
Closed 6 years ago
about:blank test should load 8 tabs
Categories
(Testing :: AWSY, enhancement)
Tracking
(firefox65 fixed)
RESOLVED
FIXED
mozilla65
Tracking | Status | |
---|---|---|
firefox65 | --- | fixed |
People
(Reporter: erahm, Assigned: erahm)
Details
Attachments
(1 file)
1.49 KB,
patch
|
bc
:
review+
|
Details | Diff | Splinter Review |
The about:blank base case test -- SY(ab) -- should load the full amount of 'dom.ipc.processCount' tabs so that we're sure there's not a preallocated content process being measured.
Assignee | ||
Comment 1•6 years ago
|
||
Attachment #9026472 -
Flags: review?(bob)
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → erahm
Status: NEW → ASSIGNED
Comment 2•6 years ago
|
||
Comment on attachment 9026472 [details] [diff] [review] Use dom.ipc.processCount to determine how many tabs to open Review of attachment 9026472 [details] [diff] [review]: ----------------------------------------------------------------- r+ lgtm. I spun my wheels for a bit trying to test this until I realized that the mach interface can't run the base test. That would nice in the future I think. Anyway, the marionette get_pref should be available in the setUp.
Attachment #9026472 -
Flags: review?(bob) → review+
Assignee | ||
Comment 3•6 years ago
|
||
(In reply to Bob Clary [:bc:] from comment #2) > Comment on attachment 9026472 [details] [diff] [review] > Use dom.ipc.processCount to determine how many tabs to open > > Review of attachment 9026472 [details] [diff] [review]: > ----------------------------------------------------------------- > > r+ lgtm. I spun my wheels for a bit trying to test this until I realized > that the mach interface can't run the base test. That would nice in the > future I think. Anyway, the marionette get_pref should be available in the > setUp. Since it's just a marionette test you can just specify the test file: > ./mach awsy-test testing/awsy/awsy/test_base_memory_usage.py To make sure it's behaving like automation you can specify a prefs file: > ./mach awsy-test --preferences testing/awsy/conf/base-prefs.js testing/awsy/awsy/test_base_memory_usage.py
Pushed by erahm@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/22ce80311bcf Use dom.ipc.processCount to determine how many tabs to open. r=bc
Comment 5•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/22ce80311bcf
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox65:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
You need to log in
before you can comment on or make changes to this bug.
Description
•