Add a new class AcceleratedStaticBitmap : StaticBitmapImage

Add a new class AcceleratedStaticBitmap : StaticBitmapImage

At this moment, the StaticBitmapImage holds RefPtr to a SkImage, or a
WebExternalTextureMailbox. So there are in total of three cases:
1). The SkImage is not texture backed.
2). The SkImage is GPU texture backed.
3). The SkImage is null, but has a valid mailbox.

It makes more sense to break this class into two classes. So this CL
adds a new class called AcceleratedStaticBitmap, inherited from the
StaticBitmapImage class, is in charge of case 2 and 3. Case 1 is handled
by the StaticBitmapImage class.

We can use the bots (including the GPU_Optional bots) to make sure this
change doesn't break anything.

BUG=632722
Committed: https://crrev.com/23b4e88d6090ae9f2b2fa9aa062152900c09f7e9
Cr-Commit-Position: refs/heads/master@{#409238}

[xidachen, 2016-08-02 02:05:11]

xidachen@chromium.org changed reviewers:
+ fmalita@chromium.org, junov@chromium.org

[xidachen, 2016-08-02 02:05:12]

PTAL

PS#2 passes many bots, and PS#3 is almost identical as PS#2.

[Justin Novosad, 2016-08-02 15:26:01]

lgtm

https://codereview.chromium.org/2196963002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/graphics/StaticBitmapImage.cpp (right):

https://codereview.chromium.org/2196963002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/StaticBitmapImage.cpp:23: return
AcceleratedStaticBitmapImage::create(image);
Excellent!

[xidachen, 2016-08-02 15:27:25]

The CQ bit was checked by xidachen@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-02 15:27:49]

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

[xidachen, 2016-08-02 15:32:57]

The CQ bit was unchecked by xidachen@chromium.org

[f(malita), 2016-08-02 15:42:08]

Thanks for refactoring this, LGTM.

[xidachen, 2016-08-02 16:47:13]

The CQ bit was checked by xidachen@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-02 16:47:42]

Dry run: 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-08-02 17:43:18]

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

[commit-bot: I haz the power, 2016-08-02 17:43:19]

Dry run: This issue passed the CQ dry run.

[xidachen, 2016-08-02 17:58:05]

The CQ bit was checked by xidachen@chromium.org

[commit-bot: I haz the power, 2016-08-02 17:58:55]

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-08-02 18:04:30]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-02 18:07:45]

Description was changed from

==========
Add a new class AcceleratedStaticBitmap : StaticBitmapImage

At this moment, the StaticBitmapImage holds RefPtr to a SkImage, or a
WebExternalTextureMailbox. So there are in total of three cases:
1). The SkImage is not texture backed.
2). The SkImage is GPU texture backed.
3). The SkImage is null, but has a valid mailbox.

It makes more sense to break this class into two classes. So this CL
adds a new class called AcceleratedStaticBitmap, inherited from the
StaticBitmapImage class, is in charge of case 2 and 3. Case 1 is handled
by the StaticBitmapImage class.

We can use the bots (including the GPU_Optional bots) to make sure this
change doesn't break anything.

BUG=632722
==========

to

==========
Add a new class AcceleratedStaticBitmap : StaticBitmapImage

At this moment, the StaticBitmapImage holds RefPtr to a SkImage, or a
WebExternalTextureMailbox. So there are in total of three cases:
1). The SkImage is not texture backed.
2). The SkImage is GPU texture backed.
3). The SkImage is null, but has a valid mailbox.

It makes more sense to break this class into two classes. So this CL
adds a new class called AcceleratedStaticBitmap, inherited from the
StaticBitmapImage class, is in charge of case 2 and 3. Case 1 is handled
by the StaticBitmapImage class.

We can use the bots (including the GPU_Optional bots) to make sure this
change doesn't break anything.

BUG=632722

Committed: https://crrev.com/23b4e88d6090ae9f2b2fa9aa062152900c09f7e9
Cr-Commit-Position: refs/heads/master@{#409238}
==========

[commit-bot: I haz the power, 2016-08-02 18:07:46]

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