Simplify/Cleanup MediaClient

content/renderer/media/render_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 CONTENT_RENDERER_MEDIA_RENDER_MEDIA_LOG_H_
 #define CONTENT_RENDERER_MEDIA_RENDER_MEDIA_LOG_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 21 matching lines @@
 //
 // It must be constructed on the render thread.
 class CONTENT_EXPORT RenderMediaLog : public media::MediaLog {
  public:
   explicit RenderMediaLog(const GURL& security_origin);
 
   // MediaLog implementation.
   void AddEvent(std::unique_ptr<media::MediaLogEvent> event) override;
   std::string GetLastErrorMessage() override;
   void RecordRapporWithSecurityOrigin(const std::string& metric) override;
+  void RecordRapporWithURL(const std::string& metric, const GURL& url) override;

[xhwang, 2017/04/03 17:47:14]

Please add a TODO to replace all usage of this one with
RecordRapporWithSecurityOrigin(), and then remove this method.

[chcunningham, 2017/04/03 18:47:39]

Implemented the TODO

 
   // Will reset |last_ipc_send_time_| with the value of NowTicks().
   void SetTickClockForTesting(std::unique_ptr<base::TickClock> tick_clock);
   void SetTaskRunnerForTesting(
       const scoped_refptr<base::SingleThreadTaskRunner>& task_runner);
 
  private:
   ~RenderMediaLog() override;
 
   // Posted as a delayed task on |task_runner_| to throttle ipc message
@@ skipping 26 matching lines @@
 
   // Holds a copy of the most recent PIPELINE_ERROR, if any.
   std::unique_ptr<media::MediaLogEvent> last_pipeline_error_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderMediaLog);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_MEDIA_RENDER_MEDIA_LOG_H_