CRs in fullscreen-video.xhtml browser-charsetmenu.inc browser-tabPreviews.js

RESOLVED FIXED

Status

()

Firefox
General
--
trivial
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: neil@parkwaycc.co.uk, Assigned: neil@parkwaycc.co.uk)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

8 years ago
fullscreen-video.html: 207 lines, 23 CRs
browser-charsetmenu.inc: 153 lines, 153 CRs
browser-tabPreviews.js: 1052 lines, 5 CRs
(Assignee)

Comment 1

8 years ago
Created attachment 495357 [details] [diff] [review]
Proposed patch
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #495357 - Flags: review?
(Assignee)

Updated

8 years ago
Attachment #495357 - Flags: review? → review?(dao)

Updated

8 years ago
Attachment #495357 - Flags: review?(dao) → review+
(Assignee)

Updated

8 years ago
Whiteboard: [needs approval]
(Assignee)

Updated

8 years ago
Attachment #495357 - Flags: approval2.0?
Comment on attachment 495357 [details] [diff] [review]
Proposed patch

This patch looks much scarier than it is. On the assumption that this is 100% whitespace, a=me.
Attachment #495357 - Flags: approval2.0? → approval2.0+
(Assignee)

Comment 3

8 years ago
Pushed changeset 24e0b001e858 to mozilla-central.
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Whiteboard: [needs approval]
You need to log in before you can comment on or make changes to this bug.