<webview>: Move back, forward, canGoBack, canGoForward, go from content to chrome

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 f4cc6ef5ffef32b23c58f148e7a9628029fe91bf..777bc46974ddc2b7dc3d6f5e00e2fc203e84dbc6 100644
--- a/chrome/renderer/resources/extensions/web_view.js
+++ b/chrome/renderer/resources/extensions/web_view.js
@@ -18,12 +18,7 @@ var WEB_VIEW_ATTRIBUTES = ['name', 'src', 'partition', 'autosize', 'minheight',
 // All exposed api methods for <webview>, these are forwarded to the browser
 // plugin.
 var WEB_VIEW_API_METHODS = [
-  'back',
-  'canGoBack',
-  'canGoForward',
-  'forward',
   'getProcessId',
-  'go',
   'reload',
   'stop',
   'terminate'
@@ -68,6 +63,13 @@ window.addEventListener('readystatechange', function(e) {
   });
 }, true /* useCapture */);
 
+
+/** @type {number} */
+WebView.prototype.entryCount_;
+
+/** @type {number} */
+WebView.prototype.currentEntryIndex_;
+
 /**
  * @constructor
  */
@@ -144,11 +146,37 @@ WebView.prototype.setupFocusPropagation_ = function() {
 WebView.prototype.setupWebviewNodeMethods_ = function() {
   // this.browserPluginNode_[apiMethod] are not necessarily defined immediately
   // after the shadow object is appended to the shadow root.
+  var webviewNode = this.webviewNode_;
+  var browserPluginNode = this.browserPluginNode_;
   var self = this;
+
+  webviewNode['canGoBack'] = function() {
+    return self.entryCount_ > 1 && self.currentEntryIndex_ > 0;

[Matt Perry, 2013/06/25 19:33:12]

I'm not familiar enough with how this works, but I'm assuming that entryCount_
and currentEntryIndex_ can't change behind the scenes (e.g. the browser
navigates but doesn't inform the renderer in time)?

[Fady Samuel, 2013/06/25 21:52:45]

Nope.

+  };
+
+  webviewNode['canGoForward'] = function() {
+    return self.currentEntryIndex_ >=0 &&
+        self.currentEntryIndex_ < (self.entryCount_ - 1);
+  };
+
+  webviewNode['back'] = function() {
+    webviewNode.go(-1);
+  };
+
+  webviewNode['forward'] = function() {
+    webviewNode.go(1);
+  };
+
+  webviewNode['go'] = function(relativeIndex) {
+    var instanceId = browserPluginNode.getGuestInstanceId();
+    if (!instanceId)
+      return;
+    chrome.webview.go(instanceId, relativeIndex);
+  };
+
   $Array.forEach(WEB_VIEW_API_METHODS, function(apiMethod) {
-    self.webviewNode_[apiMethod] = function(var_args) {
-      return $Function.apply(self.browserPluginNode_[apiMethod],
-          self.browserPluginNode_, arguments);
+    webviewNode[apiMethod] = function(var_args) {
+      return browserPluginNode[apiMethod].apply(browserPluginNode, arguments);
     };
   }, this);
   this.setupExecuteCodeAPI_();
@@ -274,6 +302,7 @@ WebView.prototype.handleBrowserPluginAttributeMutation_ = function(mutation) {
  * @private
  */
 WebView.prototype.setupWebviewNodeEvents_ = function() {
+  var self = this;
   var webviewNode = this.webviewNode_;
   // TODO(fsamuel): Generalize this further as we add more events.
   var onAttached = function(e) {
@@ -284,6 +313,8 @@ WebView.prototype.setupWebviewNodeEvents_ = function() {
       $Array.forEach(attribs, function(attribName) {
         webviewEvent[attribName] = event[attribName];
       });
+      self.currentEntryIndex_ = event.currentEntryIndex;
+      self.entryCount_ = event.entryCount;
       webviewNode.dispatchEvent(webviewEvent);
     }, {instanceId: detail.windowId});
   };