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

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

Issue 2560733002: SELECT element: Fix a bug that intrinsic width is too narrow in less-than-100% zoom level. (Closed)
Patch Set: PartScrollbarVerticalTrack -> PartScrollbarDownArrow to fix Android crash 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..868764fe3eb78d48581cae53d3cf80ffaba8d5b5 100644
--- a/third_party/WebKit/Source/core/layout/LayoutMenuList.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutMenuList.cpp
@@ -28,6 +28,7 @@
#include "core/dom/AXObjectCache.h"
#include "core/dom/NodeComputedStyle.h"
+#include "core/frame/FrameView.h"
#include "core/html/HTMLOptionElement.h"
#include "core/html/HTMLSelectElement.h"
#include "core/layout/LayoutText.h"
@@ -98,8 +99,9 @@ void LayoutMenuList::adjustInnerStyle() {
Length paddingStart =
Length(LayoutTheme::theme().popupInternalPaddingStart(styleRef()), Fixed);
- Length paddingEnd =
- Length(LayoutTheme::theme().popupInternalPaddingEnd(styleRef()), Fixed);
+ Length paddingEnd = Length(LayoutTheme::theme().popupInternalPaddingEnd(
+ frameView()->getHostWindow(), styleRef()),
+ Fixed);
innerStyle.setPaddingLeft(styleRef().direction() == LTR ? paddingStart
: paddingEnd);
innerStyle.setPaddingRight(styleRef().direction() == LTR ? paddingEnd

Powered by Google App Engine
This is Rietveld 408576698