Revert of Stop passing const-reference of SWRegistration in all GetRegistrationsCallbacks.

Revert of Stop passing const-reference of SWRegistration in all GetRegistrationsCallbacks. (patchset #4 id:80001 of https://codereview.chromium.org/2181553003/ )

Reason for revert:
Speculative revert to see if this caused crbug.com/631190. App banners use methods touched by this CL to check if a service worker is registered; this is the only relevant-looking CL in the blamelist for the app banner data fetcher tests starting to fail on the Win7 SP1 bot.

This CL will be relanded if re-enabling the disabled app banner data fetcher tests still causes failures on https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%281%29

Original issue's description:
> Stop passing const-reference of SWRegistration in all GetRegistrationsCallbacks.
>
> BUG=616502
>
> Committed: https://crrev.com/36986071183bdb219186aa9f63f9ee4b162d91b9
> Cr-Commit-Position: refs/heads/master@{#407487}

TBR=nhiroki@chromium.org,peter@chromium.org,dgozman@chromium.org,mmenke@chromium.org,jkarlin@chromium.org,horo@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=616502
Committed: https://crrev.com/0ffd7bda4112522805100efa554bdb7621de860b
Cr-Commit-Position: refs/heads/master@{#407715}

[dominickn, 2016-07-26 05:35:47]

The CQ bit was checked by dominickn@chromium.org

[dominickn, 2016-07-26 05:35:48]

Created Revert of Stop passing const-reference of SWRegistration in all
GetRegistrationsCallbacks.

[commit-bot: I haz the power, 2016-07-26 05:36: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-07-26 05:37:16]

Description was changed from

==========
Revert of Stop passing const-reference of SWRegistration in all
GetRegistrationsCallbacks. (patchset #4 id:80001 of
https://codereview.chromium.org/2181553003/ )

Reason for revert:
Speculative revert to see if this caused crbug.com/631190. App banners use
methods touched by this CL to check if a service worker is registered; this is
the only relevant-looking CL in the blamelist for the app banner data fetcher
tests starting to fail on the Win7 SP1 bot.

This CL will be relanded if re-enabling the disabled app banner data fetcher
tests still causes failures on
https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%2...

Original issue's description:
> Stop passing const-reference of SWRegistration in all
GetRegistrationsCallbacks.
> 
> BUG=616502
> 
> Committed: https://crrev.com/36986071183bdb219186aa9f63f9ee4b162d91b9
> Cr-Commit-Position: refs/heads/master@{#407487}

TBR=nhiroki@chromium.org,peter@chromium.org,dgozman@chromium.org,mmenke@chrom...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=616502
==========

to

==========
Revert of Stop passing const-reference of SWRegistration in all
GetRegistrationsCallbacks. (patchset #4 id:80001 of
https://codereview.chromium.org/2181553003/ )

Reason for revert:
Speculative revert to see if this caused crbug.com/631190. App banners use
methods touched by this CL to check if a service worker is registered; this is
the only relevant-looking CL in the blamelist for the app banner data fetcher
tests starting to fail on the Win7 SP1 bot.

This CL will be relanded if re-enabling the disabled app banner data fetcher
tests still causes failures on
https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%2...

Original issue's description:
> Stop passing const-reference of SWRegistration in all
GetRegistrationsCallbacks.
> 
> BUG=616502
> 
> Committed: https://crrev.com/36986071183bdb219186aa9f63f9ee4b162d91b9
> Cr-Commit-Position: refs/heads/master@{#407487}

TBR=nhiroki@chromium.org,peter@chromium.org,dgozman@chromium.org,mmenke@chrom...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=616502
==========

[commit-bot: I haz the power, 2016-07-26 05:37:17]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-26 05:39:36]

Description was changed from

==========
Revert of Stop passing const-reference of SWRegistration in all
GetRegistrationsCallbacks. (patchset #4 id:80001 of
https://codereview.chromium.org/2181553003/ )

Reason for revert:
Speculative revert to see if this caused crbug.com/631190. App banners use
methods touched by this CL to check if a service worker is registered; this is
the only relevant-looking CL in the blamelist for the app banner data fetcher
tests starting to fail on the Win7 SP1 bot.

This CL will be relanded if re-enabling the disabled app banner data fetcher
tests still causes failures on
https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%2...

Original issue's description:
> Stop passing const-reference of SWRegistration in all
GetRegistrationsCallbacks.
> 
> BUG=616502
> 
> Committed: https://crrev.com/36986071183bdb219186aa9f63f9ee4b162d91b9
> Cr-Commit-Position: refs/heads/master@{#407487}

TBR=nhiroki@chromium.org,peter@chromium.org,dgozman@chromium.org,mmenke@chrom...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=616502
==========

to

==========
Revert of Stop passing const-reference of SWRegistration in all
GetRegistrationsCallbacks. (patchset #4 id:80001 of
https://codereview.chromium.org/2181553003/ )

Reason for revert:
Speculative revert to see if this caused crbug.com/631190. App banners use
methods touched by this CL to check if a service worker is registered; this is
the only relevant-looking CL in the blamelist for the app banner data fetcher
tests starting to fail on the Win7 SP1 bot.

This CL will be relanded if re-enabling the disabled app banner data fetcher
tests still causes failures on
https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%2...

Original issue's description:
> Stop passing const-reference of SWRegistration in all
GetRegistrationsCallbacks.
>
> BUG=616502
>
> Committed: https://crrev.com/36986071183bdb219186aa9f63f9ee4b162d91b9
> Cr-Commit-Position: refs/heads/master@{#407487}

TBR=nhiroki@chromium.org,peter@chromium.org,dgozman@chromium.org,mmenke@chrom...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=616502

Committed: https://crrev.com/0ffd7bda4112522805100efa554bdb7621de860b
Cr-Commit-Position: refs/heads/master@{#407715}
==========

[commit-bot: I haz the power, 2016-07-26 05:39:37]

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