Upstream iOS implementation of dom_distiller

components/dom_distiller/ios/distiller_page_factory_ios.h

+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef COMPONENTS_DOM_DISTILLER_IOS_DISTILLER_PAGE_FACTORY_IOS_H_
+#define COMPONENTS_DOM_DISTILLER_IOS_DISTILLER_PAGE_FACTORY_IOS_H_
+
+#include "base/memory/scoped_ptr.h"
+#include "components/dom_distiller/core/distiller_page.h"
+
+namespace web {
+class BrowserState;
+}
+
+namespace dom_distiller {
+
+class DistillerPageIOS;
+
+class DistillerPageFactoryIOS : public DistillerPageFactory {

[bengr, 2015/02/20 20:11:38]

Add a class comment.

[sdefresne, 2015/02/23 11:08:57]

Done.

+ public:
+  DistillerPageFactoryIOS(web::BrowserState* browser_state);
+
+  scoped_ptr<DistillerPage> CreateDistillerPage(
+      const gfx::Size& view_size) const override;

[bengr, 2015/02/20 20:11:37]

Add blank line below.

[sdefresne, 2015/02/23 11:08:57]

This is the list of method overridden from DistillerPageFactory interface, the
common style throughout the Chromium code base seems to put a comment on top "//
Implementation of DistillerPageFactory" and put all the overridden methods
without blank lines between them.

Added the comment.

+  scoped_ptr<DistillerPage> CreateDistillerPageWithHandle(
+      scoped_ptr<SourcePageHandle> handle) const override;
+
+ private:
+  web::BrowserState* browser_state_;
+};
+
+}  // namespace dom_distiller
+
+#endif  // COMPONENTS_DOM_DISTILLER_IOS_DISTILLER_PAGE_FACTORY_IOS_H_