Timeline



Aug 29, 2003:

8:45 PM Changeset in webkit [4920] by vicki
  • 3 edits in trunk

tree is open for Safari-97!

4:54 PM Changeset in webkit [4919]
  • 3 copies in tags/Safari-96

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

4:54 PM Changeset in webkit [4918] by vicki
  • 8 edits in trunk

Safari-96 stamp

3:04 PM Changeset in webkit [4917] by mjs
  • 4 edits in trunk/WebCore

Reviewed by Ken.

  • fixed 3397775 21 leaks from CSS parser running PLT
  • khtml/css/css_ruleimpl.cpp: (CSSMediaRuleImpl::~CSSMediaRuleImpl): Clear the parent for all contained rules. (CSSMediaRuleImpl::append): Call insertRule. (CSSMediaRuleImpl::insertRule): set parent of the rule to this media rule. (CSSRuleListImpl::append): Call insertRule to cover all the relevant additional handling, instead of adding straight to the list.
  • khtml/css/css_ruleimpl.h:
3:02 PM Changeset in webkit [4916] by rjw
  • 3 edits in trunk/WebKit

Fixed 3401334. Use IB document key when checking for reference removal on instances of WebPreferences.

Also removed _userDefaultsKeysForIB, no longer needed by IB.

Reviewed by John.

  • WebView.subproj/WebPreferences.m: (+[WebPreferences _removeReferenceForIdentifier:]): (-[WebPreferences _postPreferencesChangesNotification]):
  • WebView.subproj/WebPreferencesPrivate.h:
2:52 PM Changeset in webkit [4915] by rjw
  • 2 edits in trunk/WebKit

Fixed 3400807. Don't release state associated with the current b/f item. We shouldn't normally have page cache state associated with the current item (3401376). This fix guarantees that we won't prematurely release the page cache state for the current item.

Reviewed byJohn Sullivan.

  • History.subproj/WebBackForwardList.m: (-[WebBackForwardList _clearPageCache]):
1:18 PM Changeset in webkit [4914] by sullivan
  • 2 edits in trunk/WebCore
  • fixed 3399880 -- Repro crash when filling out a form to download Macromedia software

Reviewed by Chris

  • kwq/WebCoreBridge.mm: (-[WebCoreBridge elementForView:]): check for nil widget before dereferencing

Aug 28, 2003:

4:17 PM Changeset in webkit [4913] by rjw
  • 6 edits in trunk/WebKit

Fixed 3399736. Fixed several problems with WebView/WebPreferences
interaction.

Reviewed by Eric Seymour.

  • Misc.subproj/WebKitLogging.h:
  • Misc.subproj/WebKitLogging.m: Added debug logging.
  • WebCoreSupport.subproj/WebTextRenderer.m: (_drawGlyphs): Checkin for 3398229 below.
  • WebView.subproj/WebPreferences.m: (-[WebPreferences initWithIdentifier:]): (-[WebPreferences initWithCoder:]): (-[WebPreferences encodeWithCoder:]): (+[WebPreferences standardPreferences]): (+[WebPreferences _getInstanceForIdentifier:]): (+[WebPreferences _setInstance:forIdentifier:]): (+[WebPreferences _concatenateKeyWithIBCreatorID:]):
  • WebView.subproj/WebView.m: (-[WebView _commonInitializationWithFrameName:groupName:]): (-[WebView initWithCoder:]): (-[WebView encodeWithCoder:]):
2:24 PM Changeset in webkit [4912] by hyatt
  • 4 edits in trunk/WebCore

Fix for 3381377, fix the calcBlockMinMaxWidth function to not use the
renderobject's computed margins except when it has to. This makes
table cells with align=right not mess up and fixes margin problems
in general with floated elements and table cells.

Since XUL boxes also use a similar function and since it had a similar
error, vertical XUL boxes have been patched as well.

Reviewed by gramps

  • khtml/rendering/render_block.cpp:
  • khtml/rendering/render_flexbox.cpp:
1:14 AM Changeset in webkit [4911] by hyatt
  • 3 edits in trunk/WebCore

Fix for 3398463, a regression caused by an unintended line movement.
<br/> was no longer being treated just like <br> in HTML.

Reviewed by gramps

  • khtml/html/htmltokenizer.cpp: (HTMLTokenizer::parseTag):

Aug 27, 2003:

8:45 PM Changeset in webkit [4910] by rjw
  • 2 edits in trunk/WebKit

Fixed 3398229. When we request a font from NSFont by name we should use a case specific name. We do case insensitve comparsion, but once a match is found we should use the actual font name, not the requested name. Two layout tests are still failing, but I don't think the failures are font related.

Reviewed by Hyatt.

  • WebCoreSupport.subproj/WebTextRendererFactory.m: (-[WebTextRendererFactory fontWithFamily:traits:size:]):
7:50 PM Changeset in webkit [4909] by mjs
  • 3 edits in trunk/JavaScriptCore

Reviewed by John

  • fixed rdar://problem/3397316 - sherlock crash: KJS::Collector::allocate(unsigned long)
  • kjs/internal.cpp: (InterpreterImp::InterpreterImp): Hold the lock a bit longer, so the call to initGlobalObject is covered.
6:06 PM Changeset in webkit [4908] by vicki
  • 3 edits in trunk

tree is open for Safari-96!

5:35 PM Changeset in webkit [4907]
  • 3 copies in tags/Safari-95

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

5:35 PM Changeset in webkit [4906] by vicki
  • 8 edits in trunk

top level:

Reviewed by Chris.

  • acconfig.h: Removed. The new autoconf in Panther doesn't need it.
  • configure.in: Removed HAVE_S_ISSOCK code; none of our code needs this define and it upsets the new autoconf if we leave out acconfig.h.

Tools:

  • Scripts/create-tarball: Leave out anything with "apple-only" in the path.
  • Scripts/run-layout-tests: Print out whether each test succeeded or failed after it runs.

JavaScriptCore:

Reviewed by John.

  • fixed 3098350 -- opt. params to date methods are ignored (can't set end date in Exchange/Outlook web cal.)
  • kjs/date_object.cpp: (DateProtoFuncImp::call): Added code to handle the optional parameters. Strangely, the table of functions already had the right number of parameters listed, but the code to look at the parameter values was missing.

Tests:

  • PDFViewer/ApplicationController.m: Added missing include.
  • PDFViewer/PDFRepresentation.m: (-[PDFView layout]): Change to use the frame of the WebFrameView instead of the frame of the NSScrollView since there's noAPI to get the scroll view any more.

WebCore:

Fixed 3359408. DON'T treat unicode whitespace as whitespace when collapsing spaces.

Reviewed by David Hyatt.

  • khtml/rendering/bidi.cpp:
  • khtml/rendering/render_text.cpp: (RenderText::trimmedMinMaxWidth): (RenderText::calcMinMaxWidth): (RenderText::containsOnlyWhitespace):
  • khtml/xml/dom_stringimpl.cpp:

WebKit:

Fixed 3397235. WebView wasn't archiving useBackForwardList.

Reviewed by Eric Seymour.

  • WebView.subproj/WebView.m: (-[WebView initWithCoder:]): (-[WebView encodeWithCoder:]):

WebBrowser:

Reviewed by John.

  • fixed 3390306 -- REGRESSION (Panther): Safari browser window titles not available via Accessibility APIs
  • BrowserWindow.m: (-[BrowserWindow accessibilityTitleAttribute]): Return the title.

WebKitExamples:

  • CarbonDownloader/English.lproj/main.nib: Changed command key equivalent for Close to Cmd-W. Since this is example code and not localized, the localization freeze should not be an issue.
  • CarbonDownloader/English.lproj/main~.nib: Removed.
3:57 PM Changeset in webkit [4905] by rjw
  • 5 edits in trunk/WebCore

Fixed 3359408. DON'T treat unicode whitespace as whitespace when collapsing spaces.

Reviewed by David Hyatt.

  • khtml/rendering/bidi.cpp:
  • khtml/rendering/render_text.cpp: (RenderText::trimmedMinMaxWidth): (RenderText::calcMinMaxWidth): (RenderText::containsOnlyWhitespace):
  • khtml/xml/dom_stringimpl.cpp:
3:07 PM Changeset in webkit [4904] by rjw
  • 2 edits in trunk/WebKit

Fixed 3397235. WebView wasn't archiving useBackForwardList.

Reviewed by Eric Seymour.

  • WebView.subproj/WebView.m: (-[WebView initWithCoder:]): (-[WebView encodeWithCoder:]):
2:00 PM Changeset in webkit [4903] by hyatt
  • 2 adds in trunk/LayoutTests/fast/forms

Add inline fieldset test.

1:53 PM Changeset in webkit [4902] by hyatt
  • 4 edits in trunk/WebCore

Fix for 3396962, hang on quote.com. Fieldsets with display:inline should be treated
like inline-blocks. This matches WinIE's behavior.

Reviewed by mjs

  • khtml/rendering/render_form.cpp: (RenderFieldset::setStyle):
  • khtml/rendering/render_form.h:
11:23 AM Changeset in webkit [4901] by rjw
  • 5 edits in trunk/WebKit

Fixed 3385478. Look for an exact match for font names (using PS names) before matching on family names.

Also added logging to help debug now resolved binding problem.

Reviewed by Maciej.

  • Misc.subproj/WebKitLogging.h:
  • Misc.subproj/WebKitLogging.m:
  • WebCoreSupport.subproj/WebTextRendererFactory.m: (-[WebTextRendererFactory fontWithFamily:traits:size:]):
  • WebView.subproj/WebView.m: (-[WebView addObserver:forKeyPath:options:context:]): (-[WebView removeObserver:forKeyPath:]):

Aug 26, 2003:

11:41 PM Changeset in webkit [4900] by darin
  • 5 edits in trunk

WebCore:

Reviewed by Maciej.

  • follow-on to fix for 3310943 -- REGRESSION (Panther): textareas in forms sometimes draw blank (bugreporter)
  • kwq/KWQListBox.mm: (QListBox::QListBox): Remove call to releaseGState. This was here to work around a problem we thought was an AppKit bug, but it turns out it was caused by NSView hackery in WebKit. We now do the WebKit part in a way that does not create the problem.

WebKit:

Reviewed by Maciej.

  • fixed 3310943 -- REGRESSION (Panther): textareas in forms sometimes draw blank (bugreporter)

There are two fixes here. The more elegant and slightly faster one that requires a new AppKit,
and the less elegant one that works without AppKit support. By including both we don't have to
worry about timing of submission of WebKit vs. AppKit but we get the good, elegant fix. Later,
we can delete the less elegant fix.

  • WebView.subproj/WebClipView.m: (+[WebClipView initialize]): Set up a boolean global so we only hav to do the "does AppKit support _focusRingVisibleRect" check one time. (-[WebClipView resetAdditionalClip]): Only do the renewGState thing if we don't have the _focusRingVisibleRect method, but if we do the renewGState thing, do it to self and all descendants using _web_renewGStateDeep. (-[WebClipView setAdditionalClip:]): Ditto. (-[WebClipView visibleRect]): Only limit this based on the additional clip if we don't have the _focusRingVisibleRect method. (-[WebClipView _focusRingVisibleRect]): Override the new method. Harmless if it's an old AppKit that doesn't have the method yet. (-[NSView _web_renewGStateDeep]): Implemented this helper method. We can get rid of it once we are entirely on the new AppKit.
10:47 PM Changeset in webkit [4899] by darin
  • 3 edits in trunk/WebCore

Reviewed by Maciej.

  • fixed 3031562 -- most forms don't work in certain configurations because of a bug in KWQVariant
  • kwq/KWQVariant.mm: (QVariant::QVariant): Set the b element of the union, not the d element, in the constructor for bool.
6:23 PM Changeset in webkit [4898] by hyatt
  • 2 adds in trunk/LayoutTests/fast/block/float

Add a test for floats/clearing/margin-collapsing under the new system

6:15 PM Changeset in webkit [4897] by hyatt
  • 6 deletes in trunk/LayoutTests/fast/block/margin-collapse

Remove margin collapsing tests that are now buggy/invalid

6:09 PM Changeset in webkit [4896] by hyatt
  • 4 edits in trunk/WebCore

Fix for 3395950, better support for self-collapsing blocks. Make them obey
clear properly and also change the definition of self-collapsing to match
the CSS2.1 specification.

Reviewed by gramps

  • khtml/rendering/render_block.cpp:
  • khtml/rendering/render_block.h:
5:47 PM Changeset in webkit [4895] by hyatt
  • 2 adds in trunk/LayoutTests/fast/invalid

Add a layout test for <script/>

5:46 PM Changeset in webkit [4894] by hyatt
  • 3 edits in trunk/WebCore

Fix for 3335175, www.calstone.com renders as blank. The fix is to deliberately
introduce an error into our HTML parsing code that causes us to honor self-closing
XML-style <script/> tags. This matches Mozilla and Opera behavior (both of them
have this bug), but it does not match WinIE.

Reviewed by mjs

  • khtml/html/htmltokenizer.cpp:
4:57 PM Changeset in webkit [4893] by hyatt
  • 2 adds in trunk/LayoutTests/fast/block/basic

Add a display:none test.

4:44 PM Changeset in webkit [4892] by hyatt
  • 2 adds in trunk/LayoutTests/fast/invalid

Commit an h2/form test case.

4:43 PM Changeset in webkit [4891] by hyatt
  • 3 edits in trunk/WebCore

Fix for 3286229, popup menu misplaced. <form> needs to be allowed as a child of
<h1>-<h6> tags. All other browsers allow this.

Reviewed by darin

  • khtml/html/dtd.cpp:
4:13 PM Changeset in webkit [4890] by darin
  • 3 edits in trunk/WebCore

Reviewed by Maciej.

  • re-fixed 3366542 -- filename with non-ASCII name left out of Content-Disposition for <input type=file>
  • khtml/html/html_formimpl.cpp: (HTMLFormElementImpl::formData): Added missing # sign.
4:09 PM Changeset in webkit [4889] by darin
  • 2 edits in trunk/WebKit

Reviewed by Maciej.

  • fixed 3321247 -- window size box disappears from Help window (caused by WebKit NSView hackery)
  • WebView.subproj/WebHTMLViewPrivate.m: (-[WebHTMLView _recursiveDisplayRectIfNeededIgnoringOpacity:isVisibleRect:rectIsVisibleRectForView:topView:]), (-[WebHTMLView _recursiveDisplayAllDirtyWithLockFocus:visRect:]): Don't propagate dirty rects at all. This was causing problems because now the AppKit uses dirty regions, not dirty rects. In AppKit-722 and newer, _setDrawsDescendants: takes care of this for us so we don't have to do anything at all.
3:19 PM Changeset in webkit [4888] by hyatt
  • 3 edits in trunk/WebCore

Fix for 3394107, make sure the display mutation code doesn't mutate objects with
display: none set.

Reviewed by darin

  • khtml/css/cssstyleselector.cpp:
11:57 AM Changeset in webkit [4887] by darin
  • 2 edits in trunk/WebKit

Reviewed by John.

  • fixed 3392650 -- REGRESSION?: assertion fails trying Apple-hosted page load test while not on Apple network
  • WebView.subproj/WebFramePrivate.m: (-[WebFrame _isLoadComplete]): Stop loading the provisional data source before blowing it away, in case there are some callbacks that haven't occurred yet. It's a waste of time to try to handle those additional callbacks, and can lead to failed assertions since the data source won't be hooked up to any frame any more.

Aug 25, 2003:

4:26 PM Changeset in webkit [4886] by rjw
  • 2 edits in trunk/WebKit

Fix for 3391609. Our rounding hack wasn't correctly reflected in the selection point
code. Piggy-backed on this fix are fixes for selection of letter-spacing, word-spacing
and justified text.

Reviewed by Hyatt.

  • WebCoreSupport.subproj/WebTextRenderer.m: (initializeCharacterWidthIterator): (widthAndGlyphForSurrogate): (widthForNextCharacter): (-[WebTextRenderer _CG_floatWidthForRun:style:widths:fonts:glyphs:startGlyph:endGlyph:numGlyphs:]):
4:08 PM Changeset in webkit [4885] by sullivan
  • 2 edits in trunk/WebKit
  • fixed 3391264 -- REGRESSION (Panther): Back/Forward buttons not updating immediately after page load

The buttons are updated in response to window update notifications.
Jaguar was sending so many extra bogus notifications that it
masked the fact that we weren't ensuring that these notifications
were sent at all in the case of non-event-based interesting
changes that might affect menu items/toolbar items/etc.

Reviewed by Richard

  • WebView.subproj/WebViewPrivate.m: (-[WebView _didStartProvisionalLoadForFrame:]): call -[NSApp setWindowsNeedUpdate:YES] so window update notices will be sent (-[WebView _didCommitLoadForFrame:]): ditto (-[WebView _didFinishLoadForFrame:]): ditto (-[WebView _didFailLoadWithError:forFrame:]): ditto (-[WebView _didFailProvisionalLoadWithError:forFrame:]): ditto
Note: See TracTimeline for information about the timeline view.