kmccullough@apple.com [Wed, 27 Feb 2008 18:06:09 +0000 (18:06 +0000)]
Landing test that was forgotten in the original patch (r30087).
Test for: http://bugs.webkit.org/show_bug.cgi?id=17191
HTML5: Client-side database queries should return values of type number
* storage/sql-data-types-expected.txt: Added.
* storage/sql-data-types.html: Added.
* storage/sql-data-types.js: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30628
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap@webkit.org [Wed, 27 Feb 2008 17:52:05 +0000 (17:52 +0000)]
Add bug number to ChangeLog.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30627
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap@webkit.org [Wed, 27 Feb 2008 17:46:43 +0000 (17:46 +0000)]
2008-02-27 Julien Chaffraix <julien.chaffraix@gmail.com>
Reviewed by Alexey.
http://bugs.webkit.org/show_bug.cgi?id=17481
Several consecutive calls to XMLHttpRequest::open should dispatch only one readyState event
Test: http/tests/xmlhttprequest/xmlhttprequest-multiple-open.html
* xml/XMLHttpRequest.cpp:
(WebCore::XMLHttpRequest::open):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30626
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap@webkit.org [Wed, 27 Feb 2008 17:13:46 +0000 (17:13 +0000)]
Reviewed by Sam Weinig.
http://bugs.webkit.org/show_bug.cgi?id=17030
Small buffer overflow within initialization
* kjs/date_object.cpp:
(KJS::DateObjectFuncImp::callAsFunction):
(KJS::parseDate):
Remove unnecessary and incorrect memset() calls - GregorianDateTime can initialize itself.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30625
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap@webkit.org [Wed, 27 Feb 2008 16:50:19 +0000 (16:50 +0000)]
2008-02-27 Sriram Neelakandan <sriram.neelakandan@gmail.com>
Reviewed, tweaked and landed by ap.
http://bugs.webkit.org/show_bug.cgi?id=17487
HTMLInput mysteriously fails to work if ICU dat file is missing
* platform/text/TextBreakIteratorICU.cpp: (WebCore::setUpIterator): Assert that the iterator
has been opened.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30624
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
aroben@apple.com [Wed, 27 Feb 2008 16:38:57 +0000 (16:38 +0000)]
Attempt to fix the GTK+ build
* webkit/webkitwebframe.cpp:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30623
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
aroben@apple.com [Wed, 27 Feb 2008 16:34:58 +0000 (16:34 +0000)]
Fix revision number in ChangeLog
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30622
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
aroben@apple.com [Wed, 27 Feb 2008 16:16:02 +0000 (16:16 +0000)]
Windows build fix after r30613
* platform/win/ClipboardWin.cpp:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30621
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
zimmermann@webkit.org [Wed, 27 Feb 2008 15:12:23 +0000 (15:12 +0000)]
Reviewed by Simon.
Small style cleanup, and obvious fix: SVGGElement reacted on
'clipPathUnits' changes, which is a funny copy&paste error.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30620
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
zimmermann@webkit.org [Wed, 27 Feb 2008 12:38:26 +0000 (12:38 +0000)]
Reviewed by Simon.
Cleanup coding style. Still on my way to fix dynamic updates of all SVG elements.
Currently at SVGF*. SVGDefs/DescElement don't need any dyn update fixes but cleanups.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30619
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
antti@apple.com [Wed, 27 Feb 2008 06:14:56 +0000 (06:14 +0000)]
Reviewed by Darin.
Fix <rdar://problem/
5761326>
REGRESSION: 1.5% -2% Sunspider regression from r30009 (ebay photo upload hang)
Generate better code with VS:
- Use ALWAYS_INLINE macro to guarantee versions of allowsAccessFrom() are reduced to single functions
- Refactor error reporting so that there is no need to construct/destruct Strings in common cases
- Add a separate version of the function for cases where error messages are not wanted
* bindings/js/JSDOMWindowCustom.cpp:
(WebCore::JSDOMWindow::customGetOwnPropertySlot):
* bindings/js/kjs_window.cpp:
(KJS::Window::allowsAccessFrom):
(KJS::Window::allowsAccessFromNoErrorMessage):
(KJS::Window::allowsAccessFromPrivate):
(KJS::Window::crossDomainAccessErrorMessage):
* bindings/js/kjs_window.h:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30617
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
weinig@apple.com [Wed, 27 Feb 2008 04:14:18 +0000 (04:14 +0000)]
Reviewed by Dave Hyatt.
- remove DeprecatedCString, DeprecatedArray, and ArrayImpl.
* GNUmakefile.am:
* WebCore.pro:
* WebCore.vcproj/WebCore.vcproj:
* WebCore.xcodeproj/project.pbxproj:
* WebCoreSources.bkl:
* platform/ArrayImpl.cpp: Removed.
* platform/ArrayImpl.h: Removed.
* platform/DeprecatedArray.h: Removed.
* platform/DeprecatedCString.cpp: Removed.
* platform/DeprecatedCString.h: Removed.
* platform/text/CString.cpp:
(WebCore::CString::length):
* platform/text/CString.h:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30616
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
hyatt@apple.com [Wed, 27 Feb 2008 03:12:00 +0000 (03:12 +0000)]
Cleanup of childrenChanged in preparation for fixing the HTML5 spec slowdown.
Reviewed by Sam Weinig
* dom/Attr.cpp:
(WebCore::Attr::childrenChanged):
* dom/ContainerNode.cpp:
(WebCore::ContainerNode::insertBefore):
(WebCore::ContainerNode::replaceChild):
(WebCore::ContainerNode::removeChild):
(WebCore::ContainerNode::removeChildren):
(WebCore::ContainerNode::appendChild):
(WebCore::ContainerNode::addChild):
(WebCore::ContainerNode::childrenChanged):
* dom/ContainerNode.h:
* dom/EventTargetNode.cpp:
(WebCore::EventTargetNode::dispatchSubtreeModifiedEvent):
* dom/EventTargetNode.h:
* dom/NamedAttrMap.cpp:
(WebCore::NamedAttrMap::addAttribute):
(WebCore::NamedAttrMap::removeAttribute):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30615
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
kevino@webkit.org [Wed, 27 Feb 2008 01:56:38 +0000 (01:56 +0000)]
Improvements to the wx build script.
http://bugs.webkit.org/show_bug.cgi?id=17492
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30614
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Wed, 27 Feb 2008 01:43:52 +0000 (01:43 +0000)]
Reviewed by Sam.
- remove use of DeprecatedCString from Windows pasteboard code
* platform/win/ClipboardUtilitiesWin.cpp:
(WebCore::createGlobalData): Added an overload for Vector<char>.
Streamlined the logic a bit.
(WebCore::append): Added. Helper functions for appending to a
Vector<char>.
(WebCore::markupToCF_HTML): Rewrote to use Vector<char> instead
of DeprecatedCString. Also simplified the logic a bit and used
macros to handle the integer width.
(WebCore::urlToMarkup): Use Vector<UChar> instead of String.
(WebCore::replaceNewlinesWithWindowsStyleNewlines): Tweaked.
* platform/win/ClipboardUtilitiesWin.h: Update type of return
value from markupToCF_HTML. Did a bit of streamlining too.
* platform/win/ClipboardWin.cpp:
(WebCore::writeURL): Update for above changes.
(WebCore::ClipboardWin::declareAndWriteDragImage): Ditto.
(WebCore::ClipboardWin::writeRange): Ditto.
* platform/win/PasteboardWin.cpp:
(WebCore::Pasteboard::writeSelection): Ditto.
(WebCore::Pasteboard::writeURL): Ditto.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30613
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
aroben@apple.com [Wed, 27 Feb 2008 00:51:05 +0000 (00:51 +0000)]
Move ResourceLoadDelegate methods to WebFrameLoaderClient
Changed all methods to use early returns where possible and COMPtr's
AdoptCOM/Query constructors.
Reviewed by Anders.
* WebCoreSupport/WebFrameLoaderClient.cpp:
(WebFrameLoaderClient::assignIdentifierToInitialRequest):
(WebFrameLoaderClient::dispatchDidReceiveAuthenticationChallenge):
(WebFrameLoaderClient::dispatchDidCancelAuthenticationChallenge):
(WebFrameLoaderClient::dispatchWillSendRequest):
(WebFrameLoaderClient::dispatchDidReceiveResponse):
(WebFrameLoaderClient::dispatchDidReceiveContentLength):
(WebFrameLoaderClient::dispatchDidFinishLoading):
(WebFrameLoaderClient::dispatchDidFailLoading):
* WebCoreSupport/WebFrameLoaderClient.h:
* WebFrame.cpp:
* WebFrame.h:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30612
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
aroben@apple.com [Wed, 27 Feb 2008 00:50:36 +0000 (00:50 +0000)]
Move two more methods to WebFrameLoaderClient
Reviewed by Anders.
* WebCoreSupport/WebFrameLoaderClient.cpp:
(kit):
(WebFrameLoaderClient::dispatchCreatePage): Changed nested ifs to
early returns, and changed to use the COMPtr Query constructor.
(WebFrameLoaderClient::dispatchDidLoadMainResource): Added an early
return, and changed to use getWebDataSource.
* WebCoreSupport/WebFrameLoaderClient.h:
* WebFrame.cpp:
* WebFrame.h:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30611
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
aroben@apple.com [Wed, 27 Feb 2008 00:50:09 +0000 (00:50 +0000)]
Get rid of IID_WebFrame
Reviewed by Anders.
* WebFrame.cpp: Removed IID_WebFrame definition.
(WebFrame::QueryInterface): Use __uuidof(WebFrame) instead of
IID_WebFrame.
(WebFrame::isDescendantOfFrame): Use COMPtr's Query constructor
instead of a manual QueryInterface.
* WebFrame.h: Removed IID_WebFrame declaration, gave WebFrame a
DECLSPEC_UUID.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30610
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
aroben@apple.com [Wed, 27 Feb 2008 00:49:43 +0000 (00:49 +0000)]
Move WebHistory-related methods to WebFrameLoaderClient
Reviewed by Anders.
* WebCoreSupport/WebFrameLoaderClient.cpp:
(WebFrameLoaderClient::updateGlobalHistory):
(WebFrameLoaderClient::shouldGoToHistoryItem):
(WebFrameLoaderClient::setTitle): Changed some nested ifs into early
returns.
(WebFrameLoaderClient::webHistory): Changed to return a COMPtr to
make the reference management foolproof.
* WebCoreSupport/WebFrameLoaderClient.h:
* WebFrame.cpp:
* WebFrame.h:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30609
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
aroben@apple.com [Wed, 27 Feb 2008 00:48:23 +0000 (00:48 +0000)]
Move cache-related methods to WebFrameLoaderClient
Reviewed by Anders.
* WebCoreSupport/WebFrameLoaderClient.cpp:
(getWebDataSource): Moved to the top of the file.
(WebFrameLoaderClient::savePlatformDataToCachedPage):
(WebFrameLoaderClient::transitionToCommittedForNewPage):
(WebFrameLoaderClient::canCachePage):
* WebCoreSupport/WebFrameLoaderClient.h:
* WebFrame.cpp:
* WebFrame.h:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30608
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
timothy@apple.com [Wed, 27 Feb 2008 00:46:16 +0000 (00:46 +0000)]
Reviewed by Adam Roben.
<rdar://problem/
5712777> REGRESSION: Arrow up/down in inspector
source list does not scroll when item at edge is reached (16572)
Use scrollIntoViewIfNeeded() instead of the hand-rolled scrollToElement
we were using in utilities.js.
* page/inspector/DocumentPanel.js: Use scrollIntoViewIfNeeded instead of scrollToElement.
* page/inspector/Resource.js: Ditto.
* page/inspector/inspector.js: Ditto.
* page/inspector/SourcePanel.js: Use scrollIntoViewIfNeeded instead
of scrollIntoView, so the source line is centered in the view if needed.
* page/inspector/utilities.js: Removed scrollToElement().
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30607
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
beidson@apple.com [Tue, 26 Feb 2008 23:47:27 +0000 (23:47 +0000)]
Add a skipped storage test for now
* platform/win/Skipped: Added storage/multiple-transactions.html
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30606
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
hyatt@apple.com [Tue, 26 Feb 2008 23:17:40 +0000 (23:17 +0000)]
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30605
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
hyatt@apple.com [Tue, 26 Feb 2008 23:17:16 +0000 (23:17 +0000)]
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30604
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
hyatt@apple.com [Tue, 26 Feb 2008 23:17:03 +0000 (23:17 +0000)]
Reviewed by Dan
http://bugs.webkit.org/show_bug.cgi?id=17485
Refine our behavior for empty inlines so that they work more correctly with margins/borders/padding.
Fixes a regression on huffingtonpost.com.
New tests added to fast/inline
* rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeLogicalBoxHeights):
(WebCore::InlineFlowBox::placeBoxesVertically):
(WebCore::InlineFlowBox::shrinkBoxesWithNoTextChildren):
* rendering/RenderFlow.cpp:
(WebCore::RenderFlow::calcMargins):
* rendering/RenderFlow.h:
* rendering/RenderObject.h:
(WebCore::RenderObject::hasHorizontalBordersPaddingOrMargin):
(WebCore::RenderObject::hasHorizontalBordersOrPadding):
* rendering/bidi.cpp:
(WebCore::getBorderPaddingMargin):
(WebCore::RenderBlock::layoutInlineChildren):
(WebCore::inlineFlowRequiresLineBox):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30603
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
timothy@apple.com [Tue, 26 Feb 2008 21:54:07 +0000 (21:54 +0000)]
.:
Reviewed by Darin Adler.
Add "64" and "64u" make rules to allow easy building of 64-bit versions.
make 64: will build Intel-only 64-bit.
make 64u: will build 4-way universal for PPC and Intel.
* Makefile:
* Makefile.shared:
WebKitTools:
Reviewed by Darin Adler.
* Scripts/build-webkit: Add --universal and --64-bit flags to make building
64-bit architectures easy. Combine the two flags to build 4-way universal.
* Makefile: Add "64" and "64u" make rules. Fix the universal rule to return
non-zero when the build failes.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30597
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
weinig@apple.com [Tue, 26 Feb 2008 21:22:04 +0000 (21:22 +0000)]
Fix Windows build.
* platform/text/PlatformString.h:
(WebCore::reverseFind):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30596
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mitz@apple.com [Tue, 26 Feb 2008 19:01:53 +0000 (19:01 +0000)]
WebCore:
Reviewed by Darin Adler.
- fix http://bugs.webkit.org/show_bug.cgi?id=17555
<rdar://problem/
5766016> REGRESSION (r27351): Thumbnails positioned wrong on Getty Images results page
Test: fast/dynamic/subtree-table-cell-height.html
* rendering/RenderObject.cpp:
(WebCore::objectIsRelayoutBoundary): Exclude table cells, since their
height depends on their contents even if the CSS height property is
fixed.
LayoutTests:
Reviewed by Darin Adler.
- test for http://bugs.webkit.org/show_bug.cgi?id=17555
<rdar://problem/
5766016> REGRESSION (r27351): Thumbnails positioned wrong on Getty Images results page
* fast/dynamic/subtree-table-cell-height.html: Added.
* platform/mac/fast/dynamic/subtree-table-cell-height-expected.checksum: Added.
* platform/mac/fast/dynamic/subtree-table-cell-height-expected.png: Added.
* platform/mac/fast/dynamic/subtree-table-cell-height-expected.txt: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30595
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
jess@apple.com [Tue, 26 Feb 2008 18:26:08 +0000 (18:26 +0000)]
Reviewed by Darin.
* Scripts/gdb-safari: Set WEBKIT_UNSET_DYLD_FRAMEWORK_PATH in gdb's environment,
so that Safari inherits it when launched. Preexisting code in WebKit checks this
environment variable, and if set, unsets DYLD_FRAMEWORK_PATH, so that applications
launched by Safari continue to use the standard system WebKit.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30594
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
weinig@apple.com [Tue, 26 Feb 2008 17:06:24 +0000 (17:06 +0000)]
JavaScriptCore:
Reviewed by Dan Bernstein.
- Add a variant of remove that takes a position and a length.
* wtf/Vector.h:
(WTF::Vector::remove):
WebCore:
Reviewed by Dan Bernstein.
Make the cleanPath function in CSSStyleSelector more efficient by using
a Vector<UChar>.
* css/CSSStyleSelector.cpp:
(WebCore::cleanPath): Make this function work on a Vector<UChar> instead
of a String. Also remove unnecessary reverseFind call that could be acomplished
with two compares.
(WebCore::checkPseudoState): Remove reserveCapacity calls now that we are using
a stack buffer.
* platform/text/PlatformString.h:
(WebCore::find): Make this find generic enough that it can be used for
String::find.
(WebCore::reverseFind): Implement reverseFind so that it can be used
for with a UChar* and length.
* platform/text/StringImpl.cpp:
(WebCore::StringImpl::find): Use implementation in PlatformString.cpp
(WebCore::StringImpl::reverseFind): Ditto,
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30593
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap@webkit.org [Tue, 26 Feb 2008 10:15:02 +0000 (10:15 +0000)]
Reviewed by Darin. Tweaked and landed by Alexey.
http://bugs.webkit.org/show_bug.cgi?id=17411
Ideographic comma and full stop should be treated as line-breakable characters
Test: fast/text/line-breaks-after-ideographic-comma-or-full-stop.html
* platform/text/CharacterNames.h: Added ideographicComma and ideographicFullStop.
* rendering/break_lines.cpp: (WebCore::shouldBreakAfter): Added a workaround for an issue in
Unicode 5.0 that is causing this.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30592
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
eric@webkit.org [Tue, 26 Feb 2008 08:56:02 +0000 (08:56 +0000)]
Reviewed by eseidel. Landed by eseidel
Fix improperly initialized m_isUpToDate member variable.
Fixs 50+ layout tests when run on Tiger.
http://bugs.webkit.org/show_bug.cgi?id=17549
* platform/network/cf/ResourceResponse.h:
(WebCore::ResourceResponse::ResourceResponse):
* platform/network/mac/ResourceResponse.h:
(WebCore::ResourceResponse::ResourceResponse):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30591
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mitz@apple.com [Tue, 26 Feb 2008 06:47:43 +0000 (06:47 +0000)]
WebCore:
Reviewed by Sam Weinig.
- fix http://bugs.webkit.org/show_bug.cgi?id=17152
Paragraphs indented with :first-letter and text-align CSS render incorrectly
Test: fast/text/apply-start-width-after-skipped-text.html
* rendering/bidi.cpp:
(WebCore::RenderBlock::findNextLineBreak): Improved the logic for when
to figure in the padding, border and margin before a text node when the
line begins inside a text node.
LayoutTests:
Reviewed by Sam Weinig.
- test for http://bugs.webkit.org/show_bug.cgi?id=17152
Paragraphs indented with :first-letter and text-align CSS render incorrectly
* fast/text/apply-start-width-after-skipped-text.html: Added.
* platform/mac/fast/text/apply-start-width-after-skipped-text-expected.checksum: Added.
* platform/mac/fast/text/apply-start-width-after-skipped-text-expected.png: Added.
* platform/mac/fast/text/apply-start-width-after-skipped-text-expected.txt: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30590
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
kevino@webkit.org [Tue, 26 Feb 2008 06:20:09 +0000 (06:20 +0000)]
Windows build fix for wx port.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30589
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mrowe@apple.com [Tue, 26 Feb 2008 05:00:21 +0000 (05:00 +0000)]
2008-02-25 Mark Rowe <mrowe@apple.com>
Windows build fix.
* platform/network/cf/ResourceResponseCFNet.cpp:
(WebCore::ResourceResponse::platformCompare): Implement platformCompare for ResourceResponseCFNet.cpp.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30588
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
bdakin@apple.com [Tue, 26 Feb 2008 04:25:25 +0000 (04:25 +0000)]
WebCore:
Reviewed by Geoff.
Fixed for <rdar://problem/
5760530> REGRESSION: Assertion failure in
Document::removePendingSheet() from r30438
r30438 added a call to CachedResource::error() from inside
Loader::didReceiveData() if a CSS file 4xxs. There was an
assumption in the loader, though, that either error() would be
called, or didFinishLoading() would be called, so some work is
duplicated in each. Now that we are calling an error() on files
that will also make it to didFinishLoading() (since they succeeded
in the network layer), we need to make sure we do not duplicate the
work. CachedCSSStyleSheet::error() calls checkNotify, which ends up
decrementing the document's pending style sheet counter.
checkNotify() was still getting called, though, through the normal
didFinishLoading code path, and the counter was being decremented
twice. Bad!
* loader/loader.cpp:
(WebCore::Loader::didFinishLoading):
(WebCore::Loader::didReceiveData):
LayoutTests:
Reviewed by Geoff.
Test for <rdar://problem/
5760530> REGRESSION: Assertion failure in
Document::removePendingSheet() from r30438
* http/tests/misc/missing-style-sheet-expected.txt: Added.
* http/tests/misc/missing-style-sheet.html: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30587
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mrowe@apple.com [Tue, 26 Feb 2008 04:01:38 +0000 (04:01 +0000)]
Fix the Gtk, wx and Qt builds.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30586
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mrowe@apple.com [Tue, 26 Feb 2008 02:22:34 +0000 (02:22 +0000)]
Fix bug number in ChangeLog.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30585
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mrowe@apple.com [Tue, 26 Feb 2008 02:21:00 +0000 (02:21 +0000)]
2008-02-25 MorganL <morganl.webkit@yahoo.com>
Reviewed by Darin Adler.
Eliminate some #ifdefs from headers in platform/network.
http://bugs.webkit.org/show_bug.cgi?id=11786
* WebCore.base.exp:
* WebCore.order:
* WebCore.vcproj/WebCore.vcproj:
* WebCore.xcodeproj/project.pbxproj:
* dom/XMLTokenizer.cpp:
* loader/ResourceLoader.h:
* platform/network/AuthenticationChallenge.cpp: Removed.
* platform/network/AuthenticationChallenge.h: Removed.
* platform/network/AuthenticationChallengeBase.cpp: Copied from platform/network/AuthenticationChallenge.cpp.
(WebCore::AuthenticationChallengeBase::AuthenticationChallengeBase):
(WebCore::AuthenticationChallengeBase::previousFailureCount):
(WebCore::AuthenticationChallengeBase::proposedCredential):
(WebCore::AuthenticationChallengeBase::protectionSpace):
(WebCore::AuthenticationChallengeBase::failureResponse):
(WebCore::AuthenticationChallengeBase::error):
(WebCore::AuthenticationChallengeBase::isNull):
(WebCore::AuthenticationChallengeBase::nullify):
(WebCore::AuthenticationChallengeBase::compare):
* platform/network/AuthenticationChallengeBase.h: Copied from platform/network/AuthenticationChallenge.h.
(WebCore::AuthenticationChallengeBase::platformCompare):
(WebCore::operator==):
(WebCore::operator!=):
* platform/network/ResourceError.cpp: Removed.
* platform/network/ResourceError.h: Removed.
* platform/network/ResourceErrorBase.cpp: Copied from platform/network/ResourceError.cpp.
(WebCore::ResourceErrorBase::lazyInit):
(WebCore::ResourceErrorBase::compare):
* platform/network/ResourceErrorBase.h: Copied from platform/network/ResourceError.h.
(WebCore::ResourceErrorBase::isNull):
(WebCore::ResourceErrorBase::domain):
(WebCore::ResourceErrorBase::errorCode):
(WebCore::ResourceErrorBase::failingURL):
(WebCore::ResourceErrorBase::localizedDescription):
(WebCore::ResourceErrorBase::ResourceErrorBase):
(WebCore::ResourceErrorBase::platformLazyInit):
(WebCore::ResourceErrorBase::platformCompare):
(WebCore::operator==):
* platform/network/ResourceResponseBase.cpp:
(WebCore::ResourceResponseBase::isHTTP):
(WebCore::ResourceResponseBase::url):
(WebCore::ResourceResponseBase::setUrl):
(WebCore::ResourceResponseBase::mimeType):
(WebCore::ResourceResponseBase::setMimeType):
(WebCore::ResourceResponseBase::expectedContentLength):
(WebCore::ResourceResponseBase::setExpectedContentLength):
(WebCore::ResourceResponseBase::textEncodingName):
(WebCore::ResourceResponseBase::setTextEncodingName):
(WebCore::ResourceResponseBase::suggestedFilename):
(WebCore::ResourceResponseBase::setSuggestedFilename):
(WebCore::ResourceResponseBase::httpStatusCode):
(WebCore::ResourceResponseBase::setHTTPStatusCode):
(WebCore::ResourceResponseBase::httpStatusText):
(WebCore::ResourceResponseBase::setHTTPStatusText):
(WebCore::ResourceResponseBase::httpHeaderField):
(WebCore::ResourceResponseBase::setHTTPHeaderField):
(WebCore::ResourceResponseBase::httpHeaderFields):
(WebCore::ResourceResponseBase::isAttachment):
(WebCore::ResourceResponseBase::setExpirationDate):
(WebCore::ResourceResponseBase::expirationDate):
(WebCore::ResourceResponseBase::setLastModifiedDate):
(WebCore::ResourceResponseBase::lastModifiedDate):
(WebCore::ResourceResponseBase::lazyInit):
(WebCore::ResourceResponseBase::compare):
* platform/network/ResourceResponseBase.h:
(WebCore::ResourceResponseBase::ResourceResponseBase):
(WebCore::ResourceResponseBase::platformLazyInit):
(WebCore::ResourceResponseBase::platformCompare):
(WebCore::operator==):
* platform/network/cf/AuthenticationCF.cpp:
(WebCore::AuthenticationChallenge::AuthenticationChallenge):
(WebCore::AuthenticationChallenge::platformCompare):
* platform/network/cf/AuthenticationChallenge.h: Added.
(WebCore::AuthenticationChallenge::AuthenticationChallenge):
(WebCore::AuthenticationChallenge::sourceHandle):
(WebCore::AuthenticationChallenge::cfURLAuthChallengeRef):
* platform/network/cf/ResourceError.h: Added.
(WebCore::ResourceError::ResourceError):
* platform/network/cf/ResourceErrorCF.cpp:
(WebCore::ResourceError::ResourceError):
(WebCore::ResourceError::platformLazyInit):
(WebCore::ResourceError::platformCompare):
(WebCore::ResourceError::operator CFStreamError):
* platform/network/cf/ResourceResponse.h:
(WebCore::ResourceResponse::ResourceResponse):
* platform/network/cf/ResourceResponseCFNet.cpp:
(WebCore::ResourceResponse::platformLazyInit):
* platform/network/curl/AuthenticationChallenge.h: Added.
(WebCore::AuthenticationChallenge::AuthenticationChallenge):
* platform/network/curl/ResourceError.h: Added.
(WebCore::ResourceError::ResourceError):
* platform/network/curl/ResourceResponse.h:
(WebCore::ResourceResponse::ResourceResponse):
* platform/network/mac/AuthenticationChallenge.h: Added.
(WebCore::AuthenticationChallenge::AuthenticationChallenge):
(WebCore::AuthenticationChallenge::sender):
(WebCore::AuthenticationChallenge::nsURLAuthenticationChallenge):
* platform/network/mac/AuthenticationMac.mm:
(WebCore::AuthenticationChallenge::AuthenticationChallenge):
(WebCore::AuthenticationChallenge::platformCompare):
* platform/network/mac/ResourceError.h: Added.
(WebCore::ResourceError::ResourceError):
* platform/network/mac/ResourceErrorMac.mm:
(WebCore::ResourceError::platformLazyInit):
(WebCore::ResourceError::platformCompare):
* platform/network/mac/ResourceResponse.h:
(WebCore::ResourceResponse::ResourceResponse):
* platform/network/mac/ResourceResponseMac.mm:
(WebCore::ResourceResponse::platformLazyInit):
(WebCore::ResourceResponse::platformCompare):
* platform/network/qt/AuthenticationChallenge.h: Added.
(WebCore::AuthenticationChallenge::AuthenticationChallenge):
* platform/network/qt/ResourceError.h: Added.
(WebCore::ResourceError::ResourceError):
* platform/network/qt/ResourceResponse.h:
(WebCore::ResourceResponse::ResourceResponse):
* xml/XSLTProcessor.cpp:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30584
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
aroben@apple.com [Tue, 26 Feb 2008 01:06:02 +0000 (01:06 +0000)]
Fix run-webkit-tests after r30394
* Scripts/webkitdirs.pm: Never append _debug to FindSafari's
executable name.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30583
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
aroben@apple.com [Tue, 26 Feb 2008 01:05:37 +0000 (01:05 +0000)]
Move plugin-related methods to WebFrameLoaderClient
Reviewed by Anders.
* WebCoreSupport/WebFrameLoaderClient.cpp:
(WebFrameLoaderClient::WebFrameLoaderClient):
(WebFrameLoaderClient::setMainDocumentError):
(WebFrameLoaderClient::committedLoad): Added some early returns.
(WebFrameLoaderClient::receivedData):
(WebFrameLoaderClient::finishedLoading): Ditto.
(getWebDataSource):
(WebFrameLoaderClient::createPlugin):
* WebCoreSupport/WebFrameLoaderClient.h:
* WebFrame.cpp:
(WebFrame::WebFramePrivate::WebFramePrivate):
* WebFrame.h:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30582
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
aroben@apple.com [Tue, 26 Feb 2008 01:05:09 +0000 (01:05 +0000)]
Move createFrame to WebFrameLoaderClient
Reviewed by Anders.
* WebCoreSupport/WebFrameLoaderClient.cpp:
(WebFrameLoaderClient::createFrame):
(WebFrameLoaderClient::loadURLIntoChild):
* WebCoreSupport/WebFrameLoaderClient.h:
* WebFrame.cpp:
* WebFrame.h:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30581
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
aroben@apple.com [Tue, 26 Feb 2008 01:04:30 +0000 (01:04 +0000)]
Move progress-related methods to WebFrameLoaderClient
Reviewed by Anders.
* WebCoreSupport/WebFrameLoaderClient.cpp:
(WebFrameLoaderClient::postProgressStartedNotification):
(WebFrameLoaderClient::postProgressEstimateChangedNotification):
(WebFrameLoaderClient::postProgressFinishedNotification):
* WebCoreSupport/WebFrameLoaderClient.h:
* WebFrame.cpp:
* WebFrame.h:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30580
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
aroben@apple.com [Tue, 26 Feb 2008 01:01:00 +0000 (01:01 +0000)]
Remove WebFrame::detachedFromParent1
This method was removed from FrameLoaderClient in r19042, but was
never removed from WebFrame.
Reviewed by Anders.
* WebFrame.cpp:
* WebFrame.h:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30579
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
beidson@apple.com [Tue, 26 Feb 2008 00:28:14 +0000 (00:28 +0000)]
Reviewed by T3h Mitz Pettel (Dan Bernstein)
Layout test for already fixed bug <rdar://problem/
5729445>
* storage/multiple-transactions-expected.txt: Added.
* storage/multiple-transactions.html: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30578
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
andersca@apple.com [Mon, 25 Feb 2008 22:57:51 +0000 (22:57 +0000)]
Reviewed by Darin.
Move remaining render style objects over to start with a refcount of 1 and get
rid of DeprecatedDataRef.
* rendering/DataRef.h:
(WebCore::DataRef::operator*):
(WebCore::DataRef::operator->):
* rendering/RenderStyle.h:
* rendering/SVGRenderStyle.cpp:
(WebCore::SVGRenderStyle::SVGRenderStyle):
* rendering/SVGRenderStyle.h:
(WebCore::SVGRenderStyle::create):
(WebCore::SVGRenderStyle::copy):
* rendering/SVGRenderStyleDefs.cpp:
(StyleFillData::StyleFillData):
(StyleStrokeData::StyleStrokeData):
(StyleStopData::StyleStopData):
(StyleTextData::StyleTextData):
(StyleClipData::StyleClipData):
(StyleMaskData::StyleMaskData):
(StyleMarkerData::StyleMarkerData):
(StyleMiscData::StyleMiscData):
* rendering/SVGRenderStyleDefs.h:
(WebCore::StyleFillData::create):
(WebCore::StyleFillData::copy):
(WebCore::StyleStrokeData::create):
(WebCore::StyleStrokeData::copy):
(WebCore::StyleStopData::create):
(WebCore::StyleStopData::copy):
(WebCore::StyleTextData::create):
(WebCore::StyleTextData::copy):
(WebCore::StyleClipData::create):
(WebCore::StyleClipData::copy):
(WebCore::StyleMaskData::create):
(WebCore::StyleMaskData::copy):
(WebCore::StyleMarkerData::create):
(WebCore::StyleMarkerData::copy):
(WebCore::StyleMiscData::create):
(WebCore::StyleMiscData::copy):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30577
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mrowe@apple.com [Mon, 25 Feb 2008 22:44:23 +0000 (22:44 +0000)]
2008-02-25 Mark Mentovai <mark@moxienet.com>
Reviewed by Mark Rowe.
Enable CollectorHeapIntrospector to build by itself, as well as in an AllInOneFile build.
http://bugs.webkit.org/show_bug.cgi?id=17538
* kjs/CollectorHeapIntrospector.cpp: Provide "using" declaration for
WTF::RemoteMemoryReader.
* kjs/collector.h: Move CollectorHeap declaration here...
* kjs/collector.cpp: ... from here.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30576
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mitz@apple.com [Mon, 25 Feb 2008 22:03:08 +0000 (22:03 +0000)]
Reviewed by Dave Hyatt.
- remove file-static BidiRun variables from bidi.cpp
* platform/text/BidiResolver.h:
(WebCore::BidiResolver::): Initialize m_logicallyLastRun.
(WebCore::BidiResolver::logicallyLastRun): Added.
(WebCore::BidiResolver::runCount): Made unsigned.
(WebCore::::reverseRuns): Changed ints to unsigned.
(WebCore::::createBidiRunsForLine): Made this function set
m_logicallyLastRun.
* rendering/RenderBlock.h:
* rendering/bidi.cpp:
(WebCore::BidiState::addRun): Removed setting of sLogicallyLastBidiRun.
(WebCore::RenderBlock::constructLine): Removed unused start parameter
and added run count and first and last run parameters. Replaced end
parameter with lastLine boolean and endObject pointer.
(WebCore::RenderBlock::computeHorizontalPositionsForLine): Added first
and logically last run parameters.
(WebCore::RenderBlock::computeVerticalPositionsForLine): Added firstRun
parameter.
(WebCore::RenderBlock::bidiReorderLine): Removed setting of static
variables.
(WebCore::RenderBlock::layoutInlineChildren): Changed to use BidiState
accessors instead of file statics.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30575
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mrowe@apple.com [Mon, 25 Feb 2008 21:58:40 +0000 (21:58 +0000)]
2008-02-25 Rodney Dawes <dobey@wayofthemonkey.com>
Reviewed by Jon Honeycutt.
http://bugs.webkit.org/show_bug.cgi?id=16924
Bug 16924: Shared PluginDatabase, PluginPackage, and PlugInInfoStore implementations
Add PluginInfoStore.cpp and new PluginDatabase.cpp to GTK+ and Qt ports.
Remove old PlugInInfoStoreQt.cpp as it is obsoleted by shared code.
Add PluginInfoStore, PluginDatabase, and PluginStream files to Wx build.
Add new PluginDatabase.cpp to Windows build.
Add temporary stubs for new PluginDatabase and PluginPackage.
shared classes to GTK+, Qt, and Wx ports.
Copy PluginDatabaseWin.cpp to PluginDatabase.cpp to preserve history.
Remove shared code from PluginDatabaseWin.cpp.
Remove Windows-specific code from PluginDatabase.cpp.
Use PlatformModule and PlatformFileTime instead of HMODULE and FILETIME.
Remove extraneous PluginPackage:: from hash() class method prototype.
Subsume storeFileVersion into PluginPackage::fetchInfo.
Add cross-platform PlatformModuleVersion type definition.
Use PlatformModuleVersion to store the module version.
Rename m_fileVersion[ML]S to m_moduleVersion.
Change compareFileVersion to use PlatformModuleVersion as the argument.
Move PluginView::determineQuirks and m_quirks to PluginPackage.
Updated determineQuirks for the PlatformModuleVersion.
* GNUmakefile.am:
* WebCore.pro:
* WebCoreSources.bkl:
* webcore-base.bkl:
* WebCore.vcproj/WebCore.vcproj:
* Platform/FileSystem.h:
* platform/qt/PlugInInfoStoreQt.cpp:
* platform/gtk/TemporaryLinkStubs.cpp:
* platform/qt/TemporaryLinkStubs.cpp:
* platform/wx/TemporaryLinkStubs.cpp:
* plugins/PluginDatabase.cpp:
* plugins/PluginDatabase.h:
* plugins/win/PluginDatabaseWin.cpp:
* plugins/PluginPackage.h:
* plugins/win/PluginPackageWin.cpp:
* plugins/PluginView.h:
* plugins/win/PluginViewWin.cpp:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30574
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
andersca@apple.com [Mon, 25 Feb 2008 21:51:42 +0000 (21:51 +0000)]
Reviewed by Darin.
Make more style objects start out with a refcount of 1.
* rendering/RenderStyle.cpp:
(WebCore::StyleSurroundData::StyleSurroundData):
(WebCore::StyleBoxData::StyleBoxData):
(WebCore::StyleVisualData::StyleVisualData):
(WebCore::StyleBackgroundData::StyleBackgroundData):
(WebCore::StyleRareNonInheritedData::StyleRareNonInheritedData):
(WebCore::StyleRareInheritedData::StyleRareInheritedData):
(WebCore::StyleInheritedData::StyleInheritedData):
* rendering/RenderStyle.h:
(WebCore::StyleSurroundData::create):
(WebCore::StyleSurroundData::copy):
(WebCore::StyleBoxData::create):
(WebCore::StyleBoxData::copy):
(WebCore::StyleVisualData::create):
(WebCore::StyleVisualData::copy):
(WebCore::StyleBackgroundData::create):
(WebCore::StyleBackgroundData::copy):
(WebCore::StyleBackgroundData::~StyleBackgroundData):
(WebCore::StyleRareNonInheritedData::create):
(WebCore::StyleRareNonInheritedData::copy):
(WebCore::StyleRareInheritedData::create):
(WebCore::StyleRareInheritedData::copy):
(WebCore::StyleInheritedData::create):
(WebCore::StyleInheritedData::copy):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30573
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
andersca@apple.com [Mon, 25 Feb 2008 20:00:19 +0000 (20:00 +0000)]
Reviewed by Darin.
Make some of the refcounted style objects start out with a refcount of 1.
* rendering/DataRef.h:
Make a DeprecatedDataRef class which is just a copy of the old DataRef class.
Change DataRef to use ::create() and ::copy() instead of the constructors.
Change DataRef's pointer to be a RefPtr instead.
* rendering/RenderStyle.cpp:
(WebCore::StyleMarqueeData::StyleMarqueeData):
(WebCore::StyleFlexibleBoxData::StyleFlexibleBoxData):
(WebCore::StyleMultiColData::StyleMultiColData):
(WebCore::StyleTransformData::StyleTransformData):
Start with a RefCount of 1.
* rendering/RenderStyle.h:
* rendering/SVGRenderStyle.h:
Add ::create() and ::copy() methods. Make not yet converted classes use DeprecatedDataRef.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30572
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Mon, 25 Feb 2008 17:33:47 +0000 (17:33 +0000)]
WebCore:
Reviewed by Anders.
Based on a patch by Tim Steele <timsteele41@gmail.com>.
- fix http://bugs.webkit.org/show_bug.cgi?id=17186
Fragment navigation within a page permanently cancels meta refresh
Test: fast/loader/meta-refresh-anchor-click.html
* loader/FrameLoader.cpp:
(WebCore::FrameLoader::load): When load type is FrameLoadTypeRedirectWithLockedHistory,
always do a real load. This makes sure that meta refresh loads are treated as real loads
rather than anchor scrolls. Also tweaked formatting a bit.
(WebCore::FrameLoader::continueFragmentScrollAfterNavigationPolicy): This function is
an alternate way to complete a load, so it needs to start the redirection timer if
redirection has already been scheduled.
LayoutTests:
Reviewed by Anders.
- test for http://bugs.webkit.org/show_bug.cgi?id=17186
Fragment navigation within a page permanently cancels meta refresh
* fast/loader/meta-refresh-anchor-click-expected.txt: Added.
* fast/loader/meta-refresh-anchor-click.html: Added.
* fast/loader/resources/meta-refresh-subframe.html: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30571
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mitz@apple.com [Mon, 25 Feb 2008 17:11:56 +0000 (17:11 +0000)]
Reviewed by Darin Adler.
- avoid counting spaces in non-justified text
* rendering/bidi.cpp:
Removed file-static numSpaces.
(WebCore::BidiState::addRun): Removed space counting.
(WebCore::RenderBlock::computeHorizontalPositionsForLine): Added space
counting, conditional on the block having the text-align: justify
property value.
(WebCore::RenderBlock::bidiReorderLine): Removed initialization of
numSpaces.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30570
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Mon, 25 Feb 2008 17:10:37 +0000 (17:10 +0000)]
Requested by Steve.
* platform/graphics/cg/ImageSourceCG.cpp:
(WebCore::ImageSource::createFrameAtIndex): Put the string into a global
variable (because CFSTR is inefficient on Windows).
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30569
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Mon, 25 Feb 2008 17:06:51 +0000 (17:06 +0000)]
JavaScriptCore:
Reviewed by Adam.
* JavaScriptCore.exp: Sort the contents of this file.
WebCore:
Reviewed by Adam.
* WebCore.base.exp: Export blankURL, and sort the file.
WebKit/mac:
Reviewed by Adam.
* WebView/WebArchiver.mm:
(+[WebArchiver archiveSelectionInFrame:]): Use blankURL.
* WebView/WebFrame.mm:
(-[WebFrame _loadData:MIMEType:textEncodingName:baseURL:unreachableURL:]): Avoid the
variable name URL to avoid clashing with the renamed KURL in the future. Also use
blankURL.
(-[WebFrame loadData:MIMEType:textEncodingName:baseURL:]): Ditto.
(-[WebFrame _loadHTMLString:baseURL:unreachableURL:]): Ditto.
(-[WebFrame loadHTMLString:baseURL:]): Ditto.
(-[WebFrame loadAlternateHTMLString:baseURL:forUnreachableURL:]): Ditto.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30568
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
aroben@apple.com [Mon, 25 Feb 2008 16:53:15 +0000 (16:53 +0000)]
MSVC build fix
* kjs/testkjs.cpp:
(functionQuit): Don't add a return statement after exit(0) for MSVC.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30567
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
aroben@apple.com [Mon, 25 Feb 2008 16:52:55 +0000 (16:52 +0000)]
Move many dispatch methods to WebFrameLoaderClient
Reviewed by Sam.
* WebCoreSupport/WebFrameLoaderClient.cpp:
(WebFrameLoaderClient::dispatchDidHandleOnloadEvents):
(WebFrameLoaderClient::dispatchDidReceiveServerRedirectForProvisionalLoad):
(WebFrameLoaderClient::dispatchDidCancelClientRedirect):
(WebFrameLoaderClient::dispatchWillPerformClientRedirect):
(WebFrameLoaderClient::dispatchDidChangeLocationWithinPage):
(WebFrameLoaderClient::dispatchWillClose):
(WebFrameLoaderClient::dispatchDidReceiveIcon):
(WebFrameLoaderClient::dispatchDidStartProvisionalLoad):
(WebFrameLoaderClient::dispatchDidReceiveTitle):
(WebFrameLoaderClient::dispatchDidCommitLoad):
(WebFrameLoaderClient::dispatchDidFinishDocumentLoad):
(WebFrameLoaderClient::dispatchDidFinishLoad):
(WebFrameLoaderClient::dispatchDidFirstLayout):
(WebFrameLoaderClient::dispatchShow):
* WebCoreSupport/WebFrameLoaderClient.h:
* WebFrame.cpp:
* WebFrame.h:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30566
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
aroben@apple.com [Mon, 25 Feb 2008 16:52:30 +0000 (16:52 +0000)]
Move two more methods to WebFrameLoaderClient
Reviewed by Sam.
* WebCoreSupport/WebFrameLoaderClient.cpp:
(WebFrameLoaderClient::hasFrameView):
(WebFrame::forceLayout):
* WebCoreSupport/WebFrameLoaderClient.h:
* WebFrame.cpp:
* WebFrame.h:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30565
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
hausmann@webkit.org [Mon, 25 Feb 2008 13:16:40 +0000 (13:16 +0000)]
Fix compilation with gcc 4.3, added stdio.h and unistd.h inclusions.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30564
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mrowe@apple.com [Mon, 25 Feb 2008 13:14:27 +0000 (13:14 +0000)]
2008-02-25 Johnny Ding <johnnyding.webkit@gmail.com>
Reviewed by Darin Adler.
- fix http://bugs.webkit.org/show_bug.cgi?id=17444
In HTMLTokenizer::write, the code checks 'pendingScripts.isEmpty()' to decide
whether to save prependingSrc or not. However, in HTMLTokenizer::scriptHandler
and HTMLTokenizer::scriptExecution, the code checks testBit:LoadingExtScript
to decide whether to save prependingSrc or not. The later behavior is not right
because, in scriptHandler and scriptExecution, even the pendingScripts queue is
empty, the testBit:LoadingExtScript might be TRUE.
Test: fast/tokenizer/nested-multiple-scripts.html
* html/HTMLTokenizer.cpp:
(WebCore::HTMLTokenizer::scriptHandler): check pendingScripts.isEmpty() instead of
checking state.loadingExtScript().
(WebCore::HTMLTokenizer::scriptExecution): check pendingScripts.isEmpty() instead of
checking state.loadingExtScript().
2008-02-25 Johnny Ding <johnnyding.webkit@gmail.com>
Reviewed by Darin Adler.
- bug http://bugs.webkit.org/show_bug.cgi?id=17444
Test for multiple nested scripts which are in a external script.
* fast/tokenizer/nested-multiple-scripts-expected.txt: Added.
* fast/tokenizer/nested-multiple-scripts.html: Added.
* fast/tokenizer/resources/external-script-1.js: Added.
* fast/tokenizer/resources/external-script-2.js: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30563
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Mon, 25 Feb 2008 07:44:58 +0000 (07:44 +0000)]
* Scripts/do-webcore-rename: Make some updates based on a trial run of
the renaming script.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30562
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Mon, 25 Feb 2008 07:17:12 +0000 (07:17 +0000)]
* Scripts/do-webcore-rename: More renaming plans.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30561
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
weinig@apple.com [Mon, 25 Feb 2008 07:11:40 +0000 (07:11 +0000)]
Reviewed by Mark Rowe.
http://bugs.webkit.org/show_bug.cgi?id=17529
Add support for reading from stdin from testkjs
* kjs/testkjs.cpp:
(GlobalObject::GlobalObject): Add readline function to global object.
(functionReadline): Added. Reads characters from stdin until a '\n' or
EOF is encountered. The input is returned as a String to the caller.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30560
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Mon, 25 Feb 2008 07:09:45 +0000 (07:09 +0000)]
* dom/Document.h: Removed stray duplicate declaration of
RegisteredEventListenerList.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30559
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Mon, 25 Feb 2008 06:52:43 +0000 (06:52 +0000)]
* dom/Range.cpp:
(WebCore::Range::surroundContents): Removed incorrect comment.
I added it a while ago, and I was just mixed up.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30558
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Mon, 25 Feb 2008 06:48:07 +0000 (06:48 +0000)]
- quick follow-up to that last check-in
* platform/graphics/cg/ImageSourceCG.cpp:
(WebCore::ImageSource::createFrameAtIndex): Use CFEqual instead of
WebCore::String's == for speed.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30557
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Mon, 25 Feb 2008 06:36:19 +0000 (06:36 +0000)]
WebCore:
Reviewed by Darin.
Bug 8749: XBM rendered incorrectly as black on white
xbm images are now created and rendered using black on a
transparent background (same behavior as in Firefox)
Test: platform/mac/fast/canvas/canvas-draw-xbm-image.html
* platform/graphics/cg/ImageSourceCG.cpp:
(WebCore::ImageSource::createFrameAtIndex):
LayoutTests:
Reviewed by Darin.
- test for http://bugs.webkit.org/show_bug.cgi?id=8749
* platform/mac/fast/canvas/canvas-draw-xbm-image-expected.checksum: Added.
* platform/mac/fast/canvas/canvas-draw-xbm-image-expected.png: Added.
* platform/mac/fast/canvas/canvas-draw-xbm-image-expected.txt: Added.
* platform/mac/fast/canvas/canvas-draw-xbm-image.html: Added.
* platform/mac/fast/canvas/resources: Added.
* platform/mac/fast/canvas/resources/smile.xbm: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30556
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
weinig@apple.com [Mon, 25 Feb 2008 05:29:20 +0000 (05:29 +0000)]
Reviewed by Mark Rowe.
http://bugs.webkit.org/show_bug.cgi?id=17528
Give testkjs a bath
* JavaScriptCore.exp:
* JavaScriptCore.xcodeproj/project.pbxproj: Make the testkjs.cpp use 4 space indentation.
* kjs/testkjs.cpp:
(StopWatch::getElapsedMS):
(GlobalObject::className):
(GlobalObject::GlobalObject):
Rename GlobalImp to GlobalObject and setup the global functions
in the GlobalObject's constructor. Also, use static functions for
the implementation so we can use the standard PrototypeFunction
class and remove TestFunctionImp.
(functionPrint): Move print() functionality here.
(functionDebug): Move debug() functionality here.
(functionGC): Move gc() functionality here.
(functionVersion): Move version() functionality here.
(functionRun): Move run() functionality here.
(functionLoad): Move load() functionality here.
(functionQuit): Move quit() functionality here.
(prettyPrintScript): Fix indentation.
(runWithScripts): Since all the functionality of createGlobalObject is
now in the GlobalObject constructor, just call new here.
(parseArguments): Fix indentation.
(kjsmain): Ditto
(fillBufferWithContentsOfFile): Ditto.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30555
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mitz@apple.com [Mon, 25 Feb 2008 03:08:37 +0000 (03:08 +0000)]
WebCore:
Reviewed by Darin Adler.
- fix http://bugs.webkit.org/show_bug.cgi?id=17471
REGRESSION (r30438): Crash in deleteLineBoxes in cursor-adjusting code browsing Acid3
Test: http/tests/misc/object-image-error.html
* html/HTMLObjectElement.cpp:
(WebCore::HTMLObjectElement::attach): Added an early return in case
updateFromElement() has changed the object to use fallback content.
LayoutTests:
Reviewed by Darin Adler.
- test for http://bugs.webkit.org/show_bug.cgi?id=17471
REGRESSION (r30438): Crash in deleteLineBoxes in cursor-adjusting code browsing Acid3
* http/tests/misc/object-image-error-expected.txt: Added.
* http/tests/misc/object-image-error.html: Added.
* http/tests/misc/resources/404image.php: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30554
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
weinig@apple.com [Mon, 25 Feb 2008 01:34:09 +0000 (01:34 +0000)]
Reviewed by Oliver Hunt and Mark Rowe.
http://bugs.webkit.org/show_bug.cgi?id=17505
Add support for getting command line arguments in testkjs
- This slightly changes the behavior of parsing arguments by requiring
a '-f' before all files.
* kjs/testkjs.cpp:
(createGlobalObject): Add a global property called 'arguments' which
contains an array with the parsed arguments as strings.
(runWithScripts): Pass in the arguments vector so that it can be passed
to the global object.
(parseArguments): Change parsing rules to require a '-f' before any script
file. After all '-f' and '-p' arguments have been parsed, the remaining
are added to the arguments vector and exposed to the script. If there is a
chance of ambiguity (the user wants to pass the string '-f' to the script),
the string '--' can be used separate the options from the pass through
arguments.
(kjsmain):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30553
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mitz@apple.com [Sun, 24 Feb 2008 22:50:02 +0000 (22:50 +0000)]
Rubber-stamped by Oliver Hunt.
- replace expected results in platform/mac-tiger and
platform/mac-leopard which were identical with a single copy in
platform/mac
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30552
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mitz@apple.com [Sun, 24 Feb 2008 18:48:27 +0000 (18:48 +0000)]
WebCore:
Reviewed by Darin Adler.
- fix float positioning when a float that does not fit on the line is
followed by a float that does
Test: fast/block/float/narrow-after-wide.html
* rendering/bidi.cpp:
(WebCore::RenderBlock::findNextLineBreak): Changed to not position any
more floats on the line once a float that does not fit is encountered.
That float should be pushed to the next line, and so should all floats
that follow, regardless of whether they can fit on the current line.
LayoutTests:
Reviewed by Darin Adler.
- test float positioning when a float that does not fit on the line is
followed by a float that does
* fast/block/float/narrow-after-wide.html: Added.
* platform/mac/fast/block/float/narrow-after-wide-expected.checksum: Added.
* platform/mac/fast/block/float/narrow-after-wide-expected.png: Added.
* platform/mac/fast/block/float/narrow-after-wide-expected.txt: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30551
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mitz@apple.com [Sun, 24 Feb 2008 18:44:26 +0000 (18:44 +0000)]
Reviewed by Darin Adler.
- fix http://bugs.webkit.org/show_bug.cgi?id=17511
REGRESSION: Reproducible crash in SegmentedSubstring::SegmentedSubstring(SegmentedSubstring const&)
* wtf/Deque.h:
(WTF::::expandCapacityIfNeeded): Fixed the case where m_start and m_end
are both zero but the buffer capacity is non-zero.
(WTF::::prepend): Added validity checks.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30550
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Sun, 24 Feb 2008 08:30:32 +0000 (08:30 +0000)]
WebCore:
Reviewed by Sam.
- fix http://bugs.webkit.org/show_bug.cgi?id=16770
Acid3 expects :visited styled links to restyle on iframe load
Disentangle global history updating from the back/forward history.
There are many cases where we don't want to create a new back/forward
item, but we do still want to add to the global history (used for visited
link coloring) in those cases.
Test: fast/history/subframe-is-visited.html
* loader/FrameLoader.cpp:
(WebCore::FrameLoader::updateGlobalHistory): Renamed from
addHistoryForCurrentLocation and removed the back/forward handling.
(WebCore::FrameLoader::updateHistoryForStandardLoad): Streamlined logic a
bit. Replaced call to addHistoryForCurrentLocation with a call to
addBackForwardItemClippedAtTarget. Added an unconditional call to
updateGlobalHistory.
(WebCore::FrameLoader::updateHistoryForClientRedirect): Added a FIXME;
why doesn't this function update global history?
(WebCore::FrameLoader::updateHistoryForBackForwardNavigation): Ditto.
(WebCore::FrameLoader::updateHistoryForReload): Replaced the direct call
the client with a call to the new updateGlobalHistory function.
(WebCore::FrameLoader::updateHistoryForRedirectWithLockedHistory): Did
the same changes as for updateHistoryForStandardLoad.
* loader/FrameLoader.h: More of the same.
* loader/FrameLoaderClient.h: Removed updateGlobalHistoryForReload and
renamed updateGlobalHistoryForStandardLoad to updateGlobalHistory.
* svg/graphics/SVGImageEmptyClients.h:
(WebCore::SVGEmptyFrameLoaderClient::updateGlobalHistory): Updated to
match the above.
WebKit/gtk:
Reviewed by Sam.
- remove separate client calls for "standard" and "reload' history
* WebCoreSupport/FrameLoaderClientGtk.cpp:
(WebKit::FrameLoaderClient::updateGlobalHistory):
* WebCoreSupport/FrameLoaderClientGtk.h:
WebKit/mac:
Reviewed by Sam.
- remove separate client calls for "standard" and "reload' history
* WebCoreSupport/WebFrameLoaderClient.h:
* WebCoreSupport/WebFrameLoaderClient.mm:
(WebFrameLoaderClient::updateGlobalHistory):
WebKit/qt:
Reviewed by Sam.
- remove separate client calls for "standard" and "reload' history
* WebCoreSupport/FrameLoaderClientQt.cpp:
(WebCore::FrameLoaderClientQt::updateGlobalHistory):
* WebCoreSupport/FrameLoaderClientQt.h:
WebKit/win:
Reviewed by Sam.
- remove separate client calls for "standard" and "reload' history
* WebFrame.cpp:
(WebFrame::updateGlobalHistory):
* WebFrame.h:
WebKit/wx:
Reviewed by Sam.
- remove separate client calls for "standard" and "reload' history
* WebKitSupport/FrameLoaderClientWx.cpp:
(WebCore::FrameLoaderClientWx::updateGlobalHistory):
* WebKitSupport/FrameLoaderClientWx.h:
LayoutTests:
Reviewed by Sam.
- test for http://bugs.webkit.org/show_bug.cgi?id=16770
Acid3 expects :visited styled links to restyle on iframe load
* fast/history/resources/subframe.html: Added.
* fast/history/subframe-is-visited-expected.txt: Added.
* fast/history/subframe-is-visited.html: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30549
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Sun, 24 Feb 2008 08:09:22 +0000 (08:09 +0000)]
- another try at fixing the build
* Api/qwebframe.cpp:
(QWebFrame::load): Replace new FormData with FormData::create.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30548
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Sun, 24 Feb 2008 07:55:40 +0000 (07:55 +0000)]
Reviewed by Sam.
- fix http://bugs.webkit.org/show_bug.cgi?id=17508
REGRESSION (r30535): mailto regression tests failing
* html/HTMLFormElement.cpp:
(WebCore::HTMLFormElement::submit): Restore the old logic, with its unpleasant
trip through String do to the replace operation. It's inefficient, but there's
no real reason to worry about the efficiency of mailto forms, since that feature
is nearly obsolete.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30547
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Sun, 24 Feb 2008 07:36:57 +0000 (07:36 +0000)]
Reviewed by Sam.
- replace SegmentedString's use of DeprecatedValueList with Deque
Testing indicates this is a slight speed-up for page loading.
* platform/text/SegmentedString.cpp:
(WebCore::SegmentedString::length): Update to use Deque.
(WebCore::SegmentedString::setExcludeLineNumbers): Ditto.
(WebCore::SegmentedString::append): Ditto.
(WebCore::SegmentedString::prepend): Ditto.
(WebCore::SegmentedString::advanceSubstring): Ditto.
(WebCore::SegmentedString::toString): Ditto.
* platform/text/SegmentedString.h: Tweak formatting. Don't bother making an entire
class private with a single friend. Change m_sbstrings from a DeprecatedValueList
to a Deque.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30546
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Sun, 24 Feb 2008 07:26:02 +0000 (07:26 +0000)]
Reviewed by Anders.
- fix http://bugs.webkit.org/show_bug.cgi?id=17506
REGRESSION (r30535): ASSERTION FAILED: i < size()
* loader/TextResourceDecoder.cpp:
(WebCore::findTextEncoding): Fix off-by-one in code that null-terminates
the C string here.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30545
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
pewtermoose@webkit.org [Sun, 24 Feb 2008 07:10:49 +0000 (07:10 +0000)]
Not reviewed, build fix.
* platform/network/cf/FormDataStreamCFNet.cpp:
(WebCore::httpBodyFromRequest):
* plugins/win/PluginViewWin.cpp:
(WebCore::PluginView::handlePost):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30544
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mitz@apple.com [Sun, 24 Feb 2008 07:07:30 +0000 (07:07 +0000)]
- fix a typo
* platform/graphics/win/GlyphPageTreeNodeCGWin.cpp:
(WebCore::GlyphPage::fill):
* platform/graphics/win/GlyphPageTreeNodeCairoWin.cpp:
(WebCore::GlyphPage::fill):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30543
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
pewtermoose@webkit.org [Sun, 24 Feb 2008 07:07:03 +0000 (07:07 +0000)]
2008-02-23 Jan Michael Alonzo <jmalonzo@unpluggable.com>
.:
Rubber stamped by Darin.
Add separator '\' after libJavaScriptCore_la_LIBADD and cleanup
whitespaces introduced in the previous commit.
* GNUmakefile.am:
JavaScriptCore:
Rubber stamped by Darin.
Add separator '\' after libJavaScriptCore_la_LIBADD and cleanup
whitespaces introduced in the previous commit.
* GNUmakefile.am:
WebKitTools:
Rubber stamped by Darin.
Add separator '\' after libJavaScriptCore_la_LIBADD and cleanup
whitespaces introduced in the previous commit.
* GNUmakefile.am:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30542
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mitz@apple.com [Sun, 24 Feb 2008 06:59:53 +0000 (06:59 +0000)]
Reviewed by Anders Carlsson.
- fix GlyphPage::fill() logic for partial page fills
* platform/graphics/win/GlyphPageTreeNodeCGWin.cpp:
(WebCore::GlyphPage::fill): Changed the test for non-BMP characters to
work correctly when the length parameter is less than a full page size.
Also updated the comment.
* platform/graphics/win/GlyphPageTreeNodeCairoWin.cpp:
(WebCore::GlyphPage::fill): Ditto.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30541
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Sun, 24 Feb 2008 05:42:29 +0000 (05:42 +0000)]
.:
Rubber-stamped by Darin.
* GNUmakefile.am: Add both GLOBALDEPS and WEBKITDEPS instead of DEPENDENCIES.
* configure.ac: Break dependent modules into GLOBALDEPS and WEBKITDEPS.
JavaScriptCore:
* GNUmakefile.am: Add GLOBALDEPS for testkjs and minidom.
WebKitTools:
Rubber-stamped by Darin.
* GNUmakefile.am: Add both GLOBALDEPS and WEBKITDEPS instead of DEPENDENCIES.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30540
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
kevino@webkit.org [Sun, 24 Feb 2008 05:38:49 +0000 (05:38 +0000)]
Move text drawing into wxcode, as we need platform-dependent APIs for drawing non-kerned text, which wx doesn't yet have.
http://bugs.webkit.org/show_bug.cgi?id=17396
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30539
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Sun, 24 Feb 2008 05:19:27 +0000 (05:19 +0000)]
Reviewed by Anders.
- http://bugs.webkit.org/show_bug.cgi?id=17496
make Deque use a circular array; add iterators
* wtf/Deque.h: Wrote an all-new version of this class that uses a circular
buffer. Growth policy is identical to vector. Added iterators.
* wtf/Vector.h: Made two small refinements while using this to implement
Deque: Made VectorBufferBase derive from Noncopyable, which would have
saved me some debugging time if it had been there. Renamed Impl and
m_impl to Buffer and m_buffer.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30538
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Sun, 24 Feb 2008 05:13:57 +0000 (05:13 +0000)]
Pointed out by Mitz.
* loader/TextResourceDecoder.cpp:
(WebCore::find): Fix a "==" that was supposed to be ">".
(WebCore::findIgnoringCase): Ditto.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30537
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Sun, 24 Feb 2008 05:12:17 +0000 (05:12 +0000)]
Reviewed by Sam.
* html/HTMLFormElement.cpp:
(WebCore::appendEncodedString): Fix buffer overrun in the code I just landed.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30536
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Sun, 24 Feb 2008 05:04:38 +0000 (05:04 +0000)]
Reviewed by Sam.
- remove all but 1 remaining use of DeprecatedCString
- change FormData to start refcount at 1
* html/HTMLFormElement.cpp:
(WebCore::appendString): Added. Helpers for Vector<char>.
(WebCore::appendEncodedString): Renamed from encodeCString and changed to
append to an existing Vector<char> to cut down a bit on memory allocation.
(WebCore::HTMLFormElement::formData): Replace the DeprecatedCString called
enc_string with a Vector<char> called encodedData. Change to use the new
FormData::create function.
(WebCore::HTMLFormElement::submit): Same thing, for the mailto form code.
* loader/TextResourceDecoder.cpp:
(WebCore::find): Added. Helper for searching in char* with length.
(WebCore::findIgnoringCase): Ditto.
(WebCore::findTextEncoding): Added. Helper for looking up a text encoding
when we have a char* with length rather than a null-terminated char*.
(WebCore::findXMLEncoding): Changed to use char* and length rather than
DeprecatedCString. Also fixed some things that would be one-character
buffer overruns in a string that's not null-terminated.
(WebCore::TextResourceDecoder::checkForCSSCharset): Change to use
findTextEncoding rather than constructing a CString to pass to the
TextEncodingconstructor.
(WebCore::TextResourceDecoder::checkForHeadCharset): Ditto.
* platform/network/FormData.cpp:
(WebCore::FormData::FormData): Changed to start refcount at 1.
(WebCore::FormData::create): Added.
(WebCore::FormData::copy): Call adoptRef since refcount starts at 1.
* platform/network/FormData.h: Made constructor private. Added create functions.
* platform/network/mac/ResourceRequestMac.mm:
(WebCore::ResourceRequest::doUpdateResourceRequest): Call FormData::create.
* xml/XMLHttpRequest.cpp:
(WebCore::XMLHttpRequest::send): Ditto.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30535
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Sun, 24 Feb 2008 05:01:27 +0000 (05:01 +0000)]
JavaScriptCore:
Reviewed by Anders.
- http://bugs.webkit.org/show_bug.cgi?id=17067
eliminate attributes parameter from JSObject::put for speed/clarity
* API/JSCallbackObject.h: Removed attribute arguments.
* API/JSCallbackObjectFunctions.h:
(KJS::JSCallbackObject<Base>::put): Ditto.
* API/JSObjectRef.cpp:
(JSObjectSetProperty): Use initializeVariable or putDirect when necessary
to set attribute values.
* JavaScriptCore.exp: Updated.
* bindings/objc/objc_runtime.h: Removed attribute arguments.
* bindings/objc/objc_runtime.mm:
(ObjcFallbackObjectImp::put): Ditto.
* bindings/runtime_array.cpp:
(RuntimeArray::put): Ditto.
* bindings/runtime_array.h: Ditto.
* bindings/runtime_object.cpp:
(RuntimeObjectImp::put): Ditto.
* bindings/runtime_object.h: Ditto. Also removed canPut which was only
called from one place in WebCore that can use hasProperty instead.
* kjs/Activation.h: Removed attribute argument from put and added the new
initializeVariable function that's used to put variables in variable objects.
Also made isActivationObject a const member.
* kjs/JSGlobalObject.cpp:
(KJS::JSGlobalObject::put): Removed attribute argument.
(KJS::JSGlobalObject::initializeVariable): Added. Used to give variables
their initial values, which can include the read-only property.
(KJS::JSGlobalObject::reset): Removed obsolete comments about flags.
Removed Internal flag, which is no longer needed.
* kjs/JSGlobalObject.h: More of the same.
* kjs/JSVariableObject.h: Added pure virtual initializeVariable function.
(KJS::JSVariableObject::symbolTablePut): Removed checkReadOnly flag; we always
check read-only.
(KJS::JSVariableObject::symbolTableInitializeVariable): Added.
* kjs/array_instance.cpp:
(KJS::ArrayInstance::put): Removed attribute argument.
* kjs/array_instance.h: Ditto.
* kjs/function.cpp:
(KJS::FunctionImp::put): Ditto.
(KJS::Arguments::put): Ditto.
(KJS::ActivationImp::put): Ditto.
(KJS::ActivationImp::initializeVariable): Added.
* kjs/function.h: Removed attribute arguments.
* kjs/function_object.cpp:
(KJS::FunctionObjectImp::construct): Removed Internal flag.
* kjs/lookup.h:
(KJS::lookupPut): Removed attributes argument. Also changed to use putDirect
instead of calling JSObject::put.
(KJS::cacheGlobalObject): Ditto.
* kjs/nodes.cpp:
(KJS::ConstDeclNode::handleSlowCase): Call initializeVariable to initialize
the constant.
(KJS::ConstDeclNode::evaluateSingle): Ditto.
(KJS::TryNode::execute): Use putDirect to set up the new object.
(KJS::FunctionBodyNode::processDeclarations): Removed Internal.
(KJS::ProgramNode::processDeclarations): Ditto.
(KJS::EvalNode::processDeclarations): Call initializeVariable to initialize
the variables and functions.
(KJS::FuncDeclNode::makeFunction): Removed Internal.
(KJS::FuncExprNode::evaluate): Ditto.
* kjs/object.cpp: Removed canPut, which was only being used in one code path,
not the normal high speed one.
(KJS::JSObject::put): Removed attribute argument. Moved the logic from
canPut here, in the one code ath that was still using it.
* kjs/object.h: Removed Internal attribute, ad canPut function. Removed the
attributes argument to the put function. Made isActivationObject const.
* kjs/regexp_object.cpp:
(KJS::RegExpImp::put): Removed attributes argument.
(KJS::RegExpImp::putValueProperty): Ditto.
(KJS::RegExpObjectImp::put): Ditto.
(KJS::RegExpObjectImp::putValueProperty): Ditto.
* kjs/regexp_object.h: Ditto.
* kjs/string_object.cpp:
(KJS::StringInstance::put): Removed attributes argument.
* kjs/string_object.h: Ditto.
WebCore:
Reviewed by Anders.
- http://bugs.webkit.org/show_bug.cgi?id=17067
eliminate attributes parameter from JSObject::put for speed/clarity
* bindings/js/JSCSSStyleDeclarationCustom.cpp:
(WebCore::JSCSSStyleDeclaration::customPut): Remove attributes argument.
* bindings/js/JSCanvasPixelArrayCustom.cpp:
(WebCore::JSCanvasPixelArray::indexGetter): Use early exit idiom.
(WebCore::JSCanvasPixelArray::indexSetter): Moved length check into the
CanvasPixelArray object, for consistency with the getter. Removed attributes
argument.
* bindings/js/JSDOMWindowCustom.cpp:
(WebCore::JSDOMWindow::customPut): Removed special case for variable
initialization, which is not needed since that does use put any more.
Removed attributes argument.
* bindings/js/JSEventTargetBase.h:
(WebCore::JSEventTargetBase::putValueProperty): Removed attributes argument.
(WebCore::JSEventTargetBase::put): Ditto.
(WebCore::JSEventTargetPrototype::self): Removed Internal flag.
* bindings/js/JSEventTargetNode.cpp:
(WebCore::JSEventTargetNode::put): Removed attributes argument.
(WebCore::JSEventTargetNode::putValueProperty): Ditto.
* bindings/js/JSEventTargetNode.h: Ditto.
* bindings/js/JSHTMLAppletElementCustom.cpp:
(WebCore::JSHTMLAppletElement::customPut): Ditto.
* bindings/js/JSHTMLEmbedElementCustom.cpp:
(WebCore::JSHTMLEmbedElement::customPut): Ditto.
* bindings/js/JSHTMLInputElementBase.cpp:
(WebCore::JSHTMLInputElementBase::put): Ditto.
(WebCore::JSHTMLInputElementBase::putValueProperty): Ditto.
* bindings/js/JSHTMLInputElementBase.h: Ditto.
* bindings/js/JSHTMLObjectElementCustom.cpp:
(WebCore::JSHTMLObjectElement::customPut): Ditto.
* bindings/js/JSHTMLOptionsCollectionCustom.cpp:
(WebCore::JSHTMLOptionsCollection::indexSetter): Ditto.
* bindings/js/JSHTMLSelectElementCustom.cpp:
(WebCore::JSHTMLSelectElement::indexSetter): Ditto.
* bindings/js/JSHistoryCustom.cpp:
(WebCore::JSHistory::customPut): Ditto.
* bindings/js/JSLocation.cpp:
(WebCore::JSLocation::put): Ditto.
* bindings/js/JSLocation.h: Ditto.
* bindings/js/JSXMLHttpRequest.cpp:
(WebCore::JSXMLHttpRequest::put): Ditto.
(WebCore::JSXMLHttpRequest::putValueProperty): Ditto.
* bindings/js/JSXMLHttpRequest.h: Ditto.
* bindings/js/kjs_dom.cpp:
(WebCore::getRuntimeObject): Changed return type to JSObject*.
* bindings/js/kjs_dom.h: Ditto.
* bindings/js/kjs_events.cpp:
(WebCore::JSClipboard::put): Removed attributes argument.
(WebCore::JSClipboard::putValueProperty): Ditto.
* bindings/js/kjs_events.h: Ditto.
* bindings/js/kjs_html.cpp:
(WebCore::runtimeObjectGetter): Updated for change to getRuntimeObject to
return a JSObject. Used early exit idiom.
(WebCore::runtimeObjectPropertyGetter): Ditto.
(WebCore::runtimeObjectCustomGetOwnPropertySlot): Ditto.
(WebCore::runtimeObjectCustomPut): Use hasProperty to check for properties
that we should put with the property syntax instead of canPut.
(WebCore::runtimeObjectImplementsCall): Ditto.
(WebCore::runtimeObjectCallAsFunction): Ditto.
* bindings/js/kjs_html.h: Removed attributes argument to runtimeObjectCustomPut.
* bindings/js/kjs_window.cpp:
(KJS::Window::put): Removed attributes argument.
* bindings/js/kjs_window.h: Ditto.
* bindings/scripts/CodeGeneratorJS.pm: Removed attributes argument from put,
putValueProperty, customPut, and indexSetter.
* html/CanvasPixelArray.h:
(WebCore::CanvasPixelArray::set): Added index checking here, as in the get
function. Before, the checking was done in the JavaScript bindings for set.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30534
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
pewtermoose@webkit.org [Sun, 24 Feb 2008 04:15:23 +0000 (04:15 +0000)]
2008-02-23 Brent Fulgham <bfulgham@gmail.com>
Reviewed by Adam and Darin.
http://bugs.webkit.org/show_bug.cgi?id=17442
Correct the Windows Cairo implementation of GraphicsContext so
that Windows 'WorldTransform' is kept in sync with the Cairo
transormations.
Also corrects an uninitialized variable in the Cairo Windows
image drag handler.
WARNING: NO TEST CASES ADDED OR CHANGED
* platform/graphics/cairo/GraphicsContextCairo.cpp: Modify
methods to call GraphicsContextPlatformPrivate calls, just
as is done for the CoreGraphics version. These calls are
nop's for everything but Windows.
(WebCore::GraphicsContext::savePlatformState):
(WebCore::GraphicsContext::restorePlatformState):
(WebCore::GraphicsContext::clip):
(WebCore::GraphicsContext::drawLineForMisspellingOrBadGrammar):
(WebCore::GraphicsContext::translate):
(WebCore::GraphicsContext::concatCTM): Re-enable for Windows
(WebCore::GraphicsContext::beginTransparencyLayer):
(WebCore::GraphicsContext::endTransparencyLayer):
(WebCore::GraphicsContext::rotate):
(WebCore::GraphicsContext::scale):
* platform/graphics/cairo/GraphicsContextPlatformPrivateCairo.h:
* platform/graphics/win/GraphicsContextCGWin.cpp: Move common
code for handling WorldTransform calls to parent class.
* platform/graphics/win/GraphicsContextCairoWin.cpp: Add common
code for handling WorldTransform calls.
(WebCore::GraphicsContextPlatformPrivate::concatCTM): Change
implementation so that it only handles WorldTransform. The
Cairo update is done in GraphicsContextCairo.cpp
* platform/graphics/win/GraphicsContextWin.cpp: Call platform-private
methods to keep WorldTransform in sync.
(WebCore::GraphicsContextPlatformPrivate::save):
(WebCore::GraphicsContextPlatformPrivate::restore):
(WebCore::GraphicsContextPlatformPrivate::clip):
(WebCore::GraphicsContextPlatformPrivate::scale):
(WebCore::GraphicsContextPlatformPrivate::rotate):
(WebCore::GraphicsContextPlatformPrivate::translate):
* platform/win/DragImageCairoWin.cpp:
(WebCore::createDragImageFromImage): Correct uninitialized value
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30533
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
pewtermoose@webkit.org [Sun, 24 Feb 2008 04:10:14 +0000 (04:10 +0000)]
2008-02-23 Brent Fulgham <bfulgham@gmail.com>
Not reviewed, build fix.
* WebIconDatabase.cpp:
* WebIconDatabase.h:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30532
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
pewtermoose@webkit.org [Sun, 24 Feb 2008 04:05:05 +0000 (04:05 +0000)]
2008-02-23 Jan Michael Alonzo <jmalonzo@unpluggable.com>
Not reviewed, Gtk build fix.
* kjs/testkjs.pro:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30531
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mitz@apple.com [Sun, 24 Feb 2008 04:00:21 +0000 (04:00 +0000)]
WebCore:
Reviewed by Dave Hyatt.
- make non-autowrapping text clear floats
Test: fast/text/whitespace/nowrap-clear-float.html
* rendering/RenderBlock.cpp:
(WebCore::RenderBlock::nextFloatBottomBelow): Renamed nearestFloat() to
this and changed to avoid comparing bottom to 0 in each iteration.
(WebCore::RenderBlock::getClearDelta): Updated comment for the rename.
* rendering/RenderBlock.h:
* rendering/bidi.cpp:
(WebCore::RenderBlock::fitBelowFloats): Added. Factored out of
findNextLineBreak() and simplified.
(WebCore::RenderBlock::findNextLineBreak): Changed to call
fitBelowFloats(). Fixed the bug by trying to fit below floats in the
case of non-wrapping text. Removed some redundancy.
LayoutTests:
Reviewed by Dave Hyatt.
- test that non-autowrapping text clear floats
* fast/text/whitespace/nowrap-clear-float.html: Added.
* platform/mac-leopard/fast/text/whitespace: Added.
* platform/mac-leopard/fast/text/whitespace/nowrap-clear-float-expected.checksum: Added.
* platform/mac-leopard/fast/text/whitespace/nowrap-clear-float-expected.png: Added.
* platform/mac/fast/text/whitespace/nowrap-clear-float-expected.txt: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30530
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
weinig@apple.com [Sun, 24 Feb 2008 01:35:07 +0000 (01:35 +0000)]
Reviewed by Oliver Hunt.
Fix for http://bugs.webkit.org/show_bug.cgi?id=17504
Speed up DOM lists array subscription syntax by using the fast
getOwnPropertySlot and set paths
- 6x speed improvement on Oliver's ImageData put test.
* bindings/scripts/CodeGeneratorJS.pm: Add fast path for getOwnPropertySlot
and put when an indexGetter or indexSetter is used. We can not use the fast
path if an overridingNameGetter is used as there would be no way to override.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30529
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
kevino@webkit.org [Sun, 24 Feb 2008 01:33:49 +0000 (01:33 +0000)]
wx build fix after JSImageData.cpp was added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30528
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap@webkit.org [Sat, 23 Feb 2008 21:52:07 +0000 (21:52 +0000)]
Windows build fix - move ThreadCondition implementation from WebCore to WTF.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30527
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap@webkit.org [Sat, 23 Feb 2008 20:49:14 +0000 (20:49 +0000)]
Build fix.
JavaScriptCore:
Touch some files, hoping that Windows build bot will create JSC headers.
* kjs/AllInOneFile.cpp:
* kjs/array_instance.cpp:
* wtf/HashTable.cpp:
WebKit/gtk:
* webkit/webkitprivate.cpp:
(webkit_init): Renamed initializeThreading to initializeThreadingAndMainThread.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30526
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap@webkit.org [Sat, 23 Feb 2008 20:29:58 +0000 (20:29 +0000)]
Qt/Wx build fix - this file was still in a wrong namespace, too.
* wtf/ThreadingNone.cpp:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30525
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap@webkit.org [Sat, 23 Feb 2008 20:21:15 +0000 (20:21 +0000)]
More build fixing - fix mismatched braces.
* JavaScriptCore.pri:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30524
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap@webkit.org [Sat, 23 Feb 2008 20:10:59 +0000 (20:10 +0000)]
Wx and Gtk build fixes.
* JavaScriptCore.pri: Don't try to compile ThreadingPthreads.
* wtf/ThreadingGtk.cpp: Use a correct namespace.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30523
268f45cc-cd09-0410-ab3c-
d52691b4dbfc