Coverity: Initialize member variables.

remoting/client/rectangle_update_decoder.cc

 // Copyright (c) 2011 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/client/rectangle_update_decoder.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "remoting/base/decoder.h"
 #include "remoting/base/decoder_row_based.h"
 #include "remoting/base/decoder_vp8.h"
 #include "remoting/base/util.h"
 #include "remoting/client/frame_consumer.h"
 #include "remoting/protocol/session_config.h"
 
 using remoting::protocol::ChannelConfig;
 using remoting::protocol::SessionConfig;
 
 namespace remoting {
 
 RectangleUpdateDecoder::RectangleUpdateDecoder(MessageLoop* message_loop,
                                                FrameConsumer* consumer)
     : message_loop_(message_loop),
       consumer_(consumer),
       frame_is_new_(false),
       frame_is_consuming_(false) {
+  memset(&clip_rect_, 0, sizeof(clip_rect_));
+  memset(&initial_screen_size_, 0, sizeof(initial_screen_size_));

[Wez, 2011/12/20 20:08:02]

Similarly, SkIRect::MakeEmpty() for |clip_rect_| and SkISize::Make(0, 0) for
|initial_screen_size|.

[James Hawkins, 2011/12/20 20:48:51]

Done.

 }
 
 RectangleUpdateDecoder::~RectangleUpdateDecoder() {
 }
 
 void RectangleUpdateDecoder::Initialize(const SessionConfig& config) {
   initial_screen_size_ = SkISize::Make(config.initial_resolution().width,
                                        config.initial_resolution().height);
 
   // Initialize decoder based on the selected codec.
@@ skipping 191 matching lines @@
     return;
   }
 
   delete rects;
 
   frame_is_consuming_ = false;
   DoRefresh();
 }
 
 }  // namespace remoting