commit 1544915bb427ec076c075e9150c8c0214584f0ac
author Bryan Ferguson <bryanf@microsoft.com> Thu Aug 08 23:41:02 2019
committer Commit Bot <commit-bot@chromium.org> Sat Aug 10 03:40:17 2019
tree 2c7ce8afd03a0b0b12dfbf19ab6d290b4b3f701d
parent e427996c6174be0ce25d08d32da3c7be4f3792db

Avoid capturing extraneous windows in CroppingWindowCapturerWin

This change reduces cases where capturing a window with the cropping
capturer captures unrelated windows from the same process. For instance:
- Capturing an Explorer window could include portions of taskbar UI,
  e.g. when an auto-hide taskbar or window preview thumbnails are shown
  overtop.
- Capturing a window from a process with multiple windows could include
  menus/tooltips from another window.

Instead of capturing any window with an empty/matching title created by
the same process, the cropping capturer will capture any window created
by the same thread. While not foolproof, this heuristic seems to capture
menus/tooltips from the window of interest while excluding those from
other top-level windows in practice (assuming those were created by a
separate thread / independent message pump).

Bug: webrtc:10856
Change-Id: I2072c79da9e0158475b442a43b5b96d6ad307bc2
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/148641
Reviewed-by: Jamie Walch <jamiewalch@chromium.org>
Commit-Queue: Jamie Walch <jamiewalch@chromium.org>
Cr-Commit-Position: refs/heads/master@{#28824}