CSS Length calculation with MediaValues

Source/core/css/CSSToLengthConversionData.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 13 matching lines @@
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "config.h"
 #include "core/css/CSSToLengthConversionData.h"
 
+#include "core/css/MediaValues.h"
 #include "core/rendering/RenderView.h"
 #include "core/rendering/style/RenderStyle.h"
 
 namespace WebCore {
 
 CSSToLengthConversionData::CSSToLengthConversionData(const RenderStyle* style, const RenderStyle* rootStyle, const RenderView* renderView, float zoom, bool computingFontSize)
     : m_style(style)
     , m_rootStyle(rootStyle)
+    , m_mediaValues(0)
     , m_viewportWidth(renderView ? renderView->layoutViewportWidth() : 0)
     , m_viewportHeight(renderView ? renderView->layoutViewportHeight() : 0)
     , m_zoom(zoom)
     , m_useEffectiveZoom(false)
     , m_computingFontSize(computingFontSize)
 {
     ASSERT(zoom > 0);
 }
 
+CSSToLengthConversionData::CSSToLengthConversionData(const MediaValues* mediaValues, float zoom, bool computingFontSize)

[esprehn, 2014/04/08 17:40:39]

This should be a reference

+    : m_style(0)
+    , m_rootStyle(0)
+    , m_mediaValues(mediaValues)
+    , m_viewportWidth(mediaValues ? mediaValues->viewportWidth() : 0)
+    , m_viewportHeight(mediaValues ? mediaValues->viewportHeight() : 0)

[esprehn, 2014/04/08 17:40:39]

These null checks make no sense, you ASSERT(mediaValues) below.

+    , m_zoom(zoom)
+    , m_useEffectiveZoom(false)
+    , m_computingFontSize(computingFontSize)
+{
+    ASSERT(zoom > 0);
+    ASSERT(mediaValues);
+}
+
 CSSToLengthConversionData::CSSToLengthConversionData(const RenderStyle* style, const RenderStyle* rootStyle, const RenderView* renderView, bool computingFontSize)
     : m_style(style)
     , m_rootStyle(rootStyle)
+    , m_mediaValues(0)
     , m_viewportWidth(renderView ? renderView->layoutViewportWidth() : 0)
     , m_viewportHeight(renderView ? renderView->layoutViewportHeight() : 0)
     , m_useEffectiveZoom(true)
     , m_computingFontSize(computingFontSize)
 {
 }
 
 CSSToLengthConversionData::CSSToLengthConversionData(const RenderStyle* style, const RenderStyle* rootStyle, float viewportWidth, float viewportHeight, float zoom, bool computingFontSize)
     : m_style(style)
     , m_rootStyle(rootStyle)
+    , m_mediaValues(0)
     , m_viewportWidth(viewportWidth)
     , m_viewportHeight(viewportHeight)
     , m_zoom(zoom)
     , m_useEffectiveZoom(false)
     , m_computingFontSize(computingFontSize)
 {
     ASSERT(zoom > 0);
 }
 
 float CSSToLengthConversionData::zoom() const
 {
     if (m_useEffectiveZoom)
         return m_style ? m_style->effectiveZoom() : 1;
     return m_zoom;
 }
 
 double CSSToLengthConversionData::viewportWidthPercent() const
 {
-    m_style->setHasViewportUnits();
+    // When this is called off main thread, the RenderStyle hasViewportUnits flag will not be set
+    if (m_style)
+        m_style->setHasViewportUnits();
     return m_viewportWidth / 100;
 }
 double CSSToLengthConversionData::viewportHeightPercent() const
 {
-    m_style->setHasViewportUnits();

[eseidel, 2014/04/08 16:51:28]

Accessors shouldn't mutate things as a side-effect.  Obviously not the fault of
your patch, but this design seems wrong.  (I know we have similar problems where
we mutate styles during matching unfortunately)

+    // When this is called off main thread, the RenderStyle hasViewportUnits flag will not be set
+    if (m_style)
+        m_style->setHasViewportUnits();
     return m_viewportHeight / 100;
 }
 double CSSToLengthConversionData::viewportMinPercent() const
 {
-    m_style->setHasViewportUnits();
+    // When this is called off main thread, the RenderStyle hasViewportUnits flag will not be set
+    if (m_style)
+        m_style->setHasViewportUnits();
     return std::min(m_viewportWidth, m_viewportHeight) / 100;
 }
 double CSSToLengthConversionData::viewportMaxPercent() const
 {
-    m_style->setHasViewportUnits();
+    // When this is called off main thread, the RenderStyle hasViewportUnits flag will not be set
+    if (m_style)
+        m_style->setHasViewportUnits();
     return std::max(m_viewportWidth, m_viewportHeight) / 100;
 }
 
+bool CSSToLengthConversionData::computingFontSize() const
+{
+    return m_computingFontSize;
+}
+
+double CSSToLengthConversionData::fontSpecifiedSize() const
+{
+    if (m_style)
+        return m_style->fontDescription().specifiedSize();
+    ASSERT(m_mediaValues);
+    return m_mediaValues->defaultFontSize();
+}
+
+double CSSToLengthConversionData::fontComputedSize() const
+{
+    if (m_style)
+        return m_style->fontDescription().computedSize();
+    ASSERT(m_mediaValues);
+    return m_mediaValues->computedFontSize();
+}
+
+double CSSToLengthConversionData::rootFontSpecifiedSize() const
+{
+    if (m_rootStyle)
+        return m_rootStyle->fontDescription().specifiedSize();
+    ASSERT(m_mediaValues);
+    return m_mediaValues->defaultFontSize();
+}
+
+double CSSToLengthConversionData::rootFontComputedSize() const
+{
+    if (m_rootStyle)
+        return m_rootStyle->fontDescription().computedSize();
+    ASSERT(m_mediaValues);
+    return m_mediaValues->computedFontSize();
+}
+
+bool CSSToLengthConversionData::hasXHeight() const
+{
+    if (m_style)
+        return m_style->fontMetrics().hasXHeight();
+    ASSERT(m_mediaValues);
+    return m_mediaValues->hasXHeight();
+}
+
+double CSSToLengthConversionData::xHeight() const
+{
+    if (m_style)
+        return m_style->fontMetrics().xHeight();
+    ASSERT(m_mediaValues);
+    return m_mediaValues->xHeight();
+}
+
+double CSSToLengthConversionData::zeroWidth() const
+{
+    if (m_style)
+        return m_style->fontMetrics().zeroWidth();
+    ASSERT(m_mediaValues);
+    return m_mediaValues->zeroWidth();
+}
+
 } // namespace WebCore