Autofill: First step toward CVC unmask prompt dialog on OSX.

chrome/browser/ui/cocoa/autofill/card_unmask_prompt_cocoa.h

+// Copyright (c) 2015 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_COCOA_CARD_UNMASK_PROMPT_COCOA_H_
+#define CHROME_BROWSER_UI_COCOA_CARD_UNMASK_PROMPT_COCOA_H_
+
+#include "base/mac/scoped_nsobject.h"
+#include "base/memory/scoped_ptr.h"
+#include "chrome/browser/ui/autofill/card_unmask_prompt_view.h"
+#include "chrome/browser/ui/cocoa/constrained_window/constrained_window_mac.h"
+
+namespace content {
+class NavigationController;
+}
+
+@class CardUnmaskPromptWindowController;
+@class GTMWidthBasedTweaker;
+
+namespace autofill {
+
+class CardUnmaskPromptCocoa : public CardUnmaskPromptView,
+                              public ConstrainedWindowMacDelegate {
+ public:
+  explicit CardUnmaskPromptCocoa(CardUnmaskPromptController* controller);
+  ~CardUnmaskPromptCocoa() override;
+
+  // CardUnmaskPromptView implementation:
+  void ControllerGone() override;
+  void DisableAndWaitForVerification() override;
+  void GotVerificationResult(bool success) override;
+
+  // ConstrainedWindowMacDelegate implementation:
+  void OnConstrainedWindowClosed(ConstrainedWindowMac* window) override;
+
+  void PerformClose();
+
+ private:
+  scoped_ptr<ConstrainedWindowMac> constrained_window_;
+  base::scoped_nsobject<CardUnmaskPromptWindowController> sheet_controller_;
+
+  // The controller |this| queries for logic and state.
+  CardUnmaskPromptController* controller_;
+};
+
+}  // autofill
+
+@interface CardUnmaskPromptWindowController : NSWindowController
+                                              <NSWindowDelegate> {

[groby-ooo-7-16, 2015/02/06 03:06:18]

That's odd formatting - git cl format?

[bondd, 2015/02/09 19:21:00]

Done. Applied 'git cl format' to entire CL.

+ @private
+  content::WebContents* webContents_;  // weak.
+  autofill::CardUnmaskPromptCocoa* cardUnmaskPrompt_;  // weak.

[groby-ooo-7-16, 2015/02/06 03:06:18]

I'd suggest calling it cardUnmaskPromptBridge_ since it bridges between the C++
and the ObjC side.

[bondd, 2015/02/09 19:21:00]

Done.

+
+  base::scoped_nsobject<GTMWidthBasedTweaker> buttonContainer_;

[groby-ooo-7-16, 2015/02/06 03:06:18]

Do you need a tweaker?

[bondd, 2015/02/09 19:21:00]

This is the same way AutofillMainContainer does it:
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/ui/...

From what I can tell this will be used to tweak button sizes for l10n. If that's
the case I'd like to leave it like this for now, instead of changing to a
different container only to switch back later. If I misunderstand or you'd still
like a different class than GTMWidthBasedTweaker used here, please let me know
which class you'd prefer. Thanks! :)

[groby-ooo-7-16, 2015/02/09 20:25:02]

It is, but by now I consider it overkill for just two buttons :) I'll not scream
bloody murder for leaving it in, though. 

(This stuff usually makes more sense for complex layouts, see:
https://code.google.com/p/google-toolbox-for-mac/wiki/UILocalization)

[bondd, 2015/02/09 23:07:32]

Done. I understand what you mean now. Changed it to an NSView.

+}
+
+// Designated initializer. The WebContents cannot be NULL.

[groby-ooo-7-16, 2015/02/06 03:06:18]

|webContents| cannot...

[bondd, 2015/02/09 19:21:00]

Done.

+- (id)initWithWebContents:(content::WebContents*)webContents
+      cardUnmaskPrompt:(autofill::CardUnmaskPromptCocoa*)cardUnmaskPrompt;

[groby-ooo-7-16, 2015/02/06 03:06:18]

Align ":"

[bondd, 2015/02/09 19:21:00]

Done.

+
+// Closes the sheet and ends the modal loop. This will also clean up the memory.

[groby-ooo-7-16, 2015/02/06 03:06:18]

Which memory?

[bondd, 2015/02/09 19:21:00]

Done.

+- (IBAction)closeSheet:(id)sender;
+
+@end
+
+#endif  // CHROME_BROWSER_UI_COCOA_CARD_UNMASK_PROMPT_COCOA_H_