Closed Bug 1168544 Opened 9 years ago Closed 9 years ago

Intermittent browser_social_activation.js | Error: operation not possible on dead CPOW

Categories

(Firefox Graveyard :: SocialAPI, defect)

Product:
Firefox Graveyard
Component:
SocialAPI
Platform:
x86_64
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(e10s?, firefox41 affected, firefox43 fixed)

Status:
RESOLVED FIXED
Milestone:
Firefox 43
Tracking Flags:
Tracking Status
e10s ? ---
firefox41 --- affected
firefox43 --- fixed

People

(Reporter: RyanVM, Assigned: markh)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

0001-Bug-1168544-fix-social-orange-by-avoiding-reaching-i.patch
1.67 KB, patch
mixedpuppy
: review+
Details | Diff | Splinter Review 
08:51:01 INFO - 711 INFO TEST-PASS | browser/base/content/test/social/browser_social_activation.js | servicesInstall-notification panel opened
08:51:01 INFO - 712 INFO TEST-PASS | browser/base/content/test/social/browser_social_activation.js | servicesInstall-notification panel hidden
08:51:01 INFO - 713 INFO TEST-PASS | browser/base/content/test/social/browser_social_activation.js | activation panel is not showing
08:51:01 INFO - 714 INFO TEST-PASS | browser/base/content/test/social/browser_social_activation.js | original provider should have been reactivated
08:51:01 INFO - 715 INFO TEST-PASS | browser/base/content/test/social/browser_social_activation.js | providers are enabled
08:51:01 INFO - 716 INFO TEST-PASS | browser/base/content/test/social/browser_social_activation.js | The Social UI tests succeeded.
08:51:01 INFO - 717 INFO TEST-PASS | browser/base/content/test/social/browser_social_activation.js | all providers removed
08:51:01 INFO - 718 INFO sub-test testActivationMultipleProvider complete
08:51:01 INFO - 719 INFO TEST-PASS | browser/base/content/test/social/browser_social_activation.js | all providers removed
08:51:01 INFO - 720 INFO pre-test: starting with 0 providers
08:51:01 INFO - 721 INFO sub-test testAddonManagerDoubleInstall starting
08:51:01 INFO - 722 INFO TEST-PASS | browser/base/content/test/social/browser_social_activation.js | Current tab should be blank tab
08:51:01 INFO - 723 INFO TEST-PASS | browser/base/content/test/social/browser_social_activation.js | about:addons should load into blank tab.
08:51:01 INFO - 724 INFO 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 be declared in the document or in the transfer protocol." {file: "https://example.com/browser/browser/base/content/test/social/social_activate.html" line: 0}]
08:51:01 INFO - 725 INFO waiting on activation panel to open/close...
08:51:01 INFO - 726 INFO TEST-PASS | browser/base/content/test/social/browser_social_activation.js | servicesInstall-notification panel opened
08:51:01 INFO - 727 INFO TEST-PASS | browser/base/content/test/social/browser_social_activation.js | servicesInstall-notification panel hidden
08:51:01 INFO - 728 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_activation.js | Error: operation not possible on dead CPOW
08:51:01 INFO - ObserverChild.observe@resource://gre/modules/RemoteAddonsChild.jsm:399:1
08:51:01 INFO - -
08:51:01 INFO - Stack trace:
08:51:01 INFO - chrome://mochitests/content/browser/browser/base/content/test/social/head.js:waitForCondition/interval<:25
08:51:01 INFO - null:null:0
08:51:01 INFO - 729 INFO TEST-UNEXPECTED-FAIL | browser/base/content/test/social/browser_social_activation.js | Error: operation not possible on dead CPOW
08:51:01 INFO - -
08:51:01 INFO - Stack trace:
08:51:01 INFO - chrome://mochitests/content/browser/browser/base/content/test/social/head.js:waitForCondition/interval<:25
08:51:01 INFO - null:null:0
Bill, anything obvious to you here?
Flags: needinfo?(wmccloskey)
See Also: → 1150875
Flags: needinfo?(mixedpuppy)
See Also: → 1158977
I don't know what's going on here. We should probably just avoid using CPOWs in this test. It doesn't look like it would be too hard. I can help out if someone else can do the work.
Flags: needinfo?(wmccloskey)
The stack trace has:

> 06:21:00 INFO - Stack trace:
> 06:21:00 INFO - chrome://mochitests/content/browser/browser/base/content/test/social/head.js:waitForCondition/interval<:25
> 06:21:00 INFO - null:null:0
> 06:21:00 INFO - Not taking screenshot here: see the one that was previously logged

And the only call to waitForCondition uses gBrowser.contentDocument.location.href, so I suspect that's the CPOW in question. Changing it to gBrowser.currentURI still allows the test to pass, so my guess is that this might fix it.
Assignee: nobody → markh
Status: NEW → ASSIGNED
Flags: needinfo?(mixedpuppy)
Attachment #8658426 - Flags: review?(mixedpuppy)
Attachment #8658426 - Flags: review?(mixedpuppy) → review+
https://hg.mozilla.org/mozilla-central/rev/8ef6aa7ee1fe
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox43: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 43
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: