[inspector] use own copy of third_party/inspector_protocol

[inspector] use own copy of third_party/inspector_protocol

When Chromium and V8 use the same version of inspector_protocol, any protocol change takes at least 4 patches and 3 waiting for rolls.
To simplify this process we need to have two diffrent versions of inspector_protocol in Chromium and V8. Current state of inspector_protocol was extracted into separate repository [1]. This CL puts last version of inspector_protocol into third_party/inspector_protocol and removes dependency on inspector_protocol in Webkit.

[1] https://chromium.googlesource.com/deps/inspector_protocol/

BUG=chromium:637032
R=dgozman@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

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

[kozy, 2016-10-26 00:23:50]

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

[commit-bot: I haz the power, 2016-10-26 00:23:59]

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

[kozy, 2016-10-26 00:26:08]

Dmitry, please take a look!

It depends on https://codereview.chromium.org/2444193006/. I will update
revision in README.chromium after landing that dependency.

[kozy, 2016-10-26 00:26:49]

Description was changed from

==========
[inspector] use own copy of thiard_party/inspector_protocol

When Chromium and V8 use the same version of inspector_protocol, any protocol
change takes at least 4 patches and 3 waiting for rolls.
To simplify this process we need to have two diffrent versions of
inspector_protocol in Chromium and V8. Current state of inspector_protocol was
extracted into separate repository [1]. This CL puts last version of
inspector_protocol into third_party/v8_inspector_protocol and removes dependency
on inspector_protocol inside Webkit.

[1] https://chromium.googlesource.com/deps/inspector_protocol/

BUG=chromium:637032
R=dgozman@chromium.org
==========

to

==========
[inspector] use own copy of thiard_party/inspector_protocol

When Chromium and V8 use the same version of inspector_protocol, any protocol
change takes at least 4 patches and 3 waiting for rolls.
To simplify this process we need to have two diffrent versions of
inspector_protocol in Chromium and V8. Current state of inspector_protocol was
extracted into separate repository [1]. This CL puts last version of
inspector_protocol into third_party/v8_inspector_protocol and removes dependency
on inspector_protocol inside Webkit.

[1] https://chromium.googlesource.com/deps/inspector_protocol/

BUG=chromium:637032
R=dgozman@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[kozy, 2016-10-26 00:27:01]

The CQ bit was unchecked by kozyatinskiy@chromium.org

[kozy, 2016-10-26 00:27:25]

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

[commit-bot: I haz the power, 2016-10-26 00:27:28]

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

[kozy, 2016-10-26 00:27:55]

Description was changed from

==========
[inspector] use own copy of thiard_party/inspector_protocol

When Chromium and V8 use the same version of inspector_protocol, any protocol
change takes at least 4 patches and 3 waiting for rolls.
To simplify this process we need to have two diffrent versions of
inspector_protocol in Chromium and V8. Current state of inspector_protocol was
extracted into separate repository [1]. This CL puts last version of
inspector_protocol into third_party/v8_inspector_protocol and removes dependency
on inspector_protocol inside Webkit.

[1] https://chromium.googlesource.com/deps/inspector_protocol/

BUG=chromium:637032
R=dgozman@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
[inspector] use own copy of third_party/inspector_protocol

When Chromium and V8 use the same version of inspector_protocol, any protocol
change takes at least 4 patches and 3 waiting for rolls.
To simplify this process we need to have two diffrent versions of
inspector_protocol in Chromium and V8. Current state of inspector_protocol was
extracted into separate repository [1]. This CL puts last version of
inspector_protocol into third_party/v8_inspector_protocol and removes dependency
on inspector_protocol inside Webkit.

[1] https://chromium.googlesource.com/deps/inspector_protocol/

BUG=chromium:637032
R=dgozman@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[kozy, 2016-10-26 00:29:51]

And we need to revert https://codereview.chromium.org/2436983002/ after landing
this.

[commit-bot: I haz the power, 2016-10-26 00:34:36]

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

[commit-bot: I haz the power, 2016-10-26 00:34:37]

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

[kozy, 2016-10-26 00:41:40]

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

[commit-bot: I haz the power, 2016-10-26 00:41:46]

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

[kozy, 2016-10-26 00:48:43]

Patchset #3 (id:40001) has been deleted

[kozy, 2016-10-26 00:48:54]

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

[commit-bot: I haz the power, 2016-10-26 00:48:56]

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

[kozy, 2016-10-26 00:49:14]

Description was changed from

==========
[inspector] use own copy of third_party/inspector_protocol

When Chromium and V8 use the same version of inspector_protocol, any protocol
change takes at least 4 patches and 3 waiting for rolls.
To simplify this process we need to have two diffrent versions of
inspector_protocol in Chromium and V8. Current state of inspector_protocol was
extracted into separate repository [1]. This CL puts last version of
inspector_protocol into third_party/v8_inspector_protocol and removes dependency
on inspector_protocol inside Webkit.

[1] https://chromium.googlesource.com/deps/inspector_protocol/

BUG=chromium:637032
R=dgozman@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
[inspector] use own copy of third_party/inspector_protocol

When Chromium and V8 use the same version of inspector_protocol, any protocol
change takes at least 4 patches and 3 waiting for rolls.
To simplify this process we need to have two diffrent versions of
inspector_protocol in Chromium and V8. Current state of inspector_protocol was
extracted into separate repository [1]. This CL puts last version of
inspector_protocol into third_party/inspector_protocol and removes dependency on
inspector_protocol in Webkit.

[1] https://chromium.googlesource.com/deps/inspector_protocol/

BUG=chromium:637032
R=dgozman@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[kozy, 2016-10-26 00:52:39]

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

[commit-bot: I haz the power, 2016-10-26 00:52:49]

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

[dgozman, 2016-10-26 01:54:30]

This should be structured differently. Take a look at
https://chromium.googlesource.com/chromium/src/+/master/third_party/closure_l...
as an example.

[commit-bot: I haz the power, 2016-10-26 02:15:09]

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

[commit-bot: I haz the power, 2016-10-26 02:15:10]

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

[kozy, 2016-10-26 17:51:26]

Changed structure, please take another look!

[kozy, 2016-10-26 19:20:56]

Please take another look.

[dgozman, 2016-10-26 19:26:36]

lgtm

https://codereview.chromium.org/2447323002/diff/140001/third_party/inspector_...
File third_party/inspector_protocol/OWNERS (right):

https://codereview.chromium.org/2447323002/diff/140001/third_party/inspector_...
third_party/inspector_protocol/OWNERS:1: set noparent
Let's use file://src/inspector/OWNERS

[kozy, 2016-10-26 21:11:12]

https://codereview.chromium.org/2447323002/diff/140001/third_party/inspector_...
File third_party/inspector_protocol/OWNERS (right):

https://codereview.chromium.org/2447323002/diff/140001/third_party/inspector_...
third_party/inspector_protocol/OWNERS:1: set noparent
On 2016/10/26 19:26:35, dgozman wrote:
> Let's use file://src/inspector/OWNERS

It won't work when V8 is part of chromium checkout and there is no file usage in
V8.

[kozy, 2016-10-26 21:11:29]

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

[kozy, 2016-10-26 21:11:30]

Jochen, please take a look!

[kozy, 2016-10-26 21:11:45]

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

[commit-bot: I haz the power, 2016-10-26 21:11:54]

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-10-26 22:01:48]

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

[commit-bot: I haz the power, 2016-10-26 22:01:49]

Dry run: This issue passed the CQ dry run.

[jochen (gone - plz use gerrit), 2016-10-27 07:41:27]

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

[jochen (gone - plz use gerrit), 2016-10-27 07:41:28]

deferring to Yang

[Yang, 2016-10-28 15:18:19]

On 2016/10/27 07:41:28, jochen wrote:
> deferring to Yang

lgtm

[Yang, 2016-10-28 15:18:20]

On 2016/10/27 07:41:28, jochen wrote:
> deferring to Yang

lgtm

[kozy, 2016-10-28 17:31:10]

The CQ bit was checked by kozyatinskiy@chromium.org

[commit-bot: I haz the power, 2016-10-28 17:31:20]

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-10-28 18:20:15]

Description was changed from

==========
[inspector] use own copy of third_party/inspector_protocol

When Chromium and V8 use the same version of inspector_protocol, any protocol
change takes at least 4 patches and 3 waiting for rolls.
To simplify this process we need to have two diffrent versions of
inspector_protocol in Chromium and V8. Current state of inspector_protocol was
extracted into separate repository [1]. This CL puts last version of
inspector_protocol into third_party/inspector_protocol and removes dependency on
inspector_protocol in Webkit.

[1] https://chromium.googlesource.com/deps/inspector_protocol/

BUG=chromium:637032
R=dgozman@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
[inspector] use own copy of third_party/inspector_protocol

When Chromium and V8 use the same version of inspector_protocol, any protocol
change takes at least 4 patches and 3 waiting for rolls.
To simplify this process we need to have two diffrent versions of
inspector_protocol in Chromium and V8. Current state of inspector_protocol was
extracted into separate repository [1]. This CL puts last version of
inspector_protocol into third_party/inspector_protocol and removes dependency on
inspector_protocol in Webkit.

[1] https://chromium.googlesource.com/deps/inspector_protocol/

BUG=chromium:637032
R=dgozman@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[commit-bot: I haz the power, 2016-10-28 18:20:19]

Committed patchset #7 (id:140001)

[commit-bot: I haz the power, 2016-11-17 22:17:13]

Description was changed from

==========
[inspector] use own copy of third_party/inspector_protocol

When Chromium and V8 use the same version of inspector_protocol, any protocol
change takes at least 4 patches and 3 waiting for rolls.
To simplify this process we need to have two diffrent versions of
inspector_protocol in Chromium and V8. Current state of inspector_protocol was
extracted into separate repository [1]. This CL puts last version of
inspector_protocol into third_party/inspector_protocol and removes dependency on
inspector_protocol in Webkit.

[1] https://chromium.googlesource.com/deps/inspector_protocol/

BUG=chromium:637032
R=dgozman@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
[inspector] use own copy of third_party/inspector_protocol

When Chromium and V8 use the same version of inspector_protocol, any protocol
change takes at least 4 patches and 3 waiting for rolls.
To simplify this process we need to have two diffrent versions of
inspector_protocol in Chromium and V8. Current state of inspector_protocol was
extracted into separate repository [1]. This CL puts last version of
inspector_protocol into third_party/inspector_protocol and removes dependency on
inspector_protocol in Webkit.

[1] https://chromium.googlesource.com/deps/inspector_protocol/

BUG=chromium:637032
R=dgozman@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

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

[commit-bot: I haz the power, 2016-11-17 22:17:15]

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