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

Unified Diff: Source/core/rendering/RenderThemeChromiumMac.mm

Issue 368003002: Check if drawFocusRingMaskWithFrame:inView: exists when drawing focus ring (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Check before calling setShowsFirstResponder Created 6 years, 6 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
« no previous file with comments | « no previous file | Source/platform/mac/ThemeMac.mm » ('j') | Source/platform/mac/WebCoreNSCellExtras.mm » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/rendering/RenderThemeChromiumMac.mm
diff --git a/Source/core/rendering/RenderThemeChromiumMac.mm b/Source/core/rendering/RenderThemeChromiumMac.mm
index 15f2aaa122c45cba71e4e4bcaf642c0aff201008..235093aee6741f239543ceaa5317d7ec40b0debb 100644
--- a/Source/core/rendering/RenderThemeChromiumMac.mm
+++ b/Source/core/rendering/RenderThemeChromiumMac.mm
@@ -626,6 +626,8 @@ void RenderThemeChromiumMac::updateEnabledState(NSCell* cell, const RenderObject
void RenderThemeChromiumMac::updateFocusedState(NSCell* cell, const RenderObject* o)
{
+ if ([cell respondsToSelector:@selector(drawFocusRingMaskWithFrame:inView:)])
+ return;
bool oldFocused = [cell showsFirstResponder];
bool focused = isFocused(o) && o->style()->outlineStyleIsAuto();
if (focused != oldFocused)
@@ -1270,9 +1272,7 @@ void RenderThemeChromiumMac::setPopupButtonCellState(const RenderObject* o, cons
updateCheckedState(popupButton, o);
updateEnabledState(popupButton, o);
updatePressedState(popupButton, o);
-#if BUTTON_CELL_DRAW_WITH_FRAME_DRAWS_FOCUS_RING
updateFocusedState(popupButton, o);
-#endif
}
const IntSize* RenderThemeChromiumMac::menuListSizes() const
« no previous file with comments | « no previous file | Source/platform/mac/ThemeMac.mm » ('j') | Source/platform/mac/WebCoreNSCellExtras.mm » ('J')

Powered by Google App Engine
This is Rietveld 408576698