ash: removes unnecessary includes and dep

ash: removes unnecessary includes and dep

BUG=670798
TEST=none
R=jamescook@chromium.org

Review-Url: https://codereview.chromium.org/2652763002
Cr-Commit-Position: refs/heads/master@{#445470}
Committed: https://chromium.googlesource.com/chromium/src/+/2ba6bc624e1a29ebe9fa66c36cc96e16224c858a

[James Cook, 2017-01-23 17:25:05]

LGTM

https://codereview.chromium.org/2652763002/diff/1/ash/BUILD.gn
File ash/BUILD.gn (left):

https://codereview.chromium.org/2652763002/diff/1/ash/BUILD.gn#oldcode862
ash/BUILD.gn:862: "//content/public/common:service_names",
Per offline discussion, can you ping ben and see if we still need this to be a
separate target? My understanding was that the reason it's separated out is to
allow ash to connect to things it shouldn't (e.g. content_browser).

[sky, 2017-01-23 17:27:41]

https://codereview.chromium.org/2652763002/diff/1/ash/BUILD.gn
File ash/BUILD.gn (left):

https://codereview.chromium.org/2652763002/diff/1/ash/BUILD.gn#oldcode862
ash/BUILD.gn:862: "//content/public/common:service_names",
On 2017/01/23 17:25:05, James Cook wrote:
> Per offline discussion, can you ping ben and see if we still need this to be a
> separate target? My understanding was that the reason it's separated out is to
> allow ash to connect to things it shouldn't (e.g. content_browser).

Will do.

[sky, 2017-01-23 17:27:45]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2017-01-23 17:28:10]

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, 2017-01-23 17:30:43]

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

[commit-bot: I haz the power, 2017-01-23 17:30:44]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[sky, 2017-01-23 19:28:41]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2017-01-23 19:29:15]

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, 2017-01-23 20:35:31]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485199721642960, "parent_rev":
"82e8eb7b5935197660fe3ef05c8ef05e9e5d8c26", "commit_rev":
"2ba6bc624e1a29ebe9fa66c36cc96e16224c858a"}

[commit-bot: I haz the power, 2017-01-23 20:36:06]

Description was changed from

==========
ash: removes unnecessary includes and dep

BUG=670798
TEST=none
R=jamescook@chromium.org
==========

to

==========
ash: removes unnecessary includes and dep

BUG=670798
TEST=none
R=jamescook@chromium.org

Review-Url: https://codereview.chromium.org/2652763002
Cr-Commit-Position: refs/heads/master@{#445470}
Committed:
https://chromium.googlesource.com/chromium/src/+/2ba6bc624e1a29ebe9fa66c36cc9...
==========

[commit-bot: I haz the power, 2017-01-23 20:36:07]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/2ba6bc624e1a29ebe9fa66c36cc9...