Chromium Code Reviews| Index: webrtc/test/testsupport/test_output.cc |
| diff --git a/webrtc/test/testsupport/test_output.cc b/webrtc/test/testsupport/test_output.cc |
| index aea5110d97d13d8915508702d61046fca73c1497..4396bb97c96750677f243a6e2a070a9c19eebe75 100644 |
| --- a/webrtc/test/testsupport/test_output.cc |
| +++ b/webrtc/test/testsupport/test_output.cc |
| @@ -12,32 +12,36 @@ |
| #include <string.h> |
| -#include "gflags/gflags.h" |
| #include "webrtc/rtc_base/file.h" |
| +#include "webrtc/rtc_base/flags.h" |
| #include "webrtc/rtc_base/logging.h" |
| #include "webrtc/rtc_base/pathutils.h" |
| #include "webrtc/test/testsupport/fileutils.h" |
| +namespace { |
| +const std::string kDefaultOutputPath = webrtc::test::OutputPath(); |
|
kwiberg-webrtc
2017/08/24 09:03:44
Another problematic global variable.
oprypin_webrtc
2017/08/26 07:38:52
Done.
|
| +} |
| + |
| DEFINE_string(test_output_dir, |
| - webrtc::test::OutputPath(), |
| + kDefaultOutputPath.c_str(), |
| "The output folder where test output should be saved."); |
| namespace webrtc { |
| namespace test { |
| bool GetTestOutputDir(std::string* out_dir) { |
| - if (FLAGS_test_output_dir.empty()) { |
| + if (strlen(FLAG_test_output_dir) == 0) { |
| LOG(LS_WARNING) << "No test_out_dir defined."; |
| return false; |
| } |
| - *out_dir = FLAGS_test_output_dir; |
| + *out_dir = FLAG_test_output_dir; |
| return true; |
| } |
| bool WriteToTestOutput(const char* filename, |
| const uint8_t* buffer, |
| size_t length) { |
| - if (FLAGS_test_output_dir.empty()) { |
| + if (strlen(FLAG_test_output_dir) == 0) { |
| LOG(LS_WARNING) << "No test_out_dir defined."; |
| return false; |
| } |
| @@ -48,7 +52,7 @@ bool WriteToTestOutput(const char* filename, |
| } |
| rtc::File output = |
| - rtc::File::Create(rtc::Pathname(FLAGS_test_output_dir, filename)); |
| + rtc::File::Create(rtc::Pathname(FLAG_test_output_dir, filename)); |
| return output.IsOpen() && output.Write(buffer, length) == length; |
| } |