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

Unified Diff: chromeos/dbus/fake_upstart_client.h

Issue 2858353002: MediaPerceptionPrivate API impl and testing. (Closed)
Patch Set: MediaPerceptionAPIManagerTest working. 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: chromeos/dbus/fake_upstart_client.h
diff --git a/chromeos/dbus/fake_upstart_client.h b/chromeos/dbus/fake_upstart_client.h
index efcb5e6a6c1344b3e19a417dd0bd3e667b7c182e..6e5de7b8366b95c439a12d5a7e913f4463ffe725 100644
--- a/chromeos/dbus/fake_upstart_client.h
+++ b/chromeos/dbus/fake_upstart_client.h
@@ -25,7 +25,13 @@ class CHROMEOS_EXPORT FakeUpstartClient : public UpstartClient {
void StartMediaAnalytics(const UpstartCallback& callback) override;
void StopMediaAnalytics() override;
+ void SetStartMediaAnalyticsWillSucceed(bool will_succeed);
+
private:
+ // Sets whether the StartMediaAnalytics method will succeed. Used for testing
+ // upstart failure case.
tbarzic 2017/05/16 20:56:30 no need for the "Used for testing" comment.
Luke Sorenson 2017/05/16 21:33:33 Done.
+ bool start_media_analytics_will_succeed_;
+
DISALLOW_COPY_AND_ASSIGN(FakeUpstartClient);
};

Powered by Google App Engine
This is Rietveld 408576698