Closed Bug 1508399 Opened 6 years ago Closed 6 years ago

about:blank test should load 8 tabs

Categories

(Testing :: AWSY, enhancement)

Version 3
enhancement
Not set
normal

Tracking

(firefox65 fixed)

RESOLVED FIXED
mozilla65
Tracking Status
firefox65 --- fixed

People

(Reporter: erahm, Assigned: erahm)

Details

Attachments

(1 file)

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: nobody → erahm
Status: NEW → ASSIGNED
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+
(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
https://hg.mozilla.org/mozilla-central/rev/22ce80311bcf
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: