Move GPU device/driver info related code from content to gpu.

Move GPU device/driver info related code from content to gpu.

Try to reland r201380 with build fix.

This has been suggested by gman, and agreed by kbr and jam, for the following reasons:

1) These are gpu related code, and are independent of content / browser, so putting them under gpu/ is the right thing to do conceptually.

2) This enables us to set up tests in various places with the correct blacklisting/driver_bug_workarounds information. Otherwise, for the moment, gpu/ has no visibility into content/ side, so we have to duplicate the driver_bug_workarounds code and hardwire them for testing purpose. This is going to cause a lot of bugs in the future, as we have the two pieces of code for the same thing (one for chrome and one for testing) and people will easily forget to update one or the other.

As for this patch, I didn't change the logic, and try to minimize the refactoring. All improvements enabled by this relocation will be done in follow-up CLs.

BUG=230477
TEST=tree
TBR=apatrick@chromium.org, joi@chromium.org, kbr@chromium.org, piman@chromium.org, sky@chromium.org

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

[Zhenyao Mo, 2013-05-22 19:39:03]

Try to reland this CL.

The patch set 1 is exactly the same as patch set 6 in
https://codereview.chromium.org/15385003/, which has been reviewed by piman,
gman, kbr, joi already. So if you guys are busy, you don't have to look at this
CL again (I'll TBR to you).

There are a few changes that I made since to fix build failures on a few bots
(unfortunately our default trybot set doesn't cover these failures).

1) ash changes (basically a path rebase): sky, owner review please.
2) remove the dead file path in command_buffer_service.gypi
3) fix the win component build dependency suggested by apatrick.  (this is the
minimal fix.  I will fix it properly in gyp in a followup patch).

Al, please review.

[piman, 2013-05-22 19:46:21]

lgtm

[Zhenyao Mo, 2013-05-22 19:48:42]

sky: ash/ owner review please

[apatrick_chromium, 2013-05-22 19:52:03]

https://codereview.chromium.org/15745014/diff/42001/content/browser/gamepad/g...
File content/browser/gamepad/gamepad_platform_data_fetcher_win.cc (right):

https://codereview.chromium.org/15745014/diff/42001/content/browser/gamepad/g...
content/browser/gamepad/gamepad_platform_data_fetcher_win.cc:23: 18 #pragma
comment(lib, "dxguid.lib")
Delete "18"

[Ken Russell (switch to Gerrit), 2013-05-22 19:53:35]

LGTM with apatrick's issue addressed.

[Zhenyao Mo, 2013-05-22 20:01:39]

Fixed.  Al: please have another look.

https://codereview.chromium.org/15745014/diff/42001/content/browser/gamepad/g...
File content/browser/gamepad/gamepad_platform_data_fetcher_win.cc (right):

https://codereview.chromium.org/15745014/diff/42001/content/browser/gamepad/g...
content/browser/gamepad/gamepad_platform_data_fetcher_win.cc:23: 18 #pragma
comment(lib, "dxguid.lib")
On 2013/05/22 19:52:03, apatrick_chromium wrote:
> Delete "18"

Done.

[apatrick_chromium, 2013-05-22 20:15:43]

lgtm

[sky, 2013-05-22 21:14:26]

LGTM

[Jói, 2013-05-22 21:17:28]

LGTM

On Wed, May 22, 2013 at 2:14 PM,  <sky@chromium.org> wrote:
> LGTM
>
> https://codereview.chromium.org/15745014/

[commit-bot: I haz the power, 2013-05-22 21:26:43]

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

[commit-bot: I haz the power, 2013-05-22 21:27:30]

Failed to apply patch for gpu/config/dx_diag_node.cc:
While running patch -p0 --forward --force --no-backup-if-mismatch;
  can't find file to patch at input line 5
  Perhaps you used the wrong -p or --strip option?
  The text leading up to this was:
  --------------------------
  |Index: gpu/config/dx_diag_node.cc
  |===================================================================
  |--- gpu/config/dx_diag_node.cc	(revision 0)
  |+++ gpu/config/dx_diag_node.cc	(working copy)
  --------------------------
  No file to patch.  Skipping patch.
  1 out of 1 hunk ignored

Patch:   N   gpu/config/dx_diag_node.cc
Index: gpu/config/dx_diag_node.cc
===================================================================
--- gpu/config/dx_diag_node.cc	(revision 0)
+++ gpu/config/dx_diag_node.cc	(working copy)
@@ -2,12 +2,12 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "content/public/common/dx_diag_node.h"
+#include "gpu/config/dx_diag_node.h"
 
-namespace content {
+namespace gpu {
 
 DxDiagNode::DxDiagNode() {}
 
 DxDiagNode::~DxDiagNode() {}
 
-}  // namespace content
+}  // namespace gpu

[Zhenyao Mo, 2013-05-23 20:05:05]

Committed patchset #3 manually as r201875.