diff options
author | jamesr <jamesr@chromium.org> | 2015-03-16 12:11:05 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-03-16 19:12:30 +0000 |
commit | 9b8fda3b7b24371afc2c2ed05d1c6a547cc6a786 (patch) | |
tree | 2507d6ec9f9990840eef087b5df7b3a873279fae /cc/cc.gyp | |
parent | 323eb85bb6dabe3fe323fbbfdb3887a7520cc27d (diff) | |
download | chromium_src-9b8fda3b7b24371afc2c2ed05d1c6a547cc6a786.zip chromium_src-9b8fda3b7b24371afc2c2ed05d1c6a547cc6a786.tar.gz chromium_src-9b8fda3b7b24371afc2c2ed05d1c6a547cc6a786.tar.bz2 |
Define cc::RendererCapabilities in its own header, fix up includes
cc::RendererCapabilities was defined in cc/trees/layer_tree_host.h
which had a few subtle implications. First, cc/output/renderer.h needed
to #include layer_tree_host even though cc/output/ logically should not
depend on cc/trees/. Second, since layer_tree_host.h includes pretty
much the entire world, everything that was #including renderer.h was
implicitly pulling in many other headers that it may have used. In the
spirit of IWYU these files should either include or forward declare
types they use.
This puts RendererCapabilities in cc/output/renderer_capabilities and
fixes up includes in many other files.
R=danakj@chromium.org
Review URL: https://codereview.chromium.org/1007593002
Cr-Commit-Position: refs/heads/master@{#320767}
Diffstat (limited to 'cc/cc.gyp')
-rw-r--r-- | cc/cc.gyp | 2 |
1 files changed, 2 insertions, 0 deletions
@@ -311,6 +311,8 @@ 'output/render_surface_filters.h', 'output/renderer.cc', 'output/renderer.h', + 'output/renderer_capabilities.cc', + 'output/renderer_capabilities.h', 'output/renderer_settings.cc', 'output/renderer_settings.h', 'output/shader.cc', |