Implement newly saved card bubble for realz and update generated card bubble to

chrome/browser/ui/autofill/new_credit_card_bubble.h

+// Copyright 2013 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.
+
+#ifndef CHROME_BROWSER_UI_AUTOFILL_NEW_CREDIT_CARD_BUBBLE_H_
+#define CHROME_BROWSER_UI_AUTOFILL_NEW_CREDIT_CARD_BUBBLE_H_
+
+#include "base/memory/weak_ptr.h"
+
+namespace autofill {
+
+class NewCreditCardBubbleController;
+
+////////////////////////////////////////////////////////////////////////////////

[Evan Stade, 2013/08/08 22:43:20]

This comment block is overkill, why not just

// class level comment lorem ipsum ...
// ...
class Foo {

};

[Dan Beam, 2013/08/09 01:47:58]

Done.

+//
+// NewCreditCardBubble
+//
+//  A cross-platform interface for a bubble that is shown when a new credit card
+//  is saved. Points at the settings menu in the toolbar and hides when
+//  activation is lost.
+//
+////////////////////////////////////////////////////////////////////////////////
+class NewCreditCardBubble {

[Evan Stade, 2013/08/08 22:43:20]

ditto on naming convention.

[Dan Beam, 2013/08/09 01:47:58]

Done.

+ public:
+  virtual ~NewCreditCardBubble();
+
+  // Visually reveals the bubble.
+  virtual void Show() = 0;
+
+  // Hides the bubble from view.
+  virtual void Hide() = 0;
+
+  // Creates a bubble that's operated by |controller| and owns itself.
+  // |controller| may be invalid while the bubble is closing.
+  static base::WeakPtr<NewCreditCardBubble> Create(
+      const base::WeakPtr<NewCreditCardBubbleController>& controller);
+};
+
+}  // namespace autofill
+
+#endif  // CHROME_BROWSER_UI_AUTOFILL_NEW_CREDIT_CARD_BUBBLE_H_