Context Navigation

← Previous Period
Next Period →

Timeline

03/19/07:

23:56 Changeset [23122] by aroben

Reviewed by Ada.

Skipped: Added tests that relay on DRT's built-in plugin, or on the QuickTime WebKit plugin.

22:58 Changeset [20338] by ap

Reviewed by Darin.

http://bugs.webkit.org/show_bug.cgi?id=13107
XPath should raise an exception when trying to parse an unknown function

Test: fast/xpath/invalid-functions.html

xml/XPathFunctions.cpp: (WebCore::XPath::createFunction): Return 0 if the function couldn't be created.
xml/XPathGrammar.y: Handle nulls returned from createFunction().

22:47 Changeset [23121] by aroben

LayoutTestsWin:

Reviewed by Ada.

Skipped: Added some more international text tests.

WebKitToolsWin:

Reviewed by Ada.

Made DRT output UTF8 strings.

DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/DumpRenderTree.cpp: (dump): Convert to UTF8 before printing.

21:01 Changeset [23120] by aroben

Reviewed by Geoff.

Added support for --threaded.

Note that this always crashes right now due to
<rdar://problem/5051141> JavaScriptCore is not thread-safe,
but that's why we need this feature!

DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/DumpRenderTree.cpp: (initialize): Changed NULL to 0. (runTest): Ditto. (pthread_equal): Added. (javaScriptThreads): Ported from Mac. (runJavaScriptThread): Ditto. (startJavaScriptThreads): Ditto. (stopJavaScriptThreads): Ditto. (main): Parse --threaded argument and start/stop threads if needed.
DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/DumpRenderTree.vcproj: Link against pthreadVC2.lib, and added include/pthreads to the include path.

19:45 Changeset [20337] by andersca

Reviewed by Dave Hyatt.

<rdar://problem/5067983>
iSale: Crash occurs at WebFrameLoaderClient::dispatchDecidePolicyForMIMEType() when attempting to load a HTML template

Restore old behavior (broke in r14533) where the resource load and download delegates are retained for as long as the
data source is loading.

WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::createDocumentLoader):
WebView/WebDocumentLoaderMac.h:
WebView/WebDocumentLoaderMac.mm: (WebDocumentLoaderMac::setDataSource): (WebDocumentLoaderMac::decreaseLoadCount):

18:05 Changeset [20336] by darin

kjs/value.h: Roll ~JSValue change out. It was causing problems. I'll do it right later.

17:50 Changeset [20335] by ggaren

Speculative fix for why ASSERT_MAIN_THREAD didn't work for me. (The
documentation says "non-zero," not "1."

Misc/WebKitLogging.m: (WebKitRunningOnMainThread):

17:46 Changeset [20334] by ggaren

Reviewed by John Sullivan.

Fixed <rdar://problem/5073380> REGRESSION: Crash occurs at WTF::fastFree()
when reloading liveconnect page (applet)

Best to use free when you use malloc, especially when malloc and delete
use completely different libraries.

bindings/jni/jni_runtime.cpp: (JavaMethod::~JavaMethod):

17:30 Changeset [20333] by adele

LayoutTests:

Reviewed by Beth.

Test for http://bugs.webkit.org/show_bug.cgi?id=13087
<rdar://problem/5064316> REGRESSION: Allow setting the checked attribute in js and in markup for unnamed radio buttons (dominos.com)

fast/forms/radio_checked_name-expected.txt: Added.
fast/forms/radio_checked_name.html: Added.

WebCore:

Reviewed by Beth.

Fix for http://bugs.webkit.org/show_bug.cgi?id=13087
<rdar://problem/5064316> REGRESSION: Allow setting the checked attribute in js and in markup for unnamed radio buttons (dominos.com)

Test: fast/forms/radio_checked_name.html

We were matching a WinIE quirk that does not allow a user to check and uncheck an unnamed radio button. But they still
allow the checked attribute to be set in html, and changed in javascript. So this change matches that behavior.

html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::parseMappedAttribute): We no longer need a special call to setChecked when parsing the name attribute, since we setChecked will now work with unnamed radio buttons. (WebCore::HTMLInputElement::preDispatchEventHandler): Added comment. (WebCore::HTMLInputElement::setChecked): Don't check for the unnamed radio button case here. (WebCore::HTMLInputElement::postDispatchEventHandler): Don't call setChecked for unnamed radio buttons. (WebCore::HTMLInputElement::defaultEventHandler): ditto.

17:27 Changeset [20332] by aroben

Rubberstamped by Oliver.

bridge/JavaScriptStatistics.cpp: Added missing header for correctness.

17:16 Changeset [20331] by andrew

JavaScriptCore:

Reviewed by Maciej.

Really set Xcode editor to use 4 space indentation (http://webkit.org/coding/coding-style.html)

JavaScriptCore.xcodeproj/project.pbxproj:

JavaScriptGlue:

Reviewed by Maciej.

Really set Xcode editor to use 4 space indentation (http://webkit.org/coding/coding-style.html)

JavaScriptGlue.xcodeproj/project.pbxproj:

WebCore:

Reviewed by Maciej.

Really set Xcode editor to use 4 space indentation (http://webkit.org/coding/coding-style.html)

WebCore.xcodeproj/project.pbxproj:

WebKit:

Reviewed by Maciej.

Really set Xcode editor to use 4 space indentation (http://webkit.org/coding/coding-style.html)

WebKit.xcodeproj/project.pbxproj:

17:13 Changeset [23119] by aroben

Build fix.

WebKit.vcproj/WebKitGUID.vcproj:

16:50 Changeset [20330] by darin

Reviewed by Geoff.

Changed list size threshold to 5 based on testing.

I was testing the i-Bench JavaScript with the list statistics
dumping on, and discovered that there were many 5-element lists.
The fast case for lists was for 4 elements and fewer. By changing
the threshold to 5 elements we get a measurable speedup. I believe
this will help real web pages too, not just the benchmark.

kjs/list.cpp: Change constant from 4 to 5.

16:36 Changeset [23118] by aroben

WebCoreWin:

Reviewed by Anders and Darin.

Added JavaScriptStatistics to the project for
<rdar://problems/5071653>.

WebCore.vcproj/WebCore.vcproj: Added include/pthreads to the include path, and added JavaScriptStatistics.(cpp|h).

WebKitToolsWin:

Reviewed by Anders and Darin.

Fix <rdar://problem/5071653> DumpRenderTree needs to be able to
trigger GC on a separate thread

DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/DumpRenderTree.vcproj: Added GCController.(cpp|h) to the project.
DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/GCController.cpp: Added. (collectCallback): (collectOnAlternateThreadCallback): (getJSObjectCountCallback): (getClass): (makeGCController):
DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/GCController.h: Added.
DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/WaitUntilDoneDelegate.cpp: (WaitUntilDoneDelegate::windowScriptObjectAvailable): Add the GCController to the window object.

WebKitWin:

Reviewed by Anders and Darin.

Added WebJavaScriptCollector for <rdar://problems/5071653>.

Interfaces/IWebJavaScriptCollector.idl: Added.
Interfaces/WebKit.idl:
WebJavaScriptCollector.cpp: Added. (WebJavaScriptCollector::WebJavaScriptCollector): (WebJavaScriptCollector::~WebJavaScriptCollector): (WebJavaScriptCollector::createInstance): (WebJavaScriptCollector::QueryInterface): (WebJavaScriptCollector::AddRef): (WebJavaScriptCollector::Release): (WebJavaScriptCollector::collect): (WebJavaScriptCollector::collectOnAlternateThread): (WebJavaScriptCollector::objectCount):
WebJavaScriptCollector.h: Added.
WebKit.vcproj/Interfaces.vcproj: Added IWebJavaScriptCollector.idl.
WebKit.vcproj/WebKit.vcproj: Added WebJavaScriptCollector.(cpp|h).
WebKit.vcproj/WebKitGUID.vcproj: Added IWebJavaScriptCollector_i.c.
WebKitClassFactory.cpp: (WebKitClassFactory::CreateInstance): Added cases for WebJavaScriptCollector.
WebKitDLL.cpp: (DllGetClassObject): Ditto.

16:28 Changeset [20329] by darin

kjs/value.h: Oops, fix build.

16:12 Changeset [20328] by darin

Reviewed by Geoff.

remove ~JSValue; tiny low-risk performance boost

kjs/value.h: Remove unneeded empty virtual destructor from JSValue. The only class derived from JSValue is JSCell and it already has a virtual destructor. Declaring an empty constructor in JSValue had one good effect: it marked the destructor private, making it a compile time error to try to destroy a JSValue; but that's not a likely mistake for someone to make. It had two bad effects: (1) it caused gcc, at least, to generate code to fix up the virtual table pointer to point to the JSValue version of the virtual table inside the destructor of all classes derived from JSValue directly or indirectly; (2) it caused JSValue to be a polymorphic class so required a virtual table for it. It's cleaner to not have either of those.

15:54 Changeset [20327] by darin

English.lproj/StringsNotToBeLocalized.txt: Update for recent changes.

15:39 Changeset [20326] by justing

LayoutTests:

Reviewed by darin

<rdar://problem/5071074>
REGRESSION: Links that aren't fully selected copy and paste as only text

editing/pasteboard/5071074-expected.checksum: Added.
editing/pasteboard/5071074-expected.png: Added.
editing/pasteboard/5071074-expected.txt: Added.
editing/pasteboard/5071074.html: Added.

WebCore:

Reviewed by darin

<rdar://problem/5071074>
REGRESSION: Links that aren't fully selected copy and paste as only text

We stopped adding markup for all commonAncestorBlocks
and the ancestors of lastClosed up to them because it
was adding a lot of unnecessary markup.

editing/markup.cpp: (WebCore::createMarkup): Also include markup for nodes up to an enclosing anchor. Cleaned up the code for including special common ancestors a bit.

14:39 Changeset [23117] by aroben

Release build fix.

DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/DumpRenderTree.vcproj: Set the include path for release to the same as for debug.

13:22 Changeset [23116] by aroben

Reviewed by Adele.

Fix eventSender.keyDown() to create WM_CHAR messages as well, since these
are needed by WebCore.

DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/EventSender.cpp: (dispatchMessage): Added. (mouseDownCallback): Call dispatchMessage instead of DispatchMessage. (doMouseUp): Ditto. (mouseClickCallback): Ditto. (doMouseMove): Ditto. (keyDownCallback): Ditto.

12:41 Changeset [20325] by zimmermann

Reviewed by Lars Knoll.

Fix crash when opening a SVG document which contains SVG elements, but no <svg> root element.
Moving the mouse was crashing WebKit. Fixes svg/hixie/dynamic/006.xml.

12:31 Changeset [20324] by adele

Reviewed by Hyatt.

fix http://bugs.webkit.org/show_bug.cgi?id=13117 REGRESSION (r14658): Flickering text fields when updating via javascript

page/Frame.cpp: (WebCore::Frame::forceLayout): Added a flag to allow partial relayout.
page/Frame.h:
page/mac/WebCoreFrameBridge.mm: (-[WebCoreFrameBridge forceLayoutAdjustingViewSize:]): Allow partial relayout if not adjusting view size. (-[WebCoreFrameBridge needsLayout]): Changed to check for pending subtree layout instead of just whether the root needs layout.

12:28 Changeset [20323] by zimmermann

Reviewed by Maciej.

Make JS function calls on POD-types (ie. SVGLength) take effect.
Calling ie. newValueInSpecifiedUnits on SVGLength works as expected.

This worked fine before we switched to SVGLength POD type, so this
can be considered as P1 regression fix.

Partly fix svg/hixie/dynamic/005-broken.svg. We match Opera now though
it's not yet completly fixed. Setting baseVal attribute on SVGAnimatedString
doesn't work as expected - needs a seperated fix.

12:07 Changeset [23115] by aroben

Reviewed by Maciej.

Skip tests with international text due to <rdar://problems/4516194>.

Skipped:

11:21 Changeset [20322] by adele

Reviewed by Adam.

Adding comments and a little cleanup from my last checkin.

html/HTMLSelectElement.cpp: (WebCore::HTMLSelectElement::setSelectedIndex): Removed commented out assert. Added comment about how we use onChange. (WebCore::HTMLSelectElement::selectAll): Added comment about how we use saveLastSelection and onChange. (WebCore::HTMLSelectElement::dispatchFocusEvent): ditto. (WebCore::HTMLSelectElement::dispatchBlurEvent): ditto. (WebCore::HTMLSelectElement::menuListDefaultEventHandler): ditto. (WebCore::HTMLSelectElement::listBoxDefaultEventHandler): ditto. (WebCore::HTMLSelectElement::menuListOnChange): Added assert that usesMenuList() is true. (WebCore::HTMLSelectElement::listBoxOnChange): Added assert that usesMenuList() is false. (WebCore::HTMLSelectElement::saveLastSelection): Added early return for menu lists.

11:14 JavaScript and DOM Benchmarks edited by timothy@apple.com

(diff)

09:57 Changeset [20321] by zimmermann

Reviewed by Maciej.

Import Hixie's SVG testsuite. It highlights one bug, one regression in JS and one crash.
The crashing file is marked with "-disabled", the broken ones with "-broken".

Landing fixes for the certain bugs/regressions seperated.

06:17 Changeset [20320] by sullivan

Reviewed by Justin

fixed <rdar://problem/5071238> REGRESSION: opt-cmd-B to show Bookmarks view does nothing when form field has focus

WebView/WebHTMLView.mm: (-[WebHTMLView _handleStyleKeyEquivalent:]): we were counting any set of modifiers plus 'b' as the standard key equivalent for toggling Bold; now we only accept command+'b'

05:46 Changeset [20319] by bdash

2007-03-19 Mitz Pettel <mitz@webkit.org>

Reviewed by Tim Hatcher.

fix http://bugs.webkit.org/show_bug.cgi?id=13091 REGRESSION (r20075): Wrong text style and pixel break in Apple Store Locator HUD

Test: fast/innerHTML/additional-inline-style.html

editing/markup.cpp: (WebCore::startMarkup): Changed to add inline style based on CSS rules only in AnnotateForInterchange mode.

2007-03-19 Mitz Pettel <mitz@webkit.org>

Reviewed by Tim Hatcher.

test and updated result for http://bugs.webkit.org/show_bug.cgi?id=13091 REGRESSION (r20075): Wrong text style and pixel break in Apple Store Locator HUD

editing/pasteboard/paste-table-002-expected.txt:
fast/innerHTML/additional-inline-style-expected.txt: Added.
fast/innerHTML/additional-inline-style.html: Added.
webarchive/test-duplicate-resources-expected.txt:
webarchive/test-frameset-expected.txt:
webarchive/test-link-href-expected.txt:

02:32 Changeset [23114] by aroben

Reviewed by Hyatt.

Implemented some more layoutTestController methods:

display
testRepaint
repaintSweepHorizontally

The last two don't have any effect yet because we don't support pixel
tests yet (<rdar://problem/5071708>).

This gets all but one test in fast/repaint passing, and fixes various
other tests that use these methods as well.

DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/DumpRenderTree.cpp: (runTest): Initialize new globals.
DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/DumpRenderTree.h: Added extern declarations.
DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/LayoutTestController.cpp: (displayCallback): Implemented. (testRepaintCallback): Implemented. (repaintSweepHorizontallyCallback): Implemented.

01:19 Changeset [20318] by zack

compile fix for qt

01:00 Changeset [20317] by aroben

WebCore:

Reviewed by Hyatt and Maciej.

Renamed WebCoreJavaScript to JavaScriptStatistics and made it C++ as
part of <rdar://problem/5071653>.

All layout tests pass.

WebCore.exp: Updated.
WebCore.xcodeproj/project.pbxproj: Updated.
bridge/JavaScriptStatistics.cpp: Moved from bridge/mac/WebCoreJavaScript.mm, and converted to C++. (WebCore::collect): (WebCore::JavaScriptStatistics::objectCount): (WebCore::JavaScriptStatistics::interpreterCount): (WebCore::JavaScriptStatistics::protectedObjectCount): (WebCore::JavaScriptStatistics::rootObjectTypeCounts): (WebCore::JavaScriptStatistics::garbageCollect): (WebCore::JavaScriptStatistics::garbageCollectOnAlternateThread): (WebCore::JavaScriptStatistics::shouldPrintExceptions): (WebCore::JavaScriptStatistics::setShouldPrintExceptions):
bridge/JavaScriptStatistics.h: Added.
bridge/mac/WebCoreJavaScript.h: Removed.
bridge/mac/WebCoreJavaScript.mm: Removed.

WebKit:

Reviewed by Hyatt and Maciej.

Updated WebCoreStatistics for the conversion of WebCoreJavaScript to
C++.

Misc/WebCoreStatistics.mm: (+[WebCoreStatistics javaScriptObjectsCount]): (+[WebCoreStatistics javaScriptInterpretersCount]): (+[WebCoreStatistics javaScriptProtectedObjectsCount]): (+[WebCoreStatistics javaScriptRootObjectTypeCounts]): Moved conversion to NSCountedSet here from WebCore. (+[WebCoreStatistics garbageCollectJavaScriptObjects]): (+[WebCoreStatistics garbageCollectJavaScriptObjectsOnAlternateThread:]): (+[WebCoreStatistics shouldPrintExceptions]): (+[WebCoreStatistics setShouldPrintExceptions:]): (+[WebCoreStatistics javaScriptReferencedObjectsCount]): (+[WebCoreStatistics javaScriptRootObjectClasses]):

00:49 Changeset [23113] by aroben

Build fix.

Interfaces/IWebView.idl:

00:40 Changeset [20316] by hyatt

Update SVG tests now that painting happens after onload.

00:36 Changeset [20315] by hyatt

Update tests for minimum font size change.

00:35 Changeset [20314] by hyatt

Update tests for minimum font size change.

00:34 Changeset [20313] by hyatt

Update tests for minimum font size change.

00:34 Changeset [20312] by hyatt

Update tests for minimum font size change.

00:32 Changeset [20311] by hyatt

Update the minimum font size pref to match the actual default setting in Safari. It should have been 1
all this time and not 9.

Reviewed by aroben

DumpRenderTree/DumpRenderTree.m: (dumpRenderTree):

03/18/07:

23:52 Changeset [23112] by aroben

WebKitToolsWin:

Reviewed by Hyatt.

Fix DRT part of <rdar://problem/5066482> DumpRenderTree on Windows
doesn't dump editing callbacks

However, we still don't dump anything because of
<rdar://problem/5071598>.

DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/DumpRenderTree.cpp: (runTest): Initialize shouldDumpEditingCallbacks. (main): Set the WebView's editing delegate.
DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/DumpRenderTree.h:
DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/DumpRenderTree.vcproj: Added EditingDelegate.(cpp|h) to the project.
DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/EditingDelegate.cpp: Added. (EditingDelegate::EditingDelegate): (EditingDelegate::QueryInterface): (EditingDelegate::AddRef): (EditingDelegate::Release): (dumpPath): (dump): (EditingDelegate::shouldBeginEditingInDOMRange): (EditingDelegate::shouldEndEditingInDOMRange): (EditingDelegate::shouldInsertNode): (EditingDelegate::shouldInsertText): (EditingDelegate::shouldDeleteDOMRange): (EditingDelegate::shouldChangeSelectedDOMRange): (EditingDelegate::shouldApplyStyle): (EditingDelegate::shouldChangeTypingStyle): (EditingDelegate::doPlatformCommand): (EditingDelegate::webViewDidBeginEditing): (EditingDelegate::webViewDidChange): (EditingDelegate::webViewDidEndEditing): (EditingDelegate::webViewDidChangeTypingStyle): (EditingDelegate::webViewDidChangeSelection):
DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/EditingDelegate.h: Added. (EditingDelegate::setAcceptsEditing): (EditingDelegate::undoManagerForWebView):
DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/LayoutTestController.cpp: (dumpEditingCallbacksCallback): Ported from Mac. (setAcceptsEditingCallback): Ditto.

WebKitWin:

Reviewed by Hyatt.

Implemented getting/setting of the editing delegate (we don't actually
use it for anything yet, though. See <rdar://problem/5071598>).

Interfaces/IWebEditingDelegate.idl: Renamed IWebViewEditingDelegate to IWebEditingDelegate to match our other delegates.
Interfaces/IWebView.idl: Updated for rename.
WebView.cpp: (WebView::setEditingDelegate): Implemented. (WebView::editingDelegate): Ditto.
WebView.h: Updated for rename.

22:43 Changeset [20310] by mjs

JavaScriptCore:

Reviewed by Mark.

avoid static construction (and global variable access) in a smarter, more portable way, to later enable MUTLI_THREAD mode to work on other platforms and compilers.

kjs/CommonIdentifiers.cpp: Added. New class to hold all the shared identifiers. (KJS::CommonIdentifiers::CommonIdentifiers): (KJS::CommonIdentifiers::shared):
kjs/CommonIdentifiers.h: Added.

kjs/ExecState.h: (KJS::ExecState::propertyNames): Hand the CommonIdentifiers instance here for easy access. (KJS::ExecState::ExecState):

API/JSObjectRef.cpp: (JSObjectMakeConstructor):
CMakeLists.txt:
JavaScriptCore.exp:
JavaScriptCore.pri:
JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:
JavaScriptCore.xcodeproj/project.pbxproj:
JavaScriptCoreSources.bkl:
bindings/runtime_array.cpp: (RuntimeArray::getOwnPropertySlot): (RuntimeArray::put):
bindings/runtime_method.cpp: (RuntimeMethod::getOwnPropertySlot):
kjs/array_object.cpp: (ArrayInstance::getOwnPropertySlot): (ArrayInstance::put): (ArrayInstance::deleteProperty): (ArrayProtoFunc::ArrayProtoFunc): (ArrayProtoFunc::callAsFunction): (ArrayObjectImp::ArrayObjectImp):
kjs/bool_object.cpp: (BooleanPrototype::BooleanPrototype): (BooleanProtoFunc::BooleanProtoFunc): (BooleanProtoFunc::callAsFunction): (BooleanObjectImp::BooleanObjectImp):
kjs/completion.h: (KJS::Completion::Completion):
kjs/date_object.cpp: (KJS::DateProtoFunc::DateProtoFunc): (KJS::DateObjectImp::DateObjectImp): (KJS::DateObjectFuncImp::DateObjectFuncImp):
kjs/error_object.cpp: (ErrorPrototype::ErrorPrototype): (ErrorProtoFunc::ErrorProtoFunc): (ErrorProtoFunc::callAsFunction): (ErrorObjectImp::ErrorObjectImp): (ErrorObjectImp::construct): (NativeErrorPrototype::NativeErrorPrototype): (NativeErrorImp::NativeErrorImp): (NativeErrorImp::construct): (NativeErrorImp::callAsFunction):
kjs/function.cpp: (KJS::FunctionImp::getOwnPropertySlot): (KJS::FunctionImp::put): (KJS::FunctionImp::deleteProperty): (KJS::FunctionImp::getParameterName): (KJS::DeclaredFunctionImp::construct): (KJS::IndexToNameMap::unMap): (KJS::Arguments::Arguments): (KJS::ActivationImp::getOwnPropertySlot): (KJS::ActivationImp::deleteProperty): (KJS::GlobalFuncImp::GlobalFuncImp):
kjs/function_object.cpp: (FunctionPrototype::FunctionPrototype): (FunctionProtoFunc::FunctionProtoFunc): (FunctionProtoFunc::callAsFunction): (FunctionObjectImp::FunctionObjectImp): (FunctionObjectImp::construct):
kjs/grammar.y:
kjs/identifier.cpp:
kjs/identifier.h:
kjs/interpreter.cpp: (KJS::Interpreter::init): (KJS::Interpreter::initGlobalObject):
kjs/interpreter.h:
kjs/lookup.h:
kjs/math_object.cpp: (MathFuncImp::MathFuncImp):
kjs/nodes.cpp: (ArrayNode::evaluate): (FuncDeclNode::processFuncDecl): (FuncExprNode::evaluate):
kjs/number_object.cpp: (NumberPrototype::NumberPrototype): (NumberProtoFunc::NumberProtoFunc): (NumberObjectImp::NumberObjectImp):
kjs/object.cpp: (KJS::JSObject::put): (KJS::JSObject::defaultValue): (KJS::JSObject::hasInstance):
kjs/object.h: (KJS::JSObject::getOwnPropertySlot):
kjs/object_object.cpp: (ObjectPrototype::ObjectPrototype): (ObjectProtoFunc::ObjectProtoFunc): (ObjectObjectImp::ObjectObjectImp):
kjs/regexp_object.cpp: (RegExpPrototype::RegExpPrototype): (RegExpProtoFunc::RegExpProtoFunc): (RegExpObjectImp::RegExpObjectImp):
kjs/string_object.cpp: (KJS::StringInstance::getOwnPropertySlot): (KJS::StringInstance::put): (KJS::StringInstance::deleteProperty): (KJS::StringPrototype::StringPrototype): (KJS::StringProtoFunc::StringProtoFunc): (KJS::StringProtoFunc::callAsFunction): (KJS::StringObjectImp::StringObjectImp): (KJS::StringObjectFuncImp::StringObjectFuncImp):
kjs/testkjs.cpp: (TestFunctionImp::TestFunctionImp):

WebCore:

Reviewed by Mark.

avoid static construction (and global variable access) in a smarter, more portable way, to later enable MUTLI_THREAD mode to work on other platforms and compilers.

bindings/js/JSCustomXPathNSResolver.cpp: (WebCore::JSCustomXPathNSResolver::lookupNamespaceURI):
bindings/js/JSHTMLInputElementBase.cpp: (WebCore::JSHTMLInputElementBaseFunction::JSHTMLInputElementBaseFunction):
bindings/js/JSHTMLOptionElementConstructor.cpp: (WebCore::JSHTMLOptionElementConstructor::JSHTMLOptionElementConstructor):
bindings/js/JSXMLHttpRequest.cpp: (KJS::JSXMLHttpRequestConstructorImp::JSXMLHttpRequestConstructorImp):
bindings/js/JSXSLTProcessor.cpp: (KJS::XSLTProcessorConstructorImp::XSLTProcessorConstructorImp):
bindings/js/kjs_dom.cpp: (KJS::DOMNamedNodeMap::getOwnPropertySlot): (KJS::DOMNamedNodesCollection::getOwnPropertySlot):
bindings/js/kjs_events.cpp: (KJS::JSAbstractEventListener::handleEvent):
bindings/js/kjs_html.cpp: (KJS::HTMLElementFunction::HTMLElementFunction): (KJS::JSHTMLCollection::getOwnPropertySlot):
bindings/js/kjs_window.cpp: (KJS::ScheduledAction::execute):
bindings/objc/WebScriptObject.mm:
bindings/scripts/CodeGeneratorJS.pm:

22:41 Changeset [23111] by hyatt

Fix the CSS2 system font for 'menu' to match Mac. Makes the system fonts layout test pass.

22:28 Changeset [23110] by aroben

Reviewed by Hyatt.

Changed the file upload button to say "Choose File" to match Mac.

platform/win/TemporaryLinkStubs.cpp: (WebCore::fileButtonChooseFileLabel):

22:05 Changeset [20309] by andrew

JavaScriptCore:

Reviewed by Mark Rowe

Set Xcode editor to use 4 space indentation (http://webkit.org/coding/coding-style.html)

JavaScriptCore.xcodeproj/project.pbxproj:

JavaScriptGlue:

Reviewed by Mark Rowe

Set Xcode editor to use 4 space indentation (http://webkit.org/coding/coding-style.html)

JavaScriptGlue.xcodeproj/project.pbxproj:

WebCore:

Reviewed by Mark Rowe

Set Xcode editor to use 4 space indentation (http://webkit.org/coding/coding-style.html)

WebCore.xcodeproj/project.pbxproj:

WebKit:

Reviewed by Mark Rowe

Set Xcode editor to use 4 space indentation (http://webkit.org/coding/coding-style.html)

WebKit.xcodeproj/project.pbxproj:

21:57 Changeset [20308] by hyatt

land update results for frame pixel changes.

21:53 Changeset [20307] by hyatt

Land updated results for frames pixel tests.

21:53 Changeset [20306] by hyatt

Add layout test for frame borders case.

21:50 Changeset [20305] by hyatt

Fix for REGRESSION bug 13108, frame borders being painted when they shouldn't be. Rework both frame borders
and resizing to be more like other browsers.

Reviewed by ggaren

fast/frames/no-frame-borders.html

html/HTMLFrameElement.cpp: (WebCore::HTMLFrameElement::HTMLFrameElement): (WebCore::HTMLFrameElement::attach): (WebCore::HTMLFrameElement::parseMappedAttribute):
html/HTMLFrameElement.h: (WebCore::HTMLFrameElement::hasFrameBorder):
html/HTMLFrameElementBase.cpp: (WebCore::HTMLFrameElementBase::HTMLFrameElementBase): (WebCore::HTMLFrameElementBase::parseMappedAttribute):
html/HTMLFrameElementBase.h:
html/HTMLFrameSetElement.cpp: (WebCore::HTMLFrameSetElement::attach):
html/HTMLFrameSetElement.h: (WebCore::HTMLFrameSetElement::hasFrameBorder):
html/HTMLIFrameElement.cpp: (WebCore::HTMLIFrameElement::HTMLIFrameElement):
rendering/RenderFrame.cpp: (WebCore::RenderFrame::edgeInfo):
rendering/RenderFrame.h:
rendering/RenderFrameSet.cpp: (WebCore::RenderFrameSet::paint): (WebCore::RenderFrameSet::GridAxis::resize): (WebCore::RenderFrameSet::fillFromEdgeInfo): (WebCore::RenderFrameSet::computeEdgeInfo): (WebCore::RenderFrameSet::edgeInfo): (WebCore::RenderFrameSet::layout): (WebCore::RenderFrameSet::startResizing): (WebCore::RenderFrameSet::continueResizing): (WebCore::RenderFrameSet::canResizeRow): (WebCore::RenderFrameSet::canResizeColumn): (WebCore::RenderFrameSet::splitPosition): (WebCore::RenderFrameSet::hitTestSplit): (WebCore::RenderFrameSet::dump):
rendering/RenderFrameSet.h: (WebCore::): (WebCore::FrameEdgeInfo::FrameEdgeInfo): (WebCore::FrameEdgeInfo::preventResize): (WebCore::FrameEdgeInfo::allowBorder): (WebCore::FrameEdgeInfo::setPreventResize): (WebCore::FrameEdgeInfo::setAllowBorder):

21:32 Changeset [20304] by bdash

2007-03-19 Mark Rowe <mrowe@apple.com>

Rubber-stamped by Brady.

Update references to bugzilla.opendarwin.org with bugs.webkit.org.

bindings/c/c_utility.cpp: (KJS::Bindings::convertUTF8ToUTF16):
kjs/function.cpp: (KJS::FunctionImp::callAsFunction):
kjs/grammar.y:
kjs/keywords.table:
kjs/lexer.cpp: (KJS::Lexer::shift):

2007-03-19 Mark Rowe <mrowe@apple.com>

Rubber-stamped by Brady.

Update references to bugzilla.opendarwin.org with bugs.webkit.org.

ChangeLog:
WebCore.vcproj/WebCore/build-generated-files.sh:
manual-tests/ATSU-bad-layout.html:
manual-tests/accidental-strict-mode.html:
manual-tests/applet-param-no-name.html:
manual-tests/bidi-parens.html:
manual-tests/bugzilla-3855.html:
manual-tests/bugzilla-4840.html:
manual-tests/bugzilla-6821.html:
manual-tests/containing-block-position-chage.html:
manual-tests/contenteditable-link.html:
manual-tests/css3-cursor-fallback-quirks.html:
manual-tests/css3-cursor-fallback-strict.html:
manual-tests/custom-cursors.html:
manual-tests/dictionary-scrolled-iframe.html:
manual-tests/dom-manipulation-on-resize.html:
manual-tests/drag-image-to-address-bar.html:
manual-tests/empty-link-target.html:
manual-tests/empty-title-popup.html:
manual-tests/first-line-style-crash.html:
manual-tests/invalid-mouse-event.html:
manual-tests/left-overflow-repaint.html:
manual-tests/linkjump-3.html:
manual-tests/log-keypress-events.html:
manual-tests/named-window-blank-target.html:
manual-tests/plain-text-paste.html:
manual-tests/plugin-controller-datasource.html:
manual-tests/pre-tab-selection-rect.html:
manual-tests/redirection-target.html:
manual-tests/redraw-page-cache-visited-links.html:
manual-tests/reset-initiatedDrag.html:
manual-tests/resources/named-window-blank-target-step2.html:
manual-tests/resources/named-window-blank-target-step3.html:
manual-tests/resources/named-window-blank-target-step4.html:
manual-tests/resources/redraw-page-cache-visited-links-2.html:
manual-tests/scrollbar-hittest.html:
manual-tests/scrollbar-hittest2.html:
manual-tests/subview-click-assertion.html:
manual-tests/tabbing-input-google.html:
manual-tests/text-field-autoscroll.html:
manual-tests/textarea-after-stylesheet-link.html:
manual-tests/textarea-focus.html:
manual-tests/whitespace-pre-affinity.html:

2007-03-19 Mark Rowe <mrowe@apple.com>

Rubber-stamped by Brady.

Update references to bugzilla.opendarwin.org with bugs.webkit.org.

WebInspector/webInspector/inspector.css:
WebView/WebHTMLView.mm: (-[WebHTMLView firstRectForCharacterRange:]):
WebView/WebView.mm: (-[WebView initWithFrame:frameName:groupName:]):

20:36 Changeset [23109] by aroben

Rubberstamped by Maciej.

Skipped: Added another crasher.

20:01 JavaScript and DOM Benchmarks edited by mjs@apple.com

(diff)

19:34 Changeset [20303] by mjs

Added a random JS benchmark.

misc/morph.html: Added.

19:18 JavaScript and DOM Benchmarks edited by mjs@apple.com

(diff)

19:14 JavaScript and DOM Benchmarks edited by mjs@apple.com

(diff)

19:11 JavaScript and DOM Benchmarks edited by mjs@apple.com

(diff)

18:53 Changeset [23108] by ggaren

Reviewed by Oliver Hunt.

Fixed <rdar://problem/5061807> STRESS (2): Crash in PlatformScrollbar::setRect

No test case because I can't seem to get the Windows DRT to cooperate. I'll try again tomorrow.

The problem was that the FrameView would try to reset the document's scrollbars after having
been removed from the page, so its page pointer would be NULL. Added NULL checks.

This situation ultimately results from the FrameView's non-trivial destructor, which
creates a fragile dependency on tear-down timing. It looks like the timing on Windows
is slightly different from the timing on Mac. I didn't see any obvious resulting bugs,
though, so the NULL checks should be sufficient for now.

platform/win/ScrollViewWin.cpp: (WebCore::ScrollView::windowResizerRect): (WebCore::ScrollView::addToDirtyRegion): (WebCore::ScrollView::scrollBackingStore): (WebCore::ScrollView::updateBackingStore):

18:50 JavaScript and DOM Benchmarks edited by mjs@apple.com

(diff)

18:50 Changeset [20302] by bdash

Roll out r20297, fix for http://bugs.webkit.org/show_bug.cgi?id=13091, as it causes regressions in layout tests.

18:40 Changeset [20301] by weinig

Reviewed by Mark.

Update results after Hyatt's patch in r20268.

tables/mozilla/bugs/bug137388-2-expected.checksum:
tables/mozilla/bugs/bug137388-2-expected.png:
tables/mozilla/bugs/bug137388-2-expected.txt:
tables/mozilla/bugs/bug137388-3-expected.checksum:
tables/mozilla/bugs/bug137388-3-expected.png:
tables/mozilla/bugs/bug137388-3-expected.txt:
tables/mozilla/bugs/bug50695-2-expected.checksum:
tables/mozilla/bugs/bug50695-2-expected.png:
tables/mozilla/bugs/bug50695-2-expected.txt:
tables/mozilla_expected_failures/bugs/bug137388-1-expected.checksum:
tables/mozilla_expected_failures/bugs/bug137388-1-expected.png:
tables/mozilla_expected_failures/bugs/bug137388-1-expected.txt:

18:32 Changeset [23107] by aroben

LayoutTestsWin:

Reviewed by Geoff.

Skipped: Added a new crasher.

WebKitToolsWin:

Reviewed by Geoff.

Added a basic implementation of eventSender. It supports mouse and key
events.

DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/DraggingInfo.h: Added. (DraggingInfo::DraggingInfo): (DraggingInfo::~DraggingInfo): (DraggingInfo::dataObject): (DraggingInfo::dropSource):
DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/DumpRenderTree.cpp: (main): Made the WebView's HWND a global so that EventSender can access it.
DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/DumpRenderTree.h: Added extern declarations.
DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/DumpRenderTree.vcproj: Added new files to project.
DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/EventSender.cpp: Added. (getDragModeCallback): (setDragModeCallback): (leapForwardCallback): (currentEventTime): (makeMsg): (mouseDownCallback): (pointl): (doMouseUp): (mouseUpCallback): (mouseClickCallback): (doMouseMove): (mouseMoveToCallback): (replaySavedEvents): (keyDownCallback): (getClass): (makeEventSender):
DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/EventSender.h: Added.
DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/LayoutTestController.cpp: Re-ordered #includes.
DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/LayoutTestController.h: Changed #include to forward declarations.
DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/WaitUntilDoneDelegate.cpp: (WaitUntilDoneDelegate::windowScriptObjectAvailable): Make the eventSender available. (WaitUntilDoneDelegate::doDragDrop): Implemented.
DumpRenderTree/DumpRenderTree.vcproj/DumpRenderTree/WaitUntilDoneDelegate.h:

WebKitWin:

Reviewed by Geoff.

Added IWebFramePrivate::layout to support eventSender in DRT.

Made IWebView inherit from IDropTarget to match Mac, and added
IWebUIDelegatePrivate::doDragDrop to support drag-and-drop in DRT.

Interfaces/IWebFramePrivate.idl: Added layout().
Interfaces/IWebUIDelegatePrivate.idl: Added doDragDrop().
Interfaces/IWebView.idl: Added inheritance from IDropTarget.
WebDragClient.cpp: (WebDragClient::startDrag): Call the UI delegate if possible, otherwise fallback to OLE's DoDragDrop.
WebFrame.cpp: (WebFrame::layout): Implemented.
WebFrame.h:
WebView.h:

18:01 JavaScript and DOM Benchmarks edited by mjs@apple.com

(diff)

17:59 JavaScript and DOM Benchmarks created by mjs@apple.com

17:59 WikiStart edited by mjs@apple.com

(diff)

17:39 Changeset [20300] by kjk

Not reviewed - gdk build fix.

platform/gdk/ScrollViewGdk.cpp: (WebCore::ScrollView::update): add a stub

17:37 Changeset [20299] by beidson

Reviewed by Hyatt

http://bugs.webkit.org/show_bug.cgi?id=13111
When stopped a load before it completes, partial images become the
broken image icon

loader/loader.cpp: (WebCore::Loader::didFail): Split off into a method that knows the difference between failed and cancelled. If a load is cancelled, don't call error() on the object - only remove it from the cache (WebCore::Loader::cancelRequests): Call didFail(loader, true)
loader/loader.h: Added didFail(SubresourceLoader*, bool cancelled)

16:42 Changeset [20298] by bdash

2007-03-18 Mitz Pettel <mitz@webkit.org>

Reviewed by Adele.

fix http://bugs.webkit.org/show_bug.cgi?id=13101 REGRESSION (NativeTextField): Specifying word-wrap:break-word makes text field wrap

Test: fast/forms/input-text-word-wrap.html

rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::createInnerTextStyle): Force the inner block to 'word-wrap:normal'.

2007-03-18 Mitz Pettel <mitz@webkit.org>

Reviewed by Adele.

test for http://bugs.webkit.org/show_bug.cgi?id=13101 REGRESSION (NativeTextField): Specifying word-wrap:break-word makes text field wrap

fast/forms/input-text-word-wrap-expected.checksum: Added.
fast/forms/input-text-word-wrap-expected.png: Added.
fast/forms/input-text-word-wrap-expected.txt: Added.
fast/forms/input-text-word-wrap.html: Added.

16:40 Changeset [20297] by bdash

2007-03-18 Mitz Pettel <mitz@webkit.org>

Reviewed by Tim Hatcher.

fix http://bugs.webkit.org/show_bug.cgi?id=13091 REGRESSION (r20075): Wrong text style and pixel break in Apple Store Locator HUD

Test: fast/innerHTML/additional-inline-style.html

editing/markup.cpp: (WebCore::startMarkup): Changed to add inline style based on CSS rules only in AnnotateForInterchange mode.

2007-03-18 Mitz Pettel <mitz@webkit.org>

Reviewed by Tim Hatcher.

test and updated result for http://bugs.webkit.org/show_bug.cgi?id=13091 REGRESSION (r20075): Wrong text style and pixel break in Apple Store Locator HUD

editing/pasteboard/paste-table-002-expected.txt:
fast/innerHTML/additional-inline-style-expected.txt: Added.
fast/innerHTML/additional-inline-style.html: Added.

16:25 Changeset [20296] by bdash

2007-03-18 Dan Waylonis <waylonis@mac.com>

Reviewed by Tim Hatcher.

Fix http://bugs.webkit.org/show_bug.cgi?id=13005
Bug 13005: WebScriptObject +throwException needs NULL check.

Add checking for NULL interpreter before throwing exception.

bindings/objc/WebScriptObject.mm: (+[WebScriptObject throwException:]):

2007-03-18 Dan Waylonis <waylonis@mac.com>

Reviewed by Tim Hatcher.

DumpRenderTree changes for http://bugs.webkit.org/show_bug.cgi?id=13005
Bug 13005: WebScriptObject +throwException needs NULL check.

Add tests to ensure that a plugin can safely throw an exception in dealloc.

DumpRenderTree/ObjCPlugin.h:
DumpRenderTree/ObjCPlugin.m: (+[ObjCPlugin webScriptNameForKey:]): (+[ObjCPlugin isKeyExcludedFromWebScript:]): (-[ObjCPlugin dealloc]):

2007-03-18 Dan Waylonis <waylonis@mac.com>

Reviewed by Tim Hatcher.

Tests for http://bugs.webkit.org/show_bug.cgi?id=13005
Bug 13005: WebScriptObject +throwException needs NULL check.

A plugin that throws on dealloc can crash WebKit.

plugins/throw-on-dealloc-expected.txt: Added.
plugins/throw-on-dealloc.html: Added.

16:02 Changeset [20295] by ggaren

JavaScriptCore:

Reviewed by Oliver Hunt.

Exposed some extra toUInt32 functionality, as part of the fix for
REGRESSION: Incomplete document.all implementation breaks abtelectronics.com
(Style Change Through JavaScript Blanks Content)

JavaScriptCore.exp:
kjs/identifier.h: (KJS::Identifier::toUInt32):

LayoutTests:

Reviewed by Oliver Hunt.

Layout test for http://bugs.webkit.org/show_bug.cgi?id=13106
REGRESSION: Incomplete document.all implementation breaks abtelectronics.com
(Style Change Through JavaScript Blanks Content)

fast/dom/collection-null-like-arguments-expected.txt: Added.
fast/dom/collection-null-like-arguments.html: Added.

WebCore:

Reviewed by Oliver Hunt.

Fixed http://bugs.webkit.org/show_bug.cgi?id=13106
REGRESSION: Incomplete document.all implementation breaks abtelectronics.com
(Style Change Through JavaScript Blanks Content)

When indexing into collections, treat the empty string as an invalid argument,
instead of the number 0.

bindings/js/kjs_html.cpp: (KJS::JSHTMLCollection::getOwnPropertySlot): (KJS::JSHTMLCollection::callAsFunction): (KJS::JSHTMLCollectionPrototypeFunction::callAsFunction):

11:42 Changeset [23106] by sfalken

Bump version for submit

11:39 Changeset [20294] by sfalken

New tag.

11:10 Changeset [20293] by ggaren

Removed duplicate export name.

JavaScriptCore.exp:

11:07 Changeset [20292] by ggaren

Reviewed by Maciej Stachowiak.

Fixed <rdar://problem/5064964> Repro ASSERT failure in JS Bindings when
closing window @ lowtrades.bptrade.com

Unfortunately, the bindings depend on UString and Identifier as string
representations. So, they need to acquire the JSLock when doing something
that will ref/deref their strings.

Layout tests, the original site, and Java, Flash, and Quicktime on the
web work. No leaks reported. No automated test for this because testing
the Java bindings, like math, is hard.

bindings/runtime.h: Made Noncopyable, just to be sure.

bindings/c/c_class.cpp: (KJS::Bindings::CClass::~CClass): Acquire the JSLock and explicitly clear the keys in our hashtable, since they're UString::Reps, and ref/deref aren't thread-safe. (KJS::Bindings::CClass::methodsNamed): Also acquire the JSLock when adding keys to the table, since the table ref's them. (KJS::Bindings::CClass::fieldNamed): ditto.

bindings/c/c_utility.cpp: Removed dead function. (KJS::Bindings::convertValueToNPVariant): Acquire the JSLock because doing it recursively is pretty cheap, and it's just too confusing to tell whether all our callers do it for us. (KJS::Bindings::convertNPVariantToValue): ditto
bindings/c/c_utility.h:

bindings/jni/jni_class.cpp: Same deal as c_class.cpp. (JavaClass::JavaClass): (JavaClass::~JavaClass):

bindings/jni/jni_instance.cpp: Same deal as c_utility.cpp. (JavaInstance::stringValue):
bindings/jni/jni_jsobject.cpp: (JavaJSObject::convertValueToJObject):

bindings/jni/jni_runtime.cpp: (JavaMethod::~JavaMethod): Moved from header, for clarity. (appendClassName): Made this static, so the set of callers is known, and we can assert that we hold the JSLock. Also changed it to take a UString reference, which makes the calling code simpler. (JavaMethod::signature): Store the ASCII value we care about instead of a UString, since UString is so much more hassle. Hold the JSLock while building up the temporary UString.

bindings/jni/jni_runtime.h: Nixed dead code in JavaMethod. (KJS::Bindings::JavaString::JavaString): Hold a UString::Rep instead of a UString, so we can acquire the JSLock and explicitly release it. (KJS::Bindings::JavaString::_commonInit): (KJS::Bindings::JavaString::~JavaString): (KJS::Bindings::JavaString::UTF8String): (KJS::Bindings::JavaString::uchars): (KJS::Bindings::JavaString::length): (KJS::Bindings::JavaString::ustring):

bindings/jni/jni_utility.cpp: (KJS::Bindings::convertArrayInstanceToJavaArray): Made this static, so the set of callers is known, and we can assert that we hold the JSLock. (KJS::Bindings::convertValueToJValue): Acquire the JSLock because doing it recursively is pretty cheap, and it's just too confusing to tell whether all our callers do it for us.

bindings/objc/objc_runtime.h: Nixed some dead code.
bindings/objc/objc_utility.mm: (KJS::Bindings::convertNSStringToString): Same drill as above.

10:39 Changeset [20291] by ap

Reviewed by Geoff.

http://bugs.webkit.org/show_bug.cgi?id=13105
REGRESSION: an exception raised when calculating base value of a dot expression is not returned

Test: fast/js/dot-node-base-exception.html

kjs/nodes.cpp: (FunctionCallDotNode::evaluate): Added the necessary KJS_CHECKEXCEPTIONVALUE.

10:36 Changeset [20290] by sfalken

2007-03-18 Steve Falkenburg <sfalken@apple.com>

Build fix.

JavaScriptCore.vcproj/testkjs/testkjs.vcproj:

04:34 Changeset [20289] by hyatt

Use fillRect instead of drawLine, since drawLine has some odd behavior (it's only used for borders).

Reviewed by aroben

platform/graphics/IntRect.h:
rendering/RenderFrameSet.cpp: (WebCore::RenderFrameSet::paintColumnBorder): (WebCore::RenderFrameSet::paintRowBorder):

04:11 Changeset [20288] by hyatt

Change the frameborder color to be light gray for framesets.

Reviewed by aroben

rendering/RenderFrameSet.cpp: (WebCore::borderFillColor): (WebCore::RenderFrameSet::paintColumnBorder): (WebCore::RenderFrameSet::paintRowBorder):

03:53 Changeset [20287] by hyatt

Make iframes work properly with frameborder="0".

Reviewed by aroben

html/HTMLIFrameElement.cpp: (WebCore::HTMLIFrameElement::mapToEntry): (WebCore::HTMLIFrameElement::parseMappedAttribute):

03:52 Changeset [20286] by hyatt

Add new iframe layout test.

03:00 Changeset [20285] by hyatt