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

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

BUG=154716
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=163727

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

lgtm

http://codereview.chromium.org/11235072/diff/1/remoting/host/video_frame_capt...
File remoting/host/video_frame_capturer_mac.mm (right):

http://codereview.chromium.org/11235072/diff/1/remoting/host/video_frame_capt...
remoting/host/video_frame_capturer_mac.mm:751: return;
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:19]

http://codereview.chromium.org/11235072/diff/1/remoting/host/video_frame_capt...
File remoting/host/video_frame_capturer_mac.mm (right):

http://codereview.chromium.org/11235072/diff/1/remoting/host/video_frame_capt...
remoting/host/video_frame_capturer_mac.mm:751: return;
On 2012/10/23 21:19:20, Jamie wrote:
> Nit: This file seems to have braces for single-line blocks in most other
places.
> We should be consistent.

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.

[commit-bot: I haz the power, 2012-10-23 21:35:02]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/garykac@chromium.org/11235072/4001

[commit-bot: I haz the power, 2012-10-23 23:42:47]

Change committed as 163727