Expose distiller functions to JavaScript

components/dom_distiller/content/renderer/distiller_page_notifier_service_impl.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_CONTENT_RENDERER_DISTILLER_PAGE_NOTIFIER_SERVICE_IMPL_H_
+#define COMPONENTS_DOM_DISTILLER_CONTENT_RENDERER_DISTILLER_PAGE_NOTIFIER_SERVICE_IMPL_H_
+
+#include "components/dom_distiller/content/common/distiller_page_notifier_service.mojom.h"
+#include "components/dom_distiller/content/renderer/distiller_js_render_frame_observer.h"
+#include "components/dom_distiller/content/renderer/distiller_native_javascript.h"
+
+namespace dom_distiller {
+
+class DistillerJsRenderFrameObserver;
+
+// DistillerPageNotifierService is responsible for listening to the browser for
+// messages about if a page is a distiller page. No message is received if the
+// page is not a distiller page. This service should be removed from the
+// registry once the page is done loading.
+class DistillerPageNotifierServiceImpl
+    : public mojo::InterfaceImpl<DistillerPageNotifierService> {
+ public:
+  DistillerPageNotifierServiceImpl(DistillerJsRenderFrameObserver* observer);

[nyquist, 2015/08/11 08:11:55]

single-argument constructors should be explicit.

[mdjones, 2015/08/11 16:02:05]

Done.

+  void Notify() override;

[nyquist, 2015/08/11 08:11:55]

Add a comment that his method is the implementation of the interface..

[mdjones, 2015/08/11 16:02:05]

Done.

+  ~DistillerPageNotifierServiceImpl() override;

[nyquist, 2015/08/11 08:11:55]

Goes together with the constructor.

[mdjones, 2015/08/11 16:02:05]

Done.

+
+ private:
+  DistillerJsRenderFrameObserver* distiller_js_observer_;
+};
+
+}  // namespace dom_distiller
+
+#endif  // COMPONENTS_DOM_DISTILLER_CONTENT_RENDERER_DISTILLER_PAGE_NOTIFIER_SERVICE_IMPL_H_