Use a scoped_refptr in the IndexedDBCallbacks constructor.

Use a scoped_refptr in the IndexedDBCallbacks constructor.

Follow-up to r433964 to change IndexedDBCallbacks to take its
IndexedDBDispatcherHost argument as a scoped_refptr as suggested by
dcheng@.

BUG=None

Committed: https://crrev.com/25f132ffa3cd92cc25dcc35ea556dabcb6d0895e
Cr-Commit-Position: refs/heads/master@{#434822}

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

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

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

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

[Reilly Grant (use Gerrit), 2016-11-22 21:01:27]

Please take a look.

[commit-bot: I haz the power, 2016-11-22 21:02:37]

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

[cmumford, 2016-11-22 21:40:10]

https://codereview.chromium.org/2521903004/diff/1/content/browser/indexed_db/...
File content/browser/indexed_db/indexed_db_callbacks.h (right):

https://codereview.chromium.org/2521903004/diff/1/content/browser/indexed_db/...
content/browser/indexed_db/indexed_db_callbacks.h:40:
scoped_refptr<IndexedDBDispatcherHost> dispatcher_host,
This change looks good, but can you also change the other two constructors to
preserve symmetry?

[Reilly Grant (use Gerrit), 2016-11-22 21:54:53]

https://codereview.chromium.org/2521903004/diff/1/content/browser/indexed_db/...
File content/browser/indexed_db/indexed_db_callbacks.h (right):

https://codereview.chromium.org/2521903004/diff/1/content/browser/indexed_db/...
content/browser/indexed_db/indexed_db_callbacks.h:40:
scoped_refptr<IndexedDBDispatcherHost> dispatcher_host,
On 2016/11/22 at 21:40:10, cmumford wrote:
> This change looks good, but can you also change the other two constructors to
preserve symmetry?

What other constructors? After landing my last Mojo conversion patch this should
be the only constructor.

[commit-bot: I haz the power, 2016-11-22 22:04:32]

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

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

Dry run: This issue passed the CQ dry run.

[cmumford, 2016-11-28 21:57:28]

lgtm

https://codereview.chromium.org/2521903004/diff/1/content/browser/indexed_db/...
File content/browser/indexed_db/indexed_db_callbacks.h (right):

https://codereview.chromium.org/2521903004/diff/1/content/browser/indexed_db/...
content/browser/indexed_db/indexed_db_callbacks.h:40:
scoped_refptr<IndexedDBDispatcherHost> dispatcher_host,
On 2016/11/22 21:54:53, Reilly Grant wrote:
> On 2016/11/22 at 21:40:10, cmumford wrote:
> > This change looks good, but can you also change the other two constructors
to
> preserve symmetry?
> 
> What other constructors? After landing my last Mojo conversion patch this
should
> be the only constructor.

Nevermind. resolved in offline discussion.

[Reilly Grant (use Gerrit), 2016-11-28 22:55:38]

The CQ bit was checked by reillyg@chromium.org

[commit-bot: I haz the power, 2016-11-28 22:56:31]

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-29 01:48:26]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480373738595350, "parent_rev":
"50a11edd81f8e1b42a5a54aba5ff5026b0b5569b", "commit_rev":
"06c31bd6e9af736bfbe90a55f3dd0c27f2d09cfd"}

[commit-bot: I haz the power, 2016-11-29 01:48:55]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-29 01:50:45]

Description was changed from

==========
Use a scoped_refptr in the IndexedDBCallbacks constructor.

Follow-up to r433964 to change IndexedDBCallbacks to take its
IndexedDBDispatcherHost argument as a scoped_refptr as suggested by
dcheng@.

BUG=None
==========

to

==========
Use a scoped_refptr in the IndexedDBCallbacks constructor.

Follow-up to r433964 to change IndexedDBCallbacks to take its
IndexedDBDispatcherHost argument as a scoped_refptr as suggested by
dcheng@.

BUG=None

Committed: https://crrev.com/25f132ffa3cd92cc25dcc35ea556dabcb6d0895e
Cr-Commit-Position: refs/heads/master@{#434822}
==========

[commit-bot: I haz the power, 2016-11-29 01:50:46]

Patchset 1 (id:??) landed as
https://crrev.com/25f132ffa3cd92cc25dcc35ea556dabcb6d0895e
Cr-Commit-Position: refs/heads/master@{#434822}