Remove TextTrackCue::notifyRegionWhenRemovingDisplayTree

Source/core/html/track/vtt/VTTCue.cpp

 /*
  * Copyright (c) 2013, Opera Software ASA. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 211 matching lines @@
     , m_text(text)
     , m_linePosition(std::numeric_limits<float>::quiet_NaN())
     , m_textPosition(std::numeric_limits<float>::quiet_NaN())
     , m_cueSize(100)
     , m_writingDirection(Horizontal)
     , m_cueAlignment(Middle)
     , m_vttNodeTree(nullptr)
     , m_cueBackgroundBox(HTMLDivElement::create(document))
     , m_snapToLines(true)
     , m_displayTreeShouldChange(true)
-    , m_notifyRegion(true)
 {
     UseCounter::count(document, UseCounter::VTTCue);
 }
 
 VTTCue::~VTTCue()
 {
     // Using oilpan, if m_displayTree is in the document it will strongly keep
     // the cue alive. Thus, if the cue is dead, either m_displayTree is not in
     // the document or the entire document is dead too.
 #if !ENABLE(OILPAN)
@@ skipping 250 matching lines @@
 void VTTCue::setRegionId(const String& regionId)
 {
     if (m_regionId == regionId)
         return;
 
     cueWillChange();
     m_regionId = regionId;
     cueDidChange();
 }
 
-void VTTCue::notifyRegionWhenRemovingDisplayTree(bool notifyRegion)
-{
-    m_notifyRegion = notifyRegion;
-}
-
 float VTTCue::calculateComputedLinePosition() const
 {
     // http://dev.w3.org/html5/webvtt/#dfn-text-track-cue-computed-line-position
     // A text track cue has a text track cue computed line position whose value
     // is that returned by the following algorithm, which is defined in terms
     // of the other aspects of the cue:
 
     // 1. If the text track cue line position is numeric, the text track cue
     //    snap-to-lines flag of the text track cue is not set, and the text
     //    track cue line position is negative or greater than 100, then return
@@ skipping 324 matching lines @@
     // normally would in CSS, it is instead forcibly wrapped at the box's edge.)
     displayTree->applyCSSProperties(displayParameters);
 
     m_displayTreeShouldChange = false;
 
     // 10.15. Let cue's text track cue display state have the CSS boxes in
     // boxes.
     return displayTree.release();
 }
 
-void VTTCue::removeDisplayTree()
+void VTTCue::removeDisplayTree(RemovalNotification removalNotification)
 {
-    if (m_notifyRegion && track()->regions()) {
+    if (removalNotification == NotifyRegion && track()->regions()) {
         // The region needs to be informed about the cue removal.
         VTTRegion* region = track()->regions()->getRegionById(m_regionId);
         if (region)
             region->willRemoveVTTCueBox(m_displayTree.get());
     }
 
     if (m_displayTree)
         m_displayTree->remove(ASSERT_NO_EXCEPTION);
 }
 
@@ skipping 263 matching lines @@
 
 DEFINE_TRACE(VTTCue)
 {
     visitor->trace(m_vttNodeTree);
     visitor->trace(m_cueBackgroundBox);
     visitor->trace(m_displayTree);
     TextTrackCue::trace(visitor);
 }
 
 } // namespace blink