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

Side by Side Diff: third_party/WebKit/Source/core/inspector/InspectorAnimationAgent.cpp

Issue 2883353002: Replace ASSERT with DCHECK_LE/GE/GT/LT/NE as appropriate (Closed)
Patch Set: DCHECK in BluetoothCharacteristicProperties.cpp 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 "core/inspector/InspectorAnimationAgent.h" 5 #include "core/inspector/InspectorAnimationAgent.h"
6 6
7 #include <memory> 7 #include <memory>
8 #include "bindings/core/v8/V8BindingForCore.h" 8 #include "bindings/core/v8/V8BindingForCore.h"
9 #include "core/animation/Animation.h" 9 #include "core/animation/Animation.h"
10 #include "core/animation/AnimationEffectReadOnly.h" 10 #include "core/animation/AnimationEffectReadOnly.h"
(...skipping 447 matching lines...) Expand 10 before | Expand all | Expand 10 after
458 static void AddStringToDigestor(WebCryptoDigestor* digestor, 458 static void AddStringToDigestor(WebCryptoDigestor* digestor,
459 const String& string) { 459 const String& string) {
460 digestor->Consume( 460 digestor->Consume(
461 reinterpret_cast<const unsigned char*>(string.Ascii().data()), 461 reinterpret_cast<const unsigned char*>(string.Ascii().data()),
462 string.length()); 462 string.length());
463 } 463 }
464 464
465 String InspectorAnimationAgent::CreateCSSId(blink::Animation& animation) { 465 String InspectorAnimationAgent::CreateCSSId(blink::Animation& animation) {
466 String type = 466 String type =
467 id_to_animation_type_.at(String::Number(animation.SequenceNumber())); 467 id_to_animation_type_.at(String::Number(animation.SequenceNumber()));
468 ASSERT(type != AnimationType::WebAnimation); 468 DCHECK_NE(type, AnimationType::WebAnimation);
469 469
470 KeyframeEffectReadOnly* effect = ToKeyframeEffectReadOnly(animation.effect()); 470 KeyframeEffectReadOnly* effect = ToKeyframeEffectReadOnly(animation.effect());
471 Vector<CSSPropertyID> css_properties; 471 Vector<CSSPropertyID> css_properties;
472 if (type == AnimationType::CSSAnimation) { 472 if (type == AnimationType::CSSAnimation) {
473 for (CSSPropertyID property : g_animation_properties) 473 for (CSSPropertyID property : g_animation_properties)
474 css_properties.push_back(property); 474 css_properties.push_back(property);
475 } else { 475 } else {
476 for (CSSPropertyID property : g_transition_properties) 476 for (CSSPropertyID property : g_transition_properties)
477 css_properties.push_back(property); 477 css_properties.push_back(property);
478 css_properties.push_back(cssPropertyID(animation.id())); 478 css_properties.push_back(cssPropertyID(animation.id()));
(...skipping 85 matching lines...) Expand 10 before | Expand all | Expand 10 after
564 564
565 DEFINE_TRACE(InspectorAnimationAgent) { 565 DEFINE_TRACE(InspectorAnimationAgent) {
566 visitor->Trace(inspected_frames_); 566 visitor->Trace(inspected_frames_);
567 visitor->Trace(css_agent_); 567 visitor->Trace(css_agent_);
568 visitor->Trace(id_to_animation_); 568 visitor->Trace(id_to_animation_);
569 visitor->Trace(id_to_animation_clone_); 569 visitor->Trace(id_to_animation_clone_);
570 InspectorBaseAgent::Trace(visitor); 570 InspectorBaseAgent::Trace(visitor);
571 } 571 }
572 572
573 } // namespace blink 573 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698