Indexed DB: Use BindOnce / OnceCallback / OnceClosure where applicable

Indexed DB: Use BindOnce / OnceCallback / OnceClosure where applicable

Within the Indexed DB back end code, use BindOnce when posting tasks
and setting mojo pipe connection error handlers, and make transaction
operations OnceCallbacks since they never execute twice.

(The storage::QuotaClient API still requires the generic base::Bind
for now even though the callbacks are logically Once.)

BUG=714018
Review-Url: https://codereview.chromium.org/2941353002
Cr-Commit-Position: refs/heads/master@{#480917}
Committed: https://chromium.googlesource.com/chromium/src/+/fd99b4a093a45f4252aca3a683333dcb0a7cb30f

[jsbell, 2017-06-16 21:15:01]

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

[commit-bot: I haz the power, 2017-06-16 21:15:15]

Dry run: CQ is trying da patch.

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

[jsbell, 2017-06-16 21:24:19]

Description was changed from

==========
Indexed DB: Use BindOnce / OnceCallback / OnceClosure where applicable

Within the Indexed DB back end code, use BindOnce when posting tasks
and make transaction operations OnceCallbacks since they should
never execute twice.

R=dmurph@chromium.org
==========

to

==========
Indexed DB: Use BindOnce / OnceCallback / OnceClosure where applicable

Within the Indexed DB back end code, use BindOnce when posting tasks
and setting mojo pipe connection error handlers, and make transaction
operations OnceCallbacks since they never execute twice.

(The storage::QuotaClient API still requires the generic base::Bind
for now even though the callbacks are logically Once.)
==========

[jsbell, 2017-06-16 21:24:30]

dmurph@ - please take a look?

[commit-bot: I haz the power, 2017-06-16 22:39:56]

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

[commit-bot: I haz the power, 2017-06-16 22:39:57]

Dry run: This issue passed the CQ dry run.

[jsbell, 2017-06-19 17:17:44]

Description was changed from

==========
Indexed DB: Use BindOnce / OnceCallback / OnceClosure where applicable

Within the Indexed DB back end code, use BindOnce when posting tasks
and setting mojo pipe connection error handlers, and make transaction
operations OnceCallbacks since they never execute twice.

(The storage::QuotaClient API still requires the generic base::Bind
for now even though the callbacks are logically Once.)
==========

to

==========
Indexed DB: Use BindOnce / OnceCallback / OnceClosure where applicable

Within the Indexed DB back end code, use BindOnce when posting tasks
and setting mojo pipe connection error handlers, and make transaction
operations OnceCallbacks since they never execute twice.

(The storage::QuotaClient API still requires the generic base::Bind
for now even though the callbacks are logically Once.)

BUG=714018
==========

[dmurph, 2017-06-20 17:59:23]

lgtm

[jsbell, 2017-06-20 18:30:40]

The CQ bit was checked by jsbell@chromium.org

[commit-bot: I haz the power, 2017-06-20 18:31:07]

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-06-20 19:55:15]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1497983440373110, "parent_rev":
"aca7a78d0cf36f18816de26361a34f0d0b9c7e6c", "commit_rev":
"fd99b4a093a45f4252aca3a683333dcb0a7cb30f"}

[commit-bot: I haz the power, 2017-06-20 19:55:27]

Description was changed from

==========
Indexed DB: Use BindOnce / OnceCallback / OnceClosure where applicable

Within the Indexed DB back end code, use BindOnce when posting tasks
and setting mojo pipe connection error handlers, and make transaction
operations OnceCallbacks since they never execute twice.

(The storage::QuotaClient API still requires the generic base::Bind
for now even though the callbacks are logically Once.)

BUG=714018
==========

to

==========
Indexed DB: Use BindOnce / OnceCallback / OnceClosure where applicable

Within the Indexed DB back end code, use BindOnce when posting tasks
and setting mojo pipe connection error handlers, and make transaction
operations OnceCallbacks since they never execute twice.

(The storage::QuotaClient API still requires the generic base::Bind
for now even though the callbacks are logically Once.)

BUG=714018

Review-Url: https://codereview.chromium.org/2941353002
Cr-Commit-Position: refs/heads/master@{#480917}
Committed:
https://chromium.googlesource.com/chromium/src/+/fd99b4a093a45f4252aca3a68333...
==========

[commit-bot: I haz the power, 2017-06-20 19:55:29]

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