Closed Bug 1465401 Opened 6 years ago Closed 6 years ago

setTabTitle shouldn't wrap data URI shortening in try/catch

Categories

(Firefox :: Tabbed Browser, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
Firefox 62
Tracking Status
firefox62 --- fixed

People

(Reporter: dao, Assigned: dao)

References

Details

Attachments

(1 file)

For the data URI shortening from bug 1408854, we don't expect exceptions, and therefore shouldn't use try/catch.
Summary: setTabTitle shouldn't wrap URI shortening in try/catch → setTabTitle shouldn't wrap data URI shortening in try/catch
Attachment #8981824 - Flags: review?(jfkthame)
Comment on attachment 8981824 [details] Bug 1465401 - setTabTitle shouldn't wrap data URI shortening in try/catch. https://reviewboard.mozilla.org/r/247878/#review254720
Attachment #8981824 - Flags: review?(jfkthame) → review+
Pushed by dgottwald@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4aa587fee045 setTabTitle shouldn't wrap data URI shortening in try/catch. r=jfkthame
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 62
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: