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

Side by Side Diff: content/browser/media/webrtc/webrtc_internals.h

Issue 1855193002: Move the call to enable the WebRTC event log from PeerConnectionFactory to PeerConnection. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added unittest for adding more PeerConnections than the maximum limit. 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CONTENT_BROWSER_MEDIA_WEBRTC_WEBRTC_INTERNALS_H_ 5 #ifndef CONTENT_BROWSER_MEDIA_WEBRTC_WEBRTC_INTERNALS_H_
6 #define CONTENT_BROWSER_MEDIA_WEBRTC_WEBRTC_INTERNALS_H_ 6 #define CONTENT_BROWSER_MEDIA_WEBRTC_WEBRTC_INTERNALS_H_
7 7
8 #include <memory> 8 #include <memory>
9 #include <queue> 9 #include <queue>
10 10
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
92 void UpdateObserver(WebRTCInternalsUIObserver* observer); 92 void UpdateObserver(WebRTCInternalsUIObserver* observer);
93 93
94 // Enables or disables diagnostic audio recordings for debugging purposes. 94 // Enables or disables diagnostic audio recordings for debugging purposes.
95 void EnableAudioDebugRecordings(content::WebContents* web_contents); 95 void EnableAudioDebugRecordings(content::WebContents* web_contents);
96 void DisableAudioDebugRecordings(); 96 void DisableAudioDebugRecordings();
97 97
98 bool IsAudioDebugRecordingsEnabled() const; 98 bool IsAudioDebugRecordingsEnabled() const;
99 const base::FilePath& GetAudioDebugRecordingsFilePath() const; 99 const base::FilePath& GetAudioDebugRecordingsFilePath() const;
100 100
101 // Enables or disables diagnostic event log. 101 // Enables or disables diagnostic event log.
102 void SetEventLogRecordings(bool enable, content::WebContents* web_contents); 102 void EnableEventLogRecordings(content::WebContents* web_contents);
103 void DisableEventLogRecordings();
103 104
104 bool IsEventLogRecordingsEnabled() const; 105 bool IsEventLogRecordingsEnabled() const;
105 const base::FilePath& GetEventLogRecordingsFilePath() const; 106 const base::FilePath& GetEventLogFilePath() const;
106 107
107 protected: 108 protected:
108 // Constructor/Destructor are protected to allow tests to derive from the 109 // Constructor/Destructor are protected to allow tests to derive from the
109 // class and do per-instance testing without having to use the global 110 // class and do per-instance testing without having to use the global
110 // instance. 111 // instance.
111 // The default ctor sets |aggregate_updates_ms| to 500ms. 112 // The default ctor sets |aggregate_updates_ms| to 500ms.
112 WebRTCInternals(); 113 WebRTCInternals();
113 explicit WebRTCInternals(int aggregate_updates_ms); 114 explicit WebRTCInternals(int aggregate_updates_ms);
114 ~WebRTCInternals() override; 115 ~WebRTCInternals() override;
115 116
(...skipping 119 matching lines...) Expand 10 before | Expand all | Expand 10 after
235 std::queue<PendingUpdate> pending_updates_; 236 std::queue<PendingUpdate> pending_updates_;
236 const int aggregate_updates_ms_; 237 const int aggregate_updates_ms_;
237 238
238 // Weak factory for this object that we use for bulking up updates. 239 // Weak factory for this object that we use for bulking up updates.
239 base::WeakPtrFactory<WebRTCInternals> weak_factory_; 240 base::WeakPtrFactory<WebRTCInternals> weak_factory_;
240 }; 241 };
241 242
242 } // namespace content 243 } // namespace content
243 244
244 #endif // CONTENT_BROWSER_MEDIA_WEBRTC_WEBRTC_INTERNALS_H_ 245 #endif // CONTENT_BROWSER_MEDIA_WEBRTC_WEBRTC_INTERNALS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698