gpu: Remove in-process GPU service support for CHROMIUM_image.

gpu: Remove in-process GPU service support for CHROMIUM_image.

This also removes the android_webview GpuMemoryBuffer
implementation.

BUG=418553
Committed: https://crrev.com/ec103f0dcc3fc00f4e6659550120540d98b0aed8
Cr-Commit-Position: refs/heads/master@{#297224}

[reveman, 2014-09-29 14:58:23]

reveman@chromium.org changed reviewers:
+ boliu@chromium.org, piman@chromium.org

[boliu, 2014-09-29 15:10:08]

lgtm

[boliu, 2014-09-29 15:47:18]

still need a bunch of compile fixes on android

https://codereview.chromium.org/608263002/diff/1/android_webview/browser/gpu_...
File android_webview/browser/gpu_memory_buffer_factory_impl.cc (left):

https://codereview.chromium.org/608263002/diff/1/android_webview/browser/gpu_...
android_webview/browser/gpu_memory_buffer_factory_impl.cc:1: // Copyright 2013
The Chromium Authors. All rights reserved.
remove this and header from android_webview/android_webview.gyp

https://codereview.chromium.org/608263002/diff/1/android_webview/lib/main/aw_...
File android_webview/lib/main/aw_main_delegate.cc (left):

https://codereview.chromium.org/608263002/diff/1/android_webview/lib/main/aw_...
android_webview/lib/main/aw_main_delegate.cc:53: CommandLine* cl =
CommandLine::ForCurrentProcess();
Still need cl to be defined.

https://codereview.chromium.org/608263002/diff/1/android_webview/lib/main/aw_...
File android_webview/lib/main/aw_main_delegate.cc (right):

https://codereview.chromium.org/608263002/diff/1/android_webview/lib/main/aw_...
android_webview/lib/main/aw_main_delegate.cc:54:
BrowserViewRenderer::CalculateTileMemoryPolicy(use_zero_copy);
Clean this up and remove the use_zero_copy branch in that function.

https://codereview.chromium.org/608263002/diff/1/android_webview/native/aw_co...
File android_webview/native/aw_contents.cc (right):

https://codereview.chromium.org/608263002/diff/1/android_webview/native/aw_co...
android_webview/native/aw_contents.cc:14: #include
"android_webview/browser/gpu_memory_buffer_factory_impl.h"
remove this

[reveman, 2014-09-29 18:32:09]

sorry, I was bit quick to upload that patch. I think this should be good now.
PTAL.

https://codereview.chromium.org/608263002/diff/1/android_webview/browser/gpu_...
File android_webview/browser/gpu_memory_buffer_factory_impl.cc (left):

https://codereview.chromium.org/608263002/diff/1/android_webview/browser/gpu_...
android_webview/browser/gpu_memory_buffer_factory_impl.cc:1: // Copyright 2013
The Chromium Authors. All rights reserved.
On 2014/09/29 15:47:18, boliu wrote:
> remove this and header from android_webview/android_webview.gyp

Done.

https://codereview.chromium.org/608263002/diff/1/android_webview/lib/main/aw_...
File android_webview/lib/main/aw_main_delegate.cc (left):

https://codereview.chromium.org/608263002/diff/1/android_webview/lib/main/aw_...
android_webview/lib/main/aw_main_delegate.cc:53: CommandLine* cl =
CommandLine::ForCurrentProcess();
On 2014/09/29 15:47:18, boliu wrote:
> Still need cl to be defined.

Done.

https://codereview.chromium.org/608263002/diff/1/android_webview/lib/main/aw_...
File android_webview/lib/main/aw_main_delegate.cc (right):

https://codereview.chromium.org/608263002/diff/1/android_webview/lib/main/aw_...
android_webview/lib/main/aw_main_delegate.cc:54:
BrowserViewRenderer::CalculateTileMemoryPolicy(use_zero_copy);
On 2014/09/29 15:47:18, boliu wrote:
> Clean this up and remove the use_zero_copy branch in that function.

Done.

https://codereview.chromium.org/608263002/diff/1/android_webview/native/aw_co...
File android_webview/native/aw_contents.cc (right):

https://codereview.chromium.org/608263002/diff/1/android_webview/native/aw_co...
android_webview/native/aw_contents.cc:14: #include
"android_webview/browser/gpu_memory_buffer_factory_impl.h"
On 2014/09/29 15:47:18, boliu wrote:
> remove this

Done.

[piman, 2014-09-29 18:33:01]

lgtm

[boliu, 2014-09-29 18:34:17]

lgtm

[reveman, 2014-09-29 18:34:24]

The CQ bit was checked by reveman@chromium.org

[commit-bot: I haz the power, 2014-09-29 18:34:55]

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

[commit-bot: I haz the power, 2014-09-29 19:12:39]

Committed patchset #4 (id:60001) as cbdcc84868b1582fc06f74bb2636546c610311b7

[commit-bot: I haz the power, 2014-09-29 19:13:29]

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