ui: Use a PaintRecorder to access the Canvas for painting.

ui: Use a PaintRecorder to access the Canvas for painting.

Don't use the canvas() from the PaintContext so that we can have the
PaintContext be backed by a DisplayItemList instead in the future.
Access to the Canvas should be done through a PaintRecorder.

R=sadrul
BUG=466426
Committed: https://crrev.com/533d7341af315b7c25bf0269314c8c2e50aa5ca5
Cr-Commit-Position: refs/heads/master@{#324137}

[sadrul, 2015-04-07 21:10:13]

lgtm

https://codereview.chromium.org/1064133003/diff/1/ui/aura/window_unittest.cc
File ui/aura/window_unittest.cc (right):

https://codereview.chromium.org/1064133003/diff/1/ui/aura/window_unittest.cc#...
ui/aura/window_unittest.cc:34: #include "ui/compositor/paint_recorder.h"
Perhaps this include isn't necessary at all?

[danakj, 2015-04-07 21:11:48]

https://codereview.chromium.org/1064133003/diff/1/ui/aura/window_unittest.cc
File ui/aura/window_unittest.cc (right):

https://codereview.chromium.org/1064133003/diff/1/ui/aura/window_unittest.cc#...
ui/aura/window_unittest.cc:34: #include "ui/compositor/paint_recorder.h"
On 2015/04/07 21:10:13, sadrul wrote:
> Perhaps this include isn't necessary at all?

Oh, seems not!

[danakj, 2015-04-07 21:12:31]

The CQ bit was checked by danakj@chromium.org

[danakj, 2015-04-07 21:12:31]

The patchset sent to the CQ was uploaded after l-g-t-m from sadrul@chromium.org
Link to the patchset: https://codereview.chromium.org/1064133003/#ps20001
(title: "ui-recorder: .")

[commit-bot: I haz the power, 2015-04-07 21:12:46]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1064133003/20001

[commit-bot: I haz the power, 2015-04-07 22:32:40]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-04-07 22:33:33]

Patchset 2 (id:??) landed as
https://crrev.com/533d7341af315b7c25bf0269314c8c2e50aa5ca5
Cr-Commit-Position: refs/heads/master@{#324137}