Remove NetLog::GetCaptureMode() and NetLogCaptureMode::None()

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/json/json_reader.h"
@@ skipping 180 matching lines @@
  private:
   // Checks state after one of the DO_START* commands.
   void VerifyFileAndStateAfterStart(
       NetLogTempFile::LogType expected_log_type,
       const std::string& expected_log_type_string,
       net::NetLogCaptureMode expected_capture_mode) const {
     EXPECT_EQ(NetLogTempFile::STATE_LOGGING, net_log_temp_file_->state());
     EXPECT_EQ("LOGGING", GetStateString());
     EXPECT_EQ(expected_log_type, net_log_temp_file_->log_type());
     EXPECT_EQ(expected_log_type_string, GetLogTypeString());
 

[mmenke, 2015/05/06 15:14:41]

Since you removed the check here, we have no check that we're actually setting
the log level, as opposed to the NetLogTemp file just claiming it did.  Maybe
add a protected method to NetLogTempFile to get write_to_file_observer_'s
CaptureMode?

[eroman, 2015/05/06 17:01:04]

Oops my bad (I meant to add a check back for this)!
Done.

-    EXPECT_EQ(expected_capture_mode, net_log_->GetCaptureMode());
-
     // Check GetFilePath returns false when still writing to the file.
     base::FilePath net_export_file_path;
     EXPECT_FALSE(net_log_temp_file_->GetFilePath(&net_export_file_path));
 
     VerifyNetExportLogExists();
   }
 
   void VerifyFileAndStateAfterDoStop(
       NetLogTempFile::LogType expected_log_type,
       const std::string& expected_log_type_string) const {
@@ skipping 177 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);
 }