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

Timeline



Dec 3, 2011:

10:46 PM Changeset in webkit [101948] by noel.gordon@gmail.com
  • 2 edits in trunk/Source/WebCore

Fix WebPImageDecoder decoder leak.
​https://bugs.webkit.org/show_bug.cgi?id=73756

Reviewed by Andreas Kling.

Delete the m_decoder member in the destructor if needed with WebPIDelete.

No new tests, it's valgrind territory.

  • platform/image-decoders/webp/WEBPImageDecoder.cpp:

(WebCore::WEBPImageDecoder::~WEBPImageDecoder):

9:49 PM Changeset in webkit [101947] by scheib@chromium.org
  • 2 edits
    2 moves
    1 add
    1 delete in trunk/LayoutTests

[Chromium] Rebaselined /accessibility/aria-checkbox-sends-notification-expected.txt

  • platform/chromium-cg-mac-leopard/accessibility/aria-checkbox-sends-notification-expected.txt: Renamed from LayoutTests/platform/chromium-mac-snowleopard/accessibility/aria-checkbox-sends-notification-expected.txt.
  • platform/chromium-cg-mac-snowleopard/accessibility/aria-checkbox-sends-notification-expected.txt: Removed.
  • platform/chromium/accessibility/aria-checkbox-sends-notification-expected.txt: Renamed from LayoutTests/platform/chromium-win/accessibility/aria-checkbox-sends-notification-expected.txt.
  • platform/chromium/test_expectations.txt:
9:20 PM Changeset in webkit [101946] by Darin Adler
  • 3 edits in trunk/Source/JavaScriptCore

Use HashMap<OwnPtr> for ScriptSampleRecordMap
​https://bugs.webkit.org/show_bug.cgi?id=73758

Reviewed by Andreas Kling.

  • bytecode/SamplingTool.cpp:

(JSC::SamplingTool::notifyOfScope): Added adoptPtr.
(JSC::SamplingTool::dump): Added get.

  • bytecode/SamplingTool.h: Changed the value type of ScriptSampleRecordMap to be OwnPtr.
9:20 PM Changeset in webkit [101945] by Darin Adler
  • 4 edits in trunk/Source/JavaScriptCore

Use HashMap<OwnPtr> for the opaqueJSClassData map
​https://bugs.webkit.org/show_bug.cgi?id=73759

Reviewed by Andreas Kling.

  • API/JSClassRef.cpp:

(OpaqueJSClass::contextData): Update types.

  • runtime/JSGlobalData.cpp:

(JSC::JSGlobalData::~JSGlobalData): Add an explicit clear of opaqueJSClassData to keep the
timing the same. If we didn't care about the order of operations, we could remove this, too.

  • runtime/JSGlobalData.h: Use OwnPtr instead of raw pointer for the mapped type in the

opaqueJSClassData map.

9:11 PM Changeset in webkit [101944] by scheib@chromium.org
  • 2 edits in trunk/LayoutTests

[Chromium] Supressing more failures via test_expectations.txt

  • platform/chromium/test_expectations.txt:

editing/spelling/markers.html, new expectations fail
svg/animations/svginteger-animation-1.html, merging overlapping lines

7:51 PM Changeset in webkit [101943] by kling@webkit.org
  • 8 edits in trunk/Source/WebCore

CSSStyleSheet: Parent rule can only ever be @import, enforce this at compile time.
<​http://webkit.org/b/73725>

Reviewed by Darin Adler.

Made StyleSheet::m_parentRule a CSSImportRule* rather than a CSSRule*
and updated getters, setters and constructors accordingly.

There is no change in behavior, this simply enforces the status quo.

  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSDOMBinding.h:
  • bindings/scripts/CodeGeneratorV8.pm:

(AddIncludesForType):

  • css/CSSStyleSheet.cpp:

(WebCore::CSSStyleSheet::CSSStyleSheet):

  • css/CSSStyleSheet.h:

(WebCore::CSSStyleSheet::create):
(WebCore::CSSStyleSheet::ownerRule):

  • css/StyleSheet.cpp:

(WebCore::StyleSheet::StyleSheet):

  • css/StyleSheet.h:

(WebCore::StyleSheet::parentRule):
(WebCore::StyleSheet::setParentRule):

7:45 PM Changeset in webkit [101942] by Darin Adler
  • 4 edits in trunk/Source/JavaScriptCore

Change HashMap implementation to use the pass type and peek type from traits for the mapped value
​https://bugs.webkit.org/show_bug.cgi?id=72474

Reviewed by Anders Carlsson.

  • wtf/HashMap.h: Added ReferenceTypeMaker struct template. Get PassInType, PassOutType,

and PeekType from the traits of the mapped value instead of hard-coding them here.
Changed inlineAdd to take a reference to the PassInType instead of the PassInType itself,
to accomodate a PassInType that can't be copied. Use the store, peek, and passOut
functions from the traits as well.

  • wtf/HashTraits.h: Updated GenericHashTraits and HashTraits for OwnPtr to include

PassInType, PassOutType, PeekType, store, passOut, and peek. Before this, the file had
an earlier version that was just PassType, PeekType, pass, and peek. Also commented
the HashTraits for RefPtr to foreshadow some work we can do there.

  • wtf/RefPtrHashMap.h: Same changes as HashMap.h.
7:25 PM Changeset in webkit [101941] by Darin Adler
  • 8 edits
    3 adds
    1 delete in trunk/LayoutTests

Update some layout test expected results
​https://bugs.webkit.org/show_bug.cgi?id=73755

Reviewed by Sam Weinig.

  • editing/spelling/markers-expected.txt: Added. We just forgot to check this in when the

test was added, and the test is skipped on most platforms so that was not noticed.

  • fast/css/getComputedStyle/computed-style-without-renderer.html: Changed this test to

share the list of properties to skip and the list of hidden properties.

  • fast/css/getComputedStyle/computed-style.html: Ditt.
  • fast/css/getComputedStyle/resources/property-names.js: Added. Besides the structural

change the change is that CSS_GRID_LAYOUT properties are now in the list to skip.

  • platform/mac/accessibility/multiselect-list-reports-active-option-expected.txt: Added.

Shows this test is failing on Mac, even though it is passing on Chromium. That needs a bug report.

  • platform/mac/fast/forms/input-placeholder-visibility-1-expected.txt:
  • platform/mac/fast/forms/input-placeholder-visibility-3-expected.txt:
  • platform/mac/fast/forms/textarea-placeholder-visibility-1-expected.txt:
  • platform/mac/fast/forms/textarea-placeholder-visibility-2-expected.txt:

Updated these results now that we don't use relative positioning for placeholders any more.
The original patch r101742 updated other platforms, including mac-snowleopard, but missed this one.

  • platform/mac/svg/css/getComputedStyle-basic-expected.txt: Removed. The Mac platform can

share the platform-independent expected results as long as we skip the CSS_GRID_LAYOUT
properties.

  • svg/css/getComputedStyle-basic.xhtml: Updated to skip the same properties that the tests

in fast/css/getComputedStyle skip.

5:22 PM Changeset in webkit [101940] by kling@webkit.org
  • 2 edits in trunk/Source/WebCore

Unreviewed, revert accidental change from r101932.

  • bindings/scripts/CodeGeneratorV8.pm:

(AddIncludesForType):

5:01 PM Changeset in webkit [101939] by mitz@apple.com
  • 4 edits
    1 add in trunk

Source/WebKit/mac: <rdar://problem/10523721> Crash at WebCore::SubresourceLoader::releaseResources
Reviewed by Darin Adler.

  • WebView/WebView.mm:

(-[WebView _removeObjectForIdentifier:]): Changed the CFRelease(self) into
a WebCFAutorelease(self). This prevents re-entry into this method due to
the WebView closing and canceling all subresource loads, including the
one we have just removed.

Tools: Added a test for <rdar://problem/10523721> Crash at WebCore::SubresourceLoader::releaseResources

Reviewed by Darin Adler.

  • TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
  • TestWebKitAPI/Tests/mac/SubresourceErrorCrash.mm: Added.

(TestWebKitAPI::TEST):

3:55 PM Changeset in webkit [101938] by Darin Adler
  • 1 edit
    11 deletes in trunk/LayoutTests

Removed some obsolete expected result files that should have been removed in r97308 and r101738.

  • platform/efl/fast/regions/content-flowed-into-regions-dynamically-added-expected.png: Removed.
  • platform/efl/fast/regions/content-flowed-into-regions-dynamically-inserted-expected.png: Removed.
  • platform/efl/fast/regions/content-flowed-into-regions-dynamically-removed-expected.png: Removed.
  • platform/efl/fast/regions/content-flowed-into-regions-expected.png: Removed.
  • platform/efl/fast/regions/content-flowed-into-regions-with-dyn-index-expected.png: Removed.
  • platform/efl/fast/regions/content-flowed-into-regions-with-index-dom-expected.png: Removed.
  • platform/efl/fast/regions/content-flowed-into-regions-with-index-expected.png: Removed.
  • platform/mac/fast/regions/content-flowed-into-regions-dynamically-added-expected.png: Removed.
  • platform/mac/fast/regions/content-flowed-into-regions-dynamically-inserted-expected.png: Removed.
  • platform/mac/fast/regions/content-flowed-into-regions-dynamically-removed-expected.png: Removed.
  • platform/mac/fast/regions/content-flowed-into-regions-expected.png: Removed.
3:32 PM Changeset in webkit [101937] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit/efl

[EFL] Revise ewk_view_fixed_layout_size_set
​https://bugs.webkit.org/show_bug.cgi?id=73401

Patch by ChangSeok Oh <ChangSeok Oh> on 2011-12-03
Reviewed by Eric Seidel.

Improve readability and replace forceLayout to setNeedsLayout,
since it sometimes causes double drawing.

  • ewk/ewk_view.cpp:

(ewk_view_fixed_layout_size_set):

3:11 PM Changeset in webkit [101936] by commit-queue@webkit.org
  • 1 edit
    5 adds in trunk/Source/WebCore

Upstream 4 files into WebCore/platform/blackberry
​https://bugs.webkit.org/show_bug.cgi?id=73614

Patch by Mary Wu <​mary.wu@torchmobile.com.cn> on 2011-12-03
Reviewed by Eric Seidel.

  • platform/blackberry/CursorBlackBerry.cpp: Added.

(WebCore::AllCursors::AllCursors):
(WebCore::getCursor):
(WebCore::Cursor::Cursor):
(WebCore::Cursor::~Cursor):
(WebCore::Cursor::operator=):
(WebCore::aliasCursor):
(WebCore::cellCursor):
(WebCore::columnResizeCursor):
(WebCore::contextMenuCursor):
(WebCore::copyCursor):
(WebCore::crossCursor):
(WebCore::eastResizeCursor):
(WebCore::eastWestResizeCursor):
(WebCore::grabbingCursor):
(WebCore::grabCursor):
(WebCore::handCursor):
(WebCore::helpCursor):
(WebCore::iBeamCursor):
(WebCore::moveCursor):
(WebCore::noDropCursor):
(WebCore::noneCursor):
(WebCore::northEastResizeCursor):
(WebCore::northEastSouthWestResizeCursor):
(WebCore::northResizeCursor):
(WebCore::northSouthResizeCursor):
(WebCore::northWestResizeCursor):
(WebCore::northWestSouthEastResizeCursor):
(WebCore::notAllowedCursor):
(WebCore::pointerCursor):
(WebCore::progressCursor):
(WebCore::rowResizeCursor):
(WebCore::southEastResizeCursor):
(WebCore::southResizeCursor):
(WebCore::southWestResizeCursor):
(WebCore::verticalTextCursor):
(WebCore::waitCursor):
(WebCore::westResizeCursor):
(WebCore::zoomInCursor):
(WebCore::zoomOutCursor):
(WebCore::middlePanningCursor):
(WebCore::eastPanningCursor):
(WebCore::northPanningCursor):
(WebCore::northEastPanningCursor):
(WebCore::northWestPanningCursor):
(WebCore::southPanningCursor):
(WebCore::southEastPanningCursor):
(WebCore::southWestPanningCursor):
(WebCore::westPanningCursor):

  • platform/blackberry/DragDataBlackBerry.cpp: Added.

(WebCore::DragData::canSmartReplace):
(WebCore::DragData::containsColor):
(WebCore::DragData::containsCompatibleContent):
(WebCore::DragData::containsFiles):
(WebCore::DragData::containsPlainText):
(WebCore::DragData::containsURL):
(WebCore::DragData::asFilenames):
(WebCore::DragData::asColor):
(WebCore::DragData::asPlainText):
(WebCore::DragData::asURL):
(WebCore::DragData::asFragment):

  • platform/blackberry/DragImageBlackBerry.cpp: Added.

(WebCore::createDragImageFromImage):
(WebCore::createDragImageIconForCachedImage):
(WebCore::deleteDragImage):
(WebCore::dissolveDragImageToFraction):
(WebCore::scaleDragImage):
(WebCore::dragImageSize):

  • platform/blackberry/EventLoopBlackBerry.cpp: Added.

(WebCore::EventLoop::platformInit):
(WebCore::EventLoop::cycle):

1:53 PM Changeset in webkit [101935] by kling@webkit.org
  • 9 edits in trunk

Cache CSSInitialValue instances per-document.
<​http://webkit.org/b/73745>

Reviewed by Oliver Hunt.

Source/WebCore:

Test: http/tests/security/cross-origin-css-primitive.html

Have CSSValuePool manage the caching of CSSInitialValue objects.

  • css/CSSInitialValue.h:

(WebCore::CSSInitialValue::createExplicit):
(WebCore::CSSInitialValue::createImplicit):
(WebCore::CSSInitialValue::CSSInitialValue):

  • css/CSSParser.cpp:

(WebCore::CSSParser::parseValue):
(WebCore::CSSParser::parseFillShorthand):
(WebCore::CSSParser::parseAnimationShorthand):
(WebCore::CSSParser::parseTransitionShorthand):
(WebCore::CSSParser::parseShorthand):

  • css/CSSValuePool.cpp:

(WebCore::CSSValuePool::CSSValuePool):
(WebCore::CSSValuePool::createImplicitInitialValue):
(WebCore::CSSValuePool::createExplicitInitialValue):

  • css/CSSValuePool.h:

LayoutTests:

Test that 'initial' and 'inherit' values are cached per-document.

  • http/tests/security/cross-origin-css-primitive-expected.txt:
  • http/tests/security/cross-origin-css-primitive.html:
  • http/tests/security/resources/cross-origin-css-primitive-iframe.html:
1:45 PM Changeset in webkit [101934] by kling@webkit.org
  • 4 edits in trunk/Source/WebCore

Keep CSSInheritedValue in the CSS value pool.
<​http://webkit.org/b/73747>

Reviewed by Antti Koivisto.

We only need one CSSInheritedValue instance per document, so cache it
in CSSValuePool and have the parser create it through there.

  • css/CSSParser.cpp:

(WebCore::CSSParser::parseValue):

  • css/CSSValuePool.cpp:

(WebCore::CSSValuePool::CSSValuePool):
(WebCore::CSSValuePool::createInheritedValue):

  • css/CSSValuePool.h:
12:06 PM Changeset in webkit [101933] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebKit/gtk

[GTK] Implement History API FrameLoaderClient methods
​https://bugs.webkit.org/show_bug.cgi?id=59017

Patch by Xan Lopez <​xlopez@igalia.com> on 2011-12-03
Reviewed by Gustavo Noronha Silva.

Fake the load process when the new HTML5 History APIs are
used. Otherwise the UA does not have a chance of updating the URL
entry, back/forward buttons, etc.

  • WebCoreSupport/FrameLoaderClientGtk.cpp:

(WebKit::FrameLoaderClient::dispatchDidNavigateWithinPage): fake a
load process.
(WebKit::FrameLoaderClient::dispatchDidPushStateWithinPage): call
dispatchDidNavigateWithinPage.
(WebKit::FrameLoaderClient::dispatchDidReplaceStateWithinPage):
ditto.
(WebKit::FrameLoaderClient::dispatchDidPopStateWithinPage): do
nothing, because push was already called, but note this in a
comment.

  • WebCoreSupport/FrameLoaderClientGtk.h: add the new method.
10:19 AM Changeset in webkit [101932] by kling@webkit.org
  • 14 edits
    2 adds
    2 deletes in trunk/Source/WebCore

Rename CSSPrimitiveValueCache to CSSValuePool.
<​http://webkit.org/b/73742>

Reviewed by Antti Koivisto.

CSSPrimitiveValueCache -> CSSValuePool
Document::primitiveValueCache() -> Document::cssValuePool()

This is in preparation for sharing more than just primitive values.

  • CMakeLists.txt:
  • GNUmakefile.list.am:
  • Target.pri:
  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/scripts/CodeGeneratorV8.pm:

(AddIncludesForType):

  • css/CSSComputedStyleDeclaration.cpp:

(WebCore::valueForNinePieceImageSlice):
(WebCore::valueForNinePieceImageQuad):
(WebCore::valueForNinePieceImageRepeat):
(WebCore::valueForNinePieceImage):
(WebCore::zoomAdjustedPixelValue):
(WebCore::zoomAdjustedNumberValue):
(WebCore::zoomAdjustedPixelValueForLength):
(WebCore::valueForReflection):
(WebCore::getPositionOffsetValue):
(WebCore::CSSComputedStyleDeclaration::currentColorOrValidColor):
(WebCore::getBorderRadiusCornerValue):
(WebCore::computedTransform):
(WebCore::CSSComputedStyleDeclaration::valueForFilter):
(WebCore::valueForGridTrackList):
(WebCore::getDelayValue):
(WebCore::getDurationValue):
(WebCore::createLineBoxContainValue):
(WebCore::CSSComputedStyleDeclaration::getFontSizeCSSValuePreferringKeyword):
(WebCore::CSSComputedStyleDeclaration::valueForShadow):
(WebCore::valueForFamily):
(WebCore::renderTextDecorationFlagsToCSSValue):
(WebCore::fillRepeatToCSSValue):
(WebCore::fillSizeToCSSValue):
(WebCore::contentToCSSValue):
(WebCore::counterToCSSValue):
(WebCore::fontFamilyFromStyle):
(WebCore::lineHeightFromStyle):
(WebCore::fontSizeFromStyle):
(WebCore::fontStyleFromStyle):
(WebCore::fontVariantFromStyle):
(WebCore::fontWeightFromStyle):
(WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):

  • css/CSSParser.cpp:

(WebCore::parseColorValue):
(WebCore::parseSimpleLengthValue):
(WebCore::CSSParser::setStyleSheet):
(WebCore::CSSParser::createPrimitiveNumericValue):
(WebCore::CSSParser::createPrimitiveStringValue):
(WebCore::CSSParser::parseValidPrimitive):
(WebCore::CSSParser::parseValue):
(WebCore::parseBackgroundClip):
(WebCore::CSSParser::parseFillShorthand):
(WebCore::CSSParser::parsePage):
(WebCore::CSSParser::parseSizeParameter):
(WebCore::CSSParser::parseContent):
(WebCore::CSSParser::parseAttr):
(WebCore::CSSParser::parseBackgroundColor):
(WebCore::CSSParser::parseFillPositionX):
(WebCore::CSSParser::parseFillPositionY):
(WebCore::CSSParser::parseFillPositionComponent):
(WebCore::CSSParser::parseFillPosition):
(WebCore::CSSParser::parseFillRepeat):
(WebCore::CSSParser::parseFillSize):
(WebCore::CSSParser::parseFillProperty):
(WebCore::CSSParser::parseAnimationDirection):
(WebCore::CSSParser::parseAnimationFillMode):
(WebCore::CSSParser::parseAnimationIterationCount):
(WebCore::CSSParser::parseAnimationName):
(WebCore::CSSParser::parseAnimationPlayState):
(WebCore::CSSParser::parseAnimationProperty):
(WebCore::CSSParser::parseAnimationTimingFunction):
(WebCore::CSSParser::parseGridTrackList):
(WebCore::CSSParser::parseDashboardRegions):
(WebCore::CSSParser::parseCounterContent):
(WebCore::CSSParser::parseShape):
(WebCore::CSSParser::parseWrapShape):
(WebCore::CSSParser::parseFont):
(WebCore::CSSParser::parseFontFamily):
(WebCore::CSSParser::parseFontStyle):
(WebCore::CSSParser::parseFontVariant):
(WebCore::CSSParser::parseFontWeight):
(WebCore::CSSParser::parseColor):
(WebCore::ShadowParseContext::ShadowParseContext):
(WebCore::ShadowParseContext::commitLength):
(WebCore::ShadowParseContext::commitStyle):
(WebCore::CSSParser::parseShadow):
(WebCore::CSSParser::parseReflect):
(WebCore::CSSParser::parseFlex):
(WebCore::BorderImageParseContext::BorderImageParseContext):
(WebCore::CSSParser::parseBorderImage):
(WebCore::CSSParser::parseBorderImageRepeat):
(WebCore::BorderImageSliceParseContext::BorderImageSliceParseContext):
(WebCore::BorderImageSliceParseContext::commitNumber):
(WebCore::BorderImageSliceParseContext::commitBorderImageSlice):
(WebCore::CSSParser::parseBorderImageSlice):
(WebCore::BorderImageQuadParseContext::BorderImageQuadParseContext):
(WebCore::BorderImageQuadParseContext::commitNumber):
(WebCore::BorderImageQuadParseContext::commitBorderImageQuad):
(WebCore::CSSParser::parseBorderImageQuad):
(WebCore::CSSParser::parseBorderRadius):
(WebCore::CSSParser::parseAspectRatio):
(WebCore::CSSParser::parseCounter):
(WebCore::parseDeprecatedGradientPoint):
(WebCore::parseDeprecatedGradientColorStop):
(WebCore::CSSParser::parseDeprecatedGradient):
(WebCore::valueFromSideKeyword):
(WebCore::parseGradientColorOrKeyword):
(WebCore::CSSParser::parseLinearGradient):
(WebCore::CSSParser::parseRadialGradient):
(WebCore::CSSParser::parseCrossfade):
(WebCore::CSSParser::parseCustomFilter):
(WebCore::CSSParser::parseFilter):
(WebCore::CSSParser::parseFlowThread):
(WebCore::CSSParser::parseRegionThread):
(WebCore::CSSParser::parseTextEmphasisStyle):
(WebCore::CSSParser::parseFontFeatureSettings):

  • css/CSSParser.h:

(WebCore::CSSParser::cssValuePool):

  • css/CSSValuePool.cpp: Renamed from Source/WebCore/css/CSSPrimitiveValueCache.cpp.

(WebCore::CSSValuePool::CSSValuePool):
(WebCore::CSSValuePool::~CSSValuePool):
(WebCore::CSSValuePool::createIdentifierValue):
(WebCore::CSSValuePool::createColorValue):
(WebCore::CSSValuePool::createValue):

  • css/CSSValuePool.h: Renamed from Source/WebCore/css/CSSPrimitiveValueCache.h.

(WebCore::CSSValuePool::create):
(WebCore::CSSValuePool::createValue):

  • css/mediaControlsGtk.css:

(audio::-webkit-media-controls-volume-slider, video::-webkit-media-controls-volume-slider):

  • dom/Document.cpp:

(WebCore::Document::cssValuePool):

  • dom/Document.h:
10:17 AM Changeset in webkit [101931] by Philippe Normand
  • 2 edits in trunk/Tools

Another GTK build fix after r101922.

Rubber-stamped by Martin James Robinson.

  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp:

(LayoutTestController::pathToLocalResource): return value needs to
be a URI.

9:23 AM BuildingGtk edited by Martin Robinson
(diff)
8:37 AM Changeset in webkit [101930] by Martin Robinson
  • 2 edits in trunk/Tools

Small fix for my previous patch. Do not try to delete a member
of a dictionary that does not exist.

Reviewed by Gustavo Noronha Silva.

  • gtk/run-with-jhbuild:

(install_and_run_jhbuild):

7:55 AM Changeset in webkit [101929] by Martin Robinson
  • 2 edits in trunk/Tools

Fix jhbuild installation for people who have unusual MAKE environment
variables such as 'make -j4' as the jhbuild build does not handle it.

Reviewed by Gustavo Noronha Silva.

  • gtk/run-with-jhbuild:

(install_and_run_jhbuild): Remove any MAKE environment variable before building jhbuild.

7:39 AM Changeset in webkit [101928] by Martin Robinson
  • 2 edits in trunk/Tools

Fix the update-webkitgtk-libs script to properly trigger a jhbuild installation
when run for the first time.

Reviewed by Gustavo Noronha Silva.

  • Scripts/update-webkitgtk-libs: Fix first run.
6:07 AM Changeset in webkit [101927] by Philippe Normand
  • 2 edits in trunk/LayoutTests

Unreviewed, skip new failing tests on GTK.

  • platform/gtk/Skipped: Skip 3 new

http/tests/security/mixedContent failing partly because of bug 73743.

5:55 AM Changeset in webkit [101926] by Philippe Normand
  • 2 edits in trunk/Tools

Unreviewed, GTK build fix after r101922.

  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp:

(LayoutTestController::pathToLocalResource): Added sentinel in
g_build_filename call.

5:02 AM Changeset in webkit [101925] by Philippe Normand
  • 2 edits in trunk/Tools

GTK API tests build fix. Run the tests inside jhbuild.

Rubber-stamped by Gustavo Noronha Silva.

  • Scripts/run-gtk-tests:

(TestRunner.init):
(TestRunner.run._error_handler):
(TestRunner):
(TestRunner.run):

4:20 AM Changeset in webkit [101924] by kling@webkit.org
  • 5 edits in trunk/Source/WebCore

Shrink CSSValueList.
<​http://webkit.org/b/73732>

Reviewed by Antti Koivisto.

Packed CSSValueList::m_isSpaceSeparated into the CSSValue bit field
and renamed it to m_isSpaceSeparatedValue, shrinking CSSValueList
by one CPU word.

Also renamed CSSValue::m_isImplicit to m_isImplicitInitialValue
for good measure.

  • css/CSSInitialValue.h:

(WebCore::CSSInitialValue::CSSInitialValue):

  • css/CSSValue.h:

(WebCore::CSSValue::isImplicitInitialValue):
(WebCore::CSSValue::CSSValue):

  • css/CSSValueList.cpp:

(WebCore::CSSValueList::CSSValueList):
(WebCore::CSSValueList::copy):
(WebCore::CSSValueList::customCssText):

  • css/CSSValueList.h:

(WebCore::CSSValueList::isSpaceSeparated):

4:12 AM Changeset in webkit [101923] by Martin Robinson
  • 2 edits in trunk/Tools

Small build fix. Properly specify the path to the locally installed jhbuild.

  • Scripts/update-webkitgtk-libs: Fix jhbuild path.
3:33 AM Changeset in webkit [101922] by commit-queue@webkit.org
  • 6 edits in trunk/Tools

[GTK][DRT] Normalize ​file:///tmp/LayoutTests in LayoutTestController::pathToLocalResource()
​https://bugs.webkit.org/show_bug.cgi?id=67256

Patch by Martin Robinson <​mrobinson@igalia.com> on 2011-12-03
Reviewed by Philippe Normand.

Implement LayoutTestController::pathToLocalResource for GTK+. Instead of passing
WEBKIT_TEST_FONTS to the GTK+ test harnesses, pass a more generic WEBKIT_TOP_LEVEL,
which points to the more generic top-level path of the WebKit checkout. This code is
duplicated between WK1 and WK2 harnesses because we do not currently have a way to
share code here.

If WEBKIT_TOP_LEVEL is not provided, we search for the top level based on the binary
location. This will cause the fallback to fail if you build into a non-typical location
or even fake it with a symlink. In this case it's important to use the environment variable.

  • DumpRenderTree/gtk/DumpRenderTree.cpp:

(getTopLevelPath): Added.
(initializeFonts): Use the new helper to get the font path.

  • DumpRenderTree/gtk/DumpRenderTreeGtk.h: Expose the new helper.
  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp:

(LayoutTestController::pathToLocalResource):Use the new helper to normalize
test paths.

  • Scripts/webkitpy/layout_tests/port/gtk.py:

(GtkPort.setup_environ_for_server):Pass the top-level path of the
checkout instead of passing the font path.

  • WebKitTestRunner/InjectedBundle/gtk/ActivateFontsGtk.cpp:

(WTR::getTopLevelPath): Added this helper.
(WTR::inititializeFontConfigSetting): Use the helper to find the font path.

3:26 AM BuildingGtk edited by kov@webkit.org
(diff)
3:19 AM BuildingGtk edited by Martin Robinson
Reorganize things (diff)
3:06 AM Changeset in webkit [101921] by Philippe Normand
  • 2 edits in trunk/LayoutTests

Unreviewed, GTK rebaseline after r101858.

  • platform/gtk/fast/writing-mode/broken-ideograph-small-caps-expected.txt:
2:39 AM BuildingGtk edited by kov@webkit.org
jhbuild update (diff)
2:29 AM WebKitGtkLayoutTests edited by kov@webkit.org
Document new jhbuild stuff (diff)
2:26 AM Changeset in webkit [101920] by Philippe Normand
  • 7 edits in trunk

[GStreamer] Fullscreen controller support for the new WebKit Fullscreen API
​https://bugs.webkit.org/show_bug.cgi?id=66968

Reviewed by Martin Robinson.

Source/WebCore:

  • GNUmakefile.am: Don't use the Quicktime fullscreen CSS anymore,

it wasn't suited for GTK in the first place anyway.

  • platform/gtk/RenderThemeGtk.cpp:

(WebCore::RenderThemeGtk::extraFullScreenStyleSheet): Simply reuse
GTK CSS.

Source/WebKit/gtk:

If the element being fullscreened is a video use the native
fullscreen controls.

  • WebCoreSupport/ChromeClientGtk.cpp:

(WebKit::ChromeClient::enterFullScreenForElement):
(WebKit::ChromeClient::exitFullScreenForElement):

LayoutTests:

  • platform/gtk/Skipped: Skip

fullscreen/video-controls-override.html now failing because of bug 73583.

1:44 AM Changeset in webkit [101919] by alex
  • 2 edits in trunk/Source/WebCore

2011-12-03 Alejandro G. Castro <​alex@igalia.com>

[GTK] TextureMapperNode should not use Qt types and functions
​https://bugs.webkit.org/show_bug.cgi?id=73713

Replace qreal with double and qMin with min.

Reviewed by Noam Rosenthal.

  • platform/graphics/texmap/TextureMapperNode.cpp: (WebCore::solveCubicBezierFunction): (WebCore::solveStepsFunction):
1:36 AM Changeset in webkit [101918] by Csaba Osztrogonác
  • 4 edits in trunk/LayoutTests

[Qt] Unreviewed weekend gardening.

  • platform/qt/Skipped: Skip new failing tests.
  • platform/qt/fast/css/font-face-opentype-expected.png: Updated.
  • platform/qt/fast/css/font-face-opentype-expected.txt: Updated.
1:22 AM Changeset in webkit [101917] by Martin Robinson
  • 6 edits in trunk

.: [GTK] Update required libsoup version
​https://bugs.webkit.org/show_bug.cgi?id=71611

Patch by Dan Winship <​danw@gnome.org> on 2011-12-03
Reviewed by Martin Robinson.

  • configure.ac: require libsoup 2.37.2.1 for SoupRequestHTTP API

changes

Source/WebCore: [GTK] Remove lots of code that is now unnecessary after
SoupRequestHTTP API changes.
​https://bugs.webkit.org/show_bug.cgi?id=71611

Patch by Dan Winship <​danw@gnome.org> on 2011-12-03
Reviewed by Martin Robinson.

No new tests; behavior is unchanged

  • platform/network/ResourceHandleInternal.h:

(WebCore::ResourceHandleInternal::ResourceHandleInternal): remove
no-longer-needed fields.

  • platform/network/soup/ResourceHandleSoup.cpp:

(WebCore::finishedCallback): remove this, and m_finished, which
we no longer need to explicitly track
(WebCore::statusWillBeHandledBySoup):
(WebCore::soupErrorShouldCauseLoadFailure): No longer needed;
SoupRequestHTTP's logic is now aligned with WebKit's.
(WebCore::gotChunkCallback): remove gotChunkCallback, which
is no longer needed
(WebCore::startHTTPRequest):
(WebCore::sendRequestCallback):
(WebCore::gotHeadersCallback):
(WebCore::contentSniffedCallback): merge the code from
gotHeadersCallback and contentSniffedCallback into
sendRequestCallback

  • platform/network/soup/ResourceResponseSoup.cpp:

(WebCore::ResourceResponse::updateFromSoupMessage): move the
sniffing override logic here

1:19 AM Changeset in webkit [101916] by Martin Robinson
  • 3 edits
    4 adds in trunk/LayoutTests

[GTK] [EFL] Update some cookie-related tests
​https://bugs.webkit.org/show_bug.cgi?id=73673

Patch by Dan Winship <​danw@gnome.org> on 2011-12-03
Reviewed by Martin Robinson.

  • platform/efl/http/tests/cookies/double-quoted-value-with-semi-colon-expected.txt:
  • platform/gtk/http/tests/cookies/double-quoted-value-with-semi-colon-expected.txt:

Copied from platform/chromium because libsoup also follows the RFC
6265 behavior.

  • platform/efl/Skipped:
  • platform/gtk/Skipped: Unskip that, and another cookie test that

has been fixed for a while.

1:06 AM Changeset in webkit [101915] by Csaba Osztrogonác
  • 4 edits in trunk

Unreviewed, rolling out r101904.
​http://trac.webkit.org/changeset/101904
​https://bugs.webkit.org/show_bug.cgi?id=73739

It broke zillions of tests on all bot (Requested by
Ossy_weekend on #webkit).

Patch by Sheriff Bot <​webkit.review.bot@gmail.com> on 2011-12-03

Source/WebCore:

  • bindings/js/JSDOMWindowCustom.cpp:

(WebCore::handlePostMessage):

LayoutTests:

  • fast/dom/Window/window-postmessage-args-expected.txt:

Dec 2, 2011:

11:22 PM Changeset in webkit [101914] by haraken@chromium.org
  • 7 edits in trunk/Source/WebCore

[Refactoring] Use join(", ", @arguments) to build a method argument string in CodeGenerator*.pm
​https://bugs.webkit.org/show_bug.cgi?id=73651

Reviewed by Darin Adler.

The code in CodeGenerator*.pm to build a method argument string is really dirty
and error-prone. It is building an argument string incrementally judging whether
", " is necessary or not, like this:

my $method = ... ? "func(" : "func(a";
if (...) {

$method .= $method =~ /\($/ ? "b" : ", b";

}
$method .= ")";

Alternatively, we can refactor the code as follows:

my $funcName = "func";
my @arguments;
push(@arguments, "a") if (...);
push(@arguments, "b") if (...);
my $method = $funcName . "(" . join(", ", @arguments) . ")";

This patch just refactors the code, and generated .h and .cpp files should be
exactly the same as the current .h and .cpp files.

Tests: bindings/scripts/test/*

  • bindings/scripts/CodeGenerator.pm:

(GetterExpression):
(SetterExpression):

  • bindings/scripts/CodeGeneratorCPP.pm:

(GenerateImplementation):

  • bindings/scripts/CodeGeneratorGObject.pm:

():

  • bindings/scripts/CodeGeneratorJS.pm:

(GenerateImplementation):

  • bindings/scripts/CodeGeneratorObjC.pm:

(GenerateImplementation):

  • bindings/scripts/CodeGeneratorV8.pm:

(GenerateNormalAttrGetter):
(GenerateNormalAttrSetter):

11:06 PM Changeset in webkit [101913] by levin@chromium.org
  • 12 edits
    2 moves
    4 adds
    4 deletes in trunk

Rename WTF class from TemporarilyChange to TemporaryChange.
​https://bugs.webkit.org/show_bug.cgi?id=73479

Reviewed by Eric Seidel.

Source/JavaScriptCore:

(WTF::TemporaryChange::TemporaryChange):
(WTF::TemporaryChange::~TemporaryChange):

Source/JavaScriptGlue:

  • ForwardingHeaders/wtf/TemporarilyChange.h: Removed.
  • ForwardingHeaders/wtf/TemporaryChange.h: Added.

Source/WebCore:

  • ForwardingHeaders/wtf/TemporarilyChange.h: Removed.
  • ForwardingHeaders/wtf/TemporaryChange.h: Added.
  • page/FrameView.cpp:

(WebCore::FrameView::forceLayoutParentViewIfNeeded):
(WebCore::FrameView::layout):
(WebCore::FrameView::setScrollPosition):
(WebCore::FrameView::autoSizeIfEnabled):

Source/WebKit/mac:

  • ForwardingHeaders/wtf/TemporarilyChange.h: Removed.
  • ForwardingHeaders/wtf/TemporaryChange.h: Added.

Tools:

  • DumpRenderTree/ForwardingHeaders/wtf/TemporarilyChange.h: Removed.
  • DumpRenderTree/ForwardingHeaders/wtf/TemporaryChange.h: Added.
  • TestWebKitAPI/TestWebKitAPI.gypi:
  • TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
  • TestWebKitAPI/Tests/WTF/TemporaryChange.cpp: Renamed from Tools/TestWebKitAPI/Tests/WTF/TemporarilyChange.cpp.

(TestWebKitAPI::TEST):

  • TestWebKitAPI/win/TestWebKitAPI.vcproj:
10:56 PM Changeset in webkit [101912] by scheib@chromium.org
  • 2 edits
    2 deletes in trunk/Source/WebKit/chromium

Unreviewed, rolling out r101906.
​http://trac.webkit.org/changeset/101906
​https://bugs.webkit.org/show_bug.cgi?id=73736

Failing chromium webkit_unit_tests (Requested by scheib on
#webkit).

Patch by Sheriff Bot <​webkit.review.bot@gmail.com> on 2011-12-02

  • WebKit.gypi:
  • tests/DispatchEventTest.cpp: Removed.
  • tests/data/event_target.html: Removed.
10:47 PM Changeset in webkit [101911] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

ASSERT fails in updateState ACTION_DRAW case
​https://bugs.webkit.org/show_bug.cgi?id=73351

Patch by Armand Navabi <​armand.navabi@gmail.com> on 2011-12-02
Reviewed by James Robinson.

Changed ASSERT in CCSchedulerStateMachine to include (
!m_visible) as discussed in bug

report. m_commitState is set to COMMIT_STATE_WAITING_FOR_FIRST_DRAW if m_needsCommit or
!m_visible, so in ACTION_DRAW the assert should have both conditions.

  • platform/graphics/chromium/cc/CCSchedulerStateMachine.cpp:

(WebCore::CCSchedulerStateMachine::updateState):

10:31 PM Changeset in webkit [101910] by mhahnenberg@apple.com
  • 4 edits in trunk/Source/JavaScriptCore

REGRESSION (r99754): All layout tests crash on Windows
​https://bugs.webkit.org/show_bug.cgi?id=72305

Reviewed by Geoffrey Garen.

Fixes a crash in release builds on Windows. Windows was optimizing the out-of-line virtual destructor in
JSFunction away, which left it with no virtual functions. Its vtable ptr was then identical to that of
a different class, therefore the optimization in the visitChildren helper function in MarkedStack.cpp was calling an
incorrect version of visitChildren on the object, which left its children unmarked, causing them to be
collected when they were still reachable.

  • runtime/JSFunction.cpp:

(JSC::JSFunction::vtableAnchor): Add a virtual function to JSFunction that Visual Studio can't optimize away.

  • runtime/JSFunction.h:
  • runtime/JSGlobalData.cpp:

(JSC::JSGlobalData::storeVPtrs): Add checks to make sure that all virtual pointers that we rely on for optimization
purposes are distinct from one another.

10:21 PM Changeset in webkit [101909] by tony@chromium.org
  • 2 edits in trunk/LayoutTests

remove a stale comment in flexbox code
​https://bugs.webkit.org/show_bug.cgi?id=73720

Reviewed by Ojan Vafai.

This was fixed by Ojan in ​https://bugs.webkit.org/show_bug.cgi?id=70754.

  • css3/flexbox/flex-flow-padding.html:
10:14 PM Changeset in webkit [101908] by commit-queue@webkit.org
  • 6 edits in trunk/Source/WebCore

[chromium] Scissor rect optimization for chromium compositor
​https://bugs.webkit.org/show_bug.cgi?id=67341

Patch by Shawn Singh <​shawnsingh@chromium.org> on 2011-12-02
Reviewed by James Robinson.

Mostly covered by damage tracker tests. Currently this relies on
manually running layout tests, because this patch requires partial
swaps that are not supported by DumpRenderTree. The feature
automatically disables if partial swap is not supported.

  • platform/graphics/chromium/LayerRendererChromium.cpp:

(WebCore::LayerRendererChromium::initialize):
(WebCore::LayerRendererChromium::trackDamageForAllSurfaces):
(WebCore::LayerRendererChromium::drawLayersOntoRenderSurfaces):
(WebCore::LayerRendererChromium::drawLayersInternal):
(WebCore::LayerRendererChromium::swapBuffers):
(WebCore::LayerRendererChromium::drawLayer):

  • platform/graphics/chromium/LayerRendererChromium.h:
  • platform/graphics/chromium/cc/CCLayerTreeHost.h:

(WebCore::CCSettings::CCSettings):
(WebCore::LayerRendererCapabilities::LayerRendererCapabilities):

  • platform/graphics/chromium/cc/CCRenderSurface.cpp:

(WebCore::CCRenderSurface::draw):

  • platform/graphics/chromium/cc/CCRenderSurface.h:
10:06 PM Changeset in webkit [101907] by tony@chromium.org
  • 7 edits in trunk/Tools

[NRWT] reftest asserts intermittently on the Qt-WK2 bot
​https://bugs.webkit.org/show_bug.cgi?id=73453

Reviewed by Ryosuke Niwa.

We used to assert if a ref test didn't generate pixel results. Instead, just claim it is a failing test.
The verbose output will include the failure reason.

  • Scripts/webkitpy/layout_tests/controllers/single_test_runner.py: Don't assert, return FailureReftestNoImagesGenerated instead.
  • Scripts/webkitpy/layout_tests/controllers/test_result_writer.py:

(write_test_result):

  • Scripts/webkitpy/layout_tests/controllers/test_result_writer_unittest.py: Remove stray letter t.
  • Scripts/webkitpy/layout_tests/models/test_failures.py: Add a new failure type: FailureReftestNoImagesGenerated

(determine_result_type):
(FailureReftestNoImagesGenerated.init):
(FailureReftestNoImagesGenerated):
(FailureReftestNoImagesGenerated.message):

  • Scripts/webkitpy/layout_tests/port/test.py: Add a test case for integration tests.
  • Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py: Update test counts.

(MainTest.test_run_singly_actually_runs_tests):
(MainTest.test_unexpected_failures):
(EndToEndTest.test_end_to_end):

9:56 PM Changeset in webkit [101906] by commit-queue@webkit.org
  • 2 edits
    2 adds in trunk/Source/WebKit/chromium

Add a unit test for creating, listening for, and dispatching events
via the chromium API.
​https://bugs.webkit.org/show_bug.cgi?id=72988

Patch by Dave Michael <​dmichael@chromium.org> on 2011-12-02
Reviewed by Darin Fisher.

  • WebKit.gypi:
  • tests/DispatchEventTest.cpp: Added.

(WebKit::MockListener::MockListener):
(WebKit::MockListener::~MockListener):
(WebKit::MockListener::events):
(WebKit::DispatchEventTest::DispatchEventTest):
(WebKit::DispatchEventTest::TearDown):
(WebKit::TEST_F):

9:51 PM Changeset in webkit [101905] by Nate Chapin
  • 3 edits in trunk/Source/WebKit/chromium

Don't call FocusController:setActive() in
WebViewImpl::setFocus(), since being active and
being focused are different concepts and chromium
now properly sends setActive IPCs on all platforms.
​https://bugs.webkit.org/show_bug.cgi?id=73687

Reviewed by Eric Seidel.

  • src/WebViewImpl.cpp:

(WebKit::WebViewImpl::setFocus):

  • tests/WebViewTest.cpp: Test how different combinations of

setIsActive() and setFocus() affect whether the document
has focus.

9:40 PM Changeset in webkit [101904] by dslomov@google.com
  • 4 edits in trunk

​https://bugs.webkit.org/show_bug.cgi?id=73691
[JSC] Implement correct order of window.postMessage arguments.

Reviewed by Geoffrey Garen.

Source/WebCore:

  • bindings/js/JSDOMWindowCustom.cpp:

(WebCore::handlePostMessage):

LayoutTests:

  • fast/dom/Window/window-postmessage-args-expected.txt:
9:34 PM Changeset in webkit [101903] by commit-queue@webkit.org
  • 5 edits
    6 adds
    2 deletes in trunk

REGRESSION (r91125): Polyline tool in google docs is broken
​https://bugs.webkit.org/show_bug.cgi?id=65796

Source/WebCore:

Patch by Stephen Chenney <​schenney@chromium.org> on 2011-12-02
Reviewed by Darin Adler.

Work around a bug in CoreGraphics, that caused incorrect bounds for paths
consisting only of move-to elements. This causes problems in SVG, when the enormous
bounds prevented the drawing of things behind.

Tests: svg/custom/path-moveto-only-rendering.svg

svg/custom/subpaths-moveto-only-rendering.svg

  • platform/graphics/cg/PathCG.cpp:

(WebCore::PathIsEmptyOrSingleMoveTester::PathIsEmptyOrSingleMoveTester): Class to
test for isEmpty accoridng ot the same rules as other platforms.
(WebCore::PathIsEmptyOrSingleMoveTester::isEmpty): Query the result
(WebCore::PathIsEmptyOrSingleMoveTester::testPathElement): Path iterator method
(WebCore::PathHasOnlyMoveToTester::PathHasOnlyMoveToTester): Class to test whether a
path contains only move-to elements, and hence should have null bounds.
(WebCore::PathHasOnlyMoveToTester::hasOnlyMoveTo): Query the result
(WebCore::PathHasOnlyMoveToTester::testPathElement): Path iterator method.
(WebCore::Path::boundingRect): Modified to check for move-to only paths
(WebCore::Path::fastBoundingRect): Modified to check for move-to only paths
(WebCore::Path::isEmpty): Now uses the method that matches other platforms.
(WebCore::Path::hasCurrentPoint): Now uses CGPathIsEmpty directly
(WebCore::Path::transform) : Now uses CGPathIsEmpty directly

LayoutTests:

Work around a bug in CoreGraphics, that caused incorrect bounds for paths
consisting only of move-to elements. This causes problems in SVG, when the enormous
bounds prevented the drawing of things behind.

Will revert expectation file when expectations are stable.

Patch by Stephen Chenney <​schenney@chromium.org> on 2011-12-02
Reviewed by Darin Adler.

  • platform/chromium-mac/svg/custom/zero-path-square-cap-rendering2-expected.txt: Removed.
  • platform/chromium-win/svg/custom/zero-path-square-cap-rendering2-expected.txt: Removed.
  • platform/mac/svg/custom/path-moveto-only-rendering-expected.png: Added.
  • platform/mac/svg/custom/path-moveto-only-rendering-expected.txt: Added.
  • platform/mac/svg/custom/subpaths-moveto-only-rendering-expected.png: Added.
  • platform/mac/svg/custom/subpaths-moveto-only-rendering-expected.txt: Added.
  • svg/custom/path-moveto-only-rendering.svg: Added.
  • svg/custom/subpaths-moveto-only-rendering.svg: Added.
  • svg/custom/zero-path-square-cap-rendering2-expected.txt: Modified text output
  • platform/chromium/test_expectations.txt: Added flakiness for new tests from this patch
9:26 PM Changeset in webkit [101902] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit2

[Qt] [WK2] MiniBrowser --touch asserts in debug build
​https://bugs.webkit.org/show_bug.cgi?id=73697

Patch by Viatcheslav Ostapenko <​ostapenko.viatcheslav@nokia.com> on 2011-12-02
Reviewed by Eric Seidel.

Add missing guard to QtViewportInteractionEngine::itemSizeChanged().

  • UIProcess/qt/QtViewportInteractionEngine.cpp:

(WebKit::QtViewportInteractionEngine::itemSizeChanged):

8:59 PM Changeset in webkit [101901] by rniwa@webkit.org
  • 6 edits in trunk/Tools

ChangeLog should be able to parse annotated changelogs
​https://bugs.webkit.org/show_bug.cgi?id=72703

Reviewed by Eric Seidel.

Make parse_entries_from_file aware of svn blame annotation such as "100000 Csaba Osztrogonác ",
and extract the revision number for each entry.

Also add a support to extract "svn blame" result from svn/git checkout to scm classes.

  • Scripts/webkitpy/common/checkout/changelog.py:
  • Scripts/webkitpy/common/checkout/changelog_unittest.py:
  • Scripts/webkitpy/common/checkout/scm/git.py:
  • Scripts/webkitpy/common/checkout/scm/scm.py:
  • Scripts/webkitpy/common/checkout/scm/svn.py:
8:54 PM Changeset in webkit [101900] by scheib@chromium.org
  • 2 edits in trunk/LayoutTests

[Chromium] test_expectations typo fix for chunked-progress-event-expectedLength.html

  • platform/chromium/test_expectations.txt:
8:46 PM Changeset in webkit [101899] by mihnea@adobe.com
  • 21 edits in trunk

[CSSRegions]More renaming for region style rules.
​https://bugs.webkit.org/show_bug.cgi?id=73526

Reviewed by Eric Seidel.

Source/WebCore:

Rename isRegionStyleRule() -> isRegionRule()
WEBKIT_REGION_STYLE_RULE -> WEBKIT_REGION_RULE.

  • bindings/objc/DOMCSS.mm:

(kitClass):

  • bindings/v8/custom/V8CSSRuleCustom.cpp:

(WebCore::toV8):

  • css/CSSGrammar.y:
  • css/CSSParser.cpp:

(WebCore::CSSParser::createRegionRule):

  • css/CSSParser.h:
  • css/CSSRule.cpp:

(WebCore::CSSRule::cssText):
(WebCore::CSSRule::destroy):

  • css/CSSRule.h:

(WebCore::CSSRule::isRegionRule):

  • css/CSSRule.idl:
  • css/CSSStyleSelector.cpp:

(WebCore::CSSStyleSelector::addRegionRule):
(WebCore::RuleSet::addRulesFromSheet):

  • css/CSSStyleSelector.h:
  • css/WebKitCSSRegionRule.cpp:

(WebCore::WebKitCSSRegionRule::WebKitCSSRegionRule):

  • css/tokenizer.flex:

LayoutTests:

Rename WEBKIT_REGION_STYLE_RULE -> WEBKIT_REGION_RULE.

  • fast/dom/Window/window-properties-expected.txt:
  • fast/regions/parsing-region-style-rule.html:
  • platform/gtk/fast/dom/Window/window-properties-expected.txt:
  • platform/mac/fast/dom/Window/window-properties-expected.txt:
  • platform/qt-wk2/fast/dom/Window/window-properties-expected.txt:
  • platform/qt/fast/dom/Window/window-properties-expected.txt:
  • platform/win/fast/dom/Window/window-properties-expected.txt:
8:06 PM Changeset in webkit [101898] by tsepez@chromium.org
  • 5 edits in trunk

XSSAuditor includes more terminating characters when truncating
attribute values.
​https://bugs.webkit.org/show_bug.cgi?id=73684

Reviewed by Daniel Bates.

Source/WebCore:

  • html/parser/XSSAuditor.cpp:

(WebCore::isTerminatingCharacter):
(WebCore::XSSAuditor::eraseDangerousAttributesIfInjected):

LayoutTests:

  • http/tests/security/xssAuditor/property-escape-comment-expected.txt:
  • http/tests/security/xssAuditor/property-escape-comment.html:
7:50 PM Changeset in webkit [101897] by tony@chromium.org
  • 18 edits in trunk

Need to implement flex-flow: column-reverse
​https://bugs.webkit.org/show_bug.cgi?id=73504

Reviewed by David Hyatt.

Source/WebCore:

We can't just change the direction of the FlexOrderIterator because we want the overflow to be
on the top. We can't just position them in the reverse location since we don't know where the
bottom edge is until we've layed out the flex items. So we do an extra pass, but it should be
fast since we're not laying out, just moving.

  • rendering/RenderFlexibleBox.cpp:

(WebCore::RenderFlexibleBox::flowAwareBorderEnd):
(WebCore::RenderFlexibleBox::flowAwarePaddingEnd):
(WebCore::RenderFlexibleBox::layoutAndPlaceChildren):
(WebCore::RenderFlexibleBox::layoutColumnReverse): Do an extra pass to reposition flexitems in the reverse order.

  • rendering/RenderFlexibleBox.h:

LayoutTests:

Enable testing of column-reverse.

  • css3/flexbox/flex-flow-expected.txt:
  • css3/flexbox/flex-flow.html: Added test cases to make sure start and end still apply to the text flow direction.
  • css3/flexbox/flex-flow-border-expected.txt:
  • css3/flexbox/flex-flow-border.html: Single flexitem, so reverse does nothing.
  • css3/flexbox/flex-flow-margins-auto-size-expected.txt:
  • css3/flexbox/flex-flow-margins-auto-size.html: Ditto. The vertical tests cases (orthogonal directions) have always failed.
  • css3/flexbox/flex-flow-margins-expected.txt:
  • css3/flexbox/flex-flow-margins.html: Ditto.
  • css3/flexbox/flex-flow-orientations-expected.txt:
  • css3/flexbox/flex-flow-orientations.html: This test has 2 flexitems, the positions of which are now flipped.
  • css3/flexbox/flex-flow-overflow-expected.txt:
  • css3/flexbox/flex-flow-overflow.html: The overflow should go in the opposite direction vertically, but the same direction horizontally.
  • css3/flexbox/flex-flow-padding-expected.txt:
  • css3/flexbox/flex-flow-padding.html: Single flexitem, no change.
7:13 PM Changeset in webkit [101896] by commit-queue@webkit.org
  • 4 edits
    2 adds in trunk/Source/WebKit/chromium

Add WebArrayBuffer to chromium API
​https://bugs.webkit.org/show_bug.cgi?id=73593

Patch by Dave Michael <​dmichael@chromium.org> on 2011-12-02
Reviewed by Darin Fisher.

  • WebKit.gyp:
  • public/WebArrayBuffer.h: Added.

(WebKit::WebArrayBuffer::~WebArrayBuffer):
(WebKit::WebArrayBuffer::WebArrayBuffer):
(WebKit::WebArrayBuffer::operator=):
(WebKit::WebArrayBuffer::isNull):

  • public/WebBindings.h:
  • src/WebArrayBuffer.cpp: Added.

(WebKit::WebArrayBuffer::create):
(WebKit::WebArrayBuffer::reset):
(WebKit::WebArrayBuffer::assign):
(WebKit::WebArrayBuffer::data):
(WebKit::WebArrayBuffer::byteLength):
(WebKit::WebArrayBuffer::toV8Value):
(WebKit::WebArrayBuffer::WebArrayBuffer):
(WebKit::WebArrayBuffer::operator=):
(WebKit::WebArrayBuffer::operator WTF::PassRefPtr<WTF::ArrayBuffer>):

  • src/WebBindings.cpp:

(WebKit::getArrayBufferImpl):
(WebKit::WebBindings::getArrayBuffer):

7:12 PM Changeset in webkit [101895] by commit-queue@webkit.org
  • 4 edits
    3 adds in trunk

Divide by zero for zero-length arcs
​https://bugs.webkit.org/show_bug.cgi?id=73021

Source/WebCore:

Patch by Stephen Chenney <​schenney@chromium.org> on 2011-12-02
Reviewed by Nikolas Zimmermann.

A zero-length path produces divide by zero, resulting in nothing being
drawn. This change modifies the behavior to produce a zero-length line
which will generate correct linecaps.

Test: svg/stroke/zero-length-arc-linecaps-rendering.svg

  • svg/SVGPathParser.cpp:

(WebCore::SVGPathParser::parseArcToSegment): Catch the case of
zero-length arcs and convert them to lines (the same as would happen
for zero arc radii).

LayoutTests:

A zero-length path produces divide by zero, resulting in nothing being
drawn. This change modifies the behavior to produce a zero-length line
which will generate correct linecaps.

Patch by Stephen Chenney <​schenney@chromium.org> on 2011-12-02
Reviewed by Nikolas Zimmermann.

  • platform/chromium/test_expectations.txt: Added flakiness for the new test until wehave expectations.
  • platform/mac/svg/stroke: Added.
  • platform/mac/svg/stroke/zero-length-arc-linecaps-rendering-expected.png: Added.
  • platform/mac/svg/stroke/zero-length-arc-linecaps-rendering-expected.txt: Added.
  • svg/stroke/zero-length-arc-linecaps-rendering.svg: Added.
6:43 PM Changeset in webkit [101894] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

-Implement the SSE optimization for vsmul and vadd.
​https://bugs.webkit.org/show_bug.cgi?id=73182

Patch by James Wei <​james.wei@intel.com> & Xingnan Wang <​xingnan.wang@intel.com> on 2011-12-02
Reviewed by Kenneth Russell.

  • platform/audio/VectorMath.cpp:

(WebCore:VectorMath):

6:34 PM Changeset in webkit [101893] by Lucas Forschler
  • 5 edits in branches/safari-534.53-branch/Source

Versioning.

6:29 PM Changeset in webkit [101892] by Lucas Forschler
  • 1 copy in tags/Safari-534.53.4

New tag.

6:25 PM Changeset in webkit [101891] by Lucas Forschler
  • 5 edits in branches/safari-534.53-branch/Source

Versioning.

6:20 PM Changeset in webkit [101890] by dgrogan@chromium.org
  • 13 edits in trunk/Source

Grant workers experimental access to IndexedDB.
​https://bugs.webkit.org/show_bug.cgi?id=73609

Reviewed by Nate Chapin.

Source/WebCore:

No new tests - there will be chromium ui tests that depend on
webkit.org/b/73297.

  • storage/IDBFactory.cpp:

(WebCore::IDBFactory::open): Call new function,
IDBFactoryBackendInterface::openFromWorker.

  • storage/IDBFactoryBackendImpl.cpp:

(WebCore::IDBFactoryBackendImpl::open):
(WebCore::IDBFactoryBackendImpl::openFromWorker):
(WebCore::IDBFactoryBackendImpl::openInternal):

  • storage/IDBFactoryBackendImpl.h:
  • storage/IDBFactoryBackendInterface.h:
  • workers/WorkerContext.cpp:

(WebCore::WorkerContext::webkitIndexedDB): Stores
IDBFactoryBackendInterface, implemented by IDBFactoryBackendProxy in
chromium, in the WorkerContext. For the Document case it is stored in
the PageGroup. Storing it in the WorkerContext causes more memory
churn, but that should be trivial. I don't know of any better
alternatives.

  • workers/WorkerContext.h:
  • workers/WorkerContext.idl:

Source/WebKit/chromium:

  • src/IDBFactoryBackendProxy.cpp:

(WebKit::IDBFactoryBackendProxy::allowIDBFromWorkerThread): Return
true while behind a runtime flag, and while we figure out how to check
permission in a thread-safe manner.
(WebKit::IDBFactoryBackendProxy::openFromWorker): Using the webFrame
from the worker seems to satisfy the weak condition in chromium that
neither it nor it's associated RenderView is NULL.

  • src/IDBFactoryBackendProxy.h:
6:12 PM Changeset in webkit [101889] by Lucas Forschler
  • 5 edits in trunk/Source

Versioning.

6:07 PM Changeset in webkit [101888] by Lucas Forschler
  • 1 copy in tags/Safari-535.11

New Tag.

5:59 PM Changeset in webkit [101887] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit/efl

[EFL] Remove entry pointer before freeing it from matrix.
​https://bugs.webkit.org/show_bug.cgi?id=73625

Patch by JungJik Lee <​jungjik.lee@samsung.com> on 2011-12-02
Reviewed by Ryosuke Niwa.

To prevent leaving dangling pointer on the matrix, remove the entry before freeing it.

  • ewk/ewk_tiled_matrix.cpp:

(ewk_tile_matrix_free):

5:47 PM Changeset in webkit [101886] by oliver@apple.com
  • 11 edits in trunk/Source/JavaScriptCore

Improve float array support in the DFG JIT
​https://bugs.webkit.org/show_bug.cgi?id=73722

Reviewed by Gavin Barraclough.

Add basic support for float typed arrays in JSC. This is currently
less optimal than it could be in the following ways:

  • float32Array1[0] = float32Array2[0] (eg. an element by element copy) promotes float to double and then back to float.
  • float64Array[0] will always perform NaN tests in order to prevent signalling NaNs from entering the engine.

We also don't support Float32Array on ARMv7

  • assembler/MacroAssemblerARMv7.h:

(JSC::MacroAssemblerARMv7::loadDouble):
(JSC::MacroAssemblerARMv7::loadFloat):
(JSC::MacroAssemblerARMv7::storeDouble):
(JSC::MacroAssemblerARMv7::storeFloat):
(JSC::MacroAssemblerARMv7::convertFloatToDouble):
(JSC::MacroAssemblerARMv7::convertDoubleToFloat):

  • assembler/MacroAssemblerX86Common.h:

(JSC::MacroAssemblerX86Common::loadDouble):
(JSC::MacroAssemblerX86Common::loadFloat):
(JSC::MacroAssemblerX86Common::storeDouble):
(JSC::MacroAssemblerX86Common::storeFloat):
(JSC::MacroAssemblerX86Common::convertDoubleToFloat):
(JSC::MacroAssemblerX86Common::convertFloatToDouble):

  • assembler/X86Assembler.h:

(JSC::X86Assembler::cvtsd2ss_rr):
(JSC::X86Assembler::cvtss2sd_rr):
(JSC::X86Assembler::movsd_rm):
(JSC::X86Assembler::movss_rm):
(JSC::X86Assembler::movsd_mr):
(JSC::X86Assembler::movss_mr):

  • dfg/DFGAbstractState.cpp:

(JSC::DFG::AbstractState::execute):

  • dfg/DFGNode.h:

(JSC::DFG::Node::shouldSpeculateFloat32Array):

  • dfg/DFGPropagator.cpp:

(JSC::DFG::Propagator::propagateNodePredictions):

  • dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compilePutByValForIntTypedArray):
(JSC::DFG::SpeculativeJIT::compileGetByValOnFloatTypedArray):
(JSC::DFG::SpeculativeJIT::compilePutByValForFloatTypedArray):

  • dfg/DFGSpeculativeJIT.h:
  • dfg/DFGSpeculativeJIT32_64.cpp:

(JSC::DFG::SpeculativeJIT::compile):

  • dfg/DFGSpeculativeJIT64.cpp:

(JSC::DFG::SpeculativeJIT::fillSpeculateIntInternal):
(JSC::DFG::SpeculativeJIT::compile):

5:42 PM Changeset in webkit [101885] by jamesr@google.com
  • 17 edits
    7 copies in trunk/Source/WebKit/chromium

[chromium] Move WebLayer APIs to platform directory
​https://bugs.webkit.org/show_bug.cgi?id=73610

Reviewed by Darin Fisher.

  • public/WebContentLayer.h:
  • public/WebContentLayerClient.h:
  • public/WebExternalTextureLayer.h:
  • public/WebLayer.h:
  • public/WebLayerClient.h:
  • public/WebLayerTreeView.h:
  • public/WebLayerTreeViewClient.h:
  • public/platform/WebContentLayer.h: Copied from Source/WebKit/chromium/public/WebContentLayer.h.

(WebKit::WebContentLayer::WebContentLayer):
(WebKit::WebContentLayer::~WebContentLayer):
(WebKit::WebContentLayer::operator=):

  • public/platform/WebContentLayerClient.h: Copied from Source/WebKit/chromium/public/WebContentLayerClient.h.

(WebKit::WebContentLayerClient::~WebContentLayerClient):

  • public/platform/WebExternalTextureLayer.h: Copied from Source/WebKit/chromium/public/WebExternalTextureLayer.h.

(WebKit::WebExternalTextureLayer::WebExternalTextureLayer):
(WebKit::WebExternalTextureLayer::~WebExternalTextureLayer):
(WebKit::WebExternalTextureLayer::operator=):

  • public/platform/WebLayer.h: Copied from Source/WebKit/chromium/public/WebLayer.h.

(WebKit::WebLayer::WebLayer):
(WebKit::WebLayer::~WebLayer):
(WebKit::WebLayer::operator=):
(WebKit::WebLayer::isNull):
(WebKit::WebLayer::to):
(WebKit::WebLayer::toConst):
(WebKit::WebLayer::unwrap):
(WebKit::WebLayer::constUnwrap):
(WebKit::operator==):
(WebKit::operator!=):

  • public/platform/WebLayerClient.h: Copied from Source/WebKit/chromium/public/WebLayerClient.h.

(WebKit::WebLayerClient::~WebLayerClient):

  • public/platform/WebLayerTreeView.h: Copied from Source/WebKit/chromium/public/WebLayerTreeView.h.

(WebKit::WebLayerTreeView::Settings::Settings):
(WebKit::WebLayerTreeView::WebLayerTreeView):
(WebKit::WebLayerTreeView::~WebLayerTreeView):
(WebKit::WebLayerTreeView::operator=):
(WebKit::operator==):
(WebKit::operator!=):

  • public/platform/WebLayerTreeViewClient.h: Copied from Source/WebKit/chromium/public/WebLayerTreeViewClient.h.

(WebKit::WebLayerTreeViewClient::~WebLayerTreeViewClient):

  • public/platform/WebPrivatePtr.h:
  • src/WebContentLayer.cpp:
  • src/WebContentLayerImpl.cpp:
  • src/WebExternalTextureLayer.cpp:
  • src/WebExternalTextureLayerImpl.cpp:
  • src/WebLayer.cpp:
  • src/WebLayerImpl.cpp:
  • src/WebLayerTreeView.cpp:
  • src/WebLayerTreeViewImpl.h:
5:42 PM Changeset in webkit [101884] by Lucas Forschler
  • 2 edits in branches/safari-534.53-branch/Source/WebCore

Merged r90959.

5:38 PM Changeset in webkit [101883] by commit-queue@webkit.org
  • 8 edits
    11 adds in trunk

Source/WebCore: Fix mixed content handling for video in Chromium by having
CachedResourceLoader & SubresourceLoader use the
ResourceRequest::TargetType when determining if a RawResource can
be requested.
​https://bugs.webkit.org/show_bug.cgi?id=72178

Patch by Aaron Colwell <​acolwell@chromium.org> on 2011-12-02
Reviewed by Adam Barth.

Tests: http/tests/security/mixedContent/insecure-video-in-iframe.html

http/tests/security/mixedContent/insecure-video-in-main-frame.html
http/tests/security/mixedContent/redirect-http-to-https-video-in-main-frame.html
http/tests/security/mixedContent/redirect-https-to-http-video-in-main-frame.html

  • loader/SubresourceLoader.cpp:

(WebCore::SubresourceLoader::willSendRequest):

  • loader/cache/CachedRawResource.cpp:

(WebCore::CachedRawResource::CachedRawResource):

  • loader/cache/CachedRawResource.h:
  • loader/cache/CachedResource.cpp:

(WebCore::defaultPriorityForResourceType):
(WebCore::cachedResourceTypeToTargetType):
(WebCore::CachedResource::targetTypeToCachedResourceType):

  • loader/cache/CachedResource.h:
  • loader/cache/CachedResourceLoader.cpp:

(WebCore::createResource):
(WebCore::CachedResourceLoader::checkInsecureContent):
(WebCore::CachedResourceLoader::canRequest):
(WebCore::CachedResourceLoader::requestResource):

LayoutTests: Adding mixed content tests for video tag.
​https://bugs.webkit.org/show_bug.cgi?id=72178

Patch by Aaron Colwell <​acolwell@chromium.org> on 2011-12-02
Reviewed by Adam Barth.

  • http/tests/security/mixedContent/insecure-video-in-iframe-expected.txt: Added.
  • http/tests/security/mixedContent/insecure-video-in-iframe.html: Added.
  • http/tests/security/mixedContent/insecure-video-in-main-frame-expected.txt: Added.
  • http/tests/security/mixedContent/insecure-video-in-main-frame.html: Added.
  • http/tests/security/mixedContent/redirect-http-to-https-video-in-main-frame-expected.txt: Added.
  • http/tests/security/mixedContent/redirect-http-to-https-video-in-main-frame.html: Added.
  • http/tests/security/mixedContent/redirect-https-to-http-video-in-main-frame-expected.txt: Added.
  • http/tests/security/mixedContent/redirect-https-to-http-video-in-main-frame.html: Added.
  • http/tests/security/mixedContent/resources/frame-with-insecure-video.html: Added.
  • http/tests/security/mixedContent/resources/frame-with-redirect-http-to-https-video.html: Added.
  • http/tests/security/mixedContent/resources/frame-with-redirect-https-to-http-video.html: Added.
5:35 PM Changeset in webkit [101882] by scheib@chromium.org
  • 1 edit
    2 copies
    1 move
    3 adds in trunk/LayoutTests

[Chromium] Rebaselining svg/custom/svg-fonts-word-spacing.html

  • platform/chromium-cg-mac-snowleopard/accessibility/aria-checkbox-sends-notification-expected.txt: Copied from LayoutTests/platform/chromium/accessibility/aria-checkbox-sends-notification-expected.txt.
  • platform/chromium-mac-snowleopard/accessibility/aria-checkbox-sends-notification-expected.txt: Copied from LayoutTests/platform/chromium/accessibility/aria-checkbox-sends-notification-expected.txt.
  • platform/chromium-win/accessibility/aria-checkbox-sends-notification-expected.txt: Renamed from LayoutTests/platform/chromium/accessibility/aria-checkbox-sends-notification-expected.txt.
5:28 PM Changeset in webkit [101881] by scheib@chromium.org
  • 2 edits in trunk/LayoutTests

[Chromium] Marking svg/W3C-SVG-1.1/fonts-elem-04-b.svg flaky for image+text

  • platform/chromium/test_expectations.txt:
5:04 PM Changeset in webkit [101880] by eric@webkit.org
  • 2 edits in trunk/Tools

Enable parallel testing for Win and Gtk bots
after confirming with Lucas and Martin the bots should be OK.

Unreviewed. (I ran the idea by Lucas and Martin.)

  • Scripts/run-webkit-tests:

(platformIsReadyForParallelTesting):

4:53 PM Changeset in webkit [101879] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebKit/chromium

[chromium] add setOpener method to WebFrame
​https://bugs.webkit.org/show_bug.cgi?id=73379

Patch by Karl Koscher <​supersat@chromium.org> on 2011-12-02
Reviewed by Darin Fisher.

  • public/WebFrame.h:

(WebKit::WebFrame::clearOpener): now an inline call to setOpener(0)

  • src/WebFrameImpl.cpp:

(WebKit::WebFrameImpl::setOpener): sets window.opener

  • src/WebFrameImpl.h:
4:41 PM Changeset in webkit [101878] by Lucas Forschler
  • 9 edits in branches/safari-534.53-branch/Source

Merge 93445.

4:21 PM Changeset in webkit [101877] by Lucas Forschler
  • 13 edits
    1 copy in branches/safari-534.53-branch/Source

Merge 93377.

4:13 PM Changeset in webkit [101876] by dcheng@chromium.org
  • 6 edits
    2 deletes in trunk

Unreviewed, rolling out r101731.
​http://trac.webkit.org/changeset/101731
​https://bugs.webkit.org/show_bug.cgi?id=73706

Broke copy and paste in chromium. (Requested by dcheng on #webkit).

Patch by Sheriff Bot <​webkit.review.bot@gmail.com> on 2011-12-02

Source/WebCore:

  • editing/SpellChecker.cpp:

(WebCore::SpellChecker::SpellChecker):
(WebCore::SpellChecker::initRequest):
(WebCore::SpellChecker::clearRequest):
(WebCore::SpellChecker::canCheckAsynchronously):
(WebCore::SpellChecker::isBusy):
(WebCore::SpellChecker::isValid):
(WebCore::SpellChecker::requestCheckingFor):
(WebCore::SpellChecker::doRequestCheckingFor):
(WebCore::SpellChecker::didCheck):

  • editing/SpellChecker.h:

LayoutTests:

  • editing/spelling/spellcheck-queue-expected.txt: Removed.
  • editing/spelling/spellcheck-queue.html: Removed.
  • platform/gtk/Skipped:
  • platform/qt/Skipped:
4:04 PM Changeset in webkit [101875] by fsamuel@chromium.org
  • 15 edits
    9 adds in trunk

When page scaling is in use position:fixed has incorrect results
​https://bugs.webkit.org/show_bug.cgi?id=68617

Reviewed by Simon Fraser.

.:

  • Source/autotools/symbols.filter:

Source/WebCore:

Add the option for position:fixed elements to be fixed to the frame
instead of the layout rectangle of the document.

Tests: fast/repaint/fixed-in-page-scale.html

fast/repaint/fixed-right-bottom-in-page-scale.html
fast/repaint/fixed-right-in-page-scale.html

  • WebCore.exp.in:
  • page/FrameView.cpp:

(WebCore::FrameView::reset):
(WebCore::FrameView::scrollXForFixedPosition):
(WebCore::FrameView::scrollYForFixedPosition):

If position:fixed elements are relative to the frame, disregard the
drag factor.

(WebCore::FrameView::setShouldLayoutFixedElementsRelativeToFrame):

  • page/FrameView.h:

(WebCore::FrameView::shouldLayoutFixedElementsRelativeToFrame):

  • rendering/RenderBox.cpp:

(WebCore::RenderBox::shouldLayoutFixedElementRelativeToFrame):
(WebCore::RenderBox::containingBlockLogicalWidthForPositioned):
(WebCore::RenderBox::containingBlockLogicalHeightForPositioned):

If position:fixed elements are relative to the frame, their container
is the frame instead of the layout rect of the document.
This allows proper positioning of these elements to the right and
bottom.

  • rendering/RenderBox.h:
  • testing/Internals.cpp:

(WebCore::Internals::setShouldLayoutFixedElementsRelativeToFrame):

  • testing/Internals.h:
  • testing/Internals.idl: Allow enabling and disabling the new behavior in layout tests.

Source/WebKit2:

  • win/WebKit2.def:
  • win/WebKit2CFLite.def:

LayoutTests:

  • fast/repaint/fixed-in-page-scale-expected.png: Added.
  • fast/repaint/fixed-in-page-scale-expected.txt: Added.
  • fast/repaint/fixed-in-page-scale.html: Added.
  • fast/repaint/fixed-right-bottom-in-page-scale-expected.png: Added.
  • fast/repaint/fixed-right-bottom-in-page-scale-expected.txt: Added.
  • fast/repaint/fixed-right-bottom-in-page-scale.html: Added.
  • fast/repaint/fixed-right-in-page-scale-expected.png: Added.
  • fast/repaint/fixed-right-in-page-scale-expected.txt: Added.
  • fast/repaint/fixed-right-in-page-scale.html: Added.
3:55 PM Changeset in webkit [101874] by benjamin@webkit.org
  • 2 edits in trunk/Source/WebCore

Build fix for SubresourceLoader when building with Core Foundation
​https://bugs.webkit.org/show_bug.cgi?id=73709

Patch by Benjamin Poulain <​bpoulain@apple.com> on 2011-12-02
Reviewed by David Kilzer.

The patch r100311 removed SubresourceClient and merged it in SubresourceLoader.
Consequently, m_client does not exist anymore and there is no need to do the check
before invoking didReceiveData().

  • loader/cf/SubresourceLoaderCF.cpp:

(WebCore::SubresourceLoader::didReceiveDataArray):

3:54 PM Changeset in webkit [101873] by Simon Fraser
  • 2 edits in trunk/Source/WebKit/mac

<rdar://problem/10423627> Layers are flipped in WebKit1 views

Reviewed by Sam Weinig.

Host the compositing layers in an NSView which is flipped, which helps AppKit
get the geometry right in some situations.

  • WebView/WebHTMLView.mm:

(-[WebLayerHostingFlippedView isFlipped]):
(-[WebHTMLView attachRootLayer:]):

3:54 PM Changeset in webkit [101872] by Lucas Forschler
  • 2 edits in branches/safari-534.53-branch/Source/WebCore

Merge 100809.

3:48 PM Changeset in webkit [101871] by mitz@apple.com
  • 1 edit in trunk/Source/WebCore/ChangeLog

Fixed a typo

3:48 PM Changeset in webkit [101870] by kling@webkit.org
  • 2 edits in trunk/Source/WebCore

StyledElement: Simplify addCSSColor().
<​http://webkit.org/b/73703>

Reviewed by Darin Adler.

The Color(const String&) constructor handles both named and 3/6-digit
hex colors, so there's no need to handle those separately here.
Also tweaked some comments and minor things.

  • dom/StyledElement.cpp:

(WebCore::StyledElement::addCSSColor):

3:45 PM Changeset in webkit [101869] by Martin Robinson
  • 2 edits in trunk

Try to fix the GTK+ debug bot. As discussed with other GTK+
hackers recently. It makes sense to just remove G_DISABLE_DEPRECATIONS
now in WebKit. The flag itself is now deprecated in GLib in favor of
the gcc attribute.

  • GNUmakefile.am: Remove G_DISABLE_DEPRECATED.
3:38 PM Changeset in webkit [101868] by scheib@chromium.org
  • 2 edits in trunk/LayoutTests

[Chromium] Marking svg/W3C-SVG-1.1/fonts-elem-04-b.svg flaky for text

  • platform/chromium/test_expectations.txt:
3:36 PM Changeset in webkit [101867] by benjamin@webkit.org
  • 5 edits
    3 adds
    1 delete in trunk/Source/WebCore

Update platform/iphone to platform/ios
​https://bugs.webkit.org/show_bug.cgi?id=73708

Patch by Benjamin Poulain <​bpoulain@apple.com> on 2011-12-02
Reviewed by Darin Adler.

The platform is now best known as iOS, update the platform layer accordingly.

  • Configurations/WebCore.xcconfig:
  • WebCore.gypi:
  • WebCore.xcodeproj/project.pbxproj:
  • platform/cocoa/KeyEventCocoa.mm:
  • platform/ios/KeyEventCodesIOS.h: Renamed from Source/WebCore/platform/iphone/KeyEventCodesIPhone.h.
  • platform/ios/KeyEventIOS.mm: Renamed from Source/WebCore/platform/iphone/KeyEventIPhone.mm.

(WebCore::keyIdentifierForKeyEvent):
(WebCore::PlatformKeyboardEvent::PlatformKeyboardEvent):
(WebCore::PlatformKeyboardEvent::disambiguateKeyDownEvent):
(WebCore::PlatformKeyboardEvent::currentCapsLockState):
(WebCore::PlatformKeyboardEvent::getCurrentModifierState):

3:31 PM Changeset in webkit [101866] by scheib@chromium.org
  • 2 edits in trunk/LayoutTests

[Chromium] Adding test expectation that svg/animations/svginteger-animation-1.html crashes leopard cg

  • platform/chromium/test_expectations.txt:
3:18 PM Changeset in webkit [101865] by scheib@chromium.org
  • 2 edits in trunk/LayoutTests

[Chromium] Marking http/tests/appcache/video.html TIMEOUT

  • platform/chromium/test_expectations.txt:
3:03 PM Changeset in webkit [101864] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebCore

[chromium] Eliminate unnecessary state on previous CL
​https://bugs.webkit.org/show_bug.cgi?id=73661

Patch by Jonathan Backer <​backer@chromium.org> on 2011-12-02
Reviewed by Kenneth Russell.

  • platform/graphics/chromium/LayerRendererChromium.cpp:

(WebCore::LayerRendererChromium::initialize):

  • platform/graphics/chromium/cc/CCLayerTreeHost.cpp:

(WebCore::CCLayerTreeHost::initialize):
(WebCore::CCLayerTreeHost::didBecomeInvisibleOnImplThread):

  • platform/graphics/chromium/cc/CCLayerTreeHost.h:

(WebCore::CCSettings::CCSettings):
(WebCore::LayerRendererCapabilities::LayerRendererCapabilities):

3:02 PM Changeset in webkit [101863] by scheib@chromium.org
  • 2 edits in trunk/LayoutTests

[Chromium] Suppressing failures in test_expectations for 101844

  • platform/chromium/test_expectations.txt:
2:59 PM Changeset in webkit [101862] by Martin Robinson
  • 2 edits in trunk/Tools

Try to fix the GTK+ build.

  • gtk/jhbuild.modules: Properly specify the glib-networking

dependency for soup.

2:58 PM Changeset in webkit [101861] by Martin Robinson
  • 2 edits in trunk/Tools

Fix a syntax error in the committers.py file.

  • Scripts/webkitpy/common/config/committers.py: Add a missing comma.
2:56 PM Changeset in webkit [101860] by ap@apple.com
  • 2 edits in trunk/Source/WebKit2

Update sandbox rules
​https://bugs.webkit.org/show_bug.cgi?id=73675
<rdar://problem/9276430>

Reviewed by Sam Weinig.

  • WebProcess/com.apple.WebProcess.sb:
2:49 PM Changeset in webkit [101859] by rniwa@webkit.org
  • 2 edits in trunk/Tools

Add Alan Stearns and Peter Linss to the contributors list.

  • Scripts/webkitpy/common/config/committers.py:
2:49 PM Changeset in webkit [101858] by mitz@apple.com
  • 21 edits in trunk

<rdar://problem/10520670> REGRESSION (r91738): didFinishLoad is called before custom fonts have finished loading
​https://bugs.webkit.org/show_bug.cgi?id=73688

Reviewed by Darin Adler.

Source/WebCore:

The problem was that after CSSFontFaceSource::getFontData() had scheduled a 0-delay timer to
begin loading the font, but before that timer fired, the subresource loader appeared to have
had no resources waiting to be loaded, and therefore didFinishLoad could be called. This change
reworks the fix for <​http://webkit.org/b/65123> so that while the load is still started on a
0-dealy timer, the subresource loader’s request count is incremented immediately, preventing
it from hitting 0 while the font load is scheduled to begin. The delayed load mechanism is
moved from CSSFontFaceSource into CSSFontSelector in order to safely handle the possibility of
the latter being decommissioned while waiting for font loading to begin.

  • css/CSSFontFaceSource.cpp:

(WebCore::CSSFontFaceSource::CSSFontFaceSource): Removed initializer for m_loadStartTimer.
(WebCore::CSSFontFaceSource::~CSSFontFaceSource): Removed stopping of m_loadStartTimer.
(WebCore::CSSFontFaceSource::getFontData): Replaced code to schedule loading on a timer with
a call to CSSFontSelector::beginLoadingFontSoon.

  • css/CSSFontFaceSource.h: Removed m_loadStartTimer and m_fontSelector member variables.
  • css/CSSFontSelector.cpp:

(WebCore::CSSFontSelector::CSSFontSelector): Added initialized for m_beginLoadingTimer.
(WebCore::CSSFontSelector::~CSSFontSelector): Added call to clearDocument(), to deal with
anything remaining in m_fontsToBeginLoading at this time.
(WebCore::CSSFontSelector::clearDocument): Now stops m_beginLoadingTimer and balances
incrementRequestCount() calls for anything remaining in m_fontsToBeginLoading.
(WebCore::CSSFontSelector::beginLoadingFontSoon): Added. Schedules the actual call to
CachedFont::beginLoadingIfNeeded on a 0-delay timer, and meanwhile increments the request count
on the CachedResourceLoader, which ensures that didFinishLoad will not be called while waiting
for the timer to fire.
(WebCore::CSSFontSelector::beginLoadTimerFired): Added. Actually calls
CachedFont::beginLoadIfNeeded and balances the incrementRequestCount() made when the timer was
scheduled.

  • css/CSSFontSelector.h:

LayoutTests:

Reverted test chnages from r91738. These tests were made to wait for the fonts to finish loading
using a DOM. Now again they do not need to.

  • fast/css/color-leakage.html:
  • fast/css/custom-font-xheight.html:
  • fast/css/font-face-multiple-faces.html:
  • fast/css/font-face-multiple-remote-sources.html:
  • fast/css/font-face-remote.html:
  • fast/css/font-face-woff.html:
  • fast/writing-mode/broken-ideograph-small-caps.html:
  • svg/W3C-SVG-1.1-SE/text-intro-09-b.svg:
  • svg/W3C-SVG-1.1/fonts-elem-07-b.svg:
  • svg/custom/svg-fonts-fallback.xhtml:
  • svg/custom/svg-fonts-in-html.html:
  • svg/custom/svg-fonts-segmented.xhtml:
  • svg/custom/svg-fonts-with-no-element-reference.html:
  • svg/custom/svg-fonts-without-missing-glyph.xhtml:
  • svg/text/text-overflow-ellipsis-svgfont.html:
2:42 PM Changeset in webkit [101857] by tkent@chromium.org
  • 2 edits in trunk/Tools

[Chromium] Fix compilation warnings for ASSERT()
​https://bugs.webkit.org/show_bug.cgi?id=73623

Reviewed by Tony Chang.

  • DumpRenderTree/chromium/ImageDiff.cpp:
    • Fix fprintf format: %s -> %d for LINE
    • Do nothing for ASSERT() if NDEBUG.
2:37 PM Changeset in webkit [101856] by eric@webkit.org
  • 2 edits in trunk/Tools

Enable parallel testing for all Mac bots.

Unreviewed.

  • Scripts/run-webkit-tests:

(platformIsReadyForParallelTesting):

2:34 PM Changeset in webkit [101855] by commit-queue@webkit.org
  • 9 edits
    2 adds in trunk

Send an AXCheckedStateChanged notification when the aria-checked attribute changes.
​https://bugs.webkit.org/show_bug.cgi?id=72754

Patch by David Tseng <​dtseng@google.com> on 2011-12-02
Reviewed by Chris Fleizach.

Source/WebCore:

Test: accessibility/aria-checkbox-sends-notification.html

  • accessibility/AXObjectCache.cpp:

(WebCore::AXObjectCache::checkedStateChanged):

  • accessibility/AXObjectCache.h:
  • dom/Element.cpp:

(WebCore::Element::updateAfterAttributeChanged):

  • html/HTMLInputElement.cpp:

(WebCore::HTMLInputElement::setChecked):

LayoutTests:

  • accessibility/aria-checkbox-sends-notification.html: Added.
  • platform/chromium/accessibility/aria-checkbox-sends-notification-expected.txt: Added.
2:30 PM Changeset in webkit [101854] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebCore

[chromium] Recycle tile-sized textures during commit to prevent reallocations
​https://bugs.webkit.org/show_bug.cgi?id=70645

Patch by Grace Kloba <​klobag@chromium.org> on 2011-12-02
Reviewed by James Robinson.

Currently texture request is capped by the high limit while we reclaim the
textures in each commit. This triggers new tiles always allocated when scrolling.
The proposal is to recycle the texture during request if the total used memory
is about to exceed the reclaim limit.

  • platform/graphics/chromium/ManagedTexture.cpp:

(WebCore::ManagedTexture::reserve):

  • platform/graphics/chromium/TextureManager.cpp:

(WebCore::TextureManager::setMemoryLimitBytes):
(WebCore::TextureManager::replaceTexture):
(WebCore::TextureManager::requestTexture):

  • platform/graphics/chromium/TextureManager.h:
2:26 PM Changeset in webkit [101853] by eric@webkit.org
  • 2 edits in trunk/Tools

Enable parallel testing for run-webkit-tests on Mac Lion.
I plan to move all the machines to parallel testing shortly
but we're starting with Lion.

Unreviewed.

  • Scripts/run-webkit-tests:

(platformIsReadyForParallelTesting):

2:17 PM Changeset in webkit [101852] by tkent@chromium.org
  • 3 edits
    2 deletes in trunk

Unreviewed, rolling out r101337.
​http://trac.webkit.org/changeset/101337
​https://bugs.webkit.org/show_bug.cgi?id=73189

It's a wrong way to fix the problem

Source/WebKit/chromium:

  • src/WebViewImpl.cpp:

(WebKit::WebViewImpl::popupOpened):

LayoutTests:

  • fast/forms/select-popup-crash-expected.txt: Removed.
  • fast/forms/select-popup-crash.html: Removed.
2:07 PM Changeset in webkit [101851] by tkent@chromium.org
  • 8 edits
    2 adds in trunk

Added helper method to identify whether the page has custom page size style.
​https://bugs.webkit.org/show_bug.cgi?id=73585

Patch by Kausalya Madhusudhanan <​kmadhusu@chromium.org> on 2011-12-02
Reviewed by Darin Fisher.

Source/WebKit/chromium:

  • public/WebFrame.h:
  • src/WebFrameImpl.cpp:

(WebKit::WebFrameImpl::hasCustomPageSizeStyle):

  • src/WebFrameImpl.h:

Tools:

  • DumpRenderTree/chromium/LayoutTestController.cpp:

(LayoutTestController::LayoutTestController):
(LayoutTestController::hasCustomPageSizeStyle):

  • DumpRenderTree/chromium/LayoutTestController.h:

LayoutTests:

  • platform/chromium/printing/custom-page-size-style-expected.txt: Added.
  • platform/chromium/printing/custom-page-size-style.html: Added.
2:02 PM Changeset in webkit [101850] by scheib@chromium.org
  • 2 edits in trunk/LayoutTests

[Chromium] Adding flaky tests to test_expectations

inspector/extensions tests even flakier
newly flaky media/event-attributes.html

  • platform/chromium/test_expectations.txt:
1:59 PM Changeset in webkit [101849] by dslomov@google.com
  • 2 edits in trunk/LayoutTests

Unreviewed: reset expectation for tests until ​https://bugs.webkit.org/show_bug.cgi?id=73691
is fixed.

  • fast/dom/Window/window-postmessage-args-expected.txt:
1:55 PM Changeset in webkit [101848] by tkent@chromium.org
  • 5 edits in trunk/Source/WebCore

[Chromium] Show placeholder even if the element is focused
​https://bugs.webkit.org/show_bug.cgi?id=73629

Reviewed by Hajime Morita.

No new tests. Need to update some existing placeholder tests.

  • rendering/RenderThemeChromiumMac.h: Add shouldShowPlaceholderWhenFocused().
  • rendering/RenderThemeChromiumMac.mm:

(WebCore::RenderThemeChromiumMac::shouldShowPlaceholderWhenFocused):
Returns true.

  • rendering/RenderThemeChromiumSkia.cpp:

(WebCore::RenderThemeChromiumSkia::shouldShowPlaceholderWhenFocused): ditto.

  • rendering/RenderThemeChromiumSkia.h: Add shouldShowPlaceholderWhenFocused().
1:49 PM Changeset in webkit [101847] by tkent@chromium.org
  • 5 edits
    2 adds in trunk

[Lion][Windows] Both of placeholder and input text are shown in <input type=number>
​https://bugs.webkit.org/show_bug.cgi?id=73615

Reviewed by Joseph Pecoraro.

Source/WebCore:

Placeholder visibility was checked by HTMLInputElement::value
emptiness. It should be innerTextValue emptiness because it is
possible that a number field has empty HTMLInputElement::value and
non-empty innerTextValue.

Tests: fast/forms/number/number-placeholder-with-unacceptable-value.html

  • html/HTMLInputElement.cpp:

(WebCore::HTMLInputElement::updateInnerTextValue):
We should update placeholder visiblity when the innerTextValue is updated.
(WebCore::HTMLInputElement::subtreeHasChanged): ditto.
(WebCore::HTMLInputElement::setSuggestedValue):
We don't need to call updatePlaceholderVisibility() because updateInnerTextValue() calls it.
(WebCore::HTMLInputElement::setValueFromRenderer):
We don't need to call updatePlaceholderVisibility() because subtreeHasChanged() calls it.

  • html/HTMLInputElement.h: Checks innerTextValue emptiness.
  • html/TextFieldInputType.cpp:

(WebCore::TextFieldInputType::setValue):
We don't need to call updatePlaceholderVisibility() because updateInnerTextValue() calls it.

LayoutTests:

  • fast/forms/number/number-placeholder-with-unacceptable-value-expected.html: Added.
  • fast/forms/number/number-placeholder-with-unacceptable-value.html: Added.
1:49 PM Changeset in webkit [101846] by tony@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

[Chromium] GYP fix for Windows 2010 build
​https://bugs.webkit.org/show_bug.cgi?id=73483

Reviewed by Adam Barth.

Patch by Carlos Pizano.

  • WebKit.gyp: Exclude the android directory and remove a file match that doesn't match anything.
1:42 PM Changeset in webkit [101845] by rniwa@webkit.org
  • 10 edits in trunk/Tools

[NRWT] reftest should support having multiple references per test
​https://bugs.webkit.org/show_bug.cgi?id=73613

Reviewed by Dirk Pranke.

Add a support for having multiple reference files for a single test.

Because a reftest succeeds when it matches at least one of expected matches and fails when it matches
at least one of expected mismatches, we compare expected mismatches first in order to minimize
the number of reference files to open on DRT.

  • Scripts/webkitpy/layout_tests/controllers/manager.py:

(interpret_test_failures): Remove checks no longer applicable.

  • Scripts/webkitpy/layout_tests/controllers/manager_unittest.py:

(ResultSummaryTest.test_interpret_test_failures): Ditto.

  • Scripts/webkitpy/layout_tests/controllers/single_test_runner.py:

(SingleTestRunner.init): Remove a bunch of code and just call port.reference_files.
(SingleTestRunner._driver_input):
(SingleTestRunner.run):
(SingleTestRunner._run_reftest): Compare the output of the test to each reference file.

  • Scripts/webkitpy/layout_tests/models/test_input.py:

(TestInput.init): Remove ref_file and is_mismatch_reftest because they are no longer used.

  • Scripts/webkitpy/layout_tests/port/base.py:

(Port.reference_files): Renamed from _reference_file_for. Returns a list of expectation, filename pairs.
(_parse_reftest_list): Now supports parsing multiple entries for a single test.

  • Scripts/webkitpy/layout_tests/port/base_unittest.py:

(PortTest.test_parse_reftest_list):

  • Scripts/webkitpy/layout_tests/port/test.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py:

(MainTest.test_unexpected_failures):
(MainTest.test_reftest_skipped_if_unlisted): Renamed from test_missing_and_unexpected_results.
(EndToEndTest.test_end_to_end):
(EndToEndTest.test_reftest_with_two_notrefs): Added.

1:08 PM Changeset in webkit [101844] by alokp@chromium.org
  • 6 edits
    1 add in trunk

Source/WebCore: [chromium] CCLayerQuad does not return FloatQuad in correct order
​https://bugs.webkit.org/show_bug.cgi?id=73247

Reviewed by James Robinson.

Returned the FloatQuad coordinates in correct order.

Covered by new unit tests in CCLayerQuadTest.cpp.

  • platform/graphics/chromium/cc/CCLayerQuad.cpp:

(WebCore::CCLayerQuad::floatQuad):

Source/WebKit/chromium: [chromium] CCLayerQuad does not return FloatQuad in correct order
​https://bugs.webkit.org/show_bug.cgi?id=73247

Reviewed by James Robinson.

  • WebKit.gypi:
  • tests/CCLayerQuadTest.cpp: Added.

(WebCore::TEST):

LayoutTests: [chromium] CCLayerQuad does not return FloatQuad in correct order
​https://bugs.webkit.org/show_bug.cgi?id=73247

Reviewed by James Robinson.

  • platform/chromium/test_expectations.txt:
11:57 AM Changeset in webkit [101843] by scheib@chromium.org
  • 2 edits in trunk/LayoutTests

[Chromium] Marking slow test: http/tests/appcache/video.html

  • platform/chromium/test_expectations.txt:
11:39 AM Changeset in webkit [101842] by adamk@chromium.org
  • 3 edits in trunk/Source/WebKit/chromium

[chromium] WebKitMutationObserver::deliverAllMutations should be exposed through the Chromium API
​https://bugs.webkit.org/show_bug.cgi?id=71242

Reviewed by Darin Fisher.

Relanding, depends on ​http://codereview.chromium.org/8748015/
to fix a sandboxing issue on Chromium/Linux.

Add addTaskObserver and removeTaskObserver to WebThread,
along with a new WebThread::TaskObserver interface.

For mutation observers, add a TaskObserver to the main thread
to deliver mutations after each task runs.

The Chromium implementation of the new WebThread methods is in
​http://codereview.chromium.org/8586038/.

  • public/platform/WebThread.h:

(WebKit::WebThread::TaskObserver::~TaskObserver):

  • src/WebKit.cpp:

(WebKit::initialize):
(WebKit::shutdown):

11:34 AM BuildBot edited by bfulgham@webkit.org
Some notes to prevent the "[executable] has crashed" dialog from … (diff)
11:31 AM Changeset in webkit [101841] by scheib@chromium.org
  • 6 edits
    1 delete in trunk

Unreviewed, rolling out r101833.
​http://trac.webkit.org/changeset/101833
​https://bugs.webkit.org/show_bug.cgi?id=73678

test_expectations file invalid: run Tools/Scripts/new-run-
webkit-tests --lint-test-files (Requested by scheib on
#webkit).

Patch by Sheriff Bot <​webkit.review.bot@gmail.com> on 2011-12-02

Source/WebCore:

  • platform/graphics/chromium/cc/CCLayerQuad.cpp:

(WebCore::CCLayerQuad::floatQuad):

Source/WebKit/chromium:

  • WebKit.gypi:
  • tests/CCLayerQuadTest.cpp: Removed.

LayoutTests:

  • platform/chromium/test_expectations.txt:
11:24 AM Changeset in webkit [101840] by kov@webkit.org
  • 2 edits in trunk/Tools

Build libsoup without gnome dependencies (like keyring).

  • gtk/jhbuild.modules:
11:21 AM Changeset in webkit [101839] by scheib@chromium.org
  • 2 edits in trunk/LayoutTests

[Chromium] Fixe test_expectations.txt parse error

  • platform/chromium/test_expectations.txt:
11:21 AM Changeset in webkit [101838] by aestes@apple.com
  • 5 edits in trunk/Source/WebKit2

WebKit2: Freeze the state of the layer tree until frame load completion if incremental rendering is suppressed
​https://bugs.webkit.org/show_bug.cgi?id=73641

Reviewed by Darin Adler.

  • WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:

(WebKit::WebFrameLoaderClient::dispatchDidFirstLayout): Do not
un-freeze the layer tree state if incremental rendering is suppressed.

  • WebProcess/WebPage/DrawingArea.h:

(WebKit::DrawingArea::layerTreeStateIsFrozen): Add a getter for
m_layerTreeStateIsFrozen.

  • WebProcess/WebPage/DrawingAreaImpl.h:

(WebKit::DrawingAreaImpl::layerTreeStateIsFrozen): Ditto.

  • WebProcess/WebPage/ca/LayerTreeHostCA.cpp:

(WebKit::LayerTreeHostCA::initialize): WebFrameLoaderClient might have
tried to freeze the state of the layer tree before the layer tree host
was created. When creating the layer tree host, only schedule a layer
flush if the layer tree state isn't frozen.

11:19 AM Changeset in webkit [101837] by kov@webkit.org
  • 2 edits in trunk/Tools

Also build gcrypt and p11-kit, and make them dependencies of
gnutls.

  • gtk/jhbuild.modules:
11:19 AM Changeset in webkit [101836] by scheib@chromium.org
  • 2 edits in trunk/LayoutTests

[Chromium] Expected flakey tests after fix for 73021 is committed
​https://bugs.webkit.org/show_bug.cgi?id=73677

Unreviewed test_expectations update.

Patch by Stephen Chenney <​schenney@chromium.org> on 2011-12-02

  • platform/chromium/test_expectations.txt: Added flakey for svg/stroke/...
11:16 AM Changeset in webkit [101835] by kevino@webkit.org
  • 6 edits in trunk

[wx] Unreviewed build fixes for Windows build.

11:10 AM Changeset in webkit [101834] by eric@webkit.org
  • 12 edits
    1 copy in trunk/Tools

Reviewed by Adam Barth.

webkit-patch post, post-commits, upload should warn when posting to a closed bug, and offer to reopen it
​https://bugs.webkit.org/show_bug.cgi?id=32006

I decided not to make it warn, and just have it re-open the bug.
That's not that different from today's behavior which will
just silently attach the patch.

This patch makes behavior between upload and land-safely consistent
(previously one would assign patches and the other would not)
as well as adds the ability for both to ensure that the bug is open.

To test this I had to add a few more methods to MockBugzilla which
(positively) affected a few other test results.

I also made AbstractStep keep a cached copy of the Bug object
and used the cached copy where appropriate (including for 'bug_title').
This should reduce the number of bug fetches we perform.

  • Scripts/webkitpy/tool/commands/download_unittest.py:
  • Scripts/webkitpy/tool/commands/upload.py:
  • Scripts/webkitpy/tool/commands/upload_unittest.py:
  • Scripts/webkitpy/tool/mocktool.py:
  • Scripts/webkitpy/tool/steps/init.py:
  • Scripts/webkitpy/tool/steps/abstractstep.py:
  • Scripts/webkitpy/tool/steps/closebug.py:
  • Scripts/webkitpy/tool/steps/ensurebugisopenandassigned.py: Added.
  • Scripts/webkitpy/tool/steps/postdiff.py:
  • Scripts/webkitpy/tool/steps/postdiffforcommit.py:
  • Scripts/webkitpy/tool/steps/preparechangelog.py:
  • Scripts/webkitpy/tool/steps/steps_unittest.py:
  • Scripts/webkitpy/tool/steps/updatechangelogswithreviewer.py:
11:08 AM Changeset in webkit [101833] by alokp@chromium.org
  • 6 edits
    1 add in trunk

[chromium] CCLayerQuad does not return FloatQuad in correct orientation
​https://bugs.webkit.org/show_bug.cgi?id=73247

Reviewed by James Robinson.

Source/WebCore:

Returned the FloatQuad coordinates in correct order.

Covered by new unit tests in CCLayerQuadTest.cpp.

  • platform/graphics/chromium/cc/CCLayerQuad.cpp:

(WebCore::CCLayerQuad::floatQuad):

Source/WebKit/chromium:

  • WebKit.gypi:
  • tests/CCLayerQuadTest.cpp: Added.

(WebCore::TEST):

LayoutTests:

  • platform/chromium/test_expectations.txt:
11:05 AM Changeset in webkit [101832] by kov@webkit.org
  • 2 edits in trunk/Tools

Also pass --no-interact to jhbuild when updating dependencies.

  • Scripts/update-webkitgtk-libs:
11:00 AM Changeset in webkit [101831] by dslomov@google.com
  • 9 edits in trunk

​https://bugs.webkit.org/show_bug.cgi?id=73589
[V8][Chromium] Adjust postMessage to the latest "implementation-ready" spec.

  • postMessage should support transfer of MessagePorts
  • the order of arguments to Window::postMessage and Window::webkitPostMessage should be (msg, targetOrigin [, transfer])

Reviewed by David Levin.

Source/WebCore:

  • bindings/v8/custom/V8DOMWindowCustom.cpp:

(WebCore::handlePostMessageCallback):

  • bindings/v8/custom/V8DedicatedWorkerContextCustom.cpp:

(WebCore::handlePostMessageCallback):

  • bindings/v8/custom/V8MessagePortCustom.cpp:

(WebCore::handlePostMessageCallback):

  • bindings/v8/custom/V8WorkerCustom.cpp:

(WebCore::handlePostMessageCallback):

LayoutTests:

  • fast/canvas/webgl/script-tests/arraybuffer-transfer-of-control.js:

(wrapSend):
(wrapFailSend):

  • fast/dom/Window/window-postmessage-args.html:
  • platform/chromium/fast/dom/Window/window-postmessage-args-expected.txt:
10:57 AM Changeset in webkit [101830] by enrica@apple.com
  • 2 edits in trunk/Source/WebCore

​https://bugs.webkit.org/show_bug.cgi?id=73497
This is a followup to the patch submitted for the bug above.
Tha patch was landed in r101575 and I missed to make one of the changes
suggested by the reviewer that had pointed out that the code was still
making use of the deprecatedNode method.

Unreviewed.

  • editing/DeleteSelectionCommand.cpp:

(WebCore::DeleteSelectionCommand::removeRedundantBlocks): Changed deprecatedNode to containerNode.

10:55 AM Changeset in webkit [101829] by tsepez@chromium.org
  • 3 edits
    2 adds in trunk

Content-security-policy script-src not enforced on workers.
​https://bugs.webkit.org/show_bug.cgi?id=73240

Reviewed by Adam Barth.

Source/WebCore:

Add a CSP check in AbstractWorker.cpp as part of resolving URL.

Test: http/tests/security/contentSecurityPolicy/worker-script-src.html

  • workers/AbstractWorker.cpp:

(WebCore::AbstractWorker::resolveURL):

LayoutTests:

Added test to verify bug.

  • http/tests/security/contentSecurityPolicy/worker-script-src-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/worker-script-src.html: Added.
10:52 AM Changeset in webkit [101828] by dcheng@chromium.org
  • 16 edits
    4 adds in trunk

[chromium] Add plumbing for supporting custom MIME types in DataTransfer.
​https://bugs.webkit.org/show_bug.cgi?id=73594

Reviewed by David Levin.

Source/WebCore:

Tests: editing/pasteboard/clipboard-customData.html

fast/events/drag-customData.html

  • platform/chromium/ChromiumDataObject.cpp:

(WebCore::ChromiumDataObject::types):
(WebCore::ChromiumDataObject::getData):
(WebCore::ChromiumDataObject::setData):

  • platform/chromium/ChromiumDataObject.h:

(WebCore::ChromiumDataObject::customData):

  • platform/chromium/PlatformSupport.h:

Source/WebKit/chromium:

  • public/platform/WebClipboard.h:

(WebKit::WebClipboard::readCustomData):

  • public/platform/WebDragData.h:
  • src/PlatformSupport.cpp:

(WebCore::PlatformSupport::clipboardReadCustomData):

  • src/WebDragData.cpp:

(WebKit::WebDragData::customData):
(WebKit::WebDragData::setCustomData):

LayoutTests:

  • editing/pasteboard/clipboard-customData-expected.txt: Added.
  • editing/pasteboard/clipboard-customData.html: Added.
  • fast/events/drag-customData-expected.txt: Added.
  • fast/events/drag-customData.html: Added.
  • platform/chromium/test_expectations.txt:
  • platform/efl/Skipped:
  • platform/gtk/Skipped:
  • platform/mac/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:
10:47 AM Changeset in webkit [101827] by aboxhall@chromium.org
  • 2 edits in trunk/Tools

Make _build_path check more thoroughly that build paths exist.
​https://bugs.webkit.org/show_bug.cgi?id=73601

Reviewed by Ojan Vafai.

  • Scripts/webkitpy/layout_tests/port/chromium_mac.py:

(ChromiumMacPort._build_path):

10:44 AM Changeset in webkit [101826] by kov@webkit.org
  • 2 edits in trunk/Tools

Add libffi to the jhbuild modules list as a glib dependency.

Rubber-stamped by Martin Robinson.

  • gtk/jhbuild.modules:
10:34 AM Changeset in webkit [101825] by Darin Adler
  • 2 edits in trunk/Source/WebCore

[Mac] Form stream data structures still not threadsafe
​https://bugs.webkit.org/show_bug.cgi?id=73674

Reviewed by Anders Carlsson.

  • platform/network/mac/FormDataStreamMac.mm:

(WebCore::streamFieldsMapMutex): Added.
(WebCore::associateStreamWithResourceHandle): Use streamFieldsMapMutex.
(WebCore::formCreate): Ditto.
(WebCore::formFinalize): Ditto.
(WebCore::httpBodyFromStream): Ditto.

10:31 AM Changeset in webkit [101824] by commit-queue@webkit.org
  • 30 edits in trunk

IndexedDB: Rename "multientry" to "multiEntry" per spec change
​https://bugs.webkit.org/show_bug.cgi?id=73578

Patch by Joshua Bell <​jsbell@chromium.org> on 2011-12-02
Reviewed by Darin Fisher.

Source/WebCore:

  • storage/IDBIndex.h:

(WebCore::IDBIndex::multiEntry):

  • storage/IDBIndex.idl:
  • storage/IDBIndexBackendImpl.cpp:

(WebCore::IDBIndexBackendImpl::IDBIndexBackendImpl):

  • storage/IDBIndexBackendImpl.h:

(WebCore::IDBIndexBackendImpl::create):
(WebCore::IDBIndexBackendImpl::multiEntry):

  • storage/IDBIndexBackendInterface.h:
  • storage/IDBLevelDBBackingStore.cpp:

(WebCore::IDBLevelDBBackingStore::getIndexes):
(WebCore::IDBLevelDBBackingStore::createIndex):

  • storage/IDBLevelDBCoding.cpp:
  • storage/IDBObjectStore.cpp:

(WebCore::IDBObjectStore::createIndex):

  • storage/IDBObjectStoreBackendImpl.cpp:

(WebCore::IDBObjectStoreBackendImpl::putInternal):
(WebCore::IDBObjectStoreBackendImpl::createIndex):
(WebCore::IDBObjectStoreBackendImpl::createIndexInternal):
(WebCore::IDBObjectStoreBackendImpl::loadIndexes):

  • storage/IDBObjectStoreBackendImpl.h:
  • storage/IDBObjectStoreBackendInterface.h:

Source/WebKit/chromium:

  • public/WebIDBIndex.h:

(WebKit::WebIDBIndex::multiEntry):

  • public/WebIDBObjectStore.h:

(WebKit::WebIDBObjectStore::createIndex):

  • src/IDBIndexBackendProxy.cpp:

(WebKit::IDBIndexBackendProxy::multiEntry):

  • src/IDBIndexBackendProxy.h:
  • src/IDBObjectStoreBackendProxy.cpp:

(WebKit::IDBObjectStoreBackendProxy::createIndex):

  • src/IDBObjectStoreBackendProxy.h:
  • src/WebIDBIndexImpl.cpp:

(WebKit::WebIDBIndexImpl::multiEntry):

  • src/WebIDBIndexImpl.h:
  • src/WebIDBObjectStoreImpl.cpp:

(WebKit::WebIDBObjectStoreImpl::createIndex):

  • src/WebIDBObjectStoreImpl.h:

LayoutTests:

  • storage/indexeddb/index-basics-expected.txt:
  • storage/indexeddb/index-basics.html:
  • storage/indexeddb/index-multientry-expected.txt:
  • storage/indexeddb/index-multientry.html:
10:30 AM Changeset in webkit [101823] by kov@webkit.org
  • 4 edits in trunk/Tools

Build fix. Use internal pcre on glib, make gnutls a dependency of
glib-networking, unset AR_FLAGS for jhbuild builds, and run jhbuild
in non-interactive mode.

  • Scripts/update-webkitgtk-libs:
  • gtk/jhbuild.modules:
  • gtk/run-with-jhbuild:
10:26 AM Changeset in webkit [101822] by Adam Roben
  • 2 edits in trunk/Tools

Fix for Python 2.6

  • BuildSlaveSupport/build.webkit.org-config/master.cfg:

(RunUnitTests.countFailures): Removed the flags parameter to re.split, which was only added
in Python 2.7, and used whitespace instead of start-of-string/end-of-string markers when
searching for "Tests that timed out:" etc. This makes our check less strict, but it
shouldn't matter given the kinds of content we parse.

10:19 AM Changeset in webkit [101821] by Adam Roben
  • 2 edits in trunk/Tools

Fix names of failed/timed out tests in run-api-tests output

Fixes <​http://webkit.org/b/73663> run-api-tests prints suite names twice for failed or timed
out tests

We were ending up with the suite name doubled (e.g., "WebKit2.WebKit2.MouseMoveAfterCrash").

Reviewed by Simon Fraser.

  • Scripts/run-api-tests:

(runTest): Don't prepend the suite name to $test. It's already been prepended at the start
of this function.

10:17 AM Changeset in webkit [101820] by Martin Robinson
  • 3 edits in trunk/Tools

[GTK] generate-gtkdoc should respect build type
​https://bugs.webkit.org/show_bug.cgi?id=73296

Reviewed by Philippe Normand.

  • Scripts/webkitdirs.pm:

(buildAutotoolsProject): Pass the --debug to the generate-gtkdoc script.

  • gtk/common.py: If --build is passed give preference to the Debug build directory.
10:11 AM Changeset in webkit [101819] by tony@chromium.org
  • 3 edits in trunk/Tools

[nrwt] fix wdiff output
​https://bugs.webkit.org/show_bug.cgi?id=73604

Reviewed by Adam Barth.

  • Scripts/webkitpy/layout_tests/port/base.py:

(Port.wdiff_text):

  • Scripts/webkitpy/layout_tests/port/base_unittest.py:

(PortTest.test_wdiff_text):

10:01 AM Changeset in webkit [101818] by Adam Roben
  • 3 edits in trunk/Tools

Teach build.webkit.org to display how many unit tests failed or timed out

Fixes <​http://webkit.org/b/73659> It's hard to tell how many unit tests are failing on
build.webkit.org

We now display something like "5 unit tests failed or timed out". Eventually we might want
to split out how many failure vs. timeouts there were.

Reviewed by Darin Adler.

  • BuildSlaveSupport/build.webkit.org-config/master.cfg:

(TestWithFailureCount): Moved this class up to the top of the file so it is before any other
classes that may want to subclass it.
(RunUnitTests): Changed to inherit from TestWithFailureCount.
(RunUnitTests.countFailures): Added. Counts the number of tests following the "Tests that
timed out:" and "Tests that failed:" lines, if present.

  • BuildSlaveSupport/build.webkit.org-config/mastercfg_unittest.py:

(StubStdio):
(StubRemoteCommand):
Added these two stub classes to mimic buildbot's RemoteCommand and log classes.

(RunUnitTestsTest.assertFailures): Helper method to check that we interpreted the results of
the test run correctly.

(RunUnitTestsTest.test_no_failures_or_timeouts):
(RunUnitTestsTest.test_one_failure):
(RunUnitTestsTest.test_multiple_failures):
(RunUnitTestsTest.test_one_timeout):
(RunUnitTestsTest.test_multiple_timeouts):
(RunUnitTestsTest.test_multiple_failures_and_timeouts):
Test various cases.

10:00 AM Changeset in webkit [101817] by Adam Roben
  • 2 edits in trunk/Tools

Teach prepare-ChangeLog to treat master.cfg as a Python file

Fixes <​http://webkit.org/b/73658> prepare-ChangeLog doesn't show modified classes/functions
in buildbot's master.cfg

Reviewed by Darin Adler.

  • Scripts/prepare-ChangeLog:

(get_function_line_ranges): Use get_function_line_ranges_for_python if the filename is
master.cfg.

9:49 AM Changeset in webkit [101816] by Darin Adler
  • 5 edits
    2 adds
    14 deletes in trunk

Unreviewed, rolling out r101805.
​http://trac.webkit.org/changeset/101805
​https://bugs.webkit.org/show_bug.cgi?id=73670

Many canvas tests started failing due to the isEmpty change
(Requested by darin on #webkit).

Patch by Sheriff Bot <​webkit.review.bot@gmail.com> on 2011-12-02

Source/WebCore:

  • platform/graphics/cg/PathCG.cpp:

(WebCore::Path::boundingRect):
(WebCore::Path::fastBoundingRect):
(WebCore::Path::isEmpty):

LayoutTests:

  • platform/chromium-mac/svg/custom/path-moveto-only-rendering-expected.png: Removed.
  • platform/chromium-mac/svg/custom/path-moveto-only-rendering-expected.txt: Removed.
  • platform/chromium-mac/svg/custom/subpaths-moveto-only-rendering-expected.png: Removed.
  • platform/chromium-mac/svg/custom/subpaths-moveto-only-rendering-expected.txt: Removed.
  • platform/chromium-mac/svg/custom/zero-path-square-cap-rendering2-expected.txt: Copied from LayoutTests/svg/custom/zero-path-square-cap-rendering2-expected.txt.
  • platform/chromium-win/svg/custom/zero-path-square-cap-rendering2-expected.txt: Copied from LayoutTests/svg/custom/zero-path-square-cap-rendering2-expected.txt.
  • platform/chromium/test_expectations.txt:
  • platform/mac/svg/custom/path-moveto-only-rendering-expected.png: Removed.
  • platform/mac/svg/custom/path-moveto-only-rendering-expected.txt: Removed.
  • platform/mac/svg/custom/subpaths-moveto-only-rendering-expected.png: Removed.
  • platform/mac/svg/custom/subpaths-moveto-only-rendering-expected.txt: Removed.
  • platform/win/svg/custom/path-moveto-only-rendering-expected.png: Removed.
  • platform/win/svg/custom/path-moveto-only-rendering-expected.txt: Removed.
  • platform/win/svg/custom/subpaths-moveto-only-rendering-expected.png: Removed.
  • platform/win/svg/custom/subpaths-moveto-only-rendering-expected.txt: Removed.
  • svg/custom/path-moveto-only-rendering.svg: Removed.
  • svg/custom/subpaths-moveto-only-rendering.svg: Removed.
  • svg/custom/zero-path-square-cap-rendering2-expected.txt:
9:44 AM Changeset in webkit [101815] by commit-queue@webkit.org
  • 5 edits in trunk/Source/WebKit2

[WebKit2][gtk] Add 'enable-dns-prefetching' property to WebKitSettings
​https://bugs.webkit.org/show_bug.cgi?id=73414

Patch by Nayan Kumar K <​nayankk@motorola.com> on 2011-12-02
Reviewed by Martin Robinson.

  • UIProcess/API/gtk/WebKitSettings.cpp:

(webKitSettingsSetProperty): Add new set function.
(webKitSettingsGetProperty): Add new get function.
(webkit_settings_class_init): Register new property.
(webkit_settings_get_enable_dns_prefetching): API to get 'enable-dns-prefetching' property.
(webkit_settings_set_enable_dns_prefetching): API to set 'enable-dns-prefetching' property.

  • UIProcess/API/gtk/WebKitSettings.h: Expose new public API's.
  • UIProcess/API/gtk/docs/webkit2gtk-sections.txt: Add new APIs to gtk-doc sections file.
  • UIProcess/API/gtk/tests/TestWebKitSettings.cpp:

(testWebKitSettings): Add new tests.

9:44 AM Changeset in webkit [101814] by Martin Robinson
  • 3 edits
    1 add in trunk/Tools

[GTK] Create a wrapper script that knows how to install jhbuild and run commands with it
​https://bugs.webkit.org/show_bug.cgi?id=73669

Reviewed by Gustavo Noronha Silva.

Add a wrapper script that knows how to install jhbuild and also to wrap commands
in the jhbuild shell.

  • Scripts/webkitdirs.pm:

(runAutogenForAutotoolsProject): Use the new wrapper script.
(mustReRunAutogen): Ditto.
(buildAutotoolsProject): Ditto.

  • Scripts/webkitpy/layout_tests/port/gtk.py:

(GtkDriver.cmd_line): Ditto.

  • gtk/run-with-jhbuild: Added.
9:42 AM Changeset in webkit [101813] by Darin Adler
  • 2 edits in trunk/Source/WebCore

[Mac] Reference count threading violation in FormDataStreamMac.mm
​https://bugs.webkit.org/show_bug.cgi?id=73627

Reviewed by Sam Weinig.

Shows up as a crash during existing layout test runs so no new tests are required.

  • platform/network/mac/FormDataStreamMac.mm:

(WebCore::streamFieldsMap): Replaced getStreamFormDataMap with this.
Use an NSMapTable instead of a HashMap because we need to remove items from this
on a non-main thread.
(WebCore::associateStreamWithResourceHandle): Use NSMapGet instead of
HashMap::contains here.
(WebCore::formCreate): FormStreamFields now stores a RefPtr to the form data.
Added the code to fill that in. Did it in a more modern way to avoid the leakRef
and adoptRef that were used before. Replaced the code that set up the stream
form data map entry with code that sets an entry in the streamFieldsMap.
(WebCore::formFinishFinalizationOnMainThread): Added. Contains the work of
finalization that must be done on the main thread, specifically, destroying the
fields structure that contains objects with RefPtr in them. We can't touch these
reference counts on non-main threads.
(WebCore::formFinalize): Changed this to use NSMapRemove on the streamFieldsMap.
Added a callOnMainThread to finish the finalization.
(WebCore::setHTTPBody): Removed the leakRef, no longer needed, that used to be
balanced by an adoptRef in formCreate.
(WebCore::httpBodyFromStream): Changed to use NSMapGet.

9:42 AM Changeset in webkit [101812] by Antti Koivisto
  • 5 edits
    2 adds in trunk

​https://bugs.webkit.org/show_bug.cgi?id=73520
REGRESSION(r101524): Article titles invisible until hover on blaze.com

Source/WebCore:

Reviewed by Darin Adler.

We need to invalidate the matched declaration cache when new web fonts are loaded.
Fonts in the cached RenderStyles may not be valid anymore.

Also renamed m_matchStyleDeclarationCache -> m_matchedStyleDeclarationCache.

Test reduction by the Reduction Fairy (aka kling).

Test: fast/css/font-face-cache-bug.html

  • css/CSSFontSelector.cpp:

(WebCore::CSSFontSelector::dispatchInvalidationCallbacks):

  • css/CSSStyleSelector.cpp:

(WebCore::CSSStyleSelector::findFromMatchedDeclarationCache):
(WebCore::CSSStyleSelector::addToMatchedDeclarationCache):
(WebCore::CSSStyleSelector::invalidateMatchedDeclarationCache):

  • css/CSSStyleSelector.h:

LayoutTests:

Reviewed by Darin Adler.

Test reduction by the Reduction Fairy (aka kling).

  • fast/css/font-face-cache-bug-expected.txt: Added.
  • fast/css/font-face-cache-bug.html: Added.
9:41 AM Changeset in webkit [101811] by kov@webkit.org
  • 2 edits in trunk/Tools

[GTK] Add glib, glib-networking, gnutls and libsoup to jhbuild
​https://bugs.webkit.org/show_bug.cgi?id=73664

Reviewed by Martin Robinson.

This patch adds our HTTP library and its main dependencies; this
should not affect layout or rendering.

  • gtk/jhbuild.modules:
9:35 AM Changeset in webkit [101810] by jer.noble@apple.com
  • 11 edits in trunk/Source/WebCore

MediaControls should use MediaController if present.
​https://bugs.webkit.org/show_bug.cgi?id=71410

Reviewed by Eric Carlson.

No new tests; covered by existing tests.

Add support for individual media controls to control the MediaController of their associated
HTMLMediaElement, if present.

The video spec requires that UA provided media controls be implemented in terms of their
HTMLMediaElement's MediaController, if present. So for each of the media controls, modify
their constructor to take a Document* instead of an HTMLMediaElement, and add an setter
taking a MediaControllerInterface.

Now that MediaControls have an abstract interface instead of an HTMLMediaElement, use toParentMediaElement
to find the controllingVideoElement.

  • accessibility/AccessibilityMediaControls.cpp:

(WebCore::AccessibilityMediaControlsContainer::controllingVideoElement):

Pass the MediaController or the HTMLMediaElement when setting up the elements controls.

  • html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::createMediaControls):
(WebCore::HTMLMediaElement::setMediaController):

The following functions have only constructor changes:

  • html/shadow/MediaControlElements.cpp:

(WebCore::MediaControlElement::MediaControlElement):
(WebCore::MediaControlPanelElement::MediaControlPanelElement):
(WebCore::MediaControlPanelElement::create):
(WebCore::MediaControlTimelineContainerElement::MediaControlTimelineContainerElement):
(WebCore::MediaControlTimelineContainerElement::create):
(WebCore::MediaControlVolumeSliderContainerElement::MediaControlVolumeSliderContainerElement):
(WebCore::MediaControlVolumeSliderContainerElement::create):
(WebCore::MediaControlStatusDisplayElement::MediaControlStatusDisplayElement):
(WebCore::MediaControlStatusDisplayElement::create):
(WebCore::MediaControlInputElement::MediaControlInputElement):
(WebCore::MediaControlMuteButtonElement::MediaControlMuteButtonElement):
(WebCore::MediaControlMuteButtonElement::defaultEventHandler):
(WebCore::MediaControlPanelMuteButtonElement::MediaControlPanelMuteButtonElement):
(WebCore::MediaControlPanelMuteButtonElement::create):
(WebCore::MediaControlVolumeSliderMuteButtonElement::MediaControlVolumeSliderMuteButtonElement):
(WebCore::MediaControlVolumeSliderMuteButtonElement::create):
(WebCore::MediaControlPlayButtonElement::MediaControlPlayButtonElement):
(WebCore::MediaControlPlayButtonElement::create):
(WebCore::MediaControlSeekButtonElement::MediaControlSeekButtonElement):
(WebCore::MediaControlSeekForwardButtonElement::MediaControlSeekForwardButtonElement):
(WebCore::MediaControlSeekForwardButtonElement::create):
(WebCore::MediaControlSeekBackButtonElement::MediaControlSeekBackButtonElement):
(WebCore::MediaControlSeekBackButtonElement::create):
(WebCore::MediaControlRewindButtonElement::MediaControlRewindButtonElement):
(WebCore::MediaControlRewindButtonElement::create):
(WebCore::MediaControlReturnToRealtimeButtonElement::MediaControlReturnToRealtimeButtonElement):
(WebCore::MediaControlReturnToRealtimeButtonElement::create):
(WebCore::MediaControlToggleClosedCaptionsButtonElement::MediaControlToggleClosedCaptionsButtonElement):
(WebCore::MediaControlToggleClosedCaptionsButtonElement::create):
(WebCore::MediaControlTimelineElement::MediaControlTimelineElement):
(WebCore::MediaControlTimelineElement::create):
(WebCore::MediaControlVolumeSliderElement::MediaControlVolumeSliderElement):
(WebCore::MediaControlVolumeSliderElement::create):
(WebCore::MediaControlVolumeSliderElement::defaultEventHandler):
(WebCore::MediaControlFullscreenVolumeSliderElement::MediaControlFullscreenVolumeSliderElement):
(WebCore::MediaControlFullscreenVolumeSliderElement::create):
(WebCore::MediaControlFullscreenButtonElement::MediaControlFullscreenButtonElement):
(WebCore::MediaControlFullscreenButtonElement::create):
(WebCore::MediaControlFullscreenVolumeMinButtonElement::MediaControlFullscreenVolumeMinButtonElement):
(WebCore::MediaControlFullscreenVolumeMinButtonElement::create):
(WebCore::MediaControlFullscreenVolumeMaxButtonElement::MediaControlFullscreenVolumeMaxButtonElement):
(WebCore::MediaControlFullscreenVolumeMaxButtonElement::create):
(WebCore::MediaControlTimeDisplayElement::MediaControlTimeDisplayElement):
(WebCore::MediaControlTimeRemainingDisplayElement::create):
(WebCore::MediaControlTimeRemainingDisplayElement::MediaControlTimeRemainingDisplayElement):
(WebCore::MediaControlCurrentTimeDisplayElement::create):
(WebCore::MediaControlCurrentTimeDisplayElement::MediaControlCurrentTimeDisplayElement):

  • html/shadow/MediaControlRootElement.cpp:

(WebCore::MediaControlRootElement::MediaControlRootElement):
(WebCore::MediaControls::create):
(WebCore::MediaControlRootElement::create):

  • html/shadow/MediaControlRootElement.h:
  • html/shadow/MediaControls.cpp:

(WebCore::MediaControls::MediaControls):

  • html/shadow/MediaControls.h:

The following functions now call MediaControllerInterface instead of HTMLMediaElement directly:

  • html/shadow/MediaControlElements.h:

(WebCore::MediaControlStatusDisplayElement::update):
(WebCore::MediaControlMuteButtonElement::updateDisplayType):
(WebCore::MediaControlPlayButtonElement::defaultEventHandler):
(WebCore::MediaControlPlayButtonElement::updateDisplayType):
(WebCore::MediaControlSeekButtonElement::startTimer):
(WebCore::MediaControlSeekButtonElement::stopTimer):
(WebCore::MediaControlSeekButtonElement::nextRate):
(WebCore::MediaControlSeekButtonElement::seekTimerFired):
(WebCore::MediaControlRewindButtonElement::defaultEventHandler):
(WebCore::MediaControlReturnToRealtimeButtonElement::defaultEventHandler):
(WebCore::MediaControlToggleClosedCaptionsButtonElement::defaultEventHandler):
(WebCore::MediaControlToggleClosedCaptionsButtonElement::updateDisplayType):
(WebCore::MediaControlTimelineElement::defaultEventHandler):
(WebCore::MediaControlFullscreenButtonElement::defaultEventHandler):
(WebCore::MediaControlFullscreenVolumeMinButtonElement::defaultEventHandler):
(WebCore::MediaControlFullscreenVolumeMaxButtonElement::defaultEventHandler):

  • html/shadow/MediaControlRootElement.cpp:

(WebCore::MediaControlRootElement::reset):
(WebCore::MediaControlRootElement::playbackStarted):
(WebCore::MediaControlRootElement::playbackProgressed):
(WebCore::MediaControlRootElement::playbackStopped):
(WebCore::MediaControlRootElement::updateTimeDisplay):
(WebCore::MediaControlRootElement::loadedMetadata):
(WebCore::MediaControlRootElement::changedVolume):
(WebCore::MediaControlRootElement::enteredFullscreen):
(WebCore::MediaControlRootElement::showVolumeSlider):
(WebCore::MediaControlRootElement::defaultEventHandler):
(WebCore::MediaControlRootElement::startHideFullscreenControlsTimer):
(WebCore::MediaControlRootElement::hideFullscreenControlsTimerFired):

  • html/shadow/MediaControlRootElementChromium.cpp:

(WebCore::MediaControlRootElementChromium::MediaControlRootElementChromium):
(WebCore::MediaControls::create):
(WebCore::MediaControlRootElementChromium::create):
(WebCore::MediaControlRootElementChromium::reset):
(WebCore::MediaControlRootElementChromium::playbackStarted):
(WebCore::MediaControlRootElementChromium::playbackProgressed):
(WebCore::MediaControlRootElementChromium::playbackStopped):
(WebCore::MediaControlRootElementChromium::updateTimeDisplay):
(WebCore::MediaControlRootElementChromium::defaultEventHandler):
(WebCore::MediaControlRootElementChromium::changedVolume):
(WebCore::MediaControlRootElementChromium::showVolumeSlider):

  • html/shadow/MediaControlRootElementChromium.h:

The following functions set the current MediaControllerInterface.

  • html/shadow/MediaControlRootElement.cpp:

(WebCore::MediaControlRootElement::setMediaController):

  • html/shadow/MediaControlElements.h:

(WebCore::MediaControlElement::setMediaController):
(WebCore::MediaControlElement::mediaController):
(WebCore::MediaControlInputElement::setMediaController):
(WebCore::MediaControlInputElement::mediaController):

  • html/shadow/MediaControlRootElementChromium.cpp:

(WebCore::MediaControlRootElementChromium::setMediaController):

9:07 AM Changeset in webkit [101809] by commit-queue@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

Unreviewed, rolling out r101801.
​http://trac.webkit.org/changeset/101801
​https://bugs.webkit.org/show_bug.cgi?id=73667

Build is still broken (Requested by Ossy on #webkit).

Patch by Sheriff Bot <​webkit.review.bot@gmail.com> on 2011-12-02

  • assembler/SH4Assembler.h:
9:04 AM Changeset in webkit [101808] by sfalken@apple.com
  • 2 edits in trunk/Source/WebKit/win

Changes to localStorageDatabasePath don't have any effect on Windows
​https://bugs.webkit.org/show_bug.cgi?id=73606
<rdar://problem/10491296>

Reviewed by Darin Adler.

  • WebView.cpp:

(WebView::notifyPreferencesChanged): Propagate local storage path change to m_page's settings.

9:01 AM Changeset in webkit [101807] by Philippe Normand
  • 2 edits in trunk/LayoutTests

Unreviewed, skip failing websocket test on GTK.

  • platform/gtk/Skipped: Skip http/tests/websocket/tests/hybi/bufferedAmount-after-close-in-busy.html
8:50 AM Changeset in webkit [101806] by Darin Adler
  • 6 edits in trunk/Source/JavaScriptCore

Prepare to deploy pass and peek types in the HashMap class
​https://bugs.webkit.org/show_bug.cgi?id=73477

Reviewed by Adam Roben.

This patch adds private typedefs inside the HashMap class,
and uses them as appropriate. A future patch will actually
tie those typedefs to hash traits, which will allow us to
make HashMap work with OwnPtr mapped values and to optimize
how HashMap works with RefPtr mapped values.

Also changed the hash translator and adapter struct templates
to use template functions to simplify them and make them more
flexible.

Also removed some unused template arguments.

This goes out of its way to not change behavior. Future patches
will change the peek type to be a reference type, which will
reduce reference count churn a bit for hash tables with RefPtr
mapped values, and then do further optimizations for RefPtr
and OwnPtr by getting types from the hash traits.

  • wtf/HashMap.h: Added MappedPassInType, MappedPassOutType,

and MappedPeekType typedefs, and used them for the arguments
and return types of the get, set, add, take, and inlineAdd
functions.
(WTF::HashMapTranslator): Changed this struct template to take
fewer arguments, and changed its member functions to be
function templates instead. This allows the compiler to
determine types more flexibly and also simplifies use of it.
(WTF::HashMapTranslatorAdapter): Ditto.
(WTF::HashMap::find): Updated to use new HashMapTranslatorAdapter.
Also reduced the arguments passed to the HashTable function template.
(WTF::HashMap::contains): Ditto.
(WTF::HashMap::inlineAdd): Ditto. Also take MappedPassInType.
(WTF::HashMap::set): Ditto.
(WTF::HashMap::add): Ditto.
(WTF::HashMap::inlineGet): Ditto, but return MappedPeekType.
(WTF::HashMap::get): Ditto.
(WTF::HashMap::take): Ditto, but return MappedPassOutType and use
that type in the implementation.
(WTF::deleteAllValues): Removed unneeded template arguments from
call to deleteAllPairSeconds.
(WTF::deleteAllKeys): Removed unneeded template arguments from
call to deleteAllPairFirsts.

  • wtf/HashSet.h:

(WTF::IdentityExtractor): Changed this to be a struct rather than
a struct template, and replaced the extract function with a function
template. This allows the compiler to deduce the type.
(WTF::HashSetTranslatorAdapter): Changed this struct template to take
fewer arguments, and changed its member functions to be
function templates instead. This allows the compiler to
determine types more flexibly and also simplifies use of it.
(WTF::HashSet::find): Updated to use new HashSetTranslatorAdapter.
Also reduced the arguments passed to the HashTable function template.
(WTF::HashSet::contains): Ditto.
(WTF::HashSet::add): Ditto.

  • wtf/HashTable.h:

(WTF::IdentityHashTranslator): Changed this struct template to take
fewer arguments, and changed its member functions to be
function templates instead. This allows the compiler to
determine types more flexibly and also simplifies use of it.
(WTF::HashTable::add): Reduced arguments passed to the function template.
(WTF::HashTable::find): Ditto, also reversed the template arguments so the
translator comes first so the compiler can deduce the other type.
(WTF::HashTable::contains): Ditto.
(WTF::HashTable::lookup): Ditto.
(WTF::HashTable::lookupForWriting): Ditto.
(WTF::HashTable::checkKey): Ditto.
(WTF::HashTable::fullLookupForWriting): Ditto.
(WTF::HashTable::add): Ditto.
(WTF::HashTable::addPassingHashCode): Ditto.
(WTF::HashTable::find): Ditto.
(WTF::HashTable::contains): Ditto.

  • wtf/ListHashSet.h:

(WTF::ListHashSetNodeHashFunctions): Changed this struct template to take
fewer arguments, and changed its member functions to be function templates
instead. This allows the compiler to determine types more flexibly and
also simplifies use of it.
(WTF::ListHashSet::find): Reduced the arguments passed to the HashTable
functon template.
(WTF::ListHashSetTranslatorAdapter): Changed this struct template in the
same way we changed ListHashSetNodeHashFunctions above.
(WTF::ListHashSetTranslatorAdapter::equal):
(WTF::::contains):
(WTF::::add):
(WTF::::insertBefore):

  • wtf/RefPtrHashMap.h: Updated comments. Removed the

RefPtrHashMapRawKeyTranslator struct template; we can use the
HashMapTranslator struct template from HashMap.h instead now that
it is more flexible. Added MappedPassInType, MappedPassOutType,
and MappedPeekType typedefs, and used them for the arguments
and return types of the get, inlineGet, set, add, take, and inlineAdd
functions. Changed the name of the RawKeyTranslator type to
Translator since it's now a class that can handle both raw keys
and conventional keys.
(WTF::HashMap::find): Changed to use Translator instead of RawKeyTranslator.
Reduced the arguments passed to the HashTable function template.
(WTF::HashMap::contains): Ditto.
(WTF::HashMap::inlineAdd): Ditto. Also take MappedPassInType.
(WTF::HashMap::set): Ditto.
(WTF::HashMap::add): Ditto.
(WTF::HashMap::inlineGet): Ditto, but return MappedPeekType.
(WTF::HashMap::get): Ditto.
(WTF::HashMap::take): Ditto, but return MappedPassOutType and use
that type in the implementation.
(WTF::deleteAllValues): Removed unneeded template arguments from
call to deleteAllPairSeconds.
(WTF::deleteAllKeys): Removed unneeded template arguments from
call to deleteAllPairFirsts.

8:41 AM WebKitGtkLayoutTests edited by kov@webkit.org
(diff)
8:37 AM Changeset in webkit [101805] by commit-queue@webkit.org
  • 5 edits
    14 adds
    2 deletes in trunk

REGRESSION (r91125): Polyline tool in google docs is broken
​https://bugs.webkit.org/show_bug.cgi?id=65796

Source/WebCore:

Patch by Stephen Chenney <​schenney@chromium.org> on 2011-12-02
Reviewed by Darin Adler.

Work around a bug in CoreGraphics, that caused incorrect bounds for paths
consisting only of move-to elements. This causes problems in SVG, when the enormous
bounds prevented the drawing of things behind.

Tests: svg/custom/path-moveto-only-rendering.svg

svg/custom/subpaths-moveto-only-rendering.svg

  • platform/graphics/cg/PathCG.cpp:

(WebCore::PathIsEmptyOrSingleMoveTester::PathIsEmptyOrSingleMoveTester): Class to
test for isEmpty accoridng ot the same rules as other platforms.
(WebCore::PathIsEmptyOrSingleMoveTester::isEmpty): Query the result
(WebCore::PathIsEmptyOrSingleMoveTester::testPathElement): Path iterator method
(WebCore::PathHasOnlyMoveToTester::PathHasOnlyMoveToTester): Class to test whether a
path contains only move-to elements, and hence should have null bounds.
(WebCore::PathHasOnlyMoveToTester::hasOnlyMoveTo): Query the result
(WebCore::PathHasOnlyMoveToTester::testPathElement): Path iterator method.
(WebCore::Path::boundingRect): Modified to check for move-to only paths
(WebCore::Path::fastBoundingRect): Modified to check for move-to only paths
(WebCore::Path::isEmpty): Now uses the method that matches other platforms.

LayoutTests:

Work around a bug in CoreGraphics, that caused incorrect bounds for paths
consisting only of move-to elements. This causes problems in SVG, when the enormous
bounds prevented the drawing of things behind.

Will revert expectation file when expectations are stable.

Patch by Stephen Chenney <​schenney@chromium.org> on 2011-12-02
Reviewed by Darin Adler.

  • platform/chromium-mac/svg/custom/zero-path-square-cap-rendering2-expected.txt: Removed.
  • platform/chromium-win/svg/custom/zero-path-square-cap-rendering2-expected.txt: Removed.
  • platform/mac/svg/custom/path-moveto-only-rendering-expected.png: Added.
  • platform/mac/svg/custom/path-moveto-only-rendering-expected.txt: Added.
  • platform/mac/svg/custom/subpaths-moveto-only-rendering-expected.png: Added.
  • platform/mac/svg/custom/subpaths-moveto-only-rendering-expected.txt: Added.
  • platform/win/svg/custom/path-moveto-only-rendering-expected.png: Added.
  • platform/win/svg/custom/path-moveto-only-rendering-expected.txt: Added.
  • platform/win/svg/custom/subpaths-moveto-only-rendering-expected.png: Added.
  • platform/win/svg/custom/subpaths-moveto-only-rendering-expected.txt: Added.
  • svg/custom/path-moveto-only-rendering.svg: Added.
  • svg/custom/subpaths-moveto-only-rendering.svg: Added.
  • svg/custom/zero-path-square-cap-rendering2-expected.txt: Modified text output
  • platform/chromium/test_expectations.txt: Added flakiness for new tests from this patch
8:36 AM Changeset in webkit [101804] by abecsi@webkit.org
  • 3 edits in trunk/Source/WebKit2

[Qt][WK2] Set event timestamps on custom input events

Reviewed by Simon Hausmann.

Since r101791 WebEventFactoryQt uses input event timestamps for WebEvents,
thus set the timestamp on the custom events as well.

  • UIProcess/qt/QtWebPageEventHandler.cpp:

(QtWebPageEventHandler::handleHoverLeaveEvent):
(QtWebPageEventHandler::handleHoverMoveEvent):

  • UIProcess/qt/WebPopupMenuProxyQtDesktop.cpp:

(WebKit::WebPopupMenuProxyQtDesktop::showPopupMenu):

8:33 AM Changeset in webkit [101803] by Adam Roben
  • 3 edits
    2 adds in trunk/Tools

Teach check-webkit-style how to check the syntax of JSON files

Fixes <​http://webkit.org/b/73590> check-webkit-style doesn't flag JSON syntax errors

I previously landed this patch as r101711, but the new tests failed on Snow Leopard. This
patch is identical to that one except for two change: assertIn(a, b) (which is new to Python
2.7) has been replaced with assertTrue(a in b), and the test_missing_closing_brace test has
been removed, since its output differs by platform.

Reviewed by Darin Adler.

  • Scripts/webkitpy/style/checker.py:

(_all_categories): Added JSONChecker's categories to the set of all categories.
(FileType): Added a JSON type. Incremented other types.
(CheckerDispatcher._file_type): Use the JSON file type for .json files.
(CheckerDispatcher._create_checker): Use a JSONChecker for JSON files.

  • Scripts/webkitpy/style/checker_unittest.py:

(CheckerDispatcherDispatchTest.assert_checker_json): Added this helper method.
(CheckerDispatcherDispatchTest.test_json_paths): Added. Based on test_python_paths.

  • Scripts/webkitpy/style/checkers/jsonchecker.py: Added. (I didn't name this just "json",

which would have matched our other checkers, because I couldn't figure out how to call
"json.loads" without hitting namespace conflicts.)
(JSONChecker.init): Turn of line filtering so that we always check the whole file, not
just the modified lines from a patch.
(JSONChecker.check): Try to parse the lines as JSON. Mark an error if there was an
exception.
(JSONChecker.line_number_from_json_exception): Parse the json modules exception message to
try to extract a line number.

  • Scripts/webkitpy/style/checkers/jsonchecker_unittest.py: Added.

(MockErrorHandler.init):
(MockErrorHandler.turn_off_line_filtering):
(MockErrorHandler.call):
Helper class. Copied from xml_unittest.py.

(JSONCheckerTest.test_line_number_from_json_exception): Test the
line_number_from_json_exception helper method.

(JSONCheckerTest.assert_no_error):
(JSONCheckerTest.assert_error):
Helper methods to assert that we did or didn't get an error.

(JSONCheckerTest.mock_handle_style_error): Helper method.

(JSONCheckerTest.test_conflict_marker):
(JSONCheckerTest.test_single_quote):
(JSONCheckerTest.test_init):
(JSONCheckerTest.test_no_error):
Test various cases.

8:30 AM Changeset in webkit [101802] by kov@webkit.org
  • 2 edits in trunk/Tools

[GTK] Also run DumpRenderTree under jhbuild
​https://bugs.webkit.org/show_bug.cgi?id=73646

Reviewed by Martin Robinson.

  • Scripts/webkitpy/layout_tests/port/gtk.py:

(GtkDriver.cmd_line):

8:30 AM Changeset in webkit [101801] by Csaba Osztrogonác
  • 2 edits in trunk/Source/JavaScriptCore

MacroAssemblerSH4 does not implement readCallTarget
​https://bugs.webkit.org/show_bug.cgi?id=73434

Patch by Zoltan Herczeg <​zherczeg@webkit.org> on 2011-12-02
Reviewed by Csaba Osztrogonác.

  • assembler/SH4Assembler.h:

(JSC::SH4Assembler::readCallTarget): Support for SH4.

8:17 AM Changeset in webkit [101800] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt] fast/forms/select/listbox-in-multi-column.html fails
​https://bugs.webkit.org/show_bug.cgi?id=73660

  • platform/qt/Skipped: Skip fast/forms/select/listbox-in-multi-column.html until fix.
8:16 AM Changeset in webkit [101799] by alex
  • 3 edits in trunk

2011-12-02 Alejandro G. Castro <​alex@igalia.com>

[GTK] Fix TextureMapperCairo compilation
​https://bugs.webkit.org/show_bug.cgi?id=73655

Reviewed by Martin Robinson.

  • GNUmakefile.am: The variables need the USE part in the name.
  • configure.ac: We can not define the variable twice.
8:14 AM Changeset in webkit [101798] by apavlov@chromium.org
  • 2 edits in trunk/LayoutTests

[Chromium] Unreviewed expectation fix after r101787.

  • platform/chromium/svg/css/getComputedStyle-basic-expected.txt:
7:51 AM Changeset in webkit [101797] by vsevik@chromium.org
  • 14 edits in trunk/Source/WebCore

Unreviewed, rolling out r101794.
​http://trac.webkit.org/changeset/101794
​https://bugs.webkit.org/show_bug.cgi?id=73656

Broke win build (Requested by vsevik on #webkit).

Patch by Sheriff Bot <​webkit.review.bot@gmail.com> on 2011-12-02

  • bindings/js/ScriptCallStackFactory.cpp:
  • bindings/js/ScriptCallStackFactory.h:
  • bindings/scripts/CodeGeneratorJS.pm:

(GenerateParametersCheck):

  • bindings/scripts/CodeGeneratorV8.pm:

(GenerateFunctionCallback):

  • bindings/scripts/test/JS/JSTestObj.cpp:

(WebCore::jsTestObjPrototypeFunctionCustomArgsAndException):

  • bindings/scripts/test/V8/V8TestObj.cpp:

(WebCore::TestObjInternal::customArgsAndExceptionCallback):

  • bindings/v8/ScriptCallStackFactory.cpp:
  • bindings/v8/ScriptCallStackFactory.h:
  • inspector/InspectorInstrumentation.cpp:
  • inspector/InspectorInstrumentation.h:
  • inspector/WorkerInspectorController.h:
  • page/Console.cpp:

(WebCore::Console::shouldCaptureFullStackTrace):

  • page/Console.h:
7:32 AM Changeset in webkit [101796] by commit-queue@webkit.org
  • 7 edits
    2 adds in trunk

Unreviewed, revert r101347.
​https://bugs.webkit.org/show_bug.cgi?id=73580

It breaks the linking of Tools/ targets due to missing functions.

Patch by Raphael Kubo da Costa <​kubo@profusion.mobi> on 2011-12-02

.:

  • Source/cmake/OptionsEfl.cmake:

Source/WebCore:

  • PlatformEfl.cmake:
  • platform/graphics/GraphicsLayer.cpp:
  • platform/graphics/GraphicsLayer.h:
  • platform/graphics/efl/GraphicsLayerEfl.cpp: Added.

(WebCore::GraphicsLayer::create):
(WebCore::GraphicsLayerEfl::GraphicsLayerEfl):
(WebCore::GraphicsLayerEfl::~GraphicsLayerEfl):
(WebCore::GraphicsLayerEfl::setNeedsDisplay):
(WebCore::GraphicsLayerEfl::setNeedsDisplayInRect):

  • platform/graphics/efl/GraphicsLayerEfl.h: Added.
  • platform/graphics/texmap/TextureMapperNode.cpp:

(WebCore::solveCubicBezierFunction):
(WebCore::solveStepsFunction):

7:27 AM Changeset in webkit [101795] by vsevik@chromium.org
  • 1 edit
    1 move in trunk/LayoutTests

Unreviewed, updated chromium test expectations.

  • platform/chromium/fast/events/offsetX-offsetY-expected.txt: Renamed from LayoutTests/platform/chromium-mac/fast/events/offsetX-offsetY-expected.txt.
7:24 AM Changeset in webkit [101794] by vsevik@chromium.org
  • 14 edits in trunk/Source/WebCore

Web Inspector: Extract default call stack creation and check for front-end from console.
​https://bugs.webkit.org/show_bug.cgi?id=73566

Reviewed by Yury Semikhatsky.

  • bindings/js/ScriptCallStackFactory.cpp:

(WebCore::createScriptCallStack):

  • bindings/js/ScriptCallStackFactory.h:
  • bindings/scripts/CodeGeneratorJS.pm:

(GenerateParametersCheck):

  • bindings/scripts/CodeGeneratorV8.pm:

(GenerateFunctionCallback):

  • bindings/scripts/test/JS/JSTestObj.cpp:

(WebCore::jsTestObjPrototypeFunctionCustomArgsAndException):

  • bindings/scripts/test/V8/V8TestObj.cpp:

(WebCore::TestObjInternal::customArgsAndExceptionCallback):

  • bindings/v8/ScriptCallStackFactory.cpp:

(WebCore::createScriptCallStack):

  • bindings/v8/ScriptCallStackFactory.h:
  • inspector/InspectorInstrumentation.cpp:

(WebCore::InspectorInstrumentation::hasFrontendForScriptContext):

  • inspector/InspectorInstrumentation.h:

(WebCore::InspectorInstrumentation::hasFrontendForScriptContext):

  • inspector/WorkerInspectorController.h:

(WebCore::WorkerInspectorController::hasFrontend):

  • page/Console.cpp:
  • page/Console.h:
7:17 AM Changeset in webkit [101793] by Philippe Normand
  • 2 edits in trunk/LayoutTests

Unreviewed, skip some new GTK multicol failures because of bug 73653.

  • platform/gtk/Skipped:
7:15 AM Changeset in webkit [101792] by gavinp@chromium.org
  • 4 edits in trunk/Source/WebCore

Remove instrumentation tracking a fixed bug
​https://bugs.webkit.org/show_bug.cgi?id=73471

The underlying bug is fixed (bug 72068), and this instrumentation was intrusive and using
memory, plus the conditional compilation made me sad.

Reviewed by Nate Chapin.

No new tests.

  • dom/ScriptElement.cpp:

(WebCore::ScriptElement::ScriptElement):
(WebCore::ScriptElement::requestScript):
(WebCore::ScriptElement::stopLoadRequest):
(WebCore::ScriptElement::notifyFinished):

  • dom/ScriptElement.h:
  • dom/ScriptRunner.cpp:

(WebCore::ScriptRunner::queueScriptForExecution):

6:45 AM Changeset in webkit [101791] by abecsi@webkit.org
  • 2 edits in trunk/Source/WebKit2

[Qt] [WK2] Use input event timestamps in WebEvents if available
​https://bugs.webkit.org/show_bug.cgi?id=73647

Reviewed by Simon Hausmann.

Qt5 input events already have a native timestamp, use this timestamp
in WebEventFactory instead of WTF::currentTime if it is available.

  • Shared/qt/WebEventFactoryQt.cpp:

(WebKit::currentTimeForEvent):
(WebKit::WebEventFactory::createWebMouseEvent):
(WebKit::WebEventFactory::createWebWheelEvent):
(WebKit::WebEventFactory::createWebKeyboardEvent):
(WebKit::WebEventFactory::createWebTouchEvent):

  • Shared/qt/WebEventFactoryQt.h:
6:44 AM Changeset in webkit [101790] by vsevik@chromium.org
  • 2 edits in trunk/Source/WebCore

Web Inspector: Split view works weirdly in network panel when resizing, showing/hiding main element.
​https://bugs.webkit.org/show_bug.cgi?id=73650

Reviewed by Pavel Feldman.

  • inspector/front-end/SplitView.js:

(WebInspector.SplitView.prototype._updateResizer):
(WebInspector.SplitView.prototype.hideMainElement):
(WebInspector.SplitView.prototype.showMainElement):
(WebInspector.SplitView.prototype.onResize):
(WebInspector.SplitView.prototype._restoreSidebarWidth):

6:15 AM Changeset in webkit [101789] by vsevik@chromium.org
  • 6 edits
    1 delete in trunk/Source/WebCore

Unreviewed, rolling out r101783.
​http://trac.webkit.org/changeset/101783
​https://bugs.webkit.org/show_bug.cgi?id=73652

Broke chromium win build. (Requested by vsevik on #webkit).

Patch by Sheriff Bot <​webkit.review.bot@gmail.com> on 2011-12-02

  • WebCore.gyp/WebCore.gyp:
  • WebCore.gyp/scripts/action_derivedsourcesallinone.py:

(main):

  • WebCore.gypi:
  • bindings/scripts/generate-bindings.pl:
  • page/DOMWindow.idl:
  • webaudio/DOMWindowWebAudio.idl: Removed.
5:37 AM Changeset in webkit [101788] by jesus@webkit.org
  • 6 edits
    2 adds in trunk/Source/WebKit2

[Qt][WK2] Split the QtWebPageProxy into PageClient and QtPageProxy
​https://bugs.webkit.org/show_bug.cgi?id=66668

Reviewed by Simon Hausmann.

Split QtPageClient out of QtWebPageProxy.
This client will live in QQuickWebViewPrivate and communicate directly
with QtWebPageProxy and QtWebPageEventHandler. The functions that need
anything else than these entities will need to call the proper implementation
through QtWebPageProxy itself.
With this we have a clear separation between PageClient and PageProxy, having
two well defined and separated entities for hooking our clients with our API layer (QtWebPageProxy)
and for hooking the WebPageProxy with our client implementations (QtPageClient).
As a positive side-effect we have a cleaner QtWebPageProxy.

  • Target.pri:
  • UIProcess/API/qt/qquickwebview.cpp:

(QQuickWebViewPrivate::QQuickWebViewPrivate):

  • UIProcess/API/qt/qquickwebview_p_p.h:
  • UIProcess/qt/QtPageClient.cpp: Added.

(QtPageClient::QtPageClient):
(QtPageClient::~QtPageClient):
(QtPageClient::createDrawingAreaProxy):
(QtPageClient::setViewNeedsDisplay):
(QtPageClient::pageDidRequestScroll):
(QtPageClient::processDidCrash):
(QtPageClient::didRelaunchProcess):
(QtPageClient::didChangeContentsSize):
(QtPageClient::didChangeViewportProperties):
(QtPageClient::startDrag):
(QtPageClient::handleDownloadRequest):
(QtPageClient::setCursor):
(QtPageClient::setCursorHiddenUntilMouseMoves):
(QtPageClient::toolTipChanged):
(QtPageClient::registerEditCommand):
(QtPageClient::clearAllEditCommands):
(QtPageClient::canUndoRedo):
(QtPageClient::executeUndoRedo):
(QtPageClient::convertToDeviceSpace):
(QtPageClient::convertToUserSpace):
(QtPageClient::screenToWindow):
(QtPageClient::windowToScreen):
(QtPageClient::createPopupMenuProxy):
(QtPageClient::createContextMenuProxy):
(QtPageClient::flashBackingStoreUpdates):
(QtPageClient::didFindZoomableArea):
(QtPageClient::didReceiveMessageFromNavigatorQtObject):
(QtPageClient::doneWithTouchEvent):
(QtPageClient::displayView):
(QtPageClient::scrollView):
(QtPageClient::viewSize):
(QtPageClient::isViewWindowActive):
(QtPageClient::isViewFocused):
(QtPageClient::isViewVisible):
(QtPageClient::isViewInWindow):
(QtPageClient::enterAcceleratedCompositingMode):
(QtPageClient::exitAcceleratedCompositingMode):

  • UIProcess/qt/QtPageClient.h: Added.

(QtPageClient::pageClosed):
(QtPageClient::doneWithKeyEvent):
(QtPageClient::setFindIndicator):
(QtPageClient::didCommitLoadForMainFrame):
(QtPageClient::didFinishLoadingDataForCustomRepresentation):
(QtPageClient::customRepresentationZoomFactor):
(QtPageClient::setCustomRepresentationZoomFactor):
(QtPageClient::didChangeScrollbarsForMainFrame):
(QtPageClient::findStringInCustomRepresentation):
(QtPageClient::countStringMatchesInCustomRepresentation):
(QtPageClient::setEventHandler):
(QtPageClient::setPageProxy):

  • UIProcess/qt/QtWebPageProxy.cpp:

(QtWebPageProxy::QtWebPageProxy):

  • UIProcess/qt/QtWebPageProxy.h:
5:32 AM Changeset in webkit [101787] by apavlov@chromium.org
  • 7 edits in trunk

Inline non-replaced elements are reported to have zero width and height
​https://bugs.webkit.org/show_bug.cgi?id=61117

Reviewed by Antti Koivisto.

Source/WebCore:

According to ​http://www.w3.org/TR/CSS21/visudet.html, the "width" and "height" properties
do not apply for inline non-replaced elements and should have their initial value of "auto"
as their computed values.

  • css/CSSComputedStyleDeclaration.cpp:

(WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):

LayoutTests:

  • fast/dom/beforeload/image-object-before-load-innerHTML.html:
  • fast/dom/beforeload/image-object-before-load.html:
  • platform/chromium-win/svg/css/getComputedStyle-basic-expected.txt:
  • svg/css/getComputedStyle-basic-expected.txt:
4:58 AM Changeset in webkit [101786] by Philippe Normand
  • 5 edits in trunk/LayoutTests

Unreviewed, GTK rebaseline.

  • platform/gtk/Skipped: plugins/invalidate_rect.html was removed. Also skip one more SVG test.
  • platform/gtk/fast/writing-mode/broken-ideograph-small-caps-expected.txt:
  • platform/gtk/fonts/custom-font-missing-glyphs-expected.txt:
  • platform/gtk/fonts/fontconfig-synthetic-bold-expected.txt:
4:34 AM Changeset in webkit [101785] by commit-queue@webkit.org
  • 7 edits in trunk/Source

[chromium] Make CCInputHandler scrolling stateful
​https://bugs.webkit.org/show_bug.cgi?id=73345

This change makes the scrolling part of CCInputHandler stateful by
replacing scrollRootLayer() with scrollBegin(), scrollBy() and
scrollEnd(). This is done in preparation for scrollable sublayers.
Specifically, scrollBegin() will allow CCLayerTreeHostImpl to perform
input event hit testing to find the layer to be scrolled.

Patch by Sami Kyostila <​skyostil@chromium.org> on 2011-12-02
Reviewed by Steve Block.

Source/WebCore:

Tested in CCLayerTreeHostImplTest.

  • platform/graphics/chromium/cc/CCInputHandler.h:
  • platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp:

(WebCore::CCLayerTreeHostImpl::CCLayerTreeHostImpl):
(WebCore::CCLayerTreeHostImpl::currentTimeMs):
(WebCore::CCLayerTreeHostImpl::setNeedsRedraw):
(WebCore::findInnermostScrollableLayerAtPoint):
(WebCore::CCLayerTreeHostImpl::scrollBegin):
(WebCore::CCLayerTreeHostImpl::scrollBy):

  • platform/graphics/chromium/cc/CCLayerTreeHostImpl.h:

Source/WebKit/chromium:

  • src/WebCompositorImpl.cpp:

(WebKit::WebCompositorImpl::handleInputEvent):

  • tests/CCLayerTreeHostImplTest.cpp:

(WebKit::TEST_F):

4:34 AM Changeset in webkit [101784] by alex
  • 2 edits in trunk

2011-12-02 Alejandro G. Castro <​alex@igalia.com>

[GTK] Improve C++0x compat warnings check
​https://bugs.webkit.org/show_bug.cgi?id=73642

Reviewed by Martin Robinson.

  • configure.ac:
4:13 AM Changeset in webkit [101783] by haraken@chromium.org
  • 6 edits
    1 add in trunk/Source/WebCore

Use the [Supplemental] IDL for webaudio attributes in Chromium
​https://bugs.webkit.org/show_bug.cgi?id=73394

Reviewed by Adam Barth.

  • Overview: Using the [Supplemental] IDL, this patch moves the attribute

declarations of webaudio from DOMWindow.idl into a new IDL file
webaudio/DOMWindowWebAudio.idl, which helps make webaudio a self-contained
feature (aka a module).

  • This patch changes the build flow of WebCore.gyp as follows:

Previous build flow:

foreach $idl (all IDL files) {

generate-bindings.pl depends on $idl;
generate-bindings.pl reads $idl;
generate-bindings.pl generates .h and .cpp files for $idl;

}

New build flow (See the discussions in bug 72138 for more details):

resolve-supplemental.pl depends on all IDL files;
resolve-supplemental.pl reads all IDL files;
resolve-supplemental.pl resolves the dependency of [Supplemental=XXXX];
resolve-supplemental.pl outputs supplemental_dependency.tmp;
foreach $idl (all IDL files) {

generate-bindings.pl depends on $idl and supplemental_dependency.tmp;
generate-bindings.pl reads $idl;
generate-bindings.pl reads supplemental_dependency.tmp;
generate-bindings.pl generates .h and .cpp files for $idl, including all attributes in IDL files whilementing $idl;

}

  • This patch introduces a temporary IDL, [Supplemented]. The [Supplemented] IDL

will be removed after build scripts for all platforms support the [Supplemental] IDL.
The motivation for the [Supplemented] IDL is as follows:

In order to support the [Supplemental] IDL, we need to
(1) run resolve-supplemental.pl and generate supplemental_dependency.tmp
(2) and run generate-bindings.pl with the supplemental_dependency.tmp.

This build flow requires a change on the following build scripts,
but changing all the build scripts all at once without any regression is too difficult:

  • DerivedSources.make
  • DerivedSources.pri
  • GNUmakefile.am
  • PlatformBlackBerry.cmake
  • UseJSC.cmake
  • UseV8.cmake
  • WebCore.vcproj/MigrateScripts
  • WebCore.vcproj/WebCore.vcproj
  • bindings/gobject/GNUmakefile.am
  • WebCore.gyp/WebCore.gyp

Thus, we are planning to change the build scripts one by one, which implies that
we need to allow the temporary state in which some build scripts support [Supplemental] IDL
but others do not. To accomplish this, we introduce a temporary IDL, [Supplemented].
The [Supplemented] IDL on an attribute means that the attribute is marked with [Supplemental]
in another IDL file somewhere, like this:

DOMWindowWebAudio.idl:

interface [

Supplemental=DOMWindow

] DOMWindowWebAudio {

attribute attr1;
attribute attr2;

};

DOMWindow.idl:

interface [
] DOMWindow {

attribute [Supplemented] attr1; This line will be removed after all build scripts support the [Su IDL
attribute [Supplemented] attr2; This line will be removed after all build scripts support the [Su IDL.
attribute attr3;
attribute attr4;

};

Assuming these IDL files, this patch implements the following logic in generate-bindings.pl:

  • If a given build script supports the [Supplemental] IDL, generate-bindings.pl ignores all attributes with the [Supplemented] IDL.
  • Otherwise, generate-bindings.pl treats all attributes with the [Supplemented] IDL as normal attributes and instead ignores all attributes with the [Supplemental] IDL (i.e. generate-bindings.pl generates nothing from the IDL file with the [Supplemental] IDL).

Tests: webaudio/*

  • WebCore.gyp/WebCore.gyp: Describes the build flow that I described above.
  • WebCore.gyp/scripts/action_derivedsourcesallinone.py:

(main): Reads the IDL file names from the input file (i.e. supplemental_dependency.tmp), which are described at the first column of each line in the input file.

  • WebCore.gypi: Added DOMWindowWebAudio.idl.
  • bindings/scripts/generate-bindings.pl: As a temporary solution, if the platform does not support the [Supplemental] IDL, the perl script ignores the [Supplemental] IDL and instead uses the [Supplemented] IDL. Otherwise, the perl script ignores the [Supplemented] IDL and instead uses the [Supplemental] IDL.
  • page/DOMWindow.idl: Added the [Supplemented] IDL to webaudio-related attributes. As I described above, the [Supplemented] IDL will be removed after all platforms support the [Supplemental] IDL.
  • webaudio/DOMWindowWebAudio.idl: Added. Describes the [Supplemental=DOMWindow] IDL. The attributes in this IDL file should be treated as if they are written in DOMWindow.idl.
4:10 AM Changeset in webkit [101782] by vsevik@chromium.org
  • 2 edits in trunk/Source/WebCore

Web Inspector: Network panel row highlighting does not work.
​https://bugs.webkit.org/show_bug.cgi?id=73644

Reviewed by Pavel Feldman.

  • inspector/front-end/networkLogView.css:

(.network-log-grid tr.highlighted-row):
(from):
(to):

3:53 AM Changeset in webkit [101781] by kenneth@webkit.org
  • 3 edits in trunk/Source/WebKit2

[Qt] Do not apply the transition state before we are unsuspended

Reviewed by Simon Hausmann.

Also remove an accessor in the private, which is unneeded as
everything in the private class is supposed to be public or
only used by the private itself.

  • UIProcess/API/qt/qquickwebview.cpp:

(QQuickWebViewPrivate::QQuickWebViewPrivate):
(QQuickWebViewPrivate::loadDidCommit):
(QQuickWebViewPrivate::didFinishFirstNonEmptyLayout):
(QQuickWebViewPrivate::_q_suspend):
(QQuickWebViewPrivate::_q_resume):
(QQuickWebViewPrivate::didChangeContentsSize):
(QQuickWebViewPrivate::didChangeViewportProperties):

  • UIProcess/API/qt/qquickwebview_p_p.h:
3:52 AM Changeset in webkit [101780] by kenneth@webkit.org
  • 2 edits in trunk/Source/WebKit2

Unreviewed buildfix, add missing include.

  • WebProcess/Notifications/WebNotificationManager.cpp:
3:50 AM Changeset in webkit [101779] by pfeldman@chromium.org
  • 2 edits in trunk/Source/WebCore

Not reviewed: remove console.timeStamp from the inspector backend dispatcher.

  • inspector/front-end/InspectorBackend.js:

(InspectorBackendClass.prototype.sendMessageObjectToBackend):

3:41 AM Changeset in webkit [101778] by morrita@google.com
  • 27 edits
    6 deletes in trunk

Unreviewed, rolling out r101751 and r101775.
​http://trac.webkit.org/changeset/101751
​http://trac.webkit.org/changeset/101775
​https://bugs.webkit.org/show_bug.cgi?id=73191

breaks Windows build

Source/JavaScriptCore:

  • JavaScriptCore.xcodeproj/project.pbxproj:
  • config.h:
  • runtime/JSExportMacros.h: Removed.
  • wtf/ExportMacros.h:
  • wtf/Platform.h:
  • wtf/WTFThreadData.h:
  • wtf/text/AtomicString.h:
  • wtf/text/StringStatics.cpp:

Source/JavaScriptGlue:

  • ForwardingHeaders/runtime/JSExportMacros.h: Removed.
  • ForwardingHeaders/wtf/ExportMacros.h: Removed.
  • config.h:

Source/WebCore:

  • ForwardingHeaders/runtime/JSExportMacros.h: Removed.
  • ForwardingHeaders/wtf/ExportMacros.h: Removed.
  • WebCore.vcproj/QTMovieWinCommon.vsprops:
  • WebCore.xcodeproj/project.pbxproj:
  • config.h:
  • platform/PlatformExportMacros.h: Removed.

Source/WebKit/cf:

  • WebCoreSupport/WebInspectorClientCF.cpp:

Source/WebKit/mac:

  • WebKitPrefix.h:

Source/WebKit/qt:

  • tests/MIMESniffing/MIMESniffing.pro:

Source/WebKit2:

  • config.h:

Tools:

  • DumpRenderTree/chromium/config.h:
  • DumpRenderTree/config.h:
  • TestWebKitAPI/config.h:
  • WebKitTestRunner/config.h:
3:37 AM Changeset in webkit [101777] by pfeldman@chromium.org
  • 6 edits in trunk/Source/WebCore

Web Inspector: introduce backend stub generation from JSON for the standalone development / remote front-ends.
​https://bugs.webkit.org/show_bug.cgi?id=73636

Reviewed by Yury Semikhatsky.

  • inspector/front-end/DOMStorage.js:
  • inspector/front-end/Database.js:
  • inspector/front-end/InspectorBackend.js:

(InspectorBackendClass):
(InspectorBackendClass.prototype.registerCommand):
(InspectorBackendClass.prototype.registerEvent):
(InspectorBackendClass.prototype.runAfterPendingDispatches):
(InspectorBackendClass.prototype.loadFromJSONIfNeeded):

  • inspector/front-end/TimelineManager.js:
  • inspector/front-end/inspector.js:
3:19 AM Changeset in webkit [101776] by vsevik@chromium.org
  • 2 edits in trunk/LayoutTests

Unreviewed, updated test expectations.

  • platform/chromium/test_expectations.txt:
3:14 AM Changeset in webkit [101775] by morrita@google.com
  • 2 edits in trunk/Source/WebCore

Unreviewed, another attempt to build fix for r101751:
Adding an include path to make PlatformExportMacros.h visibile from QTMovieWin.

Patch by MORITA Hajime <​morrita@google.com> on 2011-12-02

  • WebCore.vcproj/QTMovieWinCommon.vsprops:
3:09 AM Changeset in webkit [101774] by vsevik@chromium.org
  • 1 edit
    1 move in trunk/LayoutTests

Unreviewed test expectations.

  • platform/chromium-mac/fast/events/offsetX-offsetY-expected.txt: Renamed from LayoutTests/platform/gtk/fast/events/offsetX-offsetY-expected.txt.
3:08 AM Changeset in webkit [101773] by morrita@google.com
  • 2 edits in trunk/Source/WebCore

Unreviewed, rolling out r101772.
​http://trac.webkit.org/changeset/101772

It didn't fix the build failure

  • WebCore.vcproj/QTMovieWinCommon.vsprops:
2:55 AM Changeset in webkit [101772] by morrita@google.com
  • 2 edits in trunk/Source/WebCore

2011-12-02 MORITA Hajime <​morrita@google.com>

Unreviewed attempt to build fix for r101751:
Adding an include path to make PlatformExportMacros.h visibile from QTMovieWin.

  • WebCore.vcproj/QTMovieWinCommon.vsprops:
2:52 AM Changeset in webkit [101771] by Philippe Normand
  • 3 edits in trunk/LayoutTests

Unreviewed, GTK rebaseline after r101723 and skipping another set
of svg tests show 1px differences.

  • platform/gtk/Skipped:
  • platform/gtk/fast/dom/Window/window-properties-expected.txt:
2:20 AM Changeset in webkit [101770] by Carlos Garcia Campos
  • 2 edits in trunk/Source/WebKit2

Unreviewed. Fix GTK+ build after r101750.

  • GNUmakefile.am: Add missing files.
2:17 AM Changeset in webkit [101769] by caseq@chromium.org
  • 13 edits in trunk

Web Inspector: [Extensions API] pass preferred resource line number to extension's open resource handler
​https://bugs.webkit.org/show_bug.cgi?id=73084

Reviewed by Pavel Feldman.

Source/WebCore:

  • inspector/front-end/ExtensionAPI.js:

(injectedExtensionAPI.Panels.prototype.setOpenResourceHandler.else.callbackWrapper):
(injectedExtensionAPI.Panels.prototype.setOpenResourceHandler):

  • inspector/front-end/ExtensionServer.js:

(WebInspector.ExtensionServer.prototype._handleOpenURL):

  • inspector/front-end/HandlerRegistry.js:

(get WebInspector.HandlerRegistry.prototype.set dispatch):

  • inspector/front-end/JavaScriptSourceFrame.js:

(WebInspector.JavaScriptSourceFrame.prototype.populateTextAreaContextMenu):

  • inspector/front-end/ResourcesPanel.js:

(WebInspector.FrameResourceTreeElement.prototype._handleContextMenuEvent):

  • inspector/front-end/SourceFrame.js:

(WebInspector.SourceFrame.prototype.populateLineGutterContextMenu):
(WebInspector.SourceFrame.prototype.populateTextAreaContextMenu):
(WebInspector.TextViewerDelegateForSourceFrame.prototype.populateLineGutterContextMenu):
(WebInspector.TextViewerDelegateForSourceFrame.prototype.populateTextAreaContextMenu):

  • inspector/front-end/TextViewer.js:

(WebInspector.TextViewer.prototype._contextMenu):
(WebInspector.TextViewerDelegate.prototype.populateLineGutterContextMenu):
(WebInspector.TextViewerDelegate.prototype.populateTextAreaContextMenu):

  • inspector/front-end/externs.js:

(WebInspector.populateResourceContextMenu):

  • inspector/front-end/inspector.js:

(WebInspector.populateResourceContextMenu):
(WebInspector._showAnchorLocation):

LayoutTests:

  • inspector/extensions/extensions-resources-expected.txt:
  • inspector/extensions/extensions-resources.html:
2:14 AM Changeset in webkit [101768] by kov@webkit.org
  • 2 edits in trunk/Source/WebKit2

Unreviewed. GTK+ build fix.

  • GNUmakefile.am:
1:31 AM Changeset in webkit [101767] by haraken@chromium.org
  • 2 edits
    3 adds in trunk/Tools

Move run-bindings-tests implementation to webkitpy/bindings/main.py
​https://bugs.webkit.org/show_bug.cgi?id=73619

Reviewed by Adam Barth.

As run-bindings-tests has been growing, this patch just moves its implementation
to webkitpy/bindings/main.py to integrate with the rest of webkitpy.

No change in behavior.

  • Scripts/run-bindings-tests:

(main):

  • Scripts/webkitpy/bindings/init.py: Added.
  • Scripts/webkitpy/bindings/main.py: Added.

(BindingsTests.init):
(BindingsTests.generate_from_idl):
(BindingsTests.generate_supplemental_dependency):
(BindingsTests.detect_changes):
(BindingsTests.run_tests):

1:24 AM Changeset in webkit [101766] by vsevik@chromium.org
  • 1 edit
    1 add in trunk/LayoutTests

Unreviewed, added custom chromium test expectations.

  • platform/chromium/fast/images/image-css3-content-data-expected.png: Added.
1:18 AM Changeset in webkit [101765] by Philippe Normand
  • 4 edits in trunk/Source/WebKit/gtk

[GTK] scrollbars interfering with fullscreen (in-window) video rendering
​https://bugs.webkit.org/show_bug.cgi?id=73577

Reviewed by Martin Robinson.

Disable scrollbars when switching to fullscreen.

  • WebCoreSupport/ChromeClientGtk.cpp:

(WebKit::ChromeClient::contentsSizeChanged): Ignore size changes
when an element is being displayed fullscreen.
(WebKit::ChromeClient::enterFullScreenForElement): Disable
scrollbars when entering fullscreen.

  • WebCoreSupport/GtkAdjustmentWatcher.cpp: Added 2 methods to

enable/disable scrollbars and one method to know whether they are
enabled or not.
(WebKit::GtkAdjustmentWatcher::GtkAdjustmentWatcher):
(WebKit::updateAdjustmentFromScrollbar):
(WebKit::GtkAdjustmentWatcher::updateAdjustmentsFromScrollbars):
(WebKit::GtkAdjustmentWatcher::updateAdjustmentsFromScrollbarsLater):
(WebKit::GtkAdjustmentWatcher::disableAllScrollbars):
(WebKit::GtkAdjustmentWatcher::enableAllScrollbars):

  • WebCoreSupport/GtkAdjustmentWatcher.h:

(WebKit::GtkAdjustmentWatcher::scrollbarsDisabled):

1:16 AM Changeset in webkit [101764] by Martin Robinson
  • 7 edits in trunk

[GTK] Add compilation options to enable/disable Accelerated Compositing and to choose texture mapper implementation.
​https://bugs.webkit.org/show_bug.cgi?id=73458

Patch by Nayan Kumar K <​nayankk@motorola.com> on 2011-12-01
Reviewed by Martin Robinson.

.:

  • GNUmakefile.am: Export new compilation macros.
  • configure.ac: Provide option to choose accelerated compositing and texture mapper variations.

Source/WebCore:

No new tests added as this patch doesn't affect any functionality.

  • GNUmakefile.am: Guard the include files.
  • GNUmakefile.list.am: Guard the compilation of few files.
  • platform/graphics/GraphicsLayer.h: Guard the typedef of GraphicsLayer.
1:15 AM Changeset in webkit [101763] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebKit2

[Qt][WK2] ASSERT at QQuickWebViewPrivate::computeViewportConstraints() when API test is exercising zero sized views.
​https://bugs.webkit.org/show_bug.cgi?id=73441

Do not assert on empty available size, but return default constrain values instead.

Patch by Zalan Bujtas <​zbujtas@gmail.com> on 2011-12-02
Reviewed by Kenneth Rohde Christiansen.

  • UIProcess/API/qt/qquickwebview.cpp:

(QQuickWebViewPrivate::computeViewportConstraints):

  • WebProcess/WebCoreSupport/WebChromeClient.cpp:

(WebKit::WebChromeClient::dispatchViewportPropertiesDidChange):

1:12 AM Changeset in webkit [101762] by rniwa@webkit.org
  • 3 edits in trunk/LayoutTests

Now that r101733 fixed fast/block/child-not-removed-from-parent-lineboxes-crash.html,
remove the failing expectation on Chromium and re-enable it on Qt.

  • platform/chromium/test_expectations.txt:
  • platform/qt/Skipped:
1:09 AM Changeset in webkit [101761] by pfeldman@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

Web Inspector [chromium]: provisional fix for interactive ui tests
​https://bugs.webkit.org/show_bug.cgi?id=73633

Reviewed by Yury Semikhatsky.

  • src/js/Tests.js:

(.):
(.TestSuite.prototype._executeCodeWhenScriptsAreParsed):

1:06 AM Changeset in webkit [101760] by caseq@chromium.org
  • 13 edits in trunk

Web Inspector: use object properties, not element attributes to pass preferred panel/line/request id in linkified anchors
​https://bugs.webkit.org/show_bug.cgi?id=73556

Reviewed by Pavel Feldman.

Source/WebCore:

  • inspector/front-end/AuditFormatters.js:

(WebInspector.AuditFormatters.resourceLink):

  • inspector/front-end/ConsoleMessage.js:

(WebInspector.ConsoleMessageImpl.prototype._formatMessage.else.else.linkifier):
(WebInspector.ConsoleMessageImpl.prototype._formatMessage):

  • inspector/front-end/DebuggerPresentationModel.js:

(WebInspector.DebuggerPresentationModel.Linkifier.prototype.linkifyLocation):
(WebInspector.DebuggerPresentationModel.Linkifier.prototype._updateAnchor):

  • inspector/front-end/NetworkPanel.js:

(WebInspector.NetworkPanel.prototype._resourceByAnchor):
(WebInspector.NetworkDataGridNode.prototype._refreshInitiatorCell):

  • inspector/front-end/ResourceUtils.js:

(WebInspector.linkifyStringAsFragment):
(WebInspector.linkifyResourceAsNode):
(WebInspector.linkifyRequestAsNode):

  • inspector/front-end/ResourcesPanel.js:

(WebInspector.ResourcesPanel.prototype.showAnchorLocation):

  • inspector/front-end/inspector.js:

(WebInspector._showAnchorLocation):

LayoutTests:

  • inspector/extensions/extensions-audits-expected.txt:
  • inspector/extensions/extensions-audits.html:
12:58 AM Changeset in webkit [101759] by levin@chromium.org
  • 9 edits
    1 add in trunk/Source/WebKit/chromium

[chromium] Add WebKit API's to support the autosize algorithm in renderer process.
​https://bugs.webkit.org/show_bug.cgi?id=73058

Reviewed by Darin Fisher.

  • public/WebView.h: Expose the auto-resize method.
  • public/WebWidgetClient.h:

(WebKit::WebWidgetClient::didAutoResize):

  • src/WebFrameImpl.cpp:

(WebKit::WebFrameImpl::createFrameView): Set the auto-resize
state on the new view.

  • src/WebViewImpl.cpp:

(WebKit::WebViewImpl::WebViewImpl):
(WebKit::WebViewImpl::resize): Extracted sendResizeEventAndRepaint, so
that it can be used by layoutUpdated.
(WebKit::WebViewImpl::queueBothResizeEventAndPaint): Ditto.
(WebKit::WebViewImpl::hasHorizontalScrollbar): Added for testing purposes.
(WebKit::WebViewImpl::hasVerticalScrollbar): Ditto.
(WebKit::WebViewImpl::enableAutoResizeMode): Set-up auto-resize.
(WebKit::WebViewImpl::layoutUpdated): Handle the auto-resize case by
sending events and invalidation.

  • src/WebViewImpl.h: Expose the aut-reosize information.

(WebKit::WebViewImpl::shouldAutoResize):
(WebKit::WebViewImpl::minAutoSize):
(WebKit::WebViewImpl::maxAutoSize):

  • tests/FrameTestHelpers.cpp:

(WebKit::FrameTestHelpers::createWebViewAndLoad): Added the ability to
specify the WebViewClient.

  • tests/FrameTestHelpers.h: Ditto.
  • tests/WebViewTest.cpp: Added a simple test for auto-resize.

(WebKit::TestData::setWebView):
(WebKit::TestData::setSize): Capture the new size.
(WebKit::TestData::hasHorizontalScrollbar):
(WebKit::TestData::hasVerticalScrollbar):
(WebKit::TestData::width):
(WebKit::TestData::height):
(WebKit::AutoResizeWebViewClient::didAutoResize): Handle the resize event.
(WebKit::AutoResizeWebViewClient::testData):
(WebKit::TEST_F): The actual test.

  • tests/data/specify_size.html: Added.
12:55 AM Changeset in webkit [101758] by commit-queue@webkit.org
  • 10 edits
    2 copies
    1 add in trunk/Source/WebKit/chromium

[Chromium] Support adding/removing page overlay to WebView
​https://bugs.webkit.org/show_bug.cgi?id=73235

Patch by Xiyuan Xia <​xiyuan@chromium.org> on 2011-12-02
Reviewed by James Robinson.

  • WebKit.gyp:
  • public/WebPageOverlay.h: Added.

(WebKit::WebPageOverlay::~WebPageOverlay):

  • public/WebView.h:
  • src/PageOverlay.cpp:

(WebKit::PageOverlay::create):
(WebKit::PageOverlay::PageOverlay):
(WebKit::OverlayGraphicsLayerClientImpl::create):
(WebKit::OverlayGraphicsLayerClientImpl::paintContents):
(WebKit::OverlayGraphicsLayerClientImpl::OverlayGraphicsLayerClientImpl):
(WebKit::PageOverlay::update):
(WebKit::PageOverlay::paintWebFrame):
(WebKit::PageOverlay::invalidateWebFrame):

  • src/PageOverlay.h:

(WebKit::PageOverlay::overlay):
(WebKit::PageOverlay::setOverlay):
(WebKit::PageOverlay::zOrder):
(WebKit::PageOverlay::setZOrder):

  • src/PageOverlayList.cpp: Added.

(WebKit::PageOverlayList::create):
(WebKit::PageOverlayList::PageOverlayList):
(WebKit::PageOverlayList::~PageOverlayList):
(WebKit::PageOverlayList::add):
(WebKit::PageOverlayList::remove):
(WebKit::PageOverlayList::update):
(WebKit::PageOverlayList::paintWebFrame):
(WebKit::PageOverlayList::find):

  • src/PageOverlayList.h: Added.

(WebKit::PageOverlayList::empty):

  • src/WebDevToolsAgentImpl.cpp:

(WebKit::WebDevToolsAgentImpl::paintPageOverlay):
(WebKit::WebDevToolsAgentImpl::highlight):
(WebKit::WebDevToolsAgentImpl::hideHighlight):

  • src/WebDevToolsAgentImpl.h:
  • src/WebFrameImpl.cpp:

(WebKit::WebFrameImpl::paintWithContext):

  • src/WebViewImpl.cpp:

(WebKit::WebViewImpl::composite):
(WebKit::WebViewImpl::addPageOverlay):
(WebKit::WebViewImpl::removePageOverlay):
(WebKit::WebViewImpl::setIsAcceleratedCompositingActive):
(WebKit::WebViewImpl::didRecreateGraphicsContext):

  • src/WebViewImpl.h:

(WebKit::WebViewImpl::pageOverlays):

12:49 AM BuildingGtk edited by Martin Robinson
(diff)
12:44 AM Changeset in webkit [101757] by loislo@chromium.org
  • 7 edits in trunk/Tools

Tools/Scripts: eliminate find_test_files from Port class.
​https://bugs.webkit.org/show_bug.cgi?id=73553

rebaseline.py and chromium_gpu.py use Port.find_test_files() the same way as it is used in Port.tests()
I'd like to replace all calls to find_test_files with tests and eliminate find_tests_files.
Also I'll move _is_test_file() and related functions close to Port.tests().

Reviewed by Dirk Pranke.

  • Scripts/webkitpy/layout_tests/port/base.py:

(Port.tests):
(Port.is_reference_html_file):
(Port._has_supported_extension):
(Port._is_test_file):

  • Scripts/webkitpy/layout_tests/port/base_unittest.py:

(PortTest.test_find_no_paths_specified):
(PortTest.test_find_one_test):
(PortTest.test_find_glob):
(PortTest.test_find_with_skipped_directories):
(PortTest.test_find_with_skipped_directories_2):
(PortTest.test_is_test_file):

  • Scripts/webkitpy/layout_tests/port/chromium_gpu.py:

(_default_tests_paths):
(ChromiumGpuLinuxPort.tests):
(ChromiumGpuCgMacPort.tests):
(ChromiumGpuMacPort.tests):

  • Scripts/webkitpy/layout_tests/port/chromium_gpu_unittest.py:

(ChromiumGpuTest.test_default_tests_paths.test_paths):
(ChromiumGpuTest.test_default_tests_paths):
(ChromiumGpuTest.test_test_files.test_paths):
(ChromiumGpuTest):
(ChromiumGpuTest.test_test_files):

  • Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py:

(get_tests_run.RecordingTestDriver.run_test):

  • Scripts/webkitpy/tool/commands/rebaseline.py:

(OptimizeBaselines._optimize_baseline):
(OptimizeBaselines.execute):
(AnalyzeBaselines._analyze_baseline):
(AnalyzeBaselines.execute):

12:30 AM Changeset in webkit [101756] by pfeldman@chromium.org
  • 6 edits in trunk

Web Inspector: refactor InspectorBackendDispatcher so that it does not use JSON-serialized command templates.
​https://bugs.webkit.org/show_bug.cgi?id=73569

Reviewed by Yury Semikhatsky.

Source/WebCore:

  • inspector/CodeGeneratorInspector.py:

(Generator.process_command):

  • inspector/Inspector.json:
  • inspector/front-end/InspectorBackend.js:

(InspectorBackendClass.prototype._wrap.callback):
(InspectorBackendClass.prototype._wrap):
(InspectorBackendClass.prototype.registerCommand):
(InspectorBackendClass.prototype._invoke):
(InspectorBackendClass.prototype._sendMessageToBackend):
(InspectorBackendClass.prototype._wrapCallbackAndSendMessageObject):

LayoutTests:

  • inspector/report-API-errors-expected.txt:
12:26 AM Changeset in webkit [101755] by commit-queue@webkit.org
  • 14 edits
    8 adds in trunk

Range sliders and spin buttons don't work with multi-columns.
​https://bugs.webkit.org/show_bug.cgi?id=70898

Patch by Yosifumi Inoue <​yosin@chromium.org> on 2011-12-02
Reviewed by Dan Bernstein.

Source/WebCore:

This patch makes RenderBlock::hitTestColumns and
RenderBoxModelObject::mapAbsoluteToLocal to handle point
in multi-column same logic.

In multi-column, coordinate of box model rendering object is different
from absolute coordinate.. Columns in box model rendering object spans
vertically rather than horizontally.

When absolute point is represented in (column[i]+dx, column[0]+dy),
it is (column[0]+dx, column[0] + columnHeight + dy) in box model
rendering object coordinate.

Tests: fast/events/document-elementFromPoint.html

fast/events/offsetX-offsetY.html
fast/forms/number/spin-in-multi-column.html
fast/forms/range/slider-in-multi-column.html
fast/forms/select/listbox-in-multi-column.html

  • rendering/RenderBlock.cpp:

(WebCore::ColumnRectIterator::ColumnRectIterator): Added
(WebCore::ColumnRectIterator::advance): Added
(WebCore::ColumnRectIterator::columnRect): Added
(WebCore::ColumnRectIterator::hasMore): Added
(WebCore::ColumnRectIterator::adjust): Added
(WebCore::ColumnRectIterator::update): Added
(WebCore::RenderBlock::hitTestColumns): Use ColumnRectIterator.
(WebCore::RenderBlock::adjustForColumnRect): Added

  • rendering/RenderBlock.h: Add adjustForColumnRect.
  • rendering/RenderBox.cpp:

(WebCore::RenderBox::mapAbsoluteToLocalPoint): Call RenderBoxModelObject::mapAbsoluteToLocalPoint.

  • rendering/RenderBoxModelObject.cpp:

(WebCore::RenderBoxModelObject::mapAbsoluteToLocalPoint): Move from RenderInline::mapAbsoluteToLocalPoint and call RenderBlock::adjustForColumnRect.

  • rendering/RenderBoxModelObject.h: add mapAbsoluteToLocalPoint.
  • rendering/RenderInline.cpp: Move mapAbsoluteToLocalPoint to RenderBoxModelObject.
  • rendering/RenderInline.h: remove mapAbsoluteToLocalPoint.

LayoutTests:

  • fast/events/offsetX-offsetY-expected.txt: Change offset for "in-columns" test.
  • fast/events/offsetX-offsetY.html: Change offset for "in-columns" test.
  • fast/forms/listbox/listbox-in-multi-column-expected.txt: Added.
  • fast/forms/listbox/listbox-in-multi-column.html: Added.
  • fast/forms/number/spin-in-multi-column-expected.txt: Added.
  • fast/forms/number/spin-in-multi-column.html: Added.
  • fast/forms/range/slider-in-multi-column-expected.txt: Added.
  • fast/forms/range/slider-in-multi-column.html: Added.
  • platform/chromium-win/fast/events/offsetX-offsetY-expected.txt: Change offset for "in-columns" test.
  • platform/efl/fast/events/offsetX-offsetY-expected.txt: Change offset for "in-columns" test.
  • platform/gtk/fast/events/offsetX-offsetY-expected.txt: Change offset for "in-columns" test.
12:21 AM Changeset in webkit [101754] by pfeldman@chromium.org
  • 9 edits in trunk

InspectorController destruction order leads to use-after-free
​https://bugs.webkit.org/show_bug.cgi?id=73582

Reviewed by Yury Semikhatsky.

Source/WebCore:

  • inspector/InspectorBaseAgent.h:

(WebCore::InspectorBaseAgentInterface::discardAgent):

  • inspector/InspectorCSSAgent.cpp:

(WebCore::InspectorCSSAgent::~InspectorCSSAgent):
(WebCore::InspectorCSSAgent::discardAgent):

  • inspector/InspectorCSSAgent.h:
  • inspector/InspectorController.cpp:

(WebCore::InspectorController::~InspectorController):

  • inspector/InspectorDOMDebuggerAgent.cpp:

(WebCore::InspectorDOMDebuggerAgent::~InspectorDOMDebuggerAgent):
(WebCore::InspectorDOMDebuggerAgent::discardAgent):

  • inspector/InspectorDOMDebuggerAgent.h:

LayoutTests:

  • platform/chromium/test_expectations.txt:
12:05 AM Changeset in webkit [101753] by leo.yang@torchmobile.com.cn
  • 6 edits
    3 adds in trunk

image element with src attribute can't be replaced by content: url() style
​https://bugs.webkit.org/show_bug.cgi?id=42840

Source/WebCore:

ImageLoader were updating renderer even if the renderer's image is
style generated content. This was wrong because if an image element
with src attribute and style="content: url(...)" attribute the
src image might override content image. The correct behavior should
be showing content image.

This patch is differentiating style generated RenderImage from the
normal RenderImage and keeps the RenderImageSource untouched if the
renderer is generated content.

Reviewed by Darin Adler.

Test: fast/images/image-css3-content-data.html

  • loader/ImageLoader.cpp:

(WebCore::ImageLoader::renderImageResource):

  • rendering/RenderImage.cpp:

(WebCore::RenderImage::RenderImage):

  • rendering/RenderImage.h:

(WebCore::RenderImage::setIsGeneratedContent):
(WebCore::RenderImage::isGeneratedContent):

  • rendering/RenderObject.cpp:

(WebCore::RenderObject::createObject):

LayoutTests:

Test case contains image element with src attribute and "content: url(...)"
style attribute.

Reviewed by Darin Adler.

  • fast/images/image-css3-content-data-expected.txt: Added.
  • fast/images/image-css3-content-data.html: Added.
  • fast/images/resources/green-24x24.jpg: Added.
Note: See TracTimeline for information about the timeline view.