bindings,devtools: Shows DOM attributes' values in DevTools.

Source/bindings/core/v8/custom/V8InjectedScriptHostCustom.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 121 matching lines @@
                     result = constructorName;
             }
         }
         if (toCoreStringWithUndefinedOrNullCheck(result) == "Object" && object->IsFunction())
             result = v8AtomicString(info.GetIsolate(), "Function");
     }
 
     v8SetReturnValue(info, result);
 }
 
+void V8InjectedScriptHost::isDOMWrapperMethodCustom(const v8::FunctionCallbackInfo<v8::Value>& info)
+{
+    if (info.Length() < 1)
+        return;
+
+    v8SetReturnValue(info, V8DOMWrapper::isWrapper(info.GetIsolate(), info[0]));
+}
+
 void V8InjectedScriptHost::isHTMLAllCollectionMethodCustom(const v8::FunctionCallbackInfo<v8::Value>& info)
 {
     if (info.Length() < 1)
         return;
 
-    if (!info[0]->IsObject()) {
-        v8SetReturnValue(info, false);
-        return;
-    }
-
-    v8SetReturnValue(info, V8HTMLAllCollection::hasInstance(info[0], info.GetIsolate()));
-}
-
-void V8InjectedScriptHost::isPopularDOMObjectMethodCustom(const v8::FunctionCallbackInfo<v8::Value>& info)
-{
-    // FIXME: Stop using this function (whilelisting) once we come up with a way
-    // to distinguish DOM wrappers from other obejcts.
-
-    if (info.Length() < 1)
-        return;
-
     if (!info[0]->IsObject()) {
         v8SetReturnValue(info, false);
         return;
     }
 
-    v8::Local<v8::Value> object = info[0];
-    v8::Isolate* isolate = info.GetIsolate();
-    bool isPopularDOMObject = false;
-    static bool (*hasInstances[])(v8::Local<v8::Value>, v8::Isolate*) = {
-        V8Blob::hasInstance,
-        V8DOMError::hasInstance,
-        V8DOMException::hasInstance,
-        V8Element::hasInstance,
-        V8Event::hasInstance,
-        V8EventTarget::hasInstance,
-        V8Location::hasInstance,
-        V8Navigator::hasInstance,
-    };
-    for (size_t i = 0; i < sizeof hasInstances / sizeof *hasInstances; ++i) {
-        if (hasInstances[i](object, isolate)) {
-            isPopularDOMObject = true;
-            break;
-        }
-    }
-
-    v8SetReturnValue(info, isPopularDOMObject);
+    v8SetReturnValue(info, V8HTMLAllCollection::hasInstance(info[0], info.GetIsolate()));
 }
 
 void V8InjectedScriptHost::isTypedArrayMethodCustom(const v8::FunctionCallbackInfo<v8::Value>& info)
 {
     if (info.Length() < 1)
         return;
     v8SetReturnValue(info, info[0]->IsTypedArray());
 }
 
 void V8InjectedScriptHost::subtypeMethodCustom(const v8::FunctionCallbackInfo<v8::Value>& info)
@@ skipping 374 matching lines @@
 
 void V8InjectedScriptHost::setNonEnumPropertyMethodCustom(const v8::FunctionCallbackInfo<v8::Value>& info)
 {
     if (info.Length() < 3 || !info[0]->IsObject() || !info[1]->IsString())
         return;
 
     v8::Local<v8::Object> object = info[0]->ToObject(info.GetIsolate());
     object->ForceSet(info[1], info[2], v8::DontEnum);
 }
 
+void V8InjectedScriptHost::getDOMAttributesWithObservableSideEffectOnGetMethodCustom(const v8::FunctionCallbackInfo<v8::Value>& info)
+{
+    // Returns DOM attributes which have observable side effect on getter,
+    // in the form of
+    //   {interfaceName1: {attributeName1: true,
+    //                     attributeName2: true,
+    //                     ...},
+    //    interfaceName2: {...},
+    //    ...}
+    // These attributes should not be evaluated in the Inspector.
+
+    typedef Vector<std::pair<String, bool>> AttributeSet;

[pfeldman, 2015/03/09 11:41:41]

I don't see why this is being created here, it is way simpler to do this in the
InjectedScriptSource.js

[Yuki, 2015/03/09 12:48:13]

I misunderstood that you disliked to have this code in InjectedScriptSource.js. 
I'm happier with having this code in *.js.

+    AttributeSet requestAttributes;
+    requestAttributes.append(std::make_pair("body", true));
+    AttributeSet responseAttributes;
+    responseAttributes.append(std::make_pair("body", true));
+
+    Vector<std::pair<String, AttributeSet>> attributesWithSideEffect;
+    attributesWithSideEffect.append(std::make_pair("Request", requestAttributes));
+    attributesWithSideEffect.append(std::make_pair("Response", responseAttributes));
+
+    v8SetReturnValue(info, toV8(attributesWithSideEffect, info.Holder(), info.GetIsolate()));
+}
+
 void V8InjectedScriptHost::bindMethodCustom(const v8::FunctionCallbackInfo<v8::Value>& info)
 {
     if (info.Length() < 2 || !info[1]->IsString())
         return;
     InjectedScriptNative* injectedScriptNative = InjectedScriptNative::fromInjectedScriptHost(info.Holder());
     if (!injectedScriptNative)
         return;
 
     v8::Handle<v8::String> v8groupName = info[1]->ToString(info.GetIsolate());
     String groupName = toCoreStringWithUndefinedOrNullCheck(v8groupName);
@@ skipping 21 matching lines @@
     InjectedScriptNative* injectedScriptNative = InjectedScriptNative::fromInjectedScriptHost(info.Holder());
     if (!injectedScriptNative)
         return;
     int id = info[0]->ToInt32(info.GetIsolate())->Value();
     String groupName = injectedScriptNative->groupName(id);
     if (!groupName.isEmpty())
         info.GetReturnValue().Set(v8String(info.GetIsolate(), groupName));
 }
 
 } // namespace blink