Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #ifndef CHROME_BROWSER_UI_COCOA_EXTENSIONS_EXTENSION_INSTALL_DIALOG_CONTROLER_H_ | 5 #ifndef CHROME_BROWSER_UI_COCOA_EXTENSIONS_EXTENSION_INSTALL_DIALOG_CONTROLER_H_ |
| 6 #define CHROME_BROWSER_UI_COCOA_EXTENSIONS_EXTENSION_INSTALL_DIALOG_CONTROLER_H_ | 6 #define CHROME_BROWSER_UI_COCOA_EXTENSIONS_EXTENSION_INSTALL_DIALOG_CONTROLER_H_ |
| 7 | 7 |
| 8 #import <Cocoa/Cocoa.h> | 8 #import <Cocoa/Cocoa.h> |
| 9 | 9 |
| 10 #include "base/mac/scoped_nsobject.h" | 10 #include "base/mac/scoped_nsobject.h" |
| 11 #include "chrome/browser/extensions/extension_install_prompt.h" | 11 #include "chrome/browser/extensions/extension_install_prompt.h" |
| 12 #import "chrome/browser/ui/cocoa/constrained_window/constrained_window_mac.h" | 12 #import "chrome/browser/ui/cocoa/constrained_window/constrained_window_mac.h" |
| 13 | 13 |
| 14 namespace content { | 14 namespace content { |
| 15 class PageNavigator; | 15 class PageNavigator; |
| 16 class WebContents; | 16 class WebContents; |
| 17 } | 17 } |
| 18 | 18 |
| 19 namespace extensions { | |
| 20 class ExperienceSamplingEvent; | |
| 21 } | |
| 22 | |
| 19 @class ExtensionInstallViewController; | 23 @class ExtensionInstallViewController; |
| 20 | 24 |
| 21 // Displays an extension install prompt as a tab modal dialog. | 25 // Displays an extension install prompt as a tab modal dialog. |
| 22 class ExtensionInstallDialogController : | 26 class ExtensionInstallDialogController : |
| 23 public ExtensionInstallPrompt::Delegate, | 27 public ExtensionInstallPrompt::Delegate, |
| 24 public ConstrainedWindowMacDelegate { | 28 public ConstrainedWindowMacDelegate { |
| 25 public: | 29 public: |
| 26 ExtensionInstallDialogController( | 30 ExtensionInstallDialogController( |
| 27 const ExtensionInstallPrompt::ShowParams& show_params, | 31 const ExtensionInstallPrompt::ShowParams& show_params, |
| 28 ExtensionInstallPrompt::Delegate* delegate, | 32 ExtensionInstallPrompt::Delegate* delegate, |
| (...skipping 12 matching lines...) Expand all Loading... | |
| 41 return constrained_window_.get(); | 45 return constrained_window_.get(); |
| 42 } | 46 } |
| 43 ExtensionInstallViewController* view_controller() const { | 47 ExtensionInstallViewController* view_controller() const { |
| 44 return view_controller_; | 48 return view_controller_; |
| 45 } | 49 } |
| 46 | 50 |
| 47 private: | 51 private: |
| 48 ExtensionInstallPrompt::Delegate* delegate_; | 52 ExtensionInstallPrompt::Delegate* delegate_; |
| 49 base::scoped_nsobject<ExtensionInstallViewController> view_controller_; | 53 base::scoped_nsobject<ExtensionInstallViewController> view_controller_; |
| 50 scoped_ptr<ConstrainedWindowMac> constrained_window_; | 54 scoped_ptr<ConstrainedWindowMac> constrained_window_; |
| 55 scoped_ptr<extensions::ExperienceSamplingEvent> sampling_event_; | |
|
groby-ooo-7-16
2014/08/14 18:00:05
Question - why does this have object lifetime? Sho
Chris Thompson
2014/08/14 18:06:20
We want to also track when the UI was created/show
| |
| 51 }; | 56 }; |
| 52 | 57 |
| 53 #endif // CHROME_BROWSER_UI_COCOA_EXTENSIONS_EXTENSION_INSTALL_DIALOG_CONTROLLE R_H_ | 58 #endif // CHROME_BROWSER_UI_COCOA_EXTENSIONS_EXTENSION_INSTALL_DIALOG_CONTROLLE R_H_ |
| OLD | NEW |