Remove GPU relinquish resources infrastructure and stop GPU process.

content/browser/gpu/gpu_process_host.cc

Index: content/browser/gpu/gpu_process_host.cc
diff --git a/content/browser/gpu/gpu_process_host.cc b/content/browser/gpu/gpu_process_host.cc
index 1c37f80818bda020c0877bb305a5d8b0e1bd5da3..a3f8b370e30892dc6ac5183d6e394fd4a735b5bc 100644
--- a/content/browser/gpu/gpu_process_host.cc
+++ b/content/browser/gpu/gpu_process_host.cc
@@ -397,6 +397,7 @@ GpuProcessHost::GpuProcessHost(int host_id, GpuProcessKind kind)
       kind_(kind),
       process_launched_(false),
       initialized_(false),
+      expecting_shutdown_(false),
       gpu_crash_recorded_(false),
       uma_memory_stats_received_(false) {
   if (base::CommandLine::ForCurrentProcess()->HasSwitch(
@@ -430,7 +431,14 @@ GpuProcessHost::~GpuProcessHost() {
 
   SendOutstandingReplies();
 
-  RecordProcessCrash();
+  // Do not record crash if GPU process was asked to stop and stopped
+  // successfully.
+  int exit_code;
+  if (!expecting_shutdown_ ||
+      process_->GetTerminationStatus(false, &exit_code) !=
+          base::TERMINATION_STATUS_NORMAL_TERMINATION) {
+    RecordProcessCrash();

[no sievers, 2015/07/28 21:28:50]

That's better, but calling RecordProcessCrash() here at all seems odd.
Can you just remove it?

Then I think it needs a fix for Android in
BrowserChildProcessHostImpl::OnChildDisconnected():

Similarly to CrOS TERMINATION_STATUS_PROCESS_WAS_KILLED_BY_OOM, can you add
TERMINATION_STATUS_OOM_PROTECTED to make sure it calls OnProcessCrashed()?

And then I'd also add an explicit call to RecordProcessCrash() from
GpuProcessHost::Get() where Init() failed and we 'delete host'.
But put a TODO to revisit this behavior. It's not really a crash, but we also
don't want to regress anything in terms of fallback-to-sw behavior.

You can also get rid of |expecting_shutdown_| then.

[no sievers, 2015/07/28 21:31:46]

I forgot one:

You might also want to override OnProcessLaunchFailed() to keep the current
behavior. But it's also a bit questionable to count it as a crash.

[derekjchow1, 2015/07/29 18:35:40]

Done.
Done.
Done. I've stapled your name to the TODO. Is that ok?
Done.
Done.

+  }
 
   // In case we never started, clean up.
   while (!queued_messages_.empty()) {
@@ -903,6 +911,11 @@ void GpuProcessHost::ForceShutdown() {
   process_->ForceShutdown();
 }
 
+void GpuProcessHost::StopGpuProcess() {
+  Send(new GpuMsg_Finalize());
+  expecting_shutdown_ = true;
+}
+
 void GpuProcessHost::BeginFrameSubscription(
     int surface_id,
     base::WeakPtr<RenderWidgetHostViewFrameSubscriber> subscriber) {