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

Timeline



Aug 3, 2011:

11:35 PM Changeset in webkit [92352] by commit-queue@webkit.org
  • 5 edits in trunk

An element with role=textbox should have settable AXValue unless read-only
​https://bugs.webkit.org/show_bug.cgi?id=65664

Patch by Alice Boxhall <​aboxhall@chromium.org> on 2011-08-03
Reviewed by Chris Fleizach.

Source/WebCore:

  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::canSetValueAttribute):
Return true for non-native text field with aria-readonly not set.

LayoutTests:

  • accessibility/aria-readonly-expected.txt:
  • accessibility/aria-readonly.html:
11:08 PM Changeset in webkit [92351] by abarth@webkit.org
  • 6 edits in trunk/Tools

Use failureInfo more pervasively in garden-o-matic
​https://bugs.webkit.org/show_bug.cgi?id=65650

Reviewed by Dimitri Glazkov.

These three variable appears in concert together many places. This
patch unifies the remaining uses into being failureInfo objects, which
saves some marshalling.

  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/main.js:
  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/results.js:
  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/results_unittests.js:
  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ui.js:
  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ui_unittests.js:
10:34 PM Changeset in webkit [92350] by tkent@chromium.org
  • 2 edits in trunk/Source/WebKit/win

[Windows] Fix build without ENABLE_VIDEO.

  • WebView.cpp:

(WebView::enterFullscreenForNode):

9:40 PM Changeset in webkit [92349] by macpherson@chromium.org
  • 2 edits in trunk/Source/WebCore

Clean up value clamping in CSSStyleSelector.
​https://bugs.webkit.org/show_bug.cgi?id=65588

Reviewed by Darin Adler.

No new tests / trivial code cleanup only.

  • css/CSSStyleSelector.cpp:

(WebCore::CSSStyleSelector::applyProperty):
Replace (unsigned int)primitiveValue->getDoubleValue() with primtiveValue->getValue<unsigned>() for correct clamping.
Replace clampToInteger(primitiveValue->getDoubleValue()) with primitiveValue->getIntValue().

9:38 PM Changeset in webkit [92348] by abarth@webkit.org
  • 4 edits in trunk/Tools

garden-o-matic should be able to update expecations from the details pane
​https://bugs.webkit.org/show_bug.cgi?id=65648

Reviewed by Dimitri Glazkov.

The new button queues up the updates, which get executed all at once.

  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/garden-o-matic.html:
  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/main.js:
  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/model.js:
9:38 PM Changeset in webkit [92347] by tkent@chromium.org
  • 13 edits in trunk/Source

Fix incorrect checks for HTMLMediaElement
​https://bugs.webkit.org/show_bug.cgi?id=65590

Reviewed by Dimitri Glazkov.

<video> and <audio> can be an HTMLElement instance instead of
HTMLMediaElement if MediaPlayer::isAvailable() returns false or
the media feature is disabled at runtime.

Source/WebCore:

  • html/HTMLSourceElement.cpp:

(WebCore::HTMLSourceElement::insertedIntoTree):
(WebCore::HTMLSourceElement::willRemove):

  • html/HTMLTrackElement.cpp:

(WebCore::HTMLTrackElement::insertedIntoTree):
(WebCore::HTMLTrackElement::willRemove):

  • html/shadow/MediaControlElements.cpp:

(WebCore::toParentMediaElement):

  • page/FrameView.cpp:

(WebCore::FrameView::updateWidget):

  • platform/efl/RenderThemeEfl.cpp:

(WebCore::RenderThemeEfl::paintMediaMuteButton):

  • platform/gtk/RenderThemeGtk.cpp:

(WebCore::getMediaElementFromRenderObject):

  • rendering/RenderThemeMac.mm:

(WebCore::RenderThemeMac::paintMediaSliderTrack):

  • rendering/RenderThemeWinCE.cpp:

(WebCore::mediaElementParent):

Source/WebKit/mac:

  • WebView/WebFullScreenController.mm:

(-[WebFullScreenController _isAnyMoviePlaying]):

Source/WebKit/win:

  • WebView.cpp:

(WebView::enterFullscreenForNode):

9:16 PM Changeset in webkit [92346] by abarth@webkit.org
  • 7 edits in trunk/Tools

Add UI to garden-o-matic for updating expectations
​https://bugs.webkit.org/show_bug.cgi?id=65644

Reviewed by Dimitri Glazkov.

The patch plumbs the UI back to the gardening server, but the gardening
server endpoint is just a stub at this point.

  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/checkout.js:
  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/garden-o-matic.html:
  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/main.js:
  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/model.js:
  • Scripts/webkitpy/tool/servers/gardeningserver.py:
  • Scripts/webkitpy/tool/servers/reflectionhandler.py:
8:54 PM Changeset in webkit [92345] by jeffm@apple.com
  • 2 edits in trunk/Source/WebKit2

Use of AVFoundation should default to off on Windows
​https://bugs.webkit.org/show_bug.cgi?id=65660

Reviewed by Darin Adler.

  • Shared/WebPreferencesStore.h: Change default value for AVFoundationEnabled to false on Windows, it's still true on other platforms.
8:02 PM Changeset in webkit [92344] by abarth@webkit.org
  • 5 edits in trunk/Tools

garden-o-matic details view should queue rebaselines for later execution
​https://bugs.webkit.org/show_bug.cgi?id=65636

Reviewed by Dimitri Glazkov.

This patch doesn't contain any confirmation UI, but we'll probably want
that at some point.

  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/checkout.js:
  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/garden-o-matic.html:
  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/main.js:
  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/model.js:
7:35 PM Changeset in webkit [92343] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

Chromium Mac: Make ScrollbarOverlayUtilitiesChromiumMac.h c++ compatible
​https://bugs.webkit.org/show_bug.cgi?id=65659

Patch by Sailesh Agrawal <​sail@chromium.org> on 2011-08-03
Reviewed by James Robinson.

ScrollbarOverlayUtilitiesChromiumMac.h couldn't be included from pure c++ code because it used NSInteger. I fixed this by checking for OBJC before using NSInteger.

Currently no pure c++ files are including ScrollbarOverlayUtilitiesChromiumMac.h but this will change in future patches from ​jam@chromium.org.

  • platform/chromium/ScrollbarOverlayUtilitiesChromiumMac.h:
  • platform/chromium/ScrollbarOverlayUtilitiesChromiumMac.mm:

(wkScrollbarPainterControllerStyle):
(wkSetScrollbarPainterControllerStyle):

7:09 PM Changeset in webkit [92342] by macpherson@chromium.org
  • 3 edits in trunk/Source/WebCore

Support cast between CSSPrimitiveValue and EBoxSizing, use in CSSStyleSelector.
​https://bugs.webkit.org/show_bug.cgi?id=65657

Reviewed by Simon Fraser.

No new tests / refactoring only.

  • css/CSSPrimitiveValueMappings.h:

(WebCore::CSSPrimitiveValue::CSSPrimitiveValue):
Implement cast from EBoxSizing.
(WebCore::CSSPrimitiveValue::operator EBoxSizing):
Implement cast to EBoxSizing.

  • css/CSSStyleSelector.cpp:

(WebCore::CSSStyleSelector::applyProperty):
Use appropriate macro to simplify code using cast.

7:01 PM Changeset in webkit [92341] by commit-queue@webkit.org
  • 10 edits
    4 adds
    6 deletes in trunk

Implement EventSender.scalePageBy()
​https://bugs.webkit.org/show_bug.cgi?id=58013

Patch by Kentaro Hara <​haraken@google.com> on 2011-08-03
Reviewed by Darin Fisher.

Implemented EventSender.scalePageBy(f, x, y), which scales a page by a factor of f
and then sets a scroll position to (x, y). Enabled the tests that had been waiting
for the implementation of EventSender.scalePageBy(f, x, y).

Source/WebKit/chromium:

Tests: compositing/scaling/tiled-layer-recursion.html

fast/repaint/scale-page-shrink.html
fast/dom/Element/scale-page-client-rects.html
fast/dom/Range/scale-page-client-rects.html
fast/events/scroll-in-scaled-page-with-overflow-hidden.html
fast/dom/Element/scale-page-bounding-client-rect.html
fast/dom/Range/scale-page-bounding-client-rect.html

  • public/WebView.h:
  • src/WebViewImpl.cpp:

(WebKit::WebViewImpl::scalePage): A wrapper method for scalePage() in WebCore.

  • src/WebViewImpl.h:

Tools:

Tests: compositing/scaling/tiled-layer-recursion.html

fast/repaint/scale-page-shrink.html
fast/dom/Element/scale-page-client-rects.html
fast/dom/Range/scale-page-client-rects.html
fast/events/scroll-in-scaled-page-with-overflow-hidden.html
fast/dom/Element/scale-page-bounding-client-rect.html
fast/dom/Range/scale-page-bounding-client-rect.html

  • DumpRenderTree/chromium/EventSender.cpp:

(EventSender::EventSender): Added bindings for scalePageBy().
(EventSender::scalePageBy): A wrapper method for scalePage() in WebView.

  • DumpRenderTree/chromium/EventSender.h:
  • DumpRenderTree/chromium/TestShell.cpp:

(TestShell::resetTestController): Resets the scale factor to 1.

LayoutTests:

  • platform/chromium-linux/compositing/scaling/tiled-layer-recursion-expected.png: Added.
  • platform/chromium-linux/fast/repaint/scale-page-shrink-expected.png: Added.
  • platform/chromium-linux/fast/repaint/scale-page-shrink-expected.txt: Added.
  • platform/chromium-mac/fast/dom/Element/scale-page-bounding-client-rect-expected.txt: Removed.
  • platform/chromium-mac/fast/dom/Range/scale-page-bounding-client-rect-expected.txt: Removed.
  • platform/chromium-win/fast/dom/Element/scale-page-bounding-client-rect-expected.txt: Removed.
  • platform/chromium-win/fast/dom/Element/scale-page-client-rects-expected.txt: Removed.
  • platform/chromium-win/fast/dom/Range/scale-page-bounding-client-rect-expected.txt: Removed.
  • platform/chromium-win/fast/dom/Range/scale-page-client-rects-expected.txt: Removed.
  • platform/chromium/test_expectations.txt: Enabled one test. Enabled two tests for chromium-linux.
6:51 PM Changeset in webkit [92340] by Joseph Pecoraro
  • 10 edits
    3 adds in trunk

<​http://webkit.org/b/65006> Draw rectangular box shadow for elements with border-radius if no corners are visible

Patch by Ian Henderson <​ianh@apple.com> on 2011-08-03
Reviewed by Simon Fraser.

Source/WebCore:

Change "unroundClippedCorners" to "allCornersClippedOut" and use it to
determine when the corners' influence affects the blurred shadow in
the current drawing region.

Test: fast/box-shadow/box-shadow-clipped-slices.html

  • rendering/InlineFlowBox.cpp:

(WebCore::InlineFlowBox::paintBoxShadow): Pass PaintInfo through to
RenderBoxModelObject::paintBoxShadow.
(WebCore::InlineFlowBox::paintBoxDecorations): Ditto.

  • rendering/InlineFlowBox.h:
  • rendering/RenderBox.cpp:

(WebCore::RenderBox::paintBoxDecorations): Ditto.

  • rendering/RenderBoxModelObject.cpp:

(WebCore::allCornersClippedOut): Determines whether the bounding rects
for any of the given rounded rect's corners intersects the clip rect.
(WebCore::RenderBoxModelObject::paintBorder): Instead of "unrounding"
any corner outside the clip rect, unround all corners at once, and
only if they are all outside the clip rect.
(WebCore::RenderBoxModelObject::paintBoxShadow): Determine whether any
corner will influence shadow drawing using allCornersClippedOut on the
influenceRect, whose corners extend past the fillRect's corners by the
blur radius and inside the fillRect's corners by the blur radius.

  • rendering/RenderBoxModelObject.h:
  • rendering/RenderFieldset.cpp:

(WebCore::RenderFieldset::paintBoxDecorations): Pass PaintInfo into
RenderBoxModelObject::paintBoxShadow.

  • rendering/RenderTable.cpp:

(WebCore::RenderTable::paintBoxDecorations): Ditto.

  • rendering/RenderTableCell.cpp:

(WebCore::RenderTableCell::paintBoxDecorations): Ditto.

LayoutTests:

  • fast/box-shadow/box-shadow-clipped-slices.html: Added.
  • platform/mac/fast/box-shadow/box-shadow-clipped-slices-expected.png: Added.
  • platform/mac/fast/box-shadow/box-shadow-clipped-slices-expected.txt: Added.
6:44 PM Changeset in webkit [92339] by commit-queue@webkit.org
  • 3 edits in trunk/LayoutTests

[Chromium] fast/dom/resource-locations-in-created-html-document.html fails on win32
​https://bugs.webkit.org/show_bug.cgi?id=65521

Patch by Noel Gordon <​noel.gordon@gmail.com> on 2011-08-03
Reviewed by Adam Barth.

file: scheme URI paths on Windows include the disk drive reference (C: for example).
Refer to ​http://blogs.msdn.com/b/ie/archive/2006/12/06/file-uris-in-windows.aspx.

  • fast/dom/resource-locations-in-created-html-document.html:
  • platform/chromium/test_expectations.txt:
6:41 PM Changeset in webkit [92338] by rniwa@webkit.org
  • 2 edits in trunk/LayoutTests

Revert an erroneous rebaseline from r92315.

  • fast/dom/navigator-detached-no-crash-expected.txt:
5:22 PM Changeset in webkit [92337] by noam.rosenthal@nokia.com
  • 2 edits in trunk/Source/WebCore

[Qt][Texmap][REGRESSION] ​http://webkit.org/blog-files/transform-style.html doesn't show composited content
​https://bugs.webkit.org/show_bug.cgi?id=65629

Reviewed by Benjamin Poulain.

Some non-ES2 initialization was wrongfully #ifdefed in CPU(X86) and thus compiled-out.
When put it in the correct #ifdef, composited layers which require an intermediate buffer
work again.

No new tests. Existing opacity tests in LayoutTests/compositing test this.

  • platform/graphics/opengl/TextureMapperGL.cpp:

(WebCore::BitmapTextureGL::bind):

5:16 PM Changeset in webkit [92336] by rniwa@webkit.org
  • 1 edit
    3 deletes in trunk/LayoutTests

Chromium Windows and Linux rebaselines after r92330.

  • platform/chromium-win/editing/pasteboard/paste-pre-001-expected.txt: Removed.
  • platform/chromium-win/editing/pasteboard/paste-pre-002-expected.txt: Removed.
  • platform/chromium-win/editing/pasteboard/paste-text-011-expected.txt: Removed.
4:57 PM Changeset in webkit [92335] by enne@google.com
  • 1 edit in branches/chromium/835/Source/WebCore/platform/graphics/chromium/LayerTilerChromium.cpp

Merge 92331 - [chromium] Fix issue where switching composited tabs evicts root textures
​https://bugs.webkit.org/show_bug.cgi?id=65562

Reviewed by James Robinson.

LayerTilerChromium::protectTileTextures was hard-coding texture sizes
and texture formats. Because this didn't always match textures' actual
sizes and formats, they weren't getting protected during eviction.

  • platform/graphics/chromium/LayerTilerChromium.cpp:

(WebCore::LayerTilerChromium::protectTileTextures):

TBR=​enne@google.com
Review URL: ​http://codereview.chromium.org/7508041

4:53 PM Changeset in webkit [92334] by levin@chromium.org
  • 3 edits in trunk/Tools

Rename WEBKIT_API to WEBKIT_EXPORT in check-webkit-style.
​https://bugs.webkit.org/show_bug.cgi?id=65652

Reviewed by Adam Barth.

  • Scripts/webkitpy/style/checkers/cpp.py:
  • Scripts/webkitpy/style/checkers/cpp_unittest.py:
4:51 PM Changeset in webkit [92333] by mrowe@apple.com
  • 8 edits in trunk/Source

Bring some order to FeatureDefines.xcconfig to make it easier to follow.

Reviewed by Sam Weinig.

Source/JavaScriptCore:

  • Configurations/FeatureDefines.xcconfig:

Source/WebCore:

  • Configurations/FeatureDefines.xcconfig:

Source/WebKit/mac:

  • Configurations/FeatureDefines.xcconfig:

Source/WebKit2:

  • Configurations/FeatureDefines.xcconfig:
4:50 PM Changeset in webkit [92332] by mrowe@apple.com
  • 8 edits in trunk/Source

Clean up FeatureDefines.xcconfig to remove some unnecessary conditional settings

Reviewed by Dave Kilzer.

Source/JavaScriptCore:

  • Configurations/FeatureDefines.xcconfig:

Source/WebCore:

  • Configurations/FeatureDefines.xcconfig:

Source/WebKit/mac:

  • Configurations/FeatureDefines.xcconfig:

Source/WebKit2:

  • Configurations/FeatureDefines.xcconfig:
4:43 PM Changeset in webkit [92331] by enne@google.com
  • 2 edits in trunk/Source/WebCore

[chromium] Fix issue where switching composited tabs evicts root textures
​https://bugs.webkit.org/show_bug.cgi?id=65562

Reviewed by James Robinson.

LayerTilerChromium::protectTileTextures was hard-coding texture sizes
and texture formats. Because this didn't always match textures' actual
sizes and formats, they weren't getting protected during eviction.

  • platform/graphics/chromium/LayerTilerChromium.cpp:

(WebCore::LayerTilerChromium::protectTileTextures):

4:42 PM Changeset in webkit [92330] by rniwa@webkit.org
  • 10 edits
    2 adds in trunk

select-all, copy, paste of specialAncestorElements (e.g. pre, h1, etc) nests the element inside itself
​https://bugs.webkit.org/show_bug.cgi?id=26483

Reviewed by Enrica Casucci.

Source/WebCore:

The bug was caused by WebKit serializing pre, h1, etc... to retain structure and appearance when copying
rich content and pasting did not remove such nodes wrapping the copied contents.

Fixed the bug by extending r81887 and r83322 to remove those elements from where WebKit pastes into.

Test: editing/pasteboard/copy-paste-text-in-h1.html

  • editing/ReplaceSelectionCommand.cpp:

(WebCore::nodeHasAttributesToPreserve): Extracted from isInlineNodeWithStyle.
(WebCore::isInlineNodeWithStyle): Calls nodeHasAttributesToPreserve.
(WebCore::ReplaceSelectionCommand::doApply): Calls ancestorToRetainStructureAndAppearance.
Remove nodes copied by ancestorToRetainStructureAndAppearance at insertionPos before pasting the fragment.

  • editing/markup.cpp:

(WebCore::ancestorToRetainStructureAndAppearance): Takes ShouldIncludeParagraphSeparators.

  • editing/markup.h:

LayoutTests:

  • editing/pasteboard/5065605-expected.txt:
  • editing/pasteboard/copy-paste-text-in-h1-expected.txt: Added.
  • editing/pasteboard/copy-paste-text-in-h1.html: Added.
  • editing/pasteboard/display-block-on-spans-expected.txt:
  • editing/pasteboard/paste-pre-001-expected.txt:
  • editing/pasteboard/paste-pre-002-expected.txt:
  • editing/pasteboard/paste-text-011-expected.txt:
4:35 PM Changeset in webkit [92329] by fpizlo@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

JSC GC heap size improvement breaks build on some platforms due to
unused parameter
​https://bugs.webkit.org/show_bug.cgi?id=65641

Reviewed by Darin Adler.

Fix build on non-x86 platforms, by ensuring that the relevant
parameter always appears to be used even when it isn't.

  • heap/Heap.cpp:
4:31 PM Changeset in webkit [92328] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Remove LegacyDefaultOptionalArguments flag from Console.idl
​https://bugs.webkit.org/show_bug.cgi?id=65569

Patch by Mark Pilgrim <​pilgrim@chromium.org> on 2011-08-03
Reviewed by Adam Barth.

No new tests, all existing tests pass.

  • page/Console.idl:
4:20 PM Changeset in webkit [92327] by commit-queue@webkit.org
  • 34 edits in trunk

Remove LegacyDefaultOptionalArguments flag from HTML DOM IDL files
​https://bugs.webkit.org/show_bug.cgi?id=65338

Patch by Mark Pilgrim <​pilgrim@chromium.org> on 2011-08-03
Reviewed by Adam Barth.

Source/WebCore:

  • html/DOMFormData.idl:
  • html/DOMTokenList.idl:
  • html/DOMURL.idl:
  • html/HTMLAllCollection.idl:
  • html/HTMLAnchorElement.idl:
  • html/HTMLAudioElement.idl:
  • html/HTMLButtonElement.idl:
  • html/HTMLCanvasElement.idl:
  • html/HTMLCollection.idl:
  • html/HTMLDivElement.idl:
  • html/HTMLDocument.idl:
  • html/HTMLElement.idl:
  • html/HTMLFieldSetElement.idl:
  • html/HTMLInputElement.idl:
  • html/HTMLKeygenElement.idl:
  • html/HTMLMediaElement.idl:
  • html/HTMLObjectElement.idl:
  • html/HTMLOptionsCollection.idl:
  • html/HTMLOutputElement.idl:
  • html/HTMLSelectElement.idl:
  • html/HTMLTableElement.idl:
  • html/HTMLTableRowElement.idl:
  • html/HTMLTableSectionElement.idl:
  • html/HTMLTextAreaElement.idl:
  • html/HTMLVideoElement.idl:
  • html/TimeRanges.idl:

LayoutTests:

  • fast/files/create-blob-url-crash-expected.txt:
  • fast/files/create-blob-url-crash.html:
  • fast/files/revoke-blob-url-expected.txt:
  • fast/files/revoke-blob-url.html:
  • fast/forms/ValidityState-customError-expected.txt:
  • fast/forms/ValidityState-customError.html:
4:19 PM Changeset in webkit [92326] by fpizlo@apple.com
  • 2 edits in trunk/Tools

Adding Filip Pizlo to committer list.

Unreviewed.

  • Scripts/webkitpy/common/config/committers.py:
4:18 PM Changeset in webkit [92325] by jpfau@apple.com
  • 6 edits in trunk/Source/WebCore

Make atomic XML token
​https://bugs.webkit.org/show_bug.cgi?id=65639

Reviewed by Adam Barth.

Create a shared AtomicMarkupTokenBase that is shared by AtomicHTMLToken and the new AtomicXMLToken

  • html/parser/HTMLToken.h:

(WebCore::HTMLToken::setForceQuirks):
(WebCore::AtomicHTMLToken::AtomicHTMLToken):
(WebCore::AtomicHTMLToken::forceQuirks):

  • html/parser/HTMLTokenizer.cpp:

(WebCore::::nameForAttribute):
(WebCore::::usesName):
(WebCore::::usesAttributes):

  • xml/parser/MarkupTokenBase.h:

(WebCore::AtomicMarkupTokenBase::AtomicMarkupTokenBase):
(WebCore::AtomicMarkupTokenBase::type):
(WebCore::AtomicMarkupTokenBase::name):
(WebCore::AtomicMarkupTokenBase::setName):
(WebCore::AtomicMarkupTokenBase::selfClosing):
(WebCore::AtomicMarkupTokenBase::getAttributeItem):
(WebCore::AtomicMarkupTokenBase::attributes):
(WebCore::AtomicMarkupTokenBase::takeAtributes):
(WebCore::AtomicMarkupTokenBase::characters):
(WebCore::AtomicMarkupTokenBase::comment):
(WebCore::AtomicMarkupTokenBase::publicIdentifier):
(WebCore::AtomicMarkupTokenBase::systemIdentifier):
(WebCore::::initializeAttributes):

  • xml/parser/XMLToken.h:

(WebCore::AtomicXMLToken::AtomicXMLToken):
(WebCore::AtomicXMLToken::prefix):
(WebCore::AtomicXMLToken::target):
(WebCore::AtomicXMLToken::data):
(WebCore::AtomicXMLToken::xmlVersion):
(WebCore::AtomicXMLToken::xmlStandalone):

  • xml/parser/XMLTokenizer.cpp:

(WebCore::::nameForAttribute):
(WebCore::::usesName):
(WebCore::::usesAttributes):

4:03 PM Changeset in webkit [92324] by Dimitri Glazkov
  • 2 edits
    1 add in trunk/Tools

Add "optimize-expectations" command to webkit-patch.
​https://bugs.webkit.org/show_bug.cgi?id=65633

For now, only operates on Chromium test_expectations.txt and does not report errors/warnings.

Reviewed by Adam Barth.

  • Scripts/webkitpy/tool/commands/init.py: Added command.
  • Scripts/webkitpy/tool/commands/expectations.py: Added.
4:01 PM Changeset in webkit [92323] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebCore

Remove LegacyDefaultOptionalArguments flag from geolocation IDL files
​https://bugs.webkit.org/show_bug.cgi?id=65355

Patch by Mark Pilgrim <​pilgrim@chromium.org> on 2011-08-03
Reviewed by Adam Barth.

No new tests, all existing tests pass.

  • page/Geolocation.idl:
  • page/PositionCallback.idl:
  • page/PositionErrorCallback.idl:
3:59 PM Changeset in webkit [92322] by darin@chromium.org
  • 104 edits in trunk/Source/WebKit/chromium

2011-08-03 Darin Fisher <​darin@chromium.org>

[Chromium] Rename WEBKIT_API to WEBKIT_EXPORT.
​https://bugs.webkit.org/show_bug.cgi?id=65649

Reviewed by Adam Barth.

  • README:
  • public/WebAccessibilityCache.h:
  • public/WebAccessibilityObject.h:
  • public/WebArrayBufferView.h:
  • public/WebAttribute.h:
  • public/WebAudioBus.h:
  • public/WebBindings.h:
  • public/WebBlobData.h:
  • public/WebBlobRegistry.h:
  • public/WebCString.h:
  • public/WebCache.h:
  • public/WebColor.h:
  • public/WebCommon.h:
  • public/WebCrossOriginPreflightResultCache.h:
  • public/WebDOMEvent.h:
  • public/WebDOMEventListener.h:
  • public/WebDOMMouseEvent.h:
  • public/WebDOMMutationEvent.h:
  • public/WebDOMStringList.h:
  • public/WebData.h:
  • public/WebDatabase.h:
  • public/WebDevToolsAgent.h:
  • public/WebDevToolsFrontend.h:
  • public/WebDeviceOrientationClientMock.h:
  • public/WebDeviceOrientationController.h:
  • public/WebDocument.h:
  • public/WebDocumentType.h:
  • public/WebDragData.h:
  • public/WebElement.h:
  • public/WebFont.h:
  • public/WebFontCache.h:
  • public/WebFormControlElement.h:
  • public/WebFormElement.h:
  • public/WebFrame.h:
  • public/WebGeolocationClientMock.h:
  • public/WebGeolocationController.h:
  • public/WebGeolocationError.h:
  • public/WebGeolocationPermissionRequest.h:
  • public/WebGeolocationPermissionRequestManager.h:
  • public/WebGeolocationPosition.h:
  • public/WebGlyphCache.h:
  • public/WebHTTPBody.h:
  • public/WebHTTPLoadInfo.h:
  • public/WebHistoryItem.h:
  • public/WebIDBDatabaseError.h:
  • public/WebIDBFactory.h:
  • public/WebIDBKey.h:
  • public/WebIDBKeyPath.h:
  • public/WebIDBKeyRange.h:
  • public/WebImage.h:
  • public/WebImageDecoder.h:
  • public/WebInputElement.h:
  • public/WebInputEvent.h:
  • public/WebKit.h:
  • public/WebLabelElement.h:
  • public/WebMediaElement.h:
  • public/WebNamedNodeMap.h:
  • public/WebNetworkStateNotifier.h:
  • public/WebNode.h:
  • public/WebNodeCollection.h:
  • public/WebNodeList.h:
  • public/WebNotification.h:
  • public/WebOptionElement.h:
  • public/WebPageSerializer.h:
  • public/WebPasswordFormData.h:
  • public/WebPerformance.h:
  • public/WebPluginDocument.h:
  • public/WebPopupMenu.h:
  • public/WebRange.h:
  • public/WebRegularExpression.h:
  • public/WebRuntimeFeatures.h:
  • public/WebScriptController.h:
  • public/WebScrollbar.h:
  • public/WebSearchableFormData.h:
  • public/WebSecurityOrigin.h:
  • public/WebSecurityPolicy.h:
  • public/WebSelectElement.h:
  • public/WebSerializedScriptValue.h:
  • public/WebSharedWorker.h:
  • public/WebSpeechInputControllerMock.h:
  • public/WebSpeechInputResult.h:
  • public/WebStorageEventDispatcher.h:
  • public/WebStorageNamespace.h:
  • public/WebString.h:
  • public/WebTestingSupport.h:
  • public/WebThreadSafeData.h:
  • public/WebURLLoadTiming.h:
  • public/WebURLRequest.h:
  • public/WebURLResponse.h:
  • public/WebView.h:
  • public/WebWorker.h:
  • public/gtk/WebFontInfo.h:
  • public/gtk/WebInputEventFactory.h:
  • public/linux/WebFontRenderStyle.h:
  • public/linux/WebFontRendering.h:
  • public/linux/WebRenderTheme.h:
  • public/mac/WebInputEventFactory.h:
  • public/mac/WebScreenInfoFactory.h:
  • public/mac/WebSubstringUtil.h:
  • public/win/WebInputEventFactory.h:
  • public/win/WebScreenInfoFactory.h:
  • public/x11/WebScreenInfoFactory.h:
  • tests/WebUnitTests.h:
3:50 PM Changeset in webkit [92321] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Remove LegacyDefaultOptionalArguments flag from DOMSelection.idl
​https://bugs.webkit.org/show_bug.cgi?id=65566

Patch by Mark Pilgrim <​pilgrim@chromium.org> on 2011-08-03
Reviewed by Adam Barth.

No new tests, all existing tests pass.

  • page/DOMSelection.idl:
3:47 PM Changeset in webkit [92320] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Remove LegacyDefaultOptionalArguments flag from Location.idl
​https://bugs.webkit.org/show_bug.cgi?id=65565

Patch by Mark Pilgrim <​pilgrim@chromium.org> on 2011-08-03
Reviewed by Darin Adler.

No new tests, all existing tests pass.

  • page/Location.idl:
3:42 PM Changeset in webkit [92319] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Chromium Mac: Fix position of search tickmarks
​https://bugs.webkit.org/show_bug.cgi?id=65575

This change adjusts the position of search tickmarks so that they are correctly centered.

Also, insetting the tickmark rect by 5 pixels caused tickmarks on the overlay scrollbar to look very small. I changed this to inset by 4 pixels instead. The tickmark is the same width as the overlay scrollbar which looks much better.

Patch by Sailesh Agrawal <​sail@chromium.org> on 2011-08-03
Reviewed by James Robinson.

  • platform/chromium/ScrollbarThemeChromiumMac.mm:

(WebCore::ScrollbarThemeChromiumMac::paint):

3:36 PM Changeset in webkit [92318] by Michael Nordman
  • 3 edits in trunk/Source/WebKit/chromium

[Chromium] Minor cleanup, remove a deprecated method.
The old WebDatabase::updateDatabaseSize method has been replaced with
updateDatabaseSize, updateSpaceAvailable, and resetSpaceAvailable.
​https://bugs.webkit.org/show_bug.cgi?id=65553

Reviewed by Adam Barth.

  • public/WebDatabase.h:
  • src/WebDatabase.cpp:
3:34 PM Changeset in webkit [92317] by Dimitri Glazkov
  • 3 edits in trunk/Tools

Allow multiple bug identifiers in test expectations.
​https://bugs.webkit.org/show_bug.cgi?id=65642

Reviewed by Adam Barth.

  • Scripts/webkitpy/layout_tests/models/test_expectations.py: Tweaked to allow multiple bug ids.
  • Scripts/webkitpy/layout_tests/models/test_expectations_unittest.py: Added tests.
3:24 PM Changeset in webkit [92316] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Chromium Mac: Make sure scrollbars flash when web page loads
​https://bugs.webkit.org/show_bug.cgi?id=65586

On slow web pages scrollbars wouldn't flash when the page was done loading. The problem was that we were flashing the scrollbar 0.1 second after the load operation had started. If the page was slow to load then we might not have scrollbars to flash. To work around this I added an extra check to make sure that we had indeed finished loading the page before flashing the scrollbars.

Patch by Sailesh Agrawal <​sail@chromium.org> on 2011-08-03
Reviewed by James Robinson.

  • platform/chromium/ScrollAnimatorChromiumMac.mm:

(WebCore::ScrollAnimatorChromiumMac::initialScrollbarPaintTimerFired):

2:58 PM Changeset in webkit [92315] by commit-queue@webkit.org
  • 7 edits in trunk

Remove LegacyDefaultOptionalArguments flag from navigator IDL files
​https://bugs.webkit.org/show_bug.cgi?id=65370

Patch by Mark Pilgrim <​pilgrim@chromium.org> on 2011-08-03
Reviewed by Adam Barth.

Source/WebCore:

  • page/Navigator.idl:
  • page/NavigatorUserMediaErrorCallback.idl:
  • page/NavigatorUserMediaSuccessCallback.idl:

LayoutTests:

  • fast/dom/navigator-detached-no-crash-expected.txt:
  • platform/chromium/fast/dom/navigator-detached-no-crash-expected.txt:
2:53 PM Changeset in webkit [92314] by andersca@apple.com
  • 2 edits in trunk/Source/WebCore

webarchive/loading/missing-data.html is failing on Mac
​https://bugs.webkit.org/show_bug.cgi?id=63507

Reviewed by Adam Barth.

When loading a main resource from a web archive, make sure that the working URL is actually
the URL of the resource, and not the URL of the web archive itself.

  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::receivedFirstData):

2:50 PM Changeset in webkit [92313] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebCore

Remove LegacyDefaultOptionalArguments flag from DOMWindow.idl
​https://bugs.webkit.org/show_bug.cgi?id=65353

Patch by Mark Pilgrim <​pilgrim@chromium.org> on 2011-08-03
Reviewed by Adam Barth.

No new tests, all existing tests pass.

  • bindings/scripts/CodeGeneratorJS.pm:

(GenerateImplementation):

  • bindings/scripts/CodeGeneratorV8.pm:

(GenerateFunctionCallback):

  • page/DOMWindow.idl:
2:22 PM Changeset in webkit [92312] by Lucas Forschler
  • 2 edits in branches/safari-534.51-branch/Source/WebCore

Merge r91447.

2:16 PM Changeset in webkit [92311] by Lucas Forschler
  • 2 edits in branches/safari-534.51-branch/Source/WebCore

Merge r91438.

2:04 PM Changeset in webkit [92310] by Lucas Forschler
  • 17 edits in branches/safari-534.51-branch/Source

Merge r91435.

1:52 PM Changeset in webkit [92309] by jeffm@apple.com
  • 2 edits in trunk/Source/WebCore

Fix Chromimum and Qt builds after r92308.

  • config.h: Don't try to include WebCoreHeaderDetection.h for Chromimum and Qt.
1:15 PM Changeset in webkit [92308] by jeffm@apple.com
  • 3 edits in trunk/Source/WebCore

Generate WebCoreHeaderDetection.h on Windows to define HAVE_AVCF
​https://bugs.webkit.org/show_bug.cgi?id=65634

AVFoundation support on Windows will (eventually) be predicated on the
availability of AVFoundationCF headers and libraries in the WebKit Support
Libraries, so we test for the existence of one of the header files (AVCFBase.h)
to determine whether to AVCF is available.

For now, even if AVFoundationCF is available, leave WTF_USE_AVFOUNDATION off, since
it's not ready to be turned on yet (which is covered by ​http://webkit.org/b/65400).

Ideally, we would generate WebCoreHeaderDetection.h on all platforms so we could
include it unconditionally in WebCore's config.h, but unfortunately the Mac-only file
ExportFileGenerator.cpp depends on WTF_USE_AVFOUNDATION being set correctly, and since this
.cpp file is also generated it doesn't have access to WebCoreHeaderDetection.h.

Reviewed by Adam Roben.

No change in functionality, so new tests.

  • DerivedSources.make: Add rule to generate WebCoreHeaderDetection.h to set HAVE_AVCF on Windows.
  • config.h: Include WebCoreHeaderDetection.h on Windows, check for HAVE(AVCF) but always leave WTF_USE_AVFOUNDATION off on Windows for now.
1:13 PM Changeset in webkit [92307] by zmo@google.com
  • 2 edits in trunk/LayoutTests

Unreviewed, fix an error in test_expectations.txt.

  • platform/chromium/test_expectations.txt:
12:57 PM Changeset in webkit [92306] by zmo@google.com
  • 2 edits in trunk/LayoutTests

Unreviewed, skip fast/loader/reload-zero-byte-plugin.html for now, gardener stuff.

  • platform/chromium/test_expectations.txt:
12:55 PM Changeset in webkit [92305] by Michael Nordman
  • 10 edits in branches/chromium/835/Source/WebCore/storage

Merge 92155 - [Chromium] WebSQLDatabase version handling is broken in multi-process browsers.
​https://bugs.webkit.org/show_bug.cgi?id=65486

The WebCore::AbstractDatabase class maintains a global in-memory map of
the version numbers associated with open database files, but that map is
not reliable in a multi-process system like Chrome. So instead of relying
on the cached values in that map, we read the value from the database (and
update the cached value) where possible. There are two edge cases where that's
not possible because the scriptable interface requires synchronous access
to the version: the .version attribute getter and the .openDatabase() method.
In those cases, we have no choice but to use the potentially stale cached value.

Reviewed by Darin Fisher.

No new tests. Existing layout tests cover the version handling functionality.

  • storage/AbstractDatabase.cpp:

(WebCore::AbstractDatabase::version):
(WebCore::AbstractDatabase::performOpenAndVerify):
(WebCore::AbstractDatabase::getVersionFromDatabase):
(WebCore::AbstractDatabase::setVersionInDatabase):
(WebCore::AbstractDatabase::setExpectedVersion):
(WebCore::AbstractDatabase::getCachedVersion):
(WebCore::AbstractDatabase::setCachedVersion):
(WebCore::AbstractDatabase::getActualVersionForTransaction):

  • storage/AbstractDatabase.h:

(WebCore::AbstractDatabase::expectedVersion):

  • storage/ChangeVersionWrapper.cpp:

(WebCore::ChangeVersionWrapper::handleCommitFailedAfterPostflight):

  • storage/ChangeVersionWrapper.h:
  • storage/Database.cpp:

(WebCore::Database::openDatabase):

  • storage/DatabaseSync.cpp:

(WebCore::DatabaseSync::openDatabaseSync):
(WebCore::DatabaseSync::changeVersion):

  • storage/SQLTransaction.cpp:

(WebCore::SQLTransaction::SQLTransaction):
(WebCore::SQLTransaction::executeSQL):
(WebCore::SQLTransaction::openTransactionAndPreflight):
(WebCore::SQLTransaction::runCurrentStatement):
(WebCore::SQLTransaction::postflightAndCommit):

  • storage/SQLTransaction.h:
  • storage/SQLTransactionSync.cpp:

(WebCore::SQLTransactionSync::SQLTransactionSync):
(WebCore::SQLTransactionSync::executeSQL):
(WebCore::SQLTransactionSync::begin):

  • storage/SQLTransactionSync.h:

TBR=Michael Nordman
Review URL: ​http://codereview.chromium.org/7563014

12:54 PM Changeset in webkit [92304] by commit-queue@webkit.org
  • 27 edits
    2 copies
    7 adds in trunk

MediaStream API: Implement PeerConnection and SignalingCallback
​https://bugs.webkit.org/show_bug.cgi?id=65101

Patch by Tommy Widenflycht <​tommyw@google.com> on 2011-08-03
Reviewed by Adam Barth.

LayoutTests will be provided by a follow-up CL, see <​https://bugs.webkit.org/show_bug.cgi?id=56587>.

Source/WebCore:

  • CMakeLists.txt:
  • CodeGenerators.pri:
  • DerivedSources.cpp:
  • DerivedSources.make:
  • GNUmakefile.am:
  • GNUmakefile.list.am:
  • UseJSC.cmake:
  • WebCore.gyp/WebCore.gyp:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/generic/RuntimeEnabledFeatures.h:

(WebCore::RuntimeEnabledFeatures::peerConnectionEnabled):

  • bindings/js/JSBindingsAllInOne.cpp:
  • bindings/js/JSPeerConnectionCustom.cpp: Copied from Source/WebCore/page/MediaStreamClient.h.

(WebCore::JSPeerConnectionConstructor::constructJSPeerConnection):

  • bindings/v8/V8DOMWrapper.cpp:

(WebCore::V8DOMWrapper::convertEventTargetToV8Object):

  • bindings/v8/custom/V8PeerConnectionCustom.cpp: Added.

(WebCore::V8PeerConnection::constructorCallback):

  • dom/DispatchTask.h: Added.

(WebCore::SimpleDispatchTask::create):
(WebCore::SimpleDispatchTask::performTask):
(WebCore::SimpleDispatchTask::SimpleDispatchTask):
(WebCore::DispatchTask::create):
(WebCore::DispatchTask::performTask):
(WebCore::DispatchTask::DispatchTask):

  • dom/EventNames.h:
  • dom/EventTarget.cpp:

(WebCore::EventTarget::toPeerConnection):

  • dom/EventTarget.h:
  • p2p/PeerConnection.cpp: Added.

(WebCore::PeerConnection::create):
(WebCore::PeerConnection::PeerConnection):
(WebCore::PeerConnection::init):
(WebCore::PeerConnection::~PeerConnection):
(WebCore::PeerConnection::localStreams):
(WebCore::PeerConnection::remoteStreams):
(WebCore::PeerConnection::processSignalingMessage):
(WebCore::PeerConnection::send):
(WebCore::PeerConnection::addStream):
(WebCore::PeerConnection::removeStream):
(WebCore::PeerConnection::close):
(WebCore::PeerConnection::onNegotiationStarted):
(WebCore::PeerConnection::onNegotiationDone):
(WebCore::PeerConnection::streamAdded):
(WebCore::PeerConnection::streamRemoved):
(WebCore::PeerConnection::onMessage):
(WebCore::PeerConnection::onSignalingMessage):
(WebCore::PeerConnection::toPeerConnection):
(WebCore::PeerConnection::scriptExecutionContext):
(WebCore::PeerConnection::eventTargetData):
(WebCore::PeerConnection::ensureEventTargetData):
(WebCore::PeerConnection::postMessageEvent):
(WebCore::PeerConnection::postSignalingEvent):
(WebCore::PeerConnection::postStreamAddedEvent):
(WebCore::PeerConnection::postStreamRemovedEvent):
(WebCore::PeerConnection::postSimpleEvent):
(WebCore::PeerConnection::postStartNegotiationTask):
(WebCore::PeerConnection::dispatchMessageEvent):
(WebCore::PeerConnection::dispatchSignalingEvent):
(WebCore::PeerConnection::dispatchStreamEvent):
(WebCore::PeerConnection::dispatchSimpleEvent):
(WebCore::PeerConnection::dispatchStartNegotiationTask):

  • p2p/PeerConnection.h: Added.

(WebCore::PeerConnection::readyState):
(WebCore::PeerConnection::refEventTarget):
(WebCore::PeerConnection::derefEventTarget):

  • p2p/PeerConnection.idl: Added.
  • p2p/SignalingCallback.h: Copied from Source/WebCore/page/MediaStreamClient.h.

(WebCore::SignalingCallback::~SignalingCallback):

  • p2p/SignalingCallback.idl: Added.
  • page/DOMWindow.idl:
  • page/MediaStreamClient.h:
  • page/MediaStreamController.cpp:

(WebCore::MediaStreamController::MediaStreamController):
(WebCore::MediaStreamController::unregisterFrameController):
(WebCore::MediaStreamController::onSignalingMessage):
(WebCore::MediaStreamController::frameToPagePeerConnectionId):
(WebCore::MediaStreamController::processSignalingMessage):
(WebCore::MediaStreamController::message):
(WebCore::MediaStreamController::onMessage):
(WebCore::MediaStreamController::addStream):
(WebCore::MediaStreamController::onAddStream):
(WebCore::MediaStreamController::removeStream):
(WebCore::MediaStreamController::onRemoveStream):
(WebCore::MediaStreamController::newPeerConnection):
(WebCore::MediaStreamController::closePeerConnection):
(WebCore::MediaStreamController::startNegotiation):
(WebCore::MediaStreamController::onNegotiationStarted):
(WebCore::MediaStreamController::onNegotiationDone):

  • page/MediaStreamController.h:
  • page/MediaStreamFrameController.cpp:

(WebCore::MediaStreamFrameController::Request::isSignalingRequest):
(WebCore::MediaStreamFrameController::createPeerConnection):
(WebCore::MediaStreamFrameController::newPeerConnection):
(WebCore::MediaStreamFrameController::closePeerConnection):
(WebCore::MediaStreamFrameController::startNegotiation):
(WebCore::MediaStreamFrameController::processSignalingMessage):
(WebCore::MediaStreamFrameController::message):
(WebCore::MediaStreamFrameController::addStream):
(WebCore::MediaStreamFrameController::removeStream):
(WebCore::MediaStreamFrameController::onMessage):
(WebCore::MediaStreamFrameController::onAddStream):
(WebCore::MediaStreamFrameController::onRemoveStream):
(WebCore::MediaStreamFrameController::onSignalingMessage):
(WebCore::MediaStreamFrameController::onNegotiationStarted):
(WebCore::MediaStreamFrameController::onNegotiationDone):

  • page/MediaStreamFrameController.h:

(WebCore::MediaStreamFrameController::ClientBase::isPeerConnection):
(WebCore::MediaStreamFrameController::PeerConnectionClient::PeerConnectionClient):
(WebCore::MediaStreamFrameController::PeerConnectionClient::~PeerConnectionClient):
(WebCore::MediaStreamFrameController::PeerConnectionClient::isPeerConnection):

LayoutTests:

  • platform/chromium/fast/dom/prototype-inheritance-expected.txt:
12:38 PM Changeset in webkit [92303] by kevino@webkit.org
  • 2 edits in trunk

[wx] Unreviewed build fix after gesture recognizer changes.

12:36 PM Changeset in webkit [92302] by caio.oliveira@openbossa.org
  • 2 edits in trunk/Source/WebKit2

[Qt] [WK2] Improve position of context menu for QDesktopWebView
​https://bugs.webkit.org/show_bug.cgi?id=65635

Reviewed by Andreas Kling.

Do not assume that the QDesktopWebView takes the entire canvas when
positioning the context menu.

  • UIProcess/API/qt/qdesktopwebview.cpp:

(QDesktopWebViewPrivate::showContextMenu):

12:18 PM Changeset in webkit [92301] by Lucas Forschler
  • 4 edits in branches/safari-534.51-branch/Source

Merge r90705.

12:09 PM Changeset in webkit [92300] by Joseph Pecoraro
  • 15 edits in trunk/Source

Web Inspector: Remove InspectorFrontendHost load/saveSessionSetting
​https://bugs.webkit.org/show_bug.cgi?id=65582

Reviewed by Pavel Feldman.

Source/WebCore:

Remove stale code. WebInspector.settings uses localStorage and
handles the "debuggerEnabled" state. So remove the only remaining
usage of load/saveSessionSetting.

  • inspector/InspectorFrontendClient.h:
  • inspector/InspectorFrontendHost.cpp:
  • inspector/InspectorFrontendHost.h:
  • inspector/InspectorFrontendHost.idl:
  • inspector/front-end/InspectorFrontendHostStub.js:
  • inspector/front-end/ScriptsPanel.js:

(WebInspector.ScriptsPanel.prototype._debuggerWasDisabled):

Source/WebKit/cf:

  • WebCoreSupport/WebInspectorClientCF.cpp:

Source/WebKit/mac:

  • WebCoreSupport/WebInspectorClient.h:
  • WebCoreSupport/WebInspectorClient.mm:

Source/WebKit/win:

  • WebCoreSupport/WebInspectorClient.cpp:
  • WebCoreSupport/WebInspectorClient.h:
12:06 PM Changeset in webkit [92299] by enne@google.com
  • 1 edit in branches/chromium/835/Source/WebCore/page/FrameView.cpp

Merge 92030 - [chromium] Only force slow scrolling for iframes under --force-compositing-mode
​https://bugs.webkit.org/show_bug.cgi?id=65394

Reviewed by James Robinson.

The previous code introduced by r91591 made composited frames as well
as non-composited subframes go down the slow scrolling path. Adjust
the check to use ownerElement to test for subframe and use
hasCompositedContent() which is a more correct test for compositing
than isEnclosedInCompositingLayer.

  • page/FrameView.cpp:

(WebCore::FrameView::useSlowRepaints):

TBR=​enne@google.com
Review URL: ​http://codereview.chromium.org/7565014

11:56 AM Changeset in webkit [92298] by abarth@webkit.org
  • 3 edits
    2 adds in trunk

Crash in DocumentWriter::endIfNotLoadingMainResource
​https://bugs.webkit.org/show_bug.cgi?id=65581

Reviewed by Nate Chapin.

Source/WebCore:

This function is poorly designed because isLoadingMainResource is a
poor proxy for determing whether to flush/finish the parser. Really,
we should change how loads complete to match the model in HTML5, but
that's pretty tricky. In the meantime, this null check fixes the
crash.

I'm sure there's another bug on file about this crash, but we've never
been able to reproduce it. Thanks to Berend-Jan Wever for the test
case!

Test: fast/loader/reload-zero-byte-plugin.html

  • loader/DocumentWriter.cpp:

(WebCore::DocumentWriter::endIfNotLoadingMainResource):

LayoutTests:

  • fast/loader/reload-zero-byte-plugin-expected.txt: Added.
  • fast/loader/reload-zero-byte-plugin.html: Added.
11:40 AM Changeset in webkit [92297] by senorblanco@chromium.org
  • 1 edit in branches/chromium/835/Source/WebCore/platform/graphics/skia/ImageBufferSkia.cpp

Merge 92227 - [Chromium] Remove an unnecessary readback during accelerated
canvas-to-canvas draws.
​https://bugs.webkit.org/show_bug.cgi?id=65560

Reviewed by James Robinson.

No new tests. (Printing has few tests, sadly.)

  • platform/graphics/skia/ImageBufferSkia.cpp:

(WebCore::ImageBuffer::draw):
Revert most of the changes to this file from
​http://trac.webkit.org/changeset/91870, except for the call which makes
the source context current, so that the readPixels() still works.

TBR=​senorblanco@chromium.org
Review URL: ​http://codereview.chromium.org/7465097

11:38 AM Changeset in webkit [92296] by senorblanco@chromium.org
  • 2 edits in branches/chromium/835/Source/WebCore

Merge 91870 - Fix for printing with accelerated <canvas>
​https://bugs.webkit.org/show_bug.cgi?id=62406

Reviewed by James Robinson.

The platform-independent side of this should be covered by existing
tests. Printing itself in Chrome has few tests, sadly. :(

  • html/HTMLCanvasElement.cpp:

(WebCore::HTMLCanvasElement::paint):

  • platform/graphics/skia/ImageBufferSkia.cpp:

(WebCore::ImageBuffer::draw):

TBR=​senorblanco@chromium.org
Review URL: ​http://codereview.chromium.org/7564022

11:38 AM Changeset in webkit [92295] by andersca@apple.com
  • 2 edits in trunk/Source/WebCore

WebCore images backing CG patterns should be released on the main thread
​https://bugs.webkit.org/show_bug.cgi?id=65630

Reviewed by Sam Weinig.

  • platform/graphics/cg/PatternCG.cpp:

(WebCore::patternReleaseOnMainThreadCallback):
Deref the image.

(WebCore::patternReleaseCallback):
Call callOnMainThread.

11:11 AM Changeset in webkit [92294] by Dimitri Glazkov
  • 3 edits in trunk/Tools

Teach BuilderToPort to distinguish between Release and Debug builders.
​https://bugs.webkit.org/show_bug.cgi?id=65628

Reviewed by Adam Barth.

  • Scripts/webkitpy/tool/commands/rebaseline.py: Added a smart options object and passed it to port factory.
  • Scripts/webkitpy/tool/commands/rebaseline_unittest.py: Added tests.
10:55 AM Changeset in webkit [92293] by leviw@chromium.org
  • 16 edits
    343 deletes in branches/chromium/835

REGRESSION(87526): ASSERT(!needsLayout()) followed by graphical glitches on google charts (svg loaded in iframe)
​https://bugs.webkit.org/show_bug.cgi?id=64974

Reviewed by James Robinson.

Rolling back r87526 for Chrome 14.

Source/WebCore:

  • platform/Length.h:

(WebCore::Length::isIntrinsicOrAuto):

  • rendering/RenderPart.cpp:

(WebCore::RenderPart::embeddedContentBox):

  • rendering/RenderReplaced.cpp:

(WebCore::lengthIsSpecified):
(WebCore::RenderReplaced::computeReplacedLogicalWidth):
(WebCore::RenderReplaced::computeReplacedLogicalHeight):

  • rendering/svg/RenderSVGRoot.cpp:

(WebCore::RenderSVGRoot::RenderSVGRoot):
(WebCore::RenderSVGRoot::computeReplacedLogicalWidth):
(WebCore::RenderSVGRoot::computeReplacedLogicalHeight):
(WebCore::RenderSVGRoot::layout):

  • rendering/svg/RenderSVGRoot.h:
  • svg/SVGLength.cpp:

(WebCore::SVGLength::determineViewport):

  • svg/SVGSVGElement.cpp:

(WebCore::SVGSVGElement::SVGSVGElement):
(WebCore::SVGSVGElement::currentScale):
(WebCore::SVGSVGElement::setCurrentScale):
(WebCore::SVGSVGElement::viewBoxToViewTransform):

  • svg/SVGSVGElement.h:

LayoutTests:

  • css2.1/20110323/replaced-intrinsic-001.htm: Removed.
  • css2.1/20110323/replaced-intrinsic-002.htm: Removed.
  • css2.1/20110323/replaced-intrinsic-003.htm: Removed.
  • css2.1/20110323/replaced-intrinsic-004.htm: Removed.
  • css2.1/20110323/replaced-intrinsic-005.htm: Removed.
  • css2.1/20110323/replaced-intrinsic-ratio-001.htm: Removed.
  • http/tests/misc/object-embedding-svg-delayed-size-negotiation-2.htm: Removed.
  • http/tests/misc/object-embedding-svg-delayed-size-negotiation.xhtml: Removed.
  • http/tests/misc/resources/embedded.svg: Removed.
  • http/tests/misc/resources/intrinsic-ratio-slow.pl: Removed.
  • http/tests/misc/resources/intrinsic-ratio.svg: Removed.
  • http/tests/misc/resources/svg-slow.pl: Removed.
  • platform/chromium-linux/css2.1/20110323/replaced-intrinsic-001-expected.png: Removed.
  • platform/chromium-linux/css2.1/20110323/replaced-intrinsic-002-expected.png: Removed.
  • platform/chromium-linux/css2.1/20110323/replaced-intrinsic-003-expected.png: Removed.
  • platform/chromium-linux/css2.1/20110323/replaced-intrinsic-004-expected.png: Removed.
  • platform/chromium-linux/css2.1/20110323/replaced-intrinsic-005-expected.png: Removed.
  • platform/chromium-linux/svg/custom/object-sizing-expected.png: Removed.
  • platform/chromium-linux/svg/custom/object-sizing-explicit-height-expected.png: Removed.
  • platform/chromium-linux/svg/custom/object-sizing-explicit-width-expected.png: Removed.
  • platform/chromium-linux/svg/custom/object-sizing-explicit-width-height-expected.png: Removed.
  • platform/chromium-linux/svg/custom/object-sizing-explicit-width-height-expected.txt: Removed.
  • platform/chromium-linux/svg/custom/object-sizing-no-width-height-change-content-box-size-expected.png: Removed.
  • platform/chromium-linux/svg/custom/object-sizing-no-width-height-expected.png: Removed.
  • platform/chromium-linux/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-absolute-expected.txt: Removed.
  • platform/chromium-linux/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-expected.txt: Removed.
  • platform/chromium-linux/svg/custom/object-sizing-width-75p-height-50p-on-target-svg-absolute-expected.txt: Removed.
  • platform/chromium-linux/svg/custom/object-sizing-width-75p-height-50p-on-target-svg-expected.txt: Removed.
  • platform/chromium-linux/svg/hixie/intrinsic/001-expected.png: Removed.
  • platform/chromium-linux/svg/hixie/intrinsic/002-expected.png: Removed.
  • platform/chromium-linux/svg/hixie/intrinsic/003-expected.png: Removed.
  • platform/chromium-linux/svg/hixie/intrinsic/003-expected.txt: Removed.
  • platform/chromium-linux/svg/wicd/rightsizing-grid-expected.png: Removed.
  • platform/chromium-linux/svg/wicd/rightsizing-grid-expected.txt: Removed.
  • platform/chromium-linux/svg/wicd/test-rightsizing-a-expected.png: Removed.
  • platform/chromium-linux/svg/wicd/test-rightsizing-b-expected.png: Removed.
  • platform/chromium-linux/svg/zoom/page/zoom-replaced-intrinsic-ratio-001-expected.png: Removed.
  • platform/chromium-mac-leopard/css2.1/20110323/replaced-intrinsic-001-expected.png: Removed.
  • platform/chromium-mac-leopard/css2.1/20110323/replaced-intrinsic-002-expected.png: Removed.
  • platform/chromium-mac-leopard/css2.1/20110323/replaced-intrinsic-003-expected.png: Removed.
  • platform/chromium-mac-leopard/css2.1/20110323/replaced-intrinsic-004-expected.png: Removed.
  • platform/chromium-mac-leopard/css2.1/20110323/replaced-intrinsic-005-expected.png: Removed.
  • platform/chromium-mac-leopard/css2.1/20110323/replaced-intrinsic-ratio-001-expected.png: Removed.
  • platform/chromium-mac-leopard/svg/custom/object-sizing-expected.png: Removed.
  • platform/chromium-mac-leopard/svg/hixie/intrinsic/001-expected.png: Removed.
  • platform/chromium-mac-leopard/svg/hixie/intrinsic/002-expected.png: Removed.
  • platform/chromium-mac-leopard/svg/hixie/intrinsic/003-expected.png: Removed.
  • platform/chromium-mac-leopard/svg/wicd/rightsizing-grid-expected.png: Removed.
  • platform/chromium-mac-leopard/svg/wicd/test-rightsizing-b-expected.png: Removed.
  • platform/chromium-mac-leopard/svg/zoom/page/zoom-replaced-intrinsic-ratio-001-expected.png: Removed.
  • platform/chromium-mac/svg/wicd/rightsizing-grid-expected.png: Removed.
  • platform/chromium-mac/svg/zoom/page/zoom-replaced-intrinsic-ratio-001-expected.png: Removed.
  • platform/chromium-mac/svg/zoom/page/zoom-replaced-intrinsic-ratio-001-expected.txt: Removed.
  • platform/chromium-win-vista/svg/wicd/test-rightsizing-b-expected.png: Removed.
  • platform/chromium-win/css2.1/20110323/replaced-intrinsic-001-expected.png: Removed.
  • platform/chromium-win/css2.1/20110323/replaced-intrinsic-001-expected.txt: Removed.
  • platform/chromium-win/css2.1/20110323/replaced-intrinsic-002-expected.png: Removed.
  • platform/chromium-win/css2.1/20110323/replaced-intrinsic-002-expected.txt: Removed.
  • platform/chromium-win/css2.1/20110323/replaced-intrinsic-003-expected.png: Removed.
  • platform/chromium-win/css2.1/20110323/replaced-intrinsic-003-expected.txt: Removed.
  • platform/chromium-win/css2.1/20110323/replaced-intrinsic-004-expected.png: Removed.
  • platform/chromium-win/css2.1/20110323/replaced-intrinsic-004-expected.txt: Removed.
  • platform/chromium-win/css2.1/20110323/replaced-intrinsic-005-expected.png: Removed.
  • platform/chromium-win/css2.1/20110323/replaced-intrinsic-005-expected.txt: Removed.
  • platform/chromium-win/svg/custom/object-sizing-expected.png: Removed.
  • platform/chromium-win/svg/custom/object-sizing-expected.txt: Removed.
  • platform/chromium-win/svg/custom/object-sizing-explicit-height-expected.png: Removed.
  • platform/chromium-win/svg/custom/object-sizing-explicit-height-expected.txt: Removed.
  • platform/chromium-win/svg/custom/object-sizing-explicit-width-expected.png: Removed.
  • platform/chromium-win/svg/custom/object-sizing-explicit-width-expected.txt: Removed.
  • platform/chromium-win/svg/custom/object-sizing-explicit-width-height-expected.png: Removed.
  • platform/chromium-win/svg/custom/object-sizing-explicit-width-height-expected.txt: Removed.
  • platform/chromium-win/svg/custom/object-sizing-height-50p-on-target-svg-absolute-expected.png: Removed.
  • platform/chromium-win/svg/custom/object-sizing-height-50p-on-target-svg-absolute-expected.txt: Removed.
  • platform/chromium-win/svg/custom/object-sizing-height-50p-on-target-svg-expected.png: Removed.
  • platform/chromium-win/svg/custom/object-sizing-height-50p-on-target-svg-expected.txt: Removed.
  • platform/chromium-win/svg/custom/object-sizing-no-width-height-change-content-box-size-expected.png: Removed.
  • platform/chromium-win/svg/custom/object-sizing-no-width-height-expected.png: Removed.
  • platform/chromium-win/svg/custom/object-sizing-no-width-height-expected.txt: Removed.
  • platform/chromium-win/svg/custom/object-sizing-width-50p-height-50p-on-target-svg-absolute-expected.png: Removed.
  • platform/chromium-win/svg/custom/object-sizing-width-50p-height-50p-on-target-svg-absolute-expected.txt: Removed.
  • platform/chromium-win/svg/custom/object-sizing-width-50p-height-50p-on-target-svg-expected.png: Removed.
  • platform/chromium-win/svg/custom/object-sizing-width-50p-height-50p-on-target-svg-expected.txt: Removed.
  • platform/chromium-win/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-absolute-expected.png: Removed.
  • platform/chromium-win/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-absolute-expected.txt: Removed.
  • platform/chromium-win/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-expected.png: Removed.
  • platform/chromium-win/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-expected.txt: Removed.
  • platform/chromium-win/svg/custom/object-sizing-width-50p-on-target-svg-absolute-expected.png: Removed.
  • platform/chromium-win/svg/custom/object-sizing-width-50p-on-target-svg-absolute-expected.txt: Removed.
  • platform/chromium-win/svg/custom/object-sizing-width-50p-on-target-svg-expected.png: Removed.
  • platform/chromium-win/svg/custom/object-sizing-width-50p-on-target-svg-expected.txt: Removed.
  • platform/chromium-win/svg/custom/object-sizing-width-75p-height-50p-on-target-svg-absolute-expected.png: Removed.
  • platform/chromium-win/svg/custom/object-sizing-width-75p-height-50p-on-target-svg-absolute-expected.txt: Removed.
  • platform/chromium-win/svg/custom/object-sizing-width-75p-height-50p-on-target-svg-expected.png: Removed.
  • platform/chromium-win/svg/custom/object-sizing-width-75p-height-50p-on-target-svg-expected.txt: Removed.
  • platform/chromium-win/svg/hixie/intrinsic/001-expected.png: Removed.
  • platform/chromium-win/svg/hixie/intrinsic/001-expected.txt: Removed.
  • platform/chromium-win/svg/hixie/intrinsic/002-expected.png: Removed.
  • platform/chromium-win/svg/hixie/intrinsic/002-expected.txt: Removed.
  • platform/chromium-win/svg/hixie/intrinsic/003-expected.png: Removed.
  • platform/chromium-win/svg/hixie/intrinsic/003-expected.txt: Removed.
  • platform/chromium-win/svg/wicd/rightsizing-grid-expected.png: Removed.
  • platform/chromium-win/svg/wicd/rightsizing-grid-expected.txt: Removed.
  • platform/chromium-win/svg/wicd/test-rightsizing-a-expected.png: Removed.
  • platform/chromium-win/svg/wicd/test-rightsizing-a-expected.txt: Removed.
  • platform/chromium-win/svg/wicd/test-rightsizing-b-expected.png: Removed.
  • platform/chromium-win/svg/wicd/test-rightsizing-b-expected.txt: Removed.
  • platform/chromium-win/svg/zoom/page/zoom-replaced-intrinsic-ratio-001-expected.png: Removed.
  • platform/chromium-win/svg/zoom/page/zoom-replaced-intrinsic-ratio-001-expected.txt: Removed.
  • platform/gtk/css2.1/20110323/replaced-intrinsic-001-expected.png: Removed.
  • platform/gtk/css2.1/20110323/replaced-intrinsic-001-expected.txt: Removed.
  • platform/gtk/css2.1/20110323/replaced-intrinsic-002-expected.png: Removed.
  • platform/gtk/css2.1/20110323/replaced-intrinsic-002-expected.txt: Removed.
  • platform/gtk/css2.1/20110323/replaced-intrinsic-003-expected.png: Removed.
  • platform/gtk/css2.1/20110323/replaced-intrinsic-003-expected.txt: Removed.
  • platform/gtk/css2.1/20110323/replaced-intrinsic-004-expected.png: Removed.
  • platform/gtk/css2.1/20110323/replaced-intrinsic-004-expected.txt: Removed.
  • platform/gtk/css2.1/20110323/replaced-intrinsic-005-expected.png: Removed.
  • platform/gtk/css2.1/20110323/replaced-intrinsic-005-expected.txt: Removed.
  • platform/gtk/css2.1/20110323/replaced-intrinsic-ratio-001-expected.png: Removed.
  • platform/gtk/css2.1/20110323/replaced-intrinsic-ratio-001-expected.txt: Removed.
  • platform/gtk/svg/custom/object-sizing-expected.png: Removed.
  • platform/gtk/svg/custom/object-sizing-expected.txt: Removed.
  • platform/gtk/svg/custom/object-sizing-explicit-height-expected.png: Removed.
  • platform/gtk/svg/custom/object-sizing-explicit-height-expected.txt: Removed.
  • platform/gtk/svg/custom/object-sizing-explicit-width-expected.png: Removed.
  • platform/gtk/svg/custom/object-sizing-explicit-width-expected.txt: Removed.
  • platform/gtk/svg/custom/object-sizing-explicit-width-height-expected.png: Removed.
  • platform/gtk/svg/custom/object-sizing-explicit-width-height-expected.txt: Removed.
  • platform/gtk/svg/custom/object-sizing-height-50p-on-target-svg-absolute-expected.png: Removed.
  • platform/gtk/svg/custom/object-sizing-height-50p-on-target-svg-absolute-expected.txt: Removed.
  • platform/gtk/svg/custom/object-sizing-height-50p-on-target-svg-expected.png: Removed.
  • platform/gtk/svg/custom/object-sizing-height-50p-on-target-svg-expected.txt: Removed.
  • platform/gtk/svg/custom/object-sizing-no-width-height-change-content-box-size-expected.png: Removed.
  • platform/gtk/svg/custom/object-sizing-no-width-height-change-content-box-size-expected.txt: Removed.
  • platform/gtk/svg/custom/object-sizing-no-width-height-expected.png: Removed.
  • platform/gtk/svg/custom/object-sizing-no-width-height-expected.txt: Removed.
  • platform/gtk/svg/custom/object-sizing-width-50p-height-50p-on-target-svg-absolute-expected.png: Removed.
  • platform/gtk/svg/custom/object-sizing-width-50p-height-50p-on-target-svg-absolute-expected.txt: Removed.
  • platform/gtk/svg/custom/object-sizing-width-50p-height-50p-on-target-svg-expected.png: Removed.
  • platform/gtk/svg/custom/object-sizing-width-50p-height-50p-on-target-svg-expected.txt: Removed.
  • platform/gtk/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-absolute-expected.png: Removed.
  • platform/gtk/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-absolute-expected.txt: Removed.
  • platform/gtk/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-expected.png: Removed.
  • platform/gtk/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-expected.txt: Removed.
  • platform/gtk/svg/custom/object-sizing-width-50p-on-target-svg-absolute-expected.png: Removed.
  • platform/gtk/svg/custom/object-sizing-width-50p-on-target-svg-absolute-expected.txt: Removed.
  • platform/gtk/svg/custom/object-sizing-width-50p-on-target-svg-expected.png: Removed.
  • platform/gtk/svg/custom/object-sizing-width-50p-on-target-svg-expected.txt: Removed.
  • platform/gtk/svg/custom/object-sizing-width-75p-height-50p-on-target-svg-absolute-expected.png: Removed.
  • platform/gtk/svg/custom/object-sizing-width-75p-height-50p-on-target-svg-absolute-expected.txt: Removed.
  • platform/gtk/svg/custom/object-sizing-width-75p-height-50p-on-target-svg-expected.png: Removed.
  • platform/gtk/svg/custom/object-sizing-width-75p-height-50p-on-target-svg-expected.txt: Removed.
  • platform/gtk/svg/hixie/intrinsic/001-expected.png: Removed.
  • platform/gtk/svg/hixie/intrinsic/001-expected.txt: Removed.
  • platform/gtk/svg/hixie/intrinsic/002-expected.png: Removed.
  • platform/gtk/svg/hixie/intrinsic/002-expected.txt: Removed.
  • platform/gtk/svg/hixie/intrinsic/003-expected.png: Removed.
  • platform/gtk/svg/hixie/intrinsic/003-expected.txt: Removed.
  • platform/gtk/svg/wicd/rightsizing-grid-expected.png: Removed.
  • platform/gtk/svg/wicd/rightsizing-grid-expected.txt: Removed.
  • platform/gtk/svg/wicd/test-rightsizing-a-expected.png: Removed.
  • platform/gtk/svg/wicd/test-rightsizing-a-expected.txt: Removed.
  • platform/gtk/svg/wicd/test-rightsizing-b-expected.png: Removed.
  • platform/gtk/svg/wicd/test-rightsizing-b-expected.txt: Removed.
  • platform/mac-wk2/svg/zoom/page/zoom-replaced-intrinsic-ratio-001-expected.txt: Removed.
  • platform/mac/css2.1/20110323/replaced-intrinsic-001-expected.png: Removed.
  • platform/mac/css2.1/20110323/replaced-intrinsic-001-expected.txt: Removed.
  • platform/mac/css2.1/20110323/replaced-intrinsic-002-expected.png: Removed.
  • platform/mac/css2.1/20110323/replaced-intrinsic-002-expected.txt: Removed.
  • platform/mac/css2.1/20110323/replaced-intrinsic-003-expected.png: Removed.
  • platform/mac/css2.1/20110323/replaced-intrinsic-003-expected.txt: Removed.
  • platform/mac/css2.1/20110323/replaced-intrinsic-004-expected.png: Removed.
  • platform/mac/css2.1/20110323/replaced-intrinsic-004-expected.txt: Removed.
  • platform/mac/css2.1/20110323/replaced-intrinsic-005-expected.png: Removed.
  • platform/mac/css2.1/20110323/replaced-intrinsic-005-expected.txt: Removed.
  • platform/mac/css2.1/20110323/replaced-intrinsic-ratio-001-expected.png: Removed.
  • platform/mac/css2.1/20110323/replaced-intrinsic-ratio-001-expected.txt: Removed.
  • platform/mac/fast/block/positioning/rtl-fixed-positioning-expected.png:
  • platform/mac/fast/block/positioning/vertical-rl/fixed-positioning-expected.png:
  • platform/mac/http/tests/misc/object-embedding-svg-delayed-size-negotiation-expected.png: Removed.
  • platform/mac/http/tests/misc/object-embedding-svg-delayed-size-negotiation-expected.txt: Removed.
  • platform/mac/svg/custom/object-no-size-attributes-expected.png: Removed.
  • platform/mac/svg/custom/object-no-size-attributes-expected.txt: Removed.
  • platform/mac/svg/custom/object-sizing-expected.png: Removed.
  • platform/mac/svg/custom/object-sizing-expected.txt: Removed.
  • platform/mac/svg/custom/object-sizing-explicit-height-expected.png: Removed.
  • platform/mac/svg/custom/object-sizing-explicit-height-expected.txt: Removed.
  • platform/mac/svg/custom/object-sizing-explicit-width-expected.png: Removed.
  • platform/mac/svg/custom/object-sizing-explicit-width-expected.txt: Removed.
  • platform/mac/svg/custom/object-sizing-explicit-width-height-expected.png: Removed.
  • platform/mac/svg/custom/object-sizing-explicit-width-height-expected.txt: Removed.
  • platform/mac/svg/custom/object-sizing-height-50p-on-target-svg-absolute-expected.png: Removed.
  • platform/mac/svg/custom/object-sizing-height-50p-on-target-svg-absolute-expected.txt: Removed.
  • platform/mac/svg/custom/object-sizing-height-50p-on-target-svg-expected.png: Removed.
  • platform/mac/svg/custom/object-sizing-height-50p-on-target-svg-expected.txt: Removed.
  • platform/mac/svg/custom/object-sizing-no-width-height-change-content-box-size-expected.png: Removed.
  • platform/mac/svg/custom/object-sizing-no-width-height-change-content-box-size-expected.txt: Removed.
  • platform/mac/svg/custom/object-sizing-no-width-height-expected.png: Removed.
  • platform/mac/svg/custom/object-sizing-no-width-height-expected.txt: Removed.
  • platform/mac/svg/custom/object-sizing-width-50p-height-50p-on-target-svg-absolute-expected.png: Removed.
  • platform/mac/svg/custom/object-sizing-width-50p-height-50p-on-target-svg-absolute-expected.txt: Removed.
  • platform/mac/svg/custom/object-sizing-width-50p-height-50p-on-target-svg-expected.png: Removed.
  • platform/mac/svg/custom/object-sizing-width-50p-height-50p-on-target-svg-expected.txt: Removed.
  • platform/mac/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-absolute-expected.png: Removed.
  • platform/mac/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-absolute-expected.txt: Removed.
  • platform/mac/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-expected.png: Removed.
  • platform/mac/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-expected.txt: Removed.
  • platform/mac/svg/custom/object-sizing-width-50p-on-target-svg-absolute-expected.png: Removed.
  • platform/mac/svg/custom/object-sizing-width-50p-on-target-svg-absolute-expected.txt: Removed.
  • platform/mac/svg/custom/object-sizing-width-50p-on-target-svg-expected.png: Removed.
  • platform/mac/svg/custom/object-sizing-width-50p-on-target-svg-expected.txt: Removed.
  • platform/mac/svg/custom/object-sizing-width-75p-height-50p-on-target-svg-absolute-expected.png: Removed.
  • platform/mac/svg/custom/object-sizing-width-75p-height-50p-on-target-svg-absolute-expected.txt: Removed.
  • platform/mac/svg/custom/object-sizing-width-75p-height-50p-on-target-svg-expected.png: Removed.
  • platform/mac/svg/custom/object-sizing-width-75p-height-50p-on-target-svg-expected.txt: Removed.
  • platform/mac/svg/hixie/intrinsic/001-expected.png: Removed.
  • platform/mac/svg/hixie/intrinsic/001-expected.txt: Removed.
  • platform/mac/svg/hixie/intrinsic/002-expected.png: Removed.
  • platform/mac/svg/hixie/intrinsic/002-expected.txt: Removed.
  • platform/mac/svg/hixie/intrinsic/003-expected.png: Removed.
  • platform/mac/svg/hixie/intrinsic/003-expected.txt: Removed.
  • platform/mac/svg/hixie/text/003-expected.png: Removed.
  • platform/mac/svg/hixie/text/003-expected.txt:
  • platform/mac/svg/in-html/by-reference-expected.txt:
  • platform/mac/svg/wicd/rightsizing-grid-expected.png: Removed.
  • platform/mac/svg/wicd/rightsizing-grid-expected.txt: Removed.
  • platform/mac/svg/wicd/test-rightsizing-a-expected.png: Removed.
  • platform/mac/svg/wicd/test-rightsizing-a-expected.txt: Removed.
  • platform/mac/svg/wicd/test-rightsizing-b-expected.png: Removed.
  • platform/mac/svg/wicd/test-rightsizing-b-expected.txt: Removed.
  • platform/mac/svg/zoom/page/relative-sized-document-scrollbars-expected.txt:
  • platform/mac/svg/zoom/page/zoom-foreignObject-expected.png: Removed.
  • platform/mac/svg/zoom/page/zoom-foreignObject-expected.txt:
  • platform/mac/svg/zoom/page/zoom-replaced-intrinsic-ratio-001-expected.png: Removed.
  • platform/mac/svg/zoom/page/zoom-replaced-intrinsic-ratio-001-expected.txt: Removed.
  • platform/mac/svg/zoom/page/zoom-svg-through-object-with-absolute-size-2-expected.png: Removed.
  • platform/mac/svg/zoom/page/zoom-svg-through-object-with-absolute-size-expected.png: Removed.
  • platform/mac/svg/zoom/page/zoom-svg-through-object-with-huge-size-expected.png: Removed.
  • platform/mac/svg/zoom/page/zoom-svg-through-object-with-huge-size-expected.txt: Removed.
  • platform/mac/svg/zoom/page/zoom-svg-through-object-with-no-width-height-expected.png: Removed.
  • platform/mac/svg/zoom/page/zoom-svg-through-object-with-no-width-height-expected.txt: Removed.
  • platform/mac/svg/zoom/page/zoom-svg-through-object-with-override-size-expected.png: Removed.
  • platform/mac/svg/zoom/page/zoom-svg-through-object-with-override-size-expected.txt: Removed.
  • platform/mac/svg/zoom/page/zoom-svg-through-object-with-percentage-size-expected.png: Removed.
  • platform/mac/svg/zoom/page/zoom-svg-through-object-with-percentage-size-expected.txt: Removed.
  • platform/mac/svg/zoom/page/zoom-svg-through-object-with-text-expected.png: Removed.
  • platform/qt-4.8/svg/custom/object-sizing-expected.png: Removed.
  • platform/qt-4.8/svg/custom/object-sizing-expected.txt: Removed.
  • platform/qt-4.8/svg/wicd/test-rightsizing-a-expected.png: Removed.
  • platform/qt-4.8/svg/wicd/test-rightsizing-a-expected.txt: Removed.
  • platform/qt-4.8/svg/wicd/test-rightsizing-b-expected.png: Removed.
  • platform/qt-4.8/svg/wicd/test-rightsizing-b-expected.txt: Removed.
  • platform/qt/css2.1/20110323/replaced-intrinsic-001-expected.png: Removed.
  • platform/qt/css2.1/20110323/replaced-intrinsic-001-expected.txt: Removed.
  • platform/qt/css2.1/20110323/replaced-intrinsic-002-expected.png: Removed.
  • platform/qt/css2.1/20110323/replaced-intrinsic-002-expected.txt: Removed.
  • platform/qt/css2.1/20110323/replaced-intrinsic-003-expected.png: Removed.
  • platform/qt/css2.1/20110323/replaced-intrinsic-003-expected.txt: Removed.
  • platform/qt/css2.1/20110323/replaced-intrinsic-004-expected.png: Removed.
  • platform/qt/css2.1/20110323/replaced-intrinsic-004-expected.txt: Removed.
  • platform/qt/css2.1/20110323/replaced-intrinsic-005-expected.png: Removed.
  • platform/qt/css2.1/20110323/replaced-intrinsic-005-expected.txt: Removed.
  • platform/qt/css2.1/20110323/replaced-intrinsic-ratio-001-expected.png: Removed.
  • platform/qt/css2.1/20110323/replaced-intrinsic-ratio-001-expected.txt: Removed.
  • platform/qt/svg/custom/object-sizing-expected.png: Removed.
  • platform/qt/svg/custom/object-sizing-expected.txt: Removed.
  • platform/qt/svg/custom/object-sizing-explicit-width-height-expected.png: Removed.
  • platform/qt/svg/custom/object-sizing-explicit-width-height-expected.txt: Removed.
  • platform/qt/svg/custom/object-sizing-height-50p-on-target-svg-absolute-expected.png: Removed.
  • platform/qt/svg/custom/object-sizing-height-50p-on-target-svg-absolute-expected.txt: Removed.
  • platform/qt/svg/custom/object-sizing-height-50p-on-target-svg-expected.png: Removed.
  • platform/qt/svg/custom/object-sizing-height-50p-on-target-svg-expected.txt: Removed.
  • platform/qt/svg/custom/object-sizing-no-width-height-change-content-box-size-expected.png: Removed.
  • platform/qt/svg/custom/object-sizing-no-width-height-change-content-box-size-expected.txt: Removed.
  • platform/qt/svg/custom/object-sizing-no-width-height-expected.png: Removed.
  • platform/qt/svg/custom/object-sizing-no-width-height-expected.txt: Removed.
  • platform/qt/svg/custom/object-sizing-width-50p-height-50p-on-target-svg-absolute-expected.png: Removed.
  • platform/qt/svg/custom/object-sizing-width-50p-height-50p-on-target-svg-absolute-expected.txt: Removed.
  • platform/qt/svg/custom/object-sizing-width-50p-height-50p-on-target-svg-expected.png: Removed.
  • platform/qt/svg/custom/object-sizing-width-50p-height-50p-on-target-svg-expected.txt: Removed.
  • platform/qt/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-absolute-expected.png: Removed.
  • platform/qt/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-absolute-expected.txt: Removed.
  • platform/qt/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-expected.png: Removed.
  • platform/qt/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-expected.txt: Removed.
  • platform/qt/svg/custom/object-sizing-width-50p-on-target-svg-absolute-expected.png: Removed.
  • platform/qt/svg/custom/object-sizing-width-50p-on-target-svg-absolute-expected.txt: Removed.
  • platform/qt/svg/custom/object-sizing-width-50p-on-target-svg-expected.png: Removed.
  • platform/qt/svg/custom/object-sizing-width-50p-on-target-svg-expected.txt: Removed.
  • platform/qt/svg/custom/object-sizing-width-75p-height-50p-on-target-svg-absolute-expected.png: Removed.
  • platform/qt/svg/custom/object-sizing-width-75p-height-50p-on-target-svg-absolute-expected.txt: Removed.
  • platform/qt/svg/custom/object-sizing-width-75p-height-50p-on-target-svg-expected.png: Removed.
  • platform/qt/svg/custom/object-sizing-width-75p-height-50p-on-target-svg-expected.txt: Removed.
  • platform/qt/svg/hixie/intrinsic/001-expected.png: Removed.
  • platform/qt/svg/hixie/intrinsic/001-expected.txt: Removed.
  • platform/qt/svg/hixie/intrinsic/002-expected.png: Removed.
  • platform/qt/svg/hixie/intrinsic/002-expected.txt: Removed.
  • platform/qt/svg/hixie/intrinsic/003-expected.png: Removed.
  • platform/qt/svg/hixie/intrinsic/003-expected.txt: Removed.
  • platform/qt/svg/wicd/rightsizing-grid-expected.png: Removed.
  • platform/qt/svg/wicd/rightsizing-grid-expected.txt: Removed.
  • platform/qt/svg/wicd/test-rightsizing-a-expected.png: Removed.
  • platform/qt/svg/wicd/test-rightsizing-a-expected.txt: Removed.
  • platform/qt/svg/wicd/test-rightsizing-b-expected.png: Removed.
  • platform/qt/svg/wicd/test-rightsizing-b-expected.txt: Removed.
  • platform/qt/svg/zoom/page/zoom-replaced-intrinsic-ratio-001-expected.png: Removed.
  • platform/qt/svg/zoom/page/zoom-replaced-intrinsic-ratio-001-expected.txt: Removed.
  • platform/win/svg/custom/object-sizing-explicit-width-height-expected.txt: Removed.
  • platform/win/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-absolute-expected.txt: Removed.
  • platform/win/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-expected.txt: Removed.
  • platform/win/svg/custom/object-sizing-width-75p-height-50p-on-target-svg-absolute-expected.txt: Removed.
  • platform/win/svg/custom/object-sizing-width-75p-height-50p-on-target-svg-expected.txt: Removed.
  • platform/win/svg/hixie/intrinsic/003-expected.png: Removed.
  • platform/win/svg/hixie/intrinsic/003-expected.txt: Removed.
  • svg/custom/object-no-size-attributes.xhtml: Removed.
  • svg/custom/object-sizing-explicit-height.xhtml: Removed.
  • svg/custom/object-sizing-explicit-width-height.xhtml: Removed.
  • svg/custom/object-sizing-explicit-width.xhtml: Removed.
  • svg/custom/object-sizing-height-50p-on-target-svg-absolute.xhtml: Removed.
  • svg/custom/object-sizing-height-50p-on-target-svg.xhtml: Removed.
  • svg/custom/object-sizing-no-width-height-change-content-box-size.xhtml: Removed.
  • svg/custom/object-sizing-no-width-height.xhtml: Removed.
  • svg/custom/object-sizing-width-50p-height-50p-on-target-svg-absolute.xhtml: Removed.
  • svg/custom/object-sizing-width-50p-height-50p-on-target-svg.xhtml: Removed.
  • svg/custom/object-sizing-width-50p-height-75p-on-target-svg-absolute.xhtml: Removed.
  • svg/custom/object-sizing-width-50p-height-75p-on-target-svg.xhtml: Removed.
  • svg/custom/object-sizing-width-50p-on-target-svg-absolute.xhtml: Removed.
  • svg/custom/object-sizing-width-50p-on-target-svg.xhtml: Removed.
  • svg/custom/object-sizing-width-75p-height-50p-on-target-svg-absolute.xhtml: Removed.
  • svg/custom/object-sizing-width-75p-height-50p-on-target-svg.xhtml: Removed.
  • svg/custom/object-sizing.xhtml: Removed.
  • svg/custom/resources/embedded.svg: Removed.
  • svg/custom/resources/red-circle.svg: Removed.
  • svg/hixie/intrinsic/001.html: Removed.
  • svg/hixie/intrinsic/002.html: Removed.
  • svg/hixie/intrinsic/003.html: Removed.
  • svg/hixie/intrinsic/resources: Removed.
  • svg/hixie/intrinsic/resources/001.svg: Removed.
  • svg/hixie/intrinsic/resources/002.svg: Removed.
  • svg/hixie/intrinsic/resources/003.svg: Removed.
  • svg/hixie/intrinsic/resources/test-bl: Removed.
  • svg/hixie/intrinsic/resources/test-br: Removed.
  • svg/hixie/intrinsic/resources/test-tl: Removed.
  • svg/hixie/intrinsic/resources/test-tr: Removed.
  • svg/in-html/by-reference-expected.png: Removed.
  • svg/wicd/rightsizing-grid.xhtml: Removed.
  • svg/wicd/test-rightsizing-a.xhtml: Removed.
  • svg/wicd/test-rightsizing-b.xhtml: Removed.
  • svg/zoom/page/resources/intrinsic-ratio.svg: Removed.
  • svg/zoom/page/zoom-replaced-intrinsic-ratio-001.htm: Removed.
  • svg/zoom/page/zoom-svg-through-object-with-absolute-size-2.xhtml-disabled: Removed.
  • svg/zoom/page/zoom-svg-through-object-with-absolute-size.xhtml-disabled: Removed.
  • svg/zoom/page/zoom-svg-through-object-with-huge-size.xhtml-disabled: Removed.
  • svg/zoom/page/zoom-svg-through-object-with-override-size.html-disabled: Removed.
  • svg/zoom/page/zoom-svg-through-object-with-percentage-size.xhtml-disabled: Removed.
  • svg/zoom/page/zoom-svg-through-object-with-text.xhtml-disabled: Removed.
10:41 AM Changeset in webkit [92292] by andersca@apple.com
  • 2 edits in trunk/Source/WebKit2

Fix API tests.

Initialize threading before trying to allocate the WebContext.

  • UIProcess/WebContext.cpp:

(WebKit::WebContext::sharedProcessContext):
(WebKit::WebContext::create):

10:22 AM Changeset in webkit [92291] by Adam Roben
  • 2 edits in trunk/Source/WebKit/win

Windows nightly build fix after r91777

  • Interfaces/IWebPreferences.idl: Moved [set]PictographFontFamily to the end of the

interface to maintain binary compatibility with Safari 5.1.

10:06 AM Changeset in webkit [92290] by andersca@apple.com
  • 1 edit
    2 adds in trunk/LayoutTests

Add a Mac Lion skipped list.

  • platform/mac-lion: Added.
  • platform/mac-lion/Skipped: Added.
9:48 AM Changeset in webkit [92289] by macpherson@chromium.org
  • 2 edits in trunk/Source/WebCore

Use appropriate macro for CSSPropertyImageRendering.
​https://bugs.webkit.org/show_bug.cgi?id=65594

Reviewed by Simon Fraser.

No new tests / trivial code cleanup only.

  • css/CSSStyleSelector.cpp:

(WebCore::CSSStyleSelector::applyProperty):
Use HANDLE_INHERIT_AND_INITIAL_AND_PRIMITIVE macro.

9:48 AM Changeset in webkit [92288] by vsevik@chromium.org
  • 1 edit
    2 copies in branches/chromium/835

Merge 92167 - Web Inspector: button + "New style rule"
​https://bugs.webkit.org/show_bug.cgi?id=65409

Reviewed by Yury Semikhatsky.

Source/WebCore:

Test: inspector/styles/styles-add-new-rule.html

  • inspector/front-end/StylesSidebarPane.js:

(WebInspector.StylesSidebarPane):
(WebInspector.BlankStylePropertiesSection.prototype.editingSelectorCommitted):

LayoutTests:

  • inspector/styles/styles-add-new-rule-expected.txt: Added.
  • inspector/styles/styles-add-new-rule.html: Added.

TBR=​pfeldman@chromium.org
BUG=91217
Review URL: ​http://codereview.chromium.org/7566008

9:46 AM Changeset in webkit [92287] by mitz@apple.com
  • 2 edits in trunk/Source/WebCore

FontCache::getFontData() does not set the familyIndex to cAllFamiliesScanned if the last font is a custom font
​https://bugs.webkit.org/show_bug.cgi?id=65597

Reviewed by Simon Fraser.

No test added because behavior is unchanged, but this saves a call to getFontData().

  • platform/graphics/FontCache.cpp:

(WebCore::FontCache::getFontData): Eliminated early return inside the loop, allowing execution
to continue and familyIndex to be set.

9:27 AM Changeset in webkit [92286] by rniwa@webkit.org
  • 9 edits
    2 adds in trunk/Source/WebCore

Add RenderedPosition
​https://bugs.webkit.org/show_bug.cgi?id=65221

Reviewed by Hajime Morita.

Add the skeleton for RenderedPosition and use it in rootBoxForLine.

  • CMakeLists.txt:
  • GNUmakefile.list.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • editing/EditingAllInOne.cpp:
  • editing/RenderedPosition.cpp: Added.

(WebCore::RenderedPosition::RenderedPosition):

  • editing/RenderedPosition.h: Added.

(WebCore::RenderedPosition::isNull):
(WebCore::RenderedPosition::rootBox):
(WebCore::RenderedPosition::RenderedPosition):

  • editing/visible_units.cpp:

(WebCore::startPositionForLine):
(WebCore::endPositionForLine):
(WebCore::logicalStartPositionForLine):
(WebCore::logicalEndPositionForLine):

9:15 AM Changeset in webkit [92285] by rolandsteiner@chromium.org
  • 2 edits in trunk/Source/WebCore

[V8] V8CSSRuleCustom.cpp: not all enum values are listed in switch statement
​https://bugs.webkit.org/show_bug.cgi?id=65598

Reviewed by Dimitri Glazkov.

Added UNKNOWN_RULE to the switch statement.

No new tests. (No semantic change)

  • bindings/v8/custom/V8CSSRuleCustom.cpp:

(WebCore::toV8):

9:11 AM Changeset in webkit [92284] by pfeldman@chromium.org
  • 26 edits in trunk/Source

Web Inspector: [Timeline] Hover on paint events and see rect outline around that area.
​https://bugs.webkit.org/show_bug.cgi?id=65510

Reviewed by Yury Semikhatsky.

Source/WebCore:

  • WebCore.exp.in:
  • WebCore.order:
  • inspector/DOMNodeHighlighter.cpp:

(WebCore::DOMNodeHighlighter::drawRectHighlight):

  • inspector/DOMNodeHighlighter.h:
  • inspector/Inspector.json:
  • inspector/InspectorController.cpp:

(WebCore::InspectorController::drawHighlight):

  • inspector/InspectorController.h:
  • inspector/InspectorDOMAgent.cpp:

(WebCore::InspectorDOMAgent::highlightRect):
(WebCore::InspectorDOMAgent::hideHighlight):
(WebCore::InspectorDOMAgent::drawHighlight):

  • inspector/InspectorDOMAgent.h:
  • inspector/front-end/ResourcesPanel.js:

(WebInspector.FrameTreeElement.prototype.onselect):
(WebInspector.FrameTreeElement.prototype.set hovered):

  • inspector/front-end/TimelinePanel.js:

(WebInspector.TimelinePanel):
(WebInspector.TimelinePanel.prototype._mouseOut):
(WebInspector.TimelinePanel.prototype._mouseMove):
(WebInspector.TimelinePanel.prototype._highlightRect):
(WebInspector.TimelinePanel.prototype._hideRectHighlight):

  • inspector/front-end/inspector.js:

(WebInspector.highlightDOMNode):

Source/WebKit/chromium:

  • src/WebDevToolsAgentImpl.cpp:

(WebKit::WebDevToolsAgentImpl::paintPageOverlay):

Source/WebKit/gtk:

  • webkit/webkitwebview.cpp:

(paintWebView):

Source/WebKit/mac:

  • WebInspector/WebNodeHighlightView.mm:

(-[WebNodeHighlightView drawRect:]):

Source/WebKit/qt:

  • Api/qwebframe.cpp:

(QWebFramePrivate::renderRelativeCoords):

Source/WebKit/win:

  • WebNodeHighlight.cpp:

(WebNodeHighlight::update):

Source/WebKit2:

  • WebProcess/WebCoreSupport/WebInspectorClient.cpp:

(WebKit::WebInspectorClient::drawRect):

8:56 AM QtWebKitBuildBots edited by Csaba Osztrogonác
Update Qt and Mobility version on 4.7.x QtWebKit2.2 bot (diff)
8:49 AM Changeset in webkit [92283] by caio.oliveira@openbossa.org
  • 2 edits in trunk/Tools

[Qt] Fix running qmltests for WK2 in the bot
​https://bugs.webkit.org/show_bug.cgi?id=65621

Reviewed by Csaba Osztrogonác.

The path for QML_IMPORT_PATH was assuming we were running the WK1 tests,
but now there's the case for WK2 UIProcess tests as well.

  • Scripts/run-qtwebkit-tests: add both possible paths to the QML_IMPORT_PATH.
8:33 AM Changeset in webkit [92282] by Dimitri Glazkov
  • 3 edits in trunk/Tools

Add a way to edit test expectations.
​https://bugs.webkit.org/show_bug.cgi?id=64922

Implements a TestExpectationsEditor, which provides a standard way to
update and remove test expectations.

Reviewed by Adam Barth.

  • Scripts/webkitpy/layout_tests/models/test_expectations.py: Changed TestExpectationSerializer to skip lines that match no test configurations,

added TestExpectationLine.is_flaky, added BugManager, a simple interface to be fleshed out later, and a TestExpectationsEditor, the big
enchilada.

  • Scripts/webkitpy/layout_tests/models/test_expectations_unittest.py: Loads of tests to capture desired behavior.
8:30 AM Changeset in webkit [92281] by senorblanco@chromium.org
  • 2 edits in trunk/LayoutTests

[chromium] Added two compositing tests whose pixel results on the
Mesa bots are different following
​http://trac.webkit.org/changeset/92255.

Unreviewed.

  • platform/chromium/test_expectations.txt:
8:09 AM Changeset in webkit [92280] by pfeldman@chromium.org
  • 3 edits in trunk/Source/WebCore

Web Inspector: make text in popover selectable + make popover re-appear faster.
​https://bugs.webkit.org/show_bug.cgi?id=65603

Reviewed by Yury Semikhatsky.

  • inspector/front-end/Popover.js:

(WebInspector.PopoverHelper.prototype.setTimeout):
(WebInspector.PopoverHelper.prototype._mouseMove):
(WebInspector.PopoverHelper.prototype._handleMouseAction):

  • inspector/front-end/popover.css:

(.popover .content):

7:40 AM Changeset in webkit [92279] by Philippe Normand
  • 1 edit
    2 adds in trunk/LayoutTests

Unreviewed, GTK baselines for media/media-controls-clone.html.

  • platform/gtk/media/media-controls-clone-expected.png: Added.
  • platform/gtk/media/media-controls-clone-expected.txt: Added.
7:33 AM Changeset in webkit [92278] by vsevik@chromium.org
  • 3 edits in trunk/Source/WebCore

Web Inspector: Console scroll position should be saved when switching panels.
​https://bugs.webkit.org/show_bug.cgi?id=65607

Reviewed by Pavel Feldman.

  • inspector/front-end/ConsoleView.js:

(WebInspector.ConsoleView.prototype._formatAsArrayEntry):
(WebInspector.ConsoleView.prototype.get scrollLeft):
(WebInspector.ConsoleView.prototype.set scrollLeft):
(WebInspector.ConsoleView.prototype.get scrollTop):
(WebInspector.ConsoleView.prototype.set scrollTop):

  • inspector/front-end/Drawer.js:

(WebInspector.Drawer.prototype.show):
(WebInspector.Drawer.prototype._endStatusBarDragging):
(WebInspector.Drawer.prototype.get scrollLeft):
(WebInspector.Drawer.prototype.set scrollLeft):
(WebInspector.Drawer.prototype.get scrollTop):
(WebInspector.Drawer.prototype.set scrollTop):

7:25 AM Changeset in webkit [92277] by caio.oliveira@openbossa.org
  • 6 edits
    13 adds in trunk

[Qt] [WK2] Expose web view classes to QML
​https://bugs.webkit.org/show_bug.cgi?id=65339

Reviewed by Benjamin Poulain.

.:

  • Source/WebKit.pro: Include QML plugin subdir in the build system for WK2.

Source/WebKit2:

This patch also uses QuickTest from QtDeclarative to provide the basic autotest
infrastructure for the exposed elements.

  • UIProcess/API/qt/qmlplugin/plugin.cpp: Added.

(WebKit2QmlPlugin::registerTypes):

  • UIProcess/API/qt/qmlplugin/qmldir: Added.
  • UIProcess/API/qt/qmlplugin/qmlplugin.pro: Added.
  • UIProcess/API/qt/qtouchwebpage.h: Expose load() to QML environment.
  • UIProcess/API/qt/qtouchwebview.h: Since QTouchWebPage* doesn't change once set

in the constructor, we mark the page property as CONSTANT. This avoid QML
warnings for not having a NOTIFY signal for the property change.

  • UIProcess/API/qt/tests/tests.pro:
  • UIProcess/API/qt/tests/qmltests/DesktopWebView/tst_properties.qml: Added.
  • UIProcess/API/qt/tests/qmltests/TouchWebView/tst_properties.qml: Added.
  • UIProcess/API/qt/tests/qmltests/common/test1.html: Added.
  • UIProcess/API/qt/tests/qmltests/qmltests.pro: Added.
  • UIProcess/API/qt/tests/qmltests/tst_qmltests.cpp: Added.
6:15 AM Changeset in webkit [92276] by andreas.kling@nokia.com
  • 3 edits in trunk/Tools

[Qt] MiniBrowser: Unbreak load progress indication.
​https://bugs.webkit.org/show_bug.cgi?id=65614

Reviewed by Kenneth Rohde Christiansen.

loadProgress() was renamed to loadProgressChanged().

  • MiniBrowser/qt/BrowserWindow.cpp:

(BrowserWindow::BrowserWindow):
(BrowserWindow::onLoadProgressChanged):

  • MiniBrowser/qt/BrowserWindow.h:
6:05 AM Changeset in webkit [92275] by tkent@chromium.org
  • 2 edits in trunk/LayoutTests

[Chromium] Remove flaky failures of fast/speech/input-appearance-*.html.

  • platform/chromium/test_expectations.txt:
5:29 AM Changeset in webkit [92274] by Martin Robinson
  • 2 edits in trunk/Tools

Build fix for WebKit2GTK+.

  • WebKitTestRunner/GNUmakefile.am: Include libInternals as one of

of the library dependencies of the injected bundle.

5:20 AM Changeset in webkit [92273] by Carlos Garcia Campos
  • 10 edits in trunk

[GTK] Install web and plugin processes in libexecdir instead of bindir
​https://bugs.webkit.org/show_bug.cgi?id=65600

Reviewed by Martin Robinson.

.:

  • GNUmakefile.am: Initialize libexec_PROGRAMS, only used by

WebKit2 in this moment.

Source/WebKit2:

Launch processes from libexecdir or WEBKIT_EXEC_PATH
environment variable when it's set. This variable will be used by
MiniBrowser and unit tests to find processes without having to
install them.

  • GNUmakefile.am:
  • UIProcess/Launcher/gtk/ProcessLauncherGtk.cpp:

(WebKit::ProcessLauncher::launchProcess):

Tools:

Set WEBKIT_EXEC_PATH environment variable to use web and plugin
process from current directory instead of the intalled ones.

  • MiniBrowser/gtk/GNUmakefile.am:
  • MiniBrowser/gtk/main.c:

(main):

  • WebKitTestRunner/GNUmakefile.am:
  • WebKitTestRunner/gtk/main.cpp:

(main):

4:55 AM Changeset in webkit [92272] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt]REGRESSION(r92254): It made 2 tests timeout
​https://bugs.webkit.org/show_bug.cgi?id=65609

[Qt]REGRESSION(r92224): It made editing/text-iterator/basic-iteration.html timeout
​https://bugs.webkit.org/show_bug.cgi?id=65605

  • platform/qt/Skipped: Skip failing tests:
    • fast/frames/lots-of-objects.html
    • fast/frames/lots-of-iframes.html
    • editing/text-iterator/basic-iteration.html
4:14 AM Changeset in webkit [92271] by Philippe Normand
  • 3 edits in trunk/Source/WebCore

libsoup-CRITICAL : soup_message_io_pause: assertion `io != NULL' failed
​https://bugs.webkit.org/show_bug.cgi?id=64263

Reviewed by Martin Robinson.

Avoid any operation on not yet started messages and completed messages.

  • platform/network/ResourceHandleInternal.h: (WebCore::ResourceHandleInternal::ResourceHandleInternal):
  • platform/network/soup/ResourceHandleSoup.cpp: (WebCore::finishedCallback): (WebCore::startHTTPRequest): (WebCore::ResourceHandle::platformSetDefersLoading):
3:50 AM Changeset in webkit [92270] by vsevik@chromium.org
  • 8 edits
    1 copy in branches/chromium/835

Merge 91858 - Web Inspector: Disable cache option should only clear memory cache, not disable it.
​https://bugs.webkit.org/show_bug.cgi?id=65184

Reviewed by Pavel Feldman.

Source/WebCore:

  • inspector/InspectorClient.h:
  • inspector/InspectorResourceAgent.cpp:

(WebCore::InspectorResourceAgent::enable):
(WebCore::InspectorResourceAgent::disable):
(WebCore::InspectorResourceAgent::setCacheDisabled):
(WebCore::InspectorResourceAgent::mainFrameNavigated):

Source/WebKit/chromium:

  • src/InspectorClientImpl.cpp:
  • src/InspectorClientImpl.h:
  • src/WebDevToolsAgentImpl.cpp:
  • src/WebDevToolsAgentImpl.h:

LayoutTests:

  • http/tests/inspector/network/network-disable-cache-memory-expected.txt:
  • http/tests/inspector/network/network-disable-cache-memory.html:
  • http/tests/inspector/network/resources/random-script.php: Added.

TBR=​pfeldman@chromium.org
BUG=90757
Review URL: ​http://codereview.chromium.org/7563005

3:48 AM Changeset in webkit [92269] by jeremy@chromium.org
  • 7 edits in trunk/Source

[Chromium] Fix OOP font loading to work on 10.6.6 and above.
​https://bugs.webkit.org/show_bug.cgi?id=65543

In 10.6.6 the function used to get the unique ID for an NSFont in the
renderer was changed so it fails in the sandbox (it now tries to access
the on-disk font file). In order to work around this, we get the font
ID from the browser process.

To speed things up, we introduce 2 levels of caching in WebKit. A font
name cache where we can perform a quick lookup without the need for the
font id and a font id cache which we can only lookup in after getting
the unique ID from the browser process.

Reviewed by Kenneth Russell.

No new tests since this is not readily testable.

Source/WebCore:

  • platform/chromium/PlatformBridge.h:
  • platform/graphics/chromium/CrossProcessFontLoading.h:
  • platform/graphics/chromium/CrossProcessFontLoading.mm:

(WebCore::MemoryActivatedFont::create):
(WebCore::MemoryActivatedFont::MemoryActivatedFont):
(WebCore::MemoryActivatedFont::~MemoryActivatedFont):

Source/WebKit/chromium:

  • public/mac/WebSandboxSupport.h: Plumb font ID parameter through.
  • src/PlatformBridge.cpp:

(WebCore::PlatformBridge::loadFont): ditto.

3:46 AM Changeset in webkit [92268] by Philippe Normand
  • 31 edits
    3 adds in trunk/LayoutTests

Unreviewed, GTK svg rebaseline after r92047.

  • platform/gtk/svg/W3C-SVG-1.1/masking-path-04-b-expected.txt:
  • platform/gtk/svg/W3C-SVG-1.1/struct-use-05-b-expected.txt:
  • platform/gtk/svg/batik/filters/feTile-expected.txt:
  • platform/gtk/svg/batik/paints/patternPreserveAspectRatioA-expected.txt:
  • platform/gtk/svg/batik/paints/patternRegionA-expected.txt:
  • platform/gtk/svg/batik/paints/patternRegions-expected.txt:
  • platform/gtk/svg/batik/paints/patternRegions-positioned-objects-expected.txt:
  • platform/gtk/svg/batik/text/longTextOnPath-expected.txt:
  • platform/gtk/svg/batik/text/smallFonts-expected.txt:
  • platform/gtk/svg/batik/text/textAnchor-expected.txt:
  • platform/gtk/svg/batik/text/textDecoration-expected.txt:
  • platform/gtk/svg/batik/text/textEffect-expected.txt:
  • platform/gtk/svg/batik/text/textEffect2-expected.txt:
  • platform/gtk/svg/batik/text/textEffect3-expected.txt:
  • platform/gtk/svg/batik/text/textFeatures-expected.txt:
  • platform/gtk/svg/batik/text/textLayout-expected.txt:
  • platform/gtk/svg/batik/text/textLayout2-expected.txt:
  • platform/gtk/svg/batik/text/textLength-expected.txt:
  • platform/gtk/svg/batik/text/textOnPath-expected.txt:
  • platform/gtk/svg/batik/text/textOnPathSpaces-expected.txt:
  • platform/gtk/svg/batik/text/textPosition-expected.txt:
  • platform/gtk/svg/batik/text/textPosition2-expected.txt:
  • platform/gtk/svg/batik/text/textProperties-expected.txt:
  • platform/gtk/svg/batik/text/textProperties2-expected.txt:
  • platform/gtk/svg/batik/text/textStyles-expected.txt:
  • platform/gtk/svg/batik/text/verticalText-expected.txt:
  • platform/gtk/svg/batik/text/verticalTextOnPath-expected.txt:
  • platform/gtk/svg/custom/external-paintserver-reference-expected.txt: Added.
  • platform/gtk/svg/custom/linking-base-external-reference-expected.txt: Added.
  • platform/gtk/svg/custom/text-linking-expected.txt:
  • platform/gtk/svg/custom/zero-path-square-cap-rendering2-expected.txt: Added.
  • platform/gtk/svg/hixie/error/014-expected.txt:
  • platform/gtk/svg/hixie/use/002-expected.txt:
3:18 AM Changeset in webkit [92267] by vsevik@chromium.org
  • 1 edit in branches/chromium/835/Source/WebCore/inspector/InspectorPageAgent.cpp

Merge 91901 - Web Inspector: [REGRESSION] [Chromium] Opening link in new tab does not work properly.
​https://bugs.webkit.org/show_bug.cgi?id=65279

Reviewed by Pavel Feldman.

  • inspector/InspectorPageAgent.cpp:

(WebCore::InspectorPageAgent::open):

TBR=​vsevik@chromium.org
BUG=87345
Review URL: ​http://codereview.chromium.org/7565004

3:17 AM Changeset in webkit [92266] by hayato@chromium.org
  • 12 edits in trunk/Source/WebCore

Make EventDispatchMediator RefCounted.
​https://bugs.webkit.org/show_bug.cgi?id=65529

Reviewed by Hajime Morita.

Make EventDispatchMediator RefCounted so that it can be enqueued into a
ScopedEventQueue and called later.
A necessary change for ScopedEventQueue should be addressed in another patch.

This is a minor update of r92256, which failed on chromium win builds.

No changes to functionality so no new tests.

  • dom/Event.cpp:

(WebCore::EventDispatchMediator::create):

  • dom/Event.h:
  • dom/EventDispatcher.cpp:

(WebCore::EventDispatcher::dispatchEvent):

  • dom/EventDispatcher.h:
  • dom/KeyboardEvent.cpp:

(WebCore::KeyboardEventDispatchMediator::create):

  • dom/KeyboardEvent.h:
  • dom/MouseEvent.cpp:

(WebCore::MouseEventDispatchMediator::create):

  • dom/MouseEvent.h:
  • dom/Node.cpp:

(WebCore::Node::dispatchEvent):
(WebCore::Node::dispatchKeyEvent):
(WebCore::Node::dispatchMouseEvent):
(WebCore::Node::dispatchWheelEvent):

  • dom/WheelEvent.cpp:

(WebCore::WheelEventDispatchMediator::create):

  • dom/WheelEvent.h:
1:28 AM Changeset in webkit [92265] by pfeldman@chromium.org
  • 38 edits in trunk/Source

Web Inspector: remove Node parameter from the InspectorClient::highlight
​https://bugs.webkit.org/show_bug.cgi?id=65549

Patch by Pavel Feldman <​pfeldman@chromium.org> on 2011-08-03
Reviewed by Yury Semikhatsky.

Source/WebCore:

  • inspector/InspectorClient.h:
  • inspector/InspectorDOMAgent.cpp:

(WebCore::InspectorDOMAgent::highlight):

  • loader/EmptyClients.h:

(WebCore::EmptyInspectorClient::highlight):

Source/WebKit/chromium:

  • src/InspectorClientImpl.cpp:

(WebKit::InspectorClientImpl::highlight):

  • src/InspectorClientImpl.h:
  • src/WebDevToolsAgentImpl.cpp:

(WebKit::WebDevToolsAgentImpl::highlight):

  • src/WebDevToolsAgentImpl.h:

Source/WebKit/efl:

  • WebCoreSupport/InspectorClientEfl.cpp:

(WebCore::InspectorClientEfl::highlight):

  • WebCoreSupport/InspectorClientEfl.h:

Source/WebKit/gtk:

  • WebCoreSupport/InspectorClientGtk.cpp:

(WebKit::InspectorClient::highlight):

  • WebCoreSupport/InspectorClientGtk.h:

Source/WebKit/haiku:

  • WebCoreSupport/InspectorClientHaiku.cpp:

(WebCore::InspectorClientHaiku::highlight):

  • WebCoreSupport/InspectorClientHaiku.h:

Source/WebKit/mac:

  • WebCoreSupport/WebInspectorClient.h:
  • WebCoreSupport/WebInspectorClient.mm:

(WebInspectorClient::highlight):

  • WebInspector/WebNodeHighlighter.h:
  • WebInspector/WebNodeHighlighter.mm:

(-[WebNodeHighlighter highlight]):

Source/WebKit/qt:

  • WebCoreSupport/InspectorClientQt.cpp:

(WebCore::InspectorClientQt::highlight):

  • WebCoreSupport/InspectorClientQt.h:

Source/WebKit/win:

  • WebCoreSupport/WebInspectorClient.cpp:

(WebInspectorClient::highlight):

  • WebCoreSupport/WebInspectorClient.h:

Source/WebKit/wince:

  • WebCoreSupport/InspectorClientWinCE.cpp:

(WebKit::InspectorClientWinCE::highlight):

  • WebCoreSupport/InspectorClientWinCE.h:

Source/WebKit/wx:

  • WebKitSupport/InspectorClientWx.cpp:

(WebCore::InspectorClientWx::highlight):

  • WebKitSupport/InspectorClientWx.h:

Source/WebKit2:

  • WebProcess/WebCoreSupport/WebInspectorClient.cpp:

(WebKit::WebInspectorClient::highlight):

  • WebProcess/WebCoreSupport/WebInspectorClient.h:
1:20 AM Changeset in webkit [92264] by Antti Koivisto
  • 2 edits in trunk/Source/WebCore

​https://bugs.webkit.org/show_bug.cgi?id=65599
Image loads should trigger didFirstVisuallyNonEmptyLayout more easily

Reviewed by Dan Bernstein.

Tweak the heuristic pixel count so logo-sized images trigger didFirstVisuallyNonEmptyLayout.

  • page/FrameView.h:

(WebCore::FrameView::incrementVisuallyNonEmptyPixelCount):

12:43 AM Changeset in webkit [92263] by Carlos Garcia Campos
  • 9 edits
    2 moves in trunk

[GTK] Reorganize pkg-config files
​https://bugs.webkit.org/show_bug.cgi?id=65548

Reviewed by Martin Robinson.

.:

  • GNUmakefile.am: Initialize pkgconfigdir and pkgconfig_DATA which

are common to all libraries.

  • configure.ac: Update pkg-config files.

Source/JavaScriptCore:

  • GNUmakefile.am:
  • javascriptcoregtk.pc.in: Renamed from Source/WebKit/gtk/javascriptcoregtk.pc.in.

Source/WebKit/gtk:

  • GNUmakefile.am: Remove javascriptcore pc file references.

Source/WebKit2:

  • GNUmakefile.am: Rename pc file as webkit2gtk.pc.in. WebKit2

depends on gtk3 unconditionally so we don't need to use
WEBKITGTK_PC_NAME.

  • webkit2gtk.pc.in: Renamed from Source/WebKit2/gtk/webkit2.pc.in.
12:06 AM Changeset in webkit [92262] by commit-queue@webkit.org
  • 3 edits
    3 adds in trunk

WebSocket: Could not handle zero length text frame.
​https://bugs.webkit.org/show_bug.cgi?id=65592

Patch by Takashi Toyoshima <​toyoshim@chromium.org> on 2011-08-03
Reviewed by Kent Tamura.

Source/WebCore:

Test: http/tests/websocket/tests/hybi/zero-length-text.html

  • websockets/WebSocketChannel.cpp:

(WebCore::WebSocketChannel::processFrame):

LayoutTests:

  • http/tests/websocket/tests/hybi/zero-length-text-expected.txt: Added.
  • http/tests/websocket/tests/hybi/zero-length-text.html: Added.
  • http/tests/websocket/tests/hybi/zero-length-text_wsh.py: Added.

Aug 2, 2011:

10:07 PM Changeset in webkit [92261] by psolanki@apple.com
  • 2 edits in trunk/Source/WebCore

Crash in ResourceResponse::platformCompare() with USE(CFNETWORK)
​https://bugs.webkit.org/show_bug.cgi?id=65584

Reviewed by David Kilzer.

  • platform/network/cf/ResourceResponseCFNet.cpp:

(WebCore::ResourceResponse::platformCompare): Don't call CFEqual with 0 as an argument.

9:37 PM Changeset in webkit [92260] by commit-queue@webkit.org
  • 3 edits
    3 adds in trunk

WebSocket: Should be closed by receiving invalid continuation frame.

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

Patch by Takashi Toyoshima <​toyoshim@chromium.org> on 2011-08-02
Reviewed by Kent Tamura.

Source/WebCore:

Test: http/tests/websocket/tests/hybi/invalid-continuation.html

  • websockets/WebSocketChannel.cpp:

(WebCore::WebSocketChannel::processFrame):

LayoutTests:

  • http/tests/websocket/tests/hybi/invalid-continuation-expected.txt: Added.
  • http/tests/websocket/tests/hybi/invalid-continuation.html: Added.
  • http/tests/websocket/tests/hybi/invalid-continuation_wsh.py: Added.
9:26 PM Changeset in webkit [92259] by commit-queue@webkit.org
  • 12 edits in trunk/Source/WebCore

Unreviewed, rolling out r92256.
​http://trac.webkit.org/changeset/92256
​https://bugs.webkit.org/show_bug.cgi?id=65593

Causing tons of crashes on the chromium win bots (Requested by
jamesr on #webkit).

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

  • dom/Event.cpp:
  • dom/Event.h:
  • dom/EventDispatcher.cpp:

(WebCore::EventDispatcher::dispatchEvent):

  • dom/EventDispatcher.h:
  • dom/KeyboardEvent.cpp:
  • dom/KeyboardEvent.h:
  • dom/MouseEvent.cpp:
  • dom/MouseEvent.h:
  • dom/Node.cpp:

(WebCore::Node::dispatchEvent):
(WebCore::Node::dispatchKeyEvent):
(WebCore::Node::dispatchMouseEvent):
(WebCore::Node::dispatchWheelEvent):

  • dom/WheelEvent.cpp:
  • dom/WheelEvent.h:
8:59 PM Changeset in webkit [92258] by jamesr@google.com
  • 2 edits in trunk/LayoutTests

More expectation updates for r92255

  • platform/chromium/test_expectations.txt:
8:26 PM Changeset in webkit [92257] by jamesr@google.com
  • 2 edits in trunk/LayoutTests

Expectation follow-up for r92255

  • platform/chromium/test_expectations.txt:
8:25 PM Changeset in webkit [92256] by hayato@chromium.org
  • 12 edits in trunk/Source/WebCore

Make EventDispatchMediator RefCounted.
​https://bugs.webkit.org/show_bug.cgi?id=65529

Reviewed by Dimitri Glazkov.

Make EventDispatchMediator RefCounted so that it can be enqueued into a
ScopedEventQueue and called later.
A necessary change for ScopedEventQueue should be addressed in another patch.

No changes to functionality so no new tests.

  • dom/Event.cpp:

(WebCore::EventDispatchMediator::create):

  • dom/Event.h:
  • dom/EventDispatcher.cpp:

(WebCore::EventDispatcher::dispatchEvent):

  • dom/EventDispatcher.h:
  • dom/KeyboardEvent.cpp:

(WebCore::KeyboardEventDispatchMediator::create):

  • dom/KeyboardEvent.h:
  • dom/MouseEvent.cpp:

(WebCore::MouseEventDispatchMediator::create):

  • dom/MouseEvent.h:
  • dom/Node.cpp:

(WebCore::Node::dispatchEvent):
(WebCore::Node::dispatchKeyEvent):
(WebCore::Node::dispatchMouseEvent):
(WebCore::Node::dispatchWheelEvent):

  • dom/WheelEvent.cpp:

(WebCore::WheelEventDispatchMediator::create):

  • dom/WheelEvent.h:
7:52 PM Changeset in webkit [92255] by jamesr@google.com
  • 15 edits in trunk

Source/WebCore: [Chromium] Use edge-distance method for layer anti-aliasing.
​https://bugs.webkit.org/show_bug.cgi?id=64942

Patch by David Reveman <​reveman@chromium.org> on 2011-08-02
Reviewed by James Robinson.

Use an edge distance based method instead of the current linear
filtering method for producing anti-aliased edges on 3D transformed
layers. This removes the outer border handling from TilingData as
it's no longer needed for anti-aliasing.

Tests: platform/chromium/compositing/tiny-layer-rotated.html (existing)

platform/chromium/compositing/huge-layer-rotated.html (existing)
compositing/images/content-image-change.html (existing)
TilingDataTest in webkit_unit_tests

  • platform/graphics/chromium/LayerRendererChromium.cpp:

(WebCore::screenMatrix):
(WebCore::LayerRendererChromium::drawLayers):
(WebCore::LayerRendererChromium::setDrawViewportRect):
(WebCore::LayerRendererChromium::tilerProgramAA):
(WebCore::LayerRendererChromium::tilerProgramSwizzleAA):

  • platform/graphics/chromium/LayerRendererChromium.h:

(WebCore::LayerRendererChromium::windowMatrix):

  • platform/graphics/chromium/LayerTilerChromium.cpp:

(WebCore::LayerTilerChromium::setTileSize):
(WebCore::LayerTilerChromium::tileLayerRect):
(WebCore::LayerTilerChromium::invalidateRect):
(WebCore::LayerTilerChromium::prepareToUpdate):
(WebCore::LayerTilerChromium::updateRect):
(WebCore::LayerTilerChromium::draw):
(WebCore::LayerTilerChromium::growLayerToContain):
(WebCore::LayerTilerChromium::drawTiles):

  • platform/graphics/chromium/LayerTilerChromium.h:
  • platform/graphics/chromium/ShaderChromium.cpp:

(WebCore::VertexShaderTile::VertexShaderTile):
(WebCore::VertexShaderTile::init):
(WebCore::VertexShaderTile::getShaderString):
(WebCore::FragmentTexAlphaAABinding::FragmentTexAlphaAABinding):
(WebCore::FragmentTexAlphaAABinding::init):
(WebCore::FragmentShaderRGBATexAlphaAA::getShaderString):
(WebCore::FragmentShaderRGBATexSwizzleAlphaAA::getShaderString):

  • platform/graphics/chromium/ShaderChromium.h:

(WebCore::VertexShaderTile::vertexTexTransformLocation):
(WebCore::FragmentTexAlphaAABinding::alphaLocation):
(WebCore::FragmentTexAlphaAABinding::samplerLocation):
(WebCore::FragmentTexAlphaAABinding::fragmentTexTransformLocation):
(WebCore::FragmentTexAlphaAABinding::edgeLocation):

  • platform/graphics/chromium/TiledLayerChromium.cpp:

(WebCore::TiledLayerChromium::TiledLayerChromium):
(WebCore::TiledLayerChromium::updateTileSizeAndTilingOption):
(WebCore::TiledLayerChromium::createTilerIfNeeded):
(WebCore::TiledLayerChromium::setIsMask):

  • platform/graphics/chromium/TiledLayerChromium.h:
  • platform/graphics/gpu/TilingData.cpp:

(WebCore::computeNumTiles):
(WebCore::TilingData::tileXIndexFromSrcCoord):
(WebCore::TilingData::tileYIndexFromSrcCoord):
(WebCore::TilingData::tileSizeX):
(WebCore::TilingData::tileSizeY):
(WebCore::TilingData::intersectDrawQuad):
(WebCore::TilingData::textureOffset):

  • platform/graphics/gpu/TilingData.h:

Source/WebKit/chromium: [Chromium] Remove outer border from TilingData as it's no longer needed by layer compositor for anti-aliasing.
​https://bugs.webkit.org/show_bug.cgi?id=64942

Patch by David Reveman <​reveman@chromium.org> on 2011-08-02
Reviewed by James Robinson.

  • tests/TilingDataTest.cpp:

(WebCore::TEST):

LayoutTests: [Chromium] Use edge-distance method for layer anti-aliasing.
​https://bugs.webkit.org/show_bug.cgi?id=64942

Patch by David Reveman <​reveman@chromium.org> on 2011-08-02
Reviewed by James Robinson.

Update expected results.

  • platform/chromium/test_expectations.txt:
7:28 PM Changeset in webkit [92254] by levin@chromium.org
  • 14 edits
    3 adds in trunk/Source

Add asserts to RefCounted to make sure ref/deref happens on the right thread.
​https://bugs.webkit.org/show_bug.cgi?id=31639

Reviewed by Dmitry Titov.

Source/JavaScriptCore:

(JSC::ExecutablePool::ExecutablePool): Turned off checks for this
due to not being able to figure out what was guarding it (bug 58091).

  • parser/SourceProvider.h:

(JSC::SourceProvider::SourceProvider): Ditto.

  • wtf/CMakeLists.txt: Added new files to the build.
  • wtf/ThreadRestrictionVerifier.h: Added.

Everything is done in the header to avoid the issue with exports
that are only useful in debug but still needing to export them.

  • wtf/RefCounted.h:

(WTF::RefCountedBase::ref): Added checks using the non thread safe verifier.
and filed bug 58171 about making it stricter.
(WTF::RefCountedBase::hasOneRef): Ditto.
(WTF::RefCountedBase::refCount): Ditto.
(WTF::RefCountedBase::setMutexForVerifier): Expose a way to change the checks to be based
on a mutex. This is in the header to avoid adding more exports from JavaScriptCore.
(WTF::RefCountedBase::deprecatedTurnOffVerifier): Temporary way to turn off verification.
Filed bug 58174 to remove this method.
(WTF::RefCountedBase::derefBase):

  • wtf/SizeLimits.cpp: Adjusted the debug size check for RefCounted.
  • wtf/text/CString.h:

(WTF::CStringBuffer::CStringBuffer): Turned off checks for this while a fix is being
done in Chromium (bug 58093).

Source/JavaScriptGlue:

  • ForwardingHeaders/wtf/ThreadRestrictionVerifier.h: Added.

Source/WebCore:

No new functionality exposed so no new tests. (The change is basically adding
more testing.)

  • ForwardingHeaders/wtf/ThreadRestrictionVerifier.h: Added.
  • loader/icon/IconDatabase.cpp:

(WebCore::IconDatabase::defaultIcon): Set the mutex which does the guarding of the variable.
(WebCore::IconDatabase::setIconDataForIconURL): Ditto.
(WebCore::IconDatabase::getOrCreateIconRecord): Ditto.
(WebCore::IconDatabase::readFromDatabase): Ditto.

7:11 PM Changeset in webkit [92253] by macpherson@chromium.org
  • 3 edits in trunk/Source/WebCore

Clean up value clampling in CSSStyleSelector and CSSPrimitiveValue.
​https://bugs.webkit.org/show_bug.cgi?id=65441

Reviewed by Simon Fraser.

No new tests / refactoring only.

  • css/CSSPrimitiveValue.h:

(WebCore::CSSPrimitiveValue::getFloatValue):
Implement in terms of the new templated getValue().
(WebCore::CSSPrimitiveValue::getIntValue):
Implement in terms of the new templated getValue().
(WebCore::CSSPrimitiveValue::getValue):
Templated getValue that works for all numeric types.

  • css/CSSStyleSelector.cpp:

(WebCore::CSSStyleSelector::applyProperty):
Use getValue<short> instead of rolling-your-own clamp to short.

6:58 PM Changeset in webkit [92252] by jchaffraix@webkit.org
  • 35 edits in trunk/Source/WebCore

RenderObject::computeRectForRepaint and clippedOverflowRectForRepaint should be const
​https://bugs.webkit.org/show_bug.cgi?id=65494

Reviewed by Darin Adler.

No new tests, const-ness change only.

  • rendering/RenderObject.h:

(WebCore::RenderObject::absoluteClippedOverflowRect):
(WebCore::RenderObject::computeAbsoluteRepaintRect):
Made those 2 const as they call a const method now.

  • rendering/RenderInline.cpp:

(WebCore::RenderInline::clippedOverflowRectForRepaint): Use a const RenderObject* for iterating as
this is const.

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::rectWithOutlineForRepaint):

  • rendering/RenderBlock.h:
  • rendering/RenderBox.cpp:

(WebCore::RenderBox::clippedOverflowRectForRepaint):
(WebCore::RenderBox::computeRectForRepaint):

  • rendering/RenderBox.h:
  • rendering/RenderInline.cpp:

(WebCore::RenderInline::clippedOverflowRectForRepaint):
(WebCore::RenderInline::rectWithOutlineForRepaint):
(WebCore::RenderInline::computeRectForRepaint):

  • rendering/RenderInline.h:
  • rendering/RenderObject.cpp:

(WebCore::RenderObject::rectWithOutlineForRepaint):
(WebCore::RenderObject::clippedOverflowRectForRepaint):
(WebCore::RenderObject::computeRectForRepaint):

  • rendering/RenderReplaced.cpp:

(WebCore::RenderReplaced::clippedOverflowRectForRepaint):

  • rendering/RenderReplaced.h:
  • rendering/RenderTableCell.cpp:

(WebCore::RenderTableCell::clippedOverflowRectForRepaint):
(WebCore::RenderTableCell::computeRectForRepaint):

  • rendering/RenderTableCell.h:
  • rendering/RenderTableCol.cpp:

(WebCore::RenderTableCol::clippedOverflowRectForRepaint):

  • rendering/RenderTableCol.h:
  • rendering/RenderTableRow.cpp:

(WebCore::RenderTableRow::clippedOverflowRectForRepaint):

  • rendering/RenderTableRow.h:
  • rendering/RenderText.cpp:

(WebCore::RenderText::clippedOverflowRectForRepaint):

  • rendering/RenderText.h:
  • rendering/RenderView.cpp:

(WebCore::RenderView::computeRectForRepaint):

  • rendering/RenderView.h:
  • rendering/svg/RenderSVGForeignObject.cpp:

(WebCore::RenderSVGForeignObject::clippedOverflowRectForRepaint):
(WebCore::RenderSVGForeignObject::computeRectForRepaint):

  • rendering/svg/RenderSVGForeignObject.h:
  • rendering/svg/RenderSVGGradientStop.h:

(WebCore::RenderSVGGradientStop::clippedOverflowRectForRepaint):

  • rendering/svg/RenderSVGHiddenContainer.h:

(WebCore::RenderSVGHiddenContainer::clippedOverflowRectForRepaint):

  • rendering/svg/RenderSVGInline.cpp:

(WebCore::RenderSVGInline::clippedOverflowRectForRepaint):
(WebCore::RenderSVGInline::computeRectForRepaint):

  • rendering/svg/RenderSVGInline.h:
  • rendering/svg/RenderSVGModelObject.cpp:

(WebCore::RenderSVGModelObject::clippedOverflowRectForRepaint):
(WebCore::RenderSVGModelObject::computeRectForRepaint):

  • rendering/svg/RenderSVGModelObject.h:
  • rendering/svg/RenderSVGRoot.cpp:

(WebCore::RenderSVGRoot::clippedOverflowRectForRepaint):
(WebCore::RenderSVGRoot::computeRectForRepaint):

  • rendering/svg/RenderSVGRoot.h:
  • rendering/svg/RenderSVGText.cpp:

(WebCore::RenderSVGText::clippedOverflowRectForRepaint):
(WebCore::RenderSVGText::computeRectForRepaint):

  • rendering/svg/RenderSVGText.h:
  • rendering/svg/SVGRenderSupport.cpp:

(WebCore::SVGRenderSupport::clippedOverflowRectForRepaint):
(WebCore::SVGRenderSupport::computeRectForRepaint):

  • rendering/svg/SVGRenderSupport.h:

Updated the following method's signature to use a const object.

6:48 PM Changeset in webkit [92251] by fpizlo@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

JSC GC may not be able to reuse partially-free blocks after a
full collection
​https://bugs.webkit.org/show_bug.cgi?id=65585

Reviewed by Darin Adler.

This fixes the linked list management bug. This fix is performance
neutral on SunSpider.

  • heap/NewSpace.cpp:

(JSC::NewSpace::removeBlock):

5:41 PM Changeset in webkit [92250] by oliver@apple.com
  • 6 edits in trunk/Source/JavaScriptCore

Simplify JSFunction creation for functions written in JS
​https://bugs.webkit.org/show_bug.cgi?id=65422

Reviewed by Gavin Barraclough.

Remove hash lookups used to write name property and transition
function structure by caching the resultant structure and property
offset in JSGlobalObject. This doesn't impact performance, but
we can use this change to make other improvements later.

  • runtime/Executable.cpp:

(JSC::FunctionExecutable::FunctionExecutable):

  • runtime/Executable.h:

(JSC::ScriptExecutable::ScriptExecutable):
(JSC::FunctionExecutable::jsName):

  • runtime/JSFunction.cpp:

(JSC::JSFunction::JSFunction):

  • runtime/JSGlobalObject.cpp:

(JSC::JSGlobalObject::reset):

  • runtime/JSGlobalObject.h:

(JSC::JSGlobalObject::namedFunctionStructure):
(JSC::JSGlobalObject::functionNameOffset):

5:04 PM Changeset in webkit [92249] by jpfau@apple.com
  • 4 edits in trunk/Source/WebCore

Add namespace prefix support in XML tokenizer
​https://bugs.webkit.org/show_bug.cgi?id=65578

Reviewed by Adam Barth.

This also changes markup debug printing to print to stderr instead of stdout

  • xml/parser/MarkupTokenBase.h:

(WebCore::MarkupTokenBase::printString):

  • xml/parser/XMLToken.h:

(WebCore::XMLToken::XMLDeclarationData::XMLDeclarationData):
(WebCore::XMLToken::clear):
(WebCore::XMLToken::appendToName):
(WebCore::XMLToken::name):
(WebCore::XMLToken::target):
(WebCore::XMLToken::data):
(WebCore::XMLToken::prefix):
(WebCore::XMLToken::xmlDeclarationData):
(WebCore::XMLToken::endPrefix):
(WebCore::XMLToken::hasPrefix):
(WebCore::XMLToken::endAttributePrefix):
(WebCore::XMLToken::attributeHasPrefix):
(WebCore::XMLToken::printAttrs):
(WebCore::XMLToken::print):

  • xml/parser/XMLTokenizer.cpp:

(WebCore::XMLTokenizer::nextToken):

5:02 PM Changeset in webkit [92248] by zmo@google.com
  • 5 edits in trunk/Source/WebCore

Unreviewed, rolling out r92238.
​http://trac.webkit.org/changeset/92238
​https://bugs.webkit.org/show_bug.cgi?id=65555

fail fast/events/scrollbar-double-click.html on Mac

  • platform/chromium/ScrollbarOverlayUtilitiesChromiumMac.h:
  • platform/chromium/ScrollbarOverlayUtilitiesChromiumMac.mm:
  • platform/chromium/ScrollbarThemeChromiumMac.h:
  • platform/chromium/ScrollbarThemeChromiumMac.mm:

(WebCore::ScrollbarThemeChromiumMac::registerScrollbar):
(WebCore::ScrollbarThemeChromiumMac::setNewPainterForScrollbar):
(WebCore::toScrollbarPainterKnobStyle):
(WebCore::ScrollbarThemeChromiumMac::paint):

4:47 PM Changeset in webkit [92247] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Remove LegacyDefaultOptionalArguments flag from History.idl
​https://bugs.webkit.org/show_bug.cgi?id=65572

Patch by Mark Pilgrim <​pilgrim@chromium.org> on 2011-08-02
Reviewed by Adam Barth.

No new tests, all existing tests pass.

  • page/History.idl:
4:41 PM Changeset in webkit [92246] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Remove LegacyDefaultOptionalArguments flag from WebKitAnimationList.idl
​https://bugs.webkit.org/show_bug.cgi?id=65571

Patch by Mark Pilgrim <​pilgrim@chromium.org> on 2011-08-02
Reviewed by Adam Barth.

No new tests, all existing tests pass.

  • page/WebKitAnimationList.idl:
4:20 PM Changeset in webkit [92245] by commit-queue@webkit.org
  • 25 edits
    2 copies in trunk/Source

[chromium] Decouple layer visibility calculations from render surfaces / computed draw transforms
​https://bugs.webkit.org/show_bug.cgi?id=58840

Patch by James Robinson <​jamesr@chromium.org> on 2011-08-02
Reviewed by Kenneth Russell.

This decouples the layer visibility related calculations from the LayerChromium and CCLayerImpl trees. This is
done by making LayerRendererChromium's "calculate the world" function (renamed
calculateDrawTransformsAndVisibility()) a templated, free function that can operate on either LayerChromium +
RenderSurfaceChromium pairs or on CCLayerImpl + CCRenderSurface pairs and invoking it twice, once before
painting and once before drawing. Before painting, these calculations are used to determine which layers are
potentially visible and should be painted, which depends on the calculated opacity, render surface bounds, and
draw transforms. Before drawing, these calculations are used to figure out everything needed to draw.

RenderSurfaceChromium is now used just for visibility calculations when painting. CCRenderSurfaceChromium is
responsible for actually drawing the render surfaces.

This does mean that in the current implementation the second calculations are redundant, but in the future this
won't be the case since the pre-draw time calculations might be influenced by animations or gestures. I've
added some traces to try to monitor if this math is a significant amount of the per-frame computation. Note
that layer sorting only happens on the CCLayerImpl tree and the layer sort algorithm is the most expensive part
of the calculations currently.

This removes all of the weak back/forward pointers between LayerChromium and CCLayerImpls, the trees are now
truly decoupled.

Covered by compositing/ tests.

  • WebCore.gypi:
  • platform/graphics/chromium/CanvasLayerChromium.cpp:

(WebCore::CanvasLayerChromium::createCCLayerImpl):

  • platform/graphics/chromium/LayerChromium.cpp:

(WebCore::LayerChromium::LayerChromium):
(WebCore::LayerChromium::~LayerChromium):
(WebCore::LayerChromium::createCCLayerImpl):
(WebCore::LayerChromium::createRenderSurface):
(WebCore::LayerChromium::descendantsDrawsContent):

  • platform/graphics/chromium/LayerChromium.h:

(WebCore::LayerChromium::setUsesLayerScissor):
(WebCore::LayerChromium::usesLayerScissor):
(WebCore::LayerChromium::clearRenderSurface):
(WebCore::LayerChromium::renderSurface):
(WebCore::LayerChromium::drawOpacity):
(WebCore::LayerChromium::setDrawOpacity):
(WebCore::LayerChromium::scissorRect):
(WebCore::LayerChromium::setScissorRect):
(WebCore::LayerChromium::targetRenderSurface):
(WebCore::LayerChromium::setTargetRenderSurface):
(WebCore::LayerChromium::drawTransform):
(WebCore::LayerChromium::setDrawTransform):
(WebCore::LayerChromium::drawableContentRect):
(WebCore::LayerChromium::setDrawableContentRect):

  • platform/graphics/chromium/LayerRendererChromium.cpp:

(WebCore::LayerRendererChromium::updateLayers):
(WebCore::LayerRendererChromium::drawLayers):
(WebCore::LayerRendererChromium::paintLayerContents):
(WebCore::LayerRendererChromium::drawLayersInternal):
(WebCore::LayerRendererChromium::updateCompositorResources):
(WebCore::LayerRendererChromium::setCompositeOffscreen):
(WebCore::LayerRendererChromium::getOffscreenLayerTexture):
(WebCore::LayerRendererChromium::useRenderSurface):
(WebCore::LayerRendererChromium::drawLayer):
(WebCore::LayerRendererChromium::initializeSharedObjects):
(WebCore::LayerRendererChromium::renderSurfaceProgram):
(WebCore::LayerRendererChromium::renderSurfaceMaskProgram):
(WebCore::LayerRendererChromium::dumpRenderSurfaces):

  • platform/graphics/chromium/LayerRendererChromium.h:
  • platform/graphics/chromium/PluginLayerChromium.cpp:

(WebCore::PluginLayerChromium::createCCLayerImpl):

  • platform/graphics/chromium/RenderSurfaceChromium.cpp:

(WebCore::RenderSurfaceChromium::RenderSurfaceChromium):
(WebCore::RenderSurfaceChromium::~RenderSurfaceChromium):

  • platform/graphics/chromium/RenderSurfaceChromium.h:

(WebCore::RenderSurfaceChromium::clearLayerList):
(WebCore::RenderSurfaceChromium::layerList):
(WebCore::RenderSurfaceChromium::setMaskLayer):

  • platform/graphics/chromium/TiledLayerChromium.cpp:

(WebCore::TiledLayerChromium::createCCLayerImpl):
(WebCore::TiledLayerChromium::tilingTransform):

  • platform/graphics/chromium/TreeSynchronizer.cpp:

(WebCore::TreeSynchronizer::synchronizeTreeRecursive):

  • platform/graphics/chromium/VideoLayerChromium.cpp:

(WebCore::VideoLayerChromium::createCCLayerImpl):

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

(WebCore::CCCanvasLayerImpl::CCCanvasLayerImpl):

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

(WebCore::CCCanvasLayerImpl::create):

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

(WebCore::CCLayerImpl::CCLayerImpl):
(WebCore::CCLayerImpl::~CCLayerImpl):
(WebCore::CCLayerImpl::createRenderSurface):
(WebCore::CCLayerImpl::bindContentsTexture):

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

(WebCore::CCLayerImpl::create):
(WebCore::CCLayerImpl::renderSurface):
(WebCore::CCLayerImpl::targetRenderSurface):
(WebCore::CCLayerImpl::setTargetRenderSurface):

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

(WebCore::CCLayerSorter::createGraphNodes):

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

(WebCore::CCPluginLayerImpl::CCPluginLayerImpl):

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

(WebCore::CCPluginLayerImpl::create):

  • platform/graphics/chromium/cc/CCRenderSurface.cpp: Copied from Source/WebCore/platform/graphics/chromium/RenderSurfaceChromium.cpp.

(WebCore::CCRenderSurface::CCRenderSurface):
(WebCore::CCRenderSurface::~CCRenderSurface):
(WebCore::CCRenderSurface::cleanupResources):
(WebCore::CCRenderSurface::layerRenderer):
(WebCore::CCRenderSurface::drawableContentRect):
(WebCore::CCRenderSurface::prepareContentsTexture):
(WebCore::CCRenderSurface::releaseContentsTexture):
(WebCore::CCRenderSurface::drawSurface):
(WebCore::CCRenderSurface::draw):
(WebCore::CCRenderSurface::name):
(WebCore::writeIndent):
(WebCore::CCRenderSurface::dumpSurface):
(WebCore::CCRenderSurface::owningLayerId):

  • platform/graphics/chromium/cc/CCRenderSurface.h: Copied from Source/WebCore/platform/graphics/chromium/RenderSurfaceChromium.h.

(WebCore::CCRenderSurface::contentRectCenter):
(WebCore::CCRenderSurface::drawOpacity):
(WebCore::CCRenderSurface::setDrawOpacity):
(WebCore::CCRenderSurface::setDrawTransform):
(WebCore::CCRenderSurface::drawTransform):
(WebCore::CCRenderSurface::setReplicaDrawTransform):
(WebCore::CCRenderSurface::replicaDrawTransform):
(WebCore::CCRenderSurface::setOriginTransform):
(WebCore::CCRenderSurface::originTransform):
(WebCore::CCRenderSurface::setScissorRect):
(WebCore::CCRenderSurface::scissorRect):
(WebCore::CCRenderSurface::setContentRect):
(WebCore::CCRenderSurface::contentRect):
(WebCore::CCRenderSurface::setSkipsDraw):
(WebCore::CCRenderSurface::skipsDraw):
(WebCore::CCRenderSurface::clearLayerList):
(WebCore::CCRenderSurface::layerList):
(WebCore::CCRenderSurface::setMaskLayer):
(WebCore::CCRenderSurface::contentsTexture):

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

(WebCore::CCTiledLayerImpl::CCTiledLayerImpl):

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

(WebCore::CCTiledLayerImpl::create):

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

(WebCore::CCVideoLayerImpl::CCVideoLayerImpl):

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

(WebCore::CCVideoLayerImpl::create):

4:18 PM Changeset in webkit [92244] by andersca@apple.com
  • 3 edits in trunk/Tools

Add mac-lion platform to old-run-webkit-tests
​https://bugs.webkit.org/show_bug.cgi?id=65577

  • Scripts/old-run-webkit-tests:
  • Scripts/webkitdirs.pm:

(isLion):

4:15 PM Changeset in webkit [92243] by macpherson@chromium.org
  • 3 edits in trunk/Source/WebCore

Implement CSSPropertyWebkitTextCombine, CSSPropertyWebkitTextEmphasisPosition and CSSPropertyWebkitTextEmphasisStyle in CSSStyleApplyProperty.
​https://bugs.webkit.org/show_bug.cgi?id=65517

Reviewed by Dimitri Glazkov.

No new tests. Just refactoring and moving code around.

  • css/CSSStyleApplyProperty.cpp:

Added class ApplyPropertyTextEmphasisStyle to handle the special TextEmphasisStyle logic.
(WebCore::ApplyPropertyTextEmphasisStyle::applyInheritValue):
(WebCore::ApplyPropertyTextEmphasisStyle::applyInitialValue):
(WebCore::ApplyPropertyTextEmphasisStyle::applyValue):
(WebCore::CSSStyleApplyProperty::CSSStyleApplyProperty):
Added initializations for CSSPropertyWebkitTextCombine, CSSPropertyWebkitTextEmphasisPosition, CSSPropertyWebkitTextEmphasisStyle.

  • css/CSSStyleSelector.cpp:

(WebCore::CSSStyleSelector::applyProperty):
Remove existing implementations.

3:47 PM Changeset in webkit [92242] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Remove LegacyDefaultOptionalArguments flag from SpeechInputResultList.idl
​https://bugs.webkit.org/show_bug.cgi?id=65570

Patch by Mark Pilgrim <​pilgrim@chromium.org> on 2011-08-02
Reviewed by Adam Barth.

No new tests, all existing tests pass

  • page/SpeechInputResultList.idl:
3:41 PM Changeset in webkit [92241] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Remove LegacyDefaultOptionalArguments flag from EventSource.idl
​https://bugs.webkit.org/show_bug.cgi?id=65568

Patch by Mark Pilgrim <​pilgrim@chromium.org> on 2011-08-02
Reviewed by Adam Barth.

No new tests, all existing tests pass.

  • page/EventSource.idl:
3:19 PM Changeset in webkit [92240] by commit-queue@webkit.org
  • 2 edits in trunk

Missing null check in WebViewImpl::selectionRange
​https://bugs.webkit.org/show_bug.cgi?id=65561

Patch by Varun Jain <​varunjain@google.com> on 2011-08-02
Reviewed by Darin Fisher.

  • Source/WebKit/chromium/src/WebViewImpl.cpp:
3:17 PM Changeset in webkit [92239] by gavinp@chromium.org
  • 2 edits in trunk/Source/WebCore

fail earlier to track down null CachedScript execution
​https://bugs.webkit.org/show_bug.cgi?id=65563

Over in ​http://code.google.com/p/chromium/issues/detail?id=75604 I can't reproduce the problem. Careful reading of the code hasn't led me
to an obvious cause either. This patch should cause failure earlier, and lead to better stacks. I'll watch Chrome Canary's crash
uploads carefully, and remove this CRASH() (and fix the underlying problem) once I understand it.

Reviewed by Alexey Proskuryakov.

No new tests, as my goal here is to get telemetry on a bug that I cannot reproduce.

  • dom/ScriptRunner.cpp:

(WebCore::ScriptRunner::queueScriptForExecution):

3:11 PM Changeset in webkit [92238] by commit-queue@webkit.org
  • 5 edits in trunk/Source/WebCore

Chromium: Update forked ScrollbarThemeChromiumMac.mm
​https://bugs.webkit.org/show_bug.cgi?id=65555

Merge recent changes to ScrollbarThemeMac.mm to ScrollbarThemeChromiumMac.mm.

Patch by Sailesh Agrawal <​sail@chromium.org> on 2011-08-02
Reviewed by James Robinson.

  • platform/chromium/ScrollbarOverlayUtilitiesChromiumMac.h:
  • platform/chromium/ScrollbarOverlayUtilitiesChromiumMac.mm:

(wkScrollbarPainterSetEnabled):

  • platform/chromium/ScrollbarThemeChromiumMac.h:

(WebCore::ScrollbarThemeChromiumMac::maxOverlapBetweenPages):

  • platform/chromium/ScrollbarThemeChromiumMac.mm:

(WebCore::ScrollbarThemeChromiumMac::registerScrollbar):
(WebCore::ScrollbarThemeChromiumMac::setNewPainterForScrollbar):
(WebCore::toScrollbarPainterKnobStyle):
(WebCore::ScrollbarThemeChromiumMac::updateScrollbarOverlayStyle):
(WebCore::ScrollbarThemeChromiumMac::updateEnabledState):
(WebCore::ScrollbarThemeChromiumMac::paint):

2:36 PM Changeset in webkit [92237] by commit-queue@webkit.org
  • 19 edits in trunk/Source/WebCore

Remove LegacyDefaultOptionalArguments flag from SVG IDL files
​https://bugs.webkit.org/show_bug.cgi?id=65337

Patch by Mark Pilgrim <​pilgrim@chromium.org> on 2011-08-02
Reviewed by Nikolas Zimmermann.

No new tests, all existing tests pass.

  • svg/ElementTimeControl.idl:
  • svg/SVGAnimationElement.idl:
  • svg/SVGDocument.idl:
  • svg/SVGElementInstance.idl:
  • svg/SVGElementInstanceList.idl:
  • svg/SVGException.idl:
  • svg/SVGFEDropShadowElement.idl:
  • svg/SVGFEGaussianBlurElement.idl:
  • svg/SVGFEMorphologyElement.idl:
  • svg/SVGFilterElement.idl:
  • svg/SVGLocatable.idl:
  • svg/SVGMarkerElement.idl:
  • svg/SVGPathElement.idl:
  • svg/SVGSVGElement.idl:
  • svg/SVGStylable.idl:
  • svg/SVGStyleElement.idl:
  • svg/SVGTests.idl:
  • svg/SVGTextContentElement.idl:
2:35 PM Changeset in webkit [92236] by commit-queue@webkit.org
  • 14 edits in trunk/Source/WebCore

Remove LegacyDefaultOptionalArguments flag from CSS IDL files
​https://bugs.webkit.org/show_bug.cgi?id=65341

Patch by Mark Pilgrim <​pilgrim@chromium.org> on 2011-08-02
Reviewed by Adam Barth.

No new tests. All existing tests pass.

  • css/CSSMediaRule.idl:
  • css/CSSPrimitiveValue.idl:
  • css/CSSRuleList.idl:
  • css/CSSStyleDeclaration.idl:
  • css/CSSStyleSheet.idl:
  • css/CSSValueList.idl:
  • css/MediaList.idl:
  • css/MediaQueryList.idl:
  • css/MediaQueryListListener.idl:
  • css/StyleMedia.idl:
  • css/StyleSheetList.idl:
  • css/WebKitCSSKeyframesRule.idl:
  • css/WebKitCSSMatrix.idl:
2:33 PM Changeset in webkit [92235] by zmo@google.com
  • 2 edits in trunk/LayoutTests

Unreviewed, test expectations update, gardener stuff.

  • platform/chromium/test_expectations.txt:
2:26 PM Changeset in webkit [92234] by commit-queue@webkit.org
  • 21 edits in trunk/Source

[chromium] Layering violations in gesture recognizer
​https://bugs.webkit.org/show_bug.cgi?id=65044

Source/WebCore:

Patch by Robert Kroeger <​rjkroege@chromium.org> on 2011-08-02
Reviewed by Adam Barth.

Divided the gesture recognizer up to correct a layering
violation by moving gesture implementation from it to
EventHandler::handleGestureEvent so that the gesture recognizer
could simply be an engine for generating gesture events from
touch events.

  • dom/WheelEvent.cpp:

(WebCore::WheelEvent::WheelEvent):
(WebCore::WheelEvent::initWheelEvent):

  • dom/WheelEvent.h:

Added symbolic contstant for tick multiplier.

  • page/EventHandler.cpp:

(WebCore::EventHandler::EventHandler):
Removed construction of gesture recognizer.
(WebCore::EventHandler::handleGestureEvent):
Extended for new gesture events added in this patch.
(WebCore::EventHandler::handleTouchEvent):
Removed call into gesture recognizer.

  • page/EventHandler.h:
  • page/FrameView.cpp:

(WebCore::FrameView::reset):
Removed code to reset gesture recognizer.

  • platform/PlatformGestureEvent.h:

(WebCore::PlatformGestureEvent::PlatformGestureEvent):
(WebCore::PlatformGestureEvent::deltaX):
(WebCore::PlatformGestureEvent::deltaY):
(WebCore::PlatformGestureEvent::shiftKey):
(WebCore::PlatformGestureEvent::ctrlKey):
(WebCore::PlatformGestureEvent::altKey):
(WebCore::PlatformGestureEvent::metaKey):
Added additional gesture types and parameters.

  • platform/PlatformGestureRecognizer.h:
  • platform/chromium/FramelessScrollView.h:

Added support for displatching gesture and touch events.

  • platform/chromium/GestureRecognizerChromium.cpp:

(WebCore::InnerGestureRecognizer::InnerGestureRecognizer):
(WebCore::InnerGestureRecognizer::reset):
(WebCore::InnerGestureRecognizer::constructClickGestureEvent):
(WebCore::InnerGestureRecognizer::processTouchEventForGestures):
Removed all depedencies on EventHandler. Instead, create gesture
events from touch event stream.
(WebCore::InnerGestureRecognizer::constructScrollGesture):
(WebCore::touchDown):
(WebCore::noGesture):
(WebCore::click):
(WebCore::isClickOrScroll):
(WebCore::inScroll):
(WebCore::GestureRecognizerChromium::processTouchEventForGestures):
(WebCore::GestureRecognizerChromium::reset):

  • platform/chromium/GestureRecognizerChromium.h:
  • platform/chromium/PopupMenuChromium.cpp:

Added support to handle touch events and gesture events.
(WebCore::PopupContainer::handleTouchEvent):
(WebCore::PopupContainer::handleGestureEvent):
(WebCore::PopupListBox::handleTouchEvent):
(WebCore::PopupListBox::handleGestureEvent):

  • platform/chromium/PopupMenuChromium.h:
  • platform/mac/ScrollAnimatorMac.mm:

(WebCore::ScrollAnimatorMac::handleGestureEvent):
Ignores the newly added gesture event types.

Source/WebKit/chromium:

Reviewed by Adam Barth.

Moved gesture recognition higher up the event delivery code
path so that recognition of gestures is completely separated
from gesture handling.

Patch by Robert Kroeger <​rjkroege@chromium.org> on 2011-08-02

  • features.gypi:
  • src/WebFrameImpl.cpp:

(WebKit::WebFrameImpl::createFrameView):
Added code to reset the gesture recognizer to preserve parallel
layout test execution.

  • src/WebPopupMenuImpl.cpp:

Support dispatching touch and gesture events to Popups.
(WebKit::WebPopupMenuImpl::WebPopupMenuImpl):
(WebKit::WebPopupMenuImpl::TouchEvent):
(WebKit::WebPopupMenuImpl::handleInputEvent):

  • src/WebPopupMenuImpl.h:
  • src/WebViewImpl.cpp:

(WebKit::WebViewImpl::WebViewImpl):
Added gesture recognition creation.
(WebKit::WebViewImpl::touchEvent):
Added recognition of gestures and dispatch of gesture events.
(WebKit::WebViewImpl::resetGestureRecognizer):

  • src/WebViewImpl.h:
2:22 PM Changeset in webkit [92233] by fpizlo@apple.com
  • 9 edits in trunk/Source/JavaScriptCore

JSC GC uses dummy cells to avoid having to remember which cells
it has already destroyed
​https://bugs.webkit.org/show_bug.cgi?id=65556

Reviewed by Oliver Hunt.

This gets rid of dummy cells, and ensures that it's not necessary
to invoke a destructor on cells that have already been swept. In
the common case, a block knows that either all of its free cells
still need to have destructors called, or none of them do, which
minimizes the amount of branching that needs to happen per cell
when performing a sweep.

This is performance neutral on SunSpider and V8. It is meant as
a stepping stone to simplify the implementation of more
sophisticated sweeping algorithms.

  • heap/Heap.cpp:

(JSC::CountFunctor::ClearMarks::operator()):

  • heap/MarkedBlock.cpp:

(JSC::MarkedBlock::initForCellSize):
(JSC::MarkedBlock::callDestructor):
(JSC::MarkedBlock::specializedReset):
(JSC::MarkedBlock::reset):
(JSC::MarkedBlock::specializedSweep):
(JSC::MarkedBlock::sweep):
(JSC::MarkedBlock::produceFreeList):
(JSC::MarkedBlock::lazySweep):
(JSC::MarkedBlock::blessNewBlockForFastPath):
(JSC::MarkedBlock::blessNewBlockForSlowPath):
(JSC::MarkedBlock::canonicalizeBlock):

  • heap/MarkedBlock.h:

(JSC::MarkedBlock::FreeCell::setNoObject):
(JSC::MarkedBlock::setDestructorState):
(JSC::MarkedBlock::destructorState):
(JSC::MarkedBlock::notifyMayHaveFreshFreeCells):

  • runtime/JSCell.cpp:
  • runtime/JSCell.h:

(JSC::JSCell::JSCell::JSCell):

  • runtime/JSGlobalData.cpp:

(JSC::JSGlobalData::JSGlobalData):
(JSC::JSGlobalData::clearBuiltinStructures):

  • runtime/JSGlobalData.h:
  • runtime/Structure.h:
2:22 PM Changeset in webkit [92232] by andersca@apple.com
  • 2 edits in trunk/LayoutTests

Add scrollbars/scrollbar-drag-thumb-with-large-content.html to the Skipped list.

  • platform/mac/Skipped:
2:19 PM Changeset in webkit [92231] by msaboff@apple.com
  • 6 edits in trunk/Source

Virtual copying of FastMalloc allocated memory causes madvise MADV_FREE_REUSABLE errors
​https://bugs.webkit.org/show_bug.cgi?id=65502

Reviewed by Anders Carlsson.

Source/JavaScriptCore:

With the fix of the issues causing madvise MADV_FREE_REUSABLE to fail,
added an assert to the return code of madvise to catch any regressions.

  • wtf/TCSystemAlloc.cpp:

(TCMalloc_SystemRelease):

Source/WebCore:

Change the vm_copy in PurgeableBuffer::create to be a memcpy. The
vm_copy causes the process to have additional references to the same
memory region. These additional reference caused madvise(MADV_FREE_REUSABLE)
to fail when it encountered such pages.

No tests added this is a resource defect and not a functional issue.

  • platform/mac/PurgeableBufferMac.cpp:

(WebCore::PurgeableBuffer::create):

Source/WebKit2:

Changed OOL message to use MACH_MSG_PHYSICAL_COPY flag instead of virtual flag
so that the original memory region isn't referenced by the message and ultimately
the receiving process. The additional reference caused madvise(MADV_FREE_REUSABLE)
to fail when it encountered such pages.

  • Platform/CoreIPC/mac/ConnectionMac.cpp:

(CoreIPC::Connection::sendOutgoingMessage):

2:13 PM Changeset in webkit [92230] by andersca@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Fix Windows build.

2:08 PM Changeset in webkit [92229] by zmo@google.com
  • 2 edits in trunk/LayoutTests

Unreviewed, test expectations update, gardener stuff.

  • platform/chromium/test_expectations.txt:
2:06 PM Changeset in webkit [92228] by andersca@apple.com
  • 2 edits in trunk/LayoutTests

Reviewed by Sam Weinig.

  • fast/blockflow/japanese-rl-text-with-broken-font.html:

This test would create an empty RenderText after the change to make it asynchronous.
Fix this by putting the script inside the head.

2:06 PM Changeset in webkit [92227] by senorblanco@chromium.org
  • 2 edits in trunk/Source/WebCore

[Chromium] Remove an unnecessary readback during accelerated
canvas-to-canvas draws.
​https://bugs.webkit.org/show_bug.cgi?id=65560

Reviewed by James Robinson.

No new tests. (Printing has few tests, sadly.)

  • platform/graphics/skia/ImageBufferSkia.cpp:

(WebCore::ImageBuffer::draw):
Revert most of the changes to this file from
​http://trac.webkit.org/changeset/91870, except for the call which makes
the source context current, so that the readPixels() still works.

1:52 PM Changeset in webkit [92226] by andersca@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Fix a Windows build error.

1:47 PM Changeset in webkit [92225] by zmo@google.com
  • 2 edits in trunk/Source/WebKit/chromium

Unreviewed, mark SelectPopupMenuTest.SelectItemKeyEvent as flaky, gardener stuff.

  • tests/PopupMenuTest.cpp:

(WebKit::TEST_F):

1:40 PM Changeset in webkit [92224] by fpizlo@apple.com
  • 10 edits in trunk/Source

JSC GC is far too conservative about growing the heap size, particularly
on desktop platforms
​https://bugs.webkit.org/show_bug.cgi?id=65438

Reviewed by Oliver Hunt.

Source/JavaScriptCore:

The minimum heap size is now 16MB instead of 512KB, provided all of the
following are true:
a) ENABLE(LARGE_HEAP) is set, which currently only happens on

x86 targets, but could reasonably happen on any platform that is
known to have a decent amount of RAM.

b) JSGlobalData is initialized with HeapSize = LargeHeap, which

currently only happens when it's the JSDOMWindowBase in WebCore or
in the jsc command-line tool.

This is a 4.1% speed-up on SunSpider.

(JSC::Heap::Heap):
(JSC::Heap::collect):

  • heap/Heap.h:
  • jsc.cpp:

(main):

  • runtime/JSGlobalData.cpp:

(JSC::JSGlobalData::JSGlobalData):
(JSC::JSGlobalData::createContextGroup):
(JSC::JSGlobalData::create):
(JSC::JSGlobalData::createLeaked):
(JSC::JSGlobalData::sharedInstance):

  • runtime/JSGlobalData.h:
  • wtf/Platform.h:

Source/WebCore:

No change in behavior, thus no new tests.

Pass the LargeHeap hint to JSGlobalData when creating the JSC runtime
instance corresponding to non-worker JS code.

  • bindings/js/JSDOMWindowBase.cpp:

(WebCore::JSDOMWindowBase::commonJSGlobalData):

1:38 PM Changeset in webkit [92223] by commit-queue@webkit.org
  • 11 edits in trunk

--webkit-visual-word: ctrl-arrow is not able to reach the boundary of line
​https://bugs.webkit.org/show_bug.cgi?id=61346

Patch by Van Lam <​vanlam@google.com> on 2011-08-02
Reviewed by Ryosuke Niwa.

Source/WebCore:

When collecintg word boundaries in a box, explicitly check if the box
is at the end of line (so it should contain the end-of-line position)
and collect the end-of-line position.

  • editing/visible_units.cpp:

(WebCore::isBoxVisuallyLastInLine):
(WebCore::appendPositionAtLogicalEndOfLine):
(WebCore::collectWordBreaksInBoxInsideBlockWithSameDirectionality):
(WebCore::collectWordBreaksInBoxInsideBlockWithDifferntDirectionality):

LayoutTests:

Updated test cases to check for visual word movement to the end-of-line
position.

  • editing/selection/move-by-word-visually-multi-line-expected.txt:
  • editing/selection/move-by-word-visually-multi-line.html:
  • editing/selection/move-by-word-visually-multi-space-expected.txt:
  • editing/selection/move-by-word-visually-multi-space.html:
  • editing/selection/move-by-word-visually-single-space-inline-element-expected.txt:
  • editing/selection/move-by-word-visually-single-space-inline-element.html:
  • editing/selection/move-by-word-visually-single-space-one-element-expected.txt:
  • editing/selection/move-by-word-visually-single-space-one-element.html:
1:31 PM Changeset in webkit [92222] by zmo@google.com
  • 2 edits in trunk/LayoutTests

Unreviewed, test expectations update, gardener stuff.

  • platform/chromium/test_expectations.txt:
1:24 PM QtWebKitRelease22 edited by joel.parks@nokia.com
(diff)
1:12 PM Changeset in webkit [92221] by vsevik@chromium.org
  • 4 edits in trunk/Source/WebCore

Web Inspector: Should save scroll positions for network panel Response and Preview tabs.
​https://bugs.webkit.org/show_bug.cgi?id=65557

Reviewed by Pavel Feldman.

  • inspector/front-end/NetworkItemView.js:

(WebInspector.ResourceContentView.prototype.get innerView):
(WebInspector.ResourceContentView.prototype.set innerView):
(WebInspector.ResourceContentView.prototype.hide):

  • inspector/front-end/ResourcePreviewView.js:

(WebInspector.ResourcePreviewView.prototype.contentLoaded):

  • inspector/front-end/ResourceResponseView.js:

(WebInspector.ResourceResponseView.prototype.contentLoaded):

1:06 PM Changeset in webkit [92220] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

Buildbot: The Properties object has no "hasProperty" method in buildbot 8.3
​https://bugs.webkit.org/show_bug.cgi?id=65554

Using "has_key" instead of "hasProperty" in order to support buildbot 8.3.

Patch by Alexandru Chiculita <Alexandru Chiculita> on 2011-08-02
Reviewed by Adam Roben.

  • BuildSlaveSupport/build.webkit.org-config/master.cfg:
1:00 PM Changeset in webkit [92219] by jochen@chromium.org
  • 6 edits
    2 adds in trunk

Tools: Implement WebPermissionClient::allowPlugins for layout tests
​https://bugs.webkit.org/show_bug.cgi?id=33991

Reviewed by Adam Barth.

  • DumpRenderTree/chromium/LayoutTestController.cpp:

(LayoutTestController::LayoutTestController):
(LayoutTestController::setPluginsAllowed):

  • DumpRenderTree/chromium/LayoutTestController.h:
  • DumpRenderTree/chromium/WebPermissions.cpp:

(WebPermissions::allowPlugins):
(WebPermissions::setPluginsAllowed):
(WebPermissions::reset):

  • DumpRenderTree/chromium/WebPermissions.h:

LayoutTests: Add a test for FrameLoaderClient::allowPlugins
​https://bugs.webkit.org/show_bug.cgi?id=33991

Reviewed by Adam Barth.

  • platform/chromium/permissionclient/plugin-permission-expected.txt: Added.
  • platform/chromium/permissionclient/plugin-permission.html: Added.
12:57 PM Changeset in webkit [92218] by abarth@webkit.org
  • 2 edits in trunk/Tools

TestFailures unit tests leak a global variable
​https://bugs.webkit.org/show_bug.cgi?id=65500

Reviewed by Adam Roben.

qunit detects the leak if you check the "noglobals" box on run-unittests.html.

  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/Builder_unittests.js:
12:57 PM Changeset in webkit [92217] by fpizlo@apple.com
  • 3 edits in trunk/Source/JavaScriptCore

JSC does a GC even when the heap still has free pages
​https://bugs.webkit.org/show_bug.cgi?id=65445

Reviewed by Oliver Hunt.

If the high watermark is not reached, then we allocate new blocks as
before. If the current watermark does reach (or exceed) the high
watermark, then we check if there is a block on the free block pool.
If there is, we simply allocation from it. If there isn't, we
invoke a collectin as before. This effectively couples the elastic
scavenging to the collector's decision function. That is, if an
application rapidly varies its heap usage (sometimes using more and
sometimes less) then the collector will not thrash as it used to.
But if heap usage drops and stays low then the scavenger thread and
the GC will eventually reach a kind of consensus: the GC will set
the watermark low because of low heap usage, and the scavenger thread
will steadily eliminate pages from the free page pool, until the size
of the free pool is below the high watermark.

On command-line, this is neutral on SunSpider and Kraken and a 3% win
on V8. In browser, this is a 1% win on V8 and neutral on the other
two.

  • heap/Heap.cpp:

(JSC::Heap::allocateSlowCase):
(JSC::Heap::allocateBlock):

  • heap/Heap.h:
12:50 PM Changeset in webkit [92216] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit/efl

[EFL] Ignore empty requests on policy decisions.
​https://bugs.webkit.org/show_bug.cgi?id=64536

Patch by Raphael Kubo da Costa <​kubo@profusion.mobi> on 2011-08-02
Reviewed by Kenneth Rohde Christiansen.

This is the same fix introduced in the GTK+ port in r41313: empty
request should be ignored, so tests such as
fast/loader/empty-embed-src-attribute.html do not crash.

  • WebCoreSupport/FrameLoaderClientEfl.cpp:

(WebCore::FrameLoaderClientEfl::dispatchDecidePolicyForResponse):
(WebCore::FrameLoaderClientEfl::dispatchDecidePolicyForNewWindowAction):
(WebCore::FrameLoaderClientEfl::dispatchDecidePolicyForNavigationAction):

12:15 PM Changeset in webkit [92215] by zmo@google.com
  • 4 edits in trunk/LayoutTests

Unreviewed, rebaseline, gardener stuff.

  • platform/chromium-win-xp/fast/text/international/bidi-mirror-he-ar-expected.png:
  • platform/chromium-win-xp/fast/text/justify-ideograph-leading-expansion-expected.png:
  • platform/chromium/test_expectations.txt:
11:45 AM Changeset in webkit [92214] by Lucas Forschler
  • 11 edits in branches/safari-534.51-branch

Merge r91616.

11:41 AM Changeset in webkit [92213] by vsevik@chromium.org
  • 8 edits
    1 copy in trunk

Web Inspector: Remove initator column for JSC, fix network resource initiator tests.
​https://bugs.webkit.org/show_bug.cgi?id=65534

Reviewed by Pavel Feldman.

Source/WebCore:

  • inspector/front-end/NetworkPanel.js:

(WebInspector.NetworkLogView.prototype._createTable):
(WebInspector.NetworkLogView.prototype.switchToDetailedView):
(WebInspector.NetworkLogView.prototype.switchToBriefView):
(WebInspector.NetworkLogView.prototype._toggleGridMode):
(WebInspector.NetworkLogView.prototype._toggleViewingResourceMode):
(WebInspector.NetworkDataGridNode.prototype.createCells):
(WebInspector.NetworkDataGridNode.prototype.refreshResource):

  • inspector/front-end/Settings.js:

Source/WebKit/chromium:

  • src/js/DevTools.js:

():

LayoutTests:

  • http/tests/inspector/network/network-initiator-expected.txt:
  • http/tests/inspector/network/network-initiator.html:
  • platform/chromium/http/tests/inspector/network/network-initiator-expected.txt: Copied from LayoutTests/http/tests/inspector/network/network-initiator-expected.txt.
11:32 AM Changeset in webkit [92212] by vsevik@chromium.org
  • 5 edits in trunk/Source/WebCore

Web Inspector: SourceFrame should store saved scroll positions in View.
​https://bugs.webkit.org/show_bug.cgi?id=65472

Reviewed by Pavel Feldman.

  • inspector/front-end/ResourcesPanel.js:

(WebInspector.FrameResourceTreeElement.prototype._recreateSourceView):

  • inspector/front-end/ScriptsPanel.js:

(WebInspector.ScriptsPanel.prototype._sourceFileChanged):

  • inspector/front-end/SourceFrame.js:

(WebInspector.SourceFrame.prototype.show):
(WebInspector.SourceFrame.prototype.hide):
(WebInspector.SourceFrame.prototype.get scrollLeft):
(WebInspector.SourceFrame.prototype.set scrollLeft):
(WebInspector.SourceFrame.prototype.get scrollTop):
(WebInspector.SourceFrame.prototype.set scrollTop):

  • inspector/front-end/View.js:

(WebInspector.View.prototype._innerShow):
(WebInspector.View.prototype._innerHide):
(WebInspector.View.prototype.detach):
(WebInspector.View.prototype.storeScrollPositions):
(WebInspector.View.prototype.inheritScrollPositionsFromView):
(WebInspector.View.prototype.restoreScrollPositions):

11:32 AM Changeset in webkit [92211] by zmo@google.com
  • 2 edits in trunk/LayoutTests

Unreviewed, test expectation updates, gardener stuff.

  • platform/chromium/test_expectations.txt:
11:29 AM Changeset in webkit [92210] by jeffm@apple.com
  • 4 edits in trunk/Source

Move WTF_USE_AVFOUNDATION from JavaScriptCore/wtf/platform.h to WebCore/config.h
​https://bugs.webkit.org/show_bug.cgi?id=65552

Since this is a WebCore feature, there's no need to define it in JavaScriptCore/wtf/platform.h.

Reviewed by Adam Roben.

Source/JavaScriptCore:

  • wtf/Platform.h: Removed WTF_USE_AVFOUNDATION.

Source/WebCore:

No change in functionality, so new tests.

  • config.h: Added WTF_USE_AVFOUNDATION.
11:29 AM Changeset in webkit [92209] by andersca@apple.com
  • 1 edit
    4 adds in trunk/LayoutTests

Add Snow Leopard specific test results.

  • platform/mac-snowleopard/fast/forms/input-placeholder-visibility-1-expected.txt: Added.
  • platform/mac-snowleopard/fast/forms/input-placeholder-visibility-3-expected.txt: Added.
  • platform/mac-snowleopard/fast/forms/textarea-placeholder-visibility-1-expected.txt: Added.
  • platform/mac-snowleopard/fast/forms/textarea-placeholder-visibility-2-expected.txt: Added.
11:26 AM Changeset in webkit [92208] by rniwa@webkit.org
  • 2 edits in trunk/LayoutTests

Remove time out expectation since media/track has been added to the skipped list on GTK.

  • platform/gtk/test_expectations.txt:
11:07 AM Changeset in webkit [92207] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

Split RenderBlock::layoutRunsAndFloats into multiple methods
​https://bugs.webkit.org/show_bug.cgi?id=65376

Extracted layoutRunsAndFloatsInRange, linkToEndLineIfNeeded and repaintDirtyFloats methods and moved
most of the locals to LineLayoutState.

Patch by Alexandru Chiculita <Alexandru Chiculita> on 2011-08-02
Reviewed by David Hyatt.

No new tests needed because it is just refactoring existing code.

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

(WebCore::LineLayoutState::LineLayoutState):
(WebCore::LineLayoutState::endLineMatched):
(WebCore::LineLayoutState::setEndLineMatched):
(WebCore::LineLayoutState::checkForFloatsFromLastLine):
(WebCore::LineLayoutState::setCheckForFloatsFromLastLine):
(WebCore::LineLayoutState::lineInfo):
(WebCore::LineLayoutState::endLineLogicalTop):
(WebCore::LineLayoutState::setEndLineLogicalTop):
(WebCore::LineLayoutState::endLine):
(WebCore::LineLayoutState::setEndLine):
(WebCore::LineLayoutState::lastFloat):
(WebCore::LineLayoutState::setLastFloat):
(WebCore::LineLayoutState::floats):
(WebCore::LineLayoutState::floatIndex):
(WebCore::LineLayoutState::setFloatIndex):
(WebCore::RenderBlock::layoutRunsAndFloats):
(WebCore::RenderBlock::layoutRunsAndFloatsInRange):
(WebCore::RenderBlock::linkToEndLineIfNeeded):
(WebCore::RenderBlock::repaintDirtyFloats):
(WebCore::RenderBlock::layoutInlineChildren):
(WebCore::RenderBlock::determineStartPosition):
(WebCore::RenderBlock::determineEndPosition):
(WebCore::RenderBlock::matchedEndLine):

10:53 AM Changeset in webkit [92206] by nduca@chromium.org
  • 5 edits in trunk/Source

[chromium] Remove resizeOnscreenContent from LRC. It can be derived.
​https://bugs.webkit.org/show_bug.cgi?id=65519

Reviewed by James Robinson.

Source/WebCore:

  • platform/graphics/chromium/LayerRendererChromium.cpp:

(WebCore::LayerRendererChromium::setViewport):

  • platform/graphics/chromium/LayerRendererChromium.h:

Source/WebKit/chromium:

  • src/WebViewImpl.cpp:

(WebKit::WebViewImpl::resize):
(WebKit::WebViewImpl::setIsAcceleratedCompositingActive):

10:37 AM Changeset in webkit [92205] by enne@google.com
  • 2 edits in trunk/LayoutTests

[chromium] Remove expectations for no longer crashing tests.
​https://bugs.webkit.org/show_bug.cgi?id=65155

Unreviewed expectation update.

These were fixed after Chromium r94376, r94374, r94373.

  • platform/chromium/test_expectations.txt:
10:36 AM Changeset in webkit [92204] by simonjam@chromium.org
  • 18 edits
    2 copies in branches/chromium/835

Merge 92143 - .: REGRESSION (r39725?): Resources removed from document can not be freed until the document is deleted
​https://bugs.webkit.org/show_bug.cgi?id=61006

Patch by Scott Graham <​scottmg@chromium.org> on 2011-08-01
Reviewed by Antti Koivisto.

Update exports for test harness.

  • Source/autotools/symbols.filter:

Source/WebCore: REGRESSION (r39725?): Resources removed from document can not be freed
until the document is deleted
​https://bugs.webkit.org/show_bug.cgi?id=61006

Patch by Scott Graham <​scottmg@chromium.org> on 2011-08-01
Reviewed by Antti Koivisto.

Upon completing a load start a Timer to iterate through
CachedResourceLoader's m_documentResources map to check for any items
that have only one reference (thus being the reference in the map
itself). The map should really be weak, but because the
CachedResourceHandle achieves bookkeeping work in addition to
reference counting, this is a simpler and more localized way to free
the used memory while maintaining the other behaviour (when
CachedResource is used as proxy).

With this patch the testcase at
​https://bugs.webkit.org/attachment.cgi?id=93850 should no longer
consume 400MB of ram on load. Test added for crash discovered in
previous revision, but no tests for memory usage.

Test: http/tests/inspector/network/disabled-cache-crash.html

  • WebCore.exp.in:
  • loader/cache/CachedResource.h:

(WebCore::CachedResource::hasOneHandle):

  • loader/cache/CachedResourceLoader.cpp:

(WebCore::CachedResourceLoader::CachedResourceLoader):
(WebCore::CachedResourceLoader::loadDone):
(WebCore::CachedResourceLoader::garbageCollectDocumentResourcesTimerFired):

  • loader/cache/CachedResourceLoader.h:
  • testing/Internals.cpp:

(WebCore::Internals::disableMemoryCache):

  • testing/Internals.h:
  • testing/Internals.idl:

Source/WebKit2: REGRESSION (r39725?): Resources removed from document can not be freed until the document is deleted
​https://bugs.webkit.org/show_bug.cgi?id=61006

Patch by Scott Graham <​scottmg@chromium.org> on 2011-08-01
Reviewed by Antti Koivisto.

Update exports for test harness.

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

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

Test for CachedResourceLoader. Not caused by cache-disabling but very
difficult to reproduce when cache is active, so use cache disable in
inspector to exercise code.

Patch by Scott Graham <​scottmg@chromium.org> on 2011-08-01
Reviewed by Antti Koivisto.

  • http/tests/inspector/network/disabled-cache-crash-expected.txt: Added.
  • http/tests/inspector/network/disabled-cache-crash.html: Added.
  • platform/gtk/Skipped:
  • platform/mac/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:

TBR=​commit-queue@webkit.org
Review URL: ​http://codereview.chromium.org/7548016

10:33 AM Changeset in webkit [92203] by thakis@chromium.org
  • 2 edits in trunk/Source/WebCore

[chromium] Fix unresolved symbols error when linking to libwebkit.dylib
​https://bugs.webkit.org/show_bug.cgi?id=65498

Reviewed by Dimitri Glazkov.

  • WebCore.gyp/mac/adjust_visibility.sh:
10:31 AM Changeset in webkit [92202] by simonjam@chromium.org
  • 4 edits in branches/chromium/835/Source/WebCore

Merge 91725 - REGRESSION (r39725?): Resources removed from document can not be freed until the document is deleted
​https://bugs.webkit.org/show_bug.cgi?id=61006

Patch by Scott Graham <​scottmg@chromium.org> on 2011-07-25
Reviewed by Nate Chapin.

Rollback previous patch. Occasional crashes seen caused by
CachedResource being deleted twice.

  • loader/cache/CachedResourceLoader.cpp:

(WebCore::CachedResourceLoader::CachedResourceLoader):
(WebCore::CachedResourceLoader::loadDone):

  • loader/cache/CachedResourceLoader.h:

TBR=​commit-queue@webkit.org
Review URL: ​http://codereview.chromium.org/7542020

10:16 AM Changeset in webkit [92201] by wsiegrist@apple.com
  • 1 edit in trunk/Tools/BuildSlaveSupport/build.webkit.org-config/config.json

Removed a dangling reference to Leopard Intel Release (Build).

Unreviewed.

  • BuildSlaveSupport/build.webkit.org-config/config.json:
10:08 AM Changeset in webkit [92200] by commit-queue@webkit.org
  • 3 edits
    4 adds in trunk

hover then un-hover makes state change
​https://bugs.webkit.org/show_bug.cgi?id=56401

Patch by Mihnea Ovidenie <​mihnea@adobe.com> on 2011-08-02
Reviewed by David Hyatt.

Source/WebCore:

When a 'before' pseudo-element is re-added, we should check whether the insertion point is an anonymous
block with inline children. If it is, then we should change the insertion point to the first child of the
anonymous block, otherwise the 'before' pseudo-element ends up in a different block. We choose the insertion
point to be the first child only if the anonymous block has children, otherwise the before element ends up
in a wrong block.

Tests: fast/dynamic/hover-before-position-after-style-change.html

fast/dynamic/hover-before-position-after-style-change2.html

  • rendering/RenderObjectChildList.cpp:

(WebCore::RenderObjectChildList::updateBeforeAfterContent):

LayoutTests:

  • fast/dynamic/hover-before-position-after-style-change-expected.txt: Added.
  • fast/dynamic/hover-before-position-after-style-change.html: Added.
  • fast/dynamic/hover-before-position-after-style-change2-expected.txt: Added.
  • fast/dynamic/hover-before-position-after-style-change2.html: Added.
10:00 AM Changeset in webkit [92199] by jocelyn.turcotte@nokia.com
  • 6 edits
    1 copy in trunk/Source/WebKit2

[Qt] Add QtWebProcess in PATH at runtime for WebKit2 API auto tests.
​https://bugs.webkit.org/show_bug.cgi?id=65378

Reviewed by Benjamin Poulain.

This prevent the runner of the tests from having to add it to PATH himself.

  • UIProcess/API/qt/tests/commonviewtests/tst_commonviewtests.cpp:

(tst_CommonViewTests::tst_CommonViewTests):

  • UIProcess/API/qt/tests/qdesktopwebview/tst_qdesktopwebview.cpp:

(tst_QDesktopWebView::tst_QDesktopWebView):

  • UIProcess/API/qt/tests/qtouchwebview/tst_qtouchwebview.cpp:

(tst_QTouchWebView::tst_QTouchWebView):

  • UIProcess/API/qt/tests/tests.pri:
  • UIProcess/API/qt/tests/util.cpp: Copied from Source/WebKit2/UIProcess/API/qt/tests/util.h.

Move definitions to a cpp file to prevent unused static symbol warnings.
(addQtWebProcessToPath):
(waitForSignal):

  • UIProcess/API/qt/tests/util.h:
10:00 AM Changeset in webkit [92198] by jocelyn.turcotte@nokia.com
  • 3 edits in trunk/Source/WebKit2

TiledDrawingArea: Handle update requests in the order they were received.
​https://bugs.webkit.org/show_bug.cgi?id=64365

Reviewed by Andreas Kling.

TiledDrawingAreaProxy::createTiles() uses the distance of tiles to the
center of the viewport to decide which tile should be rendered first.
This logic is useless if the requests are not handled in the same order
as they were received.
Now use a list instead of a map to hold pending tile update requests.

  • WebProcess/WebPage/TiledDrawingArea.cpp:

(WebKit::TiledDrawingArea::tileUpdateTimerFired):
(WebKit::TiledDrawingArea::cancelTileUpdate):
(WebKit::TiledDrawingArea::requestTileUpdate):

  • WebProcess/WebPage/TiledDrawingArea.h:
9:34 AM Changeset in webkit [92197] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

[Qt] Simplify Path::translate() implementation.
​https://bugs.webkit.org/show_bug.cgi?id=65540

Patch by Andreas Kling <​kling@webkit.org> on 2011-08-02
Reviewed by Benjamin Poulain.

  • platform/graphics/qt/PathQt.cpp:

(WebCore::Path::translate):

8:59 AM Changeset in webkit [92196] by podivilov@chromium.org
  • 6 edits in trunk

Web Inspector: merge FormattedSourceFile into SourceFile.
​https://bugs.webkit.org/show_bug.cgi?id=65411

Reviewed by Pavel Feldman.

Prepare for splitting SourceFile into RawSourceCode and UISourceCode:

  • merge FormattedSourceFile into SourceFile
  • rename SourceFile to RawSourceCode
  • hide formatter worker lazy creation in ScriptFormatter

Source/WebCore:

  • inspector/front-end/DebuggerPresentationModel.js:

(WebInspector.DebuggerPresentationModel):
(WebInspector.DebuggerPresentationModel.prototype._scriptLocationToUILocation):
(WebInspector.DebuggerPresentationModel.prototype._uiLocationToScriptLocation):
(WebInspector.DebuggerPresentationModel.prototype._addScript):

  • inspector/front-end/ScriptFormatter.js:

(WebInspector.ScriptFormatter):
(WebInspector.ScriptFormatter.prototype._didFormatContent):
(WebInspector.ScriptFormatter.prototype.get _worker):

  • inspector/front-end/SourceFile.js:

(WebInspector.RawSourceCode):
(WebInspector.RawSourceCode.prototype.setFormatted):
(WebInspector.RawSourceCode.prototype.createSourceMappingIfNeeded.didRequestContent):
(WebInspector.RawSourceCode.prototype.createSourceMappingIfNeeded):
(WebInspector.RawSourceCode.prototype._didRequestContent.didFormatContent):
(WebInspector.RawSourceCode.prototype._didRequestContent):
(WebInspector.RawSourceCode.prototype._invokeRequestContentCallbacks):

LayoutTests:

  • inspector/debugger/source-file.html:
8:58 AM Changeset in webkit [92195] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

[CSSRegions]Fix build after changeset 92106
​https://bugs.webkit.org/show_bug.cgi?id=65539

Patch by Mihnea Ovidenie <​mihnea@adobe.com> on 2011-08-02
Reviewed by Adam Roben.

No new tests since it is just a build fix.

  • css/CSSStyleSelector.cpp:

(WebCore::CSSStyleSelector::applyProperty):

8:54 AM Changeset in webkit [92194] by vsevik@chromium.org
  • 3 edits
    2 adds in trunk

Web Inspector: [REGRESSION] Console evaluations leading to resource loading cause errors.
​https://bugs.webkit.org/show_bug.cgi?id=65466

Reviewed by Pavel Feldman.

Source/WebCore:

Test: http/tests/inspector/network/network-initiator-from-console.html

  • inspector/front-end/NetworkPanel.js:

(WebInspector.NetworkDataGridNode.prototype._refreshInitiatorCell):

LayoutTests:

  • http/tests/inspector/network/network-initiator-from-console-expected.txt: Added.
  • http/tests/inspector/network/network-initiator-from-console.html: Added.
8:52 AM Changeset in webkit [92193] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

Web Inspector: REGRESSION(r92105): It made 2 tests fail
​https://bugs.webkit.org/show_bug.cgi?id=65525

  • platform/qt/Skipped: Unskip tests, because r92185 fixed the bug.
8:51 AM Changeset in webkit [92192] by apavlov@chromium.org
  • 5 edits in trunk

Web Inspector: img[width=90] or align attributes shouldnt be shown above element.style block (as they have lower specificity)
​https://bugs.webkit.org/show_bug.cgi?id=65514

Reviewed by Pavel Feldman.

Source/WebCore:

  • inspector/front-end/StylesSidebarPane.js:

LayoutTests:

  • inspector/elements/elements-panel-styles-expected.txt:
  • inspector/elements/elements-panel-styles.html:
8:25 AM Changeset in webkit [92191] by commit-queue@webkit.org
  • 3 edits in trunk/Tools

CSS Regions build bot should archive and upload output files
​https://bugs.webkit.org/show_bug.cgi?id=65200

Until now the buildbots would only upload the result archive if the 'trigger' parameters was specified. The
upload was needed, because the testbots needed to download and use the archive. For CSS Regions we don't have
a testbot yet, but we want to save the archive on the server for manual testing. Added new parameter
in config.json, called "upload" that when set to "true" will force the upload to the buildmaster server.

The "features" argument is now taken into account when determining the name of the archive on the server. That
was needed in order to avoid the CSS Regions Mac buildbot overwrite the SnowLeopard Mac build.

Patch by Alexandru Chiculita <Alexandru Chiculita> on 2011-08-02
Reviewed by Adam Roben.

  • BuildSlaveSupport/build.webkit.org-config/config.json:
  • BuildSlaveSupport/build.webkit.org-config/master.cfg:
8:18 AM Changeset in webkit [92190] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebKit/efl

[EFL] Remove overlapping recursive layout function
​https://bugs.webkit.org/show_bug.cgi?id=63241

First, _ewk_view_layout_if_needed_recursive is called twice in _ewk_view_smart_calculate
and _ewk_view_tiled_updates_process_pre,so it is no problem removing one of them.
Second, In tiled backingstore configuration, _ewk_view_layout_if_needed_recursive called
in _ewk_view_smart_calculate will result in generating repaints for dirty areas
located outside of viewport since m_paintEntireContents is set.
But, the areas will not be actually painted. Thus, the layout is unnecessary except
when there is something to paint inside of viewport, which will consume CPU unnecessarily.
So,the layout func was removed from _ewk_view_smart_calculate
and added to _ewk_view_single_smart_repaints_process for single backingstore.

Patch by Eunsol Park <​eunsol47.park@samsung.com> on 2011-08-02
Reviewed by Antonio Gomes.

  • ewk/ewk_view.cpp:

(_ewk_view_smart_calculate):

  • ewk/ewk_view_single.c:

(_ewk_view_single_smart_repaints_process):

8:08 AM Changeset in webkit [92189] by gyuyoung.kim@samsung.com
  • 7 edits
    3 adds in trunk/Source

[EFL] Add fullscreen button to media control UI.
​https://bugs.webkit.org/show_bug.cgi?id=64428

Source/WebCore:

Implement paintMediaFullscreenButton and emit fullscreen signal.

Reviewed by Antonio Gomes.

  • platform/efl/RenderThemeEfl.cpp:

(WebCore::RenderThemeEfl::edjeGroupFromFormType):
(WebCore::RenderThemeEfl::emitMediaButtonSignal):
(WebCore::RenderThemeEfl::paintMediaFullscreenButton):

  • platform/efl/RenderThemeEfl.h:

Source/WebKit/efl:

Implement functions for full screen in ChromeClientEfl in order to display a full screen button on media control UI.
When full screen mode is activated by pressing the button, the functions are called by Document. So, if there is no implemented
function, full screen button will not be shown.
In addition, a .edc file and an image file are added for full screen button.

Reviewed by Antonio Gomes.

  • DefaultTheme/default.edc:
  • DefaultTheme/widget/mediacontrol/fullscreenbutton/fullscreen_button.edc: Added.
  • DefaultTheme/widget/mediacontrol/fullscreenbutton/fullscreenbutton.png: Added.
  • WebCoreSupport/ChromeClientEfl.cpp:

(WebCore::ChromeClientEfl::supportsFullScreenForElement):
(WebCore::ChromeClientEfl::enterFullScreenForElement):
(WebCore::ChromeClientEfl::exitFullScreenForElement):

  • WebCoreSupport/ChromeClientEfl.h:
7:11 AM Changeset in webkit [92188] by Patrick Gansterer
  • 2 edits in trunk/Tools

Fix _win32_check_running_pid on 64bit windows python
​https://bugs.webkit.org/show_bug.cgi?id=64950

Reviewed by Adam Roben.

Use the correct types for the PROCESSENTRY32 struct.
th32DefaultHeapID is declared as ULONG_PTR and not as DWORD.
Using 32bit for pointers instead of 64bit results in an ERROR_BAD_LENGTH
failure, when calling the Process32First function.
This is covered by the executive unittest.

  • Scripts/webkitpy/common/system/executive.py:
7:01 AM Changeset in webkit [92187] by caseq@chromium.org
  • 5 edits
    2 adds in trunk

2011-08-01 Andrey Kosyakov <​caseq@chromium.org>

Web Inspector: [Extensions API] webInspector.inspectedPage.reload() is broken
​https://bugs.webkit.org/show_bug.cgi?id=65476

  • fixed webInspector.inspectedWindow.reload(), brought tests back.
  • removed redundant setting of user agent in FrameLoader::loadResourceSynchronously()

Reviewed by Yury Semikhatsky.

Test: http/tests/inspector/extensions-useragent.html

  • inspector/front-end/ExtensionServer.js: (WebInspector.ExtensionServer.prototype._onReload):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::loadResourceSynchronously):

2011-08-01 Andrey Kosyakov <​caseq@chromium.org>

Web Inspector: [Extensions API] webInspector.inspectedPage.reload() is broken
​https://bugs.webkit.org/show_bug.cgi?id=65476

Reviewed by Yury Semikhatsky.

  • http/tests/inspector/extensions-useragent-expected.txt: Added.
  • http/tests/inspector/extensions-useragent.html: Added.
6:58 AM Changeset in webkit [92186] by Csaba Osztrogonác
  • 2 edits in trunk/Source/WebKit/qt

[Qt] Fix tst_QDeclarativeWebView::basicProperties() and historyNav() autotests
​https://bugs.webkit.org/show_bug.cgi?id=61042
Remove mark as expected fail where unexpected pass and mark failing test case as expected fail

Patch by Kristóf Kosztyó <​Kosztyo.Kristof@stud.u-szeged.hu> on 2011-08-02
Reviewed by Csaba Osztrogonác.

  • tests/qdeclarativewebview/tst_qdeclarativewebview.cpp:

(tst_QDeclarativeWebView::basicProperties):
(tst_QDeclarativeWebView::historyNav):

6:54 AM Changeset in webkit [92185] by pfeldman@chromium.org
  • 6 edits
    1 copy in trunk

2011-08-02 Pavel Feldman <​pfeldman@chromium.org>

Not reviewed: fixing inspector tests.

  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype._addOptionToFilesSelect):

2011-08-02 Pavel Feldman <​pfeldman@chromium.org>

Not reviewed: fixing inspector tests.

  • inspector/debugger/scripts-panel-expected.txt:
  • inspector/debugger/scripts-panel.html:
  • inspector/debugger/scripts-sorting-expected.txt:
  • platform/chromium/inspector/debugger/scripts-sorting-expected.txt: Copied from LayoutTests/inspector/debugger/scripts-sorting-expected.txt.
6:43 AM Changeset in webkit [92184] by pfeldman@chromium.org
  • 2 edits in trunk/Source/WebCore

2011-08-02 Pavel Feldman <​pfeldman@google.com>

Not reviewed: one line inspector style change.

  • inspector/front-end/networkLogView.css: (.network-error-row, .network-error-row .network-cell-subtitle):
6:39 AM Changeset in webkit [92183] by commit-queue@webkit.org
  • 4 edits in trunk/Source

Custom cursors cause the WebProcess to crash
​https://bugs.webkit.org/show_bug.cgi?id=64802

Patch by Amruth Raj <​amruthraj@motorola.com> on 2011-08-02
Reviewed by Martin Robinson.

Source/WebCore:

  • WebCore.exp.in:

Source/WebKit2:

  • Shared/WebCoreArgumentCoders.cpp:

(CoreIPC::::encode): In case of a NULL cursor image, encode a bool indicating that.
(CoreIPC::::decode): Decode the image only if required.

5:38 AM Changeset in webkit [92182] by Csaba Osztrogonác
  • 2 edits in trunk/Source/WebKit/qt

[Qt] Fix tst_QWebPage::viewModes() API test
​https://bugs.webkit.org/show_bug.cgi?id=65531

Patch by Kristóf Kosztyó <​Kosztyo.Kristof@stud.u-szeged.hu> on 2011-08-02
Reviewed by Csaba Osztrogonác.

  • tests/qwebpage/tst_qwebpage.cpp: Mark failing test case as expected fail

(tst_QWebPage::viewModes):

5:34 AM Changeset in webkit [92181] by hamaji@chromium.org
  • 2 edits in trunk/LayoutTests

Malformed chromium test expectation fix, again. Sorry for the spam...

  • platform/chromium/test_expectations.txt:
5:13 AM Changeset in webkit [92180] by Csaba Osztrogonác
  • 2 edits in trunk/Source/WebKit/qt

[Qt] Fix tst_QWebFrame::connectAndDisconnect() API test
​https://bugs.webkit.org/show_bug.cgi?id=65170

Patch by Kristóf Kosztyó <​Kosztyo.Kristof@stud.u-szeged.hu> on 2011-08-02
Reviewed by Csaba Osztrogonác.

  • tests/qwebframe/tst_qwebframe.cpp: Mark failing test case as expected fail

(tst_QWebFrame::connectAndDisconnect):

5:13 AM Changeset in webkit [92179] by Csaba Osztrogonác
  • 2 edits in trunk/Source/WebKit/qt

[Qt] Fix tst_QWebElement::render() API test
​https://bugs.webkit.org/show_bug.cgi?id=65243

Patch by Kristóf Kosztyó <​Kosztyo.Kristof@stud.u-szeged.hu> on 2011-08-02
Reviewed by Csaba Osztrogonác.

  • tests/qwebelement/tst_qwebelement.cpp: Mark failing test case as expected fail

(tst_QWebElement::render):

5:02 AM Changeset in webkit [92178] by hamaji@chromium.org
  • 2 edits in trunk/LayoutTests

Fix malformed chromium test expectation introduced by r92173

  • platform/chromium/test_expectations.txt:
4:06 AM Changeset in webkit [92177] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt][Mac] Unreviewed gardening. Paint the Qt Snow Leopard bot green.

  • platform/qt-mac/Skipped: Skip a failing test.
4:00 AM Changeset in webkit [92176] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt] Unreviewed gardening.

  • platform/qt/Skipped: Add comment after r92175.
3:57 AM Changeset in webkit [92175] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt] Unreviewed gardening. Paint the Qt bot green.

  • platform/qt/Skipped: Skip failing tests.
3:49 AM Changeset in webkit [92174] by hamaji@chromium.org
  • 2 edits in trunk/LayoutTests

Chromium test_expectations.txt update for r92165

  • platform/chromium/test_expectations.txt:
2:50 AM Changeset in webkit [92173] by hamaji@chromium.org
  • 2 edits in trunk/LayoutTests

Layout Test fast/text/international/bidi-mirror-he-ar.html is failing
​https://bugs.webkit.org/show_bug.cgi?id=65524

Just update test_expectations.txt

  • platform/chromium/test_expectations.txt:
2:47 AM Changeset in webkit [92172] by Csaba Osztrogonác
  • 3 edits in trunk/LayoutTests

[Qt] Unreviewed gardening after r92127.

  • platform/qt/editing/selection/4397952-expected.png: Updated.
  • platform/qt/editing/selection/4397952-expected.txt: Updated.
2:25 AM Changeset in webkit [92171] by pfeldman@chromium.org
  • 2 edits in trunk/Source/WebCore

Web Inspector: _normalizeEventTypes (via console's monitorEvents) doesnt handle all events like 'search' and other atypical ones.
​https://bugs.webkit.org/show_bug.cgi?id=65468

Reviewed by Yury Semikhatsky.

  • inspector/InjectedScriptSource.js:
2:20 AM Changeset in webkit [92170] by pfeldman@chromium.org
  • 3 edits in trunk/Source/WebCore

Web Inspector: 404/errors should get a RED visual treatment so easier to identify.
​https://bugs.webkit.org/show_bug.cgi?id=65471

Reviewed by Yury Semikhatsky.

  • inspector/front-end/NetworkPanel.js:

(WebInspector.NetworkDataGridNode.prototype._refreshStatusCell):

  • inspector/front-end/networkLogView.css:

(.network-error-cell):

1:46 AM Changeset in webkit [92169] by Csaba Osztrogonác
  • 2 edits in trunk/Source/WebKit/qt

[Qt] tst_QWebElement::style() fails because QWebElement::CascadedStyle doesn't work as expected
​https://bugs.webkit.org/show_bug.cgi?id=65244

Patch by Kristóf Kosztyó <​Kosztyo.Kristof@stud.u-szeged.hu> on 2011-08-02
Reviewed by Csaba Osztrogonác.

  • tests/qwebelement/tst_qwebelement.cpp: Mark failing test case as expected fail

(tst_QWebElement::style):

1:05 AM Changeset in webkit [92168] by pfeldman@chromium.org
  • 5 edits in trunk/Source/WebCore

Web Inspector: arrow is a pain to click in console. Drive-by selection reset fix.
​https://bugs.webkit.org/show_bug.cgi?id=65467

Reviewed by Yury Semikhatsky.

  • inspector/front-end/ConsoleView.js:

(WebInspector.ConsoleView.prototype._messagesClicked):

  • inspector/front-end/ObjectPropertiesSection.js:

(WebInspector.ObjectPropertyTreeElement):

  • inspector/front-end/inspector.css:

(.properties-tree li):
(.properties-tree li.parent::before):

  • inspector/front-end/treeoutline.js:

(TreeElement.treeElementToggled):

1:03 AM Changeset in webkit [92167] by pfeldman@chromium.org
  • 3 edits
    2 adds in trunk

Web Inspector: button + "New style rule"
​https://bugs.webkit.org/show_bug.cgi?id=65409

Reviewed by Yury Semikhatsky.

Source/WebCore:

Test: inspector/styles/styles-add-new-rule.html

  • inspector/front-end/StylesSidebarPane.js:

(WebInspector.StylesSidebarPane):
(WebInspector.BlankStylePropertiesSection.prototype.editingSelectorCommitted):

LayoutTests:

  • inspector/styles/styles-add-new-rule-expected.txt: Added.
  • inspector/styles/styles-add-new-rule.html: Added.
Note: See TracTimeline for information about the timeline view.