DevTools: move protocol-related generators into inspector subfolder.

DevTools: move protocol-related generators into inspector subfolder.

BUG=580337
NOPRESUBMIT=true

Committed: https://crrev.com/b4b1bdc22eabec57d57b92968a4fbf882446dfda
Cr-Commit-Position: refs/heads/master@{#375109}

[pfeldman, 2016-02-11 22:22:13]

pfeldman@chromium.org changed reviewers:
+ alph@chromium.org

[pfeldman, 2016-02-11 22:23:23]

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

[commit-bot: I haz the power, 2016-02-11 22:25:41]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1696513002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1696513002/1

[commit-bot: I haz the power, 2016-02-11 22:36:46]

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

[commit-bot: I haz the power, 2016-02-11 22:36:47]

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

[pfeldman, 2016-02-12 00:01:26]

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

[commit-bot: I haz the power, 2016-02-12 00:04:03]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1696513002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1696513002/20001

[alph, 2016-02-12 00:21:33]

lgtm

https://codereview.chromium.org/1696513002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/inspector/InspectorBaseAgent.h (right):

https://codereview.chromium.org/1696513002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/inspector/InspectorBaseAgent.h:46: class
Frontend;
blink::Frontend sounds too generic as well as blink::Dispatcher
Consider making a namespace. How about Remote Inspector Protocol?

https://codereview.chromium.org/1696513002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/inspector/WorkerInspectorController.h
(right):

https://codereview.chromium.org/1696513002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/inspector/WorkerInspectorController.h:46: class
InspectorFrontend;
Rename/remove?

https://codereview.chromium.org/1696513002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/inspector/WorkerInspectorController.h:47: class
FrontendChannel;
Why not just Channel then ;-)

[pfeldman, 2016-02-12 01:22:03]

The CQ bit was checked by pfeldman@chromium.org

[pfeldman, 2016-02-12 01:22:04]

The patchset sent to the CQ was uploaded after l-g-t-m from alph@chromium.org
Link to the patchset: https://codereview.chromium.org/1696513002/#ps40001
(title: " ")

[pfeldman, 2016-02-12 01:22:53]

The CQ bit was unchecked by pfeldman@chromium.org

[pfeldman, 2016-02-12 01:30:03]

The CQ bit was checked by pfeldman@chromium.org

[pfeldman, 2016-02-12 01:30:04]

The patchset sent to the CQ was uploaded after l-g-t-m from alph@chromium.org
Link to the patchset: https://codereview.chromium.org/1696513002/#ps60001
(title: " ")

[commit-bot: I haz the power, 2016-02-12 01:30:37]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1696513002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1696513002/60001

[commit-bot: I haz the power, 2016-02-12 01:36:28]

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

[commit-bot: I haz the power, 2016-02-12 01:36:29]

Try jobs failed on following builders:
  android_arm64_dbg_recipe on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)
  android_clang_dbg_recipe on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)
  android_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  cast_shell_android on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  cast_shell_linux on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  chromeos_amd64-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  chromeos_daisy_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)
  chromeos_x86-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_x86-ge...)
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
  linux_chromium_asan_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_compile_dbg_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_ozone_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_clobber_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_compile_dbg_32_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_compile_dbg_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_gn_chromeos_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[pfeldman, 2016-02-12 02:02:39]

Description was changed from

==========
DevTools: move protocol-related generators into inspector subfolder.

BUG=580337
==========

to

==========
DevTools: move protocol-related generators into inspector subfolder.

BUG=580337
NOPRESUBMIT=true
==========

[pfeldman, 2016-02-12 02:03:09]

The CQ bit was checked by pfeldman@chromium.org

[pfeldman, 2016-02-12 02:03:10]

The patchset sent to the CQ was uploaded after l-g-t-m from alph@chromium.org
Link to the patchset: https://codereview.chromium.org/1696513002/#ps80001
(title: " ")

[commit-bot: I haz the power, 2016-02-12 02:05:14]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1696513002/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1696513002/80001

[commit-bot: I haz the power, 2016-02-12 03:29:32]

Description was changed from

==========
DevTools: move protocol-related generators into inspector subfolder.

BUG=580337
NOPRESUBMIT=true
==========

to

==========
DevTools: move protocol-related generators into inspector subfolder.

BUG=580337
NOPRESUBMIT=true
==========

[commit-bot: I haz the power, 2016-02-12 03:29:33]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-02-16 22:41:17]

Description was changed from

==========
DevTools: move protocol-related generators into inspector subfolder.

BUG=580337
NOPRESUBMIT=true
==========

to

==========
DevTools: move protocol-related generators into inspector subfolder.

BUG=580337
NOPRESUBMIT=true

Committed: https://crrev.com/b4b1bdc22eabec57d57b92968a4fbf882446dfda
Cr-Commit-Position: refs/heads/master@{#375109}
==========

[commit-bot: I haz the power, 2016-02-16 22:41:18]

Patchset 5 (id:??) landed as
https://crrev.com/b4b1bdc22eabec57d57b92968a4fbf882446dfda
Cr-Commit-Position: refs/heads/master@{#375109}