Allow CopyFromBackingStore to fallback to copying from the renderer side if the accelerated surface…

chrome/browser/extensions/api/tabs/tabs_api.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 CHROME_BROWSER_EXTENSIONS_API_TABS_TABS_API_H_
 #define CHROME_BROWSER_EXTENSIONS_API_TABS_TABS_API_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 156 matching lines @@
   virtual ~TabsDetectLanguageFunction() {}
   virtual bool RunImpl() OVERRIDE;
 
   virtual void Observe(int type,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) OVERRIDE;
   void GotLanguage(const std::string& language);
   content::NotificationRegistrar registrar_;
   DECLARE_EXTENSION_FUNCTION("tabs.detectLanguage", TABS_DETECTLANGUAGE)
 };
-class TabsCaptureVisibleTabFunction : public AsyncExtensionFunction,
-                                  public content::NotificationObserver {
+class TabsCaptureVisibleTabFunction : public AsyncExtensionFunction {
  public:
   static void RegisterUserPrefs(PrefRegistrySyncable* registry);
 
  protected:
   enum ImageFormat {
     FORMAT_JPEG,
     FORMAT_PNG
   };
 
   // The default quality setting used when encoding jpegs.
   static const int kDefaultQuality;
 
   virtual ~TabsCaptureVisibleTabFunction() {}
   virtual bool RunImpl() OVERRIDE;
   virtual bool GetTabToCapture(content::WebContents** web_contents);
-  virtual void Observe(int type,
-                       const content::NotificationSource& source,
-                       const content::NotificationDetails& details) OVERRIDE;
   void SendResultFromBitmap(const SkBitmap& screen_capture);
 
  private:
   void CopyFromBackingStoreComplete(bool succeeded,
                                     const SkBitmap& bitmap);
-
-  content::NotificationRegistrar registrar_;
+  void GetSnapshotFromRendererComplete(bool succeeded,

[sky, 2013/03/22 19:39:22]

Add descriptions for these methods.

+                                       const SkBitmap& bitmap);
+  void SendInternalError();
 
   // The format (JPEG vs PNG) of the resulting image.  Set in RunImpl().
   ImageFormat image_format_;
 
   // Quality setting to use when encoding jpegs.  Set in RunImpl().
   int image_quality_;
 
   DECLARE_EXTENSION_FUNCTION("tabs.captureVisibleTab", TABS_CAPTUREVISIBLETAB)
 };
 
@@ skipping 63 matching lines @@
 };
 
 class TabsInsertCSSFunction : public ExecuteCodeInTabFunction {
  private:
   virtual ~TabsInsertCSSFunction() {}
 
   DECLARE_EXTENSION_FUNCTION("tabs.insertCSS", TABS_INSERTCSS)
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_TABS_TABS_API_H_