ServiceWorker: Stop sending the SetVersionAttributes message from SWProviderHost

ServiceWorker: Stop sending the SetVersionAttributes message from SWProviderHost

SWProviderHost had sent the SetVersionAttributes message to set version
attribtues of SWContainer (eg. navigator.serviceWorker.installing), but
these attributes were moved into SWRegistration and the message is now
sent from SWRegistrationHandle.


BUG=404982
TEST=content_unittests --gtest_filter=ServiceWorker*
TEST=run_webkit_tests.py --debug http/tests/serviceworker

Committed: https://crrev.com/f481929280d7362d77e6fb336ccea03bbeee42bd
Cr-Commit-Position: refs/heads/master@{#292110}

[nhiroki, 2014-08-22 07:26:02]

Ptal, thanks!

[falken, 2014-08-22 09:04:36]

lgtm

https://codereview.chromium.org/496983002/diff/20001/content/browser/service_...
File content/browser/service_worker/service_worker_provider_host.h (right):

https://codereview.chromium.org/496983002/diff/20001/content/browser/service_...
content/browser/service_worker/service_worker_provider_host.h:126: // versions
and removes from one of the previous versions.
I think "and removes it from the previous versions" ?

"one of" seems wrong; the function removes from all of the previous versions.

[nhiroki, 2014-08-22 10:35:18]

Thanks!

https://codereview.chromium.org/496983002/diff/20001/content/browser/service_...
File content/browser/service_worker/service_worker_provider_host.h (right):

https://codereview.chromium.org/496983002/diff/20001/content/browser/service_...
content/browser/service_worker/service_worker_provider_host.h:126: // versions
and removes from one of the previous versions.
On 2014/08/22 09:04:36, falken wrote:
> I think "and removes it from the previous versions" ?
> 
> "one of" seems wrong; the function removes from all of the previous versions.

Done.

[michaeln, 2014-08-25 21:42:51]

lgtm2

[nhiroki, 2014-08-27 05:46:00]

The CQ bit was checked by nhiroki@chromium.org

[commit-bot: I haz the power, 2014-08-27 05:46:11]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/nhiroki@chromium.org/496983002/60001

[commit-bot: I haz the power, 2014-08-27 06:58:02]

Committed patchset #3 (60001) as d94a7a95235f234c1b2f4fa6920969d561c69447

[commit-bot: I haz the power, 2014-09-10 02:50:14]

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