Fix for how webview.executeScript and webivew.insertCSS check for page navigtation when loading dat…

extensions/renderer/resources/web_view_events.js

Index: extensions/renderer/resources/web_view_events.js
diff --git a/extensions/renderer/resources/web_view_events.js b/extensions/renderer/resources/web_view_events.js
index 137baa2ee45e469dcaf1df7a4d9bcda1ecc83cfd..422a1a7bbefa02bc860fac95ce16a100cacb7358 100644
--- a/extensions/renderer/resources/web_view_events.js
+++ b/extensions/renderer/resources/web_view_events.js
@@ -297,12 +297,13 @@ WebViewEvents.prototype.handleLoadAbortEvent = function(event, webViewEvent) {
     window.console.warn(WARNING_MSG_LOAD_ABORTED.replace('%1', reason));
   };
   if (this.webViewInternal.dispatchEvent(webViewEvent)) {
-    showWarningMessage(event.reason);
+    showWarningMessqage(event.reason);

[Fady Samuel, 2014/09/25 20:09:03]

What happened here?

[paulmeyer, 2014/09/25 20:39:51]

Looks like a misguided 'q' got inserted here. My guess is that my cursor
happened to be here and I tried a Ctrl+Q but the Ctrl didn't register or
something like that.

Fixed.
On 2014/09/25 20:09:03, Fady Samuel wrote:

   }
 };
 
 WebViewEvents.prototype.handleLoadCommitEvent = function(event, webViewEvent) {
-  this.webViewInternal.onLoadCommit(event.currentEntryIndex, event.entryCount,
+  this.webViewInternal.onLoadCommit(event.baseURLForDataURL,
+                                    event.currentEntryIndex, event.entryCount,
                                     event.processId, event.url,
                                     event.isTopLevel);
   this.webViewInternal.dispatchEvent(webViewEvent);