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

Issue 4975003: Introduce a fake HostStub since there's no need to handle it now (Closed)

Created:
10 years, 1 month ago by Alpha Left Google
Modified:
9 years, 6 months ago
Reviewers:
Sergey Ulanov, garykac
CC:
chromium-reviews, Sergey Ulanov, dmac, Alpha Left Google
Visibility:
Public.

Description

Introduce a fake HostStub since there's no need to handle it now Also changed ChormotingHost to use it. BUG=None TEST=None

Patch Set 1 #

Patch Set 2 : give HostStubFake #

Unified diffs Side-by-side diffs Delta from patch set Stats (+60 lines, -4 lines) Patch
M remoting/host/chromoting_host.h View 3 chunks +5 lines, -1 line 0 comments Download
M remoting/host/chromoting_host.cc View 1 3 chunks +8 lines, -3 lines 0 comments Download
A remoting/host/host_stub_fake.h View 1 chunk +29 lines, -0 lines 0 comments Download
A remoting/host/host_stub_fake.cc View 1 chunk +16 lines, -0 lines 0 comments Download
M remoting/remoting.gyp View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Alpha Left Google
10 years, 1 month ago (2010-11-16 01:24:21 UTC) #1
awong
Punting this over to Gary and Sergey. (Hi!)
10 years, 1 month ago (2010-11-16 01:30:14 UTC) #2
garykac
10 years, 1 month ago (2010-11-16 01:36:29 UTC) #3
On 2010/11/16 01:30:14, awong wrote:
> Punting this over to Gary and Sergey.  (Hi!)

LGTM

Powered by Google App Engine
This is Rietveld 408576698