Switch to LayoutThemeMobile when emulating mobile device in DevTools.

third_party/WebKit/Source/core/layout/LayoutTheme.cpp

 /**
  * This file is part of the theme implementation for form controls in WebCore.
  *
  * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010, 2012 Apple Computer, Inc.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
@@ skipping 24 matching lines @@
 #include "core/html/HTMLDataListOptionsCollection.h"
 #include "core/html/HTMLFormControlElement.h"
 #include "core/html/HTMLInputElement.h"
 #include "core/html/HTMLMeterElement.h"
 #include "core/html/HTMLOptionElement.h"
 #include "core/html/parser/HTMLParserIdioms.h"
 #include "core/html/shadow/MediaControlElements.h"
 #include "core/html/shadow/ShadowElementNames.h"
 #include "core/html/shadow/SpinButtonElement.h"
 #include "core/html/shadow/TextControlInnerElements.h"
+#include "core/layout/LayoutThemeMobile.h"
 #include "core/page/FocusController.h"
 #include "core/page/Page.h"
 #include "core/style/ComputedStyle.h"
 #include "platform/FileMetadata.h"
 #include "platform/FloatConversion.h"
 #include "platform/RuntimeEnabledFeatures.h"
+#include "platform/Theme.h"
 #include "platform/fonts/FontSelector.h"
 #include "platform/text/PlatformLocale.h"
 #include "platform/text/StringTruncator.h"
 #include "public/platform/Platform.h"
 #include "public/platform/WebFallbackThemeEngine.h"
 #include "public/platform/WebRect.h"
 #include "wtf/text/StringBuilder.h"
 
-#if USE(NEW_THEME)
-#include "platform/Theme.h"
-#endif
-
 // The methods in this file are shared by all themes on every platform.
 
 namespace blink {
 
 using namespace HTMLNames;
 
-LayoutTheme::LayoutTheme()
+LayoutTheme& LayoutTheme::theme()
+{
+    if (RuntimeEnabledFeatures::mobileLayoutThemeEnabled()) {
+        DEFINE_STATIC_REF(LayoutTheme, layoutThemeMobile, (LayoutThemeMobile::create()));
+        return *layoutThemeMobile;
+    }
+    return nativeTheme();
+}
+
+LayoutTheme::LayoutTheme(Theme* platformTheme)
     : m_hasCustomFocusRingColor(false)
-#if USE(NEW_THEME)
-    , m_platformTheme(platformTheme())
-#endif
+    , m_platformTheme(platformTheme)
 {
 }
 
 void LayoutTheme::adjustStyle(ComputedStyle& style, Element* e)
 {
     ASSERT(style.hasAppearance());
 
     // Force inline and table display styles to be inline-block (except for table- which is block)
     ControlPart part = style.appearance();
     if (style.display() == INLINE || style.display() == INLINE_TABLE || style.display() == TABLE_ROW_GROUP
@@ skipping 12 matching lines @@
             style.setAppearance(NoControlPart);
             return;
         }
     }
 
     if (shouldUseFallbackTheme(style)) {
         adjustStyleUsingFallbackTheme(style);
         return;
     }
 
-#if USE(NEW_THEME)
-    switch (part) {
-    case CheckboxPart:
-    case InnerSpinButtonPart:
-    case RadioPart:
-    case PushButtonPart:
-    case SquareButtonPart:
-    case ButtonPart: {
-        // Border
-        LengthBox borderBox(style.borderTopWidth(), style.borderRightWidth(), style.borderBottomWidth(), style.borderLeftWidth());
-        borderBox = m_platformTheme->controlBorder(part, style.font().fontDescription(), borderBox, style.effectiveZoom());
-        if (borderBox.top().value() != static_cast<int>(style.borderTopWidth())) {
-            if (borderBox.top().value())
-                style.setBorderTopWidth(borderBox.top().value());
-            else
-                style.resetBorderTop();
+    if (m_platformTheme) {
+        switch (part) {

[dgozman, 2016/02/19 19:15:16]

This whole switch was just indented.

+        case CheckboxPart:
+        case InnerSpinButtonPart:
+        case RadioPart:
+        case PushButtonPart:
+        case SquareButtonPart:
+        case ButtonPart: {
+            // Border
+            LengthBox borderBox(style.borderTopWidth(), style.borderRightWidth(), style.borderBottomWidth(), style.borderLeftWidth());
+            borderBox = m_platformTheme->controlBorder(part, style.font().fontDescription(), borderBox, style.effectiveZoom());
+            if (borderBox.top().value() != static_cast<int>(style.borderTopWidth())) {
+                if (borderBox.top().value())
+                    style.setBorderTopWidth(borderBox.top().value());
+                else
+                    style.resetBorderTop();
+            }
+            if (borderBox.right().value() != static_cast<int>(style.borderRightWidth())) {
+                if (borderBox.right().value())
+                    style.setBorderRightWidth(borderBox.right().value());
+                else
+                    style.resetBorderRight();
+            }
+            if (borderBox.bottom().value() != static_cast<int>(style.borderBottomWidth())) {
+                style.setBorderBottomWidth(borderBox.bottom().value());
+                if (borderBox.bottom().value())
+                    style.setBorderBottomWidth(borderBox.bottom().value());
+                else
+                    style.resetBorderBottom();
+            }
+            if (borderBox.left().value() != static_cast<int>(style.borderLeftWidth())) {
+                style.setBorderLeftWidth(borderBox.left().value());
+                if (borderBox.left().value())
+                    style.setBorderLeftWidth(borderBox.left().value());
+                else
+                    style.resetBorderLeft();
+            }
+
+            // Padding
+            LengthBox paddingBox = m_platformTheme->controlPadding(part, style.font().fontDescription(), style.paddingBox(), style.effectiveZoom());
+            if (paddingBox != style.paddingBox())
+                style.setPaddingBox(paddingBox);
+
+            // Whitespace
+            if (m_platformTheme->controlRequiresPreWhiteSpace(part))
+                style.setWhiteSpace(PRE);
+
+            // Width / Height
+            // The width and height here are affected by the zoom.
+            // FIXME: Check is flawed, since it doesn't take min-width/max-width into account.
+            LengthSize controlSize = m_platformTheme->controlSize(part, style.font().fontDescription(), LengthSize(style.width(), style.height()), style.effectiveZoom());
+            if (controlSize.width() != style.width())
+                style.setWidth(controlSize.width());
+            if (controlSize.height() != style.height())
+                style.setHeight(controlSize.height());
+
+            // Min-Width / Min-Height
+            LengthSize minControlSize = m_platformTheme->minimumControlSize(part, style.font().fontDescription(), style.effectiveZoom());
+            if (minControlSize.width() != style.minWidth())
+                style.setMinWidth(minControlSize.width());
+            if (minControlSize.height() != style.minHeight())
+                style.setMinHeight(minControlSize.height());
+
+            // Font
+            FontDescription controlFont = m_platformTheme->controlFont(part, style.font().fontDescription(), style.effectiveZoom());
+            if (controlFont != style.font().fontDescription()) {
+                // Reset our line-height
+                style.setLineHeight(ComputedStyle::initialLineHeight());
+
+                // Now update our font.
+                if (style.setFontDescription(controlFont))
+                    style.font().update(nullptr);
+            }
         }
-        if (borderBox.right().value() != static_cast<int>(style.borderRightWidth())) {
-            if (borderBox.right().value())
-                style.setBorderRightWidth(borderBox.right().value());
-            else
-                style.resetBorderRight();
-        }
-        if (borderBox.bottom().value() != static_cast<int>(style.borderBottomWidth())) {
-            style.setBorderBottomWidth(borderBox.bottom().value());
-            if (borderBox.bottom().value())
-                style.setBorderBottomWidth(borderBox.bottom().value());
-            else
-                style.resetBorderBottom();
-        }
-        if (borderBox.left().value() != static_cast<int>(style.borderLeftWidth())) {
-            style.setBorderLeftWidth(borderBox.left().value());
-            if (borderBox.left().value())
-                style.setBorderLeftWidth(borderBox.left().value());
-            else
-                style.resetBorderLeft();
-        }
-
-        // Padding
-        LengthBox paddingBox = m_platformTheme->controlPadding(part, style.font().fontDescription(), style.paddingBox(), style.effectiveZoom());
-        if (paddingBox != style.paddingBox())
-            style.setPaddingBox(paddingBox);
-
-        // Whitespace
-        if (m_platformTheme->controlRequiresPreWhiteSpace(part))
-            style.setWhiteSpace(PRE);
-
-        // Width / Height
-        // The width and height here are affected by the zoom.
-        // FIXME: Check is flawed, since it doesn't take min-width/max-width into account.
-        LengthSize controlSize = m_platformTheme->controlSize(part, style.font().fontDescription(), LengthSize(style.width(), style.height()), style.effectiveZoom());
-        if (controlSize.width() != style.width())
-            style.setWidth(controlSize.width());
-        if (controlSize.height() != style.height())
-            style.setHeight(controlSize.height());
-
-        // Min-Width / Min-Height
-        LengthSize minControlSize = m_platformTheme->minimumControlSize(part, style.font().fontDescription(), style.effectiveZoom());
-        if (minControlSize.width() != style.minWidth())
-            style.setMinWidth(minControlSize.width());
-        if (minControlSize.height() != style.minHeight())
-            style.setMinHeight(minControlSize.height());
-
-        // Font
-        FontDescription controlFont = m_platformTheme->controlFont(part, style.font().fontDescription(), style.effectiveZoom());
-        if (controlFont != style.font().fontDescription()) {
-            // Reset our line-height
-            style.setLineHeight(ComputedStyle::initialLineHeight());
-
-            // Now update our font.
-            if (style.setFontDescription(controlFont))
-                style.font().update(nullptr);
+        default:
+            break;
         }
     }
-    default:
-        break;
+
+    if (!m_platformTheme) {
+        // Call the appropriate style adjustment method based off the appearance value.
+        switch (style.appearance()) {
+        case CheckboxPart:
+            return adjustCheckboxStyle(style);
+        case RadioPart:
+            return adjustRadioStyle(style);
+        case PushButtonPart:
+        case SquareButtonPart:
+        case ButtonPart:
+            return adjustButtonStyle(style);
+        case InnerSpinButtonPart:
+            return adjustInnerSpinButtonStyle(style);
+        default:
+            break;
+        }
     }
-#endif
 
     // Call the appropriate style adjustment method based off the appearance value.
     switch (style.appearance()) {
-#if !USE(NEW_THEME)
-    case CheckboxPart:
-        return adjustCheckboxStyle(style);
-    case RadioPart:
-        return adjustRadioStyle(style);
-    case PushButtonPart:
-    case SquareButtonPart:
-    case ButtonPart:
-        return adjustButtonStyle(style);
-    case InnerSpinButtonPart:
-        return adjustInnerSpinButtonStyle(style);
-#endif
     case MenulistPart:
         return adjustMenuListStyle(style, e);
     case MenulistButtonPart:
         return adjustMenuListButtonStyle(style, e);
     case SliderThumbHorizontalPart:
     case SliderThumbVerticalPart:
         return adjustSliderThumbStyle(style);
     case SearchFieldPart:
         return adjustSearchFieldStyle(style);
     case SearchFieldCancelButtonPart:
@@ skipping 152 matching lines @@
     return platformInactiveSelectionForegroundColor();
 }
 
 int LayoutTheme::baselinePosition(const LayoutObject* o) const
 {
     if (!o->isBox())
         return 0;
 
     const LayoutBox* box = toLayoutBox(o);
 
-#if USE(NEW_THEME)
-    return box->size().height() + box->marginTop() + m_platformTheme->baselinePositionAdjustment(o->style()->appearance()) * o->style()->effectiveZoom();
-#else
+    if (m_platformTheme)
+        return box->size().height() + box->marginTop() + m_platformTheme->baselinePositionAdjustment(o->style()->appearance()) * o->style()->effectiveZoom();
     return box->size().height() + box->marginTop();
-#endif
 }
 
 bool LayoutTheme::isControlContainer(ControlPart appearance) const
 {
     // There are more leaves than this, but we'll patch this function as we add support for
     // more controls.
     return appearance != CheckboxPart && appearance != RadioPart;
 }
 
 bool LayoutTheme::isControlStyled(const ComputedStyle& style) const
@@ skipping 20 matching lines @@
     case SliderVerticalPart:
         return style.boxShadow();
 
     default:
         return false;
     }
 }
 
 void LayoutTheme::addVisualOverflow(const LayoutObject& object, IntRect& borderBox)
 {
-#if USE(NEW_THEME)
-    m_platformTheme->addVisualOverflow(object.style()->appearance(), controlStatesForLayoutObject(object), object.style()->effectiveZoom(), borderBox);
-#endif
+    if (m_platformTheme)
+        m_platformTheme->addVisualOverflow(object.style()->appearance(), controlStatesForLayoutObject(object), object.style()->effectiveZoom(), borderBox);
 }
 
 bool LayoutTheme::shouldDrawDefaultFocusRing(const LayoutObject& layoutObject) const
 {
     if (themeDrawsFocusRing(layoutObject.styleRef()))
         return false;
     Node* node = layoutObject.node();
     if (!node)
         return true;
     if (!layoutObject.styleRef().hasAppearance() && !node->isLink())
@@ skipping 136 matching lines @@
 
 bool LayoutTheme::isSpinUpButtonPartHovered(const LayoutObject& o)
 {
     Node* node = o.node();
     if (!node || !node->isElementNode() || !toElement(node)->isSpinButtonElement())
         return false;
     SpinButtonElement* element = toSpinButtonElement(node);
     return element->upDownState() == SpinButtonElement::Up;
 }
 
-#if !USE(NEW_THEME)
-
 void LayoutTheme::adjustCheckboxStyle(ComputedStyle& style) const
 {
     // A summary of the rules for checkbox designed to match WinIE:
     // width/height - honored (WinIE actually scales its control for small widths, but lets it overflow for small heights.)
     // font-size - not honored (control has no text), but we use it to decide which control size to use.
     setCheckboxSize(style);
 
     // padding - not honored by WinIE, needs to be removed.
     style.resetPadding();
 
@@ skipping 17 matching lines @@
     style.resetBorder();
 }
 
 void LayoutTheme::adjustButtonStyle(ComputedStyle& style) const
 {
 }
 
 void LayoutTheme::adjustInnerSpinButtonStyle(ComputedStyle&) const
 {
 }
-#endif
 
 void LayoutTheme::adjustMenuListStyle(ComputedStyle&, Element*) const
 {
 }
 
 IntSize LayoutTheme::meterSizeForBounds(const LayoutMeter&, const IntRect& bounds) const
 {
     return bounds.size();
 }
 
@@ skipping 312 matching lines @@
 
     // padding - not honored by WinIE, needs to be removed.
     style.resetPadding();
 
     // border - honored by WinIE, but looks terrible (just paints in the control box and turns off the Windows XP theme)
     // for now, we will not honor it.
     style.resetBorder();
 }
 
 } // namespace blink