Closed
Bug 1191038
Opened 9 years ago
Closed 9 years ago
Remove the sound playing icon when the browser crashes
Categories
(Firefox :: Tabbed Browser, defect)
Firefox
Tabbed Browser
Tracking
()
VERIFIED
FIXED
Firefox 42
Tracking | Status | |
---|---|---|
firefox42 | --- | verified |
People
(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)
References
Details
Attachments
(1 file)
1.04 KB,
patch
|
jaws
:
review+
|
Details | Diff | Splinter Review |
Currently after a crash, the sound playing icon stays there in the tab. We should remove it. See bug 1188548 comment 7.
Assignee | ||
Comment 1•9 years ago
|
||
Assignee | ||
Updated•9 years ago
|
Attachment #8643234 -
Flags: review?(jaws)
Comment 2•9 years ago
|
||
Comment on attachment 8643234 [details] [diff] [review] Remove the sound playing icon from a tab when it crashes Review of attachment 8643234 [details] [diff] [review]: ----------------------------------------------------------------- ::: browser/base/content/tabbrowser.xml @@ +4252,5 @@ > browser.docShell.displayLoadError(Cr.NS_ERROR_CONTENT_CRASHED, uri, null); > browser.removeAttribute("crashedPageTitle"); > let tab = this.getTabForBrowser(browser); > tab.setAttribute("crashed", true); > + tab.removeAttribute("soundplaying"); Shouldn't the "muted" attribute be removed as well?
Assignee | ||
Comment 3•9 years ago
|
||
(In reply to Jared Wein [:jaws] (please needinfo? me) from comment #2) > Comment on attachment 8643234 [details] [diff] [review] > Remove the sound playing icon from a tab when it crashes > > Review of attachment 8643234 [details] [diff] [review]: > ----------------------------------------------------------------- > > ::: browser/base/content/tabbrowser.xml > @@ +4252,5 @@ > > browser.docShell.displayLoadError(Cr.NS_ERROR_CONTENT_CRASHED, uri, null); > > browser.removeAttribute("crashedPageTitle"); > > let tab = this.getTabForBrowser(browser); > > tab.setAttribute("crashed", true); > > + tab.removeAttribute("soundplaying"); > > Shouldn't the "muted" attribute be removed as well? No, see bug 1188548. I have filed bug 1191081 for the muted case.
Updated•9 years ago
|
Attachment #8643234 -
Flags: review?(jaws) → review+
Comment 5•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/a37ae6d18b39
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox42:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 42
Comment 6•9 years ago
|
||
Verified fixed on Windows 7 64bit, Ubuntu 13.10 32bit and Mac OSX 10.9.5 using latest Aurora 42.0a2 (buildID: 20150826004006).
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•