darin@apple.com [Tue, 19 Feb 2008 02:21:07 +0000 (02:21 +0000)]
WebCore:
Reviewed by Sam.
- reduce use of DeprecatedString and memory allocations in processing of CSS
- remove unnecessary double -> float -> double trip in the CSS parser
- cleaned up names and structure in CSS grammar
* css/CSSGrammar.y: Remove getPropertyID and getValueID. Both are now in CSSParser.cpp
instead, and they now work on ParseString and String objects and don't require the caller
to put the string into a char*. Gave members of the %union more sensible names, removed
duplicates, and sorted into a logical order. Put the %expect back in, rather than leaving
it commented out.
* css/CSSParser.cpp:
(WebCore::equalIgnoringCase): Added.
(WebCore::hasPrefix): Added.
(WebCore::CSSParser::parseTransitionProperty): Changed to call the new cssPropertyID,
which obviates the need to call lower() and utf8() or to allocate memory at all. Also
used equalIgnoringCase rather than putting the value into a String just to compare it.
(WebCore::CSSParser::lex): Replaced convertASCIIToFloat with charactersToDouble. This change
along with the CSSGrammar.y change, removes the double -> float -> double round trip, and
affects the result of one layout test.
(WebCore::cssPropertyID): Added. Gets the property ID from the gperf hash table, but
without allocating any memory.
(WebCore::cssValueKeywordID): Ditto.
* css/CSSParser.h: Removed declaration for deprecatedString function (now used only in
CSSParser.cpp; soon to be deleted). Added cssPropertyID and cssValueKeywordID functions.
* css/CSSStyleDeclaration.cpp:
(WebCore::CSSStyleDeclaration::getPropertyCSSValue): Call cssPropertyID instead of propertyID.
(WebCore::CSSStyleDeclaration::getPropertyValue): Ditto.
(WebCore::CSSStyleDeclaration::getPropertyPriority): Ditto.
(WebCore::CSSStyleDeclaration::getPropertyShorthand): Ditto.
(WebCore::CSSStyleDeclaration::isPropertyImplicit): Ditto.
(WebCore::CSSStyleDeclaration::setProperty): Ditto.
(WebCore::CSSStyleDeclaration::removeProperty): Ditto.
(WebCore::CSSStyleDeclaration::isPropertyName): Ditto.
* css/CSSStyleDeclaration.h: Removed unnecessary includes, unnecessary Noncopyable boilerplate,
and the getPropertyID function declaration along with its associated apology comment.
* css/makevalues.pl: Generate constants instead of macros for CSS value numbers (but not an
enumeration, like properties, since you rarely have any reason to handle all values, but
often have a reason to handle all properties). Renamed the constant for the number of CSS
value keywords from CSS_VAL_TOTAL to numCSSValueKeywords, and added maxCSSValueKeywordLength.
* platform/text/String.cpp:
(WebCore::charactersToDouble): Made this function more efficient by using a stack buffer
rather than a CString.
LayoutTests:
Reviewed by Sam.
- generate new results for the one test that was altered by the
double -> float -> double trip taken by numeric values in the
CSS parser
This test appears under three different names in three different
parts of our test suite.
* platform/mac/css1/box_properties/acid_test-expected.checksum: Updated.
* platform/mac/css1/box_properties/acid_test-expected.png: Ditto.
* platform/mac/css1/box_properties/acid_test-expected.txt: Ditto.
* platform/mac/css2.1/t09-c5526c-display-00-e-expected.checksum: Ditto.
* platform/mac/css2.1/t09-c5526c-display-00-e-expected.png: Ditto.
* platform/mac/css2.1/t09-c5526c-display-00-e-expected.txt: Ditto.
* platform/mac/fast/block/basic/011-expected.checksum: Ditto.
* platform/mac/fast/block/basic/011-expected.png: Ditto.
* platform/mac/fast/block/basic/011-expected.txt: Ditto.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30393
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mitz@apple.com [Tue, 19 Feb 2008 02:13:55 +0000 (02:13 +0000)]
WebCore:
Reviewed by Dave Hyatt.
- fix <rdar://problem/
5736225> crash in svgFontAndFaceElementForFontData on digitalstrom.org/cms
Test: fast/css/font-face-multiple-remote-sources.html
* css/CSSFontFace.cpp:
(WebCore::CSSFontFace::fontLoaded):
* css/CSSSegmentedFontFace.cpp:
(WebCore::CSSSegmentedFontFace::fontLoaded):
LayoutTests:
- test for <rdar://problem/
5736225> crash in svgFontAndFaceElementForFontData on digitalstrom.org/cms
* fast/css/font-face-multiple-remote-sources.html: Added.
* platform/mac/fast/css/font-face-multiple-remote-sources-expected.checksum: Added.
* platform/mac/fast/css/font-face-multiple-remote-sources-expected.png: Added.
* platform/mac/fast/css/font-face-multiple-remote-sources-expected.txt: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30392
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Tue, 19 Feb 2008 01:12:12 +0000 (01:12 +0000)]
Reviewed by Sam.
* bindings/js/JSCSSStyleDeclarationCustom.cpp:
(WebCore::hasCSSPropertyNamePrefix): Added.
(WebCore::cssPropertyName): Reimplement to not use DeprecatedString. Also made faster
by using a Vector<UChar> and eliminating all the string operations.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30391
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Tue, 19 Feb 2008 00:54:41 +0000 (00:54 +0000)]
Reviewed by Sam.
* Misc/WebNSAttributedStringExtras.mm:
(+[NSAttributedString _web_attributedStringFromRange:]): Eliminate use of
DeprecatedString.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30390
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
slewis@apple.com [Tue, 19 Feb 2008 00:48:54 +0000 (00:48 +0000)]
2008-02-18 Stephanie Lewis <slewis@apple.com>
Reviewed by Adam.
Remove workaround for <rdar://problem/
5695848>.
* platform/network/cf/ResourceResponseCFNet.cpp:
(WebCore::ResourceResponse::doUpdateResourceResponse):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30389
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Tue, 19 Feb 2008 00:48:06 +0000 (00:48 +0000)]
Reviewed by Sam.
* Api/qwebnetworkinterface.cpp:
(QWebNetworkRequestPrivate::init): Removed use of DeprecatedString.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30388
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
weinig@apple.com [Tue, 19 Feb 2008 00:25:26 +0000 (00:25 +0000)]
JavaScriptCore:
Reviewed by Geoff Garen.
Fix for http://bugs.webkit.org/show_bug.cgi?id=17419
Remove CompatMode from JavaScriptCore as it is never set to anything other than NativeMode
* kjs/JSGlobalObject.cpp:
(KJS::JSGlobalObject::init):
* kjs/JSGlobalObject.h:
(KJS::JSGlobalObject::setDebugger):
* kjs/date_object.cpp:
(KJS::dateProtoFuncGetYear):
WebCore:
Reviewed by Geoff Garen.
Fix for http://bugs.webkit.org/show_bug.cgi?id=17419
Remove CompatMode from JavaScriptCore as it is never set to anything other than NativeMode
* bindings/js/kjs_proxy.cpp:
(WebCore::KJSProxy::initScript):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30387
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
alp@webkit.org [Mon, 18 Feb 2008 23:04:37 +0000 (23:04 +0000)]
2008-02-18 Alp Toker <alp@atoker.com>
Reviewed by Mark Rowe.
http://bugs.webkit.org/show_bug.cgi?id=17381
[CURL] Regression: data URL parsing broken after DeprecatedString removal (Acid2)
This patch resolves the regression for the GTK+ port.
* platform/network/curl/ResourceHandleManager.cpp:
(WebCore::parseDataUrl):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30386
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
alp@webkit.org [Mon, 18 Feb 2008 23:03:57 +0000 (23:03 +0000)]
2008-02-18 Alp Toker <alp@atoker.com>
Reviewed by Mark Rowe.
http://bugs.webkit.org/show_bug.cgi?id=17312
[GTK] Webview Transparent Background
Add support for WebView background transparency.
* webkit/webkitprivate.h:
* webkit/webkitwebview.cpp:
* webkit/webkitwebview.h:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30385
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
alp@webkit.org [Mon, 18 Feb 2008 23:03:29 +0000 (23:03 +0000)]
2008-02-18 Alp Toker <alp@atoker.com>
Reviewed by Mark Rowe.
Implement webkit_web_view_get_focused_frame()
Equivalent to selectedFrame in the Mac API.
* webkit/webkitwebview.cpp:
* webkit/webkitwebview.h:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30384
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Mon, 18 Feb 2008 21:43:58 +0000 (21:43 +0000)]
Reviewed by Sam.
* platform/network/win/ResourceHandleWin.cpp:
(WebCore::ResourceHandle::onHandleCreated): Use String instead of DeprecatedString.
(WebCore::ResourceHandle::start): Ditto.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30383
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Mon, 18 Feb 2008 21:42:33 +0000 (21:42 +0000)]
Reviewed by Sam.
* platform/network/win/CookieJarWin.cpp:
(WebCore::setCookies): Use String instead of DeprecatedString.
(WebCore::cookies): Ditto.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30382
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Mon, 18 Feb 2008 21:40:47 +0000 (21:40 +0000)]
Reviewed by Sam.
- removed use of DeprecatedString in the Color class
* platform/graphics/Color.cpp:
(WebCore::Color::parseHexColor): Streamlined logic a bit with early returns.
Used toASCIIHexValue a character at a time rather than using toIntStrict
in base 16 mode.
(WebCore::findNamedColor): Added. Uses a fixed-size char buffer to look up
a color using the gperf-generated findColor function. Saves a memory allocation
vs. the old version that called DeprecatedString::latin1().
(WebCore::Color::setNamedColor): Changed to use findNamedColor.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30381
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Mon, 18 Feb 2008 21:28:21 +0000 (21:28 +0000)]
Reviewed by Sam.
* wtf/ASCIICType.h:
(WTF::toASCIIHexValue): Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30380
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Mon, 18 Feb 2008 21:20:32 +0000 (21:20 +0000)]
Reviewed by Sam.
* editing/htmlediting.cpp:
(WebCore::stringWithRebalancedWhitespace): Changed to use String instead of
DeprecatedString.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30379
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Mon, 18 Feb 2008 21:16:19 +0000 (21:16 +0000)]
Reviewed by Sam.
* editing/SelectionController.cpp:
(WebCore::SelectionController::debugRenderer): Changed to use String instead of
DeprecatedString.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30378
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
weinig@apple.com [Mon, 18 Feb 2008 20:13:06 +0000 (20:13 +0000)]
Reviewed and landed by Sam.
Remove DeprecatedStringList.
* GNUmakefile.am:
* WebCore.pro:
* WebCore.vcproj/WebCore.vcproj:
* WebCore.xcodeproj/project.pbxproj:
* WebCoreSources.bkl:
* editing/markup.cpp:
* platform/DeprecatedStringList.cpp: Removed.
* platform/DeprecatedStringList.h: Removed.
* platform/mac/DeprecatedStringListMac.mm: Removed.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30377
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Mon, 18 Feb 2008 19:49:52 +0000 (19:49 +0000)]
Reviewed by Sam.
* css/CSSPrimitiveValueMappings.h: Add default cases to all the switch statements.
This will ease the way some day if we decide to use an enum instead of int; otherwise
we'll have a ton of "unhandled enum value" warnings here.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30375
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
alp@webkit.org [Mon, 18 Feb 2008 19:21:05 +0000 (19:21 +0000)]
2008-02-18 Alp Toker <alp@atoker.com>
Build fix for GTK+ < 2.10. Fall back to simple text clipboard copy
with older GTK+ versions for now.
* platform/gtk/PasteboardGtk.cpp:
(WebCore::Pasteboard::writeSelection):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30374
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Mon, 18 Feb 2008 18:59:10 +0000 (18:59 +0000)]
Reviewed by Sam.
* WebCore.base.exp: Export a couple of WebCore::String functions we plan to use
in the future in WebKit.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30373
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Mon, 18 Feb 2008 18:58:02 +0000 (18:58 +0000)]
Reviewed by Sam.
* DerivedSources.make: Added the scripts to the ENABLE_SVG versions of the rules
for CSSPropertyNames.h and CSSValueKeywords.h. Somehow that got left out, so the
files would not be regenerated if the scripts were changed (but would if SVG was
disabled).
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30372
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap@webkit.org [Mon, 18 Feb 2008 18:41:09 +0000 (18:41 +0000)]
Suggested by Darin.
* platform/KURL.h: (WebCore::KURL::operator const String&): Added, to avoid unexpected
conversion via UString (as in bug 17418).
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30371
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
jhoneycutt@apple.com [Mon, 18 Feb 2008 18:23:33 +0000 (18:23 +0000)]
2008-02-18 Jon Honeycutt <jhoneycutt@apple.com>
Reviewed by Darin.
<rdar://problem/
5744899> Crash in Flash when clicking "Yes" to abort
slow script Flash 9 dialog at http://www.kidzui.com
When navigating to a new page, we stop all outstanding PluginStreams.
Flash hangs in the call to NPP_URLNotify. It eventually displays the
"slow script" dialog, which relinquishes control to the system. While
this dialog is running, the request we are in the process of cancelling
completes, and we re-enter Flash to deliver the data. When the dialog
is dismissed, the internal state of Flash has changed, and Flash
crashes with a null dereference.
To work around this, we can defer loading before entering plug-in code,
so that even if a plug-in yields to the system, we won't get callbacks
while we're handling a callback.
* plugins/PluginStream.cpp:
(WebCore::PluginStream::startStream): Defers loads while calling into
plug-in.
(WebCore::PluginStream::destroyStream): Same.
(WebCore::PluginStream::deliverData): Same.
(WebCore::PluginStream::didFail): Protect 'this' from deletion by
destroyStream. Null out m_loader only after destroyStream returns.
(WebCore::PluginStream::didFinishLoading): Same.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30370
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap@webkit.org [Mon, 18 Feb 2008 18:14:41 +0000 (18:14 +0000)]
Reviewed by Darin.
http://bugs.webkit.org/show_bug.cgi?id=17418
REGRESSION: Assertion failure dragging image (JSLock::lockCount() > 0)
* platform/win/ClipboardWin.cpp:
(WebCore::ClipboardWin::declareAndWriteDragImage): Explicitly convert from KURL to String,
as an implicit conversion uses UString and thus needs a JSLock.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30369
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
weinig@apple.com [Mon, 18 Feb 2008 06:45:13 +0000 (06:45 +0000)]
Roll out r30360.
* loader/FrameLoader.cpp:
(WebCore::FrameLoader::changeLocation):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30368
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
weinig@apple.com [Mon, 18 Feb 2008 06:25:04 +0000 (06:25 +0000)]
Mac build fix.
* WebCore.xcodeproj/project.pbxproj:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30366
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
alp@webkit.org [Mon, 18 Feb 2008 02:32:57 +0000 (02:32 +0000)]
2008-02-17 Alp Toker <alp@atoker.com>
Attempt to fix the Wx build (has been broken all weekend).
Stub out some graphics functions.
* platform/graphics/wx/GraphicsContextWx.cpp:
(WebCore::GraphicsContext::beginPath):
(WebCore::GraphicsContext::addPath):
* platform/graphics/wx/PathWx.cpp:
(WebCore::Path::isEmpty):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30363
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
alp@webkit.org [Mon, 18 Feb 2008 02:02:56 +0000 (02:02 +0000)]
2008-02-17 Julien Chaffraix <julien.chaffraix@gmail.com>
Reviewed by Alexey Proskuryakov.
http://bugs.webkit.org/show_bug.cgi?id=16989
bug 16989 : Add send() flag checks in XmlHttpRequest
- Splitted xmlhttprequest-abort-readystate in 2 test cases (xmlhttprequest-abort-readystate-shouldDispatchEvent and xmlhttprequest-abort-readystate-shouldNotDispatchEvent)
- Added test case for the 2 send() flag checks in XmlHttpRequest::send and XmlHttpRequest::setRequestHeader
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30362
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
weinig@apple.com [Mon, 18 Feb 2008 00:22:21 +0000 (00:22 +0000)]
WebCore:
Reviewed by Dan Bernstein.
Fix for http://bugs.webkit.org/show_bug.cgi?id=17365
document.createEvent("MessageEvent") throws NOT_SUPPORTED_ERR
- Updated fast/events/event-instanceof.html to test document.createEvent("MessageEvent").
* DerivedSources.make:
Generate Objective-C binding for DOMProgressEvent which was missing.
* WebCore.xcodeproj/project.pbxproj:
Add missing DOMProgressEvent files to the project.
* bindings/js/JSEventCustom.cpp:
(WebCore::toJS):
Clean up and add case for SVGZoomEvent that was missing.
* bindings/objc/DOMEvents.mm:
(+[DOMEvent _wrapEvent:WebCore::]):
Clean up and add cases for ProgressEvent and MessageEvent that were missing.
* dom/Document.cpp:
(WebCore::Document::createEvent):
Add case for MessageEvent.
WebKit/mac:
Reviewed by Dan Bernstein.
Fix for http://bugs.webkit.org/show_bug.cgi?id=17365
document.createEvent("MessageEvent") throws NOT_SUPPORTED_ERR
* MigrateHeaders.make: Migrate DOMProgressEvent.h and DOMTextPrivate.h which were
mistakenly not migrated.
LayoutTests:
Reviewed by Dan Bernstein.
Update test for http://bugs.webkit.org/show_bug.cgi?id=17365
document.createEvent("MessageEvent") throws NOT_SUPPORTED_ERR
* fast/events/event-instanceof-expected.txt:
* fast/events/resources/event-instanceof.js: Test MessageEvent as well.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30361
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
alp@webkit.org [Mon, 18 Feb 2008 00:14:43 +0000 (00:14 +0000)]
2008-02-17 Adam Treat <treat@kde.org>
Reviewed by Eric Seidel.
http://bugs.webkit.org/show_bug.cgi?id=17008
Meta refresh does not work with cache turned off
Fix for issue noticed on http://adserver.vivox.com/2
* loader/FrameLoader.cpp:
(WebCore::FrameLoader::changeLocation):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30360
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
alp@webkit.org [Sun, 17 Feb 2008 23:53:27 +0000 (23:53 +0000)]
2008-02-17 Alp Toker <alp@atoker.com>
Reviewed by Sam Weinig.
Fix for change made in r30355. Issue noticed by İsmail Dönmez.
Verify SSL certs by default, but allow checks to be disabled with an
environment variable (WEBKIT_IGNORE_SSL_ERRORS) for now.
* platform/network/curl/ResourceHandleManager.cpp:
(WebCore::ResourceHandleManager::startJob):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30359
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap@webkit.org [Sun, 17 Feb 2008 20:21:32 +0000 (20:21 +0000)]
bugs.webkit.org/show_bug.cgi?id=17397
<rdar://problem/
5748245> REGRESSION (r30236-30336): Cannot backspace/enter in forms on Windows
* WebKit.vcproj/WebKit.vcproj: Add ENABLE_CROSS_DOCUMENT_MESSAGING definitions.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30358
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mitz@apple.com [Sun, 17 Feb 2008 19:28:26 +0000 (19:28 +0000)]
Reviewed by Mark Rowe.
- disable the non-ASCII parts of a test case for the ASCII code path
because it is failing on the (Tiger) buildbots for an unrelated
reason.
* fast/text/fixed-pitch-control-characters.html:
* platform/mac/fast/text/fixed-pitch-control-characters-expected.checksum:
* platform/mac/fast/text/fixed-pitch-control-characters-expected.png:
* platform/mac/fast/text/fixed-pitch-control-characters-expected.txt:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30357
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Sun, 17 Feb 2008 17:59:10 +0000 (17:59 +0000)]
* wtf/ListHashSet.h: (WTF::swap): Removed stray return statement.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30356
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
alp@webkit.org [Sun, 17 Feb 2008 15:36:49 +0000 (15:36 +0000)]
2008-02-17 Bin Chen <binary.chen@gmail.com>
Reviewed by Alp Toker.
http://bugs.webkit.org/show_bug.cgi?id=17404
Bug 17404: curl certification problem
Disable SSL cert verification until we have a way of distributing
certs and/or reporting SSL errors to the user.
* platform/network/curl/ResourceHandleManager.cpp:
(WebCore::ResourceHandleManager::startJob):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30355
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
alp@webkit.org [Sun, 17 Feb 2008 15:09:51 +0000 (15:09 +0000)]
2008-02-17 Alp Toker <alp@atoker.com>
Reviewed by Mark Rowe.
DevHelp fails to load local files; URL truncated by one character.
Fix a file:// URL regression introduced in KURL.cpp r30243.
* platform/KURL.cpp:
(WebCore::KURL::KURL):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30354
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mrowe@apple.com [Sun, 17 Feb 2008 12:12:54 +0000 (12:12 +0000)]
2008-02-17 Jan Michael Alonzo <jmalonzo@unpluggable.com>
Reviewed by Mark Rowe.
Fix a crash introduced in changeset #29985 by moving the dereference to after
the null check.
* webkit/webkitwebhistoryitem.cpp:
(_WebKitWebHistoryItemPrivate::webkit_web_history_item_new_with_core_item):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30353
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mitz@apple.com [Sun, 17 Feb 2008 08:50:32 +0000 (08:50 +0000)]
WebCore:
Reviewed by Darin Adler.
- fix http://bugs.webkit.org/show_bug.cgi?id=17033
<rdar://problem/
5709315> REGRESSION: Really long <option> causes unnecessary page scroll bars to accommodate content
Test: fast/forms/control-clip-overflow.html
* rendering/RenderFlow.cpp:
(WebCore::RenderFlow::lowestPosition): Account for control clipping.
(WebCore::RenderFlow::rightmostPosition): Ditto.
(WebCore::RenderFlow::leftmostPosition): Ditto.
LayoutTests:
Reviewed by Darin Adler.
- test for http://bugs.webkit.org/show_bug.cgi?id=17033
<rdar://problem/
5709315> REGRESSION: Really long <option> causes unnecessary page scroll bars to accommodate content
* fast/forms/control-clip-overflow.html: Added.
* platform/mac/fast/forms/control-clip-overflow-expected.checksum: Added.
* platform/mac/fast/forms/control-clip-overflow-expected.png: Added.
* platform/mac/fast/forms/control-clip-overflow-expected.txt: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30345
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
oliver@apple.com [Sun, 17 Feb 2008 05:48:19 +0000 (05:48 +0000)]
Bug 17269: Deobfuscate CanvasRenderingContext2D.cpp
Reviewed by Eric S
Use cross-platform code to determine the dirty rects for
fill and stroke operations
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30344
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
weinig@apple.com [Sun, 17 Feb 2008 03:43:01 +0000 (03:43 +0000)]
Reviewed by Darin Adler.
Take another step in the direction of getting rid of DeprecatedString
by moving all the to{NumericType} off of it.
- Create free functions that take a UChar* buffer and length to do
the string-to-number conversions. This allows us to avoid two allocations
if we don't already have a String and is consistent with the design we would
like going forward.
- Since the toInt (and family) functions on DeprecatedString were slightly
different than the ones on String (they didn't allow trailing garbage),
an extra set of 'Strict' toInt functions were added that have this behavior.
* platform/graphics/Color.cpp:
(WebCore::Color::parseHexColor):
* platform/text/PlatformString.h:
* platform/text/String.cpp:
(WebCore::String::percentage):
(WebCore::String::toIntStrict):
(WebCore::String::toUIntStrict):
(WebCore::String::toInt64Strict):
(WebCore::String::toUInt64Strict):
(WebCore::String::toUInt):
(WebCore::String::toDouble):
(WebCore::isCharacterAllowedInBase):
(WebCore::toIntegralType):
(WebCore::lengthOfCharactersAsInteger):
(WebCore::charactersToIntStrict):
(WebCore::charactersToUIntStrict):
(WebCore::charactersToInt64Strict):
(WebCore::charactersToUInt64Strict):
(WebCore::charactersToInt):
(WebCore::charactersToUInt):
(WebCore::charactersToInt64):
(WebCore::charactersToUInt64):
(WebCore::charactersToDouble):
(WebCore::charactersToFloat):
* platform/text/StringImpl.cpp:
(WebCore::parseLength):
(WebCore::StringImpl::toIntStrict):
(WebCore::StringImpl::toUIntStrict):
(WebCore::StringImpl::toInt64Strict):
(WebCore::StringImpl::toUInt64Strict):
(WebCore::StringImpl::toInt):
(WebCore::StringImpl::toUInt):
(WebCore::StringImpl::toInt64):
(WebCore::StringImpl::toUInt64):
(WebCore::StringImpl::toDouble):
(WebCore::StringImpl::toFloat):
* platform/text/StringImpl.h:
* svg/SVGAnimationElement.cpp:
(WebCore::SVGAnimationElement::parseClockValue):
* svg/SVGFETurbulenceElement.cpp:
(WebCore::SVGFETurbulenceElement::parseMappedAttribute):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30343
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mitz@apple.com [Sun, 17 Feb 2008 03:08:37 +0000 (03:08 +0000)]
WebCore:
Reviewed by Sam Weinig.
- fix fixed-pitch font measurement of control characters that render
as zero-width space
Test: fast/text/fixed-pitch-control-characters.html
* rendering/RenderText.cpp:
(WebCore::RenderText::widthFromCache):
LayoutTests:
Reviewed by Sam Weinig.
- test fixed-pitch font measurement of control characters that render
as zero-width space
* fast/text/fixed-pitch-control-characters.html: Added.
* platform/mac/fast/text/fixed-pitch-control-characters-expected.checksum: Added.
* platform/mac/fast/text/fixed-pitch-control-characters-expected.png: Added.
* platform/mac/fast/text/fixed-pitch-control-characters-expected.txt: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30342
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
kevino@webkit.org [Sat, 16 Feb 2008 22:04:47 +0000 (22:04 +0000)]
wx build fix.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30341
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Sat, 16 Feb 2008 17:41:39 +0000 (17:41 +0000)]
- disable some failing tests; someone can re-enable after fixing the problems causing
them to fail or generating new expected results
http://bugs.webkit.org/show_bug.cgi?id=17391
* fast/frames/iframe-scroll-page-up-down.html: Removed.
* fast/frames/iframe-scroll-page-up-down.html-disabled: Copied from fast/frames/iframe-scroll-page-up-down.html.
http://bugs.webkit.org/show_bug.cgi?id=17392
* svg/batik/text/textBiDi.svg: Removed.
* svg/batik/text/textBiDi.svg-disabled: Copied from svg/batik/text/textBiDi.svg.
http://bugs.webkit.org/show_bug.cgi?id=17393
* svg/custom/use-on-symbol-inside-pattern.svg: Removed.
* svg/custom/use-on-symbol-inside-pattern.svg-disabled: Copied from svg/custom/use-on-symbol-inside-pattern.svg.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30340
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Sat, 16 Feb 2008 17:35:14 +0000 (17:35 +0000)]
Clean up some old log entries.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30339
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
alp@webkit.org [Sat, 16 Feb 2008 11:09:38 +0000 (11:09 +0000)]
2008-02-16 Jan Michael Alonzo <jmalonzo@unpluggable.com>
Reviewed by Alp Toker.
Cross document messaging GTK+/autotools build fix.
* GNUmakefile.am:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30338
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
oliver@apple.com [Sat, 16 Feb 2008 06:44:46 +0000 (06:44 +0000)]
Build fix for Qt and Cairo builds
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30337
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
oliver@apple.com [Sat, 16 Feb 2008 06:00:20 +0000 (06:00 +0000)]
Bug 17269: Deobfuscate CanvasRenderingContext2D.cpp
Refactor CanvasRenderingContext2D::drawImage(HTMLCanvasElement) to remove evil ifdefs
Reviewed by Dan B.
Add logic draw(ImageBuffer*) method to GraphicsContext to handle
painting the source canvas content.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30336
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mitz@apple.com [Sat, 16 Feb 2008 05:57:31 +0000 (05:57 +0000)]
Rubber-stamped by Oliver Hunt.
- skip a test for a Leopard-only NSHTTPURLResponse bug workaround
* platform/mac-tiger/Skipped:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30335
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
kevino@webkit.org [Sat, 16 Feb 2008 03:23:54 +0000 (03:23 +0000)]
Using GetNativeFontInfoDesc() to generate the hash value was leading to duplicate entries in the HashMap. Use the font object's pointer instead.
http://bugs.webkit.org/show_bug.cgi?id=17371
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30334
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
adachan@apple.com [Sat, 16 Feb 2008 02:51:34 +0000 (02:51 +0000)]
When parsing url we get from a CFURLRef, we need to null terminate
the string for the case when url ends with a '/'.
Reviewed by Darin.
* platform/cf/KURLCFNet.cpp:
(WebCore::KURL::KURL):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30333
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
beidson@apple.com [Sat, 16 Feb 2008 02:02:30 +0000 (02:02 +0000)]
Added 2 skipped tests
* platform/mac-tiger/Skipped:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30332
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
beidson@apple.com [Sat, 16 Feb 2008 00:51:38 +0000 (00:51 +0000)]
WebCore:
Reviewed by Darin
Fix for <rdar://problem/
5727175> and <rdar://problem/
5740495> - Database threads and callback scripts can run after
a page has closed or loaded a new document
Deciding to make the Database I/O semantic the same as loaders/XHR when a document is shut down, this patch implements
a policy of shutting down the databases in a document at the same time. This includes removing all pending transactions
in a database, cutting off an queued statements in the current transaction, and preventing further callbacks from being
made.
No new layout tests with this patch as the current layout tests were catching this issue in a plethora of ways already
(crashing, unexpected exceptions and output, etc)
* dom/Document.cpp:
(WebCore::Document::~Document): Don't actually stop the database thread here - it better have been stopped already.
Add an assertion to that effect.
(WebCore::Document::addOpenDatabase): Add a new database handle to this Document's open database set
(WebCore::Document::removeOpenDatabase): Remove such a handle
(WebCore::Document:: stopDatabases): Call "close" on all open Database handles for this document
* dom/Document.h:
* loader/FrameLoader.cpp:
(WebCore::FrameLoader::stopLoading): In addition to canceling all resource loads and XHRs, stop all database I/O
* platform/MessageQueue.h:
(WebCore::MessageQueue::killed):
* platform/sql/SQLiteTransaction.cpp:
(WebCore::SQLiteTransaction::stop): Added. Explicit stop to cut off a transaction so it won't try anymore SQL activity
* platform/sql/SQLiteTransaction.h:
* storage/Database.cpp:
(WebCore::Database::Database):
(WebCore::Database::~Database):
(WebCore::Database::markAsDeletedAndClose): Check if the thread has terminated before committing to waiting on the
thread.
(WebCore::Database::stop): Stop this database, including all queued transactions and callbacks
* storage/Database.h:
(WebCore::Database::stopped):
* storage/DatabaseThread.cpp:
(WebCore::DatabaseThread::terminationRequested):
* storage/DatabaseThread.h:
* storage/SQLTransaction.cpp:
(WebCore::SQLTransaction::executeSQL): Throw an exception if a new executeSQL comes in after a database is closed
(WebCore::SQLTransaction::checkAndHandleClosedDatabase): Added. Clears queued statements and clear the next step
when the database has been closed since the last step/callback was run. Also stops the current SQLite transaction,
if any
(WebCore::SQLTransaction::performNextStep):
(WebCore::SQLTransaction::performPendingCallback):
* storage/SQLTransaction.h:
LayoutTests:
Reviewed by Darin
Fix for <rdar://problem/
5727175> and <rdar://problem/
5740495> - Database threads and callback scripts can run after
a page has closed or loaded a new document
* storage/close-during-stress-test-expected.txt: Update results - this test contained output from a javascript callback
that never should have taken place
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30331
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
adele@apple.com [Sat, 16 Feb 2008 00:44:56 +0000 (00:44 +0000)]
WebCore:
Reviewed by Darin.
Fix for <rdar://problem/
5745072> REGRESSION (r29348): Shift + Tab does not change indent level on Google Docs
The immediate cause of this bug was that we stopped sending keypress events for the tab key when it is used to advance focus.
We had a special case for forward-tab in designMode, where the default behavior was to insert a tab key (or respect the keypress handler behavior).
This change makes the shift-tab behavior match the forward-tab behavior.
If the site had put their event handler (which does the indenting) on the keydown event, then this problem would have been avoided.
This is something we should look into and maybe advise the site on in the future. However, it's a low-risk change to just make tab and shift-tab uniform
in this respect, so I think this is the way to go for right now.
* page/EventHandler.cpp: (WebCore::EventHandler::defaultTabEventHandler):
LayoutTests:
Reviewed by Darin.
Updated test for <rdar://problem/
5745072> REGRESSION (r29348): Shift + Tab does not change indent level on Google Docs
* editing/inserting/typing-tab-designmode-expected.txt:
* editing/inserting/typing-tab-designmode.html:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30330
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
andersca@apple.com [Fri, 15 Feb 2008 23:47:12 +0000 (23:47 +0000)]
Reviewed by Alice.
<rdar://problem/
5738678>
REGRESSION: "Loading" status remains when uploading file to .Mac iDisk via Safari
Use the new CFNetwork functions for setting body parts.
* platform/network/cf/FormDataStreamCFNet.cpp:
(WebCore::setHTTPBody):
(WebCore::httpBodyFromRequest):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30329
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ggaren@apple.com [Fri, 15 Feb 2008 23:39:14 +0000 (23:39 +0000)]
WebCore:
Reviewed by Anders Carlsson.
Fixed <rdar://problem/
5725429> REGRESSION (r27898): Greenfield online
surveys no longer work due to XMLHttpRequest exceptions
Reverted some exception throwing code from r12194.
To comply with the W3C draft spec, we used to throw an exception when
trying to access responseText and responseXML at the wrong time, but
that turned out to be a compatibility problem.
Now, matching Firefox and previous versions of WebKit, we never throw
an exception when accessing responseText or responseXML.
See http://www.mail-archive.com/public-webapi@w3.org/msg02756.html.
* xml/XMLHttpRequest.cpp:
(WebCore::XMLHttpRequest::getResponseText):
(WebCore::XMLHttpRequest::getResponseXML):
LayoutTests:
Reviewed by Anders Carlsson.
Fixed <rdar://problem/
5725429> REGRESSION (r27898): Greenfield online
surveys no longer work due to XMLHttpRequest exceptions
Updated layout tests to expect an exception not to be thrown.
* http/tests/xmlhttprequest/xmlhttprequest-responseText-exception.html:
* http/tests/xmlhttprequest/xmlhttprequest-responseXML-exception.html:
* http/tests/xmlhttprequest/zero-length-response-expected.txt:
* http/tests/xmlhttprequest/zero-length-response-sync-expected.txt:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30328
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
alice.liu@apple.com [Fri, 15 Feb 2008 23:27:39 +0000 (23:27 +0000)]
minor correction to my last Changlog entry.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30327
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
justin.garcia@apple.com [Fri, 15 Feb 2008 23:25:26 +0000 (23:25 +0000)]
WebCore:
Reviewed by Dan Bernstein.
<rdar://problem/
5738768> REGRESSION (r30062): Crash in InlineTextBox::isLineBreak() when Undoing a replace
Rolled out <http://trac.webkit.org/projects/webkit/changeset/29667>
* editing/SelectionController.cpp:
(WebCore::SelectionController::nodeWillBeRemoved):
LayoutTests:
Reviewed by Dan Bernstein.
<rdar://problem/
5738768> REGRESSION (r30062): Crash in InlineTextBox::isLineBreak() when Undoing a replace
Disabled:
* editing/selection/inconsistent-in-removeChildNode.html: Removed.
* editing/selection/inconsistent-in-removeChildNode.html-disabled: Added.
Demonstrates fix:
* editing/undo/
5738768-expected.txt: Added.
* editing/undo/
5738768.html: Added.
During a shouldChangeSelection call, the old selection is no longer null because
nodeWillBeRemoved doesn't blow it away:
* platform/mac/editing/style/remove-underline-across-paragraph-in-bold-expected.txt:
* platform/mac/editing/style/remove-underline-after-paragraph-in-bold-expected.txt:
* platform/mac/editing/style/remove-underline-in-bold-expected.txt:
During a shouldChangeSelection call, the old selection is now null because
nodeWillBeRemoved blows it away:
* platform/mac/editing/style/remove-underline-across-paragraph-expected.txt:
More instances of <rdar://problem/
5729315>, where, during a shouldChangeSelection
call content that held the old selection is still around but it wouldn't make sense
to leave the selection there, so we shouldn't call shouldChangeSelection at all:
* platform/mac/editing/style/remove-underline-expected.txt:
* platform/mac/editing/style/unbold-in-bold-expected.txt:
Similar changes, but ones where I've also updated pixel results for old, unrelated fixes:
* platform/mac-leopard/editing/pasteboard/paste-RTFD-expected.txt:
* platform/mac/editing/deleting/delete-leading-ws-001-expected.checksum:
* platform/mac/editing/deleting/delete-leading-ws-001-expected.txt:
* platform/mac/editing/deleting/delete-line-011-expected.checksum:
* platform/mac/editing/deleting/delete-line-011-expected.txt:
* platform/mac/editing/execCommand/paste-1-expected.checksum:
* platform/mac/editing/execCommand/paste-1-expected.txt:
* platform/mac/editing/execCommand/paste-2-expected.checksum:
* platform/mac/editing/execCommand/paste-2-expected.txt:
* platform/mac/editing/pasteboard/emacs-ctrl-k-y-001-expected.checksum:
* platform/mac/editing/pasteboard/emacs-ctrl-k-y-001-expected.txt:
* platform/mac/editing/pasteboard/paste-RTFD-expected.txt:
* platform/mac/editing/pasteboard/paste-text-012-expected.checksum:
* platform/mac/editing/selection/
4960116-expected.checksum:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30326
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
alice.liu@apple.com [Fri, 15 Feb 2008 23:18:25 +0000 (23:18 +0000)]
Reviewed by Darin.
Fixed <rdar://problem/
5741440> REGRESSION (r28496): After deactivating JavaScript, scripts embedded in the HTML page continue to run
Before this patch, Frame::scriptProxy() would only return null in the case that javascript was
disabled and if the script proxy field wasn't set (which would only be the case if the window
hasn't loaded anything yet). Not all callers of scriptProxy() always check for a non-null return
value. Those that did check would effectively be checking if javascript was enabled before proceeding.
This fix consists of 2 elements: first, make sure that scriptProxy() will never return null, regardless
of whether javascript is disabled. This will mean that callers who don't check for null won't crash.
Second, callers who did check for null now instead check for javascript being disabled. This means that
code paths intended for preventing javascript from being run will be making the correct check. Another
minor addition to this patch is that I added a function on Frame to be a shortcut for checking if javascript
is enabled.
* bindings/js/JSCustomSQLStatementCallback.cpp:
(WebCore::JSCustomSQLStatementCallback::handleEvent):
* bindings/js/JSCustomSQLStatementErrorCallback.cpp:
(WebCore::JSCustomSQLStatementErrorCallback::handleEvent):
* bindings/js/JSCustomSQLTransactionCallback.cpp:
(WebCore::JSCustomSQLTransactionCallback::handleEvent):
* bindings/js/JSCustomSQLTransactionErrorCallback.cpp:
(WebCore::JSCustomSQLTransactionErrorCallback::handleEvent):
* bindings/js/JSCustomVoidCallback.cpp:
(WebCore::JSCustomVoidCallback::handleEvent):
* bindings/js/JSCustomXPathNSResolver.cpp:
(WebCore::JSCustomXPathNSResolver::lookupNamespaceURI):
* bindings/js/ScheduledAction.cpp:
(WebCore::ScheduledAction::execute):
* bindings/js/kjs_events.cpp:
(WebCore::JSAbstractEventListener::handleEvent):
(WebCore::JSLazyEventListener::parseCode):
* bindings/js/kjs_html.cpp:
(WebCore::runtimeObjectImplementsCall):
* bindings/js/kjs_proxy.cpp:
(WebCore::KJSProxy::isEnabled):
* bindings/js/kjs_proxy.h:
* bindings/js/kjs_window.cpp:
(KJS::Window::retrieveWindow):
(KJS::Window::retrieve):
* dom/Document.cpp:
(WebCore::Document::createHTMLEventListener):
* dom/EventTarget.cpp:
(WebCore::EventTarget::dispatchGenericEvent):
* html/HTMLPlugInElement.cpp:
(WebCore::HTMLPlugInElement::createNPObject):
* html/HTMLScriptElement.cpp:
(WebCore::HTMLScriptElement::evaluateScript):
* html/HTMLTokenizer.cpp:
(WebCore::HTMLTokenizer::parseTag):
(WebCore::HTMLTokenizer::processToken):
* loader/FrameLoader.cpp:
(WebCore::FrameLoader::executeScript):
(WebCore::FrameLoader::userGestureHint):
(WebCore::FrameLoader::open):
(WebCore::FrameLoader::dispatchWindowObjectAvailable):
(WebCore::FrameLoader::switchOutLowBandwidthDisplayIfReady):
* manual-tests/disable-javascript-reload.html: Added.
* page/Frame.cpp:
(WebCore::Frame::scriptProxy):
(WebCore::Frame::bindingRootObject):
(WebCore::Frame::windowScriptNPObject):
* page/Frame.h:
* page/InspectorController.cpp:
(WebCore::canPassNodeToJavaScript):
* page/mac/FrameMac.mm:
(WebCore::Frame::windowScriptObject):
* svg/SVGDocumentExtensions.cpp:
(WebCore::SVGDocumentExtensions::createSVGEventListener):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30325
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mitz@apple.com [Fri, 15 Feb 2008 21:30:48 +0000 (21:30 +0000)]
WebCore:
Reviewed by Alexey Proskuryakov.
- WebCore part of fixing http://bugs.webkit.org/show_bug.cgi?id=17360
<rdar://problem/
5743131> REGRESSION: mp4 file downloaded from server is downloaded as html
Test: http/tests/loading/text-content-type-with-binary-extension.html
Refined the workaround for <rdar://problem/
5321972> to exclude files
with extensions that are known to be associated with binary MIME types.
* WebCore.xcodeproj/project.pbxproj: Added WebCoreURLResponse.{h,mm}.
* platform/network/mac/ResourceResponseMac.mm:
(WebCore::ResourceResponse::doUpdateResourceResponse): Moved the
workaround logic into WebCoreURLResponse.
* platform/network/mac/WebCoreURLResponse.h: Added.
* platform/network/mac/WebCoreURLResponse.mm: Added.
(createBinaryExtensionsSet): Returns a set of extensions known to
belong to MIME types of binary data.
(-[NSURLResponse _webcore_MIMEType]):
(-[NSHTTPURLResponse _webcore_MIMEType]): Forces the MIME type from
application/octet-stream to text/plain if that is the specified
Content-Type, unless the extension is in the binary extensions set.
WebKit/mac:
Reviewed by Alexey Proskuryakov.
- WebKit part of fixing http://bugs.webkit.org/show_bug.cgi?id=17360
<rdar://problem/
5743131> REGRESSION: mp4 file downloaded from server is downloaded as html
* WebView/WebDataSource.mm:
(+[WebDataSource _representationClassForMIMEType:]):
(-[WebDataSource _responseMIMEType]):
(-[WebDataSource subresources]):
(-[WebDataSource subresourceForURL:]):
* WebView/WebResource.mm:
(-[WebResource _initWithData:URL:response:]):
* WebView/WebResourcePrivate.h:
LayoutTests:
Reviewed by Alexey Proskuryakov.
- test for http://bugs.webkit.org/show_bug.cgi?id=17360
<rdar://problem/
5743131> REGRESSION: mp4 file downloaded from server is downloaded as html
* http/tests/loading/resources/.htaccess: Added.
* http/tests/loading/resources/binaryData.m4a: Added.
* http/tests/loading/text-content-type-with-binary-extension-expected.txt: Added.
* http/tests/loading/text-content-type-with-binary-extension.html: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30323
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
weinig@apple.com [Fri, 15 Feb 2008 21:26:50 +0000 (21:26 +0000)]
Reviewed and rubber-stamped by Oliver Hunt.
Add leopard specific results for stroke-width-click.svg to work around
a CG regression.
* platform/mac-leopard/svg/custom/stroke-width-click-expected.txt: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30320
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mitz@apple.com [Fri, 15 Feb 2008 21:08:05 +0000 (21:08 +0000)]
Reviewed by Dave Hyatt.
- fix http://bugs.webkit.org/show_bug.cgi?id=17306
<rdar://problem/
5737923> Transitions between styles that have different transition-* properties behave inconsistently
* manual-tests/transitions.html: Added.
* page/AnimationController.cpp:
(WebCore::CompositeImplicitAnimation::animate): Changed to use the
transition properties of the current style rather than the target style.
(WebCore::AnimationControllerPrivate::get): Changed to not create an
animation if the style does not have transitions.
(WebCore::AnimationController::updateImplicitAnimations): Added code to
return the target style if the current style is not animating.
* rendering/RenderObject.cpp:
(WebCore::RenderObject::setAnimatableStyle): Changed to call
updateImplicitAnimations() even if the current style does not have
transitions, because we may be animating out of a style that had them.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30319
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap@webkit.org [Fri, 15 Feb 2008 20:06:10 +0000 (20:06 +0000)]
Rubber-stamped by Darin.
Remove an obsolete WebCore readme file.
* README: Removed.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30306
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
weinig@apple.com [Fri, 15 Feb 2008 18:29:46 +0000 (18:29 +0000)]
Reviewed by Darin Adler.
Remove more uses of DeprecatedString in preparation of getting rid of it.
* bridge/mac/WebCoreScriptDebugger.mm:
* css/CSSCursorImageValue.cpp:
(WebCore::isSVGCursorIdentifier):
(WebCore::CSSCursorImageValue::updateIfSVGCursorIsUsed):
* css/CSSStyleSelector.h:
* dom/Element.cpp:
(WebCore::Element::dump):
(WebCore::Element::formatForDebugger):
* dom/Position.cpp:
(WebCore::Position::debugPosition):
(WebCore::Position::formatForDebugger):
* dom/Range.cpp:
(WebCore::Range::formatForDebugger):
* dom/Text.cpp:
(WebCore::Text::formatForDebugger):
* editing/Selection.cpp:
(WebCore::Selection::debugPosition):
(WebCore::Selection::formatForDebugger):
* editing/SelectionController.cpp:
(WebCore::SelectionController::debugRenderer):
* editing/VisiblePosition.cpp:
(WebCore::VisiblePosition::debugPosition):
* html/HTMLTokenizer.cpp:
(WebCore::HTMLTokenizer::scriptHandler):
(WebCore::HTMLTokenizer::parseTag):
(WebCore::HTMLTokenizer::processToken):
(WebCore::HTMLTokenizer::notifyFinished):
* svg/SVGFontFaceElement.cpp:
(WebCore::mapAttributeToCSSProperty):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30293
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
alp@webkit.org [Fri, 15 Feb 2008 18:25:44 +0000 (18:25 +0000)]
2008-02-15 Alp Toker <alp@atoker.com>
Reviewed by Holger.
GTK+ configure script cleanups
Categorize the configuration summary printout.
Bump GTK+ requirement to 2.8.
Rename 'webkit_target' to just 'target'.
Don't check for pthread on Win32.
* configure.ac:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30291
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
aroben@apple.com [Fri, 15 Feb 2008 18:04:57 +0000 (18:04 +0000)]
* bindings/scripts/CodeGenerator.pm: Touch this to force bindings to regenerate.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30285
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
aroben@apple.com [Fri, 15 Feb 2008 17:49:10 +0000 (17:49 +0000)]
Mac build fix
JavaScriptCore:
Make JavaScriptCore's FEATURE_DEFINES match WebCore's
Reviewed by Mark.
* Configurations/JavaScriptCore.xcconfig:
WebKit/mac:
Make WebKit's FEATURE_DEFINES match WebCore's
Reviewed by Mark.
* Configurations/WebKit.xcconfig:
WebKitTools:
Fix a typo that broke the Mac build
Reviewed by Mark.
* Scripts/build-webkit:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30276
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
aroben@apple.com [Fri, 15 Feb 2008 17:04:37 +0000 (17:04 +0000)]
Try to fix Qt/GTK+ builds
* WebCore.pro: Remove MessageEvent.{idl,cpp} from the unconditional
parts of this file.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30272
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
aroben@apple.com [Fri, 15 Feb 2008 16:36:54 +0000 (16:36 +0000)]
Turn on cross-document messaging support by default
top level:
Turn on cross-document messaging support by default
Reviewed by Darin.
* configure.ac:
WebCore:
Turn on cross-document messaging support by default
Reviewed by Darin.
* Configurations/WebCore.xcconfig:
* WebCore.pro:
* WebCore.vcproj/WebCore.vcproj:
WebKitTools:
Turn on cross-document messaging support by default
Reviewed by Darin.
* Scripts/build-webkit:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30270
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
aroben@apple.com [Fri, 15 Feb 2008 16:36:07 +0000 (16:36 +0000)]
Conditionalize cross-document messaging support
top level:
Conditionalize cross-document messaging support
The cross-document messaging parts of HTML 5 are in flux and we want
ports to be able to turn off the support as needed.
Note that the support is turned off by default right now. A subsequent
commit will turn it on by default.
Reviewed by Darin.
* configure.ac:
WebCore:
Conditionalize cross-document messaging support
The cross-document messaging parts of HTML 5 are in flux and we want
ports to be able to turn off the support as needed.
Note that the support is turned off by default right now. A subsequent
commit will turn it on by default.
Reviewed by Darin.
* GNUmakefile.am:
* WebCore.vcproj/build-generated-files.sh:
* bindings/js/JSDOMWindowCustom.cpp:
(WebCore::JSDOMWindow::customGetOwnPropertySlot):
* bindings/js/JSEventCustom.cpp:
(WebCore::toJS):
* dom/Event.cpp:
(WebCore::Event::isMessageEvent):
* dom/Event.h:
* dom/MessageEvent.cpp:
* dom/MessageEvent.h:
* dom/MessageEvent.idl:
* page/DOMWindow.cpp:
(WebCore::DOMWindow::postMessage):
* page/DOMWindow.h:
* page/DOMWindow.idl:
WebKitTools:
Conditionalize cross-document messaging support
The cross-document messaging parts of HTML 5 are in flux and we want
ports to be able to turn off the support as needed.
Note that the support is turned off by default right now. A subsequent
commit will turn it on by default.
Reviewed by Darin.
* Scripts/build-webkit:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30269
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Fri, 15 Feb 2008 15:18:53 +0000 (15:18 +0000)]
Reviewed by Adam.
- quick fix for a problem causing an assertion on launch
* WebFrame.cpp:
(WebFrame::loadData): Make an empty KURL even if the BSTR is null.
Later we might want to rethink this.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30268
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
alp@webkit.org [Fri, 15 Feb 2008 11:41:23 +0000 (11:41 +0000)]
2008-02-15 Alp Toker <alp@atoker.com>
Fix the GTK+ build following breakage introduced in r30243.
* WebCoreSupport/ChromeClientGtk.cpp:
(WebKit::ChromeClient::mouseDidMoveOverElement):
* WebCoreSupport/FrameLoaderClientGtk.cpp:
(WebKit::FrameLoaderClient::objectContentType):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30267
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Fri, 15 Feb 2008 08:21:34 +0000 (08:21 +0000)]
- another Qt build fix
* platform/qt/KURLQt.cpp:
(WebCore::KURL::operator QUrl): Use the characters directly, not ascii().
Eliminate references to urlString.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30252
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Fri, 15 Feb 2008 08:12:08 +0000 (08:12 +0000)]
WebCore:
- another Qt build fix
* platform/qt/ClipboardQt.cpp:
(WebCore::ClipboardQt::declareAndWriteDragImage): Use KURL instead of String.
WebKit/gtk:
- another try at fixing the build
* webkit/webkitwebview.cpp: Added some explicit conversions to KURL.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30251
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Fri, 15 Feb 2008 07:23:34 +0000 (07:23 +0000)]
- another round of build fixes
* platform/KURL.cpp:
(WebCore::appendEncodedHostname): Fix spelling (strLen, not strlen).
* platform/network/curl/ResourceHandleManager.cpp:
(WebCore::parseDataUrl): Fix a ".." typo and call data instead of characters.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30250
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Fri, 15 Feb 2008 06:45:51 +0000 (06:45 +0000)]
- added back accidentally-removed files
* platform/DeprecatedStringList.cpp: Copied from platform/DeprecatedStringList.cpp.
* platform/DeprecatedStringList.h: Copied from platform/DeprecatedStringList.h.
* platform/mac/DeprecatedStringListMac.mm: Copied from platform/mac/DeprecatedStringListMac.mm.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30249
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Fri, 15 Feb 2008 06:38:49 +0000 (06:38 +0000)]
- more build fixes
* platform/KURL.cpp:
(WebCore::appendEncodedHostname): Use String to make a QString.
* platform/network/cf/ResourceRequest.h:
(WebCore::ResourceRequest::ResourceRequest): Removed a stray deprecatedString() call.
* platform/network/curl/ResourceHandleManager.cpp:
(WebCore::parseDataUrl): Updated to String rather than DeprecatedString functions.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30248
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
oliver@apple.com [Fri, 15 Feb 2008 06:03:11 +0000 (06:03 +0000)]
Make assign-to-window-status testcase to be more thorough
Reviewed by Jon H
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30247
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Fri, 15 Feb 2008 05:36:58 +0000 (05:36 +0000)]
- next Qt build fix
* platform/network/qt/ResourceRequest.h:
(WebCore::ResourceRequest::ResourceRequest): Removed a stray deprecatedString() call.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30246
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Fri, 15 Feb 2008 05:29:13 +0000 (05:29 +0000)]
- another build fix
* platform/network/curl/ResourceRequest.h:
(WebCore::ResourceRequest::ResourceRequest): Removed a stray deprecatedString() call.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30245
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Fri, 15 Feb 2008 05:14:14 +0000 (05:14 +0000)]
- first Qt build fix
* dom/XMLTokenizer.cpp:
(WebCore::XMLTokenizer::endElementNs): Added a missing string() call.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30244
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Fri, 15 Feb 2008 05:03:55 +0000 (05:03 +0000)]
WebCore:
Reviewed by Eric Seidel.
Based on work by Marvin Decker <marv.decker@gmail.com>
- fix http://bugs.webkit.org/show_bug.cgi?id=16538
KURL should use String instead of DeprecatedString
- fix http://bugs.webkit.org/show_bug.cgi?id=16485
DocLoader::checkForReload will crash if the URL isNull
and a similar problem in IconDatabase
- fix http://bugs.webkit.org/show_bug.cgi?id=16487
KURL doesn't preserve isNull when constructed with a DeprecatedString
- changed completeURL and various DOM getters to return KURL, to avoid
conversion back and forth from KURL to String
- changed the conversion of KURL to NSURL or NSString to be automatic,
to ease the use of KURL in Objective C DOM bindings, and eliminated
the getNSURL function
- because I had to visit the DOM bindings anyway, eliminated almost all
the use of the KJS namespace for things in WebCore
- fixed HTMLOptionElement constructor to check for undefined rather
than size of the arguments array
- eliminated some other unnecessary uses of DeprecatedString
- changed String::split to take a Vector parameter instead of returning
a Vector, for better performance
- added a couple of missing calls to do layout in SVG image handling;
I was able to reproduce these only because I had broken URLs for a
while -- not sure how to reproduce them now but the changes are
clearly needed
Performance testing shows this to be at least a 1% speedup.
Added a new function protocolIs to efficiently compare protocols
without case errors and a blankURL function so we don't have to
code "about:blank" in multiple places in the code and don't have to
construct a frash KURL each time. Moved decode_string and encode_string
out of KURL and gave them clearer names.
Made KURL constructors explicit to highlight potentially-expensive
operations and the poor semantics of KURL's constructor that takes
a String.
* WebCore.base.exp: Updated.
* bindings/js/JSAttrCustom.cpp:
(WebCore::JSAttr::setValue): Use protocolIs.
* bindings/js/JSAudioConstructor.h: KJS namespace change.
* bindings/js/JSCSSRuleCustom.cpp:
(WebCore::toJS): Ditto.
* bindings/js/JSCSSValueCustom.cpp:
(WebCore::toJS): Ditto.
* bindings/js/JSDocumentCustom.cpp:
(WebCore::JSDocument::location): Ditto.
(WebCore::JSDocument::setLocation): Updated for KURL change.
(WebCore::toJS): KJS namespace change.
* bindings/js/JSElementCustom.cpp:
(WebCore::allowSettingSrcToJavascriptURL): Use protocolIs.
(WebCore::JSElement::setAttribute): KJS namespace change.
(WebCore::JSElement::setAttributeNode): Ditto.
(WebCore::JSElement::setAttributeNS): Ditto.
(WebCore::JSElement::setAttributeNodeNS): Ditto.
* bindings/js/JSHTMLFrameElementCustom.cpp:
(WebCore::allowSettingJavascriptURL): Use protocolIs.
(WebCore::JSHTMLFrameElement::setSrc): KJS namespace change.
(WebCore::JSHTMLFrameElement::setLocation): Ditto.
* bindings/js/JSHTMLIFrameElementCustom.cpp:
(WebCore::JSHTMLIFrameElement::setSrc): Use protocolIs.
* bindings/js/JSHTMLOptionElementConstructor.cpp:
(WebCore::JSHTMLOptionElementConstructor::JSHTMLOptionElementConstructor):
(WebCore::JSHTMLOptionElementConstructor::construct): Cleaned up the
structure a bit and changed checking to check for undefined rather than
number of arguments.
* bindings/js/JSHTMLOptionElementConstructor.h: KJS namespace change.
* bindings/js/JSLocation.cpp:
(WebCore::JSLocation::put): Eliminated some DeprecatedString use,
and use protocolIs.
(WebCore::jsLocationProtoFuncReplace): Ditto.
(WebCore::jsLocationProtoFuncReload): Ditto.
(WebCore::jsLocationProtoFuncAssign): Ditto.
* bindings/js/JSLocation.h: KJS namespace change.
* bindings/js/JSNamedNodeMapCustom.cpp:
(WebCore::JSNamedNodeMap::canGetItemsForName): Ditto.
(WebCore::JSNamedNodeMap::nameGetter): Ditto.
* bindings/js/JSNamedNodesCollection.cpp:
(WebCore::JSNamedNodesCollection::JSNamedNodesCollection): Ditto.
* bindings/js/JSNamedNodesCollection.h: Ditto.
* bindings/js/JSXMLHttpRequest.cpp:
(WebCore::jsXMLHttpRequestPrototypeFunctionOpen): Removed
DeprecatedString use.
(WebCore::jsXMLHttpRequestPrototypeFunctionSend): Ditto.
* bindings/js/JSXMLHttpRequest.h: Moved this class into the WebCore
namespace.
* bindings/js/JSXSLTProcessor.cpp: Namespace change.
* bindings/js/JSXSLTProcessor.h: Ditto.
* bindings/js/kjs_binding.cpp: Updated for namespace change.
(WebCore::jsStringOrNull): Added an overload for KURL to allow DOM
classes to return KURL even if the DOM expects a string.
(WebCore::jsStringOrUndefined): Ditto.
(WebCore::jsStringOrFalse): Ditto.
* bindings/js/kjs_binding.h: Moved everything into the WebCore
namespace.
* bindings/js/kjs_css.h: Namespace change.
* bindings/js/kjs_events.cpp: Removed an include.
* bindings/js/kjs_events.h: Namespace change.
* bindings/js/kjs_html.h: Namespace change.
* bindings/js/kjs_navigator.cpp: Moved everything into the
WebCore namespace.
* bindings/js/kjs_navigator.h: Ditto.
* bindings/js/kjs_window.cpp:
(KJS::parseModalDialogFeatures): Updated for String::split change.
(KJS::createWindow): Use protocolIs and removed some DeprecatedString.
(KJS::Window::put): Ditto.
(KJS::Window::allowsAccessFrom): Ditto.
(KJS::windowProtoFuncOpen): Ditto.
* bindings/objc/DOM.mm:
(-[DOMElement _getURLAttribute:]): Removed getNSURL call.
* bindings/objc/DOMHTML.mm:
(-[DOMHTMLDocument _createDocumentFragmentWithMarkupString:baseURLString:]): Ditto.
* bindings/scripts/CodeGeneratorCOM.pm: Updated includes so conversions from
KURL will work.
* bindings/scripts/CodeGeneratorJS.pm: Updated for namespace changes, and also
updated includes so conversions from KURL will work.
* bindings/scripts/CodeGeneratorObjC.pm: Updated includes so conversions from
KURL will work.
* bridge/mac/WebCoreAXObject.mm:
(-[WebCoreAXObject accessibilityAttributeValue:]): Removed getNSURL call.
Also streamlined the logic.
(AXAttributedStringAppendText): Ditto.
* bridge/mac/WebCoreScriptDebugger.mm:
(toNSString): Tweaked.
(toNSURL): Removed getNSURL call.
* css/CSSImageValue.cpp:
(WebCore::CSSImageValue::image): Removed DeprecatedString use.
* css/CSSImportRule.cpp:
(WebCore::CSSImportRule::insertedIntoParent): Ditto.
* css/CSSParser.cpp:
(WebCore::CSSParser::parseValue): Ditto.
(WebCore::CSSParser::parseContent): Ditto.
(WebCore::CSSParser::parseBackgroundImage): Ditto.
(WebCore::CSSParser::parseFontFaceSrc): Ditto.
(WebCore::CSSParser::parseBorderImage): Ditto.
* css/CSSStyleSelector.cpp:
(WebCore::CSSStyleSelector::setEncodedURL): Ditto.
(WebCore::checkPseudoState): Ditto.
* css/CSSStyleSelector.h: Ditto.
* css/MediaList.cpp:
(WebCore::MediaList::setMediaText): Updated for String::split change.
* css/StyleBase.cpp:
(WebCore::StyleBase::baseURL): Return KURL.
* css/StyleBase.h: DItto.
* dom/Document.cpp:
(WebCore::Document::~Document): Updated for namespace change.
(WebCore::Document::documentURI): Return KURL.
(WebCore::Document::setDocumentURI): Removed DeprecatedString use.
(WebCore::Document::baseURI): Return KURL.
(WebCore::Document::open): Updated to use blankURL.
(WebCore::Document::setURL): Take KURL.
(WebCore::Document::shouldBeAllowedToLoadLocalResources): Updated for
change to use KURL
(WebCore::Document::setBaseURL): Take KURL.
(WebCore::Document::elementSheet): Updated for KURL change.
(WebCore::Document::mappedElementSheet): Ditto.
(WebCore::Document::processHttpEquiv): Ditto.
(WebCore::Document::recalcStyleSelector): Removed use of
DeprecatedString -- also noticed some dead code here!
(WebCore::Document::setCookie): Ditto.
(WebCore::Document::completeURL): Return KURL.
* dom/Document.h: Use KURL instead of String in a few places.
* dom/DocumentType.cpp:
(WebCore::DocumentType::baseURI): Return KURL.
* dom/DocumentType.h: Ditto.
* dom/Element.cpp:
(WebCore::Element::baseURI): Return KURL.
* dom/Element.h: Ditto.
* dom/Node.cpp:
(WebCore::Node::setDocument): Namespace change.
(WebCore::Node::baseURI): Return KURL.
* dom/Node.h: Ditto.
* dom/ProcessingInstruction.cpp:
(WebCore::ProcessingInstruction::checkStyleSheet): Updated for KURL change.
* dom/StyleElement.cpp:
(WebCore::StyleElement::process): Changed to use Vector<UChar> instead of
String for better performance.
(WebCore::StyleElement::createSheet): Removed use of DeprecateString.
* dom/XMLTokenizer.cpp:
(WebCore::XMLTokenizer::endElementNs): Updated for KURL change.
(WebCore::XMLTokenizer::end): Ditto.
(WebCore::xmlDocPtrForString): Removed use of DeprecateString.
* dom/XMLTokenizer.h: Ditto.
* editing/markup.cpp: Moved appendString to PlatformString.h.
(WebCore::appendQuotedURLAttributeValue): Use protocolIs.
(WebCore::completeURLs): Removed DeprecatedString use.
(WebCore::createFragmentFromMarkup): Use blankURL.
(WebCore::fillContainerFromString): Removed DeprecatedString use.
(WebCore::createFragmentFromText): Ditto.
* history/HistoryItem.cpp:
(WebCore::HistoryItem::url): Removed DeprecatedString use.
(WebCore::HistoryItem::originalURL): Ditto.
* history/HistoryItem.h: Removed include.
* html/HTMLAnchorElement.cpp:
(WebCore::HTMLAnchorElement::defaultEventHandler): Removed use of
DeprecatedString.
(WebCore::HTMLAnchorElement::href): Return KURL.
(WebCore::HTMLAnchorElement::hash): Removed DeprecatedString use.
(WebCore::HTMLAnchorElement::host): Ditto.
(WebCore::HTMLAnchorElement::hostname): Ditto.
(WebCore::HTMLAnchorElement::pathname): Ditto.
(WebCore::HTMLAnchorElement::port): Ditto.
(WebCore::HTMLAnchorElement::protocol): Ditto.
(WebCore::HTMLAnchorElement::search): Ditto.
(WebCore::HTMLAnchorElement::toString): Ditto.
* html/HTMLAnchorElement.h: Ditto.
* html/HTMLAppletElement.cpp:
(WebCore::HTMLAppletElement::createRenderer): Updated for KURL change.
* html/HTMLAreaElement.cpp:
(WebCore::HTMLAreaElement::href): Return KURL.
* html/HTMLAreaElement.h: Ditto.
* html/HTMLBaseElement.cpp:
(WebCore::HTMLBaseElement::removedFromDocument): Updated for KURL change.
(WebCore::HTMLBaseElement::process): Removed DeprecatedString use.
* html/HTMLBodyElement.cpp:
(WebCore::HTMLBodyElement::parseMappedAttribute): Updated for KURL change.
* html/HTMLEmbedElement.cpp:
(WebCore::HTMLEmbedElement::parseMappedAttribute): Removed use of
DeprecatedString.
* html/HTMLEmbedElement.h: Removed DeprecatedString use.
* html/HTMLFormElement.cpp:
(WebCore::HTMLFormElement::formWouldHaveSecureSubmission): Use protocolIs.
(WebCore::encodeCString): Updated for change to String::split.
(WebCore::HTMLFormElement::dataEncoding): Ditto.
(WebCore::HTMLFormElement::formData): Removed DeprecatedString use.
(WebCore::HTMLFormElement::isMailtoForm): Use protocolIs.
(WebCore::HTMLFormElement::submit): Updated for KURL change.
(WebCore::HTMLFormElement::reset): Ditto.
* html/HTMLFrameElementBase.cpp:
(WebCore::HTMLFrameElementBase::isURLAllowed): Updated for KURL change
and use equalIgnoringRef instead of doing a setRef to get the same effect.
(WebCore::HTMLFrameElementBase::openURL): Use blankURL.
(WebCore::HTMLFrameElementBase::location): Return KURL.
(WebCore::HTMLFrameElementBase::src): Return KURL.
* html/HTMLFrameElementBase.h: Ditto.
* html/HTMLImageElement.cpp:
(WebCore::HTMLImageElement::parseMappedAttribute): Updated for KURL change.
(WebCore::HTMLImageElement::longDesc): Return KURL.
(WebCore::HTMLImageElement::lowsrc): Return KURL.
(WebCore::HTMLImageElement::src): Return KURL.
* html/HTMLImageElement.h: Ditto. Also removed imageMap() function.
* html/HTMLInputElement.cpp:
(WebCore::HTMLInputElement::src): Return KURL.
* html/HTMLInputElement.h: Ditto.
* html/HTMLLinkElement.cpp:
(WebCore::HTMLLinkElement::parseMappedAttribute): Updated for KURL change.
(WebCore::HTMLLinkElement::tokenizeRelAttribute): Updated for String::split change.
(WebCore::HTMLLinkElement::href): Return KURL.
* html/HTMLLinkElement.h: Ditto.
* html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::src): Return KURL.
(WebCore::HTMLMediaElement::pickMedia): Updated for KURL change.
* html/HTMLMediaElement.h: Ditto.
* html/HTMLObjectElement.cpp:
(WebCore::HTMLObjectElement::isImageType): Use protocolIs.
(WebCore::HTMLObjectElement::data): Return KURL.
* html/HTMLObjectElement.h: Ditto.
* html/HTMLOptGroupElement.cpp:
(WebCore::HTMLOptGroupElement::groupLabelText): Removed DeprecatedString use.
* html/HTMLParser.cpp:
(WebCore::HTMLParser::reportErrorToConsole): Updated for KURL change.
* html/HTMLScriptElement.cpp:
(WebCore::HTMLScriptElement::insertedIntoDocument): Ditto.
(WebCore::HTMLScriptElement::text): Changed to use Vector<UChar> instead of
String for better performance.
(WebCore::HTMLScriptElement::src): Return KURL.
* html/HTMLScriptElement.h: Ditto.
* html/HTMLSourceElement.cpp:
(WebCore::HTMLSourceElement::src): Return KURL.
* html/HTMLSourceElement.h: Ditto.
* html/HTMLTableElement.cpp:
(WebCore::HTMLTableElement::parseMappedAttribute): Updated for KURL change.
* html/HTMLTablePartElement.cpp:
(WebCore::HTMLTablePartElement::parseMappedAttribute): Updated for KURL change.
* html/HTMLTextAreaElement.cpp:
(WebCore::HTMLTextAreaElement::setValue): Removed DeprecatedString use.
* html/HTMLTokenizer.cpp:
(WebCore::HTMLTokenizer::scriptExecution): Ditto.
(WebCore::HTMLTokenizer::notifyFinished): Use protocolIs.
* html/HTMLVideoElement.cpp:
(WebCore::HTMLVideoElement::poster): Return KURL.
* html/HTMLVideoElement.h: Ditto.
* html/HTMLViewSourceDocument.cpp:
(WebCore::HTMLViewSourceDocument::addText): Updated for String::split change.
* loader/DocLoader.cpp:
(WebCore::DocLoader::checkForReload): Add an explicit check for an empty URL
here to avoid problems using its string as a hash table key later.
(WebCore::DocLoader::requestResource): Removed DeprecatedString use.
* loader/FTPDirectoryDocument.cpp:
(WebCore::FTPDirectoryTokenizer::createTDForFilename): Updated for KURL change.
(WebCore::FTPDirectoryTokenizer::parseAndAppendOneLine): Removed use of
DeprecatedString.
* loader/FrameLoader.cpp:
(WebCore::FrameLoader::requestFrame): Use protocolIs.
(WebCore::FrameLoader::loadSubframe): Use blankURL.
(WebCore::FrameLoader::submitForm): Use protocolIs and removed use of
DeprecatedString.
(WebCore::FrameLoader::iconURL): Return KURL. Use protcolIs.
(WebCore::FrameLoader::didOpenURL): Use protocolIs.
(WebCore::FrameLoader::didExplicitOpen): Use blankURL.
(WebCore::FrameLoader::executeIfJavaScriptURL): Use protocolIs.
Update for name change to decodeURLEscapeSequences.
(WebCore::FrameLoader::receivedFirstData): Updated for KURL changes.
(WebCore::FrameLoader::begin): Removed DeprecatedString use.
Renamed baseurl to baseURL. Updated to use KURL more.
(WebCore::FrameLoader::gotoAnchor): Removed use of encodedHtmlRef
function, which is no different from ref.
(WebCore::FrameLoader::completeURL): Updated for KURL change.
(WebCore::FrameLoader::scheduleLocationChange): Ditto.
(WebCore::FrameLoader::canCachePage): Use protocolIs.
(WebCore::FrameLoader::updatePolicyBaseURL): Update for KURL change.
(WebCore::FrameLoader::setPolicyBaseURL): Take KURL.
(WebCore::FrameLoader::startRedirectionTimer): Removed use of
DeprecatedString.
(WebCore::FrameLoader::load): Use protocolIs.
(WebCore::FrameLoader::shouldHideReferrer): Use protocolIs.
(WebCore::FrameLoader::shouldAllowNavigation): Updated for KURL change.
(WebCore::FrameLoader::commitProvisionalLoad): Use blankURL.
(WebCore::FrameLoader::open): Use protcolIs.
(WebCore::FrameLoader::createHistoryItem): Use blankURL.
(WebCore::FrameLoader::createJavaAppletWidget): Updated for KURL change.
(WebCore::FrameLoader::switchOutLowBandwidthDisplayIfReady): Removed
DeprecatedString use.
* loader/FrameLoader.h: Ditto.
* loader/ImageDocument.cpp:
(WebCore::ImageDocument::createDocumentStructure): Updated for KURL change.
* loader/PluginDocument.cpp:
(WebCore::PluginTokenizer::createDocumentStructure): Ditto.
* loader/icon/IconDatabase.cpp:
(WebCore::IconDatabase::iconForPageURL): Added a check for an empty URL
before trying to use it as a hash table key.
* loader/icon/IconLoader.h: Tweaked includes.
* loader/loader.cpp:
(WebCore::Loader::servePendingRequests): Use protcolIs. Also removed some
code to set up a local variable that is never used (and a DeprecatedString
on to boot!).
* loader/mac/LoaderNSURLExtras.m:
(suggestedFilenameWithMIMEType): Removed unnecessary typecast.
* page/ContextMenuController.cpp: Removed include.
* page/mac/EventHandlerMac.mm:
(WebCore::EventHandler::needsKeyboardEventDisambiguationQuirks):
Use protocolIs.
* page/mac/WebCoreFrameBridge.mm:
(-[WebCoreFrameBridge URLWithAttributeString:]): Removed getNSURL call.
(-[WebCoreFrameBridge baseURL]): Ditto.
* platform/KURL.cpp:
(WebCore::isSchemeFirstChar): Fixed bug in handling of values >= 0x80.
(WebCore::isSchemeChar): Ditto.
(WebCore::isPathSegmentEndChar): Ditto.
(WebCore::hexDigitValue): Changed parameter type to UChar.
(WebCore::copyASCII): Added.
(WebCore::findFirstOf): Added.
(WebCore::KURL::protocolIs): Added.
(WebCore::KURL::KURL): Tightened logic up quite a bit. Changed parameter
types from DeprecatedString to String.
(WebCore::KURL::init): Changed parameter type to String. Preserved the
passed-in string even if the base is invalid. Cleaned up logic to determine
if the originalString should be pased in to the parse function. Simplified
by calling the new parse overload that takes String in many cases.
(WebCore::KURL::lastPathComponent): Return String.
(WebCore::KURL::protocol): Ditto.
(WebCore::KURL::host): Ditto.
(WebCore::KURL::port): Changed logic to use early return for clarity.
(WebCore::KURL::pass): Return String.
(WebCore::KURL::user): Ditto.
(WebCore::KURL::ref): Ditto.
(WebCore::assertProtocolIsGood): Added.
(WebCore::KURL::protocolIs): Added.
(WebCore::KURL::query): Return String.
(WebCore::KURL::path): Ditto.
(WebCore::KURL::setProtocol): Take String.
(WebCore::KURL::setHost): Ditto.
(WebCore::KURL::setPort): Use String.
(WebCore::KURL::setHostAndPort): Take String.
(WebCore::KURL::setUser): Ditto.
(WebCore::KURL::setPass): Ditto.
(WebCore::KURL::setRef): Ditto.
(WebCore::KURL::setQuery): Ditto.
(WebCore::KURL::setPath): Ditto.
(WebCore::KURL::prettyURL): Return String. Use Vector<UChar> to build it.
(WebCore::decodeURLEscapeSequences): Renamed from KURL::decode_string.
Return String. Use Vector<UChar> to build it.
(WebCore::KURL::isLocalFile): Use protocolIs.
(WebCore::KURL::parse): Added an overload that takes a String to replace
the use of DeprecatedString::ascii at various call sites. Updated for
name change (urlString -> m_string).
(WebCore::equalIgnoringRef): Wrote a new implementation that doesn't
do any allocation.
(WebCore::encodeWithURLEscapeSequences): Renamed from KURL::encode_string.
Return String.
(WebCore::appendEncodedHostname): Added. Replaces encodeHostname and
avoids the need to allocate a string.
(WebCore::findHostnamesInMailToURL): Update to use findFirstOf instead of
regular expressions.
(WebCore::findHostnameInHierarchicalURL): Ditto.
(WebCore::encodeHostnames): Use protocolIs and the other helpers above.
(WebCore::encodeRelativeString): Changed to put result into a CharBuffer.
(WebCore::substituteBackslashes): Updated to use String.
(WebCore::KURL::copyToBuffer): Added.
(WebCore::protocolIs): Added.
(WebCore::blankURL): Added.
(WebCore::KURL::print): Updated.
* platform/KURL.h: Added a number of comments. Reorganized the header a bit.
Made the string constructors explicit. Changed to use String instead of
DeprecatedString. Removed encodedHTMLRef. Renamed and added a few functions.
* platform/cf/KURLCFNet.cpp:
(WebCore::KURL::KURL): Streamlined the logic a bit.
(WebCore::KURL::createCFURL): Changed to use copyToBuffer.
* platform/graphics/mac/MediaPlayerPrivateQTKit.mm:
(WebCore::MediaPlayerPrivate::createQTMovie): Removed getNSURL call.
Use protocolIs.
* platform/mac/ClipboardMac.mm:
(WebCore::ClipboardMac::declareAndWriteDragImage): Removed getNSURL call.
* platform/mac/CookieJar.mm:
(WebCore::cookies): Removed getNSURL call.
(WebCore::setCookies): Removed getNSURL call.
* platform/mac/KURLMac.mm:
(WebCore::KURL::KURL): Streamlined the logic a bit.
* platform/mac/PasteboardMac.mm:
(WebCore::Pasteboard::writeURL): Removed getNSURL call.
(WebCore::Pasteboard::writeImage): Ditto.
* platform/mac/SSLKeyGeneratorMac.mm:
(WebCore::signedPublicKeyAndChallengeString): Ditto.
* platform/network/HTTPParsers.cpp:
(WebCore::filenameFromHTTPContentDisposition): Updated for String::split.
* platform/network/ResourceHandle.cpp:
(WebCore::ResourceHandle::portAllowed): Use protocolIs.
* platform/network/cf/ResourceErrorCF.cpp:
(WebCore::ResourceError::operator CFErrorRef): Removed deprecatedString call.
* platform/network/curl/ResourceHandleManager.cpp:
(WebCore::headerCallback): Ditto.
(WebCore::parseDataUrl): Use String instead of DeprecatedString.
(WebCore::ResourceHandleManager::startJob): Updated for KURL changes.
* platform/network/mac/ResourceErrorMac.mm:
(WebCore::ResourceError::operator NSError*): Removed getNSURL call.
* platform/network/mac/ResourceRequest.h:
(WebCore::ResourceRequest::ResourceRequest): Removed DeprecatedString use.
* platform/network/mac/ResourceRequestMac.mm:
(WebCore::ResourceRequest::doUpdatePlatformRequest): Removed getNSURL call.
* platform/network/mac/ResourceResponseMac.mm:
(WebCore::ResourceResponse::nsURLResponse): Removed getNSURL call.
* platform/qt/ClipboardQt.cpp:
(WebCore::ClipboardQt::writeURL): Removed deprecatedString call.
* platform/text/CString.h:
(WebCore::CStringBuffer::length): Fixed size_t/unsigned mismatch to make it
possible to compile this on Windows with higher warning level
* platform/text/PlatformString.h: Updated split to modify a result parameter
rather than returning a Vector. Added charactersAreAllASCII and an append
function that appends a String to a Vector<UChar>.
* platform/text/String.cpp:
(WebCore::String::split): Updated.
* platform/win/BString.cpp:
(WebCore::BString::BString): Added conversion from KURL.
* platform/win/BString.h: Ditto.
* platform/win/ClipboardUtilitiesWin.cpp:
(WebCore::markupToCF_HTML): Removed use of deprecatedString.
* platform/win/ClipboardWin.cpp:
(WebCore::filesystemPathFromUrlOrTitle): Ditto.
(WebCore::createGlobalHDropContent): Ditto.
(WebCore::ClipboardWin::setData): Ditto.
(WebCore::ClipboardWin::writeRange): Ditto.
* platform/win/PasteboardWin.cpp:
(WebCore::Pasteboard::writeSelection): Ditto.
* plugins/PluginStream.cpp:
(WebCore::PluginStream::startStream): Ditto.
(WebCore::PluginStream::destroyStream): Ditto.
* plugins/win/PluginViewWin.cpp:
(WebCore::scriptStringIfJavaScriptURL): Ditto.
(WebCore::PluginView::performRequest): Ditto.
(WebCore::PluginView::PluginView): Ditto.
* rendering/HitTestResult.cpp:
(WebCore::HitTestResult::absoluteImageURL): Removed DeprecatedString use.
(WebCore::HitTestResult::absoluteLinkURL): Ditto.
* rendering/RenderFrameSet.cpp:
(WebCore::RenderFrameSet::layOutAxis): Fixed comment wording.
* rendering/RenderImage.cpp:
(WebCore::RenderImage::paintReplaced): Removed use of DeperecatedString,
(WebCore::RenderImage::imageMap): Changed to call useMap instead of imageMap;
both do the same thing, and the first is standard DOM.
* rendering/RenderObject.cpp:
(WebCore::RenderObject::addPDFURLRect): Rewrote and streamlined to remove
DeprecatedString use.
* rendering/RenderObject.h: Changed addPDFURLRect to take const IntRect&.
* rendering/RenderPartObject.cpp:
(WebCore::isURLAllowed): Updated for KURL change and use equalIgnoringRef
instead of doing a setRef to get the same effect.
(WebCore::RenderPartObject::updateWidget): Updated for KURL change.
* rendering/RenderText.cpp:
(WebCore::charactersAreAllASCII): Moved the guts to PlatformString.h.
* rendering/SVGRenderSupport.cpp:
(WebCore::renderSubtreeToImage): Added missing call to do layout. I ran
into this while doing some layout tests while URL processing was broken.
* rendering/SVGRenderTreeAsText.h: Removed include.
* svg/SVGImageLoader.cpp:
(WebCore::SVGImageLoader::updateFromElement): Removed DeprecatedString use.
* svg/graphics/SVGImage.cpp:
(WebCore::SVGImage::draw): Added missing call to do layout. I ran
into this while doing some layout tests while URL processing was broken.
(WebCore::SVGImage::dataChanged): Use a null URL rather than an arbitrary
string for the document.
* xml/XMLHttpRequest.cpp:
(WebCore::XMLHttpRequest::getResponseXML): Removed DeprecatedString use.
(WebCore::XMLHttpRequest::urlMatchesDocumentDomain): Ditto.
(WebCore::XMLHttpRequest::open): Ditto.
(WebCore::XMLHttpRequest::send): Namespace change.
(WebCore::XMLHttpRequest::dropProtection): Ditto.
* xml/XMLHttpRequest.h: Removed DeprecatedString use.
* xml/XSLImportRule.cpp:
(WebCore::XSLImportRule::loadSheet): Removed DeprecatedString use.
* xml/XSLStyleSheet.cpp:
(WebCore::XSLStyleSheet::loadChildSheets): Ditto.
(WebCore::XSLStyleSheet::loadChildSheet): Ditto.
* xml/XSLStyleSheet.h: Ditto.
* xml/XSLTProcessor.cpp:
(WebCore::docLoaderFunc): Ditto.
(WebCore::xsltStylesheetPointer): Ditto.
(WebCore::xmlDocPtrFromNode): Ditto.
WebKit/gtk:
* webkit/webkitwebframe.cpp: Removed DeprecatedString use to try to keep it building.
* webkit/webkitwebview.cpp: Ditto.
WebKit/mac:
Reviewed by Eric Seidel.
- updated for WebCore KURL changes
* History/WebHistoryItem.mm:
(-[WebHistoryItem URL]): Removed getNSURL call.
* Misc/WebElementDictionary.mm:
(-[WebElementDictionary _absoluteImageURL]): Ditto.
(-[WebElementDictionary _absoluteLinkURL]): Ditto.
* Misc/WebNSAttributedStringExtras.mm:
(fileWrapperForElement): Ditto.
(+[NSAttributedString _web_attributedStringFromRange:]): Ditto.
* Misc/WebNSURLExtras.mm:
(-[NSString _webkit_stringByReplacingValidPercentEscapes]): Updated
for function name change.
* WebCoreSupport/WebContextMenuClient.mm:
(WebContextMenuClient::downloadURL): Removed getNSURL call.
* WebCoreSupport/WebDragClient.mm:
(WebDragClient::createDragImageForLink): Ditto.
* WebCoreSupport/WebFrameLoaderClient.mm:
(WebFrameLoaderClient::dispatchWillPerformClientRedirect): Ditto.
(WebFrameLoaderClient::startDownload): Ditto.
(WebFrameLoaderClient::updateGlobalHistoryForStandardLoad): Ditto.
(WebFrameLoaderClient::updateGlobalHistoryForReload): Ditto.
(WebFrameLoaderClient::cancelledError): Ditto.
(WebFrameLoaderClient::blockedError): Ditto.
(WebFrameLoaderClient::cannotShowURLError): Ditto.
(WebFrameLoaderClient::interruptForPolicyChangeError): Ditto.
(WebFrameLoaderClient::cannotShowMIMETypeError): Ditto.
(WebFrameLoaderClient::fileDoesNotExistError): Ditto.
(WebFrameLoaderClient::willUseArchive): Ditto.
(WebFrameLoaderClient::setTitle): Ditto.
(WebFrameLoaderClient::actionDictionary): Ditto.
(WebFrameLoaderClient::createFrame): Ditto.
(WebFrameLoaderClient::objectContentType): Ditto.
(WebFrameLoaderClient::createPlugin): Ditto.
(WebFrameLoaderClient::createJavaAppletWidget): Ditto.
* WebView/WebDataSource.mm:
(-[WebDataSource _URL]): Ditto.
(-[WebDataSource _initWithDocumentLoader:]): Ditto.
(-[WebDataSource unreachableURL]): Ditto.
* WebView/WebHTMLView.mm:
(-[WebHTMLView namesOfPromisedFilesDroppedAtDestination:]): Ditto.
WebKit/win:
- updated for WebCore KURL changes
* MarshallingHelpers.cpp:
(MarshallingHelpers::BSTRToKURL): Removed deprecatedString call.
(MarshallingHelpers::KURLToBSTR): Tweaked for efficiency.
* WebContextMenuClient.cpp:
(WebContextMenuClient::searchWithGoogle): Updated for KURL changes.
* WebDataSource.cpp:
(WebDataSource::unreachableURL): Ditto.
* WebDownload.cpp:
(WebDownload::init): Ditto.
(WebDownload::initWithRequest): Ditto.
* WebFrame.cpp:
(WebFrame::loadData): Ditto.
(WebFrame::loadURLIntoChild): Ditto.
(WebFrame::objectContentType): Ditto.
* WebResource.cpp:
(WebResource::initWithData): Ditto.
* WebURLResponse.cpp:
(WebURLResponse::createInstance): Ditto.
(WebURLResponse::initWithURL): Ditto.
* WebView.cpp:
(WebView::userAgentForURL): Ditto.
(WebView::copyURL): Ditto.
(WebView::notifyPreferencesChanged): Ditto.
WebKit/wx:
* WebView.cpp: Removed use of DeprecatedString to keep this compiling.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30243
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Fri, 15 Feb 2008 04:32:01 +0000 (04:32 +0000)]
- fix Windows build
* DumpRenderTree/win/UIDelegate.cpp:
(UIDelegate::setStatusText): Update parameter types to match declaration.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30242
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
darin@apple.com [Fri, 15 Feb 2008 04:30:11 +0000 (04:30 +0000)]
- update this test further, so that it succeeds
* fast/dom/assign-to-window-status-expected.txt: Updated.
* fast/dom/assign-to-window-status.html: Updated.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30241
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
adachan@apple.com [Fri, 15 Feb 2008 02:17:28 +0000 (02:17 +0000)]
<rdar://problem/
5744728> Fix leaks of RegularExpression objects in Frame.cpp.
Reviewed by Jon and Darin.
* page/Frame.cpp:
(WebCore::createRegExpForLabels):
(WebCore::Frame::searchForLabelsBeforeElement):
(WebCore::Frame::matchLabelsAgainstElement):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30240
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
slewis@apple.com [Fri, 15 Feb 2008 02:03:55 +0000 (02:03 +0000)]
2008-02-14 Stephanie Lewis <slewis@apple.com>
Reviewed by Geoff.
Update order files.
* WebKit.order:
* WebCore.order:
* JavaScriptCore.order:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30239
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
oliver@apple.com [Fri, 15 Feb 2008 02:00:13 +0000 (02:00 +0000)]
<rdar://problem/
5726608> REGRESSION (r29428): Assigning to window.status does not update status bar
Reviewed by Geoff G and Weinig.
Revert the portions of r29428 responsible for breaking the ability to
set window.status
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30238
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
andersca@apple.com [Fri, 15 Feb 2008 01:35:44 +0000 (01:35 +0000)]
Reviewed by Darin.
<rdar://problem/
5721790>
Crash in WebCore::DeprecatedString::operator= + 31 at news.google.com
Use pointers in the cache map tables. Otherwise when we rehash,
we will end up destroying Cache objects that node lists might point to.
* dom/Node.cpp:
(WebCore::NodeListsNodeData::~NodeListsNodeData):
(WebCore::Node::getElementsByName):
(WebCore::Node::getElementsByClassName):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30237
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
alp@webkit.org [Fri, 15 Feb 2008 00:30:24 +0000 (00:30 +0000)]
2008-02-14 Alp Toker <alp@atoker.com>
Reviewed by Darin.
http://bugs.webkit.org/show_bug.cgi?id=17353
XMLTokenizer installs global libxml2 callbacks that can break client applications
Patch by Mark Rowe (with a few changes).
The xmlRegisterInputCallbacks/xmlRegisterOutputCallbacks done at
init are global so we need to make sure these callbacks only get used
by XMLTokenizer and never by libxml2 calls in user applications.
This patch modifies the match and open functions to only apply when we
are certain the caller is XMLTokenizer by checking globalDocLoader and
ensuring we're on the correct thread.
Some possible issues remain. See the bug report for details.
* dom/XMLTokenizer.cpp:
(WebCore::matchFunc):
(WebCore::openFunc):
(WebCore::createStringParser):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30236
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ggaren@apple.com [Thu, 14 Feb 2008 23:29:51 +0000 (23:29 +0000)]
JavaScriptCore:
Reviewed by Sam Weinig.
Fixed <rdar://problem/
5737835> nee http://bugs.webkit.org/show_bug.cgi?id=17329
Crash in JSGlobalObject::popActivation when inserting hyperlink in Wordpress (17329)
Don't reset the "activations" stack in JSGlobalObject::reset, since we
might be executing a script during the call to reset, and the script
needs to safely run to completion.
Instead, initialize the "activations" stack when the global object is
created, and subsequently rely on pushing and popping during normal
execution to maintain the stack's state.
* kjs/JSGlobalObject.cpp:
(KJS::JSGlobalObject::init):
(KJS::JSGlobalObject::reset):
LayoutTests:
Reviewed by Sam Weinig.
Layout test for <rdar://problem/
5737835> nee http://bugs.webkit.org/show_bug.cgi?id=17329
Crash in JSGlobalObject::popActivation when inserting hyperlink in Wordpress (17329)
* fast/dom/javascript-url-crash-function.html: Added.
* fast/dom/javascript-url-crash-function-expected.txt: Added.
* fast/dom/resources/javascript-url-crash-function-iframe.html: Added.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30235
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
timothy@apple.com [Thu, 14 Feb 2008 21:00:00 +0000 (21:00 +0000)]
Reviewed by Darin Adler.
<rdar://problem/
5743768> A deadlock during storage layout tests
Make sure not to hold the m_openDatabaseMapGuard mutex when calling
Database::markAsDeletedAndClose(), since that can cause a deadlock
during the synchronous DatabaseThread call it triggers.
* storage/DatabaseTracker.cpp:
(WebCore::DatabaseTracker::deleteDatabaseFile):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30233
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
zimmermann@webkit.org [Thu, 14 Feb 2008 19:45:18 +0000 (19:45 +0000)]
Reviewed by Oliver.
Add dynamic update tests for SVGEllipseElement.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30232
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
aroben@apple.com [Thu, 14 Feb 2008 16:34:28 +0000 (16:34 +0000)]
Improve the efficiency of SecurityOriginHash
Reviewed by Alexey.
* platform/SecurityOriginHash.h:
(WebCore::SecurityOriginHash::hash): Now takes a const
RefPtr<SecurityOrigin>& to reduce ref-count churn.
(WebCore::SecurityOriginHash::equal): Ditto.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30231
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap@webkit.org [Thu, 14 Feb 2008 15:41:53 +0000 (15:41 +0000)]
* WebChromeClient.cpp: (WebChromeClient::exceededDatabaseQuota): Forgot to re-apply review comments to
the previous check-in, fixing.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30230
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ap@webkit.org [Thu, 14 Feb 2008 15:36:28 +0000 (15:36 +0000)]
Reviewed by Adam Roben.
http://bugs.webkit.org/show_bug.cgi?id=17207
Database example doesn't work (requires not-yet-released Safari)
* mac/WebCoreSupport/WebChromeClient.mm:
(WebChromeClient::exceededDatabaseQuota): Check Safari version, and allow 5 megabytes of storage
if it's too old.
* win/WebChromeClient.cpp:
(WebChromeClient::exceededDatabaseQuota): Check Safari version, and allow 5 megabytes of storage
if it's too old.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30229
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
zecke@webkit.org [Thu, 14 Feb 2008 12:37:44 +0000 (12:37 +0000)]
Correct the e-mail address in the ChangeLog.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30228
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
zecke@webkit.org [Thu, 14 Feb 2008 12:27:16 +0000 (12:27 +0000)]
* Make the handling of the coreFrame consistent. We construct the coreFrame
at the beginning and it will live until the end. Change the g_return_(val_)if
to ASSERTs as having a 0 coreFrame can never happen.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30227
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
zecke@webkit.org [Thu, 14 Feb 2008 12:26:44 +0000 (12:26 +0000)]
* Make sure that the WebCore::Frame is living as long as the WebKitWebFrame
wrapped around it. All current calls to WebCore::Frame from within the
WebKitWebFrame are safe, even in the case where the WebCore::Frame is already
detached from the WebCore::Page.
* For the mainFrame the WebKitWebView is holding the initial reference and
will unref on destruction.
* For sub-frames the FramerLoaderClientGtk.cpp will do the unrefing when the
frame gets detached from the parent.
* Make sure FrameLoader::detachFromParent gets called by calling FrameLoader::detachChildren
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30226
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
zecke@webkit.org [Thu, 14 Feb 2008 12:25:59 +0000 (12:25 +0000)]
WebCore::Frame is RefCounted, do not manually delete it!
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30225
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
zecke@webkit.org [Thu, 14 Feb 2008 12:25:28 +0000 (12:25 +0000)]
* Plug a leak
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30224
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
zecke@webkit.org [Thu, 14 Feb 2008 12:24:58 +0000 (12:24 +0000)]
* Update the Copyright Information
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30223
268f45cc-cd09-0410-ab3c-
d52691b4dbfc