Closed Bug 1368370 Opened 3 years ago Closed 3 years ago
Permaorange on Dev
Edition and beta in browser _photon _customization _context _menus .js when Gecko 55 merges to beta on 2016-06-12
https://treeherder.mozilla.org/logviewer.html#?job_id=102697503&repo=try Guessing that the test isn't expecting the developer toolbar button? Minimum to repro locally is probably changing /config/milestone.txt from 55.0a1 to 55.0 and building with ac_add_options --with-branding=browser/branding/aurora (and clobbering, and not trying to do an artifact build). Try with all three platforms is https://hg.mozilla.org/try/rev/fa9d0b5989246f52b218e875871e2bb129c2316b plus -p linux-devedition-nightly,linux64-devedition-nightly,macosx64,win32,win64 [Tracking Requested - why for this release]: merge bustage, closed tree, delayed b1.
Oh, not just DevEdition, didn't realize I was pushing a different revision of m-c than my last central-as-beta. Same timeout on opt, add in some leaked until shutdown noise for debug, https://treeherder.mozilla.org/logviewer.html#?job_id=102716347&repo=try
Summary: Permaorange on DevEdition in browser_photon_customization_context_menus.js when Gecko 55 merges to beta on 2016-06-12 → Permaorange on DevEdition and beta in browser_photon_customization_context_menus.js when Gecko 55 merges to beta on 2016-06-12
The test is the first that actively flips the photon structure pref, and it's paying for that by tripping up code that should have been removed in bug 1364090 but wasn't (even though the hidden=true attribute was removed...).
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Comment on attachment 8872314 [details] Bug 1368370 - remove leftover page action hiding, https://reviewboard.mozilla.org/r/143818/#review147508 I feel like I'm missing some context. Is there some other code hiding the button besides the ifdef? Otherwise the removed code looks like it should be a no-op, so I don't see how this would fix a test. The patch by itself seems fine regardless.
Attachment #8872314 - Flags: review?(dao+bmo) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/79c629f3a50e remove leftover page action hiding, r=dao
You need to log in before you can comment on or make changes to this bug.