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

Unified Diff: chrome/browser/ui/cocoa/location_bar/location_bar_decoration.mm

Issue 2847903003: cocoa: don't draw focus rings over the URL scheme in the location bar (Closed)
Patch Set: Created 3 years, 8 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/location_bar/location_bar_decoration.mm
diff --git a/chrome/browser/ui/cocoa/location_bar/location_bar_decoration.mm b/chrome/browser/ui/cocoa/location_bar/location_bar_decoration.mm
index 5e8d18d7addc6a578e547b5bc9894b5017abb261..58c2e230bb9619d713613fc316d4170c0ad9e4b5 100644
--- a/chrome/browser/ui/cocoa/location_bar/location_bar_decoration.mm
+++ b/chrome/browser/ui/cocoa/location_bar/location_bar_decoration.mm
@@ -104,7 +104,8 @@ - (void)drawFocusRingMask {
}
- (NSRect)focusRingMaskBounds {
- return [[self superview] convertRect:apparentFrame_ toView:self];
+ return owner_->GetRealFocusRingBounds(
+ [[self superview] convertRect:apparentFrame_ toView:self]);
}
@end
@@ -184,6 +185,10 @@ - (void)mouseExited:(NSEvent*)event {
[v setApparentFrame:apparent_frame];
}
+NSRect LocationBarDecoration::GetRealFocusRingBounds(NSRect bounds) const {
+ return bounds;
+}
+
void LocationBarDecoration::DrawInFrame(NSRect frame, NSView* control_view) {
NOTREACHED();
}

Powered by Google App Engine
This is Rietveld 408576698