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

Timeline



May 8, 2003:

10:41 PM Changeset in webkit [4328] by rjw
  • 2 edits in trunk/WebKit

Fixed 3146161. Use the AppKit to render complex
text in the simple string drawing method.

Reviewed by John.

  • Misc.subproj/WebKitNSStringExtras.m:
10:32 PM Changeset in webkit [4327] by rjw
  • 6 edits in trunk/WebKit

Make representations without intrinsic titles return nil for

  • (NSString *)title

Reviewed by John.

  • Plugins.subproj/WebNetscapePluginRepresentation.m:
  • WebView.subproj/WebDataSourcePrivate.h:
  • WebView.subproj/WebDataSourcePrivate.m:
  • WebView.subproj/WebImageRepresentation.m:
  • WebView.subproj/WebTextRepresentation.m:
6:54 PM Changeset in webkit [4326] by rjw
  • 7 edits in trunk/WebKit

Fixed 3252460. * Public API Change *
Added title method to WebDocumentRepresentation.

Fixed 3250352. Check that delegate implements method.

Reviewed by hyatt.

  • WebView.subproj/WebDataSource.m: (-[WebDataSource pageTitle]):
  • WebView.subproj/WebDataSourcePrivate.h:
  • WebView.subproj/WebDataSourcePrivate.m: (-[WebDataSource _startLoading:]): (-[WebDataSource _title]):
  • WebView.subproj/WebDocument.h:
  • WebView.subproj/WebHTMLRepresentation.m: (-[WebHTMLRepresentation title]):
  • WebView.subproj/WebImageRepresentation.m: (-[WebImageRepresentation title]):
  • WebView.subproj/WebNetscapePluginRepresentation.m: (-[WebNetscapePluginRepresentation title]):
  • WebView.subproj/WebTextRepresentation.h:
  • WebView.subproj/WebTextRepresentation.m: (-[WebTextRepresentation dealloc]): (-[WebTextRepresentation setDataSource:]): (-[WebTextRepresentation title]):
6:16 PM Changeset in webkit [4325] by hyatt
  • 9 edits in trunk/WebCore

Fix the definition of isRoot() so that it doesn't depend on
the object being in the render tree yet (and to avoid any
virtual function calls as well).

Make sure that requiresLayer() returns true if the object is
the root. This way the layer isn't destroyed when you e.g.,
hit Cmd+ to increase font size.

This fixes bug 3252496.

Reviewed by rjw

  • khtml/rendering/render_block.cpp:
  • khtml/rendering/render_box.cpp: (RenderBox::setStyle):
  • khtml/rendering/render_object.cpp: (RenderObject::isRoot):
  • khtml/rendering/render_object.h:
  • khtml/xml/dom_elementimpl.cpp: (ElementImpl::createRenderer):
6:13 PM Changeset in webkit [4324] by kocienda
  • 2 edits in trunk/WebKit/Misc.subproj

2003-05-08 Ken Kocienda <kocienda@apple.com>

WebBrowser:

Reviewed by Maciej

A number of file and class name changes in WebFoundation, as done
by running this tops script.

replace "WebAboutProtocol" with "NSAboutURLProtocol"
replace "WebCFNetworkHTTPProtocol" with "NSCFNetworkHTTPURLProtocol"
replace "WebFileProtocol" with "NSFileURLProtocol"
replace "WebHTTPProtocol" with "NSHTTPURLProtocol"
replace "WebSimpleHTTPProtocol" with "NSSimpleHTTPURLProtocol"
replace "WebSSLProxyWrapperStream" with "NSSSLProxyWrapperStream"

There were also some changes I did by hand in DebugUtilities.m, to
bring up to date some flag names used to figure out what protocols are
registered.

  • Debug/DebugUtilities.m: (-[DebugUtilities supportedProtocolHandlersSubmenuItem]): (-[BrowserDocument setCertificatePolicy:]): (-[BrowserDocument validate_setCertificatePolicy:]): (initProtocolHandlers): (-[BrowserDocument unregisterAllProtocolHandlers]): (-[BrowserDocument reregisterAllProtocolHandlers]): (-[BrowserDocument toggleAboutURLProtocolHandler:]): (-[BrowserDocument validate_toggleAboutURLProtocolHandler:]): (-[BrowserDocument toggleFileURLProtocolHandler:]): (-[BrowserDocument validate_toggleFileURLProtocolHandler:]): (-[BrowserDocument toggleCFNetworkHTTPURLProtocolHandler:]): (-[BrowserDocument validate_toggleCFNetworkHTTPURLProtocolHandler:]): (-[BrowserDocument toggleSimpleHTTPURLProtocolHandler:]): (-[BrowserDocument validate_toggleSimpleHTTPURLProtocolHandler:]):

WebFoundation:

A number of file and class name changes in WebFoundation, as done
by running this tops script.

/* Cookies */
replace "WebCookieStorage" with "NSHTTPCookieDOStorage"
replace "WebCookieStorageServer" with "NSHTTPCookieServer"

/* CacheLoader */
replace "WebNetworkSettings" with "NSNetworkSettings"
replace "WebNetworkSettingsPrivate" "NSNetworkSettingsPrivate"
replace "_settingsPrivate" "_internal"
replace "WebSynchronousDelegate" with "NSSynchronousURLConnectionDelegate"

/* Database */
replace "WebLRUFileList" with "NSLRUFileList"

/* Protocols */
replace "WebAboutProtocol" with "NSAboutURLProtocol"
replace "WebCFNetworkHTTPProtocol" with "NSCFNetworkHTTPURLProtocol"
replace "WebFileProtocol" with "NSFileURLProtocol"
replace "WebHTTPProtocol" with "NSHTTPURLProtocol"
replace "WebSimpleHTTPProtocol" with "NSSimpleHTTPURLProtocol"
replace "WebSSLProxyWrapperStream" with "NSSSLProxyWrapperStream"

Also, the NSLRUFileList files have many C functions whose names
changed from Web to NS prefixes.

  • AuthenticationManager.subproj/NSHTTPAuthenticator.m: (-[NSHTTPAuthenticator _proxyProtectionSpaceForURL:]):
  • AuthenticationManager.subproj/NSURLCredentialStorage.m:
  • CacheLoader.subproj/NSNetworkSettings.h:
  • CacheLoader.subproj/NSNetworkSettings.m: (-[NSNetworkSettings _updateProxySettings]): (proxySettingsChanged): (-[NSNetworkSettings _listenForProxySettingChanges]): (-[NSNetworkSettings _URLMatchesProxyExceptions:]): (-[NSNetworkSettings _init]): (-[NSNetworkSettings init]): (-[NSNetworkSettings dealloc]): (initSharedNetworkSettings): (+[NSNetworkSettings sharedNetworkSettings]): (-[NSNetworkSettings proxyPropertiesForURL:]): (-[NSNetworkSettings connectedToInternet:]):
  • CacheLoader.subproj/NSSynchronousURLConnectionDelegate.h:
  • CacheLoader.subproj/NSSynchronousURLConnectionDelegate.m:
  • CacheLoader.subproj/NSURLCache.m: (-[NSURLCacheInternal dealloc]): (-[NSURLCache _diskCacheSync:]): (-[NSURLCache _diskCacheCreateLRUList:]): (-[NSURLCache _diskCacheGet:]): (-[NSURLCache _diskCacheTruncate:]): (-[NSURLCache _diskCacheExecuteWrite:]): (-[NSURLCache _diskCacheExecuteRemoval:]):
  • CacheLoader.subproj/NSURLConnection.m: (+[NSURLConnection _forceSingleLoad]): (_loadThrottler):
  • CacheLoader.subproj/NSURLProtocol.m: (NSURLProtocolInit):
  • CacheLoader.subproj/WebNetworkSettings.h: Removed.
  • CacheLoader.subproj/WebNetworkSettings.m: Removed.
  • CacheLoader.subproj/WebSynchronousDelegate.h: Removed.
  • CacheLoader.subproj/WebSynchronousDelegate.m: Removed.
  • CookieManager.subproj/NSHTTPCookieDOStorage.h:
  • CookieManager.subproj/NSHTTPCookieDOStorage.m: (-[NSHTTPCookieDOStorage _updateCookiesFromServer]): (-[NSHTTPCookieDOStorage _establishConnection]): (-[NSHTTPCookieDOStorage init]):
  • CookieManager.subproj/NSHTTPCookieServer.h:
  • CookieManager.subproj/NSHTTPCookieServer.m: (checkForCookieExpiration): (-[NSHTTPCookieServer init]):
  • CookieManager.subproj/NSHTTPCookieStorage.m: (-[NSHTTPCookieStorage _connectToCookieStorage]):
  • CookieManager.subproj/WebCookieStorage.h: Removed.
  • CookieManager.subproj/WebCookieStorage.m: Removed.
  • CookieManager.subproj/WebCookieStorageServer.h: Removed.
  • CookieManager.subproj/WebCookieStorageServer.m: Removed.
  • Database.subproj/NSLRUFileList.h:
  • Database.subproj/NSLRUFileList.m: (NSLRUFileListCreate): (NSLRUFileListRelease): (NSLRUFileListRebuildFileDataUsingRootDirectory): (NSLRUFileListRemoveFileWithPath): (NSLRUFileListTouchFileWithPath): (NSLRUFileListSetFileData): (NSLRUFileListGetPathOfOldestFile): (NSLRUFileListRemoveOldestFileFromList): (NSLRUFileListContainsItem): (NSLRUFileListGetFileSize): (NSLRUFileListCountItems): (NSLRUFileListGetTotalSize): (NSLRUFileListRemoveAllFilesFromList): (compareTimes): (NSLRUFileDataEqual): (NSLRUFileListGetOldestFileData): (NSLRUFileDataReleaseApplierFunction): (NSLRUFileDataRelease): (NSLRUFileDataBinaryHeapDumpApplierFunction): (NSLRUFileDataDictDumpApplierFunction): (NSLRUFileListDescription):
  • Database.subproj/WebFileDatabase.h:
  • Database.subproj/WebFileDatabase.m: (-[WebFileDatabase _createLRUList:]): (-[WebFileDatabase _truncateToSizeLimit:]): (-[WebFileDatabase objectForKey:]): (-[WebFileDatabase performSetObject:forKey:]): (-[WebFileDatabase performRemoveObjectForKey:]): (-[WebFileDatabase close]): (-[WebFileDatabase lazySync:]): (-[WebFileDatabase sync]): (-[WebFileDatabase count]): (-[WebFileDatabase usage]):
  • Database.subproj/WebLRUFileList.h: Removed.
  • Database.subproj/WebLRUFileList.m: Removed.
  • English.lproj/StringsNotToBeLocalized.txt:
  • ProtocolHandlers.subproj/NSAboutURLProtocol.h:
  • ProtocolHandlers.subproj/NSAboutURLProtocol.m:
  • ProtocolHandlers.subproj/NSCFNetworkHTTPURLProtocol.h:
  • ProtocolHandlers.subproj/NSCFNetworkHTTPURLProtocol.m: (NSCFNetworkHTTPURLProtocolInit): (readStreamCallback): (+[NSCFNetworkHTTPURLProtocol _certificatePolicy]): (+[NSCFNetworkHTTPURLProtocol _setCertificatePolicy:]): (+[NSCFNetworkHTTPURLProtocol _supportsCertificatePolicy:]): (-[NSCFNetworkHTTPURLProtocol initWithRequest:cachedResponse:client:]): (-[NSCFNetworkHTTPURLProtocol continueBeginLoadInBackgroundAfterCreatingHTTPRequest]):
  • ProtocolHandlers.subproj/NSFTPURLProtocol.h:
  • ProtocolHandlers.subproj/NSFTPURLProtocol.m: (NSFTPURLProtocolInit): (readStreamCallback): (-[NSFTPURLProtocol initWithRequest:cachedResponse:client:]): (-[NSFTPURLProtocol startLoading]): (-[NSFTPURLProtocol determineErrorAndFail]):
  • ProtocolHandlers.subproj/NSFileURLProtocol.h:
  • ProtocolHandlers.subproj/NSFileURLProtocol.m:
  • ProtocolHandlers.subproj/NSHTTPURLProtocol.h:
  • ProtocolHandlers.subproj/NSHTTPURLProtocol.m: (-[NSHTTPURLProtocol startOriginLoad]): (-[NSHTTPURLProtocol stopLoading]): (-[NSHTTPURLProtocol buildHTTPRequest]): (-[NSHTTPURLProtocol determineErrorAndFail]): (-[NSHTTPURLProtocol didAddCredentials:toRequest:context:]): (-[NSHTTPURLProtocol cleanUpForRetry]):
  • ProtocolHandlers.subproj/NSSSLProxyWrapperStream.h:
  • ProtocolHandlers.subproj/NSSSLProxyWrapperStream.m: (wSocketStreamCallBack): (WebCFReadStreamOpenThroughSSLProxy): (WebCFReadStreamCancelOpenThroughSSLProxy):
  • ProtocolHandlers.subproj/NSSimpleHTTPURLProtocol.h:
  • ProtocolHandlers.subproj/NSSimpleHTTPURLProtocol.m: (ensureRequestStringCapacity): (appendBytesToRequestString): (appendIntegerToRequestString): (appendNSStringToRequestString): (NSSimpleHTTPURLProtocolInit): (_socketWriteCallback): (_socketReadCallback): (_addCookieHeader): (_hostNameResolved): (+[NSSimpleHTTPURLProtocol canInitWithRequest:]): (-[NSSimpleHTTPURLProtocol initWithRequest:cachedResponse:client:]):
  • ProtocolHandlers.subproj/WebAboutProtocol.h: Removed.
  • ProtocolHandlers.subproj/WebAboutProtocol.m: Removed.
  • ProtocolHandlers.subproj/WebCFNetworkHTTPProtocol.h: Removed.
  • ProtocolHandlers.subproj/WebCFNetworkHTTPProtocol.m: Removed.
  • ProtocolHandlers.subproj/WebFTPProtocol.h: Removed.
  • ProtocolHandlers.subproj/WebFTPProtocol.m: Removed.
  • ProtocolHandlers.subproj/WebFileProtocol.h: Removed.
  • ProtocolHandlers.subproj/WebFileProtocol.m: Removed.
  • ProtocolHandlers.subproj/WebHTTPProtocol.h: Removed.
  • ProtocolHandlers.subproj/WebHTTPProtocol.m: Removed.
  • ProtocolHandlers.subproj/WebSSLProxyWrapperStream.h: Removed.
  • ProtocolHandlers.subproj/WebSSLProxyWrapperStream.m: Removed.
  • ProtocolHandlers.subproj/WebSimpleHTTPProtocol.h: Removed.
  • ProtocolHandlers.subproj/WebSimpleHTTPProtocol.m: Removed.
  • WebFoundation-base.exp:
  • WebFoundation.pbproj/project.pbxproj:
5:15 PM Changeset in webkit [4323] by hyatt
  • 3 edits in trunk/WebCore

Background-position-x and y should be length|percentage. Number
is not a valid option. Need to remove this so that the quirk will
kick in that translates numbers to pixels. Fixes an issue on
diveintomark's list.

Reviewed by darin

  • khtml/css/cssparser.cpp: (CSSParser::parseValue):
5:04 PM Changeset in webkit [4322] by darin
  • 3 edits in trunk/WebCore

Reviewed by Dave.

  • fixed part of the cause of 3251979 -- switching tabs hoses page layout
  • kwq/WebCoreSettings.mm: (-[WebCoreSettings _updateAllViews]): Only update views that are using this settings object. The old way would result in re-laying out all views every time you created a new one.
4:43 PM Changeset in webkit [4321]
  • 3 copies
    6 deletes in tags/WebKit-78~1

This commit was manufactured by cvs2svn to create tag 'WebKit-78~1'.

4:43 PM Changeset in webkit [4320] by darin
  • 2 edits in branches/Safari-78~1-branch/WebKit

Rolled in change from main trunk.

2003-05-08 Darin Adler <Darin Adler>

  • WebKit.pbproj/project.pbxproj: Fixed paths for non-Panther builds.
4:33 PM Changeset in webkit [4319] by darin
  • 2 edits in trunk/WebKit
  • WebKit.pbproj/project.pbxproj: Fixed paths for non-Panther builds.
3:51 PM Changeset in webkit [4318] by darin
  • 4 edits in trunk/WebCore

Reviewed by Dave.

  • fixed regression caused by my last check-in; need to use the KHTMLView layout function
  • kwq/KWQWidget.mm: (QWidget::setFocus): Use KHTMLView's layout instead of RenderCanvas's.
  • fixed 3251913 -- when form field has focus, dragging across text elsewhere does not select it
  • khtml/xml/dom_docimpl.cpp: (DocumentImpl::setSelection): Remove setFocusNode(0) call. Apparently not needed these days. I tested and everything works fine without it, and its side effect was the bug above. (DocumentImpl::setFocusNode): Remove clearSelection() call for similar reasons.
3:23 PM Changeset in webkit [4317] by darin
  • 6 edits in trunk/WebCore

Reviewed by Maciej.

  • fixed 3251985 -- REGRESSION: tabbing doesn't scroll text fields into view any more
  • kwq/KWQTextField.h: Add a _hasFocus field, which prevents us from focusing multiple times.
  • kwq/KWQTextField.mm: (+[KWQTextField initialize]): Set KWQTextFieldCell as the cell class. We need a cell subclass to detect when editing begins. (-[KWQTextField setPasswordMode:]): Don't pass a QLineEdit to the secure text field when creating it. It's no longer needed. (-[KWQTextField controlTextDidEndEditing:]): Call setHasFocus:NO, and do the actual work in there. (-[KWQTextField control:textShouldBeginEditing:]): Remove the work that was done in here. This is only called when you actually start typing, so it's too late. (-[KWQTextField becomeFirstResponder]): Keep the code to forward first responder to the password field, but get rid of the rest, because this code path does not cover all the ways to start editing. (-[KWQTextField currentEditorForEitherField]): Moved this method inside the KWQInternal category. (-[KWQTextField selectedRange]): Ditto. (-[KWQTextField setSelectedRange:]): Ditto. (-[KWQTextField setHasFocus:]): Added. This method does all the work we want to do when we get focus and when we relinquish focus, including scrolling to make the field visible. (-[KWQTextFieldCell editWithFrame:inView:editor:delegate:event:]): Call setHasFocus:YES on the KWQTextField. (-[KWQTextFieldCell selectWithFrame:inView:editor:delegate:start:length:]): Ditto. (+[KWQSecureTextField cellClass]): Use KWQSecureTextFieldCell as the cell class. We need a cell subclass to detect when editing begins and we can't use the normal technique because NSSecureTextField will raise an exception. (-[KWQSecureTextField nextKeyView]): Simplified to just call through to KWQTextField. (-[KWQSecureTextField previousKeyView]): Ditto. (-[KWQSecureTextField nextValidKeyView]): Ditto. (-[KWQSecureTextField previousValidKeyView]): Ditto. (-[KWQSecureTextField widget]): Ditto. (-[KWQSecureTextField fieldEditorDidMouseDown:]): Ditto. (-[KWQSecureTextFieldCell editWithFrame:inView:editor:delegate:event:]): Call setHasFocus:YES on the KWQTextField. (-[KWQSecureTextFieldCell selectWithFrame:inView:editor:delegate:start:length:]): Ditto.
  • kwq/KWQWidget.mm: (QWidget::setFocus): Do a layout before positioning the widget. The code dealt with calls after a layout and before a draw, before, but not calls before a layout.
2:59 PM Changeset in webkit [4316] by hyatt
  • 3 edits in trunk/WebCore

Fix for a problem where positioned/floating children with
percentage width tables inside them don't expand to fill
the width of their containing block.

The way to solve this is to detect this
case and treat the block as though it has an infinite maxwidth.

Note that this is a quirk only, since doing this for real would
be crazy. We're basically just matching broken WinIE behavior.

Reviewed by kocienda

  • khtml/rendering/render_block.cpp:
2:03 PM Changeset in webkit [4315]
  • 3 copies in tags/Safari-78-dev-release

This commit was manufactured by cvs2svn to create tag
'Safari-78-dev-release'.

2:03 PM Changeset in webkit [4314] by sullivan
  • 7 edits in trunk/WebKit

Closed up all gaps in WebKit between "ASSERT" and "(" to
make Darin's day a little brighter.

Reviewed by Darin

  • WebCoreSupport.subproj/WebBridge.m: (-[WebBridge pluginViewWithPackage:attributes:baseURL:]): (-[WebBridge handleMouseDragged:]): (-[WebBridge mayStartDragWithMouseDragged:]):
  • WebCoreSupport.subproj/WebImageRenderer.m: (-[WebImageRenderer drawClippedToValidInRect:fromRect:]):
  • WebView.subproj/WebBaseResourceHandleDelegate.m: (-[WebBaseResourceHandleDelegate _releaseResources]): (-[WebBaseResourceHandleDelegate dealloc]):
  • WebView.subproj/WebFramePrivate.m: (-[WebFrame _transitionToCommitted:]):
  • WebView.subproj/WebViewPrivate.m: (-[WebView _preferencesChangedNotification:]):
1:52 PM Changeset in webkit [4313] by sullivan
  • 8 edits in trunk/WebKit
  • fixed 3252632 -- Registering a WebDocumentView too early breaks built-in image viewing

A startup-performance optimization was breaking the case
where clients registered WebDocumentView types before the
first WebFrameView had been created. The fix is to allow
registering WebDocumentView types without retrieving the
built-in image types.

Reviewed by Darin

  • WebView.subproj/WebFrameViewPrivate.h: remove _viewTypes; expose _viewTypesAllowImageTypeOmission. All callers have to specify the boolean now.
  • WebView.subproj/WebFrameViewPrivate.m: (+[WebFrameView _viewTypesAllowImageTypeOmission:]): assert that the list of image types isn't nil before inserting them; insert each image type only if not already present. (+[WebFrameView _viewClassForMIMEType:]): replace _viewTypes with _viewTypesAllowImageTypeOmission
  • Plugins.subproj/WebPluginDatabase.m: (-[WebPluginDatabase init]): replace _viewTypes with _viewTypesAllowImageTypeOmission
  • WebView.subproj/WebDataSourcePrivate.h: remove _repTypes; expose _repTypesAllowImageTypeOmission. All callers have to specify the boolean now.
  • WebView.subproj/WebDataSourcePrivate.m: (+[WebDataSource _representationClassForMIMEType:]): replace _repTypes with _repTypesAllowImageTypeOmission
  • WebView.subproj/WebView.m: (+[WebView registerViewClass:representationClass:forMIMEType:]): replace _viewTypes with _viewTypesAllowImageTypeOmission, and replace _repTypes with _repTypesAllowImageTypeOmission
1:44 PM Changeset in webkit [4312] by sheridan
  • 3 edits in trunk

Safari-79u stamp

12:22 PM Changeset in webkit [4311] by darin
  • 3 edits in trunk/WebKit

Reviewed by Richard.

  • fixed 3242414 -- REGRESSION: PLT times reported 10% slower after emptying cache with B/F cache enabled
  • made it only compute the page cache size once
  • History.subproj/WebBackForwardList.m: (-[WebBackForwardList init]): Set initial page cache size to special value, COMPUTE_DEFAULT_PAGE_CACHE_SIZE. (-[WebBackForwardList setPageCacheSize:]): Remove code to set pageCacheSizeModified and call to _setUsesPageCache:, not needed any more. (-[WebBackForwardList pageCacheSize]): If cache size is COMPUTE_DEFAULT_PAGE_CACHE_SIZE, then compute it. The old code would compute the cache size each time this method was called until pageCacheSizeModified was set. (-[WebBackForwardList _usesPageCache]): Just check pageCacheSize for 0 to see if we use a page cache. No need for a separate boolean any more.
  • History.subproj/WebHistoryItemPrivate.h: Remove _setUsesPageCache (gone altogether) and _clearPageCache (now only used internally).
11:54 AM Changeset in webkit [4310]
  • 3 copies in branches/Safari-78~1-branch

This commit was manufactured by cvs2svn to create branch
'Safari-78~1-branch'.

11:54 AM Changeset in webkit [4309]
  • 3 copies in tags/Safari-78~1-anchor

This commit was manufactured by cvs2svn to create tag
'Safari-78~1-anchor'.

11:54 AM Changeset in webkit [4308]
  • 3 copies in tags/Safari-78

This commit was manufactured by cvs2svn to create tag 'Safari-78'.

11:54 AM Changeset in webkit [4307] by sheridan
  • 8 edits in trunk

Safari-78 version stamp

7:57 AM Changeset in webkit [4306] by darin
  • 3 edits in trunk/WebCore

Reviewed by Ken.

  • fixed 3174769 -- Safari doesn't ignore byte order mark at start of UTF-8 HTML document
  • khtml/misc/decoder.cpp: (Decoder::decode): Added handling of the UTF-8 BOM to the code that already handled the UTF-16 BOM.

May 7, 2003:

7:19 PM Changeset in webkit [4305] by hyatt
  • 34 edits
    3 adds
    6 deletes in trunk/WebCore

The purpose of this patch is to unify XML and HTML documents' root
element handling. Instead of having a special RenderHTML object
for dealing with <html>, root element handling is now done in
base classes so that it works with XML documents as well.

The changes include:

  • Rename RenderRoot to RenderCanvas to more accurately reflect what this RenderObject represents in CSS2.
  • Rename root() to canvas()
  • Rename isHtml() to isRoot()
  • Rename isRoot() to isCanvas().
  • Eliminate RenderHTML and fold its functionality into base classes (RenderBlock and RenderBox).
  • isRoot() now means "are you the RenderObject for the root element". All isHtml() callers now say isRoot() instead.

Reviewed by gramps

  • ForwardingHeaders/rendering/render_canvas.h: Added.
  • ForwardingHeaders/rendering/render_html.h: Removed.
  • ForwardingHeaders/rendering/render_root.h: Removed.
  • WebCore.pbproj/project.pbxproj:
  • khtml/ecma/kjs_dom.cpp:
  • khtml/ecma/kjs_window.cpp:
  • khtml/html/html_baseimpl.cpp: (HTMLHtmlElementImpl::id):
  • khtml/html/html_baseimpl.h:
  • khtml/khtmlview.cpp: (KHTMLView::layout): (KHTMLView::print): (KHTMLView::slotPaletteChanged): (KHTMLView::paint): (KHTMLView::timerEvent):
  • khtml/khtmlview.h:
  • khtml/rendering/render_applet.cpp:
  • khtml/rendering/render_block.cpp:
  • khtml/rendering/render_block.h:
  • khtml/rendering/render_body.cpp:
  • khtml/rendering/render_box.cpp: (RenderBox::setStyle): (RenderBox::paintRootBoxDecorations): (RenderBox::paintBoxDecorations): (RenderBox::paintBackgroundExtended): (RenderBox::containingBlockWidth): (RenderBox::repaint): (RenderBox::calcHeight): (RenderBox::availableHeight): (RenderBox::calcAbsoluteVertical):
  • khtml/rendering/render_box.h:
  • khtml/rendering/render_canvas.cpp: Added. (RenderCanvas::RenderCanvas): (RenderCanvas::~RenderCanvas): (RenderCanvas::calcHeight): (RenderCanvas::calcWidth): (RenderCanvas::calcMinMaxWidth):
  • khtml/rendering/render_canvas.h: Added.
  • khtml/rendering/render_container.cpp: (RenderContainer::removeChildNode):
  • khtml/rendering/render_flow.cpp:
  • khtml/rendering/render_frames.cpp: (RenderFrameSet::layout): (RenderFrameSet::userResize):
  • khtml/rendering/render_html.cpp: Removed.
  • khtml/rendering/render_html.h: Removed.
  • khtml/rendering/render_image.cpp: (RenderImage::paintObject):
  • khtml/rendering/render_layer.cpp: (RenderLayer::enclosingPositionedAncestor): (RenderLayer::convertToLayerCoords): (RenderLayer::constructZTree):
  • khtml/rendering/render_layer.h:
  • khtml/rendering/render_line.cpp:
  • khtml/rendering/render_list.cpp: (RenderListMarker::paintObject):
  • khtml/rendering/render_object.cpp: (RenderObject::containingBlock): (RenderObject::canvas): (RenderObject::container): (RenderObject::removeFromObjectLists): (RenderObject::detach): (RenderObject::nodeAtPoint): (RenderObject::scheduleRelayout):
  • khtml/rendering/render_object.h:
  • khtml/rendering/render_replaced.cpp:
  • khtml/rendering/render_root.cpp: Removed.
  • khtml/rendering/render_root.h: Removed.
  • khtml/rendering/render_table.cpp: (RenderTable::layout):
  • khtml/rendering/render_text.cpp: (RenderText::paintObject):
  • khtml/rendering/table_layout.cpp: (shouldScaleColumns):
  • khtml/xml/dom_docimpl.cpp: (DocumentImpl::attach): (DocumentImpl::setSelection): (DocumentImpl::clearSelection): (DocumentImpl::prepareMouseEvent):
  • khtml/xml/dom_elementimpl.cpp: (ElementImpl::createRenderer):
  • khtml/xml/dom_nodeimpl.cpp: (NodeImpl::rendererIsNeeded):
  • kwq/KWQKHTMLPart.mm: (KWQKHTMLPart::adjustPageHeight): (KWQKHTMLPart::forceLayoutForPageWidth): (KWQKHTMLPart::selectionRect):
  • kwq/WebCoreBridge.mm: (-[WebCoreBridge _setupRootForPrinting:]):
6:01 PM Changeset in webkit [4304] by darin
  • 14 edits in trunk

WebCore:

Reviewed by John.

  • fixed 3127927 -- web view should not use primary selection color when it's not first responder
  • kwq/WebCoreBridge.h: Added usesInactiveTextBackgroundColor getter and setter.
  • kwq/WebCoreBridge.mm: (-[WebCoreBridge drawRect:]): Set the QPainter's usesInactiveTextBackgroundColor to match the value from the part. (-[WebCoreBridge selectionRect]): Fix nil-deref that happened in the no-KHTMLView case. (-[WebCoreBridge setUsesInactiveTextBackgroundColor:]): Added. (-[WebCoreBridge usesInactiveTextBackgroundColor]): Added.
  • kwq/KWQKHTMLPart.h: Added usesInactiveTextBackgroundColor, including getter and setter.
  • kwq/KWQKHTMLPart.mm: (KWQKHTMLPart::KWQKHTMLPart): Initialize usesInactiveTextBackgroundColor.
  • kwq/KWQPainter.h: Added setUsesInactiveTextBackgroundColor and made selectedTextBackgroundColor a const member function, not a static one.
  • kwq/KWQPainter.mm: (QPainter::QPainter): Initialize _usesInactiveTextBackgroundColor to false. (QPainter::selectedTextBackgroundColor): Return secondarySelectedControlColor instead of selectedTextBackgroundColor if _usesInactiveTextBackgroundColor is true.
  • khtml/rendering/render_text.cpp: (TextRun::paintSelection): Call selectedTextBackgroundColor on the painter object; it's no longer a static member function.
  • khtml/rendering/render_root.cpp: (RenderRoot::selectionRect): Fix nil-deref that happened in the no-selection case.

WebKit:

Reviewed by John.

  • fixed 3127927 -- web view should not use primary selection color when it's not first responder
  • WebView.subproj/WebHTMLView.m: (-[WebHTMLView updateTextBackgroundColor]): Added. Sets the usesInactiveTextBackgroundColor flag on the bridge, and does setNeedsDisplayInRect: of the selectionRect if the state changes. (-[WebHTMLView viewDidMoveToWindow]): Call updateTextBackgroundColor. (-[WebHTMLView windowDidBecomeKey:]): Call updateTextBackgroundColor. (-[WebHTMLView windowDidResignKey:]): Call updateTextBackgroundColor. (-[WebHTMLView becomeFirstResponder]): Call updateTextBackgroundColor. (-[WebHTMLView resignFirstResponder]): Call updateTextBackgroundColor.
  • fixed up WebHistoryItem initializers so there is a designated initializer
  • History.subproj/WebHistoryItem.m: (-[WebHistoryItem init]): Call initWithURLString:title:lastVisitedTimeInterval:, which is the designated initializer. (-[WebHistoryItem initWithURL:title:]): Ditto. (-[WebHistoryItem initWithURL:target:parent:title:]): Ditto.
  • other changes
  • History.subproj/WebHistoryPrivate.m: (-[WebHistoryPrivate addItem:]): Fixed assertion to not use _lastVisitedDate; we're trying to get rid of calls to that method.
5:13 PM Changeset in webkit [4303] by vicki
  • 8 edits in trunk

JavaScriptCore:

Reviewed by darin.

  • modify the Mixed build style to build optimized with symbols

WebFoundation:

Reviewed by darin.

  • modify the Mixed build style to build optimized with symbols
  • WebFoundation.pbproj/project.pbxproj: removed OPTIMIZATION_CFLAGS

WebCore:

Reviewed by darin.

  • modify the Mixed build style to build optimized with symbols
  • WebCore.pbproj/project.pbxproj: removed OPTIMIZATION_CFLAGS, use WebCore.exp

WebKit:

Reviewed by darin.

  • modify the Mixed build style to build optimized with symbols
  • WebKit.pbproj/project.pbxproj: removed OPTIMIZATION_CGLAGS

WebBrowser:

Reviewed by darin.

  • modify the Mixed build style to build optimized with symbols
  • WebBrowser.pbproj/project.pbxproj: removed OPTIMIZATION_CFLAGS
9:01 AM Changeset in webkit [4302] by darin
  • 5 edits in trunk/WebCore

Reviewed by Ken.

  • fixed 3109258 -- Unicode above U+FFFF not handled correctly when expressed as numeric entities
  • khtml/html/htmltokenizer.cpp: (HTMLTokenizer::parseEntity): Keep track of the entity in an unsigned integer that can hold 32 bits rather than a QChar. Convert to a pair of Unicode surrogate code points if necessary.
  • khtml/misc/stringit.h: Add a second pushed character, used for the case where we have a pair of surrogate characters for a single numeric entity.
  • khtml/html/htmltokenizer.h: Add missing DOM:: that was compiling only due to "using namespace DOM" in stringit.h. Replace EntityChar with EntityUnicodeValue.

May 6, 2003:

6:56 PM Changeset in webkit [4301]
  • 21 copies
    1 delete in tags/Safari-77~1

This commit was manufactured by cvs2svn to create tag 'Safari-77~1'.

6:56 PM Changeset in webkit [4300] by rjw
  • 2 edits in branches/Safari-77~1-branch

Revert framework references in project.pbxproj.

6:11 PM Changeset in webkit [4299] by darin
  • 12 edits in trunk/WebCore

Reviewed by Chris.

  • fixed 3211756 -- form field contents on the left instead of right on page in RtL language
  • khtml/rendering/render_form.cpp: (RenderLineEdit::updateFromElement): Call setAlignment based on the style's direction. (RenderTextArea::updateFromElement): Ditto.
  • kwq/KWQLineEdit.h: Added setAlignment.
  • kwq/KWQLineEdit.mm: (QLineEdit::setAlignment): Added. Calls setAlignment on KWQTextField.
  • kwq/KWQTextArea.h: Added setAlignment.
  • kwq/KWQTextArea.mm: (-[KWQTextArea setAlignment:]): Added. Calls setAlignment on NSTextView.
  • kwq/KWQTextEdit.h: Added setAlignment.
  • kwq/KWQTextEdit.mm: (QTextEdit::setAlignment): Added. Calls setAlignment on KWQTextField.
  • kwq/KWQTextField.mm: (-[KWQTextField setAlignment:]): Added. Call setAlignment on secure text field too.
  • fixed 3250538 -- on pages labeled "US-ASCII", non-ASCII chars are not decoded as Latin-1 as in other browsers
  • kwq/KWQTextCodec.mm: (effectiveEncoding): Added. Maps ISO Latin-1 and US-ASCII to Windows Latin-1. (QTextCodec::fromUnicode): Use effectiveEncoding. (KWQTextDecoder::convertUsingTEC): Use effectiveEncoding.
  • other changes
  • kwq/KWQCharsets.mm: (buildDictionaries): Make the first encoding in the file win for purposes of deciding which name is used. I thought this was happening before, but actually the last one was winning. Do this by using CFDictionaryAddValue, which does nothing if there's already a dictionary entry, instead of CFDictionarySetValue, which replaces.
6:06 PM Changeset in webkit [4298] by rjw
  • 8 edits in branches/Safari-77~1-branch/WebKit

Fixed build problems caused by mistaken additional merge.

5:43 PM Changeset in webkit [4297] by rjw
  • 8 edits in branches/Safari-77~1-branch

Backed out install name and release directory change.
Frameworks will be private for one more release.

  • WebKit.pbproj/project.pbxproj:
5:31 PM Changeset in webkit [4296] by rjw
  • 7 edits in branches/Safari-77~1-branch/WebKit

Merged changes from HEAD:

Fixed problem for HelpViewer. HV calls stopLoading in
a webView:resource:didReceiveResponse. This causes
the premature release of the connection delegate.

Reviewed by Ken.

  • WebCoreSupport.subproj/WebSubresourceClient.m: (-[WebSubresourceClient connectionDidFinishLoading:]):
  • WebKit.pbproj/project.pbxproj:
  • WebView.subproj/WebBaseResourceHandleDelegate.m: (-[WebBaseResourceHandleDelegate _releaseResources]): (-[WebBaseResourceHandleDelegate dealloc]):
  • WebView.subproj/WebMainResourceClient.m: (-[WebMainResourceClient stopLoadingForPolicyChange]): (-[WebMainResourceClient continueAfterContentPolicy:response:]): (-[WebMainResourceClient continueAfterContentPolicy:]): (-[WebMainResourceClient connection:didReceiveResponse:]): (-[WebMainResourceClient connectionDidFinishLoading:]):
5:15 PM Changeset in webkit [4295] by rjw
  • 8 edits in trunk/WebKit

Fixed problem for HelpViewer. HV calls stopLoading in
a webView:resource:didReceiveResponse. This causes
the premature release of the connection delegate.

Reviewed by Ken.

  • WebCoreSupport.subproj/WebSubresourceClient.m: (-[WebSubresourceClient connectionDidFinishLoading:]):
  • WebKit.pbproj/project.pbxproj:
  • WebView.subproj/WebBaseResourceHandleDelegate.m: (-[WebBaseResourceHandleDelegate _releaseResources]): (-[WebBaseResourceHandleDelegate dealloc]):
  • WebView.subproj/WebMainResourceClient.m: (-[WebMainResourceClient stopLoadingForPolicyChange]): (-[WebMainResourceClient continueAfterContentPolicy:response:]): (-[WebMainResourceClient continueAfterContentPolicy:]): (-[WebMainResourceClient connection:didReceiveResponse:]): (-[WebMainResourceClient connectionDidFinishLoading:]):
3:26 PM Changeset in webkit [4294] by hyatt
  • 11 edits in trunk/WebCore

Make negative z-index content paint above the background of
the stacking context element that encloses it. This fixes
gettyimages.com and hotwired.com.

Reviewed by kocienda

  • khtml/css/html4.css:
  • khtml/rendering/render_block.cpp:
  • khtml/rendering/render_inline.cpp: (RenderInline::paintObject):
  • khtml/rendering/render_layer.cpp: (RenderLayer::paint): (RenderLayer::nodeAtPoint): (RenderLayer::RenderZTreeNode::constructLayerList):
  • khtml/rendering/render_layer.h:
  • khtml/rendering/render_object.h:
  • khtml/rendering/render_root.cpp:
  • khtml/rendering/render_table.cpp: (RenderTable::paint):
  • kwq/KWQRenderTreeDebug.cpp: (write):
2:31 PM Changeset in webkit [4293] by darin
  • 1 edit
    1 delete in trunk/WebKit
  • removed obsolete file that's still around for some reason
  • Downloads.subproj/WebDownload.m: Removed.
1:28 PM Changeset in webkit [4292] by darin
  • 2 edits in trunk/WebKit
  • fixed crash on startup
  • History.subproj/WebHistoryItem.m: (-[WebHistoryItem initWithURLString:title:lastVisitedTimeInterval:]): Allocate WebHistoryItemPrivate.
11:26 AM Changeset in webkit [4291] by rjw
  • 8 edits in branches/Safari-77~1-branch

Changed install names and release directory to
/System/Library/Frameworks.

11:11 AM Changeset in webkit [4290] by kocienda
  • 1 edit in trunk/WebKit/Misc.subproj/WebFileDatabase.m

Reviewed by Richard

Refactored memory and disk cache. All the implementation
for the memory and disk caches are now concentrated in one
file, NSURLCache.m. This spells the end for the WebCache,
WebMemoryCache and WebDiskCache files. It also enabled
me to implement the API-approved design which calls upon
NSURLProtocol to make a determination about the cache
equivalence of cached objects.

NSURLRequest and friends are now codable.

WebLRUFileList gets a new function: WebLRUFileListContainsItem.

In WebNSFileManagerExtras, I fixed a gibberish method name:

_web_deleteBackgroundRemoveLeftoverFiles:

is now

_web_backgroundRemoveLeftoverFiles:

  • CacheLoader.subproj/NSURLCache.m: (-[NSURLCacheInternal dealloc]): (-[NSURLCacheNode initWithCachedResponse:request:key:]): (-[NSURLCacheNode isEqual:]): (-[NSURLCacheNode description]): (-[NSURLCacheNode dealloc]): (MemoryCacheRemoveNodeFromLRUList): (MemoryCacheAppendNodeToLRUList): (MemoryCacheTouchNode): (MemoryCacheTruncate): (MemoryCacheGet): (MemoryCacheSet): (MemoryCacheRemove): (MemoryCacheClear): (DiskCacheDefaultPath): (DiskCacheUniqueFilePathForKey): (DiskCacheCreateDirectory): (DiskCacheGet): (DiskCacheClear): (DiskCacheTruncate): (DiskCacheScheduleWrite): (DiskCacheExecuteWrite): (DiskCacheScheduleRemoval): (DiskCacheExecuteRemoval): (+[NSURLCache _diskCacheSyncLoop:]): (-[NSURLCache _setDiskCacheSyncTimer]): (-[NSURLCache _diskCacheSync:]): (-[NSURLCache _createDiskCacheLRUList:]): (NSURLCacheInit): (NSURLCacheCreateSharedInstance): (+[NSURLCache sharedURLCache]): (-[NSURLCache initWithMemoryCapacity:diskCapacity:diskPath:]): (-[NSURLCache cachedResponseForRequest:]): (-[NSURLCache storeCachedResponse:forRequest:]): (-[NSURLCache removeCachedResponseForRequest:]): (-[NSURLCache removeAllCachedResponses]): (-[NSURLCache memoryCapacity]): (-[NSURLCache diskCapacity]): (-[NSURLCache setMemoryCapacity:]): (-[NSURLCache setDiskCapacity:]): (-[NSURLCache currentMemoryUsage]): (-[NSURLCache currentDiskUsage]): (-[NSURLCache _cacheStatistics]):
  • CacheLoader.subproj/NSURLRequest.h:
  • CacheLoader.subproj/NSURLRequest.m: (-[NSURLRequest initWithCoder:]): (-[NSURLRequest encodeWithCoder:]): (-[NSURLRequest HTTPMethod]): (-[NSURLRequest allHTTPHeaderFields]): (-[NSURLRequest valueForHTTPHeaderField:]): (-[NSURLRequest HTTPBody]): (-[NSURLRequest allowsHTTPSCertificatesWithUnknownRoots]): (-[NSURLRequest HTTPCookiePolicyBaseURL]): (-[NSURLRequest HTTPExtraCookies]): (-[NSURLRequest HTTPPageNotFoundCacheLifetime]): (-[NSURLRequest HTTPShouldHandleCookies]): (-[NSMutableURLRequest _parametersForWriting]): (-[NSMutableURLRequest setHTTPMethod:]): (-[NSMutableURLRequest setAllHTTPHeaderFields:]): (-[NSMutableURLRequest setValue:forHTTPHeaderField:]): (-[NSMutableURLRequest addValue:forHTTPHeaderField:]): (-[NSMutableURLRequest setHTTPBody:]): (-[NSMutableURLRequest setAllowsHTTPSCertificatesWithUnknownRoots:]): (-[NSMutableURLRequest setHTTPCookiePolicyBaseURL:]): (-[NSMutableURLRequest setHTTPExtraCookies:]): (-[NSMutableURLRequest setHTTPPageNotFoundCacheLifetime:]): (-[NSMutableURLRequest setHTTPShouldHandleCookies:]): (-[NSHTTPURLRequestParameters initWithCoder:]): (-[NSHTTPURLRequestParameters encodeWithCoder:]): (-[NSHTTPURLRequestParameters copyWithZone:]): (-[NSHTTPURLRequestParameters mutableCopyWithZone:]): (-[NSHTTPURLRequestParameters isEqual:]):
  • CacheLoader.subproj/WebCache.h: Removed.
  • CacheLoader.subproj/WebCache.m: Removed.
  • CacheLoader.subproj/WebCachePrivate.h: Removed.
  • CacheLoader.subproj/WebDiskCache.h: Removed.
  • CacheLoader.subproj/WebDiskCache.m: Removed.
  • CacheLoader.subproj/WebMemoryCache.h: Removed.
  • CacheLoader.subproj/WebMemoryCache.m: Removed.
  • Database.subproj/WebFileDatabase.m: (-[WebFileDatabase open]):
  • Database.subproj/WebLRUFileList.h:
  • Database.subproj/WebLRUFileList.m: (WebLRUFileListContainsItem):
  • Misc.subproj/WebNSFileManagerExtras.h:
  • Misc.subproj/WebNSFileManagerExtras.m: (-[NSFileManager _web_backgroundRemoveLeftoverFiles:]):
  • WebFoundation.pbproj/project.pbxproj:
10:50 AM Changeset in webkit [4289] by mjs
  • 7 edits in trunk/WebCore

Reviewed by Darin.

  • fixed 3245048 - "Block Pop-Up Windows" can easily be bypassed using javascript: URLs

Closed the loophole by tracking what client redirects were
triggered by user actions, and making sure the script interpreter
knows.

  • khtml/ecma/kjs_window.cpp: (Window::put): (WindowFunc::tryCall): (Location::put): (LocationFunc::tryCall):
  • khtml/khtml_part.cpp: (KHTMLPart::executeScript): (KHTMLPart::clear): (KHTMLPart::scheduleRedirection): (KHTMLPart::slotRedirect): (KHTMLPart::urlSelected):
  • khtml/khtml_part.h:
  • khtml/khtmlpart_p.h:
  • kwq/WebCoreBridge.mm: (-[WebCoreBridge stringByEvaluatingJavaScriptFromString:]):
10:47 AM Changeset in webkit [4288]
  • 21 copies in tags/Safari-73~5

This commit was manufactured by cvs2svn to create tag 'Safari-73~5'.

10:47 AM Changeset in webkit [4287] by darin
  • 2 edits in branches/Safari-73~1-branch/WebKit

Rolled in change from trunk.

2003-05-06 Darin Adler <Darin Adler>

  • fixed 3249211 -- WebTextRenderer.h should not use "AttributeGroup" type for styleGroup
  • WebCoreSupport.subproj/WebTextRenderer.h: Update to use ATSStyleGroupPtr.
10:43 AM Changeset in webkit [4286] by darin
  • 2 edits in branches/Safari-77~1-branch/WebKit

Rolled in change from main trunk.

2003-05-06 Darin Adler <Darin Adler>

  • fixed 3249211 -- WebTextRenderer.h should not use "AttributeGroup" type for styleGroup
  • WebCoreSupport.subproj/WebTextRenderer.h: Update to use ATSStyleGroupPtr.
10:42 AM Changeset in webkit [4285] by darin
  • 2 edits in trunk/WebKit
  • fixed 3249211 -- WebTextRenderer.h should not use "AttributeGroup" type for styleGroup
  • WebCoreSupport.subproj/WebTextRenderer.h: Update to use ATSStyleGroupPtr.
9:20 AM Changeset in webkit [4284] by darin
  • 4 edits in trunk/WebKit

Reviewed by Ken.

  • used ObjectAlloc to find large numbers of allocations on startup and get rid of some
  • History.subproj/WebHistoryItem.m: (-[WebHistoryItem initFromDictionaryRepresentation:]): Use the init method that takes a URL string so we don't have to create and then destroy a URL for each item we decode.
  • History.subproj/WebHistoryPrivate.m: (-[WebHistoryPrivate addItem:]): Use URLString instead of making and destroying a URL each time this is called. (-[WebHistoryPrivate removeItem:]): Ditto. (-[WebHistoryPrivate _loadHistoryGuts:URL:error:]): Check URLString for nil; no reason to construct and then destroy a URL just to check validity.
  • WebCoreSupport.subproj/WebTextRendererFactory.m: (FontCacheKeyCopy): Added. (FontCacheKeyFree): Added. (FontCacheKeyEqual): Added. (FontCacheKeyHash): Added. (FontCacheValueRetain): Added. (FontCacheValueRelease): Added. (-[WebTextRendererFactory cachedFontFromFamily:traits:size:]): Use a C struct for the font cache key instead of using an Objective-C object. This saves us an object allocation and deallocation when doing a lookup. Also took advantage of the CFDictionary ability to store NULL and distinguish it from "not found" so we don't need a separate set for cache misses.
8:59 AM Changeset in webkit [4283] by darin
  • 3 edits in trunk/WebCore

Reviewed by Ken.

  • used ObjectAlloc to find large numbers of allocations on startup and get rid of some
  • kwq/KWQColor.mm: (QColor::getNSColor): Keep a cache of 32 colors instead of returning a new one each time. Also use special cases for black and white.

May 5, 2003:

10:28 PM Changeset in webkit [4282] by mjs
  • 3 edits in trunk/JavaScriptCore

Reviewed by Don.

  • fixed 3239961 - www.phiffer.com doesn't work; uses "var top; top = n;"
  • kjs/nodes.cpp: (VarDeclNode::evaluate): Check if the property exists with getDirect() instead of hasProperty().
9:09 PM Changeset in webkit [4281] by mjs
  • 4 edits in trunk/WebCore

Reviewed by Dave.

  • fixed 3241065 - popcap.com doesn't work in Safari, window.scrollBy fails to work in onLoad

Make sure to do a layout before accessing any window properties or
calling any window functions that require a layout to work
properly.

  • khtml/ecma/kjs_window.cpp: (Window::get): (WindowFunc::tryCall): (Window::updateLayout):
  • khtml/ecma/kjs_window.h:
7:47 PM Changeset in webkit [4280] by mjs
  • 4 edits in trunk/WebKit

Reviewed by Richard.

Fix build by separating mutating data protocol methods into
category on NSMutableURLRequest. Also, formatting fixes.

  • WebView.subproj/WebDataProtocol.h:
  • WebView.subproj/WebDataProtocol.m: (+[NSURLRequest _webDataRequestURLForData:]): (-[NSURLRequest _webDataRequestData]): (-[NSURLRequest _webDataRequestEncoding]): (-[NSURLRequest _webDataRequestMIMEType]): (-[NSURLRequest _webDataRequestBaseURL]): (-[NSURLRequest _webDataRequestExternalRequest]): (-[NSMutableURLRequest _webDataRequestSetData:]):
  • WebView.subproj/WebFrame.m:
4:26 PM Changeset in webkit [4279] by hyatt
  • 4 edits in trunk/WebCore

Fix for 3247722. Make sure the grammar accepts properties with
no values as declarations (albeit bad ones).

Reviewed by darin

  • khtml/css/parser.cpp:
  • khtml/css/parser.y:
10:51 AM Changeset in webkit [4278] by hyatt
  • 3 edits in trunk/WebCore

Fix for 3248587. Refine the crash fix for generated image
content so that we still set the image's intrinsic width
and height properly.

Reviewed by john

  • khtml/rendering/render_image.cpp: (RenderImage::setPixmap):
10:31 AM Changeset in webkit [4277] by hyatt
  • 8 edits in trunk/WebCore

When using the content property in CSS, you can string together
an intermixed list of strings and image URLs. The old code
only supported a single image URL, and if you tried to mix
text in with images, you ended up crashing.

The crash bug is 3248172. The fix is to maintain a list of
ContentData objects instead of just a single object and to
ensure that a whole list of generated content gets made (and
not just a single RenderObject).

Reviewed by john

  • khtml/css/cssstyleselector.cpp: (applyRule)
  • khtml/rendering/render_container.cpp: (RenderContainer::insertPseudoChild):
  • khtml/rendering/render_image.cpp: (RenderImage::setStyle): (RenderImage::setContentObject): (RenderImage::updateFromElement):
  • khtml/rendering/render_image.h:
  • khtml/rendering/render_style.cpp: (RenderStyle::setContent): (ContentData::clearContent):
  • khtml/rendering/render_style.h:
10:22 AM Changeset in webkit [4276] by darin
  • 3 edits in trunk/WebCore

Reviewed by John.

  • fixed replaceOccurrencesOfString crash reported in bug 3247249 (not what the bug is about)
  • kwq/KWQTextArea.mm: (-[KWQTextArea text]): Use the length of the replaced text.

May 4, 2003:

9:28 PM Changeset in webkit [4275] by darin
  • 2 edits in trunk/WebKit

Reviewed by Maciej.

  • fixed a storage leak
  • WebView.subproj/WebFramePrivate.m: (-[WebFrame _setState:]): Release the timer before setting to nil.
7:57 PM Changeset in webkit [4274] by darin
  • 4 edits in trunk/WebCore

Reviewed by John.

  • fixed 3131534 -- "javascript:window.open()" leads to page not found error
  • khtml/ecma/kjs_window.cpp: (WindowFunc::tryCall): For open() calls, if the first parameter is not present, use "" instead of "undefined" as the URL.
  • fixed 3244766 -- window.open() doesn't position window correctly along the y axis
  • kwq/KWQKHTMLPartBrowserExtension.mm: (KHTMLPartBrowserExtension::createNewWindow): Fixed the code that flips the y coordinate to take the window height into account.
3:21 PM Changeset in webkit [4273] by hyatt
  • 5 edits in trunk/WebKit

Add smallCaps boolean to the string measuring and drawing
methods. Not actually supported yet.

Reviewed by darin

  • Misc.subproj/WebKitNSStringExtras.m: (-[NSString _web_widthWithFont:]):
  • Misc.subproj/WebStringTruncator.m: (stringWidth):
  • WebCoreSupport.subproj/WebTextRenderer.h:
  • WebCoreSupport.subproj/WebTextRenderer.m: (-[WebTextRenderer widthForCharacters:length:]): (-[WebTextRenderer drawCharacters:stringLength:fromCharacterPosition:toCharacterPosition:atPoint:withPadding:withTextColor:backgroundColor:rightToLeft:letterSpacing:wordSpacing:smallCaps:fontFamilies:]): (-[WebTextRenderer floatWidthForCharacters:stringLength:characterPosition:]): (-[WebTextRenderer floatWidthForCharacters:stringLength:fromCharacterPosition:numberOfCharacters:]): (-[WebTextRenderer floatWidthForCharacters:stringLength:fromCharacterPosition:numberOfCharacters:withPadding:applyRounding:attemptFontSubstitution:widths:letterSpacing:wordSpacing:smallCaps:fontFamilies:]): (-[WebTextRenderer _floatWidthForCharacters:stringLength:fromCharacterPosition:numberOfCharacters:withPadding:applyRounding:attemptFontSubstitution:widths:fonts:glyphs:numGlyphs:letterSpacing:wordSpacing:smallCaps:fontFamilies:]):
3:17 PM Changeset in webkit [4272] by hyatt
  • 8 edits in trunk/WebCore

Add smallCaps boolean to the string drawing and measuring
routines. WebCore should be done after this patch, with
the rest of the work being done entirely in WebKit.

Reviewed by darin

  • khtml/rendering/font.cpp: (Font::drawText): (Font::floatWidth): (Font::floatCharacterWidths): (Font::width):
  • kwq/KWQFontMetrics.h:
  • kwq/KWQFontMetrics.mm: (QFontMetrics::width): (QFontMetrics::floatWidth): (QFontMetrics::floatCharacterWidths):
  • kwq/KWQPainter.h:
  • kwq/KWQPainter.mm: (QPainter::drawText):
  • kwq/WebCoreTextRenderer.h:
3:08 PM Changeset in webkit [4271] by darin
  • 3 edits in trunk/WebKit

Reviewed by John.

  • added validation to "Interface Builder" methods on WebView
  • WebView.subproj/WebView.h: Add NSUserInterfaceValidations protocol so that subclassers know that they can call [super validateUserInterfaceItem:].
  • WebView.subproj/WebView.m: (-[WebView canMakeTextSmaller]): Simplify. (-[WebView canMakeTextLarger]): Simplify. (-[WebView _isLoading]): Added. (-[WebView validateUserInterfaceItem:]): Added. Checks for the six actions we implement, and returns NO if they are not valid.
Note: See TracTimeline for information about the timeline view.