⚠ Archived content — this site is no longer maintained.   Current WebKit documentation is at docs.webkit.org.

Timeline



Jul 13, 2006:

11:48 PM Changeset in webkit [15419] by adele
  • 2 edits
    1 add in trunk/WebCore

Reviewed by Maciej.

  • Fix for <rdar://problem/4604703> REGRESSION (NativeTextField): Focus is not removed from password field after I ctrl-click into a different field

Test:

  • manual-tests/password-ctrl-click-lose-focus.html: Added.
  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::sendContextMenuEvent): If we're about to set a selection in the current view, make sure its the first responder. In this case, this will cause the password field to resign first responder at the right time.
9:42 PM Changeset in webkit [15418] by ddkilzer
  • 20 edits
    4 adds in trunk

LayoutTests:

Reviewed by Hyatt. Patch by Mitz.

  • editing/deleting/delete-to-select-table-expected.txt:
  • editing/execCommand/boldSelection-expected.txt:
  • editing/execCommand/italicizeByCharacter-expected.txt:
  • editing/execCommand/modifyForeColorByCharacter-expected.txt:
  • editing/execCommand/print-expected.txt:
  • editing/execCommand/selectAll-expected.txt:
  • editing/execCommand/strikethroughSelection-expected.txt:
  • editing/inserting/insert-div-027-expected.txt:
  • editing/selection/after-line-wrap-expected.txt:
  • editing/selection/mixed-editability-1-expected.txt:
  • editing/selection/select-all-001-expected.txt:
  • editing/selection/select-all-002-expected.txt:
  • editing/selection/select-all-003-expected.txt:
  • fast/clip/outline-overflowClip-expected.txt:
  • fast/text/international/rtl-white-space-pre-wrap-expected.checksum: Added.
  • fast/text/international/rtl-white-space-pre-wrap-expected.png: Added.
  • fast/text/international/rtl-white-space-pre-wrap-expected.txt: Added.
  • fast/text/international/rtl-white-space-pre-wrap.html: Added.
  • fast/text/whitespace/pre-wrap-overflow-selection-expected.txt:
  • fast/text/whitespace/pre-wrap-spaces-after-newline-expected.txt:

WebCore:

Reviewed by Hyatt. Patch by Mitz.

Test: fast/text/international/rtl-white-space-pre-wrap.html

  • rendering/RenderText.cpp: (WebCore::RenderText::caretRect): Fixed LTR case and added the RTL case of clipping the caret position to the text box when the caret is after the trailing space of an autowrapped line.
  • rendering/bidi.cpp: (WebCore::RenderBlock::computeHorizontalPositionsForLine): Changed to truncate- to-fit the logically last text box if it contains the trailing spaces of an autowrapped line. (WebCore::RenderBlock::bidiReorderLine): Remember the logically last text run. In the case of autowrapped text with white space that overflows beyond the line, the last text run is the one containing the overflowing white space. (WebCore::RenderBlock::findNextLineBreak): Split overflowing white space on a line that autowraps only after white space into a separate text run.
8:29 PM Changeset in webkit [15417] by ddkilzer
  • 7 edits
    3 adds in trunk

JavaScriptCore:

Reviewed by Geoffrey. Patch by Mark Rowe.

​http://bugzilla.opendarwin.org/show_bug.cgi?id=9742
Bug 9742: REGRESSION: WebKit hangs when loading <​http://www.vtbook.com>

  • kjs/value.h: (KJS::JSValue::getUInt32): Only types tagged as numeric can be converted to UInt32.

LayoutTests:

Reviewed by Geoffrey. Patch by Mark Rowe.

​http://bugzilla.opendarwin.org/show_bug.cgi?id=9742
Bug 9742: REGRESSION: WebKit hangs when loading <​http://www.vtbook.com>

This includes tests for indexing an array with immediate types, and for
setting selectField.options.length to immediate types.

  • fast/dom/select-selectedIndex-expected.txt:
  • fast/dom/select-selectedIndex-multiple-expected.txt:
  • fast/dom/select-selectedIndex-multiple.html: Update to test setting length to true and false.
  • fast/dom/select-selectedIndex.html: Ditto.
  • fast/js/array-index-immediate-types-expected.txt: Added.
  • fast/js/array-index-immediate-types.html: Added.
  • fast/js/resources/array-index-immediate-types.js: Added.
7:37 PM Changeset in webkit [15416] by bdakin
  • 2 edits in trunk/WebKit

Reviewed by Darin.

  • WebView/WebHTMLView.m: (-[WebHTMLView _updateMouseoverWithEvent:]): Return immediately if the view has already been closed.
4:17 PM Changeset in webkit [15415] by harrison
  • 4 edits in trunk/WebCore

Reviewed by Dave Hyatt.

<rdar://problem/4624203> -webkit-highlight should be behind images

  • rendering/RenderImage.cpp: (WebCore::RenderImage::paint):
  • rendering/RenderListMarker.cpp: (WebCore::RenderListMarker::paint):
  • rendering/RenderWidget.cpp: (WebCore::RenderWidget::paint): Call custom highlighter before painting the image, marker, or widget.
2:08 PM Changeset in webkit [15414] by harrison
  • 8 edits
    8 adds in trunk

LayoutTests:

Reviewed by Justin and Levi.

<rdar://problem/4620743> REGRESSION: Option-Delete doesn't delete words during typing

  • editing/deleting/delete-by-word-001-expected.checksum: Added.
  • editing/deleting/delete-by-word-001-expected.png: Added.
  • editing/deleting/delete-by-word-001-expected.txt: Added.
  • editing/deleting/delete-by-word-001.html: Added.
  • editing/deleting/delete-by-word-002-expected.checksum: Added.
  • editing/deleting/delete-by-word-002-expected.png: Added.
  • editing/deleting/delete-by-word-002-expected.txt: Added.
  • editing/deleting/delete-by-word-002.html: Added.

WebCore:

Reviewed by Justin and Levi.

<rdar://problem/4620743> REGRESSION: Option-Delete doesn't delete words during typing

  • Tests: editing/deleting/delete-by-word-001.html editing/deleting/delete-by-word-002.html
  • editing/TypingCommand.cpp: (WebCore::TypingCommand::deleteKeyPressed): (WebCore::TypingCommand::forwardDeleteKeyPressed): (WebCore::TypingCommand::doApply):
  • editing/TypingCommand.h: Delete and forward delete to use specified granularity. Undo of delete and forward delete to select what had been deleted (non-char granularity only).


  • page/Frame.cpp: (WebCore::Frame::setSelection): Close typing and end style even if selection is not changing.
  • page/Frame.h: Remove unused setSelection parameter keepTypingStyle.

WebKit:

Reviewed by Justin and Levi.

<rdar://problem/4620743> REGRESSION: Option-Delete doesn't delete words during typing

  • Tests: editing/deleting/delete-by-word-001.html editing/deleting/delete-by-word-002.html
  • WebView/WebHTMLView.m: (-[WebHTMLView _deleteRange:killRing:prepend:smartDeleteOK:deletionAction:granularity:]):
11:45 AM Changeset in webkit [15413] by thatcher
  • 3 edits in tags/WebKit-521.16.4/WebKit

Reviewed by Darin.

<rdar://problem/4616920> REGRESSION: tabbing in mail moves focus
to next control instead of inserting a tab space.

Change editible WebView's tabKeyCyclesThroughElements to NO only
if the setTabKeyCyclesThroughElements SPI wasn't called.

  • WebView/WebView.m: (-[WebView setEditable:]):
11:44 AM Changeset in webkit [15412] by thatcher
  • 3 edits in tags/WebCore-521.16.1/WebCore

Reviewed by Brady Eidson


  • fixed <rdar://problem/4611164> REGRESSION: Crash occurs when undoing a series of misspelled words (WebCore::RenderObject::repaint(bool)
  • dom/Document.cpp: (WebCore::Document::removeMarkers): put (it - markers.begin()) in a local variable before altering markers, in every case where this was happening. One of the cases like this was fixed a while back, but other cases were either missed at that time or crept in since.
11:34 AM Changeset in webkit [15411] by thatcher
  • 1 copy in tags/WebKit-521.16.4

New tag.

11:33 AM Changeset in webkit [15410] by thatcher
  • 1 copy in tags/WebCore-521.16.1/WebCore

New tag (part 2)

11:32 AM Changeset in webkit [15409] by thatcher
  • 1 add in tags/WebCore-521.16.1

New tag.

11:28 AM Changeset in webkit [15408] by lweintraub
  • 11 edits
    8 adds in trunk

LayoutTests:

Reviewed by justin

<rdar://problem/4622763> Deleting from beginning of paragraph following a table deletes rather than selects the table

  • editing/deleting/delete-block-table-expected.checksum: Added.
  • editing/deleting/delete-block-table-expected.png: Added.
  • editing/deleting/delete-block-table-expected.txt: Added.
  • editing/deleting/delete-block-table.html: Added.
  • editing/deleting/delete-hr-expected.checksum: Added.
  • editing/deleting/delete-hr-expected.png: Added.
  • editing/deleting/delete-hr-expected.txt: Added.
  • editing/deleting/delete-hr.html: Added.
  • editing/deleting/delete-to-select-table-expected.png:
  • editing/execCommand/insertHorizontalRule-expected.txt:
  • fast/lists/markers-in-selection-expected.txt:

WebCore:

Reviewed by justin

<rdar://problem/4622763> Deleting from beginning of paragraph following a table deletes rather than selects the table

  • editing/DeleteSelectionCommand.cpp: (WebCore::DeleteSelectionCommand::initializeStartEnd): Added selection expansion for HRs. (WebCore::DeleteSelectionCommand::initializePositionData): Now uses enclosingBlockFlowOrTableElement instead of enclosingBlockFlowElement. (WebCore::DeleteSelectionCommand::removeNode): Use to identify that we need a placeholder when the start or end block is removed. (WebCore::DeleteSelectionCommand::handleGeneralDelete): Added check for canHaveChildrenForEditing to keep things like HRs from being given children. (WebCore::DeleteSelectionCommand::doApply): Switched to use member variable for needPlaceholder.
  • editing/DeleteSelectionCommand.h: Made needPlaceholder a member variable.
  • editing/TypingCommand.cpp: (WebCore::TypingCommand::deleteKeyPressed): Fixed selection bug for tables and open typing commands. (WebCore::TypingCommand::forwardDeleteKeyPressed): Ditto.
  • editing/htmlediting.cpp: (WebCore::editingIgnoresContent): Added check for HRs, since it's not considered a widget.
  • editing/visible_units.cpp: (WebCore::startOfParagraph): Fix for HRs and tables. (WebCore::endOfParagraph): Ditto.
11:27 AM Changeset in webkit [15407] by thatcher
  • 3 edits in trunk/WebKit

Rolling out this earlier change (r15378) now that it is fixed on AGL's end.
Fixes <rdar://problem/4624865> Restore 64-bit OpenGL plug-in support once AGL is 64-bit

<rdar://problem/4624858> AGL isn't 64-bit yet; temporarily remove it from WebKit 64-bit build

  • Plugins/WebBaseNetscapePluginView.h:
  • Plugins/WebBaseNetscapePluginView.m:
11:06 AM Changeset in webkit [15406] by thatcher
  • 2 edits in trunk/WebKit

Reviewed by Darin.

<rdar://problem/4616920> REGRESSION: tabbing in mail moves focus
to next control instead of inserting a tab space.

Change editible WebView's tabKeyCyclesThroughElements to NO only
if the setTabKeyCyclesThroughElements SPI wasn't called.

  • WebView/WebView.m: (-[WebView setEditable:]):
8:54 AM Changeset in webkit [15405] by ap
  • 2 edits in trunk/WebCore

Reviewed by Darin.

No change in behavior, thus no test included.

  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::setStatusBarText): Use a local autorelease pool to release the temporaries - the test runs non-stop, and the enclosing pool doesn't get a chance to be drained.
1:56 AM Changeset in webkit [15404] by ggaren
  • 14 edits in trunk/JavaScriptCore

Pleasing to Maciej.


  • Renamed JSEvaluate -> JSEvaluateScript, JSCheckSyntax -> JSCheckScriptSyntax
  • Added exception out parameters to JSValueTo* and JSValueIsEqual because they can throw
  • Removed JSObjectGetDescription because it's useless and vague, and JSValueToString/JSValueIsObjectOfClass do a better job, anyway
  • Clarified comments about "IsFunction/Constructor" to indicate that they are true of all functions/constructors, not just those created by JSObjectMake*
12:11 AM Changeset in webkit [15403] by thatcher
  • 2 edits in trunk/WebKitTools

Reviewed by Timothy.

Bug 9889: [Drosera] Stepping out when paused after last statement in function skips a frame
​http://bugzilla.opendarwin.org/show_bug.cgi?id=9889

  • Drosera/debugger.js: Track whether we paused during the execution of willLeaveFrame. If so, have stepOut pause on the next call to willExecuteStatement rather than second.

Jul 12, 2006:

9:52 PM Changeset in webkit [15402] by ddkilzer
  • 3 edits
    2 adds in trunk

WebCore:

Reviewed by Adele. Patch by Mitz.

Test: fast/frames/repaint-display-none-crash.html

  • rendering/RenderView.cpp: (WebCore::RenderView::repaintViewRectangle): Added null checking of the owner element's renderer, which can be null if the iframe is set to display:none.

LayoutTests:

Reviewed by Adele. Patch by Mitz.

  • fast/frames/repaint-display-none-crash-expected.txt: Added.
  • fast/frames/repaint-display-none-crash.html: Added.
7:31 PM Changeset in webkit [15401] by justing
  • 14 edits
    2 adds in trunk

LayoutTests:

Reviewed by levi


<rdar://problem/4509393>
selected DOM range starts with <object>, 0 offset but selection should include the <object>

  • editing/selection/legal-positions-expected.txt: Added.
  • editing/selection/legal-positions.html: Added.
  • editing/deleting/delete-3608430-fix-expected.txt:
  • editing/deleting/delete-image-001-expected.txt:
  • editing/deleting/delete-image-002-expected.txt:
  • editing/deleting/delete-image-003-expected.txt:
  • editing/deleting/delete-image-004-expected.txt:
  • editing/selection/drag-to-contenteditable-iframe-expected.txt:
  • editing/selection/replaced-boundaries-3-expected.txt:
  • editing/selection/select-box-expected.txt:

WebCore:

Reviewed by levi


<rdar://problem/4509393>
selected DOM range starts with <object>, 0 offset but selection should include the <object>

  • editing/ReplaceSelectionCommand.cpp: (WebCore::ReplaceSelectionCommand::shouldMergeStart): Don't pull content out of a table cell.
  • editing/htmlediting.cpp: (WebCore::editingIgnoresContent): Added <select> nodes. (WebCore::rangeCompliantEquivalent): Convert [node, 0] positions to positionBeforeNode(node) for more types of nodes.
  • rendering/RenderContainer.cpp: (WebCore::RenderContainer::positionForCoordinates): Fix a crasher when right clicking on an anonymous table.
7:15 PM Changeset in webkit [15400] by ggaren
  • 4 edits
    2 moves in trunk/JavaScriptCore

RS by Beth.


Finished previously approved JSInternalString -> JSString conversion
by renaming the files.

  • API/JSCallbackObject.cpp:
  • API/JSInternalStringRef.cpp: Removed.
  • API/JSInternalStringRef.h: Removed.
  • API/JSStringRef.cpp: Added.
  • API/JSStringRef.h: Added.
  • API/JavaScriptCore.h:
  • JavaScriptCore.xcodeproj/project.pbxproj:
5:36 PM Changeset in webkit [15399] by sullivan
  • 1 edit
    4 adds in trunk/LayoutTests

Reviewed by Brady Eidson (and written by Justin Garcia)


  • layout test corresponding to <rdar://problem/4611164>
  • editing/undo/undo-misspellings-expected.checksum: Added.
  • editing/undo/undo-misspellings-expected.png: Added.
  • editing/undo/undo-misspellings-expected.txt: Added.
  • editing/undo/undo-misspellings.html: Added.
5:29 PM Changeset in webkit [15398] by sullivan
  • 2 edits in trunk/WebCore

Reviewed by Brady Eidson


  • fixed <rdar://problem/4611164> REGRESSION: Crash occurs when undoing a series of misspelled words (WebCore::RenderObject::repaint(bool)
  • dom/Document.cpp: (WebCore::Document::removeMarkers): put (it - markers.begin()) in a local variable before altering markers, in every case where this was happening. One of the cases like this was fixed a while back, but other cases were either missed at that time or crept in since.
5:20 PM Changeset in webkit [15397] by andersca
  • 3 edits
    4 adds in trunk

WebCore:

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

Reviewed by Justin.

​http://bugzilla.opendarwin.org/show_bug.cgi?id=9663
REGRESSION (r14948-r14956): Selection in text field remains highlighted when the text field loses focus


  • dom/Document.cpp: (WebCore::Document::updateSelection): Don't return early if the selection is empty.

LayoutTests:

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

Reviewed by Justin.

​http://bugzilla.opendarwin.org/show_bug.cgi?id=9663
REGRESSION (r14948-r14956): Selection in text field remains highlighted when the text field loses focus

  • editing/selection/clear-selection-expected.checksum: Added.
  • editing/selection/clear-selection-expected.png: Added.
  • editing/selection/clear-selection-expected.txt: Added.
  • editing/selection/clear-selection.html: Added.
2:47 PM Changeset in webkit [15396] by andersca
  • 2 edits in trunk/WebKit

2006-07-12 Anders Carlsson <​acarlsson@apple.com>

Reviewed by Darin.

​http://bugzilla.opendarwin.org/show_bug.cgi?id=9624
REGRESSION: After ctrl-clicking in a EMPTY input or textarea field, the contextual menu shows "Search in Google" and "Search in Spotlight" as active menu items


  • DefaultDelegates/WebDefaultContextMenuDelegate.m: (-[WebDefaultUIDelegate editingContextMenuItemsForElement:defaultMenuItems:]): Don't create Dictionary, Spotlight or Google lookup items if there's no selection.
10:31 AM Changeset in webkit [15395] by andersca
  • 2 edits
    1 add in trunk/WebCore

2006-07-12 Anders Carlsson <​acarlsson@apple.com>

Reviewed by Adele.

<rdar://problem/4614656> REGRESSION: onpaste() handlers don't run for textarea elements


  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::dispatchCPPEvent): If the element is a shadow node, dispatch the event to its real parent.
  • manual-tests/textarea-onpaste.html: Added.
10:00 AM Changeset in webkit [15394] by bdakin
  • 3 edits in trunk/WebCore

Reviewed by Anders.

Fix for layout test regressions after my check-in last night. Just
a silly mistake where I should have asked if we were NOT printing
instead of if we were in the listbox code.

  • WebCore.xcodeproj/project.pbxproj: Project file wars. Back to XCode 2.3
  • platform/mac/ListBoxMac.mm: (itemTextRenderer): Inverted check. (groupLabelTextRenderer): Inverted check.
9:15 AM Changeset in webkit [15393] by ddkilzer
  • 4 edits in trunk/WebKitTools

WebKitTools:

Reviewed by Darin.

  • Scripts/svn-apply: Added fixChangeLogPatch() and invoked it in the proper place.
  • Scripts/svn-create-patch: Ditto.
  • Scripts/svn-unapply: Ditto.
9:09 AM Changeset in webkit [15392] by ddkilzer
  • 3 edits in trunk/LayoutTests

LayoutTests:

Reviewed by NOBODY (layout test fix).

Remove property information left in files from svn-apply.

  • fast/table/large-rowspan-crash-expected.txt:
  • fast/table/large-rowspan-crash.html:
8:52 AM Changeset in webkit [15391] by darin
  • 1 edit in trunk/LayoutTests/ChangeLog

Fix date.

8:30 AM Changeset in webkit [15390] by darin
  • 3 edits
    2 adds in trunk

LayoutTests:

Reviewed by Maciej.

  • another large rowspan test case
  • fast/table/large-rowspan-crash-expected.txt: Added.
  • fast/table/large-rowspan-crash.html: Added.

WebCore:

Reviewed by Maciej.

Test: fast/table/large-rowspan-crash.html

  • rendering/RenderTableSection.cpp: (WebCore::RenderTableSection::ensureRows): Use sizeof(RowStruct) instead of sizeof(int). Clearly we'll need something better to solve this completely. I expect another smaller, but still huge, value will still cause a problem.
6:32 AM Changeset in webkit [15389] by ddkilzer
  • 2 edits in trunk/WebKitTools

WebKitTools:

Reviewed by ggaren.

Bug 9869: [Drosera] JS Console fails to evaluate input when paused in global scope
​http://bugzilla.opendarwin.org/show_bug.cgi?id=9869

  • Drosera/DebuggerDocument.m: (-[WebScriptObject currentFunctionStack]): Include the global frame in the stack.
4:28 AM Changeset in webkit [15388] by ddkilzer
  • 2 edits in trunk/WebKitTools

WebKitTools:

Reviewed by ggaren.

Bug 9863: Drosera needs to show something at launch
​http://bugzilla.opendarwin.org/show_bug.cgi?id=9863

  • Drosera/DebuggerApplication.m: (-[DebuggerApplication applicationDidFinishLaunching:]): Show the attach window on launch.
3:33 AM Changeset in webkit [15387] by andersca
  • 5 edits
    4 adds in trunk

WebCore:

2006-07-12 Anders Carlsson <​acarlsson@apple.com>

Reviewed by Maciej.

<rdar://problem/4586665> REGRESSION: autorestore.apple.com: Crashes Safari in WebCore::Widget::client() const

  • bridge/mac/FrameMac.h: Add focusCallResultedInViewBeingCreated argument.


  • bridge/mac/FrameMac.mm: (WebCore::FrameMac::nextKeyViewInFrame): If the call to focus() caused the node to get a native widget, set focusCallResultedInViewBeingCreated to true.


(WebCore::FrameMac::nextKeyViewInFrameHierarchy):
Don't reset the focus node if focusCallResultedInViewBeingCreated is true. Also, add magic to prevent setting
a text field as the first responder if its field editor already is the current first responder.


  • page/FrameView.cpp: (WebCore::FrameView::handleMousePressEvent): In some cases, get the event target node again after dispatching the mouse event.

LayoutTests:

2006-07-12 Anders Carlsson <​acarlsson@apple.com>

Reviewed by Maciej.

<rdar://problem/4586665> REGRESSION: autorestore.apple.com: Crashes Safari in WebCore::Widget::client() const

  • fast/forms/input-type-change-in-onfocus-keyboard-expected.txt: Added.
  • fast/forms/input-type-change-in-onfocus-keyboard.html: Added.
  • fast/forms/input-type-change-in-onfocus-mouse-expected.txt: Added.
  • fast/forms/input-type-change-in-onfocus-mouse.html: Added.
3:18 AM Changeset in webkit [15386] by andersca
  • 4 edits in trunk/JavaScriptGlue

2006-07-12 Anders Carlsson <​acarlsson@apple.com>

Build fix


  • JSUtils.cpp: (KJSValueToCFTypeInternal):
  • JSValueWrapper.cpp: (JSValueWrapper::JSObjectCopyPropertyNames):
  • UserObjectImp.cpp: (UserObjectImp::getPropertyList):
3:01 AM Changeset in webkit [15385] by ggaren
  • 16 edits in trunk/JavaScriptCore

Reviewed by Maciej.


  • Removed context and exception parameters from JSObjectGetPropertyEnumerator, removing the spurious use of ExecState inside JavaScriptCore that made us think this was necessary in the first place.

(StringInstance::getPropertyList): Use getString instead of toString because
we know we're dealing with a string -- we put it there in the first place.
While we're at it, store the string's size instead of retrieving it each time
through the loop, to avoid the unnecessary killing of puppies.

  • kjs/string_object.h:
2:55 AM Changeset in webkit [15384] by mjs
  • 7 edits in trunk/JavaScriptCore

4eviewed by Geoff.


  • add handling of hasInstance callback for API objects
  • API/JSCallbackObject.cpp: (KJS::JSCallbackObject::implementsHasInstance): Check if callback is present. (KJS::JSCallbackObject::hasInstance): Invoke appropriate callback.
  • API/JSCallbackObject.h:
  • API/JSClassRef.cpp:
  • API/JSObjectRef.h:
  • API/testapi.c: (MyObject_hasInstance): Test case; should match what construct would do.
  • API/testapi.js:
2:52 AM Changeset in webkit [15383] by bdakin
  • 6 edits in trunk/WebCore

--This Reviewed by Adele.

Fix for <rdar://problem/4615765> Input[type='search' fields
initially render too large in Widgets

Fundamentally, the problem here is that we were miscalculating
things because [NSGraphicsContext currentContextDrawingToScreen]
was evaluating to false when widgets first load. We only ever used
this check, however, to determine if we were printing or not, so it
should not have evaluated to false for loading in Dashbaord.
Instead, if we query the RenderView about whether or not we are
printing, we will get the right answer.

No test case added since this appears only to affect Dashboard.

  • bridge/mac/WebCoreFrameBridge.mm: (-[WebCoreFrameBridge drawRect:]): Ask the RenderView if we are printing instead. This is not part of the actual bug fix, but it seems wise to get rid of all calls to [NSGraphicsContext currentContextDrawingToScreen] when we are just trying to determine if we are printing or not.
  • platform/mac/ListBoxMac.mm: (itemTextRenderer):This function now takes a boolean, isPrinting. (groupLabelTextRenderer): Same as above. (ListBox::sizeForNumberOfLines): Ask the RenderView if we are printing. (-[WebCoreTableView drawRow:clipRect:]): Same as above.
  • platform/mac/PopUpButtonMac.mm: (PopUpButton::sizeHint): Same as above.
  • platform/mac/TextFieldMac.mm: (-[NSSearchFieldCell _addStringToRecentSearches:]):
  • rendering/RenderLineEdit.cpp: Same as above.

M WebCore/platform/mac/TextFieldMac.mm
M WebCore/platform/mac/PopUpButtonMac.mm
M WebCore/platform/mac/ListBoxMac.mm
M WebCore/rendering/RenderLineEdit.cpp
M WebCore/ChangeLog
M WebCore/bridge/mac/WebCoreFrameBridge.mm

2:32 AM Changeset in webkit [15382] by thatcher
  • 2 edits in trunk/WebKit

Reviewed by Timothy.

​http://bugzilla.opendarwin.org/show_bug.cgi?id=9868
Applications shown in Drosera's "Attach" window remain after exit

  • DefaultDelegates/WebScriptDebugServer.m: (-[WebScriptDebugServer init]): Register for NSApplicationWillTerminateNotification so we will know when the application is being exited. (-[WebScriptDebugServer dealloc]): Unregister notification before we are deallocated. (-[WebScriptDebugServer applicationTerminating:]): Inform anyone listening that we are going away.
1:55 AM Changeset in webkit [15381] by thatcher
  • 3 edits in trunk/WebKitTools

Reviewed by Timothy.


Lets Drosera build universal for the nightlies. Right now it is not
possible to build a universal binary on a PPC machine because of a
conflict with the universal SDK.

  • BuildSlaveSupport/build-launcher-app:
  • WebKitLauncher/WebKitLauncher.xcodeproj/project.pbxproj:
1:31 AM Changeset in webkit [15380] by thatcher
  • 3 edits in tags/WebKit-521.16.3/WebKit

Build fix for AGL. Versioning.

1:26 AM Changeset in webkit [15379] by thatcher
  • 1 copy in tags/WebKit-521.16.3

new tag.

1:25 AM Changeset in webkit [15378] by tomernic
  • 3 edits in trunk/WebKit

Reviewed by Tim Hatcher.

<rdar://problem/4624858> AGL isn't 64-bit yet; temporarily remove it from WebKit 64-bit build

Also, fixed a LOG_ERROR() so that it uses the CGL error instead of the AGL error; Tim H missed this in his build fix
from earlier.

  • Plugins/WebBaseNetscapePluginView.h:
  • Plugins/WebBaseNetscapePluginView.m:
1:17 AM Changeset in webkit [15377] by adele
  • 3 edits
    4 adds in trunk

LayoutTests:

Reviewed by Maciej.

Test for: ​http://bugzilla.opendarwin.org/show_bug.cgi?id=9813
OPTION text can paint over select element's scrollbar

  • fast/forms/option-text-clip-expected.checksum: Added.
  • fast/forms/option-text-clip-expected.png: Added.
  • fast/forms/option-text-clip-expected.txt: Added.
  • fast/forms/option-text-clip.html: Added.

WebCore:

Reviewed by Maciej.

Fix for: ​http://bugzilla.opendarwin.org/show_bug.cgi?id=9813
OPTION text can paint over select element's scrollbar

Test: fast/forms/option-text-clip.html

  • rendering/RenderMenuList.cpp: (WebCore::RenderMenuList::paintObject): When calculating the clip to apply to the button text, also consider padding. In the future, we may have separate renderers for the text part, and the arrow part of the control, and then the separation should be natural. For now, we use padding.
1:12 AM Changeset in webkit [15376] by ggaren
  • 23 edits in trunk/JavaScriptCore

Reviewed by Maciej.


  • Implemented a vast number of renames and comment clarifications suggested during API review.


JSInternalString -> JSString
JS*Make -> JSValueMake*, JSObjectMake*
JSTypeCode -> JSType
JSValueIsInstanceOf -> JSValueIsInstanceOfConstructor (reads strangely well in client code)
JSGC*Protect -> JSValue*Protect
JS*Callback -> JSObject*Callback
JSGetPropertyListCallback -> JSObjectAddPropertiesToListCallback
JSPropertyEnumeratorGetNext -> JSPropertyEnumeratorGetNextName
JSString* ->

JSStringCreateWithUTF8CString, JSStringGetUTF8CString,
JSStringGetMaximumUTF8CStringSize JSStringIsEqualToUTF8CString,
JSStringCreateWithCFString, JSStringCopyCFString, JSStringCreateWithCharacters.


  • Changed functions taking a JSValue out arg and returning a bool indicating whether it was set to simply return a JSValue or NULL.


  • Removed JSStringGetCharacters because it's more documentation than code, and it's just a glorified memcpy built on existing API functionality.


  • Moved standard library includes into the headers that actually require them.


  • Standardized use of the phrase "Create Rule."


  • Removed JSLock from make functions that don't allocate.


  • Added exception handling to JSValueToBoolean, since we now allow callback objects to throw exceptions upon converting to boolean.


  • Renamed JSGCCollect to JSGarbageCollect.
Note: See TracTimeline for information about the timeline view.