don't need native-drawing-context for CanvasSkiaPaint

don't need native-drawing-context for CanvasSkiaPaint

BUG=675977
Committed: https://crrev.com/4c5054aecae9bb99c972e7cca364053a99c4df39
Cr-Commit-Position: refs/heads/master@{#441392}

[reed1, 2017-01-04 14:08:04]

reed@google.com changed reviewers:
+ fmalita@chromium.org

[reed1, 2017-01-04 14:08:12]

The CQ bit was checked by reed@google.com to run a CQ dry run

[commit-bot: I haz the power, 2017-01-04 14:08:27]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[reed1, 2017-01-04 14:09:33]

reed@google.com changed reviewers:
+ avi@chromium.org, ccameron@chromium.org

[reed1, 2017-01-04 14:10:46]

Description was changed from

==========
don't need native-drawing-context for CanvasSkiaPaint

BUG=
==========

to

==========
don't need native-drawing-context for CanvasSkiaPaint

BUG=675977
==========

[f(malita), 2017-01-04 14:31:36]

lgtm

https://codereview.chromium.org/2612023003/diff/1/ui/gfx/canvas_paint_mac.mm
File ui/gfx/canvas_paint_mac.mm (right):

https://codereview.chromium.org/2612023003/diff/1/ui/gfx/canvas_paint_mac.mm#...
ui/gfx/canvas_paint_mac.mm:31: bool peek = canvas->peekPixels(&pixmap);
DCHECK(peek)?

[reed1, 2017-01-04 14:48:10]

The CQ bit was checked by reed@google.com to run a CQ dry run

[commit-bot: I haz the power, 2017-01-04 14:48:20]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[reed1, 2017-01-04 14:48:23]

https://codereview.chromium.org/2612023003/diff/1/ui/gfx/canvas_paint_mac.mm
File ui/gfx/canvas_paint_mac.mm (right):

https://codereview.chromium.org/2612023003/diff/1/ui/gfx/canvas_paint_mac.mm#...
ui/gfx/canvas_paint_mac.mm:31: bool peek = canvas->peekPixels(&pixmap);
On 2017/01/04 14:31:36, f(malita) wrote:
> DCHECK(peek)?

Done.

[commit-bot: I haz the power, 2017-01-04 15:33:05]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-04 15:33:06]

Dry run: This issue passed the CQ dry run.

[Avi (use Gerrit), 2017-01-04 16:32:47]

lgtm

If the experts are happy, stamp stamp

[reed1, 2017-01-04 16:36:09]

The CQ bit was checked by reed@google.com

[reed1, 2017-01-04 16:36:10]

The patchset sent to the CQ was uploaded after l-g-t-m from fmalita@chromium.org
Link to the patchset: https://codereview.chromium.org/2612023003/#ps40001
(title: "rebase")

[commit-bot: I haz the power, 2017-01-04 16:36:27]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-04 16:41:05]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1483547769664250,
"parent_rev": "4f0834b23eabc76d275c4393f791b451b1ee5ab3", "commit_rev":
"437ccc235abea0d654d3a5aac6910cb0a586d14f"}

[commit-bot: I haz the power, 2017-01-04 16:41:35]

Description was changed from

==========
don't need native-drawing-context for CanvasSkiaPaint

BUG=675977
==========

to

==========
don't need native-drawing-context for CanvasSkiaPaint

BUG=675977

Review-Url: https://codereview.chromium.org/2612023003
==========

[commit-bot: I haz the power, 2017-01-04 16:41:36]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2017-01-04 16:45:18]

Description was changed from

==========
don't need native-drawing-context for CanvasSkiaPaint

BUG=675977

Review-Url: https://codereview.chromium.org/2612023003
==========

to

==========
don't need native-drawing-context for CanvasSkiaPaint

BUG=675977

Committed: https://crrev.com/4c5054aecae9bb99c972e7cca364053a99c4df39
Cr-Commit-Position: refs/heads/master@{#441392}
==========

[commit-bot: I haz the power, 2017-01-04 16:45:19]

Patchset 3 (id:??) landed as
https://crrev.com/4c5054aecae9bb99c972e7cca364053a99c4df39
Cr-Commit-Position: refs/heads/master@{#441392}