Timeline



Apr 6, 2006:

11:51 PM Changeset in webkit [13784] by eseidel
  • 3 edits in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs

fix try/catch and forIn unwind handling as well as error throwing

10:34 PM Changeset in webkit [13783] by ggaren
  • 2 edits in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs
10:25 PM Changeset in webkit [13782] by eseidel
  • 1 edit in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs/TreeCode.cpp

fix AssignDotNodeEvaluateState

10:13 PM Changeset in webkit [13781] by ggaren
  • 4 edits in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs

loops work now -- mostly :)

9:52 PM Changeset in webkit [13724] by darin
  • 1 edit in trunk/WebKitTools/WebKitLauncher/WebKitLauncher.xcodeproj

Add the ignore property for inside this project.

9:06 PM Changeset in webkit [13780] by eseidel
  • 4 edits in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs

make value returns use a single value instead of the stack

6:21 PM Changeset in webkit [13779] by eseidel
  • 1 edit in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs/TreeCode.cpp

trying to find value leak

5:02 PM Changeset in webkit [13778] by eseidel
  • 3 edits in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs

Fix returns and completions

2:10 PM Changeset in webkit [13777] by eseidel
  • 3 edits in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs

add printStacks()

2:09 PM Changeset in webkit [13776] by ggaren
  • 1 edit in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs/completion.h

prevent the All completion mask from matching Scope, since Scope isn't a real completion

1:53 PM Changeset in webkit [13775] by ggaren
  • 1 edit in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs/internal.cpp

unwind barrier debug print function was using the wrong variable

1:46 PM Changeset in webkit [13774] by ggaren
  • 1 add in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs/TreeCode.h

forgot to add this file

1:44 PM Changeset in webkit [13773] by ggaren
  • 7 edits in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore

random bug fixes, value stack printouts are now more informative

12:36 PM Changeset in webkit [13772] by ggaren
  • 2 edits in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs

integrated for-in loop. everything compiles, but we still die bbadbeef-style on all js runs.

11:45 AM Changeset in webkit [13723] by bdakin
  • 9 edits
    4 adds in trunk

Reviewed by Darin.

Fix for ​http://bugzilla.opendarwin.org/show_bug.cgi?id=8203
REGRESSION: "Invite a friend" text field in GMail page spills out
of table

RenderReplaced::calcMinMaxWidth() sets m_minWidth to 0 when the
width is a percent, so this patch copies that behavior into
RenderTextField::calcMinMaxWidth().

4:06 AM Changeset in webkit [13771] by eseidel
  • 3 edits in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs

the beast

3:29 AM Changeset in webkit [13770] by ggaren
  • 2 edits in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs

make ReferenceList iterators hold on to their lists so that we can keep an iterator stack without having the underlying lists get destroyed

1:58 AM Changeset in webkit [13769] by eseidel
  • 4 edits in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs

add node stack and forIn

12:37 AM Changeset in webkit [13768] by eseidel
  • 5 edits in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs

go ggaren go

12:26 AM Changeset in webkit [13722] by darin
  • 2 edits in trunk/WebCore
  • try to fix Windows build
  • WebCore.vcproj/WebCore/WebCore.vcproj: Add JSDocument.cpp.
12:23 AM Changeset in webkit [13721] by darin
  • 5 edits in trunk/WebKitTools

Changes requested by Mark Rowe.

  • BuildSlaveSupport/build-launcher-app: Set executable bit, removed property change trash at the end of the file.
  • BuildSlaveSupport/build-launcher-dmg: Ditto.
  • WebKitLauncher/main.m: Removed trash at end of file.
  • WebKitLauncher/WebKitLauncher.xcodeproj/project.pbxproj: Let Xcode update comments to make name match.
12:11 AM Changeset in webkit [13720] by darin
  • 4 edits in trunk/LayoutTests
  • regenerated test results
  • fast/forms/input-table-expected.txt: Regenerated. Seems right, matches Gecko.
  • fast/forms/form-element-geometry-expected.png: Regenerated.
  • fast/forms/form-element-geometry-expected.checksum: Regenerated.
12:02 AM Changeset in webkit [13719] by darin
  • 3 edits in trunk/JavaScriptCore
  • attempt to fix Windows build
  • kxmlcore/HashMap.h: (KXMLCore::HashMap::remove): Use (*it). instead of it->.
  • kxmlcore/HashSet.h: (KXMLCore::HashSet::remove): Ditto.

Apr 5, 2006:

11:41 PM Changeset in webkit [13718] by darin
  • 2 edits in trunk/LayoutTests
  • fast/forms/input-truncate-newline-expected.txt: Restored good expected results.
11:07 PM Changeset in webkit [13717] by darin
  • 2 edits in trunk/JavaScriptCore
  • attempt to fix Windows build
  • os-win32/stdint.h: Add int8_t, uint8_t, int64_t.
11:01 PM Changeset in webkit [13716] by darin
  • 2 edits in trunk/WebKit
  • fixed a storage leak from that last check-in
  • WebView/WebUnarchivingState.m: (-[WebUnarchivingState dealloc]): Release the two dictionaries.
10:57 PM Changeset in webkit [13715] by darin
  • 19 edits
    5 adds in trunk

LayoutTests:

Reviewed by Adele.

  • updated results for improved dumping of selection location in DumpRenderTree
  • fast/forms/input-text-self-emptying-click-expected.checksum: Added.
  • fast/forms/input-text-self-emptying-click-expected.png: Added.
  • fast/forms/input-text-self-emptying-click-expected.txt: Added.
  • fast/forms/input-text-self-emptying-click.html: Added.
  • editing/selection/3690703-2-expected.txt: Updated.
  • fast/forms/input-appearance-focus-expected.txt: Updated.
  • fast/forms/input-appearance-readonly-expected.txt: Updated.
  • fast/forms/input-appearance-selection-expected.txt: Updated.
  • fast/forms/input-paste-undo-expected.txt: Updated.
  • fast/forms/input-text-double-click-expected.txt: Updated.
  • fast/forms/input-text-option-delete-expected.txt: Updated.
  • fast/forms/input-text-scroll-left-on-blur-expected.txt: Updated.
  • fast/forms/plaintext-mode-2-expected.txt: Updated.

WebCore:

Reviewed by Adele.

Test: fast/forms/input-text-self-emptying-click.html

  • page/MouseEventWithHitTestResults.h: Removed url, target, m_url and m_target, replacing them with isOverLink and m_isOverLink. Replaced innerNode with targetNode, and added logic to handle the case where the target node is removed from the document but the element the target node was in is still inside the document.
  • page/MouseEventWithHitTestResults.cpp: Added.
  • WebCore.xcodeproj/project.pbxproj: Added MouseEventWithHitTestResults.cpp.
  • WebCore.vcproj/WebCore/WebCore.vcproj: Ditto.
  • kwq/RenderTreeAsText.cpp: (nodePosition): Added a call to shadowParentNode so we correctly dump positions within shadow trees. This was needed to give a good result from my new test for this bug.
  • dom/Document.cpp: (WebCore::Document::prepareMouseEvent): Update to pass fewer parameters to the constructor for MouseEventWithHitTestResults. Now takes isOverLink boolean, and no longer takes href and target parameters.
  • page/Frame.cpp: (WebCore::Frame::handleMousePressEventDoubleClick): Use new name targetNode, instead of old name innerNode. (WebCore::Frame::handleMousePressEventTripleClick): Ditto. (WebCore::Frame::handleMousePressEventSingleClick): Ditto. Also change code to check if over a link to use !isOverLink instead of url.isNull. (WebCore::Frame::handleMousePressEvent): Ditto. Also remove unused "url" local variable. (WebCore::Frame::handleMouseMoveEvent): Ditto. (WebCore::Frame::handleMouseReleaseEvent): Ditto. (WebCore::Frame::passWidgetMouseDownEventToWidget): Ditto.
  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::handleMousePressEvent): Use new name targetNode, instead of old name innerNode. (WebCore::FrameMac::handleMouseMoveEvent): Ditto. (WebCore::FrameMac::passSubframeEventToSubframe): Ditto. (WebCore::FrameMac::sendContextMenuEvent): Ditto.
  • page/FrameView.cpp: (WebCore::subframeForEvent): Use new name targetNode, instead of old name innerNode. (WebCore::FrameView::handleMousePressEvent): Ditto. (WebCore::FrameView::handleMouseDoubleClickEvent): Ditto. (WebCore::selectCursor): Ditto. Also change code to check if over a link to use isOverLink instead of !url.isNull. (WebCore::FrameView::handleMouseMoveEvent): Ditto. (WebCore::FrameView::handleMouseReleaseEvent): Ditto. (WebCore::FrameView::updateDragAndDrop): Ditto.
9:49 PM Changeset in webkit [13714] by mjs
  • 7 edits
    2 adds in trunk/WebKit

Reviewed by Eric.



  • WebKit.xcodeproj/project.pbxproj:
  • WebView/WebDataSource.m: (-[WebDataSourcePrivate dealloc]): (-[WebDataSource _addToUnarchiveState:]): (-[WebDataSource _popSubframeArchiveWithName:]): (-[WebDataSource _documentFragmentWithArchive:]): (-[WebDataSource _setupForReplaceByMIMEType:]): (-[WebDataSource subresourceForURL:]):
  • WebView/WebDataSourcePrivate.h:
  • WebView/WebFrame.m: (-[WebFrame _loadRequest:archive:]): (-[WebFrame loadRequest:]): (-[WebFrame loadArchive:]):
  • WebView/WebFramePrivate.h:
  • WebView/WebHTMLRepresentation.m: (-[WebHTMLRepresentation loadArchive]):
  • WebView/WebUnarchivingState.h: Added.
  • WebView/WebUnarchivingState.m: Added. (-[WebUnarchivingState init]): (-[WebUnarchivingState addArchive:]): (-[WebUnarchivingState archivedResourceForURL:]): (-[WebUnarchivingState popSubframeArchiveWithFrameName:]):
9:44 PM Changeset in webkit [13713] by mjs
  • 2 edits in trunk

Fixed date in ChangeLog entries for last checkin.

9:40 PM Changeset in webkit [13712] by mjs
  • 4 edits in trunk

LayoutTests:

Reviewed and landed by Maciej.

  • fast/clip/outline-overflowClip-expected.txt: Reverted to the correct expected result.

WebCore:

Reviewed and landed by Maciej.

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::calculateRects): Reverted this part of the fix for bug 7943. The outlineRect should not be inflated.
9:36 PM Changeset in webkit [13711] by mjs
  • 3 edits
    1 add in trunk

LayoutTests:

Reviewed by Eric, landed by Maciej.

  • svg/custom/class-selector.svg: Added.

WebCore:

Reviewed by Eric, landed by Maciej.

Fix for ​http://bugzilla.opendarwin.org/show_bug.cgi?id=7627:
SVG from the W3C SVG 1.1 test suite (cubic01.svg) renders as all black

Fix this svg stylesheet to use strict-mode, as this makes sure
class selecting is done correctly.

  • ksvg2/svg/SVGStyleElement.cpp: (SVGStyleElement::childrenChanged):
9:14 PM Changeset in webkit [13710] by mjs
  • 3 edits
    10 adds in trunk/WebKitTools

Reviewed by Darin, landed by Maciej.

  • BuildSlaveSupport/build-launcher-app: Added. Builds WebKit.app from WebKitLauncher and bundles the WebKit frameworks inside it.
  • BuildSlaveSupport/build-launcher-dmg: Added. Builds, and optionally uploads, a disk image containing WebKit.app.
  • BuildSlaveSupport/run-performance-tests: Use currentSVNRevision.
  • Scripts/webkitdirs.pm: Add currentSVNRevision to retrieve the revision number of the SVN working copy.
  • WebKitLauncher: Added.
  • WebKitLauncher/Info.plist: Added.
  • WebKitLauncher/WebKitLauncher.xcodeproj: Added.
  • WebKitLauncher/WebKitLauncher.xcodeproj/project.pbxproj: Added.
  • WebKitLauncher/WebKitNightlyEnabler.m: Added. This makes up the WebKitNightlyEnabler dylib which is loaded into Safari's address space to cause LaunchServices to treat Safari.app as WebKit.app when it is run from the nightly launcher.
  • WebKitLauncher/main.m: Added. The WebKit.app launcher. It sets up the environment to have Safari use the bundled frameworks and load the WebKitNightlyEnabler dylib before exec'ing Safari
  • WebKitLauncher/start.html: Added.
  • WebKitLauncher/webkit.icns: Added.
7:31 PM Changeset in webkit [13767] by eseidel
  • 8 edits in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs

merge

6:59 PM Changeset in webkit [13766] by eseidel
  • 8 edits in branches/temp-tree-code/trunk/JavaScriptCore/kjs

commit attempt

6:52 PM Changeset in webkit [13765] by ggaren
  • 1 copy in branches/temp-tree-code/branches/evaluate-execute-merge

created branch for evaluate-execute merge

5:13 PM Changeset in webkit [13709] by darin
  • 2 edits in trunk/JavaScriptCore

Reviewed by Maciej.

  • fix memory leak introduced by the previous change
  • kxmlcore/HashTable.h: Specialize NeedsRef so that it correctly returns true when the value in question is a pair where one of the pair needs a ref and the other of the pair does not.
5:00 PM Changeset in webkit [13708] by justing
  • 2 edits in trunk/WebCore

Reviewed by mjs

I forgot to convert one of the implicit remove/inserts to an explicit remove/insert,
so we were hitting the assert I just added to InsertNodeBeforeCommand that checked for
implicit removes.

  • editing/ReplaceSelectionCommand.cpp: (WebCore::ReplaceSelectionCommand::doApply):
3:43 PM Changeset in webkit [13707] by justing
  • 1 edit in trunk/WebCore/editing/InsertNodeBeforeCommand.cpp

Should fix the build

3:26 PM Changeset in webkit [13706] by mjs
  • 23 edits
    1 add in trunk

LayoutTests:

Reviewed by Anders.

Update prototype chain for the following fix:

  • fast/dom/prototype-chain-expected.txt:

WebCore:

Reviewed by Anders.

  • also removed document.actualEncoding since it is not in any spec or implemented by any other browser


  • DerivedSources.make: add JSDocument.h to results
  • WebCore.xcodeproj/project.pbxproj: Added new files to project
  • bindings/scripts/CodeGeneratorJS.pm: Support for new stuff needed by Document.
  • dom/Document.cpp: (WebCore::Document::readyState): moved impl here from JS bindings (WebCore::Document::inputEncoding): ditto (WebCore::Document::defaultCharset): ditto (WebCore::Document::setCharset): ditto
  • dom/Document.h: (WebCore::Document::charset): added, synonym for inputEncoding. (WebCore::Document::characterSet): ditto
  • dom/Document.idl: Added. Full interface for the Document object.
  • khtml/ecma/JSXMLHttpRequest.cpp: (KJS::JSXMLHttpRequestProtoFunc::callAsFunction): JSDocument, not DOMDocument
  • khtml/ecma/JSXSLTProcessor.cpp: (KJS::XSLTProcessorProtoFunc::callAsFunction): ditto
  • khtml/ecma/kjs_binding.cpp: (KJS::jsStringOrFalse): Added this convenience for the method on Document that bizzarely returns false on failure and a string otherwise.
  • khtml/ecma/kjs_binding.h:
  • khtml/ecma/kjs_css.cpp: (KJS::toJS): renamed for consistency
  • khtml/ecma/kjs_css.h:
  • khtml/ecma/kjs_dom.cpp:
  • removed all traces of DOMDocument (KJS::toJS): JSDocument, not DOMDocument
  • khtml/ecma/kjs_dom.h:
  • khtml/ecma/kjs_html.cpp: (KJS::): (KJS::JSHTMLDocument::JSHTMLDocument): inherit from JSDocument (KJS::JSHTMLDocument::getOwnPropertySlot): ditto (KJS::JSHTMLDocument::put): ditto
  • khtml/ecma/kjs_html.h:
  • khtml/ecma/kjs_traversal.cpp: (KJS::toJS): added overloads (KJS::toNodeFilter): handle JS functions as well as impl NodeFilter objects
  • khtml/ecma/kjs_traversal.h:
  • platform/AtomicString.h: (WebCore::AtomicString::AtomicString): Allow implicit conversion from String.
2:53 PM Changeset in webkit [13705] by justing
  • 6 edits
    4 adds in trunk

LayoutTests:

Reviewed by darin


<​http://bugzilla.opendarwin.org/show_bug.cgi?id=8198>
Hitting an assert on undo paste

  • editing/pasteboard/undoable-fragment-removes-expected.checksum: Added.
  • editing/pasteboard/undoable-fragment-removes-expected.png: Added.
  • editing/pasteboard/undoable-fragment-removes-expected.txt: Added.
  • editing/pasteboard/undoable-fragment-removes.html: Added.

WebCore:

Reviewed by darin


<​http://bugzilla.opendarwin.org/show_bug.cgi?id=8198>
Hitting an assert on undo paste


ReplaceSelectionCommand was doing a combination of undoable and non-undoable
removes from the ReplacementFragment. On Undo Paste, the undoable removes
couldn't be undone because the tree was in a different state than it was
at the time of the remove. This patch makes all the removes from the fragment
non-undoable. We could make them all undoable, but I can't think of any reason
why we'd want the fragment to be reconstructed on an Undo Paste.

  • editing/AppendNodeCommand.cpp: (WebCore::AppendNodeCommand::doApply): Assert that the node to append isn't already in a tree, since if it is, it will be removed in a non-undoable way.
  • editing/InsertNodeBeforeCommand.cpp: (WebCore::InsertNodeBeforeCommand::doApply): Ditto.
  • editing/ReplaceSelectionCommand.cpp: (WebCore::ReplaceSelectionCommand::doApply): Nodes were being moved from the fragment to the document with undoable inserts. Undoable inserts implicitly remove the node (in a non-undoable way) from its old location if it is already in a tree. I now explicitly remove the nodes from the fragment before inserting them into the document to make it clear that they are being removed in a non-non-undoable way. I also changed the one undoable remove from the fragment to a non-undoable remove.
  • editing/ReplaceSelectionCommand.h: Made ReplacementFragment's non-undoable removeNode public.
2:51 PM Changeset in webkit [13704] by darin
  • 2 edits in trunk/WebCore
  • fixed the build
  • WebCore.xcodeproj/project.pbxproj: Oops! Resolved merge conflict.
2:19 PM Changeset in webkit [13703] by darin
  • 34 edits
    2 adds
    1 delete in trunk

JavaScriptCore:

Reviewed by Maciej.

Change HashMap and HashSet implementation so they fold various types together.
This allows us to implement maps and sets that use RefPtr<WebCore::StringImpl>
and WebCore::String in terms of the underlying raw pointer type, and hence use
-1 for the deleted value.

  • kxmlcore/HashTraits.h: Added a new type to HashTraits, StorageTraits, which is a type to be used when storing a value that has the same layout as the type itself. This is used only for non-key cases. In the case of keys, the hash function must also be considered. Moved emptyValue out of GenericHashTraitsBase into GenericHashTraits. Added a new bool to HashTraits, needsRef, which indicates whether the type needs explicit reference counting. If the type itself has needsRef true, but the storage type has needsRef false, then the HashSet or HashMap has to handle the reference counting explicitly. Added hash trait specializations for all signed integer values that give -1 as the deleted value. Gave all integers StorageTraits of the canonical integer type of the same size so int and long will share code. Gave all pointers and RefPtrs StorageTraits of the appropriately sized integer type. Removed redundant TraitType and emptyValue definitions in the pointer specialization for HashTraits. Added PairBaseHashTraits, which doesn't try to set up needsDestruction and deletedValue. Useful for types where we don't want to force the existence of deletedValue, such as the type of a pair in a HashMap which is not the actual storage type. Removed an unneeded parameter from the DeletedValueAssigner template. Added HashKeyStorageTraits template, which determines what type can be used to store a given hash key type with a given hash function, and specialized it for pointers and RefPtr so that pointer hash tables share an underlying HashTable that uses IntHash.
  • kxmlcore/HashTable.h: Added HashTableConstIteratorAdapter, HashTableIteratorAdapter, NeedsRef, RefCountManagerBase, RefCountManager, HashTableRefCountManagerBase, and HashTableRefCountManager. All are used by both HashSet and HashMap to handle hash tables where the type stored is not the same as the real value type.


  • kxmlcore/HashFunctions.h: Added a new struct named IntTypes that finds an integer type given a sizeof value. Renamed pointerHash to intHash and made it use overloading and take integer parameters. Added an IntHash struct which is a hash function that works for integers. Changed PtrHash to call IntHash with an appropriately sized integer. Made IntHash the default hash function for many integer types. Made PtrHash the default hash function for RefPtr as well as for raw pointers.
  • kxmlcore/HashSet.h: Changed implementation to use a separate "storage type" derived from the new traits. The HashTable will use the storage type and all necessary translation and ref/deref is done at the HashSet level. Also reorganized the file so that the HashSet is at the top and has no inline implementation inside it so it's easy to read the interface to HashSet.
  • kxmlcore/HashMap.h: Changed implementation to use a separate "storage type" derived from the new traits. The HashTable will use the storage type and all necessary translation and ref/deref is done at the HashMap level. Also reorganized the file so that the HashMap is at the top and has no inline implementation inside it so it's easy to read the interface to HashMap.
  • kxmlcore/HashMapPtrSpec.h: Removed. Superceded by optimizations in HashMap itself.
  • JavaScriptCore.xcodeproj/project.pbxproj: Remove HashMapPtrSpec.h, resort files, and also remove some unnecessary build settings from the aggregate target that generates derived sources.
  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj: Ditto.

WebCore:

Reviewed by Maciej.

  • platform/StringHash.h: Added. Moved hash functions and such for WebCore::String and friends into this file so we don't have to include the hash traits header everywhere. Changed hashing for WebCore::StringImpl and WebCore::String so that they use a raw pointer for the underlying storage type, taking advantage of the new feature added in JavaScriptCore.
  • platform/AtomicString.h: Moved StrHash specialization to StringHash.h.
  • platform/PlatformString.h: Moved StrHash specialization to StringHash.h.
  • platform/StringImpl.h: Moved StrHash, CaseInsensitiveHash, and HashTraits to StringHash.h. Left DefaultHash behind so that you can't get the wrong hash function by accident if you forget to include "StringHash.h".
  • platform/StringImpl.cpp: Added include of StringHash.h and removed RefPtr<StringImpl> HashTraits<RefPtr<StringImpl> >::_deleted, which is the object with a global initializer causing all the trouble!
  • kwq/AccessibilityObjectCache.h: Changed hash function to be IntHash instead of PtrHash.
  • dom/StyledElement.cpp: Changed MappedAttributeKeyTraits to inherit from the generic traits in KXMLCore so we get a StorageType. Also cleaned up a tiny bit by adding default values to the MappedAttributeKey constructor.
  • platform/CharsetNames.cpp: Changed hash traits here to be a new TextEncodingIDHashTraits struct rather than defining new default traits for the integer type since more integer types have default traits in HashTraits.h now. Also added a specialization so this class will share the underlying implementation (since InvalidEncoding happens to be -1).
  • bridge/mac/FrameMac.h:
  • dom/Document.h:
  • dom/xml_tokenizer.h:
  • khtml/xsl/XSLTProcessor.h:
  • kwq/JavaAppletWidget.h:
  • page/FramePrivate.h:
  • page/Page.cpp:
  • platform/AtomicString.cpp:
  • platform/TransferJob.h:
  • rendering/render_applet.h: Added include of StringHash.h.
  • WebCore.xcodeproj/project.pbxproj: Added StringHash.h. Remove unneeded CREATE_HASH_TABLE variable in build settings. Re-sorted some file lists. Added quotes to the CREATE_HASH_TABLE initialization in the rule that builds generated files. Removed various unneeded build settings for that target as well.
  • ForwardingHeaders/kxmlcore/HashTraits.h: Added.
  • other minor cleanup
  • bridge/mac/FrameMac.mm: Sorted includes.
  • dom/Node.cpp: Removed bogus symbol after #endif.
  • khtml/xsl/XSLTProcessor.cpp: Sorted includes. Removed redundant using namespace WebCore.
  • loader/Cache.cpp: Ditto.

WebKitTools:

Reviewed by Maciej.

  • Scripts/check-for-global-initializers: Remove StringImpl from the list of files that are allowed to have global initializers.
1:28 PM Changeset in webkit [13764] by ggaren
  • 4 edits in branches/temp-tree-code/trunk/JavaScriptCore/kjs

Oops. Committed wrong patch. This is with-stack.patch

1:09 PM Changeset in webkit [13763] by ggaren
  • 5 edits in branches/temp-tree-code/trunk/JavaScriptCore/kjs

Committing Eric's patch to unify how nodes know their interpreter state -- with minor tweaks to the ASSERTs. The unified method is for nodes to use a data member rather than virtual method dispatch. This is about a 7% performance win.

11:05 AM Changeset in webkit [13702] by bdakin
  • 2 edits in trunk/WebCore

Reviewed by Darin.

Fix for <rdar://problem/4502311> text-transform:capitalize needs to
treat nbsp as a regular space when ICU changes

There will be future changes in ICU to match the Unicode 4.1
standard which no longer recognizes &nbsp as a word separator. We
need to work around this with text-transform:capitalize because
words after non-breaking spaces still need to be capitalized.

No layout tests added because existing layout tests cover this.

11:02 AM Changeset in webkit [13701] by ap
  • 4 edits
    2 adds in trunk

Reviewed by Darin.

Test: fast/dom/navigator-vendorSub.html

  • khtml/ecma/kjs_navigator.cpp: (KJS::Navigator::getValueProperty): Return an empty string for vendorSub property.
  • khtml/ecma/kjs_navigator.h:
11:02 AM Changeset in webkit [13700] by ggaren
  • 2 edits in trunk/WebKitTools

Reviewed by OMG BETH

  • Scripts/run-testkjs:
  • pipe STDERR to /dev/null by default; new --verbose option overrides this behavior
  • set DYLD_FRAMEWORK_PATH to the webkit-configured path
  • output run command in a format that can be copied and pasted into the terminal to run manually
12:17 AM Changeset in webkit [13762] by ggaren
  • 6 edits
    1 delete in branches/temp-tree-code/trunk/JavaScriptCore

more steps toward integrating the evaluate and execute loops

Apr 4, 2006:

11:42 PM Changeset in webkit [13761] by eseidel
  • 1 edit in branches/temp-tree-code/trunk/JavaScriptCore/JavaScriptCore.xcodeproj

Make svn ignore xcode internal files

11:41 PM Changeset in webkit [13760] by eseidel
  • 5 edits in branches/temp-tree-code/trunk/JavaScriptCore/kjs

roll evaluateList into the loop, fix exception handling

8:55 PM Changeset in webkit [13699] by darin
  • 10 edits
    1 delete in trunk

LayoutTests:

Reviewed by Justin and Adele.

  • fast/forms/plaintext-mode-2-expected.txt: Updated because there's no longer and extra <div> element added.

WebCore:

Reviewed by Justin (editing parts) and Adele (the rest).

The smart paste code was getting confused and adding extra spaces.

  • editing/ReplaceSelectionCommand.cpp: (WebCore::ReplaceSelectionCommand::doApply): Use isStartOfParagraph instead of isStartOfLine. (WebCore::ReplaceSelectionCommand::removeLinePlaceholderIfNeeded): Ditto.
  • editing/InsertParagraphSeparatorCommand.cpp: (WebCore::enclosingEmptyListItem): Change to call isStart/EndOfParagraph instead of Line.
  • editing/InsertTextCommand.cpp: (WebCore::InsertTextCommand::input): Add a comment about how isStartOfLine is almost certainly wrong here.
  • clean up some loose ends in the Frame class from the recent renaming
  • page/Frame.h: Removed declarations of deleteMe1, deleteMe2, and handleMouseMoveEventPart2.
  • page/Frame.cpp: (WebCore::Frame::handleMouseMoveEvent): Removed handleMouseMoveEventPart2 by renaming it to handleMouseMoveEvent and removing handleMouseMoveEvent itself.
  • invoke the makefile directly, removing the generate-derived-sources script
  • WebCore.vcproj/WebCore/build-generated-files.sh: Call make directly.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • generate-derived-sources: Removed.
7:26 PM Changeset in webkit [13698] by hyatt
  • 2 edits in trunk/LayoutTests/tables

Update table results after my change to create fewer RenderTexts.

7:19 PM Changeset in webkit [13697] by hyatt
  • 14 edits in trunk/LayoutTests/fast

Update fast results after my change to create fewer RenderTexts.

7:06 PM Changeset in webkit [13696] by adele
  • 3 edits
    8 adds in trunk

LayoutTests:

  • editing/selection/caret-rtl-expected.checksum: Added.
  • editing/selection/caret-rtl-expected.png: Added.
  • editing/selection/caret-rtl-expected.txt: Added.
  • editing/selection/caret-rtl.html: Added.
  • editing/selection/caret-rtl-2-expected.checksum: Added.
  • editing/selection/caret-rtl-2-expected.png: Added.
  • editing/selection/caret-rtl-2-expected.txt: Added.
  • editing/selection/caret-rtl-2.html: Added.

WebCore:

Reviewed by Justin.

Tests:
editing/selection/caret-rtl.html
editing/selection/caret-rtl-2.html

  • rendering/RenderText.cpp: (WebCore::RenderText::positionForCoordinates): When calculating the position for the beginning or end of an InlineTextBox, we now use offsetForPosition instead of just using m_start and m_len, because offsetForPosition will take rtl text into account. I also made some formatting changes.
6:49 PM Changeset in webkit [13695] by adele
  • 2 edits in trunk/LayoutTests

New results for

  • fast/forms/input-truncate-newline-expected.txt

Filed bug:
​http://bugzilla.opendarwin.org/show_bug.cgi?id=8187
input-truncate-newline layout test fails when run as part of larger set of tests

5:47 PM Changeset in webkit [13759] by ggaren
  • 1 add in branches/temp-tree-code/trunk/JavaScriptCore/kjs/NodeType.h

forgot this file

5:47 PM Changeset in webkit [13758] by ggaren
  • 5 edits in branches/temp-tree-code/trunk/JavaScriptCore

converted all executes to giant switch statement -- nothing is integrated with the evaluate loop yet, but it builds and passes all the tests

4:47 PM Changeset in webkit [13694] by thatcher
  • 1 move in tags/JavaScriptCore-521.9.1/JavaScriptCore

Correct place.

4:47 PM Changeset in webkit [13693] by thatcher
  • 1 add in tags/JavaScriptCore-521.9.1

New tag.

4:46 PM Changeset in webkit [13692] by thatcher
  • 1 move in tags/JavaScriptCore-521.9.1-temp

Move. This was in the wrong place.

4:42 PM Changeset in webkit [13691] by thatcher
  • 5 edits in tags/JavaScriptCore-521.9.1

Reviewed by Adele.

Fixes <rdar://problem/4498338> JavaScriptCore fails to compile for ppc64
Other 64 bit build fixes.

  • kjs/collector.cpp: (KJS::Collector::markOtherThreadConservatively): test for DARWIN_UNIX03 and use r1
  • kjs/dtoa.cpp: (Bigint::): cast PRIVATE_mem to unsigned to prevent warning
  • bindings/jni/jni_utility.cpp: (KJS::Bindings::getJavaVM): cast jniError to long to prevent format warning (KJS::Bindings::getJNIEnv): cast jniError to long to prevent format warning
  • bindings/runtime_root.cpp: (KJS::Bindings::addNativeReference): cast CFDictionaryGetValue to unsigned long to prevent warning (KJS::Bindings::removeNativeReference): cast CFDictionaryGetValue to unsigned long to prevent warning
4:34 PM Changeset in webkit [13690] by thatcher
  • 1 copy in tags/JavaScriptCore-521.9.1

new tag.

4:34 PM Changeset in webkit [13689] by hyatt
  • 2 edits in trunk/WebCore

Fix for bug 8065, inline blocks incorrectly loses spaces between them.

Reviewed by beth

  • dom/Text.cpp: (WebCore::Text::rendererIsNeeded):
3:23 PM Changeset in webkit [13688] by adele
  • 3 edits
    4 adds in trunk

LayoutTests:

  • Test for:

​http://bugzilla.opendarwin.org/show_bug.cgi?id=8092
REGRESSION (NativeTextField): table contents misaligned in Netflix queue

​http://bugzilla.opendarwin.org/show_bug.cgi?id=8141
REGRESSION: Native text field fails to wrap inside table

​http://bugzilla.opendarwin.org/show_bug.cgi?id=8072
REGRESSION: text fields at connect.apple.com spill out of the containing box

  • fast/forms/input-table-expected.checksum: Added.
  • fast/forms/input-table-expected.png: Added.
  • fast/forms/input-table-expected.txt: Added.
  • fast/forms/input-table.html: Added.

WebCore:

Reviewed by Hyatt.

​http://bugzilla.opendarwin.org/show_bug.cgi?id=8141
REGRESSION: Native text field fails to wrap inside table

​http://bugzilla.opendarwin.org/show_bug.cgi?id=8072
REGRESSION: text fields at connect.apple.com spill out of the containing box

Test: fast/forms/input-table.html

Rewrote calcMinMaxWidth for text fields so it considers width,
min-width, and max-width settings as well as the size attribute.

  • rendering/RenderTextField.cpp: (WebCore::RenderTextField::calcMinMaxWidth):
3:11 PM Changeset in webkit [13687] by bdakin
  • 4 edits in trunk/WebCore

Reviewed by Darin.

This is a followup to my fix for <rdar://problem/4493218>


This patch re-names computeIntLength() and computerShortLength() to
be computeLengthInt() and computeLengthShort(), respectively, to
match the pre-existing computeLengthFloat(). This patch also adds
the slightly confusing-ly named computeLengthIntForLength() which
uses the max and min values of a 28-bit integer as bounds for
overflow. This function is necessary because Length objects expect
28-bit integers.

2:21 PM Changeset in webkit [13686] by thatcher
  • 8 edits in trunk

JavaScriptCore:

Reviewed by Darin.

The Debug and Release frameworks are now built with install paths relative to the build products directory.
This removes the need for other projects to build with -framework WebCore and -framework JavaScriptCore.

JavaScriptGlue:

Reviewed by Darin.

The Debug and Release frameworks are now built with install paths relative to the build products directory.
This removes the need for other projects to build with -framework WebCore and -framework JavaScriptCore.

  • JavaScriptGlue.xcodeproj/project.pbxproj:

WebCore:

Reviewed by Darin.

The Debug and Release frameworks are now built with install paths relative to the build products directory.
This removes the need for other projects to build with -framework WebCore and -framework JavaScriptCore.

  • WebCore.xcodeproj/project.pbxproj:

WebKit:

Reviewed by Darin.

The Debug and Release frameworks are now built with install paths relative to the build products directory.
This removes the need for other projects to build with -framework WebCore and -framework JavaScriptCore.

  • WebKit.xcodeproj/project.pbxproj:
1:52 PM Changeset in webkit [13685] by justing
  • 17 edits
    12 adds in trunk

LayoutTests:

Reviewed by darin

<​http://bugzilla.opendarwin.org/show_bug.cgi?id=6608>
REGRESSION: Line disappears when deleting

Added:

  • editing/deleting/merge-different-styles-expected.checksum: Added.
  • editing/deleting/merge-different-styles-expected.png: Added.
  • editing/deleting/merge-different-styles-expected.txt: Added.
  • editing/deleting/merge-different-styles.html: Added.
  • editing/deleting/merge-no-br-expected.checksum: Added.
  • editing/deleting/merge-no-br-expected.png: Added.
  • editing/deleting/merge-no-br-expected.txt: Added.
  • editing/deleting/merge-no-br.html: Added.
  • editing/deleting/merge-whitespace-pre-expected.checksum: Added.
  • editing/deleting/merge-whitespace-pre-expected.png: Added.
  • editing/deleting/merge-whitespace-pre-expected.txt: Added.
  • editing/deleting/merge-whitespace-pre.html: Added.

Fixes (not enough style on nodes for the fixes to be reflected in pixel results):

  • editing/deleting/delete-block-merge-contents-005-expected.txt:
  • editing/deleting/delete-block-merge-contents-006-expected.txt:
  • editing/deleting/delete-block-merge-contents-008-expected.txt:

Equivalent render trees:

  • editing/deleting/delete-3857753-fix-expected.txt:
  • editing/inserting/insert-div-026-expected.txt:


Forgot to checkin these new expected results after fixing the DRT bug:

  • fast/lists/drag-into-marker-expected.checksum:
  • fast/lists/drag-into-marker-expected.png:
  • fast/lists/drag-into-marker-expected.txt:

WebCore:

Reviewed by darin


<​http://bugzilla.opendarwin.org/show_bug.cgi?id=6608>
REGRESSION: Line disappears when deleting


Rewrote moveNodesAfterNode to address these problems:
It moved nodes without preserving their style.
It traversed over siblings looking for a br to know when
to stop merging. If the br was burried inside a span, it
wouldn't find it. If the text is whitespace:pre, it wouldn't
stop.
In theory it would crash if the "enclosingInlineElements" of the start of the
selection to delete and the end of the selection to delete were the
same. We think that this will fix these:
<rdar://problems/3950559&4498113>
CrashTracer: 2116 crashes in Mail at com.apple.WebCore: khtml::CompositeEditCommand::insertNodeAfter + 32
CrashTracer: 1569 crashes in Mail at com.apple.WebCore: khtml::DeleteSelectionCommand::moveNodesAfterNode + 340
But we haven't been able to construct a reproducible case.


  • editing/CompositeEditCommand.cpp: (WebCore::CompositeEditCommand::removeNodeAndPruneAncestors): Moved from ReplaceSelectionCommand. (WebCore::CompositeEditCommand::prune): Ditto.
  • editing/CompositeEditCommand.h:
  • editing/DeleteSelectionCommand.cpp: (WebCore::DeleteSelectionCommand::mergeParagraphs): (WebCore::DeleteSelectionCommand::doApply):
  • editing/DeleteSelectionCommand.h:
  • editing/ReplaceSelectionCommand.cpp: (WebCore::ReplaceSelectionCommand::doApply):
  • editing/ReplaceSelectionCommand.h: (WebCore::):
  • editing/markup.cpp: (WebCore::createMarkup): Was crashing when passed a collapsed range. I early return an empty string instead.
11:10 AM Changeset in webkit [13684] by eseidel
  • 3 edits in trunk/JavaScriptCore

2006-04-04 Eric Seidel <​eseidel@apple.com>

Reviewed by ggaren.

Fix win32 build.
Disable ASSERT redefinition warnings for now.

10:48 AM Changeset in webkit [13683] by sullivan
  • 2 edits in trunk/WebKit

Reviewed by Adele Peterson.

  • WebKit part of <rdar://problem/4498418> "Autosaved" searchterms are saved during private browsing
  • WebView/WebView.m: (-[WebView _updateWebCoreSettingsFromPreferences:]): Pass private browsing setting down to WebCore.
10:47 AM Changeset in webkit [13682] by sullivan
  • 6 edits in trunk/WebCore

Reviewed by Adele Peterson.


  • WebCore part of <rdar://problem/4498418> "Autosaved" searchterms are saved during private browsing
  • bridge/mac/WebCoreSettings.h:
  • bridge/mac/WebCoreSettings.mm: (-[WebCoreSettings setPrivateBrowsingEnabled:]): (-[WebCoreSettings privateBrowsingEnabled]): Teach WebCoreSettings about private browsing (WebKit knew, but WebCore didn't)


  • kwq/KWQKHTMLSettings.h: (KHTMLSettings::privateBrowsingEnabled): (KHTMLSettings::setPrivateBrowsingEnabled): Teach KHTMLSettings about private browsing


  • kwq/KWQLineEdit.mm: Fix wrong class in a category method declaration; the compiler didn't seem to mind.


  • kwq/WebCoreTextField.mm: (-[KWQSearchFieldCell _addStringToRecentSearches:]): Override this method to bail out if private browsing is enabled.
9:15 AM Changeset in webkit [13681] by darin
  • 4 edits in trunk/WebCore

Reviewed by Hyatt.

​http://bugzilla.opendarwin.org/show_bug.cgi?id=7951
REGRESSION: Safari crashes when printing a google map w/directions

Tests: none, because I believe it only happens when printing, due to the relayouts

  • rendering/RenderTable.cpp: (WebCore::RenderTable::recalcSectionsIfNeeded): Add new utility to let the cells ensure the sections' grid data is up to date.
  • rendering/RenderTable.h:
  • rendering/RenderTableCell.cpp: (WebCore::RenderTableCell::calcMinMaxWidth): Call above method.
5:11 AM Changeset in webkit [13680] by eseidel
  • 2 edits in trunk/JavaScriptCore

2006-04-04 Bjrn Graf <​bjoern.graf@gmail.com>

Reviewed by ggaren & darin. Landed by eseidel.

Integrate CURL version of gettimeofday
​http://bugzilla.opendarwin.org/show_bug.cgi?id=7399
Disable crash report dialogs for testkjs.exe in Release mode
​http://bugzilla.opendarwin.org/show_bug.cgi?id=8113

  • kjs/testkjs.cpp: (StopWatch::start): (StopWatch::stop): (StopWatch::getElapsedMS): (main): (kjsmain):
4:54 AM Changeset in webkit [13679] by eseidel
  • 2 edits in trunk/WebCore

2006-04-03 Justin Haygood <​jhaygood@spsu.edu>

Reviewed by eseidel. Landed by eseidel.

  • platform/win/ScrollViewWin.cpp: (WebCore::ScrollView::updateScrollBars):
4:48 AM Changeset in webkit [13678] by eseidel
  • 3 edits in trunk/WebCore

2006-04-04 Eric Seidel <​eseidel@apple.com>

Reviewed by andersca.

Work-around spaces-in-pathnames issue in gnumake on win32.
​http://bugzilla.opendarwin.org/show_bug.cgi?id=8173

  • WebCore.vcproj/WebCore/build-generated-files.sh:
3:08 AM Changeset in webkit [13677] by eseidel
  • 2 edits in trunk/JavaScriptCore

2006-04-04 Eric Seidel <​eseidel@apple.com>

Reviewed by mjs.

  • kjs/number_object.cpp: (NumberProtoFunc::callAsFunction): remove trunc() to fix win32.
2:03 AM Changeset in webkit [13676] by mjs
  • 2 edits in trunk/JavaScriptCore

Reviewed by Darin.


  • kjs/number_object.cpp: (intPow10): Compute integer powers of 10 using exponentiation by squaring. (NumberProtoFunc::callAsFunction): Use intPow10(n) in place of all pow(10.0, n), plus a bit of refactoring.

Apr 3, 2006:

11:54 PM Changeset in webkit [13675] by darin
  • 3 edits in trunk/WebCore
  • tried to fix build again
  • WebCore.xcodeproj/project.pbxproj: Removed JSStyleSheet files.
  • WebCore.vcproj/WebCore/WebCore.vcproj: Ditto.
11:30 PM Changeset in webkit [13674] by darin
  • 56 edits in trunk/WebCore
  • fixed properties on a bunch of files (removed allow-tabs and svn:executable from many)
  • css/css_base.cpp:
  • html/html_headimpl.cpp:
  • khtml/ecma/kjs_traversal.cpp:
  • kwq/DeprecatedPtrListImpl.cpp:
  • kwq/DeprecatedValueListImpl.cpp:
  • loader/CachedScript.h:
  • platform/ArrayImpl.cpp:
  • platform/StringImpl.cpp:
  • rendering/DataRef.h:
  • rendering/RenderContainer.cpp:
  • rendering/RenderTableCell.cpp:
  • rendering/bidi.h:
  • rendering/render_list.cpp:
  • rendering/render_style.cpp:
  • rendering/table_layout.h: Converted tabs to spaces.
10:21 PM Changeset in webkit [13673] by ap
  • 8 edits
    12 adds in trunk

Reviewed by Darin.

Tests: fast/css/invalidation-errors.html

fast/css/invalidation-errors-2.html
fast/css/invalidation-errors-3.html

  • css/CSSGrammar.y: Rollback the properties added by parseValue() when it returns false.
  • css/cssparser.h: Moved shorthand counting to ShorthandScope, a new class in cssparser.cpp.
  • css/cssparser.cpp: (WebCore::CSSParser::rollbackLastProperties): Added. (WebCore::CSSParser::parseValue): Return false if there are too many properties in the list. (WebCore::CSSParser::parseBackgroundShorthand): Use ShorthandScope. (WebCore::CSSParser::parseShorthand): Ditto. (WebCore::CSSParser::parse4Values): Ditto.
9:47 PM Changeset in webkit [13672] by thatcher
  • 1 edit in trunk/WebKitTools/BuildSlaveSupport/run-performance-tests

Bumping the PLT count back to 5.

9:07 PM Changeset in webkit [13671] by darin
  • 4 edits
    1 delete in trunk/WebCore
  • changed StyleSheet back to hand-generated since the generated toJS function was not making the right type of wrapper for CSS style sheets (fixes failing layout tests)
  • DerivedSources.make: Removed JSStyleSheet.h.
  • css/StyleSheet.idl: Removed.
  • khtml/ecma/kjs_css.cpp: Added DOMStyleSheet back in.
  • khtml/ecma/kjs_css.h: Ditto.
8:55 PM Changeset in webkit [13670] by darin
  • 3 edits in trunk/JavaScriptCore
  • tweak config.h and Platform.h to try to get buildbot working (making some small changes at the same time)
  • kjs/config.h: Removed now-unneeded HAVE_ICU.
  • kxmlcore/Platform.h: Tweak how platform gets set up. Move all the USE stuff to the end.
8:34 PM Changeset in webkit [13669] by darin
  • 2 edits in trunk/WebCore
  • fixed Macintosh build
  • WebCore.xcodeproj/project.pbxproj: Fixed paths of some files that were absolute paths from my machine.
8:14 PM Changeset in webkit [13668] by darin
  • 2 edits in trunk/WebCore
  • try to fix Windows build
  • WebCore.vcproj/WebCore/WebCore.vcproj: Add three new generated files as source files.
7:45 PM Changeset in webkit [13667] by staikos
  • 3 edits in trunk/JavaScriptCore

Reviewed by Maciej.

Fix Win32 build breakage and remove an unused forward.

6:54 PM Changeset in webkit [13666] by darin
  • 7 edits in trunk

LayoutTests:

  • add a test of RTL to the test for bug 8106
  • fast/forms/input-text-scroll-left-on-blur-expected.png: Updated.
  • fast/forms/input-text-scroll-left-on-blur-expected.txt: Updated.
  • fast/forms/input-text-scroll-left-on-blur-expected.checksum: Updated.
  • fast/forms/input-text-scroll-left-on-blur.html: Updated.

WebCore:

Reviewed by Maciej.

  • rendering/RenderTextField.cpp: (WebCore::RenderTextField::forwardEvent): Scroll to the right if RTL.
6:50 PM Changeset in webkit [13665] by darin
  • 6 edits
    4 adds in trunk/WebCore

Reviewed by Maciej.

  • DerivedSources.make: Added.
  • css/CSSPrimitiveValue.idl: Added.
  • css/Counter.idl: Added.
  • css/StyleSheet.idl: Added.
  • WebCore.xcodeproj/project.pbxproj: Added new generated files, IDLs, and the makefile.
  • bindings/scripts/CodeGeneratorJS.pm:
  • generate-derived-sources: Added license header. Removed most of the script, except for a single invocation of make.
  • khtml/ecma/kjs_css.h: Removed DOMStyleSheet, DOMCSSPrimitiveValue, CSSPrimitiveValueConstructor, and DOMCounter.
  • khtml/ecma/kjs_css.cpp: (KJS::DOMCSSStyleSheet::DOMCSSStyleSheet): Changed to use JSStyleSheet as the base class. (KJS::DOMCSSStyleSheet::getOwnPropertySlot): Ditto. (KJS::DOMCSSValueProtoFunc::callAsFunction): Added. (KJS::toJS): Changed to use JSCSSPrimitiveValue. (KJS::DOMRGBColor::getValueProperty): Changed to call toJS instead of making a DOMCSSPrimitiveValue directly.
5:30 PM Changeset in webkit [13757] by eseidel
  • 5 edits
    1 copy
    1 add in branches/temp-tree-code/trunk/JavaScriptCore

Initial commit

4:58 PM Changeset in webkit [13664] by justing
  • 10 edits
    4 adds in trunk

LayoutTests:

Reviewed by harrison


<​http://bugzilla.opendarwin.org/show_bug.cgi?id=8117>
REGRESSION (NativeTextField): Drag and drop text within a text input field modifies page
<​http://bugzilla.opendarwin.org/show_bug.cgi?id=7567>
A drag and drop in DumpRenderTree copies the source, instead of cutting it

  • editing/pasteboard/drag-drop-modifies-page-expected.checksum: Added.
  • editing/pasteboard/drag-drop-modifies-page-expected.png: Added.
  • editing/pasteboard/drag-drop-modifies-page-expected.txt: Added.
  • editing/pasteboard/drag-drop-modifies-page.html: Added.

Updated:

  • fast/lists/drag-onto-marker.html:
  • fast/lists/drag-onto-marker-expected.txt:
  • fast/lists/drag-onto-marker-expected.png:
  • fast/lists/drag-onto-marker-expected.checksum:

WebCore:

Reviewed by harrison


<​http://bugzilla.opendarwin.org/show_bug.cgi?id=8117>
REGRESSION (NativeTextField): Drag and drop text within a text input field modifies page


The frame's selection is only set after all sub-commands have been
performed. When we send the khtmlBeforeTextInsertedEvent to the root
editable element we were using frame->selection(), which may no longer
be in the document.


Had to move the construction of the ReplacementFragment to when the
replace operation is applied, because endingSelection isn't the
endingSelection of the last operation when the replace operation
is constructed.

  • editing/ReplaceSelectionCommand.cpp: (WebCore::ReplacementFragment::ReplacementFragment): (WebCore::ReplaceSelectionCommand::ReplaceSelectionCommand): (WebCore::ReplaceSelectionCommand::doApply):
  • editing/ReplaceSelectionCommand.h:

WebKitTools:

Reviewed by harrison


<​http://bugzilla.opendarwin.org/show_bug.cgi?id=7567>
A drag and drop in DumpRenderTree copies the source, instead of cutting it


Tell the source that the drag is over after the drag is performed, not before.

  • DumpRenderTree/EventSendingController.m: (-[EventSendingController mouseUp]):
4:49 PM Changeset in webkit [13663] by staikos
  • 5 edits
    9 adds in trunk/JavaScriptCore

Reviewed by Maciej.

Implement a unicode abstraction layer to make JavaScriptCore much more
easily ported to other platforms without having to take in libicu. Also
makes the unicode related code easier to understand.

4:46 PM Changeset in webkit [13662] by bdakin
  • 1 edit
    4 adds in trunk/LayoutTests

Layout test for <rdar://problem/4495644> crash when mousing over
links at nationalrealestateinvestors.com in
WebCore::RenderBlock::findNextLineBreak

4:17 PM Changeset in webkit [13661] by sullivan
  • 2 edits in trunk/WebKit

Reviewed by Tim Hatcher.


  • re-fixed <rdar://problem/4481198> REGRESSION (TOT): search results list in bookmarks view remains when search text is deleted
  • WebView/WebView.m: (-[WebView searchFor:direction:caseSensitive:wrap:]): I fixed this recently, but then broke it again by adding an early bail-out to this method. So now I'm removing the early bail-out.
4:08 PM Changeset in webkit [13756] by eseidel
  • 1502 adds in branches/temp-tree-code

initial import

3:30 PM Changeset in webkit [13660] by bdakin
  • 2 edits in trunk/WebCore

Reviewed by Hyatt.

Fix for <rdar://problem/4495644> crash when mousing over links at
nationalrealestateinvestors.com in
WebCore::RenderBlock::findNextLineBreak

This is a fix for a repro crasher where a rootLineBox had a stale
pointer to a render object.

  • rendering/RenderFlow.cpp: (WebCore::RenderFlow::dirtyLinesFromChangedChild): Only break from the function because of selfNeedsLayout() if we are not an inline flow, because if we are, we will not re-layout before bad things can happen.
3:26 PM Changeset in webkit [13659] by thatcher
  • 2 edits in trunk/WebCore

Reviewed by Maciej.

Removing idl files and some scripts from the WebCore target to prevent
them from being copied into the Resources.

  • WebCore.xcodeproj/project.pbxproj:
3:06 PM Changeset in webkit [13658] by thatcher
  • 5 edits in trunk/JavaScriptCore

Reviewed by Adele.

Fixes <rdar://problem/4498338> JavaScriptCore fails to compile for ppc64
Other 64 bit build fixes.

  • kjs/collector.cpp: (KJS::Collector::markOtherThreadConservatively): test for DARWIN_UNIX03 and use r1
  • kjs/dtoa.cpp: (Bigint::): cast PRIVATE_mem to unsigned to prevent warning
  • bindings/jni/jni_utility.cpp: (KJS::Bindings::getJavaVM): cast jniError to long to prevent format warning (KJS::Bindings::getJNIEnv): cast jniError to long to prevent format warning
  • bindings/runtime_root.cpp: (KJS::Bindings::addNativeReference): cast CFDictionaryGetValue to unsigned long to prevent warning (KJS::Bindings::removeNativeReference): cast CFDictionaryGetValue to unsigned long to prevent warning
2:58 PM Changeset in webkit [13657] by hyatt
  • 10 edits in trunk/WebCore

Win32 theme work. Bug 8162. r=anders

2:55 PM Changeset in webkit [13656] by adele
  • 3 edits
    2 adds in trunk

LayoutTests:

Reviewed by Beth.

  • fast/dom/Element/offsetTop-table-cell-expected.txt: Added.
  • fast/dom/Element/offsetTop-table-cell.html: Added.

WebCore:

Reviewed by Beth.

Test: fast/dom/Element/offsetTop-table-cell.html

  • rendering/RenderObject.cpp: (WebCore::RenderObject::offsetTop): Skip table rows when adding up the offsets, since a table cell's yPos() is relative to the table section, not the row.
2:42 PM Changeset in webkit [13655] by adele
  • 3 edits
    8 adds in trunk

LayoutTests:

Reviewed by Darin.

  • fast/repaint/inline-block-overflow-expected.png: Added.
  • fast/repaint/layer-outline-expected.checksum: Added.
  • fast/repaint/layer-outline-expected.png: Added.
  • fast/repaint/layer-outline-expected.txt: Added.
  • fast/repaint/layer-outline-horizontal-expected.checksum: Added.
  • fast/repaint/layer-outline-horizontal-expected.png: Added.
  • fast/repaint/layer-outline-horizontal-expected.txt: Added.
  • fast/repaint/layer-outline-horizontal.html: Added.
  • fast/repaint/layer-outline.html: Added.

WebCore:

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

Reviewed by Darin.

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::paintLayer): Use the outlineRect for the outline phase and do it only if the outlineRect isn't empty. (WebCore::RenderLayer::calculateRects): Actually add the outline width to the outline rect.
2:14 PM Changeset in webkit [13654] by adele
  • 3 edits
    2 adds in trunk

LayoutTests:

Reviewed by Justin.

  • fast/forms/input-truncate-newline-expected.txt: Added.
  • fast/forms/input-truncate-newline.html: Added.

WebCore:

Reviewed by Justin.

Test: fast/forms/input-truncate-newline.html

  • html/HTMLInputElement.cpp: (WebCore::minPosition): Added helper function. (WebCore::HTMLInputElement::defaultEventHandler): Searches for /r or /n and truncates the text to be inserted to the earliest newline.
10:46 AM Changeset in webkit [13653] by ap
  • 2 edits in trunk/WebCore

Fixed a comment (forgot to save the file before the previous commit).

  • xml/xmlhttprequest.cpp: (WebCore::XMLHttpRequest::open):
9:36 AM Changeset in webkit [13652] by ap
  • 3 edits
    2 adds in trunk

Reviewed by Darin.

Test: http/tests/xmlhttprequest/methods-lower-case.html

  • xml/xmlhttprequest.cpp: (WebCore::XMLHttpRequest::open): Uppercase some HTTP method names, to match a Firefox quirk. (WebCore::XMLHttpRequest::send): Account for the above change.
9:28 AM Changeset in webkit [13651] by darin
  • 2 edits
    1 add in trunk/WebCore

Reviewed by Darin.

  • rendering/RenderObject.cpp: (WebCore::RenderObject::repaintAfterLayoutIfNeeded): When an element changes size, the delta rectangles that need to be invalidated must be inflated by the outline size to ensure that the previous outline is erased, and the space where the new outline is to be drawn is also invalidated. This behaviour is identical to the behaviour of borders that was fixed in bug 6301.
  • manual-tests/outline-repaint-glitch.html: Added. Manual testcase. This is just an outline version of border-repaint-glitch.html
9:25 AM Changeset in webkit [13650] by darin
  • 7 edits
    8 adds in trunk

LayoutTests:

Reviewed by Maciej.

Adding tests for ​http://bugzilla.opendarwin.org/show_bug.cgi?id=8121
REGRESSION: 404s are not displayed
and ​http://bugzilla.opendarwin.org/show_bug.cgi?id=7739
REGRESSION: Assertion failure loading acid2 test in -[WebCoreFrameBridge installInFrame:]

In addition I tweaked the httpd conf file to allow us to exec perl CGI scripts.

  • http/conf/httpd.conf:
  • http/tests/misc/error404-expected.checksum: Added.
  • http/tests/misc/error404-expected.png: Added.
  • http/tests/misc/error404-expected.txt: Added.
  • http/tests/misc/error404.pl: Added.
  • http/tests/misc/iframe404-expected.checksum: Added.
  • http/tests/misc/iframe404-expected.png: Added.
  • http/tests/misc/iframe404-expected.txt: Added.
  • http/tests/misc/iframe404.html: Added.

WebCore:

Reviewed by Maciej.

Support for fixing ​http://bugzilla.opendarwin.org/show_bug.cgi?id=8121
REGRESSION: 404s are not displayed

  • bridge/mac/WebCoreFrameBridge.mm: (-[WebCoreFrameBridge currentForm]): Nuke redundant nil check. (-[WebCoreFrameBridge frameElement]): Tweak to not rely on our document, which gives a correct result even at the start of our loading process.
  • bindings/objc/DOM.mm: (-[DOMDocument _ownerElement]): Nuke redundant nil check.

WebKit:

Reviewed by Maciej.

fix ​http://bugzilla.opendarwin.org/show_bug.cgi?id=8121
REGRESSION: 404s are not displayed

  • WebView/WebMainResourceLoader.m: (-[WebMainResourceLoader continueAfterContentPolicy:response:]): Only cancel failed loads for object elements, not for frames or a whole page.
9:04 AM Changeset in webkit [13649] by darin
  • 2 edits in trunk/WebCore

Reviewed by Maciej.

  • page/Frame.cpp: (WebCore::Frame::reparseConfiguration): Added back updateStyleSelector call that was removed as part of the patch for bug 7907.
12:06 AM Changeset in webkit [13648] by mjs
  • 8 edits
    2 adds in trunk

LayoutTests:

Reviewed by Hyatt.


Tests for:

  • fast/forms/focus-control-to-page-expected.txt: Added.
  • fast/forms/focus-control-to-page.html: Added.

WebCore:

Reviewed by Hyatt.

  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::nextKeyViewInFrame):
  • bridge/mac/WebCoreFrameBridge.h:

WebKit:

Reviewed by Hyatt.

  • WebCoreSupport/WebFrameBridge.m: (-[WebFrameBridge willMakeFirstResponderForNodeFocus]): New method
  • let WebHTMLView know that the next time it becomes first responder, it's to change focus within the page and the right node has already been set, so it should not move focus forward or backward inside it.
  • WebView/WebHTMLView.m: (-[NSArray needsPanelToBecomeKey]): Override to return YES, oddly enough this is the right way to tell AppKit that you should be in the tab cycle loop. (-[NSArray becomeFirstResponder]): Don't move forward or back in tab cycle when this becomeFirstResponder is for tabbing from a control in the page. (-[WebHTMLView _willMakeFirstResponderForNodeFocus]): Note that the next time this view becomes first responder, it will be for in-page focus navigation.
  • WebView/WebHTMLViewInternal.h:

Apr 2, 2006:

6:52 PM Changeset in webkit [13647] by darin
  • 3 edits in trunk/LayoutTests
  • updated one more result for change in focus ring appearance
  • fast/forms/plaintext-mode-2-expected.checksum: Updated.
  • fast/forms/plaintext-mode-2-expected.png: Ditto.
6:39 PM Changeset in webkit [13646] by darin
  • 2 edits in trunk/WebCore
  • add a few stubs to get Windows closer to building
  • platform/win/TemporaryLinkStubs.cpp: (WebCore::focusRingColor): (WebCore::setFocusRingColorChangeFunction): (Frame::setNeedsReapplyStyles):
6:26 PM Changeset in webkit [13645] by darin
  • 2 edits in trunk/WebCore
  • fix the build
  • WebCore.xcodeproj/project.pbxproj: Removed a bunch of files that should not have been mentioned at all, and a bunch of others that should be in the project but not in the target.
6:17 PM Changeset in webkit [13644] by darin
  • 178 edits in trunk/LayoutTests
  • updated layout test results, mostly for the change in focus ring appearance
  • editing/input/range-for-empty-document-expected.txt:
  • editing/selection/3690703-2-expected.checksum:
  • editing/selection/3690703-2-expected.png:
  • editing/selection/3690703-2-expected.txt:
  • fast/overflow/scrollRevealButton-expected.checksum:
  • fast/overflow/scrollRevealButton-expected.png:
  • fast/overflow/scrollRevealButton-expected.txt:
  • editing/deleting/collapse-whitespace-3587601-fix-expected.checksum:
  • editing/deleting/collapse-whitespace-3587601-fix-expected.png:
  • editing/deleting/delete-3608430-fix-expected.checksum:
  • editing/deleting/delete-3608430-fix-expected.png:
  • editing/deleting/delete-3608445-fix-expected.checksum:
  • editing/deleting/delete-3608445-fix-expected.png:
  • editing/deleting/delete-3608462-fix-expected.checksum:
  • editing/deleting/delete-3608462-fix-expected.png:
  • editing/deleting/delete-3959464-fix-expected.checksum:
  • editing/deleting/delete-3959464-fix-expected.png:
  • editing/deleting/delete-4038408-fix-expected.checksum:
  • editing/deleting/delete-4038408-fix-expected.png:
  • editing/deleting/delete-4083333-fix-expected.checksum:
  • editing/deleting/delete-4083333-fix-expected.png:
  • editing/deleting/delete-after-span-ws-001-expected.checksum:
  • editing/deleting/delete-after-span-ws-001-expected.png:
  • editing/deleting/delete-after-span-ws-002-expected.checksum:
  • editing/deleting/delete-after-span-ws-002-expected.png:
  • editing/deleting/delete-after-span-ws-003-expected.checksum:
  • editing/deleting/delete-after-span-ws-003-expected.png:
  • editing/deleting/delete-and-undo-expected.checksum:
  • editing/deleting/delete-and-undo-expected.png:
  • editing/deleting/delete-at-paragraph-boundaries-001-expected.checksum:
  • editing/deleting/delete-at-paragraph-boundaries-001-expected.png:
  • editing/deleting/delete-at-paragraph-boundaries-002-expected.checksum:
  • editing/deleting/delete-at-paragraph-boundaries-002-expected.png:
  • editing/deleting/delete-at-paragraph-boundaries-003-expected.checksum:
  • editing/deleting/delete-at-paragraph-boundaries-003-expected.png:
  • editing/deleting/delete-at-paragraph-boundaries-004-expected.checksum:
  • editing/deleting/delete-at-paragraph-boundaries-004-expected.png:
  • editing/deleting/delete-at-paragraph-boundaries-005-expected.checksum:
  • editing/deleting/delete-at-paragraph-boundaries-005-expected.png:
  • editing/deleting/delete-at-paragraph-boundaries-006-expected.checksum:
  • editing/deleting/delete-at-paragraph-boundaries-006-expected.png:
  • editing/deleting/delete-at-paragraph-boundaries-007-expected.checksum:
  • editing/deleting/delete-at-paragraph-boundaries-007-expected.png:
  • editing/deleting/delete-at-paragraph-boundaries-008-expected.checksum:
  • editing/deleting/delete-at-paragraph-boundaries-008-expected.png:
  • editing/deleting/delete-at-paragraph-boundaries-009-expected.checksum:
  • editing/deleting/delete-at-paragraph-boundaries-009-expected.png:
  • editing/deleting/delete-at-paragraph-boundaries-010-expected.checksum:
  • editing/deleting/delete-at-paragraph-boundaries-010-expected.png:
  • editing/deleting/delete-at-paragraph-boundaries-011-expected.checksum:
  • editing/deleting/delete-at-paragraph-boundaries-011-expected.png:
  • editing/deleting/delete-block-contents-001-expected.checksum:
  • editing/deleting/delete-block-contents-001-expected.png:
  • editing/deleting/delete-block-contents-002-expected.checksum:
  • editing/deleting/delete-block-contents-002-expected.png:
  • editing/deleting/delete-block-contents-003-expected.checksum:
  • editing/deleting/delete-block-contents-003-expected.png:
  • editing/deleting/delete-block-merge-contents-001-expected.checksum:
  • editing/deleting/delete-block-merge-contents-001-expected.png:
  • editing/deleting/delete-block-merge-contents-002-expected.checksum:
  • editing/deleting/delete-block-merge-contents-002-expected.png:
  • editing/deleting/delete-block-merge-contents-003-expected.checksum:
  • editing/deleting/delete-block-merge-contents-003-expected.png:
  • editing/deleting/delete-block-merge-contents-004-expected.checksum:
  • editing/deleting/delete-block-merge-contents-004-expected.png:
  • editing/deleting/delete-block-merge-contents-005-expected.checksum:
  • editing/deleting/delete-block-merge-contents-005-expected.png:
  • editing/deleting/delete-block-merge-contents-006-expected.checksum:
  • editing/deleting/delete-block-merge-contents-006-expected.png:
  • editing/deleting/delete-block-merge-contents-007-expected.checksum:
  • editing/deleting/delete-block-merge-contents-007-expected.png:
  • editing/deleting/delete-block-merge-contents-008-expected.checksum:
  • editing/deleting/delete-block-merge-contents-008-expected.png:
  • editing/deleting/delete-block-merge-contents-009-expected.checksum:
  • editing/deleting/delete-block-merge-contents-009-expected.png:
  • editing/deleting/delete-block-merge-contents-010-expected.checksum:
  • editing/deleting/delete-block-merge-contents-010-expected.png:
  • editing/deleting/delete-block-merge-contents-011-expected.checksum:
  • editing/deleting/delete-block-merge-contents-011-expected.png:
  • editing/deleting/delete-block-merge-contents-012-expected.checksum:
  • editing/deleting/delete-block-merge-contents-012-expected.png:
  • editing/deleting/delete-block-merge-contents-013-expected.checksum:
  • editing/deleting/delete-block-merge-contents-013-expected.png:
  • editing/deleting/delete-block-merge-contents-014-expected.checksum:
  • editing/deleting/delete-block-merge-contents-014-expected.png:
  • editing/deleting/delete-block-merge-contents-015-expected.checksum:
  • editing/deleting/delete-block-merge-contents-015-expected.png:
  • editing/deleting/delete-block-merge-contents-016-expected.checksum:
  • editing/deleting/delete-block-merge-contents-016-expected.png:
  • editing/deleting/delete-block-merge-contents-017-expected.checksum:
  • editing/deleting/delete-block-merge-contents-017-expected.png:
  • editing/deleting/delete-br-001-expected.checksum:
  • editing/deleting/delete-br-001-expected.png:
  • editing/deleting/delete-br-002-expected.checksum:
  • editing/deleting/delete-br-002-expected.png:
  • editing/deleting/delete-br-003-expected.checksum:
  • editing/deleting/delete-br-003-expected.png:
  • editing/deleting/delete-br-004-expected.checksum:
  • editing/deleting/delete-br-004-expected.png:
  • editing/deleting/delete-br-005-expected.checksum:
  • editing/deleting/delete-br-005-expected.png:
  • editing/deleting/delete-br-006-expected.checksum:
  • editing/deleting/delete-br-006-expected.png:
  • editing/deleting/delete-br-007-expected.checksum:
  • editing/deleting/delete-br-007-expected.png:
  • editing/deleting/delete-character-001-expected.checksum:
  • editing/deleting/delete-character-001-expected.png:
  • editing/deleting/delete-contiguous-ws-001-expected.checksum:
  • editing/deleting/delete-contiguous-ws-001-expected.png:
  • editing/deleting/delete-image-001-expected.checksum:
  • editing/deleting/delete-image-001-expected.png:
  • editing/deleting/delete-image-002-expected.checksum:
  • editing/deleting/delete-image-002-expected.png:
  • editing/deleting/delete-image-003-expected.checksum:
  • editing/deleting/delete-image-003-expected.png:
  • editing/deleting/delete-image-004-expected.checksum:
  • editing/deleting/delete-image-004-expected.png:
  • editing/deleting/delete-leading-ws-001-expected.checksum:
  • editing/deleting/delete-leading-ws-001-expected.png:
  • editing/deleting/delete-line-015-expected.checksum:
  • editing/deleting/delete-line-015-expected.png:
  • editing/deleting/delete-line-016-expected.checksum:
  • editing/deleting/delete-line-016-expected.png:
  • editing/deleting/delete-line-017-expected.checksum:
  • editing/deleting/delete-line-017-expected.png:
  • editing/deleting/delete-line-end-ws-001-expected.checksum:
  • editing/deleting/delete-line-end-ws-001-expected.png:
  • editing/deleting/delete-line-end-ws-002-expected.checksum:
  • editing/deleting/delete-line-end-ws-002-expected.png:
  • editing/deleting/delete-listitem-001-expected.checksum:
  • editing/deleting/delete-listitem-001-expected.png:
  • editing/deleting/delete-listitem-002-expected.checksum:
  • editing/deleting/delete-listitem-002-expected.png:
  • editing/deleting/delete-select-all-001-expected.checksum:
  • editing/deleting/delete-select-all-001-expected.png:
  • editing/deleting/delete-select-all-002-expected.checksum:
  • editing/deleting/delete-select-all-002-expected.png:
  • editing/deleting/delete-select-all-003-expected.checksum:
  • editing/deleting/delete-select-all-003-expected.png:
  • editing/deleting/delete-selection-001-expected.checksum:
  • editing/deleting/delete-selection-001-expected.png:
  • editing/deleting/delete-tab-001-expected.checksum:
  • editing/deleting/delete-tab-001-expected.png:
  • editing/deleting/delete-tab-002-expected.checksum:
  • editing/deleting/delete-tab-002-expected.png:
  • editing/deleting/delete-tab-003-expected.checksum:
  • editing/deleting/delete-tab-003-expected.png:
  • editing/deleting/delete-tab-004-expected.checksum:
  • editing/deleting/delete-tab-004-expected.png:
  • editing/deleting/delete-trailing-ws-001-expected.checksum:
  • editing/deleting/delete-trailing-ws-001-expected.png:
  • editing/deleting/delete-trailing-ws-002-expected.checksum:
  • editing/deleting/delete-trailing-ws-002-expected.png:
  • editing/deleting/delete-ws-fixup-001-expected.checksum:
  • editing/deleting/delete-ws-fixup-001-expected.png:
  • editing/deleting/delete-ws-fixup-002-expected.checksum:
  • editing/deleting/delete-ws-fixup-002-expected.png:
  • editing/deleting/delete-ws-fixup-003-expected.checksum:
  • editing/deleting/delete-ws-fixup-003-expected.png:
  • editing/deleting/delete-ws-fixup-004-expected.checksum:
  • editing/deleting/delete-ws-fixup-004-expected.png:
  • editing/deleting/smart-delete-001-expected.checksum:
  • editing/deleting/smart-delete-001-expected.png:
  • editing/deleting/smart-delete-002-expected.checksum:
  • editing/deleting/smart-delete-002-expected.png:
  • editing/execCommand/boldSelection-expected.checksum:
  • editing/execCommand/boldSelection-expected.png:
  • editing/execCommand/insertHTML-expected.checksum:
  • editing/execCommand/insertHTML-expected.png:
  • editing/execCommand/insertImage-expected.checksum:
  • editing/execCommand/insertImage-expected.png:
  • editing/execCommand/italicizeByCharacter-expected.checksum:
  • editing/execCommand/italicizeByCharacter-expected.png:
  • editing/execCommand/modifyForeColorByCharacter-expected.checksum:
  • editing/execCommand/modifyForeColorByCharacter-expected.png:
  • editing/execCommand/print-expected.checksum:
  • editing/execCommand/print-expected.png:
  • editing/execCommand/selectAll-expected.checksum:
  • editing/execCommand/selectAll-expected.png:
  • editing/execCommand/strikethroughSelection-expected.checksum:
  • editing/execCommand/strikethroughSelection-expected.png:
  • editing/inserting/4278698-expected.checksum:
  • editing/inserting/4278698-expected.png:
  • editing/inserting/editing-empty-divs-expected.checksum:
  • editing/inserting/editing-empty-divs-expected.png:
  • editing/inserting/insert-3659587-fix-expected.checksum:
  • editing/inserting/insert-3659587-fix-expected.png:
  • editing/inserting/insert-3775316-fix-expected.checksum:
  • editing/inserting/insert-3775316-fix-expected.png:
  • editing/inserting/insert-3778059-fix-expected.checksum:
  • editing/inserting/insert-3778059-fix-expected.png:
  • editing/inserting/insert-3800346-fix-expected.checksum:
  • editing/inserting/insert-3800346-fix-expected.png:
  • editing/inserting/insert-after-delete-001-expected.checksum:
  • editing/inserting/insert-after-delete-001-expected.png:
  • editing/inserting/insert-at-end-01-expected.checksum:
  • editing/inserting/insert-at-end-01-expected.png:
  • editing/inserting/insert-at-end-02-expected.checksum:
  • editing/inserting/insert-at-end-02-expected.png:
  • editing/inserting/insert-br-001-expected.checksum:
  • editing/inserting/insert-br-001-expected.png:
  • editing/inserting/insert-br-002-expected.checksum:
  • editing/inserting/insert-br-002-expected.png:
  • editing/inserting/insert-br-003-expected.checksum:
  • editing/inserting/insert-br-003-expected.png:
  • editing/inserting/insert-br-004-expected.checksum:
  • editing/inserting/insert-br-004-expected.png:
  • editing/inserting/insert-br-005-expected.checksum:
  • editing/inserting/insert-br-005-expected.png:
  • editing/inserting/insert-br-006-expected.checksum:
  • editing/inserting/insert-br-006-expected.png:
  • editing/inserting/insert-br-007-expected.checksum:
  • editing/inserting/insert-br-007-expected.png:
  • editing/inserting/insert-br-008-expected.checksum:
  • editing/inserting/insert-br-008-expected.png:
  • editing/inserting/insert-br-at-tabspan-001-expected.checksum:
  • editing/inserting/insert-br-at-tabspan-001-expected.png:
  • editing/inserting/insert-br-at-tabspan-002-expected.checksum:
  • editing/inserting/insert-br-at-tabspan-002-expected.png:
  • editing/inserting/insert-br-at-tabspan-003-expected.checksum:
  • editing/inserting/insert-br-at-tabspan-003-expected.png:
  • editing/inserting/insert-br-quoted-001-expected.checksum:
  • editing/inserting/insert-br-quoted-001-expected.png:
  • editing/inserting/insert-br-quoted-002-expected.checksum:
  • editing/inserting/insert-br-quoted-002-expected.png:
  • editing/inserting/insert-br-quoted-003-expected.checksum:
  • editing/inserting/insert-br-quoted-003-expected.png:
  • editing/inserting/insert-br-quoted-004-expected.checksum:
  • editing/inserting/insert-br-quoted-004-expected.png:
  • editing/inserting/insert-br-quoted-005-expected.checksum:
  • editing/inserting/insert-br-quoted-005-expected.png:
  • editing/inserting/insert-br-quoted-006-expected.checksum:
  • editing/inserting/insert-br-quoted-006-expected.png:
  • editing/inserting/insert-div-025-expected.checksum:
  • editing/inserting/insert-div-025-expected.png:
  • editing/inserting/insert-div-026-expected.checksum:
  • editing/inserting/insert-div-026-expected.png:
  • editing/inserting/insert-space-in-empty-doc-expected.checksum:
  • editing/inserting/insert-space-in-empty-doc-expected.png:
  • editing/inserting/insert-tab-001-expected.checksum:
  • editing/inserting/insert-tab-001-expected.png:
  • editing/inserting/insert-tab-002-expected.checksum:
  • editing/inserting/insert-tab-002-expected.png:
  • editing/inserting/insert-tab-003-expected.checksum:
  • editing/inserting/insert-tab-003-expected.png:
  • editing/inserting/insert-tab-004-expected.checksum:
  • editing/inserting/insert-tab-004-expected.png:
  • editing/inserting/insert-text-at-tabspan-001-expected.checksum:
  • editing/inserting/insert-text-at-tabspan-001-expected.png:
  • editing/inserting/insert-text-at-tabspan-002-expected.checksum:
  • editing/inserting/insert-text-at-tabspan-002-expected.png:
  • editing/inserting/insert-text-at-tabspan-003-expected.checksum:
  • editing/inserting/insert-text-at-tabspan-003-expected.png:
  • editing/inserting/insert-text-with-newlines-expected.checksum:
  • editing/inserting/insert-text-with-newlines-expected.png:
  • editing/inserting/return-key-with-selection-001-expected.checksum:
  • editing/inserting/return-key-with-selection-001-expected.png:
  • editing/inserting/return-key-with-selection-002-expected.checksum:
  • editing/inserting/return-key-with-selection-002-expected.png:
  • editing/inserting/return-key-with-selection-003-expected.checksum:
  • editing/inserting/return-key-with-selection-003-expected.png:
  • editing/inserting/typing-001-expected.checksum:
  • editing/inserting/typing-001-expected.png:
  • editing/inserting/typing-002-expected.checksum:
  • editing/inserting/typing-002-expected.png:
  • editing/inserting/typing-003-expected.checksum:
  • editing/inserting/typing-003-expected.png:
  • editing/inserting/typing-around-br-001-expected.checksum:
  • editing/inserting/typing-around-br-001-expected.png:
  • editing/inserting/typing-around-image-001-expected.checksum:
  • editing/inserting/typing-around-image-001-expected.png:
  • editing/pasteboard/3976872-expected.checksum:
  • editing/pasteboard/3976872-expected.png:
  • editing/pasteboard/4076267-2-expected.checksum:
  • editing/pasteboard/4076267-2-expected.png:
  • editing/pasteboard/4076267-3-expected.checksum:
  • editing/pasteboard/4076267-3-expected.png:
  • editing/pasteboard/4076267-expected.checksum:
  • editing/pasteboard/4076267-expected.png:
  • editing/pasteboard/block-wrappers-necessary-expected.checksum:
  • editing/pasteboard/block-wrappers-necessary-expected.png:
  • editing/pasteboard/cut-text-001-expected.checksum:
  • editing/pasteboard/cut-text-001-expected.png:
  • editing/pasteboard/display-block-on-spans-expected.checksum:
  • editing/pasteboard/display-block-on-spans-expected.png:
  • editing/pasteboard/paste-4035648-fix-expected.checksum:
  • editing/pasteboard/paste-4035648-fix-expected.png:
  • editing/pasteboard/paste-4038267-fix-expected.checksum:
  • editing/pasteboard/paste-4038267-fix-expected.png:
  • editing/pasteboard/paste-4039777-fix-expected.checksum:
  • editing/pasteboard/paste-4039777-fix-expected.png:
  • editing/pasteboard/paste-line-endings-001-expected.checksum:
  • editing/pasteboard/paste-line-endings-001-expected.png:
  • editing/pasteboard/paste-line-endings-002-expected.checksum:
  • editing/pasteboard/paste-line-endings-002-expected.png:
  • editing/pasteboard/paste-line-endings-003-expected.checksum:
  • editing/pasteboard/paste-line-endings-003-expected.png:
  • editing/pasteboard/paste-line-endings-004-expected.checksum:
  • editing/pasteboard/paste-line-endings-004-expected.png:
  • editing/pasteboard/paste-line-endings-005-expected.checksum:
  • editing/pasteboard/paste-line-endings-005-expected.png:
  • editing/pasteboard/paste-line-endings-006-expected.checksum:
  • editing/pasteboard/paste-line-endings-006-expected.png:
  • editing/pasteboard/paste-line-endings-007-expected.checksum:
  • editing/pasteboard/paste-line-endings-007-expected.png:
  • editing/pasteboard/paste-line-endings-008-expected.checksum:
  • editing/pasteboard/paste-line-endings-008-expected.png:
  • editing/pasteboard/paste-line-endings-009-expected.checksum:
  • editing/pasteboard/paste-line-endings-009-expected.png:
  • editing/pasteboard/paste-line-endings-010-expected.checksum:
  • editing/pasteboard/paste-line-endings-010-expected.png:
  • editing/pasteboard/paste-list-001-expected.checksum:
  • editing/pasteboard/paste-list-001-expected.png:
  • editing/pasteboard/paste-match-style-001-expected.checksum:
  • editing/pasteboard/paste-match-style-001-expected.png:
  • editing/pasteboard/paste-match-style-002-expected.checksum:
  • editing/pasteboard/paste-match-style-002-expected.png:
  • editing/pasteboard/paste-pre-001-expected.checksum:
  • editing/pasteboard/paste-pre-001-expected.png:
  • editing/pasteboard/paste-pre-002-expected.checksum:
  • editing/pasteboard/paste-pre-002-expected.png:
  • editing/pasteboard/paste-table-001-expected.checksum:
  • editing/pasteboard/paste-table-001-expected.png:
  • editing/pasteboard/paste-text-001-expected.checksum:
  • editing/pasteboard/paste-text-001-expected.png:
  • editing/pasteboard/paste-text-017-expected.checksum:
  • editing/pasteboard/paste-text-017-expected.png:
  • editing/pasteboard/paste-text-018-expected.checksum:
  • editing/pasteboard/paste-text-018-expected.png:
  • editing/pasteboard/paste-text-019-expected.checksum:
  • editing/pasteboard/paste-text-019-expected.png:
  • editing/pasteboard/paste-text-at-tabspan-001-expected.checksum:
  • editing/pasteboard/paste-text-at-tabspan-001-expected.png:
  • editing/pasteboard/paste-text-at-tabspan-002-expected.checksum:
  • editing/pasteboard/paste-text-at-tabspan-002-expected.png:
  • editing/pasteboard/paste-text-at-tabspan-003-expected.checksum:
  • editing/pasteboard/paste-text-at-tabspan-003-expected.png:
  • editing/pasteboard/smart-paste-001-expected.checksum:
  • editing/pasteboard/smart-paste-001-expected.png:
  • editing/pasteboard/smart-paste-002-expected.checksum:
  • editing/pasteboard/smart-paste-002-expected.png:
  • editing/pasteboard/smart-paste-003-expected.checksum:
  • editing/pasteboard/smart-paste-003-expected.png:
  • editing/pasteboard/smart-paste-004-expected.checksum:
  • editing/pasteboard/smart-paste-004-expected.png:
  • editing/pasteboard/smart-paste-005-expected.checksum:
  • editing/pasteboard/smart-paste-005-expected.png:
  • editing/pasteboard/smart-paste-006-expected.checksum:
  • editing/pasteboard/smart-paste-006-expected.png:
  • editing/pasteboard/smart-paste-007-expected.checksum:
  • editing/pasteboard/smart-paste-007-expected.png:
  • editing/selection/3690703-expected.checksum:
  • editing/selection/3690703-expected.png:
  • editing/selection/3690719-expected.checksum:
  • editing/selection/3690719-expected.png:
  • editing/selection/6476-expected.checksum:
  • editing/selection/6476-expected.png:
  • editing/selection/after-line-wrap-expected.checksum:
  • editing/selection/after-line-wrap-expected.png:
  • editing/selection/end-of-document-expected.checksum:
  • editing/selection/end-of-document-expected.png:
  • editing/selection/expanding-selections-expected.checksum:
  • editing/selection/expanding-selections-expected.png:
  • editing/selection/expanding-selections2-expected.checksum:
  • editing/selection/expanding-selections2-expected.png:
  • editing/selection/extend-by-character-001-expected.checksum:
  • editing/selection/extend-by-character-001-expected.png:
  • editing/selection/extend-by-character-002-expected.checksum:
  • editing/selection/extend-by-character-002-expected.png:
  • editing/selection/extend-by-character-003-expected.checksum:
  • editing/selection/extend-by-character-003-expected.png:
  • editing/selection/extend-by-character-004-expected.checksum:
  • editing/selection/extend-by-character-004-expected.png:
  • editing/selection/extend-by-character-005-expected.checksum:
  • editing/selection/extend-by-character-005-expected.png:
  • editing/selection/extend-by-character-006-expected.checksum:
  • editing/selection/extend-by-character-006-expected.png:
  • editing/selection/extend-by-word-001-expected.checksum:
  • editing/selection/extend-by-word-001-expected.png:
  • editing/selection/extend-by-word-002-expected.checksum:
  • editing/selection/extend-by-word-002-expected.png:
  • editing/selection/fake-doubleclick-expected.checksum:
  • editing/selection/fake-doubleclick-expected.png:
  • editing/selection/fake-drag-expected.checksum:
  • editing/selection/fake-drag-expected.png:
  • editing/selection/iframe-expected.checksum:
  • editing/selection/iframe-expected.png:
  • editing/selection/image-before-linebreak-expected.checksum:
  • editing/selection/image-before-linebreak-expected.png:
  • editing/selection/inline-table-expected.checksum:
  • editing/selection/inline-table-expected.png:
  • editing/selection/leave-requested-block-expected.checksum:
  • editing/selection/leave-requested-block-expected.png:
  • editing/selection/move-backwords-by-word-001-expected.checksum:
  • editing/selection/move-backwords-by-word-001-expected.png:
  • editing/selection/move-between-blocks-no-001-expected.checksum:
  • editing/selection/move-between-blocks-no-001-expected.png:
  • editing/selection/move-by-character-001-expected.checksum:
  • editing/selection/move-by-character-001-expected.png:
  • editing/selection/move-by-character-002-expected.checksum:
  • editing/selection/move-by-character-002-expected.png:
  • editing/selection/move-by-character-003-expected.checksum:
  • editing/selection/move-by-character-003-expected.png:
  • editing/selection/move-by-character-004-expected.checksum:
  • editing/selection/move-by-character-004-expected.png:
  • editing/selection/move-by-character-005-expected.checksum:
  • editing/selection/move-by-character-005-expected.png:
  • editing/selection/move-by-character-6-expected.checksum:
  • editing/selection/move-by-character-6-expected.png:
  • editing/selection/move-by-line-001-expected.checksum:
  • editing/selection/move-by-line-001-expected.png:
  • editing/selection/move-by-line-002-expected.checksum:
  • editing/selection/move-by-line-002-expected.png:
  • editing/selection/move-by-word-001-expected.checksum:
  • editing/selection/move-by-word-001-expected.png:
  • editing/selection/previous-line-position-expected.checksum:
  • editing/selection/previous-line-position-expected.png:
  • editing/selection/replaced-boundaries-1-expected.checksum:
  • editing/selection/replaced-boundaries-1-expected.png:
  • editing/selection/replaced-boundaries-2-expected.checksum:
  • editing/selection/replaced-boundaries-2-expected.png:
  • editing/selection/replaced-boundaries-3-expected.checksum:
  • editing/selection/replaced-boundaries-3-expected.png:
  • editing/selection/select-all-001-expected.checksum:
  • editing/selection/select-all-001-expected.png:
  • editing/selection/select-all-002-expected.checksum:
  • editing/selection/select-all-002-expected.png:
  • editing/selection/select-box-expected.checksum:
  • editing/selection/select-box-expected.png:
  • editing/selection/selection-3748164-fix-expected.checksum:
  • editing/selection/selection-3748164-fix-expected.png:
  • editing/selection/triple-click-in-pre-expected.checksum:
  • editing/selection/triple-click-in-pre-expected.png:
  • editing/selection/unrendered-001-expected.checksum:
  • editing/selection/unrendered-001-expected.png:
  • editing/selection/unrendered-002-expected.checksum:
  • editing/selection/unrendered-002-expected.png:
  • editing/selection/unrendered-003-expected.checksum:
  • editing/selection/unrendered-003-expected.png:
  • editing/selection/unrendered-004-expected.checksum:
  • editing/selection/unrendered-004-expected.png:
  • editing/selection/unrendered-005-expected.checksum:
  • editing/selection/unrendered-005-expected.png:
  • editing/style/block-style-004-expected.checksum:
  • editing/style/block-style-004-expected.png:
  • editing/style/block-style-005-expected.checksum:
  • editing/style/block-style-005-expected.png:
  • editing/style/block-style-006-expected.checksum:
  • editing/style/block-style-006-expected.png:
  • editing/style/block-styles-007-expected.checksum:
  • editing/style/block-styles-007-expected.png:
  • editing/style/remove-underline-across-paragraph-expected.checksum:
  • editing/style/remove-underline-across-paragraph-expected.png:
  • editing/style/remove-underline-across-paragraph-in-bold-expected.checksum:
  • editing/style/remove-underline-across-paragraph-in-bold-expected.png:
  • editing/style/remove-underline-after-paragraph-expected.checksum:
  • editing/style/remove-underline-after-paragraph-expected.png:
  • editing/style/remove-underline-after-paragraph-in-bold-expected.checksum:
  • editing/style/remove-underline-after-paragraph-in-bold-expected.png:
  • editing/style/remove-underline-expected.checksum:
  • editing/style/remove-underline-expected.png:
  • editing/style/remove-underline-from-stylesheet-expected.checksum:
  • editing/style/remove-underline-from-stylesheet-expected.png:
  • editing/style/remove-underline-in-bold-expected.checksum:
  • editing/style/remove-underline-in-bold-expected.png:
  • editing/style/smoosh-styles-001-expected.checksum:
  • editing/style/smoosh-styles-001-expected.png:
  • editing/style/smoosh-styles-002-expected.checksum:
  • editing/style/smoosh-styles-002-expected.png:
  • editing/style/smoosh-styles-003-expected.checksum:
  • editing/style/smoosh-styles-003-expected.png:
  • editing/style/style-3681552-fix-001-expected.checksum:
  • editing/style/style-3681552-fix-001-expected.png:
  • editing/style/style-3681552-fix-002-expected.checksum:
  • editing/style/style-3681552-fix-002-expected.png:
  • editing/style/style-3690704-fix-expected.checksum:
  • editing/style/style-3690704-fix-expected.png:
  • editing/style/style-3998892-fix-expected.checksum:
  • editing/style/style-3998892-fix-expected.png:
  • editing/style/style-boundary-001-expected.checksum:
  • editing/style/style-boundary-001-expected.png:
  • editing/style/style-boundary-002-expected.checksum:
  • editing/style/style-boundary-002-expected.png:
  • editing/style/style-boundary-003-expected.checksum:
  • editing/style/style-boundary-003-expected.png:
  • editing/style/style-boundary-004-expected.checksum:
  • editing/style/style-boundary-004-expected.png:
  • editing/style/typing-style-001-expected.checksum:
  • editing/style/typing-style-001-expected.png:
  • editing/style/typing-style-002-expected.checksum:
  • editing/style/typing-style-002-expected.png:
  • editing/style/typing-style-003-expected.checksum:
  • editing/style/typing-style-003-expected.png:
  • editing/style/unbold-in-bold-expected.checksum:
  • editing/style/unbold-in-bold-expected.png:
  • editing/style/underline-expected.checksum:
  • editing/style/underline-expected.png:
  • editing/undo/4063751-expected.checksum:
  • editing/undo/4063751-expected.png:
  • editing/undo/redo-typing-001-expected.checksum:
  • editing/undo/redo-typing-001-expected.png:
  • editing/undo/undo-typing-001-expected.checksum:
  • editing/undo/undo-typing-001-expected.png:
  • editing/unsupported-content/list-delete-001-expected.checksum:
  • editing/unsupported-content/list-delete-001-expected.png:
  • editing/unsupported-content/list-delete-002-expected.checksum:
  • editing/unsupported-content/list-delete-002-expected.png:
  • editing/unsupported-content/list-delete-003-expected.checksum:
  • editing/unsupported-content/list-delete-003-expected.png:
  • editing/unsupported-content/list-type-after-expected.checksum:
  • editing/unsupported-content/list-type-after-expected.png:
  • editing/unsupported-content/list-type-before-expected.checksum:
  • editing/unsupported-content/list-type-before-expected.png:
  • editing/unsupported-content/table-delete-001-expected.checksum:
  • editing/unsupported-content/table-delete-001-expected.png:
  • editing/unsupported-content/table-delete-002-expected.checksum:
  • editing/unsupported-content/table-delete-002-expected.png:
  • editing/unsupported-content/table-delete-003-expected.checksum:
  • editing/unsupported-content/table-delete-003-expected.png:
  • editing/unsupported-content/table-type-after-expected.checksum:
  • editing/unsupported-content/table-type-after-expected.png:
  • editing/unsupported-content/table-type-before-expected.checksum:
  • editing/unsupported-content/table-type-before-expected.png:
  • fast/css/outline-auto-empty-rects-expected.checksum:
  • fast/css/outline-auto-empty-rects-expected.png:
  • fast/css/outline-auto-location-expected.checksum:
  • fast/css/outline-auto-location-expected.png:
  • fast/dom/focus-contenteditable-expected.checksum:
  • fast/dom/focus-contenteditable-expected.png:
  • fast/encoding/utf-16-big-endian-expected.checksum:
  • fast/encoding/utf-16-big-endian-expected.png:
  • fast/encoding/utf-16-little-endian-expected.checksum:
  • fast/encoding/utf-16-little-endian-expected.png:
  • fast/events/focusingUnloadedFrame-expected.checksum:
  • fast/events/focusingUnloadedFrame-expected.png:
  • fast/forms/input-appearance-focus-expected.checksum:
  • fast/forms/input-appearance-focus-expected.png:
  • fast/forms/input-appearance-readonly-expected.checksum: Added.
  • fast/forms/input-appearance-readonly-expected.png: Added.
  • fast/forms/input-appearance-selection-expected.checksum:
  • fast/forms/input-appearance-selection-expected.png:
  • fast/forms/input-paste-undo-expected.checksum: Added.
  • fast/forms/input-paste-undo-expected.png: Added.
  • fast/forms/input-text-double-click-expected.checksum:
  • fast/forms/input-text-double-click-expected.png:
  • fast/forms/input-text-option-delete-expected.checksum:
  • fast/forms/input-text-option-delete-expected.png:
  • fast/forms/input-text-scroll-left-on-blur-expected.checksum:
  • fast/forms/input-text-scroll-left-on-blur-expected.png:
  • fast/lists/drag-into-marker-expected.checksum:
  • fast/lists/drag-into-marker-expected.png:
  • fast/overflow/childFocusRingClip-expected.checksum:
  • fast/overflow/childFocusRingClip-expected.png:
  • fast/overflow/overflow-focus-ring-expected.checksum:
  • fast/overflow/overflow-focus-ring-expected.png:
  • fast/text/justified-text-rect-expected.checksum:
  • fast/text/justified-text-rect-expected.png:
6:16 PM Changeset in webkit [13643] by darin
  • 95 edits in trunk/LayoutTests/editing/selection
  • updated layout test results, mostly for the change in focus ring appearance
6:15 PM Changeset in webkit [13642] by darin
  • 90 edits in trunk/LayoutTests/editing/inserting
  • updated layout test results, mostly for the change in focus ring appearance
6:14 PM Changeset in webkit [13641] by darin
  • 158 edits in trunk/LayoutTests/editing/deleting
  • updated layout test results, mostly for the change in focus ring appearance
6:12 PM Changeset in webkit [13640] by darin
  • 33 edits
    4 adds in trunk/LayoutTests/fast
  • updated layout test results, mostly for the change in focus ring appearance
6:11 PM Changeset in webkit [13639] by darin
  • 23 edits in trunk/WebCore

Reviewed by Adele.

  • some cleanup to how we parse user agent style sheets
  • css/CSSValueKeywords.in: Added -webkit-focus-ring-color.
  • css/cssstyleselector.cpp: (WebCore::parseUASheet): Parse an array of chars instead of UTF-16. Cuts the size of the style sheet in half. (WebCore::CSSStyleSelector::applyProperty): Allow negative value for outline-offset. Changed shadow parsing to use getColorFromPrimitiveValue instead of repeating the same logic. (WebCore::CSSStyleSelector::getColorFromPrimitiveValue): Added a case for the focus ring color.
  • rendering/render_style.h: (WebCore::RenderStyle::setOutlineOffset): Changed to allow negative values.
  • css/html4.css: Removed a lot of excess spaces. Changed color of focus to -webkit-focus-ring-color. Changed width of focus to 5px. Added an outline-offset for <input type=text> of -2px.
  • css/cssparser.cpp: (WebCore::CSSParser::parseValue): Added focus ring color as an outline color all the time, and as any other color when not in strict mode. I'm confused about what's best for this whole strict mode policy, and I may need advice on Hyatt to perfect this one later. (WebCore::CSSParser::parseColorFromValue): Removed code to pin r, g, and b because the functions in platform already take care of that. Kept the pinning of a, though because that's done in floating point before converting to an integer. (WebCore::CSSParser::parseShadow): Allow focus ring color when not in strict mode.
  • bridge/mac/FrameMac.h: Eliminated the virtual detachFromView function.
  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::FrameMac): Eliminated code to maintain the frame instances list. (WebCore::FrameMac::~FrameMac): Ditto. (WebCore::Frame::setNeedsReapplyStyles): Added.
  • bridge/mac/WebCoreSettings.mm: (-[WebCoreSettings _updateAllViews]):
  • platform/mac/WebCoreTextRendererFactory.mm: (-[WebCoreTextRendererFactory clearCaches]): Changed to call the new Page::setNeedsReapplyStylesForSettingsChange instead of using the obsolete Frame::instances.
  • page/Frame.h: Removed instances, mutableInstances, and detachFromView.
  • page/Frame.cpp: Ditto.
  • page/FrameTree.cpp: (WebCore::FrameTree::~FrameTree): Call setView(0) instead of detachFromView(). (WebCore::FrameTree::removeChild): Ditto.
  • page/Page.h:
  • page/Page.cpp: (WebCore::Page::init): Added a set of pages instead of a page count. Also register a function for when the focus ring color changes the first time this is called. (WebCore::Page::~Page): Call setView(0) instead of detachFromView. Also update to manager the set of pages. (WebCore::Page::setNeedsReapplyStyles): Call setNeedsReapplyStyles on all frames. (WebCore::Page::setNeedsReapplyStylesForSettingsChange): Call setNeedsReapplyStyles on all frames with the passed-in settings.
  • css/make-css-file-arrays.pl: Changed to run the C preprocessor on the input files and to generate an array of char instead of unsigned short.
  • platform/PlatformString.h: Added a constructor that takes a char* and a length.
  • platform/String.cpp: (WebCore::String::String): Ditto.
  • WebCore.xcodeproj/project.pbxproj: Just some tweaks; adding in a few files like the user agent style sheets.
  • platform/Color.h: Removed all use of DeprecatedString. Cleaned up a bit. Added focusRingColor and setFocusRingColorChangeFunction.
  • platform/Color.cpp: (WebCore::makeRGB): Rewrote using max and min. (WebCore::makeRGBA): Ditto. (WebCore::parseHexColor): Cleaned up a bit; changed partway to String instead of DeprecatedString. (WebCore::Color::Color): Changed to use String and to call setNamedColor to save code. (WebCore::Color::setNamedColor): Changed to use String in the interface.
  • platform/mac/ColorMac.mm: (WebCore::observeTheme): Added. Function used to start up the observer. (WebCore::setFocusRingColorChangeFunction): Added. Used to get a call back so we can update all the views when the color changes (including recomputing style to get the color change in). (WebCore::setFocusRingColorChangeFunction): Added. Returns one of the two focus ring colors. Both of these match what AppKit uses -- neither matches what we used to have in the html4.css file. (+[WebCoreControlTintObserver controlTintDidChange]): Added. Used to update when the appearance is changed from blue to graphite and back. We keep a global so we don't have to call over to AppKit every time; that's probably overkill but we need the obsever for the color change function anyway.
12:54 PM Changeset in webkit [13638] by adele
  • 3 edits
    4 adds in trunk

LayoutTests:

Reviewed by Darin.

  • fast/inline-block/overflow-clip-expected.checksum: Added.
  • fast/inline-block/overflow-clip-expected.png: Added.
  • fast/inline-block/overflow-clip-expected.txt: Added.
  • fast/inline-block/overflow-clip.html: Added.

WebCore:

Test: fast/inline-block/overflow-clip.html

Reviewed by Darin.

  • rendering/render_line.cpp: (WebCore::InlineFlowBox::placeBoxesVertically): Don't look at interior overflow when calculating the contribution to the inline's vertical overflows.
5:13 AM Changeset in webkit [13637] by eseidel
  • 11 edits in trunk/WebCore

2006-04-02 Eric Seidel <​eseidel@apple.com>

Reviewed by andersca.

Make WebCore safe against KJS::Node in JavaScriptCore private headers.

  • editing/ReplaceSelectionCommand.h:
  • khtml/ecma/JSXMLSerializer.cpp: (KJS::XMLSerializerProtoFunc::callAsFunction):
  • khtml/ecma/JSXSLTProcessor.cpp: (KJS::XSLTProcessorProtoFunc::callAsFunction):
  • khtml/ecma/kjs_binding.cpp: (KJS::ScriptInterpreter::forgetDOMNodeForDocument): (KJS::ScriptInterpreter::putDOMNodeForDocument):
  • khtml/ecma/kjs_dom.cpp: (KJS::DOMNode::DOMNode): (KJS::DOMNode::mark): (KJS::DOMNode::getValueProperty): (KJS::DOMNode::putValueProperty): (KJS::DOMNodeProtoFunc::callAsFunction): (KJS::toNode): (KJS::DOMEventTargetNode::DOMEventTargetNode): (KJS::DOMDocumentProtoFunc::callAsFunction): (KJS::DOMElement::putValueProperty): (KJS::DOMElementProtoFunc::callAsFunction): (KJS::checkNodeSecurity): (KJS::toJS): (KJS::getRuntimeObject): (KJS::DOMNamedNodesCollection::DOMNamedNodesCollection): (KJS::DOMNamedNodesCollection::getOwnPropertySlot):
  • khtml/ecma/kjs_events.cpp: (KJS::JSLazyEventListener::JSLazyEventListener): (KJS::ClipboardProtoFunc::callAsFunction):
  • khtml/ecma/kjs_html.cpp: (KJS::JSHTMLDocument::namedItemGetter): (KJS::JSHTMLElement::framesetNameGetter): (KJS::JSHTMLElement::getOwnPropertySlot): (KJS::JSHTMLElement::pushEventHandlerScope): (KJS::JSHTMLCollection::callAsFunction): (KJS::JSHTMLCollection::getNamedItems): (KJS::JSHTMLSelectCollection::put):
  • khtml/ecma/kjs_traversal.cpp: (KJS::JSNodeFilterCondition::acceptNode):
  • khtml/ecma/kjs_window.cpp: (KJS::Window::put):
  • kwq/KWQPageState.mm: (-[KWQPageState WebCore::]):
4:41 AM Changeset in webkit [13636] by eseidel
  • 1 edit
    1 add in trunk/JavaScriptGlue

2006-04-02 Eric Seidel <​eseidel@apple.com>

Reviewed by andersca.

Add missing forwarding header.

  • kxmlcore/Vector.h: Added.
12:55 AM Changeset in webkit [13635] by ggaren
  • 5 edits
    1 add in trunk/JavaScriptGlue

Reviewed by Darin.

  • Fixed <rdar://problem/4430836> JavaScript patch crashing Quartz Composer

JSValueWrappers used to hold on to the ExecState that pertained at
the time of their creation. Since ExecStates are transient, that
design was totally bogus, and it would crash once the ExecState had
been deallocated.

Unfortunately, there's no clean solution to this problem, since
the JSGlue API has no notion of state. The solution here is to use
a shared, global ExecState for the purpose of JSGlue calls. Given
the pre-existing limitations in the JSGlue API, this design
shouldn't actually introduce any new limitations (see comments in
JSValueWrapper.cpp).

I tested with Quartz Composer and PAC files, neither of which are
layout-testable.


  • JSUtils.cpp: (KJSValueToJSObject):
  • JSValueWrapper.cpp: (JSValueWrapper::JSValueWrapper): (JSValueWrapper::GetValue): (getProcessGlobalExecState): (JSValueWrapper::JSObjectCopyPropertyNames): (JSValueWrapper::JSObjectCopyProperty): (JSValueWrapper::JSObjectSetProperty): (JSValueWrapper::JSObjectCallFunction): (JSValueWrapper::JSObjectCopyCFValue):
  • JSValueWrapper.h:

Also added a test harness file. It was helpful while I was testing,
and may come in handy in the future:


  • JavaScriptGlue.xcodeproj/project.pbxproj: testjsglue.cpp: Added.
Note: See TracTimeline for information about the timeline view.