SUCCEEDED macro is misused

webrtc/modules/desktop_capture/screen_drawer_win.cc

Index: webrtc/modules/desktop_capture/screen_drawer_win.cc
diff --git a/webrtc/modules/desktop_capture/screen_drawer_win.cc b/webrtc/modules/desktop_capture/screen_drawer_win.cc
index 0b669916479c876366ac0237557745e693f37a3a..d5f4db1d8ee0f32116fb18b474306b8be8acb4e9 100644
--- a/webrtc/modules/desktop_capture/screen_drawer_win.cc
+++ b/webrtc/modules/desktop_capture/screen_drawer_win.cc
@@ -174,8 +174,8 @@ void ScreenDrawerWin::DrawDot(DesktopVector vect, RgbaColor color) {
 }
 
 void ScreenDrawerWin::BringToFront() {
-  if (SUCCEEDED(SetWindowPos(
-      window_, HWND_TOPMOST, 0, 0, 0, 0, SWP_NOMOVE | SWP_NOSIZE))) {
+  if (SetWindowPos(window_, HWND_TOPMOST, 0, 0, 0, 0, SWP_NOMOVE | SWP_NOSIZE)
+      == TRUE) {

[Sergey Ulanov, 2016/10/20 17:31:15]

git cl format please.
It formats this code differently:
  if (SetWindowPos(window_, HWND_TOPMOST, 0, 0, 0, 0,
                   SWP_NOMOVE | SWP_NOSIZE) == TRUE) {
    return;
  }

And this looks more readable.

[Hzj_jie, 2016/10/20 18:15:18]

Done.

     return;
   }