Simplify SharedWorkerRepository code (Chromium side)

Simplify SharedWorkerRepository code (Chromium side)

- Move scattered worker repository code into one class
  (content::SharedWorkerRepository)
- Use per-page client which implements RenderViewObserver
- Implement RuntimeFeatures for shared workers
- No more WebKit:: static method to initialize repository
- Remove dead/unused code

Blink side patch (landed): https://codereview.chromium.org/40143003/

BUG=none

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=231268

[kinuko, 2013-10-25 01:34:28]

Jochen, maybe you can review this one?

[jochen (gone - plz use gerrit), 2013-10-25 11:33:29]

lgtm

https://codereview.chromium.org/40303002/diff/1/content/renderer/shared_worke...
File content/renderer/shared_worker_repository.cc (right):

https://codereview.chromium.org/40303002/diff/1/content/renderer/shared_worke...
content/renderer/shared_worker_repository.cc:38: if (url_mismatch) {
no { }

https://codereview.chromium.org/40303002/diff/1/content/renderer/shared_worke...
File content/renderer/shared_worker_repository.h (right):

https://codereview.chromium.org/40303002/diff/1/content/renderer/shared_worke...
content/renderer/shared_worker_repository.h:21:
SharedWorkerRepository(RenderViewImpl* render_view);
explicit

[kinuko, 2013-10-25 14:44:57]

Thx, updated.

https://codereview.chromium.org/40303002/diff/1/content/renderer/shared_worke...
File content/renderer/shared_worker_repository.cc (right):

https://codereview.chromium.org/40303002/diff/1/content/renderer/shared_worke...
content/renderer/shared_worker_repository.cc:38: if (url_mismatch) {
On 2013/10/25 11:33:30, jochen wrote:
> no { }

Done.

https://codereview.chromium.org/40303002/diff/1/content/renderer/shared_worke...
File content/renderer/shared_worker_repository.h (right):

https://codereview.chromium.org/40303002/diff/1/content/renderer/shared_worke...
content/renderer/shared_worker_repository.h:21:
SharedWorkerRepository(RenderViewImpl* render_view);
On 2013/10/25 11:33:30, jochen wrote:
> explicit

Done.

[kinuko, 2013-10-25 14:46:28]

jam: can you do owner review for:

content/child/runtime_features.cc
content/content_renderer.gypi

[jam, 2013-10-25 15:14:07]

lgtm

[commit-bot: I haz the power, 2013-10-27 08:56:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kinuko@chromium.org/40303002/230001

[commit-bot: I haz the power, 2013-10-27 19:34:44]

Change committed as 231268