Return the outermost embedder when returning the embedder web contents.

Return the outermost embedder when returning the embedder web contents.

Currently, ChromeConstrainedWindowViewsClient::GetEmbedderWebContents
returns the embedder WebContents of the GuestView if the initiator
WebContents is in a GuestView. However, with OOP PDF, a mime handler
view can be embedded in a webview that is embedded in an app window; in
this case, the webview WebContents is returned instead of the app window
WebContents. This CL changes GetEmbedderWebContents to return the
outermost embedder WebContents.

BUG=457672
Committed: https://crrev.com/7c0b1291c525a2b6aa3fe810fd69098952bc2d5f
Cr-Commit-Position: refs/heads/master@{#321248}

[Sam McNally, 2015-03-18 06:20:41]

sammc@chromium.org changed reviewers:
+ sky@chromium.org

[sky, 2015-03-18 15:00:36]

LGTM

https://codereview.chromium.org/1012373002/diff/1/chrome/browser/ui/views/chr...
File chrome/browser/ui/views/chrome_constrained_window_views_client.cc (right):

https://codereview.chromium.org/1012373002/diff/1/chrome/browser/ui/views/chr...
chrome/browser/ui/views/chrome_constrained_window_views_client.cc:24: while
(extensions::GuestViewBase* guest_view =
This is worth a comment.

[Sam McNally, 2015-03-18 22:40:57]

https://codereview.chromium.org/1012373002/diff/1/chrome/browser/ui/views/chr...
File chrome/browser/ui/views/chrome_constrained_window_views_client.cc (right):

https://codereview.chromium.org/1012373002/diff/1/chrome/browser/ui/views/chr...
chrome/browser/ui/views/chrome_constrained_window_views_client.cc:24: while
(extensions::GuestViewBase* guest_view =
On 2015/03/18 15:00:36, sky wrote:
> This is worth a comment.

Done.

[Sam McNally, 2015-03-18 22:41:03]

The CQ bit was checked by sammc@chromium.org

[Sam McNally, 2015-03-18 22:41:03]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/1012373002/#ps20001
(title: " ")

[commit-bot: I haz the power, 2015-03-18 22:41:29]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1012373002/20001

[commit-bot: I haz the power, 2015-03-19 00:26:46]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-19 00:27:22]

Patchset 2 (id:??) landed as
https://crrev.com/7c0b1291c525a2b6aa3fe810fd69098952bc2d5f
Cr-Commit-Position: refs/heads/master@{#321248}