Timeline
Feb 18, 2008:
- 11:17 PM Changeset in webkit [30403] by
-
- 1 edit in trunk/WebCore/ChangeLog
Reviewed by Sam Weinig's white rhino tusk stamp
SQLiteTransaction::stop() should also reset the transaction-in-progress flag in its parent SQLiteDatabase
- platform/sql/SQLiteTransaction.cpp: (WebCore::SQLiteTransaction::stop):
- 11:15 PM Changeset in webkit [30402] by
-
- 2 edits in trunk/WebCore
Reviewed by Sam Weinig's rubberstamp
SQLiteTransaction::stop() should also reset the transaction-in-progress flag in its parent SQLiteDatabase
- platform/sql/SQLiteTransaction.cpp: (WebCore::SQLiteTransaction::stop):
- 11:04 PM Changeset in webkit [30401] by
-
- 3 edits in branches/Safari-3-1-branch/LayoutTests
Merge r30399.
- 11:03 PM Changeset in webkit [30400] by
-
- 2 edits in trunk/WebKit/win
Make Drosera work on Vista.
Runtime type library registration on Vista requires use of two new call:
RegisterTypeLibraryForUser and UnRegisterTypeLibraryForUser, which write to
HKCU. LoadTypeLib[Ex] registers under HKLM, which fails under vista due to UAC.
Reviewed by Adam.
- WebKitDLL.cpp: (DllUnregisterServer): Call UnRegisterTypeLibraryForUser if available. Fix version number. (DllRegisterServer): Call RegisterTypeLibraryForUser if available.
- 10:24 PM Changeset in webkit [30399] by
-
- 3 edits in trunk/LayoutTests
Reviewed by Adam Roben in California.
- remove unnecessary and erroneous layoutTestController calls from a test
- fast/css/font-face-multiple-remote-sources.html:
- platform/mac/fast/css/font-face-multiple-remote-sources-expected.txt:
- 9:42 PM Changeset in webkit [30398] by
-
- 5 edits2 adds in branches/Safari-3-1-branch
Merge r30395.
- 9:41 PM Changeset in webkit [30397] by
-
- 4 edits4 adds in branches/Safari-3-1-branch
Merge r30392.
- 9:40 PM Changeset in webkit [30396] by
-
- 2 edits in branches/Safari-3-1-branch/WebCore
Merge r30389.
- 9:19 PM Changeset in webkit [30395] by
-
- 6 edits2 adds in trunk
WebCore:
Reviewed by Darin
Fix for <rdar://5747529> - ObjC Exception can cause JSLock to never be released
Test: platform/mac/plugins/webScriptObject-exception-deadlock.html
- bindings/objc/WebScriptObject.mm:
(-[WebScriptObject valueForKey:]): The line
resultObj = [super valueForKey:key]; // defaults to throwing an exceptionsays it all - it throws an exception. This method also happens to hold the JSLock. Problematically, when the exeception is thrown and the method exited, the JSLock is never released. Fix that without otherwise changing behavior by holding the JSLock in two individual scopes - Right before the exception and right after.
WebKitTools:
Changes by Geoff Garen, Reviewed by Darin
Fix for <rdar://5747529> - ObjC Exception can cause JSLock to never be released
DRT changes for test: platform/mac/plugins/webScriptObject-exception-deadlock.html
[WebScriptObject valueForKey:] might throw an exception, and previously might have "leaked" the global JSLock
This test calls valueForKey, then runs some arbitrary Javascript on a 2ndary thread. If the lock has leaked,
this series of method calls will deadlock. If things are good, it will complete successfully.
- DumpRenderTree/mac/ObjCController.m: (runJavaScriptThread): (+[ObjCController isSelectorExcludedFromWebScript:]): (+[ObjCController webScriptNameForSelector:]): (-[ObjCController testValueForKey]):
LayoutTests:
Reviewed by Darin
Fix for <rdar://5747529> - ObjC Exception can cause JSLock to never be released
- platform/mac-tiger/Skipped: Removed 2 hanging tests that now don't hang
- platform/mac/plugins/webScriptObject-exception-deadlock-expected.txt: Added.
- platform/mac/plugins/webScriptObject-exception-deadlock.html: Added.
- 8:07 PM Changeset in webkit [30394] by
-
- 4 edits in trunk
Remove FindSafari's Release configuration.
- 6:21 PM Changeset in webkit [30393] by
-
- 20 edits in trunk
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.
- 6:13 PM Changeset in webkit [30392] by
-
- 4 edits4 adds in trunk
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.
- 5:12 PM Changeset in webkit [30391] by
-
- 2 edits in trunk/WebCore
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.
- 4:54 PM Changeset in webkit [30390] by
-
- 2 edits in trunk/WebKit/mac
Reviewed by Sam.
- Misc/WebNSAttributedStringExtras.mm: (+[NSAttributedString _web_attributedStringFromRange:]): Eliminate use of DeprecatedString.
- 4:48 PM Changeset in webkit [30389] by
-
- 2 edits in trunk/WebCore
2008-02-18 Stephanie Lewis <Stephanie Lewis>
Reviewed by Adam.
Remove workaround for <rdar://problem/5695848>.
- platform/network/cf/ResourceResponseCFNet.cpp: (WebCore::ResourceResponse::doUpdateResourceResponse):
- 4:48 PM Changeset in webkit [30388] by
-
- 2 edits in trunk/WebKit/qt
Reviewed by Sam.
- Api/qwebnetworkinterface.cpp: (QWebNetworkRequestPrivate::init): Removed use of DeprecatedString.
- 4:25 PM Changeset in webkit [30387] by
-
- 6 edits in trunk
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):
- 3:04 PM Changeset in webkit [30386] by
-
- 2 edits in trunk/WebCore
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):
- 3:03 PM Changeset in webkit [30385] by
-
- 4 edits in trunk/WebKit/gtk
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:
- 3:03 PM Changeset in webkit [30384] by
-
- 3 edits in trunk/WebKit/gtk
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:
- 1:43 PM Changeset in webkit [30383] by
-
- 2 edits in trunk/WebCore
Reviewed by Sam.
- platform/network/win/ResourceHandleWin.cpp: (WebCore::ResourceHandle::onHandleCreated): Use String instead of DeprecatedString. (WebCore::ResourceHandle::start): Ditto.
- 1:42 PM Changeset in webkit [30382] by
-
- 2 edits in trunk/WebCore
Reviewed by Sam.
- platform/network/win/CookieJarWin.cpp: (WebCore::setCookies): Use String instead of DeprecatedString. (WebCore::cookies): Ditto.
- 1:40 PM Changeset in webkit [30381] by
-
- 2 edits in trunk/WebCore
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.
- 1:28 PM Changeset in webkit [30380] by
-
- 2 edits in trunk/JavaScriptCore
Reviewed by Sam.
- wtf/ASCIICType.h: (WTF::toASCIIHexValue): Added.
- 1:20 PM Changeset in webkit [30379] by
-
- 2 edits in trunk/WebCore
Reviewed by Sam.
- editing/htmlediting.cpp: (WebCore::stringWithRebalancedWhitespace): Changed to use String instead of DeprecatedString.
- 1:16 PM Changeset in webkit [30378] by
-
- 2 edits in trunk/WebCore
Reviewed by Sam.
- editing/SelectionController.cpp: (WebCore::SelectionController::debugRenderer): Changed to use String instead of DeprecatedString.
- 12:13 PM Changeset in webkit [30377] by
-
- 7 edits3 deletes in trunk/WebCore
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.
- 11:55 AM Changeset in webkit [30376] by
-
- 2 edits in branches/Safari-3-1-branch/WebCore
Merge r30370.
- 11:49 AM Changeset in webkit [30375] by
-
- 2 edits in trunk/WebCore
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.
- 11:21 AM Changeset in webkit [30374] by
-
- 2 edits in trunk/WebCore
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):
- 10:59 AM Changeset in webkit [30373] by
-
- 2 edits in trunk/WebCore
Reviewed by Sam.
- WebCore.base.exp: Export a couple of WebCore::String functions we plan to use in the future in WebKit.
- 10:58 AM Changeset in webkit [30372] by
-
- 2 edits in trunk/WebCore
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).
- 10:41 AM Changeset in webkit [30371] by
-
- 2 edits in trunk/WebCore
Suggested by Darin.
- platform/KURL.h: (WebCore::KURL::operator const String&): Added, to avoid unexpected conversion via UString (as in bug 17418).
- 10:23 AM Changeset in webkit [30370] by
-
- 2 edits in trunk/WebCore
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.
- 10:14 AM Changeset in webkit [30369] by
-
- 2 edits in trunk/WebCore
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.
- 7:14 AM QtWebKitTodo edited by
- (diff)
- 6:33 AM QtWebKitTodo edited by
- (diff)
- 4:15 AM QtWebKitTodo edited by
- (diff)
- 1:10 AM QtWebKitTodo edited by
- (diff)
Feb 17, 2008:
- 10:45 PM Changeset in webkit [30368] by
-
- 2 edits in trunk/WebCore
Roll out r30360.
- loader/FrameLoader.cpp: (WebCore::FrameLoader::changeLocation):
- 10:31 PM Changeset in webkit [30367] by
-
- 4 edits in branches/Safari-3-1-branch
Make JavaScriptCore and WebKit's FEATURE_DEFINES match WebCore.
- 10:25 PM Changeset in webkit [30366] by
-
- 2 edits in trunk/WebCore
Mac build fix.
- WebCore.xcodeproj/project.pbxproj:
- 9:47 PM Changeset in webkit [30365] by
-
- 1 edit in branches/Safari-3-1-branch/WebCore/ChangeLog
ChangeLog fix.
- 9:43 PM Changeset in webkit [30364] by
-
- 2 edits in branches/Safari-3-1-branch/WebCore
Build fix.
- WebCore.vcproj/build-generated-files.sh:
- 6:32 PM Changeset in webkit [30363] by
-
- 3 edits in trunk/WebCore
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):
- 6:02 PM Changeset in webkit [30362] by
-
- 4 edits2 copies2 moves2 adds in trunk
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
- 4:22 PM Changeset in webkit [30361] by
-
- 11 edits in trunk
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.
- 4:14 PM Changeset in webkit [30360] by
-
- 2 edits in trunk/WebCore
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):
- 3:53 PM Changeset in webkit [30359] by
-
- 2 edits in trunk/WebCore
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):
- 12:21 PM Changeset in webkit [30358] by
-
- 2 edits in trunk/WebKit/win
http://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.
- 11:28 AM Changeset in webkit [30357] by
-
- 5 edits in trunk/LayoutTests
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:
- 9:59 AM Changeset in webkit [30356] by
-
- 2 edits in trunk/JavaScriptCore
- wtf/ListHashSet.h: (WTF::swap): Removed stray return statement.
- 7:36 AM Changeset in webkit [30355] by
-
- 2 edits in trunk/WebCore
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):
- 7:09 AM Changeset in webkit [30354] by
-
- 2 edits in trunk/WebCore
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):
- 4:12 AM Changeset in webkit [30353] by
-
- 2 edits in trunk/WebKit/gtk
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):
- 2:46 AM Changeset in webkit [30352] by
-
- 15 edits in branches/Safari-3-1-branch
Merge r30331.
- 2:45 AM Changeset in webkit [30351] by
-
- 5 edits in branches/Safari-3-1-branch
Merge r30330.
- 2:44 AM Changeset in webkit [30350] by
-
- 2 edits in branches/Safari-3-1-branch/WebCore
Merge r30329.
- 2:43 AM Changeset in webkit [30349] by
-
- 7 edits in branches/Safari-3-1-branch
Merge r30328.
- 2:42 AM Changeset in webkit [30348] by
-
- 23 edits1 move2 adds in branches/Safari-3-1-branch
Merge r30326.
- 2:40 AM Changeset in webkit [30347] by
-
- 23 edits1 add in branches/Safari-3-1-branch/WebCore
Merge r30325.
- 2:39 AM Changeset in webkit [30346] by
-
- 8 edits1 copy5 adds in branches/Safari-3-1-branch
Merge r30323.
- 1:03 AM WebKit Team edited by
- Edited my entry to humor Sam (diff)
- 12:50 AM Changeset in webkit [30345] by
-
- 3 edits4 adds in trunk
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.