Merge gpu_switching_list into gpu_driver_bug_list.

Merge gpu_switching_list into gpu_driver_bug_list.

BUG=314901
TEST=gpu_unittests
R=bajones@chromium.org, kbr@chromium.org, piman@chromium.org, sievers@chromium.org

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

[Zhenyao Mo, 2013-11-05 02:05:17]

kbr, bajones: please review.

This is tested on a mac with dual gpus.

[Zhenyao Mo, 2013-11-05 02:06:11]

I noticed the dual gpu mechanism, including gpu_switching_manager, can be
further simplified, but I'll do that in another CL.

[Ken Russell (switch to Gerrit), 2013-11-05 02:31:55]

This is excellent. LGTM.

https://codereview.chromium.org/57633007/diff/190001/gpu/config/gpu_util_unit...
File gpu/config/gpu_util_unittest.cc (right):

https://codereview.chromium.org/57633007/diff/190001/gpu/config/gpu_util_unit...
gpu/config/gpu_util_unittest.cc:52: // zero feature.
Zero features.

https://codereview.chromium.org/57633007/diff/190001/gpu/config/gpu_util_unit...
gpu/config/gpu_util_unittest.cc:58: // One features.
One feature.

[Zhenyao Mo, 2013-11-05 02:45:15]

Daniel: gpu/ owner approval

[bajones, 2013-11-05 04:06:53]

This is a great cleanup! LGTM!

[Zhenyao Mo, 2013-11-05 19:24:07]

also add piman.

piman or sievers, whoever comes across this CL first, can you stamp as owner of
gpu/?

the changes there are two gyp related files, simply removing a few files.

[no sievers, 2013-11-05 19:41:23]

lgtm

https://codereview.chromium.org/57633007/diff/190001/content/browser/gpu/gpu_...
File content/browser/gpu/gpu_data_manager_impl_private.cc (right):

https://codereview.chromium.org/57633007/diff/190001/content/browser/gpu/gpu_...
content/browser/gpu/gpu_data_manager_impl_private.cc:312: if (option ==
switches::kGpuSwitchingOptionNameForceIntegrated) {
nit: else if

[piman, 2013-11-05 19:43:37]

LGTM. Nice cleanup.

[Zhenyao Mo, 2013-11-05 19:45:01]

https://codereview.chromium.org/57633007/diff/190001/content/browser/gpu/gpu_...
File content/browser/gpu/gpu_data_manager_impl_private.cc (right):

https://codereview.chromium.org/57633007/diff/190001/content/browser/gpu/gpu_...
content/browser/gpu/gpu_data_manager_impl_private.cc:312: if (option ==
switches::kGpuSwitchingOptionNameForceIntegrated) {
On 2013/11/05 19:41:23, sievers wrote:
> nit: else if

Done.

[commit-bot: I haz the power, 2013-11-05 20:05:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/zmo@chromium.org/57633007/420001

[commit-bot: I haz the power, 2013-11-05 20:57:32]

Sorry for I got bad news for ya.
Compile failed with a clobber build on android_dbg.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-11-05 22:17:24]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/zmo@chromium.org/57633007/690001

[Zhenyao Mo, 2013-11-05 23:53:31]

Committed patchset #6 manually as r233143 (presubmit successful).