Bug 159591 - LayoutTests jquery/attributes.html, core.html, event.html, manipulation.html, and traversing.html are flaky
Summary: LayoutTests jquery/attributes.html, core.html, event.html, manipulation.html,...
Status: RESOLVED DUPLICATE of bug 159616
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-07-08 17:28 PDT by Ryan Haddad
Modified: 2016-07-12 18:16 PDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ryan Haddad 2016-07-08 17:28:04 PDT
Regressions: Unexpected text-only failures
  jquery/attributes.html
  jquery/core.html
  jquery/event.html
  jquery/manipulation.html
  jquery/traversing.html

https://build.webkit.org/results/Apple%20El%20Capitan%20Release%20WK2%20(Tests)/r203006%20(7494)/results.html
https://build.webkit.org/builders/Apple%20Yosemite%20Release%20WK2%20%28Tests%29/builds/15964

--- /Volumes/Data/slave/elcapitan-release-tests-wk2/build/layout-test-results/jquery/core-expected.txt
+++ /Volumes/Data/slave/elcapitan-release-tests-wk2/build/layout-test-results/jquery/core-actual.txt
@@ -1,2 +1,51 @@
-ALERT: 667 tests of 667 passed, 0 failed.
+CONSOLE MESSAGE: line 900: Test map() died, exception and test follows
+CONSOLE MESSAGE: line 901: RangeError: Maximum call stack size exceeded.
+CONSOLE MESSAGE: line 902: function () {
+	expect(2);//expect(6);
 
+	same(
+		jQuery("#ap").map(function(){
+			return jQuery(this).find("a").get();
+		}).get(),
+		q("google", "groups", "anchor1", "mark"),
+		"Array Map"
+	);
+
+	same(
+		jQuery("#ap > a").map(function(){
+			return this.parentNode;
+		}).get(),
+		q("ap","ap","ap"),
+		"Single Map"
+	);
+
+	return;//these haven't been accepted yet
+
+	//for #2616
+	var keys = jQuery.map( {a:1,b:2}, function( v, k ){
+		return k;
+	}, [ ] );
+
+	equals( keys.join(""), "ab", "Map the keys from a hash to an array" );
+
+	var values = jQuery.map( {a:1,b:2}, function( v, k ){
+		return v;
+	}, [ ] );
+
+	equals( values.join(""), "12", "Map the values from a hash to an array" );
+
+	var scripts = document.getElementsByTagName("script");
+	var mapped = jQuery.map( scripts, function( v, k ){
+		return v;
+	}, {length:0} );
+
+	equals( mapped.length, scripts.length, "Map an array(-like) to a hash" );
+
+	var flat = jQuery.map( Array(4), function( v, k ){
+		return k % 2 ? k : [k,k,k];//try mixing array and regular returns
+	});
+
+	equals( flat.join(""), "00012223", "try the new flatten technique(#2616)" );
+}
+ALERT: 666 tests of 667 passed, 1 failed.
+
Comment 1 Ryan Haddad 2016-07-08 17:29:29 PDT
These tests have become flaky failures on Mac. They all fail together.
Comment 2 Alexey Proskuryakov 2016-07-12 16:36:40 PDT
When did this regress, was it <http://trac.webkit.org/r203067>?

Mark, would you be available to take a look?
Comment 3 Mark Lam 2016-07-12 18:16:11 PDT
(In reply to comment #2)
> When did this regress, was it <http://trac.webkit.org/r203067>?
> 
> Mark, would you be available to take a look?

https://build.webkit.org/results/Apple%20El%20Capitan%20Release%20WK2%20(Tests)/r203006%20(7494)/results.html is run on r203006.  Hence, this cannot be due to r203067.
Comment 4 Alexey Proskuryakov 2016-07-12 18:16:46 PDT
Indeed.

*** This bug has been marked as a duplicate of bug 159616 ***