DevTools: do not rely upon reported context having accessible injected script.

DevTools: do not rely upon reported context having accessible injected script.

NOTRY=true

Committed: https://crrev.com/26d9773b1b7d22c44f960335194b441dc58babce
Cr-Commit-Position: refs/heads/master@{#371621}

[pfeldman, 2016-01-26 19:12:08]

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

[dgozman, 2016-01-26 19:41:23]

Should we do similar checks in other places, e.g. DOM agent?

lgtm

https://codereview.chromium.org/1635173002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/inspector/v8/V8RuntimeAgentImpl.cpp (right):

https://codereview.chromium.org/1635173002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/inspector/v8/V8RuntimeAgentImpl.cpp:220:
.setOrigin(origin
Awkward formatting.

[pfeldman, 2016-01-26 20:05:50]

The CQ bit was checked by pfeldman@chromium.org

[pfeldman, 2016-01-26 20:05:50]

The patchset sent to the CQ was uploaded after l-g-t-m from dgozman@chromium.org
Link to the patchset: https://codereview.chromium.org/1635173002/#ps20001
(title: "changed keyboard")

[commit-bot: I haz the power, 2016-01-26 20:07:12]

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

[commit-bot: I haz the power, 2016-01-26 20:19:23]

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

[commit-bot: I haz the power, 2016-01-26 20:19:24]

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_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[pfeldman, 2016-01-26 22:27:54]

Description was changed from

==========
DevTools: do not rely upon reported context having accessible injected script.
==========

to

==========
DevTools: do not rely upon reported context having accessible injected script.

NOTRY=true
==========

[pfeldman, 2016-01-26 22:27:58]

The CQ bit was checked by pfeldman@chromium.org

[commit-bot: I haz the power, 2016-01-26 22:29:16]

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

[commit-bot: I haz the power, 2016-01-26 22:37:53]

Description was changed from

==========
DevTools: do not rely upon reported context having accessible injected script.

NOTRY=true
==========

to

==========
DevTools: do not rely upon reported context having accessible injected script.

NOTRY=true
==========

[commit-bot: I haz the power, 2016-01-26 22:37:53]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-26 22:39:25]

Description was changed from

==========
DevTools: do not rely upon reported context having accessible injected script.

NOTRY=true
==========

to

==========
DevTools: do not rely upon reported context having accessible injected script.

NOTRY=true

Committed: https://crrev.com/26d9773b1b7d22c44f960335194b441dc58babce
Cr-Commit-Position: refs/heads/master@{#371621}
==========

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

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