EarlGrey tests for Payment Request (base CL)

ios/chrome/browser/payments/payment_request_manager.mm

 // 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.
 
 #import "ios/chrome/browser/payments/payment_request_manager.h"
 
 #include "base/ios/ios_util.h"
 #import "base/ios/weak_nsobject.h"
 #import "base/mac/bind_objc_block.h"
 #include "base/mac/foundation_util.h"
 #include "base/mac/scoped_nsobject.h"
 #include "base/memory/ptr_util.h"
 #include "base/strings/sys_string_conversions.h"
 #import "base/values.h"
 #include "components/autofill/core/browser/personal_data_manager.h"
 #include "ios/chrome/browser/autofill/personal_data_manager_factory.h"
 #include "ios/chrome/browser/browser_state/chrome_browser_state.h"
+#include "ios/chrome/browser/experimental_flags.h"
 #import "ios/chrome/browser/payments/js_payment_request_manager.h"
 #import "ios/chrome/browser/payments/payment_request_coordinator.h"
 #import "ios/chrome/browser/payments/payment_request_web_state_observer.h"
 #include "ios/web/public/favicon_status.h"
 #include "ios/web/public/navigation_item.h"
 #include "ios/web/public/navigation_manager.h"
 #include "ios/web/public/payments/payment_request.h"
 #include "ios/web/public/ssl_status.h"
 #import "ios/web/public/url_scheme_util.h"
 #import "ios/web/public/web_state/crw_web_view_proxy.h"
@@ skipping 88 matching lines @@
 
 - (void)setWebState:(web::WebState*)webState {
   [self disconnectWebState];
   if (webState) {
     _paymentRequestJsManager.reset(
         base::mac::ObjCCastStrict<JSPaymentRequestManager>(
             [webState->GetJSInjectionReceiver()
                 instanceOfClass:[JSPaymentRequestManager class]]));
     _webState = webState;
     _webStateObserver->ObserveWebState(webState);
-    [self enableCurrentWebState];

[Justin Donnelly, 2016/12/29 18:36:57]

I'm skeptical that this is ok. The web state has a rather complex life cycle I
believe based on navigations, redirects, going back, reloads, etc. I copied this
particular setup from another JS injected feature and I assume it was designed
carefully to work in all circumstances. Maybe lpromero knows or else we can
track down someone who can better answer than I can if this will work in all
cases.

[Moe, 2017/01/06 16:16:27]

lpromero@ I was wondering if you had any feedback on the point justin made.

[lpromero, 2017/01/06 16:20:01]

eugenebut@ would probably be the right person to ask.

[Moe, 2017/01/06 16:25:21]

eugenebut@ could please chime in?

[Eugene But (OOO till 7-30), 2017/01/06 17:36:22]

I don't know what |enableCurrentWebState| does. This method has no comment. 

[Justin Donnelly, 2017/01/06 18:21:07]

I copied the existing pattern of calls and state transitions from
contextual_search_controller.mm. marq@, did you make that? Can you comment on
whether the new setup mahmadi@ is making here will work in all circumstances?

[marq (ping after 24h), 2017/01/09 17:55:38]

I'll have to look over it carefully; the use cases for
ContextualSearchController are fairly specific.

   } else {
     _webState = nullptr;
   }
 }
 
 - (void)enablePaymentRequest:(BOOL)enabled {
   // Asynchronously enables PaymentRequest, so that some preferences
   // (UIAccessibilityIsVoiceOverRunning(), for example) have time to synchronize
   // with their own notifications.
   base::WeakNSObject<PaymentRequestManager> weakSelf(self);
   dispatch_async(dispatch_get_main_queue(), ^{
     [weakSelf doEnablePaymentRequest:enabled];
   });
 }
 
 - (void)doEnablePaymentRequest:(BOOL)enabled {
   BOOL changing = _enabled != enabled;
   if (changing) {
     if (!enabled) {
       [self dismissUI];
     }
     _enabled = enabled;
-    [self enableCurrentWebState];
   }
 }
 
 - (void)cancelRequest {
   [self dismissUI];
   [_paymentRequestJsManager rejectRequestPromise:@"Request cancelled by user."
                                completionHandler:nil];
 }
 
 - (void)close {
   if (_closed)
     return;
 
   _closed = YES;
   [self disableCurrentWebState];
   [self setWebState:nil];
   [self dismissUI];
 }
 
 - (void)enableCurrentWebState {
-  if (![self webState]) {
+  if (![self webState] || !experimental_flags::IsPaymentRequestEnabled()) {
     return;
   }
 
   if (_enabled && [self webStateContentIsSecureHTML]) {
     if (!_webStateEnabled) {
       base::WeakNSObject<PaymentRequestManager> weakSelf(self);
       auto callback =
           base::BindBlock(^bool(const base::DictionaryValue& JSON,
                                 const GURL& originURL, bool userIsInteracting) {
             base::scoped_nsobject<PaymentRequestManager> strongSelf(
@@ skipping 148 matching lines @@
 - (void)pageLoadedWithStatus:(web::PageLoadCompletionStatus)loadStatus {
   if (loadStatus != web::PageLoadCompletionStatus::SUCCESS)
     return;
 
   [self dismissUI];
   _isScriptInjected = NO;
   [self enableCurrentWebState];
 }
 
 @end