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

Side by Side Diff: blimp/engine/testing/app/blimp_content_browser_client_for_test.cc

Issue 2572563006: [Blimp] Refactor Blimp test engine with embedded test server and URL rewriting (Closed)
Patch Set: John comment Created 3 years, 11 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
OLDNEW
(Empty)
1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "blimp/engine/testing/app/blimp_browser_testing_main_parts.h"
6 #include "blimp/engine/testing/app/blimp_content_browser_client_for_test.h"
7
8 namespace blimp {
9 namespace engine {
10
11 BlimpContentBrowserClientForTest::BlimpContentBrowserClientForTest() {}
12
13 content::BrowserMainParts*
14 BlimpContentBrowserClientForTest::CreateBrowserMainParts(
15 const content::MainFunctionParams& parameters) {
16 blimp_browser_main_parts_ = new BlimpBrowserTestingMainParts(parameters);
17 BlimpContentBrowserClient::SetBrowserMainPartsForTesting(
Kevin M 2017/01/09 20:17:17 Is it necessary to qualify this call with the base
shenghuazhang 2017/01/10 23:20:49 No need to specify it with base class name. Will r
18 blimp_browser_main_parts_);
19 // BrowserMainLoop takes ownership of the returned BrowserMainParts.
20 return blimp_browser_main_parts_;
21 }
22
23 } // namespace engine
24 } // namespace blimp
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698