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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 3192017: Revert 57254 - Share one thread between all AudioOutputControllers instead of... (Closed)

Created:
10 years, 4 months ago by scherkus (not reviewing)
Modified:
9 years, 7 months ago
Reviewers:
Sergey Ulanov
CC:
chromium-reviews, fbarchard, Alpha Left Google, John Grabowski, brettw-cc_chromium.org, pam+watch_chromium.org, awong, Paweł Hajdan Jr., darin-cc_chromium.org, scherkus (not reviewing)
Visibility:
Public.

Description

Revert 57254 - Share one thread between all AudioOutputControllers instead of creating one per stream. TEST=unittests BUG=39825 Review URL: http://codereview.chromium.org/3185022 TBR=sergeyu@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=57256

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+229 lines, -354 lines) Patch
M chrome/common/render_messages_params.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/renderer/media/audio_renderer_impl.h View 1 chunk +0 lines, -1 line 0 comments Download
M media/audio/audio_input_controller.h View 1 chunk +0 lines, -1 line 0 comments Download
M media/audio/audio_io.h View 1 chunk +76 lines, -0 lines 0 comments Download
D media/audio/audio_manager.h View 1 chunk +0 lines, -101 lines 0 comments Download
D media/audio/audio_manager.cc View 1 chunk +0 lines, -59 lines 0 comments Download
D media/audio/audio_manager_base.h View 1 chunk +0 lines, -35 lines 0 comments Download
D media/audio/audio_manager_base.cc View 1 chunk +0 lines, -19 lines 0 comments Download
M media/audio/audio_output_controller.h View 3 chunks +6 lines, -8 lines 0 comments Download
M media/audio/audio_output_controller.cc View 11 chunks +54 lines, -54 lines 0 comments Download
M media/audio/audio_output_controller_unittest.cc View 9 chunks +10 lines, -33 lines 0 comments Download
M media/audio/fake_audio_input_stream_unittest.cc View 2 chunks +1 line, -1 line 0 comments Download
M media/audio/linux/alsa_output.h View 1 chunk +1 line, -1 line 0 comments Download
M media/audio/linux/alsa_output.cc View 8 chunks +8 lines, -8 lines 0 comments Download
M media/audio/linux/audio_manager_linux.h View 2 chunks +7 lines, -2 lines 0 comments Download
M media/audio/linux/audio_manager_linux.cc View 6 chunks +26 lines, -7 lines 0 comments Download
M media/audio/mac/audio_manager_mac.h View 1 chunk +2 lines, -2 lines 0 comments Download
M media/audio/mac/audio_manager_mac.cc View 2 chunks +19 lines, -3 lines 0 comments Download
M media/audio/openbsd/audio_manager_openbsd.h View 1 chunk +1 line, -1 line 0 comments Download
M media/audio/openbsd/audio_manager_openbsd.cc View 3 chunks +1 line, -6 lines 0 comments Download
M media/audio/simple_sources_unittest.cc View 1 chunk +0 lines, -1 line 0 comments Download
M media/audio/win/audio_manager_win.h View 2 chunks +2 lines, -2 lines 0 comments Download
M media/audio/win/audio_manager_win.cc View 3 chunks +14 lines, -3 lines 0 comments Download
M media/filters/audio_renderer_impl.cc View 1 chunk +0 lines, -1 line 0 comments Download
M media/media.gyp View 1 chunk +0 lines, -4 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
scherkus (not reviewing)
10 years, 4 months ago (2010-08-24 23:29:44 UTC) #1 

          

          

        


      

    

    

  











      		
    

  





  
    
  






  Powered by Google App Engine





  
This is Rietveld 408576698