Changeset 93400 in webkit

Timestamp:

Aug 19, 2011 6:41:10 AM (13 years ago)

Author:

caseq@chromium.org

Message:

Web Inspector: [Extensions API][chromium] webInspector.resources deprecation warning is given even for extensions that don't use webInspector.resources
​https://bugs.webkit.org/show_bug.cgi?id=66553

Reviewed by Pavel Feldman.

• src/js/DevTools.js:

(WebInspector.platformExtensionAPI):

Location:

trunk/Source/WebKit/chromium

Files:

• ChangeLog (modified) (1 diff)
• src/js/DevTools.js (modified) (1 diff)

trunk/Source/WebKit/chromium/ChangeLog

@@ +1 @@
+2011-08-19  Andrey Kosyakov  <caseq@chromium.org>
+
+        Web Inspector: [Extensions API][chromium] webInspector.resources deprecation warning is given even for extensions that don't use webInspector.resources
+        https://bugs.webkit.org/show_bug.cgi?id=66553
+
+        Reviewed by Pavel Feldman.
+
+        * src/js/DevTools.js:
+        (WebInspector.platformExtensionAPI):
+
 2011-08-18  James Kozianski  <koz@chromium.org>
 

trunk/Source/WebKit/chromium/src/js/DevTools.js

@@ -169 +169 @@
     chrome.experimental = chrome.experimental || {};
     chrome.experimental.devtools = chrome.experimental.devtools || {};
-    for (var property in webInspector)
-        chrome.experimental.devtools[property] = webInspector[property];
+
+    var properties = Object.getOwnPropertyNames(webInspector);
+    for (var i = 0; i < properties.length; ++i) {
+        var descriptor = Object.getOwnPropertyDescriptor(webInspector, properties[i]);
+        Object.defineProperty(chrome.experimental.devtools, properties[i], descriptor);
+    }
 }