Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(39)

Issue 11155016: Merge 160986 - [Mac] Remove webcam capture output on main thread to OS race. (Closed)

Created:
8 years, 2 months ago by Scott Hess - ex-Googler
Modified:
8 years, 2 months ago
CC:
chromium-reviews, feature-media-reviews_chromium.org
Visibility:
Public.

Description

Merge 160986 - [Mac] Remove webcam capture output on main thread to OS race. When doing webcam setup and teardown on a background thread, QTKit appears to have a race condition around how it posts work to the main thread. This posts part of teardown to the main thread to work around the problem. BUG=152757 TEST=See bug. Review URL: https://chromiumcodereview.appspot.com/11094031 TBR=shess@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=161916

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+18 lines, -2 lines) Patch
M media/video/capture/mac/video_capture_device_qtkit_mac.mm View 1 chunk +18 lines, -2 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Scott Hess - ex-Googler
8 years, 2 months ago (2012-10-15 19:05:06 UTC) #1
wjia(left Chromium)
8 years, 2 months ago (2012-10-15 19:58:02 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698