Add UseCounters for certain properties on CSSStyleDeclaration

Source/bindings/v8/custom/V8CSSStyleDeclarationCustom.cpp

 /*
  * Copyright (C) 2007-2011 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 64 matching lines @@
     unsigned length = propertyName.length();
     for (unsigned i = 1; i < length; ++i) {
         if (!prefix[i])
             return isASCIIUpper(propertyName[i]);
         if (propertyName[i] != prefix[i])
             return false;
     }
     return false;
 }
 
-class CSSPropertyInfo {
-public:
-    CSSPropertyID propID;
+struct CSSPropertyInfo {
+    unsigned propID: 30; // CSSPropertyID
+    unsigned nameWithDash: 1;
+    unsigned nameWithCssPrefix: 1;
 };
 
 // When getting properties on CSSStyleDeclarations, the name used from
 // Javascript and the actual name of the property are not the same, so
 // we have to do the following translation. The translation turns upper
 // case characters into lower case characters and inserts dashes to
 // separate words.
 //
 // Example: 'backgroundPositionY' -> 'background-position-y'
 //
 // Also, certain prefixes such as 'css-' are stripped.
 static CSSPropertyInfo* cssPropertyInfo(v8::Handle<v8::String> v8PropertyName)
 {
     String propertyName = toWebCoreString(v8PropertyName);
     typedef HashMap<String, CSSPropertyInfo*> CSSPropertyInfoMap;
     DEFINE_STATIC_LOCAL(CSSPropertyInfoMap, map, ());
     CSSPropertyInfo* propInfo = map.get(propertyName);
     if (!propInfo) {
         unsigned length = propertyName.length();
         if (!length)
             return 0;
 
         StringBuilder builder;
         builder.reserveCapacity(length);
 
         unsigned i = 0;
+        bool hasSeenDash = false;
+        bool hasSeenCssPrefix = false;
 
-        if (hasCSSPropertyNamePrefix(propertyName, "css"))
+        if (hasCSSPropertyNamePrefix(propertyName, "css")) {
+            hasSeenCssPrefix = true;
             i += 3;
-        else if (hasCSSPropertyNamePrefix(propertyName, "webkit"))
+        } else if (hasCSSPropertyNamePrefix(propertyName, "webkit")) {
             builder.append('-');
-        else if (isASCIIUpper(propertyName[0]))
+        } else if (isASCIIUpper(propertyName[0])) {
             return 0;
+        }
 
         builder.append(toASCIILower(propertyName[i++]));
 
         for (; i < length; ++i) {
             UChar c = propertyName[i];
-            if (!isASCIIUpper(c))
+            if (!isASCIIUpper(c)) {
+                if (c == '-')
+                    hasSeenDash = true;
                 builder.append(c);
+            }
             else {
                 builder.append('-');
                 builder.append(toASCIILower(c));
             }
         }
 
         String propName = builder.toString();
         CSSPropertyID propertyID = cssPropertyID(propName);
         if (propertyID && RuntimeCSSEnabled::isCSSPropertyEnabled(propertyID)) {
             propInfo = new CSSPropertyInfo();
             propInfo->propID = propertyID;
+            propInfo->nameWithDash = hasSeenDash;
+            propInfo->nameWithCssPrefix = hasSeenCssPrefix;
             map.add(propertyName, propInfo);
         }
     }
+
+    if (propInfo) {
+        if (propInfo->nameWithDash)
+            UseCounter::count(activeDOMWindow(), UseCounter::CSSStyleDeclarationPropertyName);
+        if (propInfo->propID == CSSPropertyFloat && !propInfo->nameWithCssPrefix)
+            UseCounter::count(activeDOMWindow(), UseCounter::CSSStyleDeclarationFloatPropertyName);
+    }
+
     return propInfo;
 }
 
 void V8CSSStyleDeclaration::namedPropertyEnumeratorCustom(const v8::PropertyCallbackInfo<v8::Array>& info)
 {
     typedef Vector<String, numCSSProperties - 1> PreAllocatedPropertyVector;
     DEFINE_STATIC_LOCAL(PreAllocatedPropertyVector, propertyNames, ());
     static unsigned propertyNamesLength = 0;
 
     if (propertyNames.isEmpty()) {
@@ skipping 65 matching lines @@
     ExceptionState es(info.GetIsolate());
     imp->setPropertyInternal(static_cast<CSSPropertyID>(propInfo->propID), propertyValue, false, es);
 
     if (es.throwIfNeeded())
         return;
 
     v8SetReturnValue(info, value);
 }
 
 } // namespace WebCore