1. 19 Aug, 2011 40 commits
    • commit-queue@webkit.org's avatar
      Fix _path_to_apache() error reporting · 24947551
      commit-queue@webkit.org authored
      https://bugs.webkit.org/show_bug.cgi?id=66486
      
      Patch by Tom Zakrajsek <tomz@codeaurora.org> on 2011-08-19
      Reviewed by Ryosuke Niwa.
      
      * Scripts/webkitpy/layout_tests/port/webkit.py:
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93416 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      24947551
    • podivilov@chromium.org's avatar
      Web Inspector: extract breakpoint management code to a separate class and add tests. · d9fc86d9
      podivilov@chromium.org authored
      https://bugs.webkit.org/show_bug.cgi?id=66224
      
      Reviewed by Pavel Feldman.
      
      Source/WebCore:
      
      Test: inspector/debugger/breakpoint-manager.html
      
      * WebCore.gypi:
      * WebCore.vcproj/WebCore.vcproj:
      * inspector/front-end/BreakpointManager.js: Added.
      (WebInspector.BreakpointManager):
      (WebInspector.BreakpointManager.prototype.uiSourceCodeAdded):
      (WebInspector.BreakpointManager.prototype.breakpointsForUISourceCode):
      (WebInspector.BreakpointManager.prototype.setBreakpoint):
      (WebInspector.BreakpointManager.prototype.removeBreakpoint):
      (WebInspector.BreakpointManager.prototype._materializeBreakpoint):
      (WebInspector.BreakpointManager.prototype._breakpointDebuggerLocationChanged):
      (WebInspector.BreakpointManager.prototype._addBreakpointToUI):
      (WebInspector.BreakpointManager.prototype._deleteBreakpointFromUI):
      (WebInspector.BreakpointManager.prototype._moveBreakpointInUI):
      (WebInspector.BreakpointManager.prototype._breakpoints):
      (WebInspector.BreakpointManager.prototype._breakpoint):
      (WebInspector.BreakpointManager.prototype._forEachBreakpoint):
      (WebInspector.BreakpointManager.prototype._setBreakpointInDebugger):
      (WebInspector.BreakpointManager.prototype._removeBreakpointFromDebugger):
      (WebInspector.BreakpointManager.prototype._breakpointResolved):
      (WebInspector.BreakpointManager.prototype.serializeBreakpoints):
      (WebInspector.BreakpointManager.prototype.reset):
      (WebInspector.BreakpointManager.prototype.debuggerReset):
      (WebInspector.Breakpoint):
      (WebInspector.Breakpoint.prototype.serialize):
      (WebInspector.Breakpoint.deserialize):
      * inspector/front-end/DebuggerModel.js:
      (WebInspector.DebuggerModel.prototype.setBreakpointByScriptLocation):
      * inspector/front-end/SourceFile.js:
      (WebInspector.RawSourceCode.prototype.get rawSourceCode):
      * inspector/front-end/WebKit.qrc:
      * inspector/front-end/inspector.html:
      
      LayoutTests:
      
      * inspector/debugger/breakpoint-manager-expected.txt: Added.
      * inspector/debugger/breakpoint-manager.html: Added.
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93415 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      d9fc86d9
    • aroben@apple.com's avatar
      Windows build fix after r93404 · 668673fb
      aroben@apple.com authored
      * TestWebKitAPI/Tests/WebKit2/win/HideFindIndicator.cpp: Added missing #include.
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93414 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      668673fb
    • tonyg@chromium.org's avatar
      Add leandrogracia to the committers list · f8f2b268
      tonyg@chromium.org authored
      https://bugs.webkit.org/show_bug.cgi?id=66552
      
      Also fix line endings in file.
      
      No reviewed required.
      
      * Scripts/webkitpy/common/config/committers.py:
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93413 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      f8f2b268
    • commit-queue@webkit.org's avatar
      Fix compilation issue in WebKitTestRunner · a342460c
      commit-queue@webkit.org authored
      https://bugs.webkit.org/show_bug.cgi?id=66554
      
      Patch by Amruth Raj <amruthraj@motorola.com> on 2011-08-19
      Reviewed by Martin Robinson.
      
      * WebKitTestRunner/PixelDumpSupport.cpp:
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93412 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      a342460c
    • aroben@apple.com's avatar
      Fix typo · f88ac01e
      aroben@apple.com authored
      * dom/DOMAllInOne.cpp:
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93411 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      f88ac01e
    • commit-queue@webkit.org's avatar
      [EFL] Add "return" statement corresponding to abnormal condition on _ewk_frame_smart_add. · 411af019
      commit-queue@webkit.org authored
      https://bugs.webkit.org/show_bug.cgi?id=65408
      
      Bail out when we fail to allocate an Ewk_Frame_Smart_Data object rather than continuing on as if the allocation had succeeded.
      
      Patch by KwangHyuk Kim <hyuki.kim@samsung.com> on 2011-08-19
      Reviewed by Adam Roben.
      
      * ewk/ewk_frame.cpp:
      (_ewk_frame_smart_add):
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93410 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      411af019
    • commit-queue@webkit.org's avatar
      [EFL] Change condition checking logic related with both change of position and... · 47ae27d9
      commit-queue@webkit.org authored
      [EFL] Change condition checking logic related with both change of position and offset for backing store.
      https://bugs.webkit.org/show_bug.cgi?id=66028
      
      As backing store's position and offset change can occur at the same time,
      the checking of offset change shouldn't be "else if" statement.
      
      Patch by KwangHyuk Kim <hyuki.kim@samsung.com> on 2011-08-19
      Reviewed by Adam Roben.
      
      * ewk/ewk_tiled_backing_store.c:
      (_ewk_tiled_backing_store_smart_calculate):
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93409 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      47ae27d9
    • aroben@apple.com's avatar
      Windows build fix after r93385 · c59d63a3
      aroben@apple.com authored
      * dom/DOMAllInOne.cpp: Added EventDispatcMediator.cpp.
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93408 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      c59d63a3
    • commit-queue@webkit.org's avatar
      [Qt][WK2] Add a basic engine to control the content of the viewport · b9a0c591
      commit-queue@webkit.org authored
      https://bugs.webkit.org/show_bug.cgi?id=66383
      
      Patch by Benjamin Poulain <benjamin@webkit.org> on 2011-08-19
      Reviewed by Andreas Kling.
      
      Add ViewportInteractionEngine to handle the content of the viewport. The class make
      sure the QTouchWebPage stays in the viewport and in the boundaries.
      
      * UIProcess/API/qt/qdesktopwebview.cpp:
      (QDesktopWebViewPrivate::loadDidCommit):
      * UIProcess/API/qt/qdesktopwebview_p.h:
      * UIProcess/API/qt/qtouchwebpage.cpp:
      (QTouchWebPage::QTouchWebPage): Set the transform origin to the top left so
      we can asume the position 0, 0 is always the top left of the page.
      (QTouchWebPagePrivate::_q_commitScaleChange): Make commitScaleChange a slot
      in order to completely decouple the ViewportInteractionEngine and the WebKit classes.
      * UIProcess/API/qt/qtouchwebpage.h:
      * UIProcess/API/qt/qtouchwebpage_p.h:
      * UIProcess/API/qt/qtouchwebview.cpp:
      (QTouchWebViewPrivate::QTouchWebViewPrivate):
      (QTouchWebViewPrivate::loadDidCommit):
      (QTouchWebViewPrivate::_q_viewportRectUpdated): Make _q_viewportRectUpdated() a slot
      for reducing coupling.
      (QTouchWebViewPrivate::updateViewportConstraints):
      (QTouchWebViewPrivate::setViewportArguments):
      (QTouchWebView::geometryChanged):
      * UIProcess/API/qt/qtouchwebview.h:
      * UIProcess/API/qt/qtouchwebview_p.h:
      * UIProcess/qt/ClientImpl.cpp:
      (qt_wk_didCommitLoadForFrame):
      * UIProcess/qt/QtGestureRecognizer.cpp:
      (WebKit::QtGestureRecognizer::QtGestureRecognizer):
      * UIProcess/qt/QtGestureRecognizer.h: Change the gesture recognizer to talk directly
      to the ViewportInteractionEngine instead of interacting through TouchViewInterface.
      * UIProcess/qt/QtPanGestureRecognizer.cpp:
      (WebKit::QtPanGestureRecognizer::QtPanGestureRecognizer):
      (WebKit::QtPanGestureRecognizer::recognize):
      * UIProcess/qt/QtPanGestureRecognizer.h:
      * UIProcess/qt/QtPinchGestureRecognizer.cpp:
      (WebKit::QtPinchGestureRecognizer::QtPinchGestureRecognizer):
      (WebKit::QtPinchGestureRecognizer::recognize):
      * UIProcess/qt/QtPinchGestureRecognizer.h:
      * UIProcess/qt/QtWebPageProxy.cpp:
      (QtWebPageProxy::loadDidCommit): Add the loadCommited callback in order
      to reset the viewport state when the page change.
      * UIProcess/qt/QtWebPageProxy.h:
      * UIProcess/qt/TouchViewInterface.cpp:
      (WebKit::TouchViewInterface::TouchViewInterface):
      (WebKit::TouchViewInterface::contentSizeChanged):
      (WebKit::TouchViewInterface::loadDidCommit):
      * UIProcess/qt/TouchViewInterface.h:
      * UIProcess/qt/ViewInterface.h:
      * UIProcess/qt/ViewportInteractionEngine.cpp: Added.
      (WebKit::visibleRectInContentCoordinate):
      (WebKit::contentRectInViewportCoordinate):
      (WebKit::ViewportUpdateGuard::ViewportUpdateGuard):
      (WebKit::ViewportUpdateGuard::~ViewportUpdateGuard):
      (WebKit::ViewportInteractionEngine::ViewportInteractionEngine):
      (WebKit::ViewportInteractionEngine::reset):
      (WebKit::ViewportInteractionEngine::setConstraints):
      (WebKit::ViewportInteractionEngine::panGestureStarted):
      (WebKit::ViewportInteractionEngine::panGestureRequestScroll):
      (WebKit::ViewportInteractionEngine::panGestureCancelled):
      (WebKit::ViewportInteractionEngine::panGestureEnded):
      (WebKit::ViewportInteractionEngine::pinchGestureStarted):
      (WebKit::ViewportInteractionEngine::pinchGestureRequestUpdate):
      (WebKit::ViewportInteractionEngine::pinchGestureEnded):
      (WebKit::ViewportInteractionEngine::contentGeometryChanged):
      (WebKit::ViewportInteractionEngine::contentScaleChanged):
      (WebKit::ViewportInteractionEngine::updateContentIfNeeded):
      (WebKit::ViewportInteractionEngine::updateContentScaleIfNeeded):
      (WebKit::ViewportInteractionEngine::updateContentPositionIfNeeded):
      (WebKit::ViewportInteractionEngine::animateContentPositionToBoundaries):
      (WebKit::ViewportInteractionEngine::animateContentScaleToBoundaries):
      (WebKit::ViewportInteractionEngine::scaleContent):
      * UIProcess/qt/ViewportInteractionEngine.h: Added.
      (WebKit::ViewportInteractionEngine::Constraints::Constraints):
      (WebKit::operator==):
      * UIProcess/qt/qtouchwebpageproxy.cpp:
      (QTouchWebPageProxy::QTouchWebPageProxy):
      * UIProcess/qt/qtouchwebpageproxy.h:
      * WebKit2.pro:
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93407 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      b9a0c591
    • vitalyr@chromium.org's avatar
      2011-08-19 Vitaly Repeshko <vitalyr@chromium.org> · 09fba329
      vitalyr@chromium.org authored
              [chromium] Updating test expectations.
      
              Unreviewed.
      
              * platform/chromium/test_expectations.txt:
      
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93406 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      09fba329
    • aroben@apple.com's avatar
      Pull some of DynamicDeviceScaleFactor's code up into a shareable base class · 124574cf
      aroben@apple.com authored
      This will make it easier to write other WebKit-agnostic tests
      
      Fixes <http://webkit.org/b/66558> Would like to be able to reuse DynamicDeviceScaleFactor's
      code in other WebKit-agnostic tests
      
      Reviewed by Anders Carlsson.
      
      * TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj: Added new files.
      
      * TestWebKitAPI/Tests/mac/DynamicDeviceScaleFactor.mm: Fixed copyright. Moved a bunch of
      code to WebKitAgnosticTest.h/mm. Changed to inherit from WebKitAgnosticTest.
      (TestWebKitAPI::DynamicDeviceScaleFactor::url): Simple getter.
      (TestWebKitAPI::DynamicDeviceScaleFactor::didLoadURL): Calls through to runTest.
      (TestWebKitAPI::DynamicDeviceScaleFactor::runTest): Removed loading code which
      WebKitAgnosticTest handles for us.
      (TestWebKitAPI::TEST_F): Removed view-creation code which WebKitAgnosticTest handles for us.
      
      * TestWebKitAPI/mac/WebKitAgnosticTest.h: Added. Code came from DynamicDeviceScaleFactor.
      (TestWebKitAPI::WebKitAgnosticTest::loadSynchronously): Code came from
      DynamicDeviceScaleFactor::runTest.
      
      * TestWebKitAPI/mac/WebKitAgnosticTest.mm: Copied from Tools/TestWebKitAPI/Tests/mac/DynamicDeviceScaleFactor.mm.
      (TestWebKitAPI::WebKitAgnosticTest::runWebKit1Test): Code came from
      DynamicDeviceScaleFactor.WebKit.
      (TestWebKitAPI::WebKitAgnosticTest::runWebKit2Test): Code came from
      DynamicDeviceScaleFactor.WebKit2.
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93405 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      124574cf
    • aroben@apple.com's avatar
      Clean up #includes in TestWebKitAPI · 8e9bf7c3
      aroben@apple.com authored
      Fixes <http://webkit.org/b/66556> TestWebKitAPI's prefix header isn't very helpful
      
      Reviewed by Anders Carlsson.
      
      * TestWebKitAPI/TestWebKitAPIPrefix.h: Added wtf/Platform.h. Added gtest/gtest.h for C++
      files and WebKit/WebKit.h for ObjC files.
      
      * TestWebKitAPI/InjectedBundleController.cpp:
      * TestWebKitAPI/JavaScriptTest.h:
      * TestWebKitAPI/PlatformUtilities.h:
      * TestWebKitAPI/PlatformWebView.h:
      * TestWebKitAPI/Test.h:
      * TestWebKitAPI/Tests/WTF/StringOperators.cpp:
      * TestWebKitAPI/Tests/WTF/VectorBasic.cpp:
      * TestWebKitAPI/Tests/WTF/VectorReverse.cpp:
      * TestWebKitAPI/Tests/WebKit2/AboutBlankLoad.cpp:
      * TestWebKitAPI/Tests/WebKit2/CanHandleRequest.cpp:
      * TestWebKitAPI/Tests/WebKit2/CookieManager.cpp:
      * TestWebKitAPI/Tests/WebKit2/DocumentStartUserScriptAlertCrash.cpp:
      * TestWebKitAPI/Tests/WebKit2/DocumentStartUserScriptAlertCrash_Bundle.cpp:
      * TestWebKitAPI/Tests/WebKit2/DownloadDecideDestinationCrash.cpp:
      * TestWebKitAPI/Tests/WebKit2/EvaluateJavaScript.cpp:
      * TestWebKitAPI/Tests/WebKit2/FailedLoad.cpp:
      * TestWebKitAPI/Tests/WebKit2/Find.cpp:
      * TestWebKitAPI/Tests/WebKit2/ForceRepaint.cpp:
      * TestWebKitAPI/Tests/WebKit2/FrameMIMETypeHTML.cpp:
      * TestWebKitAPI/Tests/WebKit2/FrameMIMETypePNG.cpp:
      * TestWebKitAPI/Tests/WebKit2/HitTestResultNodeHandle.cpp:
      * TestWebKitAPI/Tests/WebKit2/HitTestResultNodeHandle_Bundle.cpp:
      * TestWebKitAPI/Tests/WebKit2/InjectedBundleBasic.cpp:
      * TestWebKitAPI/Tests/WebKit2/InjectedBundleBasic_Bundle.cpp:
      * TestWebKitAPI/Tests/WebKit2/MouseMoveAfterCrash.cpp:
      * TestWebKitAPI/Tests/WebKit2/PageLoadBasic.cpp:
      * TestWebKitAPI/Tests/WebKit2/PageLoadDidChangeLocationWithinPageForFrame.cpp:
      * TestWebKitAPI/Tests/WebKit2/PreventEmptyUserAgent.cpp:
      * TestWebKitAPI/Tests/WebKit2/ResponsivenessTimerDoesntFireEarly.cpp:
      * TestWebKitAPI/Tests/WebKit2/RestoreSessionStateContainingFormData.cpp:
      * TestWebKitAPI/Tests/WebKit2/SpacebarScrolling.cpp:
      * TestWebKitAPI/Tests/WebKit2/WKPreferences.cpp:
      * TestWebKitAPI/Tests/WebKit2/WKString.cpp:
      * TestWebKitAPI/Tests/WebKit2/WKStringJSString.cpp:
      * TestWebKitAPI/Tests/WebKit2/win/AltKeyGeneratesWMSysCommand.cpp:
      * TestWebKitAPI/Tests/WebKit2/win/DoNotCopyANullCFURLResponse.cpp:
      * TestWebKitAPI/Tests/WebKit2/win/HideFindIndicator.cpp:
      * TestWebKitAPI/Tests/WebKit2/win/ResizeViewWhileHidden.cpp:
      * TestWebKitAPI/Tests/WebKit2/win/TranslateMessageGeneratesWMChar.cpp:
      * TestWebKitAPI/Tests/WebKit2/win/WMCloseCallsUIClientClose.cpp:
      * TestWebKitAPI/Tests/WebKit2/win/WMPrint.cpp:
      * TestWebKitAPI/Tests/mac/DynamicDeviceScaleFactor.mm:
      * TestWebKitAPI/TestsController.cpp:
      * TestWebKitAPI/mac/PlatformUtilitiesMac.mm:
      Removed now-unnecessary #includes. Moved #includes of Test.h in with the rest of the
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93404 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      8e9bf7c3
    • vitalyr@chromium.org's avatar
      2011-08-19 Vitaly Repeshko <vitalyr@chromium.org> · 2af4c7a1
      vitalyr@chromium.org authored
              [chromium] Updating baselines.
      
              Unreviewed.
      
              * platform/chromium-cg-mac-leopard/editing/pasteboard/paste-text-008-expected.png: Added.
              * platform/chromium-cg-mac/editing/pasteboard/merge-end-list-expected.png: Added.
              * platform/chromium-cg-mac/editing/pasteboard/paste-text-008-expected.png: Added.
      
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93403 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      2af4c7a1
    • commit-queue@webkit.org's avatar
      [Qt][WK2] Change the rect argument of QtWebPageProxy::paint() to const ref · f67626d2
      commit-queue@webkit.org authored
      https://bugs.webkit.org/show_bug.cgi?id=66549
      
      Patch by Lars Knudsen <lars.knudsen@nokia.com> on 2011-08-19
      Reviewed by Benjamin Poulain.
      
      * UIProcess/qt/QtWebPageProxy.cpp:
      (QtWebPageProxy::paint):
      * UIProcess/qt/QtWebPageProxy.h:
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93402 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      f67626d2
    • commit-queue@webkit.org's avatar
      [EFL] Fix and update doxygen documentation for ewk_view. · df3f2dbe
      commit-queue@webkit.org authored
      https://bugs.webkit.org/show_bug.cgi?id=66293
      
      Patch by Grzegorz Czajkowski <g.czajkowski@samsung.com> on 2011-08-19
      Reviewed by Kent Tamura.
      
      Adds doxygen documentation to the structures in ewk_view.cpp.
      
      * ewk/ewk_view.cpp:
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93401 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      df3f2dbe
    • caseq@chromium.org's avatar
      Web Inspector: [Extensions API][chromium] webInspector.resources deprecation... · edbdfc24
      caseq@chromium.org authored
      Web Inspector: [Extensions API][chromium] webInspector.resources deprecation warning is given even for extensions that don't use webInspector.resources
      https://bugs.webkit.org/show_bug.cgi?id=66553
      
      Reviewed by Pavel Feldman.
      
      * src/js/DevTools.js:
      (WebInspector.platformExtensionAPI):
      
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93400 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      edbdfc24
    • commit-queue@webkit.org's avatar
      [EFL] Modify type of both col and row parameters for backing store's internal api. · 671301c6
      commit-queue@webkit.org authored
      https://bugs.webkit.org/show_bug.cgi?id=65302
      
      As both col and row parameter's type for both _ewk_tiled_backing_store_item_fill
      and ewk_tiled_backing_store_item_add are different from each other, I modify type of them.
      
      Patch by KwangHyuk Kim <hyuki.kim@samsung.com> on 2011-08-19
      Reviewed by Adam Roben.
      
      * ewk/ewk_tiled_backing_store.c:
      (_ewk_tiled_backing_store_item_fill):
      (_ewk_tiled_backing_store_item_add):
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93399 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      671301c6
    • pfeldman@chromium.org's avatar
      Web Inspector: getAttributes should work on a single node, not array. · be64dd3b
      pfeldman@chromium.org authored
      https://bugs.webkit.org/show_bug.cgi?id=66544
      
      Reviewed by Adam Roben.
      
      * inspector/Inspector.json:
      * inspector/InspectorDOMAgent.cpp:
      (WebCore::InspectorDOMAgent::getAttributes):
      * inspector/InspectorDOMAgent.h:
      * inspector/front-end/DOMAgent.js:
      (WebInspector.DOMAgent.prototype._loadNodeAttributes):
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93398 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      be64dd3b
    • antonm@chromium.org's avatar
      [v8] CSS wrapper objects retention · 30b7aeaf
      antonm@chromium.org authored
      https://bugs.webkit.org/show_bug.cgi?id=66377
      
      Reviewed by Pavel Feldman.
      
      Disable object grouping for CSS object wrappers.
      This logic leads to hard to debug use-after-free problems.
      
      Source/WebCore:
      
      * bindings/v8/V8GCController.cpp:
      (WebCore::GrouperVisitor::visitDOMWrapper):
      
      LayoutTests:
      
      * platform/chromium/test_expectations.txt:
      
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93397 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      30b7aeaf
    • pfeldman@chromium.org's avatar
      Web Inspector: introduce NodeId inherited from integer in the DOM domain description. · a1521dc8
      pfeldman@chromium.org authored
      https://bugs.webkit.org/show_bug.cgi?id=66491
      
      Drive-by rename of couple of DOM domain protocol methods.
      
      Reviewed by Adam Roben.
      
      * inspector/Inspector.json:
      * inspector/InspectorDOMAgent.cpp:
      (WebCore::InspectorDOMAgent::requestChildNodes):
      (WebCore::InspectorDOMAgent::getAttributes):
      (WebCore::InspectorDOMAgent::requestNode):
      (WebCore::InspectorDOMAgent::buildObjectForNode):
      * inspector/InspectorDOMAgent.h:
      * inspector/front-end/DOMAgent.js:
      (WebInspector.DOMNode):
      (WebInspector.DOMNode.prototype.getChildNodes):
      (WebInspector.DOMAgent.prototype.pushNodeToFrontend):
      (WebInspector.DOMAgent.prototype._loadNodeAttributes):
      (WebInspector.DOMAgent.prototype._setDocument):
      (WebInspector.DOMAgent.prototype._setDetachedRoot):
      * inspector/generate-inspector-idl:
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93396 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      a1521dc8
    • jochen@chromium.org's avatar
      IDBSQLiteBackingStore::deleteObjectStore should maintain referential integrity. · cf0b05aa
      jochen@chromium.org authored
      https://bugs.webkit.org/show_bug.cgi?id=66470
      
      Reviewed by Tony Gentilcore.
      
      * storage/IDBSQLiteBackingStore.cpp:
      (WebCore::IDBSQLiteBackingStore::deleteObjectStore):
      
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93395 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      cf0b05aa
    • zoltan@webkit.org's avatar
      [Qt] editing/selection/caret-at-bidi-boundary.html times out after r93369 · 9f2bc91f
      zoltan@webkit.org authored
      https://bugs.webkit.org/show_bug.cgi?id=66514
      
      Skip it.
      
      * platform/qt/Skipped:
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93394 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      9f2bc91f
    • commit-queue@webkit.org's avatar
      [WebSocket] CloseEvent's code and reason properties support. · cb8badea
      commit-queue@webkit.org authored
      https://bugs.webkit.org/show_bug.cgi?id=66362
      
      Patch by Takashi Toyoshima <toyoshim@chromium.org> on 2011-08-19
      Reviewed by Kent Tamura.
      
      Current WebSocket implementation miss code and reason properties
      in CloseEvent. This change expose incoming closing frame's code
      and reason to JavaScript API.
      
      Source/WebCore:
      
      Tests: http/tests/websocket/tests/hybi/close-code-and-reason.html
             http/tests/websocket/tests/hybi/workers/close-code-and-reason.html
      
      * websockets/CloseEvent.h:
      (WebCore::CloseEvent::initCloseEvent):
      (WebCore::CloseEvent::code):
      (WebCore::CloseEvent::reason):
      (WebCore::CloseEvent::CloseEvent):
      * websockets/CloseEvent.idl:
      Add code and reason properies.
      * websockets/ThreadableWebSocketChannelClientWrapper.cpp:
      (WebCore::ThreadableWebSocketChannelClientWrapper::didClose):
      (WebCore::ThreadableWebSocketChannelClientWrapper::didCloseCallback):
      * websockets/ThreadableWebSocketChannelClientWrapper.h:
      * websockets/WebSocket.cpp:
      (WebCore::WebSocket::didConnect):
      (WebCore::WebSocket::didClose):
      * websockets/WebSocket.h:
      Add implement to handle code and reason.
      * websockets/WebSocketChannel.cpp:
      (WebCore::WebSocketChannel::WebSocketChannel):
      (WebCore::WebSocketChannel::didCloseSocketStream):
      (WebCore::WebSocketChannel::processFrame):
      * websockets/WebSocketChannel.h:
      Add closing frame payload parser.
      * websockets/WebSocketChannelClient.h:
      (WebCore::WebSocketChannelClient::didClose):
      * websockets/WorkerThreadableWebSocketChannel.cpp:
      (WebCore::workerContextDidClose):
      (WebCore::WorkerThreadableWebSocketChannel::Peer::didClose):
      * websockets/WorkerThreadableWebSocketChannel.h:
      Add implement to handle code and reason.
      
      LayoutTests:
      
      * http/tests/websocket/tests/hybi/close-code-and-reason-expected.txt: Added.
      * http/tests/websocket/tests/hybi/close-code-and-reason.html: Added.
      * http/tests/websocket/tests/hybi/close-code-and-reason_wsh.py: Added.
      * http/tests/websocket/tests/hybi/workers/close-code-and-reason-expected.txt: Added.
      * http/tests/websocket/tests/hybi/workers/close-code-and-reason.html: Added.
      * http/tests/websocket/tests/hybi/workers/resources/close-code-and-reason.js: Added.
      (postResult):
      (ws.onopen):
      (ws.onmessage):
      (ws.onclose):
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93393 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      cb8badea
    • morrita@google.com's avatar
      .: Spell-checking doesn't recognize word boundaries on contests inserted by... · c74b6a4f
      morrita@google.com authored
      .: Spell-checking doesn't recognize word boundaries on contests inserted by execCommand('insertHTML')
      https://bugs.webkit.org/show_bug.cgi?id=65902
      
      Reviewed by Ryosuke Niwa.
      
      Add export for window.internals object.
      
      * Source/autotools/symbols.filter:
      
      Source/WebCore: Spell-checking doesn't recognize word boundaries on contests inserted by execCommand('insertHTML')
      https://bugs.webkit.org/show_bug.cgi?id=65902
      
      Reviewed by Ryosuke Niwa.
      
      markMisspellingsAndBadGrammar() was using markSpelling() and markBadGrammar().
      But these are low-level API and caller should take care of word boundary.
      This change replaced these call with overloaded version of markMisspellingsAndBadGrammar(),
      which handles word boundary correctly.
      
      Test: editing/spelling/spelling-insert-html.html
      
      * WebCore.exp.in:
      * editing/Editor.cpp:
      (WebCore::Editor::markMisspellingsAndBadGrammar):
      * testing/Internals.cpp:
      (WebCore::Internals::markerCountOf): Added.
      (WebCore::Internals::markedRangeAt): Added.
      * testing/Internals.h:
      * testing/Internals.idl:
      
      Source/WebKit2: Spell-checking against execCommand() inserted HTML doesn't care word boundary.
      https://bugs.webkit.org/show_bug.cgi?id=65902
      
      Reviewed by Ryosuke Niwa.
      
      Add exports for window.internals object.
      
      * win/WebKit2.def:
      * win/WebKit2CFLite.def:
      
      LayoutTests: Spell-checking doesn't recognize word boundaries on contests inserted by execCommand('insertHTML')
      https://bugs.webkit.org/show_bug.cgi?id=65902
      
      Reviewed by Ryosuke Niwa.
      
      Existing expectation was wrong because of some markers on substrings of words.
      With this fix, Editor now rejects such markers.
      
      * editing/spelling/spelling-insert-html-expected.txt: Added.
      * editing/spelling/spelling-insert-html.html: Added.
      * platform/mac/editing/pasteboard/merge-after-delete-1-expected.png:
      * platform/mac/editing/pasteboard/merge-after-delete-2-expected.png:
      * platform/mac/editing/pasteboard/merge-after-delete-expected.png:
      * platform/mac/editing/pasteboard/merge-end-blockquote-expected.png:
      * platform/mac/editing/pasteboard/merge-end-list-expected.png:
      * platform/mac/editing/pasteboard/merge-end-table-expected.png:
      * platform/mac/editing/pasteboard/paste-text-008-expected.png:
      
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93392 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      c74b6a4f
    • commit-queue@webkit.org's avatar
      Run-time error fix in WebKit-GTK with video support disabled. · 443ccb11
      commit-queue@webkit.org authored
      https://bugs.webkit.org/show_bug.cgi?id=66541
      
      Definition of the function extraFullScreenStyleSheet() in webkit-gtk
      lies outside ENABLE(VIDEO) guard, resulting in runtime error while
      trying to launch GtkLauncher with video support disabled.
      
      Patch by Nayan Kumar K <nayankk@motorola.com> on 2011-08-19
      Reviewed by Philippe Normand.
      
      Run-time error fix. No test cases were added.
      
      * platform/gtk/RenderThemeGtk.h:
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93391 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      443ccb11
    • commit-queue@webkit.org's avatar
      input[maxlength=0] should ignore text input. · 14426d87
      commit-queue@webkit.org authored
      https://bugs.webkit.org/show_bug.cgi?id=65497
      
      Patch by Shinya Kawanaka <shinyak@google.com> on 2011-08-19
      Reviewed by Kent Tamura.
      
      Source/WebCore:
      
      Changed the valid range of maxlength.
      
      * html/HTMLInputElement.cpp:
      (WebCore::HTMLInputElement::parseMaxLengthAttribute):
        Changed maxlength check condition.
      
      LayoutTests:
      
      * fast/forms/input-text-paste-maxlength-expected.txt:
        Added a case that maxlength=0
      * fast/forms/input-text-paste-maxlength.html: ditto.
      * fast/forms/script-tests/textarea-maxlength.js: ditto.
      (createFocusedTextAreaWithMaxLength):
      * fast/forms/textarea-maxlength-expected.txt: ditto.
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93390 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      14426d87
    • loislo@chromium.org's avatar
      Web Inspector: backend js api: an ability to skip optional arguments in the... · 3f1e785c
      loislo@chromium.org authored
      Web Inspector: backend js api: an ability to skip optional arguments in the middle of the argument list is required.
      https://bugs.webkit.org/show_bug.cgi?id=66482
      
      There are functions in the API with multiple optional arguments.
      When we call it we have to specify an optional argument as 'undefined' if we want to pass non default value for the next one.
      This can be solved with passing the arguments as an object.
      
      Reviewed by Pavel Feldman.
      
      Source/WebCore:
      
      * inspector/CodeGeneratorInspector.pm:
      * inspector/front-end/RemoteObject.js:
      
      LayoutTests:
      
      * inspector/debugger/debugger-set-breakpoint-regex.html:
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93389 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      3f1e785c
    • zoltan@webkit.org's avatar
      [Qt] Build fix after r93384. · 75112776
      zoltan@webkit.org authored
      * rendering/RenderBlockLineLayout.cpp:
      (WebCore::RenderBlock::LineBreaker::nextLineBreak):
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93388 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      75112776
    • commit-queue@webkit.org's avatar
      Computing screen-space transform for LayerChromium and CCLayerImpl · 17e2cb03
      commit-queue@webkit.org authored
      https://bugs.webkit.org/show_bug.cgi?id=66114
      
      Source/WebCore:
      
      Added a data member to LayerChromium and CCLayerImpl that holds
      the screen-space transform.   The transform is computed in
      calculateDrawTransformsAndVisibility(), which is used during
      updating (LayerChromium) and drawing (CCLayerImpl).
      
      Also fixed a FIXME in LayerRendererChromium::drawLayer, which
      was not computing the entire hierarchy of transforms to determine
      back-face visibility.  Now it simply uses the world space transform.
      
      Patch by Shawn Singh <shawnsingh@chromium.org> on 2011-08-18
      Reviewed by James Robinson.
      
      Test: compositing/backface-visibility-hierarchical-transform.html
      
      * platform/graphics/chromium/LayerChromium.h:
      (WebCore::LayerChromium::screenSpaceTransform):
      (WebCore::LayerChromium::setScreenSpaceTransform):
      * platform/graphics/chromium/LayerRendererChromium.cpp:
      (WebCore::LayerRendererChromium::updateLayers):
      (WebCore::LayerRendererChromium::drawLayersInternal):
      (WebCore::LayerRendererChromium::drawLayer):
      * platform/graphics/chromium/cc/CCLayerImpl.h:
      (WebCore::CCLayerImpl::screenSpaceTransform):
      (WebCore::CCLayerImpl::setScreenSpaceTransform):
      
      LayoutTests:
      
      This case tests if backface visibility properly accounts for a hierarchy
      of transforms.  In particular, this test forces Chromium to create a
      hierarchy of RenderSurfaces.
      
      Patch by Shawn Singh <shawnsingh@chromium.org> on 2011-08-18
      Reviewed by James Robinson.
      
      * compositing/backface-visibility-hierarchical-transform-expected.png: Added.
      * compositing/backface-visibility-hierarchical-transform-expected.txt: Added.
      * compositing/backface-visibility-hierarchical-transform.html: Added.
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93387 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      17e2cb03
    • commit-queue@webkit.org's avatar
      [skia] -webkit-transform breaks -webkit-mask · b86a9ef3
      commit-queue@webkit.org authored
      https://bugs.webkit.org/show_bug.cgi?id=66442
      
      Source/WebCore:
      
      The problem here is that in RenderBox::paintMaskImages, if we are in
      a transform with a rotation, scale or skew we set the composite mode to
      be DestinationIn and then create a transparency layer, then paint the
      mask with SourceOver and end the transparency layer. (The normal case
      is just to use DestinationIn to paint the mask.)
      
      In skia when we create transparency layers we don't pass on the composite
      mode, so when we end the transparency layer it is composited back using
      SourceOver. The fix is to pass on the composite mode when creating
      transparency layers in skia.
      
      Patch by Ben Wells <benwells@chromium.org> on 2011-08-18
      Reviewed by Stephen White.
      
      * platform/graphics/skia/GraphicsContextSkia.cpp:
      (WebCore::GraphicsContext::beginTransparencyLayer):
      * platform/graphics/skia/PlatformContextSkia.cpp:
      (WebCore::PlatformContextSkia::getXfermodeMode):
      * platform/graphics/skia/PlatformContextSkia.h:
      
      LayoutTests:
      
      Patch by Ben Wells <benwells@chromium.org> on 2011-08-18
      Reviewed by Stephen White.
      
      * platform/chromium-linux/fast/css/transformed-mask-expected.png: Added.
      * platform/chromium-linux/fast/css/transformed-mask-expected.txt: Added.
      * platform/chromium/test_expectations.txt:
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93386 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      b86a9ef3
    • hayato@chromium.org's avatar
      Isolate EventDispatchMediator into a separate file. · f4900654
      hayato@chromium.org authored
      https://bugs.webkit.org/show_bug.cgi?id=66458
      
      Reviewed by Ryosuke Niwa.
      
      Extracted EventDispatchMediator.h and EventDispatchMediator.cpp
      out of Event.h and Event.cpp Also moved FocusEventDispatchMediator
      and BlurEventDispatchMediator from Event.{h|cpp} to
      EventDispatchMediator.{h|cpp}.
      
      No new tests since this is just refactoring.
      
      * CMakeLists.txt:
      * GNUmakefile.list.am:
      * WebCore.gypi:
      * WebCore.pro:
      * WebCore.vcproj/WebCore.vcproj:
      * WebCore.xcodeproj/project.pbxproj:
      * dom/Event.cpp:
      * dom/Event.h:
      * dom/EventDispatchMediator.cpp: Added.
      (WebCore::EventDispatchMediator::create):
      (WebCore::EventDispatchMediator::EventDispatchMediator):
      (WebCore::EventDispatchMediator::dispatchEvent):
      (WebCore::FocusEventDispatchMediator::create):
      (WebCore::FocusEventDispatchMediator::FocusEventDispatchMediator):
      (WebCore::FocusEventDispatchMediator::dispatchEvent):
      (WebCore::BlurEventDispatchMediator::create):
      (WebCore::BlurEventDispatchMediator::BlurEventDispatchMediator):
      (WebCore::BlurEventDispatchMediator::dispatchEvent):
      * dom/EventDispatchMediator.h: Added.
      (WebCore::EventDispatchMediator::~EventDispatchMediator):
      (WebCore::EventDispatchMediator::event):
      (WebCore::EventDispatchMediator::EventDispatchMediator):
      (WebCore::EventDispatchMediator::setEvent):
      * dom/EventDispatcher.cpp:
      * dom/KeyboardEvent.h:
      * dom/MouseEvent.h:
      * dom/Node.cpp:
      * dom/ScopedEventQueue.cpp:
      * dom/UIEvent.h:
      * dom/WheelEvent.h:
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93385 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      f4900654
    • mitz@apple.com's avatar
      Mid-word break can occur between a character and a combining mark · ea4fd385
      mitz@apple.com authored
      https://bugs.webkit.org/show_bug.cgi?id=66529
      
      Reviewed by Simon Fraser.
      
      No test added because I could not find a combining mark with a non-zero advance in
      any of the system fonts and tests fonts. 
      
      * rendering/RenderBlockLineLayout.cpp:
      (WebCore::RenderBlock::LineBreaker::nextLineBreak): Disallow a mid-word break before a
      combining mark.
      
      
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93384 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      ea4fd385
    • dpranke@chromium.org's avatar
      new-run-webkit-tests hung while acquiring http lock on snow leopard bots · 8af72e09
      dpranke@chromium.org authored
      https://bugs.webkit.org/show_bug.cgi?id=64886
      
      Temporarily disable the http locking to work around the issue.
      I'm not actually sure if this is going to work or improve things
      much.
      
      Reviewed by Ryosuke Niwa.
      
      * Scripts/webkitpy/layout_tests/port/mac.py:
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93383 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      8af72e09
    • tkent@chromium.org's avatar
      REGRESSION(r90971): Null pointer dereference with placeholder and webkit-scrollbar-corner · 6b9574f6
      tkent@chromium.org authored
      https://bugs.webkit.org/show_bug.cgi?id=66453
      
      Reviewed by Simon Fraser.
      
      Source/WebCore:
      
      Test: fast/forms/placeholder-crash-with-scrollbar-corner.html
      
      * rendering/RenderObjectChildList.cpp:
      (WebCore::RenderObjectChildList::removeChildNode): Add a NULL check for owner's layer.
      
      LayoutTests:
      
      * fast/forms/placeholder-crash-with-scrollbar-corner-expected.txt: Added.
      * fast/forms/placeholder-crash-with-scrollbar-corner.html: Added.
      
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93382 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      6b9574f6
    • koz@chromium.org's avatar
      Chromium plumbing for webkitRequestFullScreen · 49f6dd83
      koz@chromium.org authored
      https://bugs.webkit.org/show_bug.cgi?id=66031
      
      Reviewed by Darin Fisher.
      
      * public/WebView.h:
      * public/WebViewClient.h:
      (WebKit::WebViewClient::enterFullscreenForElement):
      (WebKit::WebViewClient::exitFullscreenForElement):
      * src/ChromeClientImpl.cpp:
      (WebKit::ChromeClientImpl::enterFullScreenForElement):
      (WebKit::ChromeClientImpl::exitFullScreenForElement):
      * src/WebViewImpl.cpp:
      (WebKit::WebViewImpl::exitFullscreen):
      * src/WebViewImpl.h:
      
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93381 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      49f6dd83
    • jpfau@apple.com's avatar
      New XML parser: scripting support · f1856470
      jpfau@apple.com authored
      https://bugs.webkit.org/show_bug.cgi?id=66406
      
      Reviewed by Adam Barth.
      
      * xml/parser/NewXMLDocumentParser.cpp:
      (WebCore::NewXMLDocumentParser::NewXMLDocumentParser):
      (WebCore::NewXMLDocumentParser::resumeParsing):
      (WebCore::NewXMLDocumentParser::processScript):
      (WebCore::NewXMLDocumentParser::append): Add support for pausing the parsing
      (WebCore::NewXMLDocumentParser::finish): Add support for pausing the parsing
      (WebCore::NewXMLDocumentParser::notifyFinished):
      * xml/parser/NewXMLDocumentParser.h:
      (WebCore::NewXMLDocumentParser::pauseParsing):
      * xml/parser/XMLTreeBuilder.cpp: Add a shared function between self-closing and end tags
      (WebCore::XMLTreeBuilder::closeElement): Registers scripts and pops the stack
      (WebCore::XMLTreeBuilder::processStartTag):
      (WebCore::XMLTreeBuilder::processEndTag):
      * xml/parser/XMLTreeBuilder.h:
      
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93380 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      f1856470
    • ryuan.choi@samsung.com's avatar
      [EFL] Broken rendering occured when resized in ewk_view_single. · 1196fdfc
      ryuan.choi@samsung.com authored
      https://bugs.webkit.org/show_bug.cgi?id=66271
      
      Force rendering when ewk_view resized.
      It's because Evas_Object doesn't generate any rendering request although
      it was changed.
      
      Reviewed by Tony Chang.
      
      * ewk/ewk_view.cpp:
      (_ewk_view_smart_calculate):
      
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93379 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      1196fdfc
    • commit-queue@webkit.org's avatar
      Move allocation in constructors into separate constructorBody() methods · 82d6b809
      commit-queue@webkit.org authored
      https://bugs.webkit.org/show_bug.cgi?id=66265
      
      Patch by Mark Hahnenberg <mhahnenberg@apple.com> on 2011-08-18
      Reviewed by Oliver Hunt.
      
      Source/JavaScriptCore:
      
      Refactoring to put all allocations that need to be done after the object's
      initialization list has executed but before the object is ready for use
      into a separate constructorBody() method.  This method is still called by the constructor,
      so the patch doesn't resolve any potential issues, it's just to set up the code for further refactoring.
      
      * JavaScriptCore.exp:
      * JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def:
      * jsc.cpp:
      (GlobalObject::constructorBody):
      (GlobalObject::GlobalObject):
      * runtime/ErrorInstance.cpp:
      (JSC::ErrorInstance::ErrorInstance):
      * runtime/ErrorInstance.h:
      (JSC::ErrorInstance::constructorBody):
      * runtime/ErrorPrototype.cpp:
      (JSC::ErrorPrototype::ErrorPrototype):
      (JSC::ErrorPrototype::constructorBody):
      * runtime/ErrorPrototype.h:
      * runtime/Executable.cpp:
      (JSC::FunctionExecutable::FunctionExecutable):
      * runtime/Executable.h:
      (JSC::FunctionExecutable::constructorBody):
      * runtime/InternalFunction.cpp:
      (JSC::InternalFunction::InternalFunction):
      * runtime/InternalFunction.h:
      (JSC::InternalFunction::constructorBody):
      * runtime/JSByteArray.cpp:
      (JSC::JSByteArray::JSByteArray):
      * runtime/JSByteArray.h:
      (JSC::JSByteArray::constructorBody):
      * runtime/JSFunction.cpp:
      (JSC::JSFunction::JSFunction):
      (JSC::JSFunction::constructorBody):
      * runtime/JSFunction.h:
      * runtime/JSGlobalObject.h:
      (JSC::JSGlobalObject::JSGlobalObject):
      (JSC::JSGlobalObject::constructorBody):
      * runtime/JSPropertyNameIterator.cpp:
      (JSC::JSPropertyNameIterator::JSPropertyNameIterator):
      * runtime/JSPropertyNameIterator.h:
      (JSC::JSPropertyNameIterator::constructorBody):
      * runtime/JSString.h:
      (JSC::RopeBuilder::JSString):
      (JSC::RopeBuilder::constructorBody):
      * runtime/NativeErrorConstructor.cpp:
      (JSC::NativeErrorConstructor::NativeErrorConstructor):
      * runtime/NativeErrorConstructor.h:
      (JSC::NativeErrorConstructor::constructorBody):
      * runtime/NativeErrorPrototype.cpp:
      (JSC::NativeErrorPrototype::NativeErrorPrototype):
      (JSC::NativeErrorPrototype::constructorBody):
      * runtime/NativeErrorPrototype.h:
      * runtime/StringObject.cpp:
      * runtime/StringObject.h:
      (JSC::StringObject::create):
      * runtime/StringObjectThatMasqueradesAsUndefined.h:
      (JSC::StringObjectThatMasqueradesAsUndefined::create):
      (JSC::StringObjectThatMasqueradesAsUndefined::StringObjectThatMasqueradesAsUndefined):
      * runtime/StringPrototype.cpp:
      (JSC::StringPrototype::StringPrototype):
      * runtime/StringPrototype.h:
      (JSC::StringPrototype::create):
      
      Source/WebCore:
      
      No new tests.
      
      Refactoring to put all allocations that need to be done after the object's
      initialization list has executed but before the object is ready for use
      into a separate constructorBody() method.  This method is still called by the constructor,
      so the patch doesn't resolve any potential issues, it's just to set up the code for further refactoring.
      
      * bridge/objc/ObjCRuntimeObject.h:
      (JSC::Bindings::ObjCRuntimeObject::create):
      * bridge/objc/ObjCRuntimeObject.mm:
      * bridge/objc/objc_instance.mm:
      (ObjCRuntimeMethod::create):
      (ObjCRuntimeMethod::ObjCRuntimeMethod):
      * bridge/runtime_array.cpp:
      * bridge/runtime_array.h:
      (JSC::RuntimeArray::create):
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93378 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      82d6b809
    • bdakin@apple.com's avatar
      Source/WebCore: https://bugs.webkit.org/show_bug.cgi?id=66495 · b7dfd27f
      bdakin@apple.com authored
      Lion-specific scroller SPIs can use forward declaration instead of 
      WebKitSystemInterface
      
      Reviewed by Sam Weinig.
      
      * WebCore.exp.in:
      * WebCore.xcodeproj/project.pbxproj:
      * platform/mac/NSScrollerImpDetails.h: Added.
      * platform/mac/ScrollAnimatorMac.h:
      * platform/mac/ScrollAnimatorMac.mm:
      (-[ScrollbarPainterControllerDelegate scrollerImpPair:convertContentPoint:toScrollerImp:]):
      (-[ScrollbarPainterControllerDelegate scrollerImpPair:updateScrollerStyleForNewRecommendedScrollerStyle:]):
      (-[ScrollbarPartAnimation initWithScrollbarPainter:part:WebCore::scrollAnimator:WebCore::animateAlphaTo:duration:]):
      (-[ScrollbarPartAnimation setCurrentProgress:]):
      (-[ScrollbarPainterDelegate setUpAnimation:scrollerPainter:part:WebCore::animateAlphaTo:duration:]):
      (-[ScrollbarPainterDelegate scrollerImp:animateKnobAlphaTo:duration:]):
      (-[ScrollbarPainterDelegate scrollerImp:animateTrackAlphaTo:duration:]):
      (WebCore::ScrollAnimatorMac::ScrollAnimatorMac):
      (WebCore::ScrollAnimatorMac::~ScrollAnimatorMac):
      (WebCore::ScrollAnimatorMac::notifyPositionChanged):
      (WebCore::ScrollAnimatorMac::contentAreaWillPaint):
      (WebCore::ScrollAnimatorMac::mouseEnteredContentArea):
      (WebCore::ScrollAnimatorMac::mouseExitedContentArea):
      (WebCore::ScrollAnimatorMac::mouseMovedInContentArea):
      (WebCore::ScrollAnimatorMac::willStartLiveResize):
      (WebCore::ScrollAnimatorMac::contentsResized):
      (WebCore::ScrollAnimatorMac::willEndLiveResize):
      (WebCore::ScrollAnimatorMac::contentAreaDidShow):
      (WebCore::ScrollAnimatorMac::contentAreaDidHide):
      (WebCore::ScrollAnimatorMac::didBeginScrollGesture):
      (WebCore::ScrollAnimatorMac::didEndScrollGesture):
      (WebCore::ScrollAnimatorMac::didAddVerticalScrollbar):
      (WebCore::ScrollAnimatorMac::willRemoveVerticalScrollbar):
      (WebCore::ScrollAnimatorMac::didAddHorizontalScrollbar):
      (WebCore::ScrollAnimatorMac::willRemoveHorizontalScrollbar):
      (WebCore::ScrollAnimatorMac::updateScrollerStyle):
      (WebCore::ScrollAnimatorMac::initialScrollbarPaintTimerFired):
      * platform/mac/ScrollbarThemeMac.h:
      * platform/mac/ScrollbarThemeMac.mm:
      (WebCore::ScrollbarThemeMac::registerScrollbar):
      (WebCore::ScrollbarThemeMac::scrollbarThickness):
      (WebCore::ScrollbarThemeMac::usesOverlayScrollbars):
      (WebCore::ScrollbarThemeMac::updateScrollbarOverlayStyle):
      (WebCore::ScrollbarThemeMac::hasThumb):
      (WebCore::ScrollbarThemeMac::minimumThumbLength):
      (WebCore::ScrollbarThemeMac::updateEnabledState):
      (WebCore::scrollbarPainterPaint):
      (WebCore::ScrollbarThemeMac::paint):
      * platform/mac/WebCoreSystemInterface.h:
      * platform/mac/WebCoreSystemInterface.mm:
      
      Source/WebKit/mac: Reviewed by Sam Weinig.
      
      https://bugs.webkit.org/show_bug.cgi?id=66495
      Lion-specific scroller SPIs can use forward declaration instead of 
      WebKitSystemInterface
      
      * WebCoreSupport/WebSystemInterface.mm:
      (InitWebCoreSystemInterface):
      
      Source/WebKit2: Reviewed by Sam Weinig.
      
      https://bugs.webkit.org/show_bug.cgi?id=66495
      Lion-specific scroller SPIs can use forward declaration instead of 
      WebKitSystemInterface
      
      * WebProcess/WebCoreSupport/mac/WebSystemInterface.mm:
      (InitWebCoreSystemInterface):
      
      WebKitLibraries: Reviewed by Sam Weinig.
      
      https://bugs.webkit.org/show_bug.cgi?id=66495
      Lion-specific scroller SPIs can use forward declaration instead of 
      WebKitSystemInterface
      
      * WebKitSystemInterface.h:
      * libWebKitSystemInterfaceLeopard.a:
      * libWebKitSystemInterfaceLion.a:
      * libWebKitSystemInterfaceSnowLeopard.a:
      
      
      
      git-svn-id: svn://svn.chromium.org/blink/trunk@93377 bbb929c8-8fbe-4397-9dbb-9b2b20218538
      b7dfd27f