Extract the element implementation logic to function mods in <webview>.

extensions/renderer/resources/guest_view/web_view/web_view_iframe.js

+// Copyright 2015 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 provides --site-per-process overrides for WebView (<webview>).
+// See web_view.js for details.
+
+var IdGenerator = requireNative('id_generator');
+var WebViewImpl = require('webView').WebViewImpl;
+// NOTE: Do not remove these, we implicitly depend on these in
+// --site-per-process.
+var GuestViewIframe = require('guestViewIframe');
+var GuestViewIframeContainer = require('guestViewIframeContainer');
+
+WebViewImpl.prototype.attachWindow$ = function(opt_guestInstanceId) {
+  // If |opt_guestInstanceId| was provided, then a different existing guest is
+  // being attached to this webview, and the current one will get destroyed.
+  if (opt_guestInstanceId) {
+    if (this.guest.getId() == opt_guestInstanceId) {
+      return true;
+    }
+    this.guest.destroy();
+    this.guest = new GuestView('webview', opt_guestInstanceId);
+  }
+
+  var generatedID = IdGenerator.GetNextId();

[Fady Samuel, 2015/06/09 03:16:32]

nit: generatedId

[lazyboy, 2015/06/09 18:48:23]

Done.

+  // We do have a plugin to set our instance id, so do it here.

[Fady Samuel, 2015/06/09 03:16:32]

I wouldn't mention plugin here at all. Simply say that we generate an instance
Id for the iframe.

[lazyboy, 2015/06/09 18:48:23]

Done.

+  this.onInternalInstanceID(generatedID);

[Fady Samuel, 2015/06/09 03:16:32]

nit: Id

[lazyboy, 2015/06/09 18:48:23]

Done.

+  return true;
+};