Remove more instances of BrowserThread::FILE.

Remove more instances of BrowserThread::FILE.

These existed to support ProxyConfigServiceLinux, which used to require a
SingleThreadTaskRunner for FILE (but was recently removed).

TBR=sky@chromium.org,mkwst@chromium.org
BUG=689520
Review-Url: https://codereview.chromium.org/2946343003
Cr-Commit-Position: refs/heads/master@{#481711}
Committed: https://chromium.googlesource.com/chromium/src/+/d3bd31fdd98cd0b900581d77953eeaa3e25337ad

[eroman, 2017-06-22 20:04:35]

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

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

Dry run: CQ is trying da patch.

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

[eroman, 2017-06-22 20:13:57]

Description was changed from

==========
Remove more instances of BrowserThread::FILE.

These existed to support ProxyConfigServiceLinux requiring a
SingleThreadTaskRunner, which has since been removed.

BUG=689520
==========

to

==========
Remove more instances of BrowserThread::FILE.

These existed to support ProxyConfigServiceLinux, which used to require a
SingleThreadTaskRunner for FILE (but was recently removed).

BUG=689520
==========

[eroman, 2017-06-22 20:14:10]

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

[eroman, 2017-06-22 20:14:11]

https://codereview.chromium.org/2946343003/diff/1/content/shell/browser/shell...
File content/shell/browser/shell_url_request_context_getter.h (left):

https://codereview.chromium.org/2946343003/diff/1/content/shell/browser/shell...
content/shell/browser/shell_url_request_context_getter.h:37: // TODO(eroman):
Remove the file_task_runner parameter (unused).
This CL is really just about addressing this left-over TODO from
https://codereview.chromium.org/2944313003.

[commit-bot: I haz the power, 2017-06-22 20:16:58]

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

[commit-bot: I haz the power, 2017-06-22 20:16:58]

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

[mmenke, 2017-06-22 20:20:17]

Looks like you have more yet to do.  Fun, fun.

[eroman, 2017-06-22 20:24:00]

Sorry, I was overly optimistic that I had caught all the compile dependencies :(

Will ping when it is ready.

[mmenke, 2017-06-22 20:26:27]

Not a problem.  Done that more than a few times myself.  :)

[eroman, 2017-06-22 20:28:36]

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

[commit-bot: I haz the power, 2017-06-22 20:28:59]

Dry run: CQ is trying da patch.

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

[eroman, 2017-06-22 21:18:34]

PTAL (trybots looking good so far)

[mmenke, 2017-06-22 21:20:46]

On 2017/06/22 21:18:34, eroman wrote:
> PTAL (trybots looking good so far)

LGTM!

[eroman, 2017-06-22 22:18:57]

Description was changed from

==========
Remove more instances of BrowserThread::FILE.

These existed to support ProxyConfigServiceLinux, which used to require a
SingleThreadTaskRunner for FILE (but was recently removed).

BUG=689520
==========

to

==========
Remove more instances of BrowserThread::FILE.

These existed to support ProxyConfigServiceLinux, which used to require a
SingleThreadTaskRunner for FILE (but was recently removed).

TBR=sky@chromium.org,mkwst@chromium.org
BUG=689520
==========

[eroman, 2017-06-22 22:19:49]

eroman@chromium.org changed reviewers:
+ mkwst@chromium.org, sky@chromium.org

[eroman, 2017-06-22 22:19:49]

TBR mkwst and sky:

mkwst:
  content/shell/browser/layout_test/layout_test_browser_context.cc
  content/shell/browser/layout_test/layout_test_url_request_context_getter.cc
  content/shell/browser/layout_test/layout_test_url_request_context_getter.h
  content/shell/browser/shell_browser_context.cc
  content/shell/browser/shell_url_request_context_getter.cc
  content/shell/browser/shell_url_request_context_getter.h
sky:
  content/shell/browser/layout_test/layout_test_browser_context.cc
  content/shell/browser/layout_test/layout_test_url_request_context_getter.cc
  content/shell/browser/layout_test/layout_test_url_request_context_getter.h
  content/shell/browser/shell_browser_context.cc
  content/shell/browser/shell_url_request_context_getter.cc
  content/shell/browser/shell_url_request_context_getter.h
  extensions/shell/browser/shell_browser_context.cc
  extensions/shell/browser/shell_url_request_context_getter.cc
  extensions/shell/browser/shell_url_request_context_getter.h

[eroman, 2017-06-22 22:20:37]

The CQ bit was unchecked by eroman@chromium.org

[eroman, 2017-06-22 22:20:40]

The CQ bit was checked by eroman@chromium.org

[commit-bot: I haz the power, 2017-06-22 22:21:16]

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-06-22 23:16:20]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1498170040796340,
"parent_rev": "4312d5e0b01af9a3ad5f92a1763142155f01a7c6", "commit_rev":
"d3bd31fdd98cd0b900581d77953eeaa3e25337ad"}

[commit-bot: I haz the power, 2017-06-22 23:16:33]

Description was changed from

==========
Remove more instances of BrowserThread::FILE.

These existed to support ProxyConfigServiceLinux, which used to require a
SingleThreadTaskRunner for FILE (but was recently removed).

TBR=sky@chromium.org,mkwst@chromium.org
BUG=689520
==========

to

==========
Remove more instances of BrowserThread::FILE.

These existed to support ProxyConfigServiceLinux, which used to require a
SingleThreadTaskRunner for FILE (but was recently removed).

TBR=sky@chromium.org,mkwst@chromium.org
BUG=689520

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

[commit-bot: I haz the power, 2017-06-22 23:16:35]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/d3bd31fdd98cd0b900581d77953e...

[sky, 2017-06-23 03:25:01]

LGTM

[Mike West, 2017-06-23 06:44:32]

LGTM!