michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 1 | // Copyright 2015 The Chromium Authors. All rights reserved. |
| 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
| 5 | #include "chrome/browser/after_startup_task_utils.h" |
| 6 | |
dcheng | 4af4858 | 2016-04-19 00:29:35 | [diff] [blame] | 7 | #include <memory> |
dcheng | e73d8520c | 2015-12-27 01:19:09 | [diff] [blame] | 8 | #include <utility> |
| 9 | |
Brett Wilson | 275a137 | 2017-09-01 20:27:54 | [diff] [blame] | 10 | #include "base/containers/circular_deque.h" |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 11 | #include "base/lazy_instance.h" |
avi | e4d7b6f | 2015-12-26 00:59:18 | [diff] [blame] | 12 | #include "base/macros.h" |
dcheng | 4af4858 | 2016-04-19 00:29:35 | [diff] [blame] | 13 | #include "base/memory/ptr_util.h" |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 14 | #include "base/metrics/histogram_macros.h" |
Francois Doray | 1456375 | 2018-10-23 14:15:57 | [diff] [blame] | 15 | #include "base/process/process.h" |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 16 | #include "base/rand_util.h" |
gab | 25894fe | 2017-05-30 03:40:36 | [diff] [blame] | 17 | #include "base/sequence_checker.h" |
Gabriel Charette | e926fc1 | 2019-12-16 19:00:02 | [diff] [blame] | 18 | #include "base/sequenced_task_runner.h" |
fdoray | ef19112 | 2016-07-25 14:43:17 | [diff] [blame] | 19 | #include "base/synchronization/atomic_flag.h" |
avi | e4d7b6f | 2015-12-26 00:59:18 | [diff] [blame] | 20 | #include "build/build_config.h" |
Yuta Hijikata | 235fc62b | 2020-12-08 03:48:32 | [diff] [blame^] | 21 | #include "build/chromeos_buildflags.h" |
Eric Seckler | 8652dcd5 | 2018-09-20 10:42:28 | [diff] [blame] | 22 | #include "content/public/browser/browser_task_traits.h" |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 23 | #include "content/public/browser/browser_thread.h" |
Eric Lawrence | 45eee3a | 2019-10-14 22:46:36 | [diff] [blame] | 24 | #include "content/public/browser/navigation_handle.h" |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 25 | #include "content/public/browser/render_frame_host.h" |
| 26 | #include "content/public/browser/web_contents.h" |
| 27 | #include "content/public/browser/web_contents_observer.h" |
Collin Baker | 98457b5 | 2019-11-06 21:34:29 | [diff] [blame] | 28 | #include "content/public/common/page_visibility_state.h" |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 29 | |
Collin Baker | 8a21755 | 2019-05-29 19:47:51 | [diff] [blame] | 30 | #if !defined(OS_ANDROID) |
| 31 | #include "chrome/browser/ui/browser.h" |
| 32 | #include "chrome/browser/ui/browser_list.h" |
| 33 | #include "chrome/browser/ui/tabs/tab_strip_model.h" |
| 34 | #endif |
| 35 | |
Yuta Hijikata | 235fc62b | 2020-12-08 03:48:32 | [diff] [blame^] | 36 | // TODO(crbug.com/1052397): Revisit the macro expression once build flag switch |
| 37 | // of lacros-chrome is complete. |
| 38 | #if defined(OS_LINUX) || BUILDFLAG(IS_CHROMEOS_LACROS) |
Mario Sanchez Prada | 0a85ddf | 2018-06-07 20:32:59 | [diff] [blame] | 39 | #include "ui/views/linux_ui/linux_ui.h" |
| 40 | #endif |
| 41 | |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 42 | using content::BrowserThread; |
| 43 | using content::WebContents; |
| 44 | using content::WebContentsObserver; |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 45 | |
| 46 | namespace { |
| 47 | |
| 48 | struct AfterStartupTask { |
Brett Wilson | e1a7042 | 2017-09-12 05:10:09 | [diff] [blame] | 49 | AfterStartupTask(const base::Location& from_here, |
Gabriel Charette | e926fc1 | 2019-12-16 19:00:02 | [diff] [blame] | 50 | const scoped_refptr<base::SequencedTaskRunner>& task_runner, |
tzik | 6e42784 | 2017-04-05 10:13:21 | [diff] [blame] | 51 | base::OnceClosure task) |
tzik | 070c8ffb | 2017-03-29 05:28:12 | [diff] [blame] | 52 | : from_here(from_here), task_runner(task_runner), task(std::move(task)) {} |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 53 | ~AfterStartupTask() {} |
| 54 | |
Brett Wilson | e1a7042 | 2017-09-12 05:10:09 | [diff] [blame] | 55 | const base::Location from_here; |
Gabriel Charette | e926fc1 | 2019-12-16 19:00:02 | [diff] [blame] | 56 | const scoped_refptr<base::SequencedTaskRunner> task_runner; |
tzik | 6e42784 | 2017-04-05 10:13:21 | [diff] [blame] | 57 | base::OnceClosure task; |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 58 | }; |
| 59 | |
| 60 | // The flag may be read on any thread, but must only be set on the UI thread. |
fdoray | ef19112 | 2016-07-25 14:43:17 | [diff] [blame] | 61 | base::LazyInstance<base::AtomicFlag>::Leaky g_startup_complete_flag; |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 62 | |
| 63 | // The queue may only be accessed on the UI thread. |
Brett Wilson | 275a137 | 2017-09-01 20:27:54 | [diff] [blame] | 64 | base::LazyInstance<base::circular_deque<AfterStartupTask*>>::Leaky |
| 65 | g_after_startup_tasks; |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 66 | |
Lei Zhang | f713c64 | 2019-01-11 19:19:34 | [diff] [blame] | 67 | bool g_schedule_tasks_with_delay = true; |
| 68 | |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 69 | bool IsBrowserStartupComplete() { |
| 70 | // Be sure to initialize the LazyInstance on the main thread since the flag |
| 71 | // may only be set on it's initializing thread. |
Lukasz Anforowicz | d3e1913 | 2017-12-06 19:44:27 | [diff] [blame] | 72 | if (!g_startup_complete_flag.IsCreated()) |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 73 | return false; |
| 74 | return g_startup_complete_flag.Get().IsSet(); |
| 75 | } |
| 76 | |
dcheng | 4af4858 | 2016-04-19 00:29:35 | [diff] [blame] | 77 | void RunTask(std::unique_ptr<AfterStartupTask> queued_task) { |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 78 | // We're careful to delete the caller's |task| on the target runner's thread. |
peary2 | be58808 | 2017-05-17 01:59:49 | [diff] [blame] | 79 | DCHECK(queued_task->task_runner->RunsTasksInCurrentSequence()); |
tzik | 070c8ffb | 2017-03-29 05:28:12 | [diff] [blame] | 80 | std::move(queued_task->task).Run(); |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 81 | } |
| 82 | |
dcheng | 4af4858 | 2016-04-19 00:29:35 | [diff] [blame] | 83 | void ScheduleTask(std::unique_ptr<AfterStartupTask> queued_task) { |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 84 | // Spread their execution over a brief time. |
Lei Zhang | f713c64 | 2019-01-11 19:19:34 | [diff] [blame] | 85 | constexpr int kMinDelaySec = 0; |
| 86 | constexpr int kMaxDelaySec = 10; |
Gabriel Charette | e926fc1 | 2019-12-16 19:00:02 | [diff] [blame] | 87 | scoped_refptr<base::SequencedTaskRunner> target_runner = |
| 88 | queued_task->task_runner; |
Brett Wilson | e1a7042 | 2017-09-12 05:10:09 | [diff] [blame] | 89 | base::Location from_here = queued_task->from_here; |
Lei Zhang | f713c64 | 2019-01-11 19:19:34 | [diff] [blame] | 90 | int delay_in_seconds = g_schedule_tasks_with_delay |
| 91 | ? base::RandInt(kMinDelaySec, kMaxDelaySec) |
| 92 | : 0; |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 93 | target_runner->PostDelayedTask( |
tzik | 6d3cd756 | 2018-02-21 12:07:22 | [diff] [blame] | 94 | from_here, base::BindOnce(&RunTask, std::move(queued_task)), |
Lei Zhang | f713c64 | 2019-01-11 19:19:34 | [diff] [blame] | 95 | base::TimeDelta::FromSeconds(delay_in_seconds)); |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 96 | } |
| 97 | |
dcheng | 4af4858 | 2016-04-19 00:29:35 | [diff] [blame] | 98 | void QueueTask(std::unique_ptr<AfterStartupTask> queued_task) { |
tzik | c697696 | 2017-04-04 17:27:34 | [diff] [blame] | 99 | DCHECK(queued_task); |
tzik | 498d42b | 2017-04-13 07:42:48 | [diff] [blame] | 100 | |
| 101 | // Use CHECK instead of DCHECK to crash earlier. See http://crbug.com/711167 |
| 102 | // for details. |
| 103 | CHECK(queued_task->task); |
| 104 | |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 105 | if (!BrowserThread::CurrentlyOn(BrowserThread::UI)) { |
Eric Seckler | 0618f40 | 2018-10-29 12:08:52 | [diff] [blame] | 106 | // Posted with USER_VISIBLE priority to avoid this becoming an after startup |
| 107 | // task itself. |
Gabriel Charette | e7cdc5cd | 2020-05-27 23:35:05 | [diff] [blame] | 108 | content::GetUIThreadTaskRunner({base::TaskPriority::USER_VISIBLE}) |
| 109 | ->PostTask(FROM_HERE, |
Sami Kyostila | 7d640eb | 2019-07-31 18:50:26 | [diff] [blame] | 110 | base::BindOnce(QueueTask, std::move(queued_task))); |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 111 | return; |
| 112 | } |
| 113 | |
| 114 | // The flag may have been set while the task to invoke this method |
| 115 | // on the UI thread was inflight. |
| 116 | if (IsBrowserStartupComplete()) { |
dcheng | e73d8520c | 2015-12-27 01:19:09 | [diff] [blame] | 117 | ScheduleTask(std::move(queued_task)); |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 118 | return; |
| 119 | } |
| 120 | g_after_startup_tasks.Get().push_back(queued_task.release()); |
| 121 | } |
| 122 | |
| 123 | void SetBrowserStartupIsComplete() { |
| 124 | DCHECK_CURRENTLY_ON(BrowserThread::UI); |
Sean McAllister | df80771 | 2020-08-13 23:19:09 | [diff] [blame] | 125 | #if defined(OS_MAC) || defined(OS_WIN) || defined(OS_LINUX) || \ |
| 126 | defined(OS_CHROMEOS) |
Francois Doray | 1456375 | 2018-10-23 14:15:57 | [diff] [blame] | 127 | // Process::Current().CreationTime() is not available on all platforms. |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 128 | const base::Time process_creation_time = |
Francois Doray | 1456375 | 2018-10-23 14:15:57 | [diff] [blame] | 129 | base::Process::Current().CreationTime(); |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 130 | if (!process_creation_time.is_null()) { |
| 131 | UMA_HISTOGRAM_LONG_TIMES("Startup.AfterStartupTaskDelayedUntilTime", |
| 132 | base::Time::Now() - process_creation_time); |
| 133 | } |
Sean McAllister | df80771 | 2020-08-13 23:19:09 | [diff] [blame] | 134 | #endif // defined(OS_MAC) || defined(OS_WIN) || defined(OS_LINUX) || |
| 135 | // defined(OS_CHROMEOS) |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 136 | UMA_HISTOGRAM_COUNTS_10000("Startup.AfterStartupTaskCount", |
| 137 | g_after_startup_tasks.Get().size()); |
| 138 | g_startup_complete_flag.Get().Set(); |
| 139 | for (AfterStartupTask* queued_task : g_after_startup_tasks.Get()) |
dcheng | 4af4858 | 2016-04-19 00:29:35 | [diff] [blame] | 140 | ScheduleTask(base::WrapUnique(queued_task)); |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 141 | g_after_startup_tasks.Get().clear(); |
Lei Zhang | 3b9950b | 2018-12-21 22:24:51 | [diff] [blame] | 142 | g_after_startup_tasks.Get().shrink_to_fit(); |
Mario Sanchez Prada | 0a85ddf | 2018-06-07 20:32:59 | [diff] [blame] | 143 | |
Yuta Hijikata | 235fc62b | 2020-12-08 03:48:32 | [diff] [blame^] | 144 | // TODO(crbug.com/1052397): Revisit the macro expression once build flag switch |
| 145 | // of lacros-chrome is complete. |
| 146 | #if defined(OS_LINUX) || BUILDFLAG(IS_CHROMEOS_LACROS) |
Mario Sanchez Prada | 0a85ddf | 2018-06-07 20:32:59 | [diff] [blame] | 147 | // Make sure we complete the startup notification sequence, or launchers will |
| 148 | // get confused by not receiving the expected message from the main process. |
| 149 | views::LinuxUI* linux_ui = views::LinuxUI::instance(); |
| 150 | if (linux_ui) |
| 151 | linux_ui->NotifyWindowManagerStartupComplete(); |
| 152 | #endif |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 153 | } |
| 154 | |
| 155 | // Observes the first visible page load and sets the startup complete |
| 156 | // flag accordingly. |
gab | 25894fe | 2017-05-30 03:40:36 | [diff] [blame] | 157 | class StartupObserver : public WebContentsObserver { |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 158 | public: |
Jeremy Roman | 495db68 | 2019-07-12 16:03:24 | [diff] [blame] | 159 | StartupObserver() {} |
gab | 25894fe | 2017-05-30 03:40:36 | [diff] [blame] | 160 | ~StartupObserver() override { |
| 161 | DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_); |
| 162 | DCHECK(IsBrowserStartupComplete()); |
| 163 | } |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 164 | |
| 165 | void Start(); |
| 166 | |
| 167 | private: |
| 168 | void OnStartupComplete() { |
gab | 25894fe | 2017-05-30 03:40:36 | [diff] [blame] | 169 | DCHECK_CALLED_ON_VALID_SEQUENCE(sequence_checker_); |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 170 | SetBrowserStartupIsComplete(); |
| 171 | delete this; |
| 172 | } |
| 173 | |
| 174 | void OnFailsafeTimeout() { OnStartupComplete(); } |
| 175 | |
| 176 | // WebContentsObserver overrides |
| 177 | void DidFinishLoad(content::RenderFrameHost* render_frame_host, |
| 178 | const GURL& validated_url) override { |
| 179 | if (!render_frame_host->GetParent()) |
| 180 | OnStartupComplete(); |
| 181 | } |
| 182 | |
| 183 | void DidFailLoad(content::RenderFrameHost* render_frame_host, |
| 184 | const GURL& validated_url, |
Dave Tapuska | 924ef3c | 2020-01-22 18:20:59 | [diff] [blame] | 185 | int error_code) override { |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 186 | if (!render_frame_host->GetParent()) |
| 187 | OnStartupComplete(); |
| 188 | } |
| 189 | |
Eric Lawrence | 45eee3a | 2019-10-14 22:46:36 | [diff] [blame] | 190 | // Starting the browser with a file download url will not result in |
| 191 | // DidFinishLoad firing, so watch for this case too. crbug.com/1006954 |
| 192 | void DidFinishNavigation( |
| 193 | content::NavigationHandle* navigation_handle) override { |
| 194 | if (navigation_handle->IsInMainFrame() && navigation_handle->IsDownload()) |
| 195 | OnStartupComplete(); |
| 196 | } |
| 197 | |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 198 | void WebContentsDestroyed() override { OnStartupComplete(); } |
| 199 | |
gab | 25894fe | 2017-05-30 03:40:36 | [diff] [blame] | 200 | SEQUENCE_CHECKER(sequence_checker_); |
| 201 | |
Jeremy Roman | 495db68 | 2019-07-12 16:03:24 | [diff] [blame] | 202 | base::WeakPtrFactory<StartupObserver> weak_factory_{this}; |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 203 | |
| 204 | DISALLOW_COPY_AND_ASSIGN(StartupObserver); |
| 205 | }; |
| 206 | |
| 207 | void StartupObserver::Start() { |
| 208 | // Signal completion quickly when there is no first page to load. |
| 209 | const int kShortDelaySecs = 3; |
| 210 | base::TimeDelta delay = base::TimeDelta::FromSeconds(kShortDelaySecs); |
| 211 | |
| 212 | #if !defined(OS_ANDROID) |
| 213 | WebContents* contents = nullptr; |
scottmg | 8abbff83 | 2016-01-28 22:57:37 | [diff] [blame] | 214 | for (auto* browser : *BrowserList::GetInstance()) { |
| 215 | contents = browser->tab_strip_model()->GetActiveWebContents(); |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 216 | if (contents && contents->GetMainFrame() && |
| 217 | contents->GetMainFrame()->GetVisibilityState() == |
danakj | 0018a29a | 2018-12-01 01:03:43 | [diff] [blame] | 218 | content::PageVisibilityState::kVisible) { |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 219 | break; |
| 220 | } |
| 221 | } |
| 222 | |
| 223 | if (contents) { |
| 224 | // Give the page time to finish loading. |
| 225 | const int kLongerDelayMins = 3; |
| 226 | Observe(contents); |
| 227 | delay = base::TimeDelta::FromMinutes(kLongerDelayMins); |
| 228 | } |
| 229 | #else |
michaeln | 68bf4a8e | 2015-08-11 01:37:31 | [diff] [blame] | 230 | // Startup completion is signaled via AfterStartupTaskUtils.java, |
| 231 | // this is just a failsafe timeout. |
| 232 | const int kLongerDelayMins = 3; |
| 233 | delay = base::TimeDelta::FromMinutes(kLongerDelayMins); |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 234 | #endif // !defined(OS_ANDROID) |
| 235 | |
Gabriel Charette | e7cdc5cd | 2020-05-27 23:35:05 | [diff] [blame] | 236 | content::GetUIThreadTaskRunner({})->PostDelayedTask( |
| 237 | FROM_HERE, |
| 238 | base::BindOnce(&StartupObserver::OnFailsafeTimeout, |
| 239 | weak_factory_.GetWeakPtr()), |
| 240 | delay); |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 241 | } |
| 242 | |
| 243 | } // namespace |
| 244 | |
| 245 | void AfterStartupTaskUtils::StartMonitoringStartup() { |
| 246 | // The observer is self-deleting. |
| 247 | (new StartupObserver)->Start(); |
| 248 | } |
| 249 | |
| 250 | void AfterStartupTaskUtils::PostTask( |
Brett Wilson | e1a7042 | 2017-09-12 05:10:09 | [diff] [blame] | 251 | const base::Location& from_here, |
Gabriel Charette | e926fc1 | 2019-12-16 19:00:02 | [diff] [blame] | 252 | const scoped_refptr<base::SequencedTaskRunner>& destination_runner, |
tzik | 6e42784 | 2017-04-05 10:13:21 | [diff] [blame] | 253 | base::OnceClosure task) { |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 254 | if (IsBrowserStartupComplete()) { |
tzik | 070c8ffb | 2017-03-29 05:28:12 | [diff] [blame] | 255 | destination_runner->PostTask(from_here, std::move(task)); |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 256 | return; |
| 257 | } |
| 258 | |
dcheng | 4af4858 | 2016-04-19 00:29:35 | [diff] [blame] | 259 | std::unique_ptr<AfterStartupTask> queued_task( |
tzik | 070c8ffb | 2017-03-29 05:28:12 | [diff] [blame] | 260 | new AfterStartupTask(from_here, destination_runner, std::move(task))); |
dcheng | e73d8520c | 2015-12-27 01:19:09 | [diff] [blame] | 261 | QueueTask(std::move(queued_task)); |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 262 | } |
| 263 | |
wkorman | 8a21c4f | 2015-11-18 19:06:11 | [diff] [blame] | 264 | void AfterStartupTaskUtils::SetBrowserStartupIsCompleteForTesting() { |
| 265 | ::SetBrowserStartupIsComplete(); |
| 266 | } |
| 267 | |
michaeln | 96f887e2 | 2015-04-13 23:58:31 | [diff] [blame] | 268 | void AfterStartupTaskUtils::SetBrowserStartupIsComplete() { |
| 269 | ::SetBrowserStartupIsComplete(); |
| 270 | } |
| 271 | |
| 272 | bool AfterStartupTaskUtils::IsBrowserStartupComplete() { |
| 273 | return ::IsBrowserStartupComplete(); |
| 274 | } |
| 275 | |
| 276 | void AfterStartupTaskUtils::UnsafeResetForTesting() { |
| 277 | DCHECK(g_after_startup_tasks.Get().empty()); |
| 278 | if (!IsBrowserStartupComplete()) |
| 279 | return; |
| 280 | g_startup_complete_flag.Get().UnsafeResetForTesting(); |
| 281 | DCHECK(!IsBrowserStartupComplete()); |
| 282 | } |
Lei Zhang | f713c64 | 2019-01-11 19:19:34 | [diff] [blame] | 283 | |
| 284 | // static |
| 285 | void AfterStartupTaskUtils::DisableScheduleTaskDelayForTesting() { |
| 286 | g_schedule_tasks_with_delay = false; |
| 287 | } |