Closed
Bug 1343366
Opened 8 years ago
Closed 8 years ago
browser_parsable_css.js fails on ESR52 ASAN builds due to the ESR branding changes
Categories
(Firefox :: General, defect)
Firefox
General
Tracking
()
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | fixed |
People
(Reporter: RyanVM, Assigned: RyanVM)
References
Details
On every new ESR branch, branding changes are landed to s/Firefox/Firefox ESR the main About dialog.
https://hg.mozilla.org/releases/mozilla-esr52/rev/df0931ac8b02943468f236483f2cb5a53bb0b439
With ESR52, this change is currently causing browser_parseable_css.js to fail on ASAN builds because the About dialog changes only work with official branding, which they don't have enabled.
https://treeherder.mozilla.org/logviewer.html#?job_id=80540438&repo=mozilla-esr52
This means that ESR ASAN builds almost certainly have a legitimately broken About dialog. But given that they aren't builds we officially distribute anyway, I'm not sure we really care. So our options here are probably either adding an exception to the test for this issue or just skipping it outright on ASAN.
Having talked it over with Jared on IRC, it seems highly unlikely that ASAN builds are going to catch an issue on uplifts that other platforms wouldn't also catch (other than this one, apparently!), so I'm going to just skip the test on ASAN.
Assignee | ||
Updated•8 years ago
|
status-firefox54:
affected → ---
Summary: browser_parseable_css.js fails on ESR52 ASAN builds due to the ESR branding changes → browser_parsable_css.js fails on ESR52 ASAN builds due to the ESR branding changes
Assignee | ||
Comment 1•8 years ago
|
||
bugherder uplift |
status-firefox-esr52:
--- → fixed
Flags: in-testsuite-
Assignee | ||
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•