Chromium Code Reviews| OLD | NEW |
|---|---|
| (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 #ifndef CHROME_BROWSER_ANDROID_OFFLINE_PAGES_PRERENDERING_OFFLINER_FACTORY_H_ | |
| 6 #define CHROME_BROWSER_ANDROID_OFFLINE_PAGES_PRERENDERING_OFFLINER_FACTORY_H_ | |
| 7 | |
| 8 #include "base/macros.h" | |
| 9 #include "components/offline_pages/background/offliner_factory.h" | |
| 10 | |
| 11 namespace content { | |
| 12 class BrowserContext; | |
| 13 } | |
|
fgorski
2016/05/04 05:20:57
} // namespace content
Pete Williamson
2016/05/04 21:23:21
Done.
| |
| 14 | |
| 15 namespace offline_pages { | |
| 16 | |
| 17 class OfflinerPolicy; | |
| 18 class Offliner; | |
| 19 class PrerenderingOffliner; | |
| 20 | |
| 21 // A factory to create one unique PrerenderingOffliner | |
|
fgorski
2016/05/04 05:20:57
Can you also explain why the prerendering offliner
Pete Williamson
2016/05/04 21:23:21
Comments updated.
| |
| 22 class PrerenderingOfflinerFactory : public OfflinerFactory { | |
| 23 public: | |
| 24 explicit PrerenderingOfflinerFactory(content::BrowserContext* context); | |
| 25 ~PrerenderingOfflinerFactory() override; | |
| 26 | |
| 27 Offliner* GetOffliner(const OfflinerPolicy* policy) override; | |
| 28 | |
| 29 private: | |
| 30 // The offliner is owned by the factory, since it may be resued. | |
| 31 PrerenderingOffliner* offliner_; | |
| 32 | |
| 33 // This is an unowned pointer, expected to stay valid for the lifetime of the | |
| 34 // factory. | |
| 35 content::BrowserContext* context_; | |
| 36 | |
| 37 DISALLOW_COPY_AND_ASSIGN(PrerenderingOfflinerFactory); | |
| 38 }; | |
| 39 | |
| 40 } // namespace offline_pages | |
| 41 | |
| 42 #endif // CHROME_BROWSER_ANDROID_OFFLINE_PAGES_PRERENDERING_OFFLINER_FACTORY_H_ | |
| OLD | NEW |