Chromium Code Reviews

Side by Side Diff: content/common/gpu/media/mac_video_decode_accelerator.mm

Issue 11419224: Add missing (and remove superfluous) 'explicit' from constructors. Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase + remove non-straightforward changes Created 7 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "content/common/gpu/media/mac_video_decode_accelerator.h" 5 #include "content/common/gpu/media/mac_video_decode_accelerator.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/file_path.h" 8 #include "base/file_path.h"
9 #import "base/mac/foundation_util.h" 9 #import "base/mac/foundation_util.h"
10 #import "base/memory/ref_counted_memory.h" 10 #import "base/memory/ref_counted_memory.h"
(...skipping 16 matching lines...)
27 return ret_val; \ 27 return ret_val; \
28 } \ 28 } \
29 } while (0) 29 } while (0)
30 30
31 namespace { 31 namespace {
32 32
33 enum { kNumPictureBuffers = 4 }; 33 enum { kNumPictureBuffers = 4 };
34 34
35 class ScopedContextSetter { 35 class ScopedContextSetter {
36 public: 36 public:
37 ScopedContextSetter(CGLContextObj context) 37 explicit ScopedContextSetter(CGLContextObj context)
38 : old_context_(NULL), 38 : old_context_(NULL),
39 did_succeed_(false) { 39 did_succeed_(false) {
40 old_context_ = CGLGetCurrentContext(); 40 old_context_ = CGLGetCurrentContext();
41 did_succeed_ = CGLSetCurrentContext(context) == kCGLNoError; 41 did_succeed_ = CGLSetCurrentContext(context) == kCGLNoError;
42 } 42 }
43 43
44 ~ScopedContextSetter() { 44 ~ScopedContextSetter() {
45 if (did_succeed_) 45 if (did_succeed_)
46 CGLSetCurrentContext(old_context_); 46 CGLSetCurrentContext(old_context_);
47 } 47 }
(...skipping 333 matching lines...)
381 MacVideoDecodeAccelerator::UsedPictureInfo::~UsedPictureInfo() { 381 MacVideoDecodeAccelerator::UsedPictureInfo::~UsedPictureInfo() {
382 } 382 }
383 383
384 MacVideoDecodeAccelerator::DecodedImageInfo::DecodedImageInfo() { 384 MacVideoDecodeAccelerator::DecodedImageInfo::DecodedImageInfo() {
385 } 385 }
386 386
387 MacVideoDecodeAccelerator::DecodedImageInfo::~DecodedImageInfo() { 387 MacVideoDecodeAccelerator::DecodedImageInfo::~DecodedImageInfo() {
388 } 388 }
389 389
390 } // namespace content 390 } // namespace content
OLDNEW
« no previous file with comments | « content/common/gpu/gpu_memory_allocation.h ('k') | content/common/inter_process_time_ticks_converter.h » ('j') | no next file with comments »

Powered by Google App Engine