Finish ../platform => public/platform refactor

Finish ../platform => public/platform refactor

In https://codereview.chromium.org/2887233003, I changed most of the header
includes in the third_party/WebKit/public/web directory that started with
"../platform/" to instead start with "public/platform/" in order to make
these header includes valid under a DEPS rule that's flagging new includes
of the form "../platform/" as errors. I did not change these headers in that CL
because some build targets were not building properly. I've determined that the
problem was that these targets are missing a dependency in their BUILD.gn files.
This CL adds this dependency to those build files and finishes the codemod.

Note: I have not run git cl format to put the headers in the correct order
because I ran into problems with the original CL due to this breaking fragile
code.

BUG=724264
Review-Url: https://codereview.chromium.org/2892703002
Cr-Commit-Position: refs/heads/master@{#473333}
Committed: https://chromium.googlesource.com/chromium/src/+/38a14eb81c041f7ae7bff36226318bca3d161e79

[rlanday, 2017-05-18 07:07:54]

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

[commit-bot: I haz the power, 2017-05-18 07:08:21]

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-18 07:27:31]

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

[commit-bot: I haz the power, 2017-05-18 07:27:32]

Dry run: Try jobs failed on following builders:
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[rlanday, 2017-05-18 17:34:20]

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

[commit-bot: I haz the power, 2017-05-18 17:35:21]

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-18 17:53:43]

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

[commit-bot: I haz the power, 2017-05-18 17:53:44]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[rlanday, 2017-05-18 18:16:13]

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

[commit-bot: I haz the power, 2017-05-18 18:17:18]

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-18 18:23:09]

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

[commit-bot: I haz the power, 2017-05-18 18:23:11]

Dry run: Try jobs failed on following builders:
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[rlanday, 2017-05-18 18:47:42]

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

[commit-bot: I haz the power, 2017-05-18 18:48:45]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[rlanday, 2017-05-18 20:54:51]

Description was changed from

==========
Finish ../platform => public/platform refactor

BUG=
==========

to

==========
Finish ../platform => public/platform refactor

In https://codereview.chromium.org/2887233003, I did a bulk codemod from
../platform/ to public/platform in the third_party/WebKit/public/web/
directory to make these header includes valid under a DEPS rule that flags
../platform/ includes as errors. I reverted these headers because some
build targets were not building properly. I determined that the problem
was that these targets are missing a dependency in their BUILD.gn files.
This CL adds this dependency to those build files and finishes the codemod.
==========

[rlanday, 2017-05-18 20:54:51]

rlanday@chromium.org changed reviewers:
+ aelias@chromium.org, jochen@chromium.org

[rlanday, 2017-05-18 20:54:51]

rlanday@chromium.org changed required reviewers:
+ jochen@chromium.org

[rlanday, 2017-05-18 20:57:14]

Description was changed from

==========
Finish ../platform => public/platform refactor

In https://codereview.chromium.org/2887233003, I did a bulk codemod from
../platform/ to public/platform in the third_party/WebKit/public/web/
directory to make these header includes valid under a DEPS rule that flags
../platform/ includes as errors. I reverted these headers because some
build targets were not building properly. I determined that the problem
was that these targets are missing a dependency in their BUILD.gn files.
This CL adds this dependency to those build files and finishes the codemod.
==========

to

==========
Finish ../platform => public/platform refactor

In https://codereview.chromium.org/2887233003, I changed most of the header
includes in the third_party/WebKit/public/web directory that started with
"../platform/" to instead start with "public/platform/" in order to make
these header includes valid under a DEPS rule that's flagging new includes
of the form "../platform/" as errors. I did not change these headers in that CL
because some build targets were not building properly. I've determined that the
problem was that these targets are missing a dependency in their BUILD.gn files.
This CL adds this dependency to those build files and finishes the codemod.
==========

[rlanday, 2017-05-18 20:58:57]

Description was changed from

==========
Finish ../platform => public/platform refactor

In https://codereview.chromium.org/2887233003, I changed most of the header
includes in the third_party/WebKit/public/web directory that started with
"../platform/" to instead start with "public/platform/" in order to make
these header includes valid under a DEPS rule that's flagging new includes
of the form "../platform/" as errors. I did not change these headers in that CL
because some build targets were not building properly. I've determined that the
problem was that these targets are missing a dependency in their BUILD.gn files.
This CL adds this dependency to those build files and finishes the codemod.
==========

to

==========
Finish ../platform => public/platform refactor

In https://codereview.chromium.org/2887233003, I changed most of the header
includes in the third_party/WebKit/public/web directory that started with
"../platform/" to instead start with "public/platform/" in order to make
these header includes valid under a DEPS rule that's flagging new includes
of the form "../platform/" as errors. I did not change these headers in that CL
because some build targets were not building properly. I've determined that the
problem was that these targets are missing a dependency in their BUILD.gn files.
This CL adds this dependency to those build files and finishes the codemod.

Note: I have not run git cl format to put the headers in the correct order
because I ran into problems with the original CL due to this breaking fragile
code.
==========

[aelias_OOO_until_Jul13, 2017-05-18 21:09:16]

Description was changed from

==========
Finish ../platform => public/platform refactor

In https://codereview.chromium.org/2887233003, I changed most of the header
includes in the third_party/WebKit/public/web directory that started with
"../platform/" to instead start with "public/platform/" in order to make
these header includes valid under a DEPS rule that's flagging new includes
of the form "../platform/" as errors. I did not change these headers in that CL
because some build targets were not building properly. I've determined that the
problem was that these targets are missing a dependency in their BUILD.gn files.
This CL adds this dependency to those build files and finishes the codemod.

Note: I have not run git cl format to put the headers in the correct order
because I ran into problems with the original CL due to this breaking fragile
code.
==========

to

==========
Finish ../platform => public/platform refactor

In https://codereview.chromium.org/2887233003, I changed most of the header
includes in the third_party/WebKit/public/web directory that started with
"../platform/" to instead start with "public/platform/" in order to make
these header includes valid under a DEPS rule that's flagging new includes
of the form "../platform/" as errors. I did not change these headers in that CL
because some build targets were not building properly. I've determined that the
problem was that these targets are missing a dependency in their BUILD.gn files.
This CL adds this dependency to those build files and finishes the codemod.

Note: I have not run git cl format to put the headers in the correct order
because I ran into problems with the original CL due to this breaking fragile
code.

BUG=724264
==========

[commit-bot: I haz the power, 2017-05-18 22:15:04]

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

[commit-bot: I haz the power, 2017-05-18 22:15:05]

Dry run: 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_...)

[jochen (gone - plz use gerrit), 2017-05-19 13:35:16]

lgtm

[rlanday, 2017-05-19 20:06:31]

The CQ bit was checked by rlanday@chromium.org

[commit-bot: I haz the power, 2017-05-19 20:07:35]

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-19 21:38:06]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1495224391883520,
"parent_rev": "640dc5161179cd9058dd3c97788880a8ac4c7f4d", "commit_rev":
"38a14eb81c041f7ae7bff36226318bca3d161e79"}

[commit-bot: I haz the power, 2017-05-19 21:38:18]

Description was changed from

==========
Finish ../platform => public/platform refactor

In https://codereview.chromium.org/2887233003, I changed most of the header
includes in the third_party/WebKit/public/web directory that started with
"../platform/" to instead start with "public/platform/" in order to make
these header includes valid under a DEPS rule that's flagging new includes
of the form "../platform/" as errors. I did not change these headers in that CL
because some build targets were not building properly. I've determined that the
problem was that these targets are missing a dependency in their BUILD.gn files.
This CL adds this dependency to those build files and finishes the codemod.

Note: I have not run git cl format to put the headers in the correct order
because I ran into problems with the original CL due to this breaking fragile
code.

BUG=724264
==========

to

==========
Finish ../platform => public/platform refactor

In https://codereview.chromium.org/2887233003, I changed most of the header
includes in the third_party/WebKit/public/web directory that started with
"../platform/" to instead start with "public/platform/" in order to make
these header includes valid under a DEPS rule that's flagging new includes
of the form "../platform/" as errors. I did not change these headers in that CL
because some build targets were not building properly. I've determined that the
problem was that these targets are missing a dependency in their BUILD.gn files.
This CL adds this dependency to those build files and finishes the codemod.

Note: I have not run git cl format to put the headers in the correct order
because I ran into problems with the original CL due to this breaking fragile
code.

BUG=724264

Review-Url: https://codereview.chromium.org/2892703002
Cr-Commit-Position: refs/heads/master@{#473333}
Committed:
https://chromium.googlesource.com/chromium/src/+/38a14eb81c041f7ae7bff3622631...
==========

[commit-bot: I haz the power, 2017-05-19 21:38:20]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/38a14eb81c041f7ae7bff3622631...