De-flake IndexedDB layout tests.

De-flake IndexedDB layout tests.

Per the IndexedDB specification, a versionchange transaction is running
until the open request's transaction member is set to null, which
should happen (synchronously) after the onabort() event handler is
called. Therefore, tests can only assume that a transaction stopped
running after the onabort event handler has occurred.

BUG=657968
Committed: https://crrev.com/c6c814b48c0ec98a495202b17ce55a0aef8364d4
Cr-Commit-Position: refs/heads/master@{#437974}

[pwnall, 2016-12-07 02:29:59]

Patchset #1 (id:1) has been deleted

[pwnall, 2016-12-07 02:30:13]

The CQ bit was checked by pwnall@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-07 02:30:49]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[pwnall, 2016-12-07 02:36:04]

Description was changed from

==========
De-flake IndexedDB layout tests.

BUG=657968
==========

to

==========
De-flake IndexedDB layout tests.

Per the IndexedDB specification, a versionchange transaction is running
until the open request's transaction member is set to null, which
should happen (synchronously) after the onabort() event handler is
called. Therefore, tests can only assume that a transaction stopped
running after the onabort event handler has occurred.

BUG=657968
==========

[pwnall, 2016-12-07 02:37:37]

pwnall@chromium.org changed reviewers:
+ jsbell@chromium.org

[pwnall, 2016-12-07 02:37:38]

PTAL?

I ran each test 1,000 times.

My interpretation is based on S 5.7 of the standard.

[commit-bot: I haz the power, 2016-12-07 03:46:34]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-07 03:46:35]

Dry run: This issue passed the CQ dry run.

[jsbell, 2016-12-12 22:20:51]

lgtm

[pwnall, 2016-12-12 23:00:02]

The CQ bit was checked by pwnall@chromium.org

[commit-bot: I haz the power, 2016-12-12 23:01:51]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-12 23:08:36]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-12 23:08:37]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[pwnall, 2016-12-12 23:49:50]

The CQ bit was checked by pwnall@chromium.org

[pwnall, 2016-12-12 23:49:51]

The patchset sent to the CQ was uploaded after l-g-t-m from jsbell@chromium.org
Link to the patchset: https://codereview.chromium.org/2538343002/#ps40001
(title: "Rebased.")

[commit-bot: I haz the power, 2016-12-12 23:51:01]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-13 01:10:39]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1481586590964050,
"parent_rev": "c04768eaa820e37a97b95ca4b8e8b9dfda0ea05d", "commit_rev":
"c90c1fbccd9c711c6db7493eb082ac30c9aec337"}

[commit-bot: I haz the power, 2016-12-13 01:12:07]

Description was changed from

==========
De-flake IndexedDB layout tests.

Per the IndexedDB specification, a versionchange transaction is running
until the open request's transaction member is set to null, which
should happen (synchronously) after the onabort() event handler is
called. Therefore, tests can only assume that a transaction stopped
running after the onabort event handler has occurred.

BUG=657968
==========

to

==========
De-flake IndexedDB layout tests.

Per the IndexedDB specification, a versionchange transaction is running
until the open request's transaction member is set to null, which
should happen (synchronously) after the onabort() event handler is
called. Therefore, tests can only assume that a transaction stopped
running after the onabort event handler has occurred.

BUG=657968

Review-Url: https://codereview.chromium.org/2538343002
==========

[commit-bot: I haz the power, 2016-12-13 01:12:08]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-12-13 01:16:26]

Description was changed from

==========
De-flake IndexedDB layout tests.

Per the IndexedDB specification, a versionchange transaction is running
until the open request's transaction member is set to null, which
should happen (synchronously) after the onabort() event handler is
called. Therefore, tests can only assume that a transaction stopped
running after the onabort event handler has occurred.

BUG=657968

Review-Url: https://codereview.chromium.org/2538343002
==========

to

==========
De-flake IndexedDB layout tests.

Per the IndexedDB specification, a versionchange transaction is running
until the open request's transaction member is set to null, which
should happen (synchronously) after the onabort() event handler is
called. Therefore, tests can only assume that a transaction stopped
running after the onabort event handler has occurred.

BUG=657968

Committed: https://crrev.com/c6c814b48c0ec98a495202b17ce55a0aef8364d4
Cr-Commit-Position: refs/heads/master@{#437974}
==========

[commit-bot: I haz the power, 2016-12-13 01:16:27]

Patchset 2 (id:??) landed as
https://crrev.com/c6c814b48c0ec98a495202b17ce55a0aef8364d4
Cr-Commit-Position: refs/heads/master@{#437974}