Simplify Image::isTextureBacked()

Simplify Image::isTextureBacked()

Only AcceleratedStaticBitmapImages can be texture backed, and they
already override the method to return true.  No need for a default
baseclass impl beyond return false.

Also constify.

R=junov@chromium.org,ccameron@chromium.org

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

[f(malita), 2016-12-19 18:41:50]

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

[f(malita), 2016-12-19 18:42:05]

Description was changed from

==========
Simplify Image::isTextureBacked()

Only AcceleratedStaticBitmapImages can be texture backed, and they
already override the method to return true.  No need for a default
baseclass impl beyond return false.
==========

to

==========
Simplify Image::isTextureBacked()

Only AcceleratedStaticBitmapImages can be texture backed, and they
already override the method to return true.  No need for a default
baseclass impl beyond return false.

Also constify.
==========

[commit-bot: I haz the power, 2016-12-19 18:42:21]

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

[f(malita), 2016-12-19 18:44:49]

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

[commit-bot: I haz the power, 2016-12-19 18:45:15]

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

[f(malita), 2016-12-19 18:48:13]

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

[commit-bot: I haz the power, 2016-12-19 18:48:52]

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-12-19 20:55:04]

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

[commit-bot: I haz the power, 2016-12-19 20:55:05]

Dry run: This issue passed the CQ dry run.

[f(malita), 2016-12-19 20:56:21]

Description was changed from

==========
Simplify Image::isTextureBacked()

Only AcceleratedStaticBitmapImages can be texture backed, and they
already override the method to return true.  No need for a default
baseclass impl beyond return false.

Also constify.
==========

to

==========
Simplify Image::isTextureBacked()

Only AcceleratedStaticBitmapImages can be texture backed, and they
already override the method to return true.  No need for a default
baseclass impl beyond return false.

Also constify.

R=junov@chromium.org,ccameron@chromium.org
==========

[f(malita), 2016-12-19 20:56:22]

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

[ccameron, 2016-12-20 21:04:20]

lgtm

[f(malita), 2016-12-21 13:43:21]

fmalita@chromium.org changed reviewers:
+ schenney@chromium.org

[Justin Novosad, 2016-12-21 17:47:41]

LOL, of course!

lgtm

[f(malita), 2016-12-22 19:08:32]

The CQ bit was checked by fmalita@chromium.org

[commit-bot: I haz the power, 2016-12-22 19:08: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-12-22 19:10:52]

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

[commit-bot: I haz the power, 2016-12-22 19:10:53]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[f(malita), 2016-12-27 21:06:25]

The CQ bit was checked by fmalita@chromium.org

[f(malita), 2016-12-27 21:06:25]

The patchset sent to the CQ was uploaded after l-g-t-m from junov@chromium.org,
ccameron@chromium.org
Link to the patchset: https://codereview.chromium.org/2588103002/#ps60001
(title: "rebase")

[commit-bot: I haz the power, 2016-12-27 21:06:39]

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-27 22:28:42]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1482872785377860,
"parent_rev": "f3c21b29b0596c47c2f55da7196a842dae875509", "commit_rev":
"e9255da8ed217094d6c4479c134270502c9bb238"}

[commit-bot: I haz the power, 2016-12-27 22:29:09]

Description was changed from

==========
Simplify Image::isTextureBacked()

Only AcceleratedStaticBitmapImages can be texture backed, and they
already override the method to return true.  No need for a default
baseclass impl beyond return false.

Also constify.

R=junov@chromium.org,ccameron@chromium.org
==========

to

==========
Simplify Image::isTextureBacked()

Only AcceleratedStaticBitmapImages can be texture backed, and they
already override the method to return true.  No need for a default
baseclass impl beyond return false.

Also constify.

R=junov@chromium.org,ccameron@chromium.org

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

[commit-bot: I haz the power, 2016-12-27 22:29:10]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2017-01-02 15:47:31]

Description was changed from

==========
Simplify Image::isTextureBacked()

Only AcceleratedStaticBitmapImages can be texture backed, and they
already override the method to return true.  No need for a default
baseclass impl beyond return false.

Also constify.

R=junov@chromium.org,ccameron@chromium.org

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

to

==========
Simplify Image::isTextureBacked()

Only AcceleratedStaticBitmapImages can be texture backed, and they
already override the method to return true.  No need for a default
baseclass impl beyond return false.

Also constify.

R=junov@chromium.org,ccameron@chromium.org

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

[commit-bot: I haz the power, 2017-01-02 15:47:33]

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