deps: update v8_inspector

[targos]

Checklist

• make -j4 test (UNIX) or vcbuild test nosign (Windows) passes
• the commit message follows commit guidelines

Description of change

Pick up latest commit of v8_inspector and inspector_protocol.
This brings back compatibility with V8 5.1.

R= @ofrobots

[cjihrig]

LGTM

[mscdex]

Some questions/notes:

1. What's with the whitespace additions?
2. The associated comment for the #endif needs to be updated.
3. Why is this labeled dont-land-on-v6.x? /cc @evanlucas

[targos]

1. The whitespaces are in the original files
2. I agree with you and that's the change I proposed but another CL was merged before mine without this update (diff)

[ofrobots]

1. This is an update of the dependencies; and the whitespace exists upstream. It shows up as a diff in this patch because we have a process to land PRs with --whitespace=fix. This will show up in future PRs too. Perhaps we should land deps-only PRs without whitespace fixup.
2. @targos: Sorry for the miscommunication. I think you should update your CL to fix the comment and still land it upstream. However, I do not think we should have to wait for the comment update before landing this PR. The comment will be picked up in the next roll of the dependency.

[evanlucas]

@mscdex I marked it dont-land-on-v6.x because the original pr has not landed on v6.x and it makes it easier to reason about commits to cherry-pick for releases. That being said, I am pushing for the next v6.x release to include it and would then remove the label.

[ofrobots]

LGTM. CI: https://ci.nodejs.org/job/node-test-pull-request/3089/

[targos]

@ofrobots thanks. Maybe you could help with https://codereview.chromium.org/2084343002/. The buildbot didn't use the email address I want.

[targos]

landed in ecd639a