Summary: | [v8] document.getCSSCanvasContext doesn't need to be custom | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Adam Barth <abarth> | ||||||||
Component: | New Bugs | Assignee: | Adam Barth <abarth> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | eric, gyuyoung.kim, haraken, japhet, ojan, rakuco, webkit.review.bot | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Adam Barth
2012-09-12 14:38:58 PDT
Created attachment 163695 [details]
Patch
Created attachment 163698 [details]
Patch
Comment on attachment 163698 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=163698&action=review > Source/WebCore/bindings/v8/custom/V8CanvasRenderingContextCustom.cpp:45 > + return v8NullWithCheck(isolate); This is now null instead of undefined. Do we care? (In reply to comment #3) > (From update of attachment 163698 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=163698&action=review > > > Source/WebCore/bindings/v8/custom/V8CanvasRenderingContextCustom.cpp:45 > > + return v8NullWithCheck(isolate); > > This is now null instead of undefined. Do we care? I don't really care, but I'll add a test just for you. Created attachment 163706 [details]
Patch
Now with test. Comment on attachment 163706 [details]
Patch
OK.
Comment on attachment 163706 [details] Patch Clearing flags on attachment: 163706 Committed r128374: <http://trac.webkit.org/changeset/128374> All reviewed patches have been landed. Closing bug. |