[CacheStorage] Use appopriate type parameters for WebCallbacks (2/3).

[CacheStorage] Use appopriate type parameters for WebCallbacks (2/3).

This series of CLs fixes WebCallback parameter types used in CacheStorage.

 - WebServiceWorkerResponse* (ownership not transferred)
  => const WebServiceWorkerResponse&
 - WebVector<WebServiceWorkerResponse>* (ownership not transferred)
  => const WebVector<WebServiceWorkerResponse>&
 - WebVector<WebServiceWorkerRequest>* (ownership not transferred)
  => const WebVector<WebServiceWorkerRequest>&
 - WebServiceWorkerCache* (ownership transferred)
  => WebPassOwnPtr<WebServiceWorkerCache>
 - WebVector<WebString>* (ownership not transferred)
  => const WebVector<WebString>&
 - WebServiceWorkerCacheError* (ownership transferred)
  => WebServiceWorkerCacheError

1/3: https://codereview.chromium.org/1284173004/
2/3: https://codereview.chromium.org/1284283003/
3/3: https://codereview.chromium.org/1296723005/

BUG=493531
Committed: https://crrev.com/ab0af9a5611ac80eed35771b5f9bf169d63dbe4e
Cr-Commit-Position: refs/heads/master@{#344437}

[yhirano, 2015-08-19 10:27:14]

yhirano@chromium.org changed reviewers:
+ nhiroki@chromium.org

[nhiroki, 2015-08-19 11:29:43]

lgtm

https://codereview.chromium.org/1284283003/diff/1/content/renderer/cache_stor...
File content/renderer/cache_storage/cache_storage_dispatcher.cc (right):

https://codereview.chromium.org/1284283003/diff/1/content/renderer/cache_stor...
content/renderer/cache_storage/cache_storage_dispatcher.cc:132:
blink::WebServiceWorkerCacheErrorNotFound));
onError(blink::WebServiceWorkerCacheErrorNotFound) ?

[yhirano, 2015-08-19 11:39:41]

https://codereview.chromium.org/1284283003/diff/1/content/renderer/cache_stor...
File content/renderer/cache_storage/cache_storage_dispatcher.cc (right):

https://codereview.chromium.org/1284283003/diff/1/content/renderer/cache_stor...
content/renderer/cache_storage/cache_storage_dispatcher.cc:132:
blink::WebServiceWorkerCacheErrorNotFound));
On 2015/08/19 11:29:42, nhiroki wrote:
> onError(blink::WebServiceWorkerCacheErrorNotFound) ?

Done.

[yhirano, 2015-08-20 02:15:28]

The CQ bit was checked by yhirano@chromium.org

[yhirano, 2015-08-20 02:15:28]

The patchset sent to the CQ was uploaded after l-g-t-m from nhiroki@chromium.org
Link to the patchset: https://codereview.chromium.org/1284283003/#ps20001
(title: " ")

[commit-bot: I haz the power, 2015-08-20 02:15:53]

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

[commit-bot: I haz the power, 2015-08-20 03:06:59]

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

[commit-bot: I haz the power, 2015-08-20 03:07:00]

Try jobs failed on following builders:
  chromeos_daisy_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)

[yhirano, 2015-08-20 03:42:43]

The CQ bit was checked by yhirano@chromium.org

[commit-bot: I haz the power, 2015-08-20 03:42:56]

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

[commit-bot: I haz the power, 2015-08-20 04:17:02]

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

[commit-bot: I haz the power, 2015-08-20 04:17:03]

Try jobs failed on following builders:
  chromeos_daisy_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)

[nhiroki, 2015-08-20 04:36:54]

On 2015/08/20 04:17:03, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   chromeos_daisy_chromium_compile_only_ng on tryserver.chromium.linux
> (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)

fyi: I'm now fixing build failures on chromeos bots
https://codereview.chromium.org/1301033004/

[yhirano, 2015-08-20 05:18:41]

On 2015/08/20 04:36:54, nhiroki wrote:
> On 2015/08/20 04:17:03, commit-bot: I haz the power wrote:
> > Try jobs failed on following builders:
> >   chromeos_daisy_chromium_compile_only_ng on tryserver.chromium.linux
> > (JOB_FAILED,
> >
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)
> 
> fyi: I'm now fixing build failures on chromeos bots
> https://codereview.chromium.org/1301033004/

Thanks!

[nhiroki, 2015-08-20 05:48:12]

The CQ bit was checked by nhiroki@chromium.org

[commit-bot: I haz the power, 2015-08-20 05:48:17]

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

[commit-bot: I haz the power, 2015-08-20 06:40:03]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-08-20 06:40:35]

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