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

Timeline



Nov 13, 2010:

11:56 PM Changeset in webkit [71981] by Patrick Gansterer
  • 6 edits in trunk/WebCore

2010-11-13 Patrick Gansterer <Patrick Gansterer>

Reviewed by Yury Semikhatsky.

Buildfix for !ENABLE(INSPECTOR)
​https://bugs.webkit.org/show_bug.cgi?id=49500

Also unify the position of the condition in the cpp files.

  • inspector/ConsoleMessage.cpp:
  • inspector/InspectorResourceAgent.cpp:
  • inspector/InspectorState.cpp:
  • inspector/ScriptCallFrame.cpp:
  • inspector/ScriptCallStack.cpp:
10:14 PM Changeset in webkit [71980] by hyatt@apple.com
  • 3 edits
    4 adds in trunk

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

Reviewed by Anders Carlsson.

Repaint is broken across writing mode boundaries. It's not enough to add in flipped offsets. You have
to flip the rect itself.

WebCore:

Added fast/repaint/repaint-across-writing-mode-boundary.html

  • rendering/RenderBox.cpp:

(WebCore::RenderBox::computeRectForRepaint):

LayoutTests:

Added fast/repaint/repaint-across-writing-mode-boundary.html

  • fast/repaint/repaint-across-writing-mode-boundary.html: Added.
  • platform/mac/fast/repaint/repaint-across-writing-mode-boundary-expected.checksum: Added.
  • platform/mac/fast/repaint/repaint-across-writing-mode-boundary-expected.png: Added.
  • platform/mac/fast/repaint/repaint-across-writing-mode-boundary-expected.txt: Added.
9:31 PM Changeset in webkit [71979] by hyatt@apple.com
  • 7 edits
    4 adds in trunk

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

Reviewed by Dan Bernstein.

REGRESSION: @font-face doesn't work with the small-caps variant. This was broken by the addition of SVG fonts
support long ago. This patch fixes the SVG code so that it doesn't break non-SVG fonts.

Added fast/blockflow/broken-ideograph-small-caps.html

WebCore:

  • css/CSSFontFace.cpp:

(WebCore::CSSFontFace::hasSVGFontFaceSource):

  • css/CSSFontFace.h:
  • css/CSSFontFaceSource.cpp:

(WebCore::CSSFontFaceSource::getFontData):

  • css/CSSFontFaceSource.h:
  • css/CSSFontSelector.cpp:

(WebCore::CSSFontSelector::addFontFaceRule):
(WebCore::CSSFontSelector::getFontData):

LayoutTests:

  • fast/blockflow/broken-ideograph-small-caps.html: Added.
  • platform/mac/fast/blockflow/broken-ideograph-small-caps-expected.checksum: Added.
  • platform/mac/fast/blockflow/broken-ideograph-small-caps-expected.png: Added.
  • platform/mac/fast/blockflow/broken-ideograph-small-caps-expected.txt: Added.
8:55 PM Changeset in webkit [71978] by levin@chromium.org
  • 3 edits in trunk/WebKitTools

2010-11-13 David Levin <​levin@chromium.org>

Reviewed by Eric Seidel.

check-webkit-style function detection crashes on functions in templates.
​https://bugs.webkit.org/show_bug.cgi?id=49504

For example "template <bool shouldClose(const Element*)>".

  • Scripts/webkitpy/style/checkers/cpp.py: If not function is detected, bail out.
  • Scripts/webkitpy/style/checkers/cpp_unittest.py: Modified the complex function detection test to expose this issue.
8:35 PM Changeset in webkit [71977] by mitz@apple.com
  • 2 edits in trunk/WebCore

fast/text/hyphen* tests have different results on Leopard
​https://bugs.webkit.org/show_bug.cgi?id=43296

Reviewed by Dave Hyatt.

  • platform/text/mac/TextBreakIteratorInternalICUMac.mm:

(WebCore::topLanguagePreference): Changed to look up the value of the AppleLanguages default
through the normal NSUserDefaults search order rather than skipping the application domain.
This makes this function respect DumpRenderTree’s setting of the default in the application
domain.

8:11 PM Changeset in webkit [71976] by levin@chromium.org
  • 3 edits in trunk/WebKitTools

check-webkit-style function detection doesn't detect indented functions declaractions.
​https://bugs.webkit.org/show_bug.cgi?id=49446

Reviewed by Shinichiro Hamaji.

Indented function declarations occur inside class definitions, so
they are a pretty common (and worth detecting).

  • Scripts/webkitpy/style/checkers/cpp.py: Changed regex to allow indentation. Changed the function start detection to only happen when not in a function. Changed function end detection to work based on matching braces instead of finding a close brace at the beginning of the line. Fixed close_expression to do what it says when it doesn't find the close.
  • Scripts/webkitpy/style/checkers/cpp_unittest.py: Indented function test.
8:04 PM Changeset in webkit [71975] by hyatt@apple.com
  • 13 edits in trunk

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

Reviewed by Dan Bernstein.

broken-ideographic-font.html causes layout test flakiness in release builds. The reason for this is
that the custom font data for m_brokenIdeographFontData was never getting pruned out of the tree.
The same problem exists for custom small caps fonts, so I made sure that the small caps fonts get
pruned as well. In addition, custom small caps fonts were never getting deleted either.

I also made allowsLigatures in FontPlatformData orientation-independent, and the callers check it
instead.

Unskip fast/blockflow/broken-ideographic-font.html

  • platform/graphics/SimpleFontData.cpp:

(WebCore::SimpleFontData::~SimpleFontData):

  • platform/graphics/chromium/SimpleFontDataChromiumWin.cpp:

(WebCore::SimpleFontData::platformDestroy):
(WebCore::SimpleFontData::smallCapsFontData):

  • platform/graphics/chromium/SimpleFontDataLinux.cpp:

(WebCore::SimpleFontData::platformDestroy):
(WebCore::SimpleFontData::smallCapsFontData):

  • platform/graphics/cocoa/FontPlatformDataCocoa.mm:

(WebCore::FontPlatformData::allowsLigatures):

  • platform/graphics/freetype/SimpleFontDataFreeType.cpp:

(WebCore::SimpleFontData::platformDestroy):
(WebCore::SimpleFontData::smallCapsFontData):

  • platform/graphics/haiku/SimpleFontDataHaiku.cpp:

(WebCore::SimpleFontData::platformDestroy):
(WebCore::SimpleFontData::smallCapsFontData):

  • platform/graphics/mac/ComplexTextControllerATSUI.cpp:

(WebCore::disableLigatures):

  • platform/graphics/mac/SimpleFontDataCoreText.cpp:

(WebCore::SimpleFontData::getCFStringAttributes):

  • platform/graphics/mac/SimpleFontDataMac.mm:

(WebCore::SimpleFontData::platformDestroy):

  • platform/graphics/win/SimpleFontDataWin.cpp:

(WebCore::SimpleFontData::platformDestroy):
(WebCore::SimpleFontData::smallCapsFontData):

  • platform/graphics/wx/SimpleFontDataWx.cpp:

(WebCore::SimpleFontData::platformDestroy):
(WebCore::SimpleFontData::smallCapsFontData):

2:48 PM Changeset in webkit [71974] by hyatt@apple.com
  • 1 edit in trunk/LayoutTests/platform/mac/Skipped

Skip the broken ideograph test until I can figure out why it corrupts release builds.

12:46 PM Changeset in webkit [71973] by hyatt@apple.com
  • 2 edits in trunk/WebCore/platform/graphics

Another attempt to fix layout tests.

12:00 PM Changeset in webkit [71972] by hyatt@apple.com
  • 2 edits in trunk/WebCore

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

Fix broken-ideographic-font.html. I had the font installed locally on my machine, and this covered up
the fact that CoreText was implicitly using it as fallback in the vertical case only. The bots don't have
the font installed locally, and so they showed the bug.

Make sure the GlyphPage is filled with horizontal glyphs for a broken ideograph font, so that CoreText is
willing to fill.

  • platform/graphics/mac/SimpleFontDataCoreText.cpp:

(WebCore::SimpleFontData::getCFStringAttributes):

11:29 AM Changeset in webkit [71971] by rwlbuis@webkit.org
  • 2 edits in trunk/LayoutTests

2010-11-13 Rob Buis <​rwlbuis@gmail.com>

Unreviewed test expectation rebaseline.

  • platform/qt/svg/W3C-SVG-1.1-SE/types-dom-06-f-expected.txt:
10:45 AM Changeset in webkit [71970] by hyatt@apple.com
  • 20 edits
    9 adds in trunk

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

Reviewed by Dan Bernstein.

Fix bugs with the orientation of fonts when the font is brought in via fallback, when the font contains no
vertical tables, or when the font is loaded via font-face in both horizontal and vertical forms.

(1) Include orientation as part of the @font-face hashtables so that the fonts are treated as
distinct when used in horizontal and vertical forms.

(2) Let the orientation member of FontPlatformData represent the desired orientation rather than
the supported one. Add an orientation member to SimpleFontData that represents the actual
supported orientation of the font.

(3) For fonts with no vertical tables, add a brokenIdeographicsFont accessor that is invoked when
CJK ideographs are encountered. This is essentially identical to the small caps fallback mechanism
but invoked only for ideographs. The special ideograph font locks the orientation to vertical and
renders the glyphs upright even when the font has no vertical tables. Punctuation and such will
still look funny, and really this is a bug in the font, but at least this way it will be apparent
that the font is to blame.

Added fast/blockflow/broken-ideographic-font.html and fast/blockflow/vertical-font-fallback.html.

WebCore:

  • css/CSSFontFaceSource.cpp:

(WebCore::CSSFontFaceSource::getFontData):

  • css/CSSSegmentedFontFace.cpp:

(WebCore::CSSSegmentedFontFace::getFontData):

  • platform/graphics/Font.cpp:

(WebCore::Font::isCJKIdeograph):

  • platform/graphics/Font.h:
  • platform/graphics/FontFastPath.cpp:

(WebCore::Font::glyphDataForCharacter):

  • platform/graphics/SimpleFontData.cpp:

(WebCore::SimpleFontData::SimpleFontData):
(WebCore::SimpleFontData::~SimpleFontData):
(WebCore::SimpleFontData::brokenIdeographFontData):

  • platform/graphics/SimpleFontData.h:

(WebCore::SimpleFontData::orientation):

  • platform/graphics/cocoa/FontPlatformDataCocoa.mm:

(WebCore::FontPlatformData::FontPlatformData):

  • platform/graphics/mac/FontMac.mm:

(WebCore::showGlyphsWithAdvances):

  • platform/graphics/mac/SimpleFontDataMac.mm:

(WebCore::SimpleFontData::platformInit):
(WebCore::SimpleFontData::platformBoundsForGlyph):
(WebCore::SimpleFontData::platformWidthForGlyph):

LayoutTests:

  • fast/blockflow/broken-ideographic-font.html: Added.
  • fast/blockflow/resources/MakibaFont13.ttf: Added.
  • fast/blockflow/vertical-font-fallback.html: Added.
  • platform/mac/fast/blockflow/broken-ideographic-font-expected.checksum: Added.
  • platform/mac/fast/blockflow/broken-ideographic-font-expected.png: Added.
  • platform/mac/fast/blockflow/broken-ideographic-font-expected.txt: Added.
  • platform/mac/fast/blockflow/vertical-font-fallback-expected.checksum: Added.
  • platform/mac/fast/blockflow/vertical-font-fallback-expected.png: Added.
  • platform/mac/fast/blockflow/vertical-font-fallback-expected.txt: Added.
9:16 AM Changeset in webkit [71969] by rwlbuis@webkit.org
  • 20 edits
    78 adds
    2 deletes in trunk

2010-11-13 Rob Buis <​rwlbuis@gmail.com>

Reviewed by Dirk Schulze.

requiredFeatures does not adapt to SVGStringList changes
​https://bugs.webkit.org/show_bug.cgi?id=40887

Centralize logic of handling of SVGElement validity changes.
The logic is to detach the SVGElement when changing from valid
to invalid, and to attach when changing from invalid to valid.

Tests: svg/dynamic-updates/SVGCircleElement-dom-requiredFeatures.html

svg/dynamic-updates/SVGCircleElement-svgdom-requiredFeatures.html
svg/dynamic-updates/SVGEllipseElement-dom-requiredFeatures.html
svg/dynamic-updates/SVGEllipseElement-svgdom-requiredFeatures.html
svg/dynamic-updates/SVGForeignObjectElement-dom-requiredFeatures.html
svg/dynamic-updates/SVGForeignObjectElement-svgdom-requiredFeatures.html
svg/dynamic-updates/SVGGElement-dom-requiredFeatures.html
svg/dynamic-updates/SVGGElement-svgdom-requiredFeatures.html
svg/dynamic-updates/SVGImageElement-dom-requiredFeatures.html
svg/dynamic-updates/SVGImageElement-svgdom-requiredFeatures.html
svg/dynamic-updates/SVGLineElement-dom-requiredFeatures.html
svg/dynamic-updates/SVGLineElement-svgdom-requiredFeatures.html
svg/dynamic-updates/SVGPathElement-dom-requiredFeatures.html
svg/dynamic-updates/SVGPathElement-svgdom-requiredFeatures.html
svg/dynamic-updates/SVGPolygonElement-dom-requiredFeatures.html
svg/dynamic-updates/SVGPolygonElement-svgdom-requiredFeatures.html
svg/dynamic-updates/SVGPolylineElement-dom-requiredFeatures.html
svg/dynamic-updates/SVGPolylineElement-svgdom-requiredFeatures.html
svg/dynamic-updates/SVGRectElement-dom-requiredFeatures.html
svg/dynamic-updates/SVGRectElement-svgdom-requiredFeatures.html
svg/dynamic-updates/SVGSVGElement-dom-requiredFeatures.html
svg/dynamic-updates/SVGSVGElement-svgdom-requiredFeatures.html
svg/dynamic-updates/SVGTextElement-dom-requiredFeatures.html
svg/dynamic-updates/SVGTextElement-svgdom-requiredFeatures.html
svg/dynamic-updates/SVGUseElement-dom-requiredFeatures.html
svg/dynamic-updates/SVGUseElement-svgdom-requiredFeatures.html

8:39 AM Changeset in webkit [71968] by Patrick Gansterer
  • 2 edits in trunk/WebKit/wince

2010-11-13 Patrick Gansterer <Patrick Gansterer>

Unreviewed, build fix after r71541.

  • WebCoreSupport/ChromeClientWinCE.h:
8:32 AM Changeset in webkit [71967] by caseq@chromium.org
  • 6 edits in trunk/WebCore

2010-11-13 Andrey Kosyakov <​caseq@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: [refactoring] support Views in TabbedPane and streamline tab selection in resource view
​https://bugs.webkit.org/show_bug.cgi?id=49493

  • inspector/front-end/NetworkPanel.js: (WebInspector.NetworkPanel.prototype._showResource):
  • inspector/front-end/ResourceView.js: (WebInspector.ResourceView): (WebInspector.ResourceView.prototype.selectContentTab): (WebInspector.ResourceView.prototype._selectTab): (WebInspector.ResourceView.prototype._refreshCookies): (WebInspector.ResourceCookiesTab.prototype.show):
  • inspector/front-end/StoragePanel.js: (WebInspector.StoragePanel.prototype.showResource):
  • inspector/front-end/TabbedPane.js: (WebInspector.TabbedPane.prototype.appendTab): (WebInspector.TabbedPane.prototype.hasTab): (WebInspector.TabbedPane.prototype.selectTabById): (WebInspector.TabbedPane.prototype._hideTab): (WebInspector.TabbedPane.prototype._showTab):
  • inspector/front-end/inspector.css: (.resource-view-cookies): (.resource-view-cookies.visible):
6:50 AM Changeset in webkit [71966] by yurys@chromium.org
  • 2 edits in trunk/WebCore

2010-11-13 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Pavel Feldman.

REGRESSION(71515): Web Inspector: Safari crash calling console.log when Log JavaScript Exceptions to Console is checked
​https://bugs.webkit.org/show_bug.cgi?id=49362

  • page/Console.cpp: (WebCore::Console::addMessage): avoid accessing ScriptArguments and ScriptCallStack after their ownership has been passed to the inspector.
3:04 AM Changeset in webkit [71965] by yurys@chromium.org
  • 2 edits in trunk/WebCore

2010-11-13 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: DOM event listener name is cropped in Elements panel
​https://bugs.webkit.org/show_bug.cgi?id=49387

  • inspector/front-end/inspector.css: (.event-bar): use margin-left instead of left to fit child node into its parent
3:02 AM Changeset in webkit [71964] by hamaji@chromium.org
  • 4 edits in trunk

2010-11-13 Shinichiro Hamaji <​hamaji@chromium.org>

Reviewed by Kent Tamura.

[Chromium] print doesn't work on ​http://nodejs.org/api.html
​https://bugs.webkit.org/show_bug.cgi?id=49304

  • platform/chromium/test_expectations.txt:

2010-11-13 Shinichiro Hamaji <​hamaji@chromium.org>

Reviewed by Kent Tamura.

[Chromium] print doesn't work on ​http://nodejs.org/api.html
​https://bugs.webkit.org/show_bug.cgi?id=49304

  • src/WebFrameImpl.cpp: (WebKit::ChromePrintContext::begin): (WebKit::WebFrameImpl::printBegin):
1:47 AM Changeset in webkit [71963] by commit-queue@webkit.org
  • 2 edits
    1 add in trunk/WebCore

2010-11-13 Ilya Sherman <​isherman@chromium.org>

Reviewed by Shinichiro Hamaji.

Fixes regression from previous patch -- computes y offsets for elements
that are scrolled out of the window.
​https://bugs.webkit.org/show_bug.cgi?id=49306

  • manual-tests/select-scroll.html: Added.
  • platform/chromium/PopupMenuChromium.cpp: (WebCore::PopupListBox::layout):

Nov 12, 2010:

10:46 PM Changeset in webkit [71962] by dbates@webkit.org
  • 8 edits
    2 adds in trunk

2010-11-12 Daniel Bates <​dbates@rim.com>

Reviewed by Adam Barth.

Transferred <iframe>s may not have a unique internal name
​https://bugs.webkit.org/show_bug.cgi?id=48768

Fixes an issue where an <iframe> that is transferred from
document A to document B, where A != B, may not have a
unique internal name in B.

Test: fast/frames/iframe-reparenting-unique-name.html

  • html/HTMLFrameElementBase.cpp: (WebCore::HTMLFrameElementBase::setNameAndOpenURL): Inlined code from HTMLFrameElementBase::setName() so that we can remove setName(), which was only called being called by HTMLFrameElementBase::setNameAndOpenURL().
  • html/HTMLFrameElementBase.h: Removed HTMLFrameElementBase::setName().
  • html/HTMLFrameOwnerElement.h: Removed HTMLFrameOwnerElement::setName().
  • page/Frame.cpp: (WebCore::Frame::transferChildFrameToNewDocument): Modified to call FrameTree::transferChild().
  • page/FrameTree.cpp: (WebCore::FrameTree::transferChild): Added. (WebCore::FrameTree::appendChild): Moved the majority of the code into FrameTree::actuallyAppendChild(). (WebCore::FrameTree::actuallyAppendChild): Added.
  • page/FrameTree.h:

2010-11-12 Daniel Bates <​dbates@rim.com>

Reviewed by Adam Barth.

Transferred <iframe>s may not have a unique internal name
​https://bugs.webkit.org/show_bug.cgi?id=48768

Tests that an <iframe> which is transferred from document A to
document B, where A != B, has a unique name in B.

  • fast/frames/iframe-reparenting-unique-name-expected.txt: Added.
  • fast/frames/iframe-reparenting-unique-name.html: Added.
10:28 PM Changeset in webkit [71961] by dbates@webkit.org
  • 2 edits in trunk/WebKitTools

2010-11-12 Daniel Bates <​dbates@rim.com>

Rubber-stamped by Nikolas Zimmermann.

Make do-file-rename rename files in the directory WebKit2.

This also makes the list of searched directories in do-file-rename
consistent with the list of searched directories in the script do-webcore-rename.

  • Scripts/do-file-rename:
6:00 PM Changeset in webkit [71960] by dpranke@chromium.org
  • 3 edits
    1 add in trunk/WebKitTools

2010-11-12 Dirk Pranke <​dpranke@chromium.org>

Reviewed by James Robinson.

Attempt yet again to land the fix for bug 49360 (respecting
set-webkit-configuration). We need to handle the cases where
trying to run webkit-build-directory to find out where the
default configuration might be fails (that shows up on some
Chromium bots that apparently don't have perl installed).

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

  • Scripts/webkitpy/layout_tests/port/config.py:
  • Scripts/webkitpy/layout_tests/port/config_standalone.py: Added.
  • Scripts/webkitpy/layout_tests/port/config_unittest.py:
5:39 PM Changeset in webkit [71959] by Joseph Pecoraro
  • 3 edits in trunk/WebCore

2010-11-12 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Alexey Proskuryakov.

Some FrameLoader.h Cleanup
​https://bugs.webkit.org/show_bug.cgi?id=49484

Removed function signatures without any implementation. The functions
used to exist, and were nicely refactored into PageCache in r60688 and
the signatures were left behind.

Removed function signature setLoadType. Added in r17238 back when the
file was named WebFrameLoader, and the implementation was lost during
the transition from its mm implementation.

Removed function signature updateHistoryAfterClientRedirect. Implementation
and call points were removed in r18541 but the signature was left behind,
and jumped files!

Removed unused debug variable m_didDispatchDidCommitLoad. It was added
and used in Assertions in r28399. It was removed soon after, in r28464
because they were, "firing like crazy. Not sure why yet." It was
never used again.

Added const qualifier and moved interruptionForPolicyChangeError to
be more like its buddies.

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::FrameLoader): (WebCore::FrameLoader::checkLoadCompleteForThisFrame): (WebCore::FrameLoader::interruptionForPolicyChangeError): (WebCore::FrameLoader::dispatchDidCommitLoad):
  • loader/FrameLoader.h:
5:17 PM Changeset in webkit [71958] by mihaip@chromium.org
  • 2 edits in trunk/WebKitTools

2010-11-12 Mihai Parparita <​mihaip@chromium.org>

Reviewed by Adam Barth.

webkit-patch land logs commit message twice
​https://bugs.webkit.org/show_bug.cgi?id=49482

Don't log the comment that's used by bugzilla.close_bug_as_fixed, since
that's just the commit message, which we now output following the
actual commit.

  • Scripts/webkitpy/common/net/bugzilla/bugzilla.py:
4:57 PM Changeset in webkit [71957] by tony@chromium.org
  • 2 edits in trunk/LayoutTests

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

Unreviewed, revert changes to script-object-invoke that crash
single process chromium.

  • plugins/script-object-invoke.html:
4:47 PM Changeset in webkit [71956] by yael.aharon@nokia.com
  • 3 edits
    4 adds in trunk

2010-11-12 Yael Aharon <​yael.aharon@nokia.com>

Reviewed by David Hyatt.

Spatial Navigation: Cannot focus on some ContainerNode
​https://bugs.webkit.org/show_bug.cgi?id=48393

  • fast/events/spatial-navigation/snav-container-only-white-space-expected.txt: Added.
  • fast/events/spatial-navigation/snav-container-only-white-space.html: Added.
  • fast/events/spatial-navigation/snav-container-white-space-expected.txt: Added.
  • fast/events/spatial-navigation/snav-container-white-space.html: Added.

2010-11-12 Yael Aharon <​yael.aharon@nokia.com>

Reviewed by David Hyatt.

Spatial Navigation: Cannot focus on some ContainerNode
​https://bugs.webkit.org/show_bug.cgi?id=48393

ContainerNode::getLowerRightCorner() assumes that the lowest right corner of its last child
is its own lowest right corner. If that child is an empty text node after an image, it does
not have size and position information, so we should go to the previous child.

Tests: fast/events/spatial-navigation/snav-container-only-white-space.html

fast/events/spatial-navigation/snav-container-white-space.html

  • dom/ContainerNode.cpp: (WebCore::ContainerNode::getLowerRightCorner):
4:34 PM Changeset in webkit [71955] by tony@chromium.org
  • 4 edits
    2 moves in trunk

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

Reviewed by Adam Barth.

run platform/chromium/plugins/script-object-invoke.html on all platforms
​https://bugs.webkit.org/show_bug.cgi?id=49280

  • platform/chromium/drt_expectations.txt:
  • plugins/script-object-invoke-expected.txt: Renamed from LayoutTests/platform/chromium/plugins/script-object-invoke-expected.txt.
  • plugins/script-object-invoke.html: Renamed from LayoutTests/platform/chromium/plugins/script-object-invoke.html.

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

Reviewed by Adam Barth.

run platform/chromium/plugins/script-object-invoke.html on all platforms
​https://bugs.webkit.org/show_bug.cgi?id=49280

  • DumpRenderTree/TestNetscapePlugIn/PluginObject.cpp: (testScriptObjectInvoke): (pluginInvoke):
3:54 PM Changeset in webkit [71954] by mihaip@chromium.org
  • 3 edits in trunk/WebKitTools

2010-11-12 Mihai Parparita <​mihaip@chromium.org>

Reviewed by Adam Barth.

webkit-patch land-cowboy (and other commands) should display the committed SVN revision after landing a patch
​https://bugs.webkit.org/show_bug.cgi?id=49471

Right now there's no indication of a successful commit.

  • Scripts/webkitpy/tool/commands/download_unittest.py:
  • Scripts/webkitpy/tool/steps/commit.py:
3:43 PM Changeset in webkit [71953] by mihaip@chromium.org
  • 2 edits in trunk/LayoutTests

2010-11-12 Mihai Parparita <​mihaip@chromium.org>

Unreviewed Chromium expectations update.

fast/canvas/canvas-arc-360-winding.html needs to be skipped for the
GPU suite too.

  • platform/chromium-gpu/test_expectations.txt:
3:14 PM Changeset in webkit [71952] by kinuko@chromium.org
  • 5 edits in trunk/WebKitTools

2010-11-12 Kinuko Yasuda <​kinuko@chromium.org>

Reviewed by Ojan Vafai.

Include detailed test modifiers in results.json and enable incremental uploading for non-layout tests
​https://bugs.webkit.org/show_bug.cgi?id=49354

Also moved/integrated the upload method from run_webkit_tests.py to json_results_generator.py.

  • Scripts/webkitpy/layout_tests/layout_package/json_layout_results_generator.py:
  • Scripts/webkitpy/layout_tests/layout_package/json_results_generator.py:
  • Scripts/webkitpy/layout_tests/layout_package/json_results_generator_unittest.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
2:51 PM Changeset in webkit [71951] by commit-queue@webkit.org
  • 18 edits
    6 moves in trunk

2010-11-12 James Simonsen <​simonjam@chromium.org>

Reviewed by Darin Fisher.

[Web Timing] Rename interfaces:

  • fast/dom/Window/window-properties-performance-expected.txt: Class names changed.

2010-11-12 James Simonsen <​simonjam@chromium.org>

Reviewed by Darin Fisher.

[Web Timing] Rename interfaces:

  • DerivedSources.cpp:
  • DerivedSources.make:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • page/Performance.cpp: (WebCore::Performance::navigation): (WebCore::Performance::timing):
  • page/Performance.h:
  • page/Performance.idl:
  • page/PerformanceNavigation.cpp: Renamed from WebCore/page/Navigation.cpp. (WebCore::PerformanceNavigation::PerformanceNavigation): (WebCore::PerformanceNavigation::frame): (WebCore::PerformanceNavigation::disconnectFrame): (WebCore::PerformanceNavigation::type): (WebCore::PerformanceNavigation::redirectCount):
  • page/PerformanceNavigation.h: Renamed from WebCore/page/Navigation.h. (WebCore::PerformanceNavigation::create):
  • page/PerformanceNavigation.idl: Renamed from WebCore/page/Navigation.idl.
  • page/PerformanceTiming.cpp: Renamed from WebCore/page/Timing.cpp. (WebCore::toIntegerMilliseconds): (WebCore::getPossiblySkewedTimeInKnownRange): (WebCore::PerformanceTiming::PerformanceTiming): (WebCore::PerformanceTiming::frame): (WebCore::PerformanceTiming::disconnectFrame): (WebCore::PerformanceTiming::navigationStart): (WebCore::PerformanceTiming::unloadEventEnd): (WebCore::PerformanceTiming::redirectStart): (WebCore::PerformanceTiming::redirectEnd): (WebCore::PerformanceTiming::fetchStart): (WebCore::PerformanceTiming::domainLookupStart): (WebCore::PerformanceTiming::domainLookupEnd): (WebCore::PerformanceTiming::connectStart): (WebCore::PerformanceTiming::connectEnd): (WebCore::PerformanceTiming::requestStart): (WebCore::PerformanceTiming::requestEnd): (WebCore::PerformanceTiming::responseStart): (WebCore::PerformanceTiming::responseEnd): (WebCore::PerformanceTiming::domLoading): (WebCore::PerformanceTiming::domInteractive): (WebCore::PerformanceTiming::domContentLoaded): (WebCore::PerformanceTiming::domComplete): (WebCore::PerformanceTiming::loadEventStart): (WebCore::PerformanceTiming::loadEventEnd): (WebCore::PerformanceTiming::documentLoader): (WebCore::PerformanceTiming::documentTiming): (WebCore::PerformanceTiming::documentLoadTiming): (WebCore::PerformanceTiming::resourceLoadTiming): (WebCore::PerformanceTiming::resourceLoadTimeRelativeToAbsolute):
  • page/PerformanceTiming.h: Renamed from WebCore/page/Timing.h. (WebCore::PerformanceTiming::create):
  • page/PerformanceTiming.idl: Renamed from WebCore/page/Timing.idl.

2010-11-12 James Simonsen <​simonjam@chromium.org>

Reviewed by Darin Fisher.

[Web Timing] Rename interfaces:

  • src/WebPerformance.cpp: (WebKit::WebPerformance::navigationType):
2:40 PM Changeset in webkit [71950] by eric.carlson@apple.com
  • 3 edits in trunk/WebCore

2010-11-12 Eric Carlson <​eric.carlson@apple.com>

Reviewed by Simon Fraser.

Remove unused code from HTMLMediaElement
​https://bugs.webkit.org/show_bug.cgi?id=49451

Just removing code, so no new tests.

  • html/HTMLMediaElement.cpp: Removed unused functions.
  • html/HTMLMediaElement.h:
2:00 PM Changeset in webkit [71949] by rniwa@webkit.org
  • 2 edits in trunk/WebCore

2010-11-12 Ryosuke Niwa <​rniwa@webkit.org>

Unreviewed XCode project fix.

  • WebCore.xcodeproj/project.pbxproj:
1:59 PM Changeset in webkit [71948] by mihaip@chromium.org
  • 2 edits in trunk/LayoutTests

2010-11-12 Mihai Parparita <​mihaip@chromium.org>

Unreviewed Chromium expectations update.

Mark fast/canvas/canvas-arc-360-winding.html as failing on Linux and
Windows.

  • platform/chromium/test_expectations.txt:
1:56 PM Changeset in webkit [71947] by ap@apple.com
  • 4 edits in trunk/WebCore

Reviewed by Simon Fraser.

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

No change in behavior, so no tests.

  • css/WebKitCSSMatrix.cpp: (WebCore::WebKitCSSMatrix::WebKitCSSMatrix): We don't inherit from StyleBase any more, so no need to initialize always-null parent. (WebCore::WebKitCSSMatrix::setMatrixValue): Since there was never a parent, useStrictParsing() just returned true.
  • css/WebKitCSSMatrix.h: Just inherit from RefCounted<WebKitCSSMatrix>. Removed unused default constructor and copy constructor.
  • html/canvas/WebGLRenderingContext.h: Removed an unnneded forward declaration.
1:56 PM Changeset in webkit [71946] by jschuh@chromium.org
  • 1 edit
    2 copies in branches/chromium/552

Merge 71918 - 2010-11-12 Justin Schuh <​jschuh@chromium.org>

Reviewed by Adam Barth.

Use a RefPtr inside ContainerNode::willRemove loop
​https://bugs.webkit.org/show_bug.cgi?id=49237

Test: fast/innerHTML/innerHTML-iframe.html

  • dom/ContainerNode.cpp: (WebCore::ContainerNode::willRemove):

2010-11-12 Justin Schuh <​jschuh@chromium.org>

Reviewed by Adam Barth.

Use a RefPtr inside ContainerNode::willRemove loop
​https://bugs.webkit.org/show_bug.cgi?id=49237

  • fast/innerHTML/innerHTML-iframe-expected.txt: Added.
  • fast/innerHTML/innerHTML-iframe.html: Added.

TBR=​jschuh@chromium.org
BUG=62261

1:53 PM Changeset in webkit [71945] by jamesr@google.com
  • 3 edits
    2 copies in branches/chromium/552

Merge 71345 - 2010-11-04 Eric Seidel <​eric@webkit.org>

Reviewed by Gavin Barraclough.

REGRESSION(49798): Crash in HTMLObjectElement::parseMappedAttribute
​https://bugs.webkit.org/show_bug.cgi?id=48789

The contract for all String/AtomicString methods seems to be that it's
safe to call them, even when the String is null (impl() returns 0).
This contract was broken by r49798 (unintentionally) when optimizing
for dromeo.
This patch adds a null check to AtomicString::lower() fixing this
crash and preventing future confusion.

  • wtf/text/AtomicString.cpp: (WTF::AtomicString::lower):

2010-11-04 Eric Seidel <​eric@webkit.org>

Reviewed by Gavin Barraclough.

REGRESSION(49798): Crash in HTMLObjectElement::parseMappedAttribute
​https://bugs.webkit.org/show_bug.cgi?id=48789

  • fast/dom/HTMLObjectElement/set-type-to-null-crash-expected.txt: Added.
  • fast/dom/HTMLObjectElement/set-type-to-null-crash.html: Added.

TBR=​eric@webkit.org
BUG=55345

1:51 PM Changeset in webkit [71944] by commit-queue@webkit.org
  • 2 edits in trunk/WebKit/qt

2010-11-12 Jacob Dinu <​dinu.jacob@nokia.com>

Reviewed by Andreas Kling.

[Qt] Update .def file for Symbian
​https://bugs.webkit.org/show_bug.cgi?id=49456

Added missing symbols from revisions r69825, r70470, r71806

  • symbian/eabi/QtWebKitu.def:
1:47 PM Changeset in webkit [71943] by Dimitri Glazkov
  • 2 edits in trunk/LayoutTests

2010-11-12 Dimitri Glazkov <Dimitri Glazkov>

Because of r71945, events from shadow DOM elements now bubble correctly
unless handled, which trips over the old issue in GTK.
​https://bugs.webkit.org/show_bug.cgi?id=40601

  • platform/gtk/Skipped: Added media/controls-right-click-on-timebar.html.
1:34 PM Changeset in webkit [71942] by zmo@google.com
  • 2 edits in trunk/WebKit/chromium

2010-11-12 Zhenyao Mo <​zmo@google.com>

Reviewed by James Robinson.

WebGraphicsContext3DDefaultImpl does not run on top of OpenGL ES 2.0 implementations
​https://bugs.webkit.org/show_bug.cgi?id=48282

Simple fix for a regression introduced in r71793.

  • src/WebGraphicsContext3DDefaultImpl.cpp: (WebKit::WebGraphicsContext3DDefaultImpl::initialize):
1:26 PM Changeset in webkit [71941] by jamesr@google.com
  • 3 edits
    5 copies in branches/chromium/552

Merge 70840 - 2010-10-28 James Robinson <​jamesr@chromium.org>

Reviewed by Simon Fraser.

iframes forced into slow scrolling mode by containing RenderLayer sometimes
​https://bugs.webkit.org/show_bug.cgi?id=48570

Adds a test that verifies that the repaint region for a scrolled iframe is not
larger than necessary.

  • fast/repaint/iframe-scroll-repaint.html: Added.
  • fast/repaint/resources/iframe-scroll-repaint-iframe.html: Added.
  • platform/mac/fast/repaint/iframe-scroll-repaint-expected.checksum: Added.
  • platform/mac/fast/repaint/iframe-scroll-repaint-expected.png: Added.
  • platform/mac/fast/repaint/iframe-scroll-repaint-expected.txt: Added.

2010-10-28 James Robinson <​jamesr@chromium.org>

Reviewed by Simon Fraser.

iframes forced into slow scrolling mode by containing RenderLayer sometimes
​https://bugs.webkit.org/show_bug.cgi?id=48570

Only performs the overlap test for RenderLayers that paint themselves. This
prevents marking RenderWidgets as being overlapped unnecessarily.

Test: fast/repaint/iframe-scroll-repaint.html

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::paintLayer):

TBR=​commit-queue@webkit.org

1:25 PM Changeset in webkit [71940] by mihaip@chromium.org
  • 2 edits in trunk/LayoutTests

2010-11-12 Mihai Parparita <​mihaip@chromium.org>

Unreviewed Chromium expectations update.

Fix typo.

  • platform/chromium/test_expectations.txt:
1:07 PM Changeset in webkit [71939] by mihaip@chromium.org
  • 2 edits in trunk/LayoutTests

2010-11-12 Mihai Parparita <​mihaip@chromium.org>

Unreviewed Chromium expectations update.

Mark some more tests as flaky.

  • platform/chromium/test_expectations.txt:
1:07 PM Changeset in webkit [71938] by benjamin.poulain@nokia.com
  • 4 edits in trunk

[Qt] Need to implement WebKit::pluginWillHandleLoadError for WebKit2
​https://bugs.webkit.org/show_bug.cgi?id=48764

WebKit/qt:

Reviewed by Andreas Kling.

Add the missing error.

  • WebCoreSupport/FrameLoaderClientQt.cpp:

(WebCore::FrameLoaderClientQt::pluginWillHandleLoadError):

WebKit2:

Reviewed by NOBODY (OOPS!).

Add the missing error to Qt support.

  • WebProcess/WebCoreSupport/qt/WebErrorsQt.cpp:

(WebKit::pluginWillHandleLoadError):

12:28 PM Changeset in webkit [71937] by sullivan@apple.com
  • 2 edits in trunk/WebKit2

Fixed Target for WKBundlePageOverlay.cpp.

Reviewed by Anders Carlsson.

  • WebKit2.xcodeproj/project.pbxproj:

Changed Target for WKBundlePageOverlay.cpp from WebProcess to WebKit2.

12:23 PM Changeset in webkit [71936] by commit-queue@webkit.org
  • 3 edits
    3 adds in trunk

2010-11-12 Helder Correia <​helder@sencha.com>

Reviewed by Andreas Kling.

[Qt] Path::addArc() does not set right angle direction on full arcs
​https://bugs.webkit.org/show_bug.cgi?id=49138

New test for nonzero winding rule correctness for full arc paths.

  • fast/canvas/canvas-arc-360-winding-expected.txt: Added.
  • fast/canvas/canvas-arc-360-winding.html: Added.
  • fast/canvas/script-tests/canvas-arc-360-winding.js: Added.

2010-11-12 Helder Correia <​helder@sencha.com>

Reviewed by Andreas Kling.

[Qt] Path::addArc() does not set right angle direction on full arcs
​https://bugs.webkit.org/show_bug.cgi?id=49138

Ensure correctness of nonzero winding rule for full arc paths.
Invert the sign of the span if the direction is counterclockwise.

Test: fast/canvas/canvas-arc-360-winding.html

  • platform/graphics/qt/PathQt.cpp: (WebCore::Path::addArc):
12:14 PM Changeset in webkit [71935] by andersca@apple.com
  • 6 edits in trunk/WebKit2

Need a way that WKBundlePageOverlayRef clients can make sure they only uninstall the overlay they installed
​https://bugs.webkit.org/show_bug.cgi?id=49461
<rdar://problem/8662760>

Reviewed by John Sullivan.

  • WebProcess/InjectedBundle/API/c/WKBundlePage.cpp:

(WKBundlePageUninstallPageOverlay):

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

(WebKit::FindController::findString):
(WebKit::FindController::hideFindUI):

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::uninstallPageOverlay):

  • WebProcess/WebPage/WebPage.h:
11:49 AM Changeset in webkit [71934] by Dimitri Glazkov
  • 25 edits
    6 adds in trunk

2010-11-05 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Darin Adler.

Implement shadow DOM-aware event targeting and introduce EventContext to track the context of each event dispatch.
​https://bugs.webkit.org/show_bug.cgi?id=46015

Added more test coverage around events crossing shadow DOM boundaries and tweaked existing tests.

  • fast/events/shadow-boundary-crossing-2-expected.txt: Added.
  • fast/events/shadow-boundary-crossing-2.html: Added.
  • fast/events/shadow-boundary-crossing.html: Tuned to better reflect its point:

the event should indeed fire (it used to be swallowed), but its target
should be a non-shadow node.

  • media/audio-delete-while-slider-thumb-clicked.html : Tweaked to actually click on

the scrubber thumb (it was off by 2 pixels).

2010-11-05 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Darin Adler.

Implement shadow DOM-aware event targeting and introduce EventContext to track the context of each event dispatch.
​https://bugs.webkit.org/show_bug.cgi?id=46015

Test: fast/events/shadow-boundary-crossing-2.html

This patch adds the notion of EventContext (and a very similar-acting WindowEventContext, specifically
for DOMWindow), an abstraction that carries information around dispatching an event for any given Node.

This abstraction is necessary to ensure that events, fired from shadow DOM nodes are properly retargeted to
appear as if they are coming from their host, thus never exposing the shadow DOM nodes to the world outside.

  • Android.mk: Added EventContext, WindowEventContext files.
  • CMakeLists.txt: Ditto.
  • GNUmakefile.am: Ditto.
  • WebCore.gypi: Ditto.
  • WebCore.pro: Ditto.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • WebCore.vcproj/WebCore.vcproj: Ditto.
  • dom/ContainerNode.cpp: (WebCore::notifyChildInserted): Changed to be shadow DOM-aware.
  • dom/EventContext.cpp: Added.
  • dom/EventContext.h: Added.
  • dom/Node.cpp: (WebCore::Node::markAncestorsWithChildNeedsStyleRecalc): Changed to be shadow DOM-aware. (WebCore::Node::createRendererIfNeeded): Ditto. (WebCore::Node::parentOrHostNode): Added new helper method. (WebCore::Node::enclosingLinkEventParentOrSelf): Changed to be shadow DOM-aware. (WebCore::eventTargetRespectingSVGTargetRules): Collapsed two helper methods into one. (WebCore::Node::getEventAncestors): Renamed and refactored to collect a vector of EventContexts. (WebCore::Node::topEventContext): Added. (WebCore::eventHasListeners): Changed to use EventContexts. (WebCore::Node::dispatchGenericEvent): Ditto.
  • dom/Node.h: Removed eventParentNode that's no longer needed, added parentOrHostNode decl,

and changed signature of eventAncestors to use EventContexts.

  • dom/Text.cpp: (WebCore::Text::createRenderer): Changed to be shadow DOM-aware.
  • dom/WindowEventContext.cpp: Added.
  • dom/WindowEventContext.h: Added.
  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::getEventListenersForNode): Changed to use EventContexts.
  • page/EventHandler.cpp: (WebCore::EventHandler::updateMouseEventTargetNode): Removed code that's no longer necessary.
  • rendering/RenderTextControlMultiLine.cpp: (WebCore::RenderTextControlMultiLine::subtreeHasChanged): Removed event invocation that's

no longer necessary.

  • rendering/ShadowElement.h: Made m_shadowParent a RefPtr to avoid stale references when parent is deleted.
  • rendering/TextControlInnerElements.cpp: (WebCore::TextControlInnerTextElement::defaultEventHandler): Flipped the condition back

from where it was prior to r60418.

  • svg/SVGElement.cpp: Removed eventParentNode that's no longer needed.
  • svg/SVGElement.h: Ditto.
11:37 AM Changeset in webkit [71933] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

2010-11-12 Ryuan Choi <​ryuan.choi@samsung.com>

Reviewed by Martin Robinson.

[GTK] Remove unnecessary header for FontPlatformDataPango.cpp
​https://bugs.webkit.org/show_bug.cgi?id=46029

Remove unnecessary header includes.
No new tests, as there's no functionality.

  • platform/graphics/pango/FontPlatformDataPango.cpp:
11:10 AM Changeset in webkit [71932] by andersca@apple.com
  • 2 edits in trunk/WebKit2

Allow MessageSender to override sendMessage
​https://bugs.webkit.org/show_bug.cgi?id=49457

Reviewed by Sam Weinig.

Change MessageSender::send to encode the message and then call sendMessage.
This allows for clients to override sendMessage, something which WebProcessProxy
will be able to take advantage of.

  • Platform/CoreIPC/MessageSender.h:

(CoreIPC::MessageSender::send):
(CoreIPC::MessageSender::sendMessage):

10:52 AM Changeset in webkit [71931] by andersca@apple.com
  • 6 edits in trunk/WebKit2

CoreIPC cleanup
​https://bugs.webkit.org/show_bug.cgi?id=49455

Reviewed by Sam Weinig.

Add a static create function to ArgumentEncoder.
Factor sync message argument encoder creation out into a new function,
createSyncMessageArgumentEncoder.

  • Platform/CoreIPC/ArgumentEncoder.cpp:

(CoreIPC::ArgumentEncoder::create):

  • Platform/CoreIPC/ArgumentEncoder.h:
  • Platform/CoreIPC/Connection.cpp:

(CoreIPC::Connection::createSyncMessageArgumentEncoder):
(CoreIPC::Connection::dispatchSyncMessage):

  • Platform/CoreIPC/Connection.h:

(CoreIPC::Connection::send):
(CoreIPC::Connection::sendSync):

  • UIProcess/WebProcessProxy.h:

(WebKit::WebProcessProxy::send):

10:39 AM Changeset in webkit [71930] by weinig@apple.com
  • 4 edits in trunk/WebKit2

ASSERTION when converting some WKStringRefs returned from willSubmitForm to CFStringRef
<rdar://problem/8662180>
​https://bugs.webkit.org/show_bug.cgi?id=49454

Reviewed by Anders Carlsson.

Enforce the invariant that a WebString never contains a null WTF::String at a
lower level, in the WebString constructor, instead of relying on the callers of
WebString::create to do it.

  • Shared/API/c/WKSharedAPICast.h:

(WebKit::toAPI):
(WebKit::toCopiedAPI): Remove null string check...

  • Shared/WebString.h:

(WebKit::WebString::WebString): ... and move it to the constructor.

  • Shared/API/c/WKURL.cpp:

(WKURLCopyString): Use toCopiedAPI() instead of calling WebString directly
to match other similar uses.

10:30 AM Changeset in webkit [71929] by mihaip@chromium.org
  • 2 edits in trunk/LayoutTests

2010-11-12 Mihai Parparita <​mihaip@chromium.org>

Unreviewed Chromium expectations update.

Mark http/tests/media/video-cookie.html as flaky.

  • platform/chromium/test_expectations.txt:
10:25 AM Changeset in webkit [71928] by tony@chromium.org
  • 6 edits in trunk

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

Reviewed by Kent Tamura.

[chromium] Load a blank page when closing DRT WebViewHosts
​https://bugs.webkit.org/show_bug.cgi?id=49417

  • platform/chromium/drt_expectations.txt: Fix 2 tests in DRT.

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

Reviewed by Kent Tamura.

[chromium] Load a blank page when closing DRT WebViewHosts
​https://bugs.webkit.org/show_bug.cgi?id=49417

In test_shell, when we close a window, we first load about:blank
and run GC to fire the destruction logic of the page. In test_shell,
this happens in ~TestShell. In DRT, we manage virtual windows as
WebViewHosts, so we need to replicate this logic in ~WebViewHost.

Managing focus is a bit trickier, since normally you would get window
events to reset the focus to the right window. We do it manually
instead.

This fixes 2 plugin tests on DRT.

  • DumpRenderTree/chromium/TestShell.cpp: (TestShell::~TestShell): Move window close logic to ~WebViewHost. (TestShell::closeWindow): Move widget close() to WVH destructor and

manage focus manually.

(TestShell::closeRemainingWindows): Comment typo.

  • DumpRenderTree/chromium/WebViewHost.cpp: (WebViewHost::closeWidget): (invokeCloseWidget): (WebViewHost::closeWidgetSoon): Close the window asynchronously (matches

test_shell).

(WebViewHost::~WebViewHost):

  • DumpRenderTree/chromium/WebViewHost.h:
10:22 AM Changeset in webkit [71927] by mihaip@chromium.org
  • 5 edits
    9 adds in trunk/LayoutTests

2010-11-12 Mihai Parparita <​mihaip@chromium.org>

Unreviewed Chromium rebaseline.

Accumulated rebaselines that were missed due to lots of tests having
expected failures because of bug 48634.

  • platform/chromium-linux/fast/css/input-search-padding-expected.checksum: Added.
  • platform/chromium-linux/fast/css/input-search-padding-expected.png: Added.
  • platform/chromium-linux/fast/css/input-search-padding-expected.txt: Added.
  • platform/chromium-mac/fast/css/input-search-padding-expected.checksum: Added.
  • platform/chromium-mac/fast/css/input-search-padding-expected.png: Added.
  • platform/chromium-mac/fast/lists/markers-in-selection-expected.checksum: Added.
  • platform/chromium-mac/fast/lists/markers-in-selection-expected.png:
  • platform/chromium-mac/fast/repaint/block-layout-inline-children-replaced-expected.checksum:
  • platform/chromium-mac/fast/repaint/block-layout-inline-children-replaced-expected.png:
  • platform/chromium-win/fast/css/input-search-padding-expected.checksum: Added.
  • platform/chromium-win/fast/css/input-search-padding-expected.png: Added.
  • platform/chromium-win/fast/css/input-search-padding-expected.txt: Added.
  • platform/chromium/test_expectations.txt:
10:18 AM Changeset in webkit [71926] by levin@chromium.org
  • 2 edits in trunk/WebKitTools

cpp.py needs some tidying.
​https://bugs.webkit.org/show_bug.cgi?id=49443

Reviewed by Adam Barth.

  • Scripts/webkitpy/style/checkers/cpp.py: Fix parameter ordering for iteratively_replace_matches_with_char to mirror other similar functions (and improved comments, etc.) Also, use a python-ism (for/else) to get rid of a variable that was there to determine if something had been found in the loop.
10:18 AM Changeset in webkit [71925] by inferno@chromium.org
  • 5 edits
    3 adds in trunk

2010-11-11 Abhishek Arya <​inferno@chromium.org>

Reviewed by Adam Barth.

Not allow drag and drop across different origins.
​https://bugs.webkit.org/show_bug.cgi?id=49098

Test: http/tests/security/drag-drop-different-origin.html

  • page/DragController.cpp: (WebCore::DragController::tryDocumentDrag):
  • page/SecurityOrigin.cpp: (WebCore::SecurityOrigin::canDropOnTarget):
  • page/SecurityOrigin.h:

2010-11-10 Abhishek Arya <​inferno@chromium.org>

Reviewed by Adam Barth.

Check that drag and drop is not allowed across different origins.
​https://bugs.webkit.org/show_bug.cgi?id=49098


  • http/tests/security/drag-drop-different-origin-expected.txt: Added.
  • http/tests/security/drag-drop-different-origin.html: Added.
  • http/tests/security/resources/drag-drop.html: Added.
10:09 AM Changeset in webkit [71924] by mihaip@chromium.org
  • 2 edits in trunk/LayoutTests

2010-11-12 Mihai Parparita <​mihaip@chromium.org>

Unreviewed Chromium expectations update.

Now that we know better what is causing bug bug 48634, we can just
skip fast/images/size-failure.html instead of the 100+ tests that follow
it.

  • platform/chromium/test_expectations.txt:
9:59 AM Changeset in webkit [71923] by andersca@apple.com
  • 2 edits in trunk/JavaScriptCore

CString(const char*) crashes when passed a null pointer
​https://bugs.webkit.org/show_bug.cgi?id=49450

Reviewed by Adam Roben.

  • wtf/text/CString.cpp:

(WTF::CString::CString):
Return early if str is null.

9:55 AM Changeset in webkit [71922] by apavlov@chromium.org
  • 2 edits in trunk/WebCore

2010-11-11 Alexander Pavlov <​apavlov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: Inspect element tooltip obscures element metrics
​https://bugs.webkit.org/show_bug.cgi?id=47822

  • inspector/InspectorController.cpp: (WebCore::InspectorController::drawNodeHighlight): (WebCore::InspectorController::drawElementTitle):
9:43 AM Changeset in webkit [71921] by commit-queue@webkit.org
  • 2 edits in trunk/WebKitTools

2010-11-12 Robert Kroeger <​rjkroege@chromium.org>

Reviewed by James Robinson.

[Chromium/DRT] Make EventSendingController send complete touch events.
​https://bugs.webkit.org/show_bug.cgi?id=49285

  • DumpRenderTree/chromium/EventSender.cpp: (EventSender::addTouchPoint): (EventSender::updateTouchPoint): (EventSender::sendCurrentTouchEvent):
9:30 AM Changeset in webkit [71920] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

2010-11-12 Luiz Agostini <​luiz.agostini@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Button's background style from default Qt Mobile Theme is overriding facebook's button style
​https://bugs.webkit.org/show_bug.cgi?id=48198

Instead of creating gradients that goes from white to grey and cover the whole element the gradients
are created by varying the alpha channel of a dark color. This way the effect will not override the
background color of the elements.

  • css/themeQtMobile.css:
9:23 AM Changeset in webkit [71919] by andreas.kling@nokia.com
  • 2 edits in trunk/WebKit2

2010-11-12 Andreas Kling <​kling@webkit.org>

Reviewed by Antonio Gomes.

[Qt][WK2] Fix leaking of mapped files

MappedMemory::fileName was never set on the receiving side, which led
the MappedMemoryPool to call QFile::map() again when reusing an existing file.

  • Platform/qt/MappedMemoryPool.cpp: (WebKit::MappedMemoryPool::mapFile):
8:55 AM Changeset in webkit [71918] by jschuh@chromium.org
  • 3 edits
    2 adds in trunk

2010-11-12 Justin Schuh <​jschuh@chromium.org>

Reviewed by Adam Barth.

Use a RefPtr inside ContainerNode::willRemove loop
​https://bugs.webkit.org/show_bug.cgi?id=49237

Test: fast/innerHTML/innerHTML-iframe.html

  • dom/ContainerNode.cpp: (WebCore::ContainerNode::willRemove):

2010-11-12 Justin Schuh <​jschuh@chromium.org>

Reviewed by Adam Barth.

Use a RefPtr inside ContainerNode::willRemove loop
​https://bugs.webkit.org/show_bug.cgi?id=49237

  • fast/innerHTML/innerHTML-iframe-expected.txt: Added.
  • fast/innerHTML/innerHTML-iframe.html: Added.
8:41 AM Changeset in webkit [71917] by yurys@chromium.org
  • 7 edits in trunk/WebCore

2010-11-12 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: add more event listener breakpoint types, add support for regular breakpoint hit state, beautify hit rendering.
​https://bugs.webkit.org/show_bug.cgi?id=49130

  • English.lproj/localizedStrings.js:
  • inspector/front-end/BreakpointManager.js: (WebInspector.BreakpointManager.prototype.debuggerPaused): (WebInspector.Breakpoint.jsBreakpointId): (WebInspector.Breakpoint.prototype.get id):
  • inspector/front-end/BreakpointsSidebarPane.js: (WebInspector.EventListenerBreakpointsSidebarPane): (WebInspector.EventListenerBreakpointsSidebarPane.prototype._createCategory):
  • inspector/front-end/CallStackSidebarPane.js: (WebInspector.CallStackSidebarPane.prototype._breakpointHit): don't call populateStatusMessage for JS breakpoints since there is no specific message in that case.
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel): (WebInspector.ScriptsPanel.prototype.reset):
  • inspector/front-end/inspector.css: (#elements-sidebar): (.pane > .body): (#scripts-sidebar): (li.breakpoint-hit .breakpoint-hit-marker):
8:37 AM Changeset in webkit [71916] by mihaip@chromium.org
  • 3 edits
    1 delete in trunk/WebKitTools

2010-11-12 Mihai Parparita <​mihaip@chromium.org>

Unreviewed rollout of r71858.

Rollout out r71858 since it breaks new-run-webkit-httpd as used by the
NaCl tests.

  • Scripts/webkitpy/layout_tests/port/config.py:
  • Scripts/webkitpy/layout_tests/port/config_standalone.py: Removed.
  • Scripts/webkitpy/layout_tests/port/config_unittest.py:
8:36 AM Changeset in webkit [71915] by yael.aharon@nokia.com
  • 4 edits
    1 add in trunk/LayoutTests

2010-11-12 Yael Aharon <​yael.aharon@nokia.com>

Reviewed by Antonio Gomes.

Spatial Navigation: Need better way to control size of content in tests.
​https://bugs.webkit.org/show_bug.cgi?id=49398

Add an image and use it in a layout test to define fixed size for all ports.
This also fixes ​https://bugs.webkit.org/show_bug.cgi?id=43030

  • fast/events/spatial-navigation/resources/green.png: Added.
  • fast/events/spatial-navigation/snav-clipped-overflowed-content-expected.txt:
  • fast/events/spatial-navigation/snav-clipped-overflowed-content.html:
  • platform/qt/Skipped:
8:30 AM Changeset in webkit [71914] by Adam Roben
  • 1 edit
    1 add in trunk/LayoutTests

Skip a timing-out test on Windows XP

  • platform/win-xp/Skipped: Added. Skip

fast/workers/storage/change-version-handle-reuse-worker.html.

8:30 AM Changeset in webkit [71913] by Adam Roben
  • 1 edit
    6 adds in trunk/LayoutTests

Land new Windows results for tests added in r71851

  • platform/win/fast/repaint/japanese-rl-selection-clear-expected.checksum: Added.
  • platform/win/fast/repaint/japanese-rl-selection-clear-expected.png: Added.
  • platform/win/fast/repaint/japanese-rl-selection-clear-expected.txt: Added.
  • platform/win/fast/repaint/japanese-rl-selection-repaint-expected.checksum: Added.
  • platform/win/fast/repaint/japanese-rl-selection-repaint-expected.png: Added.
  • platform/win/fast/repaint/japanese-rl-selection-repaint-expected.txt: Added.
8:25 AM Changeset in webkit [71912] by bulach@chromium.org
  • 1 edit
    20 adds in trunk/LayoutTests

2010-11-12 Marcus Bulach <​bulach@chromium.org>

Not reviewed, Chromium test expectation update.

Update chromium baselines for svg following r71908 (missed mac in r71910).
​https://bugs.webkit.org/show_bug.cgi?id=49447

  • platform/chromium-mac/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-in-attr-expected.checksum: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-in-attr-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-in2-attr-expected.checksum: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-in2-attr-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-scale-attr-expected.checksum: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-scale-attr-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-xChannelSelector-attr-expected.checksum: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-xChannelSelector-attr-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-yChannelSelector-attr-expected.checksum: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-yChannelSelector-attr-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-in-prop-expected.checksum: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-in-prop-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-in2-prop-expected.checksum: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-in2-prop-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-scale-prop-expected.checksum: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-scale-prop-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-xChannelSelector-prop-expected.checksum: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-xChannelSelector-prop-expected.png: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-yChannelSelector-prop-expected.checksum: Added.
  • platform/chromium-mac/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-yChannelSelector-prop-expected.png: Added.
7:43 AM Changeset in webkit [71911] by caseq@chromium.org
  • 6 edits in trunk

2010-11-12 Andrey Kosyakov <​caseq@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: [Extensions API] [refactoring] generate public API interfaces automatically
Adjusting the order of properties in API interfaces dumps in the expectations.
​https://bugs.webkit.org/show_bug.cgi?id=49441

  • inspector/extensions-api-expected.txt:
  • inspector/extensions-audits-api-expected.txt:
  • inspector/extensions-expected.txt:

2010-11-12 Andrey Kosyakov <​caseq@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: [Extensions API] [refactoring] generate public API interfaces automatically
​https://bugs.webkit.org/show_bug.cgi?id=49441

7:39 AM Changeset in webkit [71910] by bulach@chromium.org
  • 1 edit
    40 adds in trunk/LayoutTests

2010-11-12 Marcus Bulach <​bulach@chromium.org>

Not reviewed, Chromium test expectation update.

Update chromium baselines for svg following r71908
​https://bugs.webkit.org/show_bug.cgi?id=49440

  • platform/chromium-linux/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-in-attr-expected.checksum: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-in-attr-expected.png: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-in2-attr-expected.checksum: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-in2-attr-expected.png: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-scale-attr-expected.checksum: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-scale-attr-expected.png: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-xChannelSelector-attr-expected.checksum: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-xChannelSelector-attr-expected.png: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-yChannelSelector-attr-expected.checksum: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-yChannelSelector-attr-expected.png: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-in-prop-expected.checksum: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-in-prop-expected.png: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-in2-prop-expected.checksum: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-in2-prop-expected.png: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-scale-prop-expected.checksum: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-scale-prop-expected.png: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-xChannelSelector-prop-expected.checksum: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-xChannelSelector-prop-expected.png: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-yChannelSelector-prop-expected.checksum: Added.
  • platform/chromium-linux/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-yChannelSelector-prop-expected.png: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-in-attr-expected.checksum: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-in-attr-expected.png: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-in2-attr-expected.checksum: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-in2-attr-expected.png: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-scale-attr-expected.checksum: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-scale-attr-expected.png: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-xChannelSelector-attr-expected.checksum: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-xChannelSelector-attr-expected.png: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-yChannelSelector-attr-expected.checksum: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-yChannelSelector-attr-expected.png: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-in-prop-expected.checksum: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-in-prop-expected.png: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-in2-prop-expected.checksum: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-in2-prop-expected.png: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-scale-prop-expected.checksum: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-scale-prop-expected.png: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-xChannelSelector-prop-expected.checksum: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-xChannelSelector-prop-expected.png: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-yChannelSelector-prop-expected.checksum: Added.
  • platform/chromium-win/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-yChannelSelector-prop-expected.png: Added.
7:25 AM Changeset in webkit [71909] by Adam Roben
  • 2 edits in trunk/WebKit2

Windows build fix

  • win/WebKit2Generated.make: Copy WKBundlePageOverlay.h.
5:52 AM Changeset in webkit [71908] by reni@webkit.org
  • 4 edits
    50 adds in trunk

WebCore: SVGFEDisplacementMapElement doesn't support dynamic invalidation
​https://bugs.webkit.org/show_bug.cgi?id=49430

Patch by Renata Hodovan <​reni@webkit.org> on 2010-11-12
Reviewed by Nikolas Zimmermann.

The dynamic changes are captured by the svgAttributeChanged function. Invalidate the filter primitive if necessary.

Tests: svg/dynamic-updates/SVGFEDisplacementMapElement-dom-in-attr.html

svg/dynamic-updates/SVGFEDisplacementMapElement-dom-in2-attr.html
svg/dynamic-updates/SVGFEDisplacementMapElement-dom-scale-attr.html
svg/dynamic-updates/SVGFEDisplacementMapElement-dom-xChannelSelector-attr.html
svg/dynamic-updates/SVGFEDisplacementMapElement-dom-yChannelSelector-attr.html
svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-in-prop.html
svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-in2-prop.html
svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-scale-prop.html
svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-xChannelSelector-prop.html
svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-yChannelSelector-prop.html

  • svg/SVGFEDisplacementMapElement.cpp:

(WebCore::SVGFEDisplacementMapElement::svgAttributeChanged):

  • svg/SVGFEDisplacementMapElement.h:

LayoutTests: SVGFEDisplacementMapElement doesn't support dynamic invalidation, when attributes change.
​https://bugs.webkit.org/show_bug.cgi?id=49430

Patch by Renata Hodovan <​reni@webkit.org> on 2010-11-12
Reviewed by Nikolas Zimmermann.

Adding layout tests for feDisplacementMap dynamic changes.

  • platform/mac/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-in-attr-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-in-attr-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-in2-attr-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-in2-attr-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-scale-attr-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-scale-attr-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-xChannelSelector-attr-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-xChannelSelector-attr-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-yChannelSelector-attr-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDisplacementMapElement-dom-yChannelSelector-attr-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-in-prop-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-in-prop-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-in2-prop-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-in2-prop-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-scale-prop-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-scale-prop-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-xChannelSelector-prop-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-xChannelSelector-prop-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-yChannelSelector-prop-expected.checksum: Added.
  • platform/mac/svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-yChannelSelector-prop-expected.png: Added.
  • svg/dynamic-updates/SVGFEDisplacementMapElement-dom-in-attr-expected.txt: Added.
  • svg/dynamic-updates/SVGFEDisplacementMapElement-dom-in-attr.html: Added.
  • svg/dynamic-updates/SVGFEDisplacementMapElement-dom-in2-attr-expected.txt: Added.
  • svg/dynamic-updates/SVGFEDisplacementMapElement-dom-in2-attr.html: Added.
  • svg/dynamic-updates/SVGFEDisplacementMapElement-dom-scale-attr-expected.txt: Added.
  • svg/dynamic-updates/SVGFEDisplacementMapElement-dom-scale-attr.html: Added.
  • svg/dynamic-updates/SVGFEDisplacementMapElement-dom-xChannelSelector-attr-expected.txt: Added.
  • svg/dynamic-updates/SVGFEDisplacementMapElement-dom-xChannelSelector-attr.html: Added.
  • svg/dynamic-updates/SVGFEDisplacementMapElement-dom-yChannelSelector-attr-expected.txt: Added.
  • svg/dynamic-updates/SVGFEDisplacementMapElement-dom-yChannelSelector-attr.html: Added.
  • svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-in-prop-expected.txt: Added.
  • svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-in-prop.html: Added.
  • svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-in2-prop-expected.txt: Added.
  • svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-in2-prop.html: Added.
  • svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-scale-prop-expected.txt: Added.
  • svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-scale-prop.html: Added.
  • svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-xChannelSelector-prop-expected.txt: Added.
  • svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-xChannelSelector-prop.html: Added.
  • svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-yChannelSelector-prop-expected.txt: Added.
  • svg/dynamic-updates/SVGFEDisplacementMapElement-svgdom-yChannelSelector-prop.html: Added.
  • svg/dynamic-updates/script-tests/SVGFEDisplacementMapElement-dom-in-attr.js: Added.

(executeTest):

  • svg/dynamic-updates/script-tests/SVGFEDisplacementMapElement-dom-in2-attr.js: Added.

(executeTest):

  • svg/dynamic-updates/script-tests/SVGFEDisplacementMapElement-dom-scale-attr.js: Added.

(executeTest):

  • svg/dynamic-updates/script-tests/SVGFEDisplacementMapElement-dom-xChannelSelector-attr.js: Added.

(executeTest):

  • svg/dynamic-updates/script-tests/SVGFEDisplacementMapElement-dom-yChannelSelector-attr.js: Added.

(executeTest):

  • svg/dynamic-updates/script-tests/SVGFEDisplacementMapElement-svgdom-in-prop.js: Added.

(executeTest):

  • svg/dynamic-updates/script-tests/SVGFEDisplacementMapElement-svgdom-in2-prop.js: Added.

(executeTest):

  • svg/dynamic-updates/script-tests/SVGFEDisplacementMapElement-svgdom-scale-prop.js: Added.

(executeTest):

  • svg/dynamic-updates/script-tests/SVGFEDisplacementMapElement-svgdom-xChannelSelector-prop.js: Added.

(executeTest):

  • svg/dynamic-updates/script-tests/SVGFEDisplacementMapElement-svgdom-yChannelSelector-prop.js: Added.

(executeTest):

5:45 AM Changeset in webkit [71907] by benjamin.poulain@nokia.com
  • 3 edits in trunk/WebKit2

2010-11-12 Benjamin Poulain <​benjamin.poulain@nokia.com>

Unreviewed fix after r71857.

  • UIProcess/qt/WebInspectorProxyQt.cpp:
  • WebProcess/WebPage/qt/WebInspectorQt.cpp:
5:34 AM Changeset in webkit [71906] by caseq@chromium.org
  • 4 edits in trunk/WebCore

2010-11-12 Andrey Kosyakov <​caseq@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: [refactoring] do not use absolute positioning for tabbed panes
​https://bugs.webkit.org/show_bug.cgi?id=49439

  • inspector/front-end/TabbedPane.js: (WebInspector.TabbedPane): (WebInspector.TabbedPane.prototype.appendTab):
  • inspector/front-end/inspector.css: (.resource-view.visible): (.resource-view .tabbed-pane-header): (.resource-view-cookies): (.tabbed-pane): (.tabbed-pane-content):
  • inspector/front-end/networkPanel.css:
4:49 AM Changeset in webkit [71905] by inferno@chromium.org
  • 11 edits
    8 copies in branches/chromium/552

Merge 71895

BUG=61313

4:42 AM Changeset in webkit [71904] by inferno@chromium.org
  • 3 edits
    3 copies in branches/chromium/552

Merge 71797

BUG=61313

4:04 AM Changeset in webkit [71903] by commit-queue@webkit.org
  • 18 edits
    5 moves in trunk

2010-11-12 John Knottenbelt <​jknotten@chromium.org>

Reviewed by Steve Block.

Rename GeolocationControllerClient to GeolocationClient.
​https://bugs.webkit.org/show_bug.cgi?id=49259

No new tests, renaming only.

  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.order:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • page/GeolocationClient.h: Renamed from WebCore/page/GeolocationControllerClient.h. (WebCore::GeolocationClient::~GeolocationClient):
  • page/GeolocationController.cpp: (WebCore::GeolocationController::GeolocationController):
  • page/GeolocationController.h:
  • page/Page.h:

2010-11-12 John Knottenbelt <​jknotten@chromium.org>

Reviewed by Steve Block.

Rename GeolocationControllerClient to GeolocationClient.
​https://bugs.webkit.org/show_bug.cgi?id=49259

  • WebKit.xcodeproj/project.pbxproj:

2010-11-12 John Knottenbelt <​jknotten@chromium.org>

Reviewed by Steve Block.

Rename GeolocationControllerClient to GeolocationClient.
​https://bugs.webkit.org/show_bug.cgi?id=49259

  • WebCoreSupport/WebGeolocationClient.h: Renamed from WebKit/mac/WebCoreSupport/WebGeolocationControllerClient.h. (WebGeolocationClient::webView): (WebGeolocationClient::setEnableHighAccuracy):
  • WebCoreSupport/WebGeolocationClient.mm: Renamed from WebKit/mac/WebCoreSupport/WebGeolocationControllerClient.mm. (WebGeolocationClient::WebGeolocationClient): (WebGeolocationClient::geolocationDestroyed): (WebGeolocationClient::startUpdating): (WebGeolocationClient::stopUpdating): (WebGeolocationClient::lastPosition):
  • WebKit.order:
  • WebView/WebView.mm: (-[WebView _commonInitializationWithFrameName:groupName:usesDocumentViews:]):

2010-11-12 John Knottenbelt <​jknotten@chromium.org>

Reviewed by Steve Block.

Rename GeolocationControllerClient to GeolocationClient.
​https://bugs.webkit.org/show_bug.cgi?id=49259

  • WebCoreSupport/WebGeolocationClient.cpp: Renamed from WebKit/win/WebCoreSupport/WebGeolocationControllerClient.cpp. (WebGeolocationClient::WebGeolocationClient): (WebGeolocationClient::geolocationDestroyed): (WebGeolocationClient::startUpdating): (WebGeolocationClient::stopUpdating): (WebGeolocationClient::lastPosition):
  • WebCoreSupport/WebGeolocationClient.h: Renamed from WebKit/win/WebCoreSupport/WebGeolocationControllerClient.h. (WebGeolocationClient::setEnableHighAccuracy):
  • WebKit.vcproj/WebKit.vcproj:
  • WebView.cpp: (WebView::initWithFrame):
3:30 AM Changeset in webkit [71902] by benjamin.poulain@nokia.com
  • 2 edits in trunk/WebKitTools

2010-11-12 Benjamin Poulain <​benjamin.poulain@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] TestController::runUntil doesn't honor the timeout parameter
​https://bugs.webkit.org/show_bug.cgi?id=48941

Implement the missing timeout of WebKitTestRunner for Qt.

A QElapsedTimer has been added to RunUntilConditionLoop in order
to measure how long the loop has been running. When the timer
is bigger or equal than the timeout, the loop ends even if the
condition is not met.

  • WebKitTestRunner/qt/TestControllerQt.cpp: (WTR::RunUntilConditionLoop::start): (WTR::RunUntilConditionLoop::run): (WTR::RunUntilConditionLoop::timerEvent): (WTR::TestController::platformRunUntil):
3:04 AM Changeset in webkit [71901] by bulach@chromium.org
  • 2 edits in trunk/LayoutTests

2010-11-12 Marcus Bulach <​bulach@chromium.org>

Not reviewed, Chromium test expectation update.

Updates test expectations for chromium following r71886
​https://bugs.webkit.org/show_bug.cgi?id=49436

Temporarily disables editing/selection/context-menu-on-text.html until we fix test_shell.

  • platform/chromium/test_expectations.txt:
2:39 AM Changeset in webkit [71900] by commit-queue@webkit.org
  • 10 edits in trunk

2010-11-12 Ilya Sherman <​isherman@chromium.org>

Reviewed by Eric Seidel.

Remove some trailing whitespace
​https://bugs.webkit.org/show_bug.cgi?id=49433

  • platform/PopupMenuClient.h:
  • platform/chromium/PopupMenuChromium.cpp: (WebCore::PopupContainer::show): (WebCore::PopupListBox::paintRow): (WebCore::PopupListBox::scrollToRevealRow):
  • platform/chromium/PopupMenuChromium.h:
  • platform/chromium/SearchPopupMenuChromium.cpp:
  • rendering/RenderMenuList.h: (WebCore::RenderMenuList::toRenderMenuList):
  • rendering/RenderTextControlSingleLine.h: (WebCore::toRenderTextControlSingleLine):

2010-11-12 Ilya Sherman <​isherman@chromium.org>

Reviewed by Eric Seidel.

Remove some trailing whitespace
​https://bugs.webkit.org/show_bug.cgi?id=49433

  • src/ChromeClientImpl.cpp: (WebKit::ChromeClientImpl::createPopupMenu):
  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::scrollRootLayerRect): (WebKit::WebViewImpl::invalidateRootLayerRect):
1:26 AM Changeset in webkit [71899] by rniwa@webkit.org
  • 4 edits in trunk

2010-11-12 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Csaba Osztrogonác.

[qt] script-clone tests added in r71895 fails on qt platform
​https://bugs.webkit.org/show_bug.cgi?id=49429

Enabled fast/dom/script-clone-rerun-self.html and fast/dom/script-clone-rerun.html

  • platform/qt/Skipped:

2010-11-12 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Csaba Osztrogonác.

[qt] script-clone tests added in r71895 fails on qt platform
​https://bugs.webkit.org/show_bug.cgi?id=49429

Fixed parseEndElement to call ScriptElement's executeScript as supposed to that of ScriptController.

  • dom/XMLDocumentParserQt.cpp: (WebCore::XMLDocumentParser::parseEndElement):
1:24 AM Changeset in webkit [71898] by morrita@google.com
  • 4 edits in trunk/WebCore

2010-11-05 MORITA Hajime <​morrita@google.com>

Reviewed by Kent Tamura.

Refactoring: Return values of TextCheckingHelper::paragraphAlignedRange should form a class.
​https://bugs.webkit.org/show_bug.cgi?id=49053

Replaced paragraphAlignedRange() function call into
TextCheckingParagraph class. The class encapsulates a range of a
paragraph, and its relation within associated checking range.
Instance variables of the class are computed lazily. So we can save unnecessary
memory allocation and DOM tree traversal.

No new tests, no behavior change.

  • editing/Editor.cpp: (WebCore::Editor::markAllMisspellingsAndBadGrammarInRanges): Replaced paragraphAlignedRange() call. (WebCore::Editor::changeBackToReplacedString): Replaced paragraphAlignedRange() call.
  • editing/TextCheckingHelper.cpp: (WebCore::expandToParagraphBoundary): (WebCore::TextCheckingParagraph::TextCheckingParagraph): (WebCore::TextCheckingParagraph::~TextCheckingParagraph): (WebCore::TextCheckingParagraph::expandRangeToNextEnd): (WebCore::TextCheckingParagraph::invalidateParagraphRangeValues): (WebCore::TextCheckingParagraph::rangeLength): (WebCore::TextCheckingParagraph::paragraphRange): (WebCore::TextCheckingParagraph::subrange): (WebCore::TextCheckingParagraph::offsetTo): (WebCore::TextCheckingParagraph::isEmpty): (WebCore::TextCheckingParagraph::offsetAsRange): (WebCore::TextCheckingParagraph::text): (WebCore::TextCheckingParagraph::checkingStart): (WebCore::TextCheckingParagraph::checkingEnd): (WebCore::TextCheckingParagraph::checkingLength): (WebCore::TextCheckingHelper::findFirstBadGrammar): Replaced paragraphAlignedRange() call. (WebCore::TextCheckingHelper::guessesForMisspelledOrUngrammaticalRange): Replaced paragraphAlignedRange() call.
  • editing/TextCheckingHelper.h: (WebCore::TextCheckingParagraph::textLength): (WebCore::TextCheckingParagraph::textSubstring): (WebCore::TextCheckingParagraph::textCharacters): (WebCore::TextCheckingParagraph::textCharAt): (WebCore::TextCheckingParagraph::checkingSubstring): (WebCore::TextCheckingParagraph::checkingRangeMatches): (WebCore::TextCheckingParagraph::isCheckingRangeCoveredBy): (WebCore::TextCheckingParagraph::checkingRangeCovers): (WebCore::TextCheckingParagraph::checkingRange): (WebCore::TextCheckingParagraph::isTextEmpty): (WebCore::TextCheckingParagraph::isRangeEmpty):
Note: See TracTimeline for information about the timeline view.