Context Navigation

Fixes <https://bugs.webkit.org/show_bug.cgi?id=15100>:

XMLHttpRequest::urlMatchesDocumentDomain raises error if port
information does not match exactly

Refactor our security check for XMLHttpRequest into SecurityOrigin so
we can reuse it in other places. This leverages our default port
technology in SecurityOrigin.

I wasn't sure how to write a test for this because the LayoutTests run
on non-default ports.

platform/SecurityOrigin.cpp: (WebCore::SecurityOrigin::canRequest):
platform/SecurityOrigin.h:
xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::send): (WebCore::XMLHttpRequest::willSendRequest):
xml/XMLHttpRequest.h:

22:39 Changeset [34531] by abarth@webkit.org

2008-06-13 Adam Barth <abarth@webkit.org>

Reviewed by Darin Adler.

Test whether we permit XMLHttpRequest once document.domain is set.

http/tests/xmlhttprequest/document-domain-set-expected.txt: Added.
http/tests/xmlhttprequest/document-domain-set.html: Added.

22:39 Changeset [34530] by abarth@webkit.org

2008-06-13 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Removed unnecessary dependencies.

platform/SecurityOrigin.cpp:
platform/SecurityOrigin.h:

22:06 Changeset [34529] by cwzwarich@webkit.org

2008-06-13 Cameron Zwarich <cwzwarich@uwaterloo.ca>

Reviewed by Maciej.

Combine TrueNode and FalseNode to make BooleanNode, and remove the
unused class PlaceholderTrueNode.

kjs/grammar.y:
kjs/nodes.cpp: (KJS::BooleanNode::emitCode):
kjs/nodes.h: (KJS::BooleanNode::): (KJS::BooleanNode::precedence):
kjs/nodes2string.cpp: (KJS::BooleanNode::streamTo):

21:40 Changeset [34528] by cwzwarich@webkit.org

2008-06-13 Cameron Zwarich <cwzwarich@uwaterloo.ca>

Reviewed by Maciej.

Eliminate the use of temporaries to store the left hand side of an
expression when the right hand side is a constant. This slightly
improves the generated bytecode for a few SunSpider tests, but it is
mostly in preparation for fixing

Bug 19484: More instructions needs to use temporary registers
<https://bugs.webkit.org/show_bug.cgi?id=19484>

VM/CodeGenerator.h: (KJS::CodeGenerator::leftHandSideNeedsCopy): (KJS::CodeGenerator::emitNodeForLeftHandSide):
kjs/nodes.cpp: (KJS::BracketAccessorNode::emitCode): (KJS::ReadModifyResolveNode::emitCode): (KJS::AssignDotNode::emitCode): (KJS::ReadModifyDotNode::emitCode): (KJS::AssignBracketNode::emitCode): (KJS::ReadModifyBracketNode::emitCode):
kjs/nodes.h: (KJS::ExpressionNode::): (KJS::FalseNode::): (KJS::TrueNode::): (KJS::NumberNode::): (KJS::StringNode::):

21:12 Changeset [34527] by mjs@apple.com

2008-06-13 Maciej Stachowiak <mjs@apple.com>

Reviewed by Oliver.

prettify opcode stats output

I changed things to be a bit more aligned, also there is a new
section listing most common opcodes and most common sequences that
include them.

VM/Opcode.cpp: (KJS::): (KJS::OpcodeStats::~OpcodeStats):
VM/Opcode.h:

16:35 Changeset [34526] by darin@apple.com

WebKit/gtk:

2008-06-13 Darin Adler <darin@apple.com>

try to fix build

WebCoreSupport/FrameLoaderClientGtk.h: Add missing argument.

WebKit/qt:

2008-06-13 Darin Adler <darin@apple.com>

try to fix build

WebCoreSupport/FrameLoaderClientQt.h: Add missing argument.

WebKit/wx:

2008-06-13 Darin Adler <darin@apple.com>

try to fix build

WebKitSupport/FrameLoaderClientWx.h: Add missing argument.

15:32 Changeset [34525] by cwzwarich@webkit.org

2008-06-13 Cameron Zwarich <cwzwarich@uwaterloo.ca>

Reviewed by Darin.

Add a test for existing loader behaviour in preparation for:

Bug 19422: Distinct redirects from the same link do not create distinct history items
<https://bugs.webkit.org/show_bug.cgi?id=19422>

http/tests/navigation/redirect-cycle-expected.txt: Added.
http/tests/navigation/redirect-cycle.html: Added.
http/tests/navigation/resources/redirect-cycle-1.pl: Added.
http/tests/navigation/resources/redirect-cycle-2.pl: Added.

15:05 Changeset [34524] by mjs@apple.com

2008-06-13 Maciej Stachowiak <mjs@apple.com>

Reviewed by Darin.

enable a bunch of disabled layout tests which now run ok https://bugs.webkit.org/show_bug.cgi?id=19540

I also made a minor change to selftxhtml.js to handle the case
where the body is a frameset properly, and to frame.xhtml to match
the HTML version of that file, and what the tests expect.

dom/html/level2/html/HTMLFormElement10-expected.txt: Added.
dom/html/level2/html/HTMLFormElement10.html: Copied from dom/html/level2/html/HTMLFormElement10.html-disabled.
dom/html/level2/html/HTMLFormElement10.html-disabled: Removed.
dom/xhtml/level2/html/HTMLFormElement10-expected.txt: Added.
dom/xhtml/level2/html/HTMLFormElement10.xhtml: Copied from dom/xhtml/level2/html/HTMLFormElement10.xhtml-disabled.
dom/xhtml/level2/html/HTMLFormElement10.xhtml-disabled: Removed.
dom/xhtml/level2/html/HTMLFrameElement01-expected.txt: Added.
dom/xhtml/level2/html/HTMLFrameElement01.xhtml: Copied from dom/xhtml/level2/html/HTMLFrameElement01.xhtml-disabled.
dom/xhtml/level2/html/HTMLFrameElement01.xhtml-disabled: Removed.
dom/xhtml/level2/html/HTMLFrameElement02-expected.txt: Added.
dom/xhtml/level2/html/HTMLFrameElement02.xhtml: Copied from dom/xhtml/level2/html/HTMLFrameElement02.xhtml-disabled.
dom/xhtml/level2/html/HTMLFrameElement02.xhtml-disabled: Removed.
dom/xhtml/level2/html/HTMLFrameElement03-expected.txt: Added.
dom/xhtml/level2/html/HTMLFrameElement03.xhtml: Copied from dom/xhtml/level2/html/HTMLFrameElement03.xhtml-disabled.
dom/xhtml/level2/html/HTMLFrameElement03.xhtml-disabled: Removed.
dom/xhtml/level2/html/HTMLFrameElement04-expected.txt: Added.
dom/xhtml/level2/html/HTMLFrameElement04.xhtml: Copied from dom/xhtml/level2/html/HTMLFrameElement04.xhtml-disabled.
dom/xhtml/level2/html/HTMLFrameElement04.xhtml-disabled: Removed.
dom/xhtml/level2/html/HTMLFrameElement05-expected.txt: Added.
dom/xhtml/level2/html/HTMLFrameElement05.xhtml: Copied from dom/xhtml/level2/html/HTMLFrameElement05.xhtml-disabled.
dom/xhtml/level2/html/HTMLFrameElement05.xhtml-disabled: Removed.
dom/xhtml/level2/html/HTMLFrameElement06-expected.txt: Added.
dom/xhtml/level2/html/HTMLFrameElement06.xhtml: Copied from dom/xhtml/level2/html/HTMLFrameElement06.xhtml-disabled.
dom/xhtml/level2/html/HTMLFrameElement06.xhtml-disabled: Removed.
dom/xhtml/level2/html/HTMLFrameElement07-expected.txt: Added.
dom/xhtml/level2/html/HTMLFrameElement07.xhtml: Copied from dom/xhtml/level2/html/HTMLFrameElement07.xhtml-disabled.
dom/xhtml/level2/html/HTMLFrameElement07.xhtml-disabled: Removed.
dom/xhtml/level2/html/HTMLFrameElement08-expected.txt: Added.
dom/xhtml/level2/html/HTMLFrameElement08.xhtml: Copied from dom/xhtml/level2/html/HTMLFrameElement08.xhtml-disabled.
dom/xhtml/level2/html/HTMLFrameElement08.xhtml-disabled: Removed.
dom/xhtml/level2/html/HTMLFrameElement09-expected.txt: Added.
dom/xhtml/level2/html/HTMLFrameElement09.xhtml: Copied from dom/xhtml/level2/html/HTMLFrameElement09.xhtml-disabled.
dom/xhtml/level2/html/HTMLFrameElement09.xhtml-disabled: Removed.
dom/xhtml/level2/html/HTMLFrameSetElement01-expected.txt: Added.
dom/xhtml/level2/html/HTMLFrameSetElement01.xhtml: Copied from dom/xhtml/level2/html/HTMLFrameSetElement01.xhtml-disabled.
dom/xhtml/level2/html/HTMLFrameSetElement01.xhtml-disabled: Removed.
dom/xhtml/level2/html/HTMLFrameSetElement02-expected.txt: Added.
dom/xhtml/level2/html/HTMLFrameSetElement02.xhtml: Copied from dom/xhtml/level2/html/HTMLFrameSetElement02.xhtml-disabled.
dom/xhtml/level2/html/HTMLFrameSetElement02.xhtml-disabled: Removed.
dom/xhtml/level2/html/resources/frame.xhtml:
dom/xhtml/level2/html/selfxhtml.js:

14:47 Changeset [34523] by darin@apple.com

WebCore:

2008-06-13 Darin Adler <darin@apple.com>

Reviewed by John Sullivan.

added FormState argument to action policy functions

loader/FrameLoader.cpp: (WebCore::FrameLoader::checkNewWindowPolicy): (WebCore::FrameLoader::checkNavigationPolicy):
loader/FrameLoaderClient.h:
svg/graphics/SVGImageEmptyClients.h: (WebCore::SVGEmptyFrameLoaderClient::dispatchDecidePolicyForNewWindowAction): (WebCore::SVGEmptyFrameLoaderClient::dispatchDecidePolicyForNavigationAction):

WebKit/gtk:

2008-06-13 Darin Adler <darin@apple.com>

Reviewed by John Sullivan.

updated for addition of FormState argument to action policy functions

WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::dispatchDecidePolicyForNewWindowAction): (WebKit::FrameLoaderClient::dispatchDecidePolicyForNavigationAction):

WebKit/mac:

2008-06-13 Darin Adler <darin@apple.com>

Reviewed by John Sullivan.

updated for addition of FormState argument to action policy functions
added WebActionFormKey

WebCoreSupport/WebFrameLoaderClient.h:
WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::dispatchDecidePolicyForNewWindowAction): (WebFrameLoaderClient::dispatchDecidePolicyForNavigationAction): (WebFrameLoaderClient::actionDictionary):
WebKit.exp:
WebView/WebPolicyDelegate.mm:
WebView/WebPolicyDelegatePrivate.h:

WebKit/qt:

2008-06-13 Darin Adler <darin@apple.com>

Reviewed by John Sullivan.

updated for addition of FormState argument to action policy functions

WebCoreSupport/FrameLoaderClientQt.cpp: (WebCore::FrameLoaderClientQt::dispatchDecidePolicyForNewWindowAction): (WebCore::FrameLoaderClientQt::dispatchDecidePolicyForNavigationAction):

WebKit/win:

2008-06-13 Darin Adler <darin@apple.com>

Reviewed by John Sullivan.

updated for addition of FormState argument to action policy functions
added WebActionFormKey

Interfaces/IWebPolicyDelegate.idl:
WebActionPropertyBag.cpp: (WebActionPropertyBag::WebActionPropertyBag): (WebActionPropertyBag::createInstance): (WebActionPropertyBag::AddRef): (WebActionPropertyBag::Release): (WebActionPropertyBag::Read):
WebActionPropertyBag.h:
WebFrame.cpp: (WebFrame::dispatchDecidePolicyForNewWindowAction): (WebFrame::dispatchDecidePolicyForNavigationAction):

WebKit/wx:

2008-06-13 Darin Adler <darin@apple.com>

Reviewed by John Sullivan.

updated for addition of FormState argument to action policy functions

WebKitSupport/FrameLoaderClientWx.cpp: (WebCore::FrameLoaderClientWx::dispatchDecidePolicyForNewWindowAction): (WebCore::FrameLoaderClientWx::dispatchDecidePolicyForNavigationAction):

11:23 Changeset [34522] by darin@apple.com

2008-06-13 Darin Adler <darin@apple.com>

Scripts/make-js-test-wrappers: Added three more exceptions.

10:45 Changeset [34521] by kmccullough@apple.com

2008-06-13 Kevin McCullough <kmccullough@apple.com>

Reviewed by Geoff.

<rdar://problem/5969992> JSProfiler: Remove the recursion limit in the
profiler.

Remove recursion from exclude(). This leaves only focus() to fix.

JavaScriptCore.exp: Change the signatures of the exported functions.
profiler/Profile.cpp: (KJS::Profile::forEach): I added a traverseNextNodePreOrder() function and so needed to distinguish the other function by labeling it traverseNextNodePostOrder(). (KJS::Profile::exclude): All new exclude that iteratively walks the tree
profiler/Profile.h: (KJS::Profile::focus): Add a null check for m_head.
profiler/ProfileNode.cpp: (KJS::ProfileNode::traverseNextNodePostOrder): Renamed (KJS::ProfileNode::traverseNextNodePreOrder): Walks the tree in pre- order, where the parent is processed before the children. (KJS::ProfileNode::setTreeVisible): Iterate over the sub-tree and set all of the nodes visible value. This changes another function that used recursion. (KJS::ProfileNode::exclude): Remove recursion from this function. Because we now check for m_visible and we are walking the tree in pre- order we do not need to check if an excluded node is in an excluded sub-tree.
profiler/ProfileNode.h: Added specific selfTime functions to facilitate exclude(). (KJS::ProfileNode::setSelfTime): (KJS::ProfileNode::setActualSelfTime): (KJS::ProfileNode::setVisibleSelfTime):

10:42 Changeset [34520] by justin.garcia@apple.com

2008-06-13 Justin Garcia <justin.garcia@apple.com>

Reviewed by Jess.

<rdar://problem/5702248> Crash on Copy Image from the contextual menu on a broken image in Mail

platform/mac/PasteboardMac.mm: (WebCore::Pasteboard::writeImage): Check to see the image loaded successfully before trying to put data for it on the Pasteboard. Mail shouldn't really include the Copy Image menu item, but if it does and the user selects it, we shouldn't crash.

05:51 Changeset [34519] by hausmann@webkit.org

2008-06-11 Marc Ordinas i Llopis <marc.ordinasillopis@collabora.co.uk>

With help from Siraj Razick <siraj.razick@collabora.co.uk>

Reviewed by Simon.

https://bugs.webkit.org/show_bug.cgi?id=19367
[Qt] Plugins are drawn over the scrollbars

01:22 WebDevelopers edited by cwzwarich@webkit.org

(diff)

06/12/08:

21:53 Changeset [34518] by darin@apple.com

2008-06-12 Darin Adler <darin@apple.com>

Reviewed by Maciej.

https://bugs.webkit.org/show_bug.cgi?id=19434 speed up SunSpider by avoiding some string boxing

Speeds up SunSpider by 1.1%.

Optimized code path for getting built-in properties from strings -- avoid
boxing with a string object in that case. We can make further changes to avoid
even more boxing, but this change alone is a win.

API/JSCallbackObjectFunctions.h: (KJS::JSCallbackObject::staticValueGetter): Use isObject instead of inherits in asssert, since the type of slotBase() is now JSValue, not JSObject. (KJS::JSCallbackObject::staticFunctionGetter): Ditto. (KJS::JSCallbackObject::callbackGetter): Ditto.

kjs/internal.cpp: (KJS::StringImp::getPrimitiveNumber): Updated for change of data member name. (KJS::StringImp::toBoolean): Ditto. (KJS::StringImp::toNumber): Ditto. (KJS::StringImp::toString): Ditto. (KJS::StringInstance::create): Added; avoids a bit of cut and paste code. (KJS::StringImp::toObject): Use StringInstance::create. (KJS::StringImp::toThisObject): Ditto. (KJS::StringImp::lengthGetter): Added. Replaces the getter that used to live in the StringInstance class. (KJS::StringImp::indexGetter): Ditto. (KJS::StringImp::indexNumericPropertyGetter): Ditto. (KJS::StringImp::getOwnPropertySlot): Added. Deals with built in properties of the string class without creating a StringInstance.

kjs/internal.h: (KJS::StringImp::getStringPropertySlot): Added. To be used by both the string and string object getOwnPropertySlot function.

kjs/lookup.h: (KJS::staticFunctionGetter): Updated since slotBase() is now a JSValue rather than a JSObject.

kjs/object.h: Removed PropertySlot::slotBase() function, which can now move back into property_slot.h where it belongs since it doesn't have to cast to JSObject*.

kjs/property_slot.cpp: (KJS::PropertySlot::functionGetter): Updated since slot.slotBase() is now a JSValue* instead of JSObject*. setGetterSlot still guarantees the base is a JSObject*.
kjs/property_slot.h: (KJS::PropertySlot::PropertySlot): Changed base to JSValue* intead of JSCell*. (KJS::PropertySlot::setStaticEntry): Ditto. (KJS::PropertySlot::setCustom): Ditto. (KJS::PropertySlot::setCustomIndex): Ditto. (KJS::PropertySlot::setCustomNumeric): Ditto. (KJS::PropertySlot::slotBase): Moved inline here since it no longer involves a downcast to JSObject*. (KJS::PropertySlot::setBase): Changed to JSValue*.

kjs/string_object.cpp: (KJS::StringInstance::getOwnPropertySlot): Changed to use getStringPropertySlot instead of coding the properties here. This allows sharing the code with StringImp.

kjs/string_object.h: Removed inlineGetOwnPropertySlot, lengthGetter, and indexGetter. Made one of the constructors protected.

kjs/value.h: Made getOwnPropertySlot private in the JSCell class -- this is better since it's not the real JSObject getOwnPropertySlot semantic and most callers shouldn't use it.

21:19 Changeset [34517] by sullivan@apple.com

2008-06-12 John Sullivan <sullivan@apple.com>

Reviewed by Dan and Darin

Clear up the confusion about _close (older private method) vs -close (newer public method).

WebView/WebView.mm: (-[WebView _isClosed]): simplified style (-[WebView _close]): added a comment about how clients and subclasses should use -close instead (-[WebView dealloc]): call -close instead of _close, so subclasses that override the public method will have the intended behavior (-[WebView close]): added a comment (-[WebView _windowWillClose:]): call -close instead of _close, so subclasses that override the public method will have the intended behavior

WebView/WebViewPrivate.h: added a comment about how clients and subclasses should use -close instead

20:28 Changeset [34516] by darin@apple.com

2008-06-12 Darin Adler <darin@apple.com>

try to fix no-SVG build

css/CSSComputedStyleDeclaration.cpp: (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue): Added a case for CSSPropertyWebkitMask.

20:10 Changeset [34515] by darin@apple.com

2008-06-12 Darin Adler <darin@apple.com>

try to fix Windows build

dom/Element.idl: Turn off ElementTraversal functions when generating COM. Not sure why they're failing, but turning them off for now seems OK.

07:34 Changeset [34514] by ap@webkit.org

Reviewed by Maciej.

Preparation to making JavaScript heap per-thread.

kjs/collector.cpp: (KJS::Collector::collect):
kjs/collector.h: (KJS::Collector::markListSet): The collector now holds the list of protected lists itself, to be made per-instance.

kjs/list.h: Changed to hold a pointer to a mark set this list is in, if any. (KJS::List::List): Explicitly initialize m_size with zero, as m_vector.size() is guaranteed to be such anyway. (KJS::List::append): Changed the fast case to only be executed as long as inline buffer is used, because otherwise, we now do more expensive checks.

kjs/list.cpp: (KJS::List::markLists): Renamed from markProtectedListsSlowCase, made it take the list set as a parameter. (KJS::List::slowAppend): If a non-immediate value is appended, the list needs to be added to an appropriate Heap's protected list. For now, a static Collector::markListSet() is used, but the code is layed out in preparation to making the switch to multiple heaps.

JavaScriptCore.exp: Updated export list.

05:52 Disk Cache edited by ben@meyerhome.net

(diff)

03:40 Changeset [34513] by abarth@webkit.org

WebCore:

2008-06-12 Adam Barth <abarth@webkit.org>

Rubberstamped by Maciej.

Roll 34504 back in.

Test: http/tests/xmlhttprequest/access-control-basic-allow-access-control-origin-header-data-url.html

platform/SecurityOrigin.cpp: (WebCore::SecurityOrigin::toString):

LayoutTests:

2008-06-12 Adam Barth <abarth@webkit.org>

Rubberstamped by Maciej.

Add the data url test back and set more agressive Cache-Control headers.

http/tests/xmlhttprequest/access-control-basic-allow-access-control-origin-header-data-url-expected.txt: Copied from LayoutTests/http/tests/xmlhttprequest/access-control-basic-allow-access-control-origin-header-data-url-expected.txt.
http/tests/xmlhttprequest/access-control-basic-allow-access-control-origin-header-data-url.html: Copied from LayoutTests/http/tests/xmlhttprequest/access-control-basic-allow-access-control-origin-header-data-url.html.
http/tests/xmlhttprequest/resources/access-control-basic-allow-access-control-origin-header.cgi:

03:02 Changeset [34512] by abarth@webkit.org

Continue backout of 34504.

02:35 Changeset [34511] by abarth@webkit.org

Partial backout of 34504. I know this will regress the *-data-url.html test, but I'd like to see if it fixes the broken test.

02:03 Changeset [34510] by cwzwarich@webkit.org

2008-06-12 Cameron Zwarich <cwzwarich@uwaterloo.ca>

Reviewed by Maciej.

Bug 19510: CodeBlock::needsFullScopeChain not always set for global code
<https://bugs.webkit.org/show_bug.cgi?id=19510>

This fixes the symptoms by using CodeGenerator::m_codeType to determine
when to use temporaries instead of CodeBlock::needsFullScopeChain, but
it does not fix the problem itself.

JavaScriptCore:

VM/CodeGenerator.h: (KJS::CodeGenerator::leftHandSideNeedsCopy):

LayoutTests:

fast/js/codegen-temporaries-multiple-global-blocks-expected.txt: Added.
fast/js/codegen-temporaries-multiple-global-blocks.html: Added.
fast/js/resources/codegen-temporaries-multiple-global-blocks-1.js: Added.
fast/js/resources/codegen-temporaries-multiple-global-blocks-2.js: Added.

02:00 Changeset [34509] by abarth@webkit.org

2008-06-12 Adam Barth <abarth@webkit.org>

Reviewed by Maciej.

Prevent caching of this resource so we always see the up-to-date value
of the Access-Control-Origin header.

http/tests/xmlhttprequest/resources/access-control-basic-allow-access-control-origin-header.cgi:

06/11/08:

23:52 Changeset [34508] by mitz@apple.com

Reverting changes I accidently made in r34507.

23:44 Changeset [34507] by mitz@apple.com

WebCore:

Reviewed by Dave Hyatt.

fix <rdar://problem/5876370> Safari 3.1.1 bad rendering of pages from http://www.legifrance.gouv.fr/

Tests: fast/parser/p-in-scope-strict.html

fast/parser/p-in-scope.html

html/HTMLParagraphElement.h: (WebCore::HTMLParagraphElement::endTagRequirement): Changed to require an end tag. The explicit closing of one P element by another is now done in error checking, and therefore takes scope into account, allowing for <p><button><p>, for example.

html/HTMLParser.cpp: (WebCore::HTMLParser::HTMLParser): Initialize m_hasPElementInScope. (WebCore::isScopingTag): Added. Returns whether the given tag represents a scoping element as defined in HTML 5 section 8.2.3.2. (WebCore::HTMLParser::formCreateErrorCheck): Added a call to pCloserCreateErrorCheck(). (WebCore::HTMLParser::ddCreateErrorCheck): Ditto. (WebCore::HTMLParser::dtCreateErrorCheck): Ditto. (WebCore::HTMLParser::nestedPCloserCreateErrorCheck): Added for use with <li>, which both closes P elements in scope and any previous LI. (WebCore::HTMLParser::pCloserCreateErrorCheck): Added. If there is a P element in scope, acts as if a </p> tag was seen. (WebCore::HTMLParser::pCloserStrictCreateErrorCheck): Ditto, but only in strict mode. Used for <table>. (WebCore::HTMLParser::getNode): Added entries for tags that close a P element in scope. (WebCore::HTMLParser::handleResidualStyleCloseTagAcrossBlocks): Added code to reset m_hasPElementInScope. (WebCore::HTMLParser::pushBlock): Added code to update m_hasPElementInScope. (WebCore::HTMLParser::popOneBlockCommon): Ditto. (WebCore::HTMLParser::checkIfHasPElementInScope): Added. Updates m_hasPElementInScope.

html/HTMLParser.h: (WebCore::HTMLParser::hasPElementInScope): Added. Calls checkIfHasPElementInScope() if needed and returns whether there is a P element in scope.

LayoutTests:

Reviewed by Dave Hyatt.

tests, updated tests and updated results for <rdar://problem/5876370> Safari 3.1.1 bad rendering of pages from http://www.legifrance.gouv.fr/

fast/block/basic/001.html:
fast/block/positioning/absolute-in-inline-ltr-2.html:
fast/block/positioning/absolute-in-inline-ltr-3.html:
fast/block/positioning/absolute-in-inline-ltr.html:
fast/block/positioning/absolute-in-inline-rtl-2.html:
fast/block/positioning/absolute-in-inline-rtl-3.html:
fast/block/positioning/absolute-in-inline-rtl.html:
fast/block/positioning/absolute-in-inline-short-ltr.html:
fast/block/positioning/absolute-in-inline-short-rtl.html:
fast/block/positioning/auto/006.html:
fast/inline/continuation-outlines-with-layers.html:
fast/inline/continuation-outlines.html:
fast/parser/p-in-scope-expected.txt: Added.
fast/parser/p-in-scope-strict-expected.txt: Added.
fast/parser/p-in-scope-strict.html: Added.
fast/parser/p-in-scope.html: Added.
fast/parser/resources/p-in-scope.css: Added.
fast/parser/resources/p-in-scope.js: Added.
fast/repaint/subtree-root-clip-2.html:
platform/mac/fast/block/basic/001-expected.txt:
platform/mac/fast/block/positioning/absolute-in-inline-ltr-2-expected.txt:
platform/mac/fast/block/positioning/absolute-in-inline-ltr-3-expected.txt:
platform/mac/fast/block/positioning/absolute-in-inline-ltr-expected.txt:
platform/mac/fast/block/positioning/absolute-in-inline-rtl-2-expected.txt:
platform/mac/fast/block/positioning/absolute-in-inline-rtl-3-expected.txt:
platform/mac/fast/block/positioning/absolute-in-inline-rtl-expected.txt:
platform/mac/fast/block/positioning/absolute-in-inline-short-ltr-expected.txt:
platform/mac/fast/block/positioning/absolute-in-inline-short-rtl-expected.txt:
platform/mac/fast/block/positioning/auto/006-expected.txt:
platform/mac/fast/inline/continuation-outlines-expected.txt:
platform/mac/fast/inline/continuation-outlines-with-layers-expected.txt:
platform/mac/fast/repaint/subtree-root-clip-2-expected.txt:

23:41 Changeset [34506] by abarth@webkit.org

WebCore:

008-06-11 Adam Barth <abarth@webkit.org>

Reviewed by Sam Weinig.

https://bugs.webkit.org/show_bug.cgi?id=19460

Update the security context of a document after calling document.open
or document.write. Basically, when a script open()s a document, the
document gains the security context of the script. Our implementation
now matches Firefox 3 on all these tests.

Tests: http/tests/security/aboutBlank/security-context-alias.html

http/tests/security/aboutBlank/security-context-grandchildren-alias.html
http/tests/security/aboutBlank/security-context-grandchildren.html
http/tests/security/aboutBlank/security-context-window-open.html
http/tests/security/aboutBlank/security-context-with-base-tag.html
http/tests/security/aboutBlank/security-context-write.html
http/tests/security/aboutBlank/security-context-writeln.html
http/tests/security/aboutBlank/security-context.html
http/tests/security/cookies/document-open.html

bindings/js/JSDOMWindowBase.cpp: (WebCore::createWindow):
bindings/js/JSHTMLDocumentCustom.cpp: (WebCore::JSHTMLDocument::open): (WebCore::JSHTMLDocument::write): (WebCore::JSHTMLDocument::writeln):
dom/Document.cpp: (WebCore::Document::open): (WebCore::Document::write): (WebCore::Document::writeln):
dom/Document.h:

LayoutTests:

008-06-11 Adam Barth <abarth@webkit.org>

Reviewed by Sam Weinig.

Tests for the security context of about:blank documents. These test
results all match Firefox 3.

fast/dom/resource-locations-in-created-html-document.html:
http/tests/security/aboutBlank/resources/iframe-with-about-blank-children.html: Added.
http/tests/security/aboutBlank/security-context-alias-expected.txt: Added.
http/tests/security/aboutBlank/security-context-alias.html: Added.
http/tests/security/aboutBlank/security-context-expected.txt: Added.
http/tests/security/aboutBlank/security-context-grandchildren-alias-expected.txt: Added.
http/tests/security/aboutBlank/security-context-grandchildren-alias.html: Added.
http/tests/security/aboutBlank/security-context-grandchildren-expected.txt: Added.
http/tests/security/aboutBlank/security-context-grandchildren.html: Added.
http/tests/security/aboutBlank/security-context-window-open-expected.txt: Added.
http/tests/security/aboutBlank/security-context-window-open.html: Added.
http/tests/security/aboutBlank/security-context-with-base-tag-expected.txt: Added.
http/tests/security/aboutBlank/security-context-with-base-tag.html: Added.
http/tests/security/aboutBlank/security-context-write-expected.txt: Added.
http/tests/security/aboutBlank/security-context-write.html: Added.
http/tests/security/aboutBlank/security-context-writeln-expected.txt: Added.
http/tests/security/aboutBlank/security-context-writeln.html: Added.
http/tests/security/aboutBlank/security-context.html: Added.
http/tests/security/aboutBlank/xss-DENIED-navigate-opener-javascript-url-expected.txt:
http/tests/security/cookies/document-open-expected.txt: Added.
http/tests/security/cookies/document-open.html: Added.

23:41 Changeset [34505] by abarth@webkit.org

WebCore:

2008-06-11 Adam Barth <abarth@webkit.org>

Reviewed by Sam Weinig.

https://bugs.webkit.org/show_bug.cgi?id=19460

Compute the cookie context at the same time as the SecurityOrigin.

Tests: http/tests/security/cookies/create-document.html

http/tests/security/cookies/xmlhttprequest.html

dom/Document.cpp: (WebCore::Document::cookie): (WebCore::Document::initSecurityOrigin):
dom/Document.h:
platform/SecurityOrigin.cpp:
platform/SecurityOrigin.h:

LayoutTests:

2008-06-11 Adam Barth <abarth@webkit.org>

Reviewed by Sam Weinig.

https://bugs.webkit.org/show_bug.cgi?id=19460

Test esoteric cookie contexts.

http/tests/security/cookies/create-document-expected.txt: Added.
http/tests/security/cookies/create-document.html: Added.
http/tests/security/cookies/resources/simple.xml: Added.
http/tests/security/cookies/xmlhttprequest-expected.txt: Added.
http/tests/security/cookies/xmlhttprequest.html: Added.

22:15 Changeset [34504] by abarth@webkit.org

WebCore:

2008-06-11 Adam Barth <abarth@webkit.org>

Reviewed and tweaked by Sam Weinig.

Fix for https://bugs.webkit.org/show_bug.cgi?id=19242
Data URLs should set an Access-Control-Origin of "null"

Correctly generate "null" as the value of the Access-Control-Origin
header for cross-site XMLHttpRequests for data URLs.

Test: http/tests/xmlhttprequest/access-control-basic-allow-access-control-origin-header-data-url.html

platform/SecurityOrigin.cpp: (WebCore::SecurityOrigin::toString):
xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::accessControlOrigin): (WebCore::XMLHttpRequest::crossSiteAccessRequest): (WebCore::XMLHttpRequest::handleAsynchronousMethodCheckResult):
xml/XMLHttpRequest.h:

LayoutTests:

2008-06-11 Adam Barth <abarth@webkit.org>

Reviewed and tweaked by Sam Weinig.

Test for https://bugs.webkit.org/show_bug.cgi?id=19242
Data URLs should set an Access-Control-Origin of "null"

Test that we correctly generate "null" as the value for the
Access-Control-Origin header when making requests from a data URL.

http/tests/xmlhttprequest/access-control-basic-allow-access-control-origin-header-data-url-expected.txt: Added.
http/tests/xmlhttprequest/access-control-basic-allow-access-control-origin-header-data-url.html: Added.
http/tests/xmlhttprequest/resources/access-control-basic-allow-access-control-origin-header.cgi:

19:51 Changeset [34503] by weinig@apple.com

Add missing newline

19:26 Changeset [34502] by weinig@apple.com

2008-06-11 Sam Weinig <sam@webkit.org>

Update results on non-leopard platforms to fix the buildbot.

http/tests/security/dataURL/xss-DENIED-to-data-url-sub-frame-uppercase-expected.txt:

19:19 Changeset [34501] by weinig@apple.com

Set executable bit

17:33 Changeset [34500] by cwzwarich@webkit.org

2008-06-11 Cameron Zwarich <cwzwarich@uwaterloo.ca>

Reviewed by Maciej.

Bug 19498: REGRESSION (r34497): crash while loading GMail
<https://bugs.webkit.org/show_bug.cgi?id=19498>

JavaScriptCore:

VM/CodeGenerator.cpp: (KJS::CodeGenerator::emitJumpIfTrueOptimized): (KJS::CodeGenerator::emitJumpIfTrue):
VM/CodeGenerator.h:
kjs/nodes.cpp: (KJS::DoWhileNode::emitCode): (KJS::WhileNode::emitCode): (KJS::ForNode::emitCode): (KJS::CaseBlockNode::emitCodeForBlock):

LayoutTests:

fast/js/logical-or-jless-expected.txt: Added.
fast/js/logical-or-jless.html: Added.
fast/js/resources/logical-or-jless.js: Added.

15:01 Changeset [34499] by darin@apple.com

2008-06-11 Darin Adler <darin@apple.com>

Reviewed by Maciej.

a little bit of cleanup and prep for some upcoming optimizations

JavaScriptCore.exp: Re-sorted this file (with sort command line tool).
VM/CodeBlock.cpp: (KJS::CodeBlock::dump): Fixed printf to avoid warnings -- to use %lu we need to make sure the type is unsigned long.
kjs/object.cpp: (KJS::Error::create): Eliminated unused error names array, and also put the strings into the code since there was already a switch statment. This also avoids having to contemplate a hypothetical access past the end of the array.
kjs/object.h: Got rid of errorNames.
kjs/property_slot.cpp: Deleted unused ungettableGetter.
kjs/property_slot.h: Ditto.
wtf/AlwaysInline.h: Added LIKELY alongside UNLIKELY.

12:55 Changeset [34498] by weinig@apple.com

WebCore:

2008-06-11 Sam Weinig <sam@webkit.org>

Reviewed by Adam Roben.

Add support for "*" wildcarding for Access-Control.

Test: http/tests/xmlhttprequest/access-control-basic-allow-star.html

xml/AccessItem.cpp: (WebCore::AccessItem::AccessItem): (WebCore::AccessItem::parseAccessItem): (WebCore::AccessItem::matches):

LayoutTests:

2008-06-11 Sam Weinig <sam@webkit.org>

Reviewed by Adam Roben.

Add basic test for "*" wildcarding for Access-Control.

http/tests/xmlhttprequest/access-control-basic-allow-star-expected.txt: Added.
http/tests/xmlhttprequest/access-control-basic-allow-star.html: Added.
http/tests/xmlhttprequest/resources/access-control-basic-allow-star.cgi: Added.

12:48 Changeset [34497] by cwzwarich@webkit.org

2008-06-11 Cameron Zwarich <cwzwarich@uwaterloo.ca>

Reviewed by Darin.

Bug 19457: Create fused opcodes for tests and conditional jumps
<https://bugs.webkit.org/show_bug.cgi?id=19457>

Add a new jless instruction, and modify the code generator to emit it
instead of the pair (less, jtrue).

Gives a 3.6% improvement on SunSpider.

VM/CodeBlock.cpp: (KJS::CodeBlock::dump):
VM/CodeGenerator.cpp: (KJS::CodeGenerator::CodeGenerator): (KJS::CodeGenerator::emitOpcode): (KJS::CodeGenerator::retrieveLastBinaryOp): (KJS::CodeGenerator::rewindBinaryOp): (KJS::CodeGenerator::emitJump): (KJS::CodeGenerator::emitJumpIfTrue): (KJS::CodeGenerator::emitJumpIfFalse): (KJS::CodeGenerator::emitMove): (KJS::CodeGenerator::emitNot): (KJS::CodeGenerator::emitEqual): (KJS::CodeGenerator::emitNotEqual): (KJS::CodeGenerator::emitStrictEqual): (KJS::CodeGenerator::emitNotStrictEqual): (KJS::CodeGenerator::emitLess): (KJS::CodeGenerator::emitLessEq): (KJS::CodeGenerator::emitPreInc): (KJS::CodeGenerator::emitPreDec): (KJS::CodeGenerator::emitPostInc): (KJS::CodeGenerator::emitPostDec): (KJS::CodeGenerator::emitToJSNumber): (KJS::CodeGenerator::emitNegate): (KJS::CodeGenerator::emitAdd): (KJS::CodeGenerator::emitMul): (KJS::CodeGenerator::emitDiv): (KJS::CodeGenerator::emitMod): (KJS::CodeGenerator::emitSub): (KJS::CodeGenerator::emitLeftShift): (KJS::CodeGenerator::emitRightShift): (KJS::CodeGenerator::emitUnsignedRightShift): (KJS::CodeGenerator::emitBitAnd): (KJS::CodeGenerator::emitBitXOr): (KJS::CodeGenerator::emitBitOr): (KJS::CodeGenerator::emitBitNot): (KJS::CodeGenerator::emitInstanceOf): (KJS::CodeGenerator::emitTypeOf): (KJS::CodeGenerator::emitIn): (KJS::CodeGenerator::emitLoad): (KJS::CodeGenerator::emitNewObject): (KJS::CodeGenerator::emitNewArray): (KJS::CodeGenerator::emitResolve): (KJS::CodeGenerator::emitGetScopedVar): (KJS::CodeGenerator::emitPutScopedVar): (KJS::CodeGenerator::emitResolveBase): (KJS::CodeGenerator::emitResolveWithBase): (KJS::CodeGenerator::emitResolveFunction): (KJS::CodeGenerator::emitGetById): (KJS::CodeGenerator::emitPutById): (KJS::CodeGenerator::emitPutGetter): (KJS::CodeGenerator::emitPutSetter): (KJS::CodeGenerator::emitDeleteById): (KJS::CodeGenerator::emitGetByVal): (KJS::CodeGenerator::emitPutByVal): (KJS::CodeGenerator::emitDeleteByVal): (KJS::CodeGenerator::emitPutByIndex): (KJS::CodeGenerator::emitNewFunction): (KJS::CodeGenerator::emitNewRegExp): (KJS::CodeGenerator::emitNewFunctionExpression): (KJS::CodeGenerator::emitCall): (KJS::CodeGenerator::emitReturn): (KJS::CodeGenerator::emitEnd): (KJS::CodeGenerator::emitConstruct): (KJS::CodeGenerator::emitPushScope): (KJS::CodeGenerator::emitPopScope): (KJS::CodeGenerator::emitDebugHook): (KJS::CodeGenerator::emitComplexJumpScopes): (KJS::CodeGenerator::emitJumpScopes): (KJS::CodeGenerator::emitNextPropertyName): (KJS::CodeGenerator::emitGetPropertyNames): (KJS::CodeGenerator::emitCatch): (KJS::CodeGenerator::emitThrow): (KJS::CodeGenerator::emitNewError): (KJS::CodeGenerator::emitJumpSubroutine): (KJS::CodeGenerator::emitSubroutineReturn):
VM/CodeGenerator.h:
VM/Machine.cpp: (KJS::Machine::privateExecute):
VM/Opcode.cpp: (KJS::):
VM/Opcode.h:

12:37 Changeset [34496] by darin@apple.com

2008-06-11 Darin Adler <darin@apple.com>

Reviewed by Alexey.

fix https://bugs.webkit.org/show_bug.cgi?id=19442 JavaScript array implementation doesn't maintain m_numValuesInVector when sorting

kjs/array_instance.cpp: (KJS::ArrayInstance::checkConsistency): Added. Empty inline version for when consistency checks are turned off. (KJS::ArrayInstance::ArrayInstance): Check consistency after construction. (KJS::ArrayInstance::~ArrayInstance): Check consistency before destruction. (KJS::ArrayInstance::put): Check consistency before and after. (KJS::ArrayInstance::deleteProperty): Ditto. (KJS::ArrayInstance::setLength): Ditto. (KJS::compareByStringPairForQSort): Use typedef for clarity. (KJS::ArrayInstance::sort): Check consistency before and after. Also broke the loop to set up sorting into two separate passes. Added FIXMEs about various exception safety issues. Added code to set m_numValuesInVector after sorting. (KJS::ArrayInstance::compactForSorting): Ditto.

kjs/array_instance.h: Added a definition of an enum for the types of consistency check and a declaration of the consistency checking function.

11:30 Changeset [34495] by justin.garcia@apple.com

WebCore:

2008-06-11 Justin Garcia <justin.garcia@apple.com>

Reviewed by Eric.

<https://bugs.webkit.org/show_bug.cgi?id=19455>
Crash at RenderObject::childAt during JustifyCenter

editing/ApplyStyleCommand.cpp: (WebCore::ApplyStyleCommand::surroundNodeRangeWithElement): Added a FIXME.
editing/CompositeEditCommand.cpp: (WebCore::CompositeEditCommand::moveParagraphs): If we're removing a line break that consists of a single '\n' in a text node by itself, remove the whole text node instead of just emptying it out.
editing/Selection.cpp: (WebCore::Selection::validate): Added a FIXME about canonicalizing to positions that aren't candidates.
editing/SelectionController.cpp: (WebCore::SelectionController::nodeWillBeRemoved): When the base and/or extent are about to be removed but the start and end aren't, change the base and extent to the start and end, but don't re-validate the selection, since doing so could move the start and end into the node that is about to be removed.

LayoutTests:

2008-06-11 Justin Garcia <justin.garcia@apple.com>

Reviewed by Eric.

<https://bugs.webkit.org/show_bug.cgi?id=19455>
Crash at RenderObject::childAt during JustifyCenter

editing/execCommand/19455-expected.txt: Added.
editing/execCommand/19455.html: Added.

11:27 Changeset [34494] by cfleizach@apple.com

<rdar://problem/5990049> AXTextMarker bytes getting messed up when getting AXTextMarkerRangeForUnorderedTextMarkersParameterizedAttribute

08:19 Changeset [34493] by hausmann@webkit.org

2008-06-11 Adriaan de Groot <groot@kde.org>

Reviewed by Simon.

Fix compilation with Sun Studio 12

It seems like the compiler instantiates a different template version. I.e.,
instead of parseUASheet<char [nnnn]>, it's probably instantiating
parseUASheet<char *>, which then passes 4 or 8 as the size value.

08:10 Changeset [34492] by hausmann@webkit.org

Benjamin Meyer <bmeyer@trolltech.com>

Fixed a crash when a slot connect to QWebPage::unsupportedContent would show a
modal dialog with an event loop.

We have a queued connection to various signals in the QNetworkReply, for which
it can happen that after releasing the QNetworkReply and disconnecting from it
a slot connected to one of the signals may still be called due to a posted
MetaCall event due to the queued connections. This patch removes the posted
events explicitly, fixes the coding style a bit and makes sure the same happens
when abort() is called.

08:10 Changeset [34491] by hausmann@webkit.org

Ariya Hidayat <ariya.hidayat@trolltech.com>

Fix left-click and middle-click mouse event are not properly accepted.

When copying or pasting text using left or middle-click, the event must be
accepted so that it will not be potentially processed further by the parent
and/or sub-classed widget.

07:57 Disk Cache edited by ben@meyerhome.net

(diff)

07:56 Changeset [34490] by hausmann@webkit.org

Ariya Hidayat <ariya.hidayat@trolltech.com>

Fix input element does not accept character typed in using AltGr.

EditorClient is modified to catch AltGr and Ctrl+Alt key combination.
This fixes http://trolltech.com/developer/task-tracker/index_html?id=207050&method=entry

07:56 Disk Cache created by ben@meyerhome.net

06/10/08:

22:51 Changeset [34489] by kevino@webkit.org

wx Linux build fix. Only use -undefined dynamic_lookup flag under Mac.

22:18 Changeset [34488] by kevino@webkit.org

wx build fix. Reorder include dirs so that WebCore/html/HTMLElementFactory.h appears before WebCore/DerivedSources/HTMLElementFactory.h. (See note in commit for more details. This is probably not the right fix, but this will get us buildinguntil the right fix is in place.)

22:16 Changeset [34487] by kevino@webkit.org

wx build fix. Link against libedit on Mac since HAVE(READLINE) is defined there.

14:31 Changeset [34486] by mitz@apple.com

fix the text encoding in this test

editing/selection/move-left-right.html:

14:06 Changeset [34485] by eric@webkit.org

Reviewed by mjs.

Fix disconnected event listener crasher
m_disconnectedNodesWithEventListeners not updated when nodes adopted
https://bugs.webkit.org/show_bug.cgi?id=19451

I failed to find a way to make this crash DRT, I can crash Debug Safari consistently.

dom/EventTarget.cpp: (WebCore::EventTarget::willMoveToNewOwnerDocument): (WebCore::EventTarget::didMoveToNewOwnerDocument):
dom/EventTarget.h:
dom/EventTargetNode.cpp: (WebCore::EventTargetNode::willMoveToNewOwnerDocument): (WebCore::EventTargetNode::didMoveToNewOwnerDocument):
dom/EventTargetNode.h:

13:49 Changeset [34484] by jchaffraix@webkit.org

WebCore:

2008-06-10 Julien Chaffraix <jchaffraix@webkit.org>

Reviewed by Eric.

Part of bug 19200: *.in files should embed more information
https://bugs.webkit.org/show_bug.cgi?id=19200

Move more parameters to the XML files.

dom/make_names.pl:
html/HTMLAttributeNames.in:
html/HTMLTagNames.in:
svg/svgattrs.in:
svg/svgtags.in:

09:56 Changeset [34483] by mjs@apple.com

2008-06-10 Maciej Stachowiak <mjs@apple.com>

Not reviewed.

tweak newly added markup.

blog/wp-content/themes/webkit/header.php:

09:54 Changeset [34482] by hausmann@webkit.org

2008-06-10 Joerg Bornemann <joerg.bornemann@trolltech.com>

Reviewed by Simon.

For the qmake based build make it possible to build against makespecs where
QMAKE_CC is defined in a configuration file included from qmake.conf.

09:47 Changeset [34481] by hausmann@webkit.org

2008-05-26 Jonathon Jongsma <jonathon.jongsma@collabora.co.uk>

Reviewed by Simon.

https://bugs.webkit.org/show_bug.cgi?id=19323

Implemented the QWebPage::editable property.

Small documentation fixes by Simon.

08:59 ApplicationsGtk edited by alp@atoker.com

Add GIMP (diff)

07:54 Changeset [34480] by ap@webkit.org

Reviewed by Darin.

https://bugs.webkit.org/show_bug.cgi?id=16503
match limit takes at least 13% of the time on the SunSpider regexp-dna test

Make the limit test slightly more efficient. It is not clear how much of a win it is,
as the improvement on regexp-dna varies from 2.3% to 0.6% depending on what revision I
apply the patch to. Today, the win on regexp-dna was minimal, but the total win was whopping
0.5%, due to random code generation changes.

pcre/pcre_exec.cpp: (match): Avoid loading a constant on each iteration.

07:22 Changeset [34479] by mjs@apple.com

2008-06-10 Maciej Stachowiak <mjs@apple.com>

Rubber stamped by Adele.

add short title and multisize icon

blog/wp-content/themes/webkit/header.php:
images/surfin-safari.icns: Added.

01:58 Changeset [34478] by hausmann@webkit.org

2008-06-09 Tor Arne Vestbø <tavestbo@trolltech.com>

Reviewed by Simon

Make sure web action in context menus have the right enablement,
while not messing up other web actions not included in the menu.

00:48 Changeset [34477] by christian@webkit.org

Apply GTK coding style to WebKit Gtk public headers

https://bugs.webkit.org/show_bug.cgi?id=16676

Reviewed by Alp Toker and Darin Adler.

06/09/08:

22:42 Changeset [34476] by alp@webkit.org

2008-06-09 Alp Toker <alp@nuanti.com>

gcc3/autotools build fix. Add explicit -O2 -fno-strict-aliasing to
each of the tools since these are no longer set globally.

21:58 Changeset [34475] by mitz@apple.com

WebCore:

Reviewed by Justin Garcia.

fix https://bugs.webkit.org/show_bug.cgi?id=19443 <rdar://problem/5994544> REGRESSION (r32531-r32652): Moving the insertion point past a newline inserted in a text area skips the next character

Tests added to editing/selection/move-left-right.html

editing/VisiblePosition.cpp: (WebCore::VisiblePosition::leftVisuallyDistinctCandidate): When falling back from visual to logical movement, restart at the original position rather than an intermediate position. (WebCore::VisiblePosition::rightVisuallyDistinctCandidate): Ditto.

LayoutTests:

Reviewed by Justin Garcia.

update test for https://bugs.webkit.org/show_bug.cgi?id=19443 <rdar://problem/5994544> REGRESSION (r32531-r32652): Moving the insertion point past a newline inserted in a text area skips the next character

editing/selection/move-left-right-expected.txt:
editing/selection/move-left-right.html:
platform/mac/editing/selection/move-left-right-expected.txt:

20:42 Changeset [34474] by mrowe@apple.com

Merge r34204.

20:41 Changeset [34473] by mrowe@apple.com

Merge r34204.

15:54 Changeset [34472] by pewtermoose@webkit.org

2008-06-07 Brent Fulgham <bfulgham@gmail.com>

Reviewed by Maciej.

Windows build fix.

platform/graphics/win/QTMovieWin.cpp: (initializeSupportedTypes):

14:02 Changeset [34471] by cwzwarich@webkit.org

2008-06-09 Cameron Zwarich <cwzwarich@uwaterloo.ca>

Reviewed by Sam.

Add an include for readline/history.h to fix the build for Darwin users
with the GNU readline library installed. Also, clean up the style of
the HAVE(READLINE) check.

kjs/testkjs.cpp: (runInteractive):

10:10 Changeset [34470] by cwzwarich@webkit.org

2008-06-09 Cameron Zwarich <cwzwarich@uwaterloo.ca>

Reviewed by Darin.

Bug 17531: Add interactive mode to testkjs
<https://bugs.webkit.org/show_bug.cgi?id=17531>

This is a cleaned up version of Sam's earlier patch to add an
interactive mode to testkjs.

Readline support is only enabled on Darwin platforms for now, but
other ports can enable it by defining HAVE_READLINE in kjs/config.h.

JavaScriptCore.xcodeproj/project.pbxproj:
kjs/config.h:
kjs/testkjs.cpp: (Options::Options): (runWithScripts): (runInteractive): (printUsageStatement): (parseArguments): (kjsmain):

07:58 Changeset [34469] by hausmann@webkit.org