Force MimeHandlerView to always use BrowserPlugin

content/public/browser/guest_mode.h

+// Copyright 2016 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_PUBLIC_BROWSER_GUEST_MODE_H_
+#define CONTENT_PUBLIC_BROWSER_GUEST_MODE_H_
+
+#include "base/macros.h"
+#include "content/common/content_export.h"
+
+namespace content {
+
+class RenderViewHost;
+class WebContents;
+
+// TODO(ekaramad): This class is temporarily added to provide a workaround for
+// allowing the existence of some guests based on browser plugin rather than
+// cross process frames. It will be removed once the issue
+// https://crbug.com/642826 is resolved.
+class CONTENT_EXPORT GuestMode {
+ public:
+  // Returns true if |web_contents| is an inner WebContents based on cross
+  // process frames.
+  static bool IsInnerWebContentsUsingCrossProcessFrames(

[Charlie Reis, 2016/09/21 22:18:21]

Given that we're calling this class GuestMode (and other "guest" references
exist in content/ names), I'm ok with the simpler IsCrossProcessFrameGuest
method name that we had before.  I think that's a bit easier to read.

(The comment can keep the "inner WebContents" reference.)

[EhsanK, 2016/09/22 22:55:44]

Acknowledged.

+      WebContents* web_contents);
+
+  // Returns true if the |rvh| belongs to an inner WebContents based on cross
+  // process frames.
+  static bool ForInnerWebContentsUsingCrossProcessFrames(RenderViewHost* rvh);

[Charlie Reis, 2016/09/21 22:18:21]

IsCrossProcessFrameGuest(rvh)

(Actually, we can remove this.  See my comment in RFHM.)

[EhsanK, 2016/09/22 22:55:44]

Acknowledged.

+};
+
+}  // namespace content
+
+#endif  // CONTENT_PUBLIC_BROWSER_GUEST_MODE_H_