| commit | 2254f0eb773a2560b789ed8ecacc11dc74bad7e0 | |
| author | lazyboy <lazyboy@chromium.org> | |
| Wed, 10 Dec 2014 07:10:54 +0000 (9 23:10 -0800) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Wed, 10 Dec 2014 07:11:13 +0000 (10 07:11 +0000) | ||
| tree | 0170962de795394cd5423ab9c2825c5a3cf78d13 | treesnapshot (tar.gz zip) |
| parent | feffe1dd547ee7b5c16d38784cd0cd679dfd7850 | commitdiff |
Fix injecting script into all frames within <webview>.
This broke after we added the check to match frame's src against
<webview>'s src.
We shouldn't be doing this for non top level frames.
This CL also makes skipping multiple loadstop-s from utils.js configurable, the option can be truned on by setting
config.SKIP_MULTIPLE_LOADSTOP = true;
BUG=433948
Test=<webview>.executeScript with allFrames: true should inject script into all frames
Review URL: https://codereview.chromium.org/792603002
Cr-Commit-Position: refs/heads/master@{#307651}
This broke after we added the check to match frame's src against
<webview>'s src.
We shouldn't be doing this for non top level frames.
This CL also makes skipping multiple loadstop-s from utils.js configurable, the option can be truned on by setting
config.SKIP_MULTIPLE_LOADSTOP = true;
BUG=433948
Test=<webview>.executeScript with allFrames: true should inject script into all frames
Review URL: https://codereview.chromium.org/792603002
Cr-Commit-Position: refs/heads/master@{#307651}