Check whether Java bitmap memory size is same to one of SkBitmap

Check whether Java bitmap memory size is same to one of SkBitmap

This is to understand why Issue 476559 is happening.

BUG=476559
Committed: https://crrev.com/a1d10e49da54d73e112c0a378b342bd73fa0f3a7
Cr-Commit-Position: refs/heads/master@{#329288}

[Jaekyun Seok (inactive), 2015-05-11 20:03:50]

jaekyun@chromium.org changed reviewers:
+ dtrainor@chromium.org, jdduke@chromium.org

[Jaekyun Seok (inactive), 2015-05-11 20:03:51]

jdduke@ and dtrainor@, could you please review this CL?

[David Trainor- moved to gerrit, 2015-05-11 22:11:20]

lgtm

[jdduke (slow), 2015-05-11 22:14:49]

Also lgtm.

https://codereview.chromium.org/1136053003/diff/1/ui/gfx/android/java_bitmap.cc
File ui/gfx/android/java_bitmap.cc (right):

https://codereview.chromium.org/1136053003/diff/1/ui/gfx/android/java_bitmap....
ui/gfx/android/java_bitmap.cc:112: CHECK_EQ(jbitmap.byte_count(), (int)
skbitmap.getSize());
nit: static_cast<int>

[Jaekyun Seok (inactive), 2015-05-11 23:05:53]

The CQ bit was checked by jaekyun@chromium.org

[Jaekyun Seok (inactive), 2015-05-11 23:05:54]

The patchset sent to the CQ was uploaded after l-g-t-m from jdduke@chromium.org,
dtrainor@chromium.org
Link to the patchset: https://codereview.chromium.org/1136053003/#ps20001
(title: "Use static_cast<int>")

[Jaekyun Seok (inactive), 2015-05-11 23:07:25]

https://codereview.chromium.org/1136053003/diff/1/ui/gfx/android/java_bitmap.cc
File ui/gfx/android/java_bitmap.cc (right):

https://codereview.chromium.org/1136053003/diff/1/ui/gfx/android/java_bitmap....
ui/gfx/android/java_bitmap.cc:112: CHECK_EQ(jbitmap.byte_count(), (int)
skbitmap.getSize());
On 2015/05/11 22:14:49, jdduke wrote:
> nit: static_cast<int>

Done.

[commit-bot: I haz the power, 2015-05-11 23:08:10]

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

[commit-bot: I haz the power, 2015-05-12 00:11:10]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-05-12 00:12:42]

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