Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2850)

Unified Diff: chrome/browser/ui/cocoa/extensions/extension_install_view_controller.mm

Issue 12303030: Merge 182576 to M26 beta branch. (Closed) Base URL: svn://svn.chromium.org/chrome/branches/1410/src/
Patch Set: Created 7 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/cocoa/extensions/extension_install_view_controller.mm
===================================================================
--- chrome/browser/ui/cocoa/extensions/extension_install_view_controller.mm (revision 183257)
+++ chrome/browser/ui/cocoa/extensions/extension_install_view_controller.mm (working copy)
@@ -186,8 +186,15 @@
- (void)awakeFromNib {
// Set control labels.
[titleField_ setStringValue:base::SysUTF16ToNSString(prompt_->GetHeading())];
- [okButton_ setTitle:base::SysUTF16ToNSString(
- prompt_->GetAcceptButtonLabel())];
+ NSRect okButtonRect;
+ if (prompt_->HasAcceptButtonLabel()) {
+ [okButton_ setTitle:base::SysUTF16ToNSString(
+ prompt_->GetAcceptButtonLabel())];
+ } else {
+ [okButton_ removeFromSuperview];
+ okButtonRect = [okButton_ frame];
+ okButton_ = nil;
+ }
[cancelButton_ setTitle:prompt_->HasAbortButtonLabel() ?
base::SysUTF16ToNSString(prompt_->GetAbortButtonLabel()) :
l10n_util::GetNSString(IDS_CANCEL)];
@@ -218,11 +225,21 @@
// Resize |okButton_| and |cancelButton_| to fit the button labels, but keep
// them right-aligned.
- NSSize buttonDelta = [GTMUILocalizerAndLayoutTweaker sizeToFitView:okButton_];
- if (buttonDelta.width) {
- [okButton_ setFrame:NSOffsetRect([okButton_ frame], -buttonDelta.width, 0)];
- [cancelButton_ setFrame:NSOffsetRect([cancelButton_ frame],
- -buttonDelta.width, 0)];
+ NSSize buttonDelta;
+ if (okButton_) {
+ buttonDelta = [GTMUILocalizerAndLayoutTweaker sizeToFitView:okButton_];
+ if (buttonDelta.width) {
+ [okButton_ setFrame:NSOffsetRect([okButton_ frame],
+ -buttonDelta.width, 0)];
+ [cancelButton_ setFrame:NSOffsetRect([cancelButton_ frame],
+ -buttonDelta.width, 0)];
+ }
+ } else {
+ // Make |cancelButton_| right-aligned in the absence of |okButton_|.
+ NSRect cancelButtonRect = [cancelButton_ frame];
+ cancelButtonRect.origin.x =
+ NSMaxX(okButtonRect) - NSWidth(cancelButtonRect);
+ [cancelButton_ setFrame:cancelButtonRect];
}
buttonDelta = [GTMUILocalizerAndLayoutTweaker sizeToFitView:cancelButton_];
if (buttonDelta.width) {

Powered by Google App Engine
This is Rietveld 408576698