Keep deinlining stuff.

webkit/glue/plugins/webplugin_print_delegate.h

 // Copyright (c) 2010 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 WEBKIT_GLUE_PLUGINS_WEBPLUGIN_PRINT_DELEGATE_H_
 #define WEBKIT_GLUE_PLUGINS_WEBPLUGIN_PRINT_DELEGATE_H_
 
 #include "base/basictypes.h"
 #include "third_party/npapi/bindings/npapi_extensions.h"
+#include "third_party/WebKit/WebKit/chromium/public/WebCanvas.h"

[Lei Zhang, 2010/12/15 00:51:35]

You can't forward declare WebKit::WebCanvas?

[Elliot Glaysher, 2010/12/15 17:34:41]

Nope. It's a bunch of ifdefed typedefs.

 
 namespace gfx {
 class Rect;
 }
 
 namespace webkit_glue {
 
 // Interface for the NPAPI print extension. This class implements "NOP"
 // versions of all these functions so it can be used seamlessly by the
 // "regular" plugin delegate while being overridden by the "pepper" one.
 class WebPluginPrintDelegate {
  public:
   // If a plugin supports print extensions, then it gets to participate fully
   // in the browser's print workflow by specifying the number of pages to be
   // printed and providing a print output for specified pages.
-  virtual bool PrintSupportsPrintExtension() {
-    return false;
-  }
+  virtual bool PrintSupportsPrintExtension();
 
   // Note: printable_area is in points (a point is 1/72 of an inch).
-  virtual int PrintBegin(const gfx::Rect& printable_area, int printer_dpi) {
-    return 0;
-  }
+  virtual int PrintBegin(const gfx::Rect& printable_area, int printer_dpi);
 
-  virtual bool PrintPage(int page_number, WebKit::WebCanvas* canvas) {
-    return false;
-  }
+  virtual bool PrintPage(int page_number, WebKit::WebCanvas* canvas);
 
-  virtual void PrintEnd() {
-  }
+  virtual void PrintEnd();
 
  protected:
   WebPluginPrintDelegate() {}
   virtual ~WebPluginPrintDelegate() {}
 };
 
 }  // namespace webkit_glue
 
 #endif  // WEBKIT_GLUE_PLUGINS_WEBPLUGIN_PRINT_DELEGATE_H_