Add unit test for WebRTCInternalsMessageHandler.

content/browser/media/webrtc/webrtc_internals.h

 // Copyright (c) 2013 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_BROWSER_MEDIA_WEBRTC_WEBRTC_INTERNALS_H_
 #define CONTENT_BROWSER_MEDIA_WEBRTC_WEBRTC_INTERNALS_H_
 
 #include <memory>
 #include <queue>
 
@@ skipping 96 matching lines @@
 
   bool IsEventLogRecordingsEnabled() const;
   const base::FilePath& GetEventLogFilePath() const;
 
  protected:
   // Constructor/Destructor are protected to allow tests to derive from the
   // class and do per-instance testing without having to use the global
   // instance.
   // The default ctor sets |aggregate_updates_ms| to 500ms.
   WebRTCInternals();
-  explicit WebRTCInternals(int aggregate_updates_ms);
+  explicit WebRTCInternals(int aggregate_updates_ms,
+                           bool should_block_power_saving);

[tommi (sloooow) - chröme, 2016/07/25 10:16:06]

remove 'explicit' (only used for a single argument ctor)

[Max Morin, 2016/07/25 10:41:18]

Done.

   ~WebRTCInternals() override;
 
  private:
   friend struct base::DefaultLazyInstanceTraits<WebRTCInternals>;
   FRIEND_TEST_ALL_PREFIXES(WebRtcAudioDebugRecordingsBrowserTest,
                            CallWithAudioDebugRecordings);
   FRIEND_TEST_ALL_PREFIXES(WebRtcAudioDebugRecordingsBrowserTest,
                            CallWithAudioDebugRecordingsEnabledThenDisabled);
   FRIEND_TEST_ALL_PREFIXES(WebRtcAudioDebugRecordingsBrowserTest,
                            TwoCallsWithAudioDebugRecordings);
@@ skipping 22 matching lines @@
 
   // Enables event log recordings on all render process hosts using
   // |event_log_recordings_file_path_|.
   void EnableEventLogRecordingsOnAllRenderProcessHosts();
 #endif
 
   // Called whenever an element is added to or removed from
   // |peer_connection_data_| to impose/release a block on suspending the current
   // application for power-saving.
   void CreateOrReleasePowerSaveBlocker();
+  const bool should_block_power_saving_;

[tommi (sloooow) - chröme, 2016/07/25 10:16:06]

keep member variables together and at the bottom of the class

[Max Morin, 2016/07/25 10:41:18]

Done.

 
   // Called on a timer to deliver updates to javascript.
   // We throttle and bulk together updates to avoid DOS like scenarios where
   // a page uses a lot of peerconnection instances with many event
   // notifications.
   void ProcessPendingUpdates();
 
   base::ObserverList<WebRTCInternalsUIObserver> observers_;
 
   // |peer_connection_data_| is a list containing all the PeerConnection
@@ skipping 69 matching lines @@
   std::queue<PendingUpdate> pending_updates_;
   const int aggregate_updates_ms_;
 
   // Weak factory for this object that we use for bulking up updates.
   base::WeakPtrFactory<WebRTCInternals> weak_factory_;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_MEDIA_WEBRTC_WEBRTC_INTERNALS_H_