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

Side by Side Diff: chrome/browser/chromeos/extensions/file_manager/private_api_base.cc

Issue 1547093002: Switch to standard integer types in chrome/browser/chromeos/. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 12 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 "chrome/browser/chromeos/extensions/file_manager/private_api_base.h" 5 #include "chrome/browser/chromeos/extensions/file_manager/private_api_base.h"
6 6
7 #include <stdint.h>
8
7 #include "base/strings/string_number_conversions.h" 9 #include "base/strings/string_number_conversions.h"
8 #include "chrome/browser/chromeos/extensions/file_manager/private_api_util.h" 10 #include "chrome/browser/chromeos/extensions/file_manager/private_api_util.h"
9 #include "components/drive/event_logger.h" 11 #include "components/drive/event_logger.h"
10 12
11 namespace extensions { 13 namespace extensions {
12 namespace { 14 namespace {
13 15
14 const int kSlowOperationThresholdMs = 500; // In ms. 16 const int kSlowOperationThresholdMs = 500; // In ms.
15 17
16 } // namespace 18 } // namespace
17 19
18 LoggedAsyncExtensionFunction::LoggedAsyncExtensionFunction() 20 LoggedAsyncExtensionFunction::LoggedAsyncExtensionFunction()
19 : log_on_completion_(false) { 21 : log_on_completion_(false) {
20 start_time_ = base::Time::Now(); 22 start_time_ = base::Time::Now();
21 } 23 }
22 24
23 LoggedAsyncExtensionFunction::~LoggedAsyncExtensionFunction() { 25 LoggedAsyncExtensionFunction::~LoggedAsyncExtensionFunction() {
24 } 26 }
25 27
26 void LoggedAsyncExtensionFunction::SendResponse(bool success) { 28 void LoggedAsyncExtensionFunction::SendResponse(bool success) {
27 drive::EventLogger* logger = file_manager::util::GetLogger(GetProfile()); 29 drive::EventLogger* logger = file_manager::util::GetLogger(GetProfile());
28 if (logger) { 30 if (logger) {
29 int64 elapsed = (base::Time::Now() - start_time_).InMilliseconds(); 31 int64_t elapsed = (base::Time::Now() - start_time_).InMilliseconds();
30 if (log_on_completion_) { 32 if (log_on_completion_) {
31 logger->Log(logging::LOG_INFO, "%s[%d] %s. (elapsed time: %sms)", name(), 33 logger->Log(logging::LOG_INFO, "%s[%d] %s. (elapsed time: %sms)", name(),
32 request_id(), success ? "succeeded" : "failed", 34 request_id(), success ? "succeeded" : "failed",
33 base::Int64ToString(elapsed).c_str()); 35 base::Int64ToString(elapsed).c_str());
34 } else if (elapsed >= kSlowOperationThresholdMs) { 36 } else if (elapsed >= kSlowOperationThresholdMs) {
35 logger->Log(logging::LOG_WARNING, 37 logger->Log(logging::LOG_WARNING,
36 "PEFORMANCE WARNING: %s[%d] was slow. (elapsed time: %sms)", 38 "PEFORMANCE WARNING: %s[%d] was slow. (elapsed time: %sms)",
37 name(), request_id(), base::Int64ToString(elapsed).c_str()); 39 name(), request_id(), base::Int64ToString(elapsed).c_str());
38 } 40 }
39 } 41 }
40 ChromeAsyncExtensionFunction::SendResponse(success); 42 ChromeAsyncExtensionFunction::SendResponse(success);
41 } 43 }
42 44
43 } // namespace extensions 45 } // namespace extensions
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698