Remove focus/blur webview handlers

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 acba9fb7b0f0aebc19461937ff220bc8266d4589..8cfc8cb441afa19109e320c0a6405f79e9878a85 100644
--- a/extensions/renderer/resources/guest_view/guest_view_container.js
+++ b/extensions/renderer/resources/guest_view/guest_view_container.js
@@ -101,14 +101,6 @@ GuestViewContainer.prototype.setupFocusPropagation = function() {
     // See http://crbug.com/231664.
     this.element.setAttribute('tabIndex', -1);
   }
-  this.element.addEventListener('focus', this.weakWrapper(function(e) {

[Fady Samuel, 2016/07/06 14:55:06]

Not LGTM. This will break focus, although I think we don't have direct test
coverage for this unfortuantely. Basically focus propagation happens through
BrowserPlugin (unless that's changed now?) and so if we don't pass along focus
state to BrowserPlugin then the guest content's focus state will not be updated.

[Fady Samuel, 2016/07/06 17:31:57]

Ah I think I see why/how this is not necessary anymore:
GuestViewContainer.prototype.focus is overridden.

I wonder if we need to override blur too?

[avallee, 2016/07/06 17:34:23]

I could see how propagating the event might make sense. It might want to know
that something blurred the window. However, calling blur() will make it lose
focus completely.

-    // Focus the BrowserPlugin when the GuestViewContainer takes focus.
-    privates(this).internalElement.focus();
-  }));
-  this.element.addEventListener('blur', this.weakWrapper(function(e) {
-    // Blur the BrowserPlugin when the GuestViewContainer loses focus.
-    privates(this).internalElement.blur();
-  }));
 };
 
 GuestViewContainer.prototype.focus = function() {