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

Timeline



Jan 25, 2010:

11:36 PM Changeset in webkit [53841] by Chris Jerdonek
  • 7 edits in trunk/WebKitTools

2010-01-25 Chris Jerdonek <Chris Jerdonek>

Reviewed by Shinichiro Hamaji.

Refactored check-webkit-style by removing the file path
parameter from the style error handler functions.

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

  • Scripts/webkitpy/style/checker.py:
    • Added _default_style_error_handler() to StyleChecker class.
    • Moved handle_style_error() to inside _default_style_error_handler().
  • Scripts/webkitpy/style/checker_unittest.py:
    • Removed file path from calls to error handler.
  • Scripts/webkitpy/style/cpp_style.py:
    • Removed file path from calls to error handler.
  • Scripts/webkitpy/style/cpp_style_unittest.py:
    • Removed file path from calls to error handler.
  • Scripts/webkitpy/style/text_style.py:
    • Removed file path from calls to error handler.
  • Scripts/webkitpy/style/text_style_unittest.py:
    • Removed file path from calls to error handler.
10:23 PM Changeset in webkit [53840] by jorlow@chromium.org
  • 17 edits
    1 copy
    21 adds
    23 deletes in trunk

2010-01-22 Jeremy Orlow <​jorlow@chromium.org>

Reviewed by Darin Adler.

Make storage events match the spec.
​https://bugs.webkit.org/show_bug.cgi?id=30546

Update the storageEvent algorithm to match the change in WebCore.

  • src/StorageAreaProxy.cpp: (WebCore::StorageAreaProxy::storageEvent):

2010-01-22 Jeremy Orlow <​jorlow@chromium.org>

Reviewed by Darin Adler.

Make storage events match the spec.
​https://bugs.webkit.org/show_bug.cgi?id=30546

This meat of the patch I just posted is very simple. It's just making events
asynchronous, not posting them to the frame that generated them, passing a null
for the key when issuing a clear storage event, and making the events
non-cancelable/non-bubbleable...all of which are clearly stated in the spec.

The asynchronous and not posting to the frame that generated them forced me to
re-write all the layout tests that dealt with storage events. There's a lot of
code there, but I tried to be fairly careful to ensure that test coverage did
not shrink in any area.

Tests: storage/domstorage/events/basic-body-attribute.html

storage/domstorage/events/basic-setattribute.html
storage/domstorage/events/basic.html
storage/domstorage/events/case-sensitive.html
storage/domstorage/events/documentURI.html

  • dom/Document.cpp: (WebCore::Document::Document): (WebCore::Document::enqueueStorageEvent): (WebCore::Document::storageEventTimerFired):
  • dom/Document.h:
  • storage/StorageEvent.cpp: (WebCore::StorageEvent::StorageEvent):
  • storage/StorageEvent.idl:
  • storage/StorageEventDispatcher.cpp: (WebCore::StorageEventDispatcher::dispatch):

2010-01-22 Jeremy Orlow <​jorlow@chromium.org>

Reviewed by Darin Adler.

Make storage events match the spec.
​https://bugs.webkit.org/show_bug.cgi?id=30546

This meat of the patch I just posted is very simple. It's just making events
asynchronous, not posting them to the frame that generated them, passing a null
for the key when issuing a clear storage event, and making the events
non-cancelable/non-bubbleable...all of which are clearly stated in the spec.

The asynchronous and not posting to the frame that generated them forced me to
re-write all the layout tests that dealt with storage events. There's a lot of
code there, but I tried to be fairly careful to ensure that test coverage did
not shrink in any area.

  • storage/domstorage/complex-values-expected.txt:
  • storage/domstorage/documentURI-expected.txt: Removed.
  • storage/domstorage/documentURI.html: Removed.
  • storage/domstorage/events: Added.
  • storage/domstorage/events/basic-body-attribute-expected.txt: Added.
  • storage/domstorage/events/basic-body-attribute.html: Added.
  • storage/domstorage/events/basic-expected.txt: Added.
  • storage/domstorage/events/basic-setattribute-expected.txt: Added.
  • storage/domstorage/events/basic-setattribute.html: Added.
  • storage/domstorage/events/basic.html: Added.
  • storage/domstorage/events/case-sensitive-expected.txt: Added.
  • storage/domstorage/events/case-sensitive.html: Added.
  • storage/domstorage/events/documentURI-expected.txt: Added.
  • storage/domstorage/events/documentURI.html: Added.
  • storage/domstorage/events/resources: Added.
  • storage/domstorage/events/resources/body-event-handler.html: Added.
  • storage/domstorage/events/resources/eventTestHarness.js: Added.
  • storage/domstorage/events/resources/setattribute-event-handler.html: Added.
  • storage/domstorage/events/script-tests: Added.
  • storage/domstorage/events/script-tests/TEMPLATE.html: Copied from LayoutTests/storage/domstorage/script-tests/TEMPLATE.html.
  • storage/domstorage/events/script-tests/basic-body-attribute.js: Added.
  • storage/domstorage/events/script-tests/basic-setattribute.js: Added.
  • storage/domstorage/events/script-tests/basic.js: Added.
  • storage/domstorage/events/script-tests/case-sensitive.js: Added.
  • storage/domstorage/events/script-tests/documentURI.js: Added.
  • storage/domstorage/localstorage/iframe-events-expected.txt: Removed.
  • storage/domstorage/localstorage/iframe-events.html: Removed.
  • storage/domstorage/localstorage/index-get-and-set-expected.txt:
  • storage/domstorage/localstorage/index-get-and-set.html:
  • storage/domstorage/localstorage/onstorage-attribute-markup-expected.txt: Removed.
  • storage/domstorage/localstorage/onstorage-attribute-markup.html: Removed.
  • storage/domstorage/localstorage/onstorage-attribute-setattribute-expected.txt: Removed.
  • storage/domstorage/localstorage/onstorage-attribute-setattribute.html: Removed.
  • storage/domstorage/localstorage/onstorage-attribute-setwindow-expected.txt: Removed.
  • storage/domstorage/localstorage/onstorage-attribute-setwindow.html: Removed.
  • storage/domstorage/localstorage/simple-events-expected.txt: Removed.
  • storage/domstorage/localstorage/simple-events.html: Removed.
  • storage/domstorage/script-tests/complex-values.js:
  • storage/domstorage/script-tests/documentURI.js: Removed.
  • storage/domstorage/sessionstorage/iframe-events-expected.txt: Removed.
  • storage/domstorage/sessionstorage/iframe-events.html: Removed.
  • storage/domstorage/sessionstorage/index-get-and-set-expected.txt:
  • storage/domstorage/sessionstorage/index-get-and-set.html:
  • storage/domstorage/sessionstorage/onstorage-attribute-markup-expected.txt: Removed.
  • storage/domstorage/sessionstorage/onstorage-attribute-markup.html: Removed.
  • storage/domstorage/sessionstorage/onstorage-attribute-setattribute-expected.txt: Removed.
  • storage/domstorage/sessionstorage/onstorage-attribute-setattribute.html: Removed.
  • storage/domstorage/sessionstorage/onstorage-attribute-setwindow-expected.txt: Removed.
  • storage/domstorage/sessionstorage/onstorage-attribute-setwindow.html: Removed.
  • storage/domstorage/sessionstorage/simple-events-expected.txt: Removed.
  • storage/domstorage/sessionstorage/simple-events.html: Removed.
  • storage/domstorage/window-attributes-exist-expected.txt:
  • storage/domstorage/window-attributes-exist.html:
10:00 PM Changeset in webkit [53839] by mitz@apple.com
  • 6 edits
    3 adds in trunk

<rdar://problem/7573493> Error with line break inside ?&raquo; pair of characters.
​https://bugs.webkit.org/show_bug.cgi?id=17475

Reviewed by Darin Adler.

WebCore:

Test: fast/text/line-break-after-question-mark.html

Instead of unconditionally allowing lines to break after a question mark, which was intended
to mimic Internet Explorer, apply the Unicode line breaking behavior after a question mark,
with one exception in the ASCII range to match IE, namely not allowing a line break between
a question mark and a vertical line.

  • rendering/break_lines.cpp:

(WebCore::shouldBreakAfter): Added a next character parameter. Changed to consult a table
for the question mark case in order to keep it fast and not require a text break iterator in
the ASCII case.
(WebCore::nextBreakablePosition): Pass the next character to shouldBreakAfter.

LayoutTests:

  • fast/text/line-break-after-question-mark-expected.txt: Added.
  • fast/text/line-break-after-question-mark.html: Added.
  • fast/text/script-tests/line-break-after-question-mark.js: Added.

():

  • platform/mac/tables/mozilla/bugs/bug6674-expected.checksum: Updated.
  • platform/mac/tables/mozilla/bugs/bug6674-expected.png: Updated.
  • platform/mac/tables/mozilla/bugs/bug6674-expected.txt: Updated.
9:57 PM Changeset in webkit [53838] by pkasting@chromium.org
  • 92 edits in trunk

WebCore: Mac scrollbar thumbs were drawn with the wrong size.
​https://bugs.webkit.org/show_bug.cgi?id=34049

Reviewed by Dan Bernstein.

  • platform/chromium/ScrollbarThemeChromiumMac.mm:

(WebCore::ScrollbarThemeChromiumMac::paint):

  • platform/mac/ScrollbarThemeMac.mm:

(WebCore::ScrollbarThemeMac::paint):

LayoutTests: Rebaseline pixel tests affected by changes in scrollbar thumb size.
​https://bugs.webkit.org/show_bug.cgi?id=34049

Reviewed by Dan Bernstein.

  • platform/mac/compositing/overflow/overflow-scroll-expected.checksum:
  • platform/mac/compositing/overflow/overflow-scroll-expected.png:
  • platform/mac/fast/backgrounds/repeat/negative-offset-repeat-transformed-expected.checksum:
  • platform/mac/fast/backgrounds/repeat/negative-offset-repeat-transformed-expected.png:
  • platform/mac/fast/body-propagation/overflow/002-expected.checksum:
  • platform/mac/fast/body-propagation/overflow/002-expected.png:
  • platform/mac/fast/body-propagation/overflow/002-xhtml-expected.checksum:
  • platform/mac/fast/body-propagation/overflow/002-xhtml-expected.png:
  • platform/mac/fast/body-propagation/overflow/003-declarative-expected.checksum:
  • platform/mac/fast/body-propagation/overflow/003-declarative-expected.png:
  • platform/mac/fast/body-propagation/overflow/003-expected.checksum:
  • platform/mac/fast/body-propagation/overflow/003-expected.png:
  • platform/mac/fast/body-propagation/overflow/003-xhtml-expected.checksum:
  • platform/mac/fast/body-propagation/overflow/003-xhtml-expected.png:
  • platform/mac/fast/body-propagation/overflow/004-declarative-expected.checksum:
  • platform/mac/fast/body-propagation/overflow/004-declarative-expected.png:
  • platform/mac/fast/body-propagation/overflow/004-expected.checksum:
  • platform/mac/fast/body-propagation/overflow/004-expected.png:
  • platform/mac/fast/body-propagation/overflow/004-xhtml-expected.checksum:
  • platform/mac/fast/body-propagation/overflow/004-xhtml-expected.png:
  • platform/mac/fast/body-propagation/overflow/007-declarative-expected.checksum:
  • platform/mac/fast/body-propagation/overflow/007-declarative-expected.png:
  • platform/mac/fast/body-propagation/overflow/007-expected.checksum:
  • platform/mac/fast/body-propagation/overflow/007-expected.png:
  • platform/mac/fast/body-propagation/overflow/007-xhtml-expected.checksum:
  • platform/mac/fast/body-propagation/overflow/007-xhtml-expected.png:
  • platform/mac/fast/clip/014-expected.checksum:
  • platform/mac/fast/clip/014-expected.png:
  • platform/mac/fast/forms/basic-textareas-expected.checksum:
  • platform/mac/fast/forms/basic-textareas-expected.png:
  • platform/mac/fast/forms/form-element-geometry-expected.checksum:
  • platform/mac/fast/forms/form-element-geometry-expected.png:
  • platform/mac/fast/forms/listbox-hit-test-zoomed-expected.checksum:
  • platform/mac/fast/forms/listbox-hit-test-zoomed-expected.png:
  • platform/mac/fast/forms/textarea-scroll-height-expected.checksum:
  • platform/mac/fast/forms/textarea-scroll-height-expected.png:
  • platform/mac/fast/inline-block/tricky-baseline-expected.checksum:
  • platform/mac/fast/inline-block/tricky-baseline-expected.png:
  • platform/mac/fast/layers/scroll-rect-to-visible-expected.checksum:
  • platform/mac/fast/layers/scroll-rect-to-visible-expected.png:
  • platform/mac/fast/overflow/002-expected.checksum:
  • platform/mac/fast/overflow/002-expected.png:
  • platform/mac/fast/overflow/003-expected.checksum:
  • platform/mac/fast/overflow/003-expected.png:
  • platform/mac/fast/overflow/005-expected.checksum:
  • platform/mac/fast/overflow/005-expected.png:
  • platform/mac/fast/overflow/007-expected.checksum:
  • platform/mac/fast/overflow/007-expected.png:
  • platform/mac/fast/overflow/008-expected.checksum:
  • platform/mac/fast/overflow/008-expected.png:
  • platform/mac/fast/overflow/childFocusRingClip-expected.checksum:
  • platform/mac/fast/overflow/childFocusRingClip-expected.png:
  • platform/mac/fast/overflow/float-in-relpositioned-expected.checksum:
  • platform/mac/fast/overflow/float-in-relpositioned-expected.png:
  • platform/mac/fast/overflow/image-selection-highlight-expected.checksum:
  • platform/mac/fast/overflow/image-selection-highlight-expected.png:
  • platform/mac/fast/overflow/overflow-auto-position-absolute-expected.checksum:
  • platform/mac/fast/overflow/overflow-auto-position-absolute-expected.png:
  • platform/mac/fast/overflow/overflow-rtl-expected.checksum:
  • platform/mac/fast/overflow/overflow-rtl-expected.png:
  • platform/mac/fast/overflow/overflow-stacking-expected.checksum:
  • platform/mac/fast/overflow/overflow-stacking-expected.png:
  • platform/mac/fast/overflow/overflow-text-hit-testing-expected.checksum:
  • platform/mac/fast/overflow/overflow-text-hit-testing-expected.png:
  • platform/mac/fast/overflow/overflow-with-local-background-attachment-expected.checksum:
  • platform/mac/fast/overflow/overflow-with-local-background-attachment-expected.png:
  • platform/mac/fast/overflow/overflow-x-y-expected.checksum:
  • platform/mac/fast/overflow/overflow-x-y-expected.png:
  • platform/mac/fast/overflow/scroll-nested-positioned-layer-in-overflow-expected.checksum:
  • platform/mac/fast/overflow/scroll-nested-positioned-layer-in-overflow-expected.png:
  • platform/mac/fast/overflow/scrollbar-position-update-expected.checksum:
  • platform/mac/fast/overflow/scrollbar-position-update-expected.png:
  • platform/mac/fast/overflow/table-overflow-float-expected.checksum:
  • platform/mac/fast/overflow/table-overflow-float-expected.png:
  • platform/mac/fast/repaint/block-selection-gap-stale-cache-2-expected.checksum:
  • platform/mac/fast/repaint/block-selection-gap-stale-cache-2-expected.png:
  • platform/mac/fast/repaint/layout-state-only-positioned-expected.checksum:
  • platform/mac/fast/repaint/layout-state-only-positioned-expected.png:
  • platform/mac/fast/repaint/selection-gap-overflow-scroll-expected.checksum:
  • platform/mac/fast/repaint/selection-gap-overflow-scroll-expected.png:
  • platform/mac/fast/table/edge-offsets-expected.checksum:
  • platform/mac/fast/table/edge-offsets-expected.png:
  • platform/mac/scrollbars/key-window-not-first-responder-expected.checksum:
  • platform/mac/scrollbars/key-window-not-first-responder-expected.png:
  • platform/mac/svg/custom/invisible-text-after-scrolling-expected.checksum:
  • platform/mac/svg/custom/invisible-text-after-scrolling-expected.png:
  • platform/mac/tables/mozilla/bugs/bug149275-1-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug149275-1-expected.png:
9:38 PM WebKit Team edited by jorlow@chromium.org
(diff)
9:15 PM Changeset in webkit [53837] by jorlow@chromium.org
  • 2 edits in trunk/WebKitTools

Adding myself as reviewer. No review necessary.

Patch by Jeremy Orlow <​jorlow@chromium.org> on 2010-01-25

  • Scripts/webkitpy/committers.py:
6:17 PM Changeset in webkit [53836] by andersca@apple.com
  • 8 edits in trunk/WebKit/mac

Featureless build fixes.

  • Misc/WebCache.mm:

(+[WebCache empty]):

  • WebCoreSupport/WebApplicationCache.mm:
  • WebCoreSupport/WebChromeClient.h:
  • WebView/WebDataSource.mm:

(-[WebDataSource _transferApplicationCache:]):

  • WebView/WebFrame.mm:

(-[WebFrame _cacheabilityDictionary]):

  • WebView/WebView.mm:

(WebKitInitializeApplicationCachePathIfNecessary):
(-[WebView _close]):

  • WebView/WebViewData.mm:

(-[WebViewPrivate dealloc]):
(-[WebViewPrivate finalize]):

6:07 PM Changeset in webkit [53835] by enrica@apple.com
  • 7 edits
    3 adds in trunk

WebCore: Cleanup unwanted tags after pasting.
​https://bugs.webkit.org/show_bug.cgi?id=34148

Reviewed by Darin Adler.

Test: editing/pasteboard/paste-noscript-xhtml.xhtml

  • dom/Element.cpp:

(WebCore::Element::setAttributeNS):

  • dom/Element.h:
  • dom/XMLTokenizer.h:
  • dom/XMLTokenizerLibxml2.cpp:

(WebCore::XMLTokenizer::XMLTokenizer):
(WebCore::handleElementNamespaces):
(WebCore::handleElementAttributes):
(WebCore::XMLTokenizer::startElementNs):
(WebCore::XMLTokenizer::endElementNs):
(WebCore::parseXMLDocumentFragment):

  • html/HTMLElement.cpp:

(WebCore::HTMLElement::createContextualFragment):

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

Reviewed by Darin Adler.

  • editing/pasteboard/paste-noscript-xhtml-expected.txt: Added.
  • editing/pasteboard/paste-noscript-xhtml.xhtml: Added.
  • editing/resources/htmlcontent.html: Added.
5:24 PM Changeset in webkit [53834] by sfalken@apple.com
  • 9 edits in trunk

2010-01-25 Steve Falkenburg <​sfalken@apple.com>

Reviewed by Simon Fraser.

A WebGeolocationControllerClient is leaked for every WebView
​https://bugs.webkit.org/show_bug.cgi?id=34145

  • WebCoreSupport/WebGeolocationControllerClient.cpp: (WebGeolocationControllerClient::geolocationDestroyed): Added.
  • WebCoreSupport/WebGeolocationControllerClient.h:

2010-01-25 Steve Falkenburg <​sfalken@apple.com>

Reviewed by Simon Fraser.

A WebGeolocationControllerClient is leaked for every WebView
​https://bugs.webkit.org/show_bug.cgi?id=34145

  • WebCoreSupport/WebGeolocationControllerClient.h:
  • WebCoreSupport/WebGeolocationControllerClient.mm: (WebGeolocationControllerClient::geolocationDestroyed):

2010-01-25 Steve Falkenburg <​sfalken@apple.com>

Reviewed by Simon Fraser.

A WebGeolocationControllerClient is leaked for every WebView
​https://bugs.webkit.org/show_bug.cgi?id=34145

  • page/GeolocationController.cpp: (WebCore::GeolocationController::~GeolocationController):
  • page/GeolocationControllerClient.h:
5:07 PM Changeset in webkit [53833] by mrowe@apple.com
  • 3 edits
    2 adds in branches/safari-4-branch

Merge r52527.

5:03 PM Changeset in webkit [53832] by mrowe@apple.com
  • 1 edit in branches/safari-4-branch/WebCore/plugins/PluginView.cpp

Merge r53581.

5:02 PM Changeset in webkit [53831] by mrowe@apple.com
  • 1 edit in branches/safari-4-branch/WebCore/ChangeLog

Merge r53581.

4:44 PM Changeset in webkit [53830] by mrowe@apple.com
  • 5 edits
    1 add in branches/safari-4-branch

Merge r46792.

4:36 PM Changeset in webkit [53829] by mrowe@apple.com
  • 3 edits
    2 adds in branches/safari-4-branch

Merge r50698.

4:33 PM Changeset in webkit [53828] by mrowe@apple.com
  • 3 edits
    2 adds in branches/safari-4-branch

Merge r52073.

4:28 PM Changeset in webkit [53827] by mrowe@apple.com
  • 3 edits
    2 adds in branches/safari-4-branch

Merge r48810.

4:09 PM Changeset in webkit [53826] by mrowe@apple.com
  • 3 edits
    2 adds in branches/safari-4-branch

Merge r51877.

4:06 PM Changeset in webkit [53825] by mrowe@apple.com
  • 5 edits in branches/safari-4-branch/WebKit/mac

Merge r53520.

4:00 PM Changeset in webkit [53824] by mrowe@apple.com
  • 4 edits in branches/safari-4-branch/WebKit

Merge r53195.

3:55 PM Changeset in webkit [53823] by mrowe@apple.com
  • 3 edits
    2 adds in branches/safari-4-branch

Merge r51727.

3:53 PM Changeset in webkit [53822] by mrowe@apple.com
  • 2 edits in branches/safari-4-branch/JavaScriptCore

Merge r53089.

3:53 PM Changeset in webkit [53821] by mrowe@apple.com
  • 2 edits in branches/safari-4-branch/WebCore

Merge r51881.

3:52 PM Changeset in webkit [53820] by eric@webkit.org
  • 2 edits in trunk/LayoutTests

2010-01-25 Kent Tamura <​tkent@chromium.org>

Reviewed by Darin Adler.

Improve portability of select-item-background-clip.html
​https://bugs.webkit.org/show_bug.cgi?id=34072

  • fast/forms/select-item-background-clip.html: Calculate the pixel height at runtime.
3:49 PM Changeset in webkit [53819] by mrowe@apple.com
  • 2 edits in branches/safari-4-branch/WebKit/mac

Merge r51616.

3:47 PM Changeset in webkit [53818] by mrowe@apple.com
  • 2 edits
    1 add in branches/safari-4-branch/WebCore

Merge r51205.

3:26 PM Changeset in webkit [53817] by mrowe@apple.com
  • 13 edits in branches/safari-4-branch

Merge r52749.

3:20 PM Changeset in webkit [53816] by Dimitri Glazkov
  • 7 edits in trunk/LayoutTests

2010-01-25 Dimitri Glazkov <Dimitri Glazkov>

Unreviewed, trivial baseline updates.

-webkit-gradient slows down scrolling when page has horizontal scrollbar
​https://bugs.webkit.org/show_bug.cgi?id=19650

Update pixel baselines. They changed slightly after ​http://trac.webkit.org/changeset/53318/
landed.

  • platform/mac/fast/gradients/border-image-gradient-expected.checksum:
  • platform/mac/fast/gradients/border-image-gradient-expected.png:
  • platform/mac/fast/gradients/border-image-gradient-sides-and-corners-expected.checksum:
  • platform/mac/fast/gradients/border-image-gradient-sides-and-corners-expected.png:
  • platform/mac/fast/gradients/simple-gradients-expected.checksum:
  • platform/mac/fast/gradients/simple-gradients-expected.png:
3:12 PM Changeset in webkit [53815] by dimich@chromium.org
  • 2 edits in trunk/JavaScriptCore

Fix Chromium Linux tests: the pthread functions on Linux produce segfault if they receive 0 thread handle.
After r53714, we can have 0 thread handles passed to pthread_join and pthread_detach if corresponding threads
were already terminated and their threadMap entries cleared.
Add a 0 check.

Reviewed by David Levin.

  • wtf/ThreadingPthreads.cpp:

(WTF::waitForThreadCompletion):
(WTF::detachThread):

3:07 PM Changeset in webkit [53814] by cmarrin@apple.com
  • 2 edits in trunk/WebCore

Fixed bug in positioning of scroll layer within the
parent root layer.
​https://bugs.webkit.org/show_bug.cgi?id=33847

1:55 PM WebKit Team edited by Michelangelo De Simone
(diff)
1:47 PM WebKit Team edited by Michelangelo De Simone
(diff)
12:54 PM WebKit Team edited by bfulgham@webkit.org
Added some (unsubstantiated) claims about what I know. (diff)
12:37 PM Changeset in webkit [53813] by ap@apple.com
  • 2 edits in trunk/WebCore

Rubber-stamped by Geoffrey Garen.

​https://bugs.webkit.org/show_bug.cgi?id=34076
An image remains accessible via form.property syntax after being removed from document.

Fix crashing regression tests (tables/mozilla/bugs/bug4527.html et al.)

  • html/HTMLImageElement.cpp: (WebCore::HTMLImageElement::insertedIntoTree): Remove incorrect assertions added in the previous patch - it's mot true that m_for is always a parent; table parsing can reparent the image element, but m_form still needs to be set.
12:07 PM Changeset in webkit [53812] by ap@apple.com
  • 4 edits
    4 adds in trunk

Reviewed by Geoffrey Garen.

​https://bugs.webkit.org/show_bug.cgi?id=34076
An image remains accessible via form.property syntax after being removed from document.

Tests: fast/forms/removed-image-as-property.html

fast/forms/reparented-image-as-property.html

  • html/HTMLImageElement.cpp: (WebCore::HTMLImageElement::~HTMLImageElement): This is called during GC - not a good time to make observable changes to DOM. (WebCore::HTMLImageElement::insertedIntoTree): This is the right place to do any work that depends on connectedness to some ancestor. We still allow for m_form to be set via constructor, which happens during parsing. (WebCore::HTMLImageElement::removedFromTree): Ditto.
  • html/HTMLImageElement.h: Added removedFromTree/insertedIntoTree, moved removedFromDocument and insertedIntoDocument to private section, as they shouldn't be called directly.
12:03 PM Changeset in webkit [53811] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-01-25 Daniel Bates <​dbates@webkit.org>

Reviewed by Adam Barth.

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

Fixes misspelled words in comments.

  • accessibility/AccessibilityListBox.cpp: (WebCore::AccessibilityListBox::addChildren): (WebCore::AccessibilityListBox::doAccessibilityHitTest):
  • accessibility/mac/AccessibilityObjectWrapper.mm: (-[AccessibilityObjectWrapper detach]): (AXAttributeStringSetElement):
  • bindings/js/JSDOMBinding.cpp: (WebCore::completeURL):
  • bindings/js/JSDOMWindowBase.cpp:
  • bindings/js/JSDOMWindowBase.h:
  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::getOwnPropertySlot):
  • bindings/js/JSSVGPODTypeWrapper.h:
  • bindings/js/ScriptController.cpp: (WebCore::ScriptController::evaluateInWorld):
  • bridge/NP_jsobject.cpp: (_NPN_SetException):
  • bridge/jni/jsc/JavaInstanceJSC.cpp: (JavaInstance::invokeMethod): (JObjectWrapper::JObjectWrapper):
  • bridge/objc/objc_instance.mm: (ObjcInstance::setValueOfUndefinedField): (ObjcInstance::getValueOfUndefinedField):
  • css/CSSFontSelector.cpp: (WebCore::CSSFontSelector::addFontFaceRule):
  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::CSSStyleSelector): (WebCore::CSSStyleSelector::sortMatchedRules): (WebCore::CSSStyleSelector::applyDeclarations): (WebCore::CSSStyleSelector::applyProperty):
  • dom/Document.cpp: (WebCore::Document::updateLayoutIgnorePendingStylesheets): (WebCore::Document::detach):
  • dom/Document.h:
  • dom/Element.cpp: (WebCore::Element::updateFocusAppearance):
  • dom/Element.h:
  • dom/MessagePortChannel.h:
  • dom/MouseRelatedEvent.cpp: (WebCore::MouseRelatedEvent::receivedTarget):
  • dom/Node.cpp: (WebCore::Node::isFocusable): (WebCore::Node::shadowAncestorNode):
  • dom/Node.h:
  • dom/Position.cpp: (WebCore::Position::upstream):
  • dom/Position.h:
  • dom/Range.cpp: (WebCore::Range::processContents):
  • dom/StyledElement.cpp: (WebCore::StyledElement::addCSSColor):
  • editing/ApplyStyleCommand.cpp: (WebCore::StyleChange::reconcileTextDecorationProperties): (WebCore::StyleChange::extractTextStyles): (WebCore::prepareEditingStyleToApplyAt):
  • editing/DeleteButtonController.cpp: (WebCore::isDeletableElement):
  • editing/DeleteSelectionCommand.cpp: (WebCore::DeleteSelectionCommand::initializeStartEnd):
  • editing/Editor.cpp: (WebCore::Editor::learnSpelling):
  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleMessage.prototype._format):
  • loader/Cache.cpp: (WebCore::Cache::pruneLiveResources): (WebCore::Cache::pruneDeadResources):
  • loader/Cache.h:
  • loader/CachedResource.cpp: (WebCore::CachedResource::makePurgeable):
  • loader/CachedResource.h:
  • loader/CachedResourceHandle.h:
  • loader/DocumentLoader.h:
  • loader/FTPDirectoryParser.cpp: (WebCore::parseOneFTPLine):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::didExplicitOpen):
  • loader/ResourceLoader.cpp: (WebCore::ResourceLoader::didCancel):
  • loader/WorkerThreadableLoader.h:
  • loader/appcache/ApplicationCacheStorage.cpp: (WebCore::ApplicationCacheStorage::spaceNeeded):
  • notifications/NotificationPresenter.h: (WebCore::NotificationPresenter::):
  • page/animation/AnimationController.cpp: (WebCore::AnimationController::updateAnimations):
  • platform/LinkHash.cpp: (WebCore::cleanPath):
  • platform/graphics/cg/GraphicsContextCG.cpp: (WebCore::calculateDrawingMode):
  • platform/graphics/cg/PatternCG.cpp: (WebCore::Pattern::createPlatformPattern):
  • platform/graphics/mac/ComplexTextController.cpp: (WebCore::ComplexTextController::offsetForPosition):
  • platform/graphics/mac/GraphicsLayerCA.h:
  • platform/graphics/mac/GraphicsLayerCA.mm: (WebCore::GraphicsLayerCA::ensureStructuralLayer):
  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::doSeek): (WebCore::MediaPlayerPrivate::setClosedCaptionsVisible): (WebCore::MediaPlayerPrivate::hasAvailableVideoFrame):
  • platform/mac/PasteboardMac.mm: (WebCore::Pasteboard::writeSelection): (WebCore::Pasteboard::writePlainText): (WebCore::Pasteboard::writeURL):
  • platform/text/TextCodecICU.cpp: (WebCore::TextCodecICU::registerExtendedEncodingNames):
  • platform/text/TextEncodingDetectorICU.cpp: (WebCore::detectTextEncoding):
  • rendering/AutoTableLayout.cpp: (WebCore::AutoTableLayout::calcEffectiveWidth): (WebCore::AutoTableLayout::layout):
  • rendering/RenderBox.cpp: (WebCore::RenderBox::calcAbsoluteHorizontalValues): (WebCore::RenderBox::calcAbsoluteVerticalValues): (WebCore::RenderBox::calcAbsoluteHorizontalReplaced): (WebCore::RenderBox::calcAbsoluteVerticalReplaced):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::scrollByRecursively): (WebCore::RenderLayer::paintLayer): (WebCore::RenderLayer::hitTestLayer):
  • rendering/RenderObject.cpp: (WebCore::RenderObject::destroy):
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::setInnerTextValue):
  • rendering/RenderTheme.cpp: (WebCore::RenderTheme::stateChanged):
  • rendering/SVGRenderSupport.cpp: (WebCore::SVGRenderBase::mapLocalToContainer):
  • rendering/SVGRootInlineBox.cpp: (WebCore::SVGRootInlineBox::buildTextChunks):
  • rendering/TransformState.cpp: (WebCore::TransformState::flattenWithTransform):
  • wml/WMLInputElement.cpp: (WebCore::WMLInputElement::dispatchBlurEvent): (WebCore::WMLInputElement::suggestedValue):
11:34 AM WebKit Team edited by darin@chromium.org
(diff)
11:32 AM WebKit Team edited by darin@chromium.org
(diff)
11:26 AM Changeset in webkit [53810] by yurys@chromium.org
  • 2 edits in trunk/LayoutTests

2010-01-25 Yury Semikhatsky <​yurys@chromium.org>

Unreviewed. Add new inspector test added in r53807 to skip
list on qt platform.

  • platform/qt/Skipped:
11:20 AM Changeset in webkit [53809] by dbates@webkit.org
  • 62 edits in trunk/WebCore

2010-01-25 Daniel Bates <​dbates@webkit.org>

Reviewed by Adam Barth.

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

Fixes misspelled words in comments.

  • accessibility/AccessibilityListBox.cpp: (WebCore::AccessibilityListBox::addChildren): (WebCore::AccessibilityListBox::doAccessibilityHitTest):
  • accessibility/mac/AccessibilityObjectWrapper.mm: (-[AccessibilityObjectWrapper detach]): (AXAttributeStringSetElement):
  • bindings/js/JSDOMBinding.cpp: (WebCore::completeURL):
  • bindings/js/JSDOMWindowBase.cpp:
  • bindings/js/JSDOMWindowBase.h:
  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::getOwnPropertySlot):
  • bindings/js/JSSVGPODTypeWrapper.h:
  • bindings/js/ScriptController.cpp: (WebCore::ScriptController::evaluateInWorld):
  • bridge/NP_jsobject.cpp: (_NPN_SetException):
  • bridge/jni/jsc/JavaInstanceJSC.cpp: (JavaInstance::invokeMethod): (JObjectWrapper::JObjectWrapper):
  • bridge/objc/objc_instance.mm: (ObjcInstance::setValueOfUndefinedField): (ObjcInstance::getValueOfUndefinedField):
  • css/CSSFontSelector.cpp: (WebCore::CSSFontSelector::addFontFaceRule):
  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::CSSStyleSelector): (WebCore::CSSStyleSelector::sortMatchedRules): (WebCore::CSSStyleSelector::applyDeclarations): (WebCore::CSSStyleSelector::applyProperty):
  • dom/Document.cpp: (WebCore::Document::updateLayoutIgnorePendingStylesheets): (WebCore::Document::detach):
  • dom/Document.h:
  • dom/Element.cpp: (WebCore::Element::updateFocusAppearance):
  • dom/Element.h:
  • dom/MessagePortChannel.h:
  • dom/MouseRelatedEvent.cpp: (WebCore::MouseRelatedEvent::receivedTarget):
  • dom/Node.cpp: (WebCore::Node::isFocusable): (WebCore::Node::shadowAncestorNode):
  • dom/Node.h:
  • dom/Position.cpp: (WebCore::Position::upstream):
  • dom/Position.h:
  • dom/Range.cpp: (WebCore::Range::processContents):
  • dom/StyledElement.cpp: (WebCore::StyledElement::addCSSColor):
  • editing/ApplyStyleCommand.cpp: (WebCore::StyleChange::reconcileTextDecorationProperties): (WebCore::StyleChange::extractTextStyles): (WebCore::prepareEditingStyleToApplyAt):
  • editing/DeleteButtonController.cpp: (WebCore::isDeletableElement):
  • editing/DeleteSelectionCommand.cpp: (WebCore::DeleteSelectionCommand::initializeStartEnd):
  • editing/Editor.cpp: (WebCore::Editor::learnSpelling):
  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleMessage.prototype._format):
  • loader/Cache.cpp: (WebCore::Cache::pruneLiveResources): (WebCore::Cache::pruneDeadResources):
  • loader/Cache.h:
  • loader/CachedResource.cpp: (WebCore::CachedResource::makePurgeable):
  • loader/CachedResource.h:
  • loader/CachedResourceHandle.h:
  • loader/DocumentLoader.h:
  • loader/FTPDirectoryParser.cpp: (WebCore::parseOneFTPLine):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::didExplicitOpen):
  • loader/ResourceLoader.cpp: (WebCore::ResourceLoader::didCancel):
  • loader/WorkerThreadableLoader.h:
  • loader/appcache/ApplicationCacheStorage.cpp: (WebCore::ApplicationCacheStorage::spaceNeeded):
  • notifications/NotificationPresenter.h: (WebCore::NotificationPresenter::):
  • page/animation/AnimationController.cpp: (WebCore::AnimationController::updateAnimations):
  • platform/LinkHash.cpp: (WebCore::cleanPath):
  • platform/graphics/cg/GraphicsContextCG.cpp: (WebCore::calculateDrawingMode):
  • platform/graphics/cg/PatternCG.cpp: (WebCore::Pattern::createPlatformPattern):
  • platform/graphics/mac/ComplexTextController.cpp: (WebCore::ComplexTextController::offsetForPosition):
  • platform/graphics/mac/GraphicsLayerCA.h:
  • platform/graphics/mac/GraphicsLayerCA.mm: (WebCore::GraphicsLayerCA::ensureStructuralLayer):
  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::doSeek): (WebCore::MediaPlayerPrivate::setClosedCaptionsVisible): (WebCore::MediaPlayerPrivate::hasAvailableVideoFrame):
  • platform/mac/PasteboardMac.mm: (WebCore::Pasteboard::writeSelection): (WebCore::Pasteboard::writePlainText): (WebCore::Pasteboard::writeURL):
  • platform/text/TextCodecICU.cpp: (WebCore::TextCodecICU::registerExtendedEncodingNames):
  • platform/text/TextEncodingDetectorICU.cpp: (WebCore::detectTextEncoding):
  • rendering/AutoTableLayout.cpp: (WebCore::AutoTableLayout::calcEffectiveWidth): (WebCore::AutoTableLayout::layout):
  • rendering/RenderBox.cpp: (WebCore::RenderBox::calcAbsoluteHorizontalValues): (WebCore::RenderBox::calcAbsoluteVerticalValues): (WebCore::RenderBox::calcAbsoluteHorizontalReplaced): (WebCore::RenderBox::calcAbsoluteVerticalReplaced):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::scrollByRecursively): (WebCore::RenderLayer::paintLayer): (WebCore::RenderLayer::hitTestLayer):
  • rendering/RenderObject.cpp: (WebCore::RenderObject::destroy):
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::setInnerTextValue):
  • rendering/RenderTheme.cpp: (WebCore::RenderTheme::stateChanged):
  • rendering/SVGRenderSupport.cpp: (WebCore::SVGRenderBase::mapLocalToContainer):
  • rendering/SVGRootInlineBox.cpp: (WebCore::SVGRootInlineBox::buildTextChunks):
  • rendering/TransformState.cpp: (WebCore::TransformState::flattenWithTransform):
  • wml/WMLInputElement.cpp: (WebCore::WMLInputElement::dispatchBlurEvent): (WebCore::WMLInputElement::suggestedValue):
10:58 AM Changeset in webkit [53808] by ggaren@apple.com
  • 2 edits in trunk/WebCore

Suggested by Darin Adler.

Fixed a spelling error, tightened up some grammar.

  • bindings/js/JSDOMBinding.cpp:

(WebCore::forgetDOMObject):
(WebCore::forgetDOMNode):

10:57 AM Changeset in webkit [53807] by eric@webkit.org
  • 3 edits
    2 adds in trunk

2010-01-25 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Timothy Hatcher.

Test that Web Inspector doesn't change methods declared by the inspected
page.
​https://bugs.webkit.org/show_bug.cgi?id=34089

  • inspector/inspected-objects-not-overriden-expected.txt: Added.
  • inspector/inspected-objects-not-overriden.html: Added.

2010-01-25 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Timothy Hatcher.

Instead of adding inspector methods to the inspected global objects
use private implementation visible from inspector code only.
​https://bugs.webkit.org/show_bug.cgi?id=34089

Test: inspector/inspected-objects-not-overriden.html

  • inspector/front-end/InjectedScript.js: (injectedScriptConstructor):
8:37 AM QtWebKitBackportingFixes edited by benjamin.poulain@nokia.com
(diff)
8:06 AM Changeset in webkit [53806] by pfeldman@chromium.org
  • 5 edits in trunk/WebCore

2010-01-25 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: don't show headers/content tabs in scripts view.

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

  • inspector/front-end/ResourceView.js: (WebInspector.ResourceView): (WebInspector.ResourceView.prototype.set headersVisible): (WebInspector.ResourceView.prototype._selectContentTab): (WebInspector.ResourceView.prototype._innerSelectContentTab):
  • inspector/front-end/ResourcesPanel.js: (WebInspector.ResourcesPanel.prototype.show): (WebInspector.ResourcesPanel.prototype.recreateViewForResourceIfNeeded): (WebInspector.ResourcesPanel.prototype.showResource):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype.show): (WebInspector.ScriptsPanel.prototype._showScriptOrResource):
  • inspector/front-end/inspector.css:
7:37 AM Changeset in webkit [53805] by Philippe Normand
  • 2 edits in trunk/LayoutTests

2010-01-25 Philippe Normand <​pnormand@igalia.com>

Rubber-stamped by Xan Lopez.

[Gtk] media/video-reverse-play-duration.html fails on and off on Gtk buildbots
​https://bugs.webkit.org/show_bug.cgi?id=34086

  • platform/gtk/Skipped: Skipping the test, requires investigation on the failure.
7:24 AM Changeset in webkit [53804] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-01-25 Simon Hausmann <Simon Hausmann>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Reduce the size of the library when compiling for Maemo5
​https://bugs.webkit.org/show_bug.cgi?id=34050

Exclude the Web Inspector's qrc file in the Maemo5 build, just
like we do for the Symbian build. Saves about ~900k.

  • WebCore.pro:
7:08 AM Changeset in webkit [53803] by eric@webkit.org
  • 5 edits in trunk/WebCore

2010-01-25 Simon Hausmann <Simon Hausmann>

Reviewed by Kenneth Rohde Christiansen.

[Qt] In the StyledPainter determine the style from the Render and Scrollbar theme instead of from the paint device
​https://bugs.webkit.org/show_bug.cgi?id=34054

Getting the style from the painter's paint device is a hack that breaks when
the paint device's style is different than the style that is used when
calculating the metries earlier when there is no painter available.

This change moves us closer to always using the same style.

  • platform/qt/RenderThemeQt.cpp: (WebCore::StylePainter::StylePainter): (WebCore::StylePainter::init): (WebCore::RenderThemeQt::paintButton): (WebCore::RenderThemeQt::paintTextField): (WebCore::RenderThemeQt::paintMenuList): (WebCore::RenderThemeQt::paintMenuListButton): (WebCore::RenderThemeQt::paintSliderTrack): (WebCore::RenderThemeQt::paintMediaMuteButton): (WebCore::RenderThemeQt::paintMediaPlayButton): (WebCore::RenderThemeQt::paintMediaSliderTrack): (WebCore::RenderThemeQt::paintMediaSliderThumb):
  • platform/qt/RenderThemeQt.h:
  • platform/qt/ScrollbarThemeQt.cpp: (WebCore::ScrollbarThemeQt::paint): (WebCore::ScrollbarThemeQt::hitTest): (WebCore::ScrollbarThemeQt::shouldCenterOnThumb): (WebCore::ScrollbarThemeQt::scrollbarThickness): (WebCore::ScrollbarThemeQt::thumbLength): (WebCore::ScrollbarThemeQt::trackPosition): (WebCore::ScrollbarThemeQt::trackLength): (WebCore::ScrollbarThemeQt::paintScrollCorner): (WebCore::ScrollbarThemeQt::style):
  • platform/qt/ScrollbarThemeQt.h:
6:42 AM Changeset in webkit [53802] by eric@webkit.org
  • 6 edits in trunk

2010-01-25 Simon Hausmann <Simon Hausmann>

Reviewed by Kenneth Rohde Christiansen.

[Qt] In RenderThemeQt determine the QStyle from the page client instead of the page's view
​https://bugs.webkit.org/show_bug.cgi?id=34053

  • platform/qt/QWebPageClient.h: Add function to return the style.
  • platform/qt/RenderThemeQt.cpp: (WebCore::RenderThemeQt::qStyle): Determine the style from the page client instead of the page's view.

2010-01-25 Simon Hausmann <Simon Hausmann>

Reviewed by Kenneth Rohde Christiansen.

[Qt] In RenderThemeQt determine the QStyle from the page client instead of the page's view
​https://bugs.webkit.org/show_bug.cgi?id=34053

  • Api/qgraphicswebview.cpp: (QGraphicsWebViewPrivate::style): Implement QWebPageClient::style() and return the graphics widget's style.
  • Api/qwebpage.cpp: (QWebPageWidgetClient::style): Implement QWebPageClient::style() and return the widget's style.
6:20 AM Changeset in webkit [53801] by Simon Hausmann
  • 3 edits in trunk/WebCore

[Qt] Clean up code to determine palette from page client in RenderThemeQt
​https://bugs.webkit.org/show_bug.cgi?id=34052

Patch by Simon Hausmann <Simon Hausmann> on 2010-01-25
Reviewed by Kenneth Christiansen.

Use the RenderTheme's associated page instead of going through the
render tree and the document.

  • platform/qt/RenderThemeQt.cpp:

(WebCore::RenderThemeQt::setPaletteFromPageClientIfExists):
(WebCore::RenderThemeQt::initializeCommonQStyleOptions):

  • platform/qt/RenderThemeQt.h:
5:56 AM QtWebKitBackportingFixes edited by Simon Hausmann
(diff)
5:53 AM QtWebKitBackportingFixes edited by Simon Hausmann
(diff)
5:18 AM Changeset in webkit [53800] by Simon Hausmann
  • 2 edits in trunk

[Qt] Fix the build on Maemo5.

Reviewed by Laszlo Gombos.

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

  • WebKit.pri: Disable the use of uitools, just like it's done for Symbian.
5:14 AM Changeset in webkit [53799] by eric@webkit.org
  • 3 edits
    1 add in trunk

2010-01-25 Janne Koskinen <​janne.p.koskinen@digia.com>

Reviewed by Simon Hausmann.

[Qt] Phone backup support for QtWebkit for Symbian devices.
​https://bugs.webkit.org/show_bug.cgi?id=34077

  • WebCore.pro:

2010-01-25 Janne Koskinen <​janne.p.koskinen@digia.com>

Reviewed by Simon Hausmann.

[Qt] Phone backup support for QtWebkit for Symbian devices.
​https://bugs.webkit.org/show_bug.cgi?id=34077

  • symbian/backup_registration.xml: Added.
5:13 AM Changeset in webkit [53798] by yurys@chromium.org
  • 2 edits
    3 copies
    2 adds in trunk/LayoutTests

2010-01-25 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Pavel Feldman.

Update inspector test expectations. Line number and source
name are present only in Chromium when console message is
logged before document load is completed.

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

  • platform/chromium/inspector/console-log-before-inspector-open-expected.txt: Added.
  • platform/gtk/inspector/console-log-before-inspector-open-expected.txt: Copied from LayoutTests/inspector/console-log-before-inspector-open-expected.txt.
  • platform/mac/inspector/console-log-before-inspector-open-expected.txt: Copied from LayoutTests/inspector/console-log-before-inspector-open-expected.txt.
  • platform/win/inspector/console-log-before-inspector-open-expected.txt: Renamed from LayoutTests/inspector/console-log-before-inspector-open-expected.txt.
4:57 AM QtWebKitBackportingFixes edited by koshuin@gmail.com
(diff)
4:37 AM Changeset in webkit [53797] by eric@webkit.org
  • 6 edits in trunk/WebCore

2010-01-25 Benjamin Poulain <​benjamin.poulain@nokia.com>

Reviewed by Antti Koivisto.

Do not render the full frame when there is some elements with fixed positioning
​https://bugs.webkit.org/show_bug.cgi?id=33150

Do not render the full frame when there is some elements with fixed positioning
​https://bugs.webkit.org/show_bug.cgi?id=33150

  • page/FrameView.cpp: (WebCore::FrameView::useSlowRepaints): (WebCore::FrameView::useSlowRepaintsIfNotOverlapped): (WebCore::FrameView::registerFixedPositionedObject): (WebCore::FrameView::unregisterFixedPositionedObject): (WebCore::FrameView::scrollContentsFastPath):
  • page/FrameView.h:
  • platform/ScrollView.cpp: (WebCore::ScrollView::scrollContents): (WebCore::ScrollView::scrollContentsFastPath):
  • platform/ScrollView.h:
  • rendering/RenderObject.cpp: (WebCore::RenderObject::styleWillChange): (WebCore::RenderObject::destroy):
4:20 AM WebKit Team edited by Chris Jerdonek
Fixed mangled special character and added missing quote. (diff)
4:16 AM WebKit Team edited by Chris Jerdonek
Added introduction and key. (diff)
3:43 AM WebKit Team edited by Chris Jerdonek
Restored page outline. (diff)
3:18 AM Changeset in webkit [53796] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-01-25 Chris Jerdonek <Chris Jerdonek>

Reviewed by Adam Barth.

Improved prepare-ChangeLog so that it preserves the relative
indentation of a git commit message.

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

  • Scripts/prepare-ChangeLog:
    • Also adjusted the script so that it does not add white space characters to empty lines.
2:58 AM Changeset in webkit [53795] by pfeldman@chromium.org
  • 9 edits
    1 add in trunk/WebCore

2010-01-24 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Add support for addMessage/clearMessages to SourceFrame2.

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

  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • inspector/front-end/SourceFrame2.js: (WebInspector.SourceFrame2): (WebInspector.SourceFrame2.prototype.addMessage): (WebInspector.SourceFrame2.prototype.clearMessages): (WebInspector.SourceFrame2.prototype._incrementMessageRepeatCount): (WebInspector.SourceFrame2.prototype._addExistingMessagesToSource): (WebInspector.SourceFrame2.prototype._addMessageToSource): (WebInspector.SourceFrame2.prototype.resize):
  • inspector/front-end/TextEditor.js: (WebInspector.TextEditor): (WebInspector.TextEditor.prototype.setDivDecoration): (WebInspector.TextEditor.prototype._lineHeight): (WebInspector.TextEditor.prototype._highlightChanged): (WebInspector.TextEditor.prototype.packAndRepaintAll): (WebInspector.TextEditor.prototype._updateSize): (WebInspector.TextEditor.prototype._repaintAll): (WebInspector.TextEditor.prototype._paint): (WebInspector.TextEditor.prototype._paintLinesContinuation): (WebInspector.TextEditor.prototype._mouseOut): (WebInspector.TextEditor.prototype._updateDivDecorations): (WebInspector.TextEditor.prototype._positionDivDecoration): (WebInspector.TextEditor.prototype._paintSelection): (WebInspector.TextEditor.prototype._replaceSelectionWith):
  • inspector/front-end/TextEditorHighlighter.js: (WebInspector.TextEditorHighlighter.prototype.updateHighlight):
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.css:
  • inspector/front-end/inspector.html:
  • inspector/front-end/textEditor.css: Added.
2:37 AM Changeset in webkit [53794] by pfeldman@chromium.org
  • 9 edits in trunk/WebCore

2010-01-24 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Replace split pane with tabbed pane in resource
contents view.

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

1:43 AM QtWebKitBackportingFixes edited by Simon Hausmann
(diff)
12:45 AM WebKit Team edited by Simon Hausmann
(diff)
12:02 AM WebKit Team edited by krit@webkit.org
(diff)

Jan 24, 2010:

11:04 PM Changeset in webkit [53793] by rolandsteiner@chromium.org
  • 17 edits in trunk/LayoutTests

build fix: rubber-stamped by Eric Seidel.

Fix expected baselines for ruby layout tests on Windows.

  • platform/win/fast/ruby/ruby-empty-rt-expected.txt:
  • platform/win/fast/ruby/ruby-length-expected.txt:
  • platform/win/fast/ruby/ruby-run-break-expected.txt:
  • platform/win/fast/ruby/ruby-runs-expected.txt:
  • platform/win/fast/ruby/ruby-runs-spans-expected.txt:
  • platform/win/fast/ruby/ruby-simple-expected.txt:
  • platform/win/fast/ruby/ruby-simple-rp-expected.txt:
  • platform/win/fast/ruby/ruby-trailing-expected.txt:
  • platform/win/fast/ruby/rubyDOM-insert-rt-expected.txt:
  • platform/win/fast/ruby/rubyDOM-insert-text1-expected.txt:
  • platform/win/fast/ruby/rubyDOM-insert-text2-expected.txt:
  • platform/win/fast/ruby/rubyDOM-insert-text3-expected.txt:
  • platform/win/fast/ruby/rubyDOM-remove-rt1-expected.txt:
  • platform/win/fast/ruby/rubyDOM-remove-rt2-expected.txt:
  • platform/win/fast/ruby/rubyDOM-remove-text1-expected.txt:
  • platform/win/fast/ruby/rubyDOM-remove-text2-expected.txt:
10:21 PM Changeset in webkit [53792] by eric.carlson@apple.com
  • 3 edits
    2 adds in trunk

2010-01-24 Eric Carlson <​eric.carlson@apple.com>

Reviewed by Dan Bernstein.

Media element duration changes to zero after playing in reverse
​https://bugs.webkit.org/show_bug.cgi?id=34041

Test: media/video-reverse-play-duration.html

  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::didEnd): QTKit's ended notification fires at time zero when playing in reverse, so don't reset duration in that case.

2010-01-24 Eric Carlson <​eric.carlson@apple.com>

Reviewed by Dan Bernstein.

Media element duration changes to zero after playing in reverse
​https://bugs.webkit.org/show_bug.cgi?id=34041

  • media/video-reverse-play-duration-expected.txt: Added.
  • media/video-reverse-play-duration.html: Added.
10:05 PM WebKit Team edited by abarth@webkit.org
(diff)
10:00 PM WebKit Team edited by abarth@webkit.org
(diff)
9:55 PM WebKit Team edited by abarth@webkit.org
(diff)
9:50 PM Changeset in webkit [53791] by rolandsteiner@chromium.org
  • 2 edits in trunk/LayoutTests

Bug 28420 - Re-enable ruby Windows layout tests
(​https://bugs.webkit.org/show_bug.cgi?id=28420, patch 5b)

Reviewed by Maciej Stachowiak.

After creating new Latin-character-only layout tests for ruby
(​https://bugs.webkit.org/show_bug.cgi?id=31865) and resolving the issue
that caused layout tests to fail then due to the addition of the ruby
layout tests (cf. ​https://bugs.webkit.org/show_bug.cgi?id=31200, problem
not actually caused by ruby), the ruby layout tests should als be
un-skipped for the Windows platform.

  • platform/win/Skipped:
9:50 PM WebKit Team edited by abarth@webkit.org
(diff)
9:43 PM WebKit Team edited by abarth@webkit.org
(diff)
9:31 PM WebKit Team edited by abarth@webkit.org
(diff)
9:25 PM WebKit Team edited by abarth@webkit.org
(diff)
9:17 PM WebKit Team edited by abarth@webkit.org
(diff)
8:11 PM Changeset in webkit [53790] by mjs@apple.com
  • 3 edits
    2 adds in trunk

2010-01-24 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Dan Bernstein.

Content with heavily nested residual style is so slow, it seems like a hang
​https://bugs.webkit.org/show_bug.cgi?id=34059
<rdar://problem/7292906>


Test cast: fast/parser/residual-style-hang.html

  • html/HTMLParser.cpp: (WebCore::HTMLParser::handleResidualStyleCloseTagAcrossBlocks): Limit the number of iterations of the main loop to 5.


The reason this limit is necessary is that otherwise, N misnested open tags followed
by N misnested close tags will cause O(N2) of work due to cloning and attaching subtrees;
at a fixed limit, the cost is at worst O(N).


The code that was in the loop originally ran exactly once - the loop was added in
r21472 to fix <​https://bugs.webkit.org/show_bug.cgi?id=13603>. I have verified that
with the iteration limit, the bug is still fixed, both with the original test case
and with the layout tests tht were added.

2010-01-24 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Dan Bernstein.

Content with heavily nested residual style is so slow, it seems like a hang
​https://bugs.webkit.org/show_bug.cgi?id=34059
<rdar://problem/7292906>

Test case for the above bug fix.

  • fast/parser/residual-style-hang-expected.txt: Added.
  • fast/parser/residual-style-hang.html: Added.
7:53 PM Changeset in webkit [53789] by eric@webkit.org
  • 2 edits in trunk/JavaScriptCore

2010-01-24 Laszlo Gombos <Laszlo Gombos>

Reviewed by Maciej Stachowiak.

Refactor JITStubs.cpp so that DEFINE_STUB_FUNCTION is only used once for each function
​https://bugs.webkit.org/show_bug.cgi?id=33866

Place the guard USE(JSVALUE32_64) inside the body of the DEFINE_STUB_FUNCTION
macro for those functions that are always present.

  • jit/JITStubs.cpp: (JSC::DEFINE_STUB_FUNCTION):
7:26 PM Changeset in webkit [53788] by eric@webkit.org
  • 2 edits
    2 adds
    3 deletes in trunk/LayoutTests

2010-01-24 Kent Tamura <​tkent@chromium.org>

Reviewed by Darin Adler.

Improve portability of listbox-selection-2.html
​https://bugs.webkit.org/show_bug.cgi?id=33994

  • Convert to dumpAsText().
  • Determine the height of an item at runtime
  • fast/forms/listbox-selection-2-expected.txt: Added.
  • fast/forms/listbox-selection-2.html:
  • fast/forms/script-tests/listbox-selection-2.js
  • platform/mac/fast/forms/listbox-selection-2-expected.checksum: Removed.
  • platform/mac/fast/forms/listbox-selection-2-expected.png: Removed.
  • platform/mac/fast/forms/listbox-selection-2-expected.txt: Removed.
6:15 PM Changeset in webkit [53787] by tkent@chromium.org
  • 7 edits in trunk

2010-01-24 Kent Tamura <​tkent@chromium.org>

Reviewed by Darin Adler.

HTMLInputElement::valueAsDate setter support for type=week.
​https://bugs.webkit.org/show_bug.cgi?id=33986

Add setter tests to input-valueasdate-datetime.js, and update the
expectation.

  • fast/forms/input-valueasdate-week-expected.txt:
  • fast/forms/script-tests/input-valueasdate-week.js:

2010-01-24 Kent Tamura <​tkent@chromium.org>

Reviewed by Darin Adler.

HTMLInputElement::valueAsDate setter support for type=week.
​https://bugs.webkit.org/show_bug.cgi?id=33986

Introduce ISODateTime::setMillisecondsSinceEpochForWeek() and add
Week type support to ISODateTime::toString().

  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::valueAsDate): Avoid default:. (WebCore::HTMLInputElement::setValueAsDate): Add WEEK type handling and avoid default:.
  • html/ISODateTime.cpp: (WebCore::offsetTo1stWeekStart): (WebCore::ISODateTime::setMillisecondsSinceEpochForWeek): (WebCore::ISODateTime::millisecondsSinceEpoch): (WebCore::ISODateTime::toString): Add Week type support and avoid default:.
  • html/ISODateTime.h:
4:41 PM WebKit Team edited by ddkilzer@webkit.org
(diff)
4:15 PM WebKit Team edited by Michelangelo De Simone
(diff)
2:54 PM Changeset in webkit [53786] by eric.carlson@apple.com
  • 2 edits in trunk/LayoutTests

2010-01-24 Eric Carlson <​eric.carlson@apple.com>

No review, build fix.

Only log the first 'timeupdate' event from audio-delete-while-slider-thumb-clicked.html
as the number fired can vary according to how long it takes to run the test.

  • media/audio-delete-while-slider-thumb-clicked.html:
2:54 PM Changeset in webkit [53785] by oliver@apple.com
  • 6 edits in trunk/WebCore

2010-01-24 Oliver Hunt <​oliver@apple.com>

Reviewed by Darin Adler.

Object Serialization assumes deserialization will always occur in the context of a webpage
​https://bugs.webkit.org/show_bug.cgi?id=34056
rdar://7573833

Added ClassInfo to JSDOMGlobalObject and make the window and worker class info
inherit from it. With this in place we're able to determine whether a given
global object is a DOMGlobalObject, and then use that information to avoid
attempting to deserialize types that require the presence of the DOM.

No test as all global objects in webkit inherit from JSDOMGlobalObject.

  • bindings/js/JSDOMGlobalObject.cpp: (WebCore::):
  • bindings/js/JSDOMGlobalObject.h: (WebCore::JSDOMGlobalObject::classInfo):
  • bindings/js/JSDOMWindowBase.cpp: (WebCore::):
  • bindings/js/JSWorkerContextBase.cpp: (WebCore::):
  • bindings/js/SerializedScriptValue.cpp: (WebCore::DeserializingTreeWalker::DeserializingTreeWalker): (WebCore::DeserializingTreeWalker::convertIfTerminal):
2:30 PM Changeset in webkit [53784] by eric@webkit.org
  • 2 edits in trunk/LayoutTests

2010-01-24 Eric Seidel <​eric@webkit.org>

No review.

Middle clicking the primary scroll bars on Chromium Linux triggers a paste event
​https://bugs.webkit.org/show_bug.cgi?id=33062

  • platform/gtk/Skipped:
    • This new test fails on Gtk for unknown reasons.
    • Skipping for now to make the bots green, until a Gtk person can look.
2:15 PM Changeset in webkit [53783] by eric@webkit.org
  • 3 edits in trunk/WebKitTools

2010-01-24 Eric Seidel <​eric@webkit.org>

No review, rolling out r53763.
​http://trac.webkit.org/changeset/53763
​https://bugs.webkit.org/show_bug.cgi?id=33895

Broke 20+ tests on Windows.

  • Scripts/run-webkit-tests:
  • Scripts/webkitdirs.pm:
2:05 PM WebKit Team edited by jpetsovits@rim.com
(diff)
1:05 PM Changeset in webkit [53782] by Chris Jerdonek
  • 3 edits in trunk/WebKitSite

Added to contributing code page a pointer to the preferred
license text. Also added information about copyright lines.

Reviewed by David Kilzer.

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

  • coding/contributing.html:
  • css/main.css:
    • Added CSS class for books.
11:42 AM Changeset in webkit [53781] by Simon Fraser
  • 4 edits in trunk/LayoutTests

2010-01-24 Simon Fraser <Simon Fraser>

No review.

Fix incorrect layout test that I checked in earlier.

  • platform/mac/compositing/reflections/load-video-in-reflection-expected.checksum:
  • platform/mac/compositing/reflections/load-video-in-reflection-expected.png:
  • platform/mac/compositing/reflections/load-video-in-reflection-expected.txt:
10:22 AM Changeset in webkit [53780] by eric.carlson@apple.com
  • 3 edits
    2 adds in trunk

2010-01-24 Eric Carlson <​eric.carlson@apple.com>

Reviewed by Simon Fraser.

​https://bugs.webkit.org/show_bug.cgi?id=34047
<rdar://problem/7573699>
Media element "endedPlayback" logic doesn't match spec

Test: media/video-timeupdate-reverse-play.html

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::ended): The 'ended' attribute should only be true when playback rate is positive. (WebCore::HTMLMediaElement::mediaPlayerTimeChanged): Always schedule a 'timeupdate' event when the media engine signals a time discontinuity, scheduleTimeupdateEvent will only queue an event if one hasn't already been posted for the current movie time. (WebCore::HTMLMediaElement::endedPlayback): Update to match the spec logic.

2010-01-24 Eric Carlson <​eric.carlson@apple.com>

Reviewed by Simon Fraser.

​https://bugs.webkit.org/show_bug.cgi?id=34047
<rdar://problem/7573699>
Media element "endedPlayback" logic doesn't match spec

  • media/video-timeupdate-reverse-play-expected.txt: Added.
  • media/video-timeupdate-reverse-play.html: Added.
9:47 AM Changeset in webkit [53779] by Simon Fraser
  • 4 edits
    4 adds in trunk

2010-01-23 Simon Fraser <Simon Fraser>

Reviewed by Darin Adler.

Video that loads inside reflection doesn't get reflected
​https://bugs.webkit.org/show_bug.cgi?id=34046

ensureCloneLayers() made an incorrect assumption that if a clone of the primaryLayer
exists already, then the structural layer and content layer can just be returned.
However, this doesn't account for the situation where a content layer may have
been create after the clone tree was last updated.

Fix by always creating the structural and clone layers on demand.

Test: compositing/reflections/load-video-in-reflection.html

  • platform/graphics/mac/GraphicsLayerCA.h: Added utility method, findOrMakeClone().
  • platform/graphics/mac/GraphicsLayerCA.mm: (WebCore::GraphicsLayerCA::findOrMakeClone): Utility that efficiently finds or creates a layer in a LayerMap. (WebCore::GraphicsLayerCA::ensureCloneLayers): Always look for or clone the structural and content layers, rather than relying on whether there's a primary layer clone already.
4:25 AM Changeset in webkit [53778] by abarth@webkit.org
  • 2 edits in trunk/WebKitTools

Rubber stamped by Eric Seidel.

More pep8 compliance.

  • Scripts/webkitpy/mock_bugzillatool.py:
4:16 AM Changeset in webkit [53777] by abarth@webkit.org
  • 3 edits in trunk/WebKitTools

Rubber stamped by Eric Seidel.

More pep8 compliance.

  • Scripts/webkitpy/executive.py:
  • Scripts/webkitpy/grammar.py:
3:57 AM Changeset in webkit [53776] by abarth@webkit.org
  • 4 edits in trunk/WebKitTools

Rubber stamped by Eric Seidel.

More pep8 compliance.

  • Scripts/webkitpy/comments.py:
  • Scripts/webkitpy/committers.py:
  • Scripts/webkitpy/credentials.py:
3:48 AM Changeset in webkit [53775] by abarth@webkit.org
  • 2 edits in trunk/WebKitTools

Rubber stamped by Eric Seidel.

Make changelogs.py pass pep8.

  • Scripts/webkitpy/changelogs.py:
Note: See TracTimeline for information about the timeline view.