Chromium Code Reviews| OLD | NEW |
|---|---|
| (Empty) | |
| 1 // Copyright 2015 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 CHROME_BROWSER_MEDIA_ROUTER_MEDIA_ROUTER_MOJO_IMPL_FACTORY_H_ | |
| 6 #define CHROME_BROWSER_MEDIA_ROUTER_MEDIA_ROUTER_MOJO_IMPL_FACTORY_H_ | |
| 7 | |
| 8 #include "base/memory/singleton.h" | |
| 9 #include "components/keyed_service/content/browser_context_keyed_service_factory .h" | |
| 10 | |
| 11 namespace content { | |
| 12 class BrowserContext; | |
| 13 } // namespace content | |
|
Wez
2015/05/20 23:37:50
nit: No need for this since the namespace is teeny
imcheng (use chromium acct)
2015/05/21 19:28:52
Done.
| |
| 14 | |
| 15 namespace media_router { | |
| 16 | |
| 17 class MediaRouterMojoImpl; | |
| 18 | |
| 19 // A factory that lazily returns a MediaRouterMojoImpl object for a given | |
| 20 // BrowserContext, creating one lazily if needed. | |
|
Wez
2015/05/20 23:37:50
nit: Can you clean up this comment, plz? :)
imcheng (use chromium acct)
2015/05/21 19:28:53
Done.
| |
| 21 class MediaRouterMojoImplFactory : public BrowserContextKeyedServiceFactory { | |
| 22 public: | |
| 23 static MediaRouterMojoImpl* GetApiForBrowserContext( | |
| 24 content::BrowserContext* context); | |
| 25 static MediaRouterMojoImplFactory* GetInstance(); | |
|
Wez
2015/05/20 23:37:50
If there are no non-static methods on MediaRouterM
imcheng (use chromium acct)
2015/05/21 19:28:52
Removed this function and inlined it in GetApiForB
| |
| 26 | |
| 27 private: | |
| 28 friend struct DefaultSingletonTraits<MediaRouterMojoImplFactory>; | |
| 29 | |
| 30 MediaRouterMojoImplFactory(); | |
| 31 ~MediaRouterMojoImplFactory() override; | |
| 32 | |
| 33 // BrowserContextKeyedBaseFactory | |
|
Wez
2015/05/20 23:37:50
... interface.
imcheng (use chromium acct)
2015/05/21 19:28:52
Done.
| |
| 34 KeyedService* BuildServiceInstanceFor( | |
| 35 content::BrowserContext* context) const override; | |
| 36 content::BrowserContext* GetBrowserContextToUse( | |
| 37 content::BrowserContext* context) const override; | |
| 38 | |
| 39 DISALLOW_COPY_AND_ASSIGN(MediaRouterMojoImplFactory); | |
| 40 }; | |
| 41 | |
| 42 } // namespace media_router | |
| 43 | |
| 44 #endif // CHROME_BROWSER_MEDIA_ROUTER_MEDIA_ROUTER_MOJO_IMPL_FACTORY_H_ | |
| OLD | NEW |