ServiceWorker: propagate provider_id to the browser process in .register + int32/int cleanup

ServiceWorker: propagate provider_id to the browser process in .register

Remotely for event/object rewiring.

This also fixes all int32 used in IPC for in-memory object IDs to int

BUG=349919
TEST=no functional changes yet

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

[kinuko, 2014-04-07 09:29:43]

Separated a cleanup/mechanical part from http://crrev.com/201303004. I'd like to
land this part separately to avoid introducing large code churn in more
important changes.

PTL, mechanical changes only

michaeln@ (or someone in the team): service_worker/*
tsepez@: service_worker_messages.h changes

[Tom Sepez, 2014-04-07 18:33:13]

Why is changing int32 to the less-specific int required?

[kinuko, 2014-04-08 12:14:52]

On 2014/04/07 18:33:13, Tom Sepez wrote:
> Why is changing int32 to the less-specific int required?

We decided to use plain int's for all these internal identifiers that do not
need to store/load to/from disk or send over network (and this should basically
matches our styleguide).

[michaeln, 2014-04-08 16:09:23]

lgtm

> PTL, mechanical changes only
> 
> michaeln@ (or someone in the team): service_worker/*
> tsepez@: service_worker_messages.h changes

[Tom Sepez, 2014-04-08 17:22:51]

Messages LGTM.

[kinuko, 2014-04-10 08:41:35]

The CQ bit was checked by kinuko@chromium.org

[commit-bot: I haz the power, 2014-04-10 08:41:43]

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

[commit-bot: I haz the power, 2014-04-10 13:50:27]

Change committed as 262973

[Jeffrey Yasskin, 2014-04-11 23:21:23]

https://codereview.chromium.org/225403014/diff/60001/content/browser/service_...
File content/browser/service_worker/service_worker_context_core.cc (right):

https://codereview.chromium.org/225403014/diff/60001/content/browser/service_...
content/browser/service_worker/service_worker_context_core.cc:91:
ServiceWorkerProviderHost* provider_host,
Note that ServiceWorkerJobCoordinator::Unregister doesn't currently use the
source_process_id: 
https://code.google.com/p/chromium/codesearch/#chromium/src/content/browser/s...
Could you double-check that it's actually going to use the ProviderHost?