Changeset 220750 in webkit
- Timestamp:
- Aug 15, 2017 11:13:57 AM (7 years ago)
- Location:
- trunk
- Files:
-
- 4 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/LayoutTests/ChangeLog
r220747 r220750 1 2017-08-15 Darin Adler <darin@apple.com> 2 3 REGRESSION(r220052): http/tests/appcache/deferred-events-delete-while-raising-timer.html is crashing. 4 https://bugs.webkit.org/show_bug.cgi?id=175107 5 6 Reviewed by Alexey Proskuryakov. 7 8 * TestExpectations: Removed the flakiness expectation for this test. 9 1 10 2017-08-15 Michael Catanzaro <mcatanzaro@igalia.com> 2 11 -
trunk/LayoutTests/TestExpectations
r220747 r220750 706 706 webkit.org/b/87633 http/tests/appcache/abort-cache-onchecking-manifest-404.html [ Pass Failure ] 707 707 webkit.org/b/87633 http/tests/appcache/abort-cache-ondownloading-resource-404.html [ Pass Failure ] 708 webkit.org/b/175107 http/tests/appcache/deferred-events-delete-while-raising-timer.html [ Pass Failure Crash ]709 708 webkit.org/b/87633 http/tests/appcache/deferred-events-delete-while-raising.html [ Pass Failure ] 710 709 webkit.org/b/87633 http/tests/appcache/deferred-events.html [ Pass Failure ] -
trunk/Source/WebCore/ChangeLog
r220746 r220750 1 2017-08-15 Darin Adler <darin@apple.com> 2 3 REGRESSION(r220052): http/tests/appcache/deferred-events-delete-while-raising-timer.html is crashing. 4 https://bugs.webkit.org/show_bug.cgi?id=175107 5 6 Reviewed by Alexey Proskuryakov. 7 8 * loader/FrameLoader.cpp: 9 (WebCore::FrameLoader::checkLoadComplete): Moved the assertion to after the null page check. 10 1 11 2017-08-15 Commit Queue <commit-queue@webkit.org> 2 12 -
trunk/Source/WebCore/loader/FrameLoader.cpp
r220052 r220750 2500 2500 void FrameLoader::checkLoadComplete() 2501 2501 { 2502 m_shouldCallCheckLoadComplete = false; 2503 2504 if (!m_frame.page()) 2505 return; 2506 2502 2507 ASSERT(m_client.hasWebView()); 2503 2508 2504 m_shouldCallCheckLoadComplete = false;2505 2506 if (!m_frame.page())2507 return;2508 2509 2509 // FIXME: Always traversing the entire frame tree is a bit inefficient, but 2510 2510 // is currently needed in order to null out the previous history item for all frames.
Note: See TracChangeset
for help on using the changeset viewer.