Prepare GuestView for removing swapped out RenderFrame.

Prepare GuestView for removing swapped out RenderFrame.

GuestView code is dependent on the usage of swapped out RenderFrame for
the out-of-process component. Since swapped out state is going away,
this CL removes this dependency.

BUG=357747
Committed: https://crrev.com/89aa05ab8d8a42d2587e5ad3cc6ac245519ce588
Cr-Commit-Position: refs/heads/master@{#348662}

[nasko, 2015-09-14 15:37:56]

The CQ bit was checked by nasko@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-09-14 15:38:26]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1337283003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1337283003/1

[nasko, 2015-09-14 15:40:20]

nasko@chromium.org changed reviewers:
+ lazyboy@chromium.org

[nasko, 2015-09-14 15:40:20]

Hey Istiaque,
Can you review this CL for me? It moves GuestView to use the WebRemoteFrame V8
context, which I introduced a special method for. This allows us to get off of
swapped out, which will come in a separate CL.
Thanks in advance!
Nasko

[Fady Samuel, 2015-09-14 15:42:46]

fsamuel@chromium.org changed reviewers:
+ fsamuel@chromium.org

[Fady Samuel, 2015-09-14 15:42:46]

https://codereview.chromium.org/1337283003/diff/1/components/guest_view/rende...
File components/guest_view/renderer/guest_view_request.cc (right):

https://codereview.chromium.org/1337283003/diff/1/components/guest_view/rende...
components/guest_view/renderer/guest_view_request.cc:93: if
(frame->isWebLocalFrame()) {
nit: comment would be helpful.

https://codereview.chromium.org/1337283003/diff/1/extensions/renderer/guest_v...
File extensions/renderer/guest_view/guest_view_internal_custom_bindings.cc
(right):

https://codereview.chromium.org/1337283003/diff/1/extensions/renderer/guest_v...
extensions/renderer/guest_view/guest_view_internal_custom_bindings.cc:318:
v8::Local<v8::Value> window;
nit: comment would be helpful

[lazyboy, 2015-09-14 17:04:42]

lgtm

[nasko, 2015-09-14 17:08:20]

The CQ bit was checked by nasko@chromium.org

[commit-bot: I haz the power, 2015-09-14 17:09:00]

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

[Fady Samuel, 2015-09-14 17:09:38]

lgtm

[commit-bot: I haz the power, 2015-09-14 18:33:51]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-09-14 18:34:30]

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

[commit-bot: I haz the power, 2015-09-23 12:34:43]

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