Separate SwipeRefreshHandler and ContentViewCore

content/browser/renderer_host/render_view_host_delegate.h

 // Copyright (c) 2012 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_RENDERER_HOST_RENDER_VIEW_HOST_DELEGATE_H_
 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_VIEW_HOST_DELEGATE_H_
 
 #include <stdint.h>
 
 #include <string>
 
 #include "base/callback.h"
 #include "base/process/kill.h"
 #include "base/strings/string16.h"
 #include "content/browser/dom_storage/session_storage_namespace_impl.h"
+#include "content/browser/renderer_host/render_view_host_delegate_view.h"

[boliu, 2016/12/03 01:05:28]

forward declaration should still be good enough?

[rlanday, 2016/12/05 19:53:56]

Oh right, this should go in
content/browser/renderer_host/render_widget_host_view_android.cc

 #include "content/common/content_export.h"
 #include "net/base/load_states.h"
 #include "third_party/WebKit/public/web/WebPopupType.h"
 #include "ui/base/window_open_disposition.h"
 
 class GURL;
 
 namespace IPC {
 class Message;
 }
 
 namespace gfx {
 class Rect;
 class Size;
 }
 
 namespace content {
 
 class BrowserContext;
 class FrameTree;
 class PageState;
 class RenderViewHost;
-class RenderViewHostDelegateView;
 class SessionStorageNamespace;
 class SiteInstance;
 class WebContents;
 struct RendererPreferences;
 
 namespace mojom {
 class CreateNewWindowParams;
 }
 
 //
@@ skipping 181 matching lines @@
   // Whether download UI should be hidden.
   virtual bool HideDownloadUI() const;
 
  protected:
   virtual ~RenderViewHostDelegate() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_RENDER_VIEW_HOST_DELEGATE_H_