Closed
Bug 687257
Opened 8 years ago
Closed 8 years ago
OSX 10.5 M-oth permaorange brought across by latest JM merge
Categories
(Core :: JavaScript Engine, defect)
Not set
Tracking
()
RESOLVED
FIXED
People
(Reporter: emorley, Assigned: bhackett)
References
Details
The latest JM merge brought with it an OSX 10.5 M-oth permaorange that had been present on Jaegermonkey for a week. Inspecting tbpl gives a rough regression range of: (do double check, as I may have made a mistake) https://hg.mozilla.org/projects/jaegermonkey/pushloghtml?fromchange=b31b25125429&tochange=ab2af24a1dfa However, due to the numerous follow-up bit by bit fixing changesets with no detailed description or bug number, it was just easier to back out the whole JM merge, rather than try and untangle everything and backout just that regression range. I'm not going to attempt to reopen whatever was backed out - given that this presumably will be resolved on JM soon - and most of the changesets don't have a bug number. Once the orange is fixed, I presume the easiest thing will just be to backout the backout at the same time as the next JM merge. { TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_popupNotification.js | Test timed out TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_popupNotification.js | Found a tab after previous test timed out: http://example.org/ TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_tabMatchesInAwesomebar.js | Test timed out TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_tab_dragdrop.js | tab0 uri - Got about:privatebrowsing, expected about:blank TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_visibleTabs_bookmarkAllTabs.js | Bookmark All Tabs should be disabled since there are two tabs with the same address - Got false, expected true TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/tabview/browser_tabview_bug597248.js | Test timed out } ...followed by numerous others. Example log: https://tbpl.mozilla.org/php/getParsedLog.php?id=6434465&tree=Firefox
Reporter | ||
Comment 1•8 years ago
|
||
The backout of the merge: https://hg.mozilla.org/mozilla-central/rev/85fb038d1dd1
Assignee | ||
Comment 2•8 years ago
|
||
Geez, sorry about this. I thought I'd looked at the orange on JM before pulling to m-c, but not closely enough it looks like. This orange was caused by bug 685358. That bug has been a never ending nightmare since it landed on JM and I tried to back it out, but there were still residual problems. Will fix this by keeping the backout on JM and just relanding the other improvements and fixes that came in since the last JM -> m-c merge. Pushed to Try (https://tbpl.mozilla.org/?tree=Try&usebuildbot=1&rev=77dd152fd50f) and it looks good but not yet to JM as I can't push anything to JM (bug 687208).
Updated•8 years ago
|
tracking-firefox9:
--- → +
Comment 3•8 years ago
|
||
Did this get fixed by a backout?
Assignee | ||
Comment 4•8 years ago
|
||
Yes.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WORKSFORME
Reporter | ||
Comment 5•8 years ago
|
||
This was fixed by the backout in comment 1, but was left open for the investigation in comment 2, which I'm presuming is now complete. Removing tracking firefox9 since the initial landing + backout both landed on 9, so no action needed for any other branches/channels.
tracking-firefox9:
+ → ---
Resolution: WORKSFORME → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•