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

Timeline



Mar 31, 2010:

11:35 PM Changeset in webkit [56899] by Chris Jerdonek
  • 4 edits
    3 adds in trunk/WebKitTools

Added Python style checking to check-webkit-style using
the third-party pep8 module (via autoinstall).

Reviewed by Dave Levin.

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

  • Scripts/webkitpy/style/checker.py:
    • Added PYTHON to FileType.
    • Updated ProcessorDispatcher to return a PythonProcessor for *.py files.
  • Scripts/webkitpy/style/checker_unittest.py:
    • Updated the ProcessorDispatcher unit tests for *.py files.
  • Scripts/webkitpy/style/processors/python.py: Added.
    • Added PythonProcessor class.
  • Scripts/webkitpy/style/processors/python_unittest.py: Added.
    • Added PythonProcessor unit tests.
  • Scripts/webkitpy/style/processors/python_unittest_input.py: Added.
    • Added a sample Python file to test the PythonProcessor.process() code path (since pep8 accepts a file path).
  • Scripts/webkitpy/style/unittests.py:
    • Updated the style unit test file to import python_unittest.py.
  • Scripts/webkitpy/style_references.py:
    • Adjusted style references to import pep8.
11:23 PM Changeset in webkit [56898] by eric@webkit.org
  • 2 edits in trunk/LayoutTests

2010-03-31 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

fast/replaced/border-radius-clip.html failed on Snow Leopard Debug bot
​https://bugs.webkit.org/show_bug.cgi?id=36642

This test is flaky (seemingly due to <video> load times)
and is thus useless until fixed. Skipping for now.

  • platform/mac/Skipped:
11:09 PM Changeset in webkit [56897] by Chris Jerdonek
  • 9 edits
    1 add
    2 deletes in trunk/WebKitTools

Rewrote autoinstall.py to support unzipping *.zip files after
download, unzipping and extracting *.tar.gz files after download,
and copying installed files to a given destination directory.

Reviewed by Dave Levin.

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

These changes will let us autoinstall pep8.py from the web and put
our third-party autoinstalled code in an explicit directory like
webkitpy/thirdparty/autoinstalled. These changes should also speed
up the execution of autoinstalled *.zip packages slightly since
*.pyc files cannot be generated when importing from zipped
packages using the current autoinstall.

  • Scripts/test-webkitpy:
    • Addressed the FIXME to enable auto-install logging once autoinstall was rewritten not to log as verbosely.
  • Scripts/webkitpy/common/net/bugzilla.py:
    • Updated mechanize import statement.
  • Scripts/webkitpy/common/net/buildbot.py:
    • Updated mechanize import statement.
  • Scripts/webkitpy/common/net/irc/ircbot.py:
    • Updated ircbot and irclib import statements.
  • Scripts/webkitpy/common/net/networktransaction.py:
    • Updated mechanize import statement.
  • Scripts/webkitpy/common/net/networktransaction_unittest.py:
    • Updated mechanize import statement.
  • Scripts/webkitpy/common/net/statusserver.py:
    • Updated mechanize import statement.
  • Scripts/webkitpy/common/system/autoinstall.py: Added.
    • Added AutoInstaller class.
    • Added sample/testing usage to the main block.
  • Scripts/webkitpy/thirdparty/init.py:
    • Updated the autoinstall lines to use the new autoinstall methods.
    • Added pep8.py to the list of auto-installed packages.
    • Added a README file to the target autoinstallation directory so users know that the directory is safe to delete.
  • Scripts/webkitpy/thirdparty/autoinstall.py: Removed.
    • This is replaced by the rewritten autoinstall webkitpy/common/system/autoinstall.py.
  • Scripts/webkitpy/thirdparty/autoinstalled/init.py: Removed.
    • The target autoinstallation directory is now auto-generated.
10:22 PM Changeset in webkit [56896] by rolandsteiner@chromium.org
  • 1 edit
    6 adds in trunk/LayoutTests

2010-04-01 Roland Steiner <​rolandsteiner@chromium.org>

Unreviewed - add missing Chromium layout test results for:

Bug 36803 - 'Inline-table' workaround for <ruby> broken with ruby support
​https://bugs.webkit.org/show_bug.cgi?id=36803

(original patch review by Darin Adler).

  • platform/chromium-linux/fast/ruby/ruby-inline-table-expected.checksum: Added.
  • platform/chromium-linux/fast/ruby/ruby-inline-table-expected.png: Added.
  • platform/chromium-linux/fast/ruby/ruby-inline-table-expected.txt: Added.
  • platform/chromium-win/fast/ruby/ruby-inline-table-expected.checksum: Added.
  • platform/chromium-win/fast/ruby/ruby-inline-table-expected.png: Added.
  • platform/chromium-win/fast/ruby/ruby-inline-table-expected.txt: Added.
10:02 PM Changeset in webkit [56895] by eric@webkit.org
  • 3 edits
    2 adds in trunk

2010-03-31 MORITA Hajime <​morrita@google.com>

Reviewed by Darin Adler.

Crash when writing into a detached TITLE element
​https://bugs.webkit.org/show_bug.cgi?id=25567

  • fast/dom/title-content-write-set-expected.txt: Added.
  • fast/dom/title-content-write-set.html: Added.

2010-03-31 MORITA Hajime <​morrita@google.com>

Reviewed by Darin Adler.

Crash when writing into a detached TITLE element
​https://bugs.webkit.org/show_bug.cgi?id=25567

Document::setTitle() invoked HTMLTitleElement::setText(), which
contains DOM tree modification, even when setTitle() is called
from HTMLTitleElement::childrenChanged(). Fix to skip setText()
when setTitle() is called childrenChanged() to avoid cascading
DOM mutations between Document and HTMLTitleElement.

Test: fast/dom/title-content-write-set.html

  • dom/Document.cpp: (WebCore::Document::setTitle):
9:50 PM Changeset in webkit [56894] by Chris Jerdonek
  • 2 edits in trunk/WebKitTools

Tweak webkitpy's logtesting.LogTesting class to get more mileage out
of our unit tests that test log messages.

Reviewed by Shinichiro Hamaji.

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

This patch adds to the LogTesting class's tearDown() method a line
asserting that the array of remaining log messages is empty. This
ensures that no extra log messages are getting logged that the caller
might not be aware of or may have forgotten to check for.

  • Scripts/webkitpy/common/system/logtesting.py:
    • Modified the tearDown() method as described above.
    • Also modified the assertMessages() method to clear the array of log messages after asserting.
9:22 PM Changeset in webkit [56893] by Chris Jerdonek
  • 4 edits in trunk/WebKitTools

Added support for a --verbose-logging flag to test-webkitpy.

Reviewed by Adam Barth.

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

Verbose logging is useful for debugging test-webkitpy code that
runs before the actual unit tests -- things like autoinstalling and
unit-test auto-detection logic. This is different from verbose
logging of the unit tests themselves (which corresponds to the
unittest module's --verbose flag).

  • Scripts/test-webkitpy:
    • In the configure_logging() method--
      • Added an is_verbose_logging parameter that sets the logging level to logging.DEBUG instead of logging.INFO.
      • Changed the method to throttle the logging level on the root logger's handler instead of directly on the root logger itself.
      • Enabled logging of the autoinstall module when the flag is set.
  • Scripts/webkitpy/thirdparty/autoinstalled/init.py:
    • Added a work-around for a bug in Python 2.6's logging module that was discovered while working on this patch.
8:56 PM Changeset in webkit [56892] by Chris Jerdonek
  • 7 edits in trunk/WebKitTools

Enabled Python's logging module for webkit-patch, and replaced
deprecated_logging with Python logging in networktransaction.py.
This eliminates some spurious output when running test-webkitpy.

Reviewed by Adam Barth.

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

  • Scripts/webkit-patch:
    • Replaced the configure_logging() function with a call to the new logutils.configure_logging() function.
  • Scripts/webkitpy/common/net/networktransaction.py:
    • Replaced the use of deprecated_logging with Python logging.
  • Scripts/webkitpy/common/system/logutils.py:
    • Added _default_handlers() which creates the default logging handler for webkitpy.
    • Added configure_logging() which configures default logging for webkitpy.
  • Scripts/webkitpy/common/system/logutils_unittest.py:
    • Added unit tests for logutils.configure_logging().
  • Scripts/webkitpy/style/checker.py:
    • Refactored check-webkit-style's configure_logging() method to call the new logutils.configure_logging().
  • Scripts/webkitpy/style_references.py:
    • Updated references as necessary.
8:49 PM Changeset in webkit [56891] by eric@webkit.org
  • 3 edits in trunk/WebKitTools

2010-03-31 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

webkit-patch land fails if not run from the root directory
​https://bugs.webkit.org/show_bug.cgi?id=35822

The root of the problem was that ChangeLog.init expects a path
relative to the current working directory, and SCM expects to
return paths relative to the SCM root. Fix it by converting from
SCM-relative to absolute paths in Checkout.modified_changelogs

  • Scripts/webkitpy/common/checkout/api.py:
  • Scripts/webkitpy/common/checkout/api_unittest.py:
8:28 PM Changeset in webkit [56890] by rolandsteiner@chromium.org
  • 3 edits
    4 adds in trunk

2010-03-29 Roland Steiner <​rolandsteiner@chromium.org>

Reviewed by Darin Adler.

Bug 36803 - 'Inline-table' workaround for <ruby> broken with ruby support
​https://bugs.webkit.org/show_bug.cgi?id=36803

Don't use ruby rendering if the underlying 'display' property has been
changed into a value other than 'inline' or 'block' (such as 'inline-table').

Test: fast/ruby/ruby-inline-table.html

  • rendering/RenderObject.cpp: (WebCore::RenderObject::createObject):

Add layout test to confirm that 'inline-table' rendering is not affected
by ruby implementation.

  • fast/ruby/ruby-inline-table.html: Added.
  • platform/mac/fast/ruby/ruby-inline-table-expected.checksum: Added.
  • platform/mac/fast/ruby/ruby-inline-table-expected.png: Added.
  • platform/mac/fast/ruby/ruby-inline-table-expected.txt: Added.
8:27 PM Changeset in webkit [56889] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

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

Reviewed by Eric Seidel.

Add more tests for webkitpy.common.checkout
​https://bugs.webkit.org/show_bug.cgi?id=36926

We don't have a great way of testing checkout, sadly.

  • Scripts/webkitpy/common/checkout/api_unittest.py:
7:55 PM Changeset in webkit [56888] by abarth@webkit.org
  • 4 edits in trunk/WebKitTools

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

Reviewed by Eric Seidel.

REGRESSION: EWS crashes on failure
​https://bugs.webkit.org/show_bug.cgi?id=36924

Turns out we need to pass one more argument. My test is kind of lame,
but at least it's there.

  • Scripts/webkitpy/tool/commands/earlywarningsystem.py:
  • Scripts/webkitpy/tool/commands/earlywarningsystem_unittest.py:
7:33 PM Changeset in webkit [56887] by Chris Jerdonek
  • 1 edit in trunk/WebKitTools/ChangeLog

Unreviewed.

Fixed typo in WebKitTools/ChangeLog: opsys -> ospath.

7:08 PM Changeset in webkit [56886] by eric@webkit.org
  • 7 edits in trunk/LayoutTests

2010-03-31 Zhenyao Mo <​zmo@google.com>

Reviewed by Eric Seidel.

Fix 3 failed tests on Snow Leopard bot from 33416
​https://bugs.webkit.org/show_bug.cgi?id=36910

  • fast/canvas/webgl/context-attributes-alpha-depth-stencil-antialias-expected.txt: Considering the case where EXT_packed_depth_stencil is unsupported.
  • fast/canvas/webgl/context-attributes-alpha-depth-stencil-antialias.html: Ditto.
  • fast/canvas/webgl/context-attributes-expected.txt: Ditto.
  • fast/canvas/webgl/context-attributes.html: Ditto.
  • fast/canvas/webgl/gl-get-calls-expected.txt: Ditto.
  • fast/canvas/webgl/gl-get-calls.html: Ditto.
6:54 PM Changeset in webkit [56885] by ggaren@apple.com
  • 4 edits
    2 adds in trunk

Crash submitting display:none textarea in a form
​https://bugs.webkit.org/show_bug.cgi?id=36905

Reviewed by Darin Adler.

WebCore:

Test: fast/forms/textarea-submit-crash.html

  • html/HTMLTextAreaElement.cpp:

(WebCore::HTMLTextAreaElement::appendFormData): Do update layout before
asking our renderer for its text, since we can't rely on our renderer's
text if layout is needed.

  • rendering/RenderTextControl.cpp:

(WebCore::RenderTextControl::textWithHardLineBreaks): Don't update layout
while being asked for our text, since doing so may delete us, causing a crash.

LayoutTests:

  • fast/forms/textarea-submit-crash-expected.txt: Added.
  • fast/forms/textarea-submit-crash.html: Added.
6:43 PM Changeset in webkit [56884] by eric@webkit.org
  • 3 edits in trunk/WebKitTools

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

Reviewed by Eric Seidel.

Make the EWS go faster by being optimistic
​https://bugs.webkit.org/show_bug.cgi?id=36916

Have the EWS be optimistic that a patch will correctly build. This
should speed up the common case by not requiring two builds for every
patch.

  • Scripts/webkitpy/tool/commands/earlywarningsystem.py:
  • Scripts/webkitpy/tool/commands/queues.py:
6:13 PM Changeset in webkit [56883] by eric@webkit.org
  • 5 edits in trunk/WebKitTools

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

Reviewed by Eric Seidel.

Add the ability to restart sheriffbot from IRC
​https://bugs.webkit.org/show_bug.cgi?id=36909

  • Scripts/webkitpy/tool/bot/irc_command.py:
  • Scripts/webkitpy/tool/bot/queueengine.py:
  • Scripts/webkitpy/tool/bot/sheriffircbot.py:
  • Scripts/webkitpy/tool/bot/sheriffircbot_unittest.py:
5:58 PM Changeset in webkit [56882] by Chris Fleizach
  • 8 edits
    2 adds in trunk

WebCore: Bug 36845 - AX: need a way to set the label of a AXWebArea through WebKit
​https://bugs.webkit.org/show_bug.cgi?id=36845

Reviewed by Darin Adler.

Provide a way through WebKit to set an accessible label that describes the web area.

Test: platform/mac/accessibility/html-with-aria-label.html

  • accessibility/AccessibilityObject.h:

(WebCore::AccessibilityObject::setAccessibleName):

  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::accessibilityDescription):
(WebCore::AccessibilityRenderObject::setAccessibleName):

  • accessibility/AccessibilityRenderObject.h:

WebKit/mac: Bug 36845 - AX: need a way to set the label of a AXWebArea through WebKit
​https://bugs.webkit.org/show_bug.cgi?id=36845

Reviewed by Darin Adler.

Provide a way through WebKit to set an accessible label that describes the web area.

  • WebView/WebFrame.mm:

(-[WebFrame setAccessibleName:]):

  • WebView/WebFramePrivate.h:

LayoutTests: Test for
Bug 36845 - AX: need a way to set the label of a AXWebArea through WebKit
​https://bugs.webkit.org/show_bug.cgi?id=36845

Reviewed by Darin Adler.

  • platform/mac/accessibility/html-with-aria-label-expected.txt: Added.
  • platform/mac/accessibility/html-with-aria-label.html: Added.
5:47 PM Changeset in webkit [56881] by eric@webkit.org
  • 4 edits in trunk/WebKitTools

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

Reviewed by Eric Seidel.

Add win-ews to QueueStatusServer
​https://bugs.webkit.org/show_bug.cgi?id=36876

The win-ews is still experimental, but it seems to be more or less
running. We should show its results to the people.

  • QueueStatusServer/model/queues.py:
  • QueueStatusServer/templates/dashboard.html:
  • QueueStatusServer/templates/statusbubble.html:
5:23 PM Changeset in webkit [56880] by hclam@chromium.org
  • 2 edits in trunk/WebKit/chromium

Not reviewed. Build fix.

Fixing a build break caused by 56872. One of the Chromium bots doesn't
like std::strstr(). Includes string.h and use strstr() instead.

  • src/WebGraphicsContext3DDefaultImpl.cpp:

(WebKit::WebGraphicsContext3DDefaultImpl::validateAttributes):
Includes string.h and use strstr().

5:13 PM Changeset in webkit [56879] by mrowe@apple.com
  • 5 edits in trunk

<​http://webkit.org/b/36878> REGRESSION: Trailing colon on hostnames (with no port specified) causes "Not allowed to use restricted network port"

Reviewed by Darin Adler.

WebCore:

  • platform/KURL.cpp:

(WebCore::KURL::port): Explicitly handle the case of a colon being present in the URL after the host name but with
no port number before the path. This is handled in the same manner as the colon and port being omitted completely.

LayoutTests:

Add explicit tests for handling of allowed ports. We currently only test the ports that map to port 80: 0, 80, and "".

  • platform/mac/security/block-test-expected.txt:
  • security/block-test.html:
5:12 PM Changeset in webkit [56878] by barraclough@apple.com
  • 9 edits in trunk/WebCore

Remove dependency on TextBreakIterator from StringImpl.cpp,
and from member methods of String.

Reviewed by Oliver Hunt.

Make 'numGraphemeClusters' & 'numCharactersInGraphemeClusters'
global function defined in PlatformString.h, rather than
member methods on String (these can be implemented purely
using the characters() and length() public interface),
and make 'makeCapitalized' a static function in RenderText.cpp.

  • dom/InputElement.cpp:

(WebCore::InputElement::sanitizeUserInputValue):
(WebCore::InputElement::handleBeforeTextInsertedEvent):

  • html/HTMLInputElement.cpp:

(WebCore::HTMLInputElement::tooLong):

  • html/HTMLTextAreaElement.cpp:

(WebCore::HTMLTextAreaElement::handleBeforeTextInsertedEvent):
(WebCore::HTMLTextAreaElement::sanitizeUserInputValue):
(WebCore::HTMLTextAreaElement::tooLong):

  • platform/text/PlatformString.h:

(WebCore::String::makeSecure):

  • platform/text/String.cpp:

(WebCore::numGraphemeClusters):
(WebCore::numCharactersInGraphemeClusters):

  • platform/text/StringImpl.cpp:
  • platform/text/StringImpl.h:
  • rendering/RenderText.cpp:

(WebCore::makeCapitalized):
(WebCore::RenderText::setTextInternal):

5:09 PM Changeset in webkit [56877] by eric@webkit.org
  • 4 edits in trunk/WebCore

2010-03-31 Vitaly Repeshko <​vitalyr@chromium.org>

Reviewed by David Levin.

[V8] SerializedScriptValue must be deserialized only once and in the right context
​https://bugs.webkit.org/show_bug.cgi?id=36892

See also ​https://bugs.webkit.org/show_bug.cgi?id=34227 for the
corresponding JSC change.

General idea: SerializedScriptValue must be deserialized only once
and in the context of the intended MessageEvent recepient. The
approach we take for now is to eagerly deserialize when a
JavaScript wrapper for MessageEvent is created.

A better fix would be to keep a reference to the context in
MessageEvent and use it when lazily deserializing. It's harder to
do since the API doesn't have a clean method to have such a reference.

Tested by fast/dom/Window/window-postmessage-clone-frames.html. This
test still fails but only for the types which we can't serialize yet.

  • bindings/scripts/CodeGeneratorV8.pm:
  • bindings/v8/SerializedScriptValue.h: (WebCore::SerializedScriptValue::deserializeAndSetProperty):
  • bindings/v8/custom/V8MessageEventCustom.cpp: (WebCore::V8MessageEvent::initMessageEventCallback):
4:45 PM Changeset in webkit [56876] by victorw@chromium.org
  • 4 edits in trunk/WebKitTools

2010-03-30 Victor Wang <​victorw@chromium.org>

Reviewed by Eric Seidel.

Update rebaseline tool to check the release image diff binary and
fallback to debug if the release version does not exist.

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

  • Scripts/webkitpy/layout_tests/port/base.py:
  • Scripts/webkitpy/layout_tests/port/chromium.py:
  • Scripts/webkitpy/layout_tests/rebaseline_chromium_webkit_tests.py:
4:22 PM Changeset in webkit [56875] by eric@webkit.org
  • 3 edits in trunk/WebCore

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

Reviewed by Darin Fisher.

Cleanup RedirectScheduler
​https://bugs.webkit.org/show_bug.cgi?id=36874

Removed the nutty ScheduledRedirection struct in favor of a hierarchy
of classes to represent the various kinds of scheduled redirects.
Doing this lets us get rid of the pseudo RTTI switch statements on
"type" in favour of calling virtual functions.

No new tests because this change should be API identical with the
existing RedirectScheduler.

  • loader/RedirectScheduler.cpp: (WebCore::ScheduledNavigation::ScheduledNavigation): (WebCore::ScheduledNavigation::~ScheduledNavigation): (WebCore::ScheduledNavigation::isLocationChange): (WebCore::ScheduledNavigation::shouldStartTimer): (WebCore::ScheduledNavigation::didStartTimer): (WebCore::ScheduledNavigation::didStopTimer): (WebCore::ScheduledNavigation::delay): (WebCore::ScheduledNavigation::lockHistory): (WebCore::ScheduledNavigation::lockBackForwardList): (WebCore::ScheduledNavigation::wasDuringLoad): (WebCore::ScheduledURLNavigation::ScheduledURLNavigation): (WebCore::ScheduledURLNavigation::fire): (WebCore::ScheduledURLNavigation::didStartTimer): (WebCore::ScheduledURLNavigation::didStopTimer): (WebCore::ScheduledURLNavigation::url): (WebCore::ScheduledURLNavigation::referrer): (WebCore::ScheduledURLNavigation::wasUserGesture): (WebCore::ScheduledRedirect::ScheduledRedirect): (WebCore::ScheduledRedirect::isLocationChange): (WebCore::ScheduledRedirect::shouldStartTimer): (WebCore::ScheduledLocationChange::ScheduledLocationChange): (WebCore::ScheduledRefresh::ScheduledRefresh): (WebCore::ScheduledRefresh::fire): (WebCore::ScheduledHistoryNavigation::ScheduledHistoryNavigation): (WebCore::ScheduledHistoryNavigation::fire): (WebCore::ScheduledFormSubmission::ScheduledFormSubmission): (WebCore::ScheduledFormSubmission::fire): (WebCore::RedirectScheduler::redirectScheduledDuringLoad): (WebCore::RedirectScheduler::clear): (WebCore::RedirectScheduler::scheduleRedirect): (WebCore::RedirectScheduler::mustLockBackForwardList): (WebCore::RedirectScheduler::scheduleLocationChange): (WebCore::RedirectScheduler::scheduleFormSubmission): (WebCore::RedirectScheduler::scheduleRefresh): (WebCore::RedirectScheduler::locationChangePending): (WebCore::RedirectScheduler::scheduleHistoryNavigation): (WebCore::RedirectScheduler::timerFired): (WebCore::RedirectScheduler::schedule): (WebCore::RedirectScheduler::startTimer): (WebCore::RedirectScheduler::cancel):
  • loader/RedirectScheduler.h:
3:55 PM Changeset in webkit [56874] by eric@webkit.org
  • 3 edits in trunk/WebKitTools

2010-03-31 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Teach buildbot.py how to parse builder activity from /one_box_per_builder
​https://bugs.webkit.org/show_bug.cgi?id=36898

I also removed some obsolete FIXMEs and
refactored one_box_per_builder parsing into multiple
methods for easier reading.

  • Scripts/webkitpy/common/net/buildbot.py:
  • Scripts/webkitpy/common/net/buildbot_unittest.py:
3:53 PM Changeset in webkit [56873] by scherkus@chromium.org
  • 2 edits
    10 adds in trunk/LayoutTests

2010-03-31 Andrew Scherkus <​scherkus@chromium.org>

Reviewed by Dmitry Titov.

Checking in Chromium media baselines for layout tests fixed in r56481.

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

  • platform/chromium-linux/media/video-no-audio-expected.checksum: Added.
  • platform/chromium-linux/media/video-no-audio-expected.png: Added.
  • platform/chromium-mac/media/video-no-audio-expected.checksum: Added.
  • platform/chromium-mac/media/video-no-audio-expected.png: Added.
  • platform/chromium-mac/media/video-no-audio-expected.txt: Added.
  • platform/chromium-win/media/audio-constructor-src-expected.txt: Added.
  • platform/chromium-win/media/video-no-audio-expected.checksum: Added.
  • platform/chromium-win/media/video-no-audio-expected.png: Added.
  • platform/chromium-win/media/video-no-audio-expected.txt: Added.
  • platform/chromium-win/media/video-source-add-src-expected.txt: Added.
  • platform/chromium/test_expectations.txt: Tests are now passing.
3:15 PM Changeset in webkit [56872] by eric@webkit.org
  • 15 edits
    2 adds in trunk

2010-03-31 Zhenyao Mo <​zmo@google.com>

Reviewed by Darin Fisher.

Hook up WebGLContextAttributes to OpenGL context creation code
​https://bugs.webkit.org/show_bug.cgi?id=33416

  • fast/canvas/webgl/context-attributes-alpha-depth-stencil-antialias-expected.txt: Added.
  • fast/canvas/webgl/context-attributes-alpha-depth-stencil-antialias.html: Added.
  • fast/canvas/webgl/context-attributes-expected.txt: WebGL context attributes behavior changed with this fix.
  • fast/canvas/webgl/context-attributes.html: Ditto.
  • fast/canvas/webgl/gl-get-calls-expected.txt: Stencil buffer is enabled, thus Stencil Bits is no longer 0.
  • fast/canvas/webgl/gl-get-calls.html: Ditto.

2010-03-31 Zhenyao Mo <​zmo@google.com>

Reviewed by Darin Fisher.

Hook up WebGLContextAttributes to OpenGL context creation code
​https://bugs.webkit.org/show_bug.cgi?id=33416

Test: fast/canvas/webgl/context-attributes-alpha-depth-stencil-antialias.html

  • bindings/v8/custom/V8HTMLCanvasElementCustom.cpp: Fix an index bug. (WebCore::V8HTMLCanvasElement::getContextCallback):
  • platform/graphics/GraphicsContext3D.h: Add members/functions for multisampling/stencil buffer purpose.
  • platform/graphics/mac/Canvas3DLayer.h: Add GraphicsContext3D as a member of Canvas3DLayer.
  • platform/graphics/mac/Canvas3DLayer.mm: Add multisampling support. (-[Canvas3DLayer drawInCGLContext:pixelFormat:forLayerTime:displayTime:]):
  • platform/graphics/mac/GraphicsContext3DMac.cpp: Hook up WebGLContextAttributes to OpenGL context creation code for Mac. (WebCore::GraphicsContext3D::GraphicsContext3D): (WebCore::GraphicsContext3D::~GraphicsContext3D): (WebCore::GraphicsContext3D::validateAttributes): (WebCore::GraphicsContext3D::reshape): (WebCore::GraphicsContext3D::prepareTexture): (WebCore::GraphicsContext3D::bindFramebuffer): (WebCore::GraphicsContext3D::readPixels):
  • platform/graphics/mac/GraphicsLayerCA.mm: Adjust to modified Canvas3DLayer init call. (WebCore::GraphicsLayerCA::setContentsToGraphicsContext3D):

2010-03-31 Zhenyao Mo <​zmo@google.com>

Reviewed by Darin Fisher.

Hook up WebGLContextAttributes to OpenGL context creation code
​https://bugs.webkit.org/show_bug.cgi?id=33416

  • src/WebGraphicsContext3DDefaultImpl.cpp: Hook up WebGLContextAttributes to OpenGL context creation code for Chrome. (WebKit::WebGraphicsContext3DDefaultImpl::WebGraphicsContext3DDefaultImpl): (WebKit::WebGraphicsContext3DDefaultImpl::~WebGraphicsContext3DDefaultImpl): (WebKit::WebGraphicsContext3DDefaultImpl::initialize): (WebKit::WebGraphicsContext3DDefaultImpl::validateAttributes): (WebKit::WebGraphicsContext3DDefaultImpl::reshape): (WebKit::WebGraphicsContext3DDefaultImpl::readBackFramebuffer): (WebKit::WebGraphicsContext3DDefaultImpl::bindFramebuffer): (WebKit::WebGraphicsContext3DDefaultImpl::readPixels): Deal with wrong returned alpha values in Mac.
  • src/WebGraphicsContext3DDefaultImpl.h: Add a function.
2:51 PM Changeset in webkit [56871] by dpranke@chromium.org
  • 4 edits in trunk/WebKitTools

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

Reviewed by Eric Seidel.

Fix logging in new-run-webkit-tests so that we don't report IMAGE
expected failures as unexpected passes when we run with pixel tests
disabled.

This change splits some of the logic embedded into the TestExpectations
classes out into separate pure functions (result_was_expected,
remove_image_failures) to make them easier to test. This also adds
a parameter to matches_an_expected_result() to indicate whether or
not pixel test results should be included in the expectations.

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

  • Scripts/webkitpy/layout_tests/layout_package/test_expectations.py:
  • Scripts/webkitpy/layout_tests/layout_package/test_expectations_test.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
2:44 PM Changeset in webkit [56870] by Chris Jerdonek
  • 4 edits
    2 adds in trunk/WebKitTools

In webkitpy, refactored two calls to os.path.relpath() replacements
to use a common method.

Reviewed by Eric Seidel.

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

  • Scripts/webkitpy/common/net/bugzilla.py:
    • Replaced the relpath implementation with a call to webkitpy.common.system.opsys.relpath().
  • Scripts/webkitpy/common/system/ospath.py: Added.
    • Moved the relpath() implementation from style/main.py.
  • Scripts/webkitpy/common/system/ospath_unittest.py: Added.
    • Moved the relpath() unit tests from style/main_unittest.py.
  • Scripts/webkitpy/style/main.py:
    • Replaced the relpath implementation with a call to webkitpy.common.system.opsys.relpath().
  • Scripts/webkitpy/style/main_unittest.py:
    • Moved the relpath unit tests to opsys_unittest.py.
2:23 PM Changeset in webkit [56869] by jianli@chromium.org
  • 6 edits
    2 adds in trunk/WebCore

Add support to create UUID string.
​https://bugs.webkit.org/show_bug.cgi?id=36472

Reviewed by Dmitry Titov.

  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • platform/UUID.cpp: Added.

(WebCore::createCanonicalUUIDString):

  • platform/UUID.h: Added.
2:15 PM Changeset in webkit [56868] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

Reviewed by NOBODY (windows build fix pt II).

1:59 PM Changeset in webkit [56867] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

Reviewed by NOBODY (windows build fix).

1:57 PM Changeset in webkit [56866] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

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

Reviewed by Eric Seidel.

Add a Builder.force_build method
​https://bugs.webkit.org/show_bug.cgi?id=36875

We plan to eventually use this in SheriffBot to break deadlocks created
by flaky tests.

  • Scripts/webkitpy/common/net/buildbot.py:
1:38 PM Changeset in webkit [56865] by eric@webkit.org
  • 3 edits
    2 adds in trunk/LayoutTests

2010-03-31 Qi Zhang <​qi.2.zhang@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] Enable LayoutTests/fast/canvas/zero-size-fill-rect.html
​https://bugs.webkit.org/show_bug.cgi?id=36837

Remove zero-size-fill-rect.html from skipped and create expected files.

  • platform/qt/Skipped:
  • platform/qt/fast/canvas/zero-size-fill-rect-expected.checksum: Added.
  • platform/qt/fast/canvas/zero-size-fill-rect-expected.png: Added.
  • platform/qt/fast/canvas/zero-size-fill-rect-expected.txt:
1:36 PM Changeset in webkit [56864] by barraclough@apple.com
  • 6 edits
    2 adds in trunk

Bug 36871 - Remove JSC::CString
Use WTF::CString instead (which until recently was WebCore::CString).

Reviewed by Geoff Garen.

JavaScriptCore:

JavaScriptGlue:

  • ForwardingHeaders/wtf/text: Added.
  • ForwardingHeaders/wtf/text/CString.h: Added.
1:28 PM Changeset in webkit [56863] by eric@webkit.org
  • 6 edits in trunk/WebKitTools

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

Reviewed by Eric Seidel.

webkit-patch reads wrong bug url from unified diff context
​https://bugs.webkit.org/show_bug.cgi?id=36477

Instead of trying to figure out the bug_id from the diff, we should
just get the information from the Checkout object, which understands
these concepts.

  • Scripts/webkitpy/common/checkout/api.py:
  • Scripts/webkitpy/common/checkout/commitinfo.py:
  • Scripts/webkitpy/tool/commands/download.py:
  • Scripts/webkitpy/tool/commands/upload.py:
  • Scripts/webkitpy/tool/mocktool.py:
1:28 PM Changeset in webkit [56862] by Darin Adler
  • 2 edits in trunk/WebCore
  • rendering/RenderThemeChromiumWin.h: Fix inconsistent line endings.
1:21 PM Changeset in webkit [56861] by hclam@chromium.org
  • 4 edits in trunk/LayoutTests

2010-03-31 Alpha Lam <​hclam@chromium.org>

Reviewed by Darin Fisher.

Update test results due to changes in r56824.
​https://bugs.webkit.org/show_bug.cgi?id=36900

  • platform/chromium-win/fast/forms/input-file-re-render-expected.checksum:
  • platform/chromium-win/fast/forms/input-file-re-render-expected.png:
  • platform/chromium/test_expectations.txt: fast/forms/input-file-re-render.html generates incorrect results on Linux, mark it so.
1:18 PM Changeset in webkit [56860] by eric@webkit.org
  • 5 edits
    3 adds in trunk/WebCore

2010-03-31 Luiz Agostini <​luiz.agostini@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Maemo5 theme - form controls style
​https://bugs.webkit.org/show_bug.cgi?id=36370

Adjusting Mameo5 form elements rendering.

  • WebCore.pri:
  • WebCore.pro:
  • css/themeQtMaemo5.css: Added. (select): (select:disabled): (select:active): (select:active:disabled): (textarea): (textarea:disabled): (textarea:active):
  • platform/qt/Maemo5Webstyle.cpp: Added. (Maemo5WebStyle::Maemo5WebStyle): (drawRectangularControlBackgorund): (Maemo5WebStyle::drawChecker): (Maemo5WebStyle::findChecker): (Maemo5WebStyle::drawRadio): (Maemo5WebStyle::findRadio): (Maemo5WebStyle::drawControl): (Maemo5WebStyle::drawComplexControl):
  • platform/qt/Maemo5Webstyle.h: Added.
  • platform/qt/RenderThemeQt.cpp: (WebCore::RenderThemeQt::RenderThemeQt): (WebCore::RenderThemeQt::isControlStyled): (WebCore::RenderThemeQt::popupInternalPaddingBottom): (WebCore::RenderThemeQt::extraDefaultStyleSheet): (WebCore::RenderThemeQt::adjustMenuListButtonStyle):
  • platform/qt/RenderThemeQt.h:
12:51 PM Changeset in webkit [56859] by ap@apple.com
  • 3 edits in trunk/WebCore

​https://bugs.webkit.org/show_bug.cgi?id=36897
<rdar://problem/7804018> REGRESSION (r56429): Flash ads are clipped when main page is scrolled (boxofficemojo.com)

Addressing additional review comments.

  • rendering/RenderWidget.cpp: Removed an unneeded include.
  • rendering/RenderWidget.h: Reworded the new comment.
12:48 PM Changeset in webkit [56858] by ap@apple.com
  • 4 edits
    2 adds in trunk/WebCore

Reviewed by Dan Bernstein.

​https://bugs.webkit.org/show_bug.cgi?id=36897
<rdar://problem/7804018> REGRESSION (r56429): Flash ads are clipped when main page is scrolled (boxofficemojo.com)

  • manual-tests/plugin-in-iframe-scroll.html: Added.
  • manual-tests/resources/plugin-in-iframe-scroll-iframe.html: Added.
  • rendering/RenderWidget.cpp: (WebCore::RenderWidget::setWidgetGeometry): This method isn't called when an iframe containing the widget moves, so we should store a value that is not dependent on current scroll offset. (WebCore::RenderWidget::windowClipRect): Apply scroll offset and window clip now.
  • rendering/RenderWidget.h: Renamed m_windowClipRect to m_clipRect, because it is no longer in window coordinates.
  • WebCore.base.exp: Export RenderWidget::windowClipRect(), since it's no longer inline.
12:32 PM Changeset in webkit [56857] by abarth@webkit.org
  • 2 edits in trunk/WebKitTools

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

Reviewed by Eric Seidel.

webkit-patch commit-queue should land patches optimistically
​https://bugs.webkit.org/show_bug.cgi?id=34187

This patch adds an optimistic path to commit-queue and a "fail twice"
requirement for rejecting patches. That means we'll land good patches
faster (via the optmistic first run) and we'll reject many fewer
patches due to flaky tests.

  • Scripts/webkitpy/tool/commands/queues.py:
12:17 PM Changeset in webkit [56856] by kenneth@webkit.org
  • 3 edits in trunk/WebCore

Unreviewed crash fix.

Crash with frame flattening on after r56854
​https://bugs.webkit.org/show_bug.cgi?id=36894

Solution tested by Greg Bolsinga, thanks.

  • page/FrameView.cpp:

(WebCore::FrameView::avoidScrollbarCreation):

  • platform/ScrollView.cpp:

(WebCore::ScrollView::setHasHorizontalScrollbar):
(WebCore::ScrollView::setHasVerticalScrollbar):

12:12 PM PythonGuidelines edited by Chris Jerdonek
Retitled, and clarified the minimum Python version requirement. (diff)
11:46 AM Changeset in webkit [56855] by Csaba Osztrogonác
  • 4 edits
    8 adds in trunk/LayoutTests

Rubber-stamped by Kenneth Rohde Christiansen.

iframe flattening doesn't flatten
​https://bugs.webkit.org/show_bug.cgi?id=36798

Platform specific expected files added, and updated after r56852.

  • platform/mac/fast/frames/flattening/frameset-flattening-subframe-resize-expected.txt:
  • platform/mac/fast/frames/flattening/iframe-flattening-fixed-height-expected.txt: Added.
  • platform/mac/fast/frames/flattening/iframe-flattening-fixed-width-and-height-expected.txt: Added.
  • platform/mac/fast/frames/flattening/iframe-flattening-fixed-width-and-height-no-scrolling-expected.txt: Added.
  • platform/mac/fast/frames/flattening/iframe-flattening-fixed-width-expected.txt: Added.
  • platform/mac/fast/frames/flattening/iframe-flattening-simple-expected.txt:
  • platform/qt/fast/frames/flattening/frameset-flattening-subframe-resize-expected.txt:
  • platform/qt/fast/frames/flattening/iframe-flattening-fixed-height-expected.txt: Added.
  • platform/qt/fast/frames/flattening/iframe-flattening-fixed-width-and-height-expected.txt: Added.
  • platform/qt/fast/frames/flattening/iframe-flattening-fixed-width-and-height-no-scrolling-expected.txt: Added.
  • platform/qt/fast/frames/flattening/iframe-flattening-fixed-width-expected.txt: Added.
11:03 AM Changeset in webkit [56854] by kenneth@webkit.org
  • 6 edits in trunk/WebCore

Do not draw scrollbars for subframes when frame flattening is
enabled. Implemented using a virtual method in ScrollView as
suggested by Dave Hyatt.

Reviewed by Antti Koivisto.

Do not suppress scrollbars as that is wrong according to
Dave Hyatt.

Covered by current tests.

  • page/FrameView.cpp:

(WebCore::FrameView::avoidScrollbarCreation):

  • page/FrameView.h:
  • platform/ScrollView.cpp:

(WebCore::ScrollView::setHasHorizontalScrollbar):
(WebCore::ScrollView::setHasVerticalScrollbar):

  • platform/ScrollView.h:

(WebCore::ScrollView::avoidScrollbarCreation):

  • rendering/RenderPart.cpp:

(WebCore::RenderPart::layoutWithFlattening):

11:00 AM Changeset in webkit [56853] by darin@chromium.org
  • 2 edits in trunk/WebKit/chromium

2010-03-31 Darin Fisher <​darin@chromium.org>

Reviewed by David Levin.

[chromium] including WebKit.gyp:webkit should automatically add Skia
and NPAPI include paths

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

  • WebKit.gyp:
10:55 AM Changeset in webkit [56852] by kenneth@webkit.org
  • 3 edits
    4 adds in trunk

iframe flattening doesn't flatten
​https://bugs.webkit.org/show_bug.cgi?id=36798

WebCore: Fix wrongly reversing logic in frame flattening check.

Reviewed by Antti Koivisto.

Tests:
fast/frames/flattening/iframe-flattening-fixed-height.html
fast/frames/flattening/iframe-flattening-fixed-width.html
fast/frames/flattening/iframe-flattening-fixed-width-and-height.html
fast/frames/flattening/iframe-flattening-fixed-width-and-height-no-scrolling.html

  • rendering/RenderPartObject.cpp:

(WebCore::RenderPartObject::flattenFrame):

LayoutTests: Add more tests for iframe flattening

Reviewed by Antti Koivisto.

  • fast/frames/flattening/iframe-flattening-fixed-height.html
  • fast/frames/flattening/iframe-flattening-fixed-width.html
  • fast/frames/flattening/iframe-flattening-fixed-width-and-height.html
  • fast/frames/flattening/iframe-flattening-fixed-width-and-height-no-scrolling.html
10:54 AM PythonGuidelines edited by Chris Jerdonek
Eliminated the Tips section (diff)
10:47 AM Changeset in webkit [56851] by eric@webkit.org
  • 23 edits in trunk

2010-03-31 Marcus Bulach <​bulach@chromium.org>

Reviewed by Jeremy Orlow.

Implements cancelGeolocationPermissionRequestForFrame.
Similar to requestGeolocationPermissionForFrame(), passes Geolocation* to cancelGeolocationPermissionRequestForFrame(),
so that the embedder can identify which Geolocation object is cancelling the pending permission request.
Calls cancelGeolocationPermissionRequestForFrame() before stopUpdating() so it better matches the startUpdating() / requestPermission() flow.
​https://bugs.webkit.org/show_bug.cgi?id=35031

  • loader/EmptyClients.h: (WebCore::EmptyChromeClient::cancelGeolocationPermissionRequestForFrame):
  • page/Chrome.cpp: (WebCore::Chrome::cancelGeolocationPermissionRequestForFrame):
  • page/Chrome.h:
  • page/ChromeClient.h:
  • page/Geolocation.cpp: (WebCore::Geolocation::disconnectFrame):

2010-03-31 Marcus Bulach <​bulach@chromium.org>

Reviewed by Jeremy Orlow.

Implements cancelGeolocationPermissionRequestForFrame.
​https://bugs.webkit.org/show_bug.cgi?id=35031

  • public/WebGeolocationService.h: (WebKit::WebGeolocationService::cancelPermissionRequestForFrame):
  • src/ChromeClientImpl.cpp: (WebKit::ChromeClientImpl::cancelGeolocationPermissionRequestForFrame):
  • src/ChromeClientImpl.h:

2010-03-31 Marcus Bulach <​bulach@chromium.org>

Reviewed by Jeremy Orlow.

Adds Geolocation param for cancelGeolocationPermissionRequestForFrame.
​https://bugs.webkit.org/show_bug.cgi?id=35031

  • WebCoreSupport/ChromeClientGtk.cpp: (WebKit::ChromeClient::cancelGeolocationPermissionRequestForFrame):
  • WebCoreSupport/ChromeClientGtk.h:

2010-03-31 Marcus Bulach <​bulach@chromium.org>

Reviewed by Jeremy Orlow.

Adds Geolocation param for cancelGeolocationPermissionRequestForFrame.
​https://bugs.webkit.org/show_bug.cgi?id=35031

  • WebCoreSupport/ChromeClientHaiku.h: (WebCore::ChromeClientHaiku::cancelGeolocationPermissionRequestForFrame):

2010-03-31 Marcus Bulach <​bulach@chromium.org>

Reviewed by Jeremy Orlow.

Adds Geolocation param for cancelGeolocationPermissionRequestForFrame.
​https://bugs.webkit.org/show_bug.cgi?id=35031

  • WebCoreSupport/WebChromeClient.h: (WebChromeClient::cancelGeolocationPermissionRequestForFrame):

2010-03-31 Marcus Bulach <​bulach@chromium.org>

Reviewed by Jeremy Orlow.

Adds Geolocation param for cancelGeolocationPermissionRequestForFrame.
​https://bugs.webkit.org/show_bug.cgi?id=35031

  • WebCoreSupport/ChromeClientQt.h: (WebCore::ChromeClientQt::cancelGeolocationPermissionRequestForFrame):

2010-03-31 Marcus Bulach <​bulach@chromium.org>

Reviewed by Jeremy Orlow.

Adds Geolocation param for cancelGeolocationPermissionRequestForFrame.
​https://bugs.webkit.org/show_bug.cgi?id=35031

  • WebCoreSupport/WebChromeClient.h: (WebChromeClient::cancelGeolocationPermissionRequestForFrame):

2010-03-31 Marcus Bulach <​bulach@chromium.org>

Reviewed by Jeremy Orlow.

Adds Geolocation param for cancelGeolocationPermissionRequestForFrame.
​https://bugs.webkit.org/show_bug.cgi?id=35031

  • WebKitSupport/ChromeClientWx.h: (WebCore::ChromeClientWx::cancelGeolocationPermissionRequestForFrame):
10:06 AM Changeset in webkit [56850] by yael.aharon@nokia.com
  • 8 edits
    2 adds in trunk/WebCore

Add animation to progress element
​https://bugs.webkit.org/show_bug.cgi?id=36664

Reviewed by Antti Koivisto.

Add a timer to control the animation. The timer is started after painting
or a state change in the progress bar, to prevent animation from running
when the progress bar is not visible.

  • html/HTMLProgressElement.cpp:
  • platform/qt/RenderThemeQt.cpp:
  • platform/qt/RenderThemeQt.h:
  • rendering/RenderProgress.cpp:
  • rendering/RenderProgress.h:
  • rendering/RenderTheme.cpp:
  • rendering/RenderTheme.h:
10:01 AM Changeset in webkit [56849] by pfeldman@chromium.org
  • 9 edits in trunk

2010-03-31 Pavel Feldman <​pfeldman@chromium.org>

Not reviewed. Rolling out r56829 since it broke chromium layout tests.

[REGRESSION] Inspector tests started crashing since r56829
​https://bugs.webkit.org/show_bug.cgi?id=36888

  • bindings/scripts/CodeGeneratorV8.pm:
  • bindings/v8/V8Utilities.cpp:
  • bindings/v8/V8Utilities.h:
  • loader/FrameLoaderClient.h: WebKit/chromium:
  • public/WebFrameClient.h:
  • src/FrameLoaderClientImpl.cpp:
  • src/FrameLoaderClientImpl.h:
9:53 AM Changeset in webkit [56848] by eric@webkit.org
  • 2 edits in trunk/LayoutTests

2010-03-31 Sterling Swigart <​sswigart@google.com>

Reviewed by David Levin.

[Chromium] Enable fast/events/onbeforeunload-focused-iframe.html
​https://bugs.webkit.org/show_bug.cgi?id=36781

  • platform/chromium/test_expectations.txt: Enable test now that callShouldCloseOnWebView is implemented in test_shell.
9:44 AM PythonGuidelines edited by Chris Jerdonek
Added section on consequences of not using Python 2.6 (diff)
9:29 AM Changeset in webkit [56847] by cmarrin@apple.com
  • 1 edit in trunk/WebKitSite/blog-files/webgl/WebGL+CSS.html

more tweaks to WebGL+CSS

9:23 AM Changeset in webkit [56846] by pfeldman@chromium.org
  • 4 edits in trunk/WebCore

2010-03-31 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: creating new style bugfixing.

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

  • InspectorDOMAgent should clear internal state upon reset (we are re-using single dom agent instance throughout entire inspector controller lifetime)
  • Brought back blank style 'refresh' processing logic - remove it by mistake earlier
  • Blocked couple of click handlers so that double-click to edit was not expanding / collapsing the pane
  • There is no need to reach out for matched rules upon 'refresh' update - getting computed style is sufficient.
  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::matchRulesForList):
  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::discardBindings):
  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylesSidebarPane.prototype.update.getStylesCallback): (WebInspector.StylesSidebarPane.prototype.update.getComputedStyleCallback): (WebInspector.StylesSidebarPane.prototype.update): (WebInspector.StylesSidebarPane.prototype._refreshUpdate): (WebInspector.StylesSidebarPane.prototype._rebuildUpdate): (WebInspector.StylesSidebarPane.prototype._refreshStyleRules): (WebInspector.StylesSidebarPane.prototype.addBlankSection): (WebInspector.StylePropertiesSection.prototype._dblclickEmptySpace): (WebInspector.StylePropertiesSection.prototype._clickSelector):
9:22 AM PythonGuidelines edited by Chris Jerdonek
Wording improvements to upgrade section. (diff)
9:16 AM PythonGuidelines edited by Chris Jerdonek
Added outline (diff)
9:10 AM Changeset in webkit [56845] by Laszlo Gombos
  • 2 edits in trunk/JavaScriptCore

2010-03-31 Laszlo Gombos <Laszlo Gombos>

Unreviewed, fix after r56842.

Add UNUSED_PARAM a to silence warning.

  • jit/JITStubs.cpp: (JSC::DEFINE_STUB_FUNCTION):
9:09 AM Changeset in webkit [56844] by yurys@chromium.org
  • 2 edits in trunk/WebCore

2010-03-31 Yury Semikhatsky <​yurys@chromium.org>

Unreviewed. Chromium build fix: create missing v8::HandleScope.

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

  • bindings/v8/custom/V8InjectedScriptHostCustom.cpp: (WebCore::InjectedScriptHost::canAccessInspectedWindow):
9:01 AM Changeset in webkit [56843] by eric@webkit.org
  • 3 edits in trunk/LayoutTests

2010-03-31 Alexander Pavlov <​apavlov@chromium.org>

Reviewed by Pavel Feldman.

Fix crash

Web Inspector: [Chromium] inspector/audits-panel-functional.html crashes test_shell
​https://bugs.webkit.org/show_bug.cgi?id=36885

  • inspector/audits-panel-functional.html:
  • platform/chromium/test_expectations.txt:
8:59 AM Changeset in webkit [56842] by Laszlo Gombos
  • 3 edits in trunk/JavaScriptCore

2010-03-31 Laszlo Gombos <Laszlo Gombos>

Unreviewed, Symbian build fix.

Refactor JITStubs.cpp so that the list of STUB_FUNCTIONs
are not dependent on the JSVALUE32_64 guard.

  • jit/JITStubs.cpp: Place the JSVALUE32_64 guard inside the body of cti_op_eq_strings.
  • jit/JITStubs.h: Remove JSVALUE32_64 guard from cti_op_eq_strings stub.
8:24 AM Changeset in webkit [56841] by yurys@chromium.org
  • 10 edits in trunk

2010-03-31 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Pavel Feldman.

Check that injected script can still access inspected window object when wrapping console object. When the window cannot be access serialize objects as strings. Also don't call InjectedScript.dispatch if the window cannot be accessed (due to frame navigation).

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

  • bindings/js/JSInjectedScriptHostCustom.cpp: (WebCore::InjectedScriptHost::canAccessInspectedWindow):
  • bindings/v8/custom/V8InjectedScriptHostCustom.cpp: (WebCore::InjectedScriptHost::canAccessInspectedWindow):
  • inspector/InjectedScript.cpp: (WebCore::InjectedScript::wrapForConsole):
  • inspector/InjectedScriptHost.h:
  • inspector/front-end/InjectedScript.js: (injectedScriptConstructor):

2010-03-30 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: get rid of "Unsafe JavaScript attempt..." message when inspecting frames whose content has changed.

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

  • http/tests/inspector-enabled/console-log-before-frame-navigation-expected.txt:
  • http/tests/inspector-enabled/resources/console-log-frame-before-navigation.html:
7:45 AM Changeset in webkit [56840] by eric@webkit.org
  • 4 edits in trunk/WebCore

2010-03-31 Mattias Nissler <​mnissler@google.com>

Reviewed by Pavel Feldman.

Allow generic resource URLs for cookie handling in the inspector. This
enables display of all cookies, not only those that are associated
with resources accessed through http, https or file.
​https://bugs.webkit.org/show_bug.cgi?id=36877

  • inspector/front-end/CookieItemsView.js: (WebInspector.CookieItemsView.prototype._filterCookiesForDomain):
  • inspector/front-end/DOMAgent.js: (WebInspector.Cookies.cookieMatchesResourceURL):
  • inspector/front-end/inspector.js: (WebInspector.updateResource):
7:42 AM Changeset in webkit [56839] by Nikolas Zimmermann
  • 1 edit
    2 adds in trunk/LayoutTests

2010-03-31 Nikolas Zimmermann <​nzimmermann@rim.com>

Not reviewed. Add missing pixel test results for svg/filters/feColorMatrix-values.svg

  • platform/mac/svg/filters/feColorMatrix-values-expected.checksum: Added.
  • platform/mac/svg/filters/feColorMatrix-values-expected.png: Added.
7:26 AM Changeset in webkit [56838] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-03-31 Diego Gonzalez <​diego.gonzalez@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Custom user agent for user agent switcher
​https://bugs.webkit.org/show_bug.cgi?id=36757

  • QtLauncher/main.cpp: (LauncherWindow::showUserAgentDialog):
6:27 AM Changeset in webkit [56837] by antti.j.koivisto@nokia.com
  • 3 edits in trunk/WebKit/qt

​https://bugs.webkit.org/show_bug.cgi?id=36446
[Qt] QWebSettings::TiledBackingStoreEnabled attribute lacks documentation and default value

Reviewed by Kenneth Rohde Christiansen.

  • Api/qgraphicswebview.cpp:
  • Api/qwebsettings.cpp:

(QWebSettings::QWebSettings):

4:04 AM Changeset in webkit [56836] by eric@webkit.org
  • 3 edits in trunk/WebKit/chromium

2010-03-31 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Pavel Feldman.

Add stubs for moveWindowBy and setAttachedWindowHeight.

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

  • src/WebDevToolsFrontendImpl.cpp: (WebKit::WebDevToolsFrontendImpl::WebDevToolsFrontendImpl): (WebKit::WebDevToolsFrontendImpl::jsSetAttachedWindowHeight): (WebKit::WebDevToolsFrontendImpl::jsMoveWindowBy):
  • src/WebDevToolsFrontendImpl.h:
3:45 AM Changeset in webkit [56835] by eric@webkit.org
  • 3 edits in trunk/WebKitTools

2010-03-31 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

path to committers.py in commit-queue rejection message is wrong
​https://bugs.webkit.org/show_bug.cgi?id=36865

This fix would have only been 3 lines long if we had

  1. Had access to an SCM object or tool to give us the checkout root
  2. Been able to depend on Python 2.6 Instead I've added a bunch of hack code, but at least now we should never have to update this string again as the location of committers.py is fully dynamically discovered. :p
  • Scripts/webkitpy/common/net/bugzilla.py:
  • Scripts/webkitpy/common/net/bugzilla_unittest.py:
3:35 AM Changeset in webkit [56834] by jorlow@chromium.org
  • 6 edits
    1 copy
    1 move in trunk/WebCore

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

Reviewed by Nate Chapin.

IndexedDB: Finish hooking up bindings for IndexedDatabaseRequest
​https://bugs.webkit.org/show_bug.cgi?id=36830

Still not enough hooked up to test. Soon!

  • WebCore.gypi:
  • bindings/v8/custom/V8CustomIDBCallback.h: Removed.
  • bindings/v8/custom/V8CustomIDBCallbacks.h: Added. (WebCore::V8CustomIDBCallbacks::create): (WebCore::V8CustomIDBCallbacks::~V8CustomIDBCallbacks): (WebCore::V8CustomIDBCallbacks::onSuccess): (WebCore::V8CustomIDBCallbacks::onError): (WebCore::V8CustomIDBCallbacks::V8CustomIDBCallbacks):
  • bindings/v8/custom/V8IndexedDatabaseRequestCustom.cpp: (WebCore::V8IndexedDatabaseRequest::openCallback):
  • storage/IDBDatabaseRequest.cpp: Added. (WebCore::IDBDatabaseRequest::IDBDatabaseRequest): (WebCore::IDBDatabaseRequest::~IDBDatabaseRequest):
  • storage/IDBDatabaseRequest.h: (WebCore::IDBDatabaseRequest::create):
  • storage/IndexedDatabaseRequest.cpp: (WebCore::IndexedDatabaseRequest::open):
  • storage/IndexedDatabaseRequest.h:
3:26 AM Changeset in webkit [56833] by eric@webkit.org
  • 5 edits in trunk

2010-03-31 Zhenyao Mo <​zmo@google.com>

Reviewed by Darin Fisher.

Implement memmove behavior for WebGLArray set()
​https://bugs.webkit.org/show_bug.cgi?id=35619

  • fast/canvas/webgl/array-setters-expected.txt: Testing shared overlapping buffer set() behaviors.
  • fast/canvas/webgl/array-setters.html: Ditto.

2010-03-31 Zhenyao Mo <​zmo@google.com>

Reviewed by Darin Fisher.

Implement memmove behavior for WebGLArray set()
​https://bugs.webkit.org/show_bug.cgi?id=35619

New test cases are added to fast/canvas/webgl/array-setters.html.

  • html/canvas/WebGLArray.cpp: Use memmove instead of memcpy; with memmove, copying takes place as if an intermediate buffer was used, allowing the destination and source to overlap. (WebCore::WebGLArray::setImpl):
3:08 AM Changeset in webkit [56832] by eric@webkit.org
  • 2 edits in trunk/LayoutTests

2010-03-31 Sterling Swigart <​sswigart@google.com>

Reviewed by David Levin.

[chromium] test_expectations.txt swept for MAC test cases becoming PASS
​https://bugs.webkit.org/show_bug.cgi?id=36863

  • platform/chromium/test_expectations.txt: Cleaned up test_expectations.txt for MAC which are now PASS.
2:50 AM Changeset in webkit [56831] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-03-31 Benjamin Poulain <​benjamin.poulain@nokia.com>

Reviewed by Darin Adler.

tryLayoutDoingPositionedMovementOnly does not have to be in RenderObject or be virtual
​https://bugs.webkit.org/show_bug.cgi?id=36810

Move tryLayoutDoingPositionedMovementOnly as a regular method of RenderBox

  • rendering/RenderBox.h: (WebCore::RenderBox::tryLayoutDoingPositionedMovementOnly):
  • rendering/RenderObject.h:
2:48 AM QtWebKitJournal edited by fboudra@gmail.com
(diff)
1:56 AM Changeset in webkit [56830] by eric@webkit.org
  • 3 edits
    2 adds in trunk/LayoutTests

2010-03-31 Qi Zhang <​qi.2.zhang@nokia.com>

Reviewed by Darin Adler.

[Qt] Remove LayoutTests/fast/canvas/image-object-in-canvas.html from Skipped
​https://bugs.webkit.org/show_bug.cgi?id=36836

Enable LayoutTests/fast/canvas/image-object-in-canvas.html at LayoutTests

  • platform/qt/Skipped:
  • platform/qt/fast/canvas/image-object-in-canvas-expected.checksum: Added.
  • platform/qt/fast/canvas/image-object-in-canvas-expected.png: Added.
  • platform/qt/fast/canvas/image-object-in-canvas-expected.txt:
1:37 AM Changeset in webkit [56829] by eric@webkit.org
  • 9 edits in trunk

2010-03-31 John Gregg <​johnnyg@google.com>

Reviewed by Darin Fisher.

[chromium] add logging of cross-frame property accesses for site isolation
​https://bugs.webkit.org/show_bug.cgi?id=35773

No new tests as no new functionality.

  • bindings/scripts/CodeGeneratorV8.pm:
  • bindings/v8/V8Utilities.cpp: (WebCore::logPropertyAccess):
  • bindings/v8/V8Utilities.h:
  • loader/FrameLoaderClient.h: (WebCore::FrameLoaderClient::logCrossFramePropertyAccess):

2010-03-31 John Gregg <​johnnyg@google.com>

Reviewed by Darin Fisher.

[chromium] add logging of cross-frame property accesses for site isolation
​https://bugs.webkit.org/show_bug.cgi?id=35773

  • public/WebFrameClient.h: (WebKit::WebFrameClient::logCrossFramePropertyAccess):
  • src/FrameLoaderClientImpl.cpp: (WebKit::FrameLoaderClientImpl::logCrossFramePropertyAccess):
  • src/FrameLoaderClientImpl.h:
12:16 AM April 2010 Meeting edited by kim.gronholm@nomovok.com
(diff)
12:14 AM Changeset in webkit [56828] by hamaji@chromium.org
  • 2 edits in trunk/WebKitTools

2010-03-31 Shinichiro Hamaji <​hamaji@chromium.org>

Unreviewed test fix for r56809.

webkit-patch what-broke throws exception
​https://bugs.webkit.org/show_bug.cgi?id=36852

  • Scripts/webkitpy/common/checkout/api_unittest.py:
12:04 AM Changeset in webkit [56827] by hamaji@chromium.org
  • 2 edits
    7 adds in trunk/LayoutTests

2010-03-30 Shinichiro Hamaji <​hamaji@chromium.org>

Reviewed by Eric Seidel.

[Chromium] fast/repaint/overflow-scroll-body-appear need to be rebaselined
​https://bugs.webkit.org/show_bug.cgi?id=36825

  • platform/chromium-linux/fast/repaint/overflow-scroll-body-appear-expected.checksum: Added.
  • platform/chromium-linux/fast/repaint/overflow-scroll-body-appear-expected.png: Added.
  • platform/chromium-mac/fast/repaint/overflow-scroll-body-appear-expected.checksum: Added.
  • platform/chromium-mac/fast/repaint/overflow-scroll-body-appear-expected.png: Added.
  • platform/chromium-win/fast/repaint/overflow-scroll-body-appear-expected.checksum: Added.
  • platform/chromium-win/fast/repaint/overflow-scroll-body-appear-expected.png: Added.
  • platform/chromium-win/fast/repaint/overflow-scroll-body-appear-expected.txt: Added.
  • platform/chromium/test_expectations.txt:

Mar 30, 2010:

11:09 PM Changeset in webkit [56826] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

Fixing b0rked version of JavaScriptCore.vcproj - added lines were truncated.

Reviewed by NOBODY (windows build fix).

10:44 PM Changeset in webkit [56825] by barraclough@apple.com
  • 259 edits
    2 moves
    3 adds in trunk

Rubber stamped by Sam Weinig.

​https://bugs.webkit.org/show_bug.cgi?id=36866
Move CString to WTF

JavaScriptCore:

(WTF::CStringBuffer::data):
(WTF::CStringBuffer::length):
(WTF::CStringBuffer::create):
(WTF::CStringBuffer::CStringBuffer):
(WTF::CStringBuffer::mutableData):
(WTF::CString::CString):
(WTF::CString::isNull):
(WTF::CString::buffer):
(WTF::operator!=):

WebCore:

  • Android.mk:
  • ForwardingHeaders/wtf/text: Added.
  • ForwardingHeaders/wtf/text/CString.h: Added.
  • GNUmakefile.am:
  • WebCore.base.exp:
  • WebCore.gypi:
  • WebCore.order:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • accessibility/gtk/AccessibilityObjectWrapperAtk.cpp:
  • bindings/js/JSDOMWindowBase.cpp:
  • bindings/js/ScheduledAction.cpp:
  • bindings/js/ScriptController.cpp:
  • bindings/v8/ScriptController.cpp:
  • bindings/v8/V8Binding.cpp:
  • bindings/v8/V8DOMWindowShell.cpp:
  • bridge/jni/JNIBridge.cpp:
  • bridge/jni/v8/JavaStringV8.h:

(JSC::Bindings::JavaStringImpl::init):

  • css/CSSParser.cpp:
  • dom/CharacterData.cpp:
  • dom/Document.cpp:
  • dom/Element.cpp:
  • dom/Node.cpp:
  • dom/Position.cpp:
  • dom/Range.cpp:
  • dom/Text.cpp:
  • dom/XMLTokenizer.cpp:
  • dom/XMLTokenizerLibxml2.cpp:
  • dom/XMLTokenizerQt.cpp:
  • editing/SelectionController.cpp:
  • editing/VisiblePosition.cpp:
  • editing/VisibleSelection.cpp:
  • history/CachedFrame.cpp:
  • history/HistoryItem.cpp:
  • history/qt/HistoryItemQt.cpp:
  • html/FormDataList.h:

(WebCore::FormDataList::appendData):
(WebCore::FormDataList::Item::Item):
(WebCore::FormDataList::Item::data):

  • html/HTMLDocument.cpp:
  • html/PreloadScanner.cpp:
  • inspector/InspectorController.cpp:
  • inspector/InspectorDOMAgent.cpp:
  • loader/DocLoader.cpp:
  • loader/FTPDirectoryDocument.cpp:
  • loader/FrameLoader.cpp:
  • loader/HistoryController.cpp:
  • loader/ProgressTracker.cpp:
  • loader/appcache/ApplicationCacheStorage.cpp:
  • loader/archive/cf/LegacyWebArchive.cpp:
  • loader/icon/wince/IconDatabaseWince.cpp:
  • loader/loader.cpp:
  • page/Console.cpp:
  • page/DOMWindow.cpp:
  • page/SecurityOrigin.cpp:
  • page/XSSAuditor.cpp:
  • page/animation/AnimationBase.cpp:
  • platform/ContextMenu.cpp:
  • platform/FileSystem.h:
  • platform/KURL.cpp:
  • platform/KURLGoogle.cpp:
  • platform/KURLGooglePrivate.h:
  • platform/Pasteboard.h:
  • platform/android/FileSystemAndroid.cpp:
  • platform/android/TemporaryLinkStubs.cpp:
  • platform/brew/KURLBrew.cpp:
  • platform/cf/FileSystemCF.cpp:
  • platform/chromium/MIMETypeRegistryChromium.cpp:
  • platform/efl/FileSystemEfl.cpp:
  • platform/efl/PasteboardEfl.cpp:
  • platform/efl/PlatformKeyboardEventEfl.cpp:
  • platform/efl/PlatformScreenEfl.cpp:
  • platform/efl/RenderThemeEfl.cpp:
  • platform/efl/ScrollbarEfl.cpp:
  • platform/efl/SharedBufferEfl.cpp:
  • platform/efl/WidgetEfl.cpp:
  • platform/graphics/GlyphPageTreeNode.cpp:
  • platform/graphics/cairo/FontPlatformDataCairo.cpp:
  • platform/graphics/cg/ImageBufferCG.cpp:
  • platform/graphics/chromium/FontCacheLinux.cpp:
  • platform/graphics/chromium/FontPlatformDataLinux.h:
  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:
  • platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp:
  • platform/graphics/gtk/FontPlatformDataPango.cpp:
  • platform/graphics/gtk/IconGtk.cpp:
  • platform/graphics/gtk/ImageGtk.cpp:
  • platform/graphics/haiku/GraphicsContextHaiku.cpp:
  • platform/graphics/mac/GraphicsContext3DMac.cpp:
  • platform/graphics/mac/GraphicsLayerCA.mm:
  • platform/graphics/qt/GraphicsContext3DQt.cpp:
  • platform/graphics/qt/ImageBufferQt.cpp:
  • platform/graphics/qt/MediaPlayerPrivatePhonon.cpp:
  • platform/graphics/qt/MediaPlayerPrivateQt.cpp:
  • platform/graphics/skia/GraphicsLayerSkia.cpp:
  • platform/graphics/win/GraphicsLayerCACF.cpp:
  • platform/graphics/win/WKCACFLayer.cpp:
  • platform/graphics/wx/FontPlatformData.h:
  • platform/gtk/ClipboardGtk.cpp:
  • platform/gtk/ContextMenuItemGtk.cpp:
  • platform/gtk/DataObjectGtk.h:
  • platform/gtk/FileChooserGtk.cpp:
  • platform/gtk/FileSystemGtk.cpp:
  • platform/gtk/GeolocationServiceGtk.cpp:
  • platform/gtk/KURLGtk.cpp:
  • platform/gtk/Language.cpp:
  • platform/gtk/LocalizedStringsGtk.cpp:
  • platform/gtk/PasteboardGtk.cpp:
  • platform/gtk/PopupMenuGtk.cpp:
  • platform/gtk/RenderThemeGtk.cpp:
  • platform/gtk/SharedBufferGtk.cpp:
  • platform/haiku/FileSystemHaiku.cpp:
  • platform/haiku/PlatformKeyboardEventHaiku.cpp:
  • platform/network/CredentialStorage.cpp:
  • platform/network/FormData.cpp:
  • platform/network/FormData.h:
  • platform/network/FormDataBuilder.cpp:
  • platform/network/FormDataBuilder.h:
  • platform/network/HTTPParsers.cpp:
  • platform/network/android/ResourceHandleAndroid.cpp:
  • platform/network/cf/FormDataStreamCFNet.cpp:
  • platform/network/cf/ResourceHandleCFNet.cpp:
  • platform/network/chromium/ResourceResponse.h:
  • platform/network/curl/FormDataStreamCurl.cpp:
  • platform/network/curl/ResourceHandleManager.cpp:
  • platform/network/curl/ResourceHandleManager.h:
  • platform/network/mac/FormDataStreamMac.mm:
  • platform/network/mac/ResourceHandleMac.mm:
  • platform/network/qt/QNetworkReplyHandler.cpp:
  • platform/network/soup/CookieJarSoup.cpp:
  • platform/network/soup/ResourceHandleSoup.cpp:
  • platform/network/soup/ResourceRequestSoup.cpp:
  • platform/network/soup/ResourceResponseSoup.cpp:
  • platform/network/win/ResourceHandleWin.cpp:
  • platform/posix/FileSystemPOSIX.cpp:
  • platform/qt/FileSystemQt.cpp:
  • platform/qt/KURLQt.cpp:
  • platform/qt/TemporaryLinkStubs.cpp:
  • platform/sql/chromium/SQLiteFileSystemChromium.cpp:
  • platform/text/CString.cpp: Removed.
  • platform/text/CString.h: Removed.
  • platform/text/PlatformString.h:
  • platform/text/String.cpp:
  • platform/text/StringImpl.cpp:
  • platform/text/TextCodec.h:
  • platform/text/TextCodecICU.cpp:
  • platform/text/TextCodecLatin1.cpp:
  • platform/text/TextCodecUTF16.cpp:
  • platform/text/TextCodecUserDefined.cpp:
  • platform/text/TextEncoding.cpp:
  • platform/text/TextEncoding.h:
  • platform/text/chromium/TextBreakIteratorInternalICUChromium.cpp:
  • platform/text/gtk/TextCodecGtk.cpp:
  • platform/text/haiku/StringHaiku.cpp:
  • platform/text/mac/TextCodecMac.cpp:
  • platform/text/qt/TextCodecQt.cpp:
  • platform/text/wx/StringWx.cpp:
  • platform/win/ClipboardUtilitiesWin.cpp:
  • platform/win/ClipboardWin.cpp:
  • platform/win/ContextMenuItemWin.cpp:
  • platform/win/ContextMenuWin.cpp:
  • platform/win/FileSystemWin.cpp:
  • platform/win/GDIObjectCounter.cpp:
  • platform/win/Language.cpp:
  • platform/win/PasteboardWin.cpp:
  • platform/wince/FileSystemWince.cpp:
  • platform/wince/KeygenWince.cpp:
  • platform/wince/PasteboardWince.cpp:
  • platform/wx/FileSystemWx.cpp:
  • platform/wx/LoggingWx.cpp:
  • plugins/PluginDebug.h:
  • plugins/PluginPackage.cpp:
  • plugins/PluginStream.cpp:
  • plugins/PluginStream.h:
  • plugins/PluginView.h:
  • plugins/gtk/PluginPackageGtk.cpp:
  • plugins/mac/PluginPackageMac.cpp:
  • plugins/qt/PluginPackageQt.cpp:
  • plugins/symbian/PluginPackageSymbian.cpp:
  • plugins/win/PluginPackageWin.cpp:
  • rendering/RenderLayer.cpp:
  • rendering/RenderLayerCompositor.cpp:
  • rendering/RenderTreeAsText.cpp:
  • storage/Database.cpp:
  • storage/SQLTransactionCoordinator.cpp:
  • storage/SQLTransactionCoordinator.h:
  • storage/StorageAreaSync.cpp:
  • storage/StorageSyncManager.cpp:
  • storage/chromium/DatabaseTrackerChromium.cpp:
  • storage/chromium/QuotaTracker.cpp:
  • storage/chromium/QuotaTracker.h:
  • svg/SVGFontFaceElement.cpp:
  • svg/SVGStyledElement.cpp:
  • svg/SVGUseElement.cpp:
  • websockets/WebSocket.cpp:
  • websockets/WebSocketChannel.cpp:
  • websockets/WebSocketHandshake.cpp:
  • wml/WMLErrorHandling.cpp:
  • wml/WMLGoElement.cpp:
  • wml/WMLPageState.cpp:
  • wml/WMLPostfieldElement.cpp:
  • wml/WMLSelectElement.cpp:
  • xml/XMLHttpRequest.cpp:
  • xml/XSLStyleSheetLibxslt.cpp:
  • xml/XSLTProcessorLibxslt.cpp:

WebKit:

  • efl/WebCoreSupport/ChromeClientEfl.cpp:
  • efl/ewk/ewk_frame.cpp:

(ewk_frame_name_get):
(ewk_frame_selection_get):
(ewk_frame_uri_changed):

  • efl/ewk/ewk_history.cpp:
  • efl/ewk/ewk_settings.cpp:

WebKit/chromium:

  • public/WebCString.h:
  • src/FrameLoaderClientImpl.cpp:
  • src/GraphicsContext3D.cpp:
  • src/WebCString.cpp:

(WebKit::WebCString::assign):
(WebKit::WebCString::WebCString):
(WebKit::WebCString::operator=):
(WebKit::WebCString::operator WTF::CString):

  • src/WebMediaPlayerClientImpl.cpp:
  • src/WebString.cpp:
  • src/WebURLError.cpp:

WebKit/gtk:

  • WebCoreSupport/ChromeClientGtk.cpp:
  • WebCoreSupport/ContextMenuClientGtk.cpp:
  • WebCoreSupport/EditorClientGtk.cpp:
  • WebCoreSupport/FrameLoaderClientGtk.cpp:
  • WebCoreSupport/InspectorClientGtk.cpp:
  • gdom/ConvertToGCharPrivate.h:
  • webkit/webkitdownload.cpp:
  • webkit/webkithittestresult.cpp:
  • webkit/webkitnetworkrequest.cpp:
  • webkit/webkitprivate.h:
  • webkit/webkitsecurityorigin.cpp:
  • webkit/webkitwebdatabase.cpp:
  • webkit/webkitwebframe.cpp:
  • webkit/webkitwebhistoryitem.cpp:

(webkit_web_history_item_finalize):
(webkit_web_history_item_get_target):

  • webkit/webkitwebresource.cpp:
  • webkit/webkitwebsettings.cpp:
  • webkit/webkitwebview.cpp:

WebKit/mac:

  • Plugins/Hosted/NetscapePluginInstanceProxy.mm:

(WebKit::NetscapePluginInstanceProxy::getCookies):
(WebKit::NetscapePluginInstanceProxy::getProxy):
(WebKit::NetscapePluginInstanceProxy::getAuthenticationInfo):
(WebKit::NetscapePluginInstanceProxy::resolveURL):

  • Plugins/WebBaseNetscapePluginView.h:
  • Plugins/WebBaseNetscapePluginView.mm:
  • Plugins/WebNetscapePluginView.mm:

(-[WebNetscapePluginView resolveURL:forTarget:]):

WebKit/qt:

  • Api/qwebelement.cpp:

WebKit/win:

  • WebDownload.cpp:
  • WebDownloadCFNet.cpp:
  • WebDownloadCurl.cpp:
  • WebHistoryItem.cpp:
  • WebLocalizableStrings.cpp:
  • WebMutableURLRequest.cpp:
  • WebPreferences.cpp:

(WebPreferences::migrateWebKitPreferencesToCFPreferences):

  • WebView.cpp:

WebKit/wx:

  • WebFrame.cpp:
  • WebView.cpp:
9:35 PM Changeset in webkit [56824] by tkent@chromium.org
  • 7 edits in trunk

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

Reviewed by Brady Eidson.

REGRESSION (r56439) - Crash when a renderer for a file upload control
with a selected file is recreated
​https://bugs.webkit.org/show_bug.cgi?id=36723

  • fast/forms/input-file-re-render.html: Force to do re-layout to check the regression.

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

Reviewed by Brady Eidson.

REGRESSION (r56439) - Crash when a renderer for a file upload control
with a selected file is recreated
​https://bugs.webkit.org/show_bug.cgi?id=36723

RenderFileUploadControl::chooseIconForFiles was called before
m_fileChooser was initialized.

  • platform/FileChooser.cpp: (WebCore::FileChooser::FileChooser): Introduce m_isInitializing flag to avoid FileChooserClient::repaint() call. (WebCore::FileChooser::loadIcon): (WebCore::FileChooser::iconLoaded):
  • platform/FileChooser.h: Add a FielChooser parameter to FileChooserClient::chooseIconForFiles().
  • rendering/RenderFileUploadControl.cpp: (WebCore::RenderFileUploadControl::chooseIconForFiles): (WebCore::RenderFileUploadControl::paintObject): Add an assertion.
  • rendering/RenderFileUploadControl.h:
9:32 PM Changeset in webkit [56823] by Chris Jerdonek
  • 2 edits in trunk/WebKitTools

Rubber stamped by Dave Levin.

Made check-webkit-style less chatty.

Examples include:

  • Scripts/webkitpy/style/checker.py:
    • Changed unrecognized file type log message from info to debug.
8:36 PM Changeset in webkit [56822] by Laszlo Gombos
  • 2 edits in trunk/WebCore

2010-03-30 Stanislav Paltis <​Stanislav.Paltis@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] qmake/symbian build failure
​https://bugs.webkit.org/show_bug.cgi?id=36745

Fix build break in Symbian build introduced in r56714,
by modifying comment escape character from ";" to "#".

  • WebCore.pro:
8:09 PM April 2010 Meeting edited by staikos@kde.org
(diff)
7:00 PM Changeset in webkit [56821] by yuzo@google.com
  • 3 edits
    2 deletes in trunk/LayoutTests

2010-03-30 Yuzo Fujishima <​yuzo@google.com>

Reviewed by Eric Seidel.

Fix LayoutTests/fast/dom/Window/window-resize.html
The test failed on Mac if Dock is placed on the left or the right because the window's width
and horizontal position can be adjusted by the OS.
This patch fixes it by using window.availWidth instead of window.width as the expected
values and by avoid using constant expected values.
As a side effect, the expected values become platform independent.

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

  • fast/dom/Window/window-resize-expected.txt:
  • fast/dom/Window/window-resize.html:
  • platform/mac/fast/dom/Window/window-resize-expected.txt: Removed.
  • platform/win/fast/dom/Window/window-resize-expected.txt: Removed.
6:25 PM Changeset in webkit [56820] by hclam@chromium.org
  • 2 edits in trunk/LayoutTests

2010-03-30 Alpha Lam <​hclam@chromium.org>

Reviewed by Dmitry Titov.

[chromium] Fix test expectations for a new test added in r56779
​https://bugs.webkit.org/show_bug.cgi?id=36868

inspector/audits-panel-functional.html is crashing on Windows. Mark it so.

  • platform/chromium/test_expectations.txt: Add expectation for inspector/audits-panel-functional.html to mark it as crashing.
6:19 PM Changeset in webkit [56819] by kevino@webkit.org
  • 6 edits in trunk

wx build fixes after new method additions and a method rename.

5:35 PM Changeset in webkit [56818] by hclam@chromium.org
  • 7 edits
    6 adds in trunk/LayoutTests

2010-03-30 Alpha Lam <​hclam@chromium.org>

Reviewed by Dmitry Titov.

[chromium] New test results due to 56776
​https://bugs.webkit.org/show_bug.cgi?id=36857

Adding new layout test results for chromium for tests added in r56776.

  • platform/chromium-linux/svg/custom/glyph-transformation-with-hkern-expected.checksum: Added.
  • platform/chromium-linux/svg/custom/glyph-transformation-with-hkern-expected.png: Added.
  • platform/chromium-mac/svg/custom/scrolling-embedded-svg-file-image-repaint-problem-expected.checksum: Added.
  • platform/chromium-mac/svg/custom/scrolling-embedded-svg-file-image-repaint-problem-expected.png: Added.
  • platform/chromium-win/svg/W3C-SVG-1.1/fonts-kern-01-t-expected.txt:
  • platform/chromium-win/svg/W3C-SVG-1.1/masking-mask-01-b-expected.txt:
  • platform/chromium-win/svg/W3C-SVG-1.1/render-groups-01-b-expected.txt:
  • platform/chromium-win/svg/W3C-SVG-1.1/render-groups-03-t-expected.txt:
  • platform/chromium-win/svg/custom/glyph-transformation-with-hkern-expected.checksum: Added.
  • platform/chromium-win/svg/custom/glyph-transformation-with-hkern-expected.png: Added.
  • platform/chromium-win/svg/custom/scrolling-embedded-svg-file-image-repaint-problem-expected.txt:
  • platform/chromium/test_expectations.txt: Removed the failing expectation for platform/chromium-win/svg/custom/scrolling-embedded-svg-file-image-repaint-problem.html because the difference in only in the scroll bar, there is not difference in the image content. Also fixed a grammar problem in last commit.
5:34 PM Changeset in webkit [56817] by hclam@chromium.org
  • 1 edit in trunk/WebKitTools/Scripts/webkitpy/layout_tests/rebaseline_chromium_webkit_tests.py

fixed rebaseline

5:27 PM Changeset in webkit [56816] by scherkus@chromium.org
  • 2 edits in trunk/LayoutTests

2010-03-30 Andrew Scherkus <​scherkus@chromium.org>

Unreviewed, expectations tweak.

Updating expectations to stop skipping tests that were fixed in r56481.

  • platform/chromium/test_expectations.txt: Don't skip fixed tests.
4:58 PM Changeset in webkit [56815] by weinig@apple.com
  • 2 edits in trunk/WebCore

Add missing export.

Reviewed by Anders Carlsson.

  • WebCore.base.exp:
4:44 PM Changeset in webkit [56814] by hclam@chromium.org
  • 2 edits in trunk/LayoutTests

2010-03-30 Alpha Lam <​hclam@chromium.org>

Reviewed by David Levin.

[chromium] Fix test_expectations for new test added in r56795
​https://bugs.webkit.org/show_bug.cgi?id=36861

  • platform/chromium/test_expectations.txt: Expects svg/animations/animate-points.html to fail because a API is missing in Chromium. See ​http://crbug.com/39904.
4:12 PM Changeset in webkit [56813] by enrica@apple.com
  • 3 edits
    2 adds in trunk

Incorrect insertion position after typing THAI CHARACTER SARA AM = 0E33.
<rdar://problem/7810749>
​https://bugs.webkit.org/show_bug.cgi?id=36858

Reviewed by Darin Adler.

WebCore:

Test: fast/text/multiglyph-characters.html

Characters with multiple glyph were not supported correctly.
Computing the advance width was producing a division by zero.

  • platform/graphics/mac/ComplexTextController.cpp:

(WebCore::ComplexTextController::advance):

LayoutTests:

  • fast/text/multiglyph-characters-expected.txt: Added.
  • fast/text/multiglyph-characters.html: Added.
3:35 PM Changeset in webkit [56812] by mrowe@apple.com
  • 4 edits in trunk

Versioning.

3:33 PM Changeset in webkit [56811] by mrowe@apple.com
  • 1 copy in tags/Safari-533.3

New tag.

2:51 PM Changeset in webkit [56810] by abarth@webkit.org
  • 3 edits
    3 adds in trunk

2010-03-30 Chris Evans <​cevans@chromium.org>

Reviewed by Adam Barth.

Add test for SVG pattern canvas tainting.

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

  • fast/canvas/svg-taint.html: Added
  • fast/canvas/svg-taint-expected.txt: Added
  • fast/canvas/resources/empty.svg: Added

2010-03-30 Chris Evans <​cevans@chromium.org>

Reviewed by Adam Barth.

Taint the canvas if an SVG-derived pattern is rendered into it.

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

Test: fast/canvas/svg-taint.html

  • html/canvas/CanvasRenderingContext2D.cpp: (WebCore::CanvasRenderingContext2D::createPattern): Take into account the image's hasSingleSecurityOrigin() property.
2:44 PM Changeset in webkit [56809] by eric@webkit.org
  • 4 edits in trunk/WebKitTools

2010-03-30 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

webkit-patch what-broke throws exception
​https://bugs.webkit.org/show_bug.cgi?id=36852

  • Scripts/webkitpy/common/checkout/api.py: Add missing import StringIO.
  • Scripts/webkitpy/common/checkout/api_unittest.py: Test the function which previously threw and exception.
2:27 PM Changeset in webkit [56808] by ap@apple.com
  • 2 edits in trunk/WebKitTools

Reviewed by Darin Adler.

​https://bugs.webkit.org/show_bug.cgi?id=36616
Dvorak-Qwerty keyboard layout gives unexpected results in javascript keydown

​https://bugs.webkit.org/show_bug.cgi?id=36797
For non-Roman layouts, keydown Event.keyCode is always 0

  • DumpRenderTree/mac/EventSendingController.mm: (-[EventSendingController keyDown:withModifiers:withLocation:]): Generate a correct keyCode for keys used in tests (we used to always pass 0 for 'A').
1:58 PM April 2010 Meeting edited by jasonr@bsquare.com
(diff)
1:48 PM Changeset in webkit [56807] by scherkus@chromium.org
  • 3 edits in trunk/LayoutTests

2010-03-30 Andrew Scherkus <​scherkus@chromium.org>

Reviewed by Darin Adler.

Updating expectations for video-no-audio.html to reflect changes made in r56481.

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

  • platform/mac-tiger/media/video-no-audio-expected.txt: Updating due to different source video dimensions.
  • platform/win/media/video-no-audio-expected.txt: Ditto.
1:37 PM Changeset in webkit [56806] by sullivan@apple.com
  • 2 edits in trunk/WebKit/mac

​https://bugs.webkit.org/show_bug.cgi?id=36848
<rdar://problem/7362913>
Menu items appropriate only for rich-content editing can appear in plain-text contexts

Reviewed by Dan Bernstein.

  • WebView/WebHTMLView.mm:

(-[WebHTMLView validRequestorForSendType:returnType:]):
Don't return self for non-string content if _canEditRichly is false.

1:10 PM Changeset in webkit [56805] by eric@webkit.org
  • 2 edits in trunk/WebKit/qt

2010-03-30 Joe Ligman <​joseph.ligman@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] QGraphicsScene mousePressEvent does not set the clickCausedFocus flag
​https://bugs.webkit.org/show_bug.cgi?id=35259

The clickCausedFocus flag is not being set in method
mousePressEvent(QGraphicsSceneMouseEvent* ev). This flag is used
in conjunction with QStyle::RSIP_OnMouseClickAndAlreadyFocused when
deciding to launch the software input panel.

  • Api/qwebpage.cpp: (QWebPagePrivate::mousePressEvent):
1:00 PM Changeset in webkit [56804] by ap@apple.com
  • 3 edits in trunk/WebCore

Reviewed by Darin Adler.

​https://bugs.webkit.org/show_bug.cgi?id=36616
Dvorak-Qwerty keyboard layout gives unexpected results in javascript keydown

​https://bugs.webkit.org/show_bug.cgi?id=36797
For non-Roman layouts, keydown Event.keyCode is always 0

  • platform/cocoa/KeyEventCocoa.mm: (WebCore::windowsKeyCodeForKeyCode): This is now the primary function for determining Windows key code - there is no reason to round trip a virtual key code via character code to get Windows virtual key code, and this can't work for non-Roman alphabets. (WebCore::windowsKeyCodeForCharCode): Removed keys that are now handled in windowsKeyCodeForKeyCode(), only keeping those for which we don't have known key codes (yet?).
  • platform/mac/KeyEventMac.mm: (WebCore::windowsKeyCodeForKeyEvent): Added comments.
12:51 PM Changeset in webkit [56803] by eric@webkit.org
  • 9 edits
    2 copies
    1 move
    1 add in trunk

2010-03-30 Marcus Bulach <​bulach@chromium.org>

Reviewed by Darin Fisher.

Rename / tidy up Geolocation bridge
Uses Geoposition instead of individual params for setLastPosition.
​https://bugs.webkit.org/show_bug.cgi?id=36535

  • platform/chromium/GeolocationServiceChromium.cpp: (WebCore::GeolocationServiceChromium::GeolocationServiceChromium): (WebCore::GeolocationServiceChromium::setIsAllowed): (WebCore::GeolocationServiceChromium::setLastPosition):
  • platform/chromium/GeolocationServiceChromium.h:

2010-03-30 Marcus Bulach <​bulach@chromium.org>

Reviewed by Darin Fisher.

Rename / tidy up Geolocation bridge:
Rename GeolocationServiceBridgeChromium.cpp to WebGeolocationServiceBridgeImpl.cpp
Uses a temporary compatibility layer in GeolocationServiceBridgeChromium.h.

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

  • WebKit.gyp:
  • public/GeolocationServiceBridgeChromium.h:
  • public/WebGeolocationService.h: Copied from WebKit/chromium/public/GeolocationServiceBridgeChromium.h. (WebKit::WebGeolocationService::detachBridge):
  • public/WebGeolocationServiceBridge.h: Copied from WebKit/chromium/public/GeolocationServiceBridgeChromium.h.
  • public/WebViewClient.h: (WebKit::WebViewClient::geolocationService):
  • src/ChromeClientImpl.cpp: (WebKit::ChromeClientImpl::requestGeolocationPermissionForFrame):
  • src/ChromiumBridge.cpp:
  • src/GeolocationServiceBridgeChromium.cpp: Removed.
  • src/WebGeolocationServiceBridgeImpl.cpp: Copied from WebKit/chromium/src/GeolocationServiceBridgeChromium.cpp. (WebKit::createGeolocationServiceBridgeImpl): (WebKit::WebGeolocationServiceBridgeImpl::WebGeolocationServiceBridgeImpl): (WebKit::WebGeolocationServiceBridgeImpl::~WebGeolocationServiceBridgeImpl): (WebKit::WebGeolocationServiceBridgeImpl::startUpdating): (WebKit::WebGeolocationServiceBridgeImpl::stopUpdating): (WebKit::WebGeolocationServiceBridgeImpl::suspend): (WebKit::WebGeolocationServiceBridgeImpl::resume): (WebKit::WebGeolocationServiceBridgeImpl::getBridgeId): (WebKit::WebGeolocationServiceBridgeImpl::setIsAllowed): (WebKit::WebGeolocationServiceBridgeImpl::setLastPosition): (WebKit::WebGeolocationServiceBridgeImpl::setLastError): (WebKit::WebGeolocationServiceBridgeImpl::getWebViewClient):
  • src/WebGeolocationServiceBridgeImpl.h: Copied from WebKit/chromium/public/GeolocationServiceBridgeChromium.h.
12:31 PM Changeset in webkit [56802] by eric@webkit.org
  • 3 edits in trunk/WebKitTools

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

Reviewed by Eric Seidel.

Implement win-ews
​https://bugs.webkit.org/show_bug.cgi?id=36809

  • Scripts/webkitpy/common/config/ports.py:
  • Scripts/webkitpy/tool/commands/earlywarningsystem.py:
12:21 PM Changeset in webkit [56801] by cmarrin@apple.com
  • 1 edit in trunk/WebKitSite/blog-files/webgl/WebGL+CSS.html

Fixed some size issues caused by how we now compute the WebGL canvas size

11:16 AM Changeset in webkit [56800] by agl@chromium.org
  • 7 edits in trunk

2010-03-30 Adam Langley <​agl@chromium.org>

Reviewed by David Levin.

[chromium] linux: prefer the "hinting" fontconfig preference.

fontconfig can give contradictory hinting information, setting
"hinting" to zero and "hintstyle" to non-zero. In this case we
should take the "hinting" preference.

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

  • platform/graphics/chromium/FontPlatformDataLinux.cpp: (WebCore::FontPlatformData::setupPaint):
10:42 AM Changeset in webkit [56799] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

2010-03-30 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Copying from the Content tab in Resources includes line numbers.

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

10:32 AM Changeset in webkit [56798] by pfeldman@chromium.org
  • 5 edits in trunk

2010-03-30 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Do not send empty matching rules for pseudo elements to frontend.

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

  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::buildArrayForPseudoElements):
  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylesSidebarPane.prototype._update):
10:09 AM Changeset in webkit [56797] by ap@apple.com
  • 2 edits in trunk/WebKit/mac

Reviewed by Darin Adler.

​https://bugs.webkit.org/show_bug.cgi?id=36791
Add assertions for instance proxy validity

Add some assertions that the instance proxy hasn't been deleted. We sometimes keep a raw
pointer to one across complicated function calls, relying on the caller to protect the
reference.

  • Plugins/Hosted/NetscapePluginInstanceProxy.h: (WebKit::NetscapePluginInstanceProxy::renderContextID): (WebKit::NetscapePluginInstanceProxy::pluginView): (WebKit::NetscapePluginInstanceProxy::hostProxy):
9:57 AM Changeset in webkit [56796] by jianli@chromium.org
  • 2 edits in trunk/LayoutTests

[chromium] Update test_expectations.txt to enable send-dragged-sliced-file.html
​https://bugs.webkit.org/show_bug.cgi?id=36764

Reviewed by Dmitry Titov.

  • platform/chromium/test_expectations.txt:
9:40 AM Changeset in webkit [56795] by krit@webkit.org
  • 8 edits
    3 adds in trunk

2010-03-30 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

Cannot animate "points" attribute for <svg:polygon>
​https://bugs.webkit.org/show_bug.cgi?id=21371

Add animation support for 'points' on polygons in SVG.

Test: svg/animations/animate-points.html

  • svg/SVGAnimateElement.cpp: (WebCore::SVGAnimateElement::determinePropertyType): (WebCore::SVGAnimateElement::calculateAnimatedValue): (WebCore::SVGAnimateElement::calculateFromAndToValues): (WebCore::SVGAnimateElement::resetToBaseValue): (WebCore::SVGAnimateElement::applyResultsToTarget):
  • svg/SVGAnimateElement.h: (WebCore::SVGAnimateElement::): added PropertyType PointsProperty
  • svg/SVGPointList.cpp: (WebCore::blendFunc): (WebCore::SVGPointList::createAnimated): calculates animated PointList
  • svg/SVGPointList.h:

2010-03-30 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

Cannot animate "points" attribute for <svg:polygon>
​https://bugs.webkit.org/show_bug.cgi?id=21371

Tests if points of a polygon are animateable.

  • svg/animations/animate-points-expected.txt: Added.
  • svg/animations/animate-points.html: Added.
  • svg/animations/script-tests/animate-points.js: Added. (sample1): (sample2): (sample3): (executeTest):
9:18 AM Changeset in webkit [56794] by Chris Jerdonek
  • 2 edits in trunk/WebKitTools

Adjusted test-webkitpy to delete all orphaned *.pyc files
from webkitpy/ prior to importing any modules from webkitpy.
This ensures that no import statements in webkitpy falsely
succeed because of leftover *.pyc files.

Reviewed by Eric Seidel.

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

  • Scripts/test-webkitpy:
    • Added _clean_pyc_files() to delete orphaned *.pyc files from a directory.
    • Added _clean_webkitpy_with_test() to call and test _clean_pyc_files().
    • Moved the "import webkitpy.python24.versioning" statement from the top of the module to the init() method -- immediately after the call to _clean_webkitpy_with_test().
8:43 AM Changeset in webkit [56793] by pfeldman@chromium.org
  • 2 edits in trunk/LayoutTests

2010-03-30 Pavel Feldman <​pfeldman@chromium.org>

Not reviewed: bring test expectation back - it still fails for some
reason.

  • platform/chromium/test_expectations.txt:
8:40 AM Changeset in webkit [56792] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-03-30 Alexander Pavlov <​apavlov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: Audits: bad CSS selector results in exception inside evaluated code
​https://bugs.webkit.org/show_bug.cgi?id=36821

  • inspector/front-end/AuditRules.js: (WebInspector.AuditRules.UnusedCssRule.prototype.doRun.evalCallback.routine): (WebInspector.AuditRules.UnusedCssRule.prototype.doRun.evalCallback):
8:23 AM Changeset in webkit [56791] by jocelyn.turcotte@nokia.com
  • 2 edits in trunk/JavaScriptCore

[Qt] Fix build break on Qt Mac.

Reviewed by nobody, build break.

DESTDIR path on Mac do not include the configuration path by default
like on Windows. Have to force it.

7:51 AM Changeset in webkit [56790] by Simon Hausmann
  • 2 edits in trunk/WebCore

[Qt] Package build fixes for Symbian

Patch by Janne Koskinen <​janne.p.koskinen@digia.com> on 2010-03-29
Reviewed by Simon Hausmann.

  • enable building standalone QtWebkit for Symbian using qtlibinfix configuration option

qtlibinfix adds possibility having multiple versions of Qt in Symbian devices.
Fix also allows unfrozen DLL exports during development.

  • Removed heap size configuration. DLLs don't have a heap.
  • Removed enforcement to def files for package builds. Instead inherit the setting from

Qt's qconfig.pri whether to use def files or not. It's still possible to override this
also at qmake time by passing CONFIG+=def_files.

  • WebCore.pro:
7:50 AM Changeset in webkit [56789] by pfeldman@chromium.org
  • 3 edits in trunk/WebCore

2010-03-30 Ilya Tikhonovsky <​loislo@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: Title for show/hide short records button was adjusted.
Some unused code was removed.
​https://bugs.webkit.org/show_bug.cgi?id=36823

  • inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel.prototype._createStatusbarButtons): (WebInspector.TimelinePanel.prototype._toggleFilterButtonClicked): (WebInspector.TimelinePanel.prototype._innerAddRecordToTimeline):
7:49 AM Changeset in webkit [56788] by yurys@chromium.org
  • 2 edits in trunk/LayoutTests

2010-03-30 Yury Semikhatsky <​yurys@chromium.org>

Unreviewed. Update Chromium test expectations.

  • platform/chromium/test_expectations.txt:
7:45 AM Changeset in webkit [56787] by pfeldman@chromium.org
  • 2 edits in trunk/LayoutTests

2010-03-30 Pavel Feldman <​pfeldman@chromium.org>

Not reviewed: removed failure expectation from Chromium.

  • platform/chromium/test_expectations.txt:
7:16 AM QtWebKitBackportingFixes edited by Simon Hausmann
(diff)
7:13 AM Changeset in webkit [56786] by Adam Roben
  • 2 edits in trunk/WebKit/win

Windows build fix

  • Interfaces/WebKit.idl: Touched this to force a build.
7:13 AM Changeset in webkit [56785] by Adam Roben
  • 2 edits in trunk/WebCore

Enable extra Direct3D debugging information in Debug builds

This makes it easier to track down memory leaks and misuses of the D3D
APIs in Debug builds.

Fixes <​http://webkit.org/b/36820>

Reviewed by John Sullivan.

  • platform/graphics/win/WKCACFLayerRenderer.cpp: Define D3D_DEBUG_INFO

in Debug builds before #including d3d9.h so that extra D3D debugging
information will be enabled.

7:10 AM April 2010 Meeting edited by ukai@chromium.org
Add Fumitoshi Ukai (diff)
7:10 AM Changeset in webkit [56784] by eric@webkit.org
  • 4 edits
    2 adds in trunk

2010-03-30 Luiz Agostini <​luiz.agostini@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Maemo5 theme - QtMaemoWebPopup class
​https://bugs.webkit.org/show_bug.cgi?id=36790

Moving maemo5 theme related code from WebCore.pri to WebCore.pro and adding
new maemo menu lists popup class source and header to the project.

The code was in .pri file because of a theme css that was conditionally included.
As the css file is been included unconditionally there is no need to this
code to be kept in webcore.pri.

  • WebCore.pri:
  • WebCore.pro:

2010-03-30 Luiz Agostini <​luiz.agostini@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Maemo5 theme - QtMaemoWebPopup class
​https://bugs.webkit.org/show_bug.cgi?id=36790

A new QtAbstractWebPopup descendant class to be used for maemo menu lists.
This is the first step. The next step will be the dialogs implementation that
will come in bug 36789 and finally to use QtMaemoWebPopup instead of QtFallbackWebPopup
that will come in bug 36368.

  • WebCoreSupport/QtMaemoWebPopup.cpp: Added. (WebCore::QtMaemoWebPopup::QtMaemoWebPopup): (WebCore::QtMaemoWebPopup::~QtMaemoWebPopup): (WebCore::QtMaemoWebPopup::createSingleSelectionPopup): (WebCore::QtMaemoWebPopup::createMultipleSelectionPopup): (WebCore::QtMaemoWebPopup::createPopup): (WebCore::QtMaemoWebPopup::show): (WebCore::QtMaemoWebPopup::hide): (WebCore::QtMaemoWebPopup::popupClosed): (WebCore::QtMaemoWebPopup::itemClicked):
  • WebCoreSupport/QtMaemoWebPopup.h: Added. (WebCore::Maemo5Popup::Maemo5Popup):
7:01 AM FAQ edited by tony@gonk.net
Fixed incorrect use of “it’s”. Changed hyphen to m-dash. (diff)
7:01 AM Changeset in webkit [56783] by jchaffraix@webkit.org
  • 2 edits
    1 add
    9 deletes in trunk/LayoutTests

fast/parser/comments.html needs some clean-up
​https://bugs.webkit.org/show_bug.cgi?id=36537

Reviewed by Darin Adler.

  • The test is converted to dumpAsText now.
  • The output is also made more explicit as to what is expected (adding some PASSED and FAILED).
  • Finally some part of the test were actually commented so added some comment endings to make sure we run all the part of the test.
  • fast/parser/comments-expected.txt: Added.
  • fast/parser/comments.html:
  • platform/chromium-linux/fast/parser/comments-expected.checksum: Removed.
  • platform/chromium-linux/fast/parser/comments-expected.png: Removed.
  • platform/chromium-win/fast/parser/comments-expected.checksum: Removed.
  • platform/chromium-win/fast/parser/comments-expected.png: Removed.
  • platform/chromium-win/fast/parser/comments-expected.txt: Removed.
  • platform/mac/fast/parser/comments-expected.checksum: Removed.
  • platform/mac/fast/parser/comments-expected.png: Removed.
  • platform/mac/fast/parser/comments-expected.txt: Removed.
  • platform/qt/fast/parser/comments-expected.txt: Removed.
6:45 AM Changeset in webkit [56782] by eric@webkit.org
  • 3 edits in trunk/WebKit/chromium

2010-03-30 Jochen Eisinger <​jochen@chromium.org>

Reviewed by Jeremy Orlow.

Remove dysfunctional implementation of canEstablishDatabase for
Workers. I postpone this implementation until Workers can actually
access Web Databases.

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

  • src/DatabaseObserver.cpp: (WebCore::DatabaseObserver::canEstablishDatabase):
  • src/WebWorkerBase.h:
6:27 AM Changeset in webkit [56781] by eric@webkit.org
  • 16 edits
    1 add in trunk

2010-03-30 Laszlo Gombos <Laszlo Gombos>

Reviewed by David Kilzer.

Guard Geolocation files with ENABLE_GEOLOCATION
​https://bugs.webkit.org/show_bug.cgi?id=25756

The intent is to guard the Geolocation implementation files
and minimize the impact on on clients/call sites.

  • DerivedSources.make:
  • WebCore.Geolocation.exp: Added. Exported interfaces if GEOLOCATION is enabled.
  • WebCore.base.exp:
  • bindings/js/JSCustomPositionCallback.cpp:
  • bindings/js/JSCustomPositionErrorCallback.cpp:
  • bindings/js/JSGeolocationCustom.cpp:
  • page/Geolocation.cpp: (WebCore::Geolocation::disconnectFrame): Stub implementation if GEOLOCATION is turned off. (WebCore::Geolocation::Geolocation): Ditto. (WebCore::Geolocation::~Geolocation): Ditto. (WebCore::Geolocation::setIsAllowed): Ditto.
  • page/Geolocation.h:
  • page/Geolocation.idl:
  • page/GeolocationPositionCache.cpp:
  • page/Geoposition.idl:
  • page/PositionError.idl:
  • platform/GeolocationService.cpp:
  • platform/mock/GeolocationServiceMock.cpp:

2010-03-30 Laszlo Gombos <Laszlo Gombos>

Reviewed by David Kilzer.

Explicit guards for ENABLE_GEOLOCATION
​https://bugs.webkit.org/show_bug.cgi?id=25756

  • WebCoreSupport/WebGeolocationMock.mm: (-[WebGeolocationMock setError:code:]): Make the body conditional on ENABLE(GEOLOCATION) (-[WebGeolocationMock setPosition:]): Ditto.
6:08 AM Changeset in webkit [56780] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-03-30 Andrey Kosyakov <​caseq@chromium.org>

Reviewed by Pavel Feldman.

Support EventTarget interface in fake workers (both for Worker object
and WorkerContext). Use MessagePort to implement message passing to
support passing ports in PostMessage.
​https://bugs.webkit.org/show_bug.cgi?id=36763

  • inspector/front-end/InjectedFakeWorker.js: (InjectedFakeWorker.Worker.onmessageGetter): (InjectedFakeWorker.Worker.onmessageSetter): (InjectedFakeWorker.Worker): (InjectedFakeWorker.FakeWorker): (InjectedFakeWorker.FakeWorker.prototype.postMessage): (InjectedFakeWorker.FakeWorker.prototype.terminate): (InjectedFakeWorker.FakeWorker.prototype._onWorkerFrameLoaded): (InjectedFakeWorker.FakeWorker.prototype._setupWorkerContext.onmessageGetter): (InjectedFakeWorker.FakeWorker.prototype._setupWorkerContext.onmessageSetter): (InjectedFakeWorker.FakeWorker.prototype._setupWorkerContext): (InjectedFakeWorker.FakeWorker.prototype._addEventListener): (InjectedFakeWorker.FakeWorker.prototype._removeEventListener): (InjectedFakeWorker.FakeWorker.prototype._callbackWrapper): (InjectedFakeWorker.FakeWorker.prototype._handleException):
5:44 AM Changeset in webkit [56779] by apavlov@chromium.org
  • 1 edit
    6 adds in trunk/LayoutTests

2010-03-30 Alexander Pavlov <​apavlov@chromium.org>

Unreviewed, test missing in r56732.

  • inspector/audits-panel-functional-expected.txt: Added.
  • inspector/audits-panel-functional.html: Added.
  • inspector/resources/audits-script1.js: Added. (foo1):
  • inspector/resources/audits-script2.js: Added. (foo2):
  • inspector/resources/audits-script3.js: Added. (foo3):
  • inspector/resources/audits-style1.css: Added.
5:44 AM Changeset in webkit [56778] by benm@google.com
  • 2 edits in trunk/WebCore

[v8] Add if ENABLE guard for Worker specific code.
​https://bugs.webkit.org/show_bug.cgi?id=36597

Patch by Kristian Monsen <​kristianm@google.com> on 2010-03-30
Reviewed by David Levin.

Build fix only, no new tests.

  • bindings/v8/V8DOMWrapper.h:
4:58 AM Changeset in webkit [56777] by jorlow@chromium.org
  • 20 edits
    1 copy
    1 move
    3 deletes in trunk

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

Reviewed by Darin Fisher.

More IndexedDB work
​https://bugs.webkit.org/show_bug.cgi?id=36770

Add the "singleton" IndexedDatabase object to PageGroup
IndexedDatabase now has a create() method since PageGroup can own the instance (at least for now)
Rip out the IDBRequest stuff (now obsolete).
DOMWindow now can instantiate indexedDatabase objects on demand.

New layout tests starting with the next patch (which should make indexedDB.open() "work").

  • WebCore.gypi:
  • bindings/v8/custom/V8IDBRequestCustom.cpp: Removed.
  • page/DOMWindow.cpp: (WebCore::DOMWindow::indexedDB):
  • page/DOMWindow.h:
  • page/PageGroup.cpp: (WebCore::PageGroup::indexedDatabase):
  • page/PageGroup.h:
  • storage/IDBDatabase.cpp:
  • storage/IDBDatabaseRequest.h: (WebCore::IDBDatabaseRequest::createObjectStore):
  • storage/IDBDatabaseRequest.idl:
  • storage/IDBRequest.cpp: Removed.
  • storage/IDBRequest.h: Removed.
  • storage/IDBRequest.idl: Removed.
  • storage/IndexedDatabase.cpp: (WebCore::IndexedDatabase::create):
  • storage/IndexedDatabase.h:
  • storage/IndexedDatabaseImpl.cpp: (WebCore::IndexedDatabaseImpl::create): (WebCore::IndexedDatabaseImpl::IndexedDatabaseImpl): (WebCore::IndexedDatabaseImpl::~IndexedDatabaseImpl):
  • storage/IndexedDatabaseImpl.h:
  • storage/IndexedDatabaseRequest.cpp: (WebCore::IndexedDatabaseRequest::IndexedDatabaseRequest):
  • storage/IndexedDatabaseRequest.h: (WebCore::IndexedDatabaseRequest::create):
  • storage/IndexedDatabaseRequest.idl:
  • storage/chromium/IndexedDatabase.cpp: (WebCore::IndexedDatabase::create):

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

Reviewed by Darin Fisher.

More IndexedDB work
​https://bugs.webkit.org/show_bug.cgi?id=36770

Start the implementation of WebIndexedDatabase (for entrance back into WebKit).

  • WebKit.gyp:
  • public/WebIndexedDatabase.h:
  • src/WebIndexedDatabaseImpl.cpp: Added. (WebKit::WebIndexedDatabase::create): (WebKit::WebIndexedDatabaseImpl::~WebIndexedDatabaseImpl): (WebKit::WebIndexedDatabaseImpl::open):
  • src/WebIndexedDatabaseImpl.h: Added.
4:30 AM Changeset in webkit [56776] by krit@webkit.org
  • 8 edits
    5 adds in trunk

2010-03-30 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVG Glyph transformations wrong on use of SVGFont
​https://bugs.webkit.org/show_bug.cgi?id=36731

Kerning on SVGFonts should be applied to the userSpace of the current Char
and not to the userSpace of the viewport. This means scaling the kerning
to match the current userSpace.

Test: svg/custom/glyph-transformation-with-hkern.svg

  • rendering/SVGRootInlineBox.cpp: (WebCore::SVGRootInlineBox::buildLayoutInformationForTextBox):

2010-03-30 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVG Glyph transformations wrong on use of SVGFont
​https://bugs.webkit.org/show_bug.cgi?id=36731

Test of correct scaled kerning, if userSpace on SVGFont doesn't
match the scaling of the viewport userSpace.

  • platform/mac/svg/W3C-SVG-1.1/fonts-kern-01-t-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/masking-mask-01-b-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/render-groups-01-b-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/render-groups-03-t-expected.txt:
  • platform/mac/svg/custom/scrolling-embedded-svg-file-image-repaint-problem-expected.txt:
  • svg/custom/glyph-transformation-with-hkern.svg: Added.
  • svg/custom/resources/SVGFreeSans.svg: Added.
4:28 AM Changeset in webkit [56775] by krit@webkit.org
  • 3 edits
    4 adds in trunk

2010-03-30 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVG Animation doesn't respect 'currentColor'
​https://bugs.webkit.org/show_bug.cgi?id=36695

Test: svg/custom/animation-currentColor.svg

SVG Animation can't handle currentColor at the moment. This patch catches the
color value of the target element and replaces 'currentColor' with it's color string.

  • svg/SVGAnimationElement.cpp: (WebCore::adjustForCurrentColor): (WebCore::SVGAnimationElement::startedActiveInterval):

2010-03-30 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVG Animation doesn't respect 'currentColor'
​https://bugs.webkit.org/show_bug.cgi?id=36695

Check if SVG animation takes the color value of the target element as 'currentColor'.

  • platform/mac/svg/custom/animation-currentColor-expected.checksum: Added.
  • platform/mac/svg/custom/animation-currentColor-expected.png: Added.
  • platform/mac/svg/custom/animation-currentColor-expected.txt: Added.
  • svg/custom/animation-currentColor.svg: Added.
2:27 AM Changeset in webkit [56774] by ukai@chromium.org
  • 2 edits in trunk/WebKitTools

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

Reviewed by Shinichiro Hamaji.

[Chromium-Win] subprocess.call should be called with stdin=open(os.devnull,'r')
​https://bugs.webkit.org/show_bug.cgi?id=36811

subproess.Popen() on Python 2.4/Windows with stdout,stdout, but no stdin will fail, because it uses return value of GetStdHandle(STD_INPUT_HANDLE), but DuplicateHandle requires integer, not the handle.
We don't need stdin, so use devnull as stdin.
Same as ​https://bugs.webkit.org/show_bug.cgi?id=36586

  • Scripts/webkitpy/layout_tests/port/chromium_win.py: open os.devnull for stdin
2:11 AM Changeset in webkit [56773] by yurys@chromium.org
  • 2 edits in trunk/LayoutTests

2010-03-30 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Pavel Feldman.

Skip http/tests/inspector-enabled on Linux and Mac where LayoutTestContoller doesn't support inspector methods yet.

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

  • platform/chromium/test_expectations.txt:
1:59 AM Changeset in webkit [56772] by pfeldman@chromium.org
  • 4 edits in trunk/WebCore

2010-03-30 Ilya Tikhonovsky <​loislo@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: Visible/captured records counter should be implemented in Timeline panel.
​https://bugs.webkit.org/show_bug.cgi?id=36708

  • English.lproj/localizedStrings.js:
  • inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel.prototype.get statusBarItems): (WebInspector.TimelinePanel.prototype._createStatusbarButtons): (WebInspector.TimelinePanel.prototype._updateRecordsCounter): (WebInspector.TimelinePanel.prototype._innerAddRecordToTimeline): (WebInspector.TimelinePanel.prototype._createRootRecord): (WebInspector.TimelinePanel.prototype._refresh): (WebInspector.TimelinePanel.prototype._filterRecords):
  • inspector/front-end/inspector.css: (.timeline-records-counter):
12:53 AM April 2010 Meeting edited by Philippe Normand
Igalia people (diff)
12:24 AM Changeset in webkit [56771] by levin@chromium.org
  • 3 edits in trunk/LayoutTests

[chromium] Many new test results are needed due to recent check-ins.
​https://bugs.webkit.org/show_bug.cgi?id=36799

Reviewed by Adam Barth.

  • platform/chromium-win/tables/mozilla_expected_failures/marvin/backgr_border-table-row-group-expected.checksum: Due to r56655.
  • platform/chromium-win/tables/mozilla_expected_failures/marvin/backgr_border-table-row-group-expected.png: Ditto.
12:09 AM Changeset in webkit [56770] by Philippe Normand
  • 2 edits in trunk/WebCore

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

[GStreamer] define static variables with DEFINE_STATIC_LOCAL macro
​https://bugs.webkit.org/show_bug.cgi?id=36653

Unreviewed, don't define the bool variable with
DEFINE_STATIC_LOCAL, it makes sense only for objects which have a
destructor. Thanks to Dan Bernstein for spotting this.

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::mimeTypeCache):
12:08 AM Changeset in webkit [56769] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-03-29 Gyuyoung Kim <​gyuyoung.kim@samsung.com>

Reviewed by Eric Seidel.

There is a build break due to the disabled() when wml feature is
enabled. This break comes from the Bug 35056 - Disabled menu options
are still submitted.(​https://bugs.webkit.org/show_bug.cgi?id=35056)
The WMLOptionElement.h needs to define the disabled() as well.
And, there is an additional break due to Bug 36177 - Multiselect Popup

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

  • wml/WMLOptionElement.h: (WebCore::WMLOptionElement::disabled):
  • wml/WMLSelectElement.h: (WebCore::WMLSelectElement::listBoxSelectItem):
12:02 AM Changeset in webkit [56768] by levin@chromium.org
  • 2 edits in trunk/LayoutTests

Rubber-stamped by Adam Barth.

Needed to fix test_expectations.txt for recent addition in r56764.

  • platform/chromium/test_expectations.txt: Added IMAGE (due to the results on OSX).
Note: See TracTimeline for information about the timeline view.