Fixed check return warnings from Coverity.

chrome/browser/policy/managed_mode_policy_provider.cc

Index: chrome/browser/policy/managed_mode_policy_provider.cc
diff --git a/chrome/browser/policy/managed_mode_policy_provider.cc b/chrome/browser/policy/managed_mode_policy_provider.cc
index e0521679330073a464e82842617f1a545d0c482a..97570c68e2101a71b235ce56a3e9db9896e82428 100644
--- a/chrome/browser/policy/managed_mode_policy_provider.cc
+++ b/chrome/browser/policy/managed_mode_policy_provider.cc
@@ -43,7 +43,8 @@ const base::Value* ManagedModePolicyProvider::GetPolicy(
     const std::string& key) const {
   base::DictionaryValue* dict = GetCachedPolicy();
   base::Value* value = NULL;
-  dict->GetWithoutPathExpansion(key, &value);
+  bool rv = dict->GetWithoutPathExpansion(key, &value);
+  DCHECK(rv);

[Mattias Nissler (ping if slow), 2012/07/26 09:37:06]

I think this is wrong, the intention here is to return NULL if the key doesn't
exist (i.e. !rv), which now DCHECKS.

[Danh Nguyen, 2012/07/26 15:46:27]

Reverted. I'll remove the DCHECK and add a note here.

   return value;
 }