Reland: [IndexedDB] Delete callbacks state on worker thread exit.

Reland: [IndexedDB] Delete callbacks state on worker thread exit.

During refactoring of r430110 the logic to actually delete the internal
state of IndexedDBCallbacksImpl and IndexedDBDatabaseCallbacksImpl
objects that lives on worker threads was accidentally removed.

This reland will fix the DCHECK encountered when actually exercising
this deletion path.

BUG=667465
Committed: https://crrev.com/6d1044b605d7dfa073437e09fd966f306137a0c9
Cr-Commit-Position: refs/heads/master@{#434030}

[Reilly Grant (use Gerrit), 2016-11-22 19:03:16]

Fix DCHECK.

[Reilly Grant (use Gerrit), 2016-11-22 19:03:26]

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

[Reilly Grant (use Gerrit), 2016-11-22 19:03:40]

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

[Reilly Grant (use Gerrit), 2016-11-22 19:03:40]

Please take another look.

[commit-bot: I haz the power, 2016-11-22 19:03:51]

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

[jsbell, 2016-11-22 19:16:12]

lgtm

[Reilly Grant (use Gerrit), 2016-11-22 19:27:26]

The CQ bit was unchecked by reillyg@chromium.org

[Reilly Grant (use Gerrit), 2016-11-22 19:27:26]

The CQ bit was unchecked by reillyg@chromium.org

[Reilly Grant (use Gerrit), 2016-11-22 19:27:48]

On 2016/11/22 at 19:16:12, jsbell wrote:
> lgtm

No, this is still broken. :(

[Reilly Grant (use Gerrit), 2016-11-22 19:37:15]

Avoid reentrancy.

[Reilly Grant (use Gerrit), 2016-11-22 19:38:17]

Tested more thoroughly this time. Please take another look. I don't like that we
need hacks like this for now.

[jsbell, 2016-11-22 19:45:13]

ugh, yeah. lgtm but there's so much clean-up ahead of us.

[Reilly Grant (use Gerrit), 2016-11-22 20:22:26]

The CQ bit was checked by reillyg@chromium.org

[commit-bot: I haz the power, 2016-11-22 20:23:16]

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-11-22 21:12:32]

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

[commit-bot: I haz the power, 2016-11-22 21:12:33]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[Reilly Grant (use Gerrit), 2016-11-22 21:40:03]

Rebased.

[Reilly Grant (use Gerrit), 2016-11-22 21:40:08]

The CQ bit was checked by reillyg@chromium.org

[Reilly Grant (use Gerrit), 2016-11-22 21:40:09]

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/2524823002/#ps60001
(title: "Rebased.")

[commit-bot: I haz the power, 2016-11-22 21:40:39]

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-11-22 23:57:16]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1479850808894350,
"parent_rev": "4ff9450829d7ab25b6bb2127b8b996fd2bd2f6ef", "commit_rev":
"a3be46ca9d35950cc57456c1ce82bfeef94d6caf"}

[commit-bot: I haz the power, 2016-11-22 23:57:37]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-11-23 00:00:42]

Description was changed from

==========
Reland: [IndexedDB] Delete callbacks state on worker thread exit.

During refactoring of r430110 the logic to actually delete the internal
state of IndexedDBCallbacksImpl and IndexedDBDatabaseCallbacksImpl
objects that lives on worker threads was accidentally removed.

This reland will fix the DCHECK encountered when actually exercising
this deletion path.

BUG=667465
==========

to

==========
Reland: [IndexedDB] Delete callbacks state on worker thread exit.

During refactoring of r430110 the logic to actually delete the internal
state of IndexedDBCallbacksImpl and IndexedDBDatabaseCallbacksImpl
objects that lives on worker threads was accidentally removed.

This reland will fix the DCHECK encountered when actually exercising
this deletion path.

BUG=667465

Committed: https://crrev.com/6d1044b605d7dfa073437e09fd966f306137a0c9
Cr-Commit-Position: refs/heads/master@{#434030}
==========

[commit-bot: I haz the power, 2016-11-23 00:00:45]

Patchset 4 (id:??) landed as
https://crrev.com/6d1044b605d7dfa073437e09fd966f306137a0c9
Cr-Commit-Position: refs/heads/master@{#434030}