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

Timeline



Apr 16, 2011:

11:29 PM Changeset in webkit [84100] by commit-queue@webkit.org
  • 3 edits
    14 adds in trunk/Source/WebKit/qt

2011-04-16 Caio Marcelo de Oliveira Filho <caio.oliveira@openbossa.org>

Reviewed by Andreas Kling.

[Qt] Autotest got missed in the handover of the QML WebView element
https://bugs.webkit.org/show_bug.cgi?id=41449

Importing WebView QML element test cases from Qt repository. I did some
style modifications and simplifications in the original code but hopefully
without changing the behavior.

Tests that do not pass are marked with QEXPECT_FAIL or QSKIP. The two major
issues currently are:

  • pixelCache() test checked the usage of pixel cache by using a subclass of QDeclarativeWebView. We can't do that right now because this class is not exported. We may need a Q_AUTOTEST_EXPORT thing for QtWebKit if we want this kind of test.
  • elementAtArea() test uses a function that is not exported. But in this case I think we should test it's user, the public method "heuristicZoom".
  • tests/qdeclarativewebview/resources/basic.html: Added.
  • tests/qdeclarativewebview/resources/basic.png: Added.
  • tests/qdeclarativewebview/resources/basic.qml: Added.
  • tests/qdeclarativewebview/resources/elements.html: Added.
  • tests/qdeclarativewebview/resources/elements.qml: Added.
  • tests/qdeclarativewebview/resources/forward.html: Added.
  • tests/qdeclarativewebview/resources/forward.png: Added.
  • tests/qdeclarativewebview/resources/javaScript.html: Added.
  • tests/qdeclarativewebview/resources/javaScript.qml: Added.
  • tests/qdeclarativewebview/resources/loadError.qml: Added.
  • tests/qdeclarativewebview/resources/newwindows.html: Added.
  • tests/qdeclarativewebview/resources/newwindows.qml: Added.
  • tests/qdeclarativewebview/resources/propertychanges.qml: Added.
  • tests/qdeclarativewebview/resources/sethtml.qml: Added.
  • tests/qdeclarativewebview/tst_qdeclarativewebview.cpp: (tst_QDeclarativeWebView::tmpDir): (strippedHtml): (fileContents): (removeRecursive): (tst_QDeclarativeWebView::cleanupTestCase): (tst_QDeclarativeWebView::basicProperties): (tst_QDeclarativeWebView::elementAreaAt): (tst_QDeclarativeWebView::historyNav): (callEvaluateJavaScript): (tst_QDeclarativeWebView::javaScript): (tst_QDeclarativeWebView::loadError): (tst_QDeclarativeWebView::multipleWindows): (tst_QDeclarativeWebView::newWindowComponent): (tst_QDeclarativeWebView::newWindowParent): (tst_QDeclarativeWebView::pressGrabTime): (tst_QDeclarativeWebView::renderingEnabled): (tst_QDeclarativeWebView::setHtml): (tst_QDeclarativeWebView::settings): (tst_QDeclarativeWebView::checkNoErrors):
  • tests/qdeclarativewebview/tst_qdeclarativewebview.qrc:
7:25 PM Changeset in webkit [84099] by abarth@webkit.org
  • 3 edits
    1 add in trunk/Source/WebCore

2011-04-16 Adam Barth <abarth@webkit.org>

Reviewed by Sam Weinig.

about:blank documents in new tabs can XHR anywhere
https://bugs.webkit.org/show_bug.cgi?id=58712

Empty security origins have supposed to be low-privilege, we should
mark them as having a unique origin.

  • manual-tests/about-blank-xhr.html: Added.
  • page/SecurityOrigin.cpp: (WebCore::SecurityOrigin::SecurityOrigin):
  • page/SecurityOrigin.h:
5:20 PM Changeset in webkit [84098] by mitz@apple.com
  • 5 edits in trunk/LayoutTests

Updated results for fast/block/float/032.html after r84096. Filed http://webkit.org/b/58736 to
track this change.

  • platform/chromium-win/fast/block/float/032-expected.txt:
  • platform/gtk/fast/block/float/032-expected.txt:
  • platform/mac/fast/block/float/032-expected.txt:
  • platform/qt/fast/block/float/032-expected.txt:
3:51 PM Changeset in webkit [84097] by kbalazs@webkit.org
  • 2 edits in trunk/LayoutTests

Unreviewed.

[Qt][WK2] Unskip fast/loader/null-request-after-willSendRequest.html since
it is passing for a long time.

  • platform/qt-wk2/Skipped:
2:59 PM Changeset in webkit [84096] by mitz@apple.com
  • 6 edits
    4 adds in trunk

<rdar://problem/9190108> Crash when hiding a float

Reviewed by Simon Fraser.

Source/WebCore:

Test: fast/block/float/overhanging-tall-block.html

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::addOverhangingFloats): Moved the call to childLogicalTop() out of the loop.
Capped the logical bottom so that we get the correct maximum.

  • rendering/RenderBlock.h: Decleared appendFloatingObjectToLastLine().
  • rendering/RenderBlockLineLayout.cpp:

(WebCore::RenderBlock::appendFloatingObjectToLastLine): Added. Ensures correct bookkeeping by
extending the float if needed so that it touches the line.
(WebCore::RenderBlock::layoutInlineChildren): Changed to call appendFloatingObjectToLastLine().
(WebCore::RenderBlock::checkFloatsInCleanLine): Capped the float height so the we mark the right
range of lines as dirty.

  • rendering/RootInlineBox.h:

(WebCore::RootInlineBox::appendFloat): Replaced the floats() accessor with this function, which
allows the creation of the vector to be combined with appending the first float.

LayoutTests:

  • fast/block/float/overhanging-tall-block.html: Added.
  • platform/mac/fast/block/float/overhanging-tall-block-expected.checksum: Added.
  • platform/mac/fast/block/float/overhanging-tall-block-expected.png: Added.
  • platform/mac/fast/block/float/overhanging-tall-block-expected.txt: Added.
2:22 PM Changeset in webkit [84095] by andersca@apple.com
  • 2 edits in trunk/Source/WebKit2

2011-04-16 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

Crash when NPP_Write returns -1 when writing the manual stream
https://bugs.webkit.org/show_bug.cgi?id=58735
<rdar://problem/9124993>

If NPP_Write returns -1 we must cancel the stream.

  • WebProcess/Plugins/Netscape/NetscapePluginStream.cpp: (WebKit::NetscapePluginStream::deliverDataToPlugin):
2:19 PM Changeset in webkit [84094] by weinig@apple.com
  • 6 edits in trunk/Source/WebKit/win

2011-04-16 Sam Weinig <sam@webkit.org>

Reviewed by Simon Fraser.

Windows should use WebCore localized strings
<rdar://problem/9119405>
https://bugs.webkit.org/show_bug.cgi?id=58733

  • WebCoreSupport/WebEditorClient.cpp: (undoNameForEditAction): (WebEditorClient::registerCommandForUndo):
  • WebPreferences.cpp: (WebPreferences::initializeDefaultSettings):
  • WebURLResponse.cpp: (CFHTTPMessageCopyLocalizedShortDescriptionForStatusCode): (WebURLResponse::localizedStringForStatusCode): Change to use WEB_UI_STRING.
  • WebCoreSupport/WebPlatformStrategies.cpp:
  • WebCoreSupport/WebPlatformStrategies.h: Remove the localization strategy in favor of using the default one.
2:15 PM Changeset in webkit [84093] by Simon Fraser
  • 1 edit
    3 adds in trunk/LayoutTests

2011-04-16 Simon Fraser <Simon Fraser>

Fix border drawing tests to drop -webkit prefix on border-radius, and make the tests compatible with Firefox
https://bugs.webkit.org/show_bug.cgi?id=58734

Add result for new test.

  • platform/mac/fast/borders/webkit-border-radius-expected.checksum: Added.
  • platform/mac/fast/borders/webkit-border-radius-expected.png: Added.
  • platform/mac/fast/borders/webkit-border-radius-expected.txt: Added.
2:13 PM Changeset in webkit [84092] by Simon Fraser
  • 34 edits
    1 add in trunk/LayoutTests

2011-04-16 Simon Fraser <Simon Fraser>

Reviewed by Sam Weinig.

Fix border drawing tests to drop -webkit prefix on border-radius, and make the tests compatible with Firefox
https://bugs.webkit.org/show_bug.cgi?id=58734

Drop the -webkit prefix on border-radius in the tests, and remove -moz-border-radius,
since Firefox supports the unprefixed property now. This avoids the problem where
-moz-border-radius and -webkit-border-radius had different behaviors when given two values.

Also unprefix box-sizing for WebKit, and add a -moz prefixed version for Firefox.

Add a test that compares the behavior of -webkit-border-radius and border-radius with
two values.

  • fast/borders/border-radius-circle.html:
  • fast/borders/border-radius-constraints.html:
  • fast/borders/border-radius-groove-01.html:
  • fast/borders/border-radius-groove-02.html:
  • fast/borders/border-radius-groove-03.html:
  • fast/borders/border-radius-wide-border-01.html:
  • fast/borders/border-radius-wide-border-02.html:
  • fast/borders/border-radius-wide-border-03.html:
  • fast/borders/border-radius-wide-border-04.html:
  • fast/borders/borderRadiusAllStylesAllCorners.html:
  • fast/borders/borderRadiusArcs01.html:
  • fast/borders/borderRadiusDashed01.html:
  • fast/borders/borderRadiusDashed02.html:
  • fast/borders/borderRadiusDashed03.html:
  • fast/borders/borderRadiusDashed04.html:
  • fast/borders/borderRadiusDashed05.html:
  • fast/borders/borderRadiusDashed06.html:
  • fast/borders/borderRadiusDotted01.html:
  • fast/borders/borderRadiusDotted02.html:
  • fast/borders/borderRadiusDotted03.html:
  • fast/borders/borderRadiusDotted04.html:
  • fast/borders/borderRadiusDotted05.html:
  • fast/borders/borderRadiusDotted06.html:
  • fast/borders/borderRadiusDouble02.html:
  • fast/borders/borderRadiusDouble03.html:
  • fast/borders/borderRadiusDouble04.html:
  • fast/borders/borderRadiusDouble05.html:
  • fast/borders/borderRadiusDouble06.html:
  • fast/borders/borderRadiusDouble07.html:
  • fast/borders/borderRadiusDouble08.html:
  • fast/borders/borderRadiusDouble09.html:
  • fast/borders/borderRadiusGroove01.html:
  • fast/borders/borderRadiusGroove02.html:
  • fast/borders/webkit-border-radius.html: Added.
  • platform/mac/fast/borders/border-radius-groove-01-expected.checksum:
  • platform/mac/fast/borders/border-radius-groove-01-expected.png:
  • platform/mac/fast/borders/border-radius-groove-02-expected.png:
  • platform/mac/fast/borders/border-radius-groove-03-expected.checksum:
  • platform/mac/fast/borders/border-radius-groove-03-expected.png:
  • platform/mac/fast/borders/border-radius-inline-flow-expected.png:
  • platform/mac/fast/borders/border-radius-split-inline-expected.png:
  • platform/mac/fast/borders/border-radius-wide-border-01-expected.png:
  • platform/mac/fast/borders/border-radius-wide-border-02-expected.png:
  • platform/mac/fast/borders/border-radius-wide-border-03-expected.checksum:
  • platform/mac/fast/borders/border-radius-wide-border-03-expected.png:
  • platform/mac/fast/borders/border-radius-wide-border-04-expected.checksum:
  • platform/mac/fast/borders/border-radius-wide-border-04-expected.png:
  • platform/mac/fast/borders/borderRadiusDashed01-expected.png:
  • platform/mac/fast/borders/borderRadiusDashed02-expected.png:
  • platform/mac/fast/borders/borderRadiusDashed03-expected.png:
  • platform/mac/fast/borders/borderRadiusDashed04-expected.png:
  • platform/mac/fast/borders/borderRadiusDashed05-expected.png:
  • platform/mac/fast/borders/borderRadiusDashed06-expected.checksum:
  • platform/mac/fast/borders/borderRadiusDashed06-expected.png:
  • platform/mac/fast/borders/borderRadiusDotted01-expected.png:
  • platform/mac/fast/borders/borderRadiusDotted02-expected.png:
  • platform/mac/fast/borders/borderRadiusDotted03-expected.png:
  • platform/mac/fast/borders/borderRadiusDotted04-expected.png:
  • platform/mac/fast/borders/borderRadiusDotted05-expected.png:
  • platform/mac/fast/borders/borderRadiusDotted06-expected.checksum:
  • platform/mac/fast/borders/borderRadiusDotted06-expected.png:
  • platform/mac/fast/borders/borderRadiusDouble01-expected.png:
  • platform/mac/fast/borders/borderRadiusDouble02-expected.png:
  • platform/mac/fast/borders/borderRadiusDouble03-expected.png:
  • platform/mac/fast/borders/borderRadiusDouble04-expected.png:
  • platform/mac/fast/borders/borderRadiusDouble05-expected.checksum:
  • platform/mac/fast/borders/borderRadiusDouble05-expected.png:
  • platform/mac/fast/borders/borderRadiusDouble06-expected.checksum:
  • platform/mac/fast/borders/borderRadiusDouble06-expected.png:
  • platform/mac/fast/borders/borderRadiusDouble07-expected.checksum:
  • platform/mac/fast/borders/borderRadiusDouble07-expected.png:
  • platform/mac/fast/borders/borderRadiusDouble08-expected.checksum:
  • platform/mac/fast/borders/borderRadiusDouble08-expected.png:
  • platform/mac/fast/borders/borderRadiusDouble09-expected.checksum:
  • platform/mac/fast/borders/borderRadiusDouble09-expected.png:
  • platform/mac/fast/borders/borderRadiusGroove01-expected.png:
  • platform/mac/fast/borders/borderRadiusGroove02-expected.png:
  • platform/mac/fast/borders/borderRadiusInset01-expected.png:
  • platform/mac/fast/borders/borderRadiusInvalidColor-expected.png:
  • platform/mac/fast/borders/borderRadiusOutset01-expected.png:
  • platform/mac/fast/borders/borderRadiusRidge01-expected.png:
  • platform/mac/fast/borders/borderRadiusSolid01-expected.png:
  • platform/mac/fast/borders/borderRadiusSolid02-expected.png:
  • platform/mac/fast/borders/borderRadiusSolid03-expected.png:
12:31 PM Changeset in webkit [84091] by weinig@apple.com
  • 9 edits in trunk/Source

2011-04-16 Sam Weinig <sam@webkit.org>

Reviewed by Simon Fraser.

Pages in the PageCache don't have the correct visited link coloring after being restored
https://bugs.webkit.org/show_bug.cgi?id=58721

  • WebCore.exp.in: Add export of markPagesForVistedLinkStyleRecalc for use by WebKit2.
  • history/CachedPage.cpp: (WebCore::CachedPage::CachedPage): (WebCore::CachedPage::restore): (WebCore::CachedPage::clear):
  • history/CachedPage.h: (WebCore::CachedPage::markForVistedLinkStyleRecalc): Add bit, set by calling markForVistedLinkStyleRecalc, which forces a visited link style recalc when being restored.
  • history/PageCache.h:
  • history/PageCache.cpp: (WebCore::PageCache::markPagesForVistedLinkStyleRecalc): Mark all pages in the page cache as requiring visited link style recalc.
  • page/PageGroup.cpp: (WebCore::PageGroup::addVisitedLink): (WebCore::PageGroup::removeVisitedLinks): (WebCore::PageGroup::removeAllVisitedLinks): Set dirty bit when changing any visited link information.


2011-04-16 Sam Weinig <sam@webkit.org>

Reviewed by Simon Fraser.

Pages in the PageCache don't have the correct visited link coloring after being restored
https://bugs.webkit.org/show_bug.cgi?id=58721

  • WebProcess/WebProcess.cpp: (WebKit::WebProcess::visitedLinkStateChanged): (WebKit::WebProcess::allVisitedLinkStateChanged): Mark all pages in the page cache as needing visited link style recalc whenever visited link information changes.
11:37 AM Changeset in webkit [84090] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk

2011-04-16 Leo Yang <leoyang.webkit@gmail.com>

Reviewed by Nikolas Zimmermann.

ASSERT failure when svg <use> element changes href
https://bugs.webkit.org/show_bug.cgi?id=58726

Test case to verify assertion at line 509 of SVGUseElement.cpp.
Test passes if no crash occurs in debug mode.

  • svg/custom/use-crash-when-href-change-expected.txt: Added.
  • svg/custom/use-crash-when-href-change.svg: Added.

2011-04-16 Leo Yang <leoyang.webkit@gmail.com>

Reviewed by Nikolas Zimmermann.

ASSERT failure when svg <use> element changes href
https://bugs.webkit.org/show_bug.cgi?id=58726

m_resourceId was not empty after the pending resource
becomes available. This will trigger the assertion at
line 509 of SVGUseElement.cpp when the <use> element's
href becomes empty or invalid.

This patch makes m_resourceId be empty after the pending
resource becomes available to avoid assertion failure.

Test: svg/custom/use-crash-when-href-change.svg

  • svg/SVGUseElement.cpp: (WebCore::SVGUseElement::buildPendingResource):
3:55 AM Changeset in webkit [84089] by Patrick Gansterer
  • 2 edits in trunk

2011-04-16 Patrick Gansterer <Patrick Gansterer>

Reviewed by Eric Seidel.

Rename PLATFORM(CAIRO) to USE(CAIRO)
https://bugs.webkit.org/show_bug.cgi?id=55192

  • Source/cmake/OptionsEfl.cmake:
3:37 AM Changeset in webkit [84088] by Patrick Gansterer
  • 41 edits in trunk

2011-04-16 Patrick Gansterer <Patrick Gansterer>

Reviewed by Eric Seidel.

Rename PLATFORM(CAIRO) to USE(CAIRO)
https://bugs.webkit.org/show_bug.cgi?id=55192

  • wtf/Platform.h:
  • wtf/gobject/GTypedefs.h:

2011-04-16 Patrick Gansterer <Patrick Gansterer>

Reviewed by Eric Seidel.

Rename PLATFORM(CAIRO) to USE(CAIRO)
https://bugs.webkit.org/show_bug.cgi?id=55192

  • CMakeListsEfl.txt:
  • config.h:
  • platform/MIMETypeRegistry.cpp: (WebCore::initializeSupportedImageMIMETypesForEncoding):
  • platform/graphics/ContextShadow.h:
  • platform/graphics/DashArray.h:
  • platform/graphics/FloatRect.h:
  • platform/graphics/FontPlatformData.h: (WebCore::FontPlatformData::FontPlatformData): (WebCore::FontPlatformData::hash): (WebCore::FontPlatformData::isHashTableDeletedValue):
  • platform/graphics/GlyphBuffer.h: (WebCore::GlyphBuffer::glyphAt): (WebCore::GlyphBuffer::add):
  • platform/graphics/Gradient.cpp:
  • platform/graphics/Gradient.h:
  • platform/graphics/GraphicsContext.cpp:
  • platform/graphics/GraphicsContext.h: (WebCore::GraphicsContextState::GraphicsContextState):
  • platform/graphics/GraphicsContext3D.h:
  • platform/graphics/ImageBufferData.h:
  • platform/graphics/ImageSource.h:
  • platform/graphics/Path.h:
  • platform/graphics/Pattern.h:
  • platform/graphics/SimpleFontData.h:
  • platform/graphics/cairo/DrawErrorUnderline.h:
  • platform/graphics/cairo/GraphicsContextCairo.cpp:
  • platform/graphics/cairo/ImageCairo.cpp:
  • platform/graphics/gstreamer/ImageGStreamer.h:
  • platform/graphics/transforms/AffineTransform.h:
  • platform/graphics/transforms/TransformationMatrix.h:
  • platform/graphics/win/FontCacheWin.cpp: (WebCore::createGDIFont): (WebCore::FontCache::createFontPlatformData):
  • platform/graphics/win/FontPlatformDataWin.cpp: (WebCore::FontPlatformData::FontPlatformData):
  • platform/graphics/win/GraphicsContextWin.cpp:
  • plugins/win/PluginViewWin.cpp: (WebCore::PluginView::paintWindowedPluginIntoContext):
  • rendering/RenderObject.h:

2011-04-16 Patrick Gansterer <Patrick Gansterer>

Reviewed by Eric Seidel.

Rename PLATFORM(CAIRO) to USE(CAIRO)
https://bugs.webkit.org/show_bug.cgi?id=55192

  • WebFrame.cpp: (WebFrame::spoolPages):
  • WebFrame.h:

2011-04-16 Patrick Gansterer <Patrick Gansterer>

Reviewed by Eric Seidel.

Rename PLATFORM(CAIRO) to USE(CAIRO)
https://bugs.webkit.org/show_bug.cgi?id=55192

  • DumpRenderTree/PixelDumpSupport.cpp:
  • DumpRenderTree/config.h:
  • DumpRenderTree/win/PixelDumpSupportWin.cpp: (createBitmapContextFromWebView):
  • WebKitTestRunner/config.h:
3:12 AM Changeset in webkit [84087] by Patrick Gansterer
  • 2 edits in trunk/Source/WebCore

2011-04-16 Patrick Gansterer <Patrick Gansterer>

Unreviewed WinCE build fix for r84033.

  • platform/win/LocalizedStringsWin.cpp: (WebCore::localizedString):
2:33 AM Changeset in webkit [84086] by ojan@chromium.org
  • 2 edits in trunk/Tools

2011-04-16 Ojan Vafai <ojan@chromium.org>

Reviewed by Eric Seidel.

show diffs+test inline in the results page
https://bugs.webkit.org/show_bug.cgi?id=58723

Put each row in it's own tbody and sort based on tbodies. That way
we can append the results to the tbody inline as another table row.

  • Scripts/webkitpy/layout_tests/layout_package/json_results.html:
2:02 AM Changeset in webkit [84085] by inferno@chromium.org
  • 4 edits
    2 adds in trunk

2011-04-16 Abhishek Arya <inferno@chromium.org>

Reviewed by Nikolas Zimmermann.

Tests that we do not crash when adding a list item block child
as a before child to svg text.
https://bugs.webkit.org/show_bug.cgi?id=58678

  • svg/text/text-block-child-crash-expected.txt: Added.
  • svg/text/text-block-child-crash.xhtml: Added.

2011-04-16 Abhishek Arya <inferno@chromium.org>

Reviewed by Nikolas Zimmermann.

Only allow inline child for SVG text since we cannot
handle block childs.
https://bugs.webkit.org/show_bug.cgi?id=58678

Test: svg/text/text-block-child-crash.xhtml

  • rendering/svg/RenderSVGText.cpp: (WebCore::RenderSVGText::isChildAllowed):
  • rendering/svg/RenderSVGText.h:
1:47 AM Changeset in webkit [84084] by sergio@webkit.org
  • 46 edits in trunk/LayoutTests

2011-04-16 Sergio Villar Senin <svillar@igalia.com>

Unreviewed, new GTK+ test expectations after r84039.

  • platform/gtk/fast/html/details-add-summary-1-and-click-expected.txt:
  • platform/gtk/fast/html/details-add-summary-1-expected.txt:
  • platform/gtk/fast/html/details-add-summary-10-and-click-expected.txt:
  • platform/gtk/fast/html/details-add-summary-10-expected.txt:
  • platform/gtk/fast/html/details-add-summary-2-and-click-expected.txt:
  • platform/gtk/fast/html/details-add-summary-2-expected.txt:
  • platform/gtk/fast/html/details-add-summary-3-and-click-expected.txt:
  • platform/gtk/fast/html/details-add-summary-3-expected.txt:
  • platform/gtk/fast/html/details-add-summary-4-and-click-expected.txt:
  • platform/gtk/fast/html/details-add-summary-4-expected.txt:
  • platform/gtk/fast/html/details-add-summary-5-and-click-expected.txt:
  • platform/gtk/fast/html/details-add-summary-5-expected.txt:
  • platform/gtk/fast/html/details-add-summary-6-and-click-expected.txt:
  • platform/gtk/fast/html/details-add-summary-6-expected.txt:
  • platform/gtk/fast/html/details-add-summary-7-and-click-expected.txt:
  • platform/gtk/fast/html/details-add-summary-7-expected.txt:
  • platform/gtk/fast/html/details-add-summary-8-and-click-expected.txt:
  • platform/gtk/fast/html/details-add-summary-8-expected.txt:
  • platform/gtk/fast/html/details-add-summary-9-and-click-expected.txt:
  • platform/gtk/fast/html/details-add-summary-9-expected.txt:
  • platform/gtk/fast/html/details-no-summary1-expected.txt:
  • platform/gtk/fast/html/details-no-summary2-expected.txt:
  • platform/gtk/fast/html/details-no-summary3-expected.txt:
  • platform/gtk/fast/html/details-no-summary4-expected.txt:
  • platform/gtk/fast/html/details-open-javascript-expected.txt:
  • platform/gtk/fast/html/details-open1-expected.txt:
  • platform/gtk/fast/html/details-open2-expected.txt:
  • platform/gtk/fast/html/details-open3-expected.txt:
  • platform/gtk/fast/html/details-open4-expected.txt:
  • platform/gtk/fast/html/details-open5-expected.txt:
  • platform/gtk/fast/html/details-open6-expected.txt:
  • platform/gtk/fast/html/details-position-expected.txt:
  • platform/gtk/fast/html/details-remove-summary-1-and-click-expected.txt:
  • platform/gtk/fast/html/details-remove-summary-1-expected.txt:
  • platform/gtk/fast/html/details-remove-summary-2-and-click-expected.txt:
  • platform/gtk/fast/html/details-remove-summary-2-expected.txt:
  • platform/gtk/fast/html/details-remove-summary-3-and-click-expected.txt:
  • platform/gtk/fast/html/details-remove-summary-3-expected.txt:
  • platform/gtk/fast/html/details-remove-summary-4-and-click-expected.txt:
  • platform/gtk/fast/html/details-remove-summary-4-expected.txt:
  • platform/gtk/fast/html/details-remove-summary-5-and-click-expected.txt:
  • platform/gtk/fast/html/details-remove-summary-5-expected.txt:
  • platform/gtk/fast/html/details-remove-summary-6-and-click-expected.txt:
  • platform/gtk/fast/html/details-remove-summary-6-expected.txt:
  • platform/gtk/fast/html/details-writing-mode-expected.txt:
12:28 AM Changeset in webkit [84083] by ojan@chromium.org
  • 2 edits in trunk/Tools

2011-04-16 Ojan Vafai <ojan@chromium.org>

Reviewed by Eric Seidel.

handle new tests in the new results html file
https://bugs.webkit.org/show_bug.cgi?id=58715

  • Scripts/webkitpy/layout_tests/layout_package/json_results.html:

Apr 15, 2011:

11:22 PM Changeset in webkit [84082] by dbates@webkit.org
  • 2 edits in trunk/Source/WebKit2

2011-04-15 Daniel Bates <dbates@webkit.org>

Attempt to fix the Qt Linux Release build after changeset 84064 <http://trac.webkit.org/changeset/84064>
(https://bugs.webkit.org/show_bug.cgi?id=58686).

  • UIProcess/API/qt/qwkpage_p.h: Remove takeFocus().
10:41 PM Changeset in webkit [84081] by loislo@chromium.org
  • 4 edits in trunk/Source/JavaScriptCore

2011-04-15 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r84067.
http://trac.webkit.org/changeset/84067
https://bugs.webkit.org/show_bug.cgi?id=58724

qt build are failing. (Requested by loislo2 on #webkit).

  • heap/MarkStack.h: (JSC::MarkStack::append):
  • runtime/JSValue.h:
  • runtime/WriteBarrier.h: (JSC::DeprecatedPtr::DeprecatedPtr): (JSC::DeprecatedPtr::get): (JSC::DeprecatedPtr::operator*): (JSC::DeprecatedPtr::operator->): (JSC::DeprecatedPtr::slot): (JSC::DeprecatedPtr::operator UnspecifiedBoolType*): (JSC::DeprecatedPtr::operator!): (JSC::operator==):
10:32 PM Changeset in webkit [84080] by loislo@chromium.org
  • 7 edits in trunk

2011-04-15 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: The list of Database entries is empty if the page opens a database just before Web Inspector.
https://bugs.webkit.org/show_bug.cgi?id=57833

  • http/tests/inspector-enabled/database-open.html:

2011-04-15 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: The list of Database entries is empty if the page opens a database just before Web Inspector.
https://bugs.webkit.org/show_bug.cgi?id=57833

  • inspector/Inspector.json:
  • inspector/InspectorDatabaseAgent.cpp: (WebCore::InspectorDatabaseAgent::didOpenDatabase): (WebCore::InspectorDatabaseAgent::InspectorDatabaseAgent): (WebCore::InspectorDatabaseAgent::setFrontend): (WebCore::InspectorDatabaseAgent::clearFrontend): (WebCore::InspectorDatabaseAgent::enable): (WebCore::InspectorDatabaseAgent::disable): (WebCore::InspectorDatabaseAgent::getDatabaseTableNames): (WebCore::InspectorDatabaseAgent::executeSQL):
  • inspector/InspectorDatabaseAgent.h:
  • inspector/front-end/inspector.js:
9:40 PM Changeset in webkit [84079] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit/chromium

2011-04-15 John Bates <jbates@google.com>

Reviewed by Kenneth Russell.

Change WebGraphicsContext3D virtuals to pure virtuals as intended
https://bugs.webkit.org/show_bug.cgi?id=58479

  • public/WebGraphicsContext3D.h:
9:35 PM Changeset in webkit [84078] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit/mac

2011-04-15 Jia Pu <jpu@apple.com>

Reviewed by Dan Bernstein.

[Mac] WebKit needs to convert the bounding box of autocorrected word to view coordinate.
https://bugs.webkit.org/show_bug.cgi?id=58717

Convert the bounding box from window coordinate to view coordinate, which is expected by NSCorrectionPanel.

  • WebCoreSupport/CorrectionPanel.mm: (CorrectionPanel::show):
9:23 PM Changeset in webkit [84077] by abarth@webkit.org
  • 5 edits
    6 adds in trunk

2011-04-15 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

CSP media-src is missing
https://bugs.webkit.org/show_bug.cgi?id=58642

These tests are adapted from the other media loading tests.

  • http/tests/security/contentSecurityPolicy/media-src-allowed-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/media-src-allowed.html: Added.
  • http/tests/security/contentSecurityPolicy/media-src-blocked-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/media-src-blocked.html: Added.

2011-04-15 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

CSP media-src is missing
https://bugs.webkit.org/show_bug.cgi?id=58642

Yay HTMLMediaElement for having the exact hook we need!

Tests: http/tests/security/contentSecurityPolicy/media-src-allowed.html

http/tests/security/contentSecurityPolicy/media-src-blocked.html

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::isSafeToLoadURL):
    • Although it's tempting to add this branch to the previous if-block, that results in the wrong error message being logged to the console (covered by the "blocked" test).
  • page/ContentSecurityPolicy.cpp: (WebCore::ContentSecurityPolicy::allowMediaFromSource): (WebCore::ContentSecurityPolicy::addDirective):
  • page/ContentSecurityPolicy.h:
    • These changes are routine.
8:59 PM Changeset in webkit [84076] by dpranke@chromium.org
  • 2 edits in trunk/Tools

2011-04-15 Dirk Pranke <dpranke@chromium.org>

Reviewed by Ojan Vafai.

old-run-webkit-tests: save the list of tests actually run to a file
https://bugs.webkit.org/show_bug.cgi?id=58692

  • Scripts/old-run-webkit-tests:
8:40 PM Changeset in webkit [84075] by dpranke@chromium.org
  • 2 edits in trunk/Tools

2011-04-15 Dirk Pranke <dpranke@chromium.org>

Reviewed by Ojan Vafai.

fix MockFileSystem.abspath() on windows
https://bugs.webkit.org/show_bug.cgi?id=58713

The existing code was borked on Windows if a path was passed
in using a drive letter (e.g., "d:\foo.txt"), and would go
into an infinite recursive loop. While win paths shouldn't
normally be passed into the mock filesystem, this just
bulletproofs things a bit (note that "/foo.txt" is considered
an absolute path on windows python as well.

  • Scripts/webkitpy/common/system/filesystem_mock.py:
8:02 PM Changeset in webkit [84074] by ojan@chromium.org
  • 3 edits in trunk/Tools

2011-04-15 Ojan Vafai <ojan@chromium.org>

Reviewed by Eric Seidel.

show stderr files and httpd log files
https://bugs.webkit.org/show_bug.cgi?id=58714

  • Scripts/webkitpy/layout_tests/layout_package/json_results.html:
  • Scripts/webkitpy/layout_tests/layout_package/test_runner.py:
7:33 PM Changeset in webkit [84073] by abarth@webkit.org
  • 9 edits
    8 adds in trunk

2011-04-15 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

CSP should block string arguments to setTimeout and setInterval unless options eval-script
https://bugs.webkit.org/show_bug.cgi?id=58610

Herein lies a four-way testing matrix.

  • http/tests/security/contentSecurityPolicy/eval-scripts-setInterval-allowed-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/eval-scripts-setInterval-allowed.html: Added.
  • http/tests/security/contentSecurityPolicy/eval-scripts-setInterval-blocked-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/eval-scripts-setInterval-blocked.html: Added.
  • http/tests/security/contentSecurityPolicy/eval-scripts-setTimeout-allowed-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/eval-scripts-setTimeout-allowed.html: Added.
  • http/tests/security/contentSecurityPolicy/eval-scripts-setTimeout-blocked-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/eval-scripts-setTimeout-blocked.html: Added.

2011-04-15 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

CSP should block string arguments to setTimeout and setInterval unless options eval-script
https://bugs.webkit.org/show_bug.cgi?id=58610

It's somewhat sadness that the JSC and V8 code for setTimeout and
setInterval are so different. I struggled for a while with how to
handle the worker case, but I decided to punt on it for now.

Tests: http/tests/security/contentSecurityPolicy/eval-scripts-setInterval-allowed.html

http/tests/security/contentSecurityPolicy/eval-scripts-setInterval-blocked.html
http/tests/security/contentSecurityPolicy/eval-scripts-setTimeout-allowed.html
http/tests/security/contentSecurityPolicy/eval-scripts-setTimeout-blocked.html

  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::setTimeout): (WebCore::JSDOMWindow::setInterval):
  • bindings/js/JSWorkerContextCustom.cpp: (WebCore::JSWorkerContext::setTimeout): (WebCore::JSWorkerContext::setInterval):
  • bindings/js/ScheduledAction.cpp: (WebCore::ScheduledAction::create):
  • bindings/js/ScheduledAction.h:
  • bindings/v8/custom/V8DOMWindowCustom.cpp: (WebCore::WindowSetTimeoutImpl):
  • page/ContentSecurityPolicy.cpp: (WebCore::ContentSecurityPolicy::allowEval):
  • page/ContentSecurityPolicy.h:
7:26 PM Changeset in webkit [84072] by commit-queue@webkit.org
  • 16 edits in trunk

2011-04-15 Shishir Agrawal <shishir@chromium.org>

Reviewed by James Robinson.

Add a flag to guard Page Visibility API changes.
https://bugs.webkit.org/show_bug.cgi?id=58464

Initial draft at:
http://lists.w3.org/Archives/Public/public-webapps/2011JanMar/0175.html

  • configure.ac:

2011-04-15 Shishir Agrawal <shishir@chromium.org>

Reviewed by James Robinson.

Add a flag to guard Page Visibility API changes.
https://bugs.webkit.org/show_bug.cgi?id=58464

  • Configurations/FeatureDefines.xcconfig:

2011-04-15 Shishir Agrawal <shishir@chromium.org>

Reviewed by James Robinson.

Add a flag to guard Page Visibility API changes.
https://bugs.webkit.org/show_bug.cgi?id=58464

  • Configurations/FeatureDefines.xcconfig:

2011-04-15 Shishir Agrawal <shishir@chromium.org>

Reviewed by James Robinson.

Add a flag to guard Page Visibility API changes.
https://bugs.webkit.org/show_bug.cgi?id=58464

  • features.gypi:

2011-04-15 Shishir Agrawal <shishir@chromium.org>

Reviewed by James Robinson.

Add a flag to guard Page Visibility API changes.
https://bugs.webkit.org/show_bug.cgi?id=58464

  • Configurations/FeatureDefines.xcconfig:

2011-04-15 Shishir Agrawal <shishir@chromium.org>

Reviewed by James Robinson.

Add a flag to guard Page Visibility API changes.
https://bugs.webkit.org/show_bug.cgi?id=58464

  • Configurations/FeatureDefines.xcconfig:

2011-04-15 Shishir Agrawal <shishir@chromium.org>

Reviewed by James Robinson.

Add a flag to guard Page Visibility API changes.
https://bugs.webkit.org/show_bug.cgi?id=58464

  • Scripts/build-webkit:

2011-04-15 Shishir Agrawal <shishir@chromium.org>

Reviewed by James Robinson.

Add a flag to guard Page Visibility API changes.
https://bugs.webkit.org/show_bug.cgi?id=58464

  • win/tools/vsprops/FeatureDefines.vsprops:
7:06 PM Changeset in webkit [84071] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebCore

2011-04-15 Fridrich Strba <fridrich.strba@bluewin.ch>

Reviewed by Martin Robinson.

Make plugins compile during WebKit GTK Windows build.
Ifdef properly relevant parts so that the PluginViewGtk compiles on Windows.
https://bugs.webkit.org/show_bug.cgi?id=58580

  • plugins/PluginView.cpp: (WebCore::PluginView::stop):
  • plugins/PluginView.h:
  • plugins/gtk/PluginViewGtk.cpp: (WebCore::PluginView::platformGetValue): (WebCore::PluginView::platformStart):
6:53 PM Changeset in webkit [84070] by commit-queue@webkit.org
  • 5 edits in trunk

2011-04-15 Chang Shu <cshu@webkit.org>

Reviewed by Alexey Proskuryakov.

When a message with url embedded is added to console, the "file:" scheme
and path should be stripped.
https://bugs.webkit.org/show_bug.cgi?id=58665

Unskip passed tests.

  • platform/mac-wk2/Skipped:
  • platform/qt-wk2/Skipped:

2011-04-15 Chang Shu <cshu@webkit.org>

Reviewed by Alexey Proskuryakov.

When a message with url embedded is added to console, the "file:" scheme
and path should be stripped.
https://bugs.webkit.org/show_bug.cgi?id=58665

  • WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp: (WTR::lastFileURLPathComponent): (WTR::InjectedBundlePage::willAddMessageToConsole):
6:19 PM Changeset in webkit [84069] by barraclough@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Errrk! - build fix from !x86-64.

  • dfg/DFGNode.h:
6:16 PM Changeset in webkit [84068] by levin@chromium.org
  • 2 edits
    2 deletes in trunk/Source/JavaScriptCore

Revert of r83974.

JavaScriptCore shouldn't depend on ../ThirdParty/gtest/xcode/gtest.xcodeproj
https://bugs.webkit.org/show_bug.cgi?id=58716

  • JavaScriptCore.xcodeproj/project.pbxproj:
  • wtf/tests/RunAllWtfTests.cpp: Removed.
  • wtf/tests/StringTests.cpp: Removed.
6:11 PM Changeset in webkit [84067] by oliver@apple.com
  • 4 edits in trunk/Source/JavaScriptCore

2011-04-15 Oliver Hunt <oliver@apple.com>

Reviewed by Gavin Barraclough.

Remove DeprecatedPtr
https://bugs.webkit.org/show_bug.cgi?id=58718

As simple as it sounds.

  • heap/MarkStack.h: (JSC::MarkStack::append):
  • runtime/JSValue.h:
  • runtime/WriteBarrier.h:
5:49 PM Changeset in webkit [84066] by morrita@google.com
  • 6 edits in trunk/Source/WebCore

2011-04-15 MORITA Hajime <morrita@google.com>

Reviewed by Simon Fraser.

Calls to WebCore::Document::mayCauseFlashOfUnstyledContent make frequently-run drawing methods slower
https://bugs.webkit.org/show_bug.cgi?id=58512

Manually rolled out r72367 at which mayCauseFlashOfUnstyledContent() was introduced.
This change keeps an small refactoring on
FrameView::shouldUpdate(), which doesn't affect on the behavior.

  • dom/Document.cpp:
  • dom/Document.h:
  • page/FrameView.cpp: (WebCore::FrameView::invalidateRect): (WebCore::FrameView::shouldUpdate):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::paintContents):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::paintLayer):
5:42 PM Changeset in webkit [84065] by crogers@google.com
  • 2 edits in trunk/Source/WebCore

2011-04-15 Chris Rogers <crogers@google.com>

Reviewed by Kenneth Russell.

Reduce default kernel size of SincResampler to favor better speed over quality
https://bugs.webkit.org/show_bug.cgi?id=58710

No new tests since audio API is not yet implemented.

  • platform/audio/SincResampler.h:
5:38 PM Changeset in webkit [84064] by jeffm@apple.com
  • 22 edits in trunk

2011-04-15 Jeff Miller <jeffm@apple.com>

Reviewed by Sam Weinig.

Add takeFocus callback to WKPageUIClient
https://bugs.webkit.org/show_bug.cgi?id=58686

On Windows, we need to handle moving focus out of the web view in the client, so add a takeFocus callback to WKPageUIClient
and stop handling taking focus in the framework in WKView.mm on the Mac.

  • UIProcess/API/C/WKPage.h: Added WKFocusDirection enum and takeFocus to WKPageUIClient.
  • UIProcess/API/mac/PageClientImpl.h: Removed takeFocus().
  • UIProcess/API/mac/PageClientImpl.mm: Removed takeFocus().
  • UIProcess/API/mac/WKView.mm: Removed _takeFocus().
  • UIProcess/API/mac/WKViewInternal.h: Removed _takeFocus().
  • UIProcess/API/qt/ClientImpl.cpp: (qt_wk_takeFocus): Added.
  • UIProcess/API/qt/ClientImpl.h: Added qt_wk_takeFocus.
  • UIProcess/API/qt/qwkpage.cpp: Remove takeFocus(). (QWKPage::QWKPage): Added takeFocus() callback.
  • UIProcess/PageClient.h: Removed takeFocus().
  • UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::takeFocus): Call m_uiClient to take focus.
  • UIProcess/WebPageProxy.h: Pass FocusDirection as a uint32_t to takeFocus().
  • UIProcess/WebPageProxy.messages.in: Pass FocusDirection as a uint32_t in TakeFocus message.
  • UIProcess/WebUIClient.cpp: (WebKit::WebUIClient::takeFocus): Added.
  • UIProcess/WebUIClient.h: Added takeFocus().
  • UIProcess/win/WebView.cpp: Removed takeFocus(), which was never implemented on Windows.
  • UIProcess/win/WebView.h: Removed takeFocus().
  • WebProcess/WebCoreSupport/WebChromeClient.cpp: (WebKit::WebChromeClient::takeFocus): Pass FocusDirection as a uint32_t in TakeFocus message.

2011-04-15 Jeff Miller <jeffm@apple.com>

Reviewed by Sam Weinig.

Add takeFocus callback to WKPageUIClient
https://bugs.webkit.org/show_bug.cgi?id=58686

Add empty WKPageUIClient entries for the takeFocus callback in the MiniBrowser and WebKitTestRunner.

  • MiniBrowser/mac/BrowserWindowController.m: (-[BrowserWindowController awakeFromNib]): Added empty takeFocus entry.
  • MiniBrowser/win/BrowserView.cpp: (BrowserView::create): Added empty takeFocus entry.
  • WebKitTestRunner/TestController.cpp: (WTR::TestController::createOtherPage): Added empty takeFocus entry. (WTR::TestController::initialize): Added empty takeFocus entry.
5:37 PM Changeset in webkit [84063] by barraclough@apple.com
  • 5 edits in trunk/Source/JavaScriptCore

Add a simple tool to gather statistics on whether functions
are completed through the new or old JIT.

Reviewed by Oliver Hunt.

  • dfg/DFGNode.h:
  • dfg/DFGNonSpeculativeJIT.cpp:

(JSC::DFG::NonSpeculativeJIT::compile):

  • dfg/DFGSpeculativeJIT.cpp:

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

  • jit/JIT.cpp:

(JSC::JIT::privateCompile):

5:33 PM Changeset in webkit [84062] by ap@apple.com
  • 2 edits in trunk/Source/WebKit2

Reviewed by Sam Weinig.

WebKit2: Stop using deprecated NSInputManager
https://bugs.webkit.org/show_bug.cgi?id=58709

Changed all mouse related functions to give NSInputContext a chance to handle the event,
added logging. Removed -rightMouseMoved: method, which was added by accident.

  • UIProcess/API/mac/WKView.mm: (-[WKView mouseMoved:]): (-[WKView mouseDown:]): (-[WKView mouseUp:]): (-[WKView mouseDragged:]):
5:26 PM Changeset in webkit [84061] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-04-15 Fridrich Strba <fridrich.strba@bluewin.ch>

Reviewed by Martin Robinson.

Expand the ntohs, ntohl, htons and htonl defines for OS(WINDOWS)
These have to be defined for Windows in general, since win32 API
does not provide them.
https://bugs.webkit.org/show_bug.cgi?id=58582

  • platform/graphics/WOFFFileFormat.cpp:
5:21 PM Changeset in webkit [84060] by commit-queue@webkit.org
  • 6 edits in trunk

2011-04-15 Vsevolod Vlasov <vsevik@chromium.org>

Reviewed by Adam Barth.

Show a console message when X-Frame-Options blocks a load
https://bugs.webkit.org/show_bug.cgi?id=39087

Added console message when X-Frame-Options headers block resource loading

  • http/tests/inspector/network/x-frame-options-deny-expected.txt:
  • http/tests/security/XFrameOptions/x-frame-options-deny-expected.txt:
  • http/tests/security/XFrameOptions/x-frame-options-parent-same-origin-deny-expected.txt:

2011-04-15 Vsevolod Vlasov <vsevik@chromium.org>

Reviewed by Adam Barth.

Show a console message when X-Frame-Options blocks a load
https://bugs.webkit.org/show_bug.cgi?id=39087

Added console message when X-Frame-Options headers block resource loading

  • loader/MainResourceLoader.cpp: (WebCore::MainResourceLoader::didReceiveResponse):
5:17 PM Changeset in webkit [84059] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-04-15 Fridrich Strba <fridrich.strba@bluewin.ch>

Reviewed by Martin Robinson.

Evaluate PLATFORM(GTK) before the all-encompassing OS(WINDOWS)
test. This allows Windows build of WebKit GTK to chose the right
path.
https://bugs.webkit.org/show_bug.cgi?id=58576

  • platform/FileSystem.h:
5:14 PM Changeset in webkit [84058] by jhoneycutt@apple.com
  • 2 edits in trunk/Source/WebKit2

Potential crash getting a JS wrapper.

<rdar://problem/8988741>

Reviewed by Brian Weinstein.

  • WebProcess/WebPage/WebFrame.cpp:

(WebKit::WebFrame::jsWrapperForWorld):
Ensure that we have a valid m_coreFrame, because it can be cleared by
invalidate().

5:05 PM Changeset in webkit [84057] by luiz@webkit.org
  • 15 edits
    2 copies
    27 adds in trunk

[Qt] QWebPage MIME type handling inconsistency with other web browsers
https://bugs.webkit.org/show_bug.cgi?id=46968

Source/WebCore:

Reviewed by Kenneth Rohde Christiansen.

Implementing mime type sniffing based on
http://tools.ietf.org/html/draft-abarth-mime-sniff-06.

  • WebCore.pro:
  • platform/network/MIMESniffing.cpp: Added.

(MagicNumbers::dataSizeNeededForImageSniffing):
(MagicNumbers::maskedCompare):
(MagicNumbers::checkSpaceOrBracket):
(MagicNumbers::compare):
(MagicNumbers::findMIMEType):
(MagicNumbers::findSimpleMIMEType):
(MagicNumbers::textOrBinaryTypeSniffingProcedure):
(MagicNumbers::unknownTypeSniffingProcedure):
(MagicNumbers::imageTypeSniffingProcedure):
(MagicNumbers::checkText):
(MagicNumbers::checkRDF):
(MagicNumbers::skipTag):
(MagicNumbers::feedTypeSniffingProcedure):
(MIMESniffer::MIMESniffer):

  • platform/network/MIMESniffing.h: Added.

(MIMESniffer::dataSize):
(MIMESniffer::sniff):
(MIMESniffer::isValid):

  • platform/network/NetworkingContext.h:
  • platform/network/qt/QNetworkReplyHandler.cpp:

(WebCore::QNetworkReplyWrapper::QNetworkReplyWrapper):
(WebCore::QNetworkReplyWrapper::release):
(WebCore::QNetworkReplyWrapper::receiveMetaData):
(WebCore::QNetworkReplyWrapper::receiveSniffedMIMEType):
(WebCore::QNetworkReplyWrapper::emitMetaDataChanged):
(WebCore::QNetworkReplyHandler::sendResponseIfNeeded):
(WebCore::QNetworkReplyHandler::sendNetworkRequest):
(WebCore::QNetworkReplyHandler::start):

  • platform/network/qt/QNetworkReplyHandler.h:

(WebCore::QNetworkReplyWrapper::advertisedMIMEType):
(WebCore::QNetworkReplyWrapper::mimeType):

  • platform/network/qt/QtMIMETypeSniffer.cpp: Added.

(QtMIMETypeSniffer::QtMIMETypeSniffer):
(QtMIMETypeSniffer::sniff):
(QtMIMETypeSniffer::trySniffing):

  • platform/network/qt/QtMIMETypeSniffer.h: Added.

(QtMIMETypeSniffer::mimeType):
(QtMIMETypeSniffer::isFinished):

Source/WebKit/qt:

Reviewed by Kenneth Rohde Christiansen.

Implementing mime type sniffing based on
http://tools.ietf.org/html/draft-abarth-mime-sniff-06.

  • WebCoreSupport/FrameLoaderClientQt.cpp:

(WebCore::FrameLoaderClientQt::createNetworkingContext):

  • WebCoreSupport/FrameNetworkingContextQt.cpp:

(WebCore::FrameNetworkingContextQt::FrameNetworkingContextQt):
(WebCore::FrameNetworkingContextQt::create):
(WebCore::FrameNetworkingContextQt::MIMESniffingEnabled):

  • WebCoreSupport/FrameNetworkingContextQt.h:
  • tests/MIMESniffing/MIMESniffing.pro: Added.
  • tests/MIMESniffing/TestData.h: Added.
  • tests/MIMESniffing/resources.qrc: Added.
  • tests/MIMESniffing/resources/application_atom+xml: Added.
  • tests/MIMESniffing/resources/application_ogg: Added.
  • tests/MIMESniffing/resources/application_pdf: Added.
  • tests/MIMESniffing/resources/application_postscript: Added.
  • tests/MIMESniffing/resources/application_rdf+xml: Added.
  • tests/MIMESniffing/resources/application_rss+xml: Added.
  • tests/MIMESniffing/resources/application_x-gzip: Added.
  • tests/MIMESniffing/resources/application_x-rar-compressed: Added.
  • tests/MIMESniffing/resources/application_zip: Added.
  • tests/MIMESniffing/resources/audio_x-wave: Added.
  • tests/MIMESniffing/resources/image_bmp: Added.
  • tests/MIMESniffing/resources/image_gif: Added.
  • tests/MIMESniffing/resources/image_jpeg: Added.
  • tests/MIMESniffing/resources/image_png: Added.
  • tests/MIMESniffing/resources/image_vnd.microsoft.icon: Added.
  • tests/MIMESniffing/resources/image_webp: Added.
  • tests/MIMESniffing/resources/text_html: Added.
  • tests/MIMESniffing/resources/text_xml: Added.
  • tests/MIMESniffing/resources/video_webm: Added.
  • tests/MIMESniffing/tst_MIMESniffing.cpp: Added.

(tst_MIMESniffing::tst_MIMESniffing):
(errorText):
(tst_MIMESniffing::testCase1):

  • tests/tests.pro:

Source/WebKit2:

Reviewed by Kenneth Rohde Christiansen.

Implementing mime type sniffing based on
http://tools.ietf.org/html/draft-abarth-mime-sniff-06.

  • WebProcess/WebCoreSupport/qt/WebFrameNetworkingContext.cpp:

(WebCore::WebFrameNetworkingContext::WebFrameNetworkingContext):
(WebCore::WebFrameNetworkingContext::MIMESniffingEnabled):

  • WebProcess/WebCoreSupport/qt/WebFrameNetworkingContext.h:

LayoutTests:

Rubberstamped by Andreas Kling.

Unskipping http/tests/loading/text-content-type-with-binary-extension.html.

  • platform/qt/Skipped:
5:03 PM Changeset in webkit [84056] by eae@chromium.org
  • 3 edits
    2 adds in trunk

2011-04-15 Emil A Eklund <eae@chromium.org>

Reviewed by Dimitri Glazkov.

input field with focus makes appendChild operation ~42x slower
https://bugs.webkit.org/show_bug.cgi?id=57059

Add performance test for cloneNode when an input field has focus.

  • perf/clone-with-focus-expected.txt: Added.
  • perf/clone-with-focus.html: Added.

2011-04-15 Emil A Eklund <eae@chromium.org>

Reviewed by Dimitri Glazkov.

input field with focus makes appendChild operation ~42x slower
https://bugs.webkit.org/show_bug.cgi?id=57059

Change ContainerNode::cloneChildNodes to only disable the
deleteButtonController if the container itself (or any of its children)
is being edited. Thus avoiding a reflow in cases where it's not.

Test: perf/clone-with-focus.html

  • dom/ContainerNode.cpp: (WebCore::ContainerNode::cloneChildNodes):
5:01 PM Changeset in webkit [84055] by commit-queue@webkit.org
  • 4 edits in trunk

2011-04-15 Charlie Reis <creis@chromium.org>

Reviewed by Darin Fisher.

[Chromium] fast/events/popup-allowed-from-gesture-initiated-form-submit.html causing debug ASSERT
https://bugs.webkit.org/show_bug.cgi?id=44079

Remove an assert whose invariant doesn't hold.

  • platform/chromium/test_expectations.txt: Re-enable test.

2011-04-15 Charlie Reis <creis@chromium.org>

Reviewed by Darin Fisher.

[Chromium] fast/events/popup-allowed-from-gesture-initiated-form-submit.html causing debug ASSERT
https://bugs.webkit.org/show_bug.cgi?id=44079

Remove an assert whose invariant doesn't hold.

  • src/FrameLoaderClientImpl.cpp:
4:59 PM Changeset in webkit [84054] by ojan@chromium.org
  • 4 edits in trunk/Tools

2011-04-15 Ojan Vafai <ojan@chromium.org>

Reviewed by Eric Seidel.

only show pretty-diff/wdiff if they were generated
https://bugs.webkit.org/show_bug.cgi?id=58682

  • Scripts/webkitpy/layout_tests/layout_package/json_results.html:
  • Scripts/webkitpy/layout_tests/layout_package/test_runner.py:
  • Scripts/webkitpy/layout_tests/port/base.py:
4:58 PM Changeset in webkit [84053] by kinuko@chromium.org
  • 23 edits
    8 adds in trunk

2011-04-15 Kinuko Yasuda <kinuko@chromium.org>

Reviewed by David Levin.

Expose unified Quota API if QUOTA build flag is enabled
https://bugs.webkit.org/show_bug.cgi?id=58648

  • src/AssertMatchingEnums.cpp: Added enum assertions.
  • src/StorageInfoChromium.cpp: Made a build fix.

2011-04-15 Kinuko Yasuda <kinuko@chromium.org>

Reviewed by David Levin.

Expose unified Quota API if QUOTA build flag is enabled
https://bugs.webkit.org/show_bug.cgi?id=58648

Test: storage/storageinfo-query-usage.html

  • CMakeLists.txt: Added new file entries.
  • DerivedSources.cpp: Added new file entries.
  • DerivedSources.make: Added new file entries.
  • GNUmakefile.am: Added new file entries.
  • WebCore.gypi: Added new file entries.
  • WebCore.vcproj/WebCore.vcproj: Added new file entries.
  • WebCore.xcodeproj/project.pbxproj: Added new file entries.
  • page/DOMWindow.cpp: (WebCore::DOMWindow::webkitStorageInfo): Added.
  • page/DOMWindow.h:
  • page/DOMWindow.idl:
  • storage/StorageInfo.h: Added storage type enum.
  • storage/StorageInfo.idl: Added.
  • storage/StorageInfoErrorCallback.idl: Added.
  • storage/StorageInfoUsageCallback.idl: Added.

2011-04-15 Kinuko Yasuda <kinuko@chromium.org>

Reviewed by David Levin.

Expose unified Quota API if QUOTA build flag is enabled
https://bugs.webkit.org/show_bug.cgi?id=58648

Added a layout test, also added skipped entries on platforms
where the QUOTA flag is not enabled.

  • platform/chromium/test_expectations.txt:
  • platform/gtk/Skipped:
  • platform/mac/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:
  • storage/script-tests/TEMPLATE.html: Added.
  • storage/script-tests/storageinfo-query-usage.js: Added.
  • storage/storageinfo-query-usage-expected.txt: Added.
  • storage/storageinfo-query-usage.html: Added.
4:55 PM Changeset in webkit [84052] by oliver@apple.com
  • 191 edits in trunk

2011-04-15 Oliver Hunt <oliver@apple.com>

GC allocate Structure
https://bugs.webkit.org/show_bug.cgi?id=58483

Rolling r83894 r83827 r83810 r83809 r83808 back in with
a workaround for the gcc bug seen by the gtk bots

4:51 PM Changeset in webkit [84051] by jberlin@webkit.org
  • 2 edits in trunk/Source/WebCore

Hang underneath ApplicationCacheStorage::writeDataToUniqueFileInDirectory when loading
http://www.webkit.org/demos/calendar.
https://bugs.webkit.org/show_bug.cgi?id=58698

Reviewed by Brian Weinstein.

  • platform/win/FileSystemWin.cpp:

(WebCore::openFile):
Add breaks to the case statement.
(WebCore::directoryName):
Remove any trailing slash in directoryName. After closer inspection of all callers, it
appears the that callers do not need the trailing slash, and those that use the length do
so only to change the direction of the slashes throughout the length of the path.

4:47 PM Changeset in webkit [84050] by rolandsteiner@chromium.org
  • 11 edits in trunk/Source/WebCore

2011-04-15 Roland Steiner <rolandsteiner@chromium.org>

Reviewed by Dimitri Glazkov.

Bug 52963 - Enable O(1) access to root from any node in shadow DOM subtree
https://bugs.webkit.org/show_bug.cgi?id=52963

.) Change base class of ShadowRoot from DocumentFragment to TreeScope.
.) Re-enable tree scope handling in Node (had ASSERT_NOT_REACHED, etc.).
.) Merged setTreeScope() with setTreeScopeRecursively()
.) Call setTreeScopeRecursively in DOM manipulation functions where applicable.

No new tests. (refactoring)

  • dom/ContainerNode.cpp: (WebCore::ContainerNode::takeAllChildrenFrom): (WebCore::ContainerNode::removeBetween): (WebCore::ContainerNode::removeChildren): (WebCore::ContainerNode::parserAddChild):
  • dom/Document.cpp: (WebCore::Document::Document): (WebCore::Document::~Document): (WebCore::Document::setDocType):
  • dom/Element.h:
  • dom/Node.cpp: (WebCore::Node::treeScope): (WebCore::Node::setTreeScopeRecursively):
  • dom/Node.h: (WebCore::Node::document):
  • dom/ShadowRoot.cpp: (WebCore::ShadowRoot::ShadowRoot): (WebCore::ShadowRoot::~ShadowRoot): (WebCore::ShadowRoot::nodeType): (WebCore::ShadowRoot::cloneNode): (WebCore::ShadowRoot::childTypeAllowed):
  • dom/ShadowRoot.h: (WebCore::toShadowRoot):
  • dom/TreeScope.cpp: (WebCore::TreeScope::TreeScope): (WebCore::TreeScope::setParentTreeScope):
  • dom/TreeScope.h:
  • rendering/RenderSlider.cpp:
4:38 PM Changeset in webkit [84049] by ggaren@apple.com
  • 6 edits in trunk/Source/WebCore

2011-04-15 Geoffrey Garen <ggaren@apple.com>

Reviewed by Oliver Hunt.

DOM object handles are never removed from cache
https://bugs.webkit.org/show_bug.cgi?id=58707

We were trying to remove hash table items by value instead of by key.

  • bindings/js/DOMWrapperWorld.cpp: (WebCore::JSNodeHandleOwner::finalize): Changed to work more like DOMObjectHandleOwner::finalize because I'm going to merge them.

(WebCore::DOMObjectHandleOwner::finalize): Remove hash table items
by key, not value. (Oops!) Use a helper function to make sure we get
this right.

  • bindings/js/JSDOMBinding.cpp: (WebCore::cacheDOMObjectWrapper): Store the hash table key as our weak handle context, so we can use it at destruction time.
  • bindings/js/JSDOMBinding.h: Removed unnecessary include.
  • bindings/js/JSNodeCustom.h: (WebCore::cacheDOMNodeWrapper): Store the hash table key as our weak handle context, so we can use it at destruction time.
  • bindings/js/ScriptWrappable.h: (WebCore::ScriptWrappable::setWrapper): Forward context parameter, to support the above.
4:35 PM Changeset in webkit [84048] by commit-queue@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

2011-04-15 Fridrich Strba <fridrich.strba@bluewin.ch>

Reviewed by Gavin Barraclough.

Correctly prefix symbols. Since gcc 4.5.0, Windows x64 symbols
are not prefixed by underscore anymore. This is consistent with
what MSVC does.
https://bugs.webkit.org/show_bug.cgi?id=58573

  • jit/JITStubs.cpp:
4:33 PM WebKit Team edited by dcheng@chromium.org
(diff)
4:10 PM Changeset in webkit [84047] by xan@webkit.org
  • 2 edits in trunk

2011-04-15 Xan Lopez <xlopez@igalia.com>

Reviewed by Martin Robinson.

  • configure.ac: bump to 1.5.0, 1.4.0 will be released from a branch and was already cut off.
4:03 PM Changeset in webkit [84046] by kbr@google.com
  • 2 edits in trunk/Source/WebCore

2011-04-15 Kenneth Russell <kbr@google.com>

Unreviewed. Chromium Linux Release build fix due to unused variables.

  • platform/audio/mkl/FFTFrameMKL.cpp: (WebCore::FFTFrame::doFFT): (WebCore::FFTFrame::doInverseFFT): (WebCore::FFTFrame::cleanup):
4:02 PM Changeset in webkit [84045] by barraclough@apple.com
  • 10 edits in trunk/Source/JavaScriptCore

Bug 58705 - DFG JIT Add support for flow control (branch, jump).

Reviewed by Geoff Garen.

Add support for control flow by breaking the CodeBlock up into multiple
basic blocks, generating code for each basic block in turn through the
speculative JIT & then the non-speculative JIT.

  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::setTemporary):
(JSC::DFG::ByteCodeParser::addToGraph):
(JSC::DFG::ByteCodeParser::parseBlock):
(JSC::DFG::ByteCodeParser::parse):

  • dfg/DFGGraph.cpp:

(JSC::DFG::Graph::dump):

  • dfg/DFGGraph.h:

(JSC::DFG::BasicBlock::BasicBlock):
(JSC::DFG::BasicBlock::getBytecodeOffset):
(JSC::DFG::Graph::blockIndexForBytecodeOffset):

  • dfg/DFGJITCodeGenerator.h:

(JSC::DFG::JITCodeGenerator::JITCodeGenerator):
(JSC::DFG::JITCodeGenerator::addBranch):
(JSC::DFG::JITCodeGenerator::linkBranches):
(JSC::DFG::JITCodeGenerator::BranchRecord::BranchRecord):

  • dfg/DFGNode.h:

(JSC::DFG::Node::Node):
(JSC::DFG::Node::isJump):
(JSC::DFG::Node::isBranch):
(JSC::DFG::Node::takenBytecodeOffset):
(JSC::DFG::Node::notTakenBytecodeOffset):

  • dfg/DFGNonSpeculativeJIT.cpp:

(JSC::DFG::NonSpeculativeJIT::compile):

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

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

  • dfg/DFGSpeculativeJIT.h:
3:56 PM Changeset in webkit [84044] by bfulgham@webkit.org
  • 2 edits in trunk/Source/WebCore

Unreviewed build change after r83945.

New 'PlatformPathCairo.cpp' was not added to the WinCairo build.

  • WebCore.vcproj/WebCore.vcproj: Add missing file to WinCairo build. Exclude for standard Apple build.
3:54 PM Changeset in webkit [84043] by andersca@apple.com
  • 3 edits in trunk/Source/WebKit2

Add API to get the bundle page overlay fade in fraction.

Reviewed by Sam Weinig.

  • WebProcess/InjectedBundle/API/c/WKBundlePageOverlay.cpp:

(WKBundlePageOverlayFractionFadedIn):

  • WebProcess/InjectedBundle/API/c/WKBundlePageOverlay.h:
3:44 PM April 2011 Meeting edited by morrita@google.com
(diff)
3:31 PM Changeset in webkit [84042] by andersca@apple.com
  • 5 edits in trunk/Source

2011-04-15 Anders Carlsson <andersca@apple.com>

Reviewed by Dan Bernstein.

Fade the find page overlay
https://bugs.webkit.org/show_bug.cgi?id=58697

Add a symbol that WebKit2 needs.

  • WebCore.exp.in:

2011-04-15 Anders Carlsson <andersca@apple.com>

Reviewed by Dan Bernstein.

Fade the find page overlay
https://bugs.webkit.org/show_bug.cgi?id=58697

  • WebProcess/WebPage/FindController.cpp: Make the color components floats.

(WebKit::overlayBackgroundColor):
(WebKit::holeShadowColor):
(WebKit::holeFillColor):
Add helper functions for returning the colors given the fraction faded in.

(WebKit::FindController::drawRect):
Use the new helper functions.

  • WebProcess/WebPage/WebPage.cpp: (WebKit::WebPage::installPageOverlay): Start the fade animation unless we're replacing an already existing page overlay with another.
3:30 PM Changeset in webkit [84041] by barraclough@apple.com
  • 7 edits in trunk/Source/JavaScriptCore

Bug 58701 - DFG JIT - add GetLocal/SetLocal nodes

Reviewed by Geoff Garen.

Use these for both access to arguments & local variables, adds ability
to set locals, such that values will persist between basic blocks.

  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::ByteCodeParser):
(JSC::DFG::ByteCodeParser::get):
(JSC::DFG::ByteCodeParser::set):
(JSC::DFG::ByteCodeParser::getVariable):
(JSC::DFG::ByteCodeParser::setVariable):
(JSC::DFG::ByteCodeParser::getArgument):
(JSC::DFG::ByteCodeParser::setArgument):
(JSC::DFG::ByteCodeParser::getThis):
(JSC::DFG::ByteCodeParser::setThis):
(JSC::DFG::ByteCodeParser::VariableRecord::VariableRecord):
(JSC::DFG::ByteCodeParser::parseBlock):

  • dfg/DFGGraph.cpp:

(JSC::DFG::Graph::dump):
(JSC::DFG::Graph::derefChildren):

  • dfg/DFGGraph.h:

(JSC::DFG::Graph::ref):
(JSC::DFG::Graph::deref):

  • dfg/DFGNode.h:

(JSC::DFG::Node::hasLocal):
(JSC::DFG::Node::local):

  • dfg/DFGNonSpeculativeJIT.cpp:

(JSC::DFG::NonSpeculativeJIT::compile):

  • dfg/DFGSpeculativeJIT.cpp:

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

3:17 PM Changeset in webkit [84040] by andreas.kling@nokia.com
  • 2 edits
    1 delete in trunk/LayoutTests

2011-04-15 Andreas Kling <kling@webkit.org>

Unreviewed, remove a Qt-specific expectation that's no longer needed.

  • platform/qt/Skipped:
  • platform/qt/fast/loader/onload-policy-ignore-for-frame-expected.txt: Removed.
2:56 PM Changeset in webkit [84039] by morrita@google.com
  • 154 edits
    2 adds
    4 deletes in trunk

2011-04-15 MORITA Hajime <morrita@google.com>

Reviewed by Dimitri Glazkov.

RenderDetailsMarker should belong to shadow element.
https://bugs.webkit.org/show_bug.cgi?id=58591

  • Render tree structure has changed.
  • Changed style rule on details-writing-mode.html to avoid to hit shadow CSS leak bug.
  • fast/html/details-writing-mode.html:
  • platform/chromium/test_expectations.txt:
  • platform/mac/fast/html/details-add-summary-1-and-click-expected.checksum:
  • platform/mac/fast/html/details-add-summary-1-and-click-expected.png:
  • platform/mac/fast/html/details-add-summary-1-and-click-expected.txt:
  • platform/mac/fast/html/details-add-summary-1-expected.checksum:
  • platform/mac/fast/html/details-add-summary-1-expected.png:
  • platform/mac/fast/html/details-add-summary-1-expected.txt:
  • platform/mac/fast/html/details-add-summary-10-and-click-expected.checksum:
  • platform/mac/fast/html/details-add-summary-10-and-click-expected.png:
  • platform/mac/fast/html/details-add-summary-10-and-click-expected.txt:
  • platform/mac/fast/html/details-add-summary-10-expected.checksum:
  • platform/mac/fast/html/details-add-summary-10-expected.png:
  • platform/mac/fast/html/details-add-summary-10-expected.txt:
  • platform/mac/fast/html/details-add-summary-2-and-click-expected.checksum:
  • platform/mac/fast/html/details-add-summary-2-and-click-expected.png:
  • platform/mac/fast/html/details-add-summary-2-and-click-expected.txt:
  • platform/mac/fast/html/details-add-summary-2-expected.checksum:
  • platform/mac/fast/html/details-add-summary-2-expected.png:
  • platform/mac/fast/html/details-add-summary-2-expected.txt:
  • platform/mac/fast/html/details-add-summary-3-and-click-expected.checksum:
  • platform/mac/fast/html/details-add-summary-3-and-click-expected.png:
  • platform/mac/fast/html/details-add-summary-3-and-click-expected.txt:
  • platform/mac/fast/html/details-add-summary-3-expected.checksum:
  • platform/mac/fast/html/details-add-summary-3-expected.png:
  • platform/mac/fast/html/details-add-summary-3-expected.txt:
  • platform/mac/fast/html/details-add-summary-4-and-click-expected.checksum:
  • platform/mac/fast/html/details-add-summary-4-and-click-expected.png:
  • platform/mac/fast/html/details-add-summary-4-and-click-expected.txt:
  • platform/mac/fast/html/details-add-summary-4-expected.checksum:
  • platform/mac/fast/html/details-add-summary-4-expected.png:
  • platform/mac/fast/html/details-add-summary-4-expected.txt:
  • platform/mac/fast/html/details-add-summary-5-and-click-expected.checksum:
  • platform/mac/fast/html/details-add-summary-5-and-click-expected.png:
  • platform/mac/fast/html/details-add-summary-5-and-click-expected.txt:
  • platform/mac/fast/html/details-add-summary-5-expected.checksum:
  • platform/mac/fast/html/details-add-summary-5-expected.png:
  • platform/mac/fast/html/details-add-summary-5-expected.txt:
  • platform/mac/fast/html/details-add-summary-6-and-click-expected.checksum:
  • platform/mac/fast/html/details-add-summary-6-and-click-expected.png:
  • platform/mac/fast/html/details-add-summary-6-and-click-expected.txt:
  • platform/mac/fast/html/details-add-summary-6-expected.checksum:
  • platform/mac/fast/html/details-add-summary-6-expected.png:
  • platform/mac/fast/html/details-add-summary-6-expected.txt:
  • platform/mac/fast/html/details-add-summary-7-and-click-expected.checksum:
  • platform/mac/fast/html/details-add-summary-7-and-click-expected.png:
  • platform/mac/fast/html/details-add-summary-7-and-click-expected.txt:
  • platform/mac/fast/html/details-add-summary-7-expected.checksum:
  • platform/mac/fast/html/details-add-summary-7-expected.png:
  • platform/mac/fast/html/details-add-summary-7-expected.txt:
  • platform/mac/fast/html/details-add-summary-8-and-click-expected.checksum:
  • platform/mac/fast/html/details-add-summary-8-and-click-expected.png:
  • platform/mac/fast/html/details-add-summary-8-and-click-expected.txt:
  • platform/mac/fast/html/details-add-summary-8-expected.checksum:
  • platform/mac/fast/html/details-add-summary-8-expected.png:
  • platform/mac/fast/html/details-add-summary-8-expected.txt:
  • platform/mac/fast/html/details-add-summary-9-and-click-expected.checksum:
  • platform/mac/fast/html/details-add-summary-9-and-click-expected.png:
  • platform/mac/fast/html/details-add-summary-9-and-click-expected.txt:
  • platform/mac/fast/html/details-add-summary-9-expected.checksum:
  • platform/mac/fast/html/details-add-summary-9-expected.png:
  • platform/mac/fast/html/details-add-summary-9-expected.txt:
  • platform/mac/fast/html/details-no-summary1-expected.checksum:
  • platform/mac/fast/html/details-no-summary1-expected.png:
  • platform/mac/fast/html/details-no-summary1-expected.txt:
  • platform/mac/fast/html/details-no-summary2-expected.checksum:
  • platform/mac/fast/html/details-no-summary2-expected.png:
  • platform/mac/fast/html/details-no-summary2-expected.txt:
  • platform/mac/fast/html/details-no-summary3-expected.checksum:
  • platform/mac/fast/html/details-no-summary3-expected.png:
  • platform/mac/fast/html/details-no-summary3-expected.txt:
  • platform/mac/fast/html/details-no-summary4-expected.checksum:
  • platform/mac/fast/html/details-no-summary4-expected.png:
  • platform/mac/fast/html/details-no-summary4-expected.txt:
  • platform/mac/fast/html/details-open-javascript-expected.checksum:
  • platform/mac/fast/html/details-open-javascript-expected.png:
  • platform/mac/fast/html/details-open-javascript-expected.txt:
  • platform/mac/fast/html/details-open1-expected.checksum:
  • platform/mac/fast/html/details-open1-expected.png:
  • platform/mac/fast/html/details-open1-expected.txt:
  • platform/mac/fast/html/details-open2-expected.checksum:
  • platform/mac/fast/html/details-open2-expected.png:
  • platform/mac/fast/html/details-open2-expected.txt:
  • platform/mac/fast/html/details-open3-expected.checksum:
  • platform/mac/fast/html/details-open3-expected.png:
  • platform/mac/fast/html/details-open3-expected.txt:
  • platform/mac/fast/html/details-open4-expected.checksum:
  • platform/mac/fast/html/details-open4-expected.png:
  • platform/mac/fast/html/details-open4-expected.txt:
  • platform/mac/fast/html/details-open5-expected.checksum:
  • platform/mac/fast/html/details-open5-expected.png:
  • platform/mac/fast/html/details-open5-expected.txt:
  • platform/mac/fast/html/details-open6-expected.checksum:
  • platform/mac/fast/html/details-open6-expected.png:
  • platform/mac/fast/html/details-open6-expected.txt:
  • platform/mac/fast/html/details-position-expected.checksum:
  • platform/mac/fast/html/details-position-expected.png:
  • platform/mac/fast/html/details-position-expected.txt:
  • platform/mac/fast/html/details-remove-summary-1-and-click-expected.checksum:
  • platform/mac/fast/html/details-remove-summary-1-and-click-expected.png:
  • platform/mac/fast/html/details-remove-summary-1-and-click-expected.txt:
  • platform/mac/fast/html/details-remove-summary-1-expected.checksum:
  • platform/mac/fast/html/details-remove-summary-1-expected.png:
  • platform/mac/fast/html/details-remove-summary-1-expected.txt:
  • platform/mac/fast/html/details-remove-summary-2-and-click-expected.checksum:
  • platform/mac/fast/html/details-remove-summary-2-and-click-expected.png:
  • platform/mac/fast/html/details-remove-summary-2-and-click-expected.txt:
  • platform/mac/fast/html/details-remove-summary-2-expected.checksum:
  • platform/mac/fast/html/details-remove-summary-2-expected.png:
  • platform/mac/fast/html/details-remove-summary-2-expected.txt:
  • platform/mac/fast/html/details-remove-summary-3-and-click-expected.checksum:
  • platform/mac/fast/html/details-remove-summary-3-and-click-expected.png:
  • platform/mac/fast/html/details-remove-summary-3-and-click-expected.txt:
  • platform/mac/fast/html/details-remove-summary-3-expected.checksum:
  • platform/mac/fast/html/details-remove-summary-3-expected.png:
  • platform/mac/fast/html/details-remove-summary-3-expected.txt:
  • platform/mac/fast/html/details-remove-summary-4-and-click-expected.checksum:
  • platform/mac/fast/html/details-remove-summary-4-and-click-expected.png:
  • platform/mac/fast/html/details-remove-summary-4-and-click-expected.txt:
  • platform/mac/fast/html/details-remove-summary-4-expected.checksum:
  • platform/mac/fast/html/details-remove-summary-4-expected.png:
  • platform/mac/fast/html/details-remove-summary-4-expected.txt:
  • platform/mac/fast/html/details-remove-summary-5-and-click-expected.checksum:
  • platform/mac/fast/html/details-remove-summary-5-and-click-expected.png:
  • platform/mac/fast/html/details-remove-summary-5-and-click-expected.txt:
  • platform/mac/fast/html/details-remove-summary-5-expected.checksum:
  • platform/mac/fast/html/details-remove-summary-5-expected.png:
  • platform/mac/fast/html/details-remove-summary-5-expected.txt:
  • platform/mac/fast/html/details-remove-summary-6-and-click-expected.checksum:
  • platform/mac/fast/html/details-remove-summary-6-and-click-expected.png:
  • platform/mac/fast/html/details-remove-summary-6-and-click-expected.txt:
  • platform/mac/fast/html/details-remove-summary-6-expected.checksum:
  • platform/mac/fast/html/details-remove-summary-6-expected.png:
  • platform/mac/fast/html/details-remove-summary-6-expected.txt:
  • platform/mac/fast/html/details-writing-mode-expected.checksum:
  • platform/mac/fast/html/details-writing-mode-expected.png:
  • platform/mac/fast/html/details-writing-mode-expected.txt:

2011-04-15 MORITA Hajime <morrita@google.com>

Reviewed by Dimitri Glazkov.

RenderDetailsMarker should belong to shadow element.
https://bugs.webkit.org/show_bug.cgi?id=58591

  • Introduced DetailsMarkerControl element, which is a shadow element of <summary>, creates RenderDetailsMarker.
  • Removed custom layout code from RenderDetails, RenderDetailsMarker, which is now done by usual CSS layout. Note that marker size is given via style for -webkit-details-marker pseudo class.
  • Converted default summary implementation from pure-renderer style to shadow of HTMLDetailsElement. Now RenderDetails knows nothing about default summary.
  • Moved event handling code from HTMLDetailsElement to HTMLSummaryElement because now the marker is always child of <summary>, and the clickable area is now <summary> itself.
  • Note that the rendering result is changed due to the conversion from custom layout code to usual CSS styling.
  • Android.mk:
  • CMakeLists.txt:
  • GNUmakefile.list.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • css/html.css: (summary::-webkit-details-marker):
  • dom/Element.cpp: (WebCore::Element::attach):
  • dom/Node.cpp: (WebCore::shouldCreateRendererFor):
  • dom/Node.h: (WebCore::Node::canHaveLightChildRendererWithShadow):
  • html/HTMLDetailsElement.cpp: (WebCore::HTMLDetailsElement::findSummaryFor): (WebCore::HTMLDetailsElement::findMainSummary): (WebCore::HTMLDetailsElement::refreshMainSummary): (WebCore::HTMLDetailsElement::createShadowSubtree): (WebCore::HTMLDetailsElement::childrenChanged): (WebCore::HTMLDetailsElement::finishParsingChildren): (WebCore::HTMLDetailsElement::toggleOpen):
  • html/HTMLDetailsElement.h: (WebCore::HTMLDetailsElement::canHaveLightChildRendererWithShadow):
  • html/HTMLSummaryElement.cpp: (WebCore::HTMLSummaryElement::create): (WebCore::HTMLSummaryElement::createShadowSubtree): (WebCore::HTMLSummaryElement::detailsElement): (WebCore::HTMLSummaryElement::isMainSummary): (WebCore::HTMLSummaryElement::defaultEventHandler):
  • html/HTMLSummaryElement.h: (WebCore::HTMLSummaryElement::canHaveLightChildRendererWithShadow):
  • html/shadow/DetailsMarkerControl.cpp: Added. (WebCore::DetailsMarkerControl::DetailsMarkerControl): (WebCore::DetailsMarkerControl::createRenderer): (WebCore::DetailsMarkerControl::rendererIsNeeded): (WebCore::DetailsMarkerControl::shadowPseudoId): (WebCore::DetailsMarkerControl::summaryElement):
  • html/shadow/DetailsMarkerControl.h: Added. (WebCore::DetailsMarkerControl::create):
  • rendering/RenderDetails.cpp: (WebCore::RenderDetails::RenderDetails): (WebCore::RenderDetails::styleDidChange): (WebCore::RenderDetails::moveSummaryToContents): (WebCore::RenderDetails::checkMainSummary): (WebCore::RenderDetails::layout):
  • rendering/RenderDetails.h:
  • rendering/RenderDetailsMarker.cpp: (WebCore::RenderDetailsMarker::RenderDetailsMarker): (WebCore::RenderDetailsMarker::isOpen): (WebCore::RenderDetailsMarker::getPath): (WebCore::RenderDetailsMarker::paint): (WebCore::RenderDetailsMarker::details):
  • rendering/RenderDetailsMarker.h:
2:52 PM Changeset in webkit [84038] by jianli@chromium.org
  • 1 edit
    2 copies in branches/chromium/696

Merge 83862 - 2011-04-14 Satish Sampath <satish@chromium.org>

Reviewed by Kent Tamura.

Disable speech input for readonly and disabled input fields.
https://bugs.webkit.org/show_bug.cgi?id=58540

  • fast/speech/input-readonly-and-disabled-expected.txt: Added.
  • fast/speech/input-readonly-and-disabled.html: Added.

2011-04-14 Satish Sampath <satish@chromium.org>

Reviewed by Kent Tamura.

Disable speech input for readonly and disabled input fields.
https://bugs.webkit.org/show_bug.cgi?id=58540

Test: fast/speech/input-readonly-and-disabled.html

  • html/shadow/TextControlInnerElements.cpp: (WebCore::InputFieldSpeechButtonElement::defaultEventHandler): (WebCore::InputFieldSpeechButtonElement::setRecognitionResult):

TBR=satish@chromium.org

2:45 PM Changeset in webkit [84037] by barraclough@apple.com
  • 4 edits in trunk/Source/JavaScriptCore

Bug 58696 - DFG JIT split handling of vars/temporaries

Reviewed by Sam Weinig.

Presently all callee registers are treated as having single block scope,
since the DFG JIT can only compile single block functions. In order to
expand the JIT to support control flow we will need to change to retaining
locals (but not temporaries) across basic block boundaries.

  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::ByteCodeParser):
(JSC::DFG::ByteCodeParser::get):
(JSC::DFG::ByteCodeParser::set):
(JSC::DFG::ByteCodeParser::getVariable):
(JSC::DFG::ByteCodeParser::setVariable):
(JSC::DFG::ByteCodeParser::getTemporary):
(JSC::DFG::ByteCodeParser::setTemporary):
(JSC::DFG::ByteCodeParser::getArgument):
(JSC::DFG::ByteCodeParser::getInt32Constant):
(JSC::DFG::ByteCodeParser::getDoubleConstant):
(JSC::DFG::ByteCodeParser::getJSConstant):
(JSC::DFG::ByteCodeParser::constantUndefined):
(JSC::DFG::ByteCodeParser::constantNull):
(JSC::DFG::ByteCodeParser::one):
(JSC::DFG::ByteCodeParser::parseBlock):
(JSC::DFG::ByteCodeParser::parse):
(JSC::DFG::parse):

  • dfg/DFGNode.h:
  • dfg/DFGScoreBoard.h:

(JSC::DFG::ScoreBoard::ScoreBoard):
(JSC::DFG::ScoreBoard::~ScoreBoard):
(JSC::DFG::ScoreBoard::allocate):
(JSC::DFG::ScoreBoard::use):

2:44 PM Changeset in webkit [84036] by andersca@apple.com
  • 2 edits in trunk/Source/WebKit2

Fix Windows build.

  • WebProcess/WebPage/PageOverlay.cpp:

(WebKit::PageOverlay::fadeAnimationTimerFired):

2:44 PM Changeset in webkit [84035] by enne@google.com
  • 19 edits
    4 adds in trunk/LayoutTests

2011-04-15 Adrienne Walker <enne@google.com>

[chromium] Unreviewed, update baselines for compositing changes.
https://bugs.webkit.org/show_bug.cgi?id=57113

  • platform/chromium-gpu-linux/compositing/geometry/fixed-position-expected.png:
  • platform/chromium-gpu-linux/compositing/geometry/horizontal-scroll-composited-expected.png:
  • platform/chromium-gpu-linux/compositing/geometry/vertical-scroll-composited-expected.png:
  • platform/chromium-gpu-linux/compositing/overflow/fixed-position-ancestor-clip-expected.png:
  • platform/chromium-gpu-linux/platform/chromium/compositing/huge-layer-rotated-expected.png: Added.
  • platform/chromium-gpu-linux/platform/chromium/compositing/layout-width-change-expected.png:
  • platform/chromium-gpu-mac/compositing/geometry/fixed-position-expected.png:
  • platform/chromium-gpu-mac/compositing/geometry/horizontal-scroll-composited-expected.png:
  • platform/chromium-gpu-mac/compositing/geometry/vertical-scroll-composited-expected.png:
  • platform/chromium-gpu-mac/compositing/overflow/fixed-position-ancestor-clip-expected.png:
  • platform/chromium-gpu-mac/compositing/tiling/huge-layer-img-expected.png: Added.
  • platform/chromium-gpu-mac/compositing/tiling/huge-layer-img-expected.txt: Added.
  • platform/chromium-gpu-mac/platform/chromium/compositing/huge-layer-rotated-expected.png:
  • platform/chromium-gpu-mac/platform/chromium/compositing/layout-width-change-expected.png:
  • platform/chromium-gpu-win/compositing/geometry/fixed-position-expected.png:
  • platform/chromium-gpu-win/compositing/geometry/horizontal-scroll-composited-expected.png:
  • platform/chromium-gpu-win/compositing/geometry/vertical-scroll-composited-expected.png:
  • platform/chromium-gpu-win/compositing/overflow/fixed-position-ancestor-clip-expected.png:
  • platform/chromium-gpu-win/compositing/tiling/huge-layer-img-expected.png: Added.
  • platform/chromium-gpu-win/platform/chromium/compositing/huge-layer-rotated-expected.png:
  • platform/chromium-gpu-win/platform/chromium/compositing/layout-width-change-expected.png:
  • platform/chromium/test_expectations.txt:
2:36 PM Changeset in webkit [84034] by andersca@apple.com
  • 7 edits in trunk/Source/WebKit2

2011-04-15 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

Add the ability for PageOverlays to fade in and out
https://bugs.webkit.org/show_bug.cgi?id=58694

  • WebProcess/InjectedBundle/API/c/WKBundlePage.cpp: (WKBundlePageUninstallPageOverlay): WebPage::uninstallPageOverlay now takes a boolean. Default to false for now.
  • WebProcess/WebPage/FindController.cpp: (WebKit::FindController::findString): Pass false to uninstallPageOverlay.

(WebKit::FindController::hideFindUI):
Pass true to uninstallPageOverlay.


  • WebProcess/WebPage/PageOverlay.cpp: (WebKit::PageOverlay::PageOverlay): Initialize new member variables.

(WebKit::PageOverlay::bounds):
Get rid of an unnecessary webPage() getter.

(WebKit::PageOverlay::setPage):
Stop the animation timer.

(WebKit::PageOverlay::startFadeInAnimation):
Update m_fractionFadedIn and call startFadeAnimation.

(WebKit::PageOverlay::startFadeOutAnimation):
Ditto.


(WebKit::PageOverlay::startFadeAnimation):
Initialize m_fadeAnimationStartTime and start the fade animation timer.

(WebKit::PageOverlay::fadeAnimationTimerFired):
Update m_fractionFadedIn and call setNeedsDisplay().

  • WebProcess/WebPage/PageOverlay.h:
  • WebProcess/WebPage/WebPage.cpp: (WebKit::WebPage::uninstallPageOverlay): If fadeOut is true, tell the page overlay to start the fade out animation. When the fade animation is complete, the page overlay will uninstall itself.
2:18 PM Changeset in webkit [84033] by weinig@apple.com
  • 2 edits in trunk/Source/WebCore

2011-04-15 Sam Weinig <sam@webkit.org>

Reviewed by Adam Roben.

Implement localize strings for windows WebKit2
https://bugs.webkit.org/show_bug.cgi?id=58688

  • platform/win/LocalizedStringsWin.cpp: (WebCore::createWebKitBundle): (WebCore::webKitBundle): (WebCore::localizedString): Add implementation of localizedString for Windows.
2:13 PM Changeset in webkit [84032] by msaboff@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Fixing build.

Inadvertantly checked in
Source/JavaScriptCore/Configurations/JavaScriptCore.xcconfig
instead of Source/JavaScriptCore/JavaScriptCore.exp

1:56 PM Changeset in webkit [84031] by andreas.kling@nokia.com
  • 2 edits in trunk/LayoutTests

2011-04-15 Andreas Kling <kling@webkit.org>

Skip failing tests after r84010 and r84012.

  • platform/qt/Skipped:
1:51 PM Changeset in webkit [84030] by msaboff@apple.com
  • 4 edits in trunk/Source/JavaScriptCore

2011-04-15 Michael Saboff <msaboff@apple.com>

Reviewed by Oliver Hunt.

globalObject moved to JSObjectWithGlobalObject.cpp inhibits inlining
https://bugs.webkit.org/show_bug.cgi?id=58677

Moved JSObjectWithGlobalObject::globalObject() to
runtime/JSObjectWithGlobalObject.h to allow the compiler to inline
it for a performance benefit. An equivalent instance had been in
a header file before r60057.

  • JavaScriptCore.exp:
  • runtime/JSObjectWithGlobalObject.cpp:
  • runtime/JSObjectWithGlobalObject.h: (JSC::JSObjectWithGlobalObject::globalObject):
1:47 PM Changeset in webkit [84029] by ggaren@apple.com
  • 19 edits in trunk/Source/WebCore

2011-04-15 Geoffrey Garen <ggaren@apple.com>

Reviewed by Oliver Hunt.

Some mechanical DOM wrapper cleanup
https://bugs.webkit.org/show_bug.cgi?id=58689

  • WebCore.exp.in: Export!
  • bindings/js/DOMWrapperWorld.cpp: (WebCore::isReachableFromDOM): Inverted the inDocument test to make the relationship of the special cases to the normal case clearer.
  • bindings/js/JSArrayBufferViewHelper.h: (WebCore::toJSArrayBufferView):
  • bindings/js/JSCSSRuleCustom.cpp: (WebCore::toJS):
  • bindings/js/JSCSSValueCustom.cpp: (WebCore::toJS):
  • bindings/js/JSDOMBinding.cpp: (WebCore::getCachedDOMObjectWrapper): (WebCore::cacheDOMObjectWrapper):
  • bindings/js/JSDOMBinding.h: (WebCore::createDOMObjectWrapper): (WebCore::getDOMObjectWrapper): (WebCore::createDOMNodeWrapper): (WebCore::getDOMNodeWrapper): Changed DOM wrapper functions to operate in terms of DOMWrapperWorlds instead of ExecStates. This is clearer, and ever-so-slightly faster.


Removed hasCachedXXX functions, now that they're unused.

  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::history): (WebCore::JSDOMWindow::location):
  • bindings/js/JSDocumentCustom.cpp: (WebCore::JSDocument::location): (WebCore::toJS):
  • bindings/js/JSElementCustom.cpp: (WebCore::toJSNewlyCreated):
  • bindings/js/JSEventCustom.cpp: (WebCore::toJS):
  • bindings/js/JSHTMLCollectionCustom.cpp: (WebCore::toJS):
  • bindings/js/JSImageDataCustom.cpp: (WebCore::toJS):
  • bindings/js/JSNodeCustom.cpp: (WebCore::createWrapperInline):
  • bindings/js/JSNodeCustom.h: (WebCore::getCachedDOMNodeWrapper): (WebCore::cacheDOMNodeWrapper): (WebCore::toJS):
  • bindings/js/JSSVGPathSegCustom.cpp: (WebCore::toJS):
  • bindings/js/JSStyleSheetCustom.cpp: (WebCore::toJS): Updated for changes above.
  • xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::dropProtection): Removed use of hasCachedDOMObjectWrapper because XHR is almost always created and used by JavaScript, so it's simpler to just always report extra cost.
1:45 PM Changeset in webkit [84028] by bweinstein@apple.com
  • 2 edits in trunk/LayoutTests

REGRESSION(r81262): plugins/refcount-leaks.html failing on Windows
https://bugs.webkit.org/show_bug.cgi?id=56478

Unskip plugins/refcount-leaks.html on Windows because it is now
passing on release and debug.

  • platform/win/Skipped:
1:33 PM Changeset in webkit [84027] by jamesr@google.com
  • 4 edits in trunk/LayoutTests

2011-04-15 James Robinson <jamesr@chromium.org>

Add chromium linux expectations for direct-image-compositing.html

  • platform/chromium-gpu-linux/compositing/direct-image-compositing-expected.checksum:
  • platform/chromium-gpu-linux/compositing/direct-image-compositing-expected.png:
  • platform/chromium/test_expectations.txt:
1:25 PM Changeset in webkit [84026] by crogers@google.com
  • 2 edits in trunk/LayoutTests

2011-04-15 Chris Rogers <crogers@google.com>

Unreviewed.

Update test_expectations.txt with more failing fast/borders tests
https://bugs.webkit.org/show_bug.cgi?id=58685

  • platform/chromium/test_expectations.txt:
1:11 PM Changeset in webkit [84025] by andreas.kling@nokia.com
  • 3 edits
    1 delete in trunk

2011-04-15 Andreas Kling <kling@webkit.org>

Rolling out accidental part of r84010.

  • platform/qt/RenderThemeQt.cpp: (WebCore::RenderThemeQt::computeSizeBasedOnStyle): (WebCore::RenderThemeQt::setButtonPadding):

2011-04-15 Andreas Kling <kling@webkit.org>

Rolling out accidental part of r84010.

  • fast/forms/button-line-break.html: Removed.
1:07 PM Changeset in webkit [84024] by oliver@apple.com
  • 2 edits in trunk/Source/WebCore

Forgot windows build fix.

1:01 PM Changeset in webkit [84023] by Joseph Pecoraro
  • 3 edits in trunk/Source/WebCore

2011-04-15 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Yury Semikhatsky.

Frontend Part of Web Inspector: Remote Web Inspector - Cross Platform InspectorServer
https://bugs.webkit.org/show_bug.cgi?id=51364

The front end should not use a transparent background when
loaded as a web page (remote debugging) but should when loaded
regularly in a custom window. This adds a "remote" style class
onto the document body when the page is loaded remotely.

  • inspector/front-end/inspector.css: (body.detached.platform-mac-snowleopard:not(.remote) #toolbar): respect the "remote" class.
  • inspector/front-end/inspector.js: add a "remote" class on the body when loaded remotely.
1:01 PM Changeset in webkit [84022] by Joseph Pecoraro
  • 2 edits in trunk/Source/WebCore

2011-04-15 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by David Kilzer.

JSLock ASSERT seen often when using the inspector for long
periods of time. We should take the JSLock whenever we
might allocate memory in the JavaScript Heap.

JSC InjectedScriptHost::nodeAsScriptValue should take JSLock before possible JavaScript Allocations
https://bugs.webkit.org/show_bug.cgi?id=58674

  • bindings/js/JSInjectedScriptHostCustom.cpp: (WebCore::InjectedScriptHost::nodeAsScriptValue): take the JSLock.
12:55 PM Changeset in webkit [84021] by oliver@apple.com
  • 11 edits in trunk/Source

2011-04-14 Oliver Hunt <oliver@apple.com>

Reviewed by Geoffrey Garen.

Make JSNodeFilterCondition handle its lifetime correctly
https://bugs.webkit.org/show_bug.cgi?id=58622

Add export

2011-04-14 Oliver Hunt <oliver@apple.com>

Reviewed by Geoffrey Garen.

Make JSNodeFilterCondition handle its lifetime correctly
https://bugs.webkit.org/show_bug.cgi?id=58622

Switch over to a WeakHandle and external roots to keep the
condition value live.

  • bindings/js/JSNodeFilterCondition.cpp: (WebCore::JSNodeFilterCondition::JSNodeFilterCondition): (WebCore::JSNodeFilterCondition::markAggregate): (WebCore::JSNodeFilterCondition::acceptNode): (WebCore::JSNodeFilterCondition::WeakOwner::isReachableFromOpaqueRoots):
  • bindings/js/JSNodeFilterCondition.h: (WebCore::JSNodeFilterCondition::create):
  • bindings/js/JSNodeFilterCustom.cpp: (WebCore::toNodeFilter):
  • bindings/scripts/CodeGeneratorJS.pm:
12:48 PM Changeset in webkit [84020] by cevans@google.com
  • 1 edit
    2 copies in branches/chromium/696

Merge 83611
BUG=79025

12:46 PM Changeset in webkit [84019] by bweinstein@apple.com
  • 3 edits in trunk/Source/WebKit2

Views should be made visible before painting in WM_PRINTCLIENT messages
https://bugs.webkit.org/show_bug.cgi?id=58676
<rdar://problem/9279211>

Reviewed by Adam Roben.

  • UIProcess/win/WebView.cpp:

(WebKit::WebView::onPrintClientEvent): If our view isn't currently visible set it to visible

before painting, then reset it back to not visible after painting.

(WebKit::WebView::onShowWindowEvent): Call setIsVisible instead of duplicating logic.
(WebKit::WebView::setIsVisible): Set the m_isVisible flag and call viewStateDidChange.

  • UIProcess/win/WebView.h:
12:42 PM Changeset in webkit [84018] by luiz@webkit.org
  • 3 edits in trunk/Source/WebCore

[Qt] QNetworkReplyHandler refactoring: some adjustments
https://bugs.webkit.org/show_bug.cgi?id=57092

Reviewed by Andreas Kling.

  • Removing flag m_redirected from QNetworkReplyHandler and using m_redirectionTargetUrl.isValid() instead.
  • Moving flag m_responseContainsData from QNetworkReplyHandler to the reply wrapper and removing the connection of

signal metaDataChanged for m_responseContainsData to be consistent.

  • Using an OwnPtr to keep the reference to the QNetworkReplyWrapper.
  • platform/network/qt/QNetworkReplyHandler.cpp:

(WebCore::QNetworkReplyWrapper::QNetworkReplyWrapper):
(WebCore::QNetworkReplyWrapper::receiveMetaData):
(WebCore::QNetworkReplyWrapper::didReceiveReadyRead):
(WebCore::QNetworkReplyHandler::QNetworkReplyHandler):
(WebCore::QNetworkReplyHandler::release):
(WebCore::shouldIgnoreHttpError):
(WebCore::QNetworkReplyHandler::finish):
(WebCore::QNetworkReplyHandler::sendResponseIfNeeded):
(WebCore::QNetworkReplyHandler::redirect):
(WebCore::QNetworkReplyHandler::forwardData):

  • platform/network/qt/QNetworkReplyHandler.h:

(WebCore::QNetworkReplyWrapper::responseContainsData):
(WebCore::QNetworkReplyWrapper::redirected):

12:37 PM Changeset in webkit [84017] by andersca@apple.com
  • 13 edits
    4 adds in trunk/Source/WebKit2

2011-04-15 Jon Lee <jonlee@apple.com>

Reviewed by Anders Carlsson.

onClick does not function with <select> elements in WebKit2
https://bugs.webkit.org/show_bug.cgi?id=57904
<rdar://problem/9217757>

Reviewed by Anders Carlsson.

  • Shared/NativeWebMouseEvent.h: Copied from Source/WebKit2/UIProcess/WebPopupMenuProxy.h. (WebKit::NativeWebMouseEvent::nativeEvent):
  • Shared/mac/NativeWebMouseEventMac.mm: Added. (WebKit::NativeWebMouseEvent::NativeWebMouseEvent):
  • Shared/qt/NativeWebMouseEventQt.cpp: Copied from Source/WebKit2/UIProcess/WebPopupMenuProxy.h. (WebKit::NativeWebMouseEvent::NativeWebMouseEvent):
  • Shared/win/NativeWebMouseEventWin.cpp: Copied from Source/WebKit2/UIProcess/WebPopupMenuProxy.h. (WebKit::NativeWebMouseEvent::NativeWebMouseEvent):
  • UIProcess/API/mac/WKView.mm: (-[WKView mouseMoved:]): (-[WKView _mouseHandler:]): Pre-compiler mouse handler calls need to use the new NativeWebMouseEvent class
  • UIProcess/API/qt/qwkpage.cpp: (QWKPagePrivate::mouseMoveEvent): (QWKPagePrivate::mousePressEvent): (QWKPagePrivate::mouseReleaseEvent): (QWKPagePrivate::mouseDoubleClickEvent):
  • UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::handleMouseEvent): keep track of mouse down event in order to be able to send a fake mouse up event on the select element (WebKit::WebPageProxy::currentMouseDownEvent): for access by the popup menu proxy to dispatch the fake mouse up event (WebKit::WebPageProxy::didReceiveEvent): clear out the cached mouse down event (WebKit::WebPageProxy::processDidCrash): clear out the cached events since we assume that didReceiveEvent did not called prior to the crash
  • UIProcess/WebPageProxy.h:
  • UIProcess/WebPopupMenuProxy.h:
  • UIProcess/mac/WebPopupMenuProxyMac.mm: (WebKit::WebPopupMenuProxyMac::showPopupMenu): after notifying the popup menu client that the value changed, fake mouse up and mouse move events are posted
  • UIProcess/win/WebPopupMenuProxyWin.cpp: (WebKit::WebPopupMenuProxyWin::showPopupMenu): after notifying the popup menu client that the value changed, fake mouse up and mouse move events are posted
  • UIProcess/win/WebView.cpp: (WebKit::WebView::onMouseEvent):
  • WebKit2.pro:
  • WebKit2.xcodeproj/project.pbxproj:
  • win/WebKit2.vcproj:
12:34 PM Changeset in webkit [84016] by ap@apple.com
  • 2 edits in trunk/Source/WebCore

Chromium build fix.

  • platform/SecureTextInput.h: CARBON_SECURE_INPUT_MODE no longer exists.
12:31 PM Changeset in webkit [84015] by cevans@google.com
  • 1 edit
    2 copies in branches/chromium/696

Merge 83699
BUG=79199

12:30 PM Changeset in webkit [84014] by ap@apple.com
  • 2 edits in trunk/Source/WebKit2

Qt build fix.

  • UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::selectionStateChanged): Wrap the variable in PLATFORM(MAC), too, as it's unused on other platforms.
12:25 PM Changeset in webkit [84013] by andreas.kling@nokia.com
  • 4 edits in trunk

2011-04-15 Andreas Kling <kling@webkit.org>

Reviewed by Antonio Gomes.

[Qt] DRT: Default custom policy delegate decision should be Ignore.

If layoutTestController.setCustomPolicyDelegate() is called with only
one argument, the second ("permissive") should default to false.

  • DumpRenderTree/qt/LayoutTestControllerQt.h:

2011-04-15 Andreas Kling <kling@webkit.org>

Reviewed by Antonio Gomes.

[Qt] DRT: Default custom policy delegate decision should be Ignore.

  • platform/qt/Skipped: Unskip now passing tests.
12:25 PM Changeset in webkit [84012] by beidson@apple.com
  • 1 edit in trunk/Source/WebKit2/ChangeLog

Get the right reviewer up in here.

12:24 PM Changeset in webkit [84011] by beidson@apple.com
  • 8 edits in trunk/Source/WebKit2

<rdar://problem/9287880> and https://bugs.webkit.org/show_bug.cgi?id=58596
WK2: Past searches not remembered for <input type=search results="5" autosave="foo">

Reviewed by Sam Weinig.

Add SaveRecentSearches and LoadRecentSearches messages:

  • UIProcess/WebPageProxy.messages.in:
  • UIProcess/WebPageProxy.h:

Message up to the UIProcess for the save/load operations:

  • WebProcess/WebCoreSupport/WebSearchPopupMenu.cpp:

(WebKit::WebSearchPopupMenu::saveRecentSearches):
(WebKit::WebSearchPopupMenu::loadRecentSearches):

  • WebProcess/WebCoreSupport/WebPopupMenu.h:

(WebKit::WebPopupMenu::page):

Save the values to disk CFPreference-style:

  • UIProcess/cf/WebPageProxyCF.cpp:

(WebKit::autosaveKey):
(WebKit::WebPageProxy::saveRecentSearches):
(WebKit::WebPageProxy::loadRecentSearches):

Stubbed out for non-CF platforms:

  • UIProcess/gtk/WebPageProxyGtk.cpp:

(WebKit::WebPageProxy::saveRecentSearches):
(WebKit::WebPageProxy::loadRecentSearches):

  • UIProcess/qt/WebPageProxyQt.cpp:

(WebKit::WebPageProxy::saveRecentSearches):
(WebKit::WebPageProxy::loadRecentSearches):

12:22 PM Changeset in webkit [84010] by andreas.kling@nokia.com
  • 5 edits
    1 add in trunk

2011-04-15 Andreas Kling <kling@webkit.org>

Reviewed by Antonio Gomes.

[Qt] DRT: Block access to external URLs.

  • platform/qt/Skipped: Unskip now passing tests.

2011-04-15 Andreas Kling <kling@webkit.org>

Reviewed by Antonio Gomes.

[Qt] DRT: Block access to external URLs.

Implement the "Blocked access to external URL" behavior for Qt's DRT,
based on what other ports are doing.

Fixes <http://webkit.org/b/57306> and <http://webkit.org/b/58523>.

  • WebCoreSupport/FrameLoaderClientQt.cpp: (WebCore::FrameLoaderClientQt::dispatchWillSendRequest):
12:16 PM Changeset in webkit [84009] by cevans@google.com
  • 3 edits
    8 copies in branches/chromium/696

Merge 83787
BUG=75186

12:16 PM Changeset in webkit [84008] by jianli@chromium.org
  • 36 edits
    3 copies
    3 deletes in branches/chromium/696

Merge 83884 - BlobBuilder should be vendor-prefixed
https://bugs.webkit.org/show_bug.cgi?id=58518

Reviewed by Darin Fisher.

Source/WebCore:

Renamed BlobBuilder to WebKitBlobBuilder.

  • Android.derived.jscbindings.mk:
  • Android.derived.v8bindings.mk:
  • CMakeLists.txt:
  • CodeGenerators.pri:
  • DerivedSources.cpp:
  • DerivedSources.make:
  • GNUmakefile.list.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • fileapi/WebKitBlobBuilder.cpp: Renamed from Source/WebCore/fileapi/BlobBuilder.cpp.

(WebCore::WebKitBlobBuilder::WebKitBlobBuilder):
(WebCore::WebKitBlobBuilder::getBuffer):
(WebCore::WebKitBlobBuilder::append):
(WebCore::WebKitBlobBuilder::getBlob):

  • fileapi/WebKitBlobBuilder.h: Renamed from Source/WebCore/fileapi/BlobBuilder.h.

(WebCore::WebKitBlobBuilder::create):

  • fileapi/WebKitBlobBuilder.idl: Renamed from Source/WebCore/fileapi/BlobBuilder.idl.
  • page/DOMWindow.idl:
  • workers/WorkerContext.idl:

LayoutTests:

  • fast/dom/Window/script-tests/window-property-descriptors.js:
  • fast/dom/Window/window-properties.html:
  • fast/dom/Window/window-property-descriptors-expected.txt:
  • fast/dom/prototype-inheritance-2-expected.txt:
  • fast/dom/prototype-inheritance-expected.txt:
  • fast/dom/script-tests/constructed-objects-prototypes.js:

(constructorPropertiesOnWindow):

  • fast/dom/script-tests/prototype-inheritance-2.js:

(constructorNamesForWindow):

  • fast/dom/script-tests/prototype-inheritance.js:
  • fast/dom/window-domurl-crash.html:
  • fast/files/blob-slice-overflow.html:
  • fast/files/blob-slice-test.html:
  • fast/files/resources/read-blob-test-cases.js:

(testReadingTripleSlicedHybridBlob):

  • fast/files/resources/read-common.js:

(buildBlob):

  • fast/files/workers/inline-worker-via-blob-url.html:
  • fast/filesystem/resources/file-writer-events.js:

(tenXBlob):
(startWrite):

  • fast/filesystem/resources/file-writer-gc-blob.js:

(tenXBlob):
(startWrite):

  • fast/filesystem/resources/file-writer-sync-truncate-extend.js:
  • fast/filesystem/resources/file-writer-sync-write-overlapped.js:
  • fast/filesystem/resources/file-writer-utils.js:
  • fast/js/global-constructors-expected.txt:
  • fast/js/script-tests/global-constructors.js:
  • http/tests/fileapi/create-blob-url-from-data-url.html:
  • http/tests/local/blob/resources/hybrid-blob-util.js:

(HybridBlobTestUtil.this.appendAndCreateBlob):
(HybridBlobTestUtil):

  • http/tests/security/resources/create-filesystem-file.html:
  • platform/chromium/fast/dom/prototype-inheritance-expected.txt:
  • platform/qt/fast/dom/Window/window-property-descriptors-expected.txt:
  • platform/qt/fast/dom/prototype-inheritance-expected.txt:
  • platform/qt/fast/js/global-constructors-expected.txt:
  • platform/win/fast/dom/window-domurl-crash-expected.txt:

TBR=jianli@chromium.org

12:13 PM Changeset in webkit [84007] by cevans@google.com
  • 3 edits
    8 deletes in branches/chromium/696

Revert 84004 - Merge 83787
BUG=75186

TBR=cevans@google.com

12:10 PM Changeset in webkit [84006] by jianli@chromium.org
  • 1 edit in branches/chromium/696/LayoutTests/fast/filesystem/resources/file-writer-utils.js

Merge 83878 - Unreviewed, fixed tests caused by r83873.

  • fast/filesystem/resources/file-writer-utils.js:

(verifyByteRangeIsZeroesHelper):
(verifyByteRangeAsStringHelper):

TBR=jianli@chromium.org

12:07 PM Changeset in webkit [84005] by jianli@chromium.org
  • 13 edits
    2 copies in branches/chromium/696

Merge 83873 - Add webkit prefix to Blob.slice method and change its semantics to be
like Array.slice
https://bugs.webkit.org/show_bug.cgi?id=58496

Reviewed by Darin Fisher.

Source/WebCore:

Renamed Blob.slice to blob.webkitSlice and made it take start and end
parameters that're like Array.slice.

Test: fast/files/blob-slice-test.html

  • fileapi/Blob.cpp:

(WebCore::Blob::webkitSlice):

  • fileapi/Blob.h:
  • fileapi/Blob.idl:

LayoutTests:

Added a new test to test the new behavior and also update all existing
tests to use the new name.

  • fast/dom/Window/window-properties.html:
  • fast/files/blob-slice-overflow.html:
  • fast/files/blob-slice-test-expected.txt: Added.
  • fast/files/blob-slice-test.html: Added.
  • fast/files/resources/read-blob-test-cases.js:

(testReadingSlicedFileBlob):
(testReadingSlicedTextBlob):
(testReadingSlicedArrayBufferBlob):
(testReadingSlicedHybridBlob):
(testReadingTripleSlicedHybridBlob):

  • fast/filesystem/resources/file-writer-utils.js:

(verifyByteRangeIsZeroesHelper):
(verifyByteRangeAsStringHelper):

  • http/tests/local/blob/script-tests/send-hybrid-blob.js:

():

  • http/tests/local/blob/script-tests/send-sliced-data-blob.js:

(runSliceTest):

  • http/tests/local/fileapi/script-tests/send-sliced-dragged-file.js:

(onStableFileDrop):
(onUnstableFileDrop):

  • http/tests/local/formdata/resources/send-form-data-common.js:

(testSendingFormData):

TBR=jianli@chromium.org

12:00 PM Changeset in webkit [84004] by cevans@google.com
  • 3 edits
    8 copies in branches/chromium/696

Merge 83787
BUG=75186

11:57 AM Changeset in webkit [84003] by cevans@google.com
  • 3 edits
    8 deletes in branches/chromium/696

Revert 84002 - Merge 83787
BUG=75186

TBR=rolandsteiner@chromium.org

TBR=cevans@google.com

11:50 AM Changeset in webkit [84002] by cevans@google.com
  • 3 edits
    8 copies in branches/chromium/696

Merge 83787
BUG=75186

TBR=rolandsteiner@chromium.org

11:47 AM Changeset in webkit [84001] by cevans@google.com
  • 3 edits
    8 deletes in branches/chromium/696

Revert 83995 - Merge 83787
BUG=75186

TBR=cevans@google.com

11:45 AM Changeset in webkit [84000] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-04-15 Mike Reed <reed@google.com>

Reviewed by Darin Fisher.

hide unused static function when SKIA_GPU is enabled
https://bugs.webkit.org/show_bug.cgi?id=58670

No new tests. Just fixes a warning (treated as an error) when SKIA_GPU is enabled

  • platform/graphics/skia/ImageSkia.cpp:
11:43 AM Changeset in webkit [83999] by jamesr@google.com
  • 2 edits in trunk/LayoutTests

2011-04-15 James Robinson <jamesr@chromium.org>

Fix duplicate expectation for compositing/tiling/huge-layer-img.html on chromium mac.

  • platform/chromium/test_expectations.txt:
11:39 AM Changeset in webkit [83998] by Simon Fraser
  • 9 edits
    12 adds in trunk

2011-04-15 Simon Fraser <Simon Fraser>

Reviewed by Beth Dakin.

Some borders with border-radius do not have rounded inner edges
https://bugs.webkit.org/show_bug.cgi?id=58457

Improve the logic used to compute the inner radii on curved
borders, to maintain borders of even thickness around the curve.

Tests: fast/borders/mixed-border-styles-radius.html

fast/borders/mixed-border-styles.html

  • rendering/RenderBoxModelObject.h:
  • rendering/RenderBoxModelObject.cpp: (WebCore::RenderBoxModelObject::computeBorderOuterRect): (WebCore::RenderBoxModelObject::computeBorderInnerRect): Two new utility functions to share some code that was in both paintBorder() and paintBoxShaadow(). getRoundedInnerBorderWithBorderWidths() now takes the outer border box as well as the inner box.

(WebCore::RenderBoxModelObject::paintBorder): Call new methods.
(WebCore::RenderBoxModelObject::paintBoxShadow): Ditto.

  • rendering/RenderObject.cpp: (WebCore::RenderObject::drawBoxSideFromPath): getRoundedInnerBorderWithBorderWidths() now takes the outer border box as well as the inner box.
  • rendering/style/RenderStyle.cpp: (WebCore::RenderStyle::getRoundedInnerBorderWithBorderWidths):
  • rendering/style/RenderStyle.h: Compute the inner radii by starting with teh radii used for the outer box, and then shrinking them down based on the border thickness.
11:38 AM Changeset in webkit [83997] by ap@apple.com
  • 13 edits in trunk/Source

Reviewed by Dan Bernstein.

WebKit2: Password field input does not switch to ASCII-compatible source
https://bugs.webkit.org/show_bug.cgi?id=58583
<rdar://problem/9059651>

The implementation is more modern than what we have in WK1. Instead of returning a null
input context (as in non-editable content), we now set page context's properties.

11:31 AM Changeset in webkit [83996] by xji@chromium.org
  • 5 edits in trunk

2011-04-13 Xiaomei Ji <xji@chromium.org>

Reviewed by Ryosuke Niwa.

Continue (3rd) experiment with moving caret by word in visual order.
https://bugs.webkit.org/show_bug.cgi?id=58294

  • editing/selection/move-by-word-visually-expected.txt:
  • editing/selection/move-by-word-visually.html:

2011-04-13 Xiaomei Ji <xji@chromium.org>

Reviewed by Ryosuke Niwa.

Continue (3rd) experiment with moving caret by word in visual order.
https://bugs.webkit.org/show_bug.cgi?id=58294

This patch along with r82588 and r83483 implements moving caret by
word in visual order.


The overall algorithm is:

  1. First get the InlineBox and offset of the pass-in VisiblePosition.
  2. Based on the position (left boundary, middle, right boundary) of the offset and the direction of the movement, look for visually adjacent word breaks. 2.1 If the offset is the minimum offset of the box,

return the rightmost word boundary in previous boxes if moving left.
return the leftmost word boundary in box and next boxes if moving right.

2.2 Similar for the case when offset is at the maximum offset of the box.
2.3 When offset is inside the box (not at boundaries), first find the previousWordPosition

or nextWordPosition based on the directionality of the box. If this word break position
is also inside the same box, return it. Otherwise (the nextWordPosition or
previousWordPosition is not in the same box or is at the box boundary), collect all the
word breaks in the box and search for the one closest to the input "offset" based on
box directionality, block directionality, and movement direction. Continue search in
adjacent boxes if needed.

Notes:

  1. Word boundaries are collected one box at a time. Only when a boundary that is closest to the input position (in the moving direction) is not available in current box, word boundaries in adjacent box will be collected. So, there is no need to save InlineBox in word boundaries. Instead, the word boundaries are saved as a pair (VisiblePosition, offset) to avoid recomputing VisiblePosition.


  1. We only collect boundaries of the right kind (i.e. left boundary of a word in LTR block and right boundary of a word in RTL block). And word boundaries are collected using previousWordPosition() and nextWordPosition(). So when box directionality is the same as block directionality, word boundaries are collected from right to left visually in a LTR box, and word boundaries are collected from left to right visually in a RTL box. It is the other way around when box directionality is different from block directionality.
  1. To find the right kinds of word boundaries, we must move back and forth between words in some situations. For example, if we're moving to the right in a LTR box in LTR block, we cannot simply return nextWordPosition() because it would return the right boundary of a word. Instead, we return nextWordPosition()'s nextWordPosition()'s previousWordPosition().
  1. When collecting word breaks inside a box, it first computes a start position, then collect the right kind of word breaks until it reaches the end of (or beyond) the box. In the meanwhile, it might need special handling on the rightmost or leftmost position based on the directionality of the box and block. These computations do not consider the box's bidi level.
  • editing/visible_units.cpp: (WebCore::nextWordBreakInBoxInsideBlockWithDifferentDirectionality): (WebCore::collectWordBreaksInBox): (WebCore::previousWordBoundaryInBox): (WebCore::nextWordBoundaryInBox): (WebCore::visuallyLastWordBoundaryInBox): (WebCore::leftWordBoundary): (WebCore::rightWordBoundary): (WebCore::leftWordPosition): (WebCore::rightWordPosition):
11:23 AM Changeset in webkit [83995] by cevans@google.com
  • 3 edits
    8 copies in branches/chromium/696

Merge 83787
BUG=75186

11:07 AM Changeset in webkit [83994] by jer.noble@apple.com
  • 7 edits in trunk/Source/WebKit2

2011-04-14 Jer Noble <jer.noble@apple.com>

Reviewed by Eric Carlson.

REGRESSION: Rendering in <video> element appears to render quickly to catch up if it has been playing in a background tab
https://bugs.webkit.org/show_bug.cgi?id=58637

Notify AVFoundation that no one will be rendering when the view or window moves off screen.

  • UIProcess/API/mac/PageClientImpl.mm: (WebKit::PageClientImpl::isViewVisible): Return false if the view's window is not visible.
  • UIProcess/API/mac/WKView.mm: (-[WKView addWindowObserversForWindow:]): Observe orderOut and orderIn events. (-[WKView removeWindowObservers]): Ditto. (-[WKView _windowDidOrderOffScreen:]): Added. (-[WKView _windowDidOrderOnScreen:]): Added.
  • WebProcess/WebPage/DrawingAreaImpl.cpp: (WebKit::DrawingAreaImpl::suspendPainting): Call LayerTreeHost::pauseRendering. (WebKit::DrawingAreaImpl::resumePainting): Call LayerTreeHost::resumeRendering.
  • WebProcess/WebPage/LayerTreeHost.h: (WebKit::LayerTreeHost::pauseRendering): Added stub. (WebKit::LayerTreeHost::resumeRendering): Added stub.
  • WebProcess/WebPage/ca/mac/LayerTreeHostCAMac.h:
  • WebProcess/WebPage/ca/mac/LayerTreeHostCAMac.mm: (WebKit::LayerTreeHostCAMac::pauseRendering): Added. Post a notification with our

intention to stop rendering.

(WebKit::LayerTreeHostCAMac::resumeRendering): Added. Post a notification with our

intention to start rendering.

11:05 AM Changeset in webkit [83993] by Simon Fraser
  • 161 edits
    4 adds in trunk/LayoutTests

2011-04-15 Simon Fraser <Simon Fraser>

Rebaseline Mac pixel results in fast/ on SnowLeopard.

  • platform/mac/fast/backgrounds/animated-svg-as-background-expected.checksum:
  • platform/mac/fast/backgrounds/animated-svg-as-background-expected.png:
  • platform/mac/fast/backgrounds/animated-svg-as-mask-expected.checksum:
  • platform/mac/fast/backgrounds/animated-svg-as-mask-expected.png:
  • platform/mac/fast/backgrounds/size/backgroundSize17-expected.checksum:
  • platform/mac/fast/backgrounds/size/backgroundSize17-expected.png:
  • platform/mac/fast/backgrounds/size/backgroundSize20-expected.checksum:
  • platform/mac/fast/backgrounds/size/backgroundSize20-expected.png:
  • platform/mac/fast/block/basic/011-expected.checksum:
  • platform/mac/fast/block/basic/011-expected.png:
  • platform/mac/fast/block/float/float-avoidance-expected.checksum:
  • platform/mac/fast/block/float/float-avoidance-expected.png:
  • platform/mac/fast/block/margin-collapse/103-expected.checksum:
  • platform/mac/fast/block/margin-collapse/103-expected.png:
  • platform/mac/fast/block/positioning/auto/005-expected.checksum:
  • platform/mac/fast/block/positioning/auto/005-expected.png:
  • platform/mac/fast/block/positioning/auto/006-expected.checksum:
  • platform/mac/fast/block/positioning/auto/006-expected.png:
  • platform/mac/fast/block/positioning/auto/vertical-lr/005-expected.checksum:
  • platform/mac/fast/block/positioning/auto/vertical-lr/005-expected.png:
  • platform/mac/fast/block/positioning/auto/vertical-lr/006-expected.checksum:
  • platform/mac/fast/block/positioning/auto/vertical-lr/006-expected.png:
  • platform/mac/fast/block/positioning/auto/vertical-rl/005-expected.checksum:
  • platform/mac/fast/block/positioning/auto/vertical-rl/005-expected.png:
  • platform/mac/fast/block/positioning/auto/vertical-rl/006-expected.checksum:
  • platform/mac/fast/block/positioning/auto/vertical-rl/006-expected.png:
  • platform/mac/fast/borders/border-radius-circle-expected.checksum:
  • platform/mac/fast/borders/border-radius-circle-expected.png:
  • platform/mac/fast/borders/border-radius-huge-assert-expected.checksum:
  • platform/mac/fast/borders/border-radius-huge-assert-expected.png:
  • platform/mac/fast/borders/svg-as-border-image-2-expected.checksum:
  • platform/mac/fast/borders/svg-as-border-image-2-expected.png:
  • platform/mac/fast/box-shadow/border-radius-big-expected.checksum:
  • platform/mac/fast/box-shadow/border-radius-big-expected.png:
  • platform/mac/fast/box-shadow/box-shadow-transformed-expected.checksum:
  • platform/mac/fast/box-shadow/box-shadow-transformed-expected.png:
  • platform/mac/fast/box-shadow/inset-with-extraordinary-radii-and-border-expected.checksum:
  • platform/mac/fast/box-shadow/inset-with-extraordinary-radii-and-border-expected.png:
  • platform/mac/fast/box-shadow/spread-expected.checksum:
  • platform/mac/fast/box-shadow/spread-expected.png:
  • platform/mac/fast/box-shadow/spread-multiple-inset-expected.checksum:
  • platform/mac/fast/box-shadow/spread-multiple-inset-expected.png:
  • platform/mac/fast/box-shadow/spread-multiple-normal-expected.checksum:
  • platform/mac/fast/box-shadow/spread-multiple-normal-expected.png:
  • platform/mac/fast/canvas/canvas-incremental-repaint-expected.checksum:
  • platform/mac/fast/canvas/canvas-incremental-repaint-expected.png:
  • platform/mac/fast/canvas/canvasDrawingIntoSelf-expected.checksum:
  • platform/mac/fast/canvas/canvasDrawingIntoSelf-expected.png:
  • platform/mac/fast/canvas/drawImage-expected.checksum:
  • platform/mac/fast/canvas/drawImage-expected.png:
  • platform/mac/fast/canvas/webgl/css-webkit-canvas-expected.checksum:
  • platform/mac/fast/canvas/webgl/css-webkit-canvas-expected.png:
  • platform/mac/fast/canvas/webgl/css-webkit-canvas-repaint-expected.checksum:
  • platform/mac/fast/canvas/webgl/css-webkit-canvas-repaint-expected.png:
  • platform/mac/fast/css/font-face-default-font-expected.checksum:
  • platform/mac/fast/css/font-face-default-font-expected.png:
  • platform/mac/fast/css/font_property_normal-expected.checksum:
  • platform/mac/fast/css/font_property_normal-expected.png:
  • platform/mac/fast/css/line-height-font-order-expected.checksum:
  • platform/mac/fast/css/line-height-font-order-expected.png:
  • platform/mac/fast/forms/001-expected.checksum:
  • platform/mac/fast/forms/001-expected.png:
  • platform/mac/fast/forms/basic-inputs-expected.checksum:
  • platform/mac/fast/forms/basic-inputs-expected.png:
  • platform/mac/fast/forms/box-shadow-override-expected.checksum:
  • platform/mac/fast/forms/box-shadow-override-expected.png:
  • platform/mac/fast/forms/checkbox-radio-onchange-expected.checksum:
  • platform/mac/fast/forms/checkbox-radio-onchange-expected.png:
  • platform/mac/fast/forms/file-input-disabled-expected.checksum:
  • platform/mac/fast/forms/file-input-disabled-expected.png:
  • platform/mac/fast/forms/form-element-geometry-expected.checksum:
  • platform/mac/fast/forms/form-element-geometry-expected.png:
  • platform/mac/fast/forms/formmove-expected.checksum:
  • platform/mac/fast/forms/formmove-expected.png:
  • platform/mac/fast/forms/formmove2-expected.checksum:
  • platform/mac/fast/forms/formmove2-expected.png:
  • platform/mac/fast/forms/indeterminate-expected.checksum:
  • platform/mac/fast/forms/indeterminate-expected.png:
  • platform/mac/fast/forms/input-appearance-height-expected.checksum:
  • platform/mac/fast/forms/input-appearance-height-expected.png:
  • platform/mac/fast/forms/input-value-expected.checksum:
  • platform/mac/fast/forms/input-value-expected.png:
  • platform/mac/fast/forms/minWidthPercent-expected.checksum:
  • platform/mac/fast/forms/minWidthPercent-expected.png:
  • platform/mac/fast/forms/radio-attr-order-expected.checksum:
  • platform/mac/fast/forms/radio-attr-order-expected.png:
  • platform/mac/fast/forms/radio-nested-labels-expected.checksum:
  • platform/mac/fast/forms/radio-nested-labels-expected.png:
  • platform/mac/fast/forms/radio_checked-expected.checksum:
  • platform/mac/fast/forms/radio_checked-expected.png:
  • platform/mac/fast/forms/radio_checked_dynamic-expected.checksum:
  • platform/mac/fast/forms/radio_checked_dynamic-expected.png:
  • platform/mac/fast/images/animated-svg-as-image-expected.checksum:
  • platform/mac/fast/images/animated-svg-as-image-expected.png:
  • platform/mac/fast/images/imagemap-case-expected.checksum:
  • platform/mac/fast/images/imagemap-case-expected.png:
  • platform/mac/fast/images/imagemap-circle-focus-ring-expected.checksum:
  • platform/mac/fast/images/imagemap-circle-focus-ring-expected.png:
  • platform/mac/fast/images/imagemap-focus-ring-zoom-expected.checksum:
  • platform/mac/fast/images/imagemap-focus-ring-zoom-expected.png:
  • platform/mac/fast/images/imagemap-polygon-focus-ring-expected.checksum:
  • platform/mac/fast/images/imagemap-polygon-focus-ring-expected.png:
  • platform/mac/fast/inline/positionedLifetime-expected.checksum:
  • platform/mac/fast/inline/positionedLifetime-expected.png:
  • platform/mac/fast/invalid/019-expected.checksum:
  • platform/mac/fast/invalid/019-expected.png:
  • platform/mac/fast/invalid/missing-address-end-tag-expected.checksum:
  • platform/mac/fast/invalid/missing-address-end-tag-expected.png:
  • platform/mac/fast/invalid/table-inside-stray-table-content-expected.checksum:
  • platform/mac/fast/invalid/table-inside-stray-table-content-expected.png:
  • platform/mac/fast/lists/003-vertical-expected.checksum:
  • platform/mac/fast/lists/003-vertical-expected.png:
  • platform/mac/fast/media/media-query-invalid-value-expected.checksum:
  • platform/mac/fast/media/media-query-invalid-value-expected.png:
  • platform/mac/fast/overflow/003-expected.checksum:
  • platform/mac/fast/overflow/003-expected.png:
  • platform/mac/fast/overflow/overflow-x-y-expected.checksum:
  • platform/mac/fast/overflow/overflow-x-y-expected.png:
  • platform/mac/fast/parser/bad-xml-slash-expected.checksum:
  • platform/mac/fast/parser/bad-xml-slash-expected.png:
  • platform/mac/fast/repaint/box-shadow-h-expected.checksum:
  • platform/mac/fast/repaint/box-shadow-h-expected.png:
  • platform/mac/fast/repaint/control-clip-expected.checksum:
  • platform/mac/fast/repaint/control-clip-expected.png:
  • platform/mac/fast/repaint/list-marker-expected.checksum:
  • platform/mac/fast/repaint/list-marker-expected.png:
  • platform/mac/fast/repaint/shadow-multiple-horizontal-expected.checksum:
  • platform/mac/fast/repaint/shadow-multiple-horizontal-expected.png:
  • platform/mac/fast/repaint/shadow-multiple-strict-horizontal-expected.checksum:
  • platform/mac/fast/repaint/shadow-multiple-strict-horizontal-expected.png:
  • platform/mac/fast/repaint/transform-replaced-shadows-expected.checksum:
  • platform/mac/fast/repaint/transform-replaced-shadows-expected.png:
  • platform/mac/fast/replaced/replaced-breaking-expected.checksum:
  • platform/mac/fast/replaced/replaced-breaking-expected.png:
  • platform/mac/fast/text/emphasis-vertical-expected.checksum:
  • platform/mac/fast/text/emphasis-vertical-expected.png:
  • platform/mac/fast/text/hyphenate-limit-before-after-expected.checksum:
  • platform/mac/fast/text/hyphenate-limit-before-after-expected.png:
  • platform/mac/fast/text/international/bidi-linebreak-001-expected.checksum:
  • platform/mac/fast/text/international/bidi-linebreak-001-expected.png:
  • platform/mac/fast/text/international/bidi-linebreak-002-expected.checksum:
  • platform/mac/fast/text/international/bidi-linebreak-002-expected.png:
  • platform/mac/fast/text/international/bidi-linebreak-003-expected.checksum:
  • platform/mac/fast/text/international/bidi-linebreak-003-expected.png:
  • platform/mac/fast/text/international/bidi-mirror-he-ar-expected.checksum:
  • platform/mac/fast/text/international/bidi-mirror-he-ar-expected.png:
  • platform/mac/fast/text/international/bidi-neutral-directionality-paragraph-start-expected.checksum:
  • platform/mac/fast/text/international/bidi-neutral-directionality-paragraph-start-expected.png:
  • platform/mac/fast/text/international/text-combine-image-test-expected.checksum:
  • platform/mac/fast/text/international/text-combine-image-test-expected.png:
  • platform/mac/fast/text/international/vertical-text-glyph-test-expected.checksum:
  • platform/mac/fast/text/international/vertical-text-glyph-test-expected.png:
  • platform/mac/fast/text/justify-ideograph-leading-expansion-expected.checksum:
  • platform/mac/fast/text/justify-ideograph-leading-expansion-expected.png:
  • platform/mac/fast/text/textIteratorNilRenderer-expected.checksum:
  • platform/mac/fast/text/textIteratorNilRenderer-expected.png:
  • platform/mac/fast/text/whitespace/normal-after-nowrap-breaking-expected.checksum:
  • platform/mac/fast/text/whitespace/normal-after-nowrap-breaking-expected.png:
  • platform/mac/fast/transforms/shadows-expected.checksum:
  • platform/mac/fast/transforms/shadows-expected.png:
11:01 AM Changeset in webkit [83992] by zmo@google.com
  • 3 edits in trunk/Source/WebCore

2011-04-14 Zhenyao Mo <zmo@google.com>

Reviewed by Kenneth Russell.

Use HTMLImageElement in Canvas 2D / WebGL before response is ready causes crash
https://bugs.webkit.org/show_bug.cgi?id=58501

  • html/canvas/WebGLRenderingContext.cpp: (WebCore::WebGLRenderingContext::texImage2D): Call validateHTMLImageElement(). (WebCore::WebGLRenderingContext::texSubImage2D): Ditto. (WebCore::WebGLRenderingContext::validateHTMLImageElement): Make sure image is ready.
  • html/canvas/WebGLRenderingContext.h:
10:57 AM Changeset in webkit [83991] by morrita@google.com
  • 5 edits in trunk/Source/WebCore

2011-04-07 MORITA Hajime <morrita@google.com>

Reviewed by Ryosuke Niwa.

DocumentMaker::AllMarkers should not be a part of DocumentMarker::MarkerType
https://bugs.webkit.org/show_bug.cgi?id=58112

  • Converted DocumentMarker::MarkerTypes from unsigned int to a class.
  • Converted DocumentMarker::AllMarkers from an enum entry to a subclass of MarkerTypes.
  • Changed type of some MarkerType argument on DocumentMarkerController API to MarkerTypes which should allow a combination of MarkerType constansts.
  • Removed some MarkerType arguments on DocumentMarkerController API which only received AllMarkers.

No new tests, no behavior change.

  • WebCore.exp.in:
  • dom/DocumentMarker.h: (WebCore::DocumentMarker::MarkerTypes::MarkerTypes): (WebCore::DocumentMarker::MarkerTypes::contains): (WebCore::DocumentMarker::MarkerTypes::intersects): (WebCore::DocumentMarker::MarkerTypes::operator==): (WebCore::DocumentMarker::MarkerTypes::add): (WebCore::DocumentMarker::MarkerTypes::remove): (WebCore::DocumentMarker::AllMarkers::AllMarkers):
  • dom/DocumentMarkerController.cpp: (WebCore::DocumentMarkerController::possiblyHasMarkers): (WebCore::DocumentMarkerController::addMarker): (WebCore::DocumentMarkerController::copyMarkers): (WebCore::DocumentMarkerController::removeMarkers): (WebCore::DocumentMarkerController::markerContainingPoint): (WebCore::DocumentMarkerController::markersInRange): (WebCore::DocumentMarkerController::renderedRectsForMarkers): (WebCore::DocumentMarkerController::removeMarkersFromMarkerMapVectorPair): (WebCore::DocumentMarkerController::repaintMarkers): (WebCore::DocumentMarkerController::shiftMarkers): (WebCore::DocumentMarkerController::setMarkersActive): (WebCore::DocumentMarkerController::hasMarkers): (WebCore::DocumentMarkerController::clearDescriptionOnMarkersIntersectingRange):
  • dom/DocumentMarkerController.h:
10:44 AM Changeset in webkit [83990] by ggaren@apple.com
  • 15 edits in trunk/Source/WebCore

2011-04-14 Geoffrey Garen <ggaren@apple.com>

Reviewed by Oliver Hunt.

Complicated hash table is complicated
https://bugs.webkit.org/show_bug.cgi?id=58631


Now that we use the opaque roots system to track node wrapper lifetime,
we can remove a lot of complicated hash-tablery that used to do the same.


Now normal world node wrappers are just set as direct properties of
ScriptWrappable, while isolated world node wrappers and other DOM object
wrappers are stored in a shared, per-world hash table.

In addition to reducing complexity, this makes DOM wrapper allocation
1.6X faster (tested with scratch-gc-dom3.html), and it reduces the memory
footprint of normal world wrappers by ~2/3, and isolated world wrappers
by ~1/3.

  • WebCore.exp.in: Paying the patch tithe.
  • bindings/js/DOMWrapperWorld.cpp: (WebCore::DOMWrapperWorld::~DOMWrapperWorld): (WebCore::DOMWrapperWorld::clearWrappers): No more per-document hash tables.

(WebCore::JSNodeHandleOwner::finalize): Changed to call a helper function,
so the code to destroy a wrapper can live next to the code to create one.

  • bindings/js/DOMWrapperWorld.h: No more per-document hash tables.
  • bindings/js/JSDOMBinding.cpp: (WebCore::uncacheDOMObjectWrapper):
  • bindings/js/JSDOMBinding.h: (WebCore::createDOMNodeWrapper): (WebCore::getDOMNodeWrapper): No more per-document hash tables. Added uncacheDOMObjectWrapper to be symmetrical with cacheDOMObjectWrapper.
  • bindings/js/JSDocumentCustom.cpp: (WebCore::toJS):
  • bindings/js/JSElementCustom.cpp: (WebCore::toJSNewlyCreated):
  • bindings/js/JSNodeCustom.cpp: (WebCore::createWrapperInline): Ditto.
  • bindings/js/JSNodeCustom.h: (WebCore::getCachedDOMNodeWrapper): (WebCore::cacheDOMNodeWrapper): (WebCore::uncacheDOMNodeWrapper): (WebCore::toJS): Implemented the scheme described above.
  • bindings/js/ScriptWrappable.h: (WebCore::ScriptWrappable::wrapper): (WebCore::ScriptWrappable::setWrapper): (WebCore::ScriptWrappable::clearWrapper): ScriptWrappable needs a handle owner now, since we don't have an extra handle living in a hash table to maintain ownership for us.
  • dom/Document.cpp: (WebCore::Document::Document): (WebCore::Document::~Document):
  • dom/Document.h:
  • dom/Node.cpp: (WebCore::Node::setDocument): No more per-document hash tables.
  • html/HTMLCanvasElement.cpp: (WebCore::HTMLCanvasElement::createImageBuffer): Removed call to hasCachedDOMNodeWrapperUnchecked because that was the old way of doing things, and I was in the mood for getting rid of the old way. It's debatable whether the check was ever a good idea. Even when a <canvas> doesn't have a direct JS wrapper, other JS references can still keep the <canvas> alive. So, it's probably best always to report extra cost.
10:42 AM Changeset in webkit [83989] by jberlin@webkit.org
  • 1 edit
    5 adds in trunk/LayoutTests

DRT on Windows doesn't block access to external URLs (https://bugs.webkit.org/show_bug.cgi?id=57334),
but WKTR does, so add the expected passing results to the win-wk2 directory to override the expected
failing results in the win directory in order to get the bots green.

  • platform/win-wk2/fast/ruby: Added.
  • platform/win-wk2/fast/ruby/after-block-doesnt-crash-expected.txt: Added.
  • platform/win-wk2/fast/ruby/after-table-doesnt-crash-expected.txt: Added.
  • platform/win-wk2/fast/ruby/before-block-doesnt-crash-expected.txt: Added.
  • platform/win-wk2/fast/ruby/before-table-doesnt-crash-expected.txt: Added.
10:41 AM Changeset in webkit [83988] by jberlin@webkit.org
  • 5 edits in trunk/Source/WebKit2

WebKit2: Need a way to keep the WebProcess alive for testing purposes.
https://bugs.webkit.org/show_bug.cgi?id=58592

Reviewed by Sam Weinig.

  • UIProcess/API/C/WKContext.cpp:

(WKContextDisableProcessTermination):
(WKContextEnableProcessTermination):

  • UIProcess/API/C/WKContextPrivate.h:
  • UIProcess/WebContext.cpp:

(WebKit::WebContext::WebContext):
Initialize m_processTerminationEnabled to true.
(WebKit::WebContext::enableProcessTermination):
Set m_processTerminationEnabled to true, and try to terminate the web process.
(WebKit::WebContext::shouldTerminate):
If !m_processTerminationEnabled, return false.

  • UIProcess/WebContext.h:

(WebKit::WebContext::disableProcessTermination):
Set m_processTerminationEnabled to false;

10:32 AM Changeset in webkit [83987] by commit-queue@webkit.org
  • 11 edits in trunk/Source

2011-04-15 Vsevolod Vlasov <vsevik@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: Rename rawRequestHeadersText and RawResponseHeadersText to requestHeadersText and responseHeadersText
https://bugs.webkit.org/show_bug.cgi?id=58650

  • inspector/Inspector.json:
  • inspector/InspectorResourceAgent.cpp: (WebCore::buildObjectForResourceResponse):
  • inspector/front-end/NetworkManager.js: (WebInspector.NetworkDispatcher.prototype._updateResourceWithResponse):
  • inspector/front-end/Resource.js: (WebInspector.Resource.prototype.get requestHeadersText): (WebInspector.Resource.prototype.set requestHeadersText): (WebInspector.Resource.prototype.get requestHeadersSize): (WebInspector.Resource.prototype.get responseHeadersText): (WebInspector.Resource.prototype.set responseHeadersText): (WebInspector.Resource.prototype.get responseHeadersSize): (WebInspector.Resource.prototype._headersSize):
  • inspector/front-end/ResourceHeadersView.js: (WebInspector.ResourceHeadersView): (WebInspector.ResourceHeadersView.prototype._refreshRequestHeaders): (WebInspector.ResourceHeadersView.prototype._refreshResponseHeaders): (WebInspector.ResourceHeadersView.prototype._refreshHeadersTitle): (WebInspector.ResourceHeadersView.prototype._refreshHeadersText): (WebInspector.ResourceHeadersView.prototype._toggleRequestHeadersText): (WebInspector.ResourceHeadersView.prototype._toggleResponseHeadersText): (WebInspector.ResourceHeadersView.prototype._createHeadersToggleButton):
  • inspector/front-end/networkPanel.css: (.resource-headers-view .outline-disclosure li.headers-text):
  • platform/network/ResourceLoadInfo.h:

2011-04-15 Vsevolod Vlasov <vsevik@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: Rename rawRequestHeadersText and RawResponseHeadersText to requestHeadersText and responseHeadersText
https://bugs.webkit.org/show_bug.cgi?id=58650

  • public/WebHTTPLoadInfo.h:
  • src/WebHTTPLoadInfo.cpp: (WebKit::WebHTTPLoadInfo::requestHeadersText): (WebKit::WebHTTPLoadInfo::setRequestHeadersText): (WebKit::WebHTTPLoadInfo::responseHeadersText): (WebKit::WebHTTPLoadInfo::setResponseHeadersText):
10:28 AM Changeset in webkit [83986] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

2011-04-15 Luke Macpherson <macpherson@chromium.org>

Reviewed by Dimitri Glazkov.

Implement css overflow properties in CSSStyleApplyProperty
https://bugs.webkit.org/show_bug.cgi?id=58633

No new tests required as no functionality changes.

  • css/CSSStyleApplyProperty.cpp: (WebCore::CSSStyleApplyProperty::CSSStyleApplyProperty): Add initializers for CSSPropertyOverflowX, CSSPropertyOverflowY and CSSPropertyOverflow.
  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::applyProperty): Delete existing implementations.
9:57 AM Changeset in webkit [83985] by weinig@apple.com
  • 6 edits in trunk/Source

2011-04-15 Sam Weinig <sam@webkit.org>

Reviewed by Maciej Stachowiak.

Make mac WebKit1 use the default localization strategy
https://bugs.webkit.org/show_bug.cgi?id=58628

  • WebCoreSupport/WebPlatformStrategies.h:
  • WebCoreSupport/WebPlatformStrategies.mm: Remove the localization strategy code. The equivalent is now in WebCore/platform/DefaultLocalizationStrategy.cpp.

2011-04-15 Sam Weinig <sam@webkit.org>

Reviewed by Maciej Stachowiak.

Make mac WebKit1 use the default localization strategy
https://bugs.webkit.org/show_bug.cgi?id=58628

  • English.lproj/Localizable.strings: Update by running update-webkit-localizable-strings.
  • platform/DefaultLocalizationStrategy.cpp: (WebCore::DefaultLocalizationStrategy::contextMenuItemTagLookUpInDictionary): (WebCore::DefaultLocalizationStrategy::keygenKeychainItemName): (WebCore::DefaultLocalizationStrategy::imageTitle): Match the WebKit1 versions of these functions by special casing CF platforms.
9:42 AM Changeset in webkit [83984] by commit-queue@webkit.org
  • 5 edits
    3 adds in trunk

2011-04-15 Vsevolod Vlasov <vsevik@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: No console message and headers in inspector when X-Frame-Options header blocks a load
https://bugs.webkit.org/show_bug.cgi?id=58136

Passed response info to inspector when X-Frame-Options header blocks resource loading.

  • http/tests/inspector/network/resources/x-frame-options-deny.cgi: Added.
  • http/tests/inspector/network/x-frame-options-deny-expected.txt: Added.
  • http/tests/inspector/network/x-frame-options-deny.html: Added.

2011-04-15 Vsevolod Vlasov <vsevik@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: No console message and headers in inspector when X-Frame-Options header blocks a load
https://bugs.webkit.org/show_bug.cgi?id=58136

Passed response info to inspector when X-Frame-Options header blocks resource loading.

Test: http/tests/inspector/network/x-frame-options-deny.html

  • inspector/InspectorInstrumentation.cpp: (WebCore::InspectorInstrumentation::continueAfterXFrameOptionsDeniedImpl):
  • inspector/InspectorInstrumentation.h: (WebCore::InspectorInstrumentation::continueAfterXFrameOptionsDenied):
  • loader/MainResourceLoader.cpp: (WebCore::MainResourceLoader::didReceiveResponse):
9:29 AM Changeset in webkit [83983] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk

2011-04-15 Andrey Adaikin <aandrey@google.com>

Reviewed by Pavel Feldman.

Web Inspector: TextViewer and TextEditorModel must support both \n and \r\n as line separators
https://bugs.webkit.org/show_bug.cgi?id=58449

  • inspector/editor/text-editor-model-expected.txt: Added.
  • inspector/editor/text-editor-model.html: Added.

2011-04-15 Andrey Adaikin <aandrey@google.com>

Reviewed by Pavel Feldman.

Web Inspector: TextViewer and TextEditorModel must support both \n and \r\n as line separators
https://bugs.webkit.org/show_bug.cgi?id=58449

Test: inspector/editor/text-editor-model.html

9:20 AM Changeset in webkit [83982] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit2

2011-04-15 Nancy Piedra <nancy.piedra@nokia.com>

Reviewed by Laszlo Gombos.

[Qt][Symbian] Need to export ViewportAttributes class
https://bugs.webkit.org/show_bug.cgi?id=58651

On Symbian, if the ViewportAttributes class is not exported we get
linking errors.

Since this is a compilation issue, no new tests added.

  • UIProcess/API/qt/qwkpage.h:
9:11 AM Changeset in webkit [83981] by eric@webkit.org
  • 2 edits in trunk/LayoutTests

2011-04-15 Eric Seidel <eric@webkit.org>

Reviewed by Eric Carlson.

Flaky Test: media/invalid-media-url-crash.html
https://bugs.webkit.org/show_bug.cgi?id=51138

Skipping this test on Snow Leopard since it's hitting
a known MediaToolbox crasher on Snow Leopard.
If Apple happens to software update SL users we can
unskip this, until then no need to run this flaky test.

  • platform/mac-snowleopard/Skipped:
9:00 AM Changeset in webkit [83980] by ojan@chromium.org
  • 3 edits in trunk/Tools

2011-04-15 Ojan Vafai <ojan@chromium.org>

Reviewed by Eric Seidel.

fix path to layout test in new results file
https://bugs.webkit.org/show_bug.cgi?id=58618

  • Scripts/webkitpy/layout_tests/layout_package/json_results.html:
  • Scripts/webkitpy/layout_tests/layout_package/test_runner.py:
8:45 AM Changeset in webkit [83979] by sergio@webkit.org
  • 2 edits in trunk/LayoutTests

2011-04-15 Sergio Villar Senin <svillar@igalia.com>

Unreviewed, skipping a test.

[GTK] editing/selection/select-out-of-editable.html failing after r83967
https://bugs.webkit.org/show_bug.cgi?id=58667

  • platform/gtk/Skipped: skipping editing/selection/select-out-of-editable.html
8:42 AM Changeset in webkit [83978] by ojan@chromium.org
  • 2 edits in trunk/Tools

2011-04-15 Ojan Vafai <ojan@chromium.org>

Reviewed by Eric Seidel.

fix sorting in new results file
https://bugs.webkit.org/show_bug.cgi?id=58616

-Fix the custom sort to return -1, 0, 1 instead of true/false.
-Secondary sort by test name when sort values are equal.
-Sort by test name by default.

  • Scripts/webkitpy/layout_tests/layout_package/json_results.html:
8:39 AM Changeset in webkit [83977] by commit-queue@webkit.org
  • 3 edits in trunk/Tools

2011-04-15 Dmitry Lomov <dslomov@google.com>

Reviewed by David Levin.

check-webkit-style shouldn't complain about not including a primary header file
if none exists
https://bugs.webkit.org/show_bug.cgi?id=39514

  • Scripts/webkitpy/style/checkers/cpp.py:
  • Scripts/webkitpy/style/checkers/cpp_unittest.py:
8:38 AM Changeset in webkit [83976] by jberlin@webkit.org
  • 1 edit
    6 adds in trunk/LayoutTests

[Windows Tests] fast/forms/input-text-drag-down.html and
editing/selection/select-from-textfield-outwards.html failing since r83967.
https://bugs.webkit.org/show_bug.cgi?id=58664

Adding Windows-specific expected failing results to get the bots green.

  • platform/win/editing/selection/select-from-textfield-outwards-expected.checksum: Added.
  • platform/win/editing/selection/select-from-textfield-outwards-expected.png: Added.
  • platform/win/editing/selection/select-from-textfield-outwards-expected.txt: Added.
  • platform/win/fast/forms/input-text-drag-down-expected.checksum: Added.
  • platform/win/fast/forms/input-text-drag-down-expected.png: Added.
  • platform/win/fast/forms/input-text-drag-down-expected.txt: Added.
8:15 AM Changeset in webkit [83975] by yurys@chromium.org
  • 14 edits
    4 adds in trunk/Source/WebCore

2011-04-15 Sergey Vorobyev <sergeyvorobyev@google.com>

Reviewed by Yury Semikhatsky.

Web Inspector: Network events don't preserves,
when inspector frontend closed and open again
https://bugs.webkit.org/show_bug.cgi?id=58064

Added InspectorFrontendProxy and EventsCollector.
They allow captured messages from InspectorResourceAgent
to frontend (or mockFrontend if frontend disabled) and
push collected data when frontend reconnect.
This functionality is disabled by default.

  • CMakeLists.txt:
  • GNUmakefile.list.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • inspector/EventsCollector.cpp: Added. (WebCore::EventsCollector::EventsCollector): (WebCore::EventsCollector::addEvent): (WebCore::EventsCollector::sendCollectedEvents):
  • inspector/EventsCollector.h: Added. (WebCore::EventsCollector::~EventsCollector):
  • inspector/Inspector.json:
  • inspector/InspectorFrontendProxy.cpp: Added. (WebCore::InspectorFrontendProxy::InspectorFrontendProxy): (WebCore::InspectorFrontendProxy::setInspectorFrontendChannel): (WebCore::InspectorFrontendProxy::setEventsCollector): (WebCore::InspectorFrontendProxy::sendMessageToFrontend):
  • inspector/InspectorFrontendProxy.h: Added. (WebCore::InspectorFrontendProxy::~InspectorFrontendProxy):
  • inspector/InspectorInstrumentation.cpp: (WebCore::InspectorInstrumentation::domContentLoadedEventFiredImpl): (WebCore::InspectorInstrumentation::loadEventFiredImpl):
  • inspector/InspectorInstrumentation.h: (WebCore::InspectorInstrumentation::willSendRequest): (WebCore::InspectorInstrumentation::willReceiveResourceResponse): (WebCore::InspectorInstrumentation::didReceiveContentLength): (WebCore::InspectorInstrumentation::didFinishLoading): (WebCore::InspectorInstrumentation::domContentLoadedEventFired): (WebCore::InspectorInstrumentation::loadEventFired): (WebCore::InspectorInstrumentation::frameDetachedFromParent): (WebCore::InspectorInstrumentation::didCreateWebSocket): (WebCore::InspectorInstrumentation::willSendWebSocketHandshakeRequest): (WebCore::InspectorInstrumentation::didReceiveWebSocketHandshakeResponse): (WebCore::InspectorInstrumentation::didCloseWebSocket):
  • inspector/InspectorResourceAgent.cpp: (WebCore::InspectorResourceAgent::setFrontend): (WebCore::InspectorResourceAgent::resourceContent): (WebCore::InspectorResourceAgent::~InspectorResourceAgent): (WebCore::InspectorResourceAgent::didReceiveResponse): (WebCore::InspectorResourceAgent::domContentEventFired): (WebCore::InspectorResourceAgent::loadEventFired): (WebCore::InspectorResourceAgent::enabledBackgoundEventsCoollection): (WebCore::InspectorResourceAgent::enable): (WebCore::InspectorResourceAgent::InspectorResourceAgent):
  • inspector/InspectorResourceAgent.h:
  • inspector/front-end/NetworkManager.js: (WebInspector.NetworkDispatcher.prototype.domContentEventFired): (WebInspector.NetworkDispatcher.prototype.loadEventFired):
  • inspector/front-end/inspector.js: (WebInspector.domContentEventFired): (WebInspector.loadEventFired):
7:55 AM Changeset in webkit [83974] by commit-queue@webkit.org
  • 2 edits
    3 adds in trunk/Source/JavaScriptCore

2011-04-15 Dmitry Lomov <dslomov@google.com>

Reviewed by David Levin.

Add a sample test case for GTest framework
https://bugs.webkit.org/show_bug.cgi?id=58509

Add an example of GTest testcase, complete with a runner, to JavaScriptCore.

  • JavaScriptCore.xcodeproj/project.pbxproj:
  • wtf/tests/RunAllWtfTests.cpp: Added. (main):
  • wtf/tests/StringTests.cpp: Added.
7:38 AM Changeset in webkit [83973] by alexis.menard@openbossa.org
  • 3 edits in trunk/Source/WebCore

2011-04-15 Alexis Menard <alexis.menard@openbossa.org>

Reviewed by Dimitri Glazkov.

REGRESSION(r83397) [Qt] When clicking on the media elements they grow 2 pixels.
https://bugs.webkit.org/show_bug.cgi?id=58477

Since r83397 the media controls elements are actual DOM elements.
Therefore the global style-sheet applies to them. html.css defines
input[type="button"]:active to be border-style: inset which means
that when the buttons are active they grow by their border size. Therefore
ports which are not using the borders must explicitly disable them in their custom
stylesheet.

  • css/mediaControlsQt.css: (audio::-webkit-media-controls-mute-button): (video::-webkit-media-controls-mute-button): (audio::-webkit-media-controls-play-button): (video::-webkit-media-controls-play-button): (video::-webkit-media-controls-fullscreen-button):
  • css/mediaControlsQuickTime.css: (audio::-webkit-media-controls-mute-button, video::-webkit-media-controls-mute-button): (audio::-webkit-media-controls-play-button, video::-webkit-media-controls-play-button): (audio::-webkit-media-controls-seek-back-button, video::-webkit-media-controls-seek-back-button): (audio::-webkit-media-controls-seek-forward-button, video::-webkit-media-controls-seek-forward-button): (audio::-webkit-media-controls-fullscreen-button, video::-webkit-media-controls-fullscreen-button): (audio::-webkit-media-controls-rewind-button, video::-webkit-media-controls-rewind-button): (audio::-webkit-media-controls-return-to-realtime-button, video::-webkit-media-controls-return-to-realtime-button): (audio::-webkit-media-controls-toggle-closed-captions-button, video::-webkit-media-controls-toggle-closed-captions-button): (audio::-webkit-media-controls-volume-slider-mute-button, video::-webkit-media-controls-volume-slider-mute-button):
7:35 AM Changeset in webkit [83972] by commit-queue@webkit.org
  • 15 edits in trunk/Source

2011-04-15 Zelidrag Hornung <zelidrag@chromium.org>

Reviewed by Darin Fisher.

Added enums for external file system type.
https://bugs.webkit.org/show_bug.cgi?id=58456

  • fileapi/DOMFileSystemBase.cpp: (WebCore::DOMFileSystemBase::crackFileSystemURL):
  • fileapi/DOMFileSystemBase.h:
  • fileapi/EntryBase.cpp: (WebCore::EntryBase::toURL):
  • page/DOMWindow.cpp: (WebCore::DOMWindow::requestFileSystem):
  • page/DOMWindow.h:
  • platform/AsyncFileSystem.h:
  • workers/WorkerContext.cpp: (WebCore::WorkerContext::requestFileSystem): (WebCore::WorkerContext::requestFileSystemSync):
  • workers/WorkerContext.h:

2011-04-15 Zelidrag Hornung <zelidrag@chromium.org>

Reviewed by Darin Fisher.

Added enums for external file system type and exposed factory function
for creating File- and DirectoryEntry objects from Chromium side.
https://bugs.webkit.org/show_bug.cgi?id=58456

  • public/WebFileSystem.h:
  • public/WebFrame.h:
  • src/AssertMatchingEnums.cpp:
  • src/WebFrameImpl.cpp: (WebKit::WebFrameImpl::createFileSystem): (WebKit::WebFrameImpl::createFileEntry):
  • src/WebFrameImpl.h:
7:30 AM Changeset in webkit [83971] by sergio@webkit.org
  • 1 edit
    6 adds in trunk/LayoutTests

2011-04-15 Sergio Villar Senin <svillar@igalia.com>

Unreviewed, added new GTK+ test expectations.

  • platform/gtk/fast/forms/select-background-none-expected.checksum: Added.
  • platform/gtk/fast/forms/select-background-none-expected.png: Added.
  • platform/gtk/fast/forms/select-background-none-expected.txt: Added.
  • platform/gtk/fast/text/zero-font-size-expected.checksum: Added.
  • platform/gtk/fast/text/zero-font-size-expected.png: Added.
  • platform/gtk/fast/text/zero-font-size-expected.txt: Added.
7:25 AM Changeset in webkit [83970] by Adam Roben
  • 2 edits in trunk/LayoutTests

Add new eventSender-dependent tests from r83967 to the mac-wks Skipped file

  • platform/mac-wk2/Skipped: Added some new editing/selection tests.
7:08 AM Changeset in webkit [83969] by pfeldman@chromium.org
  • 6 edits in trunk/Source/WebCore

2011-04-15 Pavel Feldman <pfeldman@google.com>

Reviewed by Yury Semikhatsky.

Web Inspector: make resource revisions use dedicated type (not Resource clone).
https://bugs.webkit.org/show_bug.cgi?id=58659

  • inspector/front-end/CSSStyleModel.js: (WebInspector.CSSStyleModel.prototype.setStyleSheetText):
  • inspector/front-end/Resource.js: (WebInspector.Resource.prototype.addRevision): (WebInspector.Resource.prototype._innerRequestContent.onResourceContent): (WebInspector.Resource.prototype._innerRequestContent): (WebInspector.ResourceRevision): (WebInspector.ResourceRevision.prototype.get resource): (WebInspector.ResourceRevision.prototype.get timestamp): (WebInspector.ResourceRevision.prototype.get content): (WebInspector.ResourceRevision.prototype.revertToThis): (WebInspector.ResourceRevision.prototype.requestContent.mycallback): (WebInspector.ResourceRevision.prototype.requestContent):
  • inspector/front-end/ResourceView.js: (WebInspector.ResourceView.resourceViewTypeMatchesResource): (WebInspector.ResourceView.resourceViewForResource): (WebInspector.ResourceView.recreateResourceView): (WebInspector.ResourceView.existingResourceViewForResource): (WebInspector.RevisionSourceFrame): (WebInspector.RevisionSourceFrame.prototype.get resource): (WebInspector.RevisionSourceFrame.prototype.isContentEditable): (WebInspector.RevisionSourceFrame.prototype.requestContent):
  • inspector/front-end/ResourcesPanel.js: (WebInspector.ResourcesPanel.prototype._showResourceView): (WebInspector.ResourcesPanel.prototype._showRevisionView): (WebInspector.ResourcesPanel.prototype._fetchAndApplyDiffMarkup.step1): (WebInspector.ResourcesPanel.prototype._fetchAndApplyDiffMarkup.step2): (WebInspector.ResourcesPanel.prototype._fetchAndApplyDiffMarkup): (WebInspector.ResourceRevisionTreeElement): (WebInspector.ResourceRevisionTreeElement.prototype.get itemURL): (WebInspector.ResourceRevisionTreeElement.prototype.onselect): (WebInspector.ResourceRevisionTreeElement.prototype._ondragstart): (WebInspector.ResourceRevisionTreeElement.prototype._handleContextMenuEvent):
  • inspector/front-end/SourceFrame.js: (WebInspector.SourceFrame.prototype._ensureContentLoaded): (WebInspector.SourceFrame.prototype.requestContent): (WebInspector.SourceFrame.prototype.commitEditing.didEditContent): (WebInspector.SourceFrame.prototype.commitEditing):
6:57 AM Changeset in webkit [83968] by apavlov@chromium.org
  • 2 edits in trunk/Source/WebCore

2011-04-15 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: Prevent Backspace keypresses from routing into the inspected page
https://bugs.webkit.org/show_bug.cgi?id=58653

  • inspector/front-end/inspector.js: (WebInspector.documentKeyDown):
6:44 AM Changeset in webkit [83967] by commit-queue@webkit.org
  • 3 edits
    9 adds in trunk

2011-04-15 Alice Boxhall <aboxhall@chromium.org>

Reviewed by Ryosuke Niwa.

Text selection changes unexpectedly when dragging out of the <input>
https://bugs.webkit.org/show_bug.cgi?id=55552

Tests that dragging outside of a contenteditable, input or textarea selects to the end of the
element, rather than jumping back to the beginning.

  • editing/selection/resources/select-out-of-floated-editable.js: Added. (log): (else.window.onmouseup): (getSelectionStart): (getSelectionEnd): (checkSelection):
  • editing/selection/select-out-of-editable-expected.txt: Added.
  • editing/selection/select-out-of-editable.html: Added.
  • editing/selection/select-out-of-floated-contenteditable-expected.txt: Added.
  • editing/selection/select-out-of-floated-contenteditable.html: Added.
  • editing/selection/select-out-of-floated-input-expected.txt: Added.
  • editing/selection/select-out-of-floated-input.html: Added.
  • editing/selection/select-out-of-floated-textarea-expected.txt: Added.
  • editing/selection/select-out-of-floated-textarea.html: Added.

2011-04-15 Alice Boxhall <aboxhall@chromium.org>

Reviewed by Ryosuke Niwa.

Text selection changes unexpectedly when dragging out of the <input>
https://bugs.webkit.org/show_bug.cgi?id=55552

Tests: editing/selection/select-out-of-editable.html

editing/selection/select-out-of-floated-contenteditable.html
editing/selection/select-out-of-floated-input.html
editing/selection/select-out-of-floated-textarea.html

  • page/EventHandler.cpp: (WebCore::selectionExtentRespectingEditingBoundary): When dragging from an editable element, check that the endpoint is not outside the element. If it is, translate the point into a local point within the editable element. (WebCore::EventHandler::updateSelectionForMouseDrag): Call targetPositionForSelectionEndpoint() to calculate the selection endpoint.
6:41 AM Changeset in webkit [83966] by Adam Roben
  • 2 edits in trunk/LayoutTests

Update Windows results after r83939

  • platform/win/media/media-document-audio-repaint-expected.txt:
6:34 AM Changeset in webkit [83965] by Adam Roben
  • 2 edits in trunk/LayoutTests

Skip a sometimes-failing test on Windows

<http://webkit.org/b/58441> tracks the failure.

  • platform/win/Skipped: Added fast/events/pageshow-pagehide-on-back-cached-with-frames.html.
6:28 AM Changeset in webkit [83964] by Adam Roben
  • 2 edits in trunk/LayoutTests

Skip a timing-out test on Windows

<http://webkit.org/b/58654> tracks the failure.

  • platform/win/Skipped: Added media/video-controls-in-media-document.html.
6:12 AM Changeset in webkit [83963] by Adam Roben
  • 9 edits
    8 deletes in trunk

Roll out r83954

It was causing fast/dom/Window/timer-null-script-execution-context.html to crash on multiple
bots.

See <http://webkit.org/b/58610>.

Source/WebCore:

  • bindings/js/JSDOMWindowCustom.cpp:

(WebCore::JSDOMWindow::setTimeout):
(WebCore::JSDOMWindow::setInterval):

  • bindings/js/JSWorkerContextCustom.cpp:

(WebCore::JSWorkerContext::setTimeout):
(WebCore::JSWorkerContext::setInterval):

  • bindings/js/ScheduledAction.cpp:

(WebCore::ScheduledAction::create):

  • bindings/js/ScheduledAction.h:
  • bindings/v8/custom/V8DOMWindowCustom.cpp:

(WebCore::WindowSetTimeoutImpl):

  • page/ContentSecurityPolicy.cpp:
  • page/ContentSecurityPolicy.h:

LayoutTests:

  • http/tests/security/contentSecurityPolicy/eval-scripts-setInterval-allowed-expected.txt: Removed.
  • http/tests/security/contentSecurityPolicy/eval-scripts-setInterval-allowed.html: Removed.
  • http/tests/security/contentSecurityPolicy/eval-scripts-setInterval-blocked-expected.txt: Removed.
  • http/tests/security/contentSecurityPolicy/eval-scripts-setInterval-blocked.html: Removed.
  • http/tests/security/contentSecurityPolicy/eval-scripts-setTimeout-allowed-expected.txt: Removed.
  • http/tests/security/contentSecurityPolicy/eval-scripts-setTimeout-allowed.html: Removed.
  • http/tests/security/contentSecurityPolicy/eval-scripts-setTimeout-blocked-expected.txt: Removed.
  • http/tests/security/contentSecurityPolicy/eval-scripts-setTimeout-blocked.html: Removed.
5:39 AM Changeset in webkit [83962] by pfeldman@chromium.org
  • 13 edits in trunk

2011-04-15 Pavel Feldman <pfeldman@google.com>

Reviewed by Yury Semikhatsky.

Web Inspector: refactor resource setContent / revisions infrastructure to get
rid of onRevert callback.
https://bugs.webkit.org/show_bug.cgi?id=58649

Instead, we will have DomainModel/Resource binding responsible for changing
underlying model upon resource changes and vice versa.

  • inspector/Inspector.json:
  • inspector/InspectorCSSAgent.cpp: (WebCore::InspectorCSSAgent::getStyleSheet): (WebCore::InspectorCSSAgent::getStyleSheetText): (WebCore::InspectorCSSAgent::setStyleSheetText): (WebCore::InspectorCSSAgent::setPropertyText): (WebCore::InspectorCSSAgent::toggleProperty): (WebCore::InspectorCSSAgent::setRuleSelector): (WebCore::InspectorCSSAgent::assertStyleSheetForId):
  • inspector/InspectorCSSAgent.h:
  • inspector/front-end/CSSStyleModel.js: (WebInspector.CSSStyleModel): (WebInspector.CSSStyleModel.prototype.setRuleSelector): (WebInspector.CSSStyleModel.prototype.addRule): (WebInspector.CSSStyleModel.prototype._fireStyleSheetChanged.callback): (WebInspector.CSSStyleModel.prototype._fireStyleSheetChanged): (WebInspector.CSSStyleModel.prototype.setStyleSheetText): (WebInspector.CSSStyleDeclaration.prototype.insertPropertyAt): (WebInspector.CSSProperty.prototype.setText): (WebInspector.CSSProperty.prototype.setText.callback): (WebInspector.CSSProperty.prototype.setDisabled.callback): (WebInspector.CSSProperty.prototype.setDisabled): (WebInspector.CSSStyleSheet.prototype.setText): (WebInspector.CSSStyleModelResourceBinding): (WebInspector.CSSStyleModelResourceBinding.prototype.setContent): (WebInspector.CSSStyleModelResourceBinding.prototype._frameNavigated): (WebInspector.CSSStyleModelResourceBinding.prototype._innerSetContent): (WebInspector.CSSStyleModelResourceBinding.prototype._loadStyleSheetHeaders): (WebInspector.CSSStyleModelResourceBinding.prototype._styleSheetChanged.setContent): (WebInspector.CSSStyleModelResourceBinding.prototype._styleSheetChanged):
  • inspector/front-end/DebuggerPresentationModel.js: (WebInspector.DebuggerPresentationModel): (WebInspector.DebuggerPresentationModel.prototype.editScriptSource.didEditScriptSource): (WebInspector.DebuggerPresentationModel.prototype.editScriptSource.didReceiveSource): (WebInspector.DebuggerPresentationModel.prototype.editScriptSource): (WebInspector.DebuggerPresentationModelResourceBinding): (WebInspector.DebuggerPresentationModelResourceBinding.prototype.canSetContent): (WebInspector.DebuggerPresentationModelResourceBinding.prototype.setContent):
  • inspector/front-end/Object.js: (WebInspector.Object.prototype.hasEventListeners):
  • inspector/front-end/Resource.js: (WebInspector.Resource): (WebInspector.Resource.registerDomainModelBinding): (WebInspector.Resource.prototype.isEditable): (WebInspector.Resource.prototype.setContent): (WebInspector.Resource.prototype.addRevision): (WebInspector.Resource.prototype.revertToThis.callback): (WebInspector.Resource.prototype.revertToThis): (WebInspector.ResourceDomainModelBinding): (WebInspector.ResourceDomainModelBinding.prototype.canSetContent): (WebInspector.ResourceDomainModelBinding.prototype.setContent):
  • inspector/front-end/ResourceView.js: (WebInspector.ResourceView.createResourceView): (WebInspector.ResourceView.resourceViewTypeMatchesResource): (WebInspector.ResourceSourceFrame.prototype.isContentEditable): (WebInspector.ResourceSourceFrame.prototype.editContent): (WebInspector.ResourceSourceFrame.prototype.endEditing): (WebInspector.ResourceSourceFrame.prototype._clearIncrementalUpdateTimer): (WebInspector.ResourceSourceFrame.prototype._requestContent):
  • inspector/front-end/ResourcesPanel.js: (WebInspector.ResourcesPanel.prototype._resourceAdded): (WebInspector.FrameResourceTreeElement): (WebInspector.FrameResourceTreeElement.prototype._populateRevisions): (WebInspector.FrameResourceTreeElement.prototype._revisionAdded): (WebInspector.FrameResourceTreeElement.prototype._appendRevision):
  • inspector/front-end/inspector.html:
3:05 AM Changeset in webkit [83961] by Philippe Normand
  • 2 edits
    1 copy in trunk/LayoutTests

2011-04-15 Philippe Normand <pnormand@igalia.com>

Unreviewed, GTK platform results for
fast/css/font-face-repeated-url.html

  • fast/css/font-face-repeated-url-expected.txt:
  • platform/gtk/fast/css/font-face-repeated-url-expected.txt: Copied from LayoutTests/fast/css/font-face-repeated-url-expected.txt.
2:28 AM Changeset in webkit [83960] by caseq@chromium.org
  • 8 edits in trunk

2011-04-15 Andrey Kosyakov <caseq@chromium.org>

Unreviewed, rolling out r83949.
http://trac.webkit.org/changeset/83949
https://bugs.webkit.org/show_bug.cgi?id=57960

broke 31 tests in chromium win & linux

  • platform/chromium/test_expectations.txt:

2011-04-15 Andrey Kosyakov <caseq@chromium.org>

Unreviewed, rolling out r83949.
http://trac.webkit.org/changeset/83949
https://bugs.webkit.org/show_bug.cgi?id=57960

broke 31 tests in chromium win & linux

  • platform/graphics/chromium/GLES2Canvas.cpp: (WebCore::GLES2Canvas::drawTexturedRect):
  • platform/graphics/chromium/GLES2Canvas.h:
  • platform/graphics/gpu/Texture.cpp: (WebCore::copySubRect): (WebCore::Texture::load): (WebCore::Texture::updateSubRect):
  • platform/graphics/gpu/Texture.h:
  • platform/graphics/skia/ImageBufferSkia.cpp: (WebCore::getImageData): (WebCore::putImageData): (WebCore::ImageBuffer::putUnmultipliedImageData): (WebCore::ImageBuffer::putPremultipliedImageData):
1:50 AM Changeset in webkit [83959] by Philippe Normand
  • 3 edits in trunk/LayoutTests

2011-04-15 Philippe Normand <pnormand@igalia.com>

Unreviewed, GTK rebaseline of two tests.

  • fast/css/font-face-repeated-url-expected.txt:
  • platform/gtk/media/media-document-audio-repaint-expected.txt:
1:10 AM Changeset in webkit [83958] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit/chromium

2011-04-15 Vsevolod Vlasov <vsevik@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: [Chromium] Create tests for network panel timing, size and raw headers text
https://bugs.webkit.org/show_bug.cgi?id=58566

Added network timing, size and raw headers tests for Inspector
Since chromium used different version of network stack for layout
tests, these are interactive ui tests (DevToolsSanityTest.TestNetwork*)

  • src/js/Tests.js: (.TestSuite.prototype.testNetworkSize.finishResource): (.TestSuite.prototype.testNetworkSyncSize.finishResource): (.TestSuite.prototype.testNetworkRawHeadersText.finishResource): (.TestSuite.prototype.testNetworkTiming.finishResource):
12:55 AM Changeset in webkit [83957] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-04-15 Ben Taylor <bentaylor.solx86@gmail.com>

Reviewed by Alexey Proskuryakov.

Fix building with Sun Studio 12: function pointers for extern "C" are treated differently
https://bugs.webkit.org/show_bug.cgi?id=58508

Since extern "C" makes a different type (although most compilers ignore that),
we should be more careful when passing NPAPI callback functions.

  • plugins/npapi.cpp: (NPN_PluginThreadAsyncCall):
12:54 AM Changeset in webkit [83956] by eric@webkit.org
  • 3 edits in trunk/Tools

2011-04-15 Eric Seidel <eric@webkit.org>

Reviewed by Mihai Parparita.

queues.webkit.org should display when a bot last rebooted
https://bugs.webkit.org/show_bug.cgi?id=58562

There is more repeated code here than I would like. I fear
my django-fu isn't quite up to snuff.

  • QueueStatusServer/handlers/queuestatus.py:
  • QueueStatusServer/templates/queuestatus.html:
12:43 AM Changeset in webkit [83955] by Philippe Normand
  • 191 edits in trunk

2011-04-15 Philippe Normand <pnormand@igalia.com>

Unreviewed, rollout r83894 r83827 r83810 r83809 r83808.
r83808 and its follow-up commits broke GTK Release builds.
https://bugs.webkit.org/show_bug.cgi?id=58483

12:39 AM Changeset in webkit [83954] by abarth@webkit.org
  • 9 edits
    8 adds in trunk

2011-04-15 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

CSP should block string arguments to setTimeout and setInterval unless options eval-script
https://bugs.webkit.org/show_bug.cgi?id=58610

It's somewhat sadness that the JSC and V8 code for setTimeout and
setInterval are so different. I struggled for a while with how to
handle the worker case, but I decided to punt on it for now.

Tests: http/tests/security/contentSecurityPolicy/eval-scripts-setInterval-allowed.html

http/tests/security/contentSecurityPolicy/eval-scripts-setInterval-blocked.html
http/tests/security/contentSecurityPolicy/eval-scripts-setTimeout-allowed.html
http/tests/security/contentSecurityPolicy/eval-scripts-setTimeout-blocked.html

  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::setTimeout): (WebCore::JSDOMWindow::setInterval):
  • bindings/js/JSWorkerContextCustom.cpp: (WebCore::JSWorkerContext::setTimeout): (WebCore::JSWorkerContext::setInterval):
  • bindings/js/ScheduledAction.cpp: (WebCore::ScheduledAction::create):
  • bindings/js/ScheduledAction.h:
  • bindings/v8/custom/V8DOMWindowCustom.cpp: (WebCore::WindowSetTimeoutImpl):
  • page/ContentSecurityPolicy.cpp: (WebCore::ContentSecurityPolicy::allowEval):
  • page/ContentSecurityPolicy.h:

2011-04-15 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

CSP should block string arguments to setTimeout and setInterval unless options eval-script
https://bugs.webkit.org/show_bug.cgi?id=58610

Herein lies a four-way testing matrix.

  • http/tests/security/contentSecurityPolicy/eval-scripts-setInterval-allowed-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/eval-scripts-setInterval-allowed.html: Added.
  • http/tests/security/contentSecurityPolicy/eval-scripts-setInterval-blocked-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/eval-scripts-setInterval-blocked.html: Added.
  • http/tests/security/contentSecurityPolicy/eval-scripts-setTimeout-allowed-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/eval-scripts-setTimeout-allowed.html: Added.
  • http/tests/security/contentSecurityPolicy/eval-scripts-setTimeout-blocked-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/eval-scripts-setTimeout-blocked.html: Added.
12:37 AM Changeset in webkit [83953] by abarth@webkit.org
  • 3 edits
    2 adds in trunk

2011-04-15 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Add support for CSP's 'self' source
https://bugs.webkit.org/show_bug.cgi?id=58604

  • http/tests/security/contentSecurityPolicy/script-src-self-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/script-src-self.html: Added.

2011-04-15 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Add support for CSP's 'self' source
https://bugs.webkit.org/show_bug.cgi?id=58604

This change is now trivially easy.

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

  • page/ContentSecurityPolicy.cpp: (WebCore::CSPSourceList::addSourceSelf):
12:13 AM Changeset in webkit [83952] by commit-queue@webkit.org
  • 18 edits in trunk

2011-04-15 Anna Cavender <annacc@chromium.org>

Reviewed by Eric Carlson.

Renaming TRACK feature define to VIDEO_TRACK
https://bugs.webkit.org/show_bug.cgi?id=53556

  • configure.ac:

2011-04-15 Anna Cavender <annacc@chromium.org>

Reviewed by Eric Carlson.

Renaming TRACK feature define to VIDEO_TRACK
https://bugs.webkit.org/show_bug.cgi?id=53556

  • Configurations/FeatureDefines.xcconfig:

2011-04-15 Anna Cavender <annacc@chromium.org>

Reviewed by Eric Carlson.

Renaming TRACK feature define to VIDEO_TRACK
https://bugs.webkit.org/show_bug.cgi?id=53556

No new tests. No new functionality.

  • CMakeLists.txt:
  • Configurations/FeatureDefines.xcconfig:
  • DerivedSources.make:
  • GNUmakefile.am:
  • features.pri:
  • html/HTMLTagNames.in:
  • html/HTMLTrackElement.cpp:
  • html/HTMLTrackElement.h:
  • html/HTMLTrackElement.idl:

2011-04-15 Anna Cavender <annacc@chromium.org>

Reviewed by Eric Carlson.

Renaming TRACK feature define to VIDEO_TRACK
https://bugs.webkit.org/show_bug.cgi?id=53556

  • Configurations/FeatureDefines.xcconfig:

2011-04-15 Anna Cavender <annacc@chromium.org>

Reviewed by Eric Carlson.

Renaming TRACK feature define to VIDEO_TRACK
https://bugs.webkit.org/show_bug.cgi?id=53556

  • Configurations/FeatureDefines.xcconfig:
Note: See TracTimeline for information about the timeline view.