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

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: Review comments from Tommi. 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 84 matching lines...) Expand 10 before | Expand all | Expand 10 after
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 SetEventLogRecordings(bool enable, content::WebContents* web_contents);
103 103
104 bool IsEventLogRecordingsEnabled() const; 104 bool IsEventLogRecordingsEnabled() const;
105 const base::FilePath& GetEventLogRecordingsFilePath() const;
106 105
107 protected: 106 protected:
108 // Constructor/Destructor are protected to allow tests to derive from the 107 // Constructor/Destructor are protected to allow tests to derive from the
109 // class and do per-instance testing without having to use the global 108 // class and do per-instance testing without having to use the global
110 // instance. 109 // instance.
111 // The default ctor sets |aggregate_updates_ms| to 500ms. 110 // The default ctor sets |aggregate_updates_ms| to 500ms.
112 WebRTCInternals(); 111 WebRTCInternals();
113 explicit WebRTCInternals(int aggregate_updates_ms); 112 explicit WebRTCInternals(int aggregate_updates_ms);
114 ~WebRTCInternals() override; 113 ~WebRTCInternals() override;
115 114
(...skipping 119 matching lines...) Expand 10 before | Expand all | Expand 10 after
235 std::queue<PendingUpdate> pending_updates_; 234 std::queue<PendingUpdate> pending_updates_;
236 const int aggregate_updates_ms_; 235 const int aggregate_updates_ms_;
237 236
238 // Weak factory for this object that we use for bulking up updates. 237 // Weak factory for this object that we use for bulking up updates.
239 base::WeakPtrFactory<WebRTCInternals> weak_factory_; 238 base::WeakPtrFactory<WebRTCInternals> weak_factory_;
240 }; 239 };
241 240
242 } // namespace content 241 } // namespace content
243 242
244 #endif // CONTENT_BROWSER_MEDIA_WEBRTC_WEBRTC_INTERNALS_H_ 243 #endif // CONTENT_BROWSER_MEDIA_WEBRTC_WEBRTC_INTERNALS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698