Closed
Bug 1074156
Opened 10 years ago
Closed 10 years ago
Add support for content sandbox tests to TBPL
Categories
(Tree Management Graveyard :: TBPL, defect)
Tree Management Graveyard
TBPL
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bobowen, Assigned: bobowen)
References
Details
Attachments
(1 file, 1 obsolete file)
4.10 KB,
patch
|
bobowen
:
review+
|
Details | Diff | Splinter Review |
Bug 1074147 is adding plain mochitests running with the content sandbox for win32 on holly. This is to add support to tpbl for these new tests similar to the e10s ones.
Comment 1•10 years ago
|
||
Happy mentor you on this if you need pointers, but yeah, you should be able to follow the same pattern as the e10s tests.
Assignee | ||
Comment 2•10 years ago
|
||
I've just added these for the plain mochitests at the moment, didn't think it was worth adding for the others as we may not use it and it looks like treeherder will be taking over pretty soon. Which reminds me I need to raise that bug. :)
Attachment #8498151 -
Flags: review?(ryanvm)
Comment 3•10 years ago
|
||
Comment on attachment 8498151 [details] [diff] [review] Add new content sandbox mochitest group v1 Review of attachment 8498151 [details] [diff] [review]: ----------------------------------------------------------------- r=me assuming you've tested it locally and confirmed that nothing breaks as-is :). Thanks for the patch! ::: js/Config.js @@ +301,5 @@ > "Mochitest e10s": ["Mochitest e10s", > "Mochitest e10s Browser Chrome", > "Mochitest e10s DevTools Browser Chrome", > "Mochitest e10s Other"], > + "Mochitest csb": ["Mochitest csb"], Above the e10s group. @@ +502,5 @@ > "Mochitest e10s" : "M-e10s", > "Mochitest e10s Browser Chrome" : "bc", > "Mochitest e10s DevTools Browser Chrome" : "dt", > "Mochitest e10s Other" : "oth", > + "Mochitest csb" : "M-csb", This means they will display in the UI after the M-e10s group. That's probably for the best so the UI is more consistent (all the M-e10s groups roughly in line with each other). So let's leave this as-is for now. ::: js/Data.js @@ +641,5 @@ > /mochitest-e10s-browser-chrome/i.test(name) ? "Mochitest e10s Browser Chrome" : > /mochitest-e10s-devtools-chrome/i.test(name) ? "Mochitest e10s DevTools Browser Chrome" : > /mochitest-e10s-other/i.test(name) ? "Mochitest e10s Other" : > /mochitest-e10s/i.test(name) ? "Mochitest e10s" : > + /mochitest-csb/i.test(name) ? "Mochitest csb" : Put this above the e10s lines.
Attachment #8498151 -
Flags: review?(ryanvm) → review+
Assignee | ||
Comment 4•10 years ago
|
||
(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #3) > r=me assuming you've tested it locally and confirmed that nothing breaks > as-is :). Thanks for the patch! Thanks. Yes, I've tried this locally against various trees. :) I've made the changes from your review locally. I was going to wait until the jobs were running on holly, so I could make sure the M-csb jobs appear before landing this.
Status: NEW → ASSIGNED
Assignee | ||
Comment 5•10 years ago
|
||
r=RyanVM - from comment 3 (In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #3) > Comment on attachment 8498151 [details] [diff] [review] > Add new content sandbox mochitest group v1 > > Review of attachment 8498151 [details] [diff] [review]: > ----------------------------------------------------------------- > > r=me assuming you've tested it locally and confirmed that nothing breaks > as-is :). Thanks for the patch! > > "Mochitest e10s Other"], > > + "Mochitest csb": ["Mochitest csb"], > > Above the e10s group. Moved. > > /mochitest-e10s/i.test(name) ? "Mochitest e10s" : > > + /mochitest-csb/i.test(name) ? "Mochitest csb" : > > Put this above the e10s lines. Moved.
Attachment #8498151 -
Attachment is obsolete: true
Attachment #8500891 -
Flags: review+
Assignee | ||
Comment 6•10 years ago
|
||
Landed on default: https://hg.mozilla.org/webtools/tbpl/rev/99f7932ac9c7
Updated•10 years ago
|
Product: Webtools → Tree Management
Comment 7•10 years ago
|
||
In production :)
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•9 years ago
|
Product: Tree Management → Tree Management Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•