Remove VideoDecoder::Initialize().

remoting/codec/video_decoder_verbatim.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_decoder_verbatim.h"
 
 #include "base/logging.h"
 #include "remoting/base/util.h"
 
 namespace remoting {
 
 VideoDecoderVerbatim::VideoDecoderVerbatim() {}
 
 VideoDecoderVerbatim::~VideoDecoderVerbatim() {}
 
-void VideoDecoderVerbatim::Initialize(const webrtc::DesktopSize& screen_size) {
-  updated_region_.Clear();
-  screen_buffer_.reset();
+bool VideoDecoderVerbatim::DecodePacket(const VideoPacket& packet) {
+  if (packet.format().has_screen_width() &&
+      packet.format().has_screen_height()) {
+    webrtc::DesktopSize screen_size(packet.format().screen_width(),
+                                    packet.format().screen_height());
+    // Allocate the screen buffer, if necessary.
+    if (!screen_size.equals(screen_size_)) {
+      screen_size_ = screen_size;
+      screen_buffer_.reset(new uint8[screen_size_.width() *
+                                     screen_size_.height() * kBytesPerPixel]);

[Jamie, 2015/08/19 00:16:32]

Do you need to call updated_region_.Clear() as well?

[Sergey Ulanov, 2015/08/19 00:43:53]

Done.

+    }
+  }
 
-  screen_size_ = screen_size;
-  // Allocate the screen buffer, if necessary.
-  if (!screen_size_.is_empty()) {
-    screen_buffer_.reset(
-        new uint8[screen_size_.width() * screen_size_.height() *
-                  kBytesPerPixel]);
-  }
-}
-
-bool VideoDecoderVerbatim::DecodePacket(const VideoPacket& packet) {
   webrtc::DesktopRegion region;
 
   const char* in = packet.data().data();
   int stride = kBytesPerPixel * screen_size_.width();
   for (int i = 0; i < packet.dirty_rects_size(); ++i) {
     Rect proto_rect = packet.dirty_rects(i);
     webrtc::DesktopRect rect =
         webrtc::DesktopRect::MakeXYWH(proto_rect.x(),
                                       proto_rect.y(),
                                       proto_rect.width(),
@@ skipping 65 matching lines @@
   }
 
   updated_region_.Clear();
 }
 
 const webrtc::DesktopRegion* VideoDecoderVerbatim::GetImageShape() {
   return nullptr;
 }
 
 }  // namespace remoting