Last Comment Bug 635544 - rename nsIWidget::DrawOver
: rename nsIWidget::DrawOver
Product: Core
Classification: Components
Component: Graphics (show other bugs)
: unspecified
: x86 Mac OS X
-- normal (vote)
: mozilla12
Assigned To: Benoit Girard (:BenWa)
: Milan Sreckovic [:milan]
Depends on:
  Show dependency treegraph
Reported: 2011-02-19 13:44 PST by Joe Drew (not getting mail)
Modified: 2012-01-22 12:31 PST (History)
6 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

patch (4.70 KB, patch)
2012-01-21 06:18 PST, Benoit Girard (:BenWa)
roc: review+
Details | Diff | Splinter Review

Description User image Joe Drew (not getting mail) 2011-02-19 13:44:13 PST
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?
Comment 1 User image Benoit Girard (:BenWa) 2012-01-20 12:58:13 PST
Jeff/Joe any idea what that rename should be? I'll post the patch and get this bug closed.
Comment 2 User image Robert O'Callahan (:roc) (email my personal email if necessary) 2012-01-21 00:21:40 PST
There is no DrawUnder.

Maybe DrawOver -> DrawWindowOverlay?
Comment 3 User image Benoit Girard (:BenWa) 2012-01-21 06:18:28 PST
Created attachment 590463 [details] [diff] [review]
Comment 4 User image Mozilla RelEng Bot 2012-01-21 10:00:58 PST
Try run for 43135679b839 is complete.
Detailed breakdown of the results available here:
Results (out of 4 total builds):
    success: 4
Builds (or logs if builds failed) available at:
Comment 6 User image Ed Morley [:emorley] 2012-01-22 12:31:12 PST

Note You need to log in before you can comment on or make changes to this bug.