Timeline


and

11/22/09:

22:21 Changeset [51304] by eric@webkit.org

2009-11-22 Chris Fleizach <cfleizach@apple.com>

Reviewed by Oliver Hunt.

ARIA: support aria-flowto
https://bugs.webkit.org/show_bug.cgi?id=31762

  • platform/mac/accessibility/aria-flowto-expected.txt: Added.
  • platform/mac/accessibility/aria-flowto.html: Added.

2009-11-22 Chris Fleizach <cfleizach@apple.com>

Reviewed by Oliver Hunt.

ARIA: support aria-flowto
https://bugs.webkit.org/show_bug.cgi?id=31762

Test: platform/mac/accessibility/aria-flowto.html

  • accessibility/AccessibilityObject.h: (WebCore::AccessibilityObject::ariaOwnsElements): (WebCore::AccessibilityObject::supportsARIAFlowTo): (WebCore::AccessibilityObject::ariaFlowToElements):
  • accessibility/AccessibilityRenderObject.cpp: (WebCore::AccessibilityRenderObject::linkedUIElements): (WebCore::AccessibilityRenderObject::supportsARIAFlowTo): (WebCore::AccessibilityRenderObject::ariaFlowToElements):
  • accessibility/AccessibilityRenderObject.h:
  • html/HTMLAttributeNames.in:

2009-11-22 Chris Fleizach <cfleizach@apple.com>

Reviewed by Oliver Hunt.

ARIA: support aria-flowto
https://bugs.webkit.org/show_bug.cgi?id=31762

  • DumpRenderTree/AccessibilityUIElement.cpp: (ariaFlowToElementAtIndexCallback): (AccessibilityUIElement::getJSClass):
  • DumpRenderTree/AccessibilityUIElement.h:
  • DumpRenderTree/gtk/AccessibilityUIElementGtk.cpp: (AccessibilityUIElement::ariaFlowToElementAtIndex):
  • DumpRenderTree/mac/AccessibilityUIElementMac.mm: (AccessibilityUIElement::ariaFlowToElementAtIndex):
  • DumpRenderTree/win/AccessibilityUIElementWin.cpp: (AccessibilityUIElement::ariaFlowToElementAtIndex):
20:27 Changeset [51303] by mitz@apple.com

Updated WebKitSystemInterface.

Reviewed by Eric Carlson.

  • libWebKitSystemInterfaceLeopard.a:
  • libWebKitSystemInterfaceSnowLeopard.a:
  • libWebKitSystemInterfaceTiger.a:
19:45 Changeset [51302] by laszlo.1.gombos@nokia.com

2009-11-22 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Unreviewed.

[Qt] Sort source lists and remove obsolete comments
from the build system.

17:38 HighLevelOverview edited by chris.jerdonek@gmail.com
Added information about JavaScriptGlue (diff)
17:33 Changeset [51301] by eric@webkit.org

2009-11-22 Dirk Schulze <krit@webkit.org>

Reviewed by Nikolas Zimmermann.

Tests to the filterRes implementation for SVG Filters
https://bugs.webkit.org/show_bug.cgi?id=6021

big-sized-filter: tests if filter quality is scaled down on a too

big SourceImage (SourceGraphic, SourceAlpha)

big-sized-filter-2: tests if the filter quality is scaled down

on a too big filter effect size.

filterRes: Various tests of SVG Filters with different values for

filterRes attributes. Checks

  • that kernel values (if present) are scaled with the filterRes
  • subRegion calculation is correct for effects with deeper dependencies.
  • platform/mac/svg/filters/big-sized-filter-2-expected.txt: Added.
  • platform/mac/svg/filters/big-sized-filter-expected.checksum: Added.
  • platform/mac/svg/filters/big-sized-filter-expected.png: Added.
  • platform/mac/svg/filters/big-sized-filter-expected.txt: Added.
  • platform/mac/svg/filters/filterRes-expected.checksum: Added.
  • platform/mac/svg/filters/filterRes-expected.png: Added.
  • platform/mac/svg/filters/filterRes-expected.txt: Added.
  • svg/filters/big-sized-filter-2.svg: Added.
  • svg/filters/big-sized-filter.svg: Added.
  • svg/filters/filterRes.svg: Added.
17:29 Changeset [51300] by zimmermann@webkit.org

Not reviewed. Build fix for WML enabled builds.
Adopt WebCore API changes.

15:14 Changeset [51299] by tonikitoo@webkit.org

[Qt] fast/history/back-forward-reset-after-error-handling.html failing due to WorkQueue not being un-frozen
https://bugs.webkit.org/show_bug.cgi?id=31638

Reviewed by Kenneth Christiansen.

Unfreeze WorkQueue after each test execution.

  • DumpRenderTree/qt/DumpRenderTree.cpp:

(WebCore::DumpRenderTree::resetToConsistentStateBeforeTesting):

14:29 Changeset [51298] by eric@webkit.org

2009-11-22 Jakub Wieczorek <faw217@gmail.com>

Reviewed by Adam Barth.

[Qt] DumpRenderTree should explicitly ignore any SSL certificate errors
for localhost and 127.0.0.1.
https://bugs.webkit.org/show_bug.cgi?id=31783

Unskip the http/tests/ssl/verify-ssl-enabled.php test, which is passing now.

  • platform/qt/Skipped:

2009-11-22 Jakub Wieczorek <faw217@gmail.com>

Reviewed by Adam Barth.

[Qt] DumpRenderTree should explicitly ignore any SSL certificate errors
for localhost and 127.0.0.1.
https://bugs.webkit.org/show_bug.cgi?id=31783

Unskip the http/tests/ssl/verify-ssl-enabled.php test, which is passing now.

  • DumpRenderTree/qt/DumpRenderTree.cpp: (WebCore::NetworkAccessManager::NetworkAccessManager): (WebCore::NetworkAccessManager::sslErrorsEncountered): (WebCore::WebPage::WebPage):
  • DumpRenderTree/qt/DumpRenderTree.h:
12:54 BuildingQtOnLinux edited by faw217@gmail.com
libxslt is no longer required (diff)
12:05 Changeset [51297] by pfeldman@chromium.org

2009-11-22 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Introduce sidebar background on timeline panel in order
to prevent it from flickering on scroll.

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

  • inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel): (WebInspector.TimelinePanel.prototype.setSidebarWidth): (WebInspector.TimelinePanel.prototype._onScroll): (WebInspector.TimelinePanel.prototype._scheduleRefresh):
  • inspector/front-end/inspector.css:
10:25 Changeset [51296] by pfeldman@chromium.org

2009-11-22 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Reimplement TimelinePanel to make it fast:

  • Extract grid and overview into separate files
  • Make timeline create only divs for visible rows

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

  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • inspector/front-end/AbstractTimelinePanel.js: (WebInspector.AbstractTimelinePanel.prototype.createInterface): (WebInspector.AbstractTimelinePanel.prototype.refresh): (WebInspector.AbstractTimelinePanel.prototype.set calculator):
  • inspector/front-end/TimelineGrid.js: Added. (WebInspector.TimelineGrid): (WebInspector.TimelineGrid.prototype.get itemsGraphsElement): (WebInspector.TimelineGrid.prototype.updateDividers): (WebInspector.TimelineGrid.prototype.addEventDivider): (WebInspector.TimelineGrid.prototype.setScrollAndDividerTop):
  • inspector/front-end/TimelineOverviewPane.js: Added. (WebInspector.TimelineOverviewPane): (WebInspector.TimelineOverviewPane.prototype._onCheckboxClicked): (WebInspector.TimelineOverviewPane.prototype.update): (WebInspector.TimelineOverviewPane.prototype.setSidebarWidth): (WebInspector.TimelineOverviewPane.prototype.updateMainViewWidth): (WebInspector.TimelineOverviewPane.prototype.reset): (WebInspector.TimelineOverviewPane.prototype._resizeWindow): (WebInspector.TimelineOverviewPane.prototype._windowResizeDragging): (WebInspector.TimelineOverviewPane.prototype._dragWindow): (WebInspector.TimelineOverviewPane.prototype._windowDragging): (WebInspector.TimelineOverviewPane.prototype._resizeWindowLeft): (WebInspector.TimelineOverviewPane.prototype._resizeWindowRight): (WebInspector.TimelineOverviewPane.prototype._setWindowPosition): (WebInspector.TimelineOverviewPane.prototype._endWindowDragging): (WebInspector.TimelineOverviewCalculator): (WebInspector.TimelineOverviewCalculator.prototype.computeBarGraphPercentages): (WebInspector.TimelineOverviewCalculator.prototype.reset): (WebInspector.TimelineOverviewCalculator.prototype.updateBoundaries): (WebInspector.TimelineOverviewCalculator.prototype.get boundarySpan): (WebInspector.TimelineOverviewCalculator.prototype.formatValue): (WebInspector.TimelineCategoryTreeElement): (WebInspector.TimelineCategoryTreeElement.prototype.onattach): (WebInspector.TimelineCategoryGraph): (WebInspector.TimelineCategoryGraph.prototype.get graphElement): (WebInspector.TimelineCategoryGraph.prototype.addChunk): (WebInspector.TimelineCategoryGraph.prototype.clearChunks): (WebInspector.TimelineCategoryGraph.prototype.set dimmed):
  • inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel): (WebInspector.TimelinePanel.prototype._toggleTimelineButtonClicked): (WebInspector.TimelinePanel.prototype.addRecordToTimeline): (WebInspector.TimelinePanel.prototype._formatRecord): (WebInspector.TimelinePanel.prototype.setSidebarWidth): (WebInspector.TimelinePanel.prototype.updateMainViewWidth): (WebInspector.TimelinePanel.prototype.resize): (WebInspector.TimelinePanel.prototype.reset): (WebInspector.TimelinePanel.prototype.show): (WebInspector.TimelinePanel.prototype._onScroll): (WebInspector.TimelinePanel.prototype._scheduleRefresh): (WebInspector.TimelinePanel.prototype._refresh): (WebInspector.TimelinePanel.prototype._refreshRecords): (WebInspector.TimelinePanel.prototype._adjustScrollPosition): (WebInspector.TimelineCategory): (WebInspector.TimelineCalculator): (WebInspector.TimelineCalculator.prototype.get boundarySpan): (WebInspector.TimelineRecordListRow): (WebInspector.TimelineRecordListRow.prototype.update): (WebInspector.TimelineRecordGraphRow): (WebInspector.TimelineRecordGraphRow.prototype.update):
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.css:
  • inspector/front-end/inspector.html:
09:22 Changeset [51295] by abarth@webkit.org

2009-11-22 Chris Evans <cevans@chromium.org>

Reviewed by Adam Barth.

Disable access to file:/// directory listings
https://bugs.webkit.org/show_bug.cgi?id=31329

Updates the test to make sure it is disabling universal access for
file:/// URLs (if the API is supported). This is required to do
effective intra-file:/// access tests.
Note that Safari only passes by virtual of supporting no directory
listings at all.

  • fast/xmlhttprequest/resources/xmlhttprequest-nonexistent-file-real.html: Added.
  • fast/xmlhttprequest/xmlhttprequest-nonexistent-file-expected.txt:
  • fast/xmlhttprequest/xmlhttprequest-nonexistent-file.html: Disable universal access for file:/// URLs (if API available). Run the real test in an iframe without universal access.

2009-11-22 Chris Evans <cevans@chromium.org>

Reviewed by Adam Barth.

Disable access to file:/// directory listings
https://bugs.webkit.org/show_bug.cgi?id=31329

Deny access to directory listings. This needs doing in WebKit for
WebKit clients that do permit top-level navigation to file:///dir.
This matches Firefox, plus the existing Safari implementation (which
does not support any directory access at all). It will fix the
LayoutTest named below for Chromium.

Test: fast/xmlhttprequest/xmlhttprequest-nonexistent-file.html

  • page/SecurityOrigin.cpp: Deny access to directory listings. (WebCore::SecurityOrigin::SecurityOrigin): (WebCore::SecurityOrigin::canRequest):

2009-11-22 Chris Evans <cevans@chromium.org>

Reviewed by Adam Barth.

Disable access to file:/// directory listings
https://bugs.webkit.org/show_bug.cgi?id=31329

Implemented setAllowUniversalAccessFromFileURLs to support testing of
file URL security.

  • DumpRenderTree/LayoutTestController.cpp: (setAllowUniversalAccessFromFileURLsCallback): (LayoutTestController::staticFunctions):
  • DumpRenderTree/LayoutTestController.h:
  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (LayoutTestController::setAllowUniversalAccessFromFileURLs):
  • DumpRenderTree/mac/LayoutTestControllerMac.mm: (LayoutTestController::setAllowUniversalAccessFromFileURLs):
  • DumpRenderTree/win/LayoutTestControllerWin.cpp: (LayoutTestController::setAllowUniversalAccessFromFileURLs):
  • DumpRenderTree/wx/LayoutTestControllerWx.cpp: (LayoutTestController::setAllowUniversalAccessFromFileURLs):
09:22 Changeset [51294] by abarth@webkit.org

2009-11-22 Chris Evans <cevans@chromium.org>

Reviewed by Adam Barth.

Updates the test to make sure it is disabling universal access for
file:/// URLs (if the API is supported). This is required to do
effective intra-file:/// access tests.
Note that Safari only passes by virtual of supporting no directory
listings at all.

  • fast/xmlhttprequest/resources/xmlhttprequest-nonexistent-file-real.html: Added.
  • fast/xmlhttprequest/xmlhttprequest-nonexistent-file-expected.txt:
  • fast/xmlhttprequest/xmlhttprequest-nonexistent-file.html: Disable universal access for file:/// URLs (if API available). Run the real test in an iframe without universal access.

2009-11-22 Chris Evans <cevans@chromium.org>

Reviewed by Adam Barth.

Deny access to directory listings. This needs doing in WebKit for
WebKit clients that do permit top-level navigation to file:///dir.
This matches Firefox, plus the existing Safari implementation (which
does not support any directory access at all). It will fix the
LayoutTest named below for Chromium.

Test: fast/xmlhttprequest/xmlhttprequest-nonexistent-file.html

  • page/SecurityOrigin.cpp: Deny access to directory listings. (WebCore::SecurityOrigin::SecurityOrigin): (WebCore::SecurityOrigin::canRequest):

2009-11-22 Chris Evans <cevans@chromium.org>

Reviewed by Adam Barth.

Implemented setAllowUniversalAccessFromFileURLs to support testing of
file URL security.

  • DumpRenderTree/LayoutTestController.cpp: (setAllowUniversalAccessFromFileURLsCallback): (LayoutTestController::staticFunctions):
  • DumpRenderTree/LayoutTestController.h:
  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (LayoutTestController::setAllowUniversalAccessFromFileURLs):
  • DumpRenderTree/mac/LayoutTestControllerMac.mm: (LayoutTestController::setAllowUniversalAccessFromFileURLs):
  • DumpRenderTree/win/LayoutTestControllerWin.cpp: (LayoutTestController::setAllowUniversalAccessFromFileURLs):
  • DumpRenderTree/wx/LayoutTestControllerWx.cpp: (LayoutTestController::setAllowUniversalAccessFromFileURLs):
07:16 Changeset [51293] by eric@webkit.org

2009-11-22 Jakub Wieczorek <faw217@gmail.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Fix the timeout of fast/frames/frame-navigation.html
https://bugs.webkit.org/show_bug.cgi?id=31638

The test is timeouting, because it uses the WorkQueue to load a document in one
of the child frames and once the loading is finished, the DRT does not dump the
tree. This is because it waits for the QWebFrame::loadFinished() signal from
the main frame, while it should connect to QWebPage::loadFinished().

  • DumpRenderTree/qt/DumpRenderTree.cpp: (WebCore::DumpRenderTree::DumpRenderTree):
02:04 Changeset [51292] by mitz@apple.com

Fixed a leak in WKDrawMediaUIPart().

Reviewed by Cameron Zwarich.

  • libWebKitSystemInterfaceLeopard.a:
  • libWebKitSystemInterfaceSnowLeopard.a:
  • libWebKitSystemInterfaceTiger.a:

11/21/09:

19:39 Changeset [51291] by joepeck@webkit.org

2009-11-20 Joseph Pecoraro <joepeck@webkit.org>

Reviewed by Timothy Hatcher.

Web Inspector: Support Ctrl+P and Ctrl+N, Readline keyboard shortcuts in the Console
https://bugs.webkit.org/show_bug.cgi?id=31400

Handle the following when on a Mac:

Ctrl+P = Previous (like Up arrow)
Ctrl+N = Next (like Down arrow)

No longer rerun autocompletion when just pushing a modifier key
like Control, Alt, Shift, or Meta.

Improved arrow key behavior with Multiline code in the Console, with the following behavior:

Up = Previous Command if on First Line (caret moves to the end of the first line)

otherwise default caret movement in text.

Down = Next Command if on Last Line (caret naturally moves to the end)

otherwise default caret movement in text.

  • inspector/front-end/TextPrompt.js: (WebInspector.TextPrompt.prototype.handleKeyEvent): handle new keyboard shortcuts (WebInspector.TextPrompt.prototype.isCaretOnFirstLine): check if the caret is on the top line (WebInspector.TextPrompt.prototype.isCaretOnLastLine): check if the caret is on the bottom line (WebInspector.TextPrompt.prototype._upKeyPressed): (WebInspector.TextPrompt.prototype._downKeyPressed): (WebInspector.TextPrompt.prototype._moveBackInHistory): (WebInspector.TextPrompt.prototype._moveForwardInHistory):
19:04 Changeset [51290] by jberlin@webkit.org

Adds a tooltip containing the full url of the resource to each element
in the Resources panel so that it is possible to quickly see the query
parameters.
https://bugs.webkit.org/show_bug.cgi?id=19103

Reviewed by Timothy Hatcher.

  • inspector/front-end/ResourcesPanel.js:

(WebInspector.ResourceSidebarTreeElement.prototype.refresh):
Set the tooltip to be the resource's url.

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

2009-11-21 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dan Bernstein.

CSSKeyframesRule::findRule() and deleteRule() should accept 'from' and 'to' as well as percentages
https://bugs.webkit.org/show_bug.cgi?id=31588

In WebKitCSSKeyframesRule::findRuleIndex(), map 'from' to 0% an 'to' to 100%
so that findRule() and deleteRule() work with these keywords.

  • css/WebKitCSSKeyframesRule.cpp: (WebCore::WebKitCSSKeyframesRule::findRuleIndex):
15:13 Changeset [51288] by eric@webkit.org

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

Reviewed by Eric Seidel.

[Qt][Mac] Turn on multiple JavaScript threads for QtWebkit on Mac
https://bugs.webkit.org/show_bug.cgi?id=31753

  • wtf/Platform.h:
13:34 Changeset [51287] by eric@webkit.org

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

Reviewed by Adam Barth.

bugzilla-tool --help spews way too much text
https://bugs.webkit.org/show_bug.cgi?id=31771

  • Scripts/bugzilla-tool:
    • Remove self.cached_scm initialization hack.
  • Scripts/modules/buildbot.py:
    • Make default_host accessible to callers.
  • Scripts/modules/commands/download.py:
    • Phrase help for all commands consistently and remove spurious help text punctuation.
  • Scripts/modules/commands/queries.py: Ditto.
  • Scripts/modules/commands/queues.py: Ditto.
  • Scripts/modules/commands/upload.py: Ditto.
  • Scripts/modules/multicommandtool.py:
    • Add HelpPrintingOptionParser.format_epilog to replace NonWrappingEpilogIndentedHelpFormatter and allow us to lazily initialize per-command help (thus removing the need for the cached_scm hack in BugzillaTool).
    • Make --help only show a list of commands like "svn help" and "git help" do -- previously --help was listing all commands and options.
    • Sort list of commands alphabetically.
13:32 Changeset [51286] by jhoneycutt@apple.com

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

Windows build fix. Unreviewed.

  • Interfaces/IWebSerializedJSValue.idl:
13:26 Changeset [51285] by eric@webkit.org

2009-11-21 Daniel Bates <dbates@webkit.org>

Reviewed by Eric Seidel.

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

Changed KeyboardEvent::m_keyEvent from a raw pointer to an OwnPtr.

No functionality was changed. So, no new tests.

  • dom/KeyboardEvent.cpp: (WebCore::KeyboardEvent::~KeyboardEvent):
  • dom/KeyboardEvent.h: Changed m_keyEvent to OwnPtr. (WebCore::KeyboardEvent::keyEvent):
13:12 Changeset [51284] by eric@webkit.org

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

No review. Fixing a typo from the previous patch for bug 31767.

AbstractQueue.run_bugzilla_tool throws an exception
https://bugs.webkit.org/show_bug.cgi?id=31769

  • Scripts/modules/commands/queues.py:
07:50 Changeset [51283] by eric@webkit.org

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

Reviewed by Adam Barth.

commit-queue fails to run with "permissions error" due to bad bugzilla-tool path
https://bugs.webkit.org/show_bug.cgi?id=31767

What we really want to test is BugzillaTool.path() instead of TrivialTool.path().
Since we don't have a good way to test BugzillaTool pieces, I've
left out a test for now.

  • Scripts/bugzilla-tool:
    • Add a path() implementation to expose bugzilla-tool's file path to commit-queue.
  • Scripts/modules/commands/queues.py:
    • Use tool.path() instead of file.
  • Scripts/modules/multicommandtool.py:
    • Add a new path() method to MultiComandTool.
  • Scripts/modules/multicommandtool_unittest.py:
    • Provide a path() method. Little point in testing this mock implementation.
07:37 Changeset [51282] by abarth@webkit.org

2009-11-21 Adam Barth <abarth@webkit.org>

Unreviewed "build" fix. Turns out I was testing the wrong copy of
WebKitTools.

  • Scripts/modules/commands/download.py:
07:29 Changeset [51281] by abarth@webkit.org

2009-11-21 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Convert check-style to use LandingSequence
https://bugs.webkit.org/show_bug.cgi?id=31763

Instead of manipulating the working copy by hand, we should use the
LandingSequence in CheckStyle. This will make this code eaiser to
test.

  • Scripts/modules/commands/download.py:
00:53 Web Inspector Update.ja edited by keishi@webkit.org
(diff)
00:52 WebKit nightlies support HTML5 noreferrer link relation.ja created by keishi@webkit.org
00:50 Web Inspector Update.ja edited by keishi@webkit.org
(diff)
00:48 Surfin' Safari Translations edited by keishi@webkit.org
(diff)

11/20/09:

20:50 Changeset [51280] by jhoneycutt@apple.com

DOMElement::tagName() is unimplemented

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

Reviewed by Darin Adler.

  • DOMCoreClasses.cpp:

(DOMElement::tagName):
Create a BString from the element's tag name, and release its BSTR
into result.

20:50 Changeset [51279] by jhoneycutt@apple.com

DOMNode::childNodes() is unimplemented

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

Reviewed by Darin Adler.

  • DOMCoreClasses.cpp:

(DOMNode::childNodes):
Create a DOMNodeList from the node's child nodes.

20:06 Changeset [51278] by abarth@webkit.org

2009-11-20 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Unit test query commands
https://bugs.webkit.org/show_bug.cgi?id=31755

These tests are pretty rough, but hopefully they'll grow.

  • Scripts/modules/commands/queries_unittest.py: Added.
  • Scripts/modules/mock_bugzillatool.py: Added.
  • Scripts/run-webkit-unittests:
19:53 Changeset [51277] by abarth@webkit.org

2009-11-20 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

Fix a bunch of unit test regressions from our recent bugzilla-toll hacking
https://bugs.webkit.org/show_bug.cgi?id=31758

  • Scripts/modules/multicommandtool.py:
    • Allow passing of explicit commands to MultiCommandTool.init
  • Scripts/modules/multicommandtool_unittest.py:
    • Use new Command.name naming system.
    • Test Command auto-discovery.
  • Scripts/modules/workqueue.py:
    • bug_id no longer exists, use patchbug_id? instead.
  • Scripts/modules/workqueue_unittest.py:
    • WorkQueues require names now.
    • should_proceed_with_work_item must return a patch object.
19:29 Changeset [51276] by cfleizach@apple.com

WAI-ARIA: add support for aria-owns
https://bugs.webkit.org/show_bug.cgi?id=31702

Reviewed by Beth Dakin.

WebCore:

Re-organized how accessibilityAttributeNames are returned so
that it's possible for an element to add an attribute based on
a dynamic value. In this case, we only want to add AXOwn if the
element actually supports aria-owns.

Test: platform/mac/accessibility/aria-owns.html

  • accessibility/AccessibilityObject.h:

(WebCore::AccessibilityObject::supportsARIAOwns):
(WebCore::AccessibilityObject::ariaOwnsElements):

  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::ariaOwnsElements):
(WebCore::AccessibilityRenderObject::supportsARIAOwns):

  • accessibility/AccessibilityRenderObject.h:
  • accessibility/mac/AccessibilityObjectWrapper.mm:

(-[AccessibilityObjectWrapper additionalAccessibilityAttributeNames]):
(-[AccessibilityObjectWrapper accessibilityAttributeNames]):
(-[AccessibilityObjectWrapper accessibilityAttributeValue:]):

  • html/HTMLAttributeNames.in:

WebKitTools:

  • DumpRenderTree/AccessibilityUIElement.cpp:

(ariaOwnsElementAtIndexCallback):
(AccessibilityUIElement::getJSClass):

  • DumpRenderTree/AccessibilityUIElement.h:
  • DumpRenderTree/gtk/AccessibilityUIElementGtk.cpp:

(AccessibilityUIElement::ariaOwnsElementAtIndex):

  • DumpRenderTree/mac/AccessibilityUIElementMac.mm:

(AccessibilityUIElement::ariaOwnsElementAtIndex):

  • DumpRenderTree/win/AccessibilityUIElementWin.cpp:

(AccessibilityUIElement::ariaOwnsElementAtIndex):

LayoutTests:

  • platform/mac/accessibility/aria-owns-expected.txt: Added.
  • platform/mac/accessibility/aria-owns.html: Added.
19:14 Changeset [51275] by eric@webkit.org

2009-11-20 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

Fix exception thrown when running the commit-queue.

  • Scripts/modules/statusbot.py: patch is optional.
  • Scripts/modules/workqueue.py: WorkQUeue requires a name.
18:54 Changeset [51274] by hyatt@apple.com

Change WebSerializedJSValue on Windows to have a zero-argument createInstance call. Move the
serialization into a separate serialize() function that can be called after the object has been
created.

Reviewed by Jon Honeycutt.

Fix a typo in serialize() caused when fixing build bustage (put a ! back in).

  • Interfaces/IWebSerializedJSValue.idl:
  • WebSerializedJSValue.cpp:

(WebSerializedJSValue::WebSerializedJSValue):
(WebSerializedJSValue::createInstance):
(WebSerializedJSValue::serialize):
(WebSerializedJSValue::deserialize):

  • WebSerializedJSValue.h:
18:36 Changeset [51273] by bweinstein@apple.com

Another go at the Windows Build Fix - this is a couple steps of it,
so a little more involved than most build fixes.

Reviewed by Dave Hyatt.

  • Interfaces/IWebSerializedJSValue.idl:
  • Interfaces/JavaScriptCoreAPITypes.idl:
  • WebKit.vcproj/Interfaces.vcproj:
  • WebSerializedJSValue.cpp:

(WebSerializedJSValue::deserialize):

  • WebSerializedJSValue.h:
18:07 Changeset [51272] by hyatt@apple.com

Make sure to export WebSerializedJSValue.

Reviewed by Darin Adler.

  • WebKit.exp:
17:54 Changeset [51271] by bweinstein@apple.com

Build fix for Windows - only declare JSContextRef once, instead
of in two different idl files. Create a new file to hold the
definition of JSContextRef.

Reviewed by Dave Hyatt.

  • Interfaces/IWebFrameLoadDelegate.idl:
  • Interfaces/IWebSerializedJSValue.idl:
  • Interfaces/JavaScriptCoreAPITypes.idl: Added.
  • Interfaces/WebKit.idl:
  • WebKit.vcproj/Interfaces.vcproj:
17:44 Changeset [51270] by eric.carlson@apple.com

2009-11-20 Eric Carlson <eric.carlson@apple.com>

Reviewed by Simon Fraser.

<rdar://problem/7414396> Leopard & Tiger: Support closed caption in <video> element

  • css/mediaControls.css:

Define closed caption toggle button.

  • rendering/RenderMedia.cpp: (WebCore::RenderMedia::updateControls):

Create closed caption toggle button in display order.

  • rendering/RenderThemeMac.mm: (WebCore::RenderThemeMac::shouldRenderMediaControlPart):

Don't require MediaControllerThemeQuickTime theme for captions toggle button.
No need to check if the movie has video, a closed captions track is a video
track.

2009-11-20 Eric Carlson <eric.carlson@apple.com>

Reviewed by Simon Fraser.

<rdar://problem/7414396> Leopard & Tiger: Support closed caption in <video> element

  • WebKitSystemInterface.h:
  • libWebKitSystemInterfaceLeopard.a:
  • libWebKitSystemInterfaceSnowLeopard.a:
  • libWebKitSystemInterfaceTiger.a:

2009-11-20 Eric Carlson <eric.carlson@apple.com>

Reviewed by Simon Fraser.

<rdar://problem/7414396> Leopard & Tiger: Support closed caption in <video> element

Remove closed captions test from skipped list.

  • platform/mac-leopard/Skipped:
  • platform/mac-tiger/Skipped:
17:34 Changeset [51269] by jianli@chromium.org

Not reviewed. Fix chromium build.

  • public/WebAccessibilityRole.h:

(WebKit::):

  • src/AssertMatchingEnums.cpp:
17:18 Changeset [51268] by joepeck@webkit.org

2009-11-20 Joseph Pecoraro <joepeck@webkit.org>

Reviewed by Timothy Hatcher.

Web Inspector: Running code in the Console that ends with a
single line comment no longer produces a parse error.

  • inspector/front-end/InjectedScript.js: (InjectedScript._evaluateOn):
17:17 Changeset [51267] by hyatt@apple.com

Add support for WebSerializedJSValue to WebKit. This object wraps the SerializedScriptValue functionality in WebCore
and exposes the ability to do JS value serialization/deserialization to WebKit clients.

Reviewed by Oliver Hunt and Jon Honeycutt.

WebKit:

  • WebKit.xcodeproj/project.pbxproj:

WebKit/mac:

  • WebView/WebSerializedJSValue.h: Added.
  • WebView/WebSerializedJSValue.mm: Added.

(-[WebSerializedJSValue initWithValue:context:]):
(-[WebSerializedJSValue deserialize:]):
(-[WebSerializedJSValue dealloc]):

WebKit/win:

  • Interfaces/IWebSerializedJSValue.idl: Added.
  • Interfaces/WebKit.idl:
  • WebKit.vcproj/WebKit.vcproj:
  • WebSerializedJSValue.cpp: Added.

(WebSerializedJSValue::WebSerializedJSValue):
(WebSerializedJSValue::~WebSerializedJSValue):
(WebSerializedJSValue::createInstance):
(WebSerializedJSValue::AddRef):
(WebSerializedJSValue::Release):
(WebSerializedJSValue::QueryInterface):
(WebSerializedJSValue::deserialize):

  • WebSerializedJSValue.h: Added.
17:15 Changeset [51266] by eric@webkit.org

2009-11-20 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

Teach the StatusBot how to support more than just the commit-queue
https://bugs.webkit.org/show_bug.cgi?id=31754

  • Scripts/modules/workqueue.py: Another typo.
17:09 Changeset [51265] by eric@webkit.org

2009-11-20 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

Teach the StatusBot how to support more than just the commit-queue
https://bugs.webkit.org/show_bug.cgi?id=31754

  • Scripts/modules/commands/queues.py: Fix silly typo.
17:02 Changeset [51264] by eric@webkit.org

2009-11-20 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

Teach the StatusBot how to support more than just the commit-queue
https://bugs.webkit.org/show_bug.cgi?id=31754

  • CommitQueueStatus/index.yaml:
    • Add indices required for the new queries.
  • CommitQueueStatus/queue_status.py:
    • Add a patch-status page and move update_status to update-status.
    • Only display "commit-queue" status records for the commit-queue.
    • Add support for a queue_name property on status records.
    • Fix _int_from_request to actually work.
  • CommitQueueStatus/update_status.html:
    • Add support for a queue_name on status records.
    • Remove unused list of bug ids.
    • Scripts/modules/commands/queues.py
      • Make the queues pass the patch instead of the bug_id to StatusBot.
  • Scripts/modules/statusbot.py:
    • Support passing the queue_name to the status updates.
    • Support fetching patch status with patch_status().
  • Scripts/modules/workqueue.py:
    • Pass the patch to the StatusBot instead of the bug_id.
    • Let WorkQueues have a name.
16:15 Changeset [51263] by abarth@webkit.org

2009-11-20 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Move bugzilla-tool commands into their own file
https://bugs.webkit.org/show_bug.cgi?id=31752

This will let us write unit tests.

  • Scripts/bugzilla-tool:
  • Scripts/modules/commands/init.py: Added.
  • Scripts/modules/commands/download.py: Added.
  • Scripts/modules/commands/queries.py: Added.
  • Scripts/modules/commands/queues.py: Added.
  • Scripts/modules/commands/upload.py: Added.
  • Scripts/modules/grammar.py: Added.
16:00 Changeset [51262] by eric@webkit.org

2009-11-20 Chris Fleizach <cfleizach@apple.com>

Reviewed by Beth Dakin.

WAI-ARIA: add support for 'math' role
https://bugs.webkit.org/show_bug.cgi?id=31706

  • platform/mac/accessibility/aria-grouping-roles-expected.txt:
  • platform/mac/accessibility/aria-grouping-roles.html:

2009-11-20 Chris Fleizach <cfleizach@apple.com>

Reviewed by Beth Dakin.

WAI-ARIA: add support for 'math' role
https://bugs.webkit.org/show_bug.cgi?id=31706

  • accessibility/AccessibilityObject.cpp:
  • accessibility/AccessibilityObject.h:
  • accessibility/mac/AccessibilityObjectWrapper.mm:

2009-11-20 Chris Fleizach <cfleizach@apple.com>

Reviewed by Beth Dakin.

WAI-ARIA: add support for 'math' role
https://bugs.webkit.org/show_bug.cgi?id=31706

  • English.lproj/Localizable.strings:
  • StringsNotToBeLocalized.txt:

2009-11-20 Chris Fleizach <cfleizach@apple.com>

Reviewed by Beth Dakin.

WAI-ARIA: add support for 'math' role
https://bugs.webkit.org/show_bug.cgi?id=31706

  • WebCoreSupport/WebViewFactory.mm: (-[WebViewFactory AXARIAContentGroupText:]):
14:54 Changeset [51261] by abarth@webkit.org

Fix ChangeLog

14:52 Changeset [51260] by abarth@webkit.org

2009-11-20 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.
Reviewed by Eric Seidel.

BuildQueue should check if the tree is currently buildable
https://bugs.webkit.org/show_bug.cgi?id=31744

  • Scripts/bugzilla-tool:
  • Scripts/modules/landingsequence.py:
  • Scripts/modules/webkitlandingscripts.py:
14:51 Changeset [51259] by abarth@webkit.org

2009-11-20 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Move prepare_clean_working_directory into the LandingSequence
https://bugs.webkit.org/show_bug.cgi?id=31743

  • Scripts/bugzilla-tool:
  • Scripts/modules/landingsequence.py:
14:34 Changeset [51258] by yael.aharon@nokia.com

Unreviewed fix.

Fix file names in the skipped list, for files added in http://trac.webkit.org/changeset/51257.

13:40 Changeset [51257] by yael.aharon@nokia.com

NPN_ReloadPlugins does not reload the page even if reloadPages is true.
https://bugs.webkit.org/show_bug.cgi?id=30460

Reviewed by Kenneth Rohde Christiansen.

WebCore:

Replace call to PluginDatabase::installedPlugins()->refresh() with Page::refreshPlugins().
It already refreshes plugins and reloads pages when necessary.

Tests: plugins/reloadplugins-and-pages.html

plugins/reloadplugins-no-pages.html

  • plugins/PluginInfoStore.cpp:

(WebCore::refreshPlugins):

WebKitTools:

Added code for calling NPN_ReloadPlugins with reloadPages true and false.

  • DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.cpp:

(pluginInvoke):

LayoutTests:

Added tests for calling NPN_ReloadPlugins with reloadPages being true and false.

13:32 Changeset [51256] by oliver@apple.com

<rdar://7409188> WebKit needs to be able to serialize and deserialize objects.

Reviewed by Dave Hyatt.

Expose WebCore object serialization to WebKit.

13:29 Changeset [51255] by bweinstein@apple.com

Fixes <http://webkit.org/b/31741>.
Web Inspector: User Entered Data on the console should show up no matter what filters.

Reviewed by Tim Hatcher.

Always show .console-user-command-result, no matter what the filter say
we should do with the other classes.

  • inspector/front-end/inspector.css:
13:29 Changeset [51254] by ap@apple.com

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=25381
Needs test: jQuery animation crashing Safari

Adding a regression test for a bug that doesn't reproduce in ToT.

  • fast/dynamic/jQuery-animation-crash-expected.txt: Added.
  • fast/dynamic/jQuery-animation-crash.html: Added.
13:03 Changeset [51253] by abarth@webkit.org

2009-11-20 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Pass the port information to the child process
https://bugs.webkit.org/show_bug.cgi?id=31736

We need to do this so the child process knows what to build!

  • Scripts/bugzilla-tool:
  • Scripts/modules/landingsequence.py:
  • Scripts/modules/webkitport.py:
  • Scripts/modules/webkitport_unittest.py:
12:26 Changeset [51252] by ap@apple.com

WebSocket tests still don't work on Windows, disabling for futher invesigation.

  • platform/win/Skipped:
11:55 Changeset [51251] by bweinstein@apple.com

Fix a closing tag issue in the vcproj and remove tabs.

11:51 Changeset [51250] by bweinstein@apple.com

Fixes <http://webkit.org/b/31700>.
Web Inspector: isMac should be in one central location + Cached.

Reviewed by Tim Hatcher.

We should cache the value of isMac, and make the indexOf call in one
single location, because we are calling it from multiple places, and saving
the result can save us time, and make the call sites simpler.

  • inspector/front-end/AbstractTimelinePanel.js:

(WebInspector.AbstractTimelinePanel.prototype._updateFilter):

  • inspector/front-end/ConsoleView.js:

(WebInspector.ConsoleView.prototype._updateFilter):

  • inspector/front-end/ElementsPanel.js:

(WebInspector.ElementsPanel.prototype.handleKeyEvent):

  • inspector/front-end/ScriptsPanel.js:

(WebInspector.ScriptsPanel):

  • inspector/front-end/SourceFrame.js:

(WebInspector.SourceFrame.prototype._loaded):

  • inspector/front-end/inspector.js:

(WebInspector.documentKeyDown):
(WebInspector.isMac):

11:31 Changeset [51249] by abarth@webkit.org

2009-11-20 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Support Qt port in build-queue
https://bugs.webkit.org/show_bug.cgi?id=31733

  • Scripts/bugzilla-tool:
11:22 Changeset [51248] by eric@webkit.org

2009-11-20 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Implement a build-queue
https://bugs.webkit.org/show_bug.cgi?id=31725

Currently this just builds the first 10 patches in the review queue.
We'll want to do something smarter soon.

  • Scripts/bugzilla-tool:
11:22 Changeset [51247] by ap@apple.com

Enabling WebSocket tests on Windows - let's see if they pass after the recent fixes.

  • platform/win/Skipped:
11:19 Changeset [51246] by joepeck@webkit.org

2009-11-19 Joseph Pecoraro <joepeck@webkit.org>

Removed .DS_Store accidentally added in last (r51245).

11:11 Changeset [51245] by joepeck@webkit.org

2009-11-19 Joseph Pecoraro <joepeck@webkit.org>

Reviewed by Timothy Hatcher.

Web Inspector: Add Console Only Layout Mode
https://bugs.webkit.org/show_bug.cgi?id=30282

Adds a Console Panel, which allows for a Full size Console. This
extends the Drawer to the size of a Panel. This also fixes previous
resize issues with the Drawer.

Miscellaneous changes and cleanup.

  • English.lproj/localizedStrings.js: "Console" toolbar title.
  • inspector/front-end/ConsoleView.js: removed unnecessary element reordering
  • inspector/front-end/Images/consoleIcon.png: added.
  • inspector/front-end/inspector.css: added icon.
  • inspector/front-end/inspector.html:
  • inspector/front-end/inspector.js: (WebInspector._createPanels): create console panel (WebInspector.windowResize): resize drawer if necessary (WebInspector.documentKeyDown): esc should not toggle the console when in panel mode (WebInspector.animateStyle): start animation interval, returns the interval key (WebInspector.toggleAttach): resize drawer if necessary (WebInspector.showConsolePanel): restore the panel when the inspector restarts (WebInspector.showProfileForURL): style fix

Restore the panel when the inspector restarts.

  • inspector/InspectorController.cpp: (WebCore::InspectorController::setWindowVisible): (WebCore::InspectorController::specialPanelForJSName):
  • inspector/InspectorFrontend.cpp: (WebCore::InspectorFrontend::showPanel):

Build files.

  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • inspector/front-end/WebKit.qrc:
11:06 Changeset [51244] by eric@webkit.org

2009-11-20 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Make commit-queue and style-queue show up in help
https://bugs.webkit.org/show_bug.cgi?id=31724

We need to store their names on their class to make these commands
properly register themselves with MultiCommandTool.

  • Scripts/bugzilla-tool:
10:57 Changeset [51243] by eric@webkit.org

2009-11-20 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Implement bugzilla-tool build-attachment
https://bugs.webkit.org/show_bug.cgi?id=31722

This command builds an attachment from bugzilla. It leaves the built
patch in the working copy.

  • Scripts/bugzilla-tool:
  • Scripts/modules/landingsequence.py:
  • Scripts/modules/webkitlandingscripts.py:
10:40 Changeset [51242] by bweinstein@apple.com

Fixes <http://webkit.org/b/31699>.
Web Inspector: Should Cache Values of InspectorController.platform() and port().

Reviewed by Tim Hatcher.

Refactor the Inspector to cache the value of InspectorController.platform
and InspectorController.port, because those need to call into C++, they are
being called more and more as we add platform specific keyboard shortcuts, and
they shouldn't change in the lifecycle of the Web Inspector.

  • inspector/front-end/AbstractTimelinePanel.js:

(WebInspector.AbstractTimelinePanel.prototype._updateFilter):

  • inspector/front-end/ConsoleView.js:

(WebInspector.ConsoleView.prototype._updateFilter):

  • inspector/front-end/ElementsPanel.js:

(WebInspector.ElementsPanel.prototype.handleKeyEvent):

  • inspector/front-end/ScriptsPanel.js:

(WebInspector.ScriptsPanel):

  • inspector/front-end/SourceFrame.js:

(WebInspector.SourceFrame.prototype._loaded):

  • inspector/front-end/inspector.js:

(WebInspector.get platform):
(WebInspector.get port):
(WebInspector.loaded):
(WebInspector.documentKeyDown):
(WebInspector.toolbarDragStart):

09:53 Changeset [51241] by eric@webkit.org

2009-11-20 Victor Wang <victorw@chromium.org>

Reviewed by Darin Adler.

Add expected pixel result for layout test input-file-re-render

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

  • platform/mac/fast/forms/input-file-re-render-expected.checksum: Added.
  • platform/mac/fast/forms/input-file-re-render-expected.png: Added.
09:49 Changeset [51240] by krit@webkit.org

2009-11-20 Dirk Schulze <krit@webkit.org>

Unreviewed build fix for filters enabled builds.

  • platform/graphics/filters/FEColorMatrix.cpp: (WebCore::FEColorMatrix::apply):
  • platform/graphics/filters/FEComposite.cpp: (WebCore::FEComposite::apply):
  • platform/graphics/filters/SourceGraphic.cpp: (WebCore::SourceGraphic::apply):
  • svg/graphics/SVGResourceFilter.cpp: (WebCore::SVGResourceFilter::applyFilter):
  • svg/graphics/filters/SVGFEMerge.cpp: (WebCore::FEMerge::apply):
  • svg/graphics/filters/SVGFEOffset.cpp: (WebCore::FEOffset::apply):
  • svg/graphics/filters/SVGFETile.cpp: (WebCore::FETile::apply):
09:45 Changeset [51239] by eric@webkit.org

2009-11-20 Alejandro G. Castro <alex@igalia.com>

Reviewed by Xan Lopez.

[GTK] DRT release event does not create the state correctly
https://bugs.webkit.org/show_bug.cgi?id=31717

  • WebKitTools/DumpRenderTree/gtk/EventSender.cpp:
09:27 Changeset [51238] by eric.carlson@apple.com

2009-11-20 Eric Carlson <eric.carlson@apple.com>

Reviewed by Simon Fraser.

<rdar://problem/7389945> QTKit based media engine should not claim to support
fullscreen on Leopard

  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::supportsFullscreen):

Only return true on SnowLeopard for now.

  • rendering/RenderThemeMac.mm: (WebCore::RenderThemeMac::shouldRenderMediaControlPart):

No need to special case MediaFullscreenButtonPart as the default implementation
asks the media engine if it supports fullscreen.

2009-11-20 Eric Carlson <eric.carlson@apple.com>

Reviewed by Simon Fraser.

<rdar://problem/7389945> QTKit based media engine should not claim to support fullscreen on Leopard

Update test results.

  • platform/mac/media/audio-controls-rendering-expected.checksum:
  • platform/mac/media/audio-controls-rendering-expected.png:
  • platform/mac/media/audio-controls-rendering-expected.txt:
06:54 Changeset [51237] by abarth@webkit.org

2009-11-20 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

MultiCommandTool should find Command objects automatically instead of with a manual list
https://bugs.webkit.org/show_bug.cgi?id=31710

  • Scripts/bugzilla-tool:
  • Scripts/modules/multicommandtool.py:
    • Use some wild python-fu to crawl all the known subclasses of Command.
06:45 Changeset [51236] by abarth@webkit.org

2009-11-20 Adam Barth <abarth@webkit.org>

Unreviewed "build" fix. Added missing import.

  • Scripts/bugzilla-tool:
06:44 Changeset [51235] by abarth@webkit.org

2009-11-20 Adam Barth <abarth@webkit.org>

Unreviewed "build" fix. I failed to update LandingSequence.test
properly.

  • Scripts/modules/landingsequence.py:
06:21 QtWebKitBackportingFixes created by hausmann@webkit.org
05:47 QtWebKitContrib edited by hausmann@webkit.org
(diff)
04:29 QtWebKitJournal edited by hausmann@webkit.org
(diff)
03:18 Changeset [51234] by girish@forwardbias.in

[Qt] Implement support for rendering plugins on Qt/Mac when a page is used
without a QWebView or when inside QGraphicsWebView.

Reviewed by Simon Hausmann.

Patch by Yongjun Zhang <yongjun.zhang@nokia.com> and

Girish Ramakrishnan <girish@forwardbias.in>

Currently, the code provides the cgcontext of the PlatformPluginWidget to
the plugin. This approach does not work when we are printing, or using
QWebFrame::render() to render to a QImage/QPixmap since the plugin ends
up drawing on the QWebView (i.e platformPluginWidget's cgcontext) instead
of the QPaintDevice's context.

To solve all cases and keep the code simple, we render the plugin to an
offscreen pixmap in all cases. This way, the plugin always renders to the
CGContext of the pixmap and we then use QPainter to blit the pixmap into
the QPaintDevice. We also create a fake window and set it's WindowRef in
NPWindow. Only with this WindowRef does Flash paint correctly to the QPixmap.

Now, that's the theory. In practice, ATM, mouse events do not work when using
the fake window. So, setPlatformPluginWidget() is still called when using QWebView
so that there are no regressions after this patch. Once we get mouse events
working, setPlatformPluginWidget will be removed.

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

01:01 Changeset [51233] by eric@webkit.org

2009-11-20 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Create LandingSequence as the all-sing, all-dance landing class
https://bugs.webkit.org/show_bug.cgi?id=31709

Client can inherit from this class to carefully control exactly which
steps they wish to have happen in the landing sequence.

  • Scripts/bugzilla-tool:
  • Scripts/modules/landingsequence.py: Added.
  • Scripts/modules/webkitlandingscripts.py: Added.

11/19/09:

23:01 Changeset [51232] by eric@webkit.org

2009-11-19 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Abstract AbstractPatchProcessingCommand from AbstractPatchLandingCommand
https://bugs.webkit.org/show_bug.cgi?id=31707

This is to help when we implement build-attachment.

  • Scripts/bugzilla-tool:
22:43 Changeset [51231] by eric@webkit.org

2009-11-19 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Support Qt port in bugzilla-tool
https://bugs.webkit.org/show_bug.cgi?id=31701

Now we support building with Qt!

  • Scripts/bugzilla-tool:
  • Scripts/modules/webkitport.py: Added.
  • Scripts/modules/webkitport_unittest.py: Added.
  • Scripts/run-webkit-unittests:
22:20 Changeset [51230] by eric@webkit.org

2009-11-19 Zoltan Horvath <zoltan@webkit.org>

Reviewed by Adam Barth.

Remove inserting stderr into patch in bugzilla-tool
https://bugs.webkit.org/show_bug.cgi?id=29914

Modify SCM python module's run_command function to avoid return of stderr
by default, so stderr won't be inserted into the patches.
Modify the related unit test.

  • Scripts/modules/scm.py:
  • Scripts/modules/scm_unittest.py:
22:17 Changeset [51229] by joepeck@webkit.org

2009-11-19 Joseph Pecoraro <joepeck@webkit.org>

Reviewed by Timothy Hatcher.

Web Inspector: Resync Resources Backend and Frontend
https://bugs.webkit.org/show_bug.cgi?id=31705

  • inspector/front-end/Resource.js: resync enum values with backend (WebInspector.Resource.prototype._mimeTypeIsConsistentWithType): return false if all else fails, not true in all cases
22:03 Changeset [51228] by eric@webkit.org

2009-11-19 Steve Block <steveblock@google.com>

Android port lacks configuration in Platform.h and config.h.
https://bugs.webkit.org/show_bug.cgi?id=31671

  • wtf/Platform.h: Modified. Added Android-specific configuration.

2009-11-19 Steve Block <steveblock@google.com>

Reviewed by Darin Fisher.

Android port lacks configuration in Platform.h and config.h.
https://bugs.webkit.org/show_bug.cgi?id=31671

Build change only. No new tests possible.

  • config.h: Modified. Added Android-specific configuration.
21:44 Changeset [51227] by rolandsteiner@chromium.org

2009-11-20 Roland Steiner <rolandsteiner@chromium.org>

WebCore: Change default stylesheet to reset text indentation for ruby elements by default.
(https://bugs.webkit.org/show_bug.cgi?id=31247)

Reviewed by Darin Adler.

Test: fast/ruby/ruby-text-indent.html

  • css/html.css: reset text-indent on <ruby> and <rt> elements

LayoutTests: Test checking that CSS 'text-indent' does not affect ruby rendering by default.
(https://bugs.webkit.org/show_bug.cgi?id=31247)

Reviewed by Darin Adler.

  • fast/ruby/ruby-text-indent-expected.txt: Added.
  • fast/ruby/ruby-text-indent.html: Added.
18:18 Changeset [51226] by eric@webkit.org

2009-11-19 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

bugzilla-tool needs per-command help
https://bugs.webkit.org/show_bug.cgi?id=31697

Added support for "bugzilla-tool help command-name"
and a unit test to make sure it works.

  • Scripts/modules/multicommandtool.py:
  • Scripts/modules/multicommandtool_unittest.py:
18:12 Changeset [51225] by bweinstein@apple.com

Fixes <http://webkit.org/b/22754>.
Web Inspector: Keyboard shortcut for Element finder.

Reviewed by Darin Adler.

Use Command + Shift + C as a shortcut for Node Search, to
match the Firebug shortcut to do the same thing.

  • inspector/front-end/ElementsPanel.js:

(WebInspector.ElementsPanel.prototype.handleKeyEvent):

17:09 Changeset [51224] by eric@webkit.org

2009-11-19 Yaar Schnitman <yaar@chromium.org>

Reviewed by Darin Fisher.

Make chromium/webkit/glue/FormFieldValues use the WebKit API

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

  • public/WebElement.h:
  • public/WebFormElement.h:
  • public/WebInputElement.h: (WebKit::WebInputElement::):
  • public/WebNode.h:
  • src/AssertMatchingEnums.cpp:
  • src/DOMUtilitiesPrivate.cpp: (WebKit::nameOfInputElement):
  • src/DOMUtilitiesPrivate.h:
  • src/EditorClientImpl.cpp: (WebKit::EditorClientImpl::autofill): (WebKit::EditorClientImpl::doAutofill):
  • src/WebFormElement.cpp: (WebKit::WebFormElement::name): (WebKit::WebFormElement::method): (WebKit::WebFormElement::getNamedElements): (WebKit::WebFormElement::getInputElements):
  • src/WebInputElement.cpp: (WebKit::WebInputElement::isEnabledFormControl): (WebKit::WebInputElement::inputType): (WebKit::WebInputElement::formControlType): (WebKit::WebInputElement::value): (WebKit::WebInputElement::dispatchFormControlChangeEvent): (WebKit::WebInputElement::setSelectionRange): (WebKit::WebInputElement::name): (WebKit::WebInputElement::nameForAutofill):
  • src/WebNode.cpp: (WebKit::WebNode::frame):
16:55 Changeset [51223] by eric@webkit.org

2009-11-19 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

Move MultiCommandTool and Command into a separate file and add some basic unit tests
https://bugs.webkit.org/show_bug.cgi?id=31695

  • Scripts/bugzilla-tool:
  • Scripts/modules/multicommandtool.py: Added.
  • Scripts/modules/multicommandtool_unittest.py: Added.
  • Scripts/run-webkit-unittests:
16:53 Changeset [51222] by ap@apple.com

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=31690
Make SocketStreamHandleCFNet work on Windows

  • WebCore.vcproj/WebCore.vcproj: Added LoaderRunLoopCF.
  • platform/network/ResourceHandle.h: Removed loaderRunLoop().
  • platform/network/cf/LoaderRunLoopCF.cpp: Added. (WebCore::emptyPerform): (WebCore::runLoaderThread): (WebCore::loaderRunLoop):
  • platform/network/cf/LoaderRunLoopCF.h: Added. Moved the run loop that we use for CFNetwork from ResourceHandle to its own file, because it's needed for more than just resource loading.
  • platform/network/cf/ResourceHandleCFNet.cpp: Use loaderRunLoop() from its new location.
  • platform/network/cf/DNSCFNet.cpp: (WebCore::DNSResolveQueue::resolve): Ditto.
  • platform/network/cf/SocketStreamHandle.h: Added static callbacks for forwarding events to main thread.
  • platform/network/cf/SocketStreamHandleCFNet.cpp: (WebCore::SocketStreamHandle::SocketStreamHandle): Use loaderRunLoop() on Windows instead of inoperable main run loop.

(WebCore::MainThreadEventCallbackInfo::MainThreadEventCallbackInfo):
(WebCore::SocketStreamHandle::readStreamCallback):
(WebCore::SocketStreamHandle::writeStreamCallback):
(WebCore::SocketStreamHandle::readStreamCallbackMainThread):
(WebCore::SocketStreamHandle::writeStreamCallbackMainThread):
Forward stream events to main thread on Windows.

16:47 Changeset [51221] by eric@webkit.org

2009-11-19 Avi Drissman <avi@chromium.org>

Reviewed by Darin Adler.

Quick style fix.
https://bugs.webkit.org/show_bug.cgi?id=27777

  • platform/graphics/cg/PDFDocumentImage.cpp:
16:30 Changeset [51220] by bweinstein@apple.com

Update Windows expected pixel results, and remove the RenderTree results, because they are the same as the Mac.

16:25 Changeset [51219] by bdakin@apple.com

Build fix. No review needed.

  • platform/chromium/ScrollbarThemeChromium.cpp:

(WebCore::ScrollbarThemeChromium::paintTickmarks):

  • rendering/RenderMediaControlsChromium.cpp:

(WebCore::paintMediaButton):

16:02 Changeset [51218] by bdakin@apple.com

Build fix. No review needed.

  • rendering/RenderThemeChromiumSkia.cpp:

(WebCore::RenderThemeChromiumSkia::paintCheckbox):
(WebCore::RenderThemeChromiumSkia::paintRadio):
(WebCore::RenderThemeChromiumSkia::paintSearchFieldCancelButton):
(WebCore::RenderThemeChromiumSkia::paintSearchFieldResultsDecoration):
(WebCore::RenderThemeChromiumSkia::paintSearchFieldResultsButton):

15:58 Changeset [51217] by bdakin@apple.com

Build fix. No review needed.

  • platform/graphics/qt/StillImageQt.cpp:

(WebCore::StillImage::draw):

  • platform/graphics/qt/StillImageQt.h:
15:53 Changeset [51216] by bdakin@apple.com

Build fix. No review needed.

  • rendering/RenderThemeWin.cpp:

(WebCore::RenderThemeWin::paintSearchFieldCancelButton):
(WebCore::RenderThemeWin::paintSearchFieldResultsDecoration):
(WebCore::RenderThemeWin::paintSearchFieldResultsButton):

15:50 Changeset [51215] by bdakin@apple.com

Build fix. No review needed.

  • plugins/PluginView.cpp:

(WebCore::PluginView::paintMissingPluginIcon):

15:46 Changeset [51214] by bdakin@apple.com

Build fix. No review needed.

  • platform/graphics/win/ImageCGWin.cpp:

(WebCore::BitmapImage::getHBITMAPOfSize):
(WebCore::BitmapImage::drawFrameMatchingSourceSize):

15:38 Changeset [51213] by eric@webkit.org

2009-11-19 Eric Seidel <eric@webkit.org>

No review, just adding a FIXME.

Split out command parsing and help printing from BugzillaTool
https://bugs.webkit.org/show_bug.cgi?id=31688

  • Scripts/bugzilla-tool: Add an extra comment about current design failures.
15:36 Changeset [51212] by bdakin@apple.com

WebCore: Fix for https://bugs.webkit.org/show_bug.cgi?id=31321 Make -webkit-
color-correction work with untagged images

Reviewed by Darin Adler.

Image's draw(), drawPattern(), and fillWithSolidColor() functions
now take a ColorSpace. A bunch of classes inherit from Image, so
draw() and drawPattern() functions there must also take a
ColorSpace.

  • platform/graphics/Image.cpp:

(WebCore::Image::fillWithSolidColor):
(WebCore::Image::drawTiled):

  • platform/graphics/Image.h:
  • platform/graphics/BitmapImage.h:
  • platform/graphics/GeneratedImage.cpp:

(WebCore::GeneratedImage::draw):
(WebCore::GeneratedImage::drawPattern):

  • platform/graphics/GeneratedImage.h:
  • platform/graphics/cg/PDFDocumentImage.cpp:

(WebCore::PDFDocumentImage::draw):

  • platform/graphics/cg/PDFDocumentImage.h:
  • svg/graphics/SVGImage.cpp:

(WebCore::SVGImage::draw):

  • svg/graphics/SVGImage.h:

All of the drawImage() functions and drawTiled() functions of
GraphicsContext now take a ColorSpace.

  • platform/graphics/GraphicsContext.cpp:

(WebCore::GraphicsContext::drawImage):
(WebCore::GraphicsContext::drawTiledImage):

  • platform/graphics/GraphicsContext.h:

This is where the actual work is done.

  • platform/graphics/cg/ImageCG.cpp:

(WebCore::imageWithColorSpace): New static function that returns a
copy of the given image in the given ColorSpace *IF* we should use
the given ColorSpace. We will ignore the color space if the image
is tagged and return 0.
(WebCore::BitmapImage::draw): Now takes a ColorSpace and uses it to
call imageWithColorSpace().
(WebCore::Image::drawPattern): Same.

Attempt to keep all ports building.

  • platform/graphics/cairo/ImageCairo.cpp:

(WebCore::BitmapImage::draw):
(WebCore::Image::drawPattern):

  • platform/graphics/haiku/ImageHaiku.cpp:

(WebCore::BitmapImage::draw):
(WebCore::Image::drawPattern):

  • platform/graphics/qt/ImageQt.cpp:

(WebCore::Image::drawPattern):
(WebCore::BitmapImage::draw):

  • platform/graphics/skia/BitmapImageSingleFrameSkia.h:
  • platform/graphics/skia/ImageSkia.cpp:

(WebCore::Image::drawPattern):
(WebCore::BitmapImage::draw):
(WebCore::BitmapImageSingleFrameSkia::draw):

  • platform/graphics/wx/ImageWx.cpp:

(WebCore::BitmapImage::draw):
(WebCore::BitmapImage::drawPattern):

Callers of GraphicsContext's drawImage() and drawTiled() functions
now need to send a ColorSpace.

  • html/HTMLCanvasElement.cpp:

(WebCore::HTMLCanvasElement::paint):

  • html/canvas/CanvasRenderingContext2D.cpp:

(WebCore::CanvasRenderingContext2D::drawImage):
(WebCore::CanvasRenderingContext2D::drawImageFromRect):

  • platform/ScrollView.cpp:

(WebCore::ScrollView::wheelEvent):

  • platform/graphics/filters/FEColorMatrix.cpp:

(WebCore::FEColorMatrix::apply):

  • platform/graphics/filters/FEComposite.cpp:

(WebCore::FEComposite::apply):

  • platform/graphics/filters/SourceGraphic.cpp:

(WebCore::SourceGraphic::apply):

  • platform/mac/ScrollbarThemeMac.mm:

(WebCore::ScrollbarThemeMac::paint):

  • rendering/RenderBoxModelObject.cpp:

(WebCore::RenderBoxModelObject::paintFillLayerExtended):
(WebCore::RenderBoxModelObject::paintNinePieceImage):

  • rendering/RenderImage.cpp:

(WebCore::RenderImage::paintReplaced):

  • rendering/RenderLayer.cpp:

(WebCore::RenderLayer::paintResizer):

  • rendering/RenderListMarker.cpp:

(WebCore::RenderListMarker::paint):

  • rendering/RenderSVGImage.cpp:

(WebCore::RenderSVGImage::paint):

  • rendering/RenderWidget.cpp:

(WebCore::RenderWidget::paint):

  • svg/graphics/SVGPaintServerPattern.cpp:

(WebCore::SVGPaintServerPattern::setup):

  • svg/graphics/SVGResourceFilter.cpp:

(WebCore::SVGResourceFilter::applyFilter):

  • svg/graphics/filters/SVGFEMerge.cpp:

(WebCore::FEMerge::apply):

  • svg/graphics/filters/SVGFEOffset.cpp:

(WebCore::FEOffset::apply):

  • svg/graphics/filters/SVGFETile.cpp:

(WebCore::FETile::apply):

LayoutTests: Tests for https://bugs.webkit.org/show_bug.cgi?id=31321 Make
-webkit-color-correction work with untagged images

Reviewed by Darin Adler.

  • fast/css/color-correction-on-background-image.html: Added.
  • fast/css/color-correction-untagged-images.html: Added.
  • fast/css/resources/purple-noprofile.png: Added.
  • platform/mac/fast/css/color-correction-on-background-image-expected.checksum: Added.
  • platform/mac/fast/css/color-correction-on-background-image-expected.png: Added.
  • platform/mac/fast/css/color-correction-on-background-image-expected.txt: Added.
  • platform/mac/fast/css/color-correction-untagged-images-expected.checksum: Added.
  • platform/mac/fast/css/color-correction-untagged-images-expected.png: Added.
  • platform/mac/fast/css/color-correction-untagged-images-expected.txt: Added.
15:32 Changeset [51211] by dimich@chromium.org

isMainThread() on Chromium (Mac and Linux) is so slow it timeouts LayoutTests..
https://bugs.webkit.org/show_bug.cgi?id=31693

Reviewed by David Levin.

  • wtf/ThreadingPthreads.cpp:

(WTF::initializeThreading): grab and use the pthread_t of the main thread instead of ThreadIdentifier.
(WTF::isMainThread): Ditto.

15:30 Changeset [51210] by michelangelo@webkit.org

2009-11-19 Michelangelo De Simone <michelangelo@webkit.org>

Fix for <https://bugs.webkit.org/show_bug.cgi?id=27959>
Files missed when landing patch.

  • platform/qt/Skipped:
  • platform/win/Skipped:
15:28 Changeset [51209] by eric@webkit.org

2009-11-19 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

Split out command parsing and help printing from BugzillaTool
https://bugs.webkit.org/show_bug.cgi?id=31688

  • Scripts/bugzilla-tool:
    • Add new MultiCommandTool class to contain option parsing and help printing logic.
    • Rename private methods to use _ pattern.
    • MultiCommandTool has two abstract methods should_show_command_help and should_execute_command. -
15:21 Changeset [51208] by eric@webkit.org

2009-11-19 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

Re-factor help printing to use modern python idioms
https://bugs.webkit.org/show_bug.cgi?id=31685

  • Scripts/bugzilla-tool:
15:12 Changeset [51207] by eric@webkit.org

2009-11-19 Avi Drissman <avi@chromium.org>

Reviewed by Darin Adler.

Properly create a CGImageRef on non-PLATFORM(MAC).
https://bugs.webkit.org/show_bug.cgi?id=27777

  • platform/graphics/cg/ImageSourceCG.cpp: (WebCore::sharedBufferGetBytesAtPosition): (WebCore::sharedBufferRelease): (WebCore::ImageSource::setData):
  • platform/graphics/cg/ImageSourceCG.h:
  • platform/graphics/cg/PDFDocumentImage.cpp: (WebCore::PDFDocumentImage::dataChanged):
14:16 Changeset [51206] by ap@apple.com

Trying to enable WebSocket tests on Tiger again, as the bot seems to behave now.

  • platform/mac-tiger/Skipped:
14:04 Changeset [51205] by oliver@apple.com

-webkit-user-drag: element Drag images include overflow decorations from incorrect nodes
https://bugs.webkit.org/show_bug.cgi?id=31656

Reviewed Dave Hyatt

Make sure we're a child of the painting root before drawing overflow features.

13:29 Changeset [51204] by dimich@chromium.org

Not reviewed, attempt to fix Chromium build.

  • rendering/RenderMediaControlsChromium.cpp:

(WebCore::RenderMediaControlsChromium::paintMediaControlsPart): Add new enum values for Show/HideClosedCaptionButton to make it compile.

13:02 Changeset [51203] by eric@webkit.org

2009-11-19 Eric Seidel <eric@webkit.org>

Reviewed by Darin Adler.

commit-queue empty queue logs twice
https://bugs.webkit.org/show_bug.cgi?id=31679

  • Scripts/bugzilla-tool:
12:28 Changeset [51202] by eric@webkit.org

2009-11-19 Eric Seidel <eric@webkit.org>

Reviewed by Darin Adler.

bugzilla-tool's reviewer/committer rejection message should be clearer
https://bugs.webkit.org/show_bug.cgi?id=31126

Add more explanatory prose to bugzilla-tool's flag permission rejection message.

  • Scripts/modules/bugzilla.py:
12:25 Changeset [51201] by eric@webkit.org

2009-11-19 Rahul Kuchhal <kuchhal@chromium.org>

Reviewed by Darin Adler.

Do not assert when a high number is used for roman numerals in lists.
https://bugs.webkit.org/show_bug.cgi?id=31652

  • fast/lists/ol-start-roman.html: Added.
  • platform/mac/fast/lists/ol-start-roman-expected.txt: Added.

2009-11-19 Rahul Kuchhal <kuchhal@chromium.org>

Reviewed by Darin Adler.

Do not assert when a high number is used for roman numerals in lists.
https://bugs.webkit.org/show_bug.cgi?id=31652

Test: fast/lists/ol-start-roman.html

  • rendering/RenderListMarker.cpp: (WebCore::toRoman): Increase the char array size.
12:14 Changeset [51200] by kov@webkit.org

Rubber-stamped by Oliver Hunt.

Add translatable strings to cover validationMessage, after
r51172. Already covered by existing test.

  • platform/gtk/LocalizedStringsGtk.cpp: (WebCore::validationMessageValueMissingText): (WebCore::validationMessageTypeMismatchText): (WebCore::validationMessagePatternMismatchText): (WebCore::validationMessageTooLongText): (WebCore::validationMessageRangeUnderflowText): (WebCore::validationMessageRangeOverflowText): (WebCore::validationMessageStepMismatchText):
12:12 Changeset [51199] by eric@webkit.org

2009-11-19 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Darin Adler.

Remove HAVE(STRING_H) guard from JavaScriptCore
https://bugs.webkit.org/show_bug.cgi?id=31668

  • config.h:
  • runtime/UString.cpp:
11:59 Changeset [51198] by dumi@chromium.org

Fixing a bug in MessageQueue::removeIf() that leads to an
assertion failure.

Reviewed by Dmitry Titov.

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

  • wtf/MessageQueue.h:

(WTF::MessageQueue::removeIf):

11:54 Changeset [51197] by eric@webkit.org

2009-11-19 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Darin Adler.

Remove HAVE(FLOAT_H) guard
https://bugs.webkit.org/show_bug.cgi?id=31661

JavaScriptCore has a dependency on float.h, there is
no need to guard float.h.

  • runtime/DatePrototype.cpp: Remove include directive for float.h as it is included in MathExtras.h already.
  • runtime/Operations.cpp: Ditto.
  • runtime/UString.cpp: Ditto.
  • wtf/dtoa.cpp: Ditto.
  • wtf/MathExtras.h: Remove HAVE(FLOAT_H) guard.
  • wtf/Platform.h: Ditto.

2009-11-19 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Darin Adler.

Remove HAVE(FLOAT_H) guard
https://bugs.webkit.org/show_bug.cgi?id=31661

WebCore has a dependency on float.h, there is
no need to guard float.h.

No new tests as there is no functional change.

  • html/HTMLInputElement.cpp: Remove include directive for float.h as it is included in MathExtras.h already.
11:41 Changeset [51196] by eric@webkit.org

2009-11-19 Eric Z. Ayers <zundel@google.com>

Reviewed by Pavel Feldman.

Updated timeline paint test to ignore width and height. GTK+ testing
uses a different top level window size than other platforms.

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

  • inspector/timeline-paint-expected.txt:
  • inspector/timeline-test.js:
  • platform/win/Skipped:

2009-11-19 Eric Z. Ayers <zundel@google.com>

Reviewed by Pavel Feldman.

Forces a WM_PAINT event on calling layoutTestController.display()
in order to enable the timeline-paint.html test on Windows.
::UpdateWindow() does not force an event becaue the window is
not visible.

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

  • DumpRenderTree/win/DumpRenderTree.cpp: (displayWebView):
11:28 Changeset [51195] by eric@webkit.org

2009-11-19 Chris Fleizach <cfleizach@apple.com>

Reviewed by Darin Adler.

WAI-ARIA roles not supported on image map <area>
https://bugs.webkit.org/show_bug.cgi?id=31525

  • accessibility/aria-used-on-image-maps-expected.txt: Added.
  • accessibility/aria-used-on-image-maps.html: Added.
  • platform/gtk/Skipped:
  • platform/win/Skipped:
11:03 Changeset [51194] by simon.fraser@apple.com

2009-11-19 Simon Fraser <simon.fraser@apple.com>

Reviewed by Pavel Feldman.

Instrumentation should account for painting in compositing layers
https://bugs.webkit.org/show_bug.cgi?id=31674

Add calls to InspectorTimelineAgent for painting into compositing layers,
which is not accounted for in FrameView::paintContents().

  • rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::paintIntoLayer): Avoid fetching the FrameView multiple times.

(WebCore::inspectorTimelineAgent):
(WebCore::RenderLayerBacking::paintContents):

10:21 Changeset [51193] by michelangelo@webkit.org

2009-11-19 Michelangelo De Simone <michelangelo@webkit.org>

No review needed.

Added myself to committers list.

  • Scripts/modules/committers.py:
10:16 Changeset [51192] by eric@webkit.org

2009-11-19 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Bugzilla-tool command classes should match command names
https://bugs.webkit.org/show_bug.cgi?id=31666

I renamed all the commands except CommitMessageForCurrentDiff because
the new name would conflict with an existing class.

  • Scripts/bugzilla-tool:
10:14 Changeset [51191] by eric.carlson@apple.com

2009-11-19 Eric Carlson <eric.carlson@apple.com>

Reviewed by Dan Bernstein.

<rdar://problem/7035231>
Support closed caption in <video> element

Test: media/media-captions.html

  • WebCore.base.exp:
  • accessibility/AccessibilityMediaControls.cpp: (WebCore::AccessibilityMediaControl::controlTypeName):

Define ShowClosedCaptionsButton and HideClosedCaptionsButton.

  • css/CSSPrimitiveValueMappings.h: (WebCore::CSSPrimitiveValue::CSSPrimitiveValue):

Add MediaToggleClosedCaptionsButtonPart.

  • css/CSSSelector.cpp: (WebCore::CSSSelector::extractPseudoType):

Define and match mediaControlsToggleClosedCaptionsButton.

  • css/CSSSelector.h: (WebCore::CSSSelector::):
  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::SelectorChecker::checkOneSelector):

Define and match PseudoMediaControlsToggleClosedCaptions.

  • css/CSSValueKeywords.in:

Define and use media-toggle-closed-captions-button.

  • css/mediaControls.css:
  • css/mediaControlsQuickTime.css:

Add webkit-media-controls-toggle-closed-captions-button.

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::HTMLMediaElement):

Initialize m_closedCaptionsVisible.

(WebCore::HTMLMediaElement::loadInternal):

Set m_closedCaptionsVisible to false.

(WebCore::HTMLMediaElement::hasClosedCaptions):
(WebCore::HTMLMediaElement::closedCaptionsVisible):
(WebCore::HTMLMediaElement::setClosedCaptionsVisible):

New, captions internal methods.

(WebCore::HTMLMediaElement::setWebkitClosedCaptionsVisible):
(WebCore::HTMLMediaElement::webkitClosedCaptionsVisible):
(WebCore::HTMLMediaElement::webkitHasClosedCaptions):

New, captions DOM API.

  • html/HTMLMediaElement.h:
  • html/HTMLMediaElement.idl:

Declare methods needed for captions API.

  • platform/ThemeTypes.h:

Add MediaToggleClosedCaptionsButtonPart.

  • platform/android/LocalizedStringsAndroid.cpp: (WebCore::localizedMediaControlElementString): (WebCore::localizedMediaControlElementHelpText): (WebCore::localizedMediaTimeDescription):

Add empty implmentations.

  • platform/graphics/MediaPlayer.cpp: (WebCore::NullMediaPlayerPrivate::hasClosedCaptions): (WebCore::NullMediaPlayerPrivate::setClosedCaptionsVisible):

New, empty implementations of media engine closed caption functions.

(WebCore::MediaPlayer::hasClosedCaptions):
(WebCore::MediaPlayer::setClosedCaptionsVisible):

New, call media engine closed caption functions.

  • platform/graphics/MediaPlayer.h:
  • platform/graphics/MediaPlayerPrivate.h: (WebCore::MediaPlayerPrivateInterface::hasClosedCaptions): (WebCore::MediaPlayerPrivateInterface::setClosedCaptionsVisible):

Declare new media engine methods.

  • platform/graphics/mac/MediaPlayerPrivateQTKit.h:
  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::hasClosedCaptions): (WebCore::MediaPlayerPrivate::setClosedCaptionsVisible):

New, QTKit implementation of closed caption methods.

  • platform/gtk/LocalizedStringsGtk.cpp: (WebCore::localizedMediaControlElementString): (WebCore::localizedMediaControlElementHelpText):

Add help text for ShowClosedCaptionsButton and HideClosedCaptionsButton.

  • platform/mac/WebCoreSystemInterface.h:
  • platform/mac/WebCoreSystemInterface.mm:

Declare and initialize wkQTMovieHasClosedCaptions and wkQTMovieSetShowClosedCaptions.

  • rendering/MediaControlElements.cpp: (WebCore::MediaControlInputElement::MediaControlInputElement):

Deal with MEDIA_CONTROLS_TOGGLE_CLOSED_CAPTIONS_BUTTON.

(WebCore::MediaControlToggleClosedCaptionsButtonElement::MediaControlToggleClosedCaptionsButtonElement):
(WebCore::MediaControlToggleClosedCaptionsButtonElement::defaultEventHandler):
(WebCore::MediaControlToggleClosedCaptionsButtonElement::updateDisplayType):

New, implement the closed caption toggle button,

  • rendering/MediaControlElements.h:

Define MediaShowClosedCaptionsButton and MediaHideClosedCaptionsButton,
declare MediaControlToggleClosedCaptionsButtonElement.

  • rendering/RenderMedia.cpp: (WebCore::RenderMedia::styleDidChange): (WebCore::RenderMedia::createToggleClosedCaptionsButton): (WebCore::RenderMedia::createStatusDisplay): (WebCore::RenderMedia::updateControls): (WebCore::RenderMedia::forwardEvent):
  • rendering/RenderMedia.h:

Deal with m_toggleClosedCaptionsButton.

  • rendering/RenderTheme.cpp: (WebCore::RenderTheme::paint):

Deal with MediaToggleClosedCaptionsButtonPart.

  • rendering/RenderTheme.h:
  • rendering/RenderThemeMac.h:

Declare paintMediaToggleClosedCaptionsButton.

  • rendering/RenderThemeMac.mm: (WebCore::RenderThemeMac::paintMediaToggleClosedCaptionsButton):

New.

(WebCore::RenderThemeMac::shouldRenderMediaControlPart):

Don't render captions toggle button unless we are using the new theme, the
movie has captions, and the movie is in a <video> element since we currently
rely on QTKit to render the captions.

  • rendering/style/RenderStyleConstants.h:

Define MEDIA_CONTROLS_TOGGLE_CLOSED_CAPTIONS_BUTTON.

2009-11-19 Eric Carlson <eric.carlson@apple.com>

Reviewed by Dan Bernstein.

<rdar://problem/7035231>
Support closed caption in <video> element

  • media/media-captions-expected.txt: Added.
  • media/media-captions.html: Added.
  • media/content/counting-captioned.mov: Added.
  • platform/gtk/Skipped:
  • platform/mac-leopard/Skipped:
  • platform/mac-tiger/Skipped:
  • platform/win/Skipped:

Skipped on platforms that don't support closed captions yet.

2009-11-19 Eric Carlson <eric.carlson@apple.com>

Reviewed by Dan Bernstein.

<rdar://problem/7035231>
Support closed caption in <video> element

  • English.lproj/Localizable.strings:

Add strings for toggle captions button.

2009-11-19 Eric Carlson <eric.carlson@apple.com>

Reviewed by Dan Bernstein.

<rdar://problem/7035231>
Support closed caption in <video> element

  • WebCoreSupport/WebSystemInterface.m: (InitWebCoreSystemInterface):

Add QTMovieHasClosedCaptions and QTMovieSetShowClosedCaptions.

  • WebCoreSupport/WebViewFactory.mm: (-[WebViewFactory localizedMediaControlElementString:]): (-[WebViewFactory localizedMediaControlElementHelpText:]):

Add accessibility help strings for media controller closed caption button.

2009-11-19 Eric Carlson <eric.carlson@apple.com>

Reviewed by Dan Bernstein.

<rdar://problem/7035231>
Support closed caption in <video> element

  • WebCoreLocalizedStrings.cpp: (WebCore::localizedMediaControlElementString): (WebCore::localizedMediaControlElementHelpText):

Add accessibility help strings for media controller closed caption button.

2009-11-19 Eric Carlson <eric.carlson@apple.com>

Reviewed by Dan Bernstein.

<rdar://problem/7035231>
Support closed caption in <video> element

  • WebKitSystemInterface.h:

Add prototypes for WKQTMovieHasClosedCaptions and WKQTMovieSetShowClosedCaptions,
define WKMediaUIPartToggleClosedCaptionsButton.

10:10 Changeset [51190] by kevino@webkit.org

wx build fix. Add missing header.

09:43 Changeset [51189] by ap@apple.com

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=31634
Ignore realm for proxy protection spaces

Only affects WebSocket proxy authentication, cannot be tested in DRT.

  • platform/network/ProtectionSpace.cpp: (WebCore::operator==):
  • platform/network/ProtectionSpaceHash.h: (WebCore::ProtectionSpaceHash::hash):
09:42 Changeset [51188] by eric@webkit.org

2009-11-19 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Gustavo Noronha Silva.

Web Inspector: Implement "show inspector" in WebKit GTK
API and enable console tests.

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

  • platform/gtk/Skipped:

2009-11-19 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Gustavo Noronha Silva.

Web Inspector: Implement "show inspector" in WebKit GTK
API and enable console tests.

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

  • webkit/webkitwebinspector.cpp: (webkit_web_inspector_show):
  • webkit/webkitwebinspector.h:

2009-11-19 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Gustavo Noronha Silva.

Web Inspector: Implement "show inspector" in WebKit GTK
API and enable console tests.

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

  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (LayoutTestController::showWebInspector):
06:23 Changeset [51187] by kenneth@webkit.org

Minor refactoring + more documentation.

Reviewed by Simon Hausmann.

  • DumpRenderTree/qt/DumpRenderTree.cpp:

(WebCore::clearHistory):
(WebCore::DumpRenderTree::resetToConsistentStateBeforeTesting):

06:12 Changeset [51186] by eric@webkit.org

2009-11-19 Jocelyn Turcotte <jocelyn.turcotte@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Add instantiation tests for QWebInspector.

  • tests/qwebinspector/qwebinspector.pro: Added.
  • tests/qwebinspector/tst_qwebinspector.cpp: Added. (tst_QWebInspector::attachAndDestroy):
  • tests/tests.pro:
06:04 Changeset [51185] by eric@webkit.org

2009-11-19 Jocelyn Turcotte <jocelyn.turcotte@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Fix QWebInspector destruction problem.
https://bugs.webkit.org/show_bug.cgi?id=31664

  • Api/qwebpage.cpp: (QWebPage::~QWebPage):
05:47 Changeset [51184] by hausmann@webkit.org

[Qt] Normalize signal and slot signatures.

Patch by Olivier Goffart <ogoffart@trolltech.com> on 2009-11-19
Reviewed by Simon Hausmann.

WebCore:

  • platform/graphics/qt/MediaPlayerPrivatePhonon.cpp:

(WebCore::MediaPlayerPrivate::MediaPlayerPrivate):

WebKit/qt:

  • Api/qgraphicswebview.cpp:

(QGraphicsWebView::setPage):

  • Api/qwebview.cpp:

(QWebView::setPage):

  • WebCoreSupport/FrameLoaderClientQt.cpp:

(WebCore::FrameLoaderClientQt::setFrame):

  • docs/webkitsnippets/qtwebkit_qwebinspector_snippet.cpp:

(wrapInFunction):

  • tests/qwebframe/tst_qwebframe.cpp:
  • tests/qwebpage/tst_qwebpage.cpp:

(tst_QWebPage::modified):
(tst_QWebPage::database):

05:46 Changeset [51183] by hausmann@webkit.org

Build fix for 32-bit Sparc machines: these machines are big-endian.

Patch by Thiago Macieira <thiago.macieira@nokia.com> on 2009-11-19
Reviewed by Simon Hausmann.

  • wtf/Platform.h:
05:03 Changeset [51182] by eric@webkit.org

2009-11-19 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Pavel Feldman.

Store cookie domains in the WebInspector object

The cookie domains have been moved from StoragePanel into WebInspector.
Also, the document URLs are now passed inside the WebInspector.addResource()
payload rather than pushed directly from InspectorController.
https://bugs.webkit.org/show_bug.cgi?id=31627

  • inspector/InspectorController.cpp: (WebCore::InspectorController::populateScriptObjects): (WebCore::InspectorController::didFinishLoading):
  • inspector/InspectorFrontend.cpp:
  • inspector/InspectorFrontend.h:
  • inspector/InspectorResource.cpp: (WebCore::InspectorResource::createScriptObject): (WebCore::InspectorResource::updateScriptObject):
  • inspector/front-end/StoragePanel.js: (WebInspector.StoragePanel.prototype.reset): (WebInspector.StoragePanel.prototype.addCookieDomain):
  • inspector/front-end/inspector.js: (WebInspector.addResource): (WebInspector.addCookieDomain): (WebInspector.reset):
04:32 Changeset [51181] by hamaji@chromium.org

2009-11-19 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by Eric Seidel.

Some tests in fast/css-generated-content can be dumpAsText test
https://bugs.webkit.org/show_bug.cgi?id=31663

  • fast/css-generated-content/absolute-position-inside-inline-expected.txt: Added.
  • fast/css-generated-content/absolute-position-inside-inline.html:
  • fast/css-generated-content/hit-test-generated-content-expected.txt: Added.
  • fast/css-generated-content/hit-test-generated-content.html:
  • fast/css-generated-content/positioned-background-hit-test-crash-expected.txt: Added.
  • fast/css-generated-content/positioned-background-hit-test-crash.html:
  • fast/css-generated-content/spellingToolTip-assert-expected.txt: Added.
  • fast/css-generated-content/spellingToolTip-assert.html:
  • platform/mac/fast/css-generated-content/absolute-position-inside-inline-expected.checksum: Removed.
  • platform/mac/fast/css-generated-content/absolute-position-inside-inline-expected.png: Removed.
  • platform/mac/fast/css-generated-content/absolute-position-inside-inline-expected.txt: Removed.
  • platform/mac/fast/css-generated-content/hit-test-generated-content-expected.checksum: Removed.
  • platform/mac/fast/css-generated-content/hit-test-generated-content-expected.png: Removed.
  • platform/mac/fast/css-generated-content/hit-test-generated-content-expected.txt: Removed.
  • platform/mac/fast/css-generated-content/positioned-background-hit-test-crash-expected.checksum: Removed.
  • platform/mac/fast/css-generated-content/positioned-background-hit-test-crash-expected.png: Removed.
  • platform/mac/fast/css-generated-content/positioned-background-hit-test-crash-expected.txt: Removed.
  • platform/mac/fast/css-generated-content/spellingToolTip-assert-expected.checksum: Removed.
  • platform/mac/fast/css-generated-content/spellingToolTip-assert-expected.png: Removed.
  • platform/mac/fast/css-generated-content/spellingToolTip-assert-expected.txt: Removed.
  • platform/qt/fast/css-generated-content/absolute-position-inside-inline-expected.txt: Removed.
  • platform/qt/fast/css-generated-content/positioned-background-hit-test-crash-expected.txt: Removed.
  • platform/qt/fast/css-generated-content/spellingToolTip-assert-expected.txt: Removed.
02:41 Changeset [51180] by ukai@chromium.org

2009-11-19 Fumitoshi Ukai <ukai@chromium.org>

Reviewed by Eric Seidel.

Ignore websocket tests when --no-http is specified.
https://bugs.webkit.org/show_bug.cgi?id=31662

  • Scripts/run-webkit-tests:
01:34 Changeset [51179] by zoltan@webkit.org

Allow custom memory allocation control for the other part of platform directory in WebCore
https://bugs.webkit.org/show_bug.cgi?id=31585

Reviewed by Darin Adler.

Inherits the following classes from FastAllocBase because these are
instantiated by 'new':

class RegularExpression - instantiated at: WebCore/page/Frame.cpp:415
class TransformationMatrix - instantiated at: WebCore/rendering/TransformState.cpp:62
class Path - instantiated at: WebCore/html/HTMLAreaElement.cpp:73
class FontPlatformData - instantiated at: WebCore/platform/graphics/qt/FontCacheQt.cpp:188

Inherits the following classes from Noncopyable because these are
instantiated by 'new' and no need to be copyable:

class Cursors - instantiated at: WebCore/platform/qt/CursorQt.cpp:146
class NetworkStateNotifier - instantiated at: WebCore/platform/network/NetworkStateNotifier.cpp:37
struct CrossThreadResourceRequestData - instantiated at: WebCore/platform/network/ResourceRequestBase.cpp:71

class ImageDecoder - its child class is instantiated at: WebCore/platform/graphics/qt/ImageDecoderQt.cpp:46
class MediaPlayerPrivateInterface - its child class is instantiated at: WebCore/platform/graphics/qt/MediaPlayerPrivatePhonon.cpp:119

  • platform/graphics/MediaPlayerPrivate.h:
  • platform/graphics/Path.h:
  • platform/graphics/qt/FontPlatformData.h:
  • platform/graphics/transforms/TransformationMatrix.h:
  • platform/image-decoders/ImageDecoder.h:
  • platform/network/NetworkStateNotifier.h:
  • platform/network/ResourceRequestBase.h:
  • platform/qt/CursorQt.cpp:
  • platform/text/RegularExpression.h:
00:15 Changeset [51178] by mitz@apple.com

Update the test fast/overflow/004.html to make it match the test
description and expected results.
https://bugs.webkit.org/show_bug.cgi?id=31455

Patch by Kinuko Yasuda <kinuko@google.com> on 2009-11-19
Reviewed by Dan Bernstein.

  • fast/overflow/004.html:
  • platform/mac/fast/overflow/004-expected.checksum:
  • platform/mac/fast/overflow/004-expected.png:
  • platform/mac/fast/overflow/004-expected.txt:

11/18/09:

23:18 Changeset [51177] by laszlo.1.gombos@nokia.com

2009-11-18 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Unreviewed.

[Qt] Build fix for QtWebKit after r51172.

  • platform/qt/Localizations.cpp: (WebCore::validationMessageValueMissingText): (WebCore::validationMessageTypeMismatchText): (WebCore::validationMessagePatternMismatchText): (WebCore::validationMessageTooLongText): (WebCore::validationMessageRangeUnderflowText): (WebCore::validationMessageRangeOverflowText): (WebCore::validationMessageStepMismatchText):
22:18 Changeset [51176] by eric@webkit.org

2009-11-18 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Normalize ' and " in bugzilla-tool
https://bugs.webkit.org/show_bug.cgi?id=31655

We decided " is better than ' and we should be consistent.

  • Scripts/bugzilla-tool:
22:17 Changeset [51175] by mitz@apple.com

WebCore: Fixed <rdar://problem/7398987> Assertion failure in
RenderLayer::updateClipRects when a plug-in’s enclosing layer is
enclosed by a transformed layer

Reviewed by Simon Fraser.

Tests: fast/layers/clip-rects-transformed-2.html

fast/layers/clip-rects-transformed.html

  • rendering/RenderLayer.cpp:

(WebCore::RenderLayer::clippingRoot): Added. Finds the nearest ancestor
that is either transformed or composited. During painting and hit
testing, when a transformed or composited layer is hit, it becomes the
new root layer, which ultimately gets passed to calculateRects().
(WebCore::RenderLayer::childrenClipRect): Pass the clipping root as the
root layer to calculateRects(), and return the bounding box of the
transformed rectangle.
(WebCore::RenderLayer::selfClipRect): Ditto.

  • rendering/RenderLayer.h:

LayoutTests: Test for <rdar://problem/7398987> Assertion failure in
RenderLayer::updateClipRects when a plug-in’s enclosing layer is
enclosed by a transformed layer

Reviewed by Simon Fraser.

  • fast/layers/clip-rects-transformed-2-expected.txt: Added.
  • fast/layers/clip-rects-transformed-2.html: Added.
  • fast/layers/clip-rects-transformed-expected.txt: Added.
  • fast/layers/clip-rects-transformed.html: Added.
22:03 Changeset [51174] by laszlo.1.gombos@nokia.com

2009-11-18 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Remove support for Qt v4.3 or older versions
https://bugs.webkit.org/show_bug.cgi?id=29469

2009-11-18 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Remove support for Qt v4.3 or older versions
https://bugs.webkit.org/show_bug.cgi?id=29469

  • Api/qcookiejar.cpp: Removed.
  • Api/qcookiejar.h: Removed.
  • Api/qgraphicswebview.cpp: (QGraphicsWebView::event):
  • Api/qwebframe.cpp: (QWebFrame::load):
  • Api/qwebframe.h:
  • Api/qwebkitglobal.h:
  • Api/qwebnetworkinterface.cpp: Removed.
  • Api/qwebnetworkinterface.h: Removed.
  • Api/qwebnetworkinterface_p.h: Removed.
  • Api/qwebpage.cpp: (QWebPagePrivate::QWebPagePrivate): (QWebPagePrivate::acceptNavigationRequest): (QWebPage::acceptNavigationRequest): (QWebPage::action): (QWebPage::userAgentForUrl):
  • Api/qwebpage.h:
  • Api/qwebpage_p.h:
  • Api/qwebview.cpp: (QWebView::load): (QWebView::event):
  • Api/qwebview.h:
  • QtLauncher/main.cpp: (MainWindow::print): (MainWindow::setupUI): (main):
  • WebCoreSupport/FrameLoaderClientQt.cpp: (WebCore::FrameLoaderClientQt::download): (WebCore::FrameLoaderClientQt::dispatchDecidePolicyForNewWindowAction): (WebCore::FrameLoaderClientQt::dispatchDecidePolicyForNavigationAction): (WebCore::FrameLoaderClientQt::startDownload): (WebCore::FrameLoaderClientQt::createPlugin):

2009-11-18 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Remove support for Qt v4.3 or older versions
https://bugs.webkit.org/show_bug.cgi?id=29469

No new tests as there is no change if compiled with
Qt v4.4 or later version.

  • WebCore.pro:
  • dom/XMLTokenizerQt.cpp: (WebCore::EntityResolver::resolveUndeclaredEntity): (WebCore::XMLTokenizer::XMLTokenizer): (WebCore::XMLTokenizer::~XMLTokenizer): (WebCore::XMLTokenizer::doWrite): (WebCore::XMLTokenizer::startDocument): (WebCore::XMLTokenizer::parseDtd):
  • platform/graphics/qt/FontQt.cpp:
  • platform/graphics/qt/FontQt43.cpp: Removed.
  • platform/graphics/qt/ImageQt.cpp:
  • platform/network/ResourceHandleInternal.h:
  • platform/network/qt/QNetworkReplyHandler.cpp:
  • platform/network/qt/QNetworkReplyHandler.h:
  • platform/network/qt/ResourceHandleQt.cpp: (WebCore::ResourceHandle::start): (WebCore::ResourceHandle::cancel): (WebCore::ResourceHandle::loadResourceSynchronously): (WebCore::ResourceHandle::setDefersLoading):
  • platform/network/qt/ResourceRequest.h:
  • platform/network/qt/ResourceRequestQt.cpp:
  • platform/qt/ClipboardQt.cpp: (WebCore::ClipboardQt::clearData):
  • platform/qt/CookieJarQt.cpp: (WebCore::cookieJar): (WebCore::setCookies): (WebCore::cookies): (WebCore::cookiesEnabled):
  • platform/qt/PlatformKeyboardEventQt.cpp: (WebCore::keyIdentifierForQtKeyCode): (WebCore::windowsKeyCodeForKeyEvent):
  • platform/text/qt/TextBoundaries.cpp:
  • platform/text/qt/TextBreakIteratorQt.cpp:
  • plugins/qt/PluginDataQt.cpp: (WebCore::PluginData::initPlugins):

2009-11-18 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Remove support for Qt v4.3 or older versions
https://bugs.webkit.org/show_bug.cgi?id=29469

  • DumpRenderTree/qt/DumpRenderTree.pro:
  • DumpRenderTree/qt/ImageDiff.pro:
21:47 Changeset [51173] by hamaji@chromium.org

2009-11-18 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by Eric Seidel.

GitTest.test_create_binary_patch fails if /tmp is symlink
https://bugs.webkit.org/show_bug.cgi?id=31536

  • Scripts/modules/scm_unittest.py:
21:38 Changeset [51172] by tkent@chromium.org

2009-11-18 Michelangelo De Simone <micdesim@gmail.com>

Reviewed by Darin Adler.

Fix for <https://bugs.webkit.org/show_bug.cgi?id=27959>.
Support for validationMessage attribute, as per HTML5 specs.

  • fast/dom/domListEnumeration-expected.txt: updated with new property
  • fast/dom/script-tests/domListEnumeration.js: ditto
  • fast/forms/script-tests/validationMessage.js: Added.
  • fast/forms/validationMessage-expected.txt: Added.
  • fast/forms/validationMessage.html: Added.
  • platform/gtk/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:

2009-11-18 Michelangelo De Simone <micdesim@gmail.com>

Reviewed by Darin Adler.

Fix for <https://bugs.webkit.org/show_bug.cgi?id=27959>.
Support for validationMessage attribute, as per HTML5 specs.

Test: fast/forms/validationMessage.html

  • html/HTMLButtonElement.idl: validationMessage attribute
  • html/HTMLFieldSetElement.idl: validationMessage attribute
  • html/HTMLFormControlElement.cpp:
  • html/HTMLFormControlElement.h:
  • html/HTMLInputElement.idl: validationMessage attribute
  • html/HTMLSelectElement.idl: validationMessage attribute
  • html/HTMLTextAreaElement.idl: validationMessage attribute
  • html/ValidityState.cpp: new method to retrieve pertinent localized text
  • html/ValidityState.h:
  • page/mac/WebCoreViewFactory.h:
  • platform/LocalizedStrings.h:
  • platform/android/LocalizedStringsAndroid.cpp:
  • platform/gtk/LocalizedStringsGtk.cpp:
  • platform/haiku/LocalizedStringsHaiku.cpp:
  • platform/mac/LocalizedStringsMac.mm:
  • platform/wx/LocalizedStringsWx.cpp:

2009-11-18 Michelangelo De Simone <micdesim@gmail.com>

Reviewed by Darin Adler.

Fix for <https://bugs.webkit.org/show_bug.cgi?id=27959>.
Support for validationMessage attribute, as per HTML5 specs.

  • English.lproj/Localizable.strings: new localizable strings

2009-11-18 Michelangelo De Simone <micdesim@gmail.com>

Reviewed by Darin Adler.

Fix for <https://bugs.webkit.org/show_bug.cgi?id=27959>.
Support for validationMessage attribute, as per HTML5 specs.

  • src/LocalizedStrings.cpp: (WebCore::validationMessageValueMissingText): (WebCore::validationMessageTypeMismatchText): (WebCore::validationMessagePatternMismatchText): (WebCore::validationMessageTooLongText): (WebCore::validationMessageRangeUnderflowText): (WebCore::validationMessageRangeOverflowText): (WebCore::validationMessageStepMismatchText):

2009-11-18 Michelangelo De Simone <micdesim@gmail.com>

Reviewed by Darin Adler.

Fix for <https://bugs.webkit.org/show_bug.cgi?id=27959>.
Support for validationMessage attribute, as per HTML5 specs.

  • WebCoreSupport/WebViewFactory.mm: (-[WebViewFactory validationMessageValueMissingText]): (-[WebViewFactory validationMessageTypeMismatchText]): (-[WebViewFactory validationMessagePatternMismatchText]): (-[WebViewFactory validationMessageTooLongText]): (-[WebViewFactory validationMessageRangeUnderflowText]): (-[WebViewFactory validationMessageRangeOverflowText]): (-[WebViewFactory validationMessageStepMismatchText]):

2009-11-18 Michelangelo De Simone <micdesim@gmail.com>

Reviewed by Darin Adler.

Fix for <https://bugs.webkit.org/show_bug.cgi?id=27959>.
Support for validationMessage attribute, as per HTML5 specs.

  • WebCoreLocalizedStrings.cpp: (WebCore::validationMessageValueMissingText): (WebCore::validationMessageTypeMismatchText): (WebCore::validationMessagePatternMismatchText): (WebCore::validationMessageTooLongText): (WebCore::validationMessageRangeUnderflowText): (WebCore::validationMessageRangeOverflowText): (WebCore::validationMessageStepMismatchText):
21:09 Changeset [51171] by laszlo.1.gombos@nokia.com

2009-11-18 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Unreviewed.

Fix a typo. Remove the trailing semicolon
from an include directive after r51137.

No new tests as there is no functional change.

  • platform/qt/QWebPageClient.h:
20:52 Changeset [51170] by laszlo.1.gombos@nokia.com

2009-11-18 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Unreviewed.

[Qt] Build fix for QtWebKit after r51159.

Include float.h for DBL_MANT_DIG. Not sure why this
is not a problem for other ports.

  • html/HTMLInputElement.cpp:
19:15 Changeset [51169] by rolandsteiner@chromium.org

WebCore: Bug 31574 - Crashing bug when removing <ruby> element
(https://bugs.webkit.org/show_bug.cgi?id=31574)

Reviewed by Darin Adler.

Cause of the bug:
1.) RenderBlock::destroy() of the RenderRubyRun called destroyLeftoverChildren()
2.) that called destroy() of the RenderRubyBase(), which in RenderObject::destroy() calls remove()
3.) remove() is being redirected as parent()->removeChild() in RenderObject.h
4.) this triggers the special handling of child removal in RenderRubyRun that

causes it to destroy itself

5.) On returning from all this the renderer crashes when accessing a member

or virtual function on this now illegal object.

I therefore added a flag that tracks if the ruby run is being destroyed.
If so, avoid doing the special handling in removeChild that caused this.
It's not the most elegant solution, but the easiest to implement without
touching unrelated code. Also, it's self-documenting.

Test: fast/ruby/ruby-remove.html

  • rendering/RenderRubyRun.cpp:

(WebCore::RenderRubyRun::RenderRubyRun):
(WebCore::RenderRubyRun::destroy):
(WebCore::RenderRubyRun::removeChild):

  • rendering/RenderRubyRun.h:

LayoutTests: Bug 31574 - Crashing bug when removing <ruby> element
(https://bugs.webkit.org/show_bug.cgi?id=31574)

Reviewed by Darin Adler.

Layout test to verify it no longer crashes when the <ruby> element
is being removed.

  • fast/ruby/ruby-remove-expected.txt: Added.
  • fast/ruby/ruby-remove.html: Added.
19:12 Changeset [51168] by tkent@chromium.org

2009-11-18 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Adler.

Move UString::from(double) implementation to new
WTF::doubleToStringInJavaScriptFormat(), and expose it because WebCore
code will use it.
https://bugs.webkit.org/show_bug.cgi?id=31330

  • Introduce new function createRep(const char*, unsigned) and UString::UString(const char*, unsigned) to reduce 2 calls to strlen().
  • Fix a bug that dtoa() doesn't update *rve if the input value is NaN or Infinity.

No new tests because this doesn't change the behavior.

  • JavaScriptCore.exp:
  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def:
  • runtime/UString.cpp: (JSC::createRep): (JSC::UString::UString): (JSC::UString::from): Move the code to doubleToStringInJavaScriptFormat().
  • runtime/UString.h:
  • wtf/dtoa.cpp: (WTF::dtoa): Fix a bug about rve. (WTF::append): A helper for doubleToStringInJavaScriptFormat(). (WTF::doubleToStringInJavaScriptFormat): Move the code from UString::from(double).
  • wtf/dtoa.h:
18:53 Changeset [51167] by eric@webkit.org

2009-11-18 Yaar Schnitman <yaar@chromium.org>

Reviewed by Darin Adler.

Configuration for Chromium Build Slaves.

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

  • BuildSlaveSupport/build.webkit.org-config/config.json:
  • BuildSlaveSupport/build.webkit.org-config/master.cfg:
18:46 Changeset [51166] by eric@webkit.org

2009-11-18 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Remove WTF_USE_JAVASCRIPTCORE_BINDINGS as it is no longer used
https://bugs.webkit.org/show_bug.cgi?id=31643

2009-11-18 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Remove WTF_USE_JAVASCRIPTCORE_BINDINGS as it is no longer used
https://bugs.webkit.org/show_bug.cgi?id=31643

No new tests as there is no functional change.

  • WebCore.pro:
18:38 Changeset [51165] by eric@webkit.org

2009-11-18 Yong Li <yong.li@torchmobile.com>

Reviewed by Eric Seidel.

Implement TextBoundaries for WINCE port.
https://bugs.webkit.org/show_bug.cgi?id=27371

  • platform/text/wince/TextBoundariesWince.cpp: Added.
18:30 Changeset [51164] by eric@webkit.org

2009-11-18 Yong Li <yong.li@torchmobile.com>

Reviewed by Eric Seidel.

Implement TextBreakIterator for WINCE port.
https://bugs.webkit.org/show_bug.cgi?id=27371

  • platform/text/wince/TextBreakIteratorWince.cpp: Added.
18:15 Changeset [51163] by tkent@chromium.org

2009-11-18 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Adler.

Tests for ValidityState.typeMismatch for the following types of INPUT element:

date, datetime, datetime-local, month, time and week

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

  • fast/forms/ValidityState-typeMismatch-date-expected.txt: Added.
  • fast/forms/ValidityState-typeMismatch-date.html: Added.
  • fast/forms/ValidityState-typeMismatch-datetime-expected.txt: Added.
  • fast/forms/ValidityState-typeMismatch-datetime.html: Added.
  • fast/forms/ValidityState-typeMismatch-datetimelocal-expected.txt: Added.
  • fast/forms/ValidityState-typeMismatch-datetimelocal.html: Added.
  • fast/forms/ValidityState-typeMismatch-month-expected.txt: Added.
  • fast/forms/ValidityState-typeMismatch-month.html: Added.
  • fast/forms/ValidityState-typeMismatch-time-expected.txt: Added.
  • fast/forms/ValidityState-typeMismatch-time.html: Added.
  • fast/forms/ValidityState-typeMismatch-week-expected.txt: Added.
  • fast/forms/ValidityState-typeMismatch-week.html: Added.
  • fast/forms/script-tests/ValidityState-typeMismatch-date.js: Added.
  • fast/forms/script-tests/ValidityState-typeMismatch-datetime.js: Added.
  • fast/forms/script-tests/ValidityState-typeMismatch-datetimelocal.js: Added.
  • fast/forms/script-tests/ValidityState-typeMismatch-month.js: Added.
  • fast/forms/script-tests/ValidityState-typeMismatch-time.js: Added.
  • fast/forms/script-tests/ValidityState-typeMismatch-week.js: Added.

2009-11-18 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Adler.

Add support for ValidityState.typeMismatch for the following INPUT types:

date datetime datetime-local month time week

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

Tests: fast/forms/ValidityState-typeMismatch-date.html

fast/forms/ValidityState-typeMismatch-datetime.html
fast/forms/ValidityState-typeMismatch-datetimelocal.html
fast/forms/ValidityState-typeMismatch-month.html
fast/forms/ValidityState-typeMismatch-time.html
fast/forms/ValidityState-typeMismatch-week.html

  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::formStringToISODateTime): Check the type validity of the specified type and string using the ISODateTime class.
  • html/HTMLInputElement.h:
  • html/ValidityState.cpp: (WebCore::ValidityState::typeMismatch): Check the type validity for date, datetime, datetime-local, month, time and week with HTMLInputElemtn::formStringToISODateTime()
18:13 Changeset [51162] by mjs@apple.com

2009-11-18 Maciej Stachowiak <mjs@apple.com>

Reviewed by Oliver Hunt.

Fix REGRESSION (r47022): Performance of DocumentFragment.appendChild is 1000x slower sometimes
https://bugs.webkit.org/show_bug.cgi?id=31237


Also speeds up Dromaeo DOM Core tests by 1.31x.

  • bindings/js/JSNodeCustom.cpp: (WebCore::JSNode::markChildren): Change marking algorithm to avoid O(N2) behavior. The subtree mark bit was no longer effective; instead I changed things so only a node that has no ancestors with wrappers would do marking; there should be only one in the typical case (the root of the detached subtree).
  • dom/Node.cpp: (WebCore::Node::Node): Remove now useless m_inSubtreeMark bit and related functions.
  • dom/Node.h: ditto
18:07 Changeset [51161] by darin@apple.com

Get rid of the redundant strokeType and fillType data members on
a GraphicsContext as well as the FillOrStrokeType enum. Use
null pointers instead.

Reviewed by Sam Weinig.

  • platform/graphics/GraphicsContext.cpp:

(WebCore::GraphicsContext::setStrokeColor): Null out the pattern
and gradient.
(WebCore::GraphicsContext::setFillColor): Ditto.
(WebCore::GraphicsContext::setStrokePattern): Null out the graident.
(WebCore::GraphicsContext::setFillPattern): Ditto.
(WebCore::GraphicsContext::setStrokeGradient): Null out the pattern.
(WebCore::GraphicsContext::setFillGradient): Ditto.

  • platform/graphics/GraphicsContextPrivate.h: Remove all that stuff.
  • platform/graphics/cairo/GraphicsContextCairo.cpp:

(WebCore::setPlatformFill): Use null checks on fillPattern and
fillGraident instead of a switch statement.
(WebCore::setPlatformStroke): Ditto.

  • platform/graphics/cg/GraphicsContextCG.cpp:

(WebCore::GraphicsContext::applyStrokePattern): Removed unneeded get.
(WebCore::GraphicsContext::applyFillPattern): Ditto.
(WebCore::calculateDrawingMode): Use fillPattern and strokePattern
to see if there is a pattern instead of fillType and strokeType.
(WebCore::GraphicsContext::drawPath): Use fillGradient, strokeGradient,
fillPattern, and strokePattern instead of fillType and strokeType.
(WebCore::GraphicsContext::fillPath): Added FIXME about color space that
is used in the pattern and gradient case probably erroneously. Moved
gradient code inside an if statement. Streamlined pattern code and
use an if statement. No switch any more.
(WebCore::GraphicsContext::strokePath): Ditto.
(WebCore::GraphicsContext::fillRect): Ditto.
(WebCore::GraphicsContext::strokeRect): Ditto.

  • platform/graphics/qt/GraphicsContextQt.cpp:

(WebCore::GraphicsContext::fillPath): Use null checks on fillPattern and
fillGraident instead of a switch statement.
(WebCore::GraphicsContext::strokePath): Ditto.
(WebCore::GraphicsContext::fillRect): Ditto.

  • platform/graphics/wince/GraphicsContextWince.cpp:

(WebCore::GraphicsContext::fillPath): Removed unneeded check of fillType.
(WebCore::GraphicsContext::fillRect): Ditto.

18:01 Changeset [51160] by eric@webkit.org

2009-11-18 Eric Seidel <eric@webkit.org>

Reviewed by Darin Adler.

workqueue.py results in totally broken commit-queue UI
https://bugs.webkit.org/show_bug.cgi?id=31645

  • Scripts/bugzilla-tool:
    • Remove unneeded use of PatchCollection.
    • Grab a new copy of the cq'd patches every run of the queue.
17:48 Changeset [51159] by eric@webkit.org

2009-11-18 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Adler.

Add tests for ValidityState.stepMismatch.
https://bugs.webkit.org/show_bug.cgi?id=31331

  • fast/forms/ValidityState-stepMismatch-number-expected.txt: Added.
  • fast/forms/ValidityState-stepMismatch-number.html: Added.
  • fast/forms/ValidityState-stepMismatch-range-expected.txt: Added.
  • fast/forms/ValidityState-stepMismatch-range.html: Added.
  • fast/forms/ValidityState-stepMismatch-unsupported-expected.txt: Added.
  • fast/forms/ValidityState-stepMismatch-unsupported.html: Added.
  • fast/forms/input-step-expected.txt: Added.
  • fast/forms/input-step.html: Added.
  • fast/forms/script-tests/ValidityState-stepMismatch-number.js: Added.
  • fast/forms/script-tests/ValidityState-stepMismatch-range.js: Added.
  • fast/forms/script-tests/ValidityState-stepMismatch-unsupported.js: Added.
  • fast/forms/script-tests/input-step.js: Added.

2009-11-18 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Adler.

Support for step attribute and ValidityStae.stepMismatch for
type=number and range. stepMismatch will be false if the
difference between the current value and a multiple of the step
value is very small.

Change the behavior of RenderSlider so that it always has a value
rounded to the step attribute value.

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

Tests: fast/forms/ValidityState-stepMismatch-number.html

fast/forms/ValidityState-stepMismatch-range.html
fast/forms/ValidityState-stepMismatch-unsupported.html
fast/forms/input-step.html

  • html/HTMLAttributeNames.in: Add "step".
  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::stepMismatch): (WebCore::HTMLInputElement::getStepParameters): (WebCore::HTMLInputElement::getAllowedValueStep):
  • html/HTMLInputElement.h:
  • html/HTMLInputElement.idl: Add "step".
  • html/ValidityState.cpp: (WebCore::ValidityState::stepMismatch): Forward to HTMLInputElement::stepMismatch().
  • html/ValidityState.h:
  • rendering/RenderSlider.cpp: (WebCore::SliderRange::SliderRange): (WebCore::SliderRange::clampValue):
17:27 Changeset [51158] by dimich@chromium.org

Need to ASSERT(isMainThread()) in ThreadShared methods.
https://bugs.webkit.org/show_bug.cgi?id=31637

Reviewed by Darin Adler.

Added ASSERT(IsMainThread()) to all following methods:

  • platform/TreeShared.h:

(WebCore::TreeShared::TreeShared):
(WebCore::TreeShared::~TreeShared):
(WebCore::TreeShared::ref):
(WebCore::TreeShared::deref):
(WebCore::TreeShared::setParent):
(WebCore::TreeShared::parent):

17:00 Changeset [51157] by eric@webkit.org

2009-11-18 Girish Ramakrishnan <girish@forwardbias.in>

Reviewed by Eric Seidel.

Add plugin visibility manual test

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

  • manual-tests/plugins/windowed.html:
  • manual-tests/plugins/windowless.html:
16:52 Changeset [51156] by eric@webkit.org

2009-11-18 Patrick Mueller <Patrick_Mueller@us.ibm.com>

Reviewed by Pavel Feldman.

Web Inspector - remember last script displayed in Scripts panel
https://bugs.webkit.org/show_bug.cgi?id=27552

Manual test added

  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype.showScript): (WebInspector.ScriptsPanel.prototype.showResource): (WebInspector.ScriptsPanel.prototype._showScriptOrResource): (WebInspector.ScriptsPanel.prototype._addScriptToFilesMenu): (WebInspector.ScriptsPanel.prototype._callFrameSelected): (WebInspector.ScriptsPanel.prototype._goBack): (WebInspector.ScriptsPanel.prototype._goForward):
  • manual-tests/inspector/remember-last-script.html: Added.
16:43 Changeset [51155] by eric@webkit.org

2009-11-18 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by Dimitri Glazkov.

Win chromium is slow to draw transparent texts
https://bugs.webkit.org/show_bug.cgi?id=31258

Create bounded transparency layers instead of just clipping.

No new tests because this is just a performance improvement.

  • platform/graphics/chromium/FontChromiumWin.cpp: (WebCore::TransparencyAwareFontPainter::TransparencyAwareFontPainter::initializeForGDI):
16:35 Changeset [51154] by eric@webkit.org

2009-11-18 Shu Chang <Chang.Shu@nokia.com>

Reviewed by Eric Seidel.

[Qt] Update expected result after Qt4.6 upgrade.
https://bugs.webkit.org/show_bug.cgi?id=31560

Test: LayoutTests/editing/deleting/5408255.html

  • platform/qt/editing/deleting/5408255-expected.txt:

2009-11-18 Shu Chang <Chang.Shu@nokia.com>

Reviewed by Eric Seidel.

[Qt] Add support for displaying deleteButton.
https://bugs.webkit.org/show_bug.cgi?id=31560

Test: LayoutTests/editing/deleting/5408255.html

  • WebCore.qrc:
  • platform/graphics/qt/ImageQt.cpp: (loadResourcePixmap):

2009-11-18 Shu Chang <Chang.Shu@nokia.com>

Reviewed by Eric Seidel.

[Qt] Add support for displaying deleteButton.
https://bugs.webkit.org/show_bug.cgi?id=31560

Test: LayoutTests/editing/deleting/5408255.html

  • Api/qwebsettings.cpp: (graphics):
  • Api/qwebsettings.h:
16:26 Changeset [51153] by eric@webkit.org

2009-11-18 Mikhail Naganov <mnaganov@chromium.org>

Reviewed by Eric Seidel.

Fix Qt build without JavaScript debugger.

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

  • page/Console.idl:
16:18 Changeset [51152] by eric@webkit.org

2009-11-18 Eric Seidel <eric@webkit.org>

Reviewed by David Levin.

Make 80 patches a formal requirement for reviewership
https://bugs.webkit.org/show_bug.cgi?id=31313
Per discussion on the webkit-reviewers list:
http://lists.webkit.org/mailman/private.cgi/webkit-reviewers/2009-November/000795.html

  • coding/commit-review-policy.html:
16:05 Changeset [51151] by dbates@webkit.org

2009-11-18 Daniel Bates <dbates@webkit.org>

Reviewed by Dave Hyatt.

https://bugs.webkit.org/show_bug.cgi?id=24971
And
<rdar://problem/7138265>

Fixes an issue where setting the height of a table row programmatically
(via JavaScript) causes the table to be improperly rendered because the
height of each row in the table is not recalculated with respect to the
CSS height property.

In particular, programmatically setting the height of some table row causes
the rows of that table to be rendered with a height equal to the minimum
height required by the cells in that row, regardless of any specified cell
heights. Instead, when RenderTableSection::recalcCells is called, the height
of each row should be set to the CSS height property just as we do in
RenderTableSection::addChild.

Test: fast/table/row-height-recalc2.html

  • rendering/RenderTableSection.cpp: (WebCore::setRowHeightToRowStyleHeightIfNotRelative): Added. (WebCore::RenderTableSection::addChild): Moved code that set row height into method WebCore::setRowHeightToRowStyleHeightIfNotRelative. (WebCore::RenderTableSection::recalcCells): Modified to call WebCore::setRowHeightToRowStyleHeightIfNotRelative.

2009-11-18 Daniel Bates <dbates@webkit.org>

Reviewed by Dave Hyatt.

https://bugs.webkit.org/show_bug.cgi?id=24971
And
<rdar://problem/7138265>

Tests that setting the height of a table row programmatically (via JavaScript)
recalculates the height of the rows in the table.

  • fast/table/row-height-recalc2.html: Added.
  • platform/mac/fast/table/row-height-recalc2-expected.checksum: Added.
  • platform/mac/fast/table/row-height-recalc2-expected.png: Added.
  • platform/mac/fast/table/row-height-recalc2-expected.txt: Added.
  • platform/qt/fast/table/row-height-recalc2-expected.checksum: Added.
  • platform/qt/fast/table/row-height-recalc2-expected.png: Added.
  • platform/qt/fast/table/row-height-recalc2-expected.txt: Added.
  • platform/win/fast/table/row-height-recalc2-expected.checksum: Added.
  • platform/win/fast/table/row-height-recalc2-expected.png: Added.
  • platform/win/fast/table/row-height-recalc2-expected.txt: Added.
15:57 Changeset [51150] by jparent@chromium.org

Reduce flakiness in zero-duration-with-non-zero-delay-end.html by
increasing a timeout.

Reviewed by Eric Seidel.

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

  • transitions/zero-duration-with-non-zero-delay-end.html:
15:47 Changeset [51149] by zoltan@webkit.org

Allow custom memory allocation control for the part of platform directory in WebCore
https://bugs.webkit.org/show_bug.cgi?id=31473

Reviewed by Darin Adler.

Inherits the following classes from FastAllocBase because these are
instantiated by 'new':

class AnimationList - instantiated at WebCore/rendering/style/StyleRareNonInheritedData.cpp:85
class Color - instantiated at WebCore/rendering/RenderTheme.cpp:48
struct Length - instantiated at WebCore/platform/Length.cpp:103
class PlatformKeyboardEvent - instantiated at WebCore/dom/KeyboardEvent.cpp:63
class ContextMenuItem - instantiated at WebCore/platform/ContextMenu.cpp:70
class DeprecatedPtrList - instantiated at WebCore/rendering/RenderBlock.cpp:2284

Inherits the following classes from Noncopyable because these are
instantiated by 'new' and no need to be copyable:

class GraphicsContextPrivate - instantiated at WebCore/platform/graphics/GraphicsContext.cpp:78
class FontCache - instantiated at WebCore/platform/graphics/qt/FontCacheQt.cpp:43
struct MediaPlayerFactory - instantiated at WebCore/platform/graphics/MediaPlayer.cpp:163
class DeprecatedPtrListNode - instantiated at WebCore/platform/DeprecatedPtrListImpl.cpp:53

  • platform/ContextMenuItem.h:
  • platform/DeprecatedPtrList.h:
  • platform/DeprecatedPtrListImpl.cpp:
  • platform/Length.h:
  • platform/PlatformKeyboardEvent.h:
  • platform/animation/AnimationList.h:
  • platform/graphics/Color.h:
  • platform/graphics/FontCache.h:
  • platform/graphics/GraphicsContextPrivate.h:
  • platform/graphics/MediaPlayer.cpp:
15:46 Changeset [51148] by eric@webkit.org

2009-11-18 Daniel Bates <dbates@webkit.org>

Reviewed by Darin Adler.

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

Renames RenderTextControl::m_edited and RenderTextControl::m_userEdited to
m_wasChangedSinceLastChangeEvent and m_lastChangeWasUserEdit, respectively.
These are more descriptive names so as to clear an ambiguity surrounding
their usage. Also, renames associated setters and getters so that they
coincide with the renamed fields.

No functionality was changed. So, no tests were included.

  • bindings/objc/DOMHTML.mm: (-[DOMHTMLInputElement _isEdited]): (-[DOMHTMLTextAreaElement _isEdited]):
  • dom/Document.cpp: (WebCore::Document::setFocusedNode):
  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::defaultEventHandler):
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::RenderTextControl): (WebCore::RenderTextControl::setInnerTextValue): (WebCore::RenderTextControl::setLastChangeWasUserEdit): Formerly named setUserEdited. (WebCore::RenderTextControl::subtreeHasChanged):
  • rendering/RenderTextControl.h: (WebCore::RenderTextControl::wasChangedSinceLastChangeEvent): Formerly named isEdited. (WebCore::RenderTextControl::setChangedSinceLastChangeEvent): Formerly named setEdited. (WebCore::RenderTextControl::lastChangeWasUserEdit): Formerly named setUserEdited.
  • rendering/RenderTextControlSingleLine.cpp: (WebCore::RenderTextControlSingleLine::subtreeHasChanged):
  • wml/WMLInputElement.cpp: (WebCore::WMLInputElement::defaultEventHandler):

2009-11-18 Daniel Bates <dbates@webkit.org>

Reviewed by Darin Adler.

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

Changes associated with renaming RenderTextControl::isUserEdited.

  • DOMHTMLClasses.cpp: (DOMHTMLInputElement::isUserEdited): Formerly named isUserEdited. (DOMHTMLTextAreaElement::isUserEdited): Ditto.
15:44 Changeset [51147] by weinig@apple.com

Make the Mac Geolocation API async.

Reviewed by Anders Carlsson.

WebKit:

  • WebKit.xcodeproj/project.pbxproj: Remove WebGeolocation.

WebKit/mac:

  • WebCoreSupport/WebChromeClient.mm:

(WebChromeClient::requestGeolocationPermissionForFrame):
(-[WebGeolocationPolicyListener initWithGeolocation:]):
(-[WebGeolocationPolicyListener allow]):
(-[WebGeolocationPolicyListener deny]):
Add WebGeolocationPolicyListener implementation of the new WebGeolocationPolicyListener
protocol and use if to implement requestGeolocationPermissionForFrame using the new async
API.

  • WebCoreSupport/WebGeolocation.mm: Removed.
  • WebCoreSupport/WebGeolocationInternal.h: Removed.
  • WebCoreSupport/WebGeolocationMock.mm: Remove bogus include.
  • WebCoreSupport/WebGeolocationPrivate.h: Removed.
  • WebView/WebUIDelegatePrivate.h: Remove requestGeolocationPermission and add

WebGeolocationPolicyListener protocol and decidePolicyForGeolocationRequestFromOrigin delegate
method.

WebKitTools:

Update DRT to use the new async Mac Geolocation API.

  • DumpRenderTree/mac/UIDelegate.mm:

(-[UIDelegate webView:decidePolicyForGeolocationRequestFromOrigin:frame:listener:]):

15:36 Changeset [51146] by eric@webkit.org

2009-11-18 Tony Chang <tony@chromium.org>

Reviewed by Adam Barth.

Remove V8CanvasPixelArrayCustom.cpp because it is no longer used by V8.
https://bugs.webkit.org/show_bug.cgi?id=31499

No new tests, just a build cleanup.

  • WebCore.gypi:
  • bindings/v8/custom/V8CanvasPixelArrayCustom.cpp: Removed.
  • bindings/v8/custom/V8CustomBinding.h:
15:29 Changeset [51145] by eric@webkit.org

2009-11-18 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

Add and option to toggle HTML5 datalist support to build-webkit
https://bugs.webkit.org/show_bug.cgi?id=31599

  • Scripts/build-webkit:
15:25 Changeset [51144] by japhet@chromium.org

2009-11-18 Nate Chapin <japhet@chromium.org>

Reviewed by Darin Fisher.

Remove Chromium's unnecessary dependency on wtf's tcmalloc files.

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

15:23 Applications using WebKit edited by blythe_yee@apple.com
(diff)
15:18 Changeset [51143] by kevino@webkit.org

Reviewed by Kevin Ollivier.

Enable wx plugin support using the Windows implementation as a base.

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

15:10 Changeset [51142] by eric@webkit.org

2009-11-18 Evan Stade <estade@chromium.org>

Reviewed by Darin Fisher.

Menu button shows wrong content in Chromium
https://bugs.webkit.org/show_bug.cgi?id=31448

Make context menu button (VK_APPS) show context menu for node that has
keyboard focus. This affects both the content and the positioning.

  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::sendContextMenuEvent):
15:02 Changeset [51141] by eric@webkit.org

2009-11-18 Thiago Macieira <thiago.macieira@nokia.com>

Reviewed by Gavin Barraclough.

[Qt] Implement symbol hiding for JSC's JIT functions.

These functions are implemented directly in assembly, so they need the
proper directives to enable/disable visibility. On ELF systems, it's
.hidden, whereas on Mach-O systems (Mac) it's .private_extern. On
Windows, it's not necessary since you have to explicitly export. I
also implemented the AIX idiom, though it's unlikely anyone will
implement AIX/POWER JIT.
https://bugs.webkit.org/show_bug.cgi?id=30864

  • jit/JITStubs.cpp:
14:49 Changeset [51140] by eric@webkit.org

2009-11-18 Andrei Popescu <andreip@google.com>

Reviewed by Dimitri Glazkov.

[Android] Add shared timer and sound utilities to platform/android
https://bugs.webkit.org/show_bug.cgi?id=31584

No new tests required, this is platform specific code.

  • platform/android/SharedTimerAndroid.cpp: Added. (WebCore::setSharedTimerFiredFunction): (WebCore::setSharedTimerFireTime): (WebCore::stopSharedTimer):
  • platform/android/SoundAndroid.cpp: Added. (WebCore::systemBeep):
14:41 Changeset [51139] by eric@webkit.org

2009-11-18 Mikhail Naganov <mnaganov@chromium.org>

Reviewed by Timothy Hatcher.

Fix profile tree nodes loss after focus / restore actions.

Focusing on a node is currently implemented via nodes reattaching
with some caching involved. It seems that not all code was updated
to handle this scenario correctly.

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

  • inspector/front-end/BottomUpProfileDataGridTree.js: (WebInspector.BottomUpProfileDataGridNode): (WebInspector.BottomUpProfileDataGridNode.prototype._restore): (WebInspector.BottomUpProfileDataGridNode.prototype._sharedPopulate): (WebInspector.BottomUpProfileDataGridNode.prototype._willHaveChildren):
  • inspector/front-end/DataGrid.js: (WebInspector.DataGrid.prototype.insertChild): (WebInspector.DataGridNode.prototype._detach): (WebInspector.DataGridNode.prototype.savePosition): (WebInspector.DataGridNode.prototype.restorePosition):
  • inspector/front-end/TopDownProfileDataGridTree.js: (WebInspector.TopDownProfileDataGridTree.prototype.focus): (WebInspector.TopDownProfileDataGridTree.prototype.restore):
14:37 Changeset [51138] by dbates@webkit.org

2009-11-18 Daniel Bates <dbates@webkit.org>

Reviewed by Eric Seidel.

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

Adds property "min-width" to #title so that the width of the title
element on each page is at least 790px. In particular, this makes
the page <http://webkit.org/security/security-group-members.html>
consistent, in appearance, to the rest of the site.

  • css/main.css:
14:33 Changeset [51137] by eric@webkit.org

2009-11-18 Carol Szabo <carol.szabo@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] QWebPageClient.h needs in some cases QCursor, but it does not
include it
https://bugs.webkit.org/show_bug.cgi?id=31527

No new tests as this is just a coding style fix that affects the
build of some not yet submitted patches (i.e. for bug 30173).

  • platform/qt/QWebPageClient.h:
14:25 Changeset [51136] by eric@webkit.org

2009-11-18 Nicolas Roard <nicolas@roard.com>

Reviewed by Dimitri Glazkov.

InspectorTimelineAgent.h should be guarded by ENABLE(INSPECTOR)
https://bugs.webkit.org/show_bug.cgi?id=31504

  • inspector/InspectorTimelineAgent.h:Added the guard.
14:08 Changeset [51135] by snej@chromium.org

Build fix to my previous checkin, for Windows Chromium

  • svg/SVGAnimatedProperty.h:

(WebCore::PropertySynchronizer::synchronize): Fix implicit-conversion error on AtomicString.

14:05 Changeset [51134] by joepeck@webkit.org

2009-11-18 Joseph Pecoraro <joepeck@webkit.org>

Reviewed by Pavel Feldman.

Web Inspector: Directly Access <head> Instead of Searching for It
https://bugs.webkit.org/show_bug.cgi?id=31641

Changed old access of the head element (document.getElementsByTagName)
to just use the document.head accessor.

  • inspector/front-end/FontView.js: (WebInspector.FontView):
  • inspector/front-end/InjectedScript.js: (InjectedScript.addStyleSelector):
  • inspector/front-end/SourceFrame.js: (WebInspector.SourceFrame.prototype._loaded):
  • inspector/front-end/inspector.js: (windowLoaded):
14:04 Changeset [51133] by weinig@apple.com

Fix two Geolocation assertions.

Reviewed by Anders Carlsson.

  • page/Chrome.cpp:

(WebCore::Chrome::requestGeolocationPermissionForFrame): No need to use PageGroupLoadDeferrer since this
is not called from JS.

  • page/Geolocation.cpp:

(WebCore::Geolocation::Watchers::set): The PassRefPtr was getting nulled out, so we need to put it in
a RefPtr first.

14:02 Changeset [51132] by ap@apple.com

Case sensitive file system build fix.

  • platform/network/Credential.h: It's not WTF, just wtf.
13:54 Changeset [51131] by agolden@apple.com

Add support for certificates to WebCore::Credential so we can convert between NSURLCredential
objects and WebCore::Credential objects without losing certificate information.

Reviewed by Alexey Proskuryakov

13:49 Changeset [51130] by darin@chromium.org

2009-11-17 Darin Fisher <darin@chromium.org>

Reviewed by Dmitry Titov.

Minor cleanup in WebKit API
https://bugs.webkit.org/show_bug.cgi?id=31616

  • public/WebFormElement.h: Add missing WEBKIT_API prefix.
  • public/WebNode.h: Add missing WEBKIT_API prefix.
  • src/WebFormElement.cpp: (WebKit::WebFormElement::action): Make action getter const.
  • src/WebNode.cpp: (WebKit::WebNode::frame): Utilize WebFrameImpl::fromFrame.
13:31 Changeset [51129] by oliver@apple.com

Update expected results of fixed test.

12:46 Changeset [51128] by oliver@apple.com

Interpreter may do an out of range access when throwing an exception in the profiler.
https://bugs.webkit.org/show_bug.cgi?id=31635

Reviewed by Alexey Proskuryakov.

Add bounds check.

12:25 Changeset [51127] by dimich@chromium.org

Reverting r50919 that has introduced a non-thread-safe refcounting in ScriptExecutionContext::postTaskToMainThread.
https://bugs.webkit.org/show_bug.cgi?id=31615

Reviewed by Eric Seidel.

  • dom/Document.cpp:

(WebCore::ScriptExecutionContextTaskTimer::ScriptExecutionContextTaskTimer):
(WebCore::ScriptExecutionContextTaskTimer::fired):
(WebCore::PerformTaskContext::PerformTaskContext):
(WebCore::performTask):
(WebCore::Document::postTask):

  • dom/ScriptExecutionContext.cpp:
  • dom/ScriptExecutionContext.h:
12:09 Changeset [51126] by ap@apple.com

Disabling WebSocket tests on Tiger back, they still hang.

  • platform/mac-tiger/Skipped:
11:57 Changeset [51125] by snej@chromium.org

Eliminate unnecessary String-->AtomicString conversions from generated V8 bindings,
by causing the right v8-to-WebCore conversion function to be called for every parameter.
This no longer requires any IDL metadata, so I've removed the [HintAtomic] annotations.
To enforce correctness, I added a mode that disables implicit
String-->AtomicString conversions while compiling the generated bindings.
https://bugs.webkit.org/show_bug.cgi?id=31168

Reviewed by Darin Adler.

  • bindings/scripts/CodeGeneratorV8.pm: Generate usage of V8Parameter class.
  • bindings/v8/DerivedSourcesAllInOne.cpp: Enable NO_IMPLICIT_ATOMICSTRING.
  • bindings/v8/V8Binding.h: Add V8Parameter class.
  • css/WebKitCSSKeyframesRule.h: Make AtomicString conversions explicit.
  • dom/Document.idl: Remove obsolete [HintAtomic] annotation.
  • platform/text/AtomicString.h: Added NO_IMPLICIT_ATOMICSTRING option.
  • svg/SVGAnimatedTemplate.h: Change some return types to String to avoid implicit conversion.
  • svg/SVGAnimatedProperty.h: Adapt to changed return types in SVGAnimatedTemplate.
11:49 Changeset [51124] by darin@apple.com

Move FillOrStrokeType out of public header.

Reviewed by Dan Bernstein.

  • platform/graphics/GraphicsContext.h: Updated copyright date to cover

some years we published Apple changes, sorted forward declarations,
removed FillOrStrokeType enum.

  • platform/graphics/GraphicsContextPrivate.h: Updated copyright date

to cover some years we published Apple changes, sorted includes,
moved FillOrStrokeType enum here.

11:37 Changeset [51123] by ossy@webkit.org

Fix the clobber list of cacheFlush for ARM and Thumb2 on Linux
https://bugs.webkit.org/show_bug.cgi?id=31631

Patch by Gabor Loki <loki@inf.u-szeged.hu> on 2009-11-18
Reviewed by Darin Adler.

  • jit/ExecutableAllocator.h:

(JSC::ExecutableAllocator::cacheFlush):

11:23 Changeset [51122] by cmarrin@apple.com

Add Settings for WebKitShowDebugBorders and WebKitShowRepaintCounter
https://bugs.webkit.org/show_bug.cgi?id=31601

These are used to debug accelerated compositing layers. I removed
the platform specific code from GraphicsLayerCA.mm and added calls
to GraphicsLayerClient to get it from the higher levels. The values
now get cached in RenderLayerCompositing and are queried from there
by RenderLayerBacking (which implements the GraphicsLayerClient interface).

10:26 Changeset [51121] by pfeldman@chromium.org

2009-11-18 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Get rid of metrics and properties sidebars'
flickering.

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

  • inspector/front-end/MetricsSidebarPane.js:
  • inspector/front-end/PropertiesSidebarPane.js:
10:21 Changeset [51120] by jparent@chromium.org

Eliminate flakiness in file-URL-with-port-number by removing a setTimeout.
The timeout caused the test to fail about 2% of the time on the Chromium builders.
The timeout exists to catch cases where the iframe does not load, but the test
framework already will handle this (albeit with a longer timeout).

Reviewed by Dmitry Titov.

  • fast/loader/file-URL-with-port-number.html:
10:12 Changeset [51119] by bweinstein@apple.com

Fixes <http://webkit.org/b/31606>.
Web Inspector: Enter/Return key should enter edit mode for Editable Fields.

Reviewed by Pavel Feldman.

This implements Enter starting editing mode in an editable DataGrid. If the
DataGrid is editable and the user hits return, startEditing the first child
of the selected node. Also refactored some editing functions to take an
event target instead of the event itself, because the functions only needed
the target. Lastly, added had return in editing mode stop propogation, because
when enter was hit to confirm text, it would propagate back to the datagrid
and try to start editing again.

  • inspector/front-end/DataGrid.js:

(WebInspector.DataGrid.prototype._ondblclick):
(WebInspector.DataGrid.prototype._startEditing):
(WebInspector.DataGrid.prototype.handleKeyEvent):
(WebInspector.DataGrid.prototype.dataGridNodeFromEvent):
(WebInspector.DataGrid.prototype._mouseDownInDataTable):
(WebInspector.DataGrid.prototype._clickInDataTable):

  • inspector/front-end/inspector.js:

(WebInspector.startEditing.element.handleKeyEvent):
(WebInspector.startEditing):

09:57 Changeset [51118] by benm@google.com

HTMLAnchorElement is inconsistent with its internal handling of the value returned for the href attribute.
https://bugs.webkit.org/show_bug.cgi?id=31593

Reviewed by Darin Adler.

WebCore:

Test: fast/dom/HTMLAnchorElement/set-href-attribute-whitespace.html

  • html/HTMLAnchorElement.cpp:

(WebCore::HTMLAnchorElement::href): Add call to deprecatedParseURL.

LayoutTests:

  • fast/dom/HTMLAnchorElement/script-tests/set-href-attribute-whitespace.js: Added.
  • fast/dom/HTMLAnchorElement/set-href-attribute-whitespace-expected.txt: Added.
  • fast/dom/HTMLAnchorElement/set-href-attribute-whitespace.html: Added.
09:38 Changeset [51117] by pfeldman@chromium.org

2009-11-18 Pavel Feldman <pfeldman@chromium.org>

Not reviewed. Touch InspectorController so that frontend
JS files are deployed on Windows.

  • inspector/InspectorController.cpp:
09:30 Changeset [51116] by ap@apple.com

Trying to enable WebSocket tests on Tiger again, now that bug 31603 is fixed.

  • platform/mac-tiger/Skipped:
09:25 Changeset [51115] by pfeldman@chromium.org

2009-11-18 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: clone timeline records array instead of
copying reference on invalidate all.

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

  • inspector/front-end/AbstractTimelinePanel.js: (WebInspector.AbstractTimelinePanel.prototype.invalidateAllItems):
08:38 Changeset [51114] by kenneth@webkit.org

Change the initialization order so that the controllers
will be created before exporting them to the JS DOM window.

Reviewed by Simon Hausmann.

  • DumpRenderTree/qt/DumpRenderTree.cpp:

(WebCore::DumpRenderTree::DumpRenderTree):

08:17 QtWebKitAPI edited by hausmann@webkit.org
(diff)
08:16 Changeset [51113] by kenneth@webkit.org

Fix a code copy and paste error. m_page should be page.

Reviewed by Simon Hausmann.

  • DumpRenderTree/qt/DumpRenderTree.cpp:

(WebCore::DumpRenderTree::createWindow):

08:15 QtWebKitAPI edited by hausmann@webkit.org
(diff)
08:14 QtWebKitAPI edited by hausmann@webkit.org
(diff)
08:05 QtWebKitAPI created by hausmann@webkit.org
07:49 QtWebKitContrib edited by hausmann@webkit.org
(diff)
07:02 Changeset [51112] by simon.fraser@apple.com

2009-11-18 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dan Bernstein.

Elements don't drop out of compositing layers when animation ends
https://bugs.webkit.org/show_bug.cgi?id=31613
<rdar://problem/7402913>

Avoid setting the mustOverlapCompositedLayers flag on RenderLayers which
are compositing anyway for other reasons. Doing so can cause those layers
to stay in compositing mode even after animations finish, because needsToBeComposited()
continues to return true.

No new tests because it's not possible to determine which elements are in
compositing layers from DRT output.

  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::computeCompositingRequirements):
06:27 Changeset [51111] by hausmann@webkit.org

[Qt] Make the QWebElement::render() test pass when pixmaps aren't 32 bit.

Patch by Paul Olav Tvete <paul.tvete@nokia.com> on 2009-11-18
Reviewed by Simon Hausmann.

  • tests/qwebelement/tst_qwebelement.cpp:

(tst_QWebElement::render):

05:22 Changeset [51110] by hausmann@webkit.org

[Qt] Fix detection of linux-g++

Patch by Harald Fernengel <harald.fernengel@nokia.com> on 2009-11-18
Reviewed by Simon Hausmann.

Never use "linux-g++*" to check for linux-g++, since this will break embedded
builds which use linux-arm-g++ and friends. Use 'linux*-g++*' to check for any
g++ on linux mkspec.

05:20 Changeset [51109] by kov@webkit.org

Properly localize documentation strings for the tab-key-cycles-through-elements property.

05:20 Changeset [51108] by kov@webkit.org

Ensure that tabKeyCyclesThroughElements is true by default.

05:19 Changeset [51107] by hausmann@webkit.org

Clarify and simplify the legal section in the overview documentation,
after review with our legal team.

Reviewed by Tor Arne Vestbø.

  • docs/qtwebkit.qdoc:
04:58 Changeset [51106] by vestbo@webkit.org

[Qt] Add QtLauncher support for opening links in the default browser

Reviewed by Simon Hausmann.

This can be triggered by either the context menu or by clicking a link
while holding down the Alt key. Opening a link in a new windows is
triggered by holding down Shift.

  • QtLauncher/main.cpp:
04:53 Changeset [51105] by girish@forwardbias.in

[Qt] Mac Plugins: Remove null timer

Reviewed by Tor Arne Vestbø.

A null timer was used to send mouse move events. Instead, we now use
move events to send nullEvent. This brings down CPU usage by 20-30%.

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

04:48 Changeset [51104] by kov@webkit.org

Reviewed by Xan Lopez.

[GTK] Failing test media/video-document-types.html
https://bugs.webkit.org/show_bug.cgi?id=31352

Match what Mac has been doing since r36001: cancel the main load,
and handle failures caused by 'will be handled by plugin' errors.

Covered by test media/video-document-types.html

  • WebCoreSupport/FrameLoaderClientGtk.cpp:

(WebKit::FrameLoaderClient::committedLoad):
(WebKit::FrameLoaderClient::shouldFallBack):

02:29 Changeset [51103] by hausmann@webkit.org

[Qt] WebKit crashes when loading certain SVG images

Patch by Benjamin Poulain <benjamin.poulain@nokia.com> on 2009-11-18
Reviewed by Simon Hausmann.

WebCore:

Check if the familly exist before creating the PlatformData from it.

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

Test: svg/text/text-font-invalid.html

  • platform/graphics/qt/FontFallbackListQt.cpp:

(WebCore::FontFallbackList::fontDataAt):

LayoutTests:

Added a test for a crash of QtWebkit when loading SVG images.

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

  • svg/text/resources/text-font-invalid.svg: Added.
  • svg/text/text-font-invalid-expected.txt: Added.
  • svg/text/text-font-invalid.html: Added.
Note: See TracTimeline for information about the timeline view.