Move SynchronousCompositorFactoryImpl into separate h/cc files

Move SynchronousCompositorFactoryImpl into separate h/cc files

This is a long overdue clean up to make future changes in this area
saner. Only shuffling code and these cosmetic changes:

Removed unused SynchronousCompositorImpl* argument from
SynchronousCompositorFactoryImpl::CompositorInitializedHardwareDraw
and SynchronousCompositorFactoryImpl::CompositorReleasedHardwareDraw.

Add virtual SynchronousCompositorFactory destructor.

BUG=
NOTRY=true

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

[boliu, 2014-01-06 22:23:46]

avi - stamp content_browser.gypi
tedchoc - rest of the android files

[Avi (use Gerrit), 2014-01-06 22:54:29]

lgtm

gypi stamp

[Ted C, 2014-01-07 00:31:08]

lgtm

https://codereview.chromium.org/102963010/diff/50001/content/browser/android/...
File content/browser/android/in_process/synchronous_compositor_factory_impl.cc
(right):

https://codereview.chromium.org/102963010/diff/50001/content/browser/android/...
content/browser/android/in_process/synchronous_compositor_factory_impl.cc:52: :
wrapped_gl_context_for_main_thread_(NULL), num_hardware_compositors_(0) {
does num_hardware_compositors_ need to go on the second line?

[boliu, 2014-01-07 00:39:32]

https://codereview.chromium.org/102963010/diff/50001/content/browser/android/...
File content/browser/android/in_process/synchronous_compositor_factory_impl.cc
(right):

https://codereview.chromium.org/102963010/diff/50001/content/browser/android/...
content/browser/android/in_process/synchronous_compositor_factory_impl.cc:52: :
wrapped_gl_context_for_main_thread_(NULL), num_hardware_compositors_(0) {
On 2014/01/07 00:31:09, Ted C wrote:
> does num_hardware_compositors_ need to go on the second line?

Not sure, grey area in the style guide, and clang-format did it..

Anyway, broke it into another line.

[commit-bot: I haz the power, 2014-01-07 00:41:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/boliu@chromium.org/102963010/130001

[commit-bot: I haz the power, 2014-01-07 12:30:35]

Retried try job too often on win_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2014-01-07 17:06:33]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/boliu@chromium.org/102963010/130001

[commit-bot: I haz the power, 2014-01-07 17:07:47]

Change committed as 243314