If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Kill old gfx code in nsWindowsShellService

RESOLVED FIXED

Status

()

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

People

(Reporter: sciguyryan, Assigned: sciguyryan)

Tracking

Trunk
x86
Windows XP
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

10 years ago
Remove all the old, dead gfx code from nsWindowsShellService.
(Assignee)

Comment 1

10 years ago
Created attachment 269824 [details] [diff] [review]
Patch v1.0

Patch v1.0
Attachment #269824 - Flags: review?(gavin.sharp)
(Assignee)

Updated

10 years ago
Attachment #269824 - Flags: review?(gavin.sharp) → review?(vladimir)
Comment on attachment 269824 [details] [diff] [review]
Patch v1.0

Sorry for the delay, poking me on IRC will usually help :/
Attachment #269824 - Flags: review?(vladimir) → review+
(Assignee)

Comment 3

10 years ago
(In reply to comment #2)
> (From update of attachment 269824 [details] [diff] [review])
> Sorry for the delay, poking me on IRC will usually help :/
> 

Cheers for the review Gavin. Does this require a sr or does a simple change like this not require one in this case?
Assignee: sciguyryan → nobody
Status: ASSIGNED → NEW
(Assignee)

Updated

10 years ago
Assignee: nobody → sciguyryan
No, it doesn't require SR. Code in /browser only does if the reviewer requests it.
(Assignee)

Comment 5

10 years ago
OK. Thanks Gavin.
Status: NEW → ASSIGNED
Whiteboard: [checkin needed]
Keywords: checkin-needed
Whiteboard: [checkin needed]
browser/components/shell/src/nsWindowsShellService.cpp 1.46
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Flags: in-testsuite-
You need to log in before you can comment on or make changes to this bug.