Issue 10795053: Remove EventExecutor dependency on VideoFrameCapturer.

Issue 10795053: Remove EventExecutor dependency on VideoFrameCapturer. (Closed)

Created:
8 years, 5 months ago by Wez

Modified:
8 years, 5 months ago

Reviewers:
alexeypa (please no reviews)

CC:
chromium-reviews, jamiewalch+watch_chromium.org, dcaiafa+watch_chromium.org, simonmorris+watch_chromium.org, hclam+watch_chromium.org, wez+watch_chromium.org, amit, sanjeevr, garykac+watch_chromium.org, lambroslambrou+watch_chromium.org, alexeypa+watch_chromium.org, sergeyu+watch_chromium.org

Base URL:
svn://svn.chromium.org/chrome/trunk/src

Visibility:
Public.

More Reviews

Description

Remove EventExecutor dependency on VideoFrameCapturer. BUG=137044 TEST=Manual verification that Windows host input injection works correctly. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=147906

Patch Set 1 #

Patch Set 2 : Add missing includes for Windows. #

Patch Set 3 : Update comment. #

Created: 8 years, 5 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+17 lines, -22 lines)			Patch
M	remoting/host/desktop_environment.cc	View		2 chunks	+2 lines, -4 lines	0 comments	Download
M	remoting/host/event_executor.h	View		1 chunk	+1 line, -2 lines	0 comments	Download
M	remoting/host/event_executor_linux.cc	View		1 chunk	+1 line, -2 lines	0 comments	Download
M	remoting/host/event_executor_mac.cc	View		1 chunk	+1 line, -2 lines	0 comments	Download
M	remoting/host/event_executor_win.cc	View	1 2	6 chunks	+11 lines, -11 lines	0 comments	Download
M	remoting/host/simple_host_process.cc	View		1 chunk	+1 line, -1 line	0 comments	Download

Messages

Total messages: 6 (0 generated)

Expand Messages | Collapse Messages

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/wez@chromium.org/10795053/1

8 years, 5 months ago (2012-07-21 12:25:44 UTC) #3

commit-bot: I haz the power

Try job failure for 10795053-1 (retry) on win_rel for step "compile" (clobber build). It's a ...

8 years, 5 months ago (2012-07-21 13:03:32 UTC) #4

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/wez@chromium.org/10795053/7002

8 years, 5 months ago (2012-07-23 17:44:13 UTC) #5

Change committed as 147906

Expand Messages | Collapse Messages