⚠ Archived content — this site is no longer maintained.   Current WebKit documentation is at docs.webkit.org.

Timeline



Mar 7, 2010:

10:55 PM Changeset in webkit [55656] by hamaji@chromium.org
  • 3 edits in trunk/WebKitTools

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

Reviewed by Eric Seidel.

check-webkit-style: false positive for empty loop
​https://bugs.webkit.org/show_bug.cgi?id=35717

  • Scripts/webkitpy/style/processors/cpp.py:
  • Scripts/webkitpy/style/processors/cpp_unittest.py:
9:02 PM Changeset in webkit [55655] by tkent@chromium.org
  • 1 edit
    4 adds in trunk/WebKitTools

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

Reviewed by Dimitri Glazkov.

[DRT/Chromium] Add AccessibilityController and AccessibilityUIElement
​https://bugs.webkit.org/show_bug.cgi?id=35774

Add AccessibilityController and AccessibilityUIElement classes,
which are going to be used by DumpRenderTree Chromium port. These
files are based on:

  • src/webkit/tools/test_shell/accessibility_controller.{cc,h} and
  • src/webkit/tools/test_shell/accessibility_ui_element.{cc,h} of Chromium rev.40492.
  • DumpRenderTree/chromium/AccessibilityController.cpp: Added.
  • DumpRenderTree/chromium/AccessibilityController.h: Added.
  • DumpRenderTree/chromium/AccessibilityUIElement.cpp: Added.
  • DumpRenderTree/chromium/AccessibilityUIElement.h: Added.
7:56 PM Changeset in webkit [55654] by levin@chromium.org
  • 2 edits in trunk/WebCore

Chromium Linux build fix.

  • platform/graphics/chromium/FontPlatformDataLinux.h: Add "class String" since a debug

only method returns a String.

7:06 PM Changeset in webkit [55653] by mrowe@apple.com
  • 4 edits in trunk

Chromium build fix.

WebCore:

  • platform/chromium/ClipboardChromium.cpp: Include Image.h since we're using WebCore::Image.

WebKit/chromium:

  • src/ChromiumBridge.cpp: Include SharedBuffer.h since WebCore::SharedBuffer is being used.
7:02 PM Changeset in webkit [55652] by mrowe@apple.com
  • 2 edits in trunk/WebKit/win

Windows build fix.

  • WebKitPrefix.h: Include CoreFoundation/CoreFoundation.h from the Windows prefix header

since some WebCore headers rely on the types declared within being available via the prefix
header.

6:57 PM Changeset in webkit [55651] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Gtk build fix.

  • platform/network/soup/ResourceHandleSoup.cpp: Include SharedBuffer.h since we're using WebCore::SharedBuffer.
6:51 PM Changeset in webkit [55650] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Gtk build fix.

  • platform/graphics/gtk/ImageGtk.cpp: Include SharedBuffer.h since we're using WebCore::SharedBuffer.
6:38 PM Changeset in webkit [55649] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Fix builds where USE_CG_SHADING is set.

  • platform/graphics/cg/GradientCG.cpp: Include wtf/RetainPtr.h since WTF::RetainPtr is used when USE_CG_SHADING is set.
6:33 PM Changeset in webkit [55648] by mrowe@apple.com
  • 4 edits in trunk/WebCore

Windows build fix. Add some required includes.

  • platform/graphics/win/ImageCGWin.cpp:
  • platform/network/cf/ResourceHandleCFNet.cpp:
  • platform/win/ClipboardWin.cpp:
6:29 PM Changeset in webkit [55647] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Gtk build fix.

  • platform/graphics/GraphicsContext.h: Include wtf/PassOwnPtr.h since some platforms use WTF::PassOwnPtr in this header.
6:27 PM Changeset in webkit [55646] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Qt build fix.

  • platform/network/qt/ResourceHandleQt.cpp: Include SharedBuffer.h since we're using WebCore::SharedBuffer.
6:23 PM Changeset in webkit [55645] by mrowe@apple.com
  • 2 edits in trunk/WebKit/chromium

Chromium build fix.

  • src/ResourceHandle.cpp: Include SharedBuffer.h since WebCore::SharedBuffer is being used.
6:18 PM Changeset in webkit [55644] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Gtk build fix.

  • platform/gtk/ClipboardGtk.cpp: Include Imageh since we're using WebCore::Image.
6:17 PM Changeset in webkit [55643] by mrowe@apple.com
  • 19 edits in trunk

Remove unnecessary includes from header files, adding them to the handful of implementation files that need them.

Rubber-stamped by Cameron Zwarich.

WebCore:

  • loader/CachedCSSStyleSheet.cpp:
  • loader/CachedFont.cpp:
  • loader/CachedImage.cpp:
  • loader/CachedResource.cpp:
  • loader/CachedResource.h:
  • loader/CachedScript.cpp:
  • loader/CachedXSLStyleSheet.cpp:
  • loader/icon/IconFetcher.cpp:
  • loader/loader.cpp:
  • page/Page.cpp:
  • platform/graphics/Image.cpp:
  • platform/graphics/Image.h:
  • platform/graphics/cg/GraphicsContextPlatformPrivateCG.h:
  • platform/graphics/cg/PDFDocumentImage.cpp:
  • platform/graphics/cg/PathCG.cpp:
  • platform/graphics/mac/ImageMac.mm:

WebKit/mac:

  • Misc/WebIconDatabase.mm:
6:05 PM Changeset in webkit [55642] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Chromium build fix.

  • platform/chromium/ChromiumDataObject.h: Include SharedBuffer.h since the inline constructor of this

class means that the pointed-to type of the RefPtr member must be available.

6:05 PM Changeset in webkit [55641] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Qt build fix.

  • platform/network/qt/DnsPrefetchHelper.cpp: Include PlatformString.h since we're using WebCore::String.
5:53 PM Changeset in webkit [55640] by tkent@chromium.org
  • 3 edits
    3 adds
    2 deletes in trunk

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

Reviewed by Eric Seidel.

Fix a bug that validity.valueMissing for a radio button with required
in a form element always returns true.
​https://bugs.webkit.org/show_bug.cgi?id=35472

Add tests for radio buttons in a form element, and merge an existing
radio button test to ValidityState-valueMissing-radio.html.

  • fast/forms/ValidityState-valueMissing-007-expected.txt: Removed.
  • fast/forms/ValidityState-valueMissing-007.html: Removed.
  • fast/forms/ValidityState-valueMissing-radio-expected.txt: Added.
  • fast/forms/ValidityState-valueMissing-radio.html: Added.
  • fast/forms/script-tests/ValidityState-valueMissing-radio.js: Added.

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

Reviewed by Eric Seidel.

Fix a bug that validity.valueMissing for a radio button with required
in a form element always returns true.
​https://bugs.webkit.org/show_bug.cgi?id=35472

Test: fast/forms/ValidityState-valueMissing-radio.html

  • html/HTMLInputElement.cpp: (WebCore::checkedRadioButtons): Move the location to be used by valueMissing(). (WebCore::HTMLInputElement::valueMissing): Use checkedRadioButtons() instead of document()->checkedRadioButtons().
5:52 PM Changeset in webkit [55639] by mrowe@apple.com
  • 12 edits in trunk

Remove unnecessary includes from header files, adding them to the handful of implementation files that need them.

Rubber-stamped by Cameron Zwarich.

WebCore:

  • bindings/objc/DOM.mm:
  • loader/CachedImage.h:
  • loader/EmptyClients.h:
  • platform/graphics/GlyphPageTreeNode.cpp:
  • platform/text/CString.h:
  • platform/text/String.cpp:
  • platform/text/mac/TextCodecMac.cpp:
  • svg/graphics/SVGResourceFilter.h:
  • svg/graphics/filters/SVGFEImage.h:

WebKit/mac:

  • Misc/WebNSPasteboardExtras.mm:
5:52 PM Changeset in webkit [55638] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Completely remove two files that were deleted in r55635 from the Xcode project.

  • WebCore.xcodeproj/project.pbxproj:
5:26 PM Changeset in webkit [55637] by weinig@apple.com
  • 7 edits in trunk

Remove inconsistent "Too few arguments" handling for window.atob() and window.btoa()
​https://bugs.webkit.org/show_bug.cgi?id=35848

Reviewed by Dan Bernstein.

WebCore:

  • Take the opportunity to fully autogenerate window.atob() and window.btoa().
  • bindings/js/JSDOMWindowCustom.cpp:
  • page/DOMWindow.cpp:

(WebCore::DOMWindow::btoa):
(WebCore::DOMWindow::atob):

  • page/DOMWindow.idl:

LayoutTests:

  • fast/dom/Window/atob-btoa-expected.txt:
  • fast/dom/Window/atob-btoa.html:
5:02 PM PythonGuidelines edited by Chris Jerdonek
Minor updates. Added a FIXME to include upgrade instructions. (diff)
10:15 AM Changeset in webkit [55636] by mitz@apple.com
  • 4 edits
    2 adds in trunk

<rdar://problem/7722008> Column breaking ignores floats
​https://bugs.webkit.org/show_bug.cgi?id=35837

Reviewed by Simon Fraser.

WebCore:

Test: fast/multicol/float-truncation.html

Introduce an earlier column-break if otherwise a float that could fit
inside a single column will be split between columns.

It is still possible for floats to be needlessly broken if initially
they fit in the column, but normal flow truncation then shortens the
column.

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::visibleTopOfHighestFloatExtendingBelow): Added.
Returns the visible top of the highest descendant float that visibly
extends below the given y offset, ignoring floats that are taller than
the given maximum height.
(WebCore::RenderBlock::layoutColumns): If the initial column height
would cause a float to be split, truncate above the float.

  • rendering/RenderBlock.h:

LayoutTests:

  • fast/multicol/float-truncation-expected.txt: Added.
  • fast/multicol/float-truncation.html: Added.
8:59 AM Changeset in webkit [55635] by dimich@chromium.org
  • 24 edits
    9 deletes in trunk

Not reviewed. Revert of r55593 which caused a regression of worker-cloneports.html.

REGRESSION(55593?): fast/workers/worker-cloneport.html is timing out on Leopard
​https://bugs.webkit.org/show_bug.cgi?id=35819

WebCore:

  • Android.jscbindings.mk:
  • Android.v8bindings.mk:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSDOMWindowCustom.cpp:
  • bindings/js/JSDatabaseCallback.cpp: Removed.
  • bindings/js/JSDatabaseCallback.h: Removed.
  • bindings/v8/custom/V8DOMWindowCustom.cpp:
  • bindings/v8/custom/V8DatabaseCallback.cpp: Removed.
  • bindings/v8/custom/V8DatabaseCallback.h: Removed.
  • dom/Document.cpp:

(WebCore::Document::postTask):

  • page/DOMWindow.cpp:

(WebCore::DOMWindow::openDatabase):

  • page/DOMWindow.h:
  • page/DOMWindow.idl:
  • storage/Database.cpp:

(WebCore::Database::openDatabase):
(WebCore::Database::Database):
(WebCore::Database::performOpenAndVerify):

  • storage/Database.h:
  • storage/DatabaseCallback.h: Removed.
  • workers/WorkerContext.cpp:

(WebCore::WorkerContext::openDatabase):

  • workers/WorkerContext.h:

LayoutTests:

  • fast/frames/resources/sandboxed-iframe-storage-allowed.html:
  • fast/frames/resources/sandboxed-iframe-storage-disallowed.html:
  • fast/frames/sandboxed-iframe-storage-expected.txt:
  • storage/open-database-creation-callback-expected.txt: Removed.
  • storage/open-database-creation-callback-isolated-world-expected.txt: Removed.
  • storage/open-database-creation-callback-isolated-world.html: Removed.
  • storage/open-database-creation-callback.html: Removed.
  • platform/qt/Skipped:
  • platform/mac-leopard/Skipped: Enable the temporarily disabled test back.
12:16 AM Changeset in webkit [55634] by eric@webkit.org
  • 2 edits in trunk/JavaScriptCore

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

Reviewed by Simon Hausmann.

Small performance fix in the QScriptConverter::toString().

The QByteArray was replaced by the QVarLengthArray which doesn't
have to allocate any memory on heap.

[Qt] QScriptConverter::toString() should use QVarLengthArray instead of QByteArray
​https://bugs.webkit.org/show_bug.cgi?id=35577

  • qt/api/qscriptconverter_p.h: (QScriptConverter::toString):

Mar 6, 2010:

9:44 PM Changeset in webkit [55633] by mrowe@apple.com
  • 76 edits in trunk

Remove unnecessary includes of wtf/Platform.h. This is already pulled in by config.h.

Rubber-stamped by Sam Weinig.

JavaScriptCore:

Remove unnecessary includes of wtf/Platform.h. This is already pulled in by config.h.

  • API/APICast.h:
  • API/JSCallbackFunction.cpp:
  • API/JSContextRef.cpp:
  • API/JSObjectRef.cpp:
  • API/JSValueRef.cpp:
  • assembler/ARMAssembler.h:
  • assembler/ARMv7Assembler.h:
  • assembler/AbstractMacroAssembler.h:
  • assembler/AssemblerBuffer.h:
  • assembler/AssemblerBufferWithConstantPool.h:
  • assembler/CodeLocation.h:
  • assembler/LinkBuffer.h:
  • assembler/MIPSAssembler.h:
  • assembler/MacroAssembler.h:
  • assembler/MacroAssemblerARM.h:
  • assembler/MacroAssemblerARMv7.h:
  • assembler/MacroAssemblerCodeRef.h:
  • assembler/MacroAssemblerMIPS.h:
  • assembler/MacroAssemblerX86.h:
  • assembler/MacroAssemblerX86Common.h:
  • assembler/MacroAssemblerX86_64.h:
  • assembler/RepatchBuffer.h:
  • assembler/X86Assembler.h:
  • jit/JIT.h:
  • jit/JITCode.h:
  • jit/JITInlineMethods.h:
  • jit/JITStubs.h:
  • os-win32/stdint.h:
  • runtime/JSAPIValueWrapper.h:
  • runtime/JSImmediate.h:
  • wtf/ASCIICType.h:
  • wtf/StdLibExtras.h:
  • wtf/VMTags.h:
  • yarr/RegexCompiler.h:
  • yarr/RegexInterpreter.h:
  • yarr/RegexJIT.h:
  • yarr/RegexParser.h:
  • yarr/RegexPattern.h:

WebCore:

Remove unnecessary includes of wtf/Platform.h. This is already pulled in by config.h.

  • accessibility/AccessibilityObject.h:
  • dom/XMLTokenizer.cpp:
  • dom/XMLTokenizerLibxml2.cpp:
  • dom/XMLTokenizerQt.cpp:
  • editing/TextAffinity.h:
  • loader/FrameLoaderClient.h:
  • page/FocusController.cpp:
  • page/FrameTree.cpp:
  • page/Geolocation.h:
  • page/PositionCallback.h:
  • page/PositionErrorCallback.h:
  • platform/Cursor.h:
  • platform/FileSystem.h:
  • platform/FloatConversion.h:
  • platform/KeyboardCodes.h:
  • platform/PlatformKeyboardEvent.h:
  • platform/PlatformTouchPoint.h:
  • platform/SuddenTermination.h:
  • platform/Widget.h:
  • platform/graphics/Color.h:
  • platform/graphics/FloatPoint.h:
  • platform/graphics/FloatSize.h:
  • platform/graphics/GraphicsContext.h:
  • platform/graphics/IntPoint.h:
  • platform/graphics/IntRect.h:
  • platform/graphics/IntSize.h:
  • platform/graphics/openvg/PainterOpenVG.h:
  • platform/graphics/openvg/SurfaceOpenVG.h:
  • platform/network/ResourceHandleClient.h:
  • platform/text/Base64.cpp:
  • rendering/style/SVGRenderStyle.h:
  • xml/XSLTProcessor.cpp:
  • xml/XSLTProcessorLibxslt.cpp:
  • xml/XSLTProcessorQt.cpp:

WebKit/qt:

Remove unnecessary includes of wtf/Platform.h. This is already pulled in by the prefix header.

  • WebCoreSupport/EditCommandQt.cpp:
7:31 PM Changeset in webkit [55632] by weinig@apple.com
  • 7 edits
    3 adds in trunk/WebCore

Move debug only JS wrapper-set tracking code into its own file.
​https://bugs.webkit.org/show_bug.cgi?id=35839

Reviewed by Dan Bernstein.

  • GNUmakefile.am: Added new files.
  • WebCore.gypi: Ditto.
  • WebCore.pro: Ditto.
  • WebCore.vcproj/WebCore.vcproj: Ditto.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • bindings/js/JSDOMBinding.cpp:

(WebCore::cacheDOMObjectWrapper):
(WebCore::forgetDOMObject):
(WebCore::forgetDOMNode):
(WebCore::cacheDOMNodeWrapper):
(WebCore::takeWrappers):
(WebCore::updateDOMNodeDocument):
Updated for new signature for willCacheWrapper and didUncacheWrapper.

  • bindings/js/JSDOMWrapper.cpp: Added.

(WebCore::DOMObject::~DOMObject):
(WebCore::DOMObject::defineOwnProperty):
Moved from JSDOMBinding.cpp.

  • bindings/js/JSDebugWrapperSet.cpp: Added.

(WebCore::JSDebugWrapperSet::shared):
(WebCore::JSDebugWrapperSet::JSDebugWrapperSet):

  • bindings/js/JSDebugWrapperSet.h: Added.

(WebCore::JSDebugWrapperSet::add):
(WebCore::JSDebugWrapperSet::remove):
(WebCore::JSDebugWrapperSet::contains):
(WebCore::JSDebugWrapperSet::willCacheWrapper):
(WebCore::JSDebugWrapperSet::didUncacheWrapper):
Moved from JSDOMBinding.cpp.

4:38 PM Changeset in webkit [55631] by weinig@apple.com
  • 11 edits
    5 copies in trunk/WebCore

Rubber-stamped by Dan Bernstein.

Move DOMObjectHashTableMap, DOMWrapperWorld and WebCoreJSClientData into
their own files.

  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/DOMObjectHashTableMap.cpp: Copied from bindings/js/JSDOMBinding.cpp.
  • bindings/js/DOMObjectHashTableMap.h: Copied from bindings/js/JSDOMBinding.h.

(WebCore::DOMObjectHashTableMap::~DOMObjectHashTableMap):
(WebCore::DOMObjectHashTableMap::get):

  • bindings/js/DOMWrapperWorld.cpp: Copied from bindings/js/JSDOMBinding.cpp.

(WebCore::forgetWorldOfDOMNodesForDocument):

  • bindings/js/DOMWrapperWorld.h: Copied from bindings/js/JSDOMBinding.h.

(WebCore::DOMWrapperWorld::create):
(WebCore::DOMWrapperWorld::rememberDocument):
(WebCore::DOMWrapperWorld::forgetDocument):
(WebCore::DOMWrapperWorld::isNormal):
(WebCore::debuggerWorld):
(WebCore::pluginWorld):
(WebCore::currentWorld):
(WebCore::Document::getWrapperCache):

  • bindings/js/JSDOMBinding.cpp:
  • bindings/js/JSDOMBinding.h:
  • bindings/js/JSDOMWindowBase.cpp:
  • bindings/js/ScriptController.cpp:
  • bindings/js/WebCoreJSClientData.h: Copied from bindings/js/JSDOMBinding.h.

(WebCore::WebCoreJSClientData::WebCoreJSClientData):
(WebCore::WebCoreJSClientData::~WebCoreJSClientData):
(WebCore::WebCoreJSClientData::normalWorld):
(WebCore::WebCoreJSClientData::getAllWorlds):
(WebCore::WebCoreJSClientData::rememberWorld):
(WebCore::WebCoreJSClientData::forgetWorld):

  • bindings/js/WorkerScriptController.cpp:
3:01 PM Changeset in webkit [55630] by mitz@apple.com
  • 3 edits in trunk/WebCore

Rubber-stamped by Sam Weinig.

Remove an unused method.

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::layoutBlock): Changed comment.
(WebCore::RenderBlock::floatRect): Removed.

  • rendering/RenderBlock.h: Removed floatRect().
1:45 PM Changeset in webkit [55629] by pfeldman@chromium.org
  • 2 edits in trunk/WebKit/chromium

2010-03-06 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Chromium: Restore devtools window activate upon node search complete.

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

  • src/js/DevTools.js:
1:42 PM Changeset in webkit [55628] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

2010-03-06 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: make timeline overview bars transparent to mouse events.

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

  • inspector/front-end/inspector.css:
10:14 AM Changeset in webkit [55627] by pfeldman@chromium.org
  • 3 edits in trunk/WebCore

2010-03-06 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Jeremy Orlow.

Web Inspector: switching to/from Timeline Panel moves scroller.
(Also added couple of record details items as I was fixing it).

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

  • English.lproj/localizedStrings.js:
  • inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel): (WebInspector.TimelinePanel.prototype.get defaultFocusedElement): (WebInspector.TimelinePanel.prototype.show): (WebInspector.TimelinePanel.prototype._refreshRecords):
7:30 AM Changeset in webkit [55626] by eric@webkit.org
  • 2 edits in trunk/WebKit/chromium

2010-03-06 Darin Fisher <​darin@chromium.org>

Reviewed by David Levin.

[Chromium] Embedder initiated loads should not always stop existing loads
​https://bugs.webkit.org/show_bug.cgi?id=33862

  • src/WebFrameImpl.cpp: (WebKit::WebFrameImpl::reload): (WebKit::WebFrameImpl::loadRequest): (WebKit::WebFrameImpl::loadHistoryItem): (WebKit::WebFrameImpl::loadData):
7:13 AM Changeset in webkit [55625] by eric@webkit.org
  • 2 edits in trunk/WebKit/chromium

2010-03-06 Darin Fisher <​darin@chromium.org>

Reviewed by David Levin.

[Chromium] spurious WebViewClient::didStopLoading if changing
location.hash while a subframe is still loading

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

  • src/FrameLoaderClientImpl.cpp: (WebKit::FrameLoaderClientImpl::dispatchDidChangeLocationWithinPage): Call isLoadingInAPISense, which checks subframes.
6:57 AM Changeset in webkit [55624] by eric@webkit.org
  • 1 edit
    1 delete in trunk/WebCore

2010-03-06 Patrick Gansterer <​paroga@paroga.com>

Reviewed by Eric Seidel.

Removed unnecessary WinCE file.
The same functionality is provided by platform/win/SystemTimeWin.cpp.
​https://bugs.webkit.org/show_bug.cgi?id=35799

  • platform/wince/SystemTimeWince.cpp: Removed.
6:40 AM Changeset in webkit [55623] by eric@webkit.org
  • 4 edits
    2 adds in trunk

2010-03-06 Joanmarie Diggs <​joanmarie.diggs@gmail.com>

Reviewed by Eric Seidel.

​https://bugs.webkit.org/show_bug.cgi?id=30895
[Gtk] The accessible hierarchy of tables is significantly incorrect for Atk

  • platform/gtk/accessibility/table-hierarchy.html: Added.
  • platform/gtk/accessibility/table-hierarchy-expected.txt: Added.

2010-03-06 Joanmarie Diggs <​joanmarie.diggs@gmail.com>

Reviewed by Eric Seidel.

​https://bugs.webkit.org/show_bug.cgi?id=30895
[Gtk] The accessible hierarchy of tables is significantly incorrect for Atk

This completes the fixing of the Atk table hierarchy which was started
in bug #35418.

  • accessibility/gtk/AccessibilityObjectAtk.cpp: (AccessibilityObject::accessibilityPlatformIncludesObject):
  • accessibility/AccessibilityRenderObject.cpp: (AccessibilityRenderObject::determineAccessibilityRole):
6:23 AM Changeset in webkit [55622] by eric@webkit.org
  • 2 edits in trunk/WebKit/gtk

2010-03-06 Arno Renevier <​arno@renevier.net>

Reviewed by Eric Seidel.

[Gtk] GEOLOCATION_POLICY_DECISION_CANCELLED unused
​https://bugs.webkit.org/show_bug.cgi?id=35803

  • webkit/webkitwebview.cpp: (DNDContentsRequest::webkit_web_view_class_init):
5:57 AM Changeset in webkit [55621] by eric@webkit.org
  • 2 edits
    1 copy
    2 deletes in trunk/JavaScriptCore

2010-03-06 Kwang Yul Seo <​skyul@company100.net>

Reviewed by Eric Seidel.

[BREWMP] Share OwnPtr.
​https://bugs.webkit.org/show_bug.cgi?id=35776

Share OwnPtr implementation with BREW MP and remove OwnPtrBrew.

  • wtf/OwnPtrBrew.cpp: Added. (WTF::deleteOwnedPtr):
  • wtf/OwnPtrCommon.h:
  • wtf/brew/OwnPtrBrew.cpp: Removed.
  • wtf/brew/OwnPtrBrew.h: Removed.
5:20 AM Changeset in webkit [55620] by eric@webkit.org
  • 8 edits
    2 adds in trunk

2010-03-06 Hironori Bono <​hbono@chromium.org>

Reviewed by Eric Seidel.

[Chromium] Typing into Flash with wmode = opaque|transparent and
non-latin language active outputs as if US keyboard layout active

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

Added a layout test that verifies a plugin can receive keyboard events.
Unfortunately, this test works only on Mac.

  • platform/gtk/Skipped: Skipped the new test because of the lack of eventSender.
  • platform/qt/Skipped: ditto.
  • platform/win/Skipped: Skipped the new test because NPP_HandleEvent() is not implemented.
  • plugins/keyboard-events-expected.txt: Added.
  • plugins/keyboard-events.html: Added.

2010-03-06 Hironori Bono <​hbono@chromium.org>

Reviewed by Eric Seidel.

[Chromium] Typing into Flash with wmode = opaque|transparent and
non-latin language active outputs as if US keyboard layout active

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

This change is a WebKit-side change for this issue. It dispatches
Char events to plug-ins so plug-ins can receive non-ASCII characters
as well as ASCII characters.

  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::keyEvent):

2010-03-06 Hironori Bono <​hbono@chromium.org>

Reviewed by Eric Seidel.

[Chromium] Typing into Flash with wmode = opaque|transparent and
non-latin language active outputs as if US keyboard layout active

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

To test keyboard events on the test plugin, this change implements
NPCocoaEventKeyDown and NPCocoaEventKeyUp handlers so the plugin
can write log messages.

  • DumpRenderTree/TestNetscapePlugIn.subproj/main.cpp: (handleEventCocoa): Implemented the event handlers for NPCocoaKeyDown and NPCocoaEventKeyUp.
5:03 AM Changeset in webkit [55619] by eric@webkit.org
  • 7 edits in trunk

2010-03-06 MORITA Hajime <​morrita@google.com>

Reviewed by Darin Adler.

Moved implementations of window.btoa() and window.atob() from
JSDOMWindow to DOMWindow, and make V8DOMWindow use DOMWindow
functions instead of having a separate implementation. As a side effect, the
error message has changed from "Cannot decode base64" to one that
indicates DOM Exception, which is compatible to Firefox.

Refactoring: window.btoa() and window.atob() should be implemented on DOMWindow
​https://bugs.webkit.org/show_bug.cgi?id=35723

  • fast/dom/Window/atob-btoa-expected.txt: Rebaselined expectation due to change of error message.

2010-03-06 MORITA Hajime <​morrita@google.com>

Reviewed by Darin Adler.

Moved implementations of window.btoa() and window.atob() from
JSDOMWindow to DOMWindow, and make V8DOMWindow use DOMWindow
functions instead of having a separate implementation. As a side effect, the
error message has changed from "Cannot decode base64" to one that
indicates DOM Exception, which is compatible to Firefox.

Refactoring: window.btoa() and window.atob() should be implemented on DOMWindow
​https://bugs.webkit.org/show_bug.cgi?id=35723

No new test. No new functionality.

  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::atob): (WebCore::JSDOMWindow::btoa): Moved conversion code to DOMWindow and invoke it. Argument checking remains here.
  • bindings/v8/custom/V8DOMWindowCustom.cpp: (WebCore::V8DOMWindow::atobCallback): (WebCore::V8DOMWindow::btoaCallback): Remove conversion code and call DOMWindow APIs. Although argument checking remains here.
  • page/DOMWindow.cpp: (WebCore::hasMultibyteCharacters): (WebCore::DOMWindow::btoa): (WebCore::DOMWindow::atob):
  • page/DOMWindow.h: Moved Conversion code from JSDOMWindow, modifing to fit JSC independent.
4:43 AM Changeset in webkit [55618] by eric@webkit.org
  • 2 edits
    2 adds in trunk/WebKit/chromium

2010-03-06 Kavita Kanetkar <​kkanetkar@chromium.org>

Reviewed by Darin Fisher.

Create WebKit API for WebCore::ImageDecoder
​https://bugs.webkit.org/show_bug.cgi?id=35415

  • WebKit.gyp:
  • public/WebImageDecoder.h: Added.
  • src/WebImageDecoder.cpp: Added.
4:26 AM Changeset in webkit [55617] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-03-06 Yuta Kitamura <​yutak@chromium.org>

Reviewed by Eric Seidel.

Fix decoration position in search input field.

This patch fixes rendering of input field of "search" type, whose decoration
(loupe icon and close button) was wrongly positioned when the input field was
contained in a block with -webkit-transform property.

Chromium bug: ​http://crbug.com/20439

[Chromium] Decoration of "search" input field is wrongly rendered
​https://bugs.webkit.org/show_bug.cgi?id=30245

No new tests, since this patch fixes an existing test
(fast/forms/search-transformed.html) in Chromium layout tests.

  • rendering/RenderThemeChromiumSkia.cpp: (WebCore::RenderThemeChromiumSkia::convertToPaintingRect): Added. This method does almost the same thing as RenderThemeMac::convertToPaintingRect. (WebCore::RenderThemeChromiumSkia::paintSearchFieldCancelButton): The position of the icon should not depend on its absolute position. (WebCore::RenderThemeChromiumSkia::paintSearchFieldResultsDecoration): Ditto. (WebCore::RenderThemeChromiumSkia::paintSearchFieldResultsButton): Ditto.
  • rendering/RenderThemeChromiumSkia.h: Added new method.
3:50 AM Changeset in webkit [55616] by eric@webkit.org
  • 2 edits
    3 adds in trunk/LayoutTests

2010-03-06 Shu Chang <​Chang.Shu@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Added expected results and removed the test from Skipped.
​https://bugs.webkit.org/show_bug.cgi?id=35806

  • platform/qt/Skipped:
  • platform/qt/fast/canvas/canvasDrawingIntoSelf-expected.checksum: Added.
  • platform/qt/fast/canvas/canvasDrawingIntoSelf-expected.png: Added.
  • platform/qt/fast/canvas/canvasDrawingIntoSelf-expected.txt: Added.
3:31 AM Changeset in webkit [55615] by eric@webkit.org
  • 2 edits in trunk/JavaScriptCore

2010-03-06 Patrick Gansterer <​paroga@paroga.com>

Reviewed by Eric Seidel.

Implemented JIT_OPTIMIZE_NATIVE_CALL for WinCE
​https://bugs.webkit.org/show_bug.cgi?id=33426

  • jit/JITOpcodes.cpp: (JSC::JIT::privateCompileCTIMachineTrampolines):
3:00 AM Changeset in webkit [55614] by pfeldman@chromium.org
  • 5 edits in trunk

2010-03-06 Ilya Tikhonovsky <​loislo@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: in Timeline panel, click followed with mouse move resets popover.

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

  • inspector/front-end/Popover.js: (WebInspector.PopoverHelper.prototype._mouseDown): (WebInspector.PopoverHelper.prototype._mouseMove): (WebInspector.PopoverHelper.prototype._handleMouseAction):
Note: See TracTimeline for information about the timeline view.