Fix the aec dump crash when users disable aec dump without closing the "Save File" overlay.

content/browser/media/webrtc_internals.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/media/webrtc_internals.h"
 
 #include "base/command_line.h"
 #include "base/path_service.h"
 #include "content/browser/media/webrtc_internals_ui_observer.h"
 #include "content/browser/web_contents/web_contents_view.h"
@@ skipping 228 matching lines @@
       FILE_PATH_LITERAL(""),
       web_contents->GetTopLevelNativeWindow(),
       NULL);
 #endif
 #endif
 }
 
 void WebRTCInternals::DisableAecDump() {
 #if defined(ENABLE_WEBRTC)
   aec_dump_enabled_ = false;
+  select_file_dialog_ = NULL;

[Henrik Grunell, 2014/05/30 11:24:39]

Add a comment why we do this. (If user unchecks box while dialog still open.)

[no longer working on chromium, 2014/06/02 08:34:14]

Done.

   for (RenderProcessHost::iterator i(
            content::RenderProcessHost::AllHostsIterator());
        !i.IsAtEnd(); i.Advance()) {
     i.GetCurrentValue()->DisableAecDump();
   }
 #endif
 }
 
 void WebRTCInternals::ResetForTesting() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
@@ skipping 87 matching lines @@
   aec_dump_enabled_ = true;
   for (RenderProcessHost::iterator i(
            content::RenderProcessHost::AllHostsIterator());
        !i.IsAtEnd(); i.Advance()) {
     i.GetCurrentValue()->EnableAecDump(aec_dump_file_path_);
   }
 }
 #endif
 
 }  // namespace content