Timeline


and

06/27/09:

23:05 Changeset [45321] by simon.fraser@apple.com

2009-06-27 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dan Bernstein.

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

Fix up previous change. When computeCompositingRequirements() determines
that the current layer is composited, it needs to inform its parent
by setting compositingState.m_subtreeIsCompositing() to true. That didn't
always happen after the previous patch. Clarified the logic here.

  • rendering/RenderLayerCompositor.cpp: (WebCore::CompositingState::CompositingState): (WebCore::RenderLayerCompositor::computeCompositingRequirements):
22:54 Changeset [45320] by jmalonzo@webkit.org

2009-06-27 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Jan Alonzo.

[Qt] Build fix after r45290
https://bugs.webkit.org/show_bug.cgi?id=26769

  • WebCore.pro:
22:54 Changeset [45319] by jmalonzo@webkit.org

2009-06-27 Emilio Pozuelo Monfort <pochu27@gmail.com>

Reviewed by Jan Alonzo.

[GTK] Don't use deprecated GTK+ symbols.
https://bugs.webkit.org/show_bug.cgi?id=26583

  • plugins/gtk/gtk2xtbin.c: (gtk_xtbin_class_init): (gtk_xtbin_new): (gtk_xtbin_destroy):
  • GtkLauncher/main.c: (create_toolbar):
21:38 Changeset [45318] by simon.fraser@apple.com

2009-06-27 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dan Bernstein.

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

First part: fix the RenderLayer::hasCompositingDescendant() flag to be set
correctly.

  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::computeCompositingRequirements): Do not unconditionally set compositingState.m_subtreeIsCompositing, because that can clobber the value from an earlier sibling. Add some more comments.

Remove a final use of Vector iterators.

(WebCore::RenderLayerCompositor::recursiveRepaintLayerRect):
Move the normalFlowList() processing outside the test for isStackingContext().

21:19 Changeset [45317] by mitz@apple.com

Reviewed by Simon Fraser and Antti Koivisto.

  • make paintFillLayerExtended() non-virtual and remove its clipY and clipH parameters

These parameters were computed and passed along to
paintFillLayerExtended in order to vertically constrain the rect fill to
the damage rect, because Qt cannot paint tall rectangles (see
<http://websvn.kde.org/?view=rev&revision=42721>). Since the damage rect
is passed along in the PaintInfo, the extra parameters are redundant,
and the intersection can just take place in paintFillLayerExtended().

  • rendering/InlineFlowBox.cpp: (WebCore::InlineFlowBox::paintFillLayers): (WebCore::InlineFlowBox::paintFillLayer): (WebCore::InlineFlowBox::paintBoxDecorations): (WebCore::InlineFlowBox::paintMask):
  • rendering/InlineFlowBox.h:
  • rendering/RenderBox.cpp: (WebCore::RenderBox::paintRootBoxDecorations): (WebCore::RenderBox::paintBoxDecorations): (WebCore::RenderBox::paintMask): (WebCore::RenderBox::paintMaskImages): (WebCore::RenderBox::paintFillLayers): (WebCore::RenderBox::paintFillLayer):
  • rendering/RenderBox.h:
  • rendering/RenderBoxModelObject.cpp: (WebCore::RenderBoxModelObject::paintFillLayerExtended):
  • rendering/RenderBoxModelObject.h:
  • rendering/RenderFieldset.cpp: (WebCore::RenderFieldset::paintBoxDecorations): (WebCore::RenderFieldset::paintMask):
  • rendering/RenderTable.cpp: (WebCore::RenderTable::paintBoxDecorations): (WebCore::RenderTable::paintMask):
  • rendering/RenderTableCell.cpp: (WebCore::RenderTableCell::paintBackgroundsBehindCell): (WebCore::RenderTableCell::paintMask):
20:13 Changeset [45316] by bfulgham@webkit.org

WebCore:

2009-06-27 Ryosuke Niwa <rniwa@google.com>

Reviewed by Eric Seidel.

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

Clean up for IndentOutdentCommand::indentRegion, and solved most of problems related to the bug 21712.
https://bugs.webkit.org/show_bug.cgi?id=21712

Added few utility functions to htmlediting.h/cpp

isVisibilyAdjacent checks whether the first position is visibly next to the second position.
i.e. there is no visible node between the first and second positions

canMergeLists checks whether two lists can be merged.
It checks the type of list, the editing boundary, and adjacency of the lists.

Tests: editing/execCommand/indent-nested-lists-1.html

editing/execCommand/indent-nested-lists-2.html
editing/execCommand/indent-nested-lists-3.html
editing/execCommand/indent-nested-lists-4.html
editing/execCommand/indent-nested-lists-5.html
editing/execCommand/indent-nested-lists-6.html
editing/execCommand/indent-nested-lists-7.html
editing/execCommand/outdent-nested-lists-1.html
editing/execCommand/outdent-nested-lists-2.html
editing/execCommand/outdent-nested-lists-3.html
editing/execCommand/outdent-nested-lists-4.html

  • editing/IndentOutdentCommand.cpp: (WebCore::IndentOutdentCommand::prepareBlockquoteLevelForInsertion): (WebCore::IndentOutdentCommand::tryIndentingAsListItem): (WebCore::IndentOutdentCommand::indentIntoBlockquote): (WebCore::IndentOutdentCommand::indentRegion):
  • editing/IndentOutdentCommand.h:
  • editing/htmlediting.cpp: (WebCore::enclosingListChild): (WebCore::canMergeLists): (WebCore::isVisibilyAdjacent):
  • editing/htmlediting.h:

LayoutTests:

2009-06-27 Ryosuke Niwa <set EMAIL_ADDRESS environment variable>

Reviewed by Eric Seidel.

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

Clean up for IndentOutdentCommand::indentRegion, and solved most of problems related to the bug 21712.
https://bugs.webkit.org/show_bug.cgi?id=21712


The following test cases are intended for 21712, but included here
since this patch already solved a lot of problems.

  • editing/execCommand/indent-nested-lists-1-expected.txt: Added.
  • editing/execCommand/indent-nested-lists-1.html: Added.
  • editing/execCommand/indent-nested-lists-2-expected.txt: Added.
  • editing/execCommand/indent-nested-lists-2.html: Added.
  • editing/execCommand/indent-nested-lists-3-expected.txt: Added.
  • editing/execCommand/indent-nested-lists-3.html: Added.
  • editing/execCommand/indent-nested-lists-4-expected.txt: Added.
  • editing/execCommand/indent-nested-lists-4.html: Added.
  • editing/execCommand/indent-nested-lists-5-expected.txt: Added.
  • editing/execCommand/indent-nested-lists-5.html: Added.
  • editing/execCommand/indent-nested-lists-6-expected.txt: Added.
  • editing/execCommand/indent-nested-lists-6.html: Added.
  • editing/execCommand/indent-nested-lists-7-expected.txt: Added.
  • editing/execCommand/indent-nested-lists-7.html: Added.
  • editing/execCommand/outdent-nested-lists-1-expected.txt: Added.
  • editing/execCommand/outdent-nested-lists-1.html: Added.
  • editing/execCommand/outdent-nested-lists-2-expected.txt: Added.
  • editing/execCommand/outdent-nested-lists-2.html: Added.
  • editing/execCommand/outdent-nested-lists-3-expected.txt: Added.
  • editing/execCommand/outdent-nested-lists-3.html: Added.
  • editing/execCommand/outdent-nested-lists-4-expected.txt: Added.
  • editing/execCommand/outdent-nested-lists-4.html: Added.
19:31 Changeset [45315] by bfulgham@webkit.org

2009-06-27 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

WebInspector: Fix completion when iterating options using Tab.

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

  • inspector/front-end/TextPrompt.js: (WebInspector.TextPrompt.prototype._completionsReady):
17:32 Changeset [45314] by abarth@webkit.org

2009-06-27 Daniel Bates <dbates@intudata.com>

Reviewed by Adam Barth.


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


More tests for the XSSAuditor.

  • http/tests/security/xssAuditor/anchor-url-dom-write-location-expected.txt: Added.
  • http/tests/security/xssAuditor/anchor-url-dom-write-location-inline-event-expected.txt: Added.
  • http/tests/security/xssAuditor/anchor-url-dom-write-location-inline-event.html: Added.
  • http/tests/security/xssAuditor/anchor-url-dom-write-location-javascript-URL-expected.txt: Added.
  • http/tests/security/xssAuditor/anchor-url-dom-write-location-javascript-URL.html: Added.
  • http/tests/security/xssAuditor/anchor-url-dom-write-location.html: Added.
  • http/tests/security/xssAuditor/dom-write-URL-expected.txt: Added.
  • http/tests/security/xssAuditor/dom-write-URL.html: Added.
  • http/tests/security/xssAuditor/dom-write-innerHTML-expected.txt: Added.
  • http/tests/security/xssAuditor/dom-write-innerHTML.html: Added.
  • http/tests/security/xssAuditor/dom-write-location-expected.txt: Added.
  • http/tests/security/xssAuditor/dom-write-location-inline-event-expected.txt: Added.
  • http/tests/security/xssAuditor/dom-write-location-inline-event.html: Added.
  • http/tests/security/xssAuditor/dom-write-location-javascript-URL-expected.txt: Added.
  • http/tests/security/xssAuditor/dom-write-location-javascript-URL.html: Added.
  • http/tests/security/xssAuditor/dom-write-location.html: Added.
  • http/tests/security/xssAuditor/faux-script1-expected.txt: Added.
  • http/tests/security/xssAuditor/faux-script1.html: Added.
  • http/tests/security/xssAuditor/faux-script2-expected.txt: Added.
  • http/tests/security/xssAuditor/faux-script2.html: Added.
  • http/tests/security/xssAuditor/faux-script3-expected.txt: Added.
  • http/tests/security/xssAuditor/faux-script3.html: Added.
  • http/tests/security/xssAuditor/get-from-iframe-expected.txt: Added.
  • http/tests/security/xssAuditor/get-from-iframe.html: Added.
  • http/tests/security/xssAuditor/malformed-HTML-expected.txt: Added.
  • http/tests/security/xssAuditor/malformed-HTML.html: Added.
  • http/tests/security/xssAuditor/post-from-iframe-expected.txt: Added.
  • http/tests/security/xssAuditor/post-from-iframe.html: Added.
  • http/tests/security/xssAuditor/resources/anchor-url-dom-write-location-click.html: Added.
  • http/tests/security/xssAuditor/resources/echo-dom-write-URL.html: Added.
  • http/tests/security/xssAuditor/resources/echo-dom-write-innerHTML.html: Added.
  • http/tests/security/xssAuditor/resources/echo-dom-write-location.html: Added.
  • http/tests/security/xssAuditor/resources/utilities.js: Added. (sendRequestFromIFrame):
14:38 Changeset [45313] by kov@webkit.org

2009-06-27 Gustavo Noronha Silva <gns@gnome.org>

Reviewed by Holger Freyther.

https://bugs.webkit.org/show_bug.cgi?id=25889
[GTK] scrollbar policy for main frame is not implementable

Override visibleContentRect to handle GTK+'s case, in which
scrollbars or equivalent decoration are painted by the parent
widget.

  • platform/ScrollView.cpp:
  • platform/gtk/ScrollViewGtk.cpp: (WebCore::ScrollView::visibleContentRect):
12:36 Changeset [45312] by abarth@webkit.org

WebCore:

2009-06-27 Daniel Bates <dbates@intudata.com>

Reviewed by Adam Barth.

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


Fix addresses false negatives with respect to scheme relative paths, iFrame JavaScript URLs,
and UTF-7 encoded payloads.

Tests: http/tests/security/xssAuditor/http-equiv-utf-7-encoded.html

http/tests/security/xssAuditor/iframe-javascript-url.html
http/tests/security/xssAuditor/script-tag-utf-7-encoded.html
http/tests/security/xssAuditor/script-tag-with-source-relative-scheme.html

  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::scriptHandler): Moved XSSAuditor check to HTMLTokenizer::parseTag. (WebCore::HTMLTokenizer::parseTag):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::loadSubframe): Modified to inform XSSAuditor of parent frame so as to compare against iFrame javascript URL.
  • page/XSSAuditor.cpp: Removed method XSSAuditor::isControlCharacter. Instead, exposed method isControlCharacter in ResourceResponseBase.cpp. (WebCore::XSSAuditor::XSSAuditor): (WebCore::XSSAuditor::decodeURL): Modified to decode string using specified encoder. (WebCore::XSSAuditor::findInRequest): Generalized to arbitrary frame so as to prevent execution of iFrame javascript URL.
  • page/XSSAuditor.h: Added field m_parentFrame.
  • platform/network/ResourceResponseBase.cpp: (WebCore::isControlCharacter):
  • platform/network/ResourceResponseBase.h:

LayoutTests:

2009-06-27 Daniel Bates <dbates@intudata.com>

Reviewed by Adam Barth.

Tests for https://bugs.webkit.org/show_bug.cgi?id=26708

Tests prevention of injected scripts with scheme relative paths, iFrame JavaScript URLs,
and UTF-7 encoding.

  • http/tests/security/xssAuditor/http-equiv-utf-7-encoded-expected.txt: Added.
  • http/tests/security/xssAuditor/http-equiv-utf-7-encoded.html: Added.
  • http/tests/security/xssAuditor/iframe-javascript-url-expected.txt: Added.
  • http/tests/security/xssAuditor/iframe-javascript-url.html: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag-default-encode.pl: Added.
  • http/tests/security/xssAuditor/script-tag-utf-7-encoded-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-utf-7-encoded.html: Added.
  • http/tests/security/xssAuditor/script-tag-with-source-relative-scheme-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-with-source-relative-scheme.html: Added.
01:34 Changeset [45311] by oliver@apple.com

Bug 26771: Canvas is incorrectly tainted when drawing from a video element that uses <source> elements

Reviewed by Maciej Stachowiak.

The drawImage(<video>) logic naively assumes that it just needs
to check the src attribute of the video element when in fact it
needs to look at the url that is being played instead. Failure
to do this means that video provided through source elements
taints the canvas.

Test: media/video-canvas-source.html

00:34 Changeset [45310] by hausmann@webkit.org

2009-06-27 Simon Hausmann <simon.hausmann@nokia.com>

Build fix for Qt under Windows.

  • Api/qwebhistory.h: Use consistent export linkage for the datastream operators.

06/26/09:

23:22 Changeset [45309] by simon.fraser@apple.com

2009-06-26 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dan Bernstein.

<rdar://problem/7011924> Opacity transitions should not trigger hardware compositing mode

Don't go into compositing mode just for opacity transitions, but they will be
hardware accelerated if we're already compositing.

  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::requiresCompositingLayer): (WebCore::RenderLayerCompositor::requiresCompositingForTransform): (WebCore::RenderLayerCompositor::requiresCompositingForAnimation):
  • rendering/RenderLayerCompositor.h:
22:19 Changeset [45308] by ap@webkit.org

Updating Tiger results for a recently modified test.

  • platform/mac-tiger/fast/encoding/idn-security-expected.txt:
22:15 Changeset [45307] by oliver@apple.com

<rdar://problem/7009684> REGRESSION(r45039): Crashes inside JSEvent::put on PowerPC (26746)
<https://bugs.webkit.org/show_bug.cgi?id=26746>

Reviewed by Dan Bernstein

Fix for r45039 incorrectly uncached a get_by_id by converting it to put_by_id. Clearly this
is less than correct. This patch corrects that error.

19:49 Changeset [45306] by bweinstein@apple.com

Mac build fix part 2 - add new mangled symbol

19:40 Changeset [45305] by bweinstein@apple.com

First part of Mac Build Fix - find missing symbol

19:23 Changeset [45304] by bweinstein@apple.com

2009-06-26 Brian Weinstein <bweinstein@apple.com>

Reviewed by Simon Fraser.


Changed call of scrollbarUnderMouse to scrollbarUnderPoint to match new API.

  • Api/qwebpage.cpp: (QWebPage::swallowContextMenuEvent):

2009-06-26 Brian Weinstein <bweinstein@apple.com>

Reviewed by Simon Fraser.

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

Added hit testing on scrollbars, so if you start a gesture over a scrollbar,
it isn't counted, and lets the user drag the scrollbar itself instead of a
panning gesture. Also cleaned up code in gesture.

  • WebView.cpp: (WebView::gestureNotify): (WebView::gesture):

2009-06-26 Brian Weinstein <bweinstein@apple.com>

Reviewed by Simon Fraser.

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


Added the ability to do scrollbar hit testing in EventHandler, changed the
signature of a PlatformWheelEvent constructor, and changed scrollbarUnderMouse
to scrollbarUnderPoint, and updated all calls to that function.

  • page/EventHandler.cpp: (WebCore::EventHandler::hitTestResultAtPoint): (WebCore::EventHandler::handleMousePressEvent): (WebCore::EventHandler::handleMouseMoveEvent):
  • page/EventHandler.h: (WebCore::):
  • platform/PlatformWheelEvent.h:
  • platform/ScrollView.cpp: (WebCore::ScrollView::scrollbarUnderPoint):
  • platform/ScrollView.h:
  • platform/chromium/PopupMenuChromium.cpp: (WebCore::PopupListBox::handleMouseDownEvent): (WebCore::PopupListBox::handleMouseMoveEvent):
  • platform/win/WheelEventWin.cpp: (WebCore::PlatformWheelEvent::PlatformWheelEvent):
19:02 Changeset [45303] by ggaren@apple.com

2009-06-26 Geoffrey Garen <ggaren@apple.com>

Reviewed by Maciej Stachowiak.


Standardized op_call to put { tag, payload } in { regT1, regT0 }.


SunSpider and v8 report no change.

  • jit/JIT.cpp: (JSC::JIT::privateCompileCTIMachineTrampolines):
  • jit/JITCall.cpp: (JSC::JIT::compileOpCallInitializeCallFrame): (JSC::JIT::compileOpCallSetupArgs): (JSC::JIT::compileOpConstructSetupArgs): (JSC::JIT::compileOpCallVarargsSetupArgs): (JSC::JIT::compileOpCallVarargs): (JSC::JIT::compileOpCall): (JSC::JIT::compileOpCallSlowCase):
18:52 Changeset [45302] by abarth@webkit.org

2009-06-26 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Adam Barth.

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

Fix a build break due to LocalStorage.h being included in WebKitDLL.cpp
despite it being deleted from the repo by the following patch/commit:
https://bugs.webkit.org/show_bug.cgi?id=26732
http://trac.webkit.org/changeset/45290

  • WebKitDLL.cpp:
18:49 Changeset [45301] by abarth@webkit.org

2009-06-26 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Make SVN work (again?).

  • Scripts/modules/scm.py:
17:58 Changeset [45300] by simon.fraser@apple.com

2009-06-26 Simon Fraser <simon.fraser@apple.com>

Rubber-stamped by Dave Levin

Rename ioCompState to compositingState to better match WebCore coding style.

  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::computeCompositingRequirements): (WebCore::RenderLayerCompositor::rebuildCompositingLayerTree):
17:50 Changeset [45299] by mitz@apple.com

Reviewed by Mark Rowe.

  • revert unintentional project changes from r45277
  • WebCore.xcodeproj/project.pbxproj:
17:44 Changeset [45298] by mrowe@apple.com

Fix the Windows build.

17:37 Changeset [45297] by simon.fraser@apple.com

2009-06-26 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dan Bernstein.

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

Change to use array indexing rather than Vector enumerators; the former
are preferred style.

  • rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::hasNonCompositingContent):
  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::calculateCompositedBounds): (WebCore::RenderLayerCompositor::computeCompositingRequirements): (WebCore::RenderLayerCompositor::rebuildCompositingLayerTree): (WebCore::RenderLayerCompositor::updateCompositingChildrenGeometry): (WebCore::RenderLayerCompositor::recursiveRepaintLayerRect): (WebCore::RenderLayerCompositor::layerHas3DContent):
17:17 Changeset [45296] by mitz@apple.com

WebCore:

Reviewed by Oliver Hunt.

  • fix <rdar://problem/6961476> REGRESSION (r42043): scrollWidth reported as 1 px

Test: fast/dom/Element/scrollWidth.html

Changed scrollWidth and scrollHeight to use the same logic for
visible overflow boxes that is used for clipped overflow boxes. In
particular, borders are not included and
{leftmost,rightmost,lowest}Position() are used. This logic matches IE8.

  • rendering/RenderBox.cpp: (WebCore::RenderBox::scrollWidth): (WebCore::RenderBox::scrollHeight):

LayoutTests:

Reviewed by Oliver Hunt.

  • fast/dom/Element/scrollWidth-expected.txt: Added.
  • fast/dom/Element/scrollWidth.html: Added.
17:07 Changeset [45295] by cfleizach@apple.com

Bug 26725: aria-hidden, aria-disabled, aria-readonly need to be implemented
https://bugs.webkit.org/show_bug.cgi?id=26725

17:02 Changeset [45294] by beidson@apple.com

2009-06-26 Brady Eidson <beidson@apple.com>

OMG.

  • platform/mac-tiger/fast/loader/file-url-mimetypes-expected.txt: Removed.
  • platform/mac-tiger/platform: Added.
  • platform/mac-tiger/platform/mac: Added.
  • platform/mac-tiger/platform/mac/fast: Added.
  • platform/mac-tiger/platform/mac/fast/loader: Added.
  • platform/mac-tiger/platform/mac/fast/loader/file-url-mimetypes-expected.txt: Copied from platform/mac-tiger/fast/loader/file-url-mimetypes-expected.txt.
16:09 Changeset [45293] by brettw@chromium.org

2009-06-26 Brett Wilson <brettw@chromium.org>

Reviewed by David Levin.

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

GIFImageDecoder is broken.

Make the GIFImageDecoder.repetitionCount function const to match the
base class. The mismatched definitions were causing the function to not
get called.

  • platform/image-decoders/gif/GIFImageDecoder.cpp: (WebCore::GIFImageDecoder::repetitionCount):
  • platform/image-decoders/gif/GIFImageDecoder.h:
15:58 Changeset [45292] by cmarrin@apple.com

Additional fix for https://bugs.webkit.org/show_bug.cgi?id=26651

The flag should always default to true to avoid it getting set
to false in a build with accelerated compositing turned off
and then disabling accelerated compositing when subsequently
running a build with it turned on.

15:53 Changeset [45291] by beidson@apple.com

2009-06-26 Brady Eidson <beidson@apple.com>

Reviewed by Sam Weinig.

Followup for the fix for <rdar://problem/6961578> REGRESSION (r43511): Opening .fdf files from Acrobat Professional fails

Now that other MIME type correction stuff is in our swizzled method, Tiger needs it too!

  • platform/network/mac/ResourceHandleMac.mm: (-[WebCoreResourceHandleAsDelegate connection:didReceiveResponse:]):
  • platform/network/mac/WebCoreURLResponse.h:
15:49 Changeset [45290] by dglazkov@chromium.org

2009-06-26 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Darin Fisher.

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

For the final step of https://bugs.webkit.org/show_bug.cgi?id=25376,
combine LocalStorage and SessionStorage into StorageNamespace. The
synching code (for LocalStorage) has already been removed, so these
classes are now very similar. All they do is essentially contain a
logical grouping of origins that are attached to specific contexts
(be it PageGroups for LocalStorage and Page for SessionStorage).

  • GNUmakefile.am:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:
  • page/Chrome.cpp: (WebCore::Chrome::createWindow):
  • page/DOMWindow.cpp: (WebCore::DOMWindow::localStorage):
  • page/DOMWindow.h:
  • page/Page.cpp: (WebCore::Page::sessionStorage): (WebCore::Page::setSessionStorage):
  • page/Page.h:
  • page/PageGroup.cpp: (WebCore::PageGroup::localStorage):
  • page/PageGroup.h:
  • storage/LocalStorage.cpp: Removed.
  • storage/LocalStorage.h: Removed.
  • storage/LocalStorageTask.cpp:
  • storage/LocalStorageThread.cpp:
  • storage/SessionStorage.cpp: Removed.
  • storage/SessionStorage.h: Removed.
  • storage/StorageArea.cpp: (WebCore::StorageArea::create): (WebCore::StorageArea::StorageArea): (WebCore::StorageArea::copy): (WebCore::StorageArea::length): (WebCore::StorageArea::key): (WebCore::StorageArea::getItem): (WebCore::StorageArea::setItem): (WebCore::StorageArea::removeItem): (WebCore::StorageArea::clear): (WebCore::StorageArea::contains): (WebCore::StorageArea::importItem): (WebCore::StorageArea::close): (WebCore::StorageArea::dispatchStorageEvent):
  • storage/StorageArea.h: (WebCore::):
  • storage/StorageAreaSync.cpp: (WebCore::StorageAreaSync::scheduleFinalSync):
  • storage/StorageNamespace.cpp: Copied from WebCore/storage/LocalStorage.cpp. (WebCore::localStorageNamespaceMap): (WebCore::StorageNamespace::localStorageNamespace): (WebCore::StorageNamespace::sessionStorageNamespace): (WebCore::StorageNamespace::StorageNamespace): (WebCore::StorageNamespace::~StorageNamespace): (WebCore::StorageNamespace::copy): (WebCore::StorageNamespace::storageArea): (WebCore::StorageNamespace::close):
  • storage/StorageNamespace.h: Copied from WebCore/storage/LocalStorage.h.
15:48 Changeset [45289] by beidson@apple.com

2009-06-26 Brady Eidson <beidson@apple.com>

Fix Tiger results for the new mime type layout test.

  • platform/mac-tiger/fast/loader: Added.
  • platform/mac-tiger/fast/loader/file-url-mimetypes-expected.txt: Added.
15:22 Changeset [45288] by japhet@chromium.org

2009-06-26 Nate Chapin <japhet@chromium.org>

Reviewed by David Levin.

Upstream V8Proxy. This involved updating a lot of function and variable names to match WebKit style, hence the large size.

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

15:08 Changeset [45287] by kmccullough@apple.com

2009-06-26 Kevin McCullough <kmccullough@apple.com>

Reviewed by Tim Hatcher.

<rdar://problem/7011047> Profiler shows the record button 'on' even
though it's finished

I consolidated the creation of the user initiated profile name into
its own function and then called it from console::profile instead of
calling startUserInitiatedProfiling(). This way we don't call
toggleRecordButton() which turns on the record button.

  • inspector/InspectorController.cpp: (WebCore::InspectorController::didCommitLoad): (WebCore::InspectorController::getCurrentUserInitiatedProfileName): (WebCore::InspectorController::startUserInitiatedProfiling): (WebCore::InspectorController::stopUserInitiatedProfiling):
  • inspector/InspectorController.h:
  • page/Console.cpp: (WebCore::Console::profile): (WebCore::Console::profileEnd):
15:06 Changeset [45286] by jberlin@apple.com

2009-06-26 Jessie Berlin <jberlin@apple.com>

Reviewed by Mark Rowe.


Fix the build bots.

  • editing/selection/drag-start-event-client-x-y-expected.txt:
14:53 Changeset [45285] by jeremy@chromium.org

Move focusRingColor to RenderTheme

14:51 Changeset [45284] by dimich@chromium.org

2009-06-26 Dmitry Titov <dimich@chromium.org>

Reviewed by David Levin.

https://bugs.webkit.org/show_bug.cgi?id=26761
[Chromium] Enable Dedicated Workers in Chromium.

  • bindings/v8/custom/V8WorkerCustom.cpp: (WebCore::CALLBACK_FUNC_DECL): Remove the check that prevented workers from being created w/o a command-line switch. The flag itself and methods will be removed in a subsequent patch, after corresponding change in Chromium.
14:46 Changeset [45283] by beidson@apple.com

2009-06-26 Brady Eidson <beidson@apple.com>

Step 2 (new Mac results)

  • platform/mac/fast/images/favicon-as-image-expected.checksum: Added.
  • platform/mac/fast/images/favicon-as-image-expected.png: Added.
  • platform/mac/fast/images/favicon-as-image-expected.txt: Added.
14:44 Changeset [45282] by beidson@apple.com

2009-06-26 Brady Eidson <beidson@apple.com>

1st step of fixing the layout test I broke on Mac platforms.

  • platform/mac-leopard/fast/images/favicon-as-image-expected.checksum: Removed.
  • platform/mac-leopard/fast/images/favicon-as-image-expected.png: Removed.
  • platform/mac-leopard/fast/images/favicon-as-image-expected.txt: Removed.
  • platform/mac-tiger/fast/images/favicon-as-image-expected.checksum: Removed.
  • platform/mac-tiger/fast/images/favicon-as-image-expected.png: Removed.
  • platform/mac-tiger/fast/images/favicon-as-image-expected.txt: Removed.
  • platform/mac/fast/images/favicon-as-image-expected.checksum: Removed.
  • platform/mac/fast/images/favicon-as-image-expected.png: Removed.
  • platform/mac/fast/images/favicon-as-image-expected.txt: Removed.
  • platform/win/fast/images: Added.
  • platform/win/fast/images/favicon-as-image-expected.checksum: Copied from platform/mac/fast/images/favicon-as-image-expected.checksum.
  • platform/win/fast/images/favicon-as-image-expected.png: Copied from platform/mac/fast/images/favicon-as-image-expected.png.
  • platform/win/fast/images/favicon-as-image-expected.txt: Copied from platform/mac/fast/images/favicon-as-image-expected.txt.
14:01 Moving to Git edited by treat@kde.org
(diff)
13:51 Changeset [45281] by weinig@apple.com

2009-06-26 Sam Weinig <sam@webkit.org>

Reviewed by Geoffrey Garen.

Handle multiplying by zero a little better by
inlining the case that both operands are non-negative
into the slowpath.

  • assembler/MacroAssemblerX86Common.h: (JSC::MacroAssemblerX86Common::branchOr32):
  • jit/JITArithmetic.cpp: (JSC::JIT::emit_op_mul): (JSC::JIT::emitSlow_op_mul):
13:49 Changeset [45280] by beidson@apple.com

2009-06-26 Brady Eidson <beidson@apple.com>

Fix SnowLeopard build.

  • Plugins/Hosted/HostedNetscapePluginStream.mm: (WebKit::HostedNetscapePluginStream::didReceiveResponse):
13:40 Changeset [45279] by mrowe@apple.com

Land updated results after the test change.

  • editing/selection/user-drag-element-and-user-select-none-expected.txt:
13:29 Changeset [45278] by jberlin@apple.com

2009-06-26 Jessie Berlin <jberlin@apple.com>

Fix: https://bugs.webkit.org/show_bug.cgi?id=26723
Where the m_mouseDown event was never being set on windows, so the
client X and Y coordinates were always being reported as zero in a
dragstart handler.


Reviewed by Mark Rowe.

Test: editing/selection/drag-start-event-client-x-y.html

  • page/EventHandler.cpp: (WebCore::EventHandler::handleMousePressEvent): Set the m_mouseDown event when the mouse press is handled.
  • page/mac/EventHandlerMac.mm: (WebCore::EventHandler::mouseDown): Removed now redundant setting of m_mouseDown.


Layout test for making sure the client X Y coordinates of the event
in a dragstart handler are correct.


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

  • editing/selection/drag-start-event-client-x-y-expected.txt: Added.
  • editing/selection/drag-start-event-client-x-y.html: Added.
13:13 Changeset [45277] by beidson@apple.com

2009-06-26 Brady Eidson <beidson@apple.com>

Tiger build fix

  • WebCore.xcodeproj/project.pbxproj:
  • platform/network/mac/WebCoreURLResponse.mm: (swizzleMIMETypeMethodIfNecessary):
12:43 Changeset [45276] by mrowe@apple.com

Fix user-drag-element-and-user-select-none.html to not rely on a bug in Mac DRT.

Reviewed by Sam Weinig.

  • editing/selection/user-drag-element-and-user-select-none.html:
12:38 Changeset [45275] by ap@webkit.org

Reviewed by Sam Weinig.

https://bugs.webkit.org/show_bug.cgi?id=26681
Problem updating applicationCache when server returns 304


Improve the fix, make the test pass on Tiger.

  • loader/appcache/ApplicationCacheGroup.cpp: (WebCore::ApplicationCacheGroup::didReceiveResponse): Fix another code path to remove the current item from list.
  • platform/network/mac/ResourceHandleMac.mm: (WebCore::ResourceHandle::start): On Tiger, conditional requests that cannot be cached by network layer cause errors with default cache policy.
12:23 Changeset [45274] by beidson@apple.com

WebCore:

2009-06-26 Brady Eidson <beidson@apple.com>

Reviewed by Sam Weinig

<rdar://problem/6961578> REGRESSION (r43511): Opening .fdf files from Acrobat Professional fails

When we disabled content sniffing for file urls we lost knowledge of many file extensions that we
didn't intend to lose. Turns out the CoreTypes UTI database doesn't know about every extension Gatekeeper
knew about.

By comparing CoreTypes' database to Gatekeepers, this patch adds a hardcoded list of file extension to MIME
type mappings that are missing in CoreType's database.

Test: platform/mac/fast/loader/file-url-mimetypes.html

  • platform/network/mac/ResourceHandleMac.mm: (-[WebCoreResourceHandleAsDelegate connection:didReceiveResponse:]): Move the MIME Type swizzling code to WebCoreURLResponse.
  • platform/network/mac/ResourceResponseMac.mm: (WebCore::ResourceResponse::platformLazyInit): _webcore_MIMEType -> MIMEType, as we now have only one place where we do all MIMEType correction.
  • platform/network/mac/WebCoreURLResponse.h: Remove _webcore_MIMEType, as it is now folded into the swizzled implementation of MIMEType.
  • platform/network/mac/WebCoreURLResponse.mm: (createBinaryExtensionsSet): (createExtensionToMIMETypeMap): (swizzleMIMETypeMethodIfNecessary): (webNSURLResponseMIMEType): If it's a file URL and there's no MIME type, see if the extension exists in the extension -> MIME type map before turning to the default MIME type. Also roll in what was previously implemented in _webcore_MIMEType.
  • svg/graphics/SVGImage.cpp: (WebCore::SVGImage::~SVGImage): Tweak this ASSERT - SVGImages might get destroyed without ever having a client.

WebKit/mac:

2009-06-26 Brady Eidson <beidson@apple.com>

Reviewed by Sam Weinig

<rdar://problem/6961578> REGRESSION (r43511): Opening .fdf files from Acrobat Professional fails

Replace all usage of the now obsolete [NSURLResponse _webcore_MIMEType].

  • Plugins/WebBaseNetscapePluginStream.mm: (WebNetscapePluginStream::didReceiveResponse):
  • Plugins/WebNetscapePluginView.mm: (-[WebNetscapePluginView pluginView:receivedData:]):
  • Plugins/WebPluginController.mm: (-[WebPluginController pluginView:receivedResponse:]):
  • WebView/WebDataSource.mm: (-[WebDataSource _responseMIMEType]):
  • WebView/WebResource.mm: (-[WebResource _initWithData:URL:response:]):

WebKitTools:

2009-06-26 Brady Eidson <beidson@apple.com>

Reviewed by Sam Weinig

<rdar://problem/6961578> REGRESSION (r43511): Opening .fdf files from Acrobat Professional fails

Add a dumpResourceResponseMIMETypes() mode so the ResourceLoadDelegate will dump the mime type from
the NSURLResponse. Needed for the test for this bug fix.

  • DumpRenderTree/LayoutTestController.cpp: (LayoutTestController::LayoutTestController): (dumpResourceResponseMIMETypesCallback): (LayoutTestController::staticFunctions):
  • DumpRenderTree/LayoutTestController.h: (LayoutTestController::dumpResourceResponseMIMETypes): (LayoutTestController::setDumpResourceResponseMIMETypes):
  • DumpRenderTree/mac/ResourceLoadDelegate.mm: (-[ResourceLoadDelegate webView:resource:didReceiveResponse:fromDataSource:]):

LayoutTests:

2009-06-25 Brady Eidson <beidson@apple.com>

Reviewed by Sam Weinig

<rdar://problem/6961578> REGRESSION (r43511): Opening .fdf files from Acrobat Professional fails

Add a test file for each of file extensions we think WebKit on Mac should know the MIME type for.

  • platform/mac/fast/loader/file-url-mimetypes-expected.txt: Added.
  • platform/mac/fast/loader/file-url-mimetypes.html: Added.
  • platform/mac/fast/loader/resources: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-3g2.3g2: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-3gp.3gp: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-3gp2.3gp2: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-3gpp.3gpp: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-Z.Z: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-ac3.ac3: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-ai.ai: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-aif.aif: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-aifc.aifc: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-aiff.aiff: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-asc.asc: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-asf.asf: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-asx.asx: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-au.au: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-avi.avi: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-bcpio.bcpio: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-bin.bin: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-bmp.bmp: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-bz.bz: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-bz2.bz2: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-cdf.cdf: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-class.class: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-cpgz.cpgz: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-cpio.cpio: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-cpt.cpt: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-crw.crw: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-csh.csh: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-css.css: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-dcr.dcr: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-dif.dif: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-dir.dir: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-dll.dll: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-dls.dls: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-dmg.dmg: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-dms.dms: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-doc.doc: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-docm.docm: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-docx.docx: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-dotm.dotm: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-dotx.dotx: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-dv.dv: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-dvi.dvi: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-dxr.dxr: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-efx.efx: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-eps.eps: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-etx.etx: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-exe.exe: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-ez.ez: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-fdf.fdf: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-fla.fla: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-fp.fp: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-fp2.fp2: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-fp3.fp3: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-fp4.fp4: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-fp5.fp5: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-fp6.fp6: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-fpx.fpx: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-gif.gif: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-gtar.gtar: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-gz.gz: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-gzip.gzip: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-hdf.hdf: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-hqx.hqx: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-htm.htm: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-html.html: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-ice.ice: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-ico.ico: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-ics.ics: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-ief.ief: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-iges.iges: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-igs.igs: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-iso.iso: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-j2k.j2k: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-jar.jar: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-javascript.javascript: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-jhtml.jhtml: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-jnlp.jnlp: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-jp2.jp2: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-jpe.jpe: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-jpeg.jpeg: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-jpf.jpf: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-jpg.jpg: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-jpx.jpx: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-js.js: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-jscript.jscript: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-kar.kar: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-latex.latex: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-lha.lha: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-lzh.lzh: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-m15.m15: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-m2v.m2v: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-m3u.m3u: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-m4p.m4p: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-m75.m75: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-mac.mac: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-man.man: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-me.me: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-mesh.mesh: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-mid.mid: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-midi.midi: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-mif.mif: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-mov.mov: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-movie.movie: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-mp2.mp2: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-mp3.mp3: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-mp4.mp4: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-mpe.mpe: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-mpeg.mpeg: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-mpg.mpg: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-mpga.mpga: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-ms.ms: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-msh.msh: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-mxu.mxu: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-nc.nc: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-oda.oda: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-odc.odc: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-odf.odf: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-odg.odg: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-odi.odi: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-odm.odm: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-odp.odp: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-ods.ods: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-odt.odt: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-otc.otc: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-otf.otf: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-otg.otg: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-oth.oth: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-oti.oti: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-otp.otp: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-ots.ots: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-ott.ott: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-pbm.pbm: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-pct.pct: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-pcx.pcx: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-pdb.pdb: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-pdf.pdf: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-pgm.pgm: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-pgn.pgn: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-ph3.ph3: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-ph4.ph4: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-php.php: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-php3.php3: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-php4.php4: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-phtml.phtml: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-pic.pic: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-pict.pict: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-pl.pl: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-pls.pls: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-pm.pm: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-png.png: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-pnm.pnm: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-pnt.pnt: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-pntg.pntg: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-potm.potm: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-potx.potx: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-ppm.ppm: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-ppsm.ppsm: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-ppsx.ppsx: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-ppt.ppt: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-pptm.pptm: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-pptx.pptx: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-ps.ps: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-psd.psd: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-py.py: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-qif.qif: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-qt.qt: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-qti.qti: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-qtif.qtif: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-qtz.qtz: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-ra.ra: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-ram.ram: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-ras.ras: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-rb.rb: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-rbw.rbw: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-rgb.rgb: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-rm.rm: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-roff.roff: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-rpm.rpm: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-rtf.rtf: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-rtx.rtx: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-sgi.sgi: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-sgm.sgm: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-sgml.sgml: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-sh.sh: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-shar.shar: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-shtm.shtm: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-shtml.shtml: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-silo.silo: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-sit.sit: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-sitx.sitx: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-skd.skd: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-skm.skm: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-skp.skp: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-skt.skt: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-smf.smf: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-smi.smi: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-smil.smil: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-snd.snd: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-so.so: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-spl.spl: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-src.src: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-sv4cpio.sv4cpio: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-sv4crc.sv4crc: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-svg.svg: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-svgz.svgz: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-swf.swf: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-t.t: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-tar.tar: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-targa.targa: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-tcl.tcl: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-tex.tex: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-texi.texi: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-texinfo.texinfo: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-text.text: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-tga.tga: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-tgz.tgz: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-tif.tif: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-tiff.tiff: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-torrent.torrent: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-tr.tr: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-tsv.tsv: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-txt.txt: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-ustar.ustar: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-vcard.vcard: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-vcd.vcd: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-vcf.vcf: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-vfw.vfw: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-vrml.vrml: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-wav.wav: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-wave.wave: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-wax.wax: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-wbmp.wbmp: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-wbxml.wbxml: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-webarchive.webarchive: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-wm.wm: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-wma.wma: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-wmd.wmd: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-wml.wml: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-wmlc.wmlc: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-wmls.wmls: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-wmlsc.wmlsc: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-wmp.wmp: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-wmv.wmv: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-wmx.wmx: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-wrl.wrl: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-wvx.wvx: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-xbm.xbm: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-xdp.xdp: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-xfd.xfd: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-xfdf.xfdf: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-xht.xht: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-xhtm.xhtm: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-xhtml.xhtml: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-xls.xls: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-xlsb.xlsb: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-xlsm.xlsm: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-xlsx.xlsx: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-xltm.xltm: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-xltx.xltx: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-xml.xml: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-xpm.xpm: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-xsl.xsl: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-xwd.xwd: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-xyz.xyz: Added.
  • platform/mac/fast/loader/resources/mimeTypeExamples/example-zip.zip: Added.
11:28 Changeset [45273] by simon.fraser@apple.com

2009-06-25 Pierre d'Herbemont <pdherbemont@apple.com>

Reviewed by Simon Fraser.

Show the fullscreen button only if the backend has support for it.

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

No test since this is not reachable via the DOM.

  • html/HTMLMediaElement.h: (WebCore::HTMLMediaElement::supportsFullscreen): new
  • html/HTMLVideoElement.h: (WebCore::HTMLVideoElement::supportsFullscreen): new
  • platform/graphics/MediaPlayer.cpp: (WebCore::NullMediaPlayerPrivate::supportsFullscreen): new (WebCore::MediaPlayer::supportsFullscreen): new
  • platform/graphics/MediaPlayer.h: new
  • platform/graphics/MediaPlayerPrivate.h: new (WebCore::MediaPlayerPrivateInterface::supportsFullscreen): new
  • rendering/MediaControlElements.cpp: (WebCore::MediaControlFullscreenButtonElement::rendererIsNeeded): new
  • rendering/MediaControlElements.h:
11:02 Changeset [45272] by simon.fraser@apple.com

2009-06-25 Pierre d'Herbemont <pdherbemont@apple.com>

Reviewed by Simon Fraser.

<rdar://problem/7007776> Controller doesn't automatically update counters when file
is playing ( http://www.jazzguitar.be/mp3/Michael%20Lewis%20-%20SSSJ.mp3 )

Update the time display when the movie time changes.

  • rendering/RenderMedia.cpp: (WebCore::RenderMedia::updateControls):
11:00 Changeset [45271] by simon.fraser@apple.com

2009-06-25 Pierre d'Herbemont <pdherbemont@apple.com>

Reviewed by Simon Fraser.

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

Support hidding a control bar element from the Media element controller.

Update layout tests since the fullscreen button no longer has a renderer.

  • rendering/MediaControlElements.cpp: (WebCore::MediaTextDisplayElement::update): call updateStyle() so everything is updated properly. (WebCore::MediaControlInputElement::MediaControlInputElement): (WebCore::MediaControlInputElement::update): call updateStyle() (WebCore::MediaControlInputElement::updateStyle): create the renderer properly or not depending on what rendererIsNeeded() return.
  • rendering/MediaControlElements.h:
10:48 Changeset [45270] by kmccullough@apple.com

2009-06-26 Kevin McCullough <kmccullough@apple.com>

Reviewed by Oliver Hunt.

<rdar://problem/6968137> Profiler title numbers increment even after a
reload.

  • Now the numbers are reset when the profiles are.
  • inspector/InspectorController.cpp: (WebCore::InspectorController::didCommitLoad):
10:36 Changeset [45269] by eric@webkit.org

2009-06-26 Eric Seidel <eric@webkit.org>

No review, only rolling out r45259.

Roll out r45259 after crash appeared on the bots:
plugins/undefined-property-crash.html
ASSERTION FAILED: s <= HeapConstants<heapType>::cellSize
(leopard-intel-debug-tests/build/JavaScriptCore/runtime/Collector.cpp:278
void* JSC::Heap::heapAllocate(size_t) [with JSC::HeapType heapType = PrimaryHeap])

  • runtime/DateInstance.cpp:
  • runtime/Identifier.cpp:
  • runtime/Lookup.h:
  • runtime/RegExpConstructor.cpp:
  • runtime/RegExpObject.h:
  • runtime/ScopeChain.h:
  • runtime/UString.h:
10:17 Changeset [45268] by ddkilzer@apple.com

Update build-dumprendertree to use buildXCodeProject()

Reviewed by Adam Roben.

  • Scripts/build-dumprendertree: Updated to use buildXCodeProject() when building for isAppleMacWebKit(). This provides additional command-line switch parsing for free. Also added --clean and --help switches. Updated copyright.
10:07 Changeset [45267] by cmarrin@apple.com

got rid of cruft accidentally left in from my last commit

09:53 Changeset [45266] by adele@apple.com

WebCore:

2009-06-26 Adele Peterson <adele@apple.com>

Reviewed by Darin Adler.

Fix for <rdar://problem/7000796>
REGRESSION(34681): Breaking up quoted text makes new, unquoted text blue after certain steps; repros with some messages

Test: editing/inserting/break-blockquote-after-delete.html

Keep track of whether the typing style should be preserved after the TypingCommand is applied. When adding onto an open
typing command, keep that flag up to date.

In this case, an InsertParagraphSeparatorInQuotedContent command, which should not preserve typing style,
was following an open Delete command, which does preserve the typing style. So we were applying the original
typing style (from before the delete, so blue text) to the cursor in the unquoted area after breaking up the blockquote.

  • editing/TypingCommand.cpp: (WebCore::TypingCommand::TypingCommand): (WebCore::TypingCommand::typingAddedToOpenCommand): (WebCore::TypingCommand::insertTextRunWithoutNewlines): (WebCore::TypingCommand::insertLineBreak): (WebCore::TypingCommand::insertParagraphSeparator): (WebCore::TypingCommand::insertParagraphSeparatorInQuotedContent): (WebCore::TypingCommand::deleteKeyPressed): (WebCore::TypingCommand::forwardDeleteKeyPressed): (WebCore::TypingCommand::deleteSelection): (WebCore::TypingCommand::updatePreservesTypingStyle):
  • editing/TypingCommand.h: (WebCore::TypingCommand::preservesTypingStyle):

LayoutTests:

2009-06-26 Adele Peterson <adele@apple.com>

Reviewed by Darin Adler.

Test for <rdar://problem/7000796>
REGRESSION(34681): Breaking up quoted text makes new, unquoted text blue after certain steps; repros with some messages

  • editing/inserting/break-blockquote-after-delete.html: Added.
  • platform/mac/editing/inserting/break-blockquote-after-delete-expected.checksum: Added.
  • platform/mac/editing/inserting/break-blockquote-after-delete-expected.png: Added.
  • platform/mac/editing/inserting/break-blockquote-after-delete-expected.txt: Added.
09:30 Changeset [45265] by hausmann@webkit.org

2009-06-26 Jedrzej Nowacki <jedrzej.nowacki@nokia.com>

Reviewed by Simon Hausmann.

Add support for saving and loading of QWebHistory to and from a QByteArray.

This includes streaming operators for QWebHistory. for convenience.

New autotests that test QWebHistory and QWebHistoryItem serialization.

09:29 Changeset [45264] by hausmann@webkit.org

2009-06-26 Jedrzej Nowacki <jedrzej.nowacki@nokia.com>

Reviewed by Simon Hausmann.

Fix the behaviour of QWebHistory::itemAt to interpret the specified index as absolute index.

Returns an invalid QWebHistoryItem if the index is out of range.

09:29 Changeset [45263] by hausmann@webkit.org

2009-06-26 Jedrzej Nowacki <jedrzej.nowacki@nokia.com>

Reviewed by Simon Hausmann.

Added a few autotest to QWebHistory.

09:29 Changeset [45262] by hausmann@webkit.org

JavaScriptCore:

2009-06-26 Jedrzej Nowacki <jedrzej.nowacki@nokia.com>

Reviewed by Simon Hausmann.

Add support for QDataStream operators to Vector.

WebCore:

2009-06-26 Jedrzej Nowacki <jedrzej.nowacki@nokia.com>

Reviewed by Simon Hausmann.

Add support for QDataStream operators to String and IntPoint.

09:27 Changeset [45261] by darin@chromium.org

2009-06-26 Ben Murdoch <benm@google.com>

Reviewed by Darin Fisher.

Add #if ENABLE(DOM_STORAGE) to the V8 custom bindings for local/session storage.
https://bugs.webkit.org/show_bug.cgi?id=26757

  • bindings/v8/custom/V8StorageCustom.cpp
09:27 Changeset [45260] by hausmann@webkit.org

2009-06-26 Jedrzej Nowacki <jedrzej.nowacki@nokia.com>

Reviewed by Simon Hausmann.

Fix support for documenting functions prefixed with QWEBKIT_EXPORT

Add QWEBKIT_EXPORT to the list of macros to ignore by qdoc.

03:27 Changeset [45259] by eric@webkit.org

2009-06-26 Zoltan Horvath <hzoltan@inf.u-szeged.hu>

Reviewed by Eric Seidel.

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

Allow custom memory allocation control for JavaScriptCore/runtime
directory's classes which instantiated by new:

class IdentifierTable
struct RegExpObjectData
class HashEntry
struct DataInstance::Cache
class UString
struct RegExpConstructorPrivate
class ScopeChain

  • runtime/DateInstance.cpp:
  • runtime/Identifier.cpp:
  • runtime/Lookup.h:
  • runtime/RegExpConstructor.cpp:
  • runtime/RegExpObject.h:
  • runtime/ScopeChain.h:
  • runtime/UString.h:
03:07 Changeset [45258] by eric@webkit.org

2009-06-26 Eric Seidel <eric@webkit.org>

Reviewed by Tor Arne Vestbø.

Remove non-sense --update option to land-patches
and make land-patches update before every patch application.
This makes it slightly less likely that multi-patch landings will fail.

Also updated git diff command to include staged modifications.

  • Scripts/bugzilla-tool:
  • Scripts/modules/scm.py:
03:06 Changeset [45257] by eric@webkit.org

2009-06-26 Yongjun Zhang <yongjun.zhang@nokia.com>

Reviewed by Eric Seidel.

Bug 20303: [Qt] Key events are not working in frames.

Add a layout test to test the event is sent to the right sub-frame.

  • platform/qt/fast/events/event-sender-keydown-frame-expected.txt: Added.
  • platform/qt/fast/events/event-sender-keydown-frame.html: Added.

2009-06-26 Yongjun Zhang <yongjun.zhang@nokia.com>

Reviewed by Eric Seidel.

Test: platform/qt/fast/events/event-sender-keydown-frame.html

Bug 20303: [Qt] Key events are not working in frames.

Merge scrolling handling code in qt and win port, move it to
EventHandler.

  • page/EventHandler.cpp: (WebCore::EventHandler::scrollRecursively):
  • page/EventHandler.h:

2009-06-26 Yongjun Zhang <yongjun.zhang@nokia.com>

Reviewed by Eric Seidel.

Bug 20303: [Qt] Key events are not working in frames.

Send scrolling events to current focused frame, bubble the event
up to parent frame if it is not handled. Use EventHandler's new
shared scrolling code.

  • Api/qwebpage.cpp: (QWebPagePrivate::keyPressEvent): (QWebPagePrivate::handleScrolling):
  • Api/qwebpage_p.h:

2009-06-26 Yongjun Zhang <yongjun.zhang@nokia.com>

Reviewed by Eric Seidel.

Bug 20303: [Qt] Key events are not working in frames.

Move the scroll handling code to EventHandler so that other
ports can share the functionality.

  • WebView.cpp: (WebView::keyDown): (EnumTextMatches::QueryInterface):
02:42 Changeset [45256] by rwlbuis@webkit.org

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=26682
Bug 26682: It should be possible to add image to SVG DOM programmatically (using JavaScript)

Make sure the xlink:href animated property setting syncs the corresponding attribute with the right namespace.

Test: svg/custom/createImageElement.svg

02:36 Changeset [45255] by eric@webkit.org

2009-06-26 Takeshi Yoshino <tyoshino@google.com>

Reviewed by Timothy Hatcher.

Bug 26156: In view-source mode, always render the contents using HTMLViewSourceDocument
https://bugs.webkit.org/show_bug.cgi?id=26156

When in view-source mode, render the contents using HTMLViewSourceDocument
regardless it's applicable for any plugin or not.

Chromium tells WebCore to render the contents of specified URL when
view-source: prefix is added to the URL. But currently, DOMImplementation
ignores inViewSourceMode() when the MIME type is indicating that the contents
are neither texts nor HTML family documents.

For example, we can check the contents of asf file without launching media
player. Rendering contents for view-source:-specified input is not what user
expects.

http://code.google.com/p/chromium/issues/detail?id=10545

I want to fix this issue by this patch. IMHO, regardless of this Chromium
specific issue, I think we should force use of HTMLViewSourceDocument when
inViewSourceMode() is specified.

  • fast/frames/resources/viewsource-fake-image-file.png: Added.
  • fast/frames/viewsource-on-image-file.html: Added.
  • platform/mac/fast/frames/viewsource-on-image-file-expected.txt: Added.

2009-06-26 Takeshi Yoshino <tyoshino@google.com>

Reviewed by Timothy Hatcher.

Bug 26156: In view-source mode, always render the contents using HTMLViewSourceDocument
https://bugs.webkit.org/show_bug.cgi?id=26156

When in view-source mode, render the contents using HTMLViewSourceDocument
regardless it's applicable for any plugin or not.

Chromium tells WebCore to render the contents of specified URL when
view-source: prefix is added to the URL. But currently, DOMImplementation
ignores inViewSourceMode() when the MIME type is indicating that the contents
are neither texts nor HTML family documents.

For example, we can check the contents of asf file without launching media
player. Rendering contents for view-source:-specified input is not what user
expects.

http://code.google.com/p/chromium/issues/detail?id=10545

I want to fix this issue by this patch. IMHO, regardless of this Chromium
specific issue, I think we should force use of HTMLViewSourceDocument when
inViewSourceMode() is specified.

Test: fast/frames/viewsource-on-image-file.html

  • dom/DOMImplementation.cpp: (WebCore::DOMImplementation::createDocument):
  • html/HTMLViewSourceDocument.cpp: (WebCore::HTMLViewSourceDocument::createTokenizer):
  • html/HTMLViewSourceDocument.h:
02:25 Changeset [45254] by ap@webkit.org

Reviewed by Sam Weinig.

<rdar://problem/6651201> Update lookalike character list.

  • Misc/WebNSURLExtras.mm: (isLookalikeCharacter): Added more characters to the list.
02:20 Changeset [45253] by eric@webkit.org

2009-06-26 Jungshik Shin <jshin@chromium.org>

Reviewed by Eric Seidel.

Add an additional test for bug 23786 (mistreatment of CR/LF
as zero-width characters in Chromium port). Tests using Bidi text were
added when the fix was checked in. Without the patch, Chromium used
to fail the same way with LTR complex scripts as well. (that is,
CR and LF were treated as zero-width space). To prevent a potential
regression in the future, a new test with Hindi (a LTR complex script)
is added.

The expected results for Mac and Windows are added while this test
is added to Skipped file for gtk. I guess this test just needs
a new test result generated on gtk.

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

  • fast/text/international/hindi-whitespace.html: Added.
  • platform/gtk/Skipped:
  • platform/mac/fast/text/international/hindi-whitespace-expected.checksum: Added.
  • platform/mac/fast/text/international/hindi-whitespace-expected.png: Added.
  • platform/mac/fast/text/international/hindi-whitespace-expected.txt: Added.
  • platform/win/fast/text/international/hindi-whitespace-expected.checksum: Added.
  • platform/win/fast/text/international/hindi-whitespace-expected.png: Added.
  • platform/win/fast/text/international/hindi-whitespace-expected.txt: Added.
02:12 Changeset [45252] by xan@webkit.org

2009-06-26 Xan Lopez <xlopez@igalia.com>

Reviewed by Jan Alonzo.

https://bugs.webkit.org/show_bug.cgi?id=25529
[Gtk] Expected states not exposed to assistive technologies

Add support for VISIBLE, EDITABLE and SENSITIVE states.

  • accessibility/gtk/AccessibilityObjectWrapperAtk.cpp: (setAtkStateSetFromCoreObject):
01:40 Changeset [45251] by eric@webkit.org

2009-06-26 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Darin Adler.

"Pointer to incomplete class type is not allowed" error with RVCT
https://bugs.webkit.org/show_bug.cgi?id=26721

Based on Norbert Leser's work.

  • dom/Document.cpp: (WebCore::Document::setFocusedNode):
  • dom/Node.cpp: (WebCore::Node::dispatchMouseEvent):
  • dom/Node.h: Remove the default value for PassRefPtr<Event> args, to eliminate dependency on the Event class definition
01:32 Changeset [45250] by eric@webkit.org

2009-06-25 Eric Seidel <eric@webkit.org>

Reviewed by Jan Alonzo.

Fix obsolete_attachment to work when passed a comment.
https://bugs.webkit.org/show_bug.cgi?id=26745

  • Scripts/modules/bugzilla.py:
01:28 Changeset [45249] by eric@webkit.org

2009-06-25 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Mark Rowe.

bugzilla-tool apply-patch throws exception in Linux
https://bugs.webkit.org/show_bug.cgi?id=26738

HEAD is case-sensitive in Linux. Convert uses of head to HEAD in
the scm module.

  • Scripts/modules/scm.py:
00:58 Changeset [45248] by eric@webkit.org

2009-06-26 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Maciej Stachowiak.

[Qt] Build fix after r45183
https://bugs.webkit.org/show_bug.cgi?id=26748

  • WebCore.pro:

06/25/09:

21:44 Changeset [45247] by ap@webkit.org

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=26681
<rdar://problem/7003461> Problem updating applicationCache when server returns 304

Test: http/tests/appcache/update-cache.html

  • loader/appcache/ApplicationCacheGroup.cpp: (WebCore::ApplicationCacheGroup::didReceiveResponse): We're already done with the resource, don't try to load it again.
21:21 Changeset [45246] by barraclough@apple.com

Fix up JavaScriptCore/ChangeLog, messed up slightly in the merge.

20:32 Changeset [45245] by simon.fraser@apple.com

2009-06-25 Simon Fraser <simon.fraser@apple.com>

Reviewed by Oliver Hunt.

<rdar://problem/6990481>

Handle perspective computation on non-layer objects.

Test: transforms/3d/general/perspective-non-layer.html

  • rendering/RenderObject.cpp: (WebCore::RenderObject::getTransformFromContainer):
20:03 Changeset [45244] by barraclough@apple.com

Merged r45138:45243 into nitro-extreme branch.

19:56 Changeset [45243] by barraclough@apple.com

Merged r45137:45138 into nitro-extreme branch.

19:53 Changeset [45242] by barraclough@apple.com

Merged r45039:45137 into nitro-extreme branch.

19:47 Changeset [45241] by barraclough@apple.com

Merged r45038:45039 into nitro-extreme branch.

19:35 Changeset [45240] by barraclough@apple.com

Merged r45000:45038 into nitro-extreme branch.

19:32 Changeset [45239] by eric@webkit.org

2009-06-25 Ryosuke Niwa <rniwa@google.com>

Reviewed by Eric Seidel.

The following pixel tests have been converted into dumpastext tests. This
fixes the bug 26336 partially.

  • editing/execCommand/remove-list-1-expected.txt: Added.
  • editing/execCommand/remove-list-1.html:
  • editing/execCommand/remove-list-items-expected.txt: Added.
  • editing/execCommand/remove-list-items.html:
  • platform/mac/editing/execCommand/remove-list-1-expected.checksum: Removed.
  • platform/mac/editing/execCommand/remove-list-1-expected.png: Removed.
  • platform/mac/editing/execCommand/remove-list-1-expected.txt: Removed.
  • platform/mac/editing/execCommand/remove-list-items-expected.checksum: Removed.
  • platform/mac/editing/execCommand/remove-list-items-expected.png: Removed.
  • platform/mac/editing/execCommand/remove-list-items-expected.txt: Removed.
  • platform/qt/editing/execCommand/remove-list-1-expected.txt: Removed.
  • platform/qt/editing/execCommand/remove-list-items-expected.txt: Removed.
19:31 Changeset [45238] by barraclough@apple.com

Merged r44960:45000 into nitro-extreme branch.

19:20 Changeset [45237] by barraclough@apple.com

Merged r44943:44960 into nitro-extreme branch.

19:19 Changeset [45236] by simon.fraser@apple.com

2009-06-25 Simon Fraser <simon.fraser@apple.com>

Rubber-stamped by Eric Seidel

Update this test after the video controller changes.

  • platform/mac/fast/layers/video-layer-expected.txt:
19:13 Changeset [45235] by barraclough@apple.com

Merged r44942:44943 into nitro-extreme branch.

19:12 Changeset [45234] by jmalonzo@webkit.org

2009-06-25 Jan Michael Alonzo <jmalonzo@webkit.org>

Not reviewed. Skip another failing test due to lack of eventSender impl.

  • platform/gtk/Skipped:
18:42 Changeset [45233] by barraclough@apple.com

Build fixes, mostly files not added.

18:38 Changeset [45232] by jmalonzo@webkit.org

2009-06-25 Jan Michael Alonzo <jmalonzo@webkit.org>

Not reviewed. Skip failing gtk tests due to missing DRT features.

  • platform/gtk/Skipped:
18:28 Changeset [45231] by barraclough@apple.com

Build fixes, mostly files not added.

18:16 Changeset [45230] by simon.fraser@apple.com

2009-06-25 Simon Fraser <simon.fraser@apple.com>

Rubber-stamped by Dan Bernstein.

Add Windows-specific test results, since Windows does not have
3d transforms enabled.

  • platform/win/fast/media/mq-transform-02-expected.txt: Copied from LayoutTests/platform/mac-leopard/fast/media/mq-transform-02-expected.txt.
  • platform/win/fast/media/mq-transform-03-expected.txt: Copied from LayoutTests/platform/mac-leopard/fast/media/mq-transform-03-expected.txt.
18:08 Changeset [45229] by barraclough@apple.com

Build fix

18:06 Changeset [45228] by simon.fraser@apple.com

2009-06-25 Pierre d'Herbemont <pdherbemont@apple.com>

Reviewed by Eric Seidel.

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

Use flex box in the mediaControls.css style, to nicely scale if a button gets
dynamically added or removed.

Media tests results are affected by this changes.

  • css/mediaControls.css:
  • css/mediaControlsQT.css:
18:03 Changeset [45227] by simon.fraser@apple.com

2009-06-25 Simon Fraser <simon.fraser@apple.com>

Rubber-stamped by Mark Rowe.

<rdar://problem/6999737>

Update the media controller images.

  • libWebKitSystemInterfaceLeopard.a:
  • libWebKitSystemInterfaceSnowLeopard.a:
  • libWebKitSystemInterfaceTiger.a:
17:58 Changeset [45226] by barraclough@apple.com

Merged r44920:44942 into nitro-extreme branch.

17:50 Changeset [45225] by barraclough@apple.com

Merged r44889:44920 into nitro-extreme branch.

17:49 Changeset [45224] by eric@webkit.org

2009-06-25 Ryosuke Niwa <rniwa@google.com>

Reviewed by Eric Seidel.

The following pixel tests have been converted into dumpastext tests. This
fixes the bug 26336 partially.

  • editing/execCommand/5432254-2-expected.txt: Added.
  • editing/execCommand/5432254-2.html:
  • editing/execCommand/5700414-1-expected.txt: Added.
  • editing/execCommand/5700414-1.html:
  • editing/execCommand/5700414-2-expected.txt: Added.
  • editing/execCommand/5700414-2.html:
  • platform/mac/editing/execCommand/5432254-2-expected.checksum: Removed.
  • platform/mac/editing/execCommand/5432254-2-expected.png: Removed.
  • platform/mac/editing/execCommand/5432254-2-expected.txt: Removed.
  • platform/mac/editing/execCommand/5700414-1-expected.checksum: Removed.
  • platform/mac/editing/execCommand/5700414-1-expected.png: Removed.
  • platform/mac/editing/execCommand/5700414-1-expected.txt: Removed.
  • platform/mac/editing/execCommand/5700414-2-expected.checksum: Removed.
  • platform/mac/editing/execCommand/5700414-2-expected.png: Removed.
  • platform/mac/editing/execCommand/5700414-2-expected.txt: Removed.
  • platform/qt/editing/execCommand/5432254-2-expected.txt: Removed.
17:34 Changeset [45223] by barraclough@apple.com

Merged r44888:44889 into nitro-extreme branch.

17:28 Changeset [45222] by barraclough@apple.com

Merged r44884:44888 into nitro-extreme branch.

17:26 Changeset [45221] by beidson@apple.com

2009-06-24 Brady Eidson <beidson@apple.com>

Reviewed by Mark Rowe

  • demos/index.html: Fix more typos etc.
17:24 Changeset [45220] by eric@webkit.org

2009-06-25 Jakub Wieczorek <faw217@gmail.com>

Reviewed by Adam Treat.

Add highlight functionality to the QWebPage::findText() method. Introduced is
new HighlightAllOccurrences flag which passed to the function will make it mark
all existing occurrences of specified string in the page.

  • Api/qwebpage.cpp: (QWebPage::findText):
  • Api/qwebpage.h:
  • Api/qwebview.cpp:
17:22 Changeset [45219] by jmalonzo@webkit.org

2009-06-25 Duy Nguyen <pclouds@gmail.com>

Rubber-stamped by Jan Alonzo.

Gtk port Vietnamese translation
https://bugs.webkit.org/show_bug.cgi?id=26739

  • vi.po: Added.
17:22 Changeset [45218] by barraclough@apple.com

Merged r44883:44884 into nitro-extreme branch.

17:16 Changeset [45217] by barraclough@apple.com

Merged r44844:44883 into nitro-extreme branch.

17:07 Changeset [45216] by barraclough@apple.com

Merged r44843:44844 into nitro-extreme branch.

16:58 Changeset [45215] by darin@chromium.org

2009-06-25 Albert J. Wong <ajwong@chromium.org>

Reviewed by Darin Fisher.

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

Move relavent part of setDefaultFontSize from RenderThemeChromiumWin
up into RenderThemeChromiumSkia.

  • rendering/RenderThemeChromiumSkia.cpp: (WebCore::RenderThemeChromiumSkia::setDefaultFontSize):
  • rendering/RenderThemeChromiumSkia.h:
  • rendering/RenderThemeChromiumWin.cpp: (WebCore::RenderThemeChromiumWin::setDefaultFontSize):
  • rendering/RenderThemeChromiumWin.h:
16:54 Changeset [45214] by andersca@apple.com

2009-06-25 Anders Carlsson <andersca@apple.com>

Reviewed by Oliver Hunt.

<rdar://problem/6989017>
REGRESSION (SnowLeopard): RealPlayer content replays when opening a new tab or switching back to the RealPlayer tab


If a plug-in fails in NPP_New, we would try to recreate it whenever the preferences for a web view would change.


Fix this by setting a flag when we fail to instantiate the plug-in, so we only try once.


  • Plugins/WebBaseNetscapePluginView.h:
  • Plugins/WebBaseNetscapePluginView.mm: (-[WebBaseNetscapePluginView start]):
16:49 Changeset [45213] by barraclough@apple.com

Merged r44838:44843 into nitro-extreme branch.

16:45 Changeset [45212] by ggaren@apple.com

2009-06-25 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.


Optimize x++ to ++x inside for loops.


Sadly, no measurable speedup, but this should help with result chaining.

  • parser/Nodes.cpp: (JSC::ForNode::emitBytecode):
16:39 Changeset [45211] by barraclough@apple.com

Merged r44837:44838 into nitro-extreme branch.

16:36 Changeset [45210] by jmalonzo@webkit.org

2009-06-25 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Mark Rowe.

bugzilla-tool apply-patch throws exception in Linux
https://bugs.webkit.org/show_bug.cgi?id=26738

HEAD is case-sensitive in Linux. Convert uses of head to HEAD in
the scm module.

  • Scripts/modules/scm.py:
16:24 Changeset [45209] by barraclough@apple.com

Merged r44796:44837 into nitro-extreme branch.

16:21 Changeset [45208] by darin@chromium.org

2009-06-25 Matt Perry <mpcomplete@chromium.org>

Reviewed by Darin Fisher.

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

Add V8-only methods to FrameLoaderClient that V8 can use to send
out notifications when it creates/destroys a script context.

  • loader/EmptyClients.h: (WebCore::EmptyFrameLoaderClient::didCreateScriptContext): (WebCore::EmptyFrameLoaderClient::didDestroyScriptContext):
  • loader/FrameLoaderClient.h: (WebCore::FrameLoaderClient::didCreateScriptContext): (WebCore::FrameLoaderClient::didDestroyScriptContext):
16:11 Changeset [45207] by dglazkov@chromium.org

2009-06-25 Dimitri Glazkov <dglazkov@chromium.org>

Reviewed by David Levin.

Correct a few typos that snuck in when I was reformatting CodeGeneratorV8.pm
to match WebKit style.

  • bindings/scripts/CodeGeneratorV8.pm: Corrected lots of typos.
15:42 Changeset [45206] by barraclough@apple.com

Build fix, revert part of last build fix.

15:37 Changeset [45205] by ggaren@apple.com

2009-06-25 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.


Standardized some more opcodes to put { tag, payload } in { regT1, regT0 }.

  • jit/JITArithmetic.cpp: (JSC::JIT::emitSlow_op_bitnot): (JSC::JIT::emit_op_post_inc):
15:35 Changeset [45204] by cmarrin@apple.com

Some cosmetic fixes I missed in my last checkin

15:32 Changeset [45203] by agl@chromium.org

2009-06-25 Adam Langley <agl@google.com>

TBRed: fix for Chromium tree.

Add missing include for r45199.

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

r45199 added a reference to throwError without including V8Proxy.h

  • bindings/v8/WorkerScriptController.cpp:
15:16 Changeset [45202] by barraclough@apple.com

Build fixes, mostly files not added.

15:13 Changeset [45201] by agl@chromium.org

2009-06-25 Adam Langley <agl@google.com>

TBRed: fix for Chromium tree.

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

Fix V8IsolatedWorld to point to the correct include file.

The deprecated v8_index.h was removed from the Chromium tree in r19291
and upstreamed into WebKit with r45193. However V8IsolatedWorld
slipped in between the cracks and broke the build.

  • bindings/v8/V8IsolatedWorld.h: update with new header location.
15:09 Changeset [45200] by ggaren@apple.com

2009-06-25 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.


Standardized some more opcodes to put { tag, payload } in { regT1, regT0 }.

  • jit/JITArithmetic.cpp: (JSC::JIT::emit_op_bitnot): (JSC::JIT::emit_op_post_dec): (JSC::JIT::emit_op_pre_inc): (JSC::JIT::emitSlow_op_pre_inc): (JSC::JIT::emit_op_pre_dec): (JSC::JIT::emitSlow_op_pre_dec):
15:03 Changeset [45199] by cmarrin@apple.com

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


Preference is named "WebKitAcceleratedCompositingEnabled"
and is a boolean value. When false, prevents compositing layers from
being created, which prevents hardware animation from running.
Also forces video to do software rendering. Added a cache for
the flag in RenderLayerCompositing and made it all work
on-the-fly when the flag is changed while a page is loaded.

14:54 Changeset [45198] by jianli@chromium.org

2009-06-25 Jian Li <jianli@chromium.org>

Reviewed by Dimitri Glazkov.

Bug 26701: Implement the missing code for "FIXME: Need to return an
exception" in WorkerScriptController::evaluate for v8 bindings.
https://bugs.webkit.org/show_bug.cgi?id=26701

  • bindings/v8/WorkerScriptController.cpp: (WebCore::WorkerScriptController::evaluate):
14:52 Changeset [45197] by eric@webkit.org

2009-06-25 Eric Seidel <eric@webkit.org>

Reviewed by Mark Rowe.

Call WebKitTools/Scripts scripts by their absolute paths
https://bugs.webkit.org/show_bug.cgi?id=26704

bugzilla-tool used to use whatever build-webkit was in your path.
That could end up building the wrong copy of WebKit.

  • Scripts/bugzilla-tool:
  • Scripts/modules/scm.py:
14:51 Changeset [45196] by sfalken@apple.com

New tag.

14:44 Changeset [45195] by sfalken@apple.com

2009-06-24 Mark Rowe <mrowe@apple.com>

Merge r44117.

2009-05-24 Steve Falkenburg <sfalken@apple.com>

Build fix for experimental PGO Windows target.


14:42 Changeset [45194] by ggaren@apple.com

2009-06-25 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.


Standardized some more opcodes to put { tag, payload } in { regT1, regT0 }.

  • jit/JITArithmetic.cpp: (JSC::JIT::emit_op_negate): (JSC::JIT::emitSlow_op_negate):
  • jit/JITCall.cpp: (JSC::JIT::emit_op_construct_verify): (JSC::JIT::emitSlow_op_construct_verify):
14:39 Changeset [45193] by japhet@chromium.org

2009-06-25 Nate Chapin <japhet@chromium.org>

Reviewed by Dimitri Glazkov.

Upstream V8Index.

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

  • bindings/v8/V8Index.cpp: Added. (WebCore::V8ClassIndex::GetFactory): Moved from src.chromium.org. (WebCore::V8ClassIndex::GetCache): Moved from src.chromium.org.
  • bindings/v8/V8Index.h: (WebCore::V8ClassIndex::): Moved from src.chromium.org. (WebCore::V8ClassIndex::ToInt): Moved from src.chromium.org. (WebCore::V8ClassIndex::FromInt): Moved from src.chromium.org.
14:26 Changeset [45192] by eric@webkit.org

2009-06-25 Tor Arne Vestbø <tor.arne.vestbo@nokia.com>

Reviewed by Dave Levin.

Use unicode() instead of str when reading from bugzilla.
Also add Simon Fraser as a reviewer.
https://bugs.webkit.org/show_bug.cgi?id=26719

  • Scripts/modules/bugzilla.py:
14:15 Changeset [45191] by agl@chromium.org

2009-06-25 Adam Langley <agl@google.com>

Reviewed by Darin Fisher.

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

This is hopefully the last step before our renderers can run
cleanly in a chroot.

WebKit needs to be able to ask for the correct font to use in
the case that the current font doesn't include glyphs for
certain code points. Currently we make a fontconfig call in our
WebKit port to handle this.

This patch changes this so that the call is sent our via
ChromiumBridge.

http://codereview.chromium.org/132007

This should not affect any layout tests.

  • platform/chromium/ChromiumBridge.h:
  • platform/graphics/chromium/FontCacheLinux.cpp: (WebCore::FontCache::getFontDataForCharacters):
14:02 Changeset [45190] by ggaren@apple.com

2009-06-25 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.


Standardized some more opcodes to put { tag, payload } in { regT1, regT0 }.

  • jit/JITOpcodes.cpp: (JSC::JIT::emit_op_loop_if_true): (JSC::JIT::emit_op_jfalse): (JSC::JIT::emit_op_jtrue): (JSC::JIT::emit_op_jeq_null): (JSC::JIT::emit_op_jneq_null): (JSC::JIT::emit_op_eq_null): (JSC::JIT::emit_op_neq_null):
13:34 Changeset [45189] by levin@chromium.org

2009-06-25 Albert J. Wong <ajwong@chromium.org>

Reviewed by David Levin.

https://bugs.webkit.org/show_bug.cgi?id=26566
Upstream these files from the chromium v8 code. No tests were
affected because this is essentially a code move.

  • bindings/v8/NPV8Object.cpp: Added. (allocV8NPObject): (freeV8NPObject): (listFromVariantArgs): (npIdentifierToV8Identifier): (npCreateV8ScriptObject): (NPN_Invoke): (NPN_InvokeDefault): (NPN_Evaluate): (NPN_EvaluateHelper): (NPN_GetProperty): (NPN_SetProperty): (NPN_RemoveProperty): (NPN_HasProperty): (NPN_HasMethod): (NPN_SetException): (NPN_Enumerate): (NPN_Construct):
  • bindings/v8/NPV8Object.h: Added. (PrivateIdentifier::):
  • bindings/v8/V8NPUtils.cpp: Added. (convertV8ObjectToNPVariant): (convertNPVariantToV8Object): (getStringIdentifier):
  • bindings/v8/V8NPUtils.h: Added.
13:34 Changeset [45188] by levin@chromium.org

WebCore:

2009-06-25 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by Dimitri Glazkov.

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

Windows Chromium bug fix: save context of destination canvas in
TransparencyWin::compositeTextComposite() before the function
modifies the context.

Test: fast/canvas/translate-text.html

  • platform/graphics/chromium/TransparencyWin.cpp: (WebCore::TransparencyWin::compositeTextComposite):

LayoutTests:

2009-06-25 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by Dimitri Glazkov.

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

Windows Chromium bug fix: save context of destination canvas in
TransparencyWin::compositeTextComposite() before the function
modifies the context.

  • fast/canvas/resources/translate-text.js: Added. (description.pixelValueAt): (pixelToString): (pixelShouldBe):
  • fast/canvas/translate-text-expected.txt: Added.
  • fast/canvas/translate-text.html: Added.
13:34 Changeset [45187] by levin@chromium.org

2009-06-25 Takeshi Yoshino <tyoshino@google.com>

Reviewed by David Levin.

Bug 26207: Rebaseline the expected image for fast/forms/select-style.html for Mac.
https://bugs.webkit.org/show_bug.cgi?id=26207

Replace the expected image for fast/forms/select-style-expected.html for
Mac which need to be rebaselined due to r43903
(http://trac.webkit.org/changeset/43903).

  • platform/mac/fast/forms/select-style-expected.png:
13:34 Changeset [45186] by levin@chromium.org

2009-06-25 Patrick Mueller <Patrick_Mueller@us.ibm.com>

Reviewed by Timothy Hatcher.

Show the filename and first line for "(program)" in the Profiler/Debugger
https://bugs.webkit.org/show_bug.cgi?id=25475

Add support to associate a sourceURL with an eval()'d string
via a @sourceURL comment. Currently the sourceURL is only available
in the script debugger, not in the console or profiler, but it's
most needed in the script debugger.

  • English.lproj/localizedStrings.js: added new "(program): %s" string
  • inspector/front-end/Script.js: (WebInspector.Script): if no sourceURL is available for the Script, search for a comment of the form @ sourceURL=(url) to use as the sourceURL instead.
  • manual-tests/inspector/named-evals.html: Added.
13:34 Changeset [45185] by levin@chromium.org

WebCore:

2009-06-25 John Gregg <johnnyg@google.com>

Reviewed by Sam Weinig.

Bug 23721: Changing dropdown's selectedIndex within onchange handler fires another onchange
https://bugs.webkit.org/show_bug.cgi?id=23721

onchange events fire when a SELECT element has
focus and the selectedIndex is updated by script in some way--either
during another onchange, onkeypress, onfocus, or timer--and then
focus is lost.

Fixed by making a separate method for user-driven selectedIndex
changes, leaving scripts to use one which doesn't cause onchange to
be queued.

Test: fast/forms/select-script-onchange.html

  • dom/SelectElement.cpp: check if the pending change is user driven before calling onchange (WebCore::SelectElement::menuListOnChange): (WebCore::SelectElement::setSelectedIndex):
  • dom/SelectElement.h: store whether the pending change is user driven (WebCore::SelectElementData::userDrivenChange): (WebCore::SelectElementData::setUserDrivenChange):
  • html/HTMLSelectElement.cpp: split into two methods -- script version [non-user-driven] corresponds to IDL defined property name (WebCore::HTMLSelectElement::setSelectedIndex): (WebCore::HTMLSelectElement::setSelectedIndexByUser):
  • html/HTMLSelectElement.h:
  • rendering/RenderMenuList.cpp: use ByUser method when coming through the renderer (WebCore::RenderMenuList::valueChanged):

LayoutTests:

2009-06-25 John Gregg <johnnyg@google.com>

Reviewed by Sam Weinig.

Bug 23721: Changing dropdown's selectedIndex within onchange handler fires another onchange
https://bugs.webkit.org/show_bug.cgi?id=23721

Check for unnecessary calls to onchange in response to script
actions.

  • fast/forms/select-script-onchange-expected.txt: Added.
  • fast/forms/select-script-onchange.html: Added.
13:10 Changeset [45184] by mrowe@apple.com

Versioning.

13:02 Changeset [45183] by darin@chromium.org

2009-06-25 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Darin Fisher.

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

Combined LocalStorageArea and SessionStorageArea into StorageArea since
(after my other refactorings) there are no longer substantial
differences between the two.

  • GNUmakefile.am:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:
  • storage/LocalStorage.cpp: (WebCore::LocalStorage::storageArea):
  • storage/LocalStorage.h:
  • storage/LocalStorageArea.cpp: Removed.
  • storage/LocalStorageArea.h: Removed.
  • storage/SessionStorage.cpp: (WebCore::SessionStorage::copy): (WebCore::SessionStorage::storageArea):
  • storage/SessionStorage.h:
  • storage/SessionStorageArea.cpp: Removed.
  • storage/SessionStorageArea.h: Removed.
  • storage/StorageArea.cpp: (WebCore::StorageArea::createLocalStorage): (WebCore::StorageArea::StorageArea): (WebCore::StorageArea::createSessionStorage): (WebCore::StorageArea::copy): (WebCore::StorageArea::setItem): (WebCore::StorageArea::removeItem): (WebCore::StorageArea::clear): (WebCore::StorageArea::scheduleFinalSync): (WebCore::StorageArea::blockUntilImportComplete): (WebCore::StorageArea::dispatchStorageEvent):
  • storage/StorageArea.h:
  • storage/StorageAreaSync.cpp: (WebCore::StorageAreaSync::StorageAreaSync): (WebCore::StorageAreaSync::scheduleFinalSync): (WebCore::StorageAreaSync::syncTimerFired): (WebCore::StorageAreaSync::performImport):
  • storage/StorageAreaSync.h:
  • storage/StorageSyncManager.h:
12:50 Changeset [45182] by mrowe@apple.com

Versioning.

12:48 Changeset [45181] by mrowe@apple.com

New tag.

12:45 Changeset [45180] by mitz@apple.com

WebCore:

Reviewed by Darin Adler.

Test: fast/dom/style-sheet-candidate-remove-unrendered-document.html

When a "style sheet candidate" element is removed from a document,
call Document::removeStyleSheetCandidateNode() regardless of whether
the document is rendered. Otherwise, the document's style sheet
candidate set can end up containing stale references.

  • dom/ProcessingInstruction.cpp: (WebCore::ProcessingInstruction::removedFromDocument):
  • html/HTMLLinkElement.cpp: (WebCore::HTMLLinkElement::removedFromDocument):
  • html/HTMLStyleElement.cpp: (WebCore::HTMLStyleElement::removedFromDocument):

LayoutTests:

Reviewed by Darin Adler.

  • fast/dom/style-sheet-candidate-remove-unrendered-document-expected.txt: Added.
  • fast/dom/style-sheet-candidate-remove-unrendered-document.html: Added.
11:53 Changeset [45179] by dglazkov@chromium.org

2009-06-25 Dimitri Glazkov <dglazkov@chromium.org>

Reviewed by Darin Fisher.

Update CodeGeneratorV8.pm to sync up with the changes downstream.

  • bindings/scripts/CodeGeneratorV8.pm: Added HTMLFrameSetElement check, FileList as a ref-counted type, and DataGridColumn as typeCanFailConversion.
11:14 Changeset [45178] by rwlbuis@webkit.org

Credit the correct reviewer.

10:23 Changeset [45177] by ggaren@apple.com

2009-06-25 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig (sort of, maybe).


Fixed some ASSERTs in http/tests/security.


These ASSERTs were introduced by http://trac.webkit.org/changeset/45057,
but the underlying problem was actually older. http://trac.webkit.org/changeset/45057
just exposed the problem by enabling optimization in more cases.


The ASSERTs fired because we tested PropertySlot::slotBase() for validity,
but slotBase() ASSERTs if it's invalid, so we would ASSERT before
the test could happen. Solution: Remove the ASSERT. Maybe it was valid
once, but it clearly goes against a pattern we've deployed of late.


The underlying problem was that WebCore would re-use a PropertySlot in
the case of a forwarding access, and the second use would not completely
overwrite the first use. Solution: Make sure to overwrite m_offset when
setting a value on a PropertySlot. (Other values already get implicitly
overwritten during reuse.)

  • runtime/PropertySlot.h: (JSC::PropertySlot::PropertySlot): (JSC::PropertySlot::setValueSlot): (JSC::PropertySlot::setValue): (JSC::PropertySlot::setRegisterSlot): (JSC::PropertySlot::setUndefined): (JSC::PropertySlot::slotBase): (JSC::PropertySlot::clearOffset):
09:08 ProjectVision edited by treat@kde.org
(diff)
09:03 QtWebKit edited by treat@kde.org
(diff)
09:00 QtWebKitContrib edited by treat@kde.org
(diff)
08:54 QtWebKitGitInstructions edited by treat@kde.org
(diff)
08:50 Changeset [45176] by dglazkov@chromium.org

2009-06-25 Dimitri Glazkov <dglazkov@chromium.org>

Unreviewed, build fix.

Add FileList.h include to fix Chromium build.

  • platform/chromium/ClipboardChromium.cpp: Added FileList.h include.
08:48 UsingGitWithWebKit edited by treat@kde.org
(diff)
08:46 UsingGitWithWebKit edited by treat@kde.org
(diff)
08:46 UsingGitWithWebKit edited by treat@kde.org
(diff)
08:37 UsingGitWithWebKit edited by treat@kde.org
Make this more explicit. (diff)
06:14 Changeset [45175] by vestbo@webkit.org

2009-06-25 Tor Arne Vestbø <tor.arne.vestbo@nokia.com>

Reviewed by Simon Hausmann.

Use Q_ASSERT in Qt's DumpRenderTree instead of JSC's ASSERT

The WTFReportAssertionFailure function in JSC is not exported when
building QtWebKit in both debug and release on Mac, so DRT fails to
link.

We can revert this patch once the Qt port builds JSC as a separate
library, and we add the proper export macros to Assertions.cpp

  • DumpRenderTree/qt/WorkQueue.cpp: (WorkQueue::queue): (WorkQueue::dequeue):
06:11 Changeset [45174] by vestbo@webkit.org

2009-06-25 Tor Arne Vestbø <tor.arne.vestbo@nokia.com>

Reviewed by Jan Alonzo.

bugzilla-tool: Add Simon Hausmann as reviewer

  • Scripts/modules/bugzilla.py:
05:42 Changeset [45173] by jmalonzo@webkit.org

2009-06-25 Joseph Pecoraro <joepeck02@gmail.com>

Reviewed by Jan Alonzo.

Bug 26489: Web Inspector: Typo in DatabaseQuery Error Message
https://bugs.webkit.org/show_bug.cgi?id=26489

Fixed a Typo in a Web Inspector error message.

  • English.lproj/localizedStrings.js:
  • inspector/front-end/DatabaseQueryView.js: (WebInspector.DatabaseQueryView.prototype._queryError):
04:11 Changeset [45172] by hausmann@webkit.org

2009-06-25 Simon Hausmann <simon.hausmann@nokia.com>

Fix the Qt build, add missing StorageAreaSync files to the build.

  • WebCore.pro:
03:56 Changeset [45171] by hausmann@webkit.org

2009-06-25 Simon Hausmann <simon.hausmann@nokia.com>

Reviewed by and done with Tor Arne Vestbø.

Fix shortcut keyboard handling with plugins on the Qt/Mac build.

When we receive shortcut events like Ctrl+V then the text in the QKeyEvent is
empty. If we're asked to disambiguate the event into a Char keyboard event,
we try to detect this situation and still set the text, to ensure that the
general event handling sends a key press event after this disambiguation.

  • platform/qt/PlatformKeyboardEventQt.cpp: (WebCore::PlatformKeyboardEvent::disambiguateKeyDownEvent):
03:05 Changeset [45170] by eric@webkit.org

2009-06-25 Eric Seidel <eric@webkit.org>

No review, just skipping a new test.

Skip new test on platforms missing eventSender.beginDragWithFiles
Test: http/tests/security/clipboard/clipboard-file-access.html
https://bugs.webkit.org/show_bug.cgi?id=25916

  • platform/gtk/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:
02:49 Changeset [45169] by eric@webkit.org

2009-06-25 Eric Seidel <eric@webkit.org>

Build fix only, no review.

Add FileList.h and NotImplemented.h includes in an attempt to fix bots.

  • platform/gtk/ClipboardGtk.cpp:
  • platform/qt/ClipboardQt.cpp:
  • platform/win/ClipboardWin.cpp:
  • platform/wx/ClipboardWx.cpp:
02:24 Changeset [45168] by eric@webkit.org

2009-06-24 Dan Bernstein <mitz@apple.com>

Reviewed by Simon Fraser.

  • test for <rdar://problem/7001817> REGRESSION (r41902): Base position track at UCSC Genome Browser doesn't work because image map prevents img from hit-testing
  • fast/replaced/image-map-2-expected.txt: Added.
  • fast/replaced/image-map-2.html: Added.

2009-05-21 Eric Seidel <eric@webkit.org>

Reviewed by Maciej Stachowiak.

Expose files in the clipboard in ondrop events
https://bugs.webkit.org/show_bug.cgi?id=25916

Make it possible for applications like gmail to implement
drag and drop of attachments onto email messages.

This patch exposes an event.dataTransfer.files accessor
on the drop event. No information is exposed during dragover.
This follows the HTML 5 drag and drop security model:
http://www.w3.org/TR/html5/editing.html#security-risks-in-the-drag-and-drop-model
The test http/tests/security/clipboard/clipboard-file-access.html
verifies this behavior.

Internet Explorer shows historical documentation of supporting
getData('File') as a way of exposing files on the pasteboard. The current version of their docs:
http://msdn.microsoft.com/en-us/library/ms537658(VS.85).aspx
has removed this reference (as far as I can tell IE never implemented it)
I have a printed copy of that URL from 2008 on my desk describing getData('File') in IE.
IE does not follow the HTML5 clipboard security model and always allows access to the full clipboard, even on dragover.

I choose not to use IE's getData('File') and instead added .files
so that the accessor could have a type, matching WebKit's existing
.files accessor on HTMLInputElement.

Mozilla has equivalent file access:
event.dataTransfer.mozGetDataAt("application/x-moz-file", 0);
which also does not return a typed value.
https://developer.mozilla.org/En/DragDrop/Recommended_Drag_Types#Dragging_Files

This is only implemented for Mac WebKit. All other platforms (including Apple's Win WebKit)
have incomplete Clipboard implementations and will require experts from those platforms
to add this functionality. Right now they all have Clipboard*::files() methods which call notImplemented();

Test: http/tests/security/clipboard/clipboard-file-access.html

  • dom/Clipboard.h:
  • dom/Clipboard.idl:
  • platform/chromium/ClipboardChromium.cpp: (WebCore::ClipboardChromium::files):
  • platform/chromium/ClipboardChromium.h:
  • platform/gtk/ClipboardGtk.cpp: (WebCore::ClipboardGtk::files):
  • platform/gtk/ClipboardGtk.h:
  • platform/mac/ClipboardMac.h:
  • platform/mac/ClipboardMac.mm: (WebCore::absoluteURLsFromPasteboardFilenames): (WebCore::absoluteURLsFromPasteboard): (WebCore::ClipboardMac::files):
  • platform/qt/ClipboardQt.cpp: (WebCore::ClipboardQt::files):
  • platform/qt/ClipboardQt.h:
  • platform/win/ClipboardWin.cpp: (WebCore::ClipboardWin::files):
  • platform/win/ClipboardWin.h:
  • platform/wx/ClipboardWx.cpp: (WebCore::ClipboardWx::files):
  • platform/wx/ClipboardWx.h:
01:51 Changeset [45167] by barraclough@apple.com

Merged r44794:44796 into nitro-extreme branch.

01:41 Changeset [45166] by barraclough@apple.com

Merged r44747:44794 into nitro-extreme branch.

01:27 Changeset [45165] by eric@webkit.org

2009-06-25 Eric Seidel <eric@webkit.org>

No review, only completing revert of r45144.

Add back files deleted by r45144.

  • storage/LocalStorageArea.cpp: Added. (WebCore::LocalStorageArea::create): (WebCore::LocalStorageArea::LocalStorageArea): (WebCore::LocalStorageArea::scheduleFinalSync): (WebCore::LocalStorageArea::itemChanged): (WebCore::LocalStorageArea::itemRemoved): (WebCore::LocalStorageArea::areaCleared): (WebCore::LocalStorageArea::blockUntilImportComplete): (WebCore::LocalStorageArea::dispatchStorageEvent):
  • storage/LocalStorageArea.h: Added.
  • storage/SessionStorageArea.cpp: Added. (WebCore::SessionStorageArea::copy): (WebCore::SessionStorageArea::SessionStorageArea): (WebCore::SessionStorageArea::itemChanged): (WebCore::SessionStorageArea::itemRemoved): (WebCore::SessionStorageArea::areaCleared): (WebCore::SessionStorageArea::blockUntilImportComplete): (WebCore::SessionStorageArea::dispatchStorageEvent):
  • storage/SessionStorageArea.h: Added. (WebCore::SessionStorageArea::create):
01:26 Changeset [45164] by barraclough@apple.com

Merged r44743:44747 into nitro-extreme branch.

01:22 Changeset [45163] by barraclough@apple.com

Merged r44742:44743 into nitro-extreme branch.

01:19 Changeset [45162] by eric@webkit.org

2009-06-25 Eric Seidel <eric@webkit.org>

No review, reverting r45144 only.

Roll out r45144 after 18 test failures appeared on the bots.
https://bugs.webkit.org/show_bug.cgi?id=26698

  • GNUmakefile.am:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:
  • storage/LocalStorage.cpp: (WebCore::LocalStorage::storageArea):
  • storage/LocalStorage.h:
  • storage/SessionStorage.cpp: (WebCore::SessionStorage::copy): (WebCore::SessionStorage::storageArea):
  • storage/SessionStorage.h:
  • storage/StorageArea.cpp: (WebCore::StorageArea::StorageArea): (WebCore::StorageArea::~StorageArea): (WebCore::StorageArea::setItem): (WebCore::StorageArea::removeItem): (WebCore::StorageArea::clear):
  • storage/StorageArea.h:
  • storage/StorageAreaSync.cpp: (WebCore::StorageAreaSync::StorageAreaSync): (WebCore::StorageAreaSync::scheduleFinalSync): (WebCore::StorageAreaSync::syncTimerFired): (WebCore::StorageAreaSync::performImport):
  • storage/StorageAreaSync.h:
  • storage/StorageSyncManager.h:
01:17 Changeset [45161] by barraclough@apple.com

Merged r44712:44742 into nitro-extreme branch.

01:09 Changeset [45160] by eric@webkit.org

2009-06-24 Eric Seidel <eric@webkit.org>

Reviewed by Dave Levin.

Support local commits during apply-patches
and let land-patches take multiple bug ids.
https://bugs.webkit.org/show_bug.cgi?id=26703

I also restructured parts of land-patches into
class methods and static methods in preparation
for future code sharing with other commands.

  • Scripts/bugzilla-tool:
  • Scripts/modules/bugzilla.py:
  • Scripts/modules/scm.py:
01:09 Changeset [45159] by barraclough@apple.com

Merged r44710:44712 into nitro-extreme branch.

01:03 Changeset [45158] by barraclough@apple.com

Merged r44705:44710 into nitro-extreme branch.

01:01 Changeset [45157] by barraclough@apple.com

Merged r44704:44705 into nitro-extreme branch.

00:56 Changeset [45156] by barraclough@apple.com

Build-fix, had failed to add JSONObject.cpp/.h.

00:52 Changeset [45155] by barraclough@apple.com

Merged r44700:44704 into nitro-extreme branch.

00:49 Changeset [45154] by barraclough@apple.com

Merged r44699:44700 into nitro-extreme branch.

00:45 Changeset [45153] by eric@webkit.org

2009-06-25 Eric Seidel <eric@webkit.org>

Reviewed by Tor Arne Vestbø.

Make svn-apply work with Git too
https://bugs.webkit.org/show_bug.cgi?id=26299

Add an --force option to svn-apply and otherwise make svn-apply
exit non-zero when patch application fails.
https://bugs.webkit.org/show_bug.cgi?id=26300

I did not update svn-unapply, because it makes no sense in a Git world.
You don't roll in and out patch files. You make commits and deal with those.
Git users can just git reset --hard to get the same functionality.

  • Scripts/svn-apply:
00:42 Changeset [45152] by eric@webkit.org

2009-06-25 Eric Seidel <eric@webkit.org>

Reviewed by Tor Arne Vestbø.

Remove use of os.system to fix reviewers with unicode chars in their names
https://bugs.webkit.org/show_bug.cgi?id=26713

Also change to latin1 encoding of ø to make python happy.

  • Scripts/modules/bugzilla.py:
  • Scripts/modules/scm.py:
00:42 Changeset [45151] by barraclough@apple.com

Merged r44693:44699 into nitro-extreme branch.

00:34 Changeset [45150] by barraclough@apple.com

Merged r44692:44693 into nitro-extreme branch.

00:22 Changeset [45149] by barraclough@apple.com

Merged r44526:44692 into nitro-extreme branch.

00:07 Changeset [45148] by barraclough@apple.com

Merged r44525:44526 into nitro-extreme branch.

00:06 Changeset [45147] by eric@webkit.org

2009-06-25 Eric Seidel <eric@webkit.org>

Reviewed by Tor Arne Vestbø.

Add Adam Roben and Tor Arne Vestbø to the reviewers list.

  • Scripts/modules/bugzilla.py:
00:00 Changeset [45146] by barraclough@apple.com

Merged r44523:44525 into nitro-extreme branch.

06/24/09:

23:56 Changeset [45145] by barraclough@apple.com

Merged r44520:44523 into nitro-extreme branch.

23:16 Changeset [45144] by darin@chromium.org

2009-06-24 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Darin Fisher.

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

Combined LocalStorageArea and SessionStorageArea into StorageArea since
(after my other refactorings) there are no longer substantial
differences between the two.

  • GNUmakefile.am:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:
  • storage/LocalStorage.cpp: (WebCore::LocalStorage::storageArea):
  • storage/LocalStorage.h:
  • storage/LocalStorageArea.cpp: Removed.
  • storage/LocalStorageArea.h: Removed.
  • storage/SessionStorage.cpp: (WebCore::SessionStorage::copy): (WebCore::SessionStorage::storageArea):
  • storage/SessionStorage.h:
  • storage/SessionStorageArea.cpp: Removed.
  • storage/SessionStorageArea.h: Removed.
  • storage/StorageArea.cpp: (WebCore::StorageArea::createLocalStorage): (WebCore::StorageArea::StorageArea): (WebCore::StorageArea::createSessionStorage): (WebCore::StorageArea::copy): (WebCore::StorageArea::setItem): (WebCore::StorageArea::removeItem): (WebCore::StorageArea::clear): (WebCore::StorageArea::scheduleFinalSync): (WebCore::StorageArea::blockUntilImportComplete): (WebCore::StorageArea::dispatchStorageEvent):
  • storage/StorageArea.h:
  • storage/StorageAreaSync.cpp: (WebCore::StorageAreaSync::StorageAreaSync): (WebCore::StorageAreaSync::scheduleFinalSync): (WebCore::StorageAreaSync::syncTimerFired): (WebCore::StorageAreaSync::performImport):
  • storage/StorageAreaSync.h:
  • storage/StorageSyncManager.h:
22:01 Changeset [45143] by mitz@apple.com

WebCore:

Reviewed by Simon Fraser.

  • fix <rdar://problem/7001817> REGRESSION (r41902): Base position track at UCSC Genome Browser doesn't work because image map prevents img from hit-testing

Test: fast/replaced/image-map-2.html

  • rendering/RenderImage.cpp: (WebCore::RenderImage::nodeAtPoint): Do not reset 'inside' to false if the image map failed the hit test.

LayoutTests:

Reviewed by Simon Fraser.

  • test for <rdar://problem/7001817> REGRESSION (r41902): Base position track at UCSC Genome Browser doesn't work because image map prevents img from hit-testing
  • fast/replaced/image-map-2-expected.txt: Added.
  • fast/replaced/image-map-2.html: Added.
20:41 Changeset [45142] by barraclough@apple.com

Merged r44514:44520 into nitro-extreme branch.

20:34 Changeset [45141] by barraclough@apple.com

Merged r44513:44514 into nitro-extreme branch.

20:30 Changeset [45140] by barraclough@apple.com

Merged r44492:44513 into nitro-extreme branch.

20:18 Changeset [45139] by barraclough@apple.com

Merged r44491:44492 into nitro-extreme branch.

20:07 Changeset [45138] by weinig@apple.com

2009-06-24 Sam Weinig <sam@webkit.org>

Reviewed by Gavin Barraclough.

Make the opcode sampler work once again.

  • jit/JIT.h: (JSC::JIT::compileGetByIdProto): (JSC::JIT::compileGetByIdSelfList): (JSC::JIT::compileGetByIdProtoList): (JSC::JIT::compileGetByIdChainList): (JSC::JIT::compileGetByIdChain): (JSC::JIT::compilePutByIdTransition): (JSC::JIT::compileCTIMachineTrampolines): (JSC::JIT::compilePatchGetArrayLength):
  • jit/JITStubCall.h: (JSC::JITStubCall::call):
20:06 Changeset [45137] by barraclough@apple.com

Merged r44460:44491 into nitro-extreme branch.

20:00 Changeset [45136] by barraclough@apple.com

Merged r44455:44460 into nitro-extreme branch.

19:56 Changeset [45135] by barraclough@apple.com

Merged r44454:44455 into nitro-extreme branch.

19:38 Changeset [45134] by abarth@webkit.org

2009-06-22 Adam Barth <abarth@webkit.org>

Reviewed by Dimitri Glazkov.

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

Refactor V8DOMMap to support isolated worlds.

  • bindings/v8/ScriptController.cpp: (WebCore::ScriptController::evaluateInNewWorld):
  • bindings/v8/ScriptController.h:
  • bindings/v8/V8DOMMap.cpp: (WebCore::DOMDataStore::InternalDOMWrapperMap::InternalDOMWrapperMap): (WebCore::DOMDataStore::allStores): (WebCore::DOMDataStore::allStoresMutex): (WebCore::DOMDataStore::domData): (WebCore::ScopedDOMDataStore::ScopedDOMDataStore): (WebCore::ScopedDOMDataStore::~ScopedDOMDataStore): (WebCore::StaticDOMDataStore::StaticDOMDataStore): (WebCore::): (WebCore::MainThreadDOMData::MainThreadDOMData): (WebCore::MainThreadDOMData::getStore): (WebCore::ChildThreadDOMData::ChildThreadDOMData): (WebCore::ChildThreadDOMData::getStore): (WebCore::DOMDataStore::DOMDataStore): (WebCore::DOMDataStore::~DOMDataStore): (WebCore::DOMDataStoreHandle::DOMDataStoreHandle): (WebCore::DOMDataStoreHandle::~DOMDataStoreHandle): (WebCore::::forget): (WebCore::getDOMNodeMap): (WebCore::getDOMObjectMap): (WebCore::getActiveDOMObjectMap): (WebCore::getDOMSVGElementInstanceMap): (WebCore::getDOMSVGObjectWithContextMap): (WebCore::DOMData::getCurrent): (WebCore::DOMData::handleWeakObject): (WebCore::DOMData::ensureDeref): (WebCore::weakDOMObjectCallback): (WebCore::weakActiveDOMObjectCallback): (WebCore::weakNodeCallback): (WebCore::weakSVGElementInstanceCallback): (WebCore::weakSVGObjectWithContextCallback): (WebCore::DOMData::derefObject): (WebCore::DOMData::derefDelayedObjects): (WebCore::DOMData::derefDelayedObjectsInCurrentThread): (WebCore::DOMData::removeObjectsFromWrapperMap): (WebCore::removeAllDOMObjectsInCurrentThreadHelper): (WebCore::visitDOMNodesInCurrentThread): (WebCore::visitDOMObjectsInCurrentThread): (WebCore::visitActiveDOMObjectsInCurrentThread): (WebCore::visitDOMSVGElementInstancesInCurrentThread): (WebCore::visitSVGObjectsInCurrentThread):
  • bindings/v8/V8DOMMap.h: (WebCore::DOMDataStoreHandle::getStore):
  • bindings/v8/V8IsolatedWorld.cpp: Added. (WebCore::getIsolatedWorldKey): (WebCore::contextWeakReferenceCallback): (WebCore::V8IsolatedWorld::evaluate): (WebCore::V8IsolatedWorld::V8IsolatedWorld): (WebCore::V8IsolatedWorld::~V8IsolatedWorld): (WebCore::V8IsolatedWorld::getEntered):
  • bindings/v8/V8IsolatedWorld.h: Added. (WebCore::V8IsolatedWorld::getDOMDataStore):
19:06 Changeset [45133] by mrowe@apple.com

Versioning.

19:04 Changeset [45132] by mrowe@apple.com

New tag.

18:52 Changeset [45131] by eric@webkit.org

2009-06-24 Mikhail Naganov <mnaganov@chromium.org>

Reviewed by Timothy Hatcher.

Bug 26604: Search doesn't work in Web Inspector Profiler
https://bugs.webkit.org/show_bug.cgi?id=26604

Seems like search was damaged in revision 42808.

  • inspector/front-end/ProfileView.js: (WebInspector.ProfileView.prototype.refresh): Here and in other functions: nodes we're searching in are profile data grid nodes, so there is no more need for '_dataGridNode' references. (WebInspector.ProfileView.prototype.searchCanceled): (WebInspector.ProfileView.prototype.performSearch.matchesQuery): Fixed accidental semicolon that caused 'matchesQuery' always return true. (WebInspector.ProfileView.prototype.performSearch): To perform search correctly in the case of bottom up tree, we need to populate the tree, because there's no 1-to-1 correspondence between profile nodes and data grid nodes in this case. (WebInspector.ProfileView.prototype._jumpToSearchResult):
18:32 Changeset [45130] by eric@webkit.org

2009-06-24 Zoltan Horvath <hzoltan@inf.u-szeged.hu>

Reviewed by Maciej Stachowiak.

Extend FastAllocBase.h with 'using WTF::FastAllocBase' to avoid

unnecessary WTF
usings.
Remove existing unnecessary WTF
usings.
  • interpreter/Interpreter.h:
  • profiler/CallIdentifier.h:
  • runtime/ScopeChain.h:
  • wtf/FastAllocBase.h:
18:07 Changeset [45129] by simon.fraser@apple.com

2009-06-24 Simon Fraser <simon.fraser@apple.com>

Reviewed by Darin Adler.

<rdar://problem/6450239&6574516>

Fix flashing issues caused by compositing layers rendering content before
a deferred layout has happened. Because the -viewWillDraw machinery doesn't
work for composited layers, we need to use scheduleViewUpdate() to queue
up a layout via the run loop observer in WebKit, whenever we know we
are going to be painting soon.

  • rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::setContentsNeedDisplay): (WebCore::RenderLayerBacking::setContentsNeedDisplayInRect):
17:45 Changeset [45128] by levin@chromium.org

JavaScriptCore:

2009-06-24 David Levin <levin@chromium.org>

Fix all builds.

  • bytecode/CodeBlock.h:
  • bytecompiler/BytecodeGenerator.h:
  • interpreter/Register.h:

WebCore:

2009-06-24 David Levin <levin@chromium.org>

Fix all builds.

  • ForwardingHeaders/wtf/FastAllocBase.h: Added.
17:42 Changeset [45127] by mrowe@apple.com

Versioning.

17:25 Changeset [45126] by barraclough@apple.com

Merged r44437:44454 into nitro-extreme branch.

17:20 Changeset [45125] by barraclough@apple.com

Merged r44436:44437 into nitro-extreme branch.

16:56 Changeset [45124] by darin@chromium.org

2009-06-24 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Darin Fisher.

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

Split the syncing portions of LocalStorageArea into StorageAreaSync.
This name will make more sense in the next patch (in this set) when
LocalStorageArea and SessionStorageArea are merged to become simply
StorageArea. (Thus the synching portion of StorageArea is in
StorageAreaSync.)

This looks like a big patch, but really all it's doing is splitting
code and patching split-related things up.

  • GNUmakefile.am:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:
  • storage/LocalStorageArea.cpp: (WebCore::LocalStorageArea::create): (WebCore::LocalStorageArea::LocalStorageArea): (WebCore::LocalStorageArea::scheduleFinalSync): (WebCore::LocalStorageArea::itemChanged): (WebCore::LocalStorageArea::itemRemoved): (WebCore::LocalStorageArea::areaCleared): (WebCore::LocalStorageArea::blockUntilImportComplete):
  • storage/LocalStorageArea.h:
  • storage/LocalStorageTask.cpp: (WebCore::LocalStorageTask::LocalStorageTask):
  • storage/LocalStorageTask.h: (WebCore::LocalStorageTask::createImport): (WebCore::LocalStorageTask::createSync):
  • storage/LocalStorageThread.cpp: (WebCore::LocalStorageThread::scheduleImport): (WebCore::LocalStorageThread::scheduleSync):
  • storage/LocalStorageThread.h:
  • storage/SessionStorageArea.cpp: (WebCore::SessionStorageArea::blockUntilImportComplete):
  • storage/SessionStorageArea.h:
  • storage/StorageArea.h:
  • storage/StorageAreaSync.cpp: Copied from WebCore/storage/LocalStorageArea.cpp.
  • storage/StorageAreaSync.h: Copied from WebCore/storage/LocalStorageArea.h.
  • storage/StorageSyncManager.cpp: (WebCore::StorageSyncManager::scheduleImport): (WebCore::StorageSyncManager::scheduleSync):
  • storage/StorageSyncManager.h:
16:55 Changeset [45123] by eric@webkit.org

2009-06-24 Zoltan Horvath <hzoltan@inf.u-szeged.hu>

Reviewed by Maciej Stachowiak.

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

Inherits CodeBlock class from FastAllocBase because it
has been instantiated by 'new' in JavaScriptCore/bytecode/CodeBlock.h:217.

  • bytecode/CodeBlock.h:
16:55 Changeset [45122] by eric@webkit.org

2009-06-24 Zoltan Horvath <hzoltan@inf.u-szeged.hu>

Reviewed by Maciej Stachowiak.

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

Inherits BytecodeGenerator class from FastAllocBase because it has been
instantiated by 'new' in JavaScriptCore/parser/Nodes.cpp:1892.

  • bytecompiler/BytecodeGenerator.h:
16:54 Changeset [45121] by eric@webkit.org

2009-06-24 Zoltan Horvath <hzoltan@inf.u-szeged.hu>

Reviewed by Maciej Stachowiak.

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

Inherits Register class from FastAllocBase because it has been
instantiated by 'new' in JavaScriptCore/runtime/JSVariableObject.h:149.

  • interpreter/Register.h:
16:53 Changeset [45120] by eric@webkit.org

2009-06-24 Zoltan Horvath <hzoltan@inf.u-szeged.hu>

Reviewed by Darin Adler.

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

Inherits HashMap class from FastAllocBase because it has been
instantiated by 'new' in JavaScriptCore/runtime/Structure.cpp:458.

  • wtf/HashMap.h:
16:52 Changeset [45119] by barraclough@apple.com

Merged r44412:44436 into nitro-extreme branch.

16:33 Changeset [45118] by barraclough@apple.com

Merged r44411:44412 into nitro-extreme branch.

16:32 Changeset [45117] by treat@webkit.org

2009-06-24 Adam Treat <adam.treat@torchmobile.com>

Fix Qt build.

  • WebCore.pro:
16:23 Changeset [45116] by mrowe@apple.com

Merge r44877.

16:23 Changeset [45115] by mrowe@apple.com

Merge r44845.

16:23 Changeset [45114] by mrowe@apple.com

Merge r44844.

16:23 Changeset [45113] by mrowe@apple.com

Merge r44262.

16:00 Changeset [45112] by cfleizach@apple.com

Windows build fix for fallout from

Bug 26668: AX: need a way to retrieve the language for an element

15:47 Changeset [45111] by levin@chromium.org

2009-06-24 David Levin <levin@chromium.org>

Reviewed by David Hyatt.

Bug 26696: Member functions in DataGridColumnList should return pointers instead of PassRefPtr.
https://bugs.webkit.org/show_bug.cgi?id=26696

  • html/DataGridColumnList.cpp: (WebCore::DataGridColumnList::itemWithName): (WebCore::DataGridColumnList::add):
  • html/DataGridColumnList.h: (WebCore::DataGridColumnList::item): (WebCore::DataGridColumnList::primaryColumn): (WebCore::DataGridColumnList::sortColumn):
15:19 Changeset [45110] by weinig@apple.com

2009-06-24 Sam Weinig <sam@webkit.org>

Reviewed by Dave "Messy" Hyatt.

Little bit of style cleanup.

  • html/DataGridColumn.cpp:
  • html/DataGridColumn.h:
  • html/DataGridColumn.idl:
  • html/DataGridColumnList.cpp:
  • html/DataGridColumnList.h:
  • html/DataGridColumnList.idl:
  • html/HTMLDataGridCellElement.cpp:
  • html/HTMLDataGridCellElement.h:
  • html/HTMLDataGridCellElement.idl:
  • html/HTMLDataGridColElement.cpp:
  • html/HTMLDataGridColElement.h:
  • html/HTMLDataGridColElement.idl:
  • html/HTMLDataGridElement.h:
  • html/HTMLDataGridElement.idl:
  • html/HTMLDataGridRowElement.cpp:
  • html/HTMLDataGridRowElement.h:
  • html/HTMLDataGridRowElement.idl:
15:14 Changeset [45109] by cfleizach@apple.com

Bug 26668: AX: need a way to retrieve the language for an element
https://bugs.webkit.org/show_bug.cgi?id=26668

15:12 Changeset [45108] by barraclough@apple.com

Merged r44350:44411 into nitro-extreme branch.

15:04 Changeset [45107] by beidson@apple.com

2009-06-24 Brady Eidson <beidson@apple.com>

Reviewed by Dan Bernstein.


<rdar://problem/6893811> Instead of downloading files linked from Google Earth, file contents displayed in browser window as text.

  • platform/network/mac/WebCoreURLResponse.mm: (createBinaryExtensionsSet): Add '.kmz' to the list of known-to-be-binary extensions.
15:01 Changeset [45106] by oliver@apple.com

<rdar://problem/6940519> REGRESSION (Safari 4 Public Beta - TOT): google.com/adplanner shows blank page instead of site details in "basic research'

Reviewed by Darin Adler.

The problem was caused by the page returned with a function using a
var declaration list containing around ~3000 variables. The solution
to this is to flatten the comma expression representation and make
codegen comma expressions and initializer lists iterative rather than
recursive.

14:53 Changeset [45105] by barraclough@apple.com

Merged r44340:44350 into nitro-extreme branch.

14:12 Changeset [45104] by mitz@apple.com

Rephrased change log

14:10 Changeset [45103] by eric@webkit.org

2009-06-24 Nicolas Weber <thakis@chromium.org>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=26685
Accomodate for backwards-incompatible skia api changes.

  • platform/graphics/skia/GraphicsContextSkia.cpp: (WebCore::GraphicsContext::clearRect): (WebCore::GraphicsContext::setCompositeOperation):
  • platform/graphics/skia/ImageBufferSkia.cpp: (WebCore::ImageBuffer::ImageBuffer):
  • platform/graphics/skia/ImageSkia.cpp: (WebCore::paintSkBitmap): (WebCore::Image::drawPattern):
  • platform/graphics/skia/PlatformContextSkia.cpp: (PlatformContextSkia::State::State): (PlatformContextSkia::setupPaintCommon): (PlatformContextSkia::setXfermodeMode): (PlatformContextSkia::applyClipFromImage):
  • platform/graphics/skia/PlatformContextSkia.h:
  • platform/graphics/skia/SkiaUtils.cpp: (WebCore::): (WebCore::WebCoreCompositeToSkiaComposite):
  • platform/graphics/skia/SkiaUtils.h:
14:05 Changeset [45102] by mitz@apple.com

Reviewed by Anders Carlsson.

Fix the most recently seen kind of crash seen in
<rdar://problem/5983224>

  • WebView/WebHTMLView.mm: (-[WebHTMLView adjustPageHeightNew:top:bottom:limit:]): Null-check the Frame.
14:01 Changeset [45101] by barraclough@apple.com

Merged r44170:44340 into nitro-extreme branch.

13:51 Changeset [45100] by jmalonzo@webkit.org

2009-06-24 Jan Michael Alonzo <jmalonzo@webkit.org>

Gtk build fix. Add files that were added in r45093 and r45096

  • GNUmakefile.am:
13:36 Changeset [45099] by barraclough@apple.com

Merged r44076:44170 into nitro-extreme branch.

13:33 Changeset [45098] by beidson@apple.com

2009-06-24 Brady Eidson <beidson@apple.com>

Fix 64-bit SnowLeopard build.

  • html/DataGridColumnList.cpp: (WebCore::DataGridColumnList::remove): (WebCore::DataGridColumnList::move):
13:15 Changeset [45097] by barraclough@apple.com

2009-06-24 Gavin Barraclough <barraclough@apple.com>

Reviewed by Geoff Garen.

Enable JIT_OPTIMIZE_METHOD_CALLS on the branch, implementation matches current implemenatation in ToT.

  • jit/JIT.h:
  • jit/JITPropertyAccess.cpp: (JSC::JIT::emit_op_method_check): (JSC::JIT::emitSlow_op_method_check): (JSC::JIT::emit_op_get_by_id): (JSC::JIT::compileGetByIdHotPath): (JSC::JIT::emitSlow_op_get_by_id): (JSC::JIT::compileGetByIdSlowCase):
13:06 Changeset [45096] by hyatt@apple.com

Fix Mac build.

12:44 Changeset [45095] by rwlbuis@webkit.org

Reviewed by Niko.

https://bugs.webkit.org/show_bug.cgi?id=26392
Bug 26392: In html, modification of xlink:href of an newly inserted svg image does not work.
https://bugs.webkit.org/show_bug.cgi?id=26328
Bug 26328: changing href attribute of svg images dows not work when changing display attribute as well

React to href updates even when there is no renderer, i.e. display=none.

Tests: svg/custom/js-update-image-and-display.svg

svg/custom/js-update-image-and-display2.svg
svg/custom/js-update-image-and-display3.svg

12:41 Changeset [45094] by mitz@apple.com

Add Windows test results for a couple of search field tests

12:22 Changeset [45093] by hyatt@apple.com

WebCore:

2009-06-24 David Hyatt <hyatt@apple.com>

Reviewed by Sam Weinig.

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

Add basic back-end column support to datagrid.

Added fast/dom/HTMLDataGridElement/ column tests.

  • DerivedSources.cpp:
  • DerivedSources.make:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:
  • html/DataGridColumn.cpp: Added. (WebCore::DataGridColumn::setPrimary):
  • html/DataGridColumn.h: Added. (WebCore::DataGridColumn::create): (WebCore::DataGridColumn::id): (WebCore::DataGridColumn::setId): (WebCore::DataGridColumn::label): (WebCore::DataGridColumn::setLabel): (WebCore::DataGridColumn::type): (WebCore::DataGridColumn::setType): (WebCore::DataGridColumn::sortable): (WebCore::DataGridColumn::setSortable): (WebCore::DataGridColumn::sortDirection): (WebCore::DataGridColumn::setSortDirection): (WebCore::DataGridColumn::primary): (WebCore::DataGridColumn::detachFromColumnList): (WebCore::DataGridColumn::DataGridColumn):
  • html/DataGridColumn.idl: Added.
  • html/DataGridColumnList.cpp: Added. (WebCore::DataGridColumnList::~DataGridColumnList): (WebCore::DataGridColumnList::itemWithName): (WebCore::DataGridColumnList::add): (WebCore::DataGridColumnList::remove): (WebCore::DataGridColumnList::move): (WebCore::DataGridColumnList::clear): (WebCore::DataGridColumnList::primaryColumnChanged):
  • html/DataGridColumnList.h: Added. (WebCore::DataGridColumnList::create): (WebCore::DataGridColumnList::length): (WebCore::DataGridColumnList::item): (WebCore::DataGridColumnList::primaryColumn): (WebCore::DataGridColumnList::sortColumn):
  • html/DataGridColumnList.idl: Added.
  • html/HTMLDataGridColElement.cpp: (WebCore::HTMLDataGridColElement::sortable): (WebCore::HTMLDataGridColElement::setSortable): (WebCore::HTMLDataGridColElement::sortDirection): (WebCore::HTMLDataGridColElement::setSortDirection):
  • html/HTMLDataGridColElement.h:
  • html/HTMLDataGridColElement.idl:
  • html/HTMLDataGridElement.cpp: (WebCore::HTMLDataGridElement::HTMLDataGridElement):
  • html/HTMLDataGridElement.h: (WebCore::HTMLDataGridElement::columns):
  • html/HTMLDataGridElement.idl:
  • rendering/RenderDataGrid.cpp: (WebCore::RenderDataGrid::paintObject): (WebCore::RenderDataGrid::paintColumnHeaders): (WebCore::RenderDataGrid::rebuildColumns):
  • rendering/RenderDataGrid.h: (WebCore::RenderDataGrid::gridElement):

LayoutTests:

2009-06-24 David Hyatt <hyatt@apple.com>

Reviewed by Sam Weinig.

Add test for https://bugs.webkit.org/show_bug.cgi?id=26687.

  • fast/dom/HTMLDataGridElement/DataGridColumns-basic-expected.txt: Added.
  • fast/dom/HTMLDataGridElement/DataGridColumns-basic.html: Added.
12:14 Changeset [45092] by mrowe@apple.com

Merge r44951.

12:13 Changeset [45091] by mrowe@apple.com

Merge r44941.

12:13 Changeset [45090] by mrowe@apple.com

Merge r44940.

12:13 Changeset [45089] by mrowe@apple.com

Merge r44908.

12:13 Changeset [45088] by mrowe@apple.com

Merge r44906.

10:29 Changeset [45087] by jberlin@apple.com

2009-06-24 Jessie Berlin <jberlin@apple.com>

Reviewed by Adam Roben.

Partially fixes: https://bugs.webkit.org/show_bug.cgi?id=24735
(<rdar://problem/5015942>)
Where on windows it was not possible to set an element as the drag
image using setDragImage on the dataTransfer object.


Does not "fix" the case of dragging a link where the default link image
is still used, even when the -webkit-user-drag is set to "element". This
is the same behavior as is found on OS X.


Added a manual test because it is not possible to check that what is
contained in the image snapshot is indeed the requested element.

  • dom/Clipboard.h: (WebCore::Clipboard::dragImageElement): Made getting the raw pointer from the RefPtr a const operation.
  • manual-tests/drag-with-div-or-image-as-data-image.html: Added.
  • platform/win/ClipboardWin.cpp: (WebCore::ClipboardWin::createDragImage): Get an image of the rendered element and its subtree.
09:48 Changeset [45086] by darin@chromium.org

2009-06-24 Darin Fisher <darin@chromium.org>

Reviewed by David Levin.

https://bugs.webkit.org/show_bug.cgi?id=26683
Fix Chromium build bustage: Add custom binding for HTMLDataGridElement.dataSource

This change just adds a stub implementation for now to help fix the build.

  • bindings/v8/custom/V8CustomBinding.h:
  • bindings/v8/custom/V8HTMLDataGridElementCustom.cpp: Added.
08:53 Changeset [45085] by ddkilzer@apple.com

Build fixes for ENABLE(PLUGIN_PROXY_FOR_VIDEO)

Reviewed by Adam Roben.

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::deliverNotification): Removed ExceptionCode parameter from togglePlayState(). (WebCore::HTMLMediaElement::initialURL): Don't convert a KURL object to a String when assigning to a KURL variable.
08:44 Moving to Git edited by treat@kde.org
(diff)
08:00 Changeset [45084] by zimmermann@webkit.org

Reviewed by Adam Treat.

Save memory by not storing attribute values in member variables, if not absolutely needed.
Also fixes bugs where we're substituting variables too early (noticeable with the upcoming <select> element).

07:35 Changeset [45083] by zimmermann@webkit.org

Reviewed by Adam Roben.
Forgot to initialize m_task member variable. Results in crashes sometimes.

07:34 Changeset [45082] by zimmermann@webkit.org

Not reviewed. Forgot to include within last patch.

07:30 Changeset [45081] by zimmermann@webkit.org

Reviewed by Nikolas Zimmermann.

Fixes: https://bugs.webkit.org/show_bug.cgi?id=26474
Adding WML <card> title attribute handling.

07:02 Changeset [45080] by zecke@webkit.org

2009-06-24 Jiahua Huang <jhuangjiahua@gmail.com>

Reviewed by Holger Freyther.

[Gtk] Add Undo/Redo support to WebKitGtk
https://bugs.webkit.org/show_bug.cgi?id=26573

Implement EditorClient::undo and other interested funcs.

  • WebCoreSupport/EditorClientGtk.cpp: (WebKit::EditorClient::registerCommandForUndo): (WebKit::EditorClient::registerCommandForRedo): (WebKit::EditorClient::clearUndoRedoOperations): (WebKit::EditorClient::canUndo): (WebKit::EditorClient::canRedo): (WebKit::EditorClient::undo): (WebKit::EditorClient::redo): (WebKit::EditorClient::EditorClient):
  • WebCoreSupport/EditorClientGtk.h:
03:00 Changeset [45079] by hausmann@webkit.org

2009-06-24 Simon Hausmann <simon.hausmann@nokia.com>

Reviewed by Tor Arne Vestbø.

Fix the build without ENABLE_VIDEO.

  • html/CanvasRenderingContext2D.cpp: Add #if ENABLE guards.
  • html/CanvasRenderingContext2D.h: Ditto.
02:26 Changeset [45078] by oliver@apple.com

Temporarily disable failing test

01:56 Changeset [45077] by eric@webkit.org

2009-06-24 Eric Seidel <eric@webkit.org>

Reviewed by Mark Rowe.

Add new drag tests to the Skipped lists for non-mac platforms.

Skipped tests:
fast/events/drag-to-navigate.html
fast/events/prevent-drag-to-navigate.html

  • platform/gtk/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:
01:15 Changeset [45076] by eric@webkit.org

2009-06-24 Zoltan Horvath <hzoltan@inf.u-szeged.hu>

Reviewed by Maciej Stachowiak.

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

Inherits ScopeChainNode class from FastAllocBase because it has been
instantiated by 'new' in JavaScriptCore/runtime/ScopeChain.h:95.

  • wtf/RefPtr.h:
01:12 Changeset [45075] by eric@webkit.org

2009-06-24 Siddhartha Chattopadhyay <sidchat@google.com>

Reviewed by Eric Seidel.

Fix for https://bugs.webkit.org/show_bug.cgi?id=26200 which I introduced earlier. I had
the smartReplace set TRUE initially, which was wrong since the purpose of this call is
to just replace the selection, and place the caret at the end of the selection it replaced.

  • editing/Editor.cpp: (WebCore::Editor::markMisspellingsAfterTypingToPosition):
  • manual-tests/textarea-caret-position-after-auto-spell-correct.html: Added.
01:00 Changeset [45074] by eric@webkit.org

2009-06-24 Jeff Johnson <opendarwin@lapcatsoftware.com>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=26638 WebKitErrors.m: _initWithPluginErrorCode: does not set localizedDescription

Add localized descriptions for plugin errors.
The localized strings already existed but were unused; now NSLocalizedDescriptionKey is added to the NSError userInfo.

  • Misc/WebKitErrors.m: (-[NSError _initWithPluginErrorCode:contentURL:pluginPageURL:pluginName:MIMEType:]):
00:54 Changeset [45073] by eric@webkit.org

2009-06-24 Zoltan Horvath <hzoltan@inf.u-szeged.hu>

Reviewed by Darin Adler.

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

Inherits Deque class from FastAllocBase because it has been
instantiated by 'new' with DEFINE_STATIC_LOCAL macro in
JavaScriptCore/wtf/MainThread.cpp:62.

  • wtf/Deque.h:
00:53 Changeset [45072] by eric@webkit.org

2009-06-24 Zoltan Horvath <hzoltan@inf.u-szeged.hu>

Reviewed by Darin Adler.

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

Inherits RefPtr class from FastAllocBase because it has been
instantiated by 'new' in JavaScriptCore/runtime/StructureChain.cpp:41.

  • wtf/RefPtr.h:
00:52 Changeset [45071] by oliver@apple.com

Attempting to make test more reliable.

00:47 Changeset [45070] by eric@webkit.org

2009-06-24 Kevin Watters <kevinwatters@gmail.com>

Reviewed by Kevin Ollivier.

Don't use an unsigned int for intervalInMS because we sometimes initialize
it with a negative value.

  • platform/wx/SharedTimerWx.cpp: (WebCore::setSharedTimerFireTime):
00:45 Changeset [45069] by eric@webkit.org

2009-06-24 Zoltan Horvath <hzoltan@inf.u-szeged.hu>

Reviewed by Darin Adler.

Inherits HashSet class from FastAllocBase, because it has been
instantiated by 'new' in JavaScriptCore/runtime/Collector.h:116.

  • wtf/HashSet.h:
00:40 Changeset [45068] by eric@webkit.org

2009-06-24 Zoltan Horvath <hzoltan@inf.u-szeged.hu>

Reviewed by Darin Adler.

Inherits Vector class from FastAllocBase because it has been
instantiated by 'new' in JavaScriptCore/runtime/Structure.cpp:633.

  • wtf/Vector.h:
00:34 Changeset [45067] by eric@webkit.org

2009-06-24 Eric Seidel <eric@webkit.org>

Reviewed by Maciej Stachowiak.

Spell Maciej's name right and add Brady Eidson.

  • Scripts/modules/bugzilla.py:
00:26 Changeset [45066] by eric@webkit.org

2009-06-24 Norbert Leser <norbert.leser@nokia.com>

Reviewed by Maciej Stachoviak.

The BytecodeGenerator objects were instantiated on stack, which takes up ~38kB per instance
(each instance includes copy of JSC::CodeBlock with large SymbolTable, etc.).
Specifically, since there is nested invocation (e.g., GlobalCode --> FunctionCode),
the stack overflows immediately on Symbian hardware (max. 80 kB).
Proposed change allocates generator objects on heap.
Performance impact (if any) should be negligible and change is proposed as general fix,
rather than ifdef'd for SYMBIAN.

  • parser/Nodes.cpp: (JSC::ProgramNode::generateBytecode): (JSC::EvalNode::generateBytecode): (JSC::EvalNode::bytecodeForExceptionInfoReparse): (JSC::FunctionBodyNode::generateBytecode): (JSC::FunctionBodyNode::bytecodeForExceptionInfoReparse):
00:20 Changeset [45065] by eric@webkit.org

2009-06-24 Eric Seidel <eric@webkit.org>

No review, build fix only.

The previous checkin had an improper merge. This fixes the build.

  • page/DragController.cpp: (WebCore::DragController::tryDocumentDrag):
00:07 Changeset [45064] by eric@webkit.org

2009-06-05 Eric Seidel <eric@webkit.org>

Reviewed by Oliver Hunt.

https://bugs.webkit.org/show_bug.cgi?id=25922
Fix dropEffect = "none" to work as expected.

  • fast/events/drag-to-navigate-expected.txt: Copied from LayoutTests/editing/selection/doubleclick-whitespace-img-crash-expected.txt.
  • fast/events/drag-to-navigate.html: Added.
  • fast/events/prevent-drag-to-navigate-expected.txt: Copied from LayoutTests/editing/selection/doubleclick-whitespace-img-crash-expected.txt.
  • fast/events/prevent-drag-to-navigate.html: Added.
  • fast/events/resources/file-for-drag-to-navigate.html: Added.
  • fast/events/resources/file-for-prevent-drag-to-navigate.html: Added.

2009-06-05 Eric Seidel <eric@webkit.org>

Reviewed by Oliver Hunt.

https://bugs.webkit.org/show_bug.cgi?id=25922
JS setting dropEffect = "none" causes tryDHTMLDrag
to return DragOperationNone. Which is also the value
tryDHTMLDrag previously used to indicate JS did not
want to handle the drag.

Make it possible for the DragController::try* methods
to return a bool to indicate if javascript accepted
or rejected the drag event, separate from the DragOperation.

Tests:

  • fast/events/drag-to-navigate.html
  • fast/events/prevent-drag-to-navigate.html
  • page/DragController.cpp: (WebCore::DragController::dragEnteredOrUpdated): (WebCore::DragController::tryDocumentDrag): (WebCore::defaultOperationForDrag): (WebCore::DragController::tryDHTMLDrag):
  • page/DragController.h:

06/23/09:

23:40 Changeset [45063] by oliver@apple.com

Add expected results for test i landed earlier

23:26 Changeset [45062] by eric@webkit.org

2009-06-23 Eric Seidel <eric@webkit.org>

Reviewed by Dave Levin.

Fix missing comment message during land-patches and SVN.commit_with_message respecting --dry-run
https://bugs.webkit.org/show_bug.cgi?id=26669

  • Scripts/bugzilla-tool:
  • Scripts/modules/scm.py:
23:25 Changeset [45061] by barraclough@apple.com

Merged r44075:44076 into nitro-extreme branch. (Adds infrastructure for op_method_check, but does not implement optimization.)

23:19 Changeset [45060] by oliver@apple.com

<rdar://problem/6164797> Add Canvas API to allow drawing of <video> frames
<https://bugs.webkit.org/show_bug.cgi?id=25920>

Reviewed by Sam Weinig and Dave Hyatt.

Add support for drawing the contents of the video element to the canvas
in accordance with the current HTML5 draft.

Test: media/video-canvas.html

22:55 Changeset [45059] by ggaren@apple.com

2009-06-23 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.

Bit off a tiny bit more of standardizing opcode behavior to help with result
caching.


SunSpider reports no change, v8 maybe a tiny speedup.

  • jit/JITOpcodes.cpp: (JSC::JIT::emit_op_to_jsnumber): (JSC::JIT::emitSlow_op_to_jsnumber): (JSC::JIT::emit_op_convert_this): (JSC::JIT::emitSlow_op_convert_this):
22:37 Changeset [45058] by barraclough@apple.com

Merged r44030:44075 into nitro-extreme branch.

22:31 Changeset [45057] by ggaren@apple.com

2009-06-23 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.

Bit off a tiny bit more of standardizing opcode behavior to help with result
caching -- including removing my old enemy, op_resolve_function, because
it was non-standard, and removing it felt better than helping it limp along.


SunSpider reports no change, v8 maybe a tiny speedup.


  • bytecode/CodeBlock.cpp: (JSC::CodeBlock::dump):
  • bytecode/Opcode.h:
  • bytecompiler/BytecodeGenerator.cpp:
  • bytecompiler/BytecodeGenerator.h:
  • interpreter/Interpreter.cpp: (JSC::Interpreter::privateExecute):
  • jit/JIT.cpp: (JSC::JIT::privateCompileMainPass):
  • jit/JIT.h:
  • jit/JITOpcodes.cpp: (JSC::JIT::emit_op_get_scoped_var): (JSC::JIT::emit_op_put_scoped_var): (JSC::JIT::emit_op_to_primitive): (JSC::JIT::emitSlow_op_to_primitive):
  • jit/JITStubs.cpp:
  • jit/JITStubs.h:
  • parser/Nodes.cpp: (JSC::FunctionCallResolveNode::emitBytecode):
22:16 Changeset [45056] by barraclough@apple.com

Merged r44029:44030 into nitro-extreme branch.

21:56 Changeset [45055] by barraclough@apple.com

Merged r44000:44029 into nitro-extreme branch.

21:41 Changeset [45054] by barraclough@apple.com

Merged r43853:44000 into nitro-extreme branch.

21:33 Changeset [45053] by barraclough@apple.com

Merged r43848:43853 into nitro-extreme branch.

21:26 Changeset [45052] by barraclough@apple.com

Merged r43781:43848 into nitro-extreme branch.

21:19 Changeset [45051] by ggaren@apple.com

2009-06-23 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.


Bit off a tiny bit of standardizing opcode behavior to help with result
caching.


0.6% SunSpider speedup. 0.3% v8 speedup.

  • jit/JITInlineMethods.h: (JSC::JIT::emitLoad): Accomodate a base register that overlaps with payload by loading tag before payload, to avoid stomping base/payload.
  • jit/JITOpcodes.cpp: (JSC::JIT::emit_op_mov): Abide by the standard "tag in regT1, payload in regT0" semantics.

(JSC::JIT::emit_op_get_global_var):
(JSC::JIT::emit_op_put_global_var): Ditto. Also, removed some irrelevent
loads while I was at it. The global object's "d" pointer never changes
after construction.

21:16 Changeset [45050] by barraclough@apple.com

Merged r43780:43781 into nitro-extreme branch.

20:13 Changeset [45049] by barraclough@apple.com

Merged r43750:43780 into nitro-extreme branch.

20:11 Changeset [45048] by mrowe@apple.com

Merge r44799.

20:11 Changeset [45047] by mrowe@apple.com

Merge r44749.

20:11 Changeset [45046] by mrowe@apple.com

Merge r44519.

20:11 Changeset [45045] by mrowe@apple.com

Merge r44266.

20:11 Changeset [45044] by mrowe@apple.com

Merge r44134.

20:11 Changeset [45043] by mrowe@apple.com

Merge r44133.

20:11 Changeset [45042] by mrowe@apple.com

Merge r44125.

20:11 Changeset [45041] by mrowe@apple.com

Merge r44124.

19:59 Changeset [45040] by barraclough@apple.com

Merged r43666:43750 into nitro-extreme branch.

19:47 Changeset [45039] by oliver@apple.com

<rdar://problem/6992806> REGRESSION: Enumeration can skip new properties in cases of prototypes that have more than 64 (26593)
<https://bugs.webkit.org/show_bug.cgi?id=26593>

Reviewed by Gavin Barraclough.

Add tests to ensure we correctly invalidate caching that depends on structure chains
that include dictionaries.

19:45 WikiStart edited by whzeng@126.com
(diff)
19:41 Changeset [45038] by agl@chromium.org

2009-06-23 Adam Langley <agl@google.com>

Reviewed by Eric Seidel.

Chromium: Fix crash with inherited font-size in <option>

https://bugs.webkit.org/show_bug.cgi?id=26656
http://code.google.com/p/chromium/issues/detail?id=14853

In r42597 (https://bugs.webkit.org/show_bug.cgi?id=25244), I changed
the <select> handing for Chromium to fix a rendering bug. However,
although the font-size is correctly ignored, getRowHeight wasn't
updated and so was calculating the height of the rows in an
inconsistent manner. This can lead to a crash.

  • manual-tests/optgroup-empty-and-nested.html: adding test case for crash
  • platform/chromium/PopupMenuChromium.cpp: (WebCore::PopupListBox::getRowHeight):
19:38 WikiStart edited by whzeng@126.com
Add maxthon's port. (diff)
19:37 Changeset [45037] by barraclough@apple.com

Merged r43660:43666 into nitro-extreme branch.

19:18 Changeset [45036] by barraclough@apple.com

Merged r43642:43660 into nitro-extreme branch.

19:13 Changeset [45035] by barraclough@apple.com

Merged r43641:43642 into nitro-extreme branch.

19:08 Changeset [45034] by barraclough@apple.com

Merged r43619:43641 into nitro-extreme branch.

19:02 Changeset [45033] by barraclough@apple.com

Merged r43618:43619 into nitro-extreme branch.

18:50 Changeset [45032] by eric@webkit.org

2009-06-23 Takeshi Yoshino <tyoshino@google.com>

Reviewed by Mark Rowe.

Bug 26537: Builds from command-line fail if custom build product directory is set and ~/Library/Preferences/xcodebuild.plist exists
https://bugs.webkit.org/show_bug.cgi?id=26537

Let determineBaseProductDir subroutine remove
~/Library/Preferences/xcodebuild.plist. It can prevent xcodebuild from
respecting global settings such as a custom build products directory
(<rdar://problem/5585899>).

  • Scripts/webkitdirs.pm:
18:49 Changeset [45031] by beidson@apple.com

2009-06-23 Brady Eidson <beidson@apple.com>

Patch by Antti Koivisto.
Reviewed by Brady Eidson.

<rdar://problem/6988010> Failed resources not reloaded when built against Tiger SDK

A latent bug that cache revalidation uncovered. When a CachedResource failed to load,
it should always be revalidated upon reload.

  • loader/CachedResource.cpp: (WebCore::CachedResource::canUseCacheValidator): (WebCore::CachedResource::mustRevalidate):
18:45 Changeset [45030] by barraclough@apple.com

Merged r43598:43618 into nitro-extreme branch.

18:40 Changeset [45029] by barraclough@apple.com

Merged r43583:43598 into nitro-extreme branch.

18:36 Changeset [45028] by barraclough@apple.com

Merged r43582:43583 into nitro-extreme branch.

18:33 Changeset [45027] by ojan@chromium.org

2009-06-23 Ojan Vafai <ojan@chromium.org>

Reviewed by Mark Rowe.

Resets the results for a test that was missed in
http://trac.webkit.org/changeset/45016.

  • editing/execCommand/5142012-3-expected.txt:
18:31 Changeset [45026] by eric@webkit.org

2009-06-23 Eric Seidel <eric@webkit.org>

Reviewed by Dave Levin.

Make SCM.run_command smarter, and make all previous
os.system and subprocess.popen use SCM.run_command instead.
https://bugs.webkit.org/show_bug.cgi?id=26666

This makes it easier to handle errors in a standard way throughout all the code.
Since this new code raises by default when the exit_code != 0,
we should prevent future problems of bugzilla-tool continuing after
a git or svn command failed.

  • Scripts/modules/scm.py:
18:29 Changeset [45025] by barraclough@apple.com

Merged r43560:43582 into nitro-extreme branch.

18:14 Changeset [45024] by barraclough@apple.com

Merged r43559:43560 into nitro-extreme branch.

18:00 Changeset [45023] by mrowe@apple.com

Versioning.

17:57 Changeset [45022] by mrowe@apple.com

Versioning.

17:44 Changeset [45021] by beidson@apple.com

2009-06-23 Brady Eidson <beidson@apple.com>

Reviewed by Darin Adler.

<rdar://problem/6950660> REGRESSION: iSale 5.5.3 crashes after I click a template to load

Due to some subtle WebKit changes - presumably some delegate callback behavior - a latent bug in
iSale was uncovered where they aren't properly retaining their FrameLoadDelegate, and we ended up
calling back to a dealloc'ed object.

  • WebView/WebView.mm: (-[WebView _needsAdobeFrameReloadingQuirk]): Use more intelligent C++-style initialization. (-[WebView _needsKeyboardEventDisambiguationQuirks]): Ditto. (-[WebView _needsFrameLoadDelegateRetainQuirk]): YES for iSale versions under 5.6 (-[WebView dealloc]): Release the delegate if the quirk is in effect. (-[WebView setFrameLoadDelegate:]): Retain the new delegate if the quirk is in effect.
17:41 Changeset [45020] by barraclough@apple.com

Merged r43558:43559 into nitro-extreme branch.

17:40 Changeset [45019] by mitz@apple.com

Reviewed by Darin Adler.

  • fix <rdar://problem/6634228> Crash in [NSScroller mouseDown:] beneath EventHandler::passMouseDownEventToWidget()
  • page/mac/EventHandlerMac.mm: (WebCore::EventHandler::passMouseDownEventToWidget): Surround the -mouseDown: call with calls to Widget::beforeMouseDown() and Widget::afterMouseDown(), which prevent the widget's view hierarchy from being removed from its parent view while the scroller is tracking the mouse.
17:39 Changeset [45018] by weinig@apple.com

WebCore:

2009-06-23 Sam Weinig <sam@webkit.org>

Reviewed by Dave Hyatt.

Fix for https://bugs.webkit.org/show_bug.cgi?id=26516
Add initial implementation of DataGridDataSource

  • Initial implementation only supports the initialize callback function.

Test: fast/dom/HTMLDataGridElement/DataGridDataSource-basic.html

  • WebCore.xcodeproj/project.pbxproj:
  • GNUmakefile.am:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:
  • bindings/js/JSDataGridDataSource.cpp: Added. (WebCore::JSDataGridDataSource::JSDataGridDataSource): (WebCore::JSDataGridDataSource::~JSDataGridDataSource): (WebCore::JSDataGridDataSource::initialize):
  • bindings/js/JSDataGridDataSource.h: Added. (WebCore::JSDataGridDataSource::create): (WebCore::JSDataGridDataSource::isJSDataGridDataSource): (WebCore::JSDataGridDataSource::jsDataSource): (WebCore::asJSDataGridDataSource):
  • bindings/js/JSHTMLDataGridElementCustom.cpp: Added. (WebCore::JSHTMLDataGridElement::listener): (WebCore::JSHTMLDataGridElement::setListener):
  • html/DataGridDataSource.h: Added. (WebCore::DataGridDataSource::~DataGridDataSource): (WebCore::DataGridDataSource::isJSDataGridDataSource):
  • html/HTMLDataGridElement.cpp: (WebCore::HTMLDataGridElement::HTMLDataGridElement): (WebCore::HTMLDataGridElement::setDataSource): (WebCore::HTMLDataGridElement::initializationTimerFired):
  • html/HTMLDataGridElement.h: (WebCore::HTMLDataGridElement::dataSource):
  • html/HTMLDataGridElement.idl:

LayoutTests:

2009-06-23 Sam Weinig <sam@webkit.org>

Reviewed by Dave Hyatt.

Test for https://bugs.webkit.org/show_bug.cgi?id=26516
Add initial implementation of DataGridDataSource

  • fast/dom/HTMLDataGridElement: Added.
  • fast/dom/HTMLDataGridElement/DataGridDataSource-basic-expected.txt: Added.
  • fast/dom/HTMLDataGridElement/DataGridDataSource-basic.html: Added.
17:09 Changeset [45017] by barraclough@apple.com

2009-06-23 Gavin Barraclough <barraclough@apple.com>

Reviewed by Sam Weinig.

Remove 'arguments' field from Register union (again).
This time do so without breaking tests (radical, I know).

  • interpreter/CallFrame.h: (JSC::ExecState::optionalCalleeArguments): (JSC::ExecState::setArgumentCount): (JSC::ExecState::init):
  • interpreter/Interpreter.cpp: (JSC::Interpreter::dumpRegisters): (JSC::Interpreter::unwindCallFrame): (JSC::Interpreter::privateExecute): (JSC::Interpreter::retrieveArguments):
  • interpreter/Register.h: (JSC::Register::withInt): (JSC::Register::): (JSC::Register::Register): (JSC::Register::i):
  • jit/JITStubs.cpp: (JSC::JITStubs::cti_op_tear_off_arguments):
  • runtime/Arguments.h: (JSC::JSActivation::copyRegisters): (JSC::Register::arguments):
  • runtime/JSActivation.cpp: (JSC::JSActivation::argumentsGetter):
  • runtime/JSActivation.h:
16:56 Changeset [45016] by jparent@chromium.org

WebCore:

2009-06-23 Ryosuke Niwa <rniwa@google.com>

Reviewed by Justin Garcia.

Undo the changeset 21212 to fix the regression bug 14062.
The changeset 21212 attempted to fix rdar://problem/5002441.
The changeset 21212 modified InsertTextCommand::input so as to convert
all spaces in text to non-breaking spaces for rdar://problem/5002441.
However, the bug was originally caused by rebalanceWhitespaceAt
and this workaround introduced a regression bug 14062.
Because rebalanceWhitespaceAt appears to behave correctly now,
the workaround introduced in 21212 is no longer needed.

Test: editing/inserting/space-after-removeformat.html

  • editing/InsertTextCommand.cpp: (WebCore::InsertTextCommand::input):

LayoutTests:

2009-06-23 Ryosuke Niwa <rniwa@google.com>

Reviewed by Justin Garcia.

Undo the changeset 21212 to fix the regression bug 14062.
The changeset 21212 attempted to fix rdar://problem/5002441.
The changeset 21212 modified InsertTextCommand::input so as to convert
all spaces in text to non-breaking spaces for rdar://problem/5002441.
However, the bug was originally caused by rebalanceWhitespaceAt
and this workaround introduced a regression bug 14062.
Because rebalanceWhitespaceAt appears to behave correctly now,
the workaround introduced in 21212 is no longer needed.

The following test cases are affected by this change.
This patch removes unappropriate non-breaking spaces (&nbsp;) and
inserts normal spaces instead.

  • editing/deleting/2610675-1-expected.txt: Replaced.
  • editing/deleting/2610675-2-expected.txt: Replaced.
  • editing/deleting/2610675-3-expected.txt: Replaced.
  • editing/inserting/insert-before-link-1-expected.txt: Replaced.
  • editing/inserting/space-after-removeformat-expected.txt: Added.
  • editing/inserting/space-after-removeformat.html: Added.
  • platform/mac/editing/execCommand/5482023-expected.checksum: Replaced.
  • platform/mac/editing/execCommand/5482023-expected.png: Replaced.
  • platform/mac/editing/execCommand/5482023-expected.txt: Replaced.
  • platform/mac/editing/input/text-input-controller-expected.txt: Replaced.
  • platform/mac/editing/inserting/editable-html-element-expected.checksum: Replaced.
  • platform/mac/editing/inserting/editable-html-element-expected.png: Replaced.
  • platform/mac/editing/inserting/editable-html-element-expected.txt: Replaced.
  • platform/mac/editing/pasteboard/4989774-expected.checksum: Replaced.
  • platform/mac/editing/pasteboard/4989774-expected.png: Replaced.
  • platform/mac/editing/pasteboard/4989774-expected.txt: Replaced.
  • platform/mac/editing/selection/4983858-expected.checksum: Replaced.
  • platform/mac/editing/selection/4983858-expected.png: Replaced.
  • platform/mac/editing/selection/4983858-expected.txt: Replaced.
16:42 Changeset [45015] by bdakin@apple.com

WebCore:

2009-06-23 Beth Dakin <bdakin@apple.com>

Reviewed by Dave Hyatt.

Fix for https://bugs.webkit.org/show_bug.cgi?id=26523 <col>
elements width can't be changed with javascript

This patch both fixes setting the width attribute directly in
JavaScript, and setting the width on the style attribute.

  • html/HTMLTableColElement.cpp: (WebCore::HTMLTableColElement::parseMappedAttribute): If we have a new width, call setNeedsLayoutAndPrefWidthsRecalc()

Call calcPrefWidths() on the table cols.

  • rendering/AutoTableLayout.cpp: (WebCore::AutoTableLayout::recalcColumn):
  • rendering/FixedTableLayout.cpp: (WebCore::FixedTableLayout::calcWidthArray):


Call layoutIfNeeded on the table cols.

  • rendering/RenderTable.cpp: (WebCore::RenderTable::layout):
  • rendering/RenderTableCol.cpp: (WebCore::RenderTableCol::clippedOverflowRectForRepaint): Use new table() convenience function. (WebCore::RenderTableCol::calcPrefWidths): Just call setPrefWidthsDirty(false) on ourself and our children. (WebCore::RenderTableCol::table):
  • rendering/RenderTableCol.h:

LayoutTests:

2009-06-23 Beth Dakin <bdakin@apple.com>

Reviewed by Dave Hyatt.

Test for https://bugs.webkit.org/show_bug.cgi?id=26523 <col>
elements width can't be changed with javascript

  • fast/dom/HTMLTableColElement: Added.
  • fast/dom/HTMLTableColElement/resize-table-using-col-width.html: Added.
  • platform/mac/fast/dom/HTMLTableColElement: Added.
  • platform/mac/fast/dom/HTMLTableColElement/resize-table-using-col-width-expected.checksum: Added.
  • platform/mac/fast/dom/HTMLTableColElement/resize-table-using-col-width-expected.png: Added.
  • platform/mac/fast/dom/HTMLTableColElement/resize-table-using-col-width-expected.txt: Added.
16:35 Changeset [45014] by ojan@chromium.org

WebCore:

2009-06-23 Ojan Vafai <ojan@chromium.org>

Reviewed by Dan Bernstein.

Add logic to CSSStyleDeclaration::diff to deal with font-sizes that are
keyword values. When diff is called on a CSSStyleDeclaration, we check
the keywordSize to see if font-size matches a keyword value.

This ensures that when we diff a CSSMutableStyleDeclaration returned from
copyInheritableProperties on a CSSComputedStyleDeclaration that we
correctly identify matching font-sizes.

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

Test: editing/inserting/font-size-clears-from-typing-style.html

  • css/CSSComputedStyleDeclaration.cpp: (WebCore::CSSComputedStyleDeclaration::cssPropertyMatches):
  • css/CSSComputedStyleDeclaration.h:
  • css/CSSStyleDeclaration.cpp: (WebCore::CSSStyleDeclaration::cssPropertyMatches): (WebCore::CSSStyleDeclaration::diff):
  • css/CSSStyleDeclaration.h:

LayoutTests:

2009-06-23 Ojan Vafai <ojan@chromium.org>

Reviewed by Dan Bernstein.

This test hits an edge case where typingStyle would never get cleared.
In addition to making every text insertion go into its own text node,
this caused large performance problems.

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

  • editing/execCommand/5770834-1-expected.txt:

These results match how they were before r43243. It's not clear to me
why changes to font-size are affecting text-align styling though.

  • editing/inserting/font-size-clears-from-typing-style-expected.txt: Added.
  • editing/inserting/font-size-clears-from-typing-style.html: Added.
  • editing/inserting/resources/TEMPLATE.html: Copied from LayoutTests/editing/execCommand/resources/TEMPLATE.html.
  • editing/inserting/resources/font-size-clears-from-typing-style.js: Added.
  • platform/mac/editing/execCommand/remove-formatting-2-expected.txt:
16:18 Changeset [45013] by ggaren@apple.com

2009-06-23 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.


Removed some result register tracking cruft in preparation for a new
result tracking mechanism.


SunSpider reports no change.

  • assembler/AbstractMacroAssembler.h:
  • assembler/X86Assembler.h: (JSC::X86Assembler::JmpDst::JmpDst): No need to track jump targets in machine code; we already do this in bytecode.
  • jit/JIT.cpp: (JSC::JIT::JIT): (JSC::JIT::emitTimeoutCheck): Make sure to save and restore the result registers, so an opcode with a timeout check can still benefit from result register caching.

(JSC::JIT::privateCompileMainPass):
(JSC::JIT::privateCompileSlowCases): Removed calls to killLastResultRegister()
in preparation for something new.

  • jit/JIT.h:
  • jit/JITArithmetic.cpp: (JSC::JIT::emit_op_jnless): (JSC::JIT::emit_op_jnlesseq):
  • jit/JITInlineMethods.h: (JSC::JIT::emitGetFromCallFrameHeaderPtr): (JSC::JIT::emitGetFromCallFrameHeader32):
  • jit/JITOpcodes.cpp: (JSC::JIT::emit_op_jmp): (JSC::JIT::emit_op_jfalse): (JSC::JIT::emit_op_jtrue): (JSC::JIT::emit_op_jeq_null): (JSC::JIT::emit_op_jneq_null): (JSC::JIT::emit_op_jneq_ptr): (JSC::JIT::emit_op_jsr): (JSC::JIT::emit_op_sret): (JSC::JIT::emit_op_jmp_scopes): ditto
  • jit/JITStubCall.h: (JSC::JITStubCall::JITStubCall): (JSC::JITStubCall::getArgument): added a mechanism for reloading an argument you passed to a JIT stub, for use in emitTimeoutCheck.
16:10 Changeset [45012] by treat@webkit.org

2009-06-23 Joe Mason <joe.mason@torchmobile.com>

Reviewed by Adam Treat.

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

  • Scripts/prepare-ChangeLog: Added --git-index mode to list only the changes which are already staged in the index. Useful to create an entry for what you're about to commit while ignoring unstaged changes.
16:06 Changeset [45011] by treat@webkit.org

2009-06-23 Yong Li <yong.li@torchmobile.com>

Reviewed by George Staikos.

https://bugs.webkit.org/show_bug.cgi?id=26654
Add the proper export define for the JavaScriptCore API when building for WINCE.

15:38 Changeset [45010] by andersca@apple.com

2009-06-23 Anders Carlsson <andersca@apple.com>

Reviewed by Darin Adler.

  • Fix <rdar://problem/6965672>


Defer calls to WKPCInvalidateRect, so we don't try to invalidate while waiting for a reply, since
that is too early.


  • Plugins/Hosted/NetscapePluginHostProxy.h: (WebKit::NetscapePluginHostProxy::isProcessingRequests):
  • Plugins/Hosted/NetscapePluginHostProxy.mm: (WebKit::NetscapePluginHostProxy::NetscapePluginHostProxy): (WebKit::NetscapePluginHostProxy::processRequests): (WKPCInvalidateRect):
15:25 Changeset [45009] by kmccullough@apple.com

WebCore:

2009-06-23 Kevin McCullough <kmccullough@apple.com>

Reviewed by Darin Adler.

<rdar://problem/6994790> CrashTracer: [USER] 8 crashes in Mail trying to
add a blank line to pasted Facebook content (under
CompositeEditCommand::positionAvoidingSpecialElementBoundary)

  • Just added null checks and I will create a new bug to prevent the bad behavior.
  • editing/CompositeEditCommand.cpp: (WebCore::CompositeEditCommand::positionAvoidingSpecialElementBoundary):

LayoutTests:

2009-06-23 Kevin McCullough <kmccullough@apple.com>

Reviewed by Darin Adler.

<rdar://problem/6994790> CrashTracer: [USER] 8 crashes in Mail trying to
add a blank line to pasted Facebook content (under
CompositeEditCommand::positionAvoidingSpecialElementBoundary)

  • editing/inserting/return-with-object-element-expected.txt: Added.
  • editing/inserting/return-with-object-element.html: Added.
14:51 Changeset [45008] by weinig@apple.com

2009-06-23 Sam Weinig <sam@webkit.org>

Reviewed by Geoffrey Garen.

Remove now-useless inplace variants of binary ops.

  • jit/JIT.h:
  • jit/JITArithmetic.cpp: (JSC::JIT::emit_op_bitand): (JSC::JIT::emit_op_bitor): (JSC::JIT::emit_op_bitxor): (JSC::JIT::emit_op_add): (JSC::JIT::emit_op_sub): (JSC::JIT::emit_op_mul):
14:48 Changeset [45007] by simon.fraser@apple.com

2009-06-23 Simon Fraser <simon.fraser@apple.com>

Build fix.

Wrap shouldDoSoftwarePaint() in #if USE(ACCELERATED_COMPOSITING) to avoid warning when
ACCELERATED_COMPOSITING is not defined.

  • rendering/RenderLayer.cpp:
14:43 Changeset [45006] by weinig@apple.com

2009-06-23 Sam Weinig <sam@webkit.org>

Reviewed by Geoffrey Garen.

Move off memory operands to aid in re-enabling result caching.

  • No regression measured.
  • jit/JIT.h:
  • jit/JITArithmetic.cpp: (JSC::JIT::emit_op_negate): (JSC::JIT::emit_op_jnless): (JSC::JIT::emit_op_jnlesseq): (JSC::JIT::emit_op_lshift): (JSC::JIT::emit_op_rshift): (JSC::JIT::emit_op_bitand): (JSC::JIT::emitBitAnd32Constant): (JSC::JIT::emitBitAnd32InPlace): (JSC::JIT::emit_op_bitor): (JSC::JIT::emitBitOr32Constant): (JSC::JIT::emitBitOr32InPlace): (JSC::JIT::emit_op_bitxor): (JSC::JIT::emitBitXor32Constant): (JSC::JIT::emitBitXor32InPlace): (JSC::JIT::emit_op_bitnot): (JSC::JIT::emit_op_post_inc): (JSC::JIT::emit_op_post_dec): (JSC::JIT::emit_op_pre_inc): (JSC::JIT::emitSlow_op_pre_inc): (JSC::JIT::emit_op_pre_dec): (JSC::JIT::emitSlow_op_pre_dec): (JSC::JIT::emit_op_add): (JSC::JIT::emitAdd32Constant): (JSC::JIT::emitAdd32InPlace): (JSC::JIT::emitSlow_op_add): (JSC::JIT::emitSlowAdd32Constant): (JSC::JIT::emit_op_sub): (JSC::JIT::emitSlow_op_sub): (JSC::JIT::emitSub32ConstantLeft): (JSC::JIT::emitSub32ConstantRight): (JSC::JIT::emitSub32InPlaceLeft): (JSC::JIT::emitSub32InPlaceRight): (JSC::JIT::emitBinaryDoubleOp): (JSC::JIT::emit_op_mul): (JSC::JIT::emitMul32InPlace): (JSC::JIT::emit_op_div): (JSC::JIT::emit_op_mod):
  • jit/JITCall.cpp: (JSC::JIT::compileOpCallVarargs):
  • jit/JITOpcodes.cpp: (JSC::JIT::emit_op_loop_if_less): (JSC::JIT::emit_op_loop_if_lesseq): (JSC::JIT::emit_op_instanceof): (JSC::JIT::emit_op_to_primitive): (JSC::JIT::emit_op_not): (JSC::JIT::emit_op_jneq_ptr): (JSC::JIT::emit_op_eq): (JSC::JIT::emit_op_neq): (JSC::JIT::emit_op_to_jsnumber):
  • jit/JITPropertyAccess.cpp: (JSC::JIT::emit_op_get_by_val): (JSC::JIT::emit_op_put_by_val):
14:25 Changeset [45005] by simon.fraser@apple.com

2009-06-23 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dave Hyatt.

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

Fix reflection painting on elements with compositing layers.

Test: compositing/reflections/reflection-on-composited.html

  • rendering/RenderLayer.h: Add a new PaintLayerPaintingReflection flag which is set while painting inside reflections. (WebCore::RenderLayer::setPaintingInsideReflection): Add a setter so that RenderLayerBacking can call it.
  • rendering/RenderLayer.cpp: (WebCore::shouldDoSoftwarePaint): Utility to clarify the logic of when a composited layer should do a software paint.

(WebCore::RenderLayer::paintLayer):
Use shouldDoSoftwarePaint() to determine when to software-paint a reflected layer.
Remove transform-related tests from the reflection-painting if test because they appear
to be unnecessary.

  • rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::paintIntoLayer): Add the code to paint reflections to the composited layer painting method.
  • rendering/RenderReplica.cpp: (WebCore::RenderReplica::paint): Set the PaintLayerPaintingReflection flag.
13:26 Changeset [45004] by ggaren@apple.com

2009-06-23 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.


Fixed some missing and/or misplaced labels in bytecode generation, so
we don't have to work around them in JIT code generation.

  • bytecompiler/BytecodeGenerator.cpp: (JSC::BytecodeGenerator::emitJumpSubroutine):
  • parser/Nodes.cpp: (JSC::TryNode::emitBytecode):
13:01 Changeset [45003] by antti@apple.com

WebCore:

2009-06-23 Antti Koivisto <antti@apple.com>

Reviewed by Darin Adler.


https://bugs.webkit.org/show_bug.cgi?id=26643
Memory cache should respect Cache-Control: no-store


Make resources with Cache-Control: no-store uncacheable in the memory cache.
They get evicted immediately when all clients are gone and are always
reloaded from the server.

Conforms to RFC2616 14.9.2 and seems also match Firefox behavior.

  • loader/CachedResource.cpp: (WebCore::CachedResource::removeClient): (WebCore::CachedResource::canUseCacheValidator): (WebCore::CachedResource::mustRevalidate):
  • platform/network/ResourceResponseBase.cpp: (WebCore::ResourceResponseBase::ResourceResponseBase): (WebCore::ResourceResponseBase::parseCacheControlDirectives): (WebCore::ResourceResponseBase::cacheControlContainsNoStore):
  • platform/network/ResourceResponseBase.h:

LayoutTests:

2009-06-23 Antti Koivisto <antti@apple.com>

Reviewed by Darin Adler.


https://bugs.webkit.org/show_bug.cgi?id=26643
Memory cache should respect Cache-Control: no-store


Update the test to cover no-store.

  • http/tests/cache/subresource-expiration-expected.txt:
  • http/tests/cache/subresource-expiration.html:
12:48 Changeset [45002] by beidson@apple.com

Windows build fix

12:46 Changeset [45001] by simon.fraser@apple.com

2009-06-23 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dave Hyatt

First part of https://bugs.webkit.org/show_bug.cgi?id=26652

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::paint): (WebCore::RenderLayer::paintLayer):
  • rendering/RenderLayer.h: (WebCore::RenderLayer::):
  • rendering/RenderReplica.cpp: (WebCore::RenderReplica::paint): Replace the 3 boolean params to RenderLayer::paintLayer() with a bit mask for readability.
  • rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::paintIntoLayer): (WebCore::RenderLayerBacking::paintContents):
  • rendering/RenderLayerBacking.h: Remove the unused 'haveTransparency' param to paintIntoLayer().
12:12 Changeset [45000] by kmccullough@apple.com

WebCore:

2009-06-23 Kevin McCullough <kmccullough@apple.com>

Reviewed by Adele Peterson.

<rdar://problem/6902203> REGRESSION: 'Return' insertion point incorrect
after manually quoting blank line then hitting return

If the insertion point is both the begining and ending visible position
then we need to insert at the end instead of the begining, which is what
would normally happen if the insertion point was as the begining.

  • editing/BreakBlockquoteCommand.cpp: (WebCore::BreakBlockquoteCommand::doApply):

LayoutTests:

2009-06-23 Kevin McCullough <kmccullough@apple.com>

Reviewed by Adele Peterson.

<rdar://problem/6902203> REGRESSION: 'Return' insertion point incorrect
after manually quoting blank line then hitting return

  • editing/inserting/insert-br-quoted-007-expected.txt: Added.
  • editing/inserting/insert-br-quoted-007.html: Added.
12:05 Changeset [44999] by beidson@apple.com

2009-06-23 Brady Eidson <beidson@apple.com>

Reviewed by Sam Weinig, and looked over by Eric Carlson.

<rdar://problem/6980466> Media elements should provide option for changing pitch correction

Introduce a new boolean property to HTMLMediaElement called webkitPreservesPitch for script to control this.

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::HTMLMediaElement): Currently the platforms that have a flexible pitch preservation property (Quicktime on Mac and Win) have had it on by default, so continue that default for now. (WebCore::HTMLMediaElement::loadResource): (WebCore::HTMLMediaElement::webkitPreservesPitch): (WebCore::HTMLMediaElement::setWebkitPreservesPitch):
  • html/HTMLMediaElement.h:
  • html/HTMLMediaElement.idl:
  • platform/graphics/MediaPlayer.cpp: (WebCore::NullMediaPlayerPrivate::setPreservesPitch): (WebCore::MediaPlayer::MediaPlayer): (WebCore::MediaPlayer::preservesPitch): (WebCore::MediaPlayer::setPreservesPitch):
  • platform/graphics/MediaPlayer.h:
  • platform/graphics/MediaPlayerPrivate.h: (WebCore::MediaPlayerPrivateInterface::setPreservesPitch):
  • platform/graphics/mac/MediaPlayerPrivateQTKit.h:
  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::MediaPlayerPrivate): (WebCore::MediaPlayerPrivate::createQTMovie): Rework to allow recreation of the movie with almost the exact same attributes as before, with pitch preservation being the difference. (WebCore::MediaPlayerPrivate::setPreservesPitch): Recreate the QTMovie if the flag actually changes. (WebCore::MediaPlayerPrivate::updateStates): Restore current time and rate if this is a recreated movie. (WebCore::MediaPlayerPrivate::timeChanged): Clear the time-to-be-restored, as this callback means it has been effectively overwritten.
  • platform/graphics/win/MediaPlayerPrivateQuickTimeWin.cpp: (WebCore::MediaPlayerPrivate::load): (WebCore::MediaPlayerPrivate::setPreservesPitch):
  • platform/graphics/win/MediaPlayerPrivateQuickTimeWin.h:
  • platform/graphics/win/QTMovieWin.cpp: (QTMovieWinPrivate::QTMovieWinPrivate): (QTMovieWinPrivate::~QTMovieWinPrivate): (QTMovieWinPrivate::task): Restore current time and rate if this is a re-created movie. (QTMovieWin::play): Clear the time-to-be-restored, as this effectively overwrites it. (QTMovieWin::pause): Ditto. (QTMovieWin::setRate): Ditto. (QTMovieWin::setCurrentTime): Ditto. (QTMovieWin::setPreservesPitch): Recreate the Movie if the flag actually changes. (QTMovieWin::load): Rework to allow recreation of the Movie with similar state as the movie before it, but changing the pitch preservation flag.
  • platform/graphics/win/QTMovieWin.h:
12:04 Changeset [44998] by eric.carlson@apple.com

2009-06-23 Eric Carlson <eric.carlson@apple.com>

Reviewed by Simon Fraser.

<rdar://problem/6998286> Crash after leaving page with <video>


Don't try to post events after going into the page cache.

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::documentWillBecomeInactive): Cancel pending events and stop event timers so we won't try to post events after going into the cache.
11:52 Changeset [44997] by darin@chromium.org

2009-06-23 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Brady Eidson.

Simplify the interaction between LocalStorageArea/SessionStorageArea
and StorageArea by creating a "blockUntilImportComplete()" function
rather than bouncing back and forth between the child and parent
classes in a somewhat unintuitive manner.

This patch also paves the way for LocalStorageArea and
SessionStorageArea being merged into one. It's part of several in a
set which are separating syncing (to disk) code from the rest of the
implementation so that local storage and session storage's code can be
unified.

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

  • storage/LocalStorageArea.cpp: (WebCore::LocalStorageArea::blockUntilImportComplete):
  • storage/LocalStorageArea.h:
  • storage/StorageArea.cpp: (WebCore::StorageArea::length): (WebCore::StorageArea::key): (WebCore::StorageArea::getItem): (WebCore::StorageArea::setItem): (WebCore::StorageArea::removeItem): (WebCore::StorageArea::clear): (WebCore::StorageArea::contains):
  • storage/StorageArea.h: (WebCore::StorageArea::blockUntilImportComplete):
11:36 Changeset [44996] by simon.fraser@apple.com

2009-06-23 Simon Fraser <simon.fraser@apple.com>

Reviewed by Darin Adler.

<rdar://problem/6997929> Assertion at WebCore::RenderLayer::updateClipRects with hardware acceleration

The updatingControlTints() painting pass goes through compositing layer boundaries,
so we need to ensure we don't cache clip rects computed with the wrong root by
setting temporaryClipRects to true.

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::paintLayer):
  • rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::contentOffsetInCompostingLayer): Remove a printf that I left in by mistake.
11:03 Changeset [44995] by andersca@apple.com

WebCore:

2009-06-23 Anders Carlsson <andersca@apple.com>

Reviewed by Darin Adler.

WebCore side of <rdar://problem/6946094>.


  • platform/win/ClipboardWin.cpp: (WebCore::ClipboardWin::setExternalDataObject):
  • platform/win/ClipboardWin.h:

WebKit/win:

2009-06-23 Anders Carlsson <andersca@apple.com>

Reviewed by Darin Adler.

WebKit side of <rdar://problem/6946094>.

  • Interfaces/IWebUIDelegate.idl: Add a newPasteboard out parameter to willPerformDragSourceAction.


  • WebCoreSupport/WebDragClient.cpp: (WebDragClient::willPerformDragSourceAction): Implement this. If the client returns a new pasteboard, use it for the drag.


  • WebCoreSupport/WebInspectorDelegate.h: (WebInspectorDelegate::willPerformDragSourceAction): Add the new parameter.

WebKitTools:

2009-06-23 Anders Carlsson <andersca@apple.com>

Reviewed by Darin Adler.


Update for WebKit changes.


  • DumpRenderTree/win/UIDelegate.h: (UIDelegate::willPerformDragSourceAction):
10:29 Changeset [44994] by darin@chromium.org

2009-06-23 Darin Fisher <darin@chromium.org>

Reviewed by Darin Adler.

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

Add setFormContentType and setFormData since it is awkward for Chromium
to use setFormInfoFromRequest to set the underlying members. Also, add
a clearChildren method that Chromium needs. These methods are used as
part of the Chromium code that serializes a HistoryItem for storing in
the browser's session history.

  • history/HistoryItem.cpp:
  • history/HistoryItem.h:
08:45 Changeset [44993] by treat@webkit.org

2009-06-23 Joe Mason <joe.mason@torchmobile.com>

Reviewed by Adam Treat.

Authors: Yong Li <yong.li@torchmobile.com>, Joe Mason <joe.mason@torchmobile.com>

https://bugs.webkit.org/show_bug.cgi?id=26611
Implement currentThreadStackBase on WINCE by adding a global,
g_stackBase, which must be set to the address of a local variable
by the caller before calling any WebKit function that invokes JSC.

08:44 Changeset [44992] by eric.carlson@apple.com

2009-06-23 Eric Carlson <eric.carlson@apple.com>

Reviewed by NOBODY, layout tests fix.


'timeupdate' events are, by definition, timing dependent so it isn't
a good idea to log them in layout test results.

  • media/event-attributes-expected.txt:
  • media/event-attributes.html:
07:49 Changeset [44991] by eric.carlson@apple.com

2009-06-23 Eric Carlson <eric.carlson@apple.com>

Reviewed by Darin Adler.

<rdar://problem/6978938> Need HTML 5 event attributes for media elements

Test: media/event-attributes.html

  • html/HTMLAttributeNames.in: Add media event attribute names.
  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::parseMappedAttribute): Handle media event attributes.

2009-06-23 Eric Carlson <eric.carlson@apple.com>

Reviewed by Darin Adler.


<rdar://problem/6978938> Need HTML 5 event attributes for media elements

  • media/event-attributes-expected.txt: Added.
  • media/event-attributes.html: Added.
  • media/progress-event-total.html: Cleanup, use event handler attributes.
  • media/progress-event.html: Ditto.
  • media/video-size-intrinsic-scale.html: Remove duplicate 'waitForEvent("ratechange")'
  • media/video-source-add-src.html: Ditto.
  • media/video-source-error.html: Ditto.
  • media/video-timeupdate-during-playback.html: Ditto.
  • platform/gtk/Skipped: Skip new test.
06:03 Changeset [44990] by hausmann@webkit.org

2009-06-23 Thiago Macieira <thiago.macieira@nokia.com>

Reviewed by Simon Hausmann.

Fix Qt build with Phonon.

The #include <phonon> header no longer exists. And the <Phonon> or
<Phonon/Phonon> headers have never existed (neither for us nor for the
Phonon sources). You have to select each and every header that you do
want now.

05:43 Changeset [44989] by vestbo@webkit.org

2009-06-23 Tor Arne Vestbø <tor.arne.vestbo@nokia.com>

Reviewed by Simon Hausmann.

Remove duplicated code in Qt & GTK logging initialization

We now share the getChannelFromName() function in Logging.cpp

  • platform/gtk/LoggingGtk.cpp:
  • platform/qt/LoggingQt.cpp:
05:43 Changeset [44988] by vestbo@webkit.org

2009-06-23 Tor Arne Vestbø <tor.arne.vestbo@nokia.com>

Reviewed by Simon Hausmann.

Rename the plugin logging channel to plural form 'Plugins'

This matches both the other logging channel names, as well as the
WebKit logging channel used by Safari/Mac for plugins.

  • platform/Logging.cpp:
  • platform/Logging.h:
  • platform/gtk/LoggingGtk.cpp:
  • platform/mac/LoggingMac.mm:
  • platform/qt/LoggingQt.cpp:
  • platform/win/LoggingWin.cpp:
  • plugins/PluginDebug.h:
  • plugins/gtk/PluginPackageGtk.cpp:
  • plugins/mac/PluginPackageMac.cpp:
  • plugins/mac/PluginViewMac.cpp:
  • plugins/qt/PluginPackageQt.cpp:
03:17 Changeset [44987] by vestbo@webkit.org

2009-06-23 Tor Arne Vestbø <tor.arne.vestbo@nokia.com>

Reviewed by Simon Hausmann.

Fix NPAPI mouse translation issues on Mac

The WindowRef we pass to the plugin refers to the the top level window,
so the x and y positions have to be relative to this position, but we
have to manually compensate for title bar decorations and other parents
of the QWebView since contentsToWindow() only traverses to the QWebView.

Previously we did this compensation when passing on mouse coordinates to
the plugin, which caused various issues with translations such as not
being able to close the Flash settings dialog, or the hand cursor not
appearing over links.

We now do the compensation as part of the call to NPP_SetWindow, and
then pass mouse coordinates in global context without any compensation,
similar to how both Safari and Mozilla does it.

03:11 Changeset [44986] by vestbo@webkit.org

2009-06-23 Tor Arne Vestbø <tor.arne.vestbo@nokia.com>

Rubber-stamped by Simon Hausmann.

[Qt] Resolve absolute path to library install_name manually

Since our target.path is set to the Qt directory (to be able
to do 'make install') the 'absolute_library_soname' option
to QMake did not do the right thing.

We now compute the absolute path manually using DESTDIR.

  • WebCore.pro:
02:17 Changeset [44985] by abarth@webkit.org

2009-06-23 Adam Barth <abarth@webkit.org>

Reviewed by Sam Weinig.

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

More tests for the XSSAuditor. (I converted the server side to Perl
after Sam reviewed the patch because the PHP scripts didn't agree with
the Tiger and Windows buildbots.)

  • http/tests/security/xssAuditor/img-onerror-tricky-expected.txt: Added.
  • http/tests/security/xssAuditor/img-onerror-tricky.html: Added.
  • http/tests/security/xssAuditor/link-onclick-expected.txt: Added.
  • http/tests/security/xssAuditor/link-onclick.html: Added.
  • http/tests/security/xssAuditor/property-escape-expected.txt: Added.
  • http/tests/security/xssAuditor/property-escape.html: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag-post-and-notify.pl: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag-post.pl: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag-utf-7.pl: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag.php: Removed.
  • http/tests/security/xssAuditor/resources/echo-intertag.pl: Added.
  • http/tests/security/xssAuditor/resources/echo-property.pl: Added.
  • http/tests/security/xssAuditor/resources/redir.php: Added.
  • http/tests/security/xssAuditor/resources/xss.js: Added.
  • http/tests/security/xssAuditor/script-tag-convoluted-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-convoluted.html: Added.
  • http/tests/security/xssAuditor/script-tag-open-redirect-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-open-redirect.html: Added.
  • http/tests/security/xssAuditor/script-tag-post-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-post.html: Added.
  • http/tests/security/xssAuditor/script-tag-redirect-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-redirect.html: Added.
  • http/tests/security/xssAuditor/script-tag-utf-7-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-utf-7.html: Added.
  • http/tests/security/xssAuditor/script-tag-with-source-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-with-source.html: Added.
  • http/tests/security/xssAuditor/script-tag.html:
02:14 Changeset [44984] by oliver@apple.com

Fix stupid performance problem in the LiteralParser

Reviewed by Alexey Proskuryakov.

The LiteralParser was making a new UString in order to use
toDouble, however UString's toDouble allows a much wider range
of numberic strings than the LiteralParser accepts, and requires
an additional heap allocation or two for the construciton of the
UString. To rectify this we just call WTF::dtoa directly using
a stack allocated buffer to hold the validated numeric literal.

02:11 Changeset [44983] by abarth@webkit.org

Revert 44981. The buildbots hate me.

02:01 Changeset [44982] by eric@webkit.org

2009-06-23 Eric Seidel <eric@webkit.org>

Reviewed by Mark Rowe.

  • Scripts/modules/scm.py: Fix commit_with_message to return the commit output.
01:51 Changeset [44981] by abarth@webkit.org

2009-06-23 Adam Barth <abarth@webkit.org>

Reviewed by Sam Weinig.

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

More tests for the XSSAuditor. (I converted the server side to Perl
after Sam reviewed the patch because the PHP scripts didn't agree with
the Tiger and Windows buildbots.)

  • http/tests/security/xssAuditor/img-onerror-tricky-expected.txt: Added.
  • http/tests/security/xssAuditor/img-onerror-tricky.html: Added.
  • http/tests/security/xssAuditor/link-onclick-expected.txt: Added.
  • http/tests/security/xssAuditor/link-onclick.html: Added.
  • http/tests/security/xssAuditor/property-escape-expected.txt: Added.
  • http/tests/security/xssAuditor/property-escape.html: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag-post-and-notify.pl: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag-post.pl: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag-utf-7.pl: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag.php: Removed.
  • http/tests/security/xssAuditor/resources/echo-intertag.pl: Added.
  • http/tests/security/xssAuditor/resources/echo-property.pl: Added.
  • http/tests/security/xssAuditor/resources/redir.php: Added.
  • http/tests/security/xssAuditor/resources/xss.js: Added.
  • http/tests/security/xssAuditor/script-tag-convoluted-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-convoluted.html: Added.
  • http/tests/security/xssAuditor/script-tag-open-redirect-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-open-redirect.html: Added.
  • http/tests/security/xssAuditor/script-tag-post-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-post.html: Added.
  • http/tests/security/xssAuditor/script-tag-redirect-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-redirect.html: Added.
  • http/tests/security/xssAuditor/script-tag-utf-7-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-utf-7.html: Added.
  • http/tests/security/xssAuditor/script-tag-with-source-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-with-source.html: Added.
  • http/tests/security/xssAuditor/script-tag.html:
01:46 Changeset [44980] by eric@webkit.org

2009-06-23 Eric Seidel <eric@webkit.org>

Reviewed by Mark Rowe.

  • Scripts/modules/scm.py: Fix error seen when commiting r44979.
01:40 Changeset [44979] by eric@webkit.org

2009-06-18 Eric Seidel <eric@webkit.org>

Reviewed by Dave Levin.

WebKit needs a script to interact with bugzilla and automate
parts of the patch posting and commit processes.
https://bugs.webkit.org/show_bug.cgi?id=26283

This is really a first-draft tool.
It's to the point where it's useful to more people than just me now though.
Git support works. SVN support is written, but mostly untested.

This tool requires BeautifulSoup and mechanize python modules to run:
sudo easy_install BeautifulSoup
sudo easy_install mechanize

More important than the tool itself are the Bugzilla, Git and SVN class abstractions
which I hope will allow easy writing of future tools.

The tool currently implements 10 commands, described below.

Helpers for scripting dealing with the commit queue:
bugs-to-commit Bugs in the commit queue
patches-to-commit Patches attached to bugs in the commit queue

Dealing with bugzilla:
reviewed-patches BUGID r+'d patches on a bug
apply-patches BUGID Applies all patches on a bug to the local working directory without committing.
land-and-update BUGID Lands the current working directory diff and updates the bug.
land-patches [options] BUGID Lands all patches on a bug optionally testing them first
obsolete-attachments BUGID Marks all attachments on a bug as obsolete.
commit-message Prints a commit message suitable for the uncommitted changes.

These effectively replace git-send-bugzilla:
post-diff BUGID Attaches the current working directory diff to a bug as a patch file.
post-commits BUGID COMMITISH Attaches a range of local commits to a bug as patch files.

post-diff works for SVN and Git, post-commits only works for SCMs with local-commit support (like Git)

land-* commands in a Git environment only work with simple patches due to svn-apply bugs:
https://bugs.webkit.org/show_bug.cgi?id=26299
https://bugs.webkit.org/show_bug.cgi?id=26300

This script follows python style (similar to how for Obj-C we follow AppKit style)
http://www.python.org/doc/essays/styleguide.html
The Python community has a strong style culture and the WebKit style guide is silent re: Python.

I've filed a bug to update the WebKit style guide to mention python:
https://bugs.webkit.org/show_bug.cgi?id=26524

  • Scripts/bugzilla-tool: Added.
00:48 Changeset [44978] by abarth@webkit.org

Revert 44977 because Tiger and Windows don't like the new tests.

00:11 Changeset [44977] by abarth@webkit.org

2009-06-23 Adam Barth <abarth@webkit.org>

Reviewed by Sam Weinig.

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

More tests for the XSSAuditor.

  • http/tests/security/xssAuditor/link-onclick-expected.txt: Added.
  • http/tests/security/xssAuditor/link-onclick.html: Added.
  • http/tests/security/xssAuditor/property-escape-expected.txt: Added.
  • http/tests/security/xssAuditor/property-escape.html: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag-post-and-notify.php: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag-post.php: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag-utf-7.php: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag.php:
  • http/tests/security/xssAuditor/resources/echo-property.php: Added.
  • http/tests/security/xssAuditor/resources/redir.php: Added.
  • http/tests/security/xssAuditor/resources/xss.js: Added.
  • http/tests/security/xssAuditor/script-tag-convoluted-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-convoluted.html: Added.
  • http/tests/security/xssAuditor/script-tag-open-redirect-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-open-redirect.html: Added.
  • http/tests/security/xssAuditor/script-tag-post-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-post.html: Added.
  • http/tests/security/xssAuditor/script-tag-redirect-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-redirect.html: Added.
  • http/tests/security/xssAuditor/script-tag-utf-7-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-utf-7.html: Added.
  • http/tests/security/xssAuditor/script-tag-with-source-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-with-source.html: Added.
00:00 Changeset [44976] by ggaren@apple.com

2009-06-22 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.


For member function calls, emit "this" directly into the "this" slot
for the function call, instead of moving it there later. This reduces
time spent in op_mov during certain calls, like "a.b.c()".


1%-2% speedup on v8, mostly richards and delta-blue.

  • parser/Nodes.cpp: (JSC::FunctionCallDotNode::emitBytecode):
Note: See TracTimeline for information about the timeline view.