Fix copying from interstitial pages on OSX by going through the RenderWidgetHostDelegate.

content/browser/renderer_host/render_widget_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_WIDGET_HOST_IMPL_H_
 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_IMPL_H_
 
 #include <deque>
 #include <list>
 #include <map>
@@ skipping 406 matching lines @@
     allow_privileged_mouse_lock_ = allow;
   }
 
   // Resets state variables related to tracking pending size and painting.
   //
   // We need to reset these flags when we want to repaint the contents of
   // browser plugin in this RWH. Resetting these flags will ensure we ignore
   // any previous pending acks that are not relevant upon repaint.
   void ResetSizeAndRepaintPendingFlags();
 
+  RenderWidgetHostDelegate* GetDelegate();

[nasko, 2015/05/13 17:24:08]

Simple accessors, such as this one, use hacker_case as opposed to CamelCase.

[lgarron, 2015/05/13 21:58:40]

get_delegate() or just delegate()?

[Charlie Reis, 2015/05/14 20:36:28]

delegate().

They should also be inlined in the .h file, similar to surface_id() on line 179.

[nasko, 2015/05/14 20:45:23]

Just delegate().

+
   void DetachDelegate();
 
   // Update the renderer's cache of the screen rect of the view and window.
   void SendScreenRects();
 
   // Suppreses future char events until a keydown. See
   // suppress_next_char_events_.
   void SuppressNextCharEvents();
 
   // Called by the view in response to a flush request.
@@ skipping 399 matching lines @@
   bool is_focused_;
 
   base::WeakPtrFactory<RenderWidgetHostImpl> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderWidgetHostImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_IMPL_H_