Add some additional SMIL use counters

third_party/WebKit/Source/core/svg/animation/SVGSMILElement.cpp

 /*
  * Copyright (C) 2008 Apple 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:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 299 matching lines @@
 }
 
 Node::InsertionNotificationRequest SVGSMILElement::insertedInto(ContainerNode* rootParent)
 {
     SVGElement::insertedInto(rootParent);
 
     if (!rootParent->inShadowIncludingDocument())
         return InsertionDone;
 
     Deprecation::countDeprecation(document(), UseCounter::SVGSMILElementInDocument);
+    if (document().isLoadCompleted())
+        UseCounter::count(&document(), UseCounter::SVGSMILElementInsertedAfterLoad);
 
     setAttributeName(constructQualifiedName(this, fastGetAttribute(SVGNames::attributeNameAttr)));
     SVGSVGElement* owner = ownerSVGElement();
     if (!owner)
         return InsertionDone;
 
     m_timeContainer = owner->timeContainer();
     ASSERT(m_timeContainer);
     m_timeContainer->setDocumentOrderIndexesDirty();
 
@@ skipping 133 matching lines @@
 
     Condition::Type type;
     int repeat = -1;
     if (nameString.startsWith("repeat(") && nameString.endsWith(')')) {
         repeat = nameString.substring(7, nameString.length() - 8).toUIntStrict(&ok);
         if (!ok)
             return false;
         nameString = "repeatn";
         type = Condition::EventBase;
     } else if (nameString == "begin" || nameString == "end") {
+        UseCounter::count(&document(), UseCounter::SVGSMILBeginOrEndSyncbaseValue);
         if (baseID.isEmpty())
             return false;

[shans, 2016/07/11 22:49:24]

Would it be worth counting after this point? Seems like declarations where
baseID.isEmpty() is true don't have any effect?

[suzyh_UTC10 (ex-contributor), 2016/07/13 06:04:44]

Done.

         type = Condition::Syncbase;
     } else if (nameString.startsWith("accesskey(")) {
         // FIXME: accesskey() support.
         type = Condition::AccessKey;
-    } else
+    } else {
+        UseCounter::count(&document(), UseCounter::SVGSMILBeginOrEndEventValue);

[shans, 2016/07/11 22:49:24]

Interesting that this is the catch-all condition. It means we might slightly
over-count, but that's probably OK.

         type = Condition::EventBase;
+    }
 
     m_conditions.append(Condition::create(type, beginOrEnd, baseID, nameString, offset, repeat));
 
     if (type == Condition::EventBase && beginOrEnd == End)
         m_hasEndEventConditions = true;
 
     return true;
 }
 
 void SVGSMILElement::parseBeginOrEnd(const String& parseString, BeginOrEnd beginOrEnd)
@@ skipping 866 matching lines @@
 {
     visitor->trace(m_targetElement);
     visitor->trace(m_timeContainer);
     visitor->trace(m_conditions);
     visitor->trace(m_syncBaseDependents);
     SVGElement::trace(visitor);
     SVGTests::trace(visitor);
 }
 
 } // namespace blink