Make CSSParserContext be garbage collected.

third_party/WebKit/Source/core/css/parser/CSSParserContext.h

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CSSParserContext_h
 #define CSSParserContext_h
 
 #include "core/CoreExport.h"
 #include "core/css/parser/CSSParserMode.h"
 #include "core/fetch/ResourceLoaderOptions.h"
 #include "platform/weborigin/KURL.h"
 #include "platform/weborigin/Referrer.h"
 
 namespace blink {
 
+class CSSStyleSheet;
 class Document;
+class StyleSheetContents;
 class UseCounter;
 
-class CORE_EXPORT CSSParserContext {
-  USING_FAST_MALLOC(CSSParserContext);
-
+class CORE_EXPORT CSSParserContext
+    : public GarbageCollectedFinalized<CSSParserContext> {
  public:
   // https://drafts.csswg.org/selectors/#profiles
   enum SelectorProfile { DynamicProfile, StaticProfile };
 
-  CSSParserContext(CSSParserMode,
-                   UseCounter*,
-                   SelectorProfile = DynamicProfile);
+  static CSSParserContext* create(const CSSParserContext*,
+                                  const CSSStyleSheet*);
+  static CSSParserContext* create(const CSSParserContext*,
+                                  const StyleSheetContents*);
+
+  CSSParserContext(CSSParserMode, SelectorProfile = DynamicProfile);

[haraken, 2017/01/12 04:51:13]

Put these constructors in a private section.

[Bret, 2017/01/13 02:15:28]

Done.

   // FIXME: We shouldn't need the UseCounter argument as we could infer it from
   // the Document but some callers want to disable use counting (e.g. the
   // WebInspector).
   CSSParserContext(const Document&,
-                   UseCounter*,
                    const KURL& baseURL = KURL(),
                    const String& charset = emptyString(),
                    SelectorProfile = DynamicProfile);
   // FIXME: This constructor shouldn't exist if we properly piped the UseCounter
   // through the CSS subsystem. Currently the UseCounter life time is too crazy
   // and we need a way to override it.
-  CSSParserContext(const CSSParserContext&, UseCounter*);
+  CSSParserContext(const CSSParserContext*, UseCounter*);
 
   bool operator==(const CSSParserContext&) const;
   bool operator!=(const CSSParserContext& other) const {
     return !(*this == other);
   }
 
   CSSParserMode mode() const { return m_mode; }
   CSSParserMode matchMode() const { return m_matchMode; }
   const KURL& baseURL() const { return m_baseURL; }
   const String& charset() const { return m_charset; }
@@ skipping 15 matching lines @@
   void setMode(CSSParserMode mode) { m_mode = mode; }
   void setBaseURL(const KURL& baseURL) { m_baseURL = baseURL; }
   void setCharset(const String& charset) { m_charset = charset; }
   void setReferrer(const Referrer& referrer) { m_referrer = referrer; }
 
   KURL completeURL(const String& url) const;
 
   // This may return nullptr if counting is disabled.
   // See comments on constructors.
   UseCounter* useCounter() const { return m_useCounter; }
+  bool isUseCounterRecordingEnabled() const { return m_useCounter; }
+  void setUseCounter(UseCounter* useCounter) { m_useCounter = useCounter; }
 
   ContentSecurityPolicyDisposition shouldCheckContentSecurityPolicy() const {
     return m_shouldCheckContentSecurityPolicy;
   }
 
+  DEFINE_INLINE_TRACE() {}
+
  private:
   KURL m_baseURL;
   String m_charset;
   CSSParserMode m_mode;
   CSSParserMode m_matchMode;
   SelectorProfile m_profile = DynamicProfile;
   Referrer m_referrer;
   bool m_isHTMLDocument;
   bool m_useLegacyBackgroundSizeShorthandBehavior;
   ContentSecurityPolicyDisposition m_shouldCheckContentSecurityPolicy;
 
   UseCounter* m_useCounter;
 };
 
-CORE_EXPORT const CSSParserContext& strictCSSParserContext();
+CORE_EXPORT const CSSParserContext* strictCSSParserContext();
 
 }  // namespace blink
 
-#endif  // CSSParserMode_h
+#endif  // CSSParserContext_h