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

Side by Side Diff: sync/internal_api/public/internal_components_factory_impl.h

Issue 11419224: Add missing (and remove superfluous) 'explicit' from constructors. Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase + remove non-straightforward changes Created 7 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 | Annotate | Revision Log
OLDNEW
1 // Copyright 2012 The Chromium Authors. All rights reserved. 1 // Copyright 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 // An InternalComponentsFactory implementation designed for real production / 5 // An InternalComponentsFactory implementation designed for real production /
6 // normal use. 6 // normal use.
7 7
8 #ifndef SYNC_INTERNAL_API_PUBLIC_INTERNAL_COMPONENTS_FACTORY_IMPL_H_ 8 #ifndef SYNC_INTERNAL_API_PUBLIC_INTERNAL_COMPONENTS_FACTORY_IMPL_H_
9 #define SYNC_INTERNAL_API_PUBLIC_INTERNAL_COMPONENTS_FACTORY_IMPL_H_ 9 #define SYNC_INTERNAL_API_PUBLIC_INTERNAL_COMPONENTS_FACTORY_IMPL_H_
10 10
11 #include "sync/base/sync_export.h" 11 #include "sync/base/sync_export.h"
12 #include "sync/internal_api/public/internal_components_factory.h" 12 #include "sync/internal_api/public/internal_components_factory.h"
13 13
14 namespace syncer { 14 namespace syncer {
15 15
16 class SYNC_EXPORT InternalComponentsFactoryImpl 16 class SYNC_EXPORT InternalComponentsFactoryImpl
17 : public InternalComponentsFactory { 17 : public InternalComponentsFactory {
18 public: 18 public:
19 InternalComponentsFactoryImpl(const Switches& switches); 19 explicit InternalComponentsFactoryImpl(const Switches& switches);
20 virtual ~InternalComponentsFactoryImpl(); 20 virtual ~InternalComponentsFactoryImpl();
21 21
22 virtual scoped_ptr<SyncScheduler> BuildScheduler( 22 virtual scoped_ptr<SyncScheduler> BuildScheduler(
23 const std::string& name, 23 const std::string& name,
24 sessions::SyncSessionContext* context) OVERRIDE; 24 sessions::SyncSessionContext* context) OVERRIDE;
25 25
26 virtual scoped_ptr<sessions::SyncSessionContext> BuildContext( 26 virtual scoped_ptr<sessions::SyncSessionContext> BuildContext(
27 ServerConnectionManager* connection_manager, 27 ServerConnectionManager* connection_manager,
28 syncable::Directory* directory, 28 syncable::Directory* directory,
29 const std::vector<ModelSafeWorker*> workers, 29 const std::vector<ModelSafeWorker*> workers,
(...skipping 11 matching lines...) Expand all
41 virtual Switches GetSwitches() const OVERRIDE; 41 virtual Switches GetSwitches() const OVERRIDE;
42 42
43 private: 43 private:
44 const Switches switches_; 44 const Switches switches_;
45 DISALLOW_COPY_AND_ASSIGN(InternalComponentsFactoryImpl); 45 DISALLOW_COPY_AND_ASSIGN(InternalComponentsFactoryImpl);
46 }; 46 };
47 47
48 } // namespace syncer 48 } // namespace syncer
49 49
50 #endif // SYNC_INTERNAL_API_PUBLIC_INTERNAL_COMPONENTS_FACTORY_IMPL_H_ 50 #endif // SYNC_INTERNAL_API_PUBLIC_INTERNAL_COMPONENTS_FACTORY_IMPL_H_
OLDNEW
« no previous file with comments | « skia/ext/SkFontHost_fontconfig.cpp ('k') | sync/internal_api/public/test/test_internal_components_factory.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698