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

Side by Side Diff: components/page_load_metrics/renderer/page_timing_metrics_sender.cc

Issue 2056153002: Convert PageLoadMetrics to Mojo (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 years, 5 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 2015 The Chromium Authors. All rights reserved. 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 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 "components/page_load_metrics/renderer/page_timing_metrics_sender.h" 5 #include "components/page_load_metrics/renderer/page_timing_metrics_sender.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/callback.h" 9 #include "base/callback.h"
10 #include "base/time/time.h" 10 #include "base/time/time.h"
11 #include "base/timer/timer.h" 11 #include "base/timer/timer.h"
12 #include "components/page_load_metrics/common/page_load_metrics_messages.h" 12 #include "components/page_load_metrics/common/page_load_metrics_param_traits.h"
13 #include "ipc/ipc_sender.h"
14 13
15 namespace page_load_metrics { 14 namespace page_load_metrics {
16 15
17 namespace { 16 namespace {
18 const int kInitialTimerDelayMillis = 50; 17 const int kInitialTimerDelayMillis = 50;
19 const int kTimerDelayMillis = 1000; 18 const int kTimerDelayMillis = 1000;
20 } // namespace 19 } // namespace
21 20
22 PageTimingMetricsSender::PageTimingMetricsSender( 21 PageTimingMetricsSender::PageTimingMetricsSender(
23 IPC::Sender* ipc_sender, 22 mojom::PageLoadMetrics* page_load_metrics,
24 int routing_id,
25 std::unique_ptr<base::Timer> timer, 23 std::unique_ptr<base::Timer> timer,
26 const PageLoadTiming& initial_timing) 24 const PageLoadTiming& initial_timing)
27 : ipc_sender_(ipc_sender), 25 : page_load_metrics_(page_load_metrics),
28 routing_id_(routing_id),
29 timer_(std::move(timer)), 26 timer_(std::move(timer)),
30 last_timing_(initial_timing), 27 last_timing_(initial_timing),
31 metadata_(PageLoadMetadata()) { 28 metadata_(PageLoadMetadata()) {
29 DCHECK(page_load_metrics);
32 // Send an initial IPC relatively early to help track aborts. 30 // Send an initial IPC relatively early to help track aborts.
33 EnsureSendTimer(kInitialTimerDelayMillis); 31 EnsureSendTimer(kInitialTimerDelayMillis);
34 } 32 }
35 33
36 // On destruction, we want to send any data we have if we have a timer 34 // On destruction, we want to send any data we have if we have a timer
37 // currently running (and thus are talking to a browser process) 35 // currently running (and thus are talking to a browser process)
38 PageTimingMetricsSender::~PageTimingMetricsSender() { 36 PageTimingMetricsSender::~PageTimingMetricsSender() {
39 if (timer_->IsRunning()) { 37 if (timer_->IsRunning()) {
40 timer_->Stop(); 38 timer_->Stop();
41 SendNow(); 39 SendNow();
(...skipping 25 matching lines...) Expand all
67 } 65 }
68 66
69 void PageTimingMetricsSender::EnsureSendTimer(int delay) { 67 void PageTimingMetricsSender::EnsureSendTimer(int delay) {
70 if (!timer_->IsRunning()) 68 if (!timer_->IsRunning())
71 timer_->Start( 69 timer_->Start(
72 FROM_HERE, base::TimeDelta::FromMilliseconds(delay), 70 FROM_HERE, base::TimeDelta::FromMilliseconds(delay),
73 base::Bind(&PageTimingMetricsSender::SendNow, base::Unretained(this))); 71 base::Bind(&PageTimingMetricsSender::SendNow, base::Unretained(this)));
74 } 72 }
75 73
76 void PageTimingMetricsSender::SendNow() { 74 void PageTimingMetricsSender::SendNow() {
77 ipc_sender_->Send(new PageLoadMetricsMsg_TimingUpdated( 75 page_load_metrics_->TimingUpdated(last_timing_, metadata_);
78 routing_id_, last_timing_, metadata_));
79 } 76 }
80 77
81 } // namespace page_load_metrics 78 } // namespace page_load_metrics
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698