Timeline



Aug 3, 2005:

7:37 PM Changeset in webkit [10032] by justing
  • 2 edits
    2 adds in branches/Safari-2-0-branch

Merging this fix from TOT to Safari-2-0-branch

7:04 PM Changeset in webkit [10031] by justing
  • 3 edits
    2 adds in branches/Safari-2-0-branch

Merging this fix from TOT to Safari-2-0-branch

7:02 PM Changeset in webkit [10030] by vicki
  • 1 edit in trunk/WebCore/ChangeLog-2005-08-23

add missing bug number to Changelog comments from the scrollIntoView fix

6:58 PM Changeset in webkit [10029] by vicki
  • 8 edits in branches/Safari-2-0-branch/WebCore

Merge this fix from TOT to Safari-2-0-branch

2005-07-19 Vicki Murley <vicki@apple.com>

Reviewed by Maciej.

  • fixed <rdar://problem/3471901> support scrollIntoView method for DOM elements (3474)

Test cases added:

  • manual-tests/scrollIntoView-horizontal.html: Added.
  • manual-tests/scrollIntoView-vertical.html: Added.
  • khtml/xml/dom_elementimpl.h:
  • khtml/xml/dom_elementimpl.cpp: (ElementImpl::scrollIntoView): Added.
  • khtml/ecma/kjs_dom.h: (KJS::DOMNode::): (KJS::DOMElement::):
  • khtml/ecma/kjs_dom.cpp: (KJS::DOMElementProtoFunc::tryCall): glue for new scrollIntoView function

(these two files were modified on the branch only, not required in TOT fix)

  • khtml/dom/dom_element.cpp: (Element::scrollIntoView):
  • khtml/dom/dom_element.h:
5:40 PM Changeset in webkit [10028] by rjw
  • 1 edit in trunk/JavaScriptCore/bindings/objc/objc_class.mm

Submitted by: rjw@apple.com
Reviewed by: cblu@apple.com
Fixed hard capacity limit on the method and field cache use for the ObjC bindings.

<rdar://problem/4204416> Dashboard client crashes when too many ObjC methods are called

2:49 PM Changeset in webkit [10027] by adele
  • 2 edits in branches/Safari-2-0-branch/WebKit

Merged fix from TOT. Also fixes:
<rdar://problem/4077814> Gradual slowing down of Safari eventually requiring Quit and relaunch

2005-07-26 Maciej Stachowiak <mjs@apple.com>

Reviewed by Dave Hyatt.

  • WebView.subproj/WebFrame.m: (-[WebFrame _purgePageCache]): Find the oldest candidate for purging that is not a snapback item.
2:23 PM Changeset in webkit [10026] by vicki
  • 2 edits
    1 add in trunk/WebCore

Reviewed by Beth and Hyatt.

<rdar://problem/4196646> target returns text node when it should return the parent node

Test cases added: manual-tests/mouseevents-on-textnodes.html

  • khtml/khtmlview.cpp: (KHTMLView::dispatchMouseEvent): if the targetNode is a text node, set the targetNode to the parent node
1:55 PM Changeset in webkit [10025] by adele
  • 3 edits in branches/Safari-2-0-branch/WebCore

Merged fix from TOT to Safari-2-0-branch
<rdar://problem/4128727> many DOM classes not exported (can't add categories or do other things that require class name linking)

  • WebCore-combined.exp:
  • WebCore.exp:
9:47 AM Changeset in webkit [10024] by bdakin
  • 3 edits in trunk/WebKit

Reviewed by cblu

Removing calls to WKCreateUncorrectedRGBColorSpace and
WKCreateUncorrectedGrayColorSpace in WebKit to patch up
TOT...Eric removed them from WebCore last night.

  • WebCoreSupport.subproj/WebImageData.m:
  • WebCoreSupport.subproj/WebImageRenderer.m: (WebCGColorSpaceCreateRGB): (WebCGColorSpaceCreateGray):

Aug 2, 2005:

5:07 PM Changeset in webkit [10023] by bdibello
  • 1 edit
    4 adds in trunk

Reviewed by Geoffrey Garen.

Test cases added: getting and setting button.value

  • layout-tests/fast/dom/HTMLButtonElement/value/getset-expected.txt: Added.
  • layout-tests/fast/dom/HTMLButtonElement/value/getset.html: Added.
5:02 PM Changeset in webkit [10022] by adele
  • 2 edits in branches/Safari-2-0-branch/WebCore

Merged fix from TOT to Safari-2-0-branch

2005-07-29 Geoffrey Garen <ggaren@apple.com>

  • fixed: <rdar://problem/3461547> can't use document.writeln() on windows opened via window.open("about:blank") in same function <rdar://problem/3725977> A_WINDOW.document.write() content overwritten when first parameter to window.open is provided <rdar://problem/4101498> SAP: window.write in same call chain as window.open fails to write if URL parameter is provided <rdar://problem/4179028> document.write in same function as window.open fails to write new content (4056) http://bugzilla.opendarwin.org/show_bug.cgi?id=4056 document.write in same function as window.open fails to write new content

Reviewed by darin.

Test cases added:

  • manual-tests/resources/write-after-open-popup.html: Added.
  • khtml/khtml_part.cpp: (KHTMLPart::didExplicitOpen): now cancels any pending redirection
4:10 PM Changeset in webkit [10021] by adele
  • 3 edits in branches/Safari-2-0-branch/WebCore

Merged fix from TOT to Safari-2-0-branch

2005-07-14 Justin Garcia <justin.garcia@apple.com>

Patch by Anders Carlsson and justin

Reviewed by mjs

Fixes the following related bugs:
<http://bugzilla.opendarwin.org/show_bug.cgi?id=3298> insertRow generates DOM Exception if TABLE does not possess a TBODY
<rdar://problem/4045213> insertRow fails on a <table> that has no <tbody> but does have other children in the DOM
<rdar://problem/3609576> insertRow() on table object containing any characters but no TR & TD elements fails (3476)

Does a better job of maintaining the firstBody variable so that it's not invalidated on .innerHTML = ""
Better check for when to add an implicit tbody to hold the inserted row.

Test cases added: (NONE)

  • layout-tests/fast/dom/HTMLTableElement/insert-row-expected.txt: Added.
  • layout-tests/fast/dom/HTMLTableElement/insert-row.html: Added.
  • khtml/html/html_tableimpl.cpp: (DOM::HTMLTableElementImpl::~HTMLTableElementImpl): (DOM::HTMLTableElementImpl::setTBody): (DOM::HTMLTableElementImpl::insertRow): (DOM::HTMLTableElementImpl::addChild): (DOM::HTMLTableElementImpl::childrenChanged):
  • khtml/html/html_tableimpl.h:
4:06 PM Changeset in webkit [10020] by ggaren
  • 2 edits in branches/Safari-2-0-branch/JavaScriptCore
  • minor tweak to last checkin
  • kjs/debugger.h: restored a comment I shouldn't have deleted
3:50 PM Changeset in webkit [10019] by ggaren
  • 14 edits
    7 adds in branches/Safari-2-0-branch

WebCore:

Merged fix from TOT to Safari-2-0-branch

mjs 05/07/29 20:00:34

Changes by Michael Kahl, reviewed by me.

Test cases added: No tests, this code only affects debugging.

  • fixed <rdar://problem/4164112> MASTER: JavaScript debugging support
  • WebCore-combined.exp:
  • WebCore.exp:
  • WebCore.xcodeproj/project.pbxproj:
  • kwq/WebCoreScriptDebugger.h: Added.
  • kwq/WebCoreScriptDebugger.mm: Added. (toNSString): (WebCoreScriptDebuggerImp::WebCoreScriptDebuggerImp): (WebCoreScriptDebuggerImp::sourceParsed): (WebCoreScriptDebuggerImp::callEvent): (WebCoreScriptDebuggerImp::atStatement): (WebCoreScriptDebuggerImp::returnEvent): (-[WebCoreScriptDebugger initWithDelegate:]): (-[WebCoreScriptDebugger dealloc]): (-[WebCoreScriptDebugger delegate]): (-[WebCoreScriptCallFrame _initWithGlobalObject:caller:state:]): (-[WebCoreScriptCallFrame _setWrapper:]): (-[WebCoreScriptCallFrame _convertValueToObjcValue:]): (-[WebCoreScriptCallFrame dealloc]): (-[WebCoreScriptCallFrame wrapper]): (-[WebCoreScriptCallFrame caller]): (-[WebCoreScriptCallFrame scopeChain]): (-[WebCoreScriptCallFrame functionName]): (-[WebCoreScriptCallFrame exception]): (-[WebCoreScriptCallFrame evaluateWebScript:]):

WebKit:

Merged fix from TOT to Safari-2-0-branch

mjs 05/07/29 20:00:34

Changes by Michael Kahl, reviewed by me.

Test cases added: No tests, this code only affects debugging.

  • fixed <rdar://problem/4164112> MASTER: JavaScript debugging support
  • WebCoreSupport.subproj/WebBridge.m: (-[WebBridge windowObjectCleared]):
  • WebCoreSupport.subproj/WebSubresourceLoader.m:
  • WebKit.xcodeproj/project.pbxproj:
  • WebView.subproj/WebDefaultScriptDebugDelegate.h: Added.
  • WebView.subproj/WebDefaultScriptDebugDelegate.m: Added. (+[WebDefaultScriptDebugDelegate sharedScriptDebugDelegate]): (-[WebDefaultScriptDebugDelegate webView:didParseSource:fromURL:sourceId:forWebFrame:]): (-[WebDefaultScriptDebugDelegate webView:didEnterCallFrame:sourceId:line:forWebFrame:]): (-[WebDefaultScriptDebugDelegate webView:willExecuteStatement:sourceId:line:forWebFrame:]): (-[WebDefaultScriptDebugDelegate webView:willLeaveCallFrame:sourceId:line:forWebFrame:]):
  • WebView.subproj/WebFrame.m: (-[WebFramePrivate dealloc]): (-[WebFrame _attachScriptDebugger]):
  • WebView.subproj/WebFramePrivate.h:
  • WebView.subproj/WebScriptDebugDelegate.h: Added.
  • WebView.subproj/WebScriptDebugDelegate.m: Added. (-[WebScriptDebugger initWithWebFrame:]): (-[WebScriptDebugger dealloc]): (-[WebScriptDebugger globalObject]): (-[WebScriptDebugger newWrapperForFrame:]): (-[WebScriptDebugger parsedSource:fromURL:sourceId:]): (-[WebScriptDebugger enteredFrame:sourceId:line:]): (-[WebScriptDebugger hitStatement:sourceId:line:]): (-[WebScriptDebugger leavingFrame:sourceId:line:]): (-[WebScriptCallFrame _initWithFrame:]): (-[WebScriptCallFrame dealloc]): (-[WebScriptCallFrame setUserInfo:]): (-[WebScriptCallFrame userInfo]): (-[WebScriptCallFrame caller]): (-[WebScriptCallFrame scopeChain]): (-[WebScriptCallFrame functionName]): (-[WebScriptCallFrame exception]): (-[WebScriptCallFrame evaluateWebScript:]):
  • WebView.subproj/WebScriptDebugDelegatePrivate.h: Added.
  • WebView.subproj/WebView.h:
  • WebView.subproj/WebView.m: (-[WebViewPrivate dealloc]): (-[WebView _scriptDebugDelegateForwarder]): (-[WebView setScriptDebugDelegate:]): (-[WebView scriptDebugDelegate]):
  • WebView.subproj/WebViewInternal.h:
  • WebView.subproj/WebViewPrivate.h:
3:23 PM Changeset in webkit [10018] by adele
  • 2 edits in branches/Safari-2-0-branch/WebCore

small change to fix build bustage.

  • khtml/ecma/kjs_binding.h: use '.' instead of '->'. On the branch, self returns an Object instead of an ObjectImp *
2:39 PM Changeset in webkit [10017] by adele
  • 2 edits in branches/Safari-2-0-branch/WebCore

Merged fix from TOT to Safari-2-0-branch

2005-07-25 Vicki Murley <vicki@apple.com>

Reviewed by Beth Dakin.

  • fixed <rdar://problem/4011544> selecting an item from a <select> menu with the mouse doesn't place the focus on the menu

Test cases added:

  • manual-tests/focus-select-when-clicked.html: Added.
  • kwq/KWQComboBox.mm: (-[KWQPopUpButton needsPanelToBecomeKey]): override this NSView method to return YES
2:35 PM Changeset in webkit [10016] by adele
  • 3 edits in branches/Safari-2-0-branch/WebCore

Merged fix from TOT to Safari-2-0-branch

2005-06-29 Anders Carlsson <andersca@mac.com>

Reviewed and landed by Darin Adler.

Test cases added:

  • layout-tests/fast/xsl/document-function-expected.txt: Added.
  • layout-tests/fast/xsl/document-function.xml: Added.
  • layout-tests/fast/xsl/document-function.xsl: Added.
  • layout-tests/fast/xsl/resources/document-function-source.xml: Added.
  • khtml/xsl/xslt_processorimpl.cpp: (DOM::parseErrorFunc): Stub to prevent errors from getting written to the console.

(DOM::stylesheetLoadFunc):
Fetch and parse documents.

(DOM::XSLTProcessorImpl::transformDocument):
Use the processor as the global variable instead of the
style sheet.

  • khtml/xsl/xslt_processorimpl.h: (DOM::XSLTProcessorImpl::stylesheet): (DOM::XSLTProcessorImpl::sourceDocument): Add accessors for source document and stylesheet.
2:26 PM Changeset in webkit [10015] by adele
  • 27 edits in branches/Safari-2-0-branch

JavaScriptCore:

Merged from TOT to Safari-2-0-branch.

2005-07-27 Maciej Stachowiak <mjs@apple.com>

Changes by Michael Kahl, reviewed by me.

  • fixed <rdar://problem/4194278> Need better debugging support in JavaScriptCore
  • JavaScriptCore.xcodeproj/project.pbxproj:
  • kjs/debugger.cpp: (KJS::AttachedInterpreter::AttachedInterpreter): (KJS::AttachedInterpreter::~AttachedInterpreter): (Debugger::~Debugger): (Debugger::attach): (Debugger::detach): (Debugger::sourceParsed):
  • kjs/debugger.h:
  • kjs/function.cpp: (KJS::FunctionImp::call): (KJS::GlobalFuncImp::call):
  • kjs/function_object.cpp: (FunctionObjectImp::construct):
  • kjs/grammar.y:
  • kjs/internal.cpp: (Parser::parse): (InterpreterImp::evaluate):
  • kjs/internal.h: (KJS::InterpreterImp::setDebugger):
  • kjs/interpreter.cpp:
  • kjs/interpreter.h: (KJS::Interpreter::imp):
  • kjs/nodes.cpp:

WebCore:

Merged fix from TOT to Safari-2-0-branch

2005-07-03 Maciej Stachowiak <mjs@apple.com>

Original patch from Mark Rowe <opendarwin.org@bdash.net.nz>, reviewed by me.
Fixes to patch by me, reviewed by John Sullivan.

Test cases added:

  • layout-tests/fast/js/has-own-property.html: Added - tests this change.
  • layout-tests/fast/js/eval-var-decl.html: Added - test depends on this change.
  • khtml/ecma/kjs_css.cpp: (KJS::DOMCSSStyleDeclaration::hasOwnProperty):
  • khtml/ecma/kjs_css.h:
  • khtml/ecma/kjs_dom.cpp: (KJS::DOMNodeList::hasOwnProperty): (KJS::DOMElement::tryGet): (KJS::DOMNamedNodeMap::hasOwnProperty):
  • khtml/ecma/kjs_dom.h:
  • khtml/ecma/kjs_html.cpp: (KJS::HTMLDocument::hasOwnProperty): (KJS::HTMLDocument::tryGet): (KJS::KJS::HTMLElement::hasOwnProperty): (KJS::KJS::HTMLCollection::hasOwnProperty):
  • khtml/ecma/kjs_html.h:
  • khtml/ecma/kjs_window.cpp: (KJS::Window::hasOwnProperty): (KJS::Konqueror::hasOwnProperty):
  • khtml/ecma/kjs_window.h:
1:39 PM Changeset in webkit [10014] by ggaren
  • 13 edits in branches/Safari-2-0-branch/JavaScriptCore

Merged from TOT to Safari-2-0-branch.

2005-07-27 Maciej Stachowiak <mjs@apple.com>

Changes by Michael Kahl, reviewed by me.

  • fixed <rdar://problem/4194278> Need better debugging support in JavaScriptCore
  • JavaScriptCore.xcodeproj/project.pbxproj:
  • kjs/debugger.cpp: (KJS::AttachedInterpreter::AttachedInterpreter): (KJS::AttachedInterpreter::~AttachedInterpreter): (Debugger::~Debugger): (Debugger::attach): (Debugger::detach): (Debugger::sourceParsed):
  • kjs/debugger.h:
  • kjs/function.cpp: (KJS::FunctionImp::call): (KJS::GlobalFuncImp::call):
  • kjs/function_object.cpp: (FunctionObjectImp::construct):
  • kjs/grammar.y:
  • kjs/internal.cpp: (Parser::parse): (InterpreterImp::evaluate):
  • kjs/internal.h: (KJS::InterpreterImp::setDebugger):
  • kjs/interpreter.cpp:
  • kjs/interpreter.h: (KJS::Interpreter::imp):
  • kjs/nodes.cpp:
1:36 PM Changeset in webkit [10013] by ggaren
  • 6 adds
    1 delete in /
  • fixed build failure on Safari-2-0-branch
  • DumpRenderTree/DumpRenderTree.xcodeproj/.cvsignore: Added.
  • DumpRenderTree/DumpRenderTree.xcodeproj/project.pbxproj: Added.
  • DumpRenderTree/project.pbxproj: Removed.
1:28 PM Changeset in webkit [10012] by hyatt
  • 13 edits
    3 deletes in trunk
  • landed layout tests for fix to <rdar://problem/3572585> window.open fails if name param = the name of a window just closed in same function

Fix reviewed by darin.

Test cases added:

  • manual-tests/open-after-close.html: Added.
  • manual-tests/resources/open-after-close-popup.html: Added.
10:28 AM Changeset in webkit [10011] by ggaren
  • 1 edit in trunk/WebKit/ChangeLog

Fixed Changelog entry.

10:26 AM Changeset in webkit [10010] by ggaren
  • 1 edit in trunk/WebCore/ChangeLog-2005-08-23

Fixed last Changelog entry.

10:19 AM Changeset in webkit [10009] by ggaren
  • 1 edit in trunk/WebKit/ChangeLog

Fixing previous Changelog entry.

10:14 AM Changeset in webkit [10008] by sullivan
  • 4 edits in trunk/WebKit

Reviewed by Darin Adler.

Preemptively moved some WebMenuItemTag values from SPI to API, in anticipation of
approval from macosx-api-reviewers. Retitled one of them in response to API reviewers feedback:
WebMenuItemSearchInGoogle -> WebMenuItemSearchWeb

Note that as a side effect of this change, the actual numbers used for these WebMenuItemTags has
changed from what it was in Tiger. This causes "Search in Spotlight", "Search in Google", and
"Look Up in Dictionary" to not appear in Tiger Safari if running on tip of tree WebKit.

  • WebView.subproj/WebUIDelegatePrivate.h: removed WebMenuItemTagSearchInSpotlight, WebMenuItemTagSearchInGoogle, and WebMenuItemTagLookUpInDictionary
  • WebView.subproj/WebUIDelegate.h: added WebMenuItemTagSearchInSpotlight, WebMenuItemTagSearchWeb, and WebMenuItemTagLookUpInDictionary
  • WebView.subproj/WebDefaultContextMenuDelegate.m: (-[WebDefaultUIDelegate menuItemWithTag:]): updated for rename (-[WebDefaultUIDelegate contextMenuItemsForElement:defaultMenuItems:]): ditto (-[WebDefaultUIDelegate editingContextMenuItemsForElement:defaultMenuItems:]): ditto
9:58 AM Changeset in webkit [10007] by adele
  • 4 edits in branches/Safari-2-0-branch/WebKit

Merged fix from TOT to Safari-2-0-branch

2005-05-20 Chris Blumenberg <cblu@apple.com>

Fixed: <rdar://problem/4098786> sync. XMLHttpRequest works w/o AllowNetworkAccess key because load delegate is not consulted

Synchronous loads did not cause the willSendRequest method on the resource load delegate to be called. This is the method that Dashboard uses to enforce AllowNetworkAccess and this must be called to avoid exploits.

Reviewed by sullivan.

  • WebCoreSupport.subproj/WebBridge.m: (-[WebBridge objectLoadedFromCacheWithURL:response:data:]): call [WebFrame _requestFromDelegateForRequest:identifier:error:] then [WebFrame _saveResourceAndSendRemainingDelegateMessagesWithRequest:identifier:response:data:error:] so synthetic resource load delegate methods are called and the data is saved as a WebResource for resources in the WebCore cache.

(-[WebBridge syncLoadResourceWithURL:customHeaders:postData:finalURL:responseHeaders:statusCode:]): call [WebFrame _requestFromDelegateForRequest:identifier:error:],
respect its result, do the load and then call [WebFrame _saveResourceAndSendRemainingDelegateMessagesWithRequest:identifier:response:data:error:]
for synchronous loads

  • WebView.subproj/WebFrame.m: (-[WebFrame _opened]): call [WebFrame _requestFromDelegateForRequest:identifier:error:] then [WebFrame _sendRemainingDelegateMessagesWithIdentifier:response:length:error:] so synthetic resource load delegate methods are called for subresrources in the page cache

(-[WebFrame _requestFromDelegateForRequest:identifier:error:]): new, was part of the removed _sendResourceLoadDelegateMessagesForURL:::
This method calls identifierForInitialRequest and willSendRequest.

(-[WebFrame _sendRemainingDelegateMessagesWithIdentifier:response:length:error:]): new, was part of the removed _sendResourceLoadDelegateMessagesForURL:::
This method calls the remaining resource load delegate messages.

(-[WebFrame _saveResourceAndSendRemainingDelegateMessagesWithRequest:identifier:response:data:error:]): new, saves the resource and calls
[WebFrame _sendRemainingDelegateMessagesWithIdentifier:response:length:error:]

  • WebView.subproj/WebFrameInternal.h:
9:52 AM Changeset in webkit [10006] by adele
  • 3 edits in branches/Safari-2-0-branch/WebCore

Merged fix from TOT to Safari-2-0-branch

2005-07-08 Vicki Murley <vicki@apple.com>

Test cases added: (NONE)

<rdar://problem/4109893> REGRESSION: back/forward broken at wsj.com, worked in v185
Also written as http://bugzilla.opendarwin.org/show_bug.cgi?id=3901

Broken by security fix for 4005575: Arbitrary file disclosure vulnerability due to ability to load local html from remote content

The root of this bug is that URLs for subframes of a page are sometimes added to the back/forward list. This happens a lot at
wsj.com, and I believe it would happen for many or all sites that use JS to cons up a URL for an iframe and then load it.

The security fix changed the code path to go through openURLRequest, but only openURL had the logic to guess if
the current navigation was done in response to a user gesture. openURLRequest always assumed it was a user gesture,
and thus all uses of this code path would place an item in the b/f list.

  • kwq/KWQKHTMLPart.h:
  • kwq/KWQKHTMLPart.mm: (KWQKHTMLPart::userGestureHint): Factored code from openURL. (KWQKHTMLPart::openURL): Call newly factored code. (KWQKHTMLPart::openURLRequest): Newly call newly factored code.
9:47 AM Changeset in webkit [10005] by adele
  • 2 edits in branches/Safari-2-0-branch/WebCore

Merged fix from TOT to Safari-2-0-branch

2005-07-28 Beth Dakin <Beth Dakin>

This is a fix for <rdar://problem/4190684>. Hyatt and I
(okay, mostly Hyatt) wrote this for Cayenne so that they
can change the value of a style of a rule and have everything
update dynamically without causing a crash.

Reviewed by Hyatt

Test cases added: (NONE)

  • khtml/css/css_valueimpl.cpp: (DOM::CSSMutableStyleDeclarationImpl::setChanged):
9:39 AM Changeset in webkit [10004] by adele
  • 2 edits in branches/Safari-2-0-branch/JavaScriptCore

Merged from TOT to Safari-2-0-branch.

2005-06-30 Kevin Decker <kdecker@apple.com>

Reviewed by rjw.

fixed: <rdar://problem/4166838> failed assertion in`Interpreter::lockCount() > 0

no layout test added; this is in the bindings code.

  • bindings/objc/WebScriptObject.mm: (+[WebScriptObject _convertValueToObjcValue:KJS::originExecutionContext:Bindings::executionContext:Bindings::]): make sure to lock and unlock the interpreter around allocations.

Aug 1, 2005:

5:51 PM Changeset in webkit [10003] by ggaren
  • 3 edits
    2 adds in trunk

WebCore:

-landed layout tests for fix to <rdar://problem/3572585> window.open fails if name param = the name of a window just closed in same function

Fix reviewed by darin.

Test cases added:

  • manual-tests/open-after-close.html: Added.
  • manual-tests/resources/open-after-close-popup.html: Added.

WebKit:

-fixed <rdar://problem/3572585> window.open fails if name param = the name of a window just closed in same function

Reviewed by darin.

Test cases added:

  • manual-tests/open-after-close.html: Added.
  • manual-tests/resources/open-after-close-popup.html: Added.
  • WebCoreSupport.subproj/WebBridge.m: (-[WebBridge closeWindowSoon]): We now remove a WebView from WebViewSets when the WebView is *scheduled* to close.
4:56 PM Changeset in webkit [10002] by ggaren
  • 6 edits in branches/Safari-2-0-branch

Updating projects and make files for XCode 2.1.

3:23 PM Changeset in webkit [10001] by hyatt
  • 1 edit in trunk/WebCore/khtml/rendering/render_theme_mac.h

Forgot to commit part of my previous patch.

2:31 PM Changeset in webkit [10000] by sullivan
  • 2 edits in trunk/WebKit
  • PublicHeaderChangesFromTiger.txt: added a comment about isTextField -> _isTextField
2:23 PM Changeset in webkit [9999] by hyatt
  • 4 edits in trunk/WebCore

Fix bugzilla bug 4219, crash because we tried to do the fake paint
when layout was not updated yet.

Also landing accessibility for checkboxes.

Reviewed by darin/mjs (accessibility), kocienda(crash)

  • ChangeLog:
  • khtml/rendering/render_theme.h: (khtml::RenderTheme::supportsControlTints):
  • kwq/KWQAccObject.mm: (-[KWQAccObject actionElement]): (-[KWQAccObject role]): (-[KWQAccObject roleDescription]): (-[KWQAccObject value]): (boundingBoxRect): (-[KWQAccObject accessibilityIsIgnored]): (-[KWQAccObject accessibilityAttributeValue:]): (-[KWQAccObject accessibilityIsAttributeSettable:]):
  • kwq/KWQKHTMLPart.mm: (KWQKHTMLPart::setDisplaysWithFocusAttributes):
2:19 PM Changeset in webkit [9998] by sullivan
  • 2 edits in trunk/WebKit

Patch by Trey Matteson <trey@usa.net>
Reviewed by me.

Fixed http://bugzilla.opendarwin.org/show_bug.cgi?id=4255

underlines still print too thick

The real problem here is that we have code that scales a 0 width line to always
be width=1.0 in device space. I'm leaving that in for the screen, but when printing
a width of 0.5 looks good.

  • WebCoreSupport.subproj/WebTextRenderer.m: (-[WebTextRenderer drawLineForCharacters:yOffset:width:color:thickness:]):
12:11 PM Changeset in webkit [9997] by eseidel
  • 2 edits in trunk/WebKit

Bug #: 4211
Submitted by: eseidel
Reviewed by: darin

10:50 AM Changeset in webkit [9996] by sullivan
  • 8 edits in trunk/WebKit

Patch by Trey Matteson <trey@usa.net>
Reviewed by me.

Fixed http://bugzilla.opendarwin.org/show_bug.cgi?id=4014

PDF files by default load with a poor choice of sizing

For now the various PDF viewing settings are sticky, stored in 2 new defaults. Since
there are a number of ways these settings are changed, I made a proxy for the PDFView
through which all view changing messages are sent. The proxy adds the behavior of
updating the defaults upon any change.

  • Misc.subproj/WebNSDictionaryExtras.h:
  • Misc.subproj/WebNSDictionaryExtras.m: (-[NSMutableDictionary _webkit_setFloat:forKey:]): New support method.
  • WebView.subproj/WebPDFView.h:
  • WebView.subproj/WebPDFView.m: (-[WebPDFView initWithFrame:]): Create proxy for PDFView. (-[WebPDFView dealloc]): Free proxy. (-[WebPDFView _menuItemsFromPDFKitForEvent:]): For relevant context menu items, set the

target to the proxy instead of the PDFView.

(-[WebPDFView _readPDFDefaults]): Init PDFView with settings from defaults.
(-[WebPDFView layout]): Call _readPDFDefaults, once. This turned out to be the best hook.
(-[WebPDFView _makeTextSmaller:]): Change PDFView via proxy
(-[WebPDFView _makeTextLarger:]): Ditto
(-[WebPDFView _makeTextStandardSize:]): Ditto
(-[PDFPrefUpdatingProxy initWithView:]): trivial
(-[PDFPrefUpdatingProxy forwardInvocation:]): Forward the msg, then update defaults
(-[PDFPrefUpdatingProxy methodSignatureForSelector:]): Simple forwarding support.

  • WebView.subproj/WebPreferenceKeysPrivate.h:
  • WebView.subproj/WebPreferences.m: (+[WebPreferences initialize]): Set initial values for new PDF viewing defaults. (-[WebPreferences _integerValueForKey:]): Nuke stray comment. (-[WebPreferences _floatValueForKey:]): New simple support method. (-[WebPreferences _setFloatValue:forKey:]): Ditto. (-[WebPreferences PDFScaleFactor]): 4 accessors for new defaults (-[WebPreferences setPDFScaleFactor:]): (-[WebPreferences PDFDisplayMode]): (-[WebPreferences setPDFDisplayMode:]):
  • WebView.subproj/WebPreferencesPrivate.h:
9:19 AM Changeset in webkit [9995] by justing
  • 2 edits in trunk/WebKit

Patch by Trey Matteson <trey@usa.net>

Reviewed by Maciej.

Fixed <http://bugzilla.opendarwin.org/show_bug.cgi?id=4226>

link underlines print too thickly

Reinstate the fix made by sullivan on 1/11/05. There was a merge
error with an mjs fix on 1/13/05.

  • WebCoreSupport.subproj/WebTextRenderer.m: (-[WebTextRenderer drawLineForCharacters:yOffset:width:color:thickness:]):
9:12 AM Changeset in webkit [9994] by sullivan
  • 4 edits in trunk/WebCore

Reviewed by Beth Dakin.

Test cases added: none, this only affects Safari forms autofill

Moved the recently-added isTextField method out of the public DOMExtensions.h
and into the private DOMPrivate.h for now, and renamed it with a leading
underscore. We want to make some sort of public API that covers this, but not
necessarily this exact method, so we'll leave it out of the public API until we've
thought about this more.

  • kwq/DOMExtensions.h: remove isTextField
  • kwq/DOMPrivate.h: add _isTextField. Also added some #imports that make this file more standalone.
  • kwq/DOMHTML.mm: (-[DOMHTMLInputElement _isTextField]): moved this method and added leading underscore.
2:06 AM Changeset in webkit [9993] by eseidel
  • 8 edits in trunk/WebCore

Bug #: 4211
Submitted by: eseidel
Reviewed by: darin

  • kcanvas/device/quartz/KCanvasItemQuartz.mm: (getSharedContext): moved from QPainter -> CG
  • kcanvas/device/quartz/KRenderingPaintServerGradientQuartz.mm: (CGShadingRefForLinearGradient): removed comment (CGShadingRefForRadialGradient): removed comment
  • khtml/ecma/kjs_html.cpp: (KJS::KJS::Context2DFunction::call): moved from QPainter -> CG (KJS::colorRefFromValue): moved from QPainter -> CG (KJS::Gradient::getShading): moved from QPainter -> CG
  • khtml/rendering/render_canvasimage.cpp: (RenderCanvasImage::createDrawingContext):moved from QPainter -> CG
  • kwq/KWQColor.mm: (CGColorFromNSColor): moved from QPainter -> CG
  • kwq/KWQPainter.h: removed colorspace calls
  • kwq/KWQPainter.mm: removed colorspace calls (QPainter::clearFocusRing): moved from QPainter -> CG Removed use of WebKitGraphicsBridge Colorspace SPIs, as well as corresponding QPainter colorspace "choke-point" methods. This fixes colors for WebCore+SVG (since it wasn't linking in webkit). WebKit code removal in separate commit. http://bugzilla.opendarwin.org/show_bug.cgi?id=4211

Jul 31, 2005:

10:02 PM Changeset in webkit [9992] by darin
  • 4 edits in trunk/JavaScriptCore

Reviewed by Maciej.

  • remove uses of Mac-OS-X-specific MAX macro
  • remove one of the many excess "APPLE_CHANGES" ifdefs
  • kjs/collector.cpp: (KJS::Collector::allocate): Use std::max instead of MAX.
  • kjs/property_map.cpp: (KJS::PropertyMap::rehash): Ditto.
  • kjs/ustring.cpp: (KJS::UChar::toLower): Take out non-ICU code path. (KJS::UChar::toUpper): Ditto. (KJS::UString::spliceSubstringsWithSeparators): Use std::max instead of MAX.
9:31 PM Changeset in webkit [9991] by darin
  • 4 edits in trunk/WebCore

Reviewed by Maciej.

  • remove uses of Mac-OS-X-specific MAX macro in non-Mac-OS-X-specific code
  • khtml/misc/arena.cpp: (ArenaAllocate): Use std::max instead of MAX.
  • khtml/misc/loader.cpp: (Cache::setSize): Use kMax instead of MAX.
  • khtml/rendering/render_form.cpp: (RenderLineEdit::setSelectionStart): Ditto. (RenderLineEdit::setSelectionEnd): Ditto. (RenderLineEdit::setSelectionRange): Ditto.
8:25 PM Changeset in webkit [9990] by hyatt
  • 3 edits in trunk/WebCore

No editor working

8:20 PM Changeset in webkit [9989] by justing
  • 2 edits
    2 adds in trunk

Reviewed by mjs

Fixes <http://bugzilla.opendarwin.org/show_bug.cgi?id=3359> Crash on hover with certain styles on the text applied

If a zero length render object (such as a text node that has been set to "") occured at the
end of a line, it was previously given a non-zero sized run. Iteration over such a node would cause
a crash.

Test cases added:

  • layout-tests/traversal/size-zero-run-expected.txt: Added.
  • layout-tests/traversal/size-zero-run.html: Added.
  • khtml/rendering/bidi.cpp: (khtml::appendRun):
6:58 PM Changeset in webkit [9988] by darin
  • 3 edits in trunk/WebKitTools
  • a little tools cleanup
  • DumpRenderTree/DumpRenderTree.xcodeproj/project.pbxproj: Moved options from target to project as a whole. Removed unused Default configuration.
  • Scripts/check-dom-results: Added license header, comment to explain purpose of tool.
6:56 PM Changeset in webkit [9987] by justing
  • in /

file delete-br-011-expected.txt was added on branch Safari-2-0-branch on 2005-08-04 02:37:36 +0000

6:56 PM Changeset in webkit [9986] by darin
  • 299 edits in trunk/LayoutTests

Update results for change in DumpRenderTree tool to log editing
delegate method calls.

6:56 PM Changeset in webkit [9985] by justing
  • in /

file delete-4038408-fix-expected.txt was added on branch Safari-2-0-branch on 2005-08-04 02:04:17 +0000

6:55 PM Changeset in webkit [9984] by darin
  • 2 edits in trunk/WebKitTools

Reviewed and landed by Darin Adler.

Add logging of editing delegate calls for regression checking.

  • DumpRenderTree/DumpRenderTree.m: (main): setup editing delegate

(-[DOMNode dumpPath]):
(-[DOMRange dump]):

utility for editing delegate logging

(-[EditingDelegate webView:shouldBeginEditingInDOMRange:]):
(-[EditingDelegate webView:shouldEndEditingInDOMRange:]):
(-[EditingDelegate webView:shouldInsertNode:replacingDOMRange:givenAction:]):
(-[EditingDelegate webView:shouldInsertText:replacingDOMRange:givenAction:]):
(-[EditingDelegate webView:shouldDeleteDOMRange:]):
(-[EditingDelegate webView:shouldChangeSelectedDOMRange:toDOMRange:affinity:stillSelecting:]):
(-[EditingDelegate webView:shouldApplyStyle:toElementsInDOMRange:]):
(-[EditingDelegate webView:shouldChangeTypingStyle:toStyle:]):
(-[EditingDelegate webViewDidBeginEditing:]):
(-[EditingDelegate webViewDidChange:]):
(-[EditingDelegate webViewDidEndEditing:]):
(-[EditingDelegate webViewDidChangeTypingStyle:]):

log corresponding editing delegate methods

(-[EditingDelegate webViewDidChangeSelection:]):

log selection except when clearing selection after end of test (uses existing "done" flag)

(dumpRenderTree):

added clearing of selection after test

1:02 PM Changeset in webkit [9983] by eseidel
  • 2 edits
    4 adds in trunk/WebKitTools

Bug #: 4208
Submitted by: eseidel
Reviewed by: hyatt

12:51 PM Changeset in webkit [9982] by eseidel
  • 6 edits
    1 add in trunk

Bug #: 4234
Submitted by: eseidel
Reviewed by: hyatt

  • WebCore-svg.exp: Added.
  • WebCore.xcodeproj/project.pbxproj: Uses WebCore+SVG.exp
  • kcanvas/device/quartz/KCanvasFilterQuartz.mm: (KCanvasFEFloodQuartz::getCIFilter): Fixed longjmp warning.
  • kdom/css/impl/CSSStyleSelector.cpp: (CSSStyleSelector::applyRule): Fixed "un-initialized" warning.
  • kdom/ecma/DOMLookup.h: Removed <iostream>
  • ksvg2/impl/svgpathparser.cpp: Removed <iostream> Several fixes required to make Deployment style builds compile correctly. Also WebCore-svg.exp symbols were added both when generating WebCore-combined.exp as well as for generating a new WebCore+SVG.exp export file for the Deployment style builds, allowing DrawTest to link properly. A few other small warnings were caught and fixed. http://bugzilla.opendarwin.org/show_bug.cgi?id=4234

Jul 30, 2005:

6:08 PM Changeset in webkit [9981] by hyatt
  • 7 edits in trunk/WebCore

Editor still broken.

2:19 PM Changeset in webkit [9980] by darin
  • 5 edits
    2 adds in trunk

Reviewed and landed by Darin Adler.

Test cases added:

  • layout-tests/fast/dom/css-dom-read-2.html: Added.
  • layout-tests/fast/dom/css-dom-read-2-expected.txt: Added.
  • khtml/css/css_computedstyle.cpp: (DOM::CSSComputedStyleDeclarationImpl::cssText): Implement.

(DOM::CSSComputedStyleDeclarationImpl::item):
Return the item name, not its value.

  • khtml/css/css_ruleimpl.cpp: (CSSImportRuleImpl::cssText): (CSSMediaRuleImpl::cssText): (CSSStyleRuleImpl::cssText):
  • khtml/css/css_ruleimpl.h: Implement.
  • khtml/css/css_stylesheetimpl.cpp: (MediaListImpl::mediaText): Don't put a trailing comma after lists.
1:42 PM Changeset in webkit [9979] by darin
  • 2 edits in trunk
  • rolled back a test that has been failing since we rolled back the <script/> quirk
  • layout-tests/fast/tokenizer/script_extra_close-expected.txt: Updated.
1:27 PM Changeset in webkit [9978] by darin
  • 2 edits in trunk/WebCore

Reviewed by me, change by Dave Hyatt.

  • khtml/xml/dom_nodeimpl.cpp: (DOM::NodeImpl::NodeImpl): Initialize m_hovered to false.
2:05 AM Changeset in webkit [9977] by mjs
  • 3 edits in trunk/WebCore

Missing bits of last checkin...

  • WebCore.exp:
  • WebCore.xcodeproj/project.pbxproj:
1:38 AM Changeset in webkit [9976] by ggaren
  • in /

file WebScriptDebugDelegate.m was added on branch Safari-2-0-branch on 2005-08-02 22:50:28 +0000

1:38 AM Changeset in webkit [9975] by mjs
  • 3 edits in trunk/WebKit

Build fixes for previous change (missing includes)

  • WebView.subproj/WebFrame.m:
  • WebView.subproj/WebScriptDebugDelegate.m:
12:20 AM Changeset in webkit [9974] by hyatt
  • 1 edit in trunk/WebCore/khtml/editing/htmlediting.cpp

Fix build bustage.

Note: See TracTimeline for information about the timeline view.