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

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

Issue 101063003: Add browser test for AEC dump. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 7 years 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 | Annotate | Revision Log
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_INTERNALS_H_ 5 #ifndef CONTENT_BROWSER_MEDIA_WEBRTC_INTERNALS_H_
6 #define CONTENT_BROWSER_MEDIA_WEBRTC_INTERNALS_H_ 6 #define CONTENT_BROWSER_MEDIA_WEBRTC_INTERNALS_H_
7 7
8 #include "base/gtest_prod_util.h"
8 #include "base/memory/singleton.h" 9 #include "base/memory/singleton.h"
9 #include "base/observer_list.h" 10 #include "base/observer_list.h"
10 #include "base/process/process.h" 11 #include "base/process/process.h"
11 #include "base/values.h" 12 #include "base/values.h"
12 #include "content/common/content_export.h" 13 #include "content/common/content_export.h"
13 #include "content/public/browser/browser_child_process_observer.h" 14 #include "content/public/browser/browser_child_process_observer.h"
14 #include "content/public/browser/notification_observer.h" 15 #include "content/public/browser/notification_observer.h"
15 #include "content/public/browser/notification_registrar.h" 16 #include "content/public/browser/notification_registrar.h"
16 #include "ui/shell_dialogs/select_file_dialog.h" 17 #include "ui/shell_dialogs/select_file_dialog.h"
17 18
(...skipping 61 matching lines...) Expand 10 before | Expand all | Expand 10 after
79 bool aec_dump_enabled() { 80 bool aec_dump_enabled() {
80 return aec_dump_enabled_; 81 return aec_dump_enabled_;
81 } 82 }
82 83
83 base::FilePath aec_dump_file_path() { 84 base::FilePath aec_dump_file_path() {
84 return aec_dump_file_path_; 85 return aec_dump_file_path_;
85 } 86 }
86 87
87 private: 88 private:
88 friend struct DefaultSingletonTraits<WebRTCInternals>; 89 friend struct DefaultSingletonTraits<WebRTCInternals>;
90 FRIEND_TEST_ALL_PREFIXES(WebrtcBrowserTest, CallWithAecDump);
91 FRIEND_TEST_ALL_PREFIXES(WebrtcBrowserTest,
92 CallWithAecDumpEnabledThenDisabled);
89 93
90 WebRTCInternals(); 94 WebRTCInternals();
91 virtual ~WebRTCInternals(); 95 virtual ~WebRTCInternals();
92 96
93 void SendUpdate(const std::string& command, base::Value* value); 97 void SendUpdate(const std::string& command, base::Value* value);
94 98
95 // BrowserChildProcessObserver implementation. 99 // BrowserChildProcessObserver implementation.
96 virtual void BrowserChildProcessCrashed( 100 virtual void BrowserChildProcessCrashed(
97 const ChildProcessData& data) OVERRIDE; 101 const ChildProcessData& data) OVERRIDE;
98 102
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
135 scoped_refptr<ui::SelectFileDialog> select_file_dialog_; 139 scoped_refptr<ui::SelectFileDialog> select_file_dialog_;
136 140
137 // AEC dump (diagnostic echo canceller recording) state. 141 // AEC dump (diagnostic echo canceller recording) state.
138 bool aec_dump_enabled_; 142 bool aec_dump_enabled_;
139 base::FilePath aec_dump_file_path_; 143 base::FilePath aec_dump_file_path_;
140 }; 144 };
141 145
142 } // namespace content 146 } // namespace content
143 147
144 #endif // CONTENT_BROWSER_MEDIA_WEBRTC_INTERNALS_H_ 148 #endif // CONTENT_BROWSER_MEDIA_WEBRTC_INTERNALS_H_
OLDNEW
« no previous file with comments | « content/browser/media/webrtc_browsertest.cc ('k') | content/browser/renderer_host/render_process_host_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698