diff options
| author | jkummerow <jkummerow@chromium.org> | 2015-10-26 06:03:54 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-10-26 13:04:40 +0000 |
| commit | a9914da1b0b8710b2f500c735e29953c3e56db97 (patch) | |
| tree | 9a0932c6c2305fd6d26a5ad9641eca111a513f5f /extensions/renderer/resources | |
| parent | 8f634d3856742057df389c131f6c9fc1a2a3a543 (diff) | |
| download | chromium_src-a9914da1b0b8710b2f500c735e29953c3e56db97.zip chromium_src-a9914da1b0b8710b2f500c735e29953c3e56db97.tar.gz chromium_src-a9914da1b0b8710b2f500c735e29953c3e56db97.tar.bz2 | |
Platform Apps: Disable history.state in platform_app.js
This is a follow-up to https://codereview.chromium.org/988293002 and makes sure
that all blacklisted getters are handled uniformly. The corresponding test
(chrome/test/data/extensions/platform_apps/restrictions/main.js) already asserts
that history.state is inaccessible.
R=haraken@chromium.org
BUG=547116
Review URL: https://codereview.chromium.org/1415453003
Cr-Commit-Position: refs/heads/master@{#356040}
Diffstat (limited to 'extensions/renderer/resources')
| -rw-r--r-- | extensions/renderer/resources/platform_app.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/extensions/renderer/resources/platform_app.js b/extensions/renderer/resources/platform_app.js index caf935a..21263d3 100644 --- a/extensions/renderer/resources/platform_app.js +++ b/extensions/renderer/resources/platform_app.js @@ -152,7 +152,7 @@ disableMethods(HTMLDocument.prototype, 'document', ['write', 'writeln'], true); // Disable history. Object.defineProperty(window, "history", { value: {} }); disableGetters(window.history, 'history', - ['back', 'forward', 'go', 'length', 'pushState', 'replaceState']); + ['back', 'forward', 'go', 'length', 'pushState', 'replaceState', 'state']); // Disable find. disableMethods(window, 'window', ['find']); |