Fixed some edit commands for OOPIF-<webivew> (Mac)

content/browser/renderer_host/render_widget_host_view_mac.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_VIEW_MAC_H_
 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_MAC_H_
 
 #import <Cocoa/Cocoa.h>
 #include <IOSurface/IOSurface.h>
 #include <stddef.h>
@@ skipping 469 matching lines @@
 
   // Returns the TextSelection information for the active widget. If
   // |is_guest_view_hack_| is true, then it will return the TextSelection
   // information for this RenderWidgetHostViewMac (which is serving as a
   // platform view for a guest).
   const TextInputManager::TextSelection* GetTextSelection();
 
   // Get the focused view that should be used for retrieving the text selection.
   RenderWidgetHostViewBase* GetFocusedViewForTextSelection();
 
+  // Returns the RenderWidgetHostDelegate corresponding to the currently focused
+  // RenderWidgetHost. It is different from |render_widget_host_->delegate()|
+  // when there are focused inner WebContentses on the page.

[Charlie Reis, 2017/05/15 21:36:57]

Might mention it's null if there are no focused widgets, since I didn't expect
that at first.

(Will that change behavior in any cases, or is there no way for these commands
to work without a focused widget?  Just wondering if we should default to the
main delegate rather than null.)

[EhsanK, 2017/05/16 04:34:32]

Yes. Done (fallback idea). Thanks!
But still, even with fallback plan, it seems possible that
|render_widget_host_->delegate()| might be nullptr (we were assuming this before
this patch too).

I am also not sure if I should first check if |render_widget_host_ != nullptr|
before calling |delegate()| but to be clear, we were not checking that before
either.

+  RenderWidgetHostDelegate* GetFocusedRenderWidgetHostDelegate();
+
  private:
   friend class RenderWidgetHostViewMacTest;
 
   // Returns whether this render view is a popup (autocomplete window).
   bool IsPopup() const;
 
   // Shuts down the render_widget_host_.  This is a separate function so we can
   // invoke it from the message loop.
   void ShutdownHost();
 
@@ skipping 59 matching lines @@
 
   // Factory used to safely scope delayed calls to ShutdownHost().
   base::WeakPtrFactory<RenderWidgetHostViewMac> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderWidgetHostViewMac);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_MAC_H_