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

Timeline



Apr 27, 2009:

11:05 PM Changeset in webkit [42921] by weinig@apple.com
  • 1 edit
    1 move in trunk/LayoutTests

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

Reviewed by Mark Rowe.

Disable test intermittent failing test.

  • svg/custom/filter-source-alpha.svg: Removed.
  • svg/custom/filter-source-alpha.svg-disabled: Copied from svg/custom/filter-source-alpha.svg.
10:14 PM Changeset in webkit [42920] by Simon Fraser
  • 3 edits
    5 adds in trunk/LayoutTests

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

Reviewed by Simon Fraser

Fix video-played.html layout test.

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

We implement playForMillisecs() which makes sure the video
will have been playing for a minimum time.

Note, this does not ensure a strict synchronization between the time
asked and the media playback. But given that media playback is the
slow part we just need to compensate for the delay involved by playback.
(And not the other way around.)

Moreover, we split the test in 3 in order to be more gentle which
slower machines that may tend to timeout more easily on those tests.

  • media/video-played-collapse-expected.txt: Added.
  • media/video-played-collapse.html: Added.
  • media/video-played-expected.txt:
  • media/video-played-reset-expected.txt: Added.
  • media/video-played-reset.html: Added.
  • media/video-played.html:
  • Split tests in multiple files.
  • Makes sure we always play as much as requested by using playForMillisecs();
  • media/video-played.js: Added. (testRanges): (nextTest): (pause): (canplay): (willCreateNewRange): (startPlayingInNewRange): (startPlaying): (secToMilli): (milliToSecs): (playForMillisecs.callPauseIfTimeIsReached): (playForMillisecs): (videoPlayedMain):
8:39 PM Changeset in webkit [42919] by Dimitri Glazkov
  • 2 edits in trunk/WebCore

2009-04-27 Eric Roman <​eroman@chromium.org>

Reviewed by Darin Fisher.

Fix a compile breakage.
<​https://bugs.webkit.org/show_bug.cgi?id=25384>

  • bindings/v8/custom/V8DOMWindowCustom.cpp: (WebCore::CALLBACK_FUNC_DECL):
6:01 PM Changeset in webkit [42918] by eric.carlson@apple.com
  • 3 edits in trunk/WebCore

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

Reviewed by Darin Adler.

<rdar://problem/6822344> Use of QTMovieCurrentSizeAttribute generates exception and will crash

QTMovieCurrentSizeAttribute generates an exception with some versions of QTKit, so calculate a
multiplier to scale from natural size to current size when a movie is opened and use that to
return the correct value from the naturalSize() method.

  • platform/graphics/mac/MediaPlayerPrivateQTKit.h:
  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::MediaPlayerPrivate): Initialize m_scaleFactor. (WebCore::MediaPlayerPrivate::naturalSize): Return naturalSize transformed by initial scale. (WebCore::MediaPlayerPrivate::cacheMovieScale): New, calculate difference between initial size and natural size so naturalSize() accounts for non-identity movie matrix. (WebCore::MediaPlayerPrivate::updateStates): Call cacheMovieScale when load state reaches QTMovieLoadStateLoaded for the first time.
5:54 PM Changeset in webkit [42917] by Beth Dakin
  • 3 edits
    4 adds in trunk

WebCore:

2009-04-27 Beth Dakin <Beth Dakin>

Reviewed by Dave Hyatt.

Fix for <rdar://problem/6709057> REGRESSION (4PB-TOT): ​http://www.winway.com/main3/support/faq.aspx selector doesn't look right or work

The problem here is that we had a self-painting layer nested inside
a non-self-painting layer. We ended up ignoring the self-painting
layer both during painting and hit-testing. This patch corrects
that.

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

LayoutTests:

2009-04-27 Beth Dakin <Beth Dakin>

Reviewed by Dave Hyatt.

Test for <rdar://problem/6709057> REGRESSION (4PB-TOT): ​http://www.winway.com/main3/support/faq.aspx selector doesn't look right or work

  • fast/css/nested-layers-with-hover.html: Added.
  • platform/mac/fast/css/nested-layers-with-hover-expected.checksum: Added.
  • platform/mac/fast/css/nested-layers-with-hover-expected.png: Added.
  • platform/mac/fast/css/nested-layers-with-hover-expected.txt: Added.
5:23 PM Changeset in webkit [42916] by adachan@apple.com
  • 2 edits in trunk/WebCore

If it has been set to allow any HTTPS certificates for this host, set
kCFStreamSSLAllowsExpiredCertificates to true to allow expired
certificate, and set kCFStreamSSLValidatesCertificateChain false to skip
certificate chain validation. (<rdar://problem/6382059>)

Reviewed by John Sullivan.

  • platform/network/cf/ResourceHandleCFNet.cpp: (WebCore::makeFinalRequest):
5:12 PM Changeset in webkit [42915] by Simon Fraser
  • 1 edit
    4 adds in trunk/LayoutTests

2009-04-23 Simon Fraser <Simon Fraser>

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

Add pixel test for zoomed video with controls, with and without a transform.

  • media/video-zoom-controls.html: Added.
  • platform/mac/media/video-zoom-controls-expected.checksum: Added.
  • platform/mac/media/video-zoom-controls-expected.png: Added.
  • platform/mac/media/video-zoom-controls-expected.txt: Added.
5:08 PM Changeset in webkit [42914] by Simon Fraser
  • 4 edits in trunk

2009-04-27 Simon Fraser <Simon Fraser>

Reviewed by Adele Peterson

<rdar://problem/6822225> Full page zooming of <video> element in the browser window doesn't work well

Fix a 'size changed' notification thrash that caused standalone video elements
to continually resize after full page zooming, by using the movie's natual size (which is independent
of zooming), rather than its current size (which is not).

Note that this regresses <​https://bugs.webkit.org/show_bug.cgi?id=25029>, so we have to
disable the media/video-size-intrinsic-scale.html test. However, we have to stop using
QTMovieCurrentSizeAttribute anyway; this will be addressed, and the test re-enabled via
<rdar://problem/6822344>.

  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::naturalSize):
4:58 PM Changeset in webkit [42913] by Simon Fraser
  • 3 edits in trunk/WebCore

2009-04-27 Simon Fraser <Simon Fraser>

Reviewed by Darin Adler

<rdar://problem/6642221> REGRESSION: With the Movie widget, movie trailer fails to
load if movie had been previously viewed

Fix the Dashboard quirk for display:none plug-ins by moving the code from HTMLObjectElement
to HTMLEmebedElement. It has to be in HTMLEmbedElement because the content we care about uses <embed>.

  • html/HTMLEmbedElement.cpp: (WebCore::HTMLEmbedElement::rendererIsNeeded):
  • html/HTMLObjectElement.cpp: (WebCore::HTMLObjectElement::rendererIsNeeded):
4:33 PM Changeset in webkit [42912] by weinig@apple.com
  • 14 edits in trunk/WebCore

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

Reviewed by Geoffrey Garen.

Move more window event related code into DOMWindow.

  • dom/Document.cpp: (WebCore::Document::implicitClose): (WebCore::Document::setWindowAttributeEventListener): (WebCore::Document::dispatchWindowEvent): (WebCore::Document::dispatchLoadEvent):
  • dom/Document.h:
  • dom/Element.cpp:
  • dom/Element.h:
  • dom/Node.cpp:
  • dom/Node.h:
  • html/HTMLBodyElement.cpp: (WebCore::HTMLBodyElement::parseMappedAttribute):
  • html/HTMLFrameSetElement.cpp: (WebCore::HTMLFrameSetElement::parseMappedAttribute):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::stopLoading):
  • page/DOMWindow.cpp: (WebCore::DOMWindow::dispatchAllPendingUnloadEvents): (WebCore::DOMWindow::postMessageTimerFired): (WebCore::DOMWindow::dispatchEvent): (WebCore::DOMWindow::dispatchEventWithDocumentAsTarget): (WebCore::DOMWindow::dispatchLoadEvent): (WebCore::DOMWindow::dispatchUnloadEvent): (WebCore::DOMWindow::dispatchBeforeUnloadEvent):
  • page/DOMWindow.h:
  • page/Frame.cpp: (WebCore::Frame::shouldClose):
  • svg/SVGSVGElement.cpp: (WebCore::SVGSVGElement::parseMappedAttribute):
4:14 PM Changeset in webkit [42911] by adele@apple.com
  • 25 edits in trunk

WebCore:

2009-04-27 Douglas R. Davidson <​ddavidso@apple.com>

Implement the editing mechanisms and add context menu items for
<rdar://problem/6724106> WebViews need to implement text checking

Reviewed by Justin Garcia.

  • editing/Editor.cpp:
  • editing/Editor.h:
  • editing/TypingCommand.cpp:
  • loader/EmptyClients.h:
  • page/ContextMenuController.cpp:
  • page/EditorClient.h:
  • page/mac/WebCoreViewFactory.h:
  • platform/ContextMenu.cpp:
  • platform/ContextMenuItem.h:
  • platform/LocalizedStrings.h:
  • platform/mac/LocalizedStringsMac.mm:

WebKit:

2009-04-27 Douglas R. Davidson <​ddavidso@apple.com>

Add localizable strings for
<rdar://problem/6724106> WebViews need to implement text checking

Reviewed by Justin Garcia.

  • English.lproj/Localizable.strings:

WebKit/mac:

2009-04-27 Douglas R. Davidson <​ddavidso@apple.com>

Add the various switches and context menu items needed for
<rdar://problem/6724106> WebViews need to implement text checking
and adopt updatePanels in place of old SPI _updateGrammar.

Reviewed by Justin Garcia.

  • WebCoreSupport/WebContextMenuClient.mm: (fixMenusReceivedFromOldClients):
  • WebCoreSupport/WebEditorClient.h:
  • WebCoreSupport/WebEditorClient.mm: (WebEditorClient::uppercaseWord): (WebEditorClient::lowercaseWord): (WebEditorClient::capitalizeWord): (WebEditorClient::showSubstitutionsPanel): (WebEditorClient::substitutionsPanelIsShowing): (WebEditorClient::toggleSmartInsertDelete): (WebEditorClient::isAutomaticQuoteSubstitutionEnabled): (WebEditorClient::toggleAutomaticQuoteSubstitution): (WebEditorClient::isAutomaticLinkDetectionEnabled): (WebEditorClient::toggleAutomaticLinkDetection): (WebEditorClient::isAutomaticDashSubstitutionEnabled): (WebEditorClient::toggleAutomaticDashSubstitution): (WebEditorClient::isAutomaticTextReplacementEnabled): (WebEditorClient::toggleAutomaticTextReplacement): (WebEditorClient::isAutomaticSpellingCorrectionEnabled): (WebEditorClient::toggleAutomaticSpellingCorrection): (WebEditorClient::checkTextOfParagraph):
  • WebCoreSupport/WebViewFactory.mm: (-[WebViewFactory contextMenuItemTagShowColors]): (-[WebViewFactory contextMenuItemTagCorrectSpellingAutomatically]): (-[WebViewFactory contextMenuItemTagSubstitutionsMenu]): (-[WebViewFactory contextMenuItemTagShowSubstitutions:]): (-[WebViewFactory contextMenuItemTagSmartCopyPaste]): (-[WebViewFactory contextMenuItemTagSmartQuotes]): (-[WebViewFactory contextMenuItemTagSmartDashes]): (-[WebViewFactory contextMenuItemTagSmartLinks]): (-[WebViewFactory contextMenuItemTagTextReplacement]): (-[WebViewFactory contextMenuItemTagTransformationsMenu]): (-[WebViewFactory contextMenuItemTagMakeUpperCase]): (-[WebViewFactory contextMenuItemTagMakeLowerCase]): (-[WebViewFactory contextMenuItemTagCapitalize]):
  • WebView/WebHTMLView.mm: (-[WebHTMLView validateUserInterfaceItemWithoutDelegate:]): (-[WebHTMLView isAutomaticQuoteSubstitutionEnabled]): (-[WebHTMLView setAutomaticQuoteSubstitutionEnabled:]): (-[WebHTMLView toggleAutomaticQuoteSubstitution:]): (-[WebHTMLView isAutomaticLinkDetectionEnabled]): (-[WebHTMLView setAutomaticLinkDetectionEnabled:]): (-[WebHTMLView toggleAutomaticLinkDetection:]): (-[WebHTMLView isAutomaticDashSubstitutionEnabled]): (-[WebHTMLView setAutomaticDashSubstitutionEnabled:]): (-[WebHTMLView toggleAutomaticDashSubstitution:]): (-[WebHTMLView isAutomaticTextReplacementEnabled]): (-[WebHTMLView setAutomaticTextReplacementEnabled:]): (-[WebHTMLView toggleAutomaticTextReplacement:]): (-[WebHTMLView isAutomaticSpellingCorrectionEnabled]): (-[WebHTMLView setAutomaticSpellingCorrectionEnabled:]): (-[WebHTMLView toggleAutomaticSpellingCorrection:]):
  • WebView/WebHTMLViewInternal.h:
  • WebView/WebPreferenceKeysPrivate.h:
  • WebView/WebUIDelegatePrivate.h:
  • WebView/WebView.mm: (-[WebViewPrivate init]): (-[WebView validateUserInterfaceItemWithoutDelegate:]): (-[WebView setGrammarCheckingEnabled:]): (-[WebView isAutomaticQuoteSubstitutionEnabled]): (-[WebView isAutomaticLinkDetectionEnabled]): (-[WebView isAutomaticDashSubstitutionEnabled]): (-[WebView isAutomaticTextReplacementEnabled]): (-[WebView isAutomaticSpellingCorrectionEnabled]): (-[WebView setAutomaticQuoteSubstitutionEnabled:]): (-[WebView toggleAutomaticQuoteSubstitution:]): (-[WebView setAutomaticLinkDetectionEnabled:]): (-[WebView toggleAutomaticLinkDetection:]): (-[WebView setAutomaticDashSubstitutionEnabled:]): (-[WebView toggleAutomaticDashSubstitution:]): (-[WebView setAutomaticTextReplacementEnabled:]): (-[WebView toggleAutomaticTextReplacement:]): (-[WebView setAutomaticSpellingCorrectionEnabled:]): (-[WebView toggleAutomaticSpellingCorrection:]):
  • WebView/WebViewPrivate.h:
4:12 PM Changeset in webkit [42910] by weinig@apple.com
  • 2 edits in trunk/WebCore

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

Reviewed by Geoffrey Garen.

Fix for <rdar://problem/6772944> REGRESSION (r42223): PLT 2% slower, i-Bench 3% slower

Be slightly less eager in collecting after page close/navigation by increasing
the time before collection from 0 seconds to .5 seconds.

3% speedup on the PLT.

  • bindings/js/GCController.cpp: (WebCore::GCController::garbageCollectSoon):
4:06 PM Changeset in webkit [42909] by ddkilzer@apple.com
  • 10 edits
    2 adds in trunk

Consolidate runtime application checks for Apple Mail and Safari

Reviewed by Mark Rowe and Darin Adler.

WebCore:

  • WebCore.base.exp: Added exports.
  • WebCore.xcodeproj/project.pbxproj: Added RuntimeApplicationChecks.{h|mm} source files to the project.
  • page/mac/EventHandlerMac.mm: (WebCore::EventHandler::needsKeyboardEventDisambiguationQuirks): Removed local checkedSafari and isSafari variables and switched to use applicationIsSafari().
  • platform/mac/RuntimeApplicationChecks.h: Added.
  • platform/mac/RuntimeApplicationChecks.mm: Added. (WebCore::applicationIsAppleMail): Runtime application check for Apple Mail. (WebCore::applicationIsSafari): Runtime application check for Safari.

WebKit:

  • WebCoreSupport/WebContextMenuClient.mm: (isAppleMail): Removed. (fixMenusToSendToOldClients): Switched to use applicationIsAppleMail().
  • WebView/WebFrame.mm: (-[WebFrame reload]): Switched to use applicationIsSafari().
  • WebView/WebPDFView.mm: (-[WebPDFView menuForEvent:]): Ditto.
  • WebView/WebResource.mm: (+[WebResource _needMailThreadWorkaroundIfCalledOffMainThread]): Switched to use applicationIsAppleMail().
  • WebView/WebView.mm: (runningLeopardMail): Ditto. (runningTigerMail): Ditto. (-[WebView _needsKeyboardEventDisambiguationQuirks]): Switched to use applicationIsSafari().
3:34 PM Changeset in webkit [42908] by Chris Fleizach
  • 2 edits in trunk/WebCore

Bug 25374: AX: No AXValue change sent when text is auto-inserted
​https://bugs.webkit.org/show_bug.cgi?id=25374

Updated to fix regression test failures

2:56 PM Changeset in webkit [42907] by xan@webkit.org
  • 2 edits in trunk/WebCore

2009-04-16 Xan Lopez <​xlopez@igalia.com>

Reviewed by Gustavo Noronha.

​https://bugs.webkit.org/show_bug.cgi?id=21546
[GTK] ATK accessibility enhancements

Based on a patch by Alp Toker.

Cover more WebCore role -> ATK role conversions.

  • page/gtk/AccessibilityObjectWrapperAtk.cpp:
2:40 PM Changeset in webkit [42906] by xan@webkit.org
  • 2 edits in trunk/WebCore

2009-04-18 Xan Lopez <​xlopez@igalia.com>

Reviewed by Gustavo Noronha.

​https://bugs.webkit.org/show_bug.cgi?id=21546
[GTK] ATK accessibility enhancements

Do not call ATK setters in the getters, just return the
value. Among other things calling the setter will emit the notify
signal, which is wrong.

  • page/gtk/AccessibilityObjectWrapperAtk.cpp:
2:33 PM Changeset in webkit [42905] by xan@webkit.org
  • 2 edits in trunk/WebCore

2009-04-16 Xan Lopez <​xlopez@igalia.com>

Reviewed by Gustavo Noronha.

​https://bugs.webkit.org/show_bug.cgi?id=21546
[GTK] ATK accessibility enhancements

Based on a patch by Alp Toker.

Implement AtkImage interface.

  • page/gtk/AccessibilityObjectWrapperAtk.cpp:
2:25 PM Changeset in webkit [42904] by kov@webkit.org
  • 2 edits in trunk/WebCore

2009-04-27 Gustavo Noronha Silva <Gustavo Noronha Silva>

Unreviewed debug build fix.

  • platform/gtk/gtk2drawing.c: (moz_gtk_toggle_paint):
1:12 PM Changeset in webkit [42903] by Dimitri Glazkov
  • 3 edits in trunk/WebCore

2009-04-27 Dimitri Glazkov <Dimitri Glazkov>

Build fix.

​https://bugs.webkit.org/show_bug.cgi?id=25382
Remove direct reference to CallFrame.h include to pacify Win build.

  • bindings/js/JSDOMBinding.h: Removed ScriptState.h include.
  • bindings/js/ScriptState.h: Replaced CallFrame.h with JSDOMBinding.h
12:49 PM Changeset in webkit [42902] by Chris Fleizach
  • 2 edits in trunk/WebCore

Bug 25374: AX: No AXValue change sent when text is auto-inserted
​https://bugs.webkit.org/show_bug.cgi?id=25374

12:46 PM Changeset in webkit [42901] by Dimitri Glazkov
  • 10 edits in trunk/WebCore

2009-04-27 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Eric Seidel.

​https://bugs.webkit.org/show_bug.cgi?id=25382
Move scriptStateFromNode to ScriptState.

  • bindings/js/JSDOMBinding.cpp: Moved scriptStateFromNode to ScriptState.
  • bindings/js/JSDOMBinding.h: Ditto.
  • bindings/js/ScriptState.cpp: (WebCore::scriptStateFromNode): Added.
  • bindings/js/ScriptState.h: Ditto.
  • bindings/v8/ScriptState.cpp: (WebCore::scriptStateFromNode): Added.
  • bindings/v8/ScriptState.h: Ditto.
  • dom/NodeFilter.h: Removed JSDOMBinding header include.
  • dom/NodeIterator.h: Ditto.
  • dom/TreeWalker.h: Ditto.
12:44 PM Changeset in webkit [42900] by Chris Fleizach
  • 4 edits in trunk

Bug 25428: 5 crashes in DumpRenderTree at com.apple.WebCore • -[AccessibilityObjectWrapper accessibilityAttributeValue:] + 830
​https://bugs.webkit.org/show_bug.cgi?id=25428

12:33 PM Changeset in webkit [42899] by andersca@apple.com
  • 2 edits in trunk/WebKitExamplePlugins

2009-04-27 Anders Carlsson <​andersca@apple.com>

Reviewed by Adam Roben.

Update the example plug-in for new changes to the Cocoa event model.


  • NetscapeCocoaPlugin/main.m: (handleDraw): (handleMouseEvent): (NPP_HandleEvent):
11:54 AM Changeset in webkit [42898] by kdecker@apple.com
  • 2 edits in trunk/WebKit/mac

Fix the Tiger build.


  • Plugins/WebNetscapePluginView.mm: The base class of WebNetscapePluginView.mm, WebBaseNetscapePluginView, already implemented the two methods below. But the Tiger compiler didn't know that. (-[WebNetscapePluginView webView]): (-[WebNetscapePluginView webFrame]):
10:50 AM Changeset in webkit [42897] by kdecker@apple.com
  • 10 edits
    2 adds in trunk/WebKit

Reviewed by Anders Carlsson.

<rdar://problem/6352982>


  • Plugins/WebBaseNetscapePluginView.mm: Removed checkIfAllowedToLoadURL:frame:callbackFunc, cancelCheckIfAllowedToLoadURL, and _webPluginContainerCancelCheckIfAllowedToLoadRequest from the base class. These methods now exist in the subclass WebNetscapePluginView.


Added WebNetscapeContainerCheckContextInfo, which is used as a "contextInfo" object in
-[WebNetscapePluginView checkIfAllowedToLoadURL:frame:callbackFunc:]

  • Plugins/WebNetscapeContainerCheckContextInfo.h: Added.
  • Plugins/WebNetscapeContainerCheckContextInfo.mm: Added. (-[WebNetscapeContainerCheckContextInfo initWithCheckRequestID:callbackFunc:]): Added desiginated initializer. (-[WebNetscapeContainerCheckContextInfo checkRequestID]): Added. Returns the checkRequestID. (-[WebNetscapeContainerCheckContextInfo callback]): Added. Returns the callback.


  • Plugins/WebNetscapePluginView.h: Added two new ivars: _containerChecksInProgress and _currentContainerCheckRequestID.
  • Plugins/WebNetscapePluginView.mm: #import both WebPluginContainerCheck.h and WebNetscapeContainerCheckContextInfo.h


(-[WebNetscapePluginView checkIfAllowedToLoadURL:frame:callbackFunc:]): Added. This is the implementation of WKN_CheckIfAllowedToLoadURL.
Here, we increment the request ID and start the container check.


(-[WebNetscapePluginView _containerCheckResult:contextInfo:]): Added. This is a callback method for WebPluginContainerCheck.
It's where we actually call into the plug-in and provide the allow-or-deny result.


(-[WebNetscapePluginView cancelCheckIfAllowedToLoadURL:]): Added. This is the implementation of WKN_CancelCheckIfAllowedToLoadURL.
Here we lookup the check, cancel it, and remove it from _containerChecksInProgress.


(-[WebNetscapePluginView _webPluginContainerCancelCheckIfAllowedToLoadRequest:]): Added. WebPluginContainerCheck automatically calls
this method after invoking our _containerCheckResult: selector. It works this way because calling -[WebPluginContainerCheck cancel] allows
it to do it's teardown process.


(-[WebNetscapePluginView fini]): Release _containerChecksInProgress ivar.


  • Plugins/WebPluginContainerCheck.h: Removed initWithRequest: method from header; no client was using this method directly.
  • Plugins/WebPluginContainerCheck.mm: (+[WebPluginContainerCheck checkWithRequest:target:resultObject:selector:controller:contextInfo:]): Added optional contextInfo parameter. (-[WebPluginContainerCheck _continueWithPolicy:]): If there's a contextInfo object, pass it as a parameter to resultSelector. (-[WebPluginContainerCheck cancel]): Release _contextInfo ivar. (-[WebPluginContainerCheck contextInfo]): Added new method. Returns the contextInfo object, if one so exists.
  • Plugins/WebPluginController.mm: (-[WebPluginController _webPluginContainerCheckIfAllowedToLoadRequest:inFrame:resultObject:selector:]): Pass nil for contextInfo. WebKit-style plug-ins do not need additional context information.
9:57 AM Changeset in webkit [42896] by ap@webkit.org
  • 6 edits in trunk/WebCore

Reviewed by Darin Adler.

​https://bugs.webkit.org/show_bug.cgi?id=25399
<rdar://problem/6633943> REGRESSION: Many crashes reported accessing Lexis/Nexis database,
beneath WebCore::Cache::evict

The crash happened because a cached resource handle was removed from a document's cached
resources map twice recursively, so a destructor was called for a value in a deleted bucket.
The first call was from Cache::evict, and when destroying CachedResourceHandle destroyed
CachedResource, DocLoader::removeCachedResource() was called again, with HashMap being in
an inconsistent state.

I couldn't fully reconstruct the loading sequence to make a test.

  • loader/Cache.cpp: (WebCore::Cache::revalidateResource): Assert that the resource being revalidated is in cache (it makes no sense to revalidate one that isn't). (WebCore::Cache::evict): Don't remove the resource from document's map. Removing a resource from the cache in no way implies that documents no longer use the old version. This fixes the crash, and also fixes many cases of resource content being unavailable in Web Inspector.
  • loader/CachedResource.h: (WebCore::CachedResource::setInCache): When bringing a revalidated resource back to cache, reset m_isBeingRevalidated to maintain the invariant of resources being revalidated never being in cache. This fixes another assertion I saw on LexisNexis search: in rare cases, switchClientsToRevalidatedResource() results in the same resource being requested again, but we were only enforcing CachedResource invariants after calling this function. (WebCore::CachedResource::unregisterHandle): Assert that the counter doesn't underflow.
  • loader/DocLoader.cpp: (WebCore::DocLoader::removeCachedResource): Assert that the passed resource is removed, not some other resource that happens to have the same URL (this used to fail on LexisNexis search before this patch).
  • loader/ImageDocument.cpp: (WebCore::ImageTokenizer::write): Replaced ASSERT_NOT_REACHED with notImplemented(). This method can be legally called via document.write(), and should work. LexisNexis takes this code path, but apparently has a fallback for Safari, so it doesn't affect site functionality.
  • loader/CachedResource.cpp: (WebCore::CachedResource::clearResourceToRevalidate): Don't assert that m_resourceToRevalidate is being revalidated - this may no longer be true, because we now reset this member in CachedResource::setInCache().
9:24 AM Changeset in webkit [42895] by mitz@apple.com
  • 6 edits in trunk/WebCore

Reviewed by Sam Weinig.

  • make BidiContext a RefCounted class, starting with a refcount of 1, and share the root BidiContexts.
  • platform/graphics/GraphicsContext.cpp: (WebCore::GraphicsContext::drawBidiText): Use BidiContext::create().
  • platform/text/BidiContext.cpp: (WebCore::BidiContext::create): Added. For the four "root" contexts, returns a reference to a shared static BidiContext.
  • platform/text/BidiContext.h: (WebCore::BidiContext::BidiContext):
  • platform/text/BidiResolver.h: (WebCore::::commitExplicitEmbedding): Use BidiContext::create().
  • rendering/bidi.cpp: (WebCore::RenderBlock::determineStartPosition): Ditto.
8:23 AM Changeset in webkit [42894] by jmalonzo@webkit.org
  • 3 edits in trunk/WebCore

2009-04-27 Jan Michael Alonzo <​jmalonzo@webkit.org>

Rubber-stamped by Gustavo Noronha.

Build fix for r42893: change GDK_exclaim to GDK_exclam

  • platform/chromium/KeyCodeConversionGtk.cpp: (WebCore::windowsKeyCodeForKeyEvent):
  • platform/gtk/KeyEventGtk.cpp: (WebCore::windowsKeyCodeForKeyEvent):
8:08 AM Changeset in webkit [42893] by jmalonzo@webkit.org
  • 3 edits in trunk/WebCore

2009-04-27 Fumitoshi Ukai <​ukai@google.com>

Reviewed by Xan Lopez.

Fix wrong key code conversion.

Missing conversion for GDK_exlaim ('!').
Parens are opposite: GDK_parenleft is open paren '('
and GDK_parenright is close paren ')'.
​https://bugs.webkit.org/show_bug.cgi?id=25367

  • platform/chromium/KeyCodeConversionGtk.cpp: (WebCore::windowsKeyCodeForKeyEvent):
  • platform/gtk/KeyEventGtk.cpp: (WebCore::windowsKeyCodeForKeyEvent):
8:08 AM Changeset in webkit [42892] by jmalonzo@webkit.org
  • 3 edits in trunk/WebCore

2009-04-27 Jan Michael Alonzo <​jmalonzo@webkit.org>

Reviewed by Gustavo Noronha.

[Gtk] gtkdrawing update
​https://bugs.webkit.org/show_bug.cgi?id=25408

Update to the 25 Apr 2009 version of gtk2drawing.c and
gtkdrawing.h from Mozilla (​http://hg.mozilla.org/mozilla-central)

  • platform/gtk/gtk2drawing.c: (setup_widget_prototype): (moz_gtk_get_combo_box_entry_arrow): (ensure_toolbar_widget): (ensure_tree_header_cell_widget): (moz_gtk_init): (moz_gtk_button_get_inner_border): (moz_gtk_toggle_paint): (moz_gtk_caret_paint): (moz_gtk_entry_paint): (moz_gtk_tree_header_cell_paint): (moz_gtk_combo_box_paint): (moz_gtk_toolbar_separator_paint): (moz_gtk_menu_separator_paint): (moz_gtk_get_widget_border): (moz_gtk_get_tab_scroll_arrow_size): (moz_gtk_get_toolbar_separator_width): (moz_gtk_get_menu_separator_height): (moz_gtk_widget_paint): (moz_gtk_shutdown):
  • platform/gtk/gtkdrawing.h:
8:08 AM Changeset in webkit [42891] by jmalonzo@webkit.org
  • 2 edits in trunk/WebKitTools

2009-04-27 Jan Michael Alonzo <​jmalonzo@webkit.org>

Reviewed by Gustavo Noronha.

[GTK] Make layout tests / jsc tests usable without build-webkit
​https://bugs.webkit.org/show_bug.cgi?id=21784

This requires WEBKITOUTPUTDIR to be set to the build directory.
Set the location of the product dir to the location pointed to by
WEBKITOUTPUTDIR.

  • Scripts/webkitdirs.pm:
7:54 AM Changeset in webkit [42890] by kov@webkit.org
  • 2 edits in trunk/WebKitTools

2009-04-27 Gustavo Noronha Silva <Gustavo Noronha Silva>

Reviewed by Adam Roben.

Use libsoup's facilities to resolve relative URLs, instead, since
it is a more robust method.

  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (LayoutTestController::queueLoad):
7:53 AM Changeset in webkit [42889] by kov@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-04-27 Gustavo Noronha Silva <Gustavo Noronha Silva>

Reviewed by Xan Lopez.

​https://bugs.webkit.org/show_bug.cgi?id=25409
[GTK] incorrectly fallsback on cancellation

Check whether we should fallback also on didFailLoad, so that we
don't get error pages showing up in parts of the content of some
sites because of, for instance, location changes.

  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::dispatchDidFailLoad):
7:53 AM Changeset in webkit [42888] by kov@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-04-27 Gustavo Noronha Silva <Gustavo Noronha Silva>

Reviewed by Xan Lopez.

​https://bugs.webkit.org/show_bug.cgi?id=25409
[GTK] incorrectly fallsback on cancellation

Correctly handle cancellation and frame loading interruption
caused by policy change. This also fixes
fast/frames/location-change.html.

  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::shouldFallBack):
6:55 AM Changeset in webkit [42887] by ariya@webkit.org
  • 2 edits in trunk/WebCore

2009-04-27 Ariya Hidayat <​ariya.hidayat@nokia.com>

Reviewed by Tor Arne Vestbø.

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

[Qt] Widget painting should follow the layout direction (LTR, RTL)
of the element style, not the application layout direction.

  • platform/qt/RenderThemeQt.cpp: (WebCore::RenderThemeQt::applyTheme):
6:27 AM Changeset in webkit [42886] by Adam Roben
  • 3 edits in trunk/JavaScriptCore

Windows Debug build fix

Not sure why the buildbots weren't affected by this problem.

  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj: Let VS re-order the file list, and added JavaScriptCore[_debug].def to the project. This was not necessary for the fix, but made making the fix easier.
  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore_debug.def: Removed a function that no longer exists.

Apr 26, 2009:

10:58 PM Changeset in webkit [42885] by xan@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-04-26 Xan Lopez <​xlopez@igalia.com>

Reviewed by Mark Rowe.

Plug memory leak.

  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::dispatchDidFailLoad):
10:17 PM Changeset in webkit [42884] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

2009-04-26 Gavin Barraclough <​barraclough@apple.com>

Reviewed by Weinig Sam.

Fix for ​https://bugs.webkit.org/show_bug.cgi?id=25416
"Cached prototype accesses unsafely hoist property storage load above structure checks."


Do not hoist the load of the pointer to the property storage array.

No performance impact.

  • jit/JITPropertyAccess.cpp: (JSC::JIT::privateCompileGetByIdProto): (JSC::JIT::privateCompileGetByIdProtoList):
9:32 PM Changeset in webkit [42883] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

2009-04-26 Gavin Barraclough <​barraclough@apple.com>

Reviewed by Geoffrey "Gaffe or energy?" Garen.

Randomize address requested by ExecutableAllocatorFixedVMPool.

  • jit/ExecutableAllocatorFixedVMPool.cpp: (JSC::FixedVMPoolAllocator::FixedVMPoolAllocator):
5:03 PM Changeset in webkit [42882] by weinig@apple.com
  • 10 edits
    3 adds in trunk

WebCore:

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

Reviewed by Dan Bernstein.

Fix for ​https://bugs.webkit.org/show_bug.cgi?id=25412
The Window object should be an EventTarget

Test: fast/dom/Window/dispatchEvent.html

  • bindings/js/JSEventTarget.cpp: (WebCore::toJS): Add DOMWindow case. (WebCore::toEventTarget): Ditto.
  • dom/EventTarget.cpp: (WebCore::EventTarget::toDOMWindow): Added.
  • dom/EventTarget.h:
  • page/DOMWindow.cpp: (WebCore::DOMWindow::scriptExecutionContext): Added. Returns the document. (WebCore::DOMWindow::dispatchEvent): Added.
  • page/DOMWindow.h: (WebCore::DOMWindow::toDOMWindow): (WebCore::DOMWindow::refEventTarget): (WebCore::DOMWindow::derefEventTarget):
  • page/DOMWindow.idl:

LayoutTests:

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

Reviewed by Dan Bernstein.

Tests for ​https://bugs.webkit.org/show_bug.cgi?id=25412
The Window object should be an EventTarget

  • fast/dom/Window/dispatchEvent-expected.txt: Added.
  • fast/dom/Window/dispatchEvent.html: Added.
  • fast/dom/Window/resources/dispatchEvent.js: Added.
  • fast/dom/Window/window-properties-expected.txt
  • platform/mac-leopard/fast/dom/Window/window-properties-expected.txt:
2:09 PM Changeset in webkit [42881] by weinig@apple.com
  • 2 edits
    2 deletes in trunk

JavaScriptCore:

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

Reviewed by Eric Seidel.

Remove scons-based build system.

WebCore:

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

Reviewed by Eric Seidel.

Remove scons-based build system.

  • WebCore.scons: Removed.
11:11 AM Changeset in webkit [42880] by kov@webkit.org
  • 2 edits in trunk/WebKitTools

2009-04-26 Gustavo Noronha Silva <Gustavo Noronha Silva>

Reviewed by Xan Lopez.

Resolve relative URLs when queueing loads. This fixes
post-goback-same-url.html timing out.

  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (LayoutTestController::queueLoad):
10:23 AM Changeset in webkit [42879] by xan@webkit.org
  • 5 edits in trunk/WebKit/gtk

2009-04-26 Xan Lopez <​xlopez@igalia.com>

Reviewed by Gustavo Noronha.

​https://bugs.webkit.org/show_bug.cgi?id=25391
G_SIGNAL_ACTION flag in WebKitWebView signals

Most of the signals in WebKitGTK+ have the G_SIGNAL_ACTION flag,
but I don't think this makes sense. The flag is meant for those
signals that can be emitted by the user to trigger some action in
the view, as if it were a normal API, which is not the case for
most of them.

  • webkit/webkitdownload.cpp: (_WebKitDownloadPrivate::webkit_download_class_init):
  • webkit/webkitwebframe.cpp:
  • webkit/webkitwebinspector.cpp: (_WebKitWebInspectorPrivate::webkit_web_inspector_class_init):
  • webkit/webkitwebview.cpp:
10:11 AM Changeset in webkit [42878] by xan@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-04-26 Xan Lopez <​xlopez@igalia.com>

Unreviewed, actually apply the right patch.

  • webkit/webkitwebview.cpp: (webkit_web_view_move_cursor):
10:05 AM Changeset in webkit [42877] by xan@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-04-26 Xan Lopez <​xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Pass a variable to hold the return value of the 'move-cursor'
signal, otherwise a crash can happen.

  • webkit/webkitwebview.cpp: (webkit_web_view_move_cursor):
7:25 AM Changeset in webkit [42876] by Darin Adler
  • 2 edits in trunk/WebCore

2009-04-26 Darin Adler <Darin Adler>

Suggested by Darin Fisher.

Improve the fix for bug 25355: Crash when Creating New Tab or New Window when set to open Same Page
​https://bugs.webkit.org/show_bug.cgi?id=25355
rdar://problem/6823543

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::loadItem): Allow shouldScroll to be true even if m_currentHistoryItem is 0. Not sure if when this case can really arise in practice, but it's good to match the original logic more closely.
12:02 AM QtWebKitTodo edited by Simon Hausmann
(diff)
12:01 AM Changeset in webkit [42875] by krit@webkit.org
  • 2 edits in trunk/WebCore

2009-04-25 Dirk Schulze <​krit@webkit.org>

Reviewed by Oliver Hunt.

A width or height of zero for the destination or source rect of an
image causes a not invertible pattern matrix.
This crahes newer versions of Cairo and give some graphic gliches in
Canvas.
With this patch we check if there is something to draw and return if not.

  • platform/graphics/cairo/ImageCairo.cpp: (WebCore::BitmapImage::draw):
Note: See TracTimeline for information about the timeline view.