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

Timeline



May 16, 2010:

11:10 PM Changeset in webkit [59594] by ukai@chromium.org
  • 2 edits in trunk/LayoutTests

Unreviewed

Patch by Takeshi Yoshino <​tyoshino@google.com> on 2010-05-16
Update test_expectations.txt for Chromium
fast/loader/recursive-before-unload-crash.html is flaky on Linux and Win
See ​http://code.google.com/p/chromium/issues/detail?id=44229 for detail.

  • platform/chromium/test_expectations.txt:
11:10 PM Changeset in webkit [59593] by dbates@webkit.org
  • 3 edits
    1 add in trunk/WebKitTools

2010-05-16 Daniel Bates <​dbates@rim.com>

Reviewed by Chris Jerdonek.

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

Adds function VCSUtils::parseSvnProperty to parse an SVN property with
either a single-line or multi-line value change.

  • Scripts/VCSUtils.pm:
    • Added function parseSvnProperty. We will use this function towards resolving Bug #38885 <​https://bugs.webkit.org/show_bug.cgi?id=38885>.
    • Removed FIXME comment above function parseSvnPropertyValue, since it is being used by parseSvnProperty.
    • Modified function parseSvnPropertyValue to break out of "while (<$fileHandle>)" loop when it encounters the start of the next property so that it can be processed by its caller, parseSvnPropertyValue. We reference this bullet below by (*).
  • Scripts/webkitperl/VCSUtils_unittest/parseSvnProperty.pl: Added.
    • Added unit tests.
  • Scripts/webkitperl/VCSUtils_unittest/parseSvnPropertyValue.pl:
    • Changed the name of the unit test "simple multi-line '-' change" to "single-line '-' change followed by empty line" since the former was an incorrect description of this test.
    • Added unit test "single-line '-' change followed by the next property", and "multi-line '-' change followed by the next property" to test (*) above.
9:58 PM Changeset in webkit [59592] by dbates@webkit.org
  • 23 edits
    1 add in trunk/WebCore

2010-05-16 Daniel Bates <​dbates@rim.com>

Reviewed by David Hyatt.

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

Rollout changeset 59274 <​http://trac.webkit.org/changeset/59274>.

Prepared on behalf of David Hyatt.

"Not worth the memory use." Just roll this out.

  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • editing/visible_units.cpp: (WebCore::previousLinePosition): (WebCore::nextLinePosition):
  • rendering/EllipsisBox.cpp: (WebCore::EllipsisBox::paint): (WebCore::EllipsisBox::selectionRect): (WebCore::EllipsisBox::paintSelection): (WebCore::EllipsisBox::nodeAtPoint):
  • rendering/EllipsisBox.h: (WebCore::EllipsisBox::EllipsisBox): (WebCore::EllipsisBox::height):
  • rendering/InlineBox.cpp: (WebCore::InlineBox::height): (WebCore::InlineBox::adjustPosition): (WebCore::InlineBox::canAccommodateEllipsis):
  • rendering/InlineBox.h: (WebCore::InlineBox::InlineBox): (WebCore::InlineBox::isInlineBox): (WebCore::InlineBox::hasVirtualHeight): (WebCore::InlineBox::setHasVirtualHeight): (WebCore::InlineBox::virtualHeight): (WebCore::InlineBox::setWidth): (WebCore::InlineBox::width): (WebCore::InlineBox::setX): (WebCore::InlineBox::x): (WebCore::InlineBox::setY): (WebCore::InlineBox::y): (WebCore::InlineBox::baselinePosition): (WebCore::InlineBox::lineHeight):
  • rendering/InlineFlowBox.cpp: (WebCore::InlineFlowBox::placeBoxesVertically): (WebCore::InlineFlowBox::computeVerticalOverflow): (WebCore::InlineFlowBox::nodeAtPoint): (WebCore::InlineFlowBox::paintBoxDecorations): (WebCore::InlineFlowBox::paintMask): (WebCore::InlineFlowBox::paintTextDecorations):
  • rendering/InlineFlowBox.h: (WebCore::InlineFlowBox::visibleOverflowRect): (WebCore::InlineFlowBox::topLayoutOverflow): (WebCore::InlineFlowBox::bottomLayoutOverflow): (WebCore::InlineFlowBox::leftLayoutOverflow): (WebCore::InlineFlowBox::rightLayoutOverflow): (WebCore::InlineFlowBox::layoutOverflowRect): (WebCore::InlineFlowBox::topVisualOverflow): (WebCore::InlineFlowBox::bottomVisualOverflow): (WebCore::InlineFlowBox::leftVisualOverflow): (WebCore::InlineFlowBox::rightVisualOverflow): (WebCore::InlineFlowBox::visualOverflowRect): (WebCore::InlineFlowBox::setHorizontalOverflowPositions): (WebCore::InlineFlowBox::setVerticalOverflowPositions):
  • rendering/InlineTextBox.cpp: (WebCore::InlineTextBox::selectionRect): (WebCore::InlineTextBox::placeEllipsisBox): (WebCore::InlineTextBox::nodeAtPoint): (WebCore::InlineTextBox::paint): (WebCore::InlineTextBox::paintSelection): (WebCore::InlineTextBox::paintCompositionBackground): (WebCore::InlineTextBox::paintDecoration): (WebCore::InlineTextBox::paintSpellingOrGrammarMarker): (WebCore::InlineTextBox::paintTextMatchMarker): (WebCore::InlineTextBox::computeRectForReplacementMarker): (WebCore::InlineTextBox::paintCompositionUnderline): (WebCore::InlineTextBox::offsetForPosition): (WebCore::InlineTextBox::positionForOffset):
  • rendering/InlineTextBox.h: (WebCore::InlineTextBox::setSpaceAdd):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::positionForPointWithInlineChildren):
  • rendering/RenderBlockLineLayout.cpp: (WebCore::RenderBlock::computeVerticalPositionsForLine): (WebCore::RenderBlock::layoutInlineChildren):
  • rendering/RenderSVGInline.cpp: (WebCore::RenderSVGInline::createInlineFlowBox):
  • rendering/RenderSVGInlineText.cpp: (WebCore::RenderSVGInlineText::createTextBox):
  • rendering/RenderSVGText.cpp: (WebCore::RenderSVGText::createRootInlineBox):
  • rendering/RenderText.cpp: (WebCore::RenderText::positionForPoint): (WebCore::RenderText::firstRunX): (WebCore::RenderText::firstRunY):
  • rendering/RenderTreeAsText.cpp: (WebCore::writeTextRun):
  • rendering/RootInlineBox.cpp: (WebCore::RootInlineBox::placeEllipsis): (WebCore::RootInlineBox::addHighlightOverflow): (WebCore::RootInlineBox::closestLeafChildForXPos):
  • rendering/SVGInlineFlowBox.h: (WebCore::SVGInlineFlowBox::SVGInlineFlowBox): (WebCore::SVGInlineFlowBox::virtualHeight): (WebCore::SVGInlineFlowBox::setHeight):
  • rendering/SVGInlineTextBox.h: (WebCore::SVGInlineTextBox::virtualHeight): (WebCore::SVGInlineTextBox::setHeight): (WebCore::SVGInlineTextBox::selectionTop): (WebCore::SVGInlineTextBox::selectionHeight):
  • rendering/SVGRootInlineBox.h: (WebCore::SVGRootInlineBox::SVGRootInlineBox): (WebCore::SVGRootInlineBox::virtualHeight): (WebCore::SVGRootInlineBox::setHeight):
  • rendering/TrailingFloatsRootInlineBox.h: Added. (WebCore::TrailingFloatsRootInlineBox::TrailingFloatsRootInlineBox): (WebCore::TrailingFloatsRootInlineBox::virtualHeight):
9:08 PM Changeset in webkit [59591] by tony@chromium.org
  • 3 edits
    2 adds in trunk

2010-05-13 Tony Chang <​tony@chromium.org>

Reviewed by Darin Adler.

REGRESSION: Crash by pasting to a textarea with white-space:nowrap
​https://bugs.webkit.org/show_bug.cgi?id=38992

  • editing/pasteboard/paste-plaintext-nowrap-expected.txt: Added.
  • editing/pasteboard/paste-plaintext-nowrap.html: Added.

2010-05-13 Tony Chang <​tony@chromium.org>

Reviewed by Darin Adler.

REGRESSION: Crash by pasting to a textarea with white-space:nowrap
​https://bugs.webkit.org/show_bug.cgi?id=38992

Test: editing/pasteboard/paste-plaintext-nowrap.html

  • editing/InsertParagraphSeparatorCommand.cpp: (WebCore::highestVisuallyEquivalentDivBelowRoot): (WebCore::InsertParagraphSeparatorCommand::doApply):
8:34 PM Changeset in webkit [59590] by hyatt@apple.com
  • 2 edits
    2 adds in trunk

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

Reviewed by Dan Bernstein.

Fix all of the multi-column painting and hit testing functions to properly account for RTL directionality.

Added fast/multicol/column-count-with-rules.html

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::paintColumnRules):
(WebCore::RenderBlock::paintColumnContents):
(WebCore::RenderBlock::hitTestColumns):
(WebCore::RenderBlock::adjustRectForColumns):

8:03 PM Changeset in webkit [59589] by mitz@apple.com
  • 5 edits in trunk/WebCore

Fix SVG test failures after r59588.

Reviewed by Dave Hyatt.

Updated SVG InlineBox virtual method overrides to match HTML ones. Removed
SVGInlineFlowBox::verticallyAlignBoxes() since InlineFlowBox no longer declares
verticallyAlignBoxes().

  • rendering/SVGInlineFlowBox.cpp:

(WebCore::SVGInlineFlowBox::placeBoxesHorizontally):

  • rendering/SVGInlineFlowBox.h:
  • rendering/SVGRootInlineBox.cpp:

(WebCore::SVGRootInlineBox::placeBoxesHorizontally):
(WebCore::SVGRootInlineBox::verticallyAlignBoxes):

  • rendering/SVGRootInlineBox.h:
6:43 PM Changeset in webkit [59588] by mitz@apple.com
  • 10 edits in trunk/WebCore

Move transient glyph overflow and fallback font data from a global to the stack
​https://bugs.webkit.org/show_bug.cgi?id=39050

Reviewed by Darin Adler.

  • rendering/InlineFlowBox.cpp:

(WebCore::InlineFlowBox::placeBoxesHorizontally): Added a GlyphOverflowAndFallbackFontsMap
parameter. Read glyph overflow from the map.
(WebCore::InlineFlowBox::computeLogicalBoxHeights): Added a GlyphOverflowAndFallbackFontsMap
parameter. Read fallback fonts from the map.
(WebCore::InlineFlowBox::computeVerticalOverflow): Added a GlyphOverflowAndFallbackFontsMap
parameter. Read glyph overflow from the map.

  • rendering/InlineFlowBox.h:
  • rendering/InlineTextBox.cpp: Removed setFallbackFonts(), fallbackFonts(), setGlyphOverflow(),

and glyphOverflow(). Removed s_glyphOverflowAndFallbackFontsMap.

  • rendering/InlineTextBox.h:
  • rendering/RenderBlock.h:
  • rendering/RenderBlockLineLayout.cpp:

(WebCore::RenderBlock::computeHorizontalPositionsForLine): Added a GlyphOverflowAndFallbackFontsMap
parameter. Write fallback fonts and glyph overflow into the map.
(WebCore::RenderBlock::computeVerticalPositionsForLine): Added a GlyphOverflowAndFallbackFontsMap
parameter. Pass it down to verticallyAlignBoxes().
(WebCore::RenderBlock::layoutInlineChildren): Create a GlyphOverflowAndFallbackFontsMap on the
stack and pass it down to computeHorizontalPositionsForLine(), computeVerticalPositionsForLine(),
and verticallyAlignBoxes().

  • rendering/RootInlineBox.cpp:

(WebCore::RootInlineBox::verticallyAlignBoxes): Added a GlyphOverflowAndFallbackFontsMap parameter.
Pass it down to computeLogicalBoxHeights() and computeVerticalOverflow().

  • rendering/RootInlineBox.h:
  • rendering/SVGRootInlineBox.cpp:

(WebCore::SVGRootInlineBox::layoutInlineBoxes): Updated.

6:02 PM Changeset in webkit [59587] by Simon Fraser
  • 2 edits in trunk/JavaScriptCore

2010-05-16 Simon Fraser <Simon Fraser>

Reviewed by Darin Adler.

Reduce the size of FunctionExecutable
​https://bugs.webkit.org/show_bug.cgi?id=39180

Change m_numVariables from a size_t to 31 bits in a bitfield,
packed with another bit for m_forceUsesArguments (which in turn
get packed with the base class).

Reduces the size of FunctionExecutable from 160 to 152 bytes.

  • runtime/Executable.h: (JSC::FunctionExecutable::variableCount): (JSC::FunctionExecutable::FunctionExecutable):
6:00 PM Changeset in webkit [59586] by tony@chromium.org
  • 4 edits
    2 copies
    3 adds in trunk

2010-05-16 Tony Chang <​tony@chromium.org>

Not reviewed, fixing layout test.

Add tiger and SL results. Remove Inspect Element from leopard results.

  • platform/mac-leopard/editing/spelling/context-menu-suggestions-expected.txt:
  • platform/mac-snowleopard/editing/spelling/context-menu-suggestions-expected.txt: Added.
  • platform/mac-tiger/editing/spelling/context-menu-suggestions-expected.txt: Added.

2010-05-16 Tony Chang <​tony@chromium.org>

Not reviewed, fixing layout test.

Don't output Inspect Element since this is not enabled on the bots,
but most developers probably have it installed.

  • DumpRenderTree/mac/EventSendingController.mm: (-[EventSendingController contextClick:]):
5:26 PM Changeset in webkit [59585] by tony@chromium.org
  • 9 edits
    3 adds in trunk

2010-05-12 Tony Chang <​tony@chromium.org>

Reviewed by Darin Adler.

Spellcheck disabling does not disable context menu
​https://bugs.webkit.org/show_bug.cgi?id=25639

  • editing/spelling/context-menu-suggestions.html: Added.
  • platform/chromium/test_expectations.txt:
  • platform/gtk/Skipped:
  • platform/mac-leopard/editing/spelling/context-menu-suggestions-expected.txt: Added.
  • platform/qt/Skipped:
  • platform/win/Skipped:

2010-05-12 Tony Chang <​tony@chromium.org>

Reviewed by Darin Adler.

Spellcheck disabling does not disable context menu
​https://bugs.webkit.org/show_bug.cgi?id=25639

Test: editing/spelling/context-menu-suggestions.html

  • platform/ContextMenu.cpp: (WebCore::ContextMenu::populate):

2010-05-12 Tony Chang <​tony@chromium.org>

Reviewed by Darin Adler.

Spellcheck disabling does not disable context menu
​https://bugs.webkit.org/show_bug.cgi?id=25639

  • DumpRenderTree/mac/EventSendingController.mm: (+[EventSendingController isSelectorExcludedFromWebScript:]): (+[EventSendingController webScriptNameForSelector:]): (-[EventSendingController contextClick:]): add a bool parameter that

when true, dumps the context menu items to stdout.

4:48 PM Changeset in webkit [59584] by crogers@google.com
  • 6 edits in branches/audio/WebCore/audio

compile fixes for Snow Leopard

4:42 PM Changeset in webkit [59583] by weinig@apple.com
  • 2 edits in trunk/WebCore

​https://bugs.webkit.org/show_bug.cgi?id=39190
Shave a word off of GlyphPageTreeNode
<rdar://problem/7990428>

Reviewed by Dan Bernstein.

  • platform/graphics/GlyphPageTreeNode.h:

(WebCore::GlyphPageTreeNode::GlyphPageTreeNode):

2:46 PM Changeset in webkit [59582] by eric@webkit.org
  • 5 edits
    3 adds in trunk

2010-05-16 Andreas Kling <​andreas.kling@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

Add a test to verify proper handling of invalid arguments to getImageData().
Updated old putImageData() test with correct expectations.

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

Spec link:
​http://www.whatwg.org/specs/web-apps/current-work/#pixel-manipulation

  • fast/canvas/canvas-getImageData-invalid-expected.txt: Added.
  • fast/canvas/canvas-getImageData-invalid.html: Added.
  • fast/canvas/canvas-putImageData-expected.txt:
  • fast/canvas/canvas-putImageData.js:
  • fast/canvas/script-tests/canvas-getImageData-invalid.js: Added.

2010-05-16 Andreas Kling <​andreas.kling@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

Properly handle invalid arguments to CanvasRenderingContext2D's getImageData() and putImageData().
Both should throw NOT_SUPPORTED_ERR when called with nonfinite arguments.
getImageData() should throw INDEX_SIZE_ERR if either width or height is 0.

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

Spec link:
​http://www.whatwg.org/specs/web-apps/current-work/#pixel-manipulation

Test: fast/canvas/canvas-getImageData-invalid.html

  • html/canvas/CanvasRenderingContext2D.cpp: (WebCore::CanvasRenderingContext2D::createImageData): (WebCore::CanvasRenderingContext2D::getImageData): (WebCore::CanvasRenderingContext2D::putImageData):
2:08 PM Changeset in webkit [59581] by eric@webkit.org
  • 3 edits
    4 adds in trunk

2010-05-16 Andreas Kling <​andreas.kling@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

Add a test to verify that Canvas's toDataURL() is case insensitive wrt the mimeType argument.
​https://bugs.webkit.org/show_bug.cgi?id=39153

Spec link:
​http://www.whatwg.org/specs/web-apps/current-work/#dom-canvas-todataurl

  • fast/canvas/canvas-toDataURL-case-insensitive-mimetype-expected.txt: Added.
  • fast/canvas/canvas-toDataURL-case-insensitive-mimetype.html: Added.
  • fast/canvas/script-tests/canvas-toDataURL-case-insensitive-mimetype.js: Added. (tryMimeType):
  • platform/gtk/fast/canvas/canvas-toDataURL-case-insensitive-mimetype-expected.txt: Added.

2010-05-16 Andreas Kling <​andreas.kling@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

Canvas's toDataURL() should be case insensitive wrt the mimeType argument.
​https://bugs.webkit.org/show_bug.cgi?id=39153

Spec link:
​http://www.whatwg.org/specs/web-apps/current-work/#dom-canvas-todataurl

Test: fast/canvas/canvas-toDataURL-case-insensitive-mimetype.html

  • dom/CanvasSurface.cpp: (WebCore::CanvasSurface::toDataURL):
1:13 PM Changeset in webkit [59580] by robert@webkit.org
  • 14 edits in trunk

[Qt] Fix FrameLoaderClientQt.cpp to print relative filename paths correctly.

2010-05-16 Robert Hogan <​robert@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

Fix FrameLoaderClientQt.cpp to print relative filename paths correctly.

Move qt_* functions in FrameLoaderClientQt, EditorClientQt, and NotificationPresenterClientQt
to DumpRenderTreeSupportQt. Create new functions to replace them in DumpRenderTreeSupportQt.
Support for the old function names is retained until confirmed that no one uses them outside
QtWebKit.

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

  • WebCoreSupport/DumpRenderTreeSupportQt.cpp: (DumpRenderTreeSupportQt::dumpFrameLoader): (DumpRenderTreeSupportQt::dumpResourceLoadCallbacks): (DumpRenderTreeSupportQt::dumpResourceLoadCallbacksPath): (DumpRenderTreeSupportQt::setWillSendRequestReturnsNullOnRedirect): (DumpRenderTreeSupportQt::setWillSendRequestReturnsNull): (DumpRenderTreeSupportQt::setWillSendRequestClearHeaders): (DumpRenderTreeSupportQt::dumpEditingCallbacks): (DumpRenderTreeSupportQt::dumpSetAcceptsEditing): (DumpRenderTreeSupportQt::dumpNotification): (qt_dump_frame_loader): (qt_dump_resource_load_callbacks): (qt_dump_resource_load_callbacks_path): (qt_set_will_send_request_returns_null_on_redirect): (qt_set_will_send_request_returns_null): (qt_set_will_send_request_clear_headers): (qt_dump_editing_callbacks): (qt_dump_set_accepts_editing): (qt_dump_notification):
  • WebCoreSupport/DumpRenderTreeSupportQt.h:
  • WebCoreSupport/EditorClientQt.cpp:
  • WebCoreSupport/EditorClientQt.h:
  • WebCoreSupport/FrameLoaderClientQt.cpp: (drtDescriptionSuitableForTestResult):
  • WebCoreSupport/FrameLoaderClientQt.h:
  • WebCoreSupport/NotificationPresenterClientQt.cpp:
  • WebCoreSupport/NotificationPresenterClientQt.h:

Update Qt DRT to use frame loader, editor client and notificaiton presenter
functions in DumpRenderTreeSupportQt.

  • DumpRenderTree/qt/DumpRenderTreeQt.cpp: (WebCore::DumpRenderTree::open): (WebCore::DumpRenderTree::dump):
  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::LayoutTestController): (LayoutTestController::reset): (LayoutTestController::dumpEditingCallbacks): (LayoutTestController::dumpFrameLoadCallbacks): (LayoutTestController::dumpResourceLoadCallbacks): (LayoutTestController::setWillSendRequestReturnsNullOnRedirect): (LayoutTestController::setWillSendRequestReturnsNull): (LayoutTestController::setWillSendRequestClearHeader):

Unskip fast/loader/main-document-url-for-non-http-loads.html

  • platform/qt/Skipped:
11:01 AM Changeset in webkit [59579] by eric@webkit.org
  • 4 edits in trunk

2010-05-16 Mike Belshe <​mbelshe@chromium.org>

Reviewed by Dimitri Glazkov.

Add a flag to the ResourceResponse for tracking if a request was
fetched via the TLS/Next-Protocol-Negotiation mechanism.

Also cleaned up the style in ResourceResponse to group the flags
together (and alphabetically) and hopefully improve readability of
the file.

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

  • public/WebURLResponse.h:
  • src/WebURLResponse.cpp: (WebKit::WebURLResponse::wasNpnNegotiated): (WebKit::WebURLResponse::setWasNpnNegotiated):
10:50 AM Changeset in webkit [59578] by eric@webkit.org
  • 6 edits in trunk

2010-05-16 Eric Seidel <​eric@webkit.org>

Unreviewed, rolling out r59571.
​http://trac.webkit.org/changeset/59571
​https://bugs.webkit.org/show_bug.cgi?id=39054

Broke Cr Win, but we didn't notice immediately due to
​https://bugs.webkit.org/show_bug.cgi?id=38926. It's possible
that this didn't actually break Cr Win, but rather that bug
38926 necessitates a clean compile after this and sucessive
checkins only produced a partial recompile and thus failed to
build.

  • DEPS:
  • WebKit.gyp:
  • features.gypi:

2010-05-16 Eric Seidel <​eric@webkit.org>

Unreviewed, rolling out r59571.
​http://trac.webkit.org/changeset/59571
​https://bugs.webkit.org/show_bug.cgi?id=39054

Broke Cr Win, but we didn't notice immediately due to
​https://bugs.webkit.org/show_bug.cgi?id=38926. It's possible
that this didn't actually break Cr Win, but rather that bug
38926 necessitates a clean compile after this and sucessive
checkins only produced a partial recompile and thus failed to
build.

  • DumpRenderTree/chromium/WebViewHost.cpp: (WebViewHost::canvas):
10:24 AM Changeset in webkit [59577] by Simon Fraser
  • 2 edits in trunk/WebCore

2010-05-16 Simon Fraser <Simon Fraser>

Reviewed by Anders Carlsson.

Shrink SimpleFontData
​https://bugs.webkit.org/show_bug.cgi?id=39179

Change SimpleFontData to have a pointer to the m_glyphToBoundsMap,
and to allocate this lazily. This reduces the size of the class from
5632 to 1536 bytes.

  • platform/graphics/SimpleFontData.h: (WebCore::SimpleFontData::boundsForGlyph):
8:43 AM Changeset in webkit [59576] by Simon Fraser
  • 9 edits in trunk/WebCore

2010-05-15 Simon Fraser <Simon Fraser>

Reviewed by Maciej Stachowiak.

Reduce the size of CachedResource
​https://bugs.webkit.org/show_bug.cgi?id=39171

Make protected data members of CachedResource private, to allow for
easier rearrangement, and hide the fact that enums are in bitfields.

Rearrange the data members of CachedResource to save 32 bytes per instance
in 64-bit. Also modernized the constructor code.

  • loader/CachedCSSStyleSheet.cpp: (WebCore::CachedCSSStyleSheet::didAddClient): (WebCore::CachedCSSStyleSheet::data): (WebCore::CachedCSSStyleSheet::checkNotify): (WebCore::CachedCSSStyleSheet::error):
  • loader/CachedFont.cpp: (WebCore::CachedFont::load): (WebCore::CachedFont::didAddClient): (WebCore::CachedFont::data): (WebCore::CachedFont::ensureCustomFontData): (WebCore::CachedFont::ensureSVGFontData): (WebCore::CachedFont::checkNotify): (WebCore::CachedFont::error):
  • loader/CachedImage.cpp: (WebCore::CachedImage::CachedImage): (WebCore::CachedImage::load): (WebCore::CachedImage::didAddClient): (WebCore::CachedImage::allClientsRemoved): (WebCore::CachedImage::image): (WebCore::CachedImage::data): (WebCore::CachedImage::error): (WebCore::CachedImage::checkNotify): (WebCore::CachedImage::destroyDecodedData):
  • loader/CachedImage.h: (WebCore::CachedImage::stillNeedsLoad):
  • loader/CachedResource.cpp: (WebCore::CachedResource::CachedResource):
  • loader/CachedResource.h: (WebCore::CachedResource::type): (WebCore::CachedResource::preloadResult): (WebCore::CachedResource::status): (WebCore::CachedResource::setStatus): (WebCore::CachedResource::isLoaded): (WebCore::CachedResource::isLoading): (WebCore::CachedResource::setErrorOccurred):
  • loader/CachedScript.cpp: (WebCore::CachedScript::didAddClient): (WebCore::CachedScript::data): (WebCore::CachedScript::checkNotify): (WebCore::CachedScript::error):
  • loader/CachedXSLStyleSheet.cpp: (WebCore::CachedXSLStyleSheet::didAddClient): (WebCore::CachedXSLStyleSheet::data): (WebCore::CachedXSLStyleSheet::checkNotify): (WebCore::CachedXSLStyleSheet::error):
6:44 AM Changeset in webkit [59575] by eric@webkit.org
  • 5 edits in trunk

2010-05-16 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r59574.
​http://trac.webkit.org/changeset/59574
​https://bugs.webkit.org/show_bug.cgi?id=39176

Supposedly broke cr-win, but the errors seems somewhat
unrelated (Requested by abarth on #webkit).

  • platform/network/chromium/ResourceResponse.h: (WebCore::ResourceResponse::ResourceResponse): (WebCore::ResourceResponse::setSecurityInfo): (WebCore::ResourceResponse::isContentFiltered): (WebCore::ResourceResponse::setIsContentFiltered): (WebCore::ResourceResponse::setAppCacheID): (WebCore::ResourceResponse::setAppCacheManifestURL): (WebCore::ResourceResponse::setWasFetchedViaSPDY): (WebCore::ResourceResponse::setIsMultipartPayload): (WebCore::ResourceResponse::setResponseTime):

2010-05-16 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r59574.
​http://trac.webkit.org/changeset/59574
​https://bugs.webkit.org/show_bug.cgi?id=39176

Supposedly broke cr-win, but the errors seems somewhat
unrelated (Requested by abarth on #webkit).

  • public/WebURLResponse.h:
  • src/WebURLResponse.cpp:
1:58 AM Changeset in webkit [59574] by eric@webkit.org
  • 4 edits in trunk

2010-05-16 Mike Belshe <​mbelshe@chromium.org>

Reviewed by Dimitri Glazkov.

Add a flag to the ResourceResponse for tracking if a request was
fetched via the TLS/Next-Protocol-Negotiation mechanism.

Also cleaned up the style in ResourceResponse to group the flags
together (and alphabetically) and hopefully improve readability of
the file.

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

  • public/WebURLResponse.h:
  • src/WebURLResponse.cpp: (WebKit::WebURLResponse::wasNpnNegotiated): (WebKit::WebURLResponse::setWasNpnNegotiated):
1:55 AM Changeset in webkit [59573] by robert@webkit.org
  • 8 edits in trunk

[Qt]Unskip security/set-form-autocomplete-attribute.html

2010-05-16 Robert Hogan <​robert@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt]Unskip security/set-form-autocomplete-attribute.html

Add support for layoutTestController.elementDoesAutoCompleteForElementWithId().

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

  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::elementDoesAutoCompleteForElementWithId):
  • DumpRenderTree/qt/LayoutTestControllerQt.h:
  • WebCoreSupport/DumpRenderTreeSupportQt.cpp: (DumpRenderTreeSupportQt::elementDoesAutoCompleteForElementWithId):
  • WebCoreSupport/DumpRenderTreeSupportQt.h:
  • platform/qt/Skipped:
1:49 AM Changeset in webkit [59572] by Simon Hausmann
  • 2 edits in trunk/WebKit/qt

[Qt] Prospective Qt/Mac build fix.

The friend declaration of QGraphicsWidget confuses gcc and thinks that it's an implicit
forward declaration, moving the class into the WebCore namespace (WebCore::QGraphicsWidget).

The declaration seems useless as no method of QGraphicsWidget is going to call anything
in this class.

  • WebCoreSupport/PageClientQt.h:
1:47 AM Changeset in webkit [59571] by eric@webkit.org
  • 6 edits in trunk

2010-05-16 Kent Tamura <​tkent@chromium.org>

Reviewed by Dimitri Glazkov.

[DRT/Chromium] Fix repaint, WebGL, textarea tests
​https://bugs.webkit.org/show_bug.cgi?id=39054

  • DEPS: Update to Chromium r47248 to use a fix for CommandLine assertion and textAreaResizeCorner.png
  • WebKit.gyp: Add textAreaResizeCorner.png as a Mac resource
  • features.gypi: Enable ENABLE_3D_CANVAS.

2010-05-16 Kent Tamura <​tkent@chromium.org>

Reviewed by Dimitri Glazkov.

[DRT/Chromium] Fix repaint, WebGL, textarea tests
​https://bugs.webkit.org/show_bug.cgi?id=39054

  • DumpRenderTree/chromium/WebViewHost.cpp: (WebViewHost::canvas): Remove m_paintRect initialization in canvas(). This line updated m_paintRect unexpectedly during paintRect(). We don't need to initialize m_paintRect because show() does it.
1:10 AM Changeset in webkit [59570] by eric@webkit.org
  • 4 edits in trunk/WebCore

2010-05-16 Dan Bernstein <​mitz@apple.com>

Reviewed by Darin Adler.

Scalar types allocated on the heap due to misuse of DEFINE_STATIC_LOCAL
​https://bugs.webkit.org/show_bug.cgi?id=39086

Use global variables directly.

  • bindings/v8/ScriptCallStack.cpp: (WebCore::ScriptCallStack::createUtilityContext):
  • platform/ScrollView.cpp: (WebCore::ScrollView::wheelEvent):
  • platform/graphics/mac/ComplexTextController.cpp: (WebCore::shouldUseATSUIAPI):
12:58 AM Changeset in webkit [59569] by eric@webkit.org
  • 4 edits
    3 adds in trunk

2010-05-16 Andreas Kling <​andreas.kling@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

Add a test to check proper return value (null) from invalid canvas getContext() calls.

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

  • fast/canvas/canvas-getContext-invalid-expected.txt: Added.
  • fast/canvas/canvas-getContext-invalid.html: Added.
  • fast/canvas/script-tests/canvas-getContext-invalid.js: Added.

2010-05-16 Andreas Kling <​andreas.kling@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

Canvas's getContext() must return null when called with an invalid/unsupported parameter.
(HTML5 spec 4.8.11): ​http://www.whatwg.org/specs/web-apps/current-work/multipage/the-canvas-element.html#dom-canvas-getcontext

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

Test: fast/canvas/canvas-getContext-invalid.html

  • bindings/js/JSHTMLCanvasElementCustom.cpp: (WebCore::JSHTMLCanvasElement::getContext):
  • bindings/v8/custom/V8HTMLCanvasElementCustom.cpp: (WebCore::V8HTMLCanvasElement::getContextCallback):
12:44 AM Changeset in webkit [59568] by eric@webkit.org
  • 4 edits in trunk

2010-05-16 Sergio Villar Senin <​svillar@igalia.com>

Reviewed by Gustavo Noronha Silva.

Removed 5 http/history tests from Skipped file

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

  • platform/gtk/Skipped:

2010-05-16 Sergio Villar Senin <​svillar@igalia.com>

Reviewed by Gustavo Noronha Silva.

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

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (webViewLoadStatusNotified): (createWebView): added connection to notify::load-status and signal callback

May 15, 2010:

11:57 PM Changeset in webkit [59567] by eric@webkit.org
  • 3 edits
    3 deletes in trunk

2010-05-15 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r59565.
​http://trac.webkit.org/changeset/59565
​https://bugs.webkit.org/show_bug.cgi?id=39173

Broke GTK (Requested by abarth on #webkit).

  • fast/canvas/canvas-toDataURL-case-insensitive-mimetype-expected.txt: Removed.
  • fast/canvas/canvas-toDataURL-case-insensitive-mimetype.html: Removed.
  • fast/canvas/script-tests/canvas-toDataURL-case-insensitive-mimetype.js: Removed.

2010-05-15 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r59565.
​http://trac.webkit.org/changeset/59565
​https://bugs.webkit.org/show_bug.cgi?id=39173

Broke GTK (Requested by abarth on #webkit).

  • dom/CanvasSurface.cpp: (WebCore::CanvasSurface::toDataURL):
8:50 PM Changeset in webkit [59566] by dbates@webkit.org
  • 2 edits
    1 add in trunk/WebKitTools

2010-05-15 Daniel Bates <​dbates@rim.com>

Reviewed by Chris Jerdonek.

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

Add function parseSvnPropertyValue to parse single-line and multi-line
property values of an SVN property change.

  • Scripts/VCSUtils.pm:

Added function parseSvnPropertyValue. We will use this as part of
Bug #38885 <​https://bugs.webkit.org/show_bug.cgi?id=38885>.

  • Scripts/webkitperl/VCSUtils_unittest/parseSvnPropertyValue.pl: Added.
8:15 PM Changeset in webkit [59565] by eric@webkit.org
  • 3 edits
    3 adds in trunk

2010-05-15 Andreas Kling <​andreas.kling@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

Add a test to verify that Canvas's toDataURL() is case insensitive wrt the mimeType argument.
​https://bugs.webkit.org/show_bug.cgi?id=39153

Spec link:
​http://www.whatwg.org/specs/web-apps/current-work/#dom-canvas-todataurl

  • fast/canvas/canvas-toDataURL-case-insensitive-mimetype-expected.txt: Added.
  • fast/canvas/canvas-toDataURL-case-insensitive-mimetype.html: Added.
  • fast/canvas/script-tests/canvas-toDataURL-case-insensitive-mimetype.js: Added. (tryMimeType):

2010-05-15 Andreas Kling <​andreas.kling@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

Canvas's toDataURL() should be case insensitive wrt the mimeType argument.
​https://bugs.webkit.org/show_bug.cgi?id=39153

Spec link:
​http://www.whatwg.org/specs/web-apps/current-work/#dom-canvas-todataurl

Test: fast/canvas/canvas-toDataURL-case-insensitive-mimetype.html

  • dom/CanvasSurface.cpp: (WebCore::CanvasSurface::toDataURL):
8:03 PM Changeset in webkit [59564] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-05-15 Anders Bakken <​agbakken@gmail.com>

Reviewed by Kenneth Rohde Christiansen.

QObject::objectName() returns a QString and QtField::name()
creates a temporary QByteArray object and returns constData() from it.
This is not safe.

This patch changes the signature of the function to return a
QByteArray instead of const char *.

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

  • bridge/qt/qt_instance.cpp: (JSC::Bindings::QtInstance::getPropertyNames): (JSC::Bindings::QtField::name):
  • bridge/qt/qt_runtime.h:
7:52 PM Changeset in webkit [59563] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-05-15 Anders Bakken <​agbakken@gmail.com>

Reviewed by Kenneth Rohde Christiansen.

Don't unnecessarily copy data when searching for methods in QtClass.

[Qt] JSValue QtClass::fallbackObject can be optimized
​https://bugs.webkit.org/show_bug.cgi?id=37684

  • bridge/qt/qt_class.cpp: (JSC::Bindings::QtClass::fallbackObject):
7:34 PM Changeset in webkit [59562] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-05-15 Anders Bakken <​agbakken@gmail.com>

Reviewed by Kenneth Rohde Christiansen.

Identifier::ascii() is not threadsafe and shouldn't be used for
converting to and from QString/QByteArray.

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

  • bridge/qt/qt_class.cpp: (JSC::Bindings::QtClass::fallbackObject): (JSC::Bindings::QtClass::fieldNamed):
7:22 PM Changeset in webkit [59561] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-05-15 Chang Shu <​chang.shu@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

Based on the spec and Philip Taylor's test suite, createImageData() should throw
an INDEX_SIZE_ERR exception if either the sw or sh arguments are zero.
The link to the spec:
​http://philip.html5.org/tests/canvas/suite/tests/spec.html#testrefs.2d.imageData.getcreate.zero

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

The complete test suite is in the process of checking in.
​https://bugs.webkit.org/show_bug.cgi?id=20553

  • html/canvas/CanvasRenderingContext2D.cpp: (WebCore::CanvasRenderingContext2D::createImageData):
7:08 PM Changeset in webkit [59560] by eric@webkit.org
  • 4 edits in trunk/WebKit/chromium

2010-05-15 Darin Fisher <​darin@chromium.org>

Reviewed by Adam Barth.

[chromium] Expose element accessor on WebPluginContainer
​https://bugs.webkit.org/show_bug.cgi?id=39152

  • public/WebPluginContainer.h:
  • src/WebPluginContainerImpl.cpp: (WebKit::WebPluginContainerImpl::element):
  • src/WebPluginContainerImpl.h:
6:43 PM Changeset in webkit [59559] by eric@webkit.org
  • 4 edits in trunk/WebCore

2010-05-15 Adam Barth <​abarth@webkit.org>

Reviewed by Darin Adler.

XMLHttpRequest.getResponseHeader doesn't need to be custom
​https://bugs.webkit.org/show_bug.cgi?id=39125

I don't see any reason this method needs to be custom. No new tests
because there's no behavior change.

  • bindings/js/JSXMLHttpRequestCustom.cpp:
  • bindings/v8/custom/V8XMLHttpRequestCustom.cpp:
  • xml/XMLHttpRequest.idl:
5:30 PM Changeset in webkit [59558] by abarth@webkit.org
  • 3 edits in trunk/LayoutTests

2010-05-15 Adam Barth <​abarth@webkit.org>

Reviewed by David Hyatt.

Rebaseline client-rects.html on Qt
​https://bugs.webkit.org/show_bug.cgi?id=39169

  • platform/qt/Skipped:
  • platform/qt/fast/multicol/client-rects-expected.txt:
5:19 PM Changeset in webkit [59557] by eric@webkit.org
  • 14 edits in trunk/WebCore

2010-05-15 Adam Barth <​abarth@webkit.org>

Reviewed by Darin Adler.

XMLHttpRequest.setRequestHeader shouldn't be custom
​https://bugs.webkit.org/show_bug.cgi?id=39124

I don't see any reason why this method needs to be custom. No tests
because no behavior change.

  • bindings/js/JSXMLHttpRequestCustom.cpp:
  • bindings/scripts/CodeGeneratorJS.pm:
  • bindings/scripts/CodeGeneratorV8.pm:
  • bindings/scripts/test/GObject/WebKitDOMTestObj.cpp: (webkit_dom_test_obj_method_that_requires_all_args): (webkit_dom_test_obj_method_that_requires_all_args_and_throws):
  • bindings/scripts/test/GObject/WebKitDOMTestObj.h:
  • bindings/scripts/test/JS/JSTestObj.cpp: (WebCore::): (WebCore::jsTestObjPrototypeFunctionMethodThatRequiresAllArgs): (WebCore::jsTestObjPrototypeFunctionMethodThatRequiresAllArgsAndThrows):
  • bindings/scripts/test/JS/JSTestObj.h:
  • bindings/scripts/test/ObjC/DOMTestObj.h:
  • bindings/scripts/test/ObjC/DOMTestObj.mm: (-[DOMTestObj methodThatRequiresAllArgs:objArg:]): (-[DOMTestObj methodThatRequiresAllArgsAndThrows:objArg:]):
  • bindings/scripts/test/TestObj.idl:
  • bindings/scripts/test/V8/V8TestObj.cpp: (WebCore::TestObjInternal::methodThatRequiresAllArgsCallback): (WebCore::TestObjInternal::methodThatRequiresAllArgsAndThrowsCallback): (WebCore::ConfigureV8TestObjTemplate):
  • bindings/v8/custom/V8XMLHttpRequestCustom.cpp:
  • xml/XMLHttpRequest.idl:
3:33 PM Changeset in webkit [59556] by ap@apple.com
  • 3 edits
    2 adds in trunk

Reviewed by Darin Adler.

​https://bugs.webkit.org/show_bug.cgi?id=39048
<rdar://problem/7978384> REGRESSION (r55802): XML errors on Google maps ("Entity 'nbsp' not
defined") with JavaScript disabled

Google Maps goes into mobile mode if JavaScript is disabled for some reason, and sends XHTML
content with XHTML Mobile DOCTYPE. We want to handle it whether XHTMLMP is enabled or not.

Test: fast/doctypes/xhtml-with-xhtmlmp-doctype.xhtml

  • dom/XMLTokenizerLibxml2.cpp: (WebCore::externalSubsetHandler): Restore pre-55802 behavior for builds that don't have XHTMLMP enabled. Given that r55802 negated the condition without breaking XHTMLMP, this line is actually always needed.
3:29 PM Changeset in webkit [59555] by eric@webkit.org
  • 8 edits in trunk

2010-05-15 Jochen Eisinger <​jochen@chromium.org>

Reviewed by Dmitry Titov.

Implement canEstablishDatabase call for workers.
​https://bugs.webkit.org/show_bug.cgi?id=38742

  • public/WebCommonWorkerClient.h:
  • src/DatabaseObserver.cpp: (WebCore::DatabaseObserver::canEstablishDatabase):
  • src/WebWorkerBase.cpp: (WebKit::WebWorkerBase::allowDatabase): (WebKit::WebWorkerBase::allowDatabaseTask): (WebKit::WebWorkerBase::AllowDatabaseMainThreadBridge::AllowDatabaseMainThreadBridge): (WebKit::WebWorkerBase::AllowDatabaseMainThreadBridge::cancel): (WebKit::WebWorkerBase::AllowDatabaseMainThreadBridge::result): (WebKit::WebWorkerBase::AllowDatabaseMainThreadBridge::signalCompleted): (WebKit::WebWorkerBase::AllowDatabaseMainThreadBridge::didComplete):
  • src/WebWorkerBase.h:
  • src/WebWorkerClientImpl.h: (WebKit::WebWorkerClientImpl::allowDatabase):

2010-05-15 Jochen Eisinger <​jochen@chromium.org>

Reviewed by Dmitry Titov.

Add allowDatabase method to TestWebWorker.
​https://bugs.webkit.org/show_bug.cgi?id=38742

  • DumpRenderTree/chromium/TestWebWorker.h: (TestWebWorker::allowDatabase):
3:19 PM Changeset in webkit [59554] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-05-15 Vangelis Kokkevis <​vangelis@chromium.org>

Reviewed by Darin Fisher.

Fixing LayerChromium hierarchy updates where due to mixed ownership
handling, sublayer updates didn't always properly update parent-to-child
and child-to-parent pointers in sync.
​https://bugs.webkit.org/show_bug.cgi?id=39138

  • platform/graphics/chromium/LayerChromium.cpp: (WebCore::LayerChromium::~LayerChromium): (WebCore::LayerChromium::setNeedsCommit): Typo fix. (WebCore::LayerChromium::insertSublayer): (WebCore::LayerChromium::removeFromSuperlayer): (WebCore::LayerChromium::removeSublayer): (WebCore::LayerChromium::replaceSublayer): (WebCore::LayerChromium::removeAllSublayers): (WebCore::LayerChromium::setSublayers):
  • platform/graphics/chromium/LayerChromium.h: (WebCore::LayerChromium::setSuperlayer):
3:08 PM Changeset in webkit [59553] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-05-15 Andrey Kosyakov <​caseq@chromium.org>

Reviewed by Timothy Hatcher.

Got rid of absolute positioning in favor of flex boxes where possible.
Added shadow for help window.
Changed window style names to more comprehensible.
​https://bugs.webkit.org/show_bug.cgi?id=39120

  • inspector/front-end/HelpScreen.js: (WebInspector.HelpScreen):
  • inspector/front-end/helpScreen.css: (.help-window-outer): (body.attached .help-window-outer): (.help-window-main): (body.attached .help-window-main): (.help-window-caption): (body.attached .help-window-caption): (.help-window-title): (.help-scrollable): (.help-close-button): (body.platform-mac .help-close-button): (body:not(.platform-mac) .help-close-button):
1:29 PM Changeset in webkit [59552] by Simon Hausmann
  • 1 edit
    19 deletes in trunk/WebKitTools

[Qt] Rename QtLauncher to QtTestBrowser
​https://bugs.webkit.org/show_bug.cgi?id=37665

Rubber-stamped by Antti Koivisto.

Forgot to remove the original directory after the rename.

  • QtLauncher: Removed.
1:23 PM Changeset in webkit [59551] by eric@webkit.org
  • 4 edits in trunk/WebCore

2010-05-15 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r59549.
​http://trac.webkit.org/changeset/59549
​https://bugs.webkit.org/show_bug.cgi?id=39167

Broke Chromium build (Requested by abarth on #webkit).

  • bindings/scripts/CodeGeneratorV8.pm:
  • storage/IDBRequest.cpp: (WebCore::IDBRequest::suspend): (WebCore::IDBRequest::resume): (WebCore::IDBRequest::timerFired): (WebCore::IDBRequest::onEventCommon):
  • storage/IDBRequest.h:
1:03 PM Changeset in webkit [59550] by eric@webkit.org
  • 5 edits in trunk/WebCore

2010-05-15 Vangelis Kokkevis <​vangelis@chromium.org>

Reviewed by Darin Fisher.

Adding code path for composited layers (LayerChromium) containing only
an Image element.
​https://bugs.webkit.org/show_bug.cgi?id=39142

No new tests, covered under the existing tests for composited layers.

  • platform/graphics/chromium/GraphicsLayerChromium.cpp: (WebCore::GraphicsLayerChromium::setContentsToImage): (WebCore::GraphicsLayerChromium::updateContentsImage):
  • platform/graphics/chromium/GraphicsLayerChromium.h:
  • platform/graphics/chromium/LayerChromium.cpp: (WebCore::LayerChromium::LayerChromium): (WebCore::LayerChromium::setContents):
  • platform/graphics/chromium/LayerChromium.h: (WebCore::LayerChromium::contents):
12:53 PM Changeset in webkit [59549] by eric@webkit.org
  • 4 edits in trunk/WebCore

2010-05-15 Andrei Popescu <​andreip@google.com>

Reviewed by Jeremy Orlow.

IDBRequest is not a proper ActiveDomObject
​https://bugs.webkit.org/show_bug.cgi?id=39001

Add IDBRequest interface to the IsActiveDomType
routine of the CodeGeneratorV8.pm.

Rename IDBRequest::m_stopped to m_suspended and reset
the flag when resume() is called.

No new tests, Indexed Database isn't yet testable.

  • bindings/scripts/CodeGeneratorV8.pm:
  • storage/IDBRequest.cpp: (WebCore::IDBRequest::suspend): (WebCore::IDBRequest::resume): (WebCore::IDBRequest::timerFired): (WebCore::IDBRequest::onEventCommon):
  • storage/IDBRequest.h:
12:41 PM Changeset in webkit [59548] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-05-15 Andrei Bucur <​abucur@adobe.com>

Reviewed by Gustavo Noronha Silva.

Functions using the cairo_pattern_t object returned by
Pattern::createPlatformPattern sould destroy it after the painting
operation. Failing to do so causes memory leaks.

Test: No test was added because the patch adds no new functionality.
It just completes the normal life cycle of a cairo_pattern_t object.

  • platform/graphics/cairo/FontCairo.cpp: (WebCore::Font::drawGlyphs):
  • platform/graphics/cairo/GraphicsContextCairo.cpp: (WebCore::setPlatformFill): (WebCore::setPlatformStroke):
12:22 PM Changeset in webkit [59547] by oliver@apple.com
  • 7 edits in trunk

2010-05-15 Oliver Hunt <​oliver@apple.com>

Reviewed by Maciej Stachowiak.

Incorrect codegen for slowcase of < in 64-bit
​https://bugs.webkit.org/show_bug.cgi?id=39151

Call the correct stud for the slowcases of the < operator.

  • jit/JITArithmetic.cpp: (JSC::JIT::emitSlow_op_jnless): (JSC::JIT::emitSlow_op_jless):

2010-05-15 Oliver Hunt <​oliver@apple.com>

Reviewed by Maciej Stachowiak.

Incorrect codegen for slowcase of < in 64-bit
​https://bugs.webkit.org/show_bug.cgi?id=39151

Add yet more tests for the < operator when comparing integer strings.

  • fast/js/comparison-operators-greater-expected.txt:
  • fast/js/comparison-operators-less-expected.txt:
  • fast/js/script-tests/comparison-operators-greater.js:
  • fast/js/script-tests/comparison-operators-less.js:
11:56 AM Changeset in webkit [59546] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-05-15 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r59544.
​http://trac.webkit.org/changeset/59544
​https://bugs.webkit.org/show_bug.cgi?id=39165

Cased LayoutTest to start crashing (Requested by abarth on
#webkit).

  • Scripts/old-run-webkit-tests:
11:38 AM Changeset in webkit [59545] by eric@webkit.org
  • 2 edits in trunk/JavaScriptCore

2010-05-15 Leo Yang <​leo.yang@torchmobile.com.cn>

Reviewed by Darin Adler.

Fix bug ​https://bugs.webkit.org/show_bug.cgi?id=38890

  • runtime/DatePrototype.cpp: (JSC::formatLocaleDate):
11:26 AM Changeset in webkit [59544] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-05-15 Mario Sanchez Prada <​msanchez@igalia.com>

Reviewed by Xan Lopez.

Ensure DRT loads GAIL (Gtk+ module), for a11y tests
​https://bugs.webkit.org/show_bug.cgi?id=38648

Add the GTK_MODULES envvar (set to "gail") to the clean
environment when running DRT for the Gtk+ port

  • Scripts/old-run-webkit-tests:
10:34 AM Changeset in webkit [59543] by eric@webkit.org
  • 3 edits in trunk/WebKit/chromium

2010-05-15 Jay Civelli <​jcivelli@chromium.org>

Reviewed by Darin Fisher.

[chromium] Adding a notification on WebViewClient when a new
node gets focused.
​https://bugs.webkit.org/show_bug.cgi?id=38508

  • public/WebViewClient.h: (WebKit::WebViewClient::focusedNodeChanged):
  • src/ChromeClientImpl.cpp: (WebKit::ChromeClientImpl::focusedNodeChanged):
10:01 AM Changeset in webkit [59542] by yael.aharon@nokia.com
  • 5 edits in trunk/LayoutTests

Unreviewed.

Update expected results after 59541

  • platform/qt/fast/dom/Window/window-properties-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:
9:41 AM Changeset in webkit [59541] by yael.aharon@nokia.com
  • 26 edits
    16 adds in trunk

Initial support for HTMLMeterElement
​https://bugs.webkit.org/show_bug.cgi?id=38140

Reviewed by Kent Tamura.

WebCore:

This patch adds support for HTMLMeterElement, including default rendering
of the meter element. That is because some platforms do not have a native gauge.
The feature is turned on only for the Qt platform.

Tests: fast/dom/HTMLMeterElement/meter-element.html

fast/dom/HTMLMeterElement/set-meter-properties.html

  • DerivedSources.cpp:
  • DerivedSources.make:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pri:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • css/CSSPrimitiveValueMappings.h:

(WebCore::CSSPrimitiveValue::CSSPrimitiveValue):

  • css/CSSValueKeywords.in:
  • css/html.css:

(meter):
(meter::-webkit-meter):

  • html/HTMLAttributeNames.in:
  • html/HTMLElement.cpp:

(WebCore::inlineTagList):

  • html/HTMLElementsAllInOne.cpp:
  • html/HTMLMeterElement.cpp: Added.

(WebCore::HTMLMeterElement::HTMLMeterElement):
(WebCore::HTMLMeterElement::create):
(WebCore::HTMLMeterElement::createRenderer):
(WebCore::HTMLMeterElement::formControlType):
(WebCore::HTMLMeterElement::parseMappedAttribute):
(WebCore::HTMLMeterElement::min):
(WebCore::HTMLMeterElement::setMin):
(WebCore::HTMLMeterElement::max):
(WebCore::HTMLMeterElement::setMax):
(WebCore::HTMLMeterElement::value):
(WebCore::HTMLMeterElement::setValue):
(WebCore::HTMLMeterElement::low):
(WebCore::HTMLMeterElement::setLow):
(WebCore::HTMLMeterElement::high):
(WebCore::HTMLMeterElement::setHigh):
(WebCore::HTMLMeterElement::optimum):
(WebCore::HTMLMeterElement::setOptimum):

  • html/HTMLMeterElement.h: Added.

(WebCore::HTMLMeterElement::recalcWillValidate):

  • html/HTMLMeterElement.idl: Added.
  • html/HTMLTagNames.in:
  • page/DOMWindow.idl:
  • platform/ThemeTypes.h:
  • rendering/RenderMeter.cpp: Added.

(WebCore::RenderMeter::RenderMeter):
(WebCore::RenderMeter::layout):
(WebCore::RenderMeter::updateFromElement):

  • rendering/RenderMeter.h: Added.

(WebCore::RenderMeter::renderName):
(WebCore::RenderMeter::isMeter):
(WebCore::toRenderMeter):

  • rendering/RenderObject.h:

(WebCore::RenderObject::isMeter):

  • rendering/RenderTheme.cpp:

(WebCore::RenderTheme::adjustStyle):
(WebCore::RenderTheme::paint):
(WebCore::RenderTheme::paintBorderOnly):
(WebCore::RenderTheme::paintDecorations):
(WebCore::RenderTheme::adjustMeterStyle):
(WebCore::RenderTheme::paintMeter):

  • rendering/RenderTheme.h:

LayoutTests:

  • fast/dom/HTMLMeterElement: Added.
  • fast/dom/HTMLMeterElement/meter-element.html: Added.
  • fast/dom/HTMLMeterElement/script-tests: Added.
  • fast/dom/HTMLMeterElement/script-tests/TEMPLATE.html: Added.
  • fast/dom/HTMLMeterElement/script-tests/set-meter-properties.js: Added.
  • fast/dom/HTMLMeterElement/set-meter-properties-expected.txt: Added.
  • fast/dom/HTMLMeterElement/set-meter-properties.html: Added.
  • platform/chromium/test_expectations.txt:
  • platform/gtk/Skipped:
  • platform/mac/Skipped:
  • platform/qt/fast/dom/HTMLMeterElement: Added.
  • platform/qt/fast/dom/HTMLMeterElement/meter-element-expected.checksum: Added.
  • platform/qt/fast/dom/HTMLMeterElement/meter-element-expected.png: Added.
  • platform/qt/fast/dom/HTMLMeterElement/meter-element-expected.txt: Added.
  • platform/win/Skipped:
9:36 AM Changeset in webkit [59540] by eric@webkit.org
  • 10 edits
    3 copies
    5 adds in trunk

2010-05-15 Eric Uhrhane <​ericu@chromium.org>

Reviewed by Dmitry Titov.

Add bindings for async DB API in Workers.
​https://bugs.webkit.org/show_bug.cgi?id=34992

  • storage/change-version-handle-reuse.html: Refactored to share code with the worker version of the test.
  • storage/change-version-handle-reuse-expected.html: Removed reference to reloading, which this test doesn't do.
  • storage/change-version-handle-reuse.js: This is the extracted shared core of the test.
  • storage/execute-sql-args.html: Refactored to share code with the worker version of the test.
  • storage/execute-sql-args.js: This is the extracted shared core of the test.

These are the worker versions of the tests; they're tiny shims around shared code.

  • fast/workers/storage/execute-sql-args-worker-expected.txt: Added.
  • fast/workers/storage/execute-sql-args-worker.html: Added.
  • fast/workers/storage/change-version-handle-reuse-worker-expected.txt: Added.
  • fast/workers/storage/change-version-handle-reuse-worker.html: Added. This file gets included from worker database tests; it does all the boilerplate that's needed on the DOM side.
  • fast/workers/storage/resources/database-worker-controller.js: Added. This file is the boilerplate for the worker side of the database tests. It will import and run the actual test on command, and will report back completion and exceptions.
  • fast/workers/storage/resources/database-worker.js: Added.

2010-05-15 Eric Uhrhane <​ericu@chromium.org>

Reviewed by Dmitry Titov.

Add bindings for async DB API in Workers.
​https://bugs.webkit.org/show_bug.cgi?id=34992

Tests: storage/change-version-handle-reuse-worker.html

storage/execute-sql-args-worker.html

  • bindings/js/JSWorkerContextCustom.cpp: Add openDatabase binding. (WebCore::JSWorkerContext::openDatabase):
  • bindings/v8/custom/V8WorkerContextCustom.cpp: Add openDatabase stub; Chromium will need work both in V8 and in the browser process before we can turn this on there. (WebCore::V8WorkerContext::openDatabaseCallback):

Add NoStaticTables flags to all objects now shared with workers.

  • storage/Database.idl:
  • storage/SQLError.idl:
  • storage/SQLResultSet.idl:
  • storage/SQLResultSetRowList.idl:
  • storage/SQLTransaction.idl:
  • workers/WorkerContext.h: Add databaseExceededQuota.
  • workers/WorkerContext.cpp: (WebCore::WorkerContext::databaseExceededQuota): Add stub implementation for testing; you just get 5MB for now. (WebCore::WorkerContext::openDatabase): Remove invalid assertion.

Add the IDL for the call to openDatabase.

  • workers/WorkerContext.idl:
7:00 AM Changeset in webkit [59539] by eric@webkit.org
  • 3 edits in trunk/WebKit/qt

2010-05-15 Charles Wei <​charles.wei@torchmobile.com.cn>

Reviewed by Simon Hausmann.

This patch fixes the build failure of WebKit on Linux for Qt4.5
​https://bugs.webkig.org/show_bug.cgi?id=39108

  • Api/qgraphicswebview.cpp: (QGraphicsWebViewPrivate::syncLayers):
  • WebCoreSupport/QtPlatformPlugin.cpp:
6:49 AM Changeset in webkit [59538] by eric@webkit.org
  • 1 edit
    1 add in trunk/WebKitTools

2010-05-15 Eric Seidel <​eric@webkit.org>

Reviewed by Maciej Stachowiak.

Add script to check for minimum python version and install if missing on Tiger
​https://bugs.webkit.org/show_bug.cgi?id=38886

Per Maciej's request on webkit-dev:
​https://lists.webkit.org/pipermail/webkit-dev/2010-May/012785.html
provide a script which can automatically install Python on Tiger where
the system provided version is too old to be of use.

Note this uses the official Mac Python installer from python.org.
This installs a copy of Python in /Library/Frameworks/Python.framework.
It also makes symlinks from /usr/local/bin to the Python.framework/bin.

I have tested this script on Leopard and it worked fine. I have not
tested it on Tiger as I do not have access to a Tiger machine. In
either case this should provide a great starting point for someone
wishing to upgrade their copy of Python on Tiger.

Future patches can make our scripts depend on a success return from this
script and then they can either skip tests/sections for which python is
insufficient, or they can fail themselves.

  • Scripts/ensure-valid-python: Added.
6:37 AM Changeset in webkit [59537] by eric@webkit.org
  • 6 edits
    36 adds in trunk

2010-05-15 Leandro Pereira <​leandro@profusion.mobi>

Reviewed by Adam Treat.

[EFL] Add build system for the EFL port.
​http://webkit.org/b/37945

  • CMakeLists.txt: Added.
  • cmake/FindCFLite.cmake: Added.
  • cmake/FindCairo.cmake: Added.
  • cmake/FindEFL.cmake: Added.
  • cmake/FindFreetype.cmake: Added.
  • cmake/FindGDK-PixBuf.cmake: Added.
  • cmake/FindGDK.cmake: Added.
  • cmake/FindGIO.cmake: Added.
  • cmake/FindGlib.cmake: Added.
  • cmake/FindICU.cmake: Added.
  • cmake/FindLibSoup2.cmake: Added.
  • cmake/FindLibXlst.cmake: Added.
  • cmake/FindPango.cmake: Added.
  • cmake/FindSqlite.cmake: Added.
  • cmake/LibFindMacros.cmake: Added.
  • cmake/OptionsCommon.cmake: Added.
  • cmake/OptionsEfl.cmake: Added.
  • cmake/WebKitEfl.cmake: Added.
  • cmake/WebKitFS.cmake: Added.
  • cmake/WebKitFeatures.cmake: Added.
  • cmake/WebKitGenerators.cmake: Added.
  • cmake/WebKitHelpers.cmake: Added.
  • cmake/WebKitMacros.cmake: Added.
  • cmakeconfig.h.cmake: Added.

2010-05-15 Leandro Pereira <​leandro@profusion.mobi>

Reviewed by Adam Treat.

[EFL] Add build system for the EFL port.
​http://webkit.org/b/37945

  • CMakeLists.txt: Added.
  • CMakeListsEfl.txt: Added.
  • config.h: Add conditional to look for cmakeconfig.h.
  • jsc/CMakeLists.txt: Added.
  • jsc/CMakeListsEfl.txt: Added.
  • wtf/CMakeLists.txt: Added.
  • wtf/CMakeListsEfl.txt: Added.

2010-05-15 Leandro Pereira <​leandro@profusion.mobi>

Reviewed by Adam Treat.

[EFL] Add build system for the EFL port.
​http://webkit.org/b/37945

  • CMakeLists.txt: Added.
  • CMakeListsEfl.txt: Added.
  • config.h: Add conditional to look for cmakeconfig.h.

2010-05-15 Leandro Pereira <​leandro@profusion.mobi>

Reviewed by Adam Treat.

[EFL] Add build system for the EFL port.
​http://webkit.org/b/37945

  • CMakeLists.txt: Added.
  • CMakeListsEfl.txt: Added.
6:14 AM Changeset in webkit [59536] by eric@webkit.org
  • 4 edits in trunk/WebCore

2010-05-15 Adam Barth <​abarth@webkit.org>

Reviewed by Darin Adler.

Clipboard.setData doesn't need to be custom
​https://bugs.webkit.org/show_bug.cgi?id=39133

The implementation of the custom binding for this function was a bit
different than its declaration in the IDL file. I've updated the IDL
to match the implementation, but we probably should check whether our
behavior is correct with respect to the specification.

No tests because I've maintained our current behavior.

  • bindings/js/JSClipboardCustom.cpp:
  • bindings/v8/custom/V8ClipboardCustom.cpp:
  • dom/Clipboard.idl:
5:09 AM Changeset in webkit [59535] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2010-05-15 Xan Lopez <​xlopez@igalia.com>

Skip failing test, it depends on one patch still in the commit
queue.

  • platform/gtk/Skipped:
3:42 AM Changeset in webkit [59534] by eric@webkit.org
  • 9 edits
    2 copies
    1 add in trunk/WebKitTools

2010-05-15 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Display queue position inside EWS bubbles
​https://bugs.webkit.org/show_bug.cgi?id=38979

This ended up being a lot of clean-up to our status server code.

Added a new WorkItems model, a form with which to fill it,
and taught the Attachment class how to calculate the current queue
postion for an attachment using the data in WorkItems.

I also finally made statusbubble.* and dashboard.* not use copy-paste code.

The Attachment class has this summary() method which is very
controller/view-like and does not belong in a model class.
This patch got rid of all direct uses of summary().

  • QueueStatusServer/handlers/dashboard.py:
    • Build row objects to hand off to the view instead of handing off a summary object and expecting the view to process it directly.
  • QueueStatusServer/handlers/statusbubble.py:
    • Build bubble object and hand them off to the view.
  • QueueStatusServer/handlers/updatestatus.py:
    • Code cleanup. Just move some code into _queue_status_from_request to make the main put() handler easier to read.
  • QueueStatusServer/handlers/updateworkitems.py: Added.
    • Controller to handle storing WorkItems model objects.
  • QueueStatusServer/main.py:
    • Add route for /update-work-items
  • QueueStatusServer/model/attachment.py:
    • Add new methods to replace direct summary() access.
    • Teach attachment how to calculate queue positions from WorkItems data.
  • QueueStatusServer/model/queues.py:
    • Move name_with_underscores here for easier re-use.
  • QueueStatusServer/model/workitems.py: Added.
    • New model for storing what items are currently queue for any bot.
  • QueueStatusServer/templates/dashboard.html:
    • Kill the copy/paste code!
  • QueueStatusServer/templates/statusbubble.html:
    • Kill the copy/paste code!
  • QueueStatusServer/templates/updateworkitems.html: Added.
    • Simple form for updating a queue's current work items.
3:14 AM Changeset in webkit [59533] by eric@webkit.org
  • 7 edits in trunk/WebCore

2010-05-15 Yaar Schnitman <​yaar@chromium.org>

Reviewed by Adam Barth.

V8 overload support ported to JSC. Unforking of overloaded declarations in WebGL & Canvas will be done in subsequent patches.
​https://bugs.webkit.org/show_bug.cgi?id=38920

Note that JSTestCallback.cpp/h include minor irrelevant changes too since somebody forgot to update them after changing the JS generator.

  • bindings/scripts/CodeGenerator.pm:
  • bindings/scripts/CodeGeneratorJS.pm:
  • bindings/scripts/CodeGeneratorV8.pm:
  • bindings/scripts/test/JS/JSTestCallback.cpp: (WebCore::JSTestCallback::JSTestCallback): (WebCore::JSTestCallback::~JSTestCallback):
  • bindings/scripts/test/JS/JSTestCallback.h:
  • bindings/scripts/test/JS/JSTestObj.cpp: (WebCore::): (WebCore::jsTestObjPrototypeFunctionOverloadedMethod1): (WebCore::jsTestObjPrototypeFunctionOverloadedMethod2): (WebCore::jsTestObjPrototypeFunctionOverloadedMethod3): (WebCore::jsTestObjPrototypeFunctionOverloadedMethod4): (WebCore::testObjPrototypeFunctionOverloadedMethod):
  • bindings/scripts/test/JS/JSTestObj.h:
  • bindings/scripts/test/TestObj.idl:
2:55 AM Changeset in webkit [59532] by eric@webkit.org
  • 5 edits
    2 adds in trunk

2010-05-15 Joanmarie Diggs <​joanmarie.diggs@gmail.com>

Reviewed by Xan Lopez.

​https://bugs.webkit.org/show_bug.cgi?id=30500
[Gtk] Find a way for WebKit to "announce" itself so that ATs can readily distinguish it from true Gtk/Gail

The "announcement" is now made in the form of an object attribute
associated with the AtkObject.

  • platform/gtk/accessibility/object-attributes.html: Added.
  • platform/gtk/accessibility/object-attributes-expected.txt: Added.

2010-05-15 Joanmarie Diggs <​joanmarie.diggs@gmail.com>

Reviewed by Xan Lopez.

​https://bugs.webkit.org/show_bug.cgi?id=30500
[Gtk] Find a way for WebKit to "announce" itself so that ATs can readily distinguish it from true Gtk/Gail

The "announcement" is now made in the form of an object attribute
associated with the AtkObject.

  • accessibility/gtk/AccessibilityObjectWrapperAtk.cpp: (webkit_accessible_get_attributes):

2010-05-15 Joanmarie Diggs <​joanmarie.diggs@gmail.com>

Reviewed by Xan Lopez.

​https://bugs.webkit.org/show_bug.cgi?id=30500
[Gtk] Find a way for WebKit to "announce" itself so that ATs can readily distinguish it from true Gtk/Gail

The "announcement" is now made in the form of an object attribute
associated with the AtkObject.

  • DumpRenderTree/gtk/AccessibilityUIElementGtk.cpp: (AccessibilityUIElement::allAttributes): (attributeSetToString):
2:44 AM Changeset in webkit [59531] by eric@webkit.org
  • 4 edits in trunk

2010-05-15 Mario Sanchez Prada <​msanchez@igalia.com>

Reviewed by Xan Lopez.

Accessibility: Implement isSelected in DRT for GTK
​https://bugs.webkit.org/show_bug.cgi?id=31018

Do not skip aria-controls-with-tabs.html anymore

  • platform/gtk/Skipped:

2010-05-15 Mario Sanchez Prada <​msanchez@igalia.com>

Reviewed by Xan Lopez.

Accessibility: Implement isSelected in DRT for GTK
​https://bugs.webkit.org/show_bug.cgi?id=31018

Implement AccessibilityUIElement::isSelected() for Gtk

  • DumpRenderTree/gtk/AccessibilityUIElementGtk.cpp: (AccessibilityUIElement::isSelected):
2:33 AM Changeset in webkit [59530] by eric@webkit.org
  • 4 edits in trunk/LayoutTests

2010-05-15 Yael Aharon <​yael.aharon@nokia.com>

Reviewed by Darin Adler.

Update notifications test results
​https://bugs.webkit.org/show_bug.cgi?id=39126
Add missing line endings.

  • fast/notifications/notifications-replace-expected.txt:
  • fast/notifications/notifications-request-permission-expected.txt:
  • fast/notifications/notifications-rtl-expected.txt:
2:21 AM Changeset in webkit [59529] by eric@webkit.org
  • 3 edits in trunk/WebKit/chromium

2010-05-15 Evan Stade <​estade@chromium.org>

Reviewed by Adam Barth.

[chromium] "Check spelling in this field" context menu item always checked
​https://bugs.webkit.org/show_bug.cgi?id=39018

Initialize all POD members of WebContextMenuData in a default
constructor.

No tests because this code would need some major refactoring to unit
test, and AFAICT it's not testable via layout tests. See note at top
of getCustomMenuFromDefaultItems.

  • public/WebContextMenuData.h: (WebKit::WebContextMenuData::WebContextMenuData):
  • src/ContextMenuClientImpl.cpp: (WebKit::ContextMenuClientImpl::getCustomMenuFromDefaultItems):
2:10 AM Changeset in webkit [59528] by eric@webkit.org
  • 6 edits in trunk

2010-05-15 Adam Barth <​abarth@webkit.org>

Reviewed by Darin Adler.

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

Update expected results to show the console message.

  • http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-expected.txt:
  • http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-in-body-expected.txt:
  • http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-parent-same-origin-deny-expected.txt:

2010-05-15 Adam Barth <​abarth@webkit.org>

Reviewed by Darin Adler.

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

The blank page confuses people when X-Frame-Options blocks a load. The
console message should at least give them a hint as to what's going on.

  • dom/Document.cpp: (WebCore::Document::processHttpEquiv):
1:58 AM Changeset in webkit [59527] by eric@webkit.org
  • 4 edits in trunk/JavaScriptCore

2010-05-15 Chao-ying Fu <​fu@mips.com>

Reviewed by Oliver Hunt.

Update MIPS JIT for unsigned right shift, Math.sqrt, load16
​https://bugs.webkit.org/show_bug.cgi?id=38412

Fixed MIPS build failure.

  • assembler/MIPSAssembler.h: (JSC::MIPSAssembler::srl): (JSC::MIPSAssembler::srlv): (JSC::MIPSAssembler::sqrtd):
  • assembler/MacroAssemblerMIPS.h: (JSC::MacroAssemblerMIPS::urshift32): (JSC::MacroAssemblerMIPS::sqrtDouble): (JSC::MacroAssemblerMIPS::load16): (JSC::MacroAssemblerMIPS::supportsFloatingPointSqrt):
  • jit/JSInterfaceJIT.h:
1:46 AM Changeset in webkit [59526] by eric@webkit.org
  • 11 edits
    1 copy in trunk

2010-05-15 Ilya Tikhonovsky <​loislo@chromium.org>

Reviewed by Geoffrey Garen.

WebInspector: JSC Should provide heap size info for Timeline panel.
​https://bugs.webkit.org/show_bug.cgi?id=38420

  • JavaScriptCore.exp:
  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def:
  • runtime/Collector.cpp: (JSC::Heap::size):
  • runtime/Collector.h:

2010-05-15 Ilya Tikhonovsky <​loislo@chromium.org>

Reviewed by Geoffrey Garen.

WebInspector: JSC Should provide heap size info for Timeline panel.
​https://bugs.webkit.org/show_bug.cgi?id=38420

  • GNUmakefile.am:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/ScriptGCEvent.cpp: Added. (WebCore::ScriptGCEvent::getHeapSize):
  • bindings/js/ScriptGCEvent.h:
1:25 AM Changeset in webkit [59525] by eric@webkit.org
  • 6 edits in trunk

2010-05-15 Dave Tapuska <​dtapuska@rim.com>

Reviewed by Darin Adler.

Add a new define for we need strict arena alignment. ARMv5 with
double word instructions set this value.

  • wtf/Platform.h:

2010-05-15 Dave Tapuska <​dtapuska@rim.com>

Reviewed by Darin Adler.

Fix alignment faults on ARMv5TE architectures.

In debug mode the RenderAreaDebugHeader was 12 bytes which
caused an alignment failure for RenderObjects because they
would start on a 4 byte boundary. Align the RenderArenaDebugHeader
using the ARENA_ALIGN macro.

In NDEBUG mode the RenderArena objects come from a pool which
has 4 byte alignment. Allow the alignment to be based on the
AllocAlignmentInteger. This define will be activated for ARMv5TE.

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

Note: See TracTimeline for information about the timeline view.