Separate EmbeddedWorker messages into another file to use different msg class

Separate EmbeddedWorker messages into another file to use different msg class

So that we can rely on IPC_MESSAGE_CLASS(msg) check to
determine which messages we want to relay onto another thread.

Mostly just shuffling,
- Separated EmbeddedWorker messages out of service_worker_messages.h into embedded_worker_messages.h
- Added a new message class for that
- Created a new directory content/common/service_worker (and OWNERS file) as now we have multiple header files for SW under common/

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

[kinuko, 2014-01-08 03:57:55]

Tom, (cc: alecf/michaeln)

Could you review this tiny IPC change? It turned out I want this to make my
tests run. (No changes in actual messaging). Thanks,

[Tom Sepez, 2014-01-08 20:09:33]

On 2014/01/08 03:57:55, kinuko wrote:
> Tom, (cc: alecf/michaeln)
> 
> Could you review this tiny IPC change? It turned out I want this to make my
> tests run. (No changes in actual messaging). Thanks,

This breaks them model where there is one IPC_MESSAGE_START per file, which is
previously intact.  Is there a better way of doing this?

[kinuko, 2014-01-08 23:48:20]

We could break this into multiple files, or check the specific message
types in thread dispatching code rather than relying on its msg class.  I'm
worrying that the latter is a bit error prone so am willing to separate the
part into another file... unless we have better alternatives.
2014/01/09 5:09 <tsepez@chromium.org>:

> On 2014/01/08 03:57:55, kinuko wrote:
>
>> Tom, (cc: alecf/michaeln)
>>
>
>  Could you review this tiny IPC change? It turned out I want this to make
>> my
>> tests run. (No changes in actual messaging). Thanks,
>>
>
> This breaks them model where there is one IPC_MESSAGE_START per file,
> which is
> previously intact.  Is there a better way of doing this?
>
> https://codereview.chromium.org/127583002/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[Tom Sepez, 2014-01-09 00:05:42]

Probably cleaner to make a new file, if it's not too much trouble.  LGTM
otherwise.

[kinuko, 2014-01-09 04:48:35]

Added a new file embedded_worker_messages.h out of service_worker_messages.h,
and created a new sub-dir content/common/service_worker to have them. (Resulted
as a slightly bigger change due to code churn)

Avi: Could you take a brief look at content/ changes? The change's mostly for
code shuffling.

[Avi (use Gerrit), 2014-01-09 04:53:15]

Cool. LGTM

[commit-bot: I haz the power, 2014-01-09 10:31:57]

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

[commit-bot: I haz the power, 2014-01-09 12:42:48]

Change committed as 243871