Make a context struct for ServiceWorkerCache::Put callbacks.

Make a context struct for ServiceWorkerCache::Put callbacks.

The Put callback functions took an absurd number of paramters.  This
CL simplifies by putting them in a context struct.  This is needed by
a downstream CL that needs to add another member to the struct.

Upstream of: https://codereview.chromium.org/608593003

BUG=392621
Committed: https://crrev.com/b022bc195c2fc9b587eab2bd92cc31fb4ec771cb
Cr-Commit-Position: refs/heads/master@{#297164}

[jkarlin, 2014-09-25 19:17:39]

jkarlin@chromium.org changed reviewers:
+ horo@chromium.org, michaeln@chromium.org

[jkarlin, 2014-09-25 19:17:40]

michaeln@chromium.org: Please review changes in all

horo@chromium.org: Please review changes in all

Thanks!

[horo, 2014-09-26 02:15:02]

https://codereview.chromium.org/606843002/diff/1/content/browser/service_work...
File content/browser/service_worker/service_worker_cache.cc (right):

https://codereview.chromium.org/606843002/diff/1/content/browser/service_work...
content/browser/service_worker/service_worker_cache.cc:189: disk_cache::Entry*
cache_entry;
Why don't you use ScopedEntryPtr?

[jkarlin, 2014-09-26 11:15:02]

https://codereview.chromium.org/606843002/diff/1/content/browser/service_work...
File content/browser/service_worker/service_worker_cache.cc (right):

https://codereview.chromium.org/606843002/diff/1/content/browser/service_work...
content/browser/service_worker/service_worker_cache.cc:189: disk_cache::Entry*
cache_entry;
On 2014/09/26 02:15:02, horo wrote:
> Why don't you use ScopedEntryPtr?

Because the asynchronous disk_cache API wants an Entry** for CreateEntry.  I've
added a comment.

[horo, 2014-09-29 03:13:25]

lgtm

https://codereview.chromium.org/606843002/diff/1/content/browser/service_work...
File content/browser/service_worker/service_worker_cache.cc (right):

https://codereview.chromium.org/606843002/diff/1/content/browser/service_work...
content/browser/service_worker/service_worker_cache.cc:189: disk_cache::Entry*
cache_entry;
On 2014/09/26 11:15:02, jkarlin wrote:
> On 2014/09/26 02:15:02, horo wrote:
> > Why don't you use ScopedEntryPtr?
> 
> Because the asynchronous disk_cache API wants an Entry** for CreateEntry. 
I've
> added a comment. 

Ah, I got it.

[jkarlin, 2014-09-29 12:52:20]

The CQ bit was checked by jkarlin@chromium.org

[commit-bot: I haz the power, 2014-09-29 12:53:13]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/606843002/20001

[commit-bot: I haz the power, 2014-09-29 13:22:01]

Committed patchset #2 (id:20001) as 3d8ba41cd4c816b77fcc70a3d6ec2c0809ce2965

[commit-bot: I haz the power, 2014-09-29 13:23:08]

Patchset 2 (id:??) landed as
https://crrev.com/b022bc195c2fc9b587eab2bd92cc31fb4ec771cb
Cr-Commit-Position: refs/heads/master@{#297164}