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

Side by Side Diff: content/test/test_content_client_initializer.h

Issue 10191010: Re-implement the screensaver to use WebView instead of ExtensionDialogHost. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 8 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 | Annotate | Revision Log
OLDNEW
(Empty)
1 // Copyright (c) 2011 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 #ifndef CONTENT_TEST_TEST_CONTENT_CLIENT_INITIALIZER_
6 #define CONTENT_TEST_TEST_CONTENT_CLIENT_INITIALIZER_
7 #pragma once
8
9 #include "base/basictypes.h"
10 #include "base/memory/scoped_ptr.h"
11
12 class NotificationServiceImpl;
13
14 namespace content {
jam 2012/05/01 18:55:33 put this whole class in the content namespace
rkc 2012/05/02 00:51:03 Done.
15 class ContentClient;
16 class ContentBrowserClient;
17 }
18
19 class TestContentClientInitializer {
jam 2012/05/01 18:55:33 nit: please add a comment about what this class is
rkc 2012/05/02 00:51:03 Done.
20 public:
21 TestContentClientInitializer();
22 ~TestContentClientInitializer();
23 private:
jam 2012/05/01 18:55:33 nit: blank line above private
rkc 2012/05/02 00:51:03 Done.
24 scoped_ptr<NotificationServiceImpl> notification_service_;
25 scoped_ptr<content::ContentClient> content_client_;
26 scoped_ptr<content::ContentBrowserClient> content_browser_client_;
27
28 DISALLOW_COPY_AND_ASSIGN(TestContentClientInitializer);
29 };
30
31 #endif // CONTENT_TEST_TEST_CONTENT_CLIENT_INITIALIZER_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698