OLD | NEW |
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
4 | 4 |
5 #include "remoting/host/json_host_config.h" | 5 #include "remoting/host/json_host_config.h" |
6 | 6 |
7 #include "base/bind.h" | 7 #include "base/bind.h" |
8 #include "base/file_util.h" | 8 #include "base/file_util.h" |
9 #include "base/json/json_reader.h" | 9 #include "base/json/json_reader.h" |
10 #include "base/json/json_writer.h" | 10 #include "base/json/json_writer.h" |
(...skipping 12 matching lines...) Expand all Loading... |
23 bool JsonHostConfig::Read() { | 23 bool JsonHostConfig::Read() { |
24 DCHECK(CalledOnValidThread()); | 24 DCHECK(CalledOnValidThread()); |
25 | 25 |
26 // TODO(sergeyu): Implement better error handling here. | 26 // TODO(sergeyu): Implement better error handling here. |
27 std::string file_content; | 27 std::string file_content; |
28 if (!file_util::ReadFileToString(filename_, &file_content)) { | 28 if (!file_util::ReadFileToString(filename_, &file_content)) { |
29 LOG(WARNING) << "Failed to read " << filename_.value(); | 29 LOG(WARNING) << "Failed to read " << filename_.value(); |
30 return false; | 30 return false; |
31 } | 31 } |
32 | 32 |
33 scoped_ptr<Value> value(base::JSONReader::Read(file_content, true)); | 33 scoped_ptr<Value> value( |
| 34 base::JSONReader::Read(file_content, base::JSON_ALLOW_TRAILING_COMMAS)); |
34 if (value.get() == NULL || !value->IsType(Value::TYPE_DICTIONARY)) { | 35 if (value.get() == NULL || !value->IsType(Value::TYPE_DICTIONARY)) { |
35 LOG(WARNING) << "Failed to parse " << filename_.value(); | 36 LOG(WARNING) << "Failed to parse " << filename_.value(); |
36 return false; | 37 return false; |
37 } | 38 } |
38 | 39 |
39 DictionaryValue* dictionary = static_cast<DictionaryValue*>(value.release()); | 40 DictionaryValue* dictionary = static_cast<DictionaryValue*>(value.release()); |
40 values_.reset(dictionary); | 41 values_.reset(dictionary); |
41 return true; | 42 return true; |
42 } | 43 } |
43 | 44 |
44 bool JsonHostConfig::Save() { | 45 bool JsonHostConfig::Save() { |
45 DCHECK(CalledOnValidThread()); | 46 DCHECK(CalledOnValidThread()); |
46 | 47 |
47 std::string file_content; | 48 std::string file_content; |
48 base::JSONWriter::WriteWithOptions(values_.get(), | 49 base::JSONWriter::WriteWithOptions(values_.get(), |
49 base::JSONWriter::OPTIONS_PRETTY_PRINT, | 50 base::JSONWriter::OPTIONS_PRETTY_PRINT, |
50 &file_content); | 51 &file_content); |
51 // TODO(sergeyu): Move ImportantFileWriter to base and use it here. | 52 // TODO(sergeyu): Move ImportantFileWriter to base and use it here. |
52 int result = file_util::WriteFile(filename_, file_content.c_str(), | 53 int result = file_util::WriteFile(filename_, file_content.c_str(), |
53 file_content.size()); | 54 file_content.size()); |
54 return result == static_cast<int>(file_content.size()); | 55 return result == static_cast<int>(file_content.size()); |
55 } | 56 } |
56 | 57 |
57 } // namespace remoting | 58 } // namespace remoting |
OLD | NEW |