Adding functionality to print PDF embedded in the html page.

chrome/renderer/print_web_view_helper.h

 // Copyright (c) 2011 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_RENDERER_PRINT_WEB_VIEW_HELPER_H_
 #define CHROME_RENDERER_PRINT_WEB_VIEW_HELPER_H_
 #pragma once
 
 #include <vector>
 
@@ skipping 72 matching lines @@
 // of the document and creating a new WebView with the contents.
 class PrintWebViewHelper
     : public content::RenderViewObserver,
       public content::RenderViewObserverTracker<PrintWebViewHelper>,
       public WebKit::WebViewClient,
       public WebKit::WebFrameClient {
  public:
   explicit PrintWebViewHelper(content::RenderView* render_view);
   virtual ~PrintWebViewHelper();
 
+  // Make this function a part of the public interface, since it is called
+  // directly from PPAPI for plugin printing.

[jam, 2012/01/12 22:43:01]

this comment is redundant since it's describing something that's obvious. the
comment for the method's definition should explain what the function does,
although in this particular case, the name is pretty self documenting.

see
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Implem...

+  void PrintNode(const WebKit::WebNode& node);
+
  protected:
   // WebKit::WebViewClient override:
   virtual void didStopLoading();
 
  private:
   friend class PrintWebViewHelperTestBase;
   FRIEND_TEST_ALL_PREFIXES(PrintWebViewHelperTest,
                            BlockScriptInitiatedPrinting);
   FRIEND_TEST_ALL_PREFIXES(PrintWebViewHelperTest, OnPrintPages);
 
@@ skipping 352 matching lines @@
 
     State state_;
   };
 
   PrintPreviewContext print_preview_context_;
 
   DISALLOW_COPY_AND_ASSIGN(PrintWebViewHelper);
 };
 
 #endif  // CHROME_RENDERER_PRINT_WEB_VIEW_HELPER_H_