Timeline
Apr 6, 2009:
- 4:43 PM Changeset in webkit [42261] by
-
- 5 edits in trunk/WebCore
2009-04-06 Sam Weinig <sam@webkit.org>
Reviewed by Mark Rowe.
Bottleneck dispatching mutation events in a new dispatchMutationEvent
function.
- dom/CharacterData.cpp: (WebCore::CharacterData::dispatchModifiedEvent):
- dom/ContainerNode.cpp: (WebCore::dispatchChildInsertionEvents): (WebCore::dispatchChildRemovalEvents):
- dom/Node.cpp: (WebCore::Node::dispatchSubtreeModifiedEvent): (WebCore::Node::dispatchMutationEvent):
- dom/Node.h:
- 4:11 PM Changeset in webkit [42260] by
-
- 9 edits in trunk/WebCore
2009-04-06 Dimitri Glazkov <Dimitri Glazkov>
Reviewed by Darin Fisher.
https://bugs.webkit.org/show_bug.cgi?id=25059
Remove references to dominantScript, getGenericFontForScript.
- platform/graphics/chromium/FontCacheChromiumWin.cpp: Removed references
to getGenericFontForScript.
(WebCore::FontCache::getLastResortFallbackFont): Removed calls to dominantScript.
- platform/graphics/chromium/FontCacheLinux.cpp: Ditto.
- rendering/RenderThemeChromiumLinux.cpp: (WebCore::defaultGUIFont): Removed Document* arg. (WebCore::RenderThemeChromiumLinux::systemFont): Ditto.
- rendering/RenderThemeChromiumLinux.h: Ditto.
- rendering/RenderThemeChromiumMac.h: Ditto.
- rendering/RenderThemeChromiumMac.mm: Ditto. (WebCore::RenderThemeChromiumMac::systemFont): Ditto.
- rendering/RenderThemeChromiumWin.cpp: (WebCore::defaultGUIFont): Ditto. (WebCore::RenderThemeChromiumWin::systemFont): Ditto.
- rendering/RenderThemeChromiumWin.h: Ditto.
- 4:04 PM Changeset in webkit [42259] by
-
- 4 edits in trunk
Versioning.
- 4:02 PM Changeset in webkit [42258] by
-
- 1 copy in tags/Safari-6530.5
New tag.
- 3:42 PM Changeset in webkit [42257] by
-
- 1 edit1 add in trunk/WebKit/gtk/po
2009-04-06 Christian Dywan <christian@twotoasts.de>
Rubber-stamped by Oliver Hunt.
- de.po: Added German translation for WebKitGTK+.
- 2:56 PM Changeset in webkit [42256] by
-
- 4 edits in trunk/WebCore
2009-04-06 Geoffrey Garen <ggaren@apple.com>
Reviewed by Darin Adler.
Fixed https://bugs.webkit.org/show_bug.cgi?id=21260
Unbounded memory growth when churning elements with anonymous event handler functions
The problem was that a protected event listener's scope chain would end
up with a reference to the Node protecting it, causing a cycle. The
solution is to stop protecting event listeners and rely on marking instead.
This patch converts most Node event listeners to use marking instead of
GC protection.
- bindings/js/JSDOMBinding.cpp: (WebCore::isObservableThroughDOM): (WebCore::markDOMNodesForDocument): Refactored the test for whether a Node needs marking, and added a case for marking the node in order to mark its event listeners.
- bindings/js/JSNodeCustom.cpp: (WebCore::markEventListeners): (WebCore::JSNode::addEventListener): (WebCore::JSNode::removeEventListener): (WebCore::JSNode::mark): Added code to mark a Node's event listeners when marking the Node. Changed code using protected event listeners to use event listeners.
- dom/Node.idl: Changed code using protected event listeners to use event listeners.
- 1:58 PM Changeset in webkit [42255] by
-
- 3 edits in trunk/WebCore
2009-04-06 Sam Weinig <sam@webkit.org>
Reviewed by Darin Adler.
Remove unchecked bool return values for some dispatch event functions.
- dom/Node.cpp: (WebCore::Node::dispatchSubtreeModifiedEvent): (WebCore::Node::dispatchUIEvent): (WebCore::Node::dispatchWebKitAnimationEvent): (WebCore::Node::dispatchWebKitTransitionEvent): (WebCore::Node::dispatchProgressEvent):
- dom/Node.h:
- 1:48 PM Changeset in webkit [42254] by
-
- 3 edits in trunk/WebCore
2009-04-06 Darin Fisher <darin@chromium.org>
Reviewed by Dimitri Glazkov.
https://bugs.webkit.org/show_bug.cgi?id=25058
Fix chromium build bustage caused by r42216
- bindings/scripts/CodeGeneratorJS.pm:
- page/DOMWindow.idl:
- 12:48 PM Changeset in webkit [42253] by
-
- 2 edits in trunk/LayoutTests
2009-04-06 Dimitri Glazkov <Dimitri Glazkov>
Not reviewed because it's stupidly obvious.
Correct the expected result by adding an extra line break.
- fast/dom/HTMLSelectElement/length-not-overridden-expected.txt: Added
an extra line.
- 12:39 PM Changeset in webkit [42252] by
-
- 1 edit1 add in trunk/WebKit/gtk/po
2009-04-06 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>
Rubber-stamped by Sam Weinig.
- pt_BR.po: Added Brazilian Portuguese translation for WebKitGTK+.
- 12:39 PM Changeset in webkit [42251] by
-
- 12 edits6 adds in trunk
2009-04-06 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>
Rubber-stamped by Sam Weinig.
Added rules to maintain the localization support. We cannot simply
use whatever gettextize gives us because our build system is
non-recursive.
- GNUmakefile.am:
- configure.ac:
WebCore/
2009-04-06 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>
Rubber-stamped by Sam Weinig.
https://bugs.webkit.org/show_bug.cgi?id=20575
No localization in WebKitGtk
- platform/gtk/FileChooserGtk.cpp: Remove usage of glib/gi18n.h, since there are not translatable strings.
- platform/gtk/LocalizedStringsGtk.cpp: Use the glib i18n header that is specific for libraries.
WebKit/gtk/
2009-04-06 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>
Rubber-stamped by Sam Weinig.
https://bugs.webkit.org/show_bug.cgi?id=20575
No localization in WebKitGtk
Implemented localization support for WebKitGTK+.
- WebCoreSupport/ChromeClientGtk.cpp:
- WebCoreSupport/ContextMenuClientGtk.cpp:
- po/POTFILES: Added.
- po/README: Added.
- po/webkit.pot: Added.
- webkit/webkitdownload.cpp: (_WebKitDownloadPrivate::webkit_download_class_init):
- webkit/webkitprivate.cpp: (webkit_init):
- webkit/webkitwebframe.cpp:
WebKit/gtk/po/
2009-04-06 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>
Rubber-stamped by Sam Weinig.
- GNUmakefile.am, POTFILES, webkit.pot: Initial setup of the localization infra-structure.
- 11:41 AM Changeset in webkit [42250] by
-
- 2 edits in trunk/WebCore
Revert r42249 to fix the build.
- 11:23 AM Changeset in webkit [42249] by
-
- 2 edits in trunk/WebCore
2009-04-06 Darin Fisher <darin@chromium.org>
Reviewed by Dimitri Glazkov.
https://bugs.webkit.org/show_bug.cgi?id=25058
Fix chromium build bustage caused by r42216
- page/DOMWindow.idl:
- 11:08 AM Changeset in webkit [42248] by
-
- 3 edits in trunk/WebCore
2009-04-06 Mike Belshe <mike@belshe.com>
Reviewed by Dimitri Glazkov.
https://bugs.webkit.org/show_bug.cgi?id=24978
Use custom implementations of a few Node methods so that
we don't over-aggressively create JS wrappers here.
- bindings/v8/custom/V8NodeCustom.cpp: Added custom implementations.
- dom/Node.idl: Removed JSC-specifier, making impl custom for V8 as well.
- 11:01 AM Changeset in webkit [42247] by
-
- 3 edits2 adds in trunk
WebCore:
2009-04-06 Dave Moore <davemoore@google.com>
Reviewed by Dimitri Glazkov.
https://bugs.webkit.org/show_bug.cgi?id=25031
Make the V8 element collections check for named properties *before*
checking if there are any elements with the same name. Otherwise
it is both incorrect and slow.
Test: fast/dom/HTMLSelectElement/length-not-overridden.html
Both of these interceptors were attempting to find an element in the
collection that had a name or id of the property name before checking
for a JS property with that name.
- bindings/v8/V8Collection.h: (WebCore::collectionNamedPropertyGetter): (WebCore::nodeCollectionNamedPropertyGetter):
LayoutTests:
2009-04-06 Dave Moore <davemoore@google.com>
Reviewed by Dimitri Glazkov.
https://bugs.webkit.org/show_bug.cgi?id=25031
Test for ensuring that named elements don't override properties on collections
- fast/dom/HTMLSelectElement/length-not-overridden-expected.txt: Added.
- fast/dom/HTMLSelectElement/length-not-overridden.html: Added.
- 10:51 AM Changeset in webkit [42246] by
-
- 5 edits4 adds in trunk
2009-04-06 Simon Fraser <Simon Fraser>
Reviewed by Darin Adler
Pull the call to updateGraphicsLayerGeometry() out of updateGraphicsLayers(),
because we need to call it at the end of rebuildCompositingLayerTree() once
we've determined which descendant layers are composited, otherwise
calculateCompositedBounds() can give the wrong answer.
Now that updateLayerCompositingState() doesn't end up calling updateGraphicsLayerGeometry(),
call that explicitly from styleChanged(), if we know a layer update is not pending,
and similarly from updateAfterLayout().
Test: compositing/overflow/overflow-positioning.html
- rendering/RenderLayer.cpp: (WebCore::RenderLayer::styleChanged):
- rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::updateAfterLayout): (WebCore::RenderLayerBacking::updateGraphicsLayers): (WebCore::RenderLayerBacking::updateGraphicsLayerGeometry):
- rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::rebuildCompositingLayerTree):
- 9:01 AM Changeset in webkit [42245] by
-
- 2 edits in trunk/WebCore
2009-04-06 Mike Belshe <mike@belshe.com>
Reviewed by Alexey Proskuryakov.
Add accessors to AtomicString which take String/const char* arguments
so that we don't accidentally thrash the AtomicString table.
- platform/text/AtomicString.h:
- 8:47 AM Changeset in webkit [42244] by
-
- 7 edits in trunk/WebKit/qt
2009-04-06 Tor Arne Vestbø <tor.arne.vestbo@nokia.com>
Reviewed by Simon Hausmann.
[Qt] Rename DOM API select function to findFirst, findAll, etc.
- 8:37 AM Changeset in webkit [42243] by
-
- 2 edits in trunk/WebKitTools
2009-04-06 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>
Rubber-stamped by Tor Arne Vestbø.
Add /Programs to the productDir for GTK+, instead of adding
/JavaScriptCore, when running jsc.
- Scripts/run-sunspider:
- 7:17 AM Changeset in webkit [42242] by
-
- 2 edits in trunk/WebCore
[GTK+] Correct the column and row cursor
Make col and row work in WebCore/manual-tests/cursor.html. The issue
was spotted in the WebKit inspector.
- 5:59 AM Changeset in webkit [42241] by
-
- 2 edits in trunk/WebCore
2009-04-06 Tor Arne Vestbø <tor.arne.vestbo@nokia.com>
Reviewed by Simon Hausmann.
[Qt] Don't show and hide the platformPluginWidget, as it's our QWebView
- 5:50 AM QtWebKitContrib edited by
- (diff)
- 5:50 AM QtWebKitContrib edited by
- (diff)
- 5:49 AM QtWebKitContrib edited by
- (diff)
- 4:58 AM Changeset in webkit [42240] by
-
- 2 edits in trunk/WebKit/gtk
2009-04-05 Xan Lopez <xlopez@igalia.com>
Reviewed by Holger Freyther.
https://bugs.webkit.org/show_bug.cgi?id=25042
destroying webview widget directly causes crash.
Do not connect the WebView to its own screen-changed signal, set a
default handler instead.
- webkit/webkitwebview.cpp:
- 4:57 AM Changeset in webkit [42239] by
-
- 2 edits in trunk/WebKit/gtk
2009-04-05 Xan Lopez <xlopez@igalia.com>
Reviewed by Holger Freyther.
https://bugs.webkit.org/show_bug.cgi?id=25042
destroying webview widget directly causes crash.
GtkWidget::screen-changed can be emitted when a widget is being
destroyed, protect against it.
- webkit/webkitwebview.cpp:
- 4:25 AM Changeset in webkit [42238] by
-
- 9 edits5 adds in trunk
WebCore:
2009-04-06 Simon Hausmann <simon.hausmann@nokia.com>
Reviewed by Tor Arne Vestbø.
Add new files to the Qt build.
Export helper function from the Qt JSC binding, needed in WebKit/qt.
WebKit/qt:
2009-04-06 Simon Hausmann <simon.hausmann@nokia.com>
Ariya Hidayat <ariya.hidayat@nokia.com>
Tor Arne Vestbø <tor.arne.vestbo@nokia.com>
Genevieve Mak <gen@staikos.net>
Reviewed by Tor Arne Vestbø, Simon Hausmann
First revision of new API in the Qt port to access the DOM.
- 4:13 AM Changeset in webkit [42237] by
-
- 3 edits in trunk/WebCore
2009-04-06 Simon Hausmann <simon.hausmann@nokia.com>
Reviewed by Tor Arne Vestbø.
Changed ScriptFunctionCall's last argument from a const char* as the function name to a String.
Internally the function name is stored as a WebCore::String and the Qt port wants to use this
constructor programmatically where the function name is also a WebCore::String.
- 2:51 AM Changeset in webkit [42236] by
-
- 2 edits in trunk/WebCore
2009-04-06 Mike Belshe <mike@belshe.com>
Reviewed by Eric Seidel.
HTMLCanvasElement crash when ImageBuffer creation fails.
https://bugs.webkit.org/show_bug.cgi?id=23212
Check for NULL before using the ImageBuffer as we might
be low on memory and creation may have failed.
Test case creation blocked by:
https://bugs.webkit.org/show_bug.cgi?id=25055
- html/HTMLCanvasElement.cpp: (WebCore::HTMLCanvasElement::createImageBuffer):
Apr 5, 2009:
- 3:48 PM Changeset in webkit [42235] by
-
- 13 edits in trunk/WebCore
2009-04-05 Geoffrey Garen <ggaren@apple.com>
[Originally] Reviewed by Sam Weinig.
Rolled back in r42214 with crashes fixed.
[Removed irrelevant ExecState* parameter from event handler registration code.]
[No change in behavior.]
- bindings/js/JSDOMApplicationCacheCustom.cpp: (WebCore::JSDOMApplicationCache::addEventListener): (WebCore::JSDOMApplicationCache::removeEventListener):
- bindings/js/JSDOMGlobalObject.cpp: (WebCore::JSDOMGlobalObject::findOrCreateJSProtectedEventListener): (WebCore::JSDOMGlobalObject::findJSEventListener): (WebCore::JSDOMGlobalObject::findOrCreateJSEventListener):
- bindings/js/JSDOMGlobalObject.h:
- bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::addEventListener):
- bindings/js/JSMessagePortCustom.cpp: (WebCore::JSMessagePort::addEventListener): (WebCore::JSMessagePort::removeEventListener):
- bindings/js/JSNodeCustom.cpp: (WebCore::JSNode::addEventListener):
- bindings/js/JSSVGElementInstanceCustom.cpp: (WebCore::JSSVGElementInstance::addEventListener):
- bindings/js/JSWorkerContextCustom.cpp: (WebCore::JSWorkerContext::addEventListener): (WebCore::JSWorkerContext::removeEventListener):
- bindings/js/JSWorkerCustom.cpp: (WebCore::JSWorker::addEventListener): (WebCore::JSWorker::removeEventListener):
- bindings/js/JSXMLHttpRequestCustom.cpp: (WebCore::JSXMLHttpRequest::addEventListener): (WebCore::JSXMLHttpRequest::removeEventListener):
- bindings/js/JSXMLHttpRequestUploadCustom.cpp: (WebCore::JSXMLHttpRequestUpload::addEventListener): (WebCore::JSXMLHttpRequestUpload::removeEventListener):
- bindings/scripts/CodeGeneratorJS.pm:
- 10:26 AM Changeset in webkit [42234] by
-
- 2 edits in trunk/WebCore
2009-04-05 Erik L. Bunce <elbunce@xendom.com>
Reviewed by Simon Hausmann.
Fix an assert failure when dropping an 'empty' text/uri-list on a QWebView.
- 7:56 AM Changeset in webkit [42233] by
-
- 2 edits in trunk/WebCore
Fix the Qt build
- 1:21 AM Changeset in webkit [42232] by
-
- 2 edits in trunk
2009-04-05 Mike Hommey <glandium@debian.org>
Reviewed by Holger Freyther.
Filter out all C++ symbols
https://bugs.webkit.org/show_bug.cgi?id=24960
Considering the public API is all C, we can just filter out all
C++ mangled symbols, which will avoid exporting symbols in some
corner cases such as gcc bugs on specific architectures, etc.
- autotools/symbols.filter:
- 1:16 AM Changeset in webkit [42231] by
-
- 2 edits in trunk/WebKit/gtk
2009-04-05 Jan Michael Alonzo <jmalonzo@webkit.org>
Reviewed by Holger Freyther.
[Gtk] Implement LayoutTestControllerGtk::setPrivateBrowsingEnabled
https://bugs.webkit.org/show_bug.cgi?id=24487
Updated documentation and put more information about the
private-browsing setting for WebKitGtk.
- webkit/webkitwebsettings.cpp: (_WebKitWebSettingsPrivate::webkit_web_settings_class_init):
Apr 4, 2009:
- 8:56 PM Changeset in webkit [42230] by
-
- 9 edits in trunk
Build fixes for wxMac/Tiger.
- 5:06 PM Changeset in webkit [42229] by
-
- 8 edits in trunk
wx build fixes for the Windows port.
- 1:25 PM Changeset in webkit [42228] by
-
- 2 edits in trunk/WebCore
2009-04-04 Simon Fraser <Simon Fraser>
Reviewed by Cameron Zwarich
Ensure that mapPoint(const IntPoint&) calls the FloatPoint
version to avoid infinite recursion.
No test because mapPoint(const IntPoint&) isn't ever called
on Mac, so I can't reproduce.
- platform/graphics/transforms/TransformationMatrix.h: (WebCore::TransformationMatrix::mapPoint):
- 12:33 AM Changeset in webkit [42227] by
-
- 3 edits2 adds in trunk
WebCore:
Reviewed by Mark Rowe.
- fix <rdar://problem/6757862> REGRESSION: Cannot use the mouse to place the insertion point inside a Thai character
Test: fast/text/international/thai-offsetForPosition-inside-character.html
- platform/graphics/mac/CoreTextController.cpp: (WebCore::CoreTextController::offsetForPosition): Changed to use cursorMovementIterator instead of characterBreakIterator, allowing this function to return a valid cursor position even if it is not at a character boundary.
LayoutTests:
Reviewed by Mark Rowe.
- test for <rdar://problem/6757862> REGRESSION: Cannot use the mouse to place the insertion point inside a Thai character
- fast/text/international/thai-offsetForPosition-inside-character-expected.txt: Added.
- fast/text/international/thai-offsetForPosition-inside-character.html: Added.
- 12:27 AM Changeset in webkit [42226] by
-
- 3 edits2 adds in trunk
WebCore:
Reviewed by Mark Rowe.
- fix an assertion failure when clicking at the beginning of a glyph cluster at the beginning of a text run
Test: fast/text/offsetForPosition-cluster-at-zero.html
- platform/graphics/mac/CoreTextController.cpp: (WebCore::CoreTextController::offsetForPosition): Corrected the bound check on firstGlyphBeforeCluster, letting it be 0 or -1, but not less than -1.
LayoutTests:
Reviewed by Mark Rowe.
- test for a bug where computing the text offset for a position at the beginning of a glyph cluster at the beginning of a Core Text run resulted in an attempt to read past the end of a vector
- fast/text/offsetForPosition-cluster-at-zero-expected.txt: Added.
- fast/text/offsetForPosition-cluster-at-zero.html: Added.
- 12:20 AM Changeset in webkit [42225] by
-
- 2 edits in trunk/LayoutTests
2009-04-04 Sam Weinig <sam@webkit.org>
Update test result.
- dom/xhtml/level2/html/HTMLFormElement10-expected.txt:
Apr 3, 2009:
- 6:06 PM Changeset in webkit [42224] by
-
- 2 edits in trunk/WebCore
Allows 16 HTML/CSS standard colors to be shown as swatch backgrounds
when the names are given.
Reviewed by Timothy Hatcher.
- inspector/front-end/StylesSidebarPane.js: (WebInspector.StylePropertyTreeElement.prototype.updateTitle):
- 5:56 PM Changeset in webkit [42223] by
-
- 8 edits4 adds in trunk
WebCore:
2009-04-03 Sam Weinig <sam@webkit.org>
Reviewed by Darin Adler.
<rdar://problem/6330929>
https://bugs.webkit.org/show_bug.cgi?id=21456
Don't update the document pointer for all inactive windows on navigations.
This change causes us to differ slightly from Firefox when accessing the
document from within a closure tied to a navigated context, but as all
browsers differ on this edge case, I don't foresee compatibility issues.
Test: http/tests/security/cross-frame-access-document-direct.html
- bindings/js/JSDOMWindowBase.cpp: (WebCore::JSDOMWindowBase::~JSDOMWindowBase):
- bindings/js/ScriptController.cpp: (WebCore::ScriptController::clearWindowShell): (WebCore::ScriptController::initScript): (WebCore::ScriptController::updateDocument):
- bindings/js/ScriptController.h:
LayoutTests:
2009-04-03 Sam Weinig <sam@webkit.org>
Reviewed by Darin Adler.
Update tests for <rdar://problem/6330929>
https://bugs.webkit.org/show_bug.cgi?id=21456
- dom/xhtml/level2/html/HTMLFormElement10-expected.txt:
- fast/dom/Window/dom-access-from-closure-iframe-expected.txt:
- fast/dom/Window/dom-access-from-closure-window-expected.txt:
- http/tests/security/cross-frame-access-document-direct-expected.txt: Added.
- http/tests/security/cross-frame-access-document-direct.html: Added.
- http/tests/security/resources/cross-frame-iframe-for-document-direct-test-victim.html: Added.
- http/tests/security/resources/cross-frame-iframe-for-document-direct-test.html: Added.
- 5:49 PM Changeset in webkit [42222] by
-
- 4 edits2 adds in trunk
We were starting transitions with 0 duration and delay, which is
useless. With accelerated compositing on, this went through a code
path that tried to use m_toStyle, which never got set because the
transitions was short circuited before it had a chance to. So I
both protect against using that null pointer and avoid starting
the transition in the first place.
- 5:41 PM Changeset in webkit [42221] by
-
- 2 edits in trunk/WebCore
Fixes a bug where you could not type any more in the Web Inspector's
search field after the first match is selected in the Resources panel.
https://bugs.webkit.org/show_bug.cgi?id=23982
rdar://problem/6592367
Reviewed by Darin Adler.
- inspector/front-end/SourceView.js: (WebInspector.SourceView.prototype._jumpToSearchResult): Use the window object of the frame to set the selection, not the main window.
- 5:37 PM Changeset in webkit [42220] by
-
- 2 edits in trunk/WebKit/mac
2009-04-03 Kevin Decker <kdecker@apple.com>
Reviewed by Anders Carlsson.
<rdar://problem/6756512>
- Plugins/Hosted/NetscapePluginHostManager.mm: (WebKit::NetscapePluginHostManager::spawnPluginHost): Tweaked the visibleName property.
- 5:34 PM Changeset in webkit [42219] by
-
- 2 edits in trunk/WebKitTools
2009-04-03 Stephanie Lewis <Stephanie Lewis>
Reviewed by Mark Rowe.
Fix running DumpRenderTree with a root.
- Scripts/run-webkit-tests:
- 4:49 PM Changeset in webkit [42218] by
-
- 7 edits40 adds in trunk
WebCore:
2009-04-03 Sam Weinig <sam@webkit.org>
Reviewed by Anders Carlsson.
Fix for https://bugs.webkit.org/show_bug.cgi?id=22081
<rdar://problem/6346030>
Don't allow overriding the return value of window.location.
Tests: fast/dom/Window/Location/location-override-toString-on-proto-using-defineGetter.html
fast/dom/Window/Location/location-override-toString-on-proto-using-with.html
fast/dom/Window/Location/location-override-toString-on-proto.html
fast/dom/Window/Location/location-override-toString-using-defineGetter.html
fast/dom/Window/Location/location-override-toString-using-with.html
fast/dom/Window/Location/location-override-toString.html
fast/dom/Window/Location/location-override-valueOf-on-proto-using-defineGetter.html
fast/dom/Window/Location/location-override-valueOf-on-proto-using-with.html
fast/dom/Window/Location/location-override-valueOf-on-proto.html
fast/dom/Window/Location/location-override-valueOf-using-defineGetter.html
fast/dom/Window/Location/location-override-valueOf-using-with.html
fast/dom/Window/Location/location-override-valueOf.html
fast/dom/Window/Location/window-override-location-using-defineGetter.html
fast/dom/Window/Location/window-override-window-using-defineGetter.html
fast/dom/Window/Location/window-shadow-location-using-js-object-with-toString.html
fast/dom/Window/Location/window-shadow-location-using-string.html
fast/dom/Window/Location/window-shadow-window-using-js-object-with-location-field.html
- bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::defineGetter):
- bindings/js/JSLocationCustom.cpp: (WebCore::JSLocation::customPut): (WebCore::JSLocation::defineGetter): (WebCore::JSLocationPrototype::customPut): (WebCore::JSLocationPrototype::defineGetter):
- bindings/scripts/CodeGeneratorJS.pm:
- page/Location.idl:
LayoutTests:
2009-04-03 Sam Weinig <sam@webkit.org>
Reviewed by Anders Carlsson.
Tests for https://bugs.webkit.org/show_bug.cgi?id=22081
<rdar://problem/6346030>
Don't allow overriding the return value of window.location.
- fast/dom/Window/Location: Added.
- fast/dom/Window/Location/location-override-toString-expected.txt: Added.
- fast/dom/Window/Location/location-override-toString-on-proto-expected.txt: Added.
- fast/dom/Window/Location/location-override-toString-on-proto-using-defineGetter-expected.txt: Added.
- fast/dom/Window/Location/location-override-toString-on-proto-using-defineGetter.html: Added.
- fast/dom/Window/Location/location-override-toString-on-proto-using-with-expected.txt: Added.
- fast/dom/Window/Location/location-override-toString-on-proto-using-with.html: Added.
- fast/dom/Window/Location/location-override-toString-on-proto.html: Added.
- fast/dom/Window/Location/location-override-toString-using-defineGetter-expected.txt: Added.
- fast/dom/Window/Location/location-override-toString-using-defineGetter.html: Added.
- fast/dom/Window/Location/location-override-toString-using-with-expected.txt: Added.
- fast/dom/Window/Location/location-override-toString-using-with.html: Added.
- fast/dom/Window/Location/location-override-toString.html: Added.
- fast/dom/Window/Location/location-override-valueOf-expected.txt: Added.
- fast/dom/Window/Location/location-override-valueOf-on-proto-expected.txt: Added.
- fast/dom/Window/Location/location-override-valueOf-on-proto-using-defineGetter-expected.txt: Added.
- fast/dom/Window/Location/location-override-valueOf-on-proto-using-defineGetter.html: Added.
- fast/dom/Window/Location/location-override-valueOf-on-proto-using-with-expected.txt: Added.
- fast/dom/Window/Location/location-override-valueOf-on-proto-using-with.html: Added.
- fast/dom/Window/Location/location-override-valueOf-on-proto.html: Added.
- fast/dom/Window/Location/location-override-valueOf-using-defineGetter-expected.txt: Added.
- fast/dom/Window/Location/location-override-valueOf-using-defineGetter.html: Added.
- fast/dom/Window/Location/location-override-valueOf-using-with-expected.txt: Added.
- fast/dom/Window/Location/location-override-valueOf-using-with.html: Added.
- fast/dom/Window/Location/location-override-valueOf.html: Added.
- fast/dom/Window/Location/resources: Added.
- fast/dom/Window/Location/resources/location-tests-functions.js: Added. (normalizeURL):
- fast/dom/Window/Location/resources/pass.html: Added.
- fast/dom/Window/Location/resources/window-shadow-location-using-js-object-with-toString-iframe.html: Added.
- fast/dom/Window/Location/resources/window-shadow-location-using-string-iframe.html: Added.
- fast/dom/Window/Location/window-override-location-using-defineGetter-expected.txt: Added.
- fast/dom/Window/Location/window-override-location-using-defineGetter.html: Added.
- fast/dom/Window/Location/window-override-window-using-defineGetter-expected.txt: Added.
- fast/dom/Window/Location/window-override-window-using-defineGetter.html: Added.
- fast/dom/Window/Location/window-shadow-location-using-js-object-with-toString-expected.txt: Added.
- fast/dom/Window/Location/window-shadow-location-using-js-object-with-toString.html: Added.
- fast/dom/Window/Location/window-shadow-location-using-string-expected.txt: Added.
- fast/dom/Window/Location/window-shadow-location-using-string.html: Added.
- fast/dom/Window/Location/window-shadow-window-using-js-object-with-location-field-expected.txt: Added.
- fast/dom/Window/Location/window-shadow-window-using-js-object-with-location-field.html: Added.
- http/tests/security/cross-frame-access-location-put-expected.txt:
- 4:19 PM Changeset in webkit [42217] by
-
- 13 edits in trunk/WebCore
2009-04-03 Geoffrey Garen <ggaren@apple.com>
Rolled out r42214 since it seems to have caused crashes on the buildbot.
- bindings/js/JSDOMApplicationCacheCustom.cpp: (WebCore::JSDOMApplicationCache::addEventListener): (WebCore::JSDOMApplicationCache::removeEventListener):
- bindings/js/JSDOMGlobalObject.cpp: (WebCore::JSDOMGlobalObject::findOrCreateJSProtectedEventListener): (WebCore::JSDOMGlobalObject::findJSEventListener): (WebCore::JSDOMGlobalObject::findOrCreateJSEventListener):
- bindings/js/JSDOMGlobalObject.h:
- bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::addEventListener):
- bindings/js/JSMessagePortCustom.cpp: (WebCore::JSMessagePort::addEventListener): (WebCore::JSMessagePort::removeEventListener):
- bindings/js/JSNodeCustom.cpp: (WebCore::JSNode::addEventListener): (WebCore::JSNode::removeEventListener):
- bindings/js/JSSVGElementInstanceCustom.cpp: (WebCore::JSSVGElementInstance::addEventListener):
- bindings/js/JSWorkerContextCustom.cpp: (WebCore::JSWorkerContext::addEventListener): (WebCore::JSWorkerContext::removeEventListener):
- bindings/js/JSWorkerCustom.cpp: (WebCore::JSWorker::addEventListener): (WebCore::JSWorker::removeEventListener):
- bindings/js/JSXMLHttpRequestCustom.cpp: (WebCore::JSXMLHttpRequest::addEventListener): (WebCore::JSXMLHttpRequest::removeEventListener):
- bindings/js/JSXMLHttpRequestUploadCustom.cpp: (WebCore::JSXMLHttpRequestUpload::addEventListener): (WebCore::JSXMLHttpRequestUpload::removeEventListener):
- bindings/scripts/CodeGeneratorJS.pm:
- 4:13 PM Changeset in webkit [42216] by
-
- 23 edits6 adds in trunk
WebCore:
2009-04-03 Sam Weinig <sam@webkit.org>
Reviewed by Oliver Hunt.
Fix for <rdar://problem/6476356>
https://bugs.webkit.org/show_bug.cgi?id=23148
- Use the window object the Location and History objects are directly associated with instead of the lexical global object to pick the object prototype to serve as the base of the their respective prototype chains.
- Re-factor as necessary to allow passing the correct global object to the createPrototype functions.
Tests: http/tests/security/cross-frame-access-history-prototype.html
http/tests/security/cross-frame-access-location-prototype.html
- bindings/js/JSAudioConstructor.cpp: (WebCore::JSAudioConstructor::JSAudioConstructor):
- bindings/js/JSDOMBinding.cpp: (WebCore::getCachedDOMStructure): (WebCore::cacheDOMStructure):
- bindings/js/JSDOMBinding.h: (WebCore::getDOMStructure): (WebCore::getDOMPrototype):
- bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::history): (WebCore::JSDOMWindow::location):
- bindings/js/JSDocumentCustom.cpp: (WebCore::JSDocument::location):
- bindings/js/JSImageConstructor.cpp: (WebCore::JSImageConstructor::JSImageConstructor):
- bindings/js/JSMessageChannelConstructor.cpp: (WebCore::JSMessageChannelConstructor::JSMessageChannelConstructor):
- bindings/js/JSNamedNodesCollection.h: (WebCore::JSNamedNodesCollection::createPrototype):
- bindings/js/JSOptionConstructor.cpp: (WebCore::JSOptionConstructor::JSOptionConstructor):
- bindings/js/JSRGBColor.h: (WebCore::JSRGBColor::createPrototype):
- bindings/js/JSWebKitCSSMatrixConstructor.cpp: (WebCore::JSWebKitCSSMatrixConstructor::JSWebKitCSSMatrixConstructor):
- bindings/js/JSWebKitPointConstructor.cpp: (WebCore::JSWebKitPointConstructor::JSWebKitPointConstructor):
- bindings/js/JSWorkerConstructor.cpp: (WebCore::JSWorkerConstructor::JSWorkerConstructor):
- bindings/js/JSXMLHttpRequestConstructor.cpp: (WebCore::JSXMLHttpRequestConstructor::JSXMLHttpRequestConstructor):
- bindings/js/JSXSLTProcessorConstructor.cpp: (WebCore::JSXSLTProcessorConstructor::JSXSLTProcessorConstructor):
- bindings/scripts/CodeGeneratorJS.pm:
- bridge/objc/objc_runtime.h: (JSC::Bindings::ObjcFallbackObjectImp::createPrototype):
- bridge/runtime_array.h: (JSC::RuntimeArray::createPrototype):
- bridge/runtime_method.h: (JSC::RuntimeMethod::createPrototype):
- bridge/runtime_object.h: (JSC::RuntimeObjectImp::createPrototype):
- page/DOMWindow.idl:
LayoutTests:
2009-04-03 Sam Weinig <sam@webkit.org>
Reviewed by Oliver Hunt.
Tests for <rdar://problem/6476356>
https://bugs.webkit.org/show_bug.cgi?id=23148
Test for using the correct global object for location and history object
prototype chain creation.
- http/tests/security/cross-frame-access-history-prototype-expected.txt: Added.
- http/tests/security/cross-frame-access-history-prototype.html: Added.
- http/tests/security/cross-frame-access-location-prototype-expected.txt: Added.
- http/tests/security/cross-frame-access-location-prototype.html: Added.
- http/tests/security/resources/cross-frame-history-prototype-iframe.html: Added.
- http/tests/security/resources/cross-frame-location-prototype-iframe.html: Added.
- 3:24 PM Changeset in webkit [42215] by
-
- 3 edits in trunk/WebCore
2009-04-03 Jian Li <jianli@chromium.org>
Reviewed by Darin Fisher.
https://bugs.webkit.org/show_bug.cgi?id=25009
Upstream changes to WorkerContextExecutionProxy for V8 bindings in order to use V8EventListenerList as container.
- bindings/v8/WorkerContextExecutionProxy.cpp: (WebCore::WorkerContextExecutionProxy::WorkerContextExecutionProxy): (WebCore::WorkerContextExecutionProxy::dispose): (WebCore::WorkerContextExecutionProxy::FindOrCreateEventListener): (WebCore::WorkerContextExecutionProxy::RemoveEventListener):
- bindings/v8/WorkerContextExecutionProxy.h:
- 3:09 PM Changeset in webkit [42214] by
-
- 13 edits in trunk/WebCore
2009-04-03 Geoffrey Garen <ggaren@apple.com>
Reviewed by Sam Weinig.
Removed irrelevant ExecState* parameter from event handler registration code.
No change in behavior.
- bindings/js/JSDOMApplicationCacheCustom.cpp: (WebCore::JSDOMApplicationCache::addEventListener): (WebCore::JSDOMApplicationCache::removeEventListener):
- bindings/js/JSDOMGlobalObject.cpp: (WebCore::JSDOMGlobalObject::findOrCreateJSProtectedEventListener): (WebCore::JSDOMGlobalObject::findJSEventListener): (WebCore::JSDOMGlobalObject::findOrCreateJSEventListener):
- bindings/js/JSDOMGlobalObject.h:
- bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::addEventListener):
- bindings/js/JSMessagePortCustom.cpp: (WebCore::JSMessagePort::addEventListener): (WebCore::JSMessagePort::removeEventListener):
- bindings/js/JSNodeCustom.cpp: (WebCore::JSNode::addEventListener): (WebCore::JSNode::removeEventListener):
- bindings/js/JSSVGElementInstanceCustom.cpp: (WebCore::JSSVGElementInstance::addEventListener):
- bindings/js/JSWorkerContextCustom.cpp: (WebCore::JSWorkerContext::addEventListener): (WebCore::JSWorkerContext::removeEventListener):
- bindings/js/JSWorkerCustom.cpp: (WebCore::JSWorker::addEventListener): (WebCore::JSWorker::removeEventListener):
- bindings/js/JSXMLHttpRequestCustom.cpp: (WebCore::JSXMLHttpRequest::addEventListener): (WebCore::JSXMLHttpRequest::removeEventListener):
- bindings/js/JSXMLHttpRequestUploadCustom.cpp: (WebCore::JSXMLHttpRequestUpload::addEventListener): (WebCore::JSXMLHttpRequestUpload::removeEventListener):
- bindings/scripts/CodeGeneratorJS.pm:
- 3:00 PM Changeset in webkit [42213] by
-
- 2 edits in trunk/WebKit/mac
Reviewed by Anders Carlsson.
<rdar://problem/6756512>
- Plugins/Hosted/NetscapePluginHostManager.mm: (WebKit::NetscapePluginHostManager::spawnPluginHost): Tweaked the visibleName property.
- 2:21 PM Changeset in webkit [42212] by
-
- 4 edits in trunk/LayoutTests
Skip tests that rely on layoutTestController.waitForPolicyDelegate on platforms that don't implement that functionality
Implementing is covered by the following bugs:
<https://bugs.webkit.org/show_bug.cgi?id=25036>
<https://bugs.webkit.org/show_bug.cgi?id=25037>
<https://bugs.webkit.org/show_bug.cgi?id=25038>
Rubber-stamped by Mark Rowe.
- platform/gtk/Skipped:
- platform/qt/Skipped:
- platform/win/Skipped: Skip tests that use layoutTestController.waitForPolicyDelegate.
- 2:09 PM Changeset in webkit [42211] by
-
- 3 edits2 adds in trunk
WebCore:
Reviewed by Simon Fraser.
- fix <rdar://problem/6634771> Under full-page zoom, cannot scroll MobileMe contact list
Test: fast/css/zoom-in-length-round-trip.html
- dom/Element.cpp: (WebCore::adjustForLocalZoom): For zoom factors greater than 1, ensure that the returned value is equal to the original unscaled value. (WebCore::adjustForAbsoluteZoom): Ditto.
LayoutTests:
Reviewed by Simon Fraser.
- test for <rdar://problem/6634771> Under full-page zoom, cannot scroll MobileMe contact list
- fast/css/zoom-in-length-round-trip-expected.txt: Added.
- fast/css/zoom-in-length-round-trip.html: Added.
- 1:44 PM Changeset in webkit [42210] by
-
- 2 edits in trunk/WebKit/mac
2009-04-03 John Sullivan <sullivan@apple.com>
fixed <rdar://problem/6355573> [WebView _setCacheModel:] leaks the result of _CFURLCacheCopyCacheDirectory
Reviewed by Adam Roben
- WebView/WebView.mm: (+[WebView _setCacheModel:]): use WebCFAutorelease rather than autorelease on result of method that returns CFStringRef
- 1:35 PM Changeset in webkit [42209] by
-
- 2 edits in trunk/LayoutTests
2009-04-03 Simon Fraser <Simon Fraser>
Update expected result due to rounding difference after r42207.
- platform/mac/svg/hixie/perf/007-expected.txt:
- 12:29 PM Changeset in webkit [42208] by
-
- 9 edits in trunk
This fix essentially does a -viewWillDraw call for layout. It adds
a CFRunLoopObserver which performs layout just before drawing on the
Mac platform. This makes sure layout is complete before rendering and
avoids a flash.
- 11:57 AM Changeset in webkit [42207] by
-
- 3 edits in trunk/WebCore
2009-04-02 Simon Fraser <Simon Fraser>
Reviewed by Dan Bernstein
Optimize mapPoint(), mapRect() and mapQuad() for identity matrix and translations.
- platform/graphics/transforms/TransformationMatrix.cpp: (WebCore::TransformationMatrix::mapPoint): (WebCore::TransformationMatrix::mapRect): (WebCore::TransformationMatrix::mapQuad):
- platform/graphics/transforms/TransformationMatrix.h: (WebCore::TransformationMatrix::mapPoint):
- 11:39 AM Changeset in webkit [42206] by
-
- 3 edits in trunk/WebCore
2009-04-02 Simon Fraser <Simon Fraser>
Reviewed by Dan Bernstein
Add containsPoint() and containsQuad() methods to FloatQuad.
- platform/graphics/FloatQuad.cpp: (WebCore::dot): (WebCore::isPointInTriangle): (WebCore::FloatQuad::containsPoint): (WebCore::FloatQuad::containsQuad):
- platform/graphics/FloatQuad.h:
- 11:28 AM Changeset in webkit [42205] by
-
- 3 edits in trunk/WebKit/mac
2009-04-03 Anders Carlsson <andersca@apple.com>
Reviewed by Kevin Decker.
WebKit side of <rdar://problem/6752953>.
Pass the clip rect to the plug-in host.
- Plugins/Hosted/NetscapePluginInstanceProxy.mm: (WebKit::NetscapePluginInstanceProxy::resize):
- Plugins/Hosted/WebKitPluginHost.defs:
- 11:16 AM Changeset in webkit [42204] by
-
- 2 edits in trunk/WebCore
Reviewed by Darin.
- loader/ProgressTracker.cpp: Included DocumentLoader.h (WebCore::ProgressTracker::incrementProgress): Limit the 50%-clamp rule to documents that use WebCore's layout system.
- 11:06 AM Changeset in webkit [42203] by
-
- 5 edits3 adds in trunk
2009-04-03 Eric Carlson <eric.carlson@apple.com>
Reviewed by Simon Fraser.
<rdar://problem/6336092> REGRESSION: movie does not resize correctly
Test: media/video-size-intrinsic-scale.html
- platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::naturalSize): Use QTMovieCurrentSizeAttribute instead of QTMovieNaturalSizeAttribute so we return the correct size for movies saved with a non-default matrix. (WebCore::MediaPlayerPrivate::setSize): Don't resize the QTMovieView because [QTMovieView setFrame] also resizes the movie itself, and as the renderer calls setSize immediately when a movie reports a size change (QTMovieSizeDidChangeNotification) we can get into a feedback loop observing the size change and resetting the size, and this can cause QuickTime to miss correctly setting a movie's size when the media size changes after opening (as happens with an rtsp movie once the server sends the track sizes). Instead, remember the size passed to paint() and resize the view when it changes. (WebCore::MediaPlayerPrivate::paint): Resize the view when passed a new size.
- platform/graphics/win/QTMovieWin.cpp: (QTMovieWin::getNaturalSize): Use GetMovieBox instead of GetMovieNaturalBoundsRect so we return the correct size for movies saved with non-default matrix.
2009-04-03 Eric Carlson <eric.carlson@apple.com>
Reviewed by Simon Fraser.
<rdar://problem/6336092> REGRESSION: movie does not resize correctly
- media/content/scaled-matrix.mov: Added. QuickTime movie saved with 2x scale.
- media/video-size-intrinsic-scale-expected.txt: Added.
- media/video-size-intrinsic-scale.html: Added, test that element with a scaled movie is sized correctly.
- media/video-test.js: Modify so it can be included in <head> element. (logConsole): New, wraps creating console <div>, doesn't assume document.body exists. (consoleWrite): Call logConsole().
- 10:52 AM Changeset in webkit [42202] by
-
- 2 edits in trunk/WebCore
2009-04-03 Darin Fisher <darin@chromium.org>
Reviewed by Darin Adler.
Restore FrameLoader::policyDocumentLoader to fix the Chromium build.
https://bugs.webkit.org/show_bug.cgi?id=25028
- loader/FrameLoader.h: (WebCore::FrameLoader::policyDocumentLoader):
- 9:58 AM Changeset in webkit [42201] by
-
- 2 edits in trunk/WebCore
Fixed a backwards comment in ChangeLog.
- 9:28 AM Changeset in webkit [42200] by
-
- 2 edits in trunk/WebCore
2009-04-02 Michael Nordman <Michael Nordman>
Reviewed by Alexey Proskuryakov.
Use long long, not int64, when working in webcore (sorry).
https://bugs.webkit.org/show_bug.cgi?id=25011
No change in functionality, so no tests.
- platform/network/chromium/ResourceResponse.h: (WebCore::ResourceResponse::getAppCacheID): (WebCore::ResourceResponse::setAppCacheID):
- 8:33 AM QtWebKitTodo edited by
- (diff)
- 6:00 AM Changeset in webkit [42199] by
-
- 14 edits in trunk
2009-04-03 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>
Reviewed by Holger Freyther.
Require GTK+ >= 2.10; 2.8 is already very old, and some very
useful APIs are only available since 2.10.
- configure.ac:
WebCore/
2009-04-03 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>
Reviewed by Holger Freyther.
Remove pre-processor checks for GTK+ >= 2.10, since we now require
at least that version.
- platform/graphics/gtk/FontPlatformDataGtk.cpp: (WebCore::FontPlatformData::FontPlatformData):
- platform/gtk/ContextMenuItemGtk.cpp: (WebCore::gtkStockIDFromContextMenuAction):
- platform/gtk/KeyEventGtk.cpp: (WebCore::PlatformKeyboardEvent::PlatformKeyboardEvent):
- platform/gtk/LocalizedStringsGtk.cpp: (WebCore::contextMenuItemTagSelectAll):
- platform/gtk/MouseEventGtk.cpp: (WebCore::PlatformMouseEvent::PlatformMouseEvent):
- platform/gtk/PasteboardGtk.cpp: (WebCore::Pasteboard::writeSelection): (WebCore::Pasteboard::documentFragment):
- platform/gtk/WheelEventGtk.cpp: (WebCore::PlatformWheelEvent::PlatformWheelEvent):
WebKit/gtk
2009-04-03 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>
Reviewed by Holger Freyther.
Remove pre-processor checks for GTK+ >= 2.10, since we now require
at least that version.
- WebCoreSupport/EditorClientGtk.cpp: (WebKit::clipboard_clear_contents_cb): (WebKit::EditorClient::respondToChangedSelection):
- webkit/webkitwebframe.cpp:
- webkit/webkitwebview.cpp: