Closed Bug 951655 Opened 11 years ago Closed 11 years ago

[Holly] Perma orange browser_social_workercrash.js | Test timed out

Categories

(Firefox Graveyard :: SocialAPI, defect)

x86
All
defect
Not set
normal

Tracking

(firefox29 fixed)

RESOLVED FIXED
Firefox 29
Tracking Status
firefox29 --- fixed

People

(Reporter: mconley, Assigned: mconley)

References

Details

(Whiteboard: [fixed-in-holly])

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #936193 +++

https://tbpl.mozilla.org/?tree=Holly&rev=a2a2490d8593

Seems to be across the board on Desktop.

Example from an opt run on Windows 7:

22:08:32     INFO -  ************************************************************
22:08:32     INFO -  * Call to xpconnect wrapped JSObject produced this error:  *
22:08:32     INFO -  [Exception... "'[JavaScript Error: "this._toolbarHelper.idFromOrigin is not a function" {file: "chrome://browser/content/browser.js" line: 5573}]' when calling method: [nsIObserver::observe]"  nsresult: "0x80570021 (NS_ERROR_XPC_JAVASCRIPT_ERROR_WITH_DETAILS)"  location: "JS frame :: resource://gre/modules/SocialService.jsm :: <TOP_LEVEL> :: line 757"  data: yes]
22:08:32     INFO -  ************************************************************
22:08:32     INFO -  ************************************************************
22:08:32     INFO -  * Call to xpconnect wrapped JSObject produced this error:  *
22:08:32     INFO -  [Exception... "'[JavaScript Error: "this._toolbarHelper.idFromOrigin is not a function" {file: "chrome://browser/content/browser.js" line: 5573}]' when calling method: [nsIObserver::observe]"  nsresult: "0x80570021 (NS_ERROR_XPC_JAVASCRIPT_ERROR_WITH_DETAILS)"  location: "JS frame :: resource://gre/modules/SocialService.jsm :: <TOP_LEVEL> :: line 757"  data: yes]
22:08:32     INFO -  ************************************************************
22:08:32     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/base/content/test/social/browser_social_workercrash.js | Console message: [JavaScript Error: "TypeError: this._toolbarHelper.idFromOrigin is not a function
22:08:32     INFO -  SocialStatus.reloadProvider@chrome://browser/content/browser.js:5573
22:08:32     INFO -  SocialUI_observe@chrome://browser/content/browser.js:4580
22:08:32     INFO -  @resource://gre/modules/SocialService.jsm:757
22:08:32     INFO -  reloadProvider@about:socialerror?mode=workerFailure:117
22:08:32     INFO -  tryAgainButton@about:socialerror?mode=workerFailure:107
22:08:32     INFO -  onclick@about:socialerror?mode=workerFailure:1
22:08:32     INFO -  " {file: "chrome://browser/content/browser.js" line: 4637}]
22:08:32     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/base/content/test/social/browser_social_workercrash.js | Console message: [JavaScript Error: "this._toolbarHelper.idFromOrigin is not a function" {file: "chrome://browser/content/browser.js" line: 5573}]
22:08:32     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/base/content/test/social/browser_social_workercrash.js | Console message: [JavaScript Error: "TypeError: this._toolbarHelper.idFromOrigin is not a function
22:08:32     INFO -  SocialStatus.reloadProvider@chrome://browser/content/browser.js:5573
22:08:32     INFO -  SocialUI_observe@chrome://browser/content/browser.js:4580
22:08:32     INFO -  @resource://gre/modules/SocialService.jsm:757
22:08:32     INFO -  reloadProvider@about:socialerror?mode=workerFailure:125
22:08:32     INFO -  tryAgainButton@about:socialerror?mode=workerFailure:107
22:08:32     INFO -  onclick@about:socialerror?mode=workerFailure:1
22:08:32     INFO -  " {file: "chrome://browser/content/browser.js" line: 4637}]
22:08:32     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/base/content/test/social/browser_social_workercrash.js | Console message: [JavaScript Error: "this._toolbarHelper.idFromOrigin is not a function" {file: "chrome://browser/content/browser.js" line: 5573}]
22:08:32     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/base/content/test/social/browser_social_workercrash.js | Console message: [JavaScript Error: "The character encoding of the plain text 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 file needs to be declared in the transfer protocol or file needs to use a byte order mark as an encoding signature." {file: "https://example.com/browser/browser/base/content/test/social/social_worker.js" line: 0}]
22:08:32     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/base/content/test/social/browser_social_workercrash.js | Console message: [JavaScript Error: "The character encoding of the plain text 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 file needs to be declared in the transfer protocol or file needs to use a byte order mark as an encoding signature." {file: "https://test1.example.com/browser/browser/base/content/test/social/social_worker.js" line: 0}]
22:08:36     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/base/content/test/social/browser_social_workercrash.js | Console message: [JavaScript Error: "A promise chain failed to handle a rejection.
22:08:36     INFO -  Date: Tue Dec 17 2013 22:08:32 GMT-0800 (Pacific Standard Time)
22:08:36     INFO -  Full Message: Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIMessageSender.sendAsyncMessage]
22:08:36     INFO -  Full Stack: JS frame :: resource://gre/modules/FrameWorker.jsm :: <TOP_LEVEL> :: line 161
22:08:36     INFO -  JS frame :: resource://gre/modules/Promise.jsm :: <TOP_LEVEL> :: line 767
22:08:36     INFO -  JS frame :: resource://gre/modules/Promise.jsm :: <TOP_LEVEL> :: line 531
22:08:36     INFO -  native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0" {file: "resource://gre/modules/FrameWorker.jsm" line: 161 column: 0 source: "161"}]
22:08:36     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/base/content/test/social/browser_social_workercrash.js | Console message: [JavaScript Error: "A promise chain failed to handle a rejection.
22:08:36     INFO -  Date: Tue Dec 17 2013 22:08:32 GMT-0800 (Pacific Standard Time)
22:08:36     INFO -  Full Message: Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIMessageSender.sendAsyncMessage]
22:08:36     INFO -  Full Stack: JS frame :: resource://gre/modules/FrameWorker.jsm :: <TOP_LEVEL> :: line 161
22:08:36     INFO -  JS frame :: resource://gre/modules/Promise.jsm :: <TOP_LEVEL> :: line 767
22:08:36     INFO -  JS frame :: resource://gre/modules/Promise.jsm :: <TOP_LEVEL> :: line 531
22:08:36     INFO -  native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0" {file: "resource://gre/modules/FrameWorker.jsm" line: 161 column: 0 source: "161"}]
22:08:36     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/base/content/test/social/browser_social_workercrash.js | Console message: [JavaScript Error: "A promise chain failed to handle a rejection.
22:08:36     INFO -  Date: Tue Dec 17 2013 22:08:32 GMT-0800 (Pacific Standard Time)
22:08:36     INFO -  Full Message: Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIMessageSender.sendAsyncMessage]
22:08:36     INFO -  Full Stack: JS frame :: resource://gre/modules/FrameWorker.jsm :: <TOP_LEVEL> :: line 59
22:08:36     INFO -  JS frame :: resource://gre/modules/Promise.jsm :: <TOP_LEVEL> :: line 767
22:08:36     INFO -  JS frame :: resource://gre/modules/Promise.jsm :: <TOP_LEVEL> :: line 531
22:08:36     INFO -  native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0" {file: "resource://gre/modules/FrameWorker.jsm" line: 59 column: 0 source: "59"}]
22:08:36     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/base/content/test/social/browser_social_workercrash.js | Console message: [JavaScript Error: "A promise chain failed to handle a rejection.
22:08:36     INFO -  Date: Tue Dec 17 2013 22:08:32 GMT-0800 (Pacific Standard Time)
22:08:36     INFO -  Full Message: Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIMessageSender.sendAsyncMessage]
22:08:36     INFO -  Full Stack: JS frame :: resource://gre/modules/FrameWorker.jsm :: <TOP_LEVEL> :: line 161
22:08:36     INFO -  JS frame :: resource://gre/modules/Promise.jsm :: <TOP_LEVEL> :: line 767
22:08:36     INFO -  JS frame :: resource://gre/modules/Promise.jsm :: <TOP_LEVEL> :: line 531
22:08:36     INFO -  native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0" {file: "resource://gre/modules/FrameWorker.jsm" line: 161 column: 0 source: "161"}]
22:08:36     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/base/content/test/social/browser_social_workercrash.js | Console message: [JavaScript Error: "A promise chain failed to handle a rejection.
22:08:36     INFO -  Date: Tue Dec 17 2013 22:08:32 GMT-0800 (Pacific Standard Time)
22:08:36     INFO -  Full Message: Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIMessageSender.sendAsyncMessage]
22:08:36     INFO -  Full Stack: JS frame :: resource://gre/modules/FrameWorker.jsm :: <TOP_LEVEL> :: line 161
22:08:36     INFO -  JS frame :: resource://gre/modules/Promise.jsm :: <TOP_LEVEL> :: line 767
22:08:36     INFO -  JS frame :: resource://gre/modules/Promise.jsm :: <TOP_LEVEL> :: line 531
22:08:36     INFO -  native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0" {file: "resource://gre/modules/FrameWorker.jsm" line: 161 column: 0 source: "161"}]
22:08:36     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/base/content/test/social/browser_social_workercrash.js | Console message: [JavaScript Error: "A promise chain failed to handle a rejection.
22:08:36     INFO -  Date: Tue Dec 17 2013 22:08:32 GMT-0800 (Pacific Standard Time)
22:08:36     INFO -  Full Message: Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIMessageSender.sendAsyncMessage]
22:08:36     INFO -  Full Stack: JS frame :: resource://gre/modules/FrameWorker.jsm :: <TOP_LEVEL> :: line 161
22:08:36     INFO -  JS frame :: resource://gre/modules/Promise.jsm :: <TOP_LEVEL> :: line 767
22:08:36     INFO -  JS frame :: resource://gre/modules/Promise.jsm :: <TOP_LEVEL> :: line 531
22:08:36     INFO -  native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0" {file: "resource://gre/modules/FrameWorker.jsm" line: 161 column: 0 source: "161"}]
22:08:36     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/base/content/test/social/browser_social_workercrash.js | Console message: [JavaScript Error: "A promise chain failed to handle a rejection.
22:08:36     INFO -  Date: Tue Dec 17 2013 22:08:32 GMT-0800 (Pacific Standard Time)
22:08:36     INFO -  Full Message: Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIMessageSender.sendAsyncMessage]
22:08:36     INFO -  Full Stack: JS frame :: resource://gre/modules/FrameWorker.jsm :: <TOP_LEVEL> :: line 161
22:08:36     INFO -  JS frame :: resource://gre/modules/Promise.jsm :: <TOP_LEVEL> :: line 767
22:08:36     INFO -  JS frame :: resource://gre/modules/Promise.jsm :: <TOP_LEVEL> :: line 531
22:08:36     INFO -  native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0" {file: "resource://gre/modules/FrameWorker.jsm" line: 161 column: 0 source: "161"}]
22:08:36     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/base/content/test/social/browser_social_workercrash.js | Console message: [JavaScript Error: "A promise chain failed to handle a rejection.
22:08:36     INFO -  Date: Tue Dec 17 2013 22:08:32 GMT-0800 (Pacific Standard Time)
22:08:36     INFO -  Full Message: Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIMessageSender.sendAsyncMessage]
22:08:36     INFO -  Full Stack: JS frame :: resource://gre/modules/FrameWorker.jsm :: <TOP_LEVEL> :: line 59
22:08:36     INFO -  JS frame :: resource://gre/modules/Promise.jsm :: <TOP_LEVEL> :: line 767
22:08:36     INFO -  JS frame :: resource://gre/modules/Promise.jsm :: <TOP_LEVEL> :: line 531
22:08:36     INFO -  native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0" {file: "resource://gre/modules/FrameWorker.jsm" line: 59 column: 0 source: "59"}]
22:08:36     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/base/content/test/social/browser_social_workercrash.js | Console message: [JavaScript Error: "A promise chain failed to handle a rejection.
22:08:36     INFO -  Date: Tue Dec 17 2013 22:08:32 GMT-0800 (Pacific Standard Time)
22:08:36     INFO -  Full Message: Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIMessageSender.sendAsyncMessage]
22:08:36     INFO -  Full Stack: JS frame :: resource://gre/modules/FrameWorker.jsm :: <TOP_LEVEL> :: line 161
22:08:36     INFO -  JS frame :: resource://gre/modules/Promise.jsm :: <TOP_LEVEL> :: line 767
22:08:36     INFO -  JS frame :: resource://gre/modules/Promise.jsm :: <TOP_LEVEL> :: line 531
22:08:36     INFO -  native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0" {file: "resource://gre/modules/FrameWorker.jsm" line: 161 column: 0 source: "161"}]
22:08:36     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/base/content/test/social/browser_social_workercrash.js | Console message: [JavaScript Error: "A promise chain failed to handle a rejection.
22:08:36     INFO -  Date: Tue Dec 17 2013 22:08:32 GMT-0800 (Pacific Standard Time)
22:08:36     INFO -  Full Message: Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIMessageSender.sendAsyncMessage]
22:08:36     INFO -  Full Stack: JS frame :: resource://gre/modules/FrameWorker.jsm :: <TOP_LEVEL> :: line 161
22:08:36     INFO -  JS frame :: resource://gre/modules/Promise.jsm :: <TOP_LEVEL> :: line 767
22:08:36     INFO -  JS frame :: resource://gre/modules/Promise.jsm :: <TOP_LEVEL> :: line 531
22:08:36     INFO -  native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0" {file: "resource://gre/modules/FrameWorker.jsm" line: 161 column: 0 source: "161"}]
22:08:36     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/base/content/test/social/browser_social_workercrash.js | Console message: [JavaScript Error: "A promise chain failed to handle a rejection.
22:08:36     INFO -  Date: Tue Dec 17 2013 22:08:32 GMT-0800 (Pacific Standard Time)
22:08:36     INFO -  Full Message: Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIMessageSender.sendAsyncMessage]
22:08:36     INFO -  Full Stack: JS frame :: resource://gre/modules/FrameWorker.jsm :: <TOP_LEVEL> :: line 161
22:08:36     INFO -  JS frame :: resource://gre/modules/Promise.jsm :: <TOP_LEVEL> :: line 767
22:08:36     INFO -  JS frame :: resource://gre/modules/Promise.jsm :: <TOP_LEVEL> :: line 531
22:08:36     INFO -  native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0" {file: "resource://gre/modules/FrameWorker.jsm" line: 161 column: 0 source: "161"}]
22:08:36     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/base/content/test/social/browser_social_workercrash.js | Console message: [JavaScript Error: "A promise chain failed to handle a rejection.
22:08:36     INFO -  Date: Tue Dec 17 2013 22:08:32 GMT-0800 (Pacific Standard Time)
22:08:36     INFO -  Full Message: Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIMessageSender.sendAsyncMessage]
22:08:36     INFO -  Full Stack: JS frame :: resource://gre/modules/FrameWorker.jsm :: <TOP_LEVEL> :: line 161
22:08:36     INFO -  JS frame :: resource://gre/modules/Promise.jsm :: <TOP_LEVEL> :: line 767
22:08:36     INFO -  JS frame :: resource://gre/modules/Promise.jsm :: <TOP_LEVEL> :: line 531
22:08:36     INFO -  native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0" {file: "resource://gre/modules/FrameWorker.jsm" line: 161 column: 0 source: "161"}]
[screenshot omitted]
22:09:02  WARNING -  TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/social/browser_social_workercrash.js | Test timed out
Orange seems to have been introduced in this merge changeset from last night:

https://hg.mozilla.org/projects/holly/pushloghtml?startID=107&endID=108

There's some social stuff in there marked Australis-only, but I thought all of it was backed out. Will double-check.
The error messages regarding this._toolbarHelper.idFromOrigin, coupled with the reference to this._toolbarHelper.idFromOrigin in Bug 950051 make me pretty confident that this is our culprit.

Will confirm with a local backout of bug 950051 once I also confirm that I can reproduce the failure locally.
Winner winner - backing ou tbug 950051 resolves the issue.
Blocks: 950051
So here's the story:

Bug 940820 landed this changeset: http://hg.mozilla.org/mozilla-central/rev/dd40ffa484b9

That changeset, among other things, changed the erroneously named idFromOrgin to idFromOrigin.

Unfortunately, that changeset was backed out, since it had Australis in the commit message.

A subsequent patch that is not dependent on Australis landed which uses idFromOrigin. Since Holly still has idFromOrgin, this was causing a test failure.

This patch simply changes idFromOrgin to idFromOrigin on Holly. Locally, all social tests pass with this patch.
Assignee: nobody → mconley
Status: NEW → ASSIGNED
Attachment #8349422 - Flags: feedback?(florian)
Comment on attachment 8349422 [details] [diff] [review]
Patch v1 - replace idFromOrgin with idFromOrigin

Seems reasonable, thanks!
Attachment #8349422 - Flags: feedback?(florian) → feedback+
Thanks - landed in Holly as:

https://hg.mozilla.org/projects/holly/rev/5815656487e3
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-holly]
Target Milestone: --- → Firefox 29
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: