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

Side by Side Diff: content/browser/renderer_host/render_process_host_impl.cc

Issue 1923653002: Wire up process launch error codes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix debug and clang Created 4 years, 7 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 2012 The Chromium Authors. All rights reserved. 1 // Copyright 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 // Represents the browser side of the browser <--> renderer communication 5 // Represents the browser side of the browser <--> renderer communication
6 // channel. There will be one RenderProcessHost per renderer process. 6 // channel. There will be one RenderProcessHost per renderer process.
7 7
8 #include "content/browser/renderer_host/render_process_host_impl.h" 8 #include "content/browser/renderer_host/render_process_host_impl.h"
9 9
10 #include <algorithm> 10 #include <algorithm>
(...skipping 2594 matching lines...) Expand 10 before | Expand all | Expand 10 after
2605 } 2605 }
2606 2606
2607 #if defined(ENABLE_WEBRTC) 2607 #if defined(ENABLE_WEBRTC)
2608 if (WebRTCInternals::GetInstance()->IsAudioDebugRecordingsEnabled()) { 2608 if (WebRTCInternals::GetInstance()->IsAudioDebugRecordingsEnabled()) {
2609 EnableAudioDebugRecordings( 2609 EnableAudioDebugRecordings(
2610 WebRTCInternals::GetInstance()->GetAudioDebugRecordingsFilePath()); 2610 WebRTCInternals::GetInstance()->GetAudioDebugRecordingsFilePath());
2611 } 2611 }
2612 #endif 2612 #endif
2613 } 2613 }
2614 2614
2615 void RenderProcessHostImpl::OnProcessLaunchFailed() { 2615 void RenderProcessHostImpl::OnProcessLaunchFailed(int error_code) {
2616 // If this object will be destructed soon, then observers have already been 2616 // If this object will be destructed soon, then observers have already been
2617 // sent a RenderProcessHostDestroyed notification, and we must observe our 2617 // sent a RenderProcessHostDestroyed notification, and we must observe our
2618 // contract that says that will be the last call. 2618 // contract that says that will be the last call.
2619 if (deleting_soon_) 2619 if (deleting_soon_)
2620 return; 2620 return;
2621 2621
2622 // TODO(wfh): Fill in the real error code here see crbug.com/526198. 2622 RendererClosedDetails details{base::TERMINATION_STATUS_LAUNCH_FAILED,
2623 RendererClosedDetails details{base::TERMINATION_STATUS_LAUNCH_FAILED, -1}; 2623 error_code};
2624 ProcessDied(true, &details); 2624 ProcessDied(true, &details);
2625 } 2625 }
2626 2626
2627 scoped_refptr<AudioRendererHost> RenderProcessHostImpl::audio_renderer_host() 2627 scoped_refptr<AudioRendererHost> RenderProcessHostImpl::audio_renderer_host()
2628 const { 2628 const {
2629 return audio_renderer_host_; 2629 return audio_renderer_host_;
2630 } 2630 }
2631 2631
2632 void RenderProcessHostImpl::OnUserMetricsRecordAction( 2632 void RenderProcessHostImpl::OnUserMetricsRecordAction(
2633 const std::string& action) { 2633 const std::string& action) {
(...skipping 162 matching lines...) Expand 10 before | Expand all | Expand 10 after
2796 2796
2797 // Skip widgets in other processes. 2797 // Skip widgets in other processes.
2798 if (rvh->GetProcess()->GetID() != GetID()) 2798 if (rvh->GetProcess()->GetID() != GetID())
2799 continue; 2799 continue;
2800 2800
2801 rvh->OnWebkitPreferencesChanged(); 2801 rvh->OnWebkitPreferencesChanged();
2802 } 2802 }
2803 } 2803 }
2804 2804
2805 } // namespace content 2805 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/renderer_host/render_process_host_impl.h ('k') | content/browser/utility_process_host_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698