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

Side by Side Diff: media/audio/audio_manager.h

Issue 1563493004: Disable non-crash dumps since we have a large enough list now. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Bug link. Created 4 years, 11 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
« no previous file with comments | « chrome/browser/chrome_browser_main.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 MEDIA_AUDIO_AUDIO_MANAGER_H_ 5 #ifndef MEDIA_AUDIO_AUDIO_MANAGER_H_
6 #define MEDIA_AUDIO_AUDIO_MANAGER_H_ 6 #define MEDIA_AUDIO_AUDIO_MANAGER_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/macros.h" 10 #include "base/macros.h"
(...skipping 44 matching lines...) Expand 10 before | Expand all | Expand 10 after
55 // runner to ensure the audio thread is not stuck for more than 60 seconds; if 55 // runner to ensure the audio thread is not stuck for more than 60 seconds; if
56 // a hang is detected, the process will be crashed. See EnableHangMonitor(). 56 // a hang is detected, the process will be crashed. See EnableHangMonitor().
57 static AudioManager* CreateWithHangTimer( 57 static AudioManager* CreateWithHangTimer(
58 AudioLogFactory* audio_log_factory, 58 AudioLogFactory* audio_log_factory,
59 const scoped_refptr<base::SingleThreadTaskRunner>& monitor_task_runner); 59 const scoped_refptr<base::SingleThreadTaskRunner>& monitor_task_runner);
60 60
61 // Similar to Create() except uses a FakeAudioLogFactory for testing. 61 // Similar to Create() except uses a FakeAudioLogFactory for testing.
62 static AudioManager* CreateForTesting(); 62 static AudioManager* CreateForTesting();
63 63
64 // Enables non-crash dumps when audio thread hangs are detected. 64 // Enables non-crash dumps when audio thread hangs are detected.
65 // TODO(dalecurtis): There are no callers to this function at present. A list
66 // of bad drivers has been given to Microsoft. This should be re-enabled in
67 // the future if Microsoft is able to triage third party drivers.
68 // See http://crbug.com/422522
65 static void EnableCrashKeyLoggingForAudioThreadHangs(); 69 static void EnableCrashKeyLoggingForAudioThreadHangs();
66 70
67 #if defined(OS_LINUX) 71 #if defined(OS_LINUX)
68 // Sets the name of the audio source as seen by external apps. Only actually 72 // Sets the name of the audio source as seen by external apps. Only actually
69 // used with PulseAudio as of this writing. 73 // used with PulseAudio as of this writing.
70 static void SetGlobalAppName(const std::string& app_name); 74 static void SetGlobalAppName(const std::string& app_name);
71 75
72 // Returns the app name or an empty string if it is not set. 76 // Returns the app name or an empty string if it is not set.
73 static const std::string& GetGlobalAppName(); 77 static const std::string& GetGlobalAppName();
74 #endif 78 #endif
(...skipping 153 matching lines...) Expand 10 before | Expand all | Expand 10 after
228 protected: 232 protected:
229 AudioManager(); 233 AudioManager();
230 234
231 private: 235 private:
232 DISALLOW_COPY_AND_ASSIGN(AudioManager); 236 DISALLOW_COPY_AND_ASSIGN(AudioManager);
233 }; 237 };
234 238
235 } // namespace media 239 } // namespace media
236 240
237 #endif // MEDIA_AUDIO_AUDIO_MANAGER_H_ 241 #endif // MEDIA_AUDIO_AUDIO_MANAGER_H_
OLDNEW
« no previous file with comments | « chrome/browser/chrome_browser_main.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698