1. 30 Dec, 2002 7 commits
    • hyatt's avatar
      Fix for 3129016, make sure that zero-height tables and lists · 5873ea08
      hyatt authored
      	will be skipped in a search for floats that occur before
      	them.  Makes usatoday work.
      
              Reviewed by gramps
      
              * khtml/rendering/render_flow.cpp:
              (RenderFlow::clearFloats):
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3214 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      5873ea08
    • darin's avatar
      Reviewed by Maciej. · 58521793
      darin authored
      	- fixed 3135642 -- ad frames are missing at channels.netscape.com
      
              * khtml/rendering/render_frames.cpp: (RenderPartObject::updateWidget): Updated to use the
      	same rule as Mozilla: Empty URL for an iframe means "about:blank".
      
              * khtml/ecma/kjs_window.cpp: (Window::isSafeScript): Loosen things so that it's safe for
      	a frame to manipulate an empty child frame, as well as an empty opened window. Also remove
      	the useless check for an empty document -- it never happens in practice.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3213 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      58521793
    • trey's avatar
      3135779 - REGRESSION: reproducible assertion failure, going back from ~orubin... · c4784143
      trey authored
      	3135779 - REGRESSION: reproducible assertion failure, going back from ~orubin to the main spies.com page	
      	3136218 - REGRESSION: Assertion failure in _restoreScrollPosition running browser buster
      
      	Both were caused by subtle interactions between new code for short-circuiting loading of
      	blank pages, and the WebHistoryItem manipulations we do for b/f and reload of child frames.
      
              Reviewed by Maciej
      
              * WebView.subproj/WebFramePrivate.m:
              (-[WebFrame _loadURL:intoChild:]):  Setup provisional or current item before jumping into
      	the real work of loading the URL.
              (-[WebFrame _restoreScrollPosition]):  Tweaked to make the assertion message clearer.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3212 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      c4784143
    • kocienda's avatar
      Tests: · 40efe749
      kocienda authored
              Reviewed by Darin and Gramps
      
      	Added some more test cases as part of the work to fix this bug:
      
      	Radar 3136795 (type URL with ../ in it, get assertion)
      
      	I also notice an unrelated change in ifnsurlextensions-test.chk where
      	leading whitespace in a URL is now getting removed correctly. Bonus.
      
              * TestURLs.h
              * WebFoundation-Misc/ifnsurlextensions-test.chk
              * WebFoundation-Misc/ifnsurlextensions-test.m:
              (main)
      
      WebFoundation:
      
              Reviewed by Darin and Gramps
      
              Radar 3136795 (type URL with ../ in it, get assertion)
      
              The code to remove relative portions from URLs was eating leading slashes on
              URL paths. This caused absolute paths with relative portions to get mangled,
              resulting in a URL that the engine could not use. Now, leading slashes are no
              longer removed from URL paths, resulting in usuable URLs.
      
              * Misc.subproj/WebNSURLExtras.m:
              (pathRemovingDots)
              (-[NSURL _web_URLComponents])
      
      WebCore:
      
              Reviewed by Darin and Gramps
      
      	Fix for this bug:
      
      	Radar 3136795 (type URL with ../ in it, get assertion)
      
      	The code to remove relative portions from URLs was eating leading slashes on
      	URL paths. This caused absolute paths with relative portions to get mangled,
      	resulting in a URL that the engine could not use. Now, leading slashes are no
      	longer removed from URL paths, resulting in usuable URLs.
      
              * kwq/KWQKURL.mm:
              (copyPathRemovingDots)
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3211 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      40efe749
    • darin's avatar
      Reviewed by Don and Ken. · 0a3771cb
      darin authored
      	- fixed 3136797 -- crash when Adobe SVG Viewer plug-in puts up modal dialog
      
              * Plugins.subproj/WebBaseNetscapePluginView.h: Add inSetWindow boolean.
              * Plugins.subproj/WebBaseNetscapePluginView.m:
              (-[WebBaseNetscapePluginView sendEvent:]): Don't send any events to a plug-in while it's
      	inside NPP_SetWindow. We don't want to implement more general reentrancy protection, because
      	it could cause trouble for plugins that can handle it correctly, but it's unlikely that any
      	legitimate use would require reentrant calls while inside NPP_SetWindow, and that's the case
      	that crashes for the SVG viewer plug-in when it presents its registration dialog.
              (-[WebBaseNetscapePluginView setWindow]): Set boolean.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3210 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      0a3771cb
    • darin's avatar
      WebCore: · 477551a4
      darin authored
              Reviewed by Don.
      
      	- fixed 3103287 -- body of page not rendered (page uses JavaScript trick to be both a frameset and a frame)
      
      	Our self-reference protection prevented this page from working.
      	To fix the bug, I did three things:
      
         	    1) fixed the hole in the WebCore self-reference protection, so we could remove the WebKit one
                     (the hole was that it didn't ignore #xxx suffixes)
       	    2) moved the self-reference protection into a single function so it wasn't copied and pasted all over
      	    3) allowed one level of self-reference; still enough to prevent infinite recursion
      
      	Then I tested with the site from bug 3103287 and the one from bug 3083732 to make sure that was still OK.
      
              * khtml/html/html_baseimpl.h: Added isURLAllowed().
              * khtml/html/html_baseimpl.cpp:
              (HTMLFrameElementImpl::isURLAllowed): Added. Does the self-reference check and the "local only" check.
      	This logic was repeated in three different places before. Note that since we don't use the "local only"
              feature, it's not too risky to slightly change how it works.
              (HTMLFrameElementImpl::updateForNewURL): Use isURLAllowed.
              (HTMLFrameElementImpl::attach): Ditto.
              (HTMLIFrameElementImpl::attach): Ditto.
      
      WebKit:
      
              Reviewed by Don.
      
      	- fixed 3103287 -- body of page not rendered (page uses JavaScript trick to be both a frameset and a frame)
      
      	Our self-reference checks prevented this page from working.
      	I just removed the WebKit one, since it was really just working around a problem with the WebCore
      	one that I fixed (didn't ignore #xxx suffixes).
      
              * WebCoreSupport.subproj/WebBridge.m:
              (-[WebBridge createChildFrameNamed:withURL:renderPart:allowsScrolling:marginWidth:marginHeight:]):
      	Remove self-reference check.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3209 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      477551a4
    • darin's avatar
      Reviewed by Don. · c5764f8d
      darin authored
      	- fixed 3130566 -- REGRESSION: list item numbers are overlapping the text of the list items
      
      	Moved the hack that handles list items that are not in lists out of the parser and into
      	the DOM and Render tree code itself. This fixes the issue where the parser doesn't know which
      	will be the parent node of the list item, and other problems with the old hack.
      
              * khtml/html/html_listimpl.cpp: (HTMLLIElementImpl::attach): Find the parent list (OL or UL).
      	If there is no parent list, set a "not in list" flag on the render object so it knows to render
      	"inside" no matter what the style says. Also use this parent and a safer check to make the
      	start attribute work even when there is an intervening DOM element in the hierarchy.
      
              * khtml/html/htmlparser.cpp: (KHTMLParser::getElement): Remove the code that tries to set
      	the style of an LI based on the current node at the time it's parsed, because the current node
      	is now often not the OL or the UL due to our DTD fix to allow other nodes in lists.
      
              * khtml/rendering/render_list.h: Add isInside() to RenderListMarker, setNotInList(), notInList(),
      	and _notInList to RenderListItem.
      
              * khtml/rendering/render_list.cpp:
              (RenderListItem::RenderListItem): Initialize _notInList to false.
              (RenderListMarker::paintObject): Use isInside() instead of checking listStylePosition().
              (RenderListMarker::calcMinMaxWidth): Ditto.
              (RenderListMarker::baselinePosition): Ditto.
              (RenderListMarker::isInside): Return true if we are not in a list, or if "inside" style is requested.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3208 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      c5764f8d
  2. 29 Dec, 2002 8 commits
    • darin's avatar
      Reviewed by Don. · bd2f5f11
      darin authored
      	- fixed 3134346 -- country menu at www.apple.com does not work when you go back to that page
      
              * kwq/KWQKHTMLPart.mm: (KWQKHTMLPart::openURLFromPageCache): Add call to checkCompleted().
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3207 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      bd2f5f11
    • darin's avatar
      Reviewed by Don. · 0c35e090
      darin authored
      	- fixed 3136801 -- scrolling a page that contains a QuickTime movie leaves garbage behind
      
              * Plugins.subproj/WebBaseNetscapePluginView.m:
              (-[WebBaseNetscapePluginView tellQuickTimeToChill]): Added. Calls a QuickDraw SPI
      	CallDrawingNotifications to let QuickTime know it should take a nature break.
              (-[WebBaseNetscapePluginView viewWillMoveToWindow:]): Call tellQuickTimeToChill.
              (-[WebBaseNetscapePluginView viewHasMoved:]): Call tellQuickTimeToChill.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3206 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      0c35e090
    • darin's avatar
      Reviewed by Maciej. · 45f34810
      darin authored
      	- fixed 3136391 -- PianoNanny.com screens turn to garbage when resized
      
      	Turns out browsers pass open calls to the window object if they show up at the document
      	with parameters. We now emulate that too.
      
              * khtml/ecma/kjs_html.cpp: (KJS::HTMLDocFunction::tryCall): If we get an open call with
      	parameters, pass it on to the window.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3205 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      45f34810
    • darin's avatar
      Reviewed by Don. · 73bc007f
      darin authored
      	- follow-on to my fix for 3125877 that fixes a crash I observed when a plug-in fails to NPP_New
      
              I filed bug 3136870 about the fact that we don't do a good job reporting the error.
      
              * Plugins.subproj/WebBaseNetscapePluginView.m:
              (-[WebBaseNetscapePluginView sendEvent:]): Return quietly if this is called when
      	the plug-in is not started rather than asserting (and doing bad things on Deployment).
              (-[WebBaseNetscapePluginView setWindow]): Ditto.
              (-[WebBaseNetscapePluginView viewHasMoved:]): Just call setWindow since it now checks isStarted.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3204 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      73bc007f
    • darin's avatar
      Reviewed by Don. · c5135a70
      darin authored
      	- follow-on to my fix for 3125877 that fixes a crash I observed when a plug-in fails to NPP_New
      
              * Plugins.subproj/WebBaseNetscapePluginView.m:
              (-[WebBaseNetscapePluginView sendEvent:]): Return quietly if this is called when
      	the plug-in is not started rather than asserting (and doing bad things on Deployment).
              (-[WebBaseNetscapePluginView setWindow]): Ditto.
              (-[WebBaseNetscapePluginView viewHasMoved:]): Just call setWindow since it now checks isStarted.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3203 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      c5135a70
    • darin's avatar
      Reviewed by Don. · f1002bfe
      darin authored
      	- fixed 3120630 -- spacebar scrolls the page as well as pausing the QuickTime movie
      
      	Imitate Mozilla and OmniWeb by not propagating keyboard events after passing them to
      	plug-ins regardless of what the plug-in returns, rather than imitating MacIE, which
              looks at the return value from NPP_HandleEvent.
      
              * Plugins.subproj/WebBaseNetscapePluginView.m:
              (-[WebBaseNetscapePluginView keyUp:]): Remove call to super, add comment.
              (-[WebBaseNetscapePluginView keyDown:]): Ditto.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3202 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      f1002bfe
    • darin's avatar
      Reviewed by Ken and Don. · 05df9e7f
      darin authored
      	- fixed 3136120 -- much content missing at www.olympic.org
      
      	This site gives modern CSS to "Internet Explorer" or "Netscape 6". Since we are neither of
      	those, we need to pretend to be one or the other. So we add olympic.org to our MacIE spoofing list.
      
              * WebView.subproj/WebUserAgentSpoofTable.gperf: Add olympic.org to the list of pages that give us better
      	CSS if we claim to be Internet Explorer.
              * WebView.subproj/WebUserAgentSpoofTable.c: Regenerated.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3201 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      05df9e7f
    • darin's avatar
      Reviewed by Ken. · 65e464a5
      darin authored
      	- fixed 3134560 -- Lord of the Rings desktops don't load
      
              * khtml/ecma/kjs_window.cpp: (WindowFunc::tryCall): Removed the APPLE_CHANGES here.
      	The comment claims that we don't need the code because "JavaScript will do the work
      	below on demand", but that's not true for setting the base URL. Also, the comment
      	claims that this "adds an extra entry to the back/forward list", but testing does not
      	bear that out.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3200 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      65e464a5
  3. 28 Dec, 2002 9 commits
    • kocienda's avatar
      Setting version to 47u · 42f65866
      kocienda authored
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3199 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      42f65866
    • kocienda's avatar
      Safari-46 build · fccdb286
      kocienda authored
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3197 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      fccdb286
    • kocienda's avatar
      Fixed cut-n-paste error I made when checking in Darin's patch for 3125877. · cb379e5a
      kocienda authored
      I mistakenly checked in his comment for 3133207. Doh!
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3196 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      cb379e5a
    • kocienda's avatar
      WebCore: · 11bb1581
      kocienda authored
              Reviewed by Gramps and Richard.
      
      	Fix for this bug:
      
      	Radar 3112233 (400 response when attaching files at mail.yahoo.com)
      
      	KHTML multipart form code called into KMimeType class to determine
      	the mime type for file attachments, however, our implementation of
      	KMimeType has only stubs which call ERROR.
      
      	I added some code into KWQKHTMLPart and WebCoreBridge so that the
      	code in html_formimpl can call through to the WebFoundation mime
      	file map we maintain.
      
              * khtml/html/html_formimpl.cpp:
              (HTMLFormElementImpl::formData)
              * kwq/KWQKHTMLPart.h:
              * kwq/KWQKHTMLPart.mm:
              (KWQKHTMLPart::mimeTypeForFileName)
              * kwq/WebCoreBridge.h:
      
      WebKit:
      
               Reviewed by Gramps and Richard
      
               Fix for this bug:
      
               Radar 3112233 (400 response when attaching files at mail.yahoo.com)
      
               I added the MIMETypeForPath method which accesses the WebFoundation mime
               file map we maintain. KHTML can now access this map by using the bridge.
      
               * WebCoreSupport.subproj/WebBridge.m:
               (-[WebBridge MIMETypeForPath:])
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3195 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      11bb1581
    • kocienda's avatar
      Corrected checkin date that I cut and pasted incorrectly. · c2318740
      kocienda authored
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3194 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      c2318740
    • kocienda's avatar
      Reviewed by Gramps and Ken · e0825e86
      kocienda authored
      	Checked in by Ken
      
      	- fixed 3133207 -- crash in HTMLFrameElementImpl updateForNewURL at home.real.com
      
      	My previous attempt to fix this dealt with the case where a RenderFrame object was
      	not yet created, but not with the case where a RenderFrame object was created without
      	a corresponding part. Somehow I overlooked this last time when I was doing testing.
      	Testing this time confirms this takes care of the remaining problem at movies.real.com.
      
              * khtml/html/html_baseimpl.cpp: (HTMLFrameElementImpl::updateForNewURL):
      	Use the same kind of requestFrame call here as in attach when there's a RenderFrame,
      	but no corresponding part. Maybe we can fix this an even more elegant way later.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3193 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      e0825e86
    • kocienda's avatar
      Reviewed by Gramps and Ken. · 69335a5d
      kocienda authored
      	Checked in by Ken.
      
      	- fixed 3134693 -- carsdirect.com crash on used car search, due to large JavaScript array
      
      	The parser was using recursion to handle many types of lists.
      	This meant that we crashed out of stack space when any of the lists were extra big.
      	I applied the same sort of fix we had already applied a while back for argument lists for
      	all the other types of lists, including the list of ElementNode that was the reason for
      	the crash reported here.
      
              * kjs/grammar.y: Removed ElisionNode altogether and just use a count.
      	Use specific node types for PropertyNameAndValueList and PropertyName.
      
              * kjs/grammar.cpp: Regenerated.
              * kjs/grammar.cpp.h: Regenerated.
              * kjs/grammar.h: Regenerated.
      
              * kjs/nodes.h: Elide "ElisionNode", changing objects to keep elision counts instead.
      	Make the ObjectLiteralNode list field be PropertyValueNode, not just Node.
      	Make PropertyValueNode fields have specific types. Add new reverse list functions, calls
      	to those functions in the constructors, and friend declarations as needed so the class
      	that holds the head of a list can reverse the list during parsing.
              * kjs/nodes.cpp:
              (ElementNode::ref): Use iteration instead of recursion. Also elide "elision".
              (ElementNode::deref): Ditto.
              (ElementNode::evaluate): Use iteration instead of recursion, taking advantage of
      	the fact that the linked list is reversed. Also use the elision count rather than
      	an elision list.
              (ArrayNode::reverseElementList): Reverse the list so we can iterate normally.
              (ArrayNode::ref): Elide "elision".
              (ArrayNode::deref): Ditto.
              (ArrayNode::evaluate): Use elision count instead of elision list.
              (ObjectLiteralNode::reverseList): Reverse the list so we can iterate normally.
              (PropertyValueNode::ref): Use iteration instead of recursion.
              (PropertyValueNode::deref): Use iteration instead of recursion.
              (PropertyValueNode::evaluate): Use iteration instead of recursion, taking advantage
      	of the fact that the linked list is reversed.
              (ArgumentListNode::ref): Change code to match the other similar cases we had to revise.
              (ArgumentListNode::deref): Ditto.
              (ArgumentListNode::evaluateList): Ditto.
              (ArgumentsNode::reverseList): Ditto.
              (VarDeclListNode::ref): Use iteration instead of recursion.
              (VarDeclListNode::deref): Ditto.
              (VarDeclListNode::evaluate): Use iteration instead of recursion, taking advantage
      	of the fact that the linked list is reversed.
              (VarDeclListNode::processVarDecls): Ditto.
              (VarStatementNode::reverseList): Reverse the list so we can iterate normally.
              (FunctionBodyNode::FunctionBodyNode): Use BlockNode as the base class, removing
      	most of the FunctionBodyNode class.
      
              * kjs/nodes2string.cpp:
              (ElementNode::streamTo): Update for using a count for elision, and reverse linking.
              (ArrayNode::streamTo): Update for using a count for elision.
              (PropertyValueNode::streamTo): Update for reverse linking.
              (ArgumentListNode::streamTo): Update for reverse linking. This has been wrong for
      	a while, since we added the reverse a long time ago.
              (VarDeclListNode::streamTo): Update for reverse linking.
              (ParameterNode::streamTo): Update for reverse linking.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3192 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      69335a5d
    • darin's avatar
      Reviewed by Don. · 7570c92e
      darin authored
      	- fixed 3136206 -- Can't display images in Ambrosia Software image viewer
      
      	WebKit wasn't creating the frames correctly when they weren't HTML.
      
              * WebView.subproj/WebFramePrivate.m: (-[WebFrame _makeDocumentView]): Added. Calls the
      	WebView to make the document view and then installs the new view, as WebHTMLView used to do.
              (-[WebFrame _transitionToCommitted:]): Call _makeDocumentView instead of calling WebView
      	directly. These are all the calls to _makeDocumentViewForDataSource:.
      
              * WebView.subproj/WebViewPrivate.h: Add return value to _makeDocumentViewForDataSource:.
              * WebView.subproj/WebViewPrivate.m: (-[WebView _makeDocumentViewForDataSource:]):
      	Return the newly-created view.
      
              * WebView.subproj/WebHTMLView.m: (-[WebHTMLView setDataSource:]): Removed the code here
      	since it's done by WebFrame now.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3191 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      7570c92e
    • darin's avatar
      Reviewed by Don. · ee114f40
      darin authored
      	- fixed 3136262 -- Shockwave Flash at movies.yahoo.com does not work
      
              * khtml/rendering/render_frames.cpp: (RenderPartObject::updateWidget):
      	Moved the class ID special case MIME types out of the if statement so they can be shared
      	in the case where we have an embed tag but no MIME type.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3190 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      ee114f40
  4. 26 Dec, 2002 2 commits
  5. 25 Dec, 2002 2 commits
    • darin's avatar
      Reviewed by Darin. · 5af885b5
      darin authored
      	- fixed 3133611 -- Java "ticker" applet renders badly when scrolling page
      
              * Plugins.subproj/WebBaseNetscapePluginView.m: (-[WebBaseNetscapePluginView viewHasMoved:]):
      	Call setWindow so we make a call to the plugin each time it moves, rather than just adjusting
      	the data structure we had passed to it earlier.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3187 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      5af885b5
    • rjw's avatar
      Fix for 3134543 (and widely reported dupes). Don't use render_text · 8f03ce2d
      rjw authored
              widths cache except for whitespace == PRE.  The problem was that additional
              spaces in the RenderText's string were being inappropiately measured.
              An alternate, more complex and risky, fix would account for the whitespace
              properties during scanning of the RenderText width buffer.  Considering
              the scenario for which this optimization was added (3133261), this
              simpler fix is adequate.
      
              Reviewed by Darin and Don.
      
              * khtml/rendering/render_text.cpp:
              (RenderText::computeWidths):
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3186 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      8f03ce2d
  6. 24 Dec, 2002 3 commits
    • darin's avatar
      Reviewed by Don. · 523f0cbf
      darin authored
      	- fixed 3135010 -- nil-dereference in KHTMLPart::xmlDocImpl() at channels.netscape.com
      
              * khtml/html/html_baseimpl.cpp: (HTMLFrameElementImpl::contentDocument):
              Add missing nil check. Also added lots of FIXME comments about problems I observed here.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3185 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      523f0cbf
    • darin's avatar
      Reviewed by Richard and Don. · 418f83ec
      darin authored
      	- fixed 3132192 -- HOMEPAGE: Quicktime plug in with AVI content brings Plug-ins not found panel
      
              * Plugins.subproj/WebPluginDatabase.m: (-[WebPluginDatabase pluginForKey:withEnumeratorSelector:]):
      	Lower-case the key before searching for it. This is needed for both MIME types and extensions,
      	since we want case insensitive comparison in both cases.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3184 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      418f83ec
    • darin's avatar
      Reviewed by Richard and Don. · a02ae6d8
      darin authored
      	- fixed 3133207 -- crash in HTMLFrameElementImpl updateForNewURL at home.real.com
      
      	My previous attempt to fix this dealt with the case where a RenderFrame object was
      	not yet created, but not with the case where a RenderFrame object was created without
      	a corresponding part. Somehow I overlooked this last time when I was doing testing.
      	Testing this time confirms this takes care of the remaining problem at movies.real.com.
      
              * khtml/html/html_baseimpl.cpp: (HTMLFrameElementImpl::updateForNewURL):
      	Use the same kind of requestFrame call here as in attach when there's a RenderFrame,
      	but no corresponding part. Maybe we can fix this an even more elegant way later.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3183 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      a02ae6d8
  7. 23 Dec, 2002 8 commits
    • kocienda's avatar
      Alexander 46u commit · 8724c85b
      kocienda authored
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3182 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      8724c85b
    • kocienda's avatar
      Building Alex-45 · a193bff1
      kocienda authored
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3180 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      a193bff1
    • kocienda's avatar
      Reviewed by me · 5c565391
      kocienda authored
      	Fixed ChangeLog typos in previous commit.
      
              * ChangeLog
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3179 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      5c565391
    • kocienda's avatar
      Reviewed by Darin and Gramps · 60fef03f
      kocienda authored
      	Workaround for this bug:
      
      	Radar 3134219 (MPEG-4 files don't work with the QuickTime plugin in Safari,
      	work fine in Mozilla, IE)
      
      	For beta 1, we For beta 1, when getting the MIME information for the QuickTime
      	plugin, we directly insert the information to handle MP4.
      
      	In the future, we will use the additional plugin entry points to dynamically load
      	this information from the plugin itself.
      
              * Plugins.subproj/WebNetscapePluginPackage.m:
              (-[WebNetscapePluginPackage getMIMEInformation])
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3178 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      60fef03f
    • darin's avatar
      Reviewed by Don and John. · 92285d33
      darin authored
      	- fixed 3134449 -- Date.UTC returns NaN (invalid date)
      
      	Did more testing of the date functions and made them behave like the other browsers.
      	There were three problems:
      
                  1) We did a validity check that other browsers don't do (hence the NaN).
                  2) We treated passed-in dates as local time even in Date.UTC (hence a wrong result
                     once I fixed the NaN).
                  3) The results of ToUTCString (and ToGMTString) weren't formatted quite the same
      	       as other browsers.
      
      	Also found a couple of silly but unrelated coding mistakes.
      
              * kjs/date_object.cpp:
              (timetUsingCF): Added. Has the guts of mktimeUsingCF, but without the CFGregorianDateIsValid
              check. Other browsers accept invalid dates. Also takes a time zone parameter.
              (mktimeUsingCF): Calls timetUsingCF with the current time zone.
              (timegmUsingCF): Calls timetUsingCF with the UTC time zone.
              (formatDate): Remove the includeComma flag.
              (formatDateUTCVariant): Added. For use instead of formatDate with the includeComma flag.
      	Puts the day before the month name.
              (DateProtoFuncImp::call): Use the new formatDateUTCVariant for ToGMTString and ToUTCString.
      	Without this change the date didn't match other browsers.
              (DateObjectImp::DateObjectImp): Use UTCPropertyName. Somehow I declared this and didn't use
      	it before.
              (DateObjectImp::construct): Pass -1 for is_dst literally instead of using invalidDate.
      	Changing this to invalidDate was just a mistake (although no real difference in compiled
      	code since invalidDate is just -1).
              (DateObjectFuncImp::call): Call timegm for the UTC case instead of mktime.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3177 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      92285d33
    • darin's avatar
      Reviewed by John and Don. · e72b8ee6
      darin authored
      	- fixed 3134282 -- REGRESSION: text encoding setting reverts when you go to a new location
      
              * WebView.subproj/WebFrame.m: (-[WebFrame loadRequest:]): Propagate an override encoding if there
      	was an existing data source and it had an override encoding.
              * WebView.subproj/WebFramePrivate.m: (-[WebFrame _loadRequest:triggeringAction:loadType:]): Ditto.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3176 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      e72b8ee6
    • darin's avatar
      Reviewed by John. · a5de1758
      darin authored
      	- fixed 3134558 -- REGRESSION: Lord of the Rings site problems
      
      	This regression was caused when we fixed the abcnews.com problem (3124933).
      
              * kwq/KWQKHTMLPart.mm: (KWQKHTMLPart::createEmptyDocument):
      	Despite its name, this function was never intended to overwrite an existing document with
      	an empty document. When we changed it do do the creation in a better way, we lost the
      	check that prevents it from doing anything at all if we already have a document.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3175 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      a5de1758
    • darin's avatar
      Reviewed by NOBODY (OOPS!). · 34039077
      darin authored
      	- fixed 3134558 -- REGRESSION: Lord of the Rings site problems
      
      	This regression was caused when we fixed the abcnews.com problem (3124933).
      
              * kwq/KWQKHTMLPart.mm: (KWQKHTMLPart::createEmptyDocument):
      	Despite its name, this function was never intended to overwrite an existing document with
      	an empty document. When we changed it do do the creation in a better way, we lost the
      	check that prevents it from doing anything at all if we already have a document.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3174 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      34039077
  8. 22 Dec, 2002 1 commit