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

Timeline



Dec 27, 2009:

11:59 PM Changeset in webkit [52594] by eric@webkit.org
  • 4 edits in trunk/WebKitTools

2009-12-27 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Add more awesome bug-parsing logic to bugzilla.py in preparation for assign-to-committer command
​https://bugs.webkit.org/show_bug.cgi?id=32980

  • Scripts/modules/bugzilla.py:
    • Add a new _parse_bug_page function and use it in fetch_attachments_from_bug
    • Replace fetch_title_from_bug with a new fetch_bug call instead.
    • Use list comprehensions where possible to reduce code duplication.
  • Scripts/modules/bugzilla_unittest.py:
    • Add a minimal bug parsing test.
    • Share code between bug parsing and attachment parsing tests with _assert_dictionaries_equal
  • Scripts/modules/commands/upload.py:
    • Use fetch_bug(bug_id)title instead of fetch_title_from_bug
11:05 PM Changeset in webkit [52593] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-12-27 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Status bubble shouldn't be transparent for bots that haven't run yet
​https://bugs.webkit.org/show_bug.cgi?id=32977

  • QueueStatusServer/templates/statusbubble.html:
10:58 PM Changeset in webkit [52592] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-12-27 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

[QueueStatusServer] Run gc cron job less often
​https://bugs.webkit.org/show_bug.cgi?id=32958

Now that we've cleaned out the backlog of idle messages, we don't need
to run the gc job every half hour. We could also stop logging the idle
messages, but this seems easier for the time being.

  • QueueStatusServer/cron.yaml:
8:58 PM Changeset in webkit [52591] by mjs@apple.com
  • 3 edits in trunk/LayoutTests

Rubber stamped by Adam Barth.

Put media/audio-delete-while-step-button-clicked.html back in the mac skipped list because
I think it's making other media tests fail sporadically.

  • platform/mac-tiger/Skipped:
  • platform/mac/Skipped:
7:39 PM Changeset in webkit [52590] by eric@webkit.org
  • 5 edits in trunk

2009-12-27 Jakub Wieczorek <​faw217@gmail.com>

Reviewed by Eric Seidel.

[Qt] Support upload progress events in XMLHttpRequest
​https://bugs.webkit.org/show_bug.cgi?id=32969

Make QNetworkReplyHandler report the upload progress to the client.

Already covered by tests:
http/tests/xmlhttprequest/upload-onload-event.html
http/tests/xmlhttprequest/upload-onloadstart-event.html
http/tests/xmlhttprequest/upload-progress-events.html
http/tests/xmlhttprequest/upload-onprogress-event.html

which have been unskipped.

  • platform/network/qt/QNetworkReplyHandler.cpp: (WebCore::QNetworkReplyHandler::uploadProgress): (WebCore::QNetworkReplyHandler::start):
  • platform/network/qt/QNetworkReplyHandler.h:

2009-12-27 Jakub Wieczorek <​faw217@gmail.com>

Reviewed by Eric Seidel.

[Qt] Support upload progress events in XMLHttpRequest
​https://bugs.webkit.org/show_bug.cgi?id=32969

Make QNetworkReplyHandler report the upload progress to the client.

Already covered by tests:
http/tests/xmlhttprequest/upload-onload-event.html
http/tests/xmlhttprequest/upload-onloadstart-event.html
http/tests/xmlhttprequest/upload-progress-events.html
http/tests/xmlhttprequest/upload-onprogress-event.html

which have been unskipped.

  • platform/qt/Skipped:
7:17 PM Changeset in webkit [52589] by mjs@apple.com
  • 2 edits in trunk/LayoutTests

Rubber stamped by Adam Barth.

Add media/audio-delete-while-step-button-clicked.html back to the Tiger skipped list.

  • platform/mac-tiger/Skipped:
6:28 PM Changeset in webkit [52588] by mjs@apple.com
  • 3 edits
    6 adds in trunk/LayoutTests

Rubber stamped by Adam Barth.

Removed some tests from the skipped list that are not failing on my system.

  • platform/mac-snowleopard/Skipped:
  • platform/mac/Skipped:
  • platform/mac/fast/forms/menulist-style-color-expected.checksum: Added.
  • platform/mac/fast/forms/menulist-style-color-expected.png: Added.
  • platform/mac/fast/forms/menulist-style-color-expected.txt: Added.
  • platform/mac/plugins/qt-qwidget-plugin-expected.checksum: Added.
  • platform/mac/plugins/qt-qwidget-plugin-expected.png: Added.
  • platform/mac/plugins/qt-qwidget-plugin-expected.txt: Added.
6:01 PM Changeset in webkit [52587] by dbates@webkit.org
  • 5 edits
    1 add in trunk/WebKitTools

2009-12-27 Daniel Bates <​dbates@webkit.org>

Reviewed by Adam Barth.

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

Fixes an issue where bugzilla-tool tries to read the username and password from
Git regardless of whether Git is installed. In particular, if Git is not
installed then bugzilla-tool dies (with a trace) when it attempts to query Git
for the authentication credentials to log into bugs.webkit.org.

Moreover, modifies Executive.run_command to catch and pass OSError exceptions to
the specified error handler. For instance, the specified error handler will now
be called when the command does not exist (i.e. OSError errno 2).

  • Scripts/modules/credentials.py: Added check for Git.
  • Scripts/modules/credentials_unittest.py: Added test method test_read_credentials_with_SVN.
  • Scripts/modules/executive.py: Modified method run_command to catch OSError exceptions (i.e [Errno 2] No such file or directory) and call the specified error handler.
  • Scripts/modules/executive_unittest.py: Added.
  • Scripts/run-webkit-unittests: Added import executive_unittest.py.
5:25 PM Changeset in webkit [52586] by mjs@apple.com
  • 2 edits in trunk/LayoutTests

Rubber stamped by Cameron Zwarich.

Skip fast/images/size-failure.html on SnowLeopard because it
tickles a CG bug that makes it time out.

  • platform/mac-snowleopard/Skipped:
3:48 PM Changeset in webkit [52585] by mjs@apple.com
  • 4 edits
    3 copies in trunk/LayoutTests

2009-12-27 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Dan Bernstein.

fast/forms/basic-selects.html is failing on SnowLeopard
​https://bugs.webkit.org/show_bug.cgi?id=32960

The recently added fast/forms/basic-selects.html is failing on SnowLeopard; the
result checked in is only valid for Leopard and earlier, because SnowLeopard
slightly changed the disabled text color.


Move the pre-SL result to platform/mac-leopard and generate new
SnowLeopard results in platform/mac.


  • platform/mac-leopard/fast/forms/basic-selects-expected.checksum: Copied from platform/mac/fast/forms/basic-selects-expected.checksum.
  • platform/mac-leopard/fast/forms/basic-selects-expected.png: Copied from platform/mac/fast/forms/basic-selects-expected.png.
  • platform/mac-leopard/fast/forms/basic-selects-expected.txt: Copied from platform/mac/fast/forms/basic-selects-expected.txt.
  • platform/mac/fast/forms/basic-selects-expected.checksum:
  • platform/mac/fast/forms/basic-selects-expected.png:
  • platform/mac/fast/forms/basic-selects-expected.txt:
2:22 PM Changeset in webkit [52584] by krit@webkit.org
  • 19 edits
    4 adds in trunk

2009-12-27 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVG: Stop rendering of objects with empty mask and filter content
​https://bugs.webkit.org/show_bug.cgi?id=32968

It's not neccessary to continue rendering of an object and its' childs
if the mask clips the complete content away.
The same for filters, if the filter rendering is discontinued or either
width or height are zero.
We might also stop rendering if repaintRectInLocalCoordinates is empty.
But it needs to be fixed first.

Test: svg/custom/empty-mask.svg

  • rendering/RenderPath.cpp: (WebCore::RenderPath::paint):
  • rendering/RenderSVGContainer.cpp: (WebCore::RenderSVGContainer::paint):
  • rendering/RenderSVGImage.cpp: (WebCore::RenderSVGImage::paint):
  • rendering/RenderSVGRoot.cpp: (WebCore::RenderSVGRoot::paint):
  • rendering/SVGRenderSupport.cpp: (WebCore::SVGRenderBase::prepareToRenderSVGContent):
  • rendering/SVGRenderSupport.h:
  • rendering/SVGRootInlineBox.cpp: (WebCore::SVGRootInlineBox::paint):
  • svg/SVGMaskElement.cpp: (WebCore::SVGMaskElement::drawMaskerContent):
  • svg/SVGMaskElement.h:
  • svg/graphics/SVGResourceFilter.cpp: (WebCore::SVGResourceFilter::prepareFilter): (WebCore::SVGResourceFilter::applyFilter):
  • svg/graphics/SVGResourceFilter.h:
  • svg/graphics/SVGResourceMasker.cpp: (WebCore::SVGResourceMasker::SVGResourceMasker): (WebCore::SVGResourceMasker::applyMask):
  • svg/graphics/SVGResourceMasker.h:

2009-12-27 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVG: Stop rendering of objects with empty mask and filter content
​https://bugs.webkit.org/show_bug.cgi?id=32968

Check if the content is not drawn, if the mask clips the complete
content away.
mask-excessive-malloc needed an update. This patch stops rendering
if there is no content in mask. But it's intention is to test
big masking images.

  • platform/mac/svg/custom/empty-mask-expected.checksum: Added.
  • platform/mac/svg/custom/empty-mask-expected.png: Added.
  • platform/mac/svg/custom/empty-mask-expected.txt: Added.
  • platform/mac/svg/custom/mask-excessive-malloc-expected.checksum:
  • platform/mac/svg/custom/mask-excessive-malloc-expected.png:
  • platform/mac/svg/custom/mask-excessive-malloc-expected.txt:
  • svg/custom/empty-mask.svg: Added.
  • svg/custom/mask-excessive-malloc.svg:
11:25 AM Changeset in webkit [52583] by dbates@webkit.org
  • 1 edit
    1 add in trunk/WebKitTools

2009-12-27 Daniel Bates <​dbates@webkit.org>

Unreviewed. Added missing file style_unittest.py that wasn't committed
in change set 52541 (​http://trac.webkit.org/changeset/52541) as part of
the patch for bug #32592.

  • Scripts/modules/style_unittest.py: Added.
11:12 AM Changeset in webkit [52582] by dbates@webkit.org
  • 2 edits in trunk/WebKitTools

2009-12-27 Daniel Bates <​dbates@webkit.org>

Reviewed by Eric Seidel.

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

[bzt] Updated the unit test based on the change made in bug #32951.

The change made in bug #32951 added the command-line argument:
--makeargs="-j8" to the build-webkit command. However, the associated
unit test was not updated.

  • Scripts/modules/webkitport_unittest.py:
10:19 AM Changeset in webkit [52581] by christian@webkit.org
  • 2 edits in trunk/WebCore

2009-12-27 Christian Dywan <​christian@twotoasts.de>

Reviewed by Eric Seidel.

[GTK] Spell Check Suggestions (Do not work)
​https://bugs.webkit.org/show_bug.cgi?id=30908

  • platform/gtk/ContextMenuItemGtk.cpp: (WebCore::ContextMenuItem::ContextMenuItem): (WebCore::ContextMenuItem::title): (WebCore::ContextMenuItem::setTitle): Implement the 'title' member and set the title from the native menu item.
9:08 AM Changeset in webkit [52580] by krit@webkit.org
  • 34 edits
    4 adds in trunk

2009-12-27 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVG Mask result wrong, if two different objects call the same mask id
​https://bugs.webkit.org/show_bug.cgi?id=32787

At the moment we create one resource per maskId. Some resources (like mask)
create ImageBuffers and depends on the properties of the RenderObject,
thus we can't use it for a second object.
This patch stores multiple mask resources for one maskId and different
RenderObject can use the same maskId.
Clipper, Filter and Patterns also need to store multiple resources later.

Test: svg/custom/mask-on-multiple-objects.svg

  • rendering/RenderPath.cpp: (WebCore::RenderPath::drawMarkersIfNeeded):
  • rendering/RenderSVGContainer.cpp: (WebCore::RenderSVGContainer::selfWillPaint):
  • rendering/RenderSVGGradientStop.cpp: (WebCore::RenderSVGGradientStop::styleDidChange):
  • rendering/RenderSVGRoot.cpp: (WebCore::RenderSVGRoot::selfWillPaint):
  • rendering/SVGRenderSupport.cpp: (WebCore::SVGRenderBase::prepareToRenderSVGContent): (WebCore::SVGRenderBase::filterBoundingBoxForRenderer):
  • rendering/SVGRenderTreeAsText.cpp: (WebCore::writeRenderResources):
  • svg/SVGClipPathElement.cpp: (WebCore::SVGClipPathElement::canvasResource):
  • svg/SVGClipPathElement.h:
  • svg/SVGFilterElement.cpp: (WebCore::SVGFilterElement::canvasResource):
  • svg/SVGFilterElement.h:
  • svg/SVGGradientElement.cpp: (WebCore::SVGGradientElement::canvasResource):
  • svg/SVGGradientElement.h:
  • svg/SVGMarkerElement.cpp: (WebCore::SVGMarkerElement::canvasResource):
  • svg/SVGMarkerElement.h:
  • svg/SVGMaskElement.cpp: (WebCore::SVGMaskElement::svgAttributeChanged): (WebCore::SVGMaskElement::childrenChanged): (WebCore::SVGMaskElement::canvasResource):
  • svg/SVGMaskElement.h:
  • svg/SVGPatternElement.cpp: (WebCore::SVGPatternElement::canvasResource):
  • svg/SVGPatternElement.h:
  • svg/SVGStyledElement.cpp: (WebCore::SVGStyledElement::invalidateResourcesInAncestorChain):
  • svg/SVGStyledElement.h: (WebCore::SVGStyledElement::canvasResource):
  • svg/graphics/SVGPaintServer.cpp: (WebCore::getPaintServerById): (WebCore::SVGPaintServer::fillPaintServer): (WebCore::SVGPaintServer::strokePaintServer):
  • svg/graphics/SVGPaintServer.h:
  • svg/graphics/SVGResource.cpp: (WebCore::getResourceById):
  • svg/graphics/SVGResource.h:
  • svg/graphics/SVGResourceClipper.cpp: (WebCore::getClipperById):
  • svg/graphics/SVGResourceClipper.h:
  • svg/graphics/SVGResourceFilter.cpp: (WebCore::getFilterById):
  • svg/graphics/SVGResourceFilter.h:
  • svg/graphics/SVGResourceMarker.cpp: (WebCore::getMarkerById):
  • svg/graphics/SVGResourceMarker.h:
  • svg/graphics/SVGResourceMasker.cpp: (WebCore::getMaskerById):
  • svg/graphics/SVGResourceMasker.h:

2009-12-27 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVG Mask result wrong, if two different objects call the same mask id
​https://bugs.webkit.org/show_bug.cgi?id=32787

Two rects with different sizes and positions call the same mask id. Both
rects should be masked to two circles, dependent on the properties of the
rects.

  • platform/mac/svg/custom/mask-on-multiple-objects-expected.checksum: Added.
  • platform/mac/svg/custom/mask-on-multiple-objects-expected.png: Added.
  • platform/mac/svg/custom/mask-on-multiple-objects-expected.txt: Added.
  • svg/custom/mask-on-multiple-objects.svg: Added.
6:38 AM Changeset in webkit [52579] by Nikolas Zimmermann
  • 5 edits in trunk/WebCore

2009-12-27 Nikolas Zimmermann <​nzimmermann@rim.com>

Reviewed by Dirk Schulze.

Cleanup RenderPath code, related to markers. Pass around
PaintInfo objects instead of GraphicsContext directly, to
avoid having to recreate a new PaintInfo object in
SVGResourceMarker::draw().

No layout test changes.

5:27 AM Changeset in webkit [52578] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

Rubber-stamped by Simon Hausmann.

[Qt] DRT sideeffect (authentication)
​https://bugs.webkit.org/show_bug.cgi?id=32961

  • platform/qt/Skipped: http/tests/xmlhttprequest/failed-auth.html added temorarily (until fix).
5:15 AM Changeset in webkit [52577] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

Rubber-stamped by Simon Hausmann.

Patch by Robert Hogan <​robert@roberthogan.net> on 2009-12-27
Remove successful tests from qt skipped list

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

  • platform/qt/Skipped:
2:48 AM QtBackLog edited by zecke@selfish.org
Add OProfile JIT agent to the list of things to do (diff)
1:42 AM Changeset in webkit [52576] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

Rubber-stamped by Simon Hausmann.

Patch by Robert Hogan <​robert@roberthogan.net> on 2009-12-27
Remove duplicate entries from Qt skipped list

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

  • platform/qt/Skipped:
1:11 AM Changeset in webkit [52575] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2009-12-27 Yael Aharon <​yael.aharon@nokia.com>

Reviewed by Darin Adler.

check-webkit-style should not mark moc files inclusion as errors
​https://bugs.webkit.org/show_bug.cgi?id=32669

Add a new header type for moc files, and skip them when checking the order of header files.

  • Scripts/modules/cpp_style.py:
12:55 AM Changeset in webkit [52574] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-12-27 Patrick Gansterer <​paroga@paroga.com>

Reviewed by Eric Seidel.

WinCE buildfix (added missing header)

  • platform/text/wince/TextBreakIteratorWince.cpp:
12:47 AM Changeset in webkit [52573] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-12-27 Patrick Gansterer <​paroga@paroga.com>

Reviewed by Eric Seidel.

Buildfix after r50760 and some style fixes.

  • rendering/RenderThemeWince.cpp: (WebCore::RenderThemeWince::paintSearchFieldCancelButton): (WebCore::RenderThemeWince::paintSliderTrack): (WebCore::RenderThemeWince::paintSliderThumb): (WebCore::RenderThemeWince::paintMediaMuteButton): (WebCore::RenderThemeWince::paintMediaPlayButton):
12:39 AM Changeset in webkit [52572] by eric@webkit.org
  • 4 edits in trunk/WebCore

2009-12-27 Patrick Gansterer <​paroga@paroga.com>

Reviewed by Adam Barth.

WinCE buildfixes (missing headers and unsupported HWND_MESSAGE)

  • platform/wince/MIMETypeRegistryWince.cpp:
  • platform/wince/SearchPopupMenuWince.cpp:
  • platform/wince/SharedTimerWince.cpp: (WebCore::initializeOffScreenTimerWindow):
12:32 AM Changeset in webkit [52571] by eric@webkit.org
  • 2 edits in trunk/JavaScriptCore

2009-12-27 Patrick Gansterer <​paroga@paroga.com>

Reviewed by Adam Barth.

WinCE buildfix (HWND_MESSAGE isn't supported there)

  • wtf/win/MainThreadWin.cpp: (WTF::initializeMainThreadPlatform):
12:24 AM Changeset in webkit [52570] by eric@webkit.org
  • 1 edit
    1 add in trunk/JavaScriptCore

2009-12-27 Patrick Gansterer <​paroga@paroga.com>

Reviewed by Adam Barth.

Added a file with WinMain function to link agains in WinCE.

  • os-win32/WinMain.cpp: Added. (convertToUtf8): (WinMain):

Dec 26, 2009:

11:36 PM Changeset in webkit [52569] by eric@webkit.org
  • 2 edits in trunk/WebCore

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

No review, rolling out r52554.
​http://trac.webkit.org/changeset/52554

​https://bugs.webkit.org/show_bug.cgi?id=32955
The original change was covered under
​https://bugs.webkit.org/show_bug.cgi?id=32913

r52554 was itself a rollout of r52536.
However it caused two layout tests to fail on Leopard:
storage/open-database-while-transaction-in-progress.html
svg/W3C-SVG-1.1/filters-conv-01-f.svg (no clue why?)

So I'm rolling out the rollout in an attempt to get the bots green.
r52554 did not explain why it was rolled out, so I do not know
what might fail for Chromium after this change. I warned the folks
in #chromium that it was coming.

  • storage/Database.cpp: (WebCore::Database::openDatabase): (WebCore::Database::Database): (WebCore::Database::performOpenAndVerify):
10:50 PM Changeset in webkit [52568] by abarth@webkit.org
  • 2 edits in trunk/WebKitTools

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

Reviewed by Eric Seidel.

[bzt] Make the qt build go fast
​https://bugs.webkit.org/show_bug.cgi?id=32951

Make use of multiple cores, if available.

  • Scripts/modules/webkitport.py:
7:35 PM QtBackLog edited by zecke@selfish.org
(diff)
5:28 PM Changeset in webkit [52567] by mitz@apple.com
  • 2 edits in trunk

Added Radar URLs

5:27 PM Changeset in webkit [52566] by mitz@apple.com
  • 3 edits
    4 adds in trunk

REGRESSION (r47255): Extra Large Amount of Empty Space
​https://bugs.webkit.org/show_bug.cgi?id=32690

Reviewed by Sam Weinig.

WebCore:

Test: fast/block/float/clear-to-fit.html

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::getClearDelta): Only clear floats as needed to
fit the child on the line.

LayoutTests:

  • fast/block/float/clear-to-fit-expected.checksum: Added.
  • fast/block/float/clear-to-fit-expected.png: Added.
  • fast/block/float/clear-to-fit-expected.txt: Added.
  • fast/block/float/clear-to-fit.html: Added.
1:40 AM Changeset in webkit [52565] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

Rubber-stamped by Simon Hausmann.

[Qt] Skipping some flakey tests.
​https://bugs.webkit.org/show_bug.cgi?id=32946

  • platform/qt/Skipped:
    • http/tests/cookies/simple-cookies-expired.html added.
    • storage/database-lock-after-reload.html added.
    • storage/empty-statement.html added.
    • storage/execute-sql-args.html added.
    • storage/hash-change-with-xhr.html added.
Note: See TracTimeline for information about the timeline view.