Closed Bug 1289944 Opened 9 years ago Closed 9 years ago

Remove e10s blob URL favicon hack

Categories

(Firefox :: Tabbed Browser, defect, P2)

Product:
Firefox
Component:
Tabbed Browser
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Firefox 50
Tracking Flags:
Tracking Status
e10s + ---
firefox50 --- fixed

People

(Reporter: mrbkap, Assigned: mrbkap)

Details

Attachments

(1 file)

Patch
2.48 KB, patch
Felipe
: review+
Details | Diff | Splinter Review
Bug 1184739 added some code to get around the fact that blob URLs didn't work cross process and so didn't work properly for sites that set their favicon to a blob URL. That's been fixed, thanks to baku, so we can remove the hack.
Assignee: nobody → mrbkap
Comment on attachment 8775373 [details] [diff] [review] Patch Review of attachment 8775373 [details] [diff] [review]: ----------------------------------------------------------------- \o/
Attachment #8775373 - Flags: review?(felipc) → review+
tracking-e10s: --- → +
Priority: -- → P2
Pushed by mrbkap@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/ec1fa4b01b12 Remove now-unneeded blob URL hack for favicons. r=felipe
https://hg.mozilla.org/mozilla-central/rev/ec1fa4b01b12
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox50: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 50
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: