Fix the issue that any tab/window closing hotkey will crash the Pepper Flash Fullscreen.

content/browser/renderer_host/render_view_host_impl.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_IMPL_H_
 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_VIEW_HOST_IMPL_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 106 matching lines @@
   // |routing_id| could be a valid route id, or it could be MSG_ROUTING_NONE, in
   // which case RenderWidgetHost will create a new one.  |swapped_out| indicates
   // whether the view should initially be swapped out (e.g., for an opener
   // frame being rendered by another process).
   //
   // The |session_storage_namespace| parameter allows multiple render views and
   // WebContentses to share the same session storage (part of the WebStorage
   // spec) space. This is useful when restoring contentses, but most callers
   // should pass in NULL which will cause a new SessionStorageNamespace to be
   // created.
+  //
+  // If this object outlives |widget_delegate|,
+  // RenderWidgetHostImpl::DetachDelegate() must be called when
+  // |widget_delegate| goes away.

[jam, 2012/09/22 00:01:34]

since this derives from RenderWidgetHostImpl, is this comment needed?

[yzshen1, 2012/09/22 00:28:08]

I imagine that people wouldn't be careful enough to read the comments of the
base class, and figure out this parameter would be forwarded to the base class.

Do you think it is okay to keep it?

On 2012/09/22 00:01:34, John Abd-El-Malek wrote:

[jam, 2012/09/24 16:47:00]

what I was trying to say is that this class doesn't have to do anything because
its parents class already does, so the comment is needless.

[yzshen1, 2012/09/24 16:59:41]

Removed. Thanks!
On 2012/09/24 16:47:00, John Abd-El-Malek wrote:

   RenderViewHostImpl(
       SiteInstance* instance,
       RenderViewHostDelegate* delegate,
       RenderWidgetHostDelegate* widget_delegate,
       int routing_id,
       bool swapped_out,
       SessionStorageNamespace* session_storage_namespace);
   virtual ~RenderViewHostImpl();
 
   // RenderViewHost implementation.
@@ skipping 541 matching lines @@
   DISALLOW_COPY_AND_ASSIGN(RenderViewHostImpl);
 };
 
 #if defined(COMPILER_MSVC)
 #pragma warning(pop)
 #endif
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_RENDER_VIEW_HOST_IMPL_H_