Remove base::Value::IsType in //chrome
This change removes base::Value::IsType in the //chrome directory.
It is superfluous and not part of the new API. Existing usages are
replaced by base::Value::is_* if possible (e.g. |val.is_int()| instead
of |val.IsType(base::Value::Type::INTEGER)|). Otherwise, |type()| in
combination with operator== is used (e.g. |val.type() == type| instead
of |val.IsType(type)|).
Bug: 646113
Change-Id: Id285b993173ec190fa0a68487c7be813d4952256
Reviewed-on: https://chromium-review.googlesource.com/732655
Reviewed-by: Lei Zhang <[email protected]>
Commit-Queue: Jan Wilken Dörrie <[email protected]>
Cr-Commit-Position: refs/heads/master@{#510829}diff --git a/chrome/browser/extensions/policy_handlers.cc b/chrome/browser/extensions/policy_handlers.cc
index 4a2186b..5affb86 100644
--- a/chrome/browser/extensions/policy_handlers.cc
+++ b/chrome/browser/extensions/policy_handlers.cc
@@ -234,14 +234,14 @@
// |policy_value| is expected to conform to the defined schema. But it's
// not strictly valid since there are additional restrictions.
const base::DictionaryValue* dict_value = NULL;
- DCHECK(policy_value->IsType(base::Value::Type::DICTIONARY));
+ DCHECK(policy_value->is_dict());
policy_value->GetAsDictionary(&dict_value);
for (base::DictionaryValue::Iterator it(*dict_value); !it.IsAtEnd();
it.Advance()) {
DCHECK(it.key() == schema_constants::kWildcard ||
crx_file::id_util::IdIsValid(it.key()));
- DCHECK(it.value().IsType(base::Value::Type::DICTIONARY));
+ DCHECK(it.value().is_dict());
// Extracts sub dictionary.
const base::DictionaryValue* sub_dict = NULL;
