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

Side by Side Diff: remoting/base/decoder_verbatim.cc

Issue 3141036: Revert 57112 - Rename (Host|Client)Message to Chromoting(Host|Client)Message.... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 10 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « remoting/base/decoder_verbatim.h ('k') | remoting/base/decoder_verbatim_unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "remoting/base/decoder_verbatim.h" 5 #include "remoting/base/decoder_verbatim.h"
6 6
7 #include "remoting/base/protocol_util.h" 7 #include "remoting/base/protocol_util.h"
8 8
9 namespace remoting { 9 namespace remoting {
10 10
(...skipping 20 matching lines...) Expand all
31 partial_decode_done_.reset(partial_decode_done); 31 partial_decode_done_.reset(partial_decode_done);
32 decode_done_.reset(decode_done); 32 decode_done_.reset(decode_done);
33 updated_rects_ = updated_rects; 33 updated_rects_ = updated_rects;
34 34
35 // TODO(hclam): Check if we can accept the color format of the video frame 35 // TODO(hclam): Check if we can accept the color format of the video frame
36 // and the codec. 36 // and the codec.
37 frame_ = frame; 37 frame_ = frame;
38 return true; 38 return true;
39 } 39 }
40 40
41 bool DecoderVerbatim::PartialDecode(ChromotingHostMessage* message) { 41 bool DecoderVerbatim::PartialDecode(HostMessage* message) {
42 scoped_ptr<ChromotingHostMessage> msg_deleter(message); 42 scoped_ptr<HostMessage> msg_deleter(message);
43 DCHECK(message->has_update_stream_packet()); 43 DCHECK(message->has_update_stream_packet());
44 44
45 bool ret = true; 45 bool ret = true;
46 if (message->update_stream_packet().has_begin_rect()) 46 if (message->update_stream_packet().has_begin_rect())
47 ret = HandleBeginRect(message); 47 ret = HandleBeginRect(message);
48 if (ret && message->update_stream_packet().has_rect_data()) 48 if (ret && message->update_stream_packet().has_rect_data())
49 ret = HandleRectData(message); 49 ret = HandleRectData(message);
50 if (ret && message->update_stream_packet().has_end_rect()) 50 if (ret && message->update_stream_packet().has_end_rect())
51 ret = HandleEndRect(message); 51 ret = HandleEndRect(message);
52 return ret; 52 return ret;
53 } 53 }
54 54
55 void DecoderVerbatim::EndDecode() { 55 void DecoderVerbatim::EndDecode() {
56 DCHECK_EQ(kWaitingForBeginRect, state_); 56 DCHECK_EQ(kWaitingForBeginRect, state_);
57 decode_done_->Run(); 57 decode_done_->Run();
58 58
59 partial_decode_done_.reset(); 59 partial_decode_done_.reset();
60 decode_done_.reset(); 60 decode_done_.reset();
61 frame_ = NULL; 61 frame_ = NULL;
62 updated_rects_ = NULL; 62 updated_rects_ = NULL;
63 } 63 }
64 64
65 bool DecoderVerbatim::HandleBeginRect(ChromotingHostMessage* message) { 65 bool DecoderVerbatim::HandleBeginRect(HostMessage* message) {
66 DCHECK_EQ(kWaitingForBeginRect, state_); 66 DCHECK_EQ(kWaitingForBeginRect, state_);
67 state_ = kWaitingForRectData; 67 state_ = kWaitingForRectData;
68 68
69 rect_width_ = message->update_stream_packet().begin_rect().width(); 69 rect_width_ = message->update_stream_packet().begin_rect().width();
70 rect_height_ = message->update_stream_packet().begin_rect().height(); 70 rect_height_ = message->update_stream_packet().begin_rect().height();
71 rect_x_ = message->update_stream_packet().begin_rect().x(); 71 rect_x_ = message->update_stream_packet().begin_rect().x();
72 rect_y_ = message->update_stream_packet().begin_rect().y(); 72 rect_y_ = message->update_stream_packet().begin_rect().y();
73 73
74 PixelFormat pixel_format = 74 PixelFormat pixel_format =
75 message->update_stream_packet().begin_rect().pixel_format(); 75 message->update_stream_packet().begin_rect().pixel_format();
76 76
77 if (static_cast<PixelFormat>(frame_->format()) != pixel_format) { 77 if (static_cast<PixelFormat>(frame_->format()) != pixel_format) {
78 NOTREACHED() << "Pixel format of message doesn't match the video frame. " 78 NOTREACHED() << "Pixel format of message doesn't match the video frame. "
79 "Expected vs received = " 79 "Expected vs received = "
80 << frame_->format() << " vs " << pixel_format 80 << frame_->format() << " vs " << pixel_format
81 << " Color space conversion required."; 81 << " Color space conversion required.";
82 return false; 82 return false;
83 } 83 }
84 84
85 bytes_per_pixel_ = GetBytesPerPixel(pixel_format); 85 bytes_per_pixel_ = GetBytesPerPixel(pixel_format);
86 return true; 86 return true;
87 } 87 }
88 88
89 bool DecoderVerbatim::HandleRectData(ChromotingHostMessage* message) { 89 bool DecoderVerbatim::HandleRectData(HostMessage* message) {
90 DCHECK_EQ(kWaitingForRectData, state_); 90 DCHECK_EQ(kWaitingForRectData, state_);
91 DCHECK_EQ(0, 91 DCHECK_EQ(0,
92 message->update_stream_packet().rect_data().sequence_number()); 92 message->update_stream_packet().rect_data().sequence_number());
93 93
94 // Copy the data line by line. 94 // Copy the data line by line.
95 const int src_stride = bytes_per_pixel_ * rect_width_; 95 const int src_stride = bytes_per_pixel_ * rect_width_;
96 const char* src = 96 const char* src =
97 message->update_stream_packet().rect_data().data().c_str(); 97 message->update_stream_packet().rect_data().data().c_str();
98 int src_stride_dir = src_stride; 98 int src_stride_dir = src_stride;
99 if (reverse_rows_) { 99 if (reverse_rows_) {
(...skipping 11 matching lines...) Expand all
111 src += src_stride_dir; 111 src += src_stride_dir;
112 } 112 }
113 113
114 updated_rects_->clear(); 114 updated_rects_->clear();
115 updated_rects_->push_back(gfx::Rect(rect_x_, rect_y_, 115 updated_rects_->push_back(gfx::Rect(rect_x_, rect_y_,
116 rect_width_, rect_height_)); 116 rect_width_, rect_height_));
117 partial_decode_done_->Run(); 117 partial_decode_done_->Run();
118 return true; 118 return true;
119 } 119 }
120 120
121 bool DecoderVerbatim::HandleEndRect(ChromotingHostMessage* message) { 121 bool DecoderVerbatim::HandleEndRect(HostMessage* message) {
122 DCHECK_EQ(kWaitingForRectData, state_); 122 DCHECK_EQ(kWaitingForRectData, state_);
123 state_ = kWaitingForBeginRect; 123 state_ = kWaitingForBeginRect;
124 return true; 124 return true;
125 } 125 }
126 126
127 } // namespace remoting 127 } // namespace remoting
OLDNEW
« no previous file with comments | « remoting/base/decoder_verbatim.h ('k') | remoting/base/decoder_verbatim_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698