Remove uses of FontList::Derive*(..) in views payments dialogs.

chrome/browser/ui/views/payments/payment_request_browsertest.cc

 // Copyright 2017 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 <vector>
 
 #include "base/macros.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/ui/browser_commands.h"
+#include "chrome/browser/ui/test/test_browser_dialog.h"
 #include "chrome/browser/ui/views/payments/payment_request_browsertest_base.h"
 #include "chrome/browser/ui/views/payments/payment_request_dialog_view_ids.h"
 #include "chrome/test/base/ui_test_utils.h"
 #include "components/autofill/core/browser/autofill_profile.h"
 #include "components/autofill/core/browser/autofill_test_utils.h"
 #include "components/autofill/core/browser/credit_card.h"
 #include "components/payments/content/payment_request.h"
 #include "components/payments/content/payment_request_web_contents_manager.h"
 #include "components/web_modal/web_contents_modal_dialog_manager.h"
 #include "content/public/test/browser_test_utils.h"
@@ skipping 316 matching lines @@
       GetPaymentRequests(GetActiveWebContents());
   EXPECT_EQ(1u, requests.size());
   std::vector<std::string> supported_card_networks =
       requests[0]->spec()->supported_card_networks();
   EXPECT_EQ(3u, supported_card_networks.size());
   EXPECT_EQ("mastercard", supported_card_networks[0]);
   EXPECT_EQ("visa", supported_card_networks[1]);
   EXPECT_EQ("jcb", supported_card_networks[2]);
 }
 
+// Test harness integrating with DialogBrowserTest to present the dialog in an
+// interactive manner for visual testing.
+class PaymentsRequestVisualTest
+    : public SupportsTestDialog<PaymentRequestNoShippingTest> {
+ protected:
+  PaymentsRequestVisualTest() {}
+
+  // TestBrowserDialog:
+  void ShowDialog(const std::string& name) override {
+    InvokePaymentRequestUI();
+  }
+
+  bool AlwaysCloseAsynchronously() override {
+    // Bypassing Widget::CanClose() causes payments::JourneyLogger to see the
+    // show, but not the close, resulting in a DCHECK in its destructor.
+    return true;
+  }
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(PaymentsRequestVisualTest);
+};
+
+IN_PROC_BROWSER_TEST_F(PaymentsRequestVisualTest, InvokeDialog_NoShipping) {
+  RunDialog();
+}
+
 }  // namespace payments