Split CSSPrimitiveValue into CSSPrimitiveValue and CSSIdentifierValue

third_party/WebKit/Source/core/css/resolver/ViewportStyleResolver.cpp

 /*
  * Copyright (C) 2012-2013 Intel Corporation. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1. Redistributions of source code must retain the above
  *    copyright notice, this list of conditions and the following
  *    disclaimer.
@@ skipping 129 matching lines @@
 {
     float defaultValue = ViewportDescription::ValueAuto;
 
     // UserZoom default value is CSSValueZoom, which maps to true, meaning that
     // yes, it is user scalable. When the value is set to CSSValueFixed, we
     // return false.
     if (id == CSSPropertyUserZoom)
         defaultValue = 1;
 
     const CSSValue* value = m_propertySet->getPropertyCSSValue(id);
-    if (!value || !value->isPrimitiveValue())
+    if (!value || !value->isPrimitiveValue() || !value->isIdentifierValue())

[Timothy Loh, 2016/09/21 07:55:06]

This logic (and below in the file) is wrong since now the if condition will
always be false.

[sashab, 2016/09/23 01:01:10]

Ack, I hate these confusing if-statements!! I want to write a patch to remove
them all. Fixed.

         return defaultValue;
 
+    if (value->isIdentifierValue()) {
+        switch (toCSSIdentifierValue(value)->getValueID()) {
+        case CSSValueAuto:
+            return defaultValue;
+        case CSSValueLandscape:
+            return ViewportDescription::ValueLandscape;
+        case CSSValuePortrait:
+            return ViewportDescription::ValuePortrait;
+        case CSSValueZoom:
+            return defaultValue;
+        case CSSValueInternalExtendToZoom:
+            return ViewportDescription::ValueExtendToZoom;
+        case CSSValueFixed:
+            return 0;
+        default:
+            return defaultValue;
+        }
+    }
+
     const CSSPrimitiveValue* primitiveValue = toCSSPrimitiveValue(value);
 
     if (primitiveValue->isNumber() || primitiveValue->isPx())
         return primitiveValue->getFloatValue();
 
     if (primitiveValue->isFontRelativeLength())
         return primitiveValue->getFloatValue() * m_document->computedStyle()->getFontDescription().computedSize();
 
     if (primitiveValue->isPercentage()) {
         float percentValue = primitiveValue->getFloatValue() / 100.0f;
         switch (id) {
         case CSSPropertyMaxZoom:
         case CSSPropertyMinZoom:
         case CSSPropertyZoom:
             return percentValue;
         default:
-            ASSERT_NOT_REACHED();
+            NOTREACHED();
             break;
         }
     }
 
-    switch (primitiveValue->getValueID()) {
-    case CSSValueAuto:
-        return defaultValue;
-    case CSSValueLandscape:
-        return ViewportDescription::ValueLandscape;
-    case CSSValuePortrait:
-        return ViewportDescription::ValuePortrait;
-    case CSSValueZoom:
-        return defaultValue;
-    case CSSValueInternalExtendToZoom:
-        return ViewportDescription::ValueExtendToZoom;
-    case CSSValueFixed:
-        return 0;
-    default:
-        return defaultValue;
-    }
+    NOTREACHED();
+    return defaultValue;
 }
 
 Length ViewportStyleResolver::viewportLengthValue(CSSPropertyID id) const
 {
     ASSERT(id == CSSPropertyMaxHeight
         || id == CSSPropertyMinHeight
         || id == CSSPropertyMaxWidth
         || id == CSSPropertyMinWidth);
 
     const CSSValue* value = m_propertySet->getPropertyCSSValue(id);
-    if (!value || !value->isPrimitiveValue())
+    if (!value || !value->isPrimitiveValue() || !value->isIdentifierValue())
         return Length(); // auto
 
+    if (value->isIdentifierValue()) {
+        CSSValueID valueID = toCSSIdentifierValue(value)->getValueID();
+        if (valueID == CSSValueInternalExtendToZoom)
+            return Length(ExtendToZoom);
+        if (valueID == CSSValueAuto)
+            return Length(Auto);
+    }
+
     const CSSPrimitiveValue* primitiveValue = toCSSPrimitiveValue(value);
-
-    if (primitiveValue->getValueID() == CSSValueInternalExtendToZoom)
-        return Length(ExtendToZoom);
-
     ComputedStyle* documentStyle = m_document->mutableComputedStyle();
 
     // If we have viewport units the conversion will mark the document style as having viewport units.
     bool documentStyleHasViewportUnits = documentStyle->hasViewportUnits();
     documentStyle->setHasViewportUnits(false);
 
     CSSToLengthConversionData::FontSizes fontSizes(documentStyle, documentStyle);
     CSSToLengthConversionData::ViewportSize viewportSize(m_document->layoutViewItem());
 
-    if (primitiveValue->getValueID() == CSSValueAuto)
-        return Length(Auto);
-
     Length result = primitiveValue->convertToLength(CSSToLengthConversionData(documentStyle, fontSizes, viewportSize, 1.0f));
     if (documentStyle->hasViewportUnits())
         m_document->setHasViewportUnits();
     documentStyle->setHasViewportUnits(documentStyleHasViewportUnits);
 
     return result;
 }
 
 DEFINE_TRACE(ViewportStyleResolver)
 {
     visitor->trace(m_propertySet);
     visitor->trace(m_document);
 }
 
 } // namespace blink