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

Side by Side Diff: media/remoting/rpc/proto_enum_utils.cc

Issue 2571163002: Add PIXEL_FORMAT_I422. (Closed)
Patch Set: move switch cases to align with the enum value order Created 4 years 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
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 "media/remoting/rpc/proto_enum_utils.h" 5 #include "media/remoting/rpc/proto_enum_utils.h"
6 6
7 namespace media { 7 namespace media {
8 namespace remoting { 8 namespace remoting {
9 9
10 #define CASE_RETURN_OTHER(x) \ 10 #define CASE_RETURN_OTHER(x) \
(...skipping 317 matching lines...) Expand 10 before | Expand all | Expand 10 after
328 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV420P10); 328 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV420P10);
329 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV422P9); 329 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV422P9);
330 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV422P10); 330 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV422P10);
331 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV444P9); 331 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV444P9);
332 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV444P10); 332 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV444P10);
333 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV420P12); 333 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV420P12);
334 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV422P12); 334 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV422P12);
335 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV444P12); 335 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV444P12);
336 CASE_RETURN_OTHER(PIXEL_FORMAT_Y8); 336 CASE_RETURN_OTHER(PIXEL_FORMAT_Y8);
337 CASE_RETURN_OTHER(PIXEL_FORMAT_Y16); 337 CASE_RETURN_OTHER(PIXEL_FORMAT_Y16);
338 CASE_RETURN_OTHER(PIXEL_FORMAT_I422);
338 } 339 }
339 return base::nullopt; // Not a 'default' to ensure compile-time checks. 340 return base::nullopt; // Not a 'default' to ensure compile-time checks.
340 } 341 }
341 342
342 base::Optional<pb::VideoDecoderConfig::Format> ToProtoVideoDecoderConfigFormat( 343 base::Optional<pb::VideoDecoderConfig::Format> ToProtoVideoDecoderConfigFormat(
343 ::media::VideoPixelFormat value) { 344 ::media::VideoPixelFormat value) {
344 using OriginType = ::media::VideoPixelFormat; 345 using OriginType = ::media::VideoPixelFormat;
345 using OtherType = pb::VideoDecoderConfig; 346 using OtherType = pb::VideoDecoderConfig;
346 switch (value) { 347 switch (value) {
347 CASE_RETURN_OTHER(PIXEL_FORMAT_UNKNOWN); 348 CASE_RETURN_OTHER(PIXEL_FORMAT_UNKNOWN);
(...skipping 16 matching lines...) Expand all
364 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV420P10); 365 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV420P10);
365 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV422P9); 366 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV422P9);
366 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV422P10); 367 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV422P10);
367 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV444P9); 368 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV444P9);
368 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV444P10); 369 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV444P10);
369 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV420P12); 370 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV420P12);
370 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV422P12); 371 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV422P12);
371 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV444P12); 372 CASE_RETURN_OTHER(PIXEL_FORMAT_YUV444P12);
372 CASE_RETURN_OTHER(PIXEL_FORMAT_Y8); 373 CASE_RETURN_OTHER(PIXEL_FORMAT_Y8);
373 CASE_RETURN_OTHER(PIXEL_FORMAT_Y16); 374 CASE_RETURN_OTHER(PIXEL_FORMAT_Y16);
375 CASE_RETURN_OTHER(PIXEL_FORMAT_I422);
374 } 376 }
375 return base::nullopt; // Not a 'default' to ensure compile-time checks. 377 return base::nullopt; // Not a 'default' to ensure compile-time checks.
376 } 378 }
377 379
378 base::Optional<::media::ColorSpace> ToMediaColorSpace( 380 base::Optional<::media::ColorSpace> ToMediaColorSpace(
379 pb::VideoDecoderConfig::ColorSpace value) { 381 pb::VideoDecoderConfig::ColorSpace value) {
380 using OriginType = pb::VideoDecoderConfig; 382 using OriginType = pb::VideoDecoderConfig;
381 using OtherType = ::media::ColorSpace; 383 using OtherType = ::media::ColorSpace;
382 switch (value) { 384 switch (value) {
383 CASE_RETURN_OTHER(COLOR_SPACE_UNSPECIFIED); 385 CASE_RETURN_OTHER(COLOR_SPACE_UNSPECIFIED);
(...skipping 196 matching lines...) Expand 10 before | Expand all | Expand 10 after
580 switch (value) { 582 switch (value) {
581 CASE_RETURN_OTHER(kOk); 583 CASE_RETURN_OTHER(kOk);
582 CASE_RETURN_OTHER(kAborted); 584 CASE_RETURN_OTHER(kAborted);
583 CASE_RETURN_OTHER(kConfigChanged); 585 CASE_RETURN_OTHER(kConfigChanged);
584 } 586 }
585 return base::nullopt; // Not a 'default' to ensure compile-time checks. 587 return base::nullopt; // Not a 'default' to ensure compile-time checks.
586 } 588 }
587 589
588 } // namespace remoting 590 } // namespace remoting
589 } // namespace media 591 } // namespace media
OLDNEW
« no previous file with comments | « media/remoting/proto/remoting_rpc_message.proto ('k') | media/renderers/skcanvas_video_renderer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698