Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(207)

Side by Side Diff: content/public/test/render_view_test.h

Issue 1118383002: Shut down RenderViewTest's scheduler before Blink is. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: const'ify getters Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | content/public/test/render_view_test.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CONTENT_PUBLIC_TEST_RENDER_VIEW_TEST_H_ 5 #ifndef CONTENT_PUBLIC_TEST_RENDER_VIEW_TEST_H_
6 #define CONTENT_PUBLIC_TEST_RENDER_VIEW_TEST_H_ 6 #define CONTENT_PUBLIC_TEST_RENDER_VIEW_TEST_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/command_line.h" 10 #include "base/command_line.h"
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
46 class RenderView; 46 class RenderView;
47 47
48 class RenderViewTest : public testing::Test { 48 class RenderViewTest : public testing::Test {
49 public: 49 public:
50 // A special BlinkPlatformImpl class for getting rid off the dependency to the 50 // A special BlinkPlatformImpl class for getting rid off the dependency to the
51 // sandbox, which is not available in RenderViewTest. 51 // sandbox, which is not available in RenderViewTest.
52 class RendererBlinkPlatformImplNoSandbox { 52 class RendererBlinkPlatformImplNoSandbox {
53 public: 53 public:
54 RendererBlinkPlatformImplNoSandbox(); 54 RendererBlinkPlatformImplNoSandbox();
55 ~RendererBlinkPlatformImplNoSandbox(); 55 ~RendererBlinkPlatformImplNoSandbox();
56 blink::Platform* Get(); 56 blink::Platform* Get() const;
57 scheduler::RendererScheduler* Scheduler() const;
57 58
58 private: 59 private:
59 scoped_ptr<scheduler::RendererScheduler> renderer_scheduler_; 60 scoped_ptr<scheduler::RendererScheduler> renderer_scheduler_;
60 scoped_ptr<RendererBlinkPlatformImplNoSandboxImpl> blink_platform_impl_; 61 scoped_ptr<RendererBlinkPlatformImplNoSandboxImpl> blink_platform_impl_;
61 }; 62 };
62 63
63 RenderViewTest(); 64 RenderViewTest();
64 ~RenderViewTest() override; 65 ~RenderViewTest() override;
65 66
66 protected: 67 protected:
(...skipping 117 matching lines...) Expand 10 before | Expand all | Expand 10 after
184 scoped_ptr<base::mac::ScopedNSAutoreleasePool> autorelease_pool_; 185 scoped_ptr<base::mac::ScopedNSAutoreleasePool> autorelease_pool_;
185 #endif 186 #endif
186 187
187 private: 188 private:
188 void GoToOffset(int offset, const PageState& state); 189 void GoToOffset(int offset, const PageState& state);
189 }; 190 };
190 191
191 } // namespace content 192 } // namespace content
192 193
193 #endif // CONTENT_PUBLIC_TEST_RENDER_VIEW_TEST_H_ 194 #endif // CONTENT_PUBLIC_TEST_RENDER_VIEW_TEST_H_
OLDNEW
« no previous file with comments | « no previous file | content/public/test/render_view_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698