DevTools: Fix tainted Function.prototype methods may disable console.

DevTools: Fix tainted Function.prototype methods may disable console.

BUG=401535
R=pfeldman, yurys

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=179839

[yurys, 2014-08-07 16:54:03]

lgtm

https://codereview.chromium.org/445333005/diff/20001/Source/bindings/core/v8/...
File Source/bindings/core/v8/custom/V8InjectedScriptHostCustom.cpp (right):

https://codereview.chromium.org/445333005/diff/20001/Source/bindings/core/v8/...
Source/bindings/core/v8/custom/V8InjectedScriptHostCustom.cpp:483:
ASSERT_NOT_REACHED();
In theory InjectedScriptHost can leak into the page script in which case the
page can pass whatever it wants into this method. But probably it's fine to
leave the assert to check valid cases in the tests.

[aandrey, 2014-08-07 16:57:50]

The CQ bit was checked by aandrey@chromium.org

[commit-bot: I haz the power, 2014-08-07 16:59:10]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/aandrey@chromium.org/445333005/40001

[commit-bot: I haz the power, 2014-08-07 18:35:30]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_blink_compile_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/android_blink_compile_db...)
  linux_blink_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/2...)
  win_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu/builds/43815)

[commit-bot: I haz the power, 2014-08-07 20:05:01]

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

[commit-bot: I haz the power, 2014-08-07 20:05:02]

Try jobs failed on following builders:
  linux_blink_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/2...)

[aandrey, 2014-08-07 20:33:57]

The CQ bit was checked by aandrey@chromium.org

[commit-bot: I haz the power, 2014-08-07 20:35:37]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/aandrey@chromium.org/445333005/40001

[commit-bot: I haz the power, 2014-08-07 21:46:15]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_blink_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/2...)

[commit-bot: I haz the power, 2014-08-07 22:51:09]

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

[commit-bot: I haz the power, 2014-08-07 22:51:11]

Try jobs failed on following builders:
  linux_blink_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/2...)

[aandrey, 2014-08-08 07:09:53]

The CQ bit was checked by aandrey@chromium.org

[commit-bot: I haz the power, 2014-08-08 07:10:31]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/aandrey@chromium.org/445333005/40001

[commit-bot: I haz the power, 2014-08-08 08:28:06]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_blink_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/2...)

[commit-bot: I haz the power, 2014-08-08 09:50:36]

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

[commit-bot: I haz the power, 2014-08-08 09:50:37]

Try jobs failed on following builders:
  linux_blink_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/2...)

[aandrey, 2014-08-08 10:23:36]

The CQ bit was checked by aandrey@chromium.org

[commit-bot: I haz the power, 2014-08-08 10:24:18]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/aandrey@chromium.org/445333005/40001

[aandrey, 2014-08-08 10:30:11]

The CQ bit was unchecked by aandrey@chromium.org

[aandrey, 2014-08-08 12:33:06]

The CQ bit was checked by aandrey@chromium.org

[commit-bot: I haz the power, 2014-08-08 12:34:00]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/aandrey@chromium.org/445333005/60001

[yurys, 2014-08-08 12:34:36]

PS#3 - lgtm

[aandrey, 2014-08-08 12:38:15]

The CQ bit was unchecked by aandrey@chromium.org

[aandrey, 2014-08-08 12:40:52]

The CQ bit was checked by aandrey@chromium.org

[commit-bot: I haz the power, 2014-08-08 12:42:00]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/aandrey@chromium.org/445333005/80001

[commit-bot: I haz the power, 2014-08-08 14:05:34]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/1...)

[commit-bot: I haz the power, 2014-08-08 15:10:08]

Change committed as 179839