<webview>: Give tabIndex -1 so that tab order is not affected

chrome/renderer/resources/extensions/web_view.js

 // Copyright (c) 2012 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.
 
 // Shim that simulates a <webview> tag via Mutation Observers.
 //
 // The actual tag is implemented via the browser plugin. The internals of this
 // are hidden via Shadow DOM.
 
 'use strict';
@@ skipping 186 matching lines @@
   }, this);
 
   return browserPluginNode;
 };
 
 /**
  * @private
  */
 WebViewInternal.prototype.setupFocusPropagation_ = function() {
   if (!this.webviewNode_.hasAttribute('tabIndex')) {
-    // <webview> needs a tabIndex in order to respond to keyboard focus.
-    // TODO(fsamuel): This introduces unexpected tab ordering. We need to find
-    // a way to take keyboard focus without messing with tab ordering.
+    // <webview> needs a tabIndex in order to be focusable.
+    // TODO(fsamuel): It would be nice to avoid exposing a tabIndex attribute
+    // to allow <webview> to be focusable.
     // See http://crbug.com/231664.
-    this.webviewNode_.setAttribute('tabIndex', 0);
+    this.webviewNode_.setAttribute('tabIndex', -1);
   }
   var self = this;
   this.webviewNode_.addEventListener('focus', function(e) {
     // Focus the BrowserPlugin when the <webview> takes focus.
     self.browserPluginNode_.focus();
   });
   this.webviewNode_.addEventListener('blur', function(e) {
     // Blur the BrowserPlugin when the <webview> loses focus.
     self.browserPluginNode_.blur();
   });
@@ skipping 818 matching lines @@
  * Implemented when the experimental API is available.
  * @private
  */
 WebViewInternal.prototype.maybeGetExperimentalPermissions_ = function() {
   return [];
 };
 
 exports.WebView = WebView;
 exports.WebViewInternal = WebViewInternal;
 exports.CreateEvent = CreateEvent;