Don't hold onto SkBitmap in mus_demo.

Don't hold onto SkBitmap in mus_demo.

There is no need to hold onto the SkBitmap object as a member variable
in MusDemo. It was originally retained so that the pixel buffer didn't
need to be reallocated for each frame (a copy of the pixel buffer was
made when submitting the frame). However, the pixel buffer is now locked
and ownership is shared with the gfx::ImageSkiaRep, requiring a new
pixel buffer to be allocated for each frame regardless. Remove |bitmap_|
member variable.

BUG=none

Committed: https://crrev.com/ab87dd7cd4a7d855c8b43b848618810f5c892143
Cr-Commit-Position: refs/heads/master@{#439129}

[kylechar, 2016-12-16 14:29:10]

kylechar@chromium.org changed reviewers:
+ mfomitchev@chromium.org

[mfomitchev, 2016-12-16 16:57:10]

Sure, lgtm.

[kylechar, 2016-12-16 16:57:38]

The CQ bit was checked by kylechar@chromium.org

[commit-bot: I haz the power, 2016-12-16 16:58:12]

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, 2016-12-16 17:10:31]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481907458302040,
"parent_rev": "29087e232a8cfb1407f40e92e5a697831c150487", "commit_rev":
"45f88c7a8f04ea955fb802e2d0c560b05f523dde"}

[commit-bot: I haz the power, 2016-12-16 17:10:58]

Description was changed from

==========
Don't hold onto SkBitmap in mus_demo.

There is no need to hold onto the SkBitmap object as a member variable
in MusDemo. It was originally retained so that the pixel buffer didn't
need to be reallocated for each frame (a copy of the pixel buffer was
made when submitting the frame). However, the pixel buffer is now locked
and ownership is shared with the gfx::ImageSkiaRep, requiring a new
pixel buffer to be allocated for each frame regardless. Remove |bitmap_|
member variable.

BUG=none
==========

to

==========
Don't hold onto SkBitmap in mus_demo.

There is no need to hold onto the SkBitmap object as a member variable
in MusDemo. It was originally retained so that the pixel buffer didn't
need to be reallocated for each frame (a copy of the pixel buffer was
made when submitting the frame). However, the pixel buffer is now locked
and ownership is shared with the gfx::ImageSkiaRep, requiring a new
pixel buffer to be allocated for each frame regardless. Remove |bitmap_|
member variable.

BUG=none

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

[commit-bot: I haz the power, 2016-12-16 17:10:59]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-16 17:13:13]

Description was changed from

==========
Don't hold onto SkBitmap in mus_demo.

There is no need to hold onto the SkBitmap object as a member variable
in MusDemo. It was originally retained so that the pixel buffer didn't
need to be reallocated for each frame (a copy of the pixel buffer was
made when submitting the frame). However, the pixel buffer is now locked
and ownership is shared with the gfx::ImageSkiaRep, requiring a new
pixel buffer to be allocated for each frame regardless. Remove |bitmap_|
member variable.

BUG=none

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

to

==========
Don't hold onto SkBitmap in mus_demo.

There is no need to hold onto the SkBitmap object as a member variable
in MusDemo. It was originally retained so that the pixel buffer didn't
need to be reallocated for each frame (a copy of the pixel buffer was
made when submitting the frame). However, the pixel buffer is now locked
and ownership is shared with the gfx::ImageSkiaRep, requiring a new
pixel buffer to be allocated for each frame regardless. Remove |bitmap_|
member variable.

BUG=none

Committed: https://crrev.com/ab87dd7cd4a7d855c8b43b848618810f5c892143
Cr-Commit-Position: refs/heads/master@{#439129}
==========

[commit-bot: I haz the power, 2016-12-16 17:13:14]

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