alexis.menard@openbossa.org [Wed, 27 Jul 2011 14:20:07 +0000 (14:20 +0000)]
[Qt] Unreviewed build fix for mac.
QtKit is now our default media player, the define is WTF_USE_QTKIT and it's not part
of the options passed to build-webkit, therefore DerivedSources should now generate the
includes when it's mac.
* DerivedSources.pro:
* WebCore.pro:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91843
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ossy@webkit.org [Wed, 27 Jul 2011 14:12:12 +0000 (14:12 +0000)]
[Qt] Couple of tests have different results on 64 bit and/or in debug mode compared to 32 bit and/or release mode
https://bugs.webkit.org/show_bug.cgi?id=52810
* platform/qt/Skipped: Skip a new failing test: svg/custom/image-with-transform-clip-filter.svg
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91842
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
tonyg@chromium.org [Wed, 27 Jul 2011 13:56:16 +0000 (13:56 +0000)]
Speculative win build fix after r91835
https://bugs.webkit.org/show_bug.cgi?id=65255
Unreviewed build fix.
* src/WebFontImpl.cpp:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91841
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Wed, 27 Jul 2011 13:55:06 +0000 (13:55 +0000)]
[Qt][WK2] Clean WKPageLoaderClient
https://bugs.webkit.org/show_bug.cgi?id=65246
Patch by Benjamin Poulain <benjamin@webkit.org> on 2011-07-27
Reviewed by Andreas Kling.
Remove the empty functions from Qt's WKPageLoaderClient.
* UIProcess/qt/ClientImpl.cpp:
(qt_wk_didSameDocumentNavigationForFrame): Move the function to
have all the implementations in the same order as the definitions.
* UIProcess/qt/ClientImpl.h:
* UIProcess/qt/QtWebPageProxy.cpp:
(QtWebPageProxy::init):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91840
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
yurys@chromium.org [Wed, 27 Jul 2011 13:37:03 +0000 (13:37 +0000)]
Inspector should support cd(window) in the command line
https://bugs.webkit.org/show_bug.cgi?id=19872
Source/WebCore:
Added dropdown list with all iframes to allow selecting current evaluation context for console expressions.
Reviewed by Pavel Feldman.
Test: http/tests/inspector/console-cd.html
* WebCore.gypi:
* inspector/Inspector.json:
* inspector/InspectorController.cpp:
(WebCore::InspectorController::InspectorController):
* inspector/InspectorRuntimeAgent.cpp:
(WebCore::InspectorRuntimeAgent::InspectorRuntimeAgent):
(WebCore::InspectorRuntimeAgent::evaluate):
* inspector/InspectorRuntimeAgent.h:
* inspector/WorkerInspectorController.cpp:
* inspector/front-end/ConsoleView.js:
(WebInspector.ConsoleView.prototype.addContext):
(WebInspector.ConsoleView.prototype.removeContext):
(WebInspector.ConsoleView.prototype._contextUpdated):
(WebInspector.ConsoleView.prototype.get _currentEvaluationContextId):
(WebInspector.ConsoleView.prototype.evalInInspectedWindow):
* inspector/front-end/ContextManager.js: Added.
(WebInspector.ContextManager):
(WebInspector.ContextManager.prototype._frameAdded):
(WebInspector.ContextManager.prototype._frameNavigated):
(WebInspector.ContextManager.prototype._frameDetached):
(WebInspector.FrameEvaluationContext):
(WebInspector.FrameEvaluationContext.prototype._frameNavigated):
(WebInspector.FrameEvaluationContext.prototype.get id):
(WebInspector.FrameEvaluationContext.prototype.get url):
(WebInspector.FrameEvaluationContext.prototype.get displayName):
* inspector/front-end/ResourceTreeModel.js:
(WebInspector.ResourceTreeModel.prototype._cleanupFramesAfterNavigation):
(WebInspector.ResourceTreeModel.prototype._addFramesRecursively):
* inspector/front-end/WebKit.qrc:
* inspector/front-end/inspector.html:
* inspector/front-end/inspector.js:
LayoutTests:
Added a test for console evaluation in an iframe.
Reviewed by Pavel Feldman.
* http/tests/inspector/console-cd-expected.txt: Added.
* http/tests/inspector/console-cd.html: Added.
* http/tests/inspector/resources/console-cd-iframe.html: Added.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91839
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
caio.oliveira@openbossa.org [Wed, 27 Jul 2011 13:34:41 +0000 (13:34 +0000)]
Fix Qt (and possibly Gtk) WebKit2 build after addition of -webkit-pictograph
https://bugs.webkit.org/show_bug.cgi?id=65253
Reviewed by Andreas Kling.
The -webkit-pictograph generic family was added by bug 65197.
* Shared/WebPreferencesStore.h:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91838
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Wed, 27 Jul 2011 13:07:42 +0000 (13:07 +0000)]
Source/JavaScriptCore: Added an interface to take IsWhiteSpaceFunctionPtr.
https://bugs.webkit.org/show_bug.cgi?id=57746
Patch by Shinya Kawanaka <shinyak@google.com> on 2011-07-27
Reviewed by Kent Tamura.
* wtf/text/StringImpl.cpp:
(WTF::StringImpl::stripWhiteSpace):
Added an interface to take IsWhiteSpaceFunctionPtr.
(WTF::StringImpl::simplifyWhiteSpace): ditto.
* wtf/text/StringImpl.h:
* wtf/text/WTFString.cpp:
(WTF::String::stripWhiteSpace): ditto.
(WTF::String::simplifyWhiteSpace): ditto.
* wtf/text/WTFString.h:
Source/WebCore: Uses isHTMLSpace to normalize white spaces.
https://bugs.webkit.org/show_bug.cgi?id=57746
Patch by Shinya Kawanaka <shinyak@google.com> on 2011-07-27
Reviewed by Kent Tamura.
Test: fast/forms/option-strip-unicode-spaces.html
* dom/OptionElement.cpp:
(WebCore::OptionElement::normalizeText):
Uses isHTMLSpace to trim and replace white spaces.
LayoutTests: Added a test to normalize option.text.
https://bugs.webkit.org/show_bug.cgi?id=57746
Patch by Shinya Kawanaka <shinyak@google.com> on 2011-07-27
Reviewed by Kent Tamura.
* fast/forms/option-strip-unicode-spaces-expected.txt: Added.
* fast/forms/option-strip-unicode-spaces.html: Added.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91837
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ossy@webkit.org [Wed, 27 Jul 2011 13:01:00 +0000 (13:01 +0000)]
Add Qt specific expected result for a new test introduced in r91777.
* platform/qt/fast/css/font-family-pictograph-expected.png: Added.
* platform/qt/fast/css/font-family-pictograph-expected.txt: Added.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91836
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
zecke@webkit.org [Wed, 27 Jul 2011 12:38:37 +0000 (12:38 +0000)]
[chromium] Remove include of skia/ext/platform_canvas.h
https://bugs.webkit.org/show_bug.cgi?id=60852
Reviewed by Hajime Morita.
In https://bugs.webkit.org/show_bug.cgi?id=57563 the usage of the
skia::PlatformCanvas was eliminated in favor of directly using
SkCanvas directly. The include of platform_canvas.h was not removed
leaving the dependency on Chromium in the header files. Remove
that dependency now.
* platform/graphics/skia/PlatformContextSkia.h: Remove platform_canvas.h
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91835
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
philn@webkit.org [Wed, 27 Jul 2011 11:22:58 +0000 (11:22 +0000)]
Unreviewed, forgotten GTK pixel test results for
fast/forms/input-file-re-render.html after r91833.
* platform/gtk/fast/forms/input-file-re-render-expected.png: Added.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91834
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
philn@webkit.org [Wed, 27 Jul 2011 10:51:30 +0000 (10:51 +0000)]
Unreviewed, GTK pixel tests rebaseline after r91759.
* platform/gtk/fast/forms/box-shadow-override-expected.png:
* platform/gtk/fast/forms/file-input-direction-expected.png:
* platform/gtk/fast/forms/file-input-disabled-expected.png:
* platform/gtk/fast/forms/form-element-geometry-expected.png:
* platform/gtk/fast/forms/input-appearance-height-expected.png:
* platform/gtk/fast/forms/input-value-expected.png:
* platform/gtk/test_expectations.txt:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91833
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
benjamin.poulain@nokia.com [Wed, 27 Jul 2011 10:24:08 +0000 (10:24 +0000)]
[Qt][WK2] Simplify the PageUIClient
https://bugs.webkit.org/show_bug.cgi?id=65198
Reviewed by Andreas Kling.
The only function of WKPageLoaderClient with a complete implementation
is qt_wk_setStatusText(). The other functions were what is left from
before the refactoring of QWKPage.
This patch removes the unused functions to simplify the client. Since the dependency
on QtWebPageProxy is removed from that client, it now calls the ViewInterface directly.
* UIProcess/qt/ClientImpl.cpp:
(toViewInterface):
(qt_wk_setStatusText):
* UIProcess/qt/ClientImpl.h:
* UIProcess/qt/QtWebPageProxy.cpp:
(QtWebPageProxy::QtWebPageProxy):
(QtWebPageProxy::init):
* UIProcess/qt/QtWebPageProxy.h:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91831
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Wed, 27 Jul 2011 10:03:00 +0000 (10:03 +0000)]
Unreviewed, rolling out r91792.
http://trac.webkit.org/changeset/91792
https://bugs.webkit.org/show_bug.cgi?id=65242
It broke NRWT on Qt and GTK (Requested by Ossy_night on
#webkit).
Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2011-07-27
* Scripts/webkitpy/layout_tests/models/test_configuration.py:
* Scripts/webkitpy/layout_tests/models/test_configuration_unittest.py:
* Scripts/webkitpy/layout_tests/models/test_expectations.py:
* Scripts/webkitpy/layout_tests/models/test_expectations_unittest.py:
* Scripts/webkitpy/layout_tests/port/chromium_unittest.py:
* Scripts/webkitpy/layout_tests/port/test.py:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91830
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mnaganov@chromium.org [Wed, 27 Jul 2011 10:01:51 +0000 (10:01 +0000)]
Web Inspector: [Chromium] REGRESSION: Uncaught TypeError: Cannot read property 'isTracingToWindowObjects' of undefined
https://bugs.webkit.org/show_bug.cgi?id=65213
Reviewed by Pavel Feldman.
* inspector/front-end/DetailedHeapshotView.js:
(WebInspector.HeapSnapshotRetainingPathsList.prototype.refresh):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91829
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
tonyg@chromium.org [Wed, 27 Jul 2011 09:49:49 +0000 (09:49 +0000)]
editing/selection/move-by-word-visually* tests timing out
https://bugs.webkit.org/show_bug.cgi?id=65241
Unreviewed build fix.
* platform/chromium/test_expectations.txt:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91828
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Wed, 27 Jul 2011 09:15:21 +0000 (09:15 +0000)]
Knob of a disabled or readonly range control should not be draggable.
https://bugs.webkit.org/show_bug.cgi?id=54820
Patch by Shinya Kawanaka <shinyak@google.com> on 2011-07-27
Reviewed by Kent Tamura.
Source/WebCore:
Made disabled or readonly range control undraggable.
Test: fast/forms/range-drag.html
* html/RangeInputType.cpp:
(WebCore::RangeInputType::handleMouseDownEvent):
Added check that the element is readonly or disabled.
* html/shadow/SliderThumbElement.cpp:
(WebCore::SliderThumbElement::defaultEventHandler): ditto.
LayoutTests:
Added tests to confirm disabled or readonly range control is undraggable.
* fast/forms/range-drag-expected.txt: Added.
* fast/forms/range-drag.html: Added.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91827
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
yutak@chromium.org [Wed, 27 Jul 2011 08:56:58 +0000 (08:56 +0000)]
WebSocket: CloseEvent attribute is not available on WebWorkers in JSC
https://bugs.webkit.org/show_bug.cgi?id=65232
Reviewed by Kent Tamura.
Source/WebCore:
Add [NoStaticTables] attribute to CloseEvent interface definition. This attribute
makes the code generator obtain the attributes table used for building ClassInfo
through JSGlobalData, instead of directly using a statically-allocated table.
Generally speaking, NoStaticTables needs to be specified in interfaces which
may be accessed from workers.
Tests: http/tests/websocket/tests/hybi/workers/shared-worker-simple.html (updated)
http/tests/websocket/tests/hybi/workers/worker-simple.html (updated)
* websockets/CloseEvent.idl: Fixed the formatting, too.
LayoutTests:
* http/tests/websocket/tests/hybi/workers/resources/worker-simple.js:
(runTests): Check the value of closeEvent.wasClean.
* http/tests/websocket/tests/hybi/workers/shared-worker-simple-expected.txt:
* http/tests/websocket/tests/hybi/workers/worker-simple-expected.txt:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91826
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Wed, 27 Jul 2011 07:12:38 +0000 (07:12 +0000)]
DFG JIT speculation failure code performs incorrect conversions in
the case where two registers need to be swapped.
https://bugs.webkit.org/show_bug.cgi?id=65233
Patch by Filip Pizlo <fpizlo@apple.com> on 2011-07-27
Reviewed by Gavin Barraclough.
* dfg/DFGJITCompiler.cpp:
(JSC::DFG::GeneralizedRegister::swapWith):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91825
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Wed, 27 Jul 2011 06:02:54 +0000 (06:02 +0000)]
reduce and reduceRight bind callback's this to null rather than undefined
https://bugs.webkit.org/show_bug.cgi?id=62264
Patch by Mark Hahnenberg <mhahnenberg@apple.com> on 2011-07-26
Reviewed by Oliver Hunt.
Source/JavaScriptCore:
Fixed Array.prototype.reduce and Array.prototype.reduceRight so that they behave correctly
when calling the callback function without an argument for this, which means it should
be undefined according to ES 15.4.4.21 and 15.4.4.22.
* runtime/ArrayPrototype.cpp:
(JSC::arrayProtoFuncReduce):
(JSC::arrayProtoFuncReduceRight):
LayoutTests:
Added additional tests to test Array.prototype.reduce and Array.prototype.reduceRight
when calling the callback function without an argument for this, which means it should
be undefined according to ES 15.4.4.21 and 15.4.4.22.
* fast/js/array-reduce-expected.txt:
* fast/js/array-reduceRight-expected.txt:
* fast/js/script-tests/array-reduce.js:
* fast/js/script-tests/array-reduceRight.js:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91824
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Wed, 27 Jul 2011 03:48:27 +0000 (03:48 +0000)]
[chromium] REGRESSION(90963): Content layers are not redrawn after losing the compositor context
https://bugs.webkit.org/show_bug.cgi?id=65224
Patch by James Robinson <jamesr@chromium.org> on 2011-07-26
Reviewed by Kenneth Russell.
There's no automated way to test lost compositor contexts (yet), but to test manually open up the poster circle
page and kill the GPU process.
* platform/graphics/chromium/ContentLayerChromium.cpp:
(WebCore::ContentLayerChromium::cleanupResources):
* platform/graphics/chromium/ContentLayerChromium.h:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91823
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Wed, 27 Jul 2011 03:42:22 +0000 (03:42 +0000)]
Unreviewed, rolling out r91805.
http://trac.webkit.org/changeset/91805
https://bugs.webkit.org/show_bug.cgi?id=65226
Breaks @ symbols and changes other font rendering on Windows
(Requested by enne on #webkit).
Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2011-07-26
Source/WebCore:
* platform/graphics/chromium/FontChromiumWin.cpp:
(WebCore::TransparencyAwareFontPainter::TransparencyAwareFontPainter::TransparencyAwareFontPainter):
(WebCore::TransparencyAwareFontPainter::TransparencyAwareFontPainter::init):
(WebCore::TransparencyAwareFontPainter::TransparencyAwareFontPainter::initializeForGDI):
(WebCore::TransparencyAwareFontPainter::TransparencyAwareFontPainter::~TransparencyAwareFontPainter):
(WebCore::TransparencyAwareFontPainter::TransparencyAwareGlyphPainter::TransparencyAwareGlyphPainter):
(WebCore::TransparencyAwareFontPainter::TransparencyAwareGlyphPainter::~TransparencyAwareGlyphPainter):
(WebCore::TransparencyAwareFontPainter::TransparencyAwareGlyphPainter::drawGlyphs):
(WebCore::TransparencyAwareFontPainter::TransparencyAwareUniscribePainter::hdc):
(WebCore::TransparencyAwareFontPainter::TransparencyAwareUniscribePainter::TransparencyAwareUniscribePainter):
(WebCore::TransparencyAwareFontPainter::TransparencyAwareUniscribePainter::~TransparencyAwareUniscribePainter):
(WebCore::drawGlyphsWin):
(WebCore::Font::drawComplexText):
* platform/graphics/chromium/UniscribeHelper.cpp:
(WebCore::UniscribeHelper::draw):
* platform/graphics/skia/PlatformContextSkia.cpp:
(WebCore::PlatformContextSkia::isNativeFontRenderingAllowed):
* platform/graphics/skia/PlatformContextSkia.h:
* platform/graphics/skia/SkiaFontWin.cpp:
(WebCore::windowsCanHandleDrawTextShadow):
(WebCore::windowsCanHandleTextDrawing):
(WebCore::windowsCanHandleTextDrawingWithoutShadow):
* platform/graphics/skia/SkiaFontWin.h:
LayoutTests:
* platform/chromium/test_expectations.txt:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91822
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
enne@google.com [Wed, 27 Jul 2011 03:18:22 +0000 (03:18 +0000)]
Unreviewed, rolling out r91812.
http://trac.webkit.org/changeset/91812
https://bugs.webkit.org/show_bug.cgi?id=65210
Breaks OSX GPU tests.
* platform/graphics/chromium/LayerRendererChromium.cpp:
(WebCore::LayerRendererChromium::drawLayers):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91821
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
abarth@webkit.org [Wed, 27 Jul 2011 03:05:42 +0000 (03:05 +0000)]
Fix ChangeLog.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91820
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
abarth@webkit.org [Wed, 27 Jul 2011 03:01:10 +0000 (03:01 +0000)]
enne tells me that she already rebaselined this test. The 10.5 bot is
just behind because dglazkov broke NRWT on Python 2.5.
* platform/chromium/test_expectations.txt:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91819
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
abarth@webkit.org [Wed, 27 Jul 2011 02:59:50 +0000 (02:59 +0000)]
Switch to an older idiom that works on Python 2.5.
* Scripts/webkitpy/layout_tests/models/test_configuration.py:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91818
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Wed, 27 Jul 2011 02:56:25 +0000 (02:56 +0000)]
JSC command-line tool does not come with any facility for
measuring time precisely.
https://bugs.webkit.org/show_bug.cgi?id=65223
Patch by Filip Pizlo <fpizlo@apple.com> on 2011-07-26
Reviewed by Gavin Barraclough.
Exposed WTF::currentTime() as currentTimePrecise().
* jsc.cpp:
(GlobalObject::GlobalObject):
(functionPreciseTime):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91817
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Wed, 27 Jul 2011 02:48:04 +0000 (02:48 +0000)]
Unreviewed, rolling out r91809.
http://trac.webkit.org/changeset/91809
https://bugs.webkit.org/show_bug.cgi?id=65225
Does not build on Chromium Windows (Requested by abarth on
#webkit).
Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2011-07-26
Source/WebCore:
* dom/WheelEvent.cpp:
(WebCore::WheelEvent::WheelEvent):
(WebCore::WheelEvent::initWheelEvent):
* dom/WheelEvent.h:
* page/EventHandler.cpp:
(WebCore::EventHandler::EventHandler):
(WebCore::EventHandler::handleGestureEvent):
(WebCore::EventHandler::handleTouchEvent):
(WebCore::EventHandler::resetGestureRecognizer):
* page/EventHandler.h:
* page/FrameView.cpp:
(WebCore::FrameView::reset):
* platform/PlatformGestureEvent.h:
* platform/PlatformGestureRecognizer.h:
* platform/chromium/FramelessScrollView.h:
* platform/chromium/GestureRecognizerChromium.cpp:
(WebCore::InnerGestureRecognizer::InnerGestureRecognizer):
(WebCore::InnerGestureRecognizer::reset):
(WebCore::InnerGestureRecognizer::dispatchSyntheticClick):
(WebCore::InnerGestureRecognizer::processTouchEventForGesture):
(WebCore::InnerGestureRecognizer::scrollViaTouchMotion):
(WebCore::touchDown):
(WebCore::noGesture):
(WebCore::click):
(WebCore::isClickOrScroll):
(WebCore::inScroll):
* platform/chromium/GestureRecognizerChromium.h:
(WebCore::GestureRecognizerChromium::reset):
(WebCore::GestureRecognizerChromium::processTouchEventForGesture):
* platform/chromium/PopupMenuChromium.cpp:
* platform/chromium/PopupMenuChromium.h:
* platform/mac/ScrollAnimatorMac.mm:
(WebCore::ScrollAnimatorMac::handleGestureEvent):
Source/WebKit/chromium:
* features.gypi:
* src/WebFrameImpl.cpp:
(WebKit::WebFrameImpl::createFrameView):
* src/WebPopupMenuImpl.cpp:
(WebKit::WebPopupMenuImpl::WebPopupMenuImpl):
(WebKit::WebPopupMenuImpl::handleInputEvent):
* src/WebPopupMenuImpl.h:
* src/WebViewImpl.cpp:
(WebKit::WebViewImpl::WebViewImpl):
(WebKit::WebViewImpl::touchEvent):
* src/WebViewImpl.h:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91816
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
gyuyoung.kim@samsung.com [Wed, 27 Jul 2011 02:45:32 +0000 (02:45 +0000)]
[EFL] Fix pointer operator coding style in ewk_view.
https://bugs.webkit.org/show_bug.cgi?id=65163
Move '*' operator to variable according to efl coding style. In addition,
add a space between * and variable in type casting.
Reviewed by Kent Tamura.
* ewk/ewk_view.cpp:
(_ewk_view_smart_changed):
(_ewk_view_repaints_resize):
(_ewk_view_repaint_add):
(_ewk_view_repaints_flush):
(_ewk_view_scrolls_resize):
(_ewk_view_scroll_add):
(_ewk_view_scrolls_flush):
(_ewk_view_smart_focus_in):
(_ewk_view_smart_focus_out):
(_ewk_view_smart_mouse_wheel):
(_ewk_view_smart_mouse_down):
(_ewk_view_smart_mouse_up):
(_ewk_view_smart_mouse_move):
(_ewk_view_smart_key_down):
(_ewk_view_smart_key_up):
(_ewk_view_smart_add_console_message):
(_ewk_view_smart_run_javascript_alert):
(_ewk_view_smart_run_javascript_confirm):
(_ewk_view_smart_should_interrupt_javascript):
(_ewk_view_smart_run_javascript_prompt):
(_ewk_view_on_focus_in):
(_ewk_view_on_focus_out):
(_ewk_view_on_mouse_wheel):
(_ewk_view_on_mouse_down):
(_ewk_view_on_mouse_up):
(_ewk_view_on_mouse_move):
(_ewk_view_on_key_down):
(_ewk_view_on_key_up):
(_ewk_view_core_frame_new):
(_ewk_view_priv_new):
(_ewk_view_priv_del):
(_ewk_view_smart_add):
(_ewk_view_smart_del):
(_ewk_view_smart_resize):
(_ewk_view_smart_move):
(_ewk_view_smart_calculate):
(_ewk_view_smart_contents_resize):
(_ewk_view_smart_zoom_set):
(_ewk_view_smart_flush):
(_ewk_view_smart_pre_render_region):
(_ewk_view_smart_pre_render_relative_radius):
(_ewk_view_smart_pre_render_cancel):
(_ewk_view_zoom_animated_mark_stop):
(_ewk_view_zoom_animated_finish):
(_ewk_view_zoom_animated_current):
(_ewk_view_zoom_animator_cb):
(_ewk_view_zoom_animation_start):
(_ewk_view_viewport_attributes_compute):
(ewk_view_base_smart_set):
(ewk_view_fixed_layout_size_set):
(ewk_view_fixed_layout_size_get):
(ewk_view_theme_set):
(ewk_view_theme_get):
(ewk_view_frame_main_get):
(ewk_view_frame_focused_get):
(ewk_view_uri_set):
(ewk_view_uri_get):
(ewk_view_title_get):
(ewk_view_editable_get):
(ewk_view_bg_color_set):
(ewk_view_bg_color_get):
(ewk_view_text_search):
(ewk_view_text_matches_mark):
(ewk_view_text_matches_unmark_all):
(ewk_view_text_matches_highlight_set):
(ewk_view_text_matches_highlight_get):
(ewk_view_editable_set):
(ewk_view_selection_get):
(_ewk_view_editor_command):
(ewk_view_execute_editor_command):
(ewk_view_context_menu_forward_event):
(ewk_view_load_progress_get):
(ewk_view_stop):
(ewk_view_reload):
(ewk_view_reload_full):
(ewk_view_back):
(ewk_view_forward):
(ewk_view_navigate):
(ewk_view_back_possible):
(ewk_view_forward_possible):
(ewk_view_navigate_possible):
(ewk_view_history_enable_get):
(ewk_view_history_enable_set):
(ewk_view_history_get):
(ewk_view_zoom_get):
(ewk_view_zoom_set):
(ewk_view_zoom_weak_smooth_scale_get):
(ewk_view_zoom_weak_smooth_scale_set):
(ewk_view_zoom_weak_set):
(ewk_view_zoom_animated_mark_start):
(ewk_view_zoom_animated_mark_end):
(ewk_view_zoom_animated_mark_current):
(ewk_view_zoom_animated_mark_stop):
(ewk_view_zoom_animated_set):
(ewk_view_zoom_text_only_get):
(ewk_view_zoom_text_only_set):
(ewk_view_pre_render_region):
(ewk_view_pre_render_relative_radius):
(ewk_view_pre_render_cancel):
(ewk_view_setting_user_agent_get):
(ewk_view_setting_user_agent_set):
(ewk_view_setting_user_stylesheet_get):
(ewk_view_setting_user_stylesheet_set):
(ewk_view_setting_auto_load_images_get):
(ewk_view_setting_auto_load_images_set):
(ewk_view_setting_auto_shrink_images_get):
(ewk_view_setting_auto_shrink_images_set):
(ewk_view_setting_enable_auto_resize_window_get):
(ewk_view_setting_enable_auto_resize_window_set):
(ewk_view_setting_enable_scripts_get):
(ewk_view_setting_enable_scripts_set):
(ewk_view_setting_enable_plugins_get):
(ewk_view_setting_enable_plugins_set):
(ewk_view_setting_enable_frame_flattening_get):
(ewk_view_setting_enable_frame_flattening_set):
(ewk_view_setting_scripts_window_open_get):
(ewk_view_setting_scripts_window_open_set):
(ewk_view_setting_resizable_textareas_get):
(ewk_view_setting_resizable_textareas_set):
(ewk_view_setting_private_browsing_get):
(ewk_view_setting_private_browsing_set):
(ewk_view_setting_offline_app_cache_get):
(ewk_view_setting_offline_app_cache_set):
(ewk_view_setting_caret_browsing_get):
(ewk_view_setting_caret_browsing_set):
(ewk_view_setting_encoding_custom_get):
(ewk_view_setting_encoding_custom_set):
(ewk_view_setting_encoding_default_get):
(ewk_view_setting_encoding_default_set):
(ewk_view_setting_encoding_detector_set):
(ewk_view_setting_encoding_detector_get):
(ewk_view_setting_enable_developer_extras_get):
(ewk_view_setting_enable_developer_extras_set):
(ewk_view_setting_font_minimum_size_get):
(ewk_view_setting_font_minimum_size_set):
(ewk_view_setting_font_minimum_logical_size_get):
(ewk_view_setting_font_minimum_logical_size_set):
(ewk_view_setting_font_default_size_get):
(ewk_view_setting_font_default_size_set):
(ewk_view_setting_font_monospace_size_get):
(ewk_view_setting_font_monospace_size_set):
(ewk_view_setting_font_standard_get):
(ewk_view_setting_font_standard_set):
(ewk_view_setting_font_cursive_get):
(ewk_view_setting_font_cursive_set):
(ewk_view_setting_font_fantasy_get):
(ewk_view_setting_font_fantasy_set):
(ewk_view_setting_font_monospace_get):
(ewk_view_setting_font_monospace_set):
(ewk_view_setting_font_serif_get):
(ewk_view_setting_font_serif_set):
(ewk_view_setting_font_sans_serif_get):
(ewk_view_setting_font_sans_serif_set):
(ewk_view_setting_spatial_navigation_get):
(ewk_view_setting_spatial_navigation_set):
(ewk_view_setting_local_storage_get):
(ewk_view_setting_local_storage_set):
(ewk_view_setting_page_cache_get):
(ewk_view_setting_page_cache_set):
(ewk_view_setting_local_storage_database_path_get):
(ewk_view_setting_local_storage_database_path_set):
(ewk_view_smart_data_get):
(ewk_view_repaints_get):
(ewk_view_scroll_requests_get):
(ewk_view_repaint_add):
(ewk_view_layout_if_needed_recursive):
(ewk_view_scrolls_process):
(ewk_view_paint_context_new):
(ewk_view_paint_context_free):
(ewk_view_paint_context_save):
(ewk_view_paint_context_restore):
(ewk_view_paint_context_clip):
(ewk_view_paint_context_paint):
(ewk_view_paint_context_paint_contents):
(ewk_view_paint_context_scale):
(ewk_view_paint_context_translate):
(ewk_view_paint):
(ewk_view_paint_contents):
(ewk_view_ready):
(ewk_view_input_method_state_set):
(ewk_view_title_set):
(ewk_view_uri_changed):
(ewk_view_load_started):
(ewk_view_frame_main_load_started):
(ewk_view_load_provisional):
(ewk_view_load_show):
(ewk_view_frame_main_cleared):
(ewk_view_frame_main_icon_received):
(ewk_view_load_finished):
(ewk_view_load_error):
(ewk_view_load_progress_changed):
(ewk_view_restore_state):
(ewk_view_window_create):
(ewk_view_window_close):
(ewk_view_mouse_link_hover_in):
(ewk_view_mouse_link_hover_out):
(ewk_view_toolbars_visible_set):
(ewk_view_toolbars_visible_get):
(ewk_view_statusbar_visible_set):
(ewk_view_statusbar_visible_get):
(ewk_view_statusbar_text_set):
(ewk_view_scrollbars_visible_set):
(ewk_view_scrollbars_visible_get):
(ewk_view_menubar_visible_set):
(ewk_view_menubar_visible_get):
(ewk_view_tooltip_text_set):
(ewk_view_add_console_message):
(ewk_view_run_javascript_alert):
(ewk_view_run_javascript_confirm):
(ewk_view_run_javascript_prompt):
(ewk_view_should_interrupt_javascript):
(ewk_view_exceeded_database_quota):
(ewk_view_run_open_panel):
(ewk_view_repaint):
(ewk_view_scroll):
(ewk_view_core_page_get):
(ewk_view_frame_create):
(ewk_view_plugin_create):
(ewk_view_popup_new):
(ewk_view_popup_destroy):
(ewk_view_popup_selected_set):
(ewk_view_download_request):
(ewk_view_viewport_attributes_set):
(ewk_view_viewport_attributes_get):
(ewk_view_zoom_range_set):
(ewk_view_zoom_range_min_get):
(ewk_view_zoom_range_max_get):
(ewk_view_user_scalable_set):
(ewk_view_user_scalable_get):
(ewk_view_device_pixel_ratio_get):
(ewk_view_navigation_policy_decision):
(ewk_view_page_rect_get):
(ewk_view_need_touch_events_set):
(ewk_view_need_touch_events_get):
(ewk_view_mode_set):
(ewk_view_mode_get):
(ewk_view_editor_client_selection_changed):
* ewk/ewk_view.h:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91815
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
dslomov@google.com [Wed, 27 Jul 2011 02:41:13 +0000 (02:41 +0000)]
[V8][Chromium] Run workers in a separate v8::Isolate
https://bugs.webkit.org/show_bug.cgi?id=65004
This patch allocates a new v8::Isolate for every worker and enters it on worker thread.
Reviewed by David Levin.
Covered by existing chromium tests.
* bindings/v8/DOMDataStore.cpp:
(WebCore::DOMDataStore::DOMDataStore):
(WebCore::DOMDataStore::~DOMDataStore):
* bindings/v8/StaticDOMDataStore.cpp:
(WebCore::StaticDOMDataStore::StaticDOMDataStore):
(WebCore::StaticDOMDataStore::~StaticDOMDataStore):
* bindings/v8/StaticDOMDataStore.h:
* bindings/v8/V8Binding.h:
(WebCore::V8BindingPerIsolateData::registerDOMDataStore):
(WebCore::V8BindingPerIsolateData::unregisterDOMDataStore):
* bindings/v8/V8DOMMap.cpp:
(WebCore::DOMDataStoreHandle::DOMDataStoreHandle):
(WebCore::DOMDataStoreHandle::~DOMDataStoreHandle):
* bindings/v8/WorkerContextExecutionProxy.cpp:
(WebCore::WorkerContextExecutionProxy::WorkerContextExecutionProxy):
(WebCore::WorkerContextExecutionProxy::initV8):
* bindings/v8/WorkerContextExecutionProxy.h:
* bindings/v8/WorkerScriptController.cpp:
(WebCore::WorkerScriptController::WorkerScriptController):
(WebCore::WorkerScriptController::~WorkerScriptController):
* bindings/v8/WorkerScriptController.h:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91814
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
abarth@webkit.org [Wed, 27 Jul 2011 02:37:03 +0000 (02:37 +0000)]
This test appears to fail on Leopard only.
* platform/chromium/test_expectations.txt:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91813
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
jamesr@google.com [Wed, 27 Jul 2011 02:28:07 +0000 (02:28 +0000)]
[chromium] Avoid clearing the framebuffer when compositing in release builds
https://bugs.webkit.org/show_bug.cgi?id=65210
Reviewed by Kenneth Russell.
We clear the default framebuffer to blue when compositing so it's
easier to spot regions we miss. This is useful for debugging, but eats
bandwidth unnecessarily in release builds.
* platform/graphics/chromium/LayerRendererChromium.cpp:
(WebCore::LayerRendererChromium::drawLayers):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91812
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
jpfau@apple.com [Wed, 27 Jul 2011 02:25:44 +0000 (02:25 +0000)]
New XML tokenizer
https://bugs.webkit.org/show_bug.cgi?id=64566
Reviewed by Adam Barth.
Add tokenizer for XML and tests in NewXMLDocumentParser for printing results of tokenization.
* GNUmakefile.list.am:
* WebCore.gypi:
* WebCore.pro:
* WebCore.vcproj/WebCore.vcproj:
* WebCore.xcodeproj/project.pbxproj:
* xml/parser/NewXMLDocumentParser.cpp:
(WebCore::NewXMLDocumentParser::NewXMLDocumentParser):
(WebCore::NewXMLDocumentParser::append):
* xml/parser/NewXMLDocumentParser.h:
* xml/parser/XMLToken.h:
* xml/parser/XMLTokenizer.cpp: Added.
(WebCore::XMLTokenizer::XMLTokenizer):
(WebCore::::shouldSkipNullCharacters):
(WebCore::XMLTokenizer::nextToken):
(WebCore::XMLTokenizer::bufferCharacter):
(WebCore::XMLTokenizer::parseError):
* xml/parser/XMLTokenizer.h: Added.
(WebCore::XMLTokenizer::create):
(WebCore::XMLTokenizer::gotError):
(WebCore::XMLTokenizer::shouldSkipNullCharacters):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91811
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
gyuyoung.kim@samsung.com [Wed, 27 Jul 2011 02:02:38 +0000 (02:02 +0000)]
Add an exception to coding style check for EFL port.
https://bugs.webkit.org/show_bug.cgi?id=65162
Add an exception of whitespace/declaration for EFL port. Because,
'*' operator is next to variable in EFL coding style.
Reviewed by Kent Tamura.
* Scripts/webkitpy/style/checker.py:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91810
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Wed, 27 Jul 2011 01:52:19 +0000 (01:52 +0000)]
[chromium] Layering violations in gesture recognizer
https://bugs.webkit.org/show_bug.cgi?id=65044
Patch by Robert Kroeger <rjkroege@chromium.org> on 2011-07-26
Reviewed by Adam Barth.
Source/WebCore:
Divided the gesture recognizer up to correct a layering
violation by moving gesture implementation from it to
EventHandler::handleGestureEvent so that the gesture recognizer
could simply be an engine for generating gesture events from
touch events.
* page/EventHandler.cpp:
(WebCore::EventHandler::EventHandler):
Removed construction of gesture recognizer.
(WebCore::EventHandler::handleGestureEvent):
Extended for new gesture events added in this patch.
(WebCore::EventHandler::handleTouchEvent):
Removed call into gesture recognizer.
* page/EventHandler.h:
* page/FrameView.cpp:
(WebCore::FrameView::reset):
Removed code to reset gesture recognizer.
* platform/PlatformGestureEvent.h:
(WebCore::PlatformGestureEvent::PlatformGestureEvent):
(WebCore::PlatformGestureEvent::deltaX):
(WebCore::PlatformGestureEvent::deltaY):
(WebCore::PlatformGestureEvent::shiftKey):
(WebCore::PlatformGestureEvent::ctrlKey):
(WebCore::PlatformGestureEvent::altKey):
(WebCore::PlatformGestureEvent::metaKey):
Added additional gesture types and parameters.
* platform/PlatformGestureRecognizer.h:
* platform/chromium/FramelessScrollView.h:
Added support for displatching gesture and touch events.
* platform/chromium/GestureRecognizerChromium.cpp:
(WebCore::InnerGestureRecognizer::InnerGestureRecognizer):
(WebCore::InnerGestureRecognizer::reset):
(WebCore::InnerGestureRecognizer::constructClickGestureEvent):
(WebCore::InnerGestureRecognizer::processTouchEventForGestures):
Removed all depedencies on EventHandler. Instead, create gesture
events from touch event stream.
(WebCore::InnerGestureRecognizer::constructScrollGesture):
(WebCore::touchDown):
(WebCore::noGesture):
(WebCore::click):
(WebCore::isClickOrScroll):
(WebCore::inScroll):
(WebCore::GestureRecognizerChromium::processTouchEventForGestures):
(WebCore::GestureRecognizerChromium::reset):
* platform/chromium/GestureRecognizerChromium.h:
* platform/chromium/PopupMenuChromium.cpp:
Added support to handle touch events and gesture events.
(WebCore::PopupContainer::handleTouchEvent):
(WebCore::PopupContainer::handleGestureEvent):
(WebCore::PopupListBox::handleTouchEvent):
(WebCore::PopupListBox::handleGestureEvent):
* platform/chromium/PopupMenuChromium.h:
* platform/mac/ScrollAnimatorMac.mm:
(WebCore::ScrollAnimatorMac::handleGestureEvent):
Ignores the newly added gesture event types.
Source/WebKit/chromium:
Moved gesture recognition higher up the event delivery code
path so that recognition of gestures is completely separated
from gesture handling.
* features.gypi:
* src/WebFrameImpl.cpp:
(WebKit::WebFrameImpl::createFrameView):
Added code to reset the gesture recognizer to preserve parallel
layout test execution.
* src/WebPopupMenuImpl.cpp:
Support dispatching touch and gesture events to Popups.
(WebKit::WebPopupMenuImpl::WebPopupMenuImpl):
(WebKit::WebPopupMenuImpl::TouchEvent):
(WebKit::WebPopupMenuImpl::handleInputEvent):
* src/WebPopupMenuImpl.h:
* src/WebViewImpl.cpp:
(WebKit::WebViewImpl::WebViewImpl):
Added gesture recognition creation.
(WebKit::WebViewImpl::touchEvent):
Added recognition of gestures and dispatch of gesture events.
(WebKit::WebViewImpl::resetGestureRecognizer):
* src/WebViewImpl.h:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91809
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
joepeck@webkit.org [Wed, 27 Jul 2011 01:38:30 +0000 (01:38 +0000)]
Restore Composited Layers Faster from PageCache
https://bugs.webkit.org/show_bug.cgi?id=65216
Reviewed by Simon Fraser.
Restore the layer tree hierarchy a little sooner, when
restoring from the page cache. This more closely matches
original restore behavior.
Existing test showed no change.
* page/FrameView.cpp:
(WebCore::FrameView::restoreBackingStores):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91808
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Wed, 27 Jul 2011 01:27:43 +0000 (01:27 +0000)]
DFG speculative JIT never emits inline double comparisons, even when it
would be obvious more efficient to do so.
https://bugs.webkit.org/show_bug.cgi?id=65212
Patch by Filip Pizlo <fpizlo@apple.com> on 2011-07-26
Reviewed by Gavin Barraclough.
This handles the obvious case of inlining double comparisons: it only addresses
the speculative JIT, and only for fused compare/branch sequences. But it does
handle the case where both operands are double (and there is no slow path),
or where one operand is double and the other is unknown type (in which case it
attempts to unbox the double, otherwise taking slow path). This is an 0.8%
speed-up on SunSpider.
* dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::SpeculativeJIT::convertToDouble):
(JSC::DFG::SpeculativeJIT::compilePeepHoleDoubleBranch):
(JSC::DFG::SpeculativeJIT::compare):
(JSC::DFG::SpeculativeJIT::compile):
* dfg/DFGSpeculativeJIT.h:
(JSC::DFG::SpeculativeJIT::isRegisterDataFormatDouble):
(JSC::DFG::SpeculativeJIT::shouldSpeculateInteger):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91807
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Wed, 27 Jul 2011 01:12:50 +0000 (01:12 +0000)]
[Qt] Add option to turn on disk caching in QtTestBrowser
https://bugs.webkit.org/show_bug.cgi?id=65007
Patch by Keith Kyzivat <keith.kyzivat@nokia.com> on 2011-07-26
Reviewed by Noam Rosenthal.
Add menu item and command line option to turn on disk caching in
QtTestBrowser. Simple QNetworkDiskCache added to the page's
QNetworkAccssManager.
* QtTestBrowser/launcherwindow.cpp:
(LauncherWindow::initializeView):
(LauncherWindow::createChrome):
(LauncherWindow::setDiskCache):
* QtTestBrowser/launcherwindow.h:
(WindowOptions::WindowOptions):
* QtTestBrowser/main.cpp:
(LauncherApplication::handleUserOptions):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91806
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Wed, 27 Jul 2011 01:10:04 +0000 (01:10 +0000)]
[skia] never draw with GDI, so that all text can be gpu-accelerated
https://bugs.webkit.org/show_bug.cgi?id=65203
Patch by Mike Reed <reed@google.com> on 2011-07-26
Reviewed by Kenneth Russell.
No new tests. existing tests apply to the new drawing path
* platform/graphics/chromium/FontChromiumWin.cpp:
(WebCore::TransparencyAwareFontPainter::TransparencyAwareGlyphPainter::drawGlyphs):
* platform/graphics/skia/SkiaFontWin.cpp:
(WebCore::windowsCanHandleDrawTextShadow):
(WebCore::windowsCanHandleTextDrawing):
(WebCore::windowsCanHandleTextDrawingWithoutShadow):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91805
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Wed, 27 Jul 2011 00:55:33 +0000 (00:55 +0000)]
https://bugs.webkit.org/show_bug.cgi?id=64969
DFG JIT generates inefficient code for speculation failures.
Patch by Filip Pizlo <fpizlo@apple.com> on 2011-07-26
Reviewed by Gavin Barraclough.
This implements a speculation failure strategy where (1) values spilled on
non-speculative but not spilled on speculative are spilled, (2) values that
are in registers on both paths are rearranged without ever touching memory,
and (3) values spilled on speculative but not spilled on non-speculative are
filled.
The register shuffling is the most interesting part of this patch. It
constructs a permutation graph for registers. Each node represents a
register, and each directed edge corresponds to the register's value having
to be moved to a different register as part of the shuffling. This is a
directed graph where each node may only have 0 or 1 incoming edges, and
0 or 1 outgoing edges. The algorithm then first finds maximal non-cyclic
subgraphs where all nodes in the subgraph are reachable from a start node.
Such subgraphs always resemble linked lists, and correspond to simply
moving the value in the second-to-last register into the last register, and
then moving the value in the third-to-last register into the second-to-last
register, and so on. Once these subgraphs are taken care of, the remaining
subgraphs are cycles, and are handled using either (a) conversion or no-op
if the cycle involves one node, (b) swap if it involves two nodes, or (c)
a cyclic shuffle involving a scratch register if there are three or more
nodes.
* dfg/DFGGenerationInfo.h:
(JSC::DFG::needDataFormatConversion):
* dfg/DFGJITCompiler.cpp:
(JSC::DFG::GeneralizedRegister::GeneralizedRegister):
(JSC::DFG::GeneralizedRegister::createGPR):
(JSC::DFG::GeneralizedRegister::createFPR):
(JSC::DFG::GeneralizedRegister::dump):
(JSC::DFG::GeneralizedRegister::findInSpeculationCheck):
(JSC::DFG::GeneralizedRegister::findInEntryLocation):
(JSC::DFG::GeneralizedRegister::previousDataFormat):
(JSC::DFG::GeneralizedRegister::nextDataFormat):
(JSC::DFG::GeneralizedRegister::convert):
(JSC::DFG::GeneralizedRegister::moveTo):
(JSC::DFG::GeneralizedRegister::swapWith):
(JSC::DFG::ShuffledRegister::ShuffledRegister):
(JSC::DFG::ShuffledRegister::isEndOfNonCyclingPermutation):
(JSC::DFG::ShuffledRegister::handleNonCyclingPermutation):
(JSC::DFG::ShuffledRegister::handleCyclingPermutation):
(JSC::DFG::ShuffledRegister::lookup):
(JSC::DFG::lookupForRegister):
(JSC::DFG::NodeToRegisterMap::Tuple::Tuple):
(JSC::DFG::NodeToRegisterMap::NodeToRegisterMap):
(JSC::DFG::NodeToRegisterMap::set):
(JSC::DFG::NodeToRegisterMap::end):
(JSC::DFG::NodeToRegisterMap::find):
(JSC::DFG::NodeToRegisterMap::clear):
(JSC::DFG::JITCompiler::jumpFromSpeculativeToNonSpeculative):
(JSC::DFG::JITCompiler::linkSpeculationChecks):
* dfg/DFGJITCompiler.h:
* dfg/DFGNonSpeculativeJIT.cpp:
(JSC::DFG::EntryLocation::EntryLocation):
* dfg/DFGNonSpeculativeJIT.h:
* dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::SpeculationCheck::SpeculationCheck):
* dfg/DFGSpeculativeJIT.h:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91804
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
kbr@google.com [Wed, 27 Jul 2011 00:53:33 +0000 (00:53 +0000)]
Float32Array(ArrayBuffer, index, length) constructor working incorrectly.
https://bugs.webkit.org/show_bug.cgi?id=57042
Reviewed by James Robinson.
Source/WebCore:
* bindings/js/JSArrayBufferViewHelper.h:
(WebCore::constructArrayBufferViewWithArrayBufferArgument):
* bindings/v8/custom/V8ArrayBufferViewCustom.h:
(WebCore::constructWebGLArrayWithArrayBufferArgument):
LayoutTests:
* fast/canvas/webgl/array-unit-tests-expected.txt:
* fast/canvas/webgl/array-unit-tests.html:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91803
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
crogers@google.com [Wed, 27 Jul 2011 00:25:06 +0000 (00:25 +0000)]
V8 custom bindings for AudioNode must do proper object checking and throw exception in case of error
https://bugs.webkit.org/show_bug.cgi?id=65222
Reviewed by James Robinson.
No new tests - this fixes upcoming audionode.html layout test
* bindings/v8/custom/V8AudioNodeCustom.cpp:
(WebCore::V8AudioNode::connectCallback):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91802
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
crogers@google.com [Wed, 27 Jul 2011 00:15:37 +0000 (00:15 +0000)]
AudioContext is not correctly accounting for hardware resources
https://bugs.webkit.org/show_bug.cgi?id=65217
Reviewed by Kenneth Russell.
No new tests since audio API is not yet implemented.
* webaudio/AudioContext.cpp:
(WebCore::AudioContext::create):
(WebCore::AudioContext::AudioContext):
(WebCore::AudioContext::lazyInitialize):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91801
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
macpherson@chromium.org [Wed, 27 Jul 2011 00:14:34 +0000 (00:14 +0000)]
Handle CSS Properties that can be either auto or a primitive value in CSSStyleApplyProperty
https://bugs.webkit.org/show_bug.cgi?id=65164
Reviewed by Dimitri Glazkov.
No new tests / refactoring only.
* css/CSSPrimitiveValueMappings.h:
Add casts to/from various numeric types.
(WebCore::CSSPrimitiveValue::operator short):
(WebCore::CSSPrimitiveValue::CSSPrimitiveValue):
(WebCore::CSSPrimitiveValue::operator unsigned short):
(WebCore::CSSPrimitiveValue::operator int):
(WebCore::CSSPrimitiveValue::operator float):
* css/CSSStyleApplyProperty.cpp:
Add class to handle properties that set an "auto" boolean on RenderStyle.
(WebCore::ApplyPropertyAuto::ApplyPropertyAuto):
(WebCore::ApplyPropertyAuto::applyInheritValue):
(WebCore::ApplyPropertyAuto::applyInitialValue):
(WebCore::ApplyPropertyAuto::applyValue):
(WebCore::ApplyPropertyAuto::hasAuto):
(WebCore::ApplyPropertyAuto::setAuto):
(WebCore::CSSStyleApplyProperty::CSSStyleApplyProperty):
* css/CSSStyleSelector.cpp:
Remove existing implementations.
(WebCore::CSSStyleSelector::applyProperty):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91800
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
psolanki@apple.com [Tue, 26 Jul 2011 23:43:15 +0000 (23:43 +0000)]
Add protection space authentication callback code to CFNetwork loader on Mac
https://bugs.webkit.org/show_bug.cgi?id=65190
<rdar://problem/9842424>
Reviewed by Oliver Hunt.
Port over protection space authentication callback code from ResourceHandleMac.mm to
ResourceHandleCFNet.cpp.
* platform/network/ResourceHandle.h:
* platform/network/cf/ResourceHandleCFNet.cpp:
(WebCore::canRespondToProtectionSpace):
(WebCore::ResourceHandle::createCFURLConnection):
(WebCore::ResourceHandle::canAuthenticateAgainstProtectionSpace):
(WebCore::WebCoreSynchronousLoaderClient::canAuthenticateAgainstProtectionSpace):
* platform/network/mac/ResourceHandleMac.mm:
(-[WebCoreResourceHandleAsDelegate connection:canAuthenticateAgainstProtectionSpace:]): Add logging.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91799
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
enne@google.com [Tue, 26 Jul 2011 23:30:52 +0000 (23:30 +0000)]
[chromium] Check in expectations for font-family-pictograph.
https://bugs.webkit.org/show_bug.cgi?id=65197
Unreviewed gardening.
* platform/chromium-linux/fast/css/font-family-pictograph-expected.png: Added.
* platform/chromium-linux/fast/css/font-family-pictograph-expected.txt: Added.
* platform/chromium-mac/fast/css/font-family-pictograph-expected.png: Added.
* platform/chromium-mac/fast/css/font-family-pictograph-expected.txt: Added.
* platform/chromium-win/fast/css/font-family-pictograph-expected.png: Added.
* platform/chromium-win/fast/css/font-family-pictograph-expected.txt: Added.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91798
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
abarth@webkit.org [Tue, 26 Jul 2011 23:25:49 +0000 (23:25 +0000)]
Add support for download='filename' attribute in anchors.
https://bugs.webkit.org/show_bug.cgi?id=64580
Patch by Sadrul Habib Chowdhury <sadrul@chromium.org> on 2011-07-26
Reviewed by Adam Barth.
Source/WebCore:
The download attribute allows the author of the hyperlink to cause the
browser to download the linked URL. The author can also supply a
suggested file name in the attribute value. This feature is a recent
addition to HTML to better support offline applications that use blob
URLs. Traditionally, web sites use the HTTP Content-Disposition header
to trigger downloads, but that option isn't available when working
offline.
There is some question about whether we should initiate a download when
we receive a DOM-created click event. This patch does initiate the
download, but we might revise that decision in the future as part of a
larger change in how WebKit treats the interaction between default
event handlers and DOM-created events.
Tests: fast/dom/HTMLAnchorElement/anchor-download.html
fast/dom/HTMLAnchorElement/anchor-nodownload.html
fast/dom/HTMLAnchorElement/anchor-download-unset.html
fast/dom/HTMLAnchorElement/anchor-nodownload-set.html
* html/HTMLAnchorElement.cpp:
(WebCore::HTMLAnchorElement::defaultEventHandler):
(WebCore::HTMLAnchorElement::handleClick):
* html/HTMLAnchorElement.h:
* html/HTMLAnchorElement.idl:
* html/HTMLAttributeNames.in:
* loader/EmptyClients.h:
(WebCore::EmptyFrameLoaderClient::startDownload):
* loader/FrameLoaderClient.h:
Source/WebKit/chromium:
* features.gypi:
* public/WebFrameClient.h:
(WebKit::WebFrameClient::loadURLExternally):
* src/FrameLoaderClientImpl.cpp:
(WebKit::FrameLoaderClientImpl::dispatchDecidePolicyForNavigationAction):
(WebKit::FrameLoaderClientImpl::startDownload):
* src/FrameLoaderClientImpl.h:
Source/WebKit/efl:
* WebCoreSupport/FrameLoaderClientEfl.cpp:
(WebCore::FrameLoaderClientEfl::startDownload):
* WebCoreSupport/FrameLoaderClientEfl.h:
Source/WebKit/gtk:
* WebCoreSupport/FrameLoaderClientGtk.cpp:
(WebKit::FrameLoaderClient::startDownload):
* WebCoreSupport/FrameLoaderClientGtk.h:
Source/WebKit/haiku:
* WebCoreSupport/FrameLoaderClientHaiku.cpp:
(WebCore::FrameLoaderClientHaiku::startDownload):
* WebCoreSupport/FrameLoaderClientHaiku.h:
Source/WebKit/qt:
* WebCoreSupport/FrameLoaderClientQt.cpp:
(WebCore::FrameLoaderClientQt::startDownload):
* WebCoreSupport/FrameLoaderClientQt.h:
Source/WebKit/win:
* WebFrame.cpp:
(WebFrame::startDownload):
* WebFrame.h:
Source/WebKit/wince:
* WebCoreSupport/FrameLoaderClientWinCE.cpp:
(WebKit::FrameLoaderClientWinCE::startDownload):
* WebCoreSupport/FrameLoaderClientWinCE.h:
Source/WebKit/wx:
* WebKitSupport/FrameLoaderClientWx.cpp:
(WebCore::FrameLoaderClientWx::startDownload):
* WebKitSupport/FrameLoaderClientWx.h:
Source/WebKit2:
* WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
(WebKit::WebFrameLoaderClient::startDownload):
* WebProcess/WebCoreSupport/WebFrameLoaderClient.h:
Tools:
* DumpRenderTree/chromium/WebViewHost.cpp:
(WebViewHost::loadURLExternally):
* DumpRenderTree/chromium/WebViewHost.h:
LayoutTests:
* fast/dom/HTMLAnchorElement/anchor-download-expected.txt: Added.
* fast/dom/HTMLAnchorElement/anchor-download.html: Added.
* fast/dom/HTMLAnchorElement/anchor-nodownload-expected.txt: Added.
* fast/dom/HTMLAnchorElement/anchor-nodownload.html: Added.
* fast/dom/HTMLAnchorElement/anchor-download-unset-expected.txt: Added.
* fast/dom/HTMLAnchorElement/anchor-download-unset.html: Added.
* fast/dom/HTMLAnchorElement/anchor-nodownload-set-expected.txt: Added.
* fast/dom/HTMLAnchorElement/anchor-nodownload-set.html: Added.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91797
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
enne@google.com [Tue, 26 Jul 2011 23:18:32 +0000 (23:18 +0000)]
[chromium] Mark webaudio/test-basic as crashing on Windows.
https://bugs.webkit.org/show_bug.cgi?id=65219.
Unreviewed gardening.
* platform/chromium/test_expectations.txt:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91796
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
oliver@apple.com [Tue, 26 Jul 2011 22:47:32 +0000 (22:47 +0000)]
Buffer overflow creating error messages for JSON.parse
https://bugs.webkit.org/show_bug.cgi?id=65211
Reviewed by Darin Adler.
../../../../Volumes/Data/git/WebKit/OpenSource/LayoutTests:
Update testcase to reflect actual expected output.
* fast/js/JSON-parse-expected.txt:
../../../../Volumes/Data/git/WebKit/OpenSource/Source/JavaScriptCore:
Parse string length to the UString constructor.
* runtime/LiteralParser.cpp:
(JSC::LiteralParser::parse):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91795
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mitz@apple.com [Tue, 26 Jul 2011 22:23:16 +0000 (22:23 +0000)]
Rubber-stamped by Anders Carlsson.
Update DumpRenderTree and API tests for the pictograph font family preference.
* DumpRenderTree/mac/DumpRenderTree.mm:
(resetDefaultsToConsistentValues):
* DumpRenderTree/win/DumpRenderTree.cpp:
(resetDefaultsToConsistentValues):
* TestWebKitAPI/Tests/WebKit2/WKPreferences.cpp:
(TestWebKitAPI::TEST):
* WebKitTestRunner/TestController.cpp:
(WTR::TestController::resetStateToConsistentValues):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91794
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Tue, 26 Jul 2011 22:08:28 +0000 (22:08 +0000)]
Apply r40940 to WebKit2 to work around a Silverlight crash.
https://bugs.webkit.org/show_bug.cgi?id=65205
<rdar://problem/9058370>
Patch by Tim Horton <timothy_horton@apple.com> on 2011-07-26
Reviewed by Anders Carlsson.
* Shared/Plugins/Netscape/mac/NetscapePluginModuleMac.mm:
(WebKit::NetscapePluginModule::determineQuirks):
* Shared/Plugins/PluginQuirks.h:
* WebProcess/Plugins/Netscape/NetscapePlugin.cpp:
(WebKit::NetscapePlugin::userAgent):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91793
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
dglazkov@chromium.org [Tue, 26 Jul 2011 22:07:23 +0000 (22:07 +0000)]
Replace SpecificityCalculator with TestConfiguration-driven logic.
https://bugs.webkit.org/show_bug.cgi?id=65206
Use our newly-acquired ability to expand modifiers into a set of
matching TestConfiguration instances to calculate specificity and
determine whether expectation applies to a given test configuration.
Also, store bug modifier on TestExpectationsLine.
Reviewed by Adam Barth.
* Scripts/webkitpy/layout_tests/models/test_configuration.py: Added a way to report unknown modifier errors.
* Scripts/webkitpy/layout_tests/models/test_configuration_unittest.py: Added a test for reporting errors.
* Scripts/webkitpy/layout_tests/models/test_expectations.py: Replaced the logic of calculating specificity with
test configuration matching, removed a bunch of code.
* Scripts/webkitpy/layout_tests/models/test_expectations_unittest.py: Removed a bunch of tests that aren't useful anymore.
* Scripts/webkitpy/layout_tests/port/chromium_unittest.py: Fixed an error where an invalid TestConfiguration instance
was created. 'default' is not a valid build type.
* Scripts/webkitpy/layout_tests/port/test.py: Ditto. '' is not a valid version.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91792
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
abarth@webkit.org [Tue, 26 Jul 2011 22:04:52 +0000 (22:04 +0000)]
Increase the information density in garden-o-matic
https://bugs.webkit.org/show_bug.cgi?id=65141
Reviewed by Dimitri Glazkov.
This patch changes lets us show each revision on a single line,
allowing use to show an entire day's worth of commits at once.
* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/main.css:
* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/model.js:
* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ui.js:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91791
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Tue, 26 Jul 2011 21:56:13 +0000 (21:56 +0000)]
Refactor automatically generated JS DOM bindings to replace operator new with static create methods
https://bugs.webkit.org/show_bug.cgi?id=64732
Patch by Mark Hahnenberg <mhahnenberg@apple.com> on 2011-07-26
Reviewed by Oliver Hunt.
Source/JavaScriptCore:
Replacing the public constructors in the automatically generated JS DOM bindings with static
create methods. JSByteArray is used by several of these bindings in WebCore.
* JavaScriptCore.exp:
* JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def:
* runtime/JSByteArray.cpp:
(JSC::JSByteArray::create):
* runtime/JSByteArray.h:
Source/WebCore:
No new tests.
Replacing the public constructors in the automatically generated JS DOM bindings in CodeGeneratorJS.pm
with static create methods. This is part of a larger refactoring effort to use static create methods
in the headers of the generated files (so as to be inline-able) in favor of public constructors throughout JSC.
* bindings/js/JSAudioConstructor.h:
(WebCore::JSAudioConstructor::create):
* bindings/js/JSDOMBinding.h:
(WebCore::createWrapper):
* bindings/js/JSDOMGlobalObject.h:
(WebCore::getDOMConstructor):
* bindings/js/JSDOMWindowCustom.cpp:
(WebCore::JSDOMWindow::history):
(WebCore::JSDOMWindow::location):
* bindings/js/JSDOMWindowShell.cpp:
(WebCore::JSDOMWindowShell::setWindow):
* bindings/js/JSDocumentCustom.cpp:
(WebCore::JSDocument::location):
* bindings/js/JSImageConstructor.h:
(WebCore::JSImageConstructor::create):
* bindings/js/JSImageDataCustom.cpp:
(WebCore::toJS):
* bindings/js/JSOptionConstructor.h:
(WebCore::JSOptionConstructor::create):
* bindings/js/WorkerScriptController.cpp:
(WebCore::WorkerScriptController::initScript):
* bindings/scripts/CodeGeneratorJS.pm:
(AddIncludesForTypeInImpl):
(AddIncludesForTypeInHeader):
(AddIncludesForType):
(GenerateHeader):
(GenerateImplementation):
(GenerateCallbackImplementation):
(GenerateConstructorDeclaration):
* bindings/scripts/test/JS/JSTestInterface.cpp:
(WebCore::JSTestInterfaceConstructor::create):
(WebCore::JSTestInterface::createPrototype):
* bindings/scripts/test/JS/JSTestInterface.h:
(WebCore::JSTestInterface::create):
(WebCore::JSTestInterfacePrototype::create):
* bindings/scripts/test/JS/JSTestMediaQueryListListener.cpp:
(WebCore::JSTestMediaQueryListListenerConstructor::create):
(WebCore::JSTestMediaQueryListListener::createPrototype):
* bindings/scripts/test/JS/JSTestMediaQueryListListener.h:
(WebCore::JSTestMediaQueryListListener::create):
(WebCore::JSTestMediaQueryListListenerPrototype::create):
* bindings/scripts/test/JS/JSTestObj.cpp:
(WebCore::JSTestObjConstructor::create):
(WebCore::JSTestObj::createPrototype):
* bindings/scripts/test/JS/JSTestObj.h:
(WebCore::JSTestObj::create):
(WebCore::JSTestObjPrototype::create):
* bindings/scripts/test/JS/JSTestSerializedScriptValueInterface.cpp:
(WebCore::JSTestSerializedScriptValueInterfaceConstructor::create):
(WebCore::JSTestSerializedScriptValueInterface::createPrototype):
* bindings/scripts/test/JS/JSTestSerializedScriptValueInterface.h:
(WebCore::JSTestSerializedScriptValueInterface::create):
(WebCore::JSTestSerializedScriptValueInterfacePrototype::create):
* bridge/jni/jsc/JavaArrayJSC.cpp:
(JavaArray::convertJObjectToArray):
* bridge/jsc/BridgeJSC.cpp:
(JSC::Bindings::Instance::newRuntimeObject):
* bridge/objc/objc_utility.mm:
(JSC::Bindings::convertObjcValueToValue):
* bridge/qt/qt_runtime.cpp:
(JSC::Bindings::convertQVariantToValue):
* bridge/runtime_array.h:
(JSC::RuntimeArray::create):
* bridge/runtime_object.h:
(JSC::Bindings::RuntimeObject::create):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91790
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
brettw@chromium.org [Tue, 26 Jul 2011 21:52:50 +0000 (21:52 +0000)]
Cancel the load when an associated URL loader goes out of scope.
Previously, the load would continue even when the WebURLLoader object
was deleted, and the data would just get queued up.
https://bugs.webkit.org/show_bug.cgi?id=65204
Reviewed by Darin Fisher.
* src/AssociatedURLLoader.cpp:
(WebKit::AssociatedURLLoader::~AssociatedURLLoader):
(WebKit::AssociatedURLLoader::cancel):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91789
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
xji@chromium.org [Tue, 26 Jul 2011 21:51:17 +0000 (21:51 +0000)]
--webkit-visual-word does not work in multi-line
https://bugs.webkit.org/show_bug.cgi?id=61344
Reviewed by Ryosuke Niwa.
Source/WebCore:
Replace Position(node, offset, PositionIsOffsetInAnchor) with
createPositionAvoidingIgnoredNode(node, offset) which takes care of creating position
before/after <br/> etc. editingIgnoresContent node.
When iterate InlineBox, replace prevLeafChild and nextLeafChild (which only returns
InlineBox within the same line) with leftInlineBox and rightInlineBox (which returns
InlineBox across multiple lines).
* editing/htmlediting.cpp:
(WebCore::createPositionAvoidingIgnoredNode):
* editing/htmlediting.h:
* editing/visible_units.cpp:
(WebCore::previousWordBreakInBoxInsideBlockWithSameDirectionality): Replace Position() with
createPositionAvoidingIgnoredNode().
(WebCore::leftmostPositionInRTLBoxInLTRBlock): Ditto.
(WebCore::rightmostPositionInLTRBoxInRTLBlock): Ditto.
(WebCore::nextWordBreakInBoxInsideBlockWithDifferentDirectionality): Ditto.
(WebCore::blockWithPreviousLineBox): Added.
(WebCore::previousRootInlineBox): Added.
(WebCore::blockWithNextLineBox): Added.
(WebCore::nextRootInlineBox): Added.
(WebCore::leftInlineBox): Added.
(WebCore::rightInlineBox): Added.
(WebCore::leftWordBoundary): Replace prevLeafChild/nextLeafChild with leftInlineBox()
and rightInlineBox().
(WebCore::rightWordBoundary): Ditto.
(WebCore::leftWordPositionAcrossBoundary): Ditto.
(WebCore::rightWordPositionAcrossBoundary): Ditto.
LayoutTests:
Add test for multiple-line.
* editing/selection/move-by-word-visually-null-box.html:
* editing/selection/move-by-word-visually-others-expected.txt:
* editing/selection/move-by-word-visually-others.html: Add tests for multi-line. Shorten
test id.
* editing/selection/move-by-word-visually-single-space-sigle-line.html: Shorten test id.
* editing/selection/resources/move-by-word-visually.js:
(moveByWordForEveryPosition):
(setWidth):
(runMoveLeftRight):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91788
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
alexis.menard@openbossa.org [Tue, 26 Jul 2011 21:49:03 +0000 (21:49 +0000)]
Unreviewed build fix for Qt/Linux.
On platforms with no glib and gstreamer we should not build javascriptcore
with the Glib support. This is related to http://trac.webkit.org/changeset/91752.
* wtf/wtf.pri:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91787
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
dglazkov@chromium.org [Tue, 26 Jul 2011 21:44:28 +0000 (21:44 +0000)]
TestConfigurationConverter should not barf on invalid specifiers.
https://bugs.webkit.org/show_bug.cgi?id=65194
Reviewed by Adam Barth.
* Scripts/webkitpy/layout_tests/models/test_configuration.py: Refactored TestConfigurationConverter.to_config_set a bit,
made it return empty set whenever an invalid specifier is encountered.
* Scripts/webkitpy/layout_tests/models/test_configuration_unittest.py: Added corresponding tests.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91786
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
dglazkov@chromium.org [Tue, 26 Jul 2011 21:42:29 +0000 (21:42 +0000)]
Introduce a way to convert between specifier lists and TestConfiguration sets.
https://bugs.webkit.org/show_bug.cgi?id=65094
Add TestConfigurationConverter, a facility to convert:
1) a set of specifiers into a set of TestConfiguration instances;
2) a set of TestConfiguration instances into a list of sets of specifiers.
Reviewed by Adam Barth.
* Scripts/webkitpy/layout_tests/models/test_configuration.py: Added TestConfigurationConverter.
* Scripts/webkitpy/layout_tests/models/test_configuration_unittest.py: Added tests for it.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91785
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Tue, 26 Jul 2011 21:17:14 +0000 (21:17 +0000)]
CSS Regions build bot should build with "--css-regions" and "--css-exclusion" flags
https://bugs.webkit.org/show_bug.cgi?id=65171
Added "features" parameter in config.json for "builder" objects. The parameter is optional and must be an array. All the items
of the array will be passed to the build-webkit script. The items will automatically be prefixed with "--".
Patch by Alexandru Chiculita <achicu@adobe.com> on 2011-07-26
Reviewed by Adam Roben.
* BuildSlaveSupport/build.webkit.org-config/config.json:
* BuildSlaveSupport/build.webkit.org-config/master.cfg:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91782
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
hyatt@apple.com [Tue, 26 Jul 2011 20:39:25 +0000 (20:39 +0000)]
https://bugs.webkit.org/show_bug.cgi?id=60778
Use after free because of line box culling optimization regression.
In the case of a child with no line box being removed (typically
a <br> in quirks mode), if there is no previous sibling with a line
box, then we have a potential problem with the culling optimization.
The culled inline may still have other leaf line box children, but
they may follow the removed <br>. In this case we can't rely on
them, since we need a line box that comes before the <br>.
The fix is to simply recur up to the parent if we are a culled inline
and could not find a previous line box.
Reviewed by Dan Bernstein.
Added editing/execCommand/crash-line-break-after-outdent.html
Source/WebCore:
* rendering/RenderLineBoxList.cpp:
(WebCore::RenderLineBoxList::dirtyLinesFromChangedChild):
LayoutTests:
* editing/execCommand/crash-line-break-after-outdent-expected.txt: Added.
* editing/execCommand/crash-line-break-after-outdent.html: Added.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91781
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
crogers@google.com [Tue, 26 Jul 2011 20:36:25 +0000 (20:36 +0000)]
Add first basic layout test for the web audio API
https://bugs.webkit.org/show_bug.cgi?id=57977
Reviewed by Tony Chang.
* webaudio: Added.
* webaudio/resources: Added.
* webaudio/resources/audio-testing.js: Added.
(writeString):
(writeInt16):
(writeInt32):
(writeAudioBuffer):
(createWaveFileData):
(createAudioData):
(finishAudioTest):
* webaudio/test-basic-expected.wav: Added.
* webaudio/test-basic.html: Added.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91780
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
crogers@google.com [Tue, 26 Jul 2011 20:28:15 +0000 (20:28 +0000)]
Avoid outputting #EOF twice for web audio tests in chromium DRT
https://bugs.webkit.org/show_bug.cgi?id=65201
Reviewed by Dirk Pranke.
* DumpRenderTree/chromium/TestShell.cpp:
(TestShell::dump):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91779
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mitz@apple.com [Tue, 26 Jul 2011 20:27:55 +0000 (20:27 +0000)]
Fix change log typos
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91778
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mitz@apple.com [Tue, 26 Jul 2011 20:23:50 +0000 (20:23 +0000)]
Source/WebCore: <rdar://problem/9842889> Add a generic pictograph font family Add a generic pictograph font family
https://bugs.webkit.org/show_bug.cgi?id=65197
Reviewed by Anders Carlsson.
Test: fast/css/font-family-pictograph.html
* WebCore.exp.in: Export Settings::setPictographFontFamily().
* css/CSSComputedStyleDeclaration.cpp:
(WebCore::identifierForFamily): Added -webkit-pictograph.
* css/CSSFontSelector.cpp:
(WebCore::CSSFontSelector::addFontFaceRule): Ditto.
(WebCore::fontDataForGenericFamily): Ditto.
* css/CSSStyleSelector.cpp:
(WebCore::CSSStyleSelector::applyProperty): Ditto.
* css/CSSValueKeywords.in:
* inspector/front-end/CSSKeywordCompletions.js: Ditto.
* inspector/front-end/SourceCSSTokenizer.js: Ditto.
(WebInspector.SourceCSSTokenizer):
* inspector/front-end/SourceCSSTokenizer.re2js: Ditto.
* page/Settings.cpp:
(WebCore::Settings::pictographFontFamily): Added this getter.
(WebCore::Settings::setPictographFontFamily): Added this setter.
* page/Settings.h:
* platform/graphics/FontDescription.h: Added PictographFamily to the GenericFamilyType enum.
Source/WebKit/mac: <rdar://problem/9842889> Add a generic pictograph font family
https://bugs.webkit.org/show_bug.cgi?id=65197
Reviewed by Anders Carlsson.
* WebView/WebPreferenceKeysPrivate.h: Added WebKitPictographFontPreferenceKey.
* WebView/WebPreferences.mm:
(+[WebPreferences initialize]): Set the default pictograph font family to
"Apple Color Emoji".
(-[WebPreferences pictographFontFamily]): Added this getter.
(-[WebPreferences setPictographFontFamily:]): Added this setter.
* WebView/WebPreferencesPrivate.h:
* WebView/WebView.mm:
(-[WebView _preferencesChanged:]): Pass the preference through to WebCore.
Source/WebKit/win: <rdar://problem/9842889> Add a generic pictograph font family
https://bugs.webkit.org/show_bug.cgi?id=65197
Reviewed by Anders Carlsson.
* Interfaces/IWebPreferences.idl: Added pictographFontFamily() and
setPictographFontFamily().
* WebPreferenceKeysPrivate.h: Added WebKitPictographFontPreferenceKey.
* WebPreferences.cpp:
(WebPreferences::initializeDefaultSettings): Initialize the pictograph font
preference to "Times New Roman".
(WebPreferences::pictographFontFamily): Added this getter.
(WebPreferences::setPictographFontFamily): Added this setter.
* WebPreferences.h:
* WebView.cpp:
(WebView::notifyPreferencesChanged): Pass the preference value through to
WebCore.
Source/WebKit2: <rdar://problem/9842889> Add a generic pictograph font family
https://bugs.webkit.org/show_bug.cgi?id=65197
Reviewed by Anders Carlsson.
* Shared/WebPreferencesStore.h:
* UIProcess/API/C/WKPreferences.cpp:
(WKPreferencesSetPictographFontFamily): Added.
(WKPreferencesCopyPictographFontFamily): Added.
* UIProcess/API/C/WKPreferences.h:
LayoutTests: <rdar://problem/9842889> Add a generic pictograph font family Add a generic pictograph font family
https://bugs.webkit.org/show_bug.cgi?id=65197
Reviewed by Anders Carlsson.
* fast/css/font-family-pictograph.html: Added.
* platform/mac/fast/css/font-family-pictograph-expected.png: Added.
* platform/mac/fast/css/font-family-pictograph-expected.txt: Added.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91777
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
enne@google.com [Tue, 26 Jul 2011 19:48:37 +0000 (19:48 +0000)]
[chromium] getClientRects now fails similarly on Mac after r91763.
Unreviewed gardening.
* platform/chromium/test_expectations.txt:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91775
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Tue, 26 Jul 2011 19:37:01 +0000 (19:37 +0000)]
Improve (i.e., speed up) .gyp(i) files for Chromium
https://bugs.webkit.org/show_bug.cgi?id=65032
Patch by Viet-Trung Luu <viettrungluu@chromium.org> on 2011-07-26
Reviewed by Adam Barth.
This re-applies r91595 with the source exclusion fixed.
Test by building (in all supported configurations).
* WebCore.gyp/WebCore.gyp:
* WebCore.gypi:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91774
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
enne@google.com [Tue, 26 Jul 2011 19:26:18 +0000 (19:26 +0000)]
[chromium] Fix accidental duplicate expectation from r91772.
Unreviewed gardening.
* platform/chromium/test_expectations.txt:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91773
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
enne@google.com [Tue, 26 Jul 2011 19:22:54 +0000 (19:22 +0000)]
[chromium] Mark test as flaky after r91738.
https://bugs.webkit.org/show_bug.cgi?id=65199
Unreviewed gardening.
* platform/chromium/test_expectations.txt:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91772
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
alexis.menard@openbossa.org [Tue, 26 Jul 2011 19:14:38 +0000 (19:14 +0000)]
[Qt] Disable video support on linux if the dependencies are not found.
If we can't find the necessary dependencies to build the GStreamer media player
we disable the video support. This is related to http://trac.webkit.org/changeset/91752.
Reviewed by Holger Freyther.
No new tests, it's a build fix.
* features.pri:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91771
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
evan@chromium.org [Tue, 26 Jul 2011 19:13:07 +0000 (19:13 +0000)]
regression: generating CSSValueKeywords should not print by default
https://bugs.webkit.org/show_bug.cgi?id=65195
Reviewed by Tony Chang.
Some refactoring made some callers no longer pass the final 'beQuiet'
parameter to applyPreprocessor. This flag was only used to print the
input filename anyway, so remove the flag and just make the caller
print the filename if it wants this.
* bindings/scripts/IDLParser.pm:
(Parse): print before calling applyPreprocessor, if necessary.
* bindings/scripts/preprocessor.pm:
(applyPreprocessor): remove the beQuiet flag; just always be quiet.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91770
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Tue, 26 Jul 2011 19:03:18 +0000 (19:03 +0000)]
[EFL] Do not crash on the isSlider() assert.
https://bugs.webkit.org/show_bug.cgi?id=65191
Patch by Raphael Kubo da Costa <kubo@profusion.mobi> on 2011-07-26
Reviewed by Antonio Gomes.
RenderThemeEfl's paintSliderThumb() just forwarded the call to
paintSliderTrack(), which, on its turn, called code that assumed that
object->isSlider() was true. That does not seem to be the case for
slidethumbs, so any page with an <input type="range"> tag would crash
WebKit.
We now do what RenderThemeQt does, and assume everything was properly
done in the call to paintSliderTrack().
No new tests, as this has been uncovered by an existing test.
* platform/efl/RenderThemeEfl.cpp:
(WebCore::RenderThemeEfl::paintSliderThumb):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91769
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Tue, 26 Jul 2011 18:53:55 +0000 (18:53 +0000)]
[SKIA] Make the skia GL context current when drawing text to gpu backed platform context.
https://bugs.webkit.org/show_bug.cgi?id=65182
Patch by Brian Salomon <bsalomon@google.com> on 2011-07-26
Reviewed by Stephen White.
No new tests. Not sure how to test wrong-ctx bugs in layout tests.
* platform/graphics/skia/SkiaFontWin.cpp:
(WebCore::paintSkiaText):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91768
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
enne@google.com [Tue, 26 Jul 2011 18:17:20 +0000 (18:17 +0000)]
[chromium] More flaky crashers.
https://bugs.webkit.org/show_bug.cgi?id=65155
Unreviewed gardening.
* platform/chromium/test_expectations.txt:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91767
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
abecsi@webkit.org [Tue, 26 Jul 2011 18:09:55 +0000 (18:09 +0000)]
.: [Qt] [WK2] Disable the build with Qt versions older than 5.0
https://bugs.webkit.org/show_bug.cgi?id=65189
Reviewed by Csaba Osztrogonác.
* Source/WebKit.pro: Abort if not using Qt5.
Source/WebKit2: [Qt] [WK2] Disable the build with Qt versions older than 5.0
https://bugs.webkit.org/show_bug.cgi?id=65189
Reviewed by Csaba Osztrogonác.
* Platform/qt/ModuleQt.cpp:
(WebKit::Module::platformFunctionPointer): Fix the build for Qt5.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91766
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Tue, 26 Jul 2011 17:53:40 +0000 (17:53 +0000)]
JSON errors should be informative
https://bugs.webkit.org/show_bug.cgi?id=63339
Source/JavaScriptCore:
Added error messages to the JSON Parser.
Patch by Juan C. Montemayor <jmont@apple.com> on 2011-07-26
Reviewed by Oliver Hunt.
* runtime/JSONObject.cpp:
(JSC::JSONProtoFuncParse):
* runtime/LiteralParser.cpp:
(JSC::LiteralParser::Lexer::lex):
(JSC::LiteralParser::Lexer::lexString):
(JSC::LiteralParser::Lexer::lexNumber):
(JSC::LiteralParser::parse):
* runtime/LiteralParser.h:
(JSC::LiteralParser::getErrorMessage):
(JSC::LiteralParser::Lexer::sawError):
(JSC::LiteralParser::Lexer::getErrorMessage):
LayoutTests:
Updated expected test results.
Patch by Juan C. Montemayor <jmont@apple.com> on 2011-07-26
Reviewed by Oliver Hunt.
* fast/js/JSON-parse-expected.txt:
* platform/chromium/fast/js/JSON-parse-expected.txt: Added.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91765
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
eae@chromium.org [Tue, 26 Jul 2011 17:47:03 +0000 (17:47 +0000)]
RenderText::absoluteRectsForRange() and absoluteQuadsForRange() have nearly duplicate code
https://bugs.webkit.org/show_bug.cgi?id=62478
Reviewed by Simon Fraser.
Source/WebCore:
Test: fast/dom/Range/getClientRects.html
* platform/graphics/FloatRect.h:
(WebCore::FloatRect::isZero):
Add izZero method, unlike isEmpty this checks if both the width and the
height are zero.
* rendering/RenderText.cpp:
(WebCore::absoluteQuadForTextBox):
(WebCore::RenderText::absoluteRectsForRange):
(WebCore::RenderText::absoluteQuadsForRange):
Break duplicate code out of absoluteRectsForRange and
absoluteQuadsForRange into shared static function.
Fix what I presume to be a bug in the absoluteQuadsForRange where the
logicalHeight was always used to set the size in absoluteQuadForTextBox
as oppsued to the absoluteRectsForRange implementation that uses the
logicalWidth or the logicalHeight depending on whether it's horizontal or
vertical.
LayoutTests:
* fast/dom/Range/getClientRects-expected.txt:
* fast/dom/Range/getClientRects.html:
Add test for vertical text.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91763
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
abarth@webkit.org [Tue, 26 Jul 2011 17:37:15 +0000 (17:37 +0000)]
Windows bots on build.chromium.org can't figure out the head_svn_revision
https://bugs.webkit.org/show_bug.cgi?id=65156
Reviewed by Tony Chang.
This patch is a bit of a guess as to the issue based on the
investigation in https://bugs.webkit.org/show_bug.cgi?id=65148. I'm
not sure whether it will fix the issue, but it seems worth a try.
* Scripts/webkitpy/common/checkout/scm/svn.py:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91762
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
enne@google.com [Tue, 26 Jul 2011 17:32:20 +0000 (17:32 +0000)]
[chromium] Mark media-controls-clone as flaky crashing on other platforms.
https://bugs.webkit.org/show_bug.cgi?id=65129
Unreviewed gardening.
* platform/chromium/test_expectations.txt:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91761
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Tue, 26 Jul 2011 17:11:08 +0000 (17:11 +0000)]
[CSSRegions] Collect flowed elements in different render element
https://bugs.webkit.org/show_bug.cgi?id=64516
Source/WebCore:
All the RenderObjects that have a specified "flow" name are moved out of the
normal flow. A RenderFlowThread will collect all such RenderObjects. There is
a RenderFlowThread for each flow name. The new RenderFlowThread is an anonymous
RenderBlock that is positioned absolute and will not render directly. In a
different bug/patch we will add RenderRegion that will take care of rendering the
RenderFlowThread on screen.
Patch by Alexandru Chiculita <achicu@adobe.com> on 2011-07-26
Reviewed by Simon Fraser.
Tests: fast/regions/webkit-flow-renderer-layer.html
fast/regions/webkit-flow-renderer-nested.html
fast/regions/webkit-flow-renderer.html
* WebCore.vcproj/WebCore.vcproj:
* WebCore.xcodeproj/project.pbxproj:
* dom/Node.cpp:
(WebCore::Node::diff):
* dom/NodeRenderingContext.cpp:
(WebCore::NodeRenderingContext::NodeRenderingContext):
(WebCore::NodeRenderingContext::setStyle):
(WebCore::NodeRenderingContext::nextRenderer):
(WebCore::NodeRenderingContext::previousRenderer):
(WebCore::NodeRenderingContext::parentRenderer):
(WebCore::NodeRenderingContext::moveToFlowThreadIfNeeded):
(WebCore::NodeRendererFactory::createRendererIfNeeded):
* dom/NodeRenderingContext.h:
(WebCore::NodeRenderingContext::hasFlowThreadParent):
(WebCore::NodeRenderingContext::parentFlowRenderer):
* rendering/RenderFlowThread.cpp: Added.
(WebCore::RenderFlowThread::RenderFlowThread):
(WebCore::RenderFlowThread::createFlowThreadStyle):
(WebCore::RenderFlowThread::nextRendererForNode):
(WebCore::RenderFlowThread::previousRendererForNode):
(WebCore::RenderFlowThread::addChild):
(WebCore::RenderFlowThread::removeChild):
* rendering/RenderFlowThread.h: Added.
(WebCore::toRenderFlowThread):
* rendering/RenderLayer.cpp:
(WebCore::RenderLayer::collectLayers):
* rendering/RenderObject.h:
(WebCore::RenderObject::isRenderFlowThread):
* rendering/RenderTreeAsText.cpp:
(WebCore::RenderTreeAsText::writeRenderObject):
(WebCore::writeLayers):
* rendering/RenderView.cpp:
(WebCore::RenderView::renderFlowThreadWithName):
* rendering/RenderView.h:
* rendering/style/RenderStyle.cpp:
(WebCore::RenderStyle::diff):
LayoutTests:
Patch by Alexandru Chiculita <achicu@adobe.com> on 2011-07-26
Reviewed by Simon Fraser.
* fast/regions/webkit-flow-renderer-expected.txt: Added.
* fast/regions/webkit-flow-renderer-layer-expected.txt: Added.
* fast/regions/webkit-flow-renderer-layer.html: Added.
* fast/regions/webkit-flow-renderer-nested-expected.txt: Added.
* fast/regions/webkit-flow-renderer-nested.html: Added.
* fast/regions/webkit-flow-renderer.html: Added.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91760
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Tue, 26 Jul 2011 17:08:59 +0000 (17:08 +0000)]
Source/WebCore: Filename text in file upload controls is not aligned with button text on
chromium-linux, chromium-win, and gtk.
https://bugs.webkit.org/show_bug.cgi?id=64692
The fix is to not include the margin, border, and padding in the calculation,
because those are already accounted for by RenderBlock::baselinePosition().
Patch by Patrick Dubroy <dubroy@chromium.org> on 2011-07-26
Reviewed by Dimitri Glazkov.
* rendering/RenderFileUploadControl.cpp:
(WebCore::RenderFileUploadControl::paintObject):
LayoutTests: In file upload controls, filename text was misaligned with button text.
Update chromium-linux with the new baseline images, and add temporary
exceptions for chromium-win and gtk until they can be rebaselined.
https://bugs.webkit.org/show_bug.cgi?id=64692
Patch by Patrick Dubroy <dubroy@chromium.org> on 2011-07-26
Reviewed by Dimitri Glazkov.
* platform/chromium-linux/fast/forms/box-shadow-override-expected.png:
* platform/chromium-linux/fast/forms/file-input-direction-expected.png:
* platform/chromium-linux/fast/forms/file-input-disabled-expected.png:
* platform/chromium-linux/fast/forms/form-element-geometry-expected.png:
* platform/chromium-linux/fast/forms/input-appearance-height-expected.png:
* platform/chromium-linux/fast/forms/input-file-re-render-expected.png:
* platform/chromium-linux/fast/forms/input-value-expected.png:
* platform/chromium/test_expectations.txt:
* platform/gtk/test_expectations.txt:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91759
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
benjamin.poulain@nokia.com [Tue, 26 Jul 2011 16:56:28 +0000 (16:56 +0000)]
[Qt][WK2] Make the WebContextMenuProxyQt handle the full interactions between the views and the WebPageProxy
https://bugs.webkit.org/show_bug.cgi?id=64739
Reviewed by Andreas Kling.
Change the WebContextMenuProxyQt to be the intermediary between the WebPageProxy and our views regarding
the context menu.
The actions of the context menu are no longer the responsibility of QtWebPageProxy but are instead
created and handled directly by the WebContextMenuProxyQt.
* UIProcess/qt/QtWebPageProxy.cpp:
(QtWebPageProxy::createContextMenuProxy):
(QtWebPageProxy::triggerAction):
(QtWebPageProxy::action):
* UIProcess/qt/QtWebPageProxy.h:
* UIProcess/qt/WebContextMenuProxyQt.cpp:
(WebKit::WebContextMenuProxyQt::WebContextMenuProxyQt):
(WebKit::WebContextMenuProxyQt::create):
(WebKit::WebContextMenuProxyQt::actionTriggered):
(WebKit::WebContextMenuProxyQt::showContextMenu):
(WebKit::WebContextMenuProxyQt::hideContextMenu):
(WebKit::WebContextMenuProxyQt::createContextMenu):
* UIProcess/qt/WebContextMenuProxyQt.h:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91758
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
pfeldman@chromium.org [Tue, 26 Jul 2011 16:22:01 +0000 (16:22 +0000)]
2011-07-26 Pavel Feldman <pfeldman@google.com>
Not reviewed: inspector test rebaselines.
* inspector/console/console-format-expected.txt:
* inspector/runtime/runtime-getProperties-expected.txt:
* platform/chromium/inspector/runtime/runtime-getProperties-expected.txt: Copied from LayoutTests/inspector/runtime/runtime-getProperties-expected.txt.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91757
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Tue, 26 Jul 2011 14:58:10 +0000 (14:58 +0000)]
Unreviewed, rolling out r91746.
http://trac.webkit.org/changeset/91746
https://bugs.webkit.org/show_bug.cgi?id=65180
It broke SL build (Requested by Ossy on #webkit).
Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2011-07-26
Source/JavaScriptCore:
* wtf/text/StringImpl.cpp:
(WTF::StringImpl::stripWhiteSpace):
(WTF::StringImpl::simplifyWhiteSpace):
* wtf/text/StringImpl.h:
* wtf/text/WTFString.cpp:
* wtf/text/WTFString.h:
Source/WebCore:
* dom/OptionElement.cpp:
(WebCore::OptionElement::normalizeText):
LayoutTests:
* fast/forms/option-strip-unicode-spaces-expected.txt: Removed.
* fast/forms/option-strip-unicode-spaces.html: Removed.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91756
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Tue, 26 Jul 2011 14:55:33 +0000 (14:55 +0000)]
[GTK] Remove header webkitwebviewcommon.h
https://bugs.webkit.org/show_bug.cgi?id=65177
Patch by Carlos Garcia Campos <cgarcia@igalia.com> on 2011-07-26
Reviewed by Martin Robinson.
Source/WebKit/gtk:
Move webkitwebviewcommon.h contents back to webkitwebview.h.
* GNUmakefile.am: Remove webkitwebviewcommon.h
* webkit/webkitwebview.h:
* webkit/webkitwebviewcommon.h: Removed.
Source/WebKit2:
* UIProcess/API/gtk/WebKitWebView.h: Add function prototypes
instead of including webkitwebviewcommon.h.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91755
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
pfeldman@chromium.org [Tue, 26 Jul 2011 14:44:13 +0000 (14:44 +0000)]
Web Inspector: Better represent custom getters in the properties pane
https://bugs.webkit.org/show_bug.cgi?id=16734
Reviewed by Yury Semikhatsky.
Source/WebCore:
Test: inspector/runtime/runtime-getProperties.html
* inspector/InjectedScriptSource.js:
(.):
* inspector/Inspector.json:
* inspector/front-end/ObjectPropertiesSection.js:
(WebInspector.ObjectPropertyTreeElement.prototype.update):
LayoutTests:
* inspector/runtime/runtime-getProperties-expected.txt: Added.
* inspector/runtime/runtime-getProperties.html: Added.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91754
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ossy@webkit.org [Tue, 26 Jul 2011 14:16:33 +0000 (14:16 +0000)]
[Qt] Unreviewed gardening.
* platform/qt/fast/css/font-face-opentype-expected.png: Added.
* platform/qt/fast/css/font-face-opentype-expected.txt: Updated after r91738.
* platform/qt/svg/custom/image-with-transform-clip-filter-expected.png: Updated after r91748.
* platform/qt/svg/custom/image-with-transform-clip-filter-expected.txt: Updated after r91748.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91753
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
abecsi@webkit.org [Tue, 26 Jul 2011 13:34:26 +0000 (13:34 +0000)]
Patch by Alexis Menard <alexis.menard@openbossa.org> on 2011-07-26
Reviewed by Andreas Kling.
[Qt] Change default backend to use GStreamer on Linux and QuickTime on Mac.
https://bugs.webkit.org/show_bug.cgi?id=63472
Source/JavaScriptCore:
Enable the bits needed for GStreamer only when QtMultimedia is not used.
* wtf/wtf.pri:
Source/WebCore:
Enable the GStreamer backend and the QuickTime backend as default media players
for the Qt port on Mac and Linux. QtMultimedia is now a fallback option that you
can enable by passing DEFINES+=USE_QT_MULTIMEDIA=1 to enforce its usage.
No new tests. The media layout tests are disabled on the Qt port but hopefully with this
switch we can enable them again.
* WebCore.pri:
* WebCore.pro:
* features.pri:
Source/WebKit/qt:
Use the flag WTF_USE_QTKIT=1 rather than USE_QTKIT=1 because that flag doesn't
exist anymore, it's the default. Ditto for GStreamer.
* QtWebKit.pro:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91752
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
carlosgc@webkit.org [Tue, 26 Jul 2011 12:16:28 +0000 (12:16 +0000)]
2011-07-26 Carlos Garcia Campos <cgarcia@igalia.com>
Unreviewed. Fix WebKitGtk build with GTK+ 3.x after r91707.
* DumpRenderTree/gtk/SelfScrollingWebKitWebView.cpp:
(getPreferredSizeMethod):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91751
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
pfeldman@chromium.org [Tue, 26 Jul 2011 10:20:53 +0000 (10:20 +0000)]
Web Inspector: Implement setPropertyValue using evaluate and callFunctionOn
https://bugs.webkit.org/show_bug.cgi?id=65112
Reviewed by Yury Semikhatsky.
Source/WebCore:
Test: inspector/runtime/runtime-setPropertyValue.html
* inspector/InjectedScript.cpp:
* inspector/InjectedScript.h:
* inspector/InjectedScriptSource.js:
(.):
* inspector/Inspector.json:
* inspector/InspectorRuntimeAgent.cpp:
* inspector/InspectorRuntimeAgent.h:
* inspector/front-end/RemoteObject.js:
(WebInspector.RemoteObject.prototype.setPropertyValue.evaluatedCallback.setPropertyValue):
(WebInspector.RemoteObject.prototype.setPropertyValue.evaluatedCallback):
(WebInspector.RemoteObject.prototype.setPropertyValue.propertySetCallback):
(WebInspector.RemoteObject.prototype.setPropertyValue):
LayoutTests:
* inspector/runtime/runtime-setPropertyValue-expected.txt: Added.
* inspector/runtime/runtime-setPropertyValue.html: Added.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91750
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
pfeldman@chromium.org [Tue, 26 Jul 2011 10:09:52 +0000 (10:09 +0000)]
2011-07-26 Pavel Feldman <pfeldman@google.com>
Re-applying r91665 and r91668 with JSC issues fixed.
Web Inspector: use dedicated type for object id in the protocol.
Web Inspector: refactor remote object structure to contain value for primitive values.
https://bugs.webkit.org/show_bug.cgi?id=65103
https://bugs.webkit.org/show_bug.cgi?id=65110
Reviewed by Yury Semikhatsky.
* inspector/InjectedScript.cpp:
(WebCore::InjectedScript::callFunctionOn):
* inspector/InjectedScript.h:
* inspector/InjectedScriptSource.js:
* inspector/Inspector.json:
* inspector/InspectorRuntimeAgent.cpp:
(WebCore::InspectorRuntimeAgent::callFunctionOn):
* inspector/InspectorValues.cpp:
(WebCore::InspectorBasicValue::writeJSON):
* inspector/front-end/ConsoleView.js:
(WebInspector.ConsoleView.prototype._format):
(WebInspector.ConsoleView.prototype._formatAsArrayEntry):
(WebInspector.ConsoleMessage.prototype._format):
* inspector/front-end/ElementsTreeOutline.js:
(WebInspector.ElementsTreeElement.prototype._createTooltipForNode.setTooltip):
* inspector/front-end/ObjectPropertiesSection.js:
(WebInspector.ObjectPropertyTreeElement.prototype.update):
* inspector/front-end/PropertiesSidebarPane.js:
(WebInspector.PropertiesSidebarPane.prototype.update.nodePrototypesReady):
* inspector/front-end/RemoteObject.js:
(WebInspector.RemoteObject):
(WebInspector.RemoteObject.fromPrimitiveValue):
(WebInspector.RemoteObject.fromPayload):
(WebInspector.RemoteObject.prototype.get subtype):
(WebInspector.RemoteObject.prototype.callFunction):
(WebInspector.LocalJSONObject.prototype.get description):
(WebInspector.LocalJSONObject.prototype.get type):
(WebInspector.LocalJSONObject.prototype.get subtype):
* inspector/front-end/SourceFrame.js:
(WebInspector.SourceFrame.prototype._showPopup.showObjectPopup):
(WebInspector.SourceFrame.prototype._showPopup):
* inspector/front-end/inspector.js:
(WebInspector.log.logMessage):
(WebInspector.log):
(WebInspector.inspect):
2011-07-26 Pavel Feldman <pfeldman@google.com>
Re-applying r91665 and r91668 with JSC issues fixed.
Web Inspector: use dedicated type for object id in the protocol.
Web Inspector: refactor remote object structure to contain value for primitive values.
https://bugs.webkit.org/show_bug.cgi?id=65103
https://bugs.webkit.org/show_bug.cgi?id=65110
Reviewed by Yury Semikhatsky.
* http/tests/inspector/extensions-test.js:
(initialize_ExtensionsTest.InspectorTest.runExtensionTests):
(initialize_ExtensionsTest):
* inspector/console/console-format-expected.txt:
* inspector/console/console-format.html:
* inspector/protocol/console-agent-expected.txt:
* inspector/protocol/runtime-agent-expected.txt:
* inspector/runtime/runtime-callFunctionOn-expected.txt:
* inspector/runtime/runtime-callFunctionOn.html:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91749
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
zimmermann@webkit.org [Tue, 26 Jul 2011 10:05:10 +0000 (10:05 +0000)]
2011-07-26 Nikolas Zimmermann <nzimmermann@rim.com>
Prepare animVal support - phase 2
https://bugs.webkit.org/show_bug.cgi?id=65167
Reviewed by Dirk Schulze.
Next patch preparing animVal support. Modify the SVGAnimatedProperty* classes & macros to assure that calling someElement->someProperty()
will return the animVal, when the property is being animated. Not yet hooked into SVGAnimateElement, this is just a preparation.
Doesn't affect any tests, yet.
* svg/SVGPathElement.cpp: Adapt to SVGAnimatedProperty::lookupWrapper changes (new template parameters have been added).
(WebCore::SVGPathElement::svgAttributeChanged):
* svg/SVGPathSegWithContext.h: Ditto.
(WebCore::SVGPathSegWithContext::animatedProperty):
* svg/SVGPolyElement.cpp: Ditto.
(WebCore::SVGPolyElement::parseMappedAttribute):
* svg/properties/SVGAnimatedEnumerationPropertyTearOff.h: Implement currentAnimatedValue, an abstract way to retrieve the underlying animated value for any animated type.
(WebCore::SVGAnimatedEnumerationPropertyTearOff::currentAnimatedValue): Forward call to SVGAnimatedStaticPropertyTearOff<int>, our base.
* svg/properties/SVGAnimatedListPropertyTearOff.h:
(WebCore::SVGAnimatedListPropertyTearOff::isAnimating): Returns whether this property is being animated or not.
(WebCore::SVGAnimatedListPropertyTearOff::currentAnimatedValue):
* svg/properties/SVGAnimatedProperty.h:
(WebCore::SVGAnimatedProperty::updateAnimVal): Will be used from SVGAnim*Element to update the current animVal, exposed to JS.
(WebCore::SVGAnimatedProperty::lookupOrCreateWrapper): Share more code with lookupWrapper.
(WebCore::SVGAnimatedProperty::lookupWrapper): Cleanup code, avoid doing any work for non-SVGElement derived objects, like SVGViewSpec.
* svg/properties/SVGAnimatedPropertyMacros.h: Make all lowerProperty() calls (eg. myRectElement->x()) return the animVal, if the property is being animated.
* svg/properties/SVGAnimatedPropertyTearOff.h: Main logic for the new animVal stuff.
(WebCore::SVGAnimatedPropertyTearOff::isAnimating):
(WebCore::SVGAnimatedPropertyTearOff::currentAnimatedValue):
(WebCore::SVGAnimatedPropertyTearOff::updateAnimVal):
(WebCore::SVGAnimatedPropertyTearOff::SVGAnimatedPropertyTearOff):
* svg/properties/SVGAnimatedStaticPropertyTearOff.h:
(WebCore::SVGAnimatedStaticPropertyTearOff::isAnimating):
(WebCore::SVGAnimatedStaticPropertyTearOff::currentAnimatedValue):
* svg/properties/SVGPropertyTearOff.h:
(WebCore::SVGPropertyTearOff::updateAnimVal):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91748
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
hans@chromium.org [Tue, 26 Jul 2011 09:39:27 +0000 (09:39 +0000)]
Update another test that relied on fonts loading synchronously.
https://bugs.webkit.org/show_bug.cgi?id=65169
Not Reviewed, tests fix.
* platform/chromium-linux/fast/text/international/arabic-vertical-offset.html:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91747
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Tue, 26 Jul 2011 09:10:19 +0000 (09:10 +0000)]
Source/JavaScriptCore: Added an interface to take IsWhiteSpaceFunctionPtr.
https://bugs.webkit.org/show_bug.cgi?id=57746
Patch by Shinya Kawanaka <shinyak@google.com> on 2011-07-26
Reviewed by Kent Tamura.
* wtf/text/StringImpl.cpp:
(WTF::StringImpl::stripWhiteSpace):
Added an interface to take IsWhiteSpaceFunctionPtr.
(WTF::StringImpl::simplifyWhiteSpace): ditto.
* wtf/text/StringImpl.h:
* wtf/text/WTFString.cpp:
(WTF::String::stripWhiteSpace): ditto.
(WTF::String::simplifyWhiteSpace): ditto.
* wtf/text/WTFString.h:
Source/WebCore: Uses isHTMLSpace to normalize white spaces.
https://bugs.webkit.org/show_bug.cgi?id=57746
Patch by Shinya Kawanaka <shinyak@google.com> on 2011-07-26
Reviewed by Kent Tamura.
Test: fast/forms/option-strip-unicode-spaces.html
* dom/OptionElement.cpp:
(WebCore::OptionElement::normalizeText):
Uses isHTMLSpace to trim and replace white spaces.
LayoutTests: Added a test to normalize option.text.
https://bugs.webkit.org/show_bug.cgi?id=57746
Patch by Shinya Kawanaka <shinyak@google.com> on 2011-07-26
Reviewed by Kent Tamura.
* fast/forms/option-strip-unicode-spaces-expected.txt: Added.
* fast/forms/option-strip-unicode-spaces.html: Added.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91746
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
morrita@google.com [Tue, 26 Jul 2011 07:17:22 +0000 (07:17 +0000)]
[Chromium][DRT] MockSpellCheck.cpp should be clang clean
https://bugs.webkit.org/show_bug.cgi?id=65166
Reviewed by Kent Tamura.
* DumpRenderTree/chromium/MockSpellCheck.cpp:
(MockSpellCheck::spellCheckWord): -Wsign-compare warning.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91745
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Tue, 26 Jul 2011 07:05:52 +0000 (07:05 +0000)]
Add regression test for :empty { display: none }
https://bugs.webkit.org/show_bug.cgi?id=26570
Patch by Noel Gordon <noel.gordon@gmail.com> on 2011-07-26
Reviewed by Alexey Proskuryakov.
* fast/css/pseudo-empty-display-none-expected.txt: Added.
* fast/css/pseudo-empty-display-none.html: Added.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91744
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Tue, 26 Jul 2011 05:35:00 +0000 (05:35 +0000)]
DFG non-speculative JIT emits inefficient code for arithmetic
involving two registers
https://bugs.webkit.org/show_bug.cgi?id=65160
Patch by Filip Pizlo <fpizlo@apple.com> on 2011-07-25
Reviewed by Gavin Barraclough.
The non-speculative JIT now emits inline code for double arithmetic, but
still attempts integer arithmetic first. This is a speed-up on SunSpider
(albeit a small one), and a large speed-up on Kraken.
* dfg/DFGNonSpeculativeJIT.cpp:
(JSC::DFG::NonSpeculativeJIT::basicArithOp):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91743
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
ryuan.choi@samsung.com [Tue, 26 Jul 2011 05:21:06 +0000 (05:21 +0000)]
[CMAKE] Split JSC related files from WebCore/CMakeLists.txt
https://bugs.webkit.org/show_bug.cgi?id=61350
Reviewed by Daniel Bates.
No new tests, just moving jsc related files from CMakeLists.txt to UseJSC.cmake.
* CMakeLists.txt:
* UseJSC.cmake:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91742
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
commit-queue@webkit.org [Tue, 26 Jul 2011 05:11:47 +0000 (05:11 +0000)]
[EFL] Remove unused code in ewk_tile_model.c
https://bugs.webkit.org/show_bug.cgi?id=64521
Remove unused code in ewk_tile_model.c. Dirty flag is not referenced anywhere.
Patch by JungJik Lee <jungjik.lee@samsung.com> on 2011-07-25
Reviewed by Daniel Bates.
* ewk/ewk_tiled_model.c:
(ewk_tile_unused_cache_clear):
(ewk_tile_unused_cache_flush):
(ewk_tile_unused_cache_tile_get):
(ewk_tile_unused_cache_tile_put):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91741
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mitz@apple.com [Tue, 26 Jul 2011 04:44:30 +0000 (04:44 +0000)]
Updated three more tests that relied on fonts loading synchronously.
* fast/blockflow/japanese-rl-text-with-broken-font.html:
* platform/mac/fast/text/myanmar-shaping.html:
* platform/mac/fast/text/thai-combining-mark-positioning.html:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91740
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
mitz@apple.com [Tue, 26 Jul 2011 03:37:26 +0000 (03:37 +0000)]
Fix the build by committing a rename in this file that I neglected to include in the previous commit.
* css/CSSFontFaceSource.h:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@91739
268f45cc-cd09-0410-ab3c-
d52691b4dbfc