Rewrite scoped_ptr<T>(NULL) to use the default ctor in remoting/.

remoting/codec/video_encoder_vp8.cc

 // Copyright (c) 2012 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/codec/video_encoder_vp8.h"
 
 #include "base/logging.h"
 #include "base/sys_info.h"
 #include "base/time.h"
 #include "media/base/yuv_convert.h"
@@ skipping 13 matching lines @@
 // Defines the dimension of a macro block. This is used to compute the active
 // map for the encoder.
 const int kMacroBlockSize = 16;
 
 }  // namespace remoting
 
 namespace remoting {
 
 VideoEncoderVp8::VideoEncoderVp8()
     : initialized_(false),
-      codec_(NULL),
-      image_(NULL),
       active_map_width_(0),
       active_map_height_(0),
-      last_timestamp_(0) {
-}
+      last_timestamp_(0) {}

[Wez, 2013/06/13 17:15:52]

nit: Please put the closing brace back on the next line.

[dcheng, 2013/06/13 18:28:36]

This is the result of clang-format-diff.py. If you feel strongly, I can manually
revert these by hand but future refactorings will likely keep doing the same
thing.

 
 VideoEncoderVp8::~VideoEncoderVp8() {
   Destroy();
 }
 
 void VideoEncoderVp8::Destroy() {
   if (initialized_) {
     vpx_codec_err_t ret = vpx_codec_destroy(codec_.get());
     DCHECK_EQ(ret, VPX_CODEC_OK) << "Failed to destroy codec";
     initialized_ = false;
@@ skipping 251 matching lines @@
     rect->set_x(r.rect().x());
     rect->set_y(r.rect().y());
     rect->set_width(r.rect().width());
     rect->set_height(r.rect().height());
   }
 
   data_available_callback.Run(packet.Pass());
 }
 
 }  // namespace remoting