gpu: Delete unnecessary code.

gpu: Delete unnecessary code.

gpu::GetNativeGpuMemoryBufferType() never returns SHARED_MEMORY_BUFFER.
So there's no reason to have special code that handles this case in
IsNativeGpuMemoryBufferFactoryConfigurationSupported() (which is
implemented in both content and mus). So delete this code, and update
gpu::IsNativeGpuMemoryBufferConfigurationSupported() to check that the
native type is never SHARED_MEMORY_BUFFER or EMPTY_BUFFER.

BUG=643746
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Committed: https://crrev.com/2b91090ab9e62ebf9d3577d21c286b307a3e45f5
Cr-Commit-Position: refs/heads/master@{#416804}

[sadrul, 2016-09-06 18:55:08]

Description was changed from

==========
gpu: Delete unnecessary code.

gpu::GetNativeGpuMemoryBufferType() never returns SHARED_MEMORY_BUFFER.
So there's no reason to have special code that handles this case in
IsNativeGpuMemoryBufferFactoryConfigurationSupported() (which is
implemented in both content and mus). So delete this code, and update
gpu::IsNativeGpuMemoryBufferConfigurationSupported() to check that the
native type is never SHARED_MEMORY_BUFFER or EMPTY_BUFFER.

BUG=643746
==========

to

==========
gpu: Delete unnecessary code.

gpu::GetNativeGpuMemoryBufferType() never returns SHARED_MEMORY_BUFFER.
So there's no reason to have special code that handles this case in
IsNativeGpuMemoryBufferFactoryConfigurationSupported() (which is
implemented in both content and mus). So delete this code, and update
gpu::IsNativeGpuMemoryBufferConfigurationSupported() to check that the
native type is never SHARED_MEMORY_BUFFER or EMPTY_BUFFER.

BUG=643746
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[sadrul, 2016-09-06 18:56:59]

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

[commit-bot: I haz the power, 2016-09-06 18:57:27]

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

[sadrul, 2016-09-06 20:28:53]

sadrul@chromium.org changed reviewers:
+ piman@chromium.org

[commit-bot: I haz the power, 2016-09-06 20:34:45]

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

[commit-bot: I haz the power, 2016-09-06 20:34:46]

Dry run: Try jobs failed on following builders:
  linux_optional_gpu_tests_rel on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_optional_...)

[piman, 2016-09-06 20:45:27]

lgtm

[sadrul, 2016-09-06 22:06:01]

sadrul@chromium.org changed reviewers:
+ sky@chromium.org, tsepez@chromium.org

[sadrul, 2016-09-06 22:06:02]

+tsepez@ for gpu/ipc/common

+sky@ for services/ui

[Tom Sepez, 2016-09-06 22:09:20]

lgtm

[sky, 2016-09-06 23:25:14]

LGTM

[sadrul, 2016-09-07 00:08:18]

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

[commit-bot: I haz the power, 2016-09-07 00:08:45]

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-09-07 00:28:45]

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

[commit-bot: I haz the power, 2016-09-07 00:28:45]

Dry run: This issue passed the CQ dry run.

[sadrul, 2016-09-07 00:47:31]

The CQ bit was checked by sadrul@chromium.org

[commit-bot: I haz the power, 2016-09-07 00:47:56]

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-09-07 00:52:14]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-07 00:53:40]

Description was changed from

==========
gpu: Delete unnecessary code.

gpu::GetNativeGpuMemoryBufferType() never returns SHARED_MEMORY_BUFFER.
So there's no reason to have special code that handles this case in
IsNativeGpuMemoryBufferFactoryConfigurationSupported() (which is
implemented in both content and mus). So delete this code, and update
gpu::IsNativeGpuMemoryBufferConfigurationSupported() to check that the
native type is never SHARED_MEMORY_BUFFER or EMPTY_BUFFER.

BUG=643746
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
gpu: Delete unnecessary code.

gpu::GetNativeGpuMemoryBufferType() never returns SHARED_MEMORY_BUFFER.
So there's no reason to have special code that handles this case in
IsNativeGpuMemoryBufferFactoryConfigurationSupported() (which is
implemented in both content and mus). So delete this code, and update
gpu::IsNativeGpuMemoryBufferConfigurationSupported() to check that the
native type is never SHARED_MEMORY_BUFFER or EMPTY_BUFFER.

BUG=643746
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Committed: https://crrev.com/2b91090ab9e62ebf9d3577d21c286b307a3e45f5
Cr-Commit-Position: refs/heads/master@{#416804}
==========

[commit-bot: I haz the power, 2016-09-07 00:53:42]

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