Summary: | [windows] Convert usages of GetDC to HWndDC Part 1. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | David Levin <levin> | ||||||
Component: | Platform | Assignee: | David Levin <levin> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | aroben, ddkilzer, dslomov, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 76303 | ||||||||
Attachments: |
|
Description
David Levin
2012-01-20 15:00:01 PST
Created attachment 123385 [details]
Patch
Created attachment 123394 [details]
Patch
Comment on attachment 123394 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=123394&action=review Nice! > Source/WebCore/platform/win/CursorWin.cpp:54 > HDC workingDC = CreateCompatibleDC(dc); This will be replaced with OwnDC at some point right? (In reply to comment #3) > This will be replaced with OwnDC at some point right? Ideally there would be a pass to do that. I'm not going to do that for now. (Spent too much time on this now.) I thought about filling a bug but I'm not sure it would do any good. However, there is a style check now which will encourage the right behavior! Comment on attachment 123394 [details]
Patch
Thanks for splitting this up!
Comment on attachment 123394 [details] Patch Clearing flags on attachment: 123394 Committed r105661: <http://trac.webkit.org/changeset/105661> All reviewed patches have been landed. Closing bug. |