Timeline


and

01/16/11:

23:54 Changeset [75922] by tkent@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

2011-01-16 Kent Tamura <tkent@chromium.org>

Unreviewed, Chromium-linux build fix.

  • gyp_webkit: Fix a relative path to the top directory.
23:50 Changeset [75921] by mrowe@apple.com
  • 2 edits in tags/Safari-534.15.13/WebCore

Merge r75916.

23:49 Changeset [75920] by mrowe@apple.com
  • 2 edits in tags/Safari-534.15.13/WebCore

Merge r75906.

23:49 Changeset [75919] by mrowe@apple.com
  • 5 edits in tags/Safari-534.15.13

Versioning.

23:37 Changeset [75918] by abarth@webkit.org
  • 4 edits in trunk/Tools

2011-01-16 Adam Barth <abarth@webkit.org>

Update more include paths to reflect WebKit move.

  • DumpRenderTree/chromium/EventSender.cpp: (EventSender::keyDown): (EventSender::needsShiftModifier):
  • DumpRenderTree/chromium/WebThemeEngineDRTMac.h:
  • DumpRenderTree/chromium/WebThemeEngineDRTMac.mm:
23:29 Changeset [75917] by tkent@chromium.org
  • 2 edits in trunk/Tools

2011-01-16 Kent Tamura <tkent@chromium.org>

Unreviewed, Chromium build fix.

  • Scripts/update-webkit-chromium: Webkit/chromium -> Source/WebKit/chromium
23:11 Changeset [75916] by mitz@apple.com
  • 2 edits in trunk/Source/WebCore

<rdar://problem/8871903> REGRESSION (r75897): Crash with 0-by-0 iframe in scaled WebView

Reviewed by Sam Weinig.

Avoid use of FloatRect::enclosingBoundingBox(), which can stretch an empty
quad to a non-empty rect.

  • rendering/RenderWidget.cpp:

(WebCore::RenderWidget::setWidget):
(WebCore::RenderWidget::updateWidgetPosition):

22:52 Changeset [75915] by abarth@webkit.org
  • 2 edits in trunk/Source/WebKit/chromium

2011-01-16 Adam Barth <abarth@webkit.org>

Roll Chromium DEPS to pick up the changes reflecting the new location
of WebKit.

  • DEPS:
22:24 Changeset [75914] by mrobinson@webkit.org
  • 6 edits in trunk

2011-01-16 Martin Robinson <mrobinson@igalia.com>

Reviewed by Xan Lopez.

[GTK] fast/events/scroll-after-click-on-tab-index has been failing on the bots
https://bugs.webkit.org/show_bug.cgi?id=49177

  • platform/gtk/Skipped: Unskip this test, as it's now passing.

2011-01-16 Martin Robinson <mrobinson@igalia.com>

Reviewed by Xan Lopez.

[GTK] fast/events/scroll-after-click-on-tab-index has been failing on the bots
https://bugs.webkit.org/show_bug.cgi?id=49177

  • platform/ScrollView.cpp: Move the platform guards around a little. GTK+ now shares the implementation of platformAddChild and platformRemoveChild, but has its own implementation of removeChild, which knows how to inform main frame scrollbars that they no longer control the WebCore scrollbar.
  • platform/gtk/MainFrameScrollbarGtk.cpp: (MainFrameScrollbarGtk::attachAdjustment): If we are attaching the same adjustment that we already have, bail out early. Apply the value changed signal handler after configuring the adjustment. We don't want our reset of the adjustment to stomp on WebCore values. (MainFrameScrollbarGtk::gtkValueChanged): Do not adjust the value if the WebCore state already matches ours. This prevents some unnecessary recursion
  • platform/gtk/ScrollViewGtk.cpp: (WebCore::ScrollView::removeChild): Added, special cases main frame scrollbars which need their adjustments detached.
22:23 Changeset [75913] by mrowe@apple.com
  • 1 copy in tags/Safari-534.15.13

New tag.

22:20 Changeset [75912] by yuzo@google.com
  • 5 edits in trunk/Source/WebCore

2011-01-13 Yuzo Fujishima <yuzo@google.com>

Reviewed by Antti Koivisto.

Fix for Bug 52427 - Inconsistent use of m_cache in CachedResourceLoader
https://bugs.webkit.org/show_bug.cgi?id=52427

In constructor/destructor of CachedResourceLoader, m_cache has been
used to call MemoryCache::addCachedResourceLoader/removeCachedResourceLoader
while cache() is used everywhere else.

Actually addCachedResourceLoader/removeCachedResourceLoader need not be called at all.
Remove the call sites and make MemoryCache non-friend of CachedResourceLoader.

No new tests because the behavior remains the same.

  • loader/cache/CachedResourceLoader.cpp: (WebCore::CachedResourceLoader::CachedResourceLoader): (WebCore::CachedResourceLoader::~CachedResourceLoader):
  • loader/cache/CachedResourceLoader.h:
  • loader/cache/MemoryCache.cpp:
  • loader/cache/MemoryCache.h:
21:59 Changeset [75911] by mrobinson@webkit.org
  • 2 edits in trunk/Tools

2011-01-16 Martin Robinson <mrobinson@igalia.com>

Build fix for GTK+ after source directory movement.

  • GNUmakefile.am: Correct include paths.
21:09 Changeset [75910] by abarth@webkit.org
  • 2 edits in trunk/Tools

2011-01-16 Adam Barth <abarth@webkit.org>

Teach svn-apply about WebKit move.

  • Scripts/VCSUtils.pm:
21:04 Changeset [75909] by abarth@webkit.org
  • 1 move in trunk/Source/WebKit

Move WebKit into Source

21:02 Changeset [75908] by abarth@webkit.org
  • 48 edits in trunk

2011-01-16 Adam Barth <abarth@webkit.org>

Rubber-stamped by Eric Seidel.

Move WebKit into Source
https://bugs.webkit.org/show_bug.cgi?id=52530

  • WebKit.grd:
  • WebKit.gyp:

2011-01-16 Adam Barth <abarth@webkit.org>

Rubber-stamped by Eric Seidel.

Move WebKit into Source
https://bugs.webkit.org/show_bug.cgi?id=52530

  • Api/DerivedSources.pro:
  • declarative/declarative.pro:
  • docs/docs.pri:
  • tests/tests.pri:

2011-01-16 Adam Barth <abarth@webkit.org>

Rubber-stamped by Eric Seidel.

Move WebKit into Source
https://bugs.webkit.org/show_bug.cgi?id=52530

  • GNUmakefile.am:

2011-01-16 Adam Barth <abarth@webkit.org>

Rubber-stamped by Eric Seidel.

Move WebKit into Source
https://bugs.webkit.org/show_bug.cgi?id=52530

  • GNUmakefile.am:

2011-01-16 Adam Barth <abarth@webkit.org>

Rubber-stamped by Eric Seidel.

Move WebKit into Source
https://bugs.webkit.org/show_bug.cgi?id=52530

  • Makefile:
  • scripts/generate-webkitversion.pl:

2011-01-16 Adam Barth <abarth@webkit.org>

Rubber-stamped by Eric Seidel.

Move WebKit into Source
https://bugs.webkit.org/show_bug.cgi?id=52530

  • WebKit.vcproj/Interfaces.vcproj:
  • WebKit.vcproj/WebKit.sln:
  • WebKit.vcproj/WebKit.vcproj:
  • WebKit.vcproj/WebKitGUID.vcproj:

2011-01-16 Adam Barth <abarth@webkit.org>

Rubber-stamped by Eric Seidel.

Move WebKit into Source
https://bugs.webkit.org/show_bug.cgi?id=52530

  • Android.mk:
  • CMakeLists.txt:
  • DerivedSources.pro:
  • GNUmakefile.am:
  • Makefile:
  • WebKit.pri:
  • WebKit.pro:
  • configure.ac:
  • wscript:

2011-01-16 Adam Barth <abarth@webkit.org>

Rubber-stamped by Eric Seidel.

Move WebKit into Source
https://bugs.webkit.org/show_bug.cgi?id=52530

  • DumpRenderTree/mac/InternalHeaders/WebKit/WebTypesInternal.h:
  • DumpRenderTree/qt/DumpRenderTreeQt.cpp:
  • DumpRenderTree/qt/DumpRenderTreeQt.h:
  • DumpRenderTree/qt/GCControllerQt.cpp:
  • DumpRenderTree/qt/LayoutTestControllerQt.cpp:
  • DumpRenderTree/qt/PlainTextControllerQt.cpp:
  • DumpRenderTree/qt/TextInputControllerQt.cpp:
  • DumpRenderTree/wscript:
  • QtTestBrowser/launcherwindow.h:
  • Scripts/build-webkit:
  • Scripts/webkitdirs.pm:

2011-01-16 Adam Barth <abarth@webkit.org>

Rubber-stamped by Eric Seidel.

Move WebKit into Source
https://bugs.webkit.org/show_bug.cgi?id=52530

2011-01-16 Adam Barth <abarth@webkit.org>

Rubber-stamped by Eric Seidel.

Move WebKit into Source
https://bugs.webkit.org/show_bug.cgi?id=52530

  • WebCore.gyp/WebCore.gyp:
  • WebCore.pri:
  • WebCore.pro:
20:21 Changeset [75907] by mitz@apple.com
  • 2 edits in trunk/WebKit/mac

Assertion failure (!inSetWindow) with in-process plug-in in plugins/destroy-on-setwindow.html
https://bugs.webkit.org/show_bug.cgi?id=52550

Reviewed by Simon Fraser.

  • Plugins/WebNetscapePluginView.mm:

(-[WebNetscapePluginView setWindowIfNecessary]): Removed the assertion, making sure
that inSetWindow remains YES until we exit the top-level setWindowIfNecessary.

19:49 Changeset [75906] by simon.fraser@apple.com
  • 2 edits in trunk/Source/WebCore

2011-01-16 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dan Bernstein.

frame-removed-during-resize.html test crashes (shows up as image-map-2.html crash)
https://bugs.webkit.org/show_bug.cgi?id=52549

Fix regression from r75900; m_widget->setFrameRect() can run script that
clears m_widget, so null-check it before calling setBoundsSize().

Tested by fast/replaced/frame-removed-during-resize.html

  • rendering/RenderWidget.cpp: (WebCore::RenderWidget::setWidgetGeometry):
18:31 Changeset [75905] by mrowe@apple.com
  • 3 edits in tags/Safari-534.15.12/WebKit/mac

Merge r75893.

18:31 Changeset [75904] by mrowe@apple.com
  • 11 edits in tags/Safari-534.15.12

Merge r75897.

18:31 Changeset [75903] by mrowe@apple.com
  • 4 edits
    2 adds in tags/Safari-534.15.12

Merge r75720.

18:30 Changeset [75902] by mrowe@apple.com
  • 2 edits in tags/Safari-534.15.12/WebCore

Merge r75892.

18:30 Changeset [75901] by mrowe@apple.com
  • 5 edits in tags/Safari-534.15.12

Versioning.

18:01 Changeset [75900] by simon.fraser@apple.com
  • 2 edits in trunk/Source/WebCore

2011-01-16 Simon Fraser <simon.fraser@apple.com>

Keep Leopard build happy.

  • platform/mac/WidgetMac.mm: (WebCore::Widget::setBoundsSize):
17:52 Changeset [75899] by oliver@apple.com
  • 5 edits in trunk

2011-01-16 Oliver Hunt <oliver@apple.com>

Reviewed by Sam Weinig.

[jsfunfuzz] Parser doesn't correctly validate for-loop syntax
https://bugs.webkit.org/show_bug.cgi?id=52516

Add test for the case where a syntax error immediately follows
multiple declarations in a for-loop initialiser.

  • fast/js/parser-syntax-check-expected.txt:
  • fast/js/script-tests/parser-syntax-check.js:

2011-01-16 Oliver Hunt <oliver@apple.com>

Reviewed by Sam Weinig.

[jsfunfuzz] Parser doesn't correctly validate for-loop syntax
https://bugs.webkit.org/show_bug.cgi?id=52516

Ensure that we always check for a semicolon after encountering
multiple declarations in the initialiser portion of a for-loop.

  • parser/JSParser.cpp: (JSC::JSParser::parseForStatement):
17:20 Changeset [75898] by mrowe@apple.com
  • 1 copy in tags/Safari-534.15.12

New tag.

17:07 Changeset [75897] by simon.fraser@apple.com
  • 11 edits in trunk

2011-01-16 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dan Bernstein.

Issues with iframes and plugins when the WebView is scaled.
<rdar://problem/6213380>

When _scaleWebView has been called on a WebView, iframes
in WebKit1 render and hit-test incorrectly, and plug-ins don't scale up.
This is caused by AppKit NSViews not playing nicely with the scale
applied through style.

Work around most of these issues by adjusting the bounds size
of widgets to allow iframe contents to paint with the correct scale,
and fix various places in the code where we relied on coordinate
transforms via NSViews (which ignore CSS transforms).

  • WebCore.exp.in:
  • platform/Widget.cpp: (WebCore::Widget::setBoundsSize):
  • platform/Widget.h:
  • platform/mac/WidgetMac.mm: (WebCore::Widget::setBoundsSize): (WebCore::Widget::paint):
  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::shouldPropagateCompositingToEnclosingIFrame):
  • rendering/RenderWidget.cpp: (WebCore::RenderWidget::setWidgetGeometry): (WebCore::RenderWidget::setWidget): (WebCore::RenderWidget::updateWidgetPosition):
  • rendering/RenderWidget.h:
15:54 Changeset [75896] by oliver@apple.com
  • 7 edits in trunk

2011-01-16 Oliver Hunt <oliver@apple.com>

Reviewed by Geoffrey Garen.

Strict mode restrictions on arguments and eval usage aren't complete
https://bugs.webkit.org/show_bug.cgi?id=52528

Add a helper function to aid parse failure tests by testing both direct
eval/global code, and nested code (so that we test the syntax checker
mode as well)

  • fast/js/basic-strict-mode-expected.txt:
  • fast/js/script-tests/basic-strict-mode.js: (shouldBeSyntaxError):

2011-01-16 Oliver Hunt <oliver@apple.com>

Reviewed by Geoffrey Garen.

Strict mode restrictions on arguments and eval usage aren't complete
https://bugs.webkit.org/show_bug.cgi?id=52528

Fix a few bugs in strict mode where we incorrect allow mutation of
arguments and eval in the parser.

Alas the "optimisation" used by the syntax checker for validating
binary and unary expressions was too aggressive: we do actually need
a stack for operations and operands although it needn't be as complete
as that used for the full AST builder.

Also disallow assignment to arguments in all cases as allowing arguments
to be assignable is always an error in strict mode, regardless of context.

  • parser/ASTBuilder.h: (JSC::ASTBuilder::BinaryExprContext::BinaryExprContext): (JSC::ASTBuilder::UnaryExprContext::UnaryExprContext):
  • parser/JSParser.cpp: (JSC::JSParser::parseAssignmentExpression): (JSC::JSParser::parseBinaryExpression): (JSC::JSParser::parseUnaryExpression):
  • parser/SyntaxChecker.h: (JSC::SyntaxChecker::BinaryExprContext::BinaryExprContext): (JSC::SyntaxChecker::BinaryExprContext::~BinaryExprContext): (JSC::SyntaxChecker::UnaryExprContext::UnaryExprContext): (JSC::SyntaxChecker::UnaryExprContext::~UnaryExprContext): (JSC::SyntaxChecker::appendBinaryExpressionInfo): (JSC::SyntaxChecker::operatorStackPop):
14:08 Changeset [75895] by robert@webkit.org
  • 2 edits in trunk/LayoutTests

2011-01-16 Robert Hogan <robert@webkit.org>

Unreviewed, re-skip test after r75894.

[Qt] plugins/keyboard-events.html fails after r72717

Re-skip plugins/keyboard-events.html.
Using XKeysymToKeycode(QX11Info::display(), ..) doesn't work on
a headless build, such as the buildbot. So need to solve this a
different way.

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

  • platform/qt/Skipped:
13:08 Changeset [75894] by robert@webkit.org
  • 4 edits in trunk

2011-01-16 Robert Hogan <robert@webkit.org>

Reviewed by Andreas Kling.

[Qt] plugins/keyboard-events.html fails after r72717
https://bugs.webkit.org/show_bug.cgi?id=50050

  • platform/qt/Skipped: Unskip plugins/keyboard-events.html

2011-01-16 Robert Hogan <robert@webkit.org>

Reviewed by Andreas Kling.

[Qt] plugins/keyboard-events.html fails after r72717
https://bugs.webkit.org/show_bug.cgi?id=50050

  • plugins/qt/PluginViewQt.cpp: (WebCore::setXKeyEventSpecificFields): map event text to keycode
12:59 Changeset [75893] by bdakin@apple.com
  • 3 edits in trunk/WebKit/mac

Fix for <rdar://problem/8871204>

Reviewed by Kevin Decker.

Don't try to save elasticity state. Just rely on
NSScrollElasticityAutomatic to restore the correct state.

  • WebCoreSupport/WebFrameLoaderClient.h:
  • WebCoreSupport/WebFrameLoaderClient.mm:

(WebFrameLoaderClient::WebFrameLoaderClient):
(WebFrameLoaderClient::dispatchDidFirstLayout):
(WebFrameLoaderClient::provisionalLoadStarted):

10:04 Changeset [75892] by simon.fraser@apple.com
  • 2 edits in trunk/Source/WebCore

2011-01-16 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dan Bernstein.

RenderView needs to take transforms on its layer into account
https://bugs.webkit.org/show_bug.cgi?id=52536

The RenderView's coordinate mapping methods failed to
take into account a transform on the RenderView's layer.

No tests because it's not possible to get a transform
on the RenderView's layer through content.

  • rendering/RenderView.cpp: (WebCore::RenderView::mapLocalToContainer): (WebCore::RenderView::mapAbsoluteToLocalPoint):

01/15/11:

21:11 Changeset [75891] by ggaren@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

2011-01-15 Geoffrey Garen <ggaren@apple.com>

Reviewed by Oliver Hunt.

Rolled back in r75886.
https://bugs.webkit.org/show_bug.cgi?id=52527


r75886 broke the GTK Linux bot because Linux was -- quite surprisingly --
set up to use the constants for embedded devices.

  • jit/ExecutableAllocatorFixedVMPool.cpp: (JSC::maybeModifyVMPoolSize): Separated Linux constants from embedded constants.
20:22 Changeset [75890] by commit-queue@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

2011-01-15 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r75886.
http://trac.webkit.org/changeset/75886
https://bugs.webkit.org/show_bug.cgi?id=52526

"Broke GTK+ 64bit" (Requested by xan_ on #webkit).

  • jit/ExecutableAllocatorFixedVMPool.cpp:
18:14 Changeset [75889] by ddkilzer@apple.com
  • 2 edits in trunk/LayoutTests

Skip fast/dom/Range/range-clone-contents.html on Windows

Not reviewed.

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

  • platform/win/Skipped: Skip test on Windows. The test passes

on all other bots.

16:57 Changeset [75888] by mrowe@apple.com
  • 5 edits in tags/Safari-534.15.9/WebKit/mac

Merge r75873.

16:57 Changeset [75887] by mrowe@apple.com
  • 5 edits in tags/Safari-534.15.9

Versioning.

15:48 Changeset [75886] by ggaren@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

<rdar://problem/8870429> Shrink the executable pool on embedded devices

Reviewed by Sam Weinig.

  • jit/ExecutableAllocatorFixedVMPool.cpp: Dropped the pool size from 32MB

to 16MB.

15:41 Changeset [75885] by commit-queue@webkit.org
  • 14 edits
    2 deletes in trunk

2011-01-15 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r75708.
http://trac.webkit.org/changeset/75708
https://bugs.webkit.org/show_bug.cgi?id=52521

Breaks Qt build if mobility is not installed. (Requested by
benjaminp on #webkit).

  • WebCore.pro:
  • platform/graphics/qt/MediaPlayerPrivateQt.cpp: (WebCore::MediaPlayerPrivateQt::MediaPlayerPrivateQt):
  • platform/graphics/qt/MediaPlayerPrivateQt.h: (WebCore::MediaPlayerPrivateQt::supportsFullscreen):

2011-01-15 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r75708.
http://trac.webkit.org/changeset/75708
https://bugs.webkit.org/show_bug.cgi?id=52521

Breaks Qt build if mobility is not installed. (Requested by
benjaminp on #webkit).

  • Api/qwebkitplatformplugin.h:
  • WebCoreSupport/ChromeClientQt.cpp: (WebCore::ChromeClientQt::ChromeClientQt): (WebCore::ChromeClientQt::~ChromeClientQt):
  • WebCoreSupport/ChromeClientQt.h:
  • WebCoreSupport/FullScreenVideoQt.cpp: Removed.
  • WebCoreSupport/FullScreenVideoQt.h: Removed.
  • WebCoreSupport/QtPlatformPlugin.cpp:
  • WebCoreSupport/QtPlatformPlugin.h:
  • examples/platformplugin/WebPlugin.cpp: (WebPlugin::supportsExtension): (WebPlugin::createExtension):
  • examples/platformplugin/WebPlugin.h:
  • examples/platformplugin/platformplugin.pro:
  • examples/platformplugin/qwebkitplatformplugin.h:
15:39 Changeset [75884] by oliver@apple.com
  • 5 edits in trunk

2011-01-15 Oliver Hunt <oliver@apple.com>

Reviewed by Maciej Stachowiak.

Incorrect behavior changing attributes of an accessor
https://bugs.webkit.org/show_bug.cgi?id=52515

defineProperty doesn't correctly handle changing attributes of an accessor
property. This is because we don't pass the full descriptor to the
putDescriptor helper function, which means we have insufficient information
to do the right thing. Once that's passed the correct behavior is relatively
simple to implement.

  • runtime/JSObject.cpp: (JSC::putDescriptor): (JSC::JSObject::defineOwnProperty):
15:33 Changeset [75883] by mrowe@apple.com
  • 1 copy in tags/Safari-534.15.9

New tag.

12:44 Changeset [75882] by ddkilzer@apple.com
  • 3 edits
    2 adds in trunk

<http://webkit.org/b/52512> REGRESSION(r73818): range.cloneContents() ignores end offset

Reviewed by Adele Peterson.

WebCore:

The fix for Bug 50710 in r73799 introduced an off-by-one error
when copying nodes to a local NodeVector for processing. A fix
was attempted for Bug 50854 in r73818, but instead of stopping
at the end offset, it iterates through all the sibling nodes
because the loop variable (i) is never incremented. To clean
this up, revert back to the code in r73799 and fix the
off-by-one error.

Test: fast/dom/Range/range-clone-contents.html

  • dom/Range.cpp:

(WebCore::Range::processContents): Fix the loop that copies
nodes to a local NodeVector by restoring the code from r73799
and fixing the off-by-one error.

LayoutTests:

  • fast/dom/Range/range-clone-contents-expected.txt: Added.
  • fast/dom/Range/range-clone-contents.html: Added.
12:04 Changeset [75881] by abarth@webkit.org
  • 2 edits in trunk/Tools

2011-01-15 Adam Barth <abarth@webkit.org>

Teach svn-apply about WebKit2 move.

  • Scripts/VCSUtils.pm:
12:02 Changeset [75880] by abarth@webkit.org
  • 32 edits
    1 move in trunk

Move WebKit2 into Source
https://bugs.webkit.org/show_bug.cgi?id=52438

Rubber-stamped by Eric Seidel.

.:

Update reference to WebKit2.

  • DerivedSources.pro:
  • GNUmakefile.am:
  • Makefile:
  • Source/cmake/WebKitPackaging.cmake:
  • WebKit.pro:
  • configure.ac:

Source/WebCore:

  • WebCore.pro:

Source/WebKit2:

  • DerivedSources.pro:
  • GNUmakefile.am:
  • WebKit2.pro:
  • WebProcess.pro:
  • win/WebKit2.vcproj:
  • win/WebKit2WebProcess.vcproj:

Tools:

  • MiniBrowser/qt/MiniBrowser.pro:
  • Scripts/build-webkit:
  • Scripts/do-file-rename:
  • Scripts/do-webcore-rename:
  • Scripts/test-webkitpy:
  • Scripts/webkitdirs.pm:
  • Scripts/webkitpy/style/checker.py:
  • WebKitTestRunner/DerivedSources.pro:
  • WebKitTestRunner/InjectedBundle/qt/InjectedBundle.pro:
  • WebKitTestRunner/qt/WebKitTestRunner.pro:

WebKit/win:

Update reference to WebKit2.

  • WebKit.vcproj/WebKit.sln:

Websites/bugs.webkit.org:

Remove WebKit2 from the list of top-level source directories.

  • PrettyPatch/PrettyPatch.rb:
11:26 Changeset [75879] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

2011-01-15 Joone Hur <joone.hur@collabora.co.uk>

Reviewed by Martin Robinson.

[GTK] Linux build with FileSystem API enabled fails
https://bugs.webkit.org/show_bug.cgi?id=43878

This patch allows WebKitGtk+ to build with FileSystem API option.

No new tests because no new functionality.

  • GNUmakefile.am: Included AsyncFileSystem.h,cpp and Excluded duplicated JSFileException.h,cpp.
  • bindings/js/JSDirectoryEntryCustom.cpp: Included ExceptionCode.h.
06:51 Changeset [75878] by robert@webkit.org
  • 4 edits in trunk

2011-01-15 Robert Hogan <robert@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] plugins/no-mime-with-valid-extension.html fails

Check extension of resource against installed
plugins when determining mime-type.

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

  • platform/qt/Skipped:

2011-01-15 Robert Hogan <robert@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] plugins/no-mime-with-valid-extension.html fails
Check extension of resource against installed
plugins when determining mime-type.

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

  • WebCoreSupport/FrameLoaderClientQt.cpp: (WebCore::FrameLoaderClientQt::objectContentType):
06:17 Changeset [75877] by senorblanco@chromium.org
  • 2 edits in trunk/LayoutTests

2011-01-15 Stephen White <senorblanco@chromium.org>

Unreviewed; updated chromium test expectations.

  • platform/chromium-gpu/test_expectations.txt:
06:11 Changeset [75876] by senorblanco@chromium.org
  • 3 edits in trunk/LayoutTests

2011-01-15 Stephen White <senorblanco@chromium.org>

Unreviewed; updated chromium test expectations.

  • platform/chromium-gpu/test_expectations.txt:
  • platform/chromium/test_expectations.txt:
01:33 Changeset [75875] by jochen@chromium.org
  • 5 edits in trunk

2011-01-15 Jochen Eisinger <jochen@chromium.org>

Reviewed by David Levin.

[chromium] remove obsolete createView method
https://bugs.webkit.org/show_bug.cgi?id=52437

  • DumpRenderTree/chromium/WebViewHost.cpp: (WebViewHost::createView):
  • DumpRenderTree/chromium/WebViewHost.h:

2011-01-15 Jochen Eisinger <jochen@chromium.org>

Reviewed by David Levin.

[chromium] remove obsolete createView method
https://bugs.webkit.org/show_bug.cgi?id=52437

  • public/WebViewClient.h: (WebKit::WebViewClient::createView):
00:58 Changeset [75874] by tony@chromium.org
  • 2 edits in trunk/LayoutTests

2011-01-15 Tony Chang <tony@chromium.org>

Unreviewed, add new test to GTK+ Skipped list.
https://bugs.webkit.org/show_bug.cgi?id=52508

  • platform/gtk/Skipped:

01/14/11:

22:48 Changeset [75873] by simon.fraser@apple.com
  • 5 edits in trunk/WebKit/mac

2011-01-14 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dan Bernstein.

Plugins render incorrectly with transformed ancestors
https://bugs.webkit.org/show_bug.cgi?id=52507

-[NSView visibleRect] gives the wrong answer if there are CSS transforms
in the ancestor chain of a plugin.

So use of this method with calls to -actualVisibleRectInWindow, which
maps rects through the render tree to compute the correct rect,
clipping via windowClipRect() if necessary.

Not testable, because doing so relies on the behavior of some
plugins, which stop rendering if setWindow passes an empty rect.

  • Plugins/Hosted/WebHostedNetscapePluginView.mm: (-[WebHostedNetscapePluginView updateAndSetWindow]):
  • Plugins/WebBaseNetscapePluginView.h:
  • Plugins/WebBaseNetscapePluginView.mm: (-[WebBaseNetscapePluginView actualVisibleRectInWindow]):
  • Plugins/WebNetscapePluginView.mm: (-[WebNetscapePluginView saveAndSetNewPortStateForUpdate:]):
22:35 Changeset [75872] by oliver@apple.com
  • 5 edits in trunk

2011-01-14 Oliver Hunt <oliver@apple.com>

Reviewed by Maciej Stachowiak.

[jsfunfuzz] Incorrect handling of consecutive duplicate labels
https://bugs.webkit.org/show_bug.cgi?id=52505

Add a few tests of consecutive labels containing duplicates to the parser check.

  • fast/js/parser-syntax-check-expected.txt:
  • fast/js/script-tests/parser-syntax-check.js:

2011-01-14 Oliver Hunt <oliver@apple.com>

Reviewed by Maciej Stachowiak.

[jsfunfuzz] Incorrect handling of consecutive duplicate labels
https://bugs.webkit.org/show_bug.cgi?id=52505

Compare StringImpl*'s instead of Identifier*'s when looking for duplicate
labels.

  • parser/JSParser.cpp: (JSC::JSParser::parseExpressionOrLabelStatement):
22:01 Changeset [75871] by commit-queue@webkit.org
  • 2 edits in trunk/WebKit/chromium

2011-01-14 Xiyuan Xia <xiyuan@chromium.org>

Reviewed by Darin Fisher.

Prepare Linux WebThemeEngine interface to support new theme parts.
https://bugs.webkit.org/show_bug.cgi?id=52413

  • public/linux/WebThemeEngine.h:
21:38 Changeset [75870] by commit-queue@webkit.org
  • 12 edits in trunk

2011-01-14 Sam Magnuson <smagnuso@gmail.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Compile with QT_NO_GRAPHICSVIEW
https://bugs.webkit.org/show_bug.cgi?id=49750

  • platform/graphics/qt/GraphicsLayerQt.cpp:
  • platform/graphics/qt/GraphicsLayerQt.h:
  • platform/qt/PlatformMouseEventQt.cpp:

2011-01-14 Sam Magnuson <smagnuso@gmail.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Compile with QT_NO_GRAPHICSVIEW
https://bugs.webkit.org/show_bug.cgi?id=49750

  • Api/qgraphicswebview.cpp:
  • Api/qgraphicswebview.h:
  • Api/qwebpage.cpp: (QWebPage::event):
  • Api/qwebpage_p.h:
  • WebCoreSupport/FrameLoaderClientQt.cpp: (WebCore::FrameLoaderClientQt::createPlugin):
  • WebCoreSupport/PageClientQt.cpp:
  • WebCoreSupport/PageClientQt.h:
20:20 Changeset [75869] by dimich@chromium.org
  • 5 edits
    1 delete in trunk/LayoutTests

Not reviewed, update of Chromium test expectations.
Update of files affected by mass rebaseline in r75828 and r75838

  • platform/chromium-mac/animations/3d/state-at-end-event-transform-expected.checksum:
  • platform/chromium-mac/animations/3d/state-at-end-event-transform-expected.png:
  • platform/chromium-mac/plugins/embed-attributes-style-expected.txt: Removed.
  • platform/chromium-mac/svg/custom/svg-curve-with-relative-cordinates-expected.checksum:
  • platform/chromium-mac/svg/custom/svg-curve-with-relative-cordinates-expected.png:
19:53 Changeset [75868] by simon.fraser@apple.com
  • 1 edit in trunk/Source/JavaScriptCore/ChangeLog

2011-01-14 Simon Fraser <simon.fraser@apple.com>

No review.

Initialize m_operationInProgress after r75855.

  • runtime/Heap.cpp: (JSC::Heap::Heap):
19:32 Changeset [75867] by ggaren@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Reverted accidentally committed code from my last checkin.

  • runtime/Heap.cpp:

(JSC::Heap::markRoots):

19:28 Changeset [75866] by ggaren@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Try to fix the GTK bot.

Reviewed by Sam Weinig.

  • runtime/Heap.cpp:

(JSC::Heap::Heap):
(JSC::Heap::markRoots): Kids, remember to initialize your data members.
Knowing is half the battle.

18:33 Changeset [75865] by dimich@chromium.org
  • 2 edits in trunk/LayoutTests

Not reviewed, update of test expectations.
Ignore new test fast/js/js-continue-break-restrictions.html, it fails for V8.

  • platform/chromium/test_expectations.txt:
18:26 Changeset [75864] by ericu@chromium.org
  • 2 edits
    6 adds in trunk/LayoutTests

2011-01-14 Eric Uhrhane <ericu@chromium.org>

Reviewed by David Levin.

Add tests for FileWriterSync
https://bugs.webkit.org/show_bug.cgi?id=50720

  • fast/filesystem/resources/file-writer-sync-truncate-extend.js: Added.
  • fast/filesystem/workers/file-writer-sync-truncate-extend.html: Added.
  • fast/filesystem/workers/file-writer-sync-truncate-extend-expected.txt: Added.
  • fast/filesystem/resources/file-writer-sync-write-overlapped.js: Added.
  • fast/filesystem/workers/file-writer-sync-write-overlapped.html: Added.
  • fast/filesystem/workers/file-writer-sync-write-overlapped-expected.txt: Added.
  • fast/filesystem/resources/file-writer-utils.js: (assert): Make assert throw, rather than continue. (cleanUp): Always call finishJSTest, even without fileEntryForCleanup.
18:22 Changeset [75863] by paroga@webkit.org
  • 3 edits in trunk/WebKit/efl

2011-01-14 Patrick Gansterer <paroga@webkit.org>

Unreviewed. Try to fix EFL build.

  • ewk/ewk_frame.cpp: (ewk_frame_scroll_add): Added WebCore namespace to IntSize.
  • ewk/ewk_view.cpp: (_ewk_view_viewport_attributes_compute): Ditto.
18:14 Changeset [75862] by oliver@apple.com
  • 5 edits in trunk

2011-01-14 Oliver Hunt <oliver@apple.com>

Reviewed by Stephanie Lewis.

[jsfunfuzz] We should be clearing the lexers temporary character buffers when switching to strict mode
https://bugs.webkit.org/show_bug.cgi?id=52501

Add checks for correct behaviour when the strict mode token is immediately followed
by a value that needs a temporary buffer in the lexer.

  • fast/js/basic-strict-mode-expected.txt:
  • fast/js/script-tests/basic-strict-mode.js:

2011-01-14 Oliver Hunt <oliver@apple.com>

Reviewed by Stephanie Lewis.

[jsfunfuzz] We should be clearing the lexers temporary character buffers when switching to strict mode
https://bugs.webkit.org/show_bug.cgi?id=52501

Clear the temporary character buffers used for reading escaped characters and
numbers.

  • parser/Lexer.h: (JSC::Lexer::setOffset):
18:06 Changeset [75861] by tony@chromium.org
  • 4 edits
    2 adds in trunk

2011-01-14 Tony Chang <tony@chromium.org>

Reviewed by Alexey Proskuryakov.

Strip NUL character when copying text on Windows
https://bugs.webkit.org/show_bug.cgi?id=52236

  • editing/pasteboard/copy-null-characters-expected.txt: Added.
  • editing/pasteboard/copy-null-characters.html: Added.

2011-01-14 Tony Chang <tony@chromium.org>

Reviewed by Alexey Proskuryakov.

Strip NUL character when copying text on Windows
https://bugs.webkit.org/show_bug.cgi?id=52236

Test: editing/pasteboard/copy-null-characters.html

  • editing/Editor.cpp: (WebCore::Editor::selectedText):
  • platform/mac/PasteboardMac.mm: (WebCore::Pasteboard::writeSelection): Use editor()->selectedText() which matches the other platforms.
18:00 Changeset [75860] by mrowe@apple.com
  • 4 edits in tags/Safari-534.15.8/WebCore

Merge r75758.

18:00 Changeset [75859] by mrowe@apple.com
  • 3 edits in tags/Safari-534.15.8/WebKit/mac

Merge r75829.

18:00 Changeset [75858] by mrowe@apple.com
  • 5 edits in tags/Safari-534.15.8

Versioning.

17:59 Changeset [75857] by yuzo@google.com
  • 32 edits in trunk

2011-01-14 Yuzo Fujishima <yuzo@google.com>

Reviewed by Antti Koivisto.

Rename cache() to memoryCache()
https://bugs.webkit.org/show_bug.cgi?id=52433

No new tests because the behavior remains the same.

  • WebCore.exp.in:
  • WebCore.order:
  • history/PageCache.cpp: (WebCore::PageCache::releaseAutoreleasedPagesNow):
  • inspector/InspectorResourceAgent.cpp: (WebCore::InspectorResourceAgent::cachedResource):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::tellClientAboutPastMemoryCacheLoads):
  • loader/archive/cf/LegacyWebArchive.cpp: (WebCore::LegacyWebArchive::create):
  • loader/cache/CachedImage.cpp: (WebCore::CachedImage::allClientsRemoved): (WebCore::CachedImage::data):
  • loader/cache/CachedResource.cpp: (WebCore::CachedResource::~CachedResource): (WebCore::CachedResource::addClientToSet): (WebCore::CachedResource::removeClient): (WebCore::CachedResource::setDecodedSize): (WebCore::CachedResource::setEncodedSize): (WebCore::CachedResource::didAccessDecodedData):
  • loader/cache/CachedResourceLoader.cpp: (WebCore::CachedResourceLoader::CachedResourceLoader): (WebCore::CachedResourceLoader::requestUserCSSStyleSheet): (WebCore::CachedResourceLoader::requestResource): (WebCore::CachedResourceLoader::revalidateResource): (WebCore::CachedResourceLoader::loadResource): (WebCore::CachedResourceLoader::clearPreloads): (WebCore::CachedResourceLoader::printPreloadStats):
  • loader/cache/CachedResourceRequest.cpp: (WebCore::CachedResourceRequest::load): (WebCore::CachedResourceRequest::didFail): (WebCore::CachedResourceRequest::didReceiveResponse):
  • loader/cache/MemoryCache.cpp: (WebCore::memoryCache):
  • loader/cache/MemoryCache.h:

2011-01-14 Yuzo Fujishima <yuzo@google.com>

Reviewed by Antti Koivisto.

Rename cache() to memoryCache()
https://bugs.webkit.org/show_bug.cgi?id=52433

  • src/WebCache.cpp: (WebKit::WebCache::setCapacities): (WebKit::WebCache::clear): (WebKit::WebCache::getUsageStats): (WebKit::WebCache::getResourceTypeStats):

2011-01-14 Yuzo Fujishima <yuzo@google.com>

Reviewed by Antti Koivisto.

Rename cache() to memoryCache()
https://bugs.webkit.org/show_bug.cgi?id=52433

  • webkit/webkitglobals.cpp: (webkit_set_cache_model):

2011-01-14 Yuzo Fujishima <yuzo@google.com>

Reviewed by Antti Koivisto.

Rename cache() to memoryCache()
https://bugs.webkit.org/show_bug.cgi?id=52433

  • Misc/WebCache.mm: (+[WebCache statistics]): (+[WebCache setDisabled:]): (+[WebCache isDisabled]):
  • WebView/WebView.mm: (+[WebView _setCacheModel:]):

2011-01-14 Yuzo Fujishima <yuzo@google.com>

Reviewed by Antti Koivisto.

Rename cache() to memoryCache()
https://bugs.webkit.org/show_bug.cgi?id=52433

  • Api/qwebpage.cpp: (QWebPagePrivate::_q_cleanupLeakMessages):
  • Api/qwebsettings.cpp: (QWebSettings::clearMemoryCaches): (QWebSettings::setObjectCacheCapacities):

2011-01-14 Yuzo Fujishima <yuzo@google.com>

Reviewed by Antti Koivisto.

Rename cache() to memoryCache()
https://bugs.webkit.org/show_bug.cgi?id=52433

  • WebCache.cpp: (WebCache::statistics): (WebCache::empty): (WebCache::setDisabled): (WebCache::disabled):
  • WebView.cpp: (WebView::setCacheModel):

2011-01-14 Yuzo Fujishima <yuzo@google.com>

Reviewed by Antti Koivisto.

Rename cache() to memoryCache()
https://bugs.webkit.org/show_bug.cgi?id=52433

  • WebView.cpp: (wxWebView::SetCachePolicy):

2011-01-14 Yuzo Fujishima <yuzo@google.com>

Reviewed by Antti Koivisto.

Rename cache() to memoryCache()
https://bugs.webkit.org/show_bug.cgi?id=52433

  • WebProcess/WebProcess.cpp: (WebKit::WebProcess::shutdownIfPossible): (WebKit::WebProcess::didClose):
  • WebProcess/mac/WebProcessMac.mm: (WebKit::WebProcess::platformSetCacheModel):
  • WebProcess/win/WebProcessWin.cpp: (WebKit::WebProcess::platformSetCacheModel):
17:52 Changeset [75856] by ggaren@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Try to fix non-Dtrace builds: #include Tracing.h instead of TracingDtrace.h.

  • runtime/Heap.cpp:
17:43 Changeset [75855] by ggaren@apple.com
  • 18 edits
    2 copies in trunk

2011-01-13 Geoffrey Garen <ggaren@apple.com>

Reviewed by Oliver Hunt.

Split out a MarkedSpace strategy object from Heap.
https://bugs.webkit.org/show_bug.cgi?id=52421

  • Misc/WebCoreStatistics.mm: (+[WebCoreStatistics memoryStatistics]): Updated for class move.

2011-01-13 Geoffrey Garen <ggaren@apple.com>

Reviewed by Oliver Hunt.

Split out a MarkedSpace strategy object from Heap.
https://bugs.webkit.org/show_bug.cgi?id=52421


SunSpider reports no change.

  • runtime/Heap.cpp: (JSC::Heap::Heap): (JSC::Heap::destroy): (JSC::Heap::recordExtraCost): (JSC::Heap::allocate): (JSC::Heap::markConservatively): (JSC::Heap::markRoots): (JSC::Heap::objectCount): (JSC::Heap::statistics): (JSC::Heap::size): (JSC::Heap::isBusy): (JSC::Heap::collectAllGarbage): (JSC::Heap::primaryHeapBegin): (JSC::Heap::primaryHeapEnd):
  • runtime/Heap.h: (JSC::Heap::globalData): (JSC::Heap::markedSpace): (JSC::Heap::isCellMarked): (JSC::Heap::checkMarkCell): (JSC::Heap::markCell): Moved all code pertaining to managing chunks of collector memory out of this class. Heap now just delegates to MarkedSpace.
  • runtime/JSCell.h: (JSC::JSCell::Heap::heap): Updated for MarkedSpace delegation.
  • runtime/JSValue.h: Moved the ValueStringPair typedef to help with #includes.
  • runtime/MarkedSpace.cpp: Copied from runtime/Heap.cpp. (JSC::MarkedSpace::MarkedSpace): (JSC::MarkedSpace::destroy): (JSC::MarkedSpace::allocateBlock): (JSC::MarkedSpace::freeBlock): (JSC::MarkedSpace::allocate): (JSC::MarkedSpace::resizeBlocks): (JSC::MarkedSpace::growBlocks): (JSC::MarkedSpace::shrinkBlocks): (JSC::MarkedSpace::markConservatively): (JSC::MarkedSpace::clearMarkBits): (JSC::MarkedSpace::markedCells): (JSC::MarkedSpace::sweep): (JSC::MarkedSpace::objectCount): (JSC::MarkedSpace::addToStatistics): (JSC::MarkedSpace::statistics): (JSC::MarkedSpace::size): (JSC::MarkedSpace::reset): (JSC::MarkedSpace::primaryHeapBegin): (JSC::MarkedSpace::primaryHeapEnd):
  • runtime/MarkedSpace.h: Copied from runtime/Heap.h. (JSC::MarkedSpace::globalData): (JSC::MarkedSpace::didShrink): (JSC::MarkedSpace::cellBlock): (JSC::MarkedSpace::cellOffset): (JSC::MarkedSpace::isCellMarked): (JSC::MarkedSpace::checkMarkCell): (JSC::MarkedSpace::markCell): Moved all code pertaining to managing chunks of collector memory into this class.
  • runtime/MemoryStatistics.cpp: (JSC::heapStatistics):
  • runtime/MemoryStatistics.h: Updated for MarkedSpace delegation.

2011-01-13 Geoffrey Garen <ggaren@apple.com>

Reviewed by Oliver Hunt.

Split out a MarkedSpace strategy object from Heap.
https://bugs.webkit.org/show_bug.cgi?id=52421

  • Shared/mac/WebMemorySampler.mac.mm: (WebKit::WebMemorySampler::sampleWebKit): Updated for class move.
17:40 Changeset [75854] by thakis@chromium.org
  • 5 edits in trunk/LayoutTests

2011-01-14 Nico Weber <thakis@chromium.org>

Unreviewed, test expectations.

Rebaseline gradient test due to updated skia. reed checked the test and
says the new look is right and is an intentional change.
https://bugs.webkit.org/show_bug.cgi?id=52105

  • platform/chromium-linux/svg/batik/paints/gradientLimit-expected.checksum:
  • platform/chromium-linux/svg/batik/paints/gradientLimit-expected.png:
  • platform/chromium-win/svg/batik/paints/gradientLimit-expected.checksum:
  • platform/chromium-win/svg/batik/paints/gradientLimit-expected.png:
17:31 Changeset [75853] by andersca@apple.com
  • 10 edits in trunk/WebKit2

2011-01-14 Anders Carlsson <andersca@apple.com>

Reviewed by Dan Bernstein.

Add a new PageClient::displayView function and expose it on WebPageProxy
https://bugs.webkit.org/show_bug.cgi?id=52500

  • UIProcess/API/mac/PageClientImpl.h:
  • UIProcess/API/mac/PageClientImpl.mm: (WebKit::PageClientImpl::displayView):
  • UIProcess/API/qt/qwkpage.cpp: (QWKPagePrivate::displayView):
  • UIProcess/API/qt/qwkpage_p.h:
  • UIProcess/PageClient.h:
  • UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::displayView):
  • UIProcess/WebPageProxy.h:
  • UIProcess/win/WebView.cpp: (WebKit::WebView::displayView):
  • UIProcess/win/WebView.h:
17:22 Changeset [75852] by oliver@apple.com
  • 5 edits
    3 adds in trunk

2011-01-14 Oliver Hunt <oliver@apple.com>

Reviewed by Gavin Barraclough.

[jsfunfuzz] parser doesn't enforce continue restrictions correctly.
https://bugs.webkit.org/show_bug.cgi?id=52493

Add a few tests for continue to cover the cases where continue
isn't syntactically valid.

  • fast/js/js-continue-break-restrictions-expected.txt: Added.
  • fast/js/js-continue-break-restrictions.html: Added.
  • fast/js/script-tests/js-continue-break-restrictions.js: Added.

2011-01-14 Oliver Hunt <oliver@apple.com>

Reviewed by Gavin Barraclough.

[jsfunfuzz] parser doesn't enforce continue restrictions correctly.
https://bugs.webkit.org/show_bug.cgi?id=52493

This patch reworks handling of break, continue and label statements
to correctly handle all the valid and invalid cases. Previously certain
errors would be missed by the parser in strict mode, but the bytecode
generator needed to handle those cases for non-strict code so nothing
failed, it simply became non-standard behaviour.

Now that we treat break and continue errors as early faults in non-strict
mode as well that safety net has been removed so the parser bugs result in
crashes at codegen time.

  • parser/JSParser.cpp: (JSC::JSParser::ScopeLabelInfo::ScopeLabelInfo): (JSC::JSParser::next): (JSC::JSParser::nextTokenIsColon): (JSC::JSParser::continueIsValid):

Continue is only valid in loops so we can't use breakIsValid()

(JSC::JSParser::pushLabel):

We now track whether the label is for a loop (and is therefore a
valid target for continue.

(JSC::JSParser::popLabel):
(JSC::JSParser::getLabel):

Replace hasLabel with getLabel so that we can validate the target
when parsing continue statements.

(JSC::JSParser::Scope::continueIsValid):
(JSC::JSParser::Scope::pushLabel):
(JSC::JSParser::Scope::getLabel):
(JSC::JSParser::JSParser):
(JSC::JSParser::parseBreakStatement):
(JSC::JSParser::parseContinueStatement):
(JSC::LabelInfo::LabelInfo):
(JSC::JSParser::parseExpressionOrLabelStatement):

Consecutive labels now get handled iteratively so that we can determine
whether they're valid targets for continue.

  • parser/Lexer.cpp: (JSC::Lexer::nextTokenIsColon):
  • parser/Lexer.h: (JSC::Lexer::setOffset):
17:20 Changeset [75851] by mitz@apple.com
  • 5 edits
    1 add in trunk

Source/WebCore: WebCore part of <rdar://problem/8441312> Crash in -[NSView _invalidateGStatesForTree]

Reviewed by Simon Fraser.

  • WebCore.exp.in: Export RenderWidget::suspendWidgetHierarchyUpdates() and

RenderWidget::resumeWidgetHierarchyUpdates().

  • manual-tests/plug-in-mutates-NSView-hierarchy-during-resize.html: Added.
  • rendering/RenderWidget.cpp:

(WebCore::RenderWidget::setWidgetGeometry): Removed the assertion that widget hierarchy updates
are disabled. When this assertion was added, this condition was a subset of the “calling out to
plug-in code is forbidden” condition, hence the assertion was valid. The WebKit part of this
change now suspends widget hierarchy updates even at times where plug-in code is expected to be
called, which invalidates the assertion.

WebKit/mac: WebKit/mac part of <rdar://problem/8441312> Crash in -[NSView _invalidateGStatesForTree]

Reviewed by Simon Fraser.

  • WebView/WebHTMLView.mm:

(-[WebHTMLView _invalidateGStatesForTree]): Override this NSView method, and bracket the call
to the superclass implementation with suspending WebCore Widget hierarchy updates. This ensures
that the NSView tree doesn’t change from under AppKit as it traverses it.

17:19 Changeset [75850] by commit-queue@webkit.org
  • 2 edits in trunk

2011-01-14 Ahmad Sharif <asharif.tools@gmail.com>

Reviewed by Darin Fisher.

When inlining is performed, the progress variable is
declared as uninitialized.
https://bugs.webkit.org/show_bug.cgi?id=51084

  • WebCore: Copied from WebCore.
  • WebCore/page/animation/KeyframeAnimation.cpp: (WebCore::KeyframeAnimation::animate): (WebCore::KeyframeAnimation::getAnimatedStyle):
17:18 Changeset [75849] by andersca@apple.com
  • 2 edits in trunk/WebKit2

2011-01-14 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

Implement the "should paint bounds instead of indiviual rects" algorithm from WebKit1
https://bugs.webkit.org/show_bug.cgi?id=52499

  • WebProcess/WebPage/DrawingAreaImpl.cpp: (WebKit::shouldPaintBoundsRect): Port code from -[WebView _mustDrawUnionedRect:singleRects:count:].

(WebKit::DrawingAreaImpl::display):
If shouldPaintBoundsRect returns true, clear the rects vector and append the bounds rect.

17:17 Changeset [75848] by tonyg@chromium.org
  • 4 edits in trunk

2011-01-14 Tony Gentilcore <tonyg@chromium.org>

Unreviewed build fix.

Fix Qt build after r75837
https://bugs.webkit.org/show_bug.cgi?id=52494

  • rendering/style/StyleRareInheritedData.cpp:
  • rendering/style/StyleRareNonInheritedData.cpp:
17:15 Changeset [75847] by ojan@chromium.org
  • 2 edits in trunk/Websites/bugs.webkit.org

2011-01-14 Ojan Vafai <ojan@chromium.org>

Reviewed by Adam Barth.

fix expanded lines in the code review tool
https://bugs.webkit.org/show_bug.cgi?id=52498

Fixes them to work now that we have wrapper divs.
Also fixes the long-standing bug that the line numbers
were incorrect sometimes.

  • code-review.js:
17:08 Changeset [75846] by tonyg@chromium.org
  • 4 edits in trunk

2011-01-14 Tony Gentilcore <tonyg@chromium.org>

Unreviewed build fix.

Fix Qt build after r75837
https://bugs.webkit.org/show_bug.cgi?id=52494

  • rendering/style/RenderStyle.cpp:

2011-01-14 Tony Gentilcore <tonyg@chromium.org>

Unreviewed build fix.

Fix Qt build after r75837
https://bugs.webkit.org/show_bug.cgi?id=52494

  • ewk/ewk_view.cpp:
17:02 Changeset [75845] by ojan@chromium.org
  • 3 edits in trunk/Websites/bugs.webkit.org

2011-01-14 Ojan Vafai <ojan@chromium.org>

Reviewed by Adam Barth.

put remove lines to the left of add lines in sidebyside view
https://bugs.webkit.org/show_bug.cgi?id=52458

Also, sneaking in a fix to make long lines with no whitespace wrap.

  • PrettyPatch/PrettyPatch.rb:
  • code-review.js:
17:00 Changeset [75844] by tonyg@chromium.org
  • 2 edits in trunk/Source/WebCore

2011-01-14 Tony Gentilcore <tonyg@chromium.org>

Unreviewed build fix.

Fix Qt build after r75837
https://bugs.webkit.org/show_bug.cgi?id=52494

  • editing/EditorCommand.cpp:
16:55 Changeset [75843] by tonyg@chromium.org
  • 3 edits in trunk/Source/WebCore

2011-01-14 Tony Gentilcore <tonyg@chromium.org>

Unreviewed build fix.

Fix Qt build after r75837
https://bugs.webkit.org/show_bug.cgi?id=52494

  • editing/ApplyStyleCommand.cpp:
16:44 Changeset [75842] by tonyg@chromium.org
  • 2 edits in trunk/Source/WebCore

2011-01-14 Tony Gentilcore <tonyg@chromium.org>

Unreviewed build fix.

Fix Qt build after r75837
https://bugs.webkit.org/show_bug.cgi?id=52494

  • css/CSSStyleSelector.cpp:
16:42 Changeset [75841] by mjs@apple.com
  • 3 edits in trunk/LayoutTests

2011-01-14 Maciej Stachowiak <mjs@apple.com>

Reviewed by Anders Carlsson.

Fix a few things in the WebKit2 skipped list
https://bugs.webkit.org/show_bug.cgi?id=52484

  • platform/mac-wk2/Skipped: Add/remove some tests to the WK2 skipped list.
  • editing/style/stylewithcss-without-selection.html: Fix this test to avoid referencing an undefined variable in the failure case.
16:35 Changeset [75840] by tonyg@chromium.org
  • 3 edits in trunk/Source/WebCore

2011-01-14 Tony Gentilcore <tonyg@chromium.org>

Unreviewed build fix.

Fix Qt build after r75837
https://bugs.webkit.org/show_bug.cgi?id=52494

  • rendering/InlineFlowBox.h:
  • rendering/RenderBoxModelObject.h:
16:29 Changeset [75839] by paroga@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

2011-01-14 Patrick Gansterer <paroga@webkit.org>

Reviewed by Adam Roben.

Use the Windows thread pool instead of an extra thread for FastMalloc scavenging
https://bugs.webkit.org/show_bug.cgi?id=45186

  • wtf/FastMalloc.cpp: (WTF::TCMalloc_PageHeap::scheduleScavenger): Added missing this pointer to CreateTimerQueueTimer().
16:28 Changeset [75838] by mihaip@chromium.org
  • 231 edits
    230 copies
    16 adds in trunk/LayoutTests

2011-01-14 Mihai Parparita <mihaip@chromium.org>

Update the svg/ pixel expectations for the chromium-mac port.

For the pixel tests that currently fail on Snow Leopard, this moves the
current pixel results to platform/chromium-mac-leopard and adds new
results to platform/chromium-mac.

List of files omitted due to length but composed entirely of moves from
platform/chromium-mac to platform/chromium-mac-leopard and newly added
files in platform/chromium-mac.

16:27 Changeset [75837] by tonyg@chromium.org
  • 56 edits in trunk

2011-01-14 Tony Gentilcore <tonyg@chromium.org>

Reviewed by David Levin.

Do some forward declaration in RenderStyle.h
https://bugs.webkit.org/show_bug.cgi?id=52453

No new tests because no new functionality.

  • css/CSSComputedStyleDeclaration.cpp:
  • css/CSSStyleSelector.cpp:
  • editing/Editor.cpp:
  • page/EventHandler.cpp:
  • rendering/EllipsisBox.cpp:
  • rendering/InlineBox.cpp:
  • rendering/InlineTextBox.cpp:
  • rendering/RenderBlock.cpp:
  • rendering/RenderBlock.h:
  • rendering/RenderBox.cpp:
  • rendering/RenderBox.h:
  • rendering/RenderEmbeddedObject.cpp:
  • rendering/RenderFieldset.cpp:
  • rendering/RenderFileUploadControl.cpp:
  • rendering/RenderFrameSet.cpp:
  • rendering/RenderHTMLCanvas.cpp:
  • rendering/RenderImageResource.cpp:
  • rendering/RenderInputSpeech.cpp:
  • rendering/RenderLayer.h:
  • rendering/RenderLineBoxList.cpp:
  • rendering/RenderListBox.cpp:
  • rendering/RenderMediaControlsChromium.cpp:
  • rendering/RenderObject.cpp:
  • rendering/RenderObject.h:
  • rendering/RenderObjectChildList.cpp:
  • rendering/RenderProgress.cpp:
  • rendering/RenderScrollbarPart.cpp:
  • rendering/RenderTable.cpp:
  • rendering/RenderTable.h:
  • rendering/RenderTableCell.cpp:
  • rendering/RenderTableRow.cpp:
  • rendering/RenderTableSection.cpp:
  • rendering/RenderTheme.cpp:
  • rendering/RenderThemeChromiumSkia.cpp:
  • rendering/RenderThemeChromiumWin.cpp:
  • rendering/RootInlineBox.cpp:
  • rendering/style/RenderStyle.cpp:
  • rendering/style/RenderStyle.h:
  • rendering/style/StyleRareInheritedData.cpp:
16:19 Changeset [75836] by jochen@chromium.org
  • 2 edits in trunk/WebKit/chromium

2011-01-14 Jochen Eisinger <jochen@chromium.org>

Unreviewed, roll chromium deps to r71432.

  • DEPS:
16:16 Changeset [75835] by rniwa@webkit.org
  • 7 edits in trunk/Source/WebCore

2011-01-14 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Eric Seidel.

Stop instantiating legacy editing positions in AccessibilityRenderObject.cpp, Element.cpp,
BreakBlockquoteCommand.cpp, CompositeEditCommand.cpp, and DeleteButtonController.cpp
https://bugs.webkit.org/show_bug.cgi?id=52481

Removed instantiation of legacy editing positions.
Calls to Position::Position are replaced by calls to Position's convenience functions.


Also fixed firstPositionInOrBeforeNode and lastPositionInOrAfterNode so that
they instantiate right positions for text nodes.

  • accessibility/AccessibilityRenderObject.cpp: (WebCore::AccessibilityRenderObject::setSelectedTextRange):
  • dom/Element.cpp: (WebCore::Element::updateFocusAppearance):
  • editing/BreakBlockquoteCommand.cpp: (WebCore::BreakBlockquoteCommand::doApply):
  • editing/CompositeEditCommand.cpp: (WebCore::CompositeEditCommand::inputText): (WebCore::CompositeEditCommand::rebalanceWhitespaceAt): (WebCore::CompositeEditCommand::moveParagraphContentsToNewBlockIfNecessary): (WebCore::CompositeEditCommand::moveParagraphs): (WebCore::CompositeEditCommand::breakOutOfEmptyListItem): (WebCore::CompositeEditCommand::breakOutOfEmptyMailBlockquotedParagraph): (WebCore::CompositeEditCommand::splitTreeToNode):
  • editing/DeleteButtonController.cpp: (WebCore::enclosingDeletableElement):
  • editing/htmlediting.h: (WebCore::firstPositionInOrBeforeNode): Instantiates firstPositionInNode instead of positionBeforeNode for a text node. (WebCore::lastPositionInOrAfterNode): Ditto.
16:10 Changeset [75834] by mrowe@apple.com
  • 1 copy in tags/Safari-534.15.8

New tag.

15:46 Changeset [75833] by commit-queue@webkit.org
  • 5 edits
    3 adds in trunk

2011-01-14 Helder Correia <helder@sencha.com>

Reviewed by Simon Fraser.

Shadow is not drawn when filling a path with a gradient
https://bugs.webkit.org/show_bug.cgi?id=51982

This happens in CG and is related to bug 51869, this time to be fixed
in GraphicsContext::fillPath(const Path& path). We need to draw the
gradient clipped to the path on a CGLayer first, and then draw the
layer on the GraphicsContext.

  • fast/canvas/canvas-fillPath-gradient-shadow-expected.txt: Added.
  • fast/canvas/canvas-fillPath-gradient-shadow.html: Added.
  • fast/canvas/script-tests/canvas-fillPath-gradient-shadow.js: Added.
  • platform/chromium/test_expectations.txt: Skipping new test since it fails.
  • platform/qt/Skipped: Ditto.

2011-01-14 Helder Correia <helder@sencha.com>

Reviewed by Simon Fraser.

Shadow is not drawn when filling a path with a gradient
https://bugs.webkit.org/show_bug.cgi?id=51982

This happens in CG and is related to bug 51869, this time to be fixed
in GraphicsContext::fillPath(const Path& path). We need to draw the
gradient clipped to the path on a CGLayer first, and then draw the
layer on the GraphicsContext.

Test: fast/canvas/canvas-fillPath-gradient-shadow.html

  • platform/graphics/cg/GraphicsContextCG.cpp: (WebCore::GraphicsContext::fillPath):
15:45 Changeset [75832] by simon.fraser@apple.com
  • 5 edits in trunk/Source/WebCore

2011-01-14 Simon Fraser <simon.fraser@apple.com>

Reviewed by Adam Roben.

Layer syncing should go through the compositor
https://bugs.webkit.org/show_bug.cgi?id=52486

Rather than have FrameView go directly to GraphicsLayer to
sync pending changes, route the call through RenderLayerCompositor.

Add a FIXME about an existing issue with flushing and subframes.

No behavior change, so no tests.

  • page/FrameView.cpp: (WebCore::FrameView::syncCompositingStateForThisFrame):
  • rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::notifySyncRequired):
  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::scheduleLayerFlush): (WebCore::RenderLayerCompositor::flushPendingLayerChanges):
  • rendering/RenderLayerCompositor.h: (WebCore::RenderLayerCompositor::notifySyncRequired):
15:37 Changeset [75831] by ojan@chromium.org
  • 2 edits in trunk/Websites/bugs.webkit.org

2011-01-14 Ojan Vafai <ojan@chromium.org>

Fix bad merge in r75816.

  • code-review.js:
15:36 Changeset [75830] by paroga@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

2011-01-14 Patrick Gansterer <paroga@webkit.org>

Reviewed by Adam Roben.

Use the Windows thread pool instead of an extra thread for FastMalloc scavenging
https://bugs.webkit.org/show_bug.cgi?id=45186

r75819 accidentally changed the initial state of the scavenge timer.

  • wtf/FastMalloc.cpp: (WTF::TCMalloc_PageHeap::initializeScavenger): Changed initial state of m_scavengingSuspended to true.
15:35 Changeset [75829] by bdakin@apple.com
  • 3 edits in trunk/WebKit/mac

Fix for <rdar://problem/7793902> Artifacts when scrolling
page in Safari while page is loading

Reviewed by Mark Rowe.

Call setDrawsBackground:YES on the scrollView in
dispatchDidFirstLayout() rather than waiting until
frameLoadCompleted(). Also suspend scrolling elasticity
between provisionalLoadStarted() and dispatchDidFirstLayout()

  • WebCoreSupport/WebFrameLoaderClient.h:
  • WebCoreSupport/WebFrameLoaderClient.mm:

(WebFrameLoaderClient::WebFrameLoaderClient):
(WebFrameLoaderClient::dispatchDidFirstLayout):
(WebFrameLoaderClient::provisionalLoadStarted):

15:04 Changeset [75828] by mihaip@chromium.org
  • 248 edits
    247 copies
    28 adds in trunk/LayoutTests

2011-01-14 Mihai Parparita <mihaip@chromium.org>

Update the pixel expectations for the chromium-mac port.

For the pixel tests that currently fail on Snow Leopard, this moves the
current pixel results to platform/chromium-mac-leopard and adds new
results to platform/chromium-mac.

List of files omitted due to length but composed entirely of moves from
platform/chromium-mac to platform/chromium-mac-leopard and newly added
files in platform/chromium-mac.

14:57 Changeset [75827] by paroga@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

2011-01-14 Patrick Gansterer <paroga@webkit.org>

Unreviewed Windows Release build fix.

  • wtf/FastMalloc.cpp: (WTF::TCMalloc_PageHeap::scavengerTimerFired):
14:57 Changeset [75826] by bweinstein@apple.com
  • 6 edits in trunk

ShouldLoadResourceForFrame should use strings, not URLs.
https://bugs.webkit.org/show_bug.cgi?id=52476

Reviewed by Geoff Garen.

Tools:

  • WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp:

(WTR::InjectedBundlePage::shouldLoadResourceForFrame): Update to use a WKStringRef.

  • WebKitTestRunner/InjectedBundle/InjectedBundlePage.h:

WebKit2:

  • WebProcess/InjectedBundle/API/c/WKBundlePage.h:
  • WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.cpp:

(WebKit::InjectedBundlePageLoaderClient::shouldLoadResourceForFrame): Pass resourceURL as a string instead of

a URL, and rename from subresourceURL to resourceURL.

14:49 Changeset [75825] by paroga@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

2011-01-14 Patrick Gansterer <paroga@webkit.org>

Unreviewed Windows Release build fix.

  • wtf/FastMalloc.cpp: (WTF::TCMalloc_PageHeap::scavengerTimerFired):
14:45 Changeset [75824] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

2011-01-14 Steve Lacey <sjl@chromium.org>

Reviewed by Simon Fraser.

Update media/video-zoom-controls.html to set the video source before initializing the test.
https://bugs.webkit.org/show_bug.cgi?id=52461

  • media/video-zoom-controls.html:
14:40 Changeset [75823] by inferno@chromium.org
  • 3 edits
    2 adds in trunk

2011-01-14 Abhishek Arya <inferno@chromium.org>

Reviewed by David Hyatt.

Fix parent block calculation when trying to find top most node
containing "this" float.
https://bugs.webkit.org/show_bug.cgi?id=51711

Replace use of containingBlock and traverse the parents directly
to check for float existence. containingBlock can skip parents and
jump to the RenderView directly which will cause floats to not get
cleared from intermediate parents.

Test: fast/block/float/floats-not-cleared-crash.html

  • rendering/RenderBox.cpp: (WebCore::RenderBox::removeFloatingOrPositionedChildFromBlockLists):

2011-01-14 Abhishek Arya <inferno@chromium.org>

Reviewed by Dave Hyatt.

Tests that we do not crash due to uncleared floats in parent nodes.
https://bugs.webkit.org/show_bug.cgi?id=51711

  • fast/block/float/floats-not-cleared-crash-expected.txt: Added.
  • fast/block/float/floats-not-cleared-crash.html: Added.
14:39 Changeset [75822] by andersca@apple.com
  • 6 edits in trunk/WebKit2

Fix for <rdar://problem/8711576> The web process should be able to read resources from the UI process.

Reviewed by Sam Weinig.

Based on a patch by Ivan Krstić. This also makes the NSURLCache path a sandbox parameter.

  • Shared/WebProcessCreationParameters.cpp:

(WebKit::WebProcessCreationParameters::encode):
(WebKit::WebProcessCreationParameters::decode):
Encode and decode the uiProcessBundleResourcePath member variable.

  • Shared/WebProcessCreationParameters.h:

Add uiProcessBundleResourcePath member variable.

  • UIProcess/mac/WebContextMac.mm:

(WebKit::WebContext::platformInitializeWebProcess):
Set the uiProcessBundleResourcePath member variable.

  • WebProcess/com.apple.WebProcess.sb:

Allow read-only access to UI_PROCESS_BUNDLE_RESOURCE_DIR and all subdirectories.

  • WebProcess/mac/WebProcessMac.mm:

(WebKit::initializeSandbox):
When initializing the sandbox, pass UI_PROCESS_BUNDLE_RESOURCE_DIR.

(WebKit::WebProcess::platformInitializeWebProcess):
Remove the code that would consume a sandbox extension for the url cache path.

14:32 Changeset [75821] by jberlin@webkit.org
  • 2 edits in trunk/LayoutTests

REGRESSION (r75660): compositing/reflections/load-video-in-reflection.html crashes on Windows 7
https://bugs.webkit.org/show_bug.cgi?id=52397

Unskipping tests that no longer fail on Windows 7 Release. Unreviewed.

  • platform/win/Skipped:

Un-skip compositing/reflections/load-video-in-reflection.html, which appears to be fixed by
http://trac.webkit.org/changeset/75807

14:27 Changeset [75820] by ericu@chromium.org
  • 1 edit in trunk/Tools/Scripts/webkitpy/common/config/committers.py

Add ericu@chromium.org [me] to the committers file.

14:21 Changeset [75819] by paroga@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

2011-01-14 Patrick Gansterer <paroga@webkit.org>

Reviewed by Adam Roben.

Use the Windows thread pool instead of an extra thread for FastMalloc scavenging
https://bugs.webkit.org/show_bug.cgi?id=45186

Use CreateTimerQueueTimer() to start periodicScavenge() and stop it with DeleteTimerQueueTimer().

  • wtf/FastMalloc.cpp: (WTF::TCMalloc_PageHeap::initializeScavenger): (WTF::TCMalloc_PageHeap::isScavengerSuspended): (WTF::TCMalloc_PageHeap::scheduleScavenger): (WTF::TCMalloc_PageHeap::rescheduleScavenger): (WTF::TCMalloc_PageHeap::suspendScavenger): (WTF::scavengerTimerFired): (WTF::TCMalloc_PageHeap::periodicScavenge): (WTF::TCMalloc_PageHeap::signalScavenger):
14:15 Changeset [75818] by tonyg@chromium.org
  • 2 edits in trunk/Tools

2011-01-14 Tony Gentilcore <tonyg@chromium.org>

Reviewed by David Levin.

Add Tony Gentilcore as a reviewer
https://bugs.webkit.org/show_bug.cgi?id=52480

  • Scripts/webkitpy/common/config/committers.py:
14:13 Changeset [75817] by ojan@chromium.org
  • 2 edits in trunk/Websites/bugs.webkit.org

2011-01-14 Ojan Vafai <ojan@chromium.org>

Reviewed by Adam Barth.

improve line selection with fast drags
https://bugs.webkit.org/show_bug.cgi?id=52477

  • code-review.js:
14:08 WebKit Team edited by tonyg@chromium.org
(diff)
14:00 Changeset [75816] by ojan@chromium.org
  • 2 edits in trunk/Websites/bugs.webkit.org

2011-01-14 Ojan Vafai <ojan@chromium.org>

Reviewed by Adam Barth.

tweak line selection in code review tool
https://bugs.webkit.org/show_bug.cgi?id=52462

-Improve handling of data-comment-base-line code to deal with
lines that have multiple values.
-Make it so that if you click on the line immediately above
a line that has a comment it will add a new comment instead of
adding lines to the following comment. If the last selected line
overlaps existing comment lines though they will still get
added to the existing comment.

  • code-review.js:
13:59 Changeset [75815] by commit-queue@webkit.org
  • 10 edits
    2 moves
    1 delete in trunk

2011-01-14 Adam Klein <adamk@chromium.org>

Reviewed by Darin Fisher.

[Chromium] Replace BackForwardListClient with BackForwardControllerClient/BackForwardList
https://bugs.webkit.org/show_bug.cgi?id=42237

Remove Chromium-specific implementation from WebCore.

No tests added; this refactor should be covered by existing history-exercising tests.

  • WebCore.gyp/WebCore.gyp:
  • WebCore.gypi:
  • history/BackForwardListChromium.cpp: Removed.
  • history/BackForwardListImpl.h:

2011-01-14 Adam Klein <adamk@chromium.org>

Reviewed by Darin Fisher.

[Chromium] Replace BackForwardListClient with BackForwardControllerClient/BackForwardList
https://bugs.webkit.org/show_bug.cgi?id=42237

Replace Chromium-specific BackForwardListClient in WebCore with a Chromium impl
of the BackForwardList interface.

  • WebKit.gyp:
  • src/BackForwardListChromium.cpp: Copied from WebKit/chromium/src/BackForwardListClientImpl.cpp. (WebKit::BackForwardListChromium::create): (WebKit::BackForwardListChromium::BackForwardListChromium): (WebKit::BackForwardListChromium::~BackForwardListChromium): (WebKit::BackForwardListChromium::addItem): (WebKit::BackForwardListChromium::goToItem): (WebKit::BackForwardListChromium::itemAtIndex): (WebKit::BackForwardListChromium::backListCount): (WebKit::BackForwardListChromium::forwardListCount): (WebKit::BackForwardListChromium::isActive): (WebKit::BackForwardListChromium::close):
  • src/BackForwardListChromium.h: Copied from WebKit/chromium/src/BackForwardListClientImpl.h.
  • src/BackForwardListClientImpl.cpp: Removed.
  • src/BackForwardListClientImpl.h: Removed.
  • src/FrameLoaderClientImpl.cpp:
  • src/WebFrameImpl.cpp: (WebKit::WebFrameImpl::loadHistoryItem): (WebKit::WebFrameImpl::currentHistoryItem):
  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::WebViewImpl):
  • src/WebViewImpl.h:
13:51 Changeset [75814] by andersca@apple.com
  • 3 edits in trunk/WebKit2

Delay initializing the sandbox until we get the InitializeWebProcess message.

Reviewed by Sam Weinig.

  • WebProcess/mac/WebProcessMac.mm:

(WebKit::initializeSandbox):
(WebKit::WebProcess::platformInitializeWebProcess):

  • WebProcess/mac/WebProcessMainMac.mm:

(WebKit::WebProcessMain):

13:30 Changeset [75813] by levin@chromium.org
  • 3 edits in trunk/Tools

2011-01-14 David Levin <levin@chromium.org>

Reviewed by Eric Seidel.

check-webkit-style: _FunctionState should keep track of where the function name starts.
https://bugs.webkit.org/show_bug.cgi?id=52454

  • Scripts/webkitpy/style/checkers/cpp.py: Added function_name_start_position to _FunctionState and logic for setting it correctly.
  • Scripts/webkitpy/style/checkers/cpp_unittest.py: Added tests.
12:43 Changeset [75812] by commit-queue@webkit.org
  • 2 edits in trunk/WebKit/qt

2011-01-14 Konstantin Tokarev <annulen@yandex.ru>

Reviewed by Laszlo Gombos.

[Qt] Fixed compilation when QT_NO_SHORTCUT is defined
https://bugs.webkit.org/show_bug.cgi?id=52460

This patch fixes compilation when QT_NO_SHORTCUT is defined in Qt
configuration.

  • WebCoreSupport/EditorClientQt.cpp: (WebCore::EditorClientQt::handleKeyboardEvent): Added QT_NO_SHORTCUT guards
12:39 Changeset [75811] by satish@chromium.org
  • 7 edits
    2 adds in trunk

2011-01-12 Satish Sampath <satish@chromium.org>

Reviewed by Dimitri Glazkov.

Fix a crash when accessing speech input from script.
https://bugs.webkit.org/show_bug.cgi?id=52325

Added a layout test to check enabling/disabling speech input from script.

  • fast/speech/speech-input-scripting-expected.txt: Added.
  • fast/speech/speech-input-scripting.html: Added.

2011-01-12 Satish Sampath <satish@chromium.org>

Reviewed by Dimitri Glazkov.

Fix a crash when accessing speech input from script.
https://bugs.webkit.org/show_bug.cgi?id=52325

Test: fast/speech/speech-input-scripting.html

  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::parseMappedAttribute): Recreate renderer when speech input is enabled/disabled.
  • rendering/RenderTextControlSingleLine.cpp: Remove unused code.
  • rendering/RenderTextControlSingleLine.h:
  • rendering/TextControlInnerElements.cpp: Take self references before firing events and check for renderer validity after. (WebCore::InputFieldSpeechButtonElement::defaultEventHandler): (WebCore::InputFieldSpeechButtonElement::setRecognitionResult): (WebCore::InputFieldSpeechButtonElement::detach):
12:35 Changeset [75810] by inferno@chromium.org
  • 4 edits
    2 adds in trunk

2011-01-14 Abhishek Arya <inferno@chromium.org>

Reviewed by David Hyatt.

Prevent merging of anonymous blocks if one of them is already getting
destroyed.
https://bugs.webkit.org/show_bug.cgi?id=52402

Test: fast/block/merge-anonymous-block-remove-child-crash2.html

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::RenderBlock): initialize m_beingDestroyed to false. (WebCore::RenderBlock::destroy): set m_beingDestroyed to true. (WebCore::canMergeContiguousAnonymousBlocks): do not merge if any or prev or next is being destroyed. (WebCore::RenderBlock::removeChild): remove the hack previously done for preventing oldChild merging with nextBlock's next sibling.
  • rendering/RenderBlock.h: (WebCore::RenderBlock::beingDestroyed): public function for m_beingDestroyed.

2011-01-14 Abhishek Arya <inferno@chromium.org>

Reviewed by David Hyatt.

Tests that we do not crash when trying to merge anonymous blocks, one of which
is already getting destroyed.
https://bugs.webkit.org/show_bug.cgi?id=52402

  • fast/block/merge-anonymous-block-remove-child-crash2-expected.txt: Added.
  • fast/block/merge-anonymous-block-remove-child-crash2.html: Added.
12:30 Changeset [75809] by ojan@chromium.org
  • 2 edits in trunk/Websites/bugs.webkit.org

2011-01-14 Ojan Vafai <ojan@chromium.org>

Reviewed by Adam Roben.

fix bugs going back and forth between unified and sidebyside
https://bugs.webkit.org/show_bug.cgi?id=52470

Now that we're appending DOM nodes directly, we only want to
get the first ".text" element in the subtree. In the cases
where there are multiple, their contents are the same.

Also, when making side-by-side expansion lines, we need to
clone the contents so we have a second copy to append to the DOM.

  • code-review.js:
12:27 Changeset [75808] by pfeldman@chromium.org
  • 2 edits in trunk/Source/WebCore

2011-01-14 Pavel Feldman <pfeldman@chromium.org>

Not reviewed. Follow up to r75791: fix missing dispatch.
https://bugs.webkit.org/show_bug.cgi?id=52442

  • inspector/front-end/DebuggerModel.js: (WebInspector.DebuggerDispatcher.prototype.breakpointResolved):
12:01 Changeset [75807] by simon.fraser@apple.com
  • 2 edits in trunk/Source/WebCore

2011-01-14 Simon Fraser <simon.fraser@apple.com>

Reviewed by Adam Roben.

Issues with contentsScale in GraphicsLayerCA
https://bugs.webkit.org/show_bug.cgi?id=52463

Fix various issues with the recent contentsScale changes:

  • platform/graphics/ca/GraphicsLayerCA.cpp: (WebCore::GraphicsLayerCA::GraphicsLayerCA): Initialize m_contentsScale to 1. (WebCore::GraphicsLayerCA::updateContentsScale): Only do a setNeedsDisplay() if the layer draws content, otherwise we'll create backing store for empty layers. (WebCore::GraphicsLayerCA::swapFromOrToTiledLayer): Be sure to set contentsScale on the new (tiled or untiled) layer. (WebCore::GraphicsLayerCA::cloneLayer): Clones need contentsScale too.
11:50 Changeset [75806] by laszlo.1.gombos@nokia.com
  • 4 edits in trunk

2011-01-14 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

Align import/export directives
https://bugs.webkit.org/show_bug.cgi?id=52208

  • API/JSBase.h: Align import/export directives with WebKit2/Shared/API/c/WKBase.h

2011-01-14 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

Align import/export directives
https://bugs.webkit.org/show_bug.cgi?id=52208

  • Shared/API/c/WKBase.h: Align import/export directives with JavaScriptCore/API/JSBase.h.
11:16 Changeset [75805] by tkent@chromium.org
  • 4 edits in trunk/WebKit/chromium

2011-01-14 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Fisher.

[Chromium] WebKit API: Add a function for form interactive validation setting
https://bugs.webkit.org/show_bug.cgi?id=52430

  • public/WebSettings.h: Add a pure setInteractiveFormValidationEnabled().
  • src/WebSettingsImpl.cpp: (WebKit::WebSettingsImpl::setInteractiveFormValidationEnabled): Implemented.
  • src/WebSettingsImpl.h: Add a declaration.
10:54 Changeset [75804] by simon.fraser@apple.com
  • 2 edits in trunk/Source/WebCore

2011-01-14 Simon Fraser <simon.fraser@apple.com>

Let Xcode have it's way with the project file.

  • WebCore.xcodeproj/project.pbxproj:
10:53 Changeset [75803] by simon.fraser@apple.com
  • 3 edits in trunk/Source/WebCore

2011-01-14 Simon Fraser <simon.fraser@apple.com>

Reviewed by Anders Carlsson.

Refactor some FrameView::syncCompositingState code
https://bugs.webkit.org/show_bug.cgi?id=52459

Refactor some code in FrameView related to synchronizing
compositing layer state.

  • page/FrameView.cpp: (WebCore::FrameView::syncCompositingStateForThisFrame): (WebCore::FrameView::syncCompositingStateRecursive): (WebCore::FrameView::paintContents):
  • page/FrameView.h:
10:38 Changeset [75802] by tony@chromium.org
  • 12 edits in trunk/LayoutTests

2011-01-14 Tony Chang <tony@chromium.org>

Unreviewed, updating baselines.

[chromium] rebaseline vertical tests on linux
https://bugs.webkit.org/show_bug.cgi?id=52457

After updating fonts.conf, these tests are now passing.

  • platform/chromium-linux/fast/blockflow/japanese-lr-text-expected.checksum:
  • platform/chromium-linux/fast/blockflow/japanese-lr-text-expected.png:
  • platform/chromium-linux/fast/blockflow/japanese-rl-text-expected.checksum:
  • platform/chromium-linux/fast/blockflow/japanese-rl-text-expected.png:
  • platform/chromium-linux/fast/repaint/japanese-rl-selection-clear-expected.checksum:
  • platform/chromium-linux/fast/repaint/japanese-rl-selection-clear-expected.png:
  • platform/chromium-linux/fast/repaint/japanese-rl-selection-repaint-expected.checksum:
  • platform/chromium-linux/fast/repaint/japanese-rl-selection-repaint-expected.png:
  • platform/chromium-linux/svg/batik/text/verticalText-expected.checksum:
  • platform/chromium-linux/svg/batik/text/verticalText-expected.png:
  • platform/chromium/test_expectations.txt:
10:30 Changeset [75801] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-01-14 Viatcheslav Ostapenko <ostapenko.viatcheslav@nokia.com>

Reviewed by Eric Seidel.

[Qt] Fix build problem introduced by http://trac.webkit.org/changeset/75713
https://bugs.webkit.org/show_bug.cgi?id=30179

  • plugins/symbian/PluginViewSymbian.cpp:
10:09 Changeset [75800] by commit-queue@webkit.org
  • 7 edits in trunk

2011-01-14 Vincent Scheib <scheib@chromium.org>

Reviewed by David Levin.

Add support to DumpRenderTree to use the GPU rather than software renderer
https://bugs.webkit.org/show_bug.cgi?id=52333

  • DumpRenderTree/chromium/DumpRenderTree.cpp: (main):
  • Scripts/webkitpy/layout_tests/port/chromium.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:

2011-01-14 Vincent Scheib <scheib@chromium.org>

Reviewed by David Levin.

Add support to DumpRenderTree to use the GPU rather than software renderer
https://bugs.webkit.org/show_bug.cgi?id=52333

  • DEPS: Moved forward to take required support from webkit_support.h
  • WebKit.gyp: DumpRenderTree depends on Angle on windows
09:35 Changeset [75799] by tony@chromium.org
  • 4 edits in trunk

2011-01-14 Tony Chang <tony@chromium.org>

Reviewed by Kent Tamura.

[chromium] update fonts.conf to fix 6 linux tests in drt
https://bugs.webkit.org/show_bug.cgi?id=52411

  • platform/chromium/test_expectations.txt:

2011-01-14 Tony Chang <tony@chromium.org>

Reviewed by Kent Tamura.

[chromium] update fonts.conf to fix 6 linux tests in drt
https://bugs.webkit.org/show_bug.cgi?id=52411

  • DumpRenderTree/chromium/fonts.conf: Update to match the copy in

chromium's src/webkit/tools/test_shell/resources/fonts.conf

09:27 Changeset [75798] by carlosgc@webkit.org
  • 16 edits
    1 add in trunk

2011-01-14 Philippe Normand <pnormand@igalia.com>

Reviewed by Martin Robinson.

[GTK] Add volume slider to media player
https://bugs.webkit.org/show_bug.cgi?id=51532

  • platform/gtk/Skipped:
  • platform/gtk/media/audio-controls-rendering-expected.txt:
  • platform/gtk/media/controls-after-reload-expected.txt:
  • platform/gtk/media/controls-strict-expected.txt:
  • platform/gtk/media/controls-styling-expected.txt:
  • platform/gtk/media/video-controls-rendering-expected.txt:
  • platform/gtk/media/video-display-toggle-expected.txt:
  • platform/gtk/media/video-empty-source-expected.txt:
  • platform/gtk/media/video-no-audio-expected.txt:
  • platform/gtk/media/video-volume-slider-expected.txt: Added.

2011-01-14 Carlos Garcia Campos <cgarcia@igalia.com>

Reviewed by Martin Robinson.

[GTK] Add volume slider to media player
https://bugs.webkit.org/show_bug.cgi?id=51532

  • css/mediaControlsGtk.css: (audio::-webkit-media-controls-panel, video::-webkit-media-controls-panel): (audio::-webkit-media-controls-volume-slider-container, video::-webkit-media-controls-volume-slider-container): (audio::-webkit-media-controls-volume-slider, video::-webkit-media-controls-volume-slider):
  • platform/gtk/RenderThemeGtk.cpp: (WebCore::RenderThemeGtk::adjustMediaSliderThumbSize): (WebCore::RenderThemeGtk::paintMediaVolumeSliderContainer): (WebCore::RenderThemeGtk::paintMediaVolumeSliderTrack): (WebCore::RenderThemeGtk::paintMediaVolumeSliderThumb):
  • platform/gtk/RenderThemeGtk.h:
  • platform/gtk/RenderThemeGtk2.cpp: (WebCore::RenderThemeGtk::paintSliderTrack): (WebCore::RenderThemeGtk::paintSliderThumb): (WebCore::RenderThemeGtk::adjustSliderThumbSize):
  • platform/gtk/RenderThemeGtk3.cpp: (WebCore::RenderThemeGtk::paintSliderTrack): (WebCore::RenderThemeGtk::paintSliderThumb): (WebCore::RenderThemeGtk::adjustSliderThumbSize):
09:03 Changeset [75797] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-01-14 Joone Hur <joone.hur@collabora.co.uk>

Reviewed by David Levin.

[GTK] Convert use of raw pointers to GOwnPtr in FileSystemGtk.cpp
https://bugs.webkit.org/show_bug.cgi?id=52434

No new tests. This code will be tested when an implementation
of beginDragWithFiles is complete.

  • platform/gtk/FileSystemGtk.cpp: (WebCore::filenameToString): Use GOwnPtr instead of gchar pointer. (WebCore::fileSystemRepresentation): Ditto. (WebCore::filenameForDisplay): Ditto. (WebCore::pathGetFileName): Ditto.
08:02 Changeset [75796] by msaboff@apple.com
  • 5 edits in trunk

2011-01-14 Michael Saboff <msaboff@apple.com>

Reviewed by Oliver Hunt.

Incorrect backtracking for nested alternatives
https://bugs.webkit.org/show_bug.cgi?id=52387

Added tests to verfiy backtracking of alternatives following
parenthesis.

  • fast/regex/parentheses-expected.txt:
  • fast/regex/script-tests/parentheses.js:

2011-01-14 Michael Saboff <msaboff@apple.com>

Reviewed by Oliver Hunt.

Incorrect backtracking for nested alternatives
https://bugs.webkit.org/show_bug.cgi?id=52387

In the process of propigating a datalabel it wasn't getting connected
to a destination when the destination was an indirect jump. Added
code to recognize a direct backtrack destination that was an indirect
jump and added mechanism to associate DataLabelPtrs with indirect
jump entries.
Removed dead method
BacktrackDestination::linkDataLabelToHereIfExists()

  • yarr/YarrJIT.cpp: (JSC::Yarr::YarrGenerator::IndirectJumpEntry::IndirectJumpEntry): (JSC::Yarr::YarrGenerator::IndirectJumpEntry::addDataLabel): (JSC::Yarr::YarrGenerator::GenerationState::addIndirectJumpEntry): (JSC::Yarr::YarrGenerator::GenerationState::emitIndirectJumpTable): Changes to link indirect jumps with DataLabelPtr's. (JSC::Yarr::YarrGenerator::BacktrackDestination::clearSubDataLabelPtr): (JSC::Yarr::YarrGenerator::TermGenerationState::linkDataLabelToBacktrackIfExists): Updated to handle immediate linking of indirect jumps to DataLabelPtr. (JSC::Yarr::YarrGenerator::generateParenthesesDisjunction): Changed to reflect updated linkDataLabelToBacktrackIfExists().
07:58 Changeset [75795] by podivilov@chromium.org
  • 6 edits in trunk/Source/WebCore

2011-01-14 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: breakpoint text snippet in breakpoints sidebar pane disappears after reload.
https://bugs.webkit.org/show_bug.cgi?id=52215

  • inspector/front-end/Breakpoint.js: (WebInspector.Breakpoint): (WebInspector.Breakpoint.prototype.populateLabelElement):
  • inspector/front-end/Script.js: (WebInspector.Script.prototype.get linesCount): (WebInspector.Script.prototype.sourceLine): (WebInspector.Script.prototype.sourceLine.didRequestSource): (WebInspector.Script.prototype.set source): (WebInspector.Script.prototype.requestSource.didGetScriptSource): (WebInspector.Script.prototype.requestSource):
  • inspector/front-end/ScriptView.js: (WebInspector.ScriptView.prototype.setupSourceFrameIfNeeded.didRequestSource): (WebInspector.ScriptView.prototype.setupSourceFrameIfNeeded):
  • inspector/front-end/SourceFrame.js: (WebInspector.SourceFrame.prototype._addBreakpoint):
  • inspector/front-end/utilities.js: (String.prototype.findAll):
07:49 Changeset [75794] by podivilov@chromium.org
  • 13 edits in trunk/Source

2011-01-14 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: provide script column offset to frontend.
https://bugs.webkit.org/show_bug.cgi?id=52377

  • parser/SourceCode.h: (JSC::SourceCode::SourceCode): (JSC::SourceCode::firstColumn):

2011-01-14 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: provide script column offset to frontend.
https://bugs.webkit.org/show_bug.cgi?id=52377

  • bindings/js/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::dispatchDidParseSource):
  • bindings/js/ScriptSourceCode.h: (WebCore::ScriptSourceCode::ScriptSourceCode):
  • bindings/v8/DebuggerScript.js: ():
  • bindings/v8/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::dispatchDidParseSource):
  • inspector/Inspector.idl:
  • inspector/InspectorDebuggerAgent.cpp: (WebCore::InspectorDebuggerAgent::didParseSource):
  • inspector/InspectorDebuggerAgent.h:
  • inspector/ScriptDebugListener.h:
  • inspector/front-end/DebuggerModel.js: (WebInspector.DebuggerModel.prototype.parsedScriptSource):
  • inspector/front-end/Script.js: (WebInspector.Script):
07:28 Changeset [75793] by loislo@chromium.org
  • 2 edits in trunk/Source/WebCore

2011-01-14 Ilya Tikhonovsky <loislo@chromium.org>

Unreviewed one line fix for console-xhr-logging test.

The problem was introduced at r75788.

  • inspector/InspectorBrowserDebuggerAgent.cpp: (WebCore::InspectorBrowserDebuggerAgent::willSendXMLHttpRequest):
06:57 Changeset [75792] by yurys@chromium.org
  • 22 edits
    2 adds in trunk/Source/WebCore

2011-01-13 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: extract console related functionality into InspectorConsoleAgent
https://bugs.webkit.org/show_bug.cgi?id=52282

  • CMakeLists.txt:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • inspector/CodeGeneratorInspector.pm:
  • inspector/ConsoleMessage.cpp: (WebCore::ConsoleMessage::~ConsoleMessage):
  • inspector/ConsoleMessage.h:
  • inspector/Inspector.idl:
  • inspector/InspectorConsoleAgent.cpp: Added. (WebCore::InspectorConsoleAgent::InspectorConsoleAgent): (WebCore::InspectorConsoleAgent::~InspectorConsoleAgent): (WebCore::InspectorConsoleAgent::setConsoleMessagesEnabled): (WebCore::InspectorConsoleAgent::clearConsoleMessages): (WebCore::InspectorConsoleAgent::reset): (WebCore::InspectorConsoleAgent::setFrontend): (WebCore::InspectorConsoleAgent::addMessageToConsole): (WebCore::InspectorConsoleAgent::startTiming): (WebCore::InspectorConsoleAgent::stopTiming): (WebCore::InspectorConsoleAgent::count): (WebCore::InspectorConsoleAgent::resourceRetrievedByXMLHttpRequest): (WebCore::InspectorConsoleAgent::didReceiveResponse): (WebCore::InspectorConsoleAgent::didFailLoading): (WebCore::InspectorConsoleAgent::addConsoleMessage):
  • inspector/InspectorConsoleAgent.h: Added.
  • inspector/InspectorController.cpp: (WebCore::InspectorController::InspectorController): (WebCore::InspectorController::clearConsoleMessages): (WebCore::InspectorController::connectFrontend): (WebCore::InspectorController::disconnectFrontend): (WebCore::InspectorController::didCommitLoad):
  • inspector/InspectorController.h: (WebCore::InspectorController::consoleAgent):
  • inspector/InspectorInstrumentation.cpp: (WebCore::InspectorInstrumentation::didReceiveResourceResponseImpl): (WebCore::InspectorInstrumentation::didFailLoadingImpl): (WebCore::InspectorInstrumentation::resourceRetrievedByXMLHttpRequestImpl): (WebCore::InspectorInstrumentation::addMessageToConsole): (WebCore::InspectorInstrumentation::count): (WebCore::InspectorInstrumentation::startTiming): (WebCore::InspectorInstrumentation::stopTiming): (WebCore::InspectorInstrumentation::addStartProfilingMessageToConsole):
  • inspector/InspectorInstrumentation.h:
  • inspector/InspectorProfilerAgent.cpp: (WebCore::InspectorProfilerAgent::addProfileFinishedMessageToConsole): (WebCore::InspectorProfilerAgent::addStartProfilingMessageToConsole):
  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleView): (WebInspector.ConsoleView.prototype.addConsoleMessage): (WebInspector.ConsoleView.prototype.updateConsoleMessageExpiredCount): (WebInspector.ConsoleView.prototype.consoleMessagesCleared):
  • inspector/front-end/inspector.js:
  • page/Console.cpp: (WebCore::Console::addMessage): (WebCore::Console::count): (WebCore::Console::profile): (WebCore::Console::time): (WebCore::Console::timeEnd): (WebCore::Console::group): (WebCore::Console::groupCollapsed): (WebCore::Console::groupEnd):
  • page/Console.h:
  • page/Console.idl:
06:50 Changeset [75791] by pfeldman@chromium.org
  • 11 edits in trunk

2011-01-14 Pavel Feldman <pfeldman@chromium.org>

Web Inspector: do not use this as protocol message
dispatcher in models / agents.
https://bugs.webkit.org/show_bug.cgi?id=52442

Models should never for InspectorBackend.registerDomainDispatcher("foo", this).
There should be a clear way to tell whether method is being called
from within front-end or by the backend.

  • http/tests/inspector/debugger-test2.js: (initialize_DebuggerTest.InspectorTest.startDebuggerTest.startTest): (initialize_DebuggerTest.InspectorTest.startDebuggerTest): (initialize_DebuggerTest.InspectorTest.completeDebuggerTest.disableDebugger): (initialize_DebuggerTest.InspectorTest.showScriptSource):

2011-01-14 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: do not use this as protocol message
dispatcher in models / agents.
https://bugs.webkit.org/show_bug.cgi?id=52442

Models should never for InspectorBackend.registerDomainDispatcher("foo", this).
There should be a clear way to tell whether method is being called
from within front-end or by the backend.

  • inspector/front-end/DOMAgent.js: (WebInspector.DOMAgent): (WebInspector.DOMAgent.prototype.nodeForId): (WebInspector.DOMAgent.prototype._bindNodes): (WebInspector.DOMAgent.prototype._removeBreakpoints): (WebInspector.DOMDispatcher): (WebInspector.DOMDispatcher.prototype.setDocument): (WebInspector.DOMDispatcher.prototype.attributesUpdated): (WebInspector.DOMDispatcher.prototype.characterDataModified): (WebInspector.DOMDispatcher.prototype.setChildNodes): (WebInspector.DOMDispatcher.prototype.setDetachedRoot): (WebInspector.DOMDispatcher.prototype.childNodeCountUpdated): (WebInspector.DOMDispatcher.prototype.childNodeInserted): (WebInspector.DOMDispatcher.prototype.childNodeRemoved): (WebInspector.DOMDispatcher.prototype.didCommitLoad): (WebInspector.ApplicationCacheDispatcher): (WebInspector.ApplicationCacheDispatcher.prototype.getApplicationCachesAsync): (WebInspector.ApplicationCacheDispatcher.prototype.updateApplicationCacheStatus): (WebInspector.ApplicationCacheDispatcher.prototype.updateNetworkState):
  • inspector/front-end/DOMStorage.js: (WebInspector.DOMStorageDispatcher): (WebInspector.DOMStorageDispatcher.prototype.addDOMStorage): (WebInspector.DOMStorageDispatcher.prototype.selectDOMStorage): (WebInspector.DOMStorageDispatcher.prototype.updateDOMStorage):
  • inspector/front-end/Database.js: (WebInspector.Database.prototype.executeSql): (WebInspector.DatabaseDispatcher): (WebInspector.DatabaseDispatcher.prototype.addDatabase): (WebInspector.DatabaseDispatcher.prototype.selectDatabase): (WebInspector.DatabaseDispatcher.prototype.sqlTransactionSucceeded): (WebInspector.DatabaseDispatcher.prototype.sqlTransactionFailed):
  • inspector/front-end/DebuggerModel.js: (WebInspector.DebuggerModel): (WebInspector.DebuggerModel.prototype._pausedScript): (WebInspector.DebuggerModel.prototype._resumedScript): (WebInspector.DebuggerModel.prototype._parsedScriptSource): (WebInspector.DebuggerModel.prototype._failedToParseScriptSource): (WebInspector.DebuggerDispatcher): (WebInspector.DebuggerDispatcher.prototype.pausedScript): (WebInspector.DebuggerDispatcher.prototype.resumedScript): (WebInspector.DebuggerDispatcher.prototype.parsedScriptSource): (WebInspector.DebuggerDispatcher.prototype.failedToParseScriptSource):
  • inspector/front-end/FileSystemView.js: (WebInspector.FileSystemDispatcher): (WebInspector.FileSystemDispatcher.prototype.getFileSystemPathsAsync): (WebInspector.FileSystemDispatcher.prototype.didGetFileSystemPath): (WebInspector.FileSystemDispatcher.prototype.didGetFileSystemError): (WebInspector.FileSystemDispatcher.prototype.didGetFileSystemDisabled):
  • inspector/front-end/ProfilesPanel.js: (WebInspector.ProfilesPanel): (WebInspector.ProfilesPanel.prototype._addProfileHeader): (WebInspector.ProfilesPanel.prototype._removeProfileHeader): (WebInspector.ProfilesPanel.prototype._addHeapSnapshotChunk): (WebInspector.ProfilesPanel.prototype._finishHeapSnapshot): (WebInspector.ProfilesPanel.prototype._setRecordingProfile): (WebInspector.ProfilerDispatcher): (WebInspector.ProfilerDispatcher.prototype.profilerWasEnabled): (WebInspector.ProfilerDispatcher.prototype.profilerWasDisabled): (WebInspector.ProfilerDispatcher.prototype.resetProfiles): (WebInspector.ProfilerDispatcher.prototype.addProfileHeader): (WebInspector.ProfilerDispatcher.prototype.addHeapSnapshotChunk): (WebInspector.ProfilerDispatcher.prototype.finishHeapSnapshot): (WebInspector.ProfilerDispatcher.prototype.setRecordingProfile): (WebInspector.ProfileSidebarTreeElement.prototype.ondelete):
  • inspector/front-end/ResourceTreeModel.js: (WebInspector.ResourceTreeModel):
  • inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel): (WebInspector.TimelinePanel.prototype._timelineProfilerWasStarted): (WebInspector.TimelinePanel.prototype._timelineProfilerWasStopped): (WebInspector.TimelinePanel.prototype._addRecordToTimeline): (WebInspector.TimelineDispatcher): (WebInspector.TimelineDispatcher.prototype.timelineProfilerWasStarted): (WebInspector.TimelineDispatcher.prototype.timelineProfilerWasStopped): (WebInspector.TimelineDispatcher.prototype.addRecordToTimeline):
06:25 Changeset [75790] by ossy@webkit.org
  • 2 edits in trunk/Source/WebCore

[Qt][V8] Unreviewed buildfix after r75788.

  • inspector/InspectorBrowserDebuggerAgent.cpp:
05:31 Changeset [75789] by caseq@chromium.org
  • 2 edits in trunk/LayoutTests

2011-01-14 Andrey Kosyakov <caseq@chromium.org>

Unreviewed test fix.
Filter out /favicon.ico from page resources list to avoid test expectations mismatch on GTK.

  • http/tests/inspector/extensions-resources-redirect.html:
04:54 Changeset [75788] by loislo@chromium.org
  • 14 edits
    2 adds in trunk/Source/WebCore

2011-01-12 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: Extract BrowserDebuggerAgent from InspectorController, InspectorDOMAgent and InspectorDebugger agent.
We have some methods of Debugger which are related to DOM.
Lets extract these methods to BrowserDebugger agent.

http://bugs.webkit.org/show_bug.cgi?id=52294

  • CMakeLists.txt:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • inspector/CodeGeneratorInspector.pm:
  • inspector/Inspector.idl:
  • inspector/InspectorBrowserDebuggerAgent.cpp: Added. (WebCore::InspectorBrowserDebuggerAgent::InspectorBrowserDebuggerAgent): (WebCore::InspectorBrowserDebuggerAgent::~InspectorBrowserDebuggerAgent): (WebCore::InspectorBrowserDebuggerAgent::discardBindings): (WebCore::InspectorBrowserDebuggerAgent::setEventListenerBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::removeEventListenerBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::didInsertDOMNode): (WebCore::InspectorBrowserDebuggerAgent::didRemoveDOMNode): (WebCore::InspectorBrowserDebuggerAgent::setDOMBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::removeDOMBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::willInsertDOMNode): (WebCore::InspectorBrowserDebuggerAgent::willRemoveDOMNode): (WebCore::InspectorBrowserDebuggerAgent::willModifyDOMAttr): (WebCore::InspectorBrowserDebuggerAgent::descriptionForDOMEvent): (WebCore::InspectorBrowserDebuggerAgent::hasBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::updateSubtreeBreakpoints): (WebCore::InspectorBrowserDebuggerAgent::pauseOnNativeEventIfNeeded): (WebCore::InspectorBrowserDebuggerAgent::setXHRBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::removeXHRBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::willSendXMLHttpRequest): (WebCore::InspectorBrowserDebuggerAgent::clearForPageNavigation):
  • inspector/InspectorBrowserDebuggerAgent.h: Added. (WebCore::InspectorBrowserDebuggerAgent::create):
  • inspector/InspectorController.cpp: (WebCore::InspectorController::InspectorController): (WebCore::InspectorController::inspectedPageDestroyed): (WebCore::InspectorController::didCommitLoad): (WebCore::InspectorController::enableDebuggerFromFrontend): (WebCore::InspectorController::disableDebugger): (WebCore::InspectorController::restoreStickyBreakpoints): (WebCore::InspectorController::restoreStickyBreakpoint):
  • inspector/InspectorController.h:
  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::discardBindings): (WebCore::InspectorDOMAgent::didInsertDOMNode): (WebCore::InspectorDOMAgent::didRemoveDOMNode):
  • inspector/InspectorDOMAgent.h:
  • inspector/InspectorInstrumentation.cpp: (WebCore::InspectorInstrumentation::willInsertDOMNodeImpl): (WebCore::InspectorInstrumentation::didInsertDOMNodeImpl): (WebCore::InspectorInstrumentation::willRemoveDOMNodeImpl): (WebCore::InspectorInstrumentation::didRemoveDOMNodeImpl): (WebCore::InspectorInstrumentation::willModifyDOMAttrImpl): (WebCore::InspectorInstrumentation::willSendXMLHttpRequestImpl): (WebCore::InspectorInstrumentation::pauseOnNativeEventIfNeeded):
04:33 Changeset [75787] by eric@webkit.org
  • 4 edits in trunk/Tools

2011-01-14 Eric Seidel <eric@webkit.org>

Original patch from Peter Gal <galpater@inf.u-szeged.hu>

Reviewed by Adam Barth.

Fix when running Tools/Scripts/check-webkit-style without arguments.
https://bugs.webkit.org/show_bug.cgi?id=52261#c16

  • Scripts/webkitpy/common/checkout/scm.py: Handle None for changed_files argument better in Git.create_patch method.
  • Scripts/webkitpy/common/checkout/scm_unittest.py:
04:25 Changeset [75786] by caseq@chromium.org
  • 18 edits in trunk

2011-01-14 Andrey Kosyakov <caseq@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: redirected resources not handled properly in Network panel
https://bugs.webkit.org/show_bug.cgi?id=52292

  • inspector/Inspector.idl: Do not pass isMainResource to identifierForInitialRequest() (it's useless, as we may hit provisional load)
  • inspector/InspectorInstrumentation.cpp: Ditto. (WebCore::InspectorInstrumentation::identifierForInitialRequestImpl):
  • inspector/InspectorResourceAgent.cpp: Ditto. (WebCore::InspectorResourceAgent::identifierForInitialRequest):
  • inspector/InspectorResourceAgent.h: Ditto.
  • inspector/InspectorInstrumentation.cpp: Ditto. (WebCore::InspectorInstrumentation::identifierForInitialRequestImpl): Fix main resource detection.
  • inspector/front-end/AuditsPanel.js: WebInspector.networkResources now returns array, not map.
  • inspector/front-end/ConsoleView.js: User WebInspector.resourceById() to get resource (WebInspector.ConsoleMessage.prototype._formatMessage):
  • inspector/front-end/ExtensionServer.js: Ditto. (WebInspector.ExtensionServer.prototype._onRevealAndSelectResource): (WebInspector.ExtensionServer.prototype._onGetResourceContent):
  • inspector/front-end/HAREntry.js: WebInspector.networkResources now returns array, not map (WebInspector.HARLog.prototype.build): (WebInspector.HARLog.prototype._convertResource):
  • inspector/front-end/NetworkManager.js: (WebInspector.NetworkManager): Use appendResource, not refreshResource, when adding a new resource. (WebInspector.NetworkManager.prototype.identifierForInitialRequest): (WebInspector.NetworkManager.prototype.willSendRequest): (WebInspector.NetworkManager.prototype.didLoadResourceFromMemoryCache): (WebInspector.NetworkManager.prototype.setInitialContent): (WebInspector.NetworkManager.prototype.didCommitLoadForFrame): (WebInspector.NetworkManager.prototype.didCreateWebSocket): (WebInspector.NetworkManager.prototype._createResource): always register resource URL with ResourceTreeModel (WebInspector.NetworkManager.prototype._appendRedirect):
  • inspector/front-end/NetworkPanel.js: (WebInspector.NetworkPanel): (WebInspector.NetworkPanel.prototype.get resources): (WebInspector.NetworkPanel.prototype.resourceById): (WebInspector.NetworkPanel.prototype.appendResource): Add resource as new iff appendResource was added. (WebInspector.NetworkPanel.prototype.refreshResource): ditto. (WebInspector.NetworkPanel.prototype.mainResourceChanged): Clear console upon arrival of new main resource.
  • inspector/front-end/ResourceTreeModel.js: Expose unbindResourceURL, bind resources automatically upon creation. (WebInspector.ResourceTreeModel.prototype._clearResources): (WebInspector.ResourceTreeModel.prototype.unbindResourceURL): (WebInspector.ResourceTreeModel.prototype._addFramesRecursively): (WebInspector.ResourceTreeModel.prototype.createResource):
  • inspector/front-end/inspector.js: (WebInspector.resourceById):

2011-01-14 Andrey Kosyakov <caseq@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: redirected resources not handled properly in Network panel
Added test for redirected resource presence in HAR
Changed test to request entire HAR log, not multiple HAR entries.
https://bugs.webkit.org/show_bug.cgi?id=52292

  • http/tests/inspector/extensions-resources-redirect-expected.txt:
  • http/tests/inspector/extensions-resources-redirect.html:
  • http/tests/inspector/resource-har-conversion.html:
  • inspector/extensions-resources.html:
02:48 Changeset [75785] by loislo@chromium.org
  • 14 edits
    2 deletes in trunk/Source/WebCore

2011-01-14 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r75783.
http://trac.webkit.org/changeset/75783
https://bugs.webkit.org/show_bug.cgi?id=52439

a bit strange but the same patch have broken mac builds at
linking stage. (Requested by loislo2 on #webkit).

  • CMakeLists.txt:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • inspector/CodeGeneratorInspector.pm:
  • inspector/Inspector.idl:
  • inspector/InspectorBrowserDebuggerAgent.cpp: Removed.
  • inspector/InspectorBrowserDebuggerAgent.h: Removed.
  • inspector/InspectorController.cpp: (WebCore::InspectorController::InspectorController): (WebCore::InspectorController::inspectedPageDestroyed): (WebCore::InspectorController::didCommitLoad): (WebCore::InspectorController::enableDebuggerFromFrontend): (WebCore::InspectorController::disableDebugger): (WebCore::InspectorController::restoreStickyBreakpoints): (WebCore::InspectorController::restoreStickyBreakpoint): (WebCore::InspectorController::setEventListenerBreakpoint): (WebCore::InspectorController::removeEventListenerBreakpoint): (WebCore::InspectorController::hasEventListenerBreakpoint): (WebCore::InspectorController::setXHRBreakpoint): (WebCore::InspectorController::removeXHRBreakpoint): (WebCore::InspectorController::hasXHRBreakpoint):
  • inspector/InspectorController.h:
  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::discardBindings): (WebCore::InspectorDOMAgent::setDOMBreakpoint): (WebCore::InspectorDOMAgent::removeDOMBreakpoint): (WebCore::InspectorDOMAgent::shouldBreakOnNodeInsertion): (WebCore::InspectorDOMAgent::shouldBreakOnNodeRemoval): (WebCore::InspectorDOMAgent::shouldBreakOnAttributeModification): (WebCore::InspectorDOMAgent::descriptionForDOMEvent): (WebCore::InspectorDOMAgent::didInsertDOMNode): (WebCore::InspectorDOMAgent::didRemoveDOMNode): (WebCore::InspectorDOMAgent::hasBreakpoint): (WebCore::InspectorDOMAgent::updateSubtreeBreakpoints):
  • inspector/InspectorDOMAgent.h:
  • inspector/InspectorInstrumentation.cpp: (WebCore::InspectorInstrumentation::willInsertDOMNodeImpl): (WebCore::InspectorInstrumentation::didInsertDOMNodeImpl): (WebCore::InspectorInstrumentation::willRemoveDOMNodeImpl): (WebCore::InspectorInstrumentation::didRemoveDOMNodeImpl): (WebCore::InspectorInstrumentation::willModifyDOMAttrImpl): (WebCore::InspectorInstrumentation::willSendXMLHttpRequestImpl): (WebCore::InspectorInstrumentation::pauseOnNativeEventIfNeeded):
02:24 Changeset [75784] by commit-queue@webkit.org
  • 2 edits in trunk/WebKit/chromium

2011-01-14 Finnur Thorarinsson <finnur.webkit@gmail.com>

Reviewed by Ojan Vafai.

Address some user-select-none issues better in our Find in page scoping
function.

The scoping function is in part based on Editor::countMatchesForText
(formerly Frame::markAllMatchesForText), but was rewritten with
asychronous search and interruptability in mind. At the time the
function was written, countMatchesForText didn't work well with
user-select-none style but that has now changed. While
investigating http://crbug.com/68494 I noticed WebKit had fixed this
very problem in the countMatchesForText and I believe we should make
the same changes in our platform code. Therefore, this changelist
adopts the same approach by integrating:
https://bugs.webkit.org/show_bug.cgi?id=33508
(r53142: handling user-select-none better) and
https://bugs.webkit.org/show_bug.cgi?id=51623
(r74886: handling match within textfield better).
For more details see those changelists.

This changelist, however, is tracked here:
https://bugs.webkit.org/show_bug.cgi?id=52367

The original changelists did not come with layout tests, but I'm adding
a test on the Chromium side for this.

  • src/WebFrameImpl.cpp: (WebKit::WebFrameImpl::scopeStringMatches):
02:22 Changeset [75783] by loislo@chromium.org
  • 14 edits
    2 adds in trunk/Source/WebCore

2011-01-12 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: Extract BrowserDebuggerAgent from InspectorController, InspectorDOMAgent and InspectorDebugger agent.
We have some methods of Debugger which are related to DOM.
Lets extract these methods to BrowserDebugger agent.

http://bugs.webkit.org/show_bug.cgi?id=52294

  • CMakeLists.txt:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • inspector/CodeGeneratorInspector.pm:
  • inspector/Inspector.idl:
  • inspector/InspectorBrowserDebuggerAgent.cpp: Added. (WebCore::InspectorBrowserDebuggerAgent::InspectorBrowserDebuggerAgent): (WebCore::InspectorBrowserDebuggerAgent::~InspectorBrowserDebuggerAgent): (WebCore::InspectorBrowserDebuggerAgent::discardBindings): (WebCore::InspectorBrowserDebuggerAgent::setEventListenerBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::removeEventListenerBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::didInsertDOMNode): (WebCore::InspectorBrowserDebuggerAgent::didRemoveDOMNode): (WebCore::InspectorBrowserDebuggerAgent::setDOMBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::removeDOMBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::willInsertDOMNode): (WebCore::InspectorBrowserDebuggerAgent::willRemoveDOMNode): (WebCore::InspectorBrowserDebuggerAgent::willModifyDOMAttr): (WebCore::InspectorBrowserDebuggerAgent::descriptionForDOMEvent): (WebCore::InspectorBrowserDebuggerAgent::hasBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::updateSubtreeBreakpoints): (WebCore::InspectorBrowserDebuggerAgent::pauseOnNativeEventIfNeeded): (WebCore::InspectorBrowserDebuggerAgent::setXHRBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::removeXHRBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::willSendXMLHttpRequest): (WebCore::InspectorBrowserDebuggerAgent::clearForPageNavigation):
  • inspector/InspectorBrowserDebuggerAgent.h: Added. (WebCore::InspectorBrowserDebuggerAgent::create):
  • inspector/InspectorController.cpp: (WebCore::InspectorController::InspectorController): (WebCore::InspectorController::inspectedPageDestroyed): (WebCore::InspectorController::didCommitLoad): (WebCore::InspectorController::enableDebuggerFromFrontend): (WebCore::InspectorController::disableDebugger): (WebCore::InspectorController::restoreStickyBreakpoints): (WebCore::InspectorController::restoreStickyBreakpoint):
  • inspector/InspectorController.h:
  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::discardBindings): (WebCore::InspectorDOMAgent::didInsertDOMNode): (WebCore::InspectorDOMAgent::didRemoveDOMNode):
  • inspector/InspectorDOMAgent.h:
  • inspector/InspectorInstrumentation.cpp: (WebCore::InspectorInstrumentation::willInsertDOMNodeImpl): (WebCore::InspectorInstrumentation::didInsertDOMNodeImpl): (WebCore::InspectorInstrumentation::willRemoveDOMNodeImpl): (WebCore::InspectorInstrumentation::didRemoveDOMNodeImpl): (WebCore::InspectorInstrumentation::willModifyDOMAttrImpl): (WebCore::InspectorInstrumentation::willSendXMLHttpRequestImpl): (WebCore::InspectorInstrumentation::pauseOnNativeEventIfNeeded):
02:05 Changeset [75782] by pfeldman@chromium.org
  • 2 edits in trunk/WebKit/gtk

2011-01-14 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: gtk build fix (2).

  • webkit/webkitwebview.cpp:
02:02 Changeset [75781] by jochen@chromium.org
  • 3 edits in trunk/Tools

2011-01-04 Jochen Eisinger <jochen@chromium.org>

Reviewed by David Levin.

Store the tempdir object in a variable so it doesn't get deleted too early.
https://bugs.webkit.org/show_bug.cgi?id=51875

  • Scripts/update-webkit-auxiliary-libs:
  • Scripts/update-webkit-support-libs:
01:59 Changeset [75780] by pfeldman@chromium.org
  • 2 edits in trunk/WebKit/gtk

2011-01-14 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: gtk build fix.

  • webkit/webkitwebinspector.cpp:
01:52 Changeset [75779] by eric@webkit.org
  • 2 edits in trunk/Tools

2011-01-14 Eric Seidel <eric@webkit.org>

Unreviewed. Make it work in python 2.5.

webkit-patch suggest-reviewers dies when ChangeLogs are missing
https://bugs.webkit.org/show_bug.cgi?id=49158

  • Scripts/webkitpy/common/checkout/api_unittest.py:
01:47 Changeset [75778] by pfeldman@chromium.org
  • 4 edits in trunk

2011-01-14 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: build fix.

  • WebCore.vcproj/WebCore.vcproj:
01:28 Changeset [75777] by pfeldman@chromium.org
  • 13 edits
    2 adds in trunk

2011-01-14 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: extract InspectorSettings from InspectorState,
simplify it.
https://bugs.webkit.org/show_bug.cgi?id=52429

  • CMakeLists.txt:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.xcodeproj/project.pbxproj:
  • inspector/InspectorClient.h:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::InspectorController): (WebCore::InspectorController::inspectorStartsAttached): (WebCore::InspectorController::setInspectorStartsAttached): (WebCore::InspectorController::setInspectorAttachedHeight): (WebCore::InspectorController::inspectorAttachedHeight): (WebCore::InspectorController::setMonitoringXHREnabled): (WebCore::InspectorController::restoreDebugger): (WebCore::InspectorController::restoreProfiler): (WebCore::InspectorController::ensureSettingsLoaded): (WebCore::InspectorController::enableProfiler): (WebCore::InspectorController::disableProfiler): (WebCore::InspectorController::enableDebuggerFromFrontend): (WebCore::InspectorController::disableDebugger):
  • inspector/InspectorController.h:
  • inspector/InspectorSettings.cpp: Added. (WebCore::InspectorSettings::InspectorSettings): (WebCore::InspectorSettings::getBoolean): (WebCore::InspectorSettings::setBoolean): (WebCore::InspectorSettings::getLong): (WebCore::InspectorSettings::setLong): (WebCore::InspectorSettings::registerBoolean): (WebCore::InspectorSettings::registerLong):
  • inspector/InspectorSettings.h: Added.
  • inspector/InspectorState.cpp: (WebCore::InspectorState::InspectorState): (WebCore::InspectorState::setValue): (WebCore::InspectorState::setObject): (WebCore::InspectorState::registerBoolean): (WebCore::InspectorState::registerString): (WebCore::InspectorState::registerLong): (WebCore::InspectorState::registerObject): (WebCore::InspectorState::Property::create):
  • inspector/InspectorState.h: (WebCore::InspectorState::setBoolean): (WebCore::InspectorState::setString): (WebCore::InspectorState::setLong):

2011-01-14 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: extract InspectorSettings from InspectorState,
simplify it.
https://bugs.webkit.org/show_bug.cgi?id=52429

  • WebProcess/WebCoreSupport/WebInspectorClient.cpp:
01:21 Changeset [75776] by loislo@chromium.org
  • 15 edits
    2 deletes in trunk

2011-01-14 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r75774.
http://trac.webkit.org/changeset/75774
https://bugs.webkit.org/show_bug.cgi?id=52431

gtk builds were broken (Requested by loislo2 on #webkit).

  • CMakeLists.txt:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • inspector/CodeGeneratorInspector.pm:
  • inspector/Inspector.idl:
  • inspector/InspectorBrowserDebuggerAgent.cpp: Removed.
  • inspector/InspectorBrowserDebuggerAgent.h: Removed.
  • inspector/InspectorController.cpp: (WebCore::InspectorController::InspectorController): (WebCore::InspectorController::inspectedPageDestroyed): (WebCore::InspectorController::didCommitLoad): (WebCore::InspectorController::enableDebuggerFromFrontend): (WebCore::InspectorController::disableDebugger): (WebCore::InspectorController::restoreStickyBreakpoints): (WebCore::InspectorController::restoreStickyBreakpoint): (WebCore::InspectorController::setEventListenerBreakpoint): (WebCore::InspectorController::removeEventListenerBreakpoint): (WebCore::InspectorController::hasEventListenerBreakpoint): (WebCore::InspectorController::setXHRBreakpoint): (WebCore::InspectorController::removeXHRBreakpoint): (WebCore::InspectorController::hasXHRBreakpoint):
  • inspector/InspectorController.h:
  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::discardBindings): (WebCore::InspectorDOMAgent::setDOMBreakpoint): (WebCore::InspectorDOMAgent::removeDOMBreakpoint): (WebCore::InspectorDOMAgent::shouldBreakOnNodeInsertion): (WebCore::InspectorDOMAgent::shouldBreakOnNodeRemoval): (WebCore::InspectorDOMAgent::shouldBreakOnAttributeModification): (WebCore::InspectorDOMAgent::descriptionForDOMEvent): (WebCore::InspectorDOMAgent::didInsertDOMNode): (WebCore::InspectorDOMAgent::didRemoveDOMNode): (WebCore::InspectorDOMAgent::hasBreakpoint): (WebCore::InspectorDOMAgent::updateSubtreeBreakpoints):
  • inspector/InspectorDOMAgent.h:
  • inspector/InspectorInstrumentation.cpp: (WebCore::InspectorInstrumentation::willInsertDOMNodeImpl): (WebCore::InspectorInstrumentation::didInsertDOMNodeImpl): (WebCore::InspectorInstrumentation::willRemoveDOMNodeImpl): (WebCore::InspectorInstrumentation::didRemoveDOMNodeImpl): (WebCore::InspectorInstrumentation::willModifyDOMAttrImpl): (WebCore::InspectorInstrumentation::willSendXMLHttpRequestImpl): (WebCore::InspectorInstrumentation::pauseOnNativeEventIfNeeded):

2011-01-14 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r75774.
http://trac.webkit.org/changeset/75774
https://bugs.webkit.org/show_bug.cgi?id=52431

gtk builds were broken (Requested by loislo2 on #webkit).

  • Scripts/webkitpy/common/checkout/scm.py:
00:56 Changeset [75775] by levin@chromium.org
  • 3 edits in trunk/Tools

check-webkit-style: _FunctionState should detect pure functions.
https://bugs.webkit.org/show_bug.cgi?id=52428

Reviewed by Shinichiro Hamaji.

  • Scripts/webkitpy/style/checkers/cpp.py: Added detection for if a function declaration

is a pure virtual function.

  • Scripts/webkitpy/style/checkers/cpp_unittest.py: Added tests for verifying that pure

virtual functions are correctly detected.

00:21 Changeset [75774] by loislo@chromium.org
  • 15 edits
    2 adds in trunk

2011-01-12 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: Extract BrowserDebuggerAgent from InspectorController, InspectorDOMAgent and InspectorDebugger agent.
We have some methods of Debugger which are related to DOM.
Lets extract these methods to BrowserDebugger agent.

http://bugs.webkit.org/show_bug.cgi?id=52294

  • CMakeLists.txt:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • inspector/CodeGeneratorInspector.pm:
  • inspector/Inspector.idl:
  • inspector/InspectorBrowserDebuggerAgent.cpp: Added. (WebCore::InspectorBrowserDebuggerAgent::InspectorBrowserDebuggerAgent): (WebCore::InspectorBrowserDebuggerAgent::~InspectorBrowserDebuggerAgent): (WebCore::InspectorBrowserDebuggerAgent::discardBindings): (WebCore::InspectorBrowserDebuggerAgent::setEventListenerBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::removeEventListenerBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::didInsertDOMNode): (WebCore::InspectorBrowserDebuggerAgent::didRemoveDOMNode): (WebCore::InspectorBrowserDebuggerAgent::setDOMBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::removeDOMBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::willInsertDOMNode): (WebCore::InspectorBrowserDebuggerAgent::willRemoveDOMNode): (WebCore::InspectorBrowserDebuggerAgent::willModifyDOMAttr): (WebCore::InspectorBrowserDebuggerAgent::descriptionForDOMEvent): (WebCore::InspectorBrowserDebuggerAgent::hasBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::updateSubtreeBreakpoints): (WebCore::InspectorBrowserDebuggerAgent::pauseOnNativeEventIfNeeded): (WebCore::InspectorBrowserDebuggerAgent::setXHRBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::removeXHRBreakpoint): (WebCore::InspectorBrowserDebuggerAgent::willSendXMLHttpRequest): (WebCore::InspectorBrowserDebuggerAgent::clearForPageNavigation):
  • inspector/InspectorBrowserDebuggerAgent.h: Added. (WebCore::InspectorBrowserDebuggerAgent::create):
  • inspector/InspectorController.cpp: (WebCore::InspectorController::InspectorController): (WebCore::InspectorController::inspectedPageDestroyed): (WebCore::InspectorController::didCommitLoad): (WebCore::InspectorController::enableDebuggerFromFrontend): (WebCore::InspectorController::disableDebugger): (WebCore::InspectorController::restoreStickyBreakpoints): (WebCore::InspectorController::restoreStickyBreakpoint):
  • inspector/InspectorController.h:
  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::discardBindings): (WebCore::InspectorDOMAgent::didInsertDOMNode): (WebCore::InspectorDOMAgent::didRemoveDOMNode):
  • inspector/InspectorDOMAgent.h:
  • inspector/InspectorInstrumentation.cpp: (WebCore::InspectorInstrumentation::willInsertDOMNodeImpl): (WebCore::InspectorInstrumentation::didInsertDOMNodeImpl): (WebCore::InspectorInstrumentation::willRemoveDOMNodeImpl): (WebCore::InspectorInstrumentation::didRemoveDOMNodeImpl): (WebCore::InspectorInstrumentation::willModifyDOMAttrImpl): (WebCore::InspectorInstrumentation::willSendXMLHttpRequestImpl): (WebCore::InspectorInstrumentation::pauseOnNativeEventIfNeeded):

2011-01-13 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Yury Semikhatsky.

Just small fix for problem with check-webkit-style.
It just didn't work for this patch.
Old code can't concatenate a list of strings with None.

  • Scripts/webkitpy/common/checkout/scm.py:
00:14 Changeset [75773] by hayato@chromium.org
  • 2 edits in trunk/LayoutTests

2011-01-14 Hayato Ito <hayato@chromium.org>

Unreviewed chromium expectations update.

Exclude http/tests/loading/cross-origin-XHR-willLoadRequest.html (added in r75768)

  • platform/chromium/test_expectations.txt:

01/13/11:

22:58 Changeset [75772] by levin@chromium.org
  • 3 edits in trunk/Tools

2011-01-13 David Levin <levin@chromium.org>

Reviewed by Shinichiro Hamaji.

check-webkit-style: _FunctionState should use Position to know where items are.
https://bugs.webkit.org/show_bug.cgi?id=52424

  • Scripts/webkitpy/style/checkers/cpp.py: (_FunctionState.*): Changed _FunctionState to use Position instead of line numbers. (detect_functions): Changed to pass Position's to _FunctionState.begin and did some minor clean-up. (*): Other changes are simply about converting to use the Position's in_FunctionState instead of line numbers.
  • Scripts/webkitpy/style/checkers/cpp_unittest.py: Changed the test code to verify the positions stored in _FunctionState.
22:58 Changeset [75771] by hayato@chromium.org
  • 4 edits in trunk/LayoutTests

2011-01-13 Hayato Ito <hayato@chromium.org>

Unreviewed, build fix.

Fix Skipped lists in r75768.

  • platform/gtk/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:
22:21 Changeset [75770] by tkent@chromium.org
  • 7 edits in trunk/Source/WebCore

Implement form validation message UI
https://bugs.webkit.org/show_bug.cgi?id=48980

Reviewed by Dimitri Glazkov.

  • Add four internal pseudo selectors:
    • -webkit-validation-bubble
    • -webkit-validation-bubble-message
    • -webkit-validation-bubble-top-outer-arrow
    • -webkit-validation-bubble-top-inner-arrow
  • Implement ValidationMessage functions Show the message for <the number of characters> / 20.0 seconds.

No new tests because the feature is disabled by default for now and the
new behavior is strongly timing-dependent.

  • css/CSSStyleSelector.cpp:

(WebCore::CSSStyleSelector::canShareStyleWithElement):

Do not share a style with elements with different shadowPseudoId().

  • css/html.css: Define appearance for the internal selectors.

(::-webkit-validation-bubble):
(::-webkit-validation-bubble-message):
(::-webkit-validation-bubble-top-outer-arrow):
(::-webkit-validation-bubble-top-inner-arrow):

  • dom/Node.cpp:

(WebCore::Node::createRendererIfNeeded):

Allow to add shadow renderers even if canHaveChildren() returns false.

  • html/HTMLFormControlElement.cpp:

(WebCore::HTMLFormControlElement::detach):

Remove m_validationMessage immediately because we can't use
hideVisibleValidationMessage(), which calls a ValidationMessage function later.

(WebCore::HTMLFormControlElement::updateVisibleValidationMessage):

  • Don't create ValidationMessage if the message is empty.
  • Remove the check for message equality.

(WebCore::HTMLFormControlElement::hideVisibleValidationMessage):

Don't remove m_validationMessage immediately. We shouldn't make the
element needsLayout() state in this context.

  • html/ValidationMessage.cpp:

(WebCore::ValidationMessage::~ValidationMessage):

hideMessage() -> deleteBubbleTree() renaming.

(WebCore::ValidationMessage::setMessage): Implemented.
(WebCore::ValidationMessage::setMessageDOMAndStartTimer):

Added. This updates the validation message and starts the timer to hide it.

(WebCore::ElementWithPseudoId):

Added to help implementations of styled shadow nodes.

(WebCore::ValidationMessage::buildBubbleTree): Added.
(WebCore::ValidationMessage::requestToHideMessage): Added.
(WebCore::ValidationMessage::deleteBubbleTree):

Renamed from hideMessage(), and implemented.

  • html/ValidationMessage.h: Add declarations.
21:40 Changeset [75769] by eric@webkit.org
  • 3 edits in trunk/Tools

2011-01-13 Eric Seidel <eric@webkit.org>

Reviewed by David Levin.

webkit-patch suggest-reviewers dies when ChangeLogs are missing
https://bugs.webkit.org/show_bug.cgi?id=49158

This is not the most elegant, but it is a very safe fix to this bug.
One advantage of catching ScriptError like this instead of adding a
new added_or_modified_files or fixing all changed_files callers
to use a more specific change_files variant, is that we catch
all kinds of ScriptErrors which might cause our (non-essential)
suggest-reviewers code to fail out. This should make passing
--suggest-reviewers to webkit-patch upload much more robust
and may even make it possible for us to make it default.

The root of the problem here is that SCM.changed_files includes
deleted ChangeLog paths (from moves, etc) which then when we ask
SVN/Git for the contents of the file at that revision, the command
errors out and Executive.run_command raises a ScriptError.

In the future we might fix this differently by making all current
callers of chagned_files use a more specific method for requesting
what types of changes they're interested in (adds, modifies, deletes, etc.)

  • Scripts/webkitpy/common/checkout/api.py:
  • Scripts/webkitpy/common/checkout/api_unittest.py:
21:19 Changeset [75768] by mitz@apple.com
  • 10 edits
    2 adds in trunk

<rdar://problem/8827065> REGRESSION (r71884): Cross-origin XHR fails if willSendRequest changes the URL
https://bugs.webkit.org/show_bug.cgi?id=52419

Reviewed by Alexey Proskuryakov.

Source/WebCore:

Test: http/tests/loading/cross-origin-XHR-willLoadRequest.html

  • loader/ResourceLoader.cpp:

(WebCore::ResourceLoader::init): Complete r74453 by ensuring that m_request is only set after
willSendRequest(). Otherwise, willSendRequest() ends up calling into
ThreadableDocumentRequest::willSendRequest(), which cancels the request.

Tools:

Allow to further control the behavior of the willSendRequest delegate callback by making it
set a different URL in the new request.

  • DumpRenderTree/LayoutTestController.cpp:

(addURLToRedirectCallback): Added.
(LayoutTestController::staticFunctions): Added addURLToRedirect.
(LayoutTestController::addURLToRedirect): Added. Adds the redirection to the map.
(LayoutTestController::redirectionDestinationForURL): Added this getter.

  • DumpRenderTree/LayoutTestController.h:
  • DumpRenderTree/mac/ResourceLoadDelegate.mm:

(-[ResourceLoadDelegate webView:resource:willSendRequest:redirectResponse:fromDataSource:]): Check
if there is a redirection for the request’s URL, established by addURLToRedirect(), and if so,
change the URL in the new request to the redirect destination.

LayoutTests:

  • http/tests/loading/cross-origin-XHR-willLoadRequest-expected.txt: Added.
  • http/tests/loading/cross-origin-XHR-willLoadRequest.html: Added.
  • platform/gtk/Skipped: Skip the new test.
  • platform/qt/Skipped: Ditto.
  • platform/win/Skipped: Ditto.
20:19 Changeset [75767] by mitz@apple.com
  • 2 edits in trunk/Source/WebCore

Don’t try to compile InjectedScriptSource.js into WebCore.

Reviewed by Mark Rowe.

  • WebCore.xcodeproj/project.pbxproj:
18:52 Changeset [75766] by darin@apple.com
  • 4 edits in trunk/Source/JavaScriptCore

<rdar://problem/5469576> Use PTK_FRAMEWORK_JAVASCRIPTCORE_KEY0 if available.

Reviewed by Geoff Garen.

  • DerivedSources.make: Create a header file, HeaderDetection.h, that tells

us whether pthread_machdep.h is available.

  • wtf/FastMalloc.cpp: If PTK_FRAMEWORK_JAVASCRIPTCORE_KEY0 is available, then

don't do the pthread_getspecific_function_pointer trick.
(WTF::setThreadHeap): Ditto, but set thread-specific data.
(WTF::TCMalloc_ThreadCache::GetThreadHeap): Ditto, but get rather than set.

18:24 Changeset [75765] by jschuh@chromium.org
  • 2 edits in trunk/LayoutTests

2011-01-13 Justin Schuh <jschuh@chromium.org>

Unreviewed chromium expectations update.

Several text layout tests failing on linux after r75756

  • platform/chromium/test_expectations.txt:
18:11 Changeset [75764] by mrobinson@webkit.org
  • 2 edits in trunk/LayoutTests

2011-01-13 Martin Robinson <mrobinson@igalia.com>

Unreviewed, rolling out r75740.
http://trac.webkit.org/changeset/75740

This led to GTK+ redness

  • platform/gtk/Skipped:
18:07 Changeset [75763] by robert@webkit.org
  • 2 edits in trunk/LayoutTests

2011-01-13 Robert Hogan <robert@webkit.org>

Reviewed by Andreas Kling.

[Qt] Unskip http/tests/security/originHeader/origin-header-for-empty.html
https://bugs.webkit.org/show_bug.cgi?id=52389

  • platform/qt/Skipped:
17:58 Changeset [75762] by mihaip@chromium.org
  • 1 edit
    63 adds in trunk/LayoutTests

2011-01-13 Mihai Parparita <mihaip@chromium.org>

Unreviewed chromium expectations update.

Add Chromium Snow Leopard expectations. These need to go into a new
chromium-mac-snowleopard directory, since these tests do not have
chromium-mac expectations, but currently pass on Leopard.

The differences are due to bug 52335 (which explains why we can't use
the regular mac port Snow Leopard baselines).

  • platform/chromium-mac-snowleopard/css1/box_properties/clear-expected.checksum: Added.
  • platform/chromium-mac-snowleopard/css1/box_properties/clear-expected.png: Added.
  • platform/chromium-mac-snowleopard/css1/box_properties/float-expected.checksum: Added.
  • platform/chromium-mac-snowleopard/css1/box_properties/float-expected.png: Added.
  • platform/chromium-mac-snowleopard/css1/text_properties/vertical_align-expected.checksum: Added.
  • platform/chromium-mac-snowleopard/css1/text_properties/vertical_align-expected.png: Added.
  • platform/chromium-mac-snowleopard/css2.1/t0803-c5501-imrgn-t-00-b-ag-expected.checksum: Added.
  • platform/chromium-mac-snowleopard/css2.1/t0803-c5501-imrgn-t-00-b-ag-expected.png: Added.
  • platform/chromium-mac-snowleopard/css2.1/t0803-c5502-mrgn-r-00-c-ag-expected.checksum: Added.
  • platform/chromium-mac-snowleopard/css2.1/t0803-c5502-mrgn-r-00-c-ag-expected.png: Added.
  • platform/chromium-mac-snowleopard/css2.1/t0803-c5503-imrgn-b-00-b-a-expected.checksum: Added.
  • platform/chromium-mac-snowleopard/css2.1/t0803-c5503-imrgn-b-00-b-a-expected.png: Added.
  • platform/chromium-mac-snowleopard/css2.1/t0803-c5504-mrgn-l-00-c-ag-expected.checksum: Added.
  • platform/chromium-mac-snowleopard/css2.1/t0803-c5504-mrgn-l-00-c-ag-expected.png: Added.
  • platform/chromium-mac-snowleopard/css2.1/t0803-c5505-mrgn-00-b-ag-expected.checksum: Added.
  • platform/chromium-mac-snowleopard/css2.1/t0803-c5505-mrgn-00-b-ag-expected.png: Added.
  • platform/chromium-mac-snowleopard/css2.1/t0905-c414-flt-01-d-g-expected.checksum: Added.
  • platform/chromium-mac-snowleopard/css2.1/t0905-c414-flt-01-d-g-expected.png: Added.
  • platform/chromium-mac-snowleopard/css2.1/t100304-c43-rpl-bbx-00-d-g-expected.checksum: Added.
  • platform/chromium-mac-snowleopard/css2.1/t100304-c43-rpl-bbx-00-d-g-expected.png: Added.
  • platform/chromium-mac-snowleopard/css2.1/t1008-c44-ln-box-00-d-ag-expected.checksum: Added.
  • platform/chromium-mac-snowleopard/css2.1/t1008-c44-ln-box-00-d-ag-expected.png: Added.
  • platform/chromium-mac-snowleopard/css2.1/t1008-c44-ln-box-02-d-ag-expected.checksum: Added.
  • platform/chromium-mac-snowleopard/css2.1/t1008-c44-ln-box-02-d-ag-expected.png: Added.
  • platform/chromium-mac-snowleopard/css2.1/t1008-c44-ln-box-03-d-ag-expected.checksum: Added.
  • platform/chromium-mac-snowleopard/css2.1/t1008-c44-ln-box-03-d-ag-expected.png: Added.
  • platform/chromium-mac-snowleopard/css2.1/t100801-c544-valgn-02-d-agi-expected.checksum: Added.
  • platform/chromium-mac-snowleopard/css2.1/t100801-c544-valgn-02-d-agi-expected.png: Added.
  • platform/chromium-mac-snowleopard/css2.1/t100801-c544-valgn-03-d-agi-expected.checksum: Added.
  • platform/chromium-mac-snowleopard/css2.1/t100801-c544-valgn-03-d-agi-expected.png: Added.
  • platform/chromium-mac-snowleopard/css2.1/t100801-c544-valgn-04-d-agi-expected.checksum: Added.
  • platform/chromium-mac-snowleopard/css2.1/t100801-c544-valgn-04-d-agi-expected.png: Added.
  • platform/chromium-mac-snowleopard/fast/css/resize-corner-tracking-transformed-expected.checksum: Added.
  • platform/chromium-mac-snowleopard/fast/css/resize-corner-tracking-transformed-expected.png: Added.
  • platform/chromium-mac-snowleopard/fast/forms/searchfield-heights-expected.checksum: Added.
  • platform/chromium-mac-snowleopard/fast/forms/searchfield-heights-expected.png: Added.
  • platform/chromium-mac-snowleopard/fast/forms/slider-thumb-shared-style-expected.checksum: Added.
  • platform/chromium-mac-snowleopard/fast/forms/slider-thumb-shared-style-expected.png: Added.
  • platform/chromium-mac-snowleopard/fast/gradients/background-clipped-expected.checksum: Added.
  • platform/chromium-mac-snowleopard/fast/gradients/background-clipped-expected.png: Added.
  • platform/chromium-mac-snowleopard/fast/replaced/selection-rect-transform-expected.checksum: Added.
  • platform/chromium-mac-snowleopard/fast/replaced/selection-rect-transform-expected.png: Added.
  • platform/chromium-mac-snowleopard/fast/text/whitespace/tab-character-basics-expected.checksum: Added.
  • platform/chromium-mac-snowleopard/fast/text/whitespace/tab-character-basics-expected.png: Added.
  • platform/chromium-mac-snowleopard/fast/transforms/diamond-expected.checksum: Added.
  • platform/chromium-mac-snowleopard/fast/transforms/diamond-expected.png: Added.
  • platform/chromium-mac-snowleopard/fast/transforms/shadows-expected.checksum: Added.
  • platform/chromium-mac-snowleopard/fast/transforms/shadows-expected.png: Added.
  • platform/chromium-mac-snowleopard/fast/transforms/transformed-caret-expected.checksum: Added.
  • platform/chromium-mac-snowleopard/fast/transforms/transformed-caret-expected.png: Added.
17:56 Changeset [75761] by eric@webkit.org
  • 4 edits in trunk/Tools

2011-01-13 Eric Seidel <eric@webkit.org>

Reviewed by Mihai Parparita.

webkit-patch suggest-reviewers fails due to invalid utf8 in ChangeLog files
https://bugs.webkit.org/show_bug.cgi?id=52416

Example: webkit-patch suggest-reviewers -g ca3890fe74d94d85d6bfa48f9ea497b094d8e717
UnicodeDecodeError: 'utf8' codec can't decode bytes in position 536-538: invalid data

I also added FIXMEs about the changed_files returning deleted files problem
which is causing the other most common exception for suggest-reviewers
(and is probably breaking sheriff-bots blame detection).

  • Scripts/webkitpy/common/checkout/api.py:
  • Scripts/webkitpy/common/checkout/api_unittest.py:
  • Scripts/webkitpy/common/checkout/scm.py:
17:51 Changeset [75760] by eric@webkit.org
  • 4 edits in trunk/Tools

2011-01-13 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

results archives from commit-queue are too large to upload
https://bugs.webkit.org/show_bug.cgi?id=52405

  • Scripts/webkitpy/common/system/filesystem.py:
  • Scripts/webkitpy/common/system/filesystem_mock.py:
  • Scripts/webkitpy/tool/commands/queues.py:
17:31 Changeset [75759] by jam@chromium.org
  • 2 edits in trunk/WebKit/chromium

2011-01-13 John Abd-El-Malek <jam@chromium.org>

Reviewed by Mihai Parparita.

[chromium] Fix minor type in plugin printing
https://bugs.webkit.org/show_bug.cgi?id=52410

  • src/WebFrameImpl.cpp: (WebKit::WebFrameImpl::printBegin):
17:16 Changeset [75758] by mthole@apple.com
  • 4 edits in trunk/Source/WebCore

The pageScaleFactor() should be saved/restored along with the scroll position
https://bugs.webkit.org/show_bug.cgi?id=52406
<rdar://problem/8714412>

Reviewed by Darin Adler and Brady Eidson.

  • history/HistoryItem.cpp:

(WebCore::HistoryItem::HistoryItem): Initialize m_pageScaleFactor.
(WebCore::HistoryItem::pageScaleFactor): Added getter.
(WebCore::HistoryItem::setPageScaleFactor): Added setter.
(WebCore::HistoryItem::encodeBackForwardTreeNode): Encode m_pageScaleFactor.
(WebCore::HistoryItem::decodeBackForwardTree): Decode m_pageScaleFactor.

  • history/HistoryItem.h:
  • loader/HistoryController.cpp:

(WebCore::HistoryController::saveScrollPositionAndViewStateToItem): Save the page scale factor...
(WebCore::HistoryController::restoreScrollPositionAndViewState): ...and restore it here.

17:15 Changeset [75757] by mihaip@chromium.org
  • 2 edits
    4 adds in trunk/LayoutTests

2011-01-13 Mihai Parparita <mihaip@chromium.org>

Unreviewed chromium expectations update.

Categorize remaining scrollbar-related failures. 6 are due to bug 52414
(small remaining difference between mac and chromium-mac scrollbars).
fast/text/international/thai-line-breaks.html just needed a baseline
update (done for both Leopard and Snow Leopard).

  • platform/chromium-mac-leopard/fast/text/international/thai-line-breaks-expected.checksum: Added.
  • platform/chromium-mac-leopard/fast/text/international/thai-line-breaks-expected.png: Added.
  • platform/chromium-mac/fast/text/international/thai-line-breaks-expected.checksum: Added.
  • platform/chromium-mac/fast/text/international/thai-line-breaks-expected.png: Added.
  • platform/chromium/test_expectations.txt:
16:48 Changeset [75756] by evan@chromium.org
  • 4 edits in trunk/Source/WebCore

2011-01-13 Evan Martin <evan@chromium.org>

Reviewed by Tony Chang.

[chromium] drop backwards iteration in Linux complex text code
https://bugs.webkit.org/show_bug.cgi?id=52403

ComplexTextController previously supported iterating through the text in
both directions, but this resulted in duplicate code for each path.
Instead, by being more careful about flipping signs where appropriate,
we can refactor the code into one code path.

No tests, just a refactoring; should be covered by existing tests.

  • platform/graphics/chromium/ComplexTextControllerLinux.cpp: (WebCore::ComplexTextController::ComplexTextController): (WebCore::ComplexTextController::reset): (WebCore::ComplexTextController::nextScriptRun):
  • platform/graphics/chromium/ComplexTextControllerLinux.h:
  • platform/graphics/chromium/FontLinux.cpp: (WebCore::glyphIndexForXPositionInScriptRun): (WebCore::Font::selectionRectForComplexText):
16:41 Changeset [75755] by mrowe@apple.com
  • 2 edits in tags/Safari-534.15.7/WebKit2

Build fix.

16:39 Changeset [75754] by ojan@chromium.org
  • 3 edits in trunk/Websites/bugs.webkit.org

2011-01-13 Ojan Vafai <ojan@chromium.org>

Reviewed by Adam Barth.

simplify keyboard handling in code review tool
https://bugs.webkit.org/show_bug.cgi?id=52407

Now that we have DiffBlock containers, the only things that are
focusable are previousComment nodes and DiffBlock containers
that contain add/remove lines.

Also, this means we show the focus border around the entire diff
instead of just the first line.

  • code-review.js:
16:16 Changeset [75753] by mrowe@apple.com
  • 9 edits in tags/Safari-534.15.7

Merge r75662.

16:16 Changeset [75752] by mrowe@apple.com
  • 15 edits in tags/Safari-534.15.7

Merge r75635.

16:16 Changeset [75751] by mrowe@apple.com
  • 5 edits in tags/Safari-534.15.7

Merge r74309.

16:15 Changeset [75750] by mrowe@apple.com
  • 5 edits in tags/Safari-534.15.7

Versioning.

16:14 Changeset [75749] by dglazkov@chromium.org
  • 12 edits in trunk/Source/WebCore

2011-01-13 Dimitri Glazkov <dglazkov@chromium.org>

Reviewed by Darin Adler.

Flip input[type=range] to use the new shadow DOM model.
https://bugs.webkit.org/show_bug.cgi?id=52317

This is a straight-up flip, minimal change to the logic, which will be
refactored in a follow-up patch.

Covered by existing tests.

  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::initForStyleResolve): Changed to use either

host or parent node to find parent.

  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::updateType): Added logic for destroying/creating

shadow DOM subtree.

  • html/InputType.cpp: (WebCore::InputType::createShadowSubtree): Added empty decl. (WebCore::InputType::destroyShadowSubtree): Added a method to destroy

shadow subtree.

  • html/InputType.h: Added defs.
  • html/RangeInputType.cpp: (WebCore::RangeInputType::createShadowSubtree): Added a virtual method to create

slider thumb.

  • html/RangeInputType.h: Adde def.
  • html/shadow/SliderThumbElement.cpp: Added temporary RenderSliderThumb class

to handle cascading appearance.

(WebCore::RenderSliderThumb::RenderSliderThumb): Added.
(WebCore::RenderSliderThumb::layout): Moved here the logic from

RenderSlider::createThumbStyle.

(WebCore::SliderThumbElement::createRenderer): Added virtual method to

create RenderSliderThumb instance.

(WebCore::SliderThumbElement::defaultEventHandler): Changed to use HTMLDivElement

as base class.

(WebCore::SliderThumbElement::detach): Ditto.

  • html/shadow/SliderThumbElement.h: (WebCore::SliderThumbElement::SliderThumbElement): Ditto. (WebCore::SliderThumbElement::create): Ditto. (WebCore::toSliderThumbElement): Added a casting helper.
  • rendering/MediaControlElements.cpp: (WebCore::MediaControlInputElement::attach): Added handling of shadow DOM,

since the whole method is hand-rolled.

(WebCore::MediaControlInputElement::updateStyle): Ditto.

  • rendering/RenderSlider.cpp: (WebCore::RenderSlider::~RenderSlider): Removed unneeded code. (WebCore::RenderSlider::thumbRect): Changed to use sliderThumbElement helper. (WebCore::RenderSlider::layout): Ditto. (WebCore::RenderSlider::sliderThumbElement): Added a temporary helper to access

SliderThumbElement.

(WebCore::RenderSlider::mouseEventIsInThumb): Changed to use sliderThumbElement helper.
(WebCore::RenderSlider::mouseEventOffsetToThumb): Ditto.
(WebCore::RenderSlider::setValueForPosition): Ditto.
(WebCore::RenderSlider::positionForOffset): Ditto.
(WebCore::RenderSlider::currentPosition): Ditto.
(WebCore::RenderSlider::trackSize): Ditto.
(WebCore::RenderSlider::forwardEvent): Ditto.
(WebCore::RenderSlider::inDragMode): Ditto.

  • rendering/RenderSlider.h: Added def.
16:06 BuildBot edited by lforschler@apple.com
(diff)
15:55 Changeset [75748] by abarth@webkit.org
  • 32 edits
    1 move in trunk

2011-01-13 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Rename RGBA32Buffer to ImageFrame
https://bugs.webkit.org/show_bug.cgi?id=52363

Update references to the new name.

  • src/WebImageDecoder.cpp: (WebKit::WebImageDecoder::isFrameCompleteAtIndex): (WebKit::WebImageDecoder::getFrameAtIndex):

2011-01-13 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Rename RGBA32Buffer to ImageFrame
https://bugs.webkit.org/show_bug.cgi?id=52363

RGBA32Buffer is a terrible misnomer. ImageFrame is vastly
more accurate. More cleanup to come.

  • WebCore.pro:
  • platform/graphics/ImageSource.cpp: (WebCore::ImageSource::createFrameAtIndex): (WebCore::ImageSource::frameDurationAtIndex): (WebCore::ImageSource::frameIsCompleteAtIndex):
  • platform/graphics/qt/ImageDecoderQt.cpp: (WebCore::ImageDecoderQt::frameBufferAtIndex): (WebCore::ImageDecoderQt::internalReadImage): (WebCore::ImageDecoderQt::internalHandleCurrentImage):
  • platform/graphics/qt/ImageDecoderQt.h:
  • platform/graphics/wince/ImageWinCE.cpp: (WebCore::ImageFrame::asNewNativeImage):
  • platform/image-decoders/ImageDecoder.cpp: (WebCore::ImageFrame::ImageFrame): (WebCore::ImageFrame::operator=): (WebCore::ImageFrame::clear): (WebCore::ImageFrame::zeroFill): (WebCore::ImageFrame::copyReferenceToBitmapData): (WebCore::ImageFrame::copyBitmapData): (WebCore::ImageFrame::setSize): (WebCore::ImageFrame::hasAlpha): (WebCore::ImageFrame::setHasAlpha): (WebCore::ImageFrame::setColorProfile): (WebCore::ImageFrame::setStatus): (WebCore::ImageFrame::width): (WebCore::ImageFrame::height):
  • platform/image-decoders/ImageDecoder.h: (WebCore::ImageFrame::ImageFrame):
  • platform/image-decoders/bmp/BMPImageDecoder.cpp: (WebCore::BMPImageDecoder::frameBufferAtIndex): (WebCore::BMPImageDecoder::decode):
  • platform/image-decoders/bmp/BMPImageDecoder.h:
  • platform/image-decoders/bmp/BMPImageReader.cpp: (WebCore::BMPImageReader::decodeBMP):
  • platform/image-decoders/bmp/BMPImageReader.h: (WebCore::BMPImageReader::setBuffer):
  • platform/image-decoders/cairo/ImageDecoderCairo.cpp: (WebCore::ImageFrame::asNewNativeImage):
  • platform/image-decoders/cg/ImageDecoderCG.cpp: (WebCore::getPtrAsPixelData): (WebCore::ImageFrame::copyReferenceToBitmapData): (WebCore::ImageFrame::copyBitmapData): (WebCore::ImageFrame::setSize): (WebCore::ImageFrame::asNewNativeImage):
  • platform/image-decoders/gif/GIFImageDecoder.cpp: (WebCore::GIFImageDecoder::frameBufferAtIndex): (WebCore::GIFImageDecoder::clearFrameBufferCache): (WebCore::GIFImageDecoder::haveDecodedRow): (WebCore::GIFImageDecoder::frameComplete): (WebCore::GIFImageDecoder::initFrameBuffer):
  • platform/image-decoders/gif/GIFImageDecoder.h:
  • platform/image-decoders/gif/GIFImageReader.cpp: (GIFImageReader::read):
  • platform/image-decoders/gif/GIFImageReader.h: (GIFFrameReader::GIFFrameReader):
  • platform/image-decoders/haiku/ImageDecoderHaiku.cpp: (WebCore::ImageFrame::asNewNativeImage):
  • platform/image-decoders/ico/ICOImageDecoder.cpp: (WebCore::ICOImageDecoder::frameBufferAtIndex): (WebCore::ICOImageDecoder::decode):
  • platform/image-decoders/ico/ICOImageDecoder.h:
  • platform/image-decoders/jpeg/JPEGImageDecoder.cpp: (WebCore::JPEGImageDecoder::frameBufferAtIndex): (WebCore::JPEGImageDecoder::outputScanlines): (WebCore::JPEGImageDecoder::jpegComplete): (WebCore::JPEGImageDecoder::decode):
  • platform/image-decoders/jpeg/JPEGImageDecoder.h:
  • platform/image-decoders/openvg/ImageDecoderOpenVG.cpp: (WebCore::ImageFrame::asNewNativeImage):
  • platform/image-decoders/png/PNGImageDecoder.cpp: (WebCore::PNGImageDecoder::frameBufferAtIndex): (WebCore::PNGImageDecoder::rowAvailable): (WebCore::PNGImageDecoder::pngComplete):
  • platform/image-decoders/png/PNGImageDecoder.h: (WebCore::PNGImageDecoder::isComplete):
  • platform/image-decoders/qt/ImageFrameQt.cpp: Copied from Source/WebCore/platform/image-decoders/qt/RGBA32BufferQt.cpp. (WebCore::ImageFrame::ImageFrame): (WebCore::ImageFrame::operator=): (WebCore::ImageFrame::clear): (WebCore::ImageFrame::zeroFill): (WebCore::ImageFrame::copyBitmapData): (WebCore::ImageFrame::setSize): (WebCore::ImageFrame::asNewNativeImage): (WebCore::ImageFrame::hasAlpha): (WebCore::ImageFrame::setHasAlpha): (WebCore::ImageFrame::setColorProfile): (WebCore::ImageFrame::setStatus): (WebCore::ImageFrame::setPixmap): (WebCore::ImageFrame::width): (WebCore::ImageFrame::height):
  • platform/image-decoders/qt/RGBA32BufferQt.cpp: Removed.
  • platform/image-decoders/skia/ImageDecoderSkia.cpp: (WebCore::ImageFrame::ImageFrame): (WebCore::ImageFrame::operator=): (WebCore::ImageFrame::clear): (WebCore::ImageFrame::zeroFill): (WebCore::ImageFrame::copyBitmapData): (WebCore::ImageFrame::setSize): (WebCore::ImageFrame::asNewNativeImage): (WebCore::ImageFrame::hasAlpha): (WebCore::ImageFrame::setHasAlpha): (WebCore::ImageFrame::setColorProfile): (WebCore::ImageFrame::setStatus): (WebCore::ImageFrame::width): (WebCore::ImageFrame::height):
  • platform/image-decoders/webp/WEBPImageDecoder.cpp: (WebCore::WEBPImageDecoder::frameBufferAtIndex): (WebCore::WEBPImageDecoder::decode):
  • platform/image-decoders/webp/WEBPImageDecoder.h:
  • platform/image-decoders/wx/ImageDecoderWx.cpp: (WebCore::ImageFrame::asNewNativeImage):
15:52 Changeset [75747] by ojan@chromium.org
  • 3 edits in trunk/Websites/bugs.webkit.org

2011-01-13 Ojan Vafai <ojan@chromium.org>

Reviewed by Adam Barth.

add container divs for diff blocks
https://bugs.webkit.org/show_bug.cgi?id=52400

This will help simplify a lot of code in code-review.js
and make side-by-side diffs better (i.e. put removed lines
to the left of corresponding added lines).

Also, allow for running the JS from a local file. Now you can modify code-review.js
to point to a local file and then run:
ruby prettify.rb < foo.diff > foo.html

foo.html will load a dummy code review matching foo.diff.

Before structure:
Line
Line remove
Line add
Line add
Line

After structure:
DiffBlock

DiffBlockPart shared

Line shared

DiffBlock

DiffBlockPart remove

Line remove

DiffBlockPart add

Line add
Line add

DiffBlock

DiffBlockPart shared

Line shared

  • PrettyPatch/PrettyPatch.rb:
  • code-review.js:
15:38 Changeset [75746] by dglazkov@chromium.org
  • 4 edits in trunk/Source/WebCore

2011-01-13 Dimitri Glazkov <dglazkov@chromium.org>

Reviewed by Darin Adler.

Transferring nodes between documents should be aware of the shadow DOM.
https://bugs.webkit.org/show_bug.cgi?id=52399

No visible change in behavior at the moment.

Once https://bugs.webkit.org/show_bug.cgi?id=52317 lands, the following
tests should no longer crash:

  • fast/css/pseudo-in-range-invalid-value.html
  • fast/css/pseudo-in-range.html
  • fast/forms/form-collection-elements.html
  • fast/forms/range-keyoperation.html
  • dom/Document.cpp: (WebCore::Document::adoptNode): Changed to use Node::setDocumentRecursively.
  • dom/Node.cpp: (WebCore::Node::setDocumentRecursively): Added new method, taking existing

logic and adding shadow DOM traversal.

(WebCore::Node::traverseNextNode): Style fix.
(WebCore::Node::traverseNextSibling): Ditto.
(WebCore::Node::traversePreviousNode): Ditto.
(WebCore::Node::traversePreviousNodePostOrder): Ditto.
(WebCore::Node::checkReplaceChild): Changed to use setDocumentRecursively.
(WebCore::Node::checkAddChild): Ditto.

  • dom/Node.h: Added def.
15:28 Changeset [75745] by jschuh@chromium.org
  • 2 edits in trunk/LayoutTests

2011-01-13 Justin Schuh <jschuh@chromium.org>

Unreviewed chromium expectations update.

Remove exception for canvas-large-dimensions.html.

  • platform/chromium/test_expectations.txt:
15:19 Changeset [75744] by mrowe@apple.com
  • 1 copy in tags/Safari-534.15.7

New tag.

15:14 Changeset [75743] by enrica@apple.com
  • 30 edits
    1 add in trunk

Source/WebCore: WebKit2: Add support for drag and drop
https://bugs.webkit.org/show_bug.cgi?id=52343
<rdar://problem/7660558>

Reviewed by Darin Adler.

This patch contains the changes required to support dropping content
in WebKit on the Mac. The DragData class has been extended to provide
additional context from the application (keyboard state, modal windows, etc.)
as well as information of the drag pasteboard being used.
The support for WebKit as drag source will be added with a separate patch.

  • WebCore.exp.in:
  • page/DragController.cpp:

(WebCore::DragController::dragIsMove): Added DragData parameter.
(WebCore::DragController::tryDocumentDrag): Modified call to dragIsMove.
(WebCore::DragController::concludeEditDrag): Same as above.

  • page/DragController.h: Added parameter to isCopyKeyDown.
  • page/mac/DragControllerMac.mm:

The following methods have been modified to use the information stored
in the DragData class.
(WebCore::DragController::isCopyKeyDown):
(WebCore::DragController::dragOperation):

  • platform/DragData.h:

(WebCore::DragData::flags):

  • platform/DragData.cpp: Added default flags parameter to the constructor.
  • platform/mac/DragDataMac.mm:

(WebCore::DragData::DragData): Added constructor that receives the name of the pasteboard to use.
(WebCore::DragData::canSmartReplace):
(WebCore::DragData::containsColor):
(WebCore::DragData::containsFiles):
(WebCore::DragData::asFilenames):
(WebCore::DragData::containsPlainText):
(WebCore::DragData::asPlainText):
(WebCore::DragData::containsCompatibleContent):
(WebCore::DragData::asURL):
(WebCore::DragData::asFragment):
All the following files have been modified to add the DragData
parameter to isCopyKeyDown. I plan to improve this in the future
and make isCopyKeyDown not platform specific.

  • page/android/DragControllerAndroid.cpp:

(WebCore::DragController::isCopyKeyDown):

  • page/brew/DragControllerBrew.cpp:

(WebCore::DragController::isCopyKeyDown):

  • page/chromium/DragControllerChromium.cpp:

(WebCore::DragController::isCopyKeyDown):

  • page/efl/DragControllerEfl.cpp:

(WebCore::DragController::isCopyKeyDown):

  • page/gtk/DragControllerGtk.cpp:

(WebCore::DragController::isCopyKeyDown):

  • page/haiku/DragControllerHaiku.cpp:

(WebCore::DragController::isCopyKeyDown):

  • page/mac/DragControllerMac.mm:

(WebCore::DragController::isCopyKeyDown):
(WebCore::DragController::dragOperation):

  • page/qt/DragControllerQt.cpp:

(WebCore::DragController::isCopyKeyDown):

  • page/win/DragControllerWin.cpp:

(WebCore::DragController::isCopyKeyDown):

  • page/wx/DragControllerWx.cpp:

(WebCore::DragController::isCopyKeyDown):

WebKit/mac: WebKit2: Add support for drag and drop
https://bugs.webkit.org/show_bug.cgi?id=52343
<rdar://problem/7660558>

Reviewed by Darin Adler.

The DragData class has been extended to provide
additional context from the application (keyboard state, modal windows, etc.)
as well as information of the drag pasteboard being used.
These are the changes to align the behavior for WebKit.

  • WebView/WebView.mm:

(-[WebView applicationFlags:]): Added.
(-[WebView draggingEntered:]): Added parameter to the DragData constructor.
(-[WebView draggingUpdated:]): Added parameter to the DragData constructor.
(-[WebView draggingExited:]): Added parameter to the DragData constructor.
(-[WebView performDragOperation:]): Added parameter to the DragData constructor.

WebKit2: WebKit2: Add support for drag and drop
https://bugs.webkit.org/show_bug.cgi?id=52343
<rdar://problem/7660558>

Reviewed by Darin Adler.

This patch contains the changes required to support dropping content
in WebKit on the Mac. The DragData class has been extended to provide
additional context from the application (keyboard state, modal windows, etc.)
as well as information of the drag pasteboard being used.
The support for WebKit as drag source will be added with a separate patch.

  • Shared/DragControllerAction.h: Added.
  • UIProcess/API/mac/WKView.mm:

Added implemention of the methods required to add suport for a drop target.
To maintain asynchronous communication with the WebProcess, we always return
the previous calculated value for the drag operation.
(-[WKView _registerDraggedTypes]):
(-[WKView initWithFrame:contextRef:pageGroupRef:]):
(-[WKView applicationFlags:]):
(-[WKView draggingEntered:]):
(-[WKView draggingUpdated:]):
(-[WKView draggingExited:]):
(-[WKView prepareForDragOperation:]):
(-[WKView performDragOperation:]):

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::WebPageProxy):
(WebKit::WebPageProxy::performDragControllerAction):
(WebKit::WebPageProxy::didPerformDragControllerAction):

  • UIProcess/WebPageProxy.h:

(WebKit::WebPageProxy::dragOperation):
(WebKit::WebPageProxy::resetDragOperation):

  • UIProcess/WebPageProxy.messages.in:
  • WebKit2.xcodeproj/project.pbxproj:
  • WebProcess/WebCoreSupport/WebDragClient.cpp:

(WebKit::WebDragClient::willPerformDragDestinationAction):
(WebKit::WebDragClient::willPerformDragSourceAction):
(WebKit::WebDragClient::actionMaskForDrag):
(WebKit::WebDragClient::dragSourceActionMaskForPoint):
(WebKit::WebDragClient::startDrag):

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::performDragControllerAction):

  • WebProcess/WebPage/WebPage.h:
  • WebProcess/WebPage/WebPage.messages.in:
15:11 Changeset [75742] by ddkilzer@apple.com
  • 2 edits in trunk/Source/WebCore

Add missing MathMLElementFactory.h and MathMLNames.h to Xcode project

Not reviewed.

  • WebCore.xcodeproj/project.pbxproj: Add missing headers. Note

that the corresponding source files were originally added to the
project in r48559.

15:05 Changeset [75741] by zmo@google.com
  • 17 edits in trunk/Source/WebCore

2011-01-12 Zhenyao Mo <zmo@google.com>

Reviewed by Kenneth Russell.

Use GC3D types in WebGLRenderingContext and related WebGL classes
https://bugs.webkit.org/show_bug.cgi?id=51908

  • html/canvas/WebGLActiveInfo.h: (WebCore::WebGLActiveInfo::create): (WebCore::WebGLActiveInfo::type): (WebCore::WebGLActiveInfo::size): (WebCore::WebGLActiveInfo::WebGLActiveInfo):
  • html/canvas/WebGLBuffer.cpp: (WebCore::WebGLBuffer::associateBufferDataImpl): (WebCore::WebGLBuffer::associateBufferData): (WebCore::WebGLBuffer::associateBufferSubDataImpl): (WebCore::WebGLBuffer::associateBufferSubData): (WebCore::WebGLBuffer::byteLength): (WebCore::WebGLBuffer::getCachedMaxIndex): (WebCore::WebGLBuffer::setCachedMaxIndex): (WebCore::WebGLBuffer::setTarget):
  • html/canvas/WebGLBuffer.h: (WebCore::WebGLBuffer::getTarget):
  • html/canvas/WebGLFramebuffer.cpp: (WebCore::WebGLFramebuffer::setAttachment): (WebCore::WebGLFramebuffer::getAttachment): (WebCore::WebGLFramebuffer::getWidth): (WebCore::WebGLFramebuffer::getHeight): (WebCore::WebGLFramebuffer::getColorBufferFormat): (WebCore::WebGLFramebuffer::initializeRenderbuffers):
  • html/canvas/WebGLFramebuffer.h:
  • html/canvas/WebGLProgram.cpp: (WebCore::WebGLProgram::cacheActiveAttribLocations): (WebCore::WebGLProgram::numActiveAttribLocations): (WebCore::WebGLProgram::getActiveAttribLocation): (WebCore::WebGLProgram::isUsingVertexAttrib0):
  • html/canvas/WebGLProgram.h: (WebCore::WebGLProgram::getLinkCount):
  • html/canvas/WebGLRenderbuffer.h: (WebCore::WebGLRenderbuffer::setInternalFormat): (WebCore::WebGLRenderbuffer::getInternalFormat): (WebCore::WebGLRenderbuffer::setSize): (WebCore::WebGLRenderbuffer::getWidth): (WebCore::WebGLRenderbuffer::getHeight):
  • html/canvas/WebGLRenderingContext.cpp: (WebCore::WebGLRenderingContext::initializeNewContext): (WebCore::WebGLRenderingContext::sizeInBytes): (WebCore::WebGLRenderingContext::activeTexture): (WebCore::WebGLRenderingContext::bindAttribLocation): (WebCore::WebGLRenderingContext::bindBuffer): (WebCore::WebGLRenderingContext::bindFramebuffer): (WebCore::WebGLRenderingContext::bindRenderbuffer): (WebCore::WebGLRenderingContext::bindTexture): (WebCore::WebGLRenderingContext::blendColor): (WebCore::WebGLRenderingContext::blendEquation): (WebCore::WebGLRenderingContext::blendEquationSeparate): (WebCore::WebGLRenderingContext::blendFunc): (WebCore::WebGLRenderingContext::blendFuncSeparate): (WebCore::WebGLRenderingContext::bufferData): (WebCore::WebGLRenderingContext::bufferSubData): (WebCore::WebGLRenderingContext::checkFramebufferStatus): (WebCore::WebGLRenderingContext::clear): (WebCore::WebGLRenderingContext::clearColor): (WebCore::WebGLRenderingContext::clearDepth): (WebCore::WebGLRenderingContext::clearStencil): (WebCore::WebGLRenderingContext::colorMask): (WebCore::WebGLRenderingContext::copyTexImage2D): (WebCore::WebGLRenderingContext::copyTexSubImage2D): (WebCore::WebGLRenderingContext::createShader): (WebCore::WebGLRenderingContext::cullFace): (WebCore::WebGLRenderingContext::deleteBuffer): (WebCore::WebGLRenderingContext::depthFunc): (WebCore::WebGLRenderingContext::depthMask): (WebCore::WebGLRenderingContext::depthRange): (WebCore::WebGLRenderingContext::disable): (WebCore::WebGLRenderingContext::disableVertexAttribArray): (WebCore::WebGLRenderingContext::validateElementArraySize): (WebCore::WebGLRenderingContext::validateIndexArrayConservative): (WebCore::WebGLRenderingContext::validateIndexArrayPrecise): (WebCore::WebGLRenderingContext::validateRenderingState): (WebCore::WebGLRenderingContext::drawArrays): (WebCore::WebGLRenderingContext::drawElements): (WebCore::WebGLRenderingContext::enable): (WebCore::WebGLRenderingContext::enableVertexAttribArray): (WebCore::WebGLRenderingContext::framebufferRenderbuffer): (WebCore::WebGLRenderingContext::framebufferTexture2D): (WebCore::WebGLRenderingContext::frontFace): (WebCore::WebGLRenderingContext::generateMipmap): (WebCore::WebGLRenderingContext::getActiveAttrib): (WebCore::WebGLRenderingContext::getActiveUniform): (WebCore::WebGLRenderingContext::getAttachedShaders): (WebCore::WebGLRenderingContext::getAttribLocation): (WebCore::WebGLRenderingContext::getBufferParameter): (WebCore::WebGLRenderingContext::getError): (WebCore::WebGLRenderingContext::getFramebufferAttachmentParameter): (WebCore::WebGLRenderingContext::getParameter): (WebCore::WebGLRenderingContext::getProgramParameter): (WebCore::WebGLRenderingContext::getRenderbufferParameter): (WebCore::WebGLRenderingContext::getShaderParameter): (WebCore::WebGLRenderingContext::getTexParameter): (WebCore::WebGLRenderingContext::getUniform): (WebCore::WebGLRenderingContext::getUniformLocation): (WebCore::WebGLRenderingContext::getVertexAttrib): (WebCore::WebGLRenderingContext::getVertexAttribOffset): (WebCore::WebGLRenderingContext::hint): (WebCore::WebGLRenderingContext::isBuffer): (WebCore::WebGLRenderingContext::isEnabled): (WebCore::WebGLRenderingContext::isFramebuffer): (WebCore::WebGLRenderingContext::isProgram): (WebCore::WebGLRenderingContext::isRenderbuffer): (WebCore::WebGLRenderingContext::isShader): (WebCore::WebGLRenderingContext::isTexture): (WebCore::WebGLRenderingContext::lineWidth): (WebCore::WebGLRenderingContext::linkProgram): (WebCore::WebGLRenderingContext::pixelStorei): (WebCore::WebGLRenderingContext::polygonOffset): (WebCore::WebGLRenderingContext::readPixels): (WebCore::WebGLRenderingContext::renderbufferStorage): (WebCore::WebGLRenderingContext::sampleCoverage): (WebCore::WebGLRenderingContext::scissor): (WebCore::WebGLRenderingContext::stencilFunc): (WebCore::WebGLRenderingContext::stencilFuncSeparate): (WebCore::WebGLRenderingContext::stencilMask): (WebCore::WebGLRenderingContext::stencilMaskSeparate): (WebCore::WebGLRenderingContext::stencilOp): (WebCore::WebGLRenderingContext::stencilOpSeparate): (WebCore::WebGLRenderingContext::texImage2DBase): (WebCore::WebGLRenderingContext::texImage2DImpl): (WebCore::WebGLRenderingContext::texImage2D): (WebCore::WebGLRenderingContext::texParameter): (WebCore::WebGLRenderingContext::texParameterf): (WebCore::WebGLRenderingContext::texParameteri): (WebCore::WebGLRenderingContext::texSubImage2DBase): (WebCore::WebGLRenderingContext::texSubImage2DImpl): (WebCore::WebGLRenderingContext::texSubImage2D): (WebCore::WebGLRenderingContext::uniform1f): (WebCore::WebGLRenderingContext::uniform1fv): (WebCore::WebGLRenderingContext::uniform1i): (WebCore::WebGLRenderingContext::uniform1iv): (WebCore::WebGLRenderingContext::uniform2f): (WebCore::WebGLRenderingContext::uniform2fv): (WebCore::WebGLRenderingContext::uniform2i): (WebCore::WebGLRenderingContext::uniform2iv): (WebCore::WebGLRenderingContext::uniform3f): (WebCore::WebGLRenderingContext::uniform3fv): (WebCore::WebGLRenderingContext::uniform3i): (WebCore::WebGLRenderingContext::uniform3iv): (WebCore::WebGLRenderingContext::uniform4f): (WebCore::WebGLRenderingContext::uniform4fv): (WebCore::WebGLRenderingContext::uniform4i): (WebCore::WebGLRenderingContext::uniform4iv): (WebCore::WebGLRenderingContext::uniformMatrix2fv): (WebCore::WebGLRenderingContext::uniformMatrix3fv): (WebCore::WebGLRenderingContext::uniformMatrix4fv): (WebCore::WebGLRenderingContext::vertexAttrib1f): (WebCore::WebGLRenderingContext::vertexAttrib1fv): (WebCore::WebGLRenderingContext::vertexAttrib2f): (WebCore::WebGLRenderingContext::vertexAttrib2fv): (WebCore::WebGLRenderingContext::vertexAttrib3f): (WebCore::WebGLRenderingContext::vertexAttrib3fv): (WebCore::WebGLRenderingContext::vertexAttrib4f): (WebCore::WebGLRenderingContext::vertexAttrib4fv): (WebCore::WebGLRenderingContext::vertexAttribPointer): (WebCore::WebGLRenderingContext::viewport): (WebCore::WebGLRenderingContext::getBooleanParameter): (WebCore::WebGLRenderingContext::getBooleanArrayParameter): (WebCore::WebGLRenderingContext::getFloatParameter): (WebCore::WebGLRenderingContext::getIntParameter): (WebCore::WebGLRenderingContext::getLongParameter): (WebCore::WebGLRenderingContext::getUnsignedLongParameter): (WebCore::WebGLRenderingContext::getWebGLFloatArrayParameter): (WebCore::WebGLRenderingContext::getWebGLIntArrayParameter): (WebCore::WebGLRenderingContext::isTexInternalFormatColorBufferCombinationValid): (WebCore::WebGLRenderingContext::getBoundFramebufferColorFormat): (WebCore::WebGLRenderingContext::validateTextureBinding): (WebCore::WebGLRenderingContext::validateSize): (WebCore::WebGLRenderingContext::validateTexFuncFormatAndType): (WebCore::WebGLRenderingContext::validateTexFuncLevel): (WebCore::WebGLRenderingContext::validateTexFuncParameters): (WebCore::WebGLRenderingContext::validateTexFuncData): (WebCore::WebGLRenderingContext::validateDrawMode): (WebCore::WebGLRenderingContext::validateStencilFunc): (WebCore::WebGLRenderingContext::validateFramebufferFuncParameters): (WebCore::WebGLRenderingContext::validateBlendEquation): (WebCore::WebGLRenderingContext::validateBlendFuncFactors): (WebCore::WebGLRenderingContext::validateCapability): (WebCore::WebGLRenderingContext::validateUniformParameters): (WebCore::WebGLRenderingContext::validateUniformMatrixParameters): (WebCore::WebGLRenderingContext::validateBufferDataParameters): (WebCore::WebGLRenderingContext::vertexAttribfImpl): (WebCore::WebGLRenderingContext::vertexAttribfvImpl): (WebCore::WebGLRenderingContext::simulateVertexAttrib0):
  • html/canvas/WebGLRenderingContext.h: (WebCore::WebGLRenderingContext::VertexAttribState::VertexAttribState):
  • html/canvas/WebGLShader.cpp: (WebCore::WebGLShader::create): (WebCore::WebGLShader::WebGLShader):
  • html/canvas/WebGLShader.h: (WebCore::WebGLShader::getType):
  • html/canvas/WebGLTexture.cpp: (WebCore::WebGLTexture::setTarget): (WebCore::WebGLTexture::setParameteri): (WebCore::WebGLTexture::setParameterf): (WebCore::WebGLTexture::setLevelInfo): (WebCore::WebGLTexture::generateMipmapLevelInfo): (WebCore::WebGLTexture::getInternalFormat): (WebCore::WebGLTexture::getType): (WebCore::WebGLTexture::getWidth): (WebCore::WebGLTexture::getHeight): (WebCore::WebGLTexture::isNPOT): (WebCore::WebGLTexture::mapTargetToIndex): (WebCore::WebGLTexture::computeLevelCount): (WebCore::WebGLTexture::update): (WebCore::WebGLTexture::getLevelInfo):
  • html/canvas/WebGLTexture.h: (WebCore::WebGLTexture::LevelInfo::setInfo):
  • html/canvas/WebGLUniformLocation.cpp: (WebCore::WebGLUniformLocation::create): (WebCore::WebGLUniformLocation::WebGLUniformLocation): (WebCore::WebGLUniformLocation::location):
  • html/canvas/WebGLUniformLocation.h:
  • platform/graphics/GraphicsContext3D.h: Map GC3Dboolean to bool instead of unsigned char.
15:00 Changeset [75740] by mrobinson@webkit.org
  • 2 edits in trunk/LayoutTests

2011-01-13 Martin Robinson <mrobinson@igalia.com>

Unskip editing/input/page-up-down-scrolls.html as it's passing locally.

  • platform/gtk/Skipped: Unskip test.
14:55 Changeset [75739] by jianli@chromium.org
  • 33 edits
    3 adds in trunk

Change createObjectURL and revokeObjectURL to put under webkitURL.
https://bugs.webkit.org/show_bug.cgi?id=52257

Reviewed by Darin Fisher.

Source/WebCore:

Note that we add "webkit" vendor prefix to URL that is introduced in
the latest File API spec.

For the time being, we implement webkitURL as a readonly attribute
instead of a constructor so that we will not expose prototype property,
as required by the spec.

  • Android.derived.jscbindings.mk:
  • Android.derived.v8bindings.mk:
  • Android.mk:
  • CMakeLists.txt:
  • DerivedSources.cpp:
  • DerivedSources.make:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pri:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • html/DOMURL.cpp: Added.
  • html/DOMURL.h: Added.
  • html/DOMURL.idl: Added.
  • inspector/front-end/NetworkPanel.js:
  • inspector/front-end/utilities.js:
  • page/DOMWindow.cpp:

(WebCore::DOMWindow::webkitURL):

  • page/DOMWindow.h:
  • page/DOMWindow.idl:
  • workers/WorkerContext.cpp:

(WebCore::WorkerContext::webkitURL):

  • workers/WorkerContext.h:
  • workers/WorkerContext.idl:

LayoutTests:

Change all related test scripts and results to account for this change.

  • fast/dom/Window/script-tests/window-property-descriptors.js:
  • fast/dom/Window/window-properties-expected.txt:
  • fast/dom/Window/window-properties.html:
  • fast/dom/script-tests/prototype-inheritance-2.js:
  • fast/dom/script-tests/prototype-inheritance.js:
  • fast/files/apply-blob-url-to-img.html:
  • fast/files/apply-blob-url-to-xhr.html:
  • fast/files/create-blob-url-crash.html:
  • fast/files/revoke-blob-url.html:
  • fast/files/workers/resources/worker-apply-blob-url-to-xhr.js:

(onmessage):

  • platform/qt/fast/dom/Window/window-properties-expected.txt:
14:49 Changeset [75738] by commit-queue@webkit.org
  • 6 edits
    2 adds in trunk

2011-01-13 Emil Eklund <eae@chromium.org>

Reviewed by Dimitri Glazkov.

Setting outerText should convert CR/LF to <br>
https://bugs.webkit.org/show_bug.cgi?id=52268

Add test for setting Element.outerText.

  • fast/dom/set-outer-text-expected.txt: Added.
  • fast/dom/set-outer-text.html: Added.
  • fast/dom/text-node-append-data-remove-crash-expected.txt: Changed

expectation to "didn't crash" from "threw dom exception" as test
assumed the DOMCharacterDataModified event would fire before the node
would be replaced.

  • fast/dom/text-node-append-data-remove-crash.html:

2011-01-13 Emil Eklund <eae@chromium.org>

Reviewed by Dimitri Glazkov.

Setting outerText should convert CR/LF to <br>
https://bugs.webkit.org/show_bug.cgi?id=52268

Make set outerText support line breaks (sharing the text to fragment code
with setInnerText) and handle empty text nodes the same way IE does.

Test: fast/dom/set-outer-text.html

  • html/HTMLElement.cpp: (WebCore::HTMLElement::textToFragment): Shared between setInnerText and setOuterText (WebCore::HTMLElement::setInnerText): Split out text parsing code. (WebCore::mergeWithNextTextNode): Split out text node merging code. (WebCore::HTMLElement::setOuterText): Added support for line breaks.
  • html/HTMLElement.h:
14:39 Changeset [75737] by aroben@apple.com
  • 2 edits in trunk/LayoutTests

Skip a crashing test on Windows

See <http://webkit.org/b/52397>.

  • platform/win/Skipped: Added

compositing/reflections/load-video-in-reflection.html.

14:35 Changeset [75736] by mrobinson@webkit.org
  • 2 edits in trunk/LayoutTests

2011-01-13 Martin Robinson <mrobinson@igalia.com>

Skip fast/text/emphasis.html as it shows a 1 pixel difference on the 32-bit Debug bot.

  • platform/gtk/Skipped: Skip fast/text/emphasis.html.
14:34 Changeset [75735] by zmo@google.com
  • 5 edits in trunk

2011-01-13 Zhenyao Mo <zmo@google.com>

Reviewed by Kenneth Russell.

Relax the ESSL valid charset a little bit to include " ' `
https://bugs.webkit.org/show_bug.cgi?id=52396

  • html/canvas/WebGLRenderingContext.cpp:

2011-01-13 Zhenyao Mo <zmo@google.com>

Reviewed by Kenneth Russell.

Relax the ESSL valid charset a little bit to include " ' `
https://bugs.webkit.org/show_bug.cgi?id=52396

  • fast/canvas/webgl/invalid-passed-params-expected.txt:
  • fast/canvas/webgl/invalid-passed-params.html:
13:52 Changeset [75734] by jam@chromium.org
  • 2 edits in trunk/WebKit/chromium

2011-01-13 John Abd-El-Malek <jam@chromium.org>

Reviewed by Tony Chang.

[chromium]: Only swallow ctrl+c for windowless plugins if we know the plugin has a selection
https://bugs.webkit.org/show_bug.cgi?id=52393

  • src/WebPluginContainerImpl.cpp: (WebKit::WebPluginContainerImpl::copy): (WebKit::WebPluginContainerImpl::handleKeyboardEvent):
12:51 Changeset [75733] by enne@google.com
  • 2 edits in trunk/Source/WebCore

2011-01-13 Adrienne Walker <enne@google.com>

Reviewed by Kenneth Russell.

[chromium] Attempt to fix crash in tiled compositor memcpy
https://bugs.webkit.org/show_bug.cgi?id=52379

  • platform/graphics/chromium/LayerTilerChromium.cpp: (WebCore::LayerTilerChromium::update):
12:36 Changeset [75732] by mdelaney@apple.com
  • 2 edits
    1 add in trunk/LayoutTests

2011-01-13 Matthew Delaney <mdelaney@apple.com>

Reviewed by nobody - reinstrumenting a test to save burning trees.

fast/canvas/canvas-large-dimensions.html (introduced in r75648) is failing on all platforms except Snow Leopard
https://bugs.webkit.org/show_bug.cgi?id=52341

  • fast/canvas/canvas-large-dimensions-expected.txt: Replaced.
  • fast/canvas/canvas-large-dimensions.html:
12:11 Changeset [75731] by jschuh@chromium.org
  • 2 edits in trunk/LayoutTests

2011-01-13 Justin Schuh <jschuh@chromium.org>

Unreviewed chromium expectations update.

Exclude destroy-on-setwindow.html (added in r75720).

  • platform/chromium/test_expectations.txt:
12:07 Changeset [75730] by jam@chromium.org
  • 6 edits in trunk/WebKit/chromium

2011-01-11 John Abd-El-Malek <jam@chromium.org>

Reviewed by Darin Fisher.

[chromium] Add support to allow printing just a plugin in a frame
https://bugs.webkit.org/show_bug.cgi?id=52134

  • public/WebContextMenuData.h:
  • public/WebFrame.h:
  • src/ContextMenuClientImpl.cpp: (WebKit::ContextMenuClientImpl::getCustomMenuFromDefaultItems):
  • src/WebFrameImpl.cpp: (WebKit::ChromePluginPrintContext::ChromePluginPrintContext): (WebKit::ChromePluginPrintContext::end): (WebKit::ChromePluginPrintContext::computePageRects): (WebKit::ChromePluginPrintContext::spoolPage): (WebKit::WebFrameImpl::printBegin):
11:48 Changeset [75729] by alex@webkit.org
  • 6 edits in trunk

2011-01-13 Alejandro G. Castro <alex@igalia.com>

Reviewed by Martin Robinson.

[gtk] Zero-sized font does not yet work
https://bugs.webkit.org/show_bug.cgi?id=49793

Unskip font-size-zero.html.

  • platform/gtk/Skipped:

2011-01-13 Alejandro G. Castro <alex@igalia.com>

Reviewed by Martin Robinson.

[gtk] Zero-sized font does not yet work
https://bugs.webkit.org/show_bug.cgi?id=49793

Set minimum-font-size to 0 for the DRT.

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (resetDefaultsToConsistentValues):

2011-01-13 Alejandro G. Castro <alex@igalia.com>

Reviewed by Martin Robinson.

[gtk] Zero-sized font does not yet work
https://bugs.webkit.org/show_bug.cgi?id=49793

Allow 0 as a minimum-font-size value for the settings.

  • webkit/webkitwebsettings.cpp: (webkit_web_settings_class_init):
11:38 Changeset [75728] by dglazkov@chromium.org
  • 12 edits in trunk/Source/WebCore

Unreviewed, rolling out r75725.
http://trac.webkit.org/changeset/75725
https://bugs.webkit.org/show_bug.cgi?id=52317

Caused crashes in layout tests.

  • css/CSSStyleSelector.cpp:

(WebCore::CSSStyleSelector::initForStyleResolve):

  • html/HTMLInputElement.cpp:

(WebCore::HTMLInputElement::updateType):

  • html/InputType.cpp:
  • html/InputType.h:
  • html/RangeInputType.cpp:
  • html/RangeInputType.h:
  • html/shadow/SliderThumbElement.cpp:

(WebCore::SliderThumbElement::defaultEventHandler):
(WebCore::SliderThumbElement::detach):

  • html/shadow/SliderThumbElement.h:

(WebCore::SliderThumbElement::SliderThumbElement):
(WebCore::SliderThumbElement::create):

  • rendering/MediaControlElements.cpp:

(WebCore::MediaControlInputElement::attach):
(WebCore::MediaControlInputElement::updateStyle):

  • rendering/RenderSlider.cpp:

(WebCore::RenderSlider::~RenderSlider):
(WebCore::RenderSlider::styleDidChange):
(WebCore::RenderSlider::createThumbStyle):
(WebCore::RenderSlider::thumbRect):
(WebCore::RenderSlider::layout):
(WebCore::RenderSlider::updateFromElement):
(WebCore::RenderSlider::mouseEventIsInThumb):
(WebCore::RenderSlider::mouseEventOffsetToThumb):
(WebCore::RenderSlider::setValueForPosition):
(WebCore::RenderSlider::positionForOffset):
(WebCore::RenderSlider::currentPosition):
(WebCore::RenderSlider::trackSize):
(WebCore::RenderSlider::forwardEvent):
(WebCore::RenderSlider::inDragMode):

  • rendering/RenderSlider.h:
10:56 Changeset [75727] by tony@chromium.org
  • 2 edits in trunk/LayoutTests

2011-01-13 Tony Chang <tony@chromium.org>

Unreviewed, updating chromium test_expectations.txt

  • platform/chromium/test_expectations.txt: Mark a test as slow in debug.
10:37 Changeset [75726] by ossy@webkit.org
  • 2 edits in trunk/Tools

2011-01-13 Csaba Osztrogonác <ossy@webkit.org>

Reviewed by Darin Adler.

Make old-run-webkit-tests more robust on buildbots
https://bugs.webkit.org/show_bug.cgi?id=52364

  • BuildSlaveSupport/build.webkit.org-config/master.cfg: Pass --exit-after-n-failures 500 to old-run-webkit-tests script.
10:35 Changeset [75725] by dglazkov@chromium.org
  • 12 edits in trunk/Source/WebCore

2011-01-12 Dimitri Glazkov <dglazkov@chromium.org>

Reviewed by Darin Adler.

Flip input[type=range] to use the new shadow DOM model.
https://bugs.webkit.org/show_bug.cgi?id=52317

This is a straight-up flip, minimal change to the logic, which will be
refactored in a follow-up patch.

Covered by existing tests.

  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::initForStyleResolve): Changed to use either

host or parent node to find parent.

  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::updateType): Added logic for destroying/creating

shadow DOM subtree.

  • html/InputType.cpp: (WebCore::InputType::createShadowSubtree): Added empty decl. (WebCore::InputType::destroyShadowSubtree): Added a method to destroy

shadow subtree.

  • html/InputType.h: Added defs.
  • html/RangeInputType.cpp: (WebCore::RangeInputType::createShadowSubtree): Added a virtual method to create

slider thumb.

  • html/RangeInputType.h: Adde def.
  • html/shadow/SliderThumbElement.cpp: Added temporary RenderSliderThumb class

to handle cascading appearance.

(WebCore::RenderSliderThumb::RenderSliderThumb): Added.
(WebCore::RenderSliderThumb::layout): Moved here the logic from

RenderSlider::createThumbStyle.

(WebCore::SliderThumbElement::createRenderer): Added virtual method to

create RenderSliderThumb instance.

(WebCore::SliderThumbElement::defaultEventHandler): Changed to use HTMLDivElement

as base class.

(WebCore::SliderThumbElement::detach): Ditto.

  • html/shadow/SliderThumbElement.h: (WebCore::SliderThumbElement::SliderThumbElement): Ditto. (WebCore::SliderThumbElement::create): Ditto. (WebCore::toSliderThumbElement): Added a casting helper.
  • rendering/MediaControlElements.cpp: (WebCore::MediaControlInputElement::attach): Added handling of shadow DOM,

since the whole method is hand-rolled.

(WebCore::MediaControlInputElement::updateStyle): Ditto.

  • rendering/RenderSlider.cpp: (WebCore::RenderSlider::~RenderSlider): Removed unneeded code. (WebCore::RenderSlider::thumbRect): Changed to use sliderThumbElement helper. (WebCore::RenderSlider::layout): Ditto. (WebCore::RenderSlider::sliderThumbElement): Added a temporary helper to access

SliderThumbElement.

(WebCore::RenderSlider::mouseEventIsInThumb): Changed to use sliderThumbElement helper.
(WebCore::RenderSlider::mouseEventOffsetToThumb): Ditto.
(WebCore::RenderSlider::setValueForPosition): Ditto.
(WebCore::RenderSlider::positionForOffset): Ditto.
(WebCore::RenderSlider::currentPosition): Ditto.
(WebCore::RenderSlider::trackSize): Ditto.
(WebCore::RenderSlider::forwardEvent): Ditto.
(WebCore::RenderSlider::inDragMode): Ditto.

  • rendering/RenderSlider.h: Added def.
10:34 Changeset [75724] by aroben@apple.com
  • 1 edit
    2 adds in trunk/LayoutTests

Check in Windows-XP-specific results for fast/canvas/set-colors.html

See <http://webkit.org/b/52374>.

  • platform/win-xp/fast/canvas/set-colors-expected.txt: Added.
10:33 Changeset [75723] by aroben@apple.com
  • 4 edits
    1 move
    1 delete in trunk/LayoutTests

Disable fast/profiler/throw-exception-from-eval.html

This test fails non-deterministically. See <http://webkit.org/b/49801>.

  • fast/profiler/throw-exception-from-eval.html-disabled: Renamed from LayoutTests/fast/profiler/throw-exception-from-eval.html.
  • platform/gtk/Skipped:
  • platform/mac-leopard/Skipped:
  • platform/qt/Skipped:

Un-skipped this test now that it's been disabled by other means.

  • platform/win-xp/fast/profiler/throw-exception-from-eval-expected.txt: Removed.
10:24 Changeset [75722] by mitz@apple.com
  • 2 edits in trunk/WebKit2

Incorrect assertion in DrawingAreaImpl::scroll()
https://bugs.webkit.org/show_bug.cgi?id=52376

Reviewed by John Sullivan.

  • WebProcess/WebPage/DrawingAreaImpl.cpp:

(WebKit::DrawingAreaImpl::scroll): Changed to take the fast path if the rects have the same area.
This made the assertion valid, but I removed it anyway.

10:22 Changeset [75721] by ossy@webkit.org
  • 2 edits in trunk/Tools

2011-01-13 Csaba Osztrogonác <ossy@webkit.org>

Reviewed by Darin Adler.

Make old-run-webkit-tests more robust on buildbots
https://bugs.webkit.org/show_bug.cgi?id=52364

  • Scripts/old-run-webkit-tests: Ensure that testResultsDirectory is empty.
10:21 Changeset [75720] by mitz@apple.com
  • 4 edits
    2 adds in trunk

<rdar://problem/8098442> Crash in Widget::setFrameRect()
https://bugs.webkit.org/show_bug.cgi?id=52375

Reviewed by Simon Fraser.

Source/WebCore:

Test: plugins/destroy-on-setwindow.html

  • platform/mac/WidgetMac.mm:

(WebCore::Widget::setFrameRect): Protect the Widget from being deleted as a result of calling
out to the view.
(WebCore::Widget::getOuterView): Coding style fix.
(WebCore::Widget::paint): Protect the Widget from being deleted as a result of calling out to
the views. This is precautionary, since this function does not applear to access member variables
after such calls.

  • platform/wx/WidgetWx.cpp:

(WebCore::Widget::setFrameRect): Protect the Widget from being deleted as a result of calling
out to the platform widget.

LayoutTests:

  • plugins/destroy-on-setwindow-expected.txt: Added.
  • plugins/destroy-on-setwindow.html: Added.
09:52 Changeset [75719] by tony@chromium.org
  • 6 edits
    1 delete in trunk

2011-01-13 Tony Chang <tony@chromium.org>

Reviewed by Kent Tamura.

[chromium] fix plugins/plugin-javascript-access.html on Linux and Mac
https://bugs.webkit.org/show_bug.cgi?id=52332

Fix Linux baseline. Mac should now match Safari Mac.

  • platform/chromium-linux/plugins/plugin-javascript-access-expected.txt:
  • platform/chromium/test_expectations.txt:

2011-01-13 Tony Chang <tony@chromium.org>

Reviewed by Kent Tamura.

[chromium] fix plugins/plugin-javascript-access.html on Linux and Mac
https://bugs.webkit.org/show_bug.cgi?id=52332

Forked Info.plist is not longer needed.

  • DumpRenderTree/chromium/TestNetscapePlugIn/Info.plist: Removed.

2011-01-13 Tony Chang <tony@chromium.org>

Reviewed by Kent Tamura.

[chromium] fix plugins/plugin-javascript-access.html on Linux and Mac
https://bugs.webkit.org/show_bug.cgi?id=52332

  • WebKit.gyp: Remove WebKitTestNetscapePlugIn
09:25 Changeset [75718] by ossy@webkit.org
  • 3 edits in trunk/Source/WebCore

Unreviewed buildfix after r75715.

  • WebCore.pri:
  • WebCore.pro:
09:23 Changeset [75717] by pfeldman@chromium.org
  • 3 edits in trunk/LayoutTests

2011-01-13 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: inspector extensions tests should not use
sources as test data. Commenting one out temporarily, filing a bug.

  • inspector/extensions-events-expected.txt:
  • inspector/extensions-events.html:
08:57 Changeset [75716] by pfeldman@chromium.org
  • 2 edits in trunk/Source/WebCore

2011-01-13 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: Qt build fix.

  • WebCore.pri:
08:42 Changeset [75715] by pfeldman@chromium.org
  • 22 edits
    1 move
    1 add in trunk

2011-01-12 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: embed injected script into the backend.
https://bugs.webkit.org/show_bug.cgi?id=52312

InjectedScript belongs to the backend of the system and
should be exposed by means of protocol, not InjectedScriptAccess.
This patch makes injected script a part of the backend.
It introduces single manual generation step (xxd -i) that should be
automated by the build scripts for all the platforms.

  • CMakeLists.txt:
  • DerivedSources.make:
  • GNUmakefile.am:
  • WebCore.gyp/WebCore.gyp:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSInjectedScriptHostCustom.cpp: (WebCore::InjectedScriptHost::injectedScriptFor):
  • bindings/v8/custom/V8InjectedScriptHostCustom.cpp: (WebCore::InjectedScriptHost::injectedScriptFor):
  • inspector/InjectedScriptHost.cpp: (WebCore::InjectedScriptHost::injectedScriptSource):
  • inspector/InjectedScriptHost.h:
  • inspector/Inspector.idl:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::restoreInspectorStateFromCookie):
  • inspector/InspectorState.cpp: (WebCore::InspectorState::InspectorState):
  • inspector/InspectorState.h:
  • inspector/front-end/InjectedScript.js: Removed.
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.html:
  • inspector/front-end/inspector.js: (WebInspector.doLoadedDone.propertyNamesCallback): (WebInspector.doLoadedDone):
  • inspector/xxd.pl: Added.

2011-01-12 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: embed injected script into the backend.
https://bugs.webkit.org/show_bug.cgi?id=52312

  • WebKit.grd:
07:49 Changeset [75714] by mnaganov@chromium.org
  • 15 edits in trunk

2011-01-13 Mikhail Naganov <mnaganov@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Cleanup after r75613 -- make all UI-facing strings
to be passed directly to WebInspector.UIString

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

  • English.lproj/localizedStrings.js:
  • inspector/front-end/ApplicationCacheItemsView.js: (WebInspector.ApplicationCacheItemsView.prototype._updateCallback): (WebInspector.ApplicationCacheItemsView.prototype._populateDataGrid):
  • inspector/front-end/AuditRules.js: (WebInspector.AuditRules.GzipRule.prototype.doRun): (WebInspector.AuditRules.CookieSizeRule.prototype.processCookies): (WebInspector.AuditRules.StaticCookielessRule.prototype.processCookies):
  • inspector/front-end/BreakpointsSidebarPane.js: (WebInspector.EventListenerBreakpointsSidebarPane): (WebInspector.EventListenerBreakpointsSidebarPane.prototype._createCategory):
  • inspector/front-end/CookieItemsView.js: (WebInspector.CookieItemsView.prototype._updateWithCookies):
  • inspector/front-end/ImageView.js: (WebInspector.ImageView.prototype._createContentIfNeeded.onImageLoad): (WebInspector.ImageView.prototype._createContentIfNeeded):
  • inspector/front-end/NetworkPanel.js: (WebInspector.NetworkTimeCalculator.prototype.formatValue): (WebInspector.NetworkTransferTimeCalculator.prototype.formatValue): (WebInspector.NetworkTransferDurationCalculator.prototype.formatValue):
  • inspector/front-end/ProfileDataGridTree.js: (WebInspector.ProfileDataGridNode.prototype.get data.formatMilliseconds): (WebInspector.ProfileDataGridNode.prototype.get data):
  • inspector/front-end/Resource.js: (WebInspector.Resource.Type.toUIString): (WebInspector.Resource.Type.toString):
  • inspector/front-end/TimelineOverviewPane.js: (WebInspector.TimelineOverviewCalculator.prototype.formatValue):
  • inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel): (WebInspector.TimelineCalculator.prototype.formatValue): (WebInspector.TimelinePanel.FormattedRecord.prototype._generateAggregatedInfo): (WebInspector.TimelinePanel.FormattedRecord.prototype._generatePopupContent): (WebInspector.TimelinePanel.FormattedRecord.prototype._getRecordDetails):
  • inspector/front-end/utilities.js: ():
  • Scripts/check-inspector-strings:
07:45 Changeset [75713] by commit-queue@webkit.org
  • 5 edits in trunk

2011-01-13 Viatcheslav Ostapenko <ostapenko.viatcheslav@nokia.com>

Reviewed by Simon Hausmann.

[Qt] Need a way to inform the application when a Netscape plugin is created.
https://bugs.webkit.org/show_bug.cgi?id=30179

Added "c" style static method for the application to hook up for
receiving notifications when a plugin is created.

  • plugins/symbian/PluginViewSymbian.cpp: (qtwebkit_setPluginCreatedCallback): (WebCore::PluginView::platformStart):

2011-01-13 Viatcheslav Ostapenko <ostapenko.viatcheslav@nokia.com>

Reviewed by Simon Hausmann.

[Qt] Need a way to inform the application when a Netscape plugin is created.
https://bugs.webkit.org/show_bug.cgi?id=30179

Added "c" style static method for the application to hook up for
receiving notifications when a plugin is created.

  • symbian/bwins/QtWebKitu.def:
  • symbian/eabi/QtWebKitu.def:
07:29 Changeset [75712] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-01-13 Yi Shen <yi.4.shen@nokia.com>

Reviewed by Eric Seidel.

[Qt] Use a black background color for the video element on Qt Mobile
https://bugs.webkit.org/show_bug.cgi?id=52252

Use black as the background color for the video element on Qt Mobile.

  • css/themeQtMobile.css: (video):
07:12 Changeset [75711] by commit-queue@webkit.org
  • 3 edits
    1 add in trunk/Source/WebCore

2011-01-13 Yi Shen <yi.4.shen@nokia.com>

Reviewed by Eric Seidel.

[Qt] Use a big central media play control button for video element
https://bugs.webkit.org/show_bug.cgi?id=52315

When ChromeClient::requiresFullscreenForVideoPlayback() returns true,
all other control buttons for the video element are hidden, except
a big centralized play button.

No new tests because no client implements requiresFullscreenForVideoPlayback yet.

  • WebCore.pri:
  • css/mediaControlsQtFullscreen.css: Added. (audio): (audio::-webkit-media-controls-panel): (video::-webkit-media-controls-panel): (video:-webkit-full-page-media::-webkit-media-controls-panel): (audio::-webkit-media-controls-mute-button): (video::-webkit-media-controls-mute-button): (audio::-webkit-media-controls-play-button): (video::-webkit-media-controls-play-button): (audio::-webkit-media-controls-timeline-container): (video::-webkit-media-controls-timeline-container): (audio::-webkit-media-controls-current-time-display): (video::-webkit-media-controls-current-time-display): (audio::-webkit-media-controls-time-remaining-display, video::-webkit-media-controls-time-remaining-display): (audio::-webkit-media-controls-timeline): (video::-webkit-media-controls-timeline): (audio::-webkit-media-controls-volume-slider-container): (video::-webkit-media-controls-volume-slider-container): (audio::-webkit-media-controls-volume-slider): (video::-webkit-media-controls-volume-slider): (audio::-webkit-media-controls-seek-back-button, video::-webkit-media-controls-seek-back-button): (audio::-webkit-media-controls-seek-forward-button, video::-webkit-media-controls-seek-forward-button): (audio::-webkit-media-controls-fullscreen-button): (video::-webkit-media-controls-fullscreen-button): (audio::-webkit-media-controls-rewind-button, video::-webkit-media-controls-rewind-button): (audio::-webkit-media-controls-return-to-realtime-button, video::-webkit-media-controls-return-to-realtime-button): (audio::-webkit-media-controls-toggle-closed-captions-button, video::-webkit-media-controls-toggle-closed-captions-button):
  • platform/qt/RenderThemeQt.cpp: (WebCore::RenderThemeQt::extraMediaControlsStyleSheet):
04:07 Changeset [75710] by carlosgc@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-01-13 Carlos Garcia Campos <cgarcia@igalia.com>

Reviewed by Xan Lopez.

[GTK] Paint toggle buttons inconsistent when the object state is indeterminate
https://bugs.webkit.org/show_bug.cgi?id=52361

Check whether the object state is indeterminate to set the flag
GTK_STATE_FLAG_INCONSISTENT before rendering check and radio
buttons.

  • platform/gtk/RenderThemeGtk3.cpp: (WebCore::paintToggle):
03:58 Changeset [75709] by xan@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

2011-01-13 Xan Lopez <xlopez@igalia.com>

Reviewed by Gavin Barraclough.

JIT requires VM overcommit (particularly on x86-64), Linux does not by default support this without swap?
https://bugs.webkit.org/show_bug.cgi?id=42756

The FixedVMPool Allocator does not work well on systems where
allocating very large amounts of memory upfront is not reasonable,
like Linux without overcommit enabled. As a workaround, on Linux,
default to the values used in embedded environments (in the MB
range), and only jump to the GB range if we detect at runtime that
overcommit is enabled. Should fix crashes on Linux/x86_64 with
less than 3 or 4GB of RAM.

  • jit/ExecutableAllocatorFixedVMPool.cpp: (JSC::FixedVMPoolAllocator::free): use new variables for VM pool size and coalesce limit. (JSC::ExecutableAllocator::isValid): swap the variables from embedded to generic values at runtime, on linux, if overcommit is enabled. (JSC::ExecutableAllocator::underMemoryPressure): use new variables for VM pool size and coalesce limit.
03:46 Changeset [75708] by commit-queue@webkit.org
  • 14 edits
    2 adds in trunk

2011-01-13 Yi Shen <yi.4.shen@nokia.com>

Reviewed by Simon Hausmann.

[Qt] Extend the Platform Plugin to support full screen video handler
https://bugs.webkit.org/show_bug.cgi?id=51249

Make MediaPlayerPrivateQt support a fullscreen player.

No new tests because LayoutTests/media/media-fullscreen-inline.html already exists.
However, this test failed for Qt (QtMediaPlayer) due to durationchange event getting fired twice.
So, still skip it for Qt.

  • WebCore.pro:
  • platform/graphics/qt/MediaPlayerPrivateQt.cpp: (WebCore::MediaPlayerPrivateQt::MediaPlayerPrivateQt): (WebCore::MediaPlayerPrivateQt::removeVideoItem): (WebCore::MediaPlayerPrivateQt::restoreVideoItem):
  • platform/graphics/qt/MediaPlayerPrivateQt.h: (WebCore::MediaPlayerPrivateQt::supportsFullscreen): (WebCore::MediaPlayerPrivateQt::mediaPlayer):

2011-01-13 Yi Shen <yi.4.shen@nokia.com>

Reviewed by Simon Hausmann.

[Qt] Extend the Platform Plugin to support full screen video handler
https://bugs.webkit.org/show_bug.cgi?id=51249

Add a new webkit platform plugin interface to support full screen video handler.

  • Api/qwebkitplatformplugin.h: (QWebFullScreenVideoHandler::QWebFullScreenVideoHandler): (QWebFullScreenVideoHandler::~QWebFullScreenVideoHandler):
  • WebCoreSupport/ChromeClientQt.cpp: (WebCore::ChromeClientQt::ChromeClientQt): (WebCore::ChromeClientQt::~ChromeClientQt): (WebCore::ChromeClientQt::fullScreenVideo): (WebCore::ChromeClientQt::supportsFullscreenForNode): (WebCore::ChromeClientQt::requiresFullscreenForVideoPlayback): (WebCore::ChromeClientQt::enterFullscreenForNode): (WebCore::ChromeClientQt::exitFullscreenForNode):
  • WebCoreSupport/ChromeClientQt.h:
  • WebCoreSupport/FullScreenVideoQt.cpp: Added. (WebCore::FullScreenVideoQt::FullScreenVideoQt): (WebCore::FullScreenVideoQt::~FullScreenVideoQt): (WebCore::FullScreenVideoQt::enterFullScreenForNode): (WebCore::FullScreenVideoQt::exitFullScreenForNode): (WebCore::FullScreenVideoQt::aboutToClose): (WebCore::FullScreenVideoQt::mediaPlayer): (WebCore::FullScreenVideoQt::mediaPlayerForNode): (WebCore::FullScreenVideoQt::requiresFullScreenForVideoPlayback):
  • WebCoreSupport/FullScreenVideoQt.h: Added. (WebCore::FullScreenVideoQt::isValid):
  • WebCoreSupport/QtPlatformPlugin.cpp: (WebCore::QtPlatformPlugin::createFullScreenVideoHandler):
  • WebCoreSupport/QtPlatformPlugin.h:
  • examples/platformplugin/WebPlugin.cpp: (FullScreenVideoWidget::FullScreenVideoWidget): (FullScreenVideoWidget::event): (FullScreenVideoWidget::keyPressEvent): (FullScreenVideoHandler::FullScreenVideoHandler): (FullScreenVideoHandler::~FullScreenVideoHandler): (FullScreenVideoHandler::requiresFullScreenForVideoPlayback): (FullScreenVideoHandler::enterFullScreen): (FullScreenVideoHandler::exitFullScreen): (WebPlugin::supportsExtension): (WebPlugin::createExtension):
  • examples/platformplugin/WebPlugin.h: (FullScreenVideoWidget::~FullScreenVideoWidget):
  • examples/platformplugin/platformplugin.pro:
  • examples/platformplugin/qwebkitplatformplugin.h: (QWebFullScreenVideoHandler::QWebFullScreenVideoHandler): (QWebFullScreenVideoHandler::~QWebFullScreenVideoHandler):
01:50 Changeset [75707] by mitz@apple.com
  • 3 edits
    2 adds in trunk

<rdar://problem/8858548> Disable inheritance of text emphasis marks to ruby text
https://bugs.webkit.org/show_bug.cgi?id=52359

Reviewed by Dave Hyatt.

Source/WebCore:

Test: fast/ruby/text-emphasis.html

  • css/html.css:

(rt):

LayoutTests:

  • fast/ruby/text-emphasis-expected.txt: Added.
  • fast/ruby/text-emphasis.html: Added.
00:39 Changeset [75706] by hayato@chromium.org
  • 2 edits in trunk/LayoutTests

2011-01-13 Hayato Ito <hayato@chromium.org>

Unreviewed, chromium test expectations.

Flaky expectations for html5lib/webkit-resumer.html.

  • platform/chromium/test_expectations.txt:
00:26 Changeset [75705] by commit-queue@webkit.org
  • 3 edits in trunk/Tools

2011-01-13 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r74881.
http://trac.webkit.org/changeset/74881
https://bugs.webkit.org/show_bug.cgi?id=52357

The EFL builder has been red for days (Requested by abarth on
#webkit).

  • Scripts/webkitpy/common/net/buildbot/buildbot.py:
  • Scripts/webkitpy/common/net/buildbot/buildbot_unittest.py:
00:25 Changeset [75704] by ossy@webkit.org
  • 2 edits in trunk/Source/WebCore

Trivial fix after r75682.

Rubber-stamped by David Levin.

  • loader/FrameLoader.cpp: Remove accidentally committed debug code.

(WebCore::FrameLoader::shouldClose):

01/12/11:

23:58 Changeset [75703] by mrowe@apple.com
  • 2 edits in tags/Safari-534.15.5/WebKit2

Merge r74615.

23:50 Changeset [75702] by mrowe@apple.com
  • 5 edits in tags/Safari-534.15.5

Versioning.

23:50 Changeset [75701] by mrowe@apple.com
  • 2 edits in tags/Safari-534.15.5/WebKit2

Merge r74508.

23:49 Changeset [75700] by mrowe@apple.com
  • 2 edits in tags/Safari-534.15.5/WebKit2

Merge r74329.

23:49 Changeset [75699] by mrowe@apple.com
  • 2 edits in tags/Safari-534.15.5/WebKit2

Merge r75015.

23:49 Changeset [75698] by mrowe@apple.com
  • 2 edits in tags/Safari-534.15.5/WebKit2

Merge r74531.

23:48 Changeset [75697] by mrowe@apple.com
  • 7 edits in tags/Safari-534.15.5/WebKit2

Merge r74514.

23:48 Changeset [75696] by mrowe@apple.com
  • 3 edits in tags/Safari-534.15.5/WebKit2

Merge r74591.

23:48 Changeset [75695] by mrowe@apple.com
  • 2 edits in tags/Safari-534.15.5/WebKit2

Merge r75038.

23:47 Changeset [75694] by mrowe@apple.com
  • 2 edits in tags/Safari-534.15.5/WebKit2

Merge r74303.

23:47 Changeset [75693] by mrowe@apple.com
  • 2 edits in tags/Safari-534.15.5/WebCore

Merge r75660.

23:47 Changeset [75692] by mrowe@apple.com
  • 2 edits in tags/Safari-534.15.5/WebCore

Merge r75656.

23:47 Changeset [75691] by mrowe@apple.com
  • 2 edits in tags/Safari-534.15.5/WebCore

Merge r75653.

23:46 Changeset [75690] by mrowe@apple.com
  • 3 edits in tags/Safari-534.15.5/WebCore

Merge r75652.

23:46 Changeset [75689] by mrowe@apple.com
  • 3 edits in tags/Safari-534.15.5/WebCore

Merge r75649.

23:46 Changeset [75688] by mrowe@apple.com
  • 2 edits in tags/Safari-534.15.5/WebCore

Merge r75644.

23:45 Changeset [75687] by mrowe@apple.com
  • 4 edits in tags/Safari-534.15.5/WebCore

Merge r75643.

23:45 Changeset [75686] by mrowe@apple.com
  • 14 edits in tags/Safari-534.15.5/WebCore

Merge r75639.

23:45 Changeset [75685] by mrowe@apple.com
  • 3 edits
    2 adds in tags/Safari-534.15.5

Merge r75472.

23:44 Changeset [75684] by mrowe@apple.com
  • 2 edits in tags/Safari-534.15.5/WebCore

Merge r75580.

23:27 Changeset [75683] by abarth@webkit.org
  • 2 edits in trunk/LayoutTests

2011-01-12 Adam Barth <abarth@webkit.org>

Removing this test from the Skipped list caused the next test
alphabetically to fail instead.

  • platform/mac/Skipped:
23:09 Changeset [75682] by mjs@apple.com
  • 5 edits in trunk

2011-01-12 Maciej Stachowiak <mjs@apple.com>

Reviewed by David Levin.

Add beforeunload support to WebKitTestRunner
https://bugs.webkit.org/show_bug.cgi?id=52353

  • platform/mac-wk2/Skipped: Unskip a test that was failing solely due to missing beforeunload support.

2011-01-12 Maciej Stachowiak <mjs@apple.com>

Reviewed by David Levin.

Add beforeunload support to WebKitTestRunner
https://bugs.webkit.org/show_bug.cgi?id=52353


This makes a number of currently failing tests pass.

  • WebKitTestRunner/TestController.cpp: (WTR::runBeforeUnloadConfirmPanel): (WTR::createOtherPage): (WTR::TestController::initialize):
22:49 Changeset [75681] by tkent@chromium.org
  • 3 edits
    4 adds in branches/chromium/597

Merge 75676
Because 597 branch doesn't have FormAssociatedElement.cpp, this change
was applied manually. The actual code change is in
WebCore/html/HTMLFormControlElement.cpp attributeChanged().

2011-01-12 Kenichi Ishibashi <bashi@google.com>

Reviewed by Kent Tamura.

M_formElementsWithFormAttribute not cleared when FormAssociatedElement is inserted with a null m_form and then removed.
https://bugs.webkit.org/show_bug.cgi?id=51905

Calls unregisterFormElementWithFormAttribute() when 'form' attribute
is removed.

Tests: fast/forms/form-associated-element-crash.html

fast/forms/form-associated-element-crash2.html

  • html/FormAssociatedElement.cpp: (WebCore::FormAssociatedElement::formAttributeChanged):

2011-01-12 Kenichi Ishibashi <bashi@google.com>

Reviewed by Kent Tamura.

M_formElementsWithFormAttribute not cleared when FormAssociatedElement is inserted with a null m_form and then removed.
https://bugs.webkit.org/show_bug.cgi?id=51905

Add test for crash when modifying the form attribute of a form
associated element before removing it.

  • fast/forms/form-associated-element-crash-expected.txt: Added.
  • fast/forms/form-associated-element-crash.html: Added.
  • fast/forms/form-associated-element-crash2-expected.txt: Added.
  • fast/forms/form-associated-element-crash2.html: Added.

BUG=

22:35 Changeset [75680] by tkent@chromium.org
  • 4 edits
    2 adds in branches/chromium/597

Merge 75675
Because 597 branch doesn't have FormAssociatedElement.{cpp,h}, this
change was manually applied and the ChangeLog entries are not
accurate.

2011-01-12 Kenichi Ishibashi <bashi@google.com>

Reviewed by Kent Tamura

M_formElementsWithFormAttribute not cleared when Node is moved to another document.
https://bugs.webkit.org/show_bug.cgi?id=51418

Calls unregisterFormElementWithFormAttribute() when form associated elements
are moved to another document.

Test: fast/forms/change-form-element-document-crash.html

  • html/FormAssociatedElement.cpp: (WebCore::FormAssociatedElement::willMoveToNewOwnerDocument): Added.
  • html/FormAssociatedElement.h: Added willMoveToNewOwnerDocument().
  • html/HTMLFormControlElement.cpp: (WebCore::HTMLFormControlElement::willMoveToNewOwnerDocument): Added.
  • html/HTMLFormControlElement.h: Added willMoveToNewOwnerDocument().
  • html/HTMLObjectElement.cpp: (WebCore::HTMLObjectElement::willMoveToNewOwnerDocument): Added.
  • html/HTMLObjectElement.h: Added willMoveToNewOwnerDocument().
  • html/HTMLPlugInImageElement.h: Moves willMoveToNewOwnerDocument() to protected.

2011-01-12 Kenichi Ishibashi <bashi@google.com>

Reviewed by Kent Tamura

M_formElementsWithFormAttribute not cleared when Node is moved to another document.
https://bugs.webkit.org/show_bug.cgi?id=51418

Added test for crash when moving form elements from one document to another.

  • fast/forms/change-form-element-document-crash-expected.txt: Added.
  • fast/forms/change-form-element-document-crash.html: Added.

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

21:55 Changeset [75679] by hayato@chromium.org
  • 3 edits in trunk/LayoutTests

2011-01-12 Hayato Ito <hayato@chromium.org>

Unreviewed test expectations files fix.

Fix a culprit revision number.

  • platform/chromium-gpu/test_expectations.txt:
  • platform/chromium/test_expectations.txt:
20:59 Changeset [75678] by hayato@chromium.org
  • 2 edits in trunk/LayoutTests

2011-01-12 Hayato Ito <hayato@chromium.org>

Unreviewed chromium-gpu test expectations fix.

Disable canvas-large-dimensions.html after r75639.

  • platform/chromium-gpu/test_expectations.txt:
20:51 Changeset [75677] by rniwa@webkit.org
  • 3 edits
    2 adds in trunk

2011-01-12 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Eric Seidel.

REGRESSION(r69831): focus() in onkeypress discards input (affects chaseonline.chase.com)
https://bugs.webkit.org/show_bug.cgi?id=52241

The bug was caused by RenderTextControl::selection's creating a Range with m_insertText
which is a shadow DOM div as both start and end containers. Fixed the bug by traversing
through the descendents of m_innerText and using the right start and end containers to
create a Range.

Test: fast/forms/focus-change-on-keypress.html

  • rendering/RenderTextControl.cpp: (WebCore::setContainerAndOffsetForRange): Added; a helper function. (WebCore::RenderTextControl::selection): See above.

2011-01-12 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Eric Seidel.

REGRESSION(r69831): focus() in onkeypress discards input (affects chaseonline.chase.com)
https://bugs.webkit.org/show_bug.cgi?id=52241

Added a test to ensure WebKit inserts the character typed by a user when focus's node
is changed by a presskey event handler.

  • fast/forms/focus-change-on-keypress-expected.txt: Added.
  • fast/forms/focus-change-on-keypress.html: Added.
19:55 Changeset [75676] by tkent@chromium.org
  • 3 edits
    4 adds in trunk

2011-01-12 Kenichi Ishibashi <bashi@google.com>

Reviewed by Kent Tamura.

M_formElementsWithFormAttribute not cleared when FormAssociatedElement is inserted with a null m_form and then removed.
https://bugs.webkit.org/show_bug.cgi?id=51905

Calls unregisterFormElementWithFormAttribute() when 'form' attribute
is removed.

Tests: fast/forms/form-associated-element-crash.html

fast/forms/form-associated-element-crash2.html

  • html/FormAssociatedElement.cpp: (WebCore::FormAssociatedElement::formAttributeChanged):

2011-01-12 Kenichi Ishibashi <bashi@google.com>

Reviewed by Kent Tamura.

M_formElementsWithFormAttribute not cleared when FormAssociatedElement is inserted with a null m_form and then removed.
https://bugs.webkit.org/show_bug.cgi?id=51905

Add test for crash when modifying the form attribute of a form
associated element before removing it.

  • fast/forms/form-associated-element-crash-expected.txt: Added.
  • fast/forms/form-associated-element-crash.html: Added.
  • fast/forms/form-associated-element-crash2-expected.txt: Added.
  • fast/forms/form-associated-element-crash2.html: Added.
19:48 Changeset [75675] by tkent@chromium.org
  • 9 edits
    2 adds in trunk

2011-01-12 Kenichi Ishibashi <bashi@google.com>

Reviewed by Kent Tamura.

M_formElementsWithFormAttribute not cleared when Node is moved to another document.
https://bugs.webkit.org/show_bug.cgi?id=51418

Calls unregisterFormElementWithFormAttribute() when form associated elements
are moved to another document.

Test: fast/forms/change-form-element-document-crash.html

  • html/FormAssociatedElement.cpp: (WebCore::FormAssociatedElement::willMoveToNewOwnerDocument): Added.
  • html/FormAssociatedElement.h: Added willMoveToNewOwnerDocument().
  • html/HTMLFormControlElement.cpp: (WebCore::HTMLFormControlElement::willMoveToNewOwnerDocument): Added.
  • html/HTMLFormControlElement.h: Added willMoveToNewOwnerDocument().
  • html/HTMLObjectElement.cpp: (WebCore::HTMLObjectElement::willMoveToNewOwnerDocument): Added.
  • html/HTMLObjectElement.h: Added willMoveToNewOwnerDocument().
  • html/HTMLPlugInImageElement.h: Moves willMoveToNewOwnerDocument() to protected.

2011-01-12 Kenichi Ishibashi <bashi@google.com>

Reviewed by Kent Tamura.

M_formElementsWithFormAttribute not cleared when Node is moved to another document.
https://bugs.webkit.org/show_bug.cgi?id=51418

Added test for crash when moving form elements from one document to another.

  • fast/forms/change-form-element-document-crash-expected.txt: Added.
  • fast/forms/change-form-element-document-crash.html: Added.
19:26 Changeset [75674] by rniwa@webkit.org
  • 1 edit in trunk/Source/WebCore/ChangeLog

Unreviwed. Added the missing bug id for r75672.

19:23 Changeset [75673] by kbalazs@webkit.org
  • 4 edits in trunk/WebKit2

Unreviewed.

Getting rid of the unused 'viewportSize' argument in QWKPagePrivate:init.
The argument is unused since http://trac.webkit.org/changeset/75468.

  • UIProcess/API/qt/qgraphicswkview.cpp:

(QGraphicsWKView::QGraphicsWKView):

  • UIProcess/API/qt/qwkpage.cpp:

(QWKPagePrivate::init):

  • UIProcess/API/qt/qwkpage_p.h:
19:17 Changeset [75672] by rniwa@webkit.org
  • 5 edits in trunk/Source/WebCore

2011-01-12 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Eric Seidel.

Removed instantiation of legacy editing positions.
Calls to Position::Position are replaced by calls to Position's convenience functions.

Also added firstPositionInOrBeforeNode and lastPositionInOrAfterNode,
which are functions that return firstPositionInNode and lastPositionInNode respectively
when the specified node can have children for editing purpose and return positionBeforeNode
and positionAfterNode respectively otherwise. They can be used to guarantee that we never
create a bogus position like [img, 0] or [br, 0].

No tests are added since this is a cleanup.

  • editing/ApplyBlockElementCommand.cpp: (WebCore::ApplyBlockElementCommand::formatSelection):
  • editing/ApplyStyleCommand.cpp: (WebCore::ApplyStyleCommand::applyBlockStyle): (WebCore::ApplyStyleCommand::fixRangeAndApplyInlineStyle): (WebCore::ApplyStyleCommand::highestAncestorWithConflictingInlineStyle): (WebCore::ApplyStyleCommand::removeInlineStyle): (WebCore::ApplyStyleCommand::nodeFullySelected): (WebCore::ApplyStyleCommand::nodeFullyUnselected): (WebCore::ApplyStyleCommand::splitTextAtStart): (WebCore::ApplyStyleCommand::splitTextAtEnd): (WebCore::ApplyStyleCommand::splitTextElementAtStart): (WebCore::ApplyStyleCommand::splitTextElementAtEnd): (WebCore::ApplyStyleCommand::isValidCaretPositionInTextNode): (WebCore::ApplyStyleCommand::mergeStartWithPreviousIfIdentical): (WebCore::ApplyStyleCommand::mergeEndWithNextIfIdentical): (WebCore::ApplyStyleCommand::computedFontSize): (WebCore::ApplyStyleCommand::joinChildTextNodes):
  • editing/ApplyStyleCommand.h:
  • editing/htmlediting.h: (WebCore::firstPositionInOrBeforeNode): Added. (WebCore::lastPositionInOrAfterNode): Added.
19:15 Changeset [75671] by mrowe@apple.com
  • 1 copy in tags/Safari-534.15.5

New tag.

18:50 Changeset [75670] by andersca@apple.com
  • 3 edits in trunk/WebKit2

2011-01-12 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

Implement DrawingAreaImpl::scroll
https://bugs.webkit.org/show_bug.cgi?id=52346

  • WebProcess/WebPage/DrawingAreaImpl.cpp: (WebKit::DrawingAreaImpl::scroll): If there is already an active scroll, compute the area of both scroll rects and invalidate the smallest one. Compute a new dirty region if necessary and also add the scroll repaint region to the dirty region.

(WebKit::DrawingAreaImpl::display):
Reset the scroll area and scroll delta.

  • WebProcess/WebPage/DrawingAreaImpl.h: Add scroll area and scroll delta member variables.
18:34 Changeset [75669] by andersca@apple.com
  • 3 edits in trunk/WebKit2

2011-01-12 Anders Carlsson <andersca@apple.com>

Reviewed by Dan Bernstein.

Region improvements
https://bugs.webkit.org/show_bug.cgi?id=52344

  • Platform/Region.cpp: (WebKit::Region::Shape::move): (WebKit::Region::move): Change Region::move to take an IntSize.
  • Platform/Region.h: (WebKit::intersect): (WebKit::subtract): (WebKit::move): Add convenience functions.
18:01 Changeset [75668] by weinig@apple.com
  • 2 edits in trunk/Source/WebCore

Another build fix.

  • platform/mac/WheelEventMac.mm:

(WebCore::phaseForEvent):

17:58 Changeset [75667] by rniwa@webkit.org
  • 2 edits in trunk/WebKit2

2011-01-12 Ryosuke Niwa <rniwa@webkit.org>

Unreviewed Qt build fix for r75661

  • Shared/WebEvent.h: Don't declare phase() on non-Mac platforms.
17:47 Changeset [75666] by jschuh@chromium.org
  • 2 edits in trunk/LayoutTests

2011-01-12 Justin Schuh <jschuh@chromium.org>

Unreviewed chromium test expectations fix.

Disable canvas-large-dimensions.html after r75639.

  • platform/chromium/test_expectations.txt:
17:45 Changeset [75665] by weinig@apple.com
  • 2 edits in trunk/Source/WebCore

Fix build.

  • platform/mac/WheelEventMac.mm:

(WebCore::phaseForEvent):

17:39 Changeset [75664] by abarth@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-01-12 Adam Barth <abarth@webkit.org>

Reviewed by Darin Adler.

Remove premature optimization from SchemeRegistry
https://bugs.webkit.org/show_bug.cgi?id=52283

As far as I can tell, this optimization isn't necessary. I'm slightly
unsure which benchmarks to run to demonstrate this numerically, but
none of the benchmarks Chromium tracks were affected by adding a
similar check for "display-isolated" schemes.

  • platform/SchemeRegistry.cpp: (WebCore::SchemeRegistry::shouldTreatURLSchemeAsLocal):
17:38 Changeset [75663] by enrica@apple.com
  • 1 edit in trunk/Source/WebCore/platform/mac/WheelEventMac.mm

Build fix, unreviewed.

17:34 Changeset [75662] by bdakin@apple.com
  • 9 edits in trunk

Add-on for https://bugs.webkit.org/show_bug.cgi?id=52309
Expose fixed layout through WebKit SPI
-and corresponding-
<rdar://problem/8844464>

Reviewed by Anders Carlsson.

Source/WebCore:

And now with getters!

  • WebCore.exp.in:

WebKit/mac:

And now with getters!

  • WebView/WebView.mm:

(-[WebView _useFixedLayout]):
(-[WebView _fixedLayoutSize]):

  • WebView/WebViewPrivate.h:

WebKit2:

And now with getters!

  • UIProcess/API/C/WKPage.cpp:

(WKPageUseFixedLayout):
(WKPageFixedLayoutSize):

  • UIProcess/API/C/WKPage.h:
  • UIProcess/WebPageProxy.h:

(WebKit::WebPageProxy::useFixedLayout):
(WebKit::WebPageProxy::fixedLayoutSize):

17:25 Changeset [75661] by weinig@apple.com
  • 8 edits in trunk

https://bugs.webkit.org/show_bug.cgi?id=52337
PlatformWheelEvent should know about the scroll phase on the Mac

Reviewed by Anders Carlsson.

Source/WebCore:

Add a phase parameter on Mac PlatformWheelEvents.

  • platform/PlatformWheelEvent.h:

(WebCore::PlatformWheelEvent::PlatformWheelEvent):
(WebCore::PlatformWheelEvent::phase):

  • platform/mac/WheelEventMac.mm:

(WebCore::phaseForEvent):
(WebCore::PlatformWheelEvent::PlatformWheelEvent):

WebKit2:

Add a phase parameter on Mac WebWheelEvent and pipe it down to PlatformWheelEvent.

  • Shared/WebEvent.h:

(WebKit::WebWheelEvent::phase):

  • Shared/WebEventConversion.cpp:

(WebKit::WebKit2PlatformWheelEvent::WebKit2PlatformWheelEvent):

  • Shared/WebWheelEvent.cpp:

(WebKit::WebWheelEvent::WebWheelEvent):
(WebKit::WebWheelEvent::encode):
(WebKit::WebWheelEvent::decode):

  • Shared/mac/WebEventFactory.mm:

(WebKit::phaseForEvent):
(WebKit::WebEventFactory::createWebWheelEvent):

17:24 Changeset [75660] by cmarrin@apple.com
  • 2 edits in trunk/Source/WebCore

2011-01-12 Chris Marrin <cmarrin@apple.com>

Unreviewed.

A couple of lines of the patch for https://bugs.webkit.org/show_bug.cgi?id=52321
were omitted.

  • platform/graphics/ca/GraphicsLayerCA.cpp: (WebCore::GraphicsLayerCA::commitLayerChangesBeforeSublayers):
17:02 Changeset [75659] by andersca@apple.com
  • 12 edits in trunk/WebKit2

2011-01-12 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

Get rid of an unneeded parameter from DrawingArea::scroll
https://bugs.webkit.org/show_bug.cgi?id=52336

  • WebProcess/WebCoreSupport/WebChromeClient.cpp: (WebKit::WebChromeClient::scroll):
  • WebProcess/WebCoreSupport/WebChromeClient.h:
  • WebProcess/WebPage/ChunkedUpdateDrawingArea.cpp: (WebKit::ChunkedUpdateDrawingArea::scroll):
  • WebProcess/WebPage/ChunkedUpdateDrawingArea.h:
  • WebProcess/WebPage/DrawingArea.h:
  • WebProcess/WebPage/DrawingAreaImpl.cpp: (WebKit::DrawingAreaImpl::scroll):
  • WebProcess/WebPage/DrawingAreaImpl.h:
  • WebProcess/WebPage/LayerBackedDrawingArea.cpp: (WebKit::LayerBackedDrawingArea::scroll):
  • WebProcess/WebPage/LayerBackedDrawingArea.h:
  • WebProcess/WebPage/TiledDrawingArea.cpp: (WebKit::TiledDrawingArea::scroll):
  • WebProcess/WebPage/TiledDrawingArea.h:
16:51 Changeset [75658] by mihaip@chromium.org
  • 3 edits in trunk/WebKit/chromium

2011-01-12 Mihai Parparita <mihaip@chromium.org>

Reviewed by David Levin.

[Chromium] Remove PartScrollbarHoriztonalTrack
https://bugs.webkit.org/show_bug.cgi?id=52329

Remove PartScrollbarHoriztonalTrack now that usages of it were removed
with http://crrev.com/71249.

  • DEPS:
  • public/linux/WebThemeEngine.h:
16:41 Changeset [75657] by andersca@apple.com
  • 5 edits in trunk/WebKit2

2011-01-12 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

Get the new drawing area painting to the screen
https://bugs.webkit.org/show_bug.cgi?id=52331

  • UIProcess/DrawingAreaProxyImpl.cpp: (WebKit::DrawingAreaProxyImpl::incorporateUpdate): Call setViewNeedsDisplay on all the update rects.
  • UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::setViewNeedsDisplay): Call through to the page client.

(WebKit::WebPageProxy::processDidCrash):
Null out the drawing area proxy.


  • UIProcess/mac/BackingStoreMac.mm: (WebKit::BackingStore::paint): Paint.
16:35 Changeset [75656] by cmarrin@apple.com
  • 2 edits in trunk/Source/WebCore

2011-01-12 Chris Marrin <cmarrin@apple.com>

Unreviewed.

Getting rid of an assert that was causing build failures that turns out not to be needed.

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::paintsWithTransform):
16:22 Changeset [75655] by mjs@apple.com
  • 4 edits in trunk/Tools

2011-01-12 Maciej Stachowiak <mjs@apple.com>

Reviewed by Sam Weinig.

Detect, identify and recover from WebProcess crashes in run-webkit-tests
https://bugs.webkit.org/show_bug.cgi?id=52330

  • Scripts/old-run-webkit-tests: Detect when the regression test tool reports a web process crash.
  • WebKitTestRunner/TestController.cpp: (WTR::TestController::initialize): Set up a processDidCrash callback. (WTR::TestController::processDidCrash): Report that the Web process crashed.
  • WebKitTestRunner/TestController.h:
16:20 Changeset [75654] by ojan@chromium.org
  • 3 edits in trunk/Websites/bugs.webkit.org

2011-01-12 Ojan Vafai <ojan@chromium.org>

Reviewed by Mihai Parparita.

fix assorted bugs with expansion lines
https://bugs.webkit.org/show_bug.cgi?id=52326

-Clicking "all" would collapse whitespace in the expanded lines.
-Converted the remaining line creation code to using the DOM.

  • PrettyPatch/PrettyPatch.rb:
  • code-review.js:
16:14 Changeset [75653] by cmarrin@apple.com
  • 2 edits in trunk/Source/WebCore

2011-01-12 Chris Marrin <cmarrin@apple.com>

Unreviewed.

Another fix for Gtk

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::paintsWithTransform):
16:08 Changeset [75652] by cmarrin@apple.com
  • 4 edits in trunk/Source/WebCore

2011-01-12 Chris Marrin <cmarrin@apple.com>

Unreviewed.

Fix for Gtk and Windows builds

  • page/Frame.cpp: (WebCore::Frame::scalePage):
  • page/Frame.h:
  • platform/graphics/ca/win/PlatformCALayerWin.cpp: (PlatformCALayer::contentsScale): (PlatformCALayer::setContentsScale):
15:52 Changeset [75651] by cmarrin@apple.com
  • 2 edits in trunk/Source/WebCore

2011-01-12 Chris Marrin <cmarrin@apple.com>

UNreviewed.

Fix for the Windows build

  • platform/graphics/ca/win/PlatformCALayerWin.cpp: (contentsScale): (setContentsScale):
15:48 Changeset [75650] by ojan@chromium.org
  • 2 edits in trunk/Websites/bugs.webkit.org

2011-01-12 Ojan Vafai <ojan@chromium.org>

Reviewed by Adam Barth.

review tool formatted diff doesn't match the uploaded diff
https://bugs.webkit.org/show_bug.cgi?id=51960

PrettyPatch works just fine if there is no newline at the end of the file.

  • attachment.cgi:
15:41 Changeset [75649] by cmarrin@apple.com
  • 3 edits in trunk/Source/WebCore

2011-01-12 Chris Marrin <cmarrin@apple.com>

Unreviewed.

Another build fix

  • platform/graphics/ca/GraphicsLayerCA.cpp: (WebCore::GraphicsLayerCA::clampedContentsScaleForScale):
  • platform/graphics/ca/mac/PlatformCALayerMac.mm: (PlatformCALayer::setContentsScale):
15:35 Changeset [75648] by mdelaney@apple.com
  • 3 edits
    2 adds in trunk

2011-01-11 Matthew Delaney <mdelaney@apple.com>

Reviewed by Simon Fraser.

Max area bound needed in creation of IOSurface in ImageBufferCG.cpp
https://bugs.webkit.org/show_bug.cgi?id=52172

Tests: fast/canvas/canvas-large-dimensions.html

  • platform/graphics/cg/ImageBufferCG.cpp: (WebCore::ImageBuffer::ImageBuffer):

2011-01-11 Matthew Delaney <mdelaney@apple.com>

Reviewed by Simon Fraser.

Max area bound needed in creation of IOSurface in ImageBufferCG.cpp
https://bugs.webkit.org/show_bug.cgi?id=52172

  • fast/canvas/canvas-large-dimensions.html: Added.
  • fast/canvas/canvas-large-dimensions-expected.txt: Added.
15:29 Changeset [75647] by andersca@apple.com
  • 6 edits in trunk/WebKit2

2011-01-12 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

More work on getting the drawing area proxy to paint
https://bugs.webkit.org/show_bug.cgi?id=52328

  • UIProcess/API/mac/WKView.mm: (-[WKView drawRect:]): Handle the new drawing area.
  • UIProcess/BackingStore.cpp: (WebKit::BackingStore::BackingStore): Assert that the size isn't empty.


  • UIProcess/BackingStore.h:
  • UIProcess/mac/BackingStoreMac.mm: (WebKit::BackingStore::incorporateUpdate): Paint all update rects into the bitmap context.
  • WebProcess/WebPage/DrawingAreaImpl.cpp: (WebKit::DrawingAreaImpl::display): Create a handle so the shareable bitmap will actually be transferred over.
15:27 Changeset [75646] by ojan@chromium.org
  • 3 edits in trunk/Websites/bugs.webkit.org

2011-01-12 Ojan Vafai <ojan@chromium.org>

Reviewed by Adam Barth.

Bugzilla: Add keyboard shortcuts to jump to next change
https://bugs.webkit.org/show_bug.cgi?id=52305

Comments and diff blocks go in the same queue. If you have a
comment focused, then j/k will focus the next/prev diff block
with respect to that comment.

  • PrettyPatch/PrettyPatch.rb:
  • code-review.js:
15:27 Changeset [75645] by dbates@webkit.org
  • 3 edits in trunk/Source/WebCore

2011-01-12 Daniel Bates <dbates@rim.com>

And Benjamin C Meyer <bmeyer@rim.com>

Build fix when building with Inspector disabled
https://bugs.webkit.org/show_bug.cgi?id=52313

After changeset 75604 <http://trac.webkit.org/changeset/75604> it's not
possible to build when the Inspector is disabled.

  • inspector/InspectorInstrumentation.h: (WebCore::InspectorInstrumentation::updateApplicationCacheStatus): Fix build breakage when building with Inspector disabled and Offline Web Applications enabled by moving macro guard condition ENABLE(INSPECTOR) && ENABLE(OFFLINE_WEB_APPLICATIONS) into function body. This makes us also consistent with the form we use for InspectorInstrumentation::networkStateChanged() and other InspectorInstrumentation functions.
  • loader/appcache/ApplicationCacheGroup.cpp: (WebCore::ApplicationCacheGroup::didReceiveData): Fix build breakage when building with Inspector disabled by adding ENABLE(INSPECTOR) guard around code that references the instance variable ApplicationCacheGroup::m_currentResourceIdentifier as this variable only exists when building with the Inspector. (WebCore::ApplicationCacheGroup::didFinishLoading): Ditto. (WebCore::ApplicationCacheGroup::didFail): Ditto.
15:12 Changeset [75644] by cmarrin@apple.com
  • 2 edits in trunk/Source/WebCore

2011-01-12 Chris Marrin <cmarrin@apple.com>

Unreviewed.

Fix a build issue from http://trac.webkit.org/changeset/75639.

  • platform/graphics/GraphicsLayer.h: (WebCore::GraphicsLayer::contentsScale): (WebCore::GraphicsLayer::setContentsScale):
15:06 Changeset [75643] by cmarrin@apple.com
  • 4 edits in trunk/Source/WebCore

2011-01-12 Chris Marrin <cmarrin@apple.com>

Reviewed by Kevin Decker.

Pages with accelerated layers lose subpixel-AA and become blurry when a scale factor is applied
rdar://problem/8824974

This prevents the rootLayer (with the WebKit rendered content) from going becoming
a tiled layer when zoomed, and when there is other layered content on the page.
This speeds up zooming a lot and avoids some buggy behavior in CA with a very large
number of layers.


  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::paintsWithTransform):
  • rendering/RenderLayer.h:
  • rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::paintingGoesToWindow):
14:58 Changeset [75642] by ojan@chromium.org
  • 2 edits in trunk/Websites/bugs.webkit.org

2011-01-12 Ojan Vafai <ojan@chromium.org>

Reviewed by Adam Barth.

show shared lines on both sides in code review tool
https://bugs.webkit.org/show_bug.cgi?id=52308

We were appending the same DOM node twice. Instead, make
the from column a clone of the original node.

  • code-review.js:
14:44 Changeset [75641] by andersca@apple.com
  • 2 edits in trunk/WebKit2

Fix build.

  • UIProcess/DrawingAreaProxyImpl.cpp:

(WebKit::DrawingAreaProxyImpl::incorporateUpdate):

14:34 Changeset [75640] by andersca@apple.com
  • 6 edits
    1 add in trunk/WebKit2

2011-01-12 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

Begin stubbing out some more BackingStore member functions
https://bugs.webkit.org/show_bug.cgi?id=52323

  • UIProcess/BackingStore.cpp: (WebKit::BackingStore::~BackingStore): This was accidentally made into a constructor; it should be a destructor.
  • UIProcess/DrawingAreaProxyImpl.cpp: (WebKit::DrawingAreaProxyImpl::paint): Call the backing store.

(WebKit::DrawingAreaProxyImpl::update):
Incorporate the update.

(WebKit::DrawingAreaProxyImpl::incorporateUpdate):
Create the backing store if needed and incorporate the update.

  • UIProcess/mac/BackingStoreMac.mm: Added. (WebKit::BackingStore::platformInitialize): Create a bitmap context.

(WebKit::BackingStore::paint):
(WebKit::BackingStore::incorporateUpdate):
Add stubs.

  • WebKit2.xcodeproj/project.pbxproj: Add BackingStoreMac.mm
14:27 Changeset [75639] by cmarrin@apple.com
  • 14 edits in trunk/Source/WebCore

2011-01-12 Chris Marrin <cmarrin@apple.com>

Reviewed by Simon Fraser.


Pages with accelerated layers lose subpixel-AA and become blurry when a scale factor is applied
rdar://problem/8824974

When scaled, WebKit renders the page content at the scaled up size, so there are no
scaling artifacts. But content drawn into a layer's backing store does not scale up.
This is made worse by the fact that the root page contents become layered when there
are other accelerated compositing layers present (video, plugins, etc.).


Plumb scaling factor from Frame::scalePage() down into all layers with content. It
eventually calls CALayer::setContentsScale which renders to a backing store whose dimensions
are scaled, causing them to render larger and appear pixel perfect at the scaled
page size.


  • page/Frame.cpp: (WebCore::Frame::updateContentsScale): (WebCore::Frame::scalePage):
  • page/Frame.h:
  • platform/graphics/GraphicsLayer.h:
  • platform/graphics/ca/GraphicsLayerCA.cpp: (WebCore::GraphicsLayerCA::setContentsScale): (WebCore::GraphicsLayerCA::clampedContentsScaleForScale):
  • platform/graphics/ca/GraphicsLayerCA.h: (WebCore::GraphicsLayerCA::contentsScale):
  • platform/graphics/ca/PlatformCALayer.h:
  • platform/graphics/ca/mac/PlatformCALayerMac.mm: (PlatformCALayer::contentsScale): (PlatformCALayer::setContentsScale):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::updateContentsScale):
  • rendering/RenderLayer.h:
  • rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::createGraphicsLayer): (WebCore::RenderLayerBacking::updateForegroundLayer): (WebCore::RenderLayerBacking::updateMaskLayer): (WebCore::RenderLayerBacking::updateContentsScale):
  • rendering/RenderLayerBacking.h:
  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::updateContentsScale):
  • rendering/RenderLayerCompositor.h:


14:26 Changeset [75638] by tony@chromium.org
  • 8 edits
    2 adds in trunk/LayoutTests

2011-01-12 Tony Chang <tony@chromium.org>

Reviewed by Ojan Vafai.

output the value of input and textarea nodes when using dump-as-markup
https://bugs.webkit.org/show_bug.cgi?id=52319

  • editing/inserting/5607069-2-expected.txt:
  • editing/inserting/5607069-3-expected.txt:
  • editing/selection/dump-as-markup-expected.txt:
  • editing/selection/dump-as-markup-form-text-expected.txt: Added.
  • editing/selection/dump-as-markup-form-text.html: Added.
  • fast/parser/object-with-textarea-fallback-expected.txt:
  • html5lib/runner.html:
  • html5lib/webkit-resumer.html:
  • resources/dump-as-markup.js: (Markup.useHTML5libOutputFormat): Add a flag for html5lib compatibility (Markup._get):
13:58 Changeset [75637] by senorblanco@chromium.org
  • 5 edits in trunk/Source/WebCore

2011-01-12 Stephen White <senorblanco@chromium.org>

Reviewed by James Robinson.

Fix DrawingBuffer issues for depth/stencil, extensions, etc.
https://bugs.webkit.org/show_bug.cgi?id=52310

1) GL_TEXTURE is not a valid argument to glFramebufferTexture2D()
(should be GL_TEXTURE_2D).
2) Support for the multisampling extension was being checked after
the DrawingBuffer constructor was called, which calls multisample()
to check for the extension.
3) If depth or stencil is set, the corresponding framebuffers must
be created in the DrawingBuffer constructor.
4) GL_DEPTH_STENCIL is not a valid framebuffer internal type in
GLES2. Must use GL_OES_DEPTH24_STENCIL8, and check for the
OES_packed_depth_stencil extension.

  • platform/graphics/chromium/DrawingBufferChromium.cpp: (WebCore::DrawingBuffer::DrawingBuffer): Record the extension support in the constructor. Create all the secondary drawing buffers (for multisampling, depth/stencil).
  • platform/graphics/gpu/DrawingBuffer.cpp: (WebCore::DrawingBuffer::create): Check for the extensions before calling the DrawingBuffer constructor. (WebCore::DrawingBuffer::createSecondaryBuffers): Refactor creation of the secondary buffers from DrawingBufferMac.mm to here. (WebCore::DrawingBuffer::reset): Use DEPTH24_STENCIL8 format, not DEPTH_STENCIL. Use DEPTH_COMPONENT16, not DEPTH_COMPONENT. Use GL_TEXTURE_2D, not GL_TEXTURE (for glFramebufferTexture2D).
  • platform/graphics/gpu/DrawingBuffer.h: Refactor createSecondaryBuffers(). Add extension support arguments to constructor. Add support for packed_depth_stencil extension.
  • platform/graphics/gpu/mac/DrawingBufferMac.mm: (WebCore::DrawingBuffer::DrawingBuffer): Record extension support in constructor. Refactor out creation of secondary buffers.
13:12 Changeset [75636] by robert@webkit.org
  • 1 edit
    6 adds in trunk/LayoutTests

2011-01-12 Robert Hogan <robert@webkit.org>

Unreviewed, add expected results.

[Qt] Add new expected results from r75632

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

  • platform/qt/fast/forms/search-display-none-cancel-button-expected.txt: Added.
  • platform/qt/fast/forms/search-placeholder-value-changed-expected.txt: Added.
  • platform/qt/fast/forms/search-rtl-expected.txt: Added.
  • platform/qt/fast/forms/search-styled-expected.txt: Added.
  • platform/qt/fast/forms/search-transformed-expected.txt: Added.
  • platform/qt/fast/forms/search-zoomed-expected.txt: Added.
13:04 Changeset [75635] by bdakin@apple.com
  • 15 edits in trunk

Fix for https://bugs.webkit.org/show_bug.cgi?id=52309 Expose
fixed layout through WebKit SPI
-and corresponding-
<rdar://problem/8844464>

Reviewed by Anders Carlsson.

Source/WebCore:

Export symbols for ScrollView::setUseFixedLayout() and
ScrollView::setFixedLayoutSize()

  • WebCore.exp.in:

WebKit/mac:

  • WebView/WebView.mm:

(-[WebView _setUseFixedLayout:]):
(-[WebView _setFixedLayoutSize:]):

  • WebView/WebViewPrivate.h:

WebKit2:

Store useFixedLayout and fixedLayoutSize in the
WebPageCreationParameters and the WebPageProxy.

  • Shared/WebPageCreationParameters.cpp:

(WebKit::WebPageCreationParameters::encode):
(WebKit::WebPageCreationParameters::decode):

  • Shared/WebPageCreationParameters.h:
  • UIProcess/API/C/WKPage.cpp:

(WKPageSetUseFixedLayout):
(WKPageSetFixedLayoutSize):

  • UIProcess/API/C/WKPage.h:
  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::WebPageProxy):
(WebKit::WebPageProxy::setUseFixedLayout):
(WebKit::WebPageProxy::setFixedLayoutSize):
(WebKit::WebPageProxy::creationParameters):

  • UIProcess/WebPageProxy.h:
  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::setUseFixedLayout):
(WebKit::WebPage::setFixedLayoutSize):

  • WebProcess/WebPage/WebPage.h:
  • WebProcess/WebPage/WebPage.messages.in:
13:03 Changeset [75634] by robert@webkit.org
  • 5 edits in trunk/LayoutTests

2011-01-12 Robert Hogan <robert@webkit.org>

Unreviewed, update expected results.

[Qt] Update expected results from r75632

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

  • platform/qt/fast/css/pseudo-cache-stale-expected.txt:
  • platform/qt/fast/css/text-input-with-webkit-border-radius-expected.txt:
  • platform/qt/fast/forms/search-cancel-button-style-sharing-expected.txt:
  • platform/qt/fast/forms/search-vertical-alignment-expected.txt:
12:53 Changeset [75633] by andersca@apple.com
  • 2 edits
    2 adds in trunk/WebKit2

2011-01-12 Anders Carlsson <andersca@apple.com>

Reviewed by Beth Dakin.

Add BackingStore class
https://bugs.webkit.org/show_bug.cgi?id=52318

  • UIProcess/BackingStore.cpp: Added. (WebKit::BackingStore::create): (WebKit::BackingStore::BackingStore):
  • UIProcess/BackingStore.h: Added.
  • WebKit2.xcodeproj/project.pbxproj:
12:27 Changeset [75632] by robert@webkit.org
  • 9 edits in trunk

2011-01-12 Robert Hogan <robert@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Search input field doesn't have cancel button

Add a cancel button to the search field and make it customizable
by the client. For now, use the close dialog button associated
with the application's style as the default.

Unskip a bunch of search field related tests that pass now:

fast/forms/search-cancel-button-mouseup.html
fast/forms/search-rtl.html
fast/forms/search-abs-pos-cancel-button.html
fast/forms/search-styled.html
fast/forms/search-transformed.html
fast/forms/search-zoomed.html
fast/forms/search-cancel-button-style-sharing.html
fast/forms/search-display-none-cancel-button.html
fast/forms/search-placeholder-value-changed.html
fast/repaint/search-field-cancel.html
fast/forms/search-cancel-button-events.html

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

  • platform/qt/Skipped:

2011-01-12 Robert Hogan <robert@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Search input field doesn't have cancel button

Add a cancel button to the search field and make it customizable
by the client. For now, use the close dialog button associated
with the application's style as the default.

Unskip a bunch of search field related tests that pass now:

fast/forms/search-cancel-button-mouseup.html
fast/forms/search-rtl.html
fast/forms/search-abs-pos-cancel-button.html
fast/forms/search-styled.html
fast/forms/search-transformed.html
fast/forms/search-zoomed.html
fast/forms/search-cancel-button-style-sharing.html
fast/forms/search-display-none-cancel-button.html
fast/forms/search-placeholder-value-changed.html
fast/repaint/search-field-cancel.html
fast/forms/search-cancel-button-events.html

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

  • platform/graphics/qt/ImageQt.cpp: (loadResourcePixmap):
  • platform/qt/RenderThemeQt.cpp: (WebCore::RenderThemeQt::isControlStyled): (WebCore::RenderThemeQt::computeSizeBasedOnStyle): (WebCore::RenderThemeQt::paintSearchField): (WebCore::RenderThemeQt::adjustSearchFieldStyle): (WebCore::RenderThemeQt::adjustSearchFieldCancelButtonStyle): (WebCore::RenderThemeQt::convertToPaintingRect): (WebCore::RenderThemeQt::paintSearchFieldCancelButton):
  • platform/qt/RenderThemeQt.h:

2011-01-12 Robert Hogan <robert@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Search input field doesn't have cancel button

Add a cancel button to the search field and make it customizable
by the client. For now, use the close dialog button associated
with the application's style as the default.

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

  • Api/qwebsettings.cpp: (graphics):
  • Api/qwebsettings.h:
12:12 Changeset [75631] by beidson@apple.com
  • 15 edits in trunk/WebKit2

<rdar://problem/8837307>, <rdar://problem/8637038>, and https://bugs.webkit.org/show_bug.cgi?id=52248
Back/forward list recovery after a WebProcess crash is crashy itself.

Reviewed by Sam Weinig.

Add an isEmpty() accessor so the WebProcess knows if it is actually restoring a session:

  • Shared/SessionState.cpp:

(WebKit::SessionState::isEmpty):

  • Shared/SessionState.h:

Keep track of the highest used itemID so a relaunched WebProcess can later be informed:

  • Shared/WebBackForwardListItem.cpp:

(WebKit::WebBackForwardListItem::WebBackForwardListItem):
(WebKit::WebBackForwardListItem::highedUsedItemID):

  • Shared/WebBackForwardListItem.h:
  • WebProcess/WebPage/WebBackForwardListProxy.cpp:

(WebKit::generateHistoryItemID):
(WebKit::WebBackForwardListProxy::setHighestItemIDFromUIProcess):
(WebKit::WebBackForwardListProxy::addItemFromUIProcess): Remove a now-invalid ASSERT, as the UIProcess

now tells the WebProcess that a previous WebProcess created.

  • WebProcess/WebPage/WebBackForwardListProxy.h:

Encode/decode data relevant to restoring a session on WebPage creation:

  • Shared/WebPageCreationParameters.cpp:

(WebKit::WebPageCreationParameters::encode):
(WebKit::WebPageCreationParameters::decode):

  • Shared/WebPageCreationParameters.h:

If the WebProcess has crashed since the last load, change reload and the various back/forward APIs to
update the current back/forward item (if necessary) then relaunch the process:

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::reattachToWebProcessWithItem):
(WebKit::WebPageProxy::reattachToWebProcess): Renamed from "relaunch()" and made private.
(WebKit::WebPageProxy::initializeWebPage):
(WebKit::WebPageProxy::loadURL):
(WebKit::WebPageProxy::loadURLRequest):
(WebKit::WebPageProxy::reload):
(WebKit::WebPageProxy::goForward):
(WebKit::WebPageProxy::goBack):
(WebKit::WebPageProxy::goToBackForwardItem):
(WebKit::WebPageProxy::creationParameters):

  • UIProcess/WebPageProxy.h:

Rename "RestoreSession" to "RestoreSessionAndNavigateToCurrentItem"

  • WebProcess/WebPage/WebPage.messages.in:
  • UIProcess/cf/WebPageProxyCF.cpp:

(WebKit::WebPageProxy::restoreFromSessionStateData):

Restore a session on page creation if the parameters include it:

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::WebPage): Call restoreSession if session state exists.
(WebKit::WebPage::restoreSessionAndNavigateToCurrentItem):
(WebKit::WebPage::restoreSession):

  • WebProcess/WebPage/WebPage.h:
12:11 Changeset [75630] by andersca@apple.com
  • 9 edits in trunk/WebKit2

2011-01-12 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

Send Update and DidUpdate messages
https://bugs.webkit.org/show_bug.cgi?id=52316

  • UIProcess/DrawingAreaProxy.h: (WebKit::DrawingAreaProxy::update): Add empty stub.
  • UIProcess/DrawingAreaProxy.messages.in: Add Update message.
  • UIProcess/DrawingAreaProxyImpl.cpp: (WebKit::DrawingAreaProxyImpl::update): Just send back a DidUpdate message.
  • WebProcess/WebPage/DrawingArea.h: (WebKit::DrawingArea::didUpdate): Add empty stub.
  • WebProcess/WebPage/DrawingArea.messages.in: Add DidUpdate message.
  • WebProcess/WebPage/DrawingAreaImpl.cpp: (WebKit::DrawingAreaImpl::didUpdate): Call display().

(WebKit::DrawingAreaImpl::display):
Send an Update message.

12:07 Changeset [75629] by caseq@chromium.org
  • 2 edits in trunk/Source/WebCore

2011-01-12 Andrey Kosyakov <caseq@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: cookie table is multiplied after repeated switching to cookies tab in network item view
Only create cookiesTable is it hasn't been done yet.
https://bugs.webkit.org/show_bug.cgi?id=52303

  • inspector/front-end/ResourceCookiesView.js: (WebInspector.ResourceCookiesView.prototype.show):
11:41 Changeset [75628] by levin@chromium.org
  • 3 edits in trunk/Tools

check-webkit-style -- close_expression function doesn't work correctly.
https://bugs.webkit.org/show_bug.cgi?id=52272

Reviewed by Shinichiro Hamaji.

  • Scripts/webkitpy/style/checkers/cpp.py:

(Position.str): Added a way to convert it to a string which is useful in tests.
(Position.cmp): Added a way to compare Position which is useful in tests and
generally useful (for upcoming code changes).
(close_expression): Changed to use Position for input and output.
Also, fixed many bugs such as only working correctly for parenthesis,
not working correctly if given an offset in a line and not finding
the real closing element if there were multiple closing elements in
the same line.
(detect_functions): Adjusted due to the change in arguments for
close_expression.

  • Scripts/webkitpy/style/checkers/cpp_unittest.py:

(CppStyleTestBase.assert_positions_equal): Added a way to verify that
two positions are the same.
(CppStyleTest.test_position): Added tests for the str_ and cmp methods.
(CppStyleTest.test_close_expression): Added tests to catch the issues
that were fixed.

11:41 Changeset [75627] by andersca@apple.com
  • 8 edits in trunk/WebKit2

2011-01-12 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

Make the new drawing area actually draw something
https://bugs.webkit.org/show_bug.cgi?id=52314

  • Shared/UpdateInfo.cpp: (WebKit::UpdateInfo::encode): (WebKit::UpdateInfo::decode): Encode and decode newly added fields.
  • Shared/UpdateInfo.h: Add a vector of update rects and a handle for the sharable bitmap.
  • WebProcess/WebPage/DrawingAreaImpl.cpp: (WebKit::DrawingAreaImpl::display): Get the individual rects from the region and paint them into the shareable bitmap.
  • WebProcess/WebPage/WebPage.cpp: (WebKit::WebPage::mainFrameHasCustomRepresentation):
  • WebProcess/WebPage/WebPage.h: Add helper function.
  • WebProcess/WebPage/mac/ChunkedUpdateDrawingAreaMac.cpp: (WebKit::ChunkedUpdateDrawingArea::paintIntoUpdateChunk): Use helper function.
11:25 Changeset [75626] by commit-queue@webkit.org
  • 13 edits in trunk/Tools

2011-01-12 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r75576.
http://trac.webkit.org/changeset/75576
https://bugs.webkit.org/show_bug.cgi?id=52304

broke rebaseline-chromium-webkit-test (Requested by thakis on
#webkit).

  • Scripts/webkitpy/layout_tests/layout_package/printing_unittest.py:
  • Scripts/webkitpy/layout_tests/layout_package/test_expectations.py:
  • Scripts/webkitpy/layout_tests/layout_package/test_expectations_unittest.py:
  • Scripts/webkitpy/layout_tests/layout_package/test_runner.py:
  • Scripts/webkitpy/layout_tests/port/base.py:
  • Scripts/webkitpy/layout_tests/port/chromium.py:
  • Scripts/webkitpy/layout_tests/port/port_testcase.py:
  • Scripts/webkitpy/layout_tests/port/test.py:
  • Scripts/webkitpy/layout_tests/rebaseline_chromium_webkit_tests.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
  • Scripts/webkitpy/style/checkers/test_expectations.py:
  • Scripts/webkitpy/style/checkers/test_expectations_unittest.py:
11:24 Changeset [75625] by xan@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

2011-01-12 Xan Lopez <xlopez@igalia.com>

Reviewed by Martin Robinson.

Add new Yarr.h header to the list file.

  • GNUmakefile.am: ditto.
10:57 Changeset [75624] by andersca@apple.com
  • 2 edits
    2 adds in trunk/WebKit2

2011-01-12 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

Add UpdateInfo class
https://bugs.webkit.org/show_bug.cgi?id=52306

  • Shared/UpdateInfo.cpp: Added. (WebKit::UpdateInfo::encode): (WebKit::UpdateInfo::decode):
  • Shared/UpdateInfo.h: Added. (WebKit::UpdateInfo::UpdateInfo):
  • WebKit2.xcodeproj/project.pbxproj:
10:55 Changeset [75623] by mjs@apple.com
  • 2 edits in trunk/Tools

2011-01-12 Maciej Stachowiak <mjs@apple.com>

Reviewed by Darin Adler.

Add a no-build option to run-webkit-tests, to avoid building DRT/WKTR
https://bugs.webkit.org/show_bug.cgi?id=52307

  • Scripts/old-run-webkit-tests:
10:28 Changeset [75622] by pfeldman@chromium.org
  • 2 edits in branches/chromium/597/WebCore/inspector

Follow up to r75606, fixing compilation (explicit .get() in cast + transitive import).

10:28 Changeset [75621] by thakis@chromium.org
  • 1 edit
    2 deletes in trunk/LayoutTests

2011-01-12 Nico Weber <thakis@chromium.org>

Unreviewed, test expectations.

Remove now-unnecessary vista-specific baseline.
https://bugs.webkit.org/show_bug.cgi?id=52105

  • platform/chromium-win-vista/svg/custom/js-late-gradient-and-object-creation-expected.checksum: Removed.
  • platform/chromium-win-vista/svg/custom/js-late-gradient-and-object-creation-expected.png: Removed.
10:19 Changeset [75620] by andersca@apple.com
  • 4 edits in trunk/WebKit2

2011-01-12 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

More work on the new drawing area implementation
https://bugs.webkit.org/show_bug.cgi?id=52302

  • Platform/Region.cpp: (WebKit::Region::Shape::segments_end): segments_end could end up returning an iterator that is one element past the Vector data, so we can't use operator[] since that will assert.
  • WebProcess/WebPage/DrawingAreaImpl.cpp: (WebKit::DrawingAreaImpl::DrawingAreaImpl): Initialize the timer.

(WebKit::DrawingAreaImpl::setNeedsDisplay):
Unite the rect with the dirty region and schedule a display.

(WebKit::DrawingAreaImpl::setSize):
Tell the web page to resize.

(WebKit::DrawingAreaImpl::scheduleDisplay):
Start a display timer if needed.

(WebKit::DrawingAreaImpl::display):
Add a stub.

10:11 Changeset [75619] by mrobinson@webkit.org
  • 6 edits in trunk

2011-01-12 Koan-Sin Tan <koansin.tan@gmail.com>

Reviewed by Martin Robinson.

[Gtk] WebKitGtk+ doesn't build on Mac OS X 10.6
https://bugs.webkit.org/show_bug.cgi?id=50867

  • config.h: Don't disallow ctype for (OS(DARWIN) && PLATFORM(GTK)) Many GTK+ related files include <libintl.h> or <glib/gi18n-lib.h>. On Mac, <libintl.h> includes <xlocale.h>. <xclocale.h> includes <xlocale/_ctype.h>, which uses isacii().
  • platform/UUID.cpp: (WebCore::createCanonicalUUIDString): For OS(DARWIN) && !PLATFORM(CF) use the Chromium Linux UUID generation.

2011-01-12 Koan-Sin Tan <koansin.tan@gmail.com>

Reviewed by Martin Robinson.

[Gtk] WebKitGtk+ doesn't build on Mac OS X 10.6
https://bugs.webkit.org/show_bug.cgi?id=50867

Fixed library extention for Mac in webkitdirs.pm and type conflict in
TestNetscapePlugin.cpp

  • GNUmakefile.am: Don't use AppKit when compiling TestNetscapePlugin for GTK+
  • Scripts/webkitdirs.pm:
09:57 Changeset [75618] by jschuh@chromium.org
  • 2 edits in trunk/LayoutTests

2011-01-12 Justin Schuh <jschuh@chromium.org>

Unreviewed expectatinos update.

Excluding SVG test for sporadic image failure.

  • platform/chromium/test_expectations.txt:
09:45 Changeset [75617] by mrobinson@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

2011-01-12 Martin Robinson <mrobinson@igalia.com>

Missing Source/JavaScriptCore/assembler/MacroAssemblerARMv7.h in WebKitGtk tarball
https://bugs.webkit.org/show_bug.cgi?id=52299

  • GNUmakefile.am: Fix the GTK+ build on ARMv7 by including missing source files in the source list.
09:31 Changeset [75616] by jschuh@chromium.org
  • 2 edits in trunk/LayoutTests

2011-01-12 Justin Schuh <jschuh@chromium.org>

Unreviewed chromium expectations update.

Added chromium win to exclusion for bug 48164.

  • platform/chromium/test_expectations.txt:
09:09 Changeset [75615] by simon.fraser@apple.com
  • 8 edits in trunk

2011-01-12 Simon Fraser <simon.fraser@apple.com>

Reviewed by Eric Seidel.

[CSS Gradients] One color stop should not paint anything for linear-gradient
https://bugs.webkit.org/show_bug.cgi?id=52259

Share color stop parsing code between linear and radial gradients.
Consider the gradient invalid if it has less than two color stops.

  • css/CSSParser.cpp: (WebCore::CSSParser::parseLinearGradient): (WebCore::CSSParser::parseRadialGradient): (WebCore::CSSParser::parseGradientColorStops):
  • css/CSSParser.h:
09:05 Changeset [75614] by bfulgham@webkit.org
  • 1 edit
    1 move in trunk/WebKit2

Unreviewed build fix after r75527. Rename 'BackingStoreCairo.cpp'
to 'ShareableBitmapCairo.cpp' to match VS project changes.

  • Shared/cairo/BackingStoreCairo.cpp: Removed.
  • Shared/cairo/ShareableBitmapCairo.cpp: Copied from Shared/cairo/BackingStoreCairo.cpp.
08:42 Changeset [75613] by mnaganov@chromium.org
  • 8 edits
    1 add in trunk

2011-01-12 Mikhail Naganov <mnaganov@chromium.org>

Reviewed by Pavel Feldman.

Add check-inspector-strings script.
Fix inconsistencies in Inspector strings found by the script.

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

  • English.lproj/localizedStrings.js:
  • inspector/front-end/BreakpointsSidebarPane.js: (WebInspector.EventListenerBreakpointsSidebarPane):
  • inspector/front-end/ProfilesPanel.js: (WebInspector.ProfilesPanel.prototype.setRecordingProfile):
  • inspector/front-end/Resource.js: (WebInspector.Resource.Type.toString):
  • inspector/front-end/ResourcesPanel.js: (WebInspector.ResourceRevisionTreeElement):
  • inspector/front-end/utilities.js: ():
  • Scripts/check-inspector-strings: Added.
08:21 Changeset [75612] by yurys@chromium.org
  • 3 edits in trunk/Source/WebCore

2011-01-12 Yury Semikhatsky <yurys@chromium.org>

Unreviewed. Fix WML compilation on Mac.

  • wml/WMLInputElement.cpp: (WebCore::WMLInputElement::setValue): (WebCore::WMLInputElement::setValueForUser): (WebCore::WMLInputElement::initialize): (WebCore::WMLInputElement::validateInputMask):
  • wml/WMLSelectElement.cpp: (WebCore::WMLSelectElement::listBoxSelectItem):
08:07 Changeset [75611] by jschuh@chromium.org
  • 2 edits in trunk/Source/WebCore

2011-01-12 Justin Schuh <jschuh@chromium.org>

Unreviewed build fix for debug strict compiles.

Use fprintf for debug statements instead of fwrite, which caused compile failures
in debug strict mode because of failure to check the fwrite return value.

No behavior changed.

  • rendering/CounterNode.cpp: (WebCore::showTreeAndMark):
08:06 Changeset [75610] by aroben@apple.com
  • 2 edits in trunk/Tools

Advertise the .testnetscape file extension on Windows

Fixes <http://webkit.org/b/52298>
plugins/no-mime-with-valid-extension.html fails on Windows

Reviewed by Dan Bernstein.

  • DumpRenderTree/TestNetscapePlugIn/win/TestNetscapePlugin.rc: Added a

FileExtents field to map the .testnetscape file extension to the
application/x-webkit-test-netscape MIME type, just as we do on Mac.

07:41 Changeset [75609] by ossy@webkit.org
  • 1 edit
    15 adds in trunk/LayoutTests

Unreviewed.

Add Qt specific expected results for new passing tests introduced in r74954 and r74743.
Compared to Mac png files and Mac expected files.

  • platform/qt/fast/css-generated-content/table-before-after-child-add-expected.checksum: Added.
  • platform/qt/fast/css-generated-content/table-before-after-child-add-expected.png: Added.
  • platform/qt/fast/css-generated-content/table-before-after-child-add-expected.txt: Added.
  • platform/qt/fast/css-generated-content/table-cell-before-after-child-add-expected.checksum: Added.
  • platform/qt/fast/css-generated-content/table-cell-before-after-child-add-expected.png: Added.
  • platform/qt/fast/css-generated-content/table-cell-before-after-child-add-expected.txt: Added.
  • platform/qt/fast/css-generated-content/table-row-before-after-child-add-expected.checksum: Added.
  • platform/qt/fast/css-generated-content/table-row-before-after-child-add-expected.png: Added.
  • platform/qt/fast/css-generated-content/table-row-before-after-child-add-expected.txt: Added.
  • platform/qt/fast/css-generated-content/table-table-before-after-child-add-expected.checksum: Added.
  • platform/qt/fast/css-generated-content/table-table-before-after-child-add-expected.png: Added.
  • platform/qt/fast/css-generated-content/table-table-before-after-child-add-expected.txt: Added.
  • platform/qt/fast/multicol/span/clone-anonymous-block-non-inline-child-crash-expected.checksum: Added.
  • platform/qt/fast/multicol/span/clone-anonymous-block-non-inline-child-crash-expected.png: Added.
  • platform/qt/fast/multicol/span/clone-anonymous-block-non-inline-child-crash-expected.txt: Added.
07:33 Changeset [75608] by jschuh@chromium.org
  • 2 edits in trunk/LayoutTests

2011-01-12 Justin Schuh <jschuh@chromium.org>

Unreviewed expectations update for chromium.

Exclude failing test from r75064.

  • platform/chromium/test_expectations.txt:
07:08 Changeset [75607] by philn@webkit.org
  • 2 edits in trunk/LayoutTests

2011-01-12 Philippe Normand <pnormand@igalia.com>

Unreviewed, skip failing test on GTK.

  • platform/gtk/Skipped: Skip editing/input/page-up-down-scrolls.html
06:54 Changeset [75606] by pfeldman@chromium.org
  • 3 edits in branches/chromium/597/WebCore/inspector

Merge 74637 - 2010-12-24 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Darin Adler.

inspector/debugger-step-out.html crashing intermittently in the bots
https://bugs.webkit.org/show_bug.cgi?id=50868

Use RefPtr rather instead of raw pointers when keeping references
to refcounted objects.

  • inspector/InspectorCSSAgent.cpp: (WebCore::InspectorCSSAgent::getStylesForNode2): (WebCore::InspectorCSSAgent::getComputedStyleForNode2):
  • inspector/InspectorStyleSheet.cpp: (WebCore::InspectorStyle::create): (WebCore::InspectorStyle::InspectorStyle): (WebCore::InspectorStyle::buildObjectForStyle): (WebCore::InspectorStyle::setPropertyText): (WebCore::InspectorStyle::toggleProperty): (WebCore::InspectorStyle::styleText): (WebCore::InspectorStyle::populateAllProperties): (WebCore::InspectorStyle::replacePropertyInStyleText): (WebCore::InspectorStyleSheet::create): (WebCore::InspectorStyleSheet::InspectorStyleSheet): (WebCore::InspectorStyleSheetForInlineStyle::create): (WebCore::InspectorStyleSheetForInlineStyle::InspectorStyleSheetForInlineStyle): (WebCore::InspectorStyleSheetForInlineStyle::getStyleAttributeRanges):
  • inspector/InspectorStyleSheet.h: (WebCore::InspectorStyle::cssStyle): (WebCore::InspectorStyleSheet::pageStyleSheet):

TBR=yurys@chromium.org
BUG=69296
Review URL: http://codereview.chromium.org/6205007

06:42 Changeset [75605] by philn@webkit.org
  • 2 edits in trunk/WebKit/gtk

2011-01-12 Philippe Normand <pnormand@igalia.com>

Unreviewed build fix for 32-bits Release. Include GRefPtrGtk.h so
the GdkCursor templates are found and used at runtime.

  • WebCoreSupport/FullscreenVideoController.cpp:
05:06 Changeset [75604] by yurys@chromium.org
  • 35 edits in trunk

2010-12-29 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

inspector/timeline-network-resource.html fails when run twice
https://bugs.webkit.org/show_bug.cgi?id=37394

Send didReceiveResponse notification to the timeline agent from ResourceLoadNotifier
instead of ResourceLoader::didReceiveResponse to cover the cases when resources
are loaded from memory cache.

Network notifications are now send to InspectorInstrumentation which dispatches
them to interested agents and InspectorController.
fix

  • inspector/timeline-network-resource-expected.txt:
  • inspector/timeline-network-resource.html:
  • inspector/timeline-network-resource.js:

2010-12-29 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

inspector/timeline-network-resource.html fails when run twice
https://bugs.webkit.org/show_bug.cgi?id=37394

Send didReceiveResponse notification to the timeline agent from ResourceLoadNotifier
instead of ResourceLoader::didReceiveResponse to cover the cases when resources
are loaded from memory cache.

Network notifications are now sent to InspectorInstrumentation which dispatches
them to interested agents and InspectorController.

  • inspector/Inspector.idl:
  • inspector/InspectorApplicationCacheAgent.cpp: (WebCore::InspectorApplicationCacheAgent::didReceiveManifestResponse):
  • inspector/InspectorController.cpp: (WebCore::InspectorController::InspectorController): (WebCore::InspectorController::didCommitLoad): (WebCore::InspectorController::willSendRequest): (WebCore::InspectorController::didReceiveResponse): (WebCore::InspectorController::didFailLoading): (WebCore::InspectorController::resourceRetrievedByXMLHttpRequest):
  • inspector/InspectorController.h:
  • inspector/InspectorInstrumentation.cpp: (WebCore::InspectorInstrumentation::identifierForInitialRequestImpl): (WebCore::InspectorInstrumentation::willSendRequestImpl): (WebCore::InspectorInstrumentation::markResourceAsCachedImpl): (WebCore::InspectorInstrumentation::didLoadResourceFromMemoryCacheImpl): (WebCore::InspectorInstrumentation::willReceiveResourceResponseImpl): (WebCore::InspectorInstrumentation::didReceiveContentLengthImpl): (WebCore::InspectorInstrumentation::didFinishLoadingImpl): (WebCore::InspectorInstrumentation::didFailLoadingImpl): (WebCore::InspectorInstrumentation::resourceRetrievedByXMLHttpRequestImpl): (WebCore::InspectorInstrumentation::scriptImportedImpl): (WebCore::InspectorInstrumentation::retrieveResourceAgent):
  • inspector/InspectorInstrumentation.h: (WebCore::InspectorInstrumentation::identifierForInitialRequest): (WebCore::InspectorInstrumentation::willSendRequest): (WebCore::InspectorInstrumentation::markResourceAsCached): (WebCore::InspectorInstrumentation::didLoadResourceFromMemoryCache): (WebCore::InspectorInstrumentation::willReceiveResourceResponse): (WebCore::InspectorInstrumentation::didReceiveContentLength): (WebCore::InspectorInstrumentation::didFinishLoading): (WebCore::InspectorInstrumentation::didFailLoading): (WebCore::InspectorInstrumentation::resourceRetrievedByXMLHttpRequest): (WebCore::InspectorInstrumentation::scriptImported):
  • inspector/InspectorResourceAgent.cpp: (WebCore::InspectorResourceAgent::identifierForInitialRequest):
  • inspector/InspectorResourceAgent.h:
  • inspector/InspectorTimelineAgent.cpp: (WebCore::InspectorTimelineAgent::willSendResourceRequest):
  • inspector/InspectorTimelineAgent.h:
  • inspector/TimelineRecordFactory.cpp: (WebCore::TimelineRecordFactory::createResourceSendRequestData):
  • inspector/TimelineRecordFactory.h:
  • inspector/front-end/NetworkManager.js: (WebInspector.NetworkManager.prototype.identifierForInitialRequest):
  • inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel.prototype.addRecordToTimeline):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::loadedResourceFromMemoryCache):
  • loader/ResourceLoadNotifier.cpp: (WebCore::ResourceLoadNotifier::didReceiveResponse): (WebCore::ResourceLoadNotifier::didFailToLoad): (WebCore::ResourceLoadNotifier::assignIdentifierToInitialRequest): (WebCore::ResourceLoadNotifier::dispatchWillSendRequest): (WebCore::ResourceLoadNotifier::dispatchDidReceiveResponse): (WebCore::ResourceLoadNotifier::dispatchDidReceiveContentLength): (WebCore::ResourceLoadNotifier::dispatchDidFinishLoading): (WebCore::ResourceLoadNotifier::sendRemainingDelegateMessages):
  • loader/ResourceLoader.cpp: (WebCore::ResourceLoader::didReceiveResponse):
  • loader/appcache/ApplicationCacheGroup.cpp: (WebCore::ApplicationCacheGroup::createResourceHandle): (WebCore::ApplicationCacheGroup::didReceiveResponse): (WebCore::ApplicationCacheGroup::didReceiveData): (WebCore::ApplicationCacheGroup::didFinishLoading): (WebCore::ApplicationCacheGroup::didFail):
  • loader/appcache/ApplicationCacheGroup.h:
  • workers/DefaultSharedWorkerRepository.cpp: (WebCore::SharedWorkerScriptLoader::notifyFinished):
  • workers/Worker.cpp: (WebCore::Worker::notifyFinished):
  • workers/WorkerContext.cpp: (WebCore::WorkerContext::importScripts):
  • xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::didFinishLoading):

2010-12-29 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

inspector/timeline-network-resource.html fails when run twice
https://bugs.webkit.org/show_bug.cgi?id=37394

Send didReceiveResponse notification to the timeline agent from ResourceLoadNotifier
instead of ResourceLoader::didReceiveResponse to cover the cases when resources
are loaded from memory cache.

Network notifications are now sent to InspectorInstrumentation which dispatches
them to interested agents and InspectorController.

  • src/SharedWorkerRepository.cpp: (WebCore::SharedWorkerScriptLoader::notifyFinished):
  • src/WebDevToolsAgentImpl.cpp: (WebKit::WebDevToolsAgentImpl::mainFrame): (WebKit::WebDevToolsAgentImpl::identifierForInitialRequest): (WebKit::WebDevToolsAgentImpl::willSendRequest): (WebKit::WebDevToolsAgentImpl::didReceiveData): (WebKit::WebDevToolsAgentImpl::didReceiveResponse): (WebKit::WebDevToolsAgentImpl::didFinishLoading): (WebKit::WebDevToolsAgentImpl::didFailLoading):
  • src/WebDevToolsAgentImpl.h:
05:03 Changeset [75603] by jorlow@chromium.org
  • 1 edit
    1 add in trunk/LayoutTests

2011-01-12 Jeremy Orlow <jorlow@chromium.org>

Add forgotten file.

  • storage/indexeddb/transaction-read-only-expected.txt: Added.
04:57 Changeset [75602] by pvarga@webkit.org
  • 12 edits
    1 add in trunk/Source/JavaScriptCore

2011-01-12 Peter Varga <pvarga@webkit.org>

Reviewed by Gavin Barraclough.

Add Yarr.h to YARR
https://bugs.webkit.org/show_bug.cgi?id=51021

Move other common constants and functions from YARR's different files
to Yarr.h.
Use Yarr.h header instead of including other YARR headers where it
is possible.

  • JavaScriptCore.gypi:
  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:
  • JavaScriptCore.xcodeproj/project.pbxproj:
  • runtime/RegExp.cpp:
  • yarr/Yarr.h: Added.
  • yarr/YarrInterpreter.cpp:
  • yarr/YarrInterpreter.h:
  • yarr/YarrJIT.cpp: (JSC::Yarr::jitCompile): (JSC::Yarr::execute):
  • yarr/YarrJIT.h:
  • yarr/YarrParser.h:
  • yarr/YarrPattern.cpp: (JSC::Yarr::YarrPattern::compile): (JSC::Yarr::YarrPattern::YarrPattern):
  • yarr/YarrPattern.h:
04:55 Changeset [75601] by podivilov@chromium.org
  • 2 edits in trunk/Source/WebCore

2011-01-12 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: do not replace all tabs with spaces when editing script source
https://bugs.webkit.org/show_bug.cgi?id=52291

  • inspector/front-end/SourceFrame.js: (WebInspector.SourceFrame.prototype.setContent): (WebInspector.SourceFrame.prototype._editLine):
04:29 Changeset [75600] by abarth@webkit.org
  • 2 edits in trunk/Tools

2011-01-12 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Make webkit-patch support subdirectories for SVN checkouts
https://bugs.webkit.org/show_bug.cgi?id=52261

os.relpath does not exist on Python 2.5, so we need to call our version
of the API.

  • Scripts/webkitpy/common/checkout/scm.py:
04:22 Changeset [75599] by commit-queue@webkit.org
  • 2 edits in trunk/WebKit/qt

2011-01-12 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r74344.
http://trac.webkit.org/changeset/74344
https://bugs.webkit.org/show_bug.cgi?id=52290

broke QWebPluginFactory for unusual mime-types (Requested by
kling on #webkit).

  • WebCoreSupport/FrameLoaderClientQt.cpp: (WebCore::FrameLoaderClientQt::createPlugin):
03:28 Changeset [75598] by ossy@webkit.org
  • 2 edits in trunk/LayoutTests

Unreviewed.

[Qt] plugins/no-mime-with-valid-extension.html fails
https://bugs.webkit.org/show_bug.cgi?id=52288

  • platform/qt/Skipped: plugins/no-mime-with-valid-extension.html added until fix.
03:16 Changeset [75597] by ossy@webkit.org
  • 12 edits
    1 delete in trunk/Source/JavaScriptCore

Unreviewed, rolling out r75595.
http://trac.webkit.org/changeset/75595
https://bugs.webkit.org/show_bug.cgi?id=52286

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2011-01-12
It broke fast/regex/pcre-test-1.html (Requested by Ossy on
#webkit).

  • JavaScriptCore.gypi:
  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:
  • JavaScriptCore.xcodeproj/project.pbxproj:
  • runtime/RegExp.cpp:
  • yarr/Yarr.h: Removed.
  • yarr/YarrInterpreter.cpp:
  • yarr/YarrInterpreter.h:
  • yarr/YarrJIT.cpp:

(JSC::Yarr::jitCompile):

  • yarr/YarrJIT.h:

(JSC::Yarr::execute):

  • yarr/YarrParser.h:
  • yarr/YarrPattern.cpp:

(JSC::Yarr::compile):
(JSC::Yarr::YarrPattern::YarrPattern):

  • yarr/YarrPattern.h:
02:40 Changeset [75596] by jorlow@chromium.org
  • 22 edits
    1 add in trunk

2011-01-11 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Nate Chapin.

Enforce READ_ONLY transactions in IndexedDB
https://bugs.webkit.org/show_bug.cgi?id=52229

Fix existing errors in tests + add a new test for
this issue.

  • storage/indexeddb/cursor-delete-expected.txt:
  • storage/indexeddb/cursor-delete.html:
  • storage/indexeddb/cursor-index-delete-expected.txt:
  • storage/indexeddb/cursor-index-delete.html:
  • storage/indexeddb/database-quota-expected.txt:
  • storage/indexeddb/database-quota.html:
  • storage/indexeddb/objectstore-basics-expected.txt:
  • storage/indexeddb/objectstore-basics.html:
  • storage/indexeddb/objectstore-removeobjectstore-expected.txt:
  • storage/indexeddb/objectstore-removeobjectstore.html:
  • storage/indexeddb/transaction-basics-expected.txt:
  • storage/indexeddb/transaction-basics.html:
  • storage/indexeddb/transaction-crash-on-abort-expected.txt:
  • storage/indexeddb/transaction-crash-on-abort.html:
  • storage/indexeddb/transaction-read-only.html: Added.

2011-01-11 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Nate Chapin.

Enforce READ_ONLY transactions in IndexedDB
https://bugs.webkit.org/show_bug.cgi?id=52229

Add READ_ONLY_ERR and return it whenever delete or put
are called. Note that IDBCursor's delete and update
are simply aliases for delete/put, which is why no
extra code is required to handle those cases.

Test: storage/indexeddb/transaction-read-only.html

  • dom/ExceptionCode.cpp:
  • storage/IDBCursorBackendImpl.cpp: (WebCore::IDBCursorBackendImpl::deleteFunction):
  • storage/IDBDatabase.idl:
  • storage/IDBDatabaseException.h:
  • storage/IDBDatabaseException.idl:
  • storage/IDBObjectStoreBackendImpl.cpp: (WebCore::IDBObjectStoreBackendImpl::put): (WebCore::IDBObjectStoreBackendImpl::deleteFunction): (WebCore::IDBObjectStoreBackendImpl::deleteIndex):
02:32 Changeset [75595] by pvarga@webkit.org
  • 12 edits
    1 add in trunk/Source/JavaScriptCore

2011-01-12 Peter Varga <pvarga@webkit.org>

Reviewed by Gavin Barraclough.

Add Yarr.h to YARR
https://bugs.webkit.org/show_bug.cgi?id=51021

Move other common constants and functions from YARR's different files
to Yarr.h.
Use Yarr.h header instead of including other YARR headers where it
is possible.

  • JavaScriptCore.gypi:
  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:
  • JavaScriptCore.xcodeproj/project.pbxproj:
  • runtime/RegExp.cpp:
  • yarr/Yarr.h: Added.
  • yarr/YarrInterpreter.cpp:
  • yarr/YarrInterpreter.h:
  • yarr/YarrJIT.cpp: (JSC::Yarr::jitCompile): (JSC::Yarr::execute):
  • yarr/YarrJIT.h:
  • yarr/YarrParser.h:
  • yarr/YarrPattern.cpp: (JSC::Yarr::YarrPattern::compile): (JSC::Yarr::YarrPattern::YarrPattern):
  • yarr/YarrPattern.h:
01:38 Changeset [75594] by pfeldman@chromium.org
  • 1 edit in branches/chromium/597/WebCore/inspector/front-end/ResourceManager.js

Merge 75503 - 2011-01-11 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: redirects are lost from the network panel upon navigation.
https://bugs.webkit.org/show_bug.cgi?id=52210

  • inspector/front-end/NetworkManager.js: (WebInspector.NetworkManager.prototype.didCommitLoadForFrame):

TBR=pfeldman@chromium.org
BUG=69359

01:36 Changeset [75593] by pfeldman@chromium.org
  • 2 edits in branches/chromium/597/WebCore/inspector/front-end

Merge 75498 - 2011-01-11 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: "Audit present state" always disabled
https://bugs.webkit.org/show_bug.cgi?id=52199

Removed all traces of resource tracking checks, as we have it no more.

  • English.lproj/localizedStrings.js:
  • inspector/front-end/AuditLauncherView.js: (WebInspector.AuditLauncherView.prototype._createLauncherUI):
  • inspector/front-end/AuditsPanel.js:

TBR=apavlov@chromium.org
BUG=69241

01:32 Changeset [75592] by pfeldman@chromium.org
  • 2 edits in branches/chromium/597/WebCore

Merge 75485 - 2011-01-11 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: [REGRESSION] Load timing is requested
for all resources (not only for the main one).
https://bugs.webkit.org/show_bug.cgi?id=51749

  • inspector/InspectorController.cpp: (WebCore::InspectorController::willSendRequest):
  • loader/ResourceLoadNotifier.cpp: (WebCore::ResourceLoadNotifier::dispatchWillSendRequest):

TBR=pfeldman@chromium.org
BUG=69229

01:27 Changeset [75591] by philn@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-01-11 Philippe Normand <pnormand@igalia.com>

Reviewed by Eric Seidel.

[GTK] minimal build breaks in RenderThemeGtk3
https://bugs.webkit.org/show_bug.cgi?id=52202

  • platform/gtk/RenderThemeGtk3.cpp: Added VIDEO guards around initMediaColors().
01:08 Changeset [75590] by philn@webkit.org
  • 3 edits
    1 copy in trunk/Source/WebCore

2011-01-11 Philippe Normand <pnormand@igalia.com>

Reviewed by Eric Carlson.

[GStreamer] PlatformVideoWindowMac implementation
https://bugs.webkit.org/show_bug.cgi?id=50770

This new implementation will be used on Mac OS to display the
fullscreen video window with the GStreamer-based media player.

  • platform/graphics/gstreamer/GStreamerGWorld.cpp: (WebCore::gstGWorldSyncMessageCallback):
  • platform/graphics/gstreamer/PlatformVideoWindow.h: (WebCore::PlatformVideoWindow::window):
  • platform/graphics/gstreamer/PlatformVideoWindowMac.mm: (PlatformVideoWindow::PlatformVideoWindow): (PlatformVideoWindow::~PlatformVideoWindow): (PlatformVideoWindow::prepareForOverlay):
00:37 Changeset [75589] by ojan@chromium.org
  • 3 edits in trunk/Websites/bugs.webkit.org

2011-01-12 Ojan Vafai <ojan@chromium.org>

Reviewed by Adam Barth.

fix wrapping bug in expansion lines in the code review tool
https://bugs.webkit.org/show_bug.cgi?id=52270

  • PrettyPatch/PrettyPatch.rb:
  • code-review.js:
Note: See TracTimeline for information about the timeline view.