Update Chrome Upstream flow to reflect new UI mocks

chrome/browser/ui/autofill/save_card_bubble_controller_impl.h

Index: chrome/browser/ui/autofill/save_card_bubble_controller_impl.h
diff --git a/chrome/browser/ui/autofill/save_card_bubble_controller_impl.h b/chrome/browser/ui/autofill/save_card_bubble_controller_impl.h
index aa633492bd0fbe63a7d6b39086756a216e9014ae..86e12a26616232a1940099d68b6adda3dbd864e9 100644
--- a/chrome/browser/ui/autofill/save_card_bubble_controller_impl.h
+++ b/chrome/browser/ui/autofill/save_card_bubble_controller_impl.h
@@ -66,6 +66,7 @@ class SaveCardBubbleControllerImpl
   void OnBubbleClosed() override;
 
   const LegalMessageLines& GetLegalMessageLines() const override;
+  void SetShowUploadConfirmTitle(bool show_upload_confirm_title) override;
 
   // Used to check if an entered CVC value is a valid CVC for the current card.
   // Valid CVCs are a certain length for their card type (4 for AMEX, 3
@@ -116,6 +117,10 @@ class SaveCardBubbleControllerImpl
   // The value of the CVC entered by the user (if it was requested).
   base::string16 cvc_entered_by_user_;
 
+  // Whether the normal title or the "Confirm [card]" title should be shown
+  // during upload save.
+  bool show_upload_confirm_title_{false};

[msw, 2017/06/29 20:10:24]

nit: this curly-bracket init style is very uncommon for primitive members in the
codebase (afaict); consider using | = false;| here and elsewhere in this file,
unless I missed some very new style guidance on this topic.

[Jared Saul, 2017/07/05 19:36:50]

I can't recall/find who originally told me to initialize them in this way, just
the request to do it here instead of in the ctor.  Changed to | = false;|.

+
   // Contains the details of the card that will be saved if the user accepts.
   CreditCard card_;