Timeline


and

06/14/05:

15:50 Changeset [9398]

This commit was manufactured by cvs2svn to create tag
'WebCore-315~6'.

15:50 Changeset [9397] by vicki
  • versioning for WebCore-315.6
15:37 Changeset [9396] by hyatt

The behavior of <pre> is broken so that the newline
is not always properly being eaten after the open of the tag. In addition we were eating the first newline
after all other tags. This patch stops eating newlines after other tags and refines the behavior for pre
to catch all cases.

Reviewed by sullivan

Test cases added: fast/text/whitespace/030.html

  • khtml/html/htmltokenizer.cpp: (khtml::HTMLTokenizer::parseTag): (khtml::HTMLTokenizer::write):
11:45 Changeset [9395] by darin

Changes by Anders Carlsson.
Reviewed by me.

  • Scripts/gdb-safari: Added.
11:06 Changeset [9394] by vicki
  • merged this fix from HEAD

2005-06-13 Vicki Murley <​vicki@apple.com>

Reviewed by Darin.

No test cases added since the steps to reproduce require use of showModalDialog.

  • fix for <rdar://problem/4145910> parameters not working in showModalDialog test page (dialogWidth, dialogHeight, etc.)
  • khtml/ecma/kjs_window.cpp: (KJS::parseFeatures): add a check for colon character in the feature string
11:06 Changeset [9393] by sullivan

Changes by Devin Lane.
Reviewed by me.

  • fixed <rdar://problem/3766909> PDF viewing could use a zoom control other than the one in the context menu
  • WebView.subproj/WebPDFView.h: now implements protocol _web_WebDocumentTextSizing
  • WebView.subproj/WebPDFView.m: (-[WebPDFView _updateScalingToReflectTextSize]): new method, sets the PDF scaling from the text size multiplier (-[WebPDFView setDataSource:]): call _updateScalingToReflectTextSize (-[WebPDFView _web_textSizeMultiplierChanged]): implementation of protocol _web_WebDocumentTextSizing, calls _updateScalingToReflectTextSize
10:59 Changeset [9392] by ggaren

Updated expected.html to reflect fix to <rdar://problem/4147745>.

Reviewed by cblu.

  • tests/mozilla/expected.html:
10:36 Changeset [9391] by sullivan

Reviewed by Dave Harrison.

  • WebView.subproj/WebHTMLView.m: (-[WebHTMLView _delayedEndPrintMode:]): After discussing this with Dave, I made this method both more debugger-friendly with asserts for the cases we don't think could ever happen, and more paranoid by handling these cases in deployment builds.
09:36 Changeset [9390] by ggaren

Fixed: <rdar://problem/4147745> JavaScript discards locally defined "arguments" property

No layout tests added because this change fixes existing tests:
ecma/ExecutionContexts/10.1.6.js
ecma_3/Function/regress-94506.js
js1_4/Functions/function-001.js

Reviewed by cblu.

  • kjs/function.cpp: (KJS::ActivationImp::get): get now checks for an "arguments" property defined in the local variable object before trying to return the built-in arguments array.
  • kjs/function.h: ActivationImp::put no longer overrides ObjectImp::put
09:19 Changeset [9389] by darin
  • fixed build for Xcode 2.1
  • WebKit.pbproj/project.pbxproj: Use BUILT_PRODUCTS_DIR instead of SYMROOT to search for the WebKitSystemInterface.h file. We could re-jigger this again later, but for now this is consistent with both the .a file's location and where build-webkit puts the file.

06/13/05:

19:12 Changeset [9388] by sullivan

Reviewed by Dave Harrison and Maciej.

  • WebView.subproj/WebHTMLView.m: (-[WebHTMLView _endPrintMode]): new method, extracted from identical code in beginDocument and endDocument. This method must be called once to counterbalance the code called from knowsPageRange that turns on "printing mode". (-[WebHTMLView _delayedEndPrintMode:]): new method, called from "perform after delay". Checks whether the same print operation is still underway and, if so, delays further. Otherwise calls _endPrintMode directly. (-[WebHTMLView knowsPageRange:]): after turning on "printing mode", queue up a delayed call to _delayedEndPrintMode:. If there's an early error in the print mechanism such that beginDocument is never called, this will cleanly end "printing mode" and make the webview usable again. (-[WebHTMLView beginDocument]): cancel any delayed call to _delayedEndPrintMode:. If we get this far along in printing, then we don't need the failsafe call to _delayedEndPrintMode: that was set up in knowsPageRange:. Also, call extracted method. (-[WebHTMLView endDocument]): call extracted method
18:31 Changeset [9387]

This commit was manufactured by cvs2svn to create tag
'WebCore-415~11'.

18:31 Changeset [9386]

This commit was manufactured by cvs2svn to create branch
'Safari-Cam-branch'.

18:31 Changeset [9385]

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

18:31 Changeset [9384] by adele

Versioning for Safari-2-0-branch - WebCore-415.11

18:28 Changeset [9383]

This commit was manufactured by cvs2svn to create tag 'WebKit-412~6'.

18:28 Changeset [9382] by adele

Versioning for Safari-2-0-branch - WebKit-412.6

18:27 Changeset [9381] by adele

WebCore:

Merged new fix for <rdar://problem/4142247> from TOT to Safari-2-0-branch.

2005-06-13 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Chris Blumenberg and Adele.

With this change and the matching WebKit change we'll still stop loading the moment you click
a download link, but the unload event and detaching of event handlers will not happen early any more.

  • khtml/khtml_part.cpp: (KHTMLPart::stopLoading): Factored out from closeURL, make firing of unload and detaching of event handlers optional. (KHTMLPart::closeURL): call stopLoading(true)
  • khtml/khtml_part.h:
  • kwq/WebCoreBridge.h:
  • kwq/WebCoreBridge.mm: (-[WebCoreBridge stopLoading]): Call stopLoading(false).

WebKit:

Merged new fix for <rdar://problem/4142247> from TOT to Safari-2-0-branch.

2005-06-13 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Chris Blumenberg and Adele.

With this change and the matching WebKit change we'll still stop loading the moment you click
a download link, but the unload event and detaching of event handlers will not happen early any more.

  • WebView.subproj/WebDataSource.m: (-[WebDataSource _stopLoadingInternal]): call stopLoading on bridge instead of closeURL.
  • WebView.subproj/WebFrame.m: (-[WebFrame _transitionToCommitted:]): Revert previous attempt at fix. (-[WebFrame _continueLoadRequestAfterNavigationPolicy:formState:]): ditto (-[WebFrame stopLoading]): ditto
17:29 Changeset [9380] by vicki

Reviewed by Darin.

No test cases added since the steps to reproduce require use of showModalDialog.

  • fix for <rdar://problem/4145910> parameters not working in showModalDialog test page (dialogWidth, dialogHeight, etc.)
  • khtml/ecma/kjs_window.cpp: (KJS::parseFeatures): add a check for colon character in the feature string
17:25 Changeset [9379] by mjs

Reviewed by Chris Blumenberg and Adele.

With this change and the matching WebKit change we'll still stop loading the moment you click
a download link, but the unload event and detaching of event handlers will not happen early any more.

  • WebView.subproj/WebDataSource.m: (-[WebDataSource _stopLoadingInternal]): call stopLoading on bridge instead of closeURL.
  • WebView.subproj/WebFrame.m: (-[WebFrame _transitionToCommitted:]): Revert previous attempt at fix. (-[WebFrame _continueLoadRequestAfterNavigationPolicy:formState:]): ditto (-[WebFrame stopLoading]): ditto
17:23 Changeset [9378] by mjs

Reviewed by Chris Blumenberg and Adele.

With this change and the matching WebKit change we'll still stop loading the moment you click
a download link, but the unload event and detaching of event handlers will not happen early any more.

  • khtml/khtml_part.cpp: (KHTMLPart::stopLoading): Factored out from closeURL, make firing of unload and detaching of event handlers optional. (KHTMLPart::closeURL): call stopLoading(true)
  • khtml/khtml_part.h:
  • kwq/WebCoreBridge.h:
  • kwq/WebCoreBridge.mm: (-[WebCoreBridge stopLoading]): Call stopLoading(false).
17:08 Changeset [9377] by cpeterse

Changes by Darin.
Reviewed by me.

  • fixed problems building deployment due to recent init change
  • WebView.subproj/WebArchive.m: (-[WebArchive initWithCoder:]): Put the [super init] call and check for nil outside the exception handler.
  • WebView.subproj/WebResource.m: (-[WebResource initWithCoder:]): Ditto.
08:33 Changeset [9376] by darin

Reviewed by Maciej.

  • WebCoreSupport.subproj/WebImageData.m: Got rid of use of tabs instead of spaces throughout the file. (-[WebImageData _checkSolidColor:]): Wrap use of NSGraphicsContext with an autorelease pool. (-[WebImageData _fillSolidColorInRect:compositeOperation:context:]): Ditto. (-[WebImageData tileInRect:fromPoint:context:]): Ditto. (-[WebImageData _PDFDrawFromRect:toRect:operation:alpha:flipped:context:]): Ditto.
00:27 Changeset [9375] by mjs

Patch from Andrew Wellington <​proton@wiretapped.net>
Reviewed by me.

No test cases addded since the repro steps require use of tabbed browsing.

  • khtml/khtml_part.cpp: (KHTMLPart::handleFallbackContent): check for nil

06/12/05:

20:28 Changeset [9374] by darin

Changes by Nick Zitzmann.
Reviewed by me.

  • fixed init methods that don't handle return values from the init methods they call
  • WebView.subproj/WebArchive.m: (-[WebArchive init]): Use value returned by init, check it for nil too. (-[WebArchive initWithMainResource:subresources:subframeArchives:]): Ditto. (-[WebArchive _initWithPropertyList:]): Ditto. (-[WebArchive initWithCoder:]): Ditto.
  • WebView.subproj/WebClipView.m: (-[WebClipView initWithFrame:]): Ditto.
  • WebView.subproj/WebDebugDOMNode.m: (-[WebDebugDOMNode initWithName:value:source:children:]): Ditto.
  • WebView.subproj/WebFrame.m: (-[WebFormState initWithForm:values:sourceFrame:]): Ditto. (-[WebFrame initWithName:webFrameView:webView:]): Ditto.
  • WebView.subproj/WebFrameView.m: (-[WebFrameView initWithFrame:]): Ditto.
  • WebView.subproj/WebHTMLRepresentation.m: (-[WebHTMLRepresentation init]): Ditto.
  • WebView.subproj/WebHTMLView.m: (-[WebHTMLView initWithFrame:]): Ditto. (-[WebTextCompleteController initWithHTMLView:]): Ditto.
  • WebView.subproj/WebImageView.m: (-[WebImageView initWithFrame:]): Ditto.
  • WebView.subproj/WebPreferences.m: (-[WebPreferences initWithIdentifier:]): Ditto.
  • WebView.subproj/WebRenderNode.m: (-[WebRenderNode initWithName:position:rect:view:children:]): Ditto.
  • WebView.subproj/WebResource.m: (-[WebResource init]): Ditto. (-[WebResource initWithCoder:]): Ditto.
  • WebView.subproj/WebView.m: (-[WebViewPrivate init]): Call super init. (-[_WebSafeForwarder initWithTarget:defaultTarget:templateClass:]): Use value returned by init, check it for nil too. (-[WebView initWithFrame:]): Ditto.
20:26 Changeset [9373] by darin

Changes by Stuart Morgan.
Reviewed by me.

  • Scripts/cvs-abandon: Use cwd instead of pwd.
  • Scripts/cvs-apply: Ditto.
  • Scripts/cvs-create-patch: Ditto.
  • Scripts/cvs-unapply: Ditto.
  • Scripts/run-webkit-tests: Ditto.
  • Scripts/webkitdirs.pm: Ditto. Also improve handling when there's no "Configuration" file.
20:12 Changeset [9372] by darin
  • Scripts/cvs-apply: Handle case of an empty patch better.
  • Scripts/cvs-unapply: Ditto.
19:45 Changeset [9371] by darin

Changes by Michael Gaiman.
Reviewed by me.

  • Scripts/webkitdirs.pm: Chomp off the result of pwd, and don't die when no Configuration file.
17:55 Changeset [9370] by darin
  • added expected test results for new tests
  • layout-tests/fast/selectors/159-expected.txt: Added.
  • layout-tests/fast/selectors/177a-expected.txt: Added.
  • layout-tests/fast/selectors/177b-expected.txt: Added.
17:40 Changeset [9369]

This commit was manufactured by cvs2svn to create tag 'WebKit-412~5'.

17:40 Changeset [9368] by adele

Versioning for Safari-2-0-branch - WebKit-412.5

17:10 Changeset [9367] by darin
  • Scripts/update-webkit: Make this script work when called from any directory.
15:48 Changeset [9366] by darin

Reviewed by Maciej.
Includes changes by Stuart Morgan as well as my own.

  • Scripts/build-dumprendertree: Call the new setConfiguration function, and use XcodeOptions instead of symrootXcodeOptions to set the -buildstyle option.
  • Scripts/build-webkit: Ditto. Also remove the old way of supporting Xcode 2.1 and the old --debug option. The new --development option does the same thing.
  • Scripts/run-safari: Ditto.
  • Scripts/run-webkit-tests: Ditto.
  • Scripts/update-webkit: Ditto.
  • Scripts/set-webkit-configuration: Added. Sets the default configuration to Development or Deployment.
  • Scripts/webkitdirs.pm: use FindBin to find the WebKit directory; works no matter what the current directory is when invoking a script. Add code to determine the Xcode version so we can do the right thing for 2.0 and 2.1. Change the productDir function to return the appropriate per-configuration product directory. Read the default configuration from a file, overridable by a passed-in command-line option.
14:06 Changeset [9365] by adele

Merged fix for <rdar://problem/4142247> from TOT to Safari-2-0-branch.

2005-06-12 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Chris Blumenberg.

  • WebView.subproj/WebFrame.m: (-[WebFrame _transitionToCommitted:]): Stop loading the non-provisional data source before swapping in the provisional. (-[WebFrame _continueLoadRequestAfterNavigationPolicy:formState:]): Stop only the provisional load here, we would not want to stop loading if this navigation later turns into a download or is cancelled before being committed. (-[WebFrame stopLoading]): Factored a bit. (-[WebFrame _cancelProvisionalLoad]): New method to stop only provisional load, and cancel any pending policy deicions. (-[WebFrame _stopNonProvisionalLoadOnly]): New mthod that stops only the main load.
13:27 Changeset [9364] by mjs

Reviewed by Chris Blumenberg.

  • WebView.subproj/WebFrame.m: (-[WebFrame _transitionToCommitted:]): Stop loading the non-provisional data source before swapping in the provisional. (-[WebFrame _continueLoadRequestAfterNavigationPolicy:formState:]): Stop only the provisional load here, we would not want to stop loading if this navigation later turns into a download or is cancelled before being committed. (-[WebFrame stopLoading]): Factored a bit. (-[WebFrame _cancelProvisionalLoad]): New method to stop only provisional load, and cancel any pending policy deicions. (-[WebFrame _stopNonProvisionalLoadOnly]): New mthod that stops only the main load.

06/11/05:

22:56 Changeset [9363] by darin
  • Scripts/cvs-create-patch: Improve handling of directories with mixed CVS roots by going into each directory to execute the cvs diff commands.
22:41 Changeset [9362] by darin
  • added first cuts at some cvs scripts

cvs-abandon is for throwing away changes; discards any local changes, reverting to the state in CVS
cvs-create-patch is for making patches; runs cvs diff with all the right options and handles added/deleted files
cvs-apply is for applying patches; runs patch and cvs add and cvs rm
cvs-unapply is for unapplying patches; does the opposite of cvs-apply

An argument against these is "waste of time if we switch to Subversion", but they should be good for a while.

  • Scripts/cvs-abandon: Added.
  • Scripts/cvs-apply: Added.
  • Scripts/cvs-create-patch: Added.
  • Scripts/cvs-unapply: Added.
21:07 Changeset [9361] by mjs

Patch from Nicholas Shanks <​contact@nickshanks.com>, by me.

Merge of svn log -v -r 399829 ​svn://anonsvn.kde.org/home/kde

Test cases added: Added the following from CSS selector test suite,
unfortunately 159 and 117a do not show any difference with and without the
patch since layout tests do not dump selection style (yet).

  • layout-tests/fast/selectors/159.html: Added.
  • layout-tests/fast/selectors/177a.html: Added.
  • layout-tests/fast/selectors/177b.html: Added.
  • khtml/css/css_base.cpp: (CSSSelector::specificity): (CSSSelector::extractPseudoType): (CSSSelector::selectorText):
  • khtml/css/css_base.h: (DOM::CSSSelector::): (DOM::CSSSelector::pseudoType):
  • khtml/css/cssstyleselector.cpp: (khtml::CSSStyleSelector::checkSelector): (khtml::CSSStyleSelector::checkOneSelector):
  • khtml/css/parser.y:
02:51 Changeset [9360] by mjs

Patch from Timothy Hatcher <​timothy@colloquy.info>, reviewed by me.

Test cases added:

  • layout-tests/fast/dom/children-nodes-expected.txt: Added.
  • layout-tests/fast/dom/children-nodes.html: Added.
  • khtml/html/html_miscimpl.cpp: (DOM::HTMLCollectionImpl::traverseNextItem): For "children" collection, use nextSibling() instead of traverseNextNext() to avoid doing deep traversal.
02:41 Changeset [9359] by mjs

Added missing expected results from the background image patch.

  • layout-tests/fast/backgrounds/001-expected.txt: Added.

06/10/05:

22:31 Changeset [9358] by hyatt

Fix for bugzilla bug 3335, add support for the CSS3 indirect adjacent sibling selector. Patch merge
from KHTML tree by Nick Shanks.

Reviewed by hyatt

Test cases added: fast/selectors/046.html

  • khtml/css/css_base.cpp: (CSSSelector::selectorText):
  • khtml/css/css_base.h: (DOM::CSSSelector::):
  • khtml/css/cssstyleselector.cpp: (khtml::CSSStyleSelector::checkSelector):
  • khtml/css/parser.y:
  • layout-tests/fast/selectors/046-expected.txt: Added.
  • layout-tests/fast/selectors/046.html: Added.
22:01 Changeset [9357] by hyatt

Fix for 3237, background image repeats when it shouldn't. The Radar bug is 4005553. Patch from
Nate Cook.

Reviewed by darin and hyatt

Test cases added: fast/backgrounds/001.html

  • khtml/rendering/render_box.cpp: (RenderBox::paintBackgroundExtended):
  • layout-tests/fast/backgrounds/001.html: Added.
18:58 Changeset [9356] by hyatt

Merge Allan Jensen's fix for 3236.

Reviewed by hyatt

Test cases added: None, test case already added for the bug.

  • khtml/css/cssstyleselector.cpp: (khtml::CSSStyleSelector::applyProperty):
  • khtml/rendering/render_style.cpp: (RenderStyle::setContent):
17:42 Changeset [9355] by sullivan

reviewed by Dave Harrison (first & second drafts) and Darin Adler (third draft)

  • WebKit part of fix for <rdar://problem/4145214> REGRESSION (412+): Can't drag URLs from the location bar
  • Misc.subproj/WebNSPasteboardExtras.m: (+[NSPasteboard initialize]): Reinstate variation of old code that uses CreatePasteboardFlavorTypeName to set up our new pasteboard types. The newfangled way didn't work.
17:40 Changeset [9354] by sullivan

Updated copies of WebKitSystemInterface.h and libWebKitSystemInterface.a that include the new functions needed to fix 3446192.

11:02 Changeset [9353] by darin

Change by Mark Rowe <​opendarwin.org@bdash.net.nz>.
Refined and reviewed by me.

  • layout-tests/fast/js/exception-linenums-expected.txt: Added.
  • layout-tests/fast/js/exception-linenums.html: Added.
11:02 Changeset [9352] by darin

Change by Mark Rowe <​opendarwin.org@bdash.net.nz>.
Reviewed by me.

  • further improvements to exception file/line number fix
  • kjs/nodes.h: Added setExceptionDetailsIfNeeded function.
  • kjs/nodes.cpp: Updated macros to call the new setExceptionDetailsIfNeeded function. (Node::setExceptionDetailsIfNeeded): Added.
Note: See TracTimeline for information about the timeline view.