Fix missing headers about vpn service

Fix missing headers about vpn service

Decreased the number of missing header files by 1.

BUG=661774
Review-Url: https://codereview.chromium.org/2855533002
Cr-Commit-Position: refs/heads/master@{#468512}
Committed: https://chromium.googlesource.com/chromium/src/+/e4e376d3183be96ccb95b1e0fd8f4d648a54775b

[wychen, 2017-05-01 05:16:04]

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

[commit-bot: I haz the power, 2017-05-01 05:16:15]

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, 2017-05-01 05:57:56]

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

[commit-bot: I haz the power, 2017-05-01 05:57:56]

Dry run: This issue passed the CQ dry run.

[wychen, 2017-05-01 06:00:18]

wychen@chromium.org changed reviewers:
+ jam@chromium.org

[wychen, 2017-05-01 06:00:18]

PTAL

[jam, 2017-05-01 21:23:19]

https://codereview.chromium.org/2855533002/diff/1/chrome/browser/ui/ash/syste...
File chrome/browser/ui/ash/system_tray_client.cc (right):

https://codereview.chromium.org/2855533002/diff/1/chrome/browser/ui/ash/syste...
chrome/browser/ui/ash/system_tray_client.cc:55: #if defined(OS_CHROMEOS)
system_tray_client.cc is only in chromeos builds, so it doesn't make sense that
we have this check?

https://cs.chromium.org/chromium/src/chrome/browser/ui/BUILD.gn?rcl=77654fd14...

[wychen, 2017-05-01 23:17:01]

Description was changed from

==========
Fix missing headers about vpn service

Decreased the number of missing header files by 2.

BUG=661774
==========

to

==========
Fix missing headers about vpn service

Decreased the number of missing header files by 1.

BUG=661774
==========

[wychen, 2017-05-01 23:18:21]

https://codereview.chromium.org/2855533002/diff/1/chrome/browser/ui/ash/syste...
File chrome/browser/ui/ash/system_tray_client.cc (right):

https://codereview.chromium.org/2855533002/diff/1/chrome/browser/ui/ash/syste...
chrome/browser/ui/ash/system_tray_client.cc:55: #if defined(OS_CHROMEOS)
On 2017/05/01 21:23:19, jam wrote:
> system_tray_client.cc is only in chromeos builds, so it doesn't make sense
that
> we have this check?
> 
>
https://cs.chromium.org/chromium/src/chrome/browser/ui/BUILD.gn?rcl=77654fd14...

Done.

[jam, 2017-05-01 23:47:06]

lgtm

[wychen, 2017-05-01 23:49:08]

The CQ bit was checked by wychen@chromium.org

[commit-bot: I haz the power, 2017-05-01 23:49:57]

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-05-02 00:46:11]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493682548150200,
"parent_rev": "0079dbad35b7fb64939a1c5ddc992084cc5001b1", "commit_rev":
"e4e376d3183be96ccb95b1e0fd8f4d648a54775b"}

[commit-bot: I haz the power, 2017-05-02 00:46:54]

Description was changed from

==========
Fix missing headers about vpn service

Decreased the number of missing header files by 1.

BUG=661774
==========

to

==========
Fix missing headers about vpn service

Decreased the number of missing header files by 1.

BUG=661774

Review-Url: https://codereview.chromium.org/2855533002
Cr-Commit-Position: refs/heads/master@{#468512}
Committed:
https://chromium.googlesource.com/chromium/src/+/e4e376d3183be96ccb95b1e0fd8f...
==========

[commit-bot: I haz the power, 2017-05-02 00:46:55]

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