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

Issue 2529423002: [Merge M-56] Mark IFrame support in PaymentRequest experimental (Closed)

Created:
4 years ago by please use gerrit instead
Modified:
4 years ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/pending/branch-heads/2924
Project:
chromium
Visibility:
Public.

Description

[Merge M-56] Mark IFrame support in PaymentRequest experimental IFrame support in PaymentRequest may be better served through FeaturePolicy. This patch marks the current IFrame support experimental, so it's disabled by default, but developers still can experiment with it. Once the FeaturePolicy question is resolved, the current implementation of the IFrame support will be either shipped or replaced by FeaturePolicy. BUG=652148 Review-Url: https://codereview.chromium.org/2525813003 Cr-Commit-Position: refs/heads/master@{#434257} (cherry picked from commit d2c4379ec3783f7f70b14f6fda92f13cd496bf66) Committed: https://chromium.googlesource.com/chromium/src/+/b0db3df9ef642ede2732fe6b4f357a3b7960deff

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -4 lines) Patch
M third_party/WebKit/Source/modules/payments/HTMLIFrameElementPayments.cpp View 2 chunks +3 lines, -1 line 0 comments Download
M third_party/WebKit/Source/modules/payments/HTMLIFrameElementPayments.idl View 1 chunk +1 line, -1 line 0 comments Download
M third_party/WebKit/Source/modules/payments/PaymentRequest.cpp View 2 chunks +5 lines, -2 lines 0 comments Download
M third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 2 (1 generated)
please use gerrit instead
4 years ago (2016-11-28 15:07:30 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
b0db3df9ef642ede2732fe6b4f357a3b7960deff.

Powered by Google App Engine
This is Rietveld 408576698