Timeline


and

12/27/10:

21:33 Changeset [74708] by commit-queue@webkit.org

2010-12-27 Helder Correia <helder@sencha.com>

Reviewed by Ariya Hidayat.

ContextShadow should use AffineTransform instead of TransformationMatrix
https://bugs.webkit.org/show_bug.cgi?id=51661

ContextShadow needs the CTM to make sure shadows are not affected by
transformations when drawing on a canvas. AffineTransform is sufficient
in this case.

Existing tests: fast/canvas/canvas*shadow*html

  • platform/graphics/ContextShadow.cpp: (WebCore::ContextShadow::mustUseContextShadow): (WebCore::ContextShadow::adjustBlurDistance): (WebCore::ContextShadow::calculateLayerBoundingRect):
  • platform/graphics/ContextShadow.h:
  • platform/graphics/cairo/ContextShadowCairo.cpp: (WebCore::ContextShadow::getTransformationMatrixFromContext):
  • platform/graphics/qt/ContextShadowQt.cpp: (WebCore::ContextShadow::getTransformationMatrixFromContext):
21:15 Changeset [74707] by commit-queue@webkit.org

2010-12-27 Ryuan Choi <ryuan.choi@samsung.com>

Reviewed by David Levin.

[EFL] Change path of eflsymbols.filter
https://bugs.webkit.org/show_bug.cgi?id=51659

  • Sources/cmake/OptionsEfl.cmake:
18:05 Changeset [74706] by dbates@webkit.org

2010-12-27 Daniel Bates <dbates@rim.com>

Rubber-stamped by Martin Robinson.

Append the directory Tools/DumpRenderTree to the list of directories that
do-webcore-rename should search so that it can rename DRT files and
source code.

  • Scripts/do-webcore-rename:
17:40 Changeset [74705] by mrobinson@webkit.org

2010-12-27 Martin Robinson <mrobinson@igalia.com>

Reviewed by Daniel Bates.

Try to fix my previous commit for Mac by stepping up the precision of pi even more
using Math.PI.

  • fast/canvas/canvas-save-restore-with-path.js: Use Math.PI instead of a hard-coded estimation.
17:39 Changeset [74704] by weinig@apple.com

WebKit2 snapshots should contain accelerated content
<rdar://problem/8807393>
https://bugs.webkit.org/show_bug.cgi?id=51660

Reviewed by Anders Carlsson.

When creating a snapshot, flatten the layer tree for painting so that
accelerated content is included in the snapshot.

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::snapshotInViewCoordinates):
(WebKit::WebPage::snapshotInDocumentCoordinates):

17:03 Changeset [74703] by dbates@webkit.org

2010-12-27 Daniel Bates <dbates@rim.com>

Attempt to fix the WebKit2 Windows build after changeset 74698
<http://trac.webkit.org/changeset/74698> (bug #51656).

Add "..\Shared\API\c\WKImage.h" to WebKit2Generated.make.

  • win/WebKit2Generated.make:
16:50 Changeset [74702] by kbr@google.com

2010-12-27 Kenneth Russell <kbr@google.com>

Unreviewed, Chromium test expectations update. Generalize
suppressions for Chromium bugs 67732 and 67139 to cover Linux
Debug.

  • platform/chromium/test_expectations.txt:
16:44 Changeset [74701] by dbates@webkit.org

2010-12-27 Daniel Bates <dbates@rim.com>

Fix the first sentence of the change log message for changeset 74700
<http://trac.webkit.org/changeset/74700> (bug #51634).

As pointed out by Dan Bernstein, it should read: "Rename HTMLFrameElementBase::getMarginWidth()
and HTMLFrameElementBase::getMarginHeight() to HTMLFrameElementBase::marginWidth() and
HTMLFrameElementBase::marginHeight(), respectively."

16:21 Changeset [74700] by dbates@webkit.org

2010-12-27 Daniel Bates <dbates@rim.com>

Reviewed by Antonio Gomes.

Clean up: Rename HTMLFrameElementBase::getMarginWidth() and HTMLFrameElementBase::getMarginHeight()
https://bugs.webkit.org/show_bug.cgi?id=51634

Rename HTMLFrameElementBase::getMarginWidth() and HTMLFrameElementBase::getMarginHeight()
to HTMLFrameElementBase::marginHeight() and HTMLFrameElementBase::marginHeight(), respectively.
This will make the names of these getters consistent with the naming convention we use for
getters.

  • html/HTMLBodyElement.cpp: (WebCore::HTMLBodyElement::insertedIntoDocument):
  • html/HTMLFrameElementBase.h: (WebCore::HTMLFrameElementBase::marginWidth): Renamed; Formerly getMarginWidth(). (WebCore::HTMLFrameElementBase::marginHeight): Renamed; Formerly getMarginHeight().
  • loader/SubframeLoader.cpp: (WebCore::SubframeLoader::loadSubframe):
  • page/FrameView.cpp: (WebCore::FrameView::init):
  • rendering/RenderEmbeddedObject.cpp: (WebCore::RenderEmbeddedObject::viewCleared): Also renamed variables marginw and marginh to marginWidth and marginHeight, respectively. This makes the names of these variables consistent with the names of similar variables in SubframeLoader::loadSubframe() and FrameView::init().
  • rendering/RenderFrame.cpp: (WebCore::RenderFrame::viewCleared): Ditto.
16:17 Changeset [74699] by mrobinson@webkit.org

2010-12-27 Martin Robinson <mrobinson@igalia.com>

Reviewed by Daniel Bates.

[Cairo] fast/canvas/canvas-save-restore-with-path.html fails with some versions of Cairo
https://bugs.webkit.org/show_bug.cgi?id=51655

Increase the precision of pi used in this test case. It appears that
some Cairo versions or processors are sensitive to rounding errors here.

  • fast/canvas/canvas-save-restore-with-path.js: Increase pi precision.
16:01 Changeset [74698] by weinig@apple.com

WebKit2: Add API for getting snapshots
https://bugs.webkit.org/show_bug.cgi?id=51656

Reviewed by Anders Carlsson.

  • Shared/API/c/WKBase.h:
  • Shared/API/c/WKImage.cpp: Added.

(WKImageGetTypeID):
(WKImageCreate):
(WKImageGetSize):

  • Shared/API/c/WKImage.h: Added.
  • Shared/WebImage.cpp: Added.

(WebKit::WebImage::create):
(WebKit::WebImage::size):

  • Shared/WebImage.h: Added.

(WebKit::WebImage::backingStore):
(WebKit::WebImage::WebImage):
(WebKit::WebImage::type):
Add a general purpose API image class.

  • Shared/API/c/cg: Added.
  • Shared/API/c/cg/WKImageCG.cpp: Added.

(WKImageCreateCGImage):

  • Shared/API/c/cg/WKImageCG.h: Added.

Add function to convert a WKImageRef to a CGImageRef for platforms that use CG.

  • Shared/API/c/WKSharedAPICast.h:

(WebKit::toFloatRect):
(WebKit::toIntSize):
(WebKit::toIntPoint):
(WebKit::toIntRect):
(WebKit::toImageOptions):
Make conversion functions that are ambiguous more explicit.

  • Shared/APIObject.h:
  • Shared/BackingStore.h:

(WebKit::BackingStore::isBackedBySharedMemory):

  • Shared/ImageOptions.h: Added.
  • Shared/UserMessageCoders.h:

(WebKit::UserMessageEncoder::baseEncode):
(WebKit::UserMessageDecoder::baseDecode):
Allow a WKImageRef to be passed in user messages. Right now, it only
works if the image is sharable.

  • UIProcess/API/C/WKPage.cpp:

(WKPageScaleWebView):

  • UIProcess/WebUIClient.cpp:

(WebKit::WebUIClient::windowFrame):
Use the more explicit conversion function name.

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

(WKBundlePageCreateSnapshotInViewCoordinates):
(WKBundlePageCreateSnapshotInDocumentCoordinates):

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

(WKBundlePageOverlaySetNeedsDisplay):

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::snapshotInViewCoordinates):
(WebKit::WebPage::snapshotInDocumentCoordinates):
Add function to take the snapshot.

  • WebProcess/WebPage/WebPage.h:
  • WebKit2.pro:
  • WebKit2.xcodeproj/project.pbxproj:
  • win/WebKit2.vcproj:

Add new files.

14:48 Changeset [74697] by mrobinson@webkit.org

2010-12-27 Martin Robinson <mrobinson@igalia.com>

Reviewed by Ryosuke Niwa.

[GTK] EditorClient::generateEditorCommands queues up "null string" commands
https://bugs.webkit.org/show_bug.cgi?id=51569

Prevent adding "null string" editor commands by first checking whether or not
the command string is null in generateEditorCommands before appending it to the
list of pending editor command strings.

  • WebCoreSupport/EditorClientGtk.cpp: (WebKit::EditorClient::generateEditorCommands): Check for the null string (no command for this key combination) before appending a command string to the list of pending editor commands. (WebKit::EditorClient::executePendingEditorCommands): Add an ASSERT which detects null command strings.
14:37 Changeset [74696] by mrobinson@webkit.org

2010-12-27 Martin Robinson <mrobinson@igalia.com>

Reviewed by Daniel Bates.

[GTK] RenderThemeGtk::m_HScale and RenderThemeGtk::m_VScale are not initialized
https://bugs.webkit.org/show_bug.cgi?id=51654

Properly initialize slider members of RenderThemeGtk. This can lead to test crashes
on some debug configurations.

No new tests. This is already covered by current slider tests. It isn't crashing on
the bots, but it does crash locally.

  • platform/gtk/RenderThemeGtk.cpp: (WebCore::RenderThemeGtk::RenderThemeGtk): Properly initialize members.
13:41 Changeset [74695] by darin@apple.com

2010-12-27 Jake Helfert <jake@jakeonthenet.com>

Reviewed and reworked by Darin Adler.

Building WebKit with Visual Studio 2010 fails due
to ambiguous assignment operator errors.
https://bugs.webkit.org/show_bug.cgi?id=51116

  • wtf/NullPtr.h: Added a HAVE(NULLPTR) definition for use with Platform.h HAVE macro, and included the Visual Studio 2010 compiler as one of the ones that has nullptr.
  • wtf/NullPtr.cpp: Updated condition to match.


  • wtf/PassOwnArrayPtr.h: Don't include the operator=(nullptr_t) overload if we are compiling in loose mode and the compiler has nullptr, because assignment of 0 will otherwise encounter ambiguitity with this overload and the overload for loose mode that takes a raw pointer. The conditional can be removed when we get rid of loose mode.
  • wtf/PassOwnPtr.h: Ditto.
  • wtf/PassRefPtr.h: Don't include the operator=(nullptr_t) overload if the compiler has nullptr, because assignment of 0 would be ambiguous with the overload that takes a raw pointer. The conditional can be removed if we ever decide we no longer need to support assigning 0, but might need a way to catch that usage on older compilers.
  • wtf/RefPtr.h: Ditto.
  • wtf/RetainPtr.h: Ditto
  • JavaScriptCore.xcodeproj/project.pbxproj: Added NullPtr.cpp, accidentally omitted when the file was first added.
13:13 Changeset [74694] by eric@webkit.org

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

Reviewed by Adam Barth.

build-webkit should detect missing Java SDK and explain how to install it
https://bugs.webkit.org/show_bug.cgi?id=51651

  • Scripts/build-webkit:
13:04 Changeset [74693] by weinig@apple.com

WebKit2: Add additional API for managing databases
https://bugs.webkit.org/show_bug.cgi?id=51629

Reviewed by Anders Carlsson.

  • Adds client for the database manager to get notifications of changes to databases.
  • UIProcess/API/C/WKDatabaseManager.cpp:

(WKDatabaseManagerSetClient):

  • UIProcess/API/C/WKDatabaseManager.h:
  • UIProcess/WebDatabaseManagerProxy.cpp:

(WebKit::WebDatabaseManagerProxy::initializeClient):
(WebKit::WebDatabaseManagerProxy::didModifyOrigin):
(WebKit::WebDatabaseManagerProxy::didModifyDatabase):

  • UIProcess/WebDatabaseManagerProxy.h:
  • UIProcess/WebDatabaseManagerProxy.messages.in:
  • UIProcess/WebDatabaseManagerProxyClient.cpp: Added.

(WebKit::WebDatabaseManagerProxyClient::didModifyOrigin):
(WebKit::WebDatabaseManagerProxyClient::didModifyDatabase):

  • UIProcess/WebDatabaseManagerProxyClient.h: Added.
  • WebKit2.pro:
  • WebKit2.xcodeproj/project.pbxproj:
  • WebProcess/WebCoreSupport/WebDatabaseManager.cpp:

(WebKit::WebDatabaseManager::WebDatabaseManager):
(WebKit::WebDatabaseManager::~WebDatabaseManager):
(WebKit::WebDatabaseManager::dispatchDidModifyOrigin):
(WebKit::WebDatabaseManager::dispatchDidModifyDatabase):

  • WebProcess/WebCoreSupport/WebDatabaseManager.h:
  • win/WebKit2.vcproj:
11:41 Changeset [74692] by weinig@apple.com

Fix Makefile based build.

11:21 Changeset [74691] by xan@webkit.org

2010-12-27 Xan Lopez <xlopez@igalia.com>

Reviewed by Martin Robinson.

Remove unneeded includes from our headers.

  • WebCoreSupport/PasteboardHelperGtk.h: ditto.
  • webkit/webkitprivate.h: ditto.
  • webkit/webkitsoupauthdialog.h: ditto.
11:17 Changeset [74690] by weinig@apple.com

WebKit2: Add additional API for managing databases
https://bugs.webkit.org/show_bug.cgi?id=51629

Reviewed by Anders Carlsson.

  • Adds WKDatabaseManagerGetDatabasesByOrigin, which asynchronously returns an array of dictionaries containing details about each origin using databases and details about each database. Keys were added for accessing the data in the dictionaries.
  • Adds WKDatabaseManagerDeleteDatabasesWithNameForOrigin to delete a database with a specific identifier.
  • Adds WKDatabaseManagerSetQuotaForOrigin to set the quota for an origin.
  • Shared/OriginAndDatabases.cpp: Added.

(WebKit::OriginAndDatabases::encode):
(WebKit::OriginAndDatabases::decode):

  • Shared/OriginAndDatabases.h: Added.
  • Shared/WebCoreArgumentCoders.h:
  • UIProcess/API/C/WKDatabaseManager.cpp:

(WKDatabaseManagerGetOriginKey):
(WKDatabaseManagerGetOriginQuotaKey):
(WKDatabaseManagerGetOriginUsageKey):
(WKDatabaseManagerGetDatabaseDetailsKey):
(WKDatabaseManagerGetDatabaseDetailsNameKey):
(WKDatabaseManagerGetDatabaseDetailsDisplayNameKey):
(WKDatabaseManagerGetDatabaseDetailsExpectedUsageKey):
(WKDatabaseManagerGetDatabaseDetailsCurrentUsageKey):
(WKDatabaseManagerGetDatabasesByOrigin):
(callGetDatabasesByOriginBlockAndDispose):
(WKDatabaseManagerGetDatabasesByOrigin_b):
(WKDatabaseManagerGetDatabaseOrigins):
(callGetDatabaseOriginsBlockBlockAndDispose):
(WKDatabaseManagerGetDatabaseOrigins_b):
(WKDatabaseManagerDeleteDatabasesWithNameForOrigin):
(WKDatabaseManagerDeleteDatabasesForOrigin):
(WKDatabaseManagerDeleteAllDatabases):
(WKDatabaseManagerSetQuotaForOrigin):

  • UIProcess/API/C/WKDatabaseManager.h:
  • UIProcess/WebDatabaseManagerProxy.cpp:

(WebKit::WebDatabaseManagerProxy::originKey):
(WebKit::WebDatabaseManagerProxy::originQuotaKey):
(WebKit::WebDatabaseManagerProxy::originUsageKey):
(WebKit::WebDatabaseManagerProxy::databaseDetailsKey):
(WebKit::WebDatabaseManagerProxy::databaseDetailsNameKey):
(WebKit::WebDatabaseManagerProxy::databaseDetailsDisplayNameKey):
(WebKit::WebDatabaseManagerProxy::databaseDetailsExpectedUsageKey):
(WebKit::WebDatabaseManagerProxy::databaseDetailsCurrentUsageKey):
(WebKit::WebDatabaseManagerProxy::invalidate):
(WebKit::WebDatabaseManagerProxy::getDatabasesByOrigin):
(WebKit::WebDatabaseManagerProxy::didGetDatabasesByOrigin):
(WebKit::WebDatabaseManagerProxy::getDatabaseOrigins):
(WebKit::WebDatabaseManagerProxy::didGetDatabaseOrigins):
(WebKit::WebDatabaseManagerProxy::deleteDatabaseWithNameForOrigin):
(WebKit::WebDatabaseManagerProxy::deleteDatabasesForOrigin):
(WebKit::WebDatabaseManagerProxy::setQuotaForOrigin):

  • UIProcess/WebDatabaseManagerProxy.h:
  • UIProcess/WebDatabaseManagerProxy.messages.in:
  • WebKit2.pro:
  • WebKit2.xcodeproj/project.pbxproj:
  • WebProcess/WebCoreSupport/WebDatabaseManager.cpp:

(WebKit::WebDatabaseManager::getDatabasesByOrigin):
(WebKit::WebDatabaseManager::deleteDatabaseWithNameForOrigin):
(WebKit::WebDatabaseManager::setQuotaForOrigin):

  • WebProcess/WebCoreSupport/WebDatabaseManager.h:
  • WebProcess/WebCoreSupport/WebDatabaseManager.messages.in:
  • win/WebKit2.vcproj:
11:07 Changeset [74689] by mrobinson@webkit.org

2010-12-26 Martin Robinson <mrobinson@igalia.com>

Reviewed by Daniel Bates.

[GTK] Add support for build-jsc
https://bugs.webkit.org/show_bug.cgi?id=51625

Add GTK+ support to build-jsc now that building the jsc executable as
a standalone target is easy.

  • Scripts/build-jsc: Call buildGtkProject for the GTK+ case now.
  • Scripts/build-webkit: Pass in the proper target name to buildGtkProject.
  • Scripts/webkitdirs.pm: Actually process the project parameter in buildGtkProject and pass it along to buildAutotoolsProject.
10:50 Changeset [74688] by levin@chromium.org

check-webkit-style check for meaningless variable names in function declarations.
https://bugs.webkit.org/show_bug.cgi?id=51523

Reviewed by Eric Seidel.

  • Scripts/webkitpy/style/checker.py: Exempted JavaScriptCore/jit/JITStubs.cpp

from the new check and whitespace/parens because the syntax is unusual and
produced a fair number of positives for these checks.

  • Scripts/webkitpy/style/checkers/cpp.py:

(_convert_to_lower_with_underscores): Used as a canonical form for type names
and parameter names when determining if the parameter name is useless.
(_create_acronym): Used to check for redundant variable names in cases like "ExceptionCode ec"
(Parameter.lower_with_underscores_name): Gives back the parameter name in a lower_with_underscore
format.
(_check_parameter_name_against_text): Checks to see if the parameter name is in the
text or an acronym of it.
(check_function_definition): Checks function definitions for meaningless variable names.
(process_line): Added call to check_function_definition.

  • Scripts/webkitpy/style/checkers/cpp_unittest.py:

(CppFunctionsTest.test_convert_to_lower_with_underscores): Test for _convert_to_lower_with_underscores.
(CppFunctionsTest.test_create_acronym): Test for _create_acronym.
(CppFunctionsTest.test_check_parameter_against_text): Test for _check_parameter_against_text.
(CppStyleTestBase.perform_single_line_lint): Removed the parameter name check
because when only checking a snippet, there are a lot of bogus functions.
(CppStyleTestBase.perform_multi_line_lint): Ditto and removed a bogus filter
that I put there previously and just noticed.
(WebKitStyleTest.test_parameter_names): Tests for the functionality -- both
check_function_definition and process_line.

10:46 Changeset [74687] by commit-queue@webkit.org

2010-12-27 Kenji Imasaki <imasaki@chromium.org>

Reviewed by David Levin.

remove entries for media related non-flaky tests.
https://bugs.webkit.org/show_bug.cgi?id=51225

  • platform/chromium/test_expectations.txt:
09:41 Changeset [74686] by antonm@chromium.org

2010-12-27 Anton Muhin <antonm@chromium.org>

Reviewed by David Levin.

[v8] Remove unused methods to protect/unprotect wrappers during garbage collection
https://bugs.webkit.org/show_bug.cgi?id=51648

  • bindings/v8/ScriptController.cpp:
  • bindings/v8/ScriptController.h:
  • bindings/v8/V8GCController.cpp: (WebCore::V8GCController::gcEpilogue):
  • bindings/v8/V8GCController.h:
09:37 Changeset [74685] by commit-queue@webkit.org

2010-12-27 Carlos Garcia Campos <cgarcia@igalia.com>

Reviewed by Martin Robinson.

[GTK] Disable flash plugin in GtkLauncher when using gtk3
https://bugs.webkit.org/show_bug.cgi?id=51591

Flash plugin uses gtk2 that is incompatible with gtk3 making
webkit crash when it's built with gtk3.

  • GtkLauncher/main.c: (main):
09:10 Changeset [74684] by commit-queue@webkit.org

2010-12-27 Rafael Antognolli <antognolli@profusion.mobi>

Reviewed by Kenneth Rohde Christiansen.

[EFL] Set the frame smart object base clipper to non-static.
https://bugs.webkit.org/show_bug.cgi?id=51552

The base clipper of the clipped smart object, which is the parent
class of the ewk_frame class is a static clipper. It's normally only
used for showing/hiding the object, but in the case of the ewk_frame,
it needs to clip the object to its dimension, in order to clip
scrollbars of inner frames.

Setting it to non-static makes the clipper to be automatically moved
by the parent class, thus staying on the correct position when the
frame is moved.

This change needed to be done after the adoption of the static clipper
on the clipped smart base class.

  • ewk/ewk_frame.cpp: (_ewk_frame_smart_add):
06:51 Changeset [74683] by yurys@chromium.org

2010-12-27 Yury Semikhatsky <yurys@chromium.org>

Unreviewed. Removed trailing spaces from a localized
string to make it the same as the one used in the code.

  • English.lproj/localizedStrings.js:
06:36 Changeset [74682] by philn@webkit.org

2010-12-27 Philippe Normand <pnormand@igalia.com>

Unreviewed, skip another GTK media test reporting wrong duration,
see bug 51602.

  • platform/gtk/Skipped:
06:10 Changeset [74681] by yael.aharon@nokia.com

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

Reviewed by Eric Seidel.

Browser is failing 16 baseline test from Selector API Test Suite
https://bugs.webkit.org/show_bug.cgi?id=42968

Update the expected result for the test.

  • fast/dom/SelectorAPI/resig-SelectorsAPI-test-expected.txt:

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

Reviewed by Eric Seidel.

Browser is failing 16 baseline test from Selector API Test Suite
https://bugs.webkit.org/show_bug.cgi?id=42968

Throw an exception if there is no argument to querySelector or
querySelectorAll. This replaces the current behavior of passing the string
"undefined" as the argument.

No new tests, just updating the existing results for
fast/dom/SelectorAPI/resig-SelectorAPI-test.xhtml.

  • dom/Document.idl:
  • dom/DocumentFragment.idl:
  • dom/Element.idl:
03:32 Changeset [74680] by hamaji@chromium.org

2010-12-27 Shinichiro Hamaji <shinichiro.hamaji@gmail.com>

Chromium expectation update.

Suppress fast/canvas/webgl/tex-image-with-format-and-type on linux debug

  • platform/chromium/test_expectations.txt:
00:06 Changeset [74679] by morrita@google.com

2010-12-27 MORITA Hajime <morrita@google.com>

Unreviewed expectation updates.

  • platform/chromium-linux/fast/box-shadow/spread-multiple-normal-expected.checksum: Added.
  • platform/chromium-linux/fast/box-shadow/spread-multiple-normal-expected.png: Added.
  • platform/chromium-win/fast/box-shadow/spread-multiple-normal-expected.checksum: Added.
  • platform/chromium-win/fast/box-shadow/spread-multiple-normal-expected.png: Added.

12/26/10:

23:27 Changeset [74678] by abarth@webkit.org

Fix bug number in previous commit.

23:24 Changeset [74677] by abarth@webkit.org

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

Reviewed by Eric Seidel.

[Meta] Move source code into Sources
https://bugs.webkit.org/show_bug.cgi?id=51606

  • PrettyPatch/PrettyPatch.rb:
    • Remove reference to non-existant autotools directory.

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

Reviewed by Eric Siedel.

[Meta] Move source code into Sources
https://bugs.webkit.org/show_bug.cgi?id=51606

  • .gitignore:
  • GNUmakefile.am:
  • Sources/autotools: Copied from autotools.
  • autogen.sh:
  • autotools: Removed.
  • autotools/acinclude.m4: Removed.
  • autotools/dolt.m4: Removed.
  • autotools/gsettings.m4: Removed.
  • autotools/symbols.filter: Removed.
  • autotools/webkit.m4: Removed.
  • configure.ac:
23:00 Changeset [74676] by abarth@webkit.org

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

Reviewed by Eric Seidel.

Move cmake into Sources
https://bugs.webkit.org/show_bug.cgi?id=51631

  • PrettyPatch/PrettyPatch.rb:

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

Reviewed by Eric Seidel.

Move cmake into Sources
https://bugs.webkit.org/show_bug.cgi?id=51631

  • CMakeLists.txt:
  • Sources/cmake: Copied from cmake.
  • cmake: Removed.
  • cmake/FindCFLite.cmake: Removed.
  • cmake/FindCairo.cmake: Removed.
  • cmake/FindEFL.cmake: Removed.
  • cmake/FindFontconfig.cmake: Removed.
  • cmake/FindFreetype.cmake: Removed.
  • cmake/FindGDK-PixBuf.cmake: Removed.
  • cmake/FindGDK.cmake: Removed.
  • cmake/FindGIO.cmake: Removed.
  • cmake/FindGStreamer-App.cmake: Removed.
  • cmake/FindGStreamer-Base.cmake: Removed.
  • cmake/FindGStreamer-Interfaces.cmake: Removed.
  • cmake/FindGStreamer-Pbutils.cmake: Removed.
  • cmake/FindGStreamer-Plugins-Base.cmake: Removed.
  • cmake/FindGStreamer-Video.cmake: Removed.
  • cmake/FindGStreamer.cmake: Removed.
  • cmake/FindGlib.cmake: Removed.
  • cmake/FindGperf.cmake: Removed.
  • cmake/FindGthread.cmake: Removed.
  • cmake/FindICU.cmake: Removed.
  • cmake/FindLibSoup2.cmake: Removed.
  • cmake/FindLibXlst.cmake: Removed.
  • cmake/FindPango.cmake: Removed.
  • cmake/FindSqlite.cmake: Removed.
  • cmake/LibFindMacros.cmake: Removed.
  • cmake/OptionsCommon.cmake: Removed.
  • cmake/OptionsEfl.cmake: Removed.
  • cmake/OptionsWinCE.cmake: Removed.
  • cmake/OptionsWindows.cmake: Removed.
  • cmake/WebKitEfl.cmake: Removed.
  • cmake/WebKitFS.cmake: Removed.
  • cmake/WebKitFeatures.cmake: Removed.
  • cmake/WebKitHelpers.cmake: Removed.
  • cmake/WebKitMacros.cmake: Removed.
  • cmake/WebKitPackaging.cmake: Removed.
  • cmake/eflsymbols.filter: Removed.
22:11 Changeset [74675] by abarth@webkit.org

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

Fix some references to the old location of the WebKit website in git
metadata files.

  • .gitattributes:
21:51 Changeset [74674] by abarth@webkit.org

Move ANGLE to Sources/ThirdParty
https://bugs.webkit.org/show_bug.cgi?id=51605

Reviewed by Eric Seidel.

Actually move ANGLE and update the Makefile.

  • ANGLE: Removed.
  • ANGLE/ANGLE.xcodeproj: Removed.
  • ANGLE/ANGLE.xcodeproj/project.pbxproj: Removed.
  • ANGLE/Configurations: Removed.
  • ANGLE/Configurations/ANGLE.xcconfig: Removed.
  • ANGLE/Configurations/Base.xcconfig: Removed.
  • ANGLE/Configurations/DebugRelease.xcconfig: Removed.
  • ANGLE/Makefile: Removed.
  • ANGLE/include: Removed.
  • ANGLE/include/EGL: Removed.
  • ANGLE/include/EGL/egl.h: Removed.
  • ANGLE/include/EGL/eglext.h: Removed.
  • ANGLE/include/EGL/eglplatform.h: Removed.
  • ANGLE/include/GLES2: Removed.
  • ANGLE/include/GLES2/gl2.h: Removed.
  • ANGLE/include/GLES2/gl2ext.h: Removed.
  • ANGLE/include/GLES2/gl2platform.h: Removed.
  • ANGLE/include/GLSLANG: Removed.
  • ANGLE/include/GLSLANG/ShaderLang.h: Removed.
  • ANGLE/include/KHR: Removed.
  • ANGLE/include/KHR/khrplatform.h: Removed.
  • ANGLE/src: Removed.
  • ANGLE/src/build_angle.xcodeproj: Removed.
  • ANGLE/src/build_angle.xcodeproj/project.pbxproj: Removed.
  • ANGLE/src/common: Removed.
  • ANGLE/src/common/angleutils.h: Removed.
  • ANGLE/src/common/debug.cpp: Removed.
  • ANGLE/src/common/debug.h: Removed.
  • ANGLE/src/compiler: Removed.
  • ANGLE/src/compiler/BaseTypes.h: Removed.
  • ANGLE/src/compiler/CodeGenGLSL.cpp: Removed.
  • ANGLE/src/compiler/CodeGenHLSL.cpp: Removed.
  • ANGLE/src/compiler/Common.h: Removed.
  • ANGLE/src/compiler/ConstantUnion.h: Removed.
  • ANGLE/src/compiler/InfoSink.cpp: Removed.
  • ANGLE/src/compiler/InfoSink.h: Removed.
  • ANGLE/src/compiler/Initialize.cpp: Removed.
  • ANGLE/src/compiler/Initialize.h: Removed.
  • ANGLE/src/compiler/InitializeDll.cpp: Removed.
  • ANGLE/src/compiler/InitializeDll.h: Removed.
  • ANGLE/src/compiler/InitializeGlobals.h: Removed.
  • ANGLE/src/compiler/InitializeParseContext.h: Removed.
  • ANGLE/src/compiler/IntermTraverse.cpp: Removed.
  • ANGLE/src/compiler/Intermediate.cpp: Removed.
  • ANGLE/src/compiler/MMap.h: Removed.
  • ANGLE/src/compiler/OutputGLSL.cpp: Removed.
  • ANGLE/src/compiler/OutputGLSL.h: Removed.
  • ANGLE/src/compiler/OutputHLSL.cpp: Removed.
  • ANGLE/src/compiler/OutputHLSL.h: Removed.
  • ANGLE/src/compiler/ParseHelper.cpp: Removed.
  • ANGLE/src/compiler/ParseHelper.h: Removed.
  • ANGLE/src/compiler/PoolAlloc.cpp: Removed.
  • ANGLE/src/compiler/PoolAlloc.h: Removed.
  • ANGLE/src/compiler/QualifierAlive.cpp: Removed.
  • ANGLE/src/compiler/QualifierAlive.h: Removed.
  • ANGLE/src/compiler/RemoveTree.cpp: Removed.
  • ANGLE/src/compiler/RemoveTree.h: Removed.
  • ANGLE/src/compiler/ShHandle.h: Removed.
  • ANGLE/src/compiler/ShaderLang.cpp: Removed.
  • ANGLE/src/compiler/SymbolTable.cpp: Removed.
  • ANGLE/src/compiler/SymbolTable.h: Removed.
  • ANGLE/src/compiler/TranslatorGLSL.cpp: Removed.
  • ANGLE/src/compiler/TranslatorGLSL.h: Removed.
  • ANGLE/src/compiler/TranslatorHLSL.cpp: Removed.
  • ANGLE/src/compiler/TranslatorHLSL.h: Removed.
  • ANGLE/src/compiler/Types.h: Removed.
  • ANGLE/src/compiler/UnfoldSelect.cpp: Removed.
  • ANGLE/src/compiler/UnfoldSelect.h: Removed.
  • ANGLE/src/compiler/debug.cpp: Removed.
  • ANGLE/src/compiler/debug.h: Removed.
  • ANGLE/src/compiler/glslang.l: Removed.
  • ANGLE/src/compiler/glslang.y: Removed.
  • ANGLE/src/compiler/intermOut.cpp: Removed.
  • ANGLE/src/compiler/intermediate.h: Removed.
  • ANGLE/src/compiler/localintermediate.h: Removed.
  • ANGLE/src/compiler/osinclude.h: Removed.
  • ANGLE/src/compiler/ossource_posix.cpp: Removed.
  • ANGLE/src/compiler/ossource_win.cpp: Removed.
  • ANGLE/src/compiler/parseConst.cpp: Removed.
  • ANGLE/src/compiler/preprocessor: Removed.
  • ANGLE/src/compiler/preprocessor/atom.c: Removed.
  • ANGLE/src/compiler/preprocessor/atom.h: Removed.
  • ANGLE/src/compiler/preprocessor/compile.h: Removed.
  • ANGLE/src/compiler/preprocessor/cpp.c: Removed.
  • ANGLE/src/compiler/preprocessor/cpp.h: Removed.
  • ANGLE/src/compiler/preprocessor/cppstruct.c: Removed.
  • ANGLE/src/compiler/preprocessor/memory.c: Removed.
  • ANGLE/src/compiler/preprocessor/memory.h: Removed.
  • ANGLE/src/compiler/preprocessor/parser.h: Removed.
  • ANGLE/src/compiler/preprocessor/preprocess.h: Removed.
  • ANGLE/src/compiler/preprocessor/scanner.c: Removed.
  • ANGLE/src/compiler/preprocessor/scanner.h: Removed.
  • ANGLE/src/compiler/preprocessor/slglobals.h: Removed.
  • ANGLE/src/compiler/preprocessor/symbols.c: Removed.
  • ANGLE/src/compiler/preprocessor/symbols.h: Removed.
  • ANGLE/src/compiler/preprocessor/tokens.c: Removed.
  • ANGLE/src/compiler/preprocessor/tokens.h: Removed.
  • ANGLE/src/compiler/tools: Removed.
  • ANGLE/src/compiler/unistd.h: Removed.
  • ANGLE/src/libEGL: Removed.
  • ANGLE/src/libEGL/Config.cpp: Removed.
  • ANGLE/src/libEGL/Config.h: Removed.
  • ANGLE/src/libEGL/Display.cpp: Removed.
  • ANGLE/src/libEGL/Display.h: Removed.
  • ANGLE/src/libEGL/Surface.cpp: Removed.
  • ANGLE/src/libEGL/Surface.h: Removed.
  • ANGLE/src/libEGL/libEGL.cpp: Removed.
  • ANGLE/src/libEGL/libEGL.def: Removed.
  • ANGLE/src/libEGL/libEGL.vcproj: Removed.
  • ANGLE/src/libEGL/main.cpp: Removed.
  • ANGLE/src/libEGL/main.h: Removed.
  • ANGLE/src/libGLESv2: Removed.
  • ANGLE/src/libGLESv2/Blit.cpp: Removed.
  • ANGLE/src/libGLESv2/Blit.h: Removed.
  • ANGLE/src/libGLESv2/Buffer.cpp: Removed.
  • ANGLE/src/libGLESv2/Buffer.h: Removed.
  • ANGLE/src/libGLESv2/Context.cpp: Removed.
  • ANGLE/src/libGLESv2/Context.h: Removed.
  • ANGLE/src/libGLESv2/Framebuffer.cpp: Removed.
  • ANGLE/src/libGLESv2/Framebuffer.h: Removed.
  • ANGLE/src/libGLESv2/Program.cpp: Removed.
  • ANGLE/src/libGLESv2/Program.h: Removed.
  • ANGLE/src/libGLESv2/RefCountObject.cpp: Removed.
  • ANGLE/src/libGLESv2/RefCountObject.h: Removed.
  • ANGLE/src/libGLESv2/Renderbuffer.cpp: Removed.
  • ANGLE/src/libGLESv2/Renderbuffer.h: Removed.
  • ANGLE/src/libGLESv2/ResourceManager.cpp: Removed.
  • ANGLE/src/libGLESv2/ResourceManager.h: Removed.
  • ANGLE/src/libGLESv2/Shader.cpp: Removed.
  • ANGLE/src/libGLESv2/Shader.h: Removed.
  • ANGLE/src/libGLESv2/Texture.cpp: Removed.
  • ANGLE/src/libGLESv2/Texture.h: Removed.
  • ANGLE/src/libGLESv2/geometry: Removed.
  • ANGLE/src/libGLESv2/geometry/IndexDataManager.cpp: Removed.
  • ANGLE/src/libGLESv2/geometry/IndexDataManager.h: Removed.
  • ANGLE/src/libGLESv2/geometry/VertexDataManager.cpp: Removed.
  • ANGLE/src/libGLESv2/geometry/VertexDataManager.h: Removed.
  • ANGLE/src/libGLESv2/geometry/backend.cpp: Removed.
  • ANGLE/src/libGLESv2/geometry/backend.h: Removed.
  • ANGLE/src/libGLESv2/geometry/dx9.cpp: Removed.
  • ANGLE/src/libGLESv2/geometry/dx9.h: Removed.
  • ANGLE/src/libGLESv2/geometry/vertexconversion.h: Removed.
  • ANGLE/src/libGLESv2/libGLESv2.cpp: Removed.
  • ANGLE/src/libGLESv2/libGLESv2.def: Removed.
  • ANGLE/src/libGLESv2/libGLESv2.vcproj: Removed.
  • ANGLE/src/libGLESv2/main.cpp: Removed.
  • ANGLE/src/libGLESv2/main.h: Removed.
  • ANGLE/src/libGLESv2/mathutil.h: Removed.
  • ANGLE/src/libGLESv2/utilities.cpp: Removed.
  • ANGLE/src/libGLESv2/utilities.h: Removed.
  • Makefile:
  • Sources/ThirdParty: Added.
  • Sources/ThirdParty/ANGLE: Copied from ANGLE.

Tools:

  • Scripts/build-webkit:
    • Point to the new ANGLE location.
20:42 Changeset [74673] by morrita@google.com

2010-12-26 MORITA Hajime <morrita@google.com>

Reviewed by Dan Bernstein.

For box-shadow, non-primary shadows should have specified radii.
https://bugs.webkit.org/show_bug.cgi?id=51386

  • fast/box-shadow/spread-multiple-normal-expected.txt: Added.
  • fast/box-shadow/spread-multiple-normal.html: Added.
  • fast/box-shadow/spread-multiple-normal-expected.checksum: Added.
  • fast/box-shadow/spread-multiple-normal-expected.png: Added.

2010-12-26 MORITA Hajime <morrita@google.com>

Reviewed by Dan Bernstein.

For box-shadow, non-primary shadows should have specified radii.
https://bugs.webkit.org/show_bug.cgi?id=51386

On paintBoxShadow(), the spread value accidentally accumulated accidentally
during the loop, which should be applied individually for radius of each shadow.
This change makes the radii for rects computed for each shadow.

Test: fast/box-shadow/spread-multiple-normal.html

  • rendering/RenderBoxModelObject.cpp: (WebCore::RenderBoxModelObject::paintBoxShadow):
14:13 Changeset [74672] by commit-queue@webkit.org

2010-12-26 Xan Lopez <xlopez@igalia.com>

Reviewed by Eric Seidel.

[GTK] Add standalone target for JSC
https://bugs.webkit.org/show_bug.cgi?id=51607

  • GNUmakefile.am: add convenience target to only build jsc and its dependencies.
14:00 Changeset [74671] by paroga@webkit.org

2010-12-26 Patrick Gansterer <paroga@webkit.org>

Reviewed by Eric Seidel.

Improve output of HTML parser benchmark
https://bugs.webkit.org/show_bug.cgi?id=51611

Calculate and show median, min and max values.

  • benchmarks/parser/html-parser.html:
12:27 Changeset [74670] by levin@chromium.org

2010-12-26 David Levin <levin@chromium.org>

Reviewed by Eric Seidel.

check-webkit-style should be able to parse function declaration parameters.
https://bugs.webkit.org/show_bug.cgi?id=51451

  • Scripts/webkitpy/style/checkers/cpp.py: (Position.init): Holds simple position information (row, column). (Parameter.init): Holds information about a parameter. (SingleLineView.init): Converts multiple lines into a single line for simpler searches. (SingleLineView.convert_column_to_row): Returns the original row given a column. (create_skeleton_parameters): Simplifies a parameter list for easier searching. (find_parameter_name_index): Finds where the parameter name is. (parameter_list): Generates the list of parameters for a function. (_FunctionState.begin): Added information to allow determining the parameters on demand. (_FunctionState.get_parameter_list): Returns a tuple of function parameters. (detect_functions): Improve function detection for operator functions and determine where the parameters end and pass that to _FunctionState.begin.
  • Scripts/webkitpy/style/checkers/cpp_unittest.py: (CppFunctionsTest.test_parameter): Verifies Parameter functionality. (CppFunctionsTest.test_single_line_view): Verifies SingleLineView functionality. (CppFunctionsTest.test_create_skeleton_parameters): Verifies create_skeleton_parameters. (CppFunctionsTest.test_find_parameter_name_index): Verifies find_parameter_name_index. (CppFunctionsTest.test_parameter_list): Does some minimal verification for parameter list. Much more thorough verification is done as part of FunctionDetectionTest.test_parameter_list. (FunctionDetectionTest.perform_function_detection): Added support for verifying the parameters found. (FunctionDetectionTest.test_function_declaration_detection): Added more function detection test to verify that we catch the operator functions. (FunctionDetectionTest.test_ignore_macros): Verify that function detection ignores macros. (FunctionDetectionTest.test_parameter_list): Added tests to verify the parameter parsing. (CheckForFunctionLengthsTest.test_function_length_check_definition_severity1_for_bad_test_doesnt_break): Removed because the error. The test is about the bad function name. Fixing the name makes the test exactly like test_function_length_check_definition_severity1_for_test.
11:41 Changeset [74669] by andersca@apple.com

Rename DrawingAreaMessage and DrawingAreaLegacyMessage and
DrawingAreaProxyMessage to DrawingAreaProxyLegacyMessage.

Reviewed by Dan Bernstein.

  • Platform/CoreIPC/MessageID.h:
  • Shared/CoreIPCSupport/DrawingAreaMessageKinds.h:
  • Shared/CoreIPCSupport/DrawingAreaProxyMessageKinds.h:
  • UIProcess/ChunkedUpdateDrawingAreaProxy.cpp:

(WebKit::ChunkedUpdateDrawingAreaProxy::paint):
(WebKit::ChunkedUpdateDrawingAreaProxy::setPageIsVisible):
(WebKit::ChunkedUpdateDrawingAreaProxy::update):
(WebKit::ChunkedUpdateDrawingAreaProxy::sendSetSize):
(WebKit::ChunkedUpdateDrawingAreaProxy::didReceiveMessage):

  • UIProcess/LayerBackedDrawingAreaProxy.cpp:

(WebKit::LayerBackedDrawingAreaProxy::sizeDidChange):
(WebKit::LayerBackedDrawingAreaProxy::setPageIsVisible):
(WebKit::LayerBackedDrawingAreaProxy::update):
(WebKit::LayerBackedDrawingAreaProxy::didReceiveMessage):
(WebKit::LayerBackedDrawingAreaProxy::didReceiveSyncMessage):

  • UIProcess/TiledDrawingAreaProxy.cpp:

(WebKit::TiledDrawingAreaProxy::sizeDidChange):
(WebKit::TiledDrawingAreaProxy::setPageIsVisible):
(WebKit::TiledDrawingAreaProxy::didReceiveMessage):
(WebKit::TiledDrawingAreaProxy::requestTileUpdate):
(WebKit::TiledDrawingAreaProxy::waitUntilUpdatesComplete):
(WebKit::TiledDrawingAreaProxy::takeSnapshot):
(WebKit::TiledDrawingAreaProxy::removeTile):

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::didReceiveMessage):
(WebKit::WebPageProxy::didReceiveSyncMessage):

  • WebProcess/WebPage/ChunkedUpdateDrawingArea.cpp:

(WebKit::ChunkedUpdateDrawingArea::display):
(WebKit::ChunkedUpdateDrawingArea::setSize):
(WebKit::ChunkedUpdateDrawingArea::didReceiveMessage):

  • WebProcess/WebPage/LayerBackedDrawingArea.cpp:

(WebKit::LayerBackedDrawingArea::setSize):
(WebKit::LayerBackedDrawingArea::didReceiveMessage):

  • WebProcess/WebPage/TiledDrawingArea.cpp:

(WebKit::TiledDrawingArea::display):
(WebKit::TiledDrawingArea::setSize):
(WebKit::TiledDrawingArea::updateTile):
(WebKit::TiledDrawingArea::tileUpdateTimerFired):
(WebKit::TiledDrawingArea::didReceiveMessage):

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::didReceiveMessage):

  • WebProcess/WebPage/mac/LayerBackedDrawingAreaMac.mm:

(WebKit::LayerBackedDrawingArea::attachCompositingContext):

11:26 Changeset [74668] by andersca@apple.com

Drawing area cleanup. Remove a bunch of redundant functions that just ended
up calling setNeedsDisplay.

Reviewed by Sam Weinig.

  • UIProcess/DrawingAreaProxy.h:
  • WebProcess/WebCoreSupport/WebChromeClient.cpp:

(WebKit::WebChromeClient::invalidateWindow):
(WebKit::WebChromeClient::invalidateContentsAndWindow):
(WebKit::WebChromeClient::invalidateContentsForSlowScroll):

  • WebProcess/WebPage/ChunkedUpdateDrawingArea.cpp:
  • WebProcess/WebPage/ChunkedUpdateDrawingArea.h:
  • WebProcess/WebPage/DrawingArea.h:
  • WebProcess/WebPage/LayerBackedDrawingArea.cpp:
  • WebProcess/WebPage/LayerBackedDrawingArea.h:
  • WebProcess/WebPage/TiledDrawingArea.cpp:
  • WebProcess/WebPage/TiledDrawingArea.h:
08:57 Changeset [74667] by kbr@google.com

2010-12-26 Kenneth Russell <kbr@google.com>

Unreviewed, Chromium expectations fix. Fix broken (duplicate)
expectations for
canvas/philip/tests/2d.gradient.radial.cone.top.html.

  • platform/chromium/test_expectations.txt:
08:04 Changeset [74666] by inferno@chromium.org

2010-12-26 Abhishek Arya <inferno@chromium.org>

Reviewed by Simon Fraser.

Remove the bad assert in paintPaginatedChildLayer, hitTestPaginatedChildLayer.
https://bugs.webkit.org/show_bug.cgi?id=48772

Test: fast/multicol/renderer-positioned-assert-crash.html

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::paintPaginatedChildLayer): (WebCore::RenderLayer::hitTestPaginatedChildLayer):

2010-12-26 Abhishek Arya <inferno@chromium.org>

Reviewed by Simon Fraser.

Tests passes if it does not crash.
https://bugs.webkit.org/show_bug.cgi?id=48772

  • fast/multicol/renderer-positioned-assert-crash-expected.txt: Added.
  • fast/multicol/renderer-positioned-assert-crash.html: Added.
02:04 Changeset [74665] by paroga@webkit.org

2010-12-26 Patrick Gansterer <paroga@webkit.org>

Unreviewed WinCE buildfix after r74626.

  • cmake/OptionsWinCE.cmake:

12/25/10:

21:05 Changeset [74664] by kbr@google.com

2010-12-25 Kenneth Russell <kbr@google.com>

Unreviewed, Chromium test expectations update.

  • platform/chromium-gpu/test_expectations.txt: Win Debug: Marked compositing/tiling/huge-layer-img.html as TIMEOUT. Marked fast/canvas/webgl/gl-object-get-calls.html as a flaky crash.
  • platform/chromium/test_expectations.txt: Adjusted expectations for the following tests: fast/canvas/shadow-offset-7.html svg/batik/paints/gradientLimit.svg fast/events/touch/touch-target.html canvas/philip/tests/2d.gradient.radial.cone.top.html fast/css/box-shadow.html
17:49 Changeset [74663] by commit-queue@webkit.org

2010-12-25 Peter Rybin <peter.rybin@gmail.com>

Reviewed by Adam Barth.

Adds currentColumn (and currentLine) to SegmentedString. Switches
HTMLTreeBuilder to HTMLDocumentParser::textPosition when it
needs position for a <script> tag.

SegmentedString should provide column position
https://bugs.webkit.org/show_bug.cgi?id=51311

  • html/parser/HTMLDocumentParser.cpp: (WebCore::HTMLDocumentParser::HTMLDocumentParser): (WebCore::HTMLDocumentParser::textPosition):
  • html/parser/HTMLInputStream.h: (WebCore::HTMLInputStream::current): (WebCore::InsertionPointRecord::InsertionPointRecord): (WebCore::InsertionPointRecord::~InsertionPointRecord):
  • html/parser/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::HTMLTreeBuilder): (WebCore::HTMLTreeBuilder::processScriptStartTag):
  • html/parser/HTMLTreeBuilder.h: (WebCore::HTMLTreeBuilder::create):
  • platform/text/SegmentedString.cpp: (WebCore::SegmentedString::operator=): (WebCore::SegmentedString::numberOfCharactersConsumedSlow): (WebCore::SegmentedString::advanceSlowCase): (WebCore::SegmentedString::currentLine): (WebCore::SegmentedString::currentColumn): (WebCore::SegmentedString::setCurrentPosition):
  • platform/text/SegmentedString.h: (WebCore::SegmentedString::SegmentedString): (WebCore::SegmentedString::advancePastNewline): (WebCore::SegmentedString::advance): (WebCore::SegmentedString::numberOfCharactersConsumed):
16:38 Changeset [74662] by commit-queue@webkit.org

2010-12-25 Andreas Kling <kling@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

GraphicsContext: Don't pass StrokeStyle (enum) as const reference
https://bugs.webkit.org/show_bug.cgi?id=51608

  • platform/graphics/GraphicsContext.cpp: (WebCore::GraphicsContext::setStrokeStyle): (WebCore::GraphicsContext::setPlatformStrokeStyle): (WebCore::GraphicsContext::adjustLineToPixelBoundaries):
  • platform/graphics/GraphicsContext.h:
  • platform/graphics/cairo/GraphicsContextCairo.cpp: (WebCore::GraphicsContext::setPlatformStrokeStyle):
  • platform/graphics/haiku/GraphicsContextHaiku.cpp: (WebCore::GraphicsContext::setPlatformStrokeStyle):
  • platform/graphics/openvg/GraphicsContextOpenVG.cpp: (WebCore::GraphicsContext::setPlatformStrokeStyle):
  • platform/graphics/openvg/PainterOpenVG.cpp: (WebCore::PainterOpenVG::setStrokeStyle):
  • platform/graphics/openvg/PainterOpenVG.h:
  • platform/graphics/qt/GraphicsContextQt.cpp: (WebCore::GraphicsContext::setPlatformStrokeStyle):
  • platform/graphics/skia/GraphicsContextSkia.cpp: (WebCore::GraphicsContext::setPlatformStrokeStyle):
12:53 Changeset [74661] by weinig@apple.com

Fill in some more preferences for WebKit2
https://bugs.webkit.org/show_bug.cgi?id=51610

Reviewed by Dan Bernstein.

  • Shared/WebPreferencesStore.h:
  • UIProcess/API/C/WKPreferences.cpp:

(WKPreferencesSetAuthorAndUserStylesEnabled):
(WKPreferencesGetAuthorAndUserStylesEnabled):
(WKPreferencesSetWebArchiveDebugModeEnabled):
(WKPreferencesGetWebArchiveDebugModeEnabled):
(WKPreferencesSetLocalFileContentSniffingEnabled):
(WKPreferencesGetLocalFileContentSniffingEnabled):
(WKPreferencesSetPageCacheEnabled):
(WKPreferencesGetPageCacheEnabled):
(WKPreferencesSetPaginateDuringLayoutEnabled):
(WKPreferencesGetPaginateDuringLayoutEnabled):
(WKPreferencesSetDOMPasteAllowedEnabled):
(WKPreferencesGetDOMPasteAllowedEnabled):

  • UIProcess/API/C/WKPreferences.h:
  • UIProcess/API/C/WKPreferencesPrivate.h:
  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::updatePreferences):
Pipe through some more preferences.

12:21 Changeset [74660] by paroga@webkit.org

2010-12-25 Patrick Gansterer <paroga@webkit.org>

Unreviewed WinCE buildfix after r74334.

  • CMakeListsWinCE.txt:
01:52 Changeset [74659] by philn@webkit.org

2010-12-25 Philippe Normand <pnormand@igalia.com>

Unreviewed, skip another GTK media test reporting wrong duration,
see bug 50518.

  • platform/gtk/Skipped:

12/24/10:

22:00 Changeset [74658] by kbr@google.com

2010-12-24 Kenneth Russell <kbr@google.com>

Unreviewed, Chromium test expectations update.
svg/batik/paints/gradientLimit.svg fails on Linux Debug as well.

  • platform/chromium/test_expectations.txt:
17:52 Changeset [74657] by mitz@apple.com

REGRESSION (r74648): XHR layout test failures
https://bugs.webkit.org/show_bug.cgi?id=51603

Reviewed by Alexey Proskuryakov.

Reverted r74648.

WebCore:

  • WebCore.exp.in:

WebKit2:

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::didReceiveAuthenticationChallenge):

  • WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:

(WebKit::WebFrameLoaderClient::shouldUseCredentialStorage):

  • WebProcess/mac/WebProcessMainMac.mm:

(WebKit::WebProcessMain):

14:33 Changeset [74656] by philn@webkit.org

2010-12-24 Philippe Normand <pnormand@igalia.com>

Unreviewed, skip failing media test on GTK.

  • platform/gtk/Skipped: Skip media/controls-after-reload.html.
13:43 Changeset [74655] by philn@webkit.org

2010-12-24 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r74632.
http://trac.webkit.org/changeset/74632
https://bugs.webkit.org/show_bug.cgi?id=51600

makes test-webkitpy hang forever on GTK (Requested by philn-tp
on #webkit).

  • Scripts/webkitpy/layout_tests/layout_package/message_broker.py:
  • Scripts/webkitpy/layout_tests/layout_package/message_broker_unittest.py:
13:26 Changeset [74654] by simon.fraser@apple.com

2010-12-24 Simon Fraser <simon.fraser@apple.com>

Fix the Qt build after r74650.

  • WebProcess/WebCoreSupport/WebPlatformStrategies.cpp:
13:24 Changeset [74653] by andreip@google.com

2010-12-24 Andrei Popescu <andreip@google.com>

Unreviewed, build fix.

[Chromium] http/tests/xmlhttprequest/basic-auth-nopassword.html needs Mac, Windows and Linux specific test expectations
https://bugs.webkit.org/show_bug.cgi?id=51589

  • platform/chromium-win/http/tests/xmlhttprequest/basic-auth-nopassword-expected.txt: Added.
  • platform/chromium-win/http/tests/xmlhttprequest/basic-auth-nouser-expected.txt: Added.
13:13 Changeset [74652] by mitz@apple.com

Try to fix the Qt build after r74648.

  • WebCore.gypi:
  • WebCore.pro:
  • platform/network/qt/CredentialStorageQt.cpp: Added.

(WebCore::CredentialStorage::getFromPersistentStorage):

12:57 Changeset [74651] by philn@webkit.org

2010-12-24 Philippe Normand <pnormand@igalia.com>

Unreviewed, updated GTK media tests baselines after r74643.

  • 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:
12:55 Changeset [74650] by commit-queue@webkit.org

2010-12-24 Benjamin Poulain <ikipou@gmail.com>

Reviewed by Darin Adler.

[Qt] [WK2] WebKit 2 does not build on mac, missing reference to RetainPtr
https://bugs.webkit.org/show_bug.cgi?id=51598

Fix the build, add the missing header.

  • WebProcess/WebCoreSupport/WebPlatformStrategies.cpp:
12:46 Changeset [74649] by simon.fraser@apple.com

2010-12-24 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dan Bernstein.

CSSStyleSelector cleanup
https://bugs.webkit.org/show_bug.cgi?id=51597

Make some methods const. Add assertion to checkOneSelector, with the
intention of later removing the if (!e) check (this is hot code).

  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::locateCousinList): (WebCore::CSSStyleSelector::canShareStyleWithElement): (WebCore::CSSStyleSelector::locateSharedStyle): (WebCore::CSSStyleSelector::SelectorChecker::checkOneSelector): (WebCore::CSSStyleSelector::mmLength): (WebCore::CSSStyleSelector::inchLength): (WebCore::CSSStyleSelector::getColorFromPrimitiveValue): (WebCore::CSSStyleSelector::hasSelectorForAttribute):
  • css/CSSStyleSelector.h: (WebCore::CSSStyleSelector::fontSelector):
12:24 Changeset [74648] by mitz@apple.com

<rdar://problem/8758386> The web process uses its own credential storage
https://bugs.webkit.org/show_bug.cgi?id=51599

Reviewed by Anders Carlsson.

WebCore:

  • WebCore.exp.in: Export CredentialStorage::getFromPersistentStorage(), Credential::hasPassword(),

and Credential::isEmpty().

WebKit2:

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::didReceiveAuthenticationChallenge): Try to answer the challenge using
the UI process’s credential storage first.

  • WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:

(WebKit::WebFrameLoaderClient::shouldUseCredentialStorage): Prevent the web process from using
its own credential storage.

  • WebProcess/mac/WebProcessMainMac.mm:

(WebKit::WebProcessMain): Ditto.

11:36 Changeset [74647] by darin@apple.com

Rebased Leopard to expect the same kind of stack failure
as Windows, rather than the success expected on Snow Leopard
and many other platforms.

  • platform/mac-leopard/fast/js/large-expressions-expected.txt:

Copied from platform/win/fast/js/large-expressions-expected.txt.

11:36 Changeset [74646] by yurys@chromium.org

2010-12-24 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Darin Adler.

[V8] Rename V8CustomEventListener.{h,cpp} to V8EventListener.{h,cpp}
https://bugs.webkit.org/show_bug.cgi?id=51595

  • WebCore.gypi:
  • bindings/v8/V8DOMWrapper.cpp:
  • bindings/v8/V8EventListener.cpp: Renamed from WebCore/bindings/v8/custom/V8CustomEventListener.cpp. (WebCore::V8EventListener::V8EventListener): (WebCore::V8EventListener::getListenerFunction): (WebCore::V8EventListener::callListenerFunction):
  • bindings/v8/V8EventListener.h: Renamed from WebCore/bindings/v8/custom/V8CustomEventListener.h. (WebCore::V8EventListener::create):
  • bindings/v8/V8EventListenerList.h:
  • bindings/v8/V8WorkerContextEventListener.h:
  • bindings/v8/custom/V8DOMWindowCustom.cpp:
  • bindings/v8/custom/V8LocationCustom.cpp:
  • bindings/v8/custom/V8NodeCustom.cpp:
  • bindings/v8/custom/V8NotificationCenterCustom.cpp:
11:26 Changeset [74645] by jberlin@webkit.org

Add Windows-specific (failing) expected results for
http/tests/xmlhttprequest/basic-auth-nouser.html and basic-auth-nopassword.html in order to
get the Windows 7 Release test bot green again. Unreviewed.
https://bugs.webkit.org/show_bug.cgi?id=51596

  • platform/win/http/tests/xmlhttprequest/basic-auth-nopassword-expected.txt: Added.
  • platform/win/http/tests/xmlhttprequest/basic-auth-nouser-expected.txt: Added.
11:26 Changeset [74644] by darin@apple.com

Updated makefiles used by Apple engineers to fix build with the
new JavaScriptGlue location.

  • Makefile: Told this to build JavaScriptGlue. Some day we might

want it to descend into Sources instead and use the Makefile there,
but there are problems with sequencing the project builds, so that
can wait.

  • Sources/Makefile.shared: Added.
11:02 Changeset [74643] by commit-queue@webkit.org

2010-12-24 Carlos Garcia Campos <cgarcia@igalia.com>

Reviewed by Xan Lopez.

[GTK] Show current time / total in media player
https://bugs.webkit.org/show_bug.cgi?id=51535

  • css/mediaControlsGtk.css: (audio::-webkit-media-controls-current-time-display, video::-webkit-media-controls-current-time-display):
  • platform/gtk/RenderThemeGtk.cpp: (WebCore::RenderThemeGtk::formatMediaControlsCurrentTime): (WebCore::RenderThemeGtk::paintMediaCurrentTime):
  • platform/gtk/RenderThemeGtk.h:
10:55 Changeset [74642] by commit-queue@webkit.org

2010-12-24 Jan Erik Hanssen <jhanssen@sencha.com>

Reviewed by Eric Seidel.

Clean up CSSRuleData in CSSStyleSelector.h
https://bugs.webkit.org/show_bug.cgi?id=27753

Move CSSRuleData and CSSRuleDataList from CSSStyleSelector.h to the .cpp file.

  • css/CSSStyleSelector.cpp: (WebCore::CSSRuleData::CSSRuleData): (WebCore::CSSRuleData::~CSSRuleData): (WebCore::CSSRuleData::position): (WebCore::CSSRuleData::rule): (WebCore::CSSRuleData::selector): (WebCore::CSSRuleData::next): (WebCore::CSSRuleDataList::CSSRuleDataList): (WebCore::CSSRuleDataList::~CSSRuleDataList): (WebCore::CSSRuleDataList::first): (WebCore::CSSRuleDataList::last): (WebCore::CSSRuleDataList::append):
  • css/CSSStyleSelector.h:
10:54 Changeset [74641] by jberlin@webkit.org

Skip the new editing/input/ime-composition-clearpreedit.html test since textInputController
is unimplemented in the DRT for Windows. Unreviewed.
https://bugs.webkit.org/show_bug.cgi?id=32021

  • platform/win/Skipped:
10:43 Changeset [74640] by commit-queue@webkit.org

2010-12-24 Ryuan Choi <ryuan.choi@samsung.com>

Reviewed by Kenneth Rohde Christiansen.

[EFL] Screen doesn't be rendered when changing size of EWebLauncher
https://bugs.webkit.org/show_bug.cgi?id=43528

Remove on_resize because ewk_view_fixed_layout_size_set should not be
called on callback of EVAS_CALLBACK_RESIZE. As it change size of
FrameView, rendering was ignored when _ewk_view_smart_calculate was
called.

  • EWebLauncher/main.c: (browserCreate):
10:21 Changeset [74639] by eric@webkit.org

2010-12-24 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

webkit-patch (or a pre-commit hook) needs to prevent bad ChangeLog changes
https://bugs.webkit.org/show_bug.cgi?id=28291

This is a start. At least now webkit-patch will prompt when your ChangeLog looks questionable.
We could do more advanced things, like parsing the ChangeLog (with changelog.py) and comparing that
to strings with find in the diff.
Since non-interactive always returns the default, this should cause patches with bad changelogs to fail on the commit-queue.

  • Scripts/webkitpy/common/checkout/api.py:
  • Scripts/webkitpy/common/checkout/diff_parser.py:
  • Scripts/webkitpy/tool/steps/abstractstep.py:
  • Scripts/webkitpy/tool/steps/cleanworkingdirectory.py:
  • Scripts/webkitpy/tool/steps/validatechangelogs.py: Copied from Tools/Scripts/webkitpy/tool/steps/validatereviewer.py.
  • Scripts/webkitpy/tool/steps/validatechangelogs_unittest.py: Copied from Tools/Scripts/webkitpy/tool/steps/cleanworkingdirectory.py.
  • Scripts/webkitpy/tool/steps/validatereviewer.py:
10:20 Changeset [74638] by yurys@chromium.org

2010-12-24 Yury Semikhatsky <yurys@chromium.org>

Unreviewed. Fix compilation on Windows.

  • inspector/InspectorStyleSheet.cpp: (WebCore::InspectorStyle::~InspectorStyle): added explicit empty destructor
  • inspector/InspectorStyleSheet.h:
09:51 Changeset [74637] by yurys@chromium.org

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):
09:31 Changeset [74636] by jschuh@chromium.org

2010-12-24 Justin Schuh <jschuh@chromium.org>

Reviewed by Nikolas Zimmermann.

SVGElementInstance::m_useElement should be cleared when use element is removed from document
https://bugs.webkit.org/show_bug.cgi?id=51486

Test: svg/custom/use-instanceRoot-with-use-removed.svg

  • svg/SVGElementInstance.cpp: (WebCore::SVGElementInstance::invalidateAllInstancesOfElement):
  • svg/SVGElementInstance.h: (WebCore::SVGElementInstance::clearUseElement):
  • svg/SVGUseElement.cpp: (WebCore::SVGUseElement::removedFromDocument): (WebCore::ShadowTreeUpdateBlocker::if): (WebCore::SVGUseElement::detachInstance): (WebCore::SVGUseElement::detach):
  • svg/SVGUseElement.h:

2010-12-24 Justin Schuh <jschuh@chromium.org>

Reviewed by Nikolas Zimmermann.

SVGElementInstance::m_useElement should be cleared when use element is removed from document
https://bugs.webkit.org/show_bug.cgi?id=51486

  • svg/custom/use-instanceRoot-with-use-removed-expected.txt: Added.
  • svg/custom/use-instanceRoot-with-use-removed.svg: Added.
09:01 Changeset [74635] by yurys@chromium.org

2010-12-23 Joseph Pecoraro <joepeck@webkit.org>

Reviewed by Yury Semikhatsky.

Web Inspector: Missing Localized Strings
https://bugs.webkit.org/show_bug.cgi?id=51542

Added missing strings: "WebSocket", "Frames", "Text", "Transfer",
"Preserve Log upon Navigation", and "Go to Line".

  • English.lproj/localizedStrings.js:
08:39 Changeset [74634] by apavlov@chromium.org

2010-12-24 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: Inline HTML style property out of sync with element.style in Sidebar
https://bugs.webkit.org/show_bug.cgi?id=51478

Now the element attribute modifications (including "style") are tracked, which results in the
source offsets update (in the backend) and Styles/Metrics pane updates (in the frontend)
on the "style" attribute modification through the Elements tree. Also, it appears that
a complete "style" attribute removal destroys the inline style declaration
(StyledElement::m_inlineStyleDecl), while the code used to rely on it being immutable (hence a crash).

Test: inspector/elements-delete-inline-style.html

WebCore:

  • inspector/InspectorCSSAgent.cpp: (WebCore::InspectorCSSAgent::didModifyDOMAttr):
  • inspector/InspectorCSSAgent.h:
  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::didModifyDOMAttr):
  • inspector/InspectorDOMAgent.h:
  • inspector/InspectorStyleSheet.cpp: (WebCore::InspectorStyleSheetForInlineStyle::InspectorStyleSheetForInlineStyle): (WebCore::InspectorStyleSheetForInlineStyle::didModifyElementAttribute): (WebCore::InspectorStyleSheetForInlineStyle::text): (WebCore::InspectorStyleSheetForInlineStyle::setStyleText): (WebCore::InspectorStyleSheetForInlineStyle::getStyleAttributeRanges):
  • inspector/InspectorStyleSheet.h:
  • inspector/front-end/ElementsTreeOutline.js: (WebInspector.ElementsTreeElement.prototype._attributeEditingCommitted):


LayoutTests:

  • http/tests/inspector/inspector-test2.js: (initialize_InspectorTest.InspectorTest.addResults): (initialize_InspectorTest.InspectorTest.createKeyEvent):
  • inspector/elements-delete-inline-style-expected.txt: Added.
  • inspector/elements-delete-inline-style.html: Added.
  • inspector/elements-tests2.js: Added. (initialize_ElementTest.InspectorTest.expandDOMSubtree.processChildren): (initialize_ElementTest.InspectorTest.expandDOMSubtree): (initialize_ElementTest.InspectorTest.expandDOMSubtreeAndRun): (initialize_ElementTest.InspectorTest.nodeForId): (initialize_ElementTest.InspectorTest.selectElementAndRun): (initialize_ElementTest.InspectorTest.dumpSelectedElementStyles): (initialize_ElementTest.InspectorTest.getSelectedElementStyles): (initialize_ElementTest.InspectorTest.getElementStylePropertyTreeItem): (initialize_ElementTest.InspectorTest.dumpStyleTreeOutline): (initialize_ElementTest.InspectorTest.dumpStyleTreeItem): (initialize_ElementTest):
  • inspector/styles-add-blank-property.html: Remove unnecessary function.
04:18 Changeset [74633] by commit-queue@webkit.org

2010-12-24 Benjamin Kalman <kalman@chromium.org>

Reviewed by Ryosuke Niwa.

editing/selection/caret-* tests need rebaseline for gtk
https://bugs.webkit.org/show_bug.cgi?id=51578

Rebaseline editing/selection/caret-* tests for gtk.

  • platform/gtk/editing/selection/caret-ltr-2-expected.checksum: Added.
  • platform/gtk/editing/selection/caret-ltr-2-expected.png: Added.
  • platform/gtk/editing/selection/caret-ltr-2-expected.txt: Added.
  • platform/gtk/editing/selection/caret-ltr-2-left-expected.checksum: Added.
  • platform/gtk/editing/selection/caret-ltr-2-left-expected.png: Added.
  • platform/gtk/editing/selection/caret-ltr-2-left-expected.txt: Added.
  • platform/gtk/editing/selection/caret-ltr-expected.checksum: Added.
  • platform/gtk/editing/selection/caret-ltr-expected.png: Added.
  • platform/gtk/editing/selection/caret-ltr-expected.txt: Added.
  • platform/gtk/editing/selection/caret-ltr-right-expected.checksum: Added.
  • platform/gtk/editing/selection/caret-ltr-right-expected.png: Added.
  • platform/gtk/editing/selection/caret-ltr-right-expected.txt: Added.
  • platform/gtk/editing/selection/caret-rtl-2-left-expected.checksum: Added.
  • platform/gtk/editing/selection/caret-rtl-2-left-expected.png: Added.
  • platform/gtk/editing/selection/caret-rtl-2-left-expected.txt: Added.
  • platform/gtk/editing/selection/caret-rtl-right-expected.checksum: Added.
  • platform/gtk/editing/selection/caret-rtl-right-expected.png: Added.
  • platform/gtk/editing/selection/caret-rtl-right-expected.txt: Added.
04:12 Changeset [74632] by dpranke@chromium.org

2010-12-24 Dirk Pranke <dpranke@chromium.org>

Reviewed by Kenneth Russell.

Modify new-run-webkit-tests to shut up and exit when all the threads
are wedged so that test runs complete faster and we don't end up
w/ enormous log files.

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

  • Scripts/webkitpy/layout_tests/layout_package/message_broker.py:
  • Scripts/webkitpy/layout_tests/layout_package/message_broker_unittest.py:
04:05 Changeset [74631] by commit-queue@webkit.org

2010-12-24 Noel Gordon <noel.gordon@gmail.com>

Reviewed by Andreas Kling.

[chromium] Fix a style nit in the JPEG encoder
https://bugs.webkit.org/show_bug.cgi?id=51574

  • platform/image-encoders/skia/JPEGImageEncoder.cpp: nit begone.
04:02 Changeset [74630] by apavlov@chromium.org

2010-12-23 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Maciej Stachowiak.

Web Inspector: CSS shorthand properties expand unnecessarily
https://bugs.webkit.org/show_bug.cgi?id=36304

"border-radius" is a shorthand property that has a "non-standard" format and longhands, see
http://www.w3.org/TR/css3-background/#border-radius for reference. CSSParser does not use the
standard parseShorthand() or parse4Values() methods but instead a custom parseBorderRadius() method.
This method didn't use to create a ShorthandScope instance and set the m_implicitShorthand value
appropriately when adding implicit longhands, thus they were added as normal properties
found in the CSS (neither implicit, nor longhands). This is now fixed.

WebCore:

  • css/CSSParser.cpp: (WebCore::CSSParser::parseBorderRadius):

LayoutTests:

  • inspector/elements-panel-styles-expected.txt:
  • inspector/resources/elements-panel-styles.css: (.foo):
03:54 Changeset [74629] by jschuh@chromium.org

2010-12-24 Justin Schuh <jschuh@chromium.org>

Reviewed by Darin Adler.

setTargetAttributeAnimatedValue should check for a null shadowTreeElement.
https://bugs.webkit.org/show_bug.cgi?id=51576

  • svg/custom/animate-disallowed-use-element-expected.txt: Added.
  • svg/custom/animate-disallowed-use-element.svg: Added.

2010-12-24 Justin Schuh <jschuh@chromium.org>

Reviewed by Darin Adler.

setTargetAttributeAnimatedValue should check for a null shadowTreeElement.
https://bugs.webkit.org/show_bug.cgi?id=51576

Disallowed elements may be pruned from the shadow tree, leaving a null
shadowTreeElement. We should check for that before setting attributes.

Test: svg/custom/animate-disallowed-use-element.svg

  • svg/SVGAnimationElement.cpp: (WebCore::SVGAnimationElement::setTargetAttributeAnimatedValue):
03:50 Changeset [74628] by evan@chromium.org

2010-12-24 Evan Martin <evan@chromium.org>

Reviewed by Darin Adler.

TextMetrics::width is an integer, but the member is a float
https://bugs.webkit.org/show_bug.cgi?id=51566

Change the return type to a float.

No tests, fixing a compiler warning.
(I was unable to produce a difference in the values exposed to pages
with a float versus the previous conversion to an integer.)

  • html/TextMetrics.h: (WebCore::TextMetrics::width):
03:47 Changeset [74627] by andreip@google.com

2010-12-24 Andrei Popescu <andreip@google.com>

Unreviewed build fix.

Add Chromium-mac test expectation for the layout test below.
[Chromium] http/tests/xmlhttprequest/basic-auth-nopassword.html needs Mac specific test expectations
https://bugs.webkit.org/show_bug.cgi?id=51589

  • platform/chromium-mac/http/tests/xmlhttprequest/basic-auth-nopassword-expected.txt: Added.
03:27 Changeset [74626] by commit-queue@webkit.org

2010-12-24 Ryuan Choi <ryuan.choi@samsung.com>

Reviewed by Kenneth Rohde Christiansen.

[EFL] Add option to enable Touch Events.
https://bugs.webkit.org/show_bug.cgi?id=49125

Add ENABLE_TOUCH_EVENTS option.

  • cmake/OptionsEfl.cmake:
  • cmakeconfig.h.cmake:

2010-12-24 Ryuan Choi <ryuan.choi@samsung.com>

Reviewed by Kenneth Rohde Christiansen.

[EFL] Add option to enable Touch Events.
https://bugs.webkit.org/show_bug.cgi?id=49125

Add files for EFL to support TOUCH_EVENTS.

  • CMakeLists.txt:
  • CMakeListsEfl.txt:
  • platform/PlatformTouchEvent.h: Add EFL related constructor.
  • platform/PlatformTouchPoint.h: ditto.
  • platform/efl/PlatformTouchEventEfl.cpp: Added. (WebCore::PlatformTouchEvent::PlatformTouchEvent):
  • platform/efl/PlatformTouchPointEfl.cpp: Added. (WebCore::PlatformTouchPoint::PlatformTouchPoint):

2010-12-24 Ryuan Choi <ryuan.choi@samsung.com>

Reviewed by Kenneth Rohde Christiansen.

[EFL] Add option to enable Touch Events.
https://bugs.webkit.org/show_bug.cgi?id=49125

Implement ewk_frame_feed_touch_event to feed touch events. Because EFL
doesn't have touch events, Application should generate and pass it.

  • WebCoreSupport/ChromeClientEfl.h: (WebCore::ChromeClientEfl::needTouchEvents):
  • ewk/ewk_frame.cpp: (ewk_frame_feed_touch_event):
  • ewk/ewk_frame.h:
03:23 Changeset [74625] by gyuyoung.kim@samsung.com

2010-12-24 Gyuyoung Kim <gyuyoung.kim@samsung.com>

Reviewed by Kenneth Rohde Christiansen.

[EFL] Return an empty string instead of 'application/octet-stream' in getMIMETypeForExtension()
https://bugs.webkit.org/show_bug.cgi?id=51457

WebKit EFL can't play html5 audio because of wrong mime type checking.
So, 'application/octet-stream' is changed with an empty string.

  • platform/efl/MIMETypeRegistryEfl.cpp: (WebCore::MIMETypeRegistry::getMIMETypeForExtension): Returns an empty string instead of application/octet-stream when it can't find proper mime type.
03:09 Changeset [74624] by andreip@google.com

2010-12-24 Andrei Popescu <andreip@google.com>

Unreviewed. Update Chromium test expectations.

Mark editing/input/ime-composition-clearpreedit.html.html as TEXT.
This test was added in http://trac.webkit.org/changeset/74605 and
fails in Chromium.

  • platform/chromium/test_expectations.txt:
03:05 Changeset [74623] by eric@webkit.org

2010-12-24 Eric Seidel <eric@webkit.org>

Reviewed by David Levin.

commit-queue should better explain how it will re-open bugs and follow duplicate chains
https://bugs.webkit.org/show_bug.cgi?id=51549

I've had multiple people ask me about the commit-queue's behavior
regarding reporting flaky tests. I figured it would be best to just
have the queue explain itself in the bugs it files.

  • Scripts/webkitpy/tool/bot/flakytestreporter.py:
  • Scripts/webkitpy/tool/bot/flakytestreporter_unittest.py:
02:38 Changeset [74622] by jschuh@chromium.org

2010-12-24 Justin Schuh <jschuh@chromium.org>

Reviewed by Darin Adler.

SVGFontFaceElement::rebuildFontFace() should exit when not in document
https://bugs.webkit.org/show_bug.cgi?id=51571

  • svg/custom/use-invalid-font-face-expected.txt: Added.
  • svg/custom/use-invalid-font-face.svg: Added.

2010-12-24 Justin Schuh <jschuh@chromium.org>

Reviewed by Darin Adler.

SVGFontFaceElement::rebuildFontFace() should exit when not in document
https://bugs.webkit.org/show_bug.cgi?id=51571

We were hitting a NULL deref crash. Since most of the callers checked
inDocument() anyway, I moved it into the start of rebuildFontFace.

Test: svg/custom/use-invalid-font-face.svg

  • svg/SVGFontFaceElement.cpp: (WebCore::SVGFontFaceElement::parseMappedAttribute): (WebCore::SVGFontFaceElement::rebuildFontFace): (WebCore::SVGFontFaceElement::childrenChanged):
02:19 Changeset [74621] by hamaji@chromium.org

2010-12-24 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by Eric Seidel.

check-webkit-style: should ignore "wrong" namings in JavaScriptCore/qt
https://bugs.webkit.org/show_bug.cgi?id=51467

  • Scripts/webkitpy/style/checker.py:
  • Scripts/webkitpy/style/checker_unittest.py:
02:12 Changeset [74620] by philn@webkit.org

2010-12-21 Philippe Normand <pnormand@igalia.com>

Reviewed by Eric Seidel.

[GTK] minimal build with --database fails
https://bugs.webkit.org/show_bug.cgi?id=51394

  • webkit/webkitprivate.cpp: (webkit_init): Use cacheStorage() only if the OFFLINE_WEB_APPLICATIONS feature is enabled.
02:00 Changeset [74619] by jschuh@chromium.org

2010-12-24 Justin Schuh <jschuh@chromium.org>

Reviewed by Darin Adler.

Remove unnecessary check in CSSCanvasValue::canvasDestroyed().
https://bugs.webkit.org/show_bug.cgi?id=51564

This check was duplicating an ASSERT for a case fixed in r73927.
Now that the bug is fixed the check is no longer necessary.

Removing dead code, so no new tests needed.

  • css/CSSCanvasValue.cpp: (WebCore::CSSCanvasValue::canvasDestroyed):
01:01 Changeset [74618] by abarth@webkit.org

Move JavaScriptGlue into Sources
https://bugs.webkit.org/show_bug.cgi?id=51583

Rubber-stamped by Eric Seidel.

Actually perform the move.

  • JavaScriptGlue: Removed.
  • JavaScriptGlue/Configurations: Removed.
  • JavaScriptGlue/Configurations/Base.xcconfig: Removed.
  • JavaScriptGlue/Configurations/DebugRelease.xcconfig: Removed.
  • JavaScriptGlue/Configurations/JavaScriptGlue.xcconfig: Removed.
  • JavaScriptGlue/Configurations/Version.xcconfig: Removed.
  • JavaScriptGlue/English.lproj: Removed.
  • JavaScriptGlue/English.lproj/InfoPlist.strings: Removed.
  • JavaScriptGlue/ForwardingHeaders: Removed.
  • JavaScriptGlue/ForwardingHeaders/masm: Removed.
  • JavaScriptGlue/ForwardingHeaders/masm/X86Assembler.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/profiler: Removed.
  • JavaScriptGlue/ForwardingHeaders/profiler/Profiler.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/runtime: Removed.
  • JavaScriptGlue/ForwardingHeaders/runtime/CallFrame.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/ASCIICType.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/AlwaysInline.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/Assertions.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/Atomics.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/Bitmap.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/BumpPointerAllocator.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/CrossThreadRefCounted.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/CurrentTime.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/DateInstanceCache.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/DateMath.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/FastAllocBase.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/FastMalloc.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/FixedArray.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/Forward.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/HashCountedSet.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/HashFunctions.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/HashMap.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/HashSet.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/HashTraits.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/ListHashSet.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/ListRefPtr.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/Locker.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/MainThread.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/MathExtras.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/Noncopyable.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/NotFound.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/OSAllocator.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/OwnArrayPtr.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/OwnArrayPtrCommon.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/OwnFastMallocPtr.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/OwnPtr.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/OwnPtrCommon.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/PageAllocation.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/PageAllocationAligned.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/PageBlock.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/PageReservation.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/PassOwnArrayPtr.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/PassOwnPtr.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/PassRefPtr.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/Platform.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/PossiblyNull.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/PtrAndFlags.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/RandomNumber.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/RefCounted.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/RefPtr.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/StackBounds.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/StdLibExtras.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/StringHasher.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/ThreadSafeShared.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/ThreadSpecific.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/Threading.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/ThreadingPrimitives.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/UnusedParam.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/VMTags.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/ValueCheck.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/Vector.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/VectorTraits.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/WTFThreadData.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/text: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/text/CString.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/text/StringHash.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/text/StringImpl.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/text/StringImplBase.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/text/TextPosition.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/text/WTFString.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/unicode: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/unicode/Unicode.h: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/unicode/icu: Removed.
  • JavaScriptGlue/ForwardingHeaders/wtf/unicode/icu/UnicodeIcu.h: Removed.
  • JavaScriptGlue/Info.plist: Removed.
  • JavaScriptGlue/JSBase.cpp: Removed.
  • JavaScriptGlue/JSBase.h: Removed.
  • JavaScriptGlue/JSObject.cpp: Removed.
  • JavaScriptGlue/JSObject.h: Removed.
  • JavaScriptGlue/JSRun.cpp: Removed.
  • JavaScriptGlue/JSRun.h: Removed.
  • JavaScriptGlue/JSUtils.cpp: Removed.
  • JavaScriptGlue/JSUtils.h: Removed.
  • JavaScriptGlue/JSValueWrapper.cpp: Removed.
  • JavaScriptGlue/JSValueWrapper.h: Removed.
  • JavaScriptGlue/JavaScriptGlue.cpp: Removed.
  • JavaScriptGlue/JavaScriptGlue.exp: Removed.
  • JavaScriptGlue/JavaScriptGlue.h: Removed.
  • JavaScriptGlue/JavaScriptGlue.xcodeproj: Removed.
  • JavaScriptGlue/JavaScriptGlue.xcodeproj/project.pbxproj: Removed.
  • JavaScriptGlue/LICENSE: Removed.
  • JavaScriptGlue/Makefile: Removed.
  • JavaScriptGlue/UserObjectImp.cpp: Removed.
  • JavaScriptGlue/UserObjectImp.h: Removed.
  • JavaScriptGlue/config.h: Removed.
  • JavaScriptGlue/icu: Removed.
  • JavaScriptGlue/icu/LICENSE: Removed.
  • JavaScriptGlue/icu/README: Removed.
  • JavaScriptGlue/icu/unicode: Removed.
  • JavaScriptGlue/icu/unicode/platform.h: Removed.
  • JavaScriptGlue/icu/unicode/putil.h: Removed.
  • JavaScriptGlue/icu/unicode/uchar.h: Removed.
  • JavaScriptGlue/icu/unicode/ucnv.h: Removed.
  • JavaScriptGlue/icu/unicode/ucnv_err.h: Removed.
  • JavaScriptGlue/icu/unicode/uconfig.h: Removed.
  • JavaScriptGlue/icu/unicode/uenum.h: Removed.
  • JavaScriptGlue/icu/unicode/uiter.h: Removed.
  • JavaScriptGlue/icu/unicode/umachine.h: Removed.
  • JavaScriptGlue/icu/unicode/urename.h: Removed.
  • JavaScriptGlue/icu/unicode/ustring.h: Removed.
  • JavaScriptGlue/icu/unicode/utf.h: Removed.
  • JavaScriptGlue/icu/unicode/utf16.h: Removed.
  • JavaScriptGlue/icu/unicode/utf8.h: Removed.
  • JavaScriptGlue/icu/unicode/utf_old.h: Removed.
  • JavaScriptGlue/icu/unicode/utypes.h: Removed.
  • JavaScriptGlue/icu/unicode/uversion.h: Removed.
  • JavaScriptGlue/testjsglue.cpp: Removed.
  • Makefile:
  • Sources: Added.
  • Sources/JavaScriptGlue: Copied from JavaScriptGlue.
  • Sources/Makefile: Added.

Tools:

These tools refer to JavaScriptGlue and need to be updated to its new
location.

  • Scripts/build-webkit:
  • Scripts/do-file-rename:
  • Scripts/do-webcore-rename:
  • Scripts/webkitpy/common/config/build.py:
  • Scripts/webkitpy/common/config/build_unittest.py:

Websites/bugs.webkit.org:

PrettyPatch keeps a list of the top-level directories that exist in
trac.

  • PrettyPatch/PrettyPatch.rb:
00:06 Changeset [74617] by paroga@webkit.org

2010-12-24 Patrick Gansterer <paroga@webkit.org>

Reviewed by Eric Seidel.

[WINCE] Add CPU(MIPS) detection
https://bugs.webkit.org/show_bug.cgi?id=51342

WinCE usually defines MIPS and _MIPS_.

  • wtf/Platform.h:

12/23/10:

22:14 Changeset [74616] by lforschler@apple.com

2010-12-23 Lucas Forschler <lforschler@apple.com>

Reviewed by Eric Seidel.

Bug 51539 - update iexploder scripts to point to new location

  • Scripts/run-iexploder-tests:
  • Scripts/update-iexploder-cssproperties:
22:11 Changeset [74615] by mitz@apple.com

Make dragging work again after r74591.

Reviewed by Sam Weinig.

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::didReceiveEvent): Fixed a typo.

22:05 Changeset [74614] by yutak@chromium.org

2010-12-23 Yuta Kitamura <yutak@chromium.org>

Unreviewed. Update GTK's Skipped file.

[GTK] http/tests/inspector/console-websocket-error.html hangs
https://bugs.webkit.org/show_bug.cgi?id=51380

  • platform/gtk/Skipped:
21:26 Changeset [74613] by ariya@webkit.org

Unreviewed GTK fix after r74609: put the right file name in the Skipped file.

20:59 Changeset [74612] by yutak@chromium.org

2010-12-23 Yuta Kitamura <yutak@chromium.org>

Unreviewed. Update Chromium test expectations.

Mark http/tests/inspector/console-websocket-error.html as SLOW on WIN DEBUG.

  • platform/chromium/test_expectations.txt:
20:50 Changeset [74611] by yutak@chromium.org

2010-12-22 Yuta Kitamura <yutak@chromium.org>

Reviewed by Alexey Proskuryakov.

[GTK] http/tests/inspector/console-websocket-error.html hangs
https://bugs.webkit.org/show_bug.cgi?id=51380

The test should call dumpConsoleMessages() even if WebSocket is not supported
or a WebSocket object goes to an unexpected state.

  • http/tests/inspector/console-websocket-error.html:
  • platform/gtk/Skipped:
  • platform/mac-leopard/http/tests/inspector/console-websocket-error-expected.txt:
  • platform/mac/http/tests/inspector/console-websocket-error-expected.txt:
20:31 Changeset [74610] by evan@chromium.org

2010-12-23 Evan Martin <evan@chromium.org>

Reviewed by Darin Adler.

ViewportArguments.h uses an enum to intialize a bool
https://bugs.webkit.org/show_bug.cgi?id=50982

Initialize userScalable to true, rather than -1.
No tests; fixes a compiler warning.

  • dom/ViewportArguments.h: (WebCore::ViewportArguments::ViewportArguments):
20:06 Changeset [74609] by ariya@webkit.org

[Qt] User name/password stripped from URLs
https://bugs.webkit.org/show_bug.cgi?id=36232

Patch by Jan Erik Hanssen <jhanssen@sencha.com> on 2010-12-23
Reviewed by Adam Barth.

WebCore:

If a username but no password or no username but a password is present
in a URL, QtWebKit won't send any authentication information down to Qt
as it should.

Tests: http/tests/xmlhttprequest/basic-auth-nopassword.html

http/tests/xmlhttprequest/basic-auth-nouser.html

  • platform/network/qt/ResourceHandleQt.cpp:

(WebCore::ResourceHandle::start):
(WebCore::ResourceHandle::loadResourceSynchronously):

LayoutTests:

Test authentication cases where a username but no password and
no username but a password is present.

  • http/tests/xmlhttprequest/basic-auth-nopassword-expected.txt: Added.
  • http/tests/xmlhttprequest/basic-auth-nopassword.html: Added.
  • http/tests/xmlhttprequest/basic-auth-nouser-expected.txt: Added.
  • http/tests/xmlhttprequest/basic-auth-nouser.html: Added.
  • http/tests/xmlhttprequest/resources/basic-auth-nouserpass/basic-auth-nouserpass.php: Added.
  • platform/gtk/Skipped:
  • platform/mac-leopard/http/tests/xmlhttprequest/basic-auth-nopassword-expected.txt: Added.
  • platform/mac-snowleopard/http/tests/xmlhttprequest/basic-auth-nopassword-expected.txt: Added.
  • platform/mac-tiger/http/tests/xmlhttprequest/basic-auth-nopassword-expected.txt: Added.
  • platform/qt/Skipped:
19:40 Changeset [74608] by rniwa@webkit.org

2010-12-23 Ryosuke Niwa <rniwa@webkit.org>

Unreviewed Qt rebaseline for r74593.

Added the forgotten blank line at the end.

  • platform/qt/editing/pasteboard/copy-in-password-field-expected.txt:
19:18 Changeset [74607] by rniwa@webkit.org

2010-12-23 Ryosuke Niwa <rniwa@webkit.org>

Unreviewed Qt rebaseline for r74593.

  • platform/qt/editing/pasteboard/copy-in-password-field-expected.txt: Added.
18:56 Changeset [74606] by rniwa@webkit.org

2010-12-23 Ryosuke Niwa <rniwa@webkit.org>

More unreviewed Chromium rebaselines for r74593.

  • platform/chromium-linux/editing/deleting/smart-delete-002-expected.txt:
  • platform/chromium-linux/editing/pasteboard/subframe-dragndrop-1-expected.txt:
  • platform/chromium-win/editing/pasteboard/paste-xml-expected.txt:
18:53 Changeset [74605] by commit-queue@webkit.org

2010-12-23 Jan Erik Hanssen <jhanssen@sencha.com>

Reviewed by Alexey Proskuryakov.

[Qt] Composition text is not removed from the editor when composition is cancelled
https://bugs.webkit.org/show_bug.cgi?id=29391

Ensure that the composition text is cleared when an empty QInputMethodEvent
is received and the editor is in composition mode.

  • editing/input/ime-composition-clearpreedit-expected.txt: Added.
  • editing/input/ime-composition-clearpreedit.html: Added.
  • platform/gtk/Skipped:
  • platform/mac-wk2/Skipped:
  • platform/qt-wk2/Skipped:

2010-12-23 Jan Erik Hanssen <jhanssen@sencha.com>

Reviewed by Alexey Proskuryakov.

[Qt] Composition text is not removed from the editor when composition is cancelled
https://bugs.webkit.org/show_bug.cgi?id=29391

Ensure that the composition text is cleared when an empty QInputMethodEvent
is received and the editor is in composition mode.

  • Api/qwebpage.cpp: (QWebPagePrivate::inputMethodEvent):
18:38 Changeset [74604] by rniwa@webkit.org

2010-12-23 Benjamin Kalman <kalman@chromium.org>

Reviewed by Ryosuke Niwa.

Selection becomes stale when CharacterData is manipulated directly
https://bugs.webkit.org/show_bug.cgi?id=51389

Rebaseline GTK tests.

  • platform/gtk/editing/deleting/collapse-whitespace-3587601-fix-expected.txt:
  • platform/gtk/editing/deleting/delete-3608430-fix-expected.txt:
  • platform/gtk/editing/deleting/delete-3608445-fix-expected.txt:
  • platform/gtk/editing/deleting/delete-3608462-fix-expected.txt:
  • platform/gtk/editing/deleting/delete-and-undo-expected.txt:
  • platform/gtk/editing/deleting/delete-block-merge-contents-010-expected.txt:
  • platform/gtk/editing/deleting/delete-br-007-expected.txt:
  • platform/gtk/editing/deleting/delete-contiguous-ws-001-expected.txt:
  • platform/gtk/editing/deleting/delete-leading-ws-001-expected.txt:
  • platform/gtk/editing/deleting/delete-line-end-ws-001-expected.txt:
  • platform/gtk/editing/deleting/delete-line-end-ws-002-expected.txt:
  • platform/gtk/editing/deleting/delete-listitem-001-expected.txt:
  • platform/gtk/editing/deleting/delete-tab-002-expected.txt:
  • platform/gtk/editing/deleting/delete-tab-004-expected.txt:
  • platform/gtk/editing/deleting/delete-trailing-ws-002-expected.txt:
  • platform/gtk/editing/deleting/delete-ws-fixup-001-expected.txt:
  • platform/gtk/editing/deleting/delete-ws-fixup-002-expected.txt:
  • platform/gtk/editing/deleting/forward-delete-expected.txt:
  • platform/gtk/editing/deleting/smart-delete-003-expected.txt:
  • platform/gtk/editing/deleting/smart-delete-004-expected.txt:
  • platform/gtk/editing/execCommand/insertHorizontalRule-expected.txt:
  • platform/gtk/editing/inserting/insert-3654864-fix-expected.txt:
  • platform/gtk/editing/inserting/insert-after-delete-001-expected.txt:
  • platform/gtk/editing/inserting/insert-div-003-expected.txt:
  • platform/gtk/editing/inserting/insert-div-024-expected.txt:
  • platform/gtk/editing/inserting/insert-space-in-empty-doc-expected.txt:
  • platform/gtk/editing/inserting/insert-text-with-newlines-expected.txt:
  • platform/gtk/editing/inserting/typing-001-expected.txt:
  • platform/gtk/editing/inserting/typing-002-expected.txt:
  • platform/gtk/editing/inserting/typing-003-expected.txt:
  • platform/gtk/editing/inserting/typing-around-br-001-expected.txt:
  • platform/gtk/editing/inserting/typing-around-image-001-expected.txt:
  • platform/gtk/editing/pasteboard/4076267-2-expected.txt:
  • platform/gtk/editing/pasteboard/4076267-3-expected.txt:
  • platform/gtk/editing/pasteboard/cut-text-001-expected.txt:
  • platform/gtk/editing/pasteboard/paste-text-001-expected.txt:
  • platform/gtk/editing/pasteboard/paste-xml-expected.txt:
  • platform/gtk/editing/pasteboard/subframe-dragndrop-1-expected.txt:
  • platform/gtk/editing/spelling/spelling-expected.txt:
  • platform/gtk/editing/style/style-3681552-fix-001-expected.txt:
  • platform/gtk/editing/style/style-3681552-fix-002-expected.txt:
  • platform/gtk/editing/style/typing-style-001-expected.txt:
  • platform/gtk/editing/style/typing-style-002-expected.txt:
  • platform/gtk/editing/style/unbold-in-bold-expected.txt:
  • platform/gtk/editing/undo/4063751-expected.txt:
  • platform/gtk/editing/undo/redo-typing-001-expected.txt:
  • platform/gtk/editing/undo/undo-combined-delete-boundary-expected.txt:
  • platform/gtk/editing/undo/undo-combined-delete-expected.txt:
  • platform/gtk/editing/undo/undo-delete-boundary-expected.txt:
  • platform/gtk/editing/undo/undo-delete-expected.txt:
  • platform/gtk/editing/undo/undo-forward-delete-boundary-expected.txt:
  • platform/gtk/editing/undo/undo-forward-delete-expected.txt:
  • platform/gtk/editing/undo/undo-misspellings-expected.txt:
18:33 Changeset [74603] by rniwa@webkit.org

2010-12-23 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Darin Adler.

queryCommandValue("FormatBlock") does not respect editing boundaries
https://bugs.webkit.org/show_bug.cgi?id=51219

Added test cases to ensure queryCommandValue("FormatBlock") respects editing boundaries.

  • editing/execCommand/query-format-block-expected.txt:
  • editing/execCommand/script-tests/query-format-block.js:

2010-12-23 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Darin Adler.

queryCommandValue("FormatBlock") does not respect editing boundaries
https://bugs.webkit.org/show_bug.cgi?id=51219

Fixed the bug by checking the element we found in elementForFormatBlockCommand
does not contain the editable root.

  • editing/FormatBlockCommand.cpp: (WebCore::FormatBlockCommand::elementForFormatBlockCommand):
18:17 Changeset [74602] by rniwa@webkit.org

2010-12-23 Ryosuke Niwa <rniwa@webkit.org>

Unreviewed Chromium rebaselines for r74593.

  • platform/chromium-linux/editing/deleting/smart-delete-001-expected.txt:
  • platform/chromium-linux/editing/deleting/smart-delete-003-expected.txt:
  • platform/chromium-linux/editing/deleting/smart-delete-004-expected.txt:
  • platform/chromium-win/editing/deleting/collapse-whitespace-3587601-fix-expected.txt:
  • platform/chromium-win/editing/deleting/delete-3608430-fix-expected.txt:
  • platform/chromium-win/editing/deleting/delete-3608445-fix-expected.txt:
  • platform/chromium-win/editing/deleting/delete-3608462-fix-expected.txt:
  • platform/chromium-win/editing/deleting/delete-after-span-ws-001-expected.txt:
  • platform/chromium-win/editing/deleting/delete-after-span-ws-002-expected.txt:
  • platform/chromium-win/editing/deleting/delete-after-span-ws-003-expected.txt:
  • platform/chromium-win/editing/deleting/delete-and-undo-expected.txt:
  • platform/chromium-win/editing/deleting/delete-block-merge-contents-010-expected.txt:
  • platform/chromium-win/editing/deleting/delete-block-table-expected.txt:
  • platform/chromium-win/editing/deleting/delete-br-007-expected.txt:
  • platform/chromium-win/editing/deleting/delete-character-001-expected.txt:
  • platform/chromium-win/editing/deleting/delete-contiguous-ws-001-expected.txt:
  • platform/chromium-win/editing/deleting/delete-hr-expected.txt:
  • platform/chromium-win/editing/deleting/delete-leading-ws-001-expected.txt:
  • platform/chromium-win/editing/deleting/delete-line-end-ws-001-expected.txt:
  • platform/chromium-win/editing/deleting/delete-line-end-ws-002-expected.txt:
  • platform/chromium-win/editing/deleting/delete-listitem-001-expected.txt:
  • platform/chromium-win/editing/deleting/delete-selection-001-expected.txt:
  • platform/chromium-win/editing/deleting/delete-tab-001-expected.txt:
  • platform/chromium-win/editing/deleting/delete-tab-002-expected.txt:
  • platform/chromium-win/editing/deleting/delete-tab-003-expected.txt:
  • platform/chromium-win/editing/deleting/delete-tab-004-expected.txt:
  • platform/chromium-win/editing/deleting/delete-trailing-ws-001-expected.txt:
  • platform/chromium-win/editing/deleting/delete-trailing-ws-002-expected.txt:
  • platform/chromium-win/editing/deleting/delete-ws-fixup-001-expected.txt:
  • platform/chromium-win/editing/deleting/delete-ws-fixup-002-expected.txt:
  • platform/chromium-win/editing/deleting/forward-delete-expected.txt:
  • platform/chromium-win/editing/deleting/non-smart-delete-expected.txt:
  • platform/chromium-win/editing/deleting/smart-delete-001-expected.txt:
  • platform/chromium-win/editing/deleting/smart-delete-003-expected.txt:
  • platform/chromium-win/editing/deleting/smart-delete-004-expected.txt:
  • platform/chromium-win/editing/execCommand/insertHorizontalRule-expected.txt:
  • platform/chromium-win/editing/inserting/4278698-expected.txt:
  • platform/chromium-win/editing/inserting/editing-empty-divs-expected.txt:
  • platform/chromium-win/editing/inserting/insert-3654864-fix-expected.txt:
  • platform/chromium-win/editing/inserting/insert-after-delete-001-expected.txt:
  • platform/chromium-win/editing/inserting/insert-br-003-expected.txt:
  • platform/chromium-win/editing/inserting/insert-br-006-expected.txt:
  • platform/chromium-win/editing/inserting/insert-br-007-expected.txt:
  • platform/chromium-win/editing/inserting/insert-br-008-expected.txt:
  • platform/chromium-win/editing/inserting/insert-br-009-expected.txt:
  • platform/chromium-win/editing/inserting/insert-br-at-tabspan-002-expected.txt:
  • platform/chromium-win/editing/inserting/insert-br-at-tabspan-003-expected.txt:
  • platform/chromium-win/editing/inserting/insert-div-003-expected.txt:
  • platform/chromium-win/editing/inserting/insert-div-006-expected.txt:
  • platform/chromium-win/editing/inserting/insert-div-008-expected.txt:
  • platform/chromium-win/editing/inserting/insert-div-024-expected.txt:
  • platform/chromium-win/editing/inserting/insert-space-in-empty-doc-expected.txt:
  • platform/chromium-win/editing/inserting/insert-tab-003-expected.txt:
  • platform/chromium-win/editing/inserting/insert-text-at-tabspan-001-expected.txt:
  • platform/chromium-win/editing/inserting/insert-text-at-tabspan-002-expected.txt:
  • platform/chromium-win/editing/inserting/insert-text-with-newlines-expected.txt:
  • platform/chromium-win/editing/inserting/typing-001-expected.txt:
  • platform/chromium-win/editing/inserting/typing-002-expected.txt:
  • platform/chromium-win/editing/inserting/typing-003-expected.txt:
  • platform/chromium-win/editing/inserting/typing-around-br-001-expected.txt:
  • platform/chromium-win/editing/inserting/typing-around-image-001-expected.txt:
  • platform/chromium-win/editing/pasteboard/4076267-2-expected.txt:
  • platform/chromium-win/editing/pasteboard/4076267-3-expected.txt:
  • platform/chromium-win/editing/pasteboard/bad-placeholder-expected.txt:
  • platform/chromium-win/editing/pasteboard/cut-text-001-expected.txt:
  • platform/chromium-win/editing/pasteboard/interchange-newline-1-expected.txt:
  • platform/chromium-win/editing/pasteboard/merge-end-1-expected.txt:
  • platform/chromium-win/editing/pasteboard/merge-end-2-expected.txt:
  • platform/chromium-win/editing/pasteboard/merge-end-4-expected.txt:
  • platform/chromium-win/editing/pasteboard/merge-end-list-expected.txt:
  • platform/chromium-win/editing/pasteboard/merge-end-table-expected.txt:
  • platform/chromium-win/editing/pasteboard/paste-match-style-001-expected.txt:
  • platform/chromium-win/editing/pasteboard/paste-text-001-expected.txt:
  • platform/chromium-win/editing/pasteboard/paste-text-019-expected.txt:
  • platform/chromium-win/editing/pasteboard/paste-text-at-tabspan-001-expected.txt:
  • platform/chromium-win/editing/pasteboard/paste-text-at-tabspan-002-expected.txt:
  • platform/chromium-win/editing/pasteboard/paste-text-at-tabspan-003-expected.txt:
  • platform/chromium-win/editing/pasteboard/pasting-object-expected.txt:
  • platform/chromium-win/editing/pasteboard/smart-paste-002-expected.txt:
  • platform/chromium-win/editing/pasteboard/smart-paste-006-expected.txt:
  • platform/chromium-win/editing/pasteboard/smart-paste-007-expected.txt:
  • platform/chromium-win/editing/spelling/spelling-expected.txt:
  • platform/chromium-win/editing/style/style-3681552-fix-001-expected.txt:
  • platform/chromium-win/editing/style/style-3681552-fix-002-expected.txt:
  • platform/chromium-win/editing/style/style-boundary-002-expected.txt:
  • platform/chromium-win/editing/style/style-boundary-005-expected.txt:
  • platform/chromium-win/editing/style/typing-style-001-expected.txt:
  • platform/chromium-win/editing/style/typing-style-002-expected.txt:
  • platform/chromium-win/editing/style/unbold-in-bold-expected.txt:
  • platform/chromium-win/editing/undo/4063751-expected.txt:
  • platform/chromium-win/editing/undo/redo-typing-001-expected.txt:
  • platform/chromium-win/editing/undo/undo-combined-delete-boundary-expected.txt:
  • platform/chromium-win/editing/undo/undo-combined-delete-expected.txt:
  • platform/chromium-win/editing/undo/undo-delete-boundary-expected.txt:
  • platform/chromium-win/editing/undo/undo-delete-expected.txt:
  • platform/chromium-win/editing/undo/undo-forward-delete-boundary-expected.txt:
  • platform/chromium-win/editing/undo/undo-forward-delete-expected.txt:
  • platform/chromium-win/editing/undo/undo-misspellings-expected.txt:
  • platform/chromium-win/editing/undo/undo-typing-001-expected.txt:
  • platform/chromium-win/editing/unsupported-content/list-type-after-expected.txt:
  • platform/chromium-win/editing/unsupported-content/list-type-before-expected.txt:
  • platform/chromium-win/editing/unsupported-content/table-type-after-expected.txt:
  • platform/chromium-win/editing/unsupported-content/table-type-before-expected.txt:
18:14 Changeset [74601] by rniwa@webkit.org

2010-12-23 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Darin Adler.

FormatBlock doesn't preserve typing style but it should
https://bugs.webkit.org/show_bug.cgi?id=51140

Added a test to type text after bolding the typing style and executing a FormatBlock command.
Because FormatBlock command preserves typing style, the typed text should be in bold.

  • editing/execCommand/format-block-typing-style.html: Added.

2010-12-23 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Darin Adler.

FormatBlock doesn't preserve typing style but it should
https://bugs.webkit.org/show_bug.cgi?id=51140

Fixed the bug by making preserveTypingStyle return true.

Test: editing/execCommand/format-block-typing-style.html

  • editing/FormatBlockCommand.h: (WebCore::FormatBlockCommand::preservesTypingStyle):
17:40 Changeset [74600] by barraclough@apple.com

Rename RegexCompiler.cpp to RegexPattern.cpp.
Implicitly call compileRegex from RegexPattern's constructor.

Reviewed by Sam Weinig.

(JSC::RegExp::compile):

  • yarr/RegexCompiler.cpp: Removed.
  • yarr/RegexCompiler.h: Removed.
  • yarr/RegexInterpreter.cpp:
  • yarr/RegexJIT.cpp:
  • yarr/RegexPattern.cpp: Copied from JavaScriptCore/yarr/RegexCompiler.cpp.

(JSC::Yarr::compileRegex):
(JSC::Yarr::RegexPattern::RegexPattern):

  • yarr/RegexPattern.h:
17:36 Changeset [74599] by rniwa@webkit.org

2010-12-23 Ryosuke Niwa <rniwa@webkit.org>

Unreviewed Qt rebaselines for r74593.

  • platform/qt/editing/deleting/delete-3608445-fix-expected.txt:
  • platform/qt/editing/deleting/delete-after-span-ws-001-expected.txt:
  • platform/qt/editing/deleting/delete-after-span-ws-002-expected.txt:
  • platform/qt/editing/deleting/delete-after-span-ws-003-expected.txt:
  • platform/qt/editing/deleting/delete-block-merge-contents-010-expected.txt:
  • platform/qt/editing/deleting/delete-character-001-expected.txt:
  • platform/qt/editing/deleting/delete-hr-expected.txt:
  • platform/qt/editing/deleting/delete-line-end-ws-001-expected.txt:
  • platform/qt/editing/deleting/delete-selection-001-expected.txt:
  • platform/qt/editing/deleting/delete-trailing-ws-001-expected.txt:
  • platform/qt/editing/deleting/delete-trailing-ws-002-expected.txt:
  • platform/qt/editing/deleting/non-smart-delete-expected.txt:
  • platform/qt/editing/deleting/smart-delete-001-expected.txt:
  • platform/qt/editing/deleting/smart-delete-003-expected.txt:
  • platform/qt/editing/deleting/smart-delete-004-expected.txt:
  • platform/qt/editing/inserting/insert-3654864-fix-expected.txt:
  • platform/qt/editing/inserting/insert-after-delete-001-expected.txt:
  • platform/qt/editing/inserting/insert-br-003-expected.txt:
  • platform/qt/editing/inserting/insert-br-006-expected.txt:
  • platform/qt/editing/inserting/insert-br-007-expected.txt:
  • platform/qt/editing/inserting/insert-br-008-expected.txt:
  • platform/qt/editing/inserting/insert-br-009-expected.txt:
  • platform/qt/editing/inserting/insert-br-at-tabspan-002-expected.txt:
  • platform/qt/editing/inserting/insert-br-at-tabspan-003-expected.txt:
  • platform/qt/editing/inserting/insert-div-003-expected.txt:
  • platform/qt/editing/inserting/insert-div-006-expected.txt:
  • platform/qt/editing/inserting/insert-div-008-expected.txt:
  • platform/qt/editing/inserting/insert-div-024-expected.txt:
  • platform/qt/editing/inserting/insert-space-in-empty-doc-expected.txt:
  • platform/qt/editing/inserting/insert-text-at-tabspan-001-expected.txt:
  • platform/qt/editing/inserting/insert-text-at-tabspan-002-expected.txt:
  • platform/qt/editing/inserting/typing-001-expected.txt:
  • platform/qt/editing/inserting/typing-002-expected.txt:
  • platform/qt/editing/inserting/typing-003-expected.txt:
  • platform/qt/editing/inserting/typing-around-br-001-expected.txt:
  • platform/qt/editing/inserting/typing-around-image-001-expected.txt:
  • platform/qt/editing/pasteboard/4076267-3-expected.txt:
  • platform/qt/editing/pasteboard/cut-text-001-expected.txt:
  • platform/qt/editing/pasteboard/interchange-newline-1-expected.txt:
  • platform/qt/editing/pasteboard/merge-end-1-expected.txt:
  • platform/qt/editing/pasteboard/merge-end-2-expected.txt:
  • platform/qt/editing/pasteboard/merge-end-4-expected.txt:
  • platform/qt/editing/pasteboard/merge-end-list-expected.txt:
  • platform/qt/editing/pasteboard/merge-end-table-expected.txt:
  • platform/qt/editing/pasteboard/paste-match-style-001-expected.txt:
  • platform/qt/editing/pasteboard/paste-text-019-expected.txt:
  • platform/qt/editing/pasteboard/paste-text-at-tabspan-002-expected.txt:
  • platform/qt/editing/pasteboard/paste-text-at-tabspan-003-expected.txt:
  • platform/qt/editing/pasteboard/pasting-object-expected.txt:
  • platform/qt/editing/pasteboard/smart-paste-002-expected.txt:
  • platform/qt/editing/pasteboard/smart-paste-006-expected.txt:
  • platform/qt/editing/pasteboard/smart-paste-007-expected.txt:
  • platform/qt/editing/style/remove-underline-from-stylesheet-expected.txt:
  • platform/qt/editing/undo/redo-style-expected.txt:
17:19 Changeset [74598] by evan@chromium.org

2010-12-23 Evan Martin <evan@chromium.org>

Reviewed by Ryosuke Niwa.

[chromium] LayoutTestController warning in float->int conversion
https://bugs.webkit.org/show_bug.cgi?id=51553

Rather than reading an int, converting to a float, then getting a compiler
warning on passing the float to something that expects an int, instead
just pass around ints.

No tests, fixes a compiler warning.

  • DumpRenderTree/chromium/LayoutTestController.cpp: (parsePageSizeParameters): (LayoutTestController::pageNumberForElementById): (LayoutTestController::numberOfPages):
17:16 Changeset [74597] by jberlin@webkit.org

Need WebKit2 API to create a SecurityOrigin from protocol, host, and port
https://bugs.webkit.org/show_bug.cgi?id=51563

Reviewed by Sam Weinig.

WebCore:

Add SecurityOrigin::create(const String& protocol, const String& host, int port).

  • WebCore.exp.in:

Export the symbols for WebCore::SecurityOrigin::create(String, String, int).

  • page/SecurityOrigin.cpp:

(WebCore::SecurityOrigin::createFromDatabaseIdentifier):
Use the constant MaxAllowedPort.
(WebCore::SecurityOrigin::create):
Sanitizing based off of that done in createFromDatabaseIdentifier.

  • page/SecurityOrigin.h:

WebKit2:

Add WKSecurityOriginCreate(WKStringRef protocol, WKStringRef host, int port).

  • Shared/API/c/WKSecurityOrigin.cpp:

(WKSecurityOriginCreateFromIdentifier):
Renamed so as not to clash with the new WKSecurityOriginCreate.
(WKSecurityOriginCreate):

  • Shared/API/c/WKSecurityOrigin.h:
  • Shared/WebSecurityOrigin.h:

(WebKit::WebSecurityOrigin::create):

17:11 Changeset [74596] by paroga@webkit.org

2010-12-23 Patrick Gansterer <paroga@webkit.org>

Unreviewed build fix for WinCE after r73802.

Reapply r70812:
Port ContextMenuItemWin.cpp to WinCE
https://bugs.webkit.org/show_bug.cgi?id=48407

  • platform/win/ContextMenuItemWin.cpp:
17:03 Changeset [74595] by paroga@webkit.org

2010-12-23 Patrick Gansterer <paroga@webkit.org>

Unreviewed build fix for WinCE after r74360.

Move the OS(WINDOWS) section after the OS(WINCE) section
and add missing argument to the getStackMax call.

  • wtf/StackBounds.cpp: (WTF::StackBounds::initialize):
16:55 Changeset [74594] by abarth@webkit.org

2010-12-23 Adam Barth <abarth@webkit.org>

Remove include of non-existant file.

  • bindings/v8/custom/V8DOMWindowCustom.cpp:
16:22 Changeset [74593] by rniwa@webkit.org

2010-12-23 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Darin Adler.

Selection becomes stale when CharacterData is manipulated directly
https://bugs.webkit.org/show_bug.cgi?id=51389

The bug was caused by SelectionController's not adjusting selection when CharacterData's
m_data is modified by its API.

Fixed the bug by adding textWillBeReplaced to SelectionController and calling it in
CharacterData::textWillBeReplaced. Added newLength to setDataAndUpdate, which is passed to
textWillBeReplaced because inserting, replacing, or removing data may require moving
the offset of the selection end points (done by shouldRemovePositionAfterAdoptingTextReplacement).

Also fixed the condition for determining whether or not a node intersects the selection
in respondToNodeModification, which is extracted from nodeWillBeRemoved. The old condition
failed to recognize certain cases when selection started and ended with non-zero offsets.

Tests: editing/selection/character-data-mutation-crash.html

editing/selection/character-data-mutation.html

  • dom/CharacterData.cpp: (WebCore::CharacterData::setData): Passes newLegnth to setDataAndUpdate. (WebCore::CharacterData::appendData): Ditto. (WebCore::CharacterData::insertData): Ditto. (WebCore::CharacterData::deleteData): Ditto. (WebCore::CharacterData::replaceData): Ditto. (WebCore::CharacterData::setDataAndUpdate): Added newLegnth to the argument list, and renamed lengthOfReplacedData to oldLength; calls textWillBeReplaced.
  • dom/CharacterData.h:
  • editing/SelectionController.cpp: (WebCore::removingNodeRemovesPosition): Removed indentations from blank lines. (WebCore::SelectionController::nodeWillBeRemoved): Calls nodeWillBeRemoved. (WebCore::SelectionController::respondToNodeModification): Extracted from nodeWillBeRemoved; fixed the logic to figure out whether a node is inside the range or not. (WebCore::shouldRemovePositionAfterAdoptingTextReplacement): Added. (WebCore::SelectionController::textWillBeReplaced): Added.
  • editing/SelectionController.h:

2010-12-23 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Darin Adler.

Selection becomes stale when CharacterData is manipulated directly
https://bugs.webkit.org/show_bug.cgi?id=51389

Added tests to ensure WebKit adjusts or clears selection when the start container
or the end container of the selection's first range is modified by CharacterData's API.

Many editing tests had to be rebaselined due to different editing delegate dumps
caused by WebKit's properly adjusting selection's offsets as editing commands modify the DOM.

  • editing/deleting/delete-by-word-001-expected.txt:
  • editing/deleting/delete-by-word-002-expected.txt:
  • editing/deleting/delete-ligature-001-expected.txt:
  • editing/deleting/delete-ligature-002-expected.txt:
  • editing/deleting/delete-ligature-003-expected.txt:
  • editing/deleting/skip-virama-001-expected.txt:
  • editing/execCommand/createLink-expected.txt:
  • editing/execCommand/hilitecolor-expected.txt:
  • editing/execCommand/unlink-expected.txt:
  • editing/inserting/insert-thai-characters-001-expected.txt:
  • editing/pasteboard/copy-in-password-field-expected.txt:
  • editing/pasteboard/paste-into-anchor-text-expected.txt:
  • editing/selection/character-data-mutation-crash-expected.txt: Added.
  • editing/selection/character-data-mutation-crash.html: Added.
  • editing/selection/character-data-mutation-expected.txt: Added.
  • editing/selection/character-data-mutation.html: Added.
  • editing/style/remove-underline-from-stylesheet-expected.txt:
  • editing/style/typing-style-003-expected.txt:
  • editing/undo/redo-style-expected.txt:
  • platform/mac/editing/deleting/collapse-whitespace-3587601-fix-expected.txt:
  • platform/mac/editing/deleting/delete-3608430-fix-expected.txt:
  • platform/mac/editing/deleting/delete-3608445-fix-expected.txt:
  • platform/mac/editing/deleting/delete-3608462-fix-expected.txt:
  • platform/mac/editing/deleting/delete-after-span-ws-001-expected.txt:
  • platform/mac/editing/deleting/delete-after-span-ws-002-expected.txt:
  • platform/mac/editing/deleting/delete-after-span-ws-003-expected.txt:
  • platform/mac/editing/deleting/delete-and-undo-expected.txt:
  • platform/mac/editing/deleting/delete-block-merge-contents-010-expected.txt:
  • platform/mac/editing/deleting/delete-block-table-expected.txt:
  • platform/mac/editing/deleting/delete-br-007-expected.txt:
  • platform/mac/editing/deleting/delete-character-001-expected.txt:
  • platform/mac/editing/deleting/delete-contiguous-ws-001-expected.txt:
  • platform/mac/editing/deleting/delete-hr-expected.txt:
  • platform/mac/editing/deleting/delete-leading-ws-001-expected.txt:
  • platform/mac/editing/deleting/delete-line-end-ws-001-expected.txt:
  • platform/mac/editing/deleting/delete-line-end-ws-002-expected.txt:
  • platform/mac/editing/deleting/delete-listitem-001-expected.txt:
  • platform/mac/editing/deleting/delete-selection-001-expected.txt:
  • platform/mac/editing/deleting/delete-tab-001-expected.txt:
  • platform/mac/editing/deleting/delete-tab-002-expected.txt:
  • platform/mac/editing/deleting/delete-tab-003-expected.txt:
  • platform/mac/editing/deleting/delete-tab-004-expected.txt:
  • platform/mac/editing/deleting/delete-trailing-ws-001-expected.txt:
  • platform/mac/editing/deleting/delete-trailing-ws-002-expected.txt:
  • platform/mac/editing/deleting/delete-ws-fixup-001-expected.txt:
  • platform/mac/editing/deleting/delete-ws-fixup-002-expected.txt:
  • platform/mac/editing/deleting/forward-delete-expected.txt:
  • platform/mac/editing/deleting/non-smart-delete-expected.txt:
  • platform/mac/editing/deleting/smart-delete-001-expected.txt:
  • platform/mac/editing/deleting/smart-delete-002-expected.txt:
  • platform/mac/editing/deleting/smart-delete-003-expected.txt:
  • platform/mac/editing/deleting/smart-delete-004-expected.txt:
  • platform/mac/editing/execCommand/insertHorizontalRule-expected.txt:
  • platform/mac/editing/input/firstrectforcharacterrange-styled-expected.txt:
  • platform/mac/editing/input/text-input-controller-expected.txt:
  • platform/mac/editing/inserting/4278698-expected.txt:
  • platform/mac/editing/inserting/editing-empty-divs-expected.txt:
  • platform/mac/editing/inserting/insert-3654864-fix-expected.txt:
  • platform/mac/editing/inserting/insert-after-delete-001-expected.txt:
  • platform/mac/editing/inserting/insert-br-003-expected.txt:
  • platform/mac/editing/inserting/insert-br-006-expected.txt:
  • platform/mac/editing/inserting/insert-br-007-expected.txt:
  • platform/mac/editing/inserting/insert-br-008-expected.txt:
  • platform/mac/editing/inserting/insert-br-009-expected.txt:
  • platform/mac/editing/inserting/insert-br-at-tabspan-002-expected.txt:
  • platform/mac/editing/inserting/insert-br-at-tabspan-003-expected.txt:
  • platform/mac/editing/inserting/insert-div-003-expected.txt:
  • platform/mac/editing/inserting/insert-div-006-expected.txt:
  • platform/mac/editing/inserting/insert-div-008-expected.txt:
  • platform/mac/editing/inserting/insert-div-024-expected.txt:
  • platform/mac/editing/inserting/insert-space-in-empty-doc-expected.txt:
  • platform/mac/editing/inserting/insert-tab-003-expected.txt:
  • platform/mac/editing/inserting/insert-text-at-tabspan-001-expected.txt:
  • platform/mac/editing/inserting/insert-text-at-tabspan-002-expected.txt:
  • platform/mac/editing/inserting/insert-text-with-newlines-expected.txt:
  • platform/mac/editing/inserting/typing-001-expected.txt:
  • platform/mac/editing/inserting/typing-002-expected.txt:
  • platform/mac/editing/inserting/typing-003-expected.txt:
  • platform/mac/editing/inserting/typing-around-br-001-expected.txt:
  • platform/mac/editing/inserting/typing-around-image-001-expected.txt:
  • platform/mac/editing/pasteboard/4076267-2-expected.txt:
  • platform/mac/editing/pasteboard/4076267-3-expected.txt:
  • platform/mac/editing/pasteboard/bad-placeholder-expected.txt:
  • platform/mac/editing/pasteboard/cut-text-001-expected.txt:
  • platform/mac/editing/pasteboard/emacs-ctrl-a-k-y-expected.txt:
  • platform/mac/editing/pasteboard/emacs-ctrl-k-y-001-expected.txt:
  • platform/mac/editing/pasteboard/interchange-newline-1-expected.txt:
  • platform/mac/editing/pasteboard/merge-end-1-expected.txt:
  • platform/mac/editing/pasteboard/merge-end-2-expected.txt:
  • platform/mac/editing/pasteboard/merge-end-4-expected.txt:
  • platform/mac/editing/pasteboard/merge-end-list-expected.txt:
  • platform/mac/editing/pasteboard/merge-end-table-expected.txt:
  • platform/mac/editing/pasteboard/paste-match-style-001-expected.txt:
  • platform/mac/editing/pasteboard/paste-text-001-expected.txt:
  • platform/mac/editing/pasteboard/paste-text-019-expected.txt:
  • platform/mac/editing/pasteboard/paste-text-at-tabspan-001-expected.txt:
  • platform/mac/editing/pasteboard/paste-text-at-tabspan-002-expected.txt:
  • platform/mac/editing/pasteboard/paste-text-at-tabspan-003-expected.txt:
  • platform/mac/editing/pasteboard/paste-xml-expected.txt:
  • platform/mac/editing/pasteboard/pasting-object-expected.txt:
  • platform/mac/editing/pasteboard/smart-paste-002-expected.txt:
  • platform/mac/editing/pasteboard/smart-paste-006-expected.txt:
  • platform/mac/editing/pasteboard/smart-paste-007-expected.txt:
  • platform/mac/editing/pasteboard/subframe-dragndrop-1-expected.txt:
  • platform/mac/editing/spelling/spelling-expected.txt:
  • platform/mac/editing/style/style-3681552-fix-001-expected.txt:
  • platform/mac/editing/style/style-3681552-fix-002-expected.txt:
  • platform/mac/editing/style/style-boundary-002-expected.txt:
  • platform/mac/editing/style/style-boundary-005-expected.txt:
  • platform/mac/editing/style/typing-style-001-expected.txt:
  • platform/mac/editing/style/typing-style-002-expected.txt:
  • platform/mac/editing/style/unbold-in-bold-expected.txt:
  • platform/mac/editing/undo/4063751-expected.txt:
  • platform/mac/editing/undo/redo-typing-001-expected.txt:
  • platform/mac/editing/undo/undo-combined-delete-boundary-expected.txt:
  • platform/mac/editing/undo/undo-combined-delete-expected.txt:
  • platform/mac/editing/undo/undo-delete-boundary-expected.txt:
  • platform/mac/editing/undo/undo-delete-expected.txt:
  • platform/mac/editing/undo/undo-forward-delete-boundary-expected.txt:
  • platform/mac/editing/undo/undo-forward-delete-expected.txt:
  • platform/mac/editing/undo/undo-misspellings-expected.txt:
  • platform/mac/editing/undo/undo-typing-001-expected.txt:
  • platform/mac/editing/unsupported-content/list-type-after-expected.txt:
  • platform/mac/editing/unsupported-content/list-type-before-expected.txt:
  • platform/mac/editing/unsupported-content/table-type-after-expected.txt:
  • platform/mac/editing/unsupported-content/table-type-before-expected.txt:
16:00 Changeset [74592] by abarth@webkit.org

2010-12-23 Adam Barth <abarth@webkit.org>

Reviewed by Darin Adler.

Remove unneeded V8 bindings files
https://bugs.webkit.org/show_bug.cgi?id=51570

  • WebCore.gypi:
  • bindings/generic/BindingDOMWindow.h: Removed.
  • bindings/v8/specialization/V8BindingDOMWindow.h: Removed.
15:55 Changeset [74591] by andersca@apple.com

2010-12-23 Anders Carlsson <andersca@apple.com>

Reviewed by Simon Fraser.

Ensure that we are not getting too many mouse move events
https://bugs.webkit.org/show_bug.cgi?id=51568
<rdar://problem/7881470>
<rdar://problem/8735512>

Implement throttling of mouse move events.

  • UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::WebPageProxy): Initialize m_processingMouseMoveEvent to false.

(WebKit::WebPageProxy::handleMouseEvent):
If we're already processing a mouse move events, just update
m_nextMouseMoveEvent and return.


(WebKit::WebPageProxy::didReceiveEvent):
If we received a mouse move event, send the next mouse move event if there is one.

15:43 Changeset [74590] by mitz@apple.com

Build fix.

  • editing/EditorCommand.cpp:

(WebCore::enabledDismissCorrectionPanel):

15:42 Changeset [74589] by darin@apple.com

Try to fix crash seen on GTK bot.

  • editing/EditorCommand.cpp:

(WebCore::internalCommand): Add back empty string check I accidentally deleted in r74580.

  • WebCore.xcodeproj/project.pbxproj: Let Xcode edit the file.
15:36 Changeset [74588] by bweinstein@apple.com

Assertion when loading www.yahoo.com in WebKit2 under decodeResourceRequest
https://bugs.webkit.org/show_bug.cgi?id=51562

Reviewed by Anders Carlsson.

Don't assert that our data reference isn't empty when we try to get the data from it.
If it is empty, just return 0.

All callers are safe with this function returning 0.

  • Platform/CoreIPC/DataReference.h:

(CoreIPC::DataReference::data): Change the assert to an early return.

  • Shared/cf/ArgumentCodersCF.cpp:

(CoreIPC::decode): Add an assert, and remove a ternarny operator because the function now does

this for us (if the data is empty, return 0).

15:15 Changeset [74587] by kbr@google.com

2010-12-23 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r74561.
http://trac.webkit.org/changeset/74561
https://bugs.webkit.org/show_bug.cgi?id=51565

"Broke Chromium UI tests on Vista" (Requested by kbr_google on
#webkit).

  • src/GraphicsContext3DChromium.cpp: (WebCore::GraphicsContext3DInternal::paintRenderingResultsToCanvas):

2010-12-23 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r74561.
http://trac.webkit.org/changeset/74561
https://bugs.webkit.org/show_bug.cgi?id=51565

"Broke Chromium UI tests on Vista" (Requested by kbr_google on
#webkit).

  • platform/graphics/chromium/ImageLayerChromium.cpp: (WebCore::ImageLayerChromium::updateContentsIfDirty):
  • platform/graphics/skia/GraphicsContext3DSkia.cpp: (WebCore::GraphicsContext3D::getImageData):
15:10 Changeset [74586] by commit-queue@webkit.org

2010-12-23 Yongjun Zhang <yongjun_zhang@apple.com>

Reviewed by Darin Adler.

WebKit crashes at DebuggerCallFrame::functionName() if m_callFrame is the top global callframe.
https://bugs.webkit.org/show_bug.cgi?id=38535

WebScriptDebugger in WebKit has empty implementations for willExecuteProgram and didExecuteProgram. As a result,
if the top call frame is from a program, WebKitScriptDebugger doesn't record that callframe as the top frame, and
WebScriptDebugger's callframe stack is wrong from this point. That could cause crash if we trying to access the top
call frame from this stack when an exception throws because the saved top frame could be invalid.

To fix that, we need to maintain the call frame stack in willExecuteProgram and didExecuteProgram, as we did in
callEvent and returnEvent.

  • WebView/WebScriptDebugger.mm: (WebScriptDebugger::willExecuteProgram): (WebScriptDebugger::didExecuteProgram):
15:08 Changeset [74585] by kbr@google.com

2010-12-23 Kenneth Russell <kbr@google.com>

Unreviewed Chromium test expectations update. Fix botched update
of fast/css/box-shadow.html expectation; needed to be FAIL, not
IMAGE+TEXT IMAGE.

  • platform/chromium/test_expectations.txt:
14:49 Changeset [74584] by weinig@apple.com

Fix incorrect assertion causing all pages with subframes to crash.

  • UIProcess/WebFrameProxy.cpp:

(WebKit::WebFrameProxy::appendChild):

14:40 Changeset [74583] by darin@apple.com

Fixed out-of-order ChangeLog.

14:34 Changeset [74582] by andersca@apple.com

2010-12-23 Darin Adler <darin@apple.com>

Reviewed by Sam Weinig.

WKView should not try to do asynchronous validation for selectors that are not editor commands
https://bugs.webkit.org/show_bug.cgi?id=51555

  • UIProcess/API/mac/WKView.mm: (-[WKView validateUserInterfaceItem:]): Removed the special case for startSpeaking. Added call to commandIsSupportedFromMenuOrKeyBinding so we only try to do validation for commands that are supported. Tweaked comments and added some bug numbers. (-[WKView _setUserInterfaceItemState:enabled:state:]): Tweaked comment and added bug number.
14:23 Changeset [74581] by mihaip@chromium.org

2010-12-23 Mihai Parparita <mihaip@chromium.org>

Reviewed by Kent Tamura.

[Chromium] Add WebThemeEngine for Mac to allow scrollbar rendering to be overridden for the DRT
https://bugs.webkit.org/show_bug.cgi?id=51507

Adds WebThemeEngine for the Mac (it already has parallel definitions for
Windows and Linux) so that scrollbar thumb rendering can be overridden
for the DRT (to be consistent with the NSScroller-based rendering
used by the Mac port).

No new tests since the functionality is not exposed yet. Eventually will
result in the Chromium/Mac port being able to use the same pixel
baselines for layout tests as the Mac port.

  • platform/chromium/ChromiumBridge.h:
  • platform/chromium/ScrollbarThemeChromiumMac.mm: (WebCore::scrollbarStateToThemeState): (WebCore::ScrollbarThemeChromiumMac::paint):

2010-12-23 Mihai Parparita <mihaip@chromium.org>

Reviewed by Kent Tamura.

[Chromium] Add WebThemeEngine for Mac to allow scrollbar rendering to be overridden for the DRT
https://bugs.webkit.org/show_bug.cgi?id=51507

Adds WebThemeEngine for the Mac (it already has parallel definitions for
Windows and Linux) so that scrollbar thumb rendering can be overridden
for the DRT (to be consistent with the NSScroller-based rendering
used by the Mac port).

  • public/mac/WebThemeEngine.h: Added. (WebKit::WebThemeEngine::paintScrollbarThumb):
  • src/AssertMatchingEnums.cpp:
  • src/ChromiumBridge.cpp: (WebCore::ChromiumBridge::paintScrollbarThumb):
14:20 Changeset [74580] by darin@apple.com

2010-12-23 Darin Adler <darin@apple.com>

Reviewed by Sam Weinig.

WKView should not try to do asynchronous validation for selectors that are not editor commands
https://bugs.webkit.org/show_bug.cgi?id=51555

  • WebCore.exp.in: Added commandIsSupportedFromMenuOrKeyBinding.
  • editing/Editor.h: Reordered arguments in the Editor::Command constructor and the data members too so the frame is last. Added commandIsSupportedFromMenuOrKeyBinding.
  • editing/EditorCommand.cpp: (WebCore::supported): Removed the EditorCommandSource argument. These functions are now only used when called from DOM. (WebCore::supportedFromMenuOrKeyBinding): Ditto. (WebCore::supportedCopyCut): Ditto. (WebCore::supportedPaste): Ditto. (WebCore::enabledDismissCorrectionPanel): Changed the supported function to an enabled function. It was incorrect to say that this is "supported" only when the correction panel is up. Correct to say that it is "enabled" only then. And also probably OK to enable it even when the selection is not in editable text, as long as the panel is up. (WebCore::createCommandMap): Moved conditional commands out of the main array into a separate section at the end. (WebCore::internalCommand): Added. (WebCore::Editor::command): Changed to use the new internalCommand function and simplified by relying on the null check in the Command constructor. (WebCore::Editor::commandIsSupportedFromMenuOrKeyBinding): Added. (WebCore::Editor::Command::Command): Removed unneeded initialization of m_source, which is never looked at if m_command is 0. Added feature of passing a null command pointer to the non-default constructor. (WebCore::Editor::Command::isSupported): Changed to only call the per-command isSupported function when the command source is DOM. Accordingly that function is now called isSupportedFromDOM.

2010-12-23 Darin Adler <darin@apple.com>

Reviewed by Sam Weinig.

WKView should not try to do asynchronous validation for selectors that are not editor commands
https://bugs.webkit.org/show_bug.cgi?id=51555

  • UIProcess/API/mac/WKView.mm: (-[WKView validateUserInterfaceItem:]): Removed the special case for startSpeaking. Added call to commandIsSupportedFromMenuOrKeyBinding so we only try to do validation for commands that are supported. Tweaked comments and added some bug numbers. (-[WKView _setUserInterfaceItemState:enabled:state:]): Tweaked comment and added bug number.
14:15 Changeset [74579] by mdelaney@apple.com

2010-12-23 Matthew Delaney <mdelaney@apple.com>

Reviewed by Simon Fraser.

Incorrect layer property used noted as changed in GraphicsLayerCA::setAcceleratesDrawing
https://bugs.webkit.org/show_bug.cgi?id=51560

No new tests.

  • platform/graphics/ca/GraphicsLayerCA.cpp: (WebCore::GraphicsLayerCA::setAcceleratesDrawing):
13:55 WebKit Team edited by evan@chromium.org
add myself to commiter list (diff)
13:28 Changeset [74578] by ajwong@chromium.org

Unreviewed. Chromium layout test expectations update.
https://bugs.webkit.org/show_bug.cgi?id=51500

Rebaseline media control layout tests due to pixel diff from r74535

  • platform/chromium-linux/media/audio-controls-rendering-expected.checksum:
  • platform/chromium-linux/media/audio-controls-rendering-expected.png:
  • platform/chromium-linux/media/controls-after-reload-expected.checksum:
  • platform/chromium-linux/media/controls-after-reload-expected.png:
  • platform/chromium-linux/media/controls-strict-expected.checksum:
  • platform/chromium-linux/media/controls-strict-expected.png:
  • platform/chromium-linux/media/controls-styling-expected.checksum:
  • platform/chromium-linux/media/controls-styling-expected.png:
  • platform/chromium-linux/media/controls-without-preload-expected.checksum:
  • platform/chromium-linux/media/controls-without-preload-expected.png:
  • platform/chromium-linux/media/video-controls-rendering-expected.checksum:
  • platform/chromium-linux/media/video-controls-rendering-expected.png:
  • platform/chromium-linux/media/video-display-toggle-expected.checksum:
  • platform/chromium-linux/media/video-display-toggle-expected.png:
  • platform/chromium-linux/media/video-empty-source-expected.checksum:
  • platform/chromium-linux/media/video-empty-source-expected.png:
  • platform/chromium-linux/media/video-no-audio-expected.checksum:
  • platform/chromium-linux/media/video-no-audio-expected.png:
  • platform/chromium-linux/media/video-volume-slider-expected.checksum:
  • platform/chromium-linux/media/video-volume-slider-expected.png:
  • platform/chromium-mac/media/audio-controls-rendering-expected.checksum:
  • platform/chromium-mac/media/audio-controls-rendering-expected.png:
  • platform/chromium-mac/media/audio-controls-rendering-expected.txt:
  • platform/chromium-mac/media/controls-after-reload-expected.checksum:
  • platform/chromium-mac/media/controls-after-reload-expected.png:
  • platform/chromium-mac/media/controls-after-reload-expected.txt:
  • platform/chromium-mac/media/controls-strict-expected.checksum:
  • platform/chromium-mac/media/controls-strict-expected.png:
  • platform/chromium-mac/media/controls-strict-expected.txt:
  • platform/chromium-mac/media/controls-styling-expected.checksum:
  • platform/chromium-mac/media/controls-styling-expected.png:
  • platform/chromium-mac/media/controls-styling-expected.txt:
  • platform/chromium-mac/media/controls-without-preload-expected.checksum:
  • platform/chromium-mac/media/controls-without-preload-expected.png:
  • platform/chromium-mac/media/controls-without-preload-expected.txt:
  • platform/chromium-mac/media/video-display-toggle-expected.checksum:
  • platform/chromium-mac/media/video-display-toggle-expected.png:
  • platform/chromium-mac/media/video-display-toggle-expected.txt:
  • platform/chromium-mac/media/video-empty-source-expected.checksum:
  • platform/chromium-mac/media/video-empty-source-expected.png:
  • platform/chromium-mac/media/video-empty-source-expected.txt:
  • platform/chromium-mac/media/video-no-audio-expected.checksum:
  • platform/chromium-mac/media/video-no-audio-expected.png:
  • platform/chromium-mac/media/video-no-audio-expected.txt:
  • platform/chromium-mac/media/video-volume-slider-expected.checksum:
  • platform/chromium-mac/media/video-volume-slider-expected.png:
  • platform/chromium-mac/media/video-volume-slider-expected.txt:
  • platform/chromium-win/media/audio-controls-rendering-expected.checksum:
  • platform/chromium-win/media/audio-controls-rendering-expected.png:
  • platform/chromium-win/media/audio-controls-rendering-expected.txt:
  • platform/chromium-win/media/controls-after-reload-expected.checksum:
  • platform/chromium-win/media/controls-after-reload-expected.png:
  • platform/chromium-win/media/controls-after-reload-expected.txt:
  • platform/chromium-win/media/controls-strict-expected.checksum:
  • platform/chromium-win/media/controls-strict-expected.png:
  • platform/chromium-win/media/controls-strict-expected.txt:
  • platform/chromium-win/media/controls-styling-expected.checksum:
  • platform/chromium-win/media/controls-styling-expected.png:
  • platform/chromium-win/media/controls-styling-expected.txt:
  • platform/chromium-win/media/controls-without-preload-expected.checksum:
  • platform/chromium-win/media/controls-without-preload-expected.png:
  • platform/chromium-win/media/controls-without-preload-expected.txt:
  • platform/chromium-win/media/video-controls-rendering-expected.checksum:
  • platform/chromium-win/media/video-controls-rendering-expected.png:
  • platform/chromium-win/media/video-controls-rendering-expected.txt:
  • platform/chromium-win/media/video-display-toggle-expected.checksum:
  • platform/chromium-win/media/video-display-toggle-expected.png:
  • platform/chromium-win/media/video-display-toggle-expected.txt:
  • platform/chromium-win/media/video-empty-source-expected.checksum:
  • platform/chromium-win/media/video-empty-source-expected.png:
  • platform/chromium-win/media/video-empty-source-expected.txt:
  • platform/chromium-win/media/video-no-audio-expected.checksum:
  • platform/chromium-win/media/video-no-audio-expected.png:
  • platform/chromium-win/media/video-no-audio-expected.txt:
  • platform/chromium-win/media/video-volume-slider-expected.checksum:
  • platform/chromium-win/media/video-volume-slider-expected.png:
  • platform/chromium-win/media/video-volume-slider-expected.txt:
  • platform/chromium/test_expectations.txt:
13:08 Changeset [74577] by commit-queue@webkit.org

2010-12-23 Jan Erik Hanssen <jhanssen@sencha.com>

Reviewed by Andreas Kling.

[Qt] copy and paste greek symbols to Word, TextEdit results in ?'s being shown
https://bugs.webkit.org/show_bug.cgi?id=35635

Explicitly set text/html data on QClipboard as UTF-8 and also include a <meta>
tag informing the OS that the data is in UTF-8 format.

  • platform/qt/PasteboardQt.cpp: (WebCore::Pasteboard::writeSelection):

2010-12-23 Jan Erik Hanssen <jhanssen@sencha.com>

Reviewed by Andreas Kling.

[Qt] copy and paste greek symbols to Word, TextEdit results in ?'s being shown
https://bugs.webkit.org/show_bug.cgi?id=35635

Test that QClipboard contains the charset information and unicode data
after copying unicode text.

  • tests/qwebpage/tst_qwebpage.cpp: (tst_QWebPage::macCopyUnicodeToClipboard):
13:02 Changeset [74576] by kbr@google.com

2010-12-23 Kenneth Russell <kbr@google.com>

Unreviewed Chromium expectations update. Suppress crashes seen on
Windows with fast/canvas/webgl/canvas-test.html and
fast/canvas/webgl/gl-pixelstorei.html.

  • platform/chromium/test_expectations.txt:
12:58 Changeset [74575] by weinig@apple.com

<rdar://problem/8799032>
REGRESSION(r74436) Crash opening many pages with WebKit2

Reviewed by Anders Carlsson.

  • Shared/cf/ArgumentCodersCF.cpp:

(CoreIPC::decode):
Fix paste-o. Decode into a CFDataRef not a CFDictionaryRef
when decoding CFDataRefs.

12:38 Changeset [74574] by jschuh@chromium.org

2010-12-23 Justin Schuh <jschuh@chromium.org>

Reviewed by Eric Seidel.

CSSCursorImageValue destructor should clear referenced element.
https://bugs.webkit.org/show_bug.cgi?id=51417

Calls correct method to clear image and renames method to avoid future confusion.

Test: svg/css/cursor-image-replace.svg

  • css/CSSCursorImageValue.cpp: (WebCore::CSSCursorImageValue::~CSSCursorImageValue):
  • svg/SVGElement.cpp: (WebCore::SVGElement::cursorImageValueRemoved):
  • svg/SVGElement.h:

2010-12-23 Justin Schuh <jschuh@chromium.org>

Reviewed by Eric Seidel.

CSSCursorImageValue destructor should clear referenced element.
https://bugs.webkit.org/show_bug.cgi?id=51417

  • svg/css/cursor-image-replace-expected.txt: Added.
  • svg/css/cursor-image-replace.svg: Added.
12:22 Changeset [74573] by kbr@google.com

2010-12-23 Kenneth Russell <kbr@google.com>

Unreviewed Chromium expectations update. Allow
fast/css/box-shadow.html to fail with IMAGE as well as IMAGE+TEXT.

  • platform/chromium/test_expectations.txt:
12:11 Changeset [74572] by weinig@apple.com

Fix build.

  • src/FrameLoaderClientImpl.cpp:

(WebKit::FrameLoaderClientImpl::didSaveToPageCache):
(WebKit::FrameLoaderClientImpl::didRestoreFromPageCache):

12:10 Changeset [74571] by weinig@apple.com

WebKit2 needs to mirror the frame tree in the UIProcess
https://bugs.webkit.org/show_bug.cgi?id=51546

Reviewed by Darin Adler.

WebCore:

  • Add client functions to notify that a frame has been added or removed from the page cache.
  • history/CachedFrame.cpp:

(WebCore::CachedFrameBase::restore):
(WebCore::CachedFrame::CachedFrame):

  • history/CachedFrame.h:
  • loader/EmptyClients.h:

(WebCore::EmptyFrameLoaderClient::didSaveToPageCache):
(WebCore::EmptyFrameLoaderClient::didRestoreFromPageCache):

  • loader/FrameLoaderClient.h:

WebKit/chromium:

  • Add client functions to notify that a frame has been added or removed from the page cache.
  • src/FrameLoaderClientImpl.cpp:

(WebKit::WebFrameLoaderClientImpl::didSaveToPageCache):
(WebKit::WebFrameLoaderClientImpl::didRestoreFromPageCache):

  • src/FrameLoaderClientImpl.h:

WebKit/efl:

  • Add client functions to notify that a frame has been added or removed from the page cache.
  • WebCoreSupport/FrameLoaderClientEfl.cpp:

(WebCore::FrameLoaderClientEfl::didSaveToPageCache):
(WebCore::FrameLoaderClientEfl::didRestoreFromPageCache):

  • WebCoreSupport/FrameLoaderClientEfl.h:

WebKit/gtk:

  • Add client functions to notify that a frame has been added or removed from the page cache.
  • WebCoreSupport/FrameLoaderClientGtk.cpp:

(WebKit::FrameLoaderClient::didSaveToPageCache):
(WebKit::FrameLoaderClient::didRestoreFromPageCache):

  • WebCoreSupport/FrameLoaderClientGtk.h:

WebKit/haiku:

  • Add client functions to notify that a frame has been added or removed from the page cache.
  • WebCoreSupport/FrameLoaderClientHaiku.cpp:

(WebCore::FrameLoaderClientHaiku::didSaveToPageCache):
(WebCore::FrameLoaderClientHaiku::didRestoreFromPageCache):

  • WebCoreSupport/FrameLoaderClientHaiku.h:

WebKit/mac:

  • Add client functions to notify that a frame has been added or removed from the page cache.
  • WebCoreSupport/WebFrameLoaderClient.h:
  • WebCoreSupport/WebFrameLoaderClient.mm:

(WebFrameLoaderClient::didSaveToPageCache):
(WebFrameLoaderClient::didRestoreFromPageCache):

WebKit/qt:

  • Add client functions to notify that a frame has been added or removed from the page cache.
  • WebCoreSupport/FrameLoaderClientQt.cpp:

(WebCore::FrameLoaderClientQt::didSaveToPageCache):
(WebCore::FrameLoaderClientQt::didRestoreFromPageCache):

  • WebCoreSupport/FrameLoaderClientQt.h:

WebKit/win:

  • Add client functions to notify that a frame has been added or removed from the page cache.
  • WebCoreSupport/WebFrameLoaderClient.cpp:

(WebFrameLoaderClient::didSaveToPageCache):
(WebFrameLoaderClient::didRestoreFromPageCache):

  • WebCoreSupport/WebFrameLoaderClient.h:

WebKit/wince:

  • Add client functions to notify that a frame has been added or removed from the page cache.
  • WebCoreSupport/FrameLoaderClientWinCE.cpp:

(WebKit::FrameLoaderClientWinCE::didSaveToPageCache):
(WebKit::FrameLoaderClientWinCE::didRestoreFromPageCache):

  • WebCoreSupport/FrameLoaderClientWinCE.h:

WebKit/wx:

  • Add client functions to notify that a frame has been added or removed from the page cache.
  • WebKitSupport/FrameLoaderClientWx.cpp:

(WebCore::FrameLoaderClientWx::didSaveToPageCache):
(WebCore::FrameLoaderClientWx::didRestoreFromPageCache):

  • WebKitSupport/FrameLoaderClientWx.h:

WebKit2:

  • UIProcess/API/C/WKFrame.cpp:

(WKFrameCopyChildFrames):

  • UIProcess/API/C/WKFrame.h:

Add API to get the child frames of a frame.

  • UIProcess/WebFrameProxy.cpp:

(WebKit::WebFrameProxy::WebFrameProxy):
(WebKit::WebFrameProxy::disconnect):
(WebKit::WebFrameProxy::appendChild):
(WebKit::WebFrameProxy::removeChild):
(WebKit::WebFrameProxy::isDescendantOf):
(WebKit::WebFrameProxy::dumpFrameTree):
(WebKit::WebFrameProxy::didRemoveFromHierarchy):
(WebKit::WebFrameProxy::childFrames):

  • UIProcess/WebFrameProxy.h:

(WebKit::WebFrameProxy::parentFrame): Add getter.
(WebKit::WebFrameProxy::nextSibling): Add getter.
(WebKit::WebFrameProxy::previousSibling): Add getter.
(WebKit::WebFrameProxy::firstChild): Add getter.
(WebKit::WebFrameProxy::lastChild): Add getter.
Make frames keep track of their subframes.

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::didCreateSubframe):
Use the now passed parentID to insert the frame into the hierarchy

(WebKit::WebPageProxy::didSaveFrameToPageCache):
When a frame is added to the page cache, remove it from the hierarchy.

(WebKit::WebPageProxy::didRestoreFrameFromPageCache):
When a frame is restored from the page cache, add it back to the hierarchy.

(WebKit::WebPageProxy::didRemoveFrameFromHierarchy):
Detached frames should also be removed from the hierarchy.

  • UIProcess/WebPageProxy.h:

Add new declarations for didSaveFrameToPageCache and didRestoreFrameFromPageCache.

  • UIProcess/WebPageProxy.messages.in:

Add messages for DidSaveFrameToPageCache and DidRestoreFrameFromPageCache.

  • WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:

(WebKit::WebFrameLoaderClient::savePlatformDataToCachedFrame):
(WebKit::WebFrameLoaderClient::transitionToCommittedFromCachedFrame):
Remove notImplemented, there is no work we need to do here.

(WebKit::WebFrameLoaderClient::didSaveToPageCache):
(WebKit::WebFrameLoaderClient::didRestoreFromPageCache):
Send message to UIProcess to indicate this happened.

  • WebProcess/WebCoreSupport/WebFrameLoaderClient.h:
  • WebProcess/WebPage/WebFrame.cpp:

(WebKit::WebFrame::createSubframe):
Send parent frameID in addition to our own in the message to UIProcess.

11:57 Changeset [74570] by mdelaney@apple.com

2010-12-22 Matthew Delaney <mdelaney@apple.com>

Reviewed by Simon Fraser.

Add setNeedsStyleRecalc for getting canvas contexts that use IOSurfaces
This patch also changes USE_IOSURFACE to instead follow USE(IOSURFACE)
https://bugs.webkit.org/show_bug.cgi?id=51279

  • html/HTMLCanvasElement.cpp: For IOSurface, use setNeedsStyleRecalc
  • html/canvas/CanvasRenderingContext2D.cpp: Use new flag
  • platform/graphics/ImageBuffer.h: Use new flag
  • platform/graphics/cg/ImageBufferCG.cpp: Use new flag

2010-12-22 Matthew Delaney <mdelaney@apple.com>

Reviewed by Simon Fraser.

Add setNeedsStyleRecalc for getting canvas contexts that use IOSurfaces
https://bugs.webkit.org/show_bug.cgi?id=51279

  • platform/mac-snowleopard/platform/mac/compositing: Added.
  • platform/mac-snowleopard/platform/mac/compositing/canvas: Added.
  • platform/mac-snowleopard/platform/mac/compositing/canvas/accelerated-canvas-compositing-expected.txt: Added.
  • platform/mac/compositing/canvas: Added.
  • platform/mac/compositing/canvas/accelerated-canvas-compositing-expected.txt: Added.
  • platform/mac/compositing/canvas/accelerated-canvas-compositing.html: Added.
11:33 Changeset [74569] by andersca@apple.com

2010-12-23 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

New popup windows open disproportionately big
https://bugs.webkit.org/show_bug.cgi?id=51547
<rdar://problem/8637029>

  • UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::createNewPage): Pass the correct size back to the UI process.
  • WebProcess/WebCoreSupport/WebChromeClient.cpp: (WebKit::WebChromeClient::pageRect): Implement this and return the size of the page.
11:16 Changeset [74568] by enne@google.com

2010-12-20 Adrienne Walker <enne@google.com>

Reviewed by Kenneth Russell.

[chromium] Tile root layer of the compositor.
https://bugs.webkit.org/show_bug.cgi?id=49947

Refactor root layer update and drawing from LayerRendererChromium into
LayerTilerChromium. The root layer is now drawn as multiple tiles
rather than as one single large texture. Scrollbars are now drawn
separately rather than as part of the root layer.

Test: LayoutTests/compositing/

  • WebCore.gypi:
  • page/FrameView.cpp: (WebCore::FrameView::repaintContentRectangle):
  • platform/ScrollView.cpp: (WebCore::ScrollView::ScrollView): (WebCore::ScrollView::setClipsRepaints):
  • platform/ScrollView.h: (WebCore::ScrollView::clipsRepaints):
  • platform/graphics/chromium/LayerChromium.h:
  • platform/graphics/chromium/LayerRendererChromium.cpp: (WebCore::LayerRendererChromium::LayerRendererChromium): (WebCore::LayerRendererChromium::~LayerRendererChromium): (WebCore::LayerRendererChromium::useShader): (WebCore::LayerRendererChromium::verticalScrollbarRect): (WebCore::LayerRendererChromium::horizontalScrollbarRect): (WebCore::LayerRendererChromium::invalidateRootLayerRect): (WebCore::LayerRendererChromium::updateAndDrawRootLayer): (WebCore::LayerRendererChromium::drawLayers): (WebCore::LayerRendererChromium::setRootLayer): (WebCore::LayerRendererChromium::initializeSharedObjects): (WebCore::LayerRendererChromium::cleanupSharedObjects):
  • platform/graphics/chromium/LayerRendererChromium.h:
  • platform/graphics/chromium/LayerTilerChromium.cpp: Added. (WebCore::LayerTilerChromium::create): (WebCore::LayerTilerChromium::LayerTilerChromium): (WebCore::LayerTilerChromium::~LayerTilerChromium): (WebCore::LayerTilerChromium::layerRendererContext): (WebCore::LayerTilerChromium::setTileSize): (WebCore::LayerTilerChromium::reset): (WebCore::LayerTilerChromium::createTile): (WebCore::LayerTilerChromium::invalidateTiles): (WebCore::LayerTilerChromium::contentRectToTileIndices): (WebCore::LayerTilerChromium::contentRectToLayerRect): (WebCore::LayerTilerChromium::layerRectToContentRect): (WebCore::LayerTilerChromium::tileIndex): (WebCore::LayerTilerChromium::tileContentRect): (WebCore::LayerTilerChromium::tileLayerRect): (WebCore::LayerTilerChromium::invalidateRect): (WebCore::LayerTilerChromium::invalidateEntireLayer): (WebCore::LayerTilerChromium::update): (WebCore::LayerTilerChromium::setLayerPosition): (WebCore::LayerTilerChromium::draw): (WebCore::LayerTilerChromium::resizeLayer): (WebCore::LayerTilerChromium::growLayerToContain): (WebCore::LayerTilerChromium::Tile::~Tile): (WebCore::LayerTilerChromium::Tile::releaseTextureId):
  • platform/graphics/chromium/LayerTilerChromium.h: Added. (WebCore::LayerTilerChromium::Tile::Tile): (WebCore::LayerTilerChromium::Tile::textureId): (WebCore::LayerTilerChromium::Tile::dirty): (WebCore::LayerTilerChromium::Tile::clearDirty): (WebCore::LayerTilerChromium::layerRenderer):

2010-12-20 Adrienne Walker <enne@google.com>

Reviewed by Kenneth Russell.

[chromium] Tile root layer of the compositor.
https://bugs.webkit.org/show_bug.cgi?id=49947

Refactor root layer logic out of WebViewImpl and into
LayerTilerChromium. The painting is now done through an interface
rather than directly in WebViewImpl.

  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::scrollRootLayerRect): (WebKit::WebViewImpl::invalidateRootLayerRect): (WebKit::WebViewImpl::setIsAcceleratedCompositingActive): (WebKit::WebViewImplTilePaintInterface::WebViewImplTilePaintInterface): (WebKit::WebViewImplTilePaintInterface::paint): (WebKit::WebViewImplScrollbarPaintInterface::WebViewImplScrollbarPaintInterface): (WebKit::WebViewImplScrollbarPaintInterface::paint): (WebKit::WebViewImpl::doComposite):
  • src/WebViewImpl.h:
11:06 Changeset [74567] by kbr@google.com

2010-12-23 Kenneth Russell <kbr@google.com>

Unreviewed Chromium expectations update. Fix incorrect syntax for
fast/css/box-shadow.html suppression and state that
fast/events/touch/touch-target.html may timeout.

  • platform/chromium/test_expectations.txt:
10:59 Changeset [74566] by rniwa@webkit.org

2010-12-22 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Eric Seidel.

Editor.h doesn't need to include SelectionController.h
https://bugs.webkit.org/show_bug.cgi?id=51441

Renamed SelectionController::EDirection to SelectionDirection.

  • src/WebFrameImpl.cpp: (WebKit::WebFrameImpl::executeCommand):

2010-12-22 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Eric Seidel.

Editor.h doesn't need to include SelectionController.h
https://bugs.webkit.org/show_bug.cgi?id=51441

Renamed SelectionController::EDirection to SelectionDirection.

  • WebCoreSupport/EditorClientQt.cpp: (WebCore::EditorClientQt::handleKeyboardEvent):

2010-12-22 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Eric Seidel.

Editor.h doesn't need to include SelectionController.h
https://bugs.webkit.org/show_bug.cgi?id=51441

Renamed SelectionController::EDirection to SelectionDirection.

  • WebView/WebFrame.mm: (-[WebFrame _rangeByAlteringCurrentSelection:SelectionController::direction:granularity:]): Takes SelectionDirection instead of SelectionController::EDirection.
  • WebView/WebFrameInternal.h:
  • WebView/WebTextCompletionController.mm: (-[WebTextCompletionController doCompletion]): Calls _rangeByAlteringCurrentSelection:SelectionController.

2010-12-22 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Eric Seidel.

Editor.h doesn't need to include SelectionController.h
https://bugs.webkit.org/show_bug.cgi?id=51441

Renamed SelectionController::EDirection to SelectionDirection.

  • WebCoreSupport/EditorClientHaiku.cpp: (WebCore::EditorClientHaiku::handleKeyboardEvent):

2010-12-22 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Eric Seidel.

Editor.h doesn't need to include SelectionController.h
https://bugs.webkit.org/show_bug.cgi?id=51441

Renamed SelectionController::EDirection to SelectionDirection.

  • WebCoreSupport/EditorClientWinCE.cpp: (WebKit::EditorClientWinCE::handleEditingKeyboardEvent):

2010-12-22 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Eric Seidel.

Editor.h doesn't need to include SelectionController.h
https://bugs.webkit.org/show_bug.cgi?id=51441

Renamed SelectionController::EDirection to SelectionDirection.

  • WebCoreSupport/EditorClientEfl.cpp: (WebCore::EditorClientEfl::handleEditingKeyboardEvent):

2010-12-22 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Eric Seidel.

Editor.h doesn't need to include SelectionController.h
https://bugs.webkit.org/show_bug.cgi?id=51441

Removed #include <Selection.h> from Editor.h.
Also extracted WritingDirection.h from EditingStyle.h and renamed SelectionController::EDirection
to SelectionDirection and moved its declaration from SelectionController.h to VisibleSelection.h

  • WebCore.exp.in: Signature changes.
  • WebCore.xcodeproj/project.pbxproj: Added WritingDirection.h
  • accessibility/AccessibilityRenderObject.cpp: (WebCore::AccessibilityRenderObject::visiblePositionRangeForLine): Change due to SelectionDirection.
  • editing/EditingStyle.h: Removed WritingDirection and included WritingDirection.h
  • editing/Editor.cpp: (WebCore::Editor::deleteWithDirection): Change due to SelectionDirection. (WebCore::Editor::markAllMisspellingsAndBadGrammarInRanges): Ditto. (WebCore::Editor::markMisspellingsAfterTypingToWord): Ditto.
  • editing/Editor.h: Removed SelectionController.h; added DocumentMarker.h, Timer.h, VisibleSelection.h and WritingDirection.h; forward declared CSSMutableStyleDeclaration.
  • editing/EditorCommand.cpp: (WebCore::executeDeleteBackward): Change due to SelectionDirection. (WebCore::executeDeleteBackwardByDecomposingPreviousCharacter): Ditto. (WebCore::executeDeleteForward): Ditto. (WebCore::executeDeleteToBeginningOfLine): Ditto. (WebCore::executeDeleteToBeginningOfParagraph): Ditto. (WebCore::executeDeleteToEndOfLine): Ditto. (WebCore::executeDeleteToEndOfParagraph): Ditto. (WebCore::executeDeleteWordBackward): Ditto. (WebCore::executeDeleteWordForward): Ditto. (WebCore::executeForwardDelete): Ditto. (WebCore::executeMoveBackward): Ditto. (WebCore::executeMoveBackwardAndModifySelection): Ditto. (WebCore::executeMoveDown): Ditto. (WebCore::executeMoveDownAndModifySelection): Ditto. (WebCore::executeMoveForward): Ditto. (WebCore::executeMoveForwardAndModifySelection): Ditto. (WebCore::executeMoveLeft): Ditto. (WebCore::executeMoveLeftAndModifySelection): Ditto. (WebCore::executeMoveRight): Ditto. (WebCore::executeMoveRightAndModifySelection): Ditto. (WebCore::executeMoveToBeginningOfDocument): Ditto. (WebCore::executeMoveToBeginningOfDocumentAndModifySelection): Ditto. (WebCore::executeMoveToBeginningOfLine): Ditto. (WebCore::executeMoveToBeginningOfLineAndModifySelection): Ditto. (WebCore::executeMoveToBeginningOfParagraph): Ditto. (WebCore::executeMoveToBeginningOfParagraphAndModifySelection): Ditto. (WebCore::executeMoveToBeginningOfSentence): Ditto. (WebCore::executeMoveToBeginningOfSentenceAndModifySelection): Ditto. (WebCore::executeMoveToEndOfDocument): Ditto. (WebCore::executeMoveToEndOfDocumentAndModifySelection): Ditto. (WebCore::executeMoveToEndOfSentence): Ditto. (WebCore::executeMoveToEndOfSentenceAndModifySelection): Ditto. (WebCore::executeMoveToEndOfLine): Ditto. (WebCore::executeMoveToEndOfLineAndModifySelection): Ditto. (WebCore::executeMoveToEndOfParagraph): Ditto. (WebCore::executeMoveToEndOfParagraphAndModifySelection): Ditto. (WebCore::executeMoveParagraphBackwardAndModifySelection): Ditto. (WebCore::executeMoveParagraphForwardAndModifySelection): Ditto. (WebCore::executeMoveUp): Ditto. (WebCore::executeMoveUpAndModifySelection): Ditto. (WebCore::executeMoveWordBackward): Ditto. (WebCore::executeMoveWordBackwardAndModifySelection): Ditto. (WebCore::executeMoveWordForward): Ditto. (WebCore::executeMoveWordForwardAndModifySelection): Ditto. (WebCore::executeMoveWordLeft): Ditto. (WebCore::executeMoveWordLeftAndModifySelection): Ditto. (WebCore::executeMoveWordRight): Ditto. (WebCore::executeMoveWordRightAndModifySelection): Ditto. (WebCore::executeMoveToLeftEndOfLine): Ditto. (WebCore::executeMoveToLeftEndOfLineAndModifySelection): Ditto. (WebCore::executeMoveToRightEndOfLine): Ditto. (WebCore::executeMoveToRightEndOfLineAndModifySelection): Ditto.
  • editing/SelectionController.cpp: (WebCore::SelectionController::willBeModified): Takes SelectionDirection instead of EDirection. (WebCore::SelectionController::modify): Ditto.
  • editing/SelectionController.h: Change due to SelectionDirection.
  • editing/TypingCommand.cpp: (WebCore::TypingCommand::deleteKeyPressed): Ditto. (WebCore::TypingCommand::forwardDeleteKeyPressed): Ditto.
  • editing/VisibleSelection.h: Added SelectionDirection, renamed and moved from EDirection in SelectionController.h
  • editing/WritingDirection.h: Added.
  • page/DOMSelection.cpp: (WebCore::DOMSelection::modify): Change due to SelectionDirection. (WebCore::DOMSelection::deleteFromDocument): Ditto.
  • page/EventHandler.cpp: (WebCore::EventHandler::handleKeyboardSelectionMovement): Ditto.
  • page/Frame.h: Includes SelectionController.h.
10:17 Changeset [74565] by lforschler@apple.com

2010-12-23 Lucas Forschler <lforschler@apple.com>

Reviewed by Darin Adler.

Bug 51537 - iexploder needs cleanup

  • iExploder/htdocs: Removed.
  • iExploder/htdocs/config.rb: Removed.
  • iExploder/htdocs/cssproperties.in: Removed.
  • iExploder/htdocs/cssvalues.in: Removed.
  • iExploder/htdocs/htmlattrs.in: Removed.
  • iExploder/htdocs/htmltags.in: Removed.
  • iExploder/htdocs/htmlvalues.in: Removed.
  • iExploder/htdocs/iexploder.cgi: Removed.
  • iExploder/htdocs/iexploder.rb: Removed.
  • iExploder/htdocs/index.html: Removed.
  • iExploder/htdocs/webserver.rb: Removed.
  • iExploder/iexploder-1.7.2/output: Removed.
  • iExploder/tools: Removed.
  • iExploder/tools/lasthit.rb: Removed.
  • iExploder/tools/osx_last_crash.rb: Removed.
  • iExploder/tools/showtest.rb: Removed.
10:16 Changeset [74564] by andersca@apple.com

2010-12-23 Anders Carlsson <andersca@apple.com>

Reviewed by John Sullivan.

Nothing happens clicking "launch live player" on live.cnn.com
https://bugs.webkit.org/show_bug.cgi?id=51540
<rdar://problem/8791086>

Implement NPN_PushPopupsEnabledState/NPN_PopPopupsEnabledState.

  • WebProcess/Plugins/Netscape/NetscapeBrowserFuncs.cpp: (WebKit::NPN_PushPopupsEnabledState): Call NetscapePlugin::pushPopupsEnabledState.

(WebKit::NPN_PopPopupsEnabledState):
Call NetscapePlugin::popPopupsEnabledState.

  • WebProcess/Plugins/Netscape/NetscapePlugin.cpp: (WebKit::NetscapePlugin::pushPopupsEnabledState): Append the boolean to the vector.

(WebKit::NetscapePlugin::popPopupsEnabledState):
Remove the last boolean from the vector.

(WebKit::NetscapePlugin::allowPopups):
Return the last boolean from the vector.

  • WebProcess/Plugins/Netscape/NetscapePlugin.h:
  • WebProcess/Plugins/PluginView.cpp: (WebKit::PluginView::performJavaScriptURLRequest): Update the "allowPopupsFromPlugin" state.
09:45 Changeset [74563] by abarth@webkit.org

2010-12-23 Adam Barth <abarth@webkit.org>

Reviewed by Darin Adler.

Move V8 to WebCore's implementation of showModalDialog
https://bugs.webkit.org/show_bug.cgi?id=51527

This code is basically verbatim translation from the JavaScriptCore
bindings. The only intentional difference is in the world selection
for the dialog's frame. I suspect JavaScriptCore's bindings have a
subtle bug there.

In this patch, I also remove a bunch of now-unneeded code in the
generic bindings.

  • bindings/generic/BindingDOMWindow.h:
  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::DialogHandler::dialogCreated): (WebCore::setUpDialog):
  • bindings/v8/custom/V8DOMWindowCustom.cpp: (WebCore::DialogHandler::DialogHandler): (WebCore::DialogHandler::dialogCreated): (WebCore::DialogHandler::returnValue): (WebCore::setUpDialog): (WebCore::V8DOMWindow::showModalDialogCallback): (WebCore::V8DOMWindow::openCallback):
  • bindings/v8/specialization/V8BindingDOMWindow.h:
09:26 Changeset [74562] by apavlov@chromium.org

2010-12-23 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Joseph Pecoraro.

Web Inspector: Can't backspace from CSS value to CSS key field
https://bugs.webkit.org/show_bug.cgi?id=51476

  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylePropertyTreeElement.prototype.): (WebInspector.StylePropertyTreeElement.prototype):
08:56 Changeset [74561] by commit-queue@webkit.org

2010-12-23 W. James MacLean <wjmaclean@chromium.org>

Reviewed by Kenneth Russell.

[chromium] Add asserts to test for contiguous-pixel Skia bitmaps.
https://bugs.webkit.org/show_bug.cgi?id=51186

No new tests. Behaviour not changed, but need the asserts to detect when assumptions violated.

  • platform/graphics/chromium/ImageLayerChromium.cpp: (WebCore::ImageLayerChromium::updateContentsIfDirty):
  • platform/graphics/skia/GraphicsContext3DSkia.cpp: (WebCore::GraphicsContext3D::getImageData):

2010-12-23 W. James MacLean <wjmaclean@chromium.org>

Reviewed by Kenneth Russell.

[chromium] Add asserts to test for contiguous-pixel Skia bitmaps.
https://bugs.webkit.org/show_bug.cgi?id=51186

Add asserts to detect if assumptions (about contiguous pixels in Skia bitmaps) are violated.

  • src/GraphicsContext3DChromium.cpp: (WebCore::GraphicsContext3DInternal::paintRenderingResultsToCanvas):
08:53 Changeset [74560] by jberlin@webkit.org

Add Windows-XP-specific (failing) expected results for
fast/profiler/throw-exception-from-eval-expected.html in order to get the Windows XP Debug
test bot green again. Unreviewed.
https://bugs.webkit.org/show_bug.cgi?id=49801

  • platform/win-xp/fast/profiler: Added.
  • platform/win-xp/fast/profiler/throw-exception-from-eval-expected.txt: Added.
08:39 Changeset [74559] by commit-queue@webkit.org

2010-12-23 Lucas De Marchi <lucas.demarchi@profusion.mobi>

Reviewed by Kenneth Rohde Christiansen.

[EFL] Make 'single' the default backing store in EWebLauncher
https://bugs.webkit.org/show_bug.cgi?id=51534

'single' is the simplest backing store and the one intended to be the
default in the test browser. By mistake the tiled backing store was
left as default in r72617.

  • EWebLauncher/main.c: (browserCreate): ditto.
08:23 Changeset [74558] by commit-queue@webkit.org

2010-12-23 Dawit Alemayehu <adawit@kde.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Add a 'hasSelection' function to QWebView and QWebPage.
https://bugs.webkit.org/show_bug.cgi?id=48722

This is a convenience API that optimizes the case where checking for
the presence of selected content and accessing the selected content
are two separate actions in the client. See comment #12 in the above
bug report link for details.

  • Api/qwebpage.cpp: (QWebPage::hasSelection):
  • Api/qwebpage.h:
  • Api/qwebview.cpp: (QWebView::hasSelection):
  • Api/qwebview.h:
  • tests/qwebpage/tst_qwebpage.cpp: (tst_QWebPage::textSelection):
06:45 Changeset [74557] by jeremy@chromium.org

2010-12-23 Jeremy Moskovich <jeremy@chromium.org>

Unreviewed new Chromium baselines and test expectations update.

  • platform/chromium-linux/fast/layers/video-layer-expected.checksum:
  • platform/chromium-linux/fast/layers/video-layer-expected.png:
  • platform/chromium-mac/fast/layers/video-layer-expected.checksum:
  • platform/chromium-mac/fast/layers/video-layer-expected.png:
  • platform/chromium-mac/fast/layers/video-layer-expected.txt:
  • platform/chromium-win/fast/css/getComputedStyle/computed-style-expected.txt:
  • platform/chromium-win/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
  • platform/chromium-win/fast/layers/video-layer-expected.checksum:
  • platform/chromium-win/fast/layers/video-layer-expected.png:
  • platform/chromium-win/fast/layers/video-layer-expected.txt:
  • platform/chromium-win/svg/css/getComputedStyle-basic-expected.txt:
  • platform/chromium/test_expectations.txt:
06:41 Changeset [74556] by commit-queue@webkit.org

2010-12-23 Lucas De Marchi <lucas.demarchi@profusion.mobi>

Reviewed by Kenneth Rohde Christiansen.

[EFL] Fix ContextMenu removal
https://bugs.webkit.org/show_bug.cgi?id=51530

Fix ContextMenu removal

In r74207 the support for ContextMenu was removed from EFL while the
code is being ported to the new CROSS_PLATFORM_MENUS due to build
breakage. However there were some missing removals in the shutdown
logic that introduced a bug, leading to a segmentation fault when
ContexController was destroyed.

The best way to effectively disable the ContextMenu is to conditionally
compile the meaningful stuff by surrounding with "#if
ENABLE(CONTEXT_MENUS)" and then disabling it in CMake.

  • cmake/OptionsEfl.cmake: Disable ContextMenu and don't pretend we implement CROSS_PLATFORM_MENUS yet.

2010-12-23 Lucas De Marchi <lucas.demarchi@profusion.mobi>

Reviewed by Kenneth Rohde Christiansen.

[EFL] Fix ContextMenu removal
https://bugs.webkit.org/show_bug.cgi?id=51530

Fix ContextMenu removal

In r74207 the support for ContextMenu was removed from EFL while the
code is being ported to the new CROSS_PLATFORM_MENUS due to build
breakage. However there were some missing removals in the shutdown
logic that introduced a bug, leading to a segmentation fault when
ContexController was destroyed.

The best way to effectively disable the ContextMenu is to conditionally
compile the meaningful stuff by surrounding with "#if
ENABLE(CONTEXT_MENUS)" and then disabling it in CMake.

  • ewk/ewk_contextmenu.cpp: surround need parts with #if ENABLE(CONTEXT_MENUS). (ewk_context_menu_destroy): ditto. (ewk_context_menu_item_select): ditto.
  • ewk/ewk_private.h: ditto.
  • ewk/ewk_view.cpp: ditto.
06:24 Changeset [74555] by commit-queue@webkit.org

2010-12-23 Lucas De Marchi <lucas.demarchi@profusion.mobi>

Reviewed by Kenneth Rohde Christiansen.

Fix check by undefined macro
https://bugs.webkit.org/show_bug.cgi?id=51531

Fix check by undefined macro

When building EFL port with libsoup, BUILDING_GTK is not defined,
leading to warnings during compile. We actually have to use an #ifdef
instead of an #if.

No change in functionality so no new tests.

  • platform/network/soup/cache/webkit/soup-cache.h:
06:08 Changeset [74554] by commit-queue@webkit.org

2010-12-23 Lucas De Marchi <lucas.demarchi@profusion.mobi>

Reviewed by Kenneth Rohde Christiansen.

[EFL] Clean warnings in Tiled Backing Store
https://bugs.webkit.org/show_bug.cgi?id=51529

Tiled backing store in EFL was giving the following warnings that are
gone now:

  • Static function defined but not used;
  • Comparison between signed and unsigned integer expressions;
  • ewk/ewk_tiled_backing_store.c: (_ewk_tiled_backing_store_item_fill): (_ewk_tiled_backing_store_smart_calculate):
  • ewk/ewk_tiled_matrix.c: (_ewk_tile_matrix_slicer_setup):
05:49 Changeset [74553] by commit-queue@webkit.org

2010-12-23 Jonathan Dixon <joth@chromium.org>

Reviewed by Steve Block.

TouchEvents does not support multi-touch on a page with multiple touch targets
https://bugs.webkit.org/show_bug.cgi?id=51258

  • fast/events/touch/multi-touch-grouped-targets-expected.txt: Added.
  • fast/events/touch/multi-touch-grouped-targets.html: Added.
  • fast/events/touch/script-tests/multi-touch-grouped-targets.js: Added. (declareTouchHandler.return.shouldBe): (declareTouchHandler): (touchEndHandler): Added new test that ensures both test targets receive events when hit by multiple touch points from the same platform event.
  • fast/events/touch/script-tests/touch-target-limited.js: (declareTouchStart.return.touchStartHandler): (declareTouchStart): (declareTouchMove.return.touchMoveHandler): (declareTouchMove):
  • fast/events/touch/script-tests/touch-target.js: (declareTouchStart.return.touchStartHandler): (declareTouchStart): (declareTouchMove.return.touchMoveHandler): (declareTouchMove):
  • fast/events/touch/touch-target-expected.txt:
  • fast/events/touch/touch-target-limited-expected.txt: Updated & rebaselined existing tests to reflect new expectations.

2010-12-23 Jonathan Dixon <joth@chromium.org>

Reviewed by Steve Block.

TouchEvents does not support multi-touch on a page with multiple touch targets
https://bugs.webkit.org/show_bug.cgi?id=51258

Test: fast/events/touch/multi-touch-grouped-targets.html

  • page/EventHandler.cpp: (WebCore::eventNameForTouchPointState): (WebCore::EventHandler::handleTouchEvent): Redesigned event handler to ensure all event targets involved in a given multitouch event get the appropriate event(s) fired.
  • platform/PlatformTouchPoint.h: Added end-stop marker for State enum.
05:23 Changeset [74552] by commit-queue@webkit.org

2010-12-23 Lucas De Marchi <lucas.demarchi@profusion.mobi>

Reviewed by Kenneth Rohde Christiansen.

[EFL] Subject: [webkit 4/4] [EFL] Remove non-sense workaround
https://bugs.webkit.org/show_bug.cgi?id=51533

There's no point in concatenating env("HOME") with "blah", so remove
option from EWebLauncher.

  • EWebLauncher/main.c: (main): ditto.
05:14 Changeset [74551] by philn@webkit.org

2010-12-23 Philippe Normand <pnormand@igalia.com>

Reviewed by Xan Lopez.

[GTK] media/audio-data-url.html fails since r75504
https://bugs.webkit.org/show_bug.cgi?id=51525

Make the GStreamer media-player advertize 3gpp mime-types as
supported if the 3gp caps are in the typefind factories.

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::mimeTypeCache):
05:09 Changeset [74550] by jeremy@chromium.org

2010-12-23 Jeremy Moskovich <jeremy@chromium.org>

Unreviewed test expectations update.

Fix duplicate expectation warning for media/video-controls-rendering.html introduced in r74535.

  • platform/chromium/test_expectations.txt:
04:56 Changeset [74549] by tkent@chromium.org

Refactor HTMLInputElement: Move a part of HTMLInputElement::defaultEventHandler()
to InputTypes.
https://bugs.webkit.org/show_bug.cgi?id=51093

Reviewed by Dimitri Glazkov.

Move BeforeTextInserted and Wheel event handling, event forwarding to
renderers, and the isindex form submission quirk code to InputTypes.
Also, change the return types of InputType event handlers from 'bool'
to 'void'. We use Event::defaultHandled() to decide whether event
handling should be stopped.

No new tests because this is just a refactoring.

  • html/BaseButtonInputType.cpp: bool->void changes

(WebCore::BaseButtonInputType::handleKeydownEvent):
(WebCore::BaseButtonInputType::handleKeypressEvent):
(WebCore::BaseButtonInputType::handleKeyupEvent):

  • html/BaseButtonInputType.h: ditto.
  • html/BaseCheckableInputType.cpp: ditto.

(WebCore::BaseCheckableInputType::handleKeydownEvent):
(WebCore::BaseCheckableInputType::handleKeypressEvent):

  • html/BaseCheckableInputType.h: ditto.
  • html/BaseDateAndTimeInputType.cpp:

(WebCore::BaseDateAndTimeInputType::handleKeydownEvent): ditto.
(WebCore::BaseDateAndTimeInputType::handleWheelEvent): Just calls handleWheelEventForSpinButton().

  • html/BaseDateAndTimeInputType.h: bool->void change and handleWhellEvent() addition.
  • html/CheckboxInputType.cpp: bool->void change

(WebCore::CheckboxInputType::handleKeyupEvent):

  • html/CheckboxInputType.h: ditto.
  • html/FileInputType.cpp: ditto.

(WebCore::FileInputType::handleDOMActivateEvent):

  • html/FileInputType.h: ditto.
  • html/HTMLInputElement.cpp:

(WebCore::HTMLInputElement::defaultEventHandler):

  • Change return-value handling of InputType event handlers
  • Move out beforeTextInserted/wheel event handling and isindex form quirk.
  • html/HTMLInputElement.h:

(WebCore::HTMLInputElement::handleBeforeTextInsertedEvent):

Added so that InputType can call InputElement::handleBeforeTextInsertedEvent().

  • html/ImageInputType.cpp: bool->void change

(WebCore::ImageInputType::handleDOMActivateEvent):

  • html/ImageInputType.h: ditto.
  • html/InputType.cpp: ditto.

(WebCore::InputType::handleClickEvent):
(WebCore::InputType::handleDOMActivateEvent):
(WebCore::InputType::handleKeydownEvent):
(WebCore::InputType::handleKeypressEvent):
(WebCore::InputType::handleKeyupEvent):
(WebCore::InputType::handleBeforeTextInsertedEvent):
(WebCore::InputType::handleWheelEvent): Added default implementation.
(WebCore::InputType::forwardEvent): ditto.
(WebCore::InputType::formForSubmission): ditto.

  • html/InputType.h:
  • html/IsIndexInputType.cpp:

(WebCore::IsIndexInputType::formForSubmission): Moved from HTMLInputElement.cpp.

  • html/IsIndexInputType.h:
  • html/NumberInputType.cpp:

(WebCore::isNumberCharacter): Moved from HTMLInputElement.cpp.
(WebCore::NumberInputType::handleKeydownEvent): bool->void change.
(WebCore::NumberInputType::handleBeforeTextInsertedEvent): Moved from HTMInputElement.cpp.
(WebCore::NumberInputType::handleWheelEvent): Added. Just calls handleWheelEventForSpinButton().

  • html/NumberInputType.h:
  • html/RadioInputType.cpp: bool->void changes.

(WebCore::RadioInputType::handleClickEvent):
(WebCore::RadioInputType::handleKeydownEvent):
(WebCore::RadioInputType::handleKeyupEvent):

  • html/RadioInputType.h:
  • html/RangeInputType.cpp:

(WebCore::RangeInputType::handleKeydownEvent): ditto.
(WebCore::RangeInputType::forwardEvent): Moved from HTMLInputElement.cpp.

  • html/RangeInputType.h:
  • html/ResetInputType.cpp: bool->void change.

(WebCore::ResetInputType::handleDOMActivateEvent):

  • html/ResetInputType.h: ditto.
  • html/SubmitInputType.cpp: ditto.

(WebCore::SubmitInputType::handleDOMActivateEvent):

  • html/SubmitInputType.h: ditto.
  • html/TextFieldInputType.cpp:

(WebCore::TextFieldInputType::handleKeydownEvent): bool->void change.
(WebCore::TextFieldInputType::handleKeydownEventForSpinButton): ditto.
(WebCore::TextFieldInputType::handleWheelEventForSpinButton): Moved from HTMLInputElement.cpp.
(WebCore::TextFieldInputType::forwardEvent): ditto.

  • html/TextFieldInputType.h:
03:12 Changeset [74548] by apavlov@chromium.org

2010-12-23 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Joseph Pecoraro.

Web Inspector: After scrolling new property value and cancelling, removed property remains active in page style
https://bugs.webkit.org/show_bug.cgi?id=51390

  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylePropertyTreeElement.prototype):
02:38 Changeset [74547] by ossy@webkit.org

Unreviewed. Update Qt specific expected file after r74539.

  • platform/qt/fast/dom/Window/window-properties-expected.txt:
02:20 Changeset [74546] by commit-queue@webkit.org

2010-12-23 Benjamin Kalman <kalman@chromium.org>

Reviewed by Eric Seidel.

QT linux has RTL line movement bug that causes editing/selection/extend-to-line-boundary.html to fail
https://bugs.webkit.org/show_bug.cgi?id=51514

Add different baseline to editing/selection/extend-to-line-boundary.html for qt-linux that
takes into account the RTL bug.

  • platform/qt-linux/editing/selection/extend-to-line-boundary-expected.txt: Added.
01:54 Changeset [74545] by antti@apple.com

Not reviewed.

Another attempt.

  • http/tests/local/slow-css-pass.cgi:
01:20 Changeset [74544] by loislo@chromium.org

2010-12-23 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Eric Seidel.

Web Inspector: chromium: inspector's tests are quite slow especially at windows(Debug).

Almost all inspector's tests become flaky on chromium Win(dbg) after switching to DRT.
I just discovered that we have two additional GC runs per each WebViewHost.
Test_shell had no such calls.
These gc runs eat 15% of cpu time.

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

  • DumpRenderTree/chromium/WebViewHost.cpp: (WebViewHost::~WebViewHost):
01:18 Changeset [74543] by antti@apple.com

Not reviewed.

Flaky Test: http/tests/local/stylesheet-and-script-load-order-media-print.html

Try to fix flakiness.

  • http/tests/local/slow-css-pass.cgi:
00:17 Changeset [74542] by gyuyoung.kim@samsung.com

2010-12-23 Gyuyoung Kim <gyuyoung.kim@samsung.com>

Unreviewed build warning fix.

[EFL] Remove warning messages during the webkit efl compiling
https://bugs.webkit.org/show_bug.cgi?id=51520

Remove build warning messages.

  • ewk/ewk_view.cpp: (_ewk_view_smart_show): (_ewk_view_smart_hide):
  • ewk/ewk_view.h:
00:00 Changeset [74541] by commit-queue@webkit.org

2010-12-23 Noel Gordon <noel.gordon@gmail.com>

Reviewed by Adam Barth.

[chromium] Reduce canvas.toDataURL("image/png") run-time costs 300%
https://bugs.webkit.org/show_bug.cgi?id=51455

Chromium uses the libpng compression defaults. Good compression results,
but at much larger run-time cost compared to other browsers. Chrome is 2
times slower than Mozilla 3.6.12 and 3-4 times slower than Safari 5.0.3.

libpng has supported alternative compression methods for some time, they
better meet the dual goals of good compression and run-time cost for the
on-line case, and are well-documented. Here use Huffman encoding. It's
2-20% faster than other encodings (Z_RLE, Z_FILTERED) and the compressed
size is within 1% of the Safari 5.0.3 result independent of image size.

No change in behaviour, so no new tests.

  • platform/image-encoders/skia/PNGImageEncoder.cpp: (WebCore::PNGImageEncoder::encode): Apply Huffman image encoding.
Note: See TracTimeline for information about the timeline view.