Make --log-net-log and NetLogLogger hide cookies by default,

chrome/browser/net/net_log_temp_file_unittest.cc

 // Copyright (c) 2013 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.
 
 #include "chrome/browser/net/net_log_temp_file.h"
 
 #include "base/basictypes.h"
 #include "base/files/file_path.h"
 #include "base/files/file_util.h"
 #include "base/message_loop/message_loop.h"
@@ skipping 158 matching lines @@
     EXPECT_EQ(NetLogTempFile::LOG_TYPE_NORMAL, net_log_temp_file_->log_type());
 
     base::FilePath net_export_file_path;
     EXPECT_TRUE(net_log_temp_file_->GetFilePath(&net_export_file_path));
     EXPECT_TRUE(base::PathExists(net_export_file_path));
     EXPECT_EQ(net_export_log_, net_export_file_path);
 
     VerifyNetExportLog();
   }
 
+  // Make sure the export file has been successfully initialized.
+  void VerifyFileAndStateAfterDoStopWithStripPrivateData() {
+    EXPECT_EQ("NOT_LOGGING", GetStateString());
+    EXPECT_EQ(NetLogTempFile::STATE_NOT_LOGGING, net_log_temp_file_->state());
+    EXPECT_EQ("STRIP_PRIVATE_DATA", GetLogTypeString());
+    EXPECT_EQ(NetLogTempFile::LOG_TYPE_STRIP_PRIVATE_DATA,
+              net_log_temp_file_->log_type());
+
+    base::FilePath net_export_file_path;
+    EXPECT_TRUE(net_log_temp_file_->GetFilePath(&net_export_file_path));
+    EXPECT_TRUE(base::PathExists(net_export_file_path));
+    EXPECT_EQ(net_export_log_, net_export_file_path);
+
+    VerifyNetExportLog();
+  }
+
   scoped_ptr<ChromeNetLog> net_log_;
   // |net_log_temp_file_| is initialized after |net_log_| so that it can stop
   // obvserving on destruction.
   scoped_ptr<TestNetLogTempFile> net_log_temp_file_;
   base::FilePath net_export_log_;
 
  private:
   base::MessageLoop message_loop_;
   content::TestBrowserThread file_user_blocking_thread_;
 };
@@ skipping 48 matching lines @@
   VerifyFileAndStateAfterDoStart();
 
   net_log_temp_file_->ProcessCommand(NetLogTempFile::DO_STOP);
   VerifyFileAndStateAfterDoStop();
 
   // Calling DO_STOP second time should be a no-op.
   net_log_temp_file_->ProcessCommand(NetLogTempFile::DO_STOP);
   VerifyFileAndStateAfterDoStop();
 }
 
+TEST_F(NetLogTempFileTest,
+       ProcessCommandDoStartAndStopWithPrivateDataStripping) {
+  net_log_temp_file_->ProcessCommand(
+      NetLogTempFile::DO_START_STRIP_PRIVATE_DATA);
+  VerifyFileAndStateAfterDoStartStripPrivateData();
+
+  // Calling DO_START_STRIP_PRIVATE_DATA second time should be a no-op.
+  net_log_temp_file_->ProcessCommand(
+      NetLogTempFile::DO_START_STRIP_PRIVATE_DATA);
+  VerifyFileAndStateAfterDoStartStripPrivateData();
+
+  net_log_temp_file_->ProcessCommand(NetLogTempFile::DO_STOP);
+  VerifyFileAndStateAfterDoStopWithStripPrivateData();
+
+  // Calling DO_STOP second time should be a no-op.
+  net_log_temp_file_->ProcessCommand(NetLogTempFile::DO_STOP);
+  VerifyFileAndStateAfterDoStopWithStripPrivateData();
+}
+
 TEST_F(NetLogTempFileTest, DoStartClearsFile) {
   // Verify file sizes after two consecutives start/stop are the same (even if
   // we add some junk data in between).
   net_log_temp_file_->ProcessCommand(NetLogTempFile::DO_START);
   VerifyFileAndStateAfterDoStart();
 
   int64 start_file_size;
   EXPECT_TRUE(base::GetFileSize(net_export_log_, &start_file_size));
 
   net_log_temp_file_->ProcessCommand(NetLogTempFile::DO_STOP);
@@ skipping 50 matching lines @@
   // Log an event.
   net_log_->AddGlobalEntry(net::NetLog::TYPE_CANCELLED);
 
   net_log_temp_file_->ProcessCommand(NetLogTempFile::DO_STOP);
   VerifyFileAndStateAfterDoStop();
 
   int64 new_stop_file_size;
   EXPECT_TRUE(base::GetFileSize(net_export_log_, &new_stop_file_size));
   EXPECT_GE(new_stop_file_size, stop_file_size);
 }