rename nsIWidget::DrawOver

RESOLVED FIXED in mozilla12

Status

()

Core
Graphics
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: Joe Drew (not getting mail), Assigned: BenWa)

Tracking

unspecified
mozilla12
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
Jeff thinks that DrawOver and DrawUnder are not sufficiently descriptive names.

> Renaming DrawOver might also be a good idea. The current name makes you ask
Draw what? Over what?
(Assignee)

Comment 1

5 years ago
Jeff/Joe any idea what that rename should be? I'll post the patch and get this bug closed.
There is no DrawUnder.

Maybe DrawOver -> DrawWindowOverlay?
(Assignee)

Updated

5 years ago
Summary: rename nsIWidget::DrawOver/nsIWidget::DrawUnder → rename nsIWidget::DrawOver
(Assignee)

Comment 3

5 years ago
Created attachment 590463 [details] [diff] [review]
patch
Assignee: nobody → bgirard
Status: NEW → ASSIGNED
Attachment #590463 - Flags: review?(roc)

Comment 4

5 years ago
Try run for 43135679b839 is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=43135679b839
Results (out of 4 total builds):
    success: 4
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/b56girard@gmail.com-43135679b839
Attachment #590463 - Flags: review?(roc) → review+
(Assignee)

Comment 5

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/40bc6b9b09d1
https://hg.mozilla.org/mozilla-central/rev/40bc6b9b09d1
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla12
You need to log in before you can comment on or make changes to this bug.