Fix Mac custom cursor color.

Fix Mac custom cursor color.

BUG=114598
TEST=as in bug


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=122386

[_cary, 2012-02-16 16:38:18]

lgtm

[Avi (use Gerrit), 2012-02-16 16:42:32]

On 2012/02/16 16:38:18, _cary wrote:
> lgtm

forgot to explicitly clear the bitmap if there was no data, yielding a 1x1 dot
as a cursor... :( quick look at the delta?

[_cary, 2012-02-16 16:48:44]

I was wondering about that. I thought that maybe allocPixels zeroed its data,
but that wouldn't make sense from a performance perspective. LGTM

[commit-bot: I haz the power, 2012-02-16 18:55:41]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a lowly provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[Avi (use Gerrit), 2012-02-16 19:07:44]

Robert, can you take a look for approval?

[Robert Sesek, 2012-02-16 19:36:16]

lgtm

[commit-bot: I haz the power, 2012-02-16 19:41:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/avi@chromium.org/9419010/3001

[commit-bot: I haz the power, 2012-02-16 19:41:38]

Presubmit check for 9419010-3001 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for:
webkit/glue/webcursor_mac.mm,webkit/glue/webcursor.h,ui/gfx/native_widget_types.h

[Avi (use Gerrit), 2012-02-16 19:43:32]

OWNERS
ben: ui/
darin: webkit/

[Avi (use Gerrit), 2012-02-16 20:15:52]

I need some OWNERS approvals. Sorry to be annoying but this is a happy little P1
that needs some merging.

ben or sky: ui/
darin or tony: webkit/

[tony, 2012-02-16 20:26:05]

webkit LGTM

[Ben Goodger (Google), 2012-02-16 20:32:09]

lgtm

[commit-bot: I haz the power, 2012-02-16 20:38:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/avi@chromium.org/9419010/3001

[commit-bot: I haz the power, 2012-02-16 21:21:30]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/avi@chromium.org/9419010/8

[darin (slow to review), 2012-02-16 21:23:24]

LGTM for webkit/

[commit-bot: I haz the power, 2012-02-16 23:22:10]

Change committed as 122386