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

Timeline



Mar 10, 2010:

11:53 PM Changeset in webkit [55834] by loki@webkit.org
  • 3 edits in trunk/JavaScriptCore

Buildfix for Thumb-2 after r55684. Add branch8 and branchTest8 functions.
​https://bugs.webkit.org/show_bug.cgi?id=35892

Reviewed by Gavin Barraclough.

  • assembler/ARMv7Assembler.h:

(JSC::ARMv7Assembler::):
(JSC::ARMv7Assembler::ldrb):

  • assembler/MacroAssemblerARMv7.h:

(JSC::MacroAssemblerARMv7::load8):
(JSC::MacroAssemblerARMv7::branch8):
(JSC::MacroAssemblerARMv7::branchTest8):
(JSC::MacroAssemblerARMv7::setTest8):

11:19 PM Changeset in webkit [55833] by barraclough@apple.com
  • 15 edits in trunk

Rubber stamped by Oliver Hunt.

Rename JSC::UStringImpl::data() to characters(), to match WebCore::StringImpl.

JavaScriptCore:

  • API/JSClassRef.cpp:

(OpaqueJSClassContextData::OpaqueJSClassContextData):

  • bytecompiler/BytecodeGenerator.cpp:

(JSC::keyForCharacterSwitch):

  • bytecompiler/NodesCodegen.cpp:

(JSC::processClauseList):

  • interpreter/Interpreter.cpp:

(JSC::Interpreter::privateExecute):

  • jit/JITStubs.cpp:

(JSC::DEFINE_STUB_FUNCTION):

  • runtime/ArrayPrototype.cpp:

(JSC::arrayProtoFuncToString):

  • runtime/Identifier.cpp:

(JSC::Identifier::equal):
(JSC::Identifier::addSlowCase):

  • runtime/JSString.cpp:

(JSC::JSString::resolveRope):

  • runtime/UString.cpp:

(JSC::UString::toStrictUInt32):
(JSC::equal):

  • runtime/UString.h:

(JSC::UString::data):

  • runtime/UStringImpl.h:

(JSC::UStringImpl::characters):
(JSC::UStringImpl::hash):
(JSC::UStringImpl::setHash):

WebCore:

  • bridge/jni/jsc/JavaStringJSC.h:

(JSC::Bindings::JavaStringImpl::uchars):

  • platform/text/AtomicString.cpp:

(WebCore::AtomicString::add):
(WebCore::AtomicString::find):

11:07 PM Changeset in webkit [55832] by barraclough@apple.com
  • 2 edits in trunk/WebCore

Fix for r55825, threadsafeCopy no longer needs to special-case for
empty strings (in fact, doing so results in leaks).

Reviewed by Mark Rowe.

  • platform/text/StringImpl.cpp:

(WebCore::StringImpl::threadsafeCopy):

9:30 PM Changeset in webkit [55831] by Chris Fleizach
  • 3 edits in trunk/LayoutTests

Unreviewed.

VoiceOver can navigate to hidden content in widget (WAI-ARIA)
​https://bugs.webkit.org/show_bug.cgi?id=35986

Wrong files were placed in skipped lists.

  • platform/gtk/Skipped:
  • platform/win/Skipped:
9:06 PM Changeset in webkit [55830] by Chris Fleizach
  • 3 edits in trunk/LayoutTests

Unreviewed.

VoiceOver can navigate to hidden content in widget (WAI-ARIA)
​https://bugs.webkit.org/show_bug.cgi?id=35986

Skip this test since it uses tables and lists, which are not
supported on Tiger/Leopard.

  • platform/mac-leopard/Skipped:
  • platform/mac-tiger/Skipped:
8:09 PM Changeset in webkit [55829] by dbates@webkit.org
  • 2 edits in trunk/WebKitTools

2010-03-10 Daniel Bates <​dbates@rim.com>

Reviewed by Simon Hausmann.

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

Updates the subroutine builtDylibPathForName() so that it adds the "d" suffix to
the QtWebKit library name on Windows. This change makes run-launcher work again
under Windows. Moreover, this change corresponds to the change made in change-
set 53924 <​http://trac.webkit.org/changeset/53924>.

  • Scripts/webkitdirs.pm:
8:05 PM Changeset in webkit [55828] by bweinstein@apple.com
  • 1 edit in trunk/JavaScriptCore/JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def

Windows build fix - add an missing symbol to be exported

7:23 PM Changeset in webkit [55827] by chang.shu@nokia.com
  • 6 edits in trunk

2010-03-10 Chang Shu <​chang.shu@nokia.com>

Reviewed by Darin Adler.

While calculating alpha channel, convert the floating point value to
an integer in [0, 256) with equal distribution.
​https://bugs.webkit.org/show_bug.cgi?id=22150

  • css/CSSParser.cpp: (WebCore::CSSParser::parseColorParameters):

2010-03-10 Chang Shu <​chang.shu@nokia.com>

Reviewed by Darin Adler.

While calculating alpha channel, convert the floating point value to
an integer in [0, 256) with equal distribution.
Updated test results based on this new behavior.
​https://bugs.webkit.org/show_bug.cgi?id=22150

  • fast/canvas/canvas-alphaImageData-behavior-expected.txt:
  • fast/canvas/canvas-alphaImageData-behavior.js:
  • platform/qt/Skipped:
6:38 PM Changeset in webkit [55826] by abarth@webkit.org
  • 1 edit
    2 deletes in trunk/LayoutTests

2010-03-10 Adam Barth <​abarth@webkit.org>

Unreviewed.

http/tests/security/xss-DENIED-window-open-javascript-url.html timed out on Windows Debug Bot
​https://bugs.webkit.org/show_bug.cgi?id=33349

Remove another test that has the same problem. We need to find a way
to relaibly test what happens when you mix JavaScript URLs and
window.open.

  • http/tests/security/xss-DENIED-window-open-javascript-url-leading-format-char-expected.txt: Removed.
  • http/tests/security/xss-DENIED-window-open-javascript-url-leading-format-char.html: Removed.
6:36 PM Changeset in webkit [55825] by barraclough@apple.com
  • 13 edits in trunk

​https://bugs.webkit.org/show_bug.cgi?id=35991
Would be faster to not use a thread specific to implement StringImpl::empty()

Reviewed by Darin Adler, Geoffrey Garen, Maciej Stachowiak.

JavaScriptCore:

Change JSC::UStringImpl's implementation of empty() match to match StringImpl's new implementation
(use a static defined within the empty() method), and change the interface to match too (return
a pointer not a reference).

~0% performance impact (possible minor progression from moving empty() from .h to .cpp).

(JSC::Identifier::add):
(JSC::Identifier::addSlowCase):

  • runtime/PropertyNameArray.cpp:

(JSC::PropertyNameArray::add):

  • runtime/UString.cpp:

(JSC::initializeUString):
(JSC::UString::UString):

  • runtime/UStringImpl.cpp:

(JSC::UStringImpl::empty):
(JSC::UStringImpl::create):

  • runtime/UStringImpl.h:

(JSC::UStringImpl::adopt):
(JSC::UStringImpl::createUninitialized):
(JSC::UStringImpl::tryCreateUninitialized):

WebCore:

Copy JavaScriptCore in making 'static' strings threadsafe, make the empty string a static,
shared by all threads.

~2% progression on Dromaeo DOM core & JS lib tests.

  • platform/ThreadGlobalData.cpp:

(WebCore::ThreadGlobalData::ThreadGlobalData):
(WebCore::ThreadGlobalData::~ThreadGlobalData):

  • platform/ThreadGlobalData.h:

(WebCore::ThreadGlobalData::eventNames):

  • platform/text/StringImpl.cpp:

(WebCore::StringImpl::StringImpl):
(WebCore::StringImpl::empty):

  • platform/text/StringImpl.h:

(WebCore::StringImpl::deref):
(WebCore::StringImpl::hasOneRef):

6:19 PM Changeset in webkit [55824] by Simon Fraser
  • 5 edits in trunk/WebKit/mac

2010-03-10 Simon Fraser <Simon Fraser>

Reviewed by Darin Adler.

​https://bugs.webkit.org/show_bug.cgi?id=35975
<rdar://problem/7739922> Flash 10.1b crashes when generating snapshots

Do a version check before sending a drawRect event to a Flash plugin,
since 10.1.d51 has a bug that crashes when called this way.

  • Plugins/Hosted/WebHostedNetscapePluginView.mm: (-[WebHostedNetscapePluginView drawRect:]):
  • Plugins/WebBaseNetscapePluginView.h:
  • Plugins/WebBaseNetscapePluginView.mm: (-[WebBaseNetscapePluginView supportsSnapshotting]):
  • Plugins/WebNetscapePluginView.mm: (-[WebNetscapePluginView drawRect:]):
6:18 PM Changeset in webkit [55823] by dumi@chromium.org
  • 21 edits
    9 adds in trunk

WebCore: Adding support for the optional creation callback that could be
passed to openDatabase().

Reviewed by Adam Barth.

Tests: storage/open-database-creation-callback.html

storage/open-database-creation-callback-isolated-world.html

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

  • Android.jscbindings.mk
  • Android.v8bindings.mk
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSDOMWindowCustom.cpp:

(WebCore::JSDOMWindow::openDatabase):

  • bindings/js/JSDatabaseCallback.cpp: Added.

(WebCore::JSDatabaseCallback::JSDatabaseCallback):
(WebCore::JSDatabaseCallback::~JSDatabaseCallback):
(WebCore::JSDatabaseCallback::handleEvent):

  • bindings/js/JSDatabaseCallback.h: Added.

(WebCore::JSDatabaseCallback::create):

  • bindings/v8/custom/V8DOMWindowCustom.cpp:

(WebCore::V8DOMWindow::openDatabaseCallback):

  • bindings/v8/custom/V8DatabaseCallback.cpp: Added.

(WebCore::V8DatabaseCallback::V8DatabaseCallback):
(WebCore::V8DatabaseCallback::~V8DatabaseCallback):
(WebCore::V8DatabaseCallback::handleEvent):

  • bindings/v8/custom/V8DatabaseCallback.h: Added.

(WebCore::V8DatabaseCallback::create):

  • page/DOMWindow.cpp:

(WebCore::DOMWindow::openDatabase):

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

(WebCore::DatabaseCreationCallbackTask::create):
(WebCore::DatabaseCreationCallbackTask::performTask):
(WebCore::DatabaseCreationCallbackTask::DatabaseCreationCallbackTask):
(WebCore::Database::openDatabase):
(WebCore::Database::Database):
(WebCore::Database::performOpenAndVerify):
(WebCore::Database::performCreationCallback):

  • storage/Database.h:

(WebCore::Database::isNew):

  • storage/DatabaseCallback.h: Added.

(WebCore::DatabaseCallback::~DatabaseCallback):

  • workers/WorkerContext.cpp:

(WebCore::WorkerContext::openDatabase):

  • workers/WorkerContext.h:

LayoutTests: Adding a test to test the creation callback passed to
openDatabase() and another one to make sure it is executed in the
correct world.

Reviewed by Adam Barth.

Also, fixing fast/frames/sandboxed-iframe-storage.html: calling
openDatabase() without at least 4 parameters should always fail,
because the database name, version, description and estimated size
are not optional according to the spec.

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

  • fast/frames/resources/sandboxed-iframe-storage-allowed.html:
  • fast/frames/resources/sandboxed-iframe-storage-disallowed.html:
  • fast/frames/sandboxed-iframe-storage-expected.txt:
  • platform/qt/Skipped:
  • storage/open-database-creation-callback-expected.txt: Added.
  • storage/open-database-creation-callback.html: Added.
  • storage/open-database-creation-callback-isolated-world-expected.txt: Added.
  • storage/open-database-creation-callback-isolated-world.html: Added.
6:07 PM Changeset in webkit [55822] by abarth@webkit.org
  • 3 edits
    2 adds in trunk

2010-03-10 Justin Schuh <​jschuh@chromium.org>

Reviewed by Adam Barth.

Make Chrome consistently handle leading format characters in URLs

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

  • http/tests/security/xss-DENIED-window-open-javascript-url-leading-format-char-expected.txt: Added.
  • http/tests/security/xss-DENIED-window-open-javascript-url-leading-format-char.html: Added.

2010-03-10 Justin Schuh <​jschuh@chromium.org>

Reviewed by Adam Barth.

Make Chrome consistently handle leading format characters in URLs

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

Test: http/tests/security/xss-DENIED-window-open-javascript-url-leading-format-char.html

  • platform/KURLGoogle.cpp:
5:19 PM Changeset in webkit [55821] by jianli@chromium.org
  • 14 edits
    2 adds in trunk/WebCore

Implementing DOMFormData class.
​https://bugs.webkit.org/show_bug.cgi?id=35707

Reviewed by Dmitry Titov.

This patch only addresses the implementation of DOMFormData class and
moves the FormData construction logic from HTMLFormElement::createFormData
to FormData::create() so that it can be used by both HTMLFormElement
and XMLHttpRequest.

The DOMFormData IDL interface will be exposed in another patch and the
test will be added then.

  • Android.mk:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • html/DOMFormData.cpp: Added.
  • html/DOMFormData.h: Added.
  • html/FormDataList.h:

(WebCore::FormDataList::encoding):

  • html/HTMLFormElement.cpp:

(WebCore::HTMLFormElement::createFormData):
(WebCore::HTMLFormElement::submit):

  • html/HTMLFormElement.h:
  • platform/network/FormData.cpp:

(WebCore::FormData::create):
(WebCore::FormData::createMultiPart):
(WebCore::FormData::appendDOMFormData):

  • platform/network/FormData.h:

(WebCore::FormData::boundary):

  • xml/XMLHttpRequest.cpp:

(WebCore::XMLHttpRequest::send):

  • xml/XMLHttpRequest.h:
4:59 PM Changeset in webkit [55820] by darin@chromium.org
  • 4 edits
    3 adds in trunk/WebKit/chromium

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

Reviewed by Jeremy Orlow.

Add API wrapper for WebCore::SerializedScriptValue
​https://bugs.webkit.org/show_bug.cgi?id=35988

WebSerializedScriptValue is a wrapper around SerializedScriptValue.

WebPrivatePtr is used to simplify wrapping reference counted WebCore
types. This class is used to cleanup WebHistoryItem, and will be used
to clean up other classes in the WebKit API in a follow-up patch.

  • WebKit.gyp:
  • public/WebHistoryItem.h:
  • public/WebPrivatePtr.h: Added.
  • public/WebSerializedScriptValue.h: Added.
  • src/WebHistoryItem.cpp:
  • src/WebSerializedScriptValue.cpp: Added.
4:43 PM Changeset in webkit [55819] by Chris Fleizach
  • 3 edits in trunk/LayoutTests

Skipping test on GTK and WIN. No Review.

VoiceOver can navigate to hidden content in widget (WAI-ARIA)
​https://bugs.webkit.org/show_bug.cgi?id=35986

Thought this test might pass on the other platforms, but no.

  • platform/gtk/Skipped:
  • platform/win/Skipped:
4:36 PM Changeset in webkit [55818] by Chris Fleizach
  • 18 edits
    2 adds in trunk

VoiceOver can navigate to hidden content in widget (WAI-ARIA)
​https://bugs.webkit.org/show_bug.cgi?id=35986

Reviewed by Beth Dakin.

WebCore:

Elements that are subclassers of AXRenderObject have not been respecting
aria-hidden and other cases that would make them ignored. This applies
to tables, rows, cells, select boxes, sliders, and lists. Select boxes
also need to make sure their option elements respect aria-hidden as well.

Test: accessibility/aria-hidden-with-elements.html

  • accessibility/AccessibilityList.cpp:

(WebCore::AccessibilityList::accessibilityIsIgnored):

  • accessibility/AccessibilityListBox.cpp:

(WebCore::AccessibilityListBox::addChildren):
(WebCore::AccessibilityListBox::accessibilityIsIgnored):
(WebCore::AccessibilityListBox::doAccessibilityHitTest):

  • accessibility/AccessibilityListBox.h:
  • accessibility/AccessibilityListBoxOption.cpp:

(WebCore::AccessibilityListBoxOption::accessibilityIsIgnored):

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

(WebCore::AccessibilityRenderObject::accessibilityIsIgnoredCommon):
(WebCore::AccessibilityRenderObject::accessibilityIsIgnored):
(WebCore::AccessibilityRenderObject::doAccessibilityHitTest):

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

(WebCore::AccessibilitySlider::accessibilityIsIgnored):

  • accessibility/AccessibilitySlider.h:

(WebCore::AccessibilitySlider::roleValue):
(WebCore::AccessibilitySliderThumb::roleValue):
(WebCore::AccessibilitySliderThumb::accessibilityIsIgnored):

  • accessibility/AccessibilityTable.cpp:

(WebCore::AccessibilityTable::accessibilityIsIgnored):

  • accessibility/AccessibilityTableCell.cpp:

(WebCore::AccessibilityTableCell::accessibilityIsIgnored):

  • accessibility/AccessibilityTableColumn.cpp:

(WebCore::AccessibilityTableColumn::accessibilityIsIgnored):

  • accessibility/AccessibilityTableColumn.h:

(WebCore::AccessibilityTableColumn::roleValue):

  • accessibility/AccessibilityTableHeaderContainer.cpp:

(WebCore::AccessibilityTableHeaderContainer::accessibilityIsIgnored):

  • accessibility/AccessibilityTableHeaderContainer.h:
  • accessibility/AccessibilityTableRow.cpp:

(WebCore::AccessibilityTableRow::accessibilityIsIgnored):

LayoutTests:

  • accessibility/aria-hidden-with-elements-expected.txt: Added.
  • accessibility/aria-hidden-with-elements.html: Added.
4:14 PM Changeset in webkit [55817] by dimich@chromium.org
  • 2 edits in trunk/JavaScriptCore

Not reviewed, fixing Snow Leopard build.

  • wtf/mac/MainThreadMac.mm: Forgot 'static' for a new local function.

(WTF::postTimer):

4:08 PM Changeset in webkit [55816] by dimich@chromium.org
  • 7 edits in trunk

Make Document::postTask to use a single queue of tasks, to fire them in order
​https://bugs.webkit.org/show_bug.cgi?id=35943

Reviewed by Darin Adler.

JavaScriptCore:

The patch uses CFRunLoopTimer to schedule execution of tasks instead of performSelectorOnMainThread which apparently can starve other event sources.
The timer is used when the schedule request is coming on the main thread itself. This happens when the task is posted on the main thread or
when too many tasks are posted and the queue does 'stop and re-schedule' to make sure run loop has a chance to execute other events.

  • wtf/mac/MainThreadMac.mm:

(WTF::timerFired):
(WTF::postTimer):
(WTF::scheduleDispatchFunctionsOnMainThread): Use timer posted to the current RunLoop if scheduling the task execution while on the main thread.

WebCore:

Test: existing worker-cloneport.html which was broken by initial patch in ​http://trac.webkit.org/changeset/55593.
Additional test which indirectly verifies the order of execution will come as part of ​https://bugs.webkit.org/show_bug.cgi?id=34726

  • dom/Document.cpp:

(WebCore::Document::postTask): Always use the same task queue, independent of what thread is posting the task.

LayoutTests:

Updated the test since the order of events coming from independent task sources has changed.
This is a behavior change, but is consistent with the spec (which specifically says these events may be fired
in any order) and the last version of FF (3.5.8) which was completing the test successfully.

  • http/tests/appcache/top-frame-2-expected.txt:
  • http/tests/appcache/top-frame-2.html:
3:40 PM Changeset in webkit [55815] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

Windows build fix: added new symbol.

3:28 PM Changeset in webkit [55814] by darin@chromium.org
  • 10 edits in trunk

2010-03-10 Sanjeev Radhakrishnan <​sanjeevr@chromium.org>

Reviewed by Darin Fisher.

Allow a plugin to participate in the browser's print workflow.
​https://bugs.webkit.org/show_bug.cgi?id=35550

  • loader/PluginDocument.cpp:
  • loader/PluginDocument.h:
3:27 PM Changeset in webkit [55813] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

Windows build fix: removed old symbol.

3:19 PM Changeset in webkit [55812] by eric@webkit.org
  • 1 edit
    4 adds in trunk/WebKit

2010-03-10 Leandro Pereira <​leandro@profusion.mobi>

Reviewed by Holger Freyther.

Add EFL port files to efl/WebCoreSupport.
​http://webkit.org/b/35913

  • efl/WebCoreSupport/ChromeClientEfl.cpp: Added.
  • efl/WebCoreSupport/ChromeClientEfl.h: Added.
3:11 PM Changeset in webkit [55811] by ggaren@apple.com
  • 5 edits in trunk/JavaScriptCore

Refactored fastCheckConsistency to match some review comments:

  • renamed fastCheckConsistency to fastMallocSize, and changed ValueCheck to ASSERT that a pointer's fastMallocSize is not 0.
  • implemented a version of fastMallocSize for tcmalloc.


Reviewed by Alexey Proskuryakov, Darin Adler, and Sam Weinig.

Also moved some pre-existing code around to avoid a problem related to
mismatched #define/#undef of malloc/free in this source file.

(WTF::fastMallocSize): Renamed. Fixed indentation.

(WTF::TCMalloc_PageHeap::scavenge): Removed an incorrect ASSERT that
got in the way of testing the tcmalloc implementation. (More information
on why this ASSERT is incorrect is in <rdar://problem/7165917>.)

(WTF::TCMallocStats::fastMallocSize): Implemented for tcmalloc.

  • wtf/FastMalloc.h: Updated for rename.
  • wtf/ValueCheck.h:

(WTF::): Moved the ASSERT that used to be in fastCheckConsistency here.

3:00 PM Changeset in webkit [55810] by jam@chromium.org
  • 2 edits in trunk/WebKit/chromium

2010-03-10 John Abd-El-Malek <​jam@chromium.org>

Reviewed by Nate Chapin.

[chromium]: Fix regression in mouse capture on Mac/Linux
​https://bugs.webkit.org/show_bug.cgi?id=35987

  • src/WebViewImpl.cpp:
2:44 PM Changeset in webkit [55809] by eric@webkit.org
  • 2 edits in trunk/WebKit/win

2010-03-10 Eric Uhrhane <​ericu@chromium.org>

Reviewed by David Levin.

The build fix for my patch on bug #35763 wasn't quite right--it removed
the call entirely, instead of replacing it with the new API. This adds
the call to Database::setIsAvailable.

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

2:26 PM Changeset in webkit [55808] by eric@webkit.org
  • 5 edits in trunk/WebKit/chromium

2010-03-10 Evan Stade <​estade@chromium.org>

Reviewed by David Levin.

[chromium] add functionality to activate the focused node in a WebView
​https://bugs.webkit.org/show_bug.cgi?id=35407

  • public/WebDocument.h:
  • public/WebNode.h:
  • src/WebDocument.cpp: (WebKit::WebDocument::focusedNode): Added
  • src/WebNode.cpp: (WebKit::WebNode::simulateClick): Added
2:19 PM Changeset in webkit [55807] by Adam Roben
  • 2 edits in trunk/WebKitTools

Roll out the prepare-ChangeLog part of r55870

This change wasn't needed (prepare-ChangeLog calls svn-create-patch
when it's asked to print out diffs for the user) and was screwing up
its ability to find changed function names.

  • Scripts/prepare-ChangeLog:

(diffCommand): Change the options we pass to svn-diff back to their
pre-r55870 form.

2:09 PM Changeset in webkit [55806] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-03-10 Ilya Tikhonovsky <​loislo@loislo-macbookpro.local>

Reviewed by Pavel Feldman.

Sidebar resize element height was adjusted.
Display name for Function Call details was adjusted.

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

  • inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel.prototype._refreshRecords): (WebInspector.TimelinePanel.FormattedRecord.prototype._getRecordDetails):
1:59 PM Changeset in webkit [55805] by jam@chromium.org
  • 5 edits in trunk/WebKit/chromium

2010-03-10 John Abd-El-Malek <​jam@chromium.org>

Reviewed by Darin Fisher.

​https://bugs.webkit.org/show_bug.cgi?id=35952
Propagate mouse wheel events to Pepper plugins.

  • src/WebInputEventConversion.cpp: (WebKit::WebMouseWheelEventBuilder::WebMouseWheelEventBuilder):
  • src/WebInputEventConversion.h:
  • src/WebPluginContainerImpl.cpp: (WebKit::WebPluginContainerImpl::handleEvent): (WebKit::WebPluginContainerImpl::handleMouseEvent): (WebKit::WebPluginContainerImpl::handleWheelEvent): (WebKit::WebPluginContainerImpl::handleKeyboardEvent):
  • src/WebPluginContainerImpl.h:
1:58 PM Changeset in webkit [55804] by tonikitoo@webkit.org
  • 2 edits in trunk/LayoutTests

2010-03-10 Antonio Gomes <​tonikitoo@webkit.org>

The test was introduced in r55796, it has been failing in Gtk bots since then.

Filed bug ​https://bugs.webkit.org/show_bug.cgi?id=35984.

  • platform/gtk/Skipped: fast/events/spatial-navigation/snav-iframe-with-offscreen-focusable-element.html skipped until fix.
1:55 PM Changeset in webkit [55803] by robert@webkit.org
  • 9 edits in trunk

2010-03-10 Robert Hogan <​robert@webkit.org>

Reviewed by Simon Hausmann.

[Qt] Add Support for WebKitEnableCaretBrowsing to Qt DRT

Unskip test fast/events/multiline-link-arrow-navigation.html

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

  • platform/qt/Skipped:

2010-03-10 Robert Hogan <​robert@webkit.org>

Reviewed by Simon Hausmann.

[Qt] Add Support for WebKitEnableCaretBrowsing to Qt DRT

Unskip test fast/events/multiline-link-arrow-navigation.html

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

  • DumpRenderTree/qt/DumpRenderTreeQt.cpp: (WebCore::WebPage::resetSettings):
  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::overridePreference): (LayoutTestController::setCaretBrowsingEnabled):
  • DumpRenderTree/qt/LayoutTestControllerQt.h:

2010-03-10 Robert Hogan <​robert@webkit.org>

Reviewed by Simon Hausmann.

[Qt] Add Support for WebKitEnableCaretBrowsing to Qt DRT

Unskip test fast/events/multiline-link-arrow-navigation.html
Fix typo (superfluous space) in QWebSettings docs.

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

  • Api/qwebpage.cpp: (qt_drt_enableCaretBrowsing):
  • Api/qwebsettings.cpp: (QWebSettingsPrivate::apply): fix typo in docs
1:52 PM Changeset in webkit [55802] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-03-10 Charles Wei <​charles.wei@torchmobile.com.cn>

Reviewed by George Staikos.

Fix ​https://bugs.webkig.org/show_bug.cgi?id=35207
When XHTMLMP is enabled, all xhtml documents fail to render for webkit portings wich use libxml2 tokenizer.

No new tests as all the xhtml test cases in LayoutTests/dom/xhtml fails without this fix when XHTMLMP is enabled.

  • dom/Document.cpp: (WebCore::Document::isXHTMLMPDocument):
  • dom/XMLTokenizerLibxml2.cpp: (WebCore::XMLTokenizer::internalSubset): (WebCore::externalSubsetHandler):
1:34 PM Changeset in webkit [55801] by beidson@apple.com
  • 5 edits
    1 add in trunk/LayoutTests

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

Reviewed by Sam Weinig.

Turns out the previous attempt to clear/reset cookies wasn't working because of the way
the URLs for the .js and its .cgi were being resolved.

So this is a better way to make the cookies consistent both before and after these tests.

If this fails to make cookie-related tests reliable, then we might have to bite the bullet
and give DRT itself the ability to forcibly remove cookies via API methods.

This would be bulletproof and could be automatic if a test declares itself "cookie sensitive"
but we'd lose the ability to reliably run these tests in the browser. :(

  • http/tests/cookies/resources/resetCookies.js: Added.
  • http/tests/cookies/resources/third-party-cookie-relaxing-iframe.html:
  • http/tests/cookies/third-party-cookie-relaxing.html:
  • http/tests/plugins/resources/third-party-cookie-accept-policy-iframe.html:
  • http/tests/plugins/third-party-cookie-accept-policy.html:
1:34 PM Changeset in webkit [55800] by eric@webkit.org
  • 6 edits in trunk

2010-03-10 Garret Kelly <​gdk@chromium.org>

Reviewed by Darin Fisher.

Adding all of the touch-related sources into Chromium's WebCore build.
​https://bugs.webkit.org/show_bug.cgi?id=35874

Patch tested against the try bots, but the feature is exercised in the
fast/events/touch tests.

  • WebCore.gypi:

2010-03-10 Garret Kelly <​gdk@chromium.org>

Reviewed by Darin Fisher.

Add support for converting WebTouchEvents to PlatformTouchEvents and
routing them into the EventHandler for the frame in which they
originate.
​https://bugs.webkit.org/show_bug.cgi?id=35874

  • src/ChromeClientImpl.h: (WebKit::ChromeClientImpl::needTouchEvents):
  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::touchEvent): Handle incoming WebTouchEvents, converting them to PlatformTouchEvents and sending them to the EventHandler. (WebKit::WebViewImpl::handleInputEvent): Now routes WebTouchEvents to the touchEvent handler.
  • src/WebViewImpl.h:
12:37 PM Changeset in webkit [55799] by pfeldman@chromium.org
  • 6 edits in trunk/WebCore

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

Reviewed by Timothy Hatcher.

Web Inspector: now that audits panel is added, add it to the
enums (for storing last active, etc.).

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

  • inspector/InspectorController.cpp: (WebCore::InspectorController::setWindowVisible): (WebCore::InspectorController::specialPanelForJSName):
  • inspector/InspectorController.h: (WebCore::InspectorController::):
  • inspector/InspectorFrontend.cpp: (WebCore::InspectorFrontend::showPanel):
  • inspector/front-end/inspector.js: (WebInspector.showAuditsPanel):
12:34 PM QtWebKitBugs edited by jocelyn.turcotte@nokia.com
(diff)
12:33 PM Changeset in webkit [55798] by Nate Chapin
  • 49 edits
    1 delete in trunk

2010-03-10 Nate Chapin <Nate Chapin>

Reviewed by Dimitri Glazkov.

[V8] Instead of describing the type of a wrapped v8 object
with an enum value, use a pointer to struct with more complete
information.

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

Refactoring only, so new tests.

  • bindings/scripts/CodeGeneratorV8.pm:
  • bindings/v8/DOMData.cpp:
  • bindings/v8/DOMData.h:
  • bindings/v8/NPV8Object.cpp:
  • bindings/v8/V8Collection.h:
  • bindings/v8/V8DOMWindowShell.cpp:
  • bindings/v8/V8DOMWindowShell.h:
  • bindings/v8/V8DOMWrapper.cpp:
  • bindings/v8/V8DOMWrapper.h:
  • bindings/v8/V8GCController.cpp:
  • bindings/v8/V8Helpers.cpp:
  • bindings/v8/V8Helpers.h:
  • bindings/v8/V8Index.cpp: Removed.
  • bindings/v8/V8Index.h:
  • bindings/v8/V8NPObject.cpp:
  • bindings/v8/V8Proxy.cpp:
  • bindings/v8/V8Proxy.h:
  • bindings/v8/WorkerContextExecutionProxy.cpp:
  • bindings/v8/custom/V8DOMWindowCustom.cpp:
  • bindings/v8/custom/V8HTMLAudioElementConstructor.cpp:
  • bindings/v8/custom/V8HTMLAudioElementConstructor.h:
  • bindings/v8/custom/V8HTMLImageElementConstructor.cpp:
  • bindings/v8/custom/V8HTMLImageElementConstructor.h:
  • bindings/v8/custom/V8HTMLOptionElementConstructor.cpp:
  • bindings/v8/custom/V8HTMLOptionElementConstructor.h:
  • bindings/v8/custom/V8InjectedScriptHostCustom.cpp:
  • bindings/v8/custom/V8MessageChannelConstructor.cpp:
  • bindings/v8/custom/V8SharedWorkerCustom.cpp:
  • bindings/v8/custom/V8WebGLArrayBufferCustom.cpp:
  • bindings/v8/custom/V8WebGLArrayCustom.h:
  • bindings/v8/custom/V8WebGLByteArrayCustom.cpp:
  • bindings/v8/custom/V8WebGLFloatArrayCustom.cpp:
  • bindings/v8/custom/V8WebGLIntArrayCustom.cpp:
  • bindings/v8/custom/V8WebGLShortArrayCustom.cpp:
  • bindings/v8/custom/V8WebGLUnsignedByteArrayCustom.cpp:
  • bindings/v8/custom/V8WebGLUnsignedIntArrayCustom.cpp:
  • bindings/v8/custom/V8WebGLUnsignedShortArrayCustom.cpp:
  • bindings/v8/custom/V8WebKitCSSMatrixConstructor.cpp:
  • bindings/v8/custom/V8WebKitPointConstructor.cpp:
  • bindings/v8/custom/V8WebSocketCustom.cpp:
  • bindings/v8/custom/V8WorkerCustom.cpp:
  • bindings/v8/custom/V8XMLHttpRequestConstructor.cpp:
  • bindings/v8/custom/V8XSLTProcessorCustom.cpp:
12:18 PM Changeset in webkit [55797] by tonikitoo@webkit.org
  • 1 edit
    3 adds in trunk/WebCore

Extend keyboard navigation to allow directional navigation (Manual tests)
​https://bugs.webkit.org/show_bug.cgi?id=18662

Reviewed by Dave Hyatt.
Patch by Antonio Gomes <​tonikitoo@webkit.org>

Adds a manual tests for the Spatial Navigation feature comprasing varios
Html focusable elements (e.g. <a>, <table>, <iframe>) and much of the logic
provided by the feature.

  • manual-tests/spatial-navigation/links.html: Added.
  • manual-tests/spatial-navigation/spatial-navigation-test-cases.html: Added.
12:13 PM Changeset in webkit [55796] by tonikitoo@webkit.org
  • 1 edit
    10 adds in trunk/LayoutTests

Extend keyboard navigation to allow directional navigation (LayoutTests - part V)
​https://bugs.webkit.org/show_bug.cgi?id=18662

Reviewed by Dave Hyatt.
Patch by Antonio Gomes <​tonikitoo@webkit.org>

This patch adds three LayoutTests that ensure the correctness of Spatial Navigation algorithm
in the following iframe related traversals:

1) All focusable elements in a <iframe> should be accessible.
2) Focusable elements in an <iframe> should be accessible even when

current focused element is in another <iframe> in the same page.

  • fast/events/spatial-navigation/snav-iframe-no-focusable-content.html: Added.
  • fast/events/spatial-navigation/snav-iframe-no-scrollable-content.html: Added.
  • fast/events/spatial-navigation/snav-iframe-with-offscreen-focusable-element.html: Added.
12:10 PM Changeset in webkit [55795] by tonikitoo@webkit.org
  • 1 edit
    2 adds in trunk/LayoutTests

Extend keyboard navigation to allow directional navigation (LayoutTests - part IV)
​https://bugs.webkit.org/show_bug.cgi?id=18662

Reviewed by Dave Hyatt.
Patch by Antonio Gomes <​tonikitoo@webkit.org>

This patch adds a LayoutTest that ensures that:

1) There is no unit overflow in the Spatial Navigation algorithm while

calculating the best node candidate to move focus to. To test that this
page positions some elements 10000000 pixels far from each other (distance
that can considered large enough for most of the Web Pages on the
Internet).

2) A best focusable candidate only gets focused if it is visible in the

current Viewport. Scroll-in-direction is performed otherwise.

  • fast/events/spatial-navigation/snav-unit-overflow-and-scroll-in-direction.html: Added.
12:08 PM Changeset in webkit [55794] by tonikitoo@webkit.org
  • 1 edit
    4 adds in trunk/LayoutTests

Extend keyboard navigation to allow directional navigation (LayoutTests - part III)
​https://bugs.webkit.org/show_bug.cgi?id=18662

Reviewed by Dave Hyatt.
Patch by Antonio Gomes <​tonikitoo@webkit.org>

This patch adds two LayoutTests to ensure the correctness of the "Fully aligned"
precedence logic implemented by Spatial Navigation algorithm, either vertically or
horizontally: targets whose middle falls between the top and bottom of the current
focused element are preferably to move focus to, even if it is not the shortest distance.

  • fast/events/spatial-navigation/snav-fully-aligned-vertically.html: Added.
  • fast/events/spatial-navigation/snav-fully-aligned-horizontally.html: Added.
12:06 PM Changeset in webkit [55793] by eric@webkit.org
  • 1 edit
    2 adds in trunk/WebCore

2010-03-10 Leandro Pereira <​leandro@profusion.mobi>

Reviewed by Kenneth Rohde Christiansen.

Add EFL port files to platform/efl.
​http://webkit.org/b/35890

  • platform/efl/ScrollbarEfl.h: Added.
  • platform/efl/ScrollbarThemeEfl.h: Added.
11:47 AM Changeset in webkit [55792] by eric@webkit.org
  • 2 edits in trunk/WebKit/chromium

2010-03-10 Garret Kelly <​gdk@chromium.org>

Reviewed by Darin Fisher.

Fixing build warning caused by using 0 instead of NULL as the sentiel
in a varargs call.
​https://bugs.webkit.org/show_bug.cgi?id=35898

  • src/gtk/WebInputEventFactory.cpp:
11:26 AM Changeset in webkit [55791] by robert@webkit.org
  • 2 edits in trunk/WebKitTools

2010-03-10 Robert Hogan <​robert@webkit.org>

Reviewed by Holger Freyther.

QtLauncher: Fix typo in conditional statement in

WebViewGraphicsBased::setFrameRateMeasurementEnabled.

'=' should be '=='!

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

  • QtLauncher/webview.cpp: (WebViewGraphicsBased::setFrameRateMeasurementEnabled):
11:17 AM Changeset in webkit [55790] by pfeldman@chromium.org
  • 5 edits
    1 add in trunk/WebCore

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

Reviewed by Timothy Hatcher.

Web Inspector: Audits panel needs an icon.
Landing icon attached by Timothy. Enabling panel!

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

  • WebCore.gypi:
  • inspector/front-end/Images/auditsIcon.png: Added.
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.css:
10:59 AM Changeset in webkit [55789] by kov@webkit.org
  • 3 edits in trunk/WebCore

2010-03-10 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Roll-out r55657, as Xan noticed a bad degradation in playing
youtube HTML5 videos with it, that is fixed with it
reverted. Looks like we'll need to work a bit more on this.

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::MediaPlayerPrivate::MediaPlayerPrivate): (WebCore::MediaPlayerPrivate::~MediaPlayerPrivate): (WebCore::MediaPlayerPrivate::volumeChangedTimerFired): (WebCore::MediaPlayerPrivate::volumeChanged): (WebCore::MediaPlayerPrivate::processBufferingStats): (WebCore::MediaPlayerPrivate::fillTimerFired): (WebCore::MediaPlayerPrivate::maxTimeLoaded): (WebCore::MediaPlayerPrivate::updateStates): (WebCore::MediaPlayerPrivate::muteChangedTimerFired): (WebCore::MediaPlayerPrivate::muteChanged):
  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.h:
10:41 AM Changeset in webkit [55788] by kov@webkit.org
  • 2 edits in trunk/WebCore

2010-03-10 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Rubber-stamped by Xan Lopez.

Fix ready state when buffering under PLAYING. We should not go
back to HaveNothing in that case.

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::MediaPlayerPrivate::updateStates):
10:38 AM Changeset in webkit [55787] by adachan@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Adam Roben.

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

Retrieve any dwItemData that's set on the context menu item in contextMenuItemByIdOrPosition().

  • platform/win/ContextMenuWin.cpp:
10:31 AM Changeset in webkit [55786] by Chris Jerdonek
  • 5 edits in trunk/WebKitTools

Refactored and cleaned up the code for unit-testing logging.

Reviewed by Shinichiro Hamaji.

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

  • Scripts/webkitpy/init/logtesting.py:
    • Added more information to the module docstring.
    • Added an assertMessages() method to the UnitTestLogStream class. This simplifies the calling code.
    • Renamed the UnitTestLog class to LogTesting, and reformulated it as follows:
      • Moved the logging configuration code from the init method to a new static setUp() method.
      • Documented the init method to be private.
      • Improved the code so that the root logger does not have its logging level changed. Instead we set the handler's level. This makes the unit testing more unintrusive.
      • Updated the assertMessages() method to call the UnitTestLogStream class's assertMessages() method.
      • More fully documented the class.
  • Scripts/webkitpy/style/checker.py:
    • Added a logger parameter to the configure_logging() method. This allows us to prevent test messages from being sent to the root logger during unit testing, which may be rendering to the screen, etc.
    • Simplified the code by removing the _LevelLoggingFilter class.
    • Replaced the _LevelLoggingFilter class with a one-line lambda expression in configure_logging().
  • Scripts/webkitpy/style/checker_unittest.py:
    • Changed relative imports to absolute to comply more with PEP8.
    • In the ConfigureLoggingTest class:
      • Changed the setUp() method to prevent test messages from being propagated to the root logger.
      • Changed the _log() method to a data attribute.
      • Updated to accommodate changes to logtesting.py.
  • Scripts/webkitpy/style_references.py:
    • Updated an import statement.
9:36 AM WikiStart edited by evan@chromium.org
add a gdb section (diff)
9:35 AM Changeset in webkit [55785] by evan@chromium.org
  • 1 edit
    2 adds in trunk/WebKitTools

2010-03-10 Evan Martin <​evan@chromium.org>

Reviewed by Darin Adler.

Python code for GDB 7 to support native printing of some WebCore types.

  • gdb/webcore.py: Added.
9:06 AM Changeset in webkit [55784] by jorlow@chromium.org
  • 8 edits
    8 adds in trunk

2010-03-10 Jeremy Orlow <​jorlow@chromium.org>

Reviewed by Darin Fisher.

Add IndexedDatabase class and hook it up.
​https://bugs.webkit.org/show_bug.cgi?id=35927

This change is mostly just adding the plumbing necessary for
the IndexedDatabaseRequest and IndexedDatabaseSync (not written
yet).

This code is non-functional, so no tests (yet).

  • WebCore.gyp/WebCore.gyp:
  • WebCore.gypi:
  • platform/chromium/ChromiumBridge.h:
  • storage/IndexedDatabase.cpp: Added. (WebCore::IndexedDatabase::get):
  • storage/IndexedDatabase.h: Added. (WebCore::IndexedDatabase::~IndexedDatabase):
  • storage/IndexedDatabaseImpl.cpp: Added. (WebCore::IndexedDatabaseImpl::get): (WebCore::IndexedDatabaseImpl::IndexedDatabaseImpl): (WebCore::IndexedDatabaseImpl::~IndexedDatabaseImpl): (WebCore::IndexedDatabaseImpl::open):
  • storage/IndexedDatabaseImpl.h: Added.
  • storage/chromium/IndexedDatabase.cpp: Added. (WebCore::IndexedDatabase::get):
  • WebKit.gyp:
  • public/WebIndexedDatabase.h: Added. (WebKit::WebIndexedDatabase::~WebIndexedDatabase):
  • public/WebKitClient.h: (WebKit::WebKitClient::getIndexedDatabase):
  • src/ChromiumBridge.cpp: (WebCore::ChromiumBridge::getIndexedDatabase):
  • src/IndexedDatabaseProxy.cpp: Added. (WebCore::IndexedDatabaseProxy::create): (WebCore::IndexedDatabaseProxy::IndexedDatabaseProxy): (WebCore::IndexedDatabaseProxy::~IndexedDatabaseProxy): (WebCore::IndexedDatabaseProxy::open):
  • src/IndexedDatabaseProxy.h: Added.
8:56 AM Changeset in webkit [55783] by Dimitri Glazkov
  • 3 edits
    2 adds in trunk

Add one more parent check during node removal.
​https://bugs.webkit.org/show_bug.cgi?id=35818

Reviewed by Sam Weinig.

WebCore:

Test: fast/dom/Node/mutation-blur.html

  • dom/ContainerNode.cpp:

(WebCore::ContainerNode::removeChild): Added check.

LayoutTests:

  • fast/dom/Node/mutation-blur-expected.txt: Added.
  • fast/dom/Node/mutation-blur.html: Added.
8:50 AM Changeset in webkit [55782] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt] fast/multicol/hit-test-above-or-below.html fails
​https://bugs.webkit.org/show_bug.cgi?id=35973

The test was introduced in r55546, it has been failing since then.

  • platform/qt/Skipped: fast/multicol/hit-test-above-or-below.html skipped until fix.
8:34 AM Changeset in webkit [55781] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

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

Not reviewed: added missing quote to localized strings.

  • English.lproj/localizedStrings.js:
7:47 AM Changeset in webkit [55780] by Adam Roben
  • 3 edits in trunk/WebKitTools

Make svn-create-patch and prepare-ChangeLog show better section headings for ObjC files

This makes the text at the end of each "@@" line in a diff actually
show the ObjC method or interface that contains the change, rather
than whatever the most-recently-defined C function was.

Fixes <​http://webkit.org/b/35970>.

Reviewed by John Sullivan.

  • Scripts/svn-create-patch: Pass -F'[-+@]' to diff so that it will

treat any lines starting with -, +, or @ as section heading lines.
This works well for ObjC files, and shouldn't affect other types of
files.

  • Scripts/prepare-ChangeLog: Changed the options passed to diff to

match those used in svn-create-patch.

7:38 AM Changeset in webkit [55779] by jorlow@chromium.org
  • 1 edit
    3 adds in trunk/WebCore

Commit files that were supposed to go in with the last checkin.

7:34 AM Changeset in webkit [55778] by jorlow@chromium.org
  • 7 edits in trunk/WebCore

Baby steps towards IndexedDB: Start implementing callbacks.
​https://bugs.webkit.org/show_bug.cgi?id=35911

Reviewed by Dimitry Glazkov.

This patch adds some infastructure for IndexedDB callbacks in V8.
It also adds a stub of IDBDatabaseRequest. In the near future,
I'll gut the event based implementation code, flesh out IDBReqest
further, and start plumbing IndexedDatabaseRequest.

Code is not testible because it doesn't work (yet).

  • WebCore.gypi:
  • bindings/v8/DOMObjectsInclude.h:
  • bindings/v8/V8Index.cpp:
  • bindings/v8/V8Index.h:
  • bindings/v8/custom/V8CustomIDBCallback.h: Added.

(WebCore::V8CustomIDBCallback::create):
(WebCore::V8CustomIDBCallback::~V8CustomIDBCallback):
(WebCore::V8CustomIDBCallback::handleEvent):
(WebCore::V8CustomIDBCallback::V8CustomIDBCallback):

  • bindings/v8/custom/V8IndexedDatabaseRequestCustom.cpp:

(WebCore::V8IndexedDatabaseRequest::openCallback):

  • storage/IDBDatabaseRequest.h: Added.

(WebCore::IDBDatabaseRequest::request):

  • storage/IDBDatabaseRequest.idl: Added.
  • storage/IndexedDatabaseRequest.idl:
7:17 AM Changeset in webkit [55777] by kov@webkit.org
  • 1 edit
    1 add in trunk/WebKit/gtk/po

2010-03-10 Priit Laes <​plaes@plaes.org>

Reviewed by Gustavo Noronha.

Estonian translation.

  • et.po: Added.
5:23 AM Changeset in webkit [55776] by sullivan@apple.com
  • 4 edits in trunk/WebKit

<rdar://problem/7735387> input type other than text won't work with autofill
<​https://bugs.webkit.org/show_bug.cgi?id=35963>

Reviewed by Tim Hatcher.

WebKit/mac:

  • WebView/WebHTMLRepresentation.mm:

(-[WebHTMLRepresentation elementDoesAutoComplete:]):
Return true for any text field that's not a password, rather than only
for TEXT type.

WebKit/win:

  • WebFrame.cpp:

(WebFrame::elementDoesAutoComplete):
Return true for any text field that's not a password, rather than only
for TEXT type.

4:44 AM BuildingQtOnWindows edited by jocelyn.turcotte@nokia.com
(diff)
4:10 AM Changeset in webkit [55775] by Simon Hausmann
  • 2 edits in trunk/WebKitTools

Link QtLauncher against the WebKit library using a relative rpath.

Reviewed by Tor Arne Vestbø.

This makes the launcher and the lib relocatable.

  • QtLauncher/QtLauncher.pro:
2:46 AM Changeset in webkit [55774] by zecke@webkit.org
  • 2 edits
    3 adds in trunk/WebCore

[Qt] Non animated gifs are animated in QtWebKit

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

Properly map Qt animated and non-animated values to WebCore's
understanding of animated and non-animated images. Currently
we can not map anything to the cAnimationLoopNone value.

  • manual-tests/qt/qt-anim.gif: Added.
  • manual-tests/qt/qt-gif-test.html: Added.
  • manual-tests/qt/qt-noanim.gif: Added.
  • platform/graphics/qt/ImageDecoderQt.cpp:

(WebCore::ImageDecoderQt::repetitionCount):

2:44 AM Changeset in webkit [55773] by zecke@webkit.org
  • 3 edits
    2 adds in trunk

[CAIRO] DoS on iexploder test with high text stroke width.

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

Specifying a big text stroke width can make WebKitGTK+ spend
a very long time in the cairo library for stroking the path of
the text. The best way to prevent this from happening right now
is to not stroke paths with a certain width. Samuel proposed to
not stroke with a width that is twice the width of the text. The
reason to use twice the text width is that even one stroke of
any charachter to be drawn would cover the full width.

Test: fast/text/text-stroke-width-cairo-dos.html

WebCore:

  • platform/graphics/cairo/FontCairo.cpp:

(WebCore::Font::drawGlyphs):

Add a test case with the -webkit-text-stroke-width attribute
set to a high value to illustrate the issue.

  • fast/text/text-stroke-width-cairo-dos-expected.txt: Added.
  • fast/text/text-stroke-width-cairo-dos.html: Added.
1:41 AM Changeset in webkit [55772] by zecke@webkit.org
  • 4 edits in trunk/WebKitTools

[iExploder] Add new CSS Properties and HTML Attributes

The update-iexploder-cssproperties script was used to update
the various input files. The autobuffer HTML Attribute was removed
from WebKit and I manually added it back to the htmlattrs.in like
we have done it for other attributes in the past.

1:22 AM Changeset in webkit [55771] by pfeldman@chromium.org
  • 28 edits
    1 copy
    2 adds in trunk

2010-03-10 Andrey Kosyakov <​caseq@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: display list of active workers & support debugging
with fake workers.
​https://bugs.webkit.org/show_bug.cgi?id=35568

  • English.lproj/localizedStrings.js:
  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • bindings/js/JSInjectedScriptHostCustom.cpp: (WebCore::InjectedScriptHost::createInjectedScript): (WebCore::InjectedScriptHost::injectedScriptFor):
  • bindings/v8/custom/V8InjectedScriptHostCustom.cpp: (WebCore::InjectedScriptHost::createInjectedScript): (WebCore::InjectedScriptHost::injectedScriptFor):
  • dom/Document.cpp: (WebCore::Document::inspectorController):
  • dom/Document.h:
  • dom/ScriptExecutionContext.h: (WebCore::ScriptExecutionContext::inspectorController):
  • inspector/InjectedScriptHost.cpp: (WebCore::InjectedScriptHost::InjectedScriptHost): (WebCore::InjectedScriptHost::injectScript): (WebCore::InjectedScriptHost::nextWorkerId): (WebCore::InjectedScriptHost::didCreateWorker): (WebCore::InjectedScriptHost::willDestroyWorker):
  • inspector/InjectedScriptHost.h:
  • inspector/InjectedScriptHost.idl:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::populateScriptObjects): (WebCore::InspectorController::resetScriptObjects): (WebCore::InspectorController::didCommitLoad): (WebCore::InspectorController::didCreateWorker): (WebCore::InspectorController::willDestroyWorker):
  • inspector/InspectorController.h:
  • inspector/InspectorFrontend.cpp: (WebCore::InspectorFrontend::didCreateWorker): (WebCore::InspectorFrontend::willDestroyWorker):
  • inspector/InspectorFrontend.h:
  • inspector/InspectorWorkerResource.h: Added. (WebCore::InspectorWorkerResource::create): (WebCore::InspectorWorkerResource::id): (WebCore::InspectorWorkerResource::url): (WebCore::InspectorWorkerResource::isSharedWorker): (WebCore::InspectorWorkerResource::InspectorWorkerResource):
  • inspector/front-end/Checkbox.js: Added. (WebInspector.Checkbox.callbackWrapper): (WebInspector.Checkbox): (WebInspector.Checkbox.prototype.checked):
  • inspector/front-end/InjectedFakeWorker.js: (InjectedFakeWorker.FakeWorker): (InjectedFakeWorker.FakeWorker.prototype.terminate): (InjectedFakeWorker.FakeWorker.prototype._handleException): (InjectedFakeWorker.FakeWorker.prototype._importScripts): (InjectedFakeWorker.FakeWorker.prototype._loadScript): (InjectedFakeWorker.FakeWorker.prototype._expandURLAndCheckOrigin):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel): (WebInspector.ScriptsPanel.prototype.debuggerWasEnabled): (WebInspector.ScriptsPanel.prototype.debuggerWasDisabled): (WebInspector.ScriptsPanel.prototype.reset):
  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylePropertiesSection.showInheritedToggleFunction): (WebInspector.StylePropertiesSection):
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/WorkersSidebarPane.js: Added. (WebInspector.WorkersSidebarPane): (WebInspector.WorkersSidebarPane.prototype.addWorker): (WebInspector.WorkersSidebarPane.prototype.removeWorker): (WebInspector.WorkersSidebarPane.prototype.setInstrumentation): (WebInspector.WorkersSidebarPane.prototype.reset): (WebInspector.WorkersSidebarPane.prototype._onTriggerInstrument): (WebInspector.Worker): (WebInspector.didCreateWorker): (WebInspector.willDestroyWorker):
  • inspector/front-end/inspector.css:
  • inspector/front-end/inspector.html:
  • workers/AbstractWorker.cpp: (WebCore::AbstractWorker::AbstractWorker): (WebCore::AbstractWorker::~AbstractWorker): (WebCore::AbstractWorker::onDestroyWorker): (WebCore::AbstractWorker::contextDestroyed):
  • workers/AbstractWorker.h: (WebCore::AbstractWorker::id):
  • workers/SharedWorker.cpp: (WebCore::SharedWorker::SharedWorker):
  • workers/Worker.cpp: (WebCore::Worker::Worker):

WebKit/chromium:

  • src/js/InspectorControllerImpl.js: (devtools.InspectorBackendImpl):
1:02 AM Changeset in webkit [55770] by zoltan@webkit.org
  • 3 edits in trunk/JavaScriptCore

2010-03-10 Kevin Ollivier <​kevino@theolliviers.com>

Reviewed by Eric Seidel.

Make global new/delete operators configurable for all ports and disable it
for the wx port for now.

  • wtf/FastMalloc.h:
  • wtf/Platform.h:
12:41 AM Changeset in webkit [55769] by tony@chromium.org
  • 1 edit
    1 add in trunk/WebKit/chromium

2010-03-10 Tony Chang <​tony@chromium.org>

Reviewed by Darin Fisher.

​https://bugs.webkit.org/show_bug.cgi?id=35956
Create a grd file upstream for resources that chromium uses
that are pulled directly from the upstream repository.

  • WebKit.grd: Added.
12:06 AM Changeset in webkit [55768] by Philippe Normand
  • 2 edits in trunk/WebKit/gtk

2010-03-09 Philippe Normand <​pnormand@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GTK] testkeyevents doesn't stop if input event injection fails
​https://bugs.webkit.org/show_bug.cgi?id=35922

  • tests/testkeyevents.c: (load_status_cb): Added a safeguard to exit from the test if the input event injection failed.

Mar 9, 2010:

10:32 PM Changeset in webkit [55767] by rolandsteiner@chromium.org
  • 2 edits in trunk/WebCore

Unreviewed build fix. Fix variable name change that somehow didn't
make it into the patch.

10:23 PM Changeset in webkit [55766] by rolandsteiner@chromium.org
  • 12 edits
    3 adds in trunk

Bug 28293 - [Chromium] event.datatransfer.getdata("text/uri-list") is treated the same as getdata("URL")
​https://bugs.webkit.org/show_bug.cgi?id=28293

Reviewed by David Levin.

WebCore:

Change ChromiumDataObject such that it treats types "URL" and "text/uri-list"
correctly for event.dataTransfer.getData/setData. Currently both are treated
as synonyms, but for "URL", getData is supposed to only return the first valid URL
contained within the data for "text/uri-list" (see HTML5 spec).

Tests: editing/pasteboard/dataTransfer-setData-getData.html

  • platform/chromium/ChromiumDataObject.cpp:

(WebCore::ChromiumDataObject::clear):
(WebCore::ChromiumDataObject::clearAllExceptFiles):
(WebCore::ChromiumDataObject::hasData):
(WebCore::ChromiumDataObject::ChromiumDataObject):

  • platform/chromium/ChromiumDataObject.h:

(WebCore::ChromiumDataObject::clearURL):
(WebCore::ChromiumDataObject::hasValidURL):
(WebCore::ChromiumDataObject::getURL):
(WebCore::ChromiumDataObject::setURL):

  • platform/chromium/ClipboardChromium.cpp:

(WebCore::):
(WebCore::clipboardTypeFromMIMEType):
(WebCore::ClipboardChromium::clearData):
(WebCore::ClipboardChromium::getData):
(WebCore::ClipboardChromium::setData):
(WebCore::ClipboardChromium::types):

  • platform/chromium/DragDataChromium.cpp:

(WebCore::DragData::asURL):
(WebCore::DragData::canSmartReplace):

WebKit/chromium:

Change ChromiumDataObject such that it treats types "URL" and "text/uri-list"
correctly for event.dataTransfer.getData/setData. Currently both are treated
as synonyms, but for "URL", getData is supposed to only return the first valid URL
contained within the data for "text/uri-list" (see HTML5 spec).

Tests: editing/pasteboard/dataTransfer-setData-getData.html

  • src/WebDragData.cpp:

(WebKit::WebDragData::url):
(WebKit::WebDragData::setURL):

LayoutTests:

Change ChromiumDataObject such that it treats types "URL" and "text/uri-list"
correctly for event.dataTransfer.getData/setData. Currently both are treated
as synonyms, but for "URL", getData is supposed to only return the first valid URL
contained within the data for "text/uri-list" (see HTML5 spec).

Layout test checks various combinations in a drag-n-drop operation.
Skipping on all other platforms since they don't implement this correctly, either.

  • editing/pasteboard/dataTransfer-setData-getData.html: Added.
  • editing/pasteboard/script-tests/dataTransfer-setData-getData.js: Added.

(dragTarget):
(moveMouseToCenterOfElement):
(dragOntoDragTarget):
(doDrop):
(test):
(runTest):

  • platform/mac/Skipped:
  • platform/gtk/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:
10:17 PM Changeset in webkit [55765] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

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

Reviewed by Timothy Hatcher.

Web Inspector: REGRESSION: Tall image resources are cropped over the bottom.

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

  • inspector/front-end/inspector.css:
10:13 PM Changeset in webkit [55764] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

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

Reviewed by Timothy Hatcher.

Web Inspector: REGRESSION: Pressing up/down arrow key to change numeric
value in CSS property takes focus away.

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

  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylePropertyTreeElement.prototype): (WebInspector.StylePropertyTreeElement.prototype.):
9:46 PM Changeset in webkit [55763] by ukai@chromium.org
  • 3 edits in trunk/WebKitTools

2010-03-09 Fumitoshi Ukai <​ukai@chromium.org>

Unreviewed.

Obvious fix for --cgi-paths of pywebsocket.

  • Scripts/run-webkit-websocketserver:
  • Scripts/webkitpy/layout_tests/port/websocket_server.py:
9:32 PM Changeset in webkit [55762] by tony@chromium.org
  • 10 edits
    4 adds in trunk

2010-03-09 Tony Chang <​tony@chromium.org>

Reviewed by Adam Barth.

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

Fix an editing bug where replacing a selection would result in the
new text ending up inside nodes that were not visibly included in the
selection. Instead, move our destination position out of nodes that
were not visibly included.

Two new tests to verify the new behavior. Because we're now inserting
outside of some formatting nodes, some span tags are no longer necessary
for undoing formatting caused by these formatting nodes.

  • editing/deleting/backspace-avoid-preceding-style-expected.txt: Added.
  • editing/deleting/backspace-avoid-preceding-style.html: Added.
  • editing/inserting/replace-at-visible-boundary-expected.txt: Added.
  • editing/inserting/replace-at-visible-boundary.html: Added.
  • platform/mac/editing/deleting/delete-3857753-fix-expected.txt:
  • platform/mac/editing/inserting/insert-div-026-expected.txt:
  • platform/mac/editing/pasteboard/paste-text-at-tabspan-001-expected.txt:
  • platform/mac/editing/pasteboard/paste-text-at-tabspan-002-expected.txt:
  • platform/mac/editing/style/font-family-with-space-expected.txt:
  • platform/mac/editing/style/smoosh-styles-001-expected.txt:
  • platform/mac/editing/style/style-boundary-005-expected.txt:

2010-03-09 Tony Chang <​tony@chromium.org>

Reviewed by Adam Barth.

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

Fix an editing bug where replacing a selection would result in the
new text ending up inside nodes that were not visibly included in the
selection. Instead, move our destination position out of nodes that
were not visibly included.

Tests: editing/deleting/backspace-avoid-preceding-style.html

editing/inserting/replace-at-visible-boundary.html

  • editing/ReplaceSelectionCommand.cpp: (WebCore::positionAvoidingPrecedingNodes): (WebCore::ReplaceSelectionCommand::doApply):
8:53 PM Changeset in webkit [55761] by beidson@apple.com
  • 9 edits in trunk

REGRESSION: WebInspector docking busted on Windows
<rdar://problem/7728433> and ​https://bugs.webkit.org/show_bug.cgi?id=35953

Reviewed by Tim Hatcher.

WebCore:

First off, a bit of settings-key related cleanup. If they're shared over multiple files, these
things should be properly declared Strings, not random loose char*'s.

Along with that change, we move the "inspectorStartsAttachedSettingName" from WebKit down to the
InspectorController in WebCore.

Finally, when the controller is ready to show the WebInspector window, it can use this newly
exposed settings key to call "setWindowVisible" directly instead of relying on "showWindow" to do
it indirectly.

  • WebCore.base.exp:
  • inspector/InspectorBackend.cpp:

(WebCore::InspectorBackend::saveFrontendSettings):

  • inspector/InspectorController.cpp:

(WebCore::InspectorController::frontendSettingsSettingName):
(WebCore::InspectorController::inspectorStartsAttachedSettingName):
(WebCore::InspectorController::scriptObjectReady): Using the new inspectorStartsAttachedSettingName

key and some "can I attach this?" logic, call setWindowVisible directly to display the inspector
window.

(WebCore::InspectorController::populateScriptObjects):

  • inspector/InspectorController.h:

WebKit/mac:

  • WebCoreSupport/WebInspectorClient.mm:

(-[WebInspectorWindowController showWindow:]): Use the InspectorController:: copy of the should attach settings key.
(-[WebInspectorWindowController attach]): Ditto.
(-[WebInspectorWindowController detach]): Ditto.

WebKit/win:

  • WebCoreSupport/WebInspectorClient.cpp:

(WebInspectorClient::attachWindow): Use the InspectorController:: copy of the should attach settings key.
(WebInspectorClient::detachWindow): Ditto.
(WebInspectorClient::showWindowWithoutNotifications): Ditto.

8:05 PM Changeset in webkit [55760] by ap@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Mark Rowe.

​https://bugs.webkit.org/show_bug.cgi?id=35951
<rdar://problem/7327060> Frequent crashes in Dashcode at JSC::Bindings::ObjcInstance::~ObjcInstance + 80

  • bridge/objc/objc_instance.mm: (createInstanceWrapperCache): Fix Tiger code path, too.
6:45 PM Changeset in webkit [55759] by kov@webkit.org
  • 1 copy in releases/WebKitGTK/webkit-1.1.23

Tagging 1.1.23.

5:44 PM Changeset in webkit [55758] by jam@chromium.org
  • 4 edits in trunk/WebKit/chromium

Reviewed by Darin Fisher.

Need to send mouse events to plugin when it has mouse capture
​https://bugs.webkit.org/show_bug.cgi?id=35900

  • public/WebInputEvent.h:

(WebKit::WebInputEvent::isMouseEventType):

  • src/WebViewImpl.cpp:

(WebKit::WebViewImpl::WebViewImpl):
(WebKit::WebViewImpl::mouseDown):
(WebKit::WebViewImpl::mouseUp):
(WebKit::WebViewImpl::handleInputEvent):

  • src/WebViewImpl.h:
5:29 PM Changeset in webkit [55757] by ap@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Oliver Hunt.

​https://bugs.webkit.org/show_bug.cgi?id=35951
<rdar://problem/7327060> Frequent crashes in Dashcode at JSC::Bindings::ObjcInstance::~ObjcInstance + 80

  • bridge/objc/objc_instance.mm: (createInstanceWrapperCache): It's not correct to use object personality for keys, because the key can be a mutable object, so its hash can change over its lifetime.
4:49 PM Changeset in webkit [55756] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt] http/tests/plugins/third-party-cookie-accept-policy.html was introduced in r55738,
but layoutTestController.setAlwaysAcceptCookies() is missing.

  • platform/qt/Skipped: http/tests/plugins/third-party-cookie-accept-policy.html skipped.
4:45 PM Changeset in webkit [55755] by mrowe@apple.com
  • 4 edits in trunk

Versioning.

4:43 PM Changeset in webkit [55754] by mrowe@apple.com
  • 1 copy in tags/Safari-533.2

New tag.

4:17 PM Changeset in webkit [55753] by ojan@chromium.org
  • 3 edits in trunk/WebCore

2010-03-09 Ojan Vafai <​ojan@chromium.org>

Reviewed by Darin Adler.

Fix review comment accidentally overlooked in bug 35713
​https://bugs.webkit.org/show_bug.cgi?id=35947

  • editing/DeleteSelectionCommand.cpp: (WebCore::DeleteSelectionCommand::setStartingSelectionOnSmartDelete):
  • editing/DeleteSelectionCommand.h:
3:41 PM Changeset in webkit [55752] by ojan@chromium.org
  • 5 edits
    4 adds in trunk

2010-03-03 Ojan Vafai <​ojan@chromium.org>

Reviewed by Adam Barth.

undo after smartdelete should select the deleted space
​https://bugs.webkit.org/show_bug.cgi?id=35713

  • editing/undo/undo-smart-delete-reversed-selection-expected.txt: Added.
  • editing/undo/undo-smart-delete-reversed-selection.html: Added.
  • editing/undo/undo-smart-delete-word-expected.txt: Added.
  • editing/undo/undo-smart-delete-word.html: Added.

2010-03-03 Ojan Vafai <​ojan@chromium.org>

Reviewed by Adam Barth.

undo after smartdelete should select the deleted space
​https://bugs.webkit.org/show_bug.cgi?id=35713

TextEdit behavior is to select the deleted space after a smartdelete.

Tests: editing/undo/undo-smart-delete-reversed-selection.html

editing/undo/undo-smart-delete-word.html

  • editing/DeleteSelectionCommand.cpp: (WebCore::DeleteSelectionCommand::setStartingSelectionOnSmartDelete): (WebCore::DeleteSelectionCommand::initializePositionData):
  • editing/DeleteSelectionCommand.h:
  • editing/VisibleSelection.cpp: (WebCore::VisibleSelection::setWithoutValidation): This assert looks bogus to me. undo-smart-delete-reversed-selection.html hits it but not as a result of the other changes in this patch. The granularity when deleting after making a wordgranularity selection is wordgranularity, not charactergranularity.
3:32 PM Changeset in webkit [55751] by barraclough@apple.com
  • 8 edits in trunk/JavaScriptCore

This caused a performance regression, by breaking the code
generator's logic to calculate the skip level for resolving
variables (traced by rdar:7683350) Reverting for now.

Reviewed by NOBODY (reverting r54510).

  • parser/Grammar.y:
  • parser/NodeConstructors.h:

(JSC::ContinueNode::ContinueNode):
(JSC::BreakNode::BreakNode):
(JSC::ForInNode::ForInNode):

  • runtime/CommonIdentifiers.cpp:

(JSC::CommonIdentifiers::CommonIdentifiers):

  • runtime/CommonIdentifiers.h:
  • runtime/FunctionPrototype.cpp:

(JSC::FunctionPrototype::FunctionPrototype):

  • runtime/Identifier.cpp:

(JSC::Identifier::add):

  • runtime/PropertyNameArray.cpp:

(JSC::PropertyNameArray::add):

3:07 PM Changeset in webkit [55750] by ggaren@apple.com
  • 5 edits in trunk

JavaScriptCore: Changed FastMalloc statistics reporting to be a bit clearer. We now
report:

  • Reserved VM Bytes: the VM that has been mapped into the process.
  • Committed VM Bytes: the subset of Reserved VM Bytes actually in use.
  • Free List Bytes: the subset of Committed VM Bytes in a free list.

Reviewed by Darin Adler.

  • wtf/FastMalloc.cpp:

(WTF::fastMallocStatistics):
(WTF::TCMallocStats::fastMallocStatistics): Updated to report the statistics
above. Standardized use of "ifdef WTF_CHANGES". Added a SpinLockHolder
around all statistics gathering, since it reads from the page heap.

  • wtf/FastMalloc.h: Updated to report the statistics above.

WebKit/mac: Updated for FastMalloc reporting changes.

Reviewed by Darin Adler.

  • Misc/WebCoreStatistics.mm:

(+[WebCoreStatistics memoryStatistics]):

2:40 PM Changeset in webkit [55749] by jwieczorek@webkit.org
  • 2 edits in trunk/WebKitTools

2010-03-09 Jakub Wieczorek <​jwieczorek@webkit.org>

Unreviewed.

Adding myself to committers.py.

  • Scripts/webkitpy/committers.py:
2:37 PM Changeset in webkit [55748] by darin@chromium.org
  • 3 edits
    2 adds in trunk

2010-03-09 Anton Muhin <​antonm@chromium.org>

Reviewed by Darin Fisher.

Accept and bailout NULL widgets in ChromiumBridge
​https://bugs.webkit.org/show_bug.cgi?id=35796

  • src/ChromiumBridge.cpp: (WebCore::toChromeClientImpl):
2:24 PM Changeset in webkit [55747] by sfalken@apple.com
  • 2 edits in trunk/WebCore

Rubber stamped by Adam Roben.

Load debug version of graphics library in Windows Debug_All builds.

  • platform/graphics/win/WKCACFLayerRenderer.cpp:

(WebCore::WKCACFLayerRenderer::acceleratedCompositingAvailable):

2:13 PM Google Summer of Code 2010 edited by jorlow@chromium.org
(diff)
2:02 PM CommitterTips edited by jwieczorek@webkit.org
Fix a few typos. (diff)
2:00 PM Changeset in webkit [55746] by sullivan@apple.com
  • 4 edits in trunk/WebKit

Fixed localized string key collision. update-webkit-localized-strings now
runs without errors.

Reviewed by Adam Roben.

WebKit:

  • English.lproj/Localizable.strings:

Regenerated.

WebKit/win:

  • WebCoreLocalizedStrings.cpp:

(WebCore::AXMenuListPopupActionVerb):
Used LPCTSTR_UI_STRING_KEY for the 2nd use of "press".

1:49 PM Changeset in webkit [55745] by sullivan@apple.com
  • 2 edits in trunk/WebKit
  • StringsNotToBeLocalized.txt:

Brought this file up to date. update-webkit-localizable-strings still lists
one key collision, but that's a separate issue.

1:29 PM Changeset in webkit [55744] by kov@webkit.org
  • 2 edits in trunk/LayoutTests

2010-03-09 Gustavo Noronha Silva <Gustavo Noronha Silva>

[GTK] Fails http/tests/plugins/third-party-cookie-accept-policy.html
​https://bugs.webkit.org/show_bug.cgi?id=35942

Unreviewed. Skip test that fails, while we look for the
problem. This is likely a bug in soup.

  • platform/gtk/Skipped:
1:25 PM Changeset in webkit [55743] by kov@webkit.org
  • 4 edits in trunk

Unreviewed. Versioning, and documentation control files update for 1.1.23.

1:25 PM Changeset in webkit [55742] by beidson@apple.com
  • 4 edits in trunk/LayoutTests

Fix my layouttest failures by resetting cookies before and after these new tests.
​https://bugs.webkit.org/show_bug.cgi?id=26391

Reviewed by Alexey Proskuryakov.

  • http/tests/cookies/third-party-cookie-relaxing.html:
  • http/tests/plugins/resources/third-party-cookie-accept-policy-iframe.html:
  • http/tests/plugins/third-party-cookie-accept-policy.html:
1:23 PM Changeset in webkit [55741] by dpranke@chromium.org
  • 2 edits in trunk/WebKitTools

2010-03-09 Dirk Pranke <​dpranke@chromium.org>

Reviewed by Nate Chapin.

Fix --clobber-old-results in new-run-webkit-tests.

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

  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
12:58 PM Changeset in webkit [55740] by Adam Roben
  • 2 edits in trunk/WebKitSite

Add a link to the Contributors Meeting registration form in the sidebar

Rubber-stamped in advance by Adele Peterson.

  • nav.inc: Added a link underneath "Trac".
11:56 AM Changeset in webkit [55739] by adele@apple.com
  • 2 edits in trunk/WebKitTools

Add the ability to dispatch scroll wheel events in DRT. This was
necessary in order to write a test for
​https://bugs.webkit.org/show_bug.cgi?id=34700.

Patch by Andy Estes <​aestes@apple.com> on 2010-03-09
Reviewed by Adele Peterson.

  • DumpRenderTree/mac/EventSendingController.mm: Add support for two

new methods to EventSender: mouseScrollBy(x, y) and
continuousMouseScrollBy(x, y). The API to generate scroll events on
the mac was added in 10.5, so these methods are NOOPs on Tiger.
(+[EventSendingController isSelectorExcludedFromWebScript:]):
Regiester mouseScrollByX:andY: and continuousMouseScrollByX:andY:
(+[EventSendingController webScriptNameForSelector:]): Map JavaScript
method names to ObjC selectors.
(-[EventSendingController mouseScrollByX:andY:continuously:]): Generate
a scroll wheel event using CGEventCreateScrollWheelEvent() and dispatch
it to WebKit.
(-[EventSendingController continuousMouseScrollByX:andY:]): Generate a
continuous scrolling event by x and y pixels.
(-[EventSendingController mouseScrollByX:andY:]): Generate a notchy
scrolling event by x and y lines.

11:52 AM Changeset in webkit [55738] by beidson@apple.com
  • 4 edits
    3 adds in trunk

Plug-ins don't always respect the cookie accept policy.
<rdar://problem/7338359> and ​https://bugs.webkit.org/show_bug.cgi?id=26391

Reviewed by Darin Adler.

WebCore:

The problem is that the various plug-in implementations call into a ResourceLoader
directly instead of filtering the request through FrameLoader. This bypassed the step
of adding extra fields to the requests, such as the firstPartyForCookies URL.

Since plug-in code is currently so strewn about and very platform specific, I
think reworking it needs to be a task for domain experts. I don't know the implications
of adding *all* the extra fields to plug-in requests, for example.

There's no harm in this targeted fix for the hole in our cookie accept policy until
plug-ins can more fundamentally change.

Test: http/tests/plugins/third-party-cookie-accept-policy.html

  • loader/ResourceLoader.cpp:

(WebCore::ResourceLoader::load): Don't load a resource without first giving the request

a firstPartyForCookies URL.

LayoutTests:

  • http/tests/cookies/resources/cookie-utility.php:
  • http/tests/plugins/resources/third-party-cookie-accept-policy-iframe.html: Added.
  • http/tests/plugins/third-party-cookie-accept-policy.html: Added.
11:32 AM Changeset in webkit [55737] by adele@apple.com
  • 7 edits
    2 adds in trunk

WebCore: Check for null renderer in scrollNode().
​https://bugs.webkit.org/show_bug.cgi?id=34700

Patch by Andy Estes <​aestes@apple.com> on 2010-03-09
Reviewed by Adele Peterson.

Test: fast/events/remove-child-onscroll.html

  • page/EventHandler.cpp:

(WebCore::scrollNode): Return early if node->renderer() == 0

LayoutTests: Create a test for ​https://bugs.webkit.org/show_bug.cgi?id=34700. This
test required implementing scroll wheel event dispatch in DRT and
currently only works on OS X 10.5+. Skip this test on other platforms
for the time being.

Patch by Andy Estes <​aestes@apple.com> on 2010-03-09
Reviewed by Adele Peterson.

  • fast/events/remove-child-onscroll-expected.txt: Added.
  • fast/events/remove-child-onscroll.html: Added.
  • platform/gtk/Skipped: Skip remove-child-onscroll.html
  • platform/mac-tiger/Skipped: Skip remove-child-onscroll.html
  • platform/qt/Skipped: Skip remove-child-onscroll.html
  • platform/win/Skipped: Skip remove-child-onscroll.html
11:20 AM QtWebKitContrib edited by jocelyn.turcotte@nokia.com
(diff)
11:18 AM QtWebKitContrib edited by jocelyn.turcotte@nokia.com
(diff)
11:12 AM Changeset in webkit [55736] by kov@webkit.org
  • 4 edits in trunk

Unreviewed distcheck fix

11:11 AM Changeset in webkit [55735] by eric@webkit.org
  • 2 edits in trunk/LayoutTests

2010-03-09 Garret Kelly <​gdk@chromium.org>

Reviewed by Darin Adler.

Fix send-oncancel-event. It uses js-test-post, not
js-test-post-function.
​https://bugs.webkit.org/show_bug.cgi?id=35880

  • fast/events/touch/script-tests/send-oncancel-event.js: (touchcancelHandler):
10:45 AM Changeset in webkit [55734] by kov@webkit.org
  • 2 edits in trunk/WebCore

2010-03-09 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Unreviewed build fix (only exposed by make distcheck). The

SharedWorkers files are not built, they need to be distributed.

  • GNUmakefile.am:
10:43 AM Changeset in webkit [55733] by kov@webkit.org
  • 3 edits in trunk/WebCore

2010-03-09 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Reviewed by Dirk Schulze.

[GStreamer] Buffering logic is not correct, and does not work very well
​https://bugs.webkit.org/show_bug.cgi?id=35706

Fix buffering to match GStreamer expectancies regarding the
pipeline state, so that videos which really need buffering to play
correctly also work, while maintaining the convenience of
on-disk-buffering. This required a bit of shuffling of state
change handling.

No behaviour change expected.

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::MediaPlayerPrivate::MediaPlayerPrivate): (WebCore::MediaPlayerPrivate::processBufferingStats): (WebCore::MediaPlayerPrivate::fillTimerFired): (WebCore::MediaPlayerPrivate::updateStates):
  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.h:
10:35 AM Changeset in webkit [55732] by kov@webkit.org
  • 2 edits in trunk/WebCore

2010-03-09 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Reviewed by Dirk Schulze.

[GStreamer] Buffering logic is not correct, and does not work very well
​https://bugs.webkit.org/show_bug.cgi?id=35706

Do not call pause(), but set the GStreamer state directly. This is
just a GStreamer implementation detail, and this will avoid having
side effects in case we change the pause implementation in the
future.

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::MediaPlayerPrivate::load):
10:25 AM Changeset in webkit [55731] by Simon Fraser
  • 2 edits in trunk/LayoutTests

2010-03-09 Simon Fraser <Simon Fraser>

Skip media/video-source-error.html on SnowLeopard, beacuse it fails sporadically.
This is covered by ​https://bugs.webkit.org/show_bug.cgi?id=32270.

  • platform/mac-snowleopard/Skipped:
10:11 AM Changeset in webkit [55730] by Chris Fleizach
  • 2 edits in trunk/WebKitTools

DRT build fix for Tiger. No review.

AX: hit testing a list box doesn't work anymore
​https://bugs.webkit.org/show_bug.cgi?id=35893

  • DumpRenderTree/mac/AccessibilityControllerMac.mm:

(AccessibilityController::elementAtPoint):

9:42 AM Changeset in webkit [55729] by Chris Fleizach
  • 9 edits
    2 adds in trunk

AX: hit testing a list box doesn't work anymore
​https://bugs.webkit.org/show_bug.cgi?id=35893

Reviewed by Darin Adler.

WebCore:

Since <option> elements don't have renderers, their hit testing needs
to be handled with a special case.

Test: platform/mac/accessibility/listbox-hit-test.html

  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::ariaIsHidden):
(WebCore::AccessibilityRenderObject::doAccessibilityHitTest):

WebKitTools:

  • DumpRenderTree/AccessibilityController.cpp:

(getElementAtPointCallback):
(AccessibilityController::getJSClass):

  • DumpRenderTree/AccessibilityController.h:
  • DumpRenderTree/gtk/AccessibilityControllerGtk.cpp:

(AccessibilityController::elementAtPoint):

  • DumpRenderTree/mac/AccessibilityControllerMac.mm:

(AccessibilityController::elementAtPoint):

  • DumpRenderTree/win/AccessibilityControllerWin.cpp:

(AccessibilityController::elementAtPoint):

LayoutTests:

  • platform/mac/accessibility/listbox-hit-test-expected.txt: Added.
  • platform/mac/accessibility/listbox-hit-test.html: Added.
9:37 AM Changeset in webkit [55728] by pfeldman@chromium.org
  • 1 edit in trunk/WebCore/ChangeLog

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

r55727 was landed with wrong commit messages. Should have been:

Reviewed by Timothy Hatcher.

Web Inspector: Refactor Audits panel presentation layer.
This change removes unnecessary complexity:

  • Audit scores were not used
  • Audit rule parameters are passed as rule constructor arguments
  • View management aligned with the rest of the front-end
  • Single TreeOutline is used for category results (no need to create sections for those)
  • Rules code beautified and simplified where possible
  • Some UI improvements applied (see attached screenshot)

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

9:34 AM Changeset in webkit [55727] by pfeldman@chromium.org
  • 8 edits in trunk/WebCore

2010-03-09 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Pavel Feldman.


When enable resource tracking state changes use the same method as
location.reload for reloading inspected page.

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

  • inspector/InspectorBackend.cpp: (WebCore::InspectorBackend::reloadPage):
  • inspector/InspectorController.cpp: (WebCore::InspectorController::enableResourceTracking):
9:22 AM Changeset in webkit [55726] by yurys@chromium.org
  • 7 edits in trunk

2010-03-09 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Pavel Feldman.


When enable resource tracking state changes use the same method as
location.reload for reloading inspected page.

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

  • inspector/InspectorBackend.cpp: (WebCore::InspectorBackend::reloadPage):
  • inspector/InspectorController.cpp: (WebCore::InspectorController::enableResourceTracking):

2010-03-09 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Pavel Feldman.

Use clean global object for devtools utility context instead of
trying to simulate inspected context.


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

  • src/DebuggerAgentImpl.cpp:
  • src/DebuggerAgentImpl.h:
  • src/WebDevToolsAgentImpl.cpp: (WebKit::WebDevToolsAgentImpl::resetInspectorFrontendProxy):
8:24 AM Changeset in webkit [55725] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

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

Reviewed by Timothy Hatcher.

Web Inspector: highlight text node containers while searching for node.

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

  • inspector/InspectorController.cpp: (WebCore::InspectorController::mouseDidMoveOverElement):
7:51 AM QtWebKitRelease20 edited by jocelyn.turcotte@nokia.com
(diff)
7:47 AM Changeset in webkit [55724] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt] editing/execCommand/move-selection-back-line.html fails in DRT.
​https://bugs.webkit.org/show_bug.cgi?id=35921

  • platform/qt/Skipped: editing/execCommand/move-selection-back-line.html skipped until fix.
7:46 AM QtWebKitRelease20Deployment created by jocelyn.turcotte@nokia.com
7:39 AM QtWebKitBugs edited by Simon Hausmann
(diff)
6:46 AM Changeset in webkit [55723] by kov@webkit.org
  • 1 edit
    1 add in trunk/WebKit/gtk/po

2010-03-09 Peteris Krisjanis <​pecisk@gmail.com>

Reviewed by Gustavo Noronha.

Latvian translation.

  • lv.po: Added.
6:42 AM Changeset in webkit [55722] by kov@webkit.org
  • 2 edits in trunk/WebKit/gtk/po

2010-03-09 Duy Nguyen <​pclouds@gmail.com>

Reviewed by Gustavo Noronha.

Vietnamese translation update.

  • vi.po:
6:36 AM Changeset in webkit [55721] by kov@webkit.org
  • 2 edits in trunk/WebKit/gtk/po

2010-03-09 Rimas Kudelis <​rq@akl.lt>

Reviewed by Gustavo Noronha.

Lithuanian translation update.

  • lt.po:
3:23 AM Changeset in webkit [55720] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

Unreviewed trivial fix.

[Qt] perl websocket test(s) time out
​https://bugs.webkit.org/show_bug.cgi?id=34879
It was fixed by r55719, so we can unskip websocket/tests/cookies/httponly-cookie.pl.

storage/open-database-creation-callback-isolated-world.html removed
from Skipped list, because it was removed from tree by r55635.

  • platform/qt/Skipped:
    • websocket/tests/cookies/httponly-cookie.pl unskipped.
    • storage/open-database-creation-callback-isolated-world.html unskipped.
3:20 AM QtWebKitBugs edited by jocelyn.turcotte@nokia.com
Add the all triaged bugs list (diff)
2:40 AM Changeset in webkit [55719] by ukai@chromium.org
  • 11 edits
    3 copies
    3 adds
    1 delete in trunk

2010-03-03 Fumitoshi Ukai <​ukai@chromium.org>

Reviewed by Alexey Proskuryakov.

pywebsocket should support html and cgi in the same directory.
​https://bugs.webkit.org/show_bug.cgi?id=34879

  • websocket/tests/cookies: Removed.
  • websocket/tests/cookies/echo-cookie_wsh.py: Removed.
  • websocket/tests/cookies/httponly-cookie-expected.txt: Removed.
  • websocket/tests/cookies/httponly-cookie.pl: Removed.
  • websocket/tests/echo-cookie_wsh.py: Copied from LayoutTests/websocket/tests/cookies/echo-cookie_wsh.py.
  • websocket/tests/httponly-cookie-expected.txt: Copied from LayoutTests/websocket/tests/cookies/httponly-cookie-expected.txt.
  • websocket/tests/httponly-cookie.pl: Copied from LayoutTests/websocket/tests/cookies/httponly-cookie.pl.

2010-03-03 Fumitoshi Ukai <​ukai@chromium.org>

Reviewed by Alexey Proskuryakov.

pywebsocket should support html and cgi in the same directory.
​https://bugs.webkit.org/show_bug.cgi?id=34879

Import pywebsocket 0.4.9.2
Specify --server-host 127.0.0.1, so that it only binds listening socket
to 127.0.0.1 to prevent access from non-localhost.
Change --cgi-paths from /websocket/tests/cookies to /websocket/tests,
because pywebsocket 0.4.9.2 supports html and cgi in the same directory
and only executable (httponly-cookies.pl) will be handled as cgi
script.

  • Scripts/run-webkit-tests:
  • Scripts/run-webkit-websocketserver:
  • Scripts/webkitpy/layout_tests/port/websocket_server.py:
  • Scripts/webkitpy/thirdparty/pywebsocket/README.webkit:
  • Scripts/webkitpy/thirdparty/pywebsocket/example/echo_client.py:
  • Scripts/webkitpy/thirdparty/pywebsocket/example/handler_map.txt: Added.
  • Scripts/webkitpy/thirdparty/pywebsocket/mod_pywebsocket/standalone.py:
  • Scripts/webkitpy/thirdparty/pywebsocket/mod_pywebsocket/util.py:
  • Scripts/webkitpy/thirdparty/pywebsocket/setup.py:
  • Scripts/webkitpy/thirdparty/pywebsocket/test/test_util.py:
  • Scripts/webkitpy/thirdparty/pywebsocket/test/testdata/README: Added.
  • Scripts/webkitpy/thirdparty/pywebsocket/test/testdata/hello.pl: Added.
1:36 AM Changeset in webkit [55718] by loki@webkit.org
  • 4 edits in trunk/JavaScriptCore

Rubber-stamped by Maciej Stachowiak.

Buildfix for ARM after r55684. Add branch8 and branchTest8 functions.
​https://bugs.webkit.org/show_bug.cgi?id=35892

  • assembler/ARMAssembler.cpp:

(JSC::ARMAssembler::dataTransfer32):

  • assembler/ARMAssembler.h:

(JSC::ARMAssembler::):

  • assembler/MacroAssemblerARM.h:

(JSC::MacroAssemblerARM::load8):
(JSC::MacroAssemblerARM::branch8):
(JSC::MacroAssemblerARM::branchTest8):

1:01 AM Changeset in webkit [55717] by Philippe Normand
  • 3 edits in trunk/WebCore

2010-03-09 Philippe Normand <​pnormand@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GStreamer] player code cleanups
​https://bugs.webkit.org/show_bug.cgi?id=35868

Cleaned up the private instance variables of the player.

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::MediaPlayerPrivate::MediaPlayerPrivate): (WebCore::MediaPlayerPrivate::fillTimerFired): (WebCore::MediaPlayerPrivate::mediaLocationChanged):
  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.h:
1:01 AM Changeset in webkit [55716] by Philippe Normand
  • 7 edits
    2 copies
    2 moves in trunk/WebCore

2010-03-09 Philippe Normand <​pnormand@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GStreamer] player code cleanups
​https://bugs.webkit.org/show_bug.cgi?id=35868

Splitted GOwnPtrGtk.{cpp,h} to GOwnPtr{Soup,GStreamer}.{cpp,h}.

  • GNUmakefile.am:
  • platform/graphics/gstreamer/GOwnPtrGStreamer.cpp: Added. (WTF::GstElement):
  • platform/graphics/gstreamer/GOwnPtrGStreamer.h: Added.
  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:
  • platform/gtk/GOwnPtrGtk.cpp: Removed.
  • platform/gtk/GOwnPtrGtk.h: Removed.
  • platform/network/soup/CookieJarSoup.cpp:
  • platform/network/soup/DNSSoup.cpp:
  • platform/network/soup/GOwnPtrSoup.cpp: Added. (WTF::SoupURI):
  • platform/network/soup/GOwnPtrSoup.h: Added.
  • platform/network/soup/ResourceHandleSoup.cpp:
  • platform/network/soup/ResourceRequestSoup.cpp:
12:30 AM Changeset in webkit [55715] by Chris Jerdonek
  • 3 edits in trunk/WebKitTools

Simplified check-webkit-style's argument parsing code by removing
support for the vestigial "extra flag values" parameter.

Reviewed by Shinichiro Hamaji.

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

The "extra flag values" parameter was needed before WebKit
forked check-webkit-style from Google. It was used to pass
through the option parser those command-line flags that WebKit
required but that Google's parser did not support (the --git-commit
flag in particular).

We can remove the parameter now because it is no longer

needed and unnecessarily clutters the argument-parsing code.

  • Scripts/webkitpy/style/optparser.py:
    • Removed the extra_flag_values parameter from the CommandOptionValues class's constructor.
    • Removed the extra_flags parameter from the ArgumentParser class's parse() method.
  • Scripts/webkitpy/style/optparser_unittest.py:
    • Removed from the unit tests all references to the extra_flag_values variable.
12:23 AM Changeset in webkit [55714] by hamaji@chromium.org
  • 3 edits in trunk/WebCore

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

Reviewed by Eric Seidel.

[Chromium] Should remove paddings of Win/Linux chromium's buttons
​https://bugs.webkit.org/show_bug.cgi?id=35629

No new tests, but we may need to rebaseline bunch of expectation
images in chromium's tree.

  • rendering/RenderThemeChromiumSkia.cpp:
  • rendering/RenderThemeChromiumSkia.h:
Note: See TracTimeline for information about the timeline view.