Move webfallbackthemeengine_impl.* to content/child/

Move webfallbackthemeengine_impl.* to content/child/

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

[jamesr, 2014-02-11 20:58:14]

The fallback theme engine is a strange thing.  It appears to only exist to have
a different rendering of zoomed controls when running layout tests. Anyway, this
move seems good although again if the test WebKitPlatformSupportImpl was based
on the content/renderer/ one then this could go into c/renderer.  I don't think
that we need to be able to render anything in non-renderer child ptypes (like
worker, etc).

https://codereview.chromium.org/160023003/diff/1/content/child/webkitplatform...
File content/child/webkitplatformsupport_child_impl.h (right):

https://codereview.chromium.org/160023003/diff/1/content/child/webkitplatform...
content/child/webkitplatformsupport_child_impl.h:68:
content::WebFallbackThemeEngineImpl fallback_theme_engine_;
you're already in namespace content

[pilgrim_google, 2014-02-12 21:40:41]

The CQ bit was checked by pilgrim@chromium.org

[commit-bot: I haz the power, 2014-02-12 21:45:47]

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

[commit-bot: I haz the power, 2014-02-12 21:45:48]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a lowly provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[darin (slow to review), 2014-02-12 23:58:43]

LGTM

https://codereview.chromium.org/160023003/diff/100001/content/child/webkitpla...
File content/child/webkitplatformsupport_child_impl.h (right):

https://codereview.chromium.org/160023003/diff/100001/content/child/webkitpla...
content/child/webkitplatformsupport_child_impl.h:68: WebFallbackThemeEngineImpl
fallback_theme_engine_;
I agree with jamesr. The code for the fallback theme engine should just live in
content/renderer and be hooked up in RendererWebKitPlatformSupportImpl instead
of here. That is okay to punt to a follow-up CL if you prefer, but I would
probably have included it in this CL.

[pilgrim_google, 2014-02-13 00:50:07]

The CQ bit was checked by pilgrim@chromium.org

[commit-bot: I haz the power, 2014-02-13 00:50:46]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pilgrim@chromium.org/160023003/100001

[commit-bot: I haz the power, 2014-02-13 03:57:29]

Change committed as 250930