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

Side by Side Diff: chrome/app/android/chrome_main_delegate_android.cc

Issue 23443009: [Android] Support startup histogram. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: [Android] Support startup histogram - fix missing copyright notice Created 7 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/app/android/chrome_main_delegate_android.h" 5 #include "chrome/app/android/chrome_main_delegate_android.h"
6 6
7 #include "base/android/jni_android.h" 7 #include "base/android/jni_android.h"
8 #include "base/debug/trace_event.h" 8 #include "base/debug/trace_event.h"
9 #include "chrome/browser/android/chrome_jni_registrar.h" 9 #include "chrome/browser/android/chrome_jni_registrar.h"
10 #include "chrome/browser/android/chrome_startup_flags.h" 10 #include "chrome/browser/android/chrome_startup_flags.h"
11 #include "chrome/browser/android/uma_utils.h"
12 #include "chrome/common/startup_metric_utils.h"
11 #include "content/public/browser/browser_main_runner.h" 13 #include "content/public/browser/browser_main_runner.h"
12 14
13 // ChromeMainDelegateAndroid is created when the library is loaded. It is always 15 // ChromeMainDelegateAndroid is created when the library is loaded. It is always
14 // done in the process's main Java thread. But for non browser process, e.g. 16 // done in the process's main Java thread. But for non browser process, e.g.
15 // renderer process, it is not the native Chrome's main thread. 17 // renderer process, it is not the native Chrome's main thread.
16 ChromeMainDelegateAndroid::ChromeMainDelegateAndroid() { 18 ChromeMainDelegateAndroid::ChromeMainDelegateAndroid() {
17 } 19 }
18 20
19 ChromeMainDelegateAndroid::~ChromeMainDelegateAndroid() { 21 ChromeMainDelegateAndroid::~ChromeMainDelegateAndroid() {
20 } 22 }
21 23
22 void ChromeMainDelegateAndroid::SandboxInitialized( 24 void ChromeMainDelegateAndroid::SandboxInitialized(
23 const std::string& process_type) { 25 const std::string& process_type) {
24 ChromeMainDelegate::SandboxInitialized(process_type); 26 ChromeMainDelegate::SandboxInitialized(process_type);
25 } 27 }
26 28
27 int ChromeMainDelegateAndroid::RunProcess( 29 int ChromeMainDelegateAndroid::RunProcess(
28 const std::string& process_type, 30 const std::string& process_type,
29 const content::MainFunctionParams& main_function_params) { 31 const content::MainFunctionParams& main_function_params) {
30 TRACE_EVENT0("startup", "ChromeMainDelegateAndroid::RunProcess") 32 TRACE_EVENT0("startup", "ChromeMainDelegateAndroid::RunProcess")
31 if (process_type.empty()) { 33 if (process_type.empty()) {
32 JNIEnv* env = base::android::AttachCurrentThread(); 34 JNIEnv* env = base::android::AttachCurrentThread();
33 RegisterApplicationNativeMethods(env); 35 RegisterApplicationNativeMethods(env);
34 36
35 // Because the browser process can be started asynchronously as a series of 37 // Because the browser process can be started asynchronously as a series of
36 // of UI thread tasks a second request to start it can come in while the 38 // UI thread tasks a second request to start it can come in while the
37 // first request is still being processed. We must keep the same 39 // first request is still being processed. Chrome must keep the same
38 // browser runner for the second request. 40 // browser runner for the second request.
41 // Also only record the start time the first time round, since this is the
42 // start time of the application, and will be same for all requests.
39 if (!browser_runner_.get()) { 43 if (!browser_runner_.get()) {
44 base::Time startTime = chrome::android::GetMainEntryPointTime();
45 startup_metric_utils::RecordSavedMainEntryPointTime(startTime);
40 browser_runner_.reset(content::BrowserMainRunner::Create()); 46 browser_runner_.reset(content::BrowserMainRunner::Create());
41 } 47 }
42 return browser_runner_->Initialize(main_function_params); 48 return browser_runner_->Initialize(main_function_params);
43 } 49 }
44 50
45 return ChromeMainDelegate::RunProcess(process_type, main_function_params); 51 return ChromeMainDelegate::RunProcess(process_type, main_function_params);
46 } 52 }
47 53
48 bool ChromeMainDelegateAndroid::BasicStartupComplete(int* exit_code) { 54 bool ChromeMainDelegateAndroid::BasicStartupComplete(int* exit_code) {
49 SetChromeSpecificCommandLineFlags(); 55 SetChromeSpecificCommandLineFlags();
50 return ChromeMainDelegate::BasicStartupComplete(exit_code); 56 return ChromeMainDelegate::BasicStartupComplete(exit_code);
51 } 57 }
52 58
53 bool ChromeMainDelegateAndroid::RegisterApplicationNativeMethods(JNIEnv* env) { 59 bool ChromeMainDelegateAndroid::RegisterApplicationNativeMethods(JNIEnv* env) {
54 return chrome::android::RegisterJni(env); 60 return chrome::android::RegisterJni(env);
55 } 61 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698