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

Side by Side Diff: third_party/WebKit/Source/modules/presentation/PresentationAvailability.cpp

Issue 2862963003: Replace ASSERT with DCHECK in modules/ (Closed)
Patch Set: NOTREACHED instead of DCHECK(false) Created 3 years, 7 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 "modules/presentation/PresentationAvailability.h" 5 #include "modules/presentation/PresentationAvailability.h"
6 6
7 #include "bindings/core/v8/ScriptPromiseResolver.h" 7 #include "bindings/core/v8/ScriptPromiseResolver.h"
8 #include "core/dom/Document.h" 8 #include "core/dom/Document.h"
9 #include "core/events/Event.h" 9 #include "core/events/Event.h"
10 #include "core/frame/UseCounter.h" 10 #include "core/frame/UseCounter.h"
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 47
48 PresentationAvailability::PresentationAvailability( 48 PresentationAvailability::PresentationAvailability(
49 ExecutionContext* execution_context, 49 ExecutionContext* execution_context,
50 const WTF::Vector<KURL>& urls, 50 const WTF::Vector<KURL>& urls,
51 bool value) 51 bool value)
52 : SuspendableObject(execution_context), 52 : SuspendableObject(execution_context),
53 PageVisibilityObserver(ToDocument(execution_context)->GetPage()), 53 PageVisibilityObserver(ToDocument(execution_context)->GetPage()),
54 urls_(urls), 54 urls_(urls),
55 value_(value), 55 value_(value),
56 state_(State::kActive) { 56 state_(State::kActive) {
57 ASSERT(execution_context->IsDocument()); 57 DCHECK(execution_context->IsDocument());
58 WebVector<WebURL> data(urls.size()); 58 WebVector<WebURL> data(urls.size());
59 for (size_t i = 0; i < urls.size(); ++i) 59 for (size_t i = 0; i < urls.size(); ++i)
60 data[i] = WebURL(urls[i]); 60 data[i] = WebURL(urls[i]);
61 61
62 urls_.Swap(data); 62 urls_.Swap(data);
63 } 63 }
64 64
65 PresentationAvailability::~PresentationAvailability() {} 65 PresentationAvailability::~PresentationAvailability() {}
66 66
67 const AtomicString& PresentationAvailability::InterfaceName() const { 67 const AtomicString& PresentationAvailability::InterfaceName() const {
(...skipping 70 matching lines...) Expand 10 before | Expand all | Expand 10 after
138 return value_; 138 return value_;
139 } 139 }
140 140
141 DEFINE_TRACE(PresentationAvailability) { 141 DEFINE_TRACE(PresentationAvailability) {
142 EventTargetWithInlineData::Trace(visitor); 142 EventTargetWithInlineData::Trace(visitor);
143 PageVisibilityObserver::Trace(visitor); 143 PageVisibilityObserver::Trace(visitor);
144 SuspendableObject::Trace(visitor); 144 SuspendableObject::Trace(visitor);
145 } 145 }
146 146
147 } // namespace blink 147 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698