Allow MimeHandlerViewGuest be embedded inside OOPIFs

content/browser/renderer_host/render_widget_host_view_base.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_BASE_H_
 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_BASE_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 269 matching lines @@
   // entirely, which comes first.
   virtual bool IsRenderWidgetHostViewGuest();
 
   // Subclass identifier for RenderWidgetHostViewChildFrames. This is useful
   // to be able to know if this RWHV is embedded within another RWHV. If
   // other kinds of embeddable RWHVs are created, this should be renamed to
   // a more generic term -- in which case, static casts to RWHVChildFrame will
   // need to also be resolved.
   virtual bool IsRenderWidgetHostViewChildFrame();
 
+  // TODO(ekaramad): This is used for input event routing when the target of
+  // events is a RenderWidgetHostViewGuest which is embeded in an OOPIF (the
+  // only example right now is MimeHandlerViewGuest). Same as above, this
+  // method should be removed when either direct routing is supported for
+  // RWHVG or RWHVG is removed entirely.
+  virtual RenderWidgetHostViewBase* GetEmbedderView();

[Charlie Reis, 2016/10/18 22:52:20]

Should we be using a different term here, like "outer" instead of "embedder"? 
Within content/, embedder tends to refer to the content/ embedder.

[EhsanK, 2016/10/20 21:41:17]

I think outer is better in that case. But technically the name should be:
GetTargetViewForMouseEvents() or something like that (given that that is the
only application of the method).

+
   //----------------------------------------------------------------------------
   // The following methods are related to IME.
   // TODO(ekaramad): Most of the IME methods should not stay virtual after IME
   // is implemented for OOPIF. After fixing IME, mark the corresponding methods
   // non-virtual (https://crbug.com/578168).
 
   // Updates the state of the input method attached to the view.
   virtual void TextInputStateChanged(const TextInputState& text_input_state);
 
   // Cancel the ongoing composition of the input method attached to the view.
@@ skipping 199 matching lines @@
   base::ObserverList<RenderWidgetHostViewBaseObserver> observers_;
 
   base::WeakPtrFactory<RenderWidgetHostViewBase> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderWidgetHostViewBase);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_BASE_H_