Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2795)

Unified Diff: chrome/browser/resources/net_internals/main.js

Issue 10332238: net-internals: Fix cookies not being hidden, when saving after capturing is stopped (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Oops Created 8 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/net_internals/main.js
===================================================================
--- chrome/browser/resources/net_internals/main.js (revision 137673)
+++ chrome/browser/resources/net_internals/main.js (working copy)
@@ -169,14 +169,17 @@
onLoadLog: function(opt_fileName) {
isViewingLoadedLog = true;
- SourceTracker.getInstance().setSecurityStripping(false);
this.stopCapturing();
if (opt_fileName != undefined) {
// If there's a file name, a log file was loaded, so swap out the status
- // bar to indicate we're no longer capturing events.
+ // bar to indicate we're no longer capturing events. Also disable
+ // hiding cookies, so if the log dump has them, they'll be displayed.
this.statusView_.switchToSubView('loaded').setFileName(opt_fileName);
+ SourceTracker.getInstance().setSecurityStripping(false);
} else {
// Otherwise, the "Stop Capturing" button was presumably pressed.
+ // Don't disable hiding cookies, so created log dumps won't have them,
+ // unless the user toggles the option.
this.statusView_.switchToSubView('halted');
}
},
« no previous file with comments | « chrome/browser/resources/net_internals/import_view.js ('k') | chrome/test/data/webui/net_internals/log_util.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698