Move WebEntities to platform/

Move WebEntities to platform/

Move the WebEntities class out of web/ and into platform/, since it has
no other dependencies inside web/.

BUG=712963
Review-Url: https://codereview.chromium.org/2859123002
Cr-Commit-Position: refs/heads/master@{#469288}
Committed: https://chromium.googlesource.com/chromium/src/+/265b2b33274e914c1a54d292f3164dfd96840b4e

[sashab, 2017-05-04 04:40:28]

The CQ bit was checked by sashab@chromium.org

[sashab, 2017-05-04 04:40:31]

The CQ bit was unchecked by sashab@chromium.org

[sashab, 2017-05-04 04:40:33]

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

[commit-bot: I haz the power, 2017-05-04 04:40:37]

Dry run: CQ is trying da patch.

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

[sashab, 2017-05-04 04:41:01]

sashab@chromium.org changed reviewers:
+ slangley@chromium.org

[sashab, 2017-05-04 04:41:01]

Found as a leaf node on the new graph:
https://drive.google.com/drive/u/1/folders/0B_KwtdC2J1Q6b3ZYNjVtTFIzdm8

:D

Happy to debate platform/, but I thought since it has no dependencies in core/,
why not.

[slangley, 2017-05-04 04:55:43]

On 2017/05/04 at 04:41:01, sashab wrote:
> Found as a leaf node on the new graph:
> https://drive.google.com/drive/u/1/folders/0B_KwtdC2J1Q6b3ZYNjVtTFIzdm8
> 
> :D
> 
> Happy to debate platform/, but I thought since it has no dependencies in
core/, why not.

LGTM

I doubt that platform is the right place as well (at least from my understanding
of the intent of the layering.) :) Perhaps core/editing?

[sashab, 2017-05-04 04:56:41]

sashab@chromium.org changed reviewers:
+ haraken@chromium.org

[sashab, 2017-05-04 04:56:42]

haraken ptal, wdyt of new file location?

[haraken, 2017-05-04 05:17:40]

LGTM

https://codereview.chromium.org/2859123002/diff/1/third_party/WebKit/Source/p...
File third_party/WebKit/Source/platform/text/WebEntities.h (right):

https://codereview.chromium.org/2859123002/diff/1/third_party/WebKit/Source/p...
third_party/WebKit/Source/platform/text/WebEntities.h:40: // FIXME: This class
is wrong and needs to be removed!

I'm curious about this comment but it should be a separate issue from this CL :)

[sashab, 2017-05-04 05:20:15]

The CQ bit was unchecked by sashab@chromium.org

[sashab, 2017-05-04 05:20:17]

The CQ bit was checked by sashab@chromium.org

[commit-bot: I haz the power, 2017-05-04 05:20:28]

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-04 06:54:53]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493875217935680, "parent_rev":
"45829114399facebf06f2d4abfedb08f6ac24f0f", "commit_rev":
"265b2b33274e914c1a54d292f3164dfd96840b4e"}

[commit-bot: I haz the power, 2017-05-04 06:55:03]

Description was changed from

==========
Move WebEntities to platform/

Move the WebEntities class out of web/ and into platform/, since it has
no other dependencies inside web/.

BUG=712963
==========

to

==========
Move WebEntities to platform/

Move the WebEntities class out of web/ and into platform/, since it has
no other dependencies inside web/.

BUG=712963

Review-Url: https://codereview.chromium.org/2859123002
Cr-Commit-Position: refs/heads/master@{#469288}
Committed:
https://chromium.googlesource.com/chromium/src/+/265b2b33274e914c1a54d292f316...
==========

[commit-bot: I haz the power, 2017-05-04 06:55:04]

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