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

Unified Diff: third_party/WebKit/Source/core/layout/LayoutMenuList.cpp

Issue 2555923002: Changed TextDirection to an enum class and renamed its members (Closed)
Patch Set: Added 2 changes in mac files Created 4 years 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: third_party/WebKit/Source/core/layout/LayoutMenuList.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutMenuList.cpp b/third_party/WebKit/Source/core/layout/LayoutMenuList.cpp
index c97fa7480b8270f939e251e10cacb3e66a2b1545..d88aff42cb24709938e16ca6070bd22b8f38ae40 100644
--- a/third_party/WebKit/Source/core/layout/LayoutMenuList.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutMenuList.cpp
@@ -100,10 +100,10 @@ void LayoutMenuList::adjustInnerStyle() {
Length(LayoutTheme::theme().popupInternalPaddingStart(styleRef()), Fixed);
Length paddingEnd =
Length(LayoutTheme::theme().popupInternalPaddingEnd(styleRef()), Fixed);
- innerStyle.setPaddingLeft(styleRef().direction() == LTR ? paddingStart
- : paddingEnd);
- innerStyle.setPaddingRight(styleRef().direction() == LTR ? paddingEnd
- : paddingStart);
+ innerStyle.setPaddingLeft(
+ styleRef().direction() == TextDirection::Ltr ? paddingStart : paddingEnd);
+ innerStyle.setPaddingRight(
+ styleRef().direction() == TextDirection::Ltr ? paddingEnd : paddingStart);
innerStyle.setPaddingTop(
Length(LayoutTheme::theme().popupInternalPaddingTop(styleRef()), Fixed));
innerStyle.setPaddingBottom(Length(

Powered by Google App Engine
This is Rietveld 408576698