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

Timeline



Apr 11, 2009:

10:49 PM Changeset in webkit [42425] by adele@apple.com
  • 8 edits
    2 adds in trunk

WebCore:

2009-04-11 Adele Peterson <​adele@apple.com>

Reviewed by Dan Bernstein.

Fix for <rdar://problem/6609479> Pressing return inside a table cell that's inside quoted content will split the table

Test: editing/inserting/6609479.html

Don't break a blockquote if pasting or inserting into a table.

  • editing/ReplaceSelectionCommand.cpp: (WebCore::ReplaceSelectionCommand::doApply):
  • editing/TypingCommand.cpp: (WebCore::TypingCommand::insertParagraphSeparatorInQuotedContent):

LayoutTests:

2009-04-11 Adele Peterson <​adele@apple.com>

Reviewed by Dan Bernstein.

Tests for <rdar://problem/6609479> Pressing return inside a table cell that's inside quoted content will split the table

  • editing/inserting/5418891.html: Changed test so it doesn't involve tables.
  • editing/inserting/6609479-expected.txt: Added.
  • editing/inserting/6609479.html: Added.
  • platform/mac/editing/inserting/5418891-expected.checksum:
  • platform/mac/editing/inserting/5418891-expected.png:
  • platform/mac/editing/inserting/5418891-expected.txt:
6:11 PM Changeset in webkit [42424] by mitz@apple.com
  • 3 edits
    2 adds in trunk

WebCore:

Reviewed by Darin Adler.

  • fix <rdar://problem/6643103> Unscaled values from body.scrollHeight

Test: fast/css/zoom-body-scroll.html

Adjust scroll{Left,Top,Width,Height} values for zoom in getters and
setters.

  • html/HTMLBodyElement.cpp: (WebCore::adjustForZoom): (WebCore::HTMLBodyElement::scrollLeft): (WebCore::HTMLBodyElement::setScrollLeft): (WebCore::HTMLBodyElement::scrollTop): (WebCore::HTMLBodyElement::setScrollTop): (WebCore::HTMLBodyElement::scrollHeight): (WebCore::HTMLBodyElement::scrollWidth):

LayoutTests:

Reviewed by Darin Adler.

  • test for <rdar://problem/6643103> Unscaled values from body.scrollHeight
  • fast/css/zoom-body-scroll-expected.txt: Added.
  • fast/css/zoom-body-scroll.html: Added.

Apr 10, 2009:

10:40 PM Changeset in webkit [42423] by mitz@apple.com
  • 2 edits in trunk/WebKit/mac

Reviewed by Jon Honeycutt.

  • fix <rdar://problem/6752340> Light blue-green background in content area in Mail
  • WebView/WebView.mm: (-[WebView _commonInitializationWithFrameName:groupName:usesDocumentViews:]): Use device white, rather than calibrated white, as the default background color.
10:31 PM Changeset in webkit [42422] by timothy@apple.com
  • 8 edits in trunk/WebCore

Allow page navigation and reload while stopped at a breakpoint.

<rdar://problem/6781108> Having a breakpoint active prevents page reload

Reviewed by Darin Adler.

  • inspector/InspectorController.cpp: (WebCore::InspectorController::didContinue): Call the resumedScript function.
  • inspector/InspectorController.h: Added didContinue.
  • inspector/JavaScriptDebugListener.h: Ditto.
  • inspector/JavaScriptDebugServer.cpp: (WebCore::JavaScriptDebugServer::pauseIfNeeded): Call didContinue on the listeners.
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype.debuggerResumed): Update the interface and state.
  • inspector/front-end/inspector.js: (WebInspector.resumedScript): Call ScriptsPanel.debuggerResumed.
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::continueLoadAfterNavigationPolicy): Call resumeDebugger.
7:18 PM Changeset in webkit [42421] by Adam Roben
  • 2 edits in trunk/LayoutTests

Land corrected results for complex-character-based-fallback.html

  • platform/win/fast/text/international/complex-character-based-fallback-expected.txt:
5:56 PM Changeset in webkit [42420] by Darin Adler
  • 1 edit
    1 move in trunk/LayoutTests

2009-04-10 Darin Adler <Darin Adler>

Rubber stamped by Adam Roben.

Disabled fast/dom/gc-8.html, which was using Timer in an unsafe way that now
triggers an assertion.

We'll reenable the test after fixing the problem. But this is a test-only problem.

  • fast/dom/gc-8.html: Removed.
  • fast/dom/gc-8.html-disabled: Copied from fast/dom/gc-8.html.
5:47 PM Changeset in webkit [42419] by sfalken@apple.com
  • 2 edits in trunk/WebKit/win

<rdar://problem/6676024> REGRESSION (PB-40B1): Safari crashes on launch on tablet PC


r21436 changed the semantics of AXObjectCache::get() to not create
objects on demand. Changing this get() call into a getOrCreate() call
fixes the crash.


Rubber stamped by Jon Honeycutt.

  • AccessibleDocument.cpp: (AccessibleDocument::AccessibleDocument):
5:43 PM Changeset in webkit [42418] by weinig@apple.com
  • 2 edits in trunk/WebCore

2009-04-10 Sam Weinig <​sam@webkit.org>

Reviewed by Adele Peterson.

Remove unused include for JSLock.

  • page/mac/FrameMac.mm:
5:41 PM Changeset in webkit [42417] by weinig@apple.com
  • 2 edits in trunk/WebCore

2009-04-10 Sam Weinig <​sam@webkit.org>

Reviewed by Darin Adler.

Add back the keepAlive timer for after the script execution to
protect the callers of evaluate, as was part of the original intent
of timer.

  • bindings/js/ScriptController.cpp: (WebCore::ScriptController::evaluate):
  • page/mac/FrameMac.mm:
5:39 PM Changeset in webkit [42416] by mitz@apple.com
  • 7 edits
    3 adds in trunk/LayoutTests

Reviewed by Anders Carlsson.

  • test and expected result updates following r42379
  • fast/text/offsetForPosition-cluster-at-zero-expected.txt: Updated results.
  • fast/text/offsetForPosition-cluster-at-zero.html: Removed font-dependent alert that was accidently left in this test.
  • platform/mac-tiger/fast/text/international/thai-line-breaks-expected.checksum:
  • platform/mac-tiger/fast/text/international/thai-line-breaks-expected.png:
  • platform/mac-tiger/fast/text/international/thai-line-breaks-expected.txt:
  • platform/mac-tiger/fast/text/monospace-width-cache-expected.checksum: Added.
  • platform/mac-tiger/fast/text/monospace-width-cache-expected.png: Added.
  • platform/mac-tiger/fast/text/monospace-width-cache-expected.txt: Added.
  • platform/win/Skipped:
5:26 PM Changeset in webkit [42415] by cmarrin@apple.com
  • 2 edits in trunk/WebCore

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

If we reload, the animation controller sticks around and has
a stale animation time. So the needed delay is computed wrong.
I added a reset of the animation timer on load.

I did not do a test case because this only happens on reload so I
couldn't make an automated test. I tried using an iframe and using
reload() but that doesn't tickle the bug.

5:13 PM Changeset in webkit [42414] by Adam Roben
  • 2 edits in trunk/WebKitTools

Add our new test font to the list that DRT knows about

Reviewed by Dan Bernstein.

  • DumpRenderTree/win/DumpRenderTree.cpp: (initialize): Added WebKit Layout Tests 2.ttf.
5:10 PM Changeset in webkit [42413] by Darin Adler
  • 4 edits in trunk/WebCore

2009-04-10 Grace Kloba <​klobag@gmail.com>

Reviewed by Darin Adler.

Move WORKERS header files inside ENABLE(WORKERS).
So if WORKERS is not enabled and the generated files are not generated, the build is still working.

  • bindings/js/JSDOMGlobalObject.cpp:
  • bindings/js/JSMessageChannelConstructor.cpp:
  • bindings/js/ScheduledAction.cpp:
4:59 PM Changeset in webkit [42412] by Darin Adler
  • 7 edits in trunk

WebCore:

2009-04-10 Darin Adler <Darin Adler>

Reviewed by Brady Eidson.

<rdar://problem/6773515> crash in push_heap inside WebCore when printing

The crash was due to manipulating a timer on a different thread than the one it was created on.
This adds some diagnostics so we'll catch that kind of mistake immediately.

  • platform/Timer.cpp: (WebCore::TimerBase::TimerBase): Store the thread identifier in debug versions. (WebCore::TimerBase::start): Assert it's the same thread the timer was created on. (WebCore::TimerBase::stop): Ditto. (WebCore::TimerBase::isActive): Ditto. (WebCore::TimerBase::setNextFireTime): Ditto.
  • platform/Timer.h: Added the thread identifier data member.

WebKit/mac:

2009-04-10 Darin Adler <Darin Adler>

Reviewed by Brady Eidson.

<rdar://problem/6773515> crash in push_heap inside WebCore when printing

The crash was due to manipulating a timer on a different thread than the one it was created on.

  • History/WebHistoryItem.mm: (-[WebWindowWatcher windowWillClose:]): Call later on main thread, if called on non-main thread.
  • WebView/WebHTMLView.mm: (-[WebHTMLView windowDidBecomeKey:]): Ditto. (-[WebHTMLView windowDidResignKey:]): Ditto. (-[WebHTMLView windowWillClose:]): Ditto. (-[WebHTMLView _updateControlTints]): Added. Factored out the non-thread-safe part of our override of _windowChangedKeyState. (-[WebHTMLView _windowChangedKeyState]): Call _updateControlTints later on main thread, if called on non-main thread.
  • WebView/WebPreferences.mm: (-[WebPreferences _postPreferencesChangesNotification]): Call later on main thread, if called on non-main thread
4:39 PM Changeset in webkit [42411] by Adam Roben
  • 2 edits in trunk/WebKit/win

Add our invidual IDL files back to Interfaces.vcproj, but disable them

This allows the IDL files to be found in Solution Explorer.

Rubber-stamped in advance by Steve Falkenburg.

  • WebKit.vcproj/Interfaces.vcproj:
4:19 PM Changeset in webkit [42410] by Simon Fraser
  • 5 edits
    2 adds in trunk

2009-04-10 Chris Marrin <​cmarrin@apple.com>

Reviewed by Dan Bernstein

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

If you remove a class with a transition while that transition is running
the animation timer will continue to fire after the transition is finished.
This has no visual indication, but it does drain the processor. And in some
cases it might even cause a glitch in future animations. Unfortunately there
is no way to test this without putting in printfs.

This happens because the animation logic is never traversed after a transition
is removed, so we never get a chance to cleanup. So I added cleanup in the logic
that fires the dispatch of the last style change when the animation finishes.

Test: transitions/remove-transition-style.html

  • page/animation/AnimationController.cpp: (WebCore::AnimationControllerPrivate::updateStyleIfNeededDispatcherFired):
  • page/animation/CompositeAnimation.cpp: (WebCore::CompositeAnimation::updateTransitions): (WebCore::CompositeAnimation::animate): (WebCore::CompositeAnimation::cleanupFinishedAnimations):
  • page/animation/CompositeAnimation.h:
3:52 PM Changeset in webkit [42409] by weinig@apple.com
  • 2 edits in trunk/WebCore

2009-04-10 Sam Weinig <​sam@webkit.org>

Reviewed by Geoffrey Garen.

Fix for <rdar://problem/6648858>
CrashTracer: [REGRESSION(Safari 4)] 60 crashes in Safari at com.apple.WebCore • WebCore::ScriptController::evaluate + 241

The Frame (and therefore ScriptController) were being destroyed
during JavaScript execution causing the JSDOMWindowShell to become
null.

  • bindings/js/ScriptController.cpp: (WebCore::ScriptController::evaluate): Use a RefPtr to protect the Frame instead of the keep-alive timer, since a nested event loop used from with-in JavaScript execution could cause the timer to fire before returning.
3:30 PM ApplicationsGtk edited by evandrofisico@gmail.com
Correct claws-mail webkit plugin URL (diff)
3:27 PM Changeset in webkit [42408] by Dimitri Glazkov
  • 7 edits in trunk/WebCore

2009-04-10 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Timothy Hatcher.

​https://bugs.webkit.org/show_bug.cgi?id=25133
Fix an error in InspectorController's bindings, clean up a few header
includes, make empty object construction in ScriptObject less ambiguous.

  • bindings/js/JSInspectorControllerCustom.cpp: (WebCore::JSInspectorController::setSetting): Send correct argument from

the bindings.

  • bindings/js/ScriptObject.cpp: (WebCore::ScriptObject::createNew): Added.
  • bindings/js/ScriptObject.h: Removed constructor, added decl for createNew.
  • inspector/ConsoleMessage.cpp: Remove unused header.
  • inspector/ConsoleMessage.h: Ditto.
  • inspector/InspectorResource.cpp: (WebCore::createHeadersObject): Changed to use ScriptObject::createNew.
2:50 PM Changeset in webkit [42407] by mitz@apple.com
  • 3 edits in trunk/LayoutTests

Reviewed by Adam Roben.

  • re-enable css2.1 tests that were disabled following r42379 and make them use a different fallback font for certain characters, to get the vertical metrics to match Mac
  • platform/win/Skipped:
  • platform/win/css2.1/resources/Mac-compatible-font-fallback.css:
2:44 PM Changeset in webkit [42406] by cmarrin@apple.com
  • 3 edits in trunk/WebCore

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

This both fixes a bug where I was not sending in the right time
to continue the animation and was not recomputing current time
when coming out of suspend.

1:48 PM Changeset in webkit [42405] by eric.carlson@apple.com
  • 3 edits in trunk/WebKitLibraries

2009-04-10 Eric Carlson <​eric.carlson@apple.com>

WebKitSystemInterface changes for <rdar://problem/6646998>

  • libWebKitSystemInterfaceLeopard.a:
  • libWebKitSystemInterfaceTiger.a:
1:41 PM Changeset in webkit [42404] by timothy@apple.com
  • 1 edit in trunk/WebCore/bindings/js/ScriptCachedFrameData.cpp

Revert an unintentional logic change in r42400.

1:17 PM Changeset in webkit [42403] by Simon Fraser
  • 3 edits in trunk/WebKitLibraries

2009-04-10 Simon Fraser <Simon Fraser>

Fix the leopard build by updating WebKitSystemInterface.

  • WebKitSystemInterface.h:
  • libWebKitSystemInterfaceLeopard.a:
1:07 PM Changeset in webkit [42402] by timothy@apple.com
  • 3 edits
    1 delete in trunk/WebKit

Remove DOMDocumentPrivate.h now that <rdar://problem/6730996> is fixed.

WebKit:

2009-04-10 Timothy Hatcher <​timothy@apple.com>

Remove DOMDocumentPrivate.h now that <rdar://problem/6730996> is fixed.

Rubber-stamped by Mark Rowe.

  • WebKit.xcodeproj/project.pbxproj:

WebKit/mac:

2009-04-10 Timothy Hatcher <​timothy@apple.com>

Remove DOMDocumentPrivate.h now that <rdar://problem/6730996> is fixed.

Rubber-stamped by Mark Rowe.

  • Misc/DOMDocumentPrivate.h: Removed.
1:05 PM Changeset in webkit [42401] by Adam Roben
  • 2 edits in trunk/WebCore

Give Windows's <select> popup menus drop shadows

Rubber-stamped by Steve Falkenburg.

No test possible.

  • platform/win/PopupMenuWin.cpp: (WebCore::registerPopup): Use the CS_DROPSHADOW window class style.
1:00 PM Changeset in webkit [42400] by timothy@apple.com
  • 4 edits in trunk/WebCore

Fix a bug in the Web Inspector where it would show scripts
from previous pages when toggling the debugger off and back on.

​https://bugs.webkit.org/show_bug.cgi?id=25095
rdar://problem/6769970

Reviewed by John Sullivan.

  • bindings/js/ScriptCachedFrameData.cpp: (WebCore::ScriptCachedFrameData::ScriptCachedFrameData): Null out the debugger for the scriptController. This balances the attachDebugger that was already in ScriptCachedFrameData::restore. (WebCore::ScriptCachedFrameData::restore): Clean up an if statement.
  • bindings/js/ScriptController.cpp: (WebCore::ScriptController::clearWindowShell): Clear the debugger from the current window before setting the new window.
  • inspector/JavaScriptDebugServer.cpp: (WebCore::JavaScriptDebugServer::recompileAllJSFunctions): Only accumulate source providers for code associated this as the debugger.
12:17 PM Changeset in webkit [42399] by eric.carlson@apple.com
  • 7 edits in trunk

2009-04-10 Pierre d'Herbemont <​pdherbemont@apple.com>

Reviewed by Adele Peterson.

<rdar://problem/6646998> Avoid starting QTKitServer if possible

When possible avoid asking QTKit for the MIME types that require QTKitServer
when running 64-bit.

  • WebCore.base.exp: Added new entry point.
  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::addFileTypesToCache): Add an NSArray of types to the cache. (WebCore::mimeCommonTypesCache): Add all "common" QTKit types to the cache. (WebCore::mimeModernTypesCache): Add only "modern" QTKit types to the cache (this list does not start QTKitServer if it is not already running). (WebCore::MediaPlayerPrivate::getSupportedTypes): Refactored. (WebCore::MediaPlayerPrivate::supportsType): Look in list of "modern" types first to avoid starting QTKitServer if possible.
  • platform/mac/WebCoreSystemInterface.h: Add wkQTIncludeOnlyModernMediaFileTypes.
  • platform/mac/WebCoreSystemInterface.mm: Ditto.

2009-04-10 Pierre d'Herbemont <​pdherbemont@apple.com>

Reviewed by Adele Peterson.

<rdar://problem/6646998> Avoid starting QTKitServer if possible

  • WebCoreSupport/WebSystemInterface.m: (InitWebCoreSystemInterface): Init QTIncludeOnlyModernMediaFileTypes.
11:55 AM Changeset in webkit [42398] by Adam Roben
  • 2 edits in trunk/WebCore

Turn on window animations for <select> popup menus

We turned these off back when <select> popup menus were partially
transparent. Now that they're opaque, there's no reason not to
animate them.

Reviewed by Steve Falkenburg.

No test possible.

  • platform/win/PopupMenuWin.cpp: (WebCore::PopupMenu::show): Remove the #ifdef that was disabling the window animation.
11:39 AM Changeset in webkit [42397] by mitz@apple.com
  • 2 edits in trunk/LayoutTests
  • skip failures following r42379 until they are addressed by code changes or test changes
  • platform/win/Skipped:
11:32 AM Changeset in webkit [42396] by mitz@apple.com
  • 6 edits in trunk

WebCore:

Reviewed by Sam Weinig.

  • fix a regression from r42379 which made zero-width characters affect line height

Test: fixes fast/text/wide-zero-width-space.html

  • platform/graphics/mac/FontMacATSUI.mm: (WebCore::overrideLayoutOperation): Add to the fallback font list set here, and only if the glyph is not zero-width... (WebCore::ATSULayoutParameters::initialize): ...instead of doing it here. This matches WidthIterator's behavior.

LayoutTests:

Reviewed by Sam Weinig.

  • revert broken test results that were checked in with r42379, now that the regression is fixed
  • platform/mac/fast/text/wide-zero-width-space-expected.checksum:
  • platform/mac/fast/text/wide-zero-width-space-expected.png:
  • platform/mac/fast/text/wide-zero-width-space-expected.txt:
11:31 AM Changeset in webkit [42395] by Adam Roben
  • 4 edits in trunk/LayoutTests

Update Windows results for complex-character-based-fallback.html

Rubber-stamped by Dan Bernstein.

  • platform/win/fast/text/international/complex-character-based-fallback-expected.checksum:
  • platform/win/fast/text/international/complex-character-based-fallback-expected.png:
  • platform/win/fast/text/international/complex-character-based-fallback-expected.txt: Updated.
10:39 AM Changeset in webkit [42394] by darin@chromium.org
  • 2 edits in trunk/WebCore

2009-04-10 Darin Fisher <​darin@chromium.org>

Fixing build bustage caused by warnings being treated as errors.
1- A local variable was only be used inside a debug assertion, which

caused grief for the release build.

2- Missing initializer for member 'DocumentMarker::activeMatch'

  • dom/Document.cpp: (WebCore::Document::addMarker): (WebCore::Document::setMarkersActive):
10:24 AM Changeset in webkit [42393] by darin@chromium.org
  • 9 edits in trunk/WebCore

2009-04-10 Finnur Thorarinsson <​finnur.webkit@gmail.com>

Reviewed by John Sullivan.

Fixing: ​https://bugs.webkit.org/show_bug.cgi?id=25117
(TextMatches don't have a concept of active match)

For the ports that use the DocumentMarker as highlighting for FindInPage it is
useful to have a flag that specifies which marker is active, so it can be drawn
in a different color.

This allows me to remove the hack to use the SelectionController to highlight
the active match (in the ChromiumWin port).

  • dom/Document.cpp: (WebCore::Document::setMarkersActive):
  • dom/Document.h:
  • dom/DocumentMarker.h:
  • rendering/InlineTextBox.cpp: (WebCore::InlineTextBox::paintTextMatchMarker):
  • rendering/RenderTheme.cpp: (WebCore::RenderTheme::platformActiveTextSearchHighlightColor): (WebCore::RenderTheme::platformInactiveTextSearchHighlightColor):
  • rendering/RenderTheme.h:
  • rendering/RenderThemeChromiumWin.cpp: (WebCore::RenderThemeChromiumWin::platformActiveSelectionBackgroundColor): (WebCore::RenderThemeChromiumWin::platformInactiveSelectionBackgroundColor): (WebCore::RenderThemeChromiumWin::platformActiveTextSearchHighlightColor): (WebCore::RenderThemeChromiumWin::platformInactiveTextSearchHighlightColor):
  • rendering/RenderThemeChromiumWin.h:
10:13 AM Changeset in webkit [42392] by pam@chromium.org
  • 2 edits in trunk/WebCore

2009-04-10 Pamela Greene <​pam@chromium.org>

Reviewed by Dimitri Glazkov.

Build fix for Chromium.
​https://bugs.webkit.org/show_bug.cgi?id=12440

  • page/chromium/FrameChromium.cpp: (WebCore::computePageRectsForFrame): change docWidth() to overflowWidth()
5:03 AM Changeset in webkit [42391] by kov@webkit.org
  • 13 edits in trunk/WebKit/gtk

WebKit/gtk/

2009-04-10 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Reviewed by Holger Freyther.

​https://bugs.webkit.org/show_bug.cgi?id=25124
[GTK] many strings still missing translation calls

Mark the missing strings for translation.

  • webkit/webkitdownload.cpp: (_WebKitDownloadPrivate::webkit_download_cancel):
  • webkit/webkitsoupauthdialog.c: (show_auth_dialog):
  • webkit/webkitwebhistoryitem.cpp: (_WebKitWebHistoryItemPrivate::webkit_web_history_item_class_init):
  • webkit/webkitwebinspector.cpp: (_WebKitWebInspectorPrivate::webkit_web_inspector_class_init):
  • webkit/webkitwebnavigationaction.cpp: (_WebKitWebNavigationActionPrivate::webkit_web_navigation_action_class_init):
  • webkit/webkitwebsettings.cpp: (_WebKitWebSettingsPrivate::webkit_web_settings_class_init):
  • webkit/webkitwebview.cpp:

WebKit/gtk/po/

2009-04-10 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Reviewed by Holger Freyther.

​https://bugs.webkit.org/show_bug.cgi?id=25124
[GTK] many strings still missing translation calls

Extracting translation for the files that got translation calls.

  • POTFILES:
  • de.po:
  • pt_BR.po:
  • webkit.pot:
Note: See TracTimeline for information about the timeline view.