Closed Bug 1461676 Opened 6 years ago Closed 6 years ago

Expand raptor firefox tp6 to include the other pages

Categories

(Testing :: Raptor, enhancement)

Version 3
enhancement
Not set
normal

Tracking

(firefox62 fixed)

RESOLVED FIXED
mozilla62
Tracking Status
firefox62 --- fixed

People

(Reporter: rwood, Assigned: rwood)

References

Details

Attachments

(1 file)

The raptor-firefox-tp6 test currently runs on try against the amazon test pageset. Expand it to run against the other tp6 pages (facebook, google, and youtube). Each page will be a subtest.
Depends on: 1462776
Assignee: nobody → rwood
Status: NEW → ASSIGNED
Comment on attachment 8982322 [details]
Bug 1461676 - Expand raptor firefox tp6 to include the other pages;

https://reviewboard.mozilla.org/r/248268/#review254528

do we need to update the webext rules for matching the pages/urls?  I recall *://*amazon.com* while reviewing code for benchmarking.
Attachment #8982322 - Flags: review?(jmaher) → review+
(In reply to Joel Maher ( :jmaher ) (UTC-4) from comment #3)
> Comment on attachment 8982322 [details]
> Bug 1461676 - Expand raptor firefox tp6 to include the other pages;
> 
> https://reviewboard.mozilla.org/r/248268/#review254528
> 
> do we need to update the webext rules for matching the pages/urls?  I recall
> *://*amazon.com* while reviewing code for benchmarking.

Thank you. No sir not any more, earlier I set the measure content script to <all-urls> to avoid that. :)
you changed that for the speedometer benchmarking from all_urls -> *amazon*; I just r-'d the patch
(In reply to Joel Maher ( :jmaher ) (UTC-4) from comment #5)
> you changed that for the speedometer benchmarking from all_urls -> *amazon*;
> I just r-'d the patch

Ah thanks, I'll update that again
Pushed by rwood@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3366a2b43d41
Expand raptor firefox tp6 to include the other pages; r=jmaher
https://hg.mozilla.org/mozilla-central/rev/3366a2b43d41
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: