Timeline



Dec 26, 2010:

11:27 PM Changeset in webkit [74678] by abarth@webkit.org
  • 2 edits in trunk

Fix bug number in previous commit.

11:24 PM Changeset in webkit [74677] by abarth@webkit.org
  • 7 edits
    1 move in trunk

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:
11:00 PM Changeset in webkit [74676] by abarth@webkit.org
  • 4 edits
    1 move in trunk

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.
10:11 PM Changeset in webkit [74675] by abarth@webkit.org
  • 2 edits in trunk

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

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

  • .gitattributes:
9:51 PM Changeset in webkit [74674] by abarth@webkit.org
  • 4 edits
    1 move
    1 add in trunk

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.
8:42 PM Changeset in webkit [74673] by morrita@google.com
  • 3 edits
    4 adds in trunk

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):
2:13 PM Changeset in webkit [74672] by commit-queue@webkit.org
  • 2 edits in trunk/JavaScriptCore

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.
2:00 PM Changeset in webkit [74671] by Patrick Gansterer
  • 2 edits in trunk/WebCore

2010-12-26 Patrick Gansterer <Patrick Gansterer>

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 PM Changeset in webkit [74670] by levin@chromium.org
  • 3 edits in trunk/Tools

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 AM Changeset in webkit [74669] by andersca@apple.com
  • 13 edits in trunk/WebKit2

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 AM Changeset in webkit [74668] by andersca@apple.com
  • 10 edits in trunk/WebKit2

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:
8:57 AM Changeset in webkit [74667] by kbr@google.com
  • 2 edits in trunk/LayoutTests

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:
8:04 AM Changeset in webkit [74666] by inferno@chromium.org
  • 3 edits
    2 adds in trunk

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.
2:04 AM Changeset in webkit [74665] by Patrick Gansterer
  • 2 edits in trunk

2010-12-26 Patrick Gansterer <Patrick Gansterer>

Unreviewed WinCE buildfix after r74626.

  • cmake/OptionsWinCE.cmake:

Dec 25, 2010:

9:05 PM Changeset in webkit [74664] by kbr@google.com
  • 3 edits in trunk/LayoutTests

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
5:49 PM Changeset in webkit [74663] by commit-queue@webkit.org
  • 8 edits in trunk/WebCore

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):
4:38 PM Changeset in webkit [74662] by commit-queue@webkit.org
  • 10 edits in trunk/WebCore

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 PM Changeset in webkit [74661] by weinig@apple.com
  • 6 edits in trunk/WebKit2

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 PM Changeset in webkit [74660] by Patrick Gansterer
  • 2 edits in trunk/Tools

2010-12-25 Patrick Gansterer <Patrick Gansterer>

Unreviewed WinCE buildfix after r74334.

  • CMakeListsWinCE.txt:
1:52 AM Changeset in webkit [74659] by Philippe Normand
  • 2 edits in trunk/LayoutTests

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

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

  • platform/gtk/Skipped:

Dec 24, 2010:

10:00 PM Changeset in webkit [74658] by kbr@google.com
  • 2 edits in trunk/LayoutTests

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:
5:52 PM Changeset in webkit [74657] by mitz@apple.com
  • 6 edits in trunk

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):

2:33 PM Changeset in webkit [74656] by Philippe Normand
  • 2 edits in trunk/LayoutTests

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

Unreviewed, skip failing media test on GTK.

  • platform/gtk/Skipped: Skip media/controls-after-reload.html.
1:43 PM Changeset in webkit [74655] by Philippe Normand
  • 3 edits in trunk/Tools

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:
1:26 PM Changeset in webkit [74654] by Simon Fraser
  • 2 edits in trunk/WebKit2

2010-12-24 Simon Fraser <Simon Fraser>

Fix the Qt build after r74650.

  • WebProcess/WebCoreSupport/WebPlatformStrategies.cpp:
1:24 PM Changeset in webkit [74653] by andreip@google.com
  • 1 edit
    2 adds in trunk/LayoutTests

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.
1:13 PM Changeset in webkit [74652] by mitz@apple.com
  • 3 edits
    1 add in trunk/WebCore

Try to fix the Qt build after r74648.

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

(WebCore::CredentialStorage::getFromPersistentStorage):

12:57 PM Changeset in webkit [74651] by Philippe Normand
  • 9 edits in trunk/LayoutTests

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 PM Changeset in webkit [74650] by commit-queue@webkit.org
  • 2 edits in trunk/WebKit2

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 PM Changeset in webkit [74649] by Simon Fraser
  • 3 edits in trunk/WebCore

2010-12-24 Simon Fraser <Simon Fraser>

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 PM Changeset in webkit [74648] by mitz@apple.com
  • 6 edits in trunk

<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 AM Changeset in webkit [74647] by Darin Adler
  • 1 edit
    1 copy in trunk/LayoutTests

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 AM Changeset in webkit [74646] by yurys@chromium.org
  • 9 edits
    2 moves in trunk/WebCore

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 AM Changeset in webkit [74645] by jberlin@webkit.org
  • 1 edit
    2 adds in trunk/LayoutTests

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 AM Changeset in webkit [74644] by Darin Adler
  • 2 edits
    1 add in trunk

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 AM Changeset in webkit [74643] by commit-queue@webkit.org
  • 4 edits in trunk/WebCore

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 AM Changeset in webkit [74642] by commit-queue@webkit.org
  • 3 edits in trunk/WebCore

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 AM Changeset in webkit [74641] by jberlin@webkit.org
  • 2 edits in trunk/LayoutTests

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 AM Changeset in webkit [74640] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

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 AM Changeset in webkit [74639] by eric@webkit.org
  • 12 edits
    2 copies in trunk/Tools

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 AM Changeset in webkit [74638] by yurys@chromium.org
  • 3 edits in trunk/WebCore

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:
9:51 AM Changeset in webkit [74637] by yurys@chromium.org
  • 4 edits in trunk/WebCore

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):
9:31 AM Changeset in webkit [74636] by jschuh@chromium.org
  • 6 edits
    2 adds in trunk

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.
9:01 AM Changeset in webkit [74635] by yurys@chromium.org
  • 2 edits in trunk/WebCore

2010-12-23 Joseph Pecoraro <Joseph Pecoraro>

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:
8:39 AM Changeset in webkit [74634] by apavlov@chromium.org
  • 11 edits
    3 adds in trunk

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.
4:18 AM Changeset in webkit [74633] by commit-queue@webkit.org
  • 1 edit
    18 adds in trunk/LayoutTests

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.
4:12 AM Changeset in webkit [74632] by dpranke@chromium.org
  • 3 edits in trunk/Tools

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:
4:05 AM Changeset in webkit [74631] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

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.
4:02 AM Changeset in webkit [74630] by apavlov@chromium.org
  • 5 edits in trunk

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):
3:54 AM Changeset in webkit [74629] by jschuh@chromium.org
  • 3 edits
    2 adds in trunk

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):
3:50 AM Changeset in webkit [74628] by evan@chromium.org
  • 2 edits in trunk/WebCore

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):
3:47 AM Changeset in webkit [74627] by andreip@google.com
  • 1 edit
    1 add in trunk/LayoutTests

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.
3:27 AM Changeset in webkit [74626] by commit-queue@webkit.org
  • 12 edits
    2 adds in trunk

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:
3:23 AM Changeset in webkit [74625] by gyuyoung.kim@samsung.com
  • 2 edits in trunk/WebCore

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.
3:09 AM Changeset in webkit [74624] by andreip@google.com
  • 2 edits in trunk/LayoutTests

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:
3:05 AM Changeset in webkit [74623] by eric@webkit.org
  • 3 edits in trunk/Tools

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:
2:38 AM Changeset in webkit [74622] by jschuh@chromium.org
  • 3 edits
    2 adds in trunk

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):
2:19 AM Changeset in webkit [74621] by hamaji@chromium.org
  • 3 edits in trunk/Tools

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:
2:12 AM Changeset in webkit [74620] by Philippe Normand
  • 2 edits in trunk/WebKit/gtk

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.
2:00 AM Changeset in webkit [74619] by jschuh@chromium.org
  • 2 edits in trunk/WebCore

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):
1:01 AM Changeset in webkit [74618] by abarth@webkit.org
  • 10 edits
    1 move
    2 adds in trunk

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:
12:06 AM Changeset in webkit [74617] by Patrick Gansterer
  • 2 edits in trunk/JavaScriptCore

2010-12-24 Patrick Gansterer <Patrick Gansterer>

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:

Dec 23, 2010:

10:14 PM Changeset in webkit [74616] by Lucas Forschler
  • 3 edits in trunk/Tools

2010-12-23 Lucas Forschler <Lucas Forschler>

Reviewed by Eric Seidel.

Bug 51539 - update iexploder scripts to point to new location

  • Scripts/run-iexploder-tests:
  • Scripts/update-iexploder-cssproperties:
10:11 PM Changeset in webkit [74615] by mitz@apple.com
  • 2 edits in trunk/WebKit2

Make dragging work again after r74591.

Reviewed by Sam Weinig.

  • UIProcess/WebPageProxy.cpp:

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

10:05 PM Changeset in webkit [74614] by yutak@chromium.org
  • 2 edits in trunk/LayoutTests

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:
9:26 PM Changeset in webkit [74613] by ariya@webkit.org
  • 1 edit in trunk/LayoutTests/platform/gtk/Skipped

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

8:59 PM Changeset in webkit [74612] by yutak@chromium.org
  • 2 edits in trunk/LayoutTests

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:
8:50 PM Changeset in webkit [74611] by yutak@chromium.org
  • 5 edits in trunk/LayoutTests

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:
8:31 PM Changeset in webkit [74610] by evan@chromium.org
  • 2 edits in trunk/WebCore

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):
8:06 PM Changeset in webkit [74609] by ariya@webkit.org
  • 5 edits
    13 adds in trunk

[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:
7:40 PM Changeset in webkit [74608] by rniwa@webkit.org
  • 2 edits in trunk/LayoutTests

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:
7:18 PM Changeset in webkit [74607] by rniwa@webkit.org
  • 1 edit
    1 add in trunk/LayoutTests

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

Unreviewed Qt rebaseline for r74593.

  • platform/qt/editing/pasteboard/copy-in-password-field-expected.txt: Added.
6:56 PM Changeset in webkit [74606] by rniwa@webkit.org
  • 4 edits in trunk/LayoutTests

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:
6:53 PM Changeset in webkit [74605] by commit-queue@webkit.org
  • 6 edits
    2 adds in trunk

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):
6:38 PM Changeset in webkit [74604] by rniwa@webkit.org
  • 54 edits in trunk/LayoutTests

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:
6:33 PM Changeset in webkit [74603] by rniwa@webkit.org
  • 5 edits in trunk

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):
6:17 PM Changeset in webkit [74602] by rniwa@webkit.org
  • 104 edits in trunk/LayoutTests

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:
6:14 PM Changeset in webkit [74601] by rniwa@webkit.org
  • 3 edits
    2 adds in trunk

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):
5:40 PM Changeset in webkit [74600] by barraclough@apple.com
  • 12 edits
    1 move
    1 delete in trunk/JavaScriptCore

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:
5:36 PM Changeset in webkit [74599] by rniwa@webkit.org
  • 55 edits in trunk/LayoutTests

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:
5:19 PM Changeset in webkit [74598] by evan@chromium.org
  • 2 edits in trunk/Tools

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):
5:16 PM Changeset in webkit [74597] by jberlin@webkit.org
  • 8 edits in trunk

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):

5:11 PM Changeset in webkit [74596] by Patrick Gansterer
  • 2 edits in trunk/WebCore

2010-12-23 Patrick Gansterer <Patrick Gansterer>

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:
5:03 PM Changeset in webkit [74595] by Patrick Gansterer
  • 2 edits in trunk/JavaScriptCore

2010-12-23 Patrick Gansterer <Patrick Gansterer>

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):
4:55 PM Changeset in webkit [74594] by abarth@webkit.org
  • 2 edits in trunk/WebCore

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

Remove include of non-existant file.

  • bindings/v8/custom/V8DOMWindowCustom.cpp:
4:22 PM Changeset in webkit [74593] by rniwa@webkit.org
  • 128 edits
    4 adds in trunk

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:
4:00 PM Changeset in webkit [74592] by abarth@webkit.org
  • 2 edits
    2 deletes in trunk/WebCore

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.
3:55 PM Changeset in webkit [74591] by andersca@apple.com
  • 3 edits in trunk/WebKit2

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.

3:43 PM Changeset in webkit [74590] by mitz@apple.com
  • 2 edits in trunk/WebCore

Build fix.

  • editing/EditorCommand.cpp:

(WebCore::enabledDismissCorrectionPanel):

3:42 PM Changeset in webkit [74589] by Darin Adler
  • 3 edits in trunk/WebCore

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.
3:36 PM Changeset in webkit [74588] by bweinstein@apple.com
  • 3 edits in trunk/WebKit2

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).

3:15 PM Changeset in webkit [74587] by kbr@google.com
  • 5 edits in trunk

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):
3:10 PM Changeset in webkit [74586] by commit-queue@webkit.org
  • 2 edits in trunk/WebKit/mac

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):
3:08 PM Changeset in webkit [74585] by kbr@google.com
  • 2 edits in trunk/LayoutTests

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:
2:49 PM Changeset in webkit [74584] by weinig@apple.com
  • 2 edits in trunk/WebKit2

Fix incorrect assertion causing all pages with subframes to crash.

  • UIProcess/WebFrameProxy.cpp:

(WebKit::WebFrameProxy::appendChild):

2:40 PM Changeset in webkit [74583] by Darin Adler
  • 1 edit in trunk/WebKit2/ChangeLog

Fixed out-of-order ChangeLog.

2:34 PM Changeset in webkit [74582] by andersca@apple.com
  • 18 edits
    1 add in trunk/WebKit2

2010-12-23 Darin Adler <Darin Adler>

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.
2:23 PM Changeset in webkit [74581] by mihaip@chromium.org
  • 7 edits
    1 add in trunk

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):
2:20 PM Changeset in webkit [74580] by Darin Adler
  • 6 edits in trunk

2010-12-23 Darin Adler <Darin Adler>

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 Adler>

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.
2:15 PM Changeset in webkit [74579] by mdelaney@apple.com
  • 2 edits in trunk/WebCore

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):
1:55 PM WebKit Team edited by evan@chromium.org
add myself to commiter list (diff)
1:28 PM Changeset in webkit [74578] by ajwong@chromium.org
  • 79 edits in trunk/LayoutTests

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:
1:08 PM Changeset in webkit [74577] by commit-queue@webkit.org
  • 4 edits in trunk

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):
1:02 PM Changeset in webkit [74576] by kbr@google.com
  • 2 edits in trunk/LayoutTests

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 PM Changeset in webkit [74575] by weinig@apple.com
  • 2 edits in trunk/WebKit2

<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 PM Changeset in webkit [74574] by jschuh@chromium.org
  • 6 edits
    2 adds in trunk

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 PM Changeset in webkit [74573] by kbr@google.com
  • 2 edits in trunk/LayoutTests

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 PM Changeset in webkit [74572] by weinig@apple.com
  • 2 edits in trunk/WebKit/chromium

Fix build.

  • src/FrameLoaderClientImpl.cpp:

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

12:10 PM Changeset in webkit [74571] by weinig@apple.com
  • 43 edits in trunk

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 AM Changeset in webkit [74570] by mdelaney@apple.com
  • 6 edits
    6 adds in trunk

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 AM Changeset in webkit [74569] by andersca@apple.com
  • 3 edits in trunk/WebKit2

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 AM Changeset in webkit [74568] by enne@google.com
  • 11 edits
    2 adds in trunk

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 AM Changeset in webkit [74567] by kbr@google.com
  • 2 edits in trunk/LayoutTests

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 AM Changeset in webkit [74566] by rniwa@webkit.org
  • 29 edits
    1 add in trunk

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 AM Changeset in webkit [74565] by Lucas Forschler
  • 1 edit
    13 deletes in trunk/Tools

2010-12-23 Lucas Forschler <Lucas Forschler>

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 AM Changeset in webkit [74564] by andersca@apple.com
  • 5 edits in trunk/WebKit2

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.
9:45 AM Changeset in webkit [74563] by abarth@webkit.org
  • 5 edits in trunk/WebCore

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:
9:26 AM Changeset in webkit [74562] by apavlov@chromium.org
  • 2 edits in trunk/WebCore

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):
8:56 AM Changeset in webkit [74561] by commit-queue@webkit.org
  • 5 edits in trunk

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):
8:53 AM Changeset in webkit [74560] by jberlin@webkit.org
  • 1 edit
    2 adds in trunk/LayoutTests

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.
8:39 AM Changeset in webkit [74559] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

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.
8:23 AM Changeset in webkit [74558] by commit-queue@webkit.org
  • 6 edits in trunk/WebKit/qt

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):
6:45 AM Changeset in webkit [74557] by jeremy@chromium.org
  • 13 edits in trunk/LayoutTests

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:
6:41 AM Changeset in webkit [74556] by commit-queue@webkit.org
  • 6 edits in trunk

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.
6:24 AM Changeset in webkit [74555] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

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:
6:08 AM Changeset in webkit [74554] by commit-queue@webkit.org
  • 3 edits in trunk/WebKit/efl

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):
5:49 AM Changeset in webkit [74553] by commit-queue@webkit.org
  • 8 edits
    3 adds in trunk

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.
5:23 AM Changeset in webkit [74552] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

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.
5:14 AM Changeset in webkit [74551] by Philippe Normand
  • 2 edits in trunk/WebCore

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):
5:09 AM Changeset in webkit [74550] by jeremy@chromium.org
  • 2 edits in trunk/LayoutTests

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:
4:56 AM Changeset in webkit [74549] by tkent@chromium.org
  • 31 edits in trunk/WebCore

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:
3:12 AM Changeset in webkit [74548] by apavlov@chromium.org
  • 2 edits in trunk/WebCore

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):
2:38 AM Changeset in webkit [74547] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

Unreviewed. Update Qt specific expected file after r74539.

  • platform/qt/fast/dom/Window/window-properties-expected.txt:
2:20 AM Changeset in webkit [74546] by commit-queue@webkit.org
  • 1 edit
    3 adds in trunk/LayoutTests

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.
1:54 AM Changeset in webkit [74545] by Antti Koivisto
  • 2 edits in trunk/LayoutTests

Not reviewed.

Another attempt.

  • http/tests/local/slow-css-pass.cgi:
1:20 AM Changeset in webkit [74544] by loislo@chromium.org
  • 2 edits in trunk/Tools

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):
1:18 AM Changeset in webkit [74543] by Antti Koivisto
  • 2 edits in trunk/LayoutTests

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:
12:17 AM Changeset in webkit [74542] by gyuyoung.kim@samsung.com
  • 3 edits in trunk/WebKit/efl

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:
12:00 AM Changeset in webkit [74541] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

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.

Dec 22, 2010:

11:43 PM Changeset in webkit [74540] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

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

Reviewed by Darin Fisher.

[chromium] Optimize canvas.toDataURL("image/png") unpremultiplication loop
https://bugs.webkit.org/show_bug.cgi?id=51321

Follow on from r73890, unroll the SkUnPreMultiply::PMColorToColor() call for
the "image/png" encoder case viz., compute the unpremultiplication in-place.
See https://bugs.webkit.org/show_bug.cgi?id=50804 for reference.

No new tests: canvas.toDataURL() is covered by existing tests.

  • platform/image-encoders/skia/PNGImageEncoder.cpp: (WebCore::preMultipliedBGRAtoRGBA):
10:55 PM Changeset in webkit [74539] by ariya@webkit.org
  • 8 edits in trunk

2010-12-22 Ariya Hidayat <ariya@sencha.com>

Reviewed by Simon Fraser.

Implement skewX() and skewY() for CSSMatrix.
https://bugs.webkit.org/show_bug.cgi?id=38335

  • fast/dom/Window/window-properties-expected.txt: Adjusted.
  • transforms/cssmatrix-2d-interface.xhtml: Added tests for skewX() and skewY().
  • transforms/cssmatrix-2d-interface-expected.txt: Adjusted.

2010-12-22 Ariya Hidayat <ariya@sencha.com>

Reviewed by Simon Fraser.

Implement skewX() and skewY() for CSSMatrix.
https://bugs.webkit.org/show_bug.cgi?id=38335

  • css/WebKitCSSMatrix.cpp: (WebCore::WebKitCSSMatrix::skew): Implemented skewX() and skewY().
  • css/WebKitCSSMatrix.h: Added skewX() and skewY().
  • css/WebKitCSSMatrix.idl: Added skewX() and skewY().
9:45 PM Changeset in webkit [74538] by Simon Fraser
  • 14 edits
    4 adds in trunk

2010-12-22 Simon Fraser <Simon Fraser>

Reviewed by Dan Bernstein.

Support unprefixed box-shadow property
https://bugs.webkit.org/show_bug.cgi?id=51448

Add support for the box-shadow CSS property. We'll currently render this
similarly to -webkit-box-shadow, but a later fix will adjust the blur radius
handling to match the spec.

Test: fast/css/box-shadow.html

  • css/CSSComputedStyleDeclaration.cpp: (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue): Implement getComputedStyle for box-shadow
  • css/CSSParser.cpp: (WebCore::CSSParser::parseValue): (WebCore::ShadowParseContext::ShadowParseContext): (WebCore::ShadowParseContext::commitValue): (WebCore::ShadowParseContext::commitLength): (WebCore::ShadowParseContext::commitColor): Support CSSPropertyBoxShadow
  • css/CSSPropertyNames.in: add box-shadow
  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::applyProperty): Add CSSPropertyBoxShadow, and store in the ShadowData if this is -webkit-box-shadow or not. We'll use this to adjust radii later.
  • css/SVGCSSStyleSelector.cpp: (WebCore::CSSStyleSelector::applySVGProperty): Add CSSPropertyBoxShadow.
  • page/animation/AnimationBase.cpp: (WebCore::blendFunc): (WebCore::PropertyWrapperShadow::blend): (WebCore::AnimationBase::ensurePropertyMap): Support animations of box-shadow.
  • rendering/style/ShadowData.cpp: Store a flag to say if this is a -webkit-box-shadow. (WebCore::ShadowData::ShadowData): (WebCore::ShadowData::operator==):
  • rendering/style/ShadowData.h: (WebCore::ShadowData::ShadowData): (WebCore::ShadowData::isWebkitBoxShadow):
9:39 PM Changeset in webkit [74537] by msaboff@apple.com
  • 5 edits
    2 adds in trunk

2010-12-22 Michael Saboff <msaboff@apple.com>

Reviewed by Sam Weinig.

REGRESSION: Can't shadow/overwrite window.constructor
(causes TypeError exception)
https://bugs.webkit.org/show_bug.cgi?id=48953

Added code in the javascript code generator to create a setter for
a constructor if the new extended attribute ReplaceableConstructor
is present for the interface.

  • bindings/scripts/CodeGeneratorJS.pm:
  • page/DOMWindow.idl:

2010-12-22 Michael Saboff <msaboff@apple.com>

Reviewed by Sam Weinig.

REGRESSION: Can't shadow/overwrite window.constructor
(causes TypeError exception)
https://bugs.webkit.org/show_bug.cgi?id=48953

Changed cross-frame-access-put test results to reflect that setting
the window constructor will be disallowed due to cross frame security
checking.
Added a new test to check that we can change window.constructor.

  • fast/dom/Window/window-constructor-settable-expected.txt: Added.
  • fast/dom/Window/window-constructor-settable.html: Added.
  • http/tests/security/cross-frame-access-put-expected.txt:
9:27 PM Changeset in webkit [74536] by mihaip@chromium.org
  • 2 edits in trunk/LayoutTests

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

Unreviewed test expectations update. fast/canvas/fillText-shadow.html
fails with Skia in the GPU suite too.

  • platform/chromium-gpu/test_expectations.txt:
9:25 PM Changeset in webkit [74535] by commit-queue@webkit.org
  • 4 edits in trunk

2010-12-22 Steve Lacey <sjl@chromium.org>

Reviewed by David Levin.

Change css for chromium media controls to deal with the timeline
not expanding when the current time disappears. This happens when
the width of the control drops below 240px.

This change results in a minor pixel difference - the timebar now
aligns better with the time and when the time is not displayed is
now perfectly centered. Disabling the affected tests for now
and will rebaseline.

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

  • platform/chromium/test_expectations.txt:

2010-12-22 Steve Lacey <sjl@chromium.org>

Reviewed by David Levin.

Change css for chromium media controls to deal with the timeline
not expanding when the current time disappears. This happens when
the width of the control drops below 240px.

This change results in a minor pixel difference - the timebar now
aligns better with the time and when the time is not displayed is
now perfectly centered. Disabling the affected tests for now
and will rebaseline.

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

  • css/mediaControlsChromium.css: (audio::-webkit-media-controls-timeline-container, video::-webkit-media-controls-timeline-container): (audio::-webkit-media-controls-current-time-display, video::-webkit-media-controls-current-time-display): (audio::-webkit-media-controls-time-remaining-display, video::-webkit-media-controls-time-remaining-display): (audio::-webkit-media-controls-timeline, video::-webkit-media-controls-timeline):
9:05 PM Changeset in webkit [74534] by mihaip@chromium.org
  • 2 edits in trunk/LayoutTests

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

Unreviewed test expectations update. fast/canvas/fillText-shadow.html
fails with Skia.

  • platform/chromium/test_expectations.txt:
8:01 PM Changeset in webkit [74533] by jberlin@webkit.org
  • 1 edit
    1 add in trunk/LayoutTests

Add Windows-specific (failing) expected results for media/audio-data-url.html in order to get
the Windows 7 Release test bot green again. Unreviewed.
https://bugs.webkit.org/show_bug.cgi?id=51518

  • platform/win/media/audio-data-url-expected.txt: Added.
7:25 PM Changeset in webkit [74532] by mihaip@chromium.org
  • 6 edits
    2 adds in trunk

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

Reviewed by Simon Fraser.

the shadow direction is negated in canvas context shadowOffsetY.
https://bugs.webkit.org/show_bug.cgi?id=25619

Add test to verify that shadowOffsetY is handled correctly.

  • fast/canvas/fillText-shadow-expected.txt: Added.
  • fast/canvas/fillText-shadow.html: Added.

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

Reviewed by Simon Fraser.

the shadow direction is negated in canvas context shadowOffsetY.
https://bugs.webkit.org/show_bug.cgi?id=25619

Invert shadow offset Y direction in 0-blur path in FontMac::drawGlyps
if we're not transforming shadows (i.e. are rendering in a canvas). Also
use CG shadows if there's a non-translate/flip transform applied
(shadow offsets are supposed to ignore transforms, but the naive/simple
implementation was taking them into account).

Also does the equivalent changes to FontCGWin.

Test: fast/canvas/fillText-shadow.html

  • platform/graphics/GraphicsContext.cpp: (WebCore::GraphicsContext::shadowsIgnoreTransforms):
  • platform/graphics/GraphicsContext.h:
  • platform/graphics/mac/FontMac.mm: (WebCore::Font::drawGlyphs):
  • platform/graphics/win/FontCGWin.cpp: (WebCore::Font::drawGlyphs):
6:59 PM Changeset in webkit [74531] by andersca@apple.com
  • 2 edits in trunk/WebKit2

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

Reviewed by Sam Weinig.

Webkit2: WebProcess crashes when Private browsing is turned on on yahoo.com
https://bugs.webkit.org/show_bug.cgi?id=51516
<rdar://problem/8766995>

  • WebProcess/Plugins/Netscape/NetscapePlugin.cpp: (WebKit::NetscapePlugin::NPP_SetValue): Add a null check for NPP_SetValue.
6:53 PM Changeset in webkit [74530] by andersca@apple.com
  • 2 edits in trunk/WebKit2

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

Reviewed by Sam Weinig.

REGRESSION (WK2): Plugins swallow CMD-W, CMD-Q, and probably other shortcuts
https://bugs.webkit.org/show_bug.cgi?id=51515
<rdar://problem/8740926>

Always return false for keyboard events where the command key is down.

  • WebProcess/Plugins/Netscape/mac/NetscapePluginMac.mm: (WebKit::NetscapePlugin::platformHandleKeyboardEvent):
6:44 PM Changeset in webkit [74529] by tonyg@chromium.org
  • 5 edits in trunk

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

Unreviewed, rolling out r74503.
http://trac.webkit.org/changeset/74503
https://bugs.webkit.org/show_bug.cgi?id=51513

breaks chromium mac debug compile (Requested by tonyg-cr on
#webkit).

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

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

Unreviewed, rolling out r74503.
http://trac.webkit.org/changeset/74503
https://bugs.webkit.org/show_bug.cgi?id=51513

breaks chromium mac debug compile (Requested by tonyg-cr on
#webkit).

  • src/GraphicsContext3DChromium.cpp: (WebCore::GraphicsContext3DInternal::paintRenderingResultsToCanvas):
5:57 PM Changeset in webkit [74528] by weinig@apple.com
  • 6 edits in trunk/WebKit2

WebProcess should be terminated if invalid frameIDs are
passed to the UIProcess.
https://bugs.webkit.org/show_bug.cgi?id=51508

Reviewed by Darin Adler.

  • Introduce a mechanism to mark a messages as invalid due to reasons other than in ability to decode the message, eg. an invalid frameID is passed in a message.
  • Platform/CoreIPC/Connection.cpp:

(CoreIPC::Connection::Connection):
(CoreIPC::Connection::markCurrentlyDispatchedMessageAsInvalid): Sets
the m_didReceiveInvalidMessage bit so that it can be picked up in
(CoreIPC::Connection::dispatchMessages): Check for m_didReceiveInvalidMessage in
addition to tainted arguments.

  • Platform/CoreIPC/Connection.h:

Add base macro for others to extend, that calls markCurrentlyDispatchedMessageAsInvalid()
ASSERTs, and returns.

  • UIProcess/WebPageProxy.cpp:

Add Message checks for all WebFrameProxy's gotten from frameIDs passed
over the wire.

  • UIProcess/WebProcessProxy.cpp:

(WebKit::isGoodMapKey):
Checks that the key can be inserted safely into a map (eg. not the empty or deleted value).

(WebKit::WebProcessProxy::webFrame):
Return null for bad keys (eg. 0 or -1).

(WebKit::WebProcessProxy::canCreateFrame):
Added. Checks that the ID is good and not in the map yet. Used
as a consistency check before creating frames.

(WebKit::WebProcessProxy::frameCreated):
ASSERT that the key is good, in addition to not in the map yet.

(WebKit::WebProcessProxy::didDestroyFrame):
ASSERT that the key is good.

  • UIProcess/WebProcessProxy.h:

Move WebFrameProxy HashMap type into a typedef.

5:45 PM Changeset in webkit [74527] by Darin Adler
  • 11 edits in trunk/WebKit2

2010-12-22 Darin Adler <Darin Adler>

Reviewed by Anders Carlsson.

WebKit2 needs a way to return the main resource data
https://bugs.webkit.org/show_bug.cgi?id=51510

  • UIProcess/API/C/WKFrame.cpp: (WKFrameGetMainResourceData): Added. (callGetMainResourceDataBlockAndDispose): Added. (WKFrameGetMainResourceData_b): Added.
  • UIProcess/API/C/WKFrame.h: Added WKFrameGetMainResourceData.
  • UIProcess/WebFrameProxy.cpp: (WebKit::WebFrameProxy::getMainResourceData): Added.
  • UIProcess/WebFrameProxy.h: Ditto.
  • UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::getMainResourceDataOfFrame): Added. (WebKit::WebPageProxy::dataCallback): Replaced the individual callbacks with just these two. (WebKit::WebPageProxy::stringCallback): Ditto.
  • UIProcess/WebPageProxy.h: Ditto.
  • UIProcess/WebPageProxy.messages.in: Ditto.
  • WebProcess/WebPage/WebPage.cpp: (WebKit::WebPage::runJavaScriptInMainFrame): Send the string callback message instead of a specific one. (WebKit::WebPage::getContentsAsString): Ditto. (WebKit::WebPage::getRenderTreeExternalRepresentation): Ditto. (WebKit::WebPage::getSelectionOrContentsAsString): Ditto. (WebKit::WebPage::getSourceForFrame): Ditto. (WebKit::WebPage::getMainResourceDataOfFrame): Added. (WebKit::WebPage::getWebArchiveOfFrame): Send the data callback message instead of a specific one.
  • WebProcess/WebPage/WebPage.h: Added getMainResourceDataOfFrame.
  • WebProcess/WebPage/WebPage.messages.in: Ditto.
5:35 PM Changeset in webkit [74526] by mitz@apple.com
  • 2 edits in trunk/WebCore

Replace runtime checks for hyphenation API availability with compile-time checks.

Reviewed by Darin Adler.

  • platform/text/cf/HyphenationCF.cpp:

(WebCore::::createValueForNullKey): Changed to use CFLocaleCopyCurrent() instead of using the
search locale, which is empty on Windows. On Mac, this is equivalent to using the search locale.
(WebCore::canHyphenate): Added this implementation for when the hyphenation API is not available.
It is identical to the implementation in Hyphenation.cpp.
(WebCore::lastHyphenLocation): Ditto.

5:26 PM Changeset in webkit [74525] by dpranke@chromium.org
  • 4 edits
    1 copy
    3 adds in trunk/Tools

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

Reviewed by Ojan Vafai.

This change splits out the TestRunner class into its own file.
When we fork the code for message passing, we will create a new
TestRunner, and this change will allow that to be more modular.

While we're at it, split out TestInput and ResultSummary into
their own files, to reduce the size of run_webkit_tests to
something more managable.

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

  • Scripts/webkitpy/layout_tests/layout_package/printing_unittest.py:
  • Scripts/webkitpy/layout_tests/layout_package/result_summary.py:
  • Scripts/webkitpy/layout_tests/layout_package/test_runner.py:
  • Scripts/webkitpy/layout_tests/layout_package/test_runner_unittest.py:
  • Scripts/webkitpy/layout_tests/layout_package/test_input.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests_unittest.py:

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

Reviewed by Ojan Vafai.

nrwt multiprocessing - start over, prepare to fork the code

This code cleans up the signatures and implementation of the
TestRunner class so we can easily fork it to run either the
stable implementation or the new, unstable message-passing
implementation. The two variants will have different
implementations of the run_tests() method. We will switch
between the two based on the setting for the '--worker-model'
switch. We rename the two currently valid values to 'old-inline'
and 'old-threads'.

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

  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests_unittest.py:
5:17 PM Changeset in webkit [74524] by Simon Fraser
  • 14 edits
    1 add in trunk

2010-12-22 Simon Fraser <Simon Fraser>

Reviewed by Darin Adler.

Cache snapshots of plug-ins before painting, to avoid script running during painting
https://bugs.webkit.org/show_bug.cgi?id=51493

When FrameView is asked to do a flattening paint (e.g. when Safari snapshots,
or when printing), plug-ins which otherwise use the CA rendering model
are sent a paint event. Some plug-ins may run script while handling this event,
or out of process plug-ins may process queued requests at this time. Running
script while inside layout or painting can have bad consequences, because it
can result in arbitrary changes to the render tree.

This patch avoids sending plug-ins paint events inside of painting. Instead,
we ask the plug-ins to cache a snapshot before we paint, and then the software
paint simply draws that snapshot.

Requires manual test, because the bug happens when Safari does
a page snapshot. It's not possible to tell the Test Netscape Plug-in
to paint without laying out, making an automated test impossible.

  • manual-tests/plugins/plugin-paint-causes-layout.html: Added.
  • page/FrameView.h:
  • page/FrameView.cpp: (WebCore::FrameView::paintContents): Notify all enclosed widgets that a flattening paint is going to happen. (WebCore::FrameView::notifyWidgetsInAllFrames): Utility method that tells the RenderView in all subframes to notify their widgets.
  • platform/Widget.h: (WebCore::Widget::notifyWidget): Generic method that can be used to send messages to widgets. Current messages are just 'before flattening paint' and 'after flattening paint'. Message has no payload.
  • rendering/RenderView.h:
  • rendering/RenderView.cpp: (WebCore::RenderView::getRetainedWidgets): (WebCore::RenderView::releaseWidgets): Factor code out of updateWidgetPositions(), since we use it in two places now.

(WebCore::RenderView::updateWidgetPositions): Use getRetainedWidgets() etc.
(WebCore::RenderView::notifyWidgets): Retain all the widgets, then send
them all the message.

  • rendering/RenderWidget.h:
  • rendering/RenderWidget.cpp: (WebCore::RenderWidget::notifyWidget): Pass the message to the widget.
  • Plugins/Hosted/WebHostedNetscapePluginView.mm: (-[WebHostedNetscapePluginView drawRect:]): If we have a cached snapshot, draw it. Also only send the snapshot message to the plugin proxy if we know we're snapshotting, since even if creating the snapshot image failed, we still don't want to call to the plug-in.
  • Plugins/WebBaseNetscapePluginView.h: Add a retained NSImage member for the snapshot.
  • Plugins/WebBaseNetscapePluginView.mm: (-[WebBaseNetscapePluginView cacheSnapshot]): Create an image and draw the snapshot into it. (-[WebBaseNetscapePluginView clearCachedSnapshot]): Clear the snapshot.
  • Plugins/WebNetscapePluginView.mm: (-[WebNetscapePluginView drawRect:]): If we have a cached snapshot, use it.
  • WebCoreSupport/WebFrameLoaderClient.mm: (NetscapePluginWidget::notifyWidget): Implement notifyWidget() and use it to cache and clear the snapshots.
4:55 PM Changeset in webkit [74523] by dpranke@chromium.org
  • 4 edits in trunk/Tools

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

Reviewed by Ojan Vafai.

nrwt multiprocessing - start over, prepare to fork the code

This code cleans up the signatures and implementation of the
TestRunner class so we can easily fork it to run either the
stable implementation or the new, unstable message-passing
implementation. The two variants will have different
implementations of the run_tests() method. We will switch
between the two based on the setting for the '--worker-model'
switch. We rename the two currently valid values to 'old-inline'
and 'old-threads'.

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

  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests_unittest.py:
4:50 PM Changeset in webkit [74522] by Darin Adler
  • 2 edits in trunk/WebKit2
  • WebProcess/WebPage/WebPage.cpp: Add #if to try to fix non-Mac builds.
4:44 PM Changeset in webkit [74521] by Darin Adler
  • 16 edits in trunk/WebKit2

2010-12-22 Darin Adler <Darin Adler>

Reviewed by Anders Carlsson.

Make Speech menu items work in WebKit2
https://bugs.webkit.org/show_bug.cgi?id=51504

  • UIProcess/API/C/WKFrame.cpp: (WKFrameGetWebArchive): Updated to use new-named typedef.
  • UIProcess/API/C/WKPage.cpp: (WKPageRunJavaScriptInMainFrame): Ditto. (WKPageRenderTreeExternalRepresentation): Ditto. (WKPageGetSourceForFrame): Ditto. (WKPageGetContentsAsString): Ditto.
  • UIProcess/API/mac/WKView.mm: (menuItem): Added. Helper function to make it easy to call menu item functions on arbitrary user interface items. (toolbarItem): Ditto. (-[WKView validateUserInterfaceItem:]): Added validation for startSpeaking and stopSpeaking commands. Reworked the code to fix a bug where toggleContinuousSpellChecking would always be enabled even when it should be disabled. Reworked the asynchronous validation code so it works for items other than menu items and can also handle more than one item for the same command. Added some comments. (speakString): Added. Helper function to be used as a callback when we get the selection or contents to speak. (-[WKView startSpeaking:]): Added. Uses getSelectionOrContentsAsString. (-[WKView stopSpeaking:]): Added. (-[WKView _setUserInterfaceItemState:enabled:state:]): Reworked to work with the changes to validateUserInterfaceItem above.
  • UIProcess/WebFrameProxy.cpp: (WebKit::WebFrameProxy::isDisplayingMarkupDocument): Added a FIXME about a problem I noticed. (WebKit::WebFrameProxy::didCommitLoad): Tweaked argument name. (WebKit::WebFrameProxy::getWebArchive): Changed to use new-named typedef.
  • UIProcess/WebFrameProxy.h: Changed WebArchiveCallback to DataCallaback.
  • UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::close): Invalidate the two callback maps since we only have two now. (WebKit::WebPageProxy::runJavaScriptInMainFrame): Changed to use merged maps and typedefs so we don't need a separate callback map for each function. (WebKit::WebPageProxy::getRenderTreeExternalRepresentation): Ditto. (WebKit::WebPageProxy::getSourceForFrame): Ditto. (WebKit::WebPageProxy::getContentsAsString): Ditto. (WebKit::WebPageProxy::getSelectionOrContentsAsString): Added. (WebKit::WebPageProxy::getWebArchiveOfFrame): Ditto. (WebKit::WebPageProxy::didGetContentsAsString): Ditto. (WebKit::WebPageProxy::didGetSelectionOrContentsAsString): Ditto. (WebKit::WebPageProxy::didRunJavaScriptInMainFrame): Ditto. (WebKit::WebPageProxy::didGetRenderTreeExternalRepresentation): Ditto. (WebKit::WebPageProxy::didGetSourceForFrame): Ditto. (WebKit::WebPageProxy::didGetWebArchiveOfFrame): Ditto. (WebKit::WebPageProxy::processDidCrash): Ditto.
  • UIProcess/WebPageProxy.h: Updated for changes above.
  • UIProcess/WebPageProxy.messages.in: Added DidGetSelectionOrContentsAsString, GetIsSpeaking, Speak, and StopSpeaking.
  • UIProcess/mac/WebPageProxyMac.mm: (WebKit::WebPageProxy::getIsSpeaking): Added. (WebKit::WebPageProxy::speak): Added. (WebKit::WebPageProxy::stopSpeaking): Added.
  • WebProcess/WebCoreSupport/mac/WebContextMenuClientMac.mm: (WebKit::WebContextMenuClient::isSpeaking): Call WebPage instead of doing the work directly, so it can run in the UI process. (WebKit::WebContextMenuClient::speak): Ditto. (WebKit::WebContextMenuClient::stopSpeaking): Ditto. (WebKit::WebContextMenuClient::searchWithSpotlight): Added FIXME comment and changed code to avoid a C-style cast.
  • WebProcess/WebPage/WebFrame.cpp: (WebKit::WebFrame::selectionAsString): Added.
  • WebProcess/WebPage/WebFrame.h: Ditto.
  • WebProcess/WebPage/WebPage.cpp: (WebKit::WebPage::changeAcceleratedCompositingMode): Use sendSync in the modern way instead of explicitly calling WebProcess. (WebKit::WebPage::getSelectionOrContentsAsString): Added. (WebKit::WebPage::isSpeaking): Added. (WebKit::WebPage::speak): Added. (WebKit::WebPage::stopSpeaking): Added.
  • WebProcess/WebPage/WebPage.h: Added functions.
  • WebProcess/WebPage/WebPage.messages.in: Added GetSelectionOrContentsAsString.
4:39 PM Changeset in webkit [74520] by Laszlo Gombos
  • 2 edits in trunk/JavaScriptCore

2010-12-22 Laszlo Gombos <Laszlo Gombos>

Unreviewed build fix.

[Symbian] Make sure OSAllocatorSymbian builds

This patch only addresses the build problem.

https://bugs.webkit.org/show_bug.cgi?id=51128 tracks the full
(re)implementation of the Symbian allocator.

  • wtf/OSAllocatorSymbian.cpp: (WTF::OSAllocator::reserveUncommitted): (WTF::OSAllocator::reserveAndCommit): (WTF::OSAllocator::commit):
4:36 PM Changeset in webkit [74519] by jianli@chromium.org
  • 2 edits in branches/chromium/597/LayoutTests

Merge 74435 - Rebaseline test for r74427.

  • http/tests/local/formdata/send-form-data-with-sliced-file-expected.txt: Updated.

TBR=jianli@chromium.org
Review URL: http://codereview.chromium.org/6042006

4:36 PM Changeset in webkit [74518] by andersca@apple.com
  • 2 edits in trunk/WebKit2

Fix build.

  • Platform/mac/MachUtilities.cpp:
4:33 PM Changeset in webkit [74517] by jianli@chromium.org
  • 1 edit
    3 copies in branches/chromium/597

Merge 74427 - 2010-12-21 Jian Li <jianli@chromium.org>

Reviewed by Darin Adler.

Fix bug 51366: REGRESSION (r66452): Form data no longer contains
'Content-Type' header for files with unrecognized extensions
https://bugs.webkit.org/show_bug.cgi?id=51366

Test: http/tests/local/formdata/form-data-with-unknown-file-extension.html

  • platform/network/FormData.cpp: (WebCore::FormData::appendKeyValuePairItems):

2010-12-21 Jian Li <jianli@chromium.org>

Reviewed by Darin Adler.

Fix bug 51366: REGRESSION (r66452): Form data no longer contains
'Content-Type' header for files with unrecognized extensions
https://bugs.webkit.org/show_bug.cgi?id=51366

  • http/tests/local/formdata/form-data-with-unknown-file-extension-expected.txt: Added.
  • http/tests/local/formdata/form-data-with-unknown-file-extension.html: Added.
  • http/tests/local/resources/file.invalid: Added.

TBR=jianli@chromium.org
Review URL: http://codereview.chromium.org/6077007

4:27 PM Changeset in webkit [74516] by enne@google.com
  • 2 edits in trunk/WebCore

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

Reviewed by Kenneth Russell.

Fix null pointer deref when restoring a lost WebGL context.
https://bugs.webkit.org/show_bug.cgi?id=51494

No new tests, as context restored events are not currently testable.

  • html/canvas/WebGLRenderingContext.cpp: (WebCore::WebGLRenderingContext::initializeNewContext): (WebCore::WebGLRenderingContext::restoreContext):
4:16 PM Changeset in webkit [74515] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

2010-12-22 Adam Bergkvist <adam.bergkvist@ericsson.com>

Reviewed by Martin Robinson.

[GTK] Building with Blob support fails again
https://bugs.webkit.org/show_bug.cgi?id=51392

Build fix.

  • GNUmakefile.am:
4:15 PM Changeset in webkit [74514] by andersca@apple.com
  • 7 edits in trunk/WebKit2

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

Reviewed by Darin Adler.

Detect WebProcess crashes sooner, without waiting for ReportCrash to finish
https://bugs.webkit.org/show_bug.cgi?id=51505

  • Platform/CoreIPC/CoreIPCMessageKinds.h: Add SetExceptionPort.
  • Platform/CoreIPC/mac/ConnectionMac.cpp: (CoreIPC::Connection::platformInvalidate): If we have an exception port, unregister it.

(CoreIPC::Connection::platformInitialize):
Set m_exceptionPort to null.

(CoreIPC::Connection::open):
If we have an exception port, register an event handler for it and send it over
to the other side.

(CoreIPC::Connection::receiveSourceEventHandler):
Handle the SetExceptionPort message.

(CoreIPC::Connection::exceptionSourceEventHandler):
Receive the message, forward it to the real exception handler and call connectionDidClose.

(CoreIPC::Connection::setShouldCloseConnectionOnMachExceptions):
Create an exception port.

  • Platform/mac/MachUtilities.cpp: (machExceptionPort): (setMachExceptionPort):
  • Platform/mac/MachUtilities.h: Add helper functions for getting and setting the mach exception port.
  • UIProcess/WebProcessProxy.cpp: (WebKit::WebProcessProxy::didFinishLaunching): Call setShouldCloseConnectionOnMachExceptions on Mac.
4:13 PM Changeset in webkit [74513] by mitz@apple.com
  • 2 edits in trunk/WebCore

2010-12-22 Dan Bernstein <mitz@apple.com>

Reviewed by Anders Carlsson.

canHyphenate() always returns true for the default locale
https://bugs.webkit.org/show_bug.cgi?id=51506

No test, since this change only affects performance.

  • platform/text/cf/HyphenationCF.cpp: (WebCore::::createValueForNullKey): Return 0 if hyphenation is not supported for the default locale. (WebCore::::createValueForKey): Removed whitespace.
3:58 PM Changeset in webkit [74512] by ariya@webkit.org
  • 1 edit
    8 adds in trunk/LayoutTests

2010-12-22 Ariya Hidayat <ariya@sencha.com>

Reviewed by Simon Fraser.

Transform applied via CSSMatrix does not take full page zoom into account
https://bugs.webkit.org/show_bug.cgi?id=24808

The bug has been fixed with r74430 and r74451.
These two are just the tests.

  • transforms/2d/cssmatrix-2d-zoom-expected.checksum: Added.
  • transforms/2d/cssmatrix-2d-zoom-expected.png: Added.
  • transforms/2d/cssmatrix-2d-zoom-expected.txt: Added.
  • transforms/2d/cssmatrix-2d-zoom.html: Added.
  • transforms/3d/general/cssmatrix-3d-zoom-expected.checksum: Added.
  • transforms/3d/general/cssmatrix-3d-zoom-expected.png: Added.
  • transforms/3d/general/cssmatrix-3d-zoom-expected.txt: Added.
  • transforms/3d/general/cssmatrix-3d-zoom.html: Added.
3:47 PM Changeset in webkit [74511] by victorw@chromium.org
  • 2 edits in trunk/Tools

2010-12-22 Victor Wang <victorw@chromium.org>

Reviewed by Darin Fisher.

[Chromium] Update chromium archive test result url in rebaseline script.

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

  • Scripts/webkitpy/layout_tests/rebaseline_chromium_webkit_tests.py:
3:34 PM Changeset in webkit [74510] by jianli@chromium.org
  • 3 edits
    1 add in trunk/LayoutTests

Tests that File.type returns empty string for file with invalid extension.
https://bugs.webkit.org/show_bug.cgi?id=51501

Reviewed by David Levin.

  • editing/pasteboard/file-input-files-access-expected.txt:
  • editing/pasteboard/resources/file.invalidext: Added.
  • editing/pasteboard/script-tests/file-input-files-access.js:
3:32 PM Changeset in webkit [74509] by commit-queue@webkit.org
  • 3 edits
    7 adds in trunk/LayoutTests

2010-12-22 Steve Lacey <sjl@chromium.org>

Reviewed by David Levin.

[chromium] Rebaseline chromium media/controls-styling layout test and re-enable.
https://bugs.webkit.org/show_bug.cgi?id=51427

  • platform/chromium-linux/media/controls-styling-expected.checksum: Added.
  • platform/chromium-linux/media/controls-styling-expected.png: Added.
  • platform/chromium-mac/media/controls-styling-expected.checksum: Added.
  • platform/chromium-mac/media/controls-styling-expected.png: Added.
  • platform/chromium-mac/media/controls-styling-expected.txt: Added.
  • platform/chromium-win/media/controls-styling-expected.checksum: Added.
  • platform/chromium-win/media/controls-styling-expected.png: Added.
  • platform/chromium-win/media/controls-styling-expected.txt:
  • platform/chromium/test_expectations.txt:
3:30 PM Changeset in webkit [74508] by andersca@apple.com
  • 2 edits in trunk/WebKit2

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

Reviewed by Sam Weinig.

Move Mach message receiving code out into a new function
https://bugs.webkit.org/show_bug.cgi?id=51502

  • Platform/CoreIPC/mac/ConnectionMac.cpp: (CoreIPC::readFromMachPort): (CoreIPC::Connection::receiveSourceEventHandler):
3:05 PM Changeset in webkit [74507] by commit-queue@webkit.org
  • 2 edits in trunk/WebKit2

2010-12-22 Jeff Miller <jeffm@apple.com>

Reviewed by Sam Weinig.

InjectedBundle methods should use the relative URL constructor for KURL instead of the ParsedURLStringTag version
https://bugs.webkit.org/show_bug.cgi?id=51491

The url passed to various InjectedBundle methods is not from KURL::string(), i.e. it has not already been parsed by KURL,
so we have to use the relative URL constructor for KURL instead of the ParsedURLStringTag version.

  • WebProcess/InjectedBundle/InjectedBundle.cpp: (WebKit::InjectedBundle::addUserScript): (WebKit::InjectedBundle::addUserStyleSheet): (WebKit::InjectedBundle::removeUserScript): (WebKit::InjectedBundle::removeUserStyleSheet):
2:39 PM Changeset in webkit [74506] by yurys@chromium.org
  • 2 edits in trunk/WebCore

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

Reviewed by Joseph Pecoraro.

Web Inspector: script name gets parsed as HTML in Call stack panel
https://bugs.webkit.org/show_bug.cgi?id=51468

  • inspector/front-end/Placard.js: (WebInspector.Placard.prototype.set subtitle):
2:22 PM Changeset in webkit [74505] by xan@webkit.org
  • 4 edits in trunk

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

Reviewed by Gustavo Noronha.

  • configure.ac: bump version to 1.3.9.

WebKit/gtk:

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

Reviewed by Gustavo Noronha.

  • NEWS: update for 1.3.9 release.
2:11 PM Changeset in webkit [74504] by eric.carlson@apple.com
  • 2 edits in trunk/WebCore

2010-12-22 Eric Carlson <eric.carlson@apple.com>

Reviewed by Darin Adler.

MediaPlayer should look for MIME type in data: URL
https://bugs.webkit.org/show_bug.cgi?id=51195

If no MIME type is passed from the media element for a data: URL, try to get it from
the URL itself.

No new tests are possible because the MIME type is just passed to the media engine.

  • platform/graphics/MediaPlayer.cpp: (WebCore::MediaPlayer::load):
2:03 PM Changeset in webkit [74503] by commit-queue@webkit.org
  • 5 edits in trunk

2010-12-22 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-22 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):
1:42 PM Changeset in webkit [74502] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

2010-12-22 Shane Stephens <shanestephens@google.com>

Reviewed by David Levin.

[Chromium] SLOW lines in chromium test_expectations.txt for svg tests which are no longer slow.
https://bugs.webkit.org/show_bug.cgi?id=51429

  • platform/chromium/test_expectations.txt:
1:34 PM Changeset in webkit [74501] by Chris Fleizach
  • 3 edits
    2 adds in trunk

WebCore: WAI-ARIA menubar role mapped to AXGroup, should be AXMenuBar
https://bugs.webkit.org/show_bug.cgi?id=51488

Reviewed by Darin Adler.

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

  • accessibility/AccessibilityObject.cpp:

(WebCore::createARIARoleMap):

LayoutTests: Update fast/canvas/webgl/gl-uniform-arrays.html to bring in sync with Khronos
https://bugs.webkit.org/show_bug.cgi?id=51489

Patch by Kenneth Russell <kbr@google.com> on 2010-12-22
Reviewed by David Levin.

  • fast/canvas/webgl/gl-uniform-arrays-expected.txt:
  • fast/canvas/webgl/gl-uniform-arrays.html:
1:26 PM Changeset in webkit [74500] by Darin Adler
  • 4 edits in trunk/WebKit2

Move more of WebPageProxy::didCommitLoadForFrame into WebFrameProxy::didCommitLoad.

Reviewed by Sam Weinig.

  • UIProcess/WebFrameProxy.cpp:

(WebKit::WebFrameProxy::didCommitLoad): Added type and certificate info arguments,
and set m_MIMEType, m_isFrameSet, and m_certificateInfo.

  • UIProcess/WebFrameProxy.h: Removed setMIMEType and setCertificateInfo and added

arguments to didCommitLoad.

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::didCommitLoadForFrame): Removed code that is now inside the
WebFrameProxy::didCommitLoad function.

1:13 PM Changeset in webkit [74499] by kbr@google.com
  • 3 edits in trunk/LayoutTests

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

Reviewed by David Levin.

Update fast/canvas/webgl/gl-uniform-arrays.html to bring in sync with Khronos
https://bugs.webkit.org/show_bug.cgi?id=51489

  • fast/canvas/webgl/gl-uniform-arrays-expected.txt:
  • fast/canvas/webgl/gl-uniform-arrays.html:
1:05 PM Changeset in webkit [74498] by Lucas Forschler
  • 2 edits in trunk/Tools

2010-12-22 Lucas Forschler <Lucas Forschler>

Reviewed by Alice Liu.

<rdar://problem/8633222> record-memory-win needs to record memory used by the webprocess.
Verified on Chrome, Safari, and IE.
With this change, we will record memory from a parent browser window and all child processes.

  • record-memory-win/main.cpp: (ProcessArgs): (PrintUsage): (getMemoryInfo): (printProcessInfo): (evalProcesses): (UseImage): (QueryContinuously): (ElapsedTime):
1:05 PM Changeset in webkit [74497] by zmo@google.com
  • 6 edits in trunk

2010-12-22 Zhenyao Mo <zmo@google.com>

Reviewed by Kenneth Russell.

CopyTex{Sub}Image2D check always if the bound fbo's internal color format is compatible
https://bugs.webkit.org/show_bug.cgi?id=51423

  • html/canvas/WebGLRenderingContext.cpp: (WebCore::WebGLRenderingContext::copyTexImage2D): Remove the isGLES2Compliant flag around format compatibility checking. (WebCore::WebGLRenderingContext::copyTexSubImage2D): Ditto. (WebCore::WebGLRenderingContext::getBoundFramebufferColorFormat): Helper function to get bound framebuffer's color format; deal with the case of webgl's internal fbo.
  • html/canvas/WebGLRenderingContext.h:

2010-12-22 Zhenyao Mo <zmo@google.com>

Reviewed by Kenneth Russell.

CopyTex{Sub}Image2D check always if the bound fbo's internal color format is compatible
https://bugs.webkit.org/show_bug.cgi?id=51423

  • fast/canvas/webgl/tex-input-validation-expected.txt: Add test cases for copying from webgl's internal FBO.
  • fast/canvas/webgl/tex-input-validation.html: Ditto.
12:53 PM Changeset in webkit [74496] by xan@webkit.org
  • 6 edits in trunk

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

Reviewed by Gustavo Noronha.

[GTK] Compilation fixes with GTK+ 2.91.7
https://bugs.webkit.org/show_bug.cgi?id=51487

  • configure.ac: depend on GTK+ 2.91.7 when using GTK+ 3.x.

WebCore:

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

Reviewed by Gustavo Noronha.

[GTK] Compilation fixes with GTK+ 2.91.7
https://bugs.webkit.org/show_bug.cgi?id=51487

  • platform/gtk/GtkVersioning.h: do not define GDK_DISPLAY for backwards compatibility, it's now defined again in GTK+ 3.x.
  • platform/gtk/PasteboardHelper.cpp: do not call GDK functions to initialize global static variables (!). These are called when the .so is loaded, and they now require GType to be initialized and will crash. Initialize them once from the ctor, which is a more common pattern anyway. (WebCore::initGdkAtoms): new method to initialize static variables. (WebCore::PasteboardHelper::PasteboardHelper): call it.
  • plugins/gtk/PluginViewGtk.cpp: do not include gdkconfig.h when using GTK+ 3.x, since it's no longer installed.
12:53 PM Changeset in webkit [74495] by Lucas Forschler
  • 2 edits in trunk/Tools

2010-12-22 Lucas Forschler <Lucas Forschler>

Unreviewed rollout r74489, because it was missing changelog.

  • record-memory-win/main.cpp:
12:22 PM Changeset in webkit [74494] by Darin Adler
  • 11 edits in trunk/WebKit2

2010-12-21 Darin Adler <Darin Adler>

Reviewed by Brady Eidson and Sam Weinig.

Add a way to get a web archive in WebKit2
https://bugs.webkit.org/show_bug.cgi?id=51437

  • UIProcess/API/C/WKFrame.cpp: (WKFrameGetWebArchive): Added. (callGetWebArchiveBlockAndDispose): Added. (WKFrameGetWebArchive_b): Added.
  • UIProcess/API/C/WKFrame.h: Added functions above.
  • UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::close): Clear out m_webArchiveCallbacks. (WebKit::WebPageProxy::getWebArchiveOfFrame): Added. (WebKit::WebPageProxy::didGetWebArchiveOfFrame): Added. (WebKit::WebPageProxy::processDidCrash): Clear out m_webArchiveCallbacks.
  • UIProcess/WebPageProxy.h: Added functions above.
  • UIProcess/WebPageProxy.messages.in: Added DidGetWebArchiveOfFrame.
  • WebProcess/WebPage/WebPage.cpp: (WebKit::WebPage::getWebArchiveOfFrame): Added.
  • WebProcess/WebPage/WebPage.h: Added function above.
  • WebProcess/WebPage/WebPage.messages.in: Added GetWebArchiveOfFrame.
12:15 PM Changeset in webkit [74493] by krit@webkit.org
  • 40 edits in trunk/WebCore

2010-12-22 Dirk Schulze <krit@wbekit.org>

Reviewed by Nikolas Zimmermann.

Cleanup SVG code according to the webkit style rules 2
https://bugs.webkit.org/show_bug.cgi?id=51461

Second patch to fix indention and other style issues according to the WebKit style rules in the SVG code.

No changes of functionality, so no new tests.

  • svg/SVGFEDisplacementMapElement.cpp: (WebCore::SVGFEDisplacementMapElement::stringToChannel):
  • svg/SVGFEDistantLightElement.h:
  • svg/SVGFEFuncAElement.h:
  • svg/SVGFEFuncBElement.h:
  • svg/SVGFEFuncGElement.h:
  • svg/SVGFEFuncRElement.h:
  • svg/SVGFEMergeElement.cpp: (WebCore::SVGFEMergeElement::build):
  • svg/SVGFEMorphologyElement.cpp:
  • svg/SVGFEPointLightElement.h:
  • svg/SVGFESpecularLightingElement.cpp: (WebCore::SVGFESpecularLightingElement::findLights):
  • svg/SVGFESpotLightElement.h:
  • svg/SVGFitToViewBox.cpp: (WebCore::SVGFitToViewBox::parseViewBox): (WebCore::SVGFitToViewBox::isKnownAttribute):
  • svg/SVGFontFaceElement.cpp: (WebCore::cssPropertyIdForSVGAttributeName):
  • svg/SVGFontFaceElement.h: (WebCore::SVGFontFaceElement::associatedFontElement):
  • svg/SVGFontFaceNameElement.h:
  • svg/SVGFontFaceSrcElement.cpp:
  • svg/SVGFontFaceSrcElement.h:
  • svg/SVGFontFaceUriElement.h:
  • svg/SVGForeignObjectElement.h:
  • svg/SVGGlyphElement.cpp: (WebCore::parseArabicForm): (WebCore::parseOrientation):
  • svg/SVGGlyphElement.h: (WebCore::SVGGlyphIdentifier::SVGGlyphIdentifier): (WebCore::SVGGlyphIdentifier::inheritedValue): (WebCore::SVGGlyphIdentifier::operator==): (WebCore::SVGGlyphElement::rendererIsNeeded):
  • svg/SVGGlyphMap.h: (WebCore::GlyphMapNode::GlyphMapNode): (WebCore::GlyphMapNode::create): (WebCore::SVGGlyphMap::SVGGlyphMap): (WebCore::SVGGlyphMap::add): (WebCore::SVGGlyphMap::compareGlyphPriority): (WebCore::SVGGlyphMap::get): (WebCore::SVGGlyphMap::clear):
  • svg/SVGGradientElement.cpp:
  • svg/SVGHKernElement.cpp:
  • svg/SVGHKernElement.h: (WebCore::SVGHKernElement::rendererIsNeeded):
  • svg/SVGImageElement.h:
  • svg/SVGImageLoader.cpp:
  • svg/SVGImageLoader.h:
  • svg/SVGLangSpace.cpp: (WebCore::SVGLangSpace::parseMappedAttribute): (WebCore::SVGLangSpace::isKnownAttribute):
  • svg/SVGLangSpace.h: (WebCore::SVGLangSpace::xmllang):
  • svg/SVGMetadataElement.h:
  • svg/SVGMissingGlyphElement.h: (WebCore::SVGMissingGlyphElement::rendererIsNeeded):
  • svg/SVGPaint.cpp: (WebCore::SVGPaint::cssText):
  • svg/SVGPaint.h: (WebCore::SVGPaint::create): (WebCore::SVGPaint::paintType): (WebCore::SVGPaint::isSVGPaint):
  • svg/SVGParserUtilities.cpp:
  • svg/SVGParserUtilities.h: (WebCore::isWhitespace): (WebCore::skipOptionalSpaces): (WebCore::skipOptionalSpacesOrDelimiter):
  • svg/SVGPathElement.cpp:
  • svg/SVGPathParserFactory.h:
  • svg/SVGPathSegList.h:
11:51 AM Changeset in webkit [74492] by Chris Fleizach
  • 7 edits
    2 adds in trunk

AX: WAI-ARIA textbox role needs different AXRole, depending on value of aria-multiline property
https://bugs.webkit.org/show_bug.cgi?id=51481

WebCore:

Reviewed by Sam Weinig.

Support aria-multiline and use it to change the role of an ARIA "textbox"
from AXTextArea to AXTextField.

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

  • accessibility/AccessibilityObject.cpp:

(WebCore::AccessibilityObject::ariaIsMultiline):

  • accessibility/AccessibilityObject.h:

(WebCore::AccessibilityObject::isPasswordField):
(WebCore::AccessibilityObject::isTextControl):

  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::determineAriaRoleAttribute):

  • accessibility/AccessibilityRenderObject.h:
  • html/HTMLAttributeNames.in:

LayoutTests:

Reviewed by Sam Weining.

  • platform/mac/accessibility/aria-multiline-expected.txt: Added.
  • platform/mac/accessibility/aria-multiline.html: Added.
11:26 AM WebKit Team edited by rniwa@webkit.org
Adding myself as a reviewer (diff)
11:23 AM CSS21Results edited by Simon Fraser
(diff)
11:19 AM Changeset in webkit [74491] by Simon Fraser
  • 3 edits
    2 adds in trunk

2010-12-22 Simon Fraser <Simon Fraser>

Reviewed by Dan Bernstein.

CSS 2.1 failure: counter-increment-013 fails
https://bugs.webkit.org/show_bug.cgi?id=51483

Clamp the counter-increment property to the valid range of signed
integers.

Test: fast/css/counters/counter-increment-overflow.html

  • css/CSSParser.cpp: (WebCore::clampToSignedInteger): (WebCore::CSSParser::parseCounter):
11:11 AM Changeset in webkit [74490] by rniwa@webkit.org
  • 2 edits in trunk/Tools

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

Adding myself as a reviewer.

  • Scripts/webkitpy/common/config/committers.py:
11:10 AM Changeset in webkit [74489] by Lucas Forschler
  • 1 edit in trunk/Tools/record-memory-win/main.cpp
11:03 AM Changeset in webkit [74488] by mitz@apple.com
  • 4 edits in trunk

Changed WebKitTools to Tools.

JavaScriptCore:

WebKit/win:

  • WebKit.vcproj/WebKit.sln:
10:56 AM Changeset in webkit [74487] by mitz@apple.com
  • 8 edits in trunk

Changed WebKitTools to Tools in script build phases.

Rubber-stamped by Mark Rowe.

JavaScriptCore:

JavaScriptGlue:

  • JavaScriptGlue.xcodeproj/project.pbxproj:

WebCore:

  • WebCore.xcodeproj/project.pbxproj:

WebKit:

  • WebKit.xcodeproj/project.pbxproj:
10:50 AM Changeset in webkit [74486] by chang.shu@nokia.com
  • 2 edits in trunk/LayoutTests

2010-12-22 Chang Shu <chang.shu@nokia.com>

Unreviewed layout test fix.

Revert changeset 74481: http://trac.webkit.org/changeset/74481,
as it causes regression on layout tests.

  • platform/qt/Skipped:
10:41 AM Changeset in webkit [74485] by tonyg@chromium.org
  • 2 edits in trunk/LayoutTests

2010-12-22 Tony Gentilcore <tonyg@chromium.org>

Unreviewed build fix.

[chromium] console-websocket-error sometimes times out
https://bugs.webkit.org/show_bug.cgi?id=51482

  • platform/chromium/test_expectations.txt:
10:36 AM Changeset in webkit [74484] by vangelis@chromium.org
  • 4 edits
    2 adds in trunk

2010-12-21 Vangelis Kokkevis <vangelis@chromium.org>

Reviewed by Kenneth Russell.

[chromium] Fixing crash with empty render surfaces.
https://bugs.webkit.org/show_bug.cgi?id=51432

Test: platform/chromium/compositing/empty-render-surface-crasher.html

  • platform/graphics/chromium/RenderSurfaceChromium.cpp: (WebCore::RenderSurfaceChromium::draw):

2010-12-21 Vangelis Kokkevis <vangelis@chromium.org>

Reviewed by Kenneth Russell.

[chromium] Enabling layout tests in platform/chromium/compositing
and adding new layout test to check for crash on empty render surfaces.
https://bugs.webkit.org/show_bug.cgi?id=51432

  • platform/chromium-gpu/test_expectations.txt:
  • platform/chromium/compositing/empty-render-surface-crasher-expected.txt: Added.
  • platform/chromium/compositing/empty-render-surface-crasher.html: Added.
10:16 AM Changeset in webkit [74483] by scherkus@chromium.org
  • 2 edits in trunk/Tools

2010-12-22 Andrew Scherkus <scherkus@chromium.org>

Unreviewed. Adding myself to committers.py.

  • Scripts/webkitpy/common/config/committers.py:
10:12 AM Changeset in webkit [74482] by ariya@webkit.org
  • 2 edits in trunk/Tools

2010-12-22 Ariya Hidayat <ariya@sencha.com>

Reviewed by Kenneth Rohde Christiansen.

TestNetscapePlugIn should use #if defined
https://bugs.webkit.org/show_bug.cgi?id=51471

  • DumpRenderTree/TestNetscapePlugIn/main.cpp: (NP_Initialize): (NPP_New):
8:25 AM Changeset in webkit [74481] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

2010-12-22 Krithigassree Sambamurthy <krithigassree.sambamurthy@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] LayoutTests/http/tests/appcache/auth.html failed and skipped
https://bugs.webkit.org/show_bug.cgi?id=36688

  • platform/qt/Skipped: Unskip auth.html which now passes. Using Qt4.7.
8:22 AM Changeset in webkit [74480] by loislo@chromium.org
  • 2 edits in trunk/Tools

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

Reviewed by Yury Semikhatsky.

Web Inspector: [chromium] DRT inspector layout tests are flaky in debug.

DevTools window should be closed explicitly because it has custom deinitialization code.

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

  • DumpRenderTree/chromium/TestShell.cpp: (TestShell::closeRemainingWindows):
7:41 AM Changeset in webkit [74479] by jberlin@webkit.org
  • 1 edit
    1 add in trunk/LayoutTests

Add Windows-specific expected results for fast/js/large-expressions.html in order to get
the Windows 7 Release test bot green again. Unreviewed.
https://bugs.webkit.org/show_bug.cgi?id=51469

  • platform/win/fast/js/large-expressions-expected.txt: Added.
7:31 AM Changeset in webkit [74478] by levin@chromium.org
  • 3 edits in trunk/Tools

check-webkit-style shouldn't complaint about underscores in variables in objective C files.
https://bugs.webkit.org/show_bug.cgi?id=51452

Reviewed by Shinichiro Hamaji.

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

(_FileState.init): Added the information to determine if a file is C or Objective C.
Using the file extension if possible but falling back to the file contents if we have a header file.
(_FileState.is_objective_c): Determine if we have an Objective C by examining the file contents if needed.
(_FileState.is_c_or_objective_c):
(check_using_std): Changed to using _FileState to determine the file type.
(check_max_min_macros): Ditto.
(check_for_null): Ditto.
(check_style): Changed the parameters to various calls since they now need _FileState
to determine the file type.
(check_language): Added the file_state parameter so it could be passed
to check_identifier_name_in_declaration.
(check_identifier_name_in_declaration): Don't warn about underscores in variables if
this is an Objective C file.
(_process_lines): Added information for the _FileState constructor (and moved the
call to a place that had the information).

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

(CppFunctionsTest.test_is_c_or_objective_c): Changed the tests to use FileState and exercise
its functionality.
(WebKitStyleTest.test_names): Add tests for underscores in Objective C files.

6:17 AM Changeset in webkit [74477] by podivilov@chromium.org
  • 23 edits in trunk

2010-12-21 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: persist JavaScript breakpoints in frontend settings.
https://bugs.webkit.org/show_bug.cgi?id=48434

  • http/tests/inspector/debugger-test2.js: (initialize_DebuggerTest.InspectorTest.setBreakpoint):
  • inspector/debugger-breakpoints-not-activated-on-reload.html:
  • inspector/debugger-pause-on-breakpoint.html:
  • inspector/report-protocol-errors-expected.txt:
  • inspector/report-protocol-errors.html:

2010-12-21 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: persist JavaScript breakpoints in frontend settings.
https://bugs.webkit.org/show_bug.cgi?id=48434

  • bindings/js/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::setBreakpoint): (WebCore::ScriptDebugServer::removeBreakpoint):
  • bindings/js/ScriptDebugServer.h:
  • bindings/v8/DebuggerScript.js: ():
  • bindings/v8/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::setBreakpoint): (WebCore::ScriptDebugServer::removeBreakpoint):
  • bindings/v8/ScriptDebugServer.h:
  • inspector/Inspector.idl:
  • inspector/InspectorBackend.cpp:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::InspectorController): (WebCore::InspectorController::enableDebuggerFromFrontend): (WebCore::InspectorController::setStickyBreakpoints): (WebCore::InspectorController::restoreStickyBreakpoint):
  • inspector/InspectorController.h:
  • inspector/InspectorDebuggerAgent.cpp: (WebCore::InspectorDebuggerAgent::InspectorDebuggerAgent): (WebCore::InspectorDebuggerAgent::setStickyBreakpoint): (WebCore::InspectorDebuggerAgent::setBreakpoint): (WebCore::InspectorDebuggerAgent::removeBreakpoint): (WebCore::InspectorDebuggerAgent::restoreBreakpoint): (WebCore::InspectorDebuggerAgent::clearForPageNavigation): (WebCore::InspectorDebuggerAgent::didParseSource):
  • inspector/InspectorDebuggerAgent.h:
  • inspector/front-end/Breakpoint.js: (WebInspector.Breakpoint): (WebInspector.Breakpoint.prototype.remove):
  • inspector/front-end/BreakpointManager.js: (WebInspector.BreakpointManager): (WebInspector.DOMBreakpoint.prototype._serializeToJSON): (WebInspector.JavaScriptBreakpoint): (WebInspector.XHRBreakpoint.prototype._serializeToJSON):
  • inspector/front-end/DebuggerModel.js: (WebInspector.DebuggerModel):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype._debuggerPaused):
  • inspector/front-end/SourceFrame.js:
6:00 AM Changeset in webkit [74476] by Antti Koivisto
  • 7 edits
    3 adds in trunk

WebCore: https://bugs.webkit.org/show_bug.cgi?id=51354
Don't block rendering and script execution on deferred stylesheets

Reviewed by Alexey Proskuryakov.

  • Don't add low priority stylesheets to the document pending sheet count.
  • Resolve media attribute fully for the link element stylesheet load.

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

  • html/HTMLLinkElement.cpp:

(WebCore::HTMLLinkElement::HTMLLinkElement):
(WebCore::HTMLLinkElement::~HTMLLinkElement):
(WebCore::HTMLLinkElement::setDisabledState):
(WebCore::HTMLLinkElement::process):
(WebCore::HTMLLinkElement::sheetLoaded):
(WebCore::HTMLLinkElement::addPendingSheet):
(WebCore::HTMLLinkElement::removePendingSheet):

  • html/HTMLLinkElement.h:

LayoutTests: https://bugs.webkit.org/show_bug.cgi?id=51354
Don't block rendering and script execution on deferred stylesheets

Reviewed by Alexey Proskuryakov.

  • http/tests/local/link-stylesheet-load-order-expected.txt:
  • http/tests/local/link-stylesheet-load-order-preload-expected.txt:
  • http/tests/local/resources/check-stylesheet-not-loaded.js: Added.
  • http/tests/local/stylesheet-and-script-load-order-media-print-expected.txt: Added.
  • http/tests/local/stylesheet-and-script-load-order-media-print.html: Added.
5:34 AM Changeset in webkit [74475] by podivilov@chromium.org
  • 23 edits in trunk

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

Unreviewed, rolling out r74473.
http://trac.webkit.org/changeset/74473
https://bugs.webkit.org/show_bug.cgi?id=51463

broke compilation on leopard intel release (Requested by
podivilov on #webkit).

  • http/tests/inspector/debugger-test2.js:
  • inspector/debugger-breakpoints-not-activated-on-reload.html:
  • inspector/debugger-pause-on-breakpoint.html:
  • inspector/report-protocol-errors-expected.txt:
  • inspector/report-protocol-errors.html:

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

Unreviewed, rolling out r74473.
http://trac.webkit.org/changeset/74473
https://bugs.webkit.org/show_bug.cgi?id=51463

broke compilation on leopard intel release (Requested by
podivilov on #webkit).

  • bindings/js/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::setBreakpoint): (WebCore::ScriptDebugServer::removeBreakpoint):
  • bindings/js/ScriptDebugServer.h:
  • bindings/v8/DebuggerScript.js: ():
  • bindings/v8/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::setBreakpoint): (WebCore::ScriptDebugServer::removeBreakpoint):
  • bindings/v8/ScriptDebugServer.h:
  • inspector/Inspector.idl:
  • inspector/InspectorBackend.cpp:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::InspectorController): (WebCore::InspectorController::enableDebuggerFromFrontend): (WebCore::InspectorController::setStickyBreakpoints): (WebCore::InspectorController::restoreStickyBreakpoint): (WebCore::InspectorController::breakpointsSettingKey): (WebCore::InspectorController::loadBreakpoints): (WebCore::InspectorController::saveBreakpoints):
  • inspector/InspectorController.h:
  • inspector/InspectorDebuggerAgent.cpp: (WebCore::formatBreakpointId): (WebCore::InspectorDebuggerAgent::InspectorDebuggerAgent): (WebCore::InspectorDebuggerAgent::setBreakpoint): (WebCore::InspectorDebuggerAgent::removeBreakpoint): (WebCore::InspectorDebuggerAgent::clearForPageNavigation): (WebCore::InspectorDebuggerAgent::md5Base16): (WebCore::InspectorDebuggerAgent::loadBreakpoints): (WebCore::InspectorDebuggerAgent::saveBreakpoints): (WebCore::InspectorDebuggerAgent::didParseSource):
  • inspector/InspectorDebuggerAgent.h:
  • inspector/front-end/Breakpoint.js: (WebInspector.Breakpoint): (WebInspector.Breakpoint.jsBreakpointId): (WebInspector.Breakpoint.prototype.set enabled): (WebInspector.Breakpoint.prototype.get id): (WebInspector.Breakpoint.prototype.set condition): (WebInspector.Breakpoint.prototype.remove):
  • inspector/front-end/BreakpointManager.js: (WebInspector.BreakpointManager): (WebInspector.BreakpointManager.prototype._createDOMBreakpoint): (WebInspector.BreakpointManager.prototype._createEventListenerBreakpoint): (WebInspector.BreakpointManager.prototype._createXHRBreakpoint): (WebInspector.BreakpointManager.prototype._setNativeBreakpoint): (WebInspector.BreakpointManager.prototype._setNativeBreakpointEnabled): (WebInspector.BreakpointManager.prototype._removeNativeBreakpoint): (WebInspector.BreakpointManager.prototype.breakpointViewForEventData): (WebInspector.BreakpointManager.prototype._projectChanged): (WebInspector.BreakpointManager.prototype.restoreDOMBreakpoints): (WebInspector.BreakpointManager.prototype._saveBreakpoints): (WebInspector.BreakpointManager.prototype._validateBreakpoints): (WebInspector.DOMBreakpoint.prototype._serializeToJSON): (WebInspector.EventListenerBreakpoint.prototype._serializeToJSON): (WebInspector.XHRBreakpoint.prototype._serializeToJSON): (WebInspector.NativeBreakpointView.prototype.set enabled): (WebInspector.NativeBreakpointView.prototype.remove):
  • inspector/front-end/DebuggerModel.js: (WebInspector.DebuggerModel): (WebInspector.DebuggerModel.prototype.setOneTimeBreakpoint): (WebInspector.DebuggerModel.prototype.removeOneTimeBreakpoint): (WebInspector.DebuggerModel.prototype.setBreakpoint): (WebInspector.DebuggerModel.prototype.findBreakpoint): (WebInspector.DebuggerModel.prototype.reset): (WebInspector.DebuggerModel.prototype._setBreakpoint): (WebInspector.DebuggerModel.prototype._breakpointRemoved): (WebInspector.DebuggerModel.prototype._setBreakpointOnBackend): (WebInspector.DebuggerModel.prototype.breakpointRestored): (WebInspector.DebuggerModel.prototype.pausedScript): (WebInspector.DebuggerModel.prototype.resumedScript):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype.continueToLine): (WebInspector.ScriptsPanel.prototype._debuggerPaused):
  • inspector/front-end/SourceFrame.js: (WebInspector.SourceFrame.prototype._contextMenu.addConditionalBreakpoint.didEditBreakpointCondition): (WebInspector.SourceFrame.prototype._contextMenu.addConditionalBreakpoint): (WebInspector.SourceFrame.prototype._contextMenu.else.editBreakpointCondition.didEditBreakpointCondition): (WebInspector.SourceFrame.prototype._contextMenu.else.editBreakpointCondition): (WebInspector.SourceFrame.prototype._contextMenu): (WebInspector.SourceFrame.prototype._mouseDown): (WebInspector.SourceFrame.prototype._continueToLine): (WebInspector.SourceFrame.prototype._setBreakpoint):
5:23 AM Changeset in webkit [74474] by andreip@google.com
  • 3 edits in trunk/JavaScriptCore

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

Unreviewed build fix.

Fix Chromium Linux shared library build.
[Chromium] r74431 broke the Chromium Linux shared library build
https://bugs.webkit.org/show_bug.cgi?id=51462

5:13 AM Changeset in webkit [74473] by podivilov@chromium.org
  • 23 edits in trunk

2010-12-21 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: persist JavaScript breakpoints in frontend settings.
https://bugs.webkit.org/show_bug.cgi?id=48434

  • http/tests/inspector/debugger-test2.js: (initialize_DebuggerTest.InspectorTest.setBreakpoint):
  • inspector/debugger-breakpoints-not-activated-on-reload.html:
  • inspector/debugger-pause-on-breakpoint.html:
  • inspector/report-protocol-errors-expected.txt:
  • inspector/report-protocol-errors.html:

2010-12-21 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: persist JavaScript breakpoints in frontend settings.
https://bugs.webkit.org/show_bug.cgi?id=48434

  • bindings/js/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::setBreakpoint): (WebCore::ScriptDebugServer::removeBreakpoint):
  • bindings/js/ScriptDebugServer.h:
  • bindings/v8/DebuggerScript.js: ():
  • bindings/v8/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::setBreakpoint): (WebCore::ScriptDebugServer::removeBreakpoint):
  • bindings/v8/ScriptDebugServer.h:
  • inspector/Inspector.idl:
  • inspector/InspectorBackend.cpp:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::InspectorController): (WebCore::InspectorController::enableDebuggerFromFrontend): (WebCore::InspectorController::setStickyBreakpoints): (WebCore::InspectorController::restoreStickyBreakpoint):
  • inspector/InspectorController.h:
  • inspector/InspectorDebuggerAgent.cpp: (WebCore::InspectorDebuggerAgent::InspectorDebuggerAgent): (WebCore::InspectorDebuggerAgent::setStickyBreakpoint): (WebCore::InspectorDebuggerAgent::setBreakpoint): (WebCore::InspectorDebuggerAgent::removeBreakpoint): (WebCore::InspectorDebuggerAgent::restoreBreakpoint): (WebCore::InspectorDebuggerAgent::clearForPageNavigation): (WebCore::InspectorDebuggerAgent::didParseSource):
  • inspector/InspectorDebuggerAgent.h:
  • inspector/front-end/Breakpoint.js: (WebInspector.Breakpoint): (WebInspector.Breakpoint.prototype.remove):
  • inspector/front-end/BreakpointManager.js: (WebInspector.BreakpointManager): (WebInspector.DOMBreakpoint.prototype._serializeToJSON): (WebInspector.JavaScriptBreakpoint): (WebInspector.XHRBreakpoint.prototype._serializeToJSON):
  • inspector/front-end/DebuggerModel.js: (WebInspector.DebuggerModel):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype._debuggerPaused):
  • inspector/front-end/SourceFrame.js:
4:25 AM Changeset in webkit [74472] by krit@webkit.org
  • 30 edits in trunk/WebCore

2010-12-22 Dirk Schulze <krit@wbekit.org>

Reviewed by Nikolas Zimmermann.

Cleanup SVG code according to the webkit style rules 1
https://bugs.webkit.org/show_bug.cgi?id=51411

First patch to fix indention and other style issues according to the WebKit style rules in the SVG code.

No change of functionality.

  • svg/ColorDistance.cpp: (WebCore::ColorDistance::isZero):
  • svg/ColorDistance.h:
  • svg/ElementTimeControl.h: (WebCore::ElementTimeControl::~ElementTimeControl):
  • svg/LinearGradientAttributes.h: (WebCore::LinearGradientAttributes::LinearGradientAttributes): (WebCore::LinearGradientAttributes::x1): (WebCore::LinearGradientAttributes::y1): (WebCore::LinearGradientAttributes::x2): (WebCore::LinearGradientAttributes::y2): (WebCore::LinearGradientAttributes::setX1): (WebCore::LinearGradientAttributes::setY1): (WebCore::LinearGradientAttributes::setX2): (WebCore::LinearGradientAttributes::setY2): (WebCore::LinearGradientAttributes::hasX1): (WebCore::LinearGradientAttributes::hasY1): (WebCore::LinearGradientAttributes::hasX2): (WebCore::LinearGradientAttributes::hasY2):
  • svg/RadialGradientAttributes.h: (WebCore::RadialGradientAttributes::RadialGradientAttributes): (WebCore::RadialGradientAttributes::cx): (WebCore::RadialGradientAttributes::cy): (WebCore::RadialGradientAttributes::r): (WebCore::RadialGradientAttributes::fx): (WebCore::RadialGradientAttributes::fy): (WebCore::RadialGradientAttributes::setCx): (WebCore::RadialGradientAttributes::setCy): (WebCore::RadialGradientAttributes::setR): (WebCore::RadialGradientAttributes::setFx): (WebCore::RadialGradientAttributes::setFy): (WebCore::RadialGradientAttributes::hasCx): (WebCore::RadialGradientAttributes::hasCy): (WebCore::RadialGradientAttributes::hasR): (WebCore::RadialGradientAttributes::hasFx): (WebCore::RadialGradientAttributes::hasFy):
  • svg/SVGAngle.cpp:
  • svg/SVGAnimateColorElement.h:
  • svg/SVGAnimateElement.cpp: (WebCore::SVGAnimateElement::calculateAnimatedValue): (WebCore::SVGAnimateElement::calculateDistance):
  • svg/SVGAnimateElement.h:
  • svg/SVGAnimateMotionElement.cpp:
  • svg/SVGAnimateMotionElement.h:
  • svg/SVGAnimatedNumber.h:
  • svg/SVGAnimationElement.cpp: (WebCore::parseKeyTimes): (WebCore::SVGAnimationElement::currentValuesForValuesAnimation): (WebCore::SVGAnimationElement::updateAnimation):
  • svg/SVGClipPathElement.cpp: (WebCore::SVGClipPathElement::svgAttributeChanged):
  • svg/SVGColor.h: (WebCore::SVGColor::create): (WebCore::SVGColor::createCurrentColor): (WebCore::SVGColor::colorType): (WebCore::SVGColor::setRGBColor): (WebCore::SVGColor::color): (WebCore::SVGColor::isSVGColor):
  • svg/SVGCursorElement.cpp: (WebCore::SVGCursorElement::svgAttributeChanged):
  • svg/SVGCursorElement.h:
  • svg/SVGDescElement.h: (WebCore::SVGDescElement::rendererIsNeeded):
  • svg/SVGDocument.cpp:
  • svg/SVGDocument.h: (WebCore::SVGDocument::create): (WebCore::SVGDocument::isSVGDocument):
  • svg/SVGElement.h: (WebCore::SVGElement::isStyled): (WebCore::SVGElement::isStyledTransformable): (WebCore::SVGElement::isStyledLocatable): (WebCore::SVGElement::isSVG): (WebCore::SVGElement::isFilterEffect): (WebCore::SVGElement::isGradientStop): (WebCore::SVGElement::isTextContent): (WebCore::SVGElement::isValid): (WebCore::SVGElement::svgAttributeChanged): (WebCore::SVGElement::synchronizeProperty): (WebCore::SVGElement::supplementalTransform): (WebCore::SVGElement::invalidateSVGAttributes): (WebCore::SVGElement::rendererIsNeeded): (WebCore::SVGElement::needsPendingResourceHandling): (WebCore::SVGElement::buildPendingResource):
  • svg/SVGElementInstance.h: (WebCore::SVGElementInstance::create): (WebCore::SVGElementInstance::correspondingElement): (WebCore::SVGElementInstance::correspondingUseElement): (WebCore::SVGElementInstance::shadowTreeElement): (WebCore::SVGElementInstance::parentNode): (WebCore::SVGElementInstance::previousSibling): (WebCore::SVGElementInstance::nextSibling): (WebCore::SVGElementInstance::firstChild): (WebCore::SVGElementInstance::lastChild): (WebCore::SVGElementInstance::ownerDocument): (WebCore::SVGElementInstance::toNode): (WebCore::SVGElementInstance::toSVGElementInstance): (WebCore::SVGElementInstance::hasChildNodes): (WebCore::SVGElementInstance::setFirstChild): (WebCore::SVGElementInstance::setLastChild): (WebCore::SVGElementInstance::setNextSibling): (WebCore::SVGElementInstance::setPreviousSibling): (WebCore::SVGElementInstance::refEventTarget): (WebCore::SVGElementInstance::derefEventTarget):
  • svg/SVGElementInstanceList.h: (WebCore::SVGElementInstanceList::create):
  • svg/SVGException.h: (WebCore::SVGException::create): (WebCore::SVGException::SVGException):
  • svg/SVGFEColorMatrixElement.cpp: (WebCore::SVGFEColorMatrixElement::parseMappedAttribute):
  • svg/SVGFEComponentTransferElement.cpp: (WebCore::SVGFEComponentTransferElement::build):
  • svg/SVGFEDiffuseLightingElement.cpp: (WebCore::SVGFEDiffuseLightingElement::findLights):
  • svg/SVGFont.cpp: (WebCore::convertEmUnitToPixel):
  • svg/SVGUseElement.cpp: (WebCore::isDirectReference):
4:17 AM Changeset in webkit [74471] by andreip@google.com
  • 3 edits in trunk/LayoutTests

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

Unreviewed buildfix.

[v8] trunk/LayoutTests/fast/js/large-expressions.html fails since r74402
https://bugs.webkit.org/show_bug.cgi?id=51391

  • platform/chromium-mac/fast/js/large-expressions-expected.txt:
  • platform/chromium-win/fast/js/large-expressions-expected.txt:
3:38 AM Changeset in webkit [74470] by commit-queue@webkit.org
  • 7 edits in trunk

2010-12-22 Dai Mikurube <dmikurube@google.com>

Reviewed by Kent Tamura.

https://bugs.webkit.org/show_bug.cgi?id=51289
Initial state of checkValidity() and :invalid are incorrect with <select required>

  • fast/css/pseudo-valid-007-expected.txt:
  • fast/css/pseudo-valid-007.html:
  • fast/forms/checkValidity-002-expected.txt:
  • fast/forms/checkValidity-002.html:

2010-12-22 Dai Mikurube <dmikurube@google.com>

Reviewed by Kent Tamura.

https://bugs.webkit.org/show_bug.cgi?id=51289
Initial state of checkValidity() and :invalid are incorrect with <select required>

  • html/HTMLSelectElement.cpp: (WebCore::HTMLSelectElement::childrenChanged): setNeedsValidityCheck() after <select>'s children change.
3:24 AM Changeset in webkit [74469] by kov@webkit.org
  • 1 copy in releases/WebKitGTK/webkit-1.3.8

Tagging 1.3.8.

3:05 AM Changeset in webkit [74468] by apavlov@chromium.org
  • 5 edits in trunk

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

Reviewed by Joseph Pecoraro.

Web Inspector: Styles pane fails to display properties from a broken CSS rule correctly
https://bugs.webkit.org/show_bug.cgi?id=51405

  • inspector/styles-new-API-expected.txt:
  • inspector/styles-new-API.html:

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

Reviewed by Joseph Pecoraro.

Web Inspector: Styles pane fails to display properties from a broken CSS rule correctly
https://bugs.webkit.org/show_bug.cgi?id=51405

Invoke markPropertyStart() after the parser has restored after an error.

  • css/CSSGrammar.y:
12:50 AM Changeset in webkit [74467] by gyuyoung.kim@samsung.com
  • 7 edits in trunk

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

Reviewed by Antonio Gomes.

[EFL] Sets default user agent
https://bugs.webkit.org/show_bug.cgi?id=47903

Add macros for WebKit EFL and User Agent.

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

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

Reviewed by Antonio Gomes.

[EFL] Sets default user agent
https://bugs.webkit.org/show_bug.cgi?id=47903

If applcation(e.g EWebLauncher) doesn't set user agent, WebKit EFL doesn't
set user agent. However, we need to set user agent by default. Because, some
web sites send different pages according to user agent.

  • ewk/ewk_private.h:
  • ewk/ewk_settings.cpp: (_ewk_settings_webkit_platform): (_ewk_settings_webkit_os_version): (ewk_settings_default_user_agent_get):
  • ewk/ewk_view.cpp: (_ewk_view_priv_new):
12:37 AM Changeset in webkit [74466] by commit-queue@webkit.org
  • 5 edits in trunk

2010-12-22 Dai Mikurube <dmikurube@google.com>

Reviewed by Kent Tamura.

Assertion failure with <select multiple required>
https://bugs.webkit.org/show_bug.cgi?id=51260

  • fast/forms/resources/select-live-pseudo-selectors.js:
  • fast/forms/select-live-pseudo-selectors-expected.txt:

2010-12-22 Dai Mikurube <dmikurube@google.com>

Reviewed by Kent Tamura.

Assertion failure with <select multiple required>
https://bugs.webkit.org/show_bug.cgi?id=51260

Test: fast/css/pseudo-valid-select-click.html

  • dom/SelectElement.cpp: updateValidity() after selecting some options of ListBox-typed <select> elements to update valid() cache. (WebCore::SelectElement::updateListBoxSelection):
Note: See TracTimeline for information about the timeline view.