Move SharedWorkerRepositoryClientImpl from web/ to core/exported/.

Move SharedWorkerRepositoryClientImpl from web/ to core/exported/.

SharedWorkerRepositoryClientImpl implements classes defined in public/web/ 
(WebSharedWorkerConnectListener, WebSharedWorkerRepositoryClient) so it is being 
moved to core/exported.

SharedWorkerRepositoryClientImpl.cpp did include WebLocalFrameImpl but luckily
enough didn't use it so we could just break this dependency.


BUG=708879
Review-Url: https://codereview.chromium.org/2833693002
Cr-Commit-Position: refs/heads/master@{#468274}
Committed: https://chromium.googlesource.com/chromium/src/+/2ebe3671d78db19aa97d2c4950069c1ff837c4a5

[slangley, 2017-04-20 04:45:45]

Description was changed from

==========
WebLocalFrameImpl was not being used.


git cl format


Move SharedWorkerRepositoryClientImpl from web/ to core/exported/.

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

to

==========
WebLocalFrameImpl was not being used.


git cl format


Move SharedWorkerRepositoryClientImpl from web/ to core/exported/.

BUG=708879
==========

[slangley, 2017-04-20 04:49:39]

Description was changed from

==========
WebLocalFrameImpl was not being used.


git cl format


Move SharedWorkerRepositoryClientImpl from web/ to core/exported/.

BUG=708879
==========

to

==========
Move SharedWorkerRepositoryClientImpl from web/ to core/exported/.

SharedWorkerRepositoryClientImpl implements classes defined in public/web/ 
(WebSharedWorkerConnectListener, WebSharedWorkerRepositoryClient) so it is being

moved to core/exported.

SharedWorkerRepositoryClientImpl.cpp did include WebLocalFrameImpl but luckily
enough didn't use it so we could just break this dependency.


BUG=708879
==========

[slangley, 2017-04-20 04:49:39]

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

[joelhockey, 2017-04-20 04:53:57]

lgtm

[slangley, 2017-04-20 04:58:32]

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

[slangley, 2017-04-20 04:58:32]

haraken@ - ptal

[haraken, 2017-04-20 07:38:26]

LGTM

https://codereview.chromium.org/2833693002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/exported/BUILD.gn (right):

https://codereview.chromium.org/2833693002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/exported/BUILD.gn:10:
"WebAssociatedURLLoaderImpl.cpp",

As commented in other CLs, I'd prefer moving these files to core/workers/.

[slangley, 2017-04-26 06:15:14]

The CQ bit was checked by slangley@chromium.org

[commit-bot: I haz the power, 2017-04-26 06:15:33]

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-04-26 06:18:32]

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

[commit-bot: I haz the power, 2017-04-26 06:18:33]

Try jobs failed on following builders:
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[slangley, 2017-04-26 07:19:16]

The CQ bit was checked by slangley@chromium.org

[slangley, 2017-04-26 07:19:17]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, joelhockey@chromium.org
Link to the patchset: https://codereview.chromium.org/2833693002/#ps20001
(title: "Rebase")

[commit-bot: I haz the power, 2017-04-26 07:19:38]

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-04-26 07:29:05]

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

[commit-bot: I haz the power, 2017-04-26 07:29:06]

Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[slangley, 2017-04-26 23:54:26]

The CQ bit was checked by slangley@chromium.org

[slangley, 2017-04-26 23:54:26]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, joelhockey@chromium.org
Link to the patchset: https://codereview.chromium.org/2833693002/#ps60001
(title: "Fix merge issue.")

[commit-bot: I haz the power, 2017-04-26 23:55:30]

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-04-27 01:34:56]

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

[commit-bot: I haz the power, 2017-04-27 01:34:57]

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

[slangley, 2017-04-28 06:26:35]

The CQ bit was checked by slangley@chromium.org

[slangley, 2017-04-28 06:26:35]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, joelhockey@chromium.org
Link to the patchset: https://codereview.chromium.org/2833693002/#ps80001
(title: "Make SharedWorkerRepositoryClient CORE_EXPORTed for windows builds.")

[commit-bot: I haz the power, 2017-04-28 06:26:52]

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-04-28 07:36:49]

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

[commit-bot: I haz the power, 2017-04-28 07:36:50]

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

[slangley, 2017-05-01 01:29:45]

The CQ bit was checked by slangley@chromium.org

[slangley, 2017-05-01 01:29:45]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, joelhockey@chromium.org
Link to the patchset: https://codereview.chromium.org/2833693002/#ps100001
(title: "Remove #include of WebFrameClient that was not being used, and caused
linker errors.")

[commit-bot: I haz the power, 2017-05-01 01:29:58]

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 03:20:19]

CQ is committing da patch.

Bot data: {"patchset_id": 100001, "attempt_start_ts": 1493602185313140,
"parent_rev": "7f022a2ccb293c5b31446f3f5af8be79f46555ba", "commit_rev":
"2ebe3671d78db19aa97d2c4950069c1ff837c4a5"}

[commit-bot: I haz the power, 2017-05-01 03:21:03]

Description was changed from

==========
Move SharedWorkerRepositoryClientImpl from web/ to core/exported/.

SharedWorkerRepositoryClientImpl implements classes defined in public/web/ 
(WebSharedWorkerConnectListener, WebSharedWorkerRepositoryClient) so it is being

moved to core/exported.

SharedWorkerRepositoryClientImpl.cpp did include WebLocalFrameImpl but luckily
enough didn't use it so we could just break this dependency.


BUG=708879
==========

to

==========
Move SharedWorkerRepositoryClientImpl from web/ to core/exported/.

SharedWorkerRepositoryClientImpl implements classes defined in public/web/ 
(WebSharedWorkerConnectListener, WebSharedWorkerRepositoryClient) so it is being

moved to core/exported.

SharedWorkerRepositoryClientImpl.cpp did include WebLocalFrameImpl but luckily
enough didn't use it so we could just break this dependency.


BUG=708879

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

[commit-bot: I haz the power, 2017-05-01 03:21:07]

Committed patchset #6 (id:100001) as
https://chromium.googlesource.com/chromium/src/+/2ebe3671d78db19aa97d2c495006...