Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(253)

Side by Side Diff: extensions/browser/quota_service.cc

Issue 598173003: Run clang-modernize -use-nullptr over src/extensions/. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 "extensions/browser/quota_service.h" 5 #include "extensions/browser/quota_service.h"
6 6
7 #include "base/message_loop/message_loop.h" 7 #include "base/message_loop/message_loop.h"
8 #include "base/stl_util.h" 8 #include "base/stl_util.h"
9 #include "extensions/browser/extension_function.h" 9 #include "extensions/browser/extension_function.h"
10 #include "extensions/common/error_utils.h" 10 #include "extensions/common/error_utils.h"
11 11
12 namespace { 12 namespace {
13 13
14 // If the browser stays open long enough, we reset state once a day. 14 // If the browser stays open long enough, we reset state once a day.
15 // Whatever this value is, it should be an order of magnitude longer than 15 // Whatever this value is, it should be an order of magnitude longer than
16 // the longest interval in any of the QuotaLimitHeuristics in use. 16 // the longest interval in any of the QuotaLimitHeuristics in use.
17 const int kPurgeIntervalInDays = 1; 17 const int kPurgeIntervalInDays = 1;
18 18
19 const char kOverQuotaError[] = "This request exceeds the * quota."; 19 const char kOverQuotaError[] = "This request exceeds the * quota.";
20 20
21 } // namespace 21 } // namespace
22 22
23 namespace extensions { 23 namespace extensions {
24 24
25 QuotaService::QuotaService() { 25 QuotaService::QuotaService() {
26 if (base::MessageLoop::current() != NULL) { // Null in unit tests. 26 if (base::MessageLoop::current() != nullptr) { // Null in unit tests.
27 purge_timer_.Start(FROM_HERE, 27 purge_timer_.Start(FROM_HERE,
28 base::TimeDelta::FromDays(kPurgeIntervalInDays), 28 base::TimeDelta::FromDays(kPurgeIntervalInDays),
29 this, 29 this,
30 &QuotaService::Purge); 30 &QuotaService::Purge);
31 } 31 }
32 } 32 }
33 33
34 QuotaService::~QuotaService() { 34 QuotaService::~QuotaService() {
35 DCHECK(CalledOnValidThread()); 35 DCHECK(CalledOnValidThread());
36 purge_timer_.Stop(); 36 purge_timer_.Stop();
(...skipping 18 matching lines...) Expand all
55 function->GetQuotaLimitHeuristics(&heuristics); 55 function->GetQuotaLimitHeuristics(&heuristics);
56 56
57 if (heuristics.empty()) 57 if (heuristics.empty())
58 return std::string(); // No heuristic implies no limit. 58 return std::string(); // No heuristic implies no limit.
59 59
60 ViolationErrorMap::iterator violation_error = 60 ViolationErrorMap::iterator violation_error =
61 violation_errors_.find(extension_id); 61 violation_errors_.find(extension_id);
62 if (violation_error != violation_errors_.end()) 62 if (violation_error != violation_errors_.end())
63 return violation_error->second; // Repeat offender. 63 return violation_error->second; // Repeat offender.
64 64
65 QuotaLimitHeuristic* failed_heuristic = NULL; 65 QuotaLimitHeuristic* failed_heuristic = nullptr;
66 for (QuotaLimitHeuristics::iterator heuristic = heuristics.begin(); 66 for (QuotaLimitHeuristics::iterator heuristic = heuristics.begin();
67 heuristic != heuristics.end(); 67 heuristic != heuristics.end();
68 ++heuristic) { 68 ++heuristic) {
69 // Apply heuristic to each item (bucket). 69 // Apply heuristic to each item (bucket).
70 if (!(*heuristic)->ApplyToArgs(args, event_time)) { 70 if (!(*heuristic)->ApplyToArgs(args, event_time)) {
71 failed_heuristic = *heuristic; 71 failed_heuristic = *heuristic;
72 break; 72 break;
73 } 73 }
74 } 74 }
75 75
(...skipping 107 matching lines...) Expand 10 before | Expand all | Expand 10 after
183 } 183 }
184 184
185 // We can go negative since we check has_tokens when we get to *next* bucket, 185 // We can go negative since we check has_tokens when we get to *next* bucket,
186 // and for the small interval all that matters is whether we used up all the 186 // and for the small interval all that matters is whether we used up all the
187 // tokens (which is true if num_tokens_ <= 0). 187 // tokens (which is true if num_tokens_ <= 0).
188 bucket->DeductToken(); 188 bucket->DeductToken();
189 return true; 189 return true;
190 } 190 }
191 191
192 } // namespace extensions 192 } // namespace extensions
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698