[IndexedDB] Fixed possible refcount race

[IndexedDB] Fixed possible refcount race

BUG=716728
Review-Url: https://codereview.chromium.org/2892703003
Cr-Commit-Position: refs/heads/master@{#473029}
Committed: https://chromium.googlesource.com/chromium/src/+/957b96d1ad97e774141de94caec6644174f5d651

[dmurph, 2017-05-18 20:05:58]

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

[commit-bot: I haz the power, 2017-05-18 20:07:41]

Dry run: 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, 2017-05-18 21:55:38]

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

[commit-bot: I haz the power, 2017-05-18 21:55:38]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[dmurph, 2017-05-18 23:51:42]

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

[dmurph, 2017-05-18 23:51:43]

Victor, can you PTAL?

[dmurph, 2017-05-18 23:51:56]

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

[commit-bot: I haz the power, 2017-05-18 23:52:59]

Dry run: CQ is trying da patch.

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

[pwnall, 2017-05-19 00:14:21]

On 2017/05/18 23:51:43, dmurph wrote:
> Victor, can you PTAL?

LGTM w/ nit. Thanks for fixing raciness!

[pwnall, 2017-05-19 00:16:32]

https://codereview.chromium.org/2892703003/diff/1/content/browser/indexed_db/...
File content/browser/indexed_db/indexed_db_context_impl.cc (left):

https://codereview.chromium.org/2892703003/diff/1/content/browser/indexed_db/...
content/browser/indexed_db/indexed_db_context_impl.cc:472: factory_ = NULL;
why not nullptr?

[dmurph, 2017-05-19 00:18:27]

The CQ bit was unchecked by dmurph@chromium.org

[dmurph, 2017-05-19 00:18:30]

The CQ bit was checked by dmurph@chromium.org

[commit-bot: I haz the power, 2017-05-19 00:18:46]

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, 2017-05-19 01:58:19]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495153110840560, "parent_rev":
"30250e647f84a3b5a27dea2aceb7b5ef01361f2e", "commit_rev":
"957b96d1ad97e774141de94caec6644174f5d651"}

[commit-bot: I haz the power, 2017-05-19 01:58:33]

Description was changed from

==========
[IndexedDB] Fixed possible refcount race

BUG=716728
==========

to

==========
[IndexedDB] Fixed possible refcount race

BUG=716728

Review-Url: https://codereview.chromium.org/2892703003
Cr-Commit-Position: refs/heads/master@{#473029}
Committed:
https://chromium.googlesource.com/chromium/src/+/957b96d1ad97e774141de94caec6...
==========

[commit-bot: I haz the power, 2017-05-19 01:58:34]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/957b96d1ad97e774141de94caec6...