Mark IFrame support in PaymentRequest experimental

third_party/WebKit/Source/modules/payments/HTMLIFrameElementPayments.cpp

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "modules/payments/HTMLIFrameElementPayments.h"
 
 #include "core/dom/QualifiedName.h"
 #include "core/html/HTMLIFrameElement.h"
+#include "platform/RuntimeEnabledFeatures.h"
 
 namespace blink {
 
 HTMLIFrameElementPayments::HTMLIFrameElementPayments() {}
 
 // static
 const char* HTMLIFrameElementPayments::supplementName() {
   return "HTMLIFrameElementPayments";
 }
 
@@ skipping 22 matching lines @@
   if (!supplement) {
     supplement = new HTMLIFrameElementPayments();
     provideTo(iframe, supplementName(), supplement);
   }
   return *supplement;
 }
 
 // static
 bool HTMLIFrameElementPayments::allowPaymentRequest(
     HTMLIFrameElement& element) {
-  return element.fastHasAttribute(HTMLNames::allowpaymentrequestAttr);
+  return RuntimeEnabledFeatures::paymentRequestIFrameEnabled() &&

[Rick Byers, 2016/11/23 16:56:43]

It seems like it would be nice if the bindings generator generated a method like
this automatically (so we don't have to remember to add an additional
RuntimeEnabledFeatures check), but putting the check here explicitly is nice and
simple and definitely the right place to have it.  Thanks!

[please use gerrit instead, 2016/11/23 17:27:36]

Acknowledged.

+         element.fastHasAttribute(HTMLNames::allowpaymentrequestAttr);
 }
 
 DEFINE_TRACE(HTMLIFrameElementPayments) {
   Supplement<HTMLIFrameElement>::trace(visitor);
 }
 
 }  // namespace blink