Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(117)

Issue 12899003: Attempting to stabilize node_model_test on IE9 (Closed)

Created:
7 years, 9 months ago by blois
Modified:
7 years, 9 months ago
Reviewers:
Jacob
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Attempting to stabilize node_model_test on IE9 Basically the test is passing fine on my IE9 VMs and the individual tests are passing on the bots, but it appears that when everything is run together on the bots, that the tests fail. What I believe is happening is that we're delaying the MutationEvents using window.setImmediate which is being polyfilled on IE9 to window.setTimeout(0) and running into a timing issue on slow machines with the test. We should have a much better resolution once we get a proper setImmediate from dart2js. BUG= Committed: https://code.google.com/p/dart/source/detail?r=20119

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+24 lines, -1 line) Patch
M tests/html/node_model_test.dart View 5 chunks +24 lines, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
blois
7 years, 9 months ago (2013-03-15 22:50:58 UTC) #1
Jacob
lgtm
7 years, 9 months ago (2013-03-15 22:52:29 UTC) #2
blois
7 years, 9 months ago (2013-03-15 22:52:59 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as r20119 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698