1. 18 Nov, 2002 24 commits
    • darin's avatar
      * force-js-clean-timestamp: I changed the size of ObjectImp. · 05e5f357
      darin authored
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2750 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • darin's avatar
      - property and string improvements giving a 7% or so improvement in JavaScript iBench · a6cae2cc
      darin authored
              * kjs/property_map.h: Rewrite to use a hash table.
              * kjs/property_map.cpp: Ditto.
              * kjs/string_object.h:
              * kjs/string_object.cpp:
              (StringInstanceImp::StringInstanceImp): Construct a string with the right value
      	instead of putting the string in later.
              (StringInstanceImp::get): Get the length from the string, not a separate property.
              (StringInstanceImp::put): Ignore attempts to set length, since we don't put it in
      	the property map.
              (StringInstanceImp::hasProperty): Return true for length.
              (StringInstanceImp::deleteProperty): Return false for length.
              (StringObjectImp::construct): Call new StringInstanceImp constructor. Don't try
      	to set a length property.
              * kjs/ustring.h: Make the rep deref know how to deallocate the rep.
              * kjs/ustring.cpp:
              (UString::release): Move the real work to the rep's deref, since the hash table
      	now uses the rep directly.
              * kjs/object.h: Remove unused field.
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2749 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • mjs's avatar
      Change List to completely avoid going through the GC · 1e8f66ff
      mjs authored
      	allocator. 3.6% performance improvement on JavaScript iBench.
              * kjs/internal.cpp:
              (InterpreterImp::mark): Don't mark the empty list.
      	For all the methods below I basically lifted the ListImp version
      	up to the List method with minor tweaks.
              * kjs/types.cpp:
              * kjs/types.h:
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2748 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • mjs's avatar
      Fixed a horrible leak introduced with my last change that · 6e9d9853
      mjs authored
      	somehow did not show up on my machine.
              * kjs/types.cpp:
              (List::List): Mark ListImp as GC allowed.
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2747 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • hyatt's avatar
      Fix crazyapplerumors.com. I had the logic backwards. CSS · 3c1544fd
      hyatt authored
      	padding beats cellpadding.
              * khtml/rendering/render_table.cpp:
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2746 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • hyatt's avatar
      Fix crash on espn.com. · a1993bd6
      hyatt authored
              * khtml/rendering/render_object.cpp:
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2745 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • mjs's avatar
      Another step towards the List conversion: stop inheriting from Value. · 3964a7b6
      mjs authored
              * kjs/types.cpp:
              * kjs/types.h:
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2744 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • hyatt's avatar
      Fix my <konqblock> regression on ign.com. · 07c921f4
      hyatt authored
              * khtml/html/htmlparser.cpp:
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2743 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • cblu's avatar
      Real fix for: 3104183 - Assert loading www.louisvuitton.com · 4f7065eb
      cblu authored
      	Since we don't consider plug-in content as a subresource, attaching plug-in streams to the datasource has no affect on the loading state of the datasource. Both stopLoading on WebFrame and _stopLoading datasource, would not stop plug-ins streams after the data source was done loading. Because of this, I've decided to not keep a list of plug-in streams attached to the data source and pulled my previous change. To fix this bug, WebBaseResourceHandleDelegate now retains the controller.
              * Plugins.subproj/WebNetscapePluginStream.m:
              (-[WebNetscapePluginStream start]):
              (-[WebNetscapePluginStream cancel]):
              (-[WebNetscapePluginStream handleDidFinishLoading:]):
              (-[WebNetscapePluginStream handle:didFailLoadingWithError:]):
              * WebView.subproj/WebBaseResourceHandleDelegate.h:
              * WebView.subproj/WebBaseResourceHandleDelegate.m:
              (-[WebBaseResourceHandleDelegate _releaseResources]):
              (-[WebBaseResourceHandleDelegate setDataSource:]):
              (-[WebBaseResourceHandleDelegate handle:willSendRequest:]):
              (-[WebBaseResourceHandleDelegate handleDidFinishLoading:]):
              (-[WebBaseResourceHandleDelegate handle:didFailLoadingWithError:]):
              (-[WebBaseResourceHandleDelegate _cancelWithError:]):
              * WebView.subproj/WebDataSourcePrivate.h:
              * WebView.subproj/WebDataSourcePrivate.m:
              (-[WebDataSourcePrivate dealloc]):
              (-[WebDataSource _stopLoading]):
              (-[WebDataSource _makeHandleDelegates:deferCallbacks:]):
              (-[WebDataSource _defersCallbacksChanged]):
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2742 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • mjs's avatar
      JavaScriptCore: · effd7e1c
      mjs authored
      	Partway to removing Value from List. Created a marking List
      	variant, used it in place of ListImp.
              * kjs/internal.h: Removed List stuff.
              * kjs/internal.cpp:
              (InterpreterImp::mark): Call appropriate List method to do marking of
      	empty ListImp.
              * kjs/object.h:
              * kjs/object.cpp: Use marking List instead of ListImp *.
              * kjs/types.h:
              * kjs/types.cpp:
              (List::List): New boolean needsMarking parameter.
              (List::operator=): Perform trickery related to needsMarking.
              (List::~List): Likewise.
              (List::mark): Mark the ListImp.
      	(ListImp::*): Moved here fron internal.cpp, they will be
      	integrated into the relevant List methods soon.
              * force-js-clean-timestamp: Rebuild for JSC changes.
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2741 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • darin's avatar
      - another string constant discovered that can be optimized · 2dd6a841
      darin authored
              * kjs/object.h: Add a property name constant for "__proto__".
              * kjs/object.cpp: Define it.
      	(ObjectImp::get): Use it.
      	(ObjectImp::hasProperty): Use it.
      	- prepare to turn PropertyMap into a hash table
              * kjs/object.cpp:
      	(ObjectImp::mark): Use the new PropertyMap::mark().
      	(ObjectImp::put): Use the new overload of PropertyMap::get().
      	(ObjectImp::deleteProperty): Use the new overload of PropertyMap::get().
      	(ObjectImp::propList): Use PropertyMap::addEnumerablesToReferenceList().
              * kjs/property_map.h: Remove PropertyMapNode and make all node-related methods private.
      	Add mark(), a new overload of get() that returns attributes, a clear() that takes no attributes,
      	and addEnumerablesToReferenceList().
              * kjs/property_map.cpp:
      	(PropertyMap::get): Added new overload.
      	(PropertyMap::clear): Added new overload.
      	(PropertyMap::mark): Added.
      	(PropertyMap::addEnumerablesToReferenceList): Added.
              * kjs/ustring.h: Added a hash function.
              * kjs/ustring.cpp: (KJS::hash): Added.
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2740 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • darin's avatar
      * force-js-clean-timestamp: Touch files to make the build work. · de6825fb
      darin authored
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2739 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • darin's avatar
      - simplified the ExecState class, which was showing up in profiles · 3e1ce6e5
      darin authored
              Sped up JavaScript iBench by 6%.
              * kjs/interpreter.h: Removed the level of indirection, and made it all inline.
              * kjs/interpreter.cpp: Removed ExecState implementation from here altogether.
      	- fixed an oversight in my sort speedup
              * kjs/array_object.h: Add pushUndefinedObjectsToEnd.
              * kjs/array_object.cpp:
              (ArrayInstanceImp::sort): Call pushUndefinedObjectsToEnd.
              (ArrayInstanceImp::pushUndefinedObjectsToEnd): Added.
      	Pushes all undefined to the end of the array.
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2738 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • cblu's avatar
      Fixed: 3104183 - Assert loading www.louisvuitton.com · 6d927229
      cblu authored
      	We need to treat plug-in streams like subresources except they don't change the loading state of the data source.
              * Plugins.subproj/WebNetscapePluginStream.m:
              (-[WebNetscapePluginStream start]): call _addPluginStream
              (-[WebNetscapePluginStream cancel]): call _removePluginStream
              (-[WebNetscapePluginStream handleDidFinishLoading:]): call _removePluginStream
              (-[WebNetscapePluginStream handle:didFailLoadingWithError:]): call _removePluginStream
              * WebView.subproj/WebDataSourcePrivate.h:
              * WebView.subproj/WebDataSourcePrivate.m:
              (-[WebDataSourcePrivate dealloc]): release pluginStreams
              (-[WebDataSource _addPluginStream:]): just like addSubresourceClient except don't change the loading state.
              (-[WebDataSource _removePluginStream:]): just like removeSubresourceClient except don't change the loading state.
              (-[WebDataSource _stopLoading]): stop plug-ins streams
              (-[WebDataSource _makeHandleDelegates:deferCallbacks:]):
              (-[WebDataSource _defersCallbacksChanged]): call _makeHandleDelegates:deferCallbacks: for subresources and plug-in streams.
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2737 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • darin's avatar
      - fix worst speed problems on the sort page of the iBench JavaScript test · 401641ee
      darin authored
      	Sped up JavaScript iBench by 70%, the sort page by 88%.
              * kjs/array_object.h: Add array-specific sort functions.
              * kjs/array_object.cpp:
              (compareByStringForQSort): Added.
              (ArrayInstanceImp::sort): Added.
              (compareWithCompareFunctionForQSort): Added.
              (ArrayProtoFuncImp::call): Use ArrayInstanceImp::sort if the object being
      	sorted is actually an array.
              * kjs/object.h: Add argumentsPropertyName.
              * kjs/object.cpp: Add argumentsPropertyName.
              * kjs/function.cpp:
              (FunctionImp::FunctionImp): Use argumentsPropertyName to avoid making a UString.
              (FunctionImp::call): Ditto.
              (ActivationImp::ActivationImp): Ditto.
              * kjs/function_object.cpp: (FunctionObjectImp::construct): Ditto.
              * kjs/ustring.h: Added compare function for -1/0/+1 comparison.
              * kjs/ustring.cpp: (KJS::compare): Added.
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2736 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • mjs's avatar
      Change ArgumentListNode operations to be iterative instead of · a34e69ca
      mjs authored
      	recursive. This probably fixes 3095446 (Crash in
      	KJS::ArgumentListNode::ref()) but I can't reproduce it myself so
      	I'm not 100% sure. I think the original bug was a stack overflow
      	and this change would remove that possibility.
              * kjs/nodes.cpp:
              (ArgumentListNode::ref): Make iterative.
              (ArgumentListNode::deref): Make iterative.
              (ArgumentListNode::evaluateList): Make iterative.
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2735 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • rjw's avatar
      Fixed 3103004. REGRESSION: word-spacing/letter-spacing applied when they shouldn't be · e01e824b
      rjw authored
              This was actually NOT a regression.  khtml wasn't recognizing the 'normal' value for the
              word and letter spacing properties.  So it incorrectly substituted a -1 spacing.
              * khtml/css/cssstyleselector.cpp:
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2734 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • hyatt's avatar
      Fix render_body's repaint method. · b7bce452
      hyatt authored
              * khtml/rendering/render_body.cpp:
              * khtml/rendering/render_body.h:
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2733 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • rjw's avatar
      Check for usesBackForwardList was excluding all load types. · f8dec244
      rjw authored
              * WebView.subproj/WebFramePrivate.m:
              (-[WebFrame _transitionToCommitted]):
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2732 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • cblu's avatar
      Fixed: 3098767 - REGRESSION: standalone quicktime content just shows blank window · 81cc7651
      cblu authored
      	Check the instance not the class for the class type.
              * Plugins.subproj/WebNetscapePluginRepresentation.m:
              (-[WebNetscapePluginRepresentation receivedData:withDataSource:]):
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2731 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • kocienda's avatar
      WebKit: · 357eb7e0
      kocienda authored
      	Added feature to import bookmarks/favorites from MSIE. This
      	fixes Radar 2961230 (import Mac IE bookmarks).
              * Bookmarks.subproj/WebBookmarkImporter.h: Added.
              * Bookmarks.subproj/WebBookmarkImporter.m: Added.
              (-[WebBookmarkImporter initWithPath:group:]):
              (-[WebBookmarkImporter topBookmark]):
              (-[WebBookmarkImporter error]):
              (-[WebBookmarkImporter dealloc]):
              * Bookmarks.subproj/WebBookmarkList.m:
              (-[WebBookmarkList init]): Create the bookmark list object. This
      	was not created with the -init initializer, causing bookmark lists
      	to fail.
              * English.lproj/Localizable.strings:
              * English.lproj/StringsNotToBeLocalized.txt:
              * WebKit.exp:
              * WebKit.pbproj/project.pbxproj:
      	Added calls in Debug menu to import IE Favorites.
      	No doubt that this code will move somewhere else once
      	we figure out the interaction design for importing
              * Debug/DebugUtilities.m:
              (-[DebugUtilities createDebugMenu]):
              (-[BrowserDocument importInteretExplorerBookmarks:]):
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2730 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • mjs's avatar
      - fixed 3103711 - Alexander does not complete JavaScript iBench · c1e46f02
      mjs authored
              * khtml/ecma/kjs_html.cpp:
              (KJS::HTMLDocument::putValue): Resolve URL for location property
      	relative to the base URL of the frame where the JavaScript is
      	executing, not the target frame.
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2729 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • hyatt's avatar
      Yay! Found the main cause of :hover feedback being so · 95e28653
      hyatt authored
      	slow on links.  The diff function for styles was
      	considering a color change as a containing block layout
      	instead of just a repaint.  With this fix, we will no
      	longer lay out when you hover over a link whose color
      	is different (remember that laying out causes the whole
      	page to repaint).
      	There's still some work to do to optimize this further,
      	since we do repaint the containing block still, but this
      	should be good enough for beta.
              * khtml/rendering/render_style.cpp:
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2728 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • hyatt's avatar
      Don't use the immediate painting code for now. Revert · 808edc9c
      hyatt authored
      	to non-immediate code.  Using the non-immediate path I
      	have managed to fix all the regressions except for
      	mrskin losing updates in its marquee (this works only with
      	immediate painting) and the general scrollbar regression
      	(which I just don't understand).
              * khtml/html/html_documentimpl.cpp:
              * khtml/rendering/render_object.cpp:
              * khtml/rendering/render_root.cpp:
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2727 268f45cc-cd09-0410-ab3c-d52691b4dbfc
  2. 17 Nov, 2002 13 commits
    • trey's avatar
      Code cleanup to make some internal methods return autoreleased objects. · 00178efe
      trey authored
              * WebView.subproj/WebFramePrivate.m:
              (-[WebFrame _addBackForwardItemClippedAtTarget:]):
              (-[WebFrame _createItem]):
              (-[WebFrame _createItemTreeWithTargetFrame:clippedAtTarget:]):
              (-[WebFrame _transitionToCommitted]):
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2726 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • trey's avatar
      Fixed 3102076 - REGRESSION: infinite recursion involving bridge end · c3ec7092
      trey authored
      	Fixed 3100929 - REGRESSION: serverRedirectedForDataSource: not sent on server redirects
      	Fixed 3103381 - REGRESSION: Going back from anchor doesn't restore scroll position
              * WebView.subproj/WebBaseResourceHandleDelegate.m:
              (-[WebBaseResourceHandleDelegate handle:willSendRequest:]):  Return the copy of the
      	request that we make instead of the original.
              * WebView.subproj/WebMainResourceClient.m:
              (-[WebMainResourceClient handle:willSendRequest:]):  Call super after we make our mods
      	to the request, so the copy that super makes for us includes those mods.
              * WebView.subproj/WebFramePrivate.h:  Added new state, WebFrameStateCompleting.
              * WebView.subproj/WebFramePrivate.m:
              (-[WebFrame _transitionToLayoutAcceptable]):  Completing state is a NOP.
              (-[WebFrame _transitionToCommitted]):  Completing state is a NOP.
              (-[WebFrame _isLoadComplete]):  Go to Completing state before calling [bridge end].
      	Go to Completed state afterwards, only if no new loads started in the meantime.
              (-[WebFrame _loadItem:fromItem:withLoadType:]):  When doing anchor nav to get to the
      	item, save and restore scroll state, and set the current item.
              (-[WebFrame _continueFragmentScrollAfterNavigationPolicy:request:]):
      	Reorder the steps so the right scroll state gets saved to the right place.  Also, don't
      	add a backForward item if we're doing a client redirect to an anchor.
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2725 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • hyatt's avatar
      More refinements to immediate painting code. Still doesn't · 0394b747
      hyatt authored
      	fix the scrollbar problem though. :(
              * khtml/html/html_documentimpl.cpp:
              * khtml/khtmlview.cpp:
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2724 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • hyatt's avatar
      Make immediate repainting of the root element more robust. · 177a8b90
      hyatt authored
              * khtml/html/html_documentimpl.cpp:
              * khtml/khtmlview.cpp:
              * khtml/khtmlview.h:
              * khtml/rendering/render_root.cpp:
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2723 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • hyatt's avatar
      Back out my previous fix. I have a better one that is · 1ea3cc0f
      hyatt authored
      	confined to WebCore.
              * WebView.subproj/WebFramePrivate.m:
              (-[WebFrame _isLoadComplete]):
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2722 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • hyatt's avatar
      The needsdisplay is necessary even for HTML documents. · bb86b999
      hyatt authored
      	I over-optimized here.  Fixes the directory.apple.com
      	regression and the santa clara library regression.
              * WebView.subproj/WebFramePrivate.m:
              (-[WebFrame _isLoadComplete]):
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2721 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • darin's avatar
      WebCore: · b1961817
      darin authored
      	- moved all NSURL and CFURL use out of WebCore
              * kwq/KWQKURL.h: Remove getNSURL() and getNSURLFromString().
              * kwq/KWQKURL.mm: Ditto.
              * kwq/KWQKCookieJar.mm:
              (KWQKCookieJar::cookie): Pass an NSString instead of an NSURL.
              (KWQKCookieJar::setCookie): Ditto.
              * kwq/KWQKHTMLPart.mm:
              (KWQKHTMLPart::openURL): Ditto.
              (KWQKHTMLPart::openURLRequest): Ditto.
              (KWQKHTMLPart::submitForm): Ditto.
              (KWQKHTMLPart::urlSelected): Ditto.
              (KWQKHTMLPart::createPart): Ditto.
              (KWQKHTMLPart::redirectionTimerStartedOrStopped): Ditto.
              (KWQKHTMLPart::userAgent): Ditto.
              (KWQKHTMLPart::requestedURLString): Receive an NSString instead of an NSURL.
              * kwq/KWQKHTMLPartBrowserExtension.mm:
              (KHTMLPartBrowserExtension::createNewWindow): Pass an NSString instead of an NSURL.
              (KHTMLPartBrowserExtension::setIconURL): Ditto.
              (KHTMLPartBrowserExtension::setTypedIconURL): Ditto.
              * kwq/KWQKJavaAppletWidget.h: Remove use of NSURL, just use QString instead.
              * kwq/KWQKJavaAppletWidget.mm:
              (KJavaAppletWidget::KJavaAppletWidget): No need to set the QString to nil.
              (KJavaAppletWidget::~KJavaAppletWidget): No need to release the QString.
              (KJavaAppletWidget::showApplet): Pass an NSString instead of an NSURL.
              * kwq/KWQLoader.mm:
              (KWQServeRequest): Pass an NSString instead of an NSURL.
              (KWQCheckCacheObjectStatus): Ditto.
              * kwq/WebCoreBridge.h: Change all methods to use NSString instead of NSURL.
              * kwq/WebCoreBridge.mm:
              (-[WebCoreBridge openURL:reload:headers:]): Receive an NSString instead of an NSURL.
              (-[WebCoreBridge completeURLForDOMString:]): Return an NSString instead of an NSURL.
              (-[WebCoreBridge elementAtPoint:]): WebCoreElementLinkURL and WebCoreElementImageURL
      	are now strings.
              (-[WebCoreBridge URL]): Return an NSString instead of an NSURL.
              * kwq/WebCoreCookieAdapter.h: Change all methods to use NSString instead of NSURL.
              * kwq/WebCoreCookieAdapter.m:
              (-[WebCoreCookieAdapter cookiesForURL:]): Ditto.
              (-[WebCoreCookieAdapter setCookies:forURL:policyBaseURL:]): Ditto.
              * khtml/rendering/render_applet.cpp: (RenderApplet::processArguments):
      	Tweaked this file a tiny bit, mainly because header dependencies weren't working
      	and we need this file to be recompiled due to inline function changes.
      	- update for change to WebCore API so it never uses NSURL
              * WebCoreSupport.subproj/WebBridge.m:
              (-[WebBridge createWindowWithURL:frameName:]): Take NSString instead of NSURL.
              (-[WebBridge startLoadingResource:withURL:]): Ditto.
              (-[WebBridge objectLoadedFromCacheWithURL:response:size:]): Ditto.
              (-[WebBridge reportClientRedirectToURL:delay:fireDate:]): Ditto.
              (-[WebBridge setIconURL:]): Ditto.
              (-[WebBridge setIconURL:withType:]): Ditto.
              (-[WebBridge loadURL:reload:triggeringEvent:isFormSubmission:]): Ditto.
              (-[WebBridge postWithURL:data:contentType:triggeringEvent:]): Ditto.
              (-[WebBridge createChildFrameNamed:withURL:renderPart:allowsScrolling:marginWidth:marginHeight:]): Ditto.
              (-[WebBridge userAgentForURL:]): Ditto.
              (-[WebBridge requestedURL]): Return NSString instead of NSURL.
              (-[WebBridge viewForPluginWithURL:attributes:baseURL:MIMEType:]): Take NSString instead of NSURL.
              (-[WebBridge viewForJavaAppletWithFrame:attributes:baseURL:]): Take NSString instead of NSURL.
              * WebCoreSupport.subproj/WebCookieAdapter.m:
              (-[WebCookieAdapter cookiesForURL:]): Take NSString instead of NSURL.
              (-[WebCookieAdapter setCookies:forURL:policyBaseURL:]): Ditto.
              * WebView.subproj/WebDataSourcePrivate.m:
              (-[WebDataSource _commitIfReady]): Pass NSString instead of NSURL.
              * WebView.subproj/WebFramePrivate.m:
              (-[WebFrame _continueFragmentScrollAfterNavigationPolicy:request:]): Pass NSString instead of NSURL
              (-[WebFrame _loadURL:loadType:triggeringEvent:isFormSubmission:]): Turn bridge URL into NSURL so we
      	can call _web_URLByRemovingFragment on it.
              * WebView.subproj/WebHTMLViewPrivate.m:
              (-[WebHTMLView _elementAtPoint:]): Convert NSString to NSURL when making the WebKit element dictionary.
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2720 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • hyatt's avatar
      Add support for specifying whether repaint should be immediate or · 7d5a4793
      hyatt authored
      	not.  Default to async for now to maintain current behavior.
      	For clipped object repainting (e.g., livepage.apple.com and mrskin)
      	switch to immediate repainting, since this makes the marquees
      	faster, stops dropping updates, and keeps the hover effects and
      	painting on the rest of the page fast (since the marquee rect doesn't
      	get unioned with links or images that you roll over).
              * khtml/rendering/render_box.cpp:
              * khtml/rendering/render_box.h:
              * khtml/rendering/render_html.cpp:
              * khtml/rendering/render_html.h:
              * khtml/rendering/render_object.cpp:
              * khtml/rendering/render_object.h:
              * khtml/rendering/render_root.cpp:
              * khtml/rendering/render_root.h:
              * khtml/rendering/render_table.cpp:
              * khtml/rendering/render_table.h:
              * khtml/rendering/render_text.cpp:
              * khtml/rendering/render_text.h:
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2719 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • mjs's avatar
      WebCore: · bb798fe2
      mjs authored
      	- fixed 2949193 - implement onKeyDown, onKeyPress, and onKeyUp event handlers
              * khtml/html/html_formimpl.cpp:
              * kwq/KWQEvent.h:
              * kwq/KWQEvent.mm:
              (QKeyEvent::QKeyEvent): Implemented.
              (QKeyEvent::key): Likewise.
              (QKeyEvent::state): Likewise.
              (QKeyEvent::accept): Likewise.
              (QKeyEvent::ignore): Likewise.
              (QKeyEvent::isAutoRepeat): Likewise.
              (QKeyEvent::text): Likewise.
              (QKeyEvent::ascii): Likewise.
              (QKeyEvent::count): Likewise.
              (QKeyEvent::isAccepted): Likewise.
              * kwq/KWQKHTMLPart.h:
              * kwq/KWQKHTMLPart.mm:
              (KWQKHTMLPart::keyEvent): Translate the event to a QEvent and send
      	it through the DOM event mechanism.
              * kwq/WebCoreBridge.h:
              * kwq/WebCoreBridge.mm:
              (-[WebCoreBridge interceptKeyEvent:toView:]): Pass event to the
      	KWQ part.
      	* kwq/WebCoreBridgePrivate.h: Added.
              * WebCore.pbproj/project.pbxproj: Added new file.
      	- fixed 2949193 - implement onKeyDown, onKeyPress, and onKeyUp event handlers
              * WebView.subproj/WebHTMLViewPrivate.h:
              * WebView.subproj/WebHTMLViewPrivate.m:
              (+[WebHTMLView load]): Pose as NSWindow too now.
              (-[WebHTMLView _interceptKeyEvent:toView:]): Pass event to WebCore and
      	see if WebCore wants to block it.
              (-[WebNSWindow sendEvent:]): For all key events that would go to a
      	subview of a WebHTMLView, let the WebHTMLView take a first crack
      	at it.
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2718 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • hyatt's avatar
      Don't discard linefeeds after elements with no close tag · dd93668d
      hyatt authored
      	(e.g., <img> or <input>).
              * khtml/html/htmltokenizer.cpp:
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2717 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • hyatt's avatar
      Fix for slashdot table mispositioning in the upper right. · bb3e43a4
      hyatt authored
      	Now that we fixed <td> to set to -konq-center when align="center"
      	is specified, the style rules are no longer necessary in
      	html4.css (and are in fact harmful).
              * khtml/css/html4.css:
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2716 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • hyatt's avatar
      Fix for 3038168. Duh. Remember to be case-insensitive when · 047a8b3b
      hyatt authored
      	checking the type of the <style> attribute.
              * khtml/html/html_headimpl.cpp:
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2715 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    • hyatt's avatar
      This is a fix for bug 3058470. The bug was that replaced elements · c84035d7
      hyatt authored
      	were computing their min and max widths wrong.  Specifically they
      	were not adding in borders and padding!  This means that any image
      	with a border around it would potentially break to another line
      	too early.
      	Once I fixed this for replaced elements I then went in and
      	fixed form controls.  I added two APPLE_CHANGES features for
      	our form controls.  First I overrode the border/padding functions
      	to always return 0 for all form controls, since no Aqua control
      	honors border/padding (nor should it).  This fixes the menulist
      	at the top of wired.com.
      	Second I made sure that certain controls will ignore CSS-specified
      	heights and always use their intrinsic heights.  These include
      	<select size=1> (combobox), single line text fields, and buttons.
      	I allow listboxes and textareas to honor height.
              * khtml/rendering/render_box.cpp:
              * khtml/rendering/render_form.h:
              * khtml/rendering/render_image.h:
              * khtml/rendering/render_object.h:
              * khtml/rendering/render_replaced.cpp:
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@2714 268f45cc-cd09-0410-ab3c-d52691b4dbfc
  3. 16 Nov, 2002 3 commits