Don't attempt to read the process id for a NULL handle

content/shell/webkit_test_controller.cc

 // Copyright (c) 2012 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/shell/webkit_test_controller.h"
 
 #include <iostream>
 
 #include "base/command_line.h"
 #include "base/file_util.h"
@@ skipping 255 matching lines @@
   return handled;
 }
 
 void WebKitTestController::PluginCrashed(const FilePath& plugin_path) {
   DCHECK(CalledOnValidThread());
   printer_->AddErrorMessage("#CRASHED - plugin");
 }
 
 void WebKitTestController::RenderViewCreated(RenderViewHost* render_view_host) {
   DCHECK(CalledOnValidThread());
-  // Might be kNullProcessId, in which case we will receive a notification later
-  // when the RenderProcessHost was created.
-  current_pid_ = base::GetProcId(render_view_host->GetProcess()->GetHandle());
+  // Might be kNullProcessHandle, in which case we will receive a notification
+  // later when the RenderProcessHost was created.
+  if (render_view_host->GetProcess()->GetHandle() != base::kNullProcessHandle)
+    current_pid_ = base::GetProcId(render_view_host->GetProcess()->GetHandle());
   render_view_host->Send(new ShellViewMsg_SetCurrentWorkingDirectory(
       render_view_host->GetRoutingID(), current_working_directory_));
 }
 
 void WebKitTestController::RenderViewGone(base::TerminationStatus status) {
   DCHECK(CalledOnValidThread());
   if (current_pid_ != base::kNullProcessId) {
     printer_->AddErrorMessage(std::string("#CRASHED - renderer (pid ") +
                               base::IntToString(current_pid_) + ")");
   } else {
@@ skipping 206 matching lines @@
 
 void WebKitTestController::OnNotImplemented(
     const std::string& object_name,
     const std::string& property_name) {
   printer_->AddErrorMessage(
       std::string("FAIL: NOT IMPLEMENTED: ") +
       object_name + "." + property_name);
 }
 
 }  // namespace content