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

Unified Diff: chrome/browser/ui/cocoa/chooser_content_view_cocoa.mm

Issue 2390823005: Update device permissions dialog ui for Chrome apps and extensions (Closed)
Patch Set: address more comments Created 4 years, 1 month 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
« no previous file with comments | « chrome/browser/ui/BUILD.gn ('k') | chrome/browser/ui/cocoa/extensions/chooser_dialog_cocoa.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/cocoa/chooser_content_view_cocoa.mm
diff --git a/chrome/browser/ui/cocoa/chooser_content_view_cocoa.mm b/chrome/browser/ui/cocoa/chooser_content_view_cocoa.mm
index a2ec7c0b18564436be485f8081d02db36dfef2ec..0c3db026114da7ae62d49effe144019d99f5c776 100644
--- a/chrome/browser/ui/cocoa/chooser_content_view_cocoa.mm
+++ b/chrome/browser/ui/cocoa/chooser_content_view_cocoa.mm
@@ -452,13 +452,17 @@ void ChooserContentViewController::UpdateTableView() {
separator_ = [self createSeparator];
// Help button.
- helpButton_ =
- [self createHyperlinkButtonWithText:
- l10n_util::GetNSStringF(
- IDS_DEVICE_CHOOSER_GET_HELP_LINK_WITH_SCANNING_STATUS,
- base::string16())];
- CGFloat helpButtonWidth = NSWidth([helpButton_ frame]);
- CGFloat helpButtonHeight = NSHeight([helpButton_ frame]);
+ CGFloat helpButtonWidth = 0.0f;
+ CGFloat helpButtonHeight = 0.0f;
+ if (chooserController_->ShouldShowFootnoteView()) {
+ helpButton_ =
+ [self createHyperlinkButtonWithText:
+ l10n_util::GetNSStringF(
+ IDS_DEVICE_CHOOSER_GET_HELP_LINK_WITH_SCANNING_STATUS,
+ base::string16())];
+ helpButtonWidth = NSWidth([helpButton_ frame]);
+ helpButtonHeight = NSHeight([helpButton_ frame]);
+ }
// Scanning message.
scanningMessage_ = CreateLabel(
@@ -477,12 +481,17 @@ void ChooserContentViewController::UpdateTableView() {
// ScollView embedding with TableView.
CGFloat scrollViewWidth = kChooserWidth - 2 * kMarginX;
- CGFloat scrollViewHeight = kChooserHeight - 2 * kMarginY -
- 4 * kVerticalPadding - titleHeight -
- connectButtonHeight - helpButtonHeight;
+ CGFloat scrollViewHeight =
+ kChooserHeight - 2 * kMarginY -
+ (chooserController_->ShouldShowFootnoteView() ? 4 * kVerticalPadding
+ : 2 * kVerticalPadding) -
+ titleHeight - connectButtonHeight - helpButtonHeight;
CGFloat scrollViewOriginX = kMarginX;
- CGFloat scrollViewOriginY = kMarginY + helpButtonHeight +
- 3 * kVerticalPadding + connectButtonHeight;
+ CGFloat scrollViewOriginY =
+ kMarginY + helpButtonHeight +
+ (chooserController_->ShouldShowFootnoteView() ? 3 * kVerticalPadding
+ : kVerticalPadding) +
+ connectButtonHeight;
NSRect scrollFrame = NSMakeRect(scrollViewOriginX, scrollViewOriginY,
scrollViewWidth, scrollViewHeight);
scrollView_.reset([[NSScrollView alloc] initWithFrame:scrollFrame]);
@@ -556,7 +565,9 @@ void ChooserContentViewController::UpdateTableView() {
kHorizontalPadding - connectButtonWidth -
cancelButtonWidth;
CGFloat connectButtonOriginY =
- kMarginY + helpButtonHeight + 2 * kVerticalPadding;
+ kMarginY + helpButtonHeight +
+ (chooserController_->ShouldShowFootnoteView() ? 2 * kVerticalPadding
+ : 0.0f);
[connectButton_
setFrameOrigin:NSMakePoint(connectButtonOriginX, connectButtonOriginY)];
[connectButton_ setEnabled:NO];
@@ -569,50 +580,52 @@ void ChooserContentViewController::UpdateTableView() {
setFrameOrigin:NSMakePoint(cancelButtonOriginX, cancelButtonOriginY)];
[self addSubview:cancelButton_];
- // Separator.
- CGFloat separatorOriginX = 0.0f;
- separatorOriginY_ = kMarginY + helpButtonHeight + kVerticalPadding;
- [separator_
- setFrameOrigin:NSMakePoint(separatorOriginX, separatorOriginY_)];
- [self addSubview:separator_];
-
- // Help button.
- CGFloat helpButtonOriginX = kMarginX;
- CGFloat helpButtonOriginY = (kVerticalPadding + kMarginY) / 2;
- [helpButton_
- setFrameOrigin:NSMakePoint(helpButtonOriginX, helpButtonOriginY)];
- [helpButton_ setTarget:self];
- [helpButton_ setAction:@selector(onHelpPressed:)];
- [self addSubview:helpButton_];
-
- // Scanning message.
- CGFloat scanningMessageOriginX =
- kMarginX + helpButtonWidth - kHorizontalPadding / 2;
- CGFloat scanningMessageOriginY = helpButtonOriginY;
- [scanningMessage_ setFrameOrigin:NSMakePoint(scanningMessageOriginX,
- scanningMessageOriginY)];
- [self addSubview:scanningMessage_];
- [scanningMessage_ setHidden:YES];
-
- // Word connector.
- CGFloat wordConnectorOriginX =
- kMarginX + helpButtonWidth - kHorizontalPadding / 2;
- CGFloat wordConnectorOriginY = helpButtonOriginY;
- [wordConnector_
- setFrameOrigin:NSMakePoint(wordConnectorOriginX, wordConnectorOriginY)];
- [self addSubview:wordConnector_];
- [wordConnector_ setHidden:YES];
-
- // Re-scan button.
- CGFloat reScanButtonOriginX =
- kMarginX + helpButtonWidth + wordConnectorWidth - kHorizontalPadding;
- CGFloat reScanButtonOriginY = helpButtonOriginY;
- [rescanButton_
- setFrameOrigin:NSMakePoint(reScanButtonOriginX, reScanButtonOriginY)];
- [rescanButton_ setTarget:self];
- [rescanButton_ setAction:@selector(onRescan:)];
- [self addSubview:rescanButton_];
- [rescanButton_ setHidden:YES];
+ if (chooserController_->ShouldShowFootnoteView()) {
+ // Separator.
+ CGFloat separatorOriginX = 0.0f;
+ separatorOriginY_ = kMarginY + helpButtonHeight + kVerticalPadding;
+ [separator_
+ setFrameOrigin:NSMakePoint(separatorOriginX, separatorOriginY_)];
+ [self addSubview:separator_];
+
+ // Help button.
+ CGFloat helpButtonOriginX = kMarginX;
+ CGFloat helpButtonOriginY = (kVerticalPadding + kMarginY) / 2;
+ [helpButton_
+ setFrameOrigin:NSMakePoint(helpButtonOriginX, helpButtonOriginY)];
+ [helpButton_ setTarget:self];
+ [helpButton_ setAction:@selector(onHelpPressed:)];
+ [self addSubview:helpButton_];
+
+ // Scanning message.
+ CGFloat scanningMessageOriginX =
+ kMarginX + helpButtonWidth - kHorizontalPadding / 2;
+ CGFloat scanningMessageOriginY = helpButtonOriginY;
+ [scanningMessage_ setFrameOrigin:NSMakePoint(scanningMessageOriginX,
+ scanningMessageOriginY)];
+ [self addSubview:scanningMessage_];
+ [scanningMessage_ setHidden:YES];
+
+ // Word connector.
+ CGFloat wordConnectorOriginX =
+ kMarginX + helpButtonWidth - kHorizontalPadding / 2;
+ CGFloat wordConnectorOriginY = helpButtonOriginY;
+ [wordConnector_ setFrameOrigin:NSMakePoint(wordConnectorOriginX,
+ wordConnectorOriginY)];
+ [self addSubview:wordConnector_];
+ [wordConnector_ setHidden:YES];
+
+ // Re-scan button.
+ CGFloat reScanButtonOriginX =
+ kMarginX + helpButtonWidth + wordConnectorWidth - kHorizontalPadding;
+ CGFloat reScanButtonOriginY = helpButtonOriginY;
+ [rescanButton_
+ setFrameOrigin:NSMakePoint(reScanButtonOriginX, reScanButtonOriginY)];
+ [rescanButton_ setTarget:self];
+ [rescanButton_ setAction:@selector(onRescan:)];
+ [self addSubview:rescanButton_];
+ [rescanButton_ setHidden:YES];
+ }
chooserContentViewController_.reset(new ChooserContentViewController(
chooserController_.get(), adapterOffHelpButton_.get(),
« no previous file with comments | « chrome/browser/ui/BUILD.gn ('k') | chrome/browser/ui/cocoa/extensions/chooser_dialog_cocoa.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698