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.

Sep 30, 2006:

5:41 PM Changeset in webkit [22278] by hyatt
  • 2 edits in branches/WindowsMerge/WebKitWin

Use BitBlt instead of SetDIBBitsToDevice

4:33 PM Changeset in webkit [16691] by weinig
  • 26 edits in trunk/WebCore

Reviewed by Darin.

Patch for http://bugs.webkit.org/show_bug.cgi?id=11102
Add more SVG Objective-C DOM bindings

  • Auto-generate Objective-C DOM bindings for DOMSVGAElement, DOMSVGAnimateColorElement, DOMSVGAnimateElement, DOMSVGAnimateTransformElement, DOMSVGAnimatedPathData, DOMSVGAnimatedPoints, DOMSVGAnimatedPreserveAspectRatio, DOMSVGAnimationElement, DOMSVGCircleElement, DOMSVGClipPathElement, DOMSVGColor, DOMSVGCursorElement, DOMSVGDefsElement, DOMSVGDescElement, DOMSVGDocument, DOMSVGEllipseElement, DOMSVGPreserveAspectRatio, and DOMSVGURIReference.
  • DerivedSources.make:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/objc/DOMInternal.h:
  • bindings/objc/DOMSVG.h:
  • bindings/scripts/CodeGeneratorObjC.pm:
  • ksvg2/svg/SVGAElement.idl:
  • ksvg2/svg/SVGAnimateColorElement.idl:
  • ksvg2/svg/SVGAnimateElement.idl:
  • ksvg2/svg/SVGAnimateTransformElement.idl:
  • ksvg2/svg/SVGAnimatedPathData.idl:
  • ksvg2/svg/SVGAnimatedPoints.idl:
  • ksvg2/svg/SVGAnimatedPreserveAspectRatio.idl:
  • ksvg2/svg/SVGAnimationElement.idl:
  • ksvg2/svg/SVGCircleElement.idl:
  • ksvg2/svg/SVGClipPathElement.idl:
  • ksvg2/svg/SVGColor.idl:
  • ksvg2/svg/SVGCursorElement.idl:
  • ksvg2/svg/SVGDefsElement.idl:
  • ksvg2/svg/SVGDescElement.idl:
  • ksvg2/svg/SVGDocument.idl:
  • ksvg2/svg/SVGEllipseElement.idl:
  • ksvg2/svg/SVGPointList.idl:
  • ksvg2/svg/SVGPreserveAspectRatio.idl:
  • ksvg2/svg/SVGRect.idl:
  • ksvg2/svg/SVGSVGElement.idl:
4:18 PM Changeset in webkit [16690] by mjs
  • 3 edits in trunk/WebCore

Reviewed by Adele.

  • fix build
  • platform/FloatRect.h: (WebCore::FloatRect::contains): Added; not exactly the same semantic as IntRect containment but more appropriate for path-type stuff.
  • platform/cg/PathCG.cpp: (WebCore::Path::contains): Don't use enclosingIntRect, just use FloatRect::contains.
2:14 PM Changeset in webkit [16689] by ap
  • 29 edits
    5 adds in trunk

Reviewed by Darin.

http://bugs.webkit.org/show_bug.cgi?id=11011
External CSS is parsed as iso-8859-1 even though the main document is utf-8

Test: fast/encoding/css-charset-default.xhtml

  • loader/CachedResourceClient.h: (WebCore::CachedResourceClient::setCSSStyleSheet): (WebCore::CachedResourceClient::setXSLStyleSheet): Divided setStyleSheet into setCSSStyleSheet and setXSLStyleSheet. The former takes an additional charset parameter, to be passed to CSSStyleSheet constructor.
  • css/CSSImportRule.cpp: (WebCore::CSSImportRule::setCSSStyleSheet): (WebCore::CSSImportRule::insertedIntoParent): Default to parent stylesheet's charset.
  • css/CSSImportRule.h:
  • css/CSSStyleSheet.cpp: (WebCore::CSSStyleSheet::CSSStyleSheet):
  • css/CSSStyleSheet.h: Added an m_charset member to be used when loading child stylesheets. Removed an unused m_implicit member. Changed some String parameters to const String&.
  • css/StyleSheet.cpp: (WebCore::StyleSheet::StyleSheet):
  • css/StyleSheet.h: Changed some String parameters to const String&. Removed an unused (even unimplemented) constructor.
  • css/StyleSheetList.cpp: (WebCore::StyleSheetList::length): (WebCore::StyleSheetList::item):
  • css/StyleSheetList.h: Special-casing implicit stylesheets seemed to be dead code, removed.
  • dom/Document.cpp: (WebCore::Document::setCSSStyleSheet): (WebCore::Document::recalcStyleSelector):
  • dom/Document.h: Renamed setStyleSheet() to setCSSStyleSheet().
  • dom/ProcessingInstruction.cpp: (WebCore::ProcessingInstruction::checkStyleSheet): Pass a correct charset to requestCSSStyleSheet(). (WebCore::ProcessingInstruction::setCSSStyleSheet): (WebCore::ProcessingInstruction::setXSLStyleSheet): (WebCore::ProcessingInstruction::parseStyleSheet):
  • dom/ProcessingInstruction.h:
  • html/HTMLLinkElement.cpp: (WebCore::HTMLLinkElement::process): Default to document encoding if a charset attribute is not specified. (WebCore::HTMLLinkElement::setCSSStyleSheet):
  • html/HTMLLinkElement.h:
  • html/HTMLStyleElement.cpp: (WebCore::HTMLStyleElement::childrenChanged):
  • ksvg2/svg/SVGStyleElement.cpp: (WebCore::SVGStyleElement::childrenChanged): Inherit URL and charset from the document.
  • loader/Cache.cpp: (WebCore::Cache::requestCSSStyleSheet):
  • loader/Cache.h: Adjust for setStyleSheet() renaming.
  • loader/CachedCSSStyleSheet.h: Removed an unused constructor.
  • loader/CachedCSSStyleSheet.cpp: (WebCore::CachedCSSStyleSheet::ref): (WebCore::CachedCSSStyleSheet::deref): (WebCore::CachedCSSStyleSheet::checkNotify): Pass the encoding to setCSSStyleSheet().
  • loader/CachedXSLStyleSheet.cpp: (WebCore::CachedXSLStyleSheet::ref): (WebCore::CachedXSLStyleSheet::checkNotify):
  • loader/DocLoader.cpp: (WebCore::DocLoader::requestCSSStyleSheet):
  • loader/DocLoader.h:
  • page/Frame.cpp: (WebCore::UserStyleSheetLoader::UserStyleSheetLoader): (WebCore::UserStyleSheetLoader::setCSSStyleSheet):
  • xml/XSLImportRule.cpp: (WebCore::XSLImportRule::setXSLStyleSheet): (WebCore::XSLImportRule::loadSheet):
  • xml/XSLImportRule.h: Adjust for setStyleSheet() renaming.
10:08 AM Changeset in webkit [16688] by rwlbuis
  • 3 edits
    4 adds in trunk

Reviewed by eseidel.

http://bugs.webkit.org/show_bug.cgi?id=11096
Hit testing for polylines fails

Fix Path::contains so it handles filled, non-closed paths too.

3:11 AM Changeset in webkit [16687] by hyatt
  • 1 edit in trunk/WebCore/platform/win/PlatformScrollBar.h

Fix win32.

2:49 AM Changeset in webkit [16686] by hyatt
  • 1 edit in trunk/WebKit/WebView/WebPreferences.m

Revert minimum font size pref back to 1.

2:48 AM Changeset in webkit [16685] by hyatt
  • 1 edit in trunk/WebCore/platform/win/TemporaryLinkStubs.cpp

Fix win32 bustage caused by a typo.

2:46 AM Changeset in webkit [16684] by hyatt
  • 3 edits in trunk/WebCore

Fix mac bustage in FrameMac.mm and FrameViewMac.mm

2:37 AM Changeset in webkit [22277] by hyatt
  • 2 edits in branches/WindowsMerge/WebKitWin

Revert minimum font size pref back to 1.

2:34 AM Changeset in webkit [22276] by hyatt
  • 8 edits
    1 add in branches/WindowsMerge/WebCore

Make mouse events work in iframes.

2:30 AM Changeset in webkit [16683] by hyatt
  • 12 edits in trunk/WebCore

Event refactoring.

Sep 29, 2006:

10:42 PM Changeset in webkit [16682] by adele
  • 3 edits in trunk/WebCore

Reviewed by Adele.

Fixes windows bustage:
http://bugs.webkit.org/show_bug.cgi?id=11093

  • platform/win/PlatformScrollBar.h:
  • platform/win/TemporaryLinkStubs.cpp: (PlatformScrollBar::PlatformScrollBar): (ScrollBar::ScrollBar):
9:41 PM Changeset in webkit [22275] by hyatt
  • 1 edit in branches/WindowsMerge/WebKitWin/WebPreferences.cpp

Change minimum font size value to 0 on win32 as well.

8:25 PM Changeset in webkit [16681] by thatcher
  • 2 edits in trunk/LayoutTests

Bug 11041: fast/dom/isindex-001 needs new results
http://bugs.webkit.org/show_bug.cgi?id=11041

  • fast/dom/isindex-001-expected.txt:
8:15 PM Changeset in webkit [22274] by sfalken
  • 1 edit in branches/WindowsMerge/WebCore/ChangeLog

2006-09-29 Steve Falkenburg <sfalken@apple.com>

Fix the build.

  • platform/win/PlatformScrollBarWin.cpp:
7:08 PM Changeset in webkit [16680] by thatcher
  • 2 edits in tags/Safari-521.28

Versioning

6:57 PM Changeset in webkit [16679] by hyatt
  • 2 edits in trunk/WebKit

Change the default minimum font size pref to 0 in order to allow
font-size:0 to work.

  • WebView/WebPreferences.m: (+[WebPreferences initialize]):
6:55 PM Changeset in webkit [16678] by hyatt
  • 15 edits in trunk/WebCore

Eliminate RenderLayer::gScrollBar in favor of caching the scrollbar hit
in RenderObject::NodeInfo (and then propagating that Widget to
MouseEventWithHitTestResults).

This allows RenderListBox and RenderBlock to do the same thing when the
mouse is over the scrollbar.

Also land fix to support font-size: 0.

Reviewed by mjs (scrollbar), eric (font-size)

  • dom/Document.cpp: (WebCore::Document::prepareMouseEvent):
  • page/Frame.cpp: (WebCore::Frame::passWidgetMouseDownEventToWidget):
  • page/FrameView.cpp: (WebCore::selectCursor): (WebCore::FrameView::handleMouseMoveEvent): (WebCore::FrameView::dispatchMouseEvent):
  • page/MouseEventWithHitTestResults.cpp: (WebCore::MouseEventWithHitTestResults::MouseEventWithHitTestResults):
  • page/MouseEventWithHitTestResults.h: (WebCore::MouseEventWithHitTestResults::scrollbar):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::isPointInScrollbar): (WebCore::RenderBlock::nodeAtPoint):
  • rendering/RenderBlock.h:
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::hitTest):
  • rendering/RenderLayer.h: (WebCore::RenderLayer::getHiddenBehavior):
  • rendering/RenderListBox.cpp: (WebCore::RenderListBox::RenderListBox): (WebCore::RenderListBox::~RenderListBox): (WebCore::RenderListBox::isPointInScrollbar):
  • rendering/RenderListBox.h:
  • rendering/RenderObject.h: (WebCore::RenderObject::NodeInfo::NodeInfo): (WebCore::RenderObject::NodeInfo::scrollbar): (WebCore::RenderObject::NodeInfo::setScrollbar):
6:50 PM Changeset in webkit [16677] by adele
  • 3 edits in trunk/WebCore

Reviewed by Adele.

Fixes: http://bugs.webkit.org/show_bug.cgi?id=11092

  • platform/win/ScrollViewWin.cpp: (WebCore::ScrollView::convertToContainingWindow): (WebCore::ScrollView::convertFromContainingWindow):
  • platform/win/WidgetWin.cpp: (WebCore::Widget::convertToContainingWindow): (WebCore::Widget::convertFromContainingWindow):
6:44 PM Changeset in webkit [16676] by adele
  • 2 edits in trunk/WebCore

Updated with new argument for PlatformScrollBar.

  • platform/win/TemporaryLinkStubs.cpp: (PlatformScrollBar::PlatformScrollBar):
6:03 PM Changeset in webkit [16675] by thatcher
  • 3 edits
    2 copies in tags/Safari-521.28

Merging a fix to the tag.

6:00 PM Changeset in webkit [16674] by thatcher
  • 16 edits in tags/Safari-521.28

Merging a fix to the tag.

5:44 PM Changeset in webkit [16673] by thatcher
  • 13 edits in tags

Merging fix to tag.

5:42 PM Changeset in webkit [16672] by thatcher
  • 3 edits
    2 copies in tags

LayoutTests:

Reviewed by John Sullivan.

<rdar://problem/4663772> REGRESSION: Cannot type in Japanese after replying to a particular message
<rdar://problem/4673293> REGRESSION: Can't enter the Japanese characters in Mail or Blot

  • fast/text/attributed-substring-from-range-001-expected.txt: Added.
  • fast/text/attributed-substring-from-range-001.html: Added.


Check attributed string results when starting or ending at a br element.

WebCore:

Reviewed by John Sullivan.

<rdar://problem/4663772> REGRESSION: Cannot type in Japanese after replying to a particular message
<rdar://problem/4673293> REGRESSION: Can't enter the Japanese characters in Mail or Blot

Test added

  • fast/text/attributed-substring-from-range-001.html
  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::attributedString): Handle non-zero offsets when start and/or end node is a container. Offset used to be ignored in this case. Also, add validation of the range.
5:39 PM Changeset in webkit [16671] by thatcher
  • 2 copies in tags/Safari-521.28

New tag.

5:34 PM Changeset in webkit [16670] by thatcher
  • 1 delete in tags/Safari-521.28

Bad tag.

5:29 PM Changeset in webkit [16669] by thatcher
  • 3 copies in tags/Safari-521.28

New tag.

5:11 PM Changeset in webkit [22273] by sfalken
  • 2 edits in branches/WindowsMerge/WebKitWin

2006-09-29 Steve Falkenburg <sfalken@apple.com>

Reviewed by Geoff.


Allow WebKit to query for the rect
of the resizer rather than just its size.


Fix <rdar://4753220>
Correct more/fewer button toggling in bug report sheet.

2006-09-29 Steve Falkenburg <sfalken@apple.com>

Reviewed by Geoff.

Allow WebKit to query for the rect
of the resizer rather than just its size.

  • Interfaces/IWebUIDelegatePrivate.idl:
5:06 PM Changeset in webkit [16668] by thatcher
  • 1 edit in trunk/WebCore/html/HTMLSelectElement.cpp

Build fix.

4:59 PM Changeset in webkit [16667] by adele
  • 2 edits in trunk/WebCore

Reviewed by Hyatt.

Fixing build bustage with cast to int.

  • html/HTMLSelectElement.cpp: (WebCore::HTMLSelectElement::lastSelectedListIndex):
4:45 PM Changeset in webkit [16666] by adele
  • 2 edits in trunk/WebCore

Fixing accidental include.

  • rendering/RenderListBox.cpp:
4:41 PM Changeset in webkit [16665] by adele
  • 2 edits in trunk/WebCore

Adding RenderListBox files.

4:40 PM Changeset in webkit [22272] by adele
  • 2 edits in branches/WindowsMerge/WebCore

Adding RenderListBox files

4:15 PM Changeset in webkit [16664] by bdakin
  • 1 edit in trunk/WebCore/ChangeLog

Fixing typo. No problem in the ned. Just the end.

4:15 PM Changeset in webkit [22271] by adele
  • 1 edit in branches/WindowsMerge/WebCore/ChangeLog

Adding reviewer to ChangeLog

4:15 PM Changeset in webkit [22270] by adele
  • 2 edits in branches/WindowsMerge/WebCore

Reviewed by NOBODY (OOPS!).

Updated constructor for PlatformScrollBar to include controlSize argument.

  • platform/win/PlatformScrollBar.h:
  • platform/win/PlatformScrollBarWin.cpp: (WebCore::PlatformScrollBar::PlatformScrollBar):
4:14 PM Changeset in webkit [22269] by sfalken
  • 2 edits in branches/WindowsMerge/WebKitWin

2006-09-29 Steve Falkenburg <sfalken@apple.com>

Reviewed by Adam, Geoff.


Fix <rdar://4757392>
Plug-ins flash to white while scrolling.


Set WS_CLIPCHILDREN on browser windows to clip out plug-in HWNDs
when drawing.

  • WebView.cpp: (WebView::initWithFrame):
4:13 PM Changeset in webkit [16663] by adele
  • 26 edits
    2 adds in trunk/WebCore

Reviewed by Adam.

Initial implementation of engine-based list box control.

  • WebCore.xcodeproj/project.pbxproj: Added RenderListBox.h and RenderListBox.cpp
  • bridge/mac/FrameMac.h: Added _mouseDownMayStartAutoscroll.
  • bridge/mac/FrameMac.mm: Updated autoscroll code to use renderers instead of layers, so any renderer that implements autoscroll will work. (WebCore::FrameMac::FrameMac): (WebCore::FrameMac::handleMousePressEvent): (WebCore::FrameMac::handleMouseMoveEvent): (WebCore::FrameMac::mouseDown):
  • page/Frame.cpp: (WebCore::Frame::handleMouseMoveEvent): (WebCore::Frame::scrollOverflow): Don't scroll list box here- this would cause arrow keys to scroll instead of select. (WebCore::Frame::handleAutoscroll): Updated to use a renderer instead of a layer when setting up autoscroll. (WebCore::Frame::autoscrollTimerFired): ditto. (WebCore::Frame::stopAutoscrollTimer): ditto. (WebCore::Frame::passWidgetMouseDownEventToWidget): Updated to check for list box's scroll bar.
  • page/Frame.h:
  • page/FramePrivate.h: (WebCore::FramePrivate::FramePrivate): Updated to use renderer unstead of layer for autoscroll.
  • page/FrameView.cpp: Keep track of current mouse position so this can be used for list box autoscroll. (WebCore::FrameViewPrivate::reset): (WebCore::FrameView::currentMousePosition): (WebCore::FrameView::handleMousePressEvent): (WebCore::FrameView::handleMouseDoubleClickEvent): (WebCore::selectCursor): (WebCore::FrameView::handleMouseMoveEvent): (WebCore::FrameView::handleMouseReleaseEvent):
  • page/FrameView.h:
  • platform/ScrollBar.cpp: (WebCore::ScrollBar::ScrollBar): Added controlSize argument. The list box will use a smaller scroll bar size.
  • platform/ScrollBar.h: (WebCore::): (WebCore::ScrollBar::controlSize):
  • platform/mac/PlatformScrollBar.h:
  • platform/mac/PlatformScrollBarMac.mm: (NSControlSizeForScrollBarControlSize): (-[WebCoreScrollBar initWithPlatformScrollBar:]): (WebCore::PlatformScrollBar::PlatformScrollBar):
  • rendering/RenderBlock.h:
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::createScrollbar): Updated to pass regular control size to scrollbar constructor.
  • rendering/RenderLayer.h:
  • rendering/RenderObject.cpp: (WebCore::RenderObject::shouldAutoscroll): (WebCore::RenderObject::autoscroll):
  • rendering/RenderObject.h: (WebCore::RenderObject::isListBox):
  • css/html4.css: Added properties for new list boxes.
  • html/HTMLOptionElement.cpp: (WebCore::HTMLOptionElement::setSelected): Doesn't allow selection to be changed here if the option is disabled. (WebCore::HTMLOptionElement::disabled): Added. Checks the parent's disabled status.
  • html/HTMLOptionElement.h: Added disabled method.
  • html/HTMLSelectElement.cpp: Added appearance switch for new list box implementation. (WebCore::HTMLSelectElement::recalcStyle): (WebCore::HTMLSelectElement::lastSelectedListIndex): (WebCore::HTMLSelectElement::deselectItems): (WebCore::HTMLSelectElement::setSelectedIndex): (WebCore::HTMLSelectElement::isKeyboardFocusable): (WebCore::HTMLSelectElement::isMouseFocusable): (WebCore::HTMLSelectElement::createRenderer): (WebCore::HTMLSelectElement::recalcListItems): (WebCore::HTMLSelectElement::setRecalcListItems): (WebCore::HTMLSelectElement::reset): (WebCore::HTMLSelectElement::notifyOptionSelected): (WebCore::HTMLSelectElement::defaultEventHandler): Added code to select options for list box when clicking and using arrow keys. (WebCore::HTMLSelectElement::nextSelectableListIndex): (WebCore::HTMLSelectElement::previousSelectableListIndex):
  • html/HTMLSelectElement.h:
  • rendering/RenderTheme.cpp: Added support for ListBoxAppearance (WebCore::RenderTheme::paint): (WebCore::RenderTheme::paintBorderOnly): (WebCore::RenderTheme::paintDecorations): (WebCore::RenderTheme::activeListBoxSelectionBackgroundColor): (WebCore::RenderTheme::activeListBoxSelectionForegroundColor): (WebCore::RenderTheme::inactiveListBoxSelectionBackgroundColor): (WebCore::RenderTheme::inactiveListBoxSelectionForegroundColor): (WebCore::RenderTheme::isControlStyled): (WebCore::RenderTheme::supportsFocusRing):
  • rendering/RenderTheme.h:
  • rendering/RenderThemeMac.mm: (WebCore::RenderThemeMac::isControlStyled):
  • rendering/RenderListBox.cpp: Added. (WebCore::RenderListBox::RenderListBox): (WebCore::RenderListBox::~RenderListBox): (WebCore::RenderListBox::setStyle): (WebCore::RenderListBox::updateFromElement): (WebCore::RenderListBox::calcMinMaxWidth): (WebCore::RenderListBox::size): (WebCore::RenderListBox::numItems): (WebCore::RenderListBox::calcHeight): (WebCore::RenderListBox::baselinePosition): (WebCore::RenderListBox::itemBoundingBoxRect): (WebCore::RenderListBox::paintObject): (WebCore::RenderListBox::paintScrollbar): (WebCore::RenderListBox::paintItemForeground): (WebCore::RenderListBox::paintItemBackground): (WebCore::RenderListBox::scrollBarTarget): (WebCore::RenderListBox::isPointInScrollbar): (WebCore::RenderListBox::optionAtPoint): (WebCore::RenderListBox::autoscroll): (WebCore::RenderListBox::scrollToRevealElementAtListIndex): (WebCore::RenderListBox::scroll): (WebCore::RenderListBox::valueChanged):
  • rendering/RenderListBox.h: Added. (WebCore::RenderListBox::isListBox): (WebCore::RenderListBox::selectionChanged): (WebCore::RenderListBox::setSelectionChanged): (WebCore::RenderListBox::canHaveChildren): (WebCore::RenderListBox::renderName): (WebCore::RenderListBox::setOptionsChanged): (WebCore::RenderListBox::shouldAutoscroll): (WebCore::RenderListBox::listIndexIsVisible):
4:10 PM Changeset in webkit [16662] by bdakin
  • 3 edits in trunk/WebCore

Reviewed by Darin.

We had a bug where html4.css was ignoring Media Queries. In the ned
the problem was that we were trying to send a char* to the
MediaQueryEvaluator constructor that expects a String, but the
char* got interpreted as a bool, the wrong constructor was called,
and the media type was never set.

No test case possible since this only affects html4.css

  • css/MediaQueryEvaluator.cpp: New constructor that expects takes a char* (WebCore::MediaQueryEvaluator):
  • css/MediaQueryEvaluator.h: Same.
3:57 PM Changeset in webkit [16661] by ggaren
  • 5 edits
    3 adds in trunk

LayoutTests:

Reviewed by John, Maciej.


Added test for whether a frame element reports its src attribute as a
complete, rather than relative, URL.

  • fast/frames/frame-src-attribute-expected.txt: Added.
  • fast/frames/frame-src-attribute.html: Added.
  • fast/frames/resources/frame-src-attribute-subframe.html: Added.

WebCore:

Reviewed by John, Maciej.


Integrated some frame and iframe code. I'm trying to fix up frame ownership
and loading. Reducing the number of different code paths involved
seemed like a good first step.


As a side effect, I fixed a bug where FRAME elements would report their
src attributes as relative, rather than compelete, URLs. (IFRAME elements
had the correct complete URL behavior.)

  • html/HTMLFrameElement.cpp: (WebCore::HTMLFrameElement::isURLAllowed): Fixed comment typo (WebCore::HTMLFrameElement::openURL):

(1) Removed checks that requestFrame does for us
(2) Added isURLAllowed check, to have one clear bottleneck for it
(3) Added viewsource check, to have one clear bottleneck for it

(WebCore::HTMLFrameElement::close): Changed to use the common contentFrame()
method, instead of finding our content frame in our own unique way.
(WebCore::HTMLFrameElement::setLocation): Removed isURLAllowed check,
since openURL does this for us now.
(WebCore::HTMLFrameElement::src): Return complete URL instead of relative.
This is what FF does, and it made no sense to have different behaviors
for FRAME and IFRAME elements.

  • html/HTMLIFrameElement.cpp: (WebCore::HTMLIFrameElement::HTMLIFrameElement): Removed duplicate init code.
  • html/HTMLIFrameElement.h: Removed src() and openURL() methods, since HTMLFrameElement now does everything we need.
3:33 PM Changeset in webkit [16660] by rwlbuis
  • 5 edits in trunk/WebCore

Reviewed by Hyatt.

Fix Qt/Linux build.

3:28 PM Changeset in webkit [16659] by brmorris
  • 2 edits in S60/trunk/S60Tools

Reviewed by Brad.
DESC: Update bugzilla url

  • prepare-ChangeLog.bat:
3:24 PM Changeset in webkit [16658] by thatcher
  • 1 edit in trunk/LayoutTests/ChangeLog

Replace bugzilla.opendarwin.org with bugs.webkit.org.

3:24 PM Changeset in webkit [16657] by brmorris
  • 10 edits in S60/trunk

Update Bugzilla url to bugs.webkit.org in ChangeLogs

3:22 PM Changeset in webkit [16656] by thatcher
  • 1 edit in trunk/WebKitQt/ChangeLog

Replace bugzilla.opendarwin.org with bugs.webkit.org.

3:21 PM Changeset in webkit [16655] by thatcher
  • 1 edit in trunk/WebKitLibraries/ChangeLog

Replace bugzilla.opendarwin.org with bugs.webkit.org.

3:21 PM Changeset in webkit [16654] by thatcher
  • 1 edit in trunk/WebKitSite/ChangeLog

Replace bugzilla.opendarwin.org with bugs.webkit.org.

3:20 PM Changeset in webkit [16653] by thatcher
  • 1 edit in trunk/WebKitTools/ChangeLog

Replace bugzilla.opendarwin.org with bugs.webkit.org.

3:17 PM Changeset in webkit [16652] by thatcher
  • 1 edit in trunk/JavaScriptGlue/ChangeLog

Replace bugzilla.opendarwin.org with bugs.webkit.org.

3:16 PM Changeset in webkit [16651] by thatcher
  • 1 edit in trunk/JavaScriptCore/ChangeLog

Replace bugzilla.opendarwin.org with bugs.webkit.org.

3:14 PM Changeset in webkit [16650] by thatcher
  • 2 edits in trunk/WebKit

Replace bugzilla.opendarwin.org with bugs.webkit.org.

3:12 PM Changeset in webkit [16649] by thatcher
  • 3 edits in trunk/WebCore

Replace bugzilla.opendarwin.org with bugs.webkit.org.

3:08 PM Changeset in webkit [16648] by thatcher
  • 1 edit in trunk/WebCore/ChangeLog

Replace bugzilla.opendarwin.org with bugs.webkit.org.

2:56 PM Changeset in webkit [16647] by spadma
  • 4 edits in S60/branches/3.1m/WebKit

2006-09-29 spadma <Sachin.Padma@nokia.com>

Reviewed by Joseph Ligman.
DESC: Fix resolvecomplete issues with BufferTransaction
http://bugzilla.opendarwin.org/show_bug.cgi?id=10497

  • BrowserView/src/WebKitLoader.cpp: (CWebKitLoader::LoadPageL):
  • ResourceLoader/inc/BufferTransaction.h:
  • ResourceLoader/src/BufferTransaction.cpp: (CBufferTransaction::SendDataCB): (CBufferTransaction::ConstructL): (CBufferTransaction::~CBufferTransaction): (CBufferTransaction::HandleResolveComplete):
2:49 PM Changeset in webkit [16646] by spadma
  • 4 edits in S60/trunk/WebKit

2006-09-29 spadma <Sachin.Padma@nokia.com>

Reviewed by Joseph Ligman.
DESC: Fix resolvecomplete issues with BufferTransaction
http://bugzilla.opendarwin.org/show_bug.cgi?id=10497

  • BrowserView/src/WebKitLoader.cpp: (CWebKitLoader::LoadPageL):
  • ResourceLoader/inc/BufferTransaction.h:
  • ResourceLoader/src/BufferTransaction.cpp: (CBufferTransaction::SendDataCB): (CBufferTransaction::ConstructL): (CBufferTransaction::~CBufferTransaction): (CBufferTransaction::HandleResolveComplete):
1:29 PM Changeset in webkit [16645] by weinig
  • 26 edits in trunk/WebCore

Reviewed by Tim H.

Patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=11082
Simplify Internal methods for the auto-generated Objective-C DOM code

  • Auto-generate the internal methods for more Objective-C classes.
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/objc/DOM.mm:
  • bindings/objc/DOMCSS.mm: (-[DOMStyleSheet WebCore::]): (+[DOMStyleSheet _styleSheetWith:WebCore::]): (-[DOMCSSRule WebCore::]): (-[DOMCSSRule _initWithCSSRule:WebCore::]): (+[DOMCSSRule _CSSRuleWith:WebCore::]): (-[DOMCSSValue WebCore::]): (-[DOMCSSValue _initWithCSSValue:WebCore::]): (+[DOMCSSValue _CSSValueWith:WebCore::]):
  • bindings/objc/DOMHTMLAppletElement.mm: (-[DOMHTMLAppletElement WebCore::]): (+[DOMHTMLAppletElement _HTMLOptionElementWith:WebCore::]):
  • bindings/objc/DOMHTMLEmbedElement.mm: (-[DOMHTMLEmbedElement WebCore::]): (+[DOMHTMLEmbedElement _HTMLEmbedElementWith:WebCore::]):
  • bindings/objc/DOMInternal.h:
  • bindings/objc/DOMInternal.mm:
  • bindings/scripts/CodeGeneratorObjC.pm:
  • css/CSSCharsetRule.idl:
  • css/CSSFontFaceRule.idl:
  • css/CSSImportRule.idl:
  • css/CSSMediaRule.idl:
  • css/CSSPageRule.idl:
  • css/CSSPrimitiveValue.idl:
  • css/CSSStyleRule.idl:
  • css/CSSStyleSheet.idl:
  • css/CSSUnknownRule.idl:
  • css/CSSValueList.idl:
  • dom/KeyboardEvent.idl:
  • dom/MouseEvent.idl:
  • dom/MutationEvent.idl:
  • dom/NodeIterator.idl:
  • dom/OverflowEvent.idl:
  • dom/UIEvent.idl:
  • dom/WheelEvent.idl:
10:58 AM Changeset in webkit [16644] by spadma
  • 9 edits in S60/branches/3.1m

2006-09-29 bujtas <zbujtas@gmail.com>

Reviewed by Sachin Padma.
DESC: fix intrapage navigation vs. load issue
http://bugzilla.opendarwin.org/show_bug.cgi?id=10974

  • BrowserView/src/webkitbridge.cpp: (CWebKitBridge::LoadUrlL):
  • ResourceLoader/inc/LoaderUtils.h:
  • ResourceLoader/src/LoaderUtils.cpp: (TLoaderUtils::IsIntraPageNavigation):

2006-09-29 sareen <shyam.sareen@nokia.com>

Reviewed by spadma <Sachin.Padma@nokia.com>.
DESC: Avoid passing NULL URL to webcore.
http://bugzilla.opendarwin.org/show_bug.cgi?id=11056

  • ResourceLoader/src/FileTransaction.cpp: (CFileTransaction::CreateHeaderResponseLC):

2006-09-29 zhalu <zhaosong.lu@nokia.com>

Reviewed by Sachin.
DESC:If the web page is loaded from cache, favicon is only loaded from cache,
not trying to load it from network even if cache does not have it.


http://bugzilla.opendarwin.org/show_bug.cgi?id=10355

  • BrowserView/inc/WebKitLoader.h:
  • BrowserView/src/WebKitLoader.cpp: (CWebKitLoader::CWebKitLoader): (CWebKitLoader::LoadPageL): (CWebKitLoader::LoadFavicon):

2006-09-29 spadma <Sachin.Padma@nokia.com>

Reviewed by Yongjun.
DESC: Cancel the active object before deleting it.
http://bugzilla.opendarwin.org/show_bug.cgi?id=11087

  • Src/StopScheduler.cpp: (CStopScheduler::~CStopScheduler):
10:40 AM Changeset in webkit [16643] by spadma
  • 2 edits in S60/trunk/MemoryManager

2006-09-29 spadma <Sachin.Padma@nokia.com>

Reviewed by Yongjun.
DESC: Cancel the active object before deleting it.
http://bugzilla.opendarwin.org/show_bug.cgi?id=11087

  • Src/StopScheduler.cpp: (CStopScheduler::~CStopScheduler):
9:57 AM Changeset in webkit [16642] by harrison
  • 3 edits
    2 adds in trunk

LayoutTests:

Reviewed by John Sullivan.

<rdar://problem/4663772> REGRESSION: Cannot type in Japanese after replying to a particular message
<rdar://problem/4673293> REGRESSION: Can't enter the Japanese characters in Mail or Blot

  • fast/text/attributed-substring-from-range-001-expected.txt: Added.
  • fast/text/attributed-substring-from-range-001.html: Added.


Check attributed string results when starting or ending at a br element.

WebCore:

Reviewed by John Sullivan.

<rdar://problem/4663772> REGRESSION: Cannot type in Japanese after replying to a particular message
<rdar://problem/4673293> REGRESSION: Can't enter the Japanese characters in Mail or Blot

Test added

  • fast/text/attributed-substring-from-range-001.html
  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::attributedString): Handle non-zero offsets when start and/or end node is a container. Offset used to be ignored in this case. Also, add validation of the range.
9:10 AM Changeset in webkit [16641] by thatcher
  • 5 edits
    5 adds
    8 deletes in trunk/WebKit

Reviewed by Maciej.

Switch the inspector over to use border-image with styled buttons and selects.

9:08 AM Changeset in webkit [16640] by zbujtas
  • 5 edits in S60/trunk

2006-09-25 zhalu <zhaosong.lu@nokia.com>

Reviewed by Zalan Bujtas.
DESC: For QColor object, there is a 'valid' flag indicating if the color is valid. The information for the valid flag is lost during the convertion of KWQPalette to WebCorePalette because the WebCorePalette uses only RGB values of color. This causes problem such as described in the bug report. The changes add the valid flag to the WebCorePalette class.
http://bugzilla.opendarwin.org/show_bug.cgi?id=10174


Merged from \s60\branches\3.1m r16635
WARNING: NO TEST CASES ADDED OR CHANGED

  • bridge/WebCorePalette.h: (TWebCorePalette::TWebCorePalette): (TWebCorePalette::SetBackGroundValid): (TWebCorePalette::SetForeGroundValid): (TWebCorePalette::IsBackGroundValid): (TWebCorePalette::IsForeGroundValid):
  • kwq/KWQWidget.cpp: (QWidget::setPalette):
9:03 AM Changeset in webkit [16639] by spadma
  • 4 edits in S60/trunk/WebKit

2006-09-29 bujtas <zbujtas@gmail.com>

Reviewed by Sachin Padma.
DESC: fix intrapage navigation vs. load issue
http://bugzilla.opendarwin.org/show_bug.cgi?id=10974

  • BrowserView/src/webkitbridge.cpp: (CWebKitBridge::LoadUrlL):
  • ResourceLoader/inc/LoaderUtils.h:
  • ResourceLoader/src/LoaderUtils.cpp: (TLoaderUtils::IsIntraPageNavigation):
8:59 AM Changeset in webkit [16638] by spadma
  • 4 edits in S60/trunk/WebKit

2006-09-29 sareen <shyam.sareen@nokia.com>

Reviewed by spadma <Sachin.Padma@nokia.com>.
DESC: Avoid passing NULL URL to webcore.
http://bugzilla.opendarwin.org/show_bug.cgi?id=11056

  • ResourceLoader/src/FileTransaction.cpp: (CFileTransaction::CreateHeaderResponseLC):

2006-09-29 zhalu <zhaosong.lu@nokia.com>

Reviewed by Sachin.
DESC:If the web page is loaded from cache, favicon is only loaded from cache,
not trying to load it from network even if cache does not have it.


http://bugzilla.opendarwin.org/show_bug.cgi?id=10355

  • BrowserView/inc/WebKitLoader.h:
  • BrowserView/src/WebKitLoader.cpp: (CWebKitLoader::CWebKitLoader): (CWebKitLoader::LoadPageL): (CWebKitLoader::LoadFavicon):
8:27 AM Changeset in webkit [16637] by rwlbuis
  • 1 edit in trunk/LayoutTests/fast/dom/css-insert-import-rule-expected.txt

I commited old expected results, this one is correct.

8:23 AM Changeset in webkit [16636] by brmorris
  • 2 edits in S60/trunk/S60Tools

Reviewed by Alan.
DESC: Add mk to list of makefile extenstions.
http://bugzilla.opendarwin.org/show_bug.cgi?id=10918

  • svn-create-patch.bat: *.mk permitted to have tabs
8:18 AM Changeset in webkit [16635] by zbujtas
  • 5 edits in S60/branches/3.1m

2006-09-25 zhalu <zhaosong.lu@nokia.com>

Reviewed by Zalan Bujtas.
DESC: For QColor object, there is a 'valid' flag indicating if the color is valid. The information for the valid flag is lost during the convertion of KWQPalette to WebCorePalette because the WebCorePalette uses only RGB values of color. This causes problem such as described in the bug report. The changes add the valid flag to the WebCorePalette class.
http://bugzilla.opendarwin.org/show_bug.cgi?id=10174

WARNING: NO TEST CASES ADDED OR CHANGED

  • bridge/WebCorePalette.h: (TWebCorePalette::TWebCorePalette): (TWebCorePalette::SetBackGroundValid): (TWebCorePalette::SetForeGroundValid): (TWebCorePalette::IsBackGroundValid): (TWebCorePalette::IsForeGroundValid):
  • kwq/KWQWidget.cpp: (QWidget::setPalette):
2:52 AM Changeset in webkit [22268] by hyatt
  • 3 edits in branches/WindowsMerge/WebKitWin

Fix mouse event propagation from WebKit to WebCore on Win32.

2:39 AM Changeset in webkit [22267] by aroben
  • 6 edits
    1 add in branches/WindowsMerge/WebCore

Reviewed by hyatt.

Add error logging to plugin code.

  • WebCore.vcproj/WebCore.vcproj: Added PluginDebug.h
  • platform/win/NotImplemented.h: Add surrounding #ifndef/#endif
  • plugins/win/PluginDebug.h: Added.
  • plugins/win/PluginPackageWin.cpp: Add error logging (WebCore::PluginPackageWin::load):
  • plugins/win/PluginStreamWin.cpp: Ditto. (WebCore::PluginStreamWin::startStream): (WebCore::PluginStreamWin::destroyStream): (WebCore::PluginStreamWin::deliverData): (WebCore::PluginStreamWin::receivedAllData):
  • plugins/win/PluginViewWin.cpp: Ditto. (WebCore::PluginViewWin::start): (WebCore::PluginViewWin::stop): (WebCore::PluginViewWin::newStream): (WebCore::PluginViewWin::write): (WebCore::PluginViewWin::destroyStream):
1:40 AM Changeset in webkit [16634] by hyatt
  • 1 edit in trunk/WebCore/platform/PlatformMouseEvent.h

Add the ability to set the click count after the fact on a mouse event.

1:39 AM Changeset in webkit [22266] by hyatt
  • 2 edits
    1 add in branches/WindowsMerge/WebCore

Fork mouse event.cpp

1:30 AM Changeset in webkit [16633] by rwlbuis
  • 3 edits
    3 adds in trunk

Reviewed by Maciej.

http://bugzilla.opendarwin.org/show_bug.cgi?id=10893
InsertRule can not handle @import statements

Allow @import as part of a css rule.

Sep 28, 2006:

11:30 PM Changeset in webkit [22265] by hyatt
  • 1 edit in branches/WindowsMerge/WebCore/ChangeLog

Make setValue work on win32 scrollbars. Fix buggy math.

11:16 PM Changeset in webkit [22264] by hyatt
  • 2 edits in branches/WindowsMerge/WebCore

Make scrollbars on win32 give proper hover feedback.

7:10 PM Changeset in webkit [22263] by sfalken
  • 10 edits
    5 adds in branches/WindowsMerge

2006-09-28 Steve Falkenburg <sfalken@apple.com>

Reviewed by Maciej.

Use $(ConfigSuffix) set via vsprops files to add _debug
to end of debug filenames.

7:07 PM Changeset in webkit [16632] by sfalken
  • 4 edits
    2 adds in trunk/JavaScriptCore

2006-09-28 Steve Falkenburg <sfalken@apple.com>

Reviewed by Maciej.


Use $(ConfigSuffix) set via vsprops files to add _debug
to end of debug filenames.

5:56 PM Changeset in webkit [22262] by aroben
  • 2 edits in branches/WindowsMerge/WebCore/platform/win

Forgot these in the last checkin (d'oh!)

5:55 PM Changeset in webkit [22261] by aroben
  • 1 edit in branches/WindowsMerge/WebCore/ChangeLog

Reviewed by Hyatt.

Avoid creating LOGFONTs for bitmap fonts under CG, since we don't
support them anyway.

  • platform/win/FontCacheWin.cpp: (WebCore::FontCache::createFontPlatformData): Set lfOutPrecision to OUT_TT_ONLY_PRECIS to avoid bitmap fonts.
  • platform/win/FontPlatformDataWin.cpp: (WebCore::FontPlatformData::FontPlatformData): Change the LOG_ERROR back to an ASSERT since we should never reach this condition now.
5:39 PM Changeset in webkit [16631] by hyatt
  • 3 edits in trunk/WebCore

Switch over the Mac-specific uses of viewportToContents/contentsToViewport.

  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::eventMayStartDrag): (WebCore::FrameMac::dragHysteresisExceeded): (WebCore::FrameMac::mouseDown): (WebCore::FrameMac::shouldDragAutoNode): (WebCore::FrameMac::sendContextMenuEvent):
  • bridge/mac/WebCoreFrameBridge.mm: (-[WebCoreFrameBridge _visiblePositionForPoint:]):
5:21 PM Changeset in webkit [22260] by hyatt
  • 3 edits in branches/WindowsMerge/WebCore

Eliminate viewportToContents and contentsToViewport

5:17 PM Changeset in webkit [16630] by hyatt
  • 11 edits in trunk/WebCore

Eliminate viewportToContents and contentsToViewport

4:32 PM Changeset in webkit [16629] by aliceli1
  • 2 edits in trunk/WebCore

Build fix by adding missing files for CommandByName

  • WebCore.xcodeproj/project.pbxproj:
3:47 PM Changeset in webkit [16628] by aliceli1
  • 2 edits in trunk/WebCore

Adding CommandByName files to the vcproj

  • WebCore.vcproj/WebCore/WebCore.vcproj:
3:23 PM Changeset in webkit [16627] by aliceli1
  • 2 edits in trunk/WebKit

fixing the windows build

  • COM/WebView.cpp: (WebView::mouseMoved): (WebView::mouseDown): (WebView::mouseUp): (WebView::mouseDoubleClick):
3:06 PM Changeset in webkit [22259] by aroben
  • 2 edits in branches/WindowsMerge/WebCore

Reviewed by Anders.

Fail without crashing and log an error when CG encounters a bitmap
font.

  • platform/win/FontPlatformDataWin.cpp: (WebCore::FontPlatformData::FontPlatformData): Trade LOG_ERROR for ASSERT_WITH_MESSAGE
2:41 PM Changeset in webkit [22258] by aliceli1
  • 7 edits
    1 add in branches/WindowsMerge
2:39 PM Changeset in webkit [16626] by aliceli1
  • 8 edits
    2 adds in trunk/WebCore
2:38 PM Changeset in webkit [16625] by brmorris
  • 2 edits in S60/trunk/WebCore

2006-09-28 brmorris <bradley.morrison@nokia.com>

Reviewed by Zalan.
DESC: Include color attribute value in stringified info
http://bugzilla.opendarwin.org/show_bug.cgi?id=EBUGZILLADOWN

  • khtml/rendering/render_object.cpp: (RenderObject::information): Now includes Style()->color
2:33 PM Changeset in webkit [16624] by aroben
  • 5 edits
    2 deletes in trunk/WebCore

Reviewed by Adele.

Remove no-longer-used PopUpButton.

  • WebCore.vcproj/WebCore/WebCore.vcproj: Remove PopUpButton.h
  • WebCore.xcodeproj/project.pbxproj: Remove PopUpButton.h, PopUpButtonMac.mm
  • platform/PopUpButton.h: Removed.
  • platform/mac/PopUpButtonMac.mm: Removed.
  • platform/win/TemporaryLinkStubs.cpp: Remove PopUpButton:: methods
  • rendering/DeprecatedRenderSelect.cpp: Change #include of PopUpButton.h to ListBox.h
2:31 PM Changeset in webkit [22257] by aroben
  • 3 edits in branches/WindowsMerge/WebCore

Reviewed by Adele.

Remove never-to-be-used PopUpButton

  • WebCore.vcproj/WebCore.vcproj: Remove PopUpButton.h
  • platform/win/TemporaryLinkStubs.cpp: Remove PopUpButton:: methods
2:28 PM Changeset in webkit [16623] by harrison
  • 2 edits in trunk/WebKitTools

Suggested by Darin.


Moved an extern declaration from inside a method to the top of the file.

  • DumpRenderTree/EventSendingController.m: (-[EventSendingController clearKillRing]):
2:07 PM Changeset in webkit [16622] by darin
  • 11 edits in trunk

JavaScriptCore:

Reviewed by Alice.

  • support for change that should fix <rdar://problem/4733044> REGRESSION: XML iBench shows 10% perf. regression (copying strings while decoding)
  • wtf/Vector.h: Changed VectorBuffer so that the general case contains an instance of the 0 case, since deriving from it was violating the Liskov Substitution Principle. (WTF::VectorBuffer::releaseBuffer): Added. Releases the buffer so it can be adopted by another data structure that uses the FastMalloc.h allocator. Returns 0 if the internal buffer was being used. (WTF::Vector::releaseBuffer): Added. Releases the buffer as above or creates a new one in the case where the internal buffer was being used.

WebCore:

Reviewed by Alice.

  • change that should fix <rdar://problem/4733044> REGRESSION: XML iBench shows 10% perf. regression (copying strings while decoding)

Use Vector<UChar> instead of String when building up the decoded string in
the ICU and Mac decoders. Using String leads to O(n2) behavior because
String grows the buffer every single time that append is called. Using
Vector::append instead of String::append also avoids constructing a string
each time just to append and a questionable copy that is done inside the
String::append function which also contributed to the slowness.

  • platform/PlatformString.h:
  • platform/String.cpp: (WebCore::String::adopt): Added. Makes a String from a Vector<UChar>, adopting the buffer from the vector to avoid copying and memory allocation.
  • platform/StringImpl.h:
  • platform/StringImpl.cpp: (WebCore::StringImpl::adopt): Ditto.
  • platform/StreamingTextDecoder.h:
  • platform/StreamingTextDecoder.cpp: (WebCore::TextCodec::appendOmittingBOM): Change to use a Vector<UChar> instead of a String, since vectors have better resizing performance (they store a separate capacity).
  • platform/StreamingTextDecoderICU.cpp: (WebCore::TextCodecICU::decode):
  • platform/mac/StreamingTextDecoderMac.cpp: (WebCore::TextCodecMac::decode): Change to use Vector<UChar> instead of String and then create a string at the end of the process using the new adopt function.
2:05 PM Changeset in webkit [16621] by thatcher
  • 4 edits in trunk

Explicitly specify stabs for Debug and Production.

1:25 PM Changeset in webkit [16620] by harrison
  • 4 edits in trunk

LayoutTests:

Reviewed by Justin.

Call clearKillRing() so we are guaranteed to start with empty kill ring.

  • editing/pasteboard/emacs-cntl-y-001.html:

WebKitTools:

Reviewed by Justin.


Add clearKillRing so we can test emacs support with empty kill ring.

  • DumpRenderTree/EventSendingController.m: (+[EventSendingController isSelectorExcludedFromWebScript:]): (+[EventSendingController webScriptNameForSelector:]): (-[EventSendingController clearKillRing]):
1:12 PM Changeset in webkit [16619] by brmorris
  • 1 edit
    3 adds in S60/trunk/LayoutTests

2006-09-28 brmorris <bradley.morrison@nokia.com>

Reviewed by myself :)
DESC: Added tests for tabbed navigation via:

<meta name="navigation" content="tabbed"/>

http://bugzilla.opendarwin.org/show_bug.cgi?id=10222

  • manual/10222_tabbed_navigation-mobile-expected.txt: Added.
  • manual/10222_tabbed_navigation.html: Added.
1:07 PM Changeset in webkit [22256] by aroben
  • 5 edits
    1 add in branches/WindowsMerge/WebCore

Reviewed by Alice.

Replace notImplemented() with LOG_NOIMPL() and STOP_NOIMPL() macros
which generate debug output. STOP_NOIMPL() also generates a debug
break. Replace calls to notImplemented() with
LOG_NOIMPL()/STOP_NOIMPL() as appropriate. This should stop some unnecessary
crashes where notImplemented() was called from a method that can fail
safely.

  • WebCore.vcproj/WebCore.vcproj: Added NotImplemented.h
  • platform/win/FontWin.cpp: Use LOG_NOIMPL() instead of notImplemented(). (WebCore::Font::selectionRectForComplexText): (WebCore::Font::drawComplexText): (WebCore::Font::floatWidthForComplexText): (WebCore::Font::offsetForPositionForComplexText):
  • platform/win/NotImplemented.h: Added. Contains new LOG_NOIMPL() and STOP_NOIMPL() macros.
  • platform/win/TemporaryLinkStubs.cpp: Use LOG_NOIMPL() and STOP_NOIMPL() instead of notImplemented() or nothing. (PopUpButton::focusPolicy): (PopUpButton::populate): (Widget::enableFlushDrawing): (Widget::focusPolicy): (Widget::disableFlushDrawing): (Widget::lockDrawingFocus): (Widget::unlockDrawingFocus): (JavaAppletWidget::JavaAppletWidget): (TextField::selectAll): (TextField::addSearchResult): (TextField::selectionStart): (TextField::hasSelectedText): (TextField::selectedText): (TextField::setAutoSaveName): (TextField::checksDescendantsForFocus): (TextField::setSelection): (TextField::setMaxResults): (TextField::edited): (Slider::Slider): (Slider::sizeHint): (Slider::setValue): (Slider::setMaxValue): (Slider::setMinValue): (Slider::~Slider): (Slider::setFont): (Slider::value): (ListBox::sizeForNumberOfLines): (ListBox::isSelected): (ListBox::checksDescendantsForFocus): (Slider::focusPolicy): (ListBox::focusPolicy): (TextField::focusPolicy): (Cursor::Cursor): (PlatformMouseEvent::PlatformMouseEvent): (WebCore::searchableIndexIntroduction): (WebCore::findNextSentenceFromIndex): (WebCore::findSentenceBoundary): (WebCore::findNextWordFromIndex): (WebCore::ServeSynchronousRequest): (FrameWin::focusWindow): (FrameWin::unfocusWindow): (FrameWin::locationbarVisible): (FrameWin::registerCommandForRedo): (FrameWin::getObjectInstanceForWidget): (FrameWin::getEmbedInstanceForWidget): (FrameWin::getAppletInstanceForWidget): (FrameWin::canRedo): (FrameWin::canUndo): (FrameWin::canPaste): (FrameWin::canGoBackOrForward): (FrameWin::shouldInterruptJavaScript): (FrameWin::openURL): (FrameWin::print): (FrameWin::passMouseDownEventToWidget): (FrameWin::issueCutCommand): (FrameWin::issueCopyCommand): (FrameWin::issuePasteCommand): (FrameWin::issueUndoCommand): (FrameWin::issueRedoCommand): (FrameWin::issuePasteAndMatchStyleCommand): (FrameWin::issueTransposeCommand): (FrameWin::passWheelEventToChildWidget): (FrameWin::mimeTypeForFileName): (FrameWin::scheduleClose): (FrameWin::markMisspellings): (FrameWin::menubarVisible): (FrameWin::personalbarVisible): (FrameWin::statusbarVisible): (FrameWin::toolbarVisible): (FrameWin::originalRequestURL): (FrameWin::isLoadTypeReload): (BrowserExtensionWin::canRunModal): (BrowserExtensionWin::createNewWindow): (BrowserExtensionWin::canRunModalNow): (BrowserExtensionWin::runModal): (BrowserExtensionWin::goBackOrForward): (BrowserExtensionWin::historyURL): (GraphicsContext::addRoundedRectClip): (GraphicsContext::addInnerRoundedRectClip): (WebCore::screenDepthPerComponent): (WebCore::screenIsMonochrome): (WebCore::moveCursor): (WebCore::inputElementAltText): (WebCore::findWordBoundary): (WebCore::ResourceLoader::assembleResponseHeaders): (WebCore::ResourceLoader::retrieveCharset): (FrameView::updateBorder): (FrameWin::restoreDocumentState): (FrameWin::partClearedInBegin): (FrameWin::createEmptyDocument): (FrameWin::overrideMediaType): (FrameWin::markedTextRange): (FrameWin::addMessageToConsole): (FrameWin::shouldChangeSelection): (FrameWin::respondToChangedSelection): (FrameWin::saveDocumentState): (FrameWin::registerCommandForUndo): (FrameWin::clearUndoRedoOperations): (FrameWin::incomingReferrer): (FrameWin::markMisspellingsInAdjacentWords): (FrameWin::respondToChangedContents): (BrowserExtensionWin::BrowserExtensionWin): (BrowserExtensionWin::setTypedIconURL): (BrowserExtensionWin::setIconURL): (BrowserExtensionWin::getHistoryLength): (WebCore::CheckIfReloading): (WebCore::CheckCacheObjectStatus): (Widget::setIsSelected): (ScrollView::inWindow): (ScrollView::scrollPointRecursively): (GraphicsContext::setShadow): (GraphicsContext::clearShadow): (GraphicsContext::beginTransparencyLayer): (GraphicsContext::endTransparencyLayer): (GraphicsContext::clearRect): (GraphicsContext::strokeRect): (GraphicsContext::setLineWidth): (GraphicsContext::setLineCap): (GraphicsContext::setLineJoin): (GraphicsContext::setMiterLimit): (GraphicsContext::setAlpha): (GraphicsContext::clip): (GraphicsContext::rotate): (GraphicsContext::scale): (Path::Path): (Path::~Path): (Path::contains): (Path::translate): (Path::boundingRect): (Path::operator=): (Path::clear): (Path::moveTo): (Path::addLineTo): (Path::addQuadCurveTo): (Path::addBezierCurveTo): (Path::addArcTo): (Path::closeSubpath): (Path::addArc): (Path::addRect): (Path::addEllipse): (GraphicsContext::drawFocusRing): (GraphicsContext::drawLineForMisspelling): (GraphicsContext::setCompositeOperation): (TextField::TextField): (TextField::~TextField): (TextField::setFont): (TextField::setAlignment): (TextField::setWritingDirection): (TextField::maxLength): (TextField::setMaxLength): (TextField::text): (TextField::setText): (TextField::cursorPosition): (TextField::setCursorPosition): (TextField::setEdited): (TextField::setReadOnly): (TextField::setPlaceholderString): (TextField::setColors): (TextField::sizeForCharacterWidth): (TextField::baselinePosition): (TextField::setLiveSearch): (PopUpButton::PopUpButton): (PopUpButton::~PopUpButton): (PopUpButton::setFont): (PopUpButton::baselinePosition): (PopUpButton::setWritingDirection): (PopUpButton::clear): (PopUpButton::appendItem): (PopUpButton::setCurrentItem): (PopUpButton::sizeHint): (PopUpButton::frameGeometry): (PopUpButton::setFrameGeometry): (ListBox::ListBox): (ListBox::~ListBox): (ListBox::setSelectionMode): (ListBox::setFont): (ListBox::setWritingDirection): (ListBox::clear): (ListBox::doneAppendingItems): (ListBox::setEnabled): (ListBox::appendItem): (ListBox::setSelected): (WebCore::focusRingColor): (WebCore::setFocusRingColorChangeFunction): (Frame::setNeedsReapplyStyles): (Image::drawTiled): (RenderPopupMenuWin::addSeparator): (RenderPopupMenuWin::addGroupLabel): (IconLoader::receivedResponse): (IconLoader::notifyIconChanged):
  • plugins/win/PluginViewWin.cpp: (WebCore::PluginViewWin::setValue):
12:39 PM Changeset in webkit [16618] by brmorris
  • 1 copy in S60/branches/s60-RnD-06wk39

New branch.

11:44 AM Changeset in webkit [16617] by brmorris
  • 2 edits in S60/trunk/S60Tools

2006-09-28 brmorris <bradley.morrison@nokia.com>

Reviewed by Alan.
DESC: parse the description from the commit message
http://bugzilla.opendarwin.org/show_bug.cgi?id=11050

  • svn2ccm.pl: Smarter task synopsis.
11:33 AM Changeset in webkit [16616] by weinig
  • 31 edits in trunk/WebCore

Reviewed by Tim H.

Patch for http://bugzilla.opendarwin.org/show_bug.cgi?id=11057
Auto-generate more SVG Objective-C bindings

  • Auto-generates DOMSVGAnimatedAngle, DOMSVGAnimatedBoolean, DOMSVGAnimatedEnumeration, DOMSVGAnimatedInteger, DOMSVGAnimatedLength, DOMSVGAnimatedLengthList, DOMSVGAnimatedNumber, DOMSVGAnimatedString, DOMSVGAnimatedTransformList, DOMSVGExternalResourcesRequired, DOMSVGLangSpace, DOMSVGLocatable, DOMSVGMetadataElement, DOMSVGRectElement, DOMSVGStringList, DOMSVGStylable, DOMSVGStyleElement, DOMSVGTests, and DOMSVGTransformable.
  • Adds ability to CodeGeneratorObjC.pm to handle interfaces with mulitple parent interfaces and the SVG animated classes.
  • Corrects CodeGenerator.pm to only skip the first parent for the main interface.
  • DerivedSources.make:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/objc/DOMInternal.h:
  • bindings/objc/DOMSVG.h:
  • bindings/scripts/CodeGenerator.pm:
  • bindings/scripts/CodeGeneratorObjC.pm:
  • ksvg2/svg/SVGAnimatedAngle.idl:
  • ksvg2/svg/SVGAnimatedBoolean.idl:
  • ksvg2/svg/SVGAnimatedEnumeration.idl:
  • ksvg2/svg/SVGAnimatedInteger.idl:
  • ksvg2/svg/SVGAnimatedLength.idl:
  • ksvg2/svg/SVGAnimatedLengthList.idl:
  • ksvg2/svg/SVGAnimatedNumber.idl:
  • ksvg2/svg/SVGAnimatedPathData.idl:
  • ksvg2/svg/SVGAnimatedString.idl:
  • ksvg2/svg/SVGAnimatedTransformList.idl:
  • ksvg2/svg/SVGExternalResourcesRequired.idl:
  • ksvg2/svg/SVGFilterPrimitiveStandardAttributes.idl:
  • ksvg2/svg/SVGFitToViewBox.idl:
  • ksvg2/svg/SVGLangSpace.idl:
  • ksvg2/svg/SVGLocatable.idl:
  • ksvg2/svg/SVGMetadataElement.idl:
  • ksvg2/svg/SVGStringList.idl:
  • ksvg2/svg/SVGStylable.idl:
  • ksvg2/svg/SVGStyleElement.idl:
  • ksvg2/svg/SVGTests.idl:
  • ksvg2/svg/SVGTransformable.idl:
  • ksvg2/svg/SVGURIReference.idl:
  • ksvg2/svg/SVGUnitTypes.idl:
  • ksvg2/svg/SVGZoomAndPan.idl:
11:24 AM Changeset in webkit [16615] by beidson
  • 2 edits in trunk/WebCore

Reviewed by Mitz Pettel!!!

Change the error code check for common functions from SQLResultOk to SQLResultDone

  • loader/icon/IconDatabase.cpp: (WebCore::IconDatabase::forgetPageURLQuery): (WebCore::IconDatabase::setIconIDForPageURLQuery): (WebCore::IconDatabase::addIconForIconURLQuery):
9:56 AM Changeset in webkit [22255] by aroben
  • 2 edits in branches/WindowsMerge/WebCore

Rubberstamped by sfalken.

Remove calls to DebugBreak() from plugin code.

  • plugins/win/PluginViewWin.cpp: (WebCore::PluginViewWin::getValue): (WebCore::PluginViewWin::setValue):
2:13 AM Changeset in webkit [16614] by mjs
  • 2 edits in trunk/JavaScriptCore

Reviewed by Geoff.


  • change garbage collection to happen at increments proportional to number of live objects, not always every 1000 allocations
  • kjs/collector.cpp: (KJS::Collector::allocate):
2:11 AM Changeset in webkit [16613] by mjs
  • 3 edits in trunk/JavaScriptCore

Reviewed by Mitz.

  • fixed REGRESSION (r16606): javascriptCore Crash on website load


Plus style fixes.


  • fixed some possible off-by-one bugs
  • use indexing, not iterators, for Vectors
  • store Vector by pointer instead of by value to avoid blowing out FunctionImp size


  • kjs/function.cpp: (KJS::FunctionImp::addParameter): (KJS::FunctionImp::parameterString): (KJS::FunctionImp::processParameters): (KJS::FunctionImp::lengthGetter): (KJS::FunctionImp::getParameterName):
  • kjs/function.h:
1:53 AM Changeset in webkit [22254] by hyatt
  • 4 edits in branches/WindowsMerge/WebCore

Dump the use of MapWindowPoints on Win32 and just let the outermost HWND be the viewport point of reference for events and such.

1:49 AM Changeset in webkit [16612] by hyatt
  • 3 edits in trunk/WebCore

Dump the use of MapWindowPoints on Win32 and just let the outermost HWND be the viewport point of reference for events and such.

Note: See TracTimeline for information about the timeline view.