content/aura: Use a PaintRecorder to access the Canvas for painting.

content/aura: 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=piman@chromium.org, sadrul
BUG=466426
Committed: https://crrev.com/79b6513237fc18e4b67c2a686dacfa1ee89873a3
Cr-Commit-Position: refs/heads/master@{#324153}

[piman, 2015-04-07 21:11:09]

lgtm

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

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2015-04-07 21:11:55]

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

[sadrul, 2015-04-07 21:12:15]

lgtm

https://codereview.chromium.org/1066183002/diff/1/content/browser/web_content...
File content/browser/web_contents/aura/shadow_layer_delegate.cc (right):

https://codereview.chromium.org/1066183002/diff/1/content/browser/web_content...
content/browser/web_contents/aura/shadow_layer_delegate.cc:54: paint);
I wonder if there's a reason this doesn't use canvas()->DrawRect() instead.

[danakj, 2015-04-07 21:13:25]

https://codereview.chromium.org/1066183002/diff/1/content/browser/web_content...
File content/browser/web_contents/aura/shadow_layer_delegate.cc (right):

https://codereview.chromium.org/1066183002/diff/1/content/browser/web_content...
content/browser/web_contents/aura/shadow_layer_delegate.cc:54: paint);
On 2015/04/07 21:12:15, sadrul wrote:
> I wonder if there's a reason this doesn't use canvas()->DrawRect() instead.

Oh. I didn't even read that, I agree, and I'll switch it in another CL.

[danakj, 2015-04-07 21:19:58]

The CQ bit was checked by danakj@chromium.org

[danakj, 2015-04-07 21:19:58]

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

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

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

[commit-bot: I haz the power, 2015-04-07 23:21:40]

Committed patchset #2 (id:20001)

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

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