Readability review.

remoting/host/fake_host_extension.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "remoting/host/fake_host_extension.h"
 
 #include <string>
 
 #include "base/logging.h"
 #include "remoting/codec/video_encoder.h"
@@ skipping 28 matching lines @@
 FakeExtension::Session::Session(
     FakeExtension* extension, const std::string& message_type)
   : extension_(extension),
     message_type_(message_type) {
 }
 
 scoped_ptr<webrtc::DesktopCapturer>
 FakeExtension::Session::OnCreateVideoCapturer(
     scoped_ptr<webrtc::DesktopCapturer> capturer) {
   extension_->has_wrapped_video_capturer_ = true;
   if (extension_->steal_video_capturer_) {

[Peter Kasting, 2014/08/22 06:38:27]

Nit: Completely up to you, but I think more Chromium code omits braces on
single-line conditional bodies than includes them.  (Same comment applies in
other files)

[Wez, 2014/08/25 23:23:10]

Acknowledged. General preference on the Chromoting team is to include the
braces, for safety, in new code.

     capturer.reset();
   }
   return capturer.Pass();
 }
 
 scoped_ptr<VideoEncoder> FakeExtension::Session::OnCreateVideoEncoder(
     scoped_ptr<VideoEncoder> encoder) {
   extension_->has_wrapped_video_encoder_ = true;
   return encoder.Pass();
 }
@@ skipping 53 matching lines @@
   DCHECK(was_instantiated());
   return has_wrapped_video_capturer_;
 }
 
 bool FakeExtension::has_handled_message() {
   DCHECK(was_instantiated());
   return has_handled_message_;
 }
 
 } // namespace remoting
+