Consistent CopyFromSurface() API, consolidated to RWHV

content/browser/frame_host/navigation_entry_screenshot_manager.h

 // Copyright 2013 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_FRAME_HOST_NAVIGATION_ENTRY_SCREENSHOT_MANAGER_H_
 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATION_ENTRY_SCREENSHOT_MANAGER_H_
 
 #include "base/compiler_specific.h"
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
 #include "base/time/time.h"
 #include "content/common/content_export.h"
 #include "content/public/browser/readback_types.h"
 
 class SkBitmap;
 
 namespace content {
 
 class NavigationControllerImpl;
 class NavigationEntryImpl;
-class RenderViewHost;
 class ScreenshotData;
 
 // NavigationEntryScreenshotManager takes care of taking image-captures for the
 // current navigation entry of a NavigationControllerImpl, and managing these
 // captured images. These image-captures are used for history navigation using
 // overscroll gestures.
 class CONTENT_EXPORT NavigationEntryScreenshotManager {
  public:
   explicit NavigationEntryScreenshotManager(
       NavigationControllerImpl* controller);
   virtual ~NavigationEntryScreenshotManager();
 
   // Takes a screenshot of the last-committed entry of the controller.
   void TakeScreenshot();
 
   // Clears screenshots of all navigation entries.
   void ClearAllScreenshots();
 
  protected:
-  virtual void TakeScreenshotImpl(RenderViewHost* host,
-                                  NavigationEntryImpl* entry);
-
   // Called after a screenshot has been set on an NavigationEntryImpl.
   // Overridden in tests to get notified of when a screenshot is set.
   virtual void OnScreenshotSet(NavigationEntryImpl* entry);
 
   NavigationControllerImpl* owner() { return owner_; }
 
   void SetMinScreenshotIntervalMS(int interval_ms);
 
   // The callback invoked when taking the screenshot of the page is complete.
   // This sets the screenshot on the navigation entry.
@@ skipping 29 matching lines @@
   // the callback to make sure that the screenshot/encoding completion callback
   // does not trigger on a destroyed NavigationEntryScreenshotManager.
   base::WeakPtrFactory<NavigationEntryScreenshotManager> screenshot_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(NavigationEntryScreenshotManager);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_FRAME_HOST_NAVIGATION_ENTRY_SCREENSHOT_MANAGER_H_