Timeline



Jun 13, 2006:

10:54 PM Changeset in webkit [14860] by ggaren
  • 6 edits
    2 adds in trunk

LayoutTests:

Test for createHTMLDocument.

  • fast/dom/implementation-createHTMLDocument-expected.txt: Added.
  • fast/dom/implementation-createHTMLDocument.html: Added.

WebCore:

Reviewed by Maciej.


  • Fixed <rdar://problem/4562192> Creating a new DOMHTMLDocument appears challenging.


  • Added createHTMLDocument to Objc bindings, fixed up implementation to comply with the DOM 2 Candidate Recommendation in which is was defined. (It never made the final spec.) Also removed use of deprecatedString, to avoid unnecessary killing of puppies.
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/objc/DOM.mm: (-[DOMImplementation createHTMLDocument:]):
  • bindings/objc/DOMPrivate.h:
  • dom/DOMImplementation.cpp: (WebCore::DOMImplementation::createDocument): (WebCore::DOMImplementation::createHTMLDocument):
10:03 PM Changeset in webkit [14859] by mjs
  • 2 edits
    1 add in trunk/LayoutTests
  • fixed up results of the layout test I committed for 4583892, they were guaranteed to fail
  • fast/events/no-window-load-expected.txt:
  • fast/events/resources/greenbox.png: Added.
10:01 PM Changeset in webkit [14858] by ggaren
  • 1 edit
    1 add in trunk/LayoutTests

RS by MacDome. Added missing results for this test.

  • fast/css/xml-stylesheet-pi-not-in-prolog-expected.txt: Added.
6:35 PM Changeset in webkit [14857] by aliceli1
  • 2 edits
    2 adds in trunk

Reviewed by Adele.

fixed <rdar://problem/4457902> HTMLSelectElement.remove() can't handle an option obj being passed to it, results in hang on http://www.ibc-solar.de/www_ibc/fst_solarmonitoring.jsp

  • bindings/js/kjs_html.cpp: (KJS::HTMLElementFunction::callAsFunction): support this by converting the argument to an options object before passing to remove()
6:33 PM Changeset in webkit [14856] by darin
  • 5 edits
    2 adds in trunk

LayoutTests:

Reviewed by Geoff.

  • test for <rdar://problem/4585333> Changing location for weather on yahoo.com home page redirects to another page
  • fast/events/event-listener-html-non-html-confusion-expected.txt: Added.
  • fast/events/event-listener-html-non-html-confusion.html: Added.

WebCore:

Reviewed by Geoff.

  • fix <rdar://problem/4585333> Changing location for weather on yahoo.com home page redirects to another page

This patch fixes a bug where the event listener cache does not distinguish
HTML and non-HTML listeners. Incorrect behavior where stopPropagation also
prevented default masked a case of this bug on the yahoo.com home page until
we fixed bug 5180 on 2005-10-03.

Test: fast/events/event-listener-html-non-html-confusion.html

  • bindings/js/kjs_window.h: Add additional listener maps for HTML event listeners.
  • bindings/js/kjs_window.cpp: (KJS::Window::~Window): Go through the additional maps when clearing the window object pointer in event listeners. (KJS::Window::getJSEventListener): Look in the HTML or non-HTML map depending on the argument passed. (KJS::Window::getJSUnprotectedEventListener): Ditto.
  • bindings/js/kjs_events.cpp: (KJS::JSUnprotectedEventListener::JSUnprotectedEventListener): Add to either the HTML or non-HTML map depending on the argument passed. (KJS::JSUnprotectedEventListener::~JSUnprotectedEventListener): Remove from either the HTML or non-HTML map depending on whether the HTML flag is set. (KJS::JSEventListener::JSEventListener): More of the same. (KJS::JSEventListener::~JSEventListener): Ditto. (KJS::JSLazyEventListener::parseCode): Same thing here. In a lazy event listener there is not a listener at construction time, thus the code here to put the listener into a map needs the HTML vs. non-HTML logic.
6:25 PM Changeset in webkit [14855] by mjs
  • 3 edits
    2 adds in trunk

LayoutTests:

Reviewed by Geoff.


  • test case for <rdar://problem/4583892> 10.4.7 regression: Hang occurs when attempting to load search results at mapquest.com
  • fast/events/no-window-load-expected.txt: Added.
  • fast/events/no-window-load.html: Added.

WebCore:

Reviewed by Darin.


<rdar://problem/4583892> 10.4.7 regression: Hang occurs when attempting to load search results at mapquest.com

  • dom/EventTargetNode.cpp: (WebCore::EventTargetNode::dispatchGenericEvent): Don't allow "load" events to propagate up to the window. We need this quirk to avoid site hangs, because they depend on an old Mozilla bug.
6:11 PM Changeset in webkit [14854] by thatcher
  • 1 delete in tags/WebCore-418.7/WebCore

Bad tag.

5:56 PM Changeset in webkit [14853] by thatcher
  • 1 copy in tags/WebCore-418.17/WebCore

New tag.

5:56 PM Changeset in webkit [14852] by thatcher
  • 1 delete in tags/WebCore-418.17/WebCore

Bad tag.

5:53 PM Changeset in webkit [14851] by thatcher
  • 1 copy in tags/WebCore-418.7/WebCore

New tag.

5:53 PM Changeset in webkit [14850] by thatcher
  • 1 delete in tags/WebCore-418.7/WebCore

Bad tag.

5:52 PM Changeset in webkit [14849] by thatcher
  • 1 edit in branches/Safari-2-0-branch/WebCore/ChangeLog

Versioning

5:46 PM Changeset in webkit [14848] by ggaren
  • 4 edits in branches/Safari-2-0-branch/WebCore

Patch by Darin, tweaked by me, reviewed by Harrison.

  • Fixed <rdar://problem/4584598> 10.4.7 regression: Changing location for weather on yahoo.com home page redirects to another page


We need to keep separate caches for HTML and non-HTML event listeners
so that, if you add one of each to the same node, the two don't clobber
each other.

  • khtml/ecma/kjs_events.cpp: (JSUnprotectedEventListener::JSUnprotectedEventListener): (JSUnprotectedEventListener::~JSUnprotectedEventListener): (JSEventListener::JSEventListener): (JSEventListener::~JSEventListener): (JSLazyEventListener::parseCode):
  • khtml/ecma/kjs_window.cpp: (Window::~Window): (Window::getJSEventListener): (Window::getJSUnprotectedEventListener):
  • khtml/ecma/kjs_window.h:
4:19 PM Changeset in webkit [14847] by antti
  • 7 edits
    12 adds in trunk

LayoutTests:

Reviewed by Hyatt

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

  • fast/block/positioning/relative-overflow-block-expected.checksum: Added.
  • fast/block/positioning/relative-overflow-block-expected.png: Added.
  • fast/block/positioning/relative-overflow-block-expected.txt: Added.
  • fast/block/positioning/relative-overflow-block.html: Added.
  • fast/block/positioning/relative-overflow-replaced-expected.checksum: Added.
  • fast/block/positioning/relative-overflow-replaced-expected.png: Added.
  • fast/block/positioning/relative-overflow-replaced-expected.txt: Added.
  • fast/block/positioning/relative-overflow-replaced-float-expected.checksum: Added.
  • fast/block/positioning/relative-overflow-replaced-float-expected.png: Added.
  • fast/block/positioning/relative-overflow-replaced-float-expected.txt: Added.
  • fast/block/positioning/relative-overflow-replaced-float.html: Added.
  • fast/block/positioning/relative-overflow-replaced.html: Added.

WebCore:

Reviewed by Hyatt.

http://bugzilla.opendarwin.org/show_bug.cgi?id=9314
Relative positioned block size doesnt update root layer size

  • take relative positioning into account in leftmost/rightmost/lowestPosition()
  • ignore zero width/height boxes in leftmost/rightmost/lowestPosition()
  • split relativePositionOffset() to x and y functions


  • rendering/RenderBox.cpp: (WebCore::RenderBox::absolutePosition): (WebCore::RenderBox::relativePositionOffsetX): (WebCore::RenderBox::relativePositionOffsetY): (WebCore::RenderBox::lowestPosition): (WebCore::RenderBox::rightmostPosition): (WebCore::RenderBox::leftmostPosition):
  • rendering/RenderBox.h:
  • rendering/RenderFlow.cpp: (WebCore::RenderFlow::lowestPosition): (WebCore::RenderFlow::rightmostPosition): (WebCore::RenderFlow::leftmostPosition):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::updateLayerPosition):
  • rendering/RenderObject.cpp: (WebCore::RenderObject::offsetLeft): (WebCore::RenderObject::offsetTop):
3:20 PM Changeset in webkit [14846] by andersca
  • 4 edits
    2 adds in trunk

WebCore:

2006-06-13 Anders Carlsson <acarlsson@apple.com>

Reviewed by Dave Hyatt.

http://bugzilla.opendarwin.org/show_bug.cgi?id=9427
xml-stylesheet processing instructions outside of the prolog should have no effect.

  • dom/ProcessingInstruction.cpp: (WebCore::ProcessingInstruction::checkStyleSheet): Remove todo comment.


  • dom/xml_tokenizer.cpp: (WebCore::XMLTokenizer::processingInstruction): Only check for style sheet if the root element hasn't yet been encountered.

LayoutTests:

2006-06-13 Anders Carlsson <acarlsson@apple.com>

Reviewed by Dave Hyatt.

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


  • fast/css/resources/xml-stylesheet-pi-not-in-prolog.css: Added.
  • fast/css/xml-stylesheet-pi-not-in-prolog.xml: Added.
3:19 PM Changeset in webkit [14845] by sullivan
  • 2 edits in trunk/WebKit

Reviewed by Tim O.


  • fixed <rdar://problem/4498606> REGRESSION (417.8-420+): 3 missing items (but extra separators) in context menu in Mail message body
  • WebView/WebView.m: (-[WebView _menuForElement:defaultItems:]): Add special-case hackery to recover from this SPI -> API mismatch.
2:58 PM Changeset in webkit [14844] by tomernic
  • 1 edit in trunk/WebKit/ChangeLog

Clarified my previous ChangeLog entry.

2:56 PM Changeset in webkit [14843] by tomernic
  • 2 edits in trunk/WebKit

Reviewed by Anders.

Fixed a recently-introduced assertion failure when handling 404 errors.

  • WebView/WebDataSource.m: (-[WebDataSource _handleFallbackContent]): Use the -[WebFrame _bridge] instead of -[WebDataSource _bridge]. The former is not valid until the data source has been committed, as is the case when the resource fails to load. The latter is safe to call at any time. This broke last night with Maciej's change to WebFrameResourceLoader. The old code used to call -[WebFrame _bridge].
1:17 PM Changeset in webkit [14842] by thatcher
  • 1 copy in tags/WebCore-418.17/WebCore

Tag for WebCore 418.17 (part 2 of 2.)

1:17 PM Changeset in webkit [14841] by thatcher
  • 1 add in tags/WebCore-418.17

Tag for WebCore 418.17 (part 1 of 2.)

1:15 PM Changeset in webkit [14840] by thatcher
  • 2 edits in branches/Safari-2-0-branch/WebCore

Versioning

12:56 PM Changeset in webkit [14839] by thatcher
  • 2 edits in branches/Safari-2-0-branch/WebCore

Reviewed by Harrison, landed by Tim H.

Handle window events for capture phase, except load events, this quirk is needed because
Mozilla used to not do the bubble or capture phase for image load events and sites mistakenly
put capturing load event listeners on the window

  • khtml/xml/dom_nodeimpl.cpp: (NodeImpl::dispatchGenericEvent):
6:37 AM Changeset in webkit [14838] by andersca
  • 6 edits in trunk

WebCore:

2006-06-13 Anders Carlsson <acarlsson@apple.com>

Reviewed by Geoff.

http://bugzilla.opendarwin.org/show_bug.cgi?id=9406
REGRESSION: fix for bug 9390 broke two layout tests


  • loader/PluginDocument.cpp: (WebCore::PluginTokenizer::writeRawData): Call finished() after setting up the document structure so we'll emit onload events.

WebKit:

2006-06-13 Anders Carlsson <acarlsson@apple.com>

Reviewed by Geoff.

http://bugzilla.opendarwin.org/show_bug.cgi?id=9406
REGRESSION: fix for bug 9390 broke two layout tests

  • Plugins/WebPluginDatabase.h: (-[WebPluginDatabase isMIMETypeRegistered:]): Add new function isMIMETypeRegistered which returns whether a given MIME type has a plugin registered.
  • Plugins/WebPluginDatabase.m: (-[WebPluginDatabase init]): Init set of registered MIME types.


(-[WebPluginDatabase refresh]):
Add and remove MIME types from the set of registered MIME types when registering and unregistering
plugin MIME types.


(-[WebPluginDatabase dealloc]):
Release set of registered MIME types.


  • WebCoreSupport/WebFrameBridge.m: (-[WebFrameBridge determineObjectFromMIMEType:URL:]): Use isMIMETypeRegistered here.

Jun 12, 2006:

11:37 PM Changeset in webkit [14837] by ggaren
  • 1 edit
    1 add in trunk/WebCore

build fix -- forgot to svn add this file


  • ForwardingHeaders/kjs/SavedBuiltins.h: Added.
11:27 PM Changeset in webkit [14836] by ggaren
  • 2 edits in trunk/JavaScriptCore

build fix


  • bindings/objc/WebScriptObject.mm: (+[WebScriptObject throwException:]): Restore assignment I accidentally deleted in previous commit
11:10 PM Changeset in webkit [14835] by beidson
  • 2 edits in trunk/WebCore

Reviewed by Maciej.

Fixed a bug in append(char) and append(UChar) where our intended copy-on-write semantics was ignored!

  • platform/String.cpp: (WebCore::String::append):
11:08 PM Changeset in webkit [14834] by ggaren
  • 29 edits
    2 copies
    1 move
    4 adds in trunk

JavaScriptCore:

Reviewed by TimO, Maciej.


  • Merged InterpreterImp code into Interpreter, which implements all interpreter functionality now. This is part of my continuing quest to create an external notion of JS "execution context" that is unified and simple -- something to replace the mix of Context, ContextImp, ExecState, Interpreter, InterpreterImp, and JSRun.


All tests pass. Leaks test has not regressed from its baseline ~207 leaks
with ~3460 leaked nodes.

  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:
  • JavaScriptCore.xcodeproj/project.pbxproj:
  • bindings/NP_jsobject.cpp:
  • bindings/objc/WebScriptObject.mm: (+[WebScriptObject throwException:]):
  • bindings/runtime_root.cpp:
  • bindings/runtime_root.h:
  • kjs/Context.cpp: (KJS::Context::Context):
  • kjs/ExecState.cpp: Added. (KJS::ExecState::lexicalInterpreter):
  • kjs/ExecState.h: Added. (KJS::ExecState::dynamicInterpreter):
  • kjs/SavedBuiltins.h: Added.
  • kjs/bool_object.cpp: (BooleanPrototype::BooleanPrototype):
  • kjs/collector.cpp: (KJS::Collector::collect): (KJS::Collector::numInterpreters):
  • kjs/context.h:
  • kjs/debugger.cpp: (Debugger::attach): (Debugger::detach):
  • kjs/function.cpp: (KJS::FunctionImp::callAsFunction): (KJS::GlobalFuncImp::callAsFunction):
  • kjs/function_object.cpp: (FunctionObjectImp::construct):
  • kjs/internal.cpp:
  • kjs/internal.h:
  • kjs/interpreter.cpp: (KJS::interpreterMap): (KJS::Interpreter::Interpreter): (KJS::Interpreter::init): (KJS::Interpreter::~Interpreter): (KJS::Interpreter::globalObject): (KJS::Interpreter::initGlobalObject): (KJS::Interpreter::globalExec): (KJS::Interpreter::checkSyntax): (KJS::Interpreter::evaluate): (KJS::Interpreter::builtinObject): (KJS::Interpreter::builtinFunction): (KJS::Interpreter::builtinArray): (KJS::Interpreter::builtinBoolean): (KJS::Interpreter::builtinString): (KJS::Interpreter::builtinNumber): (KJS::Interpreter::builtinDate): (KJS::Interpreter::builtinRegExp): (KJS::Interpreter::builtinError): (KJS::Interpreter::builtinObjectPrototype): (KJS::Interpreter::builtinFunctionPrototype): (KJS::Interpreter::builtinArrayPrototype): (KJS::Interpreter::builtinBooleanPrototype): (KJS::Interpreter::builtinStringPrototype): (KJS::Interpreter::builtinNumberPrototype): (KJS::Interpreter::builtinDatePrototype): (KJS::Interpreter::builtinRegExpPrototype): (KJS::Interpreter::builtinErrorPrototype): (KJS::Interpreter::builtinEvalError): (KJS::Interpreter::builtinRangeError): (KJS::Interpreter::builtinReferenceError): (KJS::Interpreter::builtinSyntaxError): (KJS::Interpreter::builtinTypeError): (KJS::Interpreter::builtinURIError): (KJS::Interpreter::builtinEvalErrorPrototype): (KJS::Interpreter::builtinRangeErrorPrototype): (KJS::Interpreter::builtinReferenceErrorPrototype): (KJS::Interpreter::builtinSyntaxErrorPrototype): (KJS::Interpreter::builtinTypeErrorPrototype): (KJS::Interpreter::builtinURIErrorPrototype): (KJS::Interpreter::mark): (KJS::Interpreter::interpreterWithGlobalObject): (KJS::Interpreter::saveBuiltins): (KJS::Interpreter::restoreBuiltins):
  • kjs/interpreter.h: (KJS::Interpreter::setCompatMode): (KJS::Interpreter::compatMode): (KJS::Interpreter::firstInterpreter): (KJS::Interpreter::nextInterpreter): (KJS::Interpreter::prevInterpreter): (KJS::Interpreter::debugger): (KJS::Interpreter::setDebugger): (KJS::Interpreter::setContext): (KJS::Interpreter::context):
  • kjs/nodes.cpp: (StatementNode::hitStatement): (RegExpNode::evaluate):
  • kjs/protect.h:

JavaScriptGlue:

Reviewed by TimO, Maciej.


  • JSGlue part of merging InterpreterImp into Interpreter.


Since there's now a kjs forwarding header, I moved all forwarding
headers into a "ForwardingHeaders" directory, like in WebCore.

  • ForwardingHeaders: Added.
  • ForwardingHeaders/kjs: Added.
  • ForwardingHeaders/kjs/ExecState.h: Added.
  • ForwardingHeaders/wtf: Added.
  • JavaScriptGlue.xcodeproj/project.pbxproj:
  • config.h:
  • wtf: Removed.
  • wtf/AlwaysInline.h: Removed.
  • wtf/Assertions.h: Removed.
  • wtf/FastMalloc.h: Removed.
  • wtf/Forward.h: Removed.
  • wtf/HashCountedSet.h: Removed.
  • wtf/HashSet.h: Removed.
  • wtf/Noncopyable.h: Removed.
  • wtf/OwnArrayPtr.h: Removed.
  • wtf/OwnPtr.h: Removed.
  • wtf/PassRefPtr.h: Removed.
  • wtf/Platform.h: Removed.
  • wtf/RefPtr.h: Removed.
  • wtf/Vector.h: Removed.

WebCore:

Reviewed by TimO, Maciej.

  • WebCore part of merging InterpreterImp into Interpreter. No test because there's no behavior change.


A substantive change here is that ScriptInterpreter::mark must now chain to
Interpreter::mark, since Interpreter needs to mark the things that
InterpreterImp used to mark.


  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/kjs_binding.cpp:
  • bindings/js/kjs_window.cpp: (KJS::Window::put):
  • bridge/mac/WebCoreFrameBridge.mm:
  • bridge/mac/WebCoreScriptDebugger.mm: (-[WebCoreScriptCallFrame evaluateWebScript:]):
  • kwq/KWQPageState.mm:
10:09 PM Changeset in webkit [14833] by mjs
  • 3 edits in trunk/WebKit
  • fix for cocoa exception (whoops)
  • WebView/WebView.m: (+[WebView _generatedMIMETypeForURLScheme:]): put this back
  • WebView/WebDataSource.m: (+[WebDataSource _generatedMIMETypeForURLScheme:]): call WebView
8:36 PM Changeset in webkit [14832] by mjs
  • 8 edits in trunk/WebKit

Reviewed by Darin.


  • remove use of WebView and related from WebMainResourceLoader
  • WebKit.xcodeproj/project.pbxproj:
  • WebView/WebDataSource.m: (+[WebDataSource _generatedMIMETypeForURLScheme:]): (+[WebDataSource _representationExistsForURLScheme:]): (+[WebDataSource _canShowMIMEType:]): (-[WebDataSource _handleFallbackContent]): (-[WebDataSource _decidePolicyForMIMEType:decisionListener:]):
  • WebView/WebDataSourceInternal.h:
  • WebView/WebFrame.m: (-[WebFrame _isMainFrame]):
  • WebView/WebFrameInternal.h:
  • WebView/WebMainResourceLoader.m: (-[WebMainResourceLoader willSendRequest:redirectResponse:]): (-[WebMainResourceLoader continueAfterContentPolicy:response:]): (-[WebMainResourceLoader checkContentPolicyForResponse:]): (-[WebMainResourceLoader loadWithRequestNow:]):
  • WebView/WebView.m:
6:04 PM Changeset in webkit [14831] by beidson
  • 8 edits in trunk/WebCore

Reviewed by Maciej.

Changed String::ascii() to return a Vector<char> instead of const char*
This allows us to use it "regularly" as the returned vector will destruct
and not leak memory like the previous approach.
We can now do a String.ascii().data() to get a char* buffer instead of
String.deprecatedString().ascii(). It doesn't improve the style much but
dumping the memory-leak issue is a plus.

  • icon/IconDatabase.cpp: (WebCore::IconDatabase::open): (WebCore::IconDatabase::clearDatabase):
  • icon/SQLDatabase.cpp: (SQLDatabase::open):
  • icon/SQLStatement.cpp: (WebCore::SQLStatement::prepare): (WebCore::SQLStatement::step): (WebCore::SQLStatement::returnTextResults): (WebCore::SQLStatement::returnTextResults16): (WebCore::SQLStatement::returnIntResults): (WebCore::SQLStatement::returnInt64Results): (WebCore::SQLStatement::returnDoubleResults): In addition to the changes to the string classes, changed my database code over to the new preferred method.
  • platform/PlatformString.h:
  • platform/String.cpp: (WebCore::String::ascii):
  • platform/StringImpl.cpp: (WebCore::StringImpl::ascii):
  • platform/StringImpl.h:
5:27 PM Changeset in webkit [14830] by tomernic
  • 2 edits in trunk/WebKit

Reviewed by Maciej.

<rdar://problem/4526052> intermittent assertion failure in -[WebBasePluginPackage dealloc] running layout tests

  • Plugins/WebNetscapePluginPackage.m: (-[WebNetscapePluginPackage _initWithPath:]): Renamed from -initWithPath:. Instead of releasing/deallocating self on error, return NO. (-[WebNetscapePluginPackage initWithPath:]): Call the new -_initWithPath:. If it returns NO, unload the plugin package before deallocating it.
4:16 PM Changeset in webkit [14829] by thatcher
  • 1 copy in tags/WebCore-418.16/WebCore

Tag for WebCore 418.16 (part 2 of 2.)

4:16 PM Changeset in webkit [14828] by thatcher
  • 1 add in tags/WebCore-418.16

Tag for WebCore 418.16 (part 1 of 2.)

4:03 PM Changeset in webkit [14827] by hyatt
  • 17 edits in trunk/WebCore

Add selection foreground color on Win32.

4:02 PM Changeset in webkit [14826] by sullivan
  • 2 edits in trunk/WebCore

Reviewed by Darin Adler.


  • page/Frame.cpp: (WebCore::Frame::markAllMatchesForText): Do a "fake" paint here so that the rectangles for the text matches will have been computed by the time this method returns.
3:43 PM Changeset in webkit [14825] by beidson
  • 4 edits in trunk/WebCore

Reviewed by Levi and Tim Omernick.

-Added a skeleton sqlite3 icon database file to IconDatabase
-Added functionality to validate and recreate this icon.db file
-Fixed some buggys in SQLDatabase.cpp

  • icon/IconDatabase.cpp: (WebCore::IconDatabase::open): (WebCore::IconDatabase::isValidDatabase): (WebCore::IconDatabase::clearDatabase): (WebCore::IconDatabase::recreateDatabase):
  • icon/IconDatabase.h:
  • icon/SQLStatement.cpp: (WebCore::SQLStatement::columnCount): (WebCore::SQLStatement::getColumnName): (WebCore::SQLStatement::getColumnName16): (WebCore::SQLStatement::getColumnText): (WebCore::SQLStatement::getColumnText16): (WebCore::SQLStatement::getColumnDouble): (WebCore::SQLStatement::getColumnInt): (WebCore::SQLStatement::getColumnInt64): (WebCore::SQLStatement::getColumnBlob): -Added checks to make sure we had a valid working sqlite3_statement as the sqlite3_* function calls weren't as error-tolerant as documentation advertised (maybe differences between the 3.3 docs I looked at and the 3.1.3 version installed on OSX)
1:54 PM Changeset in webkit [14824] by thatcher
  • 2 edits in branches/Safari-2-0-branch/WebCore

Versioning

1:53 PM Changeset in webkit [14823] by thatcher
  • 2 edits in branches/Safari-2-0-branch/WebCore

Reviewed by Maciej, Harrison.

Fixes <rdar://problem/4582172> frameset load events don't fire (can't login into Gmail account or see map at 511.org)

Framesets need to register their load, beforeunload and unload events with the window.

  • khtml/html/html_baseimpl.cpp: (HTMLFrameSetElementImpl::parseHTMLAttribute):
12:53 PM Changeset in webkit [14822] by hyatt
  • 3 edits in trunk/WebCore/platform

Fix Win32 build bustage.

12:02 PM Changeset in webkit [14821] by ggaren
  • 10 edits in trunk/JavaScriptCore

Reviewed by Maciej.


  • Have *.lut.h files #include lookup.h to eliminate surprising header include order dependency.
  • DerivedSources.make:
  • kjs/array_object.cpp:
  • kjs/date_object.cpp:
  • kjs/date_object.h: (KJS::DateProtoFunc::):
  • kjs/lexer.cpp:
  • kjs/math_object.cpp:
  • kjs/number_object.cpp:
  • kjs/regexp_object.cpp:
  • kjs/string_object.cpp:
11:07 AM Changeset in webkit [14820] by beidson
  • 7 edits
    3 adds in trunk/WebCore

Reviewed by Levi.

-Added SQLite helper wrappers to ease use of SQLite in the IconDatabase.
-Changed the base IconDatabase over to this new framework.

  • WebCore.xcodeproj/project.pbxproj:
  • bridge/mac/WebCoreIconDatabaseBridge.mm: (-[WebCoreIconDatabaseBridge closeSharedDatabase]):

-By popular request, removed an annoying log message I'd accidentally left in

  • icon/IconDatabase.cpp: (WebCore::IconDatabase::IconDatabase): (WebCore::IconDatabase::open): (WebCore::IconDatabase::close): (WebCore::IconDatabase::~IconDatabase):
  • icon/IconDatabase.h: (WebCore::IconDatabase::isOpen):

-Changed over IconDatabase from direct sqlite3_* calls to the new SQLDatabase calls

  • icon/SQLDatabase.cpp: Added. (SQLDatabase::SQLDatabase): (SQLDatabase::open): (SQLDatabase::close): (SQLDatabase::executeCommand): (SQLDatabase::tableExists):
  • icon/SQLDatabase.h: Added. (WebCore::SQLDatabase::isOpen): (WebCore::SQLDatabase::getPath): (WebCore::SQLDatabase::lastError): (WebCore::SQLDatabase::lastErrorMsg): (WebCore::SQLStatement::isPrepared): (WebCore::SQLStatement::lastError): (WebCore::SQLStatement::lastErrorMsg):
  • icon/SQLStatement.cpp: Added. (WebCore::SQLStatement::SQLStatement): (WebCore::SQLStatement::~SQLStatement): (WebCore::SQLStatement::prepare): (WebCore::SQLStatement::step): (WebCore::SQLStatement::finalize): (WebCore::SQLStatement::reset): (WebCore::SQLStatement::executeCommand): (WebCore::SQLStatement::bindBlob): (WebCore::SQLStatement::bindText): (WebCore::SQLStatement::columnCount): (WebCore::SQLStatement::getColumnName): (WebCore::SQLStatement::getColumnName16): (WebCore::SQLStatement::getColumnText): (WebCore::SQLStatement::getColumnText16): (WebCore::SQLStatement::getColumnDouble): (WebCore::SQLStatement::getColumnInt): (WebCore::SQLStatement::getColumnInt64): (WebCore::SQLStatement::getColumnBlob): (WebCore::SQLStatement::returnTextResults): (WebCore::SQLStatement::returnTextResults16): (WebCore::SQLStatement::returnIntResults): (WebCore::SQLStatement::returnInt64Results): (WebCore::SQLStatement::returnDoubleResults):

-Initial checking of SQLDatabase framework

  • platform/PlatformString.h:
  • platform/String.cpp: (WebCore::String::String):

-Added an explicit UChar* constructor to our string class as much of sqlite3's UTF16 handling is based on
null-terminated UTF16 which we didn't yet support.

Jun 11, 2006:

9:10 PM Changeset in webkit [14819] by ddkilzer
  • 2 edits in trunk/WebCore

WebCore:

Reviewed by Maciej.

http://bugzilla.opendarwin.org/show_bug.cgi?id=9408
Clean build fails with "make: * No rule to make target SVGElementFactory.cpp', needed by all'. Stop"

  • DerivedSources.make: Fix clean builds.
7:03 PM Changeset in webkit [14818] by darin
  • 10 edits in trunk/WebCore
  • another try at fixing Windows
  • loader/CachedResource.h:
  • platform/cairo/GraphicsContextCairo.cpp:
  • platform/image-decoders/ImageDecoder.h: (WebCore::ImageDecoder::setData):
  • platform/image-decoders/gif/GIFImageDecoder.cpp: (WebCore::GIFImageDecoderPrivate::decode): (WebCore::GIFImageDecoder::setData):
  • platform/image-decoders/gif/GIFImageDecoder.h:
  • platform/image-decoders/jpeg/JPEGImageDecoder.cpp: (WebCore::JPEGImageReader::decode): (WebCore::JPEGImageDecoder::setData):
  • platform/image-decoders/jpeg/JPEGImageDecoder.h:
  • platform/image-decoders/png/PNGImageDecoder.cpp: (WebCore::PNGImageReader::decode): (WebCore::PNGImageDecoder::setData):
  • platform/image-decoders/png/PNGImageDecoder.h: s/DeprecatedByteArray/Vector<char>/
6:37 PM Changeset in webkit [14817] by darin
  • 5 edits in trunk

WebCore:

  • try to fix Windows build
  • platform/cairo/ImageCairo.cpp: (WebCore::Image::loadResource): Use Vector<char> instead of DeprecatedByteArray.
  • platform/win/TemporaryLinkStubs.cpp: (KWQServeSynchronousRequest): Don't try to return 0 from a function that returns a Vector<char>.

WebKit:

  • try to fix Windows build
  • COM/WebKitDLL.cpp: (loadResourceIntoArray): Use Vector<char> instead of DeprecatedByteArray.
6:10 PM Changeset in webkit [14816] by darin
  • 59 edits
    2 deletes in trunk/WebCore

Reviewed by Hyatt, tweaked quite a bit and landed by Darin.

This removes most of the uses of DeprecatedArray and
DeprecatedByteArray, with the exception of DeprecatedCString.

No test cases added because there is no change in
functionality.

  • loader/CachedCSSStyleSheet.cpp: (WebCore::CachedCSSStyleSheet::data):
  • loader/CachedCSSStyleSheet.h:
  • loader/CachedImage.cpp: (WebCore::CachedImage::bufferData): (WebCore::CachedImage::data): (WebCore::CachedImage::checkNotify): (WebCore::CachedImage::shouldStopAnimation):
  • loader/CachedImage.h:
  • loader/CachedObject.cpp: (WebCore::CachedObject::bufferData): (WebCore::CachedObject::setExpireDate):
  • loader/CachedObject.h: (WebCore::CachedObject::setCharset):
  • loader/CachedScript.cpp: (WebCore::CachedScript::data):
  • loader/CachedScript.h:
  • loader/CachedXBLDocument.cpp: (WebCore::CachedXBLDocument::data): (WebCore::CachedXBLDocument::checkNotify):
  • loader/CachedXBLDocument.h:
  • loader/CachedXSLStyleSheet.cpp: (WebCore::CachedXSLStyleSheet::data):
  • loader/CachedXSLStyleSheet.h:
  • loader/Request.cpp: (WebCore::Request::Request): (WebCore::Request::~Request):
  • loader/Request.h: (WebCore::Request::buffer): (WebCore::Request::cachedObject): (WebCore::Request::docLoader): (WebCore::Request::isIncremental): (WebCore::Request::setIsIncremental): (WebCore::Request::isMultipart): (WebCore::Request::setIsMultipart):
  • loader/loader.cpp: (WebCore::crossDomain): (WebCore::Loader::Loader): (WebCore::Loader::load): (WebCore::Loader::servePendingRequests): (WebCore::Loader::receivedAllData): (WebCore::Loader::receivedResponse): (WebCore::Loader::receivedData): (WebCore::Loader::numRequests): (WebCore::Loader::cancelRequests): (WebCore::Loader::removeBackgroundDecodingRequest): (WebCore::Loader::jobForRequest):
  • platform/Image.cpp: (WebCore::Image::setData):
  • platform/Image.h: (WebCore::Image::dataBuffer):
  • css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::init): (WebCore::CSSStyleSelector::matchRules): (WebCore::CSSStyleSelector::matchRulesForList): (WebCore::CSSStyleSelector::sortMatchedRules): (WebCore::CSSStyleSelector::initForStyleResolve): (WebCore::CSSStyleSelector::createStyleForElement): (WebCore::CSSStyleSelector::createPseudoStyleForElement):
  • css/cssstyleselector.h: (WebCore::CSSStyleSelector::addMatchedRule): (WebCore::CSSStyleSelector::addMatchedDeclaration):
  • dom/xml_tokenizer.cpp: (WebCore::OffsetBuffer::OffsetBuffer): (WebCore::openFunc):
  • html/HTMLOptionElement.cpp: (WebCore::HTMLOptionElement::index):
  • html/HTMLSelectElement.cpp: (WebCore::HTMLSelectElement::selectedIndex): (WebCore::HTMLSelectElement::setSelectedIndex): (WebCore::HTMLSelectElement::length): (WebCore::HTMLSelectElement::remove): (WebCore::HTMLSelectElement::value): (WebCore::HTMLSelectElement::setValue): (WebCore::HTMLSelectElement::state): (WebCore::HTMLSelectElement::restoreState): (WebCore::HTMLSelectElement::appendFormData): (WebCore::HTMLSelectElement::optionToListIndex): (WebCore::HTMLSelectElement::listToOptionIndex): (WebCore::HTMLSelectElement::recalcListItems): (WebCore::HTMLSelectElement::reset): (WebCore::HTMLSelectElement::notifyOptionSelected):
  • html/HTMLSelectElement.h: (WebCore::HTMLSelectElement::listItems):
  • kwq/KWQLoader.h:
  • kwq/KWQLoader.mm: (KWQServeSynchronousRequest):
  • kwq/KWQTextStream.cpp: (QTextStream::operator<<):
  • kwq/KWQTextStream.h:
  • loader/FormData.cpp: (WebCore::FormData::flatten): (WebCore::FormData::flattenToString):
  • loader/FormData.h: (WebCore::FormDataElement::FormDataElement):
  • platform/DeprecatedString.cpp:
  • platform/DeprecatedString.h:
  • platform/IntPointArray.cpp: Removed.
  • platform/IntPointArray.h: Removed.
  • platform/TextEncoding.cpp: (WebCore::TextEncoding::toUnicode):
  • platform/TextEncoding.h:
  • platform/cairo/pixman/src/pixregion.c:
  • platform/cg/PathCG.cpp: (WebCore::Path::Path):
  • rendering/RenderObject.cpp: (WebCore::RenderObject::drawBorder):
  • rendering/RenderTable.h: (WebCore::RenderTable::colToEffCol): (WebCore::RenderTable::effColToCol):
  • rendering/RenderTableSection.cpp: (WebCore::RenderTableSection::ensureRows): (WebCore::RenderTableSection::addCell): (WebCore::RenderTableSection::setCellWidths):
  • rendering/RenderTableSection.h:
  • rendering/bidi.cpp: (WebCore::addMidpoint): (WebCore::RenderBlock::layoutInlineChildren):
  • rendering/render_form.cpp: (WebCore::RenderSelect::updateFromElement): (WebCore::RenderSelect::layout): (WebCore::RenderSelect::valueChanged): (WebCore::RenderSelect::selectionChanged): (WebCore::RenderSelect::updateSelection):
  • rendering/table_layout.cpp: (WebCore::FixedTableLayout::layout):
  • rendering/table_layout.h:
  • xml/XSLTProcessor.cpp: (WebCore::docLoaderFunc):
  • xml/xmlhttprequest.cpp: (WebCore::XMLHttpRequest::send): (WebCore::XMLHttpRequest::processSyncLoadResults):
  • xml/xmlhttprequest.h:
4:58 PM Changeset in webkit [14815] by ddkilzer
  • 122 edits in trunk/WebCore

WebCore:

Reviewed by darin.

http://bugzilla.opendarwin.org/show_bug.cgi?id=9394
Fix no-SVG build

  • DerivedSources.make: Removed tabs. Create empty SVGElementFactory.cpp on no-svg build.
  • ksvg2/bindings/js/JSSVGElementWrapperFactory.cpp: Added #if SVG_SUPPORT/#endif.
  • ksvg2/bindings/js/JSSVGElementWrapperFactory.h: Ditto.
  • ksvg2/bindings/idl/svg/SVGAElement.idl: Add "Conditional=SVG" to all interfaces.
  • ksvg2/bindings/idl/svg/SVGAnimateColorElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGAnimateElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGAnimateTransformElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGAnimatedAngle.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGAnimatedBoolean.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGAnimatedEnumeration.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGAnimatedInteger.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGAnimatedLengthList.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGAnimatedNumber.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGAnimatedNumberList.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGAnimatedPathData.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGAnimatedPoints.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGAnimatedPreserveAspectRatio.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGAnimatedRect.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGAnimatedString.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGAnimatedTransformList.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGAnimationElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGCircleElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGClipPathElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGComponentTransferFunctionElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGCursorElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGDOMImplementation.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGDefsElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGDescElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGElementInstance.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGElementInstanceList.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGEllipseElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGEvent.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGException.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGExternalResourcesRequired.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGFEBlendElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGFEColorMatrixElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGFEComponentTransferElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGFECompositeElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGFEFloodElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGFEFuncAElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGFEFuncBElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGFEFuncGElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGFEFuncRElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGFEGaussianBlurElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGFEImageElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGFEMergeElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGFEMergeNodeElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGFEOffsetElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGFETileElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGFETurbulenceElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGFilterElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGFilterPrimitiveStandardAttributes.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGFitToViewBox.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGGElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGGradientElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGICCColor.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGImageElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGLangSpace.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGLengthList.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGLineElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGLinearGradientElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGLocatable.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGMarkerElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGNumberList.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGPaint.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGPathElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGPathSeg.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGPathSegArc.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGPathSegClosePath.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGPathSegCurvetoCubic.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGPathSegCurvetoCubicSmooth.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGPathSegCurvetoQuadratic.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGPathSegCurvetoQuadraticSmooth.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGPathSegLineto.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGPathSegLinetoHorizontal.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGPathSegLinetoVertical.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGPathSegList.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGPathSegMoveto.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGPatternElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGPointList.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGPolygonElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGPolylineElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGPreserveAspectRatio.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGRadialGradientElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGRectElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGRenderingIntent.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGScriptElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGSetElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGStopElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGStringList.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGStylable.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGStyleElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGSwitchElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGSymbolElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGTSpanElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGTests.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGTextContentElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGTextElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGTextPositioningElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGTitleElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGTransformList.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGTransformable.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGURIReference.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGUnitTypes.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGUseElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGViewElement.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGZoomAndPan.idl: Ditto.
  • ksvg2/bindings/idl/svg/SVGZoomEvent.idl: Ditto.
  • ksvg2/svg/SVGAngle.idl: Ditto.
  • ksvg2/svg/SVGAnimatedLength.idl: Ditto.
  • ksvg2/svg/SVGColor.idl: Ditto.
  • ksvg2/svg/SVGDocument.idl: Ditto.
  • ksvg2/svg/SVGElement.idl: Ditto.
  • ksvg2/svg/SVGEvent.idl: Ditto.
  • ksvg2/svg/SVGLength.idl: Ditto.
  • ksvg2/svg/SVGMatrix.idl: Ditto.
  • ksvg2/svg/SVGNumber.idl: Ditto.
  • ksvg2/svg/SVGPoint.idl: Ditto.
  • ksvg2/svg/SVGRect.idl: Ditto.
  • ksvg2/svg/SVGSVGElement.idl: Ditto.
  • ksvg2/svg/SVGTransform.idl: Ditto.
4:49 PM Changeset in webkit [14814] by ddkilzer
  • 2 edits in trunk/WebKitTools

WebKitTools:

Reviewed by darin.

http://bugzilla.opendarwin.org/show_bug.cgi?id=9395
Make prepare-ChangeLog faster

  • Scripts/prepare-ChangeLog: Use "svn diff" instead of "svn stat" to find changed files, then save the diff output for reuse. Keep a status variable if changes are made to LayoutTests so "svn diff LayoutTests" doesn't have to be run to check for changes when
WebCore is updated. Added -hhelp command-line switch and help message. Move test

for no changed files closer to the beginning of the program.

1:18 PM Changeset in webkit [14813] by thatcher
  • 1 edit in trunk/WebKit/WebInspector/WebInspector.m

Removes a hardcoded path I had added for testing.

1:10 PM Changeset in webkit [14812] by thatcher
  • 2 edits in trunk/WebKit

Reviewed by Tim H.

  • WebInspector/WebInspector.m: (-[NSWindow windowWillClose:]): Added a call to setWebFrame to avoid further load progress notifications. (-[NSWindow setWebFrame:]): Changed to resign the WebView's hostWindow rather than its window for close notifications, to avoid resigning from all windows' close notifications (including the inspector window's) when the WebView is in a hidden tab. Also changed to prevent highlighting the initial focused node.
12:49 PM Changeset in webkit [14811] by andersca
  • 30 edits
    2 adds
    6 deletes in trunk

WebCore:

2006-06-11 Anders Carlsson <acarlsson@apple.com>

Reviewed by Geoff.

http://bugzilla.opendarwin.org/show_bug.cgi?id=9390
Move full-frame plugins to WebCore


  • WebCore.vcproj/WebCore/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj: Add PluginDocument


  • bridge/mac/FrameMac.h:
  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::redirectDataToPlugin): Call the bridge.


  • bridge/mac/WebCoreFrameBridge.h:
  • bridge/mac/WebCoreFrameBridge.mm: Add redirectDataToPlugin which is used to redirect incoming data to a plugin.


  • bridge/mac/WebCoreViewFactory.h: Add pluginSupportsMIMEType which returns whether any plugins support a given MIME type.


  • dom/DOMImplementation.cpp:
  • dom/DOMImplementation.h: Get rid of createTextDocument and just create a text document explicitly when needed.


  • dom/Document.h: (WebCore::Document::isPluginDocument):
  • loader/PluginDocument.cpp: Added. (WebCore::PluginTokenizer::PluginTokenizer): (WebCore::PluginTokenizer::wantsRawData): (WebCore::PluginTokenizer::write): (WebCore::PluginTokenizer::createDocumentStructure): (WebCore::PluginTokenizer::writeRawData): (WebCore::PluginTokenizer::stopParsing): (WebCore::PluginTokenizer::finish): (WebCore::PluginTokenizer::isWaitingForScripts): (WebCore::PluginDocument::PluginDocument): (WebCore::PluginDocument::createTokenizer):
  • loader/PluginDocument.h: Added. (WebCore::PluginDocument::isPluginDocument): Add PluginDocument.


  • page/Frame.cpp: (WebCore::Frame::begin): Possibly create a plugin document.


  • page/Frame.h: (WebCore::Frame::redirectDataToPlugin): Add declaration.


  • platform/PlugInInfoStore.h:
  • platform/mac/PlugInInfoStoreMac.mm: (WebCore::PlugInInfoStore::supportsMIMEType): Ask WebCoreViewFactory if the MIME type is supported.

WebKit:

2006-06-11 Anders Carlsson <acarlsson@apple.com>

Reviewed by Tim.

http://bugzilla.opendarwin.org/show_bug.cgi?id=9390
Move full-frame plugins to WebCore

  • Plugins/WebBaseNetscapePluginStream.h:
  • Plugins/WebBaseNetscapePluginStream.m: (-[WebBaseNetscapePluginStream instance]): Add instance method which returns the plugin instance.


  • Plugins/WebBasePluginPackage.h: Add WebPluginManualLoader protocol


  • Plugins/WebNetscapePluginDocumentView.h: Removed.
  • Plugins/WebNetscapePluginDocumentView.m: Removed.
  • Plugins/WebNetscapePluginEmbeddedView.h:
  • Plugins/WebNetscapePluginEmbeddedView.m: (-[WebNetscapePluginEmbeddedView initWithFrame:plugin:URL:baseURL:MIMEType:attributeKeys:attributeValues:loadManually:]): (-[WebNetscapePluginEmbeddedView dealloc]): (-[WebNetscapePluginEmbeddedView didStart]): (-[WebNetscapePluginEmbeddedView pluginView:receivedResponse:]): (-[WebNetscapePluginEmbeddedView pluginView:receivedData:]): (-[WebNetscapePluginEmbeddedView pluginView:receivedError:]): (-[WebNetscapePluginEmbeddedView pluginViewFinishedLoading:]): (-[WebNetscapePluginEmbeddedView redeliverStream]): Make WebNetscapePluginEmbeddedView support the WebPluginManualLoader protocol. It creates a plugin stream and feeds the data manually. Much of this code has been copied from WebNetscapePluginRepresentation.
  • Plugins/WebNetscapePluginRepresentation.h: Removed.
  • Plugins/WebNetscapePluginRepresentation.m: Removed.
  • Plugins/WebPluginController.h:
  • Plugins/WebPluginController.m: (-[WebPluginController pluginView:receivedResponse:]): (-[WebPluginController pluginView:receivedData:]): (-[WebPluginController pluginView:receivedError:]): (-[WebPluginController pluginViewFinishedLoading:]): Make WebPluginController support the WebPluginManualLoader protocol so it can feed data manually to WebKit plugins.
  • Plugins/WebPluginDatabase.m: (-[WebPluginDatabase refresh]): Use WebHTMLView and WebHTMLRepresentation when registering/unregistering plug-in MIME types.


  • Plugins/WebPluginDocumentView.h: Removed.
  • Plugins/WebPluginDocumentView.m: Removed.
  • WebCoreSupport/WebFrameBridge.m: (-[WebFrameBridge pluginViewWithPackage:attributeNames:attributeValues:baseURL:DOMElement:loadManually:]): (-[WebFrameBridge viewForPluginWithURL:attributeNames:attributeValues:MIMEType:DOMElement:loadManually:]): Add loadManually argument.


(-[WebFrameBridge redirectDataToPlugin:]):
Call down to the HTML representation.


(-[WebFrameBridge viewForJavaAppletWithFrame:attributeNames:attributeValues:baseURL:DOMElement:]):
Pass NO to loadManually.


(-[WebFrameBridge determineObjectFromMIMEType:URL:]):
Explicitly check if the MIME type is supported by a plug-in instead of checking the view class.


  • WebCoreSupport/WebViewFactory.m: (-[WebViewFactory pluginSupportsMIMEType:]): New function which returns whether any plugins support a given MIME type.


  • WebKit.xcodeproj/project.pbxproj: Update for removed files.


  • WebView/WebFrame.m: (-[WebFrame _reloadForPluginChanges]): Remove view type checks.


(-[WebFrame _recursive_pauseNullEventsForAllNetscapePlugins]):
(-[WebFrame _recursive_resumeNullEventsForAllNetscapePlugins]):
Remove FIXME comments.


  • WebView/WebHTMLRepresentation.m: (-[WebHTMLRepresentation _redirectDataToManualLoader:forPluginView:]): New function which redirects incoming data to a manual loader.


(-[WebHTMLRepresentation receivedData:withDataSource:]):
(-[WebHTMLRepresentation receivedError:withDataSource:]):
(-[WebHTMLRepresentation finishedLoadingWithDataSource:]):
Optionally redirect incoming data.


  • WebView/WebHTMLRepresentationPrivate.h:

Jun 10, 2006:

9:07 PM Changeset in webkit [14810] by ddkilzer
  • 3 edits
    4 adds in trunk

LayoutTests:

Patch by mitzpettel. Reviewed by hyatt.

  • fast/repaint/static-to-positioned-expected.checksum: Added.
  • fast/repaint/static-to-positioned-expected.png: Added.
  • fast/repaint/static-to-positioned-expected.txt: Added.
  • fast/repaint/static-to-positioned.html: Added.

WebCore:

Patch by mitzpettel. Reviewed by hyatt.

Test: fast/repaint/static-to-positioned.html

  • rendering/RenderObject.cpp: (WebCore::RenderObject::setStyle): If changing from static to positioned, repaint as static.
8:46 PM Changeset in webkit [14809] by ddkilzer
  • 4 edits
    9 adds in trunk

LayoutTests:

Reviewed by hyatt.

  • test repainting of layer children's outlines that extend beyond the layer
  • fast/layers: Added.
  • fast/layers/opacity-outline-expected.checksum: Added.
  • fast/layers/opacity-outline-expected.png: Added.
  • fast/layers/opacity-outline-expected.txt: Added.
  • fast/layers/opacity-outline.html: Added.
  • fast/repaint/layer-child-outline-expected.checksum: Added.
  • fast/repaint/layer-child-outline-expected.png: Added.
  • fast/repaint/layer-child-outline-expected.txt: Added.
  • fast/repaint/layer-child-outline.html: Added.

WebCore:

Reviewed by hyatt.

Test: fast/layers/opacity-outline.html

  • fix repainting of layer children's outlines that extend beyond the layer

Test: fast/repaint/layer-child-outline.html

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::intersectsDamageRect): Account for outlines. (WebCore::RenderLayer::absoluteBoundingBox): Ditto.
  • rendering/RenderObject.cpp: (WebCore::RenderObject::maximalOutlineSize): Changed to return the view's maximal outline size rather than 0 for PaintPhaseChildOutlines.
7:34 PM Changeset in webkit [14808] by sfalken
  • 3 edits in trunk/WebCore

2006-06-10 Steve Falkenburg <sfalken@apple.com>

Fix build break

  • platform/cairo/GraphicsContextCairo.cpp:
  • platform/win/TemporaryLinkStubs.cpp: (GraphicsContext::scale):
9:09 AM Changeset in webkit [14807] by ggaren
  • 28 edits in trunk

JavaScriptCore:


Fix by Mike Emmel, Reviewed by Darin.

  • JavaScriptCoreSources.bkl:
  • jscore.bkl:
  • wtf/Platform.h:

WebCore:


Fix by Mike Emmel, Reviewed by Darin.

  • Projects/gdk/webcore-gdk.bkl:
  • WebCoreSources.bkl:
  • css/maketokenizer:
  • html/HTMLCanvasElement.cpp:
  • html/HTMLImageElement.h:
  • icon/IconDatabase.cpp:
  • make-generated-sources.sh:
  • page/Frame.h:
  • page/FramePrivate.h:
  • platform/Cursor.h:
  • platform/FontData.h: (WebCore::FontData::getGlyphIndex):
  • platform/GlyphBuffer.h: (WebCore::GlyphBuffer::glyphAt): (WebCore::GlyphBuffer::advanceAt): (WebCore::GlyphBuffer::add):
  • platform/GraphicsContext.h:
  • platform/PlatformKeyboardEvent.h:
  • platform/PlatformMouseEvent.h:
  • platform/PlatformWheelEvent.h:
  • platform/ScrollView.h:
  • platform/TransferJob.h: (WebCore::TransferJob::getInternal):
  • platform/TransferJobInternal.h: (WebCore::TransferJobInternal::TransferJobInternal):
  • platform/Widget.h:
  • platform/cairo/GraphicsContextCairo.cpp: (WebCore::GraphicsContext::GraphicsContext): (WebCore::GraphicsContext::roundToDevicePixels):
  • webcore-base.bkl:
  • xpath/impl/XPathValue.cpp:
6:42 AM Changeset in webkit [14806] by sullivan
  • 5 edits in trunk/WebKit

Reviewed by Tim Omernick and Dave Hyatt.

WebKit support for computing but not highlighting rects for text matches.

  • WebView/WebHTMLViewPrivate.h: added markedTextMatchesAreHighlighted/setMarkedTextMatchesAreHighlighted and rectsForTextMatches, and renamed related methods for clarity/consistency
  • WebView/WebHTMLView.m: (-[WebHTMLView markAllMatchesForText:caseSensitive:]): renamed, calls similarly-renamed method (-[WebHTMLView setMarkedTextMatchesAreHighlighted:]): new method, calls through to bridge (-[WebHTMLView markedTextMatchesAreHighlighted]): ditto (-[WebHTMLView unmarkAllTextMatches]): renamed (-[WebHTMLView rectsForTextMatches]): new method, calls through to bridge
  • WebView/WebViewPrivate.h: added rectsForTextMatches, renamed other methods (and added highlight: parameter)
  • WebView/WebView.m: (-[WebView markAllMatchesForText:caseSensitive:highlight:]): renamed for clarity/consistency, and now has highlight: parameter, which is passed down (-[WebView unmarkAllTextMatches]): renamed for clarity/consistency, and calls similarly-renamed method lower down. diff got confused with the end of this and the end of the next method. (-[WebView rectsForTextMatches]): new method, calls through to WebHTMLView as related methods currently do
6:42 AM Changeset in webkit [14805] by sullivan
  • 6 edits in trunk/WebCore

Reviewed by Tim Omernick and Dave Hyatt.

WebCore support for computing but not highlighting rects for text matches.

  • dom/Document.h: added setRenderedRectForMarker() and renderedRectsForMarkers(), and redefined MarkerMap to be a hashtable of node -> (pair of vectors), one vector of markers and one vector of rects


  • dom/Document.cpp: (placeholderRectForMarker()): new function, returns a recognizable degenerate rect used until a real rect has been set (WebCore::Document::addMarker): Reworked for new MarkerMap data structure; now adds parallel placeholder rect along with marker (WebCore::Document::copyMarkers): Reworked for new MarkerMap data structure (WebCore::Document::removeMarkers): Reworked for new MarkerMap data structure; now removed corresponding rect along with marker (WebCore::Document::markersForNode): Reworked for new MarkerMap data structure (WebCore::Document::renderedRectsForMarkers): New method, returns an array of all non-placeholder rects for the given marker type (WebCore::Document::repaintMarkers): Reworked for new MarkerMap data structure (WebCore::Document::setRenderedRectForMarker): New method, sets the rendered rect for a given marker (WebCore::Document::shiftMarkers): Reworked for new MarkerMap data structure; resets rendered rects to placeholders.


  • rendering/InlineTextBox.cpp: (WebCore::InlineTextBox::paint): removed markedTextMatchesAreHighlighted guard; we always want to call paintTextMatchMarker now, but sometimes we will end up only computing the rect, not actually highlighting it. (Maybe some names should be improved here?) (WebCore::InlineTextBox::paintTextMatchMarker): Reorganized to move all the code that actually draws into a block that's guarded by markedTextMatchesAreHighlighted. The rest of the code computes where the highlight will go, and now we always use that computation in order to call setRenderedRectForMarker.
  • bridge/mac/WebCoreFrameBridge.h:
  • bridge/mac/WebCoreFrameBridge.mm: (-[WebCoreFrameBridge rectsForTextMatches]): New method, returns an array of NSValues representing NSRects. Gets them from Document::renderedRectsForMarkers
6:38 AM Changeset in webkit [14804] by sullivan
  • 2 edits in trunk/WebKit

<http://bugzilla.opendarwin.org/show_bug.cgi?id=9384>
WebView's initWithCoder: method does not set useBackForwardList
correctly


Reviewed by John Sullivan.

  • WebView/WebView.m: (-[WebView initWithCoder:]): Make sure that the function variable useBackForwardList is correctly set, so that the copy in the _private ivar is set.
3:24 AM Changeset in webkit [14803] by ap
  • 2 edits in trunk/WebKitTools

2006-06-10 Mitz Pettel <opendarwin.org@mitzpettel.com>

Reviewed and landed by ap.

  • make DumpRenderTree build
  • DumpRenderTree/DumpRenderTree.m: (-[WaitUntilDoneDelegate webView:didCommitLoadForFrame:]): Changed _updateFocusState to _updateActiveState (-[WaitUntilDoneDelegate webViewFocus:]): Ditto. (-[LayoutTestController setWindowIsKey:]): Ditto. (-[LayoutTestController setMainFrameIsFirstResponder:]): Ditto.
2:59 AM Changeset in webkit [14802] by andersca
  • 2 edits in trunk/WebCore

2006-06-10 Anders Carlsson <acarlsson@apple.com>

Reviewed by Eric.

  • bindings/js/kjs_html.cpp: (KJS::JSHTMLElement::put): Call WebCore::JSHTMLElement::put so that autogenerated setters will work for HTMLElement.

Jun 9, 2006:

9:52 PM Changeset in webkit [14801] by ggaren
  • 3 edits in trunk/JavaScriptCore

Build fix -- I think :).

9:34 PM Changeset in webkit [14800] by ggaren
  • 2 edits in trunk/WebCore
  • Build fix after last JSC check-in (oops!)


  • bridge/mac/WebCoreScriptDebugger.mm: (-[WebCoreScriptCallFrame scopeChain]): (-[WebCoreScriptCallFrame functionName]): (-[WebCoreScriptCallFrame evaluateWebScript:]):
8:57 PM Changeset in webkit [14799] by ggaren
  • 10 edits
    1 copy in trunk/JavaScriptCore

Reviewed by Eric (yay!).


  • Removed Context wrapper for ContextImp, renamed ContextImp to Context, split Context into its own file -- Context.cpp -- renamed _var to m_var, change ' *' to '* '.
  • JavaScriptCore.xcodeproj/project.pbxproj:
  • kjs/Context.cpp: Added. (KJS::Context::Context): (KJS::Context::~Context): (KJS::Context::mark):
  • kjs/context.h: (KJS::Context::scopeChain): (KJS::Context::variableObject): (KJS::Context::setVariableObject): (KJS::Context::thisValue): (KJS::Context::callingContext): (KJS::Context::activationObject): (KJS::Context::currentBody): (KJS::Context::function): (KJS::Context::arguments): (KJS::Context::pushScope): (KJS::Context::seenLabels):
  • kjs/function.cpp: (KJS::FunctionImp::callAsFunction): (KJS::FunctionImp::processParameters): (KJS::FunctionImp::argumentsGetter): (KJS::GlobalFuncImp::callAsFunction):
  • kjs/internal.cpp: (KJS::InterpreterImp::evaluate):
  • kjs/internal.h: (KJS::InterpreterImp::setContext): (KJS::InterpreterImp::context):
  • kjs/interpreter.cpp:
  • kjs/interpreter.h: (KJS::ExecState::context): (KJS::ExecState::ExecState):
  • kjs/nodes.cpp: (currentSourceId): (currentSourceURL): (ThisNode::evaluate): (ResolveNode::evaluate): (FunctionCallResolveNode::evaluate): (PostfixResolveNode::evaluate): (DeleteResolveNode::evaluate): (TypeOfResolveNode::evaluate): (PrefixResolveNode::evaluate): (AssignResolveNode::evaluate): (VarDeclNode::evaluate): (VarDeclNode::processVarDecls): (DoWhileNode::execute): (WhileNode::execute): (ForNode::execute): (ForInNode::execute): (ContinueNode::execute): (BreakNode::execute): (ReturnNode::execute): (WithNode::execute): (SwitchNode::execute): (LabelNode::execute): (TryNode::execute): (FuncDeclNode::processFuncDecl): (FuncExprNode::evaluate):
8:09 PM Changeset in webkit [14798] by justing
  • 1 edit in trunk/WebCore/ChangeLog
5:16 PM Changeset in webkit [14797] by thatcher
  • 1 copy in tags/WebCore-418.15/WebCore

Tag for WebCore 418.15 (part 2 of 2.)

5:16 PM Changeset in webkit [14796] by thatcher
  • 1 add in tags/WebCore-418.15

Tag for WebCore 418.15 (part 1 of 2.)

5:01 PM Changeset in webkit [14795] by hyatt
  • 1 edit in trunk/WebCore/page/Frame.cpp

Fix win32 build bustage.

4:37 PM Changeset in webkit [14794] by hyatt
  • 3 edits in trunk/WebKit

Rename updateFocusState to updateActiveState.

  • WebView/WebHTMLView.m: (-[WebHTMLView _updateActiveState]): (-[NSArray viewDidMoveToWindow]): (-[NSArray windowDidBecomeKey:]): (-[NSArray windowDidResignKey:]): (-[NSArray becomeFirstResponder]): (-[NSArray resignFirstResponder]): (-[WebHTMLView _formControlIsBecomingFirstResponder:]): (-[WebHTMLView _formControlIsResigningFirstResponder:]):
  • WebView/WebHTMLViewPrivate.h:
4:33 PM Changeset in webkit [14793] by thatcher
  • 1 edit in branches/Safari-2-0-branch/WebCore/khtml/xml/dom_nodeimpl.cpp

Correcting a merge problem.

4:30 PM Changeset in webkit [14792] by thatcher
  • 3 edits in branches/Safari-2-0-branch/WebCore

Merged fix from TOT to Safari-2-0-branch
This fixes the regression from <rdar://problem/4578533> REGRESSION: vbulletin search box closes when clicked

2006-02-23 Vicki Murley <vicki@apple.com>

Reviewed by Hyatt.

  • fix <rdar://problem/4456004> onbeforeunload should cancel, and <rdar://problem/4453579> REGRESSION (TOT): onbeforeunload is broken (7421)
  • bridge/mac/MacFrame.mm: (WebCore::MacFrame::shouldClose): Onbeforeunload is a non-bubbling window event, so we should be using handleWindowEvent rather than dispatching a generic event on the body. It seems this only worked before because of incorrect code in other parts of the tree.
  • dom/dom2_eventsimpl.cpp: (WebCore::BeforeUnloadEventImpl::BeforeUnloadEventImpl): for this constructor, pass true for the cancel parameter
4:26 PM Changeset in webkit [14791] by hyatt
  • 5 edits in trunk/WebCore

Rename m_isFocused on the frame to m_isActive, since it isn't really
about focus but is instead about whether or not the top-level window is
active.

Pull code that was incorrectly factored into Mac-only code out of FrameMac
and back up into Frame.

Reviewed by andersca

  • bridge/mac/FrameMac.h:
  • bridge/mac/FrameMac.mm:
  • page/Frame.cpp: (WebCore::Frame::setFocusNodeIfNeeded): (WebCore::Frame::isActive): (WebCore::Frame::setIsActive):
  • page/FramePrivate.h: (WebCore::FramePrivate::FramePrivate):
4:07 PM Changeset in webkit [14790] by hyatt
  • 1 edit in trunk/WebKit/COM/WebView.cpp

Fix Win32 build bustage.

3:04 PM Changeset in webkit [14789] by thatcher
  • 1 copy in tags/WebCore-418.14/WebCore

Tag for WebCore 418.14 (part 2 of 2.)

3:04 PM Changeset in webkit [14788] by thatcher
  • 1 add in tags/WebCore-418.14

Tag for WebCore 418.14 (part 1 of 2.)

3:03 PM Changeset in webkit [14787] by thatcher
  • 2 edits in branches/Safari-2-0-branch/WebCore

Versioning

3:01 PM Changeset in webkit [14786] by thatcher
  • 2 edits in branches/Safari-2-0-branch/WebCore

Rolling out this fix, this caused a Mail regression.
<rdar://problem/4578653> Forwarding loses the attached images

2006-04-26 Tim Omernick <timo@apple.com>


Reviewed by Hyatt.


<rdar://problem/4068375> Flash inserted via innerHTML Fails to Show when CSS Display
Style is Toggled via Javascript


  • html/html_objectimpl.cpp: (WebCore::HTMLObjectElement::setComplete): Set needWidgetUpdate when finished parsing, even if the object element is not in a document. That way, when the element attaches to a document, it will update its widget (creating the plug-in view if necessary). This is important when the object is being inserted via setInnerHTML, since the parsed nodes are not added to the document until the whole HTML string is parsed.
2:25 PM Changeset in webkit [14785] by thatcher
  • 1 edit in trunk/WebKitSite/coding/bsd-license.html

Removes some control characters that snuck in.

1:20 PM Changeset in webkit [14784] by hyatt
  • 34 edits in trunk

WebCore:

Rework selection coloring. Rename displaysWithFocusAttributes
to isActive. Move the white-blending code onto the Color API to make
it more convenient to mutate colors for selection blending. Eliminate all
the selection state from the GraphicsContext and move it to the RenderTheme.
Implement both Win32 and Mac theme selection colors.

Reviewed by sfalken

  • bridge/mac/FrameMac.h:
  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::setIsActive):
  • bridge/mac/WebCoreFrameBridge.h:
  • bridge/mac/WebCoreFrameBridge.mm: (-[WebCoreFrameBridge drawRect:]): (-[WebCoreFrameBridge setIsActive:]): (-[WebCoreFrameBridge selectionColor]):
  • css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::checkOneSelector):
  • page/Frame.cpp: (WebCore::Frame::isActive): (WebCore::Frame::setIsActive):
  • page/Frame.h:
  • platform/Color.cpp: (WebCore::blend): (WebCore::Color::blendWithWhite):
  • platform/Color.h: (WebCore::Color::hasAlpha):
  • platform/GraphicsContext.cpp: (WebCore::GraphicsContextPrivate::GraphicsContextPrivate):
  • platform/GraphicsContext.h:
  • platform/mac/GraphicsContextMac.mm:
  • platform/win/TemporaryLinkStubs.cpp: (GraphicsContext::endTransparencyLayer):
  • rendering/InlineTextBox.cpp: (WebCore::InlineTextBox::paintSelection):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::fillHorizontalSelectionGap): (WebCore::RenderBlock::fillVerticalSelectionGap): (WebCore::RenderBlock::fillLeftSelectionGap): (WebCore::RenderBlock::fillRightSelectionGap):
  • rendering/RenderHTMLCanvas.cpp: (WebCore::RenderHTMLCanvas::paint):
  • rendering/RenderImage.cpp: (WebCore::RenderImage::paint):
  • rendering/RenderListMarker.cpp: (WebCore::RenderListMarker::paint):
  • rendering/RenderListMarker.h:
  • rendering/RenderObject.cpp: (WebCore::RenderObject::selectionColor):
  • rendering/RenderObject.h:
  • rendering/RenderReplaced.cpp:
  • rendering/RenderReplaced.h:
  • rendering/RenderTheme.cpp: (WebCore::RenderTheme::activeSelectionColor): (WebCore::RenderTheme::inactiveSelectionColor): (WebCore::RenderTheme::platformActiveSelectionColor): (WebCore::RenderTheme::platformInactiveSelectionColor):
  • rendering/RenderTheme.h:
  • rendering/RenderThemeMac.h:
  • rendering/RenderThemeMac.mm: (WebCore::RenderThemeMac::platformActiveSelectionColor): (WebCore::RenderThemeMac::platformInactiveSelectionColor):
  • rendering/RenderThemeWin.cpp: (WebCore::RenderThemeWin::platformActiveSelectionColor): (WebCore::RenderThemeWin::platformInactiveSelectionColor):
  • rendering/RenderThemeWin.h:
  • rendering/RenderWidget.cpp: (WebCore::RenderWidget::paint):

WebKit:

Rename displaysWithFocusAttributes to isActive.

Reviewed by sfalken

  • WebKit.xcodeproj/project.pbxproj:
  • WebView/WebHTMLView.m: (-[WebHTMLView _updateFocusState]):
1:14 PM Changeset in webkit [14783] by thatcher
  • 5 edits in branches/Safari-2-0-branch/WebCore

Merged fix from TOT to Safari-2-0-branch
This fixes <rdar://problem/4578533> REGRESSION: vbulletin search box closes when clicked

2006-02-08 Adele Peterson <adele@apple.com>

Reviewed by Maciej.

  • Fixed <rdar://problem/4325160> REGRESSION(416.12-420+): window events not stopped by stopPropagation (affects macnn forums)

Tests:

  • fast/events/window-events-bubble.html
  • fast/events/window-events-bubble2.html
  • fast/events/window-events-capture.html
  • khtml/xml/DocumentImpl.cpp: (WebCore::DocumentImpl::handleWindowEvent): Added to send events directly to the window. This code used to be in defaultEventHandler, but that wasn't sending them at the right time. (WebCore::DocumentImpl::defaultEventHandler): Moved code to send window events to handleWindowEvent.
  • khtml/xml/DocumentImpl.h: Added handleWindowEvent.
  • khtml/xml/NodeImpl.cpp: (WebCore::NodeImpl::dispatchGenericEvent): fire window events in capture and bubble phases. (WebCore::NodeImpl::dispatchWindowEvent): call handleWindowEvent.
12:14 PM Changeset in webkit [14782] by justing
  • 10 edits
    4 adds in trunk

LayoutTests:

Reviewed by levi


<rdar://problem/4549980>
REGRESSION: "Find Again" can get stuck when searching for string with a trailing space

  • editing/execCommand/findString-expected.checksum: Added.
  • editing/execCommand/findString-expected.png: Added.
  • editing/execCommand/findString-expected.txt: Added.
  • editing/execCommand/findString.html: Added.

WebCore:

Reviewed by levi


<rdar://problem/4549980>
REGRESSION: "Find Again" can get stuck when searching for string with a trailing space

  • bridge/mac/FrameMac.h: Moved findString to Frame.
  • bridge/mac/FrameMac.mm: Ditto.
  • bridge/mac/WebCoreFrameBridge.mm: (-[WebCoreFrameBridge searchFor:direction:caseSensitive:wrap:]): Convert the NSString to a String.
  • editing/JSEditor.cpp: Added execCommand(FindString, ...)
  • editing/Selection.cpp: Added a constructor for Ranges (WebCore::Selection::Selection):
  • editing/Selection.h:
  • page/Frame.cpp: (WebCore::Frame::findString): Moved from FrameMac. Compare a selection created using the found range with the current selection in case the current selection is the found range minus some collapsed whitespace on the edges.
  • page/Frame.h:
12:06 PM Changeset in webkit [14781] by ggaren
  • 3 edits in trunk/WebKitTools

Reviewed by Darin, landed by Geoff.

http://bugzilla.opendarwin.org/show_bug.cgi?id=9350
Use pathcmp() when sorting paths in svn-create-patch

  • Scripts/run-webkit-tests: Fixed splitpath() to use File::Basename subroutines instead of regex.
  • Scripts/svn-create-patch: Copied numericcmp(), pathcmp() and splitpath() from run-webkit-tests. Changed sort() functions to use pathcmp(). Added subroutine prototypes. Added -h command-line switch and printUsage() subroutine.
11:40 AM Changeset in webkit [14780] by sfalken
  • 4 edits in trunk/WebCore

2006-06-09 Steve Falkenburg <sfalken@apple.com>

Fix build break

  • WebCore.vcproj/WebCore/WebCore.vcproj:
  • css/MediaQueryEvaluator.cpp:
  • platform/win/TemporaryLinkStubs.cpp: (GraphicsContext::addRoundedRectClip): (GraphicsContext::addInnerRoundedRectClip): (WebCore::screenDepthPerComponent): (WebCore::screenIsMonochrome):
10:09 AM Changeset in webkit [14779] by bdakin
  • 23 edits
    234 adds in trunk

Reviewed by Hyatt. Tweaked by Maciej. Tweaks reviewed by Beth.
Landed by Beth.

CSS3 Media Queries implementation.
<http://bugzilla.opendarwin.org/show_bug.cgi?id=4127>

  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/kjs_css.cpp: (KJS::DOMMediaList::put): (KJS::KJS::DOMMediaListProtoFunc::callAsFunction):
  • bindings/objc/DOMCSS.mm: (-[DOMMediaList setMediaText:]): (-[DOMMediaList deleteMedium:]): (-[DOMMediaList appendMedium:]):
  • css/CSSGrammar.y:
  • css/MediaFeatureNames.cpp: Added. (WebCore::MediaFeatureNames::init):
  • css/MediaFeatureNames.h: Added.
  • css/MediaList.cpp: (WebCore::MediaList::MediaList): (WebCore::MediaList::~MediaList): (WebCore::parseMediaDescriptor): (WebCore::MediaList::deleteMedium): (WebCore::MediaList::mediaText): (WebCore::MediaList::setMediaText): (WebCore::MediaList::item): (WebCore::MediaList::appendMedium): (WebCore::MediaList::appendMediaQuery):
  • css/MediaList.h: (WebCore::MediaList::MediaList): (WebCore::MediaList::length): (WebCore::MediaList::mediaQueries):
  • css/MediaQuery.cpp: Added. (WebCore::MediaQuery::MediaQuery): (WebCore::MediaQuery::~MediaQuery): (WebCore::MediaQuery::operator==): (WebCore::MediaQuery::cssText):
  • css/MediaQuery.h: Added. (WebCore::MediaQuery::): (WebCore::MediaQuery::restrictor): (WebCore::MediaQuery::expressions): (WebCore::MediaQuery::mediaType): (WebCore::MediaQuery::append):
  • css/MediaQueryEvaluator.cpp: Added. (WebCore::): (WebCore::MediaQueryEvaluator): (WebCore::MediaQueryEvaluator::~MediaQueryEvaluator): (WebCore::MediaQueryEvaluator::mediaTypeMatch): (WebCore::applyRestrictor): (WebCore::MediaQueryEvaluator::eval): (WebCore::parseAspectRatio): (WebCore::cmpvalue): (WebCore::numberValue): (WebCore::colorMediaFeatureEval): (WebCore::monochromeMediaFeatureEval): (WebCore::device_aspect_ratioMediaFeatureEval): (WebCore::gridMediaFeatureEval): (WebCore::device_heightMediaFeatureEval): (WebCore::device_widthMediaFeatureEval): (WebCore::heightMediaFeatureEval): (WebCore::widthMediaFeatureEval): (WebCore::min_colorMediaFeatureEval): (WebCore::max_colorMediaFeatureEval): (WebCore::min_monochromeMediaFeatureEval): (WebCore::max_monochromeMediaFeatureEval): (WebCore::min_device_aspect_ratioMediaFeatureEval): (WebCore::max_device_aspect_ratioMediaFeatureEval): (WebCore::min_heightMediaFeatureEval): (WebCore::max_heightMediaFeatureEval): (WebCore::min_widthMediaFeatureEval): (WebCore::max_widthMediaFeatureEval): (WebCore::min_device_heightMediaFeatureEval): (WebCore::max_device_heightMediaFeatureEval): (WebCore::min_device_widthMediaFeatureEval): (WebCore::max_device_widthMediaFeatureEval): (WebCore::createFunctionMap):
  • css/MediaQueryEvaluator.h: Added.
  • css/MediaQueryExp.cpp: Added. (WebCore::MediaQueryExp::MediaQueryExp): (WebCore::MediaQueryExp::~MediaQueryExp):
  • css/MediaQueryExp.h: Added. (WebCore::MediaQueryExp::mediaFeature): (WebCore::MediaQueryExp::value): (WebCore::MediaQueryExp::operator==):
  • css/StyleSheet.cpp: (WebCore::StyleSheet::setMedia):
  • css/cssparser.cpp: (WebCore::CSSParser::CSSParser): (WebCore::CSSParser::~CSSParser): (WebCore::CSSParser::setupParser): (WebCore::CSSParser::parseMediaQuery): (WebCore::CSSParser::createFloatingMediaQueryExp): (WebCore::CSSParser::sinkFloatingMediaQueryExp): (WebCore::CSSParser::createFloatingMediaQueryExpList): (WebCore::CSSParser::sinkFloatingMediaQueryExpList): (WebCore::CSSParser::createFloatingMediaQuery): (WebCore::CSSParser::sinkFloatingMediaQuery):
  • css/cssparser.h:
  • css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::CSSStyleSelector): (WebCore::CSSStyleSelector::init): (WebCore::CSSStyleSelector::~CSSStyleSelector): (WebCore::CSSStyleSelector::loadDefaultStyle): (WebCore::CSSStyleSelector::matchUARules): (WebCore::CSSStyleSelector::styleForElement): (WebCore::CSSStyleSelector::pseudoStyleForElement): (WebCore::CSSStyleSelector::updateFont): (WebCore::CSSStyleSelector::cacheBorderAndBackground): (WebCore::CSSStyleSelector::styleRulesForElement): (WebCore::CSSRuleSet::addRulesFromSheet):
  • css/cssstyleselector.h:
  • css/maketokenizer:
  • css/tokenizer.flex:
  • dom/DOMImplementation.cpp: (WebCore::DOMImplementation::createCSSStyleSheet):
  • html/HTMLLinkElement.cpp: (WebCore::HTMLLinkElement::process): (WebCore::HTMLLinkElement::setStyleSheet):
  • html/HTMLStyleElement.cpp: (WebCore::HTMLStyleElement::childrenChanged):
  • ksvg2/svg/SVGDOMImplementation.cpp: (SVGDOMImplementation::createCSSStyleSheet):
  • ksvg2/svg/SVGStyleElement.cpp: (WebCore::SVGStyleElement::childrenChanged):
  • page/Frame.cpp: (WebCore::Frame::Frame):
  • platform/Screen.h:
  • platform/mac/ScreenMac.mm: (WebCore::screenDepthPerComponent): (WebCore::screenIsMonochrome):
Note: See TracTimeline for information about the timeline view.