Title: [130914] trunk/Source/WebCore
Revision
130914
Author
vse...@chromium.org
Date
2012-10-10 09:22:08 -0700 (Wed, 10 Oct 2012)

Log Message

Web Inspector: [Regression] Stale revisions are not cleared when inspector is opened for the already loaded page.
https://bugs.webkit.org/show_bug.cgi?id=98915

Reviewed by Alexander Pavlov.

WorkspaceController now clears stale revisions on navigation as well.

* inspector/front-end/Workspace.js:
(WebInspector.WorkspaceController):
(WebInspector.WorkspaceController.prototype._mainFrameNavigated):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (130913 => 130914)


--- trunk/Source/WebCore/ChangeLog	2012-10-10 16:20:16 UTC (rev 130913)
+++ trunk/Source/WebCore/ChangeLog	2012-10-10 16:22:08 UTC (rev 130914)
@@ -1,5 +1,18 @@
 2012-10-10  Vsevolod Vlasov  <vse...@chromium.org>
 
+        Web Inspector: [Regression] Stale revisions are not cleared when inspector is opened for the already loaded page.
+        https://bugs.webkit.org/show_bug.cgi?id=98915
+
+        Reviewed by Alexander Pavlov.
+
+        WorkspaceController now clears stale revisions on navigation as well.
+
+        * inspector/front-end/Workspace.js:
+        (WebInspector.WorkspaceController):
+        (WebInspector.WorkspaceController.prototype._mainFrameNavigated):
+
+2012-10-10  Vsevolod Vlasov  <vse...@chromium.org>
+
         Web Inspector: Exception when committing _javascript_ editing with revision history view open.
         https://bugs.webkit.org/show_bug.cgi?id=98908
 

Modified: trunk/Source/WebCore/inspector/front-end/Workspace.js (130913 => 130914)


--- trunk/Source/WebCore/inspector/front-end/Workspace.js	2012-10-10 16:20:16 UTC (rev 130913)
+++ trunk/Source/WebCore/inspector/front-end/Workspace.js	2012-10-10 16:22:08 UTC (rev 130914)
@@ -35,6 +35,7 @@
 {
     this._workspace = workspace;
     WebInspector.resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.MainFrameNavigated, this._mainFrameNavigated, this);
+    WebInspector.resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.FrameAdded, this._frameAdded, this);
 }
 
 WebInspector.WorkspaceController.prototype = {
@@ -44,6 +45,13 @@
         this._workspace.dispatchEventToListeners(WebInspector.Workspace.Events.ProjectWillReset, this._workspace.project());
         this._workspace.project().reset();
         this._workspace.dispatchEventToListeners(WebInspector.Workspace.Events.ProjectDidReset, this._workspace.project());
+    },
+
+    _frameAdded: function(event)
+    {
+        var frame = /** @type {WebInspector.ResourceTreeFrame} */ event.data;
+        if (frame.isMainFrame())
+            WebInspector.Revision.filterOutStaleRevisions();
     }
 }
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to