Switch SupportsUserData uses to use unique_ptr.

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 2b924ab87d7a7a865ff9ab81c503911c09bf5fc0..aa633492bd0fbe63a7d6b39086756a216e9014ae 100644
--- a/chrome/browser/ui/autofill/save_card_bubble_controller_impl.h
+++ b/chrome/browser/ui/autofill/save_card_bubble_controller_impl.h
@@ -25,6 +25,8 @@ class SaveCardBubbleControllerImpl
       public content::WebContentsObserver,
       public content::WebContentsUserData<SaveCardBubbleControllerImpl> {
  public:
+  ~SaveCardBubbleControllerImpl() override;

[sky, 2017/05/01 17:34:27]

Move definition to match new position? (same comment applies to a couple of the
other files).

[Avi (use Gerrit), 2017/05/02 01:31:35]

Was aiming to avoid churn. Also, the order in the .cc file doesn't even match
the .h currently so there's not much I can do without rearranging the whole
file.

+
   // Sets up the controller for local save and shows the bubble.
   // |save_card_callback| will be invoked if and when the Save button is
   // pressed.
@@ -72,7 +74,6 @@ class SaveCardBubbleControllerImpl
 
  protected:
   explicit SaveCardBubbleControllerImpl(content::WebContents* web_contents);
-  ~SaveCardBubbleControllerImpl() override;
 
   // Returns the time elapsed since |timer_| was initialized.
   // Exists for testing.