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

Unified Diff: chrome/renderer/page_load_metrics/page_timing_metrics_sender.h

Issue 2874663005: [Page Load Metrics] Add mojom file to page load metrics. (Closed)
Patch Set: Remove unnecessary variable Created 3 years, 7 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 side-by-side diff with in-line comments
Download patch
Index: chrome/renderer/page_load_metrics/page_timing_metrics_sender.h
diff --git a/chrome/renderer/page_load_metrics/page_timing_metrics_sender.h b/chrome/renderer/page_load_metrics/page_timing_metrics_sender.h
index 75342f310678bd923296f18bc2a5836b8464240f..3f3d2f50567489ecc21868b273716b118f890a21 100644
--- a/chrome/renderer/page_load_metrics/page_timing_metrics_sender.h
+++ b/chrome/renderer/page_load_metrics/page_timing_metrics_sender.h
@@ -29,11 +29,11 @@ class PageTimingMetricsSender {
PageTimingMetricsSender(IPC::Sender* ipc_sender,
int routing_id,
std::unique_ptr<base::Timer> timer,
- const PageLoadTiming& initial_timing);
+ mojom::PageLoadTimingPtr initial_timing);
~PageTimingMetricsSender();
void DidObserveLoadingBehavior(blink::WebLoadingBehaviorFlag behavior);
- void Send(const PageLoadTiming& timing);
+ void Send(mojom::PageLoadTimingPtr timing);
protected:
base::Timer* timer() const { return timer_.get(); }
@@ -45,11 +45,11 @@ class PageTimingMetricsSender {
IPC::Sender* const ipc_sender_;
const int routing_id_;
std::unique_ptr<base::Timer> timer_;
- PageLoadTiming last_timing_;
+ mojom::PageLoadTimingPtr last_timing_;
// The the sender keep track of metadata as it comes in, because the sender is
// scoped to a single committed load.
- PageLoadMetadata metadata_;
+ mojom::PageLoadMetadata metadata_;
bool have_sent_ipc_ = false;

Powered by Google App Engine
This is Rietveld 408576698