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

Unified Diff: media/base/media_log.h

Issue 2064083002: media: Add Rappor reporting in MediaLog (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: more fixes Created 4 years, 6 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: media/base/media_log.h
diff --git a/media/base/media_log.h b/media/base/media_log.h
index d08e5426ad6ae5912cb98de6d329a40f76902640..64a3f3d4c86578fe5ade5070f883edeb27f7b955 100644
--- a/media/base/media_log.h
+++ b/media/base/media_log.h
@@ -47,6 +47,11 @@ class MEDIA_EXPORT MediaLog : public base::RefCountedThreadSafe<MediaLog> {
// Retrieve an error message, if any.
virtual std::string GetLastErrorMessage();
+ // Records the domain and registry of the current frame security origin to a
+ // Rappor privacy-preserving metric. See:
+ // https://www.chromium.org/developers/design-documents/rappor
+ virtual void RecordRapporWithSecurityOrigin(const std::string& metric);
xhwang 2016/06/15 00:19:07 Reporting security origin is consistent with our e
DaleCurtis 2016/06/15 00:33:47 I think this is better for now, as the player url
+
// Helper methods to create events and their parameters.
std::unique_ptr<MediaLogEvent> CreateEvent(MediaLogEvent::Type type);
std::unique_ptr<MediaLogEvent> CreateBooleanEvent(MediaLogEvent::Type type,

Powered by Google App Engine
This is Rietveld 408576698