Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(47)

Issue 2430923007: Disable framebusting intervention (Closed)

Created:
4 years, 2 months ago by Nate Chapin
Modified:
4 years, 2 months ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/pending/branch-heads/2883
Project:
chromium
Visibility:
Public.

Description

Disable framebusting intervention As mentioned in the Intent thread: https://groups.google.com/a/chromium.org/d/msg/blink-dev/Xi8-y4ySjA4/9_96y_F6CAAJ This is breaking enough legitimate content that we want to rethink the strategy. https://codereview.chromium.org/2392773002 is the work-in-process for that. BUG=624061 Review-Url: https://codereview.chromium.org/2413513004 Cr-Commit-Position: refs/heads/master@{#425733} (cherry picked from commit b0646f36c67c5cda685755ad15a7570c16fc0855) Committed: https://chromium.googlesource.com/chromium/src/+/02e880045c644e6a73c0a550d0870dd72df0555e

Patch Set 1 #

Patch Set 2 : Disable framebusting intervention #

Patch Set 3 : Disable framebusting intervention #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -2 lines) Patch
M content/public/common/content_features.cc View 1 chunk +1 line, -1 line 0 comments Download
M third_party/WebKit/LayoutTests/http/tests/security/frameNavigation/xss-DENIED-top-navigation-without-user-gesture.html View 1 chunk +2 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (1 generated)
Nate Chapin
4 years, 2 months ago (2016-10-19 21:39:48 UTC) #2
Message was sent while issue was closed.
Committed patchset #3 (id:40001) manually as
02e880045c644e6a73c0a550d0870dd72df0555e (tree was closed).

Powered by Google App Engine
This is Rietveld 408576698