| OLD | NEW |
| 1 // Copyright 2014 The Chromium Authors. All rights reserved. | 1 // Copyright 2014 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 "chrome/browser/extensions/api/messaging/native_messaging_policy_handle
r.h" | 5 #include "chrome/browser/extensions/api/messaging/native_messaging_policy_handle
r.h" |
| 6 | 6 |
| 7 #include <utility> | 7 #include <utility> |
| 8 | 8 |
| 9 #include "base/logging.h" | 9 #include "base/logging.h" |
| 10 #include "chrome/browser/extensions/api/messaging/native_messaging_host_manifest
.h" | 10 #include "chrome/browser/extensions/api/messaging/native_messaging_host_manifest
.h" |
| (...skipping 50 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 61 if (!value->GetAsList(&list_value)) { | 61 if (!value->GetAsList(&list_value)) { |
| 62 NOTREACHED(); | 62 NOTREACHED(); |
| 63 return false; | 63 return false; |
| 64 } | 64 } |
| 65 | 65 |
| 66 // Filter the list, rejecting any invalid native messaging host names. | 66 // Filter the list, rejecting any invalid native messaging host names. |
| 67 std::unique_ptr<base::ListValue> filtered_list(new base::ListValue()); | 67 std::unique_ptr<base::ListValue> filtered_list(new base::ListValue()); |
| 68 for (base::ListValue::const_iterator entry(list_value->begin()); | 68 for (base::ListValue::const_iterator entry(list_value->begin()); |
| 69 entry != list_value->end(); ++entry) { | 69 entry != list_value->end(); ++entry) { |
| 70 std::string name; | 70 std::string name; |
| 71 if (!entry->GetAsString(&name)) { | 71 if (!(*entry)->GetAsString(&name)) { |
| 72 errors->AddError(policy_name(), entry - list_value->begin(), | 72 errors->AddError(policy_name(), entry - list_value->begin(), |
| 73 IDS_POLICY_TYPE_ERROR, | 73 IDS_POLICY_TYPE_ERROR, |
| 74 base::Value::GetTypeName(base::Value::Type::STRING)); | 74 base::Value::GetTypeName(base::Value::Type::STRING)); |
| 75 continue; | 75 continue; |
| 76 } | 76 } |
| 77 if (!(allow_wildcards_ && name == "*") && | 77 if (!(allow_wildcards_ && name == "*") && |
| 78 !NativeMessagingHostManifest::IsValidName(name)) { | 78 !NativeMessagingHostManifest::IsValidName(name)) { |
| 79 errors->AddError(policy_name(), | 79 errors->AddError(policy_name(), |
| 80 entry - list_value->begin(), | 80 entry - list_value->begin(), |
| 81 IDS_POLICY_VALUE_FORMAT_ERROR); | 81 IDS_POLICY_VALUE_FORMAT_ERROR); |
| 82 continue; | 82 continue; |
| 83 } | 83 } |
| 84 filtered_list->AppendString(name); | 84 filtered_list->AppendString(name); |
| 85 } | 85 } |
| 86 | 86 |
| 87 if (names) | 87 if (names) |
| 88 *names = std::move(filtered_list); | 88 *names = std::move(filtered_list); |
| 89 | 89 |
| 90 return true; | 90 return true; |
| 91 } | 91 } |
| 92 | 92 |
| 93 } // namespace extensions | 93 } // namespace extensions |
| OLD | NEW |