Calling webview.focus() now works correctly

extensions/renderer/resources/guest_view/guest_view_container.js

Index: extensions/renderer/resources/guest_view/guest_view_container.js
diff --git a/extensions/renderer/resources/guest_view/guest_view_container.js b/extensions/renderer/resources/guest_view/guest_view_container.js
index f9d1a97a02a55d2cfa56530d64ced92e120aaf5b..3e471d4af01b2aa8cccd3dd3dd0148fb5ca816cb 100644
--- a/extensions/renderer/resources/guest_view/guest_view_container.js
+++ b/extensions/renderer/resources/guest_view/guest_view_container.js
@@ -111,6 +111,11 @@ GuestViewContainer.prototype.setupFocusPropagation = function() {
   }));
 };
 
+GuestViewContainer.prototype.focus = function() {
+  // Focus the internal element when focus() is called on the GuestView element.
+  privates(this).internalElement.focus();
+}
+
 GuestViewContainer.prototype.attachWindow$ = function() {
   if (!this.internalInstanceId) {
     return true;
@@ -234,7 +239,7 @@ function registerInternalElement(viewType) {
 
 // Registers the guestview container as a custom element.
 // |guestViewContainerType| is the type of guestview container
-// (e.g.WebViewImpl).
+// (e.g. WebViewImpl).
 function registerGuestViewElement(guestViewContainerType) {
   var proto = $Object.create(HTMLElement.prototype);
 
@@ -272,6 +277,15 @@ function registerGuestViewElement(guestViewContainerType) {
     internal.onElementDetached();
   };
 
+  // Override |focus| to let |internal| handle it.
+  proto.focus = function() {

[Fady Samuel, 2015/07/08 14:38:11]

What about blur? Also does this pass existing focus tests?

[paulmeyer, 2015/07/08 20:19:50]

I do not believe that doing the same thing for blur would be meaningful, since
there is not the same issue of calling blur() on an already blurred webview
leading to something else suddenly becoming unblurred. It could be added for
consistency, though I am not sure if it is warranted in this case to add extra
code that isn't needed purely for consistency.

This does pass all existing focus tests.

+    var internal = privates(this).internal;
+    if (!internal) {
+      return;
+    }
+    internal.focus();
+  };
+
   // Let the specific view type add extra functionality to its custom element
   // through |proto|.
   if (guestViewContainerType.setupElement) {