Remove usage of HostZoomMap from c/b/loader via ReadyToCommitNavigation

content/browser/host_zoom_map_host.h

+// Copyright 2016 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 CONTENT_BROWSER_HOST_ZOOM_MAP_HOST_H_
+#define CONTENT_BROWSER_HOST_ZOOM_MAP_HOST_H_
+
+#include "content/common/content_export.h"
+#include "content/common/host_zoom.mojom.h"
+#include "content/public/browser/web_contents_observer.h"
+
+namespace content {
+
+class RenderFrameHostImpl;
+
+class CONTENT_EXPORT HostZoomMapHost : private WebContentsObserver {

[nasko, 2016/09/28 01:32:04]

I find the Host suffix a bit confusing, as this object is just an observer.

[scottmg, 2016/09/28 02:44:53]

Yeah. :( Sender? Observer?

+ public:
+  HostZoomMapHost(WebContents* web_contents,
+                  RenderFrameHostImpl* render_frame_host_impl);

[nasko, 2016/09/28 01:32:04]

nit: No need for _impl, we skip that on a lot of param names inside content.

[scottmg, 2016/09/28 02:44:53]

Done.

+
+ private:
+  // WebContentsObserver implementation:
+  void ReadyToCommitNavigation(NavigationHandle* navigation_handle) override;
+
+  RenderFrameHostImpl* render_frame_host_impl_;
+  mojom::HostZoomAssociatedPtr host_zoom_;
+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_HOST_ZOOM_MAP_HOST_H_