Title: [182777] trunk/Source/WebInspectorUI
- Revision
- 182777
- Author
- [email protected]
- Date
- 2015-04-13 20:00:50 -0700 (Mon, 13 Apr 2015)
Log Message
Web Inspector: Remove unnecessary 3 second delay when archiving page
https://bugs.webkit.org/show_bug.cgi?id=143688
Patch by Joseph Pecoraro <[email protected]> on 2015-04-13
Reviewed by Timothy Hatcher.
* UserInterface/Base/Main.js:
(WebInspector.archiveMainFrame):
The delay is unnecessary and was probably just debugging code.
Modified Paths
Diff
Modified: trunk/Source/WebInspectorUI/ChangeLog (182776 => 182777)
--- trunk/Source/WebInspectorUI/ChangeLog 2015-04-14 02:44:52 UTC (rev 182776)
+++ trunk/Source/WebInspectorUI/ChangeLog 2015-04-14 03:00:50 UTC (rev 182777)
@@ -1,3 +1,14 @@
+2015-04-13 Joseph Pecoraro <[email protected]>
+
+ Web Inspector: Remove unnecessary 3 second delay when archiving page
+ https://bugs.webkit.org/show_bug.cgi?id=143688
+
+ Reviewed by Timothy Hatcher.
+
+ * UserInterface/Base/Main.js:
+ (WebInspector.archiveMainFrame):
+ The delay is unnecessary and was probably just debugging code.
+
2015-04-11 Matt Baker <[email protected]>
Web Inspector: Hide Frames Timeline if the backend doesn't support it
Modified: trunk/Source/WebInspectorUI/UserInterface/Base/Main.js (182776 => 182777)
--- trunk/Source/WebInspectorUI/UserInterface/Base/Main.js 2015-04-14 02:44:52 UTC (rev 182776)
+++ trunk/Source/WebInspectorUI/UserInterface/Base/Main.js 2015-04-14 03:00:50 UTC (rev 182777)
@@ -1874,18 +1874,16 @@
{
this.notifications.dispatchEventToListeners(WebInspector.Notification.PageArchiveStarted, event);
- setTimeout(function() {
- PageAgent.archive(function(error, data) {
- this.notifications.dispatchEventToListeners(WebInspector.Notification.PageArchiveEnded, event);
- if (error)
- return;
+ PageAgent.archive(function(error, data) {
+ this.notifications.dispatchEventToListeners(WebInspector.Notification.PageArchiveEnded, event);
+ if (error)
+ return;
- var mainFrame = WebInspector.frameResourceManager.mainFrame;
- var archiveName = mainFrame.mainResource.urlComponents.host || mainFrame.mainResource.displayName || "Archive";
- var url = "" + encodeURI(archiveName) + ".webarchive";
- InspectorFrontendHost.save(url, data, true, true);
- }.bind(this));
- }.bind(this), 3000);
+ var mainFrame = WebInspector.frameResourceManager.mainFrame;
+ var archiveName = mainFrame.mainResource.urlComponents.host || mainFrame.mainResource.displayName || "Archive";
+ var url = "" + encodeURI(archiveName) + ".webarchive";
+ InspectorFrontendHost.save(url, data, true, true);
+ }.bind(this));
};
WebInspector.canArchiveMainFrame = function()
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
