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

Timeline



Oct 2, 2006:

9:38 PM Changeset in webkit [16732] by aroben
  • 3 edits in trunk/WebKit

Reviewed by Maciej.

(Forgot this in previous checkin)

Add message paramter to WebView::mouse* methods to pass down to
PlatformMouseEvent.

  • COM/WebView.cpp: (WebView::mouseMoved): (WebView::mouseDown): (WebView::mouseUp): (WebView::mouseDoubleClick): (WebViewWndProc):
  • COM/WebView.h:
9:35 PM Changeset in webkit [22286] by aroben
  • 2 edits in branches/WindowsMerge/WebKitWin
  • WebKit.vcproj/WebKit.vcproj:
9:29 PM Changeset in webkit [22285] by aroben
  • 2 edits in branches/WindowsMerge/WebCore

Reviewed by Maciej.

Fixes <rdar://problem/4743497>
Can't check checkboxes

Let Windows determine which mouse button is pressed.

  • platform/win/PlatformMouseEventWin.cpp: (WebCore::PlatformMouseEvent::PlatformMouseEvent): Use message parameter to determine which mouse button is pressed.
9:26 PM Changeset in webkit [16731] by aroben
  • 3 edits in trunk/WebCore

Reviewed by Maciej.

Let Windows determine which mouse button is pressed.

  • platform/PlatformMouseEvent.h: Add message parameter to constructor.
  • platform/win/MouseEventWin.cpp: (WebCore::PlatformMouseEvent::PlatformMouseEvent): Use message parameter to determine which mouse button is pressed.
7:00 PM Changeset in webkit [16730] by mjs
  • 8 edits in trunk/WebKit

Reviewed by Alice.


  • take away direct knowledge of WebFrame from WebDataSource
  • Loader/WebDocumentLoadState.h:
  • Loader/WebDocumentLoadState.m: (-[WebDocumentLoadState setFrameLoader:]): (-[WebDocumentLoadState detachFromFrameLoader]):
  • Loader/WebFrameLoader.m: (-[WebFrameLoader _setDocumentLoadState:]): (-[WebFrameLoader _setProvisionalDocumentLoadState:]):
  • WebView/WebDataSource.m:
  • WebView/WebDataSourceInternal.h:
  • WebView/WebDocumentLoadStateMac.m: (-[WebDocumentLoadStateMac detachFromFrameLoader]):
  • WebView/WebFrame.m: (-[WebFrame _loadDataSource:withLoadType:formState:]):
6:38 PM Changeset in webkit [16729] by justing
  • 3 edits in trunk/WebCore

Reviewed by anders


Build fix


  • editing/SelectionController.cpp: (WebCore::SelectionController::empty):
  • editing/SelectionController.h:
6:08 PM Changeset in webkit [16728] by justing
  • 3 edits
    4 adds in trunk

LayoutTests:

Reviewed by john

  • editing/execCommand/create-list-with-hr-expected.checksum: Added.
  • editing/execCommand/create-list-with-hr-expected.png: Added.
  • editing/execCommand/create-list-with-hr-expected.txt: Added.
  • editing/execCommand/create-list-with-hr.html: Added.

WebCore:

Reviewed by john


<rdar://problem/4755287>
Writely Editor: Crash occurs at WebCore::maxDeepOffset(WebCore::Node const*) when attempting to create a OL/UL following a HR element

  • editing/markup.cpp: (WebCore::createMarkup): Migrate to isBlock (isBlockFlow is true for inline replaced elements). Don't skip blocks that aren't containers (like horizontal rules).
6:06 PM Changeset in webkit [16727] by darin
  • 5 edits in trunk/WebCore

2006-10-03 Darin Adler <Darin Adler>

Reviewed by Justin Garcia.

  • correct mistaken code that would restart blinking every time "invalidateSelection" is called even if the caret has not moved
  • editing/SelectionController.h:
  • editing/SelectionController.cpp: (WebCore::SelectionController::setSelection): Update for name change. (WebCore::SelectionController::recomputeCaretRect): New function. Computes caret rect and does any necessary invalidation if the rect changes. (WebCore::SelectionController::invalidateCaretRect): Renamed from needsCaretRepaint. Invalidates the caret rect unconditionally. Also calls recomputeCaretRect as a side effect.
  • page/Frame.cpp: (WebCore::Frame::invalidateSelection): Remove unneeded call to clearCaretRectIfNeeded. (WebCore::Frame::clearCaretRectIfNeeded): Updated for name change. (WebCore::Frame::selectionLayoutChanged): Restructured to use the new recomputeCaretRect function and not restart blinking if caret has not changed position. (WebCore::Frame::caretBlinkTimerFired): Removed some checks that are not needed since selectionLayoutChanged already checks these.
  • page/FramePrivate.h: Removed unused m_blinkCaret.
6:05 PM Changeset in webkit [16726] by bdakin
  • 2 edits in trunk/WebCore

Reviewed by Adam.

Build fix. Add counters to vcproj.

  • WebCore.vcproj/WebCore/WebCore.vcproj:
5:51 PM Changeset in webkit [22284] by hyatt
  • 1 edit in branches/WindowsMerge/WebCore/WebCore.vcproj/WebCore.vcproj

Fix build bustage on win32 caused by counters checkin.

5:38 PM Changeset in webkit [22283] by sfalken
  • 5 edits in branches/WindowsMerge

2006-10-02 Steve Falkenburg <sfalken@apple.com>

Reviewed by Geoff, Hyatt.


<rdar://4757389> Browser locks up while navigating to a new page
<rdar://4757407> Images fail to load at https://webmail.apple.com

  • WebFrame.cpp: Preserve old document until new doc has started loading. (WebFrame::loadDataSource): Don't destroy old document when a new load kicks off. (WebFrame::receivedResponse): Destroy old document when first data is received for new doc.
  • WebView.cpp: Fix painting suppression code. (WebViewWndProc): Suppress painting, mouse events until first layout instead of while loading.

2006-10-02 Steve Falkenburg <sfalken@apple.com>

Reviewed by Hyatt.

<rdar://4757387> (partial fix) Unable to sign on to my.yahoo.com
<rdar://4757405> (partial fix) Unable to log onto secure site (www.schwab.com)
<rdar://4757411> (partial fix) Unable to log onto any AppleConnect site (but using back and resubmiting works)

WinInet fixes.
Don't post data using HTTP when HTTPS should be used.
Fix redirect from POST to GET case.


There is a remaining bug with the way that we queue WinInet callbacks
still to be addressed.

  • platform/win/ResourceLoaderWin.cpp: (WebCore::addToOutstandingJobs): Skip dwContext of 0 (rollover case). (WebCore::ResourceLoaderWndProc): Call default wndproc properly.

Pass INTERNET_FLAG_SECURE when necessary for HTTPS.
Fix redirect code.
Initialize buffers struct.
Pass proper context in InternetReadFileExA

(WebCore::initializeOffScreenResourceLoaderWindow): Init a struct to {0} instead of using memset.
(WebCore::transferJobStatusCallback): Prevent future bugs by making an else a bit more explicit.
(WebCore::ResourceLoader::start): Pass proper port for HTTP vs. HTTPS.
(WebCore::ResourceLoader::cancel): Zero out handle when closing.

4:54 PM Changeset in webkit [16725] by mjs
  • 12 edits in trunk/WebKit

Reviewed by Darin.


  • move a big slice of data and logic from WebDataSource to WebDocumentLoadState


  • Loader/WebDocumentLoadState.h:
  • Loader/WebDocumentLoadState.m: (-[WebDocumentLoadState initWithRequest:]): (-[WebDocumentLoadState dealloc]): (-[WebDocumentLoadState originalRequestCopy]): (-[WebDocumentLoadState request]): (-[WebDocumentLoadState replaceRequestURLForAnchorScrollWithURL:]): (-[WebDocumentLoadState setRequest:]): (-[WebDocumentLoadState setResponse:]): (-[WebDocumentLoadState isStopping]): (-[WebDocumentLoadState bridge]): (-[WebDocumentLoadState setMainDocumentError:]): (-[WebDocumentLoadState mainDocumentError]): (-[WebDocumentLoadState clearErrors]): (-[WebDocumentLoadState mainReceivedError:complete:]): (-[WebDocumentLoadState stopLoading]): (-[WebDocumentLoadState setupForReplace]): (-[WebDocumentLoadState commitIfReady]): (-[WebDocumentLoadState finishedLoading]): (-[WebDocumentLoadState setCommitted:]): (-[WebDocumentLoadState isCommitted]): (-[WebDocumentLoadState setLoading:]): (-[WebDocumentLoadState isLoading]): (-[WebDocumentLoadState commitLoadWithData:]): (-[WebDocumentLoadState doesProgressiveLoadWithMIMEType:]): (-[WebDocumentLoadState receivedData:]): (-[WebDocumentLoadState setupForReplaceByMIMEType:]): (-[WebDocumentLoadState updateLoading]): (-[WebDocumentLoadState response]):
  • Loader/WebFrameLoader.h:
  • Loader/WebFrameLoader.m: (-[WebFrameLoader activeDocumentLoadState]): (-[WebFrameLoader activeDataSource]): (-[WebFrameLoader _archivedSubresourceForURL:]): (-[WebFrameLoader addPlugInStreamLoader:]): (-[WebFrameLoader removePlugInStreamLoader:]): (-[WebFrameLoader addSubresourceLoader:]): (-[WebFrameLoader removeSubresourceLoader:]): (-[WebFrameLoader documentLoadState]): (-[WebFrameLoader provisionalDocumentLoadState]): (-[WebFrameLoader stopLoading]): (-[WebFrameLoader _originalRequest]): (-[WebFrameLoader _receivedData:]): (-[WebFrameLoader _setRequest:]): (-[WebFrameLoader bridge]): (-[WebFrameLoader _handleFallbackContent]): (-[WebFrameLoader _isStopping]): (-[WebFrameLoader _setupForReplaceByMIMEType:]): (-[WebFrameLoader _setResponse:]): (-[WebFrameLoader _mainReceivedError:complete:]): (-[WebFrameLoader _finishedLoading]): (-[WebFrameLoader didReceiveServerRedirectForProvisionalLoadForFrame]): (-[WebFrameLoader finishedLoadingDocumentLoadState:]): (-[WebFrameLoader commitProvisitionalLoad]): (-[WebFrameLoader committedLoadWithDocumentLoadState:data:]): (-[WebFrameLoader isReplacing]): (-[WebFrameLoader setReplacing]): (-[WebFrameLoader revertToProvisionalWithDocumentLoadState:]): (-[WebFrameLoader documentLoadState:setMainDocumentError:]): (-[WebFrameLoader documentLoadState:mainReceivedCompleteError:]): (-[WebFrameLoader finalSetupForReplaceWithDocumentLoadState:]):
  • WebView/WebDataSource.m: (-[WebDataSourcePrivate dealloc]): (-[WebDataSource _prepareForLoadStart]): (-[WebDataSource _mainDocumentError]): (-[WebDataSource _finishedLoading]): (-[WebDataSource _receivedData:]): (-[WebDataSource _setMainDocumentError:]): (-[WebDataSource _clearUnarchivingState]): (-[WebDataSource _revertToProvisionalState]): (-[WebDataSource _receivedMainResourceError:complete:]): (-[WebDataSource _startLoading]): (-[WebDataSource _loadFromPageCache:]): (-[WebDataSource _bridge]): (-[WebDataSource _setPrimaryLoadComplete:]): (-[WebDataSource _URLForHistory]): (-[WebDataSource _setTitle:]): (-[WebDataSource _initWithDocumentLoadState:]): (-[WebDataSource request]): (-[WebDataSource response]): (-[WebDataSource isLoading]): (-[WebDataSource webArchive]):
  • WebView/WebDataSourceInternal.h:
  • WebView/WebDocumentLoadStateMac.m: (-[WebDocumentLoadStateMac initWithRequest:]):
  • WebView/WebFrame.m: (-[WebFrame _createItem:]): (-[WebFrame _receivedMainResourceError:]): (-[WebFrame _transitionToCommitted:]): (-[WebFrame _commitProvisionalLoad:]): (-[WebFrame _checkLoadCompleteForThisFrame]): (-[WebFrame _loadItem:withLoadType:]): (-[WebFrame _continueFragmentScrollAfterNavigationPolicy:formState:]): (-[WebFrame _didReceiveServerRedirectForProvisionalLoadForFrame]): (-[WebFrame _provisionalLoadStarted]):
  • WebView/WebFrameInternal.h:
4:51 PM Changeset in webkit [16724] by beidson
  • 2 edits in trunk/WebCore

Reviewed by Adele

If the Icon DB schema changes underneath you, the latest ToT can crash and burn when writing
icon data out because we don't correctly check SQL error conditions.

  • loader/icon/IconDataCache.cpp: (WebCore::IconDataCache::writeToDatabase):
4:36 PM Changeset in webkit [16723] by sfalken
  • 3 edits in trunk/WebCore

2006-10-02 Steve Falkenburg <sfalken@apple.com>

Reviewed by Hyatt.

Added method to find out whether a layout has occurred.

  • page/FrameView.cpp: (WebCore::FrameView::didFirstLayout):
  • page/FrameView.h:
4:25 PM Changeset in webkit [16722] by bdakin
  • 1 edit
    153 adds in trunk/LayoutTests

Minor harassment followed by abject surrender provided by ggggaren.

Tests for counters.

  • fast/css/counters/contentNone-expected.checksum: Added.
  • fast/css/counters/contentNone-expected.png: Added.
  • fast/css/counters/contentNone-expected.txt: Added.
  • fast/css/counters/contentNone.html: Added.
  • fast/css/counters/counter00-expected.checksum: Added.
  • fast/css/counters/counter00-expected.png: Added.
  • fast/css/counters/counter00-expected.txt: Added.
  • fast/css/counters/counter00.html: Added.
  • fast/css/counters/counter01-expected.checksum: Added.
  • fast/css/counters/counter01-expected.png: Added.
  • fast/css/counters/counter01-expected.txt: Added.
  • fast/css/counters/counter01.html: Added.
  • fast/css/counters/counter02-expected.checksum: Added.
  • fast/css/counters/counter02-expected.png: Added.
  • fast/css/counters/counter02-expected.txt: Added.
  • fast/css/counters/counter02.html: Added.
  • fast/css/counters/counter03-expected.checksum: Added.
  • fast/css/counters/counter03-expected.png: Added.
  • fast/css/counters/counter03-expected.txt: Added.
  • fast/css/counters/counter03.html: Added.
  • fast/css/counters/counter04-expected.checksum: Added.
  • fast/css/counters/counter04-expected.png: Added.
  • fast/css/counters/counter04-expected.txt: Added.
  • fast/css/counters/counter04.html: Added.
  • fast/css/counters/counter05-expected.checksum: Added.
  • fast/css/counters/counter05-expected.png: Added.
  • fast/css/counters/counter05-expected.txt: Added.
  • fast/css/counters/counter05.html: Added.
  • fast/css/counters/counter06-expected.checksum: Added.
  • fast/css/counters/counter06-expected.png: Added.
  • fast/css/counters/counter06-expected.txt: Added.
  • fast/css/counters/counter06.html: Added.
  • fast/css/counters/counter07-expected.checksum: Added.
  • fast/css/counters/counter07-expected.png: Added.
  • fast/css/counters/counter07-expected.txt: Added.
  • fast/css/counters/counter07.html: Added.
  • fast/css/counters/counter08-expected.checksum: Added.
  • fast/css/counters/counter08-expected.png: Added.
  • fast/css/counters/counter08-expected.txt: Added.
  • fast/css/counters/counter08.html: Added.
  • fast/css/counters/counter09-expected.checksum: Added.
  • fast/css/counters/counter09-expected.png: Added.
  • fast/css/counters/counter09-expected.txt: Added.
  • fast/css/counters/counter09.html: Added.
  • fast/css/counters/counter10-expected.checksum: Added.
  • fast/css/counters/counter10-expected.png: Added.
  • fast/css/counters/counter10-expected.txt: Added.
  • fast/css/counters/counter10.html: Added.
  • fast/css/counters/counter11-expected.checksum: Added.
  • fast/css/counters/counter11-expected.png: Added.
  • fast/css/counters/counter11-expected.txt: Added.
  • fast/css/counters/counter11.html: Added.
  • fast/css/counters/counter12-expected.checksum: Added.
  • fast/css/counters/counter12-expected.png: Added.
  • fast/css/counters/counter12-expected.txt: Added.
  • fast/css/counters/counter12.html: Added.
  • fast/css/counters/counter13-expected.checksum: Added.
  • fast/css/counters/counter13-expected.png: Added.
  • fast/css/counters/counter13-expected.txt: Added.
  • fast/css/counters/counter13.html: Added.
  • fast/css/counters/counter14-expected.checksum: Added.
  • fast/css/counters/counter14-expected.png: Added.
  • fast/css/counters/counter14-expected.txt: Added.
  • fast/css/counters/counter14.html: Added.
  • fast/css/counters/counter15-expected.checksum: Added.
  • fast/css/counters/counter15-expected.png: Added.
  • fast/css/counters/counter15-expected.txt: Added.
  • fast/css/counters/counter15.html: Added.
  • fast/css/counters/counter16-expected.checksum: Added.
  • fast/css/counters/counter16-expected.png: Added.
  • fast/css/counters/counter16-expected.txt: Added.
  • fast/css/counters/counter16.html: Added.
  • fast/css/counters/counters00-expected.checksum: Added.
  • fast/css/counters/counters00-expected.png: Added.
  • fast/css/counters/counters00-expected.txt: Added.
  • fast/css/counters/counters00.html: Added.
  • fast/css/counters/counters01-expected.checksum: Added.
  • fast/css/counters/counters01-expected.png: Added.
  • fast/css/counters/counters01-expected.txt: Added.
  • fast/css/counters/counters01.html: Added.
  • fast/css/counters/counters02-expected.checksum: Added.
  • fast/css/counters/counters02-expected.png: Added.
  • fast/css/counters/counters02-expected.txt: Added.
  • fast/css/counters/counters02.html: Added.
  • fast/css/counters/counters03-expected.checksum: Added.
  • fast/css/counters/counters03-expected.png: Added.
  • fast/css/counters/counters03-expected.txt: Added.
  • fast/css/counters/counters03.html: Added.
  • fast/css/counters/counters04-expected.checksum: Added.
  • fast/css/counters/counters04-expected.png: Added.
  • fast/css/counters/counters04-expected.txt: Added.
  • fast/css/counters/counters04.html: Added.
  • fast/css/counters/counters05-expected.checksum: Added.
  • fast/css/counters/counters05-expected.png: Added.
  • fast/css/counters/counters05-expected.txt: Added.
  • fast/css/counters/counters05.html: Added.
  • fast/css/counters/counters06-expected.checksum: Added.
  • fast/css/counters/counters06-expected.png: Added.
  • fast/css/counters/counters06-expected.txt: Added.
  • fast/css/counters/counters06.html: Added.
  • fast/css/counters/counters07-expected.checksum: Added.
  • fast/css/counters/counters07-expected.png: Added.
  • fast/css/counters/counters07-expected.txt: Added.
  • fast/css/counters/counters07.html: Added.
  • fast/css/counters/counters08-expected.checksum: Added.
  • fast/css/counters/counters08-expected.png: Added.
  • fast/css/counters/counters08-expected.txt: Added.
  • fast/css/counters/counters08.html: Added.
  • fast/css/counters/counters09-expected.checksum: Added.
  • fast/css/counters/counters09-expected.png: Added.
  • fast/css/counters/counters09-expected.txt: Added.
  • fast/css/counters/counters09.html: Added.
  • fast/css/counters/counters10-expected.checksum: Added.
  • fast/css/counters/counters10-expected.png: Added.
  • fast/css/counters/counters10-expected.txt: Added.
  • fast/css/counters/counters10.html: Added.
  • fast/css/counters/counters11-expected.checksum: Added.
  • fast/css/counters/counters11-expected.png: Added.
  • fast/css/counters/counters11-expected.txt: Added.
  • fast/css/counters/counters11.html: Added.
  • fast/css/counters/counters12-expected.checksum: Added.
  • fast/css/counters/counters12-expected.png: Added.
  • fast/css/counters/counters12-expected.txt: Added.
  • fast/css/counters/counters12.html: Added.
  • fast/css/counters/counters13-expected.checksum: Added.
  • fast/css/counters/counters13-expected.png: Added.
  • fast/css/counters/counters13-expected.txt: Added.
  • fast/css/counters/counters13.html: Added.
  • fast/css/counters/counters14-expected.checksum: Added.
  • fast/css/counters/counters14-expected.png: Added.
  • fast/css/counters/counters14-expected.txt: Added.
  • fast/css/counters/counters14.html: Added.
  • fast/css/counters/counters15-expected.checksum: Added.
  • fast/css/counters/counters15-expected.png: Added.
  • fast/css/counters/counters15-expected.txt: Added.
  • fast/css/counters/counters15.html: Added.
  • fast/css/counters/counters18-expected.checksum: Added.
  • fast/css/counters/counters18-expected.png: Added.
  • fast/css/counters/counters18-expected.txt: Added.
  • fast/css/counters/counters18.html: Added.
  • fast/css/counters/implied00-expected.checksum: Added.
  • fast/css/counters/implied00-expected.png: Added.
  • fast/css/counters/implied00-expected.txt: Added.
  • fast/css/counters/implied00.html: Added.
  • fast/css/counters/root-expected.checksum: Added.
  • fast/css/counters/root-expected.png: Added.
  • fast/css/counters/root-expected.txt: Added.
  • fast/css/counters/root.html: Added.
  • fast/css/counters/visibilityNone-expected.checksum: Added.
  • fast/css/counters/visibilityNone-expected.png: Added.
  • fast/css/counters/visibilityNone-expected.txt: Added.
  • fast/css/counters/visibilityNone.html: Added.
4:15 PM Changeset in webkit [16721] by bdakin
  • 17 edits
    7 adds in trunk/WebCore

Reviewed by Darin.

Initial implementation of CSS2 counters. See http://
bugs.webkit.org/show_bug.cgi?id=4980 for more details.

  • WebCore.xcodeproj/project.pbxproj:
  • css/CSSComputedStyleDeclaration.cpp: (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue): Return the increment/reset list now that this is implemented.
  • css/CSSPrimitiveValue.cpp: (WebCore::CSSPrimitiveValue::cssText):
  • css/Counter.h: (WebCore::Counter::Counter): (WebCore::Counter::~Counter): (WebCore::Counter::identifier): (WebCore::Counter::listStyle): (WebCore::Counter::separator): (WebCore::Counter::listStyleNumber): (WebCore::Counter::setIdentifier): (WebCore::Counter::setListStyle): (WebCore::Counter::setSeparator):
  • css/cssparser.cpp: (WebCore::CSSParser::addProperty): Take care of reset/increment (WebCore::CSSParser::parseValue): (WebCore::CSSParser::parseContent): content can now take counters (WebCore::CSSParser::parseCounterContent): Parse counter() and counters() (WebCore::CSSParser::parseCounter): Parse counter-reset and counter-increment
  • css/cssparser.h:
  • css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::matchUARules): (WebCore::CSSStyleSelector::applyProperty):
  • platform/PlatformString.h:
  • platform/String.cpp: (WebCore::String::insert): Implemented a version of insert that accepts a UChar* and a length.
  • platform/StringImpl.cpp: (WebCore::StringImpl::insert): Same as above.
  • platform/StringImpl.h:
  • rendering/CounterListItem.h: Added.
  • rendering/CounterNode.cpp: Added. (WebCore::CounterNode::CounterNode): (WebCore::CounterNode::insertAfter): (WebCore::CounterNode::removeChild): (WebCore::CounterNode::remove): (WebCore::CounterNode::setUsesSeparator): (WebCore::CounterNode::recountAndGetNext): (WebCore::CounterNode::recountTree): (WebCore::CounterNode::setSelfDirty): (WebCore::CounterNode::setParentDirty):
  • rendering/CounterNode.h: Added. (WebCore::CounterNode::~CounterNode): (WebCore::CounterNode::parent): (WebCore::CounterNode::previousSibling): (WebCore::CounterNode::nextSibling): (WebCore::CounterNode::firstChild): (WebCore::CounterNode::lastChild): (WebCore::CounterNode::value): (WebCore::CounterNode::setValue): (WebCore::CounterNode::count): (WebCore::CounterNode::setCount): (WebCore::CounterNode::setHasSeparator): (WebCore::CounterNode::isReset): (WebCore::CounterNode::hasSeparator): (WebCore::CounterNode::willNeedLayout): (WebCore::CounterNode::setWillNeedLayout): (WebCore::CounterNode::isRoot): (WebCore::CounterNode::setRenderer): (WebCore::CounterNode::renderer):
  • rendering/CounterResetNode.cpp: Added. (WebCore::CounterResetNode::CounterResetNode): (WebCore::CounterResetNode::insertAfter): (WebCore::CounterResetNode::removeChild): (WebCore::CounterResetNode::recountAndGetNext): (WebCore::CounterResetNode::setParentDirty): (WebCore::CounterResetNode::updateTotal):
  • rendering/CounterResetNode.h: Added. (WebCore::CounterResetNode::firstChild): (WebCore::CounterResetNode::lastChild): (WebCore::CounterResetNode::isReset): (WebCore::CounterResetNode::total):
  • rendering/RenderContainer.cpp: (WebCore::RenderContainer::updatePseudoChildForObject): Account for counter content.
  • rendering/RenderCounter.cpp: Added. (WebCore::RenderCounter::RenderCounter): (WebCore::RenderCounter::layout): (WebCore::toRoman): (WebCore::toLetterString): (WebCore::toHebrew): (WebCore::RenderCounter::convertValueToType): (WebCore::RenderCounter::calcMinMaxWidth):
  • rendering/RenderCounter.h: Added. (WebCore::RenderCounter::renderName): (WebCore::RenderCounter::isCounter):
  • rendering/RenderObject.cpp: (WebCore::getRenderObjectsToCounterNodeMaps): Maps RenderObjects to maps of CounterNodes (WebCore::RenderObject::RenderObject): (WebCore::RenderObject::destroy): Destroy the maps. (WebCore::RenderObject::findCounter): Finds/creates counters.
  • rendering/RenderObject.h: (WebCore::RenderObject::isCounter):
  • rendering/RenderStyle.cpp: (WebCore::StyleVisualData::StyleVisualData): (WebCore::RenderStyle::arenaDelete): (WebCore::RenderStyle::RenderStyle): (WebCore::RenderStyle::diff): (WebCore::RenderStyle::setContent): (WebCore::ContentData::clearContent): (WebCore::RenderStyle::counterDataEquivalent): (WebCore::hasCounter): (WebCore::RenderStyle::hasCounterReset): (WebCore::RenderStyle::hasCounterIncrement): (WebCore::readCounter): (WebCore::RenderStyle::counterReset): (WebCore::RenderStyle::counterIncrement):
  • rendering/RenderStyle.h: (WebCore::StyleVisualData::operator==): (WebCore::CounterData::CounterData): (WebCore::CounterData::identifier): (WebCore::CounterData::listStyle): (WebCore::CounterData::separator): (WebCore::ContentData::contentCounter): (WebCore::ContentData::): (WebCore::RenderStyle::counterIncrement): (WebCore::RenderStyle::counterReset): (WebCore::RenderStyle::setCounterIncrement): (WebCore::RenderStyle::setCounterReset): (WebCore::RenderStyle::setCounterResetList): (WebCore::RenderStyle::setCounterIncrementList): (WebCore::RenderStyle::counterResetValueList): (WebCore::RenderStyle::counterIncrementValueList):
2:03 PM Changeset in webkit [16720] by aroben
  • 3 edits in trunk/LayoutTests

Reviewed by Tim H.

This change fixes a couple of problems with the window-properties
test. window-properties' output changes depending on the presence or
non-presence of session history (see
http://bugs.webkit.org/show_bug.cgi?id=10965 for details). For this
reason, the test must always be run as part of a run of ALL layout
tests. The test results checked in were generated from an individual
run of this test, so the buildbots were always failing since they run
all the tests at once.

This change regenerates the results for the test using a full run of
layout tests. The test now also prints out object class names, and
will print an error message when the test is run by itself.

  • fast/js/window-properties-expected.txt:
  • fast/js/window-properties.html:
1:44 PM Changeset in webkit [16719] by justing
  • 2 edits in trunk/WebKit

Reviewed by john


<rdar://problem/4757583>
REGRESSION: tabbing into page focuses wrong control on 2nd pass
<rdar://problem/4757594>
REGRESSION: Form field is left with secondary selection after tabbing out of WebView

  • WebView/WebHTMLView.m: (-[NSArray maintainsInactiveSelection]): Replace code that I removed in error in the patch for 9642. Only leave inactive editable selections in the WebHTMLView if the nextResponder is in the same WebView.
1:30 PM Changeset in webkit [16718] by adele
  • 18 edits
    2 adds in trunk

LayoutTests:

Reviewed by Adam.

  • Updated results for <rdar://problem/4463841> Switch to use new list box implementation for <select multiple>
  • fast/forms/HTMLOptionElement_label05-expected.checksum:
  • fast/forms/HTMLOptionElement_label05-expected.png:
  • fast/forms/HTMLOptionElement_label05-expected.txt:
  • fast/forms/form-element-geometry-expected.checksum:
  • fast/forms/form-element-geometry-expected.png:
  • fast/forms/form-element-geometry-expected.txt:
  • fast/forms/option-strip-whitespace-expected.checksum:
  • fast/forms/option-strip-whitespace-expected.png:
  • fast/forms/option-strip-whitespace-expected.txt:
  • fast/forms/password-doubleclick-selection-expected.txt: Added.
  • fast/forms/password-doubleclick-selection.html: Added.
  • fast/forms/select-change-popup-to-listbox-expected.checksum:
  • fast/forms/select-change-popup-to-listbox-expected.png:
  • fast/forms/select-change-popup-to-listbox-expected.txt:
  • fast/text/updateNewFont-expected.checksum:
  • fast/text/updateNewFont-expected.png:
  • fast/text/updateNewFont-expected.txt:

WebCore:

Reviewed by Adam.

  • Fix for <rdar://problem/4463841> Switch to use new list box implementation for <select multiple>

There are a few outstanding issues, so I've left the old code in for now. So you can use the old-style listboxes
by using "-webkit-appearance:none".

  • css/html4.css:
11:48 AM Changeset in webkit [22282] by aroben
  • 2 edits in branches/WindowsMerge/WebKitWin

Build fix: Don't copy WebKit(_debug).dll into $WebKitSDKDir

  • WebKit.vcproj/WebKit.vcproj: Remove xcopy call
9:46 AM Changeset in webkit [16717] by aroben
  • 2 edits in trunk/WebKitTools/DumpRenderTree

Backing out these accidental commits.

9:33 AM Changeset in webkit [16716] by aroben
  • 7 edits in trunk

2006-10-02 Nikolas Zimmermann <zimmermann@kde.org>

JavaScriptCore:

Reviewed/landed by Adam.

Build testkjs on Qt/Linux.

  • CMakeLists.txt:

WebKitTools:

Reviewed/landed by Adam.

Proper build-webkit support for Qt/Linux.

Compilation process is similar to OSX now, aka.
the build directory is RootCheckoutDir/WebKitBuild now.

  • Scripts/build-webkit: Recognize Qt.
  • Scripts/webkitdirs.pm: Add buildCMakeProject() logic.
8:32 AM Changeset in webkit [16715] by harrison
  • 2 edits in trunk/WebCore

Reverted the code part of r16696, my patch for <rdar://problem/4641262>, because the problem was
addressed at a deeper level by r16696, the patch for <http://bugs.webkit.org/show_bug.cgi?id=10842>.
Left the layout test.


  • rendering/bidi.cpp: (WebCore::checkMidpoints):
8:24 AM Changeset in webkit [16714] by zbujtas
  • 3 edits in S60/trunk/WebKit

2006-10-02 sareen <shyam.sareen@nokia.com>

Reviewed by Zalan Bujtas.
DESC: Include correct path for inc directory for S60 only.
http://bugs.webkit.org/show_bug.cgi?id=11122

  • group/BrowserView.mmp:
  • group/webkit.mmp:
3:57 AM Changeset in webkit [16713] by eseidel
  • 2 edits in trunk/JavaScriptCore

2006-10-02 Nikolas Zimmermann <zimmermann@kde.org>

Reviewed by eseidel. Landed by eseidel.

Fix win32 build, which has no inttypes.h

  • wtf/Assertions.h:
3:30 AM Changeset in webkit [16712] by eseidel
  • 4 edits in trunk

2006-10-02 Nikolas Zimmermann <zimmermann@kde.org>

Reviewed by eseidel. Landed by eseidel.

Fix Qt/Linux build.

  • platform/qt/TemporaryLinkStubs.cpp: s/DeprecatedStringList/Vector<String>/ (WebCore::supportedKeySizes):
3:26 AM Changeset in webkit [16711] by eseidel
  • 7 edits in trunk

2006-10-02 Nikolas Zimmermann <zimmermann@kde.org>

Reviewed by eseidel & mjs. Landed by eseidel.

Fix Qt/Linux build with older gcc 3.3.4.
http://bugs.webkit.org/show_bug.cgi?id=11116

As discussed with Maciej, the GCC_ROOT_NS_HACK
can be completely removed, as well as the friendship
between cacheGlobalObject & the JS* objects.

  • bindings/scripts/CodeGeneratorJS.pm: Remove friendship.
  • platform/image-decoders/png/pnggccrd.c: Fix comments for gcc3. (png_read_filter_row_mmx_avg):
  • platform/image-decoders/png/pngvcrd.c: Ditto. (png_mmx_support): (png_read_filter_row_mmx_avg):
12:46 AM Changeset in webkit [16710] by mjs
  • 4 edits in trunk/WebKit

Reviewed by Anders.


  • remove webFrame field from WebDataSourcePrivate, it can get it from WebDocumentLoadState now
  • Loader/WebDocumentLoadState.h:
  • Loader/WebDocumentLoadState.m: (-[WebDocumentLoadState frameLoader]):
  • WebView/WebDataSource.m: (-[WebDataSourcePrivate dealloc]): (-[WebDataSource _revertToProvisionalState]): (-[WebDataSource _setupForReplaceByMIMEType:]): (-[WebDataSource _updateLoading]): (-[WebDataSource _startLoading]): (-[WebDataSource _setWebFrame:]): (-[WebDataSource _defersCallbacksChanged]): (-[WebDataSource _stopLoading]): (-[WebDataSource _webView]): (-[WebDataSource _stopLoadingWithError:]): (-[WebDataSource _setPrimaryLoadComplete:]): (-[WebDataSource dealloc]): (-[WebDataSource webFrame]): (-[WebDataSource isLoading]):
12:39 AM Changeset in webkit [16709] by mjs
  • 3 edits in trunk/WebKit

Reviewed by Anders.


  • fix crash on back/forward - reattach WebDocumentLoadState to data source when needed
  • WebView/WebDataSource.m: (-[WebDataSource _setWebFrame:]): (-[WebDataSource _initWithDocumentLoadState:]):
  • WebView/WebDataSourceInternal.h:
12:16 AM Changeset in webkit [16708] by mjs
  • 1 edit
    2 adds in trunk/WebKit

Build fix, not reviewed.


  • Added missing files to fix build.
  • WebView/WebDocumentLoadStateMac.h: Added.
  • WebView/WebDocumentLoadStateMac.m: Added. (-[WebDocumentLoadStateMac initWithRequest:]): (-[WebDocumentLoadStateMac dealloc]): (-[WebDocumentLoadStateMac setDataSource:]): (-[WebDocumentLoadStateMac dataSource]): (-[WebDocumentLoadStateMac setFrameLoader:]):
12:00 AM Changeset in webkit [16707] by mjs
  • 10 edits in trunk/WebKit

Reviewed by Brady and Oliver.


  • move things around so that WebDataSource and WebDocumentLoadState know about each other in the right way. This lines things up to move nearly all functionality down to WebDocumentLoadState.
  • Loader/WebDocumentLoadState.h:
  • Loader/WebDocumentLoadState.m: (-[WebDocumentLoadState initWithRequest:]): (-[WebDocumentLoadState dealloc]): (-[WebDocumentLoadState originalRequest]):
  • Loader/WebFrameLoader.h:
  • Loader/WebFrameLoader.m: (-[WebFrameLoader dealloc]): (-[WebFrameLoader addSubresourceLoader:]): (-[WebFrameLoader startLoadingMainResourceWithRequest:identifier:]): (-[WebFrameLoader dataSource]): (-[WebFrameLoader _setDocumentLoadState:]): (-[WebFrameLoader policyDataSource]): (-[WebFrameLoader _setPolicyDocumentLoadState:]): (-[WebFrameLoader clearDataSource]): (-[WebFrameLoader provisionalDataSource]): (-[WebFrameLoader _setProvisionalDocumentLoadState:]): (-[WebFrameLoader _clearProvisionalDataSource]): (-[WebFrameLoader _setState:]): (-[WebFrameLoader clearProvisionalLoad]): (-[WebFrameLoader commitProvisionalLoad]): (-[WebFrameLoader stopLoading]): (-[WebFrameLoader startLoading]): (-[WebFrameLoader startProvisionalLoad:]): (-[WebFrameLoader setupForReplace]): (-[WebFrameLoader activeDocumentLoadState]): (-[WebFrameLoader activeDataSource]): (-[WebFrameLoader _loadRequest:archive:]): (-[WebFrameLoader _loadRequest:triggeringAction:loadType:formState:]): (-[WebFrameLoader _reloadAllowingStaleDataWithOverrideEncoding:]): (-[WebFrameLoader reload]):
  • WebKit.xcodeproj/project.pbxproj:
  • WebView/WebDataSource.m: (-[WebDataSourcePrivate dealloc]): (-[WebDataSource _startLoading]): (-[WebDataSource _setWebFrame:]): (-[WebDataSource _documentLoadState]): (-[WebDataSource _initWithDocumentLoadState:]): (-[WebDataSource initWithRequest:]): (-[WebDataSource initialRequest]): (-[WebDataSource unreachableURL]):
  • WebView/WebDataSourceInternal.h:
  • WebView/WebFrame.m: (-[WebFramePrivate dealloc]): (-[WebFrame _continueLoadRequestAfterNavigationPolicy:formState:]): (-[WebFrame _loadDataSource:withLoadType:formState:]): (-[WebFrame _policyDataSource]): (-[WebFrame _shouldReloadToHandleUnreachableURLFromRequest:]): (-[WebFrame _dataSourceForDocumentLoadState:]): (-[WebFrame _createDocumentLoadStateWithRequest:]):
  • WebView/WebFrameInternal.h:

Oct 1, 2006:

10:16 PM Changeset in webkit [16706] by mjs
  • 3 edits
    2 deletes in trunk

LayoutTests:

Rubber stamped by Alexey.

  • fast/dom/css-insert-import-rule-expected.txt: Removed.
  • fast/dom/css-insert-import-rule.html: Removed.

WebCore:

Rubber stamped by Alexey.

  • css/CSSGrammar.y:
9:36 PM Changeset in webkit [16705] by eseidel
  • 3 edits in trunk/WebCore

2006-10-01 Eric Seidel <eric@webkit.org>

Reviewed by mjs.

Eliminate a few more hotspots in SVG drawing code.


No test cases were harmed in the making of this patch. (And SVG has no perf tests.)

  • kcanvas/device/quartz/KRenderingPaintServerQuartz.mm: (WebCore::KRenderingPaintServerSolidQuartz::setup): use a single shared colorspace, avoid a few mallocs
  • platform/mac/ColorMac.mm: (+[WebCoreControlTintObserver WebCore]): store a single shared colorspace instance
6:20 PM Changeset in webkit [16704] by bdash
  • 5 edits in trunk/WebKitTools

2006-10-01 Mark Rowe <opendarwin.org@bdash.net.nz>

Reviewed by Maciej.

Add QT build slave to Buildbot.

  • BuildSlaveSupport/build.webkit.org-config/webkit/builders.py: Add QT build slave, fix existing errors.
  • BuildSlaveSupport/build.webkit.org-config/webkit/factories.py: Add QT build factory.
  • BuildSlaveSupport/build.webkit.org-config/webkit/status.py: Disable forcing of builds via web interface to prevent spamming. Use the IRC bot in #webkit-build instead.
  • BuildSlaveSupport/build.webkit.org-config/webkit/steps.py: Add CMake build step.
6:02 PM Changeset in webkit [16703] by ggaren
  • 7 edits
    2 adds in trunk

LayoutTests:

Added test for the viewource attribute, which applies to <frame> and
<iframe> elements.

  • fast/frames/viewsource-attribute-expected.txt: Added.
  • fast/frames/viewsource-attribute.html: Added.
  • fast/frames/frameElement-widthheight.html: Removed stray character.

WebCore:

Reviewed by Maciej.


More frame/iframe merging.


  • Removed needWidgetUpdate and related code. needWidgetUpdate is always false, so this was dead code.
  • Removed FIXME about setInViewSourceMode inside openURL(). openURL(), rather than attach(), is the correct place for setInViewSourceMode, because openURL() is the function that creates our frame.
  • Moved IFRAME insertedIntoDocument() code into FRAME, and removed FRAME code that did the same thing in other places.
  • Made FRAME's attach() method suffuciently generic so that IFRAME could call up to it, rather than skipping its superclass and calling up directly to ELEMENT.
  • Changed a few IFRAME up-calls to ELEMENT into up-calls to FRAME.
  • Replaced ad hoc frame loading code in FRAME::attach() with call to openURL(), the designated frame loading function.

Layout tests pass. I added a layout test for viewsource mode, since I
broke it in the course of writing this patch.

5:04 PM Changeset in webkit [16702] by eseidel
  • 1 edit
    1 add in trunk

2006-10-01 Nikolas Zimmermann <zimmermann@kde.org>

Reviewed by eseidel.

  • CMakeLists.txt: add option to disable colored output when building on the buildbot
1:45 PM Changeset in webkit [16701] by andersca
  • 11 edits in trunk/WebCore

2006-10-01 Anders Carlsson <acarlsson@apple.com>

Reviewed by Geoff.

  • dom/Document.cpp: (WebCore::Document::recalcStyleSelector):
  • dom/Document.h: Get rid of availableStyleSheets() and remove DeprecatedStringList.h include.


  • dom/Node.h: Remove DeprecatedStringList class forward declaration.


  • editing/markup.cpp: Include DeprecatedStringList.h here since it's not included by Document.h anymore.


  • kcanvas/KCanvasTreeDebug.cpp:
  • kcanvas/KCanvasTreeDebug.h: Remove unused functions.


  • loader/loader.cpp:
  • page/Frame.cpp:
  • page/Frame.h:
  • page/FrameView.h: Fixup DeprecatedStringList.h includes.
9:06 AM Changeset in webkit [16700] by harrison
  • 3 edits
    4 adds in trunk

LayoutTests:

Reviewed by John Sullivan.

<rdar://problem/4641262> REGRESSION: Japanese text corrupts on wrapping point

  • fast/text/international/wrap-CJK-001-expected.checksum: Added.
  • fast/text/international/wrap-CJK-001-expected.png: Added.
  • fast/text/international/wrap-CJK-001-expected.txt: Added.
  • fast/text/international/wrap-CJK-001.html: Added.

WebCore:

Reviewed by John Sullivan.

<rdar://problem/4641262> REGRESSION: Japanese text corrupts on wrapping point

Problem was that the decision to trim was based only on whether the character
is a soft hyphen, which caused pretty much any Japanese character to go.
Changed to decide based on whether the character is ignorable whitespace.


Test:

  • fast/text/international/wrap-CJK-001.html
  • rendering/bidi.cpp: (WebCore::isTrimmableChar): New. Checks whether character is whitespace that can be ignored according to the text node's style. trimmed from the end of wrapped line. (WebCore::checkMidpoints): Call isTrimmable() rather than checking for char != SOFT_HYPHEN.
8:40 AM Changeset in webkit [16699] by andersca
  • 14 edits in trunk/WebCore

2006-10-01 Anders Carlsson <acarlsson@apple.com>

Reviewed by Mitz Pettel.

Convert the SVG code to use Vector<String> instead of DeprecatedStringList.

  • ksvg2/svg/SVGColor.cpp: (WebCore::SVGColor::setRGBColor):
  • ksvg2/svg/SVGComponentTransferFunctionElement.cpp:
  • ksvg2/svg/SVGFEDiffuseLightingElement.cpp: (WebCore::SVGFEDiffuseLightingElement::parseMappedAttribute):
  • ksvg2/svg/SVGFEGaussianBlurElement.cpp: (WebCore::SVGFEGaussianBlurElement::parseMappedAttribute):
  • ksvg2/svg/SVGFESpecularLightingElement.cpp: (SVGFESpecularLightingElement::parseMappedAttribute):
  • ksvg2/svg/SVGFETurbulenceElement.cpp: (WebCore::SVGFETurbulenceElement::parseMappedAttribute):
  • ksvg2/svg/SVGLengthList.cpp:
  • ksvg2/svg/SVGNumberList.cpp: (SVGNumberList::parse):
  • ksvg2/svg/SVGNumberList.h:
  • ksvg2/svg/SVGPreserveAspectRatio.cpp: (WebCore::SVGPreserveAspectRatio::parsePreserveAspectRatio):
  • ksvg2/svg/SVGStringList.cpp: (WebCore::SVGStringList::reset):
  • ksvg2/svg/SVGStringList.h:
  • ksvg2/svg/SVGTransformable.cpp: (SVGTransformable::parseTransformAttribute):
3:51 AM Changeset in webkit [22281] by hyatt
  • 2 edits in branches/WindowsMerge/WebCore

Finish the win32 scrollbar.

3:05 AM Changeset in webkit [16698] by ap
  • 5 edits in trunk/WebCore

2006-10-01 Nikolas Zimmermann <zimmermann@kde.org>

Reviewed by Eric, tweaked and landed by ap.

Fix Qt/Linux build.

  • CMakeLists.txt:
  • platform/qt/PlatformScrollBar.h:
  • platform/qt/TemporaryLinkStubs.cpp: (WebCore::ScrollBar::ScrollBar): (WebCore::PlatformScrollBar::PlatformScrollBar): (FrameView::passMousePressEventToSubframe): (FrameView::passMouseMoveEventToSubframe): (FrameView::passMouseReleaseEventToSubframe): (FrameView::passWheelEventToSubframe): (FrameView::passMousePressEventToScrollbar):
  • ksvg2/svg/SVGDocument.h:
2:11 AM Changeset in webkit [16697] by ap
  • 3 edits in trunk

2006-10-01 Mitz Pettel <opendarwin.org@mitzpettel.com>

Reviewed by Darin.

Fix for http://bugs.webkit.org/show_bug.cgi?id=10190
REGRESSION: Repro crash when navigating away from an image document that hasn't finished loading

No test case added (probably requires new DumpRenderTree functionality)

  • rendering/RenderImage.cpp: (WebCore::RenderImage::imageChanged): Added early return if the document is being destroyed.
2:06 AM Changeset in webkit [16696] by ap
  • 3 edits
    4 adds in trunk

Reviewed by Hyatt.

Fix for http://bugzilla.opendarwin.org/show_bug.cgi?id=10842
REGRESSION (r15418): contenteditable div truncates rightmost Japanese character


Make sure we only break after a space (and not any other valid line-break) if
the style is -webkit-line-break: after-white-space (e.g. for a contentEditable div).
I also did a logic shuffle at Mitz's request to prevent doing an if on the same expression
twice in quick succession.

Test: fast/text/line-breaks-after-white-space.html

  • rendering/bidi.cpp: (WebCore::RenderBlock::findNextLineBreak):
1:42 AM Changeset in webkit [22280] by hyatt
  • 4 edits in branches/WindowsMerge/WebCore

Make scrollbar thumb dragging work.

1:37 AM Changeset in webkit [16695] by ap
  • 2 edits in trunk

Fixed Bugzilla references in ChangeLogs for my previous commit.

1:35 AM Changeset in webkit [16694] by ap
  • 3 edits
    2 adds in trunk

Reviewed by Maciej and Darin.

http://bugzilla.opendarwin.org/show_bug.cgi?id=4372
JavaScript document.write page form does not submit in Safari

Test: fast/forms/document-write.html

  • page/Frame.cpp: (WebCore::Frame::submitForm): Don't check for parsing().
1:13 AM Changeset in webkit [16693] by ap
  • 2 edits in trunk/WebCore

Build fix.

  • bridge/mac/FrameViewMac.mm: Add missing #imports.
12:32 AM Changeset in webkit [22279] by hyatt
  • 4 edits in branches/WindowsMerge/WebCore

Hook up the pressed state of scrollbars.

12:26 AM Changeset in webkit [16692] by hyatt
  • 6 edits in trunk/WebCore

Refactor the code that sends the mouse down to scrollbars.

Note: See TracTimeline for information about the timeline view.