Lazy Parse CSS

third_party/WebKit/Source/core/css/StyleRule.cpp

 /*
  * (C) 1999-2003 Lars Knoll (knoll@kde.org)
  * (C) 2002-2003 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2002, 2005, 2006, 2008, 2012 Apple Inc. All rights reserved.
  *
  * 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 194 matching lines @@
     return rule;
 }
 
 unsigned StyleRule::averageSizeInBytes()
 {
     return sizeof(StyleRule) + sizeof(CSSSelector) + StylePropertySet::averageSizeInBytes();
 }
 
 StyleRule::StyleRule(CSSSelectorList selectorList, StylePropertySet* properties)
     : StyleRuleBase(Style)
+    , m_selectorList(std::move(selectorList))
     , m_properties(properties)
 {
-    m_selectorList = std::move(selectorList);
+}
+
+StyleRule::StyleRule(CSSSelectorList selectorList, std::unique_ptr<CSSParser::DeferredPropertiesClosure> deferredProperties)
+    : StyleRuleBase(Style)
+    , m_selectorList(std::move(selectorList))
+    , m_deferred(std::move(deferredProperties))
+{
+}
+
+const StylePropertySet& StyleRule::properties() const
+{
+    if (!m_properties) {
+        m_properties = (*m_deferred)();

[Timothy Loh, 2016/09/26 05:18:00]

Should we clear the escaped string store since it won't be used again?

[Charlie Harrison, 2016/09/26 22:14:45]

Unfortunately the escaped string pool has *all* the escaped strings from
tokenizing, so as the CL is currently designed we won't be able to clear it
until all properties are lazily parsed.

Two possible solutions:
 - don't lazily parse properties which use escaped strings
 - do a string copy and store the escaped string with the closure

Both solutions would require changing the tokenizer in ways I haven't fully
thought through (would we need to add an extra bool "hasEscapedString" to all
tokens?)

[Timothy Loh, 2016/09/27 05:15:45]

Oh right, it's fine as is then :)

+        m_deferred.reset();
+    }
+    return *m_properties;
 }
 
 StyleRule::StyleRule(const StyleRule& o)
     : StyleRuleBase(o)
-    , m_properties(o.m_properties->mutableCopy())
     , m_selectorList(o.m_selectorList.copy())
+    , m_properties(o.properties().mutableCopy())
 {
 }
 
 StyleRule::~StyleRule()
 {
 }
 
 MutableStylePropertySet& StyleRule::mutableProperties()
 {
-    if (!m_properties->isMutable())
-        m_properties = m_properties->mutableCopy();
+    // Ensure m_properties is initialized.
+    const StylePropertySet& props = properties();
+    if (!props.isMutable())
+        m_properties = props.mutableCopy();
     return *toMutableStylePropertySet(m_properties.get());
 }
 
+bool StyleRule::propertiesHaveFailedOrCanceledSubresources() const
+{
+    return m_properties && m_properties->hasFailedOrCanceledSubresources();
+}
+
+bool StyleRule::hasEmptyProperties() const
+{
+    // We only lazily parse if the property block has a nonzero amount of
+    // tokens. Thus, consider all lazily initialized properties to be non-empty.
+    return m_properties && m_properties->isEmpty();
+}
+
 DEFINE_TRACE_AFTER_DISPATCH(StyleRule)
 {
     visitor->trace(m_properties);
     StyleRuleBase::traceAfterDispatch(visitor);
 }
 
 StyleRulePage::StyleRulePage(CSSSelectorList selectorList, StylePropertySet* properties)
     : StyleRuleBase(Page)
     , m_properties(properties)
     , m_selectorList(std::move(selectorList))
@@ skipping 139 matching lines @@
     return *toMutableStylePropertySet(m_properties);
 }
 
 DEFINE_TRACE_AFTER_DISPATCH(StyleRuleViewport)
 {
     visitor->trace(m_properties);
     StyleRuleBase::traceAfterDispatch(visitor);
 }
 
 } // namespace blink