Timeline


and

04/23/09:

23:38 Changeset [42813] by adele@apple.com

2009-04-23 Adele Peterson <adele@apple.com>

Reviewed by Dan Bernstein.

Updating test to make it easier to see the blockquote in the results.

  • editing/pasteboard/paste-blockquote-into-blockquote-3.html:
  • platform/mac/editing/pasteboard/paste-blockquote-into-blockquote-3-expected.checksum:
  • platform/mac/editing/pasteboard/paste-blockquote-into-blockquote-3-expected.png:
  • platform/mac/editing/pasteboard/paste-blockquote-into-blockquote-3-expected.txt:
23:03 Changeset [42812] by mrowe@apple.com

Move deletionUI-helpers.js in to the correct location so that the
tests that use it can find it.

  • platform/mac/editing/deleting/resources/deletionUI-helpers.js: Renamed from LayoutTests/editing/deleting/resources/deletionUI-helpers.js.
22:12 Changeset [42811] by mrowe@apple.com

With great sadness and a heavy heart I switch us back from YARR to WREC in
order to restore greenness to the world once more.

  • wtf/Platform.h:
22:04 Changeset [42810] by mrowe@apple.com

More Windows build fixage.

21:16 Changeset [42809] by mrowe@apple.com

Attempt to fix the Windows build.

  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def: Remove a symbol that no longer exists.
20:47 Changeset [42808] by timothy@apple.com

BUG 24604: WebKit profiler reports incorrect total times

JavaScriptCore:

2009-04-23 Francisco Tolmasky <francisco@280north.com>

BUG 24604: WebKit profiler reports incorrect total times
<https://bugs.webkit.org/show_bug.cgi?id=24604>

Reviewed by Timothy Hatcher and Kevin McCullough.

  • JavaScriptCore.exp:
  • JavaScriptCore.xcodeproj/project.pbxproj:
  • profiler/CallIdentifier.h: (JSC::CallIdentifier::Hash::hash): (JSC::CallIdentifier::Hash::equal): (JSC::CallIdentifier::hash): (WTF::):
  • profiler/HeavyProfile.cpp: Removed.
  • profiler/HeavyProfile.h: Removed.
  • profiler/Profile.cpp: No more need for TreeProfile/HeavyProfile (JSC::Profile::create):
  • profiler/Profile.h:
  • profiler/ProfileNode.cpp:
  • profiler/ProfileNode.h:
  • profiler/TreeProfile.cpp: Removed.
  • profiler/TreeProfile.h: Removed.

WebCore:

2009-04-23 Francisco Tolmasky <francisco@280north.com>

BUG 24604: WebKit profiler reports incorrect total times
<https://bugs.webkit.org/show_bug.cgi?id=24604>

Reviewed by Timothy Hatcher and Kevin McCullough.

Made it so that most of the profiler functions now match the behavior of Shark. Most notably, in the
heavy view, child nodes now represent the statistics of the root node. Each root node of heavy view
displays flattened statistics for a particular function that ran during the profile, and each child
of these root nodes represents a callpath that lead to it. Thus, the statistics for each of these child
nodes should show how much of the root nodes values came from it. For example, if you had the following to
stacks take place during the profile:

A ->calls 1 times-> B ->calls 2 times-> C
D ->calls 4 times-> C

The tree for the C root node would look like this:

C -> B -> A

-> D

The number of calls values would look like this:

C (6) -> B (2) -> A(2)

-> D (4)

What this means is that "2 of the total 6 C calls came from B", "2 of the total C calls came from A", and
"4 of the total C calls came from D". Notice that the "A ->calls 2 time->" is completely ignored. This becomes
particularly tricky during recursive calls, because each child note can represent multiple possible paths. This
is the reason that we would get things like 40000% previously with recursion.

This is also the way gprof works, and as close as we can get to Shark's behavior (Shark is not instrumented so it
can't know exactly how many calls came from where, etc).

  • English.lproj/localizedStrings.js: Added "Average" for average times in the profile.
  • inspector/JavaScriptProfile.cpp: (WebCore::ProfileClass):
  • inspector/JavaScriptProfileNode.cpp: (WebCore::getParent): (WebCore::getHead): (WebCore::getCallUID): (WebCore::ProfileNodeClass):
  • inspector/front-end/BottomUpProfileDataGridTree.js: Added. (WebInspector.BottomUpProfileDataGridTree): (WebInspector.BottomUpProfileDataGridTree.prototype.focus): (WebInspector.BottomUpProfileDataGridNode): (WebInspector.BottomUpProfileDataGridNode.prototype._takePropertiesFromProfileDataGridNode): (WebInspector.BottomUpProfileDataGridNode.prototype._keepOnlyChild): (WebInspector.BottomUpProfileDataGridNode.prototype._exclude): (WebInspector.BottomUpProfileDataGridNode.prototype._merge): (WebInspector.BottomUpProfileDataGridNode.prototype._populate):
  • inspector/front-end/DataGrid.js: (WebInspector.DataGrid.prototype.insertChild): (WebInspector.DataGrid.prototype.removeChild): (WebInspector.DataGrid.prototype.removeChildren): (WebInspector.DataGridNode.prototype.set hasChildren): (WebInspector.DataGridNode.prototype.get hasChildren):
  • inspector/front-end/ProfileDataGridTree.js: Added. (WebInspector.ProfileDataGridNode): (WebInspector.ProfileDataGridNode.prototype.get data.formatMilliseconds): (WebInspector.ProfileDataGridNode.prototype.get data): (WebInspector.ProfileDataGridNode.prototype.createCell): (WebInspector.ProfileDataGridNode.prototype.select): (WebInspector.ProfileDataGridNode.prototype.deselect): (WebInspector.ProfileDataGridNode.prototype.expand): (WebInspector.ProfileDataGridNode.prototype.insertChild): (WebInspector.ProfileDataGridNode.prototype.removeChild): (WebInspector.ProfileDataGridNode.prototype.removeChildren): (WebInspector.ProfileDataGridNode.prototype.findChild): (WebInspector.ProfileDataGridNode.prototype.get averageTime): (WebInspector.ProfileDataGridNode.prototype.get averagePercent): (WebInspector.ProfileDataGridNode.prototype.get selfPercent): (WebInspector.ProfileDataGridNode.prototype.get totalPercent): (WebInspector.ProfileDataGridNode.prototype._save): (WebInspector.ProfileDataGridNode.prototype._restore): (WebInspector.ProfileDataGridNode.prototype._merge): (WebInspector.ProfileDataGridTree): (WebInspector.ProfileDataGridTree.prototype.get expanded): (WebInspector.ProfileDataGridTree.prototype.appendChild): (WebInspector.ProfileDataGridTree.prototype.insertChild): (WebInspector.ProfileDataGridTree.prototype.removeChildren): (WebInspector.ProfileDataGridTree.prototype.findChild.WebInspector.ProfileDataGridNode.prototype.findChild.sort.WebInspector.ProfileDataGridNode.prototype.sort._save): (WebInspector.ProfileDataGridTree.propertyComparator.comparator): (WebInspector.ProfileDataGridTree.propertyComparator.else.comparator): (WebInspector.ProfileDataGridTree.propertyComparator):
  • inspector/front-end/ProfileView.js: (WebInspector.ProfileView): (WebInspector.ProfileView.prototype.set profile): (WebInspector.ProfileView.prototype.get bottomUpProfileDataGridTree): (WebInspector.ProfileView.prototype.get topDownProfileDataGridTree): (WebInspector.ProfileView.prototype.get currentTree): (WebInspector.ProfileView.prototype.set currentTree): (WebInspector.ProfileView.prototype.get topDownTree): (WebInspector.ProfileView.prototype.get bottomUpTree): (WebInspector.ProfileView.prototype.refresh): (WebInspector.ProfileView.prototype.refreshVisibleData): (WebInspector.ProfileView.prototype.refreshShowAsPercents): (WebInspector.ProfileView.prototype.performSearch.matchesQuery): (WebInspector.ProfileView.prototype.performSearch): (WebInspector.ProfileView.prototype._changeView): (WebInspector.ProfileView.prototype._focusClicked): (WebInspector.ProfileView.prototype._excludeClicked): (WebInspector.ProfileView.prototype._resetClicked): (WebInspector.ProfileView.prototype._sortProfile):
  • inspector/front-end/ProfilesPanel.js: (WebInspector.ProfilesPanel.prototype.showProfile): (WebInspector.ProfilesPanel.prototype.showView): (WebInspector.ProfilesPanel.prototype.searchMatchFound):
  • inspector/front-end/TopDownProfileDataGridTree.js: Added. (WebInspector.TopDownProfileDataGridNode): (WebInspector.TopDownProfileDataGridNode.prototype._populate): (WebInspector.TopDownProfileDataGridNode.prototype._exclude): (WebInspector.TopDownProfileDataGridTree): (WebInspector.TopDownProfileDataGridTree.prototype.focus): (WebInspector.TopDownProfileDataGridTree.prototype.exclude):
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.css:
  • inspector/front-end/inspector.html:

LayoutTests:

2009-04-23 Francisco Tolmasky <francisco@280north.com>

BUG 24604: WebKit profiler reports incorrect total times
<https://bugs.webkit.org/show_bug.cgi?id=24604>

Reviewed by Timothy Hatcher and Kevin McCullough.

Changed profile.treeProfile to just profile, since these aren't generated in C++ anymore.
Removed heavy-view test since heavy-view isn't an actual tree that is generated in C++ land anymore,
but rather just a different display of the normal treeProfile in the JS data grid.

  • fast/profiler/heavy-view-expected.txt: Removed.
  • fast/profiler/heavy-view.html: Removed.
  • fast/profiler/resources/profiler-test-JS-resources.js: profiles[i].treeProfile -> profiles[i].treeProfile (printProfilesDataWithoutTime):
18:50 Changeset [42807] by eric@webkit.org

Reviewed by Sam Weinig.

Use static functions (and a couple templates)
to further reduce the amount of copy/paste code in SVGRenderTreeAsText

No test changes, only code cleanup.

  • rendering/SVGRenderTreeAsText.cpp: (WebCore::writeNameValuePair): (WebCore::writeNameAndQuotedValue): (WebCore::writeIfNotEmpty): (WebCore::writeIfNotDefault): (WebCore::writeStyle): (WebCore::writePositionAndStyle): (WebCore::operator<<):
18:42 Changeset [42806] by kmccullough@apple.com

2009-04-23 Kevin McCullough <kmccullough@apple.com>

  • Windows build fix. Mac only tests go in platform/mac
  • editing/deleting/deletionUI-borders.html: Removed.
  • editing/deleting/deletionUI-differing-background.html: Removed.
  • editing/deleting/deletionUI-minimum-size.html: Removed.
  • platform/mac/editing/deleting/deletionUI-borders.html: Copied from editing/deleting/deletionUI-borders.html.
  • platform/mac/editing/deleting/deletionUI-differing-background.html: Copied from editing/deleting/deletionUI-differing-background.html.
  • platform/mac/editing/deleting/deletionUI-minimum-size.html: Copied from editing/deleting/deletionUI-minimum-size.html.
18:09 Changeset [42805] by barraclough@apple.com

2009-04-23 Gavin Barraclough <barraclough@apple.com>

Not Reviewed.

Speculative Windows build fix II.

  • yarr/RegexInterpreter.cpp:
17:53 Changeset [42804] by barraclough@apple.com

2009-04-23 Gavin Barraclough <barraclough@apple.com>

Not Reviewed.

Speculative Windows build fix.

  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:
  • runtime/RegExp.cpp:
17:50 Changeset [42803] by eric@webkit.org

Reviewed by Simon Fraser.

SVGRenderTreeAsText cleanup/code sharing.

No test changes, only code cleanup.

  • rendering/RenderText.cpp: (WebCore::RenderText::firstRunOrigin):
  • rendering/RenderText.h:
  • rendering/SVGRenderTreeAsText.cpp: (WebCore::writeStandardPrefix): (WebCore::writeChildren): (WebCore::write): (WebCore::writeRenderResources):
17:39 Changeset [42802] by barraclough@apple.com

2009-04-23 Gavin Barraclough <barraclough@apple.com>

Rubber stamped by salty sea dogs Sam & Geoff.

Enable YARR_JIT by default (where supported), replacing WREC.

  • wtf/Platform.h:
17:37 Changeset [42801] by darin@apple.com

2009-04-23 Darin Adler <darin@apple.com>

Removed some stray files that Geoff missed when rolling out changes.

  • svg/custom/resources/svg-fonts-in-text-controls.js: Removed.
  • svg/custom/svg-fonts-in-text-controls-expected.txt: Removed.
17:32 Changeset [42800] by barraclough@apple.com

JavaScriptCore:

2009-04-23 Gavin Barraclough <barraclough@apple.com>

Reviewed by Geoff "Dread Pirate Roberts" Garen.

Various small fixes to YARR JIT, in preparation for enabling it by default.

  • Correctly index into the callframe when storing restart addresses for nested alternatives.
  • Allow backtracking back into matched alternatives of parentheses.
  • Fix callframe offset calculation for parenthetical assertions.
  • When a set of parenthese are quantified with a fixed and variable portion, and the variable portion is quantified once, this should not reset the pattern match on failure to match (the last match from the firxed portion should be preserved).
  • Up the pattern size limit to match PCRE's new limit.
  • Unlclosed parentheses should be reported with the message "missing )".
  • wtf/Platform.h:
  • yarr/RegexCompiler.cpp: (JSC::Yarr::RegexPatternConstructor::quantifyAtom): (JSC::Yarr::RegexPatternConstructor::setupAlternativeOffsets):
  • yarr/RegexInterpreter.cpp: (JSC::Yarr::Interpreter::matchParentheses): (JSC::Yarr::Interpreter::backtrackParentheses): (JSC::Yarr::ByteCompiler::emitDisjunction):
  • yarr/RegexJIT.cpp: (JSC::Yarr::RegexGenerator::loadFromFrameAndJump): (JSC::Yarr::RegexGenerator::generateParenthesesDisjunction): (JSC::Yarr::RegexGenerator::generateParentheticalAssertion): (JSC::Yarr::RegexGenerator::generateTerm): (JSC::Yarr::executeRegex):
  • yarr/RegexParser.h: (JSC::Yarr::Parser::): (JSC::Yarr::Parser::parseTokens): (JSC::Yarr::Parser::parse):
  • yarr/RegexPattern.h: (JSC::Yarr::PatternTerm::): (JSC::Yarr::PatternTerm::PatternTerm):

LayoutTests:

2009-04-23 Gavin Barraclough <barraclough@apple.com>

Reviewed by Geoff "Dread Pirate Roberts" Garen.

This test tries to force itself into PCRE; modify the
test so that as well as dodging WREC it can also avoid
YARR!

  • fast/js/resources/regexp-overflow-too-big.js:
17:25 Changeset [42799] by bdakin@apple.com

WebCore:

2009-04-23 Beth Dakin <bdakin@apple.com>

Reviewed by Darin Adler.

WebCore part of fix for <rdar://problem/6333461> REGRESSION
(r36864-r36869): Dragging stocks widget scrollbar drags the whole
widget

  • platform/Scrollbar.h: (WebCore::Scrollbar::isScrollbar):
  • platform/Widget.h: (WebCore::Widget::isScrollbar):

WebKit/mac:

2009-04-23 Beth Dakin <bdakin@apple.com>

Reviewed by Darin Adler.

Fix for <rdar://problem/6333461> REGRESSION (r36864-r36869):
Dragging stocks widget scrollbar drags the whole widget

Look for our new WebCore scrollbars in the WebHTMLView and add
proper Dashboard regions for them.

  • WebView/WebView.mm: (-[WebView _addScrollerDashboardRegionsForFrameView:dashboardRegions:]): (-[WebView _addScrollerDashboardRegions:from:]):
16:59 Changeset [42798] by kmccullough@apple.com

WebCore:

2009-04-23 Kevin McCullough <kmccullough@apple.com>

Reviewed by Adam Roben.

<rdar://problem/6808109> "Deletion UI" is not available for many
portions of HTML content

This patch makes the deletion UI show up in some new situations:
1) If a block's background color is different from its parent's
2) If a block has a background image.
3) If a block has a single visible border.
However the block must now not only be at least a minimum width and
height, but also exceed a minimum area. In practice this has led to
much better element selection.

  • editing/DeleteButtonController.cpp: (WebCore::isDeletableElement):

LayoutTests:

2009-04-23 Kevin McCullough <kmccullough@apple.com>

Reviewed by Adam Roben.

<rdar://problem/6808109> "Deletion UI" is not available for many
portions of HTML content

This patch makes the deletion UI show up in some new situations:
1) If a block's background color is different from its parent's
2) If a block has a background image.
3) If a block has a single visible border.
However the block must now not only be at least a minimum width and
height, but also exceed a minimum area. In practice this has led to
much better element selection.

  • editing/deleting/deletionUI-borders.html: Added.
  • editing/deleting/deletionUI-differing-background.html: Added.
  • editing/deleting/deletionUI-minimum-size.html: Added.
  • editing/deleting/resources: Added.
  • editing/deleting/resources/deletionUI-helpers.js: Added. (debug): (determineDeletionUIExistence):
  • platform/mac/editing/deleting/deletionUI-borders-expected.txt: Added.
  • platform/mac/editing/deleting/deletionUI-differing-background-expected.txt: Added.
  • platform/mac/editing/deleting/deletionUI-minimum-size-expected.txt: Added.
16:53 Changeset [42797] by kevino@webkit.org

wx build fix. Adding XMLTokenizer.cpp to the build.

16:32 Changeset [42796] by sullivan@apple.com

2009-04-23 John Sullivan <sullivan@apple.com>

fixed <rdar://problem/6822479> Assertion failure after Reset Safari in new history-writing code

Reviewed by Oliver Hunt

  • History/WebHistory.mm: (-[WebHistoryPrivate data]): Return nil immediately if there are no entries; this matches a recent Windows-platform fix.
16:14 Changeset [42795] by mrowe@apple.com

Temporarily work around <rdar://problem/6812854> by forcing our 2GB allocation up in to high memory.

Rubber-stamped by Geoff Garen.

  • jit/ExecutableAllocatorFixedVMPool.cpp:

(JSC::FixedVMPoolAllocator::FixedVMPoolAllocator):

15:46 Changeset [42794] by adele@apple.com

2009-04-23 Adele Peterson <adele@apple.com>

Reviewed by Darin Adler.

Fix for <rdar://problem/6788905> REGRESSION (41928?): hang in Position::upstream
https://bugs.webkit.org/show_bug.cgi?id=25312

I was unable to reproduce the problem, but I'm pretty sure this will fix it.

  • rendering/RenderObject.cpp: (WebCore::RenderObject::createVisiblePosition): Since VisiblePosition doesn't ensure the offset is good, we shouldn't pass max int as an offset.
15:38 Changeset [42793] by mrowe@apple.com

Versioning.

15:38 Changeset [42792] by mrowe@apple.com

New tag.

15:33 Changeset [42791] by dglazkov@chromium.org

WebCore:

2009-04-23 Dimitri Glazkov <dglazkov@chromium.org>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=25313
Missing scroll bars in GMail.

Test: fast/overflow/overflow-y-scroll.html

  • platform/ScrollView.cpp: (WebCore::ScrollView::updateScrollbars): Added check for the ScrollbarAlwaysOn scroll mode.

LayoutTests:

2009-04-23 Dimitri Glazkov <dglazkov@chromium.org>

Reviewed by Darin Adler.
https://bugs.webkit.org/show_bug.cgi?id=25313
Missing scroll bars in GMail.

  • fast/overflow/overflow-y-scroll-expected.txt: Added.
  • fast/overflow/overflow-y-scroll.html: Added.

WebKit/mac:

2009-04-23 Dimitri Glazkov <dglazkov@chromium.org>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=25313
Missing scroll bars in GMail.

  • WebView/WebDynamicScrollBarsView.m: (-[WebDynamicScrollBarsView updateScrollers]): Added check for the ScrollbarAlwaysOn

scroll mode.

15:17 Changeset [42790] by kdecker@apple.com

Fix the Tiger build

15:03 Changeset [42789] by andersca@apple.com

2009-04-23 Anders Carlsson <andersca@apple.com>

Reviewed by Geoffrey Garen.

Fix <rdar://problem/6821992>

Add a new m_inDestroy member variable. Set it to true when in destroy, and have all NPRuntime functions return false
when m_inDestroy is true.


  • Plugins/Hosted/NetscapePluginInstanceProxy.h:
  • Plugins/Hosted/NetscapePluginInstanceProxy.mm: (WebKit::NetscapePluginInstanceProxy::NetscapePluginInstanceProxy): (WebKit::NetscapePluginInstanceProxy::destroy): (WebKit::NetscapePluginInstanceProxy::invoke): (WebKit::NetscapePluginInstanceProxy::invokeDefault): (WebKit::NetscapePluginInstanceProxy::construct): (WebKit::NetscapePluginInstanceProxy::getProperty): (WebKit::NetscapePluginInstanceProxy::setProperty): (WebKit::NetscapePluginInstanceProxy::removeProperty): (WebKit::NetscapePluginInstanceProxy::hasProperty): (WebKit::NetscapePluginInstanceProxy::hasMethod): (WebKit::NetscapePluginInstanceProxy::enumerate):
14:22 Changeset [42788] by mitz@apple.com

WebCore:

Reviewed by Justin Garcia.

  • fix <rdar://problem/6081309> Mail crash when pressing down arrow in some messages in WebCore::canHaveChildrenForEditing

Test: editing/selection/extend-by-line-anonymous-content-crash.html

  • editing/visible_units.cpp: (WebCore::previousLinePosition): Null-check node. If p is not an editable position, then closestLeafChildForXPos() may have returned a non-editable box, and in particular one belonging to anonymous content. If node is 0, fall back on RenderObject::positionForPoint, which finds the closest position in non-anonymous content. (WebCore::nextLinePosition): Ditto.
  • rendering/RenderObject.cpp: (WebCore::RenderObject::createVisiblePosition): Fixed a typo.

LayoutTests:

Reviewed by Justin Garcia.

  • test for <rdar://problem/6081309> Mail crash when pressing down arrow in some messages in WebCore::canHaveChildrenForEditing
  • editing/selection/extend-by-line-anonymous-content-crash-expected.txt: Added.
  • editing/selection/extend-by-line-anonymous-content-crash.html: Added.
13:49 Changeset [42787] by hyatt@apple.com

2009-04-23 David Hyatt <hyatt@apple.com>

Reviewed by Maciej.

Fix for <rdar://problem/6789879> REGRESSION (42464): Hitting assertion when loading message in Mail + TOT WebKit


Make the Mac platform the same as all the other platforms. Instead of (incorrectly) marking a FrameView for layout
when its underlying document view changes, just mark the outermost frame view for layout when the WebView's size changes.

  • WebView/WebHTMLView.mm: (-[WebHTMLView _frameOrBoundsChanged]): (-[WebHTMLView addSuperviewObservers]):
  • WebView/WebView.mm: (-[WebView _boundsChanged]): (-[WebView removeSizeObservers]): (-[WebView addSizeObservers]):
13:08 Changeset [42786] by kdecker@apple.com

Reviewed by Tim Hatcher.

Second part of the fix for <rdar://problem/6352982>

  • Plugins/WebBaseNetscapePluginView.h: Make this class conform to WebPluginContainerCheckController
  • Plugins/WebBaseNetscapePluginView.mm: Likewise. (-[WebBaseNetscapePluginView _webPluginContainerCancelCheckIfAllowedToLoadRequest:]): Added skeleton method.
  • Plugins/WebPluginContainerCheck.h: Added protocol for <WebPluginContainerCheckController>
  • Plugins/WebPluginContainerCheck.mm: (-[WebPluginContainerCheck initWithRequest:target:resultObject:selector:controller:]):
  • Plugins/WebPluginController.h: Make this class conform to WebPluginContainerCheckController
12:58 Changeset [42785] by ap@webkit.org

Reviewed by Adele Peterson.

<rdar://problem/6791439> REGRESSION: Get an error page instead of login page navigating back in gmail

Test: http/tests/history/back-to-post.php

  • platform/network/cf/ResourceRequestCFNet.cpp: (WebCore::ResourceRequest::doUpdatePlatformRequest): Apply a new cache policy if it changed after the platform request was first created. While at it, also made sure to update timeout.
12:02 Changeset [42784] by kov@webkit.org

2009-04-23 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Rubber-stamped by Xan Lopez.

  • pt_BR.po: translation update.
11:45 Changeset [42783] by kov@webkit.org

2009-04-23 Zan Dobersek <zandobersek@gmail.com>

Reviewed by Gustavo Noronha.

When creating a new drawable object, collect system's
information about the best depth it can provide and use it
in a new colormap that the new drawable requires.

  • platform/graphics/gtk/ImageGtk.cpp: (WebCore::BitmapImage::getGdkPixbuf):
11:06 Changeset [42782] by kdecker@apple.com

Reviewed by Anders Carlsson.


First part of <rdar://problem/6352982>

  • Plugins/WebBaseNetscapePluginView.h: Imported #WebNetscapeContainerCheckPrivate.h; Added two new method: checkIfAllowedToLoadURL:url:frame:callbackFunc: and cancelCheckIfAllowedToLoadURL:
  • Plugins/WebBaseNetscapePluginView.mm: (-[WebBaseNetscapePluginView checkIfAllowedToLoadURL:frame:callbackFunc:]): Added skeleton method, does nothing interesting yet. (-[WebBaseNetscapePluginView cancelCheckIfAllowedToLoadURL:]): Likewise.
  • Plugins/WebNetscapeContainerCheckPrivate.h: Added.
  • Plugins/WebNetscapeContainerCheckPrivate.mm: Added. (browserContainerCheckFuncs): Added.
  • Plugins/WebNetscapePluginView.h: Imported #WebNetscapeContainerCheckPrivate.h;
  • Plugins/WebNetscapePluginView.mm: Imported #WebNetscapeContainerCheckPrivate.h; added WKN_CheckIfAllowedToLoadURL and WKN_CancelCheckIfAllowedToLoadURL functions. (-[WebNetscapePluginView getVariable:value:]): Return vtable for container check functions.
  • Plugins/npapi.mm: (WKN_CheckIfAllowedToLoadURL): Added new private function. (WKN_CancelCheckIfAllowedToLoadURL): Ditto.
09:42 Changeset [42781] by aroben@apple.com

Skip a newly-added test that fails on Windows

See Bug 25344: http/tests/misc/redirect-to-external-url.html fails on
Windows
<https://bugs.webkit.org/show_bug.cgi?id=25344>

Reviewed by Brady Eidson.

  • platform/win/Skipped: Added http/tests/misc/redirect-to-external-url.html.
08:39 Changeset [42780] by kov@webkit.org

2009-04-23 Zan Dobersek <zandobersek@gmail.com>

Reviewed by Gustavo Noronha.

https://bugs.webkit.org/show_bug.cgi?id=15654
GdkPixbuf support for ImageCairo

Add support for converting a Cairo surface to a GdkPixbuf.

  • platform/graphics/BitmapImage.h:
  • platform/graphics/Image.h: (WebCore::Image::getGdkPixbuf):
  • platform/graphics/gtk/ImageGtk.cpp: (WebCore::BitmapImage::getGdkPixbuf):
08:07 Changeset [42779] by vestbo@webkit.org

Fix Qt build warning

07:55 Changeset [42778] by ariya@webkit.org

2009-04-23 Kenneth Rohde Christiansen <kenneth.christiansen@openbossa.org>

Reviewed by Simon Hausmann.

Make windowed plugins (on X11, Qt) behave nicer with wheel scrolling.

Basically, what we want is to send wheel events to the plugin *only*
when it has focus, or else send it to the parent frame.

This is a work around, and basically what we do, is to steal the
wheel events from the plugin, modify the window ID, and put the
event back into the Qt event loop.

  • plugins/qt/PluginContainerQt.cpp: (PluginClientWrapper::PluginClientWrapper): (PluginClientWrapper::~PluginClientWrapper): (PluginClientWrapper::x11Event): (PluginContainerQt::PluginContainerQt): (PluginContainerQt::~PluginContainerQt): (PluginContainerQt::on_clientClosed): (PluginContainerQt::on_clientIsEmbedded): (PluginContainerQt::redirectWheelEventsToParent): (PluginContainerQt::x11Event): (PluginContainerQt::focusInEvent):
  • plugins/qt/PluginContainerQt.h:
06:53 Changeset [42777] by ariya@webkit.org

2009-04-23 Antonio Gomes <antonio.gomes@openbossa.org>

Reviewed by Ariya Hidayat.

[Qt] Added QWebElement::computedStyleProperty method.

  • Api/qwebelement.cpp: (QWebElement::computedStyleProperty):
  • Api/qwebelement.h:
  • tests/qwebelement/tst_qwebelement.cpp: (tst_QWebElement::computedStyle):
05:46 QtWebKitTodo edited by vestbo@webkit.org
(diff)
05:36 QtWebKitJournal edited by vestbo@webkit.org
(diff)
04:30 Changeset [42776] by vestbo@webkit.org

2009-04-23 Tor Arne Vestbø <tor.arne.vestbo@nokia.com>

Rubber-stamped by Ariya Hidayat.

[Qt] Rename QWebElement arguments from html to markup

00:25 Changeset [42775] by mrowe@apple.com

Fix the Windows DRT build.

  • DumpRenderTree/win/PolicyDelegate.h:
00:10 Changeset [42774] by beidson@apple.com

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

Fix Windows DRT build.

  • DumpRenderTree/win/PolicyDelegate.cpp: (PolicyDelegate::unableToImplementPolicyWithError):
00:09 Changeset [42773] by beidson@apple.com

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

Update my just-landed layout test to agree with Tiger's CFNetwork

  • http/tests/misc/resources/redirect-to-external-url.php:

04/22/09:

23:42 Changeset [42772] by beidson@apple.com

WebCore:

2009-04-22 Brady Eidson <beidson@apple.com>

Reviewed by Maciej Stachowiak

<rdar://problem/6786961> - "Are you sure you want to resubmit this form?" nag displays twice.

For most loads, we were consulting the navigation policy delegate twice. Once from FrameLoader before
the load started and once from MainResourceLoader in its willSendRequest callback.

In the past we tried removing MainResourceLoader's call altogether. This caused a regression where urls
that redirect to a url handled by an external application would no longer work in Safari. It probably
also broke other WebKit apps in subtle ways.

Changing MainResourceLoader to make the check only on redirects fixes both bugs. We now only call the
policy delegate once for most standard loads, but we do correctly call it a second time for redirects.

Tests: http/tests/misc/policy-delegate-called-twice.html

http/tests/misc/redirect-to-external-url.html

  • loader/MainResourceLoader.cpp: (WebCore::MainResourceLoader::willSendRequest): Call the navigation policy delegate only for redirects.

WebKitTools:

2009-04-22 Brady Eidson <beidson@apple.com>

Reviewed by Maciej Stachowiak

Add additional support for test:
<rdar://problem/5689748> - Cannot redirect to protocols handled by external applications.

  • DumpRenderTree/mac/PolicyDelegate.mm: (-[PolicyDelegate webView:unableToImplementPolicyWithError:frame:]): Dump formatted information when a policy cannot be implemented.
  • DumpRenderTree/win/PolicyDelegate.cpp: (PolicyDelegate::unableToImplementPolicyWithError): Ditto.
  • DumpRenderTree/win/PolicyDelegate.h:

LayoutTests:

2009-04-22 Brady Eidson <beidson@apple.com>

Reviewed by Maciej Stachowiak

With the fix for:
<rdar://problem/6786961> - "Are you sure you want to resubmit this form?" nag displays twice.
And also to make sure we never rebreak:
<rdar://problem/5689748> - Cannot redirect to protocols handled by external applications.

  • http/tests/misc/redirect-to-external-url-expected.txt: Added.
  • http/tests/misc/redirect-to-external-url.html: Added.
  • http/tests/misc/resources/redirect-to-external-url-iframe.html: Added.
  • http/tests/misc/resources/redirect-to-external-url.php: Added.
  • http/tests/misc/policy-delegate-called-twice-expected.txt: Added.
  • http/tests/misc/policy-delegate-called-twice.html: Added.
  • http/tests/misc/resources/iframe-policy-1.html: Added.
  • http/tests/misc/resources/iframe-policy-2.html: Added.
21:06 Changeset [42771] by mrowe@apple.com

Add the m_ prefix on FixedVMPoolAllocator's member variables, and fix typos in a few comments.

Rubber-stamped by Gavin Barraclough.

  • jit/ExecutableAllocatorFixedVMPool.cpp:

(JSC::FixedVMPoolAllocator::addToFreeList):
(JSC::FixedVMPoolAllocator::coalesceFreeSpace):
(JSC::FixedVMPoolAllocator::FixedVMPoolAllocator):
(JSC::FixedVMPoolAllocator::alloc):
(JSC::FixedVMPoolAllocator::free):
(JSC::FixedVMPoolAllocator::isWithinVMPool):

21:06 Changeset [42770] by mrowe@apple.com

Add some assertions to FixedVMPoolAllocator to guard against cases where we
attempt to free memory that didn't originate from the pool, or we attempt to
hand out a bogus address from alloc.

Rubber-stamped by Gavin Barraclough.

  • jit/ExecutableAllocatorFixedVMPool.cpp:

(JSC::FixedVMPoolAllocator::release):
(JSC::FixedVMPoolAllocator::FixedVMPoolAllocator):
(JSC::FixedVMPoolAllocator::alloc):
(JSC::FixedVMPoolAllocator::free):
(JSC::FixedVMPoolAllocator::isWithinVMPool):

19:41 Changeset [42769] by weinig@apple.com

2009-04-22 Sam Weinig <sam@webkit.org>

Reviewed by Mark Rowe.

Fix for <rdar://problem/6800695>

Add a hack to only allow navigating (via a link from a http page)
to feed: urls (same for feeds: and feedsearch:) that map to http:
or https: via their nested protocol. This includes both feed://example.com
and feed:http://example.com.

  • loader/FrameLoader.cpp: (WebCore::isFeedWithNestedProtocolInHTTPFamily): (WebCore::FrameLoader::loadFrameRequest):
19:34 Changeset [42768] by weinig@apple.com

2009-04-22 Sam Weinig <sam@webkit.org>

Reviewed by Anders Carlsson.

Autogenerate the XMLHttpRequest constructor getter/setter for the
WorkerContext and remove the now no longer needed JSWorkerContextBase.lut.h
file.

  • DerivedSources.make: Remove JSWorkerContextBase.lut.h
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • bindings/js/JSWorkerContextBase.cpp:
  • bindings/js/JSWorkerContextBase.h: Remove code to support hand written code for XMLHttpRequest constructor lookup and shadowing.
  • bindings/js/JSWorkerContextCustom.cpp: (WebCore::JSWorkerContext::customGetOwnPropertySlot): Remove JSC prefixes. (WebCore::JSWorkerContext::xmlHttpRequest): Added. Custom implementation of cached constructor lookup.
  • workers/WorkerContext.idl: Make the XMLHttpRequestConstructor accessor not V8 specific.
18:43 Changeset [42767] by adachan@apple.com

WebKitLibraries:

Update WebKitSystemInterface with new method that maps CFNetwork error code to localized description.


Reviewed by Darin Adler.

  • win/include/WebKitSystemInterface/WebKitSystemInterface.h:
  • win/lib/WebKitSystemInterface.lib:
  • win/lib/WebKitSystemInterface_debug.lib:

WebKit/win:

<rdar://problem/5115298> Don't get error msg when download is interrupted and cannot resume due to server unavailability
If we fail to get an error description back from CFNetwork, try to get the localized description
based on the error code.

Reviewed by Darin Adler.

  • WebError.cpp: (WebError::localizedDescription):
18:07 Changeset [42766] by darin@apple.com

2009-04-22 Darin Adler <darin@apple.com>

  • page/Navigator.idl: Touch this file to fix build since Geolocation configuration was recently turned off.
18:02 Changeset [42765] by oliver@apple.com

Include updated results

17:59 Changeset [42764] by oliver@apple.com

Modify test to only record the important mouse events

Reviewed by Geoff Garen.

17:54 Changeset [42763] by justin.garcia@apple.com

WebCore:

2009-04-22 Justin Garcia <justin.garcia@apple.com>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=25322
Can't delete empty paragraph after a block image


Empty paragraph removal would have been handled by mergeParagraphs, but we stopped short
because of some code that avoided problems that would arise from Positions that would go
bad during deletion.


We have several checks that prevent us from using bad Positions, this one in particular
was added to avoid bad Positions that result from text removal. So, I removed the check
and started correctly updating m_downstreamEnd during text removal so that it doesn't go
bad. m_upstreamStart doesn't need to be updated during text removal, since only text
after it will ever be removed.

  • editing/DeleteSelectionCommand.cpp: (WebCore::DeleteSelectionCommand::deleteTextFromNode): (WebCore::DeleteSelectionCommand::handleGeneralDelete): (WebCore::DeleteSelectionCommand::mergeParagraphs):

LayoutTests:

2009-04-22 Justin Garcia <justin.garcia@apple.com>

Reviewed by Darin Adler.


https://bugs.webkit.org/show_bug.cgi?id=25322
Can't delete empty paragraph after a block image

  • editing/deleting/25322-1-expected.txt: Added.
  • editing/deleting/25322-1.html: Added.
  • editing/deleting/25322-2-expected.txt: Added.
  • editing/deleting/25322-2.html: Added.
  • editing/deleting/25322-3-expected.txt: Added.
  • editing/deleting/25322-3.html: Added.
17:49 Changeset [42762] by darin@apple.com

2009-04-22 Darin Adler <darin@apple.com>

  • platform/mac/svg/W3C-SVG-1.1/animate-elem-36-t-expected.txt: Fixed one case Eric seems to have missed earlier today of RenderImage -> RenderSVGImage.
17:37 Changeset [42761] by oliver@apple.com

<rdar://problem/6757346> SAP: Prevent default on mouseDown does not stop iframe from capturing subsequent mouse moves

Reviewed by Darin Adler.

Make mouseUp target the correct frame when the original
mousedown made the drag non-capturing.

17:29 Changeset [42760] by oliver@apple.com

<rdar://problem/6757346> SAP: Prevent default on mouseDown does not stop iframe from capturing subsequent mouse moves

Reviewed by Darin Adler.

This is the first step of allowing drag events to match the behaviour
of mouse move events when the default action of the initial mouse down
is prevented. Remaining issue is that the final mouse up event still
targets the original root frame.

Test: fast/events/mouse-drag-from-frame-to-other-frame.html

17:01 Changeset [42759] by simon.fraser@apple.com

2009-04-22 Simon Fraser <simon.fraser@apple.com>

Reviewed by Adele Peterson

<rdar://problem/6702882> QT movie's video fails to be displayed after switching between tabs
<rdar://problem/6754957> Resizing Web browser window and hitting play will cause video blocks and artifacting

When in a media document, MediaPlayerPrivateQTKit uses a QTMovieView which may get
layer backing under some circumstances. However, drawing the view via
displayRectIgnoringOpacity:inContext: bypasses any layer setup that AppKit normally performs.
So when in the media document, we draw via displayRectIgnoringOpacity:.

  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::paint):
16:56 Changeset [42758] by mrowe@apple.com

Update test results after Blackbeard's change in r42752 to disable geolocation.

  • fast/dom/Window/window-properties-expected.txt:
  • fast/dom/navigator-detached-no-crash-expected.txt:
15:59 Changeset [42757] by simon.fraser@apple.com

2009-04-22 Pierre d'Herbemont <pdherbemont@apple.com>

Reviewed by Simon Fraser.

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

Add tests for media 'played' attribute (missed in r42619).

  • media/video-played-expected.txt: Added.
  • media/video-played.html: Added.
15:51 Changeset [42756] by andersca@apple.com

2009-04-22 Anders Carlsson <andersca@apple.com>

Reviewed by Oliver Hunt.

Fix <rdar://problem/6792694>


When we're trying to instantiate a plug-in and the plug-in host has died, we need to invalidate the
instance so that it doesn't stick around and do bad things.


  • Plugins/Hosted/NetscapePluginHostManager.mm: (WebKit::NetscapePluginHostManager::instantiatePlugin):
14:52 Changeset [42755] by barraclough@apple.com

2009-04-22 Gavin Barraclough <barraclough@apple.com>

Rubber stamped by Sam "Blackbeard" Weinig.

Although pirates do spell the word 'generate' as 'genertate',
webkit developers do not. Fixerate.

  • yarr/RegexJIT.cpp: (JSC::Yarr::RegexGenerator::generateAssertionBOL): (JSC::Yarr::RegexGenerator::generateAssertionEOL): (JSC::Yarr::RegexGenerator::generateAssertionWordBoundary): (JSC::Yarr::RegexGenerator::generatePatternCharacterSingle): (JSC::Yarr::RegexGenerator::generatePatternCharacterPair): (JSC::Yarr::RegexGenerator::generatePatternCharacterFixed): (JSC::Yarr::RegexGenerator::generatePatternCharacterGreedy): (JSC::Yarr::RegexGenerator::generatePatternCharacterNonGreedy): (JSC::Yarr::RegexGenerator::generateCharacterClassSingle): (JSC::Yarr::RegexGenerator::generateCharacterClassFixed): (JSC::Yarr::RegexGenerator::generateCharacterClassGreedy): (JSC::Yarr::RegexGenerator::generateCharacterClassNonGreedy): (JSC::Yarr::RegexGenerator::generateTerm):
14:44 Changeset [42754] by barraclough@apple.com

2009-04-22 Gavin Barraclough <barraclough@apple.com>

Reviewed by Sam "Blackbeard" Weinig.

Improvements to YARR JIT. This patch expands support in three key areas:

  • Add (temporary) support for falling back to PCRE for expressions not supported.
  • Add support for x86_64 and Windows.
  • Add support for singly quantified parentheses (? and ??), alternatives within parentheses, and parenthetical assertions.
  • runtime/RegExp.cpp: (JSC::RegExp::match):
  • yarr/RegexJIT.cpp: (JSC::Yarr::RegexGenerator::storeToFrame): (JSC::Yarr::RegexGenerator::storeToFrameWithPatch): (JSC::Yarr::RegexGenerator::loadFromFrameAndJump): (JSC::Yarr::RegexGenerator::AlternativeBacktrackRecord::AlternativeBacktrackRecord): (JSC::Yarr::RegexGenerator::TermGenerationState::resetAlternative): (JSC::Yarr::RegexGenerator::TermGenerationState::resetTerm): (JSC::Yarr::RegexGenerator::TermGenerationState::jumpToBacktrack): (JSC::Yarr::RegexGenerator::TermGenerationState::plantJumpToBacktrackIfExists): (JSC::Yarr::RegexGenerator::TermGenerationState::addBacktrackJump): (JSC::Yarr::RegexGenerator::TermGenerationState::linkAlternativeBacktracks): (JSC::Yarr::RegexGenerator::TermGenerationState::propagateBacktrackingFrom): (JSC::Yarr::RegexGenerator::genertateAssertionBOL): (JSC::Yarr::RegexGenerator::genertateAssertionEOL): (JSC::Yarr::RegexGenerator::matchAssertionWordchar): (JSC::Yarr::RegexGenerator::genertateAssertionWordBoundary): (JSC::Yarr::RegexGenerator::genertatePatternCharacterSingle): (JSC::Yarr::RegexGenerator::genertatePatternCharacterPair): (JSC::Yarr::RegexGenerator::genertatePatternCharacterFixed): (JSC::Yarr::RegexGenerator::genertatePatternCharacterGreedy): (JSC::Yarr::RegexGenerator::genertatePatternCharacterNonGreedy): (JSC::Yarr::RegexGenerator::genertateCharacterClassSingle): (JSC::Yarr::RegexGenerator::genertateCharacterClassFixed): (JSC::Yarr::RegexGenerator::genertateCharacterClassGreedy): (JSC::Yarr::RegexGenerator::genertateCharacterClassNonGreedy): (JSC::Yarr::RegexGenerator::generateParenthesesDisjunction): (JSC::Yarr::RegexGenerator::generateParenthesesSingle): (JSC::Yarr::RegexGenerator::generateParentheticalAssertion): (JSC::Yarr::RegexGenerator::generateTerm): (JSC::Yarr::RegexGenerator::generateDisjunction): (JSC::Yarr::RegexGenerator::generateEnter): (JSC::Yarr::RegexGenerator::generateReturn): (JSC::Yarr::RegexGenerator::RegexGenerator): (JSC::Yarr::RegexGenerator::generate): (JSC::Yarr::RegexGenerator::compile): (JSC::Yarr::RegexGenerator::generationFailed): (JSC::Yarr::jitCompileRegex): (JSC::Yarr::executeRegex):
  • yarr/RegexJIT.h: (JSC::Yarr::RegexCodeBlock::RegexCodeBlock): (JSC::Yarr::RegexCodeBlock::~RegexCodeBlock):
14:24 Changeset [42753] by timothy@apple.com

Fix a crash in Mail when forwarding a specific HTML email.
Caused by the fix for <rdar://problem/6512520> (r42096).
Unable to create a test case that works outside of Mail.

<rdar://problem/6796069>

Reviewed by Ada Chan.

  • editing/markup.cpp: (WebCore::createMarkup): Null check pastEnd before trying to call Range::compareBoundaryPoints.
13:44 QtWebKitJournal edited by tonikitoo@gmail.com
(diff)
12:09 Changeset [42752] by weinig@apple.com

JavaScriptCore:

2009-04-22 Sam Weinig <sam@webkit.org>

Rubber-stamped by Darin Adler.

Fix for <rdar://problem/6816957>
Turn off Geolocation by default

  • Configurations/FeatureDefines.xcconfig:

WebCore:

2009-04-22 Sam Weinig <sam@webkit.org>

Rubber-stamped by Darin Adler.

Fix for <rdar://problem/6816957>
Turn off Geolocation by default

  • Configurations/FeatureDefines.xcconfig:

WebKit/mac:

2009-04-22 Sam Weinig <sam@webkit.org>

Rubber-stamped by Darin Adler.

Fix for <rdar://problem/6816957>
Turn off Geolocation by default

  • Configurations/FeatureDefines.xcconfig:

WebKitTools:

2009-04-22 Sam Weinig <sam@webkit.org>

Rubber-stamped by Darin Adler.

Fix for <rdar://problem/6816957>
Turn off Geolocation by default

  • Scripts/build-webkit:
11:03 Changeset [42751] by eric@webkit.org

Rubber-stamped by David Hyatt.

Change RenderSVGImage to dump as RenderSVGImage instead of RenderImage

  • rendering/RenderSVGImage.h: (WebCore::RenderSVGImage::renderName):
08:48 Changeset [42750] by timothy@apple.com

Fix a crash on Gmail when they remove a "before unload" event listener when
it was never added to the pending map in the first place.

<rdar://problem/6814144>

Reviewed by Darin Adler.

  • page/DOMWindow.cpp: (WebCore::allowsPendingBeforeUnloadListeners): Renamed from shouldAddPendingBeforeUnloadListener. (WebCore::DOMWindow::addEventListener): Call allowsPendingBeforeUnloadListeners now. (WebCore::DOMWindow::removeEventListener): Call allowsPendingBeforeUnloadListeners before removing the before unload listener. (WebCore::DOMWindow::clearAttributeEventListener): Ditto.
06:39 Changeset [42749] by ariya@webkit.org

2009-04-22 Kenneth Rohde Christiansen <kenneth.christiansen@openbossa.org>

Reviewed by Ariya Hidayat.

Made windowed plugins move/resize synchronized with the painting, so
that windowed plugins are not moved before the rest of the parent
frame during scrolling.

  • plugins/qt/PluginContainerQt.cpp: (PluginContainerQt::PluginContainerQt): (PluginContainerQt::requestGeometry): (PluginContainerQt::adjustGeometry):
  • plugins/qt/PluginContainerQt.h:
  • plugins/qt/PluginViewQt.cpp: (WebCore::PluginView::updatePluginWidget): (WebCore::PluginView::paint):
05:34 Changeset [42748] by hausmann@webkit.org

2009-04-22 Simon Hausmann <simon.hausmann@nokia.com>

Reviewed by Ariya Hidayat.

Fix various bugs in the X11 Qt plugin focus handling.

Notify the focus controller on activation/deactivation of the plugin and
set focus on the plugin's element when it receives the window system
focus.

05:08 Changeset [42747] by ariya@webkit.org

2009-04-22 Tamas Szirbucz <szirbucz.tamas@stud.u-szeged.hu>

Reviewed by Ariya Hidayat.

https://bugs.webkit.org/show_bug.cgi?id=25023
Delete reply in QNetworkReplyHandler::abort() to avoid leak.

  • platform/network/qt/QNetworkReplyHandler.cpp: (WebCore::QNetworkReplyHandler::abort):
03:44 QtWebKitJournal edited by hausmann@webkit.org
(diff)
01:52 Changeset [42746] by jhoneycutt@apple.com

2009-04-21 Jon Honeycutt <jhoneycutt@apple.com>

Allow the UI delegate to control cursor-setting.

Reviewed by Ada Chan.

WebCore/:

  • page/Chrome.cpp: (WebCore::Chrome::setCursor): Pass the call to the client.
  • page/Chrome.h:
  • page/ChromeClient.h: (WebCore::ChromeClient::setCursor): Stubbed; returns false.
  • platform/Cursor.h: Define PlatformCursorHandle for each platform.
  • platform/win/WidgetWin.cpp: Reordered some includes. Added an include of Page.h. (WebCore::Widget::setCursor): Fixed a typo in the comment about ignoreNextSetCursor. Get the Page, and call its Chrome's setCursor() method with the passed cursor. Fall back to SetCursor() if the Page is not available.
  • plugins/win/PluginViewWin.cpp: (WebCore::PluginView::handleMouseEvent): Remove an unused variable. Cleaned up some whitespace.

WebKit/win/:

  • Interfaces/IWebUIDelegatePrivate.idl: Added a function to the IWebUIDelegatePrivate5 interface.
  • WebCoreSupport/WebChromeClient.cpp: (WebChromeClient::setCursor): Try to call the UI delegate's webViewSetCursor() method. If it fails, fall back to calling SetCursor() with the passed cursor.
  • WebCoreSupport/WebChromeClient.h:
  • WebView.cpp: (WebViewWndProc): Call the Chrome's setCursor() method with the last set cursor.
00:22 Changeset [42745] by oliver@apple.com

Build fix

04/21/09:

22:42 Changeset [42744] by oliver@apple.com

Build fix

22:36 Changeset [42743] by oliver@apple.com

Improve String.replace performance slightly

Reviewed by Maciej Stachowiak

Apply our vm reentry caching logic to String.replace with global
regexes.

22:22 Changeset [42742] by mitz@apple.com
  • updated another result with action element information
  • fast/encoding/mailto-always-utf-8-expected.txt:
22:05 Changeset [42741] by mitz@apple.com

WebKit/mac:

Reviewed by Jon Honeycutt.

  • Mac part of fixing for <rdar://problem/6755137> Action dictionary for policy decision is missing keys when full-page zoom is used
  • WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::actionDictionary): Use absoluteLocation() instead of pageX() and pageY(), which are adjusted for zoom.

WebKit/win:

Reviewed by Jon Honeycutt.

  • Windows part of fixing <rdar://problem/6755137> Action dictionary for policy decision is missing keys when full-page zoom is used
  • WebActionPropertyBag.cpp: (WebActionPropertyBag::Read): Use absoluteLocation() instead of clientX() and clientY(), which are adjusted for zoom.

LayoutTests:

  • test for <rdar://problem/6755137> Action dictionary for policy decision is missing keys when full-page zoom is used
  • fast/loader/policy-delegate-action-hit-test-zoomed-expected.txt: Added.
  • fast/loader/policy-delegate-action-hit-test-zoomed.html: Added.
22:04 Changeset [42740] by mitz@apple.com

WebKit/win:

Reviewed by Jon Honeycutt.

  • implement DOMNode::nodeName()
  • DOMCoreClasses.cpp: (DOMNode::nodeName):

WebKitTools:

Reviewed by Jon Honeycutt.

  • dump the node in the element dictionary in the action information passed to the policy delegate
  • DumpRenderTree/mac/PolicyDelegate.mm: (-[PolicyDelegate webView:decidePolicyForNavigationAction:request:frame:decisionListener:]):
  • DumpRenderTree/win/PolicyDelegate.cpp: (dumpPath): (PolicyDelegate::decidePolicyForNavigationAction):

LayoutTests:

Reviewed by Jon Honeycutt.

  • updated result that shows the node in the element dictionary in the action information passed to the policy delegate
  • http/tests/security/feed-urls-from-remote-expected.txt:
21:20 Changeset [42739] by dglazkov@chromium.org

2009-04-21 John Abd-El-Malek <jam@google.com>

Reviewed by Dimitri Glazkov.

Add a method to purge the Chromium plugin list cache.
https://bugs.webkit.org/show_bug.cgi?id=25318

  • plugins/chromium/PluginDataChromium.cpp: (WebCore::resetChromiumPluginCache):
21:03 Changeset [42738] by mrowe@apple.com

Fix <https://bugs.webkit.org/show_bug.cgi?id=25164> / <rdar://problem/6786319>.

Reviewed by Darin Adler.

We need to ensure that XMLTokenizer sets the current DocLoader before calling in to
any libxml2 methods that may trigger a load. The presence of a DocLoader indicates
that the load was originated by WebCore's use of libxml2 and that we should enforce
the same-origin policy on it. XMLTokenizer::initializeParserContext,
XMLTokenizer::doWrite and XMLTokenizer::doEnd were three methods that were not setting
the current DocLoader when they should have.

The XMLTokenizerScope class is introduced to simplify the pattern of saving, setting and
restoring the current DocLoader and libxml2 error handlers. The DocLoader and error handlers
are saved and set when the scope is allocated, and restored to their previous values when
the scope is exited.

Test: http/tests/security/xss-DENIED-xml-external-entity.xhtml

  • GNUmakefile.am:
  • WebCore.pro:
  • WebCore.scons:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • dom/XMLTokenizer.cpp:
  • dom/XMLTokenizer.h:
  • dom/XMLTokenizerLibxml2.cpp:

(WebCore::matchFunc):
(WebCore::shouldAllowExternalLoad):
(WebCore::openFunc):
(WebCore::XMLTokenizer::doWrite):
(WebCore::XMLTokenizer::initializeParserContext):
(WebCore::XMLTokenizer::doEnd):
(WebCore::xmlDocPtrForString):

  • dom/XMLTokenizerScope.cpp:

(WebCore::XMLTokenizerScope::XMLTokenizerScope):
(WebCore::XMLTokenizerScope::~XMLTokenizerScope):

  • dom/XMLTokenizerScope.h:
  • xml/XSLStyleSheet.cpp:

(WebCore::XSLStyleSheet::parseString):

  • xml/XSLTProcessor.cpp:
20:42 Changeset [42737] by weinig@apple.com

WebCore:

2009-04-21 Sam Weinig <sam@webkit.org>

Reviewed by Dan Bernstein.

Fix for <rdar://problem/5874009>
Add port 3659 (apple-sasl / PasswordServer) to the list of blocked ports.

  • platform/network/ResourceHandle.cpp: (WebCore::portAllowed):

LayoutTests:

2009-04-21 Sam Weinig <sam@webkit.org>

Reviewed by Dan Bernstein.

Update test and results for <rdar://problem/5874009>
Add port 3659 (apple-sasl / PasswordServer) to the list of blocked ports.

  • platform/mac-tiger/security/block-test-expected.txt:
  • platform/mac/security/block-test-expected.txt:
  • security/block-test.html:
19:18 Changeset [42736] by justin.garcia@apple.com

Corrected a mistake in my last ChangeLog entry and updated some expected results.

18:53 Changeset [42735] by justin.garcia@apple.com

WebCore:

2009-04-21 Justin Garcia <justin.garcia@apple.com>

Reviewed by Dan Bernstein.

https://bugs.webkit.org/show_bug.cgi?id=25320
Crash when changing block styles in a root that contains no visible content (but contains nodes with renderers)

  • editing/CompositeEditCommand.cpp: (WebCore::CompositeEditCommand::moveParagraphContentsToNewBlockIfNecessary): If the enclosing block is the root editable element and it contains no visible content, create a new block but don't try and move content into it, since there's nothing for moveParagraphs to move.

LayoutTests:

2009-04-21 Justin Garcia <justin.garcia@apple.com>

Reviewed by Dan Bernstein.

https://bugs.webkit.org/show_bug.cgi?id=25320
Crash when changing block styles in a root that contains no visible content (but contains nodes with renderers)

  • editing/execCommand/25320-expected.txt: Added.
  • editing/execCommand/25320.html: Added.
  • platform/mac/fast/forms/caret-rtl-expected.checksum: Replaced.
  • platform/mac/fast/forms/caret-rtl-expected.txt: Replaced.
17:18 Changeset [42734] by ggaren@apple.com

2009-04-21 Geoffrey Garen <ggaren@apple.com>

Reviewed by Cameron Zwarich and Oliver Hunt.


Re-Fixed <rdar://problem/6406045> REGRESSION: Stack overflow on PowerPC on
fast/workers/use-machine-stack.html (22531)


SunSpider reports no change.


Use a larger recursion limit on the main thread (because we can, and
there's some evidence that it may improve compatibility), and a smaller
recursion limit on secondary threads (because they tend to have smaller
stacks).

  • interpreter/Interpreter.cpp: (JSC::Interpreter::execute): (JSC::Interpreter::prepareForRepeatCall):
  • interpreter/Interpreter.h: (JSC::): Ditto. I wrote the recursion test slightly funny, so that the common case remains a simple compare to constant.
  • runtime/ArrayPrototype.cpp: (JSC::arrayProtoFuncToString): (JSC::arrayProtoFuncToLocaleString): (JSC::arrayProtoFuncJoin): Conservatively, set the array recursion limits to the lower, secondary thread limit. We can do something fancier if compatibility moves us, but this seems sufficient for now.
17:11 Changeset [42733] by sfalken@apple.com

2009-04-21 Steve Falkenburg <sfalken@apple.com>

Minor clean-ups to WebKitCreateInstance change.


Reviewed by Jon Honeycutt.

  • WebKitCOMAPI.cpp: (classFactory): (WebKitCreateInstance):
16:30 Changeset [42732] by darin@apple.com

WebCore:

2009-04-21 Darin Adler <darin@apple.com>

Reviewed by Dan Bernstein.

Bug 25162: REGRESSION (r41176-r41242): Drag-selection above/below the line does not select to beginning/end
https://bugs.webkit.org/show_bug.cgi?id=25162
rdar://problem/6764354

  • page/Settings.cpp: (WebCore::Settings::Settings): Initialize editing behavior to Mac-style for PLATFORM(MAC). Later we may want to change this default to encompass other versions running on Mac.
  • page/Settings.h: Added EditingBehavior, setEditingBehavior, editingBehavior, and m_editingBehavior. Also moved m_maximumDecodedImageSize out of the middle of all the bit fields.
  • platform/graphics/IntPoint.h: Added a constructor to turn an IntSize into an IntPoint. I'm not sure the distinction here is serving us well at the moment. When converting from global to local coordinates you want to do IntPoint - IntPoint and have the result be another IntPoint, not an IntSize. And so on.
  • rendering/RenderBlock.cpp: (WebCore::positionForPointRespectingEditingBoundaries): Changed to use pointers more and separate coordinates less. (WebCore::RenderBlock::positionForPointWithInlineChildren): Turned into a member function. Added separate cases to support the Mac and Windows behavior. (WebCore::RenderBlock::positionForPoint): Updated for the change above. Also moved the computation of pointInContents closer to the place it's used.
  • rendering/RenderBlock.h: Updated for the new private member function.

LayoutTests:

2009-04-21 Darin Adler <darin@apple.com>

Reviewed by Dan Bernstein.

Bug 25162: REGRESSION (r41176-r41242): Drag-selection above/below the line does not select to beginning/end
https://bugs.webkit.org/show_bug.cgi?id=25162
rdar://problem/6764354

  • editing/selection/resources/click-in-margins-inside-editable-div.js: Updated to expect different results on the Mac.
  • editing/selection/resources/click-in-padding-with-multiple-line-boxes.js: Ditto.
  • fast/css-generated-content/positioned-background-hit-test-crash.html: Clear the selection at the end of the test so the result won't include a caret.
  • platform/mac/editing/selection/click-in-margins-inside-editable-div-expected.txt:
  • platform/mac/editing/selection/click-in-padding-with-multiple-line-boxes-expected.txt: Copied from LayoutTests/editing/selection. Then changed to expect the different Mac results.
  • platform/mac/editing/selection/select-from-textfield-outwards-expected.checksum:
  • platform/mac/editing/selection/select-from-textfield-outwards-expected.png:
  • platform/mac/editing/selection/select-from-textfield-outwards-expected.txt: Changed to expect the different Mac results.
  • platform/mac/fast/css-generated-content/positioned-background-hit-test-crash-expected.txt: Changed to expect no caret.
  • platform/win/editing/selection/click-in-margins-inside-editable-div-expected.txt:
  • platform/win/editing/selection/click-in-padding-with-multiple-line-boxes-expected.txt: Copied from editing/selection.
  • platform/win/editing/selection/select-from-textfield-outwards-expected.checksum:
  • platform/win/editing/selection/select-from-textfield-outwards-expected.png:
  • platform/win/editing/selection/select-from-textfield-outwards-expected.txt: Copied from platform/mac/editing/selection.
14:58 Changeset [42731] by aroben@apple.com

Update StringsNotToBeLocalized.txt for recent changes

Rubber-stamped by John Sullivan.

  • StringsNotToBeLocalized.txt:
14:35 Changeset [42730] by andersca@apple.com

2009-04-21 Anders Carlsson <andersca@apple.com>

Reviewed by Darin Adler and Kevin Decker.

WebKit side of <rdar://problem/6781642>.


When we call resize with an actual changed size, block until the plug-in host is done.


  • Plugins/Hosted/NetscapePluginInstanceProxy.h:
  • Plugins/Hosted/NetscapePluginInstanceProxy.mm: (WebKit::NetscapePluginInstanceProxy::resize):
  • Plugins/Hosted/WebHostedNetscapePluginView.h:
  • Plugins/Hosted/WebHostedNetscapePluginView.mm: (-[WebHostedNetscapePluginView updateAndSetWindow]):
  • Plugins/Hosted/WebKitPluginHost.defs:
14:10 Changeset [42729] by aroben@apple.com

Windows build fix after r42726

  • DumpRenderTree/win/DumpRenderTree.cpp: Touched to force a rebuild.
  • DumpRenderTree/win/DumpRenderTree.vcproj: Delete any pre-r42726 manifest files during the pre-build event.
13:52 Changeset [42728] by timothy@apple.com

Another attempt to fix the Windows and QT builds.

13:48 Changeset [42727] by timothy@apple.com

Attempt to fix the Windows and QT builds.

13:42 Changeset [42726] by sfalken@apple.com

WebKit/win:

2009-04-21 Steve Falkenburg <sfalken@apple.com>

Add registry-free CoCreateInstance API for WebKit clients: WebKitCreateInstance.

Manifest-based registry-free COM continues to work for clients that have embedded
manifests with isolated COM information.


Reviewed by Adam Roben.

  • WebKit.vcproj/WebKit.def: Added WebKitCreateInstance.
  • WebKit.vcproj/WebKit.vcproj: Link against rpcrt4.dll for UUID hashing.
  • WebKit.vcproj/WebKit_debug.def: Added WebKitCreateInstance.
  • WebKitCOMAPI.cpp: Added. (CLSIDHash::hash): (CLSIDHash::equal): (CLSIDHashTraits::constructDeletedValue): (CLSIDHashTraits::isDeletedValue): (classFactory): (WebKitCreateInstance): New API.
  • WebKitCOMAPI.h: Added.

WebKitTools:

2009-04-21 Steve Falkenburg <sfalken@apple.com>

Use new WebKitCreateInstance API instead of CoCreateInstance
or COMPtr's createInstance to instantiate WebKit COM objects.

Reviewed by Adam Roben.

  • DumpRenderTree/win/DumpRenderTree.cpp: (initialize): (runTest): (createWebViewAndOffscreenWindow): (main):
  • DumpRenderTree/win/DumpRenderTree.vcproj:
  • DumpRenderTree/win/GCControllerWin.cpp: (GCController::collect): (GCController::collectOnAlternateThread): (GCController::getJSObjectCount):
  • DumpRenderTree/win/LayoutTestControllerWin.cpp: (LayoutTestController::keepWebHistory): (LayoutTestController::webHistoryItemCount): (LayoutTestController::setIconDatabaseEnabled):
  • DumpRenderTree/win/WorkQueueItemWin.cpp: (LoadItem::invoke):
13:21 Changeset [42725] by timothy@apple.com

Change pending unload and beforeunload counts to store the listeners in a single data structure that can be quickly iterated for dispatch.

WebCore:

2009-04-17 Timothy Hatcher <timothy@apple.com>

Change pending unload and beforeunload counts to store the listeners
in a single data structure that can be quickly iterated for dispatch.

<rdar://problem/6383352&6383379&6383940>

Reviewed by Darin Adler.

  • WebCore.base.exp: Change what symbols are exported.
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::stopLoading): Remove count updating code and move an if inside a block already checking m_frame->document().
  • page/Chrome.cpp: Remove disableSuddenTermination/enableSuddenTermination.
  • page/Chrome.h: Ditto.
  • page/ChromeClient.h: Ditto.
  • page/DOMWindow.cpp: (WebCore::pendingUnloadEventListenerMap): Create a static map. (WebCore::pendingBeforeUnloadEventListenerMap): Ditto. (WebCore::addPendingEventListener): Add the event listener to the map. (WebCore::removePendingEventListener): Remove the event listener from the map. (WebCore::removePendingEventListeners): Remove all listeners for the window. (WebCore::dispatchWindowEventToListeners): Dispatch the event to the given listeners. (WebCore::DOMWindow::dispatchAllPendingBeforeUnloadEvents): Send the beforeunload event. (WebCore::DOMWindow::pendingUnloadEventListeners): Return a count of pending listeners. (WebCore::DOMWindow::dispatchAllPendingUnloadEvents): Send the unload event. (WebCore::DOMWindow::~DOMWindow): Remove the window from both event listener maps. (WebCore::DOMWindow::addEventListener): Call addPendingEventListener when needed. (WebCore::DOMWindow::removeEventListener): Call removePendingEventListener when needed. (WebCore::DOMWindow::removeAllEventListeners): Call removePendingEventListeners. (WebCore::DOMWindow::removeInlineEventListenerForType): Call removePendingEventListener when needed.
  • page/DOMWindow.h: (WebCore::DOMWindow::frame): Changed to be const.
  • page/EventHandler.cpp: (WebCore::EventHandler::EventHandler): Remove count tracking code.
  • page/EventHandler.h: Ditto.
  • page/Page.cpp: (WebCore::Page::Page): Ditto.
  • page/Page.h: Ditto.

WebKit/mac:

2009-04-17 Timothy Hatcher <timothy@apple.com>

Change how sudden termination works with WebView teardown.

<rdar://problem/6383352&6383379&6383940>

Reviewed by Darin Adler.

  • WebCoreSupport/WebChromeClient.h: Remove disableSuddenTermination/enableSuddenTermination.
  • WebCoreSupport/WebChromeClient.mm: Ditto.
  • WebView/WebFrame.mm: (-[WebFrame _pendingFrameUnloadEventCount]): Ask the DOMWindow.
  • WebView/WebView.mm: (+[WebView canCloseAllWebViews]): Call DOMWindow::dispatchAllPendingBeforeUnloadEvents. (+[WebView closeAllWebViews]): Call DOMWindow::dispatchAllPendingUnloadEvents and call close on all the WebViews. (-[WebView _closeWithFastTeardown]): Remove code for unload event dispatch. (-[WebView _close]): Correct a comment. (+[WebView _applicationWillTerminate]): Call closeAllWebViews.
  • WebView/WebViewPrivate.h: Add canCloseAllWebViews and closeAllWebViews.
12:58 Changeset [42724] by ggaren@apple.com

JavaScriptCore:

2009-04-21 Geoffrey Garen <ggaren@apple.com>

Rubber-stamped by Adam Roben.


Disabled one more Mozilla JS test because it fails intermittently on Windows.
(See https://bugs.webkit.org/show_bug.cgi?id=25160.)

  • tests/mozilla/expected.html:

WebKitTools:

2009-04-21 Geoffrey Garen <ggaren@apple.com>

Rubber-stamped by Adam Roben.


Disabled one more Mozilla JS test because it fails intermittently on Windows.
(See https://bugs.webkit.org/show_bug.cgi?id=25160.)

  • Scripts/run-javascriptcore-tests:
12:38 Changeset [42723] by darin@apple.com

2009-04-21 Darin Adler <darin@apple.com>

Reviewed by Adele Peterson.

Fix some tests that are depending on Mac vs. Windows selection quirks accidentally.
Added failing tests to the skipped list for SnowLeopard.

  • editing/resources/select-all-iframe-src.html: Moved.
  • editing/selection/resources/select-all-iframe-src.html: Moved from editing/resources/select-all-iframe-src.html, since this is used by a test in the selection directory. Changed mouse coordinates to be farther to the right so we don't depend on the rule for what's selected when you click below text on the last line.
  • editing/selection/select-all-iframe.html: Updated for new location of file above.
  • fast/events/standalone-image-drag-to-editable.html: Changed mouse coordinates to be farther to the right so we don't depend on the rule for what's selected when you click below text on the last line.
  • platform/mac-snowleopard/Skipped: Added some tests to the Skipped list.
  • svg/custom/pointer-events-path.svg: Clear the selection at the end of the test so it's not in the test results. Also removed unneeded use of the layout test controller.
  • platform/mac/editing/selection/select-all-iframe-expected.checksum: Generated new results.
  • platform/mac/editing/selection/select-all-iframe-expected.png: Ditto.
  • platform/mac/editing/selection/select-all-iframe-expected.txt: Ditto.
  • platform/mac/fast/events/standalone-image-drag-to-editable-expected.checksum: Ditto.
  • platform/mac/fast/events/standalone-image-drag-to-editable-expected.png: Ditto.
  • platform/mac/fast/events/standalone-image-drag-to-editable-expected.txt: Ditto.
  • platform/mac/svg/custom/pointer-events-path-expected.checksum: Ditto.
  • platform/mac/svg/custom/pointer-events-path-expected.png: Ditto.
  • platform/mac/svg/custom/pointer-events-path-expected.txt: Ditto.
12:20 Changeset [42722] by justin.garcia@apple.com

WebCore:

2009-04-21 Justin Garcia <justin.garcia@apple.com>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=19644
Text copied with Select All pastes with a indent but shouldn't
<rdar://problem/6102483>
Pasting the content of an HTML message in Mail causes addition nested <div> elements to be added


To improve selectall/copy/paste fidelity of certain pages, we began wrapping copied content with
a div that held properties and attributes from the fully selected body. To fix the above issues,
only do this if if the body has certain properties or attributes. We'll begin adding to this list as
necessary. For now it's just background colors and images. Tested copy/paste of nytimes, wired,
arstechnica, and several others.


Massive nesting can still happen, it will just be much less common.

  • editing/markup.cpp: (WebCore::createMarkup):

LayoutTests:

2009-04-21 Justin Garcia <justin.garcia@apple.com>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=19644
Text copied with Select All pastes with a indent but shouldn't
<rdar://problem/6102483>
Pasting the content of an HTML message in Mail causes addition nested <div> elements to be added

  • editing/pasteboard/19644-1-expected.txt: Added.
  • editing/pasteboard/19644-1.html: Added.
  • editing/pasteboard/19644-2-expected.txt: Added.
  • editing/pasteboard/19644-2.html: Added.
  • platform/mac/editing/inserting/insert-3907422-fix-expected.txt:
  • platform/mac/editing/pasteboard/paste-text-015-expected.checksum:
  • platform/mac/editing/pasteboard/paste-text-015-expected.png:
  • platform/mac/editing/pasteboard/paste-text-015-expected.txt:
  • platform/mac/editing/style/font-family-with-space-expected.txt:
11:34 Changeset [42721] by pkasting@chromium.org

2009-04-16 Peter Kasting <pkasting@google.com>

Reviewed by David Hyatt.

https://bugs.webkit.org/show_bug.cgi?id=25250
Allow platforms to snap the scroll thumb back to the drag origin
during a drag. Implement functions for Safari/Win and Chromium/Win
to do this snapping at distances approximating the native ones.

  • platform/Scrollbar.cpp: (WebCore::Scrollbar::Scrollbar): (WebCore::Scrollbar::setValue): (WebCore::Scrollbar::scroll): (WebCore::Scrollbar::moveThumb): (WebCore::Scrollbar::setCurrentPos): (WebCore::Scrollbar::mouseMoved): (WebCore::Scrollbar::mouseDown):
  • platform/Scrollbar.h:
  • platform/ScrollbarTheme.h: (WebCore::ScrollbarTheme::shouldSnapBackToDragOrigin):
  • platform/chromium/ScrollbarThemeChromium.h:
  • platform/chromium/ScrollbarThemeChromiumLinux.cpp: (WebCore::ScrollbarThemeChromium::shouldSnapBackToDragOrigin):
  • platform/chromium/ScrollbarThemeChromiumWin.cpp: (WebCore::ScrollbarThemeChromium::shouldSnapBackToDragOrigin):
  • platform/win/ScrollbarThemeWin.cpp: (WebCore::ScrollbarThemeWin::shouldCenterOnThumb): (WebCore::ScrollbarThemeWin::shouldSnapBackToDragOrigin):
  • platform/win/ScrollbarThemeWin.h:
11:15 Changeset [42720] by aroben@apple.com

Windows build fix

  • bindings/js/JSLazyEventListener.cpp: (WebCore::eventParameterName): Remove unreachable code when SVG is enabled.
11:09 Changeset [42719] by aroben@apple.com

Fix running run-safari and run-webkit-tests from the paths recommended on webkit.org

Reviewed by Steve Falkenburg and Sam Weinig.

  • Scripts/run-webkit-nightly.cmd: Moved quotes from variable definitions to variable uses. Otherwise cmd.exe can get confused about the quoting.
  • Scripts/run-webkit-tests: Changed to use an absolute path to execAppWithEnv so that run-webkit-tests will work when run from outside the source tree.
11:09 Changeset [42718] by aroben@apple.com

Rename JavaScriptCore_debug.dll to JavaScriptCore.dll in the Debug configuration

JavaScriptCore:

Rename JavaScriptCore_debug.dll to JavaScriptCore.dll in the Debug
configuration

This matches the naming scheme for WebKit.dll, and will be necessary
once Safari links against JavaScriptCore.dll. This change also causes
run-safari not to fail (because the launcher printed by FindSafari was
always looking for JavaScriptCore.dll, never
JavaScriptCore_debug.dll).

Part of Bug 25305: can't run safari or drt on windows
<https://bugs.webkit.org/show_bug.cgi?id=25305>

Reviewed by Steve Falkenburg and Sam Weinig.

WebKit/win:

Rename JavaScriptCore_debug.dll to JavaScriptCore.dll in the Debug
configuration

This matches the naming scheme for WebKit.dll, and will be necessary
once Safari links against JavaScriptCore.dll. This change also causes
run-safari not to fail (because the launcher printed by FindSafari was
always looking for JavaScriptCore.dll, never
JavaScriptCore_debug.dll).

Part of Bug 25305: can't run safari or drt on windows
<https://bugs.webkit.org/show_bug.cgi?id=25305>

Reviewed by Steve Falkenburg and Sam Weinig.

  • WebKit.vcproj/WebKit.vcproj: Use $(WebKitDLLConfigSuffix) for naming JavaScriptCore.{dll,lib}.

WebKitTools:

Rename JavaScriptCore_debug.dll to JavaScriptCore.dll in the Debug
configuration

This matches the naming scheme for WebKit.dll, and will be necessary
once Safari links against JavaScriptCore.dll. This change also causes
run-safari not to fail (because the launcher printed by FindSafari was
always looking for JavaScriptCore.dll, never
JavaScriptCore_debug.dll).

Part of Bug 25305: can't run safari or drt on windows
<https://bugs.webkit.org/show_bug.cgi?id=25305>

Reviewed by Steve Falkenburg and Sam Weinig.

  • DumpRenderTree/win/DumpRenderTree.vcproj: Use $(WebKitDLLConfigSuffix) for naming JavaScriptCore.{dll,lib}.
11:08 Changeset [42717] by aroben@apple.com

Fix JavaScriptCore build on VC++ Express

Reviewed by Steve Falkenburg and Sam Weinig.

  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj: Link explicitly against gdi32.lib and oleaut32.lib.
11:04 BuildingOnWindows edited by aroben@apple.com
Removed now-obsolete comments about copying DLLs in order to run the … (diff)
10:26 Changeset [42716] by eric@webkit.org

2009-04-21 Dave Moore <davemoore@google.com> and Eric Seidel <eric@webkit.org>

Reviewed by Darin Adler.

Add test for (as of yet not implemented) HTML5 named option access.
https://bugs.webkit.org/show_bug.cgi?id=25191

I believe this will require separate implementations
for V8 and JSC the way our current bindings are designed.

  • fast/dom/HTMLSelectElement/named-options-expected.txt: Added.
  • fast/dom/HTMLSelectElement/named-options.html: Added.
  • fast/dom/HTMLSelectElement/resources/TEMPLATE.html: Copied from LayoutTests/fast/dom/CSSStyleDeclaration/resources/TEMPLATE.html.
  • fast/dom/HTMLSelectElement/resources/named-options.js: Added.
10:20 Changeset [42715] by kevino@webkit.org

!ENABLE(SVG) build fix.

10:00 Changeset [42714] by dglazkov@chromium.org

2009-04-21 Dave Moore <davemoore@google.com>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=25309
Better use AtomicStrings when calling from the V8
bindings into WebCore code.

  • bindings/v8/custom/V8DOMWindowCustom.cpp:
  • bindings/v8/custom/V8HTMLCollectionCustom.cpp:
  • bindings/v8/custom/V8HTMLFormElementCustom.cpp:
  • bindings/v8/custom/V8HTMLFrameSetElementCustom.cpp:
09:19 Changeset [42713] by cfleizach@apple.com

Bug 25304: AX: Safari not identifying a secure text field
https://bugs.webkit.org/show_bug.cgi?id=25304

09:10 Changeset [42712] by mitz@apple.com
  • fix the Tiger build
  • page/mac/EventHandlerMac.mm: (method_setImplementation):
08:48 Changeset [42711] by cwzwarich@webkit.org

2009-04-21 Cameron Zwarich <cwzwarich@uwaterloo.ca>

Reviewed by Darin Adler.

Add a test for bug 3235: CSS2: Table cell coalescing in CSS broken
<https://bugs.webkit.org/show_bug.cgi?id=3235>
<rdar://3299883/>

This was fixed long ago, but we should add a test to prevent a regression.

  • fast/table/cell-coalescing.html: Added.
  • platform/mac/fast/table/cell-coalescing-expected.txt: Added.
08:15 Changeset [42710] by ariya@webkit.org

2009-04-21 Antonio Gomes <antonio.gomes@openbossa.org>

Reviewed by Ariya Hidayat.

QWebElementSelection renamed to QWebElementCollection.

  • Api/qwebelement.cpp: (QWebElement::findAll): (QWebElementCollectionPrivate::QWebElementCollectionPrivate): (QWebElementCollectionPrivate::create): (QWebElementCollection::QWebElementCollection): (QWebElementCollection::operator=): (QWebElementCollection::~QWebElementCollection): (QWebElementCollection::operator+): (QWebElementCollection::append): (QWebElementCollection::count): (QWebElementCollection::at): (QWebElementCollection::toList):
  • Api/qwebelement.h:
  • Api/qwebframe.cpp: (QWebFrame::findAllElements):
  • Api/qwebframe.h:
  • QtLauncher/main.cpp: (MainWindow::selectElements):
  • tests/qwebelement/tst_qwebelement.cpp: (tst_QWebElement::simpleCollection): (tst_QWebElement::iteration): (tst_QWebElement::emptyCollection): (tst_QWebElement::appendCollection): (tst_QWebElement::nullSelect):
06:46 QtWebKitTodo edited by hausmann@webkit.org
(diff)
06:18 Changeset [42709] by ariya@webkit.org

2009-04-21 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Sam Weinig.

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

Fix the build with disabled DOM storage or disabled database support.

  • bindings/js/ScriptObjectQuarantine.cpp:
  • bindings/js/ScriptObjectQuarantine.h:
03:03 QtWebKitTodo edited by kenneth.christiansen@gmail.com
(diff)
02:27 Changeset [42708] by zecke@webkit.org

Compile fix after r42633. const on a static method does not work

01:33 QtWebKitTodo edited by kenneth.christiansen@gmail.com
(diff)
00:52 Changeset [42707] by mitz@apple.com

Reviewed by Darin Adler and John Sullivan.

  • fix <rdar://problem/5914146> Crash at -[NSScrollView scrollWheel:]

-[NSScrollView scrollWheel:] runs a nested event-tracking run loop
in a mode that allows WebCore timers to fire and NSURLConnection
callbacks to be dispatched, which can release the NSScrollView and
cause it to be deallocated (one example is a DOM timer callback that
removes a subframe from the document). This leads to a crash in
-scrollView:.

The fix is to replace the implementation of -[NSScrollView scrollWheel:]
with one that optionally retains the receiver for the duration of the
call.

  • page/mac/EventHandlerMac.mm: (WebCore::nsScrollViewScrollWheelShouldRetainSelf): Added. (WebCore::setNSScrollViewScrollWheelShouldRetainSelf): Added. Replaces the implementation of -[NSScrollView scrollWheel:] with the self-retaining one, and sets a static boolean that tells it whether to retain the receiver around the call to the original implementation. (WebCore::selfRetainingNSScrollViewScrollWheel): Added. If setNSScrollViewScrollWheelShouldRetainSelf(true) was called and this function is executing on the main thread, it retains the NSScrollView, invokes the original -[NSScrollView scrollWheel:], then releases the view. Otherwise it just calls through to the original implementation. (WebCore::EventHandler::passWheelEventToWidget): Added calls to setNSScrollViewScrollWheelShouldRetainSelf() around the call to -scrollWheel:.
00:48 Changeset [42706] by sfalken@apple.com

Update FindSafari for JavaScriptCore DLL changes.
Also removed code previously needed to support running nightlies against Safari 3 on Windows.


Reviewed by Mark Rowe.

  • FindSafari/FindSafari.cpp: (getStringValue): (getInstalledWebKitDirectory): (_tmain):
00:23 Changeset [42705] by ggaren@apple.com

JavaScriptCore:

2009-04-20 Geoffrey Garen <ggaren@apple.com>

Reviewed by Mark Rowe.


Tiger crash fix: Put VM tags in their own header file, and fixed up the
#ifdefs so they're not used on Tiger.

  • JavaScriptCore.xcodeproj/project.pbxproj:
  • interpreter/RegisterFile.h: (JSC::RegisterFile::RegisterFile):
  • jit/ExecutableAllocatorFixedVMPool.cpp: (JSC::FixedVMPoolAllocator::FixedVMPoolAllocator):
  • jit/ExecutableAllocatorPosix.cpp: (JSC::ExecutablePool::systemAlloc):
  • runtime/Collector.cpp: (JSC::allocateBlock):
  • wtf/VMTags.h: Added.

JavaScriptGlue:

2009-04-20 Geoffrey Garen <ggaren@apple.com>

Reviewed by Mark Rowe.

Tiger crash fix: Put VM tags in their own header file, and fixed up the
#ifdefs so they're not used on Tiger.

  • ForwardingHeaders/wtf/VMTags.h: Copied from JavaScriptGlue/ForwardingHeaders/wtf/HashTraits.h.

WebCore:

2009-04-20 Geoffrey Garen <ggaren@apple.com>

Reviewed by Mark Rowe.

Tiger crash fix: Put VM tags in their own header file, and fixed up the
#ifdefs so they're not used on Tiger.

  • ForwardingHeaders/wtf/VMTags.h: Copied from WebCore/ForwardingHeaders/wtf/HashTraits.h.

WebKit/mac:

2009-04-20 Geoffrey Garen <ggaren@apple.com>

Reviewed by Mark Rowe.

Tiger crash fix: Put VM tags in their own header file, and fixed up the
#ifdefs so they're not used on Tiger.

  • ForwardingHeaders/wtf/VMTags.h: Copied from ForwardingHeaders/wtf/HashTraits.h.
00:02 Changeset [42704] by xan@webkit.org

2009-04-21 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

https://bugs.webkit.org/show_bug.cgi?id=21546
[GTK] ATK accessibility enhancements

Based on a patch by Alp Toker.

Implement AtkComponent interface.

  • page/gtk/AccessibilityObjectWrapperAtk.cpp: (core):
00:01 Changeset [42703] by xan@webkit.org

2009-04-21 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

https://bugs.webkit.org/show_bug.cgi?id=21546
[GTK] ATK accessibility enhancements

Implement window<->screen coordinate transformation functions.

  • WebCoreSupport/ChromeClientGtk.cpp: (WebKit::widgetScreenPosition): (WebKit::ChromeClient::windowToScreen): (WebKit::ChromeClient::screenToWindow):
00:01 Changeset [42702] by xan@webkit.org

2009-04-21 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

https://bugs.webkit.org/show_bug.cgi?id=21546
[GTK] ATK accessibility enhancements

Only implement AtkAction interface if we actually have an action.

  • page/gtk/AccessibilityObjectWrapperAtk.cpp:

04/20/09:

23:41 Changeset [42701] by sfalken@apple.com

Windows build fix

23:34 Changeset [42700] by ap@webkit.org

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=25290
REGRESSION(r41732): Crash when constructing XMLHttpRequest in a detached document

Test: fast/dom/xmlhttprequest-constructor-in-detached-document.html

  • bindings/js/JSAudioConstructor.cpp: (WebCore::JSAudioConstructor::JSAudioConstructor):
  • bindings/js/JSAudioConstructor.h:
  • bindings/js/JSDOMGlobalObject.h: (WebCore::getDOMConstructor):
  • bindings/js/JSImageConstructor.cpp: (WebCore::JSImageConstructor::JSImageConstructor):
  • bindings/js/JSImageConstructor.h:
  • bindings/js/JSMessageChannelConstructor.cpp: (WebCore::JSMessageChannelConstructor::JSMessageChannelConstructor):
  • bindings/js/JSMessageChannelConstructor.h:
  • bindings/js/JSOptionConstructor.cpp: (WebCore::JSOptionConstructor::JSOptionConstructor):
  • bindings/js/JSOptionConstructor.h:
  • bindings/js/JSXMLHttpRequestConstructor.cpp: (WebCore::JSXMLHttpRequestConstructor::JSXMLHttpRequestConstructor):
  • bindings/js/JSXMLHttpRequestConstructor.h: Avoid accessing JSDOMGlobalObject via ScriptExecutionContext, since it may not work during frame teardown.
22:58 Changeset [42699] by ggaren@apple.com

2009-04-20 Geoffrey Garen <ggaren@apple.com>

Rubber stamped by Darin Adler and Sam Weinig.

Renamed "*InlineEventListener*" => "*AttributeEventListener*", and
"isInline" => "isAttribute".

  • bindings/js/JSDOMGlobalObject.cpp: (WebCore::JSDOMGlobalObject::createJSAttributeEventListener):
  • bindings/js/JSEventListener.cpp: (WebCore::JSEventListener::JSEventListener): (WebCore::JSEventListener::~JSEventListener): (WebCore::JSEventListener::handleEvent): (WebCore::JSEventListener::virtualisAttribute):
  • bindings/js/JSEventListener.h: (WebCore::JSEventListener::create): (WebCore::JSEventListener::isAttribute):
  • bindings/js/JSLazyEventListener.cpp: (WebCore::createAttributeEventListener):
  • bindings/js/JSLazyEventListener.h:
  • bindings/scripts/CodeGeneratorJS.pm:
  • dom/Element.cpp: (WebCore::Element::setWindowAttributeEventListener):
  • dom/Element.h:
  • dom/EventListener.h: (WebCore::EventListener::isAttribute): (WebCore::EventListener::virtualisAttribute):
  • dom/Node.cpp: (WebCore::Node::clearAttributeEventListener): (WebCore::Node::setAttributeEventListener): (WebCore::Node::getAttributeEventListener): (WebCore::Node::onabort): (WebCore::Node::setOnabort): (WebCore::Node::onblur): (WebCore::Node::setOnblur): (WebCore::Node::onchange): (WebCore::Node::setOnchange): (WebCore::Node::onclick): (WebCore::Node::setOnclick): (WebCore::Node::oncontextmenu): (WebCore::Node::setOncontextmenu): (WebCore::Node::ondblclick): (WebCore::Node::setOndblclick): (WebCore::Node::onerror): (WebCore::Node::setOnerror): (WebCore::Node::onfocus): (WebCore::Node::setOnfocus): (WebCore::Node::oninput): (WebCore::Node::setOninput): (WebCore::Node::onkeydown): (WebCore::Node::setOnkeydown): (WebCore::Node::onkeypress): (WebCore::Node::setOnkeypress): (WebCore::Node::onkeyup): (WebCore::Node::setOnkeyup): (WebCore::Node::onload): (WebCore::Node::setOnload): (WebCore::Node::onmousedown): (WebCore::Node::setOnmousedown): (WebCore::Node::onmousemove): (WebCore::Node::setOnmousemove): (WebCore::Node::onmouseout): (WebCore::Node::setOnmouseout): (WebCore::Node::onmouseover): (WebCore::Node::setOnmouseover): (WebCore::Node::onmouseup): (WebCore::Node::setOnmouseup): (WebCore::Node::onmousewheel): (WebCore::Node::setOnmousewheel): (WebCore::Node::onbeforecut): (WebCore::Node::setOnbeforecut): (WebCore::Node::oncut): (WebCore::Node::setOncut): (WebCore::Node::onbeforecopy): (WebCore::Node::setOnbeforecopy): (WebCore::Node::oncopy): (WebCore::Node::setOncopy): (WebCore::Node::onbeforepaste): (WebCore::Node::setOnbeforepaste): (WebCore::Node::onpaste): (WebCore::Node::setOnpaste): (WebCore::Node::ondragenter): (WebCore::Node::setOndragenter): (WebCore::Node::ondragover): (WebCore::Node::setOndragover): (WebCore::Node::ondragleave): (WebCore::Node::setOndragleave): (WebCore::Node::ondrop): (WebCore::Node::setOndrop): (WebCore::Node::ondragstart): (WebCore::Node::setOndragstart): (WebCore::Node::ondrag): (WebCore::Node::setOndrag): (WebCore::Node::ondragend): (WebCore::Node::setOndragend): (WebCore::Node::onreset): (WebCore::Node::setOnreset): (WebCore::Node::onresize): (WebCore::Node::setOnresize): (WebCore::Node::onscroll): (WebCore::Node::setOnscroll): (WebCore::Node::onsearch): (WebCore::Node::setOnsearch): (WebCore::Node::onselect): (WebCore::Node::setOnselect): (WebCore::Node::onselectstart): (WebCore::Node::setOnselectstart): (WebCore::Node::onsubmit): (WebCore::Node::setOnsubmit): (WebCore::Node::onunload): (WebCore::Node::setOnunload):
  • dom/Node.h:
  • editing/ReplaceSelectionCommand.cpp: (WebCore::ReplacementFragment::ReplacementFragment):
  • html/HTMLBodyElement.cpp: (WebCore::HTMLBodyElement::parseMappedAttribute):
  • html/HTMLButtonElement.cpp: (WebCore::HTMLButtonElement::parseMappedAttribute):
  • html/HTMLElement.cpp: (WebCore::HTMLElement::parseMappedAttribute):
  • html/HTMLFormElement.cpp: (WebCore::HTMLFormElement::parseMappedAttribute):
  • html/HTMLFrameElementBase.cpp: (WebCore::HTMLFrameElementBase::parseMappedAttribute):
  • html/HTMLFrameSetElement.cpp: (WebCore::HTMLFrameSetElement::parseMappedAttribute):
  • html/HTMLImageElement.cpp: (WebCore::HTMLImageElement::parseMappedAttribute):
  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::parseMappedAttribute):
  • html/HTMLObjectElement.cpp: (WebCore::HTMLObjectElement::parseMappedAttribute):
  • html/HTMLScriptElement.cpp: (WebCore::HTMLScriptElement::parseMappedAttribute):
  • html/HTMLSelectElement.cpp: (WebCore::HTMLSelectElement::parseMappedAttribute):
  • html/HTMLTextAreaElement.cpp: (WebCore::HTMLTextAreaElement::parseMappedAttribute):
  • page/AccessibilityRenderObject.cpp: (WebCore::AccessibilityRenderObject::mouseButtonListener):
  • page/DOMWindow.cpp: (WebCore::DOMWindow::setAttributeEventListener): (WebCore::DOMWindow::clearAttributeEventListener): (WebCore::DOMWindow::getAttributeEventListener): (WebCore::DOMWindow::onabort): (WebCore::DOMWindow::setOnabort): (WebCore::DOMWindow::onblur): (WebCore::DOMWindow::setOnblur): (WebCore::DOMWindow::onchange): (WebCore::DOMWindow::setOnchange): (WebCore::DOMWindow::onclick): (WebCore::DOMWindow::setOnclick): (WebCore::DOMWindow::ondblclick): (WebCore::DOMWindow::setOndblclick): (WebCore::DOMWindow::onerror): (WebCore::DOMWindow::setOnerror): (WebCore::DOMWindow::onfocus): (WebCore::DOMWindow::setOnfocus): (WebCore::DOMWindow::onkeydown): (WebCore::DOMWindow::setOnkeydown): (WebCore::DOMWindow::onkeypress): (WebCore::DOMWindow::setOnkeypress): (WebCore::DOMWindow::onkeyup): (WebCore::DOMWindow::setOnkeyup): (WebCore::DOMWindow::onload): (WebCore::DOMWindow::setOnload): (WebCore::DOMWindow::onmousedown): (WebCore::DOMWindow::setOnmousedown): (WebCore::DOMWindow::onmousemove): (WebCore::DOMWindow::setOnmousemove): (WebCore::DOMWindow::onmouseout): (WebCore::DOMWindow::setOnmouseout): (WebCore::DOMWindow::onmouseover): (WebCore::DOMWindow::setOnmouseover): (WebCore::DOMWindow::onmouseup): (WebCore::DOMWindow::setOnmouseup): (WebCore::DOMWindow::onmousewheel): (WebCore::DOMWindow::setOnmousewheel): (WebCore::DOMWindow::onreset): (WebCore::DOMWindow::setOnreset): (WebCore::DOMWindow::onresize): (WebCore::DOMWindow::setOnresize): (WebCore::DOMWindow::onscroll): (WebCore::DOMWindow::setOnscroll): (WebCore::DOMWindow::onsearch): (WebCore::DOMWindow::setOnsearch): (WebCore::DOMWindow::onselect): (WebCore::DOMWindow::setOnselect): (WebCore::DOMWindow::onsubmit): (WebCore::DOMWindow::setOnsubmit): (WebCore::DOMWindow::onunload): (WebCore::DOMWindow::setOnunload): (WebCore::DOMWindow::onbeforeunload): (WebCore::DOMWindow::setOnbeforeunload): (WebCore::DOMWindow::onwebkitanimationstart): (WebCore::DOMWindow::setOnwebkitanimationstart): (WebCore::DOMWindow::onwebkitanimationiteration): (WebCore::DOMWindow::setOnwebkitanimationiteration): (WebCore::DOMWindow::onwebkitanimationend): (WebCore::DOMWindow::setOnwebkitanimationend): (WebCore::DOMWindow::onwebkittransitionend): (WebCore::DOMWindow::setOnwebkittransitionend):
  • page/DOMWindow.h:
  • svg/SVGElement.cpp: (WebCore::SVGElement::parseMappedAttribute):
  • svg/SVGElementInstance.cpp: (WebCore::SVGElementInstance::onabort): (WebCore::SVGElementInstance::setOnabort): (WebCore::SVGElementInstance::onblur): (WebCore::SVGElementInstance::setOnblur): (WebCore::SVGElementInstance::onchange): (WebCore::SVGElementInstance::setOnchange): (WebCore::SVGElementInstance::onclick): (WebCore::SVGElementInstance::setOnclick): (WebCore::SVGElementInstance::oncontextmenu): (WebCore::SVGElementInstance::setOncontextmenu): (WebCore::SVGElementInstance::ondblclick): (WebCore::SVGElementInstance::setOndblclick): (WebCore::SVGElementInstance::onerror): (WebCore::SVGElementInstance::setOnerror): (WebCore::SVGElementInstance::onfocus): (WebCore::SVGElementInstance::setOnfocus): (WebCore::SVGElementInstance::oninput): (WebCore::SVGElementInstance::setOninput): (WebCore::SVGElementInstance::onkeydown): (WebCore::SVGElementInstance::setOnkeydown): (WebCore::SVGElementInstance::onkeypress): (WebCore::SVGElementInstance::setOnkeypress): (WebCore::SVGElementInstance::onkeyup): (WebCore::SVGElementInstance::setOnkeyup): (WebCore::SVGElementInstance::onload): (WebCore::SVGElementInstance::setOnload): (WebCore::SVGElementInstance::onmousedown): (WebCore::SVGElementInstance::setOnmousedown): (WebCore::SVGElementInstance::onmousemove): (WebCore::SVGElementInstance::setOnmousemove): (WebCore::SVGElementInstance::onmouseout): (WebCore::SVGElementInstance::setOnmouseout): (WebCore::SVGElementInstance::onmouseover): (WebCore::SVGElementInstance::setOnmouseover): (WebCore::SVGElementInstance::onmouseup): (WebCore::SVGElementInstance::setOnmouseup): (WebCore::SVGElementInstance::onmousewheel): (WebCore::SVGElementInstance::setOnmousewheel): (WebCore::SVGElementInstance::onbeforecut): (WebCore::SVGElementInstance::setOnbeforecut): (WebCore::SVGElementInstance::oncut): (WebCore::SVGElementInstance::setOncut): (WebCore::SVGElementInstance::onbeforecopy): (WebCore::SVGElementInstance::setOnbeforecopy): (WebCore::SVGElementInstance::oncopy): (WebCore::SVGElementInstance::setOncopy): (WebCore::SVGElementInstance::onbeforepaste): (WebCore::SVGElementInstance::setOnbeforepaste): (WebCore::SVGElementInstance::onpaste): (WebCore::SVGElementInstance::setOnpaste): (WebCore::SVGElementInstance::ondragenter): (WebCore::SVGElementInstance::setOndragenter): (WebCore::SVGElementInstance::ondragover): (WebCore::SVGElementInstance::setOndragover): (WebCore::SVGElementInstance::ondragleave): (WebCore::SVGElementInstance::setOndragleave): (WebCore::SVGElementInstance::ondrop): (WebCore::SVGElementInstance::setOndrop): (WebCore::SVGElementInstance::ondragstart): (WebCore::SVGElementInstance::setOndragstart): (WebCore::SVGElementInstance::ondrag): (WebCore::SVGElementInstance::setOndrag): (WebCore::SVGElementInstance::ondragend): (WebCore::SVGElementInstance::setOndragend): (WebCore::SVGElementInstance::onreset): (WebCore::SVGElementInstance::setOnreset): (WebCore::SVGElementInstance::onresize): (WebCore::SVGElementInstance::setOnresize): (WebCore::SVGElementInstance::onscroll): (WebCore::SVGElementInstance::setOnscroll): (WebCore::SVGElementInstance::onsearch): (WebCore::SVGElementInstance::setOnsearch): (WebCore::SVGElementInstance::onselect): (WebCore::SVGElementInstance::setOnselect): (WebCore::SVGElementInstance::onselectstart): (WebCore::SVGElementInstance::setOnselectstart): (WebCore::SVGElementInstance::onsubmit): (WebCore::SVGElementInstance::setOnsubmit): (WebCore::SVGElementInstance::onunload): (WebCore::SVGElementInstance::setOnunload):
  • svg/SVGSVGElement.cpp: (WebCore::SVGSVGElement::parseMappedAttribute):
22:17 Changeset [42698] by ggaren@apple.com

WebCore:

2009-04-20 Geoffrey Garen <ggaren@apple.com>

Approved by Mark Rowe, Cameron Zwarich, Oliver Hunt, and Ojan Vafai.


Used svn merge to roll out revisions 42678, 42690, 42694, 42697 because
they broke the Tiger and Windows buildbots.

  • css/html4.css:
  • css/themeWin.css:
  • platform/graphics/SimpleFontData.cpp:
  • platform/graphics/SimpleFontData.h:
  • platform/graphics/chromium/SimpleFontDataChromiumWin.cpp:
  • platform/graphics/chromium/SimpleFontDataLinux.cpp:
  • platform/graphics/gtk/SimpleFontDataGtk.cpp:
  • platform/graphics/gtk/SimpleFontDataPango.cpp:
  • platform/graphics/mac/SimpleFontDataMac.mm:
  • platform/graphics/qt/SimpleFontDataQt.cpp:
  • platform/graphics/win/SimpleFontDataCGWin.cpp:
  • platform/graphics/win/SimpleFontDataCairoWin.cpp:
  • platform/graphics/win/SimpleFontDataWin.cpp:
  • platform/graphics/wx/SimpleFontDataWx.cpp:
  • rendering/RenderTextControl.cpp:
  • rendering/RenderTextControlMultiLine.cpp:
  • rendering/RenderTextControlSingleLine.cpp:

LayoutTests:

2009-04-20 Geoffrey Garen <ggaren@apple.com>

Approved by Mark Rowe, Cameron Zwarich, Oliver Hunt, and Ojan Vafai.


Used svn merge to roll out revisions 42678, 42690, 42694, 42697 because
they broke the Tiger and Windows buildbots.

  • fast/forms/drag-out-of-textarea.html:
  • fast/forms/resources/TEMPLATE.html: Removed.
  • fast/forms/resources/text-control-intrinsic-widths.js: Removed.
  • fast/forms/resources/textarea-metrics.js: Removed.
  • fast/forms/search-abs-pos-cancel-button.html:
  • fast/forms/text-control-intrinsic-widths-expected.txt: Removed.
  • fast/forms/text-control-intrinsic-widths.html: Removed.
  • fast/forms/textarea-metrics-expected.txt: Removed.
  • fast/forms/textarea-metrics.html: Removed.
  • fast/forms/textarea-width.html:
  • fast/replaced/table-percent-height-expected.txt:
  • fast/replaced/table-percent-height.html:
  • platform/mac-leopard/fast/forms/input-text-maxlength-expected.checksum:
  • platform/mac-leopard/fast/forms/input-text-maxlength-expected.png:
  • platform/mac-leopard/fast/forms/input-text-maxlength-expected.txt:
  • platform/mac-leopard/fast/forms/input-text-paste-maxlength-expected.checksum:
  • platform/mac-leopard/fast/forms/input-text-paste-maxlength-expected.png:
  • platform/mac-leopard/fast/forms/input-text-paste-maxlength-expected.txt:
  • platform/mac/editing/deleting/5168598-expected.checksum:
  • platform/mac/editing/deleting/5168598-expected.png:
  • platform/mac/editing/deleting/5168598-expected.txt:
  • platform/mac/editing/inserting/4960120-1-expected.checksum:
  • platform/mac/editing/inserting/4960120-1-expected.png:
  • platform/mac/editing/inserting/4960120-1-expected.txt:
  • platform/mac/editing/inserting/5607069-2-expected.checksum:
  • platform/mac/editing/inserting/5607069-2-expected.png:
  • platform/mac/editing/inserting/5607069-2-expected.txt:
  • platform/mac/editing/inserting/5607069-3-expected.checksum:
  • platform/mac/editing/inserting/5607069-3-expected.png:
  • platform/mac/editing/inserting/5607069-3-expected.txt:
  • platform/mac/editing/inserting/before-after-input-element-expected.checksum:
  • platform/mac/editing/inserting/before-after-input-element-expected.png:
  • platform/mac/editing/inserting/before-after-input-element-expected.txt:
  • platform/mac/editing/inserting/typing-tab-designmode-forms-expected.checksum:
  • platform/mac/editing/inserting/typing-tab-designmode-forms-expected.png:
  • platform/mac/editing/inserting/typing-tab-designmode-forms-expected.txt:
  • platform/mac/editing/pasteboard/4806874-expected.checksum:
  • platform/mac/editing/pasteboard/4806874-expected.png:
  • platform/mac/editing/pasteboard/4806874-expected.txt:
  • platform/mac/editing/pasteboard/drop-text-without-selection-expected.checksum:
  • platform/mac/editing/pasteboard/drop-text-without-selection-expected.png:
  • platform/mac/editing/pasteboard/drop-text-without-selection-expected.txt:
  • platform/mac/editing/pasteboard/input-field-1-expected.checksum:
  • platform/mac/editing/pasteboard/input-field-1-expected.png:
  • platform/mac/editing/pasteboard/input-field-1-expected.txt:
  • platform/mac/editing/pasteboard/nested-blocks-with-text-area-expected.checksum:
  • platform/mac/editing/pasteboard/nested-blocks-with-text-area-expected.png:
  • platform/mac/editing/pasteboard/nested-blocks-with-text-area-expected.txt:
  • platform/mac/editing/pasteboard/nested-blocks-with-text-field-expected.checksum:
  • platform/mac/editing/pasteboard/nested-blocks-with-text-field-expected.png:
  • platform/mac/editing/pasteboard/nested-blocks-with-text-field-expected.txt:
  • platform/mac/editing/pasteboard/pasting-tabs-expected.checksum:
  • platform/mac/editing/pasteboard/pasting-tabs-expected.png:
  • platform/mac/editing/pasteboard/pasting-tabs-expected.txt:
  • platform/mac/editing/selection/3690703-2-expected.checksum:
  • platform/mac/editing/selection/3690703-2-expected.png:
  • platform/mac/editing/selection/3690703-2-expected.txt:
  • platform/mac/editing/selection/3690703-expected.checksum:
  • platform/mac/editing/selection/3690703-expected.png:
  • platform/mac/editing/selection/3690703-expected.txt:
  • platform/mac/editing/selection/3690719-expected.checksum:
  • platform/mac/editing/selection/3690719-expected.png:
  • platform/mac/editing/selection/3690719-expected.txt:
  • platform/mac/editing/selection/4895428-3-expected.checksum:
  • platform/mac/editing/selection/4895428-3-expected.png:
  • platform/mac/editing/selection/4895428-3-expected.txt:
  • platform/mac/editing/selection/4975120-expected.checksum:
  • platform/mac/editing/selection/4975120-expected.png:
  • platform/mac/editing/selection/4975120-expected.txt:
  • platform/mac/editing/selection/5136696-expected.checksum:
  • platform/mac/editing/selection/5136696-expected.png:
  • platform/mac/editing/selection/5136696-expected.txt:
  • platform/mac/editing/selection/5213963-expected.checksum:
  • platform/mac/editing/selection/5213963-expected.png:
  • platform/mac/editing/selection/5213963-expected.txt:
  • platform/mac/editing/selection/drag-select-1-expected.checksum:
  • platform/mac/editing/selection/drag-select-1-expected.png:
  • platform/mac/editing/selection/drag-select-1-expected.txt:
  • platform/mac/editing/selection/drag-text-delay-expected.checksum:
  • platform/mac/editing/selection/drag-text-delay-expected.png:
  • platform/mac/editing/selection/drag-text-delay-expected.txt:
  • platform/mac/editing/selection/select-from-textfield-outwards-expected.checksum:
  • platform/mac/editing/selection/select-from-textfield-outwards-expected.png:
  • platform/mac/editing/selection/select-from-textfield-outwards-expected.txt:
  • platform/mac/fast/block/margin-collapse/103-expected.txt:
  • platform/mac/fast/css/line-height-expected.checksum:
  • platform/mac/fast/css/line-height-expected.png:
  • platform/mac/fast/css/line-height-expected.txt:
  • platform/mac/fast/css/resize-corner-tracking-expected.checksum:
  • platform/mac/fast/css/resize-corner-tracking-expected.png:
  • platform/mac/fast/css/resize-corner-tracking-expected.txt:
  • platform/mac/fast/dom/HTMLTextAreaElement/reset-textarea-expected.checksum:
  • platform/mac/fast/dom/HTMLTextAreaElement/reset-textarea-expected.png:
  • platform/mac/fast/dom/HTMLTextAreaElement/reset-textarea-expected.txt:
  • platform/mac/fast/dom/isindex-002-expected.checksum:
  • platform/mac/fast/dom/isindex-002-expected.png:
  • platform/mac/fast/dom/isindex-002-expected.txt:
  • platform/mac/fast/dynamic/008-expected.checksum:
  • platform/mac/fast/dynamic/008-expected.png:
  • platform/mac/fast/dynamic/008-expected.txt:
  • platform/mac/fast/events/autoscroll-expected.checksum:
  • platform/mac/fast/events/autoscroll-expected.png:
  • platform/mac/fast/events/autoscroll-expected.txt:
  • platform/mac/fast/events/context-no-deselect-expected.checksum:
  • platform/mac/fast/events/context-no-deselect-expected.png:
  • platform/mac/fast/events/context-no-deselect-expected.txt:
  • platform/mac/fast/events/label-focus-expected.checksum:
  • platform/mac/fast/events/label-focus-expected.png:
  • platform/mac/fast/events/label-focus-expected.txt:
  • platform/mac/fast/forms/basic-textareas-expected.checksum:
  • platform/mac/fast/forms/basic-textareas-expected.png:
  • platform/mac/fast/forms/basic-textareas-expected.txt:
  • platform/mac/fast/forms/box-shadow-override-expected.checksum:
  • platform/mac/fast/forms/box-shadow-override-expected.png:
  • platform/mac/fast/forms/box-shadow-override-expected.txt:
  • platform/mac/fast/forms/control-restrict-line-height-expected.checksum:
  • platform/mac/fast/forms/control-restrict-line-height-expected.png:
  • platform/mac/fast/forms/control-restrict-line-height-expected.txt:
  • platform/mac/fast/forms/encoding-test-expected.checksum:
  • platform/mac/fast/forms/encoding-test-expected.png:
  • platform/mac/fast/forms/encoding-test-expected.txt:
  • platform/mac/fast/forms/fieldset-align-expected.checksum:
  • platform/mac/fast/forms/fieldset-align-expected.png:
  • platform/mac/fast/forms/fieldset-align-expected.txt:
  • platform/mac/fast/forms/form-element-geometry-expected.checksum:
  • platform/mac/fast/forms/form-element-geometry-expected.png:
  • platform/mac/fast/forms/form-element-geometry-expected.txt:
  • platform/mac/fast/forms/input-align-expected.checksum:
  • platform/mac/fast/forms/input-align-expected.png:
  • platform/mac/fast/forms/input-align-expected.txt:
  • platform/mac/fast/forms/input-appearance-bkcolor-expected.checksum:
  • platform/mac/fast/forms/input-appearance-bkcolor-expected.png:
  • platform/mac/fast/forms/input-appearance-bkcolor-expected.txt:
  • platform/mac/fast/forms/input-appearance-default-bkcolor-expected.checksum:
  • platform/mac/fast/forms/input-appearance-default-bkcolor-expected.png:
  • platform/mac/fast/forms/input-appearance-default-bkcolor-expected.txt:
  • platform/mac/fast/forms/input-appearance-disabled-expected.checksum:
  • platform/mac/fast/forms/input-appearance-disabled-expected.png:
  • platform/mac/fast/forms/input-appearance-disabled-expected.txt:
  • platform/mac/fast/forms/input-appearance-focus-expected.checksum:
  • platform/mac/fast/forms/input-appearance-focus-expected.png:
  • platform/mac/fast/forms/input-appearance-focus-expected.txt:
  • platform/mac/fast/forms/input-appearance-height-expected.checksum:
  • platform/mac/fast/forms/input-appearance-height-expected.png:
  • platform/mac/fast/forms/input-appearance-height-expected.txt:
  • platform/mac/fast/forms/input-appearance-preventDefault-expected.checksum:
  • platform/mac/fast/forms/input-appearance-preventDefault-expected.png:
  • platform/mac/fast/forms/input-appearance-preventDefault-expected.txt:
  • platform/mac/fast/forms/input-appearance-readonly-expected.checksum:
  • platform/mac/fast/forms/input-appearance-readonly-expected.png:
  • platform/mac/fast/forms/input-appearance-readonly-expected.txt:
  • platform/mac/fast/forms/input-appearance-selection-expected.checksum:
  • platform/mac/fast/forms/input-appearance-selection-expected.png:
  • platform/mac/fast/forms/input-appearance-selection-expected.txt:
  • platform/mac/fast/forms/input-appearance-visibility-expected.checksum:
  • platform/mac/fast/forms/input-appearance-visibility-expected.png:
  • platform/mac/fast/forms/input-appearance-visibility-expected.txt:
  • platform/mac/fast/forms/input-appearance-width-expected.checksum:
  • platform/mac/fast/forms/input-appearance-width-expected.png:
  • platform/mac/fast/forms/input-appearance-width-expected.txt:
  • platform/mac/fast/forms/input-baseline-expected.checksum:
  • platform/mac/fast/forms/input-baseline-expected.png:
  • platform/mac/fast/forms/input-baseline-expected.txt:
  • platform/mac/fast/forms/input-disabled-color-expected.checksum:
  • platform/mac/fast/forms/input-disabled-color-expected.png:
  • platform/mac/fast/forms/input-disabled-color-expected.txt:
  • platform/mac/fast/forms/input-double-click-selection-gap-bug-expected.checksum:
  • platform/mac/fast/forms/input-double-click-selection-gap-bug-expected.png:
  • platform/mac/fast/forms/input-double-click-selection-gap-bug-expected.txt:
  • platform/mac/fast/forms/input-paste-undo-expected.checksum:
  • platform/mac/fast/forms/input-paste-undo-expected.png:
  • platform/mac/fast/forms/input-paste-undo-expected.txt:
  • platform/mac/fast/forms/input-readonly-autoscroll-expected.checksum:
  • platform/mac/fast/forms/input-readonly-autoscroll-expected.png:
  • platform/mac/fast/forms/input-readonly-autoscroll-expected.txt:
  • platform/mac/fast/forms/input-readonly-dimmed-expected.checksum:
  • platform/mac/fast/forms/input-readonly-dimmed-expected.png:
  • platform/mac/fast/forms/input-readonly-dimmed-expected.txt:
  • platform/mac/fast/forms/input-readonly-empty-expected.checksum:
  • platform/mac/fast/forms/input-readonly-empty-expected.png:
  • platform/mac/fast/forms/input-readonly-empty-expected.txt:
  • platform/mac/fast/forms/input-spaces-expected.checksum:
  • platform/mac/fast/forms/input-spaces-expected.png:
  • platform/mac/fast/forms/input-spaces-expected.txt:
  • platform/mac/fast/forms/input-table-expected.txt:
  • platform/mac/fast/forms/input-text-click-inside-expected.checksum:
  • platform/mac/fast/forms/input-text-click-inside-expected.png:
  • platform/mac/fast/forms/input-text-click-inside-expected.txt:
  • platform/mac/fast/forms/input-text-click-outside-expected.checksum:
  • platform/mac/fast/forms/input-text-click-outside-expected.png:
  • platform/mac/fast/forms/input-text-click-outside-expected.txt:
  • platform/mac/fast/forms/input-text-double-click-expected.checksum:
  • platform/mac/fast/forms/input-text-double-click-expected.png:
  • platform/mac/fast/forms/input-text-double-click-expected.txt:
  • platform/mac/fast/forms/input-text-drag-down-expected.checksum:
  • platform/mac/fast/forms/input-text-drag-down-expected.png:
  • platform/mac/fast/forms/input-text-drag-down-expected.txt:
  • platform/mac/fast/forms/input-text-option-delete-expected.checksum:
  • platform/mac/fast/forms/input-text-option-delete-expected.png:
  • platform/mac/fast/forms/input-text-option-delete-expected.txt:
  • platform/mac/fast/forms/input-text-scroll-left-on-blur-expected.checksum:
  • platform/mac/fast/forms/input-text-scroll-left-on-blur-expected.png:
  • platform/mac/fast/forms/input-text-scroll-left-on-blur-expected.txt:
  • platform/mac/fast/forms/input-text-self-emptying-click-expected.checksum:
  • platform/mac/fast/forms/input-text-self-emptying-click-expected.png:
  • platform/mac/fast/forms/input-text-self-emptying-click-expected.txt:
  • platform/mac/fast/forms/input-text-word-wrap-expected.checksum:
  • platform/mac/fast/forms/input-text-word-wrap-expected.png:
  • platform/mac/fast/forms/input-text-word-wrap-expected.txt:
  • platform/mac/fast/forms/input-type-change2-expected.checksum:
  • platform/mac/fast/forms/input-type-change2-expected.png:
  • platform/mac/fast/forms/input-type-change2-expected.txt:
  • platform/mac/fast/forms/input-type-text-min-width-expected.checksum:
  • platform/mac/fast/forms/input-type-text-min-width-expected.png:
  • platform/mac/fast/forms/input-type-text-min-width-expected.txt:
  • platform/mac/fast/forms/input-value-expected.checksum:
  • platform/mac/fast/forms/input-value-expected.png:
  • platform/mac/fast/forms/input-value-expected.txt:
  • platform/mac/fast/forms/input-width-expected.checksum:
  • platform/mac/fast/forms/input-width-expected.png:
  • platform/mac/fast/forms/input-width-expected.txt:
  • platform/mac/fast/forms/linebox-overflow-in-textarea-padding-expected.checksum:
  • platform/mac/fast/forms/linebox-overflow-in-textarea-padding-expected.png:
  • platform/mac/fast/forms/linebox-overflow-in-textarea-padding-expected.txt:
  • platform/mac/fast/forms/negativeLineHeight-expected.checksum:
  • platform/mac/fast/forms/negativeLineHeight-expected.png:
  • platform/mac/fast/forms/negativeLineHeight-expected.txt:
  • platform/mac/fast/forms/password-placeholder-expected.checksum:
  • platform/mac/fast/forms/password-placeholder-expected.png:
  • platform/mac/fast/forms/password-placeholder-expected.txt:
  • platform/mac/fast/forms/password-placeholder-text-security-expected.checksum:
  • platform/mac/fast/forms/password-placeholder-text-security-expected.png:
  • platform/mac/fast/forms/password-placeholder-text-security-expected.txt:
  • platform/mac/fast/forms/placeholder-pseudo-style-expected.checksum:
  • platform/mac/fast/forms/placeholder-pseudo-style-expected.png:
  • platform/mac/fast/forms/placeholder-pseudo-style-expected.txt:
  • platform/mac/fast/forms/placeholder-set-attribute-expected.checksum:
  • platform/mac/fast/forms/placeholder-set-attribute-expected.png:
  • platform/mac/fast/forms/placeholder-set-attribute-expected.txt:
  • platform/mac/fast/forms/placeholder-set-value-expected.checksum:
  • platform/mac/fast/forms/placeholder-set-value-expected.png:
  • platform/mac/fast/forms/placeholder-set-value-expected.txt:
  • platform/mac/fast/forms/search-cancel-button-style-sharing-expected.checksum:
  • platform/mac/fast/forms/search-cancel-button-style-sharing-expected.png:
  • platform/mac/fast/forms/search-cancel-button-style-sharing-expected.txt:
  • platform/mac/fast/forms/search-display-none-cancel-button-expected.checksum:
  • platform/mac/fast/forms/search-display-none-cancel-button-expected.png:
  • platform/mac/fast/forms/search-display-none-cancel-button-expected.txt:
  • platform/mac/fast/forms/search-placeholder-value-changed-expected.checksum:
  • platform/mac/fast/forms/search-placeholder-value-changed-expected.png:
  • platform/mac/fast/forms/search-placeholder-value-changed-expected.txt:
  • platform/mac/fast/forms/search-rtl-expected.checksum:
  • platform/mac/fast/forms/search-rtl-expected.png:
  • platform/mac/fast/forms/search-rtl-expected.txt:
  • platform/mac/fast/forms/search-transformed-expected.checksum:
  • platform/mac/fast/forms/search-transformed-expected.png:
  • platform/mac/fast/forms/search-transformed-expected.txt:
  • platform/mac/fast/forms/search-zoomed-expected.checksum:
  • platform/mac/fast/forms/search-zoomed-expected.png:
  • platform/mac/fast/forms/search-zoomed-expected.txt:
  • platform/mac/fast/forms/searchfield-heights-expected.checksum:
  • platform/mac/fast/forms/searchfield-heights-expected.png:
  • platform/mac/fast/forms/searchfield-heights-expected.txt:
  • platform/mac/fast/forms/tabbing-input-iframe-expected.checksum:
  • platform/mac/fast/forms/tabbing-input-iframe-expected.png:
  • platform/mac/fast/forms/tabbing-input-iframe-expected.txt:
  • platform/mac/fast/forms/textAreaLineHeight-expected.checksum:
  • platform/mac/fast/forms/textAreaLineHeight-expected.png:
  • platform/mac/fast/forms/textAreaLineHeight-expected.txt:
  • platform/mac/fast/forms/textarea-align-expected.checksum:
  • platform/mac/fast/forms/textarea-align-expected.png:
  • platform/mac/fast/forms/textarea-align-expected.txt:
  • platform/mac/fast/forms/textarea-rows-cols-expected.checksum:
  • platform/mac/fast/forms/textarea-rows-cols-expected.png:
  • platform/mac/fast/forms/textarea-rows-cols-expected.txt:
  • platform/mac/fast/forms/textarea-scroll-height-expected.checksum:
  • platform/mac/fast/forms/textarea-scroll-height-expected.png:
  • platform/mac/fast/forms/textarea-scroll-height-expected.txt:
  • platform/mac/fast/forms/textarea-scrollbar-expected.checksum:
  • platform/mac/fast/forms/textarea-scrollbar-expected.png:
  • platform/mac/fast/forms/textarea-scrollbar-expected.txt:
  • platform/mac/fast/forms/textarea-scrolled-type-expected.checksum:
  • platform/mac/fast/forms/textarea-scrolled-type-expected.png:
  • platform/mac/fast/forms/textarea-scrolled-type-expected.txt:
  • platform/mac/fast/forms/textarea-setinnerhtml-expected.checksum:
  • platform/mac/fast/forms/textarea-setinnerhtml-expected.png:
  • platform/mac/fast/forms/textarea-setinnerhtml-expected.txt:
  • platform/mac/fast/forms/textarea-width-expected.checksum:
  • platform/mac/fast/forms/textarea-width-expected.png:
  • platform/mac/fast/forms/textarea-width-expected.txt:
  • platform/mac/fast/forms/textfield-drag-into-disabled-expected.checksum:
  • platform/mac/fast/forms/textfield-drag-into-disabled-expected.png:
  • platform/mac/fast/forms/textfield-drag-into-disabled-expected.txt:
  • platform/mac/fast/forms/textfield-outline-expected.checksum:
  • platform/mac/fast/forms/textfield-outline-expected.png:
  • platform/mac/fast/forms/textfield-outline-expected.txt:
  • platform/mac/fast/forms/textfield-overflow-expected.checksum:
  • platform/mac/fast/forms/textfield-overflow-expected.png:
  • platform/mac/fast/forms/textfield-overflow-expected.txt:
  • platform/mac/fast/forms/visual-hebrew-text-field-expected.txt:
  • platform/mac/fast/invalid/residual-style-expected.txt:
  • platform/mac/fast/lists/dynamic-marker-crash-expected.checksum:
  • platform/mac/fast/lists/dynamic-marker-crash-expected.png:
  • platform/mac/fast/lists/dynamic-marker-crash-expected.txt:
  • platform/mac/fast/overflow/overflow-x-y-expected.checksum:
  • platform/mac/fast/overflow/overflow-x-y-expected.png:
  • platform/mac/fast/overflow/overflow-x-y-expected.txt:
  • platform/mac/fast/parser/comment-in-textarea-expected.checksum:
  • platform/mac/fast/parser/comment-in-textarea-expected.png:
  • platform/mac/fast/parser/comment-in-textarea-expected.txt:
  • platform/mac/fast/parser/entity-comment-in-textarea-expected.checksum:
  • platform/mac/fast/parser/entity-comment-in-textarea-expected.png:
  • platform/mac/fast/parser/entity-comment-in-textarea-expected.txt:
  • platform/mac/fast/parser/open-comment-in-textarea-expected.checksum:
  • platform/mac/fast/parser/open-comment-in-textarea-expected.png:
  • platform/mac/fast/parser/open-comment-in-textarea-expected.txt:
  • platform/mac/fast/repaint/renderer-destruction-by-invalidateSelection-crash-expected.checksum:
  • platform/mac/fast/repaint/renderer-destruction-by-invalidateSelection-crash-expected.png:
  • platform/mac/fast/repaint/renderer-destruction-by-invalidateSelection-crash-expected.txt:
  • platform/mac/fast/repaint/search-field-cancel-expected.checksum:
  • platform/mac/fast/repaint/search-field-cancel-expected.png:
  • platform/mac/fast/repaint/search-field-cancel-expected.txt:
  • platform/mac/fast/repaint/subtree-root-skipped-expected.checksum:
  • platform/mac/fast/repaint/subtree-root-skipped-expected.png:
  • platform/mac/fast/repaint/subtree-root-skipped-expected.txt:
  • platform/mac/fast/replaced/replaced-breaking-expected.checksum:
  • platform/mac/fast/replaced/replaced-breaking-expected.png:
  • platform/mac/fast/replaced/replaced-breaking-expected.txt:
  • platform/mac/fast/replaced/replaced-breaking-mixture-expected.checksum:
  • platform/mac/fast/replaced/replaced-breaking-mixture-expected.png:
  • platform/mac/fast/replaced/replaced-breaking-mixture-expected.txt:
  • platform/mac/fast/replaced/width100percent-searchfield-expected.checksum:
  • platform/mac/fast/replaced/width100percent-searchfield-expected.png:
  • platform/mac/fast/replaced/width100percent-searchfield-expected.txt:
  • platform/mac/fast/replaced/width100percent-textarea-expected.checksum:
  • platform/mac/fast/replaced/width100percent-textarea-expected.png:
  • platform/mac/fast/replaced/width100percent-textarea-expected.txt:
  • platform/mac/fast/replaced/width100percent-textfield-expected.checksum:
  • platform/mac/fast/replaced/width100percent-textfield-expected.png:
  • platform/mac/fast/replaced/width100percent-textfield-expected.txt:
  • platform/mac/fast/table/003-expected.checksum:
  • platform/mac/fast/table/003-expected.png:
  • platform/mac/fast/table/003-expected.txt:
  • platform/mac/fast/table/colspanMinWidth-expected.checksum:
  • platform/mac/fast/table/colspanMinWidth-expected.png:
  • platform/mac/fast/table/colspanMinWidth-expected.txt:
  • platform/mac/fast/table/spanOverlapRepaint-expected.checksum:
  • platform/mac/fast/table/spanOverlapRepaint-expected.png:
  • platform/mac/fast/table/spanOverlapRepaint-expected.txt:
  • platform/mac/fast/table/text-field-baseline-expected.checksum:
  • platform/mac/fast/table/text-field-baseline-expected.png:
  • platform/mac/fast/table/text-field-baseline-expected.txt:
  • platform/mac/fast/text/international/rtl-white-space-pre-wrap-expected.checksum:
  • platform/mac/fast/text/international/rtl-white-space-pre-wrap-expected.png:
  • platform/mac/fast/text/international/rtl-white-space-pre-wrap-expected.txt:
  • platform/mac/fast/text/textIteratorNilRenderer-expected.checksum:
  • platform/mac/fast/text/textIteratorNilRenderer-expected.png:
  • platform/mac/fast/text/textIteratorNilRenderer-expected.txt:
  • platform/mac/fast/transforms/transformed-focused-text-input-expected.checksum:
  • platform/mac/fast/transforms/transformed-focused-text-input-expected.png:
  • platform/mac/fast/transforms/transformed-focused-text-input-expected.txt:
  • platform/mac/http/tests/navigation/anchor-basic-expected.checksum:
  • platform/mac/http/tests/navigation/anchor-basic-expected.png:
  • platform/mac/http/tests/navigation/anchor-basic-expected.txt:
  • platform/mac/http/tests/navigation/anchor-frames-expected.txt:
  • platform/mac/http/tests/navigation/anchor-goback-expected.checksum:
  • platform/mac/http/tests/navigation/anchor-goback-expected.png:
  • platform/mac/http/tests/navigation/anchor-goback-expected.txt:
  • platform/mac/http/tests/navigation/anchor-subframeload-expected.txt:
  • platform/mac/http/tests/navigation/javascriptlink-basic-expected.checksum:
  • platform/mac/http/tests/navigation/javascriptlink-basic-expected.png:
  • platform/mac/http/tests/navigation/javascriptlink-basic-expected.txt:
  • platform/mac/http/tests/navigation/javascriptlink-frames-expected.checksum:
  • platform/mac/http/tests/navigation/javascriptlink-frames-expected.png:
  • platform/mac/http/tests/navigation/javascriptlink-frames-expected.txt:
  • platform/mac/http/tests/navigation/javascriptlink-goback-expected.checksum:
  • platform/mac/http/tests/navigation/javascriptlink-goback-expected.png:
  • platform/mac/http/tests/navigation/javascriptlink-goback-expected.txt:
  • platform/mac/http/tests/navigation/javascriptlink-subframeload-expected.checksum:
  • platform/mac/http/tests/navigation/javascriptlink-subframeload-expected.png:
  • platform/mac/http/tests/navigation/javascriptlink-subframeload-expected.txt:
  • platform/mac/http/tests/navigation/metaredirect-basic-expected.checksum:
  • platform/mac/http/tests/navigation/metaredirect-basic-expected.png:
  • platform/mac/http/tests/navigation/metaredirect-basic-expected.txt:
  • platform/mac/http/tests/navigation/metaredirect-frames-expected.checksum:
  • platform/mac/http/tests/navigation/metaredirect-frames-expected.png:
  • platform/mac/http/tests/navigation/metaredirect-frames-expected.txt:
  • platform/mac/http/tests/navigation/metaredirect-goback-expected.checksum:
  • platform/mac/http/tests/navigation/metaredirect-goback-expected.png:
  • platform/mac/http/tests/navigation/metaredirect-goback-expected.txt:
  • platform/mac/http/tests/navigation/metaredirect-subframeload-expected.checksum:
  • platform/mac/http/tests/navigation/metaredirect-subframeload-expected.png:
  • platform/mac/http/tests/navigation/metaredirect-subframeload-expected.txt:
  • platform/mac/http/tests/navigation/post-goback2-expected.checksum:
  • platform/mac/http/tests/navigation/post-goback2-expected.png:
  • platform/mac/http/tests/navigation/post-goback2-expected.txt:
  • platform/mac/http/tests/navigation/postredirect-goback2-expected.checksum:
  • platform/mac/http/tests/navigation/postredirect-goback2-expected.png:
  • platform/mac/http/tests/navigation/postredirect-goback2-expected.txt:
  • platform/mac/http/tests/navigation/redirect302-basic-expected.checksum:
  • platform/mac/http/tests/navigation/redirect302-basic-expected.png:
  • platform/mac/http/tests/navigation/redirect302-basic-expected.txt:
  • platform/mac/http/tests/navigation/redirect302-frames-expected.checksum:
  • platform/mac/http/tests/navigation/redirect302-frames-expected.png:
  • platform/mac/http/tests/navigation/redirect302-frames-expected.txt:
  • platform/mac/http/tests/navigation/redirect302-goback-expected.checksum:
  • platform/mac/http/tests/navigation/redirect302-goback-expected.png:
  • platform/mac/http/tests/navigation/redirect302-goback-expected.txt:
  • platform/mac/http/tests/navigation/redirect302-subframeload-expected.checksum:
  • platform/mac/http/tests/navigation/redirect302-subframeload-expected.png:
  • platform/mac/http/tests/navigation/redirect302-subframeload-expected.txt:
  • platform/mac/http/tests/navigation/relativeanchor-basic-expected.checksum:
  • platform/mac/http/tests/navigation/relativeanchor-basic-expected.png:
  • platform/mac/http/tests/navigation/relativeanchor-basic-expected.txt:
  • platform/mac/http/tests/navigation/relativeanchor-frames-expected.txt:
  • platform/mac/http/tests/navigation/relativeanchor-goback-expected.checksum:
  • platform/mac/http/tests/navigation/relativeanchor-goback-expected.png:
  • platform/mac/http/tests/navigation/relativeanchor-goback-expected.txt:
  • platform/mac/http/tests/navigation/slowmetaredirect-basic-expected.checksum:
  • platform/mac/http/tests/navigation/slowmetaredirect-basic-expected.png:
  • platform/mac/http/tests/navigation/slowmetaredirect-basic-expected.txt:
  • platform/mac/http/tests/navigation/slowtimerredirect-basic-expected.checksum:
  • platform/mac/http/tests/navigation/slowtimerredirect-basic-expected.png:
  • platform/mac/http/tests/navigation/slowtimerredirect-basic-expected.txt:
  • platform/mac/http/tests/navigation/success200-basic-expected.checksum:
  • platform/mac/http/tests/navigation/success200-basic-expected.png:
  • platform/mac/http/tests/navigation/success200-basic-expected.txt:
  • platform/mac/http/tests/navigation/success200-frames-expected.checksum:
  • platform/mac/http/tests/navigation/success200-frames-expected.png:
  • platform/mac/http/tests/navigation/success200-frames-expected.txt:
  • platform/mac/http/tests/navigation/success200-frames-loadsame-expected.checksum:
  • platform/mac/http/tests/navigation/success200-frames-loadsame-expected.png:
  • platform/mac/http/tests/navigation/success200-frames-loadsame-expected.txt:
  • platform/mac/http/tests/navigation/success200-goback-expected.checksum:
  • platform/mac/http/tests/navigation/success200-goback-expected.png:
  • platform/mac/http/tests/navigation/success200-goback-expected.txt:
  • platform/mac/http/tests/navigation/success200-loadsame-expected.checksum:
  • platform/mac/http/tests/navigation/success200-loadsame-expected.png:
  • platform/mac/http/tests/navigation/success200-loadsame-expected.txt:
  • platform/mac/http/tests/navigation/success200-reload-expected.checksum:
  • platform/mac/http/tests/navigation/success200-reload-expected.png:
  • platform/mac/http/tests/navigation/success200-reload-expected.txt:
  • platform/mac/http/tests/navigation/success200-subframeload-expected.checksum:
  • platform/mac/http/tests/navigation/success200-subframeload-expected.png:
  • platform/mac/http/tests/navigation/success200-subframeload-expected.txt:
  • platform/mac/http/tests/navigation/timerredirect-basic-expected.checksum:
  • platform/mac/http/tests/navigation/timerredirect-basic-expected.png:
  • platform/mac/http/tests/navigation/timerredirect-basic-expected.txt:
  • platform/mac/http/tests/navigation/timerredirect-frames-expected.checksum:
  • platform/mac/http/tests/navigation/timerredirect-frames-expected.png:
  • platform/mac/http/tests/navigation/timerredirect-frames-expected.txt:
  • platform/mac/http/tests/navigation/timerredirect-goback-expected.checksum:
  • platform/mac/http/tests/navigation/timerredirect-goback-expected.png:
  • platform/mac/http/tests/navigation/timerredirect-goback-expected.txt:
  • platform/mac/http/tests/navigation/timerredirect-subframeload-expected.checksum:
  • platform/mac/http/tests/navigation/timerredirect-subframeload-expected.png:
  • platform/mac/http/tests/navigation/timerredirect-subframeload-expected.txt:
  • platform/mac/svg/custom/inline-svg-in-xhtml-expected.checksum:
  • platform/mac/svg/custom/inline-svg-in-xhtml-expected.png:
  • platform/mac/svg/custom/inline-svg-in-xhtml-expected.txt:
  • platform/mac/svg/hixie/mixed/003-expected.checksum:
  • platform/mac/svg/hixie/mixed/003-expected.png:
  • platform/mac/svg/hixie/mixed/003-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug1188-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug1188-expected.png:
  • platform/mac/tables/mozilla/bugs/bug1188-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug12384-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug12384-expected.png:
  • platform/mac/tables/mozilla/bugs/bug12384-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug18359-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug18359-expected.png:
  • platform/mac/tables/mozilla/bugs/bug18359-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug194024-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug194024-expected.png:
  • platform/mac/tables/mozilla/bugs/bug194024-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug24200-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug24200-expected.png:
  • platform/mac/tables/mozilla/bugs/bug24200-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug2479-2-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug2479-2-expected.png:
  • platform/mac/tables/mozilla/bugs/bug2479-2-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug2479-3-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug2479-3-expected.png:
  • platform/mac/tables/mozilla/bugs/bug2479-3-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug2479-4-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug28928-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug28928-expected.png:
  • platform/mac/tables/mozilla/bugs/bug28928-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug30559-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug30559-expected.png:
  • platform/mac/tables/mozilla/bugs/bug30559-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug30692-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug30692-expected.png:
  • platform/mac/tables/mozilla/bugs/bug30692-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug4382-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug4382-expected.png:
  • platform/mac/tables/mozilla/bugs/bug4382-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug4527-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug4527-expected.png:
  • platform/mac/tables/mozilla/bugs/bug4527-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug46368-1-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug46368-1-expected.png:
  • platform/mac/tables/mozilla/bugs/bug46368-1-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug46368-2-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug46368-2-expected.png:
  • platform/mac/tables/mozilla/bugs/bug46368-2-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug51037-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug51037-expected.png:
  • platform/mac/tables/mozilla/bugs/bug51037-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug55545-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug55545-expected.png:
  • platform/mac/tables/mozilla/bugs/bug55545-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug59354-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug59354-expected.png:
  • platform/mac/tables/mozilla/bugs/bug59354-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug7342-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug7342-expected.png:
  • platform/mac/tables/mozilla/bugs/bug7342-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug96334-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug96334-expected.png:
  • platform/mac/tables/mozilla/bugs/bug96334-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug99948-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug99948-expected.png:
  • platform/mac/tables/mozilla/bugs/bug99948-expected.txt:
  • platform/mac/tables/mozilla/dom/tableDom-expected.checksum:
  • platform/mac/tables/mozilla/dom/tableDom-expected.png:
  • platform/mac/tables/mozilla/dom/tableDom-expected.txt:
  • platform/mac/tables/mozilla/other/move_row-expected.checksum:
  • platform/mac/tables/mozilla/other/move_row-expected.png:
  • platform/mac/tables/mozilla/other/move_row-expected.txt:
  • platform/mac/tables/mozilla_expected_failures/bugs/bug2479-5-expected.txt:
  • platform/mac/tables/mozilla_expected_failures/bugs/bug92647-1-expected.checksum:
  • platform/mac/tables/mozilla_expected_failures/bugs/bug92647-1-expected.png:
  • platform/mac/tables/mozilla_expected_failures/bugs/bug92647-1-expected.txt:
  • platform/win/Skipped:
  • svg/custom/resources/svg-fonts-in-text-controls.js:
  • svg/custom/svg-fonts-in-text-controls-expected.txt:
  • svg/custom/svg-fonts-in-text-controls.html: Removed.
21:26 Changeset [42697] by dglazkov@chromium.org

2009-04-20 Dimitri Glazkov <dglazkov@chromium.org>

Not reviewed, typo fix.

Rename erroneous lines in Skipped to actual test URLs.

  • platform/win/Skipped:
20:50 Changeset [42696] by aroben@apple.com

Build fix

  • MemoryStream.cpp: (MemoryStream::Clone): Fix typo introduced when implementing suggestions from patch review.
20:48 Changeset [42695] by ggaren@apple.com

2009-04-20 Geoffrey Garen <ggaren@apple.com>

Reviewed by Adam Roben.

One last loose end, discovered using COLLECT_ON_EVERY_ALLOCATION, for
https://bugs.webkit.org/show_bug.cgi?id=21260
Unbounded memory growth when churning elements with anonymous event handler functions


  • bindings/js/JSSVGElementInstanceCustom.cpp: (WebCore::JSSVGElementInstance::mark): Don't ASSERT that an ElementInstance's correspondingElement has a JS wrapper. If a GC falls exactly between the allocation of the ElementInstance wrapper and the correspondingElement wrapper, the correspondingElement won't have a wrapper at the time we mark the ElementInstance's wrapper.
20:45 Changeset [42694] by dglazkov@chromium.org

2009-04-20 Ojan Vafai <ojan@chromium.org>

Reviewed by Adam Roben.

Add Windows failures temporarily to the Skipped list after r42678.
See https://bugs.webkit.org/show_bug.cgi?id=25306 for more
information. This is just to get the tree green while the tests
are fixed.

  • platform/win/Skipped:
20:42 Changeset [42693] by aroben@apple.com

First part of <rdar://6395825> It takes over 20 sec to launch Safari with 500KB history file

This ports the changes that were made to WebKit/mac's WebHistory
implementation in r25275 to WebKit/win. WebHistory now stores a
HashMap from CFAbsoluteTime (stored as int64_t) to CFArray of
IWebHistoryItem*. The HashMap lets us look up the CFArray for a
particular day's history in constant time rather than linear time. The
precise reasons why we store the CFAbsoluteTime as an int64_t are lost
to the mists of time, but it is likely because these CFAbsoluteTimes
never have a decimal part, and integer comparisons are faster than
floating-point comparisons, so storing them as int64_t should improve
performance without losing precision. We also now use a binary search
instead of a linear search when determining the index at which to
insert a WebHistoryItem into its day's CFArray.

This patch reduces the time needed to load a 100,000-item
History.plist from ~15 seconds to ~7.5 seconds. Further improvements
could likely be made by reducing the number of string conversions,
reducing calls to CFTimeZone functions, and removing all the
WebHistoryItemsAdded notifications that are sent (Mac doesn't send any
while loading history).

Reviewed by Darin Adler.

  • WebHistory.cpp: (WebHistoryWriter::WebHistoryWriter): Changed to take a DateToEntriesMap instead of a CFArrayRef. Initialize m_dateKeys to contain the keys from m_entriesByDate in ascending order. (WebHistoryWriter::writeHistoryItems): Now loops through m_dateKeys in reverse order and gets the entries array from the DateToEntriesMap. (WebHistory::WebHistory): Removed code to initialize m_datesWithEntries (which has been removed) and m_entriesByDate (which has been converted to a HashMap). (WebHistory::removeAllItems): Clear out m_entriesByDate and m_orderedLastVisitedDays. (WebHistory::orderedLastVisitedDays): If we don't already have a cached m_orderedLastVisitedDays array, create one by converting the keys from m_entriesByDate to DATEs and sorting them in descending order. Then copy m_orderedLastVisitedDays to the output buffer. (WebHistory::orderedItemsLastVisitedOnDay): Updated to use findKey instead of findIndex and to treat m_entriesByDate as a HashMap. Now that the items in each day's CFArray are stored in descending order (newest to oldest), we don't have to reverse them when filling in the output buffer. (The old comment about putting the items in oldest-to-newest order was incorrect -- it was putting them in newest-to-oldest order.)

(WebHistory::addItemToDateCaches):
(WebHistory::removeItemFromDateCaches):
Changed to use findKey instead of findIndex and to treat
m_entriesByDate as a HashMap. If we're adding a key to or removing a
key from m_entriesByDate, also clear m_orderedLastVisitedDays so that
we will regenerate it for the current set of keys the next time it is
needed.

(timeIntervalForBeginningOfDay): Added. Returns the CFAbsoluteTime
that corresponds to the beginning of the day in which the passed-in
DATE occurs.
(WebHistory::findKey): Returns the key used in m_entriesByDate to
store the CFArray that should contain the IWebHistoryItem that
corresponds to the passed-in DATE. Replaces findIndex.
(WebHistory::insertItem): Changed to treat m_entriesByDate as a
HashMap rather than a CFArray. We now optimize for inserting at the
beginning and end of the array, and use a binary rather than linear
search when inserting into the middle of the array.
(WebHistory::data): Changed to treat m_entriesByDate as a HashMap.

  • WebHistory.h: Changed m_entriesByDate to a DateToEntriesMap, removed m_datesWithEntries, added m_orderedLastVisitedDays.
20:42 Changeset [42692] by aroben@apple.com

Change MemoryStream::createInstance to return a COMPtr

Part of Bug 25294: All WebKit/win classes should return COMPtrs from
their static constructor members
<https://bugs.webkit.org/show_bug.cgi?id=25294>

Reviewed by Darin Adler.

  • MemoryStream.cpp: (MemoryStream::createInstance): Changed to return a COMPtr. (MemoryStream::Clone): Updated for createInstance change.
  • MemoryStream.h: Changed createInstance to return a COMPtr.
  • WebArchive.cpp: (WebArchive::data):
  • WebCoreSupport/EmbeddedWidget.cpp: (EmbeddedWidget::didReceiveData):
  • WebDataSource.cpp: (WebDataSource::data):
  • WebHistory.cpp: (WebHistory::data):
  • WebIconFetcher.cpp: (WebIconFetcherClient::finishedFetchingIcon):
  • WebResource.cpp: (WebResource::createInstance): Updated for changes to MemoryStream::createInstance.
20:42 Changeset [42691] by aroben@apple.com

Fix Bug 25295: Remove WebKit/win code that was used for Drosera support

<https://bugs.webkit.org/show_bug.cgi?id=25295>

Reviewed by Sam Weinig.

  • ForEachCoClass.h: Removed WebScriptDebugServer.
  • Interfaces/WebKit.idl: Removed WebScriptDebugServer and references to removed IDL files.
  • WebKit.vcproj/Interfaces.vcproj:
  • WebKit.vcproj/WebKit.vcproj: Removed IDL and source files for IWebScriptCallFrame, IWebScriptDebugListener, IWebScriptDebugServer, WebScriptCallFrame, and WebScriptDebugServer.
  • WebCoreSupport/WebFrameLoaderClient.cpp:
  • WebKitClassFactory.cpp:
  • WebKitDLL.cpp: Removed #includes of WebScriptDebugServer.h.
  • Interfaces/IWebScriptCallFrame.idl: Removed.
  • Interfaces/IWebScriptDebugListener.idl: Removed.
  • Interfaces/IWebScriptDebugServer.idl: Removed.
  • WebScriptCallFrame.cpp: Removed.
  • WebScriptCallFrame.h: Removed.
  • WebScriptDebugServer.cpp: Removed.
  • WebScriptDebugServer.h: Removed.
17:57 Changeset [42690] by levin@chromium.org

2009-04-20 Ojan Vafai <ojan@chromium.org>

Reviewed by Eric Seidel

Fix svg-fonts-in-text-controls.js to measure the
widths after the SVG font has loaded.

For now the expected results have the success line
printed before the metrics being measured. I'll fix
this in a following commit. See
https://bugs.webkit.org/show_bug.cgi?id=25302
for more details.

  • svg/custom/resources/svg-fonts-in-text-controls.js:
  • svg/custom/svg-fonts-in-text-controls-expected.txt:
16:36 Changeset [42689] by sfalken@apple.com

More Windows build fixes.

15:32 Changeset [42688] by beidson@apple.com

2009-04-20 Brady Eidson <beidson@apple.com>

Rubberstamped by Mark Rowe

Clean builds are broken on case-sensitive filesystems. Let's fix, shall we?

  • platform/graphics/SimpleFontData.cpp: Remove "String.h" include. Ojan probably intended to include "PlatformString.h" which was unnecessary because it is pulled in through other headers. This wasn't a build failure on case-insensitive file systems because those systems would find the system <string.h>, averting the warning.
15:32 Changeset [42687] by ggaren@apple.com

2009-04-20 Geoffrey Garen <ggaren@apple.com>

Reviewed by Darin Adler.

Last patch for https://bugs.webkit.org/show_bug.cgi?id=21260
Unbounded memory growth when churning elements with anonymous event handler functions


Converted "lazy" event listeners to be unprotected, just like all the others.

  • bindings/js/JSEventListener.cpp: (WebCore::JSEventListener::JSEventListener): (WebCore::JSEventListener::~JSEventListener): (WebCore::JSEventListener::jsFunction): (WebCore::JSEventListener::markJSFunction): (WebCore::JSEventListener::handleEvent): (WebCore::JSEventListener::virtualIsInline):
  • bindings/js/JSEventListener.h: (WebCore::JSEventListener::isInline): Merged JSAbstractEventListener into JSEventListener. Now that the only difference between JSEventListener and JSLazyEventListener is that JSLazyEventListener compiles lazily, there's no need for an abstract base class.
  • bindings/js/JSLazyEventListener.cpp: Converted JSLazyEventListener to inherit from JSEventListener and match its un-GC-protected behavior. (WebCore::JSLazyEventListener::JSLazyEventListener): ditto (WebCore::JSLazyEventListener::parseCode): ditto (WebCore::createInlineEventListener): When creating a lazy event listener, ensure that the related node has a JS wrapper to mark the listener. Since the parser makes these listeners, it's possible that no JS reference has been made to the node yet.
  • bindings/js/JSLazyEventListener.h: ditto
  • dom/EventListener.h: (WebCore::EventListener::clearJSFunction): Removed an usused function.
15:30 Changeset [42686] by justin.garcia@apple.com

Fixed typo.

15:28 Changeset [42685] by justin.garcia@apple.com

2009-04-20 Justin Garcia <justin.garcia@apple.com>

Reviewed by Dan Bernstein.

https://bugs.webkit.org/show_bug.cgi?id=25299
Don't bound caret to root line box if it is still inside its containing block


Can't yet write tests for behavior of non-standard caret widths.

  • rendering/RenderText.cpp: (WebCore::RenderText::localCaretRect):
14:47 Changeset [42684] by sfalken@apple.com

Build fix

14:47 Changeset [42683] by sfalken@apple.com

Build fix

14:46 Changeset [42682] by sfalken@apple.com

Build fix

14:36 Changeset [42681] by darin@apple.com

WebKitTools:

2009-04-20 Darin Adler <darin@apple.com>

Reviewed by Sam Weinig.

  • DumpRenderTree/DumpRenderTree.xcodeproj/project.pbxproj: Added PlainTextController.h and PlainTextController.mm.
  • DumpRenderTree/mac/FrameLoadDelegate.mm: (-[FrameLoadDelegate webView:didClearWindowObject:forFrame:]): Added PlainTextController as "window.plainText".
  • DumpRenderTree/mac/PlainTextController.h: Added.
  • DumpRenderTree/mac/PlainTextController.mm: Added.

LayoutTests:

2009-04-20 Darin Adler <darin@apple.com>

Reviewed by Sam Weinig.

  • fast/dom/Window/window-properties.html: Added a new DumpRenderTree window object property, plainText.
14:25 Changeset [42680] by sfalken@apple.com

JavaScriptCore:

2009-04-20 Steve Falkenburg <sfalken@apple.com>

Separate JavaScriptCore.dll from WebKit.dll.
Slight performance improvement or no change on benchmarks.


Allows us to break a circular dependency between CFNetwork and WebKit on Windows,
and simplifies standalone JavaScriptCore builds.

Reviewed by Oliver Hunt.

  • API/JSBase.h: Export symbols with JS_EXPORT when using MSVC.
  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:
  • JavaScriptCore.vcproj/WTF/WTF.vcproj: Build JavaScriptCore as a DLL instead of a static library.
  • config.h: Specify declspec(dllexport/dllimport) appropriately when exporting data.
  • runtime/InternalFunction.h: Specify JS_EXPORTDATA on exported data.
  • runtime/JSArray.h: Specify JS_EXPORTDATA on exported data.
  • runtime/JSFunction.h: Specify JS_EXPORTDATA on exported data.
  • runtime/StringObject.h: Specify JS_EXPORTDATA on exported data.
  • runtime/UString.h: Specify JS_EXPORTDATA on exported data.

WebCore:

2009-04-20 Steve Falkenburg <sfalken@apple.com>

Separate JavaScriptCore.dll from WebKit.dll.
Slight performance improvement or no change on benchmarks.


Allows us to break a circular dependency between CFNetwork and WebKit on Windows,
and simplifies standalone JavaScriptCore builds.

Reviewed by Oliver Hunt.

  • config.h: Specify declspec(dllexport/dllimport) appropriately when exporting data.

WebKit/cf:

2009-04-20 Steve Falkenburg <sfalken@apple.com>

Separate JavaScriptCore.dll from WebKit.dll.
Slight performance improvement or no change on benchmarks.


Allows us to break a circular dependency between CFNetwork and WebKit on Windows,
and simplifies standalone JavaScriptCore builds.


Reviewed by Oliver Hunt.

  • WebCoreSupport/WebInspectorClientCF.cpp: Workaround for lack of config.h in shared WebKit file (WebKit/win uses config.h from WebCore).

WebKit/win:

2009-04-20 Steve Falkenburg <sfalken@apple.com>

Separate JavaScriptCore.dll from WebKit.dll.
Slight performance improvement or no change on benchmarks.


Allows us to break a circular dependency between CFNetwork and WebKit on Windows,
and simplifies standalone JavaScriptCore builds.

Reviewed by Oliver Hunt.

  • WebKit.vcproj/WebKit.def: Mark JavaScriptCore re-exports as deprecated.
  • WebKit.vcproj/WebKit.vcproj: Link against JavaScriptCore.dll
  • WebKit.vcproj/WebKit_debug.def: Mark JavaScriptCore re-exports as deprecated.

WebKitLibraries:

2009-04-20 Steve Falkenburg <sfalken@apple.com>

Separate JavaScriptCore.dll from WebKit.dll.
Slight performance improvement or no change on benchmarks.


Allows us to break a circular dependency between CFNetwork and WebKit on Windows,
and simplifies standalone JavaScriptCore builds.

Reviewed by Oliver Hunt.

  • win/tools/vsprops/common.vsprops: Add BUILDING_{project} preprocessor define.

WebKitTools:

2009-04-20 Steve Falkenburg <sfalken@apple.com>

Separate JavaScriptCore.dll from WebKit.dll.
Slight performance improvement or no change on benchmarks.


Allows us to break a circular dependency between CFNetwork and WebKit on Windows,
and simplifies standalone JavaScriptCore builds.

Reviewed by Oliver Hunt.

  • DumpRenderTree/config.h: Specify JS, WebKit exported data as declspec(dllimport)
  • DumpRenderTree/win/DumpRenderTree.vcproj: Link against JavaScriptCore.dll
13:41 Changeset [42679] by andersca@apple.com

WebCore:

2009-04-20 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

<rdar://problem/6722845> In the Cocoa event model, NPWindow's window field should be null

  • bridge/npapi.h: (_NPCocoaEvent::): Add a CGContextRef member to the draw struct. Add an NPCoordinateSpace enum.


  • plugins/npfunctions.h: Add a prototype for NPN_ConvertPoint.

WebKit/mac:

2009-04-17 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

<rdar://problem/6722845>
In the Cocoa event model, NPWindow's window field should be null


  • Plugins/Hosted/NetscapePluginHostProxy.mm: (WKPCConvertPoint): Get the instance proxy and call it's convertPoint function.


  • Plugins/Hosted/NetscapePluginInstanceProxy.h:
  • Plugins/Hosted/NetscapePluginInstanceProxy.mm: (WebKit::NetscapePluginInstanceProxy::convertPoint): Call convertPoint on the plug-in view.


  • Plugins/Hosted/WebKitPluginClient.defs: Add PCConvertPoint.


  • Plugins/WebBaseNetscapePluginView.h: Add a declaration for convertFromX:andY:space:toX:andY:space:.


  • Plugins/WebBaseNetscapePluginView.mm: (-[WebBaseNetscapePluginView convertFromX:andY:space:toX:andY:space:]): Convert a point from one coordinate system to another.


  • Plugins/WebNetscapePluginEventHandler.h:
  • Plugins/WebNetscapePluginEventHandlerCarbon.h:
  • Plugins/WebNetscapePluginEventHandlerCarbon.mm: (WebNetscapePluginEventHandlerCarbon::drawRect):
  • Plugins/WebNetscapePluginEventHandlerCocoa.h: Add CGContextRef to drawRect.
  • Plugins/WebNetscapePluginEventHandlerCocoa.mm: (WebNetscapePluginEventHandlerCocoa::drawRect): Set the passed in context.


  • Plugins/WebNetscapePluginPackage.mm: (-[WebNetscapePluginPackage _tryLoad]): Add NPN_ConvertPoint to the browserFuncs vtable.


  • Plugins/WebNetscapePluginView.mm: (-[WebNetscapePluginView saveAndSetNewPortStateForUpdate:]): Only set the window for the Carbon event model.


(-[WebNetscapePluginView restorePortState:]):
It's OK for the window context to be null.


(-[WebNetscapePluginView sendDrawRectEvent:]):
Pass the CGContextRef to drawRect.

  • Plugins/npapi.mm: (NPN_ConvertPoint): Call the plug-in view method.
12:52 QtWebKitTodo edited by kenneth.christiansen@gmail.com
(diff)
12:49 Changeset [42678] by eric@webkit.org

Reviewed by Dan Bernstein.

Make textarea and text input metrics more closely match IEs.
https://bugs.webkit.org/show_bug.cgi?id=15312

This involves:
-set text input width to size*avgCharWidth + maxCharWidth - avgCharWidth
-set textarea width to cols*avgCharWidth
-Make default CSS match IEs
-Correctly initializing m_avgCharWidth and m_maxCharWidth for each platform and SVG.

Those values for textarea and inputs were derived by doing a ton of manual
testing of IE's width values for various textareas and fonts.

To get the average and max character width of a font, we do the following
for each platform:
-Win: TextMetrics expose avgCharWidth and maxCharWidth
-SVG: avgCharWidth = width of an '0', fallback on width of a space glyph, then m_xHeight

maxCharWidth = width of a 'W' for roman fonts, fallback on m_ascent

-Linux: avgCharWidth = width of an '0', fallback on m_xHeight

maxCharWidth = max of avgCharWidth and m_ascent

-Mac: look in the OS/2 table for avgCharWidth and grab the maxCharWidth off the font.

If either one is not there, then calculate the value using the Linux approach.

Linux ports could probably dig into the OS/2 table as well, but I'll leave
that up to them to implement.

Tests: fast/forms/text-control-intrinsic-widths.html

fast/forms/textarea-metrics.html
svg/custom/svg-fonts-in-text-controls.html

  • css/html4.css:
  • css/themeWin.css:
  • platform/graphics/SimpleFontData.cpp: (WebCore::SimpleFontData::SimpleFontData): (WebCore::SimpleFontData::initCharWidths):
  • platform/graphics/SimpleFontData.h: (WebCore::SimpleFontData::maxCharWidth): (WebCore::SimpleFontData::avgCharWidth):
  • platform/graphics/chromium/SimpleFontDataChromiumWin.cpp: (WebCore::SimpleFontData::platformCharWidthInit):
  • platform/graphics/chromium/SimpleFontDataLinux.cpp: (WebCore::SimpleFontData::platformCharWidthInit):
  • platform/graphics/gtk/SimpleFontDataGtk.cpp: (WebCore::SimpleFontData::platformCharWidthInit):
  • platform/graphics/gtk/SimpleFontDataPango.cpp: (WebCore::SimpleFontData::platformCharWidthInit):
  • platform/graphics/mac/SimpleFontDataMac.mm: (WebCore::SimpleFontData::platformCharWidthInit):
  • platform/graphics/qt/SimpleFontDataQt.cpp: (WebCore::SimpleFontData::platformCharWidthInit):
  • platform/graphics/win/SimpleFontDataCGWin.cpp: (WebCore::SimpleFontData::platformCharWidthInit):
  • platform/graphics/win/SimpleFontDataCairoWin.cpp: (WebCore::SimpleFontData::platformInit): (WebCore::SimpleFontData::platformCharWidthInit):
  • platform/graphics/win/SimpleFontDataWin.cpp: (WebCore::SimpleFontData::initGDIFont):
  • platform/graphics/wx/SimpleFontDataWx.cpp: (WebCore::SimpleFontData::platformCharWidthInit):
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::calcPrefWidths):
  • rendering/RenderTextControlMultiLine.cpp: (WebCore::RenderTextControlMultiLine::createInnerTextStyle):
  • rendering/RenderTextControlSingleLine.cpp: (WebCore::RenderTextControlSingleLine::preferredContentWidth):
12:13 QtWebKitContrib edited by kenneth.christiansen@gmail.com
(diff)
12:10 Changeset [42677] by eric@webkit.org

Reviewed by Simon Fraser.

Remove m_absoluteBounds hack from SVG renderers and move outlineBoundsForRepaint into RenderSVGModelObject
https://bugs.webkit.org/show_bug.cgi?id=25276

This also exposed a buggy paint check in RenderSVGContainer::layout()
we should repaint if we are self painting OR if our kids changed, not AND.

Writing real outlineBoundsForRepaint required writing a mapLocalToContainer() function

No functional changes, thus no tests.

  • rendering/RenderPath.cpp: (WebCore::RenderPath::localToParentTransform): (WebCore::RenderPath::layout):
  • rendering/RenderPath.h:
  • rendering/RenderSVGContainer.cpp: (WebCore::RenderSVGContainer::layout):
  • rendering/RenderSVGContainer.h:
  • rendering/RenderSVGModelObject.cpp: (WebCore::RenderSVGModelObject::mapLocalToContainer): (WebCore::RenderSVGModelObject::outlineBoundsForRepaint):
  • rendering/RenderSVGModelObject.h:
  • rendering/RenderSVGRoot.cpp: (WebCore::RenderSVGRoot::layout): (WebCore::RenderSVGRoot::paint): (WebCore::RenderSVGRoot::localToParentTransformWithoutCSSParentOffset): (WebCore::RenderSVGRoot::localToParentTransform): (WebCore::RenderSVGRoot::computeRectForRepaint):
  • rendering/RenderSVGRoot.h:
  • rendering/RenderSVGViewportContainer.cpp: (WebCore::RenderSVGViewportContainer::layout):
11:12 Changeset [42676] by weinig@apple.com

2009-04-20 Sam Weinig <sam@webkit.org>

Reviewed by Kevin McCullough.

Always tag mmaped memory on darwin and clean up #defines
now that they are a little bigger.

  • interpreter/RegisterFile.h: (JSC::RegisterFile::RegisterFile):
  • jit/ExecutableAllocatorFixedVMPool.cpp: (JSC::FixedVMPoolAllocator::FixedVMPoolAllocator):
  • jit/ExecutableAllocatorPosix.cpp: (JSC::ExecutablePool::systemAlloc):
  • runtime/Collector.cpp: (JSC::allocateBlock):
11:11 Changeset [42675] by simon.fraser@apple.com

2009-04-20 Simon Fraser <simon.fraser@apple.com>

Reviewed by Eric Seidel

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

outlineBoundsForRepaint() should compute a quad relative to the
repaintContainer. This fixes the repaint issues originally fixed in
https://bugs.webkit.org/show_bug.cgi?id=12885 for elements in compositing layers.
Failure was only apparent when building with ACCELERATED_COMPOSITING enabled.

Tests: compositing/repaint/content-into-overflow.html

compositing/repaint/overflow-into-content.html

  • rendering/RenderBox.cpp: (WebCore::RenderBox::outlineBoundsForRepaint):
11:06 Changeset [42674] by weinig@apple.com

JavaScriptCore:

2009-04-20 Sam Weinig <sam@webkit.org>

Rubber-stamped by Tim Hatcher.

Add licenses for xcconfig files.

  • Configurations/Base.xcconfig:
  • Configurations/DebugRelease.xcconfig:
  • Configurations/FeatureDefines.xcconfig:
  • Configurations/JavaScriptCore.xcconfig:
  • Configurations/Version.xcconfig:

JavaScriptGlue:

2009-04-20 Sam Weinig <sam@webkit.org>

Rubber-stamped by Tim Hatcher.

Add licenses for xcconfig files.

  • Configurations/Base.xcconfig:
  • Configurations/DebugRelease.xcconfig:
  • Configurations/JavaScriptGlue.xcconfig:
  • Configurations/Version.xcconfig:

WebCore:

2009-04-20 Sam Weinig <sam@webkit.org>

Rubber-stamped by Tim Hatcher.

Add licenses for xcconfig files.

  • Configurations/Base.xcconfig:
  • Configurations/DebugRelease.xcconfig:
  • Configurations/FeatureDefines.xcconfig:
  • Configurations/Version.xcconfig:
  • Configurations/WebCore.xcconfig:

WebKit/mac:

2009-04-20 Sam Weinig <sam@webkit.org>

Rubber-stamped by Tim Hatcher.

Add licenses for xcconfig files.


  • Configurations/Base.xcconfig:
  • Configurations/DebugRelease.xcconfig:
  • Configurations/FeatureDefines.xcconfig:
  • Configurations/Version.xcconfig:
  • Configurations/WebKit.xcconfig:

WebKitTools:

2009-04-20 Sam Weinig <sam@webkit.org>

Rubber-stamped by Tim Hatcher.

Add licenses for xcconfig files.

  • DumpRenderTree/mac/Configurations/Base.xcconfig:
  • DumpRenderTree/mac/Configurations/DebugRelease.xcconfig:
  • DumpRenderTree/mac/Configurations/DumpRenderTree.xcconfig:
  • DumpRenderTree/mac/Configurations/ImageDiff.xcconfig:
  • DumpRenderTree/mac/Configurations/TestNetscapePlugIn.xcconfig:
10:31 Changeset [42673] by andersca@apple.com

2009-04-20 Anders Carlsson <andersca@apple.com>

Reviewed by Dan Bernstein.

WebKit side of <rdar://problem/6781302>


  • Plugins/Hosted/NetscapePluginInstanceProxy.h:
  • Plugins/Hosted/NetscapePluginInstanceProxy.mm: (WebKit::NetscapePluginInstanceProxy::keyEvent): Pass the event keyChar.


(WebKit::NetscapePluginInstanceProxy::syntheticKeyDownWithCommandModifier):
Ditto.


(WebKit::NetscapePluginInstanceProxy::flagsChanged):
Pass a 0 keyChar.


  • Plugins/Hosted/WebHostedNetscapePluginView.mm: (-[WebHostedNetscapePluginView flagsChanged:]): Call NetscapePluginInstanceProxy::flagsChanged.


  • Plugins/Hosted/WebKitPluginHost.defs: Add a keyChar argument.
10:16 Changeset [42672] by kov@webkit.org

2009-04-20 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

https://bugs.webkit.org/show_bug.cgi?id=24786
WebKitDownload sometimes suggests peculiar filenames

Remove query and ref from the URI that is to be downloaded when
using it to provide a file name suggestion.

  • webkit/webkitdownload.cpp: (_WebKitDownloadPrivate::webkit_download_set_property):
08:27 Changeset [42671] by dglazkov@chromium.org

2009-04-20 Eric Roman <eroman@chromium.org>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=25261
Implement the V8 binding for DOMWindow.event similarly to JSC, by using
the custom getter boilerplate from the IDL file.
Also, stub out DOMWindow.crypto which is defined by the idl.

  • bindings/v8/V8AbstractEventListener.cpp: (WebCore::V8AbstractEventListener::invokeEventHandler):
  • bindings/v8/custom/V8DOMWindowCustom.cpp: (WebCore::ACCESSOR_GETTER):
08:23 Changeset [42670] by kov@webkit.org

WebCore

2009-04-20 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

https://bugs.webkit.org/show_bug.cgi?id=25243
Crash when data:// loads are cancelled

Properly handle cancellation of the load for data:// loads. This
fixes crashing in the followin test:

plugins/return-error-from-new-stream-callback-in-full-frame-plugin.html

  • platform/network/soup/ResourceHandleSoup.cpp: (WebCore::parseDataUrl):

WebKit/gtk

2009-04-20 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

Handle the case when the plugin view is destroyed during
didReceiveResponse, to avoid crashing.

  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::committedLoad):
07:30 QtWebKitTodo edited by kenneth.christiansen@gmail.com
(diff)
07:11 QtWebKitTodo edited by ariya@webkit.org
(diff)
06:24 QtWebKitJournal edited by hausmann@webkit.org
(diff)
06:17 QtWebKitTodo edited by hausmann@webkit.org
(diff)
02:27 Changeset [42669] by ariya@webkit.org

2009-04-20 Ariya Hidayat <ariya.hidayat@nokia.com>

Build fix for Qt port (after r42646). Not reviewed.

  • wtf/unicode/qt4/UnicodeQt4.h: Added U16_PREV.

04/19/09:

23:25 Changeset [42668] by adele@apple.com

2009-04-19 Adele Peterson <adele@apple.com>

Reviewed by Darin Adler.

Fix for <rdar://problem/6804809> REGRESSION: In Mail, Home and End do not scroll message

If no scrolling occurs, call tryToPerform on the next responder. Then our WebResponderChainSink
will correctly detect if no responders handle the selector.

  • WebView/WebFrameView.mm: (-[WebFrameView _scrollToBeginningOfDocument]): (-[WebFrameView _scrollToEndOfDocument]): (-[WebFrameView scrollToBeginningOfDocument:]): (-[WebFrameView scrollToEndOfDocument:]): (-[WebFrameView scrollLineUp:]): (-[WebFrameView scrollLineDown:]):
22:47 Changeset [42667] by mitz@apple.com
  • maybe fix release builds
  • dom/ScriptElement.cpp: (WebCore::ScriptElementData::notifyFinished):
22:27 Changeset [42666] by mitz@apple.com
  • fix release builds
  • dom/Document.cpp: (WebCore::Document::executeScriptSoonTimerFired):
21:20 Changeset [42665] by mitz@apple.com

WebCore:

Reviewed by Mark Rowe.

  • fix <rdar://problem/6734365> REGRESSION eBay "Save this search" link does nothing the second time search results are opened

Test: fast/dom/HTMLScriptElement/nested-execution.html

Instead of executing scripts as soon as they finish loading, which may
happen synchronously during execution of another script, defer their
execution until the next run loop iteration, using a queue that is
similar to the HTML5 notion of the "list of scripts that will execute as
soon as possible", and in my testing was consistent with how Firefox
behaved.

  • dom/Document.cpp: (WebCore::Document::Document): Initialize m_executeScriptSoonTimer. (WebCore::Document::~Document): deref() script elements in m_scriptsToExecuteSoon. (WebCore::Document::executeScriptSoonTimerFired): Added. Executes the scripts in m_scriptsToExecuteSoon and deref()s them. (WebCore::Document::executeScriptSoon): Added. Appends to m_scriptsToExecuteSoon and ref()s the script element, which keeps the ScriptElementData alive as well.
  • dom/Document.h:
  • dom/ScriptElement.cpp: (WebCore::ScriptElementData::ScriptElementData): Initialize m_requested. (WebCore::ScriptElementData::requestScript): Set m_requested to true, to prevent further load requests. (WebCore::ScriptElementData::execute): Added. Moved the code from notifyFinished() which should not execute synchronously here, to be called by the Document on a 0-interval timer. (WebCore::ScriptElementData::notifyFinished): Moved the code to dispatch events and evaluate the script, which should not execute synchronously, out of here. (WebCore::ScriptElementData::ignoresLoadRequest): Changed to test for m_requested instead of m_cachedScript, because the latter is cleared before the script is evaluated.
  • dom/ScriptElement.h:

LayoutTests:

Reviewed by Mark Rowe.

  • test and updated tests for <rdar://problem/6734365> eBay "Save this search" link does nothing the second time search results are opened
  • fast/dom/HTMLScriptElement/nested-execution-expected.txt: Added.
  • fast/dom/HTMLScriptElement/nested-execution.html: Added.
  • fast/dom/HTMLScriptElement/resources/nested-execution.js: Added.
  • fast/dom/HTMLScriptElement/script-reexecution.html:
  • fast/dom/HTMLScriptElement/script-set-src.html:
  • fast/dom/script-element-gc.html:
  • fast/loader/local-JavaScript-from-local.html:
  • svg/dom/SVGScriptElement/script-reexecution.svg:
  • svg/dom/SVGScriptElement/script-set-href.svg:
18:31 Changeset [42664] by oliver@apple.com

<rdar://problem/6127705> Excessive memory consumption on image load failure

Reviewed by Antti Koivisto and Darin Adler

When we fail to decode an image we clear most of the image data, but not the
input data itself, which can be backed by a CFData object that itself holds onto
a few hundred Kbs of memory. This patch merely ensures that this buffer gets
cleared.

16:38 Changeset [42663] by darin@apple.com

2009-04-19 Darin Adler <darin@apple.com>

  • StringsNotToBeLocalized.txt: Updated for recent changes.
15:54 Changeset [42662] by weinig@apple.com

2009-04-19 Sam Weinig <sam@webkit.org>

Reviewed by Darin Adler.

Better fix for JSStringCreateWithCFString hardening.

  • API/JSStringRefCF.cpp: (JSStringCreateWithCFString):
15:48 Changeset [42661] by darin@apple.com

ooo

15:47 Changeset [42660] by darin@apple.com

2009-04-19 Darin Adler <darin@apple.com>

Reviewed by Sam Weinig.

Add hoooks so sudden termination will not cause loss of icons or local storage.
rdar://problem/5951701

  • loader/icon/IconDatabase.cpp: Marked various constants static to give them internal linkage. (WebCore::IconDatabase::IconDatabase): Added initial value for m_syncTimer. Removed unneeded initial value for m_defaultIconRecord. Changed assertion to work cross-platform. (WebCore::IconDatabase::wakeSyncThread): Added disableSuddenTermination, so we won't terminate until the sync thread has done its thing. (WebCore::IconDatabase::scheduleOrDeferSyncTimer): Added disableSuddenTermination, so we won't terminate if we have an outstanding sync timer. (WebCore::IconDatabase::syncTimerFired): Added enableSuddenTermination, to balance the one in scheduleOrDeferSyncTimer. (WebCore::IconDatabase::syncThreadMainLoop): Added enableSuddenTermination, to balance the one in wakeSyncThread. (WebCore::IconDatabase::deleteAllPreparedStatements): Use clear() instead of set(0).
  • loader/icon/IconDatabase.h: Use a Timer instead of an OwnPtr<Timer>.
  • storage/LocalStorageArea.cpp: (WebCore::LocalStorageArea::scheduleFinalSync): Added disableSuddenTermination. (WebCore::LocalStorageArea::scheduleItemForSync): Ditto. (WebCore::LocalStorageArea::scheduleClear): Ditto. (WebCore::LocalStorageArea::syncTimerFired): Added a disableSuddenTermination if we schedule a performSync callback for later and an unconditional enableSuddenTermination to balance the ones in the schedule calls above. (WebCore::LocalStorageArea::sync): Factored out the work of the sync function so it can continue to use early return idiom. (WebCore::LocalStorageArea::performSync): Added a call to enableSuddenTermination.
  • storage/LocalStorageArea.h: Added declaration of the sync function.
15:26 Changeset [42659] by weinig@apple.com

2009-04-19 Sam Weinig <sam@webkit.org>

Reviewed by Dan Bernstein.

Fix for <rdar://problem/5860954>
Harden JSStringCreateWithCFString against malformed CFStringRefs.

  • API/JSStringRefCF.cpp: (JSStringCreateWithCFString):
10:05 Changeset [42658] by ddkilzer@apple.com

Make FEATURE_DEFINES completely dynamic

Reviewed by Darin Adler.

Make FEATURE_DEFINES depend on individual ENABLE_FEATURE_NAME
variables for each feature, making it possible to remove all
knowledge of FEATURE_DEFINES from build-webkit.

JavaScriptCore:

  • Configurations/FeatureDefines.xcconfig: Extract a variable from FEATURE_DEFINES for each feature setting.

WebCore:

  • Configurations/FeatureDefines.xcconfig: Extract a variable from FEATURE_DEFINES for each feature setting.

WebKit/mac:

  • Configurations/FeatureDefines.xcconfig: Extract a variable from FEATURE_DEFINES for each feature setting.

WebKitTools:

  • Scripts/build-webkit: Define individual ENABLE_FEATURE_NAME variables instead of the whole FEATURE_DEFINES string when building with Xcode. Miscellaneous clean up:
  • Reordered/sorted command-line switch variables.
  • Reordered/sorted command-line switches in $usage.
  • Reordered/sorted command-line switches in GetOptions().
  • Defined $svgSupport to true if any --svg-feature switch is set in order to match "Implies SVG Support" claim.
  • Build JavaScriptGlue after JavaScriptCore again!
  • Fixed --[no-]geolocation for GTK+ port and sorted GTK+ command-line switches.
  • Added "exit 0" statement at the end of the script.
00:38 Changeset [42657] by ggaren@apple.com

2009-04-18 Geoffrey Garen <ggaren@apple.com>

Reviewed by Alexey Proskuryakov.

More fix for https://bugs.webkit.org/show_bug.cgi?id=21260
Unbounded memory growth when churning elements with anonymous event handler functions


Removed a little more complexity from event handler creation and destruction.


Removed the jsProtectedEventListeners, jsProtectedInlineEventListeners,
and jsInlineEventListeners maps, and all the code for managing them.


ProtectedEventListeners don't exist anymore, so they're easy to nix.


Inline EventListeners do still exist, but there's no reason to track
them in a map. The map exists to enable 'removeEventListener' to associate
a unique JSEventListener with a given JavaScript function. But the
'removeEventListener' API only works with non-inline event listeners!


  • bindings/js/JSDOMGlobalObject.cpp: (WebCore::JSDOMGlobalObject::~JSDOMGlobalObject): (WebCore::JSDOMGlobalObject::findJSEventListener): (WebCore::JSDOMGlobalObject::findOrCreateJSEventListener): (WebCore::JSDOMGlobalObject::createJSInlineEventListener):
  • bindings/js/JSDOMGlobalObject.h:
  • bindings/js/JSEventListener.cpp: (WebCore::JSEventListener::JSEventListener): (WebCore::JSEventListener::clearJSFunctionInline):
  • bindings/js/JSLazyEventListener.cpp: (WebCore::JSLazyEventListener::~JSLazyEventListener): (WebCore::JSLazyEventListener::parseCode):
  • bindings/scripts/CodeGeneratorJS.pm:
Note: See TracTimeline for information about the timeline view.