gpu: Replace more chrome ipc with mojom API.

services/ui/gpu/interfaces/gpu_service.mojom

 // Copyright 2016 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.
 
 module ui.mojom;
 
 import "cc/ipc/display_compositor.mojom";
 import "gpu/ipc/common/gpu_info.mojom";
 import "gpu/ipc/common/memory_stats.mojom";
 import "gpu/ipc/common/surface_handle.mojom";
@@ skipping 29 matching lines @@
   GetVideoMemoryUsageStats() => (gpu.mojom.VideoMemoryUsageStats stats);
 
   // Notify GPU that a shader was loaded from disk.
   LoadedShader(string data);
 
   // Tells GPU to release the surface because it's being destroyed.
   DestroyingVideoSurface(int32 surface_id) => ();
 
   // Tells GPU to wake up the GPU because we're about to draw.
   WakeUpGpu();
+
+  // Tells GPU to destroy all channels.
+  DestroyAllChannels();
+
+  Crash();

[Fady Samuel, 2017/03/01 20:10:58]

nit: comment on what this does?

+  Hang();

[Fady Samuel, 2017/03/01 20:10:58]

nit: comment on what this does?

+  ThrowJavaException();

[Fady Samuel, 2017/03/01 20:10:58]

nit: comment on what this does?

[sadrul, 2017/03/01 20:11:56]

These should be obvious/unambiguous from the names?

[Fady Samuel, 2017/03/01 20:14:42]

Fair enough, although it could be argued "DestroyAllChannels" is also obvious
but it has a comment. I guess it would be nice to have a comment for consistency
but I (as a non-OWNER) don't feel too strongly about it.

[sadrul, 2017/03/01 20:18:30]

I agree that DestroyAllChannels() doesn't need the comment either. My preference
would be to remove that comment.

 };