Closed Bug 1168229 Opened 9 years ago Closed 8 years ago

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

Categories

(Firefox Graveyard :: SocialAPI, defect)

defect
Not set
normal

Tracking

(e10s+, firefox47 wontfix, firefox48 fixed, firefox49 fixed)

RESOLVED FIXED
Firefox 49
Tracking Status
e10s + ---
firefox47 --- wontfix
firefox48 --- fixed
firefox49 --- fixed

People

(Reporter: philor, Assigned: mixedpuppy)

References

(Blocks 1 open bug)

Details

(Keywords: intermittent-failure)

Attachments

(1 file, 1 obsolete file)

      No description provided.
Summary: Intermittent browser_aboutHome_activation.js | Error: operation not possible on dead CPOW → Intermittent e10s browser_aboutHome_activation.js | Error: operation not possible on dead CPOW
Blocks: e10s-tests
tracking-e10s: --- → +
This is a really frequent Social intermittent. Any chance you could take a look (or pass it off to someone else) before your leave, Shane?
Flags: needinfo?(mixedpuppy)
Attached patch fix dead CpOWs. (obsolete) — Splinter Review
I think this will do it.  try is currently unavailable.
Flags: needinfo?(mixedpuppy)
Attachment #8750978 - Flags: review?(markh)
Comment on attachment 8750978 [details] [diff] [review]
fix dead CpOWs.

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

rs=me

::: browser/base/content/test/social/social_activate.html
@@ +1,3 @@
>  <html>
>  <head>
> +    <meta charset="utf-8">

this indentation seems odd.

::: browser/base/content/test/social/social_postActivation.html
@@ +1,3 @@
>  <html>
>  <head>
> +    <meta charset="utf-8">

ditto here.
Attachment #8750978 - Flags: review?(markh) → review+
FWIW, finally repro'd on debug linux (talk about a snails pace).  Patch fixes the dead CPOW, but now there are docshell leaks.  Will try again.
Attached patch fix dead CpOWs.Splinter Review
This is a larger rewrite of the test and from the last try it is looking like everything is passing.  Given an r+ on it feel free to land it (depending on timing I may not get to that).
Attachment #8750978 - Attachment is obsolete: true
Attachment #8751881 - Flags: review?(markh)
Attachment #8751881 - Flags: review?(markh) → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/30601f36e851
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Assignee: nobody → mixedpuppy
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.