[ServiceWorker] Cleanup to remove old focus() code kept to not break Chromium.

[ServiceWorker] Cleanup to remove old focus() code kept to not break Chromium.

It is a three sided CL:
Part 1: https://codereview.chromium.org/897493002
Part 2: https://codereview.chromium.org/894853007
Part 3: <this>

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

[mlamouri (slow - plz ping), 2015-02-02 18:00:24]

mlamouri@chromium.org changed reviewers:
+ dominicc@chromium.org

[mlamouri (slow - plz ping), 2015-02-02 18:02:11]

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

[mlamouri (slow - plz ping), 2015-02-02 18:02:11]

+mkwst@ for non modules/ changes.

[Mike West, 2015-02-02 18:11:48]

LGTM.

[dominicc (has gone to gerrit), 2015-02-02 18:25:36]

LGTM too; one comment inline at your discretion.

https://codereview.chromium.org/868233006/diff/1/public/web/WebServiceWorkerC...
File public/web/WebServiceWorkerContextClient.h (left):

https://codereview.chromium.org/868233006/diff/1/public/web/WebServiceWorkerC...
public/web/WebServiceWorkerContextClient.h:178: // Ownership of the passed
callbacks is transferred to the callee, callee
Why not keep the comment about ownership?

[mlamouri (slow - plz ping), 2015-02-06 14:05:05]

The CQ bit was checked by mlamouri@chromium.org

[commit-bot: I haz the power, 2015-02-06 14:05:36]

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

[mlamouri (slow - plz ping), 2015-02-06 14:05:47]

https://codereview.chromium.org/868233006/diff/1/public/web/WebServiceWorkerC...
File public/web/WebServiceWorkerContextClient.h (left):

https://codereview.chromium.org/868233006/diff/1/public/web/WebServiceWorkerC...
public/web/WebServiceWorkerContextClient.h:178: // Ownership of the passed
callbacks is transferred to the callee, callee
On 2015/02/02 at 18:25:36, dominicc wrote:
> Why not keep the comment about ownership?

The comment is still, it's just a bit above. The diff make it look like I
removed it though.

[commit-bot: I haz the power, 2015-02-06 18:55:32]

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

[commit-bot: I haz the power, 2015-02-06 18:55:33]

Try jobs failed on following builders:
  mac_blink_rel on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/41746)

[mlamouri (slow - plz ping), 2015-02-07 13:51:21]

The CQ bit was checked by mlamouri@chromium.org

[commit-bot: I haz the power, 2015-02-07 13:51:43]

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

[commit-bot: I haz the power, 2015-02-07 15:13:38]

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

[commit-bot: I haz the power, 2015-02-07 15:13:39]

Try jobs failed on following builders:
  mac_blink_rel on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/41891)

[mlamouri (slow - plz ping), 2015-02-07 15:16:11]

The CQ bit was checked by mlamouri@chromium.org

[commit-bot: I haz the power, 2015-02-07 15:16:30]

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

[commit-bot: I haz the power, 2015-02-07 16:25:34]

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