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

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

Issue 11077006: Correct padding and focus ring for frameless constrained window dialog (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Fix license header Created 8 years, 2 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/web_intent_sheet_controller.mm
diff --git a/chrome/browser/ui/cocoa/web_intent_sheet_controller.mm b/chrome/browser/ui/cocoa/web_intent_sheet_controller.mm
index ab9f72effe250e7bed9e1bfec8ed28c03e5e66fa..bc876a89b0f6aaaba567b22f3c236dd63fa70090 100644
--- a/chrome/browser/ui/cocoa/web_intent_sheet_controller.mm
+++ b/chrome/browser/ui/cocoa/web_intent_sheet_controller.mm
@@ -15,6 +15,7 @@
#import "chrome/browser/ui/cocoa/info_bubble_window.h"
#include "chrome/browser/ui/cocoa/web_intent_picker_cocoa.h"
#include "chrome/browser/ui/constrained_window.h"
+#include "chrome/browser/ui/constrained_window_constants.h"
#include "chrome/browser/ui/intents/web_intent_picker_delegate.h"
#include "chrome/browser/ui/intents/web_intent_picker_model.h"
#include "chrome/browser/ui/tab_contents/tab_contents.h"
@@ -268,7 +269,8 @@ NSButton* CreateHyperlinkButton(NSString* title, const NSRect& frame) {
ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
titleField_.reset([[NSTextField alloc] initWithFrame:frame]);
ConfigureTextFieldAsLabel(titleField_);
- gfx::Font titleFont = rb.GetFont(ConstrainedWindow::kTitleFontStyle);
+ gfx::Font titleFont = rb.GetFont(
+ ConstrainedWindowConstants::kTitleFontStyle);
titleFont = titleFont.DeriveFont(0, gfx::Font::BOLD);
[titleField_ setFont:titleFont.GetNativeFont()];
@@ -276,7 +278,7 @@ NSButton* CreateHyperlinkButton(NSString* title, const NSRect& frame) {
kTextWidth, 1);
subtitleField_.reset([[NSTextField alloc] initWithFrame:frame]);
ConfigureTextFieldAsLabel(subtitleField_);
- gfx::Font textFont = rb.GetFont(ConstrainedWindow::kTextFontStyle);
+ gfx::Font textFont = rb.GetFont(ConstrainedWindowConstants::kTextFontStyle);
[subtitleField_ setFont:textFont.GetNativeFont()];
frame = NSMakeRect(0, 0, WebIntentPicker::kWindowMinWidth, 1.0);
@@ -1052,7 +1054,7 @@ const CGFloat kAddButtonWidth = 128.0;
[[NSTextField alloc] initWithFrame:titleFrame]);
ConfigureTextFieldAsLabel(title);
ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
- gfx::Font titleFont = rb.GetFont(ConstrainedWindow::kTitleFontStyle);
+ gfx::Font titleFont = rb.GetFont(ConstrainedWindowConstants::kTitleFontStyle);
titleFont = titleFont.DeriveFont(0, gfx::Font::BOLD);
[title setFont:titleFont.GetNativeFont()];
[title setStringValue:
@@ -1153,7 +1155,8 @@ const CGFloat kAddButtonWidth = 128.0;
actionTextField_.reset([[NSTextField alloc] initWithFrame:textFrame]);
ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
ConfigureTextFieldAsLabel(actionTextField_);
- gfx::Font titleFont = rb.GetFont(ConstrainedWindow::kTitleFontStyle);
+ gfx::Font titleFont = rb.GetFont(
+ ConstrainedWindowConstants::kTitleFontStyle);
titleFont = titleFont.DeriveFont(0, gfx::Font::BOLD);
[actionTextField_ setFont:titleFont.GetNativeFont()];
} else {
« no previous file with comments | « chrome/browser/ui/cocoa/intents/web_intent_service_row_view_controller.mm ('k') | chrome/browser/ui/constrained_window.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698