Fix purging logic for dom_storage caches

content/browser/dom_storage/dom_storage_context_impl.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 "content/browser/dom_storage/dom_storage_context_impl.h"
 
 #include <stddef.h>
 #include <stdlib.h>
 
 #include "base/bind.h"
@@ skipping 425 matching lines @@
   // Track the total localStorage cache size.
   UMA_HISTOGRAM_CUSTOM_COUNTS("LocalStorage.BrowserLocalStorageCacheSizeInKB",
                               initial_stats.total_cache_size / 1024, 1, 100000,
                               50);
 
   if (purge_option == PURGE_IF_NEEDED) {
     // Purging is done based on the cache sizes without including the database
     // size since it can be expensive trying to estimate the sqlite usage for
     // all databases. For low end devices purge all inactive areas.
     bool should_purge =
-        (is_low_end_device_ && initial_stats.inactive_area_count) ||
-        initial_stats.total_cache_size > kMaxCacheSize ||
-        initial_stats.total_area_count > kMaxStorageAreaCount;
+        initial_stats.inactive_area_count &&
+        (is_low_end_device_ || initial_stats.total_cache_size > kMaxCacheSize ||
+         initial_stats.total_area_count > kMaxStorageAreaCount);
     if (!should_purge)
       return;
 
     purge_option = PURGE_UNOPENED;
   }
 
   bool aggressively = purge_option == PURGE_AGGRESSIVE;
   for (const auto& it : namespaces_)
     it.second->PurgeMemory(aggressively);
 
@@ skipping 80 matching lines @@
   if (!deletable_persistent_namespace_ids_.empty()) {
     task_runner_->PostDelayedTask(
         FROM_HERE, base::Bind(
             &DOMStorageContextImpl::DeleteNextUnusedNamespace,
             this),
         base::TimeDelta::FromSeconds(kSessionStoraceScavengingSeconds));
   }
 }
 
 }  // namespace content