Android: Remove some devtools code from the build.

Android: Remove some devtools code from the build.

BUG=159847
Committed: https://crrev.com/42abef62e59d966e30f076eff4cd192c75e2d731
Cr-Commit-Position: refs/heads/master@{#301579}

[Lei Zhang, 2014-10-28 02:42:44]

thestig@chromium.org changed reviewers:
+ dgozman@chromium.org

[dgozman, 2014-10-28 05:27:07]

devtools_targets_ui.* and devtools_ui_bindings.* are not compiled on Android
(see chrome_debugger.gypi and chrome/browser/devtools/BUILD.gn), so changing
them is unnecessary.

https://codereview.chromium.org/652033004/diff/1/chrome/browser/ui/webui/chro...
File chrome/browser/ui/webui/chrome_web_ui_controller_factory.cc (right):

https://codereview.chromium.org/652033004/diff/1/chrome/browser/ui/webui/chro...
chrome/browser/ui/webui/chrome_web_ui_controller_factory.cc:463: // (Shared
Workers, extensions, etc) aren't supported.
This is not true: we support remote Shared/Service Workers, and there are no
extensions at all on mobile.

[Lei Zhang, 2014-10-28 05:52:43]

PTAL at patch set 3.

https://codereview.chromium.org/652033004/diff/1/chrome/browser/ui/webui/chro...
File chrome/browser/ui/webui/chrome_web_ui_controller_factory.cc (right):

https://codereview.chromium.org/652033004/diff/1/chrome/browser/ui/webui/chro...
chrome/browser/ui/webui/chrome_web_ui_controller_factory.cc:463: // (Shared
Workers, extensions, etc) aren't supported.
On 2014/10/28 05:27:07, dgozman wrote:
> This is not true: we support remote Shared/Service Workers, and there are no
> extensions at all on mobile.

I simply cut + pasted the comment from above, but it is also 3 years old, and
probably out of date. I rephrased the comment based on your feedback.

https://codereview.chromium.org/652033004/diff/40001/chrome/browser/ui/BUILD.gn
File chrome/browser/ui/BUILD.gn (right):

https://codereview.chromium.org/652033004/diff/40001/chrome/browser/ui/BUILD....
chrome/browser/ui/BUILD.gn:129: deps += [ "//device/bluetooth" ]
This is just to sync the GN build with GYP.

[dgozman, 2014-10-28 06:03:52]

lgtm

[Lei Zhang, 2014-10-28 06:05:42]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2014-10-28 06:06:04]

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

[commit-bot: I haz the power, 2014-10-28 06:51:14]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2014-10-28 06:52:04]

Patchset 3 (id:??) landed as
https://crrev.com/42abef62e59d966e30f076eff4cd192c75e2d731
Cr-Commit-Position: refs/heads/master@{#301579}