Worker: Factor out common functions into ThreadedObjectProxyBase

Worker: Factor out common functions into ThreadedObjectProxyBase

Several functions in InProcessWorkerObjectProxy and ThreadedWorkletObjectProxy
are common. This CL introduces ThreadedMessagingProxyBase class, factors out the
functions into the base class and makes the both proxies inherit it.

[Class relationships after this CL]

On MainThread                   / On WorkerThread
ThreadedMessagingProxyBase      / ThreadedObjectProxyBase
- InProcessWorkerMessagingProxy / - InProcessWorkerObjectProxy
- ThreadedWorkletMessagingProxy / - ThreadedWorkletObjectProxy

BUG=667357
Committed: https://crrev.com/9083392315f8b2ba599185c6b63af718ac4c6efb
Cr-Commit-Position: refs/heads/master@{#437503}

[nhiroki, 2016-12-09 05:33:17]

Description was changed from

==========
Worker: Factor out common operations into ThreadedObjectProxyBase

Several operations in InProcessWorkerObjectProxy and ThreadedWorkletObjectProxy
are common. This CL factors them out into ThreadedObjectProxyBase and makes the
both proxies inherit it.

BUG=667357
==========

to

==========
Worker: Factor out common operations into ThreadedObjectProxyBase

Several operations in InProcessWorkerObjectProxy and ThreadedWorkletObjectProxy
are common. This CL factors them out into ThreadedObjectProxyBase and makes the
both proxies inherit it.

<Class relationships after this CL>

On MainThread                   / On WorkerThread
ThreadedMessagingProxyBase      / ThreadedObjectProxyBase
- InProcessWorkerMessagingProxy / - InProcessWorkerObjectProxy
- ThreadedWorkletMessagingProxy / - ThreadedWorkletObjectProxy

BUG=667357
==========

[nhiroki, 2016-12-09 05:48:07]

The CQ bit was checked by nhiroki@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-09 05:48:19]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[nhiroki, 2016-12-09 05:48:31]

Description was changed from

==========
Worker: Factor out common operations into ThreadedObjectProxyBase

Several operations in InProcessWorkerObjectProxy and ThreadedWorkletObjectProxy
are common. This CL factors them out into ThreadedObjectProxyBase and makes the
both proxies inherit it.

<Class relationships after this CL>

On MainThread                   / On WorkerThread
ThreadedMessagingProxyBase      / ThreadedObjectProxyBase
- InProcessWorkerMessagingProxy / - InProcessWorkerObjectProxy
- ThreadedWorkletMessagingProxy / - ThreadedWorkletObjectProxy

BUG=667357
==========

to

==========
Worker: Factor out common functions into ThreadedObjectProxyBase

Several functions in InProcessWorkerObjectProxy and ThreadedWorkletObjectProxy
are common. This CL factors them out into ThreadedObjectProxyBase and makes the
both proxies inherit it.

[Class relationships after this CL]

On MainThread                   / On WorkerThread
ThreadedMessagingProxyBase      / ThreadedObjectProxyBase
- InProcessWorkerMessagingProxy / - InProcessWorkerObjectProxy
- ThreadedWorkletMessagingProxy / - ThreadedWorkletObjectProxy

BUG=667357
==========

[nhiroki, 2016-12-09 05:48:48]

Description was changed from

==========
Worker: Factor out common functions into ThreadedObjectProxyBase

Several functions in InProcessWorkerObjectProxy and ThreadedWorkletObjectProxy
are common. This CL factors them out into ThreadedObjectProxyBase and makes the
both proxies inherit it.

[Class relationships after this CL]

On MainThread                   / On WorkerThread
ThreadedMessagingProxyBase      / ThreadedObjectProxyBase
- InProcessWorkerMessagingProxy / - InProcessWorkerObjectProxy
- ThreadedWorkletMessagingProxy / - ThreadedWorkletObjectProxy

BUG=667357
==========

to

==========
Worker: Factor out common functions into ThreadedObjectProxyBase

Several functions in InProcessWorkerObjectProxy and ThreadedWorkletObjectProxy
are common. This CL factors them out into ThreadedObjectProxyBase and makes the
both proxies inherit it.

[Class relationships after this CL]

On MainThread                   / On WorkerThread
ThreadedMessagingProxyBase      / ThreadedObjectProxyBase
- InProcessWorkerMessagingProxy / - InProcessWorkerObjectProxy
- ThreadedWorkletMessagingProxy / - ThreadedWorkletObjectProxy

BUG=667357
==========

[nhiroki, 2016-12-09 05:48:55]

Patchset #1 (id:1) has been deleted

[nhiroki, 2016-12-09 05:52:58]

The CQ bit was checked by nhiroki@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-09 05:53:11]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[nhiroki, 2016-12-09 05:56:05]

Description was changed from

==========
Worker: Factor out common functions into ThreadedObjectProxyBase

Several functions in InProcessWorkerObjectProxy and ThreadedWorkletObjectProxy
are common. This CL factors them out into ThreadedObjectProxyBase and makes the
both proxies inherit it.

[Class relationships after this CL]

On MainThread                   / On WorkerThread
ThreadedMessagingProxyBase      / ThreadedObjectProxyBase
- InProcessWorkerMessagingProxy / - InProcessWorkerObjectProxy
- ThreadedWorkletMessagingProxy / - ThreadedWorkletObjectProxy

BUG=667357
==========

to

==========
Worker: Factor out common functions into ThreadedObjectProxyBase

Several functions in InProcessWorkerObjectProxy and ThreadedWorkletObjectProxy
are common. This CL introduces ThreadedMessagingProxyBase class, factors out the
functions into the base class and makes the both proxies inherit it.

[Class relationships after this CL]

On MainThread                   / On WorkerThread
ThreadedMessagingProxyBase      / ThreadedObjectProxyBase
- InProcessWorkerMessagingProxy / - InProcessWorkerObjectProxy
- ThreadedWorkletMessagingProxy / - ThreadedWorkletObjectProxy

BUG=667357
==========

[nhiroki, 2016-12-09 05:56:15]

Patchset #1 (id:20001) has been deleted

[nhiroki, 2016-12-09 06:31:18]

nhiroki@chromium.org changed reviewers:
+ falken@chromium.org

[nhiroki, 2016-12-09 06:31:19]

PTAL, thanks!

[falken, 2016-12-09 06:45:10]

lgtm

[commit-bot: I haz the power, 2016-12-09 07:25:24]

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

[commit-bot: I haz the power, 2016-12-09 07:25:25]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[nhiroki, 2016-12-09 07:39:42]

The CQ bit was checked by nhiroki@chromium.org

[nhiroki, 2016-12-09 07:39:42]

The patchset sent to the CQ was uploaded after l-g-t-m from falken@chromium.org
Link to the patchset: https://codereview.chromium.org/2566623002/#ps60001
(title: "fix layout tests")

[commit-bot: I haz the power, 2016-12-09 07:39:55]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-09 09:29:55]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1481269182077560,
"parent_rev": "09706c57f41b94fc4647844f4d9e99797445d18d", "commit_rev":
"676dc20a124203fc59f876a84fec92838e67f369"}

[commit-bot: I haz the power, 2016-12-09 09:30:21]

Committed patchset #2 (id:60001)

[commit-bot: I haz the power, 2016-12-09 09:32:10]

Description was changed from

==========
Worker: Factor out common functions into ThreadedObjectProxyBase

Several functions in InProcessWorkerObjectProxy and ThreadedWorkletObjectProxy
are common. This CL introduces ThreadedMessagingProxyBase class, factors out the
functions into the base class and makes the both proxies inherit it.

[Class relationships after this CL]

On MainThread                   / On WorkerThread
ThreadedMessagingProxyBase      / ThreadedObjectProxyBase
- InProcessWorkerMessagingProxy / - InProcessWorkerObjectProxy
- ThreadedWorkletMessagingProxy / - ThreadedWorkletObjectProxy

BUG=667357
==========

to

==========
Worker: Factor out common functions into ThreadedObjectProxyBase

Several functions in InProcessWorkerObjectProxy and ThreadedWorkletObjectProxy
are common. This CL introduces ThreadedMessagingProxyBase class, factors out the
functions into the base class and makes the both proxies inherit it.

[Class relationships after this CL]

On MainThread                   / On WorkerThread
ThreadedMessagingProxyBase      / ThreadedObjectProxyBase
- InProcessWorkerMessagingProxy / - InProcessWorkerObjectProxy
- ThreadedWorkletMessagingProxy / - ThreadedWorkletObjectProxy

BUG=667357

Committed: https://crrev.com/9083392315f8b2ba599185c6b63af718ac4c6efb
Cr-Commit-Position: refs/heads/master@{#437503}
==========

[commit-bot: I haz the power, 2016-12-09 09:32:11]

Patchset 2 (id:??) landed as
https://crrev.com/9083392315f8b2ba599185c6b63af718ac4c6efb
Cr-Commit-Position: refs/heads/master@{#437503}