Timeline


and

07/26/10:

23:04 Changeset [64108] by barraclough@apple.com

Windows build fix from Chromium/GTK build fix!

  • wtf/PageAllocation.cpp:
22:51 Changeset [64107] by barraclough@apple.com

Chromium/GTK build fix

  • wtf/PageAllocation.cpp:
22:23 Changeset [64106] by barraclough@apple.com

Build fix for !Mac platforms.

22:02 Changeset [64105] by barraclough@apple.com

Bug 43009 - Abstract out page allocation from executable allocators

Reviewed by Oliver Hunt.

JavaScriptCore:

It would be great to have a single platform abstraction for block allocation, rather than copy/paste code.

In this initial implementation I've made Symbian fall back to use malloc/free for non-executable memory.
I think this will match current behaviour for the next client we will want to port across (RegisterFile &
Collector).

(JSC::ExecutableAllocator::intializePageSize):
(JSC::ExecutablePool::systemAlloc):
(JSC::ExecutablePool::systemRelease):
(JSC::ExecutableAllocator::isValid):

  • jit/ExecutableAllocator.h:

(JSC::ExecutablePool::ExecutablePool):
(JSC::ExecutablePool::poolAllocate):

  • jit/ExecutableAllocatorFixedVMPool.cpp:

(JSC::FixedVMPoolAllocator::release):
(JSC::FixedVMPoolAllocator::reuse):
(JSC::FixedVMPoolAllocator::FixedVMPoolAllocator):
(JSC::FixedVMPoolAllocator::alloc):
(JSC::FixedVMPoolAllocator::free):
(JSC::FixedVMPoolAllocator::isValid):
(JSC::FixedVMPoolAllocator::isWithinVMPool):
(JSC::ExecutablePool::systemAlloc):
(JSC::ExecutablePool::systemRelease):

  • jit/ExecutableAllocatorPosix.cpp: Removed.
  • jit/ExecutableAllocatorSymbian.cpp: Removed.
  • jit/ExecutableAllocatorWin.cpp: Removed.
  • wscript:
  • wtf/PageAllocator.cpp: Added.

(WTF::protection):
(WTF::PageAllocation::commit):
(WTF::PageAllocation::decommit):
(WTF::PageAllocator::allocate):
(WTF::PageAllocator::reserve):
(WTF::PageAllocator::deallocate):
(WTF::PageAllocator::pagesize):

  • wtf/PageAllocator.h: Added.

(WTF::PageAllocation::PageAllocation):
(WTF::PageAllocation::base):
(WTF::PageAllocation::size):
(WTF::PageAllocation::chunk):
(WTF::PageAllocation::operator!):
(WTF::PageAllocator::):

JavaScriptGlue:

  • ForwardingHeaders/wtf/PageAllocation.h: Added.

WebCore:

  • ForwardingHeaders/wtf/PageAllocation.h: Added.
21:51 Changeset [64104] by commit-queue@webkit.org

2010-07-26 Yuta Kitamura <yutak@chromium.org>

Reviewed by Pavel Feldman.

Add WebSocket resource type to Web Inspector.

When a new WebSocket connection is established, a line for that connection
will appear in Web Inspector's Resources tab. If the resource name is
clicked, the details of handshake request and response will be shown.

Web Inspector: WebSocket in Resources tab
https://bugs.webkit.org/show_bug.cgi?id=40768

  • inspector/InspectorController.cpp: (WebCore::InspectorController::addResource): WebSocket resource does not have an associated loader, thus frame might be null. Need to check it. (WebCore::InspectorController::removeResource): Ditto. (WebCore::InspectorController::didCreateWebSocket): (WebCore::InspectorController::willSendWebSocketHandshakeRequest): (WebCore::InspectorController::didReceiveWebSocketHandshakeResponse): (WebCore::InspectorController::didCloseWebSocket):
  • inspector/InspectorController.h:
  • inspector/InspectorResource.cpp: Add null checks of m_loader and m_frame, because WebSocket does not have a loader and we need to allow null for these variables. (WebCore::createReadableStringFromBinary): (WebCore::InspectorResource::InspectorResource): (WebCore::InspectorResource::create): Factory function of regular (non-WebSocket) resources. (WebCore::InspectorResource::createWebSocket): Factory function of WebSocket resources. (WebCore::InspectorResource::updateWebSocketRequest): (WebCore::InspectorResource::updateWebSocketResponse): (WebCore::InspectorResource::updateScriptObject): m_frame->document() becomes available when Frame::setDocument() is called. We cannot obtain documentURL during the constructor or updateRequest() function, because m_frame->document() is not available yet at that point and documentURL will contain a wrong URL. As far as I know, updateScriptObject() is the only place where we can safely obtain documentURL. (WebCore::InspectorResource::cachedResource): (WebCore::InspectorResource::type): (WebCore::InspectorResource::resourceData):
  • inspector/InspectorResource.h: (WebCore::InspectorResource::): (WebCore::InspectorResource::markWebSocket):
  • inspector/front-end/Resource.js: (WebInspector.Resource.Type.toString): (WebInspector.Resource.prototype.set type): (WebInspector.Resource.prototype._mimeTypeIsConsistentWithType):
  • inspector/front-end/ResourceView.js: (WebInspector.ResourceView.prototype._refreshRequestHeaders): (WebInspector.ResourceView.prototype._refreshResponseHeaders): (WebInspector.ResourceView.prototype._refreshHeaders):
  • inspector/front-end/inspector.css: (.resources-category-websockets, .resources-category-other): (.resources-category-websockets .resources-graph-bar): (.resources-category-websockets.resource-cached .resources-graph-bar):
  • inspector/front-end/inspector.js: (WebInspector.loaded): (WebInspector.updateResource):
  • websockets/WebSocketChannel.cpp: (WebCore::WebSocketChannel::WebSocketChannel): (WebCore::WebSocketChannel::disconnect): (WebCore::WebSocketChannel::didOpen): (WebCore::WebSocketChannel::didClose): (WebCore::WebSocketChannel::processBuffer): (WebCore::WebSocketChannel::identifier):
  • websockets/WebSocketChannel.h:
21:34 Changeset [64103] by commit-queue@webkit.org

2010-07-26 Cosmin Truta <ctruta@chromium.org>

Reviewed by Eric Seidel.

Some SVGs with empty <g> elements crash Chromium on Linux
https://bugs.webkit.org/show_bug.cgi?id=41175

Remove the suppression of this bug's test on Chromium. The new fix in WebKit no longer requires
an update in Chromium, making this test safe to expose.

  • platform/chromium/test_expectations.txt:

2010-07-26 Cosmin Truta <ctruta@chromium.org>

Reviewed by Eric Seidel.

Some SVGs with empty <g> elements crash Chromium on Linux
https://bugs.webkit.org/show_bug.cgi?id=41175

Redo the old fix using a different approach, to avoid regressing
svg/batik/filters/filterRegions.svg with Chromium on Linux.
Specifically, handle empty SkBitmap objects by exiting early upon
encountering unconfigured (i.e. empty) bitmaps.

No new tests. This bug's old test (svg/filters/filter-empty-g.svg)
and the above-mentioned regressed test are expected to pass.

  • WebCore/platform/graphics/skia/ImageBufferSkia.cpp: (ImageBuffer::ImageBuffer): Undo the old fix. (getImageData): The new fix.
20:43 Changeset [64102] by simon.fraser@apple.com

2010-07-26 Simon Fraser <simon.fraser@apple.com>

Fix a couple of test results after r64095.

  • compositing/geometry/limit-layer-bounds-clipping-ancestor-expected.txt:
  • compositing/geometry/limit-layer-bounds-transformed-overflow-expected.txt:
19:53 Changeset [64101] by barraclough@apple.com

Rolling out r64097:64100, oops, more b0rked than I relized by my last changes, sorry!

(JSC::ExecutableAllocator::reprotectRegion):
(JSC::ExecutableAllocator::cacheFlush):

  • jit/ExecutableAllocator.h:

(JSC::ExecutablePool::ExecutablePool):
(JSC::ExecutablePool::poolAllocate):

  • jit/ExecutableAllocatorFixedVMPool.cpp:

(JSC::FixedVMPoolAllocator::release):
(JSC::FixedVMPoolAllocator::reuse):
(JSC::FixedVMPoolAllocator::FixedVMPoolAllocator):
(JSC::FixedVMPoolAllocator::alloc):
(JSC::FixedVMPoolAllocator::free):
(JSC::FixedVMPoolAllocator::isValid):
(JSC::FixedVMPoolAllocator::isWithinVMPool):
(JSC::ExecutablePool::systemAlloc):
(JSC::ExecutablePool::systemRelease):

  • jit/ExecutableAllocatorPosix.cpp: Added.

(JSC::ExecutableAllocator::intializePageSize):
(JSC::ExecutablePool::systemAlloc):
(JSC::ExecutablePool::systemRelease):
(JSC::ExecutableAllocator::isValid):

  • jit/ExecutableAllocatorSymbian.cpp: Added.

(JSC::ExecutableAllocator::intializePageSize):
(JSC::ExecutablePool::systemAlloc):
(JSC::ExecutablePool::systemRelease):
(JSC::ExecutableAllocator::isValid):

  • jit/ExecutableAllocatorWin.cpp: Added.

(JSC::ExecutableAllocator::intializePageSize):
(JSC::ExecutablePool::systemAlloc):
(JSC::ExecutablePool::systemRelease):
(JSC::ExecutableAllocator::isValid):

  • wscript:
  • wtf/PageAllocation.cpp: Removed.
  • wtf/PageAllocation.h: Removed.
19:39 Changeset [64100] by barraclough@apple.com

Speculative !debug build fix II.

  • wtf/PageAllocation.h:

(WTF::PageAllocation::PageAllocation):

19:37 Changeset [64099] by barraclough@apple.com

Speculative !debug build fix.

  • wtf/PageAllocation.h:

(WTF::PageAllocation::PageAllocation):

19:12 Changeset [64098] by barraclough@apple.com

Bug 43009 - Abstract out page allocation from executable allocators

Reviewed by Oliver Hunt.

It would be great to have a single platform abstraction for block allocation, rather than copy/paste code.

In this initial implementation I've made Symbian fall back to use malloc/free for non-executable memory.
I think this will match current behaviour for the next client we will want to port across (RegisterFile &
Collector).

(JSC::ExecutableAllocator::intializePageSize):
(JSC::ExecutablePool::systemAlloc):
(JSC::ExecutablePool::systemRelease):
(JSC::ExecutableAllocator::isValid):

  • jit/ExecutableAllocator.h:

(JSC::ExecutablePool::ExecutablePool):
(JSC::ExecutablePool::poolAllocate):

  • jit/ExecutableAllocatorFixedVMPool.cpp:

(JSC::FixedVMPoolAllocator::release):
(JSC::FixedVMPoolAllocator::reuse):
(JSC::FixedVMPoolAllocator::FixedVMPoolAllocator):
(JSC::FixedVMPoolAllocator::alloc):
(JSC::FixedVMPoolAllocator::free):
(JSC::FixedVMPoolAllocator::isValid):
(JSC::FixedVMPoolAllocator::isWithinVMPool):
(JSC::ExecutablePool::systemAlloc):
(JSC::ExecutablePool::systemRelease):

  • jit/ExecutableAllocatorPosix.cpp: Removed.
  • jit/ExecutableAllocatorSymbian.cpp: Removed.
  • jit/ExecutableAllocatorWin.cpp: Removed.
  • wscript:
  • wtf/PageAllocator.cpp: Added.

(WTF::protection):
(WTF::PageAllocation::commit):
(WTF::PageAllocation::decommit):
(WTF::PageAllocator::allocate):
(WTF::PageAllocator::reserve):
(WTF::PageAllocator::deallocate):
(WTF::PageAllocator::pagesize):

  • wtf/PageAllocator.h: Added.

(WTF::PageAllocation::PageAllocation):
(WTF::PageAllocation::base):
(WTF::PageAllocation::size):
(WTF::PageAllocation::chunk):
(WTF::PageAllocation::operator!):
(WTF::PageAllocator::):

19:10 Changeset [64097] by commit-queue@webkit.org

2010-07-26 Rafael Antognolli <antognolli@profusion.mobi>

Reviewed by Antonio Gomes.

[EFL] Move CMakeListsEfl.txt to WebKit/efl/
https://bugs.webkit.org/show_bug.cgi?id=43002

This should reduce the noise in WebKit/ChangeLog that comes from EFL
port. Most of the changes on EFL port should be logged in
WebKit/efl/ChangeLog.

  • CMakeLists.txt: Just change the path from where to include the file.

2010-07-26 Rafael Antognolli <antognolli@profusion.mobi>

Reviewed by Antonio Gomes.

[EFL] Move CMakeListsEfl.txt to WebKit/efl/
https://bugs.webkit.org/show_bug.cgi?id=43002

This should reduce the noise in WebKit/ChangeLog that comes from EFL
port. Most of the changes on EFL port should be logged in
WebKit/efl/ChangeLog.

  • CMakeListsEfl.txt: Renamed from WebKit/CMakeListsEfl.txt.
18:17 Changeset [64096] by darin@apple.com

WebKitTestRunner needs to support layoutTestController.counterValueForElementById
https://bugs.webkit.org/show_bug.cgi?id=42537

Reviewed by Sam Weinig.

WebKitTestRunner needs layoutTestController.markerTextForListItem
https://bugs.webkit.org/show_bug.cgi?id=42549

WebKit2:

  • UIProcess/API/C/WKString.cpp:

(WKStringIsEmpty): Added.

  • UIProcess/API/C/WKString.h: Added WKStringIsEmpty.
  • WebProcess/InjectedBundle/API/c/WKBundleFrame.cpp:

(copiedString): Added. Helper for functions that return a
WKStringRef that follows the copy rule.
(WKBundleFrameCopyName): Use copiedString.
(WKBundleFrameCopyCounterValue): Added. Calls counterValue.
(WKBundleFrameCopyMarkerText): Added. Calls markerText.

  • WebProcess/InjectedBundle/API/c/WKBundleFramePrivate.h:

Added WKBundleFrameCopyCounterValue and WKBundleFrameCopyMarkerText.

  • WebProcess/WebPage/WebFrame.cpp:

(WebKit::WebFrame::computedStyleIncludingVisitedInfo):
Renamed argument to streamline function a bit.
(WebKit::WebFrame::counterValue): Added.
(WebKit::WebFrame::markerText): Added.

  • WebProcess/WebPage/WebFrame.h: Added counterValue and markerText.

WebKitTools:

  • WebKitTestRunner/InjectedBundle/Bindings/CodeGeneratorTestRunner.pm:

Fixed _paramterExpression typo. Improved support for string type as a
return value, so we don't try to include DOMString.h and we can convert
the string to a JSValue by calling JSValueMakeStringOrNull.

  • WebKitTestRunner/InjectedBundle/Bindings/JSWrappable.h:

Added JSValueMakeStringOrNull, used by the code generator.

  • WebKitTestRunner/InjectedBundle/Bindings/LayoutTestController.idl:

Added counterValueForElementById and markerTextForListItem.

  • WebKitTestRunner/InjectedBundle/LayoutTestController.cpp:

Moved constant to the top of the file.
(WTR::toCF): Added. Converts strings to CFStringRef.
(WTR::toWK): Added. Converts strings to WKStringRef.
(WTR::toJS): Added. Converts strings to JSStringRef.
(WTR::setProperty): Moved this function to the top of the file
rather than having it down where it's used inside the class. Also
renamed it so it's a separate function rather than an overload of
JSObjectSetProperty.
(WTR::propertyValue): Copied this here from InjectedBundlePage.cpp.
Should move it somewhere we can share it.
(WTR::propertyObject): Ditto.
(WTR::getElementById): Added. Calls getElementById through the magic
of JavaScript.
(WTR::LayoutTestController::pauseAnimationAtTimeOnElementWithId):
Chagned to use toWK instead of four local variables.
(WTR::LayoutTestController::counterValueForElementById): Added.
Calls WKBundleFrameCopyCounterValue.
(WTR::LayoutTestController::markerTextForListItem): Added.
Calls WKBundleFrameCopyMarkerText.

  • WebKitTestRunner/InjectedBundle/LayoutTestController.h:

Added counterValueForElementById and markerTextForListItem.

LayoutTests:

  • platform/mac-wk2/Skipped: Removed tests that were failing due to one of the above.
17:21 Changeset [64095] by simon.fraser@apple.com

2010-07-26 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dan Bernstein.

Google image search results broken on pages with Flash
https://bugs.webkit.org/show_bug.cgi?id=43014

The test for overflow added in r63452 is wrong in that it ignores the effects
of positioning on overflow.

The correct approach is to start by using RenderView's layoutOverflowRect as the largest bounds,
then getting the oveflow rect via backgroundClipRect() relative to the root layer.

Test: compositing/geometry/limit-layer-bounds-overflow-root.html

  • rendering/RenderLayerBacking.cpp: (WebCore::layerOrAncestorIsTransformed): (WebCore::RenderLayerBacking::updateCompositedBounds):
17:19 Changeset [64094] by crogers@google.com

Get rid of AudioMixer and AudioMixerNode in derived sources

17:09 Changeset [64093] by andersca@apple.com

Plug-in views should be able to return JSObjects directly
https://bugs.webkit.org/show_bug.cgi?id=43019

Reviewed by Sam Weinig.

  • bindings/js/JSHTMLAppletElementCustom.cpp:

(WebCore::JSHTMLAppletElement::putDelegate):
(WebCore::JSHTMLAppletElement::getCallData):

  • bindings/js/JSHTMLEmbedElementCustom.cpp:

(WebCore::JSHTMLEmbedElement::putDelegate):
(WebCore::JSHTMLEmbedElement::getCallData):

  • bindings/js/JSHTMLObjectElementCustom.cpp:

(WebCore::JSHTMLObjectElement::putDelegate):
(WebCore::JSHTMLObjectElement::getCallData):
runtimeObjectCustomPut and runtimeObjectGetCallData now take a JSHTMLElement instead of a HTMLElement.

  • bindings/js/JSPluginElementFunctions.cpp:

(WebCore::pluginScriptObject):
First, ask the PluginViewBase for a JSObject and fall back to getting the JSObject from the instance.

(WebCore::runtimeObjectPropertyGetter):
(WebCore::runtimeObjectCustomGetOwnPropertySlot):
(WebCore::runtimeObjectCustomGetOwnPropertyDescriptor):
Call pluginScriptObject.

(WebCore::runtimeObjectCustomPut):
Change to take a JSHTMLElement. Call pluginScriptObject.

(WebCore::runtimeObjectGetCallData):
Change to take a JSHTMLElement.

  • bridge/jsc/BridgeJSC.cpp:
  • bridge/jsc/BridgeJSC.h:

(JSC::Bindings::Instance::createRuntimeObject):
Change this to return a JSObject.

  • html/HTMLPlugInElement.cpp:

(WebCore::HTMLPlugInElement::getInstance):
Call pluginWidget.

(WebCore::HTMLPlugInElement::pluginWidget):
Return the plug-in widget for this element.

  • html/HTMLPlugInElement.h:
  • plugins/PluginViewBase.h:

(WebCore::PluginViewBase::scriptObject):
Add default implementation of scriptObject.

17:01 Changeset [64092] by commit-queue@webkit.org

2010-07-26 Dominic Mazzoni <dmazzoni@google.com>

Reviewed by Chris Fleizach.

Add accessor to get the doctype of a WebDocument, needed to
implement Windows accessibility.

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

  • WebKit.gyp:
  • public/WebDocument.h:
  • public/WebDocumentType.h: Added. (WebKit::WebDocumentType::WebDocumentType): (WebKit::WebDocumentType::operator=): (WebKit::WebDocumentType::assign):
  • src/WebDocument.cpp: (WebKit::WebDocument::doctype):
  • src/WebDocumentType.cpp: Added. (WebKit::WebDocumentType::name): (WebKit::WebDocumentType::WebDocumentType): (WebKit::WebDocumentType::operator=): (WebKit::WebDocumentType::operator PassRefPtr<DocumentType>):
17:00 Changeset [64091] by weinig@apple.com

Part 2 of https://bugs.webkit.org/show_bug.cgi?id=43013
<rdar://problem/8152434>
Add support for scrolling using the keyboard in WebKit2

Reviewed by Anders Carlsson.

Add support for scrolling with the space bar and ensure that we don't scroll
if WebCore is handling the event in another way.

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::keyEvent):
(WebKit::getScrollMapping):
(WebKit::WebPage::performDefaultBehaviorForKeyEvent):

  • WebProcess/WebPage/WebPage.h:
16:54 Changeset [64090] by crogers@google.com

Get rid of explicit ownership (part 2)

  • now that it is unused/ignored, completely get rid of optional "owner" argument in AudioContext create methods
16:54 Changeset [64089] by crogers@google.com

Remove obsolete AudioMixerNode and AudioMixerInputNode classes

  • they have been replaced with AudioGainNode and unity-gain summing junctions in AudioNodeInput
16:33 Changeset [64088] by cfleizach@apple.com

AX: Safari does not support ARIA mixed checkboxes
https://bugs.webkit.org/show_bug.cgi?id=43016

Reviewed by Darin Adler.

WebCore:

Test: platform/mac/accessibility/mixed-checkbox.html

  • accessibility/AccessibilityObject.cpp:

(WebCore::AccessibilityObject::checkboxOrRadioValue):

  • accessibility/AccessibilityObject.h:
  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::isNativeCheckboxOrRadio):
(WebCore::AccessibilityRenderObject::checkboxOrRadioValue):

  • accessibility/AccessibilityRenderObject.h:
  • accessibility/mac/AccessibilityObjectWrapper.mm:

(-[AccessibilityObjectWrapper accessibilityAttributeValue:]):

LayoutTests:

  • platform/mac/accessibility/mixed-checkbox-expected.txt: Added.
  • platform/mac/accessibility/mixed-checkbox.html: Added.
16:11 Changeset [64087] by commit-queue@webkit.org

2010-07-26 Shimeng (Simon) Wang <swang@google.com>

Reviewed by Darin Adler.

Add android specific expected test results for auto hyphenation
tests.

  • platform/android: Added.
  • platform/android/fast: Added.
  • platform/android/fast/text: Added.
  • platform/android/fast/text/hyphenate-character-expected.txt: Added.
  • platform/android/fast/text/hyphens-expected.txt: Added.

2010-07-26 Shimeng (Simon) Wang <swang@google.com>

Reviewed by Darin Adler.

Add auto hyphenation support for Android.
https://bugs.webkit.org/show_bug.cgi?id=42800

  • platform/text/android/HyphenationAndroid.cpp: Added. (WebCore::loadHyphenationDictionary): (WebCore::lastHyphenLocation):
16:06 Changeset [64086] by weinig@apple.com

Patch for https://bugs.webkit.org/show_bug.cgi?id=43013
Part of <rdar://problem/8152434>
Add support for scrolling using the keyboard in WebKit2

Reviewed by Anders Carlsson.

WebCore:

Add a new export and make WindowsKeyboardCodes private.

  • WebCore.exp.in:
  • WebCore.xcodeproj/project.pbxproj:

WebKit2:

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::getScrollMapping):
(WebKit::WebPage::keyEvent):
Scroll the page in response to keyDown.

  • WebProcess/WebPage/WebPage.h:
  • WebProcess/WebPage/mac/WebPageMac.mm:

Use WindowsKeyboardCodes.h instead of redefining the constants.

15:58 Changeset [64085] by darin@apple.com

2010-07-26 Darin Adler <darin@apple.com>

Reviewed by Eric Carlson.

Don't handle empty URL attributes specially in reflection, even in cases such as <img src="">
https://bugs.webkit.org/show_bug.cgi?id=42087

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::insertedIntoDocument): Check if the src attribute is empty by calling getAttribute instead of calling the src function. This keeps the behavior correct with the reflection change, and has a side benefit of being more efficient as well since there is no reason to resolve the URL. (WebCore::HTMLMediaElement::loadResource): Call getNonEmptyURLAttribute directly here instead of using the poster function. (WebCore::HTMLMediaElement::getPluginProxyParams): Ditto.
  • html/HTMLMediaElement.h: Removed the poster virtual function. The code that gets the poster attribute is already video-specific and uses generic attribute fetching code, so we no longer need a virtual function. Removed the src function. Since this would give a different result from the JavaScript reflected src attribute, it's safer not to have it, and it was easy to convert all the callers.
  • html/HTMLMediaElement.idl: Removed the NonEmpty extended attribute from the src attribute.
  • html/HTMLSourceElement.cpp: Removed the src function for the same reason as above.
  • html/HTMLSourceElement.h: Ditto.
  • html/HTMLSourceElement.idl: Removed the NonEmpty extended attribute from the src attribute.
  • html/HTMLVideoElement.cpp: (WebCore::HTMLVideoElement::parseMappedAttribute): Removed code to update m_posterURL, which is no longer needed. Changed call site that used the poster() function to instead use getNonEmptyURLAttribute. (WebCore::HTMLVideoElement::updatePosterImage): Check if the poster attribute is empty by calling getAttribute instead of calling the poster function. This keeps the behavior correct with the reflection change, and has a side benefit of being more efficient as well since there is no reason to resolve the URL.
  • html/HTMLVideoElement.h: Removed the poster function for the same reason we removed the src function above. Also removed now-unneeded m_posterURL data member.
  • html/HTMLVideoElement.idl: Removed the NonEmpty extended attribute from the poster attribute.

2010-07-26 Darin Adler <darin@apple.com>

Reviewed by Eric Carlson.

Don’t handle empty URL attributes specially in reflection, even in cases such as <img src="">
https://bugs.webkit.org/show_bug.cgi?id=42087

  • fast/dom/URL-attribute-reflection-expected.txt: Updated.
  • fast/dom/script-tests/URL-attribute-reflection.js: Updated test expectations so that even for URLs where "" is invalid, we reflect the empty string resolved to the base URL.
  • media/video-poster-expected.txt: Updated.
  • media/video-poster.html: Updated to expect poster="" to reflect as the base URL.
  • media/video-source-error-no-candidate-expected.txt: Updated.
  • media/video-src-none-expected.txt: Updated.
  • media/video-src-none.html: Updated to expect src="" to reflect as the base URL.
15:49 Changeset [64084] by cfleizach@apple.com

AX: remove intValue() hasIntValue() from AccessibilityObject
https://bugs.webkit.org/show_bug.cgi?id=43006

Reviewed by Darin Adler.

These methods are very Mac centric. The mac centric code should be moved into the Mac wrapper.

No new tests, existing tests cover changes to logic.

  • accessibility/AccessibilityObject.cpp:

(WebCore::AccessibilityObject::checkboxOrRadioValue):

  • accessibility/AccessibilityObject.h:

(WebCore::):

  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::selectedRadioButton):
(WebCore::AccessibilityRenderObject::selectedTabItem):
(WebCore::AccessibilityRenderObject::checkboxOrRadioValue):

  • accessibility/AccessibilityRenderObject.h:
  • accessibility/mac/AccessibilityObjectWrapper.mm:

(-[AccessibilityObjectWrapper accessibilityAttributeValue:]):

15:47 Changeset [64083] by rniwa@webkit.org

Applying inline style to a text node whose parent is an inline editable root causes crash
https://bugs.webkit.org/show_bug.cgi?id=39989

Reviewed by Darin Adler.

WebCore:

The crash was caused by splitTextElementAtStart and splitTextElementAtEnd assuming that the parent
and the grandparent of the specified text node is editable.

Modified splitTextElementAtStart and splitTextElementAtEnd so that they call splitTextAtStart
and splitTextAtEnd respectively when the grandparent is not editable.

Also modified SplitTextNodeContainingElement to exit early if the grandparent of m_text is not editable.

Test: editing/style/style-text-node-without-editable-parent.html

  • editing/ApplyStyleCommand.cpp:

(WebCore::ApplyStyleCommand::splitTextElementAtStart):
(WebCore::ApplyStyleCommand::splitTextElementAtEnd):

  • editing/SplitTextNodeContainingElementCommand.cpp:

(WebCore::SplitTextNodeContainingElementCommand::doApply):

LayoutTests:

Added a test to apply inline styles to a text node which is a editable root's child.
The test should not crash.

Two tests require rebaseline for the editing delegates. However, the final selection is kept same.

  • editing/execCommand/hilitecolor-expected.txt:
  • editing/style/remove-underline-from-stylesheet-expected.txt:
  • editing/style/style-text-node-without-editable-parent-expected.txt: Added.
  • editing/style/style-text-node-without-editable-parent.html: Added.
  • resources/dump-as-markup.js:
15:34 Changeset [64082] by mrobinson@webkit.org

2010-07-26 Martin Robinson <mrobinson@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GTK] The EventSender should support simulating drop events
https://bugs.webkit.org/show_bug.cgi?id=39844

Unskip tests that are now passing and add baselines for new tests. Re-organize
remaining failures so they can be unskipped more easily in the future.

  • platform/gtk/Skipped:
  • platform/gtk/editing/pasteboard/dataTransfer-setData-getData-expected.txt: Added.
  • platform/gtk/editing/pasteboard/drag-drop-dead-frame-expected.txt: Added.
  • platform/gtk/editing/pasteboard/drag-drop-modifies-page-expected.txt: Added.
  • platform/gtk/editing/pasteboard/drop-text-without-selection-expected.txt: Added.
  • platform/gtk/editing/pasteboard/subframe-dragndrop-1-expected.txt: Added.
  • platform/gtk/editing/selection/drag-select-1-expected.txt: Added.
  • platform/gtk/editing/selection/drag-text-delay-expected.txt: Added.
  • platform/gtk/fast/events/bogus-dropEffect-effectAllowed-expected.txt: Removed.
  • platform/gtk/fast/forms/input-text-drag-down-expected.txt: Added.
  • platform/gtk/fast/lists/drag-into-marker-expected.txt: Added.

2010-07-26 Martin Robinson <mrobinson@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GTK] The EventSender should support simulating drop events
https://bugs.webkit.org/show_bug.cgi?id=39844

Add initial dropping support to the GTK+ EventSender.

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (createWebView): Listen for some new drag-and-drop signals.
  • DumpRenderTree/gtk/EventSender.cpp: Add currentDragSourceContext. (dispatchEvent): Detect situations where a drag is either starting or ending and massage GTK+ into sending the appropriate signals to the WebView. (replaySavedEvents): Remove comment. (makeEventSender): Clear the currentDragSourceContext when creating a new EventSender. (dragBeginCallback): Capture the new currentDragSourceContext when a drag begins. (dragEndCallback): Clear the currentDragSourceContext when a drag ends. (dragFailedCallback): Disable the GTK+ drag failed animation.
  • DumpRenderTree/gtk/EventSender.h: Add declarations for new signal callbacks.
15:23 Changeset [64081] by victorw@chromium.org

2010-07-26 Victor Wang <victorw@chromium.org>

Unreviewed. Update chromium test expectations for flaky test on linux:
tables/mozilla_expected_failures/bugs/bug89315.html

  • platform/chromium/test_expectations.txt:
15:11 Changeset [64080] by simon.fraser@apple.com

2010-07-26 Simon Fraser <simon.fraser@apple.com>

Fix tests on Leopard.

No need for these Leopard-specific results any more.

  • platform/mac-leopard/compositing/iframes/connect-compositing-iframe-delayed-expected.txt: Removed.
  • platform/mac-leopard/compositing/iframes/connect-compositing-iframe3-expected.txt: Removed.
  • platform/mac-leopard/compositing/iframes/iframe-resize-expected.txt: Removed.
  • platform/mac-leopard/compositing/iframes/iframe-size-from-zero-expected.txt: Removed.
  • platform/mac-leopard/compositing/iframes/overlapped-iframe-expected.txt: Removed.
  • platform/mac-leopard/compositing/iframes/scrolling-iframe-expected.txt: Removed.
15:03 Changeset [64079] by adachan@apple.com

Fix Windows build.

  • win/WebKit2Generated.make:
14:41 Changeset [64078] by simon.fraser@apple.com

2010-07-26 Simon Fraser <simon.fraser@apple.com>

Reviewed by Anders Carlsson.

Gmail compose is upside-down after r64054
https://bugs.webkit.org/show_bug.cgi?id=43000

We need to call setGeometryOrientation() whenever the iframe compositing layer attachment changes,
not just when requiresScrollLayer() returns false. This ensures that the orientation is updated
correctly.

Test: compositing/iframes/iframe-content-flipping.html

  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::ensureRootPlatformLayer):
14:36 Changeset [64077] by jschuh@chromium.org

2010-07-26 Justin Schuh <jschuh@chromium.org>

Reviewed by Darin Fisher.

Check history state against origin before setting
https://bugs.webkit.org/show_bug.cgi?id=42858

Tests: fast/loader/stateobjects/replacestate-base-illegal.html

fast/loader/stateobjects/replacestate-base-legal.html

  • page/History.cpp: (WebCore::History::urlForState): (WebCore::History::stateObjectAdded):

2010-07-26 Justin Schuh <jschuh@chromium.org>

Reviewed by Darin Fisher.

Check history state when base URL is changed
https://bugs.webkit.org/show_bug.cgi?id=42858

  • fast/loader/stateobjects/replacestate-base-illegal-expected.txt: Added.
  • fast/loader/stateobjects/replacestate-base-illegal.html: Added.
  • fast/loader/stateobjects/replacestate-base-legal-expected.txt: Added.
  • fast/loader/stateobjects/replacestate-base-legal.html: Added.
  • fast/loader/stateobjects/resources/replacestate-base-pass.html: Added.
14:33 Changeset [64076] by andersca@apple.com

Fix Windows build.

  • WebProcess/Plugins/Netscape/NetscapePlugin.h:

(WebKit::NetscapePlugin::platformHandleMouseEnterEvent):
(WebKit::NetscapePlugin::platformHandleMouseLeaveEvent):

14:32 Changeset [64075] by mrobinson@webkit.org

2010-07-26 Martin Robinson <mrobinson@igalia.com>

Reviewed by Xan Lopez.

[GTK] Get rid of libgdom
https://bugs.webkit.org/show_bug.cgi?id=42378

Remove the need to build the libgdom intermediate library.

  • GNUmakefile.am: Move the source list for the GObject DOM bindings to the top-level GNUmakefile.am, since they are logically part of WebKit, instead of WebCore. List all auto-generated files statically and remove no-longer- used variables. Fix an issue where some places expected webkitenumtypes.cpp to be generated in different directories.

2010-07-26 Martin Robinson <mrobinson@igalia.com>

Reviewed by Xan Lopez.

[GTK] Get rid of libgdom
https://bugs.webkit.org/show_bug.cgi?id=42378

Remove all build steps for building GObject DOM bindings.

  • GNUmakefile.am: Remove GObject DOM bindings build.
14:28 WikiStart edited by rniwa@webkit.org
Added a link "Writing DumpAsMarkup Tests" page to "WikiStart" (diff)
14:26 Writing DumpAsMarkup Tests created by rniwa@webkit.org
Added "Writing DumpAsMarkup Tests" page
14:20 Changeset [64074] by mrowe@apple.com

Disable a test that was added to verify the vendor prefix change that I rolled out in r64071.
It can be reenabled when the vendor prefix change is reintroduced in a more limited fashion.

  • fast/css/limited-vendor-prefix-behavior.html-disabled: Renamed from LayoutTests/fast/css/limited-vendor-prefix-behavior.html.
14:15 Changeset [64073] by andersca@apple.com

Handle mouseenter/mouseleave + focus events
https://bugs.webkit.org/show_bug.cgi?id=42997

Reviewed by Sam Weinig.

  • WebProcess/Plugins/Netscape/NetscapePlugin.cpp:

(WebKit::NetscapePlugin::handleMouseEnterEvent):
Call platformHandleMouseEnterEvent.

(WebKit::NetscapePlugin::handleMouseLeaveEvent):
Call platformHandleMouseExitEvent.

(WebKit::NetscapePlugin::setFocus):
Call platformSetFocus.

  • WebProcess/Plugins/Netscape/mac/NetscapePluginMac.mm:

(WebKit::fillInCocoaEventFromMouseEvent):
Factor out code that fills in an NPCocoaEvent from a WebMouseEvent.

(WebKit::initializeMouseEvent):
Call fillInCocoaEventFromMouseEvent.

(WebKit::NetscapePlugin::platformHandleMouseEnterEvent):
Send an NPCocoaEventMouseEntered event.

(WebKit::NetscapePlugin::platformHandleMouseLeaveEvent):
Send an NPCocoaEventMouseExited event.

(WebKit::NetscapePlugin::platformSetFocus):
Send an NPCocoaEventFocusChanged.

  • WebProcess/Plugins/Plugin.h:
  • WebProcess/Plugins/PluginView.cpp:

(WebKit::PluginView::handleEvent):
Dispatch mouse enter and mouse leave events. Call focusPluginElement on mousedown.

(WebKit::PluginView::focusPluginElement):
Focus the plug-in's containing frame and its element.

(WebKit::PluginView::setFocus):
Call Plugin::setFocus.

13:58 Changeset [64072] by dbates@webkit.org

2010-07-26 Daniel Bates <dbates@rim.com>

Reviewed by Darin Adler.

svn-unapply warns of uninitialized variable when unapplying
a patch that describes an svn move operation
https://bugs.webkit.org/show_bug.cgi?id=42036

Fixes Perl uninitialized variable warnings when un-applying
a patch that moves a file.

  • Scripts/svn-unapply:
    • Modified patch() so that it initializes $patch to the empty string when we don't have svnConvertedText (such as when reversing a diff that represents a svn copy/move operation).
13:53 Changeset [64071] by mrowe@apple.com

<http://webkit.org/b/42990> REGRESSION (r63854): Safari RSS layout is broken due to changes in vendor prefix handling

Reviewed by Dan Bernstein.

In r63854 support for properties with the -khtml vendor prefix was removed. This breaks the layout of
Safari's RSS interface, several Dashboard widgets that are included with Mac OS X, and at least one
popular third-party Mac OS X application. This change will need to be revisited in a manner that has
a clearer backwards-compatibility strategy. <http://webkit.org/b/42093> will be reopened to track that.

  • css/CSSParser.cpp:

(WebCore::cssPropertyID): Revert the WebCore change from r63854.

13:37 Changeset [64070] by beidson@apple.com

2010-07-26 Brady Eidson <beidson@apple.com>

Reviewed by Sam Weinig.

Add ability for WK2 to register a scheme to always be displayed as an empty document
https://bugs.webkit.org/show_bug.cgi?id=42995

No new tests. (Currently not possible to test in WebKit1 DRT, can be testing in future WK2 tester)

  • WebCore.exp.in:


  • loader/MainResourceLoader.cpp: (WebCore::shouldLoadAsEmptyDocument):


  • platform/SchemeRegistry.cpp: (WebCore::emptyDocumentSchemes): (WebCore::SchemeRegistry::registerURLSchemeAsEmptyDocument): (WebCore::SchemeRegistry::shouldLoadURLSchemeAsEmptyDocument):
  • platform/SchemeRegistry.h:

2010-07-26 Brady Eidson <beidson@apple.com>

Reviewed by Sam Weinig.

Add ability for WK2 to register a scheme to always be displayed as an empty document
https://bugs.webkit.org/show_bug.cgi?id=42995

  • Shared/CoreIPCSupport/WebProcessMessageKinds.h: (WebProcessMessage::):


  • UIProcess/API/C/WKContext.cpp: (_WKContextRegisterURLSchemeAsEmptyDocument):
  • UIProcess/API/C/WKContextPrivate.h:


  • UIProcess/WebContext.cpp: (WebKit::WebContext::registerURLSchemeAsEmptyDocument):
  • UIProcess/WebContext.h:


  • WebProcess/WebProcess.cpp: (WebKit::WebProcess::registerURLSchemeAsEmptyDocument): (WebKit::WebProcess::didReceiveMessage):
  • WebProcess/WebProcess.h:
13:16 Changeset [64069] by victorw@chromium.org

2010-07-26 Simon Fraser <simon.fraser@apple.com>

Fix compositing iframe tests on Leopard.

Add Leopard-specific results where geometry flipping causes a different test output.

  • platform/mac-leopard/compositing/iframes/connect-compositing-iframe-delayed-expected.txt: Added.
  • platform/mac-leopard/compositing/iframes/connect-compositing-iframe3-expected.txt: Added.
  • platform/mac-leopard/compositing/iframes/iframe-resize-expected.txt: Added.
  • platform/mac-leopard/compositing/iframes/iframe-size-from-zero-expected.txt: Added.
  • platform/mac-leopard/compositing/iframes/overlapped-iframe-expected.txt: Added.
  • platform/mac-leopard/compositing/iframes/scrolling-iframe-expected.txt: Added.
13:16 Changeset [64068] by andersca@apple.com

Get rid of the dummy plug-in.

Reviewed by Adam Roben.

  • WebKit2.xcodeproj/project.pbxproj:
  • WebProcess/Plugins/DummyPlugin.cpp: Removed.
  • WebProcess/Plugins/DummyPlugin.h: Removed.
  • win/WebKit2.vcproj:
13:09 Changeset [64067] by weinig@apple.com

Move some non-APIish functions into private headers.

Reviewed by Anders Carlsson.

WebKit2:

  • WebKit2.xcodeproj/project.pbxproj:
  • WebProcess/InjectedBundle/API/c/WKBundle.cpp:
  • WebProcess/InjectedBundle/API/c/WKBundle.h:
  • WebProcess/InjectedBundle/API/c/WKBundleFrame.cpp:
  • WebProcess/InjectedBundle/API/c/WKBundleFrame.h:
  • WebProcess/InjectedBundle/API/c/WKBundleFramePrivate.h: Copied from WebProcess/InjectedBundle/API/c/WKBundleFrame.h.
  • WebProcess/InjectedBundle/API/c/WKBundlePrivate.h: Copied from WebProcess/InjectedBundle/API/c/WKBundle.h.
  • win/WebKit2.vcproj:

WebKitTools:

  • WebKitTestRunner/InjectedBundle/InjectedBundle.cpp:
  • WebKitTestRunner/InjectedBundle/LayoutTestController.cpp:
12:57 Changeset [64066] by simon.fraser@apple.com

2010-07-26 Simon Fraser <simon.fraser@apple.com>

Fix compositing iframe tests on Leopard.

Add Leopard-specific results where geometry flipping causes a different test output.

  • platform/mac-leopard/compositing/iframes/connect-compositing-iframe-delayed-expected.txt: Added.
  • platform/mac-leopard/compositing/iframes/connect-compositing-iframe3-expected.txt: Added.
  • platform/mac-leopard/compositing/iframes/iframe-resize-expected.txt: Added.
  • platform/mac-leopard/compositing/iframes/iframe-size-from-zero-expected.txt: Added.
  • platform/mac-leopard/compositing/iframes/overlapped-iframe-expected.txt: Added.
  • platform/mac-leopard/compositing/iframes/scrolling-iframe-expected.txt: Added.
12:57 Changeset [64065] by weinig@apple.com

Fix for https://bugs.webkit.org/show_bug.cgi?id=42986
Add prompt and confirm client functions to WebKit2

Reviewed by Anders Carlsson.

WebKit2:

  • Shared/CoreIPCSupport/WebPageProxyMessageKinds.h:

(WebPageProxyMessage::):

  • UIProcess/API/C/WKPage.h:
  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::didReceiveSyncMessage):
(WebKit::WebPageProxy::runJavaScriptAlert):
(WebKit::WebPageProxy::runJavaScriptConfirm):
(WebKit::WebPageProxy::runJavaScriptPrompt):

  • UIProcess/WebPageProxy.h:
  • UIProcess/WebUIClient.cpp:

(WebKit::WebUIClient::runJavaScriptAlert):
(WebKit::WebUIClient::runJavaScriptConfirm):
(WebKit::WebUIClient::runJavaScriptPrompt):

  • UIProcess/WebUIClient.h:
  • WebKit2.xcodeproj/project.pbxproj:
  • WebProcess/WebCoreSupport/WebChromeClient.cpp:

(WebKit::WebChromeClient::runJavaScriptConfirm):
(WebKit::WebChromeClient::runJavaScriptPrompt):

WebKitTools:

  • MiniBrowser/mac/BrowserWindowController.m:

(-[BrowserWindowController awakeFromNib]):

12:48 Changeset [64064] by crogers@google.com

Get rid of explicit ownership (part 1)

  • implement the complex type of ref-counting necessary to deal with nodes which keep playing without JS references
  • AudioNodeInput now keeps track of "active" connections, as well as disabled connections
  • still need to address some thread safety issues, but code generally runs well
  • still need to get rid of "owner" argument (since it's now totally ignored)
12:41 Changeset [64063] by andersca@apple.com

Deliver mouse and wheel events to plug-ins
https://bugs.webkit.org/show_bug.cgi?id=42988

Reviewed by Sam Weinig.

  • Shared/WebEvent.h:

(WebKit::WebMouseEvent::WebMouseEvent):
(WebKit::WebMouseEvent::deltaX):
(WebKit::WebMouseEvent::deltaY):
(WebKit::WebMouseEvent::deltaZ):

  • Shared/mac/WebEventFactory.mm:

(WebKit::WebEventFactory::createWebMouseEvent):
Add deltaX, deltaY and deltaZ member variables to WebMosueEvent.

  • WebKit2.xcodeproj/project.pbxproj:

Rename NetscapePluginMac.cpp to NetscapePluginMac.mm.

  • WebProcess/Plugins/Netscape/NetscapePlugin.cpp:

(WebKit::NetscapePlugin::NPP_HandleEvent):
Add NPP_ wrapper.

(WebKit::NetscapePlugin::handleMouseEvent):
(WebKit::NetscapePlugin::handleWheelEvent):
Call the platform variants.

  • WebProcess/Plugins/Netscape/mac/NetscapePluginMac.mm: Added.

(WebKit::modifierFlags):
(WebKit::buttonNumber):
(WebKit::initializeMouseEvent):
(WebKit::NetscapePlugin::platformHandleMouseEvent):
Create an NPCocoaEvent and send it to the plug-in.

(WebKit::NetscapePlugin::platformHandleWheelEvent):
Ditto.

  • WebProcess/Plugins/PluginView.cpp:

(WebKit::PluginView::handleEvent):
Get the current WebEvent and send it to the plug-in if necessary.

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::currentEvent):
(WebKit::CurrentEvent::CurrentEvent):
(WebKit::CurrentEvent::~CurrentEvent):
(WebKit::WebPage::mouseEvent):
(WebKit::WebPage::wheelEvent):
(WebKit::WebPage::keyEvent):

  • WebProcess/WebPage/WebPage.h:

Add RAII object for keeping track of the current event.

12:30 Changeset [64062] by commit-queue@webkit.org

2010-07-26 Patrick Gansterer <paroga@paroga.com>

Reviewed by Kent Tamura.

[WINCE] Port ClipboardUtilities to WinCE
https://bugs.webkit.org/show_bug.cgi?id=42929

Make ClipboardUtilitiesWin more portable and remove the global namespace prefix
from GlobalLock/GlobalUnlock, since they are only macros on WinCE.

  • platform/win/ClipboardUtilitiesWin.cpp: (WebCore::urlFromPath): (WebCore::getWebLocData): (WebCore::createGlobalData): (WebCore::getURL): (WebCore::getPlainText): (WebCore::fragmentFromHTML):
12:22 Changeset [64061] by victorw@chromium.org

2010-07-26 Victor Wang <victorw@chromium.org>

Unreviewed. Update chromium test expectations fro flaky test.

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

  • platform/chromium/test_expectations.txt:
12:10 Changeset [64060] by commit-queue@webkit.org

2010-07-26 Patrick Gansterer <paroga@paroga.com>

Reviewed by Adam Roben.

[WINCE] Buildfix for BitmapImage.h
https://bugs.webkit.org/show_bug.cgi?id=42931

Use AffineTransform insted of TransformationMatrix.

  • platform/graphics/BitmapImage.h:
11:57 Changeset [64059] by aroben@apple.com

Windows build fix

WebKit2:

  • win/WebKit2.vcproj:
  • win/WebKit2WebProcess.vcproj:

Removed empty attributes so the values will be picked up from the
vsprops files. (Also removed a couple of attributes that accidentally
duplicated values from the vsprops files.)

WebKitTools:

  • MiniBrowser/MiniBrowser.vcproj:
  • WebKitTestRunner/InjectedBundle/win/InjectedBundle.vcproj:

Removed empty attributes so the values will be picked up from the
vsprops files. (Also removed a couple of attributes that accidentally
duplicated values from the vsprops files.)

11:41 Changeset [64058] by krit@webkit.org

2010-07-26 Dirk Schulze <krit@webkit.org>

Unreviewed sort of XCode project file.

  • WebCore.xcodeproj/project.pbxproj:
11:32 Changeset [64057] by ddkilzer@apple.com

<http://webkit.org/b/42982> Streamline Inspector Source build phase script should use CONFIGURATION not BUILD_STYLE
<rdar://problem/6341764>

Reviewed by Mark Rowe.

  • WebCore.xcodeproj/project.pbxproj: Switched Streamline

Inspector Source build phase script to use CONFIGURATION instead
of BUILD_STYLE.

11:28 Changeset [64056] by ojan@chromium.org

2010-07-23 Ojan Vafai <ojan@chromium.org>

Reviewed by Darin Adler.

crash in insertParagraph in a contentEditable list
https://bugs.webkit.org/show_bug.cgi?id=42919

  • editing/execCommand/crash-on-enter-in-contentEditable-list-expected.txt: Added.
  • editing/execCommand/crash-on-enter-in-contentEditable-list.html: Added.

2010-07-23 Ojan Vafai <ojan@chromium.org>

Reviewed by Darin Adler.

crash in insertParagraph in a contentEditable list
https://bugs.webkit.org/show_bug.cgi?id=42919

Don't try to break out of a list item if the list it's in
is the rootEditableElement.

  • editing/CompositeEditCommand.cpp: (WebCore::CompositeEditCommand::breakOutOfEmptyListItem):
11:15 Changeset [64055] by bulach@chromium.org

2010-07-26 Marcus Bulach <bulach@chromium.org>

Reviewed by Jeremy Orlow.

Moves IDBKeyRange to use IDBKey rather than SerializedScriptValue / IDBAny.
https://bugs.webkit.org/show_bug.cgi?id=42583

Tests: existing storage/indexeddb/idb-keyrange.html must pass.

  • storage/IDBKeyRange.cpp: (WebCore::IDBKeyRange::IDBKeyRange):
  • storage/IDBKeyRange.h: (WebCore::IDBKeyRange::create): (WebCore::IDBKeyRange::left): (WebCore::IDBKeyRange::right):
  • storage/IDBKeyRange.idl:
  • storage/IndexedDatabaseRequest.cpp: (WebCore::IndexedDatabaseRequest::makeSingleKeyRange): (WebCore::IndexedDatabaseRequest::makeLeftBoundKeyRange): (WebCore::IndexedDatabaseRequest::makeRightBoundKeyRange): (WebCore::IndexedDatabaseRequest::makeBoundKeyRange):
  • storage/IndexedDatabaseRequest.h:
  • storage/IndexedDatabaseRequest.idl:
10:48 Changeset [64054] by simon.fraser@apple.com

2010-07-26 Simon Fraser <simon.fraser@apple.com>

Reviewed by Anders Carlsson.

Composited layers don't scroll in WebKit2
https://bugs.webkit.org/show_bug.cgi?id=42771

Rename two methods on RenderLayerCompositor to make their use more clear, and call them
when the FrameView gets resized and scrolled.

  • page/FrameView.h:
  • page/FrameView.cpp: (WebCore::FrameView::setFrameRect): override so we know when the size of the FrameView changes, so that the RenderLayerCompositor can update its clipping layer. (WebCore::FrameView::scrollPositionChanged): Call compositor()->frameViewDidScroll().
  • rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::updateAfterWidgetResize): updateContentLayerOffset() was renamed.
  • rendering/RenderLayerCompositor.h:
  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::frameViewDidChangeSize): Renamed from updateContentLayerOffset(). (WebCore::RenderLayerCompositor::frameViewDidScroll): Renamed from updateContentLayerScrollPosition(). (WebCore::RenderLayerCompositor::shouldPropagateCompositingToEnclosingIFrame): Propagate compositing on Mac for any FrameView that doesn't have a native widget, which is the case with WebKit2. (WebCore::RenderLayerCompositor::requiresScrollLayer): New method to tell us whether we need to handle scrolling (and clipping) ourselves. (WebCore::RenderLayerCompositor::ensureRootPlatformLayer): Do geometry flipping on the root platform layer only if we're not doing our own scrolling, and use the new requiresScrollLayer() method. (WebCore::RenderLayerCompositor::attachRootPlatformLayer): Send out the rootPlatformLayer(), so that it returns the clipping layer if we have one.
10:37 Changeset [64053] by jorlow@chromium.org

2010-07-26 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Ojan Vafai.

Rename indexedDB tests to match new scheme
https://bugs.webkit.org/show_bug.cgi?id=42375

No changes to the tests themselves.

  • storage/indexeddb/script-tests/basics.js: Added. (openCallback): (test):
  • storage/indexeddb/script-tests/database-basics.js: Added. (openSuccess): (test):
  • storage/indexeddb/script-tests/idb-database-request.js: Removed.
  • storage/indexeddb/script-tests/idb-objectstore-request.js: Removed.
  • storage/indexeddb/script-tests/indexed-database-request.js: Removed.
  • storage/indexeddb/script-tests/keyrange.js: Renamed from LayoutTests/storage/indexeddb/script-tests/idb-keyrange.js. (checkSingleKeyRange): (checkLeftBoundKeyRange): ():
  • storage/indexeddb/script-tests/objectstore-basics.js: Added. (test): (openSuccess): (createSuccess): (addSuccess): (getSuccess): (removeSuccess):
10:19 WebKit Team edited by simon.fraser@apple.com
(diff)
10:11 Changeset [64052] by bweinstein@apple.com

Touch MiniBrowser's stdafx to fix the build.

Rubber-stamped by Adam Roben.

  • MiniBrowser/win/stdafx.h:
10:03 Changeset [64051] by tonyg@chromium.org

2010-07-26 Tony Gentilcore <tonyg@chromium.org>

Reviewed by Darin Fisher.

Move DocumentLoadTiming struct to a new file
https://bugs.webkit.org/show_bug.cgi?id=42917

Also makes DocumentLoadTiming Noncopyable.

No new tests because no new functionality.

  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • loader/DocumentLoadTiming.h: Added. (WebCore::DocumentLoadTiming::DocumentLoadTiming):
  • loader/DocumentLoader.h:
  • loader/FrameLoader.cpp:
  • loader/FrameLoaderTypes.h:
  • loader/MainResourceLoader.cpp:
  • page/Timing.cpp:
10:03 Changeset [64050] by andersca@apple.com

Clean up event handling functions
https://bugs.webkit.org/show_bug.cgi?id=42977

Reviewed by Sam Weinig.

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::mouseEvent):
(WebKit::WebPage::wheelEvent):
(WebKit::WebPage::keyEvent):
(WebKit::WebPage::didReceiveMessage):

  • WebProcess/WebPage/WebPage.h:
09:24 Changeset [64049] by mrowe@apple.com

Fix an #if in WebCore.exp.in to correctly detect whether SVG filters are enabled.

  • WebCore.exp.in:
09:01 Changeset [64048] by steveblock@google.com

2010-07-26 Steve Block <steveblock@google.com>

Reviewed by Jeremy Orlow.

DeviceOrientation event listeners should never be called synchronously from addEventListener()
https://bugs.webkit.org/show_bug.cgi?id=42304

No new tests as there are currently no implementations to test. Tests will be
added once LayoutTestController methods for providing a mock implementation are
in place, see Bug 39589.

  • dom/DeviceOrientationController.cpp: (WebCore::DeviceOrientationController::DeviceOrientationController): (WebCore::DeviceOrientationController::timerFired): (WebCore::DeviceOrientationController::addListener): (WebCore::DeviceOrientationController::removeListener): (WebCore::DeviceOrientationController::removeAllListeners):
  • dom/DeviceOrientationController.h:
08:56 Changeset [64047] by andreas.kling@nokia.com

2010-07-26 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Tor Arne Vestbø.

[Qt] Canvas: Optimize GraphicsContext::addPath()

Avoid "deep concatenating" the canvas path onto the GC path
if the GC path has no elements.

  • platform/graphics/qt/GraphicsContextQt.cpp: (WebCore::GraphicsContext::addPath):
08:43 Changeset [64046] by senorblanco@chromium.org

2010-07-23 Stephen White <senorblanco@chromium.org>

Reviewed by Darin Fisher.

Implement OpenGLES2 helper classes.
https://bugs.webkit.org/show_bug.cgi?id=42905

GLES2Canvas and GLES2Texture are some simple OpenGLES2 helper classes
for managing state, samplers and textures.

When this is hooked up, it will be covered by many layout tests.

  • WebCore.gypi: Add new files to chromium build (contents are protected by #if's for now).
  • platform/graphics/chromium/GLES2Canvas.cpp: Added. (WebCore::affineTo3x3): (WebCore::GLES2Canvas::State::State): (WebCore::GLES2Canvas::GLES2Canvas): (WebCore::GLES2Canvas::~GLES2Canvas): (WebCore::GLES2Canvas::clearRect): (WebCore::GLES2Canvas::fillRect): (WebCore::GLES2Canvas::setFillColor): (WebCore::GLES2Canvas::setAlpha): (WebCore::GLES2Canvas::translate): (WebCore::GLES2Canvas::rotate): (WebCore::GLES2Canvas::scale): (WebCore::GLES2Canvas::concatCTM): (WebCore::GLES2Canvas::save): (WebCore::GLES2Canvas::restore): (WebCore::GLES2Canvas::drawTexturedRect): (WebCore::GLES2Canvas::setCompositeOperation): (WebCore::GLES2Canvas::applyCompositeOperator): (WebCore::GLES2Canvas::getQuadVertices): (WebCore::GLES2Canvas::getQuadIndices): (WebCore::loadShader): (WebCore::GLES2Canvas::getSimpleProgram): (WebCore::GLES2Canvas::getTexProgram): (WebCore::GLES2Canvas::createTexture): (WebCore::GLES2Canvas::getTexture): (WebCore::GLES2Canvas::checkGLError):
  • platform/graphics/chromium/GLES2Canvas.h: Added. (WebCore::GLES2Canvas::gles2Context):
  • platform/graphics/chromium/GLES2Texture.cpp: Added. (WebCore::GLES2Texture::GLES2Texture): (WebCore::GLES2Texture::~GLES2Texture): (WebCore::GLES2Texture::create): (WebCore::convertFormat): (WebCore::GLES2Texture::load): (WebCore::GLES2Texture::bind):
  • platform/graphics/chromium/GLES2Texture.h: Added. (WebCore::GLES2Texture::): (WebCore::GLES2Texture::format): (WebCore::GLES2Texture::width): (WebCore::GLES2Texture::height):
07:52 EFLWebKit edited by antognolli@profusion.mobi
(diff)
07:49 Changeset [64045] by jorlow@chromium.org

2010-07-26 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Steve Block.

Fix IndexedDB release build on mac
https://bugs.webkit.org/show_bug.cgi?id=42377

  • storage/IDBDatabaseImpl.cpp: (WebCore::IDBDatabaseImpl::objectStore):
07:43 EFLWebKit edited by antognolli@profusion.mobi
(diff)
07:42 EFLWebKit edited by antognolli@profusion.mobi
(diff)
06:47 Changeset [64044] by commit-queue@webkit.org

2010-07-26 Hayato Ito <hayato@chromium.org>

Reviewed by Shinichiro Hamaji.

Add pretty printer of WTF::Vector for GDB 7.

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

  • gdb/wtf.py: Added.
05:04 Changeset [64043] by bulach@chromium.org

2010-07-26 Marcus Bulach <bulach@chromium.org>

Unreviewed. Add chromium expectations for failing tests.

tables/mozilla/other/slashlogo.html fails after r63994
https://bugs.webkit.org/show_bug.cgi?id=42964

Adds a suppression for chromium.

  • platform/chromium/test_expectations.txt:
04:52 Changeset [64042] by commit-queue@webkit.org

2010-07-26 Satish Sampath <satish@chromium.org>

Reviewed by Steve Block.

Add WebKit plumbing to connect speech requests and callbacks between WebCore and the embedder.
https://bugs.webkit.org/show_bug.cgi?id=42367

No new tests, the relevant LayoutTestController bindings will be added in the next patch.

  • page/SpeechInput.cpp: renamed a method.
  • page/SpeechInput.h: renamed a method.
  • page/SpeechInputClient.h: added an extra method to optionally let users stop recording once they have spoken.
  • page/SpeechInputClientListener.h: renamed a method.
  • page/SpeechInputListener.h: renamed a method.

2010-07-26 Satish Sampath <satish@chromium.org>

Reviewed by Steve Block.

Add WebKit plumbing to connect speech requests and callbacks between WebCore and the embedder.
https://bugs.webkit.org/show_bug.cgi?id=42367

No new tests, the relevant LayoutTestController bindings and tests will be added in the next patch.

  • public/WebSpeechInputController.h: Added new interface, implemented by embedder and called by WebKit (WebKit::WebSpeechInputController::~WebSpeechInputController):
  • public/WebSpeechInputListener.h: Added new interface, implemented by WebKit and called by embedder. (WebKit::WebSpeechInputListener::~WebSpeechInputListener):
  • public/WebViewClient.h: (WebKit::WebViewClient::createSpeechInputClient): New method to get the embedder's speech input client interface.
  • src/SpeechInputClientImpl.cpp: Added new class, implementation of a two way connector between WebCore and the embedder for requests and responses. (WebKit::SpeechInputClientImpl::SpeechInputClientImpl): (WebKit::SpeechInputClientImpl::~SpeechInputClientImpl): (WebKit::SpeechInputClientImpl::startRecognition): (WebKit::SpeechInputClientImpl::stopRecording): (WebKit::SpeechInputClientImpl::didCompleteRecording): (WebKit::SpeechInputClientImpl::setRecognitionResult): (WebKit::SpeechInputClientImpl::didCompleteRecognition):
  • src/SpeechInputClientImpl.h: Added.
  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::WebViewImpl): Pass on the above mentioned speech input connector to WebCore.
  • src/WebViewImpl.h:
04:51 Changeset [64041] by andreas.kling@nokia.com

2010-07-26 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] getImageData(): Single-pass RGB->BGR and un-premultiplication
https://bugs.webkit.org/show_bug.cgi?id=42945

Combine the two operations into a single loop over the pixel data.
Yields a considerable FPS gain on http://www.semantix.gr/statue.html

  • platform/graphics/qt/ImageBufferQt.cpp: (WebCore::getImageData):
04:21 Changeset [64040] by pfeldman@chromium.org

2010-07-26 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Shinichiro Hamaji.

Web Inspector: make sure proxy objects are JSON-stringifiable.

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

  • inspector/front-end/InjectedScript.js: (injectedScriptConstructor):
04:12 Changeset [64039] by vestbo@webkit.org

[Qt] Fix feature detection regexp

Reviewed by Simon Hausmann.

We were missing the last feature

  • Scripts/webkitdirs.pm:
02:52 Changeset [64038] by ossy@webkit.org

Unreviewed.

[Qt] DRT sideeffect revealed by r63657
https://bugs.webkit.org/show_bug.cgi?id=42578

  • platform/qt/Skipped: Add tests to the Skipped list until fix.
    • http/tests/xmlhttprequest/authorization-header.html
    • http/tests/xmlhttprequest/null-auth.php
02:37 Changeset [64037] by apavlov@chromium.org

2010-07-26 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Joseph Pecoraro.

Inspector should remember resources sorting set by the user
https://bugs.webkit.org/show_bug.cgi?id=19208

Sorting options both for time and size graphs are memorized in a single property,
which makes them possible to retrieve in a single message (e.g. useful for remote debugging).

  • inspector/front-end/ResourcesPanel.js: (WebInspector.ResourcesPanel.prototype.populateSidebar): (WebInspector.ResourcesPanel.prototype._settingsLoaded): (WebInspector.ResourcesPanel.prototype._loadSortOptions): (WebInspector.ResourcesPanel.prototype._loadSortOptionForGraph): (WebInspector.ResourcesPanel.prototype._graphSelected): (WebInspector.ResourcesPanel.prototype._changeSortingFunction): (WebInspector.ResourcesPanel.prototype._selectedOptionNameForGraph): (WebInspector.ResourcesPanel.prototype._doChangeSortingFunction):
  • inspector/front-end/Settings.js: (WebInspector.populateApplicationSettings):
01:53 Changeset [64036] by hausmann@webkit.org

[Qt] Use the default timeout interval for JS as the HTML tokenizer delay for setHtml()

Patch by Tor Arne Vestbø <tor.arne.vestbo@nokia.com> on 2009-10-30
Reviewed by Kenneth Rohde Christiansen.

This ensures that long-running JavaScript (for example due to a modal alert() dialog),
will not trigger a deferred load after only 500ms (the default tokenizer delay) while
still giving a reasonable timeout (10 seconds) to prevent deadlock.

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

JavaScriptCore:

  • runtime/TimeoutChecker.h: Add getter for the timeout interval

WebKit/qt:

  • Api/qwebframe.cpp: Document the behaviour
  • WebCoreSupport/FrameLoaderClientQt.cpp: set the custom tokenizer delay for substitute loads
  • tests/qwebframe/tst_qwebframe.cpp: Add test
01:29 Changeset [64035] by loislo@chromium.org

2010-07-26 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Pavel Feldman.

Chromium DevTools: refactor query params, add basic
support for WebSocket-based backend.

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

  • src/js/DevTools.js: (devtools.ToolsAgent.prototype.dispatchOnClient_): (WebInspector.loaded): (WebInspector.doLoadedDone): (devtools.domContentLoaded):
  • src/js/InspectorControllerImpl.js:
01:19 Changeset [64034] by rniwa@webkit.org

WebCore: Redo fails after text node is split by SplitTextNodeCommand
https://bugs.webkit.org/show_bug.cgi?id=42941

Reviewed by Darin Adler.

The bug was caused by SplitTextNodeCommand not implementing doReapply.
Implemented doReapply and added insertText1AndTrimText2 to share the code between doApply and doReapply.

Test: editing/undo/redo-split-text-node.html

editing/undo/redo-split-text-node-with-removal.html

  • editing/SplitTextNodeCommand.cpp:

(WebCore::SplitTextNodeCommand::doApply): Calls executeApply.
(WebCore::SplitTextNodeCommand::doUnapply): Don't delete m_text1, which is later used in doReapply.
(WebCore::SplitTextNodeCommand::doReapply): Added.
(WebCore::SplitTextNodeCommand::insertText1AndTrimText2): Added.

  • editing/SplitTextNodeCommand.h:

LayoutTests: Added a test that splits a text node by inserting a paragraph and applies inline style.
The test then ensures that the style is preserved after undos and redos.

Reviewed by Darin Adler.

Also added a test in which the text node is removed by the script after undo.

Redo fails after text node is split by SplitTextNodeCommand
https://bugs.webkit.org/show_bug.cgi?id=42941

  • editing/undo/redo-split-text-node-expected.txt: Added.
  • editing/undo/redo-split-text-node.html: Added.
  • editing/undo/redo-split-text-with-removal-expected.txt: Added.
  • editing/undo/redo-split-text-with-removal.html: Added.

07/25/10:

23:57 Changeset [64033] by darin@apple.com

2010-07-25 Darin Adler <darin@apple.com>

Reviewed by Dan Bernstein.

WebKitTestRunner needs to support layoutTestController.dumpSelectionRect
https://bugs.webkit.org/show_bug.cgi?id=42326

  • platform/mac-wk2/Skipped: Removed all the tests from the list that were there only because they called dumpSelectionRect. Most of them pass now.

2010-07-25 Darin Adler <darin@apple.com>

Reviewed by Dan Bernstein.

WebKitTestRunner needs to support layoutTestController.dumpSelectionRect
https://bugs.webkit.org/show_bug.cgi?id=42326

  • WebKitTestRunner/InjectedBundle/Bindings/LayoutTestController.idl: Added dumpSelectionRect.
  • WebKitTestRunner/InjectedBundle/LayoutTestController.h: (WTR::LayoutTestController::dumpSelectionRect): Added. For now, this does nothing because its purpose is to change pixel test output and we do not have pixel tests implemented yet.
23:40 Changeset [64032] by philn@webkit.org

2010-07-23 Philippe Normand <pnormand@igalia.com>

Reviewed by Eric Carlson.

[GTK] media/media-can-play-mpeg-audio.html fails
https://bugs.webkit.org/show_bug.cgi?id=42895

Fixed advertizing of mp3, mpeg and mp4 mime-types in the player.

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::mimeTypeCache):

LayoutTests:

2010-07-23 Philippe Normand <pnormand@igalia.com>

Reviewed by Eric Carlson.

[GTK] media/media-can-play-mpeg-audio.html fails
https://bugs.webkit.org/show_bug.cgi?id=42895

  • platform/gtk/Skipped: Unskip fixed test.
23:35 Changeset [64031] by philn@webkit.org

2010-07-23 Philippe Normand <pnormand@igalia.com>

Reviewed by Eric Carlson.

LayoutTests/media/media-document-audio-size.html failure
https://bugs.webkit.org/show_bug.cgi?id=38376

Advertize audio/x-wav along with audio/wav when appropriate,
otherwise WebKit attempts to use a plugin to play the media.

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::mimeTypeCache):

LayoutTests:

2010-07-23 Philippe Normand <pnormand@igalia.com>

Reviewed by Eric Carlson.

LayoutTests/media/media-document-audio-size.html failure
https://bugs.webkit.org/show_bug.cgi?id=38376

  • platform/gtk/Skipped: Unskip fixed test on GTK port.
23:13 Changeset [64030] by darin@apple.com

2010-07-25 Darin Adler <darin@apple.com>

Reviewed by Maciej Stachowiak.

Need to support unsafe access console messages in WebKitTestRunner
https://bugs.webkit.org/show_bug.cgi?id=42320

  • platform/mac-wk2/Skipped: Since unsafe access console messages are working properly, moved the failing tests listed under this bug to other sections in the file based on the reason they are failing. Removed the tests that are passing.
23:03 Changeset [64029] by darin@apple.com

2010-07-25 Darin Adler <darin@apple.com>

Reviewed by Maciej Stachowiak.

WebKitTestRunner needs to support layoutTestController.keepWebHistory
https://bugs.webkit.org/show_bug.cgi?id=42323

  • platform/mac-wk2/Skipped: Removed the tests that needed only keepWebHistory and computedStyleIncludingVisitedInfo. Moved the other tests in the keepWebHistory section of the file into the other appropriate lists.

2010-07-25 Darin Adler <darin@apple.com>

Reviewed by Maciej Stachowiak.

WebKitTestRunner needs to support layoutTestController.keepWebHistory
https://bugs.webkit.org/show_bug.cgi?id=42323

Added keepWebHistory and computedStyleIncludingVisitedInfo.

Also fixed misspellings of the word "receive".

Also tweaked the names of some of the LayoutTestController members.

  • WebKitTestRunner/InjectedBundle/Bindings/CodeGeneratorTestRunner.pm: Added support for a type named "object" that is passed and returns as a JSValueRef.
  • WebKitTestRunner/InjectedBundle/Bindings/LayoutTestController.idl: Added keepWebHistory and computedStyleIncludingVisitedInfo. Also put setAcceptsEditing up nearer the top.
  • WebKitTestRunner/InjectedBundle/InjectedBundle.cpp: (WTR::InjectedBundle::_didReceiveMessage): Fixed mispellings of receive. (WTR::InjectedBundle::initialize): Ditto. (WTR::InjectedBundle::didReceiveMessage): Ditto. (WTR::InjectedBundle::reset): Reset the state of visited links between tests. Also eliminated the unused argument to the LayoutTestController create function. (WTR::InjectedBundle::setShouldTrackVisitedLinks): Added.
  • WebKitTestRunner/InjectedBundle/InjectedBundle.h: Ditto.
  • WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp: (WTR::InjectedBundlePage::shouldBeginEditing): Call shouldAllowEditing instead of acceptsEditing. (WTR::InjectedBundlePage::shouldEndEditing): Ditto. (WTR::InjectedBundlePage::shouldInsertNode): Ditto. (WTR::InjectedBundlePage::shouldInsertText): Ditto. (WTR::InjectedBundlePage::shouldDeleteRange): Ditto. (WTR::InjectedBundlePage::shouldChangeSelectedRange): Ditto. (WTR::InjectedBundlePage::shouldApplyStyle): Ditto.
  • WebKitTestRunner/InjectedBundle/LayoutTestController.cpp: (WTR::LayoutTestController::create): Removed uneeded argument. (WTR::LayoutTestController::LayoutTestController): Removed unneeded argument. Updated for rename of m_acceptsEditing to m_shouldAllowEditing. (WTR::LayoutTestController::numberOfActiveAnimations): Added some FIXMEs about the fact that this works on the main frame. (WTR::LayoutTestController::pauseAnimationAtTimeOnElementWithId): Ditto. (WTR::LayoutTestController::keepWebHistory): Added. (WTR::LayoutTestController::computedStyleIncludingVisitedInfo): Added. (WTR::JSObjectSetProperty): Added. Helper to make the function below cleaner. (WTR::LayoutTestController::makeWindowObject): Changed to use the overload of JSObjectSetProperty above.
  • WebKitTestRunner/InjectedBundle/LayoutTestController.h: Changes to match above.
  • WebKitTestRunner/TestController.cpp: (WTR::TestController::initialize): Fixed mispellings of receive. (WTR::TestController::_didReceiveMessageFromInjectedBundle): Ditto. (WTR::TestController::didReceiveMessageFromInjectedBundle): Ditto.
  • WebKitTestRunner/TestController.h: Ditto.
  • WebKitTestRunner/TestInvocation.cpp: (WTR::TestInvocation::didReceiveMessageFromInjectedBundle): Ditto.
  • WebKitTestRunner/TestInvocation.h: Ditto.

2010-07-25 Darin Adler <darin@apple.com>

Reviewed by Maciej Stachowiak.

WebKitTestRunner needs to support layoutTestController.keepWebHistory
https://bugs.webkit.org/show_bug.cgi?id=42323

Added WKBundleSetShouldTrackVisitedLinks, WKBundleRemoveAllVisitedLinks,
and WKBundleFrameGetComputedStyleIncludingVisitedInfo.

Also fixed misspellings of the word "receive".

  • WebProcess/InjectedBundle/API/c/WKBundle.cpp: (WKBundleSetShouldTrackVisitedLinks): Added. (WKBundleRemoveAllVisitedLinks): Added.
  • WebProcess/InjectedBundle/API/c/WKBundle.h: Added functions and fixed mispellings of receive.
  • WebProcess/InjectedBundle/API/c/WKBundleFrame.cpp: (WKBundleFrameGetJavaScriptContext): Moved the code from here into WebFrame, to be consistent with the rest of the functions in this file. (WKBundleFrameGetComputedStyleIncludingVisitedInfo): Added.
  • WebProcess/InjectedBundle/API/c/WKBundleFrame.h: Added WKBundleFrameGetComputedStyleIncludingVisitedInfo and tried to rearrange the file to be more logically ordered and grouped.
  • WebProcess/InjectedBundle/InjectedBundle.cpp: (WebKit::InjectedBundle::setShouldTrackVisitedLinks): Added. (WebKit::InjectedBundle::removeAllVisitedLinks): Added. (WebKit::InjectedBundle::didReceiveMessage): Fixed mispellings of receive.
  • WebProcess/InjectedBundle/InjectedBundle.h: Ditto.
  • WebProcess/WebPage/WebFrame.cpp: (WebKit::WebFrame::jsContext): Added. Contains the code that used to be in WKBundleFrameGetJavaScriptContext. (WebKit::WebFrame::computedStyleIncludingVisitedInfo): Added.
  • WebProcess/WebPage/WebFrame.h: Ditto.
  • UIProcess/API/C/WKContext.h: Fixed misspellings of receive.
  • UIProcess/WebContext.cpp: (WebKit::WebContext::didReceiveMessageFromInjectedBundle): Ditto.
  • UIProcess/WebContext.h: Ditto.
  • UIProcess/WebContextInjectedBundleClient.cpp: (WebKit::WebContextInjectedBundleClient::didReceiveMessageFromInjectedBundle): Ditto.
  • UIProcess/WebContextInjectedBundleClient.h: Ditto.
  • UIProcess/WebProcessProxy.cpp: (WebKit::WebProcessProxy::forwardMessageToWebContext): Ditto.
  • WebProcess/WebProcess.cpp: (WebKit::WebProcess::forwardMessageToInjectedBundle): Ditto.
21:10 Changeset [64028] by rniwa@webkit.org

splitTextAt*IfNeed and splitTextElementAt*IfNeed need to be cleaned up
https://bugs.webkit.org/show_bug.cgi?id=42937

Reviewed by Kent Tamura.

Isolated the code to decide whether or not text node should be split into isValidCaretPositionInTextNode.
Moved the condition check out of *IfNeeded methods to applyRelativeFontStyleChange and applyInlineStyle.

No new tests added since this is a clean up.

  • editing/ApplyStyleCommand.cpp:

(WebCore::ApplyStyleCommand::applyRelativeFontStyleChange): Uses isValidCaretPositionInTextNode.
(WebCore::ApplyStyleCommand::applyInlineStyle): Uses isValidCaretPositionInTextNode.
(WebCore::ApplyStyleCommand::splitTextAtStart): Renamed from splitTextAtStartIfNeeded.
(WebCore::ApplyStyleCommand::splitTextAtEnd): Renamed from splitTextAtEndIfNeeded.
(WebCore::ApplyStyleCommand::splitTextElementAtStart): Renamed from splitTextElementAtStartIfNeeded.
(WebCore::ApplyStyleCommand::splitTextElementAtEnd): Renamed from splitTextElementAtEndIfNeeded.
(WebCore::ApplyStyleCommand::isValidCaretPositionInTextNode): Returns true if the position lies within a text node.

  • editing/ApplyStyleCommand.h:
21:01 Changeset [64027] by commit-queue@webkit.org

2010-07-25 Patrick Gansterer <paroga@paroga.com>

Reviewed by Kent Tamura.

[WINCE] Buildfix for JSC in release mode
https://bugs.webkit.org/show_bug.cgi?id=42934

  • jsc.cpp: Don't use try on WinCE.
20:48 Changeset [64026] by commit-queue@webkit.org

2010-07-25 Patrick Gansterer <paroga@paroga.com>

Reviewed by Kent Tamura.

[WINCE] Buildfix for ImageBufferWince
https://bugs.webkit.org/show_bug.cgi?id=42928

  • platform/graphics/wince/ImageBufferWince.cpp: (WebCore::): (WebCore::BufferedImage::draw): Pass through ColorSpace parameter. (WebCore::BufferedImage::drawPattern): Ditto. (WebCore::ImageBuffer::platformTransformColorSpace): Added.
20:33 Changeset [64025] by commit-queue@webkit.org

2010-07-25 Patrick Gansterer <paroga@paroga.com>

Reviewed by Kent Tamura.

[WINCE] Buildfix for PopupMenuWin.cpp
https://bugs.webkit.org/show_bug.cgi?id=42932

Add #if !OS(WINCE) around unsupported window messages.

  • platform/win/PopupMenuWin.cpp: (WebCore::PopupMenu::show): (WebCore::PopupMenu::wndProc):
14:51 Changeset [64024] by abarth@webkit.org

2010-07-25 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Implement adjustMathMLAttributes
https://bugs.webkit.org/show_bug.cgi?id=42952

Yay test progression.

  • html5lib/runner-expected-html5.txt:

2010-07-25 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Implement adjustMathMLAttributes
https://bugs.webkit.org/show_bug.cgi?id=42952

This is kind of mechanical overkill for adjusting one attribute name,
but it's nice to use the same machinery as SVG.

  • html/HTMLTreeBuilder.cpp:
  • mathml/mathattrs.in:
14:36 Changeset [64023] by abarth@webkit.org

2010-07-25 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Functions supporting the legacy tree builder are confusing
https://bugs.webkit.org/show_bug.cgi?id=42951

This patch just inlines these functions into their one call site and
removes the notImplemented() calls. We thought we were going to grow
these to support the new tree builder, but we took a different path.
Previous to this patch, these functions were confusing because they
looked like part of the new tree builder.

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::passTokenToLegacyParser):
  • html/HTMLTreeBuilder.h:
13:05 Changeset [64022] by abarth@webkit.org

2010-07-25 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Recent HTML5 TreeBuilder Regression? All canvas tests ASSERT
https://bugs.webkit.org/show_bug.cgi?id=42948

Remove ASSERT and replace with correct branch. I put this ASSERT in
to make sure we had test coverage for this branch. I guess we do. :)

  • html/HTMLConstructionSite.cpp: (WebCore::HTMLConstructionSite::attach):
08:46 Changeset [64021] by pfeldman@chromium.org

2010-07-25 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: check of jQuery object type prior to treating it as an array.

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

  • inspector/front-end/InjectedScript.js: (injectedScriptConstructor):
00:56 Changeset [64020] by ap@apple.com

2010-07-25 Alexey Proskuryakov <ap@apple.com>

Reviewed by Sam Weinig.

https://bugs.webkit.org/show_bug.cgi?id=42193
Support layoutTestController.dumpEditingDelegates in WebKitTestRunner

Step 2 - add the rest of editing delegates.

  • platform/mac-wk2/Skipped: Changed the bug tracking why the tests are skipped - the delegates are now implemented, but one of them isn't called for some reason.

2010-07-25 Alexey Proskuryakov <ap@apple.com>

Reviewed by Sam Weinig.

https://bugs.webkit.org/show_bug.cgi?id=42193
Support layoutTestController.dumpEditingDelegates in WebKitTestRunner

Step 2 - add the rest of editing delegates.

  • WebProcess/InjectedBundle/API/c/WKBundleAPICast.h:
  • WebProcess/InjectedBundle/API/c/WKBundleBase.h: Added enums for EditorInsertAction and EAffinity. Also added a dummy type for CSSStyleDeclaration - neither DumpRenderTree not WebKitTestRunner actually use that.
  • WebProcess/InjectedBundle/API/c/WKBundlePage.h:
  • WebProcess/InjectedBundle/InjectedBundlePageEditorClient.cpp: (WebKit::InjectedBundlePageEditorClient::shouldBeginEditing): (WebKit::InjectedBundlePageEditorClient::shouldEndEditing): (WebKit::InjectedBundlePageEditorClient::shouldInsertNode): (WebKit::InjectedBundlePageEditorClient::shouldInsertText): (WebKit::InjectedBundlePageEditorClient::shouldDeleteRange): (WebKit::InjectedBundlePageEditorClient::shouldChangeSelectedRange): (WebKit::InjectedBundlePageEditorClient::shouldApplyStyle): (WebKit::InjectedBundlePageEditorClient::didBeginEditing): (WebKit::InjectedBundlePageEditorClient::didEndEditing): (WebKit::InjectedBundlePageEditorClient::didChange): (WebKit::InjectedBundlePageEditorClient::didChangeSelection):
  • WebProcess/InjectedBundle/InjectedBundlePageEditorClient.h:
  • WebProcess/WebCoreSupport/WebEditorClient.cpp: (WebKit::WebEditorClient::shouldDeleteRange): (WebKit::WebEditorClient::shouldEndEditing): (WebKit::WebEditorClient::shouldInsertNode): (WebKit::WebEditorClient::shouldInsertText): (WebKit::WebEditorClient::shouldChangeSelectedRange): (WebKit::WebEditorClient::shouldApplyStyle): (WebKit::WebEditorClient::didBeginEditing): (WebKit::WebEditorClient::respondToChangedContents): (WebKit::WebEditorClient::respondToChangedSelection): (WebKit::WebEditorClient::didEndEditing): Added remaining delagates that are needed for WebKitTestRunner.

2010-07-25 Alexey Proskuryakov <ap@apple.com>

Reviewed by Sam Weinig.

https://bugs.webkit.org/show_bug.cgi?id=42193
Support layoutTestController.dumpEditingDelegates in WebKitTestRunner

Step 2 - add the rest of editing delegates.

  • WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp: (WTR::operator<<): Fixed to not crash when range is null. (WTR::InjectedBundlePage::InjectedBundlePage): (WTR::InjectedBundlePage::_shouldEndEditing): (WTR::InjectedBundlePage::_shouldInsertNode): (WTR::InjectedBundlePage::_shouldInsertText): (WTR::InjectedBundlePage::_shouldDeleteRange): (WTR::InjectedBundlePage::_shouldChangeSelectedRange): (WTR::InjectedBundlePage::_shouldApplyStyle): (WTR::InjectedBundlePage::_didBeginEditing): (WTR::InjectedBundlePage::_didEndEditing): (WTR::InjectedBundlePage::_didChange): (WTR::InjectedBundlePage::_didChangeSelection): (WTR::InjectedBundlePage::shouldEndEditing): (WTR::InjectedBundlePage::shouldInsertNode): (WTR::InjectedBundlePage::shouldInsertText): (WTR::InjectedBundlePage::shouldDeleteRange): (WTR::InjectedBundlePage::shouldChangeSelectedRange): (WTR::InjectedBundlePage::shouldApplyStyle): (WTR::InjectedBundlePage::didBeginEditing): (WTR::InjectedBundlePage::didEndEditing): (WTR::InjectedBundlePage::didChange): (WTR::InjectedBundlePage::didChangeSelection):
  • WebKitTestRunner/InjectedBundle/InjectedBundlePage.h:

07/24/10:

22:24 Changeset [64019] by dbates@webkit.org

2010-07-24 Daniel Bates <dbates@rim.com>

Unreviewed, build fix when building with WML support.

PageCache.cpp compile error when WML is enabled
https://bugs.webkit.org/show_bug.cgi?id=40765

Remove extraneous semicolon.

Note, building with WML support enabled is broken
even with this fix. See Bug #42943 for more details.

  • history/PageCache.cpp: (WebCore::PageCache::canCachePageContainingThisFrame):
18:24 WikiStart edited by paroga@paroga.com
(diff)
18:22 WinCE created by paroga@paroga.com
18:03 Changeset [64018] by abarth@webkit.org

2010-07-24 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Atomize strings used by the HTML5 tree builder
https://bugs.webkit.org/show_bug.cgi?id=42875

This is about a 1% improvement to the parsing benchmark.

  • html/HTMLTagNames.in:
  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTagForInBody): (WebCore::HTMLTreeBuilder::processStartTag): (WebCore::HTMLTreeBuilder::processEndTagForInBody): (WebCore::HTMLTreeBuilder::processStartTagForInHead):
17:40 Changeset [64017] by commit-queue@webkit.org

2010-07-24 Patrick Gansterer <paroga@paroga.com>

Reviewed by Darin Adler.

[MSVC] Ensure 4 byte alignment on ARM
https://bugs.webkit.org/show_bug.cgi?id=42935

  • jit/JITStubs.h: Added #pragma pack(4) around JITStackFrame.
17:08 Changeset [64016] by commit-queue@webkit.org

2010-07-24 Patrick Gansterer <paroga@paroga.com>

Reviewed by Darin Adler.

[WINCE] Cleanup defines in Platform.h
https://bugs.webkit.org/show_bug.cgi?id=42933

  • wtf/Platform.h:
16:33 Changeset [64015] by commit-queue@webkit.org

2010-07-24 Patrick Gansterer <paroga@paroga.com>

Reviewed by Darin Adler.

Buildfix for !ENABLE(INSPECTOR) after r63891.
https://bugs.webkit.org/show_bug.cgi?id=42925

  • bindings/js/ScriptValue.cpp: Added property svn:eol-style.
  • bindings/js/ScriptValue.h: Added property svn:eol-style.
  • inspector/ConsoleMessage.cpp: Added property svn:eol-style.
  • inspector/ConsoleMessage.h: Added property svn:eol-style.
15:36 Changeset [64014] by cfleizach@apple.com

AX: need a layout test testing misspelled words in attributed strings
https://bugs.webkit.org/show_bug.cgi?id=42899

Reviewed by David Kilzer.

WebKitTools:

  • DumpRenderTree/AccessibilityUIElement.cpp:

(attributedStringForRangeCallback):
(attributedStringRangeIsMisspelledCallback):
(AccessibilityUIElement::getJSClass):

  • DumpRenderTree/AccessibilityUIElement.h:
  • DumpRenderTree/gtk/AccessibilityUIElementGtk.cpp:

(AccessibilityUIElement::attributedStringForRange):
(AccessibilityUIElement::attributedStringRangeIsMisspelled):

  • DumpRenderTree/mac/AccessibilityUIElementMac.mm:

(AccessibilityUIElement::attributedStringForRange):
(AccessibilityUIElement::attributedStringRangeIsMisspelled):

  • DumpRenderTree/win/AccessibilityUIElementWin.cpp:

(AccessibilityUIElement::attributedStringForRange):
(AccessibilityUIElement::attributedStringRangeIsMisspelled):

LayoutTests:

  • platform/mac/accessibility/misspelled-attributed-string-expected.txt: Added.
  • platform/mac/accessibility/misspelled-attributed-string.html: Added.
15:17 Changeset [64013] by rniwa@webkit.org

editing/execCommand/hilitecolor.html and editing/style/remove-underline-from-stylesheet.html should be dumpAsMarkup tests
https://bugs.webkit.org/show_bug.cgi?id=42793

Reviewed by Darin Adler.

Converted the tests and removed the platform specific expected results.

  • editing/execCommand/hilitecolor-expected.txt: Added.
  • editing/execCommand/hilitecolor.html: Converted.
  • editing/style/remove-underline-from-stylesheet-expected.txt: Converted.
  • editing/style/remove-underline-from-stylesheet.html: Converted.
  • platform/chromium-linux/editing/execCommand/hilitecolor-expected.checksum: Removed.
  • platform/chromium-win/editing/execCommand/hilitecolor-expected.checksum: Removed.
  • platform/chromium-win/editing/execCommand/hilitecolor-expected.png: Removed.
  • platform/chromium-win/editing/execCommand/hilitecolor-expected.txt: Removed.
  • platform/mac/editing/execCommand/hilitecolor-expected.checksum: Removed.
  • platform/mac/editing/execCommand/hilitecolor-expected.png: Removed.
  • platform/mac/editing/execCommand/hilitecolor-expected.txt: Removed.
  • platform/qt/editing/execCommand/hilitecolor-expected.txt: Removed.
13:02 Changeset [64012] by abarth@webkit.org

2010-07-24 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Re-order whitespace checks in the tokenizer
https://bugs.webkit.org/show_bug.cgi?id=42923

This saves about 0.5% on the parsing benchmark.

  • html/HTMLTokenizer.cpp:
11:48 Changeset [64011] by andreas.kling@nokia.com

2010-07-24 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Simon Hausmann.

[Qt] tst_QWebFrame::callQtInvokable() fails
https://bugs.webkit.org/show_bug.cgi?id=41065

Converting JS objects to QVariantMaps was broken.
This is a partial revert of <http://trac.webkit.org/changeset/61478>

  • bridge/qt/qt_runtime.cpp: (JSC::Bindings::convertValueToQVariant): Remove erroneous crash guard around JSObject::getPropertyNames()
10:33 Changeset [64010] by abarth@webkit.org

2010-07-24 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Save a per-character branch in the HTML parser
https://bugs.webkit.org/show_bug.cgi?id=42921

The parsing benchmark claims this is a wash, but it makes me feel
better.

  • html/HTMLToken.h: (WebCore::HTMLToken::clear): (WebCore::HTMLToken::beginStartTag): (WebCore::HTMLToken::beginEndTag): (WebCore::HTMLToken::ensureIsCharacterToken): (WebCore::HTMLToken::beginComment): (WebCore::HTMLToken::beginDOCTYPE):
  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::bufferCharacter):
08:06 Changeset [64009] by pfeldman@chromium.org

2010-07-24 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: chromium build fix.

  • src/ApplicationCacheHost.cpp: (WebCore::ApplicationCacheHost::fillResourceList):
05:54 Changeset [64008] by commit-queue@webkit.org

2010-07-24 Riccardo Magliocchetti <riccardo.magliocchetti@gmail.com>

Reviewed by Xan Lopez.

[GTK] Enable jit compilation on arm
https://bugs.webkit.org/show_bug.cgi?id=42856

Add missing bits to configure.ac to build the arm jit.
Compile and tested on debian sid armel on a qemu vm.

  • configure.ac:

07/23/10:

21:17 Changeset [64007] by commit-queue@webkit.org

2010-07-23 Patrick Gansterer <paroga@paroga.com>

Reviewed by Darin Adler.

Buildfix for !ENABLE(FILTERS) after r63531.
https://bugs.webkit.org/show_bug.cgi?id=42914

  • rendering/RenderSVGContainer.cpp: Add missing header.
20:35 Changeset [64006] by commit-queue@webkit.org

2010-07-23 Kavita Kanetkar <kkanetkar@chromium.org>

Reviewed by Joseph Pecoraro.

Implement remaining Inspector support for chrome's appcache
https://bugs.webkit.org/show_bug.cgi?id=42426

  • inspector/InspectorApplicationCacheAgent.cpp: (WebCore::InspectorApplicationCacheAgent::getApplicationCaches): (WebCore::InspectorApplicationCacheAgent::buildObjectForApplicationCache): (WebCore::InspectorApplicationCacheAgent::buildArrayForApplicationCacheResources): (WebCore::InspectorApplicationCacheAgent::buildObjectForApplicationCacheResource):
  • inspector/InspectorApplicationCacheAgent.h: (WebCore::InspectorController::connectFrontend):
  • loader/appcache/ApplicationCacheHost.cpp: (WebCore::ApplicationCacheHost::fillResourceList): (WebCore::ApplicationCacheHost::applicationCacheInfo):
  • loader/appcache/ApplicationCacheHost.h: Changed creation + update times to double and moved appcache info here from InspectorApplicationCacheAgent.h (WebCore::ApplicationCacheHost::CacheInfo::CacheInfo): (WebCore::ApplicationCacheHost::ResourceInfo::ResourceInfo):

2010-07-23 Kavita Kanetkar <kkanetkar@chromium.org>

Reviewed by Joseph Pecoraro.

Implement remaining Inspector support for chrome's appcache
https://bugs.webkit.org/show_bug.cgi?id=42426

  • src/ApplicationCacheHost.cpp: Plugging in chrome's appcache info. (WebCore::ApplicationCacheHost::applicationCacheInfo): (WebCore::ApplicationCacheHost::fillResourceList):
  • src/js/InspectorControllerImpl.js: (devtools.InspectorBackendImpl):
18:48 Changeset [64005] by commit-queue@webkit.org

2010-07-23 Rafael Antognolli <antognolli@profusion.mobi>

Reviewed by Antonio Gomes.

[EFL] Cleanup glib support (make it optional)
https://bugs.webkit.org/show_bug.cgi?id=42480

Fixing support for compiling EFL port without glib.

  • cmake/FindGthread.cmake: Added.
  • cmake/OptionsEfl.cmake:

2010-07-23 Rafael Antognolli <antognolli@profusion.mobi>

Reviewed by Antonio Gomes.

[EFL] Cleanup glib support (make it optional)
https://bugs.webkit.org/show_bug.cgi?id=42480

Remove gobject/GRefPtr.cpp if not using soup/glib.

  • wtf/CMakeListsEfl.txt:

2010-07-23 Rafael Antognolli <antognolli@profusion.mobi>

Reviewed by Antonio Gomes.

[EFL] Cleanup glib support (make it optional)
https://bugs.webkit.org/show_bug.cgi?id=42480

No new tests, no new features added.
Just add flags and directories for glib if necessary.
Put some ifdefs on glib calls.

  • CMakeListsEfl.txt:
  • platform/graphics/cairo/FontPlatformDataCairo.cpp: (WebCore::FontPlatformData::FontPlatformData):

2010-07-23 Rafael Antognolli <antognolli@profusion.mobi>

Reviewed by Antonio Gomes.

[EFL] Cleanup glib support (make it optional)
https://bugs.webkit.org/show_bug.cgi?id=42480

Just add glib flags and directories if necessary.

  • CMakeListsEfl.txt:

2010-07-23 Rafael Antognolli <antognolli@profusion.mobi>

Reviewed by Antonio Gomes.

[EFL] Cleanup glib support (make it optional)
https://bugs.webkit.org/show_bug.cgi?id=42480

Put some ifdefs on glib calls.

  • ewk/ewk_main.cpp: (ewk_init):
17:09 Changeset [64004] by jamesr@google.com

2010-07-23 James Robinson <jamesr@chromium.org>

Reviewed by Dimitri Glazkov.

CanvasRenderingContext2D has lots of style errors
https://bugs.webkit.org/show_bug.cgi?id=42918

CanvasRenderingContext2D.h and .cpp have lots of style errors. I'm planning to post patches
to both of these files soon, so to minimize merge conflicts and additional style violations
it seems best to fix all of these errors in bulk before posting any functional changes. In
particular the header file is incorrectly indented so any correctly formatted addition will
be out of place.

  • html/canvas/CanvasRenderingContext2D.cpp: (WebCore::CanvasRenderingContext2D::CanvasRenderingContext2D): (WebCore::CanvasRenderingContext2D::setFillStyle): (WebCore::CanvasRenderingContext2D::transform): (WebCore::CanvasRenderingContext2D::setTransform): (WebCore::CanvasRenderingContext2D::arcTo): (WebCore::CanvasRenderingContext2D::arc): (WebCore::validateRectForCanvas): (WebCore::CanvasRenderingContext2D::strokeRect): (WebCore::CanvasRenderingContext2D::drawImage): (WebCore::CanvasRenderingContext2D::createRadialGradient): (WebCore::CanvasRenderingContext2D::createPattern): (WebCore::CanvasRenderingContext2D::willDraw): (WebCore::CanvasRenderingContext2D::createImageData): (WebCore::CanvasRenderingContext2D::putImageData): (WebCore::CanvasRenderingContext2D::setFont): (WebCore::CanvasRenderingContext2D::drawTextInternal):
  • html/canvas/CanvasRenderingContext2D.h: (WebCore::CanvasRenderingContext2D::is2d): (WebCore::CanvasRenderingContext2D::getLineCap): (WebCore::CanvasRenderingContext2D::getLineJoin): (WebCore::CanvasRenderingContext2D::state): (WebCore::CanvasRenderingContext2D::):
17:07 Changeset [64003] by mrowe@apple.com

Versioning.

17:05 Changeset [64002] by mrowe@apple.com

New tag.

17:03 Changeset [64001] by abarth@webkit.org

2010-07-23 Adam Barth <abarth@webkit.org>

Remove FIXME that we already fixed.

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processCharacter):
15:34 Changeset [64000] by jamesr@google.com

2010-07-23 James Robinson <jamesr@chromium.org>

Unreviewed. Add chromium expectations for new tests.

  • platform/chromium-mac/tables/layering/paint-test-layering-1-expected.checksum:
  • platform/chromium-mac/tables/layering/paint-test-layering-1-expected.png:
  • platform/chromium-mac/tables/layering/paint-test-layering-2-expected.checksum:
  • platform/chromium-mac/tables/layering/paint-test-layering-2-expected.png:
  • platform/chromium-win/tables/layering: Added.
  • platform/chromium-win/tables/layering/paint-test-layering-1-expected.checksum: Added.
  • platform/chromium-win/tables/layering/paint-test-layering-1-expected.png: Added.
  • platform/chromium-win/tables/layering/paint-test-layering-1-expected.txt: Added.
  • platform/chromium-win/tables/layering/paint-test-layering-2-expected.checksum: Added.
  • platform/chromium-win/tables/layering/paint-test-layering-2-expected.png: Added.
  • platform/chromium-win/tables/layering/paint-test-layering-2-expected.txt: Added.
15:17 Changeset [63999] by dglazkov@chromium.org

2010-06-24 Dimitri Glazkov <dglazkov@chromium.org>

Reviewed by Kent Tamura.

Move more form submission logic into FormSubmission.
https://bugs.webkit.org/show_bug.cgi?id=41110

Two major moves:

  • Fold HTMLFormElement::prepareFormSubmission into FormSubmission::create
  • Move non-static (non-FormData-building) methods of FormDataBuilder

to FormSubmission::Attributes.

Mechanical moving, no change in behavior, so no new tests.

  • WebCore.xcodeproj/project.pbxproj: Gave FormDataBuilder.h "project" and

FormSubmission.h "private" visibility.

  • html/HTMLFormElement.cpp: (WebCore::HTMLFormElement::submit): Changed to use FormSubmission::create. (WebCore::HTMLFormElement::parseMappedAttribute): Changed to use m_attributes.
  • html/HTMLFormElement.h: (WebCore::HTMLFormElement::enctype): Ditto. (WebCore::HTMLFormElement::encoding): Ditto. (WebCore::HTMLFormElement::acceptCharset): Ditto.
  • loader/FormSubmission.cpp: (WebCore::generateFormDataIdentifier): Moved from HTMLFormElement. (WebCore::appendMailtoPostFormDataToURL): Ditto. (WebCore::FormSubmission::Attributes::parseAction): Moved from FormDataBuilder. (WebCore::FormSubmission::Attributes::parseEncodingType): Ditto. (WebCore::FormSubmission::Attributes::parseMethodType): Ditto. (WebCore::FormSubmission::FormSubmission): Made inline. (WebCore::FormSubmission::create): Added logic that used to be in

HTMLFormElement::prepareFormSubmission.

  • loader/FormSubmission.h: (WebCore::FormSubmission::): (WebCore::FormSubmission::Attributes::Attributes): Added new class. (WebCore::FormSubmission::Attributes::method): Moved from FormDataBuilder. (WebCore::FormSubmission::Attributes::isMultiPartForm): Ditto. (WebCore::FormSubmission::Attributes::action): Ditto. (WebCore::FormSubmission::Attributes::target): Ditto. (WebCore::FormSubmission::Attributes::setTarget): Ditto. (WebCore::FormSubmission::Attributes::encodingType): Ditto. (WebCore::FormSubmission::Attributes::acceptCharset): Ditto. (WebCore::FormSubmission::Attributes::setAcceptCharset): Ditto. (WebCore::FormSubmission::target): Ditto. (WebCore::FormSubmission::contentType): Const-ified return type. (WebCore::FormSubmission::boundary): Ditto.
  • platform/network/FormDataBuilder.cpp: (WebCore::FormDataBuilder::encodingFromAcceptCharset): Renamed method for clarity,

made it use param instead of a class member.

  • platform/network/FormDataBuilder.h: (WebCore::FormDataBuilder::FormDataBuilder): Made constructor private.
  • wml/WMLGoElement.cpp: Changed to use FormSubmision::Attributes.
  • wml/WMLGoElement.h: Ditto.
15:14 Changeset [63998] by abarth@webkit.org

2010-07-23 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

HTML5 Parser: document.write in a asynchronous script which is specified to load before page finish blows away document
https://bugs.webkit.org/show_bug.cgi?id=42365

  • http/tests/misc/write-from-dom-script-expected.txt: Added.
  • http/tests/misc/write-from-dom-script.html: Added.
    • Test the scenario where a script added to the DOM asynchronously while the nesting level is >0. It's supposed to have its calls to document.write blocked.
  • http/tests/misc/write-while-waiting.html:
    • Writes are only prevented from scripts that are added to the DOM. Scripts that run via setTimeout do not have their writes blocked.

2010-07-23 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

HTML5 Parser: document.write in a asynchronous script which is specified to load before page finish blows away document
https://bugs.webkit.org/show_bug.cgi?id=42365

This patch implements the changes Ian made to HTML5 to address these
issues. Let's try it and see how well it works.

Test: http/tests/misc/write-from-dom-script.html

  • dom/Document.cpp: (WebCore::Document::Document): (WebCore::Document::write):
  • dom/Document.h: (WebCore::Document::writeDisabled): (WebCore::Document::setWriteDisabled):
  • dom/DocumentParser.h: (WebCore::DocumentParser::hasInsertionPoint):
  • dom/ScriptElement.cpp: (WebCore::ScriptElement::insertedIntoDocument): (WebCore::ScriptElementData::ScriptElementData): (WebCore::ScriptElementData::evaluateScript):
  • dom/ScriptElement.h: (WebCore::ScriptElementData::writeDisabled): (WebCore::ScriptElementData::setWriteDisabled):
  • html/HTMLDocumentParser.cpp: (WebCore::HTMLDocumentParser::hasInsertionPoint): (WebCore::HTMLDocumentParser::insert):
  • html/HTMLDocumentParser.h:
14:59 Changeset [63997] by ddkilzer@apple.com

<http://webkit.org/b/42911> Update ruby tools to work with shallow framework bundles

Reviewed by Mark Rowe.

  • Scripts/check-for-inappropriate-files-in-framework: Added

check for the SHALLOW_BUNDLE environment variable so that the
script will work with iOS WebKit builds.

  • Scripts/check-for-webkit-framework-include-consistency: Ditto.
14:46 Changeset [63996] by commit-queue@webkit.org

2010-07-23 Patrick Gansterer <paroga@paroga.com>

Reviewed by Adam Roben.

[WINCE] Buildfix for PopupMenuWin.cpp
https://bugs.webkit.org/show_bug.cgi?id=42906

Use correct API for BitmapInfo landed in r63944.

  • platform/win/PopupMenuWin.cpp: (WebCore::PopupMenu::paint):
14:30 Applications using WebKit edited by onlineservices@thinkingcode.ca
(diff)
14:29 Applications using WebKit edited by onlineservices@thinkingcode.ca
(diff)
13:29 Changeset [63995] by hamaji@chromium.org

2010-07-23 Shinichiro Hamaji <hamaji@chromium.org>

Unreviewed. Land the expectation for win.

Backslash isn't transcoded in <input>
https://bugs.webkit.org/show_bug.cgi?id=42878

  • platform/win/fast/text/backslash-to-yen-sign-euc-expected.txt: Added.
13:22 Changeset [63994] by jamesr@google.com

2010-07-23 fsamuel@chromium.org <fsamuel@chromium.org>

Reviewed by David Hyatt.

Updated table painting to draw cell layers in the correct order.

Table Cell Layering
https://bugs.webkit.org/show_bug.cgi?id=40775

Tests: tables/layering/paint-test-layering-1.html

tables/layering/paint-test-layering-2.html

  • accessibility/AccessibilityTable.cpp: (WebCore::AccessibilityTable::isTableExposableThroughAccessibility): (WebCore::AccessibilityTable::addChildren): (WebCore::AccessibilityTable::cellForColumnAndRow):
  • accessibility/AccessibilityTableCell.cpp: (WebCore::AccessibilityTableCell::titleUIElement):
  • accessibility/AccessibilityTableColumn.cpp: (WebCore::AccessibilityTableColumn::headerObjectForSection):
  • rendering/AutoTableLayout.cpp: (WebCore::AutoTableLayout::recalcColumn):
  • rendering/RenderTable.cpp: (WebCore::RenderTable::splitColumn): (WebCore::RenderTable::calcBorderLeft): (WebCore::RenderTable::calcBorderRight): (WebCore::RenderTable::cellAbove): (WebCore::RenderTable::cellBelow): (WebCore::RenderTable::cellBefore): (WebCore::RenderTable::cellAfter):
  • rendering/RenderTableSection.cpp: (WebCore::RenderTableSection::RenderTableSection): (WebCore::RenderTableSection::ensureRows): (WebCore::RenderTableSection::addCell): (WebCore::RenderTableSection::setCellWidths): (WebCore::RenderTableSection::calcRowHeight): (WebCore::RenderTableSection::layoutRows): (WebCore::RenderTableSection::calcOuterBorderTop): (WebCore::RenderTableSection::calcOuterBorderBottom): (WebCore::RenderTableSection::calcOuterBorderLeft): (WebCore::RenderTableSection::calcOuterBorderRight): (WebCore::RenderTableSection::firstLineBoxBaseline): (WebCore::compareCellPositions): (WebCore::RenderTableSection::paintCell): (WebCore::RenderTableSection::paintObject): (WebCore::RenderTableSection::numColumns): (WebCore::RenderTableSection::appendColumn): (WebCore::RenderTableSection::splitColumn): (WebCore::RenderTableSection::nodeAtPoint):
  • rendering/RenderTableSection.h: (WebCore::RenderTableSection::CellStruct::CellStruct): (WebCore::RenderTableSection::CellStruct::primaryCell): (WebCore::RenderTableSection::CellStruct::hasCells): (WebCore::RenderTableSection::primaryCellAt):

2010-07-23 fsamuel@chromium.org <fsamuel@chromium.org>

Reviewed by David Hyatt.

Updated table painting to draw cell layers in the correct order.

Table Cell Layering
https://bugs.webkit.org/show_bug.cgi?id=40775

  • platform/chromium-linux/tables/layering/paint-test-layering-1-expected.checksum: Added.
  • platform/chromium-linux/tables/layering/paint-test-layering-1-expected.png: Added.
  • platform/chromium-linux/tables/layering/paint-test-layering-1-expected.txt: Added.
  • platform/chromium-linux/tables/layering/paint-test-layering-2-expected.checksum: Added.
  • platform/chromium-linux/tables/layering/paint-test-layering-2-expected.png: Added.
  • platform/chromium-linux/tables/layering/paint-test-layering-2-expected.txt: Added.
  • platform/chromium-mac/tables/layering/paint-test-layering-1-expected.checksum: Added.
  • platform/chromium-mac/tables/layering/paint-test-layering-1-expected.png: Added.
  • platform/chromium-mac/tables/layering/paint-test-layering-1-expected.txt: Added.
  • platform/chromium-mac/tables/layering/paint-test-layering-2-expected.checksum: Added.
  • platform/chromium-mac/tables/layering/paint-test-layering-2-expected.png: Added.
  • platform/chromium-mac/tables/layering/paint-test-layering-2-expected.txt: Added.
  • platform/gtk/tables/layering/paint-test-layering-1-expected.checksum: Added.
  • platform/gtk/tables/layering/paint-test-layering-1-expected.png: Added.
  • platform/gtk/tables/layering/paint-test-layering-1-expected.txt: Added.
  • platform/gtk/tables/layering/paint-test-layering-2-expected.checksum: Added.
  • platform/gtk/tables/layering/paint-test-layering-2-expected.png: Added.
  • platform/gtk/tables/layering/paint-test-layering-2-expected.txt: Added.
  • platform/mac/tables/layering/paint-test-layering-1-expected.checksum: Added.
  • platform/mac/tables/layering/paint-test-layering-1-expected.png: Added.
  • platform/mac/tables/layering/paint-test-layering-1-expected.txt: Added.
  • platform/mac/tables/layering/paint-test-layering-2-expected.checksum: Added.
  • platform/mac/tables/layering/paint-test-layering-2-expected.png: Added.
  • platform/mac/tables/layering/paint-test-layering-2-expected.txt: Added.
  • tables/layering/paint-test-layering-1.html: Added.
  • tables/layering/paint-test-layering-2.html: Added.
13:16 Changeset [63993] by commit-queue@webkit.org

2010-07-23 Patrick Gansterer <paroga@paroga.com>

Reviewed by Adam Roben.

[WINCE] Buildfix for Path.h
https://bugs.webkit.org/show_bug.cgi?id=42904

  • platform/graphics/Path.h: Add missing typedef PlatformPath.
12:51 Changeset [63992] by senorblanco@chromium.org

2010-07-23 Stephen White <senorblanco@chromium.org>

Unreviewed; chromium test expectations fix.

  • platform/chromium-win/fast/borders/different-color-borders-expected.checksum: Added.
  • platform/chromium-win/fast/borders/different-color-borders-expected.png: Added.
  • platform/chromium-win/fast/borders/different-color-borders-expected.txt: Added. New baseline for new test.
  • platform/chromium/test_expectations.txt: Remove passing tests from expectations.
12:08 Changeset [63991] by abarth@webkit.org

2010-07-23 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Reserve capacity for attributes
https://bugs.webkit.org/show_bug.cgi?id=42877

This is about a 1.7% improvement to the parsing benchmark.

  • html/HTMLToken.h: (WebCore::AtomicHTMLToken::AtomicHTMLToken):
11:54 Changeset [63990] by abarth@webkit.org

2010-07-23 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Inline HTMLElementStack::top
https://bugs.webkit.org/show_bug.cgi?id=42879

According to sampling, HTMLElementStack::top is the hotest function in
the treebuilder. Inlining it is about a 0.3% improvement to the
parsing benchmark. This squares with how often it occurs in samples.

  • html/HTMLElementStack.cpp:
  • html/HTMLElementStack.h: (WebCore::HTMLElementStack::top):
11:38 Changeset [63989] by commit-queue@webkit.org

2010-07-23 Patrick Gansterer <paroga@paroga.com>

Reviewed by Adam Roben.

[WINCE] Buildfix for GraphicsContext
https://bugs.webkit.org/show_bug.cgi?id=42888

Use AffineTransform insted of TransformationMatrix and
add ColorSpace parameter to the drawBitmap methods.

  • platform/graphics/GraphicsContext.h:
  • platform/graphics/wince/GraphicsContextWince.cpp: (WebCore::GraphicsContext::affineTransform): (WebCore::GraphicsContext::fillRoundedRect): (WebCore::GraphicsContext::drawBitmap): (WebCore::GraphicsContext::drawBitmapPattern):
11:17 Changeset [63988] by commit-queue@webkit.org

2010-07-23 Rafael Antognolli <antognolli@profusion.mobi>

Reviewed by Antonio Gomes.

[EFL] Add support for using libcurl network backend.
https://bugs.webkit.org/show_bug.cgi?id=42286

Instead of just libsoup, the EFL port now can use libcurl backend.
This is a step in the direction of removing dependency on glib. Just
need to pass the option -DNETWORK_BACKEND=curl to cmake in order to
enable it.

  • cmake/OptionsEfl.cmake:

2010-07-23 Rafael Antognolli <antognolli@profusion.mobi>

Reviewed by Antonio Gomes.

[EFL] Add support for using libcurl network backend.
https://bugs.webkit.org/show_bug.cgi?id=42286

Add files and flags for soup/curl specific backend being
used.
Put some ifdefs around soup/glib calls.

No new tests, no new features implemented.

  • CMakeListsEfl.txt:
  • platform/efl/FileSystemEfl.cpp: (WebCore::fileSystemRepresentation):

2010-07-23 Rafael Antognolli <antognolli@profusion.mobi>

Reviewed by Antonio Gomes.

[EFL] Add support for using libcurl network backend.
https://bugs.webkit.org/show_bug.cgi?id=42286

Just add soup/curl directories and flags for the specific
backend being used.

  • CMakeLists.txt:
  • CMakeListsEfl.txt:

2010-07-23 Rafael Antognolli <antognolli@profusion.mobi>

Reviewed by Antonio Gomes.

[EFL] Add support for using libcurl network backend.
https://bugs.webkit.org/show_bug.cgi?id=42286

Put some ifdefs around soup calls.

  • ewk/ewk_cookies.cpp: (ewk_cookies_file_set): (ewk_cookies_clear): (ewk_cookies_get_all): (ewk_cookies_cookie_del): (ewk_cookies_cookie_free): (ewk_cookies_policy_set): (ewk_cookies_policy_get):
  • ewk/ewk_main.cpp: (ewk_init):
10:47 Changeset [63987] by abarth@webkit.org

2010-07-23 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Add fast-reject path for InputStreamPreprocessor::peek
https://bugs.webkit.org/show_bug.cgi?id=42881

This is about a 2.5% speedup on the parsing benchmark.

  • html/HTMLTokenizer.h: (WebCore::HTMLTokenizer::InputStreamPreprocessor::peek):
10:27 Changeset [63986] by commit-queue@webkit.org

2010-07-23 Hans Wennborg <hans@chromium.org>

Reviewed by Ojan Vafai.

Rebaseline fast/dom/prototype-inheritance.html for chromium
https://bugs.webkit.org/show_bug.cgi?id=42824

Since Chromium revision 53317, ENABLE_DEVICE_ORIENTATION is enabled,
which means window.DeviceOrientationEvent will show up
in this layout test.

  • platform/chromium-mac/fast/dom/prototype-inheritance-expected.txt:
  • platform/chromium-win/fast/dom/prototype-inheritance-expected.txt:
10:11 Changeset [63985] by commit-queue@webkit.org

2010-07-23 Patrick Gansterer <paroga@paroga.com>

Reviewed by Adam Roben.

[WINCE] Merge PasteboardWince.cpp into PasteboardWin.cpp
https://bugs.webkit.org/show_bug.cgi?id=42885

Make PasteboardWin more portable and remove the global namespace prefix
from GlobalLock/GlobalUnlock, since they are only macros on WinCE.

  • platform/win/PasteboardWin.cpp: (WebCore::PasteboardOwnerWndProc): (WebCore::Pasteboard::Pasteboard): (WebCore::Pasteboard::plainText): (WebCore::Pasteboard::documentFragment):
09:41 Changeset [63984] by jschuh@chromium.org

2010-07-23 Justin Schuh <jschuh@chromium.org>

Unreviewed, rolling out r63925.
http://trac.webkit.org/changeset/63925
https://bugs.webkit.org/show_bug.cgi?id=42858

Introduced a regression in history state.

  • page/History.cpp: (WebCore::History::urlForState):

2010-07-23 Justin Schuh <jschuh@chromium.org>

Unreviewed, rolling out r63925.
http://trac.webkit.org/changeset/63925
https://bugs.webkit.org/show_bug.cgi?id=42858

Introduced a regression in history state.

  • fast/history/history-replace-illegal-expected.txt: Removed.
  • fast/history/history-replace-illegal.html: Removed.
09:26 Changeset [63983] by mrobinson@webkit.org

2010-07-23 Martin Robinson <mrobinson@igalia.com>

Unreviewed build fix.

Fix make dist by aligning the source list with currently used headers.

  • GNUmakefile.am:
09:25 Changeset [63982] by jschuh@chromium.org

2010-07-23 Justin Schuh <jschuh@chromium.org>

Unreviewed, rolling out r63959.
http://trac.webkit.org/changeset/63959
https://bugs.webkit.org/show_bug.cgi?id=42884

63925 introduced a regression and is being reverted.

  • platform/chromium-mac/fast/history/history-replace-illegal-expected.txt: Removed.
  • platform/chromium-win/fast/history/history-replace-illegal-expected.txt: Removed.
09:24 Changeset [63981] by commit-queue@webkit.org

2010-07-23 Patrick Gansterer <paroga@paroga.com>

Reviewed by Adam Roben.

[WINCE] Implement TCSpinLock.
https://bugs.webkit.org/show_bug.cgi?id=41792

Implement the SpinLock with InterlockedExchange from the Windows API.

  • wtf/TCSpinLock.h: (TCMalloc_SpinLock::Lock): (TCMalloc_SpinLock::Unlock): (TCMalloc_SpinLock::IsHeld): (TCMalloc_SpinLock::Init): (TCMalloc_SlowLock):
09:10 Changeset [63980] by commit-queue@webkit.org

2010-07-23 Jedrzej Nowacki <jedrzej.nowacki@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

QScriptValue::equals benchmark crash fix.

Patch changes QScriptValue::equals implementation to cover
more edge cases.

Problem exposes an issue in our autotests (all values got
bound to an engine too fast - bug 42366).

[Qt] QScriptValue::equals asserts
https://bugs.webkit.org/show_bug.cgi?id=42363

  • api/qscriptvalue_p.h: (QScriptValuePrivate::equals):
08:57 Changeset [63979] by commit-queue@webkit.org

2010-07-23 Alex Milowski <alex@milowski.com>

Reviewed by Darin Adler.

Moved the 'mathml' group back to where it belongs as
siblings of the other major folders.

  • WebCore.xcodeproj/project.pbxproj:
08:44 Changeset [63978] by commit-queue@webkit.org

2010-07-23 Lucas De Marchi <lucas.demarchi@profusion.mobi>

Reviewed by Kenneth Rohde Christiansen.

Notify browser when window shall be closed.

This is used for window.close() events Javascript, though it would be
triggered for any event on WebCore that might ask for the window to be
closed.

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

  • WebCoreSupport/ChromeClientEfl.cpp: (WebCore::ChromeClientEfl::closeWindowSoon): call new function and moves its previous functionality there.
  • ewk/ewk_private.h:
  • ewk/ewk_view.cpp: (ewk_view_window_close): stop all loaders as was being done previously and notify browser afterwards.
  • ewk/ewk_view.h: declare new virtual method.
08:41 Changeset [63977] by ossy@webkit.org

Unreviewed.

Process link elements immediately if there are no beforeload listeners.
https://bugs.webkit.org/show_bug.cgi?id=42859

Add Qt specific expected result after r63924.

  • platform/qt/http/tests/security/mixedContent: Added.
  • platform/qt/http/tests/security/mixedContent/insecure-css-in-main-frame-expected.txt: Copied from LayoutTests/http/tests/security/mixedContent/insecure-css-in-main-frame-expected.txt and updated.
08:30 Changeset [63976] by commit-queue@webkit.org

2010-07-23 Patrick Gansterer <paroga@paroga.com>

Reviewed by Adam Roben.

Buildfix for WinCE graphics after r56015.
https://bugs.webkit.org/show_bug.cgi?id=41354

Use fabs() insetead of abs().

  • platform/graphics/wince/GraphicsContextWince.cpp: (WebCore::GraphicsContext::fillRect):
  • platform/graphics/wince/PlatformPathWince.cpp: (WebCore::PlatformPath::addArcTo):
07:50 Changeset [63975] by commit-queue@webkit.org

2010-07-23 ryuan choi <ryuan.choi@samsung.com>

Reviewed by Antonio Gomes.

[EFL]build break because of r63927
https://bugs.webkit.org/show_bug.cgi?id=42864

  • CMakeLists.txt:
07:48 Changeset [63974] by hausmann@webkit.org

Doc: Fixed incorrect QML property type.

Patch by David Boddie <dboddie@trolltech.com> on 2010-07-23
Reviewed by Simon Hausmann.

  • declarative/qdeclarativewebview.cpp:
07:36 Changeset [63973] by abecsi@webkit.org

2010-07-23 Andras Becsi <abecsi@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Enable the build of MiniBrowser.

  • WebKit.pro: add MiniBrowser.pro to SUBDIRS.
07:34 Changeset [63972] by commit-queue@webkit.org

2010-07-23 Gyuyoung Kim <gyuyoung.kim@samsung.com>

Reviewed by Kenneth Rohde Christiansen.

[EFL] Set proxy address for Soup
https://bugs.webkit.org/show_bug.cgi?id=42721

Sets the given proxy URI for libsoup network backend. This method will
support libcurl network backend later.

  • EWebLauncher/main.c: (main): Invoke proxy set method
  • ewk/ewk_settings.cpp: (ewk_settings_proxy_uri_set): Sets proxy URI.
  • ewk/ewk_settings.h:
07:27 Changeset [63971] by hausmann@webkit.org

[Qt] Build fix for Qt apps

Add webkit to QT_CONFIG. qconfig.pri will read qt_webkit_version.pri and
that's how it will pick up webkit in QT_CONFIG.

  • qt_webkit_version.pri:
07:19 Changeset [63970] by abarth@webkit.org

2010-07-23 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Inline HTMLTokenizer::InputStreamPreprocessor::peek
https://bugs.webkit.org/show_bug.cgi?id=42880

According to sampling HTMLTokenizer::InputStreamPreprocessor::peek is
the hotest function in the tokenizer. Inlining this function is about
a 2.2% speedup on the parser benchmark. Turns out the tokenizer is
much hotter than the tree builder. :)

  • html/HTMLTokenizer.h: (WebCore::HTMLTokenizer::InputStreamPreprocessor::peek):
07:04 Changeset [63969] by abecsi@webkit.org

2010-07-23 Andras Becsi <abecsi@webkit.org>

Unreviewed build fix.

[Qt] Fix WebKit2 linkage.

No new tests needed.

  • WebCore.pro: add InjectedBundlePageEditorClient.cpp to WK2 sources
06:39 Changeset [63968] by abecsi@webkit.org

2010-07-23 Balazs Kelemen <kb@inf.u-szeged.hu>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Setup the QtWebProcess

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

  • WebCore.pro: Added WebProcessMainQt.cpp to SOURCES.

2010-07-23 Balazs Kelemen <kb@inf.u-szeged.hu>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Setup the QtWebProcess

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

  • WebKit.pro: Add WebKit2.pro to the build guarded by webkit2 config variable.

2010-07-19 Balazs Kelemen <kb@inf.u-szeged.hu>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Setup the QtWebProcess

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

  • UIProcess/Launcher/qt/ProcessLauncherQt.cpp: Remove the framework entry point from here to WebProcess/qt/WebProcessMainQt.cpp to match with with the mac and the win port.
  • WebKit2.pro: Added.
  • WebProcess/qt/WebProcessMainQt.cpp: Added. (WebKit::WebProcessMainQt): Framework entry point for Qt.
  • qt/MainQt.cpp: Added. (main):
06:19 Changeset [63967] by pfeldman@chromium.org

2010-07-23 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: Chromium tests fix.

  • inspector/CodeGeneratorInspector.pm:
05:54 Changeset [63966] by commit-queue@webkit.org

2010-07-23 Patrick Gansterer <paroga@paroga.com>

Reviewed by Adam Roben.

[WINCE] Buildfix for PluginDatabaseWin.cpp
https://bugs.webkit.org/show_bug.cgi?id=41887

Define KEY_ENUMERATE_SUB_KEYS only if not defined.

  • plugins/win/PluginDatabaseWin.cpp:
05:46 Changeset [63965] by pfeldman@chromium.org

2010-07-23 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Chromium DevTools: migrate from utility context-based injected
script dispatch to upstream schema.

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

  • bindings/v8/ScriptProfile.cpp: (WebCore::ScriptProfile::title):
  • inspector/InspectorController.cpp: (WebCore::InspectorController::getProfile):
  • inspector/front-end/InjectedScriptAccess.js:

2010-07-23 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Reviewed by Yury Semikhatsky.

Chromium DevTools: migrate from utility context-based injected
script dispatch to upstream schema.

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

  • src/ToolsAgent.h:
  • src/WebDevToolsAgentImpl.cpp:
  • src/WebDevToolsAgentImpl.h:
  • src/js/DevToolsHostStub.js:
  • src/js/InspectorControllerImpl.js: (devtools.InspectorBackendImpl):
05:20 Changeset [63964] by pfeldman@chromium.org

2010-07-23 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: rolling out r63963.

  • bindings/v8/ScriptProfile.cpp: (WebCore::ScriptProfile::title):
  • inspector/InspectorController.cpp: (WebCore::InspectorController::getProfile):
  • inspector/front-end/InjectedScriptAccess.js:
05:12 Changeset [63963] by pfeldman@chromium.org

2010-07-23 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Chromium DevTools: migrate from utility context-based injected
script dispatch to upstream schema.

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

  • bindings/v8/ScriptProfile.cpp: (WebCore::ScriptProfile::title):
  • inspector/InspectorController.cpp: (WebCore::InspectorController::getProfile):
  • inspector/front-end/InjectedScriptAccess.js: (InjectedScriptAccess.get if): (InjectedScriptAccess):

2010-07-23 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Reviewed by Yury Semikhatsky.

Chromium DevTools: migrate from utility context-based injected
script dispatch to upstream schema.

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

  • src/ToolsAgent.h:
  • src/WebDevToolsAgentImpl.cpp:
  • src/WebDevToolsAgentImpl.h:
  • src/js/DevToolsHostStub.js:
  • src/js/InspectorControllerImpl.js: (devtools.InspectorBackendImpl):
04:58 Changeset [63962] by yurys@chromium.org

2010-07-23 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: error info propagation in InspectorBackendDispatch should be improved
https://bugs.webkit.org/show_bug.cgi?id=42873

  • inspector/CodeGeneratorInspector.pm: report wrong parameter count and wrong parametere type errror to the dispatch method caller
04:42 Changeset [63961] by pfeldman@chromium.org

2010-07-23 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: injectedScriptId is missing in the stack frame wrappers.

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

  • inspector/front-end/InjectedScript.js: (injectedScriptConstructor):
04:24 Changeset [63960] by ossy@webkit.org

Unreviewed.

[Qt] Search input field doesn't have cancel button
https://bugs.webkit.org/show_bug.cgi?id=42887

  • platform/qt/Skipped: fast/forms/search-cancel-button-events.html skipped until fix.
03:02 Changeset [63959] by bulach@chromium.org

2010-07-23 Marcus Bulach <bulach@chromium.org>

Unreviewed. Adds expectations for chromium.

[chromium] Updates baselines following changes in history (http://trac.webkit.org/changeset/63925)
https://bugs.webkit.org/show_bug.cgi?id=42884

  • platform/chromium-mac/fast/history/history-replace-illegal-expected.txt: Added.
  • platform/chromium-win/fast/history/history-replace-illegal-expected.txt: Added.
02:02 Changeset [63958] by bulach@chromium.org

2010-07-22 Marcus Bulach <bulach@chromium.org>

Reviewed by Dimitri Glazkov.

[chromium] Updates baselines following changes in border drawing code (http://trac.webkit.org/changeset/63864).
https://bugs.webkit.org/show_bug.cgi?id=42822

  • platform/chromium-linux/fast/borders/different-color-borders-expected.checksum: Added.
  • platform/chromium-linux/fast/borders/different-color-borders-expected.png: Added.
  • platform/chromium-linux/fast/borders/different-color-borders-expected.txt: Added.
  • platform/chromium-mac/fast/borders/border-radius-constraints-expected.checksum:
  • platform/chromium-mac/fast/borders/border-radius-constraints-expected.png:
  • platform/chromium-mac/fast/borders/border-radius-huge-assert-expected.checksum:
  • platform/chromium-mac/fast/borders/border-radius-huge-assert-expected.png:
  • platform/chromium-mac/fast/borders/border-radius-split-inline-expected.checksum:
  • platform/chromium-mac/fast/borders/border-radius-split-inline-expected.png:
  • platform/chromium-mac/fast/borders/borderRadiusAllStylesAllCorners-expected.checksum:
  • platform/chromium-mac/fast/borders/borderRadiusAllStylesAllCorners-expected.png:
  • platform/chromium-mac/fast/borders/borderRadiusArcs01-expected.checksum: Removed.
  • platform/chromium-mac/fast/borders/borderRadiusArcs01-expected.png:
  • platform/chromium-mac/fast/borders/borderRadiusDashed01-expected.checksum:
  • platform/chromium-mac/fast/borders/borderRadiusDashed01-expected.png:
  • platform/chromium-mac/fast/borders/borderRadiusDashed04-expected.checksum:
  • platform/chromium-mac/fast/borders/borderRadiusDashed04-expected.png: Added.
  • platform/chromium-mac/fast/borders/borderRadiusDotted06-expected.checksum:
  • platform/chromium-mac/fast/borders/borderRadiusDotted06-expected.png: Added.
  • platform/chromium-mac/fast/borders/borderRadiusDouble02-expected.checksum: Removed.
  • platform/chromium-mac/fast/borders/borderRadiusDouble02-expected.png:
  • platform/chromium-mac/fast/borders/borderRadiusDouble03-expected.checksum: Removed.
  • platform/chromium-mac/fast/borders/borderRadiusDouble03-expected.png:
  • platform/chromium-mac/fast/borders/borderRadiusGroove01-expected.checksum: Removed.
  • platform/chromium-mac/fast/borders/borderRadiusGroove01-expected.png:
  • platform/chromium-mac/fast/borders/borderRadiusGroove02-expected.checksum: Removed.
  • platform/chromium-mac/fast/borders/borderRadiusGroove02-expected.png:
  • platform/chromium-mac/fast/borders/borderRadiusInset01-expected.checksum: Removed.
  • platform/chromium-mac/fast/borders/borderRadiusInset01-expected.png:
  • platform/chromium-mac/fast/borders/borderRadiusInvalidColor-expected.checksum:
  • platform/chromium-mac/fast/borders/borderRadiusInvalidColor-expected.png:
  • platform/chromium-mac/fast/borders/borderRadiusOutset01-expected.checksum: Removed.
  • platform/chromium-mac/fast/borders/borderRadiusOutset01-expected.png:
  • platform/chromium-mac/fast/borders/borderRadiusRidge01-expected.checksum: Removed.
  • platform/chromium-mac/fast/borders/borderRadiusRidge01-expected.png:
  • platform/chromium-mac/fast/borders/borderRadiusSolid01-expected.checksum: Removed.
  • platform/chromium-mac/fast/borders/borderRadiusSolid01-expected.png:
  • platform/chromium-mac/fast/borders/borderRadiusSolid02-expected.checksum: Removed.
  • platform/chromium-mac/fast/borders/borderRadiusSolid02-expected.png:
  • platform/chromium-mac/fast/borders/borderRadiusSolid03-expected.checksum: Removed.
  • platform/chromium-mac/fast/borders/borderRadiusSolid03-expected.png:
  • platform/chromium-mac/fast/borders/borderRadiusSolid04-expected.checksum: Removed.
  • platform/chromium-mac/fast/borders/borderRadiusSolid04-expected.png:
  • platform/chromium-mac/fast/borders/different-color-borders-expected.checksum: Added.
  • platform/chromium-mac/fast/borders/different-color-borders-expected.png: Added.
  • platform/chromium-mac/fast/borders/fieldsetBorderRadius-expected.checksum: Removed.
  • platform/chromium-mac/fast/borders/fieldsetBorderRadius-expected.png:
  • platform/chromium-mac/fast/css/nested-rounded-corners-expected.checksum: Added.
  • platform/chromium-mac/fast/css/nested-rounded-corners-expected.png: Added.
  • platform/chromium-mac/fast/forms/basic-selects-expected.checksum:
  • platform/chromium-mac/fast/forms/basic-selects-expected.png:
  • platform/chromium-mac/fast/forms/menulist-clip-expected.checksum:
  • platform/chromium-mac/fast/forms/menulist-clip-expected.png:
  • platform/chromium-mac/fast/forms/menulist-narrow-width-expected.checksum:
  • platform/chromium-mac/fast/forms/menulist-narrow-width-expected.png:
  • platform/chromium-mac/fast/forms/menulist-option-wrap-expected.checksum:
  • platform/chromium-mac/fast/forms/menulist-option-wrap-expected.png:
  • platform/chromium-mac/fast/forms/menulist-restrict-line-height-expected.checksum:
  • platform/chromium-mac/fast/forms/menulist-restrict-line-height-expected.png:
  • platform/chromium-mac/fast/forms/menulist-separator-painting-expected.checksum:
  • platform/chromium-mac/fast/forms/menulist-separator-painting-expected.png:
  • platform/chromium-mac/fast/forms/menulist-style-color-expected.checksum:
  • platform/chromium-mac/fast/forms/menulist-style-color-expected.png:
  • platform/chromium-mac/fast/forms/select-baseline-expected.checksum:
  • platform/chromium-mac/fast/forms/select-baseline-expected.png:
  • platform/chromium-mac/fast/overflow/border-radius-clipping-expected.checksum:
  • platform/chromium-mac/fast/overflow/border-radius-clipping-expected.png:
  • platform/chromium-mac/svg/custom/svg-fonts-in-html-expected.checksum:
  • platform/chromium-mac/svg/custom/svg-fonts-in-html-expected.png:
  • platform/chromium-win-xp/fast/css/nested-rounded-corners-expected.png: Added.
  • platform/chromium-win/fast/css/nested-rounded-corners-expected.checksum: Added.
  • platform/chromium-win/fast/css/nested-rounded-corners-expected.png: Added.
  • platform/chromium/test_expectations.txt:
01:37 Changeset [63957] by hamaji@chromium.org

2010-07-23 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by Kent Tamura.

Backslash isn't transcoded in <input>
https://bugs.webkit.org/show_bug.cgi?id=42878

  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::applyProperty):

2010-07-23 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by Kent Tamura.

Backslash isn't transcoded in <input>
https://bugs.webkit.org/show_bug.cgi?id=42878

  • editing/selection/find-yensign-and-backslash-expected.txt:
  • platform/mac/fast/text/backslash-to-yen-sign-euc-expected.checksum:
  • platform/mac/fast/text/backslash-to-yen-sign-euc-expected.png:
  • platform/mac/fast/text/backslash-to-yen-sign-euc-expected.txt:
00:28 Changeset [63956] by tkent@chromium.org

2010-07-23 Kent Tamura <tkent@chromium.org>

Reviewed by Shinichiro Hamaji.

Roll Chromium revision
https://bugs.webkit.org/show_bug.cgi?id=42876

  • DEPS: Roll Chromium revision to r53434 in order to have the logger initialization change (r53303)
00:17 Changeset [63955] by hamaji@chromium.org

2010-07-23 Shinichiro Hamaji <hamaji@chromium.org>

Unreviewed. Put a failing result for now.

Backslash is transcoded into yen sign even when non japanese font is specified
https://bugs.webkit.org/show_bug.cgi?id=39654

  • editing/selection/find-yensign-and-backslash-expected.txt:
00:03 Changeset [63954] by ossy@webkit.org

Unreviewed rolling out r63947 and r63948, because they broke Qt Windows build.

  • jit/JITStubs.cpp:
  • jit/JITStubs.h:

07/22/10:

23:47 Changeset [63953] by philn@webkit.org

2010-07-22 Philippe Normand <pnormand@igalia.com>

Reviewed by Eric Carlson.

media/video-duration-known-after-eos.html test is slow
https://bugs.webkit.org/show_bug.cgi?id=42809

Seek near the end of the media to speed the test up.

  • media/video-duration-known-after-eos-expected.txt:
  • media/video-duration-known-after-eos.html:
23:39 Changeset [63952] by loislo@chromium.org

2010-07-22 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Yury Semikhatsky.

WebInspector: CodeGeneratorInspector was extended and now it can
generate InspectorBackedDispatcher's code. This dispatcher will
validate and dispatch incoming inspector commands only by native
code without using javaScript. That is necessary step for
RemoteDebugging support.
https://bugs.webkit.org/show_bug.cgi?id=42588

  • GNUmakefile.am:
  • WebCore.gyp/WebCore.gyp:
  • WebCore.gypi:
  • WebCore.pri:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • inspector/CodeGeneratorInspector.pm:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::InspectorController):
  • inspector/InspectorController.h: (WebCore::InspectorController::inspectorBackendDispatcher):
  • inspector/front-end/ElementsPanel.js: (WebInspector.ElementsPanel.prototype.setDocument):

2010-07-22 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Yury Semikhatsky.

WebInspector: CodeGeneratorInspector was extended and now it can
generate InspectorBackedDispatcher's code. This dispatcher will
validate and dispatch incoming inspector commands only by native
code without using javaScript. That is necessary step for
RemoteDebugging support.
https://bugs.webkit.org/show_bug.cgi?id=42588

  • src/ToolsAgent.h:
  • src/WebDevToolsAgentImpl.cpp: (WebKit::WebDevToolsAgentImpl::dispatchOnInspectorController):
  • src/WebDevToolsAgentImpl.h:
  • src/js/InspectorControllerImpl.js: (devtools.InspectorBackendImpl.prototype.callInspectorController_):
23:26 Changeset [63951] by hamaji@chromium.org

2010-07-22 Shinichiro Hamaji <hamaji@chromium.org>

Unreviewed. Add expectations for Qt and Gtk.

Backslash is transcoded into yen sign even when non japanese font is specified
https://bugs.webkit.org/show_bug.cgi?id=39654

  • platform/gtk/fast/text/backslash-to-yen-sign-euc-expected.txt: Added.
  • platform/qt/fast/text/backslash-to-yen-sign-euc-expected.txt: Added.
22:39 Changeset [63950] by hamaji@chromium.org

2010-07-22 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by Kent Tamura.

Backslash is transcoded into yen sign even when non japanese font is specified
https://bugs.webkit.org/show_bug.cgi?id=39654

  • fast/text/backslash-to-yen-sign-euc.html: Added.
  • platform/chromium/test_expectations.txt:
  • platform/mac/fast/text/backslash-to-yen-sign-euc-expected.checksum: Added.
  • platform/mac/fast/text/backslash-to-yen-sign-euc-expected.png: Added.
  • platform/mac/fast/text/backslash-to-yen-sign-euc-expected.txt: Added.

2010-07-22 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by Kent Tamura.

Backslash is transcoded into yen sign even when non japanese font is specified
https://bugs.webkit.org/show_bug.cgi?id=39654

Test: fast/text/backslash-to-yen-sign-euc.html

  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::applyProperty): Call FontDescriptor::setIsSpecifiedFont.
  • platform/graphics/Font.cpp: (WebCore::Font::Font):
  • platform/graphics/FontDescription.h: Added m_isSpecifiedFont and its accessors. (WebCore::FontDescription::FontDescription): (WebCore::FontDescription::isSpecifiedFont): (WebCore::FontDescription::setIsSpecifiedFont): (WebCore::FontDescription::operator==):
  • platform/text/transcoder/FontTranscoder.cpp: (WebCore::FontTranscoder::converterType): Don't transcode backslashes if a font family is explicitly specified. (WebCore::FontTranscoder::convert): (WebCore::FontTranscoder::needsTranscoding):
  • platform/text/transcoder/FontTranscoder.h: Change the arguments from AtomicString fontFamily to Font.
  • rendering/RenderText.cpp: (WebCore::RenderText::updateNeedsTranscoding): (WebCore::RenderText::setTextInternal):
20:46 Changeset [63949] by darin@apple.com

Eliminate unneeded WKBundleFrameCopyInnerText function from WebKit2
https://bugs.webkit.org/show_bug.cgi?id=42847

Reviewed by Sam Weinig.

WebKit2:

  • WebProcess/InjectedBundle/API/c/WKBundleFrame.cpp: Removed WKBundleFrameCopyInnerText.
  • WebProcess/InjectedBundle/API/c/WKBundleFrame.h: Ditto.
  • WebProcess/WebPage/WebFrame.cpp: Removed innerText.
  • WebProcess/WebPage/WebFrame.h: Ditto.

WebKitTools:

  • WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp:

(WTR::operator<<): Replaced the WKStringToUTF8 function with
a set of overloaded stream operators.
(WTR::dumpPath): Changed to use ostringstream and return a string.
(WTR::propertyValue): Added.
(WTR::propertyObject): Added.
(WTR::propertyString): Added.
(WTR::numericWindowPropertyValue): Changed to call propertyValue.
(WTR::dumpFrameScrollPosition): Removed now unneeded WKStringToUTF8 sequence
since we now have suitable streaming functions.
(WTR::dumpFrameText): Use propertyObject and propertyString instead of
WKBundleFrameCopyInnerText.
(WTR::dumpDescendantFramesText): Removed now unneeded WKStringToUTF8 sequence
since we now have suitable streaming functions.
(WTR::InjectedBundlePage::dump): Ditto.
(WTR::InjectedBundlePage::willAddMessageToConsole): Ditto.
(WTR::InjectedBundlePage::willSetStatusbarText): Ditto.
(WTR::InjectedBundlePage::willRunJavaScriptAlert): Ditto.
(WTR::InjectedBundlePage::willRunJavaScriptConfirm): Ditto.
(WTR::InjectedBundlePage::willRunJavaScriptPrompt): Ditto.

20:43 Changeset [63948] by barraclough@apple.com

Eeeep! r63947 hosed all non-x86 builds!

  • jit/JITStubs.h:
20:28 Changeset [63947] by barraclough@apple.com

Bug 42818 - [Qt] REGRESSION(63348): jsc is broken
Speculative fix, need fastcall conventions on Qt/Win.

Reviewed by Oliver Hunt.

  • jit/JITStubs.cpp:
  • jit/JITStubs.h:
20:24 Changeset [63946] by oliver@apple.com

2010-07-22 Oliver Hunt <oliver@apple.com>

Reviewed by Gavin Barraclough.

Do more constant folding
https://bugs.webkit.org/show_bug.cgi?id=42867

Constant fold a few more operations. SunSpider says this is
a win but I suspect that's just code motion at play.

  • parser/ASTBuilder.h: (JSC::ASTBuilder::makeModNode): (JSC::ASTBuilder::makeURightShiftNode): (JSC::ASTBuilder::makeBitOrNode): (JSC::ASTBuilder::makeBitAndNode): (JSC::ASTBuilder::makeBitXOrNode): (JSC::ASTBuilder::makeBinaryNode):
20:20 Changeset [63945] by jhoneycutt@apple.com

Windows build fix.

Commit some changes missed by the commit-queue in r63944.

  • platform/win/BitmapInfo.h:

(WebCore::BitmapInfo::is16bit):
(WebCore::BitmapInfo::is32bit):
(WebCore::BitmapInfo::width):
(WebCore::BitmapInfo::height):
(WebCore::BitmapInfo::size):
(WebCore::BitmapInfo::paddedWidth):
(WebCore::BitmapInfo::numPixels):
(WebCore::BitmapInfo::paddedBytesPerLine):
(WebCore::BitmapInfo::bytesPerLine):

19:55 Changeset [63944] by commit-queue@webkit.org

2010-07-22 Patrick Gansterer <paroga@paroga.com>

Reviewed by Adam Roben.

[WINCE] Add additonal methods to BitmapInfo.
https://bugs.webkit.org/show_bug.cgi?id=42071

Added a parameter to create 16bit BitmapInfo and
some additional accessors for the WinCE port.

Set bmiHeader.biSize only at constructor.

  • platform/win/BitmapInfo.cpp: Added property svn:eol-style. (WebCore::bitmapInfoForSize): (WebCore::BitmapInfo::create): (WebCore::BitmapInfo::createBottomUp):
  • platform/win/BitmapInfo.h: Added property svn:eol-style. (WebCore::BitmapInfo::is16bit): (WebCore::BitmapInfo::is32bit): (WebCore::BitmapInfo::width): (WebCore::BitmapInfo::height): (WebCore::BitmapInfo::size): (WebCore::BitmapInfo::paddedWidth): (WebCore::BitmapInfo::numPixels): (WebCore::BitmapInfo::paddedBytesPerLine): (WebCore::BitmapInfo::bytesPerLine):
19:36 Changeset [63943] by morrita@google.com

2010-07-22 MORITA Hajime <morrita@google.com>

Reviewed by Ojan Vafai.

Autoscroll on selection should take scrollbars into account as outside of the content.
https://bugs.webkit.org/show_bug.cgi?id=40403

ScrollView's scroll-bar was considered as a part of content rect,
that prevents autoscroll to happen when the pointer is over the
scroll-bar. This change excluded scroll-bar region from scrolling
delta calculation.

Test: WebCore/manual-tests/autoscroll-over-scrollbar.html

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::scrollRectToVisible):
19:29 Changeset [63942] by tkent@chromium.org

maxLength should not be applied to non-text types
https://bugs.webkit.org/show_bug.cgi?id=42814

Reviewed by Darin Adler.

WebCore:

According to the HTML5 specification, maxLength attribute should
be applied to text, search, url, tel, email, and password types.

Test: fast/forms/input-maxlength-unsupported.html

  • dom/InputElement.cpp:

(WebCore::InputElement::handleBeforeTextInsertedEvent):

Apply maxLength() only if supportsMaxLength() is true.

  • dom/InputElement.h:
  • html/HTMLInputElement.cpp:

(WebCore::HTMLInputElement::patternMismatch):

Code cleanup by isTextType().

(WebCore::HTMLInputElement::tooLong):

ditto.

(WebCore::HTMLInputElement::isTextType):

Added. This returns true for types with maxLength/pattern/placeholder.

  • html/HTMLInputElement.h:

(WebCore::HTMLInputElement::supportsMaxLength):

Added. Just calls isTextType().

(WebCore::HTMLInputElement::supportsPlaceholder):

Code cleanup by isTextType().

  • wml/WMLInputElement.h:

(WebCore::WMLInputElement::supportsMaxLength):

Added. Always returns true.

LayoutTests:

  • fast/forms/input-maxlength-unsupported-expected.txt: Added.
  • fast/forms/input-maxlength-unsupported.html: Added.
  • fast/forms/script-tests/input-maxlength-unsupported.js: Added.
19:26 Changeset [63941] by rniwa@webkit.org

[cleanup] logic to extract adjacent lists and list children in listifyParagraph and unlistifyParagraph should be isolated
https://bugs.webkit.org/show_bug.cgi?id=42841

Reviewed by Kent Tamura.

Added enclosingListChild that takes the list node containing the node for the second argument
to ensure that it only returns the enclosing list child of the specified list node.

Added adjacentEnclosingList that finds adjacent lists of the specified position.
It takes the current position, the adjacent position that may or may not have a list, and the list type,
and avoids a list that belongs to a different table cell or an outer list that contains the current position.

No new tests needed.

  • editing/InsertListCommand.cpp:

(WebCore::enclosingListChild): Added. See above.
(WebCore::InsertListCommand::unlistifyParagraph): Uses enclosingListChild.
(WebCore::adjacentEnclosingList): Added. See above.
(WebCore::InsertListCommand::listifyParagraph): Uses adjacentEnclosingList.

19:15 Changeset [63940] by commit-queue@webkit.org

2010-07-22 Patrick Gansterer <paroga@paroga.com>

Reviewed by Adam Roben.

[WIN] Correct DeleteObject usage in PasteboardWin
https://bugs.webkit.org/show_bug.cgi?id=41956

Call DeleteObject() if OpenClipboard() fails and don't
call DeleteObject() on objects returned from SelectObject().

  • platform/win/PasteboardWin.cpp: (WebCore::Pasteboard::writeImage):
19:02 Changeset [63939] by commit-queue@webkit.org

2010-07-22 Lucas De Marchi <lucas.demarchi@profusion.mobi>

Reviewed by Antonio Gomes.

[EFL] Log function is incorrectly used.
r63804 used LOG function, but it should receive one more parameter.
Used LOG_ERROR instead, since this is an error message.
https://bugs.webkit.org/show_bug.cgi?id=42842

  • platform/efl/WidgetEfl.cpp: (WebCore::Widget::applyFallbackCursor): use LOG_ERROR in place of LOG
18:56 Changeset [63938] by ap@apple.com

More Windows build fixing.

  • win/WebKit2Generated.make: Copy the new public headers to $(WEBKITOUTPUTDIR)\include\WebKit2.
18:30 Changeset [63937] by ap@apple.com

Windows build fix.

  • win/WebKit2.vcproj: Added new files.
18:24 Changeset [63936] by mrowe@apple.com

Versioning.

18:24 Changeset [63935] by commit-queue@webkit.org

2010-07-22 Cosmin Truta <ctruta@chromium.org>

Reviewed by Darin Adler.

SVG: Temporarily disable the layout test for the bug 41175 on Leopard.
https://bugs.webkit.org/show_bug.cgi?id=42802

  • platform/mac-leopard/Skipped: svg/filters/filter-empty-g.svg skipped until fix.
18:23 Changeset [63934] by mrowe@apple.com

New tag.

18:11 Changeset [63933] by commit-queue@webkit.org

2010-07-22 Lucas De Marchi <lucas.demarchi@profusion.mobi>

Reviewed by Antonio Gomes.

[EFL] Implement input method notification
https://bugs.webkit.org/show_bug.cgi?id=42640

Notify browser when keyboard should be shown/hidden. Input method
hints are updated before sending the signal. Client should be able to
determine the input type by calling ewk_view_imh_get().

  • efl/EWebLauncher/main.c: (on_inputmethod_changed): example implementation that just prints to stdout if keyboard should be shown or hidden and the imh flags. (browserCreate): listen to signal about input method changing its state.
  • efl/WebCoreSupport/EditorClientEfl.cpp: (WebCore::EditorClientEfl::setInputMethodState): call new function responsible for implementing this notification.
  • efl/ewk/ewk_private.h: ewk_view_input_method_state_set() is called only from inside WebKit.
  • efl/ewk/ewk_view.cpp: implement setters and getters (ewk_view_imh_get): (ewk_view_input_method_state_set):
  • efl/ewk/ewk_view.h: introduce Ewk_Imh enum which contains the possible input types. (_Ewk_View_Smart_Class::):
17:59 Changeset [63932] by johnnyg@google.com

2010-07-22 John Gregg <johnnyg@google.com>

Unreviewed; chromium test expectation change.

fast/forms/input-file-directory-upload.html passes on windows after r63914
https://bugs.webkit.org/show_bug.cgi?id=42768

  • platform/chromium/test_expectations.txt:
17:53 Changeset [63931] by mrowe@apple.com

Merge r63927.

17:49 Changeset [63930] by ap@apple.com

Reviewed by Sam Weinig and Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=42193
Support layoutTestController.dumpEditingDelegates in WebKitTestRunner

Step 1: Add the method, and implement one actual delegate call as proof of concept. No tests
fixed, but this makes difference one line smaller on many editing tests.

  • WebKit2.xcodeproj/project.pbxproj:
  • WebProcess/InjectedBundle/API/c/WKBundleAPICast.h:
  • WebProcess/InjectedBundle/API/c/WKBundleBase.h:
  • WebProcess/InjectedBundle/API/c/WKBundleNode.cpp: Added. (WKBundleNodeCopyNodeName): (WKBundleNodeGetParent):
  • WebProcess/InjectedBundle/API/c/WKBundleNode.h: Added.
  • WebProcess/InjectedBundle/API/c/WKBundlePage.cpp: (WKBundlePageSetEditorClient):
  • WebProcess/InjectedBundle/API/c/WKBundlePage.h:
  • WebProcess/InjectedBundle/API/c/WKBundleRange.cpp: Added. (WKBundleRangeGetStartOffset): (WKBundleRangeGetStartContainer): (WKBundleRangeGetEndOffset): (WKBundleRangeGetEndContainer):
  • WebProcess/InjectedBundle/API/c/WKBundleRange.h: Added. Added WKBundleRangeRef and WKBundleNodeRef types for injected bundles. These are just pointers to respective WebCore objects, without any WebKit2 wrappers.
  • WebProcess/InjectedBundle/InjectedBundlePageEditorClient.cpp: Added. (WebKit::InjectedBundlePageEditorClient::InjectedBundlePageEditorClient): (WebKit::InjectedBundlePageEditorClient::initialize): (WebKit::InjectedBundlePageEditorClient::shouldBeginEditing):
  • WebProcess/InjectedBundle/InjectedBundlePageEditorClient.h: Added.
  • WebProcess/WebCoreSupport/WebEditorClient.cpp: (WebKit::WebEditorClient::shouldBeginEditing):
  • WebProcess/WebPage/WebPage.cpp: (WebKit::WebPage::initializeInjectedBundleEditorClient):
  • WebProcess/WebPage/WebPage.h: (WebKit::WebPage::injectedBundleEditorClient): Added the plumbing for invoking injected bundle code for editor client. There is no real API for editor client yet.

WebKitTools:

  • WebKitTestRunner/InjectedBundle/Bindings/LayoutTestController.idl: Added dumpEditingCallbacks() and setAcceptsEditing().
  • WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp: (WTR::dumpPath): Dump path to a node in a format that's compatible with DumpRenderTree. (WTR::operator<<): Print a range. (WTR::InjectedBundlePage::InjectedBundlePage): Set editor client in addition to existing clients. (WTR::InjectedBundlePage::_shouldBeginEditing): The only client method implemented so far. (WTR::InjectedBundlePage::shouldBeginEditing): Ditto.
  • WebKitTestRunner/InjectedBundle/InjectedBundlePage.h: Added a section for editor client calls.
  • WebKitTestRunner/InjectedBundle/LayoutTestController.cpp: (WTR::LayoutTestController::LayoutTestController):
  • WebKitTestRunner/InjectedBundle/LayoutTestController.h: (WTR::LayoutTestController::dumpEditingCallbacks): (WTR::LayoutTestController::setAcceptsEditing): (WTR::LayoutTestController::acceptsEditing): (WTR::LayoutTestController::shouldDumpEditingCallbacks): Store m_acceptsEditing and m_dumpEditingCallbacks.
17:43 Changeset [63929] by tkent@chromium.org

2010-07-22 Kent Tamura <tkent@chromium.org>

Reviewed by Ojan Vafai.

Cancel button of search input field should fire input/change events
https://bugs.webkit.org/show_bug.cgi?id=34393

  • fast/forms/script-tests/search-cancel-button-events.js: Added.
  • fast/forms/search-cancel-button-events-expected.txt: Added.
  • fast/forms/search-cancel-button-events.html: Added.

2010-07-22 Kent Tamura <tkent@chromium.org>

Reviewed by Ojan Vafai.

Cancel button of search input field should fire input/change events
https://bugs.webkit.org/show_bug.cgi?id=34393

Test: fast/forms/search-cancel-button-events.html

  • rendering/TextControlInnerElements.cpp: (WebCore::SearchFieldCancelButtonElement::defaultEventHandler):
17:37 Changeset [63928] by mrowe@apple.com

Merge r63924.

17:37 Changeset [63927] by weinig@apple.com

Fix for <rdar://problem/8222626>
Send textDidChangeInTextField delegate callback only in response to typing or other forms of user text input.

Reviewed by Maciej Stachowiak.

WebCore:

  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:

Update project files.

  • dom/UserTypingGestureIndicator.cpp: Copied from dom/UserGestureIndicator.cpp.

(WebCore::UserTypingGestureIndicator::processingUserTypingGesture):
(WebCore::focusedNode):
(WebCore::UserTypingGestureIndicator::focusedElementAtGestureStart):
(WebCore::UserTypingGestureIndicator::UserTypingGestureIndicator):
(WebCore::UserTypingGestureIndicator::~UserTypingGestureIndicator):

  • dom/UserTypingGestureIndicator.h: Copied from dom/UserGestureIndicator.h.

Version of UserGestureIndicator that just tracks keyboard gestures and the focused
node they were targeting.

  • editing/Editor.cpp:

(WebCore::Editor::confirmComposition):
(WebCore::Editor::setComposition):

  • page/EventHandler.cpp:

(WebCore::EventHandler::keyEvent):
Put UserTypingGestureIndicator on the stack for user generated keyboard events.

WebKit/mac:

The function name no longer perfectly matches the behavior, but I didn't want to break any existing clients. Maybe we
should migrate to a new function name eventually

  • WebCoreSupport/WebEditorClient.mm:

(WebEditorClient::textDidChangeInTextField):

WebKit/win:

The function name no longer perfectly matches the behavior, but I didn't want to break any existing clients. Maybe we
should migrate to a new function name eventually

  • WebCoreSupport/WebEditorClient.cpp:

(WebEditorClient::textDidChangeInTextField):

17:32 Changeset [63926] by tkent@chromium.org

2010-07-22 Kent Tamura <tkent@chromium.org>

Reviewed by Ojan Vafai.

<input type=number> stepper buttons should dispatch input/change events
https://bugs.webkit.org/show_bug.cgi?id=42440

  • fast/forms/input-number-events-expected.txt: Added.
  • fast/forms/input-number-events.html: Added.
  • fast/forms/script-tests/input-number-events.js: Added.
  • platform/chromium/test_expectations.txt:
  • platform/gtk/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:

2010-07-22 Kent Tamura <tkent@chromium.org>

Reviewed by Ojan Vafai.

<input type=number> stepper buttons should dispatch input/change events
https://bugs.webkit.org/show_bug.cgi?id=42440

Test: fast/forms/input-number-events.html

  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::stepUpFromRenderer): Sets a flag to dispatch 'change' event and dispatches 'input' event if the value is changed.
17:29 Changeset [63925] by jschuh@chromium.org

2010-07-22 Justin Schuh <jschuh@chromium.org>

Reviewed by Adam Barth.

Make history state methods validate against document URL
https://bugs.webkit.org/show_bug.cgi?id=42858

Test: fast/history/history-replace-illegal.html

  • page/History.cpp: (WebCore::History::urlForState):

2010-07-22 Justin Schuh <jschuh@chromium.org>

Reviewed by Adam Barth.

Make history state methods validate against document URL
https://bugs.webkit.org/show_bug.cgi?id=42858

  • fast/history/history-replace-illegal-expected.txt: Added.
  • fast/history/history-replace-illegal.html: Added.
17:25 Changeset [63924] by aestes@apple.com

When there are no beforeload listeners on a document,
HTMLLinkElement::process() should be called immediately when the node is
inserted into the document, rather than waiting until after attach is
performed.
<https://bugs.webkit.org/show_bug.cgi?id=42859>
<rdar://problem/8194528>

Reviewed by Maciej Stachowiak.

No new tests.

  • html/HTMLLinkElement.cpp:

(WebCore::HTMLLinkElement::insertedIntoDocument): If there are no
beforeload listeners, call process() immediately. Otherwise, add it as
a post-attach callback.

17:18 Changeset [63923] by jamesr@google.com

2010-07-22 James Robinson <jamesr@chromium.org>

Reviewed by Simon Fraser.

REGRESSION: box shadows on WebGL canvas elements repainted every frame
https://bugs.webkit.org/show_bug.cgi?id=42852

Avoid calling HTMLCanvasElement::willDraw() every time a WebGL canvas'
contents update. This call is only needed in the non-compositing
rendering path.

Covered by fast/canvas/webgl/canvas-test.html

  • html/canvas/WebGLRenderingContext.cpp: (WebCore::WebGLRenderingContext::markContextChanged): (WebCore::WebGLRenderingContext::paintRenderingResultsToCanvas):
17:05 Changeset [63922] by yurys@chromium.org

2010-07-22 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Joseph Pecoraro.

Web Inspector: test that inspector will log syntax error in console without crashing
https://bugs.webkit.org/show_bug.cgi?id=42831

  • inspector/console-log-syntax-error-expected.txt: Added.
  • inspector/console-log-syntax-error.html: Added.
  • inspector/resources/syntax-error.js: Added.
  • platform/chromium/inspector/console-log-syntax-error-expected.txt: Added.
16:52 Changeset [63921] by yael.aharon@nokia.com

2010-07-22 Yael Aharon <yael.aharon@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] QtWebKit needs public API for Notifications.
https://bugs.webkit.org/show_bug.cgi?id=41413

Add a DRT API to set a flag so that notification requests are ignored,
and add a test for cancelling a pending notification request.
New API and test are disabled for Chromium port.

  • fast/notifications/notifications-cancel-request-permission-expected.txt: Added.
  • fast/notifications/notifications-cancel-request-permission.html: Added.
  • fast/notifications/resources: Added.
  • fast/notifications/resources/notifications-cancel-request-permission.html: Added.
  • platform/chromium/test_expectations.txt:

2010-07-22 Yael Aharon <yael.aharon@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] QtWebKit needs public API for Notifications.
https://bugs.webkit.org/show_bug.cgi?id=41413

Remove DumpRenderTree private API and introduce new API.
Also add support for informing the UI when a notification request is cancelled.

  • Api/qwebpage.cpp: (QWebPage::setUserPermission):
  • Api/qwebpage.h:
  • WebCoreSupport/DumpRenderTreeSupportQt.cpp:
  • WebCoreSupport/DumpRenderTreeSupportQt.h:
  • WebCoreSupport/NotificationPresenterClientQt.cpp: (WebCore::NotificationPresenterClientQt::requestPermission): (WebCore::NotificationPresenterClientQt::checkPermission): (WebCore::NotificationPresenterClientQt::cancelRequestsForPermission): (WebCore::NotificationPresenterClientQt::allowNotificationForFrame): (WebCore::NotificationPresenterClientQt::toPage): (WebCore::NotificationPresenterClientQt::toFrame):
  • WebCoreSupport/NotificationPresenterClientQt.h:

2010-07-22 Yael Aharon <yael.aharon@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] QtWebKit needs public API for Notifications.
https://bugs.webkit.org/show_bug.cgi?id=41413

Update DumpRenderTree and QtTestBrowser to use the new API.
Add new DRT API to flag that notifications permission requests should be ignored.

  • DumpRenderTree/qt/DumpRenderTreeQt.cpp: (WebCore::WebPage::WebPage): (WebCore::WebPage::requestPermission): (WebCore::WebPage::checkPermission): (WebCore::WebPage::cancelRequestsForPermission):
  • DumpRenderTree/qt/DumpRenderTreeQt.h:
  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::reset): (LayoutTestController::ignoreDesktopNotificationPermissionRequests): (LayoutTestController::checkDesktopNotificationPermission):
  • DumpRenderTree/qt/LayoutTestControllerQt.h: (LayoutTestController::ignoreReqestForPermission):
  • QtTestBrowser/main.cpp: (LauncherWindow::LauncherWindow):
  • QtTestBrowser/webpage.cpp: (WebPage::WebPage): (WebPage::requestPermission): (WebPage::checkPermission): (WebPage::cancelRequestsForPermission):
  • QtTestBrowser/webpage.h:
16:32 Changeset [63920] by andersca@apple.com

Adapt sandbox for new WebProcess bootstrap service naming scheme.
<rdar://problem/8225000>

Patch by Ivan Krstić <ike@apple.com> on 2010-07-22
Reviewed by Anders Carlsson.

  • WebProcess/com.apple.WebProcess.sb:
16:16 Changeset [63919] by commit-queue@webkit.org

2010-07-22 Patrick Gansterer <paroga@paroga.com>

Reviewed by Ojan Vafai.

Remove useless PlatformScrollBar
https://bugs.webkit.org/show_bug.cgi?id=41911

  • WebCore.gypi:
  • platform/win/PlatformScrollBar.h: Removed.
  • platform/win/PlatformScrollBarWin.cpp: Removed.
15:55 Changeset [63918] by rniwa@webkit.org

[Windows] Home hey doesn't work in first DIV inside a TABLE
https://bugs.webkit.org/show_bug.cgi?id=42767

Reviewed by Ojan Vafai.

WebCore:

The bug was caused by positionAvoidingFirstPositionInTable moving outside of the editing region.
Added a check so that it avoids the first position in the table only if the table is editable.

Test: editing/selection/home-inside-noneditable-table.html

  • editing/visible_units.cpp:

(WebCore::positionAvoidingFirstPositionInTable):

LayoutTests:

Added a test to move caret backwards to the line boundary inside a content-editable div
inside a non-editable table.

  • editing/selection/home-inside-noneditable-table-expected.txt: Added.
  • editing/selection/home-inside-noneditable-table.html: Added.
  • resources/dump-as-markup.js: Modified to support multiple calls to dump

(Markup.dump.get if):
(Markup.dump):
(Markup.notifyDone):

15:18 Changeset [63917] by andersca@apple.com

It should be possible to attach to the WebProcess in gdb during startup
https://bugs.webkit.org/show_bug.cgi?id=42853

Reviewed by Adam Roben.

Create an unique bootstrap service name and pass it to the web process, instead of registering a per process
name (which was incorrect anyway). This lets us get rid of the requirement that the UI process always should be
the parent process of the web process, something which is false when running under the debugger.

  • UIProcess/Launcher/mac/ProcessLauncherMac.mm:

(WebKit::ProcessLauncher::launchProcess):

  • WebProcess/mac/WebProcessMainMac.mm:

(WebKit::WebProcessMain):

14:58 Changeset [63916] by andersca@apple.com

Don't assert when Connection::invalidate is called before Connection::dispatchConnectionDidClose
https://bugs.webkit.org/show_bug.cgi?id=42851

Reviewed by Sam Weinig.

  • Platform/CoreIPC/Connection.cpp:

(CoreIPC::Connection::dispatchConnectionDidClose):
Just return early if m_client is 0.

14:34 Changeset [63915] by tonikitoo@webkit.org

Fix build warning for missing parentheses.

Reviewed by Laszlo Gombos.
Patch by Antonio Gomes <tonikitoo@webkit.org>

WebCore/html/HTMLInputElement.cpp:113: warning: suggest parentheses around '&&' within '
'
  • html/HTMLInputElement.cpp:

(WebCore::isNumberCharacter):

14:26 Changeset [63914] by johnnyg@google.com

2010-07-22 John Gregg <johnnyg@google.com>

Reviewed by Kent Tamura.

[chromium] fast/forms/input-file-directory-upload.html fails on win after r63454
https://bugs.webkit.org/show_bug.cgi?id=42768

Normalize path separators for the script API by convering \'s to /'s.

Covered by input-file-directory-upload.html

  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::setFileListFromRenderer):
14:25 Changeset [63913] by aroben@apple.com

Use ::MessageBox instead of ::DebugBreak to make the web process wait for a debugger

While ::DebugBreak can make it easier to attach a debugger (because it
brings up the Just-in-Time debugger window), using ::MessageBox has
two advantages:

1) It explains to the user what's going on, so it's harder to

mistake this debugging aid for a bug.

2) It allows you to debug the UI process and the web process using

the same debugger, if desired.

Special thanks to John Sullivan for help with the text that's shown in
the alert.

Fixes <http://webkit.org/b/42848> WebKit2's wait-for-debugger
debugging aid should explain to the user what's happening

Reviewed by John Sullivan and Anders Carlsson.

  • WebProcess/WebKitMain.cpp:

(WebKitMain): Use a ::MessageBox instead of a ::DebugBreak to wait for
the debugger. The alert explains to the user what's going on.

14:25 Changeset [63912] by aroben@apple.com

Make WorkQueue aware of potential errors with ::WaitForMultipleObjects

Fixes <http://webkit.org/b/42846> WorkQueue should detect
::WaitForMultipleObject failures

Reviewed by Anders Carlsson.

  • Platform/win/WorkQueueWin.cpp:

(WorkQueue::workQueueThreadBody): Added some assertions about the
various things that can fail with ::WaitForMultipleObjects,
specifically:

  • Passing too many objects
  • Timeouts (which shouldn't happen since we pass a timeout interval of INFINITE)
  • Abandoned mutexes (which shouldn't happen since we don't wait on any mutexes currently)
  • Miscellaneous failures
14:06 Changeset [63911] by dimich@chromium.org

Rubberstamped by Simon Fraser.

  • platform/PlatformStrategies.h:

(WebCore::PlatformStrategies::PlatformStrategies): Add initialization of m_localizationStrategy.

13:56 Changeset [63910] by dumi@chromium.org

WebCore: Make db.changeVersion()'s third parameter optional, as required by the spec.
https://bugs.webkit.org/show_bug.cgi?id=42787

Reviewed by Ojan Vafai.

  • bindings/js/JSDatabaseCustom.cpp:

(WebCore::JSDatabase::changeVersion):

  • bindings/js/JSDatabaseSyncCustom.cpp:

(WebCore::JSDatabaseSync::changeVersion):

  • bindings/v8/custom/V8DatabaseCustom.cpp:

(WebCore::V8Database::changeVersionCallback):

  • bindings/v8/custom/V8DatabaseSyncCustom.cpp:

(WebCore::V8DatabaseSync::changeVersionCallback):

  • storage/SQLTransaction.cpp:

(WebCore::SQLTransaction::deliverTransactionCallback):

  • storage/SQLTransactionSync.cpp:

(WebCore::SQLTransactionSync::execute):

LayoutTests: Simplify some tests, and make sure db.changeVersion()'s third parameter is optional.
https://bugs.webkit.org/show_bug.cgi?id=42787

Reviewed by Ojan Vafai.

  • fast/workers/storage/resources/change-version-sync-1.js:
  • fast/workers/storage/resources/change-version-sync-2.js:
  • storage/change-version.html:
13:43 Changeset [63909] by senorblanco@chromium.org

2010-07-22 Stephen White <senorblanco@chromium.org>

Unreviewed; test expectations fix.

  • platform/chromium/test_expectations.txt: Updated expectations to match reality on the bots.
13:33 Changeset [63908] by jberlin@webkit.org

2010-07-20 Jessie Berlin <jberlin@apple.com>

Reviewed by Darin Adler.

Code Generator: Allow negative and string constants.
Also add tests for hexadecimal numbers.

  • bindings/scripts/CodeGeneratorJS.pm: If it is a string, do not try to cast it to an int.
  • bindings/scripts/IDLStructure.pm: Add a regex just for constant values that allows strings, hexadecimal numbers, and integral numbers.
  • bindings/scripts/test/CPP/WebDOMTestObj.h: Update the tests.
  • bindings/scripts/test/JS/JSTestObj.cpp: Ditto.
  • bindings/scripts/test/JS/JSTestObj.h: Ditto.
  • bindings/scripts/test/ObjC/DOMTestObj.h: Ditto.
  • bindings/scripts/test/TestObj.idl: Add tests for negative numbers, strings, and hexadecimal numbers.
  • bindings/scripts/test/V8/V8TestObj.cpp: Update the tests.
13:17 Changeset [63907] by simon.fraser@apple.com

2010-07-22 Simon Fraser <simon.fraser@apple.com>

Reviewed by Darin Adler.

Composited layers don't scroll in WebKit2
https://bugs.webkit.org/show_bug.cgi?id=42771

More prep work: make a single scrollPositionChanged() bottleneck method, and use it to
send the scroll event, repaint fixed elements, and update compositing layers.

  • page/FrameView.cpp: (WebCore::FrameView::scrollPositionChangedViaPlatformWidget): (WebCore::FrameView::scrollPositionChanged): (WebCore::FrameView::repaintFixedElementsAfterScrolling): (WebCore::FrameView::valueChanged):
  • page/FrameView.h:
  • platform/ScrollView.cpp: (WebCore::ScrollView::valueChanged):
13:00 Changeset [63906] by darin@apple.com

2010-07-22 Darin Adler <darin@apple.com>

Reviewed by Sam Weinig.

WebKitTestRunner needs to support layoutTestController.dumpChildFramesAsText
https://bugs.webkit.org/show_bug.cgi?id=42325

  • platform/mac-wk2/Skipped: Removed a duplicate and sorted lists of tests. Added some tests to the pasteboard list. Removed the tests from the dumpChildFramesAsText list that now pass, and moved others to other lists.

2010-07-22 Darin Adler <darin@apple.com>

Reviewed by Sam Weinig.

WebKitTestRunner needs to support layoutTestController.dumpChildFramesAsText
https://bugs.webkit.org/show_bug.cgi?id=42325

  • WebKitTestRunner/InjectedBundle/Bindings/LayoutTestController.idl: Added dumpChildFramesAsText.
  • WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp: (WTR::dumpFrameText): Added. (WTR::dumpDescendantFramesText): Added. (WTR::InjectedBundlePage::dumpAllFramesText): Added. (WTR::InjectedBundlePage::dump): Changed to use the new whatToDump function instead of the shouldDumpAsText function, and added a case for AllFramesText.
  • WebKitTestRunner/InjectedBundle/InjectedBundlePage.h: Added dumpAllFramesText.
  • WebKitTestRunner/InjectedBundle/LayoutTestController.cpp: (WTR::LayoutTestController::LayoutTestController): Replaced m_dumpAsText with m_whatToDump.
  • WebKitTestRunner/InjectedBundle/LayoutTestController.h: Added dumpChildFramesAsText. Removed shouldDumpAsText, shouldDumpDOMAsWebArchive, and shouldDumpSourceAsWebArchive. Added whatToDump.
12:24 CodeReview edited by levin@chromium.org
(diff)
11:39 Changeset [63905] by darin@chromium.org

2010-07-22 Darin Fisher <darin@chromium.org>

Reviewed by David Levin.

[Chromium] Propagate return value of EventHandler::handleMouseWheelEvent
to caller of WebViewImpl::handleInputEvent.

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

  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::mouseWheel): (WebKit::WebViewImpl::handleInputEvent):
  • src/WebViewImpl.h:
11:18 Changeset [63904] by weinig@apple.com

Patch for https://bugs.webkit.org/show_bug.cgi?id=42836
Add localized strings stubs for WebKit2.

Reviewed by Anders Carlsson.

WebCore:

  • Convert WebKit (mac) to use a LocalizationStrategy rather than the WebCoreViewFactory for localized strings.

Gets us below 200 tests failing in WebKitTestRunner.

  • WebCore.exp.in:
  • WebCore.xcodeproj/project.pbxproj:
  • page/mac/WebCoreViewFactory.h:
  • platform/LocalizationStrategy.h: Added.

(WebCore::LocalizationStrategy::~LocalizationStrategy):

  • platform/PlatformStrategies.h:

(WebCore::PlatformStrategies::pluginStrategy):
(WebCore::PlatformStrategies::localizationStrategy):
(WebCore::PlatformStrategies::PlatformStrategies):
(WebCore::PlatformStrategies::~PlatformStrategies):

  • platform/mac/LocalizedStringsMac.mm:

WebKit/mac:

  • Convert WebKit (mac) to use a LocalizationStrategy rather than the WebCoreViewFactory for localized strings.

Gets us below 200 tests failing in WebKitTestRunner.

  • WebCoreSupport/WebContextMenuClient.mm:

(fixMenusReceivedFromOldClients): Use LocalizedStrings rather than the factory.

  • WebCoreSupport/WebPlatformStrategies.h:
  • WebCoreSupport/WebPlatformStrategies.mm:
  • WebCoreSupport/WebViewFactory.mm:

WebKit2:

Gets us below 200 tests failing in WebKitTestRunner.

  • WebProcess/WebCoreSupport/WebPlatformStrategies.cpp:
  • WebProcess/WebCoreSupport/WebPlatformStrategies.h:
11:08 Changeset [63903] by abecsi@webkit.org

Unreviewed trivial build fix.

[Qt] Mend remaining issues to fix the WebKit2 build.

  • Shared/qt/UpdateChunk.h: add missing function

(WebKit::UpdateChunk::isEmpty):

  • UIProcess/API/qt/qwkpage_p.h: Add stub implementation

(QWKPagePrivate::pageDidEnterAcceleratedCompositing):
(QWKPagePrivate::pageDidLeaveAcceleratedCompositing):

11:01 Changeset [63902] by abecsi@webkit.org

2010-07-22 Andras Becsi <abecsi@webkit.org>

Reviewed by Antonio Gomes.

[Qt] Implement WebContext::applicationCacheDirectory() for the Qt port of WebKit2.
https://bugs.webkit.org/show_bug.cgi?id=42830

No new tests needed.

  • WebCore.pri: add ApplicationCacheStorage.h to wk2 headers
  • WebCore.pro: add WebContextQt.cpp to sources

2010-07-22 Andras Becsi <abecsi@webkit.org>

Reviewed by Antonio Gomes.

[Qt] Implement WebContext::applicationCacheDirectory() for the Qt port of WebKit2.
https://bugs.webkit.org/show_bug.cgi?id=42830

  • UIProcess/qt/WebContextQt.cpp: Added. (WebKit::WebContext::applicationCacheDirectory):
10:48 Changeset [63901] by commit-queue@webkit.org

2010-07-22 Joone Hur <joone.hur@samsung.com>

Reviewed by Antonio Gomes.

[EFL] add support for logging
https://bugs.webkit.org/show_bug.cgi?id=40807

Enable to turn log channels on for EFL port

  • platform/efl/LoggingEfl.cpp: (WebCore::InitializeLoggingChannelsIfNecessary):
10:04 Changeset [63900] by ossy@webkit.org

Unreviewed.

[Qt] Add platform specific expected results for tests introduced in r62200.

  • platform/qt/fast/borders/border-radius-circle-expected.txt: Copied from LayoutTests/platform/mac/fast/borders/border-radius-circle-expected.txt.
  • platform/qt/fast/borders/border-radius-groove-01-expected.txt: Copied from LayoutTests/platform/mac/fast/borders/border-radius-groove-01-expected.txt.
  • platform/qt/fast/borders/border-radius-groove-02-expected.txt: Copied from LayoutTests/platform/mac/fast/borders/border-radius-groove-02-expected.txt.
  • platform/qt/fast/borders/border-radius-groove-03-expected.txt: Copied from LayoutTests/platform/mac/fast/borders/border-radius-groove-03-expected.txt.
  • platform/qt/fast/borders/border-radius-wide-border-01-expected.txt: Copied from LayoutTests/platform/mac/fast/borders/border-radius-wide-border-01-expected.txt.
  • platform/qt/fast/borders/border-radius-wide-border-02-expected.txt: Copied from LayoutTests/platform/mac/fast/borders/border-radius-wide-border-02-expected.txt.
  • platform/qt/fast/borders/border-radius-wide-border-03-expected.txt: Copied from LayoutTests/platform/mac/fast/borders/border-radius-wide-border-03-expected.txt.
  • platform/qt/fast/borders/border-radius-wide-border-04-expected.txt: Copied from LayoutTests/platform/mac/fast/borders/border-radius-wide-border-04-expected.txt.
  • platform/qt/fast/borders/borderRadiusDashed04-expected.txt: Copied from LayoutTests/platform/mac/fast/borders/borderRadiusDashed04-expected.txt.
  • platform/qt/fast/borders/borderRadiusDashed05-expected.txt: Copied from LayoutTests/platform/mac/fast/borders/borderRadiusDashed05-expected.txt.
  • platform/qt/fast/borders/borderRadiusDashed06-expected.txt: Copied from LayoutTests/platform/mac/fast/borders/borderRadiusDashed06-expected.txt.
  • platform/qt/fast/borders/borderRadiusDotted04-expected.txt: Copied from LayoutTests/platform/mac/fast/borders/borderRadiusDotted04-expected.txt.
  • platform/qt/fast/borders/borderRadiusDotted05-expected.txt: Copied from LayoutTests/platform/mac/fast/borders/borderRadiusDotted05-expected.txt.
  • platform/qt/fast/borders/borderRadiusDotted06-expected.txt: Copied from LayoutTests/platform/mac/fast/borders/borderRadiusDotted06-expected.txt.
  • platform/qt/fast/borders/borderRadiusDouble04-expected.txt: Copied from LayoutTests/platform/mac/fast/borders/borderRadiusDouble04-expected.txt.
  • platform/qt/fast/borders/borderRadiusDouble05-expected.txt: Copied from LayoutTests/platform/mac/fast/borders/borderRadiusDouble05-expected.txt.
  • platform/qt/fast/borders/borderRadiusDouble06-expected.txt: Copied from LayoutTests/platform/mac/fast/borders/borderRadiusDouble06-expected.txt.
  • platform/qt/fast/borders/borderRadiusDouble07-expected.txt: Copied from LayoutTests/platform/mac/fast/borders/borderRadiusDouble07-expected.txt.
  • platform/qt/fast/borders/borderRadiusDouble08-expected.txt: Copied from LayoutTests/platform/mac/fast/borders/borderRadiusDouble08-expected.txt.
  • platform/qt/fast/borders/borderRadiusDouble09-expected.txt: Copied from LayoutTests/platform/mac/fast/borders/borderRadiusDouble09-expected.txt.
  • platform/qt/fast/borders/different-color-borders-expected.txt: Copied from LayoutTests/platform/mac/fast/borders/different-color-borders-expected.txt and updated.
10:02 Changeset [63899] by commit-queue@webkit.org

2010-07-22 Jamey Hicks <jamey.hicks@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Enable binding of QObjects to JavaScript environment for inspector frontend

This patch enables QObjects to be bound to global variables in the
javascript environment of the inspector frontend. This is useful
for writing alternate inspector frontends in javascript which need
additional native functionality. It's a lighter-weight alternative
than using an NPAPI plugin.

The code which instantiates QWebInspector uses this by setting the
dynamic property "q_inspectorJavaScriptWindowObjects" on the
QWebInspector. The value of this property should be a
QMap<QString,QVariant> mapping global variable name to QObject.

I have used this to add support for eclipse/chrome remote
debugging in QtTestBrowser.

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

  • WebCoreSupport/InspectorClientQt.cpp: (WebCore::InspectorClientWebPage::InspectorClientWebPage): (WebCore::InspectorClientWebPage::javaScriptWindowObjectCleared): (WebCore::InspectorClientQt::openInspectorFrontend):
10:00 Changeset [63898] by abecsi@webkit.org

WebCore: Ureviewed build fix.

Patch by Balazs Kelemen <kb@inf.u-szeged.hu> on 2010-07-22
[Qt] Fix WebKit2 build

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

  • WebCore.pri:
  • WebCore.pro:

WebKit2: Unreviewed build fix.

Patch by Balazs Kelemen <kb@inf.u-szeged.hu> on 2010-07-22
[Qt] Fix WebKit2 build

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

  • UIProcess/API/qt/qwkpage_p.h:

(QWKPagePrivate::setCursor): Added empty implementation.

09:42 Changeset [63897] by darin@chromium.org

2010-07-22 Darin Fisher <darin@chromium.org>

Reviewed by Kent Tamura.

https://bugs.webkit.org/show_bug.cgi?id=42808
Pass context menu events through WebInputEvent to WebPlugin::handleInputEvent.

The DOM treats a context menu event as a type of mouse event. We therefore
do the same here, and reuse the WebMouseEvent subclass. This makes sense
since a context menu event has (x,y)-coordinates.

  • public/WebInputEvent.h: (WebKit::WebInputEvent::): (WebKit::WebInputEvent::isMouseEventType):
  • src/WebInputEventConversion.cpp: (WebKit::WebMouseEventBuilder::WebMouseEventBuilder):
09:39 Changeset [63896] by commit-queue@webkit.org

2010-07-22 Andrey Kosyakov <caseq@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: Stop granting universal access to inspector front-end page.
https://bugs.webkit.org/show_bug.cgi?id=42817

  • inspector/InspectorFrontendClientLocal.cpp: (WebCore::InspectorFrontendClientLocal::windowObjectCleared):

2010-07-22 Andrey Kosyakov <caseq@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: Stop granting universal access to inspector front-end page.
https://bugs.webkit.org/show_bug.cgi?id=42817

  • src/WebDevToolsFrontendImpl.cpp: (WebKit::WebDevToolsFrontendImpl::frontendLoaded):
09:04 Applications using WebKit edited by tom-brown@live.co.uk
(diff)
09:03 Changeset [63895] by yurys@chromium.org

2010-07-22 Yury Semikhatsky <yurys@chromium.org>

Unreviewed. Fix inspector console tests crashes.

  • bindings/v8/ScriptValue.cpp: (WebCore::v8ToInspectorValue): (WebCore::ScriptValue::toInspectorValue): enter v8 context before calling GetPropertyNames
09:02 Applications using WebKit edited by tom-brown@live.co.uk
Changed Name of TomBrowser to XtremeBrowser added features (diff)
08:19 Changeset [63894] by leandro@webkit.org

2010-07-22 Gyuyoung Kim <gyuyoung.kim@samsung.com>

Unreviewed build fix.

[EFL] Build break for SVN r63881
https://bugs.webkit.org/show_bug.cgi?id=42813

  • CMakeLists.txt: Add platform/SchemeRegistry.cpp
08:03 Changeset [63893] by commit-queue@webkit.org

2010-07-22 Balazs Kelemen <kb@inf.u-szeged.hu>

Reviewed by Antonio Gomes.

Lazy cursor creation should be guarded by #if USE(LAZY_NATIVE_CURSOR) in WebKit2

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

  • Shared/WebCoreArgumentCoders.h:
  • UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::didReceiveMessage):
  • WebProcess/WebCoreSupport/WebChromeClient.cpp: (WebKit::WebChromeClient::setCursor):
07:43 Changeset [63892] by commit-queue@webkit.org

2010-07-22 Hayato Ito <hayato@chromium.org>

Reviewed by Darin Adler.

Refactor CSSSelector's destructor and make it inline.

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

Refactor with no behavior change, thus no new tests.

  • css/CSSSelector.cpp: (WebCore::CSSSelectorBag::~CSSSelectorBag): Make sure the bag is empty. (WebCore::CSSSelectorBag::add): Renamed from append(). (WebCore::CSSSelector::releaseOwnedSelectorsToBag): (WebCore::CSSSelector::deleteReachableSelectors):
  • css/CSSSelector.h: (WebCore::CSSSelector::~CSSSelector):
07:33 Changeset [63891] by yurys@chromium.org

2010-07-22 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: should be possible to convert console message arguments to InspectorValues
https://bugs.webkit.org/show_bug.cgi?id=42457

Now it is possible to convert simple JavaScript objects to
InspectorValues.

  • bindings/js/ScriptValue.cpp: (WebCore::jsToInspectorValue): (WebCore::ScriptValue::toInspectorValue):
  • bindings/js/ScriptValue.h:
  • bindings/v8/ScriptValue.cpp: (WebCore::v8ToInspectorValue): (WebCore::ScriptValue::toInspectorValue):
  • bindings/v8/ScriptValue.h:
  • inspector/ConsoleMessage.cpp: console notifications are pushed to RemoteInspectorFrontend instead of InspectorFrontend. (WebCore::ConsoleMessage::CallFrame::buildInspectorObject): (WebCore::ConsoleMessage::addToFrontend): (WebCore::ConsoleMessage::updateRepeatCountInConsole):
  • inspector/ConsoleMessage.h:
  • inspector/InjectedScript.cpp: (WebCore::InjectedScript::wrapForConsole): return InspectorValue instead of SerializedScriptValue
  • inspector/InjectedScript.h:
  • inspector/Inspector.idl:
  • inspector/InspectorBackend.cpp: (WebCore::InspectorBackend::clearConsoleMessages): send response directly from the backend (WebCore::InspectorBackend::remoteFrontend):
  • inspector/InspectorBackend.h:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::addConsoleMessage): (WebCore::InspectorController::clearConsoleMessages): don't send notification to the front end, it will be send either from InspectorBackend.cpp if it was a user request or reset message will be send to the front end if the messages are cleared due to page navigation. (WebCore::InspectorController::populateScriptObjects):
  • inspector/front-end/ConsoleView.js:
  • inspector/front-end/inspector.js:
07:26 Changeset [63890] by ossy@webkit.org

Unreviewed.

[Qt] REGRESSION(63862): animations/play-state.html fails intermittently
https://bugs.webkit.org/show_bug.cgi?id=42821

  • platform/qt/Skipped: animations/play-state.html skipped until fix.
07:23 Changeset [63889] by apavlov@chromium.org

2010-07-22 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: Modifying settings actually modifies defaultValues
https://bugs.webkit.org/show_bug.cgi?id=42816

Avoid using live modifiable objects as property default values.

07:07 Changeset [63888] by benm@google.com

2010-07-22 Ben Murdoch <benm@google.com>

Reviewed by Simon Fraser.

Touch events do not affect the :active CSS state
https://bugs.webkit.org/show_bug.cgi?id=39493

Clean the code up by using a typedef based on comments
post landing of the original patch.

No change in functionality so no new tests.

  • rendering/HitTestRequest.h: Typedef the HitTestRequestType to unsigned.
  • WebCore.exp.in: Update export signature.
  • WebCore.order: Ditto.
  • page/EventHandler.cpp: use the new typedef.
  • page/EventHandler.h: ditto.
06:47 QtWebKitReleases edited by hausmann@webkit.org
(diff)
06:36 Changeset [63887] by hausmann@webkit.org

[Qt] Library versioning.

2.1 is branched, trunk will become 2.2.

  • Api/qwebkitglobal.h:
  • qt_webkit_version.pri:
06:29 QtWebKitRelease21 created by hausmann@webkit.org
06:28 QtWebKitBackportingFixes edited by hausmann@webkit.org
(diff)
06:26 Changeset [63886] by apavlov@chromium.org

2010-07-22 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: Rename method for CSS rule source range retrieval and fix return object format
https://bugs.webkit.org/show_bug.cgi?id=42379

WebCore:

  • inspector/Inspector.idl:
  • inspector/InspectorBackend.cpp: (WebCore::InspectorBackend::getRuleRanges):
  • inspector/InspectorBackend.h:
  • inspector/InspectorBackend.idl:
  • inspector/InspectorCSSStore.cpp: (WebCore::InspectorCSSStore::getRuleRanges):
  • inspector/InspectorCSSStore.h:
  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::getRuleRanges):
  • inspector/InspectorDOMAgent.h:
  • inspector/front-end/DOMAgent.js:

WebKit/chromium:

  • src/js/InspectorControllerImpl.js: (devtools.InspectorBackendImpl):

LayoutTests:

  • inspector/styles-source-offsets.html:
06:25 QtWebKit edited by hausmann@webkit.org
(diff)
06:01 Changeset [63885] by aroben@apple.com

Windows build fix

Inspired by r63881.

  • MiniBrowser/MiniBrowser.vcproj:
  • WebKitTestRunner/InjectedBundle/win/InjectedBundle.vcproj:

Removed pre- and post-build events so they will be inherited from the
vsprops files.

05:52 Changeset [63884] by pfeldman@chromium.org

2010-07-21 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Chromium Dev Tools: Separators are missing in the context menu.

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

  • bindings/js/JSInspectorFrontendHostCustom.cpp: (WebCore::JSInspectorFrontendHost::showContextMenu):
  • bindings/v8/custom/V8InspectorFrontendHostCustom.cpp: (WebCore::V8InspectorFrontendHost::showContextMenuCallback):
  • platform/ContextMenuItem.h: (WebCore::):

2010-07-21 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Chromium Dev Tools: Separators are missing in the context menu.

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

  • src/ContextMenuClientImpl.cpp: (WebKit::ContextMenuClientImpl::populateCustomMenuItems):
05:41 Changeset [63883] by zherczeg@webkit.org

SVGFilterElement & SVGFE*Element don't support dynamic invalidation, when attributes change
https://bugs.webkit.org/show_bug.cgi?id=42244

Reviewed by Nikolas Zimmermann.

WebCore:

Implementing svgAttributeChanged and childrenChanged for the base class of all filters,
called SVGFilterPrimitiveStandardAttributes.

Tests: svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-dom-height-attr.html

svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-dom-result-attr.html
svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-dom-width-attr.html
svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-dom-x-attr.html
svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-dom-y-attr.html
svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-svgdom-height-prop.html
svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-svgdom-result-prop.html
svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-svgdom-width-prop.html
svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-svgdom-x-prop.html
svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-svgdom-y-prop.html

  • svg/SVGFilterPrimitiveStandardAttributes.cpp:

(WebCore::SVGFilterPrimitiveStandardAttributes::svgAttributeChanged):
(WebCore::SVGFilterPrimitiveStandardAttributes::childrenChanged):

  • svg/SVGFilterPrimitiveStandardAttributes.h:

(WebCore::SVGFilterPrimitiveStandardAttributes::invalidateFilter):

LayoutTests:

  • platform/mac/svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-dom-height-attr-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-dom-height-attr-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-dom-result-attr-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-dom-result-attr-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-dom-width-attr-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-dom-width-attr-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-dom-x-attr-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-dom-x-attr-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-dom-y-attr-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-dom-y-attr-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-svgdom-height-prop-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-svgdom-height-prop-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-svgdom-result-prop-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-svgdom-result-prop-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-svgdom-width-prop-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-svgdom-width-prop-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-svgdom-x-prop-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-svgdom-x-prop-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-svgdom-y-prop-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-svgdom-y-prop-expected.png: Added.
  • svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-dom-height-attr-expected.txt: Added.
  • svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-dom-height-attr.html: Added.
  • svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-dom-result-attr-expected.txt: Added.
  • svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-dom-result-attr.html: Added.
  • svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-dom-width-attr-expected.txt: Added.
  • svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-dom-width-attr.html: Added.
  • svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-dom-x-attr-expected.txt: Added.
  • svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-dom-x-attr.html: Added.
  • svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-dom-y-attr-expected.txt: Added.
  • svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-dom-y-attr.html: Added.
  • svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-svgdom-height-prop-expected.txt: Added.
  • svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-svgdom-height-prop.html: Added.
  • svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-svgdom-result-prop-expected.txt: Added.
  • svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-svgdom-result-prop.html: Added.
  • svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-svgdom-width-prop-expected.txt: Added.
  • svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-svgdom-width-prop.html: Added.
  • svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-svgdom-x-prop-expected.txt: Added.
  • svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-svgdom-x-prop.html: Added.
  • svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-svgdom-y-prop-expected.txt: Added.
  • svg/dynamic-updates/SVGFilterPrimitiveStandardAttributes-svgdom-y-prop.html: Added.
  • svg/dynamic-updates/script-tests/SVGFilterPrimitiveStandardAttributes-dom-height-attr.js: Added.

(executeTest):

  • svg/dynamic-updates/script-tests/SVGFilterPrimitiveStandardAttributes-dom-result-attr.js: Added.

(executeTest):

  • svg/dynamic-updates/script-tests/SVGFilterPrimitiveStandardAttributes-dom-width-attr.js: Added.

(executeTest):

  • svg/dynamic-updates/script-tests/SVGFilterPrimitiveStandardAttributes-dom-x-attr.js: Added.

(executeTest):

  • svg/dynamic-updates/script-tests/SVGFilterPrimitiveStandardAttributes-dom-y-attr.js: Added.

(executeTest):

  • svg/dynamic-updates/script-tests/SVGFilterPrimitiveStandardAttributes-svgdom-height-prop.js: Added.

(executeTest):

  • svg/dynamic-updates/script-tests/SVGFilterPrimitiveStandardAttributes-svgdom-result-prop.js: Added.

(executeTest):

  • svg/dynamic-updates/script-tests/SVGFilterPrimitiveStandardAttributes-svgdom-width-prop.js: Added.

(executeTest):

  • svg/dynamic-updates/script-tests/SVGFilterPrimitiveStandardAttributes-svgdom-x-prop.js: Added.

(executeTest):

  • svg/dynamic-updates/script-tests/SVGFilterPrimitiveStandardAttributes-svgdom-y-prop.js: Added.

(executeTest):

05:08 Changeset [63882] by kent.hansen@nokia.com

2010-07-22 Kent Hansen <kent.hansen@nokia.com>

Reviewed by Kent Tamura.

Error properties of the Global Object are missing the DontEnum attribute
https://bugs.webkit.org/show_bug.cgi?id=28771

Add the attributes to become spec compliant.

  • runtime/JSGlobalObject.cpp: (JSC::JSGlobalObject::reset):

2010-07-22 Kent Hansen <kent.hansen@nokia.com>

Reviewed by Kent Tamura.

Error properties of the Global Object are missing the DontEnum attribute
https://bugs.webkit.org/show_bug.cgi?id=28771

Update expected results for tests that now pass.

  • fast/js/sputnik/Conformance/10_Execution_Contexts/10.1_Definitions/10.1.5_Global_Object/S10.1.5_A2.1_T3-expected.txt:
  • fast/js/sputnik/Conformance/10_Execution_Contexts/10.1_Definitions/10.1.5_Global_Object/S10.1.5_A2.2_T3-expected.txt:
  • fast/js/sputnik/Conformance/10_Execution_Contexts/10.1_Definitions/10.1.5_Global_Object/S10.1.5_A2.3_T3-expected.txt:
02:36 Changeset [63881] by jhoneycutt@apple.com

Build fix. Unreviewed.

  • win/WebKit2WebProcess.vcproj:

Remove the pre- and post-build events, so that they will use the value
inherited from the vsprops.

01:52 Changeset [63880] by kent.hansen@nokia.com

2010-07-22 Kent Hansen <kent.hansen@nokia.com>

Reviewed by Sam Weinig.

Improve coverage of fast/dom/prototype-inheritance test
https://bugs.webkit.org/show_bug.cgi?id=42354

Use Object.getOwnPropertyNames rather than for..in to enumerate
the window object's properties, so that non-enumerable (ECMA)
properties are covered as well.

  • fast/dom/prototype-inheritance-expected.txt:
  • fast/dom/script-tests/prototype-inheritance.js: (propertiesOnObject):
  • platform/chromium-mac/fast/dom/prototype-inheritance-expected.txt:
  • platform/chromium-win/fast/dom/prototype-inheritance-expected.txt:
  • platform/chromium/fast/dom/prototype-inheritance-expected.txt:
  • platform/gtk/fast/dom/prototype-inheritance-expected.txt:
  • platform/qt/fast/dom/prototype-inheritance-expected.txt:
  • platform/win/fast/dom/prototype-inheritance-expected.txt:
01:20 Changeset [63879] by tkent@chromium.org

2010-07-22 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Adler.

Small refactoring for input value sanitization
https://bugs.webkit.org/show_bug.cgi?id=42807

Rename some functions to clarify their roles.

  • dom/InputElement.cpp: (WebCore::replaceEOLAndLimitLength): Renamed from sanitizeUserInputValue(). (WebCore::InputElement::sanitizeValueForTextField): Renamed from sanitizeValue(), and call replaceEOLAndLimitLength() instead of sanitizeUserInputValue(). (WebCore::InputElement::sanitizeUserInputValue): Just call replaceEOLAndLimitLength(). (WebCore::InputElement::updateValueIfNeeded): Use non-static sanitizeValue() for consistency.
  • dom/InputElement.h:
  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::setInputType): Revert the change of r63876. It is not needed because of the updateValueIfNeeded() change. (WebCore::HTMLInputElement::sanitizeValue): Apply the sanitizeValue() renaming.
00:40 Changeset [63878] by tonyg@chromium.org

2010-07-22 Tony Gentilcore <tonyg@chromium.org>

Reviewed by Darin Fisher.

webkitPerformance.timing.responseEnd should not include document parse time
https://bugs.webkit.org/show_bug.cgi?id=42797

No new tests because timing based test would be flaky.

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::finishedLoading): finishedLoading() is called by the platform at the right time, but didReceiveData() synchronously invokes parsing without returning to the event loop prior to this. So by the time the didFinishLoading() method is executed, parsing is finished. The solution is to move this time to didReceiveData() prior to parsing.
  • loader/MainResourceLoader.cpp: (WebCore::MainResourceLoader::didReceiveData): Update the time each time didReceiveData() is called. (WebCore::MainResourceLoader::didFinishLoading): When finished, and after parsing, set responseEnd appropriately.
  • loader/MainResourceLoader.h:
  • page/Timing.cpp: (WebCore::Timing::resourceLoadTimeRelativeToAbsolute): Now that responseEnd is set correctly, it can be much shorter for very fast loads (like local files). This exposed the fact that this skew check was not tight enough. We need to make sure that the whole range of values from the ResourceLoadTiming API fit within fetchStart-responseEnd, not just that requestTime fits in this range.
Note: See TracTimeline for information about the timeline view.