<webview>: Make contentWindow available prior to attachment (on display:none).

extensions/renderer/resources/guest_view/guest_view.js

 // Copyright 2014 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.
 
 // This module implements a wrapper for a guestview that manages its
 // creation, attaching, and destruction.
 
 var CreateEvent = require('guestViewEvents').CreateEvent;
 var EventBindings = require('event_bindings');
 var GuestViewInternal =
@@ skipping 135 matching lines @@
     this.contentWindow = contentWindow;
 
     // Check if attaching failed.
     if (this.contentWindow === null) {
       this.state = GUEST_STATE_CREATED;
     } else {
       // Detach automatically when the container is destroyed.
       GuestViewInternalNatives.RegisterDestructionCallback(internalInstanceId,
                                                            function() {
         if (this.state == GUEST_STATE_ATTACHED) {
-          this.contentWindow = null;
           this.internalInstanceId = 0;
           this.state = GUEST_STATE_CREATED;
         }
       }.bind(this));
     }
 
     this.handleCallback(callback);
   };
 
   attachParams['instanceId'] = viewInstanceId;
@@ skipping 10 matching lines @@
 GuestViewImpl.prototype.createImpl = function(createParams, callback) {
   // Check the current state.
   if (!this.checkState('create')) {
     this.handleCallback(callback);
     return;
   }
 
   // Callback wrapper function to store the guestInstanceId from the
   // createGuest() callback, handle potential creation failure, and advance the
   // queue.
-  var callbackWrapper = function(callback, guestInstanceId) {
-    this.id = guestInstanceId;
+  var callbackWrapper = function(callback, guestInfo) {
+    this.id = guestInfo.id;
+    this.contentWindow =
+        GuestViewInternalNatives.GetContentWindow(guestInfo.contentWindowId);
 
     // Check if creation failed.
     if (this.id === 0) {
       this.state = GUEST_STATE_START;
+      this.contentWindow = null;
     }
 
     ResizeEvent.addListener(this.callOnResize, {instanceId: this.id});
     this.handleCallback(callback);
   };
 
   GuestViewInternal.createGuest(this.viewType,
                                 createParams,
                                 callbackWrapper.bind(this, callback));
 
@@ skipping 37 matching lines @@
   // Check the current state.
   if (!this.checkState('detach')) {
     this.handleCallback(callback);
     return;
   }
 
   GuestViewInternalNatives.DetachGuest(
       this.internalInstanceId,
       this.handleCallback.bind(this, callback));
 
-  this.contentWindow = null;
   this.internalInstanceId = 0;
   this.state = GUEST_STATE_CREATED;
 };
 
 // Internal implementation of setSize().
 GuestViewImpl.prototype.setSizeImpl = function(sizeParams, callback) {
   // Check the current state.
   if (!this.checkState('setSize')) {
     this.handleCallback(callback);
     return;
@@ skipping 58 matching lines @@
 };
 
 // Returns the ID for this guestview.
 GuestView.prototype.getId = function() {
   var internal = privates(this).internal;
   return internal.id;
 };
 
 // Exports
 exports.GuestView = GuestView;