Scrub deprecated Skia APIs.

Scrub deprecated Skia APIs.

Refactor clients using equivalent non-deprecated methods.

R=reed@google.com,enne@chromium.org,sky@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=269497

[f(malita), 2014-05-07 17:09:46]

https://codereview.chromium.org/271653003/diff/1/ui/gfx/canvas.h
File ui/gfx/canvas.h (left):

https://codereview.chromium.org/271653003/diff/1/ui/gfx/canvas.h#oldcode448
ui/gfx/canvas.h:448: // Builds mip map for returned image rep if necessary.
SkBitmap::buildMipMap() is essentially dead code (see
https://codereview.chromium.org/271693002/), so these methods are relegated to
thin wrappers around ImageSkia::GetRepresentation() - hence the removal.

[enne (OOO), 2014-05-07 17:14:30]

cc rslgtm

[reed1, 2014-05-07 18:06:22]

lgtm -- my thoughts about colortype can happen in a later CL

https://codereview.chromium.org/271653003/diff/1/chrome/browser/ui/app_list/f...
File chrome/browser/ui/app_list/fast_show_pickler.cc (right):

https://codereview.chromium.org/271653003/diff/1/chrome/browser/ui/app_list/f...
chrome/browser/ui/app_list/fast_show_pickler.cc:87: if
(!ConfigToFormat(it->sk_bitmap().config(), &format))
nit: can we change this more, to pass in colortype instead of config (which is
itself deprecated)? Not critical for this CL, but will need to happen
eventually.

https://codereview.chromium.org/271653003/diff/1/content/common/gpu/client/gl...
File content/common/gpu/client/gl_helper_unittest.cc (right):

https://codereview.chromium.org/271653003/diff/1/content/common/gpu/client/gl...
content/common/gpu/client/gl_helper_unittest.cc:859: if (bmp.config() ==
SkBitmap::kARGB_8888_Config) {
Can we compare using colortypes instead of configs?

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

https://codereview.chromium.org/271653003/diff/1/ui/gfx/android/java_bitmap.c...
ui/gfx/android/java_bitmap.cc:85: SkBitmap::Config bitmap_config =
skbitmap->config();
use colortypes?

[sky, 2014-05-07 20:09:00]

LGTM

[f(malita), 2014-05-07 21:16:19]

The CQ bit was checked by fmalita@chromium.org

[commit-bot: I haz the power, 2014-05-07 21:20:29]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/fmalita@chromium.org/271653003/1

[f(malita), 2014-05-07 21:31:43]

+ scherkus for media/ & piman for content/common/gpu/

[commit-bot: I haz the power, 2014-05-07 21:45:44]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  chromium_presubmit on tryserver.chromium

[commit-bot: I haz the power, 2014-05-07 21:58:24]

FYI, CQ is re-trying this CL (attempt #2).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  chromium_presubmit on tryserver.chromium

[commit-bot: I haz the power, 2014-05-07 22:01:53]

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

[commit-bot: I haz the power, 2014-05-07 22:01:53]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium

[scherkus (not reviewing), 2014-05-07 23:33:24]

media lgtm

[f(malita), 2014-05-08 22:38:24]

The CQ bit was checked by fmalita@chromium.org

[commit-bot: I haz the power, 2014-05-08 22:47:50]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/fmalita@chromium.org/271653003/1

[commit-bot: I haz the power, 2014-05-08 23:40:23]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[commit-bot: I haz the power, 2014-05-08 23:58:52]

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

[commit-bot: I haz the power, 2014-05-08 23:58:53]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[piman, 2014-05-09 00:59:52]

lgtm

[f(malita), 2014-05-09 01:00:23]

The CQ bit was checked by fmalita@chromium.org

[commit-bot: I haz the power, 2014-05-09 01:05:01]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/fmalita@chromium.org/271653003/1

[f(malita), 2014-05-09 13:26:50]

The CQ bit was unchecked by fmalita@chromium.org

[f(malita), 2014-05-09 13:27:22]

The CQ bit was checked by fmalita@chromium.org

[commit-bot: I haz the power, 2014-05-09 13:30:12]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/fmalita@chromium.org/271653003/1

[commit-bot: I haz the power, 2014-05-10 02:48:05]

Change committed as 269497