Changeset 87618 in webkit
- Timestamp:
- May 28, 2011 11:18:49 AM (13 years ago)
- Location:
- trunk
- Files:
-
- 2 added
- 7 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/LayoutTests/ChangeLog
r87617 r87618 1 2011-05-28 Gavin Peters <gavinp@chromium.org> 2 3 Reviewed by Adam Barth. 4 5 prevent HTMLLinkElement from watching multiple CachedResources 6 https://bugs.webkit.org/show_bug.cgi?id=61686 7 8 If we modify an existing link element, stop listening to the previous 9 cached resource, to prevent double notifications (which crash). 10 11 * fast/dom/HTMLLinkElement/prefetch-too-many-clients-expected.txt: Added. 12 * fast/dom/HTMLLinkElement/prefetch-too-many-clients.html: Added. 13 * platform/gtk/Skipped: 14 * platform/mac/Skipped: 15 * platform/qt/Skipped: 16 * platform/win/Skipped: 17 1 18 2011-05-28 Tonis Tiigi <tonistiigi@gmail.com> 2 19 -
trunk/LayoutTests/platform/gtk/Skipped
r87542 r87618 943 943 fast/dom/HTMLLinkElement/link-and-subresource-test.html 944 944 fast/dom/HTMLLinkElement/prefetch.html 945 fast/dom/HTMLLinkElement/prefetch-beforeload.html 945 946 fast/dom/HTMLLinkElement/prefetch-onerror.html 946 947 fast/dom/HTMLLinkElement/prefetch-onload.html 947 fast/dom/HTMLLinkElement/prefetch- beforeload.html948 fast/dom/HTMLLinkElement/prefetch-too-many-clients.html 948 949 fast/dom/HTMLLinkElement/prerender.html 949 950 fast/dom/HTMLLinkElement/subresource.html -
trunk/LayoutTests/platform/mac/Skipped
r87423 r87618 183 183 fast/dom/HTMLLinkElement/prefetch-onerror.html 184 184 fast/dom/HTMLLinkElement/prefetch-onload.html 185 fast/dom/HTMLLinkElement/prefetch-too-many-clients.html 185 186 fast/dom/HTMLLinkElement/prerender.html 186 187 fast/dom/HTMLLinkElement/subresource.html -
trunk/LayoutTests/platform/qt/Skipped
r87503 r87618 1949 1949 fast/dom/HTMLLinkElement/prefetch-onerror.html 1950 1950 fast/dom/HTMLLinkElement/prefetch-onload.html 1951 fast/dom/HTMLLinkElement/prefetch-too-many-clients.html 1951 1952 fast/dom/HTMLLinkElement/prerender.html 1952 1953 fast/dom/HTMLLinkElement/subresource.html -
trunk/LayoutTests/platform/win/Skipped
r87511 r87618 1008 1008 fast/dom/HTMLLinkElement/prefetch-onerror.html 1009 1009 fast/dom/HTMLLinkElement/prefetch-onload.html 1010 fast/dom/HTMLLinkElement/prefetch-too-many-clients.html 1010 1011 http/tests/misc/link-rel-prefetch-and-subresource.html 1011 1012 fast/dom/HTMLLinkElement/prerender.html -
trunk/Source/WebCore/ChangeLog
r87617 r87618 1 2011-05-28 Gavin Peters <gavinp@chromium.org> 2 3 Reviewed by Adam Barth. 4 5 prevent HTMLLinkElement from watching multiple CachedResources 6 https://bugs.webkit.org/show_bug.cgi?id=61686 7 8 If we modify an existing link element, stop listening to the previous 9 cached resource, to prevent double notifications (which crash). 10 11 Test: fast/dom/HTMLLinkElement/prefetch-too-many-clients.html 12 13 * html/HTMLLinkElement.cpp: 14 (WebCore::HTMLLinkElement::process): 15 1 16 2011-05-28 Tonis Tiigi <tonistiigi@gmail.com> 2 17 -
trunk/Source/WebCore/html/HTMLLinkElement.cpp
r87239 r87618 267 267 268 268 ResourceRequest linkRequest(document()->completeURL(m_url)); 269 270 if (m_cachedLinkResource) { 271 m_cachedLinkResource->removeClient(this); 272 m_cachedLinkResource = 0; 273 } 269 274 m_cachedLinkResource = document()->cachedResourceLoader()->requestLinkResource(type, linkRequest, priority); 270 275 if (m_cachedLinkResource)
Note: See TracChangeset
for help on using the changeset viewer.