Closed Bug 752409 Opened 7 years ago Closed 7 years ago

Intermittent browser/browser/components/thumbnails/test/browser_thumbnails_storage.js | an unexpected uncaught JS exception reported through window.onerror - ReferenceError: aFile is not defined at chrome://mochitests

Categories

(Firefox :: Tabbed Browser, defect)

x86
Windows 7
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 15

People

(Reporter: rnewman, Assigned: ttaubert)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js | an unexpected uncaught JS exception reported through window.onerror - ReferenceError: aFile is not defined at chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js:75 

https://tbpl.mozilla.org/php/getParsedLog.php?id=11522732&tree=Services-Central&full=1

TEST-START | chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js
TEST-INFO | chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js | Console message: [JavaScript Error: "[Exception... "'JavaScript component does not have a method named: "observe"' when calling method: [nsIObserver::observe]"  nsresult: "0x80570030 (NS_ERROR_XPC_JSOBJECT_HAS_NO_FUNCTION_NAMED)"  location: "JS frame :: chrome://browser/content/sanitize.js :: <TOP_LEVEL> :: line 191"  data: no]" {file: "chrome://browser/content/sanitize.js" line: 191}]
TEST-INFO | chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js | Console message: [JavaScript Error: "[Exception... "'JavaScript component does not have a method named: "observe"' when calling method: [nsIObserver::observe]"  nsresult: "0x80570030 (NS_ERROR_XPC_JSOBJECT_HAS_NO_FUNCTION_NAMED)"  location: "JS frame :: chrome://browser/content/sanitize.js :: <TOP_LEVEL> :: line 191"  data: no]" {file: "chrome://browser/content/sanitize.js" line: 191}]
TEST-INFO | chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js | Console message: [JavaScript Error: "[Exception... "'JavaScript component does not have a method named: "observe"' when calling method: [nsIObserver::observe]"  nsresult: "0x80570030 (NS_ERROR_XPC_JSOBJECT_HAS_NO_FUNCTION_NAMED)"  location: "JS frame :: chrome://browser/content/sanitize.js :: <TOP_LEVEL> :: line 191"  data: no]" {file: "chrome://browser/content/sanitize.js" line: 191}]
TEST-INFO | chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js | Console message: [JavaScript Error: "[Exception... "'JavaScript component does not have a method named: "observe"' when calling method: [nsIObserver::observe]"  nsresult: "0x80570030 (NS_ERROR_XPC_JSOBJECT_HAS_NO_FUNCTION_NAMED)"  location: "JS frame :: chrome://browser/content/sanitize.js :: <TOP_LEVEL> :: line 191"  data: no]" {file: "chrome://browser/content/sanitize.js" line: 191}]
TEST-INFO | chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js | Console message: [JavaScript Error: "[Exception... "'JavaScript component does not have a method named: "observe"' when calling method: [nsIObserver::observe]"  nsresult: "0x80570030 (NS_ERROR_XPC_JSOBJECT_HAS_NO_FUNCTION_NAMED)"  location: "JS frame :: chrome://browser/content/sanitize.js :: <TOP_LEVEL> :: line 191"  data: no]" {file: "chrome://browser/content/sanitize.js" line: 191}]
TEST-INFO | chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js | Console message: [JavaScript Error: "The character encoding of the HTML document was not declared. The document will render with garbled text in some browser configurations if the document contains characters from outside the US-ASCII range. The character encoding of the page must to be declared in the document or in the transfer protocol." {file: "http://mochi.test:8888/" line: 0}]
TEST-INFO | chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js | Console message: [JavaScript Error: "[Exception... "'JavaScript component does not have a method named: "observe"' when calling method: [nsIObserver::observe]"  nsresult: "0x80570030 (NS_ERROR_XPC_JSOBJECT_HAS_NO_FUNCTION_NAMED)"  location: "JS frame :: chrome://browser/content/sanitize.js :: <TOP_LEVEL> :: line 191"  data: no]" {file: "chrome://browser/content/sanitize.js" line: 191}]
TEST-INFO | chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js | Console message: [JavaScript Error: "[Exception... "'JavaScript component does not have a method named: "observe"' when calling method: [nsIObserver::observe]"  nsresult: "0x80570030 (NS_ERROR_XPC_JSOBJECT_HAS_NO_FUNCTION_NAMED)"  location: "JS frame :: chrome://browser/content/sanitize.js :: <TOP_LEVEL> :: line 191"  data: no]" {file: "chrome://browser/content/sanitize.js" line: 191}]
TEST-INFO | chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js | Console message: [JavaScript Error: "[Exception... "'JavaScript component does not have a method named: "observe"' when calling method: [nsIObserver::observe]"  nsresult: "0x80570030 (NS_ERROR_XPC_JSOBJECT_HAS_NO_FUNCTION_NAMED)"  location: "JS frame :: chrome://browser/content/sanitize.js :: <TOP_LEVEL> :: line 191"  data: no]" {file: "chrome://browser/content/sanitize.js" line: 191}]
TEST-INFO | chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js | Console message: [JavaScript Error: "[Exception... "'JavaScript component does not have a method named: "observe"' when calling method: [nsIObserver::observe]"  nsresult: "0x80570030 (NS_ERROR_XPC_JSOBJECT_HAS_NO_FUNCTION_NAMED)"  location: "JS frame :: chrome://browser/content/sanitize.js :: <TOP_LEVEL> :: line 191"  data: no]" {file: "chrome://browser/content/sanitize.js" line: 191}]
TEST-INFO | chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js | Console message: [JavaScript Error: "[Exception... "'JavaScript component does not have a method named: "observe"' when calling method: [nsIObserver::observe]"  nsresult: "0x80570030 (NS_ERROR_XPC_JSOBJECT_HAS_NO_FUNCTION_NAMED)"  location: "JS frame :: chrome://browser/content/sanitize.js :: <TOP_LEVEL> :: line 191"  data: no]" {file: "chrome://browser/content/sanitize.js" line: 191}]
TEST-INFO | chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js | Console message: [JavaScript Error: "TelemetryStopwatch: key "FX_SESSION_RESTORE_WRITE_FILE_MS" was already initialized" {file: "resource://gre/modules/TelemetryStopwatch.jsm" line: 53}]
TEST-INFO | chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js | Console message: [JavaScript Error: "The character encoding of the HTML document was not declared. The document will render with garbled text in some browser configurations if the document contains characters from outside the US-ASCII range. The character encoding of the page must to be declared in the document or in the transfer protocol." {file: "http://mochi.test:8888/" line: 0}]
TEST-PASS | chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js | thumbnail file was updated
TEST-PASS | chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js | copy of thumbnail has been removed
TEST-INFO | chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js | Console message: [JavaScript Error: "[Exception... "'JavaScript component does not have a method named: "observe"' when calling method: [nsIObserver::observe]"  nsresult: "0x80570030 (NS_ERROR_XPC_JSOBJECT_HAS_NO_FUNCTION_NAMED)"  location: "JS frame :: chrome://browser/content/sanitize.js :: <TOP_LEVEL> :: line 191"  data: no]" {file: "chrome://browser/content/sanitize.js" line: 191}]
TEST-INFO | chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js | Console message: [JavaScript Error: "[Exception... "'JavaScript component does not have a method named: "observe"' when calling method: [nsIObserver::observe]"  nsresult: "0x80570030 (NS_ERROR_XPC_JSOBJECT_HAS_NO_FUNCTION_NAMED)"  location: "JS frame :: chrome://browser/content/sanitize.js :: <TOP_LEVEL> :: line 191"  data: no]" {file: "chrome://browser/content/sanitize.js" line: 191}]
TEST-INFO | chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js | Console message: [JavaScript Error: "[Exception... "'JavaScript component does not have a method named: "observe"' when calling method: [nsIObserver::observe]"  nsresult: "0x80570030 (NS_ERROR_XPC_JSOBJECT_HAS_NO_FUNCTION_NAMED)"  location: "JS frame :: chrome://browser/content/sanitize.js :: <TOP_LEVEL> :: line 191"  data: no]" {file: "chrome://browser/content/sanitize.js" line: 191}]
TEST-INFO | chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js | Console message: [JavaScript Error: "[Exception... "'JavaScript component does not have a method named: "observe"' when calling method: [nsIObserver::observe]"  nsresult: "0x80570030 (NS_ERROR_XPC_JSOBJECT_HAS_NO_FUNCTION_NAMED)"  location: "JS frame :: chrome://browser/content/sanitize.js :: <TOP_LEVEL> :: line 191"  data: no]" {file: "chrome://browser/content/sanitize.js" line: 191}]
TEST-INFO | chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js | Console message: [JavaScript Error: "[Exception... "'JavaScript component does not have a method named: "observe"' when calling method: [nsIObserver::observe]"  nsresult: "0x80570030 (NS_ERROR_XPC_JSOBJECT_HAS_NO_FUNCTION_NAMED)"  location: "JS frame :: chrome://browser/content/sanitize.js :: <TOP_LEVEL> :: line 191"  data: no]" {file: "chrome://browser/content/sanitize.js" line: 191}]
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js | an unexpected uncaught JS exception reported through window.onerror - ReferenceError: aFile is not defined at chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js:75
Stack trace:
    JS frame :: chrome://mochikit/content/tests/SimpleTest/SimpleTest.js :: simpletestOnerror :: line 983
    native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0

INFO TEST-END | chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js | finished in 1797ms
TEST-INFO | checking window state
TEST-INFO | chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js | Console message: [JavaScript Error: "ReferenceError: aFile is not defined" {file: "chrome://mochitests/content/browser/browser/components/thumbnails/test/browser_thumbnails_storage.js" line: 75}]
TEST-START | chrome://mochitests/content/browser/browser/devtools/debugger/test/browser_dbg_bug723069_editor-breakpoints.js
Assignee: nobody → ttaubert
Status: NEW → ASSIGNED
Attached patch trivial patchSplinter Review
The aFile argument doesn't exist anymore and I forgot to remove it. Additionally, we need to yield when calling clearHistory() to wait until the history has actually been cleared.
Attachment #623645 - Flags: review?(dietrich)
Comment on attachment 623645 [details] [diff] [review]
trivial patch

Review of attachment 623645 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/components/thumbnails/test/browser_thumbnails_storage.js
@@ +71,5 @@
>    s.range = null;
>  
>    executeSoon(function () {
>      if (PageThumbsStorage.getFileForURL(URL).exists())
> +      clearHistory(aUseRange);

can things happen synchronously after this? or does next() do more executeSoon magic?
Attachment #623645 - Flags: review?(dietrich) → review+
(In reply to Dietrich Ayala (:dietrich) from comment #10)
> ::: browser/components/thumbnails/test/browser_thumbnails_storage.js
> @@ +71,5 @@
> >    s.range = null;
> >  
> >    executeSoon(function () {
> >      if (PageThumbsStorage.getFileForURL(URL).exists())
> > +      clearHistory(aUseRange);
> 
> can things happen synchronously after this? or does next() do more
> executeSoon magic?

Yeah, that's intended. We tried clearing the history and it didn't work the first time. So we'll call executeSoon() and then try again, and again, ... next() can just continue synchronously.
Pushed to fx-team:

https://hg.mozilla.org/integration/fx-team/rev/470fddafc49b

and merged:

https://hg.mozilla.org/mozilla-central/rev/470fddafc49b
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 15
Whiteboard: [orange]
You need to log in before you can comment on or make changes to this bug.