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

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

Issue 202863004: Fix "unreachable code" warnings (MSVC warning 4702) in content/. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 6 years, 9 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 | Annotate | Revision Log
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 // 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 1261 matching lines...) Expand 10 before | Expand all | Expand 10 after
1272 // died due to fast shutdown versus another cause. 1272 // died due to fast shutdown versus another cause.
1273 fast_shutdown_started_ = true; 1273 fast_shutdown_started_ = true;
1274 1274
1275 ProcessDied(false /* already_dead */); 1275 ProcessDied(false /* already_dead */);
1276 return true; 1276 return true;
1277 } 1277 }
1278 1278
1279 void RenderProcessHostImpl::DumpHandles() { 1279 void RenderProcessHostImpl::DumpHandles() {
1280 #if defined(OS_WIN) 1280 #if defined(OS_WIN)
1281 Send(new ChildProcessMsg_DumpHandles()); 1281 Send(new ChildProcessMsg_DumpHandles());
1282 return; 1282 #else
1283 NOTIMPLEMENTED();
1283 #endif 1284 #endif
1284
1285 NOTIMPLEMENTED();
1286 } 1285 }
1287 1286
1288 // This is a platform specific function for mapping a transport DIB given its id 1287 // This is a platform specific function for mapping a transport DIB given its id
1289 TransportDIB* RenderProcessHostImpl::MapTransportDIB( 1288 TransportDIB* RenderProcessHostImpl::MapTransportDIB(
1290 TransportDIB::Id dib_id) { 1289 TransportDIB::Id dib_id) {
1291 #if defined(OS_WIN) 1290 #if defined(OS_WIN)
1292 // On Windows we need to duplicate the handle from the remote process 1291 // On Windows we need to duplicate the handle from the remote process
1293 HANDLE section; 1292 HANDLE section;
1294 DuplicateHandle(GetHandle(), dib_id.handle, GetCurrentProcess(), &section, 1293 DuplicateHandle(GetHandle(), dib_id.handle, GetCurrentProcess(), &section,
1295 FILE_MAP_READ | FILE_MAP_WRITE, 1294 FILE_MAP_READ | FILE_MAP_WRITE,
(...skipping 835 matching lines...) Expand 10 before | Expand all | Expand 10 after
2131 2130
2132 void RenderProcessHostImpl::DecrementWorkerRefCount() { 2131 void RenderProcessHostImpl::DecrementWorkerRefCount() {
2133 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); 2132 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
2134 DCHECK_GT(worker_ref_count_, 0); 2133 DCHECK_GT(worker_ref_count_, 0);
2135 --worker_ref_count_; 2134 --worker_ref_count_;
2136 if (worker_ref_count_ == 0) 2135 if (worker_ref_count_ == 0)
2137 Cleanup(); 2136 Cleanup();
2138 } 2137 }
2139 2138
2140 } // namespace content 2139 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698