scheduler: Move WebFrameScheduler and WebScheduler into Blink

scheduler: Move WebFrameScheduler and WebScheduler into Blink

Move WebFrameScheduler and WebScheduler into Blink since there is no
need to expose these classes to outside components.

No functional changes.

BUG=696001, 701223
TBR=jochen@chromium.org,dalecurtis@chromium.org

Review-Url: https://codereview.chromium.org/2843503002
Cr-Commit-Position: refs/heads/master@{#467058}
Committed: https://chromium.googlesource.com/chromium/src/+/d80c41e37b7570607f6f877961f487f7f24ae12e

[Sami, 2017-04-24 15:41:44]

Description was changed from

==========
scheduler: Move WebFrameScheduler and WebScheduler into Blink

Move WebFrameScheduler and WebScheduler into Blink since there is no
need to expose these classes to outside components.

BUG=696001,701223
==========

to

==========
scheduler: Move WebFrameScheduler and WebScheduler into Blink

Move WebFrameScheduler and WebScheduler into Blink since there is no
need to expose these classes to outside components.

No functional changes.

BUG=696001,701223
==========

[Sami, 2017-04-24 15:44:52]

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

[Sami, 2017-04-24 15:44:56]

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

[Sami, 2017-04-24 15:45:09]

PTAL.

[commit-bot: I haz the power, 2017-04-24 15:45:31]

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-04-24 15:56:04]

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

[commit-bot: I haz the power, 2017-04-24 15:56:05]

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

[Sami, 2017-04-24 16:11:41]

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

[commit-bot: I haz the power, 2017-04-24 16:12:43]

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-04-24 16:24:26]

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

[commit-bot: I haz the power, 2017-04-24 16:24:27]

Dry run: 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...)
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[Sami, 2017-04-25 11:52:26]

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

[commit-bot: I haz the power, 2017-04-25 11:52:41]

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-04-25 12:03:55]

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

[commit-bot: I haz the power, 2017-04-25 12:03:56]

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

[Sami, 2017-04-25 13:45:43]

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

[commit-bot: I haz the power, 2017-04-25 13:46:01]

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-04-25 16:04:10]

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

[commit-bot: I haz the power, 2017-04-25 16:04:11]

Dry run: This issue passed the CQ dry run.

[haraken, 2017-04-25 17:45:39]

LGTM. Sorry about the review delay.

Our plan is to remove WebScheduler, right? What about WebViewScheduler? How is
it used?

[Sami, 2017-04-25 18:10:44]

On 2017/04/25 17:45:39, haraken wrote:
> LGTM. Sorry about the review delay.

Thanks, no problem!

> Our plan is to remove WebScheduler, right? What about WebViewScheduler? How is
> it used?

Yeah, I think we want to make RendererScheduler be the main interface to the
scheduler inside Blink and provide some really thin interface to content/.
WebViewScheduler should probably remain in some form because it's the next level
in the hierarchy (there can be several WebViews in one renderer).

[Sami, 2017-04-25 18:10:51]

The CQ bit was checked by skyostil@chromium.org

[commit-bot: I haz the power, 2017-04-25 18:11:31]

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-25 18:22:52]

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

[commit-bot: I haz the power, 2017-04-25 18:22:54]

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

[Sami, 2017-04-25 18:27:38]

Description was changed from

==========
scheduler: Move WebFrameScheduler and WebScheduler into Blink

Move WebFrameScheduler and WebScheduler into Blink since there is no
need to expose these classes to outside components.

No functional changes.

BUG=696001,701223
==========

to

==========
scheduler: Move WebFrameScheduler and WebScheduler into Blink

Move WebFrameScheduler and WebScheduler into Blink since there is no
need to expose these classes to outside components.

No functional changes.

BUG=696001,701223
TBR=jochen@chromium.org,dalecurtis@chromium.org
==========

[Sami, 2017-04-25 18:27:38]

skyostil@chromium.org changed reviewers:
+ dalecurtis@chromium.org, jochen@chromium.org

[Sami, 2017-04-25 18:28:01]

TBR'ing content/ and media/ owners since the changes are trivial.

[Sami, 2017-04-25 18:28:07]

The CQ bit was checked by skyostil@chromium.org

[commit-bot: I haz the power, 2017-04-25 18:28:56]

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-25 18:35:41]

CQ is committing da patch.

Bot data: {"patchset_id": 50001, "attempt_start_ts": 1493144887724430,
"parent_rev": "1995a63cc353629e27d14a58e0476f864acd5c25", "commit_rev":
"d80c41e37b7570607f6f877961f487f7f24ae12e"}

[commit-bot: I haz the power, 2017-04-25 18:35:52]

Description was changed from

==========
scheduler: Move WebFrameScheduler and WebScheduler into Blink

Move WebFrameScheduler and WebScheduler into Blink since there is no
need to expose these classes to outside components.

No functional changes.

BUG=696001,701223
TBR=jochen@chromium.org,dalecurtis@chromium.org
==========

to

==========
scheduler: Move WebFrameScheduler and WebScheduler into Blink

Move WebFrameScheduler and WebScheduler into Blink since there is no
need to expose these classes to outside components.

No functional changes.

BUG=696001,701223
TBR=jochen@chromium.org,dalecurtis@chromium.org

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

[commit-bot: I haz the power, 2017-04-25 18:35:54]

Committed patchset #4 (id:50001) as
https://chromium.googlesource.com/chromium/src/+/d80c41e37b7570607f6f877961f4...