.: Expose an export for the iconUrl list so Internals can use it
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 10 Jul 2012 01:37:36 +0000 (01:37 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 10 Jul 2012 01:37:36 +0000 (01:37 +0000)
commit2c8d702814789ba675a6cf8f5c92a819ea260f40
tree7c42d7da8e318535e1ddca542c4c59639efa000d
parent1a0b1e27e22f5fbf71b678dec67a54f13227e9cb
.: Expose an export for the iconUrl list so Internals can use it
https://bugs.webkit.org/show_bug.cgi?id=88665

Patch by Pete Williamson <petewil@google.com> on 2012-07-09
Reviewed by Kent Tamura.

* Source/autotools/symbols.filter: export iconURLs

Source/WebCore: Changed the behavior of iconURLs to always recalculate the list.
https://bugs.webkit.org/show_bug.cgi?id=88665

Patch by Pete Williamson <petewil@google.com> on 2012-07-09
Reviewed by Kent Tamura..

As it turns out, it can contain stale URLs in the case that some script
manipulates the DOM, which breaks scripts trying to reset the favicon
URL. Also added a method in Internals to allow tests to get the list of
icon

Tests: fast/dom/icon-url-change.html
       fast/dom/icon-url-list.html

* WebCore.exp.in: export Document::iconURLs on the mac for the Internals class
* dom/Document.cpp:
(WebCore::Document::iconURLs): Changed the method to recalculate the iconURL list every time
(WebCore::Document::addIconURL): we no longer need to add to the internal list since we recalculate it
(WebCore::Document::setUseSecureKeyboardEntryWhenActive): removed extra whitespace
* dom/Document.h:
(Document): removed the addIconURL method which is no longer used
* html/HTMLLinkElement.cpp:
(WebCore::HTMLLinkElement::iconType): exposed the icon type with an accessor
(WebCore):
(WebCore::HTMLLinkElement::iconSizes): exposed the icon sizes with an accessor
* html/HTMLLinkElement.h:
(HTMLLinkElement): declared the icon type and size accessors
* testing/Internals.cpp:
(WebCore::Internals::iconURLs): made a method to be used by unit tests for inspecting the icon URL list
(WebCore):
* testing/Internals.h:
(Internals): declared the method for unit testing the icon URL list
* testing/Internals.idl: exported the Document::iconURLs function

Source/WebKit2: Export the iconURL list to make it available to the Internals class for testing
https://bugs.webkit.org/show_bug.cgi?id=88665

Patch by Pete Williamson <petewil@google.com> on 2012-07-09
Reviewed by Kent Tamura.

* win/WebKit2.def: export the DocumentL::iconURLs function

LayoutTests: Add some new unit tests to test the favicon changing dynamically
https://bugs.webkit.org/show_bug.cgi?id=88665

Patch by Pete Williamson <petewil@google.com> on 2012-07-09
Reviewed by Kent Tamura.

* fast/dom/icon-url-change-expected.txt: Added.
* fast/dom/icon-url-change.html: Added a new test for changing the favicon dynamically
* fast/dom/icon-url-list-expected.txt: Added.
* fast/dom/icon-url-list.html: Added a new test for multiple favicons in the HTML header
* fast/dom/icon-url-property-expected.txt: update unit test expectations
* fast/dom/icon-url-property.html: update and enable existing favicon test
* platform/chromium/TestExpectations: reenable the url-property test

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@122178 268f45cc-cd09-0410-ab3c-d52691b4dbfc
21 files changed:
ChangeLog
LayoutTests/ChangeLog
LayoutTests/fast/dom/icon-url-change-expected.txt [new file with mode: 0644]
LayoutTests/fast/dom/icon-url-change.html [new file with mode: 0644]
LayoutTests/fast/dom/icon-url-list-expected.txt [new file with mode: 0644]
LayoutTests/fast/dom/icon-url-list.html [new file with mode: 0644]
LayoutTests/fast/dom/icon-url-property-expected.txt
LayoutTests/fast/dom/icon-url-property.html
LayoutTests/platform/chromium/TestExpectations
Source/WebCore/ChangeLog
Source/WebCore/WebCore.exp.in
Source/WebCore/dom/Document.cpp
Source/WebCore/dom/Document.h
Source/WebCore/html/HTMLLinkElement.cpp
Source/WebCore/html/HTMLLinkElement.h
Source/WebCore/testing/Internals.cpp
Source/WebCore/testing/Internals.h
Source/WebCore/testing/Internals.idl
Source/WebKit2/ChangeLog
Source/WebKit2/win/WebKit2.def
Source/autotools/symbols.filter