Closed
Bug 1378163
Opened 7 years ago
Closed 7 years ago
Permanent failure on Cedar: browser/base/content/test/static/browser_all_files_referenced.js | there should be no unreferenced files
Categories
(Firefox :: General, defect)
Firefox
General
Tracking
()
RESOLVED
FIXED
Firefox 56
Tracking | Status | |
---|---|---|
firefox56 | --- | fixed |
People
(Reporter: mconley, Assigned: mconley)
References
Details
Attachments
(1 file)
Snipped from the log:
07:50:04 INFO - Entering test bound checkAllTheFiles
07:50:04 INFO - Buffered messages finished
07:50:04 INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/static/browser_all_files_referenced.js | there should be no unreferenced files - Got 6, expected 0
07:50:04 INFO - Stack trace:
07:50:04 INFO - chrome://mochikit/content/browser-test.js:test_is:967
07:50:04 INFO - chrome://mochitests/content/browser/browser/base/content/test/static/browser_all_files_referenced.js:checkAllTheFiles:615
07:50:04 INFO - Not taking screenshot here: see the one that was previously logged
07:50:04 INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/static/browser_all_files_referenced.js | unreferenced file: chrome://browser/skin/chevron-animation.svg -
07:50:04 INFO - Stack trace:
07:50:04 INFO - chrome://mochitests/content/browser/browser/base/content/test/static/browser_all_files_referenced.js:checkAllTheFiles:617
07:50:04 INFO - Not taking screenshot here: see the one that was previously logged
07:50:04 INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/static/browser_all_files_referenced.js | unreferenced file: chrome://browser/skin/device-desktop.svg -
07:50:04 INFO - Stack trace:
07:50:04 INFO - chrome://mochitests/content/browser/browser/base/content/test/static/browser_all_files_referenced.js:checkAllTheFiles:617
07:50:04 INFO - Not taking screenshot here: see the one that was previously logged
07:50:04 INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/static/browser_all_files_referenced.js | unreferenced file: chrome://browser/skin/email-link.svg -
07:50:04 INFO - Stack trace:
07:50:04 INFO - chrome://mochitests/content/browser/browser/base/content/test/static/browser_all_files_referenced.js:checkAllTheFiles:617
07:50:04 INFO - Not taking screenshot here: see the one that was previously logged
07:50:04 INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/static/browser_all_files_referenced.js | unreferenced file: chrome://browser/skin/link.svg -
07:50:04 INFO - Stack trace:
07:50:04 INFO - chrome://mochitests/content/browser/browser/base/content/test/static/browser_all_files_referenced.js:checkAllTheFiles:617
07:50:04 INFO - Not taking screenshot here: see the one that was previously logged
07:50:04 INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/static/browser_all_files_referenced.js | unreferenced file: chrome://browser/skin/page-action.svg -
07:50:04 INFO - Stack trace:
07:50:04 INFO - chrome://mochitests/content/browser/browser/base/content/test/static/browser_all_files_referenced.js:checkAllTheFiles:617
07:50:04 INFO - Not taking screenshot here: see the one that was previously logged
07:50:04 INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/static/browser_all_files_referenced.js | unreferenced file: chrome://browser/skin/tabbrowser/indicator-tab-attention.svg -
07:50:04 INFO - Stack trace:
07:50:04 INFO - chrome://mochitests/content/browser/browser/base/content/test/static/browser_all_files_referenced.js:checkAllTheFiles:617
07:50:04 INFO - ignored unused whitelist entry: chrome://xslt-qa/content/buster/result-view.xul
07:50:04 INFO - ignored unused whitelist entry: chrome://xslt-qa/content/xslt-qa-overlay.xul
07:50:04 INFO - ignored unused whitelist entry: chrome://global/locale/crashes.dtd
07:50:04 INFO - ignored unused whitelist entry: chrome://global/locale/crashes.properties
07:50:04 INFO - ignored unused whitelist entry: resource://app/blocklist.xml
07:50:04 INFO - ignored unused whitelist entry: resource://gre/gmp-clearkey/0.1/manifest.json
07:50:04 INFO - ignored unused whitelist entry: resource://gre/res/test.properties
07:50:04 INFO - missing file with code reference: chrome://flyweb/content/icon-64.png
07:50:04 INFO - missing file with code reference: chrome://communicator/content/directory/directory.xul
07:50:04 INFO - missing file with code reference: chrome://help/content/help.js
07:50:04 INFO - missing file with code reference: resource://gre/modules/E10SUtils.jsm
07:50:04 INFO - missing file with code reference: chrome://global/skin/expander.css
07:50:04 INFO - missing file with code reference: chrome://global/skin/printPreview.css
07:50:04 INFO - missing file with code reference: resource://services-sync/)%20==%200)%20%7B
07:50:04 INFO - missing file with code reference: resource://gre/modules/worker/myModule.js
07:50:04 INFO - missing file with code reference: resource://gre/modules/workers/SimpleTest.js
07:50:04 INFO - missing file with code reference: resource://gre/modules/workers/Logger.js
07:50:04 INFO - missing file with code reference: resource://pdf.js.components/
07:50:04 INFO - missing file with code reference: resource://gre/modules/glenn.htm
07:50:04 INFO - missing file with code reference: resource://gre/modules/jane.html
07:50:04 INFO - missing file with code reference: resource://gre/modules/jane.jpeg
07:50:04 INFO - missing file with code reference: resource://gre/modules/glenn.html
07:50:04 INFO - missing file with code reference: chrome://gfxsanity/content/sanityparent.html
07:50:04 INFO - missing file with code reference: resource://gre/modules/Messaging.jsm
07:50:04 INFO - missing file with code reference: resource://addon-1/data/index.html
07:50:04 INFO - missing file with code reference: resource://addon-1/data/main.html
07:50:04 INFO - missing file with code reference: resource://gre/modules/WindowsRegistry.jsm
07:50:04 INFO - missing file with code reference: resource://gre/modules/OSCrypto_win.js
07:50:04 INFO - missing file with code reference: resource://gre/modules/osfile/osfile_win_allthreads.jsm
07:50:04 INFO - missing file with code reference: resource://gre/modules/subprocess/subprocess_win.jsm
07:50:04 INFO - missing file with code reference: resource://gre/modules/DownloadImport.jsm
07:50:04 INFO - missing file with code reference: resource://android/
07:50:04 INFO - missing file with code reference: resource://gre/modules/PushServiceAndroidGCM.jsm
07:50:04 INFO - missing file with code reference: chrome://global/skin/plugins.css
07:50:04 INFO - missing file with code reference: chrome://browser/skin/customizableui/density-normal.svg
07:50:04 INFO - missing file with code reference: chrome://browser/skin/customizableui/density-compact.svg
07:50:04 INFO - missing file with code reference: chrome://browser/skin/customizableui/density-touch.svg
07:50:04 INFO - missing file with code reference: chrome://browser/locale/preferences.properties
07:50:04 INFO - missing file with code reference: resource://gre/modules/WebNavigationFrames.js
07:50:04 INFO - missing file with code reference: resource:///modules/WindowsPreviewPerTab.jsm
07:50:04 INFO - missing file with code reference: resource:///modules/Windows8WindowFrameColor.jsm
07:50:04 INFO - missing file with code reference: resource://testing-common/content-task.js%20line%2052%20%3E%20eval
07:50:04 INFO - missing file with code reference: resource://gre/modules/systemlibs.js
07:50:04 INFO - missing file with code reference: resource:///modules/WindowsJumpLists.jsm
07:50:04 INFO - missing file with code reference: chrome://branding/content/icon
07:50:04 INFO - missing file with code reference: chrome://b2g/content/shell.html
07:50:04 INFO - missing file with code reference: chrome://browser/content/pref/pref.xul
07:50:04 INFO - missing file with code reference: chrome://browser/content/preferences/preferences.xul
07:50:04 INFO - missing file with code reference: resource://test/
07:50:04 INFO - missing file with code reference: resource://app/some/link.html
07:50:04 INFO - missing file with code reference: chrome://browser/locale/uiDensity.properties
07:50:04 INFO - missing file with code reference: resource://gre/modules/SystemAppProxy.jsm
07:50:04 INFO - missing file with code reference: chrome://browser/content/extension-win-panel.css
07:50:04 INFO - missing file with code reference: chrome://browser/content/webrtcIndicator.xul
07:50:04 INFO - missing file with code reference: resource://screenshots-addon/
07:50:04 INFO - Leaving test bound checkAllTheFiles
07:50:04 INFO - Console message: No chrome package registered for chrome://flyweb/content/icon-64.png
07:50:04 INFO - Console message: No chrome package registered for chrome://navigator/content/navigator.xul
07:50:04 INFO - Console message: No chrome package registered for chrome://communicator/content/directory/directory.xul
07:50:04 INFO - Console message: No chrome package registered for chrome://help/content/help.js
07:50:04 INFO - Console message: No chrome package registered for chrome://gfxsanity/content/sanityparent.html
07:50:04 INFO - Console message: No chrome package registered for chrome://b2g/content/shell.html
07:50:04 INFO - GECKO(1710) | MEMORY STAT vsizeMaxContiguous not supported in this build configuration.
07:50:04 INFO - GECKO(1710) | MEMORY STAT | vsize 4736MB | residentFast 715MB | heapAllocated 477MB
07:50:04 INFO - TEST-OK | browser/base/content/test/static/browser_all_files_referenced.js | took 9421ms
07:50:05 INFO - checking window state
07:50:05 INFO - TEST-START | browser/base/content/test/static/browser_misused_characters_in_strings.js
07:50:05 INFO - GECKO(1710) | MEMORY STAT | vsize 4745MB | residentFast 725MB | heapAllocated 457MB
07:50:05 INFO - TEST-OK | browser/base/content/test/static/browser_misused_characters_in_strings.js | took 959ms
07:50:06 INFO - checking window state
07:50:06 INFO - TEST-START | browser/base/content/test/static/browser_parsable_css.js
07:50:06 INFO - Not taking screenshot here: see the one that was previously logged
07:50:06 INFO - <snipped 21 output lines - if you need more context, please use SimpleTest.requestCompleteLog() in your test>
07:50:06 INFO - Buffered messages logged at 07:50:06
07:50:06 INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/static/browser_parsable_css.js | missing chrome://browser/skin/customizableui/density-normal.svg referenced from chrome://browser/skin/browser.css -
07:50:06 INFO - Stack trace:
07:50:06 INFO - chrome://mochitests/content/browser/browser/base/content/test/static/browser_parsable_css.js:checkAllTheCSS:340
07:50:06 INFO - Not taking screenshot here: see the one that was previously logged
07:50:06 INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/static/browser_parsable_css.js | missing chrome://browser/skin/customizableui/density-compact.svg referenced from chrome://browser/skin/browser.css -
07:50:06 INFO - Stack trace:
07:50:06 INFO - chrome://mochitests/content/browser/browser/base/content/test/static/browser_parsable_css.js:checkAllTheCSS:340
07:50:06 INFO - Not taking screenshot here: see the one that was previously logged
07:50:06 INFO - TEST-UNEXPECTED-FAIL | browser/base/content/test/static/browser_parsable_css.js | missing chrome://browser/skin/customizableui/density-touch.svg referenced from chrome://browser/skin/browser.css -
07:50:06 INFO - Stack trace:
07:50:06 INFO - chrome://mochitests/content/browser/browser/base/content/test/static/browser_parsable_css.js:checkAllTheCSS:340
07:50:06 INFO - Ignored error for resource://gre-resources/html.css because of filter.
Assignee | ||
Updated•7 years ago
|
Summary: Permanent failure browser/base/content/test/static/browser_all_files_referenced.js | there should be no unreferenced files → Permanent failure on Cedar: browser/base/content/test/static/browser_all_files_referenced.js | there should be no unreferenced files
Assignee | ||
Comment 1•7 years ago
|
||
Assignee | ||
Comment 2•7 years ago
|
||
I suspect we're just going to need to update the whitelist here of files that can be unreferenced in the non-Photon build configuration.
Here's an easier to read list of unreferenced files being complained about here, along with the bugs that added them:
Bug 1375152 - chrome://browser/skin/chevron-animation.svg
Bug 1363182 - chrome://browser/skin/device-desktop.svg
Bug 1355323 - chrome://browser/skin/email-link.svg
Bug 1367587 - chrome://browser/skin/link.svg
Bug 1355322 - chrome://browser/skin/page-action.svg
Bug 1375893 - chrome://browser/skin/tabbrowser/indicator-tab-attention.svg
Assignee | ||
Comment 3•7 years ago
|
||
Okay, I think I've sorted this:
chrome://browser/skin/chevron-animation.svg:
Recently added in bug 1375152. Need to wrap the jar.mn entry in MOZ_PHOTON_ANIMATIONS, since this is only ever used in Photon.
chrome://browser/skin/device-desktop.svg
chrome://browser/skin/email-link.svg
chrome://browser/skin/link.svg
chrome://browser/skin/page-action.svg
Moved CSS rules into an ifdef recently in bug 1377165. The jar.mn entries should also go behind MOZ_PHOTON_THEME.
chrome://browser/skin/tabbrowser/indicator-tab-attention.svg:
Recently added in bug 1375893. The jar.mn entry needs to be wrapped by MOZ_PHOTON_THEME.
Assignee: nobody → mconley
Comment hidden (mozreview-request) |
Comment 5•7 years ago
|
||
mozreview-review |
Comment on attachment 8883381 [details]
Bug 1378163 - Only package images for Photon when Photon is enabled.
https://reviewboard.mozilla.org/r/154264/#review159356
Attachment #8883381 -
Flags: review?(gijskruitbosch+bugs) → review+
Pushed by mconley@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/79df9efc194b
Only package images for Photon when Photon is enabled. r=Gijs
Comment 7•7 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox56:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
Comment hidden (Intermittent Failures Robot) |
You need to log in
before you can comment on or make changes to this bug.
Description
•