Remove focus/blur webview handlers

extensions/renderer/resources/guest_view/guest_view_container.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 the shared functionality for different guestview
 // containers, such as web_view, app_view, etc.
 
 var DocumentNatives = requireNative('document_natives');
 var GuestView = require('guestView').GuestView;
 var GuestViewInternalNatives = requireNative('guest_view_internal');
@@ skipping 83 matching lines @@
 };
 
 GuestViewContainer.prototype.setupFocusPropagation = function() {
   if (!this.element.hasAttribute('tabIndex')) {
     // GuestViewContainer needs a tabIndex in order to be focusable.
     // TODO(fsamuel): It would be nice to avoid exposing a tabIndex attribute
     // to allow GuestViewContainer to be focusable.
     // 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() {
   // 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;
@@ skipping 180 matching lines @@
   // Delete the callbacks so developers cannot call them and produce unexpected
   // behavior.
   delete proto.createdCallback;
   delete proto.attachedCallback;
   delete proto.detachedCallback;
   delete proto.attributeChangedCallback;
 }
 
 // Exports.
 exports.$set('GuestViewContainer', GuestViewContainer);