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

Timeline



Feb 6, 2008:

11:17 PM Changeset in webkit [30062] by justin.garcia@apple.com
  • 11 edits in trunk

WebCore:

Reviewed by Darin Adler.

<rdar://problem/5195056> Huge plain text pastes are slow


This was fixed in r27369 and then r29367 and r29667 caused performance to
regress.

  • editing/EditCommand.cpp: (WebCore::EditCommand::apply): Only updateLayout() for high level commands. (WebCore::EditCommand::unapply): Ditto. (WebCore::EditCommand::reapply): Ditto.
  • editing/Editor.cpp: (WebCore::Editor::appliedEditing): Added a note about shouldChangeSelection calls that shouldn't be made, a bug I filed as <rdar://problem/5729315>. (WebCore::Editor::unappliedEditing): Ditto. (WebCore::Editor::reappliedEditing): Ditto.
  • editing/SelectionController.cpp: (WebCore::SelectionController::nodeWillBeRemoved): Don't try to test the selection base and extent with the expensive isCandidate operation if the node that will be removed is in a fragment, since such a removal is guaranteed to have no effect on a selection. This is to speed up the paste operation, which does many removes from a fragment.

LayoutTests:

Reviewed by Darin Adler.


<rdar://problem/5195056> Huge plain text pastes are slow


The changes made for this fix exposed several more cases of:
<rdar://problem/5729315> Some shouldChangeSelectedDOMRange contain Ranges for selections that are no longer valid

  • platform/mac/editing/deleting/collapse-whitespace-3587601-fix-expected.txt:
  • platform/mac/editing/deleting/delete-3608462-fix-expected.txt:
  • platform/mac/editing/deleting/delete-4083333-fix-expected.txt:
  • platform/mac/editing/execCommand/find-after-replace-expected.txt:
  • platform/mac/editing/selection/move-between-blocks-no-001-expected.txt:
  • platform/mac/editing/selection/replace-selection-1-expected.txt:
10:16 PM Changeset in webkit [30061] by ap@webkit.org
  • 2 edits in trunk/LayoutTests

Landing updated (improved) results for this test. The results changed in r30013,
but this wasn't immediately noticed because of the test being on Leopard skipped list.

Interestingly, js-test-pre.js has a workaround for the issue fixed in this revision in
its debug() function, but not in description(). Keeping the workaround for now to let tests
keep running in shipping Safari/WebKit.

  • svg/css/glyph-orientation-rounding-test-expected.txt:
8:12 PM Changeset in webkit [30060] by kevino@webkit.org
  • 4 edits in trunk/WebCore

Switch from directly handling wx scroll wheel events to handling the PlatformWheelEvent instead to give JS, etc. a chance to handle it.
​http://bugs.webkit.org/show_bug.cgi?id=17179

8:00 PM Changeset in webkit [30059] by mrowe@apple.com
  • 2 edits in trunk/WebCore

2008-02-06 Mark Rowe <​mrowe@apple.com>

Fix Windows builds.

  • WebCore.vcproj/WebCore.vcproj: Unbreak the XML of the project file.
6:40 PM Changeset in webkit [30058] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Mac build fix. Track rename that happened in r30056.

6:00 PM Changeset in webkit [30057] by oliver@apple.com
  • 4 edits in trunk/LayoutTests

Fix expected output for layout tests to match tiger expected (and so work on the build bots)

RS=Stephanie

5:52 PM Changeset in webkit [30056] by mrowe@apple.com
  • 11 edits
    7 copies
    1 move
    6 adds in trunk/WebCore

2008-02-06 Brent Fulgham <​bfulgham@gmail.com>

Reviewed by Adam Roben.

​http://bugs.webkit.org/show_bug.cgi?id=16979
Conditionalize CoreGraphics vs Cairo support in Windows port.

5:36 PM Changeset in webkit [30055] by andersca@apple.com
  • 4 edits in trunk/WebCore

Reviewed by Darin.

Change httpBodyFromStream to take the request instead of the stream.


  • platform/network/cf/FormDataStreamCFNet.cpp: (WebCore::httpBodyFromRequest):
  • platform/network/cf/FormDataStreamCFNet.h:
  • platform/network/cf/ResourceRequestCFNet.cpp: (WebCore::ResourceRequest::doUpdateResourceRequest):
5:24 PM Changeset in webkit [30054] by kmccullough@apple.com
  • 1 edit
    2 adds in trunk/WebCore

Reviewed by Darin.

  • Added manual tests for <rdar://problem/5556374> REGRESSION: cross-domain error when one URL uses an explicit port number and another doesn't
  • manual-tests/Default-port-frame.html: Added.
  • manual-tests/resources/Default-port-frame-contents.html: Added.
2:23 PM Changeset in webkit [30053] by oliver@apple.com
  • 2 edits in trunk/WebCore

Update build flags to fix windows build

2:03 PM Changeset in webkit [30052] by kmccullough@apple.com
  • 5 edits
    7 deletes in trunk

WebCore:

Rubberstamped by Darin.

<rdar://problem/5727708> REGRESSION (r29952): Can't send message from Yahoo Mail beta

  • Reverted the change that caused the regression.
  • dom/Document.cpp: (WebCore::Document::createElement): (WebCore::Document::createElementNS): (WebCore::Document::getElementById): (WebCore::Document::parseQualifiedName): (WebCore::Document::createAttributeNS):
  • dom/Document.idl:

LayoutTests:

Rubberstamped by Darin.

<rdar://problem/5727708> REGRESSION (r29952): Can't send message from Yahoo Mail beta

  • Reverted the change that caused the regression.
  • dom/xhtml/level3/core/documentsetstricterrorchecking02-expected.txt:
  • fast/dom/Document/createAttributeNS-namespace-err-expected.txt: Removed.
  • fast/dom/Document/createAttributeNS-namespace-err.html: Removed.
  • fast/dom/Document/createElementNS-namespace-err-expected.txt: Removed.
  • fast/dom/Document/createElementNS-namespace-err.html: Removed.
  • fast/dom/Document/resources/TEMPLATE.html: Removed.
  • fast/dom/Document/resources/createAttributeNS-namespace-err.js: Removed.
  • fast/dom/Document/resources/createElementNS-namespace-err.js: Removed.
1:46 PM Changeset in webkit [30051] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

Reviewed by Oliver Hunt.


Added an ASSERT to catch refCount underflow, since it caused a leak in
my last check-in.

  • wtf/RefCounted.h: (WTF::RefCounted::deref):
1:24 PM Changeset in webkit [30050] by ggaren@apple.com
  • 4 edits in trunk/WebCore

Reviewed by Oliver Hunt.


Fixed <rdar://problem/5728081> REGRESSION: Many leaks on buildbot

The problem was refCount underflow in NamedAttrMap.


Neither our regression tests nor the stress test have yet discovered
another instance of this problem.

  • dom/NamedAttrMap.cpp: (WebCore::NamedAttrMap::addAttribute): Changed to use PassRefPtr, for fast and correct refCount management. Also, change a rediculously slow malloc to a slightly less rediculously slow realloc.
  • dom/NamedAttrMap.h: (WebCore::NamedAttrMap::insertAttribute): Changed to use PassRefPtr, for fast and correct refCount management.
  • html/HTMLTokenizer.cpp: (WebCore::Token::addAttribute): Use a RefPtr, to guarantee that the object starts with a refCount of 1.
1:21 PM Changeset in webkit [30049] by mrowe@apple.com
  • 1 edit in trunk/WebCore/page/InspectorController.cpp

Fix typo in comment added in r30047.

12:19 PM Changeset in webkit [30048] by timothy@apple.com
  • 1 edit
    1180 adds in trunk

Dump of bugs.webkit.org's Bugzilla instance.

11:29 AM Changeset in webkit [30047] by Darin Adler
  • 2 edits in trunk/WebCore

Reviewed by Tim Hatcher.

  • fix <rdar://problem/5723293> NULL-deref crash in PropertyMap::put opening web inspector with View Source window as target
  • page/InspectorController.cpp: (WebCore::canPassNodeToJavaScript): Added. Returns false if the node is in a document with JavaScript disabled. (WebCore::InspectorController::inspect): Check canPassNodeToJavaScript and do nothing if it returns false.
11:27 AM Changeset in webkit [30046] by hyatt@apple.com
  • 1 delete in trunk/LayoutTests/dom/xhtml/level2/html/object06.xhtml

Remove bad layout test.:

11:26 AM Changeset in webkit [30045] by hyatt@apple.com
  • 1 delete in trunk/LayoutTests/dom/html/level2/html/object06.html

Remove bad layout test.:

11:26 AM Changeset in webkit [30044] by hyatt@apple.com
  • 4 edits in trunk/WebCore

Fix for bug 16799, object elements should return absolute URLs from .data.

Reviewed by Mark Rowe

  • dom/Document.cpp: (WebCore::Document::completeURL):
  • html/HTMLAnchorElement.cpp: (WebCore::HTMLAnchorElement::href):
  • html/HTMLObjectElement.cpp: (WebCore::HTMLObjectElement::data):
10:17 AM Changeset in webkit [30043] by Antti Koivisto
  • 3 edits in trunk/WebCore

Reviewed by Darin.

<rdar://problem/5726340>
<video autoplay controls> left in unplayable state if navigated away, then back to, before video finished loading


When moving document in and out from the page cache:

  • Cancel incomplete load by deleting the media player. This guarantees everything is in consistent state.
  • Restart the load if it was aborted in the middle.
  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::~HTMLMediaElement): (WebCore::HTMLMediaElement::load): (WebCore::HTMLMediaElement::willSaveToCache): (WebCore::HTMLMediaElement::didRestoreFromCache):
  • html/HTMLMediaElement.h: (WebCore::HTMLMediaElement::player):
9:53 AM Changeset in webkit [30042] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

Reviewed by Darin Adler.

PLT speedup related to <rdar://problem/5659272> REGRESSION: PLT .4%
slower due to r28884 (global variable symbol table optimization)


Tweaked RefCounted::deref() to be a little more efficient.

1% - 1.5% speedup on my machine. .7% speedup on Stephanie's machine.


  • wtf/RefCounted.h: (WTF::RefCounted::deref): Don't modify m_refCount if we're just going to delete the object anyway. Also, use a simple == test, which might be faster than <= on some hardware.
9:40 AM Changeset in webkit [30041] by Darin Adler
  • 4 edits
    3 adds in trunk

JavaScriptCore:

Reviewed by Sam.

Test results match Gecko with very few obscure exceptions that seem to be
bugs in Gecko.

Test: fast/js/array-functions-non-arrays.html

  • kjs/array_object.cpp: (KJS::arrayProtoFuncConcat): Removed DontEnum and DontDelete from the call to set length. (KJS::arrayProtoFuncPop): Ditto. Also added missing call to deleteProperty, which is not needed for real arrays, but is needed for non-arrays. (KJS::arrayProtoFuncPush): Ditto. (KJS::arrayProtoFuncShift): Ditto. (KJS::arrayProtoFuncSlice): Ditto. (KJS::arrayProtoFuncSort): Removed incorrect call to set length when the array has no elements. (KJS::arrayProtoFuncSplice): Removed DontEnum and DontDelete from the call to set length. (KJS::arrayProtoFuncUnShift): Ditto. Also added a check for 0 arguments to make behavior match the specification in that case.
  • kjs/nodes.cpp: (KJS::ArrayNode::evaluate): Removed DontEnum and DontDelete from the call to set length.

LayoutTests:

Reviewed by Sam.

  • fast/js/array-functions-non-arrays-expected.txt: Added.
  • fast/js/array-functions-non-arrays.html: Added.
  • fast/js/resources/array-functions-non-arrays.js: Added.
9:33 AM Changeset in webkit [30040] by Darin Adler
  • 17 edits in trunk

JavaScriptCore:

Reviewed by Sam.

  • replace calls to put to set up properties with calls to putDirect, to prepare for a future change where put won't take attributes any more, and for a slight performance boost
  • API/JSObjectRef.cpp: (JSObjectMakeConstructor): Use putDirect instead of put.
  • kjs/CommonIdentifiers.h: Removed lastIndex.
  • kjs/JSGlobalObject.cpp: (KJS::JSGlobalObject::reset): Use putDirect instead of put.
  • kjs/array_object.cpp: (KJS::arrayProtoFuncConcat): Took out extra call to get length (unused). (KJS::ArrayObjectImp::ArrayObjectImp): Use putDirect instead of put.
  • kjs/error_object.cpp: (KJS::ErrorPrototype::ErrorPrototype): Use putDirect instead of put.
  • kjs/function.cpp: (KJS::Arguments::Arguments): Use putDirect instead of put. (KJS::PrototypeFunction::PrototypeFunction): Use putDirect instead of put.
  • kjs/function_object.cpp: (KJS::FunctionObjectImp::construct): Use putDirect instead of put.
  • kjs/nodes.cpp: (KJS::FuncDeclNode::makeFunction): Use putDirect instead of put. (KJS::FuncExprNode::evaluate): Use putDirect instead of put.
  • kjs/regexp_object.cpp: (KJS::regExpProtoFuncCompile): Use setLastIndex instead of put(lastIndex). (KJS::RegExpImp::match): Get and set lastIndex by using m_lastIndex instead of calling get and put.
  • kjs/regexp_object.h: (KJS::RegExpImp::setLastIndex): Added.
  • kjs/string_object.cpp: (KJS::stringProtoFuncMatch): Use setLastIndex instead of put(lastIndex).

WebCore:

Reviewed by Sam.

  • replace calls to put to set up properties with calls to putDirect, to prepare for a future change where put won't take attributes any more, and for a slight performance boost
  • bindings/js/JSAudioConstructor.cpp: (WebCore::JSAudioConstructor::JSAudioConstructor): Use putDirect instead of put.
  • bindings/js/JSEventTargetBase.h: (WebCore::JSEventTargetPrototype::self): Ditto.
  • bindings/js/JSHTMLOptionElementConstructor.cpp: (WebCore::JSHTMLOptionElementConstructor::JSHTMLOptionElementConstructor): Ditto.
  • bindings/js/JSSQLResultSetRowListCustom.cpp: (WebCore::JSSQLResultSetRowList::item): Ditto.
8:01 AM Changeset in webkit [30039] by mitz@apple.com
  • 3 edits
    4 adds in trunk

WebCore:

Reviewed by Darin Adler.

Test: fast/css/transition-color-unspecified.html

  • page/AnimationController.cpp: (WebCore::ImplicitAnimation::animate): Changed to use the value of the 'color' property in the source or destination style as the source or destination value of properties whose inital value is defined to be the computed value of 'color'.

LayoutTests:

Reviewed by Darin Adler.

  • fast/css/transition-color-unspecified.html: Added.
  • platform/mac/fast/css/transition-color-unspecified-expected.checksum: Added.
  • platform/mac/fast/css/transition-color-unspecified-expected.png: Added.
  • platform/mac/fast/css/transition-color-unspecified-expected.txt: Added.
7:46 AM Changeset in webkit [30038] by mitz@apple.com
  • 2 edits in trunk/WebCore

Build fix.

  • svg/svgtags.in:

Feb 5, 2008:

11:08 PM Changeset in webkit [30037] by oliver@apple.com
  • 2 edits in trunk/WebCore

Build fix -- touch generate-bindings to trigger regeneration of bindings

10:18 PM Changeset in webkit [30036] by mitz@apple.com
  • 224 edits in trunk/LayoutTests/platform/mac/svg/custom

Added the svn:mime-type property to PNGs that did not have it.

10:01 PM Changeset in webkit [30035] by oliver@apple.com
  • 66 edits in trunk

RS=Eric.

Re-enable foreignObject by default as it is needed for a number of
non-fO related SVG tests and none of the old known crashes occur
anymore.

8:36 PM Changeset in webkit [30034] by weinig@apple.com
  • 7 edits
    4 adds in trunk

JavaScriptCore:

Reviewed by Anders Carlsson.

Fix for ​http://bugs.webkit.org/show_bug.cgi?id=8080
NodeList (and other DOM lists) items are not enumeratable using for..in

WebCore:

Reviewed by Anders Carlsson.

Fix for ​http://bugs.webkit.org/show_bug.cgi?id=8080
NodeList (and other DOM lists) items are not enumeratable using for..in

  • Match Firefox when enumerating DOM interfaces with indexGetters (support for the array bracket, nodeList[0], notation) by including all the items in the list before the attributes and methods of the interface.

Test: fast/dom/domListEnumeration.html

  • ForwardingHeaders/kjs/PropertyNameArray.h: Added.
  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::customGetPropertyNames): Use the new custom method model.
  • bindings/js/JSHistoryCustom.cpp: (WebCore::JSHistory::customGetPropertyNames): ditto.
  • bindings/scripts/CodeGeneratorJS.pm: Instead of just adding a declaration of getProperyNames and implementing the method in the Custom.cpp, move to a the model used by generated getOwnPropertySlot() and put() where the custom code is written in a separate customGetPropertyNames which returns a bool indicating whether to call up to the base class. This enables adding the list indexes to the PropertyNameArray for interfaces with indexGetters automatically.

LayoutTests:

Reviewed by Anders Carlsson.

Test for ​http://bugs.webkit.org/show_bug.cgi?id=8080
NodeList (and other DOM lists) items are not enumeratable using for..in

  • fast/dom/domListEnumeration-expected.txt: Added.
  • fast/dom/domListEnumeration.html: Added.
  • fast/dom/resources/domListEnumeration.js: Added.
6:13 PM Changeset in webkit [30033] by Nikolas Zimmermann
  • 1 edit
    10 adds in trunk/LayoutTests

Reviewed by Oliver.
Add new dynamic update tests, for SVGClipPathElement.

5:29 PM Changeset in webkit [30032] by weinig@apple.com
  • 9 edits
    3 adds in trunk

WebCore:

Reviewed by Darin Adler.

Fix for <rdar://problem/5726604>
dom-checker: deleting properties of the window object cross-domain should not be allowed

Fix flaw found while testing with dom-checker testing tool (​http://code.google.com/p/dom-checker/).

Test: http/tests/security/cross-frame-access-delete.html

  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::deleteProperty): Override deleteProperty to not delete cross-domain.
  • bindings/js/JSHistoryCustom.cpp: (WebCore::JSHistory::deleteProperty): ditto.
  • bindings/js/JSLocation.cpp: (WebCore::JSLocation::deleteProperty): ditto.
  • bindings/js/JSLocation.h:
  • bindings/scripts/CodeGeneratorJS.pm: Add deleteProperty declaration when CustomDeleteProperty is used.
  • page/DOMWindow.idl: Add CustomDeleteProperty extended attribute.
  • page/History.idl: ditto.

LayoutTests:

Reviewed by Darin Adler.

Test for <rdar://problem/5726604>
dom-checker: deleting properties of the window object cross-domain should not be allowed

  • http/tests/security/cross-frame-access-delete-expected.txt: Added.
  • http/tests/security/cross-frame-access-delete.html: Added.
  • http/tests/security/resources/cross-frame-iframe-for-delete-test.html: Added.
5:24 PM Changeset in webkit [30031] by mrowe@apple.com
  • 4 edits in trunk/WebKit/gtk

2008-02-05 Mark Rowe <​mrowe@apple.com>

Reviewed by Alp Toker.

Fix warnings seen on the Gtk port by declaring variables and using constants of the correct types.

  • webkit/webkitwebbackforwardlist.cpp: (_WebKitWebBackForwardListPrivate::webkit_web_back_forward_list_contains_item):
  • webkit/webkitwebhistoryitem.cpp:
  • webkit/webkitwebview.cpp:
5:16 PM Changeset in webkit [30030] by sfalken@apple.com
  • 2 edits in trunk/WebKitTools

Build fix. Don't override intermediate directory.

  • Drosera/win/Drosera.vcproj/Drosera.vcproj:
4:57 PM Changeset in webkit [30029] by alp@webkit.org
  • 2 edits in trunk/WebCore

2008-02-05 Alp Toker <​alp@atoker.com>

Rubber-stamped by Mark Rowe.

Fix a "missing sentinel in function call" warning by using NULL
instead of 0.

  • platform/gtk/RenderThemeGtk.cpp: (WebCore::RenderThemeGtk::caretBlinkFrequency):
3:09 PM Changeset in webkit [30028] by Nikolas Zimmermann
  • 5 edits
    30 adds in trunk

Reviewed by Oliver.
Fix dynamic updates of <circle> element's properties.

Added tests: svg/dynamic-updates/SVGCircleElement*

2:31 PM Changeset in webkit [30027] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Wx build fix. Add WebCore/plugins to the include path.

2:28 PM Changeset in webkit [30026] by zecke@webkit.org
  • 3 edits in trunk/WebKitTools

Build fix for Windows. willCloseFrame is a pure virtual and we are
forced to have an implementation.

This partially reverts r30014.

2:24 PM Changeset in webkit [30025] by mrowe@apple.com
  • 4 edits in tags/Safari-5525.7.2

Versioning.

2:23 PM Changeset in webkit [30024] by mrowe@apple.com
  • 4 edits in trunk/WebCore

2008-02-05 Mark Rowe <​mrowe@apple.com>

Mac build fix. Update the Xcode project to follow a moved file,
and track a rename in two files that were missed earlier.

  • WebCore.xcodeproj/project.pbxproj:
  • page/mac/WebCoreFrameBridge.mm: (-[WebCoreFrameBridge canProvideDocumentSource]):
  • platform/mac/PlugInInfoStoreMac.mm: (WebCore::PluginInfoStore::pluginNameForMIMEType):
2:22 PM Changeset in webkit [30023] by mrowe@apple.com
  • 2 edits in tags/Safari-5525.7.2/WebKit/mac

Merge r30020.

1:57 PM Changeset in webkit [30022] by Nikolas Zimmermann
  • 1 edit in trunk/WebCore/ChangeLog

Forgot to mention bug

1:56 PM Changeset in webkit [30021] by Nikolas Zimmermann
  • 6 edits
    25 adds in trunk

Reviewed by Holger.

Fix dynamic SVG DOM updates of <a> element's href property.

This commit also introduces a new SVG Tests framework: LayoutTests/svg/dynamic-updates
Its purpose is to provide dynamic updating tests for each SVG class & property, aka.
one test per property per class. As a first pass it's sufficient to add testcases for all
SVG*Element classes, and their properties - and only common-used base-class properties.

For SVGAElement - for example - it makes most sense to test scripting its parent
SVGURIReference object ('href' property) and its own property 'target'.

Adding tests to see - for example - if SVGAElement reacts on changes on ie. its parent
SVGExternalResourcesRequired interface, should really be done in a second pass.

All tests within the new framework are supposed to be created using make-js-tests-wrappers.

1:54 PM Changeset in webkit [30020] by mrowe@apple.com
  • 2 edits in trunk/WebKit/mac

Unreviewed build fix. Add missing #import.

1:53 PM Changeset in webkit [30019] by mrowe@apple.com
  • 1 copy in tags/Safari-5525.7.2

New tag.

1:53 PM Changeset in webkit [30018] by mrowe@apple.com
  • 1 delete in tags/Safari-5525.72

Remove bogus tag.

1:52 PM Changeset in webkit [30017] by mrowe@apple.com
  • 1 copy in tags/Safari-5525.72

New tag.

1:51 PM Changeset in webkit [30016] by justin.garcia@apple.com
  • 4 edits
    6 adds in trunk

WebCore:

Reviewed by Alexey Proskuryakov.

<rdar://problem/5685601> webkit-block-placeholder class on placeholders seems unnecessary (12317)

  • editing/CompositeEditCommand.cpp: (WebCore::createBlockPlaceholderElement): The khtml-block-placeholder class was needed when the editing code actively looked for <br>s of that type in order to handle them specially. That is no longer the case.
  • editing/InsertParagraphSeparatorCommand.cpp: (WebCore::InsertParagraphSeparatorCommand::doApply): Fixed a bug where an extra block would be inserted when hitting return inside an editable root held open by a placeholder <br> or '\n'. Added test cases for the bug and for the special case code that was incorrectly triggered to cause the bug.

LayoutTests:

Reviewed by Alexey Proskuryakov.


<rdar://problem/5685601> webkit-block-placeholder class on placeholders seems unnecessary (12317)

  • editing/inserting/5685601-1-expected.txt: Added.
  • editing/inserting/5685601-1.html: Added.
  • editing/inserting/5685601-2-expected.txt: Added.
  • editing/inserting/5685601-2.html: Added.
  • editing/inserting/5685601-3-expected.txt: Added.
  • editing/inserting/5685601-3.html: Added.
1:46 PM Changeset in webkit [30015] by mrowe@apple.com
  • 20 edits
    6 moves in trunk

2008-02-05 Rodney Dawes <​dobey@wayofthemonkey.com>

Reviewed by Anders Carlsson.

Part one of ​http://bugs.webkit.org/show_bug.cgi?id=16924.
Shared PluginDatabase, PluginInfoStore and PluginPackage implementations.

Remove the Win suffix on several plugin-related classes that will
soon be refactored to be more portable.

12:47 PM Changeset in webkit [30014] by zecke@webkit.org
  • 14 edits in trunk

WebKitTools/
2008-02-05 Holger Freyther <​zecke@selfish.org>

Reviewed by Darin.

In ​http://bugs.webkit.org/show_bug.cgi?id=16853 it was identified that the
the output of the willCloseFrame and didClearWindowObject FrameLoadDelegate
depends on the order the tests are executed.

Propose to remove willCloseFrame and didlCearWindowObject output from the
FrameLoadDelegate and update the test results. The output of didClearWindowObject
is not interesting for any of the current tests and we have other ways to find
out if a frame was closed or not.

  • DumpRenderTree/mac/FrameLoadDelegate.mm:
  • DumpRenderTree/win/FrameLoadDelegate.cpp: (FrameLoadDelegate::didClearWindowObject):
  • DumpRenderTree/win/FrameLoadDelegate.h:

LayoutTest:
2008-02-05 Holger Freyther <​zecke@selfish.org>

Reviewed by Darin.

In ​http://bugs.webkit.org/show_bug.cgi?id=16853 it was identified that the
the output of the willCloseFrame and didClearWindowObject FrameLoadDelegate
depends on the order the tests are executed.

Propose to remove willCloseFrame and didlCearWindowObject output from the
FrameLoadDelegate and update the test results. The output of didClearWindowObject
is not interesting for any of the current tests and we have other ways to find
out if a frame was closed or not.

  • http/tests/loading/bad-scheme-subframe-expected.txt:
  • http/tests/loading/bad-server-subframe-expected.txt:
  • http/tests/loading/basic-expected.txt:
  • http/tests/loading/empty-subframe-expected.txt:
  • http/tests/loading/gmail-assert-on-load-expected.txt:
  • http/tests/loading/onload-vs-immediate-refresh-expected.txt:
  • http/tests/loading/slow-parsing-subframe-expected.txt:
  • platform/mac/http/tests/loading/simple-subframe-expected.txt:
  • webarchive/loading/test-loading-archive-expected.txt:
11:52 AM Changeset in webkit [30013] by ap@webkit.org
  • 3 edits
    2 adds in trunk

Reviewed by Darin.

​http://bugs.webkit.org/show_bug.cgi?id=17158
Setting innerHTML in a detached XHTML element doesn't use the right namespace

Test: fast/dom/innerHTML-detached-element.xhtml

  • dom/XMLTokenizer.cpp: (WebCore::XMLTokenizer::XMLTokenizer): Take parent element namespace into account.
11:45 AM Changeset in webkit [30012] by mrowe@apple.com
  • 4 edits in tags/Safari-5525.7.1

Versioning.

11:44 AM Changeset in webkit [30011] by mrowe@apple.com
  • 8 edits in tags/Safari-5525.7.1

Merge r30003.

11:41 AM Changeset in webkit [30010] by mrowe@apple.com
  • 2 edits in tags/Safari-5525.7.1/WebKit/mac

Merge r29869.

11:32 AM Changeset in webkit [30009] by Antti Koivisto
  • 7 edits
    6 adds in trunk

WebCore:

Reviewed by Geoff.


Fix <rdar://problem/5698200>
eBay photo uploading hangs and causes slow script warning to pop up


In a case like this


var f = window.parent.parentFunction;
document.domain = document.domain; this makes window.parent inaccessible
f();


Firefox allows parentFunction to access parents properties. Match this behavior.


In a domain security check against the dynamic global object fails for the specific reason that one of
the frames has written to the document.domain property and another has not (but they match otherwise),
then recheck against the lexical global object.

Test: http/tests/security/cross-frame-access-callback-explicit-domain-ALLOW.html

http/tests/security/cross-frame-access-callback-explicit-domain-DENY.html

  • bindings/js/kjs_window.cpp: (KJS::Window::allowsAccessFrom): (KJS::Window::printErrorMessage):
  • bindings/js/kjs_window.h:
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::shouldAllowNavigation):
  • platform/SecurityOrigin.cpp: (WebCore::SecurityOrigin::canAccess):
  • platform/SecurityOrigin.h: (WebCore::SecurityOrigin::):

LayoutTests:

Reviewed by Geoff.


Test for <rdar://problem/5698200>
eBay photo uploading hangs and causes slow script warning to pop up

  • http/tests/security/cross-frame-access-callback-explicit-domain-ALLOW-expected.txt: Added.
  • http/tests/security/cross-frame-access-callback-explicit-domain-ALLOW.html: Added.
  • http/tests/security/cross-frame-access-callback-explicit-domain-DENY-expected.txt: Added.
  • http/tests/security/cross-frame-access-callback-explicit-domain-DENY.html: Added.
  • http/tests/security/resources/cross-frame-iframe-callback-explicit-domain-ALLOW.html: Added.
  • http/tests/security/resources/cross-frame-iframe-callback-explicit-domain-DENY.html: Added.
11:24 AM Changeset in webkit [30008] by mrowe@apple.com
  • 1 copy in tags/Safari-5525.7.1

New tag.

11:18 AM Changeset in webkit [30007] by Adam Roben
  • 3 edits in trunk/WebCore

Remove an unused member from Document

Reviewed by Mitz.

  • dom/Document.cpp: (WebCore::Document::recalcStyleSelector): There's no need to take m_printSheet into account anymore as it's never anything but the null string.
  • dom/Document.h: Removed m_printSheet and methods relating to it.
10:36 AM Changeset in webkit [30006] by Darin Adler
  • 2 edits in trunk/WebKit/win

Reviewed by Darin.

​http://bugs.webkit.org/show_bug.cgi?id=15248
<rdar://problem/5497032> Can not enter accented characters using alt-numeric keypad (take two)

  • WebView.cpp: (WebViewWndProc): KeyUp messages need to be translated, too.
9:31 AM Changeset in webkit [30005] by mitz@apple.com
  • 7 edits in trunk

WebCore:

Reviewed by Darin Adler.

  • WebCore part of <rdar://problem/5724303> Should implement writing direction shortcuts
  • WebCore.base.exp: Added Editor::setBaseWritingDirection() and Frame::baseWritingDirectionForSelectionStart().
  • page/mac/WebCoreFrameBridge.h: Removed -baseWritingDirectionForSelectionStart.
  • page/mac/WebCoreFrameBridge.mm: Ditto.

WebKit/mac:

Reviewed by Darin Adler.

  • WebKit part of <rdar://problem/5724303> Should implement writing direction shortcuts

The key bindings are Command-Control-left arrow and
Command-Control-right arrow. To match AppKit, the bindings are enabled
only when certain user defaults are set.

  • WebView/WebHTMLView.mm: (-[WebHTMLView validateUserInterfaceItemWithoutDelegate:]): (-[WebHTMLView toggleBaseWritingDirection:]): Changed to call Frame::baseWritingDirectionForSelectionStart() and Editor::setBaseWritingDirection() directly. (-[WebHTMLView changeBaseWritingDirection:]): Ditto. (writingDirectionKeyBindingsEnabled): Added. (-[WebHTMLView _changeBaseWritingDirectionTo:]): Added this helper method. (-[WebHTMLView changeBaseWritingDirectionToLTR:]): Added. (-[WebHTMLView changeBaseWritingDirectionToRTL:]): Added.
  • WebView/WebView.mm:
9:01 AM Changeset in webkit [30004] by mrowe@apple.com
  • 6 edits in trunk/WebKitSite

Update WordPress to 2.3.3 which includes security fixes.

7:52 AM Changeset in webkit [30003] by mrowe@apple.com
  • 8 edits in trunk

Update versioning to support the mysterious future.

Reviewed by Oliver Hunt.

  • Configurations/Version.xcconfig: Add SYSTEM_VERSION_PREFIX_1060.
2:42 AM Changeset in webkit [30002] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Fix the wxWidget Mac build by avoiding using ICU functions that were added after ICU 3.2.

  • editing/SmartReplaceICU.cpp:

(WebCore::addAllCodePoints): Implement a replacement for uset_addAllCodePoints.
(WebCore::getSmartSet): Use addAllCodePoints instead of uset_addAllCodePoints.

1:56 AM Applications using WebKit edited by ap@webkit.org
Added Microsoft products. (diff)
1:34 AM Changeset in webkit [30001] by ap@webkit.org
  • 9 edits
    10 deletes in trunk

Pretend that r29998 never happened.

12:20 AM Changeset in webkit [30000] by mrowe@apple.com
  • 1 edit in trunk/WebCore/ChangeLog

30,000!

12:19 AM Changeset in webkit [29999] by mrowe@apple.com
  • 2 edits in trunk/WebKitTools

Workaround for ​http://bugs.webkit.org/show_bug.cgi?id=16842
Bug 16842: Hang in DRT in leaks mode due to signal handler doing unsafe things

Reviewed by Oliver Hunt.

Don't install the signal handler unless we are running the pixel tests.
It is currently only used to restore the color profile, but by catching
signals we can trigger a deadlock in DRT while running leak tests.
The deadlock is tracked by ​http://bugs.webkit.org/show_bug.cgi?id=16842,
and a more complete fix will need to be developed to address this for
pixel tests as well.

  • DumpRenderTree/mac/DumpRenderTree.mm:

(dumpRenderTree):

12:16 AM Changeset in webkit [29998] by mrowe@apple.com
  • 9 edits
    10 adds in trunk

2008-02-04 Matt Perry <​mpComplete@gmail.com>

Reviewed by Darin Adler.

Fix for ​http://bugs.webkit.org/show_bug.cgi?id=14959
No back forward entry added for pages created in javascript

A new HistoryItem is created for calls to Document::open. Calls to
Document::write save the written data to a SharedBuffer that is also
stored on the HistoryItem. When the user navigates back to a
HistoryItem that has a valid buffer, that data is used for the page
content.

Tests: http/tests/navigation/document-open-adds-history-item.html

http/tests/navigation/document-open-delayed-adds-history-item.html
http/tests/navigation/document-open-new-window-adds-history-item.html
http/tests/navigation/document-open-replace-no-history-item.html

  • bindings/js/JSHTMLDocumentCustom.cpp: (WebCore::JSHTMLDocument::open):
  • dom/Document.cpp: (WebCore::Document::open): (WebCore::Document::write): (WebCore::Document::clear):
  • dom/Document.h:
  • history/HistoryItem.cpp: (WebCore::HistoryItem::HistoryItem): (WebCore::HistoryItem::substituteData): (WebCore::HistoryItem::setSubstituteData):
  • history/HistoryItem.h:
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::didExplicitOpen): (WebCore::FrameLoader::load): (WebCore::FrameLoader::reloadAllowingStaleData): (WebCore::FrameLoader::reload): (WebCore::FrameLoader::shouldTreatURLAsSameAsCurrent): (WebCore::FrameLoader::loadItem):
  • loader/FrameLoader.h:

2008-02-04 Matt Perry <​mpComplete@gmail.com>

Reviewed by Darin Adler.

Test cases for fix to ​http://bugs.webkit.org/show_bug.cgi?id=14959
No back forward entry added for pages created in javascript.

  • http/tests/navigation/document-open-adds-history-item-expected.txt: Added.
  • http/tests/navigation/document-open-adds-history-item.html: Added.
  • http/tests/navigation/document-open-delayed-adds-history-item-expected.txt: Added.
  • http/tests/navigation/document-open-delayed-adds-history-item.html: Added.
  • http/tests/navigation/document-open-new-window-adds-history-item-expected.txt: Added.
  • http/tests/navigation/document-open-new-window-adds-history-item.html: Added.
  • http/tests/navigation/document-open-replace-no-history-item-expected.txt: Added.
  • http/tests/navigation/document-open-replace-no-history-item.html: Added.
Note: See TracTimeline for information about the timeline view.