WebKit-https.git
6 years agoDFG should only have two mechanisms for describing effectfulness of nodes; previously...
fpizlo@apple.com [Mon, 9 Feb 2015 19:57:41 +0000 (19:57 +0000)]
DFG should only have two mechanisms for describing effectfulness of nodes; previously there were three
https://bugs.webkit.org/show_bug.cgi?id=141369

Reviewed by Michael Saboff.

We previously used the NodeMightClobber and NodeClobbersWorld NodeFlags to describe
effectfulness.  Starting over a year ago, we introduced a more powerful mechanism - the
DFG::clobberize() function.  Now we only have one remaining client of the old NodeFlags,
and everyone else uses DFG::clobberize().  We should get rid of those NodeFlags and
finally switch everyone over to DFG::clobberize().

Unfortunately there is still another place where effectfulness of nodes is described: the
AbstractInterpreter. This is because the AbstractInterpreter has special tuning both for
compile time performance and there are places where the AI is more precise than
clobberize() because of its flow-sensitivity.

This means that after this change there will be only two places, rather than three, where
the effectfulness of a node has to be described:

- DFG::clobberize()
- DFG::AbstractInterpreter

* dfg/DFGClobberize.cpp:
(JSC::DFG::clobbersWorld):
* dfg/DFGClobberize.h:
* dfg/DFGDoesGC.cpp:
(JSC::DFG::doesGC):
* dfg/DFGFixupPhase.cpp:
(JSC::DFG::FixupPhase::fixupNode):
(JSC::DFG::FixupPhase::attemptToMakeGetTypedArrayByteLength):
(JSC::DFG::FixupPhase::convertToGetArrayLength):
(JSC::DFG::FixupPhase::attemptToMakeGetTypedArrayByteOffset):
* dfg/DFGGraph.h:
(JSC::DFG::Graph::isPredictedNumerical): Deleted.
(JSC::DFG::Graph::byValIsPure): Deleted.
(JSC::DFG::Graph::clobbersWorld): Deleted.
* dfg/DFGNode.h:
(JSC::DFG::Node::convertToConstant):
(JSC::DFG::Node::convertToGetLocalUnlinked):
(JSC::DFG::Node::convertToGetByOffset):
(JSC::DFG::Node::convertToMultiGetByOffset):
(JSC::DFG::Node::convertToPutByOffset):
(JSC::DFG::Node::convertToMultiPutByOffset):
* dfg/DFGNodeFlags.cpp:
(JSC::DFG::dumpNodeFlags):
* dfg/DFGNodeFlags.h:
* dfg/DFGNodeType.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179840 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoUnreviewed, rolling out r179494.
commit-queue@webkit.org [Mon, 9 Feb 2015 19:53:28 +0000 (19:53 +0000)]
Unreviewed, rolling out r179494.
https://bugs.webkit.org/show_bug.cgi?id=141395

Caused slowdown in a WebKit client test scenario (Requested by
kling on #webkit).

Reverted changeset:

"[Cocoa] Make decoded image data purgeable ASAP."
https://bugs.webkit.org/show_bug.cgi?id=140298
http://trac.webkit.org/changeset/179494

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179839 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years ago[WebAudio] AudioBufferSourceNodes should accurately play backwards if given a negativ...
jer.noble@apple.com [Mon, 9 Feb 2015 19:45:54 +0000 (19:45 +0000)]
[WebAudio] AudioBufferSourceNodes should accurately play backwards if given a negative playbackRate.
https://bugs.webkit.org/show_bug.cgi?id=140955

Reviewed by Eric Carlson.

Source/WebCore:

Tests: webaudio/audiobuffersource-negative-playbackrate-interpolated.html
       webaudio/audiobuffersource-negative-playbackrate.html

Add support for playing an AudioBufferSourceNode at a negative playbackRate. Change the meaning of
start() to set the initial playback position at the end of the play range if the rate of playback
is negtive.

* Modules/webaudio/AudioBufferSourceNode.cpp:
(WebCore::AudioBufferSourceNode::AudioBufferSourceNode): Allow the playbackRate AudioParam to range from [-32, 32].
(WebCore::AudioBufferSourceNode::renderFromBuffer): Change variable names from "start" and "end" to "min" and "max"
    for clarity. Add a non-interpolated and interpolated render step for negative playback.
(WebCore::AudioBufferSourceNode::start): Drive-by fix: default value of grainDuration is not 0.02.
(WebCore::AudioBufferSourceNode::startPlaying): Start playing at the end of the buffer for negative playback.
(WebCore::AudioBufferSourceNode::totalPitchRate): Allow the pitch to be negative.

LayoutTests:

* webaudio/audiobuffersource-negative-playbackrate-expected.txt: Added.
* webaudio/audiobuffersource-negative-playbackrate-interpolated-expected.txt: Added.
* webaudio/audiobuffersource-negative-playbackrate-interpolated-loop-expected.txt: Added.
* webaudio/audiobuffersource-negative-playbackrate-interpolated-loop.html: Added.
* webaudio/audiobuffersource-negative-playbackrate-interpolated.html:
* webaudio/audiobuffersource-negative-playbackrate-loop-expected.txt: Added.
* webaudio/audiobuffersource-negative-playbackrate-loop.html: Added.
* webaudio/audiobuffersource-negative-playbackrate.html:
* webaudio/resources/audiobuffersource-testing.js:
(createRamp):

Get rid of extra HRTF padding as it's now unnecessary.

* webaudio/resources/note-grain-on-testing.js:
(createSignalBuffer):
(verifyStartAndEndFrames):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179838 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years ago[iOS] Gardening: css3/masking/mask-repeat-space-padding.html
ddkilzer@apple.com [Mon, 9 Feb 2015 18:59:44 +0000 (18:59 +0000)]
[iOS] Gardening: css3/masking/mask-repeat-space-padding.html

* platform/ios-simulator-wk2/TestExpectations: Remove
expectation since this is covered in
platform/ios-simulator/TestExpectations as an ImageOnlyFailure.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179837 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years ago[iOS] Gardening for editing/execCommand/remove-list-item-1.html
ddkilzer@apple.com [Mon, 9 Feb 2015 18:53:03 +0000 (18:53 +0000)]
[iOS] Gardening for editing/execCommand/remove-list-item-1.html

* platform/ios-simulator-wk1/TestExpectations:
(editing/execCommand/remove-list-item-1.html): Mark as flakey.
When run (with or without --run-singly), this test always passes
the first time, but fails all subsequent times due to
EDITING DELEGATE: shouldChangeSelectedDOMRange:range
running in a different order.
* platform/ios-simulator-wk2/editing/execCommand/remove-list-item-1-expected.txt: Update.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179836 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoREGRESSION (r179771): [iOS] Gardening for compositing/layer-creation/subpixel-adjacen...
ddkilzer@apple.com [Mon, 9 Feb 2015 18:22:00 +0000 (18:22 +0000)]
REGRESSION (r179771): [iOS] Gardening for compositing/layer-creation/subpixel-adjacent-layers-overlap.html

Test recently added for:
Convert the compositing overlap map to use LayoutRects
<http://webkit.org/b/141346>

* platform/ios-simulator/compositing/layer-creation/subpixel-adjacent-layers-overlap-expected.txt:
Add platform-specific results for ios-simulator.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179832 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoFix the !ENABLE(DFG_JIT) build
ossy@webkit.org [Mon, 9 Feb 2015 18:07:46 +0000 (18:07 +0000)]
Fix the !ENABLE(DFG_JIT) build
https://bugs.webkit.org/show_bug.cgi?id=141387

Reviewed by Darin Adler.

* jit/Repatch.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179831 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agorun-jsc-stress-tests shell test runner should run tests in fixed order
ossy@webkit.org [Mon, 9 Feb 2015 17:52:18 +0000 (17:52 +0000)]
run-jsc-stress-tests shell test runner should run tests in fixed order
https://bugs.webkit.org/show_bug.cgi?id=141383

Reviewed by Darin Adler.

* Scripts/jsc-stress-test-helpers/shell-runner.sh:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179830 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoTry to fix build on platforms that use SVG "all in one" file (Windows).
darin@apple.com [Mon, 9 Feb 2015 17:15:28 +0000 (17:15 +0000)]
Try to fix build on platforms that use SVG "all in one" file (Windows).

* svg/SVGAElement.cpp: Don't do "using namespace HTMLNames;" outside of
function boundaries, because that will be inherited by other files.
(WebCore::SVGAElement::isURLAttribute): Use XLinkNames directly here
instead of using HTMLNames implicitly.

* svg/SVGElement.cpp: Don't do "using namespace HTMLNames;" outside of
function boundaries, because that will be inherited by other files.
(WebCore::populateAttributeNameToCSSPropertyIDMap): Instead do it in here.
(WebCore::populateAttributeNameToAnimatedPropertyTypeMap): And here.
(WebCore::populateCSSPropertyWithSVGDOMNameToAnimatedPropertyTypeMap): And here.
(WebCore::SVGElement::parseAttribute): And use HTMLNames directly here
instead of implicitly.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179829 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoREGRESSION(r179705): 2nd-level inspector availability no longer controlled by Develop...
burg@cs.washington.edu [Mon, 9 Feb 2015 16:37:41 +0000 (16:37 +0000)]
REGRESSION(r179705): 2nd-level inspector availability no longer controlled by DeveloperExtrasEnabled user default
https://bugs.webkit.org/show_bug.cgi?id=141343

Reviewed by Timothy Hatcher.

The regression was caused by the switch to using WKWebViewConfiguration and
its default WebPreferences object, which is used to populate the inspector page's
Settings object. This WebPreferences is initialized with no identifier, so
only preferences in the FOR_EACH_WEBKIT_DEBUG_*_PREFERENCE macros are populated
from NSUserDefaults.

The simplest fix is to move DeveloperExtrasEnabled into the DEBUG group.

Previously, each inspector level had a unique identifier such as
__WebInspectorPageGroupLevelN__, and the n+1 level inspector was enabled
by toggling __WebInspectorPageGroupLevelN__.WebKit2DeveloperExrasEnabled.
With the move to the DEBUG group, the preference becomes simply
WebKitDeveloperExtrasEnabled, which enables any level of inspector.
(This does not clash with Safari's "Show Develop Menu" preference, which uses
the key "WebKitDeveloperExtrasEnabledPreferenceKey")

* Shared/WebPreferencesDefinitions.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179828 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years ago[iOS] exit from fullscreen when player view controller calls delegate
eric.carlson@apple.com [Mon, 9 Feb 2015 16:22:53 +0000 (16:22 +0000)]
[iOS] exit from fullscreen when player view controller calls delegate
https://bugs.webkit.org/show_bug.cgi?id=141350

Reviewed by Jer Noble.

* platform/ios/WebVideoFullscreenInterfaceAVKit.mm:
(-[WebAVPlayerController playerViewControllerWillCancelOptimizedFullscree:]): New, ask delegate
    to exit from fullscreen.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179827 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoASSERTION FAILED: resolvedInitialPosition <= resolvedFinalPosition in WebCore::GridSp...
svillar@igalia.com [Mon, 9 Feb 2015 14:05:49 +0000 (14:05 +0000)]
ASSERTION FAILED: resolvedInitialPosition <= resolvedFinalPosition in WebCore::GridSpan::GridSpan
https://bugs.webkit.org/show_bug.cgi?id=141328

Reviewed by Darin Adler.

.:

Added as manual test because it involves a huge grid allocation
which is very slow on Debug bots, the only ones capable to trigger
the assertion.

* ManualTests/css-grid-layout-item-with-huge-span-crash.html: Added.

Source/WebCore:

Whenever
GridResolvedPosition::resolveGridPositionsFromAutoPlacementPosition()
was trying to place an item with span, it was completely ignoring
the resolvedInitialPosition returned by
GridResolvedPosition::resolveGridPositionAgainstOppositePosition()
and only using the finalResolvedPosition. This works with an
unlimited grid which can indefinitely grow. But if the item spans
over the grid track limits, then it might happen that the final
resolved position is placed before the initial resolved position,
something that is forbidden.

The solution is to directly use the GridSpan returned by
GridResolvedPosition::resolveGridPositionAgainstOppositePosition(), if the item
does not surpass the track limits then the returned initialResolvedPosition
is identical to the provided one, otherwise it's properly corrected to respect
track boundaries.

* rendering/style/GridResolvedPosition.cpp:
(WebCore::GridResolvedPosition::resolveGridPositionsFromAutoPlacementPosition):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179826 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoUnreviewed GTK Gardening.
clopez@igalia.com [Mon, 9 Feb 2015 13:18:19 +0000 (13:18 +0000)]
Unreviewed GTK Gardening.

* platform/gtk/TestExpectations: Mark new test failing
* platform/gtk/css2.1/t1508-c527-font-00-b-expected.txt: Added. Rebaseline after r177774.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179825 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years ago[CSS Grid Layout] Tracks' growth limits must be >= base sizes
svillar@igalia.com [Mon, 9 Feb 2015 11:34:11 +0000 (11:34 +0000)]
[CSS Grid Layout] Tracks' growth limits must be >= base sizes
https://bugs.webkit.org/show_bug.cgi?id=140540

Reviewed by Antti Koivisto.

Source/WebCore:

The track sizing algorithm is supposed to avoid those situations
but they easily (specially when we mix absolute lengths and
intrinsic lengths in min and max track sizing functions) and
frequently appear. In those cases the outcome from the algorithm
is wrong, tracks are not correctly sized.

In order to fulfill the restriction, m_usedBreadth and
m_maxBreadth are now private members of GridTrack and the class
now provides a couple of methods to modify them respecting the
growthLimit >= baseSize precondition.

Apart from that, the members and methods of GridTrack were also
renamed to match the ones used in the recent algorithm rewrite:
usedBreadth became baseSize and maxBreadth is now growthLimit.

Although the algorithm was not modified at all, this change
detected and fixed several invalid results (tracks and/or grids
bigger than expected).

* rendering/RenderGrid.cpp:
(WebCore::GridTrack::GridTrack): Renamed fields and methods. Added
assertions.
(WebCore::GridTrack::baseSize): Renamed from usedBreadth.
(WebCore::GridTrack::growthLimit): Renamed from maxBreadth.
(WebCore::GridTrack::setBaseSize):
(WebCore::GridTrack::setGrowthLimit):
(WebCore::GridTrack::growBaseSize): Renamed from growUsedBreadth.
(WebCore::GridTrack::growGrowthLimit): Renamed from growMaxBreadth.
(WebCore::GridTrack::growthLimitIsInfinite): New helper method.
(WebCore::GridTrack::growthLimitIfNotInfinite): Renamed from
maxBreadthIfNotInfinite.
(WebCore::GridTrack::isGrowthLimitBiggerThanBaseSize): New helper
method to verify ASSERTs are true.
(WebCore::GridTrack::ensureGrowthLimitIsBiggerThanBaseSize): Ditto.
(WebCore::GridTrackForNormalization::GridTrackForNormalization):
(WebCore::RenderGrid::computeIntrinsicLogicalWidths):
(WebCore::RenderGrid::computeUsedBreadthOfGridTracks):
(WebCore::RenderGrid::computeNormalizedFractionBreadth):
(WebCore::RenderGrid::resolveContentBasedTrackSizingFunctions):
(WebCore::sortByGridTrackGrowthPotential):
(WebCore::RenderGrid::distributeSpaceToTracks):
(WebCore::RenderGrid::tracksAreWiderThanMinTrackBreadth):
(WebCore::RenderGrid::layoutGridItems):
(WebCore::RenderGrid::gridAreaBreadthForChild):
(WebCore::RenderGrid::populateGridPositions):
(WebCore::GridTrack::growUsedBreadth): Renamed to growBaseSize.
(WebCore::GridTrack::usedBreadth): Renamed to baseSize.
(WebCore::GridTrack::growMaxBreadth): Renamed to growGrowthLimit.
(WebCore::GridTrack::maxBreadthIfNotInfinite): Renamed to
growthLimitIfNotInfinite.
* rendering/RenderGrid.h:

LayoutTests:

* fast/css-grid-layout/grid-content-sized-columns-resolution-expected.txt:
* fast/css-grid-layout/grid-content-sized-columns-resolution.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179824 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoMeasure cache size more accurately
antti@apple.com [Mon, 9 Feb 2015 11:03:06 +0000 (11:03 +0000)]
Measure cache size more accurately
https://bugs.webkit.org/show_bug.cgi?id=141378
<rdar://problem/19760224>

Reviewed by Chris Dumez.

Estimate the cache disk space usage from the actual entry sizes instead of the item count.
This prevents large cache items from making the cache grow beyond its bounds.

* NetworkProcess/cache/NetworkCacheStorage.h:
* NetworkProcess/cache/NetworkCacheStorageCocoa.mm:
(WebKit::NetworkCacheStorage::initialize):
(WebKit::NetworkCacheStorage::removeEntry):
(WebKit::NetworkCacheStorage::store):
(WebKit::NetworkCacheStorage::clear):
(WebKit::NetworkCacheStorage::shrinkIfNeeded):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179823 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoUnreviewed, add new baseline since r179796.
gyuyoung.kim@samsung.com [Mon, 9 Feb 2015 08:18:20 +0000 (08:18 +0000)]
Unreviewed, add new baseline since r179796.

New baseline was added by r179796. EFL port supports the test as well.
This patch adds new baseline based on EFL port.

* platform/efl/fast/css/focus-ring-exists-for-search-field-expected.png: Added.
* platform/efl/fast/css/focus-ring-exists-for-search-field-expected.txt: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179822 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoREGRESSION(r179705): [GTK] The Web Inspector doesn't work after r179705
carlosgc@webkit.org [Mon, 9 Feb 2015 08:02:34 +0000 (08:02 +0000)]
REGRESSION(r179705): [GTK] The Web Inspector doesn't work after r179705
https://bugs.webkit.org/show_bug.cgi?id=141333

Reviewed by Žan Doberšek.

Create an initialize WebPreferences for the inspector page. This
was moved from cross-platform code to platform specific code.

* UIProcess/gtk/WebInspectorProxyGtk.cpp:
(WebKit::WebInspectorProxy::platformCreateInspectorPage):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179821 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoAX: VoiceOver appears unresponsive when JavaScript alerts are triggered via focus...
cfleizach@apple.com [Mon, 9 Feb 2015 07:09:50 +0000 (07:09 +0000)]
AX: VoiceOver appears unresponsive when JavaScript alerts are triggered via focus or blur events
https://bugs.webkit.org/show_bug.cgi?id=140485

Reviewed by Anders Carlsson.

Source/WebCore:

If setting an accessibility attribute results in a modal alert being displayed, it can cause VoiceOver
to hang. A simple solution is perform the actual work after a short delay, which will ensure the call
returns without hanging.

Test: platform/mac/accessibility/setting-attributes-is-asynchronous.html

* accessibility/mac/WebAccessibilityObjectWrapperMac.mm:
(-[WebAccessibilityObjectWrapper accessibilitySetValue:forAttribute:]):
(-[WebAccessibilityObjectWrapper _accessibilitySetValue:forAttribute:]):

Tools:

Implement takeFocus() as a way to set focus through accessibility wrappers.

* DumpRenderTree/mac/AccessibilityUIElementMac.mm:
(AccessibilityUIElement::takeFocus):
* WebKitTestRunner/InjectedBundle/mac/AccessibilityUIElementMac.mm:
(WTR::AccessibilityUIElement::takeFocus):

LayoutTests:

Modify tests that relied on setting behavior and immediately checking results. Those
tests now need to retrieve results after a short timeout.

* accessibility/textarea-selected-text-range-expected.txt:
* accessibility/textarea-selected-text-range.html:
* platform/mac/accessibility/select-element-selection-with-optgroups.html:
* platform/mac/accessibility/setting-attributes-is-asynchronous-expected.txt: Added.
* platform/mac/accessibility/setting-attributes-is-asynchronous.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179820 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoAdd parsing support for CSS Selector L4's case-insensitive attribute
benjamin@webkit.org [Mon, 9 Feb 2015 03:44:57 +0000 (03:44 +0000)]
Add parsing support for CSS Selector L4's case-insensitive attribute
https://bugs.webkit.org/show_bug.cgi?id=141373

Reviewed by Darin Adler.

Source/WebCore:

This patch adds parsing for the case-insensitive attribute value
matching of CSS Selectors Level 4: http://dev.w3.org/csswg/selectors-4/#attribute-case
Excuse of a grammar: http://dev.w3.org/csswg/selectors-4/#grammar

This patch also covers serialization for CSSOM. The serialization
is defined here: http://dev.w3.org/csswg/cssom/#serializing-selectors

Matching is completely ignored in this patch. All the simple selectors
are treated as regular attribute selectors.

Tests: fast/css/parsing-css-attribute-case-insensitive-value-1.html
       fast/css/parsing-css-attribute-case-insensitive-value-2.html
       fast/css/parsing-css-attribute-case-insensitive-value-3.html
       fast/css/parsing-css-attribute-case-insensitive-value-4.html

* css/CSSGrammar.y.in:
* css/CSSParserValues.h:
(WebCore::CSSParserSelector::setAttributeValueMatchingIsCaseInsensitive):
* css/CSSSelector.cpp:
(WebCore::CSSSelector::CSSSelector):
(WebCore::CSSSelector::selectorText):
* css/CSSSelector.h:
(WebCore::CSSSelector::CSSSelector):
(WebCore::CSSSelector::setAttributeValueMatchingIsCaseInsensitive):
(WebCore::CSSSelector::attributeValueMatchingIsCaseInsensitive):

LayoutTests:

* fast/css/css-selector-text-expected.txt:
* fast/css/css-selector-text.html:
* fast/css/css-set-selector-text-expected.txt:
* fast/css/css-set-selector-text.html:
Basic round-trip serialization through CSSOM.

* fast/css/parsing-css-attribute-case-insensitive-value-1-expected.txt: Added.
* fast/css/parsing-css-attribute-case-insensitive-value-1.html: Added.
Simple cases by themself and used in complex selectors.

* fast/css/parsing-css-attribute-case-insensitive-value-2-expected.txt: Added.
* fast/css/parsing-css-attribute-case-insensitive-value-2.html: Added.
Less simple cases, all kinds of valid syntax for case-insensitive attributes.

* fast/css/parsing-css-attribute-case-insensitive-value-3-expected.txt: Added.
* fast/css/parsing-css-attribute-case-insensitive-value-3.html: Added.
Cases that must be treated as invalid selectors.

* fast/css/parsing-css-attribute-case-insensitive-value-4-expected.txt: Added.
* fast/css/parsing-css-attribute-case-insensitive-value-4.html: Added.
Verify that invalid rules do not affect surrounding valid rules.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179819 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years ago[iOS] Skip js/dom/create-lots-of-workers.html on ios-simulator
ddkilzer@apple.com [Mon, 9 Feb 2015 02:56:05 +0000 (02:56 +0000)]
[iOS] Skip js/dom/create-lots-of-workers.html on ios-simulator

Already skipped on mac; marked as flakey on efl and gtk.

Fix tracked by:
REGRESSION: js/dom/create-lots-of-workers.html frequently crashes (sometimes in js/dom/cross-frame-bad-time.html)
<http://webkit.org/b/129758>
<rdar://problem/19760988>

* platform/ios-simulator/TestExpectations: Skip test.
* platform/mac/TestExpectations: Add comment about skipping on
ios-simulator.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179818 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoRemove a few duplicate propagation steps from the DFG's PredictionPropagation phase
benjamin@webkit.org [Mon, 9 Feb 2015 02:42:22 +0000 (02:42 +0000)]
Remove a few duplicate propagation steps from the DFG's PredictionPropagation phase
https://bugs.webkit.org/show_bug.cgi?id=141363

Reviewed by Darin Adler.

* dfg/DFGPredictionPropagationPhase.cpp:
(JSC::DFG::PredictionPropagationPhase::propagate):
Some blocks were duplicated, they probably evolved separately
to the same state.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179817 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoRemove useless declarations and a stale comment from DFGByteCodeParser.h
benjamin@webkit.org [Mon, 9 Feb 2015 02:40:09 +0000 (02:40 +0000)]
Remove useless declarations and a stale comment from DFGByteCodeParser.h
https://bugs.webkit.org/show_bug.cgi?id=141361

Reviewed by Darin Adler.

The comment refers to the original form of the ByteCodeParser:
    parse(Graph&, JSGlobalData*, CodeBlock*, unsigned startIndex);

That form is long dead, the comment is more misleading than anything.

* dfg/DFGByteCodeParser.cpp:
* dfg/DFGByteCodeParser.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179816 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoEncapsulate DFG::Plan's beforeFTL timestamp
benjamin@webkit.org [Mon, 9 Feb 2015 02:38:47 +0000 (02:38 +0000)]
Encapsulate DFG::Plan's beforeFTL timestamp
https://bugs.webkit.org/show_bug.cgi?id=141360

Reviewed by Darin Adler.

Make the attribute private, it is an internal state.

Rename beforeFTL->timeBeforeFTL for readability.

* dfg/DFGPlan.cpp:
(JSC::DFG::Plan::compileInThread):
(JSC::DFG::Plan::compileInThreadImpl):
* dfg/DFGPlan.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179815 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoRemove DFGNode::hasArithNodeFlags()
benjamin@webkit.org [Mon, 9 Feb 2015 02:37:03 +0000 (02:37 +0000)]
Remove DFGNode::hasArithNodeFlags()
https://bugs.webkit.org/show_bug.cgi?id=141319

Patch by Benjamin Poulain <bpoulain@apple.com> on 2015-02-08
Reviewed by Michael Saboff.

* dfg/DFGNode.h:
(JSC::DFG::Node::hasArithNodeFlags): Deleted.
Unused code is unused.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179814 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoFix CMake-based build.
darin@apple.com [Mon, 9 Feb 2015 01:50:50 +0000 (01:50 +0000)]
Fix CMake-based build.

* CMakeLists.txt: Added a dependency on the CMakeLists.txt itself, analogous
to the one I added in DerivedSources.make.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179813 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoFix build.
darin@apple.com [Mon, 9 Feb 2015 01:40:07 +0000 (01:40 +0000)]
Fix build.

* bindings/js/JSEventListener.h: Removed a call to forwardEventListeners.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179812 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agofullscreen/full-screen-plugin.html is very flaky on Yosemite WK2
ap@apple.com [Mon, 9 Feb 2015 01:25:17 +0000 (01:25 +0000)]
fullscreen/full-screen-plugin.html is very flaky on Yosemite WK2
https://bugs.webkit.org/show_bug.cgi?id=141364

Reviewed by Sam Weinig.

Make the test wait for the plug-in to become available.

* fullscreen/full-screen-plugin.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179811 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoRemove the SVG instance tree
darin@apple.com [Mon, 9 Feb 2015 01:13:35 +0000 (01:13 +0000)]
Remove the SVG instance tree
https://bugs.webkit.org/show_bug.cgi?id=140602

Reviewed by Dean Jackson.

Source/WebCore:

* CMakeLists.txt: Removed SVGElementInstance source files.
* DerivedSources.cpp: Ditto.
* DerivedSources.make: Ditto.
* WebCore.vcxproj/WebCore.vcxproj: Ditto.
* WebCore.vcxproj/WebCore.vcxproj.filters: Ditto.
* WebCore.xcodeproj/project.pbxproj: Ditto.
* bindings/js/JSBindingsAllInOne.cpp: Ditto.

* bindings/js/JSEventListener.cpp:
(WebCore::forwardsEventListeners): Deleted. Only returned true for JSSVGElementInstance.
(WebCore::correspondingElementWrapper): Deleted. Only used for JSSVGElementInstance.
(WebCore::createJSEventListenerForAttribute): Deleted. Argument type was JSSVGElementInstance.
(WebCore::createJSEventListenerForAdd): Removed most of the code; later we can delete this entirely.

* bindings/js/JSEventListener.h: Removed the overload of createJSEventListenerForAttribute
that takes a JSSVGElementInstance.

* bindings/js/JSSVGElementInstanceCustom.cpp: Removed.

* dom/ContainerNodeAlgorithms.h: Updated comment to reflect the fact that
this code is really now only used for ContainerNode and no longer needs to
exist in a generic form.

* dom/EventTarget.h: Removed forward declaration of SVGElementInstance.
* svg/SVGElement.h: Ditto.

* dom/EventTargetFactory.in: Removed SVGElementInstance.

* svg/SVGElementInstance.cpp: Removed.
* svg/SVGElementInstance.h: Removed.
* svg/SVGElementInstance.idl: Removed.

* svg/SVGUseElement.cpp:
(WebCore::SVGUseElement::insertedInto): Removed obsolete comment.
(WebCore::SVGUseElement::instanceTreeIsLoading): Deleted. Unused
function that I forgot to delete in my last patch. It also had a
glaring mistake, a missing "return" before the recursive call to
itself that would cause it to return false when it should return true.

* svg/SVGUseElement.h: Removed instanceTreeIsLoading.

* dom/EventDispatcher.cpp: Removed include of SVGElementInstance.h.
* page/EventHandler.cpp: Ditto.
* rendering/svg/RenderSVGViewportContainer.cpp: Ditto.
* svg/SVGAElement.cpp: Ditto.
* svg/SVGAllInOne.cpp: Ditto.
* svg/SVGAnimateMotionElement.cpp: Ditto.
* svg/SVGAnimatedTypeAnimator.h: Ditto.
* svg/SVGAnimationElement.cpp: Ditto.
* svg/SVGCircleElement.cpp: Ditto.
* svg/SVGClipPathElement.cpp: Ditto.
* svg/SVGComponentTransferFunctionElement.cpp: Ditto.
* svg/SVGCursorElement.cpp: Ditto.
* svg/SVGElement.cpp: Ditto.
* svg/SVGEllipseElement.cpp: Ditto.
* svg/SVGFEBlendElement.cpp: Ditto.
* svg/SVGFEColorMatrixElement.cpp: Ditto.
* svg/SVGFECompositeElement.cpp: Ditto.
* svg/SVGFEConvolveMatrixElement.cpp: Ditto.
* svg/SVGFEDiffuseLightingElement.cpp: Ditto.
* svg/SVGFEDisplacementMapElement.cpp: Ditto.
* svg/SVGFEDropShadowElement.cpp: Ditto.
* svg/SVGFEGaussianBlurElement.cpp: Ditto.
* svg/SVGFEImageElement.cpp: Ditto.
* svg/SVGFELightElement.cpp: Ditto.
* svg/SVGFEMergeNodeElement.cpp: Ditto.
* svg/SVGFEMorphologyElement.cpp: Ditto.
* svg/SVGFEOffsetElement.cpp: Ditto.
* svg/SVGFESpecularLightingElement.cpp: Ditto.
* svg/SVGFETileElement.cpp: Ditto.
* svg/SVGFETurbulenceElement.cpp: Ditto.
* svg/SVGFilterElement.cpp: Ditto.
* svg/SVGFilterPrimitiveStandardAttributes.cpp: Ditto.
* svg/SVGForeignObjectElement.cpp: Ditto.
* svg/SVGGElement.cpp: Ditto.
* svg/SVGGradientElement.cpp: Ditto.
* svg/SVGGraphicsElement.cpp: Ditto.
* svg/SVGImageElement.cpp: Ditto.
* svg/SVGLineElement.cpp: Ditto.
* svg/SVGLinearGradientElement.cpp: Ditto.
* svg/SVGMarkerElement.cpp: Ditto.
* svg/SVGMaskElement.cpp: Ditto.
* svg/SVGPathElement.cpp: Ditto.
* svg/SVGPatternElement.cpp: Ditto.
* svg/SVGPolyElement.cpp: Ditto.
* svg/SVGRadialGradientElement.cpp: Ditto.
* svg/SVGRectElement.cpp: Ditto.
* svg/SVGSVGElement.cpp: Ditto.
* svg/SVGScriptElement.cpp: Ditto.
* svg/SVGStopElement.cpp: Ditto.
* svg/SVGSymbolElement.cpp: Ditto.
* svg/SVGTRefElement.cpp: Ditto.
* svg/SVGTextContentElement.cpp: Ditto.
* svg/SVGTextElement.cpp: Ditto.
* svg/SVGTextPathElement.cpp: Ditto.
* svg/SVGTextPositioningElement.cpp: Ditto.

Tools:

* Scripts/check-for-global-initializers: Removed special case for
SVGElementInstance.o.

LayoutTests:

Last step: Remove SVGElementInstance class itself.

* js/dom/global-constructors-attributes-expected.txt: Removed SVGElementInstance.
* platform/efl/js/dom/global-constructors-attributes-expected.txt: Ditto.
* platform/gtk/js/dom/global-constructors-attributes-expected.txt: Ditto.
* platform/ios-sim-deprecated/fast/dom/Window/window-property-descriptors-expected.txt: Ditto.
* platform/ios-sim-deprecated/fast/js/global-constructors-expected.txt: Ditto.
* platform/ios-sim-deprecated/js/dom/global-constructors-attributes-expected.txt: Ditto.
* platform/mac-mavericks/js/dom/global-constructors-attributes-expected.txt: Ditto.
* platform/mac-mountainlion/js/dom/global-constructors-attributes-expected.txt: Ditto.
* platform/mac/js/dom/global-constructors-attributes-expected.txt: Ditto.
* platform/win/js/dom/global-constructors-attributes-expected.txt: Ditto.
* svg/custom/global-constructors-expected.txt: Ditto.
* svg/custom/script-tests/global-constructors.js: Ditto.
* svg/dom/svg2-inheritance-expected.txt: Ditto.
* svg/dom/svg2-inheritance.html: Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179810 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoNull deref in _clearImmediateActionState when closing a view with a DataDetectors...
timothy_horton@apple.com [Mon, 9 Feb 2015 00:30:46 +0000 (00:30 +0000)]
Null deref in _clearImmediateActionState when closing a view with a DataDetectors popover open
https://bugs.webkit.org/show_bug.cgi?id=141377
<rdar://problem/19711203>

Reviewed by Darin Adler.

* UIProcess/mac/WKImmediateActionController.mm:
(-[WKImmediateActionController _clearImmediateActionState]):
We can have already detached the page when DataDetectors calls us back
in interactionStoppedHandler. While we have kept a strong reference to the
page in the interactionStoppedHandler block, _page is nulled out.
It's OK to avoid doing this work, in any case, because closing a page
tears down the TextIndicator anyway.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179809 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoTweak inline playback controls to match system spec
dino@apple.com [Sun, 8 Feb 2015 23:40:01 +0000 (23:40 +0000)]
Tweak inline playback controls to match system spec
https://bugs.webkit.org/show_bug.cgi?id=141375
<rdar://problem/19760754>

Reviewed by Sam Weinig.

Rework the UI of the inline media controls on iOS, to
better match the system specification. I've batched a
few changes into one patch because many of them are
inter-dependent, and not very aggressive. Changes are:

- updated artwork for the buttons.
- separate artwork for normal and active states.
- background images are now explicitly sized and positioned
  in the middle of the element, allowing audio and video
  to use the same glyphs even though the elements are
  different sizes.
- use plus-darker blend mode on the button glyphs.
- rearranged some of the rules to group things in a
  logical order.
- time should front-pad a "0" character, if less than 10.
- no need for an "active" class on the Airplay button (although
  I won't be surprised if this changes back).

* Modules/mediacontrols/mediaControlsiOS.css:
(::-webkit-media-controls):
(video::-webkit-media-controls-wireless-playback-picker-button.active): Deleted.
(audio::-webkit-media-controls-wireless-playback-picker-button.active): Deleted.
(audio::-webkit-media-controls-play-button:active): Deleted.
(audio::-webkit-media-controls-play-button.paused): Deleted.
(video::-webkit-media-controls-timeline): Deleted.
* Modules/mediacontrols/mediaControlsiOS.js:
(ControllerIOS.prototype.updateWirelessPlaybackStatus): No need
for the "active" class.
(ControllerIOS.prototype.formatTime): Pad with a leading zero.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179808 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoMake SVGUseElement work without creating any SVGElementInstance objects
darin@apple.com [Sun, 8 Feb 2015 23:22:24 +0000 (23:22 +0000)]
Make SVGUseElement work without creating any SVGElementInstance objects
https://bugs.webkit.org/show_bug.cgi?id=141374

Reviewed by Sam Weinig.

Source/WebCore:

* dom/ElementIterator.h: Changed the * and -> operators to be const.
There is no need for the iterator itself to be modified just to dereference it.

* dom/TypedElementDescendantIterator.h: Added DoubleTypedElementDescendantIterator.
This allows callers to call descendantsOfType on two elements, as long as the caller
can guarantee that both have the same number of descendants of that type. It's handy
for walking a tree of cloned elements to set up something between each original and
its clone. In the future we might instead change the cloning machinery so it can do
this work as we clone, and if so, we could consider deleting this.

* svg/SVGElement.cpp:
(WebCore::SVGElement::correspondingElement): Made this const.
(WebCore::SVGElement::invalidateInstances): Got rid of the rule that said "this can
only be done for an element in a document", since it's useful to do this on an element
that has just been removed from a document. Removed the "updateStyleIfNeeded" call
here now that the other changes make it no longer needed. Removed an unimportant
assertion that we only invalidate use elements that are in a document; that's not
a necessary restriction. Streamlined the logic a bit.

* svg/SVGElement.h: Made correspondingElement const.

* svg/SVGUseElement.cpp:
(WebCore::SVGUseElement::insertedInto): Removed an assertion about
m_targetElementInstance since that's gone now.
(WebCore::SVGUseElement::svgAttributeChanged): Changed code that transfers
size attributes to the shadow tree to use shadowTreeTargetClone instead of
m_targetElementInstance.
(WebCore::SVGUseElement::clearResourceReferences): Removed code to detach
m_targetElementInstance, and also the call to removeAllTargetReferencesForElement,
because we no longer use those.
(WebCore::SVGUseElement::buildPendingResource): Moved the code to build the
shadow tree in here and deleted the buildShadowAndInstanceTree function.
Also changed logic so that we use a pending resource any time the target is not
a valid one. That helps us correctly handle cases where we initially have an
invalid target, but later get a value one
(WebCore::SVGUseElement::buildShadowAndInstanceTree): Deleted. The code here
was greatly simplified and moved into buildPendingResource.
(WebCore::SVGUseElement::buildInstanceTree): Deleted.
(WebCore::SVGUseElement::hasCycleUseReferencing): Deleted. Cycles are now
detected by the new isValidTarget function and so there's no need for a
separate explicit check for a cycle.
(WebCore::associateClonesWithOriginals): Added. Helper that makes
functions that build the shadow tree simpler and easier to read.
(WebCore::associateReplacementCloneWithOriginal): Added. Helper to
make associateReplacementClonesWithOriginals simple.
(WebCore::associateReplacementClonesWithOriginals): Added. Helper that
makes functions that build the shadow tree simpler and easier to read.
(WebCore::SVGUseElement::buildShadowTree): Call associateClonesWithOriginals
since associateInstancesWithShadowTreeElements no longer does this.
(WebCore::SVGUseElement::isValidTarget): Added. Covers all the different
reasons a target might not be valid: type of element, reference cycles, and
also "not in document" (refactored in here; not sure when that can happen
in practice, might be possible to remove it later).
(WebCore::SVGUseElement::expandUseElementsInShadowTree): Add checks for
documents that are still loading; this used to be checked when building the
instance tree. Added calls to associateReplacementClonesWithOriginals and
associateClonesWithOriginals; that used to be done by later in the
associateInstancesWithShadowTreeElements function. Use isValidTarget so
we handle cycles as well as invalid target types.
(WebCore::SVGUseElement::expandSymbolElementsInShadowTree): Added a call to
associateReplacementClonesWithOriginals, since we can no longer do that in
associateInstancesWithShadowTreeElements.
(WebCore::SVGUseElement::associateInstancesWithShadowTreeElements): Deleted.
(WebCore::SVGUseElement::instanceForShadowTreeElement): Deleted.
(WebCore::SVGUseElement::invalidateDependentShadowTrees): Removed a comment
that simply restated the name of the function.

* svg/SVGUseElement.h: Removed instanceForShadowTreeElement,
buildShadowAndInstanceTree, detachInstance, buildInstanceTree,
hasCycleUseReferencing, associateInstancesWithShadowTreeElements,
instanceForShadowTreeElement, and m_targetElementInstance. Added isValidTarget.

LayoutTests:

Results changed on some tests that expected the old "remove all content if a cycle is detected"
behavior from the <use> element. The new behavior is to inhibit cycles, but render everything
else, which is much easier to implement correctly and also makes logical sense. Changed all
those tests to be reference tests, which makes sense since they are focusing on what gets
rendered in these complex cases, and the expected results are a lot easier to understand in
SVG form than they were in txt/png form. This also means we can remove a lot of platform-specific
results since reference tests aren't sensitive to small platform differences in rendering.

* platform/efl/svg/W3C-SVG-1.2-Tiny/struct-use-recursion-01-t-expected.png: Removed.
* platform/efl/svg/W3C-SVG-1.2-Tiny/struct-use-recursion-02-t-expected.png: Removed.
* platform/efl/svg/W3C-SVG-1.2-Tiny/struct-use-recursion-03-t-expected.png: Removed.
* platform/efl/svg/hixie/error/017-expected.png: Removed.
* platform/gtk/svg/W3C-SVG-1.2-Tiny/struct-use-recursion-01-t-expected.png: Removed.
* platform/gtk/svg/W3C-SVG-1.2-Tiny/struct-use-recursion-02-t-expected.png: Removed.
* platform/gtk/svg/W3C-SVG-1.2-Tiny/struct-use-recursion-03-t-expected.png: Removed.
* platform/gtk/svg/custom/use-on-disallowed-foreign-object-3-expected.png: Removed.
* platform/gtk/svg/custom/use-on-disallowed-foreign-object-3-expected.txt: Removed.
* platform/gtk/svg/custom/use-recursion-1-expected.png: Removed.
* platform/gtk/svg/custom/use-recursion-1-expected.txt: Removed.
* platform/gtk/svg/custom/use-recursion-2-expected.png: Removed.
* platform/gtk/svg/custom/use-recursion-2-expected.txt: Removed.
* platform/gtk/svg/custom/use-recursion-3-expected.png: Removed.
* platform/gtk/svg/custom/use-recursion-3-expected.txt: Removed.
* platform/gtk/svg/custom/use-recursion-4-expected.png: Removed.
* platform/gtk/svg/custom/use-recursion-4-expected.txt: Removed.
* platform/gtk/svg/hixie/error/017-expected.png: Removed.
* platform/ios-sim-deprecated/svg/W3C-SVG-1.2-Tiny/struct-use-recursion-01-t-expected.txt: Removed.
* platform/ios-sim-deprecated/svg/W3C-SVG-1.2-Tiny/struct-use-recursion-02-t-expected.txt: Removed.
* platform/ios-sim-deprecated/svg/W3C-SVG-1.2-Tiny/struct-use-recursion-03-t-expected.txt: Removed.
* platform/ios-sim-deprecated/svg/custom/use-on-disallowed-foreign-object-3-expected.txt: Removed.
* platform/ios-sim-deprecated/svg/custom/use-recursion-1-expected.txt: Removed.
* platform/ios-sim-deprecated/svg/custom/use-recursion-2-expected.txt: Removed.
* platform/ios-sim-deprecated/svg/custom/use-recursion-3-expected.txt: Removed.
* platform/ios-sim-deprecated/svg/custom/use-recursion-4-expected.txt: Removed.
* platform/ios-sim-deprecated/svg/hixie/error/017-expected.txt: Removed.
* platform/ios-simulator/svg/custom/use-on-disallowed-foreign-object-3-expected.txt: Removed.
* platform/ios-simulator/svg/custom/use-recursion-1-expected.txt: Removed.
* platform/ios-simulator/svg/custom/use-recursion-2-expected.txt: Removed.
* platform/ios-simulator/svg/custom/use-recursion-3-expected.txt: Removed.
* platform/ios-simulator/svg/custom/use-recursion-4-expected.txt: Removed.
* platform/mac-mountainlion/svg/W3C-SVG-1.2-Tiny/struct-use-recursion-01-t-expected.txt: Removed.
* platform/mac-mountainlion/svg/W3C-SVG-1.2-Tiny/struct-use-recursion-02-t-expected.txt: Removed.
* platform/mac-mountainlion/svg/W3C-SVG-1.2-Tiny/struct-use-recursion-03-t-expected.txt: Removed.
* platform/mac-mountainlion/svg/custom/use-on-disallowed-foreign-object-3-expected.txt: Removed.
* platform/mac-mountainlion/svg/custom/use-recursion-1-expected.txt: Removed.
* platform/mac-mountainlion/svg/custom/use-recursion-2-expected.txt: Removed.
* platform/mac-mountainlion/svg/custom/use-recursion-3-expected.txt: Removed.
* platform/mac-mountainlion/svg/custom/use-recursion-4-expected.txt: Removed.
* platform/mac-mountainlion/svg/hixie/error/017-expected.txt: Removed.
* platform/mac/svg/W3C-SVG-1.2-Tiny/struct-use-recursion-01-t-expected.png: Removed.
* platform/mac/svg/W3C-SVG-1.2-Tiny/struct-use-recursion-01-t-expected.txt: Removed.
* platform/mac/svg/W3C-SVG-1.2-Tiny/struct-use-recursion-02-t-expected.png: Removed.
* platform/mac/svg/W3C-SVG-1.2-Tiny/struct-use-recursion-02-t-expected.txt: Removed.
* platform/mac/svg/W3C-SVG-1.2-Tiny/struct-use-recursion-03-t-expected.png: Removed.
* platform/mac/svg/W3C-SVG-1.2-Tiny/struct-use-recursion-03-t-expected.txt: Removed.
* platform/mac/svg/custom/use-on-disallowed-foreign-object-3-expected.png: Removed.
* platform/mac/svg/custom/use-on-disallowed-foreign-object-3-expected.txt: Removed.
* platform/mac/svg/custom/use-recursion-1-expected.png: Removed.
* platform/mac/svg/custom/use-recursion-1-expected.txt: Removed.
* platform/mac/svg/custom/use-recursion-2-expected.png: Removed.
* platform/mac/svg/custom/use-recursion-2-expected.txt: Removed.
* platform/mac/svg/custom/use-recursion-3-expected.png: Removed.
* platform/mac/svg/custom/use-recursion-3-expected.txt: Removed.
* platform/mac/svg/custom/use-recursion-4-expected.png: Removed.
* platform/mac/svg/custom/use-recursion-4-expected.txt: Removed.
* platform/mac/svg/hixie/error/017-expected.png: Removed.
* platform/mac/svg/hixie/error/017-expected.txt: Removed.

* svg/W3C-SVG-1.2-Tiny/struct-use-recursion-01-t-expected.svg: Added. Made this be a reference test,
and made it expect more of the recursion to work.
* svg/W3C-SVG-1.2-Tiny/struct-use-recursion-01-t-expected.txt: Removed.
* svg/W3C-SVG-1.2-Tiny/struct-use-recursion-02-t-expected.svg: Added. More of the same.
* svg/W3C-SVG-1.2-Tiny/struct-use-recursion-02-t-expected.txt: Removed.
* svg/W3C-SVG-1.2-Tiny/struct-use-recursion-03-t-expected.svg: Added. More of the same.
* svg/W3C-SVG-1.2-Tiny/struct-use-recursion-03-t-expected.txt: Removed.
* svg/custom/use-on-disallowed-foreign-object-3-expected.svg: Added. More of the same.
* svg/custom/use-recursion-1-expected.svg: Added. More of the same.
* svg/custom/use-recursion-2-expected.svg: Added. More of the same.
* svg/custom/use-recursion-3-expected.svg: Added. More of the same.
* svg/custom/use-recursion-4-expected.svg: Added. More of the same.
* svg/hixie/error/017-expected.txt: Removed.
* svg/hixie/error/017-expected.xml: Added. More of the same.

* svg/in-html/defs-after-use.html: Updated incorrect bug number in this test.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179807 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoSkip fast/parser/document-open-in-unload.html on all WK2 platforms
ddkilzer@apple.com [Sun, 8 Feb 2015 23:21:02 +0000 (23:21 +0000)]
Skip fast/parser/document-open-in-unload.html on all WK2 platforms

Tracked by:
[WK2] fast/parser/document-open-in-unload.html makes the following test crash
<http://webkit.org/b/98345>

* platform/mac-wk2/TestExpectations: Move Skip expectation from here...
* platform/wk2/TestExpectations: ...to here with updated bug number.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179806 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoUpdate section headers for mac-wk2/TestExpectations
ddkilzer@apple.com [Sun, 8 Feb 2015 23:20:57 +0000 (23:20 +0000)]
Update section headers for mac-wk2/TestExpectations

* platform/mac-wk2/TestExpectations:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179805 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years ago[WK2] Add logging to validate the network cache efficacy (Part 1)
cdumez@apple.com [Sun, 8 Feb 2015 22:06:06 +0000 (22:06 +0000)]
[WK2] Add logging to validate the network cache efficacy (Part 1)
https://bugs.webkit.org/show_bug.cgi?id=141269
<rdar://problem/19632080>

Reviewed by Antti Koivisto.

Source/WebCore:

Export an extra symbol.

* WebCore.exp.in:

Source/WebKit2:

Add console logging to validate the network cache efficacy. This will
tell us if how the network cache satisties requests, in particular:
- Request cannot be handled by the cache
- Entry was not in the cache but is no longer there (pruned)
- Entry is in the cache but is not usable
- Entry is in the cache and is used.

This patch introduces a SQLite-based network cache statistics storage
that is used to store requests we have seen before, and query if we
have seen a request before. The storage is lightweight as it only
stores hashes in the database, in a background thread.

The statistics cache is initially bootstapped from the network disk
cache so that we have data initially and get as accurate statistics
as possible from the start.

To maintain an acceptable level of performance, we have a hard limit
on the number of unique requests that are retained set to 100000.

Diagnostic logging for this will be added in a follow-up patch.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179804 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years ago[iOS] Gardening: Some MathML tests crash in RenderMathMLOperator::advanceForGlyph...
ddkilzer@apple.com [Sun, 8 Feb 2015 20:22:32 +0000 (20:22 +0000)]
[iOS] Gardening: Some MathML tests crash in RenderMathMLOperator::advanceForGlyph() or boundsForGlyph()

Tracked by: <http://webkit.org/b/141371>

* platform/ios-simulator-wk2/TestExpectations: Mark tests as
crashing.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179803 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoREGRESSION (r179391): Remove references to deleted SVG tests
ddkilzer@apple.com [Sun, 8 Feb 2015 20:22:29 +0000 (20:22 +0000)]
REGRESSION (r179391): Remove references to deleted SVG tests

Fixes the following lint warnings:

    --lint-test-files warnings:
    LayoutTests/platform/ios-simulator-wk2/TestExpectations:412 Path does not exist. svg/custom/use-elementInstance-event-target.svg
    LayoutTests/platform/ios-simulator-wk2/TestExpectations:413 Path does not exist. svg/custom/use-elementInstance-methods.svg
    LayoutTests/platform/ios-simulator-wk2/TestExpectations:417 Path does not exist. svg/custom/use-instanceRoot-event-listeners.xhtml

* platform/ios-simulator-wk2/TestExpectations: Remove deleted
tests.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179802 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoRemoving an accidentally committed ChangeLog.
ap@apple.com [Sun, 8 Feb 2015 20:00:56 +0000 (20:00 +0000)]
Removing an accidentally committed ChangeLog.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179801 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoAX: The input element with type="search" has no default focus outline
ap@apple.com [Sun, 8 Feb 2015 19:33:59 +0000 (19:33 +0000)]
AX: The input element with type="search" has no default focus outline
https://bugs.webkit.org/show_bug.cgi?id=140326

Adding results for Mavericks.

* platform/mac-mavericks/fast/css/focus-ring-exists-for-search-field-expected.txt: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179800 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agohttp/tests/security/appcache-in-private-browsing.html is flaky
ap@apple.com [Sun, 8 Feb 2015 19:17:57 +0000 (19:17 +0000)]
http/tests/security/appcache-in-private-browsing.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=141370

* TestExpectations: Marking as such.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179799 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoApplication cache abort() tests are flaky
ap@apple.com [Sun, 8 Feb 2015 19:11:41 +0000 (19:11 +0000)]
Application cache abort() tests are flaky
https://bugs.webkit.org/show_bug.cgi?id=87633

Moved expectations form Efl to root TestExpectations file. Also, removed Crash
expectation, as no bot hits that now.

Removed an erroneously added expectation for abort-cache-onchecking-resource-404.html -
this test doesn't happen to fail on the bots, although it also doesn't appear to
be very robust.

* TestExpectations:
* platform/efl/TestExpectations:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179798 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agohttp/tests/appcache/abort-cache-onchecking-resource-404.html is flaky
ap@apple.com [Sun, 8 Feb 2015 18:47:01 +0000 (18:47 +0000)]
http/tests/appcache/abort-cache-onchecking-resource-404.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=141368

* TestExpectations: This test is intrinsically unreliable, but at least it checks
that there is no crash is any of the code paths that it takes.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179797 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoAX: The input element with type="search" has no default focus outline
cfleizach@apple.com [Sun, 8 Feb 2015 08:02:48 +0000 (08:02 +0000)]
AX: The input element with type="search" has no default focus outline
https://bugs.webkit.org/show_bug.cgi?id=140326

Reviewed by Darin Adler.

Source/WebCore:

The platform RenderTheme takes care of the search field, and that code
was missing a check for whether the element was focused.

Test: fast/css/focus-ring-exists-for-search-field.html

* rendering/RenderThemeMac.mm:
(WebCore::RenderThemeMac::paintSearchField):

LayoutTests:

* fast/css/focus-ring-exists-for-search-field.html: Added.
* platform/mac/fast/css/focus-ring-exists-for-search-field-expected.png: Added.
* platform/mac/fast/css/focus-ring-exists-for-search-field-expected.txt: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179796 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years ago[ARM] GC sometimes stuck in an infinite loop if parallel GC is enabled
ossy@webkit.org [Sun, 8 Feb 2015 07:54:20 +0000 (07:54 +0000)]
[ARM] GC sometimes stuck in an infinite loop if parallel GC is enabled
https://bugs.webkit.org/show_bug.cgi?id=141290

Reviewed by Anders Carlsson.

* wtf/Platform.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179795 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agofullscreen/full-screen-plugin.html is very flaky on Yosemite WK2
ap@apple.com [Sun, 8 Feb 2015 05:24:25 +0000 (05:24 +0000)]
fullscreen/full-screen-plugin.html is very flaky on Yosemite WK2
https://bugs.webkit.org/show_bug.cgi?id=141364

Reviewed by Zalan Bujtas.

Speculative fix. Make sure that the plug-in has loaded before using it.

* fullscreen/full-screen-plugin.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179794 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years ago[iOS] run-webkit-tests fails due to simulator devices from previous SDK installs...
ddkilzer@apple.com [Sun, 8 Feb 2015 03:26:51 +0000 (03:26 +0000)]
[iOS] run-webkit-tests fails due to simulator devices from previous SDK installs being marked as unavailable
<http://webkit.org/b/141365>

Reviewed by Daniel Bates.

* Scripts/webkitpy/xcode/simulator.py:
(Simulator): Add unavailable_version_re precompiled regex.
(Simulator._parse_devices): Check for unavailable versions and
ignore them if found when parsing the output of
`xcrun simctl list`.
* Scripts/webkitpy/xcode/simulator_unittest.py:
(test_unavailable_devices): Add test with output from
`xcrun simctl list` with unavailable runtimes that fails before
the fix.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179793 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoAdd some dictionary lookup tests
timothy_horton@apple.com [Sun, 8 Feb 2015 02:43:40 +0000 (02:43 +0000)]
Add some dictionary lookup tests
https://bugs.webkit.org/show_bug.cgi?id=141355

Reviewed by Darin Adler.

Tests: platform/mac/editing/dictionary-lookup/dictionary-lookup-input.html
       platform/mac/editing/dictionary-lookup/dictionary-lookup-inside-selection.html
       platform/mac/editing/dictionary-lookup/dictionary-lookup-outside-selection.html
       platform/mac/editing/dictionary-lookup/dictionary-lookup-rtl.html
       platform/mac/editing/dictionary-lookup/dictionary-lookup.html

* WebCore.exp.in:
Remove an unneeded export.

* editing/mac/DictionaryLookup.h:
Use OBJC_CLASS instead of @class so that this can be included in pure-C++ files.

* testing/Internals.cpp:
(WebCore::Internals::rangeForDictionaryLookupAtLocation):
* testing/Internals.h:
* testing/Internals.idl:
Expose rangeForDictionaryLookupAtHitTestResult fairly directly to JavaScript.

* platform/mac/editing/dictionary-lookup/dictionary-lookup-expected.txt: Added.
* platform/mac/editing/dictionary-lookup/dictionary-lookup-input-expected.txt: Added.
* platform/mac/editing/dictionary-lookup/dictionary-lookup-input.html: Added.
* platform/mac/editing/dictionary-lookup/dictionary-lookup-inside-selection-expected.txt: Added.
* platform/mac/editing/dictionary-lookup/dictionary-lookup-inside-selection.html: Added.
* platform/mac/editing/dictionary-lookup/dictionary-lookup-outside-selection-expected.txt: Added.
* platform/mac/editing/dictionary-lookup/dictionary-lookup-outside-selection.html: Added.
* platform/mac/editing/dictionary-lookup/dictionary-lookup-rtl-expected.txt: Added.
* platform/mac/editing/dictionary-lookup/dictionary-lookup-rtl.html: Added.
* platform/mac/editing/dictionary-lookup/dictionary-lookup.html: Added.
* platform/mac/editing/dictionary-lookup/lookup-test.js: Added.
(runTest):
Add tests for various cases that we've had trouble with in the past.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179792 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoAdd Vector::removeFirstMatching() / removeAllMatching() methods taking lambda functions
cdumez@apple.com [Sun, 8 Feb 2015 02:28:48 +0000 (02:28 +0000)]
Add Vector::removeFirstMatching() / removeAllMatching() methods taking lambda functions
https://bugs.webkit.org/show_bug.cgi?id=141321

Reviewed by Darin Adler.

Source/JavaScriptCore:

Use new Vector::removeFirstMatching() / removeAllMatching() methods.

Source/WebCore:

Use new Vector::removeFirstMatching() / removeAllMatching() methods.

Source/WebKit/win:

Use new Vector::removeFirstMatching() / removeAllMatching() methods.

Source/WebKit2:

Use new Vector::removeFirstMatching() / removeAllMatching() methods.

Source/WTF:

Add Vector::removeFirstMatching() / removeAllMatching() methods taking
lambda functions to match the element(s) to remove. This simplifies the
code a bit. Vector::removeAllMatching() is also more efficient than the
manual removal alternative.

* wtf/Vector.h:

Tools:

Use new Vector::removeFirstMatching() / removeAllMatching() methods.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179791 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoUnreviewed gardening.
zalan@apple.com [Sun, 8 Feb 2015 01:31:58 +0000 (01:31 +0000)]
Unreviewed gardening.
Remove svg/custom/use-events-crash.svg. It has no value anymore.
See webkit.org/b/141108

* platform/gtk/svg/custom/use-events-crash-expected.png: Removed.
* platform/gtk/svg/custom/use-events-crash-expected.txt: Removed.
* platform/ios-sim-deprecated/svg/custom/use-events-crash-expected.txt: Removed.
* platform/ios-simulator/svg/custom/use-events-crash-expected.txt: Removed.
* platform/mac-mountainlion/svg/custom/use-events-crash-expected.txt: Removed.
* platform/mac/svg/custom/use-events-crash-expected.png: Removed.
* platform/mac/svg/custom/use-events-crash-expected.txt: Removed.
* svg/custom/use-events-crash.svg: Removed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179790 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agohttp/tests/xmlhttprequest/event-listener-gc.html is flaky
ap@apple.com [Sun, 8 Feb 2015 00:53:49 +0000 (00:53 +0000)]
http/tests/xmlhttprequest/event-listener-gc.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=33342

Reviewed by Darin Adler.

This test relies on a zero-delay timer being a lot faster than fetching from network.
Force a layout before starting the test, because otherwise, the layout can significantly
delay the timer (I've seen 50-60 milliseconds being a common delay in debug builds).

Also, changed the resource URL to avoid Apache error log spew.

* http/tests/xmlhttprequest/event-listener-gc.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179789 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years ago[iOS] Make Simulator class testable
ddkilzer@apple.com [Sun, 8 Feb 2015 00:44:06 +0000 (00:44 +0000)]
[iOS] Make Simulator class testable
<http://webkit.org/b/141358>

Rubber-stamped by Darin Adler.

* Scripts/webkitpy/common/system/platforminfo.py:
(PlatformInfo.xcode_simctl_list): Move `xcrun simctl list`
command to here from Simulator.refresh() in xcode/simulator.py
so that the output of the command can be mocked.

* Scripts/webkitpy/common/system/platforminfo_mock.py:
(MockPlatformInfo.__init__): Set self.expected_xcode_simctl_list
to None.
(MockPlatformInfo.xcode_simctl_list): Add method that returns
self.expected_xcode_simctl_list expectation.

* Scripts/webkitpy/xcode/simulator.py: Add missing copyright
and license header.
(Simulator.__init__): Add optional 'host' parameter to make it
possible to pass in a mock object for testing.  Set self._host
to 'host' parameter or create Host() object.
(Simulator.refresh): Call new PlatformInfo.xcode_simctl_list()
method.

* Scripts/webkitpy/xcode/simulator_unittest.py: Add unit test
for current code.
(SimulatorTest):
(SimulatorTest.setUp):
(SimulatorTest._set_expected_xcrun_simctl_list):
(SimulatorTest.test_simulator_device_types):
(test_invalid_device_types_header):
(test_invalid_runtimes_header):
(test_invalid_devices_header):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179788 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agofast/images/animated-gif-iframe-webkit-transform.html is flaky
cdumez@apple.com [Sun, 8 Feb 2015 00:32:54 +0000 (00:32 +0000)]
fast/images/animated-gif-iframe-webkit-transform.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=141323

Reviewed by Alexey Proskuryakov.

Use shouldBecomeEqual() instead of shouldBe() to check for initial test
conditions. This should address the flakiness.

* fast/images/animated-gif-iframe-webkit-transform-expected.txt:
* fast/images/animated-gif-iframe-webkit-transform.html:
* platform/mac/TestExpectations:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179787 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoAdd API::HistoryClient and split some things out of API::NavigationClient
timothy_horton@apple.com [Sat, 7 Feb 2015 23:46:18 +0000 (23:46 +0000)]
Add API::HistoryClient and split some things out of API::NavigationClient
https://bugs.webkit.org/show_bug.cgi?id=141264

Reviewed by Darin Adler.

* UIProcess/API/APIHistoryClient.h: Added.
(API::HistoryClient::~HistoryClient):
(API::HistoryClient::didNavigateWithNavigationData):
(API::HistoryClient::didPerformClientRedirect):
(API::HistoryClient::didPerformServerRedirect):
(API::HistoryClient::didUpdateHistoryTitle):
* UIProcess/API/APINavigationClient.h:
(API::NavigationClient::didNavigateWithNavigationData): Deleted.
(API::NavigationClient::didPerformClientRedirect): Deleted.
(API::NavigationClient::didPerformServerRedirect): Deleted.
(API::NavigationClient::didUpdateHistoryTitle): Deleted.
* UIProcess/Cocoa/NavigationState.h:
* UIProcess/Cocoa/NavigationState.mm:
(WebKit::NavigationState::createHistoryClient):
(WebKit::NavigationState::HistoryClient::HistoryClient):
(WebKit::NavigationState::HistoryClient::~HistoryClient):
(WebKit::NavigationState::HistoryClient::didNavigateWithNavigationData):
(WebKit::NavigationState::HistoryClient::didPerformClientRedirect):
(WebKit::NavigationState::HistoryClient::didPerformServerRedirect):
(WebKit::NavigationState::HistoryClient::didUpdateHistoryTitle):
(WebKit::NavigationState::NavigationClient::didNavigateWithNavigationData): Deleted.
(WebKit::NavigationState::NavigationClient::didPerformClientRedirect): Deleted.
(WebKit::NavigationState::NavigationClient::didPerformServerRedirect): Deleted.
(WebKit::NavigationState::NavigationClient::didUpdateHistoryTitle): Deleted.
* UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::setHistoryClient):
(WebKit::WebPageProxy::didNavigateWithNavigationData):
(WebKit::WebPageProxy::didPerformClientRedirect):
(WebKit::WebPageProxy::didPerformServerRedirect):
(WebKit::WebPageProxy::didUpdateHistoryTitle):
* UIProcess/WebPageProxy.h:
Add a API::HistoryClient and move the few things that belong on it out of API::NavigationClient.
Adjust accordingly in WebPageProxy and NavigationState.

* UIProcess/API/Cocoa/WKWebView.mm:
(-[WKWebView initWithFrame:configuration:]):
(-[WKWebView setNavigationDelegate:]):
(-[WKWebView setUIDelegate:]):
(-[WKWebView _setHistoryDelegate:]):
Lazily push the NavigationState/UIDelegate clients down to WebPageProxy upon
installation of a delegate, so that alternative (C SPI) delegate setters can
be separately created.

* WebKit2.xcodeproj/project.pbxproj:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179786 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoSource/WebCore:
darin@apple.com [Sat, 7 Feb 2015 23:17:26 +0000 (23:17 +0000)]
Source/WebCore:
Stop dispatching events to with SVGElementInstance objects as their targets
https://bugs.webkit.org/show_bug.cgi?id=141108

Reviewed by Anders Carlsson.

Test: svg/custom/use-event-retargeting.html

* dom/EventDispatcher.cpp:
(WebCore::eventTargetRespectingTargetRules): Replaced the code that retargeted
events at SVGElementInstance objects with code that retargets them at the use
element instead. Also wrote the code in a simpler way.

LayoutTests:
Stop dispatching events with SVGElementInstance objects as their targets
https://bugs.webkit.org/show_bug.cgi?id=141108

Reviewed by Anders Carlsson.

Many tests are no longer relevant once we aren't doing this any more.

* platform/gtk/svg/custom/use-instanceRoot-event-bubbling-expected.png: Removed.
* platform/gtk/svg/custom/use-instanceRoot-modifications-expected.png: Removed.
* platform/gtk/svg/custom/use-instanceRoot-modifications-expected.txt: Removed.
* platform/ios-sim-deprecated/svg/custom/use-instanceRoot-modifications-expected.txt: Removed.
* platform/ios-simulator/svg/custom/use-instanceRoot-modifications-expected.txt: Removed.
* platform/mac-mountainlion/svg/custom/use-instanceRoot-modifications-expected.txt: Removed.
* platform/mac/svg/custom/use-instanceRoot-event-bubbling-expected.png: Removed.
* platform/mac/svg/custom/use-instanceRoot-modifications-expected.png: Removed.
* platform/mac/svg/custom/use-instanceRoot-modifications-expected.txt: Removed.
* svg/custom/use-instanceRoot-modifications.svg: Removed.
* svg/custom/use-instanceRoot-with-use-removed-expected.txt: Removed.
* svg/custom/use-instanceRoot-with-use-removed.svg: Removed.

* svg/custom/resources/use-instanceRoot-event-bubbling.js: Updated this test to expect
the events to be dispatched with the SVGUseElement as the target. I talked this over with
Sam Weinig and we decided this is good behavior for now, and it almost matches what the
spec says. Might be worth refining later.
* svg/custom/use-instanceRoot-event-bubbling-expected.txt: Updated expected results.
* svg/custom/use-instanceRoot-event-bubbling.xhtml: Tweaked the test a little. It still
could use improvement; it's like half a "repaint test", which is strange.

* svg/custom/use-event-retargeting-expected.txt: Added. Got this test from Blink.
* svg/custom/use-event-retargeting.html: Added. Ditto.

* svg/custom/use-events-crash.svg: Added some more events, a second click, so that we
don't hang with the context menu up when running this. Also converted line endings to
use LF instead of CRLF.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179785 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years ago[Mac] Set -contentsScale on AVPlayerLayer to allow AVPlayer to select the appropriate...
jer.noble@apple.com [Sat, 7 Feb 2015 22:48:36 +0000 (22:48 +0000)]
[Mac] Set -contentsScale on AVPlayerLayer to allow AVPlayer to select the appropriate HLS variant.
https://bugs.webkit.org/show_bug.cgi?id=141354
rdar://problem/19717591

Reviewed by Darin Adler.

AVPlayer will try to determine the correct HLS variant based on the bounds of an AVPlayerLayer.
When not in a layer tree, AVFoundation is not able to determine the correct mapping from logical
units to pixel values. To provide AVPlayer with that scaling value, set -contentsScale based on
both the current device scale and the current page scale.

Since this needs to be set at initialization time, before the AVPlayer is has any AVPlayerItems,
add some plumbing up from MediaPlayer to as the HTMLMediaElement for the appropriate contents
scale.

* html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::mediaPlayerContentsScale):
* html/HTMLMediaElement.h:
* platform/graphics/MediaPlayer.h:
(WebCore::MediaPlayerClient::mediaPlayerContentsScale):
* platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:
(WebCore::MediaPlayerPrivateAVFoundationObjC::createAVPlayerLayer):
* platform/graphics/ca/GraphicsLayerCA.cpp:
(WebCore::GraphicsLayerCA::updateContentsScale):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179784 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoASan complains about plugins/snapshotting/snapshot-plugin-not-quite-blocked-by-image...
ap@apple.com [Sat, 7 Feb 2015 20:21:24 +0000 (20:21 +0000)]
ASan complains about plugins/snapshotting/snapshot-plugin-not-quite-blocked-by-image.html
https://bugs.webkit.org/show_bug.cgi?id=141352
rdar://problem/19717490

Reviewed by Anders Carlsson.

* dom/Document.cpp: (WebCore::Document::ensurePlugInsInjectedScript): This string
is not null terminated.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179783 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoAnd as a further followup restore the 8bit test too.
antti@apple.com [Sat, 7 Feb 2015 19:50:42 +0000 (19:50 +0000)]
And as a further followup restore the 8bit test too.

* NetworkProcess/cache/NetworkCacheKey.cpp:
(WebKit::hashString):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179782 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoUse longer hashes for cache keys
antti@apple.com [Sat, 7 Feb 2015 19:45:15 +0000 (19:45 +0000)]
Use longer hashes for cache keys
https://bugs.webkit.org/show_bug.cgi?id=141356

Rubber-stamped by Darin Adler.

Folloup and build fix.

* NetworkProcess/cache/NetworkCacheCoders.h:
* NetworkProcess/cache/NetworkCacheKey.cpp:
(WebKit::hashString):

    Use containsOnlyASCII instead of is8Bit so both paths always compute the same hash.

* NetworkProcess/cache/NetworkCacheKey.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179781 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoRemove a printf.
antti@apple.com [Sat, 7 Feb 2015 19:19:43 +0000 (19:19 +0000)]
Remove a printf.

* NetworkProcess/cache/NetworkCache.cpp:
(WebKit::NetworkCache::retrieve):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179780 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoUse longer hashes for cache keys
antti@apple.com [Sat, 7 Feb 2015 19:18:30 +0000 (19:18 +0000)]
Use longer hashes for cache keys
https://bugs.webkit.org/show_bug.cgi?id=141356

Reviewed by Darin Adler.

The current key hashes are 32bit. We should use longer hashes to eliminate collisions.

This patch switches us to using MD5 digests for the cache key hashes. As a result the file names for the cache
entries grow from 8 to 32 character.

Note that we don't need a cryptographic hash (full cache keys are verified against the entries).
MD5 just happens to be fast, convenient and available.

The patch also moves the whole cache hierarchy down to a versioned subdirectory ("WebKitCache/Version 2")
and deletes any old style cache files if they exist.

* NetworkProcess/cache/NetworkCacheCoders.cpp:
(WebKit::NetworkCacheCoder<MD5::Digest>::encode):
(WebKit::NetworkCacheCoder<MD5::Digest>::decode):
* NetworkProcess/cache/NetworkCacheCoders.h:
* NetworkProcess/cache/NetworkCacheKey.cpp:
(WebKit::NetworkCacheKey::NetworkCacheKey):
(WebKit::hashString):
(WebKit::NetworkCacheKey::computeHash):
(WebKit::NetworkCacheKey::hashAsString):
(WebKit::NetworkCacheKey::stringToHash):
* NetworkProcess/cache/NetworkCacheKey.h:
(WebKit::NetworkCacheKey::shortHash):
(WebKit::NetworkCacheKey::toShortHash):

    32bit hash to use in the bloom filter.

(WebKit::NetworkCacheKey::hashStringLength):
* NetworkProcess/cache/NetworkCacheStorage.h:

    Bump the version.

* NetworkProcess/cache/NetworkCacheStorageCocoa.mm:
(WebKit::traverseCacheFiles):
(WebKit::makeVersionedDirectoryPath):
(WebKit::NetworkCacheStorage::NetworkCacheStorage):
(WebKit::NetworkCacheStorage::initialize):
(WebKit::NetworkCacheStorage::removeEntry):
(WebKit::NetworkCacheStorage::retrieve):
(WebKit::NetworkCacheStorage::store):
(WebKit::NetworkCacheStorage::update):
(WebKit::NetworkCacheStorage::shrinkIfNeeded):
(WebKit::NetworkCacheStorage::deleteOldVersions):

    Wipe out the version 1 cache.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179779 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agorun-jsc-stress-tests --remote should use the default ssh port
ossy@webkit.org [Sat, 7 Feb 2015 17:01:37 +0000 (17:01 +0000)]
run-jsc-stress-tests --remote should use the default ssh port
https://bugs.webkit.org/show_bug.cgi?id=141287

Reviewed by Darin Adler.

* Scripts/run-jsc-stress-tests: Extended URI module to be able to handle ssh scheme with the default 22 port number.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179778 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoREGRESSION (OS X 10.10.2): http/tests/media/video-query-url.html frequently times out
ap@apple.com [Sat, 7 Feb 2015 08:32:54 +0000 (08:32 +0000)]
REGRESSION (OS X 10.10.2): http/tests/media/video-query-url.html frequently times out
https://bugs.webkit.org/show_bug.cgi?id=141085

* platform/mac-wk2/TestExpectations: Added an expectation.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179777 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoASSERT repaintContainer->hasLayer() in WebCore::RenderObject::repaintUsingContainer
zalan@apple.com [Sat, 7 Feb 2015 04:35:58 +0000 (04:35 +0000)]
ASSERT repaintContainer->hasLayer() in WebCore::RenderObject::repaintUsingContainer
https://bugs.webkit.org/show_bug.cgi?id=140750

Reviewed by Simon Fraser.

There's a short period of time when RenderObject::layer() still returns a valid pointer
even though we already cleared the hasLayer() flag.
Do not use the layer as repaint container in such cases.

Source/WebCore:

Test: compositing/repaint-container-assertion-when-toggling-compositing.html

* rendering/RenderObject.cpp:
(WebCore::RenderObject::enclosingLayer):

LayoutTests:

* compositing/repaint-container-assertion-when-toggling-compositing-expected.txt: Added.
* compositing/repaint-container-assertion-when-toggling-compositing.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179776 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agodashboard: BuildbotTesterQueueView crashesOnly logic is wrong
dburkart@apple.com [Sat, 7 Feb 2015 04:35:30 +0000 (04:35 +0000)]
dashboard: BuildbotTesterQueueView crashesOnly logic is wrong
https://bugs.webkit.org/show_bug.cgi?id=141349

Reviewed by Alexey Proskuryakov.

* BuildSlaveSupport/build.webkit.org-config/public_html/dashboard/Scripts/BuildbotTesterQueueView.js:
(BuildbotTesterQueueView.prototype.update.appendBuilderQueueStatus):
(BuildbotTesterQueueView.prototype.update):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179775 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoHave SQLiteStatement::database() return a reference
cdumez@apple.com [Sat, 7 Feb 2015 03:06:58 +0000 (03:06 +0000)]
Have SQLiteStatement::database() return a reference
https://bugs.webkit.org/show_bug.cgi?id=141348

Reviewed by Andreas Kling.

Have SQLiteStatement::database() return a reference as it can never
return null.

Source/WebCore:

* loader/icon/IconDatabase.cpp:
(WebCore::readySQLiteStatement):
* platform/sql/SQLiteStatement.h:
(WebCore::SQLiteStatement::database):

Source/WebKit2:

* DatabaseProcess/IndexedDB/sqlite/SQLiteIDBCursor.cpp:
(WebKit::SQLiteIDBCursor::internalAdvanceOnce):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179774 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoAdd youtube-nocookie URL to isYouTubeURL predicate
bfulgham@apple.com [Sat, 7 Feb 2015 01:38:15 +0000 (01:38 +0000)]
Add youtube-nocookie URL to isYouTubeURL predicate
https://bugs.webkit.org/show_bug.cgi?id=141347
<rdar://problem/19430657>

Reviewed by Eric Carlson.

* Modules/plugins/YouTubePluginReplacement.cpp:
(WebCore::isYouTubeURL): Update for additional youtube-nocookie site.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179773 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agogit-svn-id: https://svn.webkit.org/repository/webkit/trunk@179772 268f45cc-cd09-0410...
said@apple.com [Sat, 7 Feb 2015 01:24:48 +0000 (01:24 +0000)]
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179772 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoConvert the compositing overlap map to use LayoutRects
simon.fraser@apple.com [Sat, 7 Feb 2015 01:11:07 +0000 (01:11 +0000)]
Convert the compositing overlap map to use LayoutRects
https://bugs.webkit.org/show_bug.cgi?id=141346
Source/WebCore:

rdar://problem/18206365

Reviewed by Zalan Bujtas.

If two compositing layers were adjoining but not overlapping, but happened to
have non-integral offsets, then using enclosing IntRects in the overlap map
would cause us to think they are overlapping, and create unnecessary backing store.

Fix by converting the overlap map to use LayoutRects.

Test: compositing/layer-creation/subpixel-adjacent-layers-overlap.html

* rendering/RenderLayerCompositor.cpp:
(WebCore::OverlapMapContainer::add):
(WebCore::OverlapMapContainer::overlapsLayers):
(WebCore::RenderLayerCompositor::OverlapMap::add):
(WebCore::RenderLayerCompositor::OverlapMap::overlapsLayers):
(WebCore::RenderLayerCompositor::OverlapMap::RectList::append):
(WebCore::RenderLayerCompositor::OverlapMap::RectList::intersects):
(WebCore::RenderLayerCompositor::logLayerInfo):
(WebCore::RenderLayerCompositor::addToOverlapMap):
(WebCore::RenderLayerCompositor::addToOverlapMapRecursive):
(WebCore::RenderLayerCompositor::computeCompositingRequirements):
* rendering/RenderLayerCompositor.h:

LayoutTests:

Reviewed by Zalan Bujtas.

Test with adjacent layers on non-pixel boundaries.

* compositing/layer-creation/subpixel-adjacent-layers-overlap-expected.txt: Added.
* compositing/layer-creation/subpixel-adjacent-layers-overlap.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179771 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoRef-ify various getters that return HTMLCollection.
akling@apple.com [Sat, 7 Feb 2015 01:08:52 +0000 (01:08 +0000)]
Ref-ify various getters that return HTMLCollection.
<https://webkit.org/b/141336>

Reviewed by Anders Carlsson.

Make all the getters that return HTMLCollection objects (and never return nullptr)
return Ref instead of RefPtr.

Removed a couple of useless null checks that were exposed by this change.

* accessibility/AccessibilityRenderObject.cpp:
(WebCore::AccessibilityRenderObject::getDocumentLinks):
* bindings/js/JSDOMWindowCustom.cpp:
(WebCore::namedItemGetter):
* bindings/js/JSHTMLDocumentCustom.cpp:
(WebCore::JSHTMLDocument::nameGetter):
* dom/Document.cpp:
(WebCore::Document::ensureCachedCollection):
(WebCore::Document::images):
(WebCore::Document::applets):
(WebCore::Document::embeds):
(WebCore::Document::plugins):
(WebCore::Document::scripts):
(WebCore::Document::links):
(WebCore::Document::forms):
(WebCore::Document::anchors):
(WebCore::Document::all):
(WebCore::Document::windowNamedItems):
(WebCore::Document::documentNamedItems):
(WebCore::Document::iconURLs):
* dom/Document.h:
* dom/Element.cpp:
(WebCore::Element::ensureCachedHTMLCollection):
* dom/Element.h:
* html/ColorInputType.cpp:
(WebCore::ColorInputType::suggestions):
* html/HTMLDataListElement.cpp:
(WebCore::HTMLDataListElement::options):
* html/HTMLDataListElement.h:
* html/HTMLElement.cpp:
(WebCore::HTMLElement::children):
* html/HTMLElement.h:
* html/HTMLFieldSetElement.cpp:
(WebCore::HTMLFieldSetElement::elements):
* html/HTMLFieldSetElement.h:
* html/HTMLFormElement.cpp:
(WebCore::HTMLFormElement::elements):
* html/HTMLFormElement.h:
* html/HTMLInputElement.cpp:
(WebCore::HTMLInputElement::setupDateTimeChooserParameters):
* html/HTMLMapElement.cpp:
(WebCore::HTMLMapElement::areas):
* html/HTMLMapElement.h:
* html/HTMLSelectElement.cpp:
(WebCore::HTMLSelectElement::selectedOptions):
(WebCore::HTMLSelectElement::options):
* html/HTMLSelectElement.h:
* html/HTMLTableElement.cpp:
(WebCore::HTMLTableElement::rows):
(WebCore::HTMLTableElement::tBodies):
* html/HTMLTableElement.h:
* html/HTMLTableRowElement.cpp:
(WebCore::HTMLTableRowElement::insertCell):
(WebCore::HTMLTableRowElement::deleteCell):
(WebCore::HTMLTableRowElement::cells):
* html/HTMLTableRowElement.h:
* html/HTMLTableSectionElement.cpp:
(WebCore::HTMLTableSectionElement::insertRow):
(WebCore::HTMLTableSectionElement::deleteRow):
(WebCore::HTMLTableSectionElement::rows):
* html/HTMLTableSectionElement.h:
* html/RangeInputType.cpp:
(WebCore::RangeInputType::updateTickMarkValues):
* rendering/RenderTheme.cpp:
(WebCore::RenderTheme::paintSliderTicks):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179770 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years ago[iOS] Implement audio track selection in fullscreen.
bfulgham@apple.com [Sat, 7 Feb 2015 00:39:59 +0000 (00:39 +0000)]
[iOS] Implement audio track selection in fullscreen.
https://bugs.webkit.org/show_bug.cgi?id=131236
<rdar://problem/16552632>

Reviewed by Eric Carlson.

* platform/ios/WebVideoFullscreenModelVideoElement.h:
* platform/ios/WebVideoFullscreenModelVideoElement.mm:
(WebVideoFullscreenModelVideoElement::selectAudioMediaOption): Provide implementation.
(WebVideoFullscreenModelVideoElement::updateLegibleOptions): Add audio track information
to menu displayed to user.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179768 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoDFG SSA shouldn't have SetArgument nodes
fpizlo@apple.com [Fri, 6 Feb 2015 23:37:39 +0000 (23:37 +0000)]
DFG SSA shouldn't have SetArgument nodes
https://bugs.webkit.org/show_bug.cgi?id=141342

Reviewed by Mark Lam.

I was wondering why we kept the SetArgument around for captured
variables. It turns out we did so because we thought we had to, even
though we didn't have to. The node is meaningless in SSA.

* dfg/DFGSSAConversionPhase.cpp:
(JSC::DFG::SSAConversionPhase::run):
* ftl/FTLLowerDFGToLLVM.cpp:
(JSC::FTL::LowerDFGToLLVM::compileNode):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179767 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoThe delta value in the chart pane sometimes doens't show '+' for a positive delta
rniwa@webkit.org [Fri, 6 Feb 2015 23:31:30 +0000 (23:31 +0000)]
The delta value in the chart pane sometimes doens't show '+' for a positive delta
https://bugs.webkit.org/show_bug.cgi?id=141340

Reviewed by Andreas Kling.

The bug was caused by computeStatus prefixing the value delta with '+' if it's greater than 0 after
it had already been formatted. Fixed the bug by using a formatter that always emits a sign symbol.

* public/v2/app.js:
(App.Pane.computeStatus):
(App.createChartData):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179766 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoUnreviewed build fix. currentPoint wasn't defined when selectedPoints was used to...
rniwa@webkit.org [Fri, 6 Feb 2015 22:55:57 +0000 (22:55 +0000)]
Unreviewed build fix. currentPoint wasn't defined when selectedPoints was used to find points.

* public/v2/app.js:
(App.PaneController._updateDetails):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179765 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoUnreviewed. Commit the forgotten change.
rniwa@webkit.org [Fri, 6 Feb 2015 22:47:31 +0000 (22:47 +0000)]
Unreviewed. Commit the forgotten change.

* public/include/manifest.php:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179764 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoNew perf dashboard should have multiple dashboard pages
rniwa@webkit.org [Fri, 6 Feb 2015 22:43:08 +0000 (22:43 +0000)]
New perf dashboard should have multiple dashboard pages
https://bugs.webkit.org/show_bug.cgi?id=141339

Reviewed by Chris Dumez.

Added the support for multiple dashboard pages. Also added the status of the latest data point.
e.g. "5% better than target"

* public/v2/app.css: Tweaked the styles to work around the fact Ember.js creates empty script elements.
Also hid the border lines around charts on the dashboard page for a cleaner look.

* public/v2/app.js:
(App.IndexRoute): Added. Navigate to /dashboard/<defaultDashboardName> once the manifest.json is loaded.
(App.IndexRoute.beforeModel): Added.
(App.DashboardRoute): Added.
(App.DashboardRoute.model): Added. Return the dashboard specified by the name.
(App.CustomDashboardRoute): Added. This route is used for a customized dashboard specified by "grid".
(App.CustomDashboardRoute.model): Create a dashboard model from "grid" query parameter.
(App.CustomDashboardRoute.renderTemplate): Use the dashboard template.
(App.DashboardController): Renamed from App.IndexController.
(App.DashboardController.modelChanged): Renamed from gridChanged. Removed the code to deal with "grid"
and "defaultDashboard" as these are taken care of by newly added routers.
(App.DashboardController.computeGrid): Renamed from updateGrid. No longer updates "grid" since this is
now done in actions.toggleEditMode.
(App.DashboardController.actions.toggleEditMode): Navigate to CustomDashboardRoute when start editing
an existing dashboard.

(App.Pane.computeStatus): Moved from App.PaneController so that to be called in App.Pane.latestStatus.
Also moved the code to compute the delta with respect to the previous data point from _updateDetails.
(App.Pane._relativeDifferentToLaterPointInTimeSeries): Ditto.
(App.Pane.latestStatus): Added. Used by the dashboard template to show the status of the latest result.

(App.createChartData): Added deltaFormatter to show less significant digits for differences.

(App.PaneController._updateDetails): Updated per changes to computeStatus.

* public/v2/chart-pane.css: Added style rules for the status labels on the dashboard.

* public/v2/data.js:
(TimeSeries.prototype.lastPoint): Added.

* public/v2/index.html: Prefetch manifest.json as soon as possible, show the latest data points' status
on the dashboard, and enumerate all predefined dashboards.

* public/v2/interactive-chart.js:
(App.InteractiveChartComponent._relayoutDataAndAxes): Slightly adjust the offset at which we show unit
for the dashboard page.

* public/v2/manifest.js:
(App.Dashboard): Inherit from App.NameLabelModel now that each predefined dashboard has a name.
(App.MetricSerializer.normalizePayload): Parse all predefined dashboards instead of a single dashboard.
IDs are generated for each dashboard for forward compatibility.
(App.Manifest):
(App.Manifest.dashboardByName): Added.
(App.Manifest.defaultDashboardName): Added.
(App.Manifest._fetchedManifest): Create dashboard model objects for all predefined ones.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179763 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years ago[MSE] Implement Append Error algorithm.
commit-queue@webkit.org [Fri, 6 Feb 2015 22:14:20 +0000 (22:14 +0000)]
[MSE] Implement Append Error algorithm.
https://bugs.webkit.org/show_bug.cgi?id=139439

Patch by Bartlomiej Gajda <b.gajda@samsung.com> on 2015-02-06
Reviewed by Jer Noble.

If Source Buffer has not received first init segment, then it shall call endOfStream after receiving
Media Segment, as per Media Source spec. (from 17 July 2014) in paragraph 3.5.1 point 6.1.
Source/WebCore:

Based this change on Editor's Draft 12 December 2014, as it clarifies order of events.

Test: media/media-source/media-source-append-media-segment-without-init.html

* Modules/mediasource/MediaSource.cpp:
(WebCore::MediaSource::streamEndedWithError):
* Modules/mediasource/MediaSource.h:
* Modules/mediasource/SourceBuffer.cpp:
(WebCore::SourceBuffer::sourceBufferPrivateAppendComplete):
(WebCore::SourceBuffer::sourceBufferPrivateDidReceiveInitializationSegment):
(WebCore::SourceBuffer::validateInitializationSegment):
(WebCore::SourceBuffer::appendError):
* Modules/mediasource/SourceBuffer.h:

LayoutTests:

Added test which after creating SourceBuffer sends media sample, without any init segments.
Updated existing tests, so they correctly expect updateend and error as per Append Error algorithm.

* media/media-source/media-source-append-failed-expected.txt:
* media/media-source/media-source-append-failed.html:
* media/media-source/media-source-append-media-segment-without-init-expected.txt: Added.
* media/media-source/media-source-append-media-segment-without-init.html: Added.
* media/media-source/media-source-multiple-initialization-segments-expected.txt:
* media/media-source/media-source-multiple-initialization-segments.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179762 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoREGRESSION: Lookup doesn't work in RTL
timothy_horton@apple.com [Fri, 6 Feb 2015 21:54:45 +0000 (21:54 +0000)]
REGRESSION: Lookup doesn't work in RTL
https://bugs.webkit.org/show_bug.cgi?id=141338
<rdar://problem/19738407>

Reviewed by Dan Bernstein.

* editing/Editor.cpp:
(WebCore::Editor::scanSelectionForTelephoneNumbers):
* editing/mac/DictionaryLookup.mm:
(WebCore::rangeExpandedAroundPositionByCharacters):
Positions are independent of writing direction, so we don't
need to (and shouldn't) do anything special for RTL here.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179758 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoCorrect expectations for inspector/css/selector-dynamic-specificity.html.
ap@apple.com [Fri, 6 Feb 2015 21:48:41 +0000 (21:48 +0000)]
Correct expectations for inspector/css/selector-dynamic-specificity.html.

"Slow Pass Timeout" is not currently valid - if we expect a flaky timeout, we need
to expect it within the regular period of time with "Pass Timeout".

* platform/mac-wk2/TestExpectations:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179757 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoIt should be possible to use the DFG SetArgument node to indicate that someone set...
fpizlo@apple.com [Fri, 6 Feb 2015 21:39:04 +0000 (21:39 +0000)]
It should be possible to use the DFG SetArgument node to indicate that someone set the value of a local out-of-band
https://bugs.webkit.org/show_bug.cgi?id=141337

Reviewed by Mark Lam.

This mainly involved ensuring that SetArgument behaves just like SetLocal from a CPS standpoint, but with a special case for those SetArguments that
are associated with the prologue.

* dfg/DFGCPSRethreadingPhase.cpp:
(JSC::DFG::CPSRethreadingPhase::run):
(JSC::DFG::CPSRethreadingPhase::canonicalizeSet):
(JSC::DFG::CPSRethreadingPhase::canonicalizeLocalsInBlock):
(JSC::DFG::CPSRethreadingPhase::specialCaseArguments):
(JSC::DFG::CPSRethreadingPhase::canonicalizeSetLocal): Deleted.
(JSC::DFG::CPSRethreadingPhase::canonicalizeSetArgument): Deleted.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179756 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoWeb Inspector: REGRESSION: CSS Resource appears as empty after editing it via Styles...
jonowells@apple.com [Fri, 6 Feb 2015 21:33:26 +0000 (21:33 +0000)]
Web Inspector: REGRESSION: CSS Resource appears as empty after editing it via Styles sidebar
https://bugs.webkit.org/show_bug.cgi?id=140586

Reviewed by Timothy Hatcher.

Update SourceCode#_processContent to properly handle the promise returned from CSSAgent so that the content
will properly update when a style has changed. Properly clear the existing _requestContentPromise on the
stylesheet so that the content will update correctly.

* UserInterface/Controllers/CSSStyleManager.js: Drive-by style updates.
(WebInspector.CSSStyleManager.prototype._fetchInfoForAllStyleSheets):
* UserInterface/Models/CSSStyleSheet.js: Drive-by inheritance style update.
(WebInspector.CSSStyleSheet.prototype.requestContentFromBackend): Remove unnecessary backend promise function call.
* UserInterface/Models/Resource.js: Drive-by removal of unused variable.
* UserInterface/Models/SourceCode.js:
(WebInspector.SourceCode.prototype.markContentAsStale): Clear _requestContentPromise.
(WebInspector.SourceCode.prototype._processContent): Handle `parameters.text` correctly.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179755 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoReport network process crashes during layout tests
ap@apple.com [Fri, 6 Feb 2015 21:21:08 +0000 (21:21 +0000)]
Report network process crashes during layout tests
https://bugs.webkit.org/show_bug.cgi?id=139646

Reviewed by Anders Carlsson.

Source/WebKit2:

Added a way to get network process pid, modeled after how we do this for web process.

* UIProcess/API/C/mac/WKContextPrivateMac.h:
* UIProcess/API/C/mac/WKContextPrivateMac.mm:
(WKContextGetNetworkProcessIdentifier):
* UIProcess/WebProcessPool.cpp:
(WebKit::WebProcessPool::networkProcessCrashed): Don't reset m_networkProcess until
after calling the client, so that the client could retrieve its pid.
(WebKit::WebProcessPool::networkProcessIdentifier):
* UIProcess/WebProcessPool.h:

Tools:

* WebKitTestRunner/TestController.cpp:
(WTR::TestController::initialize):
(WTR::TestController::networkProcessName):
(WTR::TestController::networkProcessDidCrash):
* WebKitTestRunner/TestController.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179754 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoMachineThreads should be ref counted.
mark.lam@apple.com [Fri, 6 Feb 2015 20:57:45 +0000 (20:57 +0000)]
MachineThreads should be ref counted.
<https://webkit.org/b/141317>

Reviewed by Filip Pizlo.

The VM's MachineThreads registry object is being referenced from other
threads as a raw pointer.  In a scenario where the VM is destructed on
the main thread, there is no guarantee that another thread isn't still
holding a reference to the registry and will eventually invoke
removeThread() on it on thread exit.  Hence, there's a possible use
after free scenario here.

The fix is to make MachineThreads ThreadSafeRefCounted, and have all
threads that references keep a RefPtr to it to ensure that it stays
alive until the very last thread is done with it.

* API/tests/testapi.mm:
(useVMFromOtherThread): - Renamed to be more descriptive.
(useVMFromOtherThreadAndOutliveVM):
- Added a test that has another thread which uses the VM outlive the
  VM to confirm that there is no crash.

  However, I was not actually able to get the VM to crash without this
  patch because I wasn't always able to the thread destructor to be
  called.  With this patch applied, I did verify with some logging that
  the MachineThreads registry is only destructed after all threads
  have removed themselves from it.

(threadMain): Deleted.

* heap/Heap.cpp:
(JSC::Heap::Heap):
(JSC::Heap::~Heap):
(JSC::Heap::gatherStackRoots):
* heap/Heap.h:
(JSC::Heap::machineThreads):
* heap/MachineStackMarker.cpp:
(JSC::MachineThreads::Thread::Thread):
(JSC::MachineThreads::addCurrentThread):
(JSC::MachineThreads::removeCurrentThread):
* heap/MachineStackMarker.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179753 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoSupport overriding the deviceScaleFactor per WKWebView/WKView
timothy@apple.com [Fri, 6 Feb 2015 20:46:19 +0000 (20:46 +0000)]
Support overriding the deviceScaleFactor per WKWebView/WKView
https://bugs.webkit.org/show_bug.cgi?id=141311

Reviewed by Tim Horton.

* UIProcess/API/Cocoa/WKViewPrivate.h:
* UIProcess/API/Cocoa/WKWebView.mm:
(-[WKWebView _setOverrideDeviceScaleFactor:]):
(-[WKWebView _overrideDeviceScaleFactor]):
* UIProcess/API/Cocoa/WKWebViewPrivate.h:
* UIProcess/API/mac/WKView.mm:
(-[WKView _intrinsicDeviceScaleFactor]):
(-[WKView _setOverrideDeviceScaleFactor:]):
(-[WKView _overrideDeviceScaleFactor]):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179752 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoUnreviewed, rolling out r179743.
commit-queue@webkit.org [Fri, 6 Feb 2015 20:25:05 +0000 (20:25 +0000)]
Unreviewed, rolling out r179743.
https://bugs.webkit.org/show_bug.cgi?id=141335

caused missing symbols in non-WebKit clients of WTF::Vector
(Requested by kling on #webkit).

Reverted changeset:

"Remove WTF::fastMallocGoodSize()."
https://bugs.webkit.org/show_bug.cgi?id=141020
http://trac.webkit.org/changeset/179743

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179751 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoREGRESSION(r179706): Caused memory corruption on some tests (Requested by _ap_ on...
mjs@apple.com [Fri, 6 Feb 2015 18:56:48 +0000 (18:56 +0000)]
REGRESSION(r179706): Caused memory corruption on some tests (Requested by _ap_ on #webkit).
https://bugs.webkit.org/show_bug.cgi?id=141324

Reviewed by Alexey Proskuryakov.

No new tests. This is caught by existing tests under ASAN, and I don't know how to reproduce
it without ASAN.

* rendering/RenderLineBoxList.cpp:
(WebCore::RenderLineBoxList::dirtyLinesFromChangedChild): Give up
and just always invalidate the next line. It's too hard to come up
with the condition that catches all needed cases, doesn't itself
cause a crash, and isn't overzealous. And we do this for the
previous line anyway.  Also clean up the code a bit since it
confusingly reuses a variable, and declares it uninitialized, for
no good reason.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179750 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agohttp/tests/xmlhttprequest/event-listener-gc.html is flaky
ap@apple.com [Fri, 6 Feb 2015 18:50:25 +0000 (18:50 +0000)]
http/tests/xmlhttprequest/event-listener-gc.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=33342

Tweaked test output a little, hopefully this will shed some light on what happens in failure case.

* http/tests/xmlhttprequest/event-listener-gc-expected.txt:
* http/tests/xmlhttprequest/event-listener-gc.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179749 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agorun-jsc-stress-tests --remote should create remote directory before copying the bundle
ossy@webkit.org [Fri, 6 Feb 2015 16:27:54 +0000 (16:27 +0000)]
run-jsc-stress-tests --remote should create remote directory before copying the bundle
https://bugs.webkit.org/show_bug.cgi?id=141329

Reviewed by Michael Saboff.

* Scripts/run-jsc-stress-tests:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179747 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoRemove BytecodeGenerator::preserveLastVar() and replace it with a more robust mechani...
fpizlo@apple.com [Fri, 6 Feb 2015 16:04:39 +0000 (16:04 +0000)]
Remove BytecodeGenerator::preserveLastVar() and replace it with a more robust mechanism for preserving non-temporary registers
https://bugs.webkit.org/show_bug.cgi?id=141211

Reviewed by Mark Lam.

Previously, the way non-temporary registers were preserved (i.e. not reclaimed anytime
we did newTemporary()) by calling preserveLastVar() after all non-temps are created. It
would raise the refcount on the last (highest-numbered) variable created, and rely on
the fact that register reclamation started at higher-numbered registers and worked its
way down. So any retained register would block any lower-numbered registers from being
reclaimed.

Also, preserveLastVar() sets a thing called m_firstConstantIndex. It's unused.

This removes preserveLastVar() and makes addVar() retain each register it creates. This
is more explicit, since addVar() is the mechanism for creating non-temporary registers.

To make this work I had to remove an assertion that Register::setIndex() can only be
called when the refcount is zero. This method might be called after a var is created to
change its index. This previously worked because preserveLastVar() would be called after
we had already made all index changes, so the vars would still have refcount zero. Now
they have refcount 1. I think it's OK to lose this assertion; I can't remember this
assertion ever firing in a way that alerted me to a serious issue.

* bytecompiler/BytecodeGenerator.cpp:
(JSC::BytecodeGenerator::BytecodeGenerator):
(JSC::BytecodeGenerator::preserveLastVar): Deleted.
* bytecompiler/BytecodeGenerator.h:
(JSC::BytecodeGenerator::addVar):
* bytecompiler/RegisterID.h:
(JSC::RegisterID::setIndex):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179746 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years ago[GTK] Remove WebKitWebView::close-notification signal
carlosgc@webkit.org [Fri, 6 Feb 2015 14:50:20 +0000 (14:50 +0000)]
[GTK] Remove WebKitWebView::close-notification signal
https://bugs.webkit.org/show_bug.cgi?id=141330

Reviewed by Gustavo Noronha Silva.

Source/WebKit2:

In favor of a WebKitNotification::closed signal and
webkit_notification_close() method that both applications and
WebKit can use to close a notification. This also fixes the
onclose event that was not fired when the notification was
closed. It also brings back padding space in WebKitWebViewClass.

* UIProcess/API/gtk/WebKitNotification.cpp:
(webkit_notification_class_init): Add WebKitNotification::closed signal.
(webkit_notification_close): Emit WebKitNotification::closed.
* UIProcess/API/gtk/WebKitNotification.h:
* UIProcess/API/gtk/WebKitNotificationProvider.cpp:
(WebKitNotificationProvider::notificationCloseCallback): Callback
for WebKitNotification::closed signal that notifies the WebProcess
and removes the notification from the map.
(WebKitNotificationProvider::show): Connect to WebKitNotification::closed.
(WebKitNotificationProvider::cancelNotificationByID): Call webkit_notification_close().
* UIProcess/API/gtk/WebKitNotificationProvider.h:
* UIProcess/API/gtk/WebKitWebView.cpp:
(notifyNotificationClosed): The user closed the annotation, call
webkit_notification_close().
(webNotificationClosed): The WebKitNotification has been closed,
close the libnotify notification if it hasn't been closed yet.
(webkitWebViewShowNotification): Create the libnotifiy
notification if needed and associate it to the WebKitNotification
as user data. Connect to the closed signal of both, the libnotifiy
notification and the WebKit notification.
(webkitWebViewCloseNotification): Deleted.
(webkit_web_view_class_init): Remove close-notification signal and
the default hanlder.
(webkitWebViewEmitCloseNotification): Deleted.
* UIProcess/API/gtk/WebKitWebView.h:
* UIProcess/API/gtk/WebKitWebViewPrivate.h:
* UIProcess/API/gtk/docs/webkit2gtk-4.0-sections.txt: Add webkit_notification_close.

Tools:

Update notifications unit tests according to the API changes, and
add a test case to check that onclose event is fired when a
notification is closed by the user.

* TestWebKitAPI/Tests/WebKit2Gtk/TestWebKitWebView.cpp:
(testWebViewNotification):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179745 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoASSERTION FAILED: !m_adoptionIsRequired in WTF::RefCountedBase::ref
carlosgc@webkit.org [Fri, 6 Feb 2015 14:46:17 +0000 (14:46 +0000)]
ASSERTION FAILED: !m_adoptionIsRequired in WTF::RefCountedBase::ref
https://bugs.webkit.org/show_bug.cgi?id=141035

Reviewed by Sergio Villar Senin.

Rename PrinterListGtk::singleton() as PrinterListGtk::getOrCreate(), and
make it return nullptr when the shared PrinterListGtk object is
still being created. This can happen if the nested loop used by
gtk_enumerate_printers dispatches a GSource that starts a new
synchronous print operation. In that case we just ignore the
second print operation, since there's already one ongoing.

* WebProcess/WebCoreSupport/WebChromeClient.cpp:
(WebKit::WebChromeClient::print): Return early if
PrinterListGtk::getOrCreate() return nullptr.
* WebProcess/WebPage/gtk/PrinterListGtk.cpp:
(WebKit::PrinterListGtk::getOrCreate): Return nullptr if the
PrinterListGtk is still enumerating the printers.
(WebKit::PrinterListGtk::PrinterListGtk): Initialize
m_enumeratingPrinters to true before calling
gtk_enumerate_printers, and to false once it finishes.
(WebKit::PrinterListGtk::singleton): Deleted.
(WebKit::PrinterListGtk::enumeratePrintersFunction): Deleted.
* WebProcess/WebPage/gtk/PrinterListGtk.h:
* WebProcess/WebPage/gtk/WebPrintOperationGtk.cpp: Add an
assertion here since PrinterListGtk::getOrCreate() should never
return nullptr at this point.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179744 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoRemove WTF::fastMallocGoodSize().
akling@apple.com [Fri, 6 Feb 2015 10:03:56 +0000 (10:03 +0000)]
Remove WTF::fastMallocGoodSize().
<https://webkit.org/b/141020>

Reviewed by Anders Carlsson.

Source/JavaScriptCore:

* assembler/AssemblerBuffer.h:
(JSC::AssemblerData::AssemblerData):
(JSC::AssemblerData::grow):

Source/WTF:

bmalloc's good-size API just returns exactly whatever you pass it,
so it's of no utility to us anymore.

This gets rid of a bunch of pointless out-of-line calls in Vector
construction and growth.

* wtf/Compression.cpp:
(WTF::GenericCompressedData::create):
* wtf/FastMalloc.cpp:
(WTF::fastMallocGoodSize): Deleted.
* wtf/FastMalloc.h:
* wtf/Vector.h:
(WTF::VectorBufferBase::allocateBuffer):
(WTF::VectorBufferBase::tryAllocateBuffer):
(WTF::VectorBufferBase::reallocateBuffer):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179743 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agohttp/tests/xmlhttprequest/timeout/xmlhttprequest-timeout-abortedonmain.html is flaky
ap@apple.com [Fri, 6 Feb 2015 08:31:30 +0000 (08:31 +0000)]
http/tests/xmlhttprequest/timeout/xmlhttprequest-timeout-abortedonmain.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=141325

* platform/mac-wk2/TestExpectations: Mark it as such.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179742 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoinspector/css/selector-dynamic-specificity.html is very slow on Yosemite
ap@apple.com [Fri, 6 Feb 2015 08:14:11 +0000 (08:14 +0000)]
inspector/css/selector-dynamic-specificity.html is very slow on Yosemite
https://bugs.webkit.org/show_bug.cgi?id=141252

* platform/mac-wk2/TestExpectations: Add weaker expectations for debug builds.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179741 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoTest gardening for issues uncovered by disabling retries on debug bots.
ap@apple.com [Fri, 6 Feb 2015 04:32:46 +0000 (04:32 +0000)]
Test gardening for issues uncovered by disabling retries on debug bots.

* TestExpectations:
* platform/mac-wk2/TestExpectations:
* platform/mac/TestExpectations:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179740 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoRemove duplicate loop after r179532
diorahman@rockybars.com [Fri, 6 Feb 2015 04:07:09 +0000 (04:07 +0000)]
Remove duplicate loop after r179532
https://bugs.webkit.org/show_bug.cgi?id=141300

Reviewed by Benjamin Poulain.

No new tests, no behavior changed.

* css/SelectorCheckerTestFunctions.h:
(WebCore::matchesLangPseudoClass):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179739 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoUnreviewed, rolling out r179725.
commit-queue@webkit.org [Fri, 6 Feb 2015 03:55:10 +0000 (03:55 +0000)]
Unreviewed, rolling out r179725.
https://bugs.webkit.org/show_bug.cgi?id=141320

caused 2 layout tests to fail (Requested by zalan on #webkit).

Reverted changeset:

"[MSE] Implement Append Error algorithm."
https://bugs.webkit.org/show_bug.cgi?id=139439
http://trac.webkit.org/changeset/179725

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179738 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years ago[iOS] Run a full garbage collection on memory warning.
akling@apple.com [Fri, 6 Feb 2015 03:14:22 +0000 (03:14 +0000)]
[iOS] Run a full garbage collection on memory warning.
<https://webkit.org/b/141313>
<rdar://problem/19738024>

Reviewed by Chris Dumez.

Make sure that we run a full GC when trying to free up memory, as this might
be our last chance to execute before the kernel suspends this process.

This aligns WebKit2 with the old WebKit1 behavior.

* platform/cocoa/MemoryPressureHandlerCocoa.mm:
(WebCore::MemoryPressureHandler::platformReleaseMemory):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179737 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoNull deref in ViewGestureController::beginSwipeGesture when swiping while script...
timothy_horton@apple.com [Fri, 6 Feb 2015 02:21:40 +0000 (02:21 +0000)]
Null deref in ViewGestureController::beginSwipeGesture when swiping while script is navigating
https://bugs.webkit.org/show_bug.cgi?id=141308
<rdar://problem/18460046>

Reviewed by Simon Fraser.

* UIProcess/mac/ViewGestureControllerMac.mm:
(WebKit::ViewGestureController::trackSwipeGesture):
If script navigates (history.back, probably other cases too) while in the middle of
building up enough scroll events to start a swipe, it can destroy the history item
that we were planning to swipe to. If this happens, bail from the swipe.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179736 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoUnreviewed, EFL gardening. Unskip 4 passing tests regarding webgl.
gyuyoung.kim@samsung.com [Fri, 6 Feb 2015 01:34:07 +0000 (01:34 +0000)]
Unreviewed, EFL gardening. Unskip 4 passing tests regarding webgl.

Those tests were marked with *CRASH* on r179465 though, it looks
wrong.

* platform/efl/TestExpectations:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179733 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoDon't pass architecture to development plug-in XPC services
ap@apple.com [Fri, 6 Feb 2015 01:24:25 +0000 (01:24 +0000)]
Don't pass architecture to development plug-in XPC services
https://bugs.webkit.org/show_bug.cgi?id=141309

Reviewed by Anders Carlsson.

We now have separate services for 32-bit and 64-bit.

* Shared/EntryPointUtilities/mac/XPCService/XPCServiceMain.Development.mm:
(WebKit::reexec):
(WebKit::XPCServiceEventHandler):
* UIProcess/Launcher/mac/ProcessLauncherMac.mm:
(WebKit::connectToReExecService):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179732 268f45cc-cd09-0410-ab3c-d52691b4dbfc

6 years agoMove commits viewer to the end of details view
rniwa@webkit.org [Fri, 6 Feb 2015 01:23:31 +0000 (01:23 +0000)]
Move commits viewer to the end of details view
https://bugs.webkit.org/show_bug.cgi?id=141315

Rubber-stamped by Andreas Kling.

Show the difference instead of the old value per kling's request.

* public/v2/app.js:
(App.PaneController._updateDetails):
* public/v2/index.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@179731 268f45cc-cd09-0410-ab3c-d52691b4dbfc