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

Timeline



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:

Mar 8, 2010:

10:52 PM Changeset in webkit [55713] by tkent@chromium.org
  • 1 edit
    2 adds in trunk/WebKitTools

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

Reviewed by Dimitri Glazkov.

[DRT/Chromium] Add EventSender
​https://bugs.webkit.org/show_bug.cgi?id=35859

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

  • src/webkit/tools/test_shell/event_sending_controller.cc
  • src/webkit/tools/test_shell/event_sending_controller.h of Chromium rev.40492.
  • DumpRenderTree/chromium/EventSender.cpp: Added.
  • DumpRenderTree/chromium/EventSender.h: Added.
10:39 PM Changeset in webkit [55712] by tkent@chromium.org
  • 1 edit
    4 adds in trunk/WebKitTools

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

Reviewed by Dimitri Glazkov.

[DRT/Chromium] Add PlainTextController and TextInputController
​https://bugs.webkit.org/show_bug.cgi?id=35852

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

  • src/webkit/tools/test_shell/plain_text_controller.{cc,h} and
  • src/webkit/tools/test_shell/text_input_controller.{cc,h} of Chromium rev.40492.
  • DumpRenderTree/chromium/PlainTextController.cpp: Added.
  • DumpRenderTree/chromium/PlainTextController.h: Added.
  • DumpRenderTree/chromium/TextInputController.cpp: Added.
  • DumpRenderTree/chromium/TextInputController.h: Added.
10:29 PM Changeset in webkit [55711] by tkent@chromium.org
  • 1 edit in trunk/LayoutTests/ChangeLog

Unreviewed. Fix a wrong bug number.

10:24 PM Changeset in webkit [55710] by tkent@chromium.org
  • 7 edits
    3 adds in trunk

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

Reviewed by Adam Barth.

Implement HTML5 <hgroup> element.
​https://bugs.webkit.org/show_bug.cgi?id=32943

The new test file tests:

  • <p> closing,
  • Residual style, and
  • FormatBlock.
  • fast/html/hgroup-element-expected.txt: Added.
  • fast/html/hgroup-element.html: Added.
  • fast/html/script-tests/hgroup-element.js: Added.

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

Reviewed by Adam Barth.

Implement HTML5 <hgroup> element.
​https://bugs.webkit.org/show_bug.cgi?id=33369

<hgroup> should behave the same as <nav>, <section>, <article>, and <aside>.
<hgroup> has no specific parsing rules.

Test: fast/html/hgroup-element.html

  • css/html.css: Add hgroup as a block element.
  • editing/htmlediting.cpp: (WebCore::validBlockTag): Add hgroupTag.
  • html/HTMLElement.cpp: (WebCore::createTagPriorityMap): Returns 5 for hgroupTag. (WebCore::blockTagList): Add hgroupTag.
  • html/HTMLParser.cpp: (WebCore::HTMLParser::getNode): Add hgroupTag.
  • html/HTMLTagNames.in: Add hgroup.
9:34 PM Changeset in webkit [55709] by dumi@chromium.org
  • 2 edits in trunk/WebKitTools

Unreviewed, Chromium build fix.

Reverting r55689.

  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
9:15 PM Changeset in webkit [55708] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

Windows build fix: 'P' is not a type. Luckily, 'void' is.

  • wtf/FastMalloc.cpp:

(WTF::fastCheckConsistency):

9:10 PM Changeset in webkit [55707] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

Windows build fix: export a new symbol.

8:57 PM Changeset in webkit [55706] by ggaren@apple.com
  • 5 edits in trunk/JavaScriptCore

Switching malloc implementations requires a world rebuild
​https://bugs.webkit.org/show_bug.cgi?id=35899

Reviewed by Maciej Stachowiak.

  • wtf/FastMalloc.cpp:

(WTF::fastCheckConsistency):
(WTF::TCMallocStats::fastCheckConsistency):

  • wtf/FastMalloc.h:
  • wtf/ValueCheck.h:

(WTF::): Moved pointer checking into a helper function in FastMalloc.cpp,
so you can switch malloc implementations without rebuilding the world.

8:32 PM Changeset in webkit [55705] by tony@chromium.org
  • 4 edits
    2 adds in trunk

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

Reviewed by Adam Barth.

​https://bugs.webkit.org/show_bug.cgi?id=32131
Work around a crash when inserting an ordered list. This was caused
by incorrect logic when trying to restore a range from a location.
We compute the offset using TextIterator, but were sometimes using
VisiblePosition::next() to iterate instead.

  • editing/execCommand/insert-ordered-list-expected.txt: Added.
  • editing/execCommand/insert-ordered-list.html: Added.

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

Reviewed by Adam Barth.

​https://bugs.webkit.org/show_bug.cgi?id=32131
Crash when inserting an ordered list.

Test: editing/execCommand/insert-ordered-list.html

  • editing/CompositeEditCommand.cpp: (WebCore::CompositeEditCommand::moveParagraphs):
  • editing/TextIterator.cpp: (WebCore::TextIterator::rangeFromLocationAndLength):
8:21 PM Changeset in webkit [55704] by ukai@chromium.org
  • 2 edits in trunk/WebKitTools

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

Reviewed by Adam Barth.

Fix webkit-patch post and land to work well with security bug
​https://bugs.webkit.org/show_bug.cgi?id=35733

Bugzilla requires authentication to access security bug page,
so call authenticate() if it failed to fetch bug page.

  • Scripts/webkitpy/bugzilla.py:
5:47 PM Changeset in webkit [55703] by Simon Fraser
  • 2 edits in trunk/WebKit/mac

2010-03-08 Simon Fraser <Simon Fraser>

Reviewed by Mark Rowe.

Move the new method to the end of the MIG definitions file, to avoid breaking
compatibility between WebKit and older versions of WebKitPluginHost.

  • Plugins/Hosted/WebKitPluginHost.defs:
5:33 PM Changeset in webkit [55702] by Simon Fraser
  • 8 edits in trunk/WebKit/mac

2010-03-08 Simon Fraser <Simon Fraser>

Reviewed by Kevin Decker.

<rdar://problem/7714340> Need to grab image snapshot of Core Animation plugins

Allow plug-ins using the Core Animation drawing model to be captured when doing a flattening paint,
by sending them a drawRect event as if they were software-painting.

  • Plugins/Hosted/NetscapePluginInstanceProxy.h:
  • Plugins/Hosted/NetscapePluginInstanceProxy.mm: (WebKit::NetscapePluginInstanceProxy::snapshot): New snapshot() method that calls over to the plugin host, and then draws the image that comes back.
  • Plugins/Hosted/WebHostedNetscapePluginView.mm: (-[WebHostedNetscapePluginView drawRect:]): If we don't have a software renderer, but we're doing a flattening paint, then call the snapshot method.
  • Plugins/Hosted/WebKitPluginHost.defs: Added snapshot method.
  • Plugins/WebBaseNetscapePluginView.h:
  • Plugins/WebBaseNetscapePluginView.mm: (-[WebBaseNetscapePluginView inFlatteningPaint]): New utility method that asks the FrameView whether the current paint behavior is flattening.
  • Plugins/WebNetscapePluginView.mm: (-[WebNetscapePluginView drawRect:]): If the plug-in is using CA but this is a flattening paint, go ahead and send a drawRect event to the plug-in.
5:14 PM Changeset in webkit [55701] by Darin Adler
  • 3 edits
    1 add in trunk

WebCore: Added a test for crash when you quit inside an unload handler.
rdar://problem/6958347

Reviewed by Dan Bernstein.

  • manual-tests/quit-inside-unload.html: Added.

WebKit/mac: Fix crash when you quit inside an unload handler.
rdar://problem/6958347

Reviewed by Dan Bernstein.

Test: manual-tests/quit-inside-unload.html

  • WebView/WebView.mm:

(-[WebView _closeWithFastTeardown]): Removed code to set
_private->closed since _close now does this earlier, before
calling this method.
(-[WebView _close]): Moved code to set _private->closed to the
top of this method.
(-[WebView setHostWindow:]): Rewrote the code that forbade
setting the host window after closing so that it only forbids
non-nil host windows. That way, the code to clear away the host
window can run safely after setting _private->closed, yet client
code cannot set a new host window after closing.

5:08 PM Changeset in webkit [55700] by Darin Adler
  • 2 edits in trunk/WebKit/mac

Roll out a file I checked in by accident.

  • WebView/WebView.mm: Back to previous version.
5:06 PM Changeset in webkit [55699] by Darin Adler
  • 4 edits
    2 adds in trunk

2010-03-08 Darin Adler <Darin Adler>

Reviewed by Jon Honeycutt.

Don't auto-play <audio> and <video> elements loaded in background tabs
​https://bugs.webkit.org/show_bug.cgi?id=35886
rdar://problem/7117745

  • manual-tests/video-in-non-frontmost-tab.html: Added.
  • manual-tests/resources/video-tab.html: Added.
  • html/HTMLMediaElement.h: Added MediaCanStartListener as a base class, and added the mediaCanStart function as well as a boolean, m_isWaitingUntilMediaCanStart.
  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::HTMLMediaElement): Initialize m_isWaitingUntilMediaCanStart. (WebCore::HTMLMediaElement::~HTMLMediaElement): Call removeMediaCanStartListener if m_isWaitingUntilMediaCanStart is true. (WebCore::HTMLMediaElement::loadInternal): Set m_isWaitingUntilMediaCanStart and call addMediaCanStartListener if canStartMedia is false. (WebCore::HTMLMediaElement::mediaCanStart): Clear m_isWaitingUntilMediaCanStart and call loadInternal.
5:00 PM Changeset in webkit [55698] by cmarrin@apple.com
  • 1 edit in trunk/LayoutTests/platform/mac-leopard/Skipped

Fixed path in mac-leopard Skipped file. Was animation, now animations

4:56 PM Changeset in webkit [55697] by cmarrin@apple.com
  • 2 edits in trunk

This turns back on HW comp on Leopard (which I turned off in ​http://trac.webkit.org/changeset/55696) if you have WebGL turned on. I've added the failing tests to the Skipped file instead

4:39 PM Changeset in webkit [55696] by cmarrin@apple.com
  • 1 edit in trunk/WebKit/mac/WebView/WebView.mm

Backed out the half of ​http://trac.webkit.org/changeset/55690 that turned on HW accel for Leopard which was causing LayoutTests to fail

4:38 PM Changeset in webkit [55695] by ap@apple.com
  • 2 edits in trunk/WebKit/chromium

Chromium build fix.

Updated for the removal of "mightDownloadFromHandle".

4:35 PM Changeset in webkit [55694] by Csaba Osztrogonác
  • 2 edits in trunk/WebKit/gtk

[GTK] Unreviewed buildfix after r55688.

  • webkit/webkitdownload.cpp:

(webkit_download_start):

4:30 PM Changeset in webkit [55693] by Csaba Osztrogonác
  • 2 edits in trunk/WebCore

[GTK] Unreviewed buildfix after r55688.

  • platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp:

(webKitWebSrcStart):

4:26 PM Changeset in webkit [55692] by Csaba Osztrogonác
  • 2 edits in trunk/WebCore

[GTK] Unreviewed buildfix after r55688.

  • platform/network/soup/ResourceHandleSoup.cpp:

(WebCore::):

4:19 PM Changeset in webkit [55691] by Csaba Osztrogonác
  • 2 edits in trunk/WebCore

[Qt] Unreviewed buildfix after r55688.

  • platform/network/qt/ResourceHandleQt.cpp:

(WebCore::ResourceHandle::loadResourceSynchronously):

4:11 PM Changeset in webkit [55690] by cmarrin@apple.com
  • 1 edit in trunk/LayoutTests/platform/mac-leopard/Skipped

Added animation/fill-mode-transform.html to Skipped list for Leopard until ​https://bugs.webkit.org/show_bug.cgi?id=35714 is fixed

3:56 PM Changeset in webkit [55689] by dpranke@chromium.org
  • 2 edits in trunk/WebKitTools

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

Reviewed by Dimitry Glazkov.

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:
3:52 PM Changeset in webkit [55688] by ap@apple.com
  • 8 edits in trunk/WebCore

Reviewed by Darin Adler.

​https://bugs.webkit.org/show_bug.cgi?id=35879
Eliminate m_mightDownloadFromHandle

It was only used on Mac, and unnecessarily complicated the code.

No change in behavior, thus no test.

  • loader/MainResourceLoader.cpp: (WebCore::MainResourceLoader::loadNow):
  • loader/ResourceLoader.cpp: (WebCore::ResourceLoader::load):
  • loader/appcache/ApplicationCacheGroup.cpp: (WebCore::ApplicationCacheGroup::createResourceHandle):
  • platform/network/ResourceHandle.cpp: (WebCore::ResourceHandle::ResourceHandle): (WebCore::ResourceHandle::create):
  • platform/network/ResourceHandle.h:
  • platform/network/ResourceHandleInternal.h: (WebCore::ResourceHandleInternal::ResourceHandleInternal):
  • platform/network/mac/ResourceHandleMac.mm: (WebCore::ResourceHandle::start):
3:44 PM Changeset in webkit [55687] by pkasting@chromium.org
  • 10 edits in trunk/WebCore

Clean up usage of m_failed in open-source image decoders, part 1.
​https://bugs.webkit.org/show_bug.cgi?id=35411

Reviewed by Adam Barth.

Makes setFailed() virtual so subclasses can override it (none do yet) to
do automatic cleanup on failure; makes it return a bool for easy
tailcalling; makes failed() the only way to access m_failed so
subclasses are assured setFailed() won't be bypassed. Plus one or two
other tiny cleanup bits.

Overriding setFailed() is coming in a subsequent patch because it can be
hairy and needs close review.

No functional change, so no tests.

  • platform/graphics/qt/ImageDecoderQt.cpp:

(WebCore::ImageDecoderQt::setData):
(WebCore::ImageDecoderQt::frameBufferAtIndex):
(WebCore::ImageDecoderQt::internalDecodeSize):
(WebCore::ImageDecoderQt::internalReadImage):
(WebCore::ImageDecoderQt::internalHandleCurrentImage):
(WebCore::ImageDecoderQt::forceLoadEverything):
(WebCore::ImageDecoderQt::clearPointers):

  • platform/graphics/qt/ImageDecoderQt.h:
  • platform/image-decoders/ImageDecoder.h:

(WebCore::ImageDecoder::ImageDecoder):
(WebCore::ImageDecoder::setData):
(WebCore::ImageDecoder::setSize):
(WebCore::ImageDecoder::setFailed):
(WebCore::ImageDecoder::failed):

  • platform/image-decoders/bmp/BMPImageDecoder.cpp:

(WebCore::BMPImageDecoder::isSizeAvailable):
(WebCore::BMPImageDecoder::frameBufferAtIndex):
(WebCore::BMPImageDecoder::processFileHeader):

  • platform/image-decoders/gif/GIFImageDecoder.cpp:

(WebCore::GIFImageDecoder::setData):
(WebCore::GIFImageDecoder::isSizeAvailable):
(WebCore::GIFImageDecoder::frameBufferAtIndex):
(WebCore::GIFImageDecoder::decode):
(WebCore::GIFImageDecoder::initFrameBuffer):

  • platform/image-decoders/ico/ICOImageDecoder.cpp:

(WebCore::ICOImageDecoder::decodeAtIndex):
(WebCore::ICOImageDecoder::processDirectory):
(WebCore::ICOImageDecoder::processDirectoryEntries):

  • platform/image-decoders/jpeg/JPEGImageDecoder.cpp:

(WebCore::JPEGImageReader::decode):
(WebCore::JPEGImageDecoder::setData):
(WebCore::JPEGImageDecoder::isSizeAvailable):
(WebCore::JPEGImageDecoder::frameBufferAtIndex):
(WebCore::JPEGImageDecoder::outputScanlines):
(WebCore::JPEGImageDecoder::decode):

  • platform/image-decoders/png/PNGImageDecoder.cpp:

(WebCore::decodingFailed):
(WebCore::PNGImageReader::decode):
(WebCore::PNGImageDecoder::setData):
(WebCore::PNGImageDecoder::isSizeAvailable):
(WebCore::PNGImageDecoder::frameBufferAtIndex):
(WebCore::PNGImageDecoder::headerAvailable):
(WebCore::PNGImageDecoder::rowAvailable):
(WebCore::PNGImageDecoder::decode):

  • platform/image-decoders/png/PNGImageDecoder.h:
3:14 PM Changeset in webkit [55686] by Dimitri Glazkov
  • 7 edits
    2 adds in trunk

[Chromium] Reset multiple form submission protection on mouse events.
​https://bugs.webkit.org/show_bug.cgi?id=35128

Reviewed by Darin Fisher.

WebKit/chromium:

  • src/WebViewImpl.cpp:

(WebKit::WebViewImpl::mouseDown):

LayoutTests:

  • fast/forms/multiple-form-submission-protection-mouse-expected.txt: Copied from LayoutTests/editing/execCommand/copy-without-selection-expected.txt.
  • fast/forms/multiple-form-submission-protection-mouse.html: Added.
2:54 PM Changeset in webkit [55685] by cmarrin@apple.com
  • 4 edits in trunk

When WebGL is enabled, enable hardware compositing bypassing check for CoreVideo bug fix
​https://bugs.webkit.org/show_bug.cgi?id=35759

2:54 PM PythonGuidelines edited by Chris Jerdonek
Added preliminary instructions on how to install new versions of … (diff)
2:45 PM Changeset in webkit [55684] by oliver@apple.com
  • 6 edits in trunk/JavaScriptCore

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

Reviewed by Darin Adler.

TypeInfo is unnecessarily large
​https://bugs.webkit.org/show_bug.cgi?id=35850

Reduce the size of the type and flags members to a single
byte each, reducing the size of Structure by 8 bytes.

  • assembler/MacroAssemblerX86Common.h: (JSC::MacroAssemblerX86Common::branch8): (JSC::MacroAssemblerX86Common::branchTest8): (JSC::MacroAssemblerX86Common::setTest8): Add single byte branches, and correct setTest8 to do a single byte read from memory, and actually store the result
  • assembler/X86Assembler.h: (JSC::X86Assembler::): (JSC::X86Assembler::cmpb_im): (JSC::X86Assembler::testb_im):
  • jit/JITCall.cpp: (JSC::JIT::emit_op_construct_verify):
  • jit/JITOpcodes.cpp: (JSC::JIT::emit_op_instanceof): (JSC::JIT::emit_op_jeq_null): (JSC::JIT::emit_op_jneq_null): (JSC::JIT::emit_op_get_pnames): (JSC::JIT::emit_op_convert_this): (JSC::JIT::emit_op_construct_verify): (JSC::JIT::emit_op_to_jsnumber): (JSC::JIT::emit_op_eq_null): (JSC::JIT::emit_op_neq_null):
  • runtime/JSTypeInfo.h: (JSC::TypeInfo::TypeInfo): (JSC::TypeInfo::type):
2:38 PM Changeset in webkit [55683] by darin@chromium.org
  • 3 edits in trunk/WebKit/chromium

2010-03-08 Michael Nordman <Michael Nordman>

Reviewed by Darin Fisher.

Quick fix for a chromium unit test failures around r55675 (repaint refactoring).
This change restores the tests for empty rects that were deleted.

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

  • src/ChromeClientImpl.cpp: (WebKit::ChromeClientImpl::invalidateContentsAndWindow):
  • src/WebPopupMenuImpl.cpp: (WebKit::WebPopupMenuImpl::invalidateContentsAndWindow):
2:16 PM Changeset in webkit [55682] by Darin Adler
  • 5 edits in trunk/WebCore

2010-03-08 Darin Adler <Darin Adler>

Reviewed by Jon Honeycutt.

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

Fix minor style issues in HTMLMediaElement and classes derived from it.
Made many public members private and protected.

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::screenRect): Got rid of a stray "const" and retstructured the function to use early return and get rid of a local.
  • html/HTMLMediaElement.h: Made lots of members private and some protected. Also use private inheritance instead of public. Removed some unneeded includes.
  • html/HTMLVideoElement.cpp: (WebCore::HTMLVideoElement::parseMappedAttribute): Use player() instead of m_player; HTMLMediaElement data members are now private, not protected. (WebCore::HTMLVideoElement::supportsFullscreen): Ditto. (WebCore::HTMLVideoElement::videoWidth): Ditto. (WebCore::HTMLVideoElement::videoHeight): Ditto. (WebCore::HTMLVideoElement::hasAvailableVideoFrame): Ditto. (WebCore::HTMLVideoElement::webkitEnterFullScreen): Use isFullscreen() instead of m_isFullscreen; same reason. (WebCore::HTMLVideoElement::webkitExitFullScreen): Ditto. (WebCore::HTMLVideoElement::webkitDisplayingFullscreen): Ditto.
  • html/HTMLVideoElement.h: Removed an unneeded include. Made many public functions private. Got rid of unused paint function, which was replaced with paintCurrentFrameInContext a while back.
2:07 PM Changeset in webkit [55681] by jam@chromium.org
  • 1 edit in trunk/WebKit/chromium/public/WebPluginContainer.h

take out temporary define

1:06 PM Changeset in webkit [55680] by eric@webkit.org
  • 10 edits in trunk

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

Reviewed by Xan Lopez.

​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:
  • platform/gtk/accessibility/table-hierarchy-expected.txt:

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

Reviewed by Xan Lopez.

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

Gives platforms the ability to exclude parts of an AccessibilityTable
from the accessible hierarchy.

  • accessibility/gtk/AccessibilityObjectAtk.cpp: (AccessibilityObject::accessibilityPlatformIncludesObject):
  • accessibility/AccessibilityRenderObject.cpp: (AccessibilityRenderObject::determineAccessibilityRole):
  • accessibility/AccessibilityTable.cpp: (AccessibilityTable::addChildren):
  • accessibility/AccessibilityTableColumn.h: (accessibilityIsIgnored):
  • accessibility/AccessibilityTableHeaderContainer.h: (accessibilityIsIgnored):
  • accessibility/AccessibilityTableRow.cpp: (accessibilityIsIgnored):
12:31 PM Changeset in webkit [55679] by barraclough@apple.com
  • 5 edits in trunk/JavaScriptCore

Reverting 55035, this caused a regression.
(​https://bugs.webkit.org/show_bug.cgi?id=35843)

Reviewed by NOBODY (reverting regression).

  • runtime/JSString.cpp:

(JSC::JSString::resolveRope):
(JSC::JSString::getStringPropertyDescriptor):

  • runtime/JSString.h:

(JSC::jsSingleCharacterSubstring):
(JSC::JSString::getIndex):
(JSC::JSString::getStringPropertySlot):

  • runtime/UStringImpl.cpp:
  • runtime/UStringImpl.h:
12:22 PM Changeset in webkit [55678] by beidson@apple.com
  • 2 edits in trunk/WebKitTools

Followup to ​https://bugs.webkit.org/show_bug.cgi?id=35532

Reviewed by NOBODY (but suggested by Steve Falkenburg and fixing a boneheaded mistake on my part last week)

  • DumpRenderTree/win/LayoutTestControllerWin.cpp:

(LayoutTestController::apiTestGoToCurrentBackForwardItem): Can't pass in a null BOOL to WebKit APIs.

12:21 PM Changeset in webkit [55677] by treat@webkit.org
  • 2 edits in trunk/WebKit/win

Unreviewed build fix for Windows.

Patch by Adam Treat <​atreat@rim.com> on 2010-03-08

  • WebCoreSupport/WebChromeClient.cpp:

(WebChromeClient::invalidateContents):

12:03 PM Changeset in webkit [55676] by jianli@chromium.org
  • 3 edits in trunk/WebCore

No review. Fix build break on Tiger intel release.

  • html/Blob.cpp:
  • html/Blob.h:
11:43 AM Changeset in webkit [55675] by treat@webkit.org
  • 33 edits in trunk

WebCore: Refactor the HostWindow methods for repaint, scroll, invalidate and blit
of backingstore and window by eliminating the three bools that currently
exist as params of the repaint method.
​https://bugs.webkit.org/show_bug.cgi?id=34214

Patch by Adam Treat <​atreat@rim.com> on 2010-03-02
Reviewed by Dave Hyatt.

I've added extra methods to provide the hosts with more semantic
information of what is being requested thus eliminating the need for
these bools.

No tests as this change should not introduce any behavior changes in any
of the ports.

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

  • loader/EmptyClients.h:

(WebCore::EmptyChromeClient::invalidateContents):
(WebCore::EmptyChromeClient::invalidateWindow):
(WebCore::EmptyChromeClient::invalidateContentsAndWindow):
(WebCore::EmptyChromeClient::invalidateContentsForSlowScroll):

  • page/Chrome.cpp:

(WebCore::Chrome::invalidateContents):
(WebCore::Chrome::invalidateWindow):
(WebCore::Chrome::invalidateContentsAndWindow):
(WebCore::Chrome::invalidateContentsForSlowScroll):

  • page/Chrome.h:
  • page/ChromeClient.h:
  • page/FrameView.cpp:

(WebCore::FrameView::invalidateRect):

  • platform/HostWindow.h:
  • platform/ScrollView.cpp:

(WebCore::ScrollView::scrollContents):
(WebCore::ScrollView::wheelEvent):

WebKit/chromium: Adapt the chromium port to the refactoring of repaint methods.

Patch by Adam Treat <​atreat@rim.com> on 2010-03-02
Reviewed by Dave Hyatt.

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

  • src/ChromeClientImpl.cpp:

(WebKit::ChromeClientImpl::invalidateContents):
(WebKit::ChromeClientImpl::invalidateWindow):
(WebKit::ChromeClientImpl::invalidateContentsAndWindow):
(WebKit::ChromeClient::invalidateContentsForSlowScroll):

  • src/ChromeClientImpl.h:

WebKit/gtk: Adapt the gtk port to the refactoring of repaint methods.

Patch by Adam Treat <​atreat@rim.com> on 2010-03-02
Reviewed by Dave Hyatt.

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

  • WebCoreSupport/ChromeClientGtk.cpp:
  • WebCoreSupport/ChromeClientGtk.h:

WebKit/haiku: Adapt the haiku port to the refactoring of repaint methods.

Patch by Adam Treat <​atreat@rim.com> on 2010-03-02
Reviewed by Dave Hyatt.

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

  • WebCoreSupport/ChromeClientHaiku.cpp:

(WebCore::ChromeClientHaiku::invalidateContents):
(WebCore::ChromeClientHaiku::invalidateWindow):
(WebCore::ChromeClientHaiku::invalidateContentsAndWindow):
(WebCore::ChromeClient::invalidateContentsForSlowScroll):

  • WebCoreSupport/ChromeClientHaiku.h:

WebKit/mac: Adapt the mac port to the refactoring of repaint methods.

Patch by Adam Treat <​atreat@rim.com> on 2010-03-02
Reviewed by Dave Hyatt.

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

  • WebCoreSupport/WebChromeClient.h:
  • WebCoreSupport/WebChromeClient.mm:

(WebChromeClient::invalidateContents):
(WebChromeClient::invalidateWindow):
(WebChromeClient::invalidateContentsAndWindow):
(WebChromeClient::invalidateContentsForSlowScroll):

WebKit/qt: Adapt the qt port to the refactoring of repaint methods.

Patch by Adam Treat <​atreat@rim.com> on 2010-03-02
Reviewed by Dave Hyatt.

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

  • WebCoreSupport/ChromeClientQt.cpp:

(WebCore::ChromeClientQt::invalidateContents):
(WebCore::ChromeClientQt::invalidateWindow):
(WebCore::ChromeClientQt::invalidateContentsAndWindow):
(WebCore::ChromeClientQt::invalidateContentsForSlowScroll):

  • WebCoreSupport/ChromeClientQt.h:

WebKit/win: Adapt the win port to the refactoring of repaint methods.

Patch by Adam Treat <​atreat@rim.com> on 2010-03-02
Reviewed by Dave Hyatt.

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

  • WebCoreSupport/WebChromeClient.cpp:

(WebChromeClient::invalidateContents):
(WebChromeClient::invalidateWindow):
(WebChromeClient::invalidateContentsAndWindow):
(WebChromeClient::invalidateContentsForSlowScroll):

  • WebCoreSupport/WebChromeClient.h:

WebKit/wx: Adapt the wx port to the refactoring of repaint methods.

Patch by Adam Treat <​atreat@rim.com> on 2010-03-02
Reviewed by Dave Hyatt.

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

  • WebKitSupport/ChromeClientWx.cpp:

(WebCore::ChromeClientWx::invalidateContents):
(WebCore::ChromeClientWx::invalidateWindow):
(WebCore::ChromeClientWx::invalidateContentsForSlowScroll):
(WebCore::ChromeClientWx::invalidateContentsAndWindow):

  • WebKitSupport/ChromeClientWx.h:
11:40 AM Changeset in webkit [55674] by eric@webkit.org
  • 3 edits
    2 adds in trunk

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

Reviewed by Nate Chapin.

[V8] Block popups from inline script
​https://bugs.webkit.org/show_bug.cgi?id=35474

Test that we block popups generated from <script>window.open(...)</script>.

  • http/tests/security/popup-blocked-from-window-open-expected.txt: Added.
  • http/tests/security/popup-blocked-from-window-open.html: Added.

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

Reviewed by Nate Chapin.

[V8] Block popups from inline script
​https://bugs.webkit.org/show_bug.cgi?id=35474

Apparently, we're supposed to look at the sourceURL to figure out
whether we're running a script tag or a hyperlink. This logic is
copied from the JSC version.

Test: http/tests/security/popup-blocked-from-window-open.html

  • bindings/v8/ScriptController.cpp: (WebCore::ScriptController::processingUserGesture): (WebCore::ScriptController::evaluate):
11:22 AM Changeset in webkit [55673] by eric@webkit.org
  • 10 edits
    2 adds in trunk

2010-03-08 Stuart Morgan <​stuartmorgan@chromium.org>

Reviewed by Darin Adler.

Added a new USE definition for secure text mode on the Mac.
​https://bugs.webkit.org/show_bug.cgi?id=31265

  • wtf/Platform.h:

2010-03-08 Stuart Morgan <​stuartmorgan@chromium.org>

Reviewed by Darin Adler.

Move the details of secure text mode into WebCore/platform.
Move the higher-level logic for secure text mode from Frame
to SelectionController.

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

No new tests: no functional changes.

  • WebCore.gypi:
  • WebCore.xcodeproj/project.pbxproj:
  • dom/Document.cpp: (WebCore::Document::setUseSecureKeyboardEntryWhenActive):
  • editing/SelectionController.cpp: (WebCore::SelectionController::focusedOrActiveStateChanged): (WebCore::SelectionController::updateSecureKeyboardEntryIfActive): (WebCore::SelectionController::setUseSecureKeyboardEntry):
  • editing/SelectionController.h:
  • page/Frame.cpp: (WebCore::Frame::setDocument):
  • page/Frame.h:
  • platform/SecureTextInput.cpp: Added. (WebCore::enableSecureTextInput): (WebCore::disableSecureTextInput):
  • platform/SecureTextInput.h: Added. (WebCore::enableSecureTextInput): (WebCore::disableSecureTextInput):
10:49 AM Changeset in webkit [55672] by beidson@apple.com
  • 6 edits
    4 adds in trunk

Relax the 3rd party cookie policy in cases where it won't add a new tracking vector.
<rdar://problem/7163012> and ​https://bugs.webkit.org/show_bug.cgi?id=35824

Reviewed by Anders Carlsson.

WebCore:

Test: http/tests/cookies/third-party-cookie-relaxing.html

If the 3rd-party domain in question already has a cookie set, allow changes
by setting the first party url of the request to be the url of the request itself:

  • platform/network/cf/ResourceHandleCFNet.cpp:

(WebCore::makeFinalRequest):

Ditto:

  • platform/network/mac/ResourceHandleMac.mm:

(WebCore::ResourceHandle::start):
(WebCore::ResourceHandle::loadResourceSynchronously):

I've filed <rdar://problem/7728508> to track changing the policy in our networking layer.

LayoutTests:

  • http/tests/cookies/resources/cookie-utility.php: Added.
  • http/tests/cookies/resources/third-party-cookie-relaxing-iframe.html: Added.
  • http/tests/cookies/third-party-cookie-relaxing-expected.txt: Added.
  • http/tests/cookies/third-party-cookie-relaxing.html: Added.
  • platform/gtk/Skipped:
  • platform/qt/Skipped:
10:32 AM Changeset in webkit [55671] by Chris Jerdonek
  • 2 edits in trunk/WebKitTools

Unreviewed.

Fixed incorrect import statement in validate-committer-lists:
webkitpy.BeautifulSoup -> webkitpy.thirdparty.BeautifulSoup.

  • Scripts/validate-committer-lists:
10:13 AM Changeset in webkit [55670] by jianli@chromium.org
  • 33 edits
    2 copies
    4 adds in trunk

.: Blob.slice support.
​https://bugs.webkit.org/show_bug.cgi?id=32993

Reviewed by Dmitry Titov.

Add ENABLE_BLOB_SLICE feature define.

  • configure.ac:

JavaScriptCore: Blob.slice support.
​https://bugs.webkit.org/show_bug.cgi?id=32993

Reviewed by Dmitry Titov.

Add ENABLE_BLOB_SLICE feature define.
Also fix a problem that JSValue.toInteger is not exposed on Windows.

  • Configurations/FeatureDefines.xcconfig:
  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def:

WebCore: Blob.slice support.
​https://bugs.webkit.org/show_bug.cgi?id=32993

Reviewed by Dmitry Titov.

The following semantic is adopted per the discussions on public-webapps:
1) File.slice() does a synchronous IO to capture the current size and

modification time and cache them in the resulting Blob.

2) Subsequent Blob operations, like Blob.slice and Blob.size simply

use the cached values.

3) When the underlying file data are accessed, like in XHR.send(), the

UA will check the cached modification time against the current
modification time to determine if the file has been changed or not.
An error or exception will be thrown if needed.

Also add ENABLE_BLOB_SLICE feature define.

Test: http/tests/local/send-sliced-dragged-file.html

  • Configurations/FeatureDefines.xcconfig:
  • GNUmakefile.am:
  • WebCore.pri:
  • html/Blob.cpp:

(WebCore::Blob::Blob):
(WebCore::Blob::size):
(WebCore::Blob::slice):

  • html/Blob.h:

(WebCore::Blob::start):
(WebCore::Blob::length):
(WebCore::Blob::modificationTime):

  • html/Blob.idl:
  • platform/network/FormData.cpp:

(WebCore::FormData::deepCopy):
(WebCore::FormData::appendFile):
(WebCore::FormData::appendFileRange):

  • platform/network/FormData.h:

(WebCore::FormDataElement::FormDataElement):
(WebCore::operator==):

  • platform/network/mac/FormDataStreamMac.mm:

(WebCore::advanceCurrentStream):
(WebCore::openNextStream):
(WebCore::formCreate):
(WebCore::formOpen):
(WebCore::formRead):
(WebCore::setHTTPBody):

  • xml/XMLHttpRequest.cpp:

(WebCore::XMLHttpRequest::send):

WebKit/chromium: Blob.slice support.
​https://bugs.webkit.org/show_bug.cgi?id=32993

Reviewed by Dmitry Titov.

Add ENABLE_BLOB_SLICE feature define.

  • features.gypi:

WebKit/mac: Blob.slice support.
​https://bugs.webkit.org/show_bug.cgi?id=32993

Reviewed by Dmitry Titov.

Add ENABLE_BLOB_SLICE feature define.

  • Configurations/FeatureDefines.xcconfig:

WebKitLibraries: Blob.slice support.
​https://bugs.webkit.org/show_bug.cgi?id=32993

Reviewed by Dmitry Titov.

Add ENABLE_BLOB_SLICE feature define.

  • win/tools/vsprops/FeatureDefines.vsprops:
  • win/tools/vsprops/FeatureDefinesCairo.vsprops:

WebKitTools: Blob.slice support.
​https://bugs.webkit.org/show_bug.cgi?id=32993

Reviewed by Dmitry Titov.

Add ENABLE_BLOB_SLICE feature define and flag for build-webkit. It is
currently only turned on for Mac.

  • Scripts/build-webkit:

LayoutTests: Blob.slice support.
​https://bugs.webkit.org/show_bug.cgi?id=32993

Reviewed by Dmitry Titov.

Add a new layout test to test slicing and sending a file. In addition,
two new scripts are introduced to create and touch the temp file.

Move some code from send-dragged-file.js to a new file
setup-file-input-element-for-drag.js so that it can be used by both
send-dragged-file.js and send-sliced-dragged-file.js to setup the input
element for the file dragging.

Since the Blob.slice is only enabled for Mac, we disable the new test
for all other platforms.

  • fast/dom/Window/window-properties.html:
  • http/tests/local/resources/file-for-drag-to-send.txt:
  • http/tests/local/resources/send-dragged-file.js:
  • http/tests/local/resources/send-sliced-dragged-file.js: Added.
  • http/tests/local/resources/setup-file-input-element-for-drag.js: Added.
  • http/tests/local/send-dragged-file.html:
  • http/tests/local/send-sliced-dragged-file-expected.txt: Added.
  • http/tests/local/send-sliced-dragged-file.html: Added.
  • http/tests/resources/touch-temp-file.php: Added.
  • http/tests/resources/write-temp-file.php: Added.
  • platform/gtk/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:
9:54 AM Changeset in webkit [55669] by dbates@webkit.org
  • 4 edits
    4 adds in trunk

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

Reviewed by Simon Fraser.

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

Fixes an issue where we repaint the caret rectangle even if the associated
selection is not in a content editable element. This is extraneous since the
caret is only visible when the selection is in a content editable element.
Hence, we should only repaint the caret rectangle when the associated selection
is in a content editable element.

Note, we always paint the caret when caret browsing is enabled.

Test: fast/repaint/no-caret-repaint-in-non-content-editable-element.html

  • editing/SelectionController.cpp: (WebCore::SelectionController::recomputeCaretRect): Modified to call method SelectionController::shouldRepaintCaret. (WebCore::SelectionController::shouldRepaintCaret): Added. (WebCore::SelectionController::invalidateCaretRect): Modified to call method SelectionController::shouldRepaintCaret.
  • editing/SelectionController.h:

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

Reviewed by Simon Fraser.

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

Tests that we do not repaint the caret rectangle when the associated selection is
not in a content editable element.

  • fast/repaint/no-caret-repaint-in-non-content-editable-element.html: Added.
  • platform/mac/fast/repaint/no-caret-repaint-in-non-content-editable-element-expected.checksum: Added.
  • platform/mac/fast/repaint/no-caret-repaint-in-non-content-editable-element-expected.png: Added.
  • platform/mac/fast/repaint/no-caret-repaint-in-non-content-editable-element-expected.txt: Added.
9:36 AM Changeset in webkit [55668] by eric.carlson@apple.com
  • 3 edits in trunk/WebCore

2010-03-08 Eric Carlson <​eric.carlson@apple.com>

Reviewed by Darin Adler.

YouTube HTML5 video never starts playing on Windows
​https://bugs.webkit.org/show_bug.cgi?id=33954

  • platform/graphics/win/MediaPlayerPrivateQuickTimeWin.cpp: (WebCore::MediaPlayerPrivate::rfc2616DateStringFromTime): New, create an rfc 2616 formatted string for an absolute time value. (WebCore::addCookieParam): New, add a cookie param and value to a string builder. (WebCore::MediaPlayerPrivate::setUpCookiesForQuickTime): Copy cookies for the movie to

be loaded from CFNetwork into WinINet so they are available when QuickTime tries to
download the movie.

(WebCore::MediaPlayerPrivate::load): Call setupCookiesForQuickTime.

  • platform/graphics/win/MediaPlayerPrivateQuickTimeWin.h:
9:26 AM Changeset in webkit [55667] by dbates@webkit.org
  • 2 edits in trunk/WebKit/win

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

Unreviewed, build fix.

Attempt to fix the Windows builds by applying the corresponding change
made in bug #35763 <​https://bugs.webkit.org/show_bug.cgi?id=35763>.

8:41 AM Changeset in webkit [55666] by eric@webkit.org
  • 14 edits in trunk

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

Reviewed by David Levin.

Remove the now-redundant Settings fields for the Database
​https://bugs.webkit.org/show_bug.cgi?id=35763

No new tests; this code isn't called.

  • WebCore.base.exp:
  • page/Settings.cpp: (WebCore::Settings::Settings):
  • page/Settings.h:

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

Reviewed by David Levin.

Remove the now-redundant Settings fields for the Database
​https://bugs.webkit.org/show_bug.cgi?id=35763

No new tests; this code isn't called.

  • public/WebSettings.h:
  • src/WebSettingsImpl.cpp:
  • src/WebSettingsImpl.h:

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

Reviewed by David Levin.

Remove the now-redundant Settings fields for the Database
​https://bugs.webkit.org/show_bug.cgi?id=35763

No new tests; this code isn't called.

  • webkit/webkitwebview.cpp: Remove the calls into Settings. (DNDContentsRequest::webkit_web_view_update_settings): (DNDContentsRequest::webkit_web_view_settings_notify):

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

Reviewed by David Levin.

Remove the now-redundant Settings fields for the Database
​https://bugs.webkit.org/show_bug.cgi?id=35763

No new tests; this code isn't called.

  • WebView/WebView.mm: Remove the call into Settings. (-[WebView _preferencesChangedNotification:]):

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

Reviewed by David Levin.

Remove the now-redundant Settings fields for the Database
​https://bugs.webkit.org/show_bug.cgi?id=35763

No new tests; this code isn't called.

  • Api/qwebsettings.cpp: (QWebSettingsPrivate::apply): Remove the call into Settings.
7:57 AM Changeset in webkit [55665] by eric@webkit.org
  • 2 edits in trunk/WebKit/qt

2010-03-08 Simon Hausmann <​simon.hausmann@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] Binary incompatibility between Qt 4.6's WebKit and trunk in QWebSettings
​https://bugs.webkit.org/show_bug.cgi?id=35858

Moved the enum value added in r54873 to the end of the enum, to preserve
binary compatibility. DnsPrefetchEnabled was in the last release and needs
to remain after LocalContentCanAccessRemoteUrls.

  • Api/qwebsettings.h:
7:41 AM Changeset in webkit [55664] by eric@webkit.org
  • 2 edits
    3 adds in trunk/LayoutTests

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

Reviewed by Holger Freyther.

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

  • platform/qt/Skipped:
  • platform/qt/fast/canvas/canvas-incremental-repaint-2-expected.checksum: Added.
  • platform/qt/fast/canvas/canvas-incremental-repaint-2-expected.png: Added.
  • platform/qt/fast/canvas/canvas-incremental-repaint-2-expected.txt: Added.
7:07 AM Changeset in webkit [55663] by kov@webkit.org
  • 4 edits in trunk

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

No review, rolling out r55662.
​http://trac.webkit.org/changeset/55662
​https://bugs.webkit.org/show_bug.cgi?id=35863

Need to be coordinated with bots setup

  • http/conf/apache2-debian-httpd.conf:

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

No review, rolling out r55662.
​http://trac.webkit.org/changeset/55662
​https://bugs.webkit.org/show_bug.cgi?id=35863

Need to be coordinated with bots setup

  • Scripts/run-webkit-tests:
6:23 AM Changeset in webkit [55662] by kov@webkit.org
  • 4 edits in trunk

LayoutTests

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

Reviewed by Holger Freyther.

[GTK] Tests fail when running with ipv6 addresses available, on Debian systems
​https://bugs.webkit.org/show_bug.cgi?id=35863

Also listen for SSL requests on the ipv6 address.

  • http/conf/apache2-debian-httpd.conf:

WebKitTools

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

Reviewed by Holger Freyther.

[GTK] Tests fail when running with ipv6 addresses available, on Debian systems
​https://bugs.webkit.org/show_bug.cgi?id=35863

If running on a Debian-based system, also listen on the ipv6 address.

  • Scripts/run-webkit-tests:
6:08 AM QtWebKitBugs edited by jocelyn.turcotte@nokia.com
(diff)
5:21 AM Changeset in webkit [55661] by kov@webkit.org
  • 2 edits in trunk/LayoutTests

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

[GTK] DRT needs to support synthesizing double clicks
​https://bugs.webkit.org/show_bug.cgi?id=35862

Skip test that needs a new feature in DRT.

  • platform/gtk/Skipped:
5:16 AM Changeset in webkit [55660] by kov@webkit.org
  • 3 edits in trunk/LayoutTests

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

Unreviewed. Rebaseline two tests that had their expected results
changed.

  • platform/gtk/inspector/timeline-network-resource-expected.txt:
  • platform/gtk/svg/css/getComputedStyle-basic-expected.txt:
4:19 AM Changeset in webkit [55659] by jorlow@chromium.org
  • 5 edits in trunk/WebKit/chromium

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

Reviewed by Darin Fisher.

Pass the WebFrame into WebStorageArea::setItem so we can figure out the routing ID
​https://bugs.webkit.org/show_bug.cgi?id=35758

This is necessary since setItem is sometimes blocked and the embedder might need
to display some piece of UI associated with such an event.

  • public/WebStorageArea.h: (WebKit::WebStorageArea::setItem):
  • src/StorageAreaProxy.cpp: (WebCore::StorageAreaProxy::setItem):
  • src/WebStorageAreaImpl.cpp: (WebKit::WebStorageAreaImpl::setItem):
  • src/WebStorageAreaImpl.h:
3:39 AM QtWebKitBugs edited by vestbo@webkit.org
(diff)
3:23 AM QtWebKitBugs edited by vestbo@webkit.org
(diff)
3:19 AM QtWebKitBugs edited by vestbo@webkit.org
(diff)
3:17 AM QtWebKitBugs edited by vestbo@webkit.org
(diff)
2:07 AM QtWebKitBackportingFixes edited by Simon Hausmann
(diff)
2:06 AM QtWebKitRelease20 created by Simon Hausmann
2:06 AM QtWebKit edited by Simon Hausmann
(diff)
2:04 AM QtWebKitBackportingFixes edited by Simon Hausmann
(diff)
1:57 AM QtWebKit edited by Simon Hausmann
(diff)
12:16 AM Changeset in webkit [55658] by zecke@webkit.org
  • 2 edits in trunk/WebKitTools

[iexploder] Automatically update htmltags.in and htmlattrs.in too

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

Change the update-iexploder-cssproperites script to update
the htmlattrs.in and htmltags.in of WebKitTools/iExploder/htdocs
automatically as well.

Change the reading and writing code to work with parameters
and extend the method that is parsing the .in files to handle
the HTMLTagNames.in and the HTMLAttributeNames.in files.

Remove custom code to determine the revision of files with a
utility of VCUtils.pm to determine the revision of the directory
these files are located in. This will also work with git checkout.

  • Scripts/update-iexploder-cssproperties:
12:13 AM Changeset in webkit [55657] by Philippe Normand
  • 3 edits in trunk/WebCore

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

Reviewed by Eric Seidel.

[GStreamer] replace g_idle_add / g_timeout_add calls with Timers in the gstreamer player
​https://bugs.webkit.org/show_bug.cgi?id=35735

Replaced g_idle_add calls with Timers immediately started once
only. The g_timeout_add was replaced by a periodic Timer.

  • 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:
Note: See TracTimeline for information about the timeline view.