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

Side by Side Diff: content/renderer/gpu/queue_message_swap_promise.cc

Issue 842523002: base: Change DCHECK_IS_ON to a macro DCHECK_IS_ON(). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: dcheck2: withoutandroidchange Created 5 years, 11 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
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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/renderer/gpu/queue_message_swap_promise.h" 5 #include "content/renderer/gpu/queue_message_swap_promise.h"
6 6
7 #include "content/renderer/gpu/frame_swap_message_queue.h" 7 #include "content/renderer/gpu/frame_swap_message_queue.h"
8 #include "ipc/ipc_sync_message_filter.h" 8 #include "ipc/ipc_sync_message_filter.h"
9 9
10 namespace content { 10 namespace content {
11 11
12 QueueMessageSwapPromise::QueueMessageSwapPromise( 12 QueueMessageSwapPromise::QueueMessageSwapPromise(
13 scoped_refptr<IPC::SyncMessageFilter> message_sender, 13 scoped_refptr<IPC::SyncMessageFilter> message_sender,
14 scoped_refptr<content::FrameSwapMessageQueue> message_queue, 14 scoped_refptr<content::FrameSwapMessageQueue> message_queue,
15 int source_frame_number) 15 int source_frame_number)
16 : message_sender_(message_sender), 16 : message_sender_(message_sender),
17 message_queue_(message_queue), 17 message_queue_(message_queue),
18 source_frame_number_(source_frame_number) 18 source_frame_number_(source_frame_number)
19 #if DCHECK_IS_ON 19 #if DCHECK_IS_ON()
20 , 20 ,
21 completed_(false) 21 completed_(false)
22 #endif 22 #endif
23 { 23 {
24 DCHECK(message_sender_.get()); 24 DCHECK(message_sender_.get());
25 DCHECK(message_queue_.get()); 25 DCHECK(message_queue_.get());
26 } 26 }
27 27
28 QueueMessageSwapPromise::~QueueMessageSwapPromise() { 28 QueueMessageSwapPromise::~QueueMessageSwapPromise() {
29 // The promise should have either been kept or broken before it's deleted. 29 // The promise should have either been kept or broken before it's deleted.
30 #if DCHECK_IS_ON 30 #if DCHECK_IS_ON()
31 DCHECK(completed_); 31 DCHECK(completed_);
32 #endif 32 #endif
33 } 33 }
34 34
35 void QueueMessageSwapPromise::DidSwap(cc::CompositorFrameMetadata* metadata) { 35 void QueueMessageSwapPromise::DidSwap(cc::CompositorFrameMetadata* metadata) {
36 #if DCHECK_IS_ON 36 #if DCHECK_IS_ON()
37 DCHECK(!completed_); 37 DCHECK(!completed_);
38 #endif 38 #endif
39 message_queue_->DidSwap(source_frame_number_); 39 message_queue_->DidSwap(source_frame_number_);
40 // The OutputSurface will take care of the Drain+Send. 40 // The OutputSurface will take care of the Drain+Send.
41 PromiseCompleted(); 41 PromiseCompleted();
42 } 42 }
43 43
44 void QueueMessageSwapPromise::DidNotSwap(DidNotSwapReason reason) { 44 void QueueMessageSwapPromise::DidNotSwap(DidNotSwapReason reason) {
45 #if DCHECK_IS_ON 45 #if DCHECK_IS_ON()
46 DCHECK(!completed_); 46 DCHECK(!completed_);
47 #endif 47 #endif
48 ScopedVector<IPC::Message> messages; 48 ScopedVector<IPC::Message> messages;
49 message_queue_->DidNotSwap(source_frame_number_, reason, &messages); 49 message_queue_->DidNotSwap(source_frame_number_, reason, &messages);
50 for (ScopedVector<IPC::Message>::iterator i = messages.begin(); 50 for (ScopedVector<IPC::Message>::iterator i = messages.begin();
51 i != messages.end(); 51 i != messages.end();
52 ++i) { 52 ++i) {
53 message_sender_->Send(*i); 53 message_sender_->Send(*i);
54 } 54 }
55 messages.weak_clear(); 55 messages.weak_clear();
56 PromiseCompleted(); 56 PromiseCompleted();
57 } 57 }
58 58
59 void QueueMessageSwapPromise::PromiseCompleted() { 59 void QueueMessageSwapPromise::PromiseCompleted() {
60 #if DCHECK_IS_ON 60 #if DCHECK_IS_ON()
61 completed_ = true; 61 completed_ = true;
62 #endif 62 #endif
63 } 63 }
64 64
65 int64 QueueMessageSwapPromise::TraceId() const { 65 int64 QueueMessageSwapPromise::TraceId() const {
66 return 0; 66 return 0;
67 } 67 }
68 68
69 } // namespace content 69 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698