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

Side by Side Diff: chrome/browser/ui/cocoa/bookmarks/bookmark_sync_promo_controller.mm

Issue 865373002: Removed calls to [HyperlinkTextView setMessageAndLink:withLink:...] (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Removed unnecessary checks, changed declaration Created 5 years, 11 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 #import "chrome/browser/ui/cocoa/bookmarks/bookmark_sync_promo_controller.h" 5 #import "chrome/browser/ui/cocoa/bookmarks/bookmark_sync_promo_controller.h"
6 6
7 #include "base/strings/sys_string_conversions.h" 7 #include "base/strings/sys_string_conversions.h"
8 #include "chrome/browser/signin/signin_promo.h" 8 #include "chrome/browser/signin/signin_promo.h"
9 #include "chrome/browser/ui/chrome_pages.h" 9 #include "chrome/browser/ui/chrome_pages.h"
10 #include "chrome/browser/ui/chrome_style.h" 10 #include "chrome/browser/ui/chrome_style.h"
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
74 [promoView setBorderWidth:kBorderWidth]; 74 [promoView setBorderWidth:kBorderWidth];
75 [promoView setBorderColor:gfx::SkColorToDeviceNSColor(kBorderColor)]; 75 [promoView setBorderColor:gfx::SkColorToDeviceNSColor(kBorderColor)];
76 76
77 // Add the sync promo text. 77 // Add the sync promo text.
78 size_t offset; 78 size_t offset;
79 const base::string16 linkText = l10n_util::GetStringUTF16( 79 const base::string16 linkText = l10n_util::GetStringUTF16(
80 IDS_BOOKMARK_SYNC_PROMO_LINK); 80 IDS_BOOKMARK_SYNC_PROMO_LINK);
81 const base::string16 promoText = l10n_util::GetStringFUTF16( 81 const base::string16 promoText = l10n_util::GetStringFUTF16(
82 IDS_BOOKMARK_SYNC_PROMO_MESSAGE, 82 IDS_BOOKMARK_SYNC_PROMO_MESSAGE,
83 linkText, &offset); 83 linkText, &offset);
84 const base::string16 promoTextWithoutLink = 84 NSString* nsPromoText = SysUTF16ToNSString(promoText);
85 promoText.substr(0, offset) + 85 NSString* nsLinkText = SysUTF16ToNSString(linkText);
86 promoText.substr(offset + linkText.size());
87
88 NSFont* font = [NSFont labelFontOfSize:kFontSize]; 86 NSFont* font = [NSFont labelFontOfSize:kFontSize];
89 NSColor* linkColor = gfx::SkColorToCalibratedNSColor( 87 NSColor* linkColor = gfx::SkColorToCalibratedNSColor(
90 chrome_style::GetLinkColor()); 88 chrome_style::GetLinkColor());
91 89
92 textView_.reset([[HyperlinkTextView alloc] init]); 90 textView_.reset([[HyperlinkTextView alloc] init]);
93 [textView_ setMessageAndLink:base::SysUTF16ToNSString(promoTextWithoutLink) 91 [textView_ setMessage:nsPromoText
94 withLink:base::SysUTF16ToNSString(linkText) 92 withFont:font
95 atOffset:offset 93 messageColor:gfx::SkColorToDeviceNSColor(kTextColor)];
96 font:font 94 if ([nsLinkText length] != 0) {
Alexei Svitkine (slow) 2015/01/26 18:11:19 You don't need this if either.
97 messageColor:gfx::SkColorToDeviceNSColor(kTextColor) 95 [textView_ addLinkRange:NSMakeRange(offset, [nsLinkText length])
98 linkColor:linkColor]; 96 withName:@""
97 linkColor:linkColor];
98 }
99 [textView_ setRefusesFirstResponder:YES]; 99 [textView_ setRefusesFirstResponder:YES];
100 [[textView_ textContainer] setLineFragmentPadding:0.0]; 100 [[textView_ textContainer] setLineFragmentPadding:0.0];
101 RemoveUnderlining(textView_, offset, linkText.size()); 101 RemoveUnderlining(textView_, offset, linkText.size());
102 [textView_ setDelegate:self]; 102 [textView_ setDelegate:self];
103 103
104 [promoView setContentView:textView_]; 104 [promoView setContentView:textView_];
105 105
106 [self setView:promoView]; 106 [self setView:promoView];
107 } 107 }
108 108
109 - (BOOL)textView:(NSTextView *)textView 109 - (BOOL)textView:(NSTextView *)textView
110 clickedOnLink:(id)link 110 clickedOnLink:(id)link
111 atIndex:(NSUInteger)charIndex { 111 atIndex:(NSUInteger)charIndex {
112 chrome::ShowBrowserSignin(browser_, signin_metrics::SOURCE_BOOKMARK_BUBBLE); 112 chrome::ShowBrowserSignin(browser_, signin_metrics::SOURCE_BOOKMARK_BUBBLE);
113 return YES; 113 return YES;
114 } 114 }
115 115
116 @end 116 @end
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698