Index: chrome/browser/ui/cocoa/website_settings/chooser_bubble_ui_cocoa.mm |
diff --git a/chrome/browser/ui/cocoa/website_settings/chooser_bubble_ui_cocoa.mm b/chrome/browser/ui/cocoa/website_settings/chooser_bubble_ui_cocoa.mm |
index f54bbbda7f9b9fe4d9ed455acd61549df22f5983..92f2d48b65de7160896252c11e117540d3a3acfa 100644 |
--- a/chrome/browser/ui/cocoa/website_settings/chooser_bubble_ui_cocoa.mm |
+++ b/chrome/browser/ui/cocoa/website_settings/chooser_bubble_ui_cocoa.mm |
@@ -67,13 +67,12 @@ scoped_ptr<BubbleUi> ChooserBubbleDelegate::BuildBubbleUi() { |
base::scoped_nsobject<NSButton> cancelButton_; |
Browser* browser_; // Weak. |
- ChooserBubbleDelegate* chooser_bubble_delegate_; // Weak. |
+ ChooserBubbleDelegate* chooserBubbleDelegate_; // Weak. |
} |
// Designated initializer. |browser| and |bridge| must both be non-nil. |
- (id)initWithBrowser:(Browser*)browser |
- initWithChooserBubbleDelegate: |
- (ChooserBubbleDelegate*)chooser_bubble_delegate |
+ initWithChooserBubbleDelegate:(ChooserBubbleDelegate*)chooserBubbleDelegate |
bridge:(ChooserBubbleUiCocoa*)bridge; |
// Makes the bubble visible. |
@@ -132,18 +131,14 @@ scoped_ptr<BubbleUi> ChooserBubbleDelegate::BuildBubbleUi() { |
@implementation ChooserBubbleUiController |
- (id)initWithBrowser:(Browser*)browser |
- initWithChooserBubbleDelegate: |
- (ChooserBubbleDelegate*)chooser_bubble_delegate |
+ initWithChooserBubbleDelegate:(ChooserBubbleDelegate*)chooserBubbleDelegate |
bridge:(ChooserBubbleUiCocoa*)bridge { |
DCHECK(browser); |
- DCHECK(chooser_bubble_delegate); |
+ DCHECK(chooserBubbleDelegate); |
DCHECK(bridge); |
- if (browser == nil || chooser_bubble_delegate == nil || bridge == nil) |
- return nil; |
- |
browser_ = browser; |
- chooser_bubble_delegate_ = chooser_bubble_delegate; |
+ chooserBubbleDelegate_ = chooserBubbleDelegate; |
base::scoped_nsobject<InfoBubbleWindow> window([[InfoBubbleWindow alloc] |
initWithContentRect:ui::kWindowSizeDeterminedLater |
@@ -169,6 +164,7 @@ scoped_ptr<BubbleUi> ChooserBubbleDelegate::BuildBubbleUi() { |
} |
- (void)windowWillClose:(NSNotification*)notification { |
+ [[NSNotificationCenter defaultCenter] removeObserver:self]; |
bridge_->OnBubbleClosing(); |
[super windowWillClose:notification]; |
} |
@@ -188,7 +184,6 @@ scoped_ptr<BubbleUi> ChooserBubbleDelegate::BuildBubbleUi() { |
- (void)show { |
NSView* view = [[self window] contentView]; |
- [view setSubviews:@[]]; |
// ------------------------------------ |
// | Chooser bubble title | |
@@ -294,7 +289,7 @@ scoped_ptr<BubbleUi> ChooserBubbleDelegate::BuildBubbleUi() { |
- (NSInteger)numberOfRowsInTableView:(NSTableView*)tableView { |
const std::vector<base::string16>& device_names = |
- chooser_bubble_delegate_->GetOptions(); |
+ chooserBubbleDelegate_->GetOptions(); |
if (device_names.empty()) { |
return 1; |
} else { |
@@ -306,7 +301,7 @@ scoped_ptr<BubbleUi> ChooserBubbleDelegate::BuildBubbleUi() { |
objectValueForTableColumn:(NSTableColumn*)tableColumn |
row:(NSInteger)rowIndex { |
const std::vector<base::string16>& device_names = |
- chooser_bubble_delegate_->GetOptions(); |
+ chooserBubbleDelegate_->GetOptions(); |
if (device_names.empty()) { |
DCHECK(rowIndex == 0); |
return l10n_util::GetNSString(IDS_CHOOSER_BUBBLE_NO_DEVICES_FOUND_PROMPT); |
@@ -346,7 +341,7 @@ scoped_ptr<BubbleUi> ChooserBubbleDelegate::BuildBubbleUi() { |
- (void)updateTableView { |
const std::vector<base::string16>& device_names = |
- chooser_bubble_delegate_->GetOptions(); |
+ chooserBubbleDelegate_->GetOptions(); |
[tableView_ setEnabled:!device_names.empty()]; |
[tableView_ reloadData]; |
} |
@@ -363,9 +358,9 @@ scoped_ptr<BubbleUi> ChooserBubbleDelegate::BuildBubbleUi() { |
- (NSPoint)getExpectedAnchorPoint { |
NSPoint anchor; |
if ([self hasLocationBar]) { |
- LocationBarViewMac* location_bar = |
+ LocationBarViewMac* locationBar = |
[[[self getExpectedParentWindow] windowController] locationBarBridge]; |
- anchor = location_bar->GetPageInfoBubblePoint(); |
+ anchor = locationBar->GetPageInfoBubblePoint(); |
} else { |
// Center the bubble if there's no location bar. |
NSRect contentFrame = [[[self getExpectedParentWindow] contentView] frame]; |
@@ -447,12 +442,12 @@ scoped_ptr<BubbleUi> ChooserBubbleDelegate::BuildBubbleUi() { |
- (void)onConnect:(id)sender { |
NSInteger row = [tableView_ selectedRow]; |
- chooser_bubble_delegate_->Select(row); |
+ chooserBubbleDelegate_->Select(row); |
[self close]; |
} |
- (void)onCancel:(id)sender { |
- chooser_bubble_delegate_->Cancel(); |
+ chooserBubbleDelegate_->Cancel(); |
[self close]; |
} |
@@ -460,60 +455,54 @@ scoped_ptr<BubbleUi> ChooserBubbleDelegate::BuildBubbleUi() { |
ChooserBubbleUiCocoa::ChooserBubbleUiCocoa( |
Browser* browser, |
- ChooserBubbleDelegate* chooser_bubble_delegate) |
+ ChooserBubbleDelegate* chooserBubbleDelegate) |
: browser_(browser), |
- chooser_bubble_delegate_(chooser_bubble_delegate), |
- chooser_bubble_ui_controller_(nil) { |
+ chooserBubbleDelegate_(chooserBubbleDelegate), |
+ chooserBubbleUiController_(nil) { |
DCHECK(browser); |
- DCHECK(chooser_bubble_delegate); |
- chooser_bubble_delegate_->set_observer(this); |
+ DCHECK(chooserBubbleDelegate); |
+ chooserBubbleDelegate_->set_observer(this); |
} |
ChooserBubbleUiCocoa::~ChooserBubbleUiCocoa() { |
- chooser_bubble_delegate_->set_observer(nullptr); |
- if (chooser_bubble_ui_controller_) { |
- [chooser_bubble_ui_controller_ close]; |
- chooser_bubble_ui_controller_ = nil; |
- } |
+ chooserBubbleDelegate_->set_observer(nullptr); |
+ [chooserBubbleUiController_ close]; |
+ chooserBubbleUiController_ = nil; |
} |
void ChooserBubbleUiCocoa::Show(BubbleReference bubble_reference) { |
- if (!chooser_bubble_ui_controller_) { |
- chooser_bubble_ui_controller_ = [[ChooserBubbleUiController alloc] |
+ if (!chooserBubbleUiController_) { |
+ chooserBubbleUiController_ = [[ChooserBubbleUiController alloc] |
initWithBrowser:browser_ |
- initWithChooserBubbleDelegate:chooser_bubble_delegate_ |
+ initWithChooserBubbleDelegate:chooserBubbleDelegate_ |
bridge:this]; |
} |
- if (chooser_bubble_ui_controller_) { |
- [chooser_bubble_ui_controller_ show]; |
- [chooser_bubble_ui_controller_ updateTableView]; |
- } |
+ [chooserBubbleUiController_ show]; |
+ [chooserBubbleUiController_ updateTableView]; |
} |
void ChooserBubbleUiCocoa::Close() { |
- if (chooser_bubble_ui_controller_) { |
- [chooser_bubble_ui_controller_ close]; |
- chooser_bubble_ui_controller_ = nil; |
- } |
+ [chooserBubbleUiController_ close]; |
+ chooserBubbleUiController_ = nil; |
} |
void ChooserBubbleUiCocoa::UpdateAnchorPosition() { |
- [chooser_bubble_ui_controller_ updateAnchorPosition]; |
+ [chooserBubbleUiController_ updateAnchorPosition]; |
} |
void ChooserBubbleUiCocoa::OnOptionsInitialized() { |
- [chooser_bubble_ui_controller_ onOptionsInitialized]; |
+ [chooserBubbleUiController_ onOptionsInitialized]; |
} |
void ChooserBubbleUiCocoa::OnOptionAdded(int index) { |
- [chooser_bubble_ui_controller_ onOptionAdded:index]; |
+ [chooserBubbleUiController_ onOptionAdded:index]; |
} |
void ChooserBubbleUiCocoa::OnOptionRemoved(int index) { |
- [chooser_bubble_ui_controller_ onOptionRemoved:index]; |
+ [chooserBubbleUiController_ onOptionRemoved:index]; |
} |
void ChooserBubbleUiCocoa::OnBubbleClosing() { |
- chooser_bubble_ui_controller_ = nil; |
+ chooserBubbleUiController_ = nil; |
} |