Timeline
Aug 12, 2008:
- 9:35 PM Changeset in webkit [35704] by
-
- 2 edits in trunk/LayoutTests
2008-08-12 Sam Weinig <sam@webkit.org>
Reviewed by Anders Carlsson.
- platform/mac/Skipped: Add fast/dom/cssTarget-crash.html to the skipped list.
- 8:17 PM Changeset in webkit [35703] by
-
- 3 edits in trunk/JavaScriptCore
Add peephole optimisation to 'op_not... jfalse...' (eg. if(!...) )
Reviewed by Geoff Garen
This is a very slight win in sunspider, and a fairly substantial win
in hot code that does if(!...), etc.
- 3:49 PM Changeset in webkit [35702] by
-
- 5 edits in trunk/LayoutTests
Not reviewed.
Revert the last SVGLineElement commit. It's not needed, the first run on buildbot showed them as failures
the next build doesn't complain anymore (this was before adjusting the offsets.) No idea what's going on,
but if it's fixed I'm happy.
While I'm at it, regenerate struct-image-02-b.svg (invisible differences) to have a clean baseline.
All SVG pixel test results are accurate on Leopard now.
- 3:26 PM Changeset in webkit [35701] by
-
- 3 edits in trunk/LayoutTests
Not reviewed. The last SVGLineElement* fix attempt, fixed 4/8 failing tests.
Trying two variations in two tests to find out the right offset to make the other four tests work on Tiger.
- 3:18 PM Changeset in webkit [35700] by
-
- 1 edit40 adds in trunk/LayoutTests
Rubber stamped by Eric.
Add new dynamic-update layout tests covering SVGRectElement.
- 3:13 PM Changeset in webkit [35699] by
-
- 9 edits in trunk/LayoutTests
Not reviewed. Trying to modify mouseevent target location, to work around known CG/Tiger hit test problem.
- 2:59 PM Changeset in webkit [35698] by
-
- 5 edits in trunk/LayoutTests
Reviewed by Sam.
Forgot to include these in my last commit.
- 2:49 PM Changeset in webkit [35697] by
-
- 4 edits2 adds in trunk
WebCore:
2008-08-12 Mihnea Ovidenie <mihnea@adobe.com>
Fix for https://bugs.webkit.org/show_bug.cgi?id=19891
Broken HTML object elements cause de-reference of pointer to freed memory.
If we fail to load an image for an object tag and we no longer believe the object tag points at
an image, then clear m_imageLoader in the HTMLObjectElement so that we attempt to render the
fall back content.
Reviewed by Dave Hyatt and Alexey Proskuryakov.
Test: http/tests/misc/object-image-error-with-onload.html
- html/HTMLObjectElement.cpp: (WebCore::HTMLObjectElement::renderFallbackContent):
- page/Frame.cpp: (WebCore::Frame::Frame):
LayoutTests:
2008-08-12 Mihnea Ovidenie <mihnea@adobe.com>
Test for https://bugs.webkit.org/show_bug.cgi?id=19891
Broken HTML object elements cause de-reference of pointer to freed memory.
Reviewed by Dave Hyatt and Alexey Proskuryakov.
- http/tests/misc/object-image-error-with-onload-expected.txt: Added.
- http/tests/misc/object-image-error-with-onload.html: Added.
- 2:46 PM Changeset in webkit [35696] by
-
- 84 edits in trunk/LayoutTests
Reviewed by Eric.
Fix typo in most files: "mesage" -> "message".
Remove unused code in SVGTestCase.js
- 2:18 PM Changeset in webkit [35695] by
-
- 1 edit70 adds in trunk/LayoutTests
Rubber stamped by Eric.
Add new dynamic-update layout tests covering SVGPatternElement.
- 2:11 PM Changeset in webkit [35694] by
-
- 13 edits in trunk
Reviewed by Dave.
Fixes: https://bugs.webkit.org/show_bug.cgi?id=19798
Masks are translated, and the mask images are swapped on the y-axis.
Turned out that http://trac.webkit.org/changeset/31830/trunk/WebCore/svg/graphics/cg/SVGResourceMaskerCg.mm
is guilty. GraphicsContext::clipToImageBuffer() does some extra transformations that SVGResourcesMaskerCg does not want.
Long term goal is to remove the SVGResource*/SVGPaintServer* classes anyway, so it's okay to duplicate
the "clip to image buffer" functionality, in the CG specific SVGResourceMaskerCg class - as it was before.
- 2:05 PM Changeset in webkit [35693] by
-
- 2 edits in trunk/LayoutTests
Reviewed by Sam.
Speculative fix for hit testing problems in dynamic-updates/SVGLineElement* on Tiger.
Supposed to fix the hanging tests on the buildbot.
- 2:00 PM Changeset in webkit [35692] by
-
- 2 edits in trunk/LayoutTests
2008-08-12 Sam Weinig <sam@webkit.org>
- fast/dom/Window/window-properties-expected.txt: Update results.
- 1:54 PM Changeset in webkit [35691] by
-
- 11 edits in trunk
- JavaScriptCore part of <rdar://problem/6121636> Make fast*alloc() abort() on failure and add "try" variants that return NULL on failure.
Reviewed by Darin Adler.
- JavaScriptCore.exp: Exported tryFastCalloc().
- VM/RegisterFile.h: (KJS::RegisterFile::RegisterFile): Removed an ASSERT().
- kjs/JSArray.cpp: (KJS::JSArray::putSlowCase): Changed to use tryFastRealloc(). (KJS::JSArray::increaseVectorLength): Ditto.
- kjs/ustring.cpp: (KJS::allocChars): Changed to use tryFastMalloc(). (KJS::reallocChars): Changed to use tryFastRealloc().
- wtf/FastMalloc.cpp: (WTF::fastZeroedMalloc): Removed null checking of fastMalloc()'s result and removed extra call to InvokeNewHook(). (WTF::tryFastZeroedMalloc): Added. Uses tryFastMalloc(). (WTF::tryFastMalloc): Renamed fastMalloc() to this. (WTF::fastMalloc): Added. This version abort()s if allocation fails. (WTF::tryFastCalloc): Renamed fastCalloc() to this. (WTF::fastCalloc): Added. This version abort()s if allocation fails. (WTF::tryFastRealloc): Renamed fastRealloc() to this. (WTF::fastRealloc): Added. This version abort()s if allocation fails. (WTF::do_malloc): Made this a function template. When the abortOnFailure template parameter is set, the function abort()s on failure to allocate. Otherwise, it sets errno to ENOMEM and returns zero. (WTF::TCMallocStats::fastMalloc): Defined to abort() on failure. (WTF::TCMallocStats::tryFastMalloc): Added. Does not abort() on failure. (WTF::TCMallocStats::fastCalloc): Defined to abort() on failure. (WTF::TCMallocStats::tryFastCalloc): Added. Does not abort() on failure. (WTF::TCMallocStats::fastRealloc): Defined to abort() on failure. (WTF::TCMallocStats::tryFastRealloc): Added. Does not abort() on failure.
- wtf/FastMalloc.h: Declared the "try" variants.
WebCore:
- WebCore part of <rdar://problem/6121636> Make fast*alloc() abort() on failure and add "try" variants that return NULL on failure.
Reviewed by Darin Adler.
- platform/Arena.cpp: (WebCore::ArenaAllocate): Removed null checking of fastMalloc()'s result.
- platform/graphics/cg/ImageBufferCG.cpp: (WebCore::ImageBuffer::create): Changed to use tryFastCalloc().
- 11:27 AM Changeset in webkit [35690] by
-
- 3 edits4 adds in trunk
WebCore:
Reviewed by Darin Adler.
- fix https://bugs.webkit.org/show_bug.cgi?id=19348 <rdar://problem/5978447> REGRESSION (r34193): Setting the size of a frame with javascript document.body.row no longer works
Test: fast/frames/frameset-style-recalc.html
- html/HTMLFrameSetElement.cpp: (WebCore::HTMLFrameSetElement::recalcStyle): Changed to call the base class implementation after marking for layout.
LayoutTests:
Reviewed by Darin Adler.
- test for https://bugs.webkit.org/show_bug.cgi?id=19348 <rdar://problem/5978447> REGRESSION (r34193): Setting the size of a frame with javascript document.body.row no longer works
- fast/frames/frameset-style-recalc.html: Added.
- platform/mac/fast/frames/frameset-style-recalc-expected.checksum: Added.
- platform/mac/fast/frames/frameset-style-recalc-expected.png: Added.
- platform/mac/fast/frames/frameset-style-recalc-expected.txt: Added.
- 10:45 AM Changeset in webkit [35689] by
-
- 4 edits in trunk/WebKit/mac
Remove unneeded header imports from some Web Inspector files.
Reviewed by Adam Roben.
- WebCoreSupport/WebInspectorClient.mm:
- WebInspector/WebInspector.mm:
- WebInspector/WebNodeHighlightView.m:
- 10:45 AM Changeset in webkit [35688] by
-
- 2 edits in trunk/WebKit/mac
Remove the call to WKNSWindowMakeBottomCornersSquare on the
Web Inspector's window. This isn't needed anymore since the
window style masks used always have square bottom corners.
Reviewed by Adam Roben.
- WebCoreSupport/WebInspectorClient.mm: (-[WebInspectorWindowController window]): Remove the call to WKNSWindowMakeBottomCornersSquare.
- 10:45 AM Changeset in webkit [35687] by
-
- 2 edits in trunk/WebKit/mac
Make attaching and detaching the Web Inspector instantaneous.
This also preserves the current view, other state, and keeps
the script debugger attached.
Reviewed by Adam Roben.
- WebCoreSupport/WebInspectorClient.mm: (-[WebInspectorWindowController close]): Only call setWindowVisible(false) when not moving windows.
- 10:13 AM Changeset in webkit [35686] by
-
- 2 edits in trunk/WebKit/win
Export WTFLog in Debug builds
Reviewed by John Sullivan.
- WebKit.vcproj/WebKit_debug.def:
- 10:04 AM Changeset in webkit [35685] by
-
- 19 edits in trunk
WebCore:
Reviewed by John Sullivan.
- move shouldUpdateWhenOffscreen from Settings to FrameView and rename it shouldUpdateWhileHidden
- WebCore.base.exp:
- page/FrameView.cpp: (WebCore::FrameViewPrivate::FrameViewPrivate): (WebCore::FrameView::shouldUpdateWhileHidden): (WebCore::FrameView::setShouldUpdateWhileHidden):
- page/FrameView.h:
- page/Settings.cpp:
- page/Settings.h:
WebKit/mac:
Reviewed by John Sullivan.
- replace -[WebPreferences updatesWhenOffscreen] with -[WebView shouldUpdateWhileHidden]
- WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::transitionToCommittedForNewPage):
- WebView/WebFrame.mm: (-[WebFrame _updateBackgroundAndUpdatesWhileHidden]):
- WebView/WebFrameInternal.h:
- WebView/WebHTMLView.mm: (-[WebHTMLView windowWillOrderOnScreen:]):
- WebView/WebPreferenceKeysPrivate.h:
- WebView/WebPreferences.mm: (+[WebPreferences initialize]):
- WebView/WebPreferencesPrivate.h:
- WebView/WebView.mm: (-[WebViewPrivate init]): (-[WebView _preferencesChangedNotification:]): (-[WebView setBackgroundColor:]): (-[WebView setDrawsBackground:]): (-[WebView shouldUpdateWhileHidden]): (-[WebView setShouldUpdateWhileHidden:]):
- WebView/WebViewPrivate.h:
- 7:58 AM Changeset in webkit [35684] by
-
- 2 edits in trunk/WebCore
Windows build fix
- bindings/js/JSSVGPODTypeWrapper.h: Align JSSVGDynamicPODTypeWrapper's and JSSVGStaticPODTypeWrapperWithParent's members on 16-byte boundaries to avoid an alignment warning.
- 7:40 AM Changeset in webkit [35683] by
-
- 3 edits20 adds in trunk/LayoutTests
Rubber stamped by Anders.
Add new dynamic-update layout tests covering SVGCursorElement.
Scale svg/webarchive/resources/green-checker.png to a value < 128x128,
as this is the maximum hardcoded cursor size (fixes manual testing of svg-cursor-subresources.png).
- 5:01 AM Changeset in webkit [35682] by
-
- 1 edit4 moves60 adds in trunk/LayoutTests
Rubber stamped by Oliver.
Add new dynamice-update layout tests covering SVGMaskElement.
Move pixel tests results of SVGMarkerElement* to the right positions, while I'm at it.
- 4:59 AM Changeset in webkit [35681] by
-
- 3 edits in trunk/WebKit/qt
2008-08-12 Urs Wolfer <uwolfer@kde.org>
Reviewed by Simon.
Fix crash in QWebPage in case contextMenuEvent has been overwritten
because context menu has the view as parent and thus is deleted too early.
Add testcase for this crash.
- 3:35 AM Changeset in webkit [35680] by
-
- 3 edits75 adds in trunk
Reviewed by Oliver.
Add new dynamice-update layout tests covering SVGMarkerElement.
Fix bug: SVGMarkerElement's SVG DOM function calls don't update rendering.
Fix orientAngle/orientType confusion: "auto" orient should always return "0" as angle.
- 3:27 AM Changeset in webkit [35679] by
-
- 3 edits50 adds in trunk
Reviewed by Oliver.
Add new dynamic-update layout tests covering SVGImageElement.
Fix bug: SVGImageElement doesn't react on 'preserveAspectRatio' changes.
Aug 11, 2008:
- 9:28 PM Changeset in webkit [35678] by
-
- 2 edits in trunk/WebCore
Changed Option/Alt-Up or Down in CSS editing when the value is
near zero to jump to the next integer.
Reviewed by Tim Hatcher.
- page/inspector/StylesSidebarPane.js:
- 9:20 PM Changeset in webkit [35677] by
-
- 2 edits in trunk/WebCore
Changed the line highlight transition for an easier animation.
Reviewed by Tim Hatcher.
- page/inspector/SourceFrame.js:
- 8:57 PM Changeset in webkit [35676] by
-
- 2 edits in trunk/WebCore
Added support for some Firebug Command Line APIs.
Reviewed by Tim Hatcher.
https://bugs.webkit.org/show_bug.cgi?id=19867
https://bugs.webkit.org/show_bug.cgi?id=19868
https://bugs.webkit.org/show_bug.cgi?id=19869
https://bugs.webkit.org/show_bug.cgi?id=19875
https://bugs.webkit.org/show_bug.cgi?id=19876
https://bugs.webkit.org/show_bug.cgi?id=19880
- page/inspector/Console.js: (WebInspector.Console.prototype._evalInInspectedWindow): Create an object on the inspected window that holds the console command line API functions. This object is used in a with statement around the typed expression.
- 7:55 PM Changeset in webkit [35675] by
-
- 1 edit110 adds in trunk/LayoutTests
Rubber stamped by Oliver.
Add a bunch of new dynamic-updates tests covering SVGLineElement & SVGRadialGradientElement.
No functional changes needed, all work out of the box.
- 7:22 PM Changeset in webkit [35674] by
-
- 11 edits15 adds in trunk
Reviewed by Antti.
Fixes: http://bugs.webkit.org/show_bug.cgi?id=17736
JS wrapper objects around SVG POD types, that contain other SVG POD types with writable properties
failed to update. Modification of the values were completly ignored (ie. transform.matrix.a = 50, didn't take any effect)
Added tests: svg/custom/svg-modify-currentTranslate.html
svg/custom/tearoffs-with-tearoffs.html
svg/custom/immutable-properties.html
Fixed tests: svg/dynamic-updates/SVGLinearGradientElement-svgdom-gradientTransform-prop.html
- 5:34 PM Changeset in webkit [35673] by
-
- 3 edits in trunk/WebCore
2008-08-11 Brady Eidson <beidson@apple.com>
Reviewed by John and Anders
Fix for <rdar://problem/6141797>
When WebArchives were entirely a WebKit concept, there was a guarantee that a WebResource
would never have nil data.
When they were pushed down into WebCore, that guarantee was lost, subtly changing a few
semantics with some WebKit applications.
The guarantee was a good one and should be restored.
Note that ApplicationCacheResource doesn't need any updates to follow this rule as it already
creates an empty data object in the case of null data for its own purposes.
- loader/SubstituteResource.h: (WebCore::SubstituteResource::SubstituteResource): ASSERT that the data is not null. This well help any future subclassers not make this mistake.
- loader/archive/ArchiveResource.cpp: (WebCore::ArchiveResource::create): Return 0 if the data is null.
- 3:44 PM Changeset in webkit [35672] by
-
- 3 edits2 adds in trunk
2008-08-11 Simon Fraser <Simon Fraser>
Reviewed by Dave Hyatt
https://bugs.webkit.org/show_bug.cgi?id=20328
Fix a problem when an 'all' transition transition with more than
one property changing is interrupted, and did some AnimationController
cleanup.
Test: transitions/interrupted-all-transition.html
- page/AnimationController.cpp: (WebCore::ImplicitAnimation::ImplicitAnimation): (WebCore::AnimationControllerPrivate::blendProperties): (WebCore::CompositeAnimation::updateTransitions): (WebCore::CompositeAnimation::cleanupFinishedAnimations): (WebCore::CompositeAnimation::setTransitionStartTime): (WebCore::CompositeAnimation::overrideImplicitAnimations): (WebCore::CompositeAnimation::resumeOverriddenImplicitAnimations): (WebCore::ImplicitAnimation::animate): (WebCore::ImplicitAnimation::onAnimationEnd): (WebCore::ImplicitAnimation::sendTransitionEvent): (WebCore::ImplicitAnimation::affectsProperty): (WebCore::KeyframeAnimation::endAnimation): (WebCore::KeyframeAnimation::onAnimationEnd):
- 1:45 PM Changeset in webkit [35671] by
-
- 3 edits in trunk/WebCore
2008-08-11 Kevin McCullough <kmccullough@apple.com>
Reviewed by Tim.
- Because console messages have group levels now, newly created messages that do not specify the level lose their message since the number of arguments is wrong.
- page/inspector/Console.js:
- page/inspector/Resource.js:
- 12:52 PM Changeset in webkit [35670] by
-
- 2 edits in trunk/WebCore
2008-08-11 Alp Toker <alp@nuanti.com>
Build fix. Add new files from r35666 (WebKitAnimationEvent). Also take
the opportunity to sort the sources lists.
- GNUmakefile.am:
- 11:48 AM Changeset in webkit [35669] by
-
- 2 edits in trunk/WebCore
Speed up the the JavaScript syntax highlighter by generating
the finders only once per script instead of per line.
Reviewed by Adam Roben.
- page/inspector/SourceFrame.js: (WebInspector.SourceFrame.prototype._syntaxHighlightJavascriptLine): Removed, factored into syntaxHighlightJavascript as an inline function. (WebInspector.SourceFrame.prototype.syntaxHighlightJavascript): Pulled in the _syntaxHighlightJavascriptLine so it will create a closure. Generate the finders before iterating the lines.
- 11:31 AM Changeset in webkit [35668] by
-
- 2 edits in trunk/WebCore
Windows build fix
- WebCore.vcproj/WebCore.vcproj: Added JSWebKitAnimationEvent.cpp and JSWebKitTransitionEvent.cpp to the project. Let VS reformat the file, too.
- 11:04 AM Changeset in webkit [35667] by
-
- 7 edits4 adds in trunk
WebCore:
Reviewed by Darin Adler.
- fix <rdar://problem/6131096> Reproducible crash in CounterNode::isReset under guard malloc
Test: fast/css/counters/invalidate-cached-counter-node.html
- rendering/RenderContainer.cpp: (WebCore::RenderContainer::invalidateCounters): Added. Invalidates all RenderCounters in :before and :after content.
- rendering/RenderContainer.h:
- rendering/RenderCounter.cpp: (WebCore::RenderCounter::isCounter): Renamed isRenderCounter() to this to match the RenderObject method. (WebCore::RenderCounter::invalidate): Added. Resets the cached CounterNode and invalidates the object's layout and preferred widths. (WebCore::destroyCounterNodeChildren): Added a call to invalidateCounters().
- rendering/RenderCounter.h:
- rendering/RenderObject.h: (WebCore::RenderObject::invalidateCounters):
LayoutTests:
Reviewed by Darin Adler.
- test for <rdar://problem/6131096> Reproducible crash in CounterNode::isReset under guard malloc
- fast/css/counters/invalidate-cached-counter-node.html: Added.
- platform/mac/fast/css/counters/invalidate-cached-counter-node-expected.checksum: Added.
- platform/mac/fast/css/counters/invalidate-cached-counter-node-expected.png: Added.
- platform/mac/fast/css/counters/invalidate-cached-counter-node-expected.txt: Added.
- 10:24 AM Changeset in webkit [35666] by
-
- 19 edits7 adds in trunk/WebCore
Implement CSS Animation and Transition Events
https://bugs.webkit.org/show_bug.cgi?id=20337
Implement the events defined in the CSS Animations
and Transitions specifications so code can react
to animations and transitions.
Reviewed by Tim Hatcher and Dave Hyatt.
- DerivedSources.make:
- GNUmakefile.am:
- WebCore.pro:
- WebCore.vcproj/WebCore.vcproj:
- WebCore.xcodeproj/project.pbxproj:
- WebCoreSources.bkl:
Build configs for new files
- bindings/js/JSDOMWindowBase.cpp:
- bindings/js/JSDOMWindowBase.h:
- bindings/js/JSEventCustom.cpp:
- dom/Document.h:
- dom/Event.cpp:
- dom/Event.h:
- dom/EventTarget.cpp:
- dom/EventTargetNode.cpp:
- dom/EventTargetNode.h:
- html/HTMLElement.cpp:
- page/AnimationController.cpp:
do all the new event stuff
- html/HTMLAttributeNames.in:
the onwebkitanimation* and onwebkittransitionend attrs
- dom/WebKitAnimationEvent.cpp: Added.
- dom/WebKitAnimationEvent.h: Added.
- dom/WebKitAnimationEvent.idl: Added.
- dom/WebKitTransitionEvent.cpp: Added.
- dom/WebKitTransitionEvent.h: Added.
- dom/WebKitTransitionEvent.idl: Added.
New files for the events
- manual-tests/transition-events.html: Added.
New testfile
- 10:21 AM Changeset in webkit [35665] by
-
- 5 edits2 adds in trunk
Move WTF::notFound into its own header so that it can be used independently of Vector
Move WTF::notFound into its own header so that it can be used
independently of Vector
Rubberstamped by Darin Adler.
- JavaScriptCore.vcproj/WTF/WTF.vcproj:
- JavaScriptCore.xcodeproj/project.pbxproj: Added NotFound.h to the project.
- wtf/NotFound.h: Added. Moved the notFound constant here...
- wtf/Vector.h: ...from here.
WebCore:
Add a ForwardingHeader for wtf/NotFound.h
Rubberstamped by Darin Adler.
- ForwardingHeaders/wtf/NotFound.h: Added.
- 9:53 AM Changeset in webkit [35664] by
-
- 2 edits2 adds in trunk/WebCore
Fixes a bug where error bubbles in JavaScript resources would
be clobbered by the syntax highlighter.
Reviewed by Adam Roben.
- manual-tests/inspector/resources/script-error.js: Added.
- manual-tests/inspector/styled-error-bubbles-in-scripts.html: Added.
- page/inspector/SourceFrame.js: (WebInspector.SourceFrame.prototype._addMessageToSource): Check the nodeType and not the nodeName, this is less fragile. (WebInspector.SourceFrame.prototype._syntaxHighlightJavascriptLine): Check if the lastChild is an error bubble, if so remove it before getting the line's textContent. Add the error bubble back at the end.
- 5:01 AM Changeset in webkit [35663] by
-
- 3 edits in trunk/JavaScriptCore
Reviewed by Mark Rowe.
<rdar://problem/6130393> REGRESSION: PhotoBooth hangs after launching under TOT Webkit
- API/JSContextRef.cpp: (JSGlobalContextRelease): Corrected a comment.
- kjs/collector.cpp: (KJS::Heap::~Heap): Ensure that JSGlobalData is not deleted while sweeping the heap.