diff options
author | rsleevi@chromium.org <rsleevi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-02-06 09:51:55 +0000 |
---|---|---|
committer | rsleevi@chromium.org <rsleevi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-02-06 09:51:55 +0000 |
commit | 9f93027279b04fab04d10e80d9ba166439ebe685 (patch) | |
tree | af382ddabdd38b7ab916c6dc50b939f92bf608b5 | |
parent | ba91a797de94b0ed95c180e7b9c3e0d8546915e2 (diff) | |
download | chromium_src-9f93027279b04fab04d10e80d9ba166439ebe685.zip chromium_src-9f93027279b04fab04d10e80d9ba166439ebe685.tar.gz chromium_src-9f93027279b04fab04d10e80d9ba166439ebe685.tar.bz2 |
Linux/ChromeOS Chromium style checker cleanup, cc/ edition.
Automated clean up of style checker errors that were missed due to the plugin
not being executed on implementation files.
BUG=115047
Review URL: https://chromiumcodereview.appspot.com/12218028
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180948 0039d316-1c4b-4281-b951-d872f2087c98
-rw-r--r-- | cc/frame_rate_controller_unittest.cc | 4 | ||||
-rw-r--r-- | cc/gl_renderer_pixeltest.cc | 2 | ||||
-rw-r--r-- | cc/gl_renderer_unittest.cc | 2 | ||||
-rw-r--r-- | cc/layer_tree_host_impl_unittest.cc | 8 | ||||
-rw-r--r-- | cc/layer_tree_host_unittest_context.cc | 12 | ||||
-rw-r--r-- | cc/picture_layer_impl_unittest.cc | 4 | ||||
-rw-r--r-- | cc/resource_update_controller_unittest.cc | 2 | ||||
-rw-r--r-- | cc/test/layer_tree_test_common.cc | 2 | ||||
-rw-r--r-- | cc/tree_synchronizer_unittest.cc | 2 |
9 files changed, 20 insertions, 18 deletions
diff --git a/cc/frame_rate_controller_unittest.cc b/cc/frame_rate_controller_unittest.cc index 17168dd..d150248 100644 --- a/cc/frame_rate_controller_unittest.cc +++ b/cc/frame_rate_controller_unittest.cc @@ -17,7 +17,9 @@ public: void reset() { m_vsyncTicked = false; } bool vsyncTicked() const { return m_vsyncTicked; } - virtual void vsyncTick(bool throttled) { m_vsyncTicked = !throttled; } + virtual void vsyncTick(bool throttled) OVERRIDE { + m_vsyncTicked = !throttled; + } protected: bool m_vsyncTicked; diff --git a/cc/gl_renderer_pixeltest.cc b/cc/gl_renderer_pixeltest.cc index 3507d77..74ec0b9 100644 --- a/cc/gl_renderer_pixeltest.cc +++ b/cc/gl_renderer_pixeltest.cc @@ -42,7 +42,7 @@ class FakeRendererClient : public RendererClient { virtual void enforceManagedMemoryPolicy( const ManagedMemoryPolicy&) OVERRIDE {} virtual bool hasImplThread() const OVERRIDE { return false; } - virtual bool shouldClearRootRenderPass() const { return true; } + virtual bool shouldClearRootRenderPass() const OVERRIDE { return true; } virtual CompositorFrameMetadata makeCompositorFrameMetadata() const OVERRIDE { return CompositorFrameMetadata(); } }; diff --git a/cc/gl_renderer_unittest.cc b/cc/gl_renderer_unittest.cc index e9665ab..23c48f7 100644 --- a/cc/gl_renderer_unittest.cc +++ b/cc/gl_renderer_unittest.cc @@ -581,7 +581,7 @@ TEST(GLRendererTest2, activeTextureState) class NoClearRootRenderPassFakeClient : public FakeRendererClient { public: - virtual bool shouldClearRootRenderPass() const { return false; } + virtual bool shouldClearRootRenderPass() const OVERRIDE { return false; } }; class NoClearRootRenderPassMockContext : public FakeWebGraphicsContext3D { diff --git a/cc/layer_tree_host_impl_unittest.cc b/cc/layer_tree_host_impl_unittest.cc index 06a4e96..b3b0cab 100644 --- a/cc/layer_tree_host_impl_unittest.cc +++ b/cc/layer_tree_host_impl_unittest.cc @@ -2405,14 +2405,14 @@ TEST_P(LayerTreeHostImplTest, partialSwap) class PartialSwapContext : public FakeWebGraphicsContext3D { public: - WebKit::WebString getString(WebKit::WGC3Denum name) + virtual WebKit::WebString getString(WebKit::WGC3Denum name) { if (name == GL_EXTENSIONS) return WebKit::WebString("GL_CHROMIUM_post_sub_buffer"); return WebKit::WebString(); } - WebKit::WebString getRequestableExtensionsCHROMIUM() + virtual WebKit::WebString getRequestableExtensionsCHROMIUM() { return WebKit::WebString("GL_CHROMIUM_post_sub_buffer"); } @@ -4017,7 +4017,7 @@ TEST_P(LayerTreeHostImplTest, testRemoveRenderPasses) class LayerTreeHostImplTestWithDelegatingRenderer : public LayerTreeHostImplTest { protected: - virtual scoped_ptr<OutputSurface> createOutputSurface() + virtual scoped_ptr<OutputSurface> createOutputSurface() OVERRIDE { // Creates an output surface with a parent to use a delegating renderer. WebKit::WebGraphicsContext3D::Attributes attrs; @@ -4061,7 +4061,7 @@ public: return make_scoped_ptr(new FakeMaskLayerImpl(treeImpl, id)); } - virtual ResourceProvider::ResourceId contentsResourceId() const { return 0; } + virtual ResourceProvider::ResourceId contentsResourceId() const OVERRIDE { return 0; } private: FakeMaskLayerImpl(LayerTreeImpl* treeImpl, int id) : LayerImpl(treeImpl, id) { } diff --git a/cc/layer_tree_host_unittest_context.cc b/cc/layer_tree_host_unittest_context.cc index 751e7d7..7b84656 100644 --- a/cc/layer_tree_host_unittest_context.cc +++ b/cc/layer_tree_host_unittest_context.cc @@ -300,7 +300,7 @@ class LayerTreeHostContextTestLostContextSucceedsWithContent : LayerTreeHostContextTestLostContextSucceeds::InvalidateAndSetNeedsCommit(); } - virtual void drawLayersOnThread(LayerTreeHostImpl* host_impl) { + virtual void drawLayersOnThread(LayerTreeHostImpl* host_impl) OVERRIDE { FakeContentLayerImpl* content_impl = static_cast<FakeContentLayerImpl*>( host_impl->rootLayer()->children()[0]); // Even though the context was lost, we should have a resource. The @@ -549,7 +549,7 @@ class LayerTreeHostContextTestLostContextWhileUpdatingResources : times_to_lose_on_end_query_(3) { } - virtual scoped_ptr<FakeWebGraphicsContext3D> CreateContext3d() { + virtual scoped_ptr<FakeWebGraphicsContext3D> CreateContext3d() OVERRIDE { scoped_ptr<FakeWebGraphicsContext3D> context = LayerTreeHostContextTest::CreateContext3d(); if (times_to_lose_on_end_query_) { @@ -559,7 +559,7 @@ class LayerTreeHostContextTestLostContextWhileUpdatingResources : return context.Pass(); } - virtual void setupTree() { + virtual void setupTree() OVERRIDE { parent_->setBounds(gfx::Size(num_children_, 1)); for (int i = 0; i < num_children_; i++) { @@ -574,11 +574,11 @@ class LayerTreeHostContextTestLostContextWhileUpdatingResources : LayerTreeHostContextTest::setupTree(); } - virtual void beginTest() { + virtual void beginTest() OVERRIDE { postSetNeedsCommitToMainThread(); } - virtual void commitCompleteOnThread(LayerTreeHostImpl* impl) { + virtual void commitCompleteOnThread(LayerTreeHostImpl* impl) OVERRIDE { endTest(); } @@ -586,7 +586,7 @@ class LayerTreeHostContextTestLostContextWhileUpdatingResources : EXPECT_TRUE(succeeded); } - virtual void afterTest() { + virtual void afterTest() OVERRIDE { EXPECT_EQ(0, times_to_lose_on_end_query_); } diff --git a/cc/picture_layer_impl_unittest.cc b/cc/picture_layer_impl_unittest.cc index 340e60c..c1845ce 100644 --- a/cc/picture_layer_impl_unittest.cc +++ b/cc/picture_layer_impl_unittest.cc @@ -112,7 +112,7 @@ class TestablePicturePileImpl : public PicturePileImpl { } protected: - ~TestablePicturePileImpl() { + virtual ~TestablePicturePileImpl() { } }; @@ -131,7 +131,7 @@ class PictureLayerImplTest : public testing::Test { host_impl_.initializeRenderer(createFakeOutputSurface()); } - ~PictureLayerImplTest() { + virtual ~PictureLayerImplTest() { } void SetupTrees( diff --git a/cc/resource_update_controller_unittest.cc b/cc/resource_update_controller_unittest.cc index 2da1460a..8fb7d18 100644 --- a/cc/resource_update_controller_unittest.cc +++ b/cc/resource_update_controller_unittest.cc @@ -78,7 +78,7 @@ public: { } - ~ResourceUpdateControllerTest() + virtual ~ResourceUpdateControllerTest() { DebugScopedSetImplThreadAndMainThreadBlocked implThreadAndMainThreadBlocked(&m_proxy); diff --git a/cc/test/layer_tree_test_common.cc b/cc/test/layer_tree_test_common.cc index f710f4b..c7528fc 100644 --- a/cc/test/layer_tree_test_common.cc +++ b/cc/test/layer_tree_test_common.cc @@ -121,7 +121,7 @@ public: return layerTreeHost.Pass(); } - virtual scoped_ptr<cc::LayerTreeHostImpl> createLayerTreeHostImpl(cc::LayerTreeHostImplClient* client) + virtual scoped_ptr<cc::LayerTreeHostImpl> createLayerTreeHostImpl(cc::LayerTreeHostImplClient* client) OVERRIDE { return MockLayerTreeHostImpl::create(m_testHooks, settings(), client, proxy()).PassAs<cc::LayerTreeHostImpl>(); } diff --git a/cc/tree_synchronizer_unittest.cc b/cc/tree_synchronizer_unittest.cc index 30c09f6..d4e9def 100644 --- a/cc/tree_synchronizer_unittest.cc +++ b/cc/tree_synchronizer_unittest.cc @@ -91,7 +91,7 @@ private: virtual ~FakeLayerAnimationController() { } - virtual void pushAnimationUpdatesTo(LayerAnimationController* controllerImpl) + virtual void pushAnimationUpdatesTo(LayerAnimationController* controllerImpl) OVERRIDE { LayerAnimationController::pushAnimationUpdatesTo(controllerImpl); m_synchronizedAnimations = true; |