Chromium Code Reviews| Index: cc/test/mock_occlusion_tracker.h |
| diff --git a/cc/test/mock_occlusion_tracker.h b/cc/test/mock_occlusion_tracker.h |
| index d46f5b7923af141cec0e61b8a269387273348180..4624a5627e8e38b20e70300cfb6c42417e1c6d00 100644 |
| --- a/cc/test/mock_occlusion_tracker.h |
| +++ b/cc/test/mock_occlusion_tracker.h |
| @@ -10,7 +10,7 @@ |
| namespace cc { |
| template <typename LayerType> |
| -class MockOcclusionTracker : public OcclusionTracker<LayerType> { |
| +class CC_EXPORT MockOcclusionTracker : public OcclusionTracker<LayerType> { |
| // This class is used for testing only. It fakes just enough information to |
| // calculate unoccluded content rect and unoccluded contributing surface |
| // content rect. It calls the helper function of occlusion tracker to avoid |
| @@ -42,7 +42,7 @@ class MockOcclusionTracker : public OcclusionTracker<LayerType> { |
| .occlusion_from_inside_target = occluded; |
| } |
| - gfx::Rect UnoccludedContentRect( |
| + virtual gfx::Rect UnoccludedContentRect( |
|
enne (OOO)
2014/05/30 21:37:43
style nit: don't inline virtuals
Also, isn't this
weiliangc
2014/05/30 21:46:07
In unittest the OcclusionTracker and LayerImpls ar
enne (OOO)
2014/05/30 22:11:10
Hmm, ok. I'm always saddened by virtual-for-testi
|
| const LayerType* render_target, |
| const gfx::Rect& content_rect, |
| const gfx::Transform& draw_transform) const OVERRIDE { |
| @@ -50,13 +50,16 @@ class MockOcclusionTracker : public OcclusionTracker<LayerType> { |
| render_target, content_rect, draw_transform); |
| } |
| - gfx::Rect UnoccludedContributingSurfaceContentRect( |
| + virtual gfx::Rect UnoccludedContributingSurfaceContentRect( |
| const LayerType* layer, |
| const gfx::Rect& content_rect, |
| const gfx::Transform& draw_transform) const OVERRIDE { |
| return this->UnoccludedContributingSurfaceContentRectHelper( |
| layer, content_rect, draw_transform); |
| } |
| + |
| + private: |
| + DISALLOW_COPY_AND_ASSIGN(MockOcclusionTracker); |
| }; |
| } // namespace cc |