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

extensions/renderer/resources/web_view_events.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.
 
 // Event management for WebViewInternal.
 
 var EventBindings = require('event_bindings');
 var MessagingNatives = requireNative('messaging_natives');
 var WebView = require('webViewInternal').WebView;
 
@@ skipping 279 matching lines @@
   }
 };
 
 WebViewEvents.prototype.handleLoadAbortEvent = function(event, webViewEvent) {
   var showWarningMessage = function(reason) {
     var WARNING_MSG_LOAD_ABORTED = '<webview>: ' +
         'The load has aborted with reason "%1".';
     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);
 };
 
 WebViewEvents.prototype.handleNewWindowEvent = function(event, webViewEvent) {
   var ERROR_MSG_NEWWINDOW_ACTION_ALREADY_TAKEN = '<webview>: ' +
       'An action has already been taken for this "newwindow" event.';
 
   var ERROR_MSG_NEWWINDOW_UNABLE_TO_ATTACH = '<webview>: ' +
@@ skipping 206 matching lines @@
   }
 };
 
 WebViewEvents.prototype.handleSizeChangedEvent = function(
     event, webViewEvent) {
   this.webViewInternal.onSizeChanged(webViewEvent);
 };
 
 exports.WebViewEvents = WebViewEvents;
 exports.CreateEvent = CreateEvent;