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

Timeline



Apr 21, 2011:

11:46 PM Changeset in webkit [84606] by commit-queue@webkit.org
  • 8 edits
    1 copy
    9 adds in trunk/LayoutTests

2011-04-21 Chris Weber <​chris@lookout.net>

Reviewed by Adam Barth.

Add more test coverage to fast/url
​https://bugs.webkit.org/show_bug.cgi?id=59142

The PASS/FAIL results for host, path, and port are set to match
Chromium. The PASS/FAIL results for idna2003 and idna2008 are set to
match those specifications.

  • fast/url/host-expected.txt:
  • fast/url/idna2003-expected.txt: Added.
  • fast/url/idna2003.html: Added.
  • fast/url/idna2008-expected.txt: Added.
  • fast/url/idna2008.html: Added.
  • fast/url/path-expected.txt:
  • fast/url/port-expected.txt:
  • fast/url/script-tests/host.js:
  • fast/url/script-tests/idna2003.js: Added.
  • fast/url/script-tests/idna2008.js: Added.
  • fast/url/script-tests/path.js:
  • fast/url/script-tests/port.js:
11:37 PM Changeset in webkit [84605] by tony@chromium.org
  • 1 edit
    1 delete in trunk/Tools

2011-04-21 Tony Chang <​tony@chromium.org>

Reviewed by Mihai Parparita.

Remove find-mismatched-layout-test-results since it's not needed in a
checksum free world.
​https://bugs.webkit.org/show_bug.cgi?id=59156

  • Scripts/find-mismatched-layout-test-results: Removed.
11:29 PM Changeset in webkit [84604] by Beth Dakin
  • 3 edits
    2 adds in trunk

​https://bugs.webkit.org/show_bug.cgi?id=57898
REGRESSION (r82185): Scroll position not restored on navigation back to a page in
the page cache
-and corresponding-
<rdar://problem/9226652>

Reviewed by Dan Bernstein.

Source/WebCore:

The original fix for this change broke scroll-position restore for pages that are
*not* in the page cache. This patch fixes both cached and non-cached pages by
using the cachedScrollPosition() only for cached pages.

  • loader/HistoryController.cpp:

(WebCore::HistoryController::saveScrollPositionAndViewStateToItem):

LayoutTests:

  • fast/loader/scroll-position-restored-on-back-non-cached-expected.txt: Added.
  • fast/loader/scroll-position-restored-on-back-non-cached.html: Added.
11:13 PM Changeset in webkit [84603] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

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

Reviewed by Eric Seidel.

Remove redundant calls to RenderStyle::setDisplay()
​https://bugs.webkit.org/show_bug.cgi?id=59060

No new tests / no new functionality added.

  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::applyProperty): Remove redundant calls to RenderStyle::setDisplay()
11:08 PM Changeset in webkit [84602] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

2011-04-21 John Bauman <​jbauman@chromium.org>

Reviewed by Kenneth Russell.

WebGL context drawing buffer may not be cleared if FBO bound
​https://bugs.webkit.org/show_bug.cgi?id=58956

Clear the drawing buffer even if it's not currently bound, by
switching between FBOs around the clear. Also, make sure that we
handle depth and stencil masks properly.

No new tests, because testing this under DRT doesn't work.

  • html/canvas/WebGLRenderingContext.cpp: (WebCore::WebGLRenderingContext::initializeNewContext): (WebCore::WebGLRenderingContext::clearIfComposited): (WebCore::WebGLRenderingContext::depthMask):
  • html/canvas/WebGLRenderingContext.h:
10:37 PM Changeset in webkit [84601] by bfulgham@webkit.org
  • 4 edits in trunk/Tools

Teach build-webkit and update-webkit that the WinCairo port does
not need the non-redistributable WebKitSupportLibraries.zip file.
Also append the _Cairo_CFLite label to the build target for the
default (no argument) case when building with '--wincairo'

Reviewed by Martin Robinson.

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

  • Scripts/build-webkit: Don't die if WebKitSupportLibraries.zip doesn't exist.
  • Scripts/webkitdirs.pm: Append '_Cairo_CFLite' to the default 'Release' build target when passed the '--wincairo' flag.
  • Scripts/update-webkit: Don't die if WebKitSupportLibraries.zip doesn't exist.
10:01 PM Changeset in webkit [84600] by Chris Fleizach
  • 8 edits in trunk

WebKit is erroneously modifying the DOM; adding 'aria-selected' to tables
​https://bugs.webkit.org/show_bug.cgi?id=59140

Reviewed by Beth Dakin.

Source/WebCore:

WebKit should not modify the DOM when the screen reader requests certain things be set. There is no
specification that allows this behavior yet, but we are working on something for the future.

  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::setARIAGrabbed):
(WebCore::AccessibilityRenderObject::setIsExpanded):
(WebCore::AccessibilityRenderObject::setSelected):
(WebCore::AccessibilityRenderObject::setValue):

LayoutTests:

  • accessibility/aria-combobox.html:
  • platform/mac/accessibility/aria-combobox-expected.txt:
  • platform/mac/accessibility/aria-slider-value-change-expected.txt:
  • platform/mac/accessibility/aria-slider-value-change.html:
9:54 PM Changeset in webkit [84599] by commit-queue@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

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

Unreviewed, rolling out r84583.
​http://trac.webkit.org/changeset/84583
​https://bugs.webkit.org/show_bug.cgi?id=59173

"broke
​http://trac.webkit.org/export/84593/trunk/LayoutTests/fast/js
/Object-create.html" (Requested by ggaren on #webkit).

  • runtime/ObjectConstructor.cpp: (JSC::objectConstructorCreate):
9:26 PM Changeset in webkit [84598] by ap@apple.com
  • 2 edits in trunk/Source/WebKit2

Minimize synchronous calls during loading
​https://bugs.webkit.org/show_bug.cgi?id=59169

<rdar://problem/9320128> REGRESSION(r84526): WK2 windows unresponsive

  • UIProcess/API/mac/WKView.mm: (-[WKView resignFirstResponder]): (-[WKView _resetTextInputState]): Made _resetTextInputState deal with client side state only, letting Editor composition be discarded more discriminately.
8:45 PM Changeset in webkit [84597] by dpranke@chromium.org
  • 2 edits in trunk/LayoutTests

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

Unreviewed, expectations change - new baselines needed as
a result of r84589.

  • platform/chromium/test_expectations.txt:
8:29 PM Changeset in webkit [84596] by dpranke@chromium.org
  • 3 edits
    2 moves in trunk/Tools

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

Reviewed by Eric Seidel.

new-run-webkit-tests: obsolete old code, part 5: rename message_broker2 to message_broker
​https://bugs.webkit.org/show_bug.cgi?id=58758

Since worker_mixin is now only used by worker, there's no point
in having it split across two files.

  • Scripts/webkitpy/layout_tests/layout_package/manager_worker_broker.py:
  • Scripts/webkitpy/layout_tests/layout_package/manager_worker_broker_unittest.py:
  • Scripts/webkitpy/layout_tests/layout_package/message_broker.py: Added
  • Scripts/webkitpy/layout_tests/layout_package/message_broker2.py: Removed.
  • Scripts/webkitpy/layout_tests/layout_package/message_broker_unittest.py: * Added
  • Scripts/webkitpy/layout_tests/layout_package/message_broker2_unittest.py: Removed.
8:28 PM Changeset in webkit [84595] by dpranke@chromium.org
  • 2 edits
    1 delete in trunk/Tools

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

Reviewed by Tony Chang.

new-run-webkit-tests: obsolete old code, part 4: merge worker_mixin into worker
​https://bugs.webkit.org/show_bug.cgi?id=58756

Since worker_mixin is now only used by worker, there's no point
in having it split across two files.

  • Scripts/webkitpy/layout_tests/layout_package/worker.py:
  • Scripts/webkitpy/layout_tests/layout_package/worker_mixin.py: Removed.
8:26 PM Changeset in webkit [84594] by dpranke@chromium.org
  • 3 edits
    2 moves in trunk/Tools

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

Reviewed by Tony Chang.

new-run-webkit-tests: obsolete old code, part 3 - rename test_runner to manager
​https://bugs.webkit.org/show_bug.cgi?id=58754

test_runner hasn't been a great name for some time, since all of
the test running is actually done in worker and
single_test_runner. Rename to manager to match the terminology
elsewhere.

  • Scripts/webkitpy/layout_tests/layout_package/manager.py: Added.
  • Scripts/webkitpy/layout_tests/layout_package/manager_unittest.py: Added.
  • Scripts/webkitpy/layout_tests/layout_package/printing_unittest.py:
  • Scripts/webkitpy/layout_tests/layout_package/test_runner.py: Removed.
  • Scripts/webkitpy/layout_tests/layout_package/test_runner_unittest.py: Removed.
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
8:13 PM Changeset in webkit [84593] by Chris Fleizach
  • 4 edits in trunk/Source

AX: order AccessibilityRole alphabetically
​https://bugs.webkit.org/show_bug.cgi?id=31524

Reviewed by Beth Dakin.

Source/WebCore:

  • accessibility/AccessibilityObject.h:

Source/WebKit/chromium:

  • public/WebAccessibilityRole.h:
7:49 PM Changeset in webkit [84592] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebCore

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

Reviewed by Maciej Stachowiak.

[Mac] correctionPanelTimerFired() crashes due to rangeToBeReplaced being cleared.
​https://bugs.webkit.org/show_bug.cgi?id=59155
<rdar://problem/9261698>

Make sure the range isn't cleared before calling windowRectForRange() on it. Also removed unused functions in Editor.h,
and fixed a typo in function name.

  • editing/Editor.h:
  • editing/SpellingCorrectionController.cpp: (WebCore::SpellingCorrectionController::correctionPanelTimerFired): (WebCore::SpellingCorrectionController::respondToChangedSelection):
  • editing/SpellingCorrectionController.h: (WebCore::SpellingCorrectionController::shouldStartTimerFor):
7:48 PM Changeset in webkit [84591] by dpranke@chromium.org
  • 3 edits
    1 delete in trunk/Tools

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

Reviewed by Tony Chang.

new-run-webkit-tests: obsolete old threading code, part 2
​https://bugs.webkit.org/show_bug.cgi?id=58753

This merges the test_runner2 code back into test_runner since
there's only a single type of test runner now.

  • Scripts/webkitpy/layout_tests/layout_package/test_runner.py:
  • Scripts/webkitpy/layout_tests/layout_package/test_runner2.py: Removed.
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
7:41 PM Changeset in webkit [84590] by dpranke@chromium.org
  • 2 edits in trunk/LayoutTests

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

Unreviewed, expectations change. Triage failure for
media/video-zoom.html on chromium gpu bots.

  • platform/chromium/test_expectations.txt:
7:30 PM Changeset in webkit [84589] by mjs@apple.com
  • 34 edits in trunk

2011-04-21 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Adam Roben.

Add a feature define to allow <details> and <summary> to be disabled
​https://bugs.webkit.org/show_bug.cgi?id=59118
<rdar://problem/9257045>

  • configure.ac:

2011-04-21 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Adam Roben.

Add a feature define to allow <details> and <summary> to be disabled
​https://bugs.webkit.org/show_bug.cgi?id=59118
<rdar://problem/9257045>

  • Configurations/FeatureDefines.xcconfig:

2011-04-21 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Adam Roben.

Add a feature define to allow <details> and <summary> to be disabled
​https://bugs.webkit.org/show_bug.cgi?id=59118
<rdar://problem/9257045>

  • features.gypi:

2011-04-21 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Adam Roben.

Add a feature define to allow <details> and <summary> to be disabled
​https://bugs.webkit.org/show_bug.cgi?id=59118
<rdar://problem/9257045>

  • Configurations/FeatureDefines.xcconfig:

2011-04-21 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Adam Roben.

Add a feature define to allow <details> and <summary> to be disabled
​https://bugs.webkit.org/show_bug.cgi?id=59118
<rdar://problem/9257045>

No new tests; no behavior change.

  • Configurations/FeatureDefines.xcconfig:
  • DerivedSources.make:
  • GNUmakefile.am:
  • features.pri:
  • html/HTMLDetailsElement.cpp:
  • html/HTMLSummaryElement.cpp:
  • html/HTMLTagNames.in:
  • html/shadow/DetailsMarkerControl.cpp:
  • rendering/RenderBlock.cpp: (WebCore::canMergeContiguousAnonymousBlocks):
  • rendering/RenderDetails.cpp:
  • rendering/RenderDetails.h:
  • rendering/RenderDetailsMarker.cpp:
  • rendering/RenderDetailsMarker.h:
  • rendering/RenderObject.h:
  • rendering/RenderSummary.cpp:
  • rendering/RenderSummary.h:
  • rendering/RenderTreeAsText.cpp: (WebCore::RenderTreeAsText::writeRenderObject):

2011-04-21 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Adam Roben.

Add a feature define to allow <details> and <summary> to be disabled
​https://bugs.webkit.org/show_bug.cgi?id=59118
<rdar://problem/9257045>

  • Configurations/FeatureDefines.xcconfig:

2011-04-21 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Adam Roben.

Add a feature define to allow <details> and <summary> to be disabled
​https://bugs.webkit.org/show_bug.cgi?id=59118
<rdar://problem/9257045>

  • Scripts/build-webkit:
  • Tools.pro:

2011-04-21 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Adam Roben.

Add a feature define to allow <details> and <summary> to be disabled
​https://bugs.webkit.org/show_bug.cgi?id=59118
<rdar://problem/9257045>


  • win/tools/vsprops/FeatureDefines.vsprops:
  • win/tools/vsprops/FeatureDefinesCairo.vsprops:
7:29 PM Changeset in webkit [84588] by dpranke@chromium.org
  • 2 edits in trunk/LayoutTests

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

Unreviewed, expectations change. Suppress failures on chromium
win/linux from bug 59166.

  • platform/chromium/test_expectations.txt:
7:01 PM Changeset in webkit [84587] by Dimitri Glazkov
  • 5 edits in trunk

2011-04-21 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Adele Peterson.

REGRESSION (r83386): Typing in WSJ.com search field doesn't clear placeholder text
​https://bugs.webkit.org/show_bug.cgi?id=59053

  • fast/events/shadow-boundary-crossing-expected.txt: Updated expected results.
  • fast/events/shadow-boundary-crossing.html: Added new sub-test.

2011-04-21 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Adele Peterson.

REGRESSION (r83386): Typing in WSJ.com search field doesn't clear placeholder text
​https://bugs.webkit.org/show_bug.cgi?id=59053

Turns out, the relatedTarget of an event could be also an ancestor of an event,
not just its parent. Duh.

  • dom/EventDispatcher.cpp: (WebCore::EventDispatcher::adjustToShadowBoundaries): Changed to use shadowHost

explicitly.

6:42 PM Changeset in webkit [84586] by msaboff@apple.com
  • 5 edits in trunk/Source/WebCore

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

Reviewed by Maciej Stachowiak.

Qualified names used for all TagName access, yet namespace usage is rare
​https://bugs.webkit.org/show_bug.cgi?id=58997

The methods getElementsByTagName and getElementsByTagNameNS where
always creating and using QualifiedNames. QualifiedName::init
was consistently in the top 3 routines when running the Dromaeo
DOM-query benchmark. Split out the functionality so that
getElementsByTagName uses just the local name, an implied "*"
namespace and a separate TagNodeListCache keyed by an atomic name
instead of a QualifiedName. Access to elements via
getElementsByTagNameNS that have "*" namespace are forwarded to
getElementsByTagName as well. This provides ~10% speed up in that
Dromaeo test.

No new tests added, existing tests have coverage. The changes are
an optimization of existing functionality.

  • dom/Node.cpp: (WebCore::Node::removeCachedTagNodeList): (WebCore::Node::getElementsByTagName): (WebCore::Node::getElementsByTagNameNS): (WebCore::NodeListsNodeData::invalidateCaches): (WebCore::NodeListsNodeData::isEmpty):
  • dom/Node.h:
  • dom/NodeRareData.h:
  • dom/TagNodeList.cpp: (WebCore::TagNodeList::~TagNodeList):
6:31 PM Changeset in webkit [84585] by rniwa@webkit.org
  • 3 edits in trunk/Source/WebKit/wince

2011-04-21 Ryosuke Niwa <​rniwa@webkit.org>

WinCE build fix after r84574.

  • WebCoreSupport/EditorClientWinCE.cpp: (WebKit::EditorClientWinCE::canCopyCut): (WebKit::EditorClientWinCE::canPaste):
  • WebCoreSupport/EditorClientWinCE.h:
6:30 PM Changeset in webkit [84584] by ggaren@apple.com
  • 16 edits in trunk/Source/WebCore

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

Reviewed by Oliver Hunt.

Removed a use of markDOMObjectWrapper: more style- and node-related stuff
​https://bugs.webkit.org/show_bug.cgi?id=59127

  • bindings/js/JSCanvasRenderingContextCustom.cpp:
  • bindings/js/JSCanvasRenderingContextCustom.cpp: (WebCore::JSCanvasRenderingContextOwner::isReachableFromOpaqueRoots): (WebCore::JSCanvasRenderingContextOwner::finalize): (WebCore::wrapperOwner): (WebCore::wrapperContext): Use opaque roots for marking canvas rendering contexts, so JSNode isn't responsible for it.
  • bindings/js/JSElementCustom.cpp: (WebCore::JSElement::markChildren):
  • bindings/js/JSHTMLCanvasElementCustom.cpp:
  • bindings/js/JSHTMLLinkElementCustom.cpp:
  • bindings/js/JSHTMLStyleElementCustom.cpp: Removed obsolete marking code.
  • bindings/js/JSNodeCustom.cpp: (WebCore::isObservable): (WebCore::isReachableFromDOM): (WebCore::JSNodeOwner::isReachableFromOpaqueRoots): Removed a bunch of CSS object model related special cases, since the CSS object model now tracks its own lifetime through the opaque roots system. Removed canvas special case, covered above.
  • bindings/js/JSProcessingInstructionCustom.cpp:
  • bindings/js/JSWebKitAnimationListCustom.cpp:
  • dom/ProcessingInstruction.idl:
  • html/HTMLCanvasElement.idl:
  • html/HTMLLinkElement.idl:
  • html/HTMLStyleElement.idl:
  • page/WebKitAnimationList.idl: Removed obsolete marking code.
6:30 PM Changeset in webkit [84583] by oliver@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

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

Reviewed by Geoffrey Garen.

Object.create creates uncachable objects
​https://bugs.webkit.org/show_bug.cgi?id=59164

Use the prototype object's inheritorID, as we
should always have done

  • runtime/ObjectConstructor.cpp: (JSC::objectConstructorCreate):
6:05 PM Changeset in webkit [84582] by vangelis@chromium.org
  • 15 edits in trunk/Source/WebCore

2011-04-21 Vangelis Kokkevis <​vangelis@chromium.org>

Reviewed by James Robinson.

[chromium] Only update the contents of layers that are going to
be drawn. Remove individual texture unreserves that happen
after the layer draw calls with a single unreserve for all the
textures in the texture manager.
​https://bugs.webkit.org/show_bug.cgi?id=59129

Tests: No new functionality. Covered by existing layout tests.

  • platform/graphics/chromium/ContentLayerChromium.cpp: (WebCore::ContentLayerChromium::draw):
  • platform/graphics/chromium/ContentLayerChromium.h:
  • platform/graphics/chromium/ImageLayerChromium.cpp: (WebCore::ImageLayerChromium::paintContentsIfDirty): (WebCore::ImageLayerChromium::updateCompositorResources):
  • platform/graphics/chromium/LayerRendererChromium.cpp: (WebCore::LayerRendererChromium::drawRootLayer): (WebCore::LayerRendererChromium::updateAndDrawLayers): (WebCore::LayerRendererChromium::updateLayers): (WebCore::LayerRendererChromium::paintLayerContents):
  • platform/graphics/chromium/LayerRendererChromium.h:
  • platform/graphics/chromium/LayerTexture.cpp:
  • platform/graphics/chromium/LayerTexture.h:
  • platform/graphics/chromium/LayerTilerChromium.cpp:
  • platform/graphics/chromium/LayerTilerChromium.h:
  • platform/graphics/chromium/RenderSurfaceChromium.cpp: (WebCore::RenderSurfaceChromium::drawSurface):
  • platform/graphics/chromium/TextureManager.cpp: (WebCore::TextureManager::unprotectAllTextures):
  • platform/graphics/chromium/TextureManager.h:
  • platform/graphics/chromium/cc/CCHeadsUpDisplay.cpp: (WebCore::CCHeadsUpDisplay::draw):
  • platform/graphics/chromium/cc/CCLayerImpl.h: (WebCore::CCLayerImpl::owner):
5:50 PM Changeset in webkit [84581] by Lucas Forschler
  • 14 edits in tags/Safari-534.30/Source/WebKit2

Rollout 84526.

5:49 PM Changeset in webkit [84580] by ike@apple.com
  • 2 edits in trunk/Source/WebKit2

<rdar://problem/9311103>

Reviewed by Maciej Stachowiak.

Remove unnecessary QTKitServer and Apple Events permissions
from the WebProcess sandbox.

  • WebProcess/com.apple.WebProcess.sb:
5:45 PM Changeset in webkit [84579] by Lucas Forschler
  • 5 edits in trunk/Source

Versioning.

5:43 PM Changeset in webkit [84578] by rniwa@webkit.org
  • 2 edits in trunk/Source/WebKit/win

2011-04-21 Ryosuke Niwa <​rniwa@webkit.org>

Windows build fix.

  • WebCoreSupport/WebEditorClient.h:
5:42 PM Changeset in webkit [84577] by ike@apple.com
  • 2 edits in trunk/Source/WebKit2

<rdar://problem/9033535>

Reviewed by Maciej Stachowiak.

Mute sandbox violations when attempting to read a temporary
CFNetwork keychain.

  • WebProcess/com.apple.WebProcess.sb:
5:38 PM Changeset in webkit [84576] by Lucas Forschler
  • 1 copy in tags/Safari-534.30

New tag.

5:34 PM Changeset in webkit [84575] by abarth@webkit.org
  • 5 edits in trunk/Tools

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

Reviewed by Eric Seidel.

webkit-patch upload should assign the bug to you
​https://bugs.webkit.org/show_bug.cgi?id=33699

This patch only re-assigns the bug if it is currently unassigned. We
can be more aggressive in the future if people want us to be more
agressive.

  • Scripts/webkitpy/common/net/bugzilla/bugzilla.py:
  • Scripts/webkitpy/tool/commands/upload_unittest.py:
  • Scripts/webkitpy/tool/mocktool.py:
  • Scripts/webkitpy/tool/steps/postdiff.py:
5:33 PM Changeset in webkit [84574] by rniwa@webkit.org
  • 31 edits in trunk/Source

2011-04-21 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Sam Weinig.

Add Frame* to the argument lists of canCopyCut and canPaste
​https://bugs.webkit.org/show_bug.cgi?id=59153

  • src/EditorClientImpl.cpp: (WebKit::EditorClientImpl::canCopyCut): (WebKit::EditorClientImpl::canPaste):
  • src/EditorClientImpl.h:

2011-04-21 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Sam Weinig.

Add Frame* to the argument lists of canCopyCut and canPaste
​https://bugs.webkit.org/show_bug.cgi?id=59153

  • WebCoreSupport/EditorClientQt.cpp: (WebCore::EditorClientQt::canCopyCut): (WebCore::EditorClientQt::canPaste):
  • WebCoreSupport/EditorClientQt.h:

2011-04-21 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Sam Weinig.

Add Frame* to the argument lists of canCopyCut and canPaste
​https://bugs.webkit.org/show_bug.cgi?id=59153

  • WebCoreSupport/EditorClientGtk.cpp: (WebKit::EditorClient::canCopyCut): (WebKit::EditorClient::canPaste):
  • WebCoreSupport/EditorClientGtk.h:

2011-04-21 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Sam Weinig.

Add Frame* to the argument lists of canCopyCut and canPaste
​https://bugs.webkit.org/show_bug.cgi?id=59153

  • WebCoreSupport/WebEditorClient.cpp: (WebEditorClient::canCopyCut): (WebEditorClient::canPaste):
  • WebCoreSupport/WebEditorClient.h:

2011-04-21 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Sam Weinig.

Add Frame* to the argument lists of canCopyCut and canPaste
​https://bugs.webkit.org/show_bug.cgi?id=59153

  • WebKitSupport/EditorClientWx.cpp: (WebCore::EditorClientWx::canCopyCut): (WebCore::EditorClientWx::canPaste):
  • WebKitSupport/EditorClientWx.h:

2011-04-21 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Sam Weinig.

Add Frame* to the argument lists of canCopyCut and canPaste
​https://bugs.webkit.org/show_bug.cgi?id=59153

  • WebCoreSupport/WebEditorClient.h:
  • WebCoreSupport/WebEditorClient.mm: (WebEditorClient::canCopyCut): (WebEditorClient::canPaste):

2011-04-21 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Sam Weinig.

Add Frame* to the argument lists of canCopyCut and canPaste
​https://bugs.webkit.org/show_bug.cgi?id=59153

  • WebCoreSupport/EditorClientHaiku.cpp: (WebCore::EditorClientHaiku::canCopyCut): (WebCore::EditorClientHaiku::canPaste):
  • WebCoreSupport/EditorClientHaiku.h:

2011-04-21 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Sam Weinig.

Add Frame* to the argument lists of canCopyCut and canPaste
​https://bugs.webkit.org/show_bug.cgi?id=59153

  • WebCoreSupport/EditorClientEfl.cpp: (WebCore::EditorClientEfl::canCopyCut): (WebCore::EditorClientEfl::canPaste):
  • WebCoreSupport/EditorClientEfl.h:

2011-04-21 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Sam Weinig.

Add Frame* to the argument lists of canCopyCut and canPaste
​https://bugs.webkit.org/show_bug.cgi?id=59153

canCopyCut and canPaste now takes Frame* for which supportedCopyCut and supportedPaste are called.
This allows editor client to grant copy/cut and paste permissions per origin.

  • editing/EditorCommand.cpp: (WebCore::supportedCopyCut): (WebCore::supportedPaste):
  • loader/EmptyClients.h: (WebCore::EmptyEditorClient::canCopyCut): (WebCore::EmptyEditorClient::canPaste):
  • page/EditorClient.h:

2011-04-21 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Sam Weinig.

Add Frame* to the argument lists of canCopyCut and canPaste
​https://bugs.webkit.org/show_bug.cgi?id=59153

  • WebProcess/WebCoreSupport/WebEditorClient.cpp: (WebKit::WebEditorClient::canCopyCut): (WebKit::WebEditorClient::canPaste):
  • WebProcess/WebCoreSupport/WebEditorClient.h:
5:22 PM Changeset in webkit [84573] by dpranke@chromium.org
  • 1 edit
    13 adds in trunk/LayoutTests

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

Unreviwed, expecations change. Pick up chromium-win baselines
for r84522.

  • platform/chromium-win/svg/dynamic-updates/SVGFEDropShadowElement-dom-dx-attr-expected.png: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDropShadowElement-dom-dy-attr-expected.png: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDropShadowElement-dom-in-attr-expected.png: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDropShadowElement-dom-shadow-color-attr-expected.png: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDropShadowElement-dom-shadow-opacity-attr-expected.png: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDropShadowElement-dom-stdDeviation-attr-expected.png: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-dx-prop-expected.png: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-dy-prop-expected.png: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-in-prop-expected.png: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-shadow-color-prop-expected.png: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-shadow-opacity-prop-expected.png: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-stdDeviation-prop-expected.png: Added.
  • platform/chromium-win/svg/filters/feDropShadow-expected.png: Added.
5:17 PM Changeset in webkit [84572] by dpranke@chromium.org
  • 2 edits in trunk/LayoutTests

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

Unreviewed, expectations change.

Suppress failures possibly caused by bug 59158 on chromium win/linux:
fast/canvas/canvas-stroke{Path,Rect}-alpha-shadow.html = TEXT

  • platform/chromium/test_expectations.txt:
5:10 PM Changeset in webkit [84571] by ike@apple.com
  • 2 edits in trunk/Source/WebKit2

<rdar://problem/8774116>

Reviewed by Maciej Stachowiak.

The WebProcess sandbox should only allow access to the
parts of the Darwin cache directory needed by the Security
framework.

  • WebProcess/com.apple.WebProcess.sb:
5:06 PM Changeset in webkit [84570] by ike@apple.com
  • 2 edits in trunk/Source/WebKit2

<rdar://problem/9311053>

Reviewed by Maciej Stachowiak.

The WebProcess sandbox should not allow opening arbitrary
IOKit user clients. Restrict to a whitelist required for normal
WebProcess operation, including sound and accelerated graphics.

  • WebProcess/com.apple.WebProcess.sb:
5:03 PM Changeset in webkit [84569] by ike@apple.com
  • 2 edits in trunk/Source/WebKit2

<rdar://problem/9311037>

Reviewed by Maciej Stachowiak.

The WebProcess sandbox should not allow modification of the
CoreAudio component cache.

  • WebProcess/com.apple.WebProcess.sb:
4:58 PM Changeset in webkit [84568] by ojan@chromium.org
  • 3 edits
    2 moves
    1 add in trunk

fix

4:58 PM Changeset in webkit [84567] by ike@apple.com
  • 2 edits in trunk/Source/WebKit2

<rdar://problem/9263428>

Reviewed by Maciej Stachowiak.

The WebProcess sandbox should only allow outbound networking
to remote destinations and local kernel controls.

  • WebProcess/com.apple.WebProcess.sb:
4:54 PM Changeset in webkit [84566] by oliver@apple.com
  • 3 edits in trunk/Source/WebCore

Whoops, change lost during merge

4:49 PM Changeset in webkit [84565] by dpranke@chromium.org
  • 1 edit
    1 add in trunk/LayoutTests

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

Update baselines to accomodate the change in r84506. I wonder
how much having platform/mac in the search path hurts
chromium-win/-linux.

  • platform/chromium-win/canvas/philip/tests/toDataURL.jpeg.alpha-expected.txt: Added.
4:44 PM Changeset in webkit [84564] by mdelaney@apple.com
  • 1 edit
    1 move in trunk/LayoutTests

2011-04-21 Matthew Delaney <​mdelaney@apple.com>

Reviewed by Simon Fraser.

Move canvas test set-colors.html expected results out of platform/mac-leopard and into platform/mac
This test expected result pertains to all of mac, not just leopard.
​https://bugs.webkit.org/show_bug.cgi?id=59139

  • platform/mac-leopard/fast/canvas/set-colors-expected.txt: Removed.
  • platform/mac/fast/canvas/set-colors-expected.txt: Copied from LayoutTests/platform/mac-leopard/fast/canvas/set-colors-expected.txt.
4:36 PM Changeset in webkit [84563] by tony@chromium.org
  • 22 edits in trunk/LayoutTests

2011-04-21 Tony Chang <​tony@chromium.org>

Embed checksums in more png files. Not sure how these were missed earlier.

  • animations/missing-values-first-keyframe-expected.png:
  • animations/missing-values-last-keyframe-expected.png:
  • animations/opacity-transform-animation-expected.png:
  • animations/suspend-transform-animation-expected.png:
  • compositing/backface-visibility-expected.png:
  • compositing/checkerboard-expected.png:
  • compositing/geometry/ancestor-overflow-change-expected.png:
  • compositing/geometry/tall-page-composited-expected.png:
  • compositing/iframes/composited-iframe-scroll-expected.png:
  • compositing/iframes/iframe-content-flipping-expected.png:
  • compositing/iframes/nested-iframe-scrolling-expected.png:
  • compositing/images/direct-pdf-image-expected.png:
  • compositing/images/direct-svg-image-expected.png:
  • compositing/overflow/zero-size-overflow-expected.png:
  • compositing/reflections/backface-hidden-reflection-expected.png:
  • compositing/reflections/remove-add-reflection-expected.png:
  • compositing/tiling/constrained-layer-size-expected.png:
  • compositing/z-order/negative-z-index-expected.png:
  • transforms/3d/general/cssmatrix-3d-zoom-expected.png:
  • transforms/3d/general/matrix-with-zoom-3d-expected.png:
  • transitions/suspend-transform-transition-expected.png:
4:31 PM April 2011 Meeting edited by andreas.kling@nokia.com
(diff)
4:29 PM Changeset in webkit [84562] by dpranke@chromium.org
  • 2 edits in trunk/LayoutTests

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

Reviewed by NOBODY (OOPS!).

Unreviewed, expectations change - update chromium SVG baselines
after r84522.

  • platform/chromium/fast/dom/prototype-inheritance-expected.txt:
4:24 PM Changeset in webkit [84561] by commit-queue@webkit.org
  • 4 edits in trunk/Source

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

Unreviewed, rolling out r82052.
​http://trac.webkit.org/changeset/82052
​https://bugs.webkit.org/show_bug.cgi?id=59135

New full screen api should be opt-in, not opt-out. (Requested
by jernoble on #webkit).

  • WebView/WebPreferences.mm: (+[WebPreferences initialize]):

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

Unreviewed, rolling out r82052.
​http://trac.webkit.org/changeset/82052
​https://bugs.webkit.org/show_bug.cgi?id=59135

New full screen api should be opt-in, not opt-out. (Requested
by jernoble on #webkit).

  • Shared/WebPreferencesStore.h:
4:24 PM Changeset in webkit [84560] by ojan@chromium.org
  • 3 edits in trunk/Tools

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

Reviewed by Eric Seidel.

ValidateChangeLogs doesn't work on SVN
​https://bugs.webkit.org/show_bug.cgi?id=59115

svn-create-patch lies about the diff in the working copy by moving
ChangeLog entries to the top of the diff. That's fine on most cases,
but causes problems for ValidateChangeLogs, which is trying validate
the where the ChangeLog entry appears.

I haven't added a test for this change because I couldn't figure out
how to write one. The issue is more of an integration issue, which
we're not really set up to test in our unit testing framework. If this
patch had worked around the output from svn-create-patch, then I could
have tested that we behave correctly on sample svn-create-patch output,
but, in this case, I've removed the dependency on svn-create-patch. I
could test that we behave correctly on "svn diff" output, but we
already have those tests.

  • Scripts/webkitpy/tool/commands/download_unittest.py:
  • Scripts/webkitpy/tool/mocktool.py:
  • Scripts/webkitpy/tool/steps/validatechangelogs.py:
4:22 PM Changeset in webkit [84559] by dpranke@chromium.org
  • 1 edit
    18 adds in trunk/LayoutTests

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

Unreviewed, expectations change - update chromium SVG baselines
after r84522.

  • platform/chromium-linux/svg/dynamic-updates/SVGFEDropShadowElement-dom-dx-attr-expected.png: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDropShadowElement-dom-dy-attr-expected.png: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDropShadowElement-dom-in-attr-expected.png: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDropShadowElement-dom-shadow-color-attr-expected.png: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDropShadowElement-dom-shadow-opacity-attr-expected.png: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDropShadowElement-dom-stdDeviation-attr-expected.png: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-dx-prop-expected.png: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-dy-prop-expected.png: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-in-prop-expected.png: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-shadow-color-prop-expected.png: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-shadow-opacity-prop-expected.png: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-stdDeviation-prop-expected.png: Added.
  • platform/chromium-linux/svg/filters/feDropShadow-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEDropShadowElement-dom-shadow-color-attr-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEDropShadowElement-dom-shadow-opacity-attr-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-shadow-color-prop-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-shadow-opacity-prop-expected.png: Added.
  • platform/chromium-mac/svg/filters/feDropShadow-expected.png: Added.
4:20 PM Changeset in webkit [84558] by ggaren@apple.com
  • 5 edits in trunk

Fixed a failure seen in fast/dom/prototype-inheritance-2.html due to
wrong global object use.

Reviewed by Sam Weinig.

Source/WebCore:

  • bindings/js/JSHTMLDocumentCustom.cpp:

(WebCore::JSHTMLDocument::nameGetter):
(WebCore::JSHTMLDocument::all):

  • bindings/js/JSHTMLFormElementCustom.cpp:

(WebCore::JSHTMLFormElement::nameGetter): Use the right global object.

LayoutTests:

  • fast/dom/prototype-inheritance-2-expected.txt: Expect success!
4:10 PM Changeset in webkit [84557] by dpranke@chromium.org
  • 2 edits in trunk/LayoutTests

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

Unreviewed, expectations change - flaky tests.

  • platform/chromium/test_expectations.txt:
4:08 PM Changeset in webkit [84556] by oliver@apple.com
  • 120 edits in trunk/Source

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

Reviewed by Geoffrey Garen.

Start moving to a general visitor pattern for GC traversal
​https://bugs.webkit.org/show_bug.cgi?id=59141

This is just a rename:

markChildren -> visitChildren
markAggregate -> visitAggregate
markStack -> visitor
MarkStack -> typedef'd to SlotVisitor

  • API/JSCallbackObject.h: (JSC::JSCallbackObjectData::visitChildren): (JSC::JSCallbackObjectData::JSPrivatePropertyMap::visitChildren): (JSC::JSCallbackObject::visitChildren):
  • JavaScriptCore.exp:
  • bytecode/CodeBlock.cpp: (JSC::CodeBlock::visitStructures): (JSC::EvalCodeCache::visitAggregate): (JSC::CodeBlock::visitAggregate):
  • bytecode/CodeBlock.h:
  • bytecode/EvalCodeCache.h:
  • bytecode/Instruction.h: (JSC::PolymorphicAccessStructureList::visitAggregate):
  • bytecode/StructureStubInfo.cpp: (JSC::StructureStubInfo::visitAggregate):
  • bytecode/StructureStubInfo.h:
  • debugger/DebuggerActivation.cpp: (JSC::DebuggerActivation::visitChildren):
  • debugger/DebuggerActivation.h:
  • heap/HandleHeap.cpp: (JSC::WeakHandleOwner::isReachableFromOpaqueRoots): (JSC::HandleHeap::markStrongHandles): (JSC::HandleHeap::markWeakHandles):
  • heap/HandleHeap.h:
  • heap/HandleStack.cpp: (JSC::HandleStack::mark):
  • heap/HandleStack.h:
  • heap/Heap.cpp: (JSC::Heap::markProtectedObjects): (JSC::Heap::markTempSortVectors): (JSC::Heap::markRoots):
  • heap/Heap.h:
  • heap/MarkStack.cpp: (JSC::MarkStack::visitChildren): (JSC::MarkStack::drain):
  • heap/MarkStack.h: (JSC::HeapRootVisitor::HeapRootVisitor): (JSC::HeapRootVisitor::mark): (JSC::HeapRootVisitor::visitor):
  • heap/MarkedSpace.h:
  • runtime/ArgList.cpp: (JSC::MarkedArgumentBuffer::markLists):
  • runtime/ArgList.h:
  • runtime/Arguments.cpp: (JSC::Arguments::visitChildren):
  • runtime/Arguments.h:
  • runtime/Executable.cpp: (JSC::EvalExecutable::visitChildren): (JSC::ProgramExecutable::visitChildren): (JSC::FunctionExecutable::visitChildren):
  • runtime/Executable.h:
  • runtime/GetterSetter.cpp: (JSC::GetterSetter::visitChildren):
  • runtime/GetterSetter.h: (JSC::GetterSetter::createStructure):
  • runtime/JSAPIValueWrapper.h: (JSC::JSAPIValueWrapper::createStructure):
  • runtime/JSActivation.cpp: (JSC::JSActivation::visitChildren):
  • runtime/JSActivation.h:
  • runtime/JSArray.cpp: (JSC::JSArray::visitChildren):
  • runtime/JSArray.h: (JSC::JSArray::visitDirect):
  • runtime/JSCell.h: (JSC::JSCell::JSCell::visitChildren):
  • runtime/JSFunction.cpp: (JSC::JSFunction::visitChildren):
  • runtime/JSFunction.h:
  • runtime/JSGlobalObject.cpp: (JSC::visitIfNeeded): (JSC::JSGlobalObject::visitChildren):
  • runtime/JSGlobalObject.h:
  • runtime/JSONObject.cpp:
  • runtime/JSObject.cpp: (JSC::JSObject::visitChildren):
  • runtime/JSObject.h: (JSC::JSObject::visitDirect):
  • runtime/JSPropertyNameIterator.cpp: (JSC::JSPropertyNameIterator::visitChildren):
  • runtime/JSPropertyNameIterator.h: (JSC::JSPropertyNameIterator::createStructure):
  • runtime/JSStaticScopeObject.cpp: (JSC::JSStaticScopeObject::visitChildren):
  • runtime/JSStaticScopeObject.h:
  • runtime/JSTypeInfo.h: (JSC::TypeInfo::TypeInfo): (JSC::TypeInfo::overridesVisitChildren):
  • runtime/JSWrapperObject.cpp: (JSC::JSWrapperObject::visitChildren):
  • runtime/JSWrapperObject.h:
  • runtime/JSZombie.h: (JSC::JSZombie::visitChildren):
  • runtime/NativeErrorConstructor.cpp: (JSC::NativeErrorConstructor::visitChildren):
  • runtime/NativeErrorConstructor.h:
  • runtime/RegExpObject.cpp: (JSC::RegExpObject::visitChildren):
  • runtime/RegExpObject.h:
  • runtime/ScopeChain.cpp: (JSC::ScopeChainNode::visitChildren):
  • runtime/ScopeChain.h:
  • runtime/SmallStrings.cpp: (JSC::SmallStrings::visitChildren):
  • runtime/SmallStrings.h:
  • runtime/Structure.cpp: (JSC::Structure::Structure): (JSC::Structure::visitChildren):
  • runtime/Structure.h:
  • runtime/StructureChain.cpp: (JSC::StructureChain::visitChildren):
  • runtime/StructureChain.h: (JSC::StructureChain::createStructure):

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

Reviewed by Geoffrey Garen.

Start moving to a general visitor pattern for GC traversal
​https://bugs.webkit.org/show_bug.cgi?id=59141

This is just a rename:

markChildren -> visitChildren
markAggregate -> visitAggregate
markStack -> visitor
MarkStack -> typedef'd to SlotVisitor

  • UserObjectImp.cpp: (UserObjectImp::visitChildren):

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

Reviewed by Geoffrey Garen.

Start moving to a general visitor pattern for GC traversal
​https://bugs.webkit.org/show_bug.cgi?id=59141

This is just a rename:

markChildren -> visitChildren
markAggregate -> visitAggregate
markStack -> visitor
MarkStack -> typedef'd to SlotVisitor

  • bindings/js/JSAttrCustom.cpp: (WebCore::JSAttr::visitChildren):
  • bindings/js/JSAudioConstructor.cpp: (WebCore::constructAudio):
  • bindings/js/JSCSSRuleCustom.cpp: (WebCore::JSCSSRule::visitChildren):
  • bindings/js/JSCSSStyleDeclarationCustom.cpp: (WebCore::JSCSSStyleDeclaration::visitChildren):
  • bindings/js/JSDOMBinding.cpp: (WebCore::visitActiveObjectsForContext): (WebCore::markDOMObjectWrapper):
  • bindings/js/JSDOMBinding.h:
  • bindings/js/JSDOMGlobalObject.cpp: (WebCore::JSDOMGlobalObject::visitChildren):
  • bindings/js/JSDOMGlobalObject.h:
  • bindings/js/JSDOMImplementationCustom.cpp: (WebCore::JSDOMImplementation::visitChildren):
  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::visitChildren):
  • bindings/js/JSDOMWindowShell.cpp: (WebCore::JSDOMWindowShell::visitChildren):
  • bindings/js/JSDOMWindowShell.h:
  • bindings/js/JSDocumentCustom.cpp: (WebCore::JSDocument::visitChildren):
  • bindings/js/JSElementCustom.cpp: (WebCore::JSElement::visitChildren):
  • bindings/js/JSEventListener.cpp: (WebCore::JSEventListener::markJSFunction):
  • bindings/js/JSEventListener.h:
  • bindings/js/JSHTMLCanvasElementCustom.cpp: (WebCore::JSHTMLCanvasElement::visitChildren):
  • bindings/js/JSHTMLLinkElementCustom.cpp: (WebCore::JSHTMLLinkElement::visitChildren):
  • bindings/js/JSHTMLStyleElementCustom.cpp: (WebCore::JSHTMLStyleElement::visitChildren):
  • bindings/js/JSImageConstructor.cpp: (WebCore::constructImage):
  • bindings/js/JSJavaScriptAudioNodeCustom.cpp: (WebCore::JSJavaScriptAudioNode::visitChildren):
  • bindings/js/JSMessageChannelCustom.cpp: (WebCore::JSMessageChannel::visitChildren):
  • bindings/js/JSMessagePortCustom.cpp: (WebCore::JSMessagePort::visitChildren):
  • bindings/js/JSNamedNodeMapCustom.cpp: (WebCore::JSNamedNodeMapOwner::isReachableFromOpaqueRoots): (WebCore::JSNamedNodeMap::visitChildren):
  • bindings/js/JSNavigatorCustom.cpp: (WebCore::JSNavigator::visitChildren):
  • bindings/js/JSNodeCustom.cpp: (WebCore::isObservable): (WebCore::isReachableFromDOM): (WebCore::JSNodeOwner::isReachableFromOpaqueRoots): (WebCore::JSNode::visitChildren):
  • bindings/js/JSNodeCustom.h:
  • bindings/js/JSNodeFilterCondition.cpp: (WebCore::JSNodeFilterCondition::WeakOwner::isReachableFromOpaqueRoots):
  • bindings/js/JSNodeFilterCondition.h:
  • bindings/js/JSNodeFilterCustom.cpp: (WebCore::JSNodeFilter::visitChildren):
  • bindings/js/JSNodeIteratorCustom.cpp: (WebCore::JSNodeIterator::visitChildren):
  • bindings/js/JSNodeListCustom.cpp: (WebCore::JSNodeListOwner::isReachableFromOpaqueRoots):
  • bindings/js/JSProcessingInstructionCustom.cpp: (WebCore::JSProcessingInstruction::visitChildren):
  • bindings/js/JSSVGElementInstanceCustom.cpp: (WebCore::JSSVGElementInstance::visitChildren):
  • bindings/js/JSSharedWorkerCustom.cpp: (WebCore::JSSharedWorker::visitChildren):
  • bindings/js/JSStyleSheetCustom.cpp: (WebCore::JSStyleSheet::visitChildren):
  • bindings/js/JSTreeWalkerCustom.cpp: (WebCore::JSTreeWalker::visitChildren):
  • bindings/js/JSWebGLRenderingContextCustom.cpp: (WebCore::JSWebGLRenderingContext::visitChildren):
  • bindings/js/JSWebKitAnimationListCustom.cpp: (WebCore::JSWebKitAnimationList::visitChildren):
  • bindings/js/JSWorkerContextCustom.cpp: (WebCore::JSWorkerContext::visitChildren):
  • bindings/js/JSXMLHttpRequestCustom.cpp: (WebCore::JSXMLHttpRequest::visitChildren):
  • bindings/js/JSXMLHttpRequestUploadCustom.cpp: (WebCore::JSXMLHttpRequestUpload::visitChildren):
  • bindings/scripts/CodeGeneratorJS.pm:
  • bridge/qt/qt_instance.cpp: (JSC::Bindings::QtRuntimeObject::visitChildren): (JSC::Bindings::QtInstance::markAggregate):
  • bridge/qt/qt_instance.h:
  • bridge/qt/qt_pixmapruntime.cpp:
  • bridge/qt/qt_runtime.cpp: (JSC::Bindings::QtRuntimeMetaMethod::visitChildren):
  • bridge/qt/qt_runtime.h:
  • dom/EventListener.h: (WebCore::EventListener::visitJSFunction):
  • dom/EventTarget.h: (WebCore::EventTarget::visitJSEventListeners):
  • dom/Node.h:
  • dom/NodeFilterCondition.h: (WebCore::NodeFilterCondition::visitAggregate):
  • page/DOMWindow.h:
  • workers/WorkerContext.h:
4:07 PM Changeset in webkit [84555] by abarth@webkit.org
  • 2 edits
    2 adds in trunk/LayoutTests

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

Update test results to show that we're using ogv. I would have
preferred to make this test independent of which video codec we have,
but I didn't see a way to do that.

  • platform/chromium/http/tests/security/contentSecurityPolicy/media-src-blocked-expected.txt: Added.
  • platform/chromium/media/csp-blocks-video-expected.txt: Added.
  • platform/chromium/test_expectations.txt:
3:52 PM Changeset in webkit [84554] by tony@chromium.org
  • 1 edit
    12822 deletes in trunk/LayoutTests

2011-04-21 Tony Chang <​tony@chromium.org>

Unreviewed, deleting .checksum files in platform/chromium*

3:49 PM Changeset in webkit [84553] by Beth Dakin
  • 20 edits in trunk/Source

​https://bugs.webkit.org/show_bug.cgi?id=59048
Need to track whether overlay scrollbar is currently visible and in lower-
righthand corner
-and corresponding-
<rdar://problem/9211232>

Reviewed by Dan Bernstein.

Source/WebCore:

Whenever we start a new ScrollbarPartAnimation, check if the scrollbar is vertical
and showing (alpha of 1), and if so, call setVisibleScrollerThumbRect with the
knob rect, otherwise with an empty rect. ScrollAnimatorMac has a local rect to
track this value so that it can avoid sending messages across the ChromeClient to
the UIProcess if the rect has not changed.

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

(-[ScrollbarPainterDelegate setUpAnimation:scrollerPainter:part:WebCore::animateAlphaTo:duration:]):
(WebCore::ScrollAnimatorMac::ScrollAnimatorMac):
(WebCore::ScrollAnimatorMac::setVisibleScrollerThumbRect):

Plumbing for the chrome client.

  • page/ChromeClient.h:

(WebCore::ChromeClient::notifyScrollerThumbIsVisibleInRect):

  • page/FrameView.cpp:

(WebCore::FrameView::setVisibleScrollerThumbRect):

  • page/FrameView.h:
  • platform/ScrollableArea.h:

(WebCore::ScrollableArea::setVisibleScrollerThumbRect):

New WebCoreSystemInterface function to get the knob rect.

  • WebCore.exp.in:
  • platform/mac/WebCoreSystemInterface.h:
  • platform/mac/WebCoreSystemInterface.mm:

Source/WebKit/mac:

A new WebSystemInterface is needed for this.

  • WebCoreSupport/WebSystemInterface.mm:

(InitWebCoreSystemInterface):

Source/WebKit2:

WebPageProxy now stores an IntRect of the scroller thumb set by WebCore through
the client.

_windowResizeMouseLocationIsInVisibleScrollerThumb tests if the parameter point is
inside the cached scroller thumb rect.

  • UIProcess/API/mac/WKView.mm:

(-[WKView _windowResizeMouseLocationIsInVisibleScrollerThumb:]):

Client work to cache the rect.

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::WebPageProxy):
(WebKit::WebPageProxy::close):
(WebKit::WebPageProxy::processDidCrash):
(WebKit::WebPageProxy::notifyScrollerThumbIsVisibleInRect):

  • UIProcess/WebPageProxy.h:

(WebKit::WebPageProxy::visibleScrollerThumbRect):

  • UIProcess/WebPageProxy.messages.in:
  • WebProcess/WebCoreSupport/WebChromeClient.cpp:

(WebKit::WebChromeClient::notifyScrollerThumbIsVisibleInRect):

  • WebProcess/WebCoreSupport/WebChromeClient.h:

New WebSystemInterface is needed by WebCore for this.

  • WebProcess/WebCoreSupport/mac/WebSystemInterface.mm:

(InitWebCoreSystemInterface):

3:47 PM Changeset in webkit [84552] by enne@google.com
  • 3 edits in branches/chromium/742/Source/WebCore/platform/graphics/chromium

Merge 84542 - 2011-04-21 Adrienne Walker <​enne@google.com> Reviewed by James Robinson. [chromium] Don't upload partially drawn tiles in the compositor ​https://bugs.webkit.org/show_bug.cgi?id=59112 Use the same set of tiles for painting as is used for updating. Before, a larger set of tiles were being uploaded, causing some to only be partially undirtied because they were not fully painted. As tiles are clear their dirty rect after being unpainted, this caused artifacts to appear when scrolling. Unfortunately, no layout test can test this behavior yet because scrolling invalidations are done differently in test shell. * platform/graphics/chromium/ImageLayerChromium.cpp: (WebCore::ImageLayerChromium::updateCompositorResources): * platform/graphics/chromium/LayerTilerChromium.cpp: (WebCore::LayerTilerChromium::update): (WebCore::LayerTilerChromium::uploadCanvas): (WebCore::LayerTilerChromium::updateFromPixels): * platform/graphics/chromium/LayerTilerChromium.h:
TBR=​enne@google.com

3:44 PM Changeset in webkit [84551] by enne@google.com
  • 1 edit in branches/chromium/742/Source/WebCore/platform/graphics/chromium/ContentLayerChromium.cpp

Merge 84529 - 2011-04-21 Adrienne Walker <​enne@google.com> Reviewed by James Robinson. [chromium] Clamp dirty regions on content layers to layer bounds ​https://bugs.webkit.org/show_bug.cgi?id=58933 This prevents garbage being rendered when layers are resized and a previous invalidation (now off-layer) is no longer valid. * platform/graphics/chromium/ContentLayerChromium.cpp: (WebCore::ContentLayerChromium::paintContentsIfDirty):
TBR=​enne@google.com

3:13 PM Changeset in webkit [84550] by adamk@chromium.org
  • 2 edits in trunk/Source/JavaScriptCore

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

Unreviewed, rolling out r84548.
​http://trac.webkit.org/changeset/84548
​https://bugs.webkit.org/show_bug.cgi?id=59144

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

  • wtf/Platform.h:
3:00 PM Changeset in webkit [84549] by abarth@webkit.org
  • 4 edits in trunk/Tools

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

Reviewed by Eric Seidel.

ValidateChangeLogs doesn't work on SVN
​https://bugs.webkit.org/show_bug.cgi?id=59115

svn-create-patch lies about the diff in the working copy by moving
ChangeLog entries to the top of the diff. That's fine on most cases,
but causes problems for ValidateChangeLogs, which is trying validate
the where the ChangeLog entry appears.

I haven't added a test for this change because I couldn't figure out
how to write one. The issue is more of an integration issue, which
we're not really set up to test in our unit testing framework. If this
patch had worked around the output from svn-create-patch, then I could
have tested that we behave correctly on sample svn-create-patch output,
but, in this case, I've removed the dependency on svn-create-patch. I
could test that we behave correctly on "svn diff" output, but we
already have those tests.

  • Scripts/webkitpy/tool/commands/download_unittest.py:
  • Scripts/webkitpy/tool/mocktool.py:
  • Scripts/webkitpy/tool/steps/validatechangelogs.py:
2:39 PM Changeset in webkit [84548] by adamk@chromium.org
  • 2 edits in trunk/Source/JavaScriptCore

2011-04-21 Adam Klein <​adamk@chromium.org>

Reviewed by David Levin.

[fileapi] Worker File API calls that create Blobs fail in debug builds due to random number generator thread assertion
​https://bugs.webkit.org/show_bug.cgi?id=55728

Enable WTF_MULTIPLE_THREADS for Chromium.

  • wtf/Platform.h:
2:28 PM Changeset in webkit [84547] by tony@chromium.org
  • 69 edits in trunk/LayoutTests

2011-04-21 Tony Chang <​tony@chromium.org>

Unreviewed, embedding checksums in png files not in LayoutTests/platform.

2:27 PM April 2011 Meeting edited by sam@webkit.org
(diff)
2:22 PM Changeset in webkit [84546] by commit-queue@webkit.org
  • 5 edits in trunk/Source/WebCore

2011-04-21 Nat Duca <​nduca@chromium.org>

Reviewed by James Robinson.

[chromium] Fix artefacts in compositor HUD when resizing
​https://bugs.webkit.org/show_bug.cgi?id=58969

HUD was using the tiler program and not setting the texture
transform. The HUD would thus draw with a texture transform based
on the last tile drawn, leading sometimes to distortion.

  • platform/graphics/chromium/LayerRendererChromium.cpp: (WebCore::LayerRendererChromium::initializeSharedObjects): (WebCore::LayerRendererChromium::cleanupSharedObjects):
  • platform/graphics/chromium/LayerRendererChromium.h: (WebCore::LayerRendererChromium::headsUpDisplayProgram):
  • platform/graphics/chromium/cc/CCHeadsUpDisplay.cpp: (WebCore::CCHeadsUpDisplay::draw):
  • platform/graphics/chromium/cc/CCHeadsUpDisplay.h:
2:09 PM Changeset in webkit [84545] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

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

Reviewed by Kenneth Russell.

only return false if we actually in a skia-gpu accelerated canvas for isNativeFontRenderingAllowed
​https://bugs.webkit.org/show_bug.cgi?id=59024

No new tests. The bug (and fix) depend on enabling SKIA_GPU. When that is enabled, the bug
shows all text w/o lcd rendering. This is a regression from a recent CL when we switch to
calling skia::SupportsPlatformPaint. This CL fixes that.
There should be no change for the default builds when SKIA_GPU is not enabled.

  • platform/graphics/skia/PlatformContextSkia.cpp: (WebCore::PlatformContextSkia::isNativeFontRenderingAllowed):
2:08 PM Changeset in webkit [84544] by tony@chromium.org
  • 7 edits in trunk/Tools

2011-04-21 Tony Chang <​tony@chromium.org>

Reviewed by Ojan Vafai.

old-run-webkit-tests/new-run-webkit-tests shouldn't read or write .checksum files
​https://bugs.webkit.org/show_bug.cgi?id=58402

  • Scripts/old-run-webkit-tests: Don't read or write .checksum files.
  • Scripts/webkitpy/layout_tests/layout_package/single_test_runner.py: Don't write .checksum files.
  • Scripts/webkitpy/layout_tests/port/base.py: Don't read .checksum files.
  • Scripts/webkitpy/layout_tests/port/mock_drt_unittest.py:
  • Scripts/webkitpy/layout_tests/port/test.py: Update passing pngs to include a checksum.
  • Scripts/webkitpy/layout_tests/run_webkit_tests_unittest.py:
1:44 PM Changeset in webkit [84543] by ojan@chromium.org
  • 3 edits in trunk/Tools

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

Reviewed by Tony Chang.

only expand visible rows
​https://bugs.webkit.org/show_bug.cgi?id=59130

For the chromium port, where there are many expected failures,
expanding all of them is quite slow. Only expand the visible ones.
This makes the default case. Also, make the expanding async to
avoid totally killing the process.

  • Scripts/webkitpy/layout_tests/layout_package/json_results.html:
  • Scripts/webkitpy/layout_tests/layout_package/json_results_test.js:
1:19 PM Changeset in webkit [84542] by enne@google.com
  • 4 edits in trunk/Source/WebCore

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

Reviewed by James Robinson.

[chromium] Don't upload partially drawn tiles in the compositor
​https://bugs.webkit.org/show_bug.cgi?id=59112

Use the same set of tiles for painting as is used for updating.
Before, a larger set of tiles were being uploaded, causing some to
only be partially undirtied because they were not fully painted.
As tiles are clear their dirty rect after being unpainted, this caused
artifacts to appear when scrolling.

Unfortunately, no layout test can test this behavior yet because
scrolling invalidations are done differently in test shell.

  • platform/graphics/chromium/ImageLayerChromium.cpp: (WebCore::ImageLayerChromium::updateCompositorResources):
  • platform/graphics/chromium/LayerTilerChromium.cpp: (WebCore::LayerTilerChromium::update): (WebCore::LayerTilerChromium::uploadCanvas): (WebCore::LayerTilerChromium::updateFromPixels):
  • platform/graphics/chromium/LayerTilerChromium.h:
12:49 PM Changeset in webkit [84541] by krit@webkit.org
  • 5 edits in trunk/LayoutTests

2011-04-21 Dirk Schulze <​krit@webkit.org>

Uneviewed rebaseline of win.

SVG feDropShadow implementation of SVG Filters 1.2
​https://bugs.webkit.org/show_bug.cgi?id=52513

  • platform/win/fast/dom/Window/window-property-descriptors-expected.txt:
  • platform/win/fast/dom/prototype-inheritance-2-expected.txt:
  • platform/win/fast/dom/prototype-inheritance-expected.txt:
  • platform/win/fast/js/global-constructors-expected.txt:
12:35 PM Changeset in webkit [84540] by Martin Robinson
  • 7 edits
    3 deletes in trunk

2011-04-21 Martin Robinson <​mrobinson@igalia.com>

[GTK] DRT: implement eventSender.scalePageBy
​https://bugs.webkit.org/show_bug.cgi?id=59082

Rolling out r84507 since it broke many GTK+ tests.

  • platform/gtk/Skipped:
  • platform/gtk/fast/repaint/scale-page-shrink-expected.checksum: Removed.
  • platform/gtk/fast/repaint/scale-page-shrink-expected.png: Removed.
  • platform/gtk/fast/repaint/scale-page-shrink-expected.txt: Removed.

2011-04-21 Martin Robinson <​mrobinson@igalia.com>

[GTK] DRT: implement eventSender.scalePageBy
​https://bugs.webkit.org/show_bug.cgi?id=59082

Rolling out r84507 since it broke many GTK+ tests.

  • WebCoreSupport/DumpRenderTreeSupportGtk.cpp:
  • WebCoreSupport/DumpRenderTreeSupportGtk.h:

2011-04-21 Martin Robinson <​mrobinson@igalia.com>

[GTK] DRT: implement eventSender.scalePageBy
​https://bugs.webkit.org/show_bug.cgi?id=59082

Rolling out r84507 since it broke many GTK+ tests.

  • DumpRenderTree/gtk/EventSender.cpp:
12:31 PM Changeset in webkit [84539] by andreas.kling@nokia.com
  • 4 edits in trunk/Source/WebKit2

2011-04-21 Andreas Kling <​andreas.kling@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt][WK2] Emit QWKPage::urlChanged() when navigating within the same document.
​https://bugs.webkit.org/show_bug.cgi?id=59044

Hook up the didSameDocumentNavigationForFrame callback to QWKPage::urlChanged().

  • UIProcess/API/qt/ClientImpl.cpp: (qt_wk_didSameDocumentNavigationForFrame):
  • UIProcess/API/qt/ClientImpl.h:
  • UIProcess/API/qt/qwkpage.cpp: (QWKPage::QWKPage):
12:27 PM Changeset in webkit [84538] by commit-queue@webkit.org
  • 8 edits in trunk/Source/WebCore

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

Reviewed by Alexey Proskuryakov.

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

Fix a compile issue on Solaris 10 with Sun Studio 12,
ternary operators (?:) must have the same types

  • loader/DocumentLoader.cpp: (WebCore::DocumentLoader::popArchiveForSubframe):
  • loader/archive/ArchiveFactory.cpp: (WebCore::ArchiveFactory::create):
  • loader/icon/IconDatabase.cpp: (WebCore::IconDatabase::setIconDataForIconURL):
  • page/animation/AnimationBase.cpp: (WebCore::blendFunc):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::updateScrollCornerStyle): (WebCore::RenderLayer::updateResizerStyle):
  • rendering/RenderScrollbar.cpp: (WebCore::RenderScrollbar::updateScrollbarPart):
  • rendering/RenderTextFragment.cpp: (WebCore::RenderTextFragment::RenderTextFragment):
12:22 PM Changeset in webkit [84537] by krit@webkit.org
  • 6 edits in trunk/LayoutTests

2011-04-21 Dirk Schulze <​krit@webkit.org>

Unreviewed rebaseline.

SVG feDropShadow implementation of SVG Filters 1.2
​https://bugs.webkit.org/show_bug.cgi?id=52513

  • fast/dom/Window/window-properties-expected.txt:
  • fast/dom/Window/window-property-descriptors-expected.txt:
  • fast/dom/prototype-inheritance-2-expected.txt:
  • fast/dom/prototype-inheritance-expected.txt:
  • fast/js/global-constructors-expected.txt:
12:15 PM Changeset in webkit [84536] by Csaba Osztrogonác
  • 5 edits in trunk/LayoutTests

[Qt] Unreviewed. Update expected files after r84522.

  • platform/qt/fast/dom/Window/window-properties-expected.txt:
  • platform/qt/fast/dom/Window/window-property-descriptors-expected.txt:
  • platform/qt/fast/dom/prototype-inheritance-expected.txt:
  • platform/qt/fast/js/global-constructors-expected.txt:
12:09 PM Changeset in webkit [84535] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

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

Reviewed by Kenneth Russell.

[chromium, win] Multiple failures of shadow rendering layout tests
​https://bugs.webkit.org/show_bug.cgi?id=58772

No new tests. This CL fixes a regression in the following tests
fast/multicol/shadow-breaking.html
fast/repaint/shadow-multiple-horizontal.html
fast/repaint/shadow-multiple-strict-horizontal.html
fast/repaint/shadow-multiple-strict-vertical.html
fast/repaint/shadow-multiple-vertical.html
fast/text/shadow-translucent-fill.html
fast/transforms/shadows.html
svg/css/group-with-shadow.svg
svg/css/shadow-changes.svg
svg/css/text-shadow-multiple.xhtml
transitions/svg-text-shadow-transition.html
svg/css/composite-shadow-text.svg

  • platform/graphics/skia/GraphicsContextSkia.cpp: (WebCore::GraphicsContext::setPlatformShadow):
12:08 PM Changeset in webkit [84534] by weinig@apple.com
  • 6 edits in trunk/Tools

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

Reviewed by Simon Fraser.

Add way to garbage collect between each test when running layout tests
​https://bugs.webkit.org/show_bug.cgi?id=59126

  • DumpRenderTree/mac/DumpRenderTree.mm: (initializeGlobalsFromCommandLineOptions): (runTest):
  • Scripts/old-run-webkit-tests:
  • WebKitTestRunner/InjectedBundle/InjectedBundle.cpp: (WTR::InjectedBundle::didReceiveMessage):
  • WebKitTestRunner/TestController.cpp: (WTR::TestController::TestController): (WTR::TestController::initialize): (WTR::TestController::resetStateToConsistentValues):
  • WebKitTestRunner/TestController.h:
12:01 PM Changeset in webkit [84533] by rolandsteiner@chromium.org
  • 2 edits in trunk/LayoutTests

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

Unreviewed: skip tests on Windows.

fast/dom/shadow/layout-tests-can-access-shadow.html is failing on Windows
​https://bugs.webkit.org/show_bug.cgi?id=59081

Skip all tests in fast/dom/shadow on Windows.
(shadowRoot accessor is not implemented on Windows yet)

  • platform/win/Skipped:
11:56 AM Changeset in webkit [84532] by dpranke@chromium.org
  • 2 edits in trunk/LayoutTests

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

Unreviewed, expectations change.

Roll in downstream failures as of Chromium r82532.

  • platform/chromium/test_expectations.txt:
11:46 AM Changeset in webkit [84531] by ggaren@apple.com
  • 3 edits in trunk/Source/WebCore

Try to fix the Qt build.

  • UseJSC.cmake:
  • WebCore.pro: Use correct file name.
11:45 AM Changeset in webkit [84530] by krit@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-04-21 Dirk Schulze <​krit@webkit.org>

Unreviewed Qt build fix.

SVG feDropShadow implementation of SVG Filters 1.2
​https://bugs.webkit.org/show_bug.cgi?id=52513

  • CodeGenerators.pri:
11:39 AM Changeset in webkit [84529] by enne@google.com
  • 2 edits in trunk/Source/WebCore

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

Reviewed by James Robinson.

[chromium] Clamp dirty regions on content layers to layer bounds
​https://bugs.webkit.org/show_bug.cgi?id=58933

This prevents garbage being rendered when layers are resized and a
previous invalidation (now off-layer) is no longer valid.

  • platform/graphics/chromium/ContentLayerChromium.cpp: (WebCore::ContentLayerChromium::paintContentsIfDirty):
11:32 AM Changeset in webkit [84528] by rolandsteiner@chromium.org
  • 4 edits
    2 adds in trunk

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

Reviewed by Kent Tamura.

Bug 59012 - Shadows should be imported along with their host nodes
​https://bugs.webkit.org/show_bug.cgi?id=59012

Verify that the shadow of a bound element is imported along with the element.

  • fast/dom/shadow/import-shadowroot-expected.txt: Added.
  • fast/dom/shadow/import-shadowroot.html: Added.

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

Reviewed by Kent Tamura.

Bug 59012 - Shadows should be imported along with their host nodes
​https://bugs.webkit.org/show_bug.cgi?id=59012

Copy the shadow tree of an element (even on a shallow copy!).

Test: fast/dom/shadow/import-shadowroot.html

  • dom/Element.cpp: (WebCore::Element::copyNonAttributeProperties):
  • dom/Element.h:
11:29 AM Changeset in webkit [84527] by ggaren@apple.com
  • 38 edits
    1 add in trunk/Source/WebCore

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

Reviewed by Oliver Hunt.

Removed a use of markDOMObjectWrapper: CSS Object Model
​https://bugs.webkit.org/show_bug.cgi?id=59057

This is 24 hours of my life I will never get back. Thank you, W3C.

  • GNUmakefile.list.am:
  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj: Build!
  • bindings/js/JSCSSFontFaceRuleCustom.cpp:
  • bindings/js/JSCSSImportRuleCustom.cpp:
  • bindings/js/JSCSSMediaRuleCustom.cpp:
  • bindings/js/JSCSSPageRuleCustom.cpp:
  • bindings/js/JSCSSRuleCustom.cpp: (WebCore::JSCSSRuleOwner::isReachableFromOpaqueRoots): (WebCore::JSCSSRuleOwner::finalize): (WebCore::wrapperOwner): (WebCore::wrapperContext): (WebCore::JSCSSRule::markChildren):
  • bindings/js/JSCSSRuleListCustom.cpp: (WebCore::JSCSSRuleListOwner::isReachableFromOpaqueRoots): (WebCore::JSCSSRuleListOwner::finalize): (WebCore::wrapperOwner): (WebCore::wrapperContext): (WebCore::toJS):
  • bindings/js/JSCSSStyleDeclarationCustom.cpp: (WebCore::JSCSSStyleDeclarationOwner::isReachableFromOpaqueRoots): (WebCore::JSCSSStyleDeclarationOwner::finalize): (WebCore::wrapperOwner): (WebCore::wrapperContext): (WebCore::JSCSSStyleDeclaration::markChildren): Removed custom mark functions, and replaced with use of the opaque roots system. This has the nice benefit of allowing us to eagerly recycle far more CSS wrappers and their associated DOM wrappers.

(WebCore::JSCSSStyleDeclaration::getPropertyCSSValue): Made this function
custom because it must maintain the sorrowful invariant that primitive
values can be mapped back to objects that are reachable through the DOM.
We maintain this invariant through an external hash table because
making all primitive values bigger would be a too-large memory use
penalty. Luckily, nobody uses this feature, so the cost of the hash table
is no big deal.

(WebCore::toJS): We need a custom toJS so we can specify our own WeakHandleOwner.

  • bindings/js/JSCSSStyleRuleCustom.cpp: Removed custom mark, as above.
  • bindings/js/JSCSSValueCustom.cpp: (WebCore::cssValueRoots): (WebCore::JSCSSValueOwner::isReachableFromOpaqueRoots): (WebCore::JSCSSValueOwner::finalize): (WebCore::wrapperOwner): (WebCore::wrapperContext): Use the opaque roots system. Piggy-back on the external hash table filled by JSCSSStyleDeclaration::getPropertyCSSValue.
  • bindings/js/JSDocumentCustom.cpp: (WebCore::JSDocument::markChildren): No need to mark our stylesheet list, since all style-related objects can use our opaque root to determine liveness.
  • bindings/js/JSMediaListCustom.cpp: Added. (WebCore::JSMediaListOwner::isReachableFromOpaqueRoots): (WebCore::JSMediaListOwner::finalize): (WebCore::wrapperOwner): (WebCore::wrapperContext): (WebCore::toJS): Use the opaque roots system.
  • bindings/js/JSNodeCustom.h: (WebCore::root): Added helpers for computing style-related roots.
  • bindings/js/JSStyleSheetCustom.cpp: (WebCore::JSStyleSheetOwner::isReachableFromOpaqueRoots): (WebCore::JSStyleSheetOwner::finalize): (WebCore::wrapperOwner): (WebCore::wrapperContext): (WebCore::JSStyleSheet::markChildren): Use the opaque roots system instead of direct marking.


  • bindings/js/JSStyleSheetListCustom.cpp: (WebCore::JSStyleSheetListOwner::isReachableFromOpaqueRoots): (WebCore::JSStyleSheetListOwner::finalize): (WebCore::wrapperOwner): (WebCore::wrapperContext): (WebCore::toJS): Ditto.
  • bindings/js/JSWebKitCSSKeyframeRuleCustom.cpp:
  • bindings/js/JSWebKitCSSKeyframesRuleCustom.cpp:
  • css/CSSFontFaceRule.idl:
  • css/CSSImportRule.idl: No more custom mark, since we use the opaque roots system instead.
  • css/CSSMediaRule.cpp: (WebCore::CSSMediaRule::CSSMediaRule): Fixed a bug where a media rule would incorrectly have no parent, since this got in the way of correctly computing the media rule's root.
  • css/CSSMediaRule.idl:
  • css/CSSPageRule.idl:
  • css/CSSRuleList.idl:
  • css/CSSStyleDeclaration.idl:
  • css/CSSStyleRule.idl:
  • css/MediaList.idl: Match behavior changes above.
  • css/StyleBase.cpp: (WebCore::StyleBase::node): Helper function for computing roots of the style system. This object model is pretty weird and not very well factored, but I'm pretty sure this covers all the relevant cases.
  • css/StyleBase.h:
  • css/StyleSheetList.idl:
  • css/WebKitCSSKeyframeRule.idl:
  • css/WebKitCSSKeyframesRule.idl: Updated to match the behavior changes above.
11:29 AM Changeset in webkit [84526] by ap@apple.com
  • 14 edits in trunk/Source/WebKit2

Reviewed by Maciej Stachowiak.

WebKit2: Unconfirmed inline input session persists after reload
​https://bugs.webkit.org/show_bug.cgi?id=59054
<rdar://problem/9291086>

Makes the inline input get properly confirmed or discarded when changing selection,
navigating to another page, focusing another view in the window, or after a web process
crash.

  • Shared/EditorState.h: (WebKit::EditorState::EditorState):
  • WebProcess/WebPage/WebPage.cpp: (WebKit::WebPage::editorState): Added a variable that Editor sets while processing input method calls. We don't want selection changes caused by -setMarkedText: discard the composition!


  • UIProcess/PageClient.h:
  • UIProcess/API/mac/PageClientImpl.h:
  • UIProcess/API/mac/PageClientImpl.mm: (WebKit::PageClientImpl::updateTextInputState): (WebKit::PageClientImpl::resetTextInputState): Extended updateSecureInputState() to potentially confirm the current inline input session. Added resetTextInputState, which is to be called when we get out of the current web page.
  • UIProcess/API/mac/WKView.mm: (-[WKView resignFirstResponder]): Just call _resetTextInputState instead of dealing with input state here. (-[WKView _updateSecureInputState]): Use _isFocused to make the code simpler. (-[WKView _updateTextInputStateIncludingSecureInputState:]): Confirm the current inline input session if selection changed for any other reason but an input method changing the composition. We will want to do other work here, like dismissing a dictionary pop-up, updating a font panel etc. (-[WKView _resetTextInputState]): Added. We don't want secure text input persist after navigations, crashes an such, and we also don't want input methods to remember inline input state. Relying on selection changed notifications is fragile and insufficient.
  • UIProcess/API/mac/WKViewInternal.h: Added internal functions to keep the build unbroken.
  • UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::didCommitLoadForFrame): This is the best chance we have to reset input state for a new page. This doesn't work well for cached page loads, but WK1 is quite broken in that case, too. (WebKit::WebPageProxy::editorStateChanged): Moved some logic up to the client, since the function now does much more than before.
  • UIProcess/WebPageProxy.h:
  • UIProcess/mac/WebPageProxyMac.mm: (WebKit::WebPageProxy::confirmCompositionWithoutDisturbingSelection):
  • WebProcess/WebPage/WebPage.h:
  • WebProcess/WebPage/WebPage.messages.in:
  • WebProcess/WebPage/mac/WebPageMac.mm: (WebKit::WebPage::confirmCompositionWithoutDisturbingSelection): Added confirmCompositionWithoutDisturbingSelection(). When we confirm the composition due to selection change, we obvisouly don't want the selection to move back.
11:28 AM Changeset in webkit [84525] by ojan@chromium.org
  • 3 edits in trunk/Tools

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

Reviewed by Mihai Parparita.

fix another innocuous JS error in expand/collapseAllExpectations
​https://bugs.webkit.org/show_bug.cgi?id=59122

  • Scripts/webkitpy/layout_tests/layout_package/json_results.html:
  • Scripts/webkitpy/layout_tests/layout_package/json_results_test.js:
11:19 AM Changeset in webkit [84524] by weinig@apple.com
  • 5 edits in trunk/Source/WebKit2

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

Reviewed by Simon Fraser.

Remove warnings from DrawingAreaImpl and related classes
​https://bugs.webkit.org/show_bug.cgi?id=59120

DrawingAreaImpl is mature enough for others to adopt. Remove warnings.

  • UIProcess/BackingStore.cpp:
  • UIProcess/DrawingAreaProxyImpl.cpp:
  • WebProcess/WebPage/DrawingAreaImpl.cpp:
  • WebProcess/WebPage/LayerTreeHost.cpp:
10:57 AM Changeset in webkit [84523] by weinig@apple.com
  • 6 edits in trunk/Source/WebKit2

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

Reviewed by Simon Fraser.

Deploy GraphicsContextStateSaver in WebKit2
​https://bugs.webkit.org/show_bug.cgi?id=59116

  • UIProcess/FindIndicator.cpp: (WebKit::FindIndicator::draw):
  • WebProcess/Plugins/PluginView.cpp: (WebKit::PluginView::paint):
  • WebProcess/WebPage/FindController.cpp: (WebKit::FindController::drawRect):
  • WebProcess/WebPage/PageOverlay.cpp: (WebKit::PageOverlay::drawRect):
  • WebProcess/WebPage/WebPage.cpp: (WebKit::WebPage::drawRect): (WebKit::WebPage::drawPageOverlay):
10:56 AM Changeset in webkit [84522] by krit@webkit.org
  • 27 edits
    69 adds in trunk

2011-04-21 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVG feDropShadow implementation of SVG Filters 1.2
​https://bugs.webkit.org/show_bug.cgi?id=52513

Added dynamic update tests for FEDropShadow as well as a basic pixel test.

  • platform/mac/svg/dynamic-updates/SVGFEDropShadowElement-dom-dx-attr-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDropShadowElement-dom-dx-attr-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDropShadowElement-dom-dy-attr-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDropShadowElement-dom-dy-attr-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDropShadowElement-dom-in-attr-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDropShadowElement-dom-in-attr-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDropShadowElement-dom-shadow-color-attr-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDropShadowElement-dom-shadow-color-attr-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDropShadowElement-dom-shadow-opacity-attr-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDropShadowElement-dom-shadow-opacity-attr-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDropShadowElement-dom-stdDeviation-attr-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDropShadowElement-dom-stdDeviation-attr-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-dx-prop-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-dx-prop-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-dy-prop-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-dy-prop-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-in-prop-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-in-prop-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-shadow-color-prop-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-shadow-color-prop-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-shadow-opacity-prop-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-shadow-opacity-prop-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-stdDeviation-prop-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDropShadowElement-svgdom-stdDeviation-prop-expected.png: Added.
  • platform/mac/svg/filters/feDropShadow-expected.checksum: Added.
  • platform/mac/svg/filters/feDropShadow-expected.png: Added.
  • platform/mac/svg/filters/feDropShadow-expected.txt: Added.
  • svg/dynamic-updates/SVGFEDropShadowElement-dom-dx-attr-expected.txt: Added.
  • svg/dynamic-updates/SVGFEDropShadowElement-dom-dx-attr.html: Added.
  • svg/dynamic-updates/SVGFEDropShadowElement-dom-dy-attr-expected.txt: Added.
  • svg/dynamic-updates/SVGFEDropShadowElement-dom-dy-attr.html: Added.
  • svg/dynamic-updates/SVGFEDropShadowElement-dom-in-attr-expected.txt: Added.
  • svg/dynamic-updates/SVGFEDropShadowElement-dom-in-attr.html: Added.
  • svg/dynamic-updates/SVGFEDropShadowElement-dom-shadow-color-attr-expected.txt: Added.
  • svg/dynamic-updates/SVGFEDropShadowElement-dom-shadow-color-attr.html: Added.
  • svg/dynamic-updates/SVGFEDropShadowElement-dom-shadow-opacity-attr-expected.txt: Added.
  • svg/dynamic-updates/SVGFEDropShadowElement-dom-shadow-opacity-attr.html: Added.
  • svg/dynamic-updates/SVGFEDropShadowElement-dom-stdDeviation-attr-expected.txt: Added.
  • svg/dynamic-updates/SVGFEDropShadowElement-dom-stdDeviation-attr.html: Added.
  • svg/dynamic-updates/SVGFEDropShadowElement-svgdom-dx-prop-expected.txt: Added.
  • svg/dynamic-updates/SVGFEDropShadowElement-svgdom-dx-prop.html: Added.
  • svg/dynamic-updates/SVGFEDropShadowElement-svgdom-dy-prop-expected.txt: Added.
  • svg/dynamic-updates/SVGFEDropShadowElement-svgdom-dy-prop.html: Added.
  • svg/dynamic-updates/SVGFEDropShadowElement-svgdom-in-prop-expected.txt: Added.
  • svg/dynamic-updates/SVGFEDropShadowElement-svgdom-in-prop.html: Added.
  • svg/dynamic-updates/SVGFEDropShadowElement-svgdom-shadow-color-prop-expected.txt: Added.
  • svg/dynamic-updates/SVGFEDropShadowElement-svgdom-shadow-color-prop.html: Added.
  • svg/dynamic-updates/SVGFEDropShadowElement-svgdom-shadow-opacity-prop-expected.txt: Added.
  • svg/dynamic-updates/SVGFEDropShadowElement-svgdom-shadow-opacity-prop.html: Added.
  • svg/dynamic-updates/SVGFEDropShadowElement-svgdom-stdDeviation-prop-expected.txt: Added.
  • svg/dynamic-updates/SVGFEDropShadowElement-svgdom-stdDeviation-prop.html: Added.
  • svg/dynamic-updates/SVGFETurbulenceElement-svgdom-numOctaves-prop.html:
  • svg/dynamic-updates/SVGFETurbulenceElement-svgdom-seed-prop.html:
  • svg/dynamic-updates/SVGFETurbulenceElement-svgdom-stitchTiles-prop.html:
  • svg/dynamic-updates/SVGFETurbulenceElement-svgdom-type-prop.html:
  • svg/dynamic-updates/script-tests/SVGFEDropShadowElement-dom-dx-attr.js: Added. (executeTest):
  • svg/dynamic-updates/script-tests/SVGFEDropShadowElement-dom-dy-attr.js: Added. (executeTest):
  • svg/dynamic-updates/script-tests/SVGFEDropShadowElement-dom-in-attr.js: Added. (executeTest):
  • svg/dynamic-updates/script-tests/SVGFEDropShadowElement-dom-shadow-color-attr.js: Added. (executeTest):
  • svg/dynamic-updates/script-tests/SVGFEDropShadowElement-dom-shadow-opacity-attr.js: Added. (executeTest):
  • svg/dynamic-updates/script-tests/SVGFEDropShadowElement-dom-stdDeviation-attr.js: Added. (executeTest):
  • svg/dynamic-updates/script-tests/SVGFEDropShadowElement-svgdom-dx-prop.js: Added. (executeTest):
  • svg/dynamic-updates/script-tests/SVGFEDropShadowElement-svgdom-dy-prop.js: Added. (executeTest):
  • svg/dynamic-updates/script-tests/SVGFEDropShadowElement-svgdom-in-prop.js: Added. (executeTest):
  • svg/dynamic-updates/script-tests/SVGFEDropShadowElement-svgdom-shadow-color-prop.js: Added. (executeTest):
  • svg/dynamic-updates/script-tests/SVGFEDropShadowElement-svgdom-shadow-opacity-prop.js: Added. (executeTest):
  • svg/dynamic-updates/script-tests/SVGFEDropShadowElement-svgdom-stdDeviation-prop.js: Added. (executeTest):
  • svg/filters/feDropShadow.svg: Added.

2011-04-21 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVG feDropShadow implementation of SVG Filters 1.2
​https://bugs.webkit.org/show_bug.cgi?id=52513

  • MigrateHeaders.make:

2011-04-21 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVG feDropShadow implementation of SVG Filters 1.2
​https://bugs.webkit.org/show_bug.cgi?id=52513

This is the implementation of feDropShadow, a shorthand filter and a combination of feGaussianBlur,
feOffset, feFlood and feComposite to create a drop shadow of an input effect.
Creating this new filter effect makes it possible to optimize the often used workflow. ShadowBlur, with its
fast blurring algorithm, is used to blur the shadow and composite operators are responsible for combining the
source with different colored shadows.
Shadow color and opacity can be set with the properties flood-color and flood-opacity.

Tests: svg/dynamic-updates/SVGFEDropShadowElement-dom-dx-attr.html

svg/dynamic-updates/SVGFEDropShadowElement-dom-dy-attr.html
svg/dynamic-updates/SVGFEDropShadowElement-dom-in-attr.html
svg/dynamic-updates/SVGFEDropShadowElement-dom-shadow-color-attr.html
svg/dynamic-updates/SVGFEDropShadowElement-dom-shadow-opacity-attr.html
svg/dynamic-updates/SVGFEDropShadowElement-dom-stdDeviation-attr.html
svg/dynamic-updates/SVGFEDropShadowElement-svgdom-dx-prop.html
svg/dynamic-updates/SVGFEDropShadowElement-svgdom-dy-prop.html
svg/dynamic-updates/SVGFEDropShadowElement-svgdom-in-prop.html
svg/dynamic-updates/SVGFEDropShadowElement-svgdom-shadow-color-prop.html
svg/dynamic-updates/SVGFEDropShadowElement-svgdom-shadow-opacity-prop.html
svg/dynamic-updates/SVGFEDropShadowElement-svgdom-stdDeviation-prop.html
svg/filters/feDropShadow.svg

  • Android.mk:
  • CMakeLists.txt:
  • DerivedSources.cpp:
  • DerivedSources.make:
  • GNUmakefile.list.am:
  • WebCore.exp.in:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/objc/DOM.mm: (WebCore::createElementClassMap):
  • bindings/objc/DOMSVG.h:
  • page/DOMWindow.idl:
  • platform/graphics/ShadowBlur.cpp: Added missing copyright after r84410.
  • platform/graphics/ShadowBlur.h: Made blurLayerImage public.
  • platform/graphics/filters/FEDropShadow.cpp: Added. (WebCore::FEDropShadow::FEDropShadow): (WebCore::FEDropShadow::create): (WebCore::FEDropShadow::determineAbsolutePaintRect): (WebCore::FEDropShadow::apply): (WebCore::FEDropShadow::dump): (WebCore::FEDropShadow::externalRepresentation):
  • platform/graphics/filters/FEDropShadow.h: Added. (WebCore::FEDropShadow::stdDeviationX): (WebCore::FEDropShadow::setStdDeviationX): (WebCore::FEDropShadow::stdDeviationY): (WebCore::FEDropShadow::setStdDeviationY): (WebCore::FEDropShadow::dx): (WebCore::FEDropShadow::setDx): (WebCore::FEDropShadow::dy): (WebCore::FEDropShadow::setDy): (WebCore::FEDropShadow::shadowColor): (WebCore::FEDropShadow::setShadowColor): (WebCore::FEDropShadow::shadowOpacity): (WebCore::FEDropShadow::setShadowOpacity):
  • platform/graphics/filters/FEGaussianBlur.cpp: (WebCore::FEGaussianBlur::calculateKernelSize): Make kernel calculations accessible for feDropShadow.
  • platform/graphics/filters/FEGaussianBlur.h:
  • svg/SVGAllInOne.cpp:
  • svg/SVGFEDropShadowElement.cpp: Added. (WebCore::SVGFEDropShadowElement::SVGFEDropShadowElement): (WebCore::SVGFEDropShadowElement::create): (WebCore::SVGFEDropShadowElement::stdDeviationXIdentifier): (WebCore::SVGFEDropShadowElement::stdDeviationYIdentifier): (WebCore::SVGFEDropShadowElement::setStdDeviation): (WebCore::SVGFEDropShadowElement::parseMappedAttribute): (WebCore::SVGFEDropShadowElement::svgAttributeChanged): (WebCore::SVGFEDropShadowElement::synchronizeProperty): (WebCore::SVGFEDropShadowElement::build):
  • svg/SVGFEDropShadowElement.h: Added.
  • svg/SVGFEDropShadowElement.idl: Added.
  • svg/svgtags.in:
10:47 AM Changeset in webkit [84521] by morrita@google.com
  • 3 edits in trunk/LayoutTests

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

Unreviewed expectations update following r84511.

  • platform/gtk/fast/html/details-open2-expected.txt:
  • platform/qt/fast/html/details-open2-expected.txt:
10:44 AM Changeset in webkit [84520] by rolandsteiner@chromium.org
  • 18 edits
    2 adds in trunk

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

Reviewed by Dimitri Glazkov.

Bug 58704 - Decide on node type for ShadowRoot
​https://bugs.webkit.org/show_bug.cgi?id=58704

Test node types of shadow host, shadow root and first shadow child.

  • fast/dom/shadow/nodetype-expected.txt: Added.
  • fast/dom/shadow/nodetype.html: Added.

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

Reviewed by Dimitri Glazkov.

Bug 58704 - Decide on node type for ShadowRoot
​https://bugs.webkit.org/show_bug.cgi?id=58704

Add a new node type SHADOW_ROOT_NODE.
Adapt all calling sites of nodeType().

Test: fast/dom/shadow/nodetype.html

  • bindings/js/JSNodeCustom.cpp: (WebCore::createWrapperInline):
  • bindings/objc/DOM.mm: (kitClass):
  • bindings/v8/custom/V8NodeCustom.cpp: (WebCore::toV8Slow):
  • dom/Document.cpp: (WebCore::Document::importNode): (WebCore::Document::childTypeAllowed): (WebCore::Document::canReplaceChild):
  • dom/Node.cpp: (WebCore::Node::dumpStatistics): (WebCore::Node::isDefaultNamespace): (WebCore::Node::lookupPrefix): (WebCore::Node::lookupNamespaceURI): (WebCore::appendTextContent): (WebCore::Node::setTextContent):
  • dom/Node.h:
  • dom/Range.cpp: (WebCore::lengthOfContentsInNode): (WebCore::Range::processContentsBetweenOffsets): (WebCore::Range::insertNode): (WebCore::Range::checkNodeWOffset): (WebCore::Range::checkNodeBA): (WebCore::Range::selectNode): (WebCore::Range::selectNodeContents): (WebCore::Range::surroundContents):
  • dom/ShadowRoot.cpp: (WebCore::ShadowRoot::nodeType):
  • dom/Traversal.cpp: (WebCore::Traversal::acceptNode):
  • editing/MarkupAccumulator.cpp: (WebCore::MarkupAccumulator::appendStartMarkup):
  • html/parser/HTMLElementStack.cpp: (WebCore::HTMLNames::isRootNode): (WebCore::HTMLElementStack::pushRootNode):
  • html/parser/HTMLElementStack.h: (WebCore::isInHTMLNamespace):
  • xml/XPathUtil.cpp: (WebCore::XPath::isValidContextNode):

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

Reviewed by Dimitri Glazkov.

Bug 58704 - Decide on node type for ShadowRoot
​https://bugs.webkit.org/show_bug.cgi?id=58704

Add a new enum entry to WebNode::NodeType for shadow roots.
Adapt calling sites for new SHADOW_ROOT_NODE node type.

  • public/WebNode.h:
  • src/WebPageSerializerImpl.cpp: (WebKit::WebPageSerializerImpl::buildContentForNode):
10:43 AM Changeset in webkit [84519] by jberlin@webkit.org
  • 8 edits in trunk/Source/WebKit2

WebKit2: Need a way to get a snapshot of the visible contents of a page from the UI Process.
​https://bugs.webkit.org/show_bug.cgi?id=59035

Reviewed by Sam Weinig.

Added a sync call from the UI Process to the Web Process with comments saying that it should
be removed as soon as possible.

  • UIProcess/API/C/WKPage.cpp:

(WKPageVisibleContentsSnapshot):

  • UIProcess/API/C/WKPagePrivate.h:
  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::createSnapshotOfVisibleContent):

  • UIProcess/WebPageProxy.h:
  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::createSnapshotOfVisibleContent):
Ask the FrameView for the visibleContentRect, and call scaledSnapshotInDocumentCoordinates
with a scale factor of 1 (which preserves the current scaling of the page - e.g. if the
page is already zoomed then the snapshot will reflect the zoomed content).

  • WebProcess/WebPage/WebPage.h:
  • WebProcess/WebPage/WebPage.messages.in:
10:43 AM Changeset in webkit [84518] by ojan@chromium.org
  • 3 edits in trunk/Tools

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

Reviewed by Tony Chang.

put unexpected passes into their own table
​https://bugs.webkit.org/show_bug.cgi?id=59016

  • Scripts/webkitpy/layout_tests/layout_package/json_results.html:
10:37 AM Changeset in webkit [84517] by Dimitri Glazkov
  • 8 edits
    2 adds in trunk

2011-04-20 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Kent Tamura.

REGRESSION (r75543): Styles bleed into new shadow DOM (like slider and video)
​https://bugs.webkit.org/show_bug.cgi?id=52917

  • fast/css/shadow-dom-scope-expected.txt: Added.
  • fast/css/shadow-dom-scope.html: Added.

2011-04-20 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Kent Tamura.

REGRESSION (r75543): Styles bleed into new shadow DOM (like slider and video)
​https://bugs.webkit.org/show_bug.cgi?id=52917

In order to determine whether a rule should apply inside of a shadow DOM subtree,
we collect three pieces of information:

1) Are we parsing UA sheets?
2) Does the current tree scope allows applying author sheets?
3) Does the rule explicitly reach into shadow DOM (using pseudo-elements, for example)?

If the answer to all of these is no, we ignore the rule.

Test: fast/css/shadow-dom-scope.html

  • css/CSSStyleSelector.cpp: (WebCore::MatchingUARulesScope::MatchingUARulesScope): Added a scope helper to

track whether we are currently matching UA rules.

(WebCore::CSSStyleSelector::matchRulesForList): Added the check for three

conditions mentioned above.

(WebCore::CSSStyleSelector::SelectorChecker::SelectorChecker): Added

initializer for the flag, which helps us determine whether the rule
explicitly reaches into shadow DOM.

(WebCore::CSSStyleSelector::matchUARules): Started using the scope helper.
(WebCore::CSSStyleSelector::checkSelector): Added initializing of the

flag.

(WebCore::CSSStyleSelector::SelectorChecker::checkOneSelector): Added setting

of the flag when we encounter unknown pseudo-elements.

  • css/CSSStyleSelector.h: Adjusted decls.
  • dom/ShadowRoot.cpp: (WebCore::ShadowRoot::applyAuthorSheets): Added.
  • dom/ShadowRoot.h: Adjusted decls.
  • dom/TreeScope.cpp: (WebCore::TreeScope::applyAuthorSheets): Added.
  • dom/TreeScope.h: Adjusted decls.
10:24 AM Changeset in webkit [84516] by leviw@chromium.org
  • 3 edits
    4 adds in trunk

2011-04-21 Levi Weintraub <​leviw@chromium.org>

Reviewed by Simon Fraser.

position fixed element does not render properly when dynamically updated via javascript
​https://bugs.webkit.org/show_bug.cgi?id=54297

Disabling the LayoutState fast-path for fixed-position content.

Tests: fast/repaint/fixed-contents-changed-after-scroll.html

fast/repaint/transformed-contents-changed-after-scroll.html

  • rendering/RenderBox.cpp: (WebCore::RenderBox::computeRectForRepaint):

2011-04-21 Levi Weintraub <​leviw@chromium.org>

Reviewed by Simon Fraser.

position fixed element does not render properly when dynamically updated via javascript
​https://bugs.webkit.org/show_bug.cgi?id=54297

Disabling the LayoutState fast-path for fixed-position content.

  • fast/repaint/fixed-contents-changed-after-scroll.html: Added.
  • fast/repaint/transformed-contents-changed-after-scroll.html: Added.
  • platform/mac/fast/repaint/fixed-contents-changed-after-scroll-expected.checksum: Added.
  • platform/mac/fast/repaint/fixed-contents-changed-after-scroll-expected.png: Added.
  • platform/mac/fast/repaint/fixed-contents-changed-after-scroll-expected.txt: Added.
  • platform/mac/fast/repaint/transformed-contents-changed-after-scroll-expected.checksum: Added.
  • platform/mac/fast/repaint/transformed-contents-changed-after-scroll-expected.png: Added.
  • platform/mac/fast/repaint/transformed-contents-changed-after-scroll-expected.txt: Added.
10:23 AM Changeset in webkit [84515] by msaboff@apple.com
  • 3 edits in trunk/Source/JavaScriptCore

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

Reviewed by Geoff Garen.

JSString::resolveRope inefficient for common 2 fiber case
​https://bugs.webkit.org/show_bug.cgi?id=58994

Split JSString::resolveRope into three routines.
resolveRope allocates the new buffer and handles the 1 or 2
fiber case with single level fibers.
resolveRopeSlowCase handles the general case.
outOfMemory handles the rare out of memory exception case.

  • runtime/JSString.cpp: (JSC::JSString::resolveRope): (JSC::JSString::resolveRopeSlowCase): (JSC::JSString::outOfMemory):
  • runtime/JSString.h:
10:20 AM Rebaseline edited by senorblanco@chromium.org
(diff)
10:16 AM Changeset in webkit [84514] by ojan@chromium.org
  • 2 edits
    1 add in trunk/Tools

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

Reviewed by Tony Chang.

initial set of tests for new results.html file
​https://bugs.webkit.org/show_bug.cgi?id=59031

Also, fix the bugs (mostly innocuous JS errors) the tests found.

  • Scripts/webkitpy/layout_tests/layout_package/json_results.html:
  • Scripts/webkitpy/layout_tests/layout_package/json_results_test.js: Added.
10:12 AM Changeset in webkit [84513] by tony@chromium.org
  • 2 edits
    6 adds
    7 deletes in trunk/LayoutTests

2011-04-21 Tony Chang <​tony@chromium.org>

Unreviewed. Fixed the last set of tests that are missing pngs.
Created new png baselines on my Leopard machine.

  • platform/chromium-mac-leopard/fast/blockflow/english-bt-text-expected.checksum: Removed.
  • platform/chromium-mac-leopard/fast/blockflow/english-bt-text-expected.png: Added.
  • platform/chromium-mac-leopard/fast/borders/svg-as-border-image-expected.checksum: Removed.
  • platform/chromium-mac-leopard/fast/borders/svg-as-border-image-expected.png: Added.
  • platform/chromium-mac-leopard/fast/css/background-shorthand-invalid-url-expected.checksum: Removed.
  • platform/chromium-mac-leopard/fast/css/background-shorthand-invalid-url-expected.png: Added.
  • platform/chromium-mac-leopard/fast/css/value-list-out-of-bounds-crash-expected.checksum: Removed.
  • platform/chromium-mac-leopard/fast/css/value-list-out-of-bounds-crash-expected.png: Added.
  • platform/chromium-mac-leopard/svg/custom/group-opacity-expected.checksum: Removed.
  • platform/chromium-mac-leopard/svg/custom/group-opacity-expected.png: Added.
  • platform/chromium-mac-leopard/svg/custom/image-with-transform-clip-filter-expected.checksum: Removed.
  • platform/chromium-mac-leopard/svg/custom/image-with-transform-clip-filter-expected.png: Added.
  • platform/chromium-mac/editing/selection/extend-by-word-002-expected.checksum: Removed. Matches platform/mac.
  • platform/chromium/test_expectations.txt:
10:11 AM Rebaseline edited by senorblanco@chromium.org
(diff)
9:57 AM Changeset in webkit [84512] by mdelaney@apple.com
  • 3 edits
    2 adds in trunk

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

Reviewed by Dimitri Glazkov.

Content of <summary> should be forwarded through the shadow DOM
​https://bugs.webkit.org/show_bug.cgi?id=58914

  • Added test cases for dynamic DOM change.
  • Updated expectations details-open2.html because generated RenderTree slightly changed.
  • fast/html/details-add-summary-child-1.html: Added.
  • fast/html/details-add-summary-child-2.html: Added.
  • fast/html/details-remove-summary-child-1.html: Added.
  • fast/html/details-remove-summary-child-2.html: Added.
  • platform/chromium/test_expectations.txt:
  • platform/mac/fast/html/details-add-summary-child-1-expected.checksum: Added.
  • platform/mac/fast/html/details-add-summary-child-1-expected.png: Added.
  • platform/mac/fast/html/details-add-summary-child-1-expected.txt: Added.
  • platform/mac/fast/html/details-add-summary-child-2-expected.checksum: Added.
  • platform/mac/fast/html/details-add-summary-child-2-expected.png: Added.
  • platform/mac/fast/html/details-add-summary-child-2-expected.txt: Added.
  • platform/mac/fast/html/details-open2-expected.txt:
  • platform/mac/fast/html/details-remove-summary-child-1-expected.checksum: Added.
  • platform/mac/fast/html/details-remove-summary-child-1-expected.png: Added.
  • platform/mac/fast/html/details-remove-summary-child-1-expected.txt: Added.
  • platform/mac/fast/html/details-remove-summary-child-2-expected.checksum: Added.
  • platform/mac/fast/html/details-remove-summary-child-2-expected.png: Added.
  • platform/mac/fast/html/details-remove-summary-child-2-expected.txt: Added.

2011-04-20 Matthew Delaney <​mdelaney@apple.com>

Reviewed by Maciej Stachowiak.

arc() should add a circle to the path when start and end angles are far enough apart
​https://bugs.webkit.org/show_bug.cgi?id=58934

Test: fast/canvas/canvas_arc_largeangles.html

  • html/canvas/CanvasRenderingContext2D.cpp: (WebCore::CanvasRenderingContext2D::arc):
9:51 AM Changeset in webkit [84511] by morrita@google.com
  • 14 edits
    1 copy
    16 adds in trunk

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

Reviewed by Dimitri Glazkov.

Content of <summary> should be forwarded through the shadow DOM
​https://bugs.webkit.org/show_bug.cgi?id=58914

  • Added test cases for dynamic DOM change.
  • Updated expectations details-open2.html because generated RenderTree slightly changed.
  • fast/html/details-add-summary-child-1.html: Added.
  • fast/html/details-add-summary-child-2.html: Added.
  • fast/html/details-remove-summary-child-1.html: Added.
  • fast/html/details-remove-summary-child-2.html: Added.
  • platform/chromium/test_expectations.txt:
  • platform/mac/fast/html/details-add-summary-child-1-expected.checksum: Added.
  • platform/mac/fast/html/details-add-summary-child-1-expected.png: Added.
  • platform/mac/fast/html/details-add-summary-child-1-expected.txt: Added.
  • platform/mac/fast/html/details-add-summary-child-2-expected.checksum: Added.
  • platform/mac/fast/html/details-add-summary-child-2-expected.png: Added.
  • platform/mac/fast/html/details-add-summary-child-2-expected.txt: Added.
  • platform/mac/fast/html/details-open2-expected.txt:
  • platform/mac/fast/html/details-remove-summary-child-1-expected.checksum: Added.
  • platform/mac/fast/html/details-remove-summary-child-1-expected.png: Added.
  • platform/mac/fast/html/details-remove-summary-child-1-expected.txt: Added.
  • platform/mac/fast/html/details-remove-summary-child-2-expected.checksum: Added.
  • platform/mac/fast/html/details-remove-summary-child-2-expected.png: Added.
  • platform/mac/fast/html/details-remove-summary-child-2-expected.txt: Added.

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

Reviewed by Dimitri Glazkov.

Content of <summary> should be forwarded through the shadow DOM
​https://bugs.webkit.org/show_bug.cgi?id=58914

  • Introduced ShadowContentElement which hosts forwarded children of <summary>
  • Encapsulated Node::parentNodeForRenderingAndStyle() and shouldCreateRendererFor() into NodeVisualParentLookupResult class.
  • The parent lookup is also aware of node forwarding. If the visual parent node has a shadow root, the node is possibly forwarded to ShadowContentElement
  • ShadowContentElement lookup is implemented inside ShadowRoot.

Tests: fast/html/details-add-summary-child-1.html

fast/html/details-add-summary-child-2.html
fast/html/details-remove-summary-child-1.html
fast/html/details-remove-summary-child-2.html

  • GNUmakefile.list.am:
  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • dom/Element.cpp: (WebCore::Element::childrenChanged):
  • dom/Node.cpp: (WebCore::NodeRenderParentDetector::NodeRenderParentDetector): (WebCore::NodeRenderParentDetector::parentNodeForRenderingAndStyle): (WebCore::NodeRenderParentDetector::findVisualParent): (WebCore::NodeRenderParentDetector::shouldCreateRenderer): (WebCore::Node::parentNodeForRenderingAndStyle): (WebCore::Node::createRendererAndStyle):
  • dom/ShadowRoot.cpp: (WebCore::ShadowRoot::contentContainerFor): (WebCore::ShadowRoot::hostChildrenChanged): (WebCore::ShadowRoot::firstContentElement):
  • dom/ShadowRoot.h:
  • html/HTMLSummaryElement.cpp: (WebCore::HTMLSummaryElement::createShadowSubtree):
  • html/HTMLSummaryElement.h:
  • html/shadow/ShadowContentElement.h: Copied from Source/WebCore/dom/ShadowRoot.h. (WebCore::ShadowContentElement::ShadowContentElement): (WebCore::ShadowContentElement::isShadowBoundary): (WebCore::ShadowContentElement::createRenderer): (WebCore::ShadowContentElement::create):
9:40 AM Changeset in webkit [84510] by caseq@chromium.org
  • 6 edits in trunk

011-04-21 Andrey Kosyakov <​caseq@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: fix calls to WebInspector.ObjectPropertiesSection constructor
​https://bugs.webkit.org/show_bug.cgi?id=59105

  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleView.prototype._formatobject):
  • inspector/front-end/ExtensionPanel.js: (WebInspector.ExtensionSidebarPane.prototype._setObject):
  • inspector/front-end/ResourceJSONView.js: (WebInspector.ResourceJSONView.parseJSON.WebInspector.ResourceJSONView.prototype._initialize):
  • inspector/front-end/SourceFrame.js: (WebInspector.SourceFrame.prototype._showPopup.showObjectPopup): (WebInspector.SourceFrame.prototype._showPopup):
9:38 AM Changeset in webkit [84509] by mdelaney@apple.com
  • 1 edit
    1 move in trunk/LayoutTests

2011-04-20 Matthew Delaney <​mdelaney@apple.com>

Reviewed by Adam Roben.

Move toDataURL.jpeg.alpha expected result out of /mac-snowleopard and into /mac
​https://bugs.webkit.org/show_bug.cgi?id=59038

  • platform/mac-snowleopard/canvas/philip/tests/toDataURL.jpeg.alpha-expected.txt: Removed.
  • platform/mac/canvas/philip/tests/toDataURL.jpeg.alpha-expected.txt: Copied from LayoutTests/platform/mac-snowleopard/canvas/philip/tests/toDataURL.jpeg.alpha-expected.txt.
9:32 AM Changeset in webkit [84508] by Philippe Normand
  • 7 edits in trunk

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

Reviewed by Martin Robinson.

[GTK] missing LayoutTestController::callShouldCloseOnWebView()
​https://bugs.webkit.org/show_bug.cgi?id=59086

  • platform/gtk/Skipped: Unskip fast/events/onbeforeunload-focused-iframe.html

callShouldCloseOnWebView() implementation.

  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (LayoutTestController::callShouldCloseOnWebView):

DRT::shouldClose(), used by the LayoutController.

  • WebCoreSupport/DumpRenderTreeSupportGtk.cpp: (DumpRenderTreeSupportGtk::shouldClose):
  • WebCoreSupport/DumpRenderTreeSupportGtk.h:
9:24 AM Changeset in webkit [84507] by Philippe Normand
  • 7 edits
    3 adds in trunk

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

Reviewed by Martin Robinson.

[GTK] DRT: implement eventSender.scalePageBy
​https://bugs.webkit.org/show_bug.cgi?id=59082

Unskipped tests that should pass now that
eventSender.scalePageBy() is supported in GTK.

  • platform/gtk/Skipped:
  • platform/gtk/fast/repaint/scale-page-shrink-expected.checksum: Added.
  • platform/gtk/fast/repaint/scale-page-shrink-expected.png: Added.
  • platform/gtk/fast/repaint/scale-page-shrink-expected.txt: Added.
  • WebCoreSupport/DumpRenderTreeSupportGtk.cpp: (DumpRenderTreeSupportGtk::scalePageBy):
  • WebCoreSupport/DumpRenderTreeSupportGtk.h:
  • DumpRenderTree/gtk/EventSender.cpp: (scalePageByCallback): scalePageBy() support.
9:18 AM Changeset in webkit [84506] by abarth@webkit.org
  • 4 edits in trunk

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

Update results to show failure.

  • http/tests/security/contentSecurityPolicy/frame-src-blocked-expected.txt:

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

Remove support for CSP's frame-src. This approach causes too many
extra console messages to be logged. I'm leaving in the test (which
now shows failure).

  • html/HTMLFrameElementBase.cpp: (WebCore::HTMLFrameElementBase::isURLAllowed):
8:58 AM Changeset in webkit [84505] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-04-21 Eunsol Park <​eunsol47.park@samsung.com>

Reviewed by Antonio Gomes.

[EFL] Scroll doesn't work on tiled backing store.
​https://bugs.webkit.org/show_bug.cgi?id=55021

It was added from Changeset 72242, but it worked wrong in EFL port.
The visible rect should not be returned as content rect for scrolling in EFL tiled backing store,
because it works differently from QT.

  • platform/ScrollView.cpp: (WebCore::ScrollView::visibleContentRect):
8:55 AM Changeset in webkit [84504] by Simon Fraser
  • 45 edits in trunk/Source/WebCore

2011-04-21 Simon Fraser <Simon Fraser>

Reviewed by Eric Seidel.

Create RIAA class for GraphicsContext::save() and restore()
​https://bugs.webkit.org/show_bug.cgi?id=58807

Use GraphicsContextStateSaver in a lot more places.

  • html/canvas/CanvasRenderingContext2D.cpp: (WebCore::CanvasRenderingContext2D::drawImage): (WebCore::CanvasRenderingContext2D::drawTextInternal):
  • platform/ScrollView.cpp: (WebCore::ScrollView::wheelEvent):
  • platform/graphics/GeneratedImage.cpp: (WebCore::GeneratedImage::draw):
  • platform/graphics/GraphicsContext.h: (WebCore::GraphicsContextStateSaver::save): (WebCore::GraphicsContextStateSaver::restore):
  • platform/graphics/ShadowBlur.cpp: (WebCore::ShadowBlur::drawShadowBuffer): (WebCore::ShadowBlur::drawRectShadowWithoutTiling): (WebCore::ShadowBlur::drawInsetShadowWithoutTiling): (WebCore::ShadowBlur::drawInsetShadowWithTiling): (WebCore::ShadowBlur::drawRectShadowWithTiling): (WebCore::ShadowBlur::drawLayerPieces):
  • platform/graphics/avfoundation/MediaPlayerPrivateAVFoundationObjC.mm: (WebCore::MediaPlayerPrivateAVFoundationObjC::paint):
  • platform/graphics/cg/ImageCG.cpp: (WebCore::BitmapImage::draw): (WebCore::Image::drawPattern):
  • platform/graphics/cg/PDFDocumentImage.cpp: (WebCore::PDFDocumentImage::draw):
  • platform/graphics/filters/FEComposite.cpp: (WebCore::FEComposite::apply):
  • platform/graphics/filters/SourceAlpha.cpp: (WebCore::SourceAlpha::apply):
  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivateQTKit::paint):
  • platform/mac/ScrollbarThemeMac.mm: (WebCore::ScrollbarThemeMac::paint):
  • platform/mac/ThemeMac.mm: (WebCore::paintCheckbox): (WebCore::paintRadio): (WebCore::paintStepper):
  • rendering/EllipsisBox.cpp: (WebCore::EllipsisBox::paintSelection):
  • rendering/InlineFlowBox.cpp: (WebCore::InlineFlowBox::paintFillLayer): (WebCore::InlineFlowBox::paintBoxDecorations): (WebCore::InlineFlowBox::paintMask):
  • rendering/InlineTextBox.cpp: (WebCore::InlineTextBox::paint): (WebCore::InlineTextBox::paintSelection): (WebCore::InlineTextBox::paintCompositionBackground): (WebCore::InlineTextBox::paintDecoration): (WebCore::InlineTextBox::paintTextMatchMarker):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::paintColumnContents): (WebCore::RenderBlock::paintSelection):
  • rendering/RenderBox.cpp: (WebCore::RenderBox::paintBoxDecorationsWithSize):
  • rendering/RenderBoxModelObject.cpp: (WebCore::RenderBoxModelObject::paintFillLayerExtended): (WebCore::RenderBoxModelObject::paintBorder):
  • rendering/RenderEmbeddedObject.cpp: (WebCore::RenderEmbeddedObject::paintReplaced):
  • rendering/RenderFieldset.cpp: (WebCore::RenderFieldset::paintBoxDecorations):
  • rendering/RenderFileUploadControl.cpp: (WebCore::RenderFileUploadControl::paintObject):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::paintResizer): (WebCore::RenderLayer::paintLayer): (WebCore::RenderLayer::paintChildLayerIntoColumns):
  • rendering/RenderListMarker.cpp: (WebCore::RenderListMarker::paint):
  • rendering/RenderMediaControls.cpp: (WebCore::RenderMediaControls::paintMediaControlsPart):
  • rendering/RenderTableCell.cpp: (WebCore::RenderTableCell::paintBackgroundsBehindCell):
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::paintPlaceholder):
  • rendering/RenderThemeMac.mm: (WebCore::RenderThemeMac::paintMenuList): (WebCore::RenderThemeMac::paintMeter): (WebCore::RenderThemeMac::paintProgressBar): (WebCore::RenderThemeMac::paintMenuListButtonGradients): (WebCore::RenderThemeMac::paintMenuListButton): (WebCore::RenderThemeMac::paintSliderTrack): (WebCore::RenderThemeMac::paintSliderThumb): (WebCore::RenderThemeMac::paintSearchField): (WebCore::RenderThemeMac::paintSearchFieldCancelButton): (WebCore::RenderThemeMac::paintSearchFieldResultsButton): (WebCore::RenderThemeMac::paintMediaSliderTrack): (WebCore::RenderThemeMac::paintMediaCurrentTime): (WebCore::RenderThemeMac::paintMediaTimeRemaining):
  • rendering/mathml/RenderMathMLBlock.cpp: (WebCore::RenderMathMLBlock::paint):
  • rendering/mathml/RenderMathMLFraction.cpp: (WebCore::RenderMathMLFraction::paint):
  • rendering/mathml/RenderMathMLRoot.cpp: (WebCore::RenderMathMLRoot::paint):
  • rendering/mathml/RenderMathMLSquareRoot.cpp: (WebCore::RenderMathMLSquareRoot::paint):
  • rendering/svg/RenderSVGContainer.cpp: (WebCore::RenderSVGContainer::paint):
  • rendering/svg/RenderSVGForeignObject.cpp: (WebCore::RenderSVGForeignObject::paint):
  • rendering/svg/RenderSVGImage.cpp: (WebCore::RenderSVGImage::paint):
  • rendering/svg/RenderSVGPath.cpp: (WebCore::RenderSVGPath::fillAndStrokePath): (WebCore::RenderSVGPath::paint):
  • rendering/svg/RenderSVGResourceClipper.cpp: (WebCore::RenderSVGResourceClipper::applyClippingToContext):
  • rendering/svg/RenderSVGResourceMarker.cpp: (WebCore::RenderSVGResourceMarker::draw):
  • rendering/svg/RenderSVGText.cpp: (WebCore::RenderSVGText::paint):
  • rendering/svg/SVGInlineFlowBox.cpp: (WebCore::SVGInlineFlowBox::paint):
  • rendering/svg/SVGInlineTextBox.cpp: (WebCore::SVGInlineTextBox::paintSelectionBackground): (WebCore::SVGInlineTextBox::paint): (WebCore::SVGInlineTextBox::paintDecorationWithStyle):
  • rendering/svg/SVGRootInlineBox.cpp: (WebCore::SVGRootInlineBox::paint):
  • svg/SVGFont.cpp: (WebCore::Font::drawTextUsingSVGFont):
  • svg/graphics/SVGImage.cpp: (WebCore::SVGImage::draw):
8:39 AM Changeset in webkit [84503] by tonyg@chromium.org
  • 2 edits in trunk/LayoutTests

2011-04-21 Tony Gentilcore <​tonyg@chromium.org>

[chromium] inspector/console/console-log-before-inspector-open.html times out on mac
​https://bugs.webkit.org/show_bug.cgi?id=59106

Seems to have started timing out in 84470-84471 which were expectations
updates.

  • platform/chromium/test_expectations.txt:
8:38 AM Changeset in webkit [84502] by abarth@webkit.org
  • 5 edits in trunk/LayoutTests

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

Reviewed by Tony Gentilcore.

Make test robust to PHP magic quotes
​https://bugs.webkit.org/show_bug.cgi?id=59104

  • http/tests/security/contentSecurityPolicy/report-uri-expected.txt:
  • http/tests/security/contentSecurityPolicy/report-uri-from-child-frame-expected.txt:
  • http/tests/security/contentSecurityPolicy/resources/save-report.php:
8:21 AM Changeset in webkit [84501] by yi.4.shen@nokia.com
  • 2 edits in trunk/Source/WebKit/qt

2011-04-21 Yi Shen <​yi.4.shen@nokia.com>

Reviewed by Antonio Gomes.

[Qt] Fix the style issue for Api/qwebkitplatformplugin.h
​https://bugs.webkit.org/show_bug.cgi?id=59097

Fix the style issues for Api/qwebkitplatformplugin.h found by the webkit-check-style.

  • Api/qwebkitplatformplugin.h:
8:07 AM Changeset in webkit [84500] by tonyg@chromium.org
  • 2 edits in trunk/LayoutTests

2011-04-21 Tony Gentilcore <​tonyg@chromium.org>

[chromium] Temporarily skip new contentSecurityPolicy tests
​https://bugs.webkit.org/show_bug.cgi?id=59103

Tests introduced by ​http://trac.webkit.org/changeset/84478

  • platform/chromium/test_expectations.txt:
7:59 AM Changeset in webkit [84499] by caseq@chromium.org
  • 5 edits in trunk

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

Reviewed by Pavel Feldman.

Web Inspector: [Extensions API] console API is not exposed to code evaluated by sidebar.setExpression()
​https://bugs.webkit.org/show_bug.cgi?id=59006

  • inspector/front-end/ExtensionPanel.js: (WebInspector.ExtensionSidebarPane.prototype.setExpression):

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

Reviewed by Pavel Feldman.

Expose console API to the code evaluated by sidebar.setExpression()

Web Inspector: [Extensions API] console API is not exposed to code evaluated by sidebar.setExpression()
​https://bugs.webkit.org/show_bug.cgi?id=59006

  • inspector/extensions/extensions-expected.txt:
  • inspector/extensions/extensions.html:
7:22 AM Changeset in webkit [84498] by Nikolas Zimmermann
  • 15 edits
    2 copies in trunk/Source/WebCore

2011-04-21 Nikolas Zimmermann <​nzimmermann@rim.com>

Reviewed by Dirk Schulze.

Split SVGGlyphIdentifier class out of SVGGlyphElement.h
​https://bugs.webkit.org/show_bug.cgi?id=59101

Move SVGGlyphIdentifier class out of SVGGlyphElement.h, place it in platform/graphics/SVGGlyph.h
and rename the contained struct to SVGGlyph. It will be used in follow-up patches that will integrate SVGGlyph & GlyphPage.

Several utility functions are moved from SVGFont.cpp to SVGGlyph.cpp, as that's the place where they really belong to.
Renamed nameLength to unicodeStringLength in the SVGGlyph struct for clarity.

  • Android.mk: Add SVGGlyph.cpp/h to build.
  • CMakeLists.txt: Ditto.
  • GNUmakefile.list.am: Ditto.
  • WebCore.gypi: Ditto.
  • WebCore.pro: Ditto.
  • WebCore.vcproj/WebCore.vcproj: Ditto.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • platform/graphics/SVGGlyph.cpp: Copied from svg/SVGFont.cpp. Removed everything except the methods listed below, that belong here. (WebCore::processArabicFormDetection): (WebCore::charactersWithArabicForm): (WebCore::isCompatibleArabicForm): (WebCore::isCompatibleGlyph):
  • platform/graphics/SVGGlyph.h: Copied from svg/SVGGlyphElement.h. Renamed SVGGlyphIdentifier to SVGGlyph. (WebCore::SVGGlyph::SVGGlyph): (WebCore::SVGGlyph::operator==):
  • platform/graphics/SimpleFontData.cpp: Adapt for SVGGlyphIdentifier -> SVGGlyph and nameLength -> unicodeStringLength rename. (WebCore::SimpleFontData::SimpleFontData):
  • svg/SVGFont.cpp: Ditto. Remove helper functions that are now living in SVGGlyph.cpp (WebCore::SVGTextRunWalker::walk): (WebCore::floatWidthUsingSVGFontCallback): (WebCore::drawTextUsingSVGFontCallback): (WebCore::drawTextMissingGlyphCallback): (WebCore::Font::drawTextUsingSVGFont):
  • svg/SVGFontElement.cpp: Ditto. (WebCore::SVGFontElement::getGlyphIdentifiersForString):
  • svg/SVGFontElement.h: Ditto.
  • svg/SVGGlyphElement.cpp: Ditto. (WebCore::parseArabicForm): (WebCore::parseOrientation): (WebCore::SVGGlyphElement::inheritUnspecifiedAttributes): (WebCore::parseSVGGlyphAttribute): (WebCore::SVGGlyphElement::buildGenericGlyphIdentifier): (WebCore::SVGGlyphElement::buildGlyphIdentifier):
  • svg/SVGGlyphElement.h: Ditto.
  • svg/SVGGlyphMap.h: Ditto. (WebCore::SVGGlyphMap::add): (WebCore::SVGGlyphMap::compareGlyphPriority): (WebCore::SVGGlyphMap::get):
7:10 AM Changeset in webkit [84497] by eric.carlson@apple.com
  • 4 edits in trunk/Source/WebCore

2011-04-21 Eric Carlson <​eric.carlson@apple.com>

Reviewed by Dan Bernstein.

Live streams won't play with AVFoundation backend
​https://bugs.webkit.org/show_bug.cgi?id=58961
rdar://problem/9284748

Tested manually because we don't have any live streams to test in DRT.

  • platform/graphics/avfoundation/MediaPlayerPrivateAVFoundation.cpp: (WebCore::MediaPlayerPrivateAVFoundation::seek): Do nothing when asked to seek to the

current time.

(WebCore::MediaPlayerPrivateAVFoundation::updateStates): If the readyState is HaveEnoughData,

don't revert when the new state is "ready to play" or "buffer full".

(WebCore::MediaPlayerPrivateAVFoundation::movieLoadType): Use new convenience function isLiveStream.

  • platform/graphics/avfoundation/MediaPlayerPrivateAVFoundation.h: (WebCore::MediaPlayerPrivateAVFoundation::isLiveStream): New.
  • platform/graphics/avfoundation/MediaPlayerPrivateAVFoundationObjC.mm: (WebCore::MediaPlayerPrivateAVFoundationObjC::tracksChanged): Don't use AVAsset's

tracksWithMediaCharacteristic because it doesn't give us what we need with live
streams.

7:08 AM Changeset in webkit [84496] by tonyg@chromium.org
  • 2 edits in trunk/LayoutTests

2011-04-21 Tony Gentilcore <​tonyg@chromium.org>

[chromium] Correct expecations for editing/selection/designmode-no-caret.html
​https://bugs.webkit.org/show_bug.cgi?id=59088

Straighten out bad override in designmode-no-caret.html

  • platform/chromium/test_expectations.txt:
6:51 AM Changeset in webkit [84495] by pfeldman@chromium.org
  • 2 edits in trunk/Source/WebCore

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

Reviewed by Yury Semikhatsky.

Web Inspector: Cookies section in empty on page re-launch.
​https://bugs.webkit.org/show_bug.cgi?id=59095

  • inspector/front-end/ResourcesPanel.js: (WebInspector.ResourcesPanel.prototype._reset): (WebInspector.ResourcesPanel.prototype._frameNavigated):
6:49 AM Changeset in webkit [84494] by pfeldman@chromium.org
  • 3 edits in trunk/Source/WebCore

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

Reviewed by Yury Semikhatsky.

Web Inspector: rename De-obfuscate to Pretty print.
​https://bugs.webkit.org/show_bug.cgi?id=59093

  • English.lproj/localizedStrings.js:
  • inspector/front-end/SourceFrame.js: (WebInspector.SourceFrame.prototype._populateTextAreaContextMenu):
6:48 AM Changeset in webkit [84493] by pfeldman@chromium.org
  • 3 edits in trunk/Source/WebCore

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

Reviewed by Yury Semikhatsky.

Web Inspector: following Image URL link in the network panel does nothing.
​https://bugs.webkit.org/show_bug.cgi?id=59092

  • inspector/front-end/ImageView.js: (WebInspector.ImageView.prototype._createContentIfNeeded.onImageLoad): (WebInspector.ImageView.prototype._createContentIfNeeded):
  • inspector/front-end/inspector.js: (WebInspector.linkifyURLAsNode):
6:46 AM Changeset in webkit [84492] by pfeldman@chromium.org
  • 6 edits in trunk/Source/WebCore

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

Reviewed by Yury Semikhatsky.

Web Inspector: implement Go To Line for network panel.
​https://bugs.webkit.org/show_bug.cgi?id=59090

  • inspector/front-end/GoToLineDialog.js: (WebInspector.GoToLineDialog.prototype._highlightSelectedLine):
  • inspector/front-end/NetworkItemView.js: (WebInspector.NetworkItemView.prototype._tabSelected): (WebInspector.NetworkItemView.prototype._installHighlightSupport):
  • inspector/front-end/NetworkPanel.js: (WebInspector.NetworkPanel):
  • inspector/front-end/TabbedPane.js: (WebInspector.TabbedPane.prototype.appendTab): (WebInspector.TabbedPane.prototype.selectTab):
  • inspector/front-end/TextViewer.js: (WebInspector.TextViewer.prototype.highlightLine):
6:45 AM Changeset in webkit [84491] by Adam Roben
  • 2 edits in trunk/LayoutTests

Skip some newly-failing tests on Windows

<​http://webkit.org/b/59098> and <​http://webkit.org/b/59099> track the failures.

  • platform/win/Skipped: Added fast/transforms/selection-bounds-in-transformed-view.html and

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

6:34 AM Changeset in webkit [84490] by Nikolas Zimmermann
  • 6 edits
    1 copy in trunk/Source/WebCore

2011-04-21 Nikolas Zimmermann <​nzimmermann@rim.com>

Reviewed by Dirk Schulze.

Split GlyphPage from GlyphPageTreeNode.h into its own header
​https://bugs.webkit.org/show_bug.cgi?id=59096

  • GNUmakefile.list.am: Add GlyphPage.h to build.
  • WebCore.gypi: Ditto.
  • WebCore.vcproj/WebCore.vcproj: Ditto.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • platform/graphics/GlyphPage.h: Copied from platform/graphics/GlyphPageTreeNode.h. Removed GlyphPageTreeNode class, leaving only GlyphPage.
  • platform/graphics/GlyphPageTreeNode.h: Removed GlyphPage class.
6:07 AM Changeset in webkit [84489] by Nikolas Zimmermann
  • 8 edits
    1 add in trunk/Source/WebCore

2011-04-21 Nikolas Zimmermann <​nzimmermann@rim.com>

Reviewed by Dirk Schulze.

Centralize typedef unsigned short Glyph in Glyph.h
​https://bugs.webkit.org/show_bug.cgi?id=59087

  • GNUmakefile.list.am: Add Glyph.h to build.
  • WebCore.gypi: Ditto.
  • WebCore.vcproj/WebCore.vcproj: Ditto.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • platform/graphics/Glyph.h: Added.
  • platform/graphics/GlyphBuffer.h: Include Glyph.h instead of duplicating the Glyph typedef.
  • platform/graphics/GlyphMetricsMap.h: Ditto.
  • platform/graphics/GlyphPageTreeNode.h: Ditto.
5:45 AM Changeset in webkit [84488] by tonyg@chromium.org
  • 2 edits in trunk/LayoutTests

2011-04-21 Tony Gentilcore <​tonyg@chromium.org>

BitmapImage::destroyMetadataAndNotify should clear m_checkedForSolidColor
​https://bugs.webkit.org/show_bug.cgi?id=58926

Remove chromium's crashing expectations.

  • platform/chromium/test_expectations.txt:
5:33 AM Changeset in webkit [84487] by tonyg@chromium.org
  • 2 edits in trunk/LayoutTests

2011-04-21 Tony Gentilcore <​tonyg@chromium.org>

[chromium] Correct expecations for editing/selection/designmode-no-caret.html
​https://bugs.webkit.org/show_bug.cgi?id=59088

​http://trac.webkit.org/changeset/84470 seemed to have caused a conflict. This
has always been failing on Win Debug.

  • platform/chromium/test_expectations.txt:
5:28 AM Changeset in webkit [84486] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt][WK2] Unreviewed.

REGRESSION (WK2): Animated GIF stops when navigating forward, back
​https://bugs.webkit.org/show_bug.cgi?id=59042

  • platform/qt-wk2/Skipped: Add fast/images/animated-gif-restored-from-bfcache.html,

because WebKit2 since it doesn't implement layoutTestController.overridePreference().

5:24 AM Changeset in webkit [84485] by pfeldman@chromium.org
  • 2 edits in trunk/Source/WebCore

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

Reviewed by Yury Semikhatsky.

Web Inspector: toolbar background is blank on Mac Chromium port.
​https://bugs.webkit.org/show_bug.cgi?id=59089

  • inspector/front-end/inspector.css: (body.detached.platform-mac-snowleopard:not(.remote) #toolbar):
5:05 AM Changeset in webkit [84484] by pfeldman@chromium.org
  • 9 edits in trunk/Source/WebCore

2011-04-21 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: add support for Go To Line in Resources panel.
​https://bugs.webkit.org/show_bug.cgi?id=59077

  • inspector/front-end/CallStackSidebarPane.js: (WebInspector.CallStackSidebarPane.prototype.registerShortcuts):
  • inspector/front-end/GoToLineDialog.js: (WebInspector.GoToLineDialog.show): (WebInspector.GoToLineDialog.createShortcut): (WebInspector.GoToLineDialog.prototype._onKeyDown):
  • inspector/front-end/NetworkPanel.js: (WebInspector.NetworkPanel.prototype.handleShortcut):
  • inspector/front-end/Panel.js: (WebInspector.Panel): (WebInspector.Panel.prototype._restoreScrollPositions): (WebInspector.Panel.prototype.handleShortcut): (WebInspector.Panel.prototype.registerShortcuts): (WebInspector.Panel.prototype._showGoToLineDialog):
  • inspector/front-end/ProfilesPanel.js: (WebInspector.ProfilesPanel.prototype.handleShortcut):
  • inspector/front-end/ResourcesPanel.js: (WebInspector.ResourcesPanel):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype._registerShortcuts): (WebInspector.ScriptsPanel.prototype.jumpToPreviousSearchResult):
  • inspector/front-end/inspector.js: (WebInspector._registerShortcuts): (WebInspector.documentKeyDown):
4:43 AM Changeset in webkit [84483] by pfeldman@chromium.org
  • 5 edits in trunk/Source/WebCore

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

Reviewed by Yury Semikhatsky.

Web Inspector: persist local resource history between inspector / browser launches.
​https://bugs.webkit.org/show_bug.cgi?id=58993

  • inspector/front-end/Resource.js: (WebInspector.Resource.prototype._restoreRevisions):
3:51 AM Changeset in webkit [84482] by Philippe Normand
  • 1 edit
    3 adds in trunk/LayoutTests

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

Unreviewed, GTK baselines for test added in r84398.

  • platform/gtk/fast/borders/bidi-012-expected.checksum: Added.
  • platform/gtk/fast/borders/bidi-012-expected.png: Added.
  • platform/gtk/fast/borders/bidi-012-expected.txt: Added.
2:53 AM Changeset in webkit [84481] by apavlov@chromium.org
  • 12 edits in trunk/Source/WebCore

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

Reviewed by Pavel Feldman.

Web Inspector: Metrics pane editing and visual feedback improvements
​https://bugs.webkit.org/show_bug.cgi?id=58992

Added visual feedback when hovering over the box model components of the Metrics pane
and handling of the [Page]Up/Down keys when editing the box model values.

  • inspector/DOMNodeHighlighter.cpp: (WebCore::DOMNodeHighlighter::DrawNodeHighlight):
  • inspector/DOMNodeHighlighter.h:
  • inspector/Inspector.json:
  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::mouseDidMoveOverElement): (WebCore::InspectorDOMAgent::highlight): (WebCore::InspectorDOMAgent::highlightDOMNode): (WebCore::InspectorDOMAgent::highlightFrame): (WebCore::InspectorDOMAgent::drawNodeHighlight):
  • inspector/InspectorDOMAgent.h:
  • inspector/front-end/CSSStyleModel.js: (WebInspector.CSSProperty.prototype.setText.callback): (WebInspector.CSSProperty.prototype.setText): (WebInspector.CSSProperty.prototype.setValue): (WebInspector.CSSProperty.prototype.setDisabled.callback): (WebInspector.CSSProperty.prototype.setDisabled):
  • inspector/front-end/ElementsPanel.js: (WebInspector.ElementsPanel.prototype.startEditingStyle): (WebInspector.ElementsPanel.prototype.endEditingStyle): (WebInspector.ElementsPanel.prototype._attributesUpdated):
  • inspector/front-end/MetricsSidebarPane.js: (WebInspector.MetricsSidebarPane): (WebInspector.MetricsSidebarPane.prototype._highlightDOMNode): (WebInspector.MetricsSidebarPane.prototype._highlightDOMNode.getBoxRectangleElement): (WebInspector.MetricsSidebarPane.prototype._update.createBoxPartElement): (WebInspector.MetricsSidebarPane.prototype._update): (WebInspector.MetricsSidebarPane.prototype.startEditing): (WebInspector.MetricsSidebarPane.prototype._handleKeyDown): (WebInspector.MetricsSidebarPane.prototype.editingEnded): (WebInspector.MetricsSidebarPane.prototype.editingCancelled): (WebInspector.MetricsSidebarPane.prototype._applyUserInput.callback): (WebInspector.MetricsSidebarPane.prototype._applyUserInput): (WebInspector.MetricsSidebarPane.prototype.editingCommitted):
  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylesSidebarPane.alteredFloatNumber): (WebInspector.StylesSidebarPane.alteredHexNumber): (WebInspector.StylePropertyTreeElement.prototype): (WebInspector.StylePropertyTreeElement.prototype.element.userInput.previousContent.context.moveDirection):
  • inspector/front-end/inspector.css: (.metrics .hovered > .label): (.metrics .position): (.metrics .margin): (.metrics .margin.hovered): (.metrics .border): (.metrics .border.hovered): (.metrics .padding): (.metrics .padding.hovered): (.metrics .content): (.metrics .content.hovered):
  • inspector/front-end/inspector.js: (WebInspector.highlightDOMNode):
2:39 AM Changeset in webkit [84480] by Philippe Normand
  • 1 edit
    3 adds in trunk/LayoutTests

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

Unreviewed, GTK baselines for
http/tests/security/contentSecurityPolicy tests that don't escape
the quote character.

  • platform/gtk/http/tests/security/contentSecurityPolicy/report-uri-expected.txt: Added.
  • platform/gtk/http/tests/security/contentSecurityPolicy/report-uri-from-child-frame-expected.txt: Added.
1:50 AM Changeset in webkit [84479] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt] Missing eventSender.scalePageBy() implementation

  • platform/qt/Skipped: Add fast/transforms/selection-bounds-in-transformed-view.html.
1:34 AM Changeset in webkit [84478] by abarth@webkit.org
  • 6 edits
    10 adds in trunk

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

Reviewed by Eric Seidel.

CSP report-uri is missing
​https://bugs.webkit.org/show_bug.cgi?id=58639

Test that report-uri sends a report. Testing approach stolen from the
ping tests.

  • http/tests/security/contentSecurityPolicy/report-uri-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/report-uri.html: Added.
  • http/tests/security/contentSecurityPolicy/resources/echo-report.php: Added.
  • http/tests/security/contentSecurityPolicy/resources/go-to-echo-report.js: Added.
  • http/tests/security/contentSecurityPolicy/resources/save-report.php: Added.

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

Reviewed by Eric Seidel.

CSP report-uri is missing
​https://bugs.webkit.org/show_bug.cgi?id=58639

Our implementation of report-uri differs from what's currently in the
spec. I sent the working group an email explaining why. Generally,
we're using normal form encoding instead of JSON and we're sending less
information to a wider set of URLs. Specifically, we send the current
document's URL as well as the directive that was violated. The spec
(currently) tells us to send the raw HTTP headers and the URL that
caused the violation, but both of these pieces of data could contain
information that's sensitive, so we omit them for now.

Test: http/tests/security/contentSecurityPolicy/report-uri.html

  • loader/PingLoader.cpp: (WebCore::PingLoader::reportContentSecurityPolicyViolation):
  • loader/PingLoader.h:
  • page/ContentSecurityPolicy.cpp: (WebCore::CSPDirective::CSPDirective): (WebCore::CSPDirective::text): (WebCore::ContentSecurityPolicy::reportViolation): (WebCore::ContentSecurityPolicy::allowJavaScriptURLs): (WebCore::ContentSecurityPolicy::allowInlineEventHandlers): (WebCore::ContentSecurityPolicy::allowInlineScript): (WebCore::ContentSecurityPolicy::allowEval): (WebCore::ContentSecurityPolicy::allowScriptFromSource): (WebCore::ContentSecurityPolicy::allowObjectFromSource): (WebCore::ContentSecurityPolicy::allowChildFrameFromSource): (WebCore::ContentSecurityPolicy::allowImageFromSource): (WebCore::ContentSecurityPolicy::allowStyleFromSource): (WebCore::ContentSecurityPolicy::allowFontFromSource): (WebCore::ContentSecurityPolicy::allowMediaFromSource): (WebCore::ContentSecurityPolicy::parseReportURI): (WebCore::ContentSecurityPolicy::addDirective):
  • page/ContentSecurityPolicy.h:
1:07 AM Changeset in webkit [84477] by Philippe Normand
  • 2 edits in trunk/LayoutTests

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

Unreviewed, skip
fast/transforms/selection-bounds-in-transformed-view.html on GTK as it
depends on eventSender.scalePageBy().

  • platform/gtk/Skipped:
12:45 AM Changeset in webkit [84476] by yutak@chromium.org
  • 2 edits in trunk/LayoutTests

2011-04-21 Yuta Kitamura <​yutak@chromium.org>

Unreviewed, skip fast/dom/shadow/layout-tests-can-access-shadow.html on Windows.

fast/dom/shadow/layout-tests-can-access-shadow.html is failing on Windows
​https://bugs.webkit.org/show_bug.cgi?id=59081

  • platform/win/Skipped:
12:29 AM Changeset in webkit [84475] by commit-queue@webkit.org
  • 6 edits in trunk/Source/WebKit/qt

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

Reviewed by Csaba Osztrogonác.

[Qt] REGRESSION(84057): It made 4 API tests fail
​https://bugs.webkit.org/show_bug.cgi?id=58848

After MIME sniffing support was added, some autotests are failing because
the sniffer doesn't recognize our HTML pages.

This fixes tst_QWebPage::backActionUpdate(), tst_QWebPage::errorPageExtensionInFrameset()
and tst_QWebFrame::horizontalScrollAfterBack().

  • tests/qwebframe/resources/testiframe.html:
  • tests/qwebframe/resources/testiframe2.html: Remove spurious </html> in beginning of the document.
  • tests/qwebpage/resources/framedindex.html:
  • tests/qwebpage/resources/index.html: Add <html> (and </html>) tags, <frameset> is not recognized by the sniffer as a possible starting tag for HTML document.
  • tests/qwebpage/tst_qwebpage.cpp: (tst_QWebPage::errorPageExtensionInFrameset): Verify if we do have a main frame and the main frame has the children frames we expect. This will avoid crashing if for some reason children frames are not loaded.
12:24 AM Changeset in webkit [84474] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

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

Reviewed by Maciej Stachowiak.

remove unneeded function from graphicscontext3d.cpp (59076)
​https://bugs.webkit.org/show_bug.cgi?id=59076
<rdar://problem/9315681>

  • platform/graphics/GraphicsContext3D.cpp: removing unused function unpackOneRowOfRGBA8ToRGBA8
12:03 AM Changeset in webkit [84473] by ap@apple.com
  • 5 edits in trunk

2011-04-21 Alexey Proskuryakov <​ap@apple.com>

Reviewed by Dan Bernstein.

REGRESSION: Korean encoding alias KSC5601 does not work
​https://bugs.webkit.org/show_bug.cgi?id=59075
<rdar://problem/9303233>

  • fast/encoding/char-decoding-expected.txt:
  • fast/encoding/char-decoding.html: Added tests for KSC_5601 (which worked, but wasn't tested), and KSC5601 (which was broken).

2011-04-21 Alexey Proskuryakov <​ap@apple.com>

Reviewed by Dan Bernstein.

REGRESSION: Korean encoding alias KSC5601 does not work
​https://bugs.webkit.org/show_bug.cgi?id=59075
<rdar://problem/9303233>

  • platform/text/TextCodecICU.cpp: (WebCore::TextCodecICU::registerEncodingNames): Added a KSC5601 alias for KSC_5601, which effectively makes it windows-949.

Apr 20, 2011:

11:33 PM Changeset in webkit [84472] by commit-queue@webkit.org
  • 33 edits
    2 adds in trunk

2011-04-20 Dominic Cooney <​dominicc@chromium.org>

Reviewed by Dimitri Glazkov.

layoutTestController can create and destroy shadow DOM
​https://bugs.webkit.org/show_bug.cgi?id=59058

Exercise the new layoutTestController methods.

  • fast/dom/shadow/layout-tests-can-access-shadow-expected.txt: Added.
  • fast/dom/shadow/layout-tests-can-access-shadow.html: Added.

2011-04-20 Dominic Cooney <​dominicc@chromium.org>

Reviewed by Dimitri Glazkov.

layoutTestController can create and destroy shadow DOM
​https://bugs.webkit.org/show_bug.cgi?id=59058

Test: fast/dom/shadow/layout-tests-can-access-shadow.html

  • WebCore.exp.in: Mac DRT needs to see Element::ensure/removeShadowRoot

2011-04-20 Dominic Cooney <​dominicc@chromium.org>

Reviewed by Dimitri Glazkov.

layoutTestController can create and destroy shadow DOM
​https://bugs.webkit.org/show_bug.cgi?id=59058

Chromium DRT relies on the Chromium WebKit API

  • public/WebElement.h:
  • src/WebElement.cpp: (WebKit::WebElement::shadowRoot): don't steal a zero refcount (WebKit::WebElement::ensureShadowRoot): added (WebKit::WebElement::removeShadowRoot): added

2011-04-20 Dominic Cooney <​dominicc@chromium.org>

Reviewed by Dimitri Glazkov.

layoutTestController can create and destroy shadow DOM
​https://bugs.webkit.org/show_bug.cgi?id=59058

Support for new methods in GTK DRT.

  • WebCoreSupport/DumpRenderTreeSupportGtk.cpp: (DumpRenderTreeSupportGtk::ensureShadowRoot): (DumpRenderTreeSupportGtk::removeShadowRoot):
  • WebCoreSupport/DumpRenderTreeSupportGtk.h:

2011-04-20 Dominic Cooney <​dominicc@chromium.org>

Reviewed by Dimitri Glazkov.

layoutTestController can create and destroy shadow DOM
​https://bugs.webkit.org/show_bug.cgi?id=59058

Support for new methods in Mac DRT.

  • DOM/WebDOMOperations.mm: (-[DOMElement _ensureShadowRoot:]): (-[DOMElement _removeShadowRoot]):
  • DOM/WebDOMOperationsPrivate.h:

2011-04-20 Dominic Cooney <​dominicc@chromium.org>

Reviewed by Dimitri Glazkov.

layoutTestController can create and destroy shadow DOM
​https://bugs.webkit.org/show_bug.cgi?id=59058

Support for new methods in Qt DRT.

  • WebCoreSupport/DumpRenderTreeSupportQt.cpp: (DumpRenderTreeSupportQt::ensureShadowRoot): (DumpRenderTreeSupportQt::removeShadowRoot):
  • WebCoreSupport/DumpRenderTreeSupportQt.h:

2011-04-20 Dominic Cooney <​dominicc@chromium.org>

Reviewed by Dimitri Glazkov.

layoutTestController can create and destroy shadow DOM
​https://bugs.webkit.org/show_bug.cgi?id=59058

Support for new methods in WK2 WebKitTestRunner.

  • WebProcess/InjectedBundle/API/c/WKBundleNodeHandle.cpp: (WKBundleNodeHandleCopyElementEnsureShadowRoot): (WKBundleNodeHandleRemoveShadowRoot):
  • WebProcess/InjectedBundle/API/c/WKBundleNodeHandlePrivate.h:
  • WebProcess/InjectedBundle/DOM/InjectedBundleNodeHandle.cpp: (WebKit::InjectedBundleNodeHandle::elementShadowRoot): (WebKit::InjectedBundleNodeHandle::elementEnsureShadowRoot): (WebKit::InjectedBundleNodeHandle::elementRemoveShadowRoot):
  • WebProcess/InjectedBundle/DOM/InjectedBundleNodeHandle.h:

2011-04-20 Dominic Cooney <​dominicc@chromium.org>

Reviewed by Dimitri Glazkov.

layoutTestController can create and destroy shadow DOM
​https://bugs.webkit.org/show_bug.cgi?id=59058

Chromium, Mac, GTK and Qt DRT; and WK2 test runner.

  • DumpRenderTree/LayoutTestController.cpp: (ensureShadowRootCallback): (removeShadowRootCallback): (LayoutTestController::staticFunctions):
  • DumpRenderTree/LayoutTestController.h:
  • DumpRenderTree/chromium/LayoutTestController.cpp: (LayoutTestController::LayoutTestController): (LayoutTestController::ensureShadowRoot): (LayoutTestController::removeShadowRoot):
  • DumpRenderTree/chromium/LayoutTestController.h:
  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (LayoutTestController::ensureShadowRoot): (LayoutTestController::removeShadowRoot):
  • DumpRenderTree/mac/LayoutTestControllerMac.mm: (LayoutTestController::shadowRoot): (LayoutTestController::ensureShadowRoot): (LayoutTestController::removeShadowRoot):
  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::ensureShadowRoot): (LayoutTestController::removeShadowRoot):
  • DumpRenderTree/qt/LayoutTestControllerQt.h:
  • DumpRenderTree/win/LayoutTestControllerWin.cpp: (LayoutTestController::ensureShadowRoot): (LayoutTestController::removeShadowRoot):
  • WebKitTestRunner/InjectedBundle/Bindings/LayoutTestController.idl:
  • WebKitTestRunner/InjectedBundle/LayoutTestController.cpp: (WTR::LayoutTestController::shadowRoot): (WTR::LayoutTestController::ensureShadowRoot): (WTR::LayoutTestController::removeShadowRoot):
  • WebKitTestRunner/InjectedBundle/LayoutTestController.h:
10:49 PM Changeset in webkit [84471] by dpranke@chromium.org
  • 2 edits in trunk/LayoutTests

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

Make video/image-zoom fail on GPU on all ports.

  • platform/chromium/test_expectations.txt:
10:42 PM Changeset in webkit [84470] by dpranke@chromium.org
  • 2 edits in trunk/LayoutTests

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

Unreviewed, expectations change.

  • platform/chromium/test_expectations.txt:
10:29 PM Changeset in webkit [84469] by dpranke@chromium.org
  • 2 edits in trunk/LayoutTests

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

Unreviewed, expectations change.

Add test failures for r84454, r84457, flakiness on deps bots.

  • platform/chromium/test_expectations.txt:
10:20 PM Changeset in webkit [84468] by psolanki@apple.com
  • 2 edits in trunk/Source/WebCore

Follow on fix for ​https://bugs.webkit.org/show_bug.cgi?id=58926

Reviewed by Ryosuke Niwa.

Fix assertions in Chromium builds.

  • platform/graphics/Image.cpp:

(WebCore::Image::drawTiled): Skia and Wx do not have proper implementations of
BitmapImage::checkForSolidColor() so we shouldn't assert on those platforms.

10:19 PM Writing Reftests edited by hayato@chromium.org
(diff)
9:50 PM Changeset in webkit [84467] by dpranke@chromium.org
  • 3 edits in trunk/Tools

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

Reviewed by Eric Seidel.

new-run-webkit-tests: handle ctrl-c more cleanly
​https://bugs.webkit.org/show_bug.cgi?id=58849

  • Scripts/new-run-webkit-tests:
  • Scripts/webkitpy/layout_tests/layout_package/worker.py:
9:20 PM Changeset in webkit [84466] by adamk@chromium.org
  • 7 edits in trunk/Source/JavaScriptCore

2011-04-20 Adam Klein <​adamk@chromium.org>

Reviewed by David Levin.

Rename all uses of JSC_MULTIPLE_THREADS under wtf/... to WTF_MULTIPLE_THREADS
​https://bugs.webkit.org/show_bug.cgi?id=59040

This will be used to fix ​https://bugs.webkit.org/show_bug.cgi?id=55728
by enabling WTF_MULTIPLE_THREADS for Chromium.

  • wtf/CryptographicallyRandomNumber.cpp: (WTF::ARC4Stream::ARC4RandomNumberGenerator::randomNumber): (WTF::ARC4Stream::ARC4RandomNumberGenerator::randomValues):
  • wtf/FastMalloc.cpp:
  • wtf/Platform.h: Enable WTF_MULTIPLE_THREADS whenever JSC_MULTIPLE_THREADS is enabled.
  • wtf/RandomNumber.cpp: (WTF::randomNumber):
  • wtf/RefCountedLeakCounter.cpp: (WTF::RefCountedLeakCounter::increment): (WTF::RefCountedLeakCounter::decrement):
  • wtf/dtoa.cpp: (WTF::pow5mult):
9:14 PM Changeset in webkit [84465] by gyuyoung.kim@samsung.com
  • 2 edits in trunk/Source/WebCore

2011-04-20 Gyuyoung Kim <​gyuyoung.kim@samsung.com>

Reviewed by Eric Seidel.

[EFL] Add timeline to mediaControlsEfl.css
​https://bugs.webkit.org/show_bug.cgi?id=58865

Add timeline to mediaControlsEfl.css in order to show playing time of media
content.

  • css/mediaControlsEfl.css: (audio): (audio::-webkit-media-controls-panel, video::-webkit-media-controls-panel): (audio::-webkit-media-controls-mute-button, video::-webkit-media-controls-mute-button): (audio::-webkit-media-controls-play-button, video::-webkit-media-controls-play-button): (audio::-webkit-media-controls-timeline-container, video::-webkit-media-controls-timeline-container): (audio::-webkit-media-controls-current-time-display, video::-webkit-media-controls-current-time-display): (audio::-webkit-media-controls-timeline, video::-webkit-media-controls-timeline): (audio::-webkit-media-controls-seek-back-button, video::-webkit-media-controls-seek-back-button): (audio::-webkit-media-controls-seek-forward-button, video::-webkit-media-controls-seek-forward-button): (audio::-webkit-media-controls-fullscreen-button, video::-webkit-media-controls-fullscreen-button): (audio::-webkit-media-controls-toggle-closed-captions-button, video::-webkit-media-controls-toggle-closed-captions-button):
8:49 PM Changeset in webkit [84464] by dpranke@chromium.org
  • 2 edits in trunk/LayoutTests

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

Unreviewed, expectations change.

Suppress failures for inspector/debugger/live-edit.html
and inspector/console/console-object-constructor-name.html

  • platform/chromium/test_expectations.txt:
8:41 PM Changeset in webkit [84463] by Simon Fraser
  • 3 edits in trunk/Source/WebCore

2011-04-20 Simon Fraser <Simon Fraser>

Reviewed by Eric Seidel.

Create RIAA class for GraphicsContext::save() and restore()
​https://bugs.webkit.org/show_bug.cgi?id=58807

Make a stack-based class that can be used to save and restore
the graphics state. It has conditional behavior to deal with the many cases
where we decide to save/restore based on some runtime condition.

Change RenderBoxModelObject to use it for now; it can be used more widely
once committed.

  • platform/graphics/GraphicsContext.h: (WebCore::GraphicsContextStateSaver::GraphicsContextStateSaver): (WebCore::GraphicsContextStateSaver::~GraphicsContextStateSaver): (WebCore::GraphicsContextStateSaver::saveState):
  • rendering/RenderBoxModelObject.cpp: (WebCore::RenderBoxModelObject::paintFillLayerExtended): (WebCore::RenderBoxModelObject::paintOneBorderSide): (WebCore::RenderBoxModelObject::paintBorder): (WebCore::RenderBoxModelObject::drawBoxSideFromPath): (WebCore::RenderBoxModelObject::paintBoxShadow):
8:33 PM Changeset in webkit [84462] by barraclough@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Bug 59069 - DFG JIT - register allocate r8, r9, r10

Rubber stamped by Geoff Garen

  • dfg/DFGJITCompiler.h:

(JSC::DFG::JITCompiler::gprToRegisterID):

8:26 PM Changeset in webkit [84461] by dpranke@chromium.org
  • 2 edits in trunk/LayoutTests

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

Unreviewed, expectations change.

Suppress failures for media/csp-blocks-video.html and
plugins/embed-attributes-style.html

  • platform/chromium/test_expectations.txt:
8:21 PM Changeset in webkit [84460] by abarth@webkit.org
  • 5 edits
    6 adds in trunk

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

Reviewed by Eric Seidel.

CSP frame-src is missing
​https://bugs.webkit.org/show_bug.cgi?id=58643

Tests the basic functionality of frame-src.

  • http/tests/security/contentSecurityPolicy/frame-src-allowed-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/frame-src-allowed.html: Added.
  • http/tests/security/contentSecurityPolicy/frame-src-blocked-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/frame-src-blocked.html: Added.
  • http/tests/security/contentSecurityPolicy/resources/alert-fail.html: Added.
  • http/tests/security/contentSecurityPolicy/resources/alert-pass.html: Added.

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

Reviewed by Eric Seidel.

CSP frame-src is missing
​https://bugs.webkit.org/show_bug.cgi?id=58643

This is a first cut at an implementation of frame-src. There are a
couple things that will need to be improved:

1) I don't think we're handling in-frame navigation properly. This

patch only covers setting the src attribute of the frame, but I
think the intent of the spec is to cover navigation as well.

2) The console message is printed twice, once when we try to load the

frame and again when we attach the frame to the render tree.

I'll file bugs about these issues (blocking
​https://bugs.webkit.org/show_bug.cgi?id=53572) once this patch lands.

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

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

  • html/HTMLFrameElementBase.cpp: (WebCore::HTMLFrameElementBase::isURLAllowed):
  • page/ContentSecurityPolicy.cpp: (WebCore::ContentSecurityPolicy::allowChildFrameFromSource): (WebCore::ContentSecurityPolicy::addDirective):
  • page/ContentSecurityPolicy.h:
8:02 PM Changeset in webkit [84459] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebCore

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

Reviewed by Maciej Stachowiak.

[Mac] Correction panel shows up at incorrect location after webpage is scrolled.
​https://bugs.webkit.org/show_bug.cgi?id=59052
<rdar://problem/9312513>

Fixed a regression caused by interaction between patch for bug 57353 and patch for bug 55571.
See bug webpage for detail.

  • editing/Editor.cpp: (WebCore::Editor::selectionStartHasMarkerFor):
  • editing/SpellingCorrectionController.cpp: (WebCore::SpellingCorrectionController::windowRectForRange):
7:47 PM Changeset in webkit [84458] by dpranke@chromium.org
  • 2 edits in trunk/LayoutTests

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

Unreviewed, expectations change.

  • platform/chromium/test_expectations.txt:
7:33 PM Changeset in webkit [84457] by abarth@webkit.org
  • 22 edits
    2 copies
    12 adds
    4 deletes in trunk

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

Reviewed by Eric Seidel.

CSP policy violations should log to the console
​https://bugs.webkit.org/show_bug.cgi?id=58646

Now with console messages. I had to split a bunch of these tests into
smaller pieces to avoid race conditions in the new test output.

  • http/tests/security/contentSecurityPolicy/directive-parsing-expected.txt:
  • http/tests/security/contentSecurityPolicy/eval-scripts-setInterval-blocked-expected.txt:
  • http/tests/security/contentSecurityPolicy/eval-scripts-setTimeout-blocked-expected.txt:
  • http/tests/security/contentSecurityPolicy/image-blocked-expected.txt:
  • http/tests/security/contentSecurityPolicy/inline-script-blocked-expected.txt:
  • http/tests/security/contentSecurityPolicy/inline-script-blocked-goofy-expected.txt:
  • http/tests/security/contentSecurityPolicy/inline-script-blocked-goofy.html:
  • http/tests/security/contentSecurityPolicy/inline-script-blocked-javascript-url-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/inline-script-blocked-javascript-url.html: Copied from LayoutTests/http/tests/security/contentSecurityPolicy/inline-script-blocked-goofy.html.
  • http/tests/security/contentSecurityPolicy/javascript-url-allowed-expected.txt: Copied from LayoutTests/http/tests/security/contentSecurityPolicy/eval-scripts-setTimeout-blocked-expected.txt.
  • http/tests/security/contentSecurityPolicy/javascript-url-allowed.html: Added.
  • http/tests/security/contentSecurityPolicy/javascript-url-blocked-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/javascript-url-blocked.html: Added.
  • http/tests/security/contentSecurityPolicy/javascript-url-expected.txt: Removed.
  • http/tests/security/contentSecurityPolicy/javascript-url.html: Removed.
  • http/tests/security/contentSecurityPolicy/media-src-blocked-expected.txt:
  • http/tests/security/contentSecurityPolicy/object-src-none-allowed-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/object-src-none-allowed.html: Copied from LayoutTests/http/tests/security/contentSecurityPolicy/object-src-none.html.
  • http/tests/security/contentSecurityPolicy/object-src-none-blocked-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/object-src-none-blocked.html: Renamed from LayoutTests/http/tests/security/contentSecurityPolicy/object-src-none.html.
  • http/tests/security/contentSecurityPolicy/object-src-none-expected.txt: Removed.
  • http/tests/security/contentSecurityPolicy/script-src-none-expected.txt:
  • http/tests/security/contentSecurityPolicy/script-src-none-inline-event-expected.txt:
  • http/tests/security/contentSecurityPolicy/script-src-redirect-expected.txt:
  • http/tests/security/contentSecurityPolicy/script-src-self-blocked-01-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/script-src-self-blocked-01.html: Added.
  • http/tests/security/contentSecurityPolicy/script-src-self-blocked-02-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/script-src-self-blocked-02.html: Added.
  • http/tests/security/contentSecurityPolicy/script-src-self-blocked-03-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/script-src-self-blocked-03.html: Added.
  • http/tests/security/contentSecurityPolicy/script-src-self-expected.txt:
  • http/tests/security/contentSecurityPolicy/script-src-self.html:
  • http/tests/security/contentSecurityPolicy/source-list-parsing-expected.txt:
  • http/tests/security/contentSecurityPolicy/style-blocked-expected.txt:
  • http/tests/security/contentSecurityPolicy/xsl-blocked-expected.txt:
  • media/csp-blocks-video-expected.txt:

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

Reviewed by Eric Seidel.

CSP policy violations should log to the console
​https://bugs.webkit.org/show_bug.cgi?id=58646

We now log policy violations to the JavaScript console to help
developers debug what's going on with their Content-Security-Policy.

Tests: http/tests/security/contentSecurityPolicy/inline-script-blocked-javascript-url.html

http/tests/security/contentSecurityPolicy/javascript-url-allowed.html
http/tests/security/contentSecurityPolicy/javascript-url-blocked.html
http/tests/security/contentSecurityPolicy/object-src-none-allowed.html
http/tests/security/contentSecurityPolicy/object-src-none-blocked.html
http/tests/security/contentSecurityPolicy/script-src-self-blocked-01.html
http/tests/security/contentSecurityPolicy/script-src-self-blocked-02.html
http/tests/security/contentSecurityPolicy/script-src-self-blocked-03.html

  • dom/Document.cpp: (WebCore::Document::initSecurityContext):
  • page/ContentSecurityPolicy.cpp: (WebCore::ContentSecurityPolicy::ContentSecurityPolicy): (WebCore::ContentSecurityPolicy::allowJavaScriptURLs): (WebCore::ContentSecurityPolicy::allowInlineEventHandlers): (WebCore::ContentSecurityPolicy::allowInlineScript): (WebCore::ContentSecurityPolicy::allowEval): (WebCore::ContentSecurityPolicy::allowScriptFromSource): (WebCore::ContentSecurityPolicy::allowObjectFromSource): (WebCore::ContentSecurityPolicy::allowImageFromSource): (WebCore::ContentSecurityPolicy::allowStyleFromSource): (WebCore::ContentSecurityPolicy::allowFontFromSource): (WebCore::ContentSecurityPolicy::allowMediaFromSource): (WebCore::ContentSecurityPolicy::addDirective):
  • page/ContentSecurityPolicy.h: (WebCore::ContentSecurityPolicy::create):
7:23 PM Changeset in webkit [84456] by aestes@apple.com
  • 2 edits in trunk/LayoutTests

fast/images/animated-gif-restored-from-bfcache.html fails on Windows bots
​https://bugs.webkit.org/show_bug.cgi?id=59062

  • platform/win/Skipped:
7:20 PM Changeset in webkit [84455] by commit-queue@webkit.org
  • 5 edits in trunk/Tools

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

Unreviewed, rolling out r84431.
​http://trac.webkit.org/changeset/84431
​https://bugs.webkit.org/show_bug.cgi?id=59061

Breaks for folks who don't have editbugs (Requested by abarth
on #webkit).

  • Scripts/webkitpy/common/net/bugzilla/bugzilla.py:
  • Scripts/webkitpy/tool/commands/upload_unittest.py:
  • Scripts/webkitpy/tool/mocktool.py:
  • Scripts/webkitpy/tool/steps/postdiff.py:
7:13 PM Changeset in webkit [84454] by mitz@apple.com
  • 4 edits
    2 adds in trunk

<rdar://problem/9095366> With a non-1 page scale, scrolling to reveal selection fails
​https://bugs.webkit.org/show_bug.cgi?id=59046

Reviewed by Maciej Stachowiak.

Source/WebCore:

Test: fast/transforms/selection-bounds-in-transformed-view.html

  • rendering/RenderObject.cpp:

(WebCore::RenderObject::repaintUsingContainer): If the repaint container is the RenderView, and
it has a composited layer that paints straight to the window, then translate from view coordinates
to window coordinates here.

  • rendering/RenderView.cpp:

(WebCore::RenderView::mapLocalToContainer): Do not apply our transform if we are the painting root.
(WebCore::RenderView::computeRectForRepaint): Ditto. Applying the transform here was compensating
for not applying it in RenderObject::repaintUsingContainer(), but for purposes other than repainting,
such as computing selection bounds, this function was returning the wrong results.

LayoutTests:

  • fast/transforms/selection-bounds-in-transformed-view-expected.txt: Added.
  • fast/transforms/selection-bounds-in-transformed-view.html: Added.
7:07 PM Changeset in webkit [84453] by leo.yang@torchmobile.com.cn
  • 2 edits in trunk/Tools

2011-04-20 Leo Yang <​leo.yang@torchmobile.com.cn>

Unreviewed.

Adding myself to committers.py.

  • Scripts/webkitpy/common/config/committers.py:
7:03 PM Changeset in webkit [84452] by mitz@apple.com
  • 2 edits in trunk/Source/WebCore

<rdar://problem/9304956> media/broken-video.html crashes

Reviewed by Eric Carlson.

  • platform/graphics/MediaPlayer.cpp:

(WebCore::nextMediaEngine): Fixed an off-by-one error.

6:58 PM Changeset in webkit [84451] by dpranke@chromium.org
  • 2 edits in trunk/LayoutTests

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

Unreviewed, expectations change.

Merge forward more suppressions from the chromium overrides
file.

  • platform/chromium/test_expectations.txt:
6:55 PM Changeset in webkit [84450] by eric@webkit.org
  • 2 edits in trunk/Source/WebCore

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

Reviewed by Adam Barth.

Split out isIteratorTarget from bidiNext and bidiFirst
​https://bugs.webkit.org/show_bug.cgi?id=59033

This is the first (tiny) step of many. When I tried to
make larger changes to this function I broke stuff, so
doing this in tiny pieces so we know it's right.

  • rendering/InlineIterator.h: (WebCore::isIteratorTarget): (WebCore::bidiNext): (WebCore::bidiFirstSkippingInlines): (WebCore::bidiFirstNotSkippingInlines):
6:38 PM Changeset in webkit [84449] by commit-queue@webkit.org
  • 5 edits in trunk/Source

2011-04-20 Alexey Marinichev <​amarinichev@chromium.org>

Reviewed by Kenneth Russell.

Check for null m_context in WebGLLayerChromium and add resetting platform layer's context in the destructor of GraphicsContext3D.
​https://bugs.webkit.org/show_bug.cgi?id=59045

Lose context extension test might not catch this bug, more reliable
way is forcing the crash of the GPU process, which is done by Chromium
browser tests.

  • platform/graphics/chromium/WebGLLayerChromium.cpp: (WebCore::WebGLLayerChromium::updateCompositorResources): (WebCore::WebGLLayerChromium::setContext):
  • platform/graphics/chromium/WebGLLayerChromium.h:

2011-04-20 Alexey Marinichev <​amarinichev@chromium.org>

Reviewed by Kenneth Russell.

Check for null m_context in WebGLLayerChromium and add resetting platform layer's context in the destructor of GraphicsContext3D.
​https://bugs.webkit.org/show_bug.cgi?id=59045

  • src/GraphicsContext3DChromium.cpp: (WebCore::GraphicsContext3D::~GraphicsContext3D): call into platform layer and unregister itself.
6:28 PM Changeset in webkit [84448] by commit-queue@webkit.org
  • 2 edits
    1 add in trunk/Source/WebCore

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

Reviewed by Maciej Stachowiak.

[Mac]Regression: extending word that is already marked misspelled doesn't remove misspelling underline.
​https://bugs.webkit.org/show_bug.cgi?id=59032
<rdar://problem/9312444>

This fixes a regression introduced in recent refactoring work in Editor. See bug webpage for detail.

manual test: Source/WebCore/manual-tests/autocorrection/remove-misspelling-marker-after-appending-letter.html

  • editing/Editor.cpp: (WebCore::Editor::insertTextWithoutSendingTextEvent):
  • manual-tests/autocorrection/remove-misspelling-marker-after-appending-letter.html: Added.
6:10 PM Changeset in webkit [84447] by adamk@chromium.org
  • 1 edit in trunk/Tools/Scripts/webkitpy/common/config/committers.py

Unreviewed. Add myself to the committers script.

6:04 PM Changeset in webkit [84446] by barraclough@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Build fix - revert accidental change.

  • wtf/Platform.h:
6:03 PM Changeset in webkit [84445] by barraclough@apple.com
  • 6 edits in trunk/Source/JavaScriptCore

Add SAMPLING_FLAGS tool to DFG JIT.

Reviewed by Sam Weinig.

  • bytecode/SamplingTool.h:

(JSC::SamplingFlags::addressOfFlags):

  • dfg/DFGJITCompiler.cpp:

(JSC::DFG::JITCompiler::setSamplingFlag):
(JSC::DFG::JITCompiler::clearSamplingFlag):

  • dfg/DFGJITCompiler.h:
  • jit/JITInlineMethods.h:

(JSC::JIT::setSamplingFlag):
(JSC::JIT::clearSamplingFlag):

  • wtf/Platform.h:
5:54 PM Changeset in webkit [84444] by Chris Fleizach
  • 3 edits in trunk/Source/WebCore

2011-04-20 Chris Fleizach <Chris Fleizach>

Reviewed by Beth Dakin.

CrashTracer: 301 crashes in Safari at com.apple.WebCore: WebCore::AccessibilityRenderObject::visiblePositionForPoint const + 297
​https://bugs.webkit.org/show_bug.cgi?id=57405

The offending line in this crash was a null pointer access in

m_renderer->document()->topDocument()->renderer()->view()->frameView();

It seems likely that one of those calls was invalid. I could not reproduce and there was no
information on reproducible steps, hence the absence of a layout test.

  • accessibility/AccessibilityRenderObject.cpp: (WebCore::AccessibilityRenderObject::topRenderer): (WebCore::AccessibilityRenderObject::topDocument): (WebCore::AccessibilityRenderObject::topDocumentFrameView): (WebCore::AccessibilityRenderObject::visiblePositionForPoint):
  • accessibility/AccessibilityRenderObject.h:
5:46 PM Changeset in webkit [84443] by aestes@apple.com
  • 4 edits
    3 adds in trunk

2011-04-20 Andy Estes <​aestes@apple.com>

Reviewed by Maciej Stachowiak.

REGRESSION (WK2): Animated GIF stops when navigating forward, back
​https://bugs.webkit.org/show_bug.cgi?id=59042

  • fast/images/animated-gif-restored-from-bfcache-expected.txt: Added.
  • fast/images/animated-gif-restored-from-bfcache.html: Added.
  • fast/images/resources/animated-10color.gif: Added.
  • platform/mac-wk2/Skipped: Sadly, this test for a WebKit2-specific fix has to be skipped for now on WebKit2 since it doesn't implement layoutTestController.overridePreference().

2011-04-20 Andy Estes <​aestes@apple.com>

Reviewed by Maciej Stachowiak.

REGRESSION (WK2): Animated GIF stops when navigating forward, back
​https://bugs.webkit.org/show_bug.cgi?id=59042

When loading the page from the b/f cache,
document()->view()->isOffscreen() returns true in
RenderObject::willRenderImage(), so BitmapImage::startAnimation()
returns without actually kicking off the animation.

In WebKit1, the ScrollView is a platform widget, so
ScrollView::isOffscreen() calls [NSWindow isVisible]. In WebKit2, the
ScrollView isn't a platform widget so it asks the WebCore Widget
whether or not it is visible. It always returns false since
setParentVisible(false) was called on the old ScrollView before
navigating to a new page, but setParentVisible(true) is never called
when restoring it from the back/forward cache. This is probably a
long-standing bug.

Fix this by calling setParentVisible(true) when a cached main frame is
being restored.

Test: fast/images/animated-gif-restored-from-bfcache.html

  • history/CachedFrame.cpp: (WebCore::CachedFrameBase::restore): If the cached frame is the main frame, call setParentVisible(true) on the frame's view.
5:35 PM Changeset in webkit [84442] by weinig@apple.com
  • 2 edits in trunk/Source/WebCore

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

Reviewed by Anders Carlsson.

WebKit2: Can't scroll page after printing on Yahoo.com and other sites
<rdar://problem/9233291>
​https://bugs.webkit.org/show_bug.cgi?id=59049

  • page/Frame.cpp: (WebCore::Frame::setPrinting): Match Mac WebKit1 code when ending printing, and call forceLayout() instead of forceLayoutForPagination().
5:28 PM Changeset in webkit [84441] by kbr@google.com
  • 2 edits in trunk/Source/WebCore

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

Reviewed by James Robinson.

Reduce size of GraphicsContext3D's compiled code
​https://bugs.webkit.org/show_bug.cgi?id=59029

Changed format conversion operations to work line-by-line instead
of pixel-by-pixel, and passed them as function pointers rather
than template parameters. Simplified computation of source
increment, since elements per row is now all that is needed.

These changes reduce the size of GraphicsContext3D.o in release
mode on Linux from 299 KB to 53 KB.

No new tests. Existing layout tests cover these format conversions
well, and caught one typo in the restructuring. Also ran WebGL
conformance suite and other demos.

  • platform/graphics/GraphicsContext3D.cpp: (WebCore::doUnpackingAndPacking): (WebCore::computeSourceElementsPerRow): (WebCore::doPacking): (WebCore::doFloatingPointPacking): (WebCore::GraphicsContext3D::packPixels):
5:23 PM Changeset in webkit [84440] by Dimitri Glazkov
  • 3 edits
    2 adds in trunk

2011-04-20 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Kent Tamura.

REGRESSION(r82376): Clicking on a label, associated with a disabled input element changes state of the element.
​https://bugs.webkit.org/show_bug.cgi?id=59037

  • fast/events/simulated-click-disabled-expected.txt: Added.
  • fast/events/simulated-click-disabled.html: Added.

2011-04-20 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Kent Tamura.

REGRESSION(r82376): Clicking on a label, associated with a disabled input element changes state of the element.
​https://bugs.webkit.org/show_bug.cgi?id=59037

Test: fast/events/simulated-click-disabled.html

  • dom/EventDispatcher.cpp: (WebCore::EventDispatcher::dispatchSimulatedClick): Added check for disabled element.
5:21 PM Changeset in webkit [84439] by dpranke@chromium.org
  • 2 edits in trunk/LayoutTests

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

Pull in more flaky expectations.

  • platform/chromium/test_expectations.txt:
4:55 PM Changeset in webkit [84438] by dpranke@chromium.org
  • 2 edits in trunk/LayoutTests

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

Unreviewed, expectations change. Pull in expectations from
overrides file.

  • platform/chromium/test_expectations.txt:
4:50 PM Changeset in webkit [84437] by andersca@apple.com
  • 3 edits in trunk/Source/WebKit2

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

Reviewed by Sam Weinig.

Don't discard backing stores for hung web processes
​https://bugs.webkit.org/show_bug.cgi?id=59039
<rdar://problem/9237090>

  • UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::viewStateDidChange): (WebKit::WebPageProxy::processDidBecomeUnresponsive): (WebKit::WebPageProxy::processDidBecomeResponsive): Call updateBackingStoreDiscardableState.

(WebKit::WebPageProxy::updateBackingStoreDiscardableState):
Never mark the backing store as discardable if the web process is hung

4:48 PM Changeset in webkit [84436] by eric@webkit.org
  • 4 edits in trunk/Source/WebCore

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

Reviewed by Ryosuke Niwa.

Split bidiFirst into two functions
​https://bugs.webkit.org/show_bug.cgi?id=59025

I don't have a full understanding of the skipInlines case yet
(hence it's not as pretty as the non-skipping case), but
this is clearly a win, as both functions are much simpler
than their combined version.

Both of these functions still have bad names. However the
bidiNext/bidiFirst names date all the way back to revision 1
of the WebKit repository. :) So I don't feel too bad keeping
them for the moment.

  • rendering/InlineIterator.h: (WebCore::bidiNext): (WebCore::bidiFirstSkippingInlines): (WebCore::bidiFirstNotSkippingInlines):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::simplifiedNormalFlowLayout):
  • rendering/RenderBlockLineLayout.cpp: (WebCore::RenderBlock::layoutInlineChildren): (WebCore::RenderBlock::determineStartPosition):
4:43 PM Changeset in webkit [84435] by bfulgham@webkit.org
  • 2 edits in trunk/Source/WebKit/win

Provide missing implementation of WebMutableURLRequest::mutableCopy.
This was preventing DumpRenderTree from working properly under
the WinCairo port.
​https://bugs.webkit.org/show_bug.cgi?id=59034

Reviewed by Martin Robinson.

  • WebMutableURLRequest.cpp:

(WebMutableURLRequest::mutableCopy): Implement cURL version of the
copy operation.

4:41 PM Changeset in webkit [84434] by mjs@apple.com
  • 3 edits in trunk/Source/WebKit2

2011-04-20 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Sam Weinig.

Reproducible crash under WebPageProxy::didReceiveEvent
​https://bugs.webkit.org/show_bug.cgi?id=59036
<rdar://problem/9252083>

  • UIProcess/API/mac/WKView.mm: (-[WKView _resendKeyDownEvent:]): Protect against self-destruction when re-dispatching the event.
  • UIProcess/WebProcessProxy.cpp: (WebKit::WebProcessProxy::createWebPage): Fixed a type mismatch I noticed while debugging.
4:35 PM Changeset in webkit [84433] by commit-queue@webkit.org
  • 7 edits in trunk/Source/WebCore

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

Reviewed by Adele Peterson.

[Mac] Like autocorrection, other types of substitution need to respect undo.
​https://bugs.webkit.org/show_bug.cgi?id=59009
<rdar://problem/8601537>

Similar to autocorrection, we use SpellingCorrectionCommand to carry out other types of text substitution, so that
if user undoes the substitution, it will not be applied again. For CreateLinkCommand, we add marker when it's undone
so that we won't automatically add link again.

  • editing/CreateLinkCommand.h: (WebCore::CreateLinkCommand::isCreateLinkCommand):
  • editing/EditCommand.cpp: (WebCore::EditCommand::isCreateLinkCommand):
  • editing/EditCommand.h:
  • editing/Editor.cpp: (WebCore::Editor::unappliedEditing): (WebCore::Editor::markAllMisspellingsAndBadGrammarInRanges):
  • editing/SpellingCorrectionController.cpp: (WebCore::SpellingCorrectionController::respondToAppliedEditing): (WebCore::SpellingCorrectionController::respondToUnappliedEditing):
  • editing/SpellingCorrectionController.h: (WebCore::SpellingCorrectionController::UNLESS_ENABLED):
4:20 PM Changeset in webkit [84432] by tony@chromium.org
  • 2 edits
    1 add
    2 deletes in trunk/LayoutTests

2011-04-20 Tony Chang <​tony@chromium.org>

Update 2 more tests that are missing png files.

  • platform/chromium-mac/fast/borders/borderRadiusDotted06-expected.checksum: Removed.
  • platform/chromium-mac/fast/borders/borderRadiusDotted06-expected.png: Added. png with embedded checksum pulled from the bot.
  • platform/chromium-win/compositing/repaint/become-overlay-composited-layer-expected.checksum: Removed. This file is not used since we don't

run compositing tests w/CPU rendering and there's a GPU result in chromium-gpu-win.

  • platform/chromium/test_expectations.txt: Remove these tests.
4:08 PM Changeset in webkit [84431] by abarth@webkit.org
  • 5 edits in trunk/Tools

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

Reviewed by Eric Seidel.

webkit-patch upload should assign the bug to you
​https://bugs.webkit.org/show_bug.cgi?id=33699

This patch only re-assigns the bug if it is currently unassigned. We
can be more aggressive in the future if people want us to be more
agressive.

  • Scripts/webkitpy/common/net/bugzilla/bugzilla.py:
  • Scripts/webkitpy/tool/commands/upload_unittest.py:
  • Scripts/webkitpy/tool/mocktool.py:
  • Scripts/webkitpy/tool/steps/postdiff.py:
4:07 PM Changeset in webkit [84430] by jianli@chromium.org
  • 2 edits in trunk/LayoutTests

Unreviewed, fixed typo in the failed test added to chromium test
expectations.

  • platform/chromium/test_expectations.txt:
3:57 PM Changeset in webkit [84429] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk/Source/WebCore

2011-04-20 Steve Lacey <​sjl@chromium.org>

Reviewed by Eric Carlson.

[chromium] Move Chromium media controls into own root element.
​https://bugs.webkit.org/show_bug.cgi?id=59018

No new tests as there is no functional change

  • WebCore.gypi:
  • css/mediaControlsChromium.css: (audio::-webkit-media-controls-timeline-container, video::-webkit-media-controls-timeline-container):
  • html/shadow/MediaControlRootElementChromium.cpp: Added. Clone of html/shadow/MediaControlRootElement.cpp with all unused code removed.
  • html/shadow/MediaControlRootElementChromium.h: Added.
3:54 PM Changeset in webkit [84428] by tony@chromium.org
  • 2 edits
    1 add
    1 delete in trunk/LayoutTests

2011-04-20 Tony Chang <​tony@chromium.org>

More .checksum file cleanup.

  • fast/css/background-clip-values-expected.checksum: Removed. This is a duplicate of the .checksum

file in platform/mac.

  • platform/mac-leopard/fast/backgrounds/size/contain-and-cover-expected.checksum: Correct checksum by

regenerating on my leopard machine.

  • platform/mac-leopard/fast/backgrounds/size/contain-and-cover-expected.png: Added.
3:32 PM Changeset in webkit [84427] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebKit/chromium

2011-04-20 Yuzhu Shen <​yzshen@chromium.org>

Reviewed by Tony Chang.

[chromium] linux chromium doesn't set click count for mouse up events.
​https://bugs.webkit.org/show_bug.cgi?id=58921

The reason why we need such a fix is that the Pepper API passes this information to plugins,
and some consumers (e.g., Flash) need it to detect double-clicks.

  • src/gtk/WebInputEventFactory.cpp: set click count for mouse up events.
3:26 PM Changeset in webkit [84426] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-04-20 Charlie Lee <​chocobo@chromium.org>

Reviewed by Dirk Schulze.

LEFilter change doesn't compile on ARM neon targets
​https://bugs.webkit.org/show_bug.cgi?id=58671

Build fix for Chromium ARM/NEON platform. Added platform/graphics/filters/arm directory to build system.

  • WebCore.gyp/WebCore.gyp:
3:20 PM Changeset in webkit [84425] by xji@chromium.org
  • 5 edits in trunk

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

Reviewed by Ryosuke Niwa.

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

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


The overall algorithm is:

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

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

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

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

Notes:

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


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

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

Reviewed by Ryosuke Niwa.

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

  • editing/selection/move-by-word-visually-expected.txt:
  • editing/selection/move-by-word-visually.html:
3:15 PM April 2011 Meeting edited by Dimitri Glazkov
(diff)
3:09 PM Changeset in webkit [84424] by timothy@apple.com
  • 11 edits in trunk/Source/WebKit2

Make the Web Insepctor come to the front when a breakpoint is hit.

<​https://webkit.org/b/57480> REGRESSION (WebKit2): Inspector doesn't come to the front when a breakpoint is hit in the debugger
<rdar://problem/9209472>

Reviewed by Adam Roben.

  • UIProcess/WebInspectorProxy.cpp:

(WebKit::WebInspectorProxy::bringToFront): Added. Call platformBringToFront.

  • UIProcess/WebInspectorProxy.h:
  • UIProcess/WebInspectorProxy.messages.in: Added BringToFront.
  • UIProcess/mac/WebInspectorProxyMac.mm:

(WebKit::WebInspectorProxy::platformBringToFront): Added. Call orderFront:.

  • UIProcess/win/WebInspectorProxyWin.cpp:

(WebKit::WebInspectorProxy::platformBringToFront): Added. Call SetWindowPos.

  • WebProcess/WebCoreSupport/WebInspectorFrontendClient.cpp:

(WebKit::WebInspectorFrontendClient::bringToFront): Call WebInspector::bringToFront.

  • WebProcess/WebPage/WebInspector.cpp:

(WebKit::WebInspector::bringToFront): Added. Send WebInspectorProxy::BringToFront message.

  • WebProcess/WebPage/WebInspector.h: Added bringToFront.
2:57 PM Changeset in webkit [84423] by Csaba Osztrogonác
  • 2 edits in trunk/Tools

Modify builder.html template to allow easy access
to list more builds on the builder's page.

Patch by Peter Gal <​galpeter@inf.u-szeged.hu> on 2011-04-20
Reviewed by Csaba Osztrogonác.

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

  • BuildSlaveSupport/build.webkit.org-config/templates/builder.html:
2:57 PM Changeset in webkit [84422] by cdn@chromium.org
  • 3 edits
    2 adds in trunk

2011-04-20 Cris Neckar <​cdn@chromium.org>

Reviewed by Dirk Schulze.

Tests for crash when copying a filter effect after applying tranforms.
​https://bugs.webkit.org/show_bug.cgi?id=57885

  • svg/filters/svg-transform-blur-crash-expected.txt: Added.
  • svg/filters/svg-transform-blur-crash.xhtml: Added.

2011-04-20 Cris Neckar <​cdn@chromium.org>

Reviewed by Dirk Schulze.

Return early when the paint rect and the source rect do not overlap as no bytes need to be copied.
​https://bugs.webkit.org/show_bug.cgi?id=57885

Test: svg/filters/svg-transform-blur-crash.xhtml

  • platform/graphics/filters/FilterEffect.cpp: (WebCore::FilterEffect::copyImageBytes):
2:48 PM Changeset in webkit [84421] by Csaba Osztrogonác
  • 1 edit
    1 add in trunk/Tools

2011-04-20 Peter Gal <​galpeter@inf.u-szeged.hu>

Reviewed by Csaba Osztrogonác.

Add the default template for builder page.

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

  • BuildSlaveSupport/build.webkit.org-config/templates/builder.html: Added.
2:45 PM Changeset in webkit [84420] by ojan@chromium.org
  • 2 edits in trunk/Tools

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

Reviewed by Eric Seidel.

move page generation into a single function
​https://bugs.webkit.org/show_bug.cgi?id=59027

Now executing the script doesn't create any state. We should then
be able to write tests for this by setting custom results data.

  • Scripts/webkitpy/layout_tests/layout_package/json_results.html:
2:39 PM Changeset in webkit [84419] by ojan@chromium.org
  • 2 edits in trunk/Tools

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

Reviewed by Tony Chang.

move global state into a shared object
​https://bugs.webkit.org/show_bug.cgi?id=59023

This makes the code more testable.

  • Scripts/webkitpy/layout_tests/layout_package/json_results.html:
2:38 PM Changeset in webkit [84418] by jianli@chromium.org
  • 4 edits
    3 adds in trunk

Support DataView in workers
​https://bugs.webkit.org/show_bug.cgi?id=59014

Reviewed by Kenneth Russell.

Source/WebCore:

Test: fast/canvas/webgl/worker-data-view-test.html

  • workers/WorkerContext.idl: Expose DataView constructor.

LayoutTests:

  • fast/canvas/webgl/resources/worker-data-view-test.js: Added.
  • fast/canvas/webgl/worker-data-view-test-expected.txt: Added.
  • fast/canvas/webgl/worker-data-view-test.html: Added.
  • platform/chromium/test_expectations.txt:
2:33 PM Changeset in webkit [84417] by tony@chromium.org
  • 2 edits
    12 deletes in trunk/LayoutTests

2011-04-20 Tony Chang <​tony@chromium.org>

Cleanup .checksum files that don't have .png files.

If the test is marked as failing, just delete the .checksum file.
If the test is marked as passing, put it in test_expectations.txt so
the bots will generate results.

  • platform/chromium-mac-leopard/fast/borders/border-radius-split-inline-expected.checksum: Removed.
  • platform/chromium-mac-leopard/fast/repaint/positioned-document-element-expected.checksum: Removed.
  • platform/chromium-mac-leopard/fast/text/basic/014-expected.checksum: Removed.
  • platform/chromium-mac/fast/dom/HTMLMeterElement/meter-appearances-capacity-expected.checksum: Removed.
  • platform/chromium-mac/fast/dom/HTMLMeterElement/meter-element-expected.checksum: Removed.
  • platform/chromium-mac/fast/dom/HTMLMeterElement/meter-optimums-expected.checksum: Removed.
  • platform/chromium-mac/fast/dom/HTMLMeterElement/meter-styles-changing-pseudo-expected.checksum: Removed.
  • platform/chromium-mac/fast/dom/HTMLMeterElement/meter-styles-expected.checksum: Removed.
  • platform/chromium-mac/fast/forms/input-appearance-bkcolor-expected.checksum: Removed.
  • platform/chromium-mac/fast/images/imagemap-focus-ring-expected.checksum: Removed.
  • platform/chromium-mac/tables/mozilla/core/misc-expected.checksum: Removed.
  • platform/chromium-win/fast/forms/searchfield-heights-expected.checksum: Removed.
  • platform/chromium/test_expectations.txt:
2:09 PM Changeset in webkit [84416] by cmarrin@apple.com
  • 10 edits in trunk/Source

2011-04-20 Chris Marrin <​cmarrin@apple.com>

Reviewed by Adam Roben.

REGRESSION (WebKit2): CSS animations on pages that use accelerated compositing stop after switching tabs
​https://bugs.webkit.org/show_bug.cgi?id=57868


Running CACFAnimations aren't remembered when layer tree is unparented and
then reparented. Added a new ensureAnimationsSubmitted function to
PlatformCALayer to traverse the tree and resubmit animations, setting the
beginTime to the original start time of the animation. This is called in WK2
when layer tree is newly parented.


I also consolidated currentTimeToMediaTime into PlatformCALayer so it can be shared
between PlatformCALayer and GraphicsLayerCA.

  • platform/graphics/ca/GraphicsLayerCA.cpp:
  • platform/graphics/ca/PlatformCAAnimation.h:

Remember actual animation start time in CACFAnimation beginTime property

  • platform/graphics/ca/PlatformCALayer.h:

New ensureAnimationsSubmitted function which traverses layers and resubmits all animations

  • platform/graphics/ca/mac/PlatformCALayerMac.mm:
  • platform/graphics/ca/win/PlatformCALayerWin.cpp:

2011-04-20 Chris Marrin <​cmarrin@apple.com>

Reviewed by Adam Roben.

REGRESSION (WebKit2): CSS animations on pages that use accelerated compositing stop after switching tabs
​https://bugs.webkit.org/show_bug.cgi?id=57868


Subclassed setRootCompositingLayer so it can call ensureAnimationsSubmitted on Win.

  • WebProcess/WebPage/ca/LayerTreeHostCA.h:

Make setRootCompositingLayer protected so it can be called from subclass

  • WebProcess/WebPage/ca/win/LayerTreeHostCAWin.cpp:

Override setRootCompositingLayer to add call to root layer's
PlatformCALayer::ensureAnimationsSubmitted method

  • WebProcess/WebPage/ca/win/LayerTreeHostCAWin.h:
2:03 PM ThreadCommunication edited by levin@chromium.org
Filled in api for sync case. (diff)
2:02 PM Changeset in webkit [84415] by eric.carlson@apple.com
  • 14 edits in trunk

2011-04-20 Eric Carlson <​eric.carlson@apple.com>

Reviewed by Dan Bernstein.

Adopt QTKit API for listing and deleting file in its media cache.
<rdar://problem/9130029>
​https://bugs.webkit.org/show_bug.cgi?id=58795

Not possible to test automatically, verified manually.

  • WebCore.exp.in: Export new media download cache functions.
  • platform/graphics/mac/MediaPlayerPrivateQTKit.h:
  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivateQTKit::registerMediaEngine): Register cache management functions. (WebCore::MediaPlayerPrivateQTKit::getSitesInMediaCache): New, return cache origins. (WebCore::MediaPlayerPrivateQTKit::clearMediaCache): New, empty the media cache. (WebCore::MediaPlayerPrivateQTKit::clearMediaCacheForSite): New, delete files in the media

cache for one origin.

  • platform/mac/WebCoreSystemInterface.h: Add new WKSI functions.
  • platform/mac/WebCoreSystemInterface.mm: Ditto.

2011-04-20 Eric Carlson <​eric.carlson@apple.com>

Reviewed by Dan Bernstein.

Adopt QTKit API for listing and deleting file in its media cache.
<rdar://problem/9130029>
​https://bugs.webkit.org/show_bug.cgi?id=58795

  • WebKitSystemInterface.h:
  • libWebKitSystemInterfaceLeopard.a:
  • libWebKitSystemInterfaceSnowLeopard.a:

2011-04-20 Eric Carlson <​eric.carlson@apple.com>

Reviewed by Dan Bernstein.

Adopt QTKit API for listing and deleting file in its media cache.
<rdar://problem/9130029>

  • WebCoreSupport/WebSystemInterface.mm: (InitWebCoreSystemInterface): Initialize new functions.

2011-04-18 Eric Carlson <​eric.carlson@apple.com>

Reviewed by Dan Bernstein.

Adopt QTKit API for listing and deleting file in its media cache.
<rdar://problem/9130029>

  • WebProcess/WebCoreSupport/mac/WebSystemInterface.mm: (InitWebCoreSystemInterface): Initialize new functions.
2:00 PM Changeset in webkit [84414] by dpranke@chromium.org
  • 2 edits in trunk/LayoutTests

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

Unreviewed, fix duplicate expectations.

  • platform/chromium/test_expectations.txt:
1:47 PM Changeset in webkit [84413] by barraclough@apple.com
  • 5 edits in trunk/Source/JavaScriptCore

Bug 59022 - DFG JIT - Optimize branch-on-relational-compare

Reviewed by Oliver Hunt.

If a relational compare (< or <=) is immediately followed by a branch,
we can combine the two, avoiding generation of a boolean into a register.

  • assembler/MacroAssemblerX86Common.h:

(JSC::MacroAssemblerX86Common::branch32):
(JSC::MacroAssemblerX86Common::invert):
(JSC::MacroAssemblerX86Common::commute):

  • dfg/DFGNode.h:

(JSC::DFG::Node::adjustedRefCount):

  • dfg/DFGSpeculativeJIT.cpp:

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

  • dfg/DFGSpeculativeJIT.h:

(JSC::DFG::SpeculativeJIT::isJSConstantWithInt32Value):
(JSC::DFG::SpeculativeJIT::detectPeepHoleBranch):

1:47 PM Changeset in webkit [84412] by tony@chromium.org
  • 6191 edits in trunk/LayoutTests

2011-04-20 Tony Chang <​tony@chromium.org>

Unreviewed, embedding checksums in png files in platform/mac.

1:28 PM Changeset in webkit [84411] by robert@webkit.org
  • 2 edits
    1 add in trunk/LayoutTests

2011-04-20 Robert Hogan <​robert@webkit.org>

Reviewed by Antonio Gomes.

[Qt] DRT sideeffect revealed by r63657 and r75305
​https://bugs.webkit.org/show_bug.cgi?id=42578

Unskip tests passing since r76958

The NetworkAccessManager's authentication cache is getting
cleared now on every page load, so these no longer leak
credentials between tests.

  • platform/qt/Skipped:
1:05 PM Changeset in webkit [84410] by krit@webkit.org
  • 9 edits in trunk/Source/WebCore

2011-04-20 Dirk Schulze <​krit@webkit.org>

Reviewed by Simon Fraser.

SVG feDropShadow implementation of SVG Filters 1.2
​https://bugs.webkit.org/show_bug.cgi?id=52513

Support different radii for horizontal and vertical blurring in ShadowBlur. This is a preperation for feDropShadow.
feDropShadow is a new shorthand filter effect for the up-comming Filter 1.0 specification and will use ShadowBlur.
feGaussianBlur might make use of ShadowBlur as well later (on SourceAlpha input).
Added ShadowBlur to remaining platforms.

The changes can't be tested before the changes in feGaussianBlur or feDropShadow. Current behavior is not affected.

  • Android.mk:
  • CMakeLists.txt:
  • GNUmakefile.list.am:
  • WebCore.pro:
  • platform/graphics/FloatSize.h: (WebCore::FloatSize::scale):
  • platform/graphics/ShadowBlur.cpp: (WebCore::ScratchBuffer::ScratchBuffer): (WebCore::ScratchBuffer::setLastShadowValues): (WebCore::ScratchBuffer::setLastInsetShadowValues): (WebCore::ScratchBuffer::matchesLastShadow): (WebCore::ScratchBuffer::matchesLastInsetShadow): (WebCore::ScratchBuffer::clearScratchBuffer): (WebCore::ShadowBlur::ShadowBlur): (WebCore::calculateLobes): (WebCore::ShadowBlur::blurLayerImage): (WebCore::ShadowBlur::adjustBlurRadius): (WebCore::ShadowBlur::calculateLayerBoundingRect): (WebCore::computeSliceSizesFromRadii): (WebCore::ShadowBlur::templateSize): (WebCore::ShadowBlur::drawInsetShadowWithTiling): (WebCore::ShadowBlur::drawRectShadowWithTiling): (WebCore::ShadowBlur::drawLayerPieces):
  • platform/graphics/ShadowBlur.h:
  • platform/graphics/cg/GraphicsContextCG.cpp: (WebCore::GraphicsContext::fillRect): (WebCore::GraphicsContext::fillRoundedRect): (WebCore::GraphicsContext::fillRectWithRoundedHole):
12:56 PM Changeset in webkit [84409] by vrk@chromium.org
  • 4 edits in trunk/LayoutTests

2011-03-07 Victoria Kirst <​vrk@google.com>

Reviewed by Mihai Parparita.

Make LayoutTests robust to latency btwn playing event and frame rendering
​https://bugs.webkit.org/show_bug.cgi?id=55918

  • media/controls-after-reload.html:
  • media/controls-without-preload.html:
  • media/video-paint-test.js:
  • platform/chromium/test_expectations.txt:
12:54 PM Changeset in webkit [84408] by commit-queue@webkit.org
  • 5 edits in trunk/LayoutTests

2011-04-20 Taiju TSUIKI <​tzik@google.com>

Reviewed by David Levin.

Add overwrite copy/move tests for FileSystem API
​https://bugs.webkit.org/show_bug.cgi?id=58864

  • fast/filesystem/op-copy-expected.txt:
  • fast/filesystem/op-move-expected.txt:
  • fast/filesystem/resources/op-copy.js:
  • fast/filesystem/resources/op-move.js:
12:52 PM WebKit Team edited by mahesh.kulkarni@nokia.com
(diff)
12:48 PM Changeset in webkit [84407] by mahesh.kulkarni@nokia.com
  • 1 edit in trunk/Tools/Scripts/webkitpy/common/config/committers.py

Adding myself to committer list

12:38 PM Changeset in webkit [84406] by tony@chromium.org
  • 5331 edits in trunk/LayoutTests

2011-04-20 Tony Chang <​tony@chromium.org>

Unreviewed, embedding checksums in png files in platform/chromium-linux*.

12:21 PM Changeset in webkit [84405] by ojan@chromium.org
  • 2 edits in trunk/Tools

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

Reviewed by Eric Seidel.

add expandAll/collapseAll links
​https://bugs.webkit.org/show_bug.cgi?id=59011

  • Scripts/webkitpy/layout_tests/layout_package/json_results.html:
12:18 PM Changeset in webkit [84404] by yael.aharon@nokia.com
  • 1 edit
    2 adds in trunk/LayoutTests

Unreviewed.

Adjust Windows results after r84398.

  • platform/win/fast/borders: Added.
  • platform/win/fast/borders/bidi-012-expected.txt: Added.
12:13 PM Changeset in webkit [84403] by tony@chromium.org
  • 5595 edits in trunk/LayoutTests

2011-04-20 Tony Chang <​tony@chromium.org>

Unreviewed, embedding checksums in png files in platform/chromium-win*.

12:09 PM Changeset in webkit [84402] by barraclough@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

ARMv7 build fix II.

  • jit/JITOpcodes32_64.cpp:

(JSC::JIT::softModulo):

12:02 PM Changeset in webkit [84401] by barraclough@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

ARMv7 build fix.

  • assembler/MacroAssemblerARM.h:

(JSC::MacroAssemblerARM::test8):

11:54 AM Changeset in webkit [84400] by evan@chromium.org
  • 4 edits in trunk/Source/WebKit/chromium

2011-04-20 Evan Martin <​evan@chromium.org>

Reviewed by Tony Chang.

[chromium] expose title text direction on WebDataSource
​https://bugs.webkit.org/show_bug.cgi?id=59010

  • public/WebDataSource.h:
  • src/WebDataSourceImpl.cpp: (WebKit::WebDataSourceImpl::pageTitle): (WebKit::WebDataSourceImpl::pageTitleDirection):
  • src/WebDataSourceImpl.h:
11:44 AM Changeset in webkit [84399] by barraclough@apple.com
  • 12 edits in trunk/Source/JavaScriptCore

Rationalize MacroAssembler branch methods
​https://bugs.webkit.org/show_bug.cgi?id=58950

Reviewed by Oliver Hunt.

The MacroAssembler currently exposes x86's weird behaviour that the 'setcc'
instruction only sets the low 8 bits of a register. Stop that.

Having done so, to clarify remove the 'set32' prefix from test & compare
instructions - these methods all now set a full 32/64 bit register (Ptr size).
The size in the function name should indicate the amount of data being compared.

Also split out the 'Condition' enum into 'RelationalCondition' and
'ResultCondition'. The former is used in binary comparison, the latter is a unary
condition check on the result of an operation.

  • JavaScriptCore.xcodeproj/project.pbxproj:
  • assembler/MacroAssembler.h:

(JSC::MacroAssembler::branchPtr):
(JSC::MacroAssembler::branch32):
(JSC::MacroAssembler::branch16):
(JSC::MacroAssembler::branchTestPtr):
(JSC::MacroAssembler::comparePtr):
(JSC::MacroAssembler::branchAddPtr):
(JSC::MacroAssembler::branchSubPtr):
(JSC::MacroAssembler::branchTest8):

  • assembler/MacroAssemblerARM.h:

(JSC::MacroAssemblerARM::branch8):
(JSC::MacroAssemblerARM::branch32):
(JSC::MacroAssemblerARM::branch32WithUnalignedHalfWords):
(JSC::MacroAssemblerARM::branch16):
(JSC::MacroAssemblerARM::branchTest8):
(JSC::MacroAssemblerARM::branchTest32):
(JSC::MacroAssemblerARM::branchAdd32):
(JSC::MacroAssemblerARM::branchMul32):
(JSC::MacroAssemblerARM::branchSub32):
(JSC::MacroAssemblerARM::branchNeg32):
(JSC::MacroAssemblerARM::branchOr32):
(JSC::MacroAssemblerARM::compare32):
(JSC::MacroAssemblerARM::test32):
(JSC::MacroAssemblerARM::test8):
(JSC::MacroAssemblerARM::branchPtrWithPatch):
(JSC::MacroAssemblerARM::ARMCondition):

  • assembler/MacroAssemblerARMv7.h:

(JSC::MacroAssemblerARMv7::branch32):
(JSC::MacroAssemblerARMv7::branch32WithUnalignedHalfWords):
(JSC::MacroAssemblerARMv7::branch16):
(JSC::MacroAssemblerARMv7::branch8):
(JSC::MacroAssemblerARMv7::branchTest32):
(JSC::MacroAssemblerARMv7::branchTest8):
(JSC::MacroAssemblerARMv7::branchAdd32):
(JSC::MacroAssemblerARMv7::branchMul32):
(JSC::MacroAssemblerARMv7::branchOr32):
(JSC::MacroAssemblerARMv7::branchSub32):
(JSC::MacroAssemblerARMv7::compare32):
(JSC::MacroAssemblerARMv7::test32):
(JSC::MacroAssemblerARMv7::test8):
(JSC::MacroAssemblerARMv7::branchPtrWithPatch):
(JSC::MacroAssemblerARMv7::makeBranch):
(JSC::MacroAssemblerARMv7::armV7Condition):

  • assembler/MacroAssemblerMIPS.h:

(JSC::MacroAssemblerMIPS::branch8):
(JSC::MacroAssemblerMIPS::branch32):
(JSC::MacroAssemblerMIPS::branch32WithUnalignedHalfWords):
(JSC::MacroAssemblerMIPS::branch16):
(JSC::MacroAssemblerMIPS::branchTest32):
(JSC::MacroAssemblerMIPS::branchTest8):
(JSC::MacroAssemblerMIPS::branchAdd32):
(JSC::MacroAssemblerMIPS::branchMul32):
(JSC::MacroAssemblerMIPS::branchSub32):
(JSC::MacroAssemblerMIPS::branchOr32):
(JSC::MacroAssemblerMIPS::compare32):
(JSC::MacroAssemblerMIPS::test8):
(JSC::MacroAssemblerMIPS::test32):
(JSC::MacroAssemblerMIPS::branchPtrWithPatch):

  • assembler/MacroAssemblerX86.h:

(JSC::MacroAssemblerX86::branch32):
(JSC::MacroAssemblerX86::branchPtrWithPatch):

  • assembler/MacroAssemblerX86Common.h:

(JSC::MacroAssemblerX86Common::branch8):
(JSC::MacroAssemblerX86Common::branch32):
(JSC::MacroAssemblerX86Common::branch32WithUnalignedHalfWords):
(JSC::MacroAssemblerX86Common::branch16):
(JSC::MacroAssemblerX86Common::branchTest32):
(JSC::MacroAssemblerX86Common::branchTest8):
(JSC::MacroAssemblerX86Common::branchAdd32):
(JSC::MacroAssemblerX86Common::branchMul32):
(JSC::MacroAssemblerX86Common::branchSub32):
(JSC::MacroAssemblerX86Common::branchNeg32):
(JSC::MacroAssemblerX86Common::branchOr32):
(JSC::MacroAssemblerX86Common::compare32):
(JSC::MacroAssemblerX86Common::test8):
(JSC::MacroAssemblerX86Common::test32):
(JSC::MacroAssemblerX86Common::x86Condition):

  • assembler/MacroAssemblerX86_64.h:

(JSC::MacroAssemblerX86_64::comparePtr):
(JSC::MacroAssemblerX86_64::branchPtr):
(JSC::MacroAssemblerX86_64::branchTestPtr):
(JSC::MacroAssemblerX86_64::branchAddPtr):
(JSC::MacroAssemblerX86_64::branchSubPtr):
(JSC::MacroAssemblerX86_64::branchPtrWithPatch):
(JSC::MacroAssemblerX86_64::branchTest8):

  • dfg/DFGSpeculativeJIT.cpp:

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

  • jit/JITOpcodes.cpp:

(JSC::JIT::emit_op_eq):
(JSC::JIT::emit_op_neq):
(JSC::JIT::compileOpStrictEq):
(JSC::JIT::emit_op_eq_null):
(JSC::JIT::emit_op_neq_null):

  • jit/JITOpcodes32_64.cpp:

(JSC::JIT::emit_op_eq):
(JSC::JIT::emit_op_neq):
(JSC::JIT::compileOpStrictEq):
(JSC::JIT::emit_op_eq_null):
(JSC::JIT::emit_op_neq_null):

11:30 AM ThreadCommunication edited by levin@chromium.org
Small fix. (diff)
11:15 AM Changeset in webkit [84398] by yael.aharon@nokia.com
  • 1 edit
    4 adds
    11 deletes in trunk/LayoutTests

Replace bidi-004.html with its new version.
​https://bugs.webkit.org/show_bug.cgi?id=58984

Reviewed by Eric Seidel.

  • fast/borders/bidi-004.html: Removed.
  • fast/borders/bidi-012.html: Added.
  • platform/chromium-linux/fast/borders/bidi-004-expected.png: Removed.
  • platform/chromium-mac-leopard/fast/borders/bidi-004-expected.png: Removed.
  • platform/chromium-win/fast/borders/bidi-004-expected.png: Removed.
  • platform/chromium-win/fast/borders/bidi-004-expected.txt: Removed.
  • platform/gtk/fast/borders/bidi-004-expected.checksum: Removed.
  • platform/gtk/fast/borders/bidi-004-expected.png: Removed.
  • platform/gtk/fast/borders/bidi-004-expected.txt: Removed.
  • platform/mac/fast/borders/bidi-004-expected.checksum: Removed.
  • platform/mac/fast/borders/bidi-004-expected.png: Removed.
  • platform/mac/fast/borders/bidi-004-expected.txt: Removed.
  • platform/mac/fast/borders/bidi-012-expected.checksum: Added.
  • platform/mac/fast/borders/bidi-012-expected.png: Added.
  • platform/mac/fast/borders/bidi-012-expected.txt: Added.
11:09 AM Changeset in webkit [84397] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

Unreviewed typo fix.

  • platform/qt-wk2/Skipped:
10:57 AM Changeset in webkit [84396] by kevino@webkit.org
  • 7 edits in trunk/Tools

[wx] Unreviewed build fixes for wx 2.9.1, Cocoa and 64-bit builds.

10:46 AM Changeset in webkit [84395] by tony@chromium.org
  • 1841 edits in trunk/LayoutTests

2011-04-20 Tony Chang <​tony@chromium.org>

Unreviewed, embedding checksums in png files in platform/google*,
platform/chromium-gpu*, platform/chromium-mac*.

10:25 AM Changeset in webkit [84394] by rolandsteiner@chromium.org
  • 13 edits in trunk/Source/WebCore

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

Reviewed by Dimitri Glazkov.

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

.) Change base class of ShadowRoot from DocumentFragment to TreeScope.
.) Re-enable tree scope handling in Node (had ASSERT_NOT_REACHED, etc.).
.) Merged setTreeScope() with setTreeScopeRecursively()
.) Call setTreeScopeRecursively in DOM manipulation functions where applicable.
.) Adapt JavaScript Node wrappers (temporary code - see ​https://bugs.webkit.org/show_bug.cgi?id=58704).

No new tests. (refactoring)

  • bindings/js/JSNodeCustom.cpp: (WebCore::createWrapperInline):
  • bindings/v8/custom/V8NodeCustom.cpp: (WebCore::toV8Slow):
  • dom/ContainerNode.cpp: (WebCore::ContainerNode::takeAllChildrenFrom): (WebCore::ContainerNode::removeBetween): (WebCore::ContainerNode::removeChildren): (WebCore::ContainerNode::parserAddChild):
  • dom/Document.cpp: (WebCore::Document::Document): (WebCore::Document::~Document): (WebCore::Document::setDocType):
  • dom/Element.h:
  • dom/Node.cpp: (WebCore::Node::treeScope): (WebCore::Node::setTreeScopeRecursively):
  • dom/Node.h: (WebCore::Node::document):
  • dom/ShadowRoot.cpp: (WebCore::ShadowRoot::ShadowRoot): (WebCore::ShadowRoot::~ShadowRoot): (WebCore::ShadowRoot::nodeType): (WebCore::ShadowRoot::cloneNode): (WebCore::ShadowRoot::childTypeAllowed):
  • dom/ShadowRoot.h: (WebCore::toShadowRoot):
  • dom/TreeScope.cpp: (WebCore::TreeScope::TreeScope): (WebCore::TreeScope::setParentTreeScope):
  • dom/TreeScope.h:
  • rendering/RenderSlider.cpp:
10:24 AM Changeset in webkit [84393] by tony@chromium.org
  • 4555 edits in trunk/LayoutTests

2011-04-20 Tony Chang <​tony@chromium.org>

Unreviewed, embedding checksums in png files in platform/mac-*.

10:16 AM Changeset in webkit [84392] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt][WK2] WebProcess crashes intermittently: "FAIL: TestControllerRunLoop timed out."

  • platform/qt-wk2/Skipped: Add http/tests/security/contentSecurityPolicy/media-src-blocked.html.
10:09 AM Changeset in webkit [84391] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt][WK2] Skip new failing tests.

  • platform/qt-wk2/Skipped:
10:06 AM Changeset in webkit [84390] by mdelaney@apple.com
  • 3 edits in trunk/LayoutTests

2011-04-20 Matthew Delaney <​mdelaney@apple.com>

Reviewed by Dan Bernstein.

Update fast/canvas/font-update.html to use dumpAsText.
​https://bugs.webkit.org/show_bug.cgi?id=59003

  • fast/canvas/font-update-expected.txt:
  • fast/canvas/font-update.html: This test is only useful as a pixel test. Dumping the render tree will always show the same thing regardless of what's done on the canvas in the test. However, with new render tree schemes (canvases in layers and such) the render trees will even look different but not actually failing what the test tests.
10:00 AM Changeset in webkit [84389] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

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

Reviewed by Pavel Feldman.

Web Inspector: Update front-end files list in WebCore.vcproj
​https://bugs.webkit.org/show_bug.cgi?id=58896

  • WebCore.vcproj/WebCore.vcproj:
9:56 AM Changeset in webkit [84388] by kerz@chromium.org
  • 1 copy in branches/chromium/742

branching for chromium m12

9:55 AM Changeset in webkit [84387] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

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

Reviewed by Steve Block.

prepare tests for rebaselining
​https://bugs.webkit.org/show_bug.cgi?id=58989

  • platform/chromium/test_expectations.txt:
9:47 AM Changeset in webkit [84386] by tony@chromium.org
  • 2 edits in trunk/LayoutTests

2011-04-20 Tony Chang <​tony@chromium.org>

Fixed lint errors in test_expectations.txt.

  • platform/chromium/test_expectations.txt:
9:46 AM Changeset in webkit [84385] by Dimitri Glazkov
  • 2 edits in trunk/Source/WebCore

2011-04-20 Dimitri Glazkov <Dimitri Glazkov>

Qt build fix after r84380.

  • css/CSSPrimitiveValueMappings.h: Moved EBorderCollapse-related casts from under ENABLE(SVG) guard.
9:41 AM Changeset in webkit [84384] by jknotten@chromium.org
  • 2 edits in trunk/LayoutTests

2011-04-20 John Knottenbelt <​jknotten@chromium.org>

Unreviewed.

[Chromium] Remove crash expectation for http/tests/local/formdata/send-form-data-with-sliced-file.html
​https://bugs.webkit.org/show_bug.cgi?id=59004

  • platform/chromium/test_expectations.txt:
9:39 AM Changeset in webkit [84383] by alex
  • 2 edits in trunk/Source/WebKit2

2011-04-20 Alejandro G. Castro <​alex@igalia.com>

Fixed GTK+ compilation after r84206.

  • GNUmakefile.am:
9:24 AM Changeset in webkit [84382] by jknotten@chromium.org
  • 2 edits in trunk/LayoutTests

2011-04-20 John Knottenbelt <​jknotten@chromium.org>

Unreviewed.

[Chromium] Adjust test expectations.
​https://bugs.webkit.org/show_bug.cgi?id=59002

  • platform/chromium/test_expectations.txt:
9:21 AM Changeset in webkit [84381] by Carlos Garcia Campos
  • 4 edits in trunk/Source

2011-04-20 Carlos Garcia Campos <​cgarcia@igalia.com>

Reviewed by Martin Robinson.

[GTK] Build libWebCore with the gtk sources too
​https://bugs.webkit.org/show_bug.cgi?id=58968

In some cases webcore files are compiled twice, so it improves the
build time.

  • GNUmakefile.am: Add webcore gtk sources to libWebCore sources.

2011-04-20 Carlos Garcia Campos <​cgarcia@igalia.com>

Reviewed by Martin Robinson.

[GTK] Build libWebCore with the gtk sources too
​https://bugs.webkit.org/show_bug.cgi?id=58968

In some cases webcore files are compiled twice, so it improves the
build time.

  • GNUmakefile.am: Remove webcore gtk sources from libwebkitgtk sources.
9:14 AM Changeset in webkit [84380] by commit-queue@webkit.org
  • 5 edits in trunk/Source/WebCore

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

Reviewed by Dimitri Glazkov.

Allow casting between CSSPrimitiveValue and EBorderCollapse to remove special-case logic from CSSStyleSelector.
​https://bugs.webkit.org/show_bug.cgi?id=58964

No new tests as no new functionality added.

  • css/CSSPrimitiveValueMappings.h: Support casting to/from EBorderCollapse.
  • css/CSSStyleSelector.cpp: Use new casting ability to simplify code.
  • rendering/style/RenderStyle.h: Use EBorderCollapse instead of a bool for representing border collapse member variable.
  • rendering/style/RenderStyleConstants.h: Added EBorderCollapse enum.
9:11 AM Changeset in webkit [84379] by Adam Roben
  • 19 edits
    18 copies
    4 adds
    1 delete in trunk/LayoutTests

Add Windows 7 SP1 baselines for some SVG tests that progressed in SP1

These baselines were previously thought to be WebKit2-specific. However, it turned out that
the WebKit2 bots had Windows 7 SP1 installed, while the WebKit1 bots just had Windows 7
installed, and that was the crucial difference. So this patch reclassifies the new baselines
as being SP1 vs. not-SP1, rather than WK2 vs. WK1. I also added pixel results.

Fixes <​http://webkit.org/b/56617> Several tests fail on Windows 7 SP1 due to font
differences from Windows 7 "SP0"

  • platform/win-7sp0/svg/W3C-I18N/text-dirLTR-ubOverride-expected.checksum:
  • platform/win-7sp0/svg/W3C-I18N/text-dirLTR-ubOverride-expected.png:
  • platform/win-7sp0/svg/W3C-I18N/text-dirLTR-ubOverride-expected.txt:
  • platform/win-7sp0/svg/W3C-I18N/tspan-dirLTR-ubOverride-in-default-context-expected.checksum:
  • platform/win-7sp0/svg/W3C-I18N/tspan-dirLTR-ubOverride-in-default-context-expected.png:
  • platform/win-7sp0/svg/W3C-I18N/tspan-dirLTR-ubOverride-in-default-context-expected.txt:
  • platform/win-7sp0/svg/W3C-I18N/tspan-dirLTR-ubOverride-in-ltr-context-expected.checksum:
  • platform/win-7sp0/svg/W3C-I18N/tspan-dirLTR-ubOverride-in-ltr-context-expected.png:
  • platform/win-7sp0/svg/W3C-I18N/tspan-dirLTR-ubOverride-in-ltr-context-expected.txt:
  • platform/win-7sp0/svg/W3C-I18N/tspan-dirNone-ubOverride-in-default-context-expected.checksum:
  • platform/win-7sp0/svg/W3C-I18N/tspan-dirNone-ubOverride-in-default-context-expected.png:
  • platform/win-7sp0/svg/W3C-I18N/tspan-dirNone-ubOverride-in-default-context-expected.txt:
  • platform/win-7sp0/svg/W3C-I18N/tspan-dirNone-ubOverride-in-ltr-context-expected.checksum:
  • platform/win-7sp0/svg/W3C-I18N/tspan-dirNone-ubOverride-in-ltr-context-expected.png:
  • platform/win-7sp0/svg/W3C-I18N/tspan-dirNone-ubOverride-in-ltr-context-expected.txt:
  • platform/win-7sp0/svg/text/bidi-text-query-expected.checksum:
  • platform/win-7sp0/svg/text/bidi-text-query-expected.png:
  • platform/win-7sp0/svg/text/bidi-text-query-expected.txt:

Copied these results from platform/win.

  • platform/win-wk2/svg/W3C-I18N/text-dirLTR-ubOverride-expected.txt:
  • platform/win-wk2/svg/W3C-I18N/tspan-dirLTR-ubOverride-in-default-context-expected.txt:
  • platform/win-wk2/svg/W3C-I18N/tspan-dirLTR-ubOverride-in-ltr-context-expected.txt:
  • platform/win-wk2/svg/W3C-I18N/tspan-dirNone-ubOverride-in-default-context-expected.txt:
  • platform/win-wk2/svg/W3C-I18N/tspan-dirNone-ubOverride-in-ltr-context-expected.txt:
  • platform/win-wk2/svg/text/bidi-text-query-expected.txt:

Moved these results to platform/win.

  • platform/win/svg/W3C-I18N/text-dirLTR-ubOverride-expected.checksum:
  • platform/win/svg/W3C-I18N/text-dirLTR-ubOverride-expected.png:
  • platform/win/svg/W3C-I18N/text-dirLTR-ubOverride-expected.txt:
  • platform/win/svg/W3C-I18N/tspan-dirLTR-ubOverride-in-default-context-expected.checksum:
  • platform/win/svg/W3C-I18N/tspan-dirLTR-ubOverride-in-default-context-expected.png:
  • platform/win/svg/W3C-I18N/tspan-dirLTR-ubOverride-in-default-context-expected.txt:
  • platform/win/svg/W3C-I18N/tspan-dirLTR-ubOverride-in-ltr-context-expected.checksum:
  • platform/win/svg/W3C-I18N/tspan-dirLTR-ubOverride-in-ltr-context-expected.png:
  • platform/win/svg/W3C-I18N/tspan-dirLTR-ubOverride-in-ltr-context-expected.txt:
  • platform/win/svg/W3C-I18N/tspan-dirNone-ubOverride-in-default-context-expected.checksum:
  • platform/win/svg/W3C-I18N/tspan-dirNone-ubOverride-in-default-context-expected.png:
  • platform/win/svg/W3C-I18N/tspan-dirNone-ubOverride-in-default-context-expected.txt:
  • platform/win/svg/W3C-I18N/tspan-dirNone-ubOverride-in-ltr-context-expected.checksum:
  • platform/win/svg/W3C-I18N/tspan-dirNone-ubOverride-in-ltr-context-expected.png:
  • platform/win/svg/W3C-I18N/tspan-dirNone-ubOverride-in-ltr-context-expected.txt:
  • platform/win/svg/text/bidi-text-query-expected.checksum:
  • platform/win/svg/text/bidi-text-query-expected.png:
  • platform/win/svg/text/bidi-text-query-expected.txt:

The .txt files came from platform/win-wk2, and the .png/.checksum files were
newly-generated.

9:05 AM Changeset in webkit [84378] by Martin Robinson
  • 1 edit
    3 deletes in trunk/LayoutTests

2011-04-20 Martin Robinson <​mrobinson@igalia.com>

platform/gtk/fast/dom/gc-10-expected.png is corrupt
​https://bugs.webkit.org/show_bug.cgi?id=58940

Remove expected results for fast/dom/gc-10.html since it is crashing on GTK+.

  • platform/gtk/fast/dom/gc-10-expected.checksum: Removed.
  • platform/gtk/fast/dom/gc-10-expected.png: Removed.
  • platform/gtk/fast/dom/gc-10-expected.txt: Removed.
9:05 AM Changeset in webkit [84377] by Adam Roben
  • 3 edits in trunk/Tools

Replace old-run-webkit-tests's win-7 platform with win-7sp0

It was an error that we had a win-7 platform to begin with. Windows 7 was the most recent
version of Windows we cared about, so its results should just go in win. But now that
Windows 7 SP1 exists (and has different results for some tests), we need a place to store
results for Windows 7 without SP1 (and older versions of Windows).

Part of <​http://webkit.org/b/56617> Several tests fail on Windows 7 SP1 due to font
differences from Windows 7 "SP0"

Reviewed by David Kilzer.

  • Scripts/old-run-webkit-tests: Replaced win-7 with win-7sp0. Newer versions of Windows 7

will just use "win".

  • Scripts/webkitdirs.pm:

(isWindows7SP0): Added. Replaces isWindows7, because that function had no remaining callers.

8:44 AM Changeset in webkit [84376] by jknotten@chromium.org
  • 2 edits in trunk/LayoutTests

2011-04-20 John Knottenbelt <​jknotten@chromium.org>

Unreviewed.

[Chromium] Mark css2.1/t140201-c533-bgimage-00-a.html as crash on Linux, Win Debug
​https://bugs.webkit.org/show_bug.cgi?id=58995

Accidentally indicated MAC instead of WIN.

  • platform/chromium/test_expectations.txt:
8:39 AM Changeset in webkit [84375] by jknotten@chromium.org
  • 2 edits in trunk/LayoutTests

2011-04-20 John Knottenbelt <​jknotten@chromium.org>

Unreviewed.

[Chromium] Mark css2.1/t140201-c533-bgimage-00-a.html as crash on Linux, Win Debug
​https://bugs.webkit.org/show_bug.cgi?id=58995

  • platform/chromium/test_expectations.txt:
7:54 AM Changeset in webkit [84374] by senorblanco@chromium.org
  • 7 edits
    2 moves in trunk/Source/WebCore

2011-04-19 Stephen White <​senorblanco@chromium.org>

Reviewed by Kenneth Russell.

Move and rename GLES2Canvas.
​https://bugs.webkit.org/show_bug.cgi?id=58927

Move GLES2Canvas to platform/graphcs/gpu, where all the other
gpu-related files live, and rename it to GraphicsContextGPU to better
reflect its purpose.

Covered by existing tests.

  • WebCore.gypi:
  • platform/graphics/chromium/GLES2Canvas.cpp:
  • platform/graphics/chromium/GLES2Canvas.h:
  • platform/graphics/skia/GraphicsContextSkia.cpp: (WebCore::GraphicsContext::fillPath): (WebCore::GraphicsContext::setPlatformShadow):
  • platform/graphics/skia/ImageBufferSkia.cpp:
  • platform/graphics/skia/ImageSkia.cpp: (WebCore::drawBitmapGLES2):
  • platform/graphics/skia/PlatformContextSkia.cpp: (WebCore::PlatformContextSkia::setSharedGraphicsContext3D):
  • platform/graphics/skia/PlatformContextSkia.h: (WebCore::PlatformContextSkia::gpuCanvas):
7:46 AM Changeset in webkit [84373] by benjamin.poulain@nokia.com
  • 2 edits in trunk/Source/WebCore

[Qt] Clean the style of our cookie methods
​https://bugs.webkit.org/show_bug.cgi?id=58987

Reviewed by Andreas Kling.

Got rid of the variables named "u" in favor just converting the url when needed.

  • platform/qt/CookieJarQt.cpp:

(WebCore::setCookies): The variable p was unused.
The ::toAscii() depends on the default codec, use toLatin1() instead.
(WebCore::cookies): foreach() should use const reference.
(WebCore::cookieRequestHeaderFieldValue):

7:44 AM Changeset in webkit [84372] by Adam Roben
  • 2 edits in trunk/Tools

Teach webkitdirs::winVersion about build numbers

The build number isn't currently used, but soon will be.

As a bonus, winVersion now works on non-Cygwin variants of Perl!

Fixes <​http://webkit.org/b/58991> Need a way to distinguish between Windows 7 and Windows 7
SP1

Reviewed by Anders Carlsson.

  • Scripts/webkitdirs.pm:

(isAnyWindows): Added. Returns true if we're running on any Windows-ish Perl.
(determineWinVersion): Use cmd /c ver to get the version number, including the build
number. Parse the version number into a hash, similar to how osXVersion() works.

(isWindows7):
(isWindowsVista):
(isWindowsXP):
Updated for changes to winVersion().

7:40 AM Changeset in webkit [84371] by mihaip@chromium.org
  • 37 edits in trunk/Source/WebCore

2011-04-19 Mihai Parparita <​mihaip@chromium.org>

Reviewed by Adam Barth.

ScriptController.h is included in too many places
​https://bugs.webkit.org/show_bug.cgi?id=58963

Remove ScriptController.h #includes from a bunch of .cpp files where it
doesn't appear to be necessary.

  • bindings/js/JSCustomPositionCallback.cpp:
  • bindings/js/JSCustomPositionErrorCallback.cpp:
  • bindings/js/JSCustomVoidCallback.cpp:
  • bindings/js/JSCustomXPathNSResolver.cpp:
  • bindings/js/JSDOMWindowBase.cpp:
  • bindings/js/JSDOMWindowShell.cpp:
  • bindings/js/JSDocumentCustom.cpp:
  • bindings/js/ScriptDebugServer.cpp:
  • bindings/v8/NPV8Object.cpp:
  • bindings/v8/ScriptCachedFrameData.cpp:
  • bindings/v8/V8DOMWindowShell.cpp:
  • bindings/v8/V8DOMWrapper.cpp:
  • bindings/v8/V8IsolatedContext.cpp:
  • bindings/v8/V8Proxy.cpp:
  • bindings/v8/custom/V8LocationCustom.cpp:
  • dom/Node.cpp:
  • dom/ScriptElement.cpp:
  • dom/XMLDocumentParser.cpp:
  • dom/XMLDocumentParserQt.cpp:
  • html/HTMLEmbedElement.cpp:
  • html/HTMLObjectElement.cpp:
  • html/HTMLPlugInElement.cpp:
  • html/parser/HTMLConstructionSite.cpp:
  • html/parser/HTMLTreeBuilder.cpp:
  • inspector/InspectorProfilerAgent.cpp:
  • loader/icon/IconDatabase.cpp:
  • page/Chrome.cpp:
  • page/Navigator.cpp:
  • page/Page.cpp:
  • plugins/PluginView.cpp:
  • plugins/gtk/PluginViewGtk.cpp:
  • plugins/qt/PluginViewQt.cpp:
  • plugins/symbian/PluginViewSymbian.cpp:
  • plugins/win/PluginViewWin.cpp:
  • svg/SVGDocumentExtensions.cpp:
  • xml/XMLTreeViewer.cpp:
7:23 AM Changeset in webkit [84370] by Adam Roben
  • 2 edits in trunk/Source/WebKit2

Remove some unnecessary VS junk related to NativeWebMouseEventWin.cpp

  • win/WebKit2.vcproj:
7:21 AM Changeset in webkit [84369] by jberlin@webkit.org
  • 2 edits in trunk/LayoutTests

[Snow Leopard Intel WebKit2 Release Tests] editing/execCommand/convert-style-elements-to-spans.html
flakey.
​https://bugs.webkit.org/show_bug.cgi?id=58183

Skip the flakey test in order to get the bots green.

  • platform/mac-wk2/Skipped:
7:17 AM Changeset in webkit [84368] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

2011-04-20 Leon Scroggins <​scroggo@google.com>

Reviewed by Kenneth Russell.

Mark tests as failing for rebaselining.
​https://bugs.webkit.org/show_bug.cgi?id=58839

We are making a change to the way gradients
are calculated, so disable tests for now.

  • platform/chromium/test_expectations.txt:
7:11 AM Changeset in webkit [84367] by Philippe Normand
  • 2 edits in trunk/LayoutTests

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

Unreviewed, skip GTK crashing test
storage/open-database-set-empty-version.html.

  • platform/gtk/Skipped:
7:04 AM Changeset in webkit [84366] by jberlin@webkit.org
  • 2 edits in trunk/LayoutTests

WebKit2 needs layoutTestController.overridePreference
​https://bugs.webkit.org/show_bug.cgi?id=42197

Add another test that uses layoutTestController.overridePreference
(fast/loader/scroll-position-restored-on-back.html) to the Skipped list to get the bots
green.

  • platform/mac-wk2/Skipped:
6:49 AM Changeset in webkit [84365] by pfeldman@chromium.org
  • 3 edits in trunk/Source/WebCore

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

Reviewed by Yury Semikhatsky.

Web Inspector: add Save As to the Resources panel context menu.
​https://bugs.webkit.org/show_bug.cgi?id=58983

  • English.lproj/localizedStrings.js:
  • inspector/front-end/ResourcesPanel.js: (WebInspector.FrameResourceTreeElement.prototype.onattach): (WebInspector.FrameResourceTreeElement.prototype._handleContextMenuEvent.save): (WebInspector.FrameResourceTreeElement.prototype._handleContextMenuEvent): (WebInspector.ResourceRevisionTreeElement.prototype._handleContextMenuEvent.save): (WebInspector.ResourceRevisionTreeElement.prototype._handleContextMenuEvent):
6:46 AM Changeset in webkit [84364] by kbalazs@webkit.org
  • 8 edits in trunk/Source

2011-04-20 Balazs Kelemen <​kbalazs@webkit.org>

Reviewed by Csaba Osztrogonác.

[Qt] Cleanup includepath adjustment for generated files
​https://bugs.webkit.org/show_bug.cgi?id=58869

  • JavaScriptCore.pri: Add the directory of generated files to the include path with absolute path to make it valid in the final build step.

2011-04-20 Balazs Kelemen <​kbalazs@webkit.org>

Reviewed by Csaba Osztrogonác.

[Qt] Cleanup includepath adjustment for generated files
​https://bugs.webkit.org/show_bug.cgi?id=58869

Build cleanup, no new tests.

  • WebCore.pri: Add the directory of generated files to the include path with absolute path to make it valid in the final build step. Remove unnecessary logic.

2011-04-20 Balazs Kelemen <​kbalazs@webkit.org>

Reviewed by Csaba Osztrogonác.

[Qt] Cleanup includepath adjustment for generated files
​https://bugs.webkit.org/show_bug.cgi?id=58869

  • QtWebKit.pro: Revert the hacky fix in r84174. Remove unnecessary logic.

2011-04-20 Balazs Kelemen <​kbalazs@webkit.org>

Reviewed by Csaba Osztrogonác.

[Qt] Cleanup includepath adjustment for generated files
​https://bugs.webkit.org/show_bug.cgi?id=58869

  • WebKit2.pri: Add the directory of generated files to the include path with absolute path to make it valid in the final build step.
6:42 AM Changeset in webkit [84363] by jknotten@chromium.org
  • 8 edits
    1 move
    8 deletes in trunk/LayoutTests

2011-04-20 John Knottenbelt <​jknotten@chromium.org>

Unreviewed.

[Chromium] Rebaseline some border-related tests on Mac
​https://bugs.webkit.org/show_bug.cgi?id=58985

Most likely related to ​http://trac.webkit.org/changeset/84341/
"background color of elements with border-radius shows around outer edge of border at corners"

  • platform/chromium-mac-leopard/fast/forms/menulist-clip-expected.png:
  • platform/chromium-mac-leopard/fast/forms/menulist-narrow-width-expected.png:
  • platform/chromium-mac-leopard/fast/forms/menulist-option-wrap-expected.png:
  • platform/chromium-mac-leopard/fast/forms/menulist-restrict-line-height-expected.png:
  • platform/chromium-mac-leopard/fast/forms/menulist-style-color-expected.png:
  • platform/chromium-mac-leopard/fast/forms/select-baseline-expected.png:
  • platform/chromium-mac/fast/borders/borderRadiusDashed01-expected.checksum: Removed.
  • platform/chromium-mac/fast/borders/borderRadiusDashed01-expected.png:
  • platform/chromium-mac/fast/forms/menulist-clip-expected.png: Removed.
  • platform/chromium-mac/fast/forms/menulist-narrow-width-expected.png: Removed.
  • platform/chromium-mac/fast/forms/menulist-option-wrap-expected.png: Removed.
  • platform/chromium-mac/fast/forms/menulist-restrict-line-height-expected.png: Removed.
  • platform/chromium-mac/fast/forms/menulist-style-color-expected.png: Removed.
  • platform/chromium-mac/fast/forms/select-baseline-expected.png: Removed.
  • platform/chromium-mac/fast/forms/select-empty-option-height-expected.png: Removed.
6:20 AM Changeset in webkit [84362] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-04-20 Mariusz Grzegorczyk <​mariusz.g@samsung.com>

Reviewed by Andreas Kling.

[EFL] Initializing m_window member field in PluginView.cpp for efl port
​https://bugs.webkit.org/show_bug.cgi?id=58311

No new tests needed: only initializing class member.

  • plugins/PluginView.cpp: (WebCore::PluginView::PluginView):
6:17 AM Changeset in webkit [84361] by wjmaclean@chromium.org
  • 6 edits in trunk/Tools

2011-04-20 W. James MacLean <​wjmaclean@chromium.org>

Reviewed by Kenneth Russell.

Add support for composite-to-texture to DumpRenderTree.
​https://bugs.webkit.org/show_bug.cgi?id=58675

  • DumpRenderTree/chromium/DumpRenderTree.cpp: (main):
  • DumpRenderTree/chromium/TestShell.cpp: (TestShell::TestShell): (TestShell::resetWebSettings):
  • DumpRenderTree/chromium/TestShell.h: (TestShell::setCompositeToTexture):
  • DumpRenderTree/chromium/WebPreferences.cpp: (WebPreferences::reset): (WebPreferences::applyTo):
  • DumpRenderTree/chromium/WebPreferences.h:
5:45 AM Changeset in webkit [84360] by yurys@chromium.org
  • 2 edits in trunk/LayoutTests

2011-04-20 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Tony Gentilcore.

http/tests/misc/unloadable-script.html fails if <script> error event doesn't bubble
​https://bugs.webkit.org/show_bug.cgi?id=58981

  • http/tests/misc/unloadable-script.html: canged the test so that it uses <script onerror> instead od <body onerror>
5:40 AM Changeset in webkit [84359] by pfeldman@chromium.org
  • 3 edits in trunk/Source/WebCore

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

Not reviewed: fix Leopard compile.

  • inspector/InspectorFrontendClient.h:
  • inspector/InspectorFrontendClientLocal.h: (WebCore::InspectorFrontendClientLocal::saveAs):
5:27 AM Changeset in webkit [84358] by pfeldman@chromium.org
  • 15 edits in trunk/Source

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

Reviewed by Yury Semikhatsky.

Web Inspector: add saveAs into the InspectorFrontendHost binding.
​https://bugs.webkit.org/show_bug.cgi?id=58979

  • English.lproj/localizedStrings.js:
  • inspector/InspectorFrontendClient.h: (WebCore::InspectorFrontendClient::saveAs): (WebCore::InspectorFrontendClient::saveSessionSetting): (WebCore::InspectorFrontendClient::loadSessionSetting):
  • inspector/InspectorFrontendHost.cpp: (WebCore::InspectorFrontendHost::saveAs):
  • inspector/InspectorFrontendHost.h:
  • inspector/InspectorFrontendHost.idl:
  • inspector/front-end/InspectorFrontendHostStub.js: (.WebInspector.InspectorFrontendHostStub.prototype.saveAs.fr.onload): (.WebInspector.InspectorFrontendHostStub.prototype.saveAs):
  • inspector/front-end/NetworkPanel.js: (WebInspector.NetworkPanel.prototype._contextMenu): (WebInspector.NetworkPanel.prototype._copyAll): (WebInspector.NetworkPanel.prototype._copyResource): (WebInspector.NetworkPanel.prototype._exportAll): (WebInspector.NetworkPanel.prototype._exportResource):
  • inspector/front-end/Settings.js:
  • inspector/front-end/utilities.js: ():

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

Reviewed by Yury Semikhatsky.

Web Inspector: add saveAs into the InspectorFrontendHost binding.
​https://bugs.webkit.org/show_bug.cgi?id=58979

  • public/WebDevToolsFrontendClient.h: (WebKit::WebDevToolsFrontendClient::saveAs):
  • src/InspectorFrontendClientImpl.cpp: (WebKit::InspectorFrontendClientImpl::saveAs):
  • src/InspectorFrontendClientImpl.h:
  • src/js/DevTools.js:
4:52 AM Changeset in webkit [84357] by yurys@chromium.org
  • 12 edits
    2 adds in trunk

2011-04-20 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Tony Gentilcore.

Error event in <script> element shouldn't bubble
​https://bugs.webkit.org/show_bug.cgi?id=51040

  • fast/events/onerror-bubbling-expected.txt:
  • fast/events/onerror-bubbling.html:
  • svg/dom/SVGScriptElement/script-load-and-error-events.svg: error event is dispatched on the corresponding SVGScriptElement and shouldn't bubble, so the global error handler should never be invoked for script load errors. This behaviour is consistent with HTMLScriptElement element.
  • svg/dom/SVGScriptElement/script-onerror-bubbling-expected.txt: Added.
  • svg/dom/SVGScriptElement/script-onerror-bubbling.svg: Added.

2011-04-20 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Tony Gentilcore.

Error event in <script> element shouldn't bubble
​https://bugs.webkit.org/show_bug.cgi?id=51040

Test: svg/dom/SVGScriptElement/script-onerror-bubbling.svg

  • dom/ScriptElement.cpp: (WebCore::ScriptElement::dispatchErrorEvent): the method now has same implementation for all script elements, script load error event doesn' bubble.
  • dom/ScriptElement.h:
  • html/HTMLScriptElement.cpp:
  • html/HTMLScriptElement.h:
  • html/parser/HTMLScriptRunner.cpp: (WebCore::HTMLScriptRunner::executePendingScriptAndDispatchEvent):
  • svg/SVGScriptElement.cpp: (WebCore::SVGScriptElement::parseMappedAttribute): SVG script element now supports onerror hanlder which behaves the same way as onerror in HTML script element: it's invoked when script loading fails and the event doesn't bubble.
  • svg/SVGScriptElement.h:
4:10 AM Changeset in webkit [84356] by jknotten@chromium.org
  • 2 edits
    3 adds in trunk/LayoutTests

2011-04-20 John Knottenbelt <​jknotten@chromium.org>

Unreviewed.

[Chromium] Rebaseline fast/repaint/trailing-floats-root-line-box-overflow fast/canvas/webgl/css-webkit-canvas fast/canvas/webgl/css-webkit-canvas-repaint
​https://bugs.webkit.org/show_bug.cgi?id=58978

Windows bots are now generating baselines, so include them.
fast/repaint/trailing-floats-root-line-box-overflow looks like correct output.

  • platform/chromium-mac/fast/repaint/trailing-floats-root-line-box-overflow-expected.png: Added.
  • platform/chromium-win/fast/canvas/webgl/css-webkit-canvas-expected.png: Added.
  • platform/chromium-win/fast/canvas/webgl/css-webkit-canvas-repaint-expected.png: Added.
  • platform/chromium/test_expectations.txt:
3:57 AM Changeset in webkit [84355] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt] REGRESSION(r84296): platform/qt/fast/history/back-to-unreachable-url-then-forward.html fails
​https://bugs.webkit.org/show_bug.cgi?id=58977

  • platform/qt/Skipped: Add platform/qt/fast/history/back-to-unreachable-url-then-forward.html.
3:00 AM Changeset in webkit [84354] by Philippe Normand
  • 2 edits in trunk/LayoutTests

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

Unreviewed, skip failing test on GTK.

[GTK] fast/events/overflow-viewport-renderer-deleted.html crashes
​https://bugs.webkit.org/show_bug.cgi?id=58976

  • platform/gtk/Skipped:
2:44 AM Changeset in webkit [84353] by Philippe Normand
  • 5 edits
    21 adds in trunk/LayoutTests

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

Unreviewed, new GTK baselines for tests added in r84273.
Also skip a couple of failing tests.

  • platform/gtk/Skipped:
  • platform/gtk/fast/backgrounds/background-leakage-expected.checksum: Added.
  • platform/gtk/fast/backgrounds/background-leakage-expected.png: Added.
  • platform/gtk/fast/backgrounds/background-leakage-expected.txt: Added.
  • platform/gtk/fast/backgrounds/background-leakage-transforms-expected.checksum: Added.
  • platform/gtk/fast/backgrounds/background-leakage-transforms-expected.png: Added.
  • platform/gtk/fast/backgrounds/background-leakage-transforms-expected.txt: Added.
  • platform/gtk/fast/backgrounds/border-radius-split-background-expected.checksum: Added.
  • platform/gtk/fast/backgrounds/border-radius-split-background-expected.png: Added.
  • platform/gtk/fast/backgrounds/border-radius-split-background-expected.txt: Added.
  • platform/gtk/fast/backgrounds/border-radius-split-background-image-expected.checksum: Added.
  • platform/gtk/fast/backgrounds/border-radius-split-background-image-expected.png: Added.
  • platform/gtk/fast/backgrounds/border-radius-split-background-image-expected.txt: Added.
  • platform/gtk/fast/borders/border-radius-different-width-001-expected.checksum: Added.
  • platform/gtk/fast/borders/border-radius-different-width-001-expected.png: Added.
  • platform/gtk/fast/borders/border-radius-different-width-001-expected.txt: Added.
  • platform/gtk/fast/borders/border-radius-inset-outset-expected.checksum: Added.
  • platform/gtk/fast/borders/border-radius-inset-outset-expected.png: Added.
  • platform/gtk/fast/borders/border-radius-inset-outset-expected.txt: Added.
  • platform/gtk/fast/borders/table-borders-expected.checksum: Added.
  • platform/gtk/fast/borders/table-borders-expected.png: Added.
  • platform/gtk/fast/borders/table-borders-expected.txt: Added.
  • platform/gtk/fast/blockflow/border-styles-vertical-lr-expected.txt:
  • platform/gtk/fast/blockflow/border-styles-vertical-rl-expected.txt:
  • platform/gtk/fast/borders/border-styles-split-expected.txt:
2:35 AM Changeset in webkit [84352] by jknotten@chromium.org
  • 2 edits
    2 adds in trunk/LayoutTests

2011-04-20 John Knottenbelt <​jknotten@chromium.org>

Unreviewed.

[Chromium] Rebaseline overhanging-tall-block
​https://bugs.webkit.org/show_bug.cgi?id=58974

  • platform/chromium-win/fast/block/float/overhanging-tall-block-expected.png: Added.
  • platform/chromium-win/fast/block/float/overhanging-tall-block-expected.txt: Added.
  • platform/chromium/test_expectations.txt:
2:21 AM Changeset in webkit [84351] by pfeldman@chromium.org
  • 3 edits in trunk/Source/WebCore

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

Reviewed by Yury Semikhatsky.

Web Inspector: minor protocol cleanups (refined types).
​https://bugs.webkit.org/show_bug.cgi?id=58970

  • inspector/InjectedScriptSource.js:
  • inspector/Inspector.json:
1:32 AM WebKit Team edited by leo.yang@torchmobile.com.cn
(diff)
1:16 AM Changeset in webkit [84350] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-04-20 Leo Yang <​leo.yang@torchmobile.com.cn>

Reviewed by Eric Seidel.

Leaks beneath RenderSVGShadowTreeRootContainer::updateFromElement seen on SnowLeopard Intel Leaks
​https://bugs.webkit.org/show_bug.cgi?id=56672

~RenderSVGShadowTreeRootContainer() cleared shadow host only if
the shadow root is attached. This is wrong because it will
leak shadow root when the root is not attached. For example, it
leaks shadow root of <use> element which is pending on resource.

This patch is clearing shadow host if m_shadowRoot is valid to
prevent leaking.

Fixing leaking, no new tests.

  • rendering/svg/RenderSVGShadowTreeRootContainer.cpp: (WebCore::RenderSVGShadowTreeRootContainer::~RenderSVGShadowTreeRootContainer):
12:39 AM Changeset in webkit [84349] by mjs@apple.com
  • 2 edits in trunk/Source/WebCore

2011-04-19 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Antti Koivisto.

Reproducible crash two-finger zooming at multiple sites in WebCore::RenderLayer::isTransparent
​https://bugs.webkit.org/show_bug.cgi?id=58868

I could not figure out how to make a test.

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::paintLayer): Check for existence of a parent before calling a method on it.
12:18 AM Changeset in webkit [84348] by gyuyoung.kim@samsung.com
  • 3 edits in trunk/Source/WebCore

2011-04-20 Gyuyoung Kim <​gyuyoung.kim@samsung.com>

Reviewed by Daniel Bates.

[EFL] Add current time to media control panel.
​https://bugs.webkit.org/show_bug.cgi?id=58866

Implement paintMediaCurrentTime() to show playing time of media content.

  • platform/efl/RenderThemeEfl.cpp: (WebCore::RenderThemeEfl::RenderThemeEfl): (WebCore::RenderThemeEfl::formatMediaControlsCurrentTime): (WebCore::RenderThemeEfl::paintMediaCurrentTime):
  • platform/efl/RenderThemeEfl.h:
Note: See TracTimeline for information about the timeline view.