media: Add Rappor reporting in MediaLog

media/base/media_log.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef MEDIA_BASE_MEDIA_LOG_H_
 #define MEDIA_BASE_MEDIA_LOG_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 29 matching lines @@
 
   MediaLog();
 
   // Add an event to this log. Overriden by inheritors to actually do something
   // with it.
   virtual void AddEvent(std::unique_ptr<MediaLogEvent> event);
 
   // 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 existing rappor metrics. We can
also report the WebMediaPlayerSource URL, but I am not sure how useful that will
be.

[DaleCurtis, 2016/06/15 00:33:47]

I think this is better for now, as the player url is often a cdn redirect which
may not be useful in tracking down the main page.

+
   // Helper methods to create events and their parameters.
   std::unique_ptr<MediaLogEvent> CreateEvent(MediaLogEvent::Type type);
   std::unique_ptr<MediaLogEvent> CreateBooleanEvent(MediaLogEvent::Type type,
                                                     const std::string& property,
                                                     bool value);
   std::unique_ptr<MediaLogEvent> CreateStringEvent(MediaLogEvent::Type type,
                                                    const std::string& property,
                                                    const std::string& value);
   std::unique_ptr<MediaLogEvent> CreateTimeEvent(MediaLogEvent::Type type,
                                                  const std::string& property,
@@ skipping 65 matching lines @@
 #define LIMITED_MEDIA_LOG(level, media_log, count, max)                       \
   LAZY_STREAM(MEDIA_LOG(level, media_log),                                    \
               (count) < (max) && ((count)++ || true))                         \
       << (((count) == (max)) ? "(Log limit reached. Further similar entries " \
                                "may be suppressed): "                         \
                              : "")
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_MEDIA_LOG_H_