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

Unified Diff: chrome/renderer/page_load_metrics/metrics_render_frame_observer_unittest.cc

Issue 2823523003: [Page Load Metrics] PageLoadMetrics Mojofication. (Closed)
Patch Set: rebase 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/metrics_render_frame_observer_unittest.cc
diff --git a/chrome/renderer/page_load_metrics/metrics_render_frame_observer_unittest.cc b/chrome/renderer/page_load_metrics/metrics_render_frame_observer_unittest.cc
index 5ad379d051e1237005e939f827479eda387f7b29..90f0859832a3c1bd7155348bdc4b4a8a52c4ed76 100644
--- a/chrome/renderer/page_load_metrics/metrics_render_frame_observer_unittest.cc
+++ b/chrome/renderer/page_load_metrics/metrics_render_frame_observer_unittest.cc
@@ -11,7 +11,7 @@
#include "base/time/time.h"
#include "base/timer/mock_timer.h"
#include "chrome/common/page_load_metrics/page_load_timing.h"
-#include "chrome/renderer/page_load_metrics/fake_page_timing_metrics_ipc_sender.h"
+#include "chrome/renderer/page_load_metrics/fake_page_timing_sender.h"
#include "testing/gtest/include/gtest/gtest.h"
namespace page_load_metrics {
@@ -29,11 +29,9 @@ class TestMetricsRenderFrameObserver : public MetricsRenderFrameObserver {
return std::move(mock_timer_);
}
- // We intercept sent messages and dispatch them to our
- // FakePageTimingMetricsIPCSender, which we use to verify that the expected
- // IPC messages get sent.
- bool Send(IPC::Message* message) override {
- return fake_timing_ipc_sender_.Send(message);
+ std::unique_ptr<PageTimingSender> CreatePageTimingSender() override {
+ return base::WrapUnique<PageTimingSender>(
+ new FakePageTimingSender(&validator_));
}
void set_mock_timer(std::unique_ptr<base::Timer> timer) {
@@ -43,7 +41,7 @@ class TestMetricsRenderFrameObserver : public MetricsRenderFrameObserver {
void ExpectPageLoadTiming(const mojom::PageLoadTiming& timing) {
SetFakePageLoadTiming(timing);
- fake_timing_ipc_sender_.ExpectPageLoadTiming(timing);
+ validator_.ExpectPageLoadTiming(timing);
}
void SetFakePageLoadTiming(const mojom::PageLoadTiming& timing) {
@@ -58,14 +56,14 @@ class TestMetricsRenderFrameObserver : public MetricsRenderFrameObserver {
void VerifyExpectedTimings() const {
EXPECT_EQ(nullptr, fake_timing_.get());
- fake_timing_ipc_sender_.VerifyExpectedTimings();
+ validator_.VerifyExpectedTimings();
}
bool ShouldSendMetrics() const override { return true; }
bool HasNoRenderFrame() const override { return false; }
private:
- FakePageTimingMetricsIPCSender fake_timing_ipc_sender_;
+ FakePageTimingSender::PageTimingValidator validator_;
mutable mojom::PageLoadTimingPtr fake_timing_;
mutable std::unique_ptr<base::Timer> mock_timer_;
};

Powered by Google App Engine
This is Rietveld 408576698