DevTools: add correct include dirs to protocol handler generator for V8.

DevTools: add correct include dirs to protocol handler generator for V8.

Jinja2 and markupsafe will be pulled into v8/third_party/ via DEPS,
while the protocol handler generator will be pulled into
v8/platform/inspector_protocol/. In order for the protocol handler
generator to work when building standalone V8, it needs to include
them correctly.

R=pfeldman@chromium.org
BUG=580337
Committed: https://crrev.com/322f425f3406a70a1513d53c35bf55bd5c3b2165
Cr-Commit-Position: refs/heads/master@{#407582}

[pfeldman, 2016-07-22 17:07:34]

https://codereview.chromium.org/2174773002/diff/1/third_party/WebKit/Source/p...
File third_party/WebKit/Source/platform/inspector_protocol/CodeGenerator.py
(right):

https://codereview.chromium.org/2174773002/diff/1/third_party/WebKit/Source/p...
third_party/WebKit/Source/platform/inspector_protocol/CodeGenerator.py:35:
module_path, os.pardir, os.pardir, "tools"))
I don't think we are allowed to have third party outside of third_party folder.

[Yang, 2016-07-25 04:55:56]

On 2016/07/22 17:07:34, pfeldman wrote:
>
https://codereview.chromium.org/2174773002/diff/1/third_party/WebKit/Source/p...
> File third_party/WebKit/Source/platform/inspector_protocol/CodeGenerator.py
> (right):
> 
>
https://codereview.chromium.org/2174773002/diff/1/third_party/WebKit/Source/p...
> third_party/WebKit/Source/platform/inspector_protocol/CodeGenerator.py:35:
> module_path, os.pardir, os.pardir, "tools"))
> I don't think we are allowed to have third party outside of third_party
folder.

You are right. I changed the include path to v8/third_party.

[Yang, 2016-07-25 04:56:33]

Description was changed from

==========
DevTools: add correct include dirs to protocol handler generator for V8.

Jinja2 and markupsafe will be pulled into v8/tools/ via DEPS, while the
protocol handler generator will be pulled into
v8/platform/inspector_protocol/. In order for the protocol handler
generator to work when building standalone V8, it needs to include them
correctly.

R=pfeldman@chromium.org
BUG=580337
==========

to

==========
DevTools: add correct include dirs to protocol handler generator for V8.

Jinja2 and markupsafe will be pulled into v8/third_party/ via DEPS,
while the protocol handler generator will be pulled into
v8/platform/inspector_protocol/. In order for the protocol handler
generator to work when building standalone V8, it needs to include
them correctly.

R=pfeldman@chromium.org
BUG=580337
==========

[pfeldman, 2016-07-25 17:13:33]

lgtm

[Yang, 2016-07-25 19:01:31]

The CQ bit was checked by yangguo@chromium.org

[commit-bot: I haz the power, 2016-07-25 19:02: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-07-25 21:25:38]

Description was changed from

==========
DevTools: add correct include dirs to protocol handler generator for V8.

Jinja2 and markupsafe will be pulled into v8/third_party/ via DEPS,
while the protocol handler generator will be pulled into
v8/platform/inspector_protocol/. In order for the protocol handler
generator to work when building standalone V8, it needs to include
them correctly.

R=pfeldman@chromium.org
BUG=580337
==========

to

==========
DevTools: add correct include dirs to protocol handler generator for V8.

Jinja2 and markupsafe will be pulled into v8/third_party/ via DEPS,
while the protocol handler generator will be pulled into
v8/platform/inspector_protocol/. In order for the protocol handler
generator to work when building standalone V8, it needs to include
them correctly.

R=pfeldman@chromium.org
BUG=580337
==========

[commit-bot: I haz the power, 2016-07-25 21:25:39]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-25 21:28:33]

Description was changed from

==========
DevTools: add correct include dirs to protocol handler generator for V8.

Jinja2 and markupsafe will be pulled into v8/third_party/ via DEPS,
while the protocol handler generator will be pulled into
v8/platform/inspector_protocol/. In order for the protocol handler
generator to work when building standalone V8, it needs to include
them correctly.

R=pfeldman@chromium.org
BUG=580337
==========

to

==========
DevTools: add correct include dirs to protocol handler generator for V8.

Jinja2 and markupsafe will be pulled into v8/third_party/ via DEPS,
while the protocol handler generator will be pulled into
v8/platform/inspector_protocol/. In order for the protocol handler
generator to work when building standalone V8, it needs to include
them correctly.

R=pfeldman@chromium.org
BUG=580337

Committed: https://crrev.com/322f425f3406a70a1513d53c35bf55bd5c3b2165
Cr-Commit-Position: refs/heads/master@{#407582}
==========

[commit-bot: I haz the power, 2016-07-25 21:28:33]

Patchset 2 (id:??) landed as
https://crrev.com/322f425f3406a70a1513d53c35bf55bd5c3b2165
Cr-Commit-Position: refs/heads/master@{#407582}