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

Issue 8601011: Add OVERRIDE to remoting/. (Closed)

Created:
9 years, 1 month ago by Avi (use Gerrit)
Modified:
9 years, 1 month ago
Reviewers:
awong
CC:
chromium-reviews, jamiewalch+watch_chromium.org, hclam+watch_chromium.org, simonmorris+watch_chromium.org, wez+watch_chromium.org, garykac+watch_chromium.org, lambroslambrou+watch_chromium.org, sergeyu+watch_chromium.org
Visibility:
Public.

Description

Add OVERRIDE to remoting/. BUG=104314 TEST=no change Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=110791

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+110 lines, -98 lines) Patch
M remoting/base/compound_buffer.h View 1 chunk +4 lines, -4 lines 0 comments Download
M remoting/base/compressor_verbatim.h View 2 chunks +11 lines, -6 lines 0 comments Download
M remoting/base/compressor_zlib.h View 1 chunk +9 lines, -5 lines 0 comments Download
M remoting/base/decoder_row_based.h View 1 chunk +6 lines, -6 lines 0 comments Download
M remoting/base/decompressor_verbatim.h View 1 chunk +2 lines, -2 lines 0 comments Download
M remoting/base/decompressor_zlib.h View 1 chunk +2 lines, -2 lines 0 comments Download
M remoting/client/plugin/chromoting_scriptable_object.h View 1 chunk +6 lines, -6 lines 0 comments Download
M remoting/client/plugin/pepper_input_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M remoting/client/plugin/pepper_xmpp_proxy.h View 1 chunk +4 lines, -3 lines 0 comments Download
M remoting/host/capturer_fake_ascii.h View 1 chunk +1 line, -1 line 0 comments Download
M remoting/host/client_session.h View 1 chunk +2 lines, -2 lines 0 comments Download
M remoting/host/in_memory_host_config.h View 1 chunk +7 lines, -5 lines 0 comments Download
M remoting/host/json_host_config.h View 1 chunk +1 line, -1 line 0 comments Download
M remoting/jingle_glue/javascript_signal_strategy.h View 1 chunk +1 line, -1 line 0 comments Download
M remoting/jingle_glue/jingle_thread.h View 2 chunks +5 lines, -5 lines 0 comments Download
M remoting/jingle_glue/ssl_socket_adapter.h View 3 chunks +21 lines, -21 lines 0 comments Download
M remoting/jingle_glue/xmpp_socket_adapter.h View 2 chunks +9 lines, -9 lines 0 comments Download
M remoting/protocol/buffered_socket_writer.h View 2 chunks +6 lines, -6 lines 0 comments Download
M remoting/protocol/channel_authenticator.h View 1 chunk +1 line, -1 line 0 comments Download
M remoting/protocol/client_event_dispatcher.h View 1 chunk +2 lines, -2 lines 0 comments Download
M remoting/protocol/fake_session.h View 2 chunks +8 lines, -8 lines 0 comments Download
M remoting/protocol/rtp_reader.h View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
Avi (use Gerrit)
Just adding OVERRIDE annotations. Nothing more.
9 years, 1 month ago (2011-11-18 23:29:12 UTC) #1
garykac
On 2011/11/18 23:29:12, Avi wrote: > Just adding OVERRIDE annotations. Nothing more. LGTM
9 years, 1 month ago (2011-11-18 23:41:36 UTC) #2
awong
9 years, 1 month ago (2011-11-19 00:24:35 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld 408576698