Remove stale commented-out DCHECK

third_party/WebKit/Source/core/animation/css/CSSAnimations.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 738 matching lines @@
   if (animatingElement->document().printing() ||
       animatingElement->document().wasPrinting())
     return;
 
   ElementAnimations* elementAnimations = animatingElement->elementAnimations();
   const TransitionMap* activeTransitions =
       elementAnimations ? &elementAnimations->cssAnimations().m_transitions
                         : nullptr;
   const CSSTransitionData* transitionData = style.transitions();
 
-#if DCHECK_IS_ON()
-  // In debug builds we verify that it would have been safe to avoid populating
-  // and testing listedProperties if the style recalc is due to animation.
-  const bool animationStyleRecalc = false;
-#else
-  // In release builds we avoid the cost of checking for new and interrupted
-  // transitions if the style recalc is due to animation.
   const bool animationStyleRecalc =
       elementAnimations && elementAnimations->isAnimationStyleChange();
-#endif
 
   std::bitset<numCSSProperties> listedProperties;
   bool anyTransitionHadTransitionAll = false;
   const LayoutObject* layoutObject = animatingElement->layoutObject();
   if (!animationStyleRecalc && style.display() != EDisplay::None &&
       layoutObject && layoutObject->style() && transitionData) {
     const ComputedStyle& oldStyle = *layoutObject->style();
 
     for (size_t i = 0; i < transitionData->propertyList().size(); ++i) {
       const CSSTransitionData::TransitionProperty& transitionProperty =
@@ skipping 39 matching lines @@
         }
       }
     }
   }
 
   if (activeTransitions) {
     for (const auto& entry : *activeTransitions) {
       CSSPropertyID id = entry.key;
       if (!anyTransitionHadTransitionAll && !animationStyleRecalc &&
           !listedProperties.test(id - firstCSSProperty)) {
-        // TODO(crbug.com/365507): Figure out why this fails on Chrome OS login
-        // page.
-        // DCHECK(animation.playStateInternal() == Animation::Finished ||
-        //       !(elementAnimations &&
-        //         elementAnimations->isAnimationStyleChange()));
         update.cancelTransition(id);
       } else if (entry.value.animation->finishedInternal()) {
         update.finishTransition(id);
       }
     }
   }
 }
 
 void CSSAnimations::cancel() {
   for (const auto& runningAnimation : m_runningAnimations) {
@@ skipping 254 matching lines @@
   return toShadowRoot(treeScope.rootNode()).host() == element;
 }
 
 DEFINE_TRACE(CSSAnimations) {
   visitor->trace(m_transitions);
   visitor->trace(m_pendingUpdate);
   visitor->trace(m_runningAnimations);
 }
 
 }  // namespace blink