Closed Bug 1289944 Opened 4 years ago Closed 4 years ago

Remove e10s blob URL favicon hack

Categories

(Firefox :: Tabbed Browser, defect, P2)

defect

Tracking

()

RESOLVED FIXED
Firefox 50
Tracking Status
e10s + ---
firefox50 --- fixed

People

(Reporter: mrbkap, Assigned: mrbkap)

Details

Attachments

(1 file)

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: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 50
You need to log in before you can comment on or make changes to this bug.