Simplify WebServiceWorker{Registration}Proxy interfaces.

Simplify WebServiceWorker{Registration}Proxy interfaces.

Turn WebServiceWorkerRegistrationProxy and WebServiceWorkerProxy into
pure abstract interfaces, letting the embedder call the "proxy"
interfaces on the ServiceWorkerRegistration and ServiceWorker Blink
objects, respectively. That is, no longer keep private (and bare)
pointers on these platform proxy objects; unnecessary and runs up
against overall platform layering constraints.

Like before, the ownership of these proxy objects resides with Blink.

R=nhiroki,haraken,mkwst
BUG=410257
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=184587

[sof, 2014-10-28 13:17:05]

sigbjornf@opera.com changed reviewers:
+ falken@chromium.org, haraken@chromium.org, nhiroki@chromium.org

[sof, 2014-10-28 13:17:06]

Please take a look.

Perhaps I'm overlooking some aspect wrt keeping these m_private bare pointers,
but they do appear unnecessary.

[nhiroki, 2014-10-29 03:40:57]

LGTM, thank you for working on this!

[haraken, 2014-10-29 04:19:55]

LGTM

[sof, 2014-10-29 06:49:37]

sigbjornf@opera.com changed reviewers:
+ jochen@chromium.org, tkent@chromium.org

[sof, 2014-10-29 06:49:38]

Thanks for the reviews.

tkent, jochen: could you look at the public/platform/ changes when you have some
time? tia

https://codereview.chromium.org/682003003/diff/20001/public/platform/WebServi...
File public/platform/WebServiceWorkerProxy.h (right):

https://codereview.chromium.org/682003003/diff/20001/public/platform/WebServi...
public/platform/WebServiceWorkerProxy.h:10: // FIXME: move to public/web/ ?
Removed this addition. Not a correct place for it, this header file belongs in
public/platform/.

[Mike West, 2014-10-29 07:59:43]

mkwst@chromium.org changed reviewers:
+ mkwst@chromium.org

[Mike West, 2014-10-29 07:59:44]

public/platform LGTM.

[sof, 2014-10-29 08:03:08]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2014-10-29 08:04:12]

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

[commit-bot: I haz the power, 2014-10-29 11:19:09]

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

[commit-bot: I haz the power, 2014-10-29 11:19:11]

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/34130)

[sof, 2014-10-29 11:32:35]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2014-10-29 11:33:29]

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

[commit-bot: I haz the power, 2014-10-29 13:20:51]

Committed patchset #3 (id:40001) as 184587