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

Timeline



Jun 29, 2011:

11:30 PM Changeset in webkit [90089] by tkent@chromium.org
  • 41 edits in trunk

2011-06-29 Kent Tamura <​tkent@chromium.org>

Reviewed by Dimitri Glazkov.

Avoid custom layout code of RenderTextControlSingleLine
​https://bugs.webkit.org/show_bug.cgi?id=61415

Update tests including <input type=search> or <input type=number>. Their
internal structures are changed.
Slight position changes for search result buttons and search cancel
buttons are expected.

  • fast/css/text-input-with-webkit-border-radius-expected.txt:
  • platform/chromium/test_expectations.txt:
  • platform/mac/fast/css/input-search-padding-expected.png:
  • platform/mac/fast/css/input-search-padding-expected.txt:
  • platform/mac/fast/css/pseudo-cache-stale-expected.txt:
  • platform/mac/fast/forms/box-shadow-override-expected.txt:
  • platform/mac/fast/forms/control-restrict-line-height-expected.txt:
  • platform/mac/fast/forms/input-appearance-height-expected.txt:
  • platform/mac/fast/forms/input-appearance-spinbutton-disabled-readonly-expected.txt:
  • platform/mac/fast/forms/input-appearance-spinbutton-expected.txt:
  • platform/mac/fast/forms/input-appearance-spinbutton-layer-expected.txt:
  • platform/mac/fast/forms/input-appearance-spinbutton-up-expected.txt:
  • platform/mac/fast/forms/input-appearance-spinbutton-visibility-expected.txt:
  • platform/mac/fast/forms/placeholder-position-expected.txt:
  • platform/mac/fast/forms/placeholder-pseudo-style-expected.txt:
  • platform/mac/fast/forms/placeholder-set-value-expected.txt:
  • platform/mac/fast/forms/search-cancel-button-style-sharing-expected.txt:
  • platform/mac/fast/forms/search-display-none-cancel-button-expected.txt:
  • platform/mac/fast/forms/search-placeholder-value-changed-expected.txt:
  • platform/mac/fast/forms/search-rtl-expected.txt:
  • platform/mac/fast/forms/search-styled-expected.txt:
  • platform/mac/fast/forms/search-vertical-alignment-expected.png:
  • platform/mac/fast/forms/search-vertical-alignment-expected.txt:
  • platform/mac/fast/forms/searchfield-heights-expected.txt:
  • platform/mac/fast/repaint/search-field-cancel-expected.png:
  • platform/mac/fast/repaint/search-field-cancel-expected.txt:
  • platform/mac/fast/replaced/width100percent-searchfield-expected.txt:
  • platform/mac/fast/table/colspanMinWidth-vertical-expected.png:
  • platform/mac/fast/table/colspanMinWidth-vertical-expected.txt:

2011-06-29 Kent Tamura <​tkent@chromium.org>

Reviewed by Dimitri Glazkov.

Avoid custom layout code of RenderTextControlSingleLine
​https://bugs.webkit.org/show_bug.cgi?id=61415

If a text field has a search results button, a search cancel
button, a speech button, or a spin button, the editable inner text
element and the buttons are wrapped by a flexible box container,
and the editable inner text element has flexibility. As the result
of applying the flexible box, we don't need any horizontal layout
code in RenderTextControlSingleLine except the inner spin button.

Also, we remove custom hit-testing code in nodeAtPoint(). If a
user clicks on a position left of the inner editable text but not
on the search results button, the search results button doesn't
work though it worked.

  • css/html.css: (input::-webkit-textfield-decoration-container): New style definition for the "container" element. (input[type="search"]::-webkit-search-cancel-button): It should be a block element because it's a child of the flexible box. Also, this doesn't have flexibility. (input[type="search"]::-webkit-search-decoration): ditto. (input[type="search"]::-webkit-search-results-decoration): ditto. (input[type="search"]::-webkit-search-results-button): ditto. (input::-webkit-inner-spin-button): ditto. (input::-webkit-input-speech-button): ditto.
  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::containerElement): Add an accessor of the flexible box container. It just calls InputType::containerElement().
  • html/HTMLInputElement.h: Declare it.
  • html/InputType.h: (WebCore::InputType::containerElement): New accessor for a shadow element.
  • html/SearchInputType.cpp: (WebCore::SearchInputType::SearchInputType): m_innerBlock is moved to TextFieldInputType. (WebCore::SearchInputType::needsContainer): A search field always needs the flexible box container because of the results button and the cancel button. (WebCore::SearchInputType::createShadowSubtree): Use TextFieldInputType::createShadowSubtree(), then modify the tree. (WebCore::SearchInputType::destroyShadowSubtree): m_innerBlock is moved to TextFieldInputType.
  • html/SearchInputType.h:
  • html/TextFieldInputType.cpp: (WebCore::TextFieldInputType::TextFieldInputType): Initialize the new members. (WebCore::TextFieldInputType::needsContainer): Returns true if x-webkit-speech. We check a spin button separately. (WebCore::TextFieldInputType::createShadowSubtree): Wrap everything with the flexible box container if needsContainer() or we need a spin button. (WebCore::TextFieldInputType::containerElement): Added an accessor. (WebCore::TextFieldInputType::innerBlockElement): ditto. (WebCore::TextFieldInputType::innerTextElement): ditto. (WebCore::TextFieldInputType::innerSpinButtonElement): ditto. (WebCore::TextFieldInputType::speechButtonElement): ditto. (WebCore::TextFieldInputType::destroyShadowSubtree): Clear references to new shadow nodes.
  • html/TextFieldInputType.h:
  • rendering/RenderTextControlSingleLine.cpp: (WebCore::RenderTextControlSingleLine::containerElement): Added an accessor. (WebCore::RenderTextControlSingleLine::innerTextElement): (WebCore::RenderTextControlSingleLine::layout):
    • Correct heights check Comparing with height(), not contentHeigth(), was incorrect.
    • Adjust the container height like the editable inner text.
    • Remove width adjustment
    • Remove the speech button layout code The flexible box layout is enough.

(WebCore::RenderTextControlSingleLine::nodeAtPoint):

Remove custom code for results/cancel/speech buttons, and
Simplify comments.

(WebCore::RenderTextControlSingleLine::forwardEvent):

Remove custom forwarding code for results/cancel buttons.

(WebCore::RenderTextControlSingleLine::styleDidChange):

Clear the style of the container because we update it in layout().

(WebCore::RenderTextControlSingleLine::hasControlClip):

Decoration buttons should be clipped by the container.

(WebCore::RenderTextControlSingleLine::controlClipRect): ditto.
(WebCore::RenderTextControlSingleLine::createInnerTextStyle):

We don't need to use display:inline-block any more.

(WebCore::RenderTextControlSingleLine::createInnerBlockStyle):

The text wrapper element should have flexibility.

(WebCore::RenderTextControlSingleLine::textBlockInsetTop):

Updated for the tree structure change.

  • rendering/RenderTextControlSingleLine.h:
10:39 PM Changeset in webkit [90088] by tkent@chromium.org
  • 2 edits in trunk/Source/WebCore

2011-06-29 Kent Tamura <​tkent@chromium.org>

[Mac] Attempt to fix Leopard build.

  • platform/mac/CursorMac.mm: (WebCore::Cursor::ensurePlatformCursor):
10:12 PM Changeset in webkit [90087] by Darin Adler
  • 11 edits
    2 deletes in trunk/Source

2011-06-29 Darin Adler <Darin Adler>

Reviewed by Dan Bernstein.

[Mac] Use system cursors instead of custom bitmaps where possible
​https://bugs.webkit.org/show_bug.cgi?id=63679

  • WebCoreSupport/WebSystemInterface.mm: (InitWebCoreSystemInterface): Added WKCursor.

2011-06-29 Darin Adler <Darin Adler>

Reviewed by Dan Bernstein.

[Mac] Use system cursors instead of custom bitmaps where possible
​https://bugs.webkit.org/show_bug.cgi?id=63679

  • Resources/crossHairCursor.png: Removed.
  • Resources/notAllowedCursor.png: Removed.
  • WebCore.exp.in: Added wkCursor.
  • WebCore/WebCore.gyp/WebCore.gyp: Removed the two now-unused PNG files.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • platform/mac/CursorMac.mm: (WebCore::Cursor::ensurePlatformCursor): Use NSCursor API for Cross, Hand, VerticalText, ContextMenu, Copy, and NotAllowed. Use wkCursor for Move, Alias, Progress, and the resize and panning cursors.
  • platform/mac/WebCoreSystemInterface.h: Added wkCursor.
  • platform/mac/WebCoreSystemInterface.mm: Ditto.

2011-06-29 Darin Adler <Darin Adler>

Reviewed by Dan Bernstein.

[Mac] Use system cursors instead of custom bitmaps where possible
​https://bugs.webkit.org/show_bug.cgi?id=63679

  • WebProcess/WebCoreSupport/mac/WebSystemInterface.mm: (InitWebCoreSystemInterface): Added WKCursor.
9:16 PM Changeset in webkit [90086] by dpranke@chromium.org
  • 2 edits in trunk/Tools

2011-06-29 Dirk Pranke <​dpranke@chromium.org>

Reviewed by Adam Barth.

test-webkitpy is spamming lots of messages about updating test expectations
​https://bugs.webkit.org/show_bug.cgi?id=63680

This patch changes the passing_run() function to capture the
output to stdout and stderr and assert that it is empty, and
also specifies --no-new-test-results by default to avoid getting
"generating baseline" messages for the expected-missing results.

Also check that we are logging messages when we generate new
baselines.

Also fix a minor bug in the way test_tolerance() was
checking for whether the tests were passing or not.

  • Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py:
8:40 PM Changeset in webkit [90085] by yutak@chromium.org
  • 6 edits in trunk/Tools

2011-06-29 Yuta Kitamura <​yutak@chromium.org>

Reviewed by Kent Tamura.

DRT: Make Hixie76WebSocketProtocolEnabled preference flag configurable from LayoutTestController
​https://bugs.webkit.org/show_bug.cgi?id=63532

LayoutTestController::overridePreference() will be used to change the value of
the Hixie76WebSocketProtocolEnabled preference flag from layout tests. In this way,
we can avoid writing a lot of boilerplate code in each port's DumpRenderTree, but
at least we need to reset preference values in DumpRenderTree every time we start
a new test because preference values changed in some test should not be leaked
to the next test.

  • DumpRenderTree/chromium/LayoutTestController.cpp: (LayoutTestController::overridePreference): The name of preference key is defined in WebKit/{mac/WebView,win}/WebPreferenceKeysPrivate.h.
  • DumpRenderTree/chromium/WebPreferences.cpp: (WebPreferences::reset): Set the default value. This will be called from TestShell::resetTestController(). (WebPreferences::applyTo):
  • DumpRenderTree/chromium/WebPreferences.h:
  • DumpRenderTree/mac/DumpRenderTree.mm: (resetDefaultsToConsistentValues): Set the default value.
  • DumpRenderTree/win/DumpRenderTree.cpp: (resetDefaultsToConsistentValues): Ditto.
8:34 PM Changeset in webkit [90084] by Simon Fraser
  • 3 edits in trunk/LayoutTests

2011-06-29 Simon Fraser <Simon Fraser>

Fix the test description.

  • fast/layers/negative-scroll-positions-expected.txt:
  • fast/layers/script-tests/negative-scroll-positions.js:
8:07 PM Changeset in webkit [90083] by bashi@chromium.org
  • 3 edits in trunk/Tools

2011-06-29 Kenichi Ishibashi <​bashi@chromium.org>

Reviewed by Adam Barth.

check-webkit-style fails to detect ​https://webkit.org/b/XXXXX as a bug number
​https://webkit.org/b/57360

Modify the regular expression to match https scheme.

  • Scripts/webkitpy/common/checkout/changelog_unittest.py: Added a test case for ​https://webkit.org/b/XXXXX.
  • Scripts/webkitpy/common/config/urls.py: Modified bug_url_short to match https scheme.
8:05 PM Changeset in webkit [90082] by keishi@webkit.org
  • 6 edits
    3 adds in trunk

2011-06-29 Keishi Hattori <​keishi@webkit.org>

Reviewed by Adam Barth.

Remove extra space at EOL in view source mode
​https://bugs.webkit.org/show_bug.cgi?id=63620

  • fast/frames/resources/viewsource-frame-5.html: Added.
  • fast/frames/viewsource-empty-attribute-value-expected.txt: Regenerated.
  • fast/frames/viewsource-linebreak-expected.txt: Added.
  • fast/frames/viewsource-linebreak.html: Added. Test that there are no extra spaces at EOL. And that the multiple consecutive line breaks are maintained in the frame.

2011-06-29 Keishi Hattori <​keishi@webkit.org>

Reviewed by Adam Barth.

Remove extra space at EOL in view source mode
​https://bugs.webkit.org/show_bug.cgi?id=63620

Test: fast/frames/viewsource-linebreak.html

  • css/view-source.css: (tbody:last-child .webkit-line-content:empty:before): If we don't do this the last line will be empty and the height will be shorter than the rest.
  • html/HTMLViewSourceDocument.cpp: (WebCore::HTMLViewSourceDocument::addSource): If source file is empty, add an empty line. (WebCore::HTMLViewSourceDocument::finishLine): Call when you are done with the current line. Adds a BR element if the line is empty and sets current to tbody. (WebCore::HTMLViewSourceDocument::addText):
  • html/HTMLViewSourceDocument.h:
7:30 PM Changeset in webkit [90081] by abarth@webkit.org
  • 2 edits in trunk/Tools

2011-06-29 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

new-run-webkit-tests: images page switch actual and expected images before they're loaded
​https://bugs.webkit.org/show_bug.cgi?id=63199

We now preload the images and keep the DOM nodes in memory instead of
going back to the MemoryCache all the time. (Also, remove some nutty
</img> HTML.)

  • Scripts/webkitpy/layout_tests/layout_package/test_result_writer.py:
7:06 PM Changeset in webkit [90080] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebKit/efl

2011-06-29 Kangil Han <​kangil.han@samsung.com>

Reviewed by Kenneth Rohde Christiansen.

[EFL] Add APIs to get/set view mode
​https://bugs.webkit.org/show_bug.cgi?id=63229

Implement public API for getting/setting the view-mode media feature as specified in W3C.

  • ewk/ewk_view.cpp: (ewk_view_mode_set): (ewk_view_mode_get):
  • ewk/ewk_view.h:
7:02 PM Changeset in webkit [90079] by abarth@webkit.org
  • 2 edits in trunk/Tools

2011-06-29 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

new-run-webkit-tests should tell you when it creates new expectations
​https://bugs.webkit.org/show_bug.cgi?id=63675

Unforunately, this patch creates a bunch of test-webkitpy spam because
we don't understand how Python logging works well enough to stop the
spam. :(

  • Scripts/webkitpy/layout_tests/layout_package/single_test_runner.py:
6:48 PM Changeset in webkit [90078] by eric@webkit.org
  • 2 edits in trunk/Tools

2011-06-29 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Refactor WebKitDriver.run_test into smaller pieces
​https://bugs.webkit.org/show_bug.cgi?id=63673

There is no functional change here. Just moving code around.
This is in preparation for adding support for #CRASHED and #CRASHED - WebProcess

  • Scripts/webkitpy/layout_tests/port/webkit.py:
6:36 PM Changeset in webkit [90077] by eric@webkit.org
  • 15 edits in trunk/Tools

2011-06-29 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Simplify skipped file finding in preparation for adding wk2 skipped list fallback
​https://bugs.webkit.org/show_bug.cgi?id=63501

Re-landing after fixing a bug in WebKitPort.baseline_search_path which broke
the rebaselineserver unit tests. Also fixed nits brought up by Dirk after previous landing.

The goal was to add support for wk2 skipped lists.
However, I found that our skipped list computation was a
manual hack (which only worked for the mac port).

So I fixed a FIXME to move the skipped list finding
into WebKitPort instead of MacPort.
Doing so required the concept of a "port_name", since previously
the only name accessible from a port object was name()
which includes many things beyond "mac" or "chromium", etc.

Eventually I believe we'll want to re-think the way that we pass
in a port_name argument to Port subclasses and expect them to parse
it. But for now I just added a cls.port_name variable which contains
the static information needed to compute wk2 names as well as
compute Skipped list fallback which works for Mac/Win/Qt and Gtk.

In order to test my new _skipped_file_search_paths method, I
fixed another FIXME by making it return relative paths.

I also fixed the test_expectations_path code in WebKitPort to use port_name.
It was using name() which would return PORT-VERSION so MacPort was overriding
it to use just PORT. After fixing test_expectations_path to use port_name
(and making it aware of webkit2) I was able to remove the MacPort implementation.

  • Scripts/webkitpy/layout_tests/port/base.py:
    • Add port_name() to access "mac", since name() returns "mac-leopard" etc.
    • Document that real_name() seems to have no purpose.
  • Scripts/webkitpy/layout_tests/port/chromium_linux.py:
    • Add _parse_port_name(), eventually we might call this from WebKitPort directly.
  • Scripts/webkitpy/layout_tests/port/chromium_mac.py:
    • Add _parse_port_name.
  • Scripts/webkitpy/layout_tests/port/chromium_win.py:
    • Add _parse_port_name.
  • Scripts/webkitpy/layout_tests/port/gtk.py:
  • Scripts/webkitpy/layout_tests/port/mac.py:
    • Move Skipped-file finding code down to WebKitPort
  • Scripts/webkitpy/layout_tests/port/mac_unittest.py:
  • Scripts/webkitpy/layout_tests/port/qt.py:
  • Scripts/webkitpy/layout_tests/port/webkit.py:
  • Scripts/webkitpy/layout_tests/port/webkit_unittest.py:
6:11 PM Changeset in webkit [90076] by commit-queue@webkit.org
  • 14 edits in trunk/Tools

2011-06-29 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r90070.
​http://trac.webkit.org/changeset/90070
​https://bugs.webkit.org/show_bug.cgi?id=63676

Broke test-webkitpy rebaseline tests (Requested by eseidel2 on
#webkit).

  • Scripts/webkitpy/layout_tests/port/base.py:
  • Scripts/webkitpy/layout_tests/port/chromium.py:
  • Scripts/webkitpy/layout_tests/port/chromium_linux.py:
  • Scripts/webkitpy/layout_tests/port/chromium_mac.py:
  • Scripts/webkitpy/layout_tests/port/chromium_win.py:
  • Scripts/webkitpy/layout_tests/port/gtk.py:
  • Scripts/webkitpy/layout_tests/port/mac.py:
  • Scripts/webkitpy/layout_tests/port/mac_unittest.py:
  • Scripts/webkitpy/layout_tests/port/qt.py:
  • Scripts/webkitpy/layout_tests/port/webkit.py:
  • Scripts/webkitpy/layout_tests/port/webkit_unittest.py:
  • Scripts/webkitpy/layout_tests/port/win.py:
  • Scripts/webkitpy/tool/commands/rebaselineserver_unittest.py:
5:46 PM Changeset in webkit [90075] by bfulgham@webkit.org
  • 3 edits in trunk/Source/WebCore

[WinCE] Correct logical state of WINCE test.

  • platform/win/ScrollbarThemeWin.cpp: The #if OS(WINCE) should have been #if !OS(WINCE)

(WebCore::ScrollbarThemeWin::paintTrackPiece):
(WebCore::ScrollbarThemeWin::paintButton):
(WebCore::ScrollbarThemeWin::paintThumb):

  • rendering/RenderThemeWin.cpp: ditto.

(WebCore::drawControl):

5:42 PM Changeset in webkit [90074] by abarth@webkit.org
  • 4 edits in trunk/Tools

2011-06-29 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

new-run-webkit-tests doesn't support sample-on-timeout
​https://bugs.webkit.org/show_bug.cgi?id=56731

  • Scripts/webkitpy/layout_tests/port/server_process.py:
  • Scripts/webkitpy/layout_tests/port/server_process_unittest.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
5:30 PM Changeset in webkit [90073] by justin.garcia@apple.com
  • 2 edits in trunk/Source/WebCore

2011-06-29 Justin Garcia <​justin.garcia@apple.com>

Reviewed by Enrica Casucci.


I forgot to include these (reviewed) style fix-ups in my previous check-in.

  • editing/htmlediting.cpp: (WebCore::indexForVisiblePosition): (WebCore::visiblePositionForIndex):
5:10 PM Changeset in webkit [90072] by justin.garcia@apple.com
  • 6 edits in trunk/Source/WebCore

2011-06-29 Justin Garcia <​justin.garcia@apple.com>

Reviewed by Enrica Casucci.

​https://bugs.webkit.org/show_bug.cgi?id=62922
indexForVisiblePosition(const VisiblePosition& visiblePosition) does not consider shadow content


VisiblePositions can be inside web form text regions, which use shadow trees. Made indexForVisiblePosition
aware of this, and added a new parameter to obtain the scope for a VisiblePosition, in addition to its index.


Added visiblePositionForIndex to go in the opposite direction, taking into account the scope
used to compute the index.


These two functions use TextIterators to convert between VisiblePositions and indices. But
TextIterator iteration using TextIteratorEmitsCharactersBetweenAllVisiblePositions does not
exactly match VisiblePosition iteration, so using them to preserve a selection during an
editing operation is unreliable. This can be seen in the expected results for:


editing/execCommand/indent-pre-list.html
editing/execCommand/crash-indenting-list-item.html


TextIterator's TextIteratorEmitsCharactersBetweenAllVisiblePositions mode needs to be fixed, or
these functions need to be changed to iterate using actual VisiblePositions. See:


​https://bugs.webkit.org/show_bug.cgi?id=63590
TextIterators in TextIteratorEmitsCharactersBetweenAllVisiblePositions do not exactly match VisiblePositions


Also:


​https://bugs.webkit.org/show_bug.cgi?id=63592
Use visiblePositionForIndex and indexForVisiblePosition everywhere that TextIterators are used to convert between VisiblePositions and indices


No new tests added because indexForVisiblePosition is currently only used for editing operations
that cannot be performed inside web form fields.

  • editing/ApplyBlockElementCommand.cpp: (WebCore::ApplyBlockElementCommand::doApply):
  • editing/InsertListCommand.cpp: (WebCore::InsertListCommand::doApply):
  • editing/htmlediting.cpp: (WebCore::indexForVisiblePosition): (WebCore::visiblePositionForIndex):
  • editing/htmlediting.h:
5:03 PM Changeset in webkit [90071] by Dimitri Glazkov
  • 11 edits
    2 deletes in trunk/Source/WebCore

2011-06-29 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Kent Tamura.

Remove ElementWithPseudoId, since now an Element can store it all by itself.
​https://bugs.webkit.org/show_bug.cgi?id=63657

Refactoring, covered by existing tests.

  • CMakeLists.txt: Removed ElementWithPseudoId from build.
  • GNUmakefile.list.am: Ditto.
  • WebCore.gypi: Ditto.
  • WebCore.order: Ditto.
  • WebCore.pro: Ditto.
  • WebCore.vcproj/WebCore.vcproj: Ditto.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • html/ColorInputType.cpp: (WebCore::ColorInputType::createShadowSubtree): Changed to use plain-old HTMLDivElement.
  • html/RangeInputType.cpp: (WebCore::RangeInputType::createShadowSubtree): Ditto.
  • html/ValidationMessage.cpp: (WebCore::ValidationMessage::buildBubbleTree): Ditto.
  • html/shadow/ElementWithPseudoId.cpp: Removed.
  • html/shadow/ElementWithPseudoId.h: Removed.
5:02 PM Changeset in webkit [90070] by eric@webkit.org
  • 14 edits in trunk/Tools

2011-06-29 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Simplify skipped file finding in preparation for adding wk2 skipped list fallback
​https://bugs.webkit.org/show_bug.cgi?id=63501

The goal was to add support for wk2 skipped lists.
However, I found that our skipped list computation was a
manual hack (which only worked for the mac port).

So I fixed a FIXME to move the skipped list finding
into WebKitPort instead of MacPort.
Doing so required the concept of a "port_name", since previously
the only name accessible from a port object was name()
which includes many things beyond "mac" or "chromium", etc.

Eventually I believe we'll want to re-think the way that we pass
in a port_name argument to Port subclasses and expect them to parse
it. But for now I just added a cls.port_name variable which contains
the static information needed to compute wk2 names as well as
compute Skipped list fallback which works for Mac/Win/Qt and Gtk.

In order to test my new _skipped_file_search_paths method, I
fixed another FIXME by making it return relative paths.

I also fixed the test_expectations_path code in WebKitPort to use port_name.
It was using name() which would return PORT-VERSION so MacPort was overriding
it to use just PORT. After fixing test_expectations_path to use port_name
(and making it aware of webkit2) I was able to remove the MacPort implementation.

  • Scripts/webkitpy/layout_tests/port/base.py:
    • Add port_name() to access "mac", since name() returns "mac-leopard" etc.
    • Document that real_name() seems to have no purpose.
  • Scripts/webkitpy/layout_tests/port/chromium_linux.py:
    • Add _parse_port_name(), eventually we might call this from WebKitPort directly.
  • Scripts/webkitpy/layout_tests/port/chromium_mac.py:
    • Add _parse_port_name.
  • Scripts/webkitpy/layout_tests/port/chromium_win.py:
    • Add _parse_port_name.
  • Scripts/webkitpy/layout_tests/port/gtk.py:
  • Scripts/webkitpy/layout_tests/port/mac.py:
    • Move Skipped-file finding code down to WebKitPort
  • Scripts/webkitpy/layout_tests/port/mac_unittest.py:
  • Scripts/webkitpy/layout_tests/port/qt.py:
  • Scripts/webkitpy/layout_tests/port/webkit.py:
  • Scripts/webkitpy/layout_tests/port/webkit_unittest.py:
4:59 PM Changeset in webkit [90069] by leviw@chromium.org
  • 24 edits in trunk/Source/WebCore

2011-06-29 Levi Weintraub <​leviw@chromium.org>

Reviewed by Eric Seidel.

Switch computeWidth and computeHeight functions to new layout types
​https://bugs.webkit.org/show_bug.cgi?id=63655

Switch computeWidth and computeHeight functions over to the new layout unit abstraction.

Making a point to mark rounding that should eventually be removed by linking to a new
tracking bug: 63645.

No new tests, no functionality changes.

  • rendering/AutoTableLayout.cpp: (WebCore::AutoTableLayout::computePreferredLogicalWidths):
  • rendering/AutoTableLayout.h:
  • rendering/FixedTableLayout.cpp: (WebCore::FixedTableLayout::computePreferredLogicalWidths):
  • rendering/FixedTableLayout.h:
  • rendering/LayoutTypes.h:
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::computePreferredLogicalWidths): (WebCore::RenderBlock::computeInlinePreferredLogicalWidths): (WebCore::RenderBlock::computeBlockPreferredLogicalWidths):
  • rendering/RenderBox.cpp: (WebCore::RenderBox::computeLogicalWidthUsing): (WebCore::RenderBox::computeLogicalHeight): (WebCore::RenderBox::computeLogicalHeightUsing): (WebCore::RenderBox::computePercentageLogicalHeight): (WebCore::RenderBox::computeReplacedLogicalWidth): (WebCore::RenderBox::computeReplacedLogicalWidthRespectingMinMaxWidth): (WebCore::RenderBox::computeReplacedLogicalWidthUsing): (WebCore::RenderBox::computeReplacedLogicalHeight): (WebCore::RenderBox::computeReplacedLogicalHeightRespectingMinMaxHeight): (WebCore::RenderBox::computeReplacedLogicalHeightUsing):
  • rendering/RenderBox.h:
  • rendering/RenderIFrame.cpp: (WebCore::RenderIFrame::computeLogicalWidth):
  • rendering/RenderImage.cpp: (WebCore::RenderImage::computeReplacedLogicalWidth): (WebCore::RenderImage::computeReplacedLogicalHeight):
  • rendering/RenderImage.h:
  • rendering/RenderListBox.cpp: (WebCore::RenderListBox::computePreferredLogicalWidths):
  • rendering/RenderListMarker.cpp: (WebCore::RenderListMarker::computePreferredLogicalWidths):
  • rendering/RenderMenuList.cpp: (WebCore::RenderMenuList::computePreferredLogicalWidths):
  • rendering/RenderReplaced.cpp: (WebCore::RenderReplaced::computeReplacedLogicalWidth): (WebCore::RenderReplaced::computeReplacedLogicalHeight): (WebCore::RenderReplaced::computePreferredLogicalWidths):
  • rendering/RenderReplaced.h:
  • rendering/RenderSlider.cpp: (WebCore::RenderSlider::computePreferredLogicalWidths):
  • rendering/RenderTable.cpp: (WebCore::RenderTable::computeLogicalWidth):
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::computePreferredLogicalWidths):
  • rendering/RenderVideo.cpp: (WebCore::RenderVideo::computeReplacedLogicalWidth): (WebCore::RenderVideo::computeReplacedLogicalHeight):
  • rendering/RenderVideo.h:
  • rendering/svg/RenderSVGForeignObject.cpp: (WebCore::RenderSVGForeignObject::computeLogicalWidth): (WebCore::RenderSVGForeignObject::computeLogicalHeight):
  • rendering/svg/RenderSVGRoot.cpp: (WebCore::RenderSVGRoot::computePreferredLogicalWidths): (WebCore::RenderSVGRoot::computeIntrinsicWidth): (WebCore::RenderSVGRoot::computeIntrinsicHeight): (WebCore::RenderSVGRoot::computeReplacedLogicalWidth): (WebCore::RenderSVGRoot::computeReplacedLogicalHeight): (WebCore::RenderSVGRoot::layout):
  • rendering/svg/RenderSVGRoot.h:
4:58 PM Changeset in webkit [90068] by inferno@chromium.org
  • 6 edits
    4 adds in trunk

2011-06-23 Abhishek Arya <​inferno@chromium.org>

Reviewed by James Robinson.

In RenderBlock, RenderWidget and RenderReplaced destroy functions,
call dirtyLinesFromChangedChild to tell our parent that we are going away.
​https://bugs.webkit.org/show_bug.cgi?id=60307

Test: fast/block/child-not-removed-from-parent-lineboxes-crash.html

fast/block/block-not-removed-from-parent-lineboxes-crash.html

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::destroy):
  • rendering/RenderReplaced.cpp: (WebCore::RenderReplaced::destroy):
  • rendering/RenderReplaced.h:
  • rendering/RenderWidget.cpp: (WebCore::RenderWidget::destroy):

2011-06-23 Abhishek Arya <​inferno@chromium.org>

Reviewed by James Robinson.

Tests that we do not crash when failing to remove the linebreak
object(widget, block, replaced, etc) from parent block lineboxes. The
testcase tries to fuzz over the list of html and svg tags.
​https://bugs.webkit.org/show_bug.cgi?id=60307

  • fast/block/block-not-removed-from-parent-lineboxes-crash-expected.txt: Added.
  • fast/block/block-not-removed-from-parent-lineboxes-crash.html: Added.
  • fast/block/child-not-removed-from-parent-lineboxes-crash-expected.txt: Added.
  • fast/block/child-not-removed-from-parent-lineboxes-crash.html: Added.
4:54 PM Changeset in webkit [90067] by commit-queue@webkit.org
  • 6 edits
    2 adds in trunk/LayoutTests

2011-06-29 Alan Stearns <​stearns@adobe.com>

Reviewed by Tony Chang.

Modify Skipped files so that future css regions layout tests are
skipped by default.

[CSS Regions] Add support for skipped tests
​https://bugs.webkit.org/show_bug.cgi?id=63632

  • fast/regions/flow-content-basic.html: Added.
  • platform/chromium/test_expectations.txt:
  • platform/gtk/Skipped:
  • platform/mac/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:
4:52 PM Changeset in webkit [90066] by bfulgham@webkit.org
  • 2 edits in trunk/Source/WebCore

Speculative WinCE build correction.

  • rendering/RenderThemeWin.cpp:

(WebCore::drawControl): Don't call setRGBABitmapAlpha

for WinCE case.

4:50 PM Changeset in webkit [90065] by barraclough@apple.com
  • 3 edits in trunk/Source/JavaScriptCore

​https://bugs.webkit.org/show_bug.cgi?id=63669
DFG JIT - fix spectral-norm regression

Reviewed by Geoff Garen.

The problem is a mis-speculation leading to us falling off the speculative path.
Make the speculation logic slightly smarter, don't predict int if one of the
operands is already loaded as a double (we use this logic already for compares).

  • dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compile):

  • dfg/DFGSpeculativeJIT.h:

(JSC::DFG::SpeculativeJIT::shouldSpeculateInteger):

4:45 PM Changeset in webkit [90064] by rniwa@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-06-29 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Darin Adler.

REGRESSION(r89952): r89952 added a wrong assertion in visible_units.cpp
​https://bugs.webkit.org/show_bug.cgi?id=63668

We need to assert the offset to be zero only when the position is NOT offset in node.

  • editing/visible_units.cpp: (WebCore::startOfParagraph):
4:45 PM Changeset in webkit [90063] by commit-queue@webkit.org
  • 4 edits in trunk/Source/JavaScriptCore

2011-06-29 Filip Pizlo <​fpizlo@apple.com>

Reviewed by Gavin Barraclough.

DFG JIT does not do put_by_id transition caching.
​https://bugs.webkit.org/show_bug.cgi?id=63662

  • dfg/DFGJITCodeGenerator.cpp: (JSC::DFG::JITCodeGenerator::cachedPutById):
  • dfg/DFGJITCompiler.h: (JSC::DFG::JITCompiler::addPropertyAccess):
  • dfg/DFGRepatch.cpp: (JSC::DFG::testPrototype): (JSC::DFG::tryCachePutByID):
4:42 PM Changeset in webkit [90062] by bfulgham@webkit.org
  • 3 edits in trunk/Source/WebCore

Speculative WinCE build correction after r90057.

  • CMakeListsWinCE.txt: Roll back r90061, since GetWorldTransform is not available in WinCE.
  • platform/win/ScrollbarThemeWin.cpp: Conditionally exclude calls to DIBPixelData::setRGBABitmapAlpha for WinCE.

(WebCore::ScrollbarThemeWin::paintTrackPiece):
(WebCore::ScrollbarThemeWin::paintButton):
(WebCore::ScrollbarThemeWin::paintThumb):

4:27 PM Changeset in webkit [90061] by bfulgham@webkit.org
  • 2 edits in trunk/Source/WebCore

Speculative WinCE build correction after r90057.

  • CMakeListsWinCE.txt: Add missing DIBPixelData.cpp to

WinCE build to support setting bitmap image alpha value.

4:11 PM Changeset in webkit [90060] by ggaren@apple.com
  • 4 edits in trunk/Source/JavaScriptCore

2011-06-29 Geoffrey Garen <​ggaren@apple.com>

Reviewed by Oliver Hunt.

Added a dummy write barrier emitting function in all the right places in the old JIT
​https://bugs.webkit.org/show_bug.cgi?id=63667


SunSpider reports no change.

  • jit/JIT.h:
  • jit/JITPropertyAccess.cpp: (JSC::JIT::emit_op_put_by_id): (JSC::JIT::emit_op_put_scoped_var): Do it.

(JSC::JIT::emit_op_put_global_var): Global object needs to be in a register
for the sake of the write barrier.

(JSC::JIT::emitWriteBarrier): Empty for now. Not for long!

  • jit/JITPropertyAccess32_64.cpp: (JSC::JIT::emit_op_put_by_val): (JSC::JIT::emit_op_put_by_id): (JSC::JIT::emit_op_put_scoped_var): Do it.

(JSC::JIT::emit_op_put_global_var): Global object needs to be in a register
for the sake of the write barrier.

(JSC::JIT::emitWriteBarrier): Empty for now. Not for long!

3:56 PM Changeset in webkit [90059] by Simon Fraser
  • 6 edits
    5 adds in trunk

2011-06-29 Simon Fraser <Simon Fraser>

Reviewed by Beth Dakin.

Lay some groundwork for rubber-banding in overflow:scroll divs
​https://bugs.webkit.org/show_bug.cgi?id=63654

Make it possible for scrollTop and scrollLeft on a RenderLayer to be
set to negative values, which we will do during rubber-banding in
future. Continue to clamp programmatic scrolls, and scroll positions
set by layout between 0 and the max value.

Test: fast/layers/negative-scroll-positions.html

  • rendering/RenderBox.cpp: (WebCore::RenderBox::setScrollLeft): (WebCore::RenderBox::setScrollTop):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::scrollToOffset): (WebCore::RenderLayer::scrollTo):
  • rendering/RenderLayer.h: (WebCore::RenderLayer::scrollToXOffset): (WebCore::RenderLayer::scrollToYOffset):
  • rendering/RenderTextControlSingleLine.cpp: (WebCore::RenderTextControlSingleLine::forwardEvent):
3:53 PM Changeset in webkit [90058] by andersca@apple.com
  • 3 edits in trunk/Source/WebCore

2011-06-29 Anders Carlsson <​andersca@apple.com>

Reviewed by Simon Fraser.

RenderLayer::maximumScrollPosition() is wrong
​https://bugs.webkit.org/show_bug.cgi?id=62667

  • platform/ScrollableArea.h: (WebCore::ScrollableArea::isVerticalScrollerPinnedToMinimumPosition): (WebCore::ScrollableArea::isVerticalScrollerPinnedToMaximumPosition): Add these for good measure; they will be needed in an upcoming patch.
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::maximumScrollPosition): Subtract the visible content rect, including the scroll bars.
3:52 PM Changeset in webkit [90057] by bfulgham@webkit.org
  • 2 edits in trunk/Source/WebCore

Speculative WinCE build correction after r90053.

  • platform/graphics/GraphicsContext.h:

(WebCore::GraphicsContext::inTransparencyLayer): Provide
stub implementation for 'inTransparencyLayer'.

3:42 PM Changeset in webkit [90056] by jberlin@webkit.org
  • 2 edits in trunk/LayoutTests

eventSender.keyDown is unimplemented (WKTR)
​https://bugs.webkit.org/show_bug.cgi?id=57515

Add another test that uses eventSender.keyDown to the mac-wk2 Skipped list.

  • platform/mac-wk2/Skipped:
3:25 PM Changeset in webkit [90055] by abarth@webkit.org
  • 8 edits in trunk/Tools

2011-06-29 Adam Barth <​abarth@webkit.org>

Reviewed by Dirk Pranke.

new-run-webkit-tests complains about missing pixel results instead of plopping down new expectations
​https://bugs.webkit.org/show_bug.cgi?id=38063

This patch changes new-run-webkit-tests to match old-run-webkit-tests
in generating new expected results when the expected results are
missing.

There are still a couple details that are different:

1) Image baselines aren't generated unless you run with -p.
2) Render tree dumps are places in the cross-platform directory instead
of the platform-specific directory.

I'm inclined to deal with both of these issues in follow-up patches.

  • Scripts/webkitpy/common/net/layouttestresults.py:
  • Scripts/webkitpy/layout_tests/layout_package/manager.py:
  • Scripts/webkitpy/layout_tests/layout_package/single_test_runner.py:
  • Scripts/webkitpy/layout_tests/layout_package/test_results.py:
  • Scripts/webkitpy/layout_tests/port/test.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py:
3:24 PM Changeset in webkit [90054] by Adam Roben
  • 5 edits
    1 add in trunk/Tools

Teach TestFailures to detect possibly flaky tests and list them separately

Fixes <​http://webkit.org/b/61061> <rdar://problem/9452796> TestFailures page blames
arbitrary revisions for breaking flaky tests

Reviewed by Dan Bates.

  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/FlakyLayoutTestDetector.js: Added.

(FlakyLayoutTestDetector): This class identifies flaky tests when given the test results
from various builds (in reverse-chronological order).
(FlakyLayoutTestDetector.prototype.incorporateTestResults): Detects flaky tests. Tests move
monotonically through three states: LastSeenFailing, LastSeenPassing, and PossiblyFlaky.
(FlakyLayoutTestDetector.prototype.flakinessExamples): Finds examples of flakiness for the
given test. Essentially, finds all the transitions from passing to failing (or vice-versa)
and puts them in an array in reverse-chronological order.
(FlakyLayoutTestDetector.prototype.get possiblyFlakyTests): Returns all tests we've detected
to be possibly flaky.

  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/LayoutTestHistoryAnalyzer.js:

(LayoutTestHistoryAnalyzer): Initialize new members.
(LayoutTestHistoryAnalyzer.prototype.start): Now passes the callback an object with two
properties: history and possiblyFlaky. history holds the data this function used to pass to
the callback, while possiblyFlaky lists all tests that might be flaky and examples of their
flakiness. Updated documentation comment to match.
(LayoutTestHistoryAnalyzer.prototype._incorporateBuildHistory): Now uses a
FlakyLayoutTestDetector to identify possibly flaky tests. Any possibly flaky tests are
removed from the failure history, since when they started failing is no longer meaningful.
We tell our caller to keep calling until all current failures have been explained and we've
gone through 5 builds without any new flaky tests being identified.

  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/Utilities.js:

(sorted): New helper function to return a sorted copy of an array.
(Array.prototype.findLast): New helper function. Like findFirst, but finds the last item
that matches the predicate.

  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ViewController.js:

(ViewController.prototype._displayBuilder): Updated for change in the object passed to us by
the analyzer. Now puts the list of possibly flaky tests after the failure history.
(ViewController.prototype._domForFailedTest): Moved some code from here...
(ViewController.prototype._domForFailureDiagnosis): ...to here.
(ViewController.prototype._domForPossiblyFlakyTests): New function, builds up a list of
possibly flaky tests and examples of their flakiness and returns it.

  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/index.html: Pull in

FlakyLayoutTestDetector.js.

3:11 PM Changeset in webkit [90053] by bfulgham@webkit.org
  • 5 edits in trunk/Source/WebCore

2011-06-29 Xu Fan <​qifuren1985@gmail.com>

Reviewed by Brent Fulgham.

Fix Scrollbars Transparent in Windows XP if WebKit is using Layered Window
​https://bugs.webkit.org/show_bug.cgi?id=61136

  • platform/graphics/win/DIBPixelData.h: add a static method setRGBABitmapAlpha
  • platform/graphics/win/DIBPixelData.cpp: add implementation
  • platform/win/ScrollbarThemeWin.cpp: fix scroolbar transparent
  • rendering/RenderThemeWin.cpp: fix textbox and button transparent
3:04 PM Changeset in webkit [90052] by eric@webkit.org
  • 3 edits in trunk/Tools

2011-06-29 Eric Seidel <​eric@webkit.org>

Adam says cowboys don't review (or unit test).

  • Scripts/webkitpy/tool/commands/download.py:
2:56 PM Changeset in webkit [90051] by eric@webkit.org
  • 2 edits in trunk/Tools

2011-06-29 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Make port/config.py mockable for easier unit testing
​https://bugs.webkit.org/show_bug.cgi?id=63661

Fix an exception from the unit tests.

  • Scripts/webkitpy/layout_tests/port/config.py:
2:48 PM Changeset in webkit [90050] by bfulgham@webkit.org
  • 6 edits in trunk/Tools

2011-06-29 Brent Fulgham <​bfulgham@webkit.org>

Unreviewed build correction.

  • DumpRenderTree/win/ImageDiffWinCairoCommon.vsprops: Link against zlib1.dll, not the static zlib.lib.
2:41 PM Changeset in webkit [90049] by eae@chromium.org
  • 10 edits in trunk/Source/WebCore

2011-06-29 Emil A Eklund <​eae@chromium.org>

Reviewed by Eric Seidel.

Switch RenderBoxModelObject::margin* and setMargin* to to new layout types
​https://bugs.webkit.org/show_bug.cgi?id=63652

Switch margin methods for render tree over to the new layout unit abstraction.

No new tests, no functionality changes.

  • rendering/InlineFlowBox.h:
  • rendering/RenderBlock.cpp:
  • rendering/RenderBlock.h:
  • rendering/RenderBox.cpp:
  • rendering/RenderBox.h:
  • rendering/RenderBoxModelObject.h:
  • rendering/RenderInline.cpp:
  • rendering/RenderInline.h:
  • rendering/RenderText.h:
2:33 PM Changeset in webkit [90048] by leviw@chromium.org
  • 52 edits in trunk/Source/WebCore

2011-06-29 Levi Weintraub <​leviw@chromium.org>

Reviewed by Eric Seidel.

Switch paint to new layout types
​https://bugs.webkit.org/show_bug.cgi?id=63645

Switch vanilla paint function over to the new layout unit abstraction.

No new tests, no functionality changes.

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::paint):
  • rendering/RenderBlock.h:
  • rendering/RenderBox.cpp: (WebCore::RenderBox::paint):
  • rendering/RenderBox.h:
  • rendering/RenderDetailsMarker.cpp: (WebCore::RenderDetailsMarker::paint):
  • rendering/RenderDetailsMarker.h:
  • rendering/RenderEmbeddedObject.cpp: (WebCore::RenderEmbeddedObject::paint):
  • rendering/RenderEmbeddedObject.h:
  • rendering/RenderFrameSet.cpp: (WebCore::RenderFrameSet::paint):
  • rendering/RenderFrameSet.h:
  • rendering/RenderImage.cpp: (WebCore::RenderImage::paint):
  • rendering/RenderImage.h:
  • rendering/RenderInline.cpp: (WebCore::RenderInline::paint):
  • rendering/RenderInline.h:
  • rendering/RenderLineBoxList.h:
  • rendering/RenderListItem.cpp: (WebCore::RenderListItem::paint):
  • rendering/RenderListItem.h:
  • rendering/RenderListMarker.cpp: (WebCore::RenderListMarker::paint):
  • rendering/RenderListMarker.h:
  • rendering/RenderObject.h:
  • rendering/RenderReplaced.cpp: (WebCore::RenderReplaced::paint):
  • rendering/RenderReplaced.h:
  • rendering/RenderReplica.cpp: (WebCore::RenderReplica::paint):
  • rendering/RenderReplica.h:
  • rendering/RenderTable.cpp: (WebCore::RenderTable::paint):
  • rendering/RenderTable.h:
  • rendering/RenderTableCell.cpp: (WebCore::RenderTableCell::paint):
  • rendering/RenderTableCell.h:
  • rendering/RenderTableRow.cpp: (WebCore::RenderTableRow::paint):
  • rendering/RenderTableRow.h:
  • rendering/RenderTableSection.cpp: (WebCore::RenderTableSection::paint):
  • rendering/RenderTableSection.h:
  • rendering/RenderTextControlSingleLine.cpp: (WebCore::RenderTextControlSingleLine::paint):
  • rendering/RenderTextControlSingleLine.h:
  • rendering/RenderTheme.h:
  • rendering/RenderView.cpp: (WebCore::RenderView::paint):
  • rendering/RenderView.h:
  • rendering/RenderWidget.cpp: (WebCore::RenderWidget::paint):
  • rendering/RenderWidget.h:
  • rendering/mathml/RenderMathMLBlock.h:
  • rendering/mathml/RenderMathMLFraction.h:
  • rendering/mathml/RenderMathMLRoot.h:
  • rendering/mathml/RenderMathMLSquareRoot.h:
  • rendering/svg/RenderSVGContainer.h:
  • rendering/svg/RenderSVGForeignObject.h:
  • rendering/svg/RenderSVGHiddenContainer.h:
  • rendering/svg/RenderSVGImage.h:
  • rendering/svg/RenderSVGPath.h:
  • rendering/svg/RenderSVGRoot.cpp: (WebCore::RenderSVGRoot::paint):
  • rendering/svg/RenderSVGRoot.h:
  • rendering/svg/RenderSVGText.h:
2:29 PM Changeset in webkit [90047] by bfulgham@webkit.org
  • 2 edits in trunk/Tools

Unreviewed build correction.

  • DumpRenderTree/win/ImageDiffWinCairoCommon.vsprops:

Link against zlib1.dll, not the static zlib.lib.

2:28 PM Changeset in webkit [90046] by jianli@chromium.org
  • 5 edits
    3 deletes in trunk

WebGL test failing on SL bots: fast/canvas/webgl/data-view-test.html
​https://bugs.webkit.org/show_bug.cgi?id=63151

Reviewed by Kenneth Russell.

Source/WebCore:

The fix is to change to throw TypeError in custom JS/V8 bindings.

  • bindings/js/JSDataViewCustom.cpp:

(WebCore::getDataViewMember):
(WebCore::setDataViewMember):

  • bindings/v8/custom/V8DataViewCustom.cpp:

(WebCore::V8DataView::constructorCallback):
(WebCore::V8DataView::getInt8Callback):

LayoutTests:

Updated the expected result and removed the rebaselined files that are not needed.

  • fast/canvas/webgl/data-view-test-expected.txt:
  • platform/chromium-mac/fast/canvas/webgl/data-view-test-expected.txt: Removed.
  • platform/chromium-win/fast/canvas/webgl/data-view-test-expected.txt: Removed.
  • platform/mac/fast/canvas/webgl/data-view-test-expected.txt: Removed.
2:25 PM Changeset in webkit [90045] by macpherson@chromium.org
  • 2 edits in trunk/Source/WebCore

2011-06-29 Luke Macpherson <​macpherson@chromium.org>

Reviewed by Darin Adler.

Convert uses of computeLengthIntForLength() to computeLength<Length>()
​https://bugs.webkit.org/show_bug.cgi?id=63641

No new tests / refactoring only.

  • css/CSSStyleSelector.cpp: (WebCore::convertToLength): (WebCore::CSSStyleSelector::applyProperty): (WebCore::CSSStyleSelector::applyPageSizeProperty): (WebCore::CSSStyleSelector::mmLength): (WebCore::CSSStyleSelector::inchLength): (WebCore::CSSStyleSelector::mapFillSize): (WebCore::CSSStyleSelector::mapFillXPosition): (WebCore::CSSStyleSelector::mapFillYPosition):
2:25 PM Changeset in webkit [90044] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

2011-06-29 Jeff Timanus <​twiz@chromium.org>

Reviewed by Stephen White.

[Chromium] Remove flakiness marker from fast/canvas/canvas-longlived-context.html
​https://bugs.webkit.org/show_bug.cgi?id=63649

  • platform/chromium/test_expectations.txt:
2:22 PM Changeset in webkit [90043] by eric@webkit.org
  • 2 edits in trunk/Tools

2011-06-29 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Remove duplicate methods in filesystem.py
​https://bugs.webkit.org/show_bug.cgi?id=63658

Looks like there was a bad merge at some point.

I also removed a bunch of redundant docstrings.

  • Scripts/webkitpy/common/system/filesystem.py:
2:20 PM Changeset in webkit [90042] by crogers@google.com
  • 3 edits in trunk/Source/WebCore

2011-06-29 Chris Rogers <​crogers@google.com>

Reviewed by Kenneth Russell.

Simplify AudioBufferSourceNode rendering
​https://bugs.webkit.org/show_bug.cgi?id=63586

No new tests since this doesn't change API.

  • webaudio/AudioBufferSourceNode.cpp: (WebCore::AudioBufferSourceNode::AudioBufferSourceNode): (WebCore::AudioBufferSourceNode::process): (WebCore::AudioBufferSourceNode::renderFromBuffer): (WebCore::AudioBufferSourceNode::reset): (WebCore::AudioBufferSourceNode::setBuffer): (WebCore::AudioBufferSourceNode::noteOn): (WebCore::AudioBufferSourceNode::noteGrainOn): (WebCore::AudioBufferSourceNode::totalPitchRate):
  • webaudio/AudioBufferSourceNode.h:
2:03 PM Changeset in webkit [90041] by commit-queue@webkit.org
  • 6 edits in trunk/Source/WebCore

2011-06-29 Martin Robinson <​mrobinson@igalia.com>

Reviewed by Xan Lopez.

[Freetype] Many tests report ERROR: Failed to get glyph page zero.
​https://bugs.webkit.org/show_bug.cgi?id=63498

Instead of conditionally instantiating the m_scaledFont member of FontPlatformData,
always instantiate it. For times when the size is 0 and the instantiation would lead
to a Cairo error, we use size 1 and just make operations which would fail conditional
on the m_size > 0.

No new tests. This change is covered by existing tests. We should
observe no errors on the bots after landing.

  • platform/graphics/cairo/FontCairo.cpp: (WebCore::Font::drawGlyphs): Instead of looking at whether or not there is a scaled font to determine whether to draw, just look at the size.
  • platform/graphics/freetype/FontPlatformData.h: Remove the m_font member.
  • platform/graphics/freetype/FontPlatformDataFreeType.cpp: Always instantiate a scaled font member and remove references to m_font. If we are instantiating a scaled font for a 0 size font, instantiate a size 1 font, so that we can still use Cairo to query font properties.
  • platform/graphics/freetype/GlyphPageTreeNodeFreeType.cpp: (WebCore::GlyphPage::fill): We can assume there is always a scaled font now.
  • platform/graphics/freetype/SimpleFontDataFreeType.cpp: (WebCore::SimpleFontData::platformInit): Ditto. (WebCore::SimpleFontData::createScaledFontData): Ditto. (WebCore::SimpleFontData::containsCharacters): Ditto. (WebCore::SimpleFontData::platformWidthForGlyph): Ditto.
1:55 PM Changeset in webkit [90040] by abarth@webkit.org
  • 2 edits in trunk/Tools

2011-06-29 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Pass --no-new-test-results to NRWT on build.webkit.org
​https://bugs.webkit.org/show_bug.cgi?id=63653

This is in preparation for ​https://bugs.webkit.org/show_bug.cgi?id=38063.

  • BuildSlaveSupport/build.webkit.org-config/master.cfg:
1:52 PM Changeset in webkit [90039] by eric.carlson@apple.com
  • 5 edits
    4 adds in trunk

2011-06-29 Eric Carlson <​eric.carlson@apple.com>

Reviewed by Darin Adler.

Media element loads blocked by a resource load delegate do not generate an error event
​https://bugs.webkit.org/show_bug.cgi?id=63558

Tests: media/media-blocked-by-beforeload.html

media/media-blocked-by-willsendrequest.html

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::selectMediaResource): Call mediaLoadingFailed instead of noneSupported

so all failures follow the same code path.

(WebCore::HTMLMediaElement::loadResource): Call mediaLoadingFailed when there is no frame, when

the loader delegate rejects the url, or when MediaPlayer doesn't find a media engine that
supports the type.

(WebCore::HTMLMediaElement::selectNextSourceChild): Fix incorrect logging.
(WebCore::HTMLMediaElement::mediaLoadingFailed): Change name of parameter from "state" to

"error" to make its purpose obvious.

  • platform/graphics/MediaPlayer.cpp: (WebCore::MediaPlayer::load): Return a bool indicating whether or not the url was given to

a media engine.

(WebCore::MediaPlayer::loadWithNextMediaEngine): Add some logging.

  • platform/graphics/MediaPlayer.h:

2011-06-29 Eric Carlson <​eric.carlson@apple.com>

Reviewed by Darin Adler.

Media element loads blocked by a resource load delegate do not generate an error event
​https://bugs.webkit.org/show_bug.cgi?id=63558

  • media/media-blocked-by-beforeload-expected.txt: Added.
  • media/media-blocked-by-beforeload.html: Added.
  • media/media-blocked-by-willsendrequest-expected.txt: Added.
  • media/media-blocked-by-willsendrequest.html: Added.
1:46 PM Changeset in webkit [90038] by Darin Adler
  • 5 edits
    2 adds in trunk

2011-06-29 Darin Adler <Darin Adler>

Reviewed by Anders Carlsson.

[WebKit2] Crash loading page that adds/removes frame in DOMContentLoaded/loaded
​https://bugs.webkit.org/show_bug.cgi?id=63483

  • fast/loader/create-frame-in-DOMContentLoaded-expected.txt: Added.
  • fast/loader/create-frame-in-DOMContentLoaded.html: Added.

2011-06-29 Darin Adler <Darin Adler>

Reviewed by Anders Carlsson.

[WebKit2] Crash loading page that adds/removes frame in DOMContentLoaded/loaded
​https://bugs.webkit.org/show_bug.cgi?id=63483

Test: fast/loader/create-frame-in-DOMContentLoaded.html

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::init): Added an assertion. (WebCore::FrameLoader::finishedLoadingDocument): Removed a non-helpful #if statement. The rule that we do not call the client when creating the initial empty document was nominally specific to Windows and Chromium but is needed for all platforms.

2011-06-29 Darin Adler <Darin Adler>

Reviewed by Anders Carlsson.

[WebKit2] Crash loading page that adds/removes frame in DOMContentLoaded/loaded
​https://bugs.webkit.org/show_bug.cgi?id=63483

  • WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp: (WebKit::WebFrameLoaderClient::createFrame): Added a check for null when creating a frame. Also added a check that was in the WebKit1 code, but not here. Strangely, the comment from the second check was still here, but not the code.
1:14 PM Changeset in webkit [90037] by enne@google.com
  • 1 edit
    3 copies in branches/chromium/782

Merge 88698 - 2011-06-13 Adrienne Walker <​enne@google.com>

Reviewed by Simon Fraser.

Overlap test needs to consider children of composited layers
​https://bugs.webkit.org/show_bug.cgi?id=62465

  • compositing/layer-creation/overlap-child-layer-expected.png: Added.
  • compositing/layer-creation/overlap-child-layer-expected.txt: Added.
  • compositing/layer-creation/overlap-child-layer.html: Added.

2011-06-13 Adrienne Walker <​enne@google.com>

Reviewed by Simon Fraser.

Overlap test needs to consider children of composited layers
​https://bugs.webkit.org/show_bug.cgi?id=62465

Any child layer with a compositing ancestor will be put into a
composited layer even though they themselves don't need compositing.
As layer bounds aren't hierarchical, these layer bounds themselves
need to be put into the overlap map because the composited layer
itself is not sufficient.

Test: compositing/layer-creation/overlap-animated-layer.html

  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::computeCompositingRequirements):

TBR=​enne@google.com

12:47 PM Changeset in webkit [90036] by xan@webkit.org
  • 2 edits in trunk/Tools

2011-06-29 Xan Lopez <​xlopez@igalia.com>

Reviewed by Adam Barth.

[GTK] overlapping drag&drop tests fail on NRWT
​https://bugs.webkit.org/show_bug.cgi?id=57640

Make a GTK test driver that spawns one Xvfb instance per
thread. This avoids bad interactions in DnD tests between threads.

  • Scripts/webkitpy/layout_tests/port/gtk.py:
12:46 PM Changeset in webkit [90035] by commit-queue@webkit.org
  • 7 edits in trunk/Source/JavaScriptCore

2011-06-29 Filip Pizlo <​fpizlo@apple.com>

Reviewed by Gavin Barraclough.

DFG JIT does not perform get_by_id self list caching.
​https://bugs.webkit.org/show_bug.cgi?id=63605

  • bytecode/StructureStubInfo.h:
  • dfg/DFGJITCompiler.cpp: (JSC::DFG::JITCompiler::compileFunction):
  • dfg/DFGOperations.cpp:
  • dfg/DFGOperations.h:
  • dfg/DFGRepatch.cpp: (JSC::DFG::tryCacheGetByID): (JSC::DFG::tryBuildGetByIDList): (JSC::DFG::dfgBuildGetByIDList):
  • dfg/DFGRepatch.h:
12:46 PM QtWebKitRelease20 edited by Ademar Reis
(diff)
12:45 PM QtWebKitRelease21 edited by Ademar Reis
(diff)
12:20 PM Changeset in webkit [90034] by nduca@chromium.org
  • 2 edits in trunk/LayoutTests

2011-06-29 Nat Duca <​nduca@chromium.org>

[chromium] Unreviewed, mark fast/canvas/webgl/context-lost-restored failing
​https://bugs.webkit.org/show_bug.cgi?id=63643

  • platform/chromium/test_expectations.txt:
12:18 PM Changeset in webkit [90033] by alexis.menard@openbossa.org
  • 2 edits in trunk/Source/WebKit/qt

2011-06-29 Alexis Menard <​alexis.menard@openbossa.org>

Reviewed by Benjamin Poulain.

[Qt] Add Qt dependencies in QtWebKit's main pro file.
​https://bugs.webkit.org/show_bug.cgi?id=63639

syncqt, the script which generates the headers inside Qt parses
the main pro file of QtWebKit to check the Qt dependencies. It used
to be WebCore.pro but after the build reorganization QtWebKit.pro is
the new main pro file so we need to add the network dependency just like
we did in WebCore.pro.

12:07 PM Changeset in webkit [90032] by nduca@chromium.org
  • 2 edits in trunk/LayoutTests

2011-06-29 Nat Duca <​nduca@chromium.org>

[chromium] Unreviewed, mark index-validation-with-resized-buffer as failing.
​https://bugs.webkit.org/show_bug.cgi?id=63638

  • platform/chromium/test_expectations.txt:
11:58 AM Changeset in webkit [90031] by commit-queue@webkit.org
  • 2 edits
    1 move
    9 deletes in trunk/LayoutTests

2011-06-29 Wyatt Carss <​wcarss@chromium.org>

Reviewed by Ryosuke Niwa.

convert editing/deleting/paragraph-in-preserveNewline.html to dumpAsText
​https://bugs.webkit.org/show_bug.cgi?id=63362

Converted editing/deleting/paragraph-in-preserveNewline.html to dump-as-markup,
added doctype, added new baseline, and removed old baselines.

  • editing/deleting/paragraph-in-preserveNewline.html:
  • editing/deleting/paragraph-in-preserveNewline-expected.html: Added.
  • platform/chromium-linux/editing/deleting/paragraph-in-preserveNewline-expected.png: Removed.
  • platform/chromium-win/editing/deleting/paragraph-in-preserveNewline-expected.png: Removed.
  • platform/chromium-win/editing/deleting/paragraph-in-preserveNewline-expected.txt: Removed.
  • platform/gtk/editing/deleting/paragraph-in-preserveNewline-expected.png: Removed.
  • platform/mac-leopard/editing/deleting/paragraph-in-preserveNewline-expected.png: Removed.
  • platform/mac/editing/deleting/paragraph-in-preserveNewline-expected.png: Removed.
  • platform/mac/editing/deleting/paragraph-in-preserveNewline-expected.txt: Removed.
  • platform/qt/editing/deleting/paragraph-in-preserveNewline-expected.png: Removed.
  • platform/qt/editing/deleting/paragraph-in-preserveNewline-expected.txt: Removed.
11:58 AM Changeset in webkit [90030] by eae@chromium.org
  • 6 edits in trunk/Source/WebCore

2011-06-29 Emil A Eklund <​eae@chromium.org>

Reviewed by Darin Adler.

Switch hitTestColumns/Contents/Floats to to new layout types
​https://bugs.webkit.org/show_bug.cgi?id=63589

Switch location and size methods for RenderBox over to the new layout unit abstraction.

No new tests, no functionality changes.

  • rendering/ColumnInfo.h: (WebCore::ColumnInfo::desiredColumnWidth): (WebCore::ColumnInfo::setDesiredColumnWidth): (WebCore::ColumnInfo::columnHeight): (WebCore::ColumnInfo::setColumnCountAndHeight): (WebCore::ColumnInfo::setColumnHeight): (WebCore::ColumnInfo::updateMinimumColumnHeight): (WebCore::ColumnInfo::minimumColumnHeight):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::addOverflowFromChildren): (WebCore::RenderBlock::paintColumnRules): (WebCore::RenderBlock::paintColumnContents): (WebCore::RenderBlock::hitTestFloats): (WebCore::RenderBlock::hitTestColumns): (WebCore::RenderBlock::hitTestContents): (WebCore::RenderBlock::columnRectAt):
  • rendering/RenderBlock.h: (WebCore::RenderBlock::xPositionForFloatIncludingMargin): (WebCore::RenderBlock::yPositionForFloatIncludingMargin):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::hitTestContents):
  • rendering/RenderLayer.h:
11:30 AM Changeset in webkit [90029] by enne@google.com
  • 2 edits in trunk/Source/WebCore

2011-06-29 Adrienne Walker <​enne@google.com>

Reviewed by James Robinson.

[chromium] Divide by zero in TilingData
​https://bugs.webkit.org/show_bug.cgi?id=63305

Add safety checks for empty layers.

  • platform/graphics/chromium/LayerTilerChromium.cpp: (WebCore::LayerTilerChromium::prepareToUpdate): (WebCore::LayerTilerChromium::reset): (WebCore::LayerTilerChromium::updateRect):
11:21 AM Changeset in webkit [90028] by Nate Chapin
  • 3 edits in trunk/LayoutTests

2011-06-29 Nate Chapin <Nate Chapin>

Unreviewed, more chromium test updates.

  • platform/chromium-linux/svg/foreignObject/text-tref-02-b-expected.png:
  • platform/chromium-win/svg/foreignObject/text-tref-02-b-expected.png:
10:57 AM Changeset in webkit [90027] by commit-queue@webkit.org
  • 2 edits in trunk/Websites/webkit.org

2011-06-29 Leandro Pereira <​leandro@profusion.mobi>

Reviewed by Darin Adler.

Add note about cases where enums are preferred over bools on function parameters.
​https://bugs.webkit.org/show_bug.cgi?id=63564

This matches the outcome of this webkit-dev thread:
​https://lists.webkit.org/pipermail/webkit-dev/2010-December/015192.html

  • coding/coding-style.html:
10:46 AM Changeset in webkit [90026] by Dimitri Glazkov
  • 3 edits in trunk/Source/WebCore

2011-06-29 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Adam Barth.

Extract setShadowRoot function, harden it in preparation to be exposed to script.
​https://bugs.webkit.org/show_bug.cgi?id=63596

Refactoring, covered by existing tests.

  • dom/Element.cpp: (WebCore::validateShadowRoot): Added. (WebCore::Element::setShadowRoot): Extracted from ensureShadowRoot. (WebCore::Element::ensureShadowRoot): Changed to use setShadowRoot.
  • dom/Element.h: Added decl.
10:44 AM Changeset in webkit [90025] by jhawkins@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

2011-06-29 James Hawkins <​jhawkins@chromium.org>

Reviewed by Pavel Feldman.

[Chromium] Fix a crash on when executeCommand(copy) is called on a null node.
​https://bugs.webkit.org/show_bug.cgi?id=63626

  • src/WebFrameImpl.cpp: (WebKit::pluginContainerFromNode):
10:42 AM Changeset in webkit [90024] by commit-queue@webkit.org
  • 4 edits in trunk/Tools

2011-06-29 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r89888.
​http://trac.webkit.org/changeset/89888
​https://bugs.webkit.org/show_bug.cgi?id=63609

"Seems to be causing merging issues on the dashboard, e.g.

​http://test-
results.appspot.com/dashboards/flakiness_dashboard.html#group=%40ToT%20-%20chromium.org&tests=fast%2Fcanvas"
(Requested by aboxhall on #webkit).

  • 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:
10:34 AM Changeset in webkit [90023] by Nate Chapin
  • 3 edits
    3 adds in trunk/LayoutTests

2011-06-29 Nate Chapin <Nate Chapin>

Unreviewed, chromium test updates.

  • platform/chromium-mac-leopard/svg/foreignObject: Added.
  • platform/chromium-mac-leopard/svg/foreignObject/text-tref-02-b-expected.png: Added.
  • platform/chromium-mac-leopard/svg/foreignObject/text-tref-02-b-expected.txt: Added.
  • platform/chromium-mac/svg/foreignObject/text-tref-02-b-expected.png:
  • platform/chromium/test_expectations.txt:
9:22 AM Changeset in webkit [90022] by commit-queue@webkit.org
  • 3 edits
    3 adds in trunk

2011-06-29 Vsevolod Vlasov <​vsevik@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: REGRESSION Resource revision fails to request content if it was not yet loaded for resource.
​https://bugs.webkit.org/show_bug.cgi?id=63631

  • http/tests/inspector/network/network-request-revision-content-expected.txt: Added.
  • http/tests/inspector/network/network-request-revision-content.html: Added.
  • http/tests/inspector/network/resources/script.js: Added.

2011-06-29 Vsevolod Vlasov <​vsevik@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: REGRESSION Resource revision fails to request content if it was not yet loaded for resource.
​https://bugs.webkit.org/show_bug.cgi?id=63631

Test: http/tests/inspector/network/network-request-revision-content.html

  • inspector/front-end/Resource.js: (WebInspector.ResourceRevision.prototype.requestContent):
9:15 AM Changeset in webkit [90021] by kbalazs@webkit.org
  • 2 edits
    2 adds in trunk/LayoutTests

2011-06-29 Balazs Kelemen <​kbalazs@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt][WK2] plugins/document-open.html need new result for wk2
​https://bugs.webkit.org/show_bug.cgi?id=63633

  • platform/qt-wk2/Skipped:
  • platform/qt-wk2/plugins/document-open-expected.txt: Added.
8:56 AM Changeset in webkit [90020] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

2011-06-29 Igor Oliveira <​igor.oliveira@openbossa.org>

Reviewed by Andreas Kling.

[Qt] GraphicsContext::clipToImageBuffer does not support scale transformation
​https://bugs.webkit.org/show_bug.cgi?id=63555

Scale alphaMask if GraphicsContext is scaled

  • platform/graphics/qt/GraphicsContextQt.cpp: (WebCore::GraphicsContext::pushTransparencyLayerInternal):
  • platform/graphics/qt/ImageBufferQt.cpp: (WebCore::ImageBuffer::clip):
8:20 AM Changeset in webkit [90019] by loislo@chromium.org
  • 3 edits in branches/chromium/782/Source/WebCore/inspector/front-end

Merge 89666 - 2011-06-23 Mikhail Naganov <​mnaganov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: [Chromium] Shorten DOMWindow URLs in heap profiles.
​https://bugs.webkit.org/show_bug.cgi?id=63238

  • inspector/front-end/DetailedHeapshotGridNodes.js: (WebInspector.HeapSnapshotGridNode.prototype.hasHoverMessage.false.hoverMessage): (WebInspector.HeapSnapshotGenericObjectNode): (WebInspector.HeapSnapshotGenericObjectNode.prototype.hoverMessage): (WebInspector.HeapSnapshotGenericObjectNode.prototype._updateHasChildren): (WebInspector.HeapSnapshotGenericObjectNode.prototype.isDOMWindow): (WebInspector.HeapSnapshotGenericObjectNode.prototype.shortenWindowURL):
  • inspector/front-end/DetailedHeapshotView.js: (WebInspector.HeapSnapshotRetainingPathsList.prototype.showNext.pathFound): (WebInspector.DetailedHeapshotView.prototype._getHoverAnchor): (WebInspector.DetailedHeapshotView.prototype._showStringContentPopup.displayString): (WebInspector.DetailedHeapshotView.prototype._showStringContentPopup):
  • inspector/front-end/utilities.js: ():

TBR=​mnaganov@chromium.org
BUG=87584
Review URL: ​http://codereview.chromium.org/7218041

8:17 AM Changeset in webkit [90018] by loislo@chromium.org
  • 3 edits in branches/chromium/782

Merge 90013 - 2011-06-29 Pavel Feldman <​pfeldman@google.com>

Reviewed by Yury Semikhatsky.

Web Inspector: tab crash after deleting trailing quote when editing attribute
​https://bugs.webkit.org/show_bug.cgi?id=63544

  • inspector/elements/set-attribute-expected.txt:
  • inspector/elements/set-attribute.html:

2011-06-29 Pavel Feldman <​pfeldman@google.com>

Reviewed by Yury Semikhatsky.

Web Inspector: tab crash after deleting trailing quote when editing attribute
​https://bugs.webkit.org/show_bug.cgi?id=63544

  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::setAttributesText):
  • inspector/front-end/ElementsTreeOutline.js: (WebInspector.ElementsTreeElement.prototype._attributeEditingCommitted.moveToNextAttributeIfNeeded): (WebInspector.ElementsTreeElement.prototype._attributeEditingCommitted):

TBR=​pfeldman@chromium.org
BUG=87895
Review URL: ​http://codereview.chromium.org/7280007

7:38 AM Changeset in webkit [90017] by caseq@chromium.org
  • 9 edits in trunk/Source

2011-06-29 Andrey Kosyakov <​caseq@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: backend needs to provide system-unique object ids, so these remain unique across navigation
​https://bugs.webkit.org/show_bug.cgi?id=62894

  • inspector/InspectorController.cpp: (WebCore::InspectorController::setAgentProcessIdentifier):
  • inspector/InspectorController.h:
  • inspector/InspectorPageAgent.cpp: (WebCore::InspectorPageAgent::setAgentIdentifier): (WebCore::InspectorPageAgent::createIdentifier): (WebCore::InspectorPageAgent::frameForId): (WebCore::InspectorPageAgent::frameId): (WebCore::InspectorPageAgent::frameDestroyed):
  • inspector/InspectorPageAgent.h:

2011-06-29 Andrey Kosyakov <​caseq@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: backend needs to provide system-unique object ids, so these remain unique across navigation
​https://bugs.webkit.org/show_bug.cgi?id=62894

  • public/WebDevToolsAgent.h:
  • src/WebDevToolsAgentImpl.cpp: (WebKit::WebDevToolsAgentImpl::setAgentProcessIdentifier):
  • src/WebDevToolsAgentImpl.h:
7:18 AM Changeset in webkit [90016] by Philippe Normand
  • 2 edits in trunk/Tools

Reviewed by Martin Robinson.

[webkitpy] Executive.running_pids() sometimes doesn't correctly process "ps" output
​https://bugs.webkit.org/show_bug.cgi?id=63624

  • Scripts/webkitpy/common/system/executive.py: In some cases the ps output lines can contain one or more leading white-spaces, so strip it before splitting the pid and command name.
7:12 AM Changeset in webkit [90015] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebKit/efl

2011-06-29 Michal Pakula vel Rutka <Michał Pakuła vel Rutka>

Reviewed by Kenneth Rohde Christiansen.

[EFL] Add ewk_frame_text_selection_type_get function
​https://bugs.webkit.org/show_bug.cgi?id=63462

Add a method to obtain current WebCore::VisibleSelection text
selection type.

  • ewk/ewk_frame.cpp: (ewk_frame_text_selection_type_get):
  • ewk/ewk_frame.h:
6:58 AM Changeset in webkit [90014] by pfeldman@chromium.org
  • 6 edits in trunk/Source/WebCore

2011-06-29 Pavel Feldman <​pfeldman@google.com>

Reviewed by Yury Semikhatsky.

Web Inspector: create status bar buttons programmatically. Add "Inspect" button to all panels.
​https://bugs.webkit.org/show_bug.cgi?id=63450

Buttons should be created in code, not in HTML. Also making Inspect Element
action available on all panels (as an experiment).

  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleView.prototype.show): (WebInspector.ConsoleView.prototype.hide):
  • inspector/front-end/ElementsPanel.js: (WebInspector.ElementsPanel): (WebInspector.ElementsPanel.prototype.get statusBarItems): (WebInspector.ElementsPanel.prototype.updateFocusedNode): (WebInspector.ElementsPanel.prototype._setSearchingForNode): (WebInspector.ElementsPanel.prototype.toggleSearchingForNode):
  • inspector/front-end/inspector.css: (button.dock-status-bar-item.status-bar-item .glyph): (button.dock-status-bar-item.status-bar-item.toggled-on .glyph): (.console-status-bar-item .glyph):
  • inspector/front-end/inspector.html:
  • inspector/front-end/inspector.js: (WebInspector._createGlobalStatusBarItems): (WebInspector.set attached):
6:47 AM Changeset in webkit [90013] by pfeldman@chromium.org
  • 6 edits in trunk

2011-06-29 Pavel Feldman <​pfeldman@google.com>

Reviewed by Yury Semikhatsky.

Web Inspector: tab crash after deleting trailing quote when editing attribute
​https://bugs.webkit.org/show_bug.cgi?id=63544

  • inspector/elements/set-attribute-expected.txt:
  • inspector/elements/set-attribute.html:

2011-06-29 Pavel Feldman <​pfeldman@google.com>

Reviewed by Yury Semikhatsky.

Web Inspector: tab crash after deleting trailing quote when editing attribute
​https://bugs.webkit.org/show_bug.cgi?id=63544

  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::setAttributesText):
  • inspector/front-end/ElementsTreeOutline.js: (WebInspector.ElementsTreeElement.prototype._attributeEditingCommitted.moveToNextAttributeIfNeeded): (WebInspector.ElementsTreeElement.prototype._attributeEditingCommitted):
6:42 AM Changeset in webkit [90012] by Philippe Normand
  • 2 edits in trunk/LayoutTests

Unreviewed, remove erroneous test name from GTK Skipped list.

  • platform/gtk/Skipped:
6:13 AM Changeset in webkit [90011] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-06-29 Vsevolod Vlasov <​vsevik@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: REGRESSION Inspector failing to load resource content after navigation. Safari.
​https://bugs.webkit.org/show_bug.cgi?id=63480

  • inspector/front-end/NetworkManager.js: (WebInspector.NetworkDispatcher): (WebInspector.NetworkDispatcher.prototype.resourceLoadedFromMemoryCache):
5:43 AM Changeset in webkit [90010] by pfeldman@chromium.org
  • 9 edits
    2 adds in trunk

2011-06-29 Mike West <​mkwst@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: Missing fields in HAR
​https://bugs.webkit.org/show_bug.cgi?id=58127

Setting correct expectations for the following fields newly added to
the HAR implementation:

  • entry.request.httpVersion
  • entry.request.headersSize
  • entry.request.bodySize
  • entry.response.httpVersion
  • entry.response.headersSize
  • entry.response.bodySize
  • entry.response.content.compression
  • http/tests/inspector/network/network-size.html:
  • http/tests/inspector/resource-har-conversion-expected.txt:
  • http/tests/inspector/resource-har-conversion.html:
  • http/tests/inspector/resource-har-headers-expected.txt: Added.
  • http/tests/inspector/resource-har-headers.html: Added.
  • http/tests/inspector/resource-parameters-expected.txt:
  • http/tests/inspector/resources-test.js:

2011-06-29 Mike West <​mkwst@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: Missing fields in HAR
​https://bugs.webkit.org/show_bug.cgi?id=58127

Specifically, this patch adds:

  • entry.request.httpVersion
  • entry.request.headersSize
  • entry.request.bodySize
  • entry.response.httpVersion
  • entry.response.headersSize
  • entry.response.bodySize
  • entry.response.content.compression

Tests: http/tests/inspector/resource-har-conversion.html

http/tests/inspector/resource-har-headers.html
http/tests/inspector/resource-parameters.html
http/tests/inspector/network/network-size.html

4:44 AM Changeset in webkit [90009] by Philippe Normand
  • 2 edits in trunk/LayoutTests

Unreviewed, skip 2 acid tests on GTK due to bug 54119. They were
failing only in 32-Bits Release though.

  • platform/gtk/Skipped:
4:05 AM Changeset in webkit [90008] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt][Mac] Qt specific results committed in r90007, but fail on Qt-Mac platform

  • platform/qt-mac/Skipped: Skip failing tests.
3:44 AM Changeset in webkit [90007] by Csaba Osztrogonác
  • 3 edits
    10 adds in trunk/LayoutTests

[Qt] Add platform specific expected results for new passing tests.

  • platform/qt/svg/W3C-SVG-1.1-SE/text-tref-03-b-expected.png: Added.
  • platform/qt/svg/W3C-SVG-1.1-SE/text-tref-03-b-expected.txt: Added.
  • platform/qt/svg/custom/text-tref-03-b-change-href-dom-expected.png: Added.
  • platform/qt/svg/custom/text-tref-03-b-change-href-dom-expected.txt: Added.
  • platform/qt/svg/custom/text-tref-03-b-change-href-expected.png: Added.
  • platform/qt/svg/custom/text-tref-03-b-change-href-expected.txt: Added.
  • platform/qt/svg/custom/text-tref-03-b-referenced-element-removal-expected.png: Added.
  • platform/qt/svg/custom/text-tref-03-b-referenced-element-removal-expected.txt: Added.
  • platform/qt/svg/custom/text-tref-03-b-tref-removal-expected.png: Added.
  • platform/qt/svg/custom/text-tref-03-b-tref-removal-expected.txt: Added.
  • platform/qt/svg/foreignObject/text-tref-02-b-expected.png:
  • platform/qt/svg/foreignObject/text-tref-02-b-expected.txt:
3:24 AM Changeset in webkit [90006] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

2011-06-29 Noel Gordon <​noel.gordon@gmail.com>

Reviewed by Kent Tamura.

[Chromium] [DRT] ImageDiff should compile in VS2010
​https://bugs.webkit.org/show_bug.cgi?id=63614

  • DumpRenderTree/chromium/ImageDiff.cpp: Include <iterator> for back_inserter.
2:58 AM Changeset in webkit [90005] by Philippe Normand
  • 3 edits in trunk/LayoutTests

Unreviewed, GTK rebaseline of 2 more tests.

  • inspector/console/console-trace-in-eval-expected.txt:
  • platform/gtk/svg/foreignObject/text-tref-02-b-expected.txt:
2:39 AM Changeset in webkit [90004] by hayato@chromium.org
  • 5 edits
    2 adds in trunk

2011-06-29 Hayato Ito <​hayato@chromium.org>

Reviewed by Hajime Morita.

Make a shadow host transfer a focus to the first focusable element in the shadow root when its focus() is called.
​https://bugs.webkit.org/show_bug.cgi?id=62358.

This patch doesn't take an effect on the following elements to keep a compatibility.

  • <input>, <textarea>, <video> and <audio> elements We'll address these elements separately after re-targeting focus events (bug 61421) is implemented.

A shadow root's <content> is not considered in this patch.
That should be addressed in another patch. See bug 63522.

  • fast/dom/shadow/shadow-host-transfer-focus-expected.txt: Added.
  • fast/dom/shadow/shadow-host-transfer-focus.html: Added.

2011-06-29 Hayato Ito <​hayato@chromium.org>

Reviewed by Hajime Morita.

Make a shadow host transfer a focus to the first focusable element in the shadow root when its focus() is called.
​https://bugs.webkit.org/show_bug.cgi?id=62358.

This patch doesn't take an effect on the following elements to keep a compatibility.

  • <input>, <textarea>, <video> and <audio> elements We'll address these elements separately after re-targeting focus events (bug 61421) is implemented.

A shadow root's <content> is not considered in this patch.
That should be addressed in another patch. See bug 63522.

Test: fast/dom/shadow/shadow-host-transfer-focus.html

  • dom/Element.cpp: (WebCore::Element::focus):
  • page/FocusController.cpp: (WebCore::shadowRoot): (WebCore::isTreeScopeOwner): (WebCore::FocusController::transferFocusToElementInShadowRoot): (WebCore::hasCustomFocusLogic): (WebCore::FocusController::findFocusableNodeDecendingDownIntoFrameDocumentOrShadowRoot): (WebCore::FocusController::advanceFocusInDocumentOrder): (WebCore::ownerOfTreeScope): (WebCore::FocusController::findFocusableNodeAcrossTreeScope):
  • page/FocusController.h:
2:32 AM Changeset in webkit [90003] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-06-29 Ryan Sleevi <​rsleevi@chromium.org>

Reviewed by Dirk Schulze.

SVGRadialGradientElement::selfHasRelativeLengths() doesn't consider if cx() is relative
​https://bugs.webkit.org/show_bug.cgi?id=61600

Correct the logic for checking for relative lengths in a SVGRadialGradientElement to consider cx() in addition to cy()

No new tests, as there is no way to currently test this. With the bug present, a side-effect is causing the
expected/correct image to be rendered. This is because the cached SVG resource is getting invalided through a
different path, forcing a repaint. The repaint takes into consideration the proper dimensions, hiding the bug.

  • svg/SVGRadialGradientElement.cpp: (WebCore::SVGRadialGradientElement::selfHasRelativeLengths):
2:29 AM Changeset in webkit [90002] by rolandsteiner@chromium.org
  • 1 edit
    3 adds in trunk/LayoutTests

2011-06-29 Roland Steiner <​rolandsteiner@chromium.org>

Unreviewed: last last ruby expectations for WebKit2 Win.

  • platform/win-wk2/fast/ruby/generated-after-counter-doesnt-crash-expected.txt: Added.
  • platform/win-wk2/fast/ruby/generated-before-and-after-counter-doesnt-crash-expected.txt: Added.
  • platform/win-wk2/fast/ruby/generated-before-counter-doesnt-crash-expected.txt: Added.
2:22 AM Changeset in webkit [90001] by Philippe Normand
  • 2 edits in trunk/LayoutTests

Unreviewed, skip
http/tests/history/cross-origin-replace-history-object-child.html
on GTK, due to bug 61937.

  • platform/gtk/Skipped:
2:18 AM Changeset in webkit [90000] by Philippe Normand
  • 4 edits in trunk/LayoutTests

Unreviewed, GTK rebaseline of 3 svg/custom/linking tests.

  • platform/gtk/svg/custom/linking-a-03-b-all-expected.txt:
  • platform/gtk/svg/custom/linking-a-03-b-transform-expected.txt:
  • platform/gtk/svg/custom/linking-a-03-b-viewBox-transform-expected.txt:
1:59 AM Changeset in webkit [89999] by Philippe Normand
  • 2 edits in trunk/LayoutTests

Unreviewed, GTK rebaseline of
fast/html/details-open-javascript.html

  • platform/gtk/fast/html/details-open-javascript-expected.txt:
1:30 AM Changeset in webkit [89998] by Philippe Normand
  • 2 edits in trunk/LayoutTests

Unreviewed, skip crashing tests on GTK because of
​https://bugs.webkit.org/show_bug.cgi?id=63611

  • platform/gtk/Skipped: skip 5 editing tests
12:57 AM Changeset in webkit [89997] by rolandsteiner@chromium.org
  • 5 edits in trunk/LayoutTests

2011-06-29 Roland Steiner <​rolandsteiner@chromium.org>

Unreviewed: update the (hopefully) last ruby expectation stragglers in WebKit2

  • platform/win-wk2/fast/ruby/after-block-doesnt-crash-expected.txt:
  • platform/win-wk2/fast/ruby/after-table-doesnt-crash-expected.txt:
  • platform/win-wk2/fast/ruby/before-block-doesnt-crash-expected.txt:
  • platform/win-wk2/fast/ruby/before-table-doesnt-crash-expected.txt:
12:31 AM Changeset in webkit [89996] by loislo@chromium.org
  • 2 edits in branches/chromium/782/Source/WebCore/inspector/front-end

Merge 89562 - 2011-06-23 Mikhail Naganov <​mnaganov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: [Chromium] Fix showing dominator leaf nodes after r89457
​https://bugs.webkit.org/show_bug.cgi?id=63250

  • inspector/front-end/DetailedHeapshotView.js:
  • inspector/front-end/HeapSnapshot.js: (WebInspector.HeapSnapshotFilteredOrderedIterator.prototype.get isEmpty):

TBR=​mnaganov@chromium.org
BUG=87583
Review URL: ​http://codereview.chromium.org/7280004

12:21 AM Changeset in webkit [89995] by rolandsteiner@chromium.org
  • 1 edit
    17 adds in trunk/LayoutTests

2011-06-29 Roland Steiner <​rolandsteiner@chromium.org>

Unreviewed: more juggling with ruby expectation files, trying to
find my way out of the expectations dependency jungle...

  • platform/chromium-mac/fast/ruby/after-block-doesnt-crash-expected.txt: Added.
  • platform/chromium-mac/fast/ruby/after-table-doesnt-crash-expected.txt: Added.
  • platform/chromium-mac/fast/ruby/before-block-doesnt-crash-expected.txt: Added.
  • platform/chromium-mac/fast/ruby/before-table-doesnt-crash-expected.txt: Added.
  • platform/chromium-mac/fast/ruby/generated-after-counter-doesnt-crash-expected.txt: Added.
  • platform/chromium-mac/fast/ruby/generated-before-and-after-counter-doesnt-crash-expected.txt: Added.
  • platform/chromium-mac/fast/ruby/generated-before-counter-doesnt-crash-expected.txt: Added.
  • platform/chromium-win/fast/ruby/after-block-doesnt-crash-expected.txt: Added.
  • platform/chromium-win/fast/ruby/after-table-doesnt-crash-expected.txt: Added.
  • platform/chromium-win/fast/ruby/before-block-doesnt-crash-expected.txt: Added.
  • platform/chromium-win/fast/ruby/before-table-doesnt-crash-expected.txt: Added.
  • platform/chromium-win/fast/ruby/generated-after-counter-doesnt-crash-expected.txt: Added.
  • platform/chromium-win/fast/ruby/generated-before-and-after-counter-doesnt-crash-expected.txt: Added.
  • platform/chromium-win/fast/ruby/generated-before-counter-doesnt-crash-expected.txt: Added.
  • platform/win/fast/ruby/generated-after-counter-doesnt-crash-expected.txt: Added.
  • platform/win/fast/ruby/generated-before-and-after-counter-doesnt-crash-expected.txt: Added.
  • platform/win/fast/ruby/generated-before-counter-doesnt-crash-expected.txt: Added.
12:11 AM Changeset in webkit [89994] by Darin Adler
  • 4 edits
    3 moves in trunk/LayoutTests

2011-06-29 Darin Adler <Darin Adler>

Reviewed by Adam Barth.

Consolidate the fast/loader and fast/loading directories
​https://bugs.webkit.org/show_bug.cgi?id=63587

  • fast/loader/resources/subframe-that-removes-itself.html: Copied from LayoutTests/fast/loading/resources/subframe-that-removes-itself.html.
  • fast/loader/subframe-removes-itself-expected.txt: Copied from LayoutTests/fast/loading/subframe-removes-itself-expected.txt.
  • fast/loader/subframe-removes-itself.html: Copied from LayoutTests/fast/loading/subframe-removes-itself.html.
  • fast/loading/resources/subframe-that-removes-itself.html: Removed.
  • fast/loading/subframe-removes-itself-expected.txt: Removed.
  • fast/loading/subframe-removes-itself.html: Removed.
  • platform/gtk/Skipped: Updated for new location of test.
  • platform/mac-wk2/Skipped: Ditto.
  • platform/qt-wk2/Skipped: Ditto.

Jun 28, 2011:

11:54 PM Changeset in webkit [89993] by krit@webkit.org
  • 16 edits
    4 adds in trunk

2011-06-28 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVGAnimatedType should support SVGBoolean animation
​https://bugs.webkit.org/show_bug.cgi?id=63529

Follow up of "SVGAnimation should use direct unit animation for SVGLength": ​https://bugs.webkit.org/show_bug.cgi?id=61368
This patch continues the conversion to the new concept of SVGAnimatorFactory with SVGBoolean.


The new animator does not affect any behavior, since it was animated with SVGAnimatedString before. The new animator is still needed
to support animVal and baseVal later.

Test: svg/animations/svgboolean-animation-1.html

  • CMakeLists.txt: Add new file to build system.
  • GNUmakefile.list.am: Ditto.
  • WebCore.gypi: Ditto.
  • WebCore.pro: Ditto.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • svg/SVGAllInOne.cpp: Ditto.
  • svg/SVGAnimateElement.cpp: Make use of AnimatedBoolean. (WebCore::SVGAnimateElement::determineAnimatedAttributeType): (WebCore::SVGAnimateElement::calculateAnimatedValue): (WebCore::SVGAnimateElement::calculateFromAndToValues): (WebCore::SVGAnimateElement::calculateFromAndByValues): (WebCore::SVGAnimateElement::resetToBaseValue): (WebCore::SVGAnimateElement::applyResultsToTarget): (WebCore::SVGAnimateElement::calculateDistance):
  • svg/SVGAnimatedBoolean.cpp: Added. New animator for SVGBoolean. (WebCore::SVGAnimatedBooleanAnimator::SVGAnimatedBooleanAnimator): (WebCore::SVGAnimatedBooleanAnimator::constructFromString): (WebCore::SVGAnimatedBooleanAnimator::calculateFromAndToValues): (WebCore::SVGAnimatedBooleanAnimator::calculateFromAndByValues): (WebCore::SVGAnimatedBooleanAnimator::calculateAnimatedValue): (WebCore::SVGAnimatedBooleanAnimator::calculateDistance):
  • svg/SVGAnimatedBoolean.h: (WebCore::SVGAnimatedBooleanAnimator::~SVGAnimatedBooleanAnimator):
  • svg/SVGAnimatedType.cpp: Use bool for SVGBoolean. (WebCore::SVGAnimatedType::~SVGAnimatedType): (WebCore::SVGAnimatedType::createBoolean): (WebCore::SVGAnimatedType::boolean): (WebCore::SVGAnimatedType::valueAsString): (WebCore::SVGAnimatedType::setValueAsString):
  • svg/SVGAnimatedType.h:
  • svg/SVGAnimatorFactory.h: New crate function for SVGAnimatedSVGBooleanAnimator. (WebCore::SVGAnimatorFactory::create):

2011-06-28 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVGAnimatedType should support SVGBoolean animation
​https://bugs.webkit.org/show_bug.cgi?id=63529


Test animation of SVGBoolean. Add the test to the skip list of wk2.
Convert animate-number-calcMode-discrete-keyTimes to common style, no change
on the test itself.

  • platform/mac-wk2/Skipped:
  • svg/animations/animate-number-calcMode-discrete-keyTimes-expected.txt:
  • svg/animations/animate-number-calcMode-discrete-keyTimes.html:
  • svg/animations/script-tests/svgboolean-animation-1.js: Added. (sample1): (sample2): (sample3): (executeTest):
  • svg/animations/svgboolean-animation-1-expected.txt: Added.
  • svg/animations/svgboolean-animation-1.html: Added.
11:46 PM Changeset in webkit [89992] by Dimitri Glazkov
  • 3 edits in trunk/Source/WebCore

2011-06-28 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Adam Barth.

Make applyAuthorSheets settable on ShadowRoot.
​https://bugs.webkit.org/show_bug.cgi?id=63599

Refactoring and adding not-yet used function, tests to follow when API lands.

  • dom/ShadowRoot.cpp: (WebCore::ShadowRoot::ShadowRoot): Added initializer. (WebCore::ShadowRoot::applyAuthorSheets): Made to use member variable. (WebCore::ShadowRoot::setApplyAuthorSheets): Added.
  • dom/ShadowRoot.h: Tweaked, added decls.
11:35 PM Changeset in webkit [89991] by isherman@chromium.org
  • 22 edits
    2 adds in trunk

2011-06-28 Ilya Sherman <​isherman@chromium.org>

Reviewed by Adam Barth.

form.autocomplete="off" does not work
​https://bugs.webkit.org/show_bug.cgi?id=35823

  • fast/forms/autocomplete-expected.txt: Added.
  • fast/forms/autocomplete.html: Added.

2011-06-28 Ilya Sherman <​isherman@chromium.org>

Reviewed by Adam Barth.

form.autocomplete="off" does not work
Rename the autoComplete() method for HTMLFormElement, HTMLFormControlElementWithState, and HTMLInputElement to shouldAutocomplete()
Add the autocomplete attribute to HTMLFormElement.idl and HTMLInputElement.idl
​https://bugs.webkit.org/show_bug.cgi?id=35823

Test: fast/forms/autocomplete.html

  • html/HTMLFormControlElement.cpp: (WebCore::HTMLFormControlElementWithState::shouldAutocomplete): Renamed from ::autoComplete() (WebCore::HTMLFormControlElementWithState::shouldSaveAndRestoreFormControlState): Update callsite.
  • html/HTMLFormControlElement.h:
  • html/HTMLFormElement.cpp: (WebCore::HTMLFormElement::~HTMLFormElement): Update callsite. (WebCore::HTMLFormElement::parseMappedAttribute): Update callsite. (WebCore::HTMLFormElement::documentDidBecomeActive): Update callsite. (WebCore::HTMLFormElement::willMoveToNewOwnerDocument): Update callsite. (WebCore::HTMLFormElement::didMoveToNewOwnerDocument): Update callsite. (WebCore::HTMLFormElement::shouldAutocomplete): Renamed from ::autoComplete()
  • html/HTMLFormElement.h:
  • html/HTMLFormElement.idl: Bind the autocomplete attribute to a JavaScript property.
  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::shouldAutocomplete): Renamed from ::autoComplete()
  • html/HTMLInputElement.h:
  • html/HTMLInputElement.idl: Bind the autocomplete attribute to a JavaScript property.
  • WebCore.exp.in: Updated autoComplete() -> shouldAutocomplete()

2011-06-28 Ilya Sherman <​isherman@chromium.org>

Reviewed by Adam Barth.

form.autocomplete="off" does not work
​https://bugs.webkit.org/show_bug.cgi?id=35823

  • src/EditorClientImpl.cpp: (WebKit::EditorClientImpl::autofill): Update callsite.
  • src/WebFormElement.cpp: (WebKit::WebFormElement::autoComplete): Update callsite.
  • src/WebInputElement.cpp: (WebKit::WebInputElement::autoComplete): Update callsite.
  • src/WebPasswordFormUtils.cpp: (WebKit::findPasswordFormFields): Update callsite.

2011-06-28 Ilya Sherman <​isherman@chromium.org>

Reviewed by Adam Barth.

form.autocomplete="off" does not work
​https://bugs.webkit.org/show_bug.cgi?id=35823

  • WebView/WebHTMLRepresentation.mm: (-[WebHTMLRepresentation elementDoesAutoComplete:]): Update callsite.

2011-06-28 Ilya Sherman <​isherman@chromium.org>

Reviewed by Adam Barth.

form.autocomplete="off" does not work
​https://bugs.webkit.org/show_bug.cgi?id=35823

  • WebCoreSupport/DumpRenderTreeSupportQt.cpp: (DumpRenderTreeSupportQt::elementDoesAutoCompleteForElementWithId): Update callsite.

2011-06-28 Ilya Sherman <​isherman@chromium.org>

Reviewed by Adam Barth.

form.autocomplete="off" does not work
​https://bugs.webkit.org/show_bug.cgi?id=35823

  • WebFrame.cpp: (WebFrame::elementDoesAutoComplete): Updated callsite.
11:06 PM Changeset in webkit [89990] by rolandsteiner@chromium.org
  • 15 edits
    8 deletes in trunk/LayoutTests

2011-06-28 Roland Steiner <​rolandsteiner@chromium.org>

Unreviewed: change URLs in ruby layout tests to lower case in order
to avoid having to have separate Chromium expectations.

  • fast/ruby/after-block-doesnt-crash-expected.txt:
  • fast/ruby/after-block-doesnt-crash.html:
  • fast/ruby/after-table-doesnt-crash-expected.txt:
  • fast/ruby/after-table-doesnt-crash.html:
  • fast/ruby/before-block-doesnt-crash-expected.txt:
  • fast/ruby/before-block-doesnt-crash.html:
  • fast/ruby/before-table-doesnt-crash-expected.txt:
  • fast/ruby/before-table-doesnt-crash.html:
  • fast/ruby/generated-after-counter-doesnt-crash-expected.txt:
  • fast/ruby/generated-after-counter-doesnt-crash.html:
  • fast/ruby/generated-before-and-after-counter-doesnt-crash-expected.txt:
  • fast/ruby/generated-before-and-after-counter-doesnt-crash.html:
  • fast/ruby/generated-before-counter-doesnt-crash-expected.txt:
  • fast/ruby/generated-before-counter-doesnt-crash.html:
  • platform/chromium-mac/fast/ruby/after-block-doesnt-crash-expected.txt: Removed.
  • platform/chromium-mac/fast/ruby/after-table-doesnt-crash-expected.txt: Removed.
  • platform/chromium-mac/fast/ruby/before-block-doesnt-crash-expected.txt: Removed.
  • platform/chromium-mac/fast/ruby/before-table-doesnt-crash-expected.txt: Removed.
  • platform/chromium-win/fast/ruby/after-block-doesnt-crash-expected.txt: Removed.
  • platform/chromium-win/fast/ruby/after-table-doesnt-crash-expected.txt: Removed.
  • platform/chromium-win/fast/ruby/before-block-doesnt-crash-expected.txt: Removed.
  • platform/chromium-win/fast/ruby/before-table-doesnt-crash-expected.txt: Removed.
11:05 PM Changeset in webkit [89989] by Dimitri Glazkov
  • 6 edits in trunk/Source/WebCore

2011-06-28 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Kent Tamura.

Store shadowPseudoId in ElementRareData.
​https://bugs.webkit.org/show_bug.cgi?id=63604

No new tests, because the code isn't yet used.

  • css/CSSSelector.cpp: (WebCore::CSSSelector::isUnknownPseudoType): Added convenience helper.
  • css/CSSSelector.h: Added decl.
  • dom/Element.cpp: (WebCore::Element::shadowPseudoId): Changed to use ElementRareData. (WebCore::Element::setShadowPseudoId): Ditto.
  • dom/Element.h: Added decls.
  • dom/ElementRareData.h: Added a member to keep shadowPseudoId.
11:01 PM Changeset in webkit [89988] by bashi@chromium.org
  • 9 edits in trunk/Source/WebCore

2011-06-28 Kenichi Ishibashi <​bashi@chromium.org>

Reviewed by Tony Chang.

[Chromium] Remove dependencies on harfbuzz from FontPlatformDataLinux and FontLinux
​https://bugs.webkit.org/show_bug.cgi?id=62530

  • Moved codes of FontLinux which depend on harfbuzz APIs to ComplexTextController. Removed Some methods of ComplexTextController(advances() and logClusters()) since they are no longer needed.
  • Moved RefCountedHarfbuzzFace from FontPlatformDataLinux to HarfbuzzSkia and renamed it to HarfbuzzFace.

No new tests because there is no behavior change (The existing tests should cover the changes).

  • platform/graphics/chromium/ComplexTextControllerLinux.cpp: (WebCore::ComplexTextController::ComplexTextController): Added arguments so eliminating setter invocations. (WebCore::ComplexTextController::setupForRTL): Added. (WebCore::ComplexTextController::setupFontForScriptRun): Adopt the change of HarfbuzzFace class. (WebCore::ComplexTextController::setGlyphPositions): Use m_item.log_clusters instead of removed method. (WebCore::ComplexTextController::glyphIndexForXPositionInScriptRun): Added. (WebCore::ComplexTextController::offsetForPosition): Ditto. (WebCore::ComplexTextController::selectionRect): Ditto.
  • platform/graphics/chromium/ComplexTextControllerLinux.h: (WebCore::ComplexTextController::width):
  • platform/graphics/chromium/FontLinux.cpp: Removed truncateFixedPointToInteger(). (WebCore::Font::drawComplexText): Removed setter invocations of ComplexTextController. (WebCore::Font::floatWidthForComplexText): Ditto. (WebCore::Font::offsetForPositionForComplexText): Moved harfbuzz dependent code to ComplexTextController. (WebCore::Font::selectionRectForComplexText): Ditto.
  • platform/graphics/chromium/FontPlatformDataLinux.cpp: (WebCore::FontPlatformData::harfbuzzFace): Wrapped up HB_FaceRec in HarfbuzzFace class.
  • platform/graphics/chromium/FontPlatformDataLinux.h: Moved RefCountedHarfbuzzFace class and renamed to HarfbuzzFace.
  • platform/graphics/chromium/HarfbuzzSkia.cpp: (WebCore::allocHarfbuzzFont): Moved from ComplexTextControllerLinux.cpp. (WebCore::HarfbuzzFace::HarfbuzzFace): Added. (WebCore::HarfbuzzFace::~HarfbuzzFace): Added.
  • platform/graphics/chromium/HarfbuzzSkia.h: (WebCore::HarfbuzzFace::create): Added. (WebCore::HarfbuzzFace::face): Added.
  • platform/graphics/skia/GlyphPageTreeNodeSkia.cpp: (WebCore::substituteWithVerticalGlyphs): Adopt the change of HarfbuzzFace class.
11:00 PM Writing Layout Tests for DumpRenderTree edited by dominicc@chromium.org
Documented overridePreference. (diff)
9:34 PM Changeset in webkit [89987] by rolandsteiner@chromium.org
  • 3 edits
    6 adds in trunk

2011-06-28 Roland Steiner <​rolandsteiner@chromium.org>

Reviewed by Eric Seidel.

Bug 55930 - (CVE-2011-1440) Incorrect handling of 'display:' property within nested <ruby> tags
​https://bugs.webkit.org/show_bug.cgi?id=55930

Test that a generated block child + counter within a <ruby> doesn't crash.
(Test as provided by original reporter).

  • fast/ruby/generated-after-counter-doesnt-crash-expected.txt: Added.
  • fast/ruby/generated-after-counter-doesnt-crash.html: Added.
  • fast/ruby/generated-before-and-after-counter-doesnt-crash-expected.txt: Added.
  • fast/ruby/generated-before-and-after-counter-doesnt-crash.html: Added.
  • fast/ruby/generated-before-counter-doesnt-crash-expected.txt: Added.
  • fast/ruby/generated-before-counter-doesnt-crash.html: Added.

2011-06-28 Roland Steiner <​rolandsteiner@chromium.org>

Reviewed by Eric Seidel.

Bug 55930 - (CVE-2011-1440) Incorrect handling of 'display:' property within nested <ruby> tags
​https://bugs.webkit.org/show_bug.cgi?id=55930

Don't set style type BEFORE/AFTER on anonymous wrapper block.
Rather, check style type on generated wrapped child.

Tests: fast/ruby/generated-after-counter-doesnt-crash.html

fast/ruby/generated-before-and-after-counter-doesnt-crash.html
fast/ruby/generated-before-counter-doesnt-crash.html

  • rendering/RenderRuby.cpp: (WebCore::isAnonymousRubyInlineBlock): (WebCore::isRubyBeforeBlock): (WebCore::isRubyAfterBlock): (WebCore::rubyBeforeBlock): (WebCore::rubyAfterBlock): (WebCore::createAnonymousRubyInlineBlock): (WebCore::RenderRubyAsInline::addChild): (WebCore::RenderRubyAsBlock::addChild):
8:18 PM Changeset in webkit [89986] by barraclough@apple.com
  • 8 edits in trunk/Source/JavaScriptCore

DFG JIT lacks array.length caching.
​https://bugs.webkit.org/show_bug.cgi?id=63505

Patch by Filip Pizlo <​fpizlo@apple.com> on 2011-06-28
Reviewed by Gavin Barraclough.

  • bytecode/StructureStubInfo.h:
  • dfg/DFGJITCodeGenerator.cpp:

(JSC::DFG::JITCodeGenerator::cachedGetById):
(JSC::DFG::JITCodeGenerator::cachedPutById):

  • dfg/DFGJITCodeGenerator.h:

(JSC::DFG::JITCodeGenerator::tryAllocate):
(JSC::DFG::JITCodeGenerator::selectScratchGPR):
(JSC::DFG::JITCodeGenerator::silentSpillAllRegisters):

  • dfg/DFGJITCompiler.cpp:

(JSC::DFG::JITCompiler::compileFunction):

  • dfg/DFGJITCompiler.h:

(JSC::DFG::JITCompiler::addPropertyAccess):
(JSC::DFG::JITCompiler::PropertyAccessRecord::PropertyAccessRecord):

  • dfg/DFGRegisterBank.h:

(JSC::DFG::RegisterBank::tryAllocate):

  • dfg/DFGRepatch.cpp:

(JSC::DFG::tryCacheGetByID):

7:41 PM Changeset in webkit [89985] by commit-queue@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

2011-06-28 Pierre Rossi <​pierre.rossi@gmail.com>

Reviewed by Eric Seidel.

Warnings in JSC's JIT on 32 bit
​https://bugs.webkit.org/show_bug.cgi?id=63259

Fairly straightforward, just use ASSERT_JIT_OFFSET_UNUSED when it applies.

  • jit/JITPropertyAccess32_64.cpp: (JSC::JIT::emit_op_method_check): (JSC::JIT::compileGetByIdHotPath): (JSC::JIT::emit_op_put_by_id):
6:28 PM Changeset in webkit [89984] by eae@chromium.org
  • 5 edits in trunk/Source/WebCore

2011-06-28 Emil A Eklund <​eae@chromium.org>

Reviewed by Eric Seidel.

Switch RenderLayer position/size to to new layout types
​https://bugs.webkit.org/show_bug.cgi?id=63578

Switch location and size methods for RenderLayer over to the new layout unit abstraction.

No new tests, no functionality changes.

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::resize): (WebCore::RenderLayer::visibleHeight): (WebCore::RenderLayer::visibleWidth): (WebCore::RenderLayer::offsetFromResizeCorner):
  • rendering/RenderLayer.h: (WebCore::RenderLayer::location): (WebCore::RenderLayer::setLocation): (WebCore::RenderLayer::size): (WebCore::RenderLayer::setSize): (WebCore::RenderLayer::rect): (WebCore::RenderLayer::renderBoxLocation): (WebCore::RenderLayer::renderBoxX): (WebCore::RenderLayer::renderBoxY):
  • rendering/RenderListBox.cpp: (WebCore::RenderListBox::visibleHeight): (WebCore::RenderListBox::visibleWidth):
  • rendering/RenderListBox.h:
6:06 PM Changeset in webkit [89983] by leviw@chromium.org
  • 23 edits in trunk/Source/WebCore

2011-06-28 Levi Weintraub <​leviw@chromium.org>

Reviewed by Eric Seidel.

Switch paintObject and paintReplaced to new layout types
​https://bugs.webkit.org/show_bug.cgi?id=63585

Switching paintObject and paintReplaced to layout type abstractions from more integral types.

No new tests as this is just moving to an abstraction.

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::paintObject):
  • rendering/RenderBlock.h:
  • rendering/RenderBox.h: (WebCore::RenderBox::paintObject):
  • rendering/RenderEmbeddedObject.cpp: (WebCore::RenderEmbeddedObject::paintReplaced):
  • rendering/RenderEmbeddedObject.h:
  • rendering/RenderFileUploadControl.cpp: (WebCore::RenderFileUploadControl::paintObject):
  • rendering/RenderFileUploadControl.h:
  • rendering/RenderHTMLCanvas.cpp: (WebCore::RenderHTMLCanvas::paintReplaced):
  • rendering/RenderHTMLCanvas.h:
  • rendering/RenderImage.cpp: (WebCore::RenderImage::paintReplaced):
  • rendering/RenderImage.h:
  • rendering/RenderListBox.cpp: (WebCore::RenderListBox::paintObject):
  • rendering/RenderListBox.h:
  • rendering/RenderReplaced.h: (WebCore::RenderReplaced::paintReplaced):
  • rendering/RenderTable.cpp: (WebCore::RenderTable::paintObject):
  • rendering/RenderTable.h:
  • rendering/RenderTableSection.cpp: (WebCore::RenderTableSection::paintObject):
  • rendering/RenderTableSection.h:
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::paintObject):
  • rendering/RenderTextControl.h:
  • rendering/RenderVideo.cpp: (WebCore::RenderVideo::paintReplaced):
  • rendering/RenderVideo.h:
6:03 PM Changeset in webkit [89982] by agl@chromium.org
  • 3 edits
    6 adds in trunk

2011-06-28 Adam Langley <​agl@chromium.org>

Reviewed by Adam Barth.

Skip cache validation on back with HTTPS.
​https://bugs.webkit.org/show_bug.cgi?id=63537

  • http/tests/cache/history-only-cached-subresource-loads-max-age-https-expected.txt: Added.
  • http/tests/cache/history-only-cached-subresource-loads-max-age-https.html: Added.
  • http/tests/cache/resources/max-age-resource-forward.html: Added.
  • http/tests/cache/resources/max-age-resource-next.html: Added.
  • http/tests/cache/resources/max-age-resource.html: Added.
  • http/tests/cache/resources/random-max-age.cgi: Added.

2011-06-28 Adam Langley <​agl@chromium.org>

Reviewed by Adam Barth.

Skip cache validation on back with HTTPS.
​https://bugs.webkit.org/show_bug.cgi?id=63537

For back navigations over HTTP, WebKit sets a flag to use cached data
irrespective of whether it has expired. However, this isn't currently
done for HTTPS.

The logic in question was added in
​https://bugs.webkit.org/show_bug.cgi?id=33993 and appears to have been
precautionary. However, both Firefox and IE will use this back
navigation trick for HTTPS as well and so we should probably behave
likewise.

Test: http/tests/cache/history-only-cached-subresource-loads-max-age-https.html

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::addExtraFieldsToRequest):
5:58 PM Changeset in webkit [89981] by Dimitri Glazkov
  • 2 edits in trunk/Source/WebCore

2011-06-28 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Adam Barth.

Remove errant isShadowBoundary on ShadowRoot.
​https://bugs.webkit.org/show_bug.cgi?id=63588

Removing dead code, no tests for that.

  • dom/ShadowRoot.h: Removed a function that's no longer used.
5:57 PM Changeset in webkit [89980] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

2011-06-28 Zeng Huiqing <​huiqing.zeng@intel.com>

Reviewed by Kenneth Russell.

[chromium]Optimize GraphicsLayerChromium::setChildren()
​https://bugs.webkit.org/show_bug.cgi?id=62837

No new tests.

  • platform/graphics/chromium/GraphicsLayerChromium.cpp: (WebCore::GraphicsLayerChromium::GraphicsLayerChromium): (WebCore::GraphicsLayerChromium::setChildren): (WebCore::GraphicsLayerChromium::addChild):
  • platform/graphics/chromium/GraphicsLayerChromium.h:
5:53 PM Changeset in webkit [89979] by leviw@chromium.org
  • 14 edits in trunk/Source/WebCore

2011-06-28 Levi Weintraub <​leviw@chromium.org>

Reviewed by Eric Seidel.

Switch paintBoxShadow and paintBoxDecorations to new layout types
​https://bugs.webkit.org/show_bug.cgi?id=63577

Switching paintBoxShadow and paintBoxDecorations to layout type abstraction
from more integral types.

No new tests as this is just moving to an abstraction.

  • rendering/InlineFlowBox.cpp: (WebCore::InlineFlowBox::paintBoxShadow): (WebCore::InlineFlowBox::paintBoxDecorations):
  • rendering/InlineFlowBox.h:
  • rendering/RenderBox.cpp: (WebCore::RenderBox::paintBoxDecorations):
  • rendering/RenderBox.h:
  • rendering/RenderBoxModelObject.cpp: (WebCore::RenderBoxModelObject::paintBoxShadow):
  • rendering/RenderBoxModelObject.h:
  • rendering/RenderFieldset.cpp: (WebCore::RenderFieldset::paintBoxDecorations):
  • rendering/RenderFieldset.h:
  • rendering/RenderTable.cpp: (WebCore::RenderTable::paintBoxDecorations):
  • rendering/RenderTable.h:
  • rendering/RenderTableCell.cpp: (WebCore::RenderTableCell::paintBoxDecorations):
  • rendering/RenderTableCell.h:
  • rendering/RenderView.cpp: (WebCore::RenderView::paintBoxDecorations):
  • rendering/RenderView.h:
5:49 PM Changeset in webkit [89978] by rniwa@webkit.org
  • 2 edits in trunk/Source/WebKit/chromium

2011-06-28 Ryosuke Niwa <​rniwa@webkit.org>

Rolled DEPS.

  • DEPS:
5:44 PM WebKit Team edited by gavinp@chromium.org
add moi (diff)
5:40 PM WebKit Team edited by bweinstein@apple.com
(diff)
4:56 PM Changeset in webkit [89977] by eae@chromium.org
  • 9 edits in trunk/Source/WebCore

2011-06-28 Emil A Eklund <​eae@chromium.org>

Reviewed by Eric Seidel.

Switch RenderBox position/size to to new layout types
​https://bugs.webkit.org/show_bug.cgi?id=63571

Switch location and size methods for RenderBox over to the new layout unit abstraction.

No new tests, no functionality changes.

  • rendering/RenderBox.cpp:
  • rendering/RenderBox.h:
  • rendering/RenderBoxModelObject.cpp:
  • rendering/RenderBoxModelObject.h:
  • rendering/RenderInline.cpp:
  • rendering/RenderInline.h:
  • rendering/RenderVideo.cpp:
  • rendering/RenderVideo.h:
4:52 PM Changeset in webkit [89976] by bfulgham@webkit.org
  • 2 edits in trunk/WebKitLibraries

Unreviewed build correction. Link to correct library for
the zlib1.dll.

  • win/tools/vsprops/WinCairo.vsprops: Don't link to the static

zlib.lib when building for WinCairo.

4:39 PM Changeset in webkit [89975] by commit-queue@webkit.org
  • 27 edits in trunk/Source

2011-06-28 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r89968.
​http://trac.webkit.org/changeset/89968
​https://bugs.webkit.org/show_bug.cgi?id=63581

Broke chromium windows compile (Requested by jamesr on
#webkit).

  • wtf/Platform.h:

2011-06-28 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r89968.
​http://trac.webkit.org/changeset/89968
​https://bugs.webkit.org/show_bug.cgi?id=63581

Broke chromium windows compile (Requested by jamesr on
#webkit).

  • WebCore.gyp/WebCore.gyp:
  • loader/cache/CachedFont.cpp:
  • platform/chromium/DragImageRef.h:
  • platform/graphics/FloatPoint.h:
  • platform/graphics/FloatRect.h:
  • platform/graphics/FloatSize.h:
  • platform/graphics/FontPlatformData.h: (WebCore::FontPlatformData::hash):
  • platform/graphics/GlyphBuffer.h: (WebCore::GlyphBuffer::advanceAt): (WebCore::GlyphBuffer::add): (WebCore::GlyphBuffer::expandLastAdvance):
  • platform/graphics/IntPoint.h:
  • platform/graphics/IntRect.h:
  • platform/graphics/IntSize.h:
  • platform/graphics/SimpleFontData.h:
  • platform/graphics/cg/FloatPointCG.cpp:
  • platform/graphics/cg/FloatRectCG.cpp:
  • platform/graphics/cg/FloatSizeCG.cpp:
  • platform/graphics/cg/IntPointCG.cpp:
  • platform/graphics/cg/IntRectCG.cpp:
  • platform/graphics/cg/IntSizeCG.cpp:
  • platform/graphics/mac/FontCustomPlatformData.cpp: (WebCore::FontCustomPlatformData::~FontCustomPlatformData): (WebCore::createFontCustomPlatformData):
  • platform/graphics/mac/FontCustomPlatformData.h: (WebCore::FontCustomPlatformData::FontCustomPlatformData):

2011-06-28 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r89968.
​http://trac.webkit.org/changeset/89968
​https://bugs.webkit.org/show_bug.cgi?id=63581

Broke chromium windows compile (Requested by jamesr on
#webkit).

  • WebKit.gyp:
  • features.gypi:
  • public/WebCommon.h:
4:36 PM Changeset in webkit [89974] by leviw@chromium.org
  • 11 edits in trunk/Source/WebCore

2011-06-28 Levi Weintraub <​leviw@chromium.org>

Reviewed by Eric Seidel.

Switch PaintMask* to new layout types
​https://bugs.webkit.org/show_bug.cgi?id=63576

Switching paintMask* to layout type abstraction from more integral types.

No new tests as this is just moving to an abstraction.

  • rendering/InlineFlowBox.cpp: (WebCore::InlineFlowBox::paintMask):
  • rendering/InlineFlowBox.h:
  • rendering/RenderBox.cpp: (WebCore::RenderBox::paintMask): (WebCore::RenderBox::paintMaskImages):
  • rendering/RenderBox.h:
  • rendering/RenderFieldset.cpp: (WebCore::RenderFieldset::paintMask):
  • rendering/RenderFieldset.h:
  • rendering/RenderTable.cpp: (WebCore::RenderTable::paintMask):
  • rendering/RenderTable.h:
  • rendering/RenderTableCell.cpp: (WebCore::RenderTableCell::paintMask):
  • rendering/RenderTableCell.h:
4:26 PM Changeset in webkit [89973] by oliver@apple.com
  • 5 edits in trunk/Source/JavaScriptCore

2011-06-28 Oliver Hunt <​oliver@apple.com>

Reviewed by Gavin Barraclough.

Fix sampling build
​https://bugs.webkit.org/show_bug.cgi?id=63579

Gets opcode sampling building again, doesn't seem to work alas

  • bytecode/SamplingTool.cpp: (JSC::SamplingTool::notifyOfScope):
  • bytecode/SamplingTool.h: (JSC::SamplingTool::SamplingTool):
  • interpreter/Interpreter.cpp: (JSC::Interpreter::enableSampler):
  • runtime/Executable.h: (JSC::ScriptExecutable::ScriptExecutable):
4:11 PM Changeset in webkit [89972] by diegohcg@webkit.org
  • 7 edits in trunk

2011-06-27 Diego Gonzalez <​diegohcg@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] DRT support for setInteractiveFormValidationEnabled
​https://bugs.webkit.org/show_bug.cgi?id=63496

Unskipped:

fast/forms/interactive-validation-prevented.html
fast/forms/interactive-validation-attach-assertion.html
fast/forms/interactive-validation-assertion-by-validate-twice.html
fast/forms/interactive-validation-select-crash.html
fast/forms/interactive-validation-crash-by-style-override.html

Source/WebKit/qt:

  • WebCoreSupport/DumpRenderTreeSupportQt.cpp: (DumpRenderTreeSupportQt::setInteractiveFormValidationEnabled):
  • WebCoreSupport/DumpRenderTreeSupportQt.h:

Tools:

  • DumpRenderTree/qt/DumpRenderTreeQt.cpp: (WebCore::DumpRenderTree::DumpRenderTree):

LayoutTests:

  • platform/qt/Skipped:
4:06 PM Changeset in webkit [89971] by eric@webkit.org
  • 4 edits in trunk/Tools

2011-06-28 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Make new-run-webkit-tests -2 actually use WebKitTestRunner
​https://bugs.webkit.org/show_bug.cgi?id=63572

new-run-webkit-tests -2 works with this change. However a zillion
tests fail since it can't find the right skipped lists for wk2.
The Skipped-list fixing is covered in bug 63501.

  • Scripts/webkitpy/layout_tests/port/gtk.py:
  • Scripts/webkitpy/layout_tests/port/qt.py:
  • Scripts/webkitpy/layout_tests/port/webkit.py:
3:41 PM Changeset in webkit [89970] by leviw@chromium.org
  • 8 edits in trunk/Source/WebCore

2011-06-28 Levi Weintraub <​leviw@chromium.org>

Reviewed by Eric Seidel.

Switch paintFillLayer* to new layout types
​https://bugs.webkit.org/show_bug.cgi?id=63570

Switching painFillLayer* to layout type abstraction from more integral types.

No new tests as this is just moving to an abstraction.

  • rendering/InlineFlowBox.cpp: (WebCore::InlineFlowBox::paintFillLayers): (WebCore::InlineFlowBox::paintFillLayer):
  • rendering/InlineFlowBox.h:
  • rendering/RenderBox.cpp: (WebCore::RenderBox::paintFillLayers): (WebCore::RenderBox::paintFillLayer):
  • rendering/RenderBox.h:
  • rendering/RenderBoxModelObject.cpp: (WebCore::RenderBoxModelObject::paintFillLayerExtended):
  • rendering/RenderBoxModelObject.h:
  • rendering/RenderObject.h:
3:33 PM Changeset in webkit [89969] by macpherson@chromium.org
  • 2 edits in trunk/Source/WebCore

2011-06-28 Luke Macpherson <​macpherson@chromium.org>

Reviewed by Eric Seidel.

Use appropriate macro for CSSPropertyWebkitAppearance in CSSStyleSelector::applyProperty()
​https://bugs.webkit.org/show_bug.cgi?id=63197

No new tests / no functionality changed.

  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::applyProperty): Use HANDLE_INHERIT_AND_INITIAL_AND_PRIMITIVE macro and delete duplicate code.
3:27 PM Changeset in webkit [89968] by commit-queue@webkit.org
  • 27 edits in trunk/Source

2011-06-28 Cary Clark <​caryclark@google.com>

Reviewed by James Robinson.

Use Skia if Skia on Mac Chrome is enabled
​https://bugs.webkit.org/show_bug.cgi?id=62999

  • wtf/Platform.h: Add switch to use Skia if, externally, Skia has been enabled by a gyp define.

2011-06-28 Cary Clark <​caryclark@google.com>

Reviewed by James Robinson.

Use Skia if Skia on Mac Chrome is enabled
​https://bugs.webkit.org/show_bug.cgi?id=62999

No new tests. This does not affect existing
functionality.

  • WebCore.gyp/WebCore.gyp: Include Skia and related files and exclude CG and related files when building Chromium for Skia on the Mac.
  • loader/cache/CachedFont.cpp: Rename CHROME to CHROMIUM.
  • platform/chromium/DragImageRef.h:
  • platform/graphics/FloatPoint.h:
  • platform/graphics/FloatRect.h:
  • platform/graphics/FloatSize.h:
  • platform/graphics/FontPlatformData.h: (WebCore::FontPlatformData::hash):
  • platform/graphics/GlyphBuffer.h: (WebCore::GlyphBuffer::advanceAt): (WebCore::GlyphBuffer::add): (WebCore::GlyphBuffer::expandLastAdvance):
  • platform/graphics/IntPoint.h:
  • platform/graphics/IntRect.h:
  • platform/graphics/IntSize.h:
  • platform/graphics/SimpleFontData.h:
  • platform/graphics/cg/FloatPointCG.cpp:
  • platform/graphics/cg/FloatRectCG.cpp:
  • platform/graphics/cg/FloatSizeCG.cpp:
  • platform/graphics/cg/IntPointCG.cpp:
  • platform/graphics/cg/IntRectCG.cpp:
  • platform/graphics/cg/IntSizeCG.cpp:
  • platform/graphics/mac/FontCustomPlatformData.cpp: (WebCore::FontCustomPlatformData::~FontCustomPlatformData): (WebCore::createFontCustomPlatformData):
  • platform/graphics/mac/FontCustomPlatformData.h: (WebCore::FontCustomPlatformData::FontCustomPlatformData): Ditto.

2011-06-28 Cary Clark <​caryclark@google.com>

Reviewed by James Robinson.

Use Skia if Skia on Mac Chrome is enabled
​https://bugs.webkit.org/show_bug.cgi?id=62999

  • WebKit.gyp: If building for Skia, exclude CG files. If not, exclude Skia files.
  • features.gypi: Enable Skia on Chromium Mac if so instructed by the gyp define. Allow Skia to be hardware accelerated on the Mac.
  • public/WebCommon.h: If Skia is defined on the Mac, define it in the WebXXX interfaces as well.
3:15 PM Changeset in webkit [89967] by Nate Chapin
  • 1 edit
    16 adds in trunk/LayoutTests

2011-06-28 Nate Chapin <Nate Chapin>

Unreviewed, chromium test update for r89951.

  • platform/chromium-linux/svg/W3C-SVG-1.1-SE/text-tref-03-b-expected.png: Added.
  • platform/chromium-linux/svg/custom/text-tref-03-b-change-href-dom-expected.png: Added.
  • platform/chromium-linux/svg/custom/text-tref-03-b-change-href-expected.png: Added.
  • platform/chromium-linux/svg/custom/text-tref-03-b-referenced-element-removal-expected.png: Added.
  • platform/chromium-linux/svg/custom/text-tref-03-b-tref-removal-expected.png: Added.
  • platform/chromium-mac-leopard/svg/W3C-SVG-1.1-SE/text-tref-03-b-expected.png: Added.
  • platform/chromium-win/svg/W3C-SVG-1.1-SE/text-tref-03-b-expected.png: Added.
  • platform/chromium-win/svg/W3C-SVG-1.1-SE/text-tref-03-b-expected.txt: Added.
  • platform/chromium-win/svg/custom/text-tref-03-b-change-href-dom-expected.png: Added.
  • platform/chromium-win/svg/custom/text-tref-03-b-change-href-dom-expected.txt: Added.
  • platform/chromium-win/svg/custom/text-tref-03-b-change-href-expected.png: Added.
  • platform/chromium-win/svg/custom/text-tref-03-b-change-href-expected.txt: Added.
  • platform/chromium-win/svg/custom/text-tref-03-b-referenced-element-removal-expected.png: Added.
  • platform/chromium-win/svg/custom/text-tref-03-b-referenced-element-removal-expected.txt: Added.
  • platform/chromium-win/svg/custom/text-tref-03-b-tref-removal-expected.png: Added.
  • platform/chromium-win/svg/custom/text-tref-03-b-tref-removal-expected.txt: Added.
3:09 PM Changeset in webkit [89966] by rniwa@webkit.org
  • 2 edits in trunk/Tools

2011-06-28 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Adam Roben.

Add more machines to Chromium Mac Release and Chromium Mac Release (Tests)
​https://bugs.webkit.org/show_bug.cgi?id=63500

Added google-mac-3 and google-mac-4.

  • BuildSlaveSupport/build.webkit.org-config/config.json:
3:06 PM Changeset in webkit [89965] by eric@webkit.org
  • 10 edits in trunk/Tools

2011-06-28 Eric Seidel <​eric@webkit.org>

Reviewed by Dirk Pranke.

Remove webkitpy's tiger support now that WebKit no longer supports tiger
​https://bugs.webkit.org/show_bug.cgi?id=63568

I don't think our support actually fully worked anyway. Regardless
tiger is dead now (thankfully)!

I also unwrapped some short lines and added a couple FIXMEs while
touring the code.

No new tests, only removing old ones which tested our Tiger support.

  • Scripts/webkitpy/layout_tests/port/base.py:
  • Scripts/webkitpy/layout_tests/port/builders.py:
  • Scripts/webkitpy/layout_tests/port/chromium_linux.py:
  • Scripts/webkitpy/layout_tests/port/chromium_mac.py:
  • Scripts/webkitpy/layout_tests/port/chromium_unittest.py:
  • Scripts/webkitpy/layout_tests/port/mac.py:
  • Scripts/webkitpy/layout_tests/port/mac_unittest.py:
  • Scripts/webkitpy/tool/commands/rebaseline.py:
  • Scripts/webkitpy/tool/commands/rebaselineserver_unittest.py:
2:51 PM Changeset in webkit [89964] by oliver@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

2011-06-28 Oliver Hunt <​oliver@apple.com>

Reviewed by Gavin Barraclough.

ASSERT when launching debug builds with interpreter and jit enabled
​https://bugs.webkit.org/show_bug.cgi?id=63566

Add appropriate guards to the various Executable's memory reporting
logic.

  • runtime/Executable.cpp: (JSC::EvalExecutable::compileInternal): (JSC::ProgramExecutable::compileInternal): (JSC::FunctionExecutable::compileForCallInternal): (JSC::FunctionExecutable::compileForConstructInternal):
2:51 PM Changeset in webkit [89963] by eric@webkit.org
  • 3 edits in trunk/Tools

2011-06-28 Eric Seidel <​eric@webkit.org>

Unreviewed, just fixing a typo.

Fix misspellings of "thead" to "thread".

  • Scripts/webkitpy/common/checkout/scm/git.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
2:44 PM Changeset in webkit [89962] by eric@webkit.org
  • 2 edits in trunk/Tools

2011-06-28 Eric Seidel <​eric@webkit.org>

Reviewed by Tony Chang.

new-run-webkit-tests needs a --webkit-test-runner option
​https://bugs.webkit.org/show_bug.cgi?id=63439

This was accidentally fixed in bug 63501 as well.
The root problem here was that the webkit unit tests were not using
enough mocks. I've fixed them here to use more mocks
which should unbreak windows.

  • Scripts/webkitpy/layout_tests/port/webkit_unittest.py:
2:33 PM Changeset in webkit [89961] by barraclough@apple.com
  • 6 edits in trunk/Source/JavaScriptCore

​https://bugs.webkit.org/show_bug.cgi?id=63563
DFG JIT - add support for double arith to speculative path

Reviewed by Oliver Hunt.

Add integer support for div & mod, add double support for div, mod,
add, sub & mul, dynamically selecting based on operand types.

  • dfg/DFGJITCodeGenerator.cpp:

(JSC::DFG::FPRTemporary::FPRTemporary):

  • dfg/DFGJITCodeGenerator.h:
  • dfg/DFGJITCompiler.h:

(JSC::DFG::JITCompiler::assembler):

  • dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::fillSpeculateDouble):
(JSC::DFG::SpeculativeJIT::compile):

  • dfg/DFGSpeculativeJIT.h:

(JSC::DFG::SpeculateDoubleOperand::SpeculateDoubleOperand):
(JSC::DFG::SpeculateDoubleOperand::~SpeculateDoubleOperand):
(JSC::DFG::SpeculateDoubleOperand::index):
(JSC::DFG::SpeculateDoubleOperand::fpr):

2:28 PM Changeset in webkit [89960] by jamesr@google.com
  • 10 edits in branches/chromium/782/Source

Merge 89635 - 2011-06-23 John Bates <​jbates@google.com>

Reviewed by James Robinson.

Fix latch deadlock when GPU process crashes or context is lost.
​https://bugs.webkit.org/show_bug.cgi?id=63189
The main bug fix is to only set/wait latches if the child context has no errors.
Additionally, the LayerChromium classes needed to be modified to not continue drawing when
their corresponding contexts have errors. Otherwise, they would draw with invalid texture ids.

Test: open particles WebGL demo in chrome, kill GPU process from Task Manager; observe no deadlock.

  • platform/graphics/chromium/LayerRendererChromium.cpp: (WebCore::LayerRendererChromium::LayerRendererChromium): (WebCore::LayerRendererChromium::updateAndDrawLayers): (WebCore::LayerRendererChromium::updateLayers): (WebCore::LayerRendererChromium::isCompositorContextLost):
  • platform/graphics/chromium/LayerRendererChromium.h:
  • platform/graphics/chromium/WebGLLayerChromium.cpp: (WebCore::WebGLLayerChromium::drawsContent): (WebCore::WebGLLayerChromium::updateCompositorResources): (WebCore::WebGLLayerChromium::setContext):
  • platform/graphics/chromium/WebGLLayerChromium.h:
  • platform/graphics/chromium/Canvas2DLayerChromium.cpp: (WebCore::Canvas2DLayerChromium::drawsContent):
  • platform/graphics/chromium/Canvas2DLayerChromium.h:

2011-06-23 John Bates <​jbates@google.com>

Reviewed by James Robinson.

Fix latch deadlock when GPU process crashes or context is lost
​https://bugs.webkit.org/show_bug.cgi?id=63189

  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::composite):

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

2:18 PM Changeset in webkit [89959] by oliver@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Fix interpreter build.

2:13 PM Changeset in webkit [89958] by alexis.menard@openbossa.org
  • 3 edits in trunk/Source/WebKit/qt

2011-06-28 Alexis Menard <​alexis.menard@openbossa.org>

Reviewed by Andreas Kling.

[Qt] Use <QtX/x.h> style of headers for Qt tests which rely on it.
​https://bugs.webkit.org/show_bug.cgi?id=63562

Some Qt auto-tests rely on <QtX/x.h> types of includes.

  • Api/qwebkitplatformplugin.h:
  • symbian/platformplugin/qwebkitplatformplugin.h:
2:06 PM Changeset in webkit [89957] by barraclough@apple.com
  • 3 edits in trunk/Source/JavaScriptCore

​https://bugs.webkit.org/show_bug.cgi?id=63561
DFG JIT - don't always assume integer in relational compare

Reviewed by Oliver Hunt.

If neither operand is known integer, or either is in double representation,
then at least use a function call (don't bail off the speculative path).

  • dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compilePeepHoleCall):
(JSC::DFG::SpeculativeJIT::compile):

  • dfg/DFGSpeculativeJIT.h:

(JSC::DFG::SpeculativeJIT::isDataFormatDouble):
(JSC::DFG::SpeculativeJIT::compareIsInteger):

1:47 PM Changeset in webkit [89956] by barraclough@apple.com
  • 5 edits in trunk/Source/JavaScriptCore

Make constant array optimisation less strict about what constitutes a constant
​https://bugs.webkit.org/show_bug.cgi?id=63554

Patch by Oliver Hunt <​oliver@apple.com> on 2011-06-28
Reviewed by Gavin Barraclough.

Now allow string constants in array literals to actually be considered constant,
and so avoid codegen in array literals with strings in them.

  • bytecode/CodeBlock.h:

(JSC::CodeBlock::addConstantBuffer):
(JSC::CodeBlock::constantBuffer):

  • bytecompiler/BytecodeGenerator.cpp:

(JSC::BytecodeGenerator::addConstantBuffer):
(JSC::BytecodeGenerator::addStringConstant):
(JSC::BytecodeGenerator::emitNewArray):

  • bytecompiler/BytecodeGenerator.h:
  • interpreter/Interpreter.cpp:

(JSC::Interpreter::privateExecute):

  • jit/JITStubs.cpp:

(JSC::DEFINE_STUB_FUNCTION):

1:34 PM Changeset in webkit [89955] by Nate Chapin
  • 1 edit
    2 adds in trunk/LayoutTests

2011-06-28 Nate Chapin <Nate Chapin>

Unreviewed, chromium test update for r89946.

  • platform/chromium/fast/regex: Added.
  • platform/chromium/fast/regex/constructor-expected.txt: Added.
1:21 PM Changeset in webkit [89954] by oliver@apple.com
  • 6 edits in trunk/Source/JavaScriptCore

2011-06-28 Oliver Hunt <​oliver@apple.com>

Reviewed by Gavin Barraclough.

Make constant array optimisation less strict about what constitutes a constant
​https://bugs.webkit.org/show_bug.cgi?id=63554

Now allow string constants in array literals to actually be considered constant,
and so avoid codegen in array literals with strings in them.

  • bytecode/CodeBlock.h: (JSC::CodeBlock::addConstantBuffer): (JSC::CodeBlock::constantBuffer):
  • bytecompiler/BytecodeGenerator.cpp: (JSC::BytecodeGenerator::addConstantBuffer): (JSC::BytecodeGenerator::addStringConstant): (JSC::BytecodeGenerator::emitNewArray):
  • bytecompiler/BytecodeGenerator.h:
  • interpreter/Interpreter.cpp: (JSC::Interpreter::privateExecute):
  • jit/JITStubs.cpp: (JSC::DEFINE_STUB_FUNCTION):
1:09 PM Changeset in webkit [89953] by rniwa@webkit.org
  • 2 edits in trunk/Source/WebKit/chromium

2011-06-28 Ryosuke Niwa <​rniwa@webkit.org>

Rolled DEPS.

  • DEPS:
1:09 PM QtWebKitContrib edited by Ademar Reis
(diff)
1:07 PM Changeset in webkit [89952] by rniwa@webkit.org
  • 13 edits in trunk/Source/WebCore

2011-06-28 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Darin Adler.

Stop instantiating Position with PositionIsOffsetInAnchor in various files
​https://bugs.webkit.org/show_bug.cgi?id=63384

Refactoring.

Removed many if conditions that compared the anchor type to Position::PositionIsOffsetInAnchor
because there were also checking that containerNode is a text node.

Also added Position::containerText() to avoid manually casting containerNode().

  • dom/Position.cpp: (WebCore::Position::Position): Added an assertion to ensure BeforeChildren/AfterChildren anchor type won't be used for a node whose contents is ignored by editing. (WebCore::Position::containerText): Added.
  • dom/Position.h:
  • editing/ApplyBlockElementCommand.cpp: (WebCore::isNewLineAtPosition): Removed a redundant comparison of anchor type. (WebCore::ApplyBlockElementCommand::rangeForParagraphSplittingTextNodesIfNeeded): Since containerNode is always a text node when renderStyleOfEnclosingTextNode returns a render style, use new Position constructor that takes Text* and offset. (WebCore::ApplyBlockElementCommand::endOfNextParagrahSplittingTextNodesIfNeeded): Removed redundant comparison of anchor type and use new Position constructor that takes Text* and offset.
  • editing/ApplyStyleCommand.cpp: (WebCore::ApplyStyleCommand::splitTextAtStart): Replaced an assertion that compared anchor type by an assertion that the container node is a text node; use new constructor. (WebCore::ApplyStyleCommand::splitTextAtEnd): Ditto; also added early exits in the case script modified DOM. (WebCore::ApplyStyleCommand::splitTextElementAtStart): Ditto. (WebCore::ApplyStyleCommand::splitTextElementAtEnd): Ditto. (WebCore::ApplyStyleCommand::joinChildTextNodes): Use new constructor.
  • editing/CompositeEditCommand.cpp: (WebCore::Editor::replaceSelectedTextInNode): Calls containerText instead of manually casting containerNode.
  • editing/Editor.cpp: (WebCore::Editor::canDeleteRange): Call Range::startPosition instead of manually instantiating Position by calling startContainer and startOffset.
  • editing/FrameSelection.cpp: (WebCore::FrameSelection::setSelectedRange): Ditto.
  • editing/InsertTextCommand.cpp: (WebCore::InsertTextCommand::input): Calls containerText instead of manually casting containerNode.
  • editing/ReplaceSelectionCommand.cpp: (WebCore::ReplaceSelectionCommand::doApply): Calls containerText instead of manually casting containerNode.
  • editing/VisiblePosition.cpp: (WebCore::VisiblePosition::characterAfter): Removed a redundant anchor type comparison. (WebCore::startVisiblePosition): Call Range::startPosition instead of startContainer and startOffset. (WebCore::endVisiblePosition): Call Range::endPosition instead of endContainer and endOffset.
  • editing/htmlediting.cpp: (WebCore::firstInSpecialElement): Call containerNode() to address <rdar://problem/5027702>. (WebCore::lastInSpecialElement): Ditto; use new constructor.
  • editing/visible_units.cpp: (WebCore::endPositionForLine): Use new constructor. (WebCore::startOfParagraph): Ditto. (WebCore::endOfParagraph): Ditto.
1:04 PM Changeset in webkit [89951] by rwlbuis@webkit.org
  • 6 edits
    19 adds in trunk

2011-06-28 Rob Buis <​rbuis@rim.com>

Reviewed by Nikolas Zimmermann.

SVG1.1SE test text-tref-03-b.svg fails
​https://bugs.webkit.org/show_bug.cgi?id=63390

Support tref on forward referenced content by listening to DOMSubtreeModified events. This also
allows updating after modifications in the referenced content subtree.

Tests: svg/W3C-SVG-1.1-SE/text-tref-03-b.svg

svg/custom/text-tref-03-b-change-href-dom.svg
svg/custom/text-tref-03-b-change-href.svg
svg/custom/text-tref-03-b-dynamic.svg
svg/custom/text-tref-03-b-dynamic2.svg
svg/custom/text-tref-03-b-referenced-element-removal.svg
svg/custom/text-tref-03-b-tref-removal.svg

  • svg/SVGTRefElement.cpp: (WebCore::SubtreeModificationEventListener::create): (WebCore::SubtreeModificationEventListener::cast): (WebCore::SubtreeModificationEventListener::removeFromTarget): (WebCore::SubtreeModificationEventListener::SubtreeModificationEventListener): (WebCore::SubtreeModificationEventListener::operator==): (WebCore::SubtreeModificationEventListener::handleEvent): (WebCore::SVGTRefElement::updateReferencedText): (WebCore::SVGTRefElement::parseMappedAttribute): (WebCore::SVGTRefElement::svgAttributeChanged): (WebCore::SVGTRefElement::buildPendingResource): (WebCore::SVGTRefElement::removedFromDocument):
  • svg/SVGTRefElement.h:

2011-06-28 Rob Buis <​rbuis@rim.com>

Reviewed by Nikolas Zimmermann.

SVG1.1SE test text-tref-03-b.svg fails
​https://bugs.webkit.org/show_bug.cgi?id=63390

  • text-tref-03-b.svg, the w3c SVG1.1 testcase mentioned in the bug.
  • text-tref-03-b-dynamic.svg, tests dynamically adding the tref.
  • text-tref-03-b-dynamic2.svg, tests dynamically adding the referenced content.
  • text-tref-03-b-tref-removal.svg, tests removal of tref.
  • text-tref-03-b-referenced-element-removal.svg, tests removal of referenced content.
  • text-tref-03-b-change-href.svg, tests changing href attribute dynamically through setAttribute.
  • text-tref-03-b-change-href-dom.svg, tests changing href attribute dynamically through href animated property.
  • foreignObject/text-tref-02-b.svg has improved result, but is still not perfect.
  • platform/mac/svg/W3C-SVG-1.1-SE/text-tref-03-b-expected.png: Added.
  • platform/mac/svg/W3C-SVG-1.1-SE/text-tref-03-b-expected.txt: Added.
  • platform/mac/svg/custom/text-tref-03-b-change-href-dom-expected.png: Added.
  • platform/mac/svg/custom/text-tref-03-b-change-href-dom-expected.txt: Added.
  • platform/mac/svg/custom/text-tref-03-b-change-href-expected.png: Added.
  • platform/mac/svg/custom/text-tref-03-b-change-href-expected.txt: Added.
  • platform/mac/svg/custom/text-tref-03-b-referenced-element-removal-expected.png: Added.
  • platform/mac/svg/custom/text-tref-03-b-referenced-element-removal-expected.txt: Added.
  • platform/mac/svg/custom/text-tref-03-b-tref-removal-expected.png: Added.
  • platform/mac/svg/custom/text-tref-03-b-tref-removal-expected.txt: Added.
  • platform/mac/svg/foreignObject/text-tref-02-b-expected.png:
  • platform/mac/svg/foreignObject/text-tref-02-b-expected.txt:
  • svg/W3C-SVG-1.1-SE/text-tref-03-b.svg: Added.
  • svg/custom/text-tref-03-b-change-href-dom.svg: Added.
  • svg/custom/text-tref-03-b-change-href.svg: Added.
  • svg/custom/text-tref-03-b-dynamic-expected.txt: Added.
  • svg/custom/text-tref-03-b-dynamic.svg: Added.
  • svg/custom/text-tref-03-b-dynamic2-expected.txt: Added.
  • svg/custom/text-tref-03-b-dynamic2.svg: Added.
  • svg/custom/text-tref-03-b-referenced-element-removal.svg: Added.
  • svg/custom/text-tref-03-b-tref-removal.svg: Added.
12:37 PM Changeset in webkit [89950] by inferno@chromium.org
  • 3 edits in trunk/Source/WebCore

2011-06-28 Abhishek Arya <​inferno@chromium.org>

Reviewed by Anders Carlsson.

Split the string creation into a local (while creating text runs)
so that it is not destroyed prematurely.
​https://bugs.webkit.org/show_bug.cgi?id=63543

Run fast/forms/text-control-intrinsic-widths.html under valgrind/ASAN.

  • rendering/RenderFileUploadControl.cpp: (WebCore::RenderFileUploadControl::computePreferredLogicalWidths): this one is just done for caution, in case, we don't do the same mistake when adding legacy rounding hacks option.
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::getAvgCharWidth):
12:33 PM Changeset in webkit [89949] by eric@webkit.org
  • 4 edits in trunk/Tools

2011-06-28 Eric Seidel <​eric@webkit.org>

Reviewed by Tony Chang.

new-run-webkit-tests should not run mthml tests when MHTML support is disabled
​https://bugs.webkit.org/show_bug.cgi?id=63549

This way mhtml tests won't show up as "missing" results on all non-chromium platforms.
I happen to go looking for what code was deciding that .mht was a supported
extension, found test_files.py and then removed some redundant doc-strings and
if branching.

  • Scripts/webkitpy/layout_tests/port/test_files.py:
  • Scripts/webkitpy/layout_tests/port/webkit.py:
  • Scripts/webkitpy/layout_tests/port/webkit_unittest.py:
12:19 PM Changeset in webkit [89948] by commit-queue@webkit.org
  • 11 edits in trunk

2011-06-28 Greg Simon <​gregsimon@chromium.org>

Reviewed by Dimitri Glazkov.

Update migration LayoutTest to include indexes and successful migration.
​https://bugs.webkit.org/show_bug.cgi?id=62780

  • storage/indexeddb/migrate-basics-expected.txt:
  • storage/indexeddb/migrate-basics.html:

2011-06-28 Greg Simon <​gregsimon@chromium.org>

Reviewed by Dimitri Glazkov.

Migrate SQLite backing store to LevelDB backing store for Indexeddb.
​https://bugs.webkit.org/show_bug.cgi?id=62780

  • storage/IDBFactoryBackendImpl.cpp: (WebCore::computeFileIdentifier): (WebCore::computeUniqueIdentifier): (WebCore::IDBFactoryBackendImpl::open): (WebCore::migrateObjectStores): (WebCore::IDBFactoryBackendImpl::migrateFromSQLiteToLevelDB):
  • storage/IDBLevelDBBackingStore.cpp: (WebCore::IDBLevelDBBackingStore::backingStoreExists):
  • storage/IDBLevelDBBackingStore.h:
  • storage/IDBObjectStoreBackendImpl.cpp: (WebCore::IDBObjectStoreBackendImpl::populateIndex):
  • storage/IDBObjectStoreBackendImpl.h:
  • storage/IDBSQLiteBackingStore.cpp: (WebCore::IDBSQLiteBackingStore::backingStoreExists):
  • storage/IDBSQLiteBackingStore.h:
12:01 PM Changeset in webkit [89947] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit2

2011-06-28 Chang Shu <​cshu@webkit.org>

Reviewed by Andreas Kling.

[Qt][WK2] Random crashes in fast/frames/ tests
​https://bugs.webkit.org/show_bug.cgi?id=63459

Investigation on the crashes has revealed that sometimes, a frame could
have been destroyed before it is removed from the hierachy in Qt WebKit2 UI process.
The root cause lies in the fact that the order of the two contiguous asynchronous
messages may not be respected on the receiving side. And it is Qt's implementation,
which uses QTimer::startTimer(0), that causes the reverse order. This patch replaces
the timer approach with QMetaObject::invokeMethod approach and no crash happens afterwards.

  • Platform/qt/WorkQueueQt.cpp: (WorkQueue::scheduleWork):
11:35 AM Changeset in webkit [89946] by barraclough@apple.com
  • 4 edits
    3 adds in trunk

​https://bugs.webkit.org/show_bug.cgi?id=55040
RegExp constructor returns the argument regexp instead of a new object

Reviewed by Oliver Hunt.

Per 15.10.3.1, our current behaviour is correct if called as a function,
but incorrect when called as a constructor.

Source/JavaScriptCore:

  • runtime/RegExpConstructor.cpp:

(JSC::constructRegExp):
(JSC::constructWithRegExpConstructor):

  • runtime/RegExpConstructor.h:

LayoutTests:

  • fast/regex/constructor-expected.txt: Added.
  • fast/regex/constructor.html: Added.
  • fast/regex/script-tests/constructor.js: Added.
11:31 AM Changeset in webkit [89945] by leviw@chromium.org
  • 8 edits
    1 add in trunk/Source/WebCore

2011-06-28 Levi Weintraub <​leviw@chromium.org>

Reviewed by Darin Adler.

Create intermediate classes as a path towards getting off of pixel offsets
​https://bugs.webkit.org/show_bug.cgi?id=61896

Creating three intermediate typedefs to aid in moving layout and hit testing from integers
to floats. We'll transition the render tree over to these intermediate types and add
infrastructure to support the new types behind USE(FLOAT_LAYOUT_OFFSETS). This allows the ultimate
changeover to be easier.

Switching LayoutState over to this abstraction to verify it works correctly.

No new tests since this is creating a new abstraction, not changing current behavior.

  • GNUmakefile.list.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • rendering/LayoutState.cpp: (WebCore::LayoutState::LayoutState): (WebCore::LayoutState::pageLogicalOffset): (WebCore::LayoutState::addForcedColumnBreak):
  • rendering/LayoutState.h:
  • rendering/LayoutTypes.h: Added.
11:29 AM Changeset in webkit [89944] by rolandsteiner@chromium.org
  • 2 edits in trunk/Tools

2011-06-28 Roland Steiner <​rolandsteiner@chromium.org>

Reviewed by Tony Chang.

Fix filesystem_unittest.FileSystemTest test_read_and_write_file()
​https://bugs.webkit.org/show_bug.cgi?id=63514

  • change write_text_file to write_binary_file
  • check that text_path and binary_path are actually existant files
  • change cleanup block from 'except' to 'finally'
  • Scripts/webkitpy/common/system/filesystem_unittest.py:
11:26 AM Changeset in webkit [89943] by macpherson@chromium.org
  • 2 edits in trunk/Source/JavaScriptCore

2011-06-28 Luke Macpherson <​macpherson@chromium.org>

Reviewed by Darin Adler.

Clean up integer clamping functions in MathExtras.h and support arbitrary numeric types and limits.
​https://bugs.webkit.org/show_bug.cgi?id=63469

  • wtf/MathExtras.h: (defaultMinimumForClamp): Version of std::numeric_limits::min() that returns the largest negative value for floating point types. (defaultMaximumForClamp): Symmetric alias for std::numeric_limits::max() (clampTo): New templated clamping function that supports arbitrary output types. (clampToInteger): Use new clampTo template. (clampToFloat): Use new clampTo template. (clampToPositiveInteger): Use new clampTo template.
11:24 AM Changeset in webkit [89942] by jeffm@apple.com
  • 2 edits in trunk/Source/WebCore

Add fullscreen.css and fullscreenQuickTime.css to the WebCore Visual Studio project for convenience.
Looks like Visual Studio also re-sorted one more file since my last checkin.

  • WebCore.vcproj/WebCore.vcproj:
11:21 AM Changeset in webkit [89941] by jeffm@apple.com
  • 2 edits in trunk/Source/WebCore

Re-sort the WebCore project with Visual Studio after recent changes that I assume were done by manually editing the XML.

  • WebCore.vcproj/WebCore.vcproj:
10:55 AM Changeset in webkit [89940] by joone.hur@collabora.co.uk
  • 4 edits in trunk

2011-06-28 Joone Hur <​joone.hur@collabora.co.uk>

Reviewed by Martin Robinson.

[GTK] Rename 3D transforms to 3D rendering
​https://bugs.webkit.org/show_bug.cgi?id=63508

WebKitGtk+ provides the 3D transforms feature, but it was
renamed to 3D Rendering. This features supports CSS 3D trasnforms.

  • configure.ac: Rename ENABLE_3D_TRANSFORMS to ENABLE_3D_RENDERING.

2011-06-28 Joone Hur <​joone.hur@collabora.co.uk>

Reviewed by Martin Robinson.

[GTK] Rename 3D transforms to 3D rendering
​https://bugs.webkit.org/show_bug.cgi?id=63508

This patch allows to include ENABLE_3D_RENDERING to build WebKitGtk+.

  • GNUmakefile.am: Add ENABLE_3D_RENDERING
9:58 AM Changeset in webkit [89939] by jhawkins@chromium.org
  • 4 edits in trunk/Source/WebKit/chromium

2011-06-27 James Hawkins <​jhawkins@chromium.org>

Reviewed by Darin Fisher.

[Chromium] Pass an optional WebNode that is the node receiving the context menu action.
​https://bugs.webkit.org/show_bug.cgi?id=63503

  • public/WebFrame.h:
  • src/WebFrameImpl.cpp: (WebKit::pluginContainerFromNode): (WebKit::WebFrameImpl::executeCommand): (WebKit::WebFrameImpl::printBegin):
  • src/WebFrameImpl.h:
9:48 AM Changeset in webkit [89938] by podivilov@chromium.org
  • 4 edits in trunk/Source/WebCore

2011-06-22 Pavel Podivilov <​podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: use column numbers when linking to script location in timeline and console.
​https://bugs.webkit.org/show_bug.cgi?id=63126

  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleMessage.prototype._populateStackTraceTreeElement):
  • inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel.FormattedRecord.prototype._getRecordDetails): (WebInspector.TimelinePanel.PopupContentHelper.prototype._appendStackTrace):
9:27 AM Changeset in webkit [89937] by joone.hur@collabora.co.uk
  • 5 edits
    3 adds in trunk/Source

2011-06-28 Joone Hur <​joone.hur@collabora.co.uk>

Reviewed by Martin Robinson.

[GTK] Allow to build WebKitGtk+ with accelerated compositing
​https://bugs.webkit.org/show_bug.cgi?id=63404

WebKitGtk+ needs to implement accelerated compositing for running CSS3 3D transforms
and animation through GPU acceleration. We could implement GraphicsLayer using Clutter.

  • GNUmakefile.list.am:
  • platform/graphics/clutter/GraphicsLayerClutter.cpp: Added. (WebCore::GraphicsLayer::create): Not implemented yet. (WebCore::GraphicsLayerClutter::GraphicsLayerClutter): (WebCore::GraphicsLayerClutter::~GraphicsLayerClutter): (WebCore::GraphicsLayerClutter::setNeedsDisplay): Ditto. (WebCore::GraphicsLayerClutter::setNeedsDisplayInRect): Ditto.
  • platform/graphics/clutter/GraphicsLayerClutter.h: Added.

2011-06-28 Joone Hur <​joone.hur@collabora.co.uk>

Reviewed by Martin Robinson.

[GTK] Allow to build WebKitGtk+ with accelerated compositing
​https://bugs.webkit.org/show_bug.cgi?id=63404

WebKitGtk+ needs to implement accelerated compositing for running CSS3 3D transforms
and animation through GPU acceleration. We could implement GraphicsLayer using Clutter.

  • WebCoreSupport/ChromeClientGtk.cpp: (WebKit::ChromeClient::attachRootGraphicsLayer): not implemented yet. (WebKit::ChromeClient::setNeedsOneShotDrawingSynchronization): Ditto. (WebKit::ChromeClient::scheduleCompositingLayerSync): Ditto. (WebKit::ChromeClient::allowedCompositingTriggers): Ditto.
  • WebCoreSupport/ChromeClientGtk.h:
9:23 AM Changeset in webkit [89936] by jberlin@webkit.org
  • 2 edits
    1 add in trunk/LayoutTests

Better Windows rebaseline for the changes in r89864, with pixel results, to get the bots
green.

  • platform/win/fast/text/international/unicode-bidi-plaintext-expected.png: Added.
  • platform/win/fast/text/international/unicode-bidi-plaintext-expected.txt:
9:08 AM Changeset in webkit [89935] by jberlin@webkit.org
  • 2 edits in trunk/LayoutTests

WebKitTestRunner needs to implement undo client (Windows).
​https://bugs.webkit.org/show_bug.cgi?id=58103

Add another undo test to the win-wk2 skipped list to get the bots green.

  • platform/win-wk2/Skipped:
9:01 AM Changeset in webkit [89934] by senorblanco@chromium.org
  • 2 edits in trunk/LayoutTests

2011-06-28 Stephen White <​senorblanco@chromium.org>

Unreviewed.

Remove some now-passing tests from Chromium's test expectations.

  • platform/chromium/test_expectations.txt:
8:56 AM Changeset in webkit [89933] by mitz@apple.com
  • 2 edits in trunk/Source/WebCore

REGRESSION (r89733): fast/text/zero-font-size.html is failing on Leopard
​https://bugs.webkit.org/show_bug.cgi?id=63512

Reviewed by Darin Adler.

  • platform/graphics/mac/SimpleFontDataMac.mm:

(WebCore::SimpleFontData::platformWidthForGlyph): Initialize the advance local variable, because
on Leopard wkGetGlyphTransformedAdvances can return successfully yet not update the variable when
the font size is 0. Presumably, r89733 exposed this by changing the values put on the stack just
before this function is called.

8:42 AM Changeset in webkit [89932] by bfulgham@webkit.org
  • 2 edits in trunk/Source/WebKit/win

2011-06-27 Brent Fulgham <​bfulgham@webkit.org>

Reviewed by Adam Roben.

Avoid attempt to create a zero-size DIB.
​https://bugs.webkit.org/show_bug.cgi?id=63484.

  • WebNodeHighlight.cpp: (WebNodeHighlight::update): Check for zero-height (or zero-width) region for DIB, and exit early if attempting to create a zero-size DIB/BITMAP.
8:24 AM Changeset in webkit [89931] by caseq@chromium.org
  • 4 edits
    1 delete in trunk

2011-06-24 Vsevolod Vlasov <​vsevik@chromium.org>

Reviewed by Pavel Feldman.

REGRESSION (r89449): http/tests/inspector/extensions-resources-redirect.html failing on SnowLeopard Intel Release (Tests), Windows 7 Release (Tests), Qt
​https://bugs.webkit.org/show_bug.cgi?id=63178

  • http/tests/inspector/extensions-resources-redirect-expected.txt:
  • platform/mac/http/tests/inspector/extensions-resources-redirect-expected.txt: Removed.
  • platform/qt/Skipped:
8:15 AM Changeset in webkit [89930] by rniwa@webkit.org
  • 3 edits in trunk/LayoutTests

2011-06-28 Ryosuke Niwa <​rniwa@webkit.org>

Fix the test added by the previous commit. Also add it to the WebKit2's skipped list
since it uses a eventSender method that hasn't been implemented by TestRunner.

  • fast/events/selectstart-prevent-selection-on-right-click.html:
  • platform/mac-wk2/Skipped:
8:09 AM Changeset in webkit [89929] by rniwa@webkit.org
  • 1 edit
    2 adds in trunk/LayoutTests

2011-06-28 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Darin Adler.

Add a test for canceling selectstart when a word is selected by right click
​https://bugs.webkit.org/show_bug.cgi?id=63530

Add a regression now that the bug no longer reproduces.

  • fast/events/selectstart-prevent-selection-on-right-click-expected.txt: Added.
  • fast/events/selectstart-prevent-selection-on-right-click.html: Added.
7:51 AM Changeset in webkit [89928] by senorblanco@chromium.org
  • 2 edits in trunk/LayoutTests

2011-06-28 Stephen White <​senorblanco@chromium.org>

Unreviewed.

Chromium test_expectations update.

  • platform/chromium/test_expectations.txt:
7:39 AM Changeset in webkit [89927] by Adam Roben
  • 2 edits in trunk/Source/JavaScriptCore

Windows Debug build fix after r89885

JSGlobalData::releaseExecutableMemory for jsc.exe's benefit.

7:39 AM Changeset in webkit [89926] by Adam Roben
  • 4 edits in trunk/Tools

Roll out r89924 and r89925

I didn't mean to commit them!

  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/LayoutTestHistoryAnalyzer.js:
  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/Utilities.js:
  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ViewController.js:
7:34 AM Changeset in webkit [89925] by Adam Roben
  • 2 edits in trunk/Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures

Keep track of all builds

7:33 AM Changeset in webkit [89924] by Adam Roben
  • 3 edits in trunk/Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures

Start categorizing flaky tests

7:22 AM Changeset in webkit [89923] by alexis.menard@openbossa.org
  • 4 edits in trunk/Source/WebKit/qt

2011-06-28 Alexis Menard <​alexis.menard@openbossa.org>

Reviewed by Andreas Kling.

[Qt] Use QtQuick imports in Api tests rather than old deprecated Qt 4.x imports.
​https://bugs.webkit.org/show_bug.cgi?id=63533

We need to use the new QtQuick 1.x imports rather than the old deprecated
version.

  • tests/qdeclarativewebview/resources/webviewbackgroundcolor.qml:
  • tests/qdeclarativewebview/resources/webviewtest.qml:
  • tests/qdeclarativewebview/resources/webviewtestdefault.qml:
5:07 AM Changeset in webkit [89922] by apavlov@chromium.org
  • 11 edits in trunk

2011-06-27 Alexander Pavlov <​apavlov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: Change the inspector model API and backend to allow CSS pseudoclass inspection
​https://bugs.webkit.org/show_bug.cgi?id=63446

  • inspector/styles/get-set-stylesheet-text.html:
  • inspector/styles/styles-new-API.html:
  • inspector/styles/styles-source-offsets.html:

2011-06-27 Alexander Pavlov <​apavlov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: Change the inspector model API and backend to allow CSS pseudoclass inspection
​https://bugs.webkit.org/show_bug.cgi?id=63446

  • inspector/Inspector.json:
  • inspector/InspectorCSSAgent.cpp: (WebCore::computePseudoClassMask): (WebCore::InspectorCSSAgent::getStylesForNode):
  • inspector/InspectorCSSAgent.h:
  • inspector/front-end/AuditRules.js: (WebInspector.AuditRules.ImageDimensionsRule.prototype.doRun.getStyles):
  • inspector/front-end/CSSStyleModel.js: (WebInspector.CSSStyleModel.prototype.getStylesAsync):
  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylesSidebarPane.prototype.update):
4:42 AM Changeset in webkit [89921] by kbalazs@webkit.org
  • 4 edits in trunk

2011-06-28 Balazs Kelemen <​kbalazs@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[X11][WK2] X error with plugin tests
​https://bugs.webkit.org/show_bug.cgi?id=63520

  • platform/qt-wk2/Skipped:

2011-06-28 Balazs Kelemen <​kbalazs@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[X11][WK2] X error with plugin tests
​https://bugs.webkit.org/show_bug.cgi?id=63520

  • WebProcess/Plugins/Netscape/x11/NetscapePluginX11.cpp: Never create a pixmap with (0,0) dimensions. (WebKit::NetscapePlugin::platformGeometryDidChange): (WebKit::NetscapePlugin::platformPaint):
4:38 AM Changeset in webkit [89920] by commit-queue@webkit.org
  • 5 edits in trunk/Source/WebKit/efl

2011-06-28 Michal Pakula vel Rutka <Michał Pakuła vel Rutka>

Reviewed by Kenneth Rohde Christiansen.

[EFL] Implement EditorClientEfl::respondToChangedSelection
​https://bugs.webkit.org/show_bug.cgi?id=63525

This patch:

  • implements EditorClientEfl::respondToChangedSelection,
  • adds ewk_frame_editor_client_selection_changed and
  • ewk_view_editor_client_selection_changed
  • WebCoreSupport/EditorClientEfl.cpp: (WebCore::EditorClientEfl::respondToChangedSelection):
  • ewk/ewk_frame.cpp: (ewk_frame_editor_client_selection_changed):
  • ewk/ewk_private.h:
  • ewk/ewk_view.cpp: (ewk_view_editor_client_selection_changed):
2:59 AM Changeset in webkit [89919] by tkent@chromium.org
  • 5 edits
    2 adds in trunk/LayoutTests

2011-06-28 Kent Tamura <​tkent@chromium.org>

Reviewed by Hajime Morita.

Verify cloning elements with validation messages works.
​https://bugs.webkit.org/show_bug.cgi?id=61988

  • fast/forms/validation-message-clone-expected.txt: Added.
  • fast/forms/validation-message-clone.html: Added.
  • platform/gtk/Skipped:
  • platform/mac-wk2/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:
2:54 AM Changeset in webkit [89918] by commit-queue@webkit.org
  • 1 edit
    1 move
    1 add
    11 deletes in trunk/LayoutTests

2011-06-28 Wyatt Carss <​wcarss@chromium.org>

Reviewed by Hajime Morita.

convert editing/deleting/4845371.html to runDumpAsTextEditingTest
​https://bugs.webkit.org/show_bug.cgi?id=63214

Renamed editing/deleting/4845371.html to editing/deleting/delete-cell-contents.html
and converted editing/deleting/delete-cell-contents.html to runDumpAsTextEditingTest; left
it without a doctype, etc, because they changed functionality of the image test. Also
added the rdar number.

  • editing/deleting/delete-cell-contents-expected.txt: Added.
  • editing/deleting/delete-cell-contents.html: Added.
  • editing/deleting/4845371.html: Removed.
  • platform/chromium-linux/editing/deleting/4845371-expected.png: Removed.
  • platform/chromium-mac-leopard/editing/deleting/4845371-expected.png: Removed.
  • platform/chromium-win/editing/deleting/4845371-expected.png: Removed.
  • platform/chromium-win/editing/deleting/4845371-expected.txt: Removed.
  • platform/gtk/editing/deleting/4845371-expected.png: Removed.
  • platform/gtk/editing/deleting/4845371-expected.txt: Removed.
  • platform/mac-leopard/editing/deleting/4845371-expected.png: Removed.
  • platform/mac/editing/deleting/4845371-expected.png: Removed.
  • platform/mac/editing/deleting/4845371-expected.txt: Removed.
  • platform/qt/editing/deleting/4845371-expected.png: Removed.
  • platform/qt/editing/deleting/4845371-expected.txt: Removed.
2:53 AM Changeset in webkit [89917] by tkent@chromium.org
  • 2 edits in trunk/Source/WebCore

2011-06-28 Kent Tamura <​tkent@chromium.org>

Fix a build error on Mac by r89907.

  • svg/animation/SVGSMILElement.cpp: (WebCore::SVGSMILElement::endListChanged):
2:52 AM Changeset in webkit [89916] by apavlov@chromium.org
  • 3 edits in trunk/Source/WebCore

2011-06-27 Alexander Pavlov <​apavlov@chromium.org>

Reviewed by Kent Tamura.

[Chromium] SELECT or autofill popup is trimmed by screen edge on Windows
​https://bugs.webkit.org/show_bug.cgi?id=63438

If the popup is calculated to be trimmed by a screen edge, an attempt is made
to alter its vertical edge alignment (set to the right edge for LTR or to the left edge for RTL)
to see if the trimmed portion becomes smaller than that with the original layout.
The change involves remembering the original frameRect for the popup and restoring it in refresh().
This is due to the fact that the frameRect originally set in showInRect() is overwritten
by layoutAndGetRTLOffset(), which breaks the originally requested popup container layout metrics.
The max width is reset on every layoutAndCalculateWidgetRect(), as it can be constrained by the screen edge,
and thus should be re-checked every time the popup is displayed, in case the browser window has been moved.

Tests: manual-tests/popup-width-restriction-within-screen.html partly covers the fix (should not regress).
Other than that, there is no way to unit-test the platform-specific native code.

  • platform/chromium/PopupMenuChromium.cpp: (WebCore::PopupListBox::setMaxWidth): Added. Avoid duplicate popup content layouts (in contrast with setMaxWidthAndLayout()). (WebCore::PopupListBox::PopupListBox): (WebCore::PopupContainer::layoutAndCalculateWidgetRect): Attempt a left-right popup alignment inversion to minimize the trimmed content. Also restore a default max popup width. (WebCore::PopupContainer::layoutAndGetRTLOffset): Always return the popup listbox offset for the RTL (right alignment) case. The method rename is due to the return value semantics change. (WebCore::PopupContainer::showInRect): Store the originally requested frameRect for the popup. (WebCore::PopupContainer::refresh): Restore the original popup frameRect to avoid layout artifacts on refresh. (WebCore::PopupContainer::isRTL): This check should be made by the layoutAndGetRTLOffset() clients.
  • platform/chromium/PopupMenuChromium.h:
2:46 AM Changeset in webkit [89915] by commit-queue@webkit.org
  • 8 edits in trunk

2011-06-28 Kentaro Hara <​haraken@google.com>

Reviewed by Kent Tamura.

Allow spaces between e-mail addresses in 'email' input type.
​https://bugs.webkit.org/show_bug.cgi?id=55987

Allowed spaces between e-mail addresses in 'email' input type,
e.g. ' ​a@p.com , ​b@p.com ' is a valid value. Implemented the
value sanitization algorithm for an e-mail value. Added more unit
tests that check the e-mail value parser.

  • fast/forms/ValidityState-typeMismatch-email-expected.txt:
  • fast/forms/resources/ValidityState-typeMismatch-email.js:

2011-06-28 Kentaro Hara <​haraken@google.com>

Reviewed by Kent Tamura.

Allow spaces between e-mail addresses in 'email' input type.
​https://bugs.webkit.org/show_bug.cgi?id=55987

Allowed spaces between e-mail addresses in 'email' input type,
e.g. ' ​a@p.com , ​b@p.com ' is a valid value. Implemented the
value sanitization algorithm for an e-mail value. Added more unit
tests that check the e-mail value parser.

Test: fast/forms/ValidityState-typeMismatch-email.html

  • html/EmailInputType.cpp: (WebCore::EmailInputType::typeMismatchFor): (WebCore::EmailInputType::sanitizeValue):
  • html/EmailInputType.h:
  • html/parser/HTMLParserIdioms.cpp: (WebCore::stripHTMLLineBreaks):
  • html/parser/HTMLParserIdioms.h: (WebCore::isHTMLLineBreak):
2:44 AM Changeset in webkit [89914] by Csaba Osztrogonác
  • 3 edits in trunk/LayoutTests

Qt specific results committed in r89901, but some of them fail on Qt-WK2 and/or Qt-Mac platform

  • platform/qt-mac/Skipped: Skip failing tests and typo fix.
  • platform/qt-wk2/Skipped: Skip failing tests.
2:30 AM Changeset in webkit [89913] by tkent@chromium.org
  • 3 edits in trunk/Source/WebCore

2011-06-28 Kent Tamura <​tkent@chromium.org>

Reviewed by Hajime Morita.

Refactor RenderTextControl::text().
​https://bugs.webkit.org/show_bug.cgi?id=63519

No new tests because this is just a refactoring.

  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::setInnerTextValue): Resuce the number of text() calls. (WebCore::finishText):
    • Use StringBuilder.
    • Make this a static function. This function doesn't access anything of RenderTextControl.

(WebCore::RenderTextControl::text):

Use emptyString() and StringBuilder.

(WebCore::RenderTextControl::textWithHardLineBreaks): ditto.

  • rendering/RenderTextControl.h: Remove finishText().
2:28 AM Changeset in webkit [89912] by yurys@chromium.org
  • 5 edits in trunk

2011-06-28 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: console.group messages should never be coalesced
​https://bugs.webkit.org/show_bug.cgi?id=63521

  • inspector/console/console-nested-group-expected.txt:
  • inspector/console/console-nested-group.html:

2011-06-28 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: console.group messages should never be coalesced
​https://bugs.webkit.org/show_bug.cgi?id=63521

  • inspector/InspectorConsoleAgent.cpp: (WebCore::isGroupMessage): (WebCore::InspectorConsoleAgent::addConsoleMessage):
2:26 AM Changeset in webkit [89911] by yurys@chromium.org
  • 3 edits in trunk/Source/WebCore

2011-06-28 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: console group boundaries should be more distinct in UI
​https://bugs.webkit.org/show_bug.cgi?id=63455

Show grey bracket to on the left side of a console group message to highlight
the group extent.

  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleGroup):
  • inspector/front-end/inspector.css: (#console-prompt): (.console-message, .console-user-command): (.console-message::before, .console-user-command::before, #console-prompt::before, .console-group-title::before): (.console-group-title::before): (.console-group): (.console-group-bracket): (.console-group.collapsed > .console-group-bracket):
2:20 AM Changeset in webkit [89910] by tkent@chromium.org
  • 5 edits in trunk/Source/WebCore

2011-06-28 Kent Tamura <​tkent@chromium.org>

Reviewed by Hajime Morita.

Use RefPtr for the HTMLElement data members of TextFieldInptType
and SearchInputType.
​https://bugs.webkit.org/show_bug.cgi?id=63511

No new tests because of no behavior change.

  • html/SearchInputType.cpp: (WebCore::SearchInputType::SearchInputType): No need to initialize the element members explicitly because RefPtr constructor works well. (WebCore::SearchInputType::createShadowSubtree): Substitute an element to a data member, and don't call RefPtr<>::release(). (WebCore::SearchInputType::innerBlockElement): Moved from the header file, and adjustment for RefPtr<>. (WebCore::SearchInputType::resultsButtonElement): ditto. (WebCore::SearchInputType::cancelButtonElement): ditto. (WebCore::SearchInputType::destroyShadowSubtree): Adjustment for RefPtr<>.
  • html/SearchInputType.h:
    • Move accessor functions to the cpp file because they are virtual.
    • Change element data member types: HTMLElement* -> RefPtr<HTMLElement>.
  • html/TextFieldInputType.cpp: (WebCore::TextFieldInputType::TextFieldInputType): No need to initialize the element members explicitly because RefPtr constructor works well. (WebCore::TextFieldInputType::~TextFieldInputType): Explicityly defines the destructor in the cpp file to avoid to make it implicitly in the header file. If the destructor was made in the header file, we would need to include HTMLElement.h in the header file. (WebCore::TextFieldInputType::createShadowSubtree):
  • Don't call RenderTheme::themeForPage().
  • Substitute an element to a data member, and don't call RefPtr<>::release(). (WebCore::TextFieldInputType::setInnerTextElement): Moved from the header file to avoid to include HTMLElement.h. (WebCore::TextFieldInputType::setSpeechButtonElement): ditto. (WebCore::TextFieldInputType::innerTextElement): Moved from the header file, and adjustment for RefPtr<>. (WebCore::TextFieldInputType::innerSpinButtonElement): ditto. (WebCore::TextFieldInputType::speechButtonElement): ditto. (WebCore::TextFieldInputType::destroyShadowSubtree): Adjustment for RefPtr<>.
  • html/TextFieldInputType.h:
    • Move accessor functions to the cpp file because they are virtual.
    • Change element data member types: HTMLElement* -> RefPtr<HTMLElement>.
2:19 AM Changeset in webkit [89909] by pfeldman@chromium.org
  • 2 edits in trunk/LayoutTests

2011-06-28 Pavel Feldman <​pfeldman@google.com>

Not reviewed: added chromium test suppression.

  • platform/chromium/test_expectations.txt:
2:12 AM Changeset in webkit [89908] by kbalazs@webkit.org
  • 4 edits in trunk

2011-06-28 Balazs Kelemen <​kbalazs@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[X11][WK2] plugins/plugin-javascript-access fails
​https://bugs.webkit.org/show_bug.cgi?id=63464

  • platform/qt-wk2/Skipped:

2011-06-28 Balazs Kelemen <​kbalazs@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[X11][WK2] plugins/plugin-javascript-access fails
​https://bugs.webkit.org/show_bug.cgi?id=63464

  • Shared/Plugins/Netscape/x11/NetscapePluginModuleX11.cpp: (WebKit::NetscapePluginModule::getPluginInfo): Determine the name of the plugin.
1:58 AM Changeset in webkit [89907] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebCore

2011-06-28 Felician Marton <​marton.felician.zoltan@stud.u-szeged.hu>

Reviewed by Nikolas Zimmermann.

SVG animation flickers when restarting
​https://bugs.webkit.org/show_bug.cgi?id=18450

Add new parameter to the following methods. The new parameter is the time of adding a "begin" or "end" time to the SVG animation.

  • WebCore::SVGSMILElement::addBeginTime
  • WebCore::SVGSMILElement::beginListChanged
  • WebCore::SVGSMILElement::addEndTime
  • WebCore::SVGSMILElement::endListChanged

In case of beginTime and beginListChanged it's necessary, because the delay between the event of adding a new begin time
(in WebCore::SVGAnimationElement::beginElementAt) and the actual processing (in WebCore::SVGSMILElement::endListChanged)
is significant. In lack of event time we can't decide that we should consider the new time or not.
If the new begin time is smaller than the event time, we musn't begin the animation, else we shoud do further calculations.

The following methods just modified for consistency:

  • WebCore::SVGSMILElement::addEndTime
  • WebCore::SVGSMILElement::endListChanged

No new tests, because currently there is no way to test SVG flickering at zero time.

  • svg/SVGAnimationElement.cpp: (WebCore::SVGAnimationElement::beginElementAt): (WebCore::SVGAnimationElement::endElementAt):
  • svg/animation/SVGSMILElement.cpp: (WebCore::SVGSMILElement::attributeChanged): (WebCore::SVGSMILElement::addBeginTime): (WebCore::SVGSMILElement::addEndTime): (WebCore::SVGSMILElement::beginListChanged): (WebCore::SVGSMILElement::endListChanged): The body changed:
    • Use new parameter, the time of adding the new begin time.
    • Cancel some incorrect optimalization. We should also do further calculations when elapsed >= m_intervalBegin.

(WebCore::SVGSMILElement::createInstanceTimesFromSyncbase):
(WebCore::SVGSMILElement::handleConditionEvent):
(WebCore::SVGSMILElement::beginByLinkActivation):

  • svg/animation/SVGSMILElement.h:
1:53 AM Changeset in webkit [89906] by commit-queue@webkit.org
  • 5 edits in trunk/Source/JavaScriptCore

2011-06-28 Shinya Kawanaka <​shinyak@google.com>

Reviewed by Kent Tamura.

Add const to show() method in WTFString and AtomicString.
​https://bugs.webkit.org/show_bug.cgi?id=63515

The lack of const in show() method is painful when
doing something like printf-debug.

  • wtf/text/AtomicString.cpp: (WTF::AtomicString::show):
  • wtf/text/AtomicString.h:
  • wtf/text/WTFString.cpp: (String::show):
  • wtf/text/WTFString.h:
1:48 AM Changeset in webkit [89905] by gyuyoung.kim@samsung.com
  • 4 edits in trunk/Source/WebKit/efl

2011-06-28 Gyuyoung Kim <​gyuyoung.kim@samsung.com>

Reviewed by Kent Tamura.

[EFL] Change permission of some ewk files.
​https://bugs.webkit.org/show_bug.cgi?id=63517

Change some file's permission same with other file's.

  • ewk/ewk_tiled_backing_store.c:
  • ewk/ewk_tiled_model.c:
  • ewk/ewk_view_single.c:
1:46 AM Changeset in webkit [89904] by Csaba Osztrogonác
  • 2 edits in trunk/Source/WebCore

Adding MHTML reading support.
​https://bugs.webkit.org/show_bug.cgi?id=7168

Reviewed by Andreas Kling.

  • features.pri: Disable MHTML on Qt by default in a proper way.
1:46 AM Changeset in webkit [89903] by Csaba Osztrogonác
  • 2 edits in trunk/Source/WebCore

[Qt] Remove ENABLE_XBL, because there aren't ENABLE(XBL) macros anymore.

Reviewed by Andreas Kling.

  • features.pri:
1:02 AM Changeset in webkit [89902] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt] http/tests/misc/link-rel-icon-beforeload.html is flakey
​https://bugs.webkit.org/show_bug.cgi?id=63518

  • platform/qt/Skipped: Add http/tests/misc/link-rel-icon-beforeload.html until fix.
12:55 AM Changeset in webkit [89901] by Csaba Osztrogonác
  • 2 edits
    30 adds in trunk/LayoutTests

[Qt] Unreviewed daily gardening. Add platform specific expected files
for new passing tests and add failing new tests to the Skipped list.

  • platform/qt/Skipped:
  • platform/qt/editing/selection/extend-inside-transforms-backward-expected.png: Added.
  • platform/qt/editing/selection/extend-inside-transforms-backward-expected.txt: Added.
  • platform/qt/editing/selection/extend-inside-transforms-forward-expected.png: Added.
  • platform/qt/editing/selection/extend-inside-transforms-forward-expected.txt: Added.
  • platform/qt/fast/borders/only-one-border-with-width-expected.png: Added.
  • platform/qt/fast/borders/only-one-border-with-width-expected.txt: Added.
  • platform/qt/fast/css/color-leakage-expected.png: Added.
  • platform/qt/fast/css/color-leakage-expected.txt: Added.
  • platform/qt/fast/dom/HTMLProgressElement/indeterminate-progress-001-expected.png: Added.
  • platform/qt/fast/dom/HTMLProgressElement/indeterminate-progress-001-expected.txt: Added.
  • platform/qt/fast/inline/inline-wrap-with-parent-padding-expected.png: Added.
  • platform/qt/fast/inline/inline-wrap-with-parent-padding-expected.txt: Added.
  • platform/qt/fast/multicol/span/span-as-nested-inline-block-child-expected.png: Added.
  • platform/qt/fast/multicol/span/span-as-nested-inline-block-child-expected.txt: Added.
  • platform/qt/http/tests/misc/object-embedding-svg-delayed-size-negotiation-2-expected.png: Added.
  • platform/qt/http/tests/misc/object-embedding-svg-delayed-size-negotiation-2-expected.txt: Added.
  • platform/qt/svg/W3C-SVG-1.1-SE/interact-pointer-03-t-expected.png: Added.
  • platform/qt/svg/W3C-SVG-1.1-SE/interact-pointer-03-t-expected.txt: Added.
  • platform/qt/svg/W3C-SVG-1.1-SE/linking-uri-01-b-expected.png: Added.
  • platform/qt/svg/W3C-SVG-1.1-SE/linking-uri-01-b-expected.txt: Added.
  • platform/qt/svg/W3C-SVG-1.1-SE/styling-css-04-f-expected.png: Added.
  • platform/qt/svg/W3C-SVG-1.1-SE/styling-css-04-f-expected.txt: Added.
  • platform/qt/svg/custom/invalid-dasharray-expected.png: Added.
  • platform/qt/svg/custom/invalid-dasharray-expected.txt: Added.
  • platform/qt/svg/custom/linking-uri-01-b-expected.png: Added.
  • platform/qt/svg/custom/linking-uri-01-b-expected.txt: Added.
  • platform/qt/svg/text/text-overflow-ellipsis-svgfont-expected.png: Added.
  • platform/qt/svg/text/text-overflow-ellipsis-svgfont-expected.txt: Added.
  • platform/qt/svg/zoom/page/zoom-replaced-intrinsic-ratio-001-expected.png: Added.
  • platform/qt/svg/zoom/page/zoom-replaced-intrinsic-ratio-001-expected.txt: Added.
12:35 AM Changeset in webkit [89900] by kbalazs@webkit.org
  • 4 edits in trunk/LayoutTests

2011-06-28 Balazs Kelemen <​kbalazs@webkit.org>

Reviewed by Csaba Osztrogonác.

WebKitTestRunner needs layoutTestController.setCallCloseOnWebViews
​https://bugs.webkit.org/show_bug.cgi?id=46714

Skip these tests via the mac list to make them skipped on every wk2 platform.

  • platform/mac-wk2/Skipped:
  • platform/win-wk2/Skipped:
12:30 AM Changeset in webkit [89899] by abarth@webkit.org
  • 9 edits in trunk/Tools

2011-06-27 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

NRWT should wait for ReportCrash
​https://bugs.webkit.org/show_bug.cgi?id=63495

When a test crashes on Mac, ReportCrash confuses us into thinking the
test times out.

I'm not sure how to test this in a meaningful way in our current test
harness...

  • Scripts/webkitpy/common/system/crashlogs.py:
  • Scripts/webkitpy/common/system/crashlogs_unittest.py:
  • Scripts/webkitpy/layout_tests/layout_package/test_result_writer.py:
  • Scripts/webkitpy/layout_tests/port/base.py:
  • Scripts/webkitpy/layout_tests/port/mac.py:
  • Scripts/webkitpy/layout_tests/port/server_process.py:
  • Scripts/webkitpy/tool/commands/queries.py:
12:25 AM Changeset in webkit [89898] by cevans@google.com
  • 2 edits
    2 copies in branches/chromium/782

Merge 89595
BUG=87227
Review URL: ​http://codereview.chromium.org/7274018

Note: See TracTimeline for information about the timeline view.