AtomicString usage improvements in v8 interceptors.

third_party/WebKit/WebCore/bindings/v8/custom/V8HTMLDocumentCustom.cpp

 /*
  * Copyright (C) 2007, 2008, 2009 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 30 matching lines @@
 
 #include <wtf/RefPtr.h>
 
 namespace WebCore {
 
 NAMED_PROPERTY_DELETER(HTMLDocument)
 {
     // Only handle document.all.  Insert the marker object into the
     // shadow internal field to signal that document.all is no longer
     // shadowed.
-    String key = toWebCoreString(name);
-    if (key != "all")
+    AtomicString key = v8StringToAtomicWebCoreString(name);
+    static const AtomicString all("all");
+    if (key != all)
         return deletionNotHandledByInterceptor();
 
     ASSERT(info.Holder()->InternalFieldCount() == kHTMLDocumentInternalFieldCount);
     v8::Local<v8::Value> marker = info.Holder()->GetInternalField(kHTMLDocumentMarkerIndex);
     info.Holder()->SetInternalField(kHTMLDocumentShadowIndex, marker);
     return v8::True();
 }
 
 NAMED_PROPERTY_GETTER(HTMLDocument)
 {
     INC_STATS("DOM.HTMLDocument.NamedPropertyGetter");
-    AtomicString key = toWebCoreString(name);
+    AtomicString key = v8StringToAtomicWebCoreString(name);
 
     // Special case for document.all.  If the value in the shadow
     // internal field is not the marker object, then document.all has
     // been temporarily shadowed and we return the value.
-    if (key == "all") {
+    static const AtomicString all("all");
+    if (key == all) {
         ASSERT(info.Holder()->InternalFieldCount() == kHTMLDocumentInternalFieldCount);
         v8::Local<v8::Value> marker = info.Holder()->GetInternalField(kHTMLDocumentMarkerIndex);
         v8::Local<v8::Value> value = info.Holder()->GetInternalField(kHTMLDocumentShadowIndex);
         if (marker != value)
             return value;
     }
 
     HTMLDocument* htmlDocument = V8Proxy::DOMWrapperToNode<HTMLDocument>(info.Holder());
 
     // Fast case for named elements that are not there.
@@ skipping 102 matching lines @@
 
 ACCESSOR_SETTER(HTMLDocumentAll)
 {
     INC_STATS("DOM.HTMLDocument.all._set");
     v8::Handle<v8::Object> holder = info.Holder();
     ASSERT(info.Holder()->InternalFieldCount() == kHTMLDocumentInternalFieldCount);
     info.Holder()->SetInternalField(kHTMLDocumentShadowIndex, value);
 }
 
 } // namespace WebCore