[ServiceWorker] Add WebDataSource argument to isControlledByServiceWorker() [3/3 blink].

[ServiceWorker] Add WebDataSource argument to isControlledByServiceWorker() [3/3 blink]

In current implementation, while the new page is loading the request from the existing previous page may refer to the provisional dataSource in isControlledByServiceWorker().
But this is wrong.
We should pass WebDataSource to this method to avoid the ambiguity.

1/3 blink: https://codereview.chromium.org/634813002
2/3 chromium: https://codereview.chromium.org/629953003
3/3 blink: https://codereview.chromium.org/635693003

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

[horo, 2014-10-07 02:22:25]

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

[horo, 2014-10-07 02:22:26]

michaeln@
Could you please review this?

[Mike West, 2014-10-07 15:45:58]

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

[Mike West, 2014-10-07 15:45:58]

And, assuming the first two land, this obviously LGTM as well.

As an aside, I really appreciate the links you put in the CL descriptions. It
makes reviewing the whole set quite a bit simpler. Thank you. I've started
pointing other people to your reviews as an example of how to document 3-sided
patches. :)

[michaeln, 2014-10-07 19:32:21]

lgtm, but you'll need a public api owner

[horo, 2014-10-07 22:50:55]

horo@chromium.org changed reviewers:
+ tkent@chromium.org

[horo, 2014-10-07 22:50:55]

tkent@
Could you please review this cl?
Thank you!

[tkent, 2014-10-08 03:09:49]

lgtm

[horo, 2014-10-08 22:31:26]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2014-10-08 22:31:59]

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

[commit-bot: I haz the power, 2014-10-09 03:24:30]

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

[commit-bot: I haz the power, 2014-10-09 03:24:31]

Exceeded time limit waiting for builds to trigger.

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

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2014-10-09 03:26:37]

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

[commit-bot: I haz the power, 2014-10-09 06:31:24]

Committed patchset #3 (id:40001) as 183436