1378163 - Permanent failure on Cedar: browser/base/content/test/static/browser_all_files_referenced.js | there should be no unreferenced files

Assignee

Description

•

7 years ago

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.

Mike Conley (:mconley) (:⚙️)

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

Mike Conley (:mconley) (:⚙️)

Assignee

Comment 1

•

7 years ago

Example job: https://treeherder.mozilla.org/#/jobs?repo=cedar&selectedJob=111728756

Mike Conley (:mconley) (:⚙️)

Assignee

Updated

•

7 years ago

Blocks: 1378197

Mike Conley (:mconley) (:⚙️)

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

Mike Conley (:mconley) (:⚙️)

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)

Review commit: https://reviewboard.mozilla.org/r/154264/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/154264/

:Gijs (he/him)

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+

Pulsebot

Comment 6

•

7 years ago

Pushed by mconley@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/79df9efc194b
Only package images for Photon when Photon is enabled. r=Gijs

Phil Ringnalda (:philor)

Comment 7

•

7 years ago

bugherder

https://hg.mozilla.org/mozilla-central/rev/79df9efc194b

Status: NEW → RESOLVED

Closed: 7 years ago

status-firefox56: --- → fixed

Resolution: --- → FIXED

Target Milestone: --- → Firefox 56

Comment hidden (Intermittent Failures Robot)

7 failures in 656 pushes (0.011 failures/push) were associated with this bug in the last 7 days.   

Repository breakdown:
* cedar: 7

Platform breakdown:
* linux64: 3
* windows8-64: 2
* windows7-32-vm: 1
* osx-10-10: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1378163&startday=2017-07-03&endday=2017-07-09&tree=all

Bugzilla

Quick Search

Permanent failure on Cedar: browser/base/content/test/static/browser_all_files_referenced.js | there should be no unreferenced files

Categories

(Firefox :: General, defect)

Tracking

()

People

(Reporter: mconley, Assigned: mconley)

References

Details

Crash Data

Security

(public)

User Story

Attachments

(1 file)

Description

Updated

Comment 1

Updated

Comment 2

Comment 3

Comment 4

Comment 5

Comment 6

Comment 7

Comment 9

Attachment

General

Description

File Name

Content Type