[Chromoting] Change ScreenRefresh to use 'if' instead of 'DCHECK'.

remoting/host/video_frame_capturer_mac.mm

Index: remoting/host/video_frame_capturer_mac.mm
===================================================================
--- remoting/host/video_frame_capturer_mac.mm	(revision 163603)
+++ remoting/host/video_frame_capturer_mac.mm	(working copy)
@@ -747,7 +747,8 @@
 
 void VideoFrameCapturerMac::ScreenRefresh(CGRectCount count,
                                           const CGRect* rect_array) {
-  DCHECK(!desktop_bounds_.isEmpty());
+  if (desktop_bounds_.isEmpty())
+    return;

[Jamie, 2012/10/23 21:19:20]

Nit: This file seems to have braces for single-line blocks in most other places.
We should be consistent.

[garykac, 2012/10/23 21:34:20]

I looked at the CGBlitP*Lion routines to check for consistency here and they
don't use braces. Looking more carefully at the whole file, I can see that it
isn't consistent.

I'll add braces here and then make the braces consistent with my follow-up cl.

   SkIRect skirect_array[count];
   for (CGRectCount i = 0; i < count; ++i) {
     skirect_array[i] = CGRectToSkIRect(rect_array[i]);