NOTREACHED() in VarTracker::ReleaseVar() can be reached. Remove it.

ppapi/shared_impl/var_tracker.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ppapi/shared_impl/var_tracker.h"
 
 #include <string.h>
 
 #include <limits>
 
@@ skipping 76 matching lines @@
     return false;
   return AddRefVar(static_cast<int32>(var.value.as_id));
 }
 
 bool VarTracker::ReleaseVar(int32 var_id) {
   DCHECK(CalledOnValidThread());
 
   DLOG_IF(ERROR, !CheckIdType(var_id, PP_ID_TYPE_VAR))
       << var_id << " is not a PP_Var ID.";
   VarMap::iterator found = live_vars_.find(var_id);
-  if (found == live_vars_.end()) {
-    NOTREACHED() << "Unref-ing an invalid var";
+  if (found == live_vars_.end())
     return false;
-  }
 
   VarInfo& info = found->second;
   if (info.ref_count == 0) {
     NOTREACHED() << "Releasing an object with zero ref";
     return false;
   }
   info.ref_count--;
 
   if (info.ref_count == 0) {
     if (info.var->GetType() == PP_VARTYPE_OBJECT) {
@@ skipping 111 matching lines @@
 bool VarTracker::DeleteObjectInfoIfNecessary(VarMap::iterator iter) {
   if (iter->second.ref_count != 0 ||
       iter->second.track_with_no_reference_count != 0)
     return false;  // Object still alive.
   iter->second.var->ResetVarID();
   live_vars_.erase(iter);
   return true;
 }
 
 }  // namespace ppapi