desktop_capture: crop border in window_capture on Win8/10

webrtc/modules/desktop_capture/win/window_capture_utils.cc

Index: webrtc/modules/desktop_capture/win/window_capture_utils.cc
diff --git a/webrtc/modules/desktop_capture/win/window_capture_utils.cc b/webrtc/modules/desktop_capture/win/window_capture_utils.cc
index 83922ea7f806e2b8eb0f916e48d70f68a1af7342..0777538b5430bdf7177deca2637fe8ce745c9863 100644
--- a/webrtc/modules/desktop_capture/win/window_capture_utils.cc
+++ b/webrtc/modules/desktop_capture/win/window_capture_utils.cc
@@ -9,6 +9,7 @@
  */
 
 #include "webrtc/modules/desktop_capture/win/window_capture_utils.h"
+#include "webrtc/base/win32.h"

[Do not use (sergeyu), 2017/07/06 20:35:49]

add empty line here.
https://google.github.io/styleguide/cppguide.html#Names_and_Order_of_Includes

[Do not use (sergeyu), 2017/07/06 20:35:49]

I think you want to include webrtc/rtc_base/win32.h . webrtc/base/win32.h is
deprecated

[braveyao1, 2017/07/06 22:10:47]

Done.

[braveyao1, 2017/07/06 22:10:47]

Done.

 
 namespace webrtc {
 
@@ -29,7 +30,8 @@ GetCroppedWindowRect(HWND window,
   *original_rect = DesktopRect::MakeLTRB(
       rect.left, rect.top, rect.right, rect.bottom);
 
-  if (window_placement.showCmd == SW_SHOWMAXIMIZED) {
+  if (rtc::IsWindows8OrLater() ||

[Do not use (sergeyu), 2017/07/06 20:36:37]

Please add a comment here to explain why we want to crop the border only after
win8

[braveyao1, 2017/07/06 22:10:47]

Done.

+      window_placement.showCmd == SW_SHOWMAXIMIZED) {
     DesktopSize border = DesktopSize(GetSystemMetrics(SM_CXSIZEFRAME),
                                      GetSystemMetrics(SM_CYSIZEFRAME));
     *cropped_rect = DesktopRect::MakeLTRB(