If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Backout Bug 698187 due to mozilla-central back out of parts of bug 698986 to resolve bug 716945.

RESOLVED FIXED in seamonkey2.10

Status

SeaMonkey
General
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Philip Chee, Assigned: Philip Chee)

Tracking

Trunk
seamonkey2.10
Dependency tree / graph

SeaMonkey Tracking Flags

(seamonkey2.7 wontfix, seamonkey2.8 fixed, seamonkey2.9 fixed, seamonkey2.10+ fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Mozilla-central plans to backout out parts of bug 698986 to resolve bug 716945.
q.v.:
Bug 716945 - Figure out why Flash hang volume dropped very significantly in 10.0a1 trunk on 2011-10-29
Bug 665196 - Change in-content plugin crash UI to submit a crash report and reload the page in one step.
SM Bug 698187 Sync pluginCrashedUI with changes from Firefox (Bug 665196 and Bug 688083)

Comment 1

6 years ago
FYI, this has already been backed out from Firefox 11 (now Beta).
(Assignee)

Comment 2

6 years ago
Created attachment 594205 [details] [diff] [review]
Patch v1.0 Backout.

Attaching patch for check-in for when m-c backs out their core patch.
(Assignee)

Comment 3

6 years ago
Comment on attachment 594205 [details] [diff] [review]
Patch v1.0 Backout.

> FYI, this has already been backed out from Firefox 11 (now Beta).
In that case, asking for approval for comm-beta (SeaMonkey 2.8)
Attachment #594205 - Flags: review?(neil)
Attachment #594205 - Flags: approval-comm-beta?

Updated

6 years ago
Attachment #594205 - Flags: review?(neil) → review+
Hum, why is this bug marked "status-seamonkey2.7: fixed"? And don't we want approval-comm-aurora too/first?
(Assignee)

Comment 5

6 years ago
> Hum, why is this bug marked "status-seamonkey2.7: fixed"?
I must have forgotten to reset the flags when I cloned the bug.
> And don't we want approval-comm-aurora too/first?
Only if MoCo backs out the underlying patch from mozilla-aurora.
status-seamonkey2.7: fixed → ---
(Assignee)

Comment 6

6 years ago
Comment on attachment 594205 [details] [diff] [review]
Patch v1.0 Backout.

> Only if MoCo backs out the underlying patch from mozilla-aurora.
Bug 716945 Comment 25 has approval for mozilla-aurora.
Asking for comm-aurora.
Attachment #594205 - Flags: approval-comm-aurora?

Comment 7

6 years ago
(In reply to Philip Chee from comment #6)
> Comment on attachment 594205 [details] [diff] [review]
> Patch v1.0 Backout.
> 
> > Only if MoCo backs out the underlying patch from mozilla-aurora.
> Bug 716945 Comment 25 has approval for mozilla-aurora.
> Asking for comm-aurora.

Do we have a way of making flash crash so we can see if we fail at submitting too?
(Assignee)

Comment 8

6 years ago
> Do we have a way of making flash crash so we can see if we fail at submitting too?
In Win7 I used the task manager to kill the plugin container.
(Assignee)

Comment 9

6 years ago
The underlying bug was backed out of mozilla-central: https://hg.mozilla.org/mozilla-central/rev/2087f423a1a4
We need to do the same for suite.

Comment 10

6 years ago
Yes, this has been backed out of all non-release channels now, i.e. Mozilla toolkit 11 and higher.
(Assignee)

Comment 11

6 years ago
Pushed to comm-central:
http://hg.mozilla.org/comm-central/rev/243b4f14dab8

Still needs approval for comm-aurora and comm-central!
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
status-seamonkey2.7: --- → wontfix
status-seamonkey2.8: --- → affected
status-seamonkey2.9: --- → affected
Depends on: 716945
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.10
Comment on attachment 594205 [details] [diff] [review]
Patch v1.0 Backout.

I didn't verify that the m-c bug was backed out on these branches, but assuming it was, land away.
Attachment #594205 - Flags: approval-comm-beta?
Attachment #594205 - Flags: approval-comm-beta+
Attachment #594205 - Flags: approval-comm-aurora?
Attachment #594205 - Flags: approval-comm-aurora+
(Assignee)

Comment 13

6 years ago
REOPENED Still needs to land on comm-beta and comm-aurora.

> I didn't verify that the m-c bug was backed out on these branches
According to KaiRo, it was.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(In reply to Philip Chee from comment #13)
> REOPENED Still needs to land on comm-beta and comm-aurora.

Main 'Status' is for m-c; 'status-seamonkey*' are for branches:
so current status of this bug should actually be "Fixed"...
(Assignee)

Comment 15

6 years ago
I'm keeping this open to remind *ME* to check this in to the branches.
(Assignee)

Comment 16

6 years ago
Pushed to comm-aurora:
http://hg.mozilla.org/releases/comm-aurora/rev/58d020405ee4
Pushed to comm-beta:
http://hg.mozilla.org/releases/comm-beta/rev/9f3e1bfd37aa
Status: REOPENED → RESOLVED
Last Resolved: 6 years ago6 years ago
status-seamonkey2.8: affected → fixed
status-seamonkey2.9: affected → fixed
Resolution: --- → FIXED
status-seamonkey2.10: --- → fixed
You need to log in before you can comment on or make changes to this bug.