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

Timeline



Feb 25, 2008:

11:30 PM Google Summer of Code 2008 edited by eric@webkit.org
(diff)
11:05 PM Google Summer of Code 2008 created by eric@webkit.org
10:47 PM Changeset in webkit [30590] by mitz@apple.com
  • 3 edits
    4 adds in trunk

WebCore:

Reviewed by Sam Weinig.

Test: fast/text/apply-start-width-after-skipped-text.html

  • rendering/bidi.cpp: (WebCore::RenderBlock::findNextLineBreak): Improved the logic for when to figure in the padding, border and margin before a text node when the line begins inside a text node.

LayoutTests:

Reviewed by Sam Weinig.

  • fast/text/apply-start-width-after-skipped-text.html: Added.
  • platform/mac/fast/text/apply-start-width-after-skipped-text-expected.checksum: Added.
  • platform/mac/fast/text/apply-start-width-after-skipped-text-expected.png: Added.
  • platform/mac/fast/text/apply-start-width-after-skipped-text-expected.txt: Added.
10:47 PM WikiStart edited by eric@webkit.org
(diff)
10:20 PM Changeset in webkit [30589] by kevino@webkit.org
  • 2 edits in trunk/WebCore

Windows build fix for wx port.

9:00 PM Changeset in webkit [30588] by mrowe@apple.com
  • 2 edits in trunk/WebCore

2008-02-25 Mark Rowe <​mrowe@apple.com>

Windows build fix.

  • platform/network/cf/ResourceResponseCFNet.cpp: (WebCore::ResourceResponse::platformCompare): Implement platformCompare for ResourceResponseCFNet.cpp.
8:25 PM Changeset in webkit [30587] by Beth Dakin
  • 4 edits
    2 adds in trunk

WebCore:

Reviewed by Geoff.

Fixed for <rdar://problem/5760530> REGRESSION: Assertion failure in
Document::removePendingSheet() from r30438

r30438 added a call to CachedResource::error() from inside
Loader::didReceiveData() if a CSS file 4xxs. There was an
assumption in the loader, though, that either error() would be
called, or didFinishLoading() would be called, so some work is
duplicated in each. Now that we are calling an error() on files
that will also make it to didFinishLoading() (since they succeeded
in the network layer), we need to make sure we do not duplicate the
work. CachedCSSStyleSheet::error() calls checkNotify, which ends up
decrementing the document's pending style sheet counter.
checkNotify() was still getting called, though, through the normal
didFinishLoading code path, and the counter was being decremented
twice. Bad!

  • loader/loader.cpp: (WebCore::Loader::didFinishLoading): (WebCore::Loader::didReceiveData):

LayoutTests:

Reviewed by Geoff.

Test for <rdar://problem/5760530> REGRESSION: Assertion failure in
Document::removePendingSheet() from r30438

  • http/tests/misc/missing-style-sheet-expected.txt: Added.
  • http/tests/misc/missing-style-sheet.html: Added.
8:01 PM Changeset in webkit [30586] by mrowe@apple.com
  • 5 edits in trunk/WebCore

Fix the Gtk, wx and Qt builds.

6:22 PM Changeset in webkit [30585] by mrowe@apple.com
  • 1 edit in trunk/WebCore/ChangeLog

Fix bug number in ChangeLog.

6:21 PM Changeset in webkit [30584] by mrowe@apple.com
  • 20 edits
    4 moves
    8 adds in trunk/WebCore

2008-02-25 MorganL <​morganl.webkit@yahoo.com>

Reviewed by Darin Adler.

Eliminate some #ifdefs from headers in platform/network.
​http://bugs.webkit.org/show_bug.cgi?id=11786

  • WebCore.base.exp:
  • WebCore.order:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • dom/XMLTokenizer.cpp:
  • loader/ResourceLoader.h:
  • platform/network/AuthenticationChallenge.cpp: Removed.
  • platform/network/AuthenticationChallenge.h: Removed.
  • platform/network/AuthenticationChallengeBase.cpp: Copied from platform/network/AuthenticationChallenge.cpp. (WebCore::AuthenticationChallengeBase::AuthenticationChallengeBase): (WebCore::AuthenticationChallengeBase::previousFailureCount): (WebCore::AuthenticationChallengeBase::proposedCredential): (WebCore::AuthenticationChallengeBase::protectionSpace): (WebCore::AuthenticationChallengeBase::failureResponse): (WebCore::AuthenticationChallengeBase::error): (WebCore::AuthenticationChallengeBase::isNull): (WebCore::AuthenticationChallengeBase::nullify): (WebCore::AuthenticationChallengeBase::compare):
  • platform/network/AuthenticationChallengeBase.h: Copied from platform/network/AuthenticationChallenge.h. (WebCore::AuthenticationChallengeBase::platformCompare): (WebCore::operator==): (WebCore::operator!=):
  • platform/network/ResourceError.cpp: Removed.
  • platform/network/ResourceError.h: Removed.
  • platform/network/ResourceErrorBase.cpp: Copied from platform/network/ResourceError.cpp. (WebCore::ResourceErrorBase::lazyInit): (WebCore::ResourceErrorBase::compare):
  • platform/network/ResourceErrorBase.h: Copied from platform/network/ResourceError.h. (WebCore::ResourceErrorBase::isNull): (WebCore::ResourceErrorBase::domain): (WebCore::ResourceErrorBase::errorCode): (WebCore::ResourceErrorBase::failingURL): (WebCore::ResourceErrorBase::localizedDescription): (WebCore::ResourceErrorBase::ResourceErrorBase): (WebCore::ResourceErrorBase::platformLazyInit): (WebCore::ResourceErrorBase::platformCompare): (WebCore::operator==):
  • platform/network/ResourceResponseBase.cpp: (WebCore::ResourceResponseBase::isHTTP): (WebCore::ResourceResponseBase::url): (WebCore::ResourceResponseBase::setUrl): (WebCore::ResourceResponseBase::mimeType): (WebCore::ResourceResponseBase::setMimeType): (WebCore::ResourceResponseBase::expectedContentLength): (WebCore::ResourceResponseBase::setExpectedContentLength): (WebCore::ResourceResponseBase::textEncodingName): (WebCore::ResourceResponseBase::setTextEncodingName): (WebCore::ResourceResponseBase::suggestedFilename): (WebCore::ResourceResponseBase::setSuggestedFilename): (WebCore::ResourceResponseBase::httpStatusCode): (WebCore::ResourceResponseBase::setHTTPStatusCode): (WebCore::ResourceResponseBase::httpStatusText): (WebCore::ResourceResponseBase::setHTTPStatusText): (WebCore::ResourceResponseBase::httpHeaderField): (WebCore::ResourceResponseBase::setHTTPHeaderField): (WebCore::ResourceResponseBase::httpHeaderFields): (WebCore::ResourceResponseBase::isAttachment): (WebCore::ResourceResponseBase::setExpirationDate): (WebCore::ResourceResponseBase::expirationDate): (WebCore::ResourceResponseBase::setLastModifiedDate): (WebCore::ResourceResponseBase::lastModifiedDate): (WebCore::ResourceResponseBase::lazyInit): (WebCore::ResourceResponseBase::compare):
  • platform/network/ResourceResponseBase.h: (WebCore::ResourceResponseBase::ResourceResponseBase): (WebCore::ResourceResponseBase::platformLazyInit): (WebCore::ResourceResponseBase::platformCompare): (WebCore::operator==):
  • platform/network/cf/AuthenticationCF.cpp: (WebCore::AuthenticationChallenge::AuthenticationChallenge): (WebCore::AuthenticationChallenge::platformCompare):
  • platform/network/cf/AuthenticationChallenge.h: Added. (WebCore::AuthenticationChallenge::AuthenticationChallenge): (WebCore::AuthenticationChallenge::sourceHandle): (WebCore::AuthenticationChallenge::cfURLAuthChallengeRef):
  • platform/network/cf/ResourceError.h: Added. (WebCore::ResourceError::ResourceError):
  • platform/network/cf/ResourceErrorCF.cpp: (WebCore::ResourceError::ResourceError): (WebCore::ResourceError::platformLazyInit): (WebCore::ResourceError::platformCompare): (WebCore::ResourceError::operator CFStreamError):
  • platform/network/cf/ResourceResponse.h: (WebCore::ResourceResponse::ResourceResponse):
  • platform/network/cf/ResourceResponseCFNet.cpp: (WebCore::ResourceResponse::platformLazyInit):
  • platform/network/curl/AuthenticationChallenge.h: Added. (WebCore::AuthenticationChallenge::AuthenticationChallenge):
  • platform/network/curl/ResourceError.h: Added. (WebCore::ResourceError::ResourceError):
  • platform/network/curl/ResourceResponse.h: (WebCore::ResourceResponse::ResourceResponse):
  • platform/network/mac/AuthenticationChallenge.h: Added. (WebCore::AuthenticationChallenge::AuthenticationChallenge): (WebCore::AuthenticationChallenge::sender): (WebCore::AuthenticationChallenge::nsURLAuthenticationChallenge):
  • platform/network/mac/AuthenticationMac.mm: (WebCore::AuthenticationChallenge::AuthenticationChallenge): (WebCore::AuthenticationChallenge::platformCompare):
  • platform/network/mac/ResourceError.h: Added. (WebCore::ResourceError::ResourceError):
  • platform/network/mac/ResourceErrorMac.mm: (WebCore::ResourceError::platformLazyInit): (WebCore::ResourceError::platformCompare):
  • platform/network/mac/ResourceResponse.h: (WebCore::ResourceResponse::ResourceResponse):
  • platform/network/mac/ResourceResponseMac.mm: (WebCore::ResourceResponse::platformLazyInit): (WebCore::ResourceResponse::platformCompare):
  • platform/network/qt/AuthenticationChallenge.h: Added. (WebCore::AuthenticationChallenge::AuthenticationChallenge):
  • platform/network/qt/ResourceError.h: Added. (WebCore::ResourceError::ResourceError):
  • platform/network/qt/ResourceResponse.h: (WebCore::ResourceResponse::ResourceResponse):
  • xml/XSLTProcessor.cpp:
5:06 PM Changeset in webkit [30583] by Adam Roben
  • 2 edits in trunk/WebKitTools

Fix run-webkit-tests after r30394

  • Scripts/webkitdirs.pm: Never append _debug to FindSafari's executable name.
5:05 PM Changeset in webkit [30582] by Adam Roben
  • 5 edits in trunk/WebKit/win

Move plugin-related methods to WebFrameLoaderClient

Reviewed by Anders.

  • WebCoreSupport/WebFrameLoaderClient.cpp: (WebFrameLoaderClient::WebFrameLoaderClient): (WebFrameLoaderClient::setMainDocumentError): (WebFrameLoaderClient::committedLoad): Added some early returns. (WebFrameLoaderClient::receivedData): (WebFrameLoaderClient::finishedLoading): Ditto. (getWebDataSource): (WebFrameLoaderClient::createPlugin):
  • WebCoreSupport/WebFrameLoaderClient.h:
  • WebFrame.cpp: (WebFrame::WebFramePrivate::WebFramePrivate):
  • WebFrame.h:
5:05 PM Changeset in webkit [30581] by Adam Roben
  • 5 edits in trunk/WebKit/win

Move createFrame to WebFrameLoaderClient

Reviewed by Anders.

  • WebCoreSupport/WebFrameLoaderClient.cpp: (WebFrameLoaderClient::createFrame): (WebFrameLoaderClient::loadURLIntoChild):
  • WebCoreSupport/WebFrameLoaderClient.h:
  • WebFrame.cpp:
  • WebFrame.h:
5:04 PM Changeset in webkit [30580] by Adam Roben
  • 5 edits in trunk/WebKit/win

Move progress-related methods to WebFrameLoaderClient

Reviewed by Anders.

  • WebCoreSupport/WebFrameLoaderClient.cpp: (WebFrameLoaderClient::postProgressStartedNotification): (WebFrameLoaderClient::postProgressEstimateChangedNotification): (WebFrameLoaderClient::postProgressFinishedNotification):
  • WebCoreSupport/WebFrameLoaderClient.h:
  • WebFrame.cpp:
  • WebFrame.h:
5:01 PM Changeset in webkit [30579] by Adam Roben
  • 3 edits in trunk/WebKit/win

Remove WebFrame::detachedFromParent1

This method was removed from FrameLoaderClient in r19042, but was
never removed from WebFrame.

Reviewed by Anders.

  • WebFrame.cpp:
  • WebFrame.h:
4:52 PM BuildingCairoOnWindows edited by bfulgham@macports.org
(diff)
4:28 PM Changeset in webkit [30578] by beidson@apple.com
  • 1 edit
    2 adds in trunk/LayoutTests

Reviewed by T3h Mitz Pettel (Dan Bernstein)

Layout test for already fixed bug <rdar://problem/5729445>

  • storage/multiple-transactions-expected.txt: Added.
  • storage/multiple-transactions.html: Added.
2:57 PM Changeset in webkit [30577] by andersca@apple.com
  • 7 edits in trunk/WebCore

Reviewed by Darin.

Move remaining render style objects over to start with a refcount of 1 and get
rid of DeprecatedDataRef.


  • rendering/DataRef.h: (WebCore::DataRef::operator*): (WebCore::DataRef::operator->):
  • rendering/RenderStyle.h:
  • rendering/SVGRenderStyle.cpp: (WebCore::SVGRenderStyle::SVGRenderStyle):
  • rendering/SVGRenderStyle.h: (WebCore::SVGRenderStyle::create): (WebCore::SVGRenderStyle::copy):
  • rendering/SVGRenderStyleDefs.cpp: (StyleFillData::StyleFillData): (StyleStrokeData::StyleStrokeData): (StyleStopData::StyleStopData): (StyleTextData::StyleTextData): (StyleClipData::StyleClipData): (StyleMaskData::StyleMaskData): (StyleMarkerData::StyleMarkerData): (StyleMiscData::StyleMiscData):
  • rendering/SVGRenderStyleDefs.h: (WebCore::StyleFillData::create): (WebCore::StyleFillData::copy): (WebCore::StyleStrokeData::create): (WebCore::StyleStrokeData::copy): (WebCore::StyleStopData::create): (WebCore::StyleStopData::copy): (WebCore::StyleTextData::create): (WebCore::StyleTextData::copy): (WebCore::StyleClipData::create): (WebCore::StyleClipData::copy): (WebCore::StyleMaskData::create): (WebCore::StyleMaskData::copy): (WebCore::StyleMarkerData::create): (WebCore::StyleMarkerData::copy): (WebCore::StyleMiscData::create): (WebCore::StyleMiscData::copy):
2:44 PM Changeset in webkit [30576] by mrowe@apple.com
  • 4 edits in trunk/JavaScriptCore

2008-02-25 Mark Mentovai <​mark@moxienet.com>

Reviewed by Mark Rowe.

Enable CollectorHeapIntrospector to build by itself, as well as in an AllInOneFile build.
​http://bugs.webkit.org/show_bug.cgi?id=17538

  • kjs/CollectorHeapIntrospector.cpp: Provide "using" declaration for WTF::RemoteMemoryReader.
  • kjs/collector.h: Move CollectorHeap declaration here...
  • kjs/collector.cpp: ... from here.
2:03 PM Changeset in webkit [30575] by mitz@apple.com
  • 4 edits in trunk/WebCore

Reviewed by Dave Hyatt.

  • remove file-static BidiRun variables from bidi.cpp
  • platform/text/BidiResolver.h: (WebCore::BidiResolver::): Initialize m_logicallyLastRun. (WebCore::BidiResolver::logicallyLastRun): Added. (WebCore::BidiResolver::runCount): Made unsigned. (WebCore::::reverseRuns): Changed ints to unsigned. (WebCore::::createBidiRunsForLine): Made this function set m_logicallyLastRun.
  • rendering/RenderBlock.h:
  • rendering/bidi.cpp: (WebCore::BidiState::addRun): Removed setting of sLogicallyLastBidiRun. (WebCore::RenderBlock::constructLine): Removed unused start parameter and added run count and first and last run parameters. Replaced end parameter with lastLine boolean and endObject pointer. (WebCore::RenderBlock::computeHorizontalPositionsForLine): Added first and logically last run parameters. (WebCore::RenderBlock::computeVerticalPositionsForLine): Added firstRun parameter. (WebCore::RenderBlock::bidiReorderLine): Removed setting of static variables. (WebCore::RenderBlock::layoutInlineChildren): Changed to use BidiState accessors instead of file statics.
1:58 PM Changeset in webkit [30574] by mrowe@apple.com
  • 16 edits
    1 copy
    1 delete in trunk/WebCore

2008-02-25 Rodney Dawes <​dobey@wayofthemonkey.com>

Reviewed by Jon Honeycutt.

​http://bugs.webkit.org/show_bug.cgi?id=16924
Bug 16924: Shared PluginDatabase, PluginPackage, and PlugInInfoStore implementations

Add PluginInfoStore.cpp and new PluginDatabase.cpp to GTK+ and Qt ports.
Remove old PlugInInfoStoreQt.cpp as it is obsoleted by shared code.
Add PluginInfoStore, PluginDatabase, and PluginStream files to Wx build.
Add new PluginDatabase.cpp to Windows build.
Add temporary stubs for new PluginDatabase and PluginPackage.
shared classes to GTK+, Qt, and Wx ports.
Copy PluginDatabaseWin.cpp to PluginDatabase.cpp to preserve history.
Remove shared code from PluginDatabaseWin.cpp.
Remove Windows-specific code from PluginDatabase.cpp.
Use PlatformModule and PlatformFileTime instead of HMODULE and FILETIME.

Remove extraneous PluginPackage
from hash() class method prototype. Subsume storeFileVersion into PluginPackage::fetchInfo. Add cross-platform PlatformModuleVersion type definition. Use PlatformModuleVersion to store the module version. Rename m_fileVersion[ML]S to m_moduleVersion. Change compareFileVersion to use PlatformModuleVersion as the argument. Move PluginView::determineQuirks and m_quirks to PluginPackage. Updated determineQuirks for the PlatformModuleVersion.
  • GNUmakefile.am:
  • WebCore.pro:
  • WebCoreSources.bkl:
  • webcore-base.bkl:
  • WebCore.vcproj/WebCore.vcproj:
  • Platform/FileSystem.h:
  • platform/qt/PlugInInfoStoreQt.cpp:
  • platform/gtk/TemporaryLinkStubs.cpp:
  • platform/qt/TemporaryLinkStubs.cpp:
  • platform/wx/TemporaryLinkStubs.cpp:
  • plugins/PluginDatabase.cpp:
  • plugins/PluginDatabase.h:
  • plugins/win/PluginDatabaseWin.cpp:
  • plugins/PluginPackage.h:
  • plugins/win/PluginPackageWin.cpp:
  • plugins/PluginView.h:
  • plugins/win/PluginViewWin.cpp:
1:51 PM Changeset in webkit [30573] by andersca@apple.com
  • 3 edits in trunk/WebCore

Reviewed by Darin.

Make more style objects start out with a refcount of 1.


  • rendering/RenderStyle.cpp: (WebCore::StyleSurroundData::StyleSurroundData): (WebCore::StyleBoxData::StyleBoxData): (WebCore::StyleVisualData::StyleVisualData): (WebCore::StyleBackgroundData::StyleBackgroundData): (WebCore::StyleRareNonInheritedData::StyleRareNonInheritedData): (WebCore::StyleRareInheritedData::StyleRareInheritedData): (WebCore::StyleInheritedData::StyleInheritedData):
  • rendering/RenderStyle.h: (WebCore::StyleSurroundData::create): (WebCore::StyleSurroundData::copy): (WebCore::StyleBoxData::create): (WebCore::StyleBoxData::copy): (WebCore::StyleVisualData::create): (WebCore::StyleVisualData::copy): (WebCore::StyleBackgroundData::create): (WebCore::StyleBackgroundData::copy): (WebCore::StyleBackgroundData::~StyleBackgroundData): (WebCore::StyleRareNonInheritedData::create): (WebCore::StyleRareNonInheritedData::copy): (WebCore::StyleRareInheritedData::create): (WebCore::StyleRareInheritedData::copy): (WebCore::StyleInheritedData::create): (WebCore::StyleInheritedData::copy):
12:00 PM Changeset in webkit [30572] by andersca@apple.com
  • 5 edits in trunk/WebCore

Reviewed by Darin.

Make some of the refcounted style objects start out with a refcount of 1.

  • rendering/DataRef.h: Make a DeprecatedDataRef class which is just a copy of the old DataRef class. Change DataRef to use ::create() and ::copy() instead of the constructors. Change DataRef's pointer to be a RefPtr instead.
  • rendering/RenderStyle.cpp: (WebCore::StyleMarqueeData::StyleMarqueeData): (WebCore::StyleFlexibleBoxData::StyleFlexibleBoxData): (WebCore::StyleMultiColData::StyleMultiColData): (WebCore::StyleTransformData::StyleTransformData): Start with a RefCount of 1.


  • rendering/RenderStyle.h:
  • rendering/SVGRenderStyle.h: Add ::create() and ::copy() methods. Make not yet converted classes use DeprecatedDataRef.
9:33 AM Changeset in webkit [30571] by Darin Adler
  • 3 edits
    3 adds in trunk

WebCore:

Reviewed by Anders.
Based on a patch by Tim Steele <​timsteele41@gmail.com>.

Test: fast/loader/meta-refresh-anchor-click.html

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::load): When load type is FrameLoadTypeRedirectWithLockedHistory, always do a real load. This makes sure that meta refresh loads are treated as real loads rather than anchor scrolls. Also tweaked formatting a bit. (WebCore::FrameLoader::continueFragmentScrollAfterNavigationPolicy): This function is an alternate way to complete a load, so it needs to start the redirection timer if redirection has already been scheduled.

LayoutTests:

Reviewed by Anders.

  • fast/loader/meta-refresh-anchor-click-expected.txt: Added.
  • fast/loader/meta-refresh-anchor-click.html: Added.
  • fast/loader/resources/meta-refresh-subframe.html: Added.
9:11 AM Changeset in webkit [30570] by mitz@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Darin Adler.

  • avoid counting spaces in non-justified text
  • rendering/bidi.cpp: Removed file-static numSpaces. (WebCore::BidiState::addRun): Removed space counting. (WebCore::RenderBlock::computeHorizontalPositionsForLine): Added space counting, conditional on the block having the text-align: justify property value. (WebCore::RenderBlock::bidiReorderLine): Removed initialization of numSpaces.
9:10 AM Changeset in webkit [30569] by Darin Adler
  • 2 edits in trunk/WebCore

Requested by Steve.

  • platform/graphics/cg/ImageSourceCG.cpp: (WebCore::ImageSource::createFrameAtIndex): Put the string into a global variable (because CFSTR is inefficient on Windows).
9:06 AM Changeset in webkit [30568] by Darin Adler
  • 6 edits in trunk

JavaScriptCore:

Reviewed by Adam.

WebCore:

Reviewed by Adam.

  • WebCore.base.exp: Export blankURL, and sort the file.

WebKit/mac:

Reviewed by Adam.

  • WebView/WebArchiver.mm: (+[WebArchiver archiveSelectionInFrame:]): Use blankURL.
  • WebView/WebFrame.mm: (-[WebFrame _loadData:MIMEType:textEncodingName:baseURL:unreachableURL:]): Avoid the variable name URL to avoid clashing with the renamed KURL in the future. Also use blankURL. (-[WebFrame loadData:MIMEType:textEncodingName:baseURL:]): Ditto. (-[WebFrame _loadHTMLString:baseURL:unreachableURL:]): Ditto. (-[WebFrame loadHTMLString:baseURL:]): Ditto. (-[WebFrame loadAlternateHTMLString:baseURL:forUnreachableURL:]): Ditto.
8:53 AM Changeset in webkit [30567] by Adam Roben
  • 2 edits in trunk/JavaScriptCore

MSVC build fix

  • kjs/testkjs.cpp: (functionQuit): Don't add a return statement after exit(0) for MSVC.
8:52 AM Changeset in webkit [30566] by Adam Roben
  • 5 edits in trunk/WebKit/win

Move many dispatch methods to WebFrameLoaderClient

Reviewed by Sam.

  • WebCoreSupport/WebFrameLoaderClient.cpp: (WebFrameLoaderClient::dispatchDidHandleOnloadEvents): (WebFrameLoaderClient::dispatchDidReceiveServerRedirectForProvisionalLoad): (WebFrameLoaderClient::dispatchDidCancelClientRedirect): (WebFrameLoaderClient::dispatchWillPerformClientRedirect): (WebFrameLoaderClient::dispatchDidChangeLocationWithinPage): (WebFrameLoaderClient::dispatchWillClose): (WebFrameLoaderClient::dispatchDidReceiveIcon): (WebFrameLoaderClient::dispatchDidStartProvisionalLoad): (WebFrameLoaderClient::dispatchDidReceiveTitle): (WebFrameLoaderClient::dispatchDidCommitLoad): (WebFrameLoaderClient::dispatchDidFinishDocumentLoad): (WebFrameLoaderClient::dispatchDidFinishLoad): (WebFrameLoaderClient::dispatchDidFirstLayout): (WebFrameLoaderClient::dispatchShow):
  • WebCoreSupport/WebFrameLoaderClient.h:
  • WebFrame.cpp:
  • WebFrame.h:
8:52 AM Changeset in webkit [30565] by Adam Roben
  • 5 edits in trunk/WebKit/win

Move two more methods to WebFrameLoaderClient

Reviewed by Sam.

  • WebCoreSupport/WebFrameLoaderClient.cpp: (WebFrameLoaderClient::hasFrameView): (WebFrame::forceLayout):
  • WebCoreSupport/WebFrameLoaderClient.h:
  • WebFrame.cpp:
  • WebFrame.h:
5:16 AM Changeset in webkit [30564] by Simon Hausmann
  • 16 edits in trunk/WebCore

Fix compilation with gcc 4.3, added stdio.h and unistd.h inclusions.

5:14 AM Changeset in webkit [30563] by mrowe@apple.com
  • 3 edits
    4 adds in trunk

2008-02-25 Johnny Ding <​johnnyding.webkit@gmail.com>

Reviewed by Darin Adler.

In HTMLTokenizer::write, the code checks 'pendingScripts.isEmpty()' to decide
whether to save prependingSrc or not. However, in HTMLTokenizer::scriptHandler
and HTMLTokenizer::scriptExecution, the code checks testBit:LoadingExtScript
to decide whether to save prependingSrc or not. The later behavior is not right
because, in scriptHandler and scriptExecution, even the pendingScripts queue is
empty, the testBit:LoadingExtScript might be TRUE.

Test: fast/tokenizer/nested-multiple-scripts.html

  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::scriptHandler): check pendingScripts.isEmpty() instead of checking state.loadingExtScript(). (WebCore::HTMLTokenizer::scriptExecution): check pendingScripts.isEmpty() instead of checking state.loadingExtScript().

2008-02-25 Johnny Ding <​johnnyding.webkit@gmail.com>

Reviewed by Darin Adler.

  • fast/tokenizer/nested-multiple-scripts-expected.txt: Added.
  • fast/tokenizer/nested-multiple-scripts.html: Added.
  • fast/tokenizer/resources/external-script-1.js: Added.
  • fast/tokenizer/resources/external-script-2.js: Added.

Feb 24, 2008:

11:44 PM Changeset in webkit [30562] by Darin Adler
  • 2 edits in trunk/WebKitTools
  • Scripts/do-webcore-rename: Make some updates based on a trial run of the renaming script.
11:17 PM Changeset in webkit [30561] by Darin Adler
  • 2 edits in trunk/WebKitTools
  • Scripts/do-webcore-rename: More renaming plans.
11:11 PM Changeset in webkit [30560] by weinig@apple.com
  • 2 edits in trunk/JavaScriptCore

Reviewed by Mark Rowe.

​http://bugs.webkit.org/show_bug.cgi?id=17529
Add support for reading from stdin from testkjs

  • kjs/testkjs.cpp: (GlobalObject::GlobalObject): Add readline function to global object. (functionReadline): Added. Reads characters from stdin until a '\n' or EOF is encountered. The input is returned as a String to the caller.
11:09 PM Changeset in webkit [30559] by Darin Adler
  • 2 edits in trunk/WebCore
  • dom/Document.h: Removed stray duplicate declaration of RegisteredEventListenerList.
10:52 PM Changeset in webkit [30558] by Darin Adler
  • 2 edits in trunk/WebCore
  • dom/Range.cpp: (WebCore::Range::surroundContents): Removed incorrect comment. I added it a while ago, and I was just mixed up.
10:48 PM Changeset in webkit [30557] by Darin Adler
  • 2 edits in trunk/WebCore
  • quick follow-up to that last check-in
  • platform/graphics/cg/ImageSourceCG.cpp: (WebCore::ImageSource::createFrameAtIndex): Use CFEqual instead of WebCore::String's == for speed.
10:36 PM Changeset in webkit [30556] by Darin Adler
  • 3 edits
    6 adds in trunk

WebCore:

Reviewed by Darin.

Bug 8749: XBM rendered incorrectly as black on white

xbm images are now created and rendered using black on a
transparent background (same behavior as in Firefox)

Test: platform/mac/fast/canvas/canvas-draw-xbm-image.html

  • platform/graphics/cg/ImageSourceCG.cpp: (WebCore::ImageSource::createFrameAtIndex):

LayoutTests:

Reviewed by Darin.


  • platform/mac/fast/canvas/canvas-draw-xbm-image-expected.checksum: Added.
  • platform/mac/fast/canvas/canvas-draw-xbm-image-expected.png: Added.
  • platform/mac/fast/canvas/canvas-draw-xbm-image-expected.txt: Added.
  • platform/mac/fast/canvas/canvas-draw-xbm-image.html: Added.
  • platform/mac/fast/canvas/resources: Added.
  • platform/mac/fast/canvas/resources/smile.xbm: Added.
9:29 PM Changeset in webkit [30555] by weinig@apple.com
  • 4 edits in trunk/JavaScriptCore

Reviewed by Mark Rowe.

​http://bugs.webkit.org/show_bug.cgi?id=17528
Give testkjs a bath

  • JavaScriptCore.exp:
  • JavaScriptCore.xcodeproj/project.pbxproj: Make the testkjs.cpp use 4 space indentation.
  • kjs/testkjs.cpp: (StopWatch::getElapsedMS): (GlobalObject::className): (GlobalObject::GlobalObject): Rename GlobalImp to GlobalObject and setup the global functions in the GlobalObject's constructor. Also, use static functions for the implementation so we can use the standard PrototypeFunction class and remove TestFunctionImp. (functionPrint): Move print() functionality here. (functionDebug): Move debug() functionality here. (functionGC): Move gc() functionality here. (functionVersion): Move version() functionality here. (functionRun): Move run() functionality here. (functionLoad): Move load() functionality here. (functionQuit): Move quit() functionality here. (prettyPrintScript): Fix indentation. (runWithScripts): Since all the functionality of createGlobalObject is now in the GlobalObject constructor, just call new here. (parseArguments): Fix indentation. (kjsmain): Ditto (fillBufferWithContentsOfFile): Ditto.
7:08 PM Changeset in webkit [30554] by mitz@apple.com
  • 3 edits
    3 adds in trunk

WebCore:

Reviewed by Darin Adler.

Test: http/tests/misc/object-image-error.html

  • html/HTMLObjectElement.cpp: (WebCore::HTMLObjectElement::attach): Added an early return in case updateFromElement() has changed the object to use fallback content.

LayoutTests:

Reviewed by Darin Adler.

  • http/tests/misc/object-image-error-expected.txt: Added.
  • http/tests/misc/object-image-error.html: Added.
  • http/tests/misc/resources/404image.php: Added.
5:34 PM Changeset in webkit [30553] by weinig@apple.com
  • 2 edits in trunk/JavaScriptCore

Reviewed by Oliver Hunt and Mark Rowe.

​http://bugs.webkit.org/show_bug.cgi?id=17505
Add support for getting command line arguments in testkjs

  • This slightly changes the behavior of parsing arguments by requiring a '-f' before all files.
  • kjs/testkjs.cpp: (createGlobalObject): Add a global property called 'arguments' which contains an array with the parsed arguments as strings. (runWithScripts): Pass in the arguments vector so that it can be passed to the global object. (parseArguments): Change parsing rules to require a '-f' before any script file. After all '-f' and '-p' arguments have been parsed, the remaining are added to the arguments vector and exposed to the script. If there is a chance of ambiguity (the user wants to pass the string '-f' to the script), the string '--' can be used separate the options from the pass through arguments. (kjsmain):
2:50 PM Changeset in webkit [30552] by mitz@apple.com
  • 1 edit
    651 moves
    651 deletes in trunk/LayoutTests

Rubber-stamped by Oliver Hunt.

  • replace expected results in platform/mac-tiger and platform/mac-leopard which were identical with a single copy in platform/mac
10:48 AM Changeset in webkit [30551] by mitz@apple.com
  • 3 edits
    4 adds in trunk

WebCore:

Reviewed by Darin Adler.

  • fix float positioning when a float that does not fit on the line is followed by a float that does

Test: fast/block/float/narrow-after-wide.html

  • rendering/bidi.cpp: (WebCore::RenderBlock::findNextLineBreak): Changed to not position any more floats on the line once a float that does not fit is encountered. That float should be pushed to the next line, and so should all floats that follow, regardless of whether they can fit on the current line.

LayoutTests:

Reviewed by Darin Adler.

  • test float positioning when a float that does not fit on the line is followed by a float that does
  • fast/block/float/narrow-after-wide.html: Added.
  • platform/mac/fast/block/float/narrow-after-wide-expected.checksum: Added.
  • platform/mac/fast/block/float/narrow-after-wide-expected.png: Added.
  • platform/mac/fast/block/float/narrow-after-wide-expected.txt: Added.
10:44 AM Changeset in webkit [30550] by mitz@apple.com
  • 2 edits in trunk/JavaScriptCore

Reviewed by Darin Adler.

  • wtf/Deque.h: (WTF::::expandCapacityIfNeeded): Fixed the case where m_start and m_end are both zero but the buffer capacity is non-zero. (WTF::::prepend): Added validity checks.
12:30 AM Changeset in webkit [30549] by Darin Adler
  • 21 edits
    3 adds in trunk

WebCore:

Reviewed by Sam.

Disentangle global history updating from the back/forward history.
There are many cases where we don't want to create a new back/forward
item, but we do still want to add to the global history (used for visited
link coloring) in those cases.

Test: fast/history/subframe-is-visited.html

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::updateGlobalHistory): Renamed from addHistoryForCurrentLocation and removed the back/forward handling. (WebCore::FrameLoader::updateHistoryForStandardLoad): Streamlined logic a bit. Replaced call to addHistoryForCurrentLocation with a call to addBackForwardItemClippedAtTarget. Added an unconditional call to updateGlobalHistory. (WebCore::FrameLoader::updateHistoryForClientRedirect): Added a FIXME; why doesn't this function update global history? (WebCore::FrameLoader::updateHistoryForBackForwardNavigation): Ditto. (WebCore::FrameLoader::updateHistoryForReload): Replaced the direct call the client with a call to the new updateGlobalHistory function. (WebCore::FrameLoader::updateHistoryForRedirectWithLockedHistory): Did the same changes as for updateHistoryForStandardLoad.
  • loader/FrameLoader.h: More of the same.
  • loader/FrameLoaderClient.h: Removed updateGlobalHistoryForReload and renamed updateGlobalHistoryForStandardLoad to updateGlobalHistory.
  • svg/graphics/SVGImageEmptyClients.h: (WebCore::SVGEmptyFrameLoaderClient::updateGlobalHistory): Updated to match the above.

WebKit/gtk:

Reviewed by Sam.

  • remove separate client calls for "standard" and "reload' history
  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::updateGlobalHistory):
  • WebCoreSupport/FrameLoaderClientGtk.h:

WebKit/mac:

Reviewed by Sam.

  • remove separate client calls for "standard" and "reload' history
  • WebCoreSupport/WebFrameLoaderClient.h:
  • WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::updateGlobalHistory):

WebKit/qt:

Reviewed by Sam.

  • remove separate client calls for "standard" and "reload' history
  • WebCoreSupport/FrameLoaderClientQt.cpp: (WebCore::FrameLoaderClientQt::updateGlobalHistory):
  • WebCoreSupport/FrameLoaderClientQt.h:

WebKit/win:

Reviewed by Sam.

  • remove separate client calls for "standard" and "reload' history
  • WebFrame.cpp: (WebFrame::updateGlobalHistory):
  • WebFrame.h:

WebKit/wx:

Reviewed by Sam.

  • remove separate client calls for "standard" and "reload' history
  • WebKitSupport/FrameLoaderClientWx.cpp: (WebCore::FrameLoaderClientWx::updateGlobalHistory):
  • WebKitSupport/FrameLoaderClientWx.h:

LayoutTests:

Reviewed by Sam.

  • fast/history/resources/subframe.html: Added.
  • fast/history/subframe-is-visited-expected.txt: Added.
  • fast/history/subframe-is-visited.html: Added.
12:09 AM Changeset in webkit [30548] by Darin Adler
  • 2 edits in trunk/WebKit/qt
  • another try at fixing the build
  • Api/qwebframe.cpp: (QWebFrame::load): Replace new FormData with FormData::create.
Note: See TracTimeline for information about the timeline view.