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

Timeline



Jul 28, 2012:

9:59 PM Changeset in webkit [123977] by mitz@apple.com
  • 3 edits
    4 adds in trunk

Source/WebCore: RenderBlock::offsetForContents() is wrong in flipped blocks writing modes
https://bugs.webkit.org/show_bug.cgi?id=92588

Reviewed by Sam Weinig.

Tests: fast/multicol/hit-test-block-axis-flipped.html

fast/writing-mode/flipped-blocks-hit-test-overflow-scroll.html

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::offsetForContents): Flip the point, which puts it back in unflipped
coordinates, before adding the scroll offset and adjusting for columns, then flip it back.
(WebCore::RenderBlock::adjustPointToColumnContents): Changed the inline-axis cases to
work with an unflipped point.

LayoutTests: Hit test results are wrong in flipped blocks writing modes with block column axis
https://bugs.webkit.org/show_bug.cgi?id=92588

Reviewed by Sam Weinig.

  • fast/multicol/hit-test-block-axis-flipped-expected.txt: Added.
  • fast/multicol/hit-test-block-axis-flipped.html: Added.
8:38 PM Changeset in webkit [123976] by weinig@apple.com
  • 2 edits in trunk/Source/WebKit2

Stop copying unnecessary files in WebKit2 build
https://bugs.webkit.org/show_bug.cgi?id=92587

Reviewed by Dan Bernstein.

  • WebKit2.xcodeproj/project.pbxproj:

We were accidentally copying ColorSpaceData.h and EventDispatcher.messages.in into the
build products. Stop this.

6:35 PM Changeset in webkit [123975] by pkasting@chromium.org
  • 2 edits
    14 adds in trunk/LayoutTests

[Chromium] New baselines and test expectations update.
https://bugs.webkit.org/show_bug.cgi?id=72039
https://bugs.webkit.org/show_bug.cgi?id=92569
https://bugs.webkit.org/show_bug.cgi?id=92583

Unreviewed, test expectations update.

  • platform/chromium-linux-x86/compositing/geometry/bounds-ignores-hidden-composited-descendant-expected.txt: Added.
  • platform/chromium-linux-x86/compositing/geometry/bounds-ignores-hidden-dynamic-negzindex-expected.png: Added.
  • platform/chromium-linux-x86/compositing/geometry/bounds-ignores-hidden-dynamic-negzindex-expected.txt: Added.
  • platform/chromium-linux/compositing/geometry/bounds-ignores-hidden-composited-descendant-expected.txt: Added.
  • platform/chromium-linux/compositing/geometry/bounds-ignores-hidden-dynamic-negzindex-expected.png: Added.
  • platform/chromium-linux/compositing/geometry/bounds-ignores-hidden-dynamic-negzindex-expected.txt: Added.
  • platform/chromium-mac-snowleopard/compositing/geometry/bounds-ignores-hidden-dynamic-negzindex-expected.png: Added.
  • platform/chromium-mac/compositing/geometry/bounds-ignores-hidden-dynamic-negzindex-expected.png: Added.
  • platform/chromium-win-xp/compositing/geometry/bounds-ignores-hidden-composited-descendant-expected.txt: Added.
  • platform/chromium-win-xp/compositing/geometry/bounds-ignores-hidden-dynamic-negzindex-expected.png: Added.
  • platform/chromium-win-xp/compositing/geometry/bounds-ignores-hidden-dynamic-negzindex-expected.txt: Added.
  • platform/chromium-win/compositing/geometry/bounds-ignores-hidden-composited-descendant-expected.txt: Added.
  • platform/chromium-win/compositing/geometry/bounds-ignores-hidden-dynamic-negzindex-expected.png: Added.
  • platform/chromium-win/compositing/geometry/bounds-ignores-hidden-dynamic-negzindex-expected.txt: Added.
  • platform/chromium/TestExpectations:
4:28 PM Changeset in webkit [123974] by ryuan.choi@samsung.com
  • 3 edits in trunk/Source/WebKit2

[WK2][EFL] Add ewk_view_scale_{get|set} to EwkView.
https://bugs.webkit.org/show_bug.cgi?id=92446

Reviewed by Kenneth Rohde Christiansen.

Add methods to Ewk_View to get or set the scale factor of page.

  • UIProcess/API/efl/ewk_view.cpp:

(ewk_view_scale_set):
(ewk_view_scale_get):

  • UIProcess/API/efl/ewk_view.h:
2:40 PM Changeset in webkit [123973] by mitz@apple.com
  • 5 edits in trunk

In flipped lines writing modes, hit testing at the beginning of a column may return a result from the previous column
https://bugs.webkit.org/show_bug.cgi?id=92566

Reviewed by Simon Fraser.

Source/WebCore:

Enhanced the fix for <http://webkit.org/b/92524> to work with flipped lines.

Extended fast/multicol/hit-test-end-of-column-with-line-height.html.

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::positionForPointWithInlineChildren): For flipped lines, check if the
hit line is the last one before a page break, and in that case, check if the hit point was
after the break.

LayoutTests:

  • fast/multicol/hit-test-end-of-column-with-line-height-expected.txt: Updated.
  • fast/multicol/hit-test-end-of-column-with-line-height.html: Extended with tests for

horizontal-bt.

2:32 PM Changeset in webkit [123972] by Simon Fraser
  • 3 edits
    4 adds in trunk

Size changes on a layer with negative z-index children don't repaint correctly
https://bugs.webkit.org/show_bug.cgi?id=92583

Reviewed by Dean Jackson.

Source/WebCore:

Negative z-index descendants cause a composited layer to have a separate
layer for foreground content. If something caused that foreground layer
to change size (like toggling visibility on a descendant), we failed to
repaint the foreground layer, resulting in stretched content.

Test: compositing/geometry/bounds-ignores-hidden-dynamic-negzindex.html

  • rendering/RenderLayerBacking.cpp:

(WebCore::RenderLayerBacking::updateGraphicsLayerGeometry): Repaint the foreground
layer if its size changes, like we do for the main layer.

LayoutTests:

Test that causes a composited layer with a foreground layer (by virtue of having
negative z-index children) to resize.

Add missing pixel result for bounds-ignores-hidden-dynamic.html too.

  • compositing/geometry/bounds-ignores-hidden-dynamic-expected.png: Added.
  • compositing/geometry/bounds-ignores-hidden-dynamic-negzindex-expected.png: Added.
  • compositing/geometry/bounds-ignores-hidden-dynamic-negzindex-expected.txt: Added.
  • compositing/geometry/bounds-ignores-hidden-dynamic-negzindex.html: Added.
12:44 PM Changeset in webkit [123971] by Simon Fraser
  • 5 edits
    6 adds in trunk

Ignore visibility:hidden elements when computing compositing layer bounds
https://bugs.webkit.org/show_bug.cgi?id=92569

Reviewed by Dan Bernstein.

Source/WebCore:

When computing the bounds of compositing layers, we would take into account
layers with no visible content (visibility:hidden, and no non-hidden descendants).
This caused some huge layers in mobile Google maps.

Tests: compositing/geometry/bounds-ignores-hidden-composited-descendant.html

compositing/geometry/bounds-ignores-hidden-dynamic.html
compositing/geometry/bounds-ignores-hidden.html

  • rendering/RenderLayer.cpp:

(WebCore::RenderLayer::calculateLayerBounds): If the ExcludeHiddenDescendants flag is set, return
the empty rect for layers with no visible content or descendants, as long as the layer is not
the root of the subtree whose bounds are being computed.
calculateLayerBounds() currently (I think incorrectly) passes the default flags to the recursive calls.
It should probably just pass 'flags', but to make this change safe, OR in the ExcludeHiddenDescendants
flag.

  • rendering/RenderLayer.h: Add an ExcludeHiddenDescendants flag to the CalculateLayerBoundsFlags.
  • rendering/RenderLayerCompositor.cpp:

(WebCore::RenderLayerCompositor::calculateCompositedBounds): Pass the ExcludeHiddenDescendants
flag.

LayoutTests:

Tests related to visibility:hidden's affect on composited layer bounds.

  • compositing/geometry/bounds-ignores-hidden-composited-expected.txt: Added.
  • compositing/geometry/bounds-ignores-hidden-composited-descendant.html: Added.

Test with visibility:visible descendants of a visibility:hidden element in a
compositing layer.

  • compositing/geometry/bounds-ignores-hidden-dynamic-expected.txt: Added.
  • compositing/geometry/bounds-ignores-hidden-dynamic.html: Added.

Test that changes visibility on descendants of a compositing layer, to test dynamic changes.

  • compositing/geometry/bounds-ignores-hidden-expected.txt: Added.
  • compositing/geometry/bounds-ignores-hidden.html: Added.

Test with a compositing layer having visibility:hidden descendants, which dumps
layer tree information.

12:35 PM Changeset in webkit [123970] by pkasting@chromium.org
  • 2 edits in trunk/LayoutTests

[Chromium] Widen expectation for crashing test.
http://crbug.com/139493

Unreviewed, test expectations update.

  • platform/chromium/TestExpectations:
12:23 PM Changeset in webkit [123969] by commit-queue@webkit.org
  • 17 edits
    2 adds in trunk/Source/WebCore

Add an overlay play button to media controls on android
https://bugs.webkit.org/show_bug.cgi?id=92132

Patch by Min Qin <qinmin@chromium.org> on 2012-07-28
Reviewed by Adam Barth.

This change adds an overlay play button to the media controls for android.
No tests for now as this setting is not being used by other port.
Will add a test and adjust all the test expectations when we upstream the layout tests for android.

  • WebCore.gypi:
  • css/CSSPrimitiveValueMappings.h:

(WebCore::CSSPrimitiveValue::CSSPrimitiveValue):

  • css/CSSValueKeywords.in:
  • css/mediaControls.css:

(audio::-webkit-media-controls-overlay-play-button, video::-webkit-media-controls-overlay-play-button):

  • css/mediaControlsChromiumAndroid.css:

(audio::-webkit-media-controls-overlay-enclosure):
(video::-webkit-media-controls-overlay-enclosure):
(audio::-webkit-media-controls-play-button, video::-webkit-media-controls-play-button):
(video::-webkit-media-controls-overlay-play-button):

  • html/shadow/MediaControlElements.cpp:

(WebCore::MediaControlOverlayPlayButtonElement::MediaControlOverlayPlayButtonElement):
(WebCore):
(WebCore::MediaControlOverlayPlayButtonElement::create):
(WebCore::MediaControlOverlayPlayButtonElement::defaultEventHandler):
(WebCore::MediaControlOverlayPlayButtonElement::updateDisplayType):
(WebCore::MediaControlOverlayPlayButtonElement::shadowPseudoId):

  • html/shadow/MediaControlElements.h:

(MediaControlOverlayPlayButtonElement):
(WebCore):

  • html/shadow/MediaControlRootElementChromium.cpp:

(WebCore::MediaControlPanelEnclosureElement::MediaControlPanelEnclosureElement):
(WebCore::MediaControlPanelEnclosureElement::create):
(WebCore):
(WebCore::MediaControlPanelEnclosureElement::shadowPseudoId):
(WebCore::MediaControlRootElementChromium::create):
(WebCore::MediaControlRootElementChromium::initializeControls):

  • html/shadow/MediaControlRootElementChromium.h:

(MediaControlChromiumEnclosureElement):
(WebCore):
(MediaControlPanelEnclosureElement):
(MediaControlRootElementChromium):

  • html/shadow/MediaControlRootElementChromiumAndroid.cpp: Added.

(WebCore):
(WebCore::MediaControlOverlayEnclosureElement::MediaControlOverlayEnclosureElement):
(WebCore::MediaControlOverlayEnclosureElement::create):
(WebCore::MediaControlOverlayEnclosureElement::shadowPseudoId):
(WebCore::MediaControlRootElementChromiumAndroid::MediaControlRootElementChromiumAndroid):
(WebCore::MediaControls::create):
(WebCore::MediaControlRootElementChromiumAndroid::create):
(WebCore::MediaControlRootElementChromiumAndroid::setMediaController):
(WebCore::MediaControlRootElementChromiumAndroid::playbackStarted):
(WebCore::MediaControlRootElementChromiumAndroid::playbackStopped):

  • html/shadow/MediaControlRootElementChromiumAndroid.h: Added.

(WebCore):
(MediaControlOverlayEnclosureElement):
(MediaControlRootElementChromiumAndroid):

  • platform/ThemeTypes.h:
  • rendering/RenderMediaControlsChromium.cpp:

(WebCore::paintMediaOverlayPlayButton):
(WebCore):
(WebCore::RenderMediaControlsChromium::paintMediaControlsPart):

  • rendering/RenderTheme.cpp:

(WebCore::RenderTheme::paint):

  • rendering/RenderTheme.h:

(WebCore::RenderTheme::paintMediaOverlayPlayButton):

  • rendering/RenderThemeChromiumAndroid.cpp:

(WebCore::RenderThemeChromiumAndroid::paintMediaOverlayPlayButton):
(WebCore):

  • rendering/RenderThemeChromiumAndroid.h:
12:22 PM Changeset in webkit [123968] by pkasting@chromium.org
  • 2 edits in trunk/Tools

Fix Chromium/Win compile.
https://bugs.webkit.org/show_bug.cgi?id=88787

Unreviewed, build fix.

Chromium/Win needs _USE_MATH_DEFINES but doesn't define PLATFORM(WIN).
The other files in WebKit that define this macro (both in
WebCore/platform/graphics/cg) do it by unconditionally #defining it to 1
above the #include of "config.h", so copy that pattern here.

  • TestWebKitAPI/Tests/WTF/MediaTime.cpp:
11:45 AM Changeset in webkit [123967] by mario@webkit.org
  • 10 edits
    2 adds in trunk

[WK2][GTK] Implement a new spell checker API for WebKit2GTK+
https://bugs.webkit.org/show_bug.cgi?id=90268

Reviewed by Martin Robinson.

Source/WebKit2:

Add a simple spell checking API to WK2, allowing to enable/disable
this feature and to define a list of languages associated to it.

  • GNUmakefile.am: Add flags to handle the SPELLCHECK feature.
  • GNUmakefile.list.am: Added new files

Added new internal class that will act both as the implementation
of TextCheckerClient and as the object WebKitWebContext will
depend on to implement the newly added API.

  • UIProcess/API/gtk/WebKitTextChecker.cpp: Added.

(toTextChecker):
(continuousSpellCheckingEnabledCallback):
(setContinuousSpellCheckingEnabledCallback):
(checkSpellingOfStringCallback):
(guessesForWordCallback):
(learnWordCallback):
(ignoreWordCallback):
(WebKitTextChecker::~WebKitTextChecker):
(WebKitTextChecker::create):
(WebKitTextChecker::WebKitTextChecker):
(WebKitTextChecker::checkSpellingOfString):
(WebKitTextChecker::getGuessesForWord):
(WebKitTextChecker::learnWord):
(WebKitTextChecker::ignoreWord):
(WebKitTextChecker::setSpellCheckingEnabled):
(WebKitTextChecker::setSpellCheckingLanguages):

  • UIProcess/API/gtk/WebKitTextChecker.h: Added.

(WebKitTextChecker):
(WebKitTextChecker::isSpellCheckingEnabled):
(WebKitTextChecker::getSpellCheckingLanguages):

Added WKTextChecker to WebKitPrivate.h, needed in WebKitTextChecker.

  • UIProcess/API/gtk/WebKitPrivate.h:

Add new API to WebKitWebContext to allow enabling/disabling this
spell checking feature, and to set/get the list of languages used
to decide which dictionaries will be consulted.

  • UIProcess/API/gtk/WebKitWebContext.cpp:

(_WebKitWebContextPrivate):
(createDefaultWebContext):
(webkit_web_context_get_spell_checking_enabled): New API.
(webkit_web_context_set_spell_checking_enabled): Ditto.
(webkit_web_context_get_spell_checking_languages): Ditto.
(webkit_web_context_set_spell_checking_languages): Ditto.

  • UIProcess/API/gtk/WebKitWebContext.h:

New tests for checking the new API added to WebKitWebContext.

  • UIProcess/API/gtk/tests/TestWebKitWebContext.cpp:

(testWebContextSpellChecker):
(beforeAll):

Update documentation with new sections and symbols.

  • UIProcess/API/gtk/docs/webkit2gtk-docs.sgml:
  • UIProcess/API/gtk/docs/webkit2gtk-sections.txt:

Tools:

Ignore WebKitTextChecker.* private files for gtk-doc.

  • gtk/generate-gtkdoc:

(get_webkit2_options):

11:28 AM Changeset in webkit [123966] by mario@webkit.org
  • 5 edits
    2 adds in trunk/Source

[GTK] Add a new and reusable enchant-based spellchecker in WebCore
https://bugs.webkit.org/show_bug.cgi?id=90269

Reviewed by Martin Robinson.

Source/WebCore:

Move enchant specific code down to WebCore, into a new class
TextCheckerEnchant, that we can use from WK1 and WK2.

  • GNUmakefile.am: Add flags to handle the SPELLCHECK feature.
  • GNUmakefile.list.am: Added new files.
  • platform/text/gtk/TextCheckerEnchant.cpp: Added.

(getAvailableDictionariesCallback):
(TextCheckerEnchant::TextCheckerEnchant):
(TextCheckerEnchant::~TextCheckerEnchant):
(TextCheckerEnchant::ignoreWord):
(TextCheckerEnchant::learnWord):
(TextCheckerEnchant::checkSpellingOfString):
(TextCheckerEnchant::getGuessesForWord):
(TextCheckerEnchant::updateSpellCheckingLanguages):
(TextCheckerEnchant::freeEnchantBrokerDictionaries):

  • platform/text/gtk/TextCheckerEnchant.h: Added.

(WebCore):
(TextCheckerEnchant):
(WebCore::TextCheckerEnchant::create):

Source/WebKit/gtk:

Remove enchant specific code from WebKitSpellCheckerEnchant and
implement it relying in the new TextCheckerEnchant class in WebCore.

  • webkit/webkitspellcheckerenchant.cpp:

(_WebKitSpellCheckerEnchantPrivate):
(webkit_spell_checker_enchant_finalize):
(webkit_spell_checker_enchant_class_init):
(webkit_spell_checker_enchant_init):
(checkSpellingOfString):
(getGuessesForWord):
(updateSpellCheckingLanguages):
(learnWord):
(ignoreWord):

11:05 AM Changeset in webkit [123965] by pkasting@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

Sigh. Just disable all the AutoResize tests after the first three.
https://bugs.webkit.org/show_bug.cgi?id=92318

Unreviewed, making bots happier.

  • tests/WebViewTest.cpp:
10:36 AM Changeset in webkit [123964] by pkasting@chromium.org
  • 15 edits
    13 adds
    2 deletes in trunk/LayoutTests

[Chromium] Rebaselining and test expectations update.
https://bugs.webkit.org/show_bug.cgi?id=77188
https://bugs.webkit.org/show_bug.cgi?id=85856
https://bugs.webkit.org/show_bug.cgi?id=87844
https://bugs.webkit.org/show_bug.cgi?id=91756
http://crbug.com/10364
http://crbug.com/138159
http://crbug.com/139162
http://crbug.com/139493

Unreviewed, test expectations update.

  • fast/overflow/unreachable-overflow-rtl-bug-expected.txt: Added.
  • platform/chromium-linux-x86/platform/chromium: Added.
  • platform/chromium-linux-x86/platform/chromium/virtual: Added.
  • platform/chromium-linux-x86/platform/chromium/virtual/gpu: Added.
  • platform/chromium-linux-x86/platform/chromium/virtual/gpu/fast: Added.
  • platform/chromium-linux-x86/platform/chromium/virtual/gpu/fast/canvas: Added.
  • platform/chromium-linux-x86/platform/chromium/virtual/gpu/fast/canvas/canvas-text-alignment-expected.png: Added.
  • platform/chromium-linux-x86/platform/chromium/virtual/gpu/fast/canvas/gradient-add-second-start-end-stop-expected.png: Added.
  • platform/chromium-linux/platform/chromium/virtual/gpu/fast/canvas/canvas-text-alignment-expected.png:
  • platform/chromium-linux/platform/chromium/virtual/gpu/fast/canvas/gradient-add-second-start-end-stop-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/filters-example-01-b-expected.png:
  • platform/chromium-mac-snowleopard/fast/block/float/026-expected.png:
  • platform/chromium-mac-snowleopard/fast/block/float/028-expected.png:
  • platform/chromium-mac-snowleopard/fast/forms/datalist: Added.
  • platform/chromium-mac-snowleopard/fast/forms/datalist/input-appearance-range-with-datalist-expected.png: Added.
  • platform/chromium-mac-snowleopard/fast/forms/datalist/input-appearance-range-with-datalist-zoomed-expected.png: Added.
  • platform/chromium-mac-snowleopard/platform/chromium/virtual/gpu/fast/canvas/canvas-text-alignment-expected.png:
  • platform/chromium-mac/fast/block/float/026-expected.png:
  • platform/chromium-mac/fast/block/float/028-expected.png:
  • platform/chromium-mac/fast/overflow/unreachable-overflow-rtl-bug-expected.png:
  • platform/chromium-mac/fast/overflow/unreachable-overflow-rtl-bug-expected.txt:
  • platform/chromium-mac/platform/chromium/virtual/gpu/fast/canvas/canvas-text-alignment-expected.png:
  • platform/chromium-mac/platform/chromium/virtual/gpu/fast/canvas/gradient-add-second-start-end-stop-expected.png: Added.
  • platform/chromium-win/fast/forms/box-shadow-override-expected.png:
  • platform/chromium-win/platform/chromium/virtual/gpu/fast/canvas/canvas-text-alignment-expected.png:
  • platform/chromium-win/platform/chromium/virtual/gpu/fast/canvas/gradient-add-second-start-end-stop-expected.png:
  • platform/chromium/TestExpectations:
  • platform/efl/fast/overflow/unreachable-overflow-rtl-bug-expected.txt: Removed.
  • platform/gtk/fast/overflow/unreachable-overflow-rtl-bug-expected.txt: Removed.
9:19 AM Changeset in webkit [123963] by pkasting@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

Try disabling different test in hopes of tracking down the real problem.
https://bugs.webkit.org/show_bug.cgi?id=92318

Unreviewed, making bots happier.

  • tests/WebViewTest.cpp:
9:11 AM Changeset in webkit [123962] by commit-queue@webkit.org
  • 11 edits in trunk/Source/WebCore

[Clamp] support in binding generator.
https://bugs.webkit.org/show_bug.cgi?id=77605

Patch by Vineet Chaudhary <Vineet> on 2012-07-28
Reviewed by Kentaro Hara.

Currently [Clamp] extended attribute in IDL files.
Spec: http://www.w3.org/TR/2012/CR-WebIDL-20120419/#Clamp

Tests: TestObj.idl

  • bindings/scripts/CodeGeneratorCPP.pm:

(SkipFunction): Skips methods with [Clamp] parameters.

  • bindings/scripts/CodeGeneratorGObject.pm:

(SkipFunction): Skips methods with [Clamp] parameters.

  • bindings/scripts/CodeGeneratorJS.pm:

(GenerateParametersCheck): Support [Clamp] for JSBindings.

  • bindings/scripts/CodeGeneratorObjC.pm:

(SkipFunction): Skips methods with [Clamp] parameters.

  • bindings/scripts/CodeGeneratorV8.pm:

(GenerateParametersCheck): Support [Clamp] for V8Bindings.

  • bindings/scripts/IDLAttributes.txt: Added [Clamp] to attributes list.
  • bindings/scripts/test/JS/JSTestObj.cpp: Test.

(WebCore::jsTestObjPrototypeFunctionClassMethodWithClamp):

  • bindings/scripts/test/JS/JSTestObj.h:
  • bindings/scripts/test/TestObj.idl:
  • bindings/scripts/test/V8/V8TestObj.cpp:

(WebCore::TestObjV8Internal::classMethodWithClampCallback):
(TestObjV8Internal):

8:18 AM Changeset in webkit [123961] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt] Unreviewed gardening.

  • platform/qt-4.8/Skipped: Unskip now passing tests. (bug85203)
6:54 AM Changeset in webkit [123960] by Philippe Normand
  • 2 edits in trunk/LayoutTests

Unreviewed, GTK rebaseline after r123937.

  • platform/gtk/fast/dom/navigator-detached-no-crash-expected.txt:
6:09 AM Changeset in webkit [123959] by Philippe Normand
  • 2 edits in trunk/Tools

[GTK] Transparent Media controls timeline and panel
https://bugs.webkit.org/show_bug.cgi?id=85279

Reviewed by Martin Robinson.

  • gtk/jhbuild.modules: Provide gnome-themes-standard build

support, this is necessary so the media controls panel colors are
properly initialized from the GTK+ theme.

5:34 AM Changeset in webkit [123958] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WTF

Remove obsolete functions from WTF::Unicode
https://bugs.webkit.org/show_bug.cgi?id=92571

Patch by Patrick Gansterer <Patrick Gansterer> on 2012-07-28
Reviewed by Kentaro Hara.

Remove hasLineBreakingPropertyComplexContextOrIdeographic() and digitValue(),
since they are never used and defined for a few Unicode backends only.

  • wtf/unicode/glib/UnicodeGLib.h:
  • wtf/unicode/icu/UnicodeIcu.h:
12:50 AM EFLWebKit edited by kangil.han@samsung.com
(diff)

Jul 27, 2012:

11:55 PM Changeset in webkit [123957] by pkasting@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

Disable failing test.
https://bugs.webkit.org/show_bug.cgi?id=92318

Unreviewed, making bots happier.

  • tests/WebViewTest.cpp:
11:55 PM Changeset in webkit [123956] by commit-queue@webkit.org
  • 3 edits in trunk/LayoutTests

[WebGL] Align contents of drawbuffer-test.html layout test with the one in the WebGL conformance suite.
https://bugs.webkit.org/show_bug.cgi?id=92375

Patch by Nayan Kumar K <nayankk@motorola.com> on 2012-07-27
Reviewed by Kenneth Russell.

  • fast/canvas/webgl/drawingbuffer-test-expected.txt:
  • fast/canvas/webgl/drawingbuffer-test.html:
11:16 PM Changeset in webkit [123955] by pkasting@chromium.org
  • 2 edits in trunk/LayoutTests

Chromium rebaseline after r123285.
https://bugs.webkit.org/show_bug.cgi?id=90624

Unreviewed, rebaselining.

  • platform/chromium-mac/fast/borders/border-image-rotate-transform-expected.png:
11:11 PM Changeset in webkit [123954] by msaboff@apple.com
  • 2 edits in trunk/LayoutTests

https://bugs.webkit.org/show_bug.cgi?id=92573
Mac Lion: http/tests/misc/submit-post-keygen.html fails on buildbots

  • platform/mac-lion/Skipped:
11:00 PM Changeset in webkit [123953] by commit-queue@webkit.org
  • 8 edits in trunk

[EFL] Support for HTML media capture
https://bugs.webkit.org/show_bug.cgi?id=91842

Patch by Kihong Kwon <kihong.kwon@samsung.com> on 2012-07-27
Reviewed by Ryosuke Niwa.

.:

Add HTML media capture feature, and enable that on the Efl port.

  • Source/cmake/OptionsEfl.cmake:
  • Source/cmake/WebKitFeatures.cmake:
  • Source/cmakeconfig.h.cmake:

Tools:

Add HTML media capture feature to the feature list.
In addition, enable that on the Efl port.

  • Scripts/webkitperl/FeatureList.pm:

LayoutTests:

Remove file-input-capture.html from skipped list.

  • platform/efl/Skipped:
10:48 PM Changeset in webkit [123952] by commit-queue@webkit.org
  • 16 edits
    2 deletes in trunk/Source/WebCore

Unreviewed, rolling out r123908.
http://trac.webkit.org/changeset/123908
https://bugs.webkit.org/show_bug.cgi?id=92572

Broke Chromium Linux compile (Requested by pkasting on
#webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-07-27

  • WebCore.gypi:
  • css/CSSPrimitiveValueMappings.h:

(WebCore::CSSPrimitiveValue::CSSPrimitiveValue):

  • css/CSSValueKeywords.in:
  • css/mediaControls.css:
  • css/mediaControlsChromiumAndroid.css:

(audio::-webkit-media-controls-play-button, video::-webkit-media-controls-play-button):

  • html/shadow/MediaControlElements.cpp:

(WebCore):

  • html/shadow/MediaControlElements.h:

(WebCore):

  • html/shadow/MediaControlRootElementChromium.cpp:

(WebCore::MediaControlChromiumEnclosureElement::create):
(WebCore):
(WebCore::MediaControlChromiumEnclosureElement::shadowPseudoId):
(WebCore::MediaControls::create):
(WebCore::MediaControlRootElementChromium::create):

  • html/shadow/MediaControlRootElementChromium.h:

(MediaControlChromiumEnclosureElement):
(MediaControlRootElementChromium):

  • html/shadow/MediaControlRootElementChromiumAndroid.cpp: Removed.
  • html/shadow/MediaControlRootElementChromiumAndroid.h: Removed.
  • platform/ThemeTypes.h:
  • rendering/RenderMediaControlsChromium.cpp:

(WebCore::RenderMediaControlsChromium::paintMediaControlsPart):

  • rendering/RenderTheme.cpp:

(WebCore::RenderTheme::paint):

  • rendering/RenderTheme.h:
  • rendering/RenderThemeChromiumAndroid.cpp:
  • rendering/RenderThemeChromiumAndroid.h:
10:12 PM Changeset in webkit [123951] by pkasting@chromium.org
  • 2 edits in trunk/LayoutTests

Mark three tests as flakily timing out.
https://bugs.webkit.org/show_bug.cgi?id=92570

Unreviewed, Chromium test expectations update.

  • platform/chromium/TestExpectations:
9:30 PM Changeset in webkit [123950] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebCore

[BlackBerry] Use new LocaleHandler.
https://bugs.webkit.org/show_bug.cgi?id=92548

Patch by Eli Fidler <efidler@rim.com> on 2012-07-27
Reviewed by Yong Li.

RIM PR 128381

  • platform/blackberry/LocalizedStringsBlackBerry.cpp:

(WebCore::platformLanguage):

  • platform/network/blackberry/ResourceRequestBlackBerry.cpp:

(WebCore::ResourceRequest::initializePlatformRequest):

  • platform/text/blackberry/TextBreakIteratorInternalICUBlackBerry.cpp:

(WebCore::currentSearchLocaleID):
(WebCore::currentTextBreakLocaleID):

9:29 PM Changeset in webkit [123949] by pkasting@chromium.org
  • 2 edits in trunk/LayoutTests

Fix broken script reference after test move.
https://bugs.webkit.org/show_bug.cgi?id=92111

Unreviewed, fixing bad test move.

  • platform/win/fast/events/panScroll-click-hyperlink.html:
9:20 PM Changeset in webkit [123948] by pkasting@chromium.org
  • 1 edit
    8 adds in trunk/LayoutTests

Chromium rebaselines after r123912.
https://bugs.webkit.org/show_bug.cgi?id=92327

Unreviewed, rebaselining.

  • platform/chromium-linux-x86/fast/hidpi: Added.
  • platform/chromium-linux-x86/fast/hidpi/gradient-with-scaled-ancestor-expected.png: Added.
  • platform/chromium-linux/fast/hidpi/gradient-with-scaled-ancestor-expected.png: Added.
  • platform/chromium-mac-snowleopard/fast/hidpi/gradient-with-scaled-ancestor-expected.png: Added.
  • platform/chromium-mac/fast/hidpi/gradient-with-scaled-ancestor-expected.png: Added.
  • platform/chromium-win-xp/fast/hidpi: Added.
  • platform/chromium-win-xp/fast/hidpi/gradient-with-scaled-ancestor-expected.png: Added.
  • platform/chromium-win/fast/hidpi/gradient-with-scaled-ancestor-expected.png: Added.
9:12 PM Changeset in webkit [123947] by pkasting@chromium.org
  • 2 edits in trunk/LayoutTests

Chromium rebaseline after r123706.
https://bugs.webkit.org/show_bug.cgi?id=92236

Unreviewed, rebaselining.

  • platform/chromium-win/fast/dom/HTMLProgressElement/progress-element-expected.png:
8:19 PM Changeset in webkit [123946] by pkasting@chromium.org
  • 1 edit
    10 adds in trunk/LayoutTests

Chromium rebaselines after r123790.
https://bugs.webkit.org/show_bug.cgi?id=92059

Unreviewed, rebaselining.

  • platform/chromium-linux-x86/css3/filters/effect-brightness-clamping-expected.txt: Added.
  • platform/chromium-linux-x86/css3/filters/effect-brightness-clamping-hw-expected.txt: Added.
  • platform/chromium-linux/css3/filters/effect-brightness-clamping-expected.txt: Added.
  • platform/chromium-linux/css3/filters/effect-brightness-clamping-hw-expected.txt: Added.
  • platform/chromium-win-xp/css3: Added.
  • platform/chromium-win-xp/css3/filters: Added.
  • platform/chromium-win-xp/css3/filters/effect-brightness-clamping-expected.txt: Added.
  • platform/chromium-win-xp/css3/filters/effect-brightness-clamping-hw-expected.txt: Added.
  • platform/chromium-win/css3/filters/effect-brightness-clamping-expected.txt: Added.
  • platform/chromium-win/css3/filters/effect-brightness-clamping-hw-expected.txt: Added.
6:23 PM Changeset in webkit [123945] by commit-queue@webkit.org
  • 32 edits in trunk/Source/WebCore

Change constructor with explicit constructor in WebCore/Modules
https://bugs.webkit.org/show_bug.cgi?id=92529

Patch by Gyuyoung Kim <gyuyoung.kim@samsung.com> on 2012-07-27
Reviewed by Kentaro Hara.

As a step to let *explicit* keyword add to constructors which have a parameter,
*explicit* keyword is added to Source/WebCore/Modules in order to avoid implicit
type conversion.

No new tests. Covered by existing tests.

  • Modules/filesystem/LocalFileSystem.h:

(WebCore::LocalFileSystem::LocalFileSystem):

  • Modules/filesystem/Metadata.h:

(WebCore::Metadata::Metadata):

  • Modules/filesystem/SyncCallbackHelper.h:

(WebCore::SyncCallbackHelper::SuccessCallbackImpl::SuccessCallbackImpl):
(WebCore::SyncCallbackHelper::ErrorCallbackImpl::ErrorCallbackImpl):

  • Modules/mediastream/IceCandidate.h:

(IceCandidate):

  • Modules/mediastream/NavigatorUserMediaError.h:

(WebCore::NavigatorUserMediaError::NavigatorUserMediaError):

  • Modules/mediastream/SessionDescription.h:

(SessionDescription):

  • Modules/mediastream/UserMediaController.h:

(UserMediaController):

  • Modules/notifications/NotificationController.h:

(NotificationController):

  • Modules/speech/SpeechRecognition.h:

(SpeechRecognition):

  • Modules/speech/SpeechRecognitionController.h:

(SpeechRecognitionController):

  • Modules/speech/SpeechRecognitionEvent.h:

(SpeechRecognitionEvent):

  • Modules/speech/SpeechRecognitionResultList.h:

(SpeechRecognitionResultList):

  • Modules/vibration/Vibration.h:

(Vibration):

  • Modules/webaudio/AudioBuffer.h:

(AudioBuffer):

  • Modules/webaudio/AudioContext.h:

(AudioContext):

  • Modules/webaudio/AudioNodeInput.h:

(AudioNodeInput):

  • Modules/webaudio/AudioSummingJunction.h:

(AudioSummingJunction):

  • Modules/webaudio/BiquadDSPKernel.h:

(WebCore::BiquadDSPKernel::BiquadDSPKernel):

  • Modules/webaudio/DefaultAudioDestinationNode.h:

(DefaultAudioDestinationNode):

  • Modules/webaudio/DelayDSPKernel.h:

(DelayDSPKernel):

  • Modules/webaudio/OfflineAudioCompletionEvent.h:

(OfflineAudioCompletionEvent):

  • Modules/webaudio/WaveShaperDSPKernel.h:

(WebCore::WaveShaperDSPKernel::WaveShaperDSPKernel):

  • Modules/webaudio/WaveShaperNode.h:

(WaveShaperNode):

  • Modules/webaudio/WaveTable.h:

(WaveTable):

  • Modules/webdatabase/DatabaseAuthorizer.h:

(DatabaseAuthorizer):

  • Modules/webdatabase/DatabaseTask.h:

(Database::DatabaseTransactionTask):

  • Modules/webdatabase/DatabaseTracker.h:

(DatabaseTracker):

  • Modules/webdatabase/SQLCallbackWrapper.h:

(WebCore::SQLCallbackWrapper::SafeReleaseTask::SafeReleaseTask):

  • Modules/webdatabase/SQLException.h:

(WebCore::SQLException::SQLException):

  • Modules/websockets/WebSocket.h:
  • Modules/websockets/WebSocketExtensionProcessor.h:

(WebCore::WebSocketExtensionProcessor::WebSocketExtensionProcessor):

6:21 PM Changeset in webkit [123944] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

[V8] Optimize Element::getAttributeNS() by replacing String with AtomicString
https://bugs.webkit.org/show_bug.cgi?id=90276

Patch by Kentaro Hara <haraken@chromium.org> on 2012-07-27
Reviewed by Adam Barth.

In r121439, we changed the argument type of Element::getAttribute()
from String to AtomicString for optimization. This patch changes the
argument type of Element::getAttributeNS() from String to AtomicString.

Actually this change affects V8 performance only.

[JSC without the patch]
The binding code converts a given JavaScript string to a String
-> the String is passed to getAttributeNS()
-> the String is converted to an AtomicString when it is passed to
QualifiedName() constructor.

[JSC with the patch]
The binding code converts a given JavaScript string to a String
-> the String is converted to an AtomicString when it is passed to
getAttributeNS()
-> the AtomicString is passed to QualifiedName() constructor.

[V8 without the patch]
The binding code retrieves a cached external String for a given
JavaScript string
-> the String is passed to getAttributeNS()
-> the String is converted to AtomicString when it is passed to
QualifiedName() constructor.

[V8 with the patch]
The binding code retrieves a cached external AtomicString for a given
JavaScript string
-> the AtomicString is passed to getAttributeNS()
-> the AtomicString is passed to QualifiedName() constructor.

In this way, in JSC, the change affects where a String is converted to
an AtomicString. On the other hand, in V8, (since the AtomicString is already
cached,) we can skip the conversion step.

Performance test: https://bugs.webkit.org/attachment.cgi?id=150158

getAttributeNS (Chromium/Linux):
577.80ms => 454.60ms

  • dom/Element.cpp:

(WebCore::Element::getAttributeNS):

  • dom/Element.h:

(Element):

6:02 PM Changeset in webkit [123943] by commit-queue@webkit.org
  • 12 edits in trunk/Source

Unreviewed, rolling out r123679.
http://trac.webkit.org/changeset/123679
https://bugs.webkit.org/show_bug.cgi?id=92565

Slowed down HTML parsing by 3.6% (Requested by abarth on
#webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-07-27

Source/WebCore:

  • html/FTPDirectoryDocument.cpp:

(WebCore::FTPDirectoryDocumentParser::append):

  • html/parser/HTMLEntityParser.cpp:

(WebCore::HTMLEntityParser::consumeNamedEntity):

  • html/parser/HTMLParserIdioms.cpp:

(WebCore::stripLeadingAndTrailingHTMLSpaces):

  • html/parser/HTMLSourceTracker.cpp:

(WebCore::HTMLSourceTracker::sourceForToken):

  • loader/cache/CachedScript.cpp:

(WebCore::CachedScript::script):

  • platform/text/SegmentedString.cpp:

(WebCore::SegmentedString::SegmentedString):
(WebCore::SegmentedString::operator=):
(WebCore::SegmentedString::clear):
(WebCore::SegmentedString::append):
(WebCore::SegmentedString::prepend):
(WebCore::SegmentedString::advanceSubstring):
(WebCore::SegmentedString::advance):
(WebCore::SegmentedString::advanceSlowCase):
(WebCore::SegmentedString::advanceAndUpdateLineNumberSlowCase):

  • platform/text/SegmentedString.h:

(WebCore::SegmentedSubstring::SegmentedSubstring):
(WebCore::SegmentedSubstring::clear):
(WebCore::SegmentedSubstring::appendTo):
(SegmentedSubstring):
(WebCore::SegmentedString::SegmentedString):
(WebCore::SegmentedString::push):
(WebCore::SegmentedString::isEmpty):
(WebCore::SegmentedString::lookAhead):
(WebCore::SegmentedString::lookAheadIgnoringCase):
(WebCore::SegmentedString::advance):
(WebCore::SegmentedString::advanceAndASSERT):
(WebCore::SegmentedString::advanceAndASSERTIgnoringCase):
(WebCore::SegmentedString::advancePastNewlineAndUpdateLineNumber):
(SegmentedString):
(WebCore::SegmentedString::advancePastNonNewline):
(WebCore::SegmentedString::advanceAndUpdateLineNumber):
(WebCore::SegmentedString::operator*):
(WebCore::SegmentedString::operator->):
(WebCore::SegmentedString::current):
(WebCore::SegmentedString::equalsLiterally):
(WebCore::SegmentedString::equalsIgnoringCase):
(WebCore::SegmentedString::lookAheadInline):
(WebCore::SegmentedString::lookAheadSlowCase):

  • xml/parser/CharacterReferenceParserInlineMethods.h:

(WebCore::consumeCharacterReference):

  • xml/parser/MarkupTokenizerBase.h:

(WebCore::MarkupTokenizerBase::InputStreamPreprocessor::nextInputCharacter):
(WebCore::MarkupTokenizerBase::InputStreamPreprocessor::peek):
(WebCore::MarkupTokenizerBase::InputStreamPreprocessor::advance):

Source/WTF:

  • wtf/text/WTFString.h:
5:41 PM Changeset in webkit [123942] by jer.noble@apple.com
  • 4 edits in trunk/Source/WebCore

Reset the set of "seen" plugins when the main frame load is committed.
https://bugs.webkit.org/show_bug.cgi?id=92564

Reviewed by Anders Carlsson.

Because the Page object is re-used across navigation and reload, reset the list
of seen plugins when the main frame load commits. This gives a good baseline to
compare against the number of pages loaded.

No new tests; the "seen" plugin list is for diagnostic purposes only.

  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::dispatchDidCommitLoad):

  • page/Page.cpp:

(WebCore::Page::resetSeenPlugins):

  • page/Page.h:
5:34 PM Changeset in webkit [123941] by dpranke@chromium.org
  • 3 edits in trunk/Tools

"webkit-patch rebaseline <test>" fails with exit_code 1
https://bugs.webkit.org/show_bug.cgi?id=91560

Reviewed by Adam Barth.

webkit-patch rebaseline-json crashes in an svn checkout if it
tries to add files and there are no files to add; arguably
svn.add() should be robust against this, but rebaseline-json
shouldn't be calling it, either. This patch fixes the latter and
adds better debugging info to garden-o-matic so we can see what
caused the crash.

  • Scripts/webkitpy/tool/commands/rebaseline.py:

(AbstractParallelRebaselineCommand._rebaseline):

  • Scripts/webkitpy/tool/servers/gardeningserver.py:

(GardeningHTTPRequestHandler.rebaselineall):
(GardeningHTTPRequestHandler.rebaselineall.error_handler):
(GardeningHTTPRequestHandler):

5:32 PM Changeset in webkit [123940] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk

execCommand copies the backgroung-color of the enclosing element to the element being edited.
https://bugs.webkit.org/show_bug.cgi?id=91783

Patch by Sukolsak Sakshuwong <sukolsak@google.com> on 2012-07-27
Reviewed by Ryosuke Niwa.

Source/WebCore:

InsertListCommand and some other edit commands remove redundant
style properties after their DOM modification by using
StylePropertySet::removeEquivalentProperties. However, this method
does not remove redundant background-color properties such as
"background-color: white" and "background-color: #fff" because
it compares them using cssText and thus treats them as different.
Add code to convert background colors to RGBA and then compare them.

Test: editing/style/redundant-background-color.html

  • editing/EditingStyle.cpp:

(WebCore::getRGBABackgroundColor):
(WebCore):
(WebCore::getPropertiesNotIn):

LayoutTests:

This tests to make sure that WebKit removes redundant "background-color"
property after execCommand.

  • editing/style/redundant-background-color-expected.txt: Added.
  • editing/style/redundant-background-color.html: Added.
5:30 PM Changeset in webkit [123939] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

[EFL] Return 0 when network is offline
https://bugs.webkit.org/show_bug.cgi?id=92454

Patch by Gyuyoung Kim <gyuyoung.kim@samsung.com> on 2012-07-27
Reviewed by Kenneth Rohde Christiansen.

Network Info latest spec says 0 is returned when network is offline.
r123769 modified to return infinity value when network is offiline.

  • platform/efl/NetworkInfoProviderEfl.cpp:

(WebCore::NetworkInfoProviderEfl::bandwidth):

5:08 PM Changeset in webkit [123938] by rniwa@webkit.org
  • 5 edits in trunk

Unreviewed perf-o-matic build fix for dashboard image generation.
Also, only show Parser/html5-full-render on the dashboard by default
so that it won't take forever to load it.

  • Websites/webkit-perf.appspot.com/controller.py:

(schedule_runs_update):
(RunsChartHandler.post):

  • Websites/webkit-perf.appspot.com/models.py:

(Runs.chart_params):
(DashboardImage.get_image):

  • Websites/webkit-perf.appspot.com/models_unittest.py:

(RunsTest.test_chart_params_with_value):

4:51 PM Changeset in webkit [123937] by commit-queue@webkit.org
  • 13 edits in trunk

Change navigator.webkitGamepads[] to navigator.webkitGetGamepads()
https://bugs.webkit.org/show_bug.cgi?id=92533

Patch by Scott Graham <scottmg@chromium.org> on 2012-07-27
Reviewed by Adam Barth.

Source/WebCore:

Per discussion on public-webapps. Pages that enumerate navigator were
causing background resources to be created to poll hardware
unintentionally. By changing to an explicit function call, this is
avoided (and the API becomes more clear too). This change updates the
implementation to match the current ED spec.

  • Modules/gamepad/NavigatorGamepad.cpp:

(WebCore::NavigatorGamepad::webkitGetGamepads):

  • Modules/gamepad/NavigatorGamepad.h:

(NavigatorGamepad):

  • Modules/gamepad/NavigatorGamepad.idl:
  • bindings/generic/RuntimeEnabledFeatures.h:

(WebCore::RuntimeEnabledFeatures::setWebkitGetGamepadsEnabled):
(WebCore::RuntimeEnabledFeatures::webkitGetGamepadsEnabled):

Source/WebKit/chromium:

Renames for updated access point.

  • src/WebRuntimeFeatures.cpp:

(WebKit::WebRuntimeFeatures::enableGamepad):
(WebKit::WebRuntimeFeatures::isGamepadEnabled):

LayoutTests:

Updates for renamed access point.

  • gamepad/gamepad-api-expected.txt:
  • gamepad/gamepad-api.html:
  • gamepad/gamepad-polling-access-expected.txt:
  • gamepad/gamepad-polling-access.html:
  • platform/chromium/fast/dom/navigator-detached-no-crash-expected.txt:
4:44 PM Changeset in webkit [123936] by beidson@apple.com
  • 14 edits
    3 adds in trunk

Plugins should not be allowed to override standard properties/attributes in non-standard worlds
<rdar://problem/11975252> and https://bugs.webkit.org/show_bug.cgi?id=92519

Reviewed by Anders Carlsson.

Source/WebCore:

Change the 3 plugin-owning element's custom bindings to prefer built-in properties if they exist.
When they do they don't give the plugin a chance to override.

Test: plugins/npruntime/overrides-all-properties.html

Add plugin custom functions to prefer built-in properties over plugin scriptable object properties:

  • bindings/js/JSPluginElementFunctions.h:

(WebCore::pluginElementCustomGetOwnPropertySlot):
(WebCore::pluginElementCustomGetOwnPropertyDescriptor):

Use those new custom functions for getting properties:

  • bindings/js/JSHTMLAppletElementCustom.cpp:

(WebCore::JSHTMLAppletElement::getOwnPropertySlotDelegate):
(WebCore::JSHTMLAppletElement::getOwnPropertyDescriptorDelegate):

  • bindings/js/JSHTMLEmbedElementCustom.cpp:

(WebCore::JSHTMLEmbedElement::getOwnPropertySlotDelegate):
(WebCore::JSHTMLEmbedElement::getOwnPropertyDescriptorDelegate):

  • bindings/js/JSHTMLObjectElementCustom.cpp:

(WebCore::JSHTMLObjectElement::getOwnPropertySlotDelegate):
(WebCore::JSHTMLObjectElement::getOwnPropertyDescriptorDelegate):

Tools:

Add a test plugin which overrides all properties it is asked about:

  • DumpRenderTree/TestNetscapePlugIn/Tests/PluginScriptableObjectOverridesAllProperties.cpp: Added.

(PluginScriptableObjectOverridesAllProperties):
(PluginScriptableObjectOverridesAllProperties::PluginScriptableObjectOverridesAllProperties):
(PluginObject):
(PluginScriptableObjectOverridesAllProperties::PluginObject::PluginObject):
(PluginScriptableObjectOverridesAllProperties::PluginObject::~PluginObject):
(PluginScriptableObjectOverridesAllProperties::PluginObject::hasProperty):
(PluginScriptableObjectOverridesAllProperties::PluginObject::getProperty):
(PluginScriptableObjectOverridesAllProperties::NPP_GetValue):

Expose NPN_MemAlloc to the test plugin:

  • DumpRenderTree/TestNetscapePlugIn/PluginTest.cpp:

(PluginTest::NPN_MemAlloc):

  • DumpRenderTree/TestNetscapePlugIn/PluginTest.h:

(PluginTest):

  • DumpRenderTree/DumpRenderTree.gypi:
  • DumpRenderTree/DumpRenderTree.xcodeproj/project.pbxproj:
  • DumpRenderTree/TestNetscapePlugIn/win/TestNetscapePlugin.vcproj:
  • DumpRenderTree/qt/TestNetscapePlugin/TestNetscapePlugin.pro:

LayoutTests:

  • plugins/npruntime/overrides-all-properties-expected.txt: Added.
  • plugins/npruntime/overrides-all-properties.html: Added.
  • platform/chromium/TestExpectations: Test behaves different in V8, so skip on chromium
4:32 PM Changeset in webkit [123935] by commit-queue@webkit.org
  • 14 edits
    3 adds in trunk

use createUninitialized when creating TypedArray from another array
https://bugs.webkit.org/show_bug.cgi?id=92518

Patch by Arnaud Renevier <a.renevier@sisa.samsung.com> on 2012-07-27
Reviewed by Kenneth Russell.

PerformanceTests:

  • Bindings/typed-array-construct-from-array.html: Added.

Source/WebCore:

When creating a Typed Array from another array, we fill it with values
from the source array. So, we can create it uninitialized. This is
make constructor about 35% faster when constructed with another typed
array.

Test: fast/canvas/webgl/array-constructor.html

  • bindings/js/JSArrayBufferViewHelper.h:

(WebCore::constructArrayBufferViewWithTypedArrayArgument):
(WebCore::constructArrayBufferView):

Source/WTF:

Expose a createUninitialized static method on TypedArray classes.

  • wtf/Float32Array.h:

(Float32Array):
(WTF::Float32Array::createUninitialized):
(WTF):

  • wtf/Float64Array.h:

(Float64Array):
(WTF::Float64Array::createUninitialized):
(WTF):

  • wtf/Int16Array.h:

(Int16Array):
(WTF::Int16Array::createUninitialized):
(WTF):

  • wtf/Int32Array.h:

(Int32Array):
(WTF::Int32Array::createUninitialized):
(WTF):

  • wtf/Int8Array.h:

(Int8Array):
(WTF::Int8Array::createUninitialized):
(WTF):

  • wtf/Uint16Array.h:

(Uint16Array):
(WTF::Uint16Array::createUninitialized):
(WTF):

  • wtf/Uint32Array.h:

(Uint32Array):
(WTF::Uint32Array::createUninitialized):
(WTF):

  • wtf/Uint8Array.h:

(Uint8Array):
(WTF::Uint8Array::createUninitialized):
(WTF):

  • wtf/Uint8ClampedArray.h:

(Uint8ClampedArray):

LayoutTests:

Add test to check that constructing a Typed Array with nan values does
not result in random values.

  • fast/canvas/webgl/array-constructor-expected.txt: Added.
  • fast/canvas/webgl/array-constructor.html: Added.
4:29 PM Changeset in webkit [123934] by commit-queue@webkit.org
  • 4 edits in trunk/Source

[BlackBerry] Adapt to change in the FontInfo platform API.
https://bugs.webkit.org/show_bug.cgi?id=92547

Patch by Eli Fidler <efidler@rim.com> on 2012-07-27
Reviewed by Yong Li

RIM PR 161263

Source/WebCore:

  • page/blackberry/SettingsBlackBerry.cpp:

(WebCore::Settings::initializeDefaultFontFamilies):

Source/WebKit/blackberry:

  • Api/WebSettings.cpp:

(BlackBerry::WebKit::WebSettings::standardSettings):

4:12 PM Changeset in webkit [123933] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit2

[EFL][WK2] Assertion hit in ewk_cookie_manager.cpp
https://bugs.webkit.org/show_bug.cgi?id=92544

Patch by Christophe Dumez <Christophe Dumez> on 2012-07-27
Reviewed by Kenneth Rohde Christiansen.

Removed wrong assertion in ewk_cookie_manager.cpp
and replace it by an if check. The assertion was
sometimes hit by our unit tests.

  • UIProcess/API/efl/ewk_cookie_manager.cpp:

(cookiesDidChange):

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

[chromium] Fix color matrix filter computation so that filters that don't need clamping are correctly detected as such
https://bugs.webkit.org/show_bug.cgi?id=92452

Patch by Antoine Labour <piman@chromium.org> on 2012-07-27
Reviewed by Stephen White.

Because of precision issues, some filters were returning true to
matrixNeedsClamping when they can return false. This fixes it.

No new tests in this CL, a follow-up change refactors the code to add
several tests.

  • platform/graphics/chromium/cc/CCRenderSurfaceFilters.cpp:
3:59 PM Changeset in webkit [123931] by mhahnenberg@apple.com
  • 3 edits in trunk/Source/JavaScriptCore

Split functionality of MarkedAllocator::m_currentBlock
https://bugs.webkit.org/show_bug.cgi?id=92550

Reviewed by Filip Pizlo.

MarkedAllocator::m_currentBlock serves two purposes right now; it indicates the block that is currently
being used for allocation and the beginning of the list of blocks that need to be swept. We should split
these two functionalities into two separate fields.

  • heap/MarkedAllocator.cpp:

(JSC::MarkedAllocator::tryAllocateHelper): Use m_blocksToSweep instead of m_currentBlock as the
initializer/reference of the loop. Only change m_currentBlock when we know what the result will be.
(JSC::MarkedAllocator::addBlock): When we add a new block we know that both m_blocksToSweep and
m_currentBlock are null. In order to preserve the invariant that m_currentBlock <= m_blocksToSweep,
we assign both of them to point to the new block.
(JSC::MarkedAllocator::removeBlock): We need a separate check to see if the block we're removing is
m_blocksToSweep and if so, advance it to the next block in the list.

  • heap/MarkedAllocator.h:

(MarkedAllocator): Initialize m_blocksToSweep.
(JSC::MarkedAllocator::MarkedAllocator):
(JSC::MarkedAllocator::reset): We set m_blocksToSweep to be the head of our list. This function is called
at the end of a collection, so all of the blocks in our allocator need to be swept. We need to sweep a
block before we can start allocating, so m_currentBlock is set to null. We also set the freeList to
the empty FreeList to emphasize the fact that we can't start allocating until we do some sweeping.

3:58 PM Changeset in webkit [123930] by jer.noble@apple.com
  • 8 edits in trunk/Source/WebCore

Add diagnostic logging for plugins-per-page.
https://bugs.webkit.org/show_bug.cgi?id=92538

Reviewed by Anders Carlsson.

Add some diagnostic logging for whether a page has seen a plugin, and
whether a page has seen a plugin of a specific type.

Move the diagnostic logging out of the elemements themselves:

  • html/HTMLEmbedElement.cpp:

(WebCore::HTMLEmbedElement::updateWidget):

  • html/HTMLObjectElement.cpp:

(WebCore::HTMLObjectElement::updateWidget):

Instead, log when the plugin is requested, thereby catching plugins which are
rejected because, e.g., Java is disabled or not installed:

  • loader/SubframeLoader.cpp:

(WebCore::logPluginRequest):
(WebCore::SubframeLoader::requestObject):
(WebCore::SubframeLoader::createJavaAppletWidget):

Add new diagnostic key values:

  • page/DiagnosticLoggingKeys.cpp:

(WebCore::DiagnosticLoggingKeys::pageContainsPluginKey):
(WebCore::DiagnosticLoggingKeys::pageContainsAtLeastOnePluginKey):

  • page/DiagnosticLoggingKeys.h:

Add a map of plugin types seen per-page for diagnostic purposes:

  • page/Page.cpp:

(WebCore::Page::hasSeenAnyPlugin):
(WebCore::Page::hasSeenPlugin):
(WebCore::Page::sawPlugin):

  • page/Page.h:
3:46 PM Changeset in webkit [123929] by commit-queue@webkit.org
  • 4 edits in trunk/Source

[WINCE] Use macros from ICU instead of defining the same functionality again
https://bugs.webkit.org/show_bug.cgi?id=92530

Patch by Patrick Gansterer <Patrick Gansterer> on 2012-07-27
Reviewed by Ryosuke Niwa.

Replace isHighSurrogate() with U16_IS_LEAD(), isLowSurrogate() with
U16_IS_TRAIL() and surrogateToUcs4() with U16_GET_SUPPLEMENTARY().

Source/WebCore:

  • html/FTPDirectoryDocument.cpp:

(WebCore::processFileDateString):

  • loader/archive/mhtml/MHTMLArchive.cpp:

(WebCore::MHTMLArchive::generateMHTMLData):

  • platform/graphics/wince/FontWinCE.cpp:

(WebCore::generateComponents):

Source/WTF:

  • wtf/unicode/wince/UnicodeWinCE.h:
3:41 PM Changeset in webkit [123928] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Build warning in CSSPrimitiveValueMappings.h when CSS_STICKY_POSITION is disabled
https://bugs.webkit.org/show_bug.cgi?id=92523

Patch by Christophe Dumez <Christophe Dumez> on 2012-07-27
Reviewed by Simon Fraser.

Move #ifdef in CSSPrimitiveValueMappings.h to silent compilation warning
when CSS_STICKY_POSITION is disabled.

No new tests, no behavior change.

  • css/CSSPrimitiveValueMappings.h:

(WebCore::CSSPrimitiveValue::CSSPrimitiveValue):

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

DragController should ascertain dragClient to be non-null
https://bugs.webkit.org/show_bug.cgi?id=92474

Patch by Vivek Galatage <vivekgalatage@gmail.com> on 2012-07-27
Reviewed by Tony Chang.

The DragController must always ensure a valid dragClient object. Hence
adding the ASSERT to assure that.

No new tests. Added assertion for PageClient.dragClient.

  • page/DragController.cpp:

(WebCore::DragController::DragController):

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

[chromium] Clean up LayerRendererChromium::drawTexturedQuad
https://bugs.webkit.org/show_bug.cgi?id=91823

Patch by Adrienne Walker <enne@google.com> on 2012-07-27
Reviewed by James Robinson.

LayerRendererChromium::drawTexturedQuad has been around forever and
over time has developed several problems:

  • Confusingly named relative to drawTextureQuad
  • All but one callers identically manipulate the transform matrix
  • Lots of optional parameters that only few functions use

This patch breaks up drawTexturedQuad into setShaderOpacity (for
optional opacity values sent to the shader), setShaderFloatQuad (the
full FloatQuad used by AA shaders), and drawQuadGeometry (actually
draw the geometry, and wrap the transform logic that was duplicated
everywhere).

No new tests; no change in functionality.

  • platform/graphics/chromium/LayerRendererChromium.cpp:

(WebCore::LayerRendererChromium::drawCheckerboardQuad):
(WebCore::LayerRendererChromium::drawBackgroundFilters):
(WebCore::LayerRendererChromium::drawRenderPassQuad):
(WebCore::LayerRendererChromium::drawSolidColorQuad):
(WebCore::LayerRendererChromium::drawTileQuad):
(WebCore::LayerRendererChromium::drawYUVVideoQuad):
(WebCore::LayerRendererChromium::drawStreamVideoQuad):
(WebCore::LayerRendererChromium::drawTextureQuad):
(WebCore::LayerRendererChromium::drawIOSurfaceQuad):
(WebCore::LayerRendererChromium::drawHeadsUpDisplay):
(WebCore::LayerRendererChromium::setShaderFloatQuad):
(WebCore::LayerRendererChromium::setShaderOpacity):
(WebCore::LayerRendererChromium::drawQuadGeometry):
(WebCore::LayerRendererChromium::copyTextureToFramebuffer):

  • platform/graphics/chromium/LayerRendererChromium.h:

(LayerRendererChromium):

3:20 PM Changeset in webkit [123925] by caio.oliveira@openbossa.org
  • 2 edits in trunk/Source/WTF

[Qt] Fix build after r123917
https://bugs.webkit.org/show_bug.cgi?id=92555

Unreviewed.

  • wtf/GregorianDateTime.h: Needs string.h for memset().
3:10 PM Changeset in webkit [123924] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

"webkit-patch upload" should strip colors from patch if necessary
https://bugs.webkit.org/show_bug.cgi?id=92306

Patch by Rafael Brandao <rafael.lobo@openbossa.org> on 2012-07-27
Reviewed by Adam Barth.

  • Scripts/webkitpy/common/checkout/scm/git.py: Explicity remove colors from diff.
3:07 PM Changeset in webkit [123923] by caio.oliveira@openbossa.org
  • 26 edits
    2 deletes in trunk/LayoutTests

[Qt] Remaining rebaselines after new test fonts
https://bugs.webkit.org/show_bug.cgi?id=92552

Unreviewed.

Unskip a bunch of tests, organize the remaining skips. The mentions to bug 85203 were
removed now. A last batch of WK1 results were associated with the bug 91610.

  • platform/qt-5.0-wk1/Skipped:
  • platform/qt-5.0-wk2/Skipped:
  • platform/qt-5.0-wk2/fullscreen/full-screen-placeholder-expected.txt:
  • platform/qt-5.0/Skipped:
  • platform/qt/editing/input/caret-at-the-edge-of-input-expected.txt:
  • platform/qt/editing/input/reveal-caret-of-multiline-input-expected.txt:
  • platform/qt/http/tests/cache/cancel-during-revalidation-succeeded-expected.txt: Removed.
  • platform/qt/http/tests/misc/object-embedding-svg-delayed-size-negotiation-2-expected.txt:
  • platform/qt/http/tests/security/xss-DENIED-xsl-document-securityOrigin-expected.txt: Removed.
  • platform/qt/svg/text/select-textLength-spacing-squeeze-1-expected.txt:
  • platform/qt/svg/text/select-textLength-spacing-squeeze-2-expected.txt:
  • platform/qt/svg/text/select-textLength-spacing-squeeze-3-expected.txt:
  • platform/qt/svg/text/select-textLength-spacing-squeeze-4-expected.txt:
  • platform/qt/svg/text/select-textLength-spacing-stretch-1-expected.txt:
  • platform/qt/svg/text/select-textLength-spacing-stretch-2-expected.txt:
  • platform/qt/svg/text/select-textLength-spacing-stretch-3-expected.txt:
  • platform/qt/svg/text/select-textLength-spacing-stretch-4-expected.txt:
  • platform/qt/svg/text/select-textLength-spacingAndGlyphs-stretch-4-expected.txt:
  • platform/qt/svg/text/select-x-list-1-expected.txt:
  • platform/qt/svg/text/select-x-list-2-expected.txt:
  • platform/qt/svg/text/select-x-list-3-expected.txt:
  • platform/qt/svg/text/select-x-list-4-expected.txt:
  • platform/qt/svg/text/select-x-list-with-tspans-1-expected.txt:
  • platform/qt/svg/text/select-x-list-with-tspans-2-expected.txt:
  • platform/qt/svg/text/select-x-list-with-tspans-3-expected.txt:
  • platform/qt/svg/text/select-x-list-with-tspans-4-expected.txt:
  • platform/qt/svg/text/selection-doubleclick-expected.txt:
3:07 PM Changeset in webkit [123922] by Lucas Forschler
  • 3 edits
    2 copies in branches/safari-536.26-branch

Merged r116357. <rdar://problem/11942044>

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

[chromium] CCLayerTreeHost placeholder scissor should be deviceViewportSize instead of viewportSize
https://bugs.webkit.org/show_bug.cgi?id=92469

Patch by Shawn Singh <shawnsingh@chromium.org> on 2012-07-27
Reviewed by Adrienne Walker.

We were accidentally using viewportSize instead of
deviceViewportSize in a particular place in code. Recent
refactoring to use tighter scissor rects exposed this problem.

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

(WebCore::CCLayerTreeHost::updateLayers):

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

clean-pending-commit throws exception when encountering Ossy
https://bugs.webkit.org/show_bug.cgi?id=92534

Patch by Eric Seidel <eric@webkit.org> on 2012-07-27
Reviewed by Adam Barth.

Mark a couple strings as unicode so that reviewers with unicode names do not cause
clean-pending-commit to barf.

  • Scripts/webkitpy/tool/commands/upload.py:

(CleanPendingCommit._flags_to_clear_on_patch):
(CleanPendingCommit.execute):
(AssignToCommitter._assign_bug_to_last_patch_attacher):

2:49 PM Changeset in webkit [123919] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk

Search cancel button is hard to activate with a tap gesture even if touch adjustment is enabled.
https://bugs.webkit.org/show_bug.cgi?id=91894

Patch by Kevin Ellis <kevers@chromium.org> on 2012-07-27
Reviewed by Antonio Gomes.

Source/WebCore:

Update check for determining if a node responds to tap gestures.
Previously, the cancel button was being discarded as a candidate for
touch adjustment if the touch area clipped the text input region
because the candidate pruning process failed to account for the cancel
button having a default mouse handler. Possibly an interim fix,
depending on resolution to 92093.

Test: touchadjustment/search-cancel.html

  • page/TouchAdjustment.cpp:

(TouchAdjustment):
(WebCore::TouchAdjustment::nodeRespondsToTapGesture):

LayoutTests:

Add test for touch adjustment in the vicinity of a search cancel button.
Prior to the patch, the cancel button was pruned as a candidate if the
touch area clipped the input field.

  • touchadjustment/search-cancel-expected.txt: Added.
  • touchadjustment/search-cancel.html: Added.
2:48 PM Changeset in webkit [123918] by dpranke@chromium.org
  • 1 edit
    1 add in trunk/Tools

chromium win compile broken after removing webkit.py
https://bugs.webkit.org/show_bug.cgi?id=92549

Unreviewed, build fix.

It turns out that this file is listed in the chromium downstream
'browser_tests.isolate' file, and so deleting it broke gyp.
Adding a stub file back in until we can remove the entry from
the file.

  • Scripts/webkitpy/layout_tests/port/webkit.py: Added.
2:40 PM Changeset in webkit [123917] by Patrick Gansterer
  • 2 edits in trunk/Source/WTF

MinGW build fails because of missing header in GregorianDateTime.h
https://bugs.webkit.org/show_bug.cgi?id=92531

Patch by Paweł Forysiuk <tuxator@o2.pl> on 2012-07-27

  • wtf/GregorianDateTime.h: Include time.h
2:10 PM CommitterTips edited by peter@chromium.org
Clarify that new committers should manually commit the change to … (diff)
2:04 PM Changeset in webkit [123916] by dpranke@chromium.org
  • 13 edits
    2 deletes in trunk/Tools

nrwt: remove 'webkit.py' and the 'WebKitPort'
https://bugs.webkit.org/show_bug.cgi?id=92450

Reviewed by Ryosuke Niwa.

This patch finishes the work of merging WebKitPort into Port.
Now Port is way too big :).

This is all cutting and pasting; no new functionality and
no changes in the tests.

  • Scripts/webkitpy/layout_tests/port/apple.py:

(ApplePort):
(ApplePort.init):

  • Scripts/webkitpy/layout_tests/port/base.py:

(Port.skipped_layout_tests):
(Port):
(Port._port_flag_for_scripts):
(Port._arguments_for_configuration):
(Port._run_script):
(Port._build_driver):
(Port._build_driver_flags):
(Port._tests_for_other_platforms):
(Port._runtime_feature_list):
(Port.nm_command):
(Port._modules_to_search_for_symbols):
(Port._symbols_string):
(Port._missing_feature_to_skipped_tests):
(Port._missing_symbol_to_skipped_tests):
(Port._has_test_in_directories):
(Port._skipped_tests_for_unsupported_features):
(Port._wk2_port_name):
(Port._skipped_file_search_paths):

  • Scripts/webkitpy/layout_tests/port/chromium.py:

(ChromiumPort):

  • Scripts/webkitpy/layout_tests/port/chromium_android.py:
  • Scripts/webkitpy/layout_tests/port/chromium_port_testcase.py:

(ChromiumPortTestCase.test_check_build):

  • Scripts/webkitpy/layout_tests/port/driver_unittest.py:
  • Scripts/webkitpy/layout_tests/port/efl.py:

(EflPort):
(EflPort.init):

  • Scripts/webkitpy/layout_tests/port/gtk.py:

(GtkPort):
(GtkPort.setup_environ_for_server):

  • Scripts/webkitpy/layout_tests/port/port_testcase.py:

(TestWebKitPort):
(TestWebKitPort.init):
(TestWebKitPort.all_test_configurations):
(TestWebKitPort._symbols_string):
(TestWebKitPort._tests_for_other_platforms):
(TestWebKitPort._tests_for_disabled_features):
(PortTestCase):
(PortTestCase.test_diff_image):
(PortTestCase.test_path_to_test_expectations_file):
(PortTestCase.test_skipped_directories_for_symbols):
(test_skipped_directories_for_features):
(test_skipped_directories_for_features_no_matching_tests_in_test_list):
(test_skipped_tests_for_unsupported_features_empty_test_list):
(test_skipped_layout_tests):
(test_skipped_file_search_paths):
(test_root_option):
(test_test_expectations):
(test_build_driver):
(_assert_config_file_for_platform):
(test_linux_distro_detection):
(test_apache_config_file_name_for_platform):
(test_path_to_apache_config_file):

  • Scripts/webkitpy/layout_tests/port/qt.py:

(QtPort):
(QtPort.init):
(QtPort.setup_environ_for_server):

  • Scripts/webkitpy/layout_tests/port/webkit.py: Removed.
  • Scripts/webkitpy/layout_tests/port/webkit_unittest.py: Removed.
  • Scripts/webkitpy/tool/servers/rebaselineserver.py:

(get_test_baselines.AllPlatformsPort):
(get_test_baselines.AllPlatformsPort.init):

  • Scripts/webkitpy/tool/servers/rebaselineserver_unittest.py:

(get_test_config.TestMacPort):

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

Don't use an NSUserDefault for disabling the web process sandbox
https://bugs.webkit.org/show_bug.cgi?id=92542
<rdar://problem/11976060>

Reviewed by Mark Rowe.

The user default was really helpful during WebKit2 bringup, but we no longer need to pay the IPC cost of initializing NSUserDefaults at startup,
so get rid of the preference and use a good old #ifdef instead.

  • WebProcess/mac/WebProcessMac.mm:

(WebKit::initializeSandbox):

1:43 PM Changeset in webkit [123914] by commit-queue@webkit.org
  • 4 edits
    4 adds in trunk

Animated SVGs do not clear previous frame completely in hidpi mode.
https://bugs.webkit.org/show_bug.cgi?id=92395

Patch by Varun Jain <varunjain@chromium.org> on 2012-07-27
Reviewed by Darin Adler.

Source/WebCore:

The rect thats is cleared in subsequent frames needed to be scaled with device scale.

Test: svg/as-image/animated-svg-repaints-completely-in-hidpi.html

  • svg/graphics/SVGImage.cpp:

(WebCore::SVGImage::drawSVGToImageBuffer):

LayoutTests:

  • platform/chromium/TestExpectations:
  • platform/mac/svg/as-image/animated-svg-repaints-completely-in-hidpi-expected.png: Added.
  • platform/mac/svg/as-image/animated-svg-repaints-completely-in-hidpi-expected.txt: Added.
  • svg/as-image/animated-svg-repaints-completely-in-hidpi.html: Added.
  • svg/as-image/resources/animated-rect-fixed-size-3.svg: Added.
1:41 PM Changeset in webkit [123913] by mitz@apple.com
  • 2 edits in trunk/Source/WebCore

Fix COMPILE_ASSERT for InlineFlowBox growing
https://bugs.webkit.org/show_bug.cgi?id=92541

Patch by Scott Graham <scottmg@chromium.org> on 2012-07-27
Reviewed by Dan Bernstein.

Use unsigned instead of bool to keep size small on Windows.

No new tests.

  • rendering/InlineFlowBox.h:

(InlineFlowBox):

1:33 PM Changeset in webkit [123912] by Beth Dakin
  • 13 edits
    3 adds in trunk

https://bugs.webkit.org/show_bug.cgi?id=92327
-webkit-background-clip:text is blurry in WebKit 1 apps when
deviceScaleFactor > 1
-and corresponding-
<rdar://problem/11683788>

Reviewed by Simon Fraser.

Source/WebCore:

The bug here is that the code to make createCompatibleBuffer() HiDPI-savvy
assumed that the deviceScaleFactor would always be baked into the CTM of the
GraphicsContext. But that is NOT the case in WebKit 1.

createCompatibleBuffer() is used for clip text and gradients.

Now getCTM() takes a parameter indicating whether the result should definitely
include the device scale, or if it should possibly included the device scale,
which is the option that matches old behavior.

  • platform/graphics/GraphicsContext.h:

(GraphicsContext):

  • platform/graphics/cairo/GraphicsContextCairo.cpp:

(WebCore::GraphicsContext::getCTM):

  • platform/graphics/openvg/GraphicsContextOpenVG.cpp:

(WebCore::GraphicsContext::getCTM):

  • platform/graphics/qt/GraphicsContextQt.cpp:

(WebCore::GraphicsContext::getCTM):

  • platform/graphics/skia/GraphicsContextSkia.cpp:

(WebCore::GraphicsContext::getCTM):

  • platform/graphics/wince/GraphicsContextWinCE.cpp:

(WebCore::GraphicsContext::getCTM):

  • platform/graphics/wx/GraphicsContextWx.cpp:

(WebCore::GraphicsContext::getCTM):

Actually use the new parameter in the CG implementation. Use CG API to get a
matrix that definitely includes the device scale when that is required.

  • platform/graphics/cg/GraphicsContextCG.cpp:

(WebCore::GraphicsContext::getCTM):

Remove some symbol cruft that doesn't seem to require a replacement.

  • WebCore.exp.in:

Use DefinitelyIncludeDeviceScale when getting the CTM in the buggy spot.

  • platform/graphics/GraphicsContext.cpp:

(WebCore::GraphicsContext::createCompatibleBuffer):

The ImageBuffer for gradients is created using createCompatibleBuffer(), and since
createCompatibleBuffer() now uses getCTM(DefinitelyIncludeDeviceScale) to
determine appropriate sizing, drawPattern() should use that same matrix to
determine pattern sizing.

  • platform/graphics/GeneratorGeneratedImage.cpp:

(WebCore::GeneratorGeneratedImage::drawPattern):

LayoutTests:

  • fast/hidpi/gradient-with-scaled-ancestor.html: Added.
  • platform/mac/fast/hidpi/gradient-with-scaled-ancestor-expected.png: Added.
  • platform/mac/fast/hidpi/gradient-with-scaled-ancestor-expected.txt: Added.
1:10 PM Changeset in webkit [123911] by commit-queue@webkit.org
  • 10 edits in trunk

[WK2][WTR] LayoutTestController.deliverWebIntent() needs to be implemented
https://bugs.webkit.org/show_bug.cgi?id=92228

Patch by Christophe Dumez <Christophe Dumez> on 2012-07-27
Reviewed by Anders Carlsson.

Source/WebKit2:

Add C API to create a WKIntentData and to deliver
it to a frame. This is needed by WebKitTestRunner
to implement LayoutTestController.deliverWebIntent().

  • UIProcess/API/C/WKIntentData.cpp:

(WKIntentDataCreate):

  • UIProcess/API/C/WKIntentData.h:
  • WebProcess/InjectedBundle/API/c/WKBundlePage.cpp:

(WKBundlePageDeliverIntentToFrame):

  • WebProcess/InjectedBundle/API/c/WKBundlePage.h:
  • WebProcess/WebPage/WebPage.h:

(WebPage): Make deliverIntentToFrame() public so that it can
be used by WKBundlePage.

Tools:

Implement LayoutTestController.deliverWebIntent() to
allow test cases to deliver a Web intent and test this
functionality.

This functionality is needed by:

  • webintents/web-intents-delivery.html
  • WebKitTestRunner/InjectedBundle/Bindings/LayoutTestController.idl:
  • WebKitTestRunner/InjectedBundle/LayoutTestController.cpp:

(WTR::LayoutTestController::deliverWebIntent):
(WTR):

  • WebKitTestRunner/InjectedBundle/LayoutTestController.h:

(LayoutTestController):

12:49 PM Changeset in webkit [123910] by bweinstein@apple.com
  • 1 edit
    1 add in trunk/LayoutTests

Add Mac results for fast/table/table-row-outline-paint.html.

  • platform/mac/fast/table/table-row-outline-paint-expected.txt: Added.
12:46 PM Changeset in webkit [123909] by commit-queue@webkit.org
  • 5 edits
    2 adds in trunk

Source/WebCore: flex-wrap: wrap not wrapping for % sized items in column flow
https://bugs.webkit.org/show_bug.cgi?id=92324

Patch by Tony Chang <tony@chromium.org> on 2012-07-27
Reviewed by Ojan Vafai.

We were using trying to use the value of contentLogicalHeight() before having called computeLogicalHeight()
in a few places. Fix this in mainAxisContentExtent() rather than at the callers.

Test: css3/flexbox/percentage-heights.html

  • rendering/RenderFlexibleBox.cpp:

(WebCore::RenderFlexibleBox::mainAxisContentExtent):
(WebCore::RenderFlexibleBox::preferredMainAxisContentExtentForChild): If we haven't called computeLogicalHeight, we try to compute
the height based on the fixed flexbox value. min/max don't matter here since percent values are only based on height/width.

  • rendering/RenderFlexibleBox.h: Drop const because computeContentLogicalHeightUsing is not const. I can

try making computeContentLogicalHeightUsing const in a follow up change (might be non-trivial).

Source/WebKit2: Show the unavailable plug-in indicator for Java applets as well
https://bugs.webkit.org/show_bug.cgi?id=92521

Patch by Anders Carlsson <andersca@apple.com> on 2012-07-27
Reviewed by Sam Weinig.

  • WebProcess/WebCoreSupport/WebChromeClient.cpp:

(WebKit::WebChromeClient::unavailablePluginButtonClicked):
This can now be called on applet elements as well.

  • WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:

(WebKit::WebFrameLoaderClient::createJavaAppletWidget):
Get the MIME type from the applet element.

LayoutTests: flex-wrap: wrap not wrapping for % sized items in column flow
https://bugs.webkit.org/show_bug.cgi?id=92324

Patch by Tony Chang <tony@chromium.org> on 2012-07-27
Reviewed by Ojan Vafai.

Test cases for column flow with percentage heights. There are three
test cases for the three callers of mainAxisContentExtent() that were
broken by this.

  • css3/flexbox/percentage-heights-expected.txt: Added.
  • css3/flexbox/percentage-heights.html: Added.
12:38 PM Changeset in webkit [123908] by commit-queue@webkit.org
  • 16 edits
    2 adds in trunk/Source/WebCore

Add an overlay play button to media controls on android
https://bugs.webkit.org/show_bug.cgi?id=92132

Patch by Min Qin <qinmin@chromium.org> on 2012-07-27
Reviewed by Adam Barth.

This change adds an overlay play button to the media controls for android.
No tests for now as this setting is not being used by other port.
Will add a test and adjust all the test expectations when we upstream the layout tests for android.

  • WebCore.gypi:
  • css/CSSPrimitiveValueMappings.h:

(WebCore::CSSPrimitiveValue::CSSPrimitiveValue):

  • css/CSSValueKeywords.in:
  • css/mediaControls.css:

(audio::-webkit-media-controls-overlay-play-button, video::-webkit-media-controls-overlay-play-button):

  • css/mediaControlsChromiumAndroid.css:

(audio::-webkit-media-controls-overlay-enclosure):
(video::-webkit-media-controls-overlay-enclosure):
(audio::-webkit-media-controls-play-button, video::-webkit-media-controls-play-button):
(video::-webkit-media-controls-overlay-play-button):

  • html/shadow/MediaControlElements.cpp:

(WebCore::MediaControlOverlayPlayButtonElement::MediaControlOverlayPlayButtonElement):
(WebCore):
(WebCore::MediaControlOverlayPlayButtonElement::create):
(WebCore::MediaControlOverlayPlayButtonElement::defaultEventHandler):
(WebCore::MediaControlOverlayPlayButtonElement::updateDisplayType):
(WebCore::MediaControlOverlayPlayButtonElement::shadowPseudoId):

  • html/shadow/MediaControlElements.h:

(MediaControlOverlayPlayButtonElement):
(WebCore):

  • html/shadow/MediaControlRootElementChromium.cpp:

(WebCore::MediaControlPanelEnclosureElement::MediaControlPanelEnclosureElement):
(WebCore::MediaControlPanelEnclosureElement::create):
(WebCore):
(WebCore::MediaControlPanelEnclosureElement::shadowPseudoId):
(WebCore::MediaControlRootElementChromium::create):
(WebCore::MediaControlRootElementChromium::initializeControls):

  • html/shadow/MediaControlRootElementChromium.h:

(MediaControlChromiumEnclosureElement):
(WebCore):
(MediaControlPanelEnclosureElement):
(MediaControlRootElementChromium):

  • html/shadow/MediaControlRootElementChromiumAndroid.cpp: Added.

(WebCore):
(WebCore::MediaControlOverlayEnclosureElement::MediaControlOverlayEnclosureElement):
(WebCore::MediaControlOverlayEnclosureElement::create):
(WebCore::MediaControlOverlayEnclosureElement::shadowPseudoId):
(WebCore::MediaControlRootElementChromiumAndroid::MediaControlRootElementChromiumAndroid):
(WebCore::MediaControls::create):
(WebCore::MediaControlRootElementChromiumAndroid::create):
(WebCore::MediaControlRootElementChromiumAndroid::setMediaController):
(WebCore::MediaControlRootElementChromiumAndroid::playbackStarted):
(WebCore::MediaControlRootElementChromiumAndroid::playbackStopped):

  • html/shadow/MediaControlRootElementChromiumAndroid.h: Added.

(WebCore):
(MediaControlOverlayEnclosureElement):
(MediaControlRootElementChromiumAndroid):

  • platform/ThemeTypes.h:
  • rendering/RenderMediaControlsChromium.cpp:

(WebCore::paintMediaOverlayPlayButton):
(WebCore):
(WebCore::RenderMediaControlsChromium::paintMediaControlsPart):

  • rendering/RenderTheme.cpp:

(WebCore::RenderTheme::paint):

  • rendering/RenderTheme.h:

(WebCore::RenderTheme::paintMediaOverlayPlayButton):

  • rendering/RenderThemeChromiumAndroid.cpp:

(WebCore::RenderThemeChromiumAndroid::paintMediaOverlayPlayButton):
(WebCore):

  • rendering/RenderThemeChromiumAndroid.h:
12:30 PM Changeset in webkit [123907] by andersca@apple.com
  • 9 edits in trunk/Source

Show the unavailable plug-in indicator for Java applets as well
https://bugs.webkit.org/show_bug.cgi?id=92521

Reviewed by Sam Weinig.

Source/WebCore:

Now that <applet> behaves more like <embed> and <object>, make sure that we show the unavailable plug-in indicator
and call the correct error callbacks if we fail to instantiate the plug-in.

  • WebCore.exp.in:

Export a symbol needed by WebKit2.

  • html/HTMLAppletElement.cpp:

(WebCore::HTMLAppletElement::HTMLAppletElement):
Set the correct service type.

  • loader/SubframeLoader.cpp:

(WebCore::SubframeLoader::createJavaAppletWidget):
Enable the unavailable plug-in indicator if we fail to create the java applet widget.

Source/WebKit/mac:

Check if the Java plug-in is blocked before trying to instantiate it.

  • WebCoreSupport/WebFrameLoaderClient.mm:

Source/WebKit2:

  • WebProcess/WebCoreSupport/WebChromeClient.cpp:

(WebKit::WebChromeClient::unavailablePluginButtonClicked):
This can now be called on applet elements as well.

  • WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:

(WebKit::WebFrameLoaderClient::createJavaAppletWidget):
Get the MIME type from the applet element.

12:30 PM Changeset in webkit [123906] by jer.noble@apple.com
  • 4 edits in trunk

Unreviewed build fix.

Source/WTF:

Use MathExtras.h and the non std:: versions of isnan and signbit.

  • wtf/MediaTime.cpp:

(WTF::MediaTime::createWithFloat):
(WTF::MediaTime::createWithDouble):

Tools:

Wrap MSVC-specific fixes is a COMPILER() check, not a PLATFORM() check.

  • TestWebKitAPI/Tests/WTF/MediaTime.cpp:
12:29 PM Changeset in webkit [123905] by mhahnenberg@apple.com
  • 3 edits in trunk/Source/JavaScriptCore

Increase inline storage for JSFinalObjects by one
https://bugs.webkit.org/show_bug.cgi?id=92526

Reviewed by Geoffrey Garen.

Now that we've removed the inheritorID from objects, we can increase our inline storage for JSFinalObjects on
64-bit platforms by 1.

  • llint/LowLevelInterpreter.asm: Change the constant.
  • runtime/PropertyOffset.h: Change the constant.

(JSC):

12:25 PM Changeset in webkit [123904] by mitz@apple.com
  • 5 edits
    2 adds in trunk

Hit testing near a column break can return a result from an adjacent column when there is leading
https://bugs.webkit.org/show_bug.cgi?id=92524

Reviewed by Anders Carlsson.

Source/WebCore:

The fix for <http://webkit.org/b/92311> relied on the existence of a pagination strut for
detecting that a line was at the beginning of a new column. However, when a line naturally
falls at the beginning of a column, there is no pagination strut, and the check failed.

Test: fast/multicol/hit-test-end-of-column-with-line-height.html

  • rendering/InlineFlowBox.h:

(WebCore::InlineFlowBox::InlineFlowBox): Added initializer for new member variable.
(InlineFlowBox): Added m_isFirstAfterPageBreak member variable.

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::positionForPointWithInlineChildren): Changed the test for whether a
line was at the beginning of a column from relying on a pagination strut to checking
isFirstAfterPageBreak(). Also refined the hit test itself to include the leading above such
a line.
(WebCore::RenderBlock::adjustLinePositionForPagination): Added calls to
setIsFirstAfterPageBreak() to first reset this flag, then set it to true if necessary.

  • rendering/RootInlineBox.h:

(WebCore::RootInlineBox::isFirstAfterPageBreak): Added this accessor.
(WebCore::RootInlineBox::setIsFirstAfterPageBreak): Ditto.

LayoutTests:

  • fast/multicol/hit-test-end-of-column-with-line-height-expected.txt: Added.
  • fast/multicol/hit-test-end-of-column-with-line-height.html: Added.
12:03 PM Changeset in webkit [123903] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit2

[WK2] Fix build warning in WebEventConversion.cpp
https://bugs.webkit.org/show_bug.cgi?id=92517

Patch by Gyuyoung Kim <gyuyoung.kim@samsung.com> on 2012-07-27
Reviewed by Darin Adler.

There is a build warning related to comparison between signed and unsigned integer expressions.

  • Shared/WebEventConversion.cpp:

(WebKit::WebKit2PlatformTouchEvent::WebKit2PlatformTouchEvent):

12:02 PM Changeset in webkit [123902] by commit-queue@webkit.org
  • 3 edits in trunk/Source

Web Inspector: Allow front_end to be loaded into an iframe
https://bugs.webkit.org/show_bug.cgi?id=92437

Patch by John J. Barton <johnjbarton@johnjbarton.com> on 2012-07-27
Reviewed by Pavel Feldman.

This change only affects 'embedders' of the inspector/front_end. No change of function for WebKit ports.

  • inspector/front-end/ExtensionAPI.js:

Replace window.top with window.parent two places. When WebInspector is loaded as the main window content,
then extension iframes have window.top === window.parent; when WebInspector is loaded as an iframe, then
extensions are iframes in iframes and we need to use a relative address window.parent.
(injectedExtensionAPI.ExtensionViewImpl.dispatchShowEvent):
(injectedExtensionAPI.ExtensionViewImpl):
(injectedExtensionAPI.ExtensionServerClient):
WebKit/chromium/scripts/generate_devtools_extension_api.py:
The current number of frames is used in a dynamically created identifier. Again we replace 'top' by
window.parent.

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

Web Inspector: InspectorPageAgent should ascertain document to be non-null in updateViewMetrics()
https://bugs.webkit.org/show_bug.cgi?id=92476

Patch by Vivek Galatage <vivekgalatage@gmail.com> on 2012-07-27
Reviewed by Pavel Feldman.

InspectorPageAgent should check for document object being null. As pages created with blank URL might
not have the document object, this causes a crash.

No new tests as added null check.

  • inspector/InspectorPageAgent.cpp:

(WebCore::InspectorPageAgent::updateViewMetrics):

11:42 AM Changeset in webkit [123900] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

[Chromium-Android] NRWT better handling of DRT deadlocks and crashes
https://bugs.webkit.org/show_bug.cgi?id=92299

Patch by Xianzhu Wang <wangxianzhu@chromium.org> on 2012-07-27
Reviewed by Dirk Pranke.

  1. Changed the deadlock detector thread target from a method of driver to a function to avoid its reference to the current driver (see the bug for the problem of the reference.
  2. Let the deadlock detector thread exit early when the normal_startup_event is set to avoid it from blocking the main thread when the main thread is exiting.
  3. Extracted common loop until timeout logic into ChromiumAndroidDriver._loop_with_timeout().
  4. Use the "for i in range(n)" style suggested by dpranke in https://bugs.webkit.org/show_bug.cgi?id=89124 in several functions.
  • Scripts/webkitpy/layout_tests/port/chromium_android.py:

(ChromiumAndroidPort.init):
(ChromiumAndroidPort._run_adb_command): Reduced the verbose output.
(ChromiumAndroidDriver.init):
(ChromiumAndroidDriver._loop_with_timeout): Extracted common loop until timeout logic here.
(ChromiumAndroidDriver._all_pipes_created):
(ChromiumAndroidDriver._remove_all_pipes):
(ChromiumAndroidDriver._start): Changed the retry loop style.
(ChromiumAndroidDriver._start_once): Changed the loop style and code related to deadlock detector.
(ChromiumAndroidDriver._start_once.deadlock_detector): Moved the original ChromiumAndroidDriver._deadlock_detector() here.
(ChromiumAndroidDriver.stop): Changed the loop style.

11:38 AM Changeset in webkit [123899] by commit-queue@webkit.org
  • 4 edits
    2 adds in trunk

CSP directives containing invalid characters should log an error.
https://bugs.webkit.org/show_bug.cgi?id=92487

Patch by Mike West <mkwst@chromium.org> on 2012-07-27
Reviewed by Adam Barth.

Source/WebCore:

Unknown directive names are currently logged, but we exit the directive
parser early without logging if we hit an invalid character inside a
directive. script-src: ..., for example, was ignored without letting
the developer know what happened. This patch changes that behavior,
logging the whole name (in this case script-src:) as an unknown
directive.

Test: http/tests/security/contentSecurityPolicy/directive-parsing-04.html

  • page/ContentSecurityPolicy.cpp:

(WebCore::CSPDirectiveList::parseDirective):

LayoutTests:

  • http/tests/security/contentSecurityPolicy/directive-parsing-03-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/directive-parsing-04-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/directive-parsing-04.html: Added.

Adding a test specifically for script-src: ... which bit me, and
updating test #3, which now has a better error message.

11:26 AM Changeset in webkit [123898] by dpranke@chromium.org
  • 4 edits in trunk/Tools

nrwt: move collect_tests(), parse_expectations() into manager
https://bugs.webkit.org/show_bug.cgi?id=92423

Reviewed by Ojan Vafai.

This is the first in a series of patches refactoring
run_webkit_tests.py and manager.py to make them more
maintainable; all this patch does is move a couple of functions
into manager.run_tests() so manager's public interface is
simple.

  • Scripts/webkitpy/layout_tests/controllers/manager.py:

(Manager._collect_tests):
(Manager._parse_expectations):
(Manager._split_into_chunks_if_necessary):
(Manager.run):
(read_test_files):

  • Scripts/webkitpy/layout_tests/controllers/manager_unittest.py:

(ManagerTest.test_http_locking):
(ManagerTest.test_perf_locking):
(ManagerTest.integration_test_needs_servers.get_manager_with_tests):
(ManagerTest.test_look_for_new_crash_logs.get_manager_with_tests):

  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:

(run):

11:25 AM Changeset in webkit [123897] by commit-queue@webkit.org
  • 1 edit
    5 adds in trunk/PerformanceTests

Add more Russian replay performance tests
https://bugs.webkit.org/show_bug.cgi?id=92462

Patch by Ryosuke Niwa <rniwa@webkit.org> on 2012-07-27
Reviewed by Tony Chang.

Add more Russian sites as suggested by loislo.

  • Replay/Russian/lenta.ru.replay: Added.
  • Replay/Russian/www.ixbt.com.replay: Added.
  • Replay/Russian/www.kp.ru.replay: Added.
  • Replay/Russian/www.liveinternet.ru.replay: Added.
  • Replay/Russian/www.pravda.ru.replay: Added.
11:25 AM Changeset in webkit [123896] by commit-queue@webkit.org
  • 404 edits
    2 copies
    1 move
    10 adds
    23 deletes in trunk/LayoutTests

[Qt] platform/qt/svg/custom tests needs update after font change.
https://bugs.webkit.org/show_bug.cgi?id=91090.

Patch by Ádám Kallai <kadam@inf.u-szeged.hu> on 2012-07-27
Reviewed by Zoltan Herczeg.

  • platform/qt-5.0-wk1/Skipped:
  • platform/qt-5.0-wk2/Skipped:
  • platform/qt-5.0-wk2/svg/custom/altglyph-expected.png: Removed.
  • platform/qt-5.0-wk2/svg/custom/altglyph-expected.txt: Removed.
  • platform/qt-5.0-wk2/svg/custom/font-face-simple-expected.png: Removed.
  • platform/qt-5.0-wk2/svg/custom/font-face-simple-expected.txt: Removed.
  • platform/qt-5.0-wk2/svg/custom/inline-svg-in-xhtml-expected.png: Added.
  • platform/qt-5.0-wk2/svg/custom/inline-svg-in-xhtml-expected.txt: Copied from LayoutTests/platform/qt/svg/custom/inline-svg-in-xhtml-expected.txt.
  • platform/qt-5.0/Skipped:
  • platform/qt-5.0/svg/custom/glyph-selection-arabic-forms-expected.png: Removed.
  • platform/qt-5.0/svg/custom/glyph-selection-arabic-forms-expected.txt: Removed.
  • platform/qt-5.0/svg/custom/glyph-selection-bidi-mirror-expected.png: Removed.
  • platform/qt-5.0/svg/custom/glyph-selection-bidi-mirror-expected.txt: Removed.
  • platform/qt-5.0/svg/custom/glyph-selection-non-bmp-expected.txt: Removed.
  • platform/qt-5.0/svg/custom/glyph-transformation-with-hkern-expected.png: Removed.
  • platform/qt-5.0/svg/custom/glyph-transformation-with-hkern-expected.txt: Removed.
  • platform/qt-5.0/svg/custom/preserve-aspect-ratio-syntax-expected.png: Removed.
  • platform/qt-5.0/svg/custom/preserve-aspect-ratio-syntax-expected.txt: Removed.
  • platform/qt-5.0/svg/custom/scrolling-embedded-svg-file-image-repaint-problem-expected.png: Removed.
  • platform/qt-5.0/svg/custom/scrolling-embedded-svg-file-image-repaint-problem-expected.txt: Removed.
  • platform/qt-5.0/svg/custom/svg-features-expected.txt: Removed.
  • platform/qt-5.0/svg/custom/svg-fonts-in-html-expected.png: Removed.
  • platform/qt-5.0/svg/custom/svg-fonts-in-html-expected.txt: Removed.
  • platform/qt-5.0/svg/custom/svg-fonts-in-text-controls-expected.txt: Removed.
  • platform/qt-5.0/svg/custom/svg-fonts-segmented-expected.png: Removed.
  • platform/qt-5.0/svg/custom/svg-fonts-segmented-expected.txt: Removed.
  • platform/qt-5.0/svg/custom/text-linking-expected.png: Removed.
  • platform/qt-5.0/svg/custom/text-linking-expected.txt: Removed.
  • platform/qt/svg/custom/SVGMatrix-interface-expected.png:
  • platform/qt/svg/custom/SVGMatrix-interface-expected.txt:
  • platform/qt/svg/custom/SVGPoint-matrixTransform-expected.png:
  • platform/qt/svg/custom/SVGPoint-matrixTransform-expected.txt:
  • platform/qt/svg/custom/absolute-sized-content-with-resources-expected.png:
  • platform/qt/svg/custom/absolute-sized-content-with-resources-expected.txt:
  • platform/qt/svg/custom/absolute-sized-svg-in-xhtml-expected.png:
  • platform/qt/svg/custom/absolute-sized-svg-in-xhtml-expected.txt:
  • platform/qt/svg/custom/alignment-baseline-modes-expected.png:
  • platform/qt/svg/custom/alignment-baseline-modes-expected.txt:
  • platform/qt/svg/custom/altglyph-expected.png:
  • platform/qt/svg/custom/altglyph-expected.txt:
  • platform/qt/svg/custom/baseval-animval-equality-expected.png:
  • platform/qt/svg/custom/baseval-animval-equality-expected.txt:
  • platform/qt/svg/custom/broken-internal-references-expected.txt:
  • platform/qt/svg/custom/bug45331-expected.png: Added.
  • platform/qt/svg/custom/bug45331-expected.txt: Added.
  • platform/qt/svg/custom/bug86119-expected.txt:
  • platform/qt/svg/custom/circle-move-invalidation-expected.png:
  • platform/qt/svg/custom/circle-move-invalidation-expected.txt:
  • platform/qt/svg/custom/clip-path-referencing-use-expected.png:
  • platform/qt/svg/custom/clip-path-referencing-use-expected.txt:
  • platform/qt/svg/custom/clip-path-referencing-use2-expected.png:
  • platform/qt/svg/custom/clip-path-referencing-use2-expected.txt:
  • platform/qt/svg/custom/clone-element-with-animated-svg-properties-expected.png:
  • platform/qt/svg/custom/clone-element-with-animated-svg-properties-expected.txt:
  • platform/qt/svg/custom/container-opacity-clip-viewBox-expected.png:
  • platform/qt/svg/custom/container-opacity-clip-viewBox-expected.txt:
  • platform/qt/svg/custom/coords-relative-units-transforms-expected.png:
  • platform/qt/svg/custom/coords-relative-units-transforms-expected.txt:
  • platform/qt/svg/custom/createImageElement2-expected.png: Added.
  • platform/qt/svg/custom/createImageElement2-expected.txt:
  • platform/qt/svg/custom/dominant-baseline-hanging-expected.png:
  • platform/qt/svg/custom/dominant-baseline-hanging-expected.txt:
  • platform/qt/svg/custom/dominant-baseline-modes-expected.png:
  • platform/qt/svg/custom/dominant-baseline-modes-expected.txt:
  • platform/qt/svg/custom/dynamic-svg-document-creation-expected.png:
  • platform/qt/svg/custom/dynamic-svg-document-creation-expected.txt:
  • platform/qt/svg/custom/embedding-external-svgs-expected.png:
  • platform/qt/svg/custom/embedding-external-svgs-expected.txt:
  • platform/qt/svg/custom/empty-clip-path-expected.png:
  • platform/qt/svg/custom/empty-clip-path-expected.txt:
  • platform/qt/svg/custom/empty-mask-expected.png: Added.
  • platform/qt/svg/custom/empty-mask-expected.txt:
  • platform/qt/svg/custom/external-paintserver-reference-expected.png:
  • platform/qt/svg/custom/external-paintserver-reference-expected.txt:
  • platform/qt/svg/custom/fill-SVGPaint-interface-expected.png:
  • platform/qt/svg/custom/fill-SVGPaint-interface-expected.txt:
  • platform/qt/svg/custom/fill-fallback-expected.png:
  • platform/qt/svg/custom/fill-fallback-expected.txt:
  • platform/qt/svg/custom/focus-ring-expected.png:
  • platform/qt/svg/custom/focus-ring-expected.txt:
  • platform/qt/svg/custom/font-face-cascade-order-expected.png:
  • platform/qt/svg/custom/font-face-cascade-order-expected.txt:
  • platform/qt/svg/custom/font-face-simple-expected.png:
  • platform/qt/svg/custom/font-face-simple-expected.txt:
  • platform/qt/svg/custom/foreign-object-skew-expected.png:
  • platform/qt/svg/custom/foreign-object-skew-expected.txt:
  • platform/qt/svg/custom/foreignObject-crash-on-hover-expected.png:
  • platform/qt/svg/custom/foreignObject-crash-on-hover-expected.txt:
  • platform/qt/svg/custom/getPresentationAttribute-expected.png:
  • platform/qt/svg/custom/getPresentationAttribute-expected.txt:
  • platform/qt/svg/custom/getTransformToElement-expected.png:
  • platform/qt/svg/custom/getTransformToElement-expected.txt:
  • platform/qt/svg/custom/getscreenctm-in-mixed-content-expected.png:
  • platform/qt/svg/custom/getscreenctm-in-mixed-content-expected.txt:
  • platform/qt/svg/custom/getscreenctm-in-scrollable-div-area-expected.png:
  • platform/qt/svg/custom/getscreenctm-in-scrollable-div-area-expected.txt:
  • platform/qt/svg/custom/getscreenctm-in-scrollable-div-area-nested-expected.png:
  • platform/qt/svg/custom/getscreenctm-in-scrollable-div-area-nested-expected.txt:
  • platform/qt/svg/custom/getscreenctm-in-scrollable-svg-area-expected.png:
  • platform/qt/svg/custom/getscreenctm-in-scrollable-svg-area-expected.txt:
  • platform/qt/svg/custom/getsvgdocument-expected.png:
  • platform/qt/svg/custom/getsvgdocument-expected.txt:
  • platform/qt/svg/custom/glyph-selection-arabic-forms-expected.png: Added.
  • platform/qt/svg/custom/glyph-selection-arabic-forms-expected.txt:
  • platform/qt/svg/custom/glyph-selection-bidi-mirror-expected.png: Copied from LayoutTests/platform/qt-5.0/svg/custom/glyph-selection-non-bmp-expected.png.
  • platform/qt/svg/custom/glyph-selection-bidi-mirror-expected.txt:
  • platform/qt/svg/custom/glyph-selection-non-bmp-expected.png: Renamed from LayoutTests/platform/qt-5.0/svg/custom/glyph-selection-non-bmp-expected.png.
  • platform/qt/svg/custom/glyph-selection-non-bmp-expected.txt:
  • platform/qt/svg/custom/glyph-transformation-with-hkern-expected.png:
  • platform/qt/svg/custom/glyph-transformation-with-hkern-expected.txt:
  • platform/qt/svg/custom/gradient-cycle-detection-expected.png:
  • platform/qt/svg/custom/gradient-cycle-detection-expected.txt:
  • platform/qt/svg/custom/gradient-deep-referencing-expected.png:
  • platform/qt/svg/custom/gradient-deep-referencing-expected.txt:
  • platform/qt/svg/custom/gradient-with-1d-boundingbox-expected.png: Added.
  • platform/qt/svg/custom/gradient-with-1d-boundingbox-expected.txt:
  • platform/qt/svg/custom/hit-test-path-expected.png:
  • platform/qt/svg/custom/hit-test-path-expected.txt:
  • platform/qt/svg/custom/hit-test-path-stroke-expected.png:
  • platform/qt/svg/custom/hit-test-path-stroke-expected.txt:
  • platform/qt/svg/custom/hit-test-unclosed-subpaths-expected.png:
  • platform/qt/svg/custom/hit-test-unclosed-subpaths-expected.txt:
  • platform/qt/svg/custom/image-parent-translation-expected.png:
  • platform/qt/svg/custom/image-parent-translation-expected.txt:
  • platform/qt/svg/custom/image-rescale-clip-expected.png:
  • platform/qt/svg/custom/image-rescale-clip-expected.txt:
  • platform/qt/svg/custom/image-rescale-scroll-expected.png:
  • platform/qt/svg/custom/image-rescale-scroll-expected.txt:
  • platform/qt/svg/custom/image-small-width-height-expected.png:
  • platform/qt/svg/custom/image-small-width-height-expected.txt:
  • platform/qt/svg/custom/image-with-transform-clip-filter-expected.png:
  • platform/qt/svg/custom/image-with-transform-clip-filter-expected.txt:
  • platform/qt/svg/custom/inline-svg-in-xhtml-expected.png:
  • platform/qt/svg/custom/inline-svg-in-xhtml-expected.txt:
  • platform/qt/svg/custom/invalid-css-expected.png:
  • platform/qt/svg/custom/invalid-css-expected.txt:
  • platform/qt/svg/custom/invalid-fill-expected.png:
  • platform/qt/svg/custom/invalid-fill-expected.txt:
  • platform/qt/svg/custom/invalid-fill-hex-expected.png:
  • platform/qt/svg/custom/invalid-fill-hex-expected.txt:
  • platform/qt/svg/custom/invalid-lengthlist-expected.png:
  • platform/qt/svg/custom/invalid-lengthlist-expected.txt:
  • platform/qt/svg/custom/invalid-stroke-hex-expected.png: Added.
  • platform/qt/svg/custom/invalid-stroke-hex-expected.txt:
  • platform/qt/svg/custom/invalid-uri-stroke-expected.png: Added.
  • platform/qt/svg/custom/invalid-uri-stroke-expected.txt:
  • platform/qt/svg/custom/invisible-text-after-scrolling-expected.png:
  • platform/qt/svg/custom/invisible-text-after-scrolling-expected.txt:
  • platform/qt/svg/custom/js-late-clipPath-and-object-creation-expected.png:
  • platform/qt/svg/custom/js-late-clipPath-and-object-creation-expected.txt:
  • platform/qt/svg/custom/js-late-clipPath-creation-expected.png:
  • platform/qt/svg/custom/js-late-clipPath-creation-expected.txt:
  • platform/qt/svg/custom/js-late-gradient-and-object-creation-expected.png:
  • platform/qt/svg/custom/js-late-gradient-and-object-creation-expected.txt:
  • platform/qt/svg/custom/js-late-gradient-creation-expected.png:
  • platform/qt/svg/custom/js-late-gradient-creation-expected.txt:
  • platform/qt/svg/custom/js-update-stop-linked-gradient-expected.png:
  • platform/qt/svg/custom/js-update-stop-linked-gradient-expected.txt:
  • platform/qt/svg/custom/linking-a-03-b-preserveAspectRatio-expected.png:
  • platform/qt/svg/custom/linking-a-03-b-preserveAspectRatio-expected.txt:
  • platform/qt/svg/custom/linking-a-03-b-transform-expected.png:
  • platform/qt/svg/custom/linking-a-03-b-transform-expected.txt:
  • platform/qt/svg/custom/linking-a-03-b-viewBox-expected.png:
  • platform/qt/svg/custom/linking-a-03-b-viewBox-expected.txt:
  • platform/qt/svg/custom/linking-a-03-b-viewBox-transform-expected.png:
  • platform/qt/svg/custom/linking-a-03-b-viewBox-transform-expected.txt:
  • platform/qt/svg/custom/linking-a-03-b-viewTarget-expected.png:
  • platform/qt/svg/custom/linking-a-03-b-viewTarget-expected.txt:
  • platform/qt/svg/custom/linking-a-03-b-zoomAndPan-expected.png:
  • platform/qt/svg/custom/linking-a-03-b-zoomAndPan-expected.txt:
  • platform/qt/svg/custom/linking-base-external-reference-expected.png:
  • platform/qt/svg/custom/linking-base-external-reference-expected.txt:
  • platform/qt/svg/custom/linking-uri-01-b-expected.png:
  • platform/qt/svg/custom/linking-uri-01-b-expected.txt:
  • platform/qt/svg/custom/marker-default-width-height-expected.png:
  • platform/qt/svg/custom/marker-default-width-height-expected.txt:
  • platform/qt/svg/custom/marker-overflow-clip-expected.png:
  • platform/qt/svg/custom/marker-overflow-clip-expected.txt:
  • platform/qt/svg/custom/mouse-move-on-svg-container-expected.png:
  • platform/qt/svg/custom/mouse-move-on-svg-container-expected.txt:
  • platform/qt/svg/custom/mouse-move-on-svg-container-standalone-expected.png:
  • platform/qt/svg/custom/mouse-move-on-svg-container-standalone-expected.txt:
  • platform/qt/svg/custom/mouse-move-on-svg-root-expected.png:
  • platform/qt/svg/custom/mouse-move-on-svg-root-expected.txt:
  • platform/qt/svg/custom/mouse-move-on-svg-root-standalone-expected.png:
  • platform/qt/svg/custom/mouse-move-on-svg-root-standalone-expected.txt:
  • platform/qt/svg/custom/no-inherited-dashed-stroke-expected.png:
  • platform/qt/svg/custom/no-inherited-dashed-stroke-expected.txt:
  • platform/qt/svg/custom/non-opaque-filters-expected.png:
  • platform/qt/svg/custom/non-opaque-filters-expected.txt:
  • platform/qt/svg/custom/object-no-size-attributes-expected.png:
  • platform/qt/svg/custom/object-no-size-attributes-expected.txt:
  • platform/qt/svg/custom/object-sizing-expected.png:
  • platform/qt/svg/custom/object-sizing-expected.txt:
  • platform/qt/svg/custom/object-sizing-explicit-height-expected.png:
  • platform/qt/svg/custom/object-sizing-explicit-height-expected.txt:
  • platform/qt/svg/custom/object-sizing-explicit-width-expected.png:
  • platform/qt/svg/custom/object-sizing-explicit-width-expected.txt:
  • platform/qt/svg/custom/object-sizing-explicit-width-height-expected.png:
  • platform/qt/svg/custom/object-sizing-explicit-width-height-expected.txt:
  • platform/qt/svg/custom/object-sizing-height-50p-on-target-svg-absolute-expected.png:
  • platform/qt/svg/custom/object-sizing-height-50p-on-target-svg-absolute-expected.txt:
  • platform/qt/svg/custom/object-sizing-height-50p-on-target-svg-expected.png:
  • platform/qt/svg/custom/object-sizing-height-50p-on-target-svg-expected.txt:
  • platform/qt/svg/custom/object-sizing-no-width-height-expected.png:
  • platform/qt/svg/custom/object-sizing-no-width-height-expected.txt:
  • platform/qt/svg/custom/object-sizing-width-50p-height-50p-on-target-svg-absolute-expected.png:
  • platform/qt/svg/custom/object-sizing-width-50p-height-50p-on-target-svg-absolute-expected.txt:
  • platform/qt/svg/custom/object-sizing-width-50p-height-50p-on-target-svg-expected.png:
  • platform/qt/svg/custom/object-sizing-width-50p-height-50p-on-target-svg-expected.txt:
  • platform/qt/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-absolute-expected.txt:
  • platform/qt/svg/custom/object-sizing-width-50p-height-75p-on-target-svg-expected.txt:
  • platform/qt/svg/custom/object-sizing-width-50p-on-target-svg-absolute-expected.png:
  • platform/qt/svg/custom/object-sizing-width-50p-on-target-svg-absolute-expected.txt:
  • platform/qt/svg/custom/object-sizing-width-50p-on-target-svg-expected.png:
  • platform/qt/svg/custom/object-sizing-width-50p-on-target-svg-expected.txt:
  • platform/qt/svg/custom/object-sizing-width-75p-height-50p-on-target-svg-absolute-expected.txt:
  • platform/qt/svg/custom/object-sizing-width-75p-height-50p-on-target-svg-expected.png:
  • platform/qt/svg/custom/object-sizing-width-75p-height-50p-on-target-svg-expected.txt:
  • platform/qt/svg/custom/path-textPath-simulation-expected.png:
  • platform/qt/svg/custom/path-textPath-simulation-expected.txt:
  • platform/qt/svg/custom/pattern-cycle-detection-expected.png:
  • platform/qt/svg/custom/pattern-cycle-detection-expected.txt:
  • platform/qt/svg/custom/pattern-deep-referencing-expected.png:
  • platform/qt/svg/custom/pattern-deep-referencing-expected.txt:
  • platform/qt/svg/custom/pattern-incorrect-tiling-expected.png:
  • platform/qt/svg/custom/pattern-incorrect-tiling-expected.txt:
  • platform/qt/svg/custom/pattern-rotate-expected.png:
  • platform/qt/svg/custom/pattern-rotate-expected.txt:
  • platform/qt/svg/custom/pattern-rotate-gaps-expected.png:
  • platform/qt/svg/custom/pattern-rotate-gaps-expected.txt:
  • platform/qt/svg/custom/pattern-userSpaceOnUse-userToBaseTransform-expected.png:
  • platform/qt/svg/custom/pattern-userSpaceOnUse-userToBaseTransform-expected.txt:
  • platform/qt/svg/custom/pattern-with-transformation-expected.png:
  • platform/qt/svg/custom/pattern-with-transformation-expected.txt:
  • platform/qt/svg/custom/percentage-of-html-parent-expected.png:
  • platform/qt/svg/custom/percentage-of-html-parent-expected.txt:
  • platform/qt/svg/custom/pointer-events-image-css-transform-expected.png:
  • platform/qt/svg/custom/pointer-events-image-css-transform-expected.txt:
  • platform/qt/svg/custom/pointer-events-image-expected.png:
  • platform/qt/svg/custom/pointer-events-image-expected.txt:
  • platform/qt/svg/custom/pointer-events-path-expected.png:
  • platform/qt/svg/custom/pointer-events-path-expected.txt:
  • platform/qt/svg/custom/pointer-events-text-css-transform-expected.png:
  • platform/qt/svg/custom/pointer-events-text-css-transform-expected.txt:
  • platform/qt/svg/custom/pointer-events-text-expected.png:
  • platform/qt/svg/custom/pointer-events-text-expected.txt:
  • platform/qt/svg/custom/preserve-aspect-ratio-syntax-expected.png:
  • platform/qt/svg/custom/preserve-aspect-ratio-syntax-expected.txt:
  • platform/qt/svg/custom/recursive-clippath-expected.png:
  • platform/qt/svg/custom/recursive-clippath-expected.txt:
  • platform/qt/svg/custom/recursive-filter-expected.png:
  • platform/qt/svg/custom/recursive-filter-expected.txt:
  • platform/qt/svg/custom/recursive-gradient-expected.png:
  • platform/qt/svg/custom/recursive-gradient-expected.txt:
  • platform/qt/svg/custom/recursive-mask-expected.png:
  • platform/qt/svg/custom/recursive-mask-expected.txt:
  • platform/qt/svg/custom/recursive-pattern-expected.png:
  • platform/qt/svg/custom/recursive-pattern-expected.txt:
  • platform/qt/svg/custom/relative-sized-content-expected.png: Added.
  • platform/qt/svg/custom/relative-sized-content-expected.txt:
  • platform/qt/svg/custom/relative-sized-content-with-resources-expected.png:
  • platform/qt/svg/custom/relative-sized-content-with-resources-expected.txt:
  • platform/qt/svg/custom/relative-sized-deep-shadow-tree-content-expected.png:
  • platform/qt/svg/custom/relative-sized-deep-shadow-tree-content-expected.txt:
  • platform/qt/svg/custom/relative-sized-image-expected.png:
  • platform/qt/svg/custom/relative-sized-image-expected.txt:
  • platform/qt/svg/custom/relative-sized-inner-svg-expected.png:
  • platform/qt/svg/custom/relative-sized-inner-svg-expected.txt:
  • platform/qt/svg/custom/relative-sized-shadow-tree-content-expected.png:
  • platform/qt/svg/custom/relative-sized-shadow-tree-content-expected.txt:
  • platform/qt/svg/custom/relative-sized-shadow-tree-content-with-symbol-expected.png:
  • platform/qt/svg/custom/relative-sized-shadow-tree-content-with-symbol-expected.txt:
  • platform/qt/svg/custom/relative-sized-use-on-symbol-expected.png:
  • platform/qt/svg/custom/relative-sized-use-on-symbol-expected.txt:
  • platform/qt/svg/custom/relative-sized-use-without-attributes-on-symbol-expected.png:
  • platform/qt/svg/custom/relative-sized-use-without-attributes-on-symbol-expected.txt:
  • platform/qt/svg/custom/rootmost-svg-xy-attrs-expected.png:
  • platform/qt/svg/custom/rootmost-svg-xy-attrs-expected.txt:
  • platform/qt/svg/custom/scroll-hit-test-expected.png:
  • platform/qt/svg/custom/scroll-hit-test-expected.txt:
  • platform/qt/svg/custom/scrolling-embedded-svg-file-image-repaint-problem-expected.png:
  • platform/qt/svg/custom/scrolling-embedded-svg-file-image-repaint-problem-expected.txt:
  • platform/qt/svg/custom/second-inline-text-expected.png:
  • platform/qt/svg/custom/second-inline-text-expected.txt:
  • platform/qt/svg/custom/shape-rendering-expected.png:
  • platform/qt/svg/custom/shape-rendering-expected.txt:
  • platform/qt/svg/custom/shapes-supporting-markers-expected.png:
  • platform/qt/svg/custom/shapes-supporting-markers-expected.txt:
  • platform/qt/svg/custom/simple-text-double-shadow-expected.png:
  • platform/qt/svg/custom/simple-text-double-shadow-expected.txt:
  • platform/qt/svg/custom/simpleCDF-expected.png:
  • platform/qt/svg/custom/simpleCDF-expected.txt:
  • platform/qt/svg/custom/stroke-fallback-expected.png:
  • platform/qt/svg/custom/stroke-fallback-expected.txt:
  • platform/qt/svg/custom/stroke-width-large-expected.png:
  • platform/qt/svg/custom/stroke-width-large-expected.txt:
  • platform/qt/svg/custom/stroked-pattern-expected.png:
  • platform/qt/svg/custom/stroked-pattern-expected.txt:
  • platform/qt/svg/custom/style-attribute-font-size-expected.png:
  • platform/qt/svg/custom/style-attribute-font-size-expected.txt:
  • platform/qt/svg/custom/svg-curve-with-relative-cordinates-expected.png:
  • platform/qt/svg/custom/svg-curve-with-relative-cordinates-expected.txt:
  • platform/qt/svg/custom/svg-features-expected.txt:
  • platform/qt/svg/custom/svg-float-border-padding-expected.png:
  • platform/qt/svg/custom/svg-float-border-padding-expected.txt:
  • platform/qt/svg/custom/svg-fonts-in-html-expected.png:
  • platform/qt/svg/custom/svg-fonts-in-html-expected.txt:
  • platform/qt/svg/custom/svg-fonts-in-text-controls-expected.txt:
  • platform/qt/svg/custom/svg-fonts-segmented-expected.png:
  • platform/qt/svg/custom/svg-fonts-segmented-expected.txt:
  • platform/qt/svg/custom/svg-fonts-without-missing-glyph-expected.png:
  • platform/qt/svg/custom/svg-fonts-without-missing-glyph-expected.txt:
  • platform/qt/svg/custom/svg-fonts-word-spacing-expected.png:
  • platform/qt/svg/custom/svg-fonts-word-spacing-expected.txt:
  • platform/qt/svg/custom/svg-overflow-types-expected.png:
  • platform/qt/svg/custom/svg-overflow-types-expected.txt:
  • platform/qt/svg/custom/text-clip-expected.png:
  • platform/qt/svg/custom/text-clip-expected.txt:
  • platform/qt/svg/custom/text-ctm-expected.png:
  • platform/qt/svg/custom/text-ctm-expected.txt:
  • platform/qt/svg/custom/text-decoration-visibility-expected.png:
  • platform/qt/svg/custom/text-decoration-visibility-expected.txt:
  • platform/qt/svg/custom/text-dom-01-f-expected.png:
  • platform/qt/svg/custom/text-dom-01-f-expected.txt:
  • platform/qt/svg/custom/text-filter-expected.png:
  • platform/qt/svg/custom/text-filter-expected.txt:
  • platform/qt/svg/custom/text-hit-test-expected.png:
  • platform/qt/svg/custom/text-hit-test-expected.txt:
  • platform/qt/svg/custom/text-image-opacity-expected.png:
  • platform/qt/svg/custom/text-image-opacity-expected.txt:
  • platform/qt/svg/custom/text-letter-spacing-expected.png:
  • platform/qt/svg/custom/text-letter-spacing-expected.txt:
  • platform/qt/svg/custom/text-linking-expected.png:
  • platform/qt/svg/custom/text-linking-expected.txt:
  • platform/qt/svg/custom/text-repaint-including-stroke-expected.png:
  • platform/qt/svg/custom/text-repaint-including-stroke-expected.txt:
  • platform/qt/svg/custom/text-rotated-gradient-expected.png:
  • platform/qt/svg/custom/text-rotated-gradient-expected.txt:
  • platform/qt/svg/custom/text-rotation-expected.png:
  • platform/qt/svg/custom/text-rotation-expected.txt:
  • platform/qt/svg/custom/text-tref-03-b-change-href-dom-expected.png:
  • platform/qt/svg/custom/text-tref-03-b-change-href-dom-expected.txt:
  • platform/qt/svg/custom/text-tref-03-b-change-href-expected.png:
  • platform/qt/svg/custom/text-tref-03-b-change-href-expected.txt:
  • platform/qt/svg/custom/text-tref-03-b-referenced-element-removal-expected.png:
  • platform/qt/svg/custom/text-tref-03-b-referenced-element-removal-expected.txt:
  • platform/qt/svg/custom/text-tref-03-b-tref-removal-expected.png:
  • platform/qt/svg/custom/text-tref-03-b-tref-removal-expected.txt:
  • platform/qt/svg/custom/text-whitespace-handling-expected.png:
  • platform/qt/svg/custom/text-whitespace-handling-expected.txt:
  • platform/qt/svg/custom/text-x-dx-lists-expected.png:
  • platform/qt/svg/custom/text-x-dx-lists-expected.txt:
  • platform/qt/svg/custom/text-x-dy-lists-expected.png:
  • platform/qt/svg/custom/text-x-dy-lists-expected.txt:
  • platform/qt/svg/custom/text-x-override-in-tspan-child-expected.png:
  • platform/qt/svg/custom/text-x-override-in-tspan-child-expected.txt:
  • platform/qt/svg/custom/text-xy-updates-SVGList-expected.png:
  • platform/qt/svg/custom/text-xy-updates-SVGList-expected.txt:
  • platform/qt/svg/custom/text-xy-updates-expected.png:
  • platform/qt/svg/custom/text-xy-updates-expected.txt:
  • platform/qt/svg/custom/tref-own-content-removal-expected.png:
  • platform/qt/svg/custom/tref-own-content-removal-expected.txt:
  • platform/qt/svg/custom/tref-update-expected.png:
  • platform/qt/svg/custom/tref-update-expected.txt:
  • platform/qt/svg/custom/use-clipped-hit-expected.png:
  • platform/qt/svg/custom/use-clipped-hit-expected.txt:
  • platform/qt/svg/custom/use-detach-expected.png:
  • platform/qt/svg/custom/use-detach-expected.txt:
  • platform/qt/svg/custom/use-dynamic-append-expected.png:
  • platform/qt/svg/custom/use-dynamic-append-expected.txt:
  • platform/qt/svg/custom/use-elementInstance-event-target-expected.png:
  • platform/qt/svg/custom/use-elementInstance-event-target-expected.txt:
  • platform/qt/svg/custom/use-elementInstance-methods-expected.png:
  • platform/qt/svg/custom/use-elementInstance-methods-expected.txt:
  • platform/qt/svg/custom/use-event-handler-on-referenced-element-expected.png:
  • platform/qt/svg/custom/use-event-handler-on-referenced-element-expected.txt:
  • platform/qt/svg/custom/use-event-handler-on-use-element-expected.png:
  • platform/qt/svg/custom/use-event-handler-on-use-element-expected.txt:
  • platform/qt/svg/custom/use-events-crash-expected.png:
  • platform/qt/svg/custom/use-events-crash-expected.txt:
  • platform/qt/svg/custom/use-instanceRoot-modifications-expected.png:
  • platform/qt/svg/custom/use-instanceRoot-modifications-expected.txt:
  • platform/qt/svg/custom/use-modify-container-in-target-expected.png:
  • platform/qt/svg/custom/use-modify-container-in-target-expected.txt:
  • platform/qt/svg/custom/use-modify-target-container-expected.png:
  • platform/qt/svg/custom/use-modify-target-container-expected.txt:
  • platform/qt/svg/custom/use-modify-target-symbol-expected.png:
  • platform/qt/svg/custom/use-modify-target-symbol-expected.txt:
  • platform/qt/svg/custom/use-on-disallowed-foreign-object-1-expected.png:
  • platform/qt/svg/custom/use-on-disallowed-foreign-object-1-expected.txt:
  • platform/qt/svg/custom/use-on-disallowed-foreign-object-2-expected.png:
  • platform/qt/svg/custom/use-on-disallowed-foreign-object-2-expected.txt:
  • platform/qt/svg/custom/use-on-disallowed-foreign-object-3-expected.png:
  • platform/qt/svg/custom/use-on-disallowed-foreign-object-3-expected.txt:
  • platform/qt/svg/custom/use-on-disallowed-foreign-object-4-expected.png:
  • platform/qt/svg/custom/use-on-disallowed-foreign-object-4-expected.txt:
  • platform/qt/svg/custom/use-on-disallowed-foreign-object-5-expected.png:
  • platform/qt/svg/custom/use-on-disallowed-foreign-object-5-expected.txt:
  • platform/qt/svg/custom/use-on-disallowed-foreign-object-6-expected.png:
  • platform/qt/svg/custom/use-on-disallowed-foreign-object-6-expected.txt:
  • platform/qt/svg/custom/use-on-g-containing-symbol-expected.png:
  • platform/qt/svg/custom/use-on-g-containing-symbol-expected.txt:
  • platform/qt/svg/custom/use-on-g-containing-use-expected.png:
  • platform/qt/svg/custom/use-on-g-containing-use-expected.txt:
  • platform/qt/svg/custom/use-on-g-expected.png:
  • platform/qt/svg/custom/use-on-g-expected.txt:
  • platform/qt/svg/custom/use-on-non-svg-namespaced-element-expected.png:
  • platform/qt/svg/custom/use-on-non-svg-namespaced-element-expected.txt:
  • platform/qt/svg/custom/use-on-rect-expected.png:
  • platform/qt/svg/custom/use-on-rect-expected.txt:
  • platform/qt/svg/custom/use-on-symbol-expected.png:
  • platform/qt/svg/custom/use-on-symbol-expected.txt:
  • platform/qt/svg/custom/use-on-text-expected.png:
  • platform/qt/svg/custom/use-on-text-expected.txt:
  • platform/qt/svg/custom/use-on-use-expected.png:
  • platform/qt/svg/custom/use-on-use-expected.txt:
  • platform/qt/svg/custom/use-property-changes-through-dom-expected.png:
  • platform/qt/svg/custom/use-property-changes-through-dom-expected.txt:
  • platform/qt/svg/custom/use-property-changes-through-svg-dom-expected.png:
  • platform/qt/svg/custom/use-property-changes-through-svg-dom-expected.txt:
  • platform/qt/svg/custom/use-recursion-1-expected.png:
  • platform/qt/svg/custom/use-recursion-1-expected.txt:
  • platform/qt/svg/custom/use-recursion-2-expected.png:
  • platform/qt/svg/custom/use-recursion-2-expected.txt:
  • platform/qt/svg/custom/use-recursion-3-expected.png:
  • platform/qt/svg/custom/use-recursion-3-expected.txt:
  • platform/qt/svg/custom/use-recursion-4-expected.png:
  • platform/qt/svg/custom/use-recursion-4-expected.txt:
  • platform/qt/svg/custom/use-referencing-nonexisting-symbol-expected.png:
  • platform/qt/svg/custom/use-referencing-nonexisting-symbol-expected.txt:
  • platform/qt/svg/custom/use-setAttribute-crash-expected.png:
  • platform/qt/svg/custom/use-setAttribute-crash-expected.txt:
  • platform/qt/svg/custom/use-transform-expected.png:
  • platform/qt/svg/custom/use-transform-expected.txt:
  • platform/qt/svg/custom/viewbox-syntax-expected.png:
  • platform/qt/svg/custom/viewbox-syntax-expected.txt:
  • platform/qt/svg/custom/viewport-em-expected.png:
  • platform/qt/svg/custom/viewport-em-expected.txt:
11:23 AM Changeset in webkit [123895] by dpranke@chromium.org
  • 10 edits in trunk/Tools

nrwt: print filtered stderr after a crash correctly
https://bugs.webkit.org/show_bug.cgi?id=92428

Reviewed by Adam Barth.

Some ports will run the stderr output from DRT/WTR through a
filter after a crash in order to print a more useful stack
trace; while that gets written to the crash log correctly, we
don't log it to the debug output properly. This change fixes
that and makes it clearer that we're logging stderr, not the
crash log.

  • Scripts/webkitpy/layout_tests/controllers/single_test_runner.py:

(SingleTestRunner._handle_error):

  • Scripts/webkitpy/layout_tests/port/base.py:

(Port._get_crash_log):

  • Scripts/webkitpy/layout_tests/port/chromium.py:

(ChromiumPort._get_crash_log):

  • Scripts/webkitpy/layout_tests/port/chromium_android_unittest.py:

(ChromiumAndroidPortTest.test_get_crash_log):

  • Scripts/webkitpy/layout_tests/port/driver.py:

(Driver.run_test):

  • Scripts/webkitpy/layout_tests/port/gtk.py:
  • Scripts/webkitpy/layout_tests/port/gtk_unittest.py:
  • Scripts/webkitpy/layout_tests/port/mac.py:

(MacPort._get_crash_log):

  • Scripts/webkitpy/layout_tests/port/port_testcase.py:

(PortTestCase.test_get_crash_log):

11:22 AM Changeset in webkit [123894] by noam.rosenthal@nokia.com
  • 2 edits in trunk/Source/WebCore

[Texmap] Performance regression in texture uploads after r121223
https://bugs.webkit.org/show_bug.cgi?id=91897

Reviewed by Jocelyn Turcotte.

Rolling out r121223, with some conflict fixes.

  • platform/graphics/texmap/TextureMapperGL.cpp:

(WebCore::swizzleBGRAToRGBA):
(WebCore):
(WebCore::driverSupportsBGRASwizzling):
(WebCore::BitmapTextureGL::didReset):
(WebCore::BitmapTextureGL::updateContents):

11:21 AM Changeset in webkit [123893] by dpranke@chromium.org
  • 10 edits
    3 adds in trunk/Tools

nrwt: move image diffing code to a separate module
https://bugs.webkit.org/show_bug.cgi?id=92447

Reviewed by Ryosuke Niwa.

This patch moves the code to talk to ImageDiff into its own
module, and adds more tests for it. In addition, the patch
modifies diff_image() so that we don't automatically stop
ImageDiff after a single invocation, and thus subsequent
diffs may be slightly faster. (Note that the chromium ports
don't use any of this code; that is not changed by this patch).

The main motivation for this change is to move more "generic"
code out of the port/* classes, and in particular to move more
code out of webkit.py so that we can eventually eliminate it by
merging it into base.py.

This patch also splits MockServerProcess out from driver_unittest.py
so that it can be re-used.

  • Scripts/webkitpy/layout_tests/port/base.py:

(Port.init):
(Port.diff_image):
(Port.clean_up_test_run):

  • Scripts/webkitpy/layout_tests/port/driver.py:

(Driver.init):
(Driver._start):

  • Scripts/webkitpy/layout_tests/port/driver_unittest.py:

(DriverTest.test_stop_cleans_up_properly):
(DriverTest.test_two_starts_cleans_up_properly):
(DriverTest.test_start_actually_starts):

  • Scripts/webkitpy/layout_tests/port/efl.py:

(EflPort.clean_up_test_run):

  • Scripts/webkitpy/layout_tests/port/gtk.py:

(GtkPort.clean_up_test_run):

  • Scripts/webkitpy/layout_tests/port/image_diff.py: Added.

(ImageDiffer):
(ImageDiffer.init):
(ImageDiffer.diff_image):
(ImageDiffer._start):
(ImageDiffer._read):
(ImageDiffer.stop):

  • Scripts/webkitpy/layout_tests/port/image_diff_unittest.py: Added.

(for):
(FakePort):
(FakePort.init):
(FakePort._path_to_image_diff):
(FakePort.setup_environ_for_server):
(TestImageDiffer):
(TestImageDiffer.test_diff_image):

  • Scripts/webkitpy/layout_tests/port/mock_drt_unittest.py:

(MockDRTPortTest.test_diff_image):

  • Scripts/webkitpy/layout_tests/port/port_testcase.py:

(PortTestCase.test_diff_imagemissing_both):
(PortTestCase.test_diff_image):
(PortTestCase.test_diff_image.make_proc):

  • Scripts/webkitpy/layout_tests/port/server_process.py:

(ServerProcess._start):

  • Scripts/webkitpy/layout_tests/port/server_process_mock.py: Added.

(MockServerProcess):
(MockServerProcess.init):
(MockServerProcess.write):
(MockServerProcess.has_crashed):
(MockServerProcess.read_stdout_line):
(MockServerProcess.read_stdout):
(MockServerProcess.pop_all_buffered_stderr):
(MockServerProcess.read_either_stdout_or_stderr_line):
(MockServerProcess.start):
(MockServerProcess.stop):
(MockServerProcess.kill):

  • Scripts/webkitpy/layout_tests/port/webkit.py:

(WebKitPort._build_driver_flags):
(WebKitPort._symbols_string):

11:17 AM Changeset in webkit [123892] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

Activate committer bit granted April 6th so I can garden.
https://bugs.webkit.org/show_bug.cgi?id=92500

Patch by Tom Hudson <hudson@google.com> on 2012-07-27
Reviewed by Jochen Eisinger.

  • Scripts/webkitpy/common/config/committers.py:
11:13 AM Changeset in webkit [123891] by commit-queue@webkit.org
  • 5 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Should support onbeforeunload event and show confirmation dialog
https://bugs.webkit.org/show_bug.cgi?id=92510

Patch by Yong Li <yoli@rim.com> on 2012-07-27
Reviewed by Antonio Gomes.

RIM PR# 155878.

  1. Export dispatchBeforeUnloadEvent() so client can call it before going to close the page.
  2. Forward runBeforeUnloadConfirmPanel() call to client so it can show a dialog.
  • Api/WebPage.cpp:

(BlackBerry::WebKit::WebPage::dispatchBeforeUnloadEvent):
(WebKit):

  • Api/WebPage.h:
  • Api/WebPageClient.h:
  • WebCoreSupport/ChromeClientBlackBerry.cpp:

(WebCore::ChromeClientBlackBerry::canRunBeforeUnloadConfirmPanel):
(WebCore::ChromeClientBlackBerry::runBeforeUnloadConfirmPanel):

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

[BlackBerry] Update NetworkJob::sendRequestWithCredentials to use new getProxyAddress API
https://bugs.webkit.org/show_bug.cgi?id=92457

Patch by Joe Mason <jmason@rim.com> on 2012-07-27
Reviewed by Yong Li.

getProxyAddress now returns "host:port", and getProxyPort is gone. Update NetworkJob to use
the new interface.

RIM PR: 176166
Internally reviewed by Jonathan Dong <Jonathan Dong>

  • platform/network/blackberry/NetworkJob.cpp:

(WebCore::NetworkJob::sendRequestWithCredentials):

10:59 AM Changeset in webkit [123889] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk

Improve touch adjustment for targetting small controls.
https://bugs.webkit.org/show_bug.cgi?id=92293

Patch by Kevin Ellis <kevers@chromium.org> on 2012-07-27
Reviewed by Antonio Gomes.

Source/WebCore:

Modifies the touch adjustment algorithm to better discriminate small targets without introducing
a bias towards shorter links. The revised scoring algorihtm uses a normalized distance to
center-line score and a normalized overlap score. The better of the two scores is used for
determining the best candidate for target adjustment. The center-line score works well
for discriminating elements with partial overlaps, where relying solely on percent overlap
could bias towards the smaller element. Conversely, a high percentage in overlap provides a
strong measure of confidence in a candidate target. An additonal tie-break algorithm is
introduced to minimize the adjustment distance if two equally suitable candidates are found.

Test: touchadjustment/small-target-test.html

  • page/TouchAdjustment.cpp:

(TouchAdjustment):
(WebCore::TouchAdjustment::nodeRespondsToTapGesture):
(WebCore::TouchAdjustment::hybridDistanceFunction):
(WebCore::TouchAdjustment::findNodeWithLowestDistanceMetric):
(WebCore::findBestClickableCandidate):

LayoutTests:

Adds a test for cases that were problematic for the touch adjustment algorithm. Each example
in the test involves a small control immediately adjacent to a larger control. When the touch
area is significantly larger than the small control, it was possible for the touch element to
completely overlap the control yet select an adjacent element with a partial overlap.

  • touchadjustment/small-target-test-expected.txt: Added.
  • touchadjustment/small-target-test.html: Added.
10:56 AM Changeset in webkit [123888] by commit-queue@webkit.org
  • 4 edits in trunk/Source

[chromium] Calendar for input type=date should be larger on devices that support touch input.
https://bugs.webkit.org/show_bug.cgi?id=92424

Patch by Kevin Ellis <kevers@chromium.org> on 2012-07-27
Reviewed by Kent Tamura.

Source/WebCore:

Adds touch specific CSS rules to enlarge entries in the calendar on
screens that support touch input.

Manually tested with touch support enabled/disabled.

  • Resources/calendarPicker.css:

(@media (pointer:coarse)):

  • html/shadow/CalendarPickerElement.cpp:

(WebCore::CalendarPickerElement::contentSize):

Source/WebKit/chromium:

Plumbs the deviceSupportsTouch setting through to the popup page so
that the Calendar popup can be enlarged on screens that support touch
input.

  • src/WebPagePopupImpl.cpp:

(WebKit::WebPagePopupImpl::initPage):

10:50 AM Changeset in webkit [123887] by commit-queue@webkit.org
  • 1 edit
    2 adds in trunk/LayoutTests

[EFL] Add missing baselines for fast/table/table-row-outline-paint.html
https://bugs.webkit.org/show_bug.cgi?id=92509

Unreviewed EFL gardening, add missing results for
fast/table/table-row-outline-paint.html after r123793.

Patch by Sudarsana Nagineni <sudarsana.nagineni@linux.intel.com> on 2012-07-27

  • platform/efl/fast/table/table-row-outline-paint-expected.png: Added.
  • platform/efl/fast/table/table-row-outline-paint-expected.txt: Added.
10:48 AM Changeset in webkit [123886] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

Lose a timer in prerender-remove-after-stop.html
https://bugs.webkit.org/show_bug.cgi?id=92425

Patch by Gavin Peters <gavinp@chromium.org> on 2012-07-27
Reviewed by Nate Chapin.

japhet suggested that the timer wasn't needed, and testing revealed that the test works without it. I've tested that if the bug is reintroduced
that it crashes, and of course it doesn't crash with the fix. So bye-bye setTimeout!

  • fast/dom/HTMLLinkElement/prerender-remove-after-stop.html:
10:39 AM Changeset in webkit [123885] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Switch InputHandler malloc use to fastMalloc for cases that should never fail
https://bugs.webkit.org/show_bug.cgi?id=92508

Patch by Mike Fenton <mifenton@rim.com> on 2012-07-27
Reviewed by Yong Li.

Replace common uses of malloc with fastMalloc rather
than trying to recover gracefully. If we are truly
out of memory, crash before corruption occurs.

  • WebKitSupport/InputHandler.cpp:

(BlackBerry::WebKit::InputHandler::spannableTextInRange):
(BlackBerry::WebKit::InputHandler::extractedTextRequest):

10:39 AM Changeset in webkit [123884] by Chris Fleizach
  • 2 edits in trunk/Source/WebCore

WebKit should expose @title as label (AXTitle or AXDescription) sometimes instead of AXHelp, according to the ARIA text alt computation
https://bugs.webkit.org/show_bug.cgi?id=91911

Reviewed by Darin Adler.

Update comments around code to explain rationale.

  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::helpText):
(WebCore::AccessibilityRenderObject::accessibilityDescription):

10:34 AM Changeset in webkit [123883] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit/efl

[EFL] Regression(r123858): Default navigation policy decision was switched to "reject"
https://bugs.webkit.org/show_bug.cgi?id=92507

Patch by Christophe Dumez <Christophe Dumez> on 2012-07-27
Reviewed by Kentaro Hara.

After r92466, the default navigation policy is to reject the request if the
client does not make a decision explicitly. The default policy should be
to accept. This broke ewk_view unit tests.

  • ewk/ewk_view.cpp:

(ewk_view_navigation_policy_decision):

10:24 AM Changeset in webkit [123882] by commit-queue@webkit.org
  • 15 edits
    7 adds in trunk/Source/WebKit2

[EFL][WK2] Add download client for Ewk_Context
https://bugs.webkit.org/show_bug.cgi?id=91345

Patch by Christophe Dumez <Christophe Dumez> on 2012-07-27
Reviewed by Kenneth Rohde Christiansen.

Add download client for Ewk_Context so that
information about downloads can be reported
to the client application via Ewk_View signals.

The client application needs to listen for
"download,request" signal on the view and set
the download path for the download in the
callback in order to accept it. If the signal
is ignored or if the download path is not set
the download will fail.

A new Ewk_Download_Job type is introduced to provide
information relative to a download to the client
and to allow the client to interact with it (e.g.
set its download path, cancel it, ...).

  • PlatformEfl.cmake:
  • UIProcess/API/efl/EWebKit2.h:
  • UIProcess/API/efl/PageClientImpl.cpp:

(WebKit::PageClientImpl::handleDownloadRequest):
(WebKit):

  • UIProcess/API/efl/PageClientImpl.h:

(PageClientImpl):

  • UIProcess/API/efl/ewk_context.cpp:

(_Ewk_Context):
(_Ewk_Context::_Ewk_Context):
(_Ewk_Context::~_Ewk_Context):
(ewk_context_download_job_add):
(ewk_context_download_job_get):
(ewk_context_download_job_remove):

  • UIProcess/API/efl/ewk_context_download_client.cpp: Added.

(toEwkContext):
(decideDestinationWithSuggestedFilename):
(didReceiveResponse):
(didCreateDestination):
(didReceiveData):
(didFail):
(didCancel):
(didFinish):
(ewk_context_download_client_attach):

  • UIProcess/API/efl/ewk_context_download_client_private.h: Added.
  • UIProcess/API/efl/ewk_context_private.h:
  • UIProcess/API/efl/ewk_download_job.cpp: Added.

(_Ewk_Download_Job):
(_Ewk_Download_Job::_Ewk_Download_Job):
(_Ewk_Download_Job::~_Ewk_Download_Job):
(ewk_download_job_ref):
(ewk_download_job_unref):
(ewk_download_job_id_get):
(ewk_download_job_view_get):
(ewk_download_job_state_get):
(ewk_download_job_request_get):
(ewk_download_job_response_get):
(ewk_download_job_destination_get):
(ewk_download_job_destination_set):
(ewk_download_job_suggested_filename_get):
(ewk_download_job_cancel):
(ewk_download_job_estimated_progress_get):
(ewk_download_job_elapsed_time_get):
(ewk_download_job_response_set):
(ewk_download_job_suggested_filename_set):
(ewk_download_job_received_data):
(ewk_download_job_state_set):
(ewk_download_job_new):

  • UIProcess/API/efl/ewk_download_job.h: Added.
  • UIProcess/API/efl/ewk_download_job_private.h: Added.

(WebKit):

  • UIProcess/API/efl/ewk_url_response.cpp:

(ewk_url_response_content_length_get): Add a function to
retrieve the content length of a URL response.

  • UIProcess/API/efl/ewk_url_response.h:
  • UIProcess/API/efl/ewk_view.cpp:

(ewk_view_download_job_cancelled):
(ewk_view_download_job_requested):
(ewk_view_download_job_failed):
(ewk_view_download_job_finished):

  • UIProcess/API/efl/ewk_view.h:
  • UIProcess/API/efl/ewk_view_private.h:
  • UIProcess/API/efl/tests/resources/test.pdf: Added.
  • UIProcess/API/efl/tests/test_ewk2_download_job.cpp: Added. This tests

the download functionality.
(DownloadTestData):
(fileExists):
(serverCallback):
(on_download_requested):
(on_download_cancelled):
(on_download_failed):
(on_download_finished):
(TEST_F):

  • UIProcess/PageClient.h:

(PageClient):

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::receivedPolicyDecision):
(WebKit):
(WebKit::WebPageProxy::handleDownloadRequest):

  • UIProcess/WebPageProxy.h:

(WebPageProxy):

10:19 AM Changeset in webkit [123881] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebCore

Add runtime flag to enable/disable JS memory information
https://bugs.webkit.org/show_bug.cgi?id=92479

Patch by Zoltan Horvath <zoltan@webkit.org> on 2012-07-27
Reviewed by Simon Hausmann.

We need console.memory JS object to be accessible because of the memory consumption
measurements of the performance tests, so I added a toggle to InternalSettings.

  • testing/InternalSettings.cpp:

(WebCore::InternalSettings::setMemoryInfoEnabled):
(WebCore):

  • testing/InternalSettings.h:

(InternalSettings):

  • testing/InternalSettings.idl:
10:17 AM Changeset in webkit [123880] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

Microdata: Remove toJs() and toV8Object() custom methods from JSHTMLElementCustom.cpp and V8HTMLElementCustom.cpp respectively.
https://bugs.webkit.org/show_bug.cgi?id=92482

Patch by Arko Saha <arko@motorola.com> on 2012-07-27
Reviewed by Kentaro Hara.

Removed custom methods toJS() and toV8Object(). We should use toJS() method defined in
JSMicroDataItemValueCustom.cpp and toV8() method defined in V8MicroDataItemValueCustom.cpp
in place of custom toJs() and toV8Object().

Existing tests :

fast/dom/MicroData/itemvalue-reflects-data-attr-on-object-element.html
fast/dom/MicroData/itemvalue-reflects-href-attr.html
fast/dom/MicroData/itemvalue-reflects-src-attribute-on-img-element.html
fast/dom/MicroData/itemvalue-reflects-the-content-attr-on-meta-element.html
fast/dom/MicroData/itemvalue-reflects-the-src-attr.html
fast/dom/MicroData/itemvalue-returns-element-itself.html
fast/dom/MicroData/itemvalue-returns-null.html
fast/dom/MicroData/itemvalue-throws-excpetion-onsetting-001.html
fast/dom/MicroData/itemvalue-throws-excpetion-onsetting-002.html

  • bindings/js/JSHTMLElementCustom.cpp:

(WebCore):

  • bindings/v8/custom/V8HTMLElementCustom.cpp:

(WebCore):
(WebCore::V8HTMLElement::itemValueAccessorGetter):

10:14 AM Changeset in webkit [123879] by commit-queue@webkit.org
  • 3 edits in trunk/Source

Source/WebKit: [EFL] WebKit API unit tests needs a timeout
https://bugs.webkit.org/show_bug.cgi?id=92496

Patch by Thiago Marcos P. Santos <thiago.santos@intel.com> on 2012-07-27
Reviewed by Antonio Gomes.

Added the same timeout value that we have for WebKit2.

  • PlatformEfl.cmake:

Source/WTF: Support a rational time class for use by media elements.
https://bugs.webkit.org/show_bug.cgi?id=88787

Patch by Jer Noble <jer.noble@apple.com> on 2012-07-27
Reviewed by Eric Carlson.

Add a new MediaTime class which implements rational math operations.

Add common constructors and assignment operators:

  • wtf/MediaTime.cpp: Added.

(WTF::MediaTime::MediaTime):
(WTF::MediaTime::~MediaTime):
(WTF::MediaTime::operator=):

Add common math operators:

  • wtf/MediaTime.cpp:

(WTF::MediaTime::operator+):
(WTF::MediaTime::operator-):
(WTF::MediaTime::operator<):
(WTF::MediaTime::operator>):
(WTF::MediaTime::operator==):
(WTF::MediaTime::operator>=):
(WTF::MediaTime::operator<=):

Add functions to check the MeiaTime time type flags:

  • wtf/MediaTime.h:

(WTF::MediaTime::isValid):
(WTF::MediaTime::isInvalid):
(WTF::MediaTime::hasBeenRounde):
(WTF::MediaTime::isPositiveInfinite):
(WTF::MediaTime::isNegativeInfinite):
(WTF::MediaTime::isIndefinite):

Add constants for commonly used MediaTime values:
(WTF::MediaTime::zeroTime):
(WTF::MediaTime::invalidTime):
(WTF::MediaTime::positiveInfiniteTime):
(WTF::MediaTime::negativeInfiniteTime):
(WTF::MediaTime::indefiniteTime):

Add explicit conversion functions to convert to and from floating point values.

  • wtf/MediaTime.cpp:

(WTF::MediaTime::createWithFloat):
(WTF::MediaTime::createWithDouble):
(WTF::MediaTime::toFloat):
(WTF::MediaTime::toDouble):

Add some useful exported functions:

  • wtf/MediaTime.cpp:

(WTF::MediaTime::compare): Master function for the comparison operators above.
(WTF::MediaTime::setTimeScale): Rescale the time value to a new time scale.
(WTF::abs): Return an absolute value for the current MediaTime.

Static utility functions to implement the above:

  • wtf/MediaTime.cpp:

(WTF::greatestCommonDivisor):
(WTF::leastCommonMultiple):
(WTF::signum):

Windows-only implementations of isinf and signbit:

  • wtf/MediaTime.cpp:

(std::isinf):
(std::signbit):

Add the new class to platform build files:

  • GNUmakefile.list.am:
  • WTF.gypi:
  • WTF.pro:
  • WTF.vcproj/WTF.vcproj:
  • WTF.xcodeproj/project.pbxproj:
9:31 AM Changeset in webkit [123878] by jer.noble@apple.com
  • 14 edits
    3 adds in trunk

Support a rational time class for use by media elements.
https://bugs.webkit.org/show_bug.cgi?id=88787

Source/JavaScriptCore:

Re-export WTF::MediaTime from JavaScriptCore.

Reviewed by Eric Carlson.

Source/WTF:

Reviewed by Eric Carlson.

Add a new MediaTime class which implements rational math operations.

Add common constructors and assignment operators:

  • wtf/MediaTime.cpp: Added.

(WTF::MediaTime::MediaTime):
(WTF::MediaTime::~MediaTime):
(WTF::MediaTime::operator=):

Add common math operators:

  • wtf/MediaTime.cpp:

(WTF::MediaTime::operator+):
(WTF::MediaTime::operator-):
(WTF::MediaTime::operator<):
(WTF::MediaTime::operator>):
(WTF::MediaTime::operator==):
(WTF::MediaTime::operator>=):
(WTF::MediaTime::operator<=):

Add functions to check the MeiaTime time type flags:

  • wtf/MediaTime.h:

(WTF::MediaTime::isValid):
(WTF::MediaTime::isInvalid):
(WTF::MediaTime::hasBeenRounde):
(WTF::MediaTime::isPositiveInfinite):
(WTF::MediaTime::isNegativeInfinite):
(WTF::MediaTime::isIndefinite):

Add constants for commonly used MediaTime values:
(WTF::MediaTime::zeroTime):
(WTF::MediaTime::invalidTime):
(WTF::MediaTime::positiveInfiniteTime):
(WTF::MediaTime::negativeInfiniteTime):
(WTF::MediaTime::indefiniteTime):

Add explicit conversion functions to convert to and from floating point values.

  • wtf/MediaTime.cpp:

(WTF::MediaTime::createWithFloat):
(WTF::MediaTime::createWithDouble):
(WTF::MediaTime::toFloat):
(WTF::MediaTime::toDouble):

Add some useful exported functions:

  • wtf/MediaTime.cpp:

(WTF::MediaTime::compare): Master function for the comparison operators above.
(WTF::MediaTime::setTimeScale): Rescale the time value to a new time scale.
(WTF::abs): Return an absolute value for the current MediaTime.

Static utility functions to implement the above:

  • wtf/MediaTime.cpp:

(WTF::greatestCommonDivisor):
(WTF::leastCommonMultiple):
(WTF::signum):

Windows-only implementations of isinf and signbit:

  • wtf/MediaTime.cpp:

(std::isinf):
(std::signbit):

Add the new class to platform build files:

  • GNUmakefile.list.am:
  • WTF.gypi:
  • WTF.pro:
  • WTF.vcproj/WTF.vcproj:
  • WTF.xcodeproj/project.pbxproj:

Tools:

Add unit tests for the WTF::MediaTime class.

Reviewed by Eric Carlson.

  • TestWebKitAPI/GNUmakefile.am:
  • TestWebKitAPI/TestWebKitAPI.gypi:
  • TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
  • TestWebKitAPI/Tests/WTF/MediaTime.cpp: Added.
  • TestWebKitAPI/win/TestWebKitAPI.vcproj:

(WTF::operator<<):
(TestWebKitAPI):
(TestWebKitAPI::TEST):

9:22 AM Changeset in webkit [123877] by kbalazs@webkit.org
  • 2 edits in trunk/Tools

[NRWT] should have a way to restrict pixel tests for individual directories
https://bugs.webkit.org/show_bug.cgi?id=91754

Unreviewed attempt to fix the chaos I introduced.

  • DumpRenderTree/qt/DumpRenderTreeQt.cpp:

(WebCore::DumpRenderTree::processLine):

7:15 AM Changeset in webkit [123876] by zherczeg@webkit.org
  • 2 edits in trunk/LayoutTests

[Qt][WK2] Gardening after r123786. It made 5 fast/animation tests fail
https://bugs.webkit.org/show_bug.cgi?id=92490

Patch by Balazs Ankes <bank@inf.u-szeged.hu> on 2012-07-27
Rubber stamped by Zoltan Herczeg.

  • platform/qt-5.0-wk2/Skipped:
7:14 AM Changeset in webkit [123875] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WTF

set WTF_USE_LOCKFREE_THREADSAFEREFCOUNTED for chromium android
https://bugs.webkit.org/show_bug.cgi?id=91746

Patch by Wei James <james.wei@intel.com> on 2012-07-27
Reviewed by Adam Barth.

This patch is part of efforts to enable web audio for chromium android.
Web audio component needs to use atomicIncrement and atomicDecrement,
which are enabled by this MACRO.

As mentioned in https://bugs.webkit.org/show_bug.cgi?id=89428#c19, make
this patch standalone.

  • wtf/Atomics.h:

(WTF):

7:04 AM Changeset in webkit [123874] by commit-queue@webkit.org
  • 61 edits in trunk/LayoutTests

[EFL] Rebaseline after harfbuzz-ng complex font support is enabled
https://bugs.webkit.org/show_bug.cgi?id=92120

Unreviewed, EFL rebaselining.

Now that we have complex font support using harfbuzz, we need a couple of new expected results.

Patch by Dominik Röttsches <dominik.rottsches@intel.com> on 2012-07-27

  • platform/efl/TestExpectations:
  • platform/efl/fast/dom/52776-expected.png:
  • platform/efl/fast/dom/52776-expected.txt:
  • platform/efl/fast/text/atsui-kerning-and-ligatures-expected.png:
  • platform/efl/fast/text/atsui-kerning-and-ligatures-expected.txt:
  • platform/efl/fast/text/atsui-multiple-renderers-expected.png:
  • platform/efl/fast/text/atsui-multiple-renderers-expected.txt:
  • platform/efl/fast/text/atsui-negative-spacing-features-expected.png:
  • platform/efl/fast/text/atsui-negative-spacing-features-expected.txt:
  • platform/efl/fast/text/atsui-partial-selection-expected.png:
  • platform/efl/fast/text/atsui-partial-selection-expected.txt:
  • platform/efl/fast/text/atsui-pointtooffset-calls-cg-expected.png:
  • platform/efl/fast/text/atsui-pointtooffset-calls-cg-expected.txt:
  • platform/efl/fast/text/atsui-rtl-override-selection-expected.png:
  • platform/efl/fast/text/atsui-rtl-override-selection-expected.txt:
  • platform/efl/fast/text/atsui-small-caps-punctuation-size-expected.png:
  • platform/efl/fast/text/atsui-small-caps-punctuation-size-expected.txt:
  • platform/efl/fast/text/atsui-spacing-features-expected.png:
  • platform/efl/fast/text/atsui-spacing-features-expected.txt:
  • platform/efl/fast/text/cg-vs-atsui-expected.png:
  • platform/efl/fast/text/cg-vs-atsui-expected.txt:
  • platform/efl/fast/text/complex-preferred-logical-widths-expected.png:
  • platform/efl/fast/text/complex-preferred-logical-widths-expected.txt:
  • platform/efl/fast/text/justify-ideograph-leading-expansion-expected.png:
  • platform/efl/fast/text/justify-ideograph-leading-expansion-expected.txt:
  • platform/efl/fast/text/midword-break-before-surrogate-pair-2-expected.png:
  • platform/efl/fast/text/midword-break-before-surrogate-pair-2-expected.txt:
  • platform/efl/fast/text/should-use-atsui-expected.png:
  • platform/efl/fast/text/should-use-atsui-expected.txt:
  • platform/efl/fast/text/stroking-decorations-expected.png:
  • platform/efl/fast/text/stroking-decorations-expected.txt:
  • platform/efl/fast/text/stroking-expected.png:
  • platform/efl/fast/text/stroking-expected.txt:
  • platform/efl/fast/text/wide-zero-width-space-expected.png:
  • platform/efl/fast/text/wide-zero-width-space-expected.txt:
  • platform/efl/svg/W3C-I18N/g-dirLTR-ubNone-expected.txt:
  • platform/efl/svg/W3C-I18N/g-dirLTR-ubOverride-expected.txt:
  • platform/efl/svg/W3C-I18N/text-dirLTR-ubNone-expected.txt:
  • platform/efl/svg/W3C-I18N/text-dirLTR-ubOverride-expected.txt:
  • platform/efl/svg/W3C-I18N/text-dirRTL-ubNone-expected.txt:
  • platform/efl/svg/W3C-I18N/text-dirRTL-ubOverride-expected.txt:
  • platform/efl/svg/W3C-I18N/tspan-dirLTR-ubEmbed-in-rtl-context-expected.txt:
  • platform/efl/svg/W3C-I18N/tspan-dirLTR-ubNone-in-rtl-context-expected.txt:
  • platform/efl/svg/W3C-I18N/tspan-dirLTR-ubOverride-in-default-context-expected.txt:
  • platform/efl/svg/W3C-I18N/tspan-dirLTR-ubOverride-in-ltr-context-expected.txt:
  • platform/efl/svg/W3C-I18N/tspan-dirLTR-ubOverride-in-rtl-context-expected.txt:
  • platform/efl/svg/W3C-I18N/tspan-dirNone-ubOverride-in-default-context-expected.txt:
  • platform/efl/svg/W3C-I18N/tspan-dirNone-ubOverride-in-ltr-context-expected.txt:
  • platform/efl/svg/W3C-I18N/tspan-dirNone-ubOverride-in-rtl-context-expected.txt:
  • platform/efl/svg/W3C-I18N/tspan-dirRTL-ubEmbed-in-default-context-expected.txt:
  • platform/efl/svg/W3C-I18N/tspan-dirRTL-ubEmbed-in-ltr-context-expected.txt:
  • platform/efl/svg/W3C-I18N/tspan-dirRTL-ubNone-in-default-context-expected.txt:
  • platform/efl/svg/W3C-I18N/tspan-dirRTL-ubNone-in-ltr-context-expected.txt:
  • platform/efl/svg/W3C-I18N/tspan-dirRTL-ubOverride-in-default-context-expected.txt:
  • platform/efl/svg/W3C-I18N/tspan-dirRTL-ubOverride-in-ltr-context-expected.txt:
  • platform/efl/svg/W3C-I18N/tspan-dirRTL-ubOverride-in-rtl-context-expected.txt:
  • platform/efl/svg/W3C-I18N/tspan-direction-ltr-expected.txt:
  • platform/efl/svg/W3C-I18N/tspan-direction-rtl-expected.txt:
  • platform/efl/transforms/2d/hindi-rotated-expected.png:
  • platform/efl/transforms/2d/hindi-rotated-expected.txt:
6:56 AM Changeset in webkit [123873] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

REGRESSION(r123856): Breaks the build when disabling inspector
https://bugs.webkit.org/show_bug.cgi?id=92493

Patch by Thiago Marcos P. Santos <thiago.santos@intel.com> on 2012-07-27
Reviewed by Kentaro Hara.

  • bindings/js/ScriptGCEvent.h:

(WebCore):

  • bindings/v8/ScriptGCEvent.h:

(WebCore):

6:45 AM Changeset in webkit [123872] by commit-queue@webkit.org
  • 8 edits
    1 copy in trunk

[WK2][EFL] Add an ACCELERATED_COMPOSITING implementation for Efl WebKit2
https://bugs.webkit.org/show_bug.cgi?id=91581

Patch by YoungTaeck Song <youngtaeck.song@samsung.com> on 2012-07-27
Reviewed by Noam Rosenthal.

Source/WebKit2:

This patch is a subset of Efl's UI_SIDE_COMPOSITING implementation.
When enter accelerated compositing mode, create evas_gl, evas_gl_context, and evas_gl_surface.

  • PlatformEfl.cmake:
  • Shared/LayerTreeContext.h:

(LayerTreeContext):

  • UIProcess/API/efl/PageClientImpl.cpp:

(WebKit::PageClientImpl::viewSize):
(WebKit::PageClientImpl::enterAcceleratedCompositingMode):
(WebKit::PageClientImpl::exitAcceleratedCompositingMode):

  • UIProcess/API/efl/ewk_view.cpp:

(_Ewk_View_Private_Data):
(_Ewk_View_Private_Data::_Ewk_View_Private_Data):
(ewk_view_size_get):
(ewk_view_create_gl_surface):
(ewk_view_enter_accelerated_compositing_mode):
(ewk_view_exit_accelerated_compositing_mode):
(_ewk_view_smart_calculate):

  • UIProcess/API/efl/ewk_view_private.h:

Tools:

Add a MiniBrowser's option for selecting evas engine.

  • MiniBrowser/efl/main.c:

(browserCreate):
(main):

6:41 AM Changeset in webkit [123871] by kbalazs@webkit.org
  • 14 edits in trunk/Tools

Unreviewed, rolling out r123869.
http://trac.webkit.org/changeset/123869
https://bugs.webkit.org/show_bug.cgi?id=92501

"it did not fix anything but made things even worst"
(Requested by kbalazs on #webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-07-27

  • DumpRenderTree/qt/DumpRenderTreeQt.cpp:

(WebCore::DumpRenderTree::DumpRenderTree):
(WebCore::DumpRenderTree::open):
(WebCore::DumpRenderTree::processLine):
(WebCore::DumpRenderTree::setDumpPixelsForAllTests):
(WebCore::DumpRenderTree::dump):

  • DumpRenderTree/qt/DumpRenderTreeQt.h:

(DumpRenderTree):

  • DumpRenderTree/qt/main.cpp:

(main):

  • Scripts/webkitpy/layout_tests/controllers/single_test_runner.py:

(SingleTestRunner._should_fetch_expected_checksum):
(SingleTestRunner._overwrite_baselines):
(SingleTestRunner._compare_output):

  • Scripts/webkitpy/layout_tests/controllers/worker.py:

(Worker._update_test_input):

  • Scripts/webkitpy/layout_tests/port/base.py:

(Port.supports_switching_pixel_tests_per_test):
(Port):
(Port._supports_switching_pixel_tests_per_test):
(Port.should_run_as_pixel_test):
(Port._should_run_as_pixel_test):

  • Scripts/webkitpy/layout_tests/port/driver.py:

(Driver.cmd_line):
(Driver._command_from_driver_input):

  • Scripts/webkitpy/layout_tests/port/qt.py:

(QtPort.check_sys_deps):
(QtPort):
(QtPort._supports_switching_pixel_tests_per_test):
(QtPort._should_run_as_pixel_test):
(QtPort._default_pixel_test_directories):

  • Scripts/webkitpy/layout_tests/port/test.py:

(TestPort.supports_switching_pixel_tests_per_test):

  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:

(_set_up_derived_options):
(parse_args):

  • Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py:

(MainTest.test_run_singly_actually_runs_tests):
(MainTest.test_pixel_test_directories):

  • WebKitTestRunner/TestController.cpp:

(WTR::TestController::TestController):
(WTR::TestController::initialize):
(WTR::TestController::runTest):

  • WebKitTestRunner/TestController.h:

(TestController):

6:39 AM Changeset in webkit [123870] by commit-queue@webkit.org
  • 7 edits in trunk/Source

[EFL][WK2] Add API to Ewk_Cookie_Manager to watch for cookie changes
https://bugs.webkit.org/show_bug.cgi?id=92484

Patch by Christophe Dumez <Christophe Dumez> on 2012-07-27
Reviewed by Kenneth Rohde Christiansen.

Source/WebCore:

Make use of soup/CookieStorageSoup.cpp for EFL port instead
of defining empty implementations for cookie storage
functions.

No new tests, no behavior change.

  • PlatformEfl.cmake:
  • platform/efl/TemporaryLinkStubs.cpp:

Source/WebKit2:

Add API tp Ewk_Cookie_Manager so that the client can watch
for cookie changes.

  • UIProcess/API/efl/ewk_cookie_manager.cpp:
  • UIProcess/API/efl/ewk_cookie_manager.h:
  • UIProcess/API/efl/tests/test_ewk2_cookie_manager.cpp: Add unit tests

for cookie changes watching.

6:11 AM Changeset in webkit [123869] by kbalazs@webkit.org
  • 14 edits in trunk/Tools

[NRWT] should have a way to restrict pixel tests for individual directories
https://bugs.webkit.org/show_bug.cgi?id=91754

Reviewed by Zoltan Herczeg.

Rollout r123729 because it made Qt debug bots crasy.

  • DumpRenderTree/qt/DumpRenderTreeQt.cpp:

(WebCore::DumpRenderTree::DumpRenderTree):
(WebCore::DumpRenderTree::open):
(WebCore::DumpRenderTree::processLine):
(WebCore::DumpRenderTree::setDumpPixels):
(WebCore::DumpRenderTree::dump):

  • DumpRenderTree/qt/DumpRenderTreeQt.h:

(DumpRenderTree):

  • DumpRenderTree/qt/main.cpp:

(main):

  • Scripts/webkitpy/layout_tests/controllers/single_test_runner.py:

(SingleTestRunner._should_fetch_expected_checksum):
(SingleTestRunner._overwrite_baselines):
(SingleTestRunner._compare_output):

  • Scripts/webkitpy/layout_tests/controllers/worker.py:

(Worker._update_test_input):

  • Scripts/webkitpy/layout_tests/port/base.py:

(Port.lookup_virtual_test_args):

  • Scripts/webkitpy/layout_tests/port/driver.py:

(Driver.cmd_line):
(Driver._command_from_driver_input):

  • Scripts/webkitpy/layout_tests/port/qt.py:

(QtPort.check_sys_deps):

  • Scripts/webkitpy/layout_tests/port/test.py:

(TestPort.virtual_test_suites):

  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:

(_set_up_derived_options):
(parse_args):

  • Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py:

(MainTest.test_run_singly_actually_runs_tests):
(MainTest.test_missing_and_unexpected_results):

  • WebKitTestRunner/TestController.cpp:

(WTR::TestController::TestController):
(WTR::TestController::initialize):
(WTR::TestController::runTest):

  • WebKitTestRunner/TestController.h:

(TestController):

5:51 AM Changeset in webkit [123868] by zherczeg@webkit.org
  • 2 edits in trunk/LayoutTests

[Qt] Gardening after r123786. It made 3 fast/animation tests fail
https://bugs.webkit.org/show_bug.cgi?id=92490

Patch by János Badics <János Badics> on 2012-07-27
Rubber stamped by Zoltan Herczeg.

  • platform/qt/Skipped:
5:27 AM Changeset in webkit [123867] by keishi@webkit.org
  • 4 edits in trunk/Source/WebCore

Replace ColorChooserClient::elementRectRelativeToWindow with elementRectRelativeToRootView
https://bugs.webkit.org/show_bug.cgi?id=92488

Reviewed by Kent Tamura.

Changing ColorChooserClient interface to return element rectangle that is relative to root view instead of window.

No new tests. Method is not used yet.

  • html/ColorInputType.cpp:

(WebCore::ColorInputType::elementRectRelativeToRootView): Replaced elementRectRelativeToWindow. Returns element rectangle relative to root view.

  • html/ColorInputType.h:

(ColorInputType):

  • platform/ColorChooserClient.h:

(ColorChooserClient):

5:25 AM EFLWebKit edited by dominik.rottsches@intel.com
ragel (diff)
4:52 AM Changeset in webkit [123866] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit2

[EFL][WK2] Stop using C API to interact with the page in Ewk_View
https://bugs.webkit.org/show_bug.cgi?id=92463

Patch by Christophe Dumez <Christophe Dumez> on 2012-07-27
Reviewed by Simon Hausmann.

Stop using C API to interact with the PageProxy in Ewk_View
and use the PageProxy object directly. This avoids useless
converting to WK type using toAPI() all the time.

  • UIProcess/API/efl/ewk_view.cpp:

(ewk_view_base_add):
(ewk_view_uri_update):
(ewk_view_uri_set):
(ewk_view_reload):
(ewk_view_reload_bypass_cache):
(ewk_view_stop):
(ewk_view_load_progress_get):
(ewk_view_device_pixel_ratio_set):
(ewk_view_theme_get):
(ewk_view_cursor_set):
(ewk_view_back):
(ewk_view_forward):
(ewk_view_intent_deliver):
(ewk_view_back_possible):
(ewk_view_forward_possible):
(ewk_view_setting_encoding_custom_get):
(ewk_view_setting_encoding_custom_set):

4:42 AM Changeset in webkit [123865] by abarth@webkit.org
  • 3 edits in trunk/Tools

webkit.review.bot should run clean-review-queue and clean-pending-commit periodically
https://bugs.webkit.org/show_bug.cgi?id=92472

Reviewed by Eric Seidel.

We need to run these command periodically to keep cruft from
accumulating in bugs.webkit.org. Rather than running them manually, we
should just have a bot run them. This patch has the style-queue run the
commands, which admittedly is a bit odd but it doesn't seem worthwhile
to create another bot specifically for this purposes.

  • Scripts/webkitpy/tool/commands/queues.py:

(AbstractReviewQueue):
(StyleQueue.begin_work_queue):
(StyleQueue):
(StyleQueue.clean_bugzilla):

  • Scripts/webkitpy/tool/commands/queues_unittest.py:

(StyleQueueTest.test_style_queue_with_style_exception):
(test_style_queue_with_watch_list_exception):

4:32 AM Changeset in webkit [123864] by commit-queue@webkit.org
  • 15 edits
    3 adds
    1 delete in trunk

[Cairo] Add complex font drawing using HarfbuzzNG
https://bugs.webkit.org/show_bug.cgi?id=91864

Patch by Dominik Röttsches <dominik.rottsches@intel.com> on 2012-07-27
Reviewed by Simon Hausmann and Martin Robinson.

.:

Configuring Freetype backend to use HarfBuzz by default.

  • Source/cmake/FindHarfBuzz.cmake: Added pkgconfig based discovery of HarfBuzz.
  • Source/cmake/OptionsEfl.cmake: Adding Harfbuzz configuration.

Source/WebCore:

Adding Harfbuzz support to EFL by implementing it with the help of cairo and cairo-ft.
Reusing Chromium's Harfbuzz-NG support.

No new tests, complex font support is covered by existing tests.

  • CMakeLists.txt: Adding new cairo based implementations, removing emtpy FontEfl.cpp
  • WebCore.gypi: Removing FontEfl.cpp
  • platform/graphics/cairo/FontCairoHarfbuzzNG.cpp: Added. New implementation that implements cairo & harfbuzz-ng based complex font drawing.

(WebCore):
(WebCore::Font::drawComplexText):
(WebCore::Font::drawEmphasisMarksForComplexText):
(WebCore::Font::canReturnFallbackFontsForComplexText):
(WebCore::Font::canExpandAroundIdeographsInComplexText):
(WebCore::Font::floatWidthForComplexText):
(WebCore::Font::offsetForPositionForComplexText):
(WebCore::Font::selectionRectForComplexText):

  • platform/graphics/efl/FontEfl.cpp: Removed.
  • platform/graphics/freetype/FontPlatformData.h: Adding a getter that retrieves a harfbuzz face.

(FontPlatformData):

  • platform/graphics/freetype/FontPlatformDataFreeType.cpp: Adding a getter that retrieves a harfbuzz face.

(WebCore::FontPlatformData::operator=):
(WebCore::FontPlatformData::FontPlatformData):
(WebCore):
(WebCore::FontPlatformData::harfbuzzFace):

  • platform/graphics/harfbuzz/ng/HarfBuzzNGFaceCairo.cpp: Added. Cairo-freetype based approach to get complex font metrics.

(WebCore):
(CairoFtFaceLocker):
(WebCore::CairoFtFaceLocker::CairoFtFaceLocker):
(WebCore::CairoFtFaceLocker::lock):
(WebCore::CairoFtFaceLocker::~CairoFtFaceLocker):
(WebCore::floatToHarfBuzzPosition):
(WebCore::doubleToHarfBuzzPosition):
(WebCore::CairoGetGlyphWidthAndExtents):
(WebCore::harfbuzzGetGlyph):
(WebCore::harfbuzzGetGlyphHorizontalAdvance):
(WebCore::harfbuzzGetGlyphHorizontalOrigin):
(WebCore::harfbuzzGetGlyphExtents):
(WebCore::harfbuzzCairoTextGetFontFuncs):
(WebCore::harfbuzzCairoGetTable):
(WebCore::HarfBuzzNGFace::createFace):
(WebCore::HarfBuzzNGFace::createFont):
(WebCore::HarfBuzzShaper::createGlyphBufferAdvance):

Source/WebKit:

Adding includes for harfbuzz folders.

  • CMakeLists.txt:

Source/WebKit2:

Adding includes for harfbuzz folders.

  • CMakeLists.txt:

Tools:

Bringing Harfbuzz support to EFL with this patch, so we need HarfBuzz in the pulled in jhbuild dependencies.

  • WebKitTestRunner/CMakeLists.txt: Adding additional header include directories.
  • efl/jhbuild.modules: Adding source for HarfBuzz release version 0.9.0.
4:12 AM Changeset in webkit [123863] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebKit2

[EFL][WK2] Add more Ewk_View unit tests
https://bugs.webkit.org/show_bug.cgi?id=92407

Patch by Christophe Dumez <Christophe Dumez> on 2012-07-27
Reviewed by Kenneth Rohde Christiansen.

Add more unit tests for WebKit2 Ewk_View API.
In particular, navigation, HTML loading and
device pixel ratio functions are now tested.

  • UIProcess/API/efl/tests/UnitTestUtils/EWK2UnitTestBase.cpp:

(EWK2UnitTest::EWK2UnitTestBase::SetUp): Call ewk_init() instead
of duplicating initialization code.
(EWK2UnitTest::EWK2UnitTestBase::TearDown): call ewk_shutdown().
(TitleChangedData):
(EWK2UnitTest):
(EWK2UnitTest::onTitleChanged):
(EWK2UnitTest::EWK2UnitTestBase::waitUntilTitleChangedTo): Add convenience
method to wait until the view main frame title changes to a given value.

  • UIProcess/API/efl/tests/UnitTestUtils/EWK2UnitTestBase.h:

(EWK2UnitTestBase):

  • UIProcess/API/efl/tests/test_ewk2_view.cpp:

(TEST_F):
(serverCallbackNavigation):

4:09 AM Changeset in webkit [123862] by commit-queue@webkit.org
  • 114 edits
    1 copy
    46 adds
    9 deletes in trunk/LayoutTests

[Qt] platform/qt/editing/pasteboard tests needs update after font change.
https://bugs.webkit.org/show_bug.cgi?id=91399.

Patch by Ádám Kallai <kadam@inf.u-szeged.hu> on 2012-07-27
Reviewed by Csaba Osztrogonác.

  • platform/qt-5.0-wk1/Skipped:
  • platform/qt-5.0-wk2/Skipped:
  • platform/qt-5.0-wk2/editing/pasteboard/4806874-expected.txt: Added.
  • platform/qt-5.0-wk2/editing/pasteboard/4944770-1-expected.png: Removed.
  • platform/qt-5.0-wk2/editing/pasteboard/4944770-1-expected.txt: Removed.
  • platform/qt-5.0-wk2/editing/pasteboard/4944770-2-expected.png: Removed.
  • platform/qt-5.0-wk2/editing/pasteboard/4944770-2-expected.txt: Removed.
  • platform/qt-5.0-wk2/editing/pasteboard/input-field-1-expected.txt: Copied from LayoutTests/platform/qt/editing/pasteboard/input-field-1-expected.txt.
  • platform/qt-5.0/Skipped:
  • platform/qt-5.0/editing/pasteboard/4944770-1-expected.png: Removed.
  • platform/qt-5.0/editing/pasteboard/4944770-1-expected.txt: Removed.
  • platform/qt-5.0/editing/pasteboard/4944770-2-expected.png: Removed.
  • platform/qt-5.0/editing/pasteboard/4944770-2-expected.txt: Removed.
  • platform/qt-5.0/editing/pasteboard/paste-text-at-tabspan-003-expected.txt: Removed.
  • platform/qt/editing/pasteboard/3976872-expected.png: Added.
  • platform/qt/editing/pasteboard/3976872-expected.txt:
  • platform/qt/editing/pasteboard/4076267-2-expected.png:
  • platform/qt/editing/pasteboard/4076267-2-expected.txt:
  • platform/qt/editing/pasteboard/4076267-3-expected.png:
  • platform/qt/editing/pasteboard/4076267-3-expected.txt:
  • platform/qt/editing/pasteboard/4076267-expected.png:
  • platform/qt/editing/pasteboard/4076267-expected.txt:
  • platform/qt/editing/pasteboard/4242293-expected.png: Added.
  • platform/qt/editing/pasteboard/4242293-expected.txt:
  • platform/qt/editing/pasteboard/4631972-expected.png: Added.
  • platform/qt/editing/pasteboard/4631972-expected.txt:
  • platform/qt/editing/pasteboard/4641033-expected.png:
  • platform/qt/editing/pasteboard/4641033-expected.txt:
  • platform/qt/editing/pasteboard/4806874-expected.png:
  • platform/qt/editing/pasteboard/4806874-expected.txt:
  • platform/qt/editing/pasteboard/4944770-1-expected.png:
  • platform/qt/editing/pasteboard/4944770-1-expected.txt:
  • platform/qt/editing/pasteboard/4944770-2-expected.png:
  • platform/qt/editing/pasteboard/4944770-2-expected.txt:
  • platform/qt/editing/pasteboard/4989774-expected.png: Added.
  • platform/qt/editing/pasteboard/4989774-expected.txt:
  • platform/qt/editing/pasteboard/5006779-expected.png: Added.
  • platform/qt/editing/pasteboard/5006779-expected.txt:
  • platform/qt/editing/pasteboard/5028447-expected.png: Added.
  • platform/qt/editing/pasteboard/5028447-expected.txt:
  • platform/qt/editing/pasteboard/5032095-expected.png: Added.
  • platform/qt/editing/pasteboard/5032095-expected.txt:
  • platform/qt/editing/pasteboard/5071074-2-expected.png: Added.
  • platform/qt/editing/pasteboard/5071074-2-expected.txt:
  • platform/qt/editing/pasteboard/5071074-expected.png: Added.
  • platform/qt/editing/pasteboard/5071074-expected.txt:
  • platform/qt/editing/pasteboard/5075944-expected.png: Added.
  • platform/qt/editing/pasteboard/5075944-expected.txt:
  • platform/qt/editing/pasteboard/5134759-expected.png: Added.
  • platform/qt/editing/pasteboard/5134759-expected.txt:
  • platform/qt/editing/pasteboard/5156401-1-expected.png: Added.
  • platform/qt/editing/pasteboard/5156401-1-expected.txt:
  • platform/qt/editing/pasteboard/5247341-expected.png: Added.
  • platform/qt/editing/pasteboard/5247341-expected.txt:
  • platform/qt/editing/pasteboard/5387578-expected.png:
  • platform/qt/editing/pasteboard/5387578-expected.txt:
  • platform/qt/editing/pasteboard/5478250-expected.png:
  • platform/qt/editing/pasteboard/5478250-expected.txt:
  • platform/qt/editing/pasteboard/5601583-1-expected.png: Added.
  • platform/qt/editing/pasteboard/5601583-1-expected.txt:
  • platform/qt/editing/pasteboard/5780697-2-expected.txt:
  • platform/qt/editing/pasteboard/7955-expected.png: Added.
  • platform/qt/editing/pasteboard/7955-expected.txt:
  • platform/qt/editing/pasteboard/8145-1-expected.png: Added.
  • platform/qt/editing/pasteboard/8145-1-expected.txt:
  • platform/qt/editing/pasteboard/8145-2-expected.png: Added.
  • platform/qt/editing/pasteboard/8145-2-expected.txt:
  • platform/qt/editing/pasteboard/8145-3-expected.png:
  • platform/qt/editing/pasteboard/8145-3-expected.txt:
  • platform/qt/editing/pasteboard/bad-placeholder-expected.png:
  • platform/qt/editing/pasteboard/bad-placeholder-expected.txt:
  • platform/qt/editing/pasteboard/block-wrappers-necessary-expected.txt:
  • platform/qt/editing/pasteboard/cut-text-001-expected.png:
  • platform/qt/editing/pasteboard/cut-text-001-expected.txt:
  • platform/qt/editing/pasteboard/displaced-generic-placeholder-expected.png: Added.
  • platform/qt/editing/pasteboard/displaced-generic-placeholder-expected.txt:
  • platform/qt/editing/pasteboard/displaced-placeholder-expected.png: Added.
  • platform/qt/editing/pasteboard/displaced-placeholder-expected.txt:
  • platform/qt/editing/pasteboard/emacs-cntl-y-001-expected.png: Added.
  • platform/qt/editing/pasteboard/emacs-cntl-y-001-expected.txt:
  • platform/qt/editing/pasteboard/innerText-inline-table-expected.png:
  • platform/qt/editing/pasteboard/innerText-inline-table-expected.txt:
  • platform/qt/editing/pasteboard/input-field-1-expected.png:
  • platform/qt/editing/pasteboard/input-field-1-expected.txt:
  • platform/qt/editing/pasteboard/merge-after-delete-1-expected.png: Added.
  • platform/qt/editing/pasteboard/merge-after-delete-1-expected.txt:
  • platform/qt/editing/pasteboard/merge-after-delete-2-expected.png: Added.
  • platform/qt/editing/pasteboard/merge-after-delete-2-expected.txt:
  • platform/qt/editing/pasteboard/merge-after-delete-expected.png: Added.
  • platform/qt/editing/pasteboard/merge-after-delete-expected.txt:
  • platform/qt/editing/pasteboard/merge-end-blockquote-expected.png:
  • platform/qt/editing/pasteboard/merge-end-blockquote-expected.txt:
  • platform/qt/editing/pasteboard/merge-end-borders-expected.png:
  • platform/qt/editing/pasteboard/merge-end-borders-expected.txt:
  • platform/qt/editing/pasteboard/merge-start-blockquote-expected.png: Added.
  • platform/qt/editing/pasteboard/merge-start-blockquote-expected.txt:
  • platform/qt/editing/pasteboard/merge-start-list-expected.png: Added.
  • platform/qt/editing/pasteboard/merge-start-list-expected.txt:
  • platform/qt/editing/pasteboard/paste-4035648-fix-expected.png: Added.
  • platform/qt/editing/pasteboard/paste-4035648-fix-expected.txt:
  • platform/qt/editing/pasteboard/paste-4038267-fix-expected.png: Added.
  • platform/qt/editing/pasteboard/paste-4038267-fix-expected.txt:
  • platform/qt/editing/pasteboard/paste-blockquote-after-blockquote-expected.png: Added.
  • platform/qt/editing/pasteboard/paste-blockquote-after-blockquote-expected.txt:
  • platform/qt/editing/pasteboard/paste-blockquote-into-blockquote-4-expected.png: Added.
  • platform/qt/editing/pasteboard/paste-blockquote-into-blockquote-4-expected.txt:
  • platform/qt/editing/pasteboard/paste-line-endings-001-expected.png: Added.
  • platform/qt/editing/pasteboard/paste-line-endings-001-expected.txt:
  • platform/qt/editing/pasteboard/paste-line-endings-002-expected.png: Added.
  • platform/qt/editing/pasteboard/paste-line-endings-002-expected.txt:
  • platform/qt/editing/pasteboard/paste-line-endings-003-expected.png: Added.
  • platform/qt/editing/pasteboard/paste-line-endings-003-expected.txt:
  • platform/qt/editing/pasteboard/paste-line-endings-004-expected.png: Added.
  • platform/qt/editing/pasteboard/paste-line-endings-004-expected.txt:
  • platform/qt/editing/pasteboard/paste-line-endings-005-expected.png: Added.
  • platform/qt/editing/pasteboard/paste-line-endings-005-expected.txt:
  • platform/qt/editing/pasteboard/paste-line-endings-006-expected.png: Added.
  • platform/qt/editing/pasteboard/paste-line-endings-006-expected.txt:
  • platform/qt/editing/pasteboard/paste-line-endings-007-expected.png:
  • platform/qt/editing/pasteboard/paste-line-endings-007-expected.txt:
  • platform/qt/editing/pasteboard/paste-line-endings-008-expected.png:
  • platform/qt/editing/pasteboard/paste-line-endings-008-expected.txt:
  • platform/qt/editing/pasteboard/paste-line-endings-009-expected.png:
  • platform/qt/editing/pasteboard/paste-line-endings-009-expected.txt:
  • platform/qt/editing/pasteboard/paste-line-endings-010-expected.png:
  • platform/qt/editing/pasteboard/paste-line-endings-010-expected.txt:
  • platform/qt/editing/pasteboard/paste-match-style-001-expected.png:
  • platform/qt/editing/pasteboard/paste-match-style-001-expected.txt:
  • platform/qt/editing/pasteboard/paste-match-style-002-expected.png: Added.
  • platform/qt/editing/pasteboard/paste-match-style-002-expected.txt:
  • platform/qt/editing/pasteboard/paste-table-003-expected.png:
  • platform/qt/editing/pasteboard/paste-table-003-expected.txt:
  • platform/qt/editing/pasteboard/paste-table-cells-expected.png:
  • platform/qt/editing/pasteboard/paste-table-cells-expected.txt:
  • platform/qt/editing/pasteboard/paste-text-004-expected.png:
  • platform/qt/editing/pasteboard/paste-text-004-expected.txt:
  • platform/qt/editing/pasteboard/paste-text-005-expected.png:
  • platform/qt/editing/pasteboard/paste-text-005-expected.txt:
  • platform/qt/editing/pasteboard/paste-text-006-expected.png: Added.
  • platform/qt/editing/pasteboard/paste-text-006-expected.txt:
  • platform/qt/editing/pasteboard/paste-text-007-expected.png: Added.
  • platform/qt/editing/pasteboard/paste-text-007-expected.txt:
  • platform/qt/editing/pasteboard/paste-text-008-expected.png:
  • platform/qt/editing/pasteboard/paste-text-008-expected.txt:
  • platform/qt/editing/pasteboard/paste-text-009-expected.png:
  • platform/qt/editing/pasteboard/paste-text-009-expected.txt:
  • platform/qt/editing/pasteboard/paste-text-013-expected.png: Added.
  • platform/qt/editing/pasteboard/paste-text-013-expected.txt:
  • platform/qt/editing/pasteboard/paste-text-014-expected.png:
  • platform/qt/editing/pasteboard/paste-text-014-expected.txt:
  • platform/qt/editing/pasteboard/paste-text-016-expected.png: Added.
  • platform/qt/editing/pasteboard/paste-text-016-expected.txt:
  • platform/qt/editing/pasteboard/paste-text-019-expected.png:
  • platform/qt/editing/pasteboard/paste-text-019-expected.txt:
  • platform/qt/editing/pasteboard/paste-text-at-tabspan-003-expected.png:
  • platform/qt/editing/pasteboard/paste-text-at-tabspan-003-expected.txt:
  • platform/qt/editing/pasteboard/pasting-object-expected.png:
  • platform/qt/editing/pasteboard/pasting-object-expected.txt:
  • platform/qt/editing/pasteboard/quirks-mode-br-1-expected.png: Added.
  • platform/qt/editing/pasteboard/quirks-mode-br-1-expected.txt:
  • platform/qt/editing/pasteboard/smart-paste-004-expected.txt: Added.
  • platform/qt/editing/pasteboard/smart-paste-007-expected.png:
  • platform/qt/editing/pasteboard/smart-paste-007-expected.txt:
  • platform/qt/editing/pasteboard/styled-element-markup-expected.png: Added.
  • platform/qt/editing/pasteboard/styled-element-markup-expected.txt:
  • platform/qt/editing/pasteboard/undoable-fragment-removes-expected.png: Added.
  • platform/qt/editing/pasteboard/undoable-fragment-removes-expected.txt:
  • platform/qt/editing/pasteboard/unrendered-br-expected.png: Added.
  • platform/qt/editing/pasteboard/unrendered-br-expected.txt:
4:04 AM Changeset in webkit [123861] by peter@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

Unreviewed. Rolled DEPS.

  • DEPS:
3:47 AM Changeset in webkit [123860] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit2

[EFL][WK2] Keyboard events miss key location data.
https://bugs.webkit.org/show_bug.cgi?id=92235

Patch by Mikhail Pozdnyakov <mikhail.pozdnyakov@intel.com> on 2012-07-27
Reviewed by Simon Hausmann.

EFL WK2 did not initialize WebKeyboardEvent::m_isKeypad field.
This caused failure of fast/events/keydown-numpad-keys.html test.

  • Shared/efl/WebEventFactory.cpp: WebKeyboardEvent::m_isKeypad is initialized.

(WebKit::WebEventFactory::createWebKeyboardEvent):

3:44 AM Changeset in webkit [123859] by commit-queue@webkit.org
  • 2 edits
    1 add in trunk/Tools

[WK2][EFL][WTR] TestNetscapePlugin is required.
https://bugs.webkit.org/show_bug.cgi?id=88756

Patch by KwangYong Choi <ky0.choi@samsung.com> on 2012-07-27
Reviewed by Simon Hausmann.

libTestNetscapePlugin.so should be required for DRT/WTR/EFL.

  • CMakeLists.txt:
  • DumpRenderTree/TestNetscapePlugin/CMakeList.txt:
2:55 AM Changeset in webkit [123858] by gyuyoung.kim@samsung.com
  • 6 edits in trunk/Source

[EFL] Fix wrong return value in EWK_VIEW_XXX macro.
https://bugs.webkit.org/show_bug.cgi?id=92466

Reviewed by Simon Hausmann.

There are wrong return values in EWK_VIEW_XXX macros. The wrong return values
should be fixed. In addition, API description is also modified according to returned
value change.

Source/WebKit/efl:

  • ewk/ewk_view.cpp:

(ewk_view_setting_minimum_timer_interval_get):
(ewk_view_navigation_policy_decision):
(ewk_view_page_rect_get):
(ewk_view_setting_enable_xss_auditor_get):

  • ewk/ewk_view.h:

Source/WebKit2:

  • UIProcess/API/efl/ewk_view.cpp:

(ewk_view_uri_set):
(ewk_view_load_progress_get):
(ewk_view_device_pixel_ratio_get):

  • UIProcess/API/efl/ewk_view.h:
2:41 AM Changeset in webkit [123857] by zandobersek@gmail.com
  • 2 edits in trunk/LayoutTests

Unreviewed GTK gardening, adding a text failure expectation for an accessibility
test that started failing after r123767.

  • platform/gtk/TestExpectations:
2:37 AM Changeset in webkit [123856] by abarth@webkit.org
  • 12 edits
    1 copy in trunk/Source

Add a Setting to expose quantized, rate-limited MemoryInfo values
https://bugs.webkit.org/show_bug.cgi?id=80444

Reviewed by Eric Seidel.

We do not currently expose real MemoryInfo objects to the web unless
the user opts in because we're worried that this memory information
could be used in side-channel attacks.

We've gotten feedback from a number of web app developers that this
information is very useful in tracking the performance of their
applications. These developers use the setting in their testing labs
and regression harnesses to catch memory leaks and regressiosn early in
their development cycle.

Some of these developers have experimented with enabling this feature
within their enterprise and have found the memory data from the field
extremely useful in tracking down memory issues that slip through their
testing.

Based on this experience, they've asked whether we can enable this
functionality on a wider scale so they catch even more problems
including problems that don't manifest within their enterprise.
Because we're still worried about side-channel attacks, we don't want
to expose the raw data, so we've talked with these folks in more detail
to understand what information they find most valuable.

This patch is the result of those discussions. In particular, this
patch adds an option to expose quantized and rate-limited memory
information to web pages. Web pages can only learn new data every 20
minutes, which helps mitigate attacks where the attacker compares two
or readings to extract side-channel information. The patch also only
reports 100 distinct memory values, which (combined with the rate
limts) makes it difficult for attackers to learn about small changes in
memory use.

  • page/MemoryInfo.cpp:

(WebCore):
(HeapSizeCache):
(WebCore::HeapSizeCache::HeapSizeCache):
(WebCore::HeapSizeCache::getCachedHeapSize):
(WebCore::HeapSizeCache::maybeUpdate):
(WebCore::HeapSizeCache::update):
(WebCore::HeapSizeCache::quantize):
(WebCore::MemoryInfo::MemoryInfo):

  • page/Settings.cpp:

(WebCore::Settings::Settings):

  • page/Settings.h:

(WebCore::Settings::setQuantizedMemoryInfoEnabled):
(WebCore::Settings::quantizedMemoryInfoEnabled):
(Settings):

2:36 AM Changeset in webkit [123855] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit/chromium

[chromium] Fix issues in visiblePositionForWindowPoint (offscreen points / iframes).
https://bugs.webkit.org/show_bug.cgi?id=79117

Fix two issues in WebFrameImpl::visiblePositionForWindowPoint.

1) If the point is off-screen, the visiblePosition is clipped to the screen.

  • The fix is to add HitTestRequest::IgnoreClipping.

2) The result is incorrect if the point is within an iframe.

  • This is because we are calling convertFromContainingWindow twice: once in windowToContents, and then explicitly as well.
  • This patch removes the explicit call to convertFromContainingWindow.

Patch by Oli Lan <olilan@chromium.org> on 2012-07-27
Reviewed by Adam Barth.

  • src/WebFrameImpl.cpp:

(WebKit::WebFrameImpl::visiblePositionForWindowPoint):

2:32 AM Changeset in webkit [123854] by peter@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

Unreviewed. Rolled DEPS.

  • DEPS:
2:26 AM Changeset in webkit [123853] by vsevik@chromium.org
  • 4 edits in trunk

Source/WebCore: Web Inspector: Move formatting support from JavaScriptSource to UISourceCode.
https://bugs.webkit.org/show_bug.cgi?id=92373

Reviewed by Pavel Feldman.

Moved formatting support from JavaScriptSource to UISourceCode.
Extracted Formatter interface from ScriptFormatter and created IdentityFormatter
implememntation and Formatter.createFormatter(contentType) formatter factory.
Introduced virtual formattedChanged() method on UISourceCode.

  • inspector/front-end/JavaScriptSource.js:

(WebInspector.JavaScriptSource):
(WebInspector.JavaScriptSource.prototype.uiLocationToRawLocation):
(WebInspector.JavaScriptSource.prototype.breakpointStorageId):
(WebInspector.JavaScriptSource.prototype.searchInContent):
(WebInspector.JavaScriptSource.prototype.formattedChanged):

  • inspector/front-end/ScriptFormatter.js:

(WebInspector.Formatter):
(WebInspector.Formatter.createFormatter):
(WebInspector.Formatter.locationToPosition):
(WebInspector.Formatter.positionToLocation):
(WebInspector.Formatter.prototype.formatContent):
(WebInspector.ScriptFormatter):
(WebInspector.IdentityFormatter):
(WebInspector.IdentityFormatter.prototype.formatContent):
(WebInspector.FormatterSourceMappingImpl.prototype.originalToFormatted):
(WebInspector.FormatterSourceMappingImpl.prototype.formattedToOriginal):

  • inspector/front-end/UISourceCode.js:

(WebInspector.UISourceCode):
(WebInspector.UISourceCode.prototype.requestContent):
(WebInspector.UISourceCode.prototype._fireContentAvailable):
(WebInspector.UISourceCode.prototype.uiLocationToRawLocation):
(WebInspector.UISourceCode.prototype.overrideLocation):
(WebInspector.UISourceCode.prototype.togglingFormatter):
(WebInspector.UISourceCode.prototype.formatted):
(WebInspector.UISourceCode.prototype.setFormatted.if):
(WebInspector.UISourceCode.prototype.setFormatted.didGetContent.formattedChanged):
(WebInspector.UISourceCode.prototype.setFormatted.didGetContent):
(WebInspector.UISourceCode.prototype.setFormatted):
(WebInspector.UISourceCode.prototype.createFormatter):
(WebInspector.UISourceCode.prototype.formattedChanged):

Source/WebKit2: [EFL][WK2] Add unit tests for Ewk_View form client
https://bugs.webkit.org/show_bug.cgi?id=92468

Reviewed by Simon Hausmann.

Add unit test for form client functionality in
Ewk_View.

  • UIProcess/API/efl/tests/test_ewk2_view.cpp:

(onFormAboutToBeSubmitted):
(TEST_F):

LayoutTests: Web Inspector: Move formatting support from JavaScriptSource to UISourceCode.
https://bugs.webkit.org/show_bug.cgi?id=92373

Reviewed by Pavel Feldman.

  • inspector/debugger/script-formatter.html:
1:50 AM Changeset in webkit [123852] by vsevik@chromium.org
  • 6 edits in trunk

Web Inspector: Move formatting support from JavaScriptSource to UISourceCode.
https://bugs.webkit.org/show_bug.cgi?id=92373

Reviewed by Pavel Feldman.

Source/WebCore:

Moved formatting support from JavaScriptSource to UISourceCode.
Extracted Formatter interface from ScriptFormatter and created IdentityFormatter
implememntation and Formatter.createFormatter(contentType) formatter factory.
Introduced virtual formattedChanged() method on UISourceCode.

  • inspector/front-end/JavaScriptSource.js:

(WebInspector.JavaScriptSource):
(WebInspector.JavaScriptSource.prototype.uiLocationToRawLocation):
(WebInspector.JavaScriptSource.prototype.breakpointStorageId):
(WebInspector.JavaScriptSource.prototype.searchInContent):
(WebInspector.JavaScriptSource.prototype.formattedChanged):

  • inspector/front-end/ScriptFormatter.js:

(WebInspector.Formatter):
(WebInspector.Formatter.createFormatter):
(WebInspector.Formatter.locationToPosition):
(WebInspector.Formatter.positionToLocation):
(WebInspector.Formatter.prototype.formatContent):
(WebInspector.ScriptFormatter):
(WebInspector.IdentityFormatter):
(WebInspector.IdentityFormatter.prototype.formatContent):
(WebInspector.FormatterSourceMappingImpl.prototype.originalToFormatted):
(WebInspector.FormatterSourceMappingImpl.prototype.formattedToOriginal):

  • inspector/front-end/UISourceCode.js:

(WebInspector.UISourceCode):
(WebInspector.UISourceCode.prototype.requestContent):
(WebInspector.UISourceCode.prototype._fireContentAvailable):
(WebInspector.UISourceCode.prototype.uiLocationToRawLocation):
(WebInspector.UISourceCode.prototype.overrideLocation):
(WebInspector.UISourceCode.prototype.togglingFormatter):
(WebInspector.UISourceCode.prototype.formatted):
(WebInspector.UISourceCode.prototype.setFormatted.if):
(WebInspector.UISourceCode.prototype.setFormatted.didGetContent.formattedChanged):
(WebInspector.UISourceCode.prototype.setFormatted.didGetContent):
(WebInspector.UISourceCode.prototype.setFormatted):
(WebInspector.UISourceCode.prototype.createFormatter):
(WebInspector.UISourceCode.prototype.formattedChanged):

LayoutTests:

  • inspector/debugger/script-formatter.html:
1:41 AM Changeset in webkit [123851] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

[EFL] Memory leaks in EFL DRT
https://bugs.webkit.org/show_bug.cgi?id=92347

Patch by Sudarsana Nagineni <sudarsana.nagineni@linux.intel.com> on 2012-07-27
Reviewed by Simon Hausmann.

Fix a memory leak in EFL's DRT code by adopting an allocation
of JSString created with JSStringCreateWithUTF8CString.

  • DumpRenderTree/efl/DumpRenderTreeChrome.cpp:

(DumpRenderTreeChrome::onWindowObjectCleared):

1:34 AM Changeset in webkit [123850] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Page object should ascertain EditorClient to be non-null
https://bugs.webkit.org/show_bug.cgi?id=92458

Patch by Vivek Galatage <vivekgalatage@gmail.com> on 2012-07-27
Reviewed by Ryosuke Niwa.

The EditorClient should always be available in Page object. Added the assertion to ascertain this.

No new tests. Added assertion to make sure EditorClient always exist.

  • page/Page.cpp:

(WebCore::Page::Page):

1:28 AM Changeset in webkit [123849] by zherczeg@webkit.org
  • 1 edit
    4 adds in trunk/LayoutTests

[Qt] Unreviewed, rebaseline for tests introduced in r123790

Patch by János Badics <János Badics> on 2012-07-27
Rubber stamped by Zoltan Herczeg.

  • platform/qt/css3/filters/effect-brightness-clamping-expected.png: Added.
  • platform/qt/css3/filters/effect-brightness-clamping-expected.txt: Added.
  • platform/qt/css3/filters/effect-brightness-clamping-hw-expected.png: Added.
  • platform/qt/css3/filters/effect-brightness-clamping-hw-expected.txt: Added.
1:25 AM Changeset in webkit [123848] by commit-queue@webkit.org
  • 9 edits in trunk/Source/WebCore

Gather the duplicated timer code into CachedResource.
https://bugs.webkit.org/show_bug.cgi?id=92332

Patch by Huang Dongsung <luxtella@company100.net> on 2012-07-27
Reviewed by Nate Chapin.

Internal review by Jae Hyun Park.

When all clients are removed, CachedImage, CachedScript and CachedCSSStyleSheet
start the timer to destroy decoded data. Those three classes have their own
timer.
Changed CachedCSSStyleSheet::didAddClient to call super class method in order to
stop the timer. This change does not have any side effect because
CachedResource::didAddClient only stops the timer in this case.

No new tests - no new testable functionality.

  • loader/cache/CachedCSSStyleSheet.cpp:

(WebCore::CachedCSSStyleSheet::CachedCSSStyleSheet):
(WebCore::CachedCSSStyleSheet::didAddClient):

  • loader/cache/CachedCSSStyleSheet.h:

(CachedCSSStyleSheet):

  • loader/cache/CachedImage.cpp:

(WebCore::CachedImage::CachedImage):
(WebCore::CachedImage::didAddClient):
(WebCore::CachedImage::allClientsRemoved):

  • loader/cache/CachedImage.h:

(CachedImage):

  • loader/cache/CachedResource.cpp:

(WebCore::CachedResource::CachedResource):
(WebCore::CachedResource::didAddClient):
(WebCore::CachedResource::removeClient):
(WebCore::CachedResource::destroyDecodedDataIfNeeded):
(WebCore):
(WebCore::CachedResource::decodedDataDeletionTimerFired):

  • loader/cache/CachedResource.h:

(CachedResource):

  • loader/cache/CachedScript.cpp:

(WebCore::CachedScript::CachedScript):

  • loader/cache/CachedScript.h:

(CachedScript):

12:56 AM Changeset in webkit [123847] by commit-queue@webkit.org
  • 5 edits in trunk/Source/WebKit/efl

[EFL] Change prototype of run_open_panel
https://bugs.webkit.org/show_bug.cgi?id=91956

Patch by Kihong Kwon <kihong.kwon@samsung.com> on 2012-07-27
Reviewed by Kentaro Hara.

In order to support all of the file chooser attributes,
change the types of parameters in run_open_panel method.
i.e., change "Eina_Bool allows_multiple_files, Eina_List *accept_types"
to "Ewk_File_Chooser *file_chooser"
In addition, Efl can support the capture attribute for HTML media capture.

  • WebCoreSupport/ChromeClientEfl.cpp:

(WebCore::ChromeClientEfl::runOpenPanel):

  • ewk/ewk_view.cpp:

(ewk_view_run_open_panel):

  • ewk/ewk_view.h:
  • ewk/ewk_view_private.h:
12:51 AM Changeset in webkit [123846] by abarth@webkit.org
  • 2 edits in trunk/Websites/webkit.org

Writing a blog post for new reviewers doesn't add value to the blog
https://bugs.webkit.org/show_bug.cgi?id=92305

Reviewed by Eric Seidel.

Now that we have Peter posting weekly updates to the blog, having an
announcement on the blog for each reviewer is more noise than signal.

  • coding/commit-review-policy.html:
12:48 AM Changeset in webkit [123845] by abarth@webkit.org
  • 4 edits in trunk/Tools

Simplify ExpectedFailures
https://bugs.webkit.org/show_bug.cgi?id=92216

Reviewed by Eric Seidel.

This patch simplifies the ExpectedFailures class we use to remember
which tests are currently failing on the bots. When we wrote this code
originally, we weren't entirely sure how it would work. Now that we
understand it more clearly, we can write the code more clearly.

  • Scripts/webkitpy/tool/bot/expectedfailures.py:

(_has_failures):
(_is_trustworthy):
(ExpectedFailures.init):
(ExpectedFailures.failures_were_expected):
(ExpectedFailures.unexpected_failures_observed):
(ExpectedFailures.update):

  • Scripts/webkitpy/tool/bot/expectedfailures_unittest.py:

(ExpectedFailuresTest._assert_can_trust):
(ExpectedFailuresTest.test_failures_were_expected):
(ExpectedFailuresTest.test_unexpected_failures_observed):
(ExpectedFailuresTest.test_unexpected_failures_observed_when_tree_is_hosed):

  • Scripts/webkitpy/tool/bot/patchanalysistask.py:

(PatchAnalysisTask._test):
(PatchAnalysisTask._build_and_test_without_patch):
(PatchAnalysisTask._test_patch):

12:44 AM Changeset in webkit [123844] by danakj@chromium.org
  • 2 edits in trunk/Source/WebCore

[chromium] Don't add a HUD layer when there is no rootLayer
https://bugs.webkit.org/show_bug.cgi?id=92442

Reviewed by Adrienne Walker.

When the compositor is shutting down in threaded mode, it does a commit
without a root layer. We should not try stick the hud layer onto the
null root layer.

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

(WebCore::CCLayerTreeHost::willCommit):

12:30 AM Changeset in webkit [123843] by commit-queue@webkit.org
  • 31 edits
    3 adds in trunk

IndexedDB: inject index keys on cursor/objectstore/index get success handlers
https://bugs.webkit.org/show_bug.cgi?id=91123

Patch by Alec Flett <alecflett@chromium.org> on 2012-07-27
Reviewed by Tony Chang.

Source/WebCore:

Move key-injection into the frontend, by injecting the key into
the value on get rather than set for autoincremented keys. This
gives a potential performance win by avoiding an IPC call to the
V8 utility process on chromium, and generally cleans up the key
management.

Note that this duplicates the logic of generateIndexKeys into the
frontend until that method can be expunged from the backend. This
will be cleaned up in a later patch.

In order to account for both old data (which has the duplicate
primary key already injected and serialized) and any new data, an
assertion was removed from IDBBindingUtilities.

This also includes some minor refactoring such as the deprecation
of IDBCursorBackendInterface::update in favor of a direct call to
IDBObjectStoreBackendImpl::put, and removal of the now-defunct
IDBBackingStore::getObjectViaIndex.

Tests: storage/indexdb/index-duplicate-keypaths.html

  • Modules/indexeddb/IDBBackingStore.h:

(IDBBackingStore):

  • Modules/indexeddb/IDBCursor.cpp:

(WebCore::IDBCursor::update):
(WebCore::IDBCursor::setValueReady):

  • Modules/indexeddb/IDBCursorBackendImpl.cpp:

(WebCore::IDBCursorBackendImpl::update):

  • Modules/indexeddb/IDBIndexBackendImpl.cpp:

(WebCore::IDBIndexBackendImpl::getInternal):

  • Modules/indexeddb/IDBLevelDBBackingStore.cpp:
  • Modules/indexeddb/IDBLevelDBBackingStore.h:

(IDBLevelDBBackingStore):

  • Modules/indexeddb/IDBObjectStore.cpp:

(WebCore::generateIndexKeysForValue):
(WebCore):
(WebCore::IDBObjectStore::add):
(WebCore::IDBObjectStore::put):

  • Modules/indexeddb/IDBObjectStore.h:

(IDBObjectStore):

  • Modules/indexeddb/IDBObjectStoreBackendImpl.cpp:

(WebCore::IDBObjectStoreBackendImpl::put):
(WebCore::IDBObjectStoreBackendImpl::putWithIndexKeys):
(WebCore):
(WebCore::IDBObjectStoreBackendImpl::putInternal):

  • Modules/indexeddb/IDBRequest.cpp:

(WebCore::IDBRequest::onSuccess):

  • bindings/v8/IDBBindingUtilities.cpp:

(WebCore):

Source/WebKit/chromium:

Remove method no longer used.

  • tests/IDBFakeBackingStore.h:

LayoutTests:

One new test to make sure that indexes with the same keypath as an
autoIncrement'ed objectStore still index their values. Also minor
tweaks to tests to make it easier to figure out which key is being
updated.

  • storage/indexeddb/index-duplicate-keypaths-expected.txt: Added.
  • storage/indexeddb/index-duplicate-keypaths.html: Added.
  • storage/indexeddb/mozilla/autoincrement-indexes-expected.txt:
  • storage/indexeddb/mozilla/resources/autoincrement-indexes.js:
  • storage/indexeddb/resources/index-duplicate-keypaths.js: Added.
12:27 AM Changeset in webkit [123842] by tony@chromium.org
  • 4 edits
    2 adds in trunk

changing -webkit-order should change the paint order of flex items
https://bugs.webkit.org/show_bug.cgi?id=92041

Reviewed by Ojan Vafai.

Source/WebCore:

Override paintChildren and use the flex order iterator to determine the order to paint the children.

Test: css3/flexbox/order-painting.html

  • rendering/RenderFlexibleBox.cpp:

(WebCore::RenderFlexibleBox::layoutBlock): Save a reference to the order iterator.
(WebCore::RenderFlexibleBox::paintChildren):

  • rendering/RenderFlexibleBox.h:

(RenderFlexibleBox): Hold a reference to the order iterator so we don't have to recreate it at paint time.

Also mark all the virtual methods with OVERRIDE.

LayoutTests:

Use a ref test since this is testing paint behavior.

  • css3/flexbox/order-painting-expected.html: Added.
  • css3/flexbox/order-painting.html: Added.
12:10 AM Changeset in webkit [123841] by Csaba Osztrogonác
  • 7 edits in trunk

[Qt][WK2] REGRESSION(r119127): resetting window.internals settings between tests doesn't work properly
https://bugs.webkit.org/show_bug.cgi?id=88064

Reviewed by Zoltan Herczeg.

Source/WebKit/qt:

Add missing DumpRenderTreeSupportQt::resetInternalsObject(JSContextRef context) function.

  • WebCoreSupport/DumpRenderTreeSupportQt.cpp:

(DumpRenderTreeSupportQt::resetInternalsObject):

  • WebCoreSupport/DumpRenderTreeSupportQt.h:

Tools:

Use resetInternalsObject() instead of injectInternalsObject().

  • WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp:

(WTR::InjectedBundlePage::resetAfterTest):

LayoutTests:

  • platform/qt-5.0-wk2/Skipped: Unskip now passing tests.
Note: See TracTimeline for information about the timeline view.