Timeline
Apr 8, 2006:
- 10:08 PM Changeset in webkit [13743] by
-
- 17 edits8 adds in trunk
LayoutTests:
Reviewed by Beth.
- tests for http://bugzilla.opendarwin.org/show_bug.cgi?id=7622 REGRESSION: New text fields should compute maxlength considering composed character sequences
- fast/forms/input-text-maxlength-expected.checksum: Added.
- fast/forms/input-text-maxlength-expected.png: Added.
- fast/forms/input-text-maxlength-expected.txt: Added.
- fast/forms/input-text-maxlength.html: Added.
- fast/forms/input-text-paste-maxlength-expected.checksum: Added.
- fast/forms/input-text-paste-maxlength-expected.png: Added.
- fast/forms/input-text-paste-maxlength-expected.txt: Added.
- fast/forms/input-text-paste-maxlength.html: Added.
WebCore:
Reviewed by Beth.
- fix http://bugzilla.opendarwin.org/show_bug.cgi?id=7622 REGRESSION: New text fields should compute maxlength considering composed character sequences
Test: fast/forms/input-text-maxlength.html
Test: fast/forms/input-text-paste-maxlength.html
- html/HTMLInputElement.h: Removed all friend classes (not needed any more). Renamed typeEnum to InputType. Made init() function private. Tweaked parameter names and formatting in many function declarations. Made canHaveSelection, selectionStart, and selectionEnd const. Made data members except for m_name private instead of protected. Added private functions constrainValue and recheckValue. Removed unused isEditable function.
- html/HTMLInputElement.cpp: (WebCore::numGraphemeClusters): Added. (WebCore::numCharactersInGraphemeClusters): Added. (WebCore::HTMLInputElement::isKeyboardFocusable): Use inputType() instead of using m_type directly. (WebCore::HTMLInputElement::isMouseFocusable): Ditto. (WebCore::HTMLInputElement::focus): Ditto. (WebCore::HTMLInputElement::setInputType): Added code to call constrainValue or recheckValue so we will enforce maxLen if changing from a type that doesn't have maxLen to one that does. (WebCore::HTMLInputElement::type): More-standard formatting for switch statement. Put cases into alphabetical order. (WebCore::HTMLInputElement::state): Changed switch statement to include all case values and not include a default case to take advantage of gcc's missing case warning. (WebCore::HTMLInputElement::restoreState): Ditto. (WebCore::HTMLInputElement::canHaveSelection): Ditto. (WebCore::HTMLInputElement::selectionStart): Ditto. (WebCore::HTMLInputElement::selectionEnd): Ditto. (WebCore::HTMLInputElement::setSelectionStart): Ditto. (WebCore::HTMLInputElement::setSelectionEnd): Ditto. (WebCore::HTMLInputElement::select): Ditto. (WebCore::HTMLInputElement::setSelectionRange): Ditto. (WebCore::HTMLInputElement::click): Ditto. (WebCore::HTMLInputElement::accessKeyAction): Ditto. (WebCore::HTMLInputElement::parseMappedAttribute): Use inputType() instead of using m_type directly. Added call to recheckValue when parsing a new value for the maxlength attribute. (WebCore::HTMLInputElement::rendererIsNeeded): Changed switch statement to include all case values and not include a default case to take advantage of gcc's missing case warning. (WebCore::HTMLInputElement::createRenderer): Ditto. (WebCore::HTMLInputElement::attach): Remove code to condition the value attribute when done parsing. This is now all handled by constrainValue and recheckValue as needed. (WebCore::HTMLInputElement::isSuccessfulSubmitButton): Use inputType() instead of using m_type directly. (WebCore::HTMLInputElement::appendFormData): Ditto. Rearranged code a little. (WebCore::HTMLInputElement::setChecked): Ditto. (WebCore::HTMLInputElement::setIndeterminate): Ditto. (WebCore::HTMLInputElement::value): Ditto. Call constrainValue when reading the value out of the value attribute. (WebCore::HTMLInputElement::valueWithDefault): Use inputType() instead of using m_type directly. Changed switch statement to include all case values and not include a default case to take advantage of gcc's missing case warning. (WebCore::HTMLInputElement::setValue): Ditto. Call constrainValue when storing a value. (WebCore::HTMLInputElement::setValueFromRenderer): Added an assertion. (WebCore::HTMLInputElement::storesValueSeparateFromAttribute): Use inputType() instead of using m_type directly. (WebCore::HTMLInputElement::preDispatchEventHandler): Ditto. (WebCore::HTMLInputElement::postDispatchEventHandler): Ditto. (WebCore::HTMLInputElement::defaultEventHandler): Ditto. Changed code to truncate inserted text in a BeforeTextInsertedEvent to use the new constrainValue function and also the numGraphemeClusters function, so it's based on grapheme clusters instead of characters and shares code. (WebCore::HTMLInputElement::constrainValue): Added. (WebCore::HTMLInputElement::recheckValue): Added.
- html/HTMLIsIndexElement.cpp: (WebCore::HTMLIsIndexElement::HTMLIsIndexElement): Removed unneeded code to set m_type to TEXT, which is already what it gets set to by the base class's constructor.
- html/HTMLGenericFormElement.h: Removed unused isEditable function.
- html/HTMLGenericFormElement.cpp: Ditto.
- html/HTMLTextAreaElement.h: Ditto.
- html/HTMLTextAreaElement.cpp: Ditto.
- platform/StringImpl.cpp: (WebCore::StringImpl::truncate): Changed > to >= so that truncating to the size of the string does nothing, efficiently.
- rendering/RenderText.h: Added declaration of characterBreakIterator.
- rendering/RenderText.cpp: (WebCore::characterBreakIterator): Made this public so it can be used in other files. Maybe we should also move it to another source file later. Also renamed to remove the "get" from the title. (WebCore::RenderText::previousOffset): Updated for name change. (WebCore::RenderText::nextOffset): Updated for name change.
- rendering/RenderTextField.cpp: (WebCore::RenderTextField::updateFromElement): Removed code to implement maxlength checking. That's handled entirely in the DOM now. Also moved down the code to get the value into a string so that it's done only in the case where the string is used.
- rendering/render_form.cpp: (WebCore::RenderFileButton::valueChanged): Use setValueFromRenderer instead of setting the value directly in the input element. We changed this for all the other types a while ago, and it works just as well for the input element.
- dom/BeforeTextInsertedEvent.h: Added setText function. Previously, clients changed the text by modifying the text object in place, but going forward we'd like to avoid that sort of thing.
- editing/ReplaceSelectionCommand.cpp: (WebCore::ReplacementFragment::ReplacementFragment): Changed code to assume clients will change the text in the event rather than mutating the text object itself. This is compatible with possible future changes to String to be copy on write.
- editing/TypingCommand.cpp: (WebCore::TypingCommand::insertText): Ditto.
- 1:51 PM Changeset in webkit [13801] by
-
- 1 edit in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs/TreeCode.cpp
added missing fall through to case code
- 1:50 PM Changeset in webkit [13800] by
-
- 2 edits in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs
refined rules for what the current completion state is
- 1:40 PM Changeset in webkit [13799] by
-
- 2 edits in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore
added shrinkTo and growTo to vector for speed gain
- 12:08 PM Changeset in webkit [13798] by
-
- 2 edits in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs
fix try and switch
- 12:04 PM Changeset in webkit [13742] by
-
- 2 edits in trunk/JavaScriptCore
Reviewed by Darin.
One more attempt - use reinterpret_cast, rather than static_cast.
- 11:54 AM Changeset in webkit [13741] by
-
- 2 edits in trunk/JavaScriptCore
Reviewed by Darin.
An attempt to fix Win32 build - ICU uses wchar_t on Windows, so we need a type cast.
- kxmlcore/unicode/icu/UnicodeIcu.h: (KXMLCore::Unicode::toLower): (KXMLCore::Unicode::toUpper):
- 10:21 AM Changeset in webkit [13740] by
-
- 11 edits3 copies3 adds in trunk
Reviewed by Darin.
- fix http://bugzilla.opendarwin.org/show_bug.cgi?id=8264 toLowerCase and toUpperCase don't honor special mappings
Test: fast/js/string-capitalization.html
- JavaScriptCore.xcodeproj/project.pbxproj: Added KXMLCore::Unicode headers to the project.
- icu/unicode/putil.h: Added (copied from WebCore).
- icu/unicode/uiter.h: Ditto.
- icu/unicode/ustring.h: Ditto.
- kjs/string_object.cpp: (StringProtoFunc::callAsFunction): Use the new KXMLCore::Unicode::toUpper() and toLower().
- kjs/ustring.cpp: Removed unused (and evil) UChar::toLower() and toUpper().
- kjs/ustring.h: Ditto.
- kxmlcore/unicode/Unicode.h: Corrected capitalization of the word Unicode.
- kxmlcore/unicode/UnicodeCategory.h: Renamed include guard macro to match file name.
- kxmlcore/unicode/icu/UnicodeIcu.h: (KXMLCore::Unicode::toLower): Work on strings, not individual characters. Use ICU root locale. (KXMLCore::Unicode::toUpper): Ditto. (KXMLCore::Unicode::isFormatChar): Use int32_t, which can hold a complete code point. (KXMLCore::Unicode::isSeparatorSpace): Ditto. (KXMLCore::Unicode::category): Ditto.
- kxmlcore/unicode/qt4/UnicodeQt4.h: (KXMLCore::Unicode::toLower): Work on strings, not individual characters. (KXMLCore::Unicode::toUpper): Ditto. (KXMLCore::Unicode::isFormatChar): Use int32_t, which can hold a complete code point. (KXMLCore::Unicode::isSeparatorSpace): Ditto. (KXMLCore::Unicode::category): Ditto.
- tests/mozilla/ecma/String/15.5.4.12-1.js: Corrected expected results.
- tests/mozilla/ecma/String/15.5.4.12-5.js: Corrected expected results.
- 9:49 AM Changeset in webkit [13739] by
-
- 2 edits in trunk/WebKit
Reviewed by Adele Peterson.
- fixed http://bugzilla.opendarwin.org/show_bug.cgi?id=8260 REGRESSION: Assertion failure: ![_private->iconsToSaveWithURLs containsObject:iconURLString] in WebIconDatabase.m:695-[WebIconDatabase(WebInternal) _retainIconForIconURLString:]
- Misc/WebIconDatabase.m: (-[WebIconDatabase _retainIconForIconURLString:]): This new assertion was one block too high; moved it in.
Apr 7, 2006:
- 6:40 PM Changeset in webkit [13797] by
-
- 1 edit in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs/TreeCode.cpp
fix assertion in ForInNodeExecutePopBreakUnwindBarrierState
- 6:31 PM Changeset in webkit [13796] by
-
- 1 edit in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs/TreeCode.cpp
Fix FALL_THROUGH in debug builds
- 6:26 PM Changeset in webkit [13795] by
-
- 2 edits in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs
fixed labeled break and continue
- 6:16 PM Changeset in webkit [13794] by
-
- 1 edit in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs/TreeCode.cpp
add missing break in CommaNodeEvaluateState
- 6:02 PM Changeset in webkit [13793] by
-
- 2 edits in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs
Add FALL_THROUGH macro
- 5:47 PM Changeset in webkit [13792] by
-
- 1 edit in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs/TreeCode.cpp
fixed do-while by adding fake true value at the front
- 5:34 PM Changeset in webkit [13738] by
-
- 4 edits4 adds in trunk
LayoutTests:
Reviewed by Hyatt.
- test for http://bugzilla.opendarwin.org/show_bug.cgi?id=8134 REGRESSION: dragging down from the middle of a text field does not select to end of field
- fast/forms/input-text-drag-down-expected.checksum: Added.
- fast/forms/input-text-drag-down-expected.png: Added.
- fast/forms/input-text-drag-down-expected.txt: Added.
- fast/forms/input-text-drag-down.html: Added.
WebCore:
Reviewed by Hyatt.
- test for http://bugzilla.opendarwin.org/show_bug.cgi?id=8134 REGRESSION: dragging down from the middle of a text field does not select to end of field
- editing/Selection.cpp: (WebCore::comparePositions): Added. Takes shadow content into account. (WebCore::Selection::validate): Changed to call comparePositions instead of calling Range::compareBoundaryPoints directly. Also removed unneeded code to redundantly set m_start and m_end to null and did a bit of reformatting.
- some tiny efficiency improvements to the tokenizer -- no measurable speedup, but removes a little bit of unneeded code
- html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::parseTag): Changed all the places that do "unsigned short x = *c" to "unsigned short x = c->unicode()" when c is a QChar, otherwise we do an unnecessary conversion to char (which requires a branch to see if the c fits in a char).
- 5:27 PM Changeset in webkit [13791] by
-
- 3 edits in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs
fix exception handling and loop continue states
- 5:21 PM Changeset in webkit [13737] by
-
- 6 edits in trunk
Reviewed by harrison
Rolled the fix for 8250 back in and fixed a bug:
The local variables for the first and last nodes in the fragment
need to be reset when the fragment is changed for plaintext-only mode
or a change from the beforetextinserted event handler.
- editing/ReplaceSelectionCommand.cpp: (WebCore::ReplacementFragment::ReplacementFragment):
- 5:16 PM Changeset in webkit [13736] by
-
- 2 edits in trunk/WebKit
A fix that makes coalesced updates work much better (and makes our
single animated GIF timer work better). The new rect painting algorithm
for WebHTMLView will use the single unioned rect if the # of rects exceeds
a threshold (10 is my initial cut), or if the union has enough "wasted"
additional pixels (conservatively set at 75%).
Reviewed by darin
- WebView/WebHTMLView.m: (-[NSArray drawSingleRect:]): (-[NSArray drawRect:]):
- 5:11 PM Changeset in webkit [13790] by
-
- 1 edit in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs/TreeCode.cpp
fixed missing break in property list node
- 5:08 PM Changeset in webkit [13789] by
-
- 1 edit in branches/temp-tree-code/trunk/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj
fixing missing break statement in property list node
- 4:55 PM Changeset in webkit [13735] by
-
- 3 edits2 adds in trunk
LayoutTests:
Reviewed by Darin.
Test case for this fix:
- fixed REGRESSION: offsetParent on element with no offset parent crashes
- fast/dom/Element/null-offset-parent.html: Added.
WebCore:
Reviewed by Darin.
- fixed REGRESSION: offsetParent on element with no offset parent crashes
- dom/Element.cpp: (WebCore::Element::offsetParent): Add missing null check.
- 4:50 PM Changeset in webkit [13734] by
-
- 3 edits in trunk/WebKit
Reviewed by Darin.
- simplify archive loading
- WebView/WebDataSource.m: (-[WebDataSource representation]): fix whitespace
- WebView/WebHTMLRepresentation.m: (-[WebHTMLRepresentation loadArchive]): Use WebFrame's loadArchive: instead of trying to do a manual load here. (-[WebHTMLRepresentation documentSource]): No more special case for WebArchive.
- 4:42 PM Changeset in webkit [13788] by
-
- 2 edits in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs
fixed up switch / case statements
- 4:35 PM Changeset in webkit [13787] by
-
- 2 edits in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs
much better unwind barrier printing
- 4:01 PM Changeset in webkit [13733] by
-
- 2 edits in trunk/WebKit
Reviewed by Tim Omernick.
- fixed <rdar://problem/4507054> If an icon file is removed from disk behind Safari's back, Safari will not try to refetch it
In addition to letting WebIconDatabase recover from disk/memory mismatches as per 4507054, I also found
a bug in the way icons are remembered that could account for some of the other cases where
site icons didn't appear (and you'd get an ERROR on debug builds about WebIconDatabase saying it
had some icon when it really doesn't).
- Misc/WebIconDatabase.m: (-[WebIconDatabase iconForURL:withSize:cache:]): When the icon file for a previously-saved site icon isn't found, forget about the darn icon URL thoroughly so that this situation is self-correcting. Formerly, once you got into this state WebKit would never find the icon again (short of removing the entire icon database cleanly). Note that this does *not* change the behavior of sites that didn't have a site icon when WebKit checked -- these will continue to not return a site icon indefinitely (that's a separate, possibly performance-sensitive issue). (-[WebIconDatabase _retainIconForIconURLString:]): This code did the wrong thing in the case where an icon was in the process of being forgotten about. In that case, the icon would still be in _private->iconsOnDiskWithURLs, so we wouldn't try to save it here, so the new icon wouldn't stick.
- 3:25 PM Changeset in webkit [13732] by
-
- 3 edits4 adds in trunk
LayoutTests:
Reviewed by adele
- fast/forms/input-maxlength-2-expected.checksum: Added.
- fast/forms/input-maxlength-2-expected.png: Added.
- fast/forms/input-maxlength-2-expected.txt: Added.
- fast/forms/input-maxlength-2.html: Added.
WebCore:
Reviewed by adele
maxlength truncation in text fields didn't work if the fragment was a
single text node.
- editing/ReplaceSelectionCommand.cpp: (WebCore::ReplacementFragment::ReplacementFragment):
- 2:57 PM Changeset in webkit [13731] by
-
- 1 edit in trunk/WebCore/editing/ReplaceSelectionCommand.cpp
Rolling out the patch I just checked in.
- 1:46 PM Changeset in webkit [13730] by
-
- 3 edits4 adds in trunk
LayoutTests:
Reviewed by harrison
<http://bugzilla.opendarwin.org/show_bug.cgi?id=8250>
REGRESSION: Interchange newlines aren't passed with the khtmlBeforeTextInsertedEvent
- fast/forms/8250-expected.checksum: Added.
- fast/forms/8250-expected.png: Added.
- fast/forms/8250-expected.txt: Added.
- fast/forms/8250.html: Added.
WebCore:
Reviewed by harrison
<http://bugzilla.opendarwin.org/show_bug.cgi?id=8250>
REGRESSION: Interchange newlines aren't passed with the khtmlBeforeTextInsertedEvent
- editing/ReplaceSelectionCommand.cpp: (WebCore::ReplacementFragment::ReplacementFragment): Interchange content removal happened before khtmlBeforeTextInsertedEvent was sent.
- 12:51 PM Changeset in webkit [13786] by
-
- 2 edits in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs
unify exception handling
- 12:36 PM Changeset in webkit [13729] by
-
- 7 edits in trunk
LayoutTests:
Reviewed by adele
<http://bugzilla.opendarwin.org/show_bug.cgi?id=8219>
REGRESSION: Two extra newlines added when pasting a single styled line into a plaintext-only region
Updated to relfect fix:
- editing/pasteboard/8145-1-expected.checksum:
- editing/pasteboard/8145-1-expected.png:
- editing/pasteboard/8145-1-expected.txt:
- editing/pasteboard/8145-1.html:
WebCore:
Reviewed by adele
<http://bugzilla.opendarwin.org/show_bug.cgi?id=8219>
REGRESSION: Two extra newlines added when pasting a single styled line into a plaintext-only region
- editing/ReplaceSelectionCommand.cpp: (WebCore::ReplacementFragment::ReplacementFragment): Converting the fragment to plaintext introduced an extraneous newline because the range passed to plainText ended after the paragraph containing the fragment built from the markup that TextEdit put on the paste board. TextIterator will emit a newline when it exits a paragraph. Two extra newlines were added because the '\n' in the plaintext string turns into an interchange newline, which isn't removed because of 8250, and the interchange newline looks like inline content that requires the insertion of a paragraph separator during paste. Fixed by creating a range using VisiblePositions at the start and the end of the node that holds the fragment during paste's test rendering.
- 11:41 AM Changeset in webkit [13728] by
-
- 9 edits12 adds in trunk
LayoutTests:
Reviewed by harrison
<http://bugzilla.opendarwin.org/show_bug.cgi?id=8145>
REGRESSION: Pasting text from TextEdit with a bold word into text field results in crash
- editing/pasteboard/8145-1-expected.checksum: Added.
- editing/pasteboard/8145-1-expected.png: Added.
- editing/pasteboard/8145-1-expected.txt: Added.
- editing/pasteboard/8145-1.html: Added.
- editing/pasteboard/8145-2-expected.checksum: Added.
- editing/pasteboard/8145-2-expected.png: Added.
- editing/pasteboard/8145-2-expected.txt: Added.
- editing/pasteboard/8145-2.html: Added.
- editing/pasteboard/8145-3-expected.checksum: Added.
- editing/pasteboard/8145-3-expected.png: Added.
- editing/pasteboard/8145-3-expected.txt: Added.
- editing/pasteboard/8145-3.html: Added.
- fast/forms/plaintext-mode-1-expected.txt:
- fast/forms/plaintext-mode-1.html:
WebCore:
Reviewed by harrison
<http://bugzilla.opendarwin.org/show_bug.cgi?id=8145>
REGRESSION: Pasting text from TextEdit with a bold word into text field results in crash
- editing/AppendNodeCommand.cpp: (WebCore::AppendNodeCommand::doApply): Assert that the node will be placed somewhere that's contenteditable.
- editing/InsertNodeBeforeCommand.cpp: (WebCore::InsertNodeBeforeCommand::doApply): Ditto
- editing/JSEditor.cpp: Enabled insertHTML for plaintext-only regions since it's useful for debugging rich content pastes into the new text fields.
- editing/ReplaceSelectionCommand.cpp: (WebCore::ReplaceSelectionCommand::doApply): 1) After the first paragraph of the fragment has been merged with the first part of the paragraph where the paste occured, if the next node to be inserted is inline, we put it in a new paragraph because it was at the start of a paragraph in the fragment. The change is to insert a paragraph separator if insertionPos.next() is null or outside of the current editable region. 2) Before the paste begins, a paragraph separator is inserted in order to avoid nesting blocks from the fragment to be pasted inside the block where the paste will occur. I made two fixes to the code that decides whether or not to insert the paragraph separator and added testcases for each. Added a fixme because it appears that this code is also used to ensure that the aforementioned insertionPos will be at the end of a paragraph. This code should only be about preventing nesting.
- 10:06 AM Changeset in webkit [13727] by
-
- 1 add in branches/Safari-2-0-branch/JavaScriptGlue/Makefile
Missing a makefile.
- 1:31 AM Changeset in webkit [13726] by
-
- 13 edits3 adds in trunk
LayoutTests:
Reviewed by Anders.
Added test case for incidental fix from this change:
- Convert Element JS bindings to be almost completely autogenerated http://bugzilla.opendarwin.org/show_bug.cgi?id=8227
- fast/dom/Element/dimension-properties-unrendered-expected.txt: Added.
- fast/dom/Element/dimension-properties-unrendered.html: Added.
- fast/dom/Element/resources/dimension-properties-unrendered.js: Added.
WebCore:
Reviewed by Anders.
- Convert Element JS bindings to be almost completely autogenerated http://bugzilla.opendarwin.org/show_bug.cgi?id=8227
- dom/Element.idl: Declare full interface in IDL.
- dom/Element.h: (WebCore::Element::tagQName): Renamed from tagName, so the real DOM method can be called tagName. (WebCore::Element::tagName): inline alias for nodeName.
- dom/Element.cpp: (WebCore::Element::scrollByUnits): Moved logic from JS bindings to core DOM. (WebCore::Element::scrollByLines): ditto (WebCore::Element::scrollByPages): ditto (WebCore::Element::offsetLeft): ditto, plus make unrendered elements return 0 not undefined (WebCore::Element::offsetTop): ditto, plus make unrendered elements return 0 not undefined (WebCore::Element::offsetWidth): ditto, plus make unrendered elements return 0 not undefined (WebCore::Element::offsetHeight): ditto, plus make unrendered elements return 0 not undefined (WebCore::Element::offsetParent): ditto (WebCore::Element::clientWidth): ditto, plus make unrendered elements return 0 not undefined (WebCore::Element::clientHeight): ditto, plus make unrendered elements return 0 not undefined (WebCore::Element::scrollLeft): ditto (WebCore::Element::scrollTop): ditto (WebCore::Element::setScrollLeft): ditto (WebCore::Element::setScrollTop): ditto (WebCore::Element::scrollWidth): ditto, plus make unrendered elements return 0 not undefined (WebCore::Element::scrollHeight): ditto, plus make unrendered elements return 0 not undefined
- bindings/scripts/CodeGeneratorJS.pm: Added support for ConvertUndefinedToTrue, for benefit of scroll methods where omitted arguments should be treated as true. However, maybe explicit overloading in the IDL would be a better long-term approach for optional arguments.
- khtml/ecma/kjs_binding.cpp: (KJS::valueToStringWithNullCheck): fixed formatting (KJS::valueToBooleanTreatUndefinedAsTrue): added for binding of scrollIntoView (isn't this lame?)
- khtml/ecma/kjs_binding.h: (KJS::toJS): fixed formatting, added new stuff
- khtml/ecma/kjs_dom.cpp: (KJS::DOMElement::getValueProperty): removed most of contents (KJS::DOMElement::putValueProperty): ditto (KJS::DOMElementProtoFunc::callAsFunction): ditto
- css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::canShareStyleWithElement): updated for rename of tagName to tagQName (WebCore::CSSStyleSelector::checkOneSelector): ditto
- dom/Document.cpp: (WebCore::Document::importNode): ditto
- editing/ApplyStyleCommand.cpp: (WebCore::ApplyStyleCommand::removeInlineStyle): ditto (WebCore::areIdenticalElements): ditto
- html/HTMLElement.cpp: (WebCore::HTMLElement::inEitherTagList): ditto (WebCore::HTMLElement::inInlineTagList): ditto (WebCore::HTMLElement::inBlockTagList): ditto
- 1:04 AM Changeset in webkit [13725] by
-
- 2 edits in trunk/WebKit
Re-enable coalesced updates. Our move to a single timer has essentially
implemented coalescing of updates anyway, so the regression has already
come back. Since we have to deal with that now anyway, there's no reason
to visually tear also. :)
Reviewed by darin
- ChangeLog:
- WebView/WebView.m: (-[WebView _commonInitializationWithFrameName:groupName:]):
- 12:26 AM Changeset in webkit [13785] by
-
- 3 edits in branches/temp-tree-code/branches/evaluate-execute-merge/JavaScriptCore/kjs
further fixes to exception handling, as well as doWhile and new expressions