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

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

Issue 2496233003: Destroy the old RenderWidgetHostView when swapping out a main frame. (Closed)
Patch Set: rebase Created 4 years 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
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_TEST_RENDERER_HOST_H_ 5 #ifndef CONTENT_PUBLIC_TEST_TEST_RENDERER_HOST_H_
6 #define CONTENT_PUBLIC_TEST_TEST_RENDERER_HOST_H_ 6 #define CONTENT_PUBLIC_TEST_TEST_RENDERER_HOST_H_
7 7
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <memory> 10 #include <memory>
(...skipping 157 matching lines...) Expand 10 before | Expand all | Expand 10 after
168 // RenderViewHostTester and RenderFrameHostTester respectively. 168 // RenderViewHostTester and RenderFrameHostTester respectively.
169 class RenderViewHostTestEnabler { 169 class RenderViewHostTestEnabler {
170 public: 170 public:
171 RenderViewHostTestEnabler(); 171 RenderViewHostTestEnabler();
172 ~RenderViewHostTestEnabler(); 172 ~RenderViewHostTestEnabler();
173 173
174 private: 174 private:
175 DISALLOW_COPY_AND_ASSIGN(RenderViewHostTestEnabler); 175 DISALLOW_COPY_AND_ASSIGN(RenderViewHostTestEnabler);
176 friend class RenderViewHostTestHarness; 176 friend class RenderViewHostTestHarness;
177 177
178 #if defined(OS_ANDROID)
179 std::unique_ptr<MockGpuChannelEstablishFactory> gpu_channel_factory_;
Charlie Reis 2016/11/23 07:25:22 Similarly, what made this change necessary?
lfg 2016/11/29 17:41:22 Same as the other change, this is necessary to all
180 #endif
178 std::unique_ptr<MockRenderProcessHostFactory> rph_factory_; 181 std::unique_ptr<MockRenderProcessHostFactory> rph_factory_;
179 std::unique_ptr<TestRenderViewHostFactory> rvh_factory_; 182 std::unique_ptr<TestRenderViewHostFactory> rvh_factory_;
180 std::unique_ptr<TestRenderFrameHostFactory> rfh_factory_; 183 std::unique_ptr<TestRenderFrameHostFactory> rfh_factory_;
181 }; 184 };
182 185
183 // RenderViewHostTestHarness --------------------------------------------------- 186 // RenderViewHostTestHarness ---------------------------------------------------
184 class RenderViewHostTestHarness : public testing::Test { 187 class RenderViewHostTestHarness : public testing::Test {
185 public: 188 public:
186 RenderViewHostTestHarness(); 189 RenderViewHostTestHarness();
187 ~RenderViewHostTestHarness() override; 190 ~RenderViewHostTestHarness() override;
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
269 272
270 std::unique_ptr<BrowserContext> browser_context_; 273 std::unique_ptr<BrowserContext> browser_context_;
271 274
272 std::unique_ptr<WebContents> contents_; 275 std::unique_ptr<WebContents> contents_;
273 #if defined(OS_WIN) 276 #if defined(OS_WIN)
274 std::unique_ptr<ui::ScopedOleInitializer> ole_initializer_; 277 std::unique_ptr<ui::ScopedOleInitializer> ole_initializer_;
275 #endif 278 #endif
276 #if defined(USE_AURA) 279 #if defined(USE_AURA)
277 std::unique_ptr<aura::test::AuraTestHelper> aura_test_helper_; 280 std::unique_ptr<aura::test::AuraTestHelper> aura_test_helper_;
278 #endif 281 #endif
279 #if defined(OS_ANDROID)
280 std::unique_ptr<MockGpuChannelEstablishFactory> gpu_channel_factory_;
281 #endif
282 RenderViewHostTestEnabler rvh_test_enabler_; 282 RenderViewHostTestEnabler rvh_test_enabler_;
283 283
284 DISALLOW_COPY_AND_ASSIGN(RenderViewHostTestHarness); 284 DISALLOW_COPY_AND_ASSIGN(RenderViewHostTestHarness);
285 }; 285 };
286 286
287 } // namespace content 287 } // namespace content
288 288
289 #endif // CONTENT_PUBLIC_TEST_TEST_RENDERER_HOST_H_ 289 #endif // CONTENT_PUBLIC_TEST_TEST_RENDERER_HOST_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698