Title: [125186] trunk
Revision
125186
Author
[email protected]
Date
2012-08-09 10:56:04 -0700 (Thu, 09 Aug 2012)

Log Message

Web Inspector: follow up to r125174 - fix subtype use.

Source/WebCore:

Not reviewed, trivial fix.

* inspector/InjectedScriptSource.js:
(.):

LayoutTests:

Not reviewed.

* inspector/console/console-format-expected.txt:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (125185 => 125186)


--- trunk/LayoutTests/ChangeLog	2012-08-09 17:37:06 UTC (rev 125185)
+++ trunk/LayoutTests/ChangeLog	2012-08-09 17:56:04 UTC (rev 125186)
@@ -1,3 +1,11 @@
+2012-08-09  'Pavel Feldman'  <[email protected]>
+
+        Web Inspector: follow up to r125174 - fix subtype use.
+
+        Not reviewed.
+
+        * inspector/console/console-format-expected.txt:
+
 2012-08-09  Brady Eidson  <[email protected]>
 
         Layout Test fast/forms/autocomplete-off-with-default-value-does-not-clear.html is failing (on Chromium)

Modified: trunk/LayoutTests/inspector/console/console-format-expected.txt (125185 => 125186)


--- trunk/LayoutTests/inspector/console/console-format-expected.txt	2012-08-09 17:37:06 UTC (rev 125185)
+++ trunk/LayoutTests/inspector/console/console-format-expected.txt	2012-08-09 17:56:04 UTC (rev 125186)
@@ -55,7 +55,7 @@
 ["test", "test2", undefined × 2, "test4", undefined × 5] console-format.html:21
 Array[10] console-format.html:22
 /^url\(\s*(?:(?:"(?:[^\\\"]|(?:\\[\da-f]{1,6}\s?|\.))*"|'(?:[^\\\']|(?:\\[\da-f]{1,6}\s?|\.))*')|(?:[!#$%&*-~\w]|(?:\\[\da-f]{1,6}\s?|\.))*)\s*\)/i console-format.html:54
-[/^url\(\s*(?:(?:"(?:[^\\\"]|(?:\\[\da-f]{1,6}\s?|\.))*"|'(?:[^\\\']|(?:\\[\da-f]{1,6}\s?|\.))*')|(?:…] console-format.html:55
+[/^url\(\s*(?:(?:"(?:[^\\\"]|(?:\\[\da-f]{1,6}\s?|\…?:[!#$%&*-~\w]|(?:\\[\da-f]{1,6}\s?|\.))*)\s*\)/i] console-format.html:55
 globals[0]
 /^url\(\s*(?:(?:"(?:[^\\\"]|(?:\\[\da-f]{1,6}\s?|\.))*"|'(?:[^\\\']|(?:\\[\da-f]{1,6}\s?|\.))*')|(?:[!#$%&*-~\w]|(?:\\[\da-f]{1,6}\s?|\.))*)\s*\)/i
 /foo\\bar\sbaz/i console-format.html:54

Modified: trunk/Source/WebCore/ChangeLog (125185 => 125186)


--- trunk/Source/WebCore/ChangeLog	2012-08-09 17:37:06 UTC (rev 125185)
+++ trunk/Source/WebCore/ChangeLog	2012-08-09 17:56:04 UTC (rev 125186)
@@ -1,3 +1,12 @@
+2012-08-09  'Pavel Feldman'  <[email protected]>
+
+        Web Inspector: follow up to r125174 - fix subtype use.
+
+        Not reviewed, trivial fix.
+
+        * inspector/InjectedScriptSource.js:
+        (.):
+
 2012-08-09  Yury Semikhatsky  <[email protected]>
 
         Web Inspector: DocumentEventQueue memory instrumentation

Modified: trunk/Source/WebCore/inspector/InjectedScriptSource.js (125185 => 125186)


--- trunk/Source/WebCore/inspector/InjectedScriptSource.js	2012-08-09 17:37:06 UTC (rev 125185)
+++ trunk/Source/WebCore/inspector/InjectedScriptSource.js	2012-08-09 17:56:04 UTC (rev 125186)
@@ -782,8 +782,8 @@
                 if (type === "function")
                     continue;
 
+                var subtype = injectedScript._subtype(value);
                 var property = { name: name, type: type, value: this._abbreviateString(injectedScript._describe(value), maxLength, subtype === "regexp") };
-                var subtype = injectedScript._subtype(value);
                 if (subtype)
                     property.subtype = subtype;
                 properties.push(property);
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to