Changeset 150067 in webkit

Timestamp:

May 14, 2013, 5:51:59 AM (12 years ago)

Author:

abucur@adobe.com

Message:

Modify checkLayout to receive the log container as an optional parameter
​https://bugs.webkit.org/show_bug.cgi?id=112793

Reviewed by Darin Adler.

Add a way to specify the container manually for checkLayout. This is helpful to measure regions for nested
named flows (and probably other cases as well).

• resources/check-layout.js: Added an optional parameter to window.checkLayout.

Location:

trunk/LayoutTests

Files:

• ChangeLog (modified) (1 diff)
• resources/check-layout.js (modified) (2 diffs)

trunk/LayoutTests/ChangeLog

@@ +1 @@
+2013-05-14  Andrei Bucur  <abucur@adobe.com>
+
+        Modify checkLayout to receive the log container as an optional parameter
+        https://bugs.webkit.org/show_bug.cgi?id=112793
+
+        Reviewed by Darin Adler.
+
+        Add a way to specify the container manually for checkLayout. This is helpful to measure regions for nested
+        named flows (and probably other cases as well).
+
+        * resources/check-layout.js: Added an optional parameter to window.checkLayout.
+
 2013-04-30  Balazs Kelemen  <b.kelemen@sisa.samsung.com>
 

trunk/LayoutTests/resources/check-layout.js

@@ -180 +180 @@
 }
 
-window.checkLayout = function(selectorList)
+window.checkLayout = function(selectorList, overrideContainer)
 {
     if (!selectorList) {
@@ -195 +195 @@
         checkedLayout |= checkSubtreeExpectedValues(node, failures);
 
-        var container = node.parentNode.className == 'container' ? node.parentNode : node;
+        var container = overrideContainer || (node.parentNode.className == 'container' ? node.parentNode : node);
 
         var pre = document.createElement('pre');