<webview>: Add name attribute

chrome/renderer/resources/extensions/web_view.js

Index: chrome/renderer/resources/extensions/web_view.js
diff --git a/chrome/renderer/resources/extensions/web_view.js b/chrome/renderer/resources/extensions/web_view.js
index 99c504b6dca27094aa2af1b3b7bd2a646a766012..0d2fac0e2492939115c0ff7d1ed2f3d526a28ef7 100644
--- a/chrome/renderer/resources/extensions/web_view.js
+++ b/chrome/renderer/resources/extensions/web_view.js
@@ -9,7 +9,7 @@
 
 var watchForTag = require("tagWatcher").watchForTag;
 
-var WEB_VIEW_ATTRIBUTES = ['src', 'partition'];
+var WEB_VIEW_ATTRIBUTES = ['name', 'src', 'partition'];
 
 // All exposed api methods for <webview>, these are forwarded to the browser
 // plugin.
@@ -125,8 +125,12 @@ WebView.prototype.handleMutation_ = function(mutation) {
  * @private
  */
 WebView.prototype.handleObjectMutation_ = function(mutation) {
-  this.node_.setAttribute(mutation.attributeName,
-      this.objectNode_.getAttribute(mutation.attributeName));
+  if (!this.objectNode_.hasAttribute(mutation.attributeName)) {

[Charlie Reis, 2012/12/13 01:11:42]

I'm not entirely sure which type of attribute change this is preventing.  (Is it
coming from within the BrowserPlugin or from JS code?)  Perhaps a comment with
an example of why we need this would help?

[Fady Samuel, 2012/12/13 17:46:45]

Done.

+    this.node_.removeAttribute(mutation.attributeName);
+  } else {
+    this.node_.setAttribute(mutation.attributeName,
+        this.objectNode_.getAttribute(mutation.attributeName));
+  }
 };
 
 /**