Created attachment 8515140 [details] [diff] [review] Fix keyword in test
Attachment #8515140 - Flags: review?(dbaron)
Attachment #8515140 - Flags: review?(dbaron) → review+
(In reply to Rik Cabanier from comment #2) > try build: > https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=27553f066dab You probably don't need an all-platform try build for that. (Also, it doesn't seem like the try push will actually test anything useful given that you didn't push the pref change in that try push.) See also https://secure.pub.build.mozilla.org/builddata/reports/reportor/daily/highscores/highscores.html ; we don't want to use resources unnecessarily, though we should certainly keep using what we need. Honestly, I think you can just push this without a try push given that it's pref'd out. But you'd want a try push of bug 1091885 *and* this patch before pushing bug 1091885.
Comment on attachment 8515140 [details] [diff] [review] Fix keyword in test https://hg.mozilla.org/integration/mozilla-inbound/rev/6dda3b5dc0a9
Attachment #8515140 - Flags: checkin+
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in before you can comment on or make changes to this bug.