Summary: | Get rid of HIGH_DPI_CANVAS leftovers | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | mitz | ||||||
Component: | Canvas | Assignee: | mitz | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | benjamin, cmarcelo, commit-queue, dino, esprehn+autocc, gyuyoung.kim, kondapallykalyan, rakuco, roger_fong, ryuan.choi, sergio, thorton | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
mitz
2014-09-03 11:56:22 PDT
Created attachment 237567 [details]
I. Remove EANBLE_HIGH_DPI_CANVAS
Comment on attachment 237567 [details] I. Remove EANBLE_HIGH_DPI_CANVAS View in context: https://bugs.webkit.org/attachment.cgi?id=237567&action=review > Source/WebCore/html/HTMLCanvasElement.cpp:351 > float HTMLCanvasElement::targetDeviceScaleFactor() const Can't you remove all the "target device scale factor" stuff from canvas? (In reply to comment #2) > (From update of attachment 237567 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=237567&action=review > > > Source/WebCore/html/HTMLCanvasElement.cpp:351 > > float HTMLCanvasElement::targetDeviceScaleFactor() const > > Can't you remove all the "target device scale factor" stuff from canvas? I can. Committed attachment 237567 [details] as <http://trac.webkit.org/r173215>. Created attachment 237577 [details]
II. Remove HTMLCanvasElement::deviceScaleFactor()
Comment on attachment 237577 [details]
II. Remove HTMLCanvasElement::deviceScaleFactor()
Neat!
Committed attachment 237577 [details] as <http://trac.webkit.org/r173256>. |