⚠ Archived content — this site is no longer maintained.   Current WebKit documentation is at docs.webkit.org.

Timeline



Mar 28, 2010:

9:11 PM Changeset in webkit [56706] by ap@apple.com
  • 2 edits in trunk/WebKit/chromium

Build fix. Removed extraneous includes of KeyboardCodesWin.h and KeyboardCodesPosix.h -
these no longer exist, but they weren't needed even before reshuffling KeyboardCodes headers.

8:53 PM Changeset in webkit [56705] by ap@apple.com
  • 10 edits in trunk/WebKit

Build fix. Include WindowsKeyboardCodes.h instead of KeyboardCodes.h.

8:41 PM Changeset in webkit [56704] by ap@apple.com
  • 13 edits
    1 move
    1 add
    5 deletes in trunk/WebCore

Reviewed by Sam Weinig.

​https://bugs.webkit.org/show_bug.cgi?id=36733
Use a common header for Windows virtual key codes on all platforms

No change in functionality, so no new tests.

  • platform/WindowsKeyboardCodes.h: Added. Define VK_* key codes as macros, matching Windows.
  • platform/KeyboardCodes.h: Removed.
  • platform/android/KeyboardCodes.h: Removed.
  • platform/chromium/KeyboardCodesPosix.h: Removed.
  • platform/chromium/KeyboardCodesWin.h: Removed.
  • platform/haiku/KeyboardCodes.h: Removed.
  • platform/wx/KeyboardCodes.h: Removed. Removed platform specific attempts to do the same thing. In fact, platform/KeyboardCodes.h was supposed to be cross-platform, but it didn't have a sufficiently descriptive name, and its name was the same as platform-specific versions, so I chose a new one.
  • platform/chromium/KeyboardCodes.h: Copied from WebCore/platform/chromium/KeyboardCodesWin.h. Chromium redefines Windows constants to different names; keeping the header doing that to avoid breaking Chromium code.
  • GNUmakefile.am:
  • WebCore.gypi: Replaced KeyboardCodes.h with WindowsKeyboardCodes.h.
  • WebCore.xcodeproj/project.pbxproj:
  • platform/cocoa/KeyEventCocoa.mm: (WebCore::windowsKeyCodeForKeyCode): (WebCore::windowsKeyCodeForCharCode): Mac used to hardcode the key codes; changed to use VK_* macros.
  • config.h: Removed a hack to include KeyboardCodes.h, it shouldn't be necessary any more.
  • platform/android/KeyEventAndroid.cpp:
  • platform/brew/PlatformKeyboardEventBrew.cpp:
  • platform/efl/PlatformKeyboardEventEfl.cpp:
  • platform/gtk/KeyEventGtk.cpp:
  • platform/haiku/PlatformKeyboardEventHaiku.cpp:
  • platform/qt/PlatformKeyboardEventQt.cpp:
  • platform/wx/KeyboardEventWx.cpp: Changed to include WindowsKeyboardCodes.h.
3:48 PM Changeset in webkit [56703] by kov@webkit.org
  • 2 edits in trunk/LayoutTests

2010-03-28 Gustavo Noronha Silva <Gustavo Noronha Silva>

[GTK] Improve reporting of frame loader callbacks in DRT
​https://bugs.webkit.org/show_bug.cgi?id=36454

Skip tests that need more load notifications than we provide in
our DRT currently.

  • platform/gtk/Skipped:
3:36 PM Changeset in webkit [56702] by kov@webkit.org
  • 2 edits in trunk/LayoutTests

2010-03-28 Gustavo Noronha Silva <Gustavo Noronha Silva>

Skipping new tests.

  • platform/gtk/Skipped:
3:29 PM Changeset in webkit [56701] by kov@webkit.org
  • 2 edits in trunk/WebKitTools

2010-03-28 Gustavo Noronha Silva <Gustavo Noronha Silva>

No review, rolling out r56679.
​http://trac.webkit.org/changeset/56679
​https://bugs.webkit.org/show_bug.cgi?id=36454

Lots of tests broken.

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (createWebView):
2:41 PM Changeset in webkit [56700] by eric@webkit.org
  • 4 edits in trunk

2010-03-28 Laszlo Gombos <Laszlo Gombos>

Reviewed by Simon Hausmann.

[Qt] Remove the definition of WTF_CHANGES guards from the build system
​https://bugs.webkit.org/show_bug.cgi?id=31670

  • JavaScriptCore.pro: Remove the definition of WTF_CHANGES as it is already defined in config.h

2010-03-28 Laszlo Gombos <Laszlo Gombos>

Reviewed by Simon Hausmann.

[Qt] Remove the definition of WTF_CHANGES guards from the build system
​https://bugs.webkit.org/show_bug.cgi?id=31670

No new tests as there is no new functionality.

  • WebCore.pri: Remove the definition of WTF_CHANGES as it is already defined in config.h
1:00 PM QtWebKitJournal edited by Simon Hausmann
(diff)
12:19 PM Changeset in webkit [56699] by Chris Jerdonek
  • 2 edits in trunk/WebKitTools

Unreviewed.

Fixed typo in deprecated_logging_unittest.py, which is masking
sys.stderr while running test-webkitpy.

See also-- ​https://bugs.webkit.org/show_bug.cgi?id=36725#c3

  • Scripts/webkitpy/common/system/deprecated_logging_unittest.py:
10:47 AM Changeset in webkit [56698] by weinig@apple.com
  • 2 edits in trunk/WebCore

Clean up the Xcode project a bit.

Reviewed by Dan Bernstein.

  • WebCore.xcodeproj/project.pbxproj:
10:41 AM Changeset in webkit [56697] by pfeldman@chromium.org
  • 6 edits in trunk/WebCore

2010-03-28 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Toggling style properties on/off does not always work.

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

  • inspector/InspectorController.cpp:
  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::toggleStyleEnabled): (WebCore::InspectorDOMAgent::buildObjectForStyle): (WebCore::InspectorDOMAgent::buildArrayForDisabledStyleProperties):
  • inspector/InspectorDOMAgent.h:
  • inspector/front-end/DOMAgent.js: (WebInspector.CSSStyleDeclaration):
10:32 AM Changeset in webkit [56696] by pfeldman@chromium.org
  • 3 edits in trunk/WebCore

2010-03-28 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: resource links should navigate to resource panel enabler when resource tracking is off.

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

9:59 AM Changeset in webkit [56695] by krit@webkit.org
  • 2 edits in trunk/LayoutTests

2010-03-28 Dirk Schulze <​krit@webkit.org>

Unreviewed test update for Win.

Out of bounds read in SVG feColorMatrix filter
​https://bugs.webkit.org/show_bug.cgi?id=32714

May mixed up the win bot with the win-chromium bot.

  • platform/win/svg/filters/feColorMatrix-values-expected.txt:
9:30 AM Changeset in webkit [56694] by krit@webkit.org
  • 6 edits in trunk

2010-03-28 Dirk Schulze <​krit@webkit.org>

Unreviewed build-fix for chromium linux.

SVGResourceMarker needs to be moved to RenderersSVGResourceMarker
​https://bugs.webkit.org/show_bug.cgi?id=36185

  • svg/SVGStyledElement.cpp: (WebCore::SVGStyledElement::invalidateResources):

2010-03-28 Dirk Schulze <​krit@webkit.org>

Unreviewed test update.

SVGResourceMarker needs to be moved to RenderersSVGResourceMarker
​https://bugs.webkit.org/show_bug.cgi?id=36185

Marker results update for Qt port.

  • platform/qt/svg/custom/marker-opacity-expected.txt:
  • platform/qt/svg/custom/marker-referencePoint-expected.txt:
  • platform/qt/svg/custom/marker-strokeWidth-changes-expected.txt:
9:01 AM Changeset in webkit [56693] by krit@webkit.org
  • 43 edits
    2 adds
    2 deletes in trunk

2010-03-28 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVGResourceMarker needs to be moved to RenderersSVGResourceMarker
​https://bugs.webkit.org/show_bug.cgi?id=36185

This moves SVGResourceMarker to RenderersSVGResourceMarker. This follows other
SVG resources like masker and clipper and makes DRT results more readable and
adds additonal informations.
Some marker specific code also moved out from RenderSVGViewportContainer to
RenderersSVGResourceMarker.

  • Android.mk:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • rendering/RenderPath.cpp: (WebCore::RenderPath::calculateMarkerBoundsIfNeeded):
  • rendering/RenderSVGImage.cpp: (WebCore::RenderSVGImage::destroy):
  • rendering/RenderSVGResource.h: (WebCore::):
  • rendering/RenderSVGResourceMarker.cpp: Added. (WebCore::RenderSVGResourceMarker::RenderSVGResourceMarker): (WebCore::RenderSVGResourceMarker::~RenderSVGResourceMarker): (WebCore::RenderSVGResourceMarker::layout): (WebCore::RenderSVGResourceMarker::addClient): (WebCore::RenderSVGResourceMarker::invalidateClients): (WebCore::RenderSVGResourceMarker::invalidateClient): (WebCore::RenderSVGResourceMarker::applyViewportClip): (WebCore::RenderSVGResourceMarker::markerBoundaries): (WebCore::RenderSVGResourceMarker::localToParentTransform): (WebCore::RenderSVGResourceMarker::referencePoint): (WebCore::RenderSVGResourceMarker::angle): (WebCore::RenderSVGResourceMarker::markerTransformation): (WebCore::RenderSVGResourceMarker::draw): (WebCore::RenderSVGResourceMarker::markerContentTransformation): (WebCore::RenderSVGResourceMarker::viewportTransform): (WebCore::RenderSVGResourceMarker::calcViewport):
  • rendering/RenderSVGResourceMarker.h: Added. (WebCore::RenderSVGResourceMarker::renderName): (WebCore::RenderSVGResourceMarker::applyResource): (WebCore::RenderSVGResourceMarker::resourceBoundingBox): (WebCore::RenderSVGResourceMarker::markerUnits): (WebCore::RenderSVGResourceMarker::resourceType):
  • rendering/RenderSVGViewportContainer.cpp: (WebCore::RenderSVGViewportContainer::calcViewport): (WebCore::RenderSVGViewportContainer::viewportTransform):
  • rendering/RenderSVGViewportContainer.h:
  • rendering/SVGMarkerData.h: (WebCore::SVGMarkerData::SVGMarkerData): (WebCore::SVGMarkerData::marker): (WebCore::SVGMarkerData::updateTypeAndMarker):
  • rendering/SVGMarkerLayoutInfo.cpp: (WebCore::processStartAndMidMarkers): (WebCore::SVGMarkerLayoutInfo::calculateBoundaries): (WebCore::SVGMarkerLayoutInfo::addLayoutedMarker):
  • rendering/SVGMarkerLayoutInfo.h: (WebCore::MarkerLayout::MarkerLayout): (WebCore::SVGMarkerLayoutInfo::midMarker):
  • rendering/SVGRenderSupport.cpp: (WebCore::deregisterFromResources):
  • rendering/SVGRenderSupport.h:
  • rendering/SVGRenderTreeAsText.cpp: (WebCore::operator<<): (WebCore::writeSVGResource):
  • svg/SVGMarkerElement.cpp: (WebCore::SVGMarkerElement::svgAttributeChanged): (WebCore::SVGMarkerElement::childrenChanged): (WebCore::SVGMarkerElement::setOrientToAuto): (WebCore::SVGMarkerElement::setOrientToAngle): (WebCore::SVGMarkerElement::createRenderer):
  • svg/SVGMarkerElement.h: (WebCore::SVGMarkerElement::):
  • svg/SVGStyledElement.cpp: (WebCore::SVGStyledElement::invalidateResources):
  • svg/graphics/SVGResource.h: (WebCore::): (WebCore::SVGResource::isFilter):
  • svg/graphics/SVGResourceMarker.cpp: Removed.
  • svg/graphics/SVGResourceMarker.h: Removed.

2010-03-28 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVGResourceMarker needs to be moved to RenderersSVGResourceMarker
​https://bugs.webkit.org/show_bug.cgi?id=36185

DRT results changed after moving SVGResourceMarker and need updates.

  • platform/mac/svg/W3C-SVG-1.1/painting-marker-01-f-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/painting-marker-02-f-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/painting-marker-03-f-expected.txt:
  • platform/mac/svg/custom/circular-marker-reference-1-expected.txt:
  • platform/mac/svg/custom/circular-marker-reference-2-expected.txt:
  • platform/mac/svg/custom/circular-marker-reference-3-expected.txt:
  • platform/mac/svg/custom/circular-marker-reference-4-expected.txt:
  • platform/mac/svg/custom/invalid-css-expected.txt:
  • platform/mac/svg/custom/js-late-marker-and-object-creation-expected.txt:
  • platform/mac/svg/custom/js-late-marker-creation-expected.txt:
  • platform/mac/svg/custom/marker-changes-expected.txt:
  • platform/mac/svg/custom/marker-child-changes-expected.txt:
  • platform/mac/svg/custom/marker-default-width-height-expected.txt:
  • platform/mac/svg/custom/marker-opacity-expected.txt:
  • platform/mac/svg/custom/marker-overflow-clip-expected.txt:
  • platform/mac/svg/custom/marker-referencePoint-expected.txt:
  • platform/mac/svg/custom/marker-strokeWidth-changes-expected.txt:
  • platform/mac/svg/custom/marker-viewBox-changes-expected.txt:
  • platform/mac/svg/custom/non-circular-marker-reference-expected.txt:
  • platform/mac/svg/custom/shapes-supporting-markers-expected.txt:
8:23 AM Changeset in webkit [56692] by Chris Jerdonek
  • 10 edits in trunk/WebKitTools

Renamed check-webkit-style's --debug flag to --verbose to be more
in line with other WebKit scripts. Also renamed the current
--verbose flag to --min-confidence to allow the --debug rename.

Reviewed by Shinichiro Hamaji.

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

We also renamed the internal "verbose" variables to "confidence" or
"min_confidence," as appropriate, to avoid confusion with the
--verbose flag, and because the new names are more accurate.

  • Scripts/check-webkit-style:
    • Renamed is_debug to is_verbose.
  • Scripts/webkitpy/style/checker.py:
    • Renamed _DEFAULT_VERBOSITY to _DEFAULT_CONFIDENCE.
    • Renamed "verbosity" parameters to "min_confidence" throughout.
    • Renamed configure_logging()'s is_debug parameter to is_verbose.
  • Scripts/webkitpy/style/checker_unittest.py:
    • Updated the unit tests as necessary.
  • Scripts/webkitpy/style/error_handlers.py:
    • Updated the call to StyleCheckerConfiguration.write_style_error().
  • Scripts/webkitpy/style/error_handlers_unittest.py:
    • Updated the unit tests as necessary.
  • Scripts/webkitpy/style/optparser.py:
    • Updated the usage string with the new flag names.
    • Renamed the verbosity parameter to min_confidence throughout.
    • Renamed the is_debug parameter to is_verbose throughout.
  • Scripts/webkitpy/style/optparser_unittest.py:
    • Updated the unit tests as necessary.
  • Scripts/webkitpy/style/processors/cpp.py:
    • Renamed the verbosity parameter to min_confidence throughout.
  • Scripts/webkitpy/style/processors/cpp_unittest.py:
    • Updated the unit tests as necessary.
8:05 AM Changeset in webkit [56691] by Chris Jerdonek
  • 2 edits in trunk/WebKitTools

Unreviewed.

Added back a line that accidentally got deleted in r56690.

  • Scripts/check-webkit-style:
7:51 AM Changeset in webkit [56690] by Chris Jerdonek
  • 5 edits in trunk/WebKitTools

Add to check-webkit-style support for checking directories.

Reviewed by Shinichiro Hamaji.

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

  • Scripts/check-webkit-style:
    • Replaced the call to check_file() with a call to check_paths().
  • Scripts/webkitpy/style/checker.py:
    • In the StyleChecker class:
      • Added a check_paths() method that accepts a list of paths to files and directories.
      • Added a _check_directory() method that checks the files in a directory.
  • Scripts/webkitpy/style/checker_unittest.py:
    • Added a StyleCheckerCheckPathsTest to unit-test the new check_paths() method.
  • Scripts/webkitpy/style/optparser.py:
    • Updated the usage string.
3:43 AM Changeset in webkit [56689] by eric@webkit.org
  • 5 edits in trunk/JavaScriptCore

2010-03-28 Kent Hansen <​kent.hansen@nokia.com>

Reviewed by Simon Hausmann.

[Qt] Add API for reporting additional memory cost of JavaScript objects
​https://bugs.webkit.org/show_bug.cgi?id=36650

  • qt/api/qscriptengine.cpp: (QScriptEngine::reportAdditionalMemoryCost):
  • qt/api/qscriptengine.h:
  • qt/api/qscriptengine_p.h: (QScriptEnginePrivate::reportAdditionalMemoryCost):
  • qt/tests/qscriptengine/tst_qscriptengine.cpp: (tst_QScriptEngine::reportAdditionalMemoryCost):
3:33 AM Changeset in webkit [56688] by Joseph Pecoraro
  • 2 edits in trunk/WebCore

2010-03-28 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Pavel Feldman.

Web Inspector: Consolidate Tree Hierarchy Attribute HTML Generation
​https://bugs.webkit.org/show_bug.cgi?id=36719

Consolidates the generation of Element Tree Hierarchy attribute code
generation to a single function: _attributeHTML.

  • inspector/front-end/ElementsTreeOutline.js: (WebInspector.ElementsTreeElement.prototype._addNewAttribute): convert to use attributeHTML. (WebInspector.ElementsTreeElement.prototype._attributeEditingCommitted.regenerateStyledAttribute): convert to use attributeHTML. (WebInspector.ElementsTreeElement.prototype._attributeEditingCommitted): (WebInspector.ElementsTreeElement.prototype._attributeHTML): a single point for generating the html for attributes.
3:33 AM Changeset in webkit [56687] by Joseph Pecoraro
  • 2 edits in trunk/WebCore

2010-03-28 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Pavel Feldman.

Web Inspector: Edit Tag Names
​https://bugs.webkit.org/show_bug.cgi?id=36481

Ensure blacklisting happens by checking it in _startEditingTagName
instead of all the possible entry points.

  • inspector/front-end/ElementsTreeOutline.js: (WebInspector.ElementsTreeElement.prototype._startEditingFromEvent): don't check blacklist here. (WebInspector.ElementsTreeElement.prototype._startEditingTagName): check blacklist here.
3:03 AM Changeset in webkit [56686] by eric@webkit.org
  • 6 edits in trunk/JavaScriptCore

2010-03-28 Jedrzej Nowacki <​jedrzej.nowacki@nokia.com>

Reviewed by Simon Hausmann.

QScriptEngine API was enriched by globalObject() method
which give an access to the global object.

[Qt] QScriptEngine doesn't give an access to global object
​https://bugs.webkit.org/show_bug.cgi?id=36603

  • qt/api/qscriptengine.cpp: (QScriptEngine::globalObject):
  • qt/api/qscriptengine.h:
  • qt/api/qscriptengine_p.cpp: (QScriptEnginePrivate::globalObject):
  • qt/api/qscriptengine_p.h:
  • qt/tests/qscriptengine/tst_qscriptengine.cpp: (tst_QScriptEngine::globalObject):
2:46 AM Changeset in webkit [56685] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-03-28 Kim Grönholm <​kim.gronholm@nomovok.com>

Reviewed by Simon Hausmann.

[Qt] GraphicsLayer: Timing functions don't work with transitions
​https://bugs.webkit.org/show_bug.cgi?id=36589

If the animation value doesn't have timing function set, we need to
use the animation's timing function.

Fixed also a bug in passing the duration to solveCubicBezierFunction.
The duration was divided two times by 1000 and meanwhile casted to int.
It needs to be kept as double and divided by 1000 only once.

Test case: ​https://bugs.webkit.org/attachment.cgi?id=51619

  • platform/graphics/qt/GraphicsLayerQt.cpp: (WebCore::applyTimingFunction): (WebCore::AnimationQt:::AnimationQtBase): (WebCore::AnimationQt::updateCurrentTime):
1:49 AM Changeset in webkit [56684] by Joseph Pecoraro
  • 3 edits in trunk/WebCore

2010-03-27 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Pavel Feldman.

Web Inspector: Edit Tag Names
​https://bugs.webkit.org/show_bug.cgi?id=36481

Allow tabbing back and forth between the tag name and element
attributes when editing. Also, fixed a number of minor issues with
editing attributes and tabbing.

  • inspector/front-end/ElementsTreeOutline.js: (WebInspector.ElementsTreeElement.prototype._startEditingTagName): find the tag name if it wasn't provided. (WebInspector.ElementsTreeElement.prototype._attributeEditingCommitted): fix tabbing issues, enable tab to tag name. (WebInspector.ElementsTreeElement.prototype._attributeEditingCommitted.regenerateStyledAttribute): cleanup styles when tabbing.
  • inspector/front-end/inspector.css: do not display <br>s while editing
1:49 AM Changeset in webkit [56683] by Joseph Pecoraro
  • 11 edits in trunk/WebCore

2010-03-27 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Pavel Feldman.

Web Inspector: Edit Tag Names
​https://bugs.webkit.org/show_bug.cgi?id=36481

Allow editing an Element's Tag Name by double clicking
on the tag name in the Element's Tree Hierarchy.

The usual asynchronous InspectorBackend, InspectorDOMAgent, InspectorFrontend flow.

  • inspector/InspectorBackend.cpp: moved DOM manipulation to InspectorDOMAgent (WebCore::InspectorBackend::removeNode): (WebCore::InspectorBackend::changeTagName):
  • inspector/InspectorBackend.h:
  • inspector/InspectorBackend.idl:
  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::removeNode): (WebCore::InspectorDOMAgent::changeTagName):
  • inspector/InspectorDOMAgent.h:
  • inspector/InspectorFrontend.cpp: (WebCore::InspectorFrontend::didChangeTagName):
  • inspector/InspectorFrontend.h:
  • inspector/front-end/DOMAgent.js:

Handle the UI for editing an Element's tag name.

  • inspector/front-end/ElementsTreeOutline.js: (WebInspector.ElementsTreeElement.prototype._startEditingFromEvent): allow editing from double click. (WebInspector.ElementsTreeElement.prototype._startEditingTagName.keyupListener): update the closing tag (WebInspector.ElementsTreeElement.prototype._startEditingTagName.editingComitted): remove extra listener and commit (WebInspector.ElementsTreeElement.prototype._startEditingTagName.editingCancelled): remove extra listener and cancel (WebInspector.ElementsTreeElement.prototype._startEditingTagName): (WebInspector.ElementsTreeElement.prototype._tagNameEditingCommitted.cancel): (WebInspector.ElementsTreeElement.prototype._tagNameEditingCommitted.moveToNextAttributeIfNeeded): (WebInspector.ElementsTreeElement.prototype._tagNameEditingCommitted.editTagNameCallback): (WebInspector.ElementsTreeElement.prototype._tagNameEditingCommitted): (WebInspector.ElementsTreeElement.prototype._distinctClosingTagElement): get the closing tag for an opening tag (WebInspector.ElementsTreeElement.prototype._nodeTitleInfo): wrap the tagName in span.webkit-html-tag-name

Miscellaneous updates.

  • inspector/front-end/treeoutline.js: fixed a typo.
12:54 AM Changeset in webkit [56682] by Chris Jerdonek
  • 7 edits
    2 adds in trunk/WebKitTools

Changed check-webkit-style to convert paths to paths relative to
the checkout root when invoking check-webkit-style with path
arguments. Also added warning messages where appropriate.

Reviewed by Shinichiro Hamaji.

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

Converting paths to paths relative to the checkout root ensures
that style checking will behave as expected (since path-specific
rules assume input paths are relative to a source root).

  • Scripts/check-webkit-style:
    • Added debug logging of whether the current directory was found to be in a WebKit checkout.
    • Added the found_checkout parameter to the call to parser.parse().
    • Renamed the files variable to paths.
    • Added a call to change_directory() prior to checking style.
  • Scripts/webkitpy/style/checker.py:
    • For StyleChecker.check_file():
      • Updated the docstring.
      • Added two log messages.
  • Scripts/webkitpy/style/checker_unittest.py:
    • Updated a call to parser.parse() with the found_checkout parameter.
  • Scripts/webkitpy/style/main.py: Added.
    • Added a new file so the code in this patch could be unit-tested, as opposed to adding new code to check-webkit-style.
    • Added the method _rel_path() as a substitute for os.path.relpath(), which is available only in Python 2.6.
    • Added the method change_directory(), which contains most of the new functionality in this patch.
  • Scripts/webkitpy/style/main_unittest.py: Added.
    • Added RelPathTest to test main._rel_path().
    • Added ChangeDirectoryTest to test main.change_directory().
  • Scripts/webkitpy/style/optparser.py:
    • Updated check-webkit-style's usage string.
    • For the ArgumentParser.parse() method:
      • Added a found_checkout parameter.
      • Renamed filenames to paths.
      • Added logic so that an error is raised if no paths are passed if found_checkout is False.
  • Scripts/webkitpy/style/optparser_unittest.py:
    • Updated the ArgumentParser.parse() unit tests to include coverage for the new found_checkout parameter.
  • Scripts/webkitpy/style/unittests.py:
    • Added an import statement for main_unittest.
  • Scripts/webkitpy/style_references.py:
    • Renamed SimpleScm to WebKitCheckout.
    • Added a detect_checkout() function to allow returning None instead of a WebKitCheckout instance if no checkout is found.
    • Renamed checkout_root to root_path.

Mar 27, 2010:

11:09 PM Changeset in webkit [56681] by eric@webkit.org
  • 7 edits in trunk

2010-03-27 Sergio Villar Senin <​svillar@igalia.com>

Reviewed by Eric Seidel.

Removed 5 http/history tests from Skipped file

[GTK] Improve reporting of frame loader callbacks in DRT
​https://bugs.webkit.org/show_bug.cgi?id=36454

  • platform/gtk/Skipped:

2010-03-27 Sergio Villar Senin <​svillar@igalia.com>

Reviewed by Eric Seidel.

FrameLoader emits onload-event when handling
dispatchDidHandleOnloadEvents

[GTK] Improve reporting of frame loader callbacks in DRT
​https://bugs.webkit.org/show_bug.cgi?id=36454

  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::dispatchDidHandleOnloadEvents): implemented, now it emits onload-event signal
  • webkit/webkitwebview.cpp: added onload-event signal

2010-03-27 Sergio Villar Senin <​svillar@igalia.com>

Reviewed by Eric Seidel.

Print didHandleOnloadEventsForFrame in the callback of
onload-event signal comming from frame loader

[GTK] Improve reporting of frame loader callbacks in DRT
​https://bugs.webkit.org/show_bug.cgi?id=36454

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (webViewOnloadEvent): (createWebView): added connection to signal::onload-event and signal callback
10:51 PM Changeset in webkit [56680] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-03-27 Sergio Villar Senin <​svillar@igalia.com>

Reviewed by Eric Seidel.

Print didCommitLoadForFrame in the callback of signal::load-committed

[GTK] Improve reporting of frame loader callbacks in DRT
​https://bugs.webkit.org/show_bug.cgi?id=36454

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (webViewLoadCommitted): (createWebView): added connection to sinal::load-committed and signal callback
10:34 PM Changeset in webkit [56679] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-03-27 Sergio Villar Senin <​svillar@igalia.com>

Reviewed by Eric Seidel.

Print didStartProvisionalLoadForFrame in the callback of
notify::load-status property change notification

[GTK] Improve reporting of frame loader callbacks in DRT
​https://bugs.webkit.org/show_bug.cgi?id=36454

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (webInspectorCloseWindow): (webInspectorInspectWebView): (createWebView): added connection to notify::load-status and signal callback
9:16 PM Changeset in webkit [56678] by kbr@google.com
  • 2 edits in trunk/WebKit/chromium

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

Reviewed by Darin Fisher.

Use WebKitClient to instantiate WebGraphicsContext3D
​https://bugs.webkit.org/show_bug.cgi?id=36669

  • src/GraphicsContext3D.cpp: (WebCore::GraphicsContext3DInternal::initialize):
7:50 PM Changeset in webkit [56677] by eric@webkit.org
  • 1 edit
    8 deletes in trunk/WebKitTools

2010-03-27 Chris Jerdonek <Chris Jerdonek>

Reviewed by Adam Barth.

In webkitpy/, deleted the /unittests.py files since test-webkitpy
now auto-detects all *_unittest.py files.

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

  • Scripts/webkitpy/common/net/irc/unittests.py: Removed.
  • Scripts/webkitpy/common/system/unittests.py: Removed.
  • Scripts/webkitpy/common/thread/unittests.py: Removed.
  • Scripts/webkitpy/common/unittests.py: Removed.
  • Scripts/webkitpy/python24/unittests.py: Removed.
  • Scripts/webkitpy/style/unittests.py: Removed.
  • Scripts/webkitpy/tool/unittests.py: Removed.
  • Scripts/webkitpy/unittests.py: Removed.
7:32 PM Changeset in webkit [56676] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-03-27 Dmitry Gorbik <​socket.h@gmail.com>

Reviewed by Pavel Feldman.

Fix the regression caused by r28078: a global variable
definition masks a local one in an inspector console
​https://bugs.webkit.org/show_bug.cgi?id=32442

  • inspector/front-end/InjectedScript.js: (injectedScriptConstructor):
6:56 PM Changeset in webkit [56675] by mitz@apple.com
  • 5 edits in trunk/LayoutTests

More updated results for r56673.

  • platform/chromium-mac/svg/css/getComputedStyle-basic-expected.txt:
  • platform/chromium-win/svg/css/getComputedStyle-basic-expected.txt:
  • platform/gtk/svg/css/getComputedStyle-basic-expected.txt:
  • svg/css/getComputedStyle-basic-expected.txt:
6:07 PM Changeset in webkit [56674] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-03-27 Sergio Villar Senin <​svillar@igalia.com>

Reviewed by Eric Seidel.

Add a CR after printing didFinishDocumentLoadForFrame

[GTK] Improve reporting of frame loader callbacks in DRT
​https://bugs.webkit.org/show_bug.cgi?id=36454

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (webViewLoadFinished):
5:59 PM Changeset in webkit [56673] by mitz@apple.com
  • 12 edits in trunk

WebCore: <rdar://problem/7801598> REGRESSION (r56182): iWeb shadow drawn around image rectangle, not around opaque part of the image
​https://bugs.webkit.org/show_bug.cgi?id=36700

Reviewed by Darin Adler.

  • css/CSSPrimitiveValue.cpp:

(WebCore::CSSPrimitiveValue::createColor): Reverted to returning an RGB color value rather than an
identifier for the transparent color.

LayoutTests: Updated results and one test for
<rdar://problem/7801598> REGRESSION (r56182): iWeb shadow drawn around image rectangle, not around opaque part of the image
​https://bugs.webkit.org/show_bug.cgi?id=36700

Reviewed by Darin Adler.

  • fast/css/getComputedStyle/computed-style-expected.txt:
  • fast/css/invalid-import-rule-insertion.html:
  • http/tests/mime/standard-mode-does-not-load-stylesheet-with-text-plain-and-css-extension-expected.txt:
  • http/tests/mime/standard-mode-does-not-load-stylesheet-with-text-plain-expected.txt:
  • http/tests/security/cross-origin-css-expected.txt:
  • http/tests/security/cross-origin-css-in-xml-expected.txt:
  • platform/mac/fast/css/getComputedStyle/computed-style-expected.txt:
  • platform/qt/fast/css/getComputedStyle/computed-style-expected.txt:
  • platform/win/fast/css/getComputedStyle/computed-style-expected.txt:
  • svg/css/getComputedStyle-basic-expected.txt:
4:29 PM Changeset in webkit [56672] by Chris Jerdonek
  • 1 edit
    1 move in trunk/WebKitTools

Rubber-stamped by Adam Barth.

Fixed the name of a unit test file in webkitpy.

  • Scripts/webkitpy/tool/steps/updatechangelogswithreview_unittest.py: Copied from WebKitTools/Scripts/webkitpy/tool/steps/updatechangelogswithreview_unittests.py.
  • Scripts/webkitpy/tool/steps/updatechangelogswithreview_unittests.py: Removed.
3:56 PM Changeset in webkit [56671] by Chris Jerdonek
  • 2 edits
    3 adds in trunk/WebKitTools

2010-03-26 Chris Jerdonek <Chris Jerdonek>

Reviewed by Adam Barth.

The test-webkitpy script now automatically detects all unit-test
files in webkitpy/. This lets us eliminate the need to have and
maintain all of the unittests.py files.

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

  • Scripts/test-webkitpy:
    • Replaced the "from webkitpy.unittests import *" with a call to webkitpy.test.main.Tester().run_tests().
  • Scripts/webkitpy/test/init.py: Copied from WebKitTools/QueueStatusServer/filters/init.py.
    • Required file for the new webkitpy/test/ directory.
  • Scripts/webkitpy/test/main.py: Added.
    • Added a Tester class that contains the following methods:
      • _find_unittest_files() to detect all the unit-test files.
      • _modules_from_paths() to convert the paths to the unit-test files to fully-qualified module names.
      • run_tests() which calls the above two methods and then passes the module names to Python's unittest module.
2:42 PM Changeset in webkit [56670] by Joseph Pecoraro
  • 6 edits in trunk/WebCore

2010-03-26 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Pavel Feldman.

Web Inspector: Converge Inspector ids to be long, not ints
​https://bugs.webkit.org/show_bug.cgi?id=36706

Part 1: Converge on long for callIds, nodeIds, and other Ids.

  • inspector/InspectorController.cpp:
  • inspector/InspectorController.h:
  • inspector/InspectorDOMAgent.cpp:
  • inspector/InspectorFrontend.cpp:
  • inspector/InspectorFrontend.h:
12:51 PM Changeset in webkit [56669] by mrowe@apple.com
  • 2 edits in trunk/LayoutTests

Fix the newly-added test to not trip over a bug that causes some subsequent tests to fail.

Rubber-stamped by Dan Bernstein.

  • fast/dom/script-tests/setPrimitiveValue-exceptions.js:
12:31 PM Changeset in webkit [56668] by robert@webkit.org
  • 5 edits in trunk

2010-03-27 Robert Hogan <​robert@webkit.org>

Reviewed by Simon Fraser.

[Qt/Win] Add support to unix and windows NS plugin for executing scripts on setWindow.

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

  • DumpRenderTree/unix/TestNetscapePlugin/TestNetscapePlugin.cpp: (webkit_test_plugin_new_instance): Add onSetWindow. (webkit_test_plugin_destroy_instance): Add onSetWindow. (webkit_test_plugin_set_window): Add onSetWindow.
  • DumpRenderTree/win/TestNetscapePlugin/main.cpp: (NPP_New): Add onSetWindow. (NPP_Destroy): Add onSetWindow. (NPP_SetWindow): Add onSetWindow.

2010-03-27 Robert Hogan <​robert@webkit.org>

Unreviewed skip failing test.

[Qt] Skip test added in r56646. Tracked by ​https://bugs.webkit.org/show_bug.cgi?id=36702

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

  • platform/qt/Skipped: Add plugins/reentrant-update-widget-positions.html
12:24 PM Changeset in webkit [56667] by Darin Adler
  • 2 edits in trunk/WebKit/mac
  • Misc/WebNSFileManagerExtras.m:

(-[NSFileManager _webkit_pathWithUniqueFilenameForPath:]):
Removed stray "!". How did that get in there?

12:20 PM Changeset in webkit [56666] by Darin Adler
  • 3 edits in trunk/WebKit/mac

WebKit NSFileManager methods don't handle broken symlinks correctly.
Part of <rdar://problem/7574046>.

Reviewed by Mark Rowe.

  • Misc/WebNSFileManagerExtras.h: Removed unused defines and methods.
  • Misc/WebNSFileManagerExtras.m: Removed unused methods.

(fileExists): Added. For use instead of fileExistsAtPath: for cases where we'd like
to treat a broken symlink as a file that does indeed exist.
(-[NSFileManager _webkit_pathWithUniqueFilenameForPath:]): Use fileExists.

12:17 PM Changeset in webkit [56665] by pfeldman@chromium.org
  • 2 edits
    1 add in trunk

2010-03-27 Pavel Feldman <​pfeldman@chromium.org>

Not reviewed: added missing test resource file.

  • inspector/resources/elements-panel-styles.css: Added.
11:24 AM Changeset in webkit [56664] by pfeldman@chromium.org
  • 7 edits
    2 adds in trunk

2010-03-27 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: add layout test for styles panel.

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

Test: inspector/elements-panel-styles.html

  • inspector/front-end/inspector.css: (#elements-content): (#elements-sidebar):
  • inspector/front-end/inspector.js: (WebInspector.displayNameForURL):
  • inspector/front-end/utilities.js: (String.prototype.trimURL):
11:21 AM Changeset in webkit [56663] by mrowe@apple.com
  • 3 edits
    3 adds in trunk

CSSPrimitiveValue::setFloatValue/setStringValue should throw an exception if passed an incorrect unit type.
<​http://webkit.org/b/36680> / <rdar://problem/7801016>

Reviewed by Darin Adler.

WebCore:

Test: fast/dom/setPrimitiveValue-exceptions.html

  • css/CSSPrimitiveValue.cpp:

(WebCore::CSSPrimitiveValue::setFloatValue): Throw an INVALID_ACCESS_ERR if either of our type or the passed-in
type is not a numeric type.
(WebCore::CSSPrimitiveValue::setStringValue): Throw an INVALID_ACCESS_ERR if either of our type or the passed-in
type is not a string type.

LayoutTests:

  • fast/dom/script-tests/setPrimitiveValue-exceptions.js: Added.
  • fast/dom/setPrimitiveValue-exceptions-expected.txt: Added.
  • fast/dom/setPrimitiveValue-exceptions.html: Copied from LayoutTests/fast/dom/event-attribute-availability.html.
6:06 AM Changeset in webkit [56662] by robert@webkit.org
  • 2 edits in trunk/WebCore

2010-03-27 Robert Hogan <​robert@webkit.org>

Reviewed by nobody, fix typo in previous commit.

Allow plugins implemented by the application, such as mimetype 'x-qt-plugin',

when pluginsEnabled is false

Fix parentheses typo in r56661. This happened while rebasing and was not present
in the reviewed patch, so committing unreviewed.

​https://bugs.webkit.org/attachment.cgi?id=49515

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::requestObject):
5:46 AM Changeset in webkit [56661] by robert@webkit.org
  • 18 edits in trunk

2010-03-26 Robert Hogan <​robert@roberthogan.net>

Reviewed by Simon Hausmann.

Allow plugins implemented by the application, such as mimetype 'x-qt-plugin',

when pluginsEnabled is false.

The purpose of disabling plugins is to prevent the execution of third-party code
that may be untrustworthy. Qt plugins are implemented by the client rather than
loaded from an external source, so the client should have the opportunity to
consider them separately from other plugins.

Add a function MimeTypeRegistry::isApplicationPluginMIMEType() that WebKit
uses in conjunction with arePluginsEnabled() to determine if it should attempt
to load a plugin. If isApplicationPluginMIMEType() returns true, WebKit will load
the plugin even if arePluginsEnabled() is false.

Currently, only Qt has application-implemented plugins: these use the mimetype
'x-qt-plugin' and 'x-qt-styled-widget'. This patch permits Qt clients'
reimplementation of QWebPage::createPlugin() to decide whether or not
to create a Qt plugin, even when arePluginsEnabled is false.

For all platforms apart from Qt, isApplicationPluginMIMEType() returns false.

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

Test: plugins/application-plugin-plugins-disabled.html

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::requestObject):
  • platform/MIMETypeRegistry.h:
  • platform/brew/MIMETypeRegistryBrew.cpp: (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
  • platform/chromium/MIMETypeRegistryChromium.cpp: (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
  • platform/gtk/MIMETypeRegistryGtk.cpp: (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
  • platform/haiku/MIMETypeRegistryHaiku.cpp: (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
  • platform/mac/MIMETypeRegistryMac.mm: (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
  • platform/qt/MIMETypeRegistryQt.cpp: (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
  • platform/win/MIMETypeRegistryWin.cpp: (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
  • platform/wince/MIMETypeRegistryWince.cpp: (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
  • platform/wx/MimeTypeRegistryWx.cpp: (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):

2010-03-26 Robert Hogan <​robert@roberthogan.net>

Reviewed by Simon Hausmann.

Allow plugins implemented by the application, such as mimetype 'x-qt-plugin',

when pluginsEnabled is false.

For all platforms apart from Qt, isApplicationPluginMIMEType() returns false.

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

  • platform/qt/plugins/application-plugin-plugins-disabled-expected.txt: Added.
  • plugins/application-plugin-plugins-disabled-expected.txt: Added.
  • plugins/application-plugin-plugins-disabled.html: Added.

2010-03-26 Robert Hogan <​robert@roberthogan.net>

Reviewed by Simon Hausmann.

Allow plugins implemented by the application, such as mimetype 'x-qt-plugin',

when pluginsEnabled is false.

Add support for LayoutTestController.WebKitPluginsEnabled

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

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (resetDefaultsToConsistentValues):
  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (copyWebSettingKey):
  • DumpRenderTree/qt/DumpRenderTreeQt.cpp: (WebCore::WebPage::resetSettings):
  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::overridePreference):
5:36 AM Changeset in webkit [56660] by yael.aharon@nokia.com
  • 2 edits in trunk/LayoutTests

Unreviewed.
Skip HTMLProgressElement tests on Windows.

  • platform/win/Skipped:
2:06 AM Changeset in webkit [56659] by mitz@apple.com
  • 2 edits in trunk/WebCore

Fix Acid3 text-shadow rendering regression introduced in r56597.

Reviewed by Mark Rowe.

  • platform/graphics/mac/FontMac.mm:

(WebCore::Font::drawGlyphs): Revert to using the simple shadow logic for zero-blur shadows,
except for color bitmap fonts.

Note: See TracTimeline for information about the timeline view.