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

Unified Diff: content/browser/media/webrtc/webrtc_internals.h

Issue 2163983006: Add unit test for WebRTCInternalsMessageHandler. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add unit test for WebRTCInternalsMessageHandler. Created 4 years, 5 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: content/browser/media/webrtc/webrtc_internals.h
diff --git a/content/browser/media/webrtc/webrtc_internals.h b/content/browser/media/webrtc/webrtc_internals.h
index 35a1d897f070249d8279c411adf37d55799c99d4..ebc887521c35c44b87b8087de7d57756ceb56604 100644
--- a/content/browser/media/webrtc/webrtc_internals.h
+++ b/content/browser/media/webrtc/webrtc_internals.h
@@ -114,7 +114,8 @@ class CONTENT_EXPORT WebRTCInternals : public RenderProcessHostObserver,
// 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
Max Morin 2016/07/25 10:41:18 Done.
~WebRTCInternals() override;
private:
@@ -157,6 +158,7 @@ class CONTENT_EXPORT WebRTCInternals : public RenderProcessHostObserver,
// |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 o
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

Powered by Google App Engine
This is Rietveld 408576698