Remove unused thread.h include in trace_event_impl.h and fix IWUU.

Remove unneeded thread.h includes and fix IWUU.

Include removed from trace_event_impl.h and leveldb_mojo_proxy.h

Removing unnecessary thread.h includes allows including trace_event_impl.h
in pending_task.h without introducing a cyclic dependency
(in a follow-up CL).

BUG=649084
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

TBR=wez,pastarmovj,rockot,sadrul (ref. https://codereview.chromium.org/2365623003/#msg68)

Committed: https://crrev.com/4d92485f89ba1de9d4e4f0f37789a0055e19daae
Cr-Commit-Position: refs/heads/master@{#420989}

[gab, 2016-09-22 20:30:51]

Description was changed from

==========
Remove unused thread.h include in trace_event_impl.h and fix IWUU.

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

to

==========
Remove unneeded thread.h includes and fix IWUU.

Removing unnecessary thread.h includes allows including trace_event_impl.h 
in  pending_task.h without introducing a cyclic dependency 
(in a follow-up CL).

BUG=649084
==========

[gab, 2016-09-22 20:30:57]

Description was changed from

==========
Remove unneeded thread.h includes and fix IWUU.

Removing unnecessary thread.h includes allows including trace_event_impl.h 
in  pending_task.h without introducing a cyclic dependency 
(in a follow-up CL).

BUG=649084
==========

to

==========
Remove unneeded thread.h includes and fix IWUU.

Removing unnecessary thread.h includes allows including trace_event_impl.h 
in pending_task.h without introducing a cyclic dependency 
(in a follow-up CL).

BUG=649084
==========

[gab, 2016-09-22 20:33:13]

gab@chromium.org changed reviewers:
+ dalecurtis@chromium.org, danakj@chromium.org, erg@chromium.org,
oysteine@chromium.org

[gab, 2016-09-22 20:33:14]

oysteine PTAL at whole CL as owner of base/trace_event

TBR:
 - danakj for base/ side-effects
 - erg for components/level_db side-effects
 - dalecurtis for media/gpu side-effects

[gab, 2016-09-22 20:33:18]

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

[commit-bot: I haz the power, 2016-09-22 20:33:52]

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

[DaleCurtis, 2016-09-22 20:34:06]

media/ lgtm

[danakj, 2016-09-22 20:35:25]

LGTM

[Elliot Glaysher, 2016-09-22 20:37:22]

lgtm

[oystein (OOO til 10th of July), 2016-09-22 20:47:40]

lgtm

[commit-bot: I haz the power, 2016-09-22 20:49:57]

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

[commit-bot: I haz the power, 2016-09-22 20:49:58]

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

[gab, 2016-09-23 17:29:37]

Description was changed from

==========
Remove unneeded thread.h includes and fix IWUU.

Removing unnecessary thread.h includes allows including trace_event_impl.h 
in pending_task.h without introducing a cyclic dependency 
(in a follow-up CL).

BUG=649084
==========

to

==========
Remove unneeded thread.h includes and fix IWUU.

Removing unnecessary thread.h includes allows including trace_event_impl.h 
in pending_task.h without introducing a cyclic dependency 
(in a follow-up CL).

BUG=649084
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[gab, 2016-09-23 17:29:40]

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

[commit-bot: I haz the power, 2016-09-23 17:30:13]

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

[gab, 2016-09-23 17:32:07]

gab@chromium.org changed reviewers:
+ mattm@chromium.org, pastarmovj@chromium.org, rockot@chromium.org

[gab, 2016-09-23 17:32:08]

TBR for more IWUU fixes falling out of this:

@oystein:
   base/trace_event/trace_event_android.cc

@danakj:
   cc/test/fake_external_begin_frame_source.h
   cc/tiles/picture_layer_tiling_set.h
   cc/trees/remote_channel_impl.h

@mattm:
   chrome/browser/safe_browsing/protocol_manager_unittest.cc

@pastarmovj:
   components/policy/core/common/cloud/policy_header_io_helper_unittest.cc
   components/policy/core/common/cloud/policy_header_service_unittest.cc

@rockot:
   ipc/attachment_broker_mac_unittest.cc
   ipc/ipc_send_fds_test.cc
   ipc/ipc_test_channel_listener.cc

[commit-bot: I haz the power, 2016-09-23 17:47:14]

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

[commit-bot: I haz the power, 2016-09-23 17:47:17]

Dry run: Try jobs failed on following builders:
  android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)

[mattm, 2016-09-23 18:01:30]

safe_browsing lgtm

[gab, 2016-09-23 18:47:10]

Patchset #3 (id:40001) has been deleted

[gab, 2016-09-23 18:48:21]

Description was changed from

==========
Remove unneeded thread.h includes and fix IWUU.

Removing unnecessary thread.h includes allows including trace_event_impl.h 
in pending_task.h without introducing a cyclic dependency 
(in a follow-up CL).

BUG=649084
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
Remove unneeded thread.h includes and fix IWUU.

Include removed from trace_event_impl.h and leveldb_mojo_proxy.h

Removing unnecessary thread.h includes allows including trace_event_impl.h 
in pending_task.h without introducing a cyclic dependency 
(in a follow-up CL).

BUG=649084
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[gab, 2016-09-23 18:48:43]

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

[commit-bot: I haz the power, 2016-09-23 18:49:33]

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

[gab, 2016-09-23 18:51:31]

gab@chromium.org changed reviewers:
+ sadrul@chromium.org, zmo@chromium.org

[gab, 2016-09-23 18:51:32]

On 2016/09/23 17:32:08, gab wrote:
> TBR for more IWUU fixes falling out of this:
> 
> @oystein:
>    base/trace_event/trace_event_android.cc
> 
> @danakj:
>    cc/test/fake_external_begin_frame_source.h
>    cc/tiles/picture_layer_tiling_set.h
>    cc/trees/remote_channel_impl.h
> 
> @pastarmovj:
>    components/policy/core/common/cloud/policy_header_io_helper_unittest.cc
>    components/policy/core/common/cloud/policy_header_service_unittest.cc
> 
> @rockot:
>    ipc/attachment_broker_mac_unittest.cc
>    ipc/ipc_send_fds_test.cc
>    ipc/ipc_test_channel_listener.cc

and more..!

@danakj:
 cc/raster/raster_buffer_provider_unittest.cc

@zmo:
 content/browser/gpu/gpu_process_host.h

@dalecurtis:
 media/gpu/android_video_decode_accelerator.cc

@sadrul:
 ui/events/ozone/device/udev/device_manager_udev.cc
 ui/gl/gl_context_cgl.cc
 ui/ozone/public/ozone_gpu_test_helper.cc

[commit-bot: I haz the power, 2016-09-23 19:12:55]

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

[commit-bot: I haz the power, 2016-09-23 19:12:57]

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

[gab, 2016-09-23 19:22:29]

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

[commit-bot: I haz the power, 2016-09-23 19:23:04]

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

[gab, 2016-09-23 19:24:31]

gab@chromium.org changed reviewers:
+ hubbe@chromium.org, nick@chromium.org, wez@chromium.org

[gab, 2016-09-23 19:24:32]

On 2016/09/23 18:51:32, gab wrote:
> On 2016/09/23 17:32:08, gab wrote:
> > TBR for more IWUU fixes falling out of this:
> > 
> > @oystein:
> >    base/trace_event/trace_event_android.cc
> > 
> > @danakj:
> >    cc/test/fake_external_begin_frame_source.h
> >    cc/tiles/picture_layer_tiling_set.h
> >    cc/trees/remote_channel_impl.h
> > 
> > @pastarmovj:
> >    components/policy/core/common/cloud/policy_header_io_helper_unittest.cc
> >    components/policy/core/common/cloud/policy_header_service_unittest.cc
> > 
> > @rockot:
> >    ipc/attachment_broker_mac_unittest.cc
> >    ipc/ipc_send_fds_test.cc
> >    ipc/ipc_test_channel_listener.cc
> 
> and more..!
> 
> @danakj:
>  cc/raster/raster_buffer_provider_unittest.cc
> 
> @zmo:
>  content/browser/gpu/gpu_process_host.h
> 
> @dalecurtis:
>  media/gpu/android_video_decode_accelerator.cc
> 
> @sadrul:
>  ui/events/ozone/device/udev/device_manager_udev.cc
>  ui/gl/gl_context_cgl.cc
>  ui/ozone/public/ozone_gpu_test_helper.cc

and more..!

@wez:
 blimp/client/core/compositor/blimp_compositor_frame_sink_unittest.cc
 blimp/client/core/compositor/blimp_compositor_unittest.cc

@nick:
 content/browser/renderer_host/render_message_filter.cc

@hubbe:
 media/cast/test/end2end_unittest.cc

[hubbe, 2016-09-23 19:27:14]

media/cast/test/end2end_unittest.cc LGTM

[ncarter (slow), 2016-09-23 19:48:53]

content/ lgtm

[commit-bot: I haz the power, 2016-09-23 20:02:55]

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

[commit-bot: I haz the power, 2016-09-23 20:02:57]

Dry run: Try jobs failed on following builders:
  android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[gab, 2016-09-23 20:26:05]

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

[commit-bot: I haz the power, 2016-09-23 20:26:30]

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, 2016-09-23 20:52:42]

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

[commit-bot: I haz the power, 2016-09-23 20:52:43]

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

[gab, 2016-09-24 04:15:36]

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

[commit-bot: I haz the power, 2016-09-24 04:15:43]

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

[gab, 2016-09-24 04:17:26]

Patchset #6 (id:120001) has been deleted

[gab, 2016-09-24 04:17:27]

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

[commit-bot: I haz the power, 2016-09-24 04:17:35]

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

[gab, 2016-09-24 04:19:42]

Patchset #6 (id:140001) has been deleted

[gab, 2016-09-24 04:21:30]

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

[commit-bot: I haz the power, 2016-09-24 04:21:40]

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

[gab, 2016-09-24 04:26:26]

The CQ bit was unchecked by gab@chromium.org

[gab, 2016-09-24 13:03:55]

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

[commit-bot: I haz the power, 2016-09-24 13:04:03]

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, 2016-09-24 13:12:53]

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

[commit-bot: I haz the power, 2016-09-24 13:12:55]

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

[gab, 2016-09-24 13:16:25]

gab@chromium.org changed reviewers:
+ esprehn@chromium.org

[gab, 2016-09-24 13:16:26]

Left to review:

@esprehn (this currently breaks DEPS presubmit... what's the right thing to do
here? it clearly used these already by transitivity instead of IWUU...):
 third_party/WebKit/Source/web/tests/TimerPerfTest.cpp
 third_party/WebKit/Source/web/tests/VirtualTimeTest.cpp


Trivial side-effects (I'll TBR remaining ones once CQ is green):

@wez:
   blimp/client/core/compositor/blimp_compositor_frame_sink_unittest.cc
   blimp/client/core/compositor/blimp_compositor_unittest.cc
   
@pastarmovj:
   components/policy/core/common/cloud/policy_header_io_helper_unittest.cc
   components/policy/core/common/cloud/policy_header_service_unittest.cc

@rockot:
   extensions/browser/api/display_source/display_source_apitestbase.cc
   ipc/attachment_broker_mac_unittest.cc
   ipc/ipc_send_fds_test.cc
   ipc/ipc_test_channel_listener.cc

@sadrul:
   ui/events/ozone/device/udev/device_manager_udev.cc
   ui/gl/gl_context_cgl.cc
   ui/ozone/public/ozone_gpu_test_helper.cc

[gab, 2016-09-24 13:23:11]

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

[commit-bot: I haz the power, 2016-09-24 13:23:25]

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

[esprehn, 2016-09-24 14:10:12]

Hmm you're not really supposed to include base in web, but I see this is tests.
Maybe we should whitelist web/tests in general?

[commit-bot: I haz the power, 2016-09-24 14:57:07]

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

[commit-bot: I haz the power, 2016-09-24 14:57:08]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[gab, 2016-09-26 13:54:47]

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

[commit-bot: I haz the power, 2016-09-26 13:55:02]

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

[gab, 2016-09-26 13:55:14]

On 2016/09/24 14:10:12, esprehn wrote:
> Hmm you're not really supposed to include base in web, but I see this is
tests.
> Maybe we should whitelist web/tests in general?

Looks like it was only these two tests [1] but I'm fine whitelisting base/ for
web/tests/ if you think that's okay. Done in latest PS, PTAnL, thanks!

[1]
https://cs.chromium.org/search/?q=case:yes+file:src/third_party/WebKit/Source...

[commit-bot: I haz the power, 2016-09-26 16:48:35]

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

[commit-bot: I haz the power, 2016-09-26 16:48:37]

Dry run: This issue passed the CQ dry run.

[gab, 2016-09-26 19:38:32]

On 2016/09/26 13:55:14, gab wrote:
> On 2016/09/24 14:10:12, esprehn wrote:
> > Hmm you're not really supposed to include base in web, but I see this is
> tests.
> > Maybe we should whitelist web/tests in general?
> 
> Looks like it was only these two tests [1] but I'm fine whitelisting base/ for
> web/tests/ if you think that's okay. Done in latest PS, PTAnL, thanks!
> 
> [1]
>
https://cs.chromium.org/search/?q=case:yes+file:src/third_party/WebKit/Source...

ping esprehn, thanks!

[esprehn, 2016-09-26 20:48:37]

lgtm

[gab, 2016-09-26 20:51:12]

A few TBRs for other benign IWUU:

@wez:
   blimp/client/core/compositor/blimp_compositor_frame_sink_unittest.cc
   blimp/client/core/compositor/blimp_compositor_unittest.cc
   
@pastarmovj:
   components/policy/core/common/cloud/policy_header_io_helper_unittest.cc
   components/policy/core/common/cloud/policy_header_service_unittest.cc

@rockot:
   extensions/browser/api/display_source/display_source_apitestbase.cc
   ipc/attachment_broker_mac_unittest.cc
   ipc/ipc_send_fds_test.cc
   ipc/ipc_test_channel_listener.cc

@sadrul:
   ui/events/ozone/device/udev/device_manager_udev.cc
   ui/gl/gl_context_cgl.cc
   ui/ozone/public/ozone_gpu_test_helper.cc

Thanks!

[gab, 2016-09-26 20:51:53]

Description was changed from

==========
Remove unneeded thread.h includes and fix IWUU.

Include removed from trace_event_impl.h and leveldb_mojo_proxy.h

Removing unnecessary thread.h includes allows including trace_event_impl.h 
in pending_task.h without introducing a cyclic dependency 
(in a follow-up CL).

BUG=649084
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
Remove unneeded thread.h includes and fix IWUU.

Include removed from trace_event_impl.h and leveldb_mojo_proxy.h

Removing unnecessary thread.h includes allows including trace_event_impl.h 
in pending_task.h without introducing a cyclic dependency 
(in a follow-up CL).

BUG=649084
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

TBR=wez,pastarmovj,rockot,sadrul

For trivial IWUU:

@wez:
   blimp/client/core/compositor/blimp_compositor_frame_sink_unittest.cc
   blimp/client/core/compositor/blimp_compositor_unittest.cc
   
@pastarmovj:
   components/policy/core/common/cloud/policy_header_io_helper_unittest.cc
   components/policy/core/common/cloud/policy_header_service_unittest.cc

@rockot:
   extensions/browser/api/display_source/display_source_apitestbase.cc
   ipc/attachment_broker_mac_unittest.cc
   ipc/ipc_send_fds_test.cc
   ipc/ipc_test_channel_listener.cc

@sadrul:
   ui/events/ozone/device/udev/device_manager_udev.cc
   ui/gl/gl_context_cgl.cc
   ui/ozone/public/ozone_gpu_test_helper.cc
==========

[gab, 2016-09-26 20:52:18]

Description was changed from

==========
Remove unneeded thread.h includes and fix IWUU.

Include removed from trace_event_impl.h and leveldb_mojo_proxy.h

Removing unnecessary thread.h includes allows including trace_event_impl.h 
in pending_task.h without introducing a cyclic dependency 
(in a follow-up CL).

BUG=649084
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

TBR=wez,pastarmovj,rockot,sadrul

For trivial IWUU:

@wez:
   blimp/client/core/compositor/blimp_compositor_frame_sink_unittest.cc
   blimp/client/core/compositor/blimp_compositor_unittest.cc
   
@pastarmovj:
   components/policy/core/common/cloud/policy_header_io_helper_unittest.cc
   components/policy/core/common/cloud/policy_header_service_unittest.cc

@rockot:
   extensions/browser/api/display_source/display_source_apitestbase.cc
   ipc/attachment_broker_mac_unittest.cc
   ipc/ipc_send_fds_test.cc
   ipc/ipc_test_channel_listener.cc

@sadrul:
   ui/events/ozone/device/udev/device_manager_udev.cc
   ui/gl/gl_context_cgl.cc
   ui/ozone/public/ozone_gpu_test_helper.cc
==========

to

==========
Remove unneeded thread.h includes and fix IWUU.

Include removed from trace_event_impl.h and leveldb_mojo_proxy.h

Removing unnecessary thread.h includes allows including trace_event_impl.h 
in pending_task.h without introducing a cyclic dependency 
(in a follow-up CL).

BUG=649084
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

TBR=wez,pastarmovj,rockot,sadrul (ref.
https://codereview.chromium.org/2365623003/#msg68)
==========

[gab, 2016-09-26 20:52:27]

The CQ bit was checked by gab@chromium.org

[gab, 2016-09-26 20:52:28]

The patchset sent to the CQ was uploaded after l-g-t-m from danakj@chromium.org,
oysteine@chromium.org, erg@chromium.org, dalecurtis@chromium.org,
mattm@chromium.org, nick@chromium.org, hubbe@chromium.org
Link to the patchset: https://codereview.chromium.org/2365623003/#ps240001
(title: "Whitelist base DEPS for all web/tests")

[commit-bot: I haz the power, 2016-09-26 20:53:13]

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-09-26 21:01:07]

Description was changed from

==========
Remove unneeded thread.h includes and fix IWUU.

Include removed from trace_event_impl.h and leveldb_mojo_proxy.h

Removing unnecessary thread.h includes allows including trace_event_impl.h 
in pending_task.h without introducing a cyclic dependency 
(in a follow-up CL).

BUG=649084
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

TBR=wez,pastarmovj,rockot,sadrul (ref.
https://codereview.chromium.org/2365623003/#msg68)
==========

to

==========
Remove unneeded thread.h includes and fix IWUU.

Include removed from trace_event_impl.h and leveldb_mojo_proxy.h

Removing unnecessary thread.h includes allows including trace_event_impl.h 
in pending_task.h without introducing a cyclic dependency 
(in a follow-up CL).

BUG=649084
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

TBR=wez,pastarmovj,rockot,sadrul (ref.
https://codereview.chromium.org/2365623003/#msg68)
==========

[commit-bot: I haz the power, 2016-09-26 21:01:09]

Committed patchset #10 (id:240001)

[commit-bot: I haz the power, 2016-09-26 21:04:23]

Description was changed from

==========
Remove unneeded thread.h includes and fix IWUU.

Include removed from trace_event_impl.h and leveldb_mojo_proxy.h

Removing unnecessary thread.h includes allows including trace_event_impl.h 
in pending_task.h without introducing a cyclic dependency 
(in a follow-up CL).

BUG=649084
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

TBR=wez,pastarmovj,rockot,sadrul (ref.
https://codereview.chromium.org/2365623003/#msg68)
==========

to

==========
Remove unneeded thread.h includes and fix IWUU.

Include removed from trace_event_impl.h and leveldb_mojo_proxy.h

Removing unnecessary thread.h includes allows including trace_event_impl.h
in pending_task.h without introducing a cyclic dependency
(in a follow-up CL).

BUG=649084
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

TBR=wez,pastarmovj,rockot,sadrul (ref.
https://codereview.chromium.org/2365623003/#msg68)

Committed: https://crrev.com/4d92485f89ba1de9d4e4f0f37789a0055e19daae
Cr-Commit-Position: refs/heads/master@{#420989}
==========

[commit-bot: I haz the power, 2016-09-26 21:04:25]

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

[sadrul, 2016-09-27 04:01:52]

lgtm

[Wez, 2016-10-08 01:21:15]

blimp/ LGTM

[pastarmovj, 2016-10-10 09:59:19]

lgtm