Make CSSParserContext be garbage collected.

third_party/WebKit/Source/core/css/FontFace.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 48 matching lines @@
 #include "core/frame/UseCounter.h"
 #include "platform/FontFamilyNames.h"
 #include "platform/Histogram.h"
 #include "platform/SharedBuffer.h"
 
 namespace blink {
 
 static const CSSValue* parseCSSValue(const Document* document,
                                      const String& value,
                                      CSSPropertyID propertyID) {
-  CSSParserContext context(*document, UseCounter::getFrom(document));
+  CSSParserContext* context = new CSSParserContext(*document);
+  context->setUseCounter(UseCounter::getFrom(document));

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

It looks not really nice that we have to create CSSParserContext and call
setUserCounter separately.

Once your refactoring is done, what will it look like in the end?

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

Yes, I'm wasn't entirely happy with this. I changed around the construction
interface when I made all the factories so now it doesn't need to be a separate
statement.

   return CSSParser::parseFontFaceDescriptor(propertyID, value, context);
 }
 
 FontFace* FontFace::create(ExecutionContext* context,
                            const AtomicString& family,
                            StringOrArrayBufferOrArrayBufferView& source,
                            const FontFaceDescriptors& descriptors) {
   if (source.isString())
     return create(context, family, source.getAsString(), descriptors);
   if (source.isArrayBuffer())
@@ skipping 560 matching lines @@
 
 bool FontFace::hadBlankText() const {
   return m_cssFontFace->hadBlankText();
 }
 
 bool FontFace::hasPendingActivity() const {
   return m_status == Loading && getExecutionContext();
 }
 
 }  // namespace blink