Unexport resource_controller.h

Unexport resource_controller.h

This is a follow up for https://codereview.chromium.org/2535723005/
suggested by mmenke@.

ResourceController is no longer used by any file outside of
content/browser/. Move it to content/browser/loader/.

BUG=646277
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/3b3ee8e99fca8912ad38f2bdc6137189c5043790
Cr-Commit-Position: refs/heads/master@{#436967}

[tyoshino (SeeGerritForStatus), 2016-12-07 08:33:48]

Description was changed from

==========
Unexport resource_controller.h

This is a follow up for https://codereview.chromium.org/2535723005/

ResourceController is no longer used by any file outside of
content/browser/.

BUG=646277
==========

to

==========
Unexport resource_controller.h

This is a follow up for https://codereview.chromium.org/2535723005/

ResourceController is no longer used by any file outside of
content/browser/.

BUG=646277
==========

[tyoshino (SeeGerritForStatus), 2016-12-07 08:33:48]

tyoshino@chromium.org changed reviewers:
+ jochen@chromium.org

[tyoshino (SeeGerritForStatus), 2016-12-07 08:34:35]

tyoshino@chromium.org changed reviewers:
+ mmenke@chromium.org

[tyoshino (SeeGerritForStatus), 2016-12-07 08:34:39]

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

[commit-bot: I haz the power, 2016-12-07 08:34:59]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tyoshino (SeeGerritForStatus), 2016-12-07 08:36:07]

Description was changed from

==========
Unexport resource_controller.h

This is a follow up for https://codereview.chromium.org/2535723005/

ResourceController is no longer used by any file outside of
content/browser/.

BUG=646277
==========

to

==========
Unexport resource_controller.h

This is a follow up for https://codereview.chromium.org/2535723005/
suggested by mmenke@.

ResourceController is no longer used by any file outside of
content/browser/. Move it to content/browser/loader/.

BUG=646277
==========

[commit-bot: I haz the power, 2016-12-07 08:38:50]

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

[commit-bot: I haz the power, 2016-12-07 08:38:50]

Dry run: Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)
  android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)

[tyoshino (SeeGerritForStatus), 2016-12-07 08:45:53]

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

[commit-bot: I haz the power, 2016-12-07 08:46:13]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tyoshino (SeeGerritForStatus), 2016-12-07 08:46:17]

Oops. Forgot to git add. PTAL at ps2.

[commit-bot: I haz the power, 2016-12-07 08:58:46]

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

[commit-bot: I haz the power, 2016-12-07 08:58:48]

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_...)

[tyoshino (SeeGerritForStatus), 2016-12-07 09:06:50]

Description was changed from

==========
Unexport resource_controller.h

This is a follow up for https://codereview.chromium.org/2535723005/
suggested by mmenke@.

ResourceController is no longer used by any file outside of
content/browser/. Move it to content/browser/loader/.

BUG=646277
==========

to

==========
Unexport resource_controller.h

This is a follow up for https://codereview.chromium.org/2535723005/
suggested by mmenke@.

ResourceController is no longer used by any file outside of
content/browser/. Move it to content/browser/loader/.

BUG=646277
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[tyoshino (SeeGerritForStatus), 2016-12-07 09:09:13]

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

[commit-bot: I haz the power, 2016-12-07 09:09:33]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tyoshino (SeeGerritForStatus), 2016-12-07 09:09:42]

On 2016/12/07 08:46:17, tyoshino wrote:
> Oops. Forgot to git add. PTAL at ps2.

Sorry for spamming. Fixed browser_tests and content_unittests build.

[commit-bot: I haz the power, 2016-12-07 10:53:59]

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

[commit-bot: I haz the power, 2016-12-07 10:54:00]

Dry run: This issue passed the CQ dry run.

[jochen (gone - plz use gerrit), 2016-12-07 15:37:42]

lgtm

[mmenke, 2016-12-07 15:52:23]

On 2016/12/07 15:37:42, jochen wrote:
> lgtm

LGTM, thanks for doing this!

[mmenke, 2016-12-07 15:58:48]

On 2016/12/07 15:52:23, mmenke wrote:
> On 2016/12/07 15:37:42, jochen wrote:
> > lgtm
> 
> LGTM, thanks for doing this!

Going to go ahead and CQ this - as I mentioned, I'm doing some work here, and
don't want our CLs to fight with each other.

[mmenke, 2016-12-07 15:58:53]

The CQ bit was checked by mmenke@chromium.org

[commit-bot: I haz the power, 2016-12-07 15:59:11]

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, 2016-12-07 16:04:15]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1481126333399110,
"parent_rev": "8a31a33ec6f65ec98a87b1492f38d56596861a24", "commit_rev":
"6dd44f06aab1cf9cbdf5c632130b6cec339f6ae1"}

[commit-bot: I haz the power, 2016-12-07 16:04:43]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-12-07 16:06:47]

Description was changed from

==========
Unexport resource_controller.h

This is a follow up for https://codereview.chromium.org/2535723005/
suggested by mmenke@.

ResourceController is no longer used by any file outside of
content/browser/. Move it to content/browser/loader/.

BUG=646277
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Unexport resource_controller.h

This is a follow up for https://codereview.chromium.org/2535723005/
suggested by mmenke@.

ResourceController is no longer used by any file outside of
content/browser/. Move it to content/browser/loader/.

BUG=646277
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/3b3ee8e99fca8912ad38f2bdc6137189c5043790
Cr-Commit-Position: refs/heads/master@{#436967}
==========

[commit-bot: I haz the power, 2016-12-07 16:06:48]

Patchset 4 (id:??) landed as
https://crrev.com/3b3ee8e99fca8912ad38f2bdc6137189c5043790
Cr-Commit-Position: refs/heads/master@{#436967}

[tyoshino (SeeGerritForStatus), 2016-12-08 14:32:44]

On 2016/12/07 15:58:48, mmenke wrote:
> On 2016/12/07 15:52:23, mmenke wrote:
> > On 2016/12/07 15:37:42, jochen wrote:
> > > lgtm
> > 
> > LGTM, thanks for doing this!
> 
> Going to go ahead and CQ this - as I mentioned, I'm doing some work here, and
> don't want our CLs to fight with each other.

Thanks!