Changeset 251768 in webkit
- Timestamp:
- Oct 29, 2019 10:12:28 PM (4 years ago)
- Location:
- trunk/Source/WebCore
- Files:
-
- 4 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/Source/WebCore/ChangeLog
r251765 r251768 1 2019-10-29 Ryosuke Niwa <rniwa@webkit.org> 2 3 Remove HTMLMediaElementDestructorScope 4 https://bugs.webkit.org/show_bug.cgi?id=203600 5 6 Reviewed by Chris Dumez. 7 8 Removed the workaround introduced in r231392 for the bug 185284 now that it's been fixed. 9 10 No new tests since this code should be dead code. 11 12 * html/HTMLMediaElement.cpp: 13 (WebCore::HTMLMediaElement::~HTMLMediaElement): 14 (WebCore::HTMLMediaElement::isRunningDestructor): Deleted. 15 (WebCore::HTMLMediaElementDestructorScope::HTMLMediaElementDestructorScope): Deleted. 16 (WebCore::HTMLMediaElementDestructorScope::~HTMLMediaElementDestructorScope): Deleted. 17 * html/HTMLMediaElement.h: 18 * loader/FrameLoader.cpp: 19 (WebCore::FrameLoader::checkCompleted): 20 1 21 2019-10-29 Andy Estes <aestes@apple.com> 2 22 -
trunk/Source/WebCore/html/HTMLMediaElement.cpp
r251737 r251768 555 555 } 556 556 557 // FIXME: Remove this code once https://webkit.org/b/185284 is fixed.558 static unsigned s_destructorCount = 0;559 560 bool HTMLMediaElement::isRunningDestructor()561 {562 return !!s_destructorCount;563 }564 565 class HTMLMediaElementDestructorScope {566 public:567 HTMLMediaElementDestructorScope() { ++s_destructorCount; }568 ~HTMLMediaElementDestructorScope() { --s_destructorCount; }569 };570 571 557 HTMLMediaElement::~HTMLMediaElement() 572 558 { 573 HTMLMediaElementDestructorScope destructorScope;574 559 ALWAYS_LOG(LOGIDENTIFIER); 575 560 -
trunk/Source/WebCore/html/HTMLMediaElement.h
r251737 r251768 167 167 WEBCORE_EXPORT static RefPtr<HTMLMediaElement> bestMediaElementForShowingPlaybackControlsManager(MediaElementSession::PlaybackControlsPurpose); 168 168 169 static bool isRunningDestructor();170 171 169 WEBCORE_EXPORT void rewind(double timeDelta); 172 170 WEBCORE_EXPORT void returnToRealtime() override; -
trunk/Source/WebCore/loader/FrameLoader.cpp
r251220 r251768 843 843 if (m_isComplete) 844 844 return; 845 846 #if ENABLE(VIDEO)847 // FIXME: Remove this code once https://webkit.org/b/185284 is fixed.848 if (HTMLMediaElement::isRunningDestructor()) {849 ASSERT_NOT_REACHED();850 scheduleCheckCompleted();851 return;852 }853 #endif854 845 855 846 // FIXME: It would be better if resource loads were kicked off after render tree update (or didn't complete synchronously).
Note: See TracChangeset
for help on using the changeset viewer.