[BackgroundSync] Use appopriate type parameters for WebCallbacks (4/4)

[BackgroundSync] Use appopriate type parameters for WebCallbacks (4/4)

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

 - WebSyncRegistration* (ownership transferred)
  => PassOwnPtr<WebServiceWorkerResponse>
 - bool* (ownership transferred)
  => bool
 - WebVector<WebSyncRegstration*>* (ownership transferred)
  => const WebVector<WebSyncRegistration*>&
 - WebPermissionStatus* (ownership transferred)
  => WebPermissionStatus
 - WebSyncError* (ownership transferred)
  => const WebSyncError&

1/4: https://codereview.chromium.org/1309143002/
2/4: https://codereview.chromium.org/1311053002/
3/4: https://codereview.chromium.org/1308273002/
4/4: https://codereview.chromium.org/1308283002/

BUG=493531
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=201330

[yhirano, 2015-08-24 08:47:21]

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

[jkarlin, 2015-08-26 13:21:41]

https://codereview.chromium.org/1308283002/diff/1/public/platform/modules/bac...
File public/platform/modules/background_sync/WebSyncProvider.h (right):

https://codereview.chromium.org/1308283002/diff/1/public/platform/modules/bac...
public/platform/modules/background_sync/WebSyncProvider.h:54:
callbacks->onError(WebSyncError(WebSyncError::ErrorTypeAbort, "Function not
implemented."));
Shouldn't this fix be in the second CL?

[yhirano, 2015-08-27 02:23:05]

https://codereview.chromium.org/1308283002/diff/1/public/platform/modules/bac...
File public/platform/modules/background_sync/WebSyncProvider.h (right):

https://codereview.chromium.org/1308283002/diff/1/public/platform/modules/bac...
public/platform/modules/background_sync/WebSyncProvider.h:54:
callbacks->onError(WebSyncError(WebSyncError::ErrorTypeAbort, "Function not
implemented."));
On 2015/08/26 13:21:41, jkarlin wrote:
> Shouldn't this fix be in the second CL?

Done.

[jkarlin, 2015-08-27 11:25:27]

lgtm

[yhirano, 2015-08-27 18:50:28]

The CQ bit was checked by yhirano@chromium.org

[commit-bot: I haz the power, 2015-08-27 18:50:39]

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

[commit-bot: I haz the power, 2015-08-27 19:31:31]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=201330