Timeline



Aug 7, 2008:

9:05 PM Changeset in webkit [35636] by timothy@apple.com
  • 1 edit in trunk/WebCore/ChangeLog

Fix bug numbers.

9:04 PM Changeset in webkit [35635] by mrowe@apple.com
  • 1 edit in tags/Safari-6528.0.3/WebCore/DerivedSources.cpp

Build fix.

9:04 PM Changeset in webkit [35634] by mrowe@apple.com
  • 4 edits in tags/Safari-6528.0.3

Versioning.

8:56 PM Changeset in webkit [35633] by timothy@apple.com
  • 2 edits in trunk/WebCore

Change highlight time from 300ms to 500ms and fade effect from 1s to 2s

https://bugs.webkit.org/show_bug.cgi?id=17589

Reviewed by Timothy Hatcher.

  • page/inspector/SourceFrame.js:
8:55 PM Changeset in webkit [35632] by timothy@apple.com
  • 3 edits in trunk/WebCore

The search is only performed if the field contains more than 3
letters or if Enter is pressed.

Search is now performed through a keyup event.

Added a WebInspector.lastQuery to remember the last query and not
perform it again (eg. the user hits a modifier key).

https://bugs.webkit.org/show_bug.cgi?id=18548

Reviewed by Timothy Hatcher.

  • page/inspector/inspector.html: Removed the onsearch and incremental attributes
  • page/inspector/inspector.js:
8:30 PM Changeset in webkit [35631] by mrowe@apple.com
  • 1 copy in tags/Safari-6528.0.3

New tag.

5:35 PM Changeset in webkit [35630] by mrowe@apple.com
  • 2 edits
    1 add in tags/Safari-6528.0.2/WebCore

Merge r35629.

4:38 PM Changeset in webkit [35629] by sfalken@apple.com
  • 2 edits
    1 add in trunk/WebCore

Compile derived sources through an all-in-one cpp file.
Fixes Windows release build on non-x64 systems by reducing the size of WebCore.lib from 1.5GB to 1GB.


Reviewed by Mark Rowe.

  • DerivedSources.cpp: Added.
  • WebCore.vcproj/WebCore.vcproj:
3:29 PM Changeset in webkit [35628] by kmccullough@apple.com
  • 4 edits in trunk/WebCore

2008-08-07 Kevin McCullough <kmccullough@apple.com>

Reviewed by Adam.

<rdar://problem/5712917> Confusing console error message clear
behavior (16791)
https://bugs.webkit.org/show_bug.cgi?id=16791
<rdar://problem/5831899> The Web Inspector does not correctly clear the
console messages

  • page/InspectorController.cpp: Clear the message so they don't show up in the inspector after it has cleared them. (WebCore::clearMessages): (WebCore::InspectorController::clearConsoleMessages): (WebCore::InspectorController::windowScriptObjectAvailable):
  • page/InspectorController.h:
  • page/inspector/Console.js: Tell the InspectorController to clear the messages it's holding onto.
2:55 PM Changeset in webkit [35627] by Simon Fraser
  • 10 edits
    2 adds in trunk/WebCore

2008-08-07 Simon Fraser <Simon Fraser>

Reviewed by Sam Weinig

Rename Marquee to RenderMarquee and move to its own file.
https://bugs.webkit.org/show_bug.cgi?id=20319

  • GNUmakefile.am:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:
  • html/HTMLMarqueeElement.cpp:
  • rendering/RenderBlock.cpp:
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::styleChanged):
  • rendering/RenderLayer.h:
  • rendering/RenderMarquee.cpp: Added. (WebCore::RenderMarquee::RenderMarquee): (WebCore::RenderMarquee::marqueeSpeed): (WebCore::RenderMarquee::direction): (WebCore::RenderMarquee::isHorizontal): (WebCore::RenderMarquee::computePosition): (WebCore::RenderMarquee::start): (WebCore::RenderMarquee::suspend): (WebCore::RenderMarquee::stop): (WebCore::RenderMarquee::updateMarqueePosition): (WebCore::RenderMarquee::updateMarqueeStyle): (WebCore::RenderMarquee::timerFired):
  • rendering/RenderMarquee.h: Added. (WebCore::RenderMarquee::speed): (WebCore::RenderMarquee::reverseDirection): (WebCore::RenderMarquee::setEnd):
1:32 PM Changeset in webkit [35626] by mitz@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Dave Hyatt.

  • fix <rdar://problem/6118816> Initial empty page or bookmarks view window has white strip where vertical scrollbar would be (which vanishes on resize)
  • page/Frame.cpp: (WebCore::Frame::setZoomFactor): If the view has never had a layout with the old zoom factor, then there is no need to explicitly call layout() here.
10:28 AM Changeset in webkit [35625] by kmccullough@apple.com
  • 2 edits in trunk/WebCore

2008-08-07 Kevin McCullough <kmccullough@apple.com>

Reviewed by Geoff.

<rdar://problem/6115981> Start the profiler in Heavy view

  • page/inspector/ProfileView.js:
9:04 AM Changeset in webkit [35624] by sfalken@apple.com
  • 2 edits in trunk/WebKit/win

Delete backing stores before 5 second WM_TIMER fires if more than 2 are already queued.


WM_TIMER-based timers can be starved if enough other events are keeping
our message loop busy. This change prevents many backing store deletes from stacking
up in this case.


Reviewed by Geoff Garen.

  • WebView.cpp: (WebView::deleteBackingStore): (WebView::deleteBackingStoreSoon): (WebView::cancelDeleteBackingStoreSoon):
5:09 AM Changeset in webkit [35623] by jmalonzo@webkit.org
  • 2 edits in trunk/WebCore

2008-08-07 Alp Toker <alp@nuanti.com>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=20313
Add null check in String::fromUTF8()

Make String::fromUTF8() consistent with other constructors by
returning a null String when the input is null instead of crashing.

  • platform/text/String.cpp: (WebCore::String::fromUTF8):
5:08 AM Changeset in webkit [35622] by jmalonzo@webkit.org
  • 3 edits in trunk

2008-08-07 Kalle Vahlman <zuh@iki.fi>

Reviewed by David Kilzer.

Check for correct flex version to avoid faulty builds

A correct Webkit build requires flex 2.5.33 but autotools build doesn't check for it
https://bugs.webkit.org/show_bug.cgi?id=20253

acinclude.m4: Import AX_COMPARE_VERSION macro
configure.ac: Check for flex version

  • acinclude.m4:
  • configure.ac:
4:31 AM Changeset in webkit [35621] by Simon Hausmann
  • 3 edits
    8 adds in trunk

2008-08-07 Simon Hausmann <Simon Hausmann>

Rubber-stamped by Lars.

Added API tests for QWebPage/QWebFrame based on QTestLib.

Aug 6, 2008:

8:52 PM Changeset in webkit [35620] by kevino@webkit.org
  • 4 edits
    2 adds in trunk/WebKit/wx

Reviewed by Kevin Ollivier.

Create a wxWebFrame API to match other ports and to prepare for frames support.
Also fixes a frame leak in wx port on trunk.

https://bugs.webkit.org/show_bug.cgi?id=19041

5:46 PM Changeset in webkit [35619] by mrowe@apple.com
  • 1 edit in trunk/WebKitLibraries/win/tools/scripts/VERSION

Versioning.

4:48 PM Changeset in webkit [35618] by kevino@webkit.org
  • 2 edits in trunk/WebCore

wx build fix after Frame -> ScriptController moves. Also, remove a character which was in an unknown encoding and replace it with an ANSI representation (which the author used for their Bugzilla account too).

4:08 PM Changeset in webkit [35617] by kmccullough@apple.com
  • 2 edits in trunk/WebCore

2008-08-06 Kevin McCullough <kmccullough@apple.com>

Reviewed by Tim.

  • Make XHR console message clearer.
  • xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::didFinishLoading):
4:07 PM Changeset in webkit [35616] by kmccullough@apple.com
  • 4 edits in trunk/WebCore

2008-08-06 Kevin McCullough <kmccullough@apple.com>

Reviewed by Tim.

  • Created a new function to create DOM links from text without converting back and forth from HTML and messing with escaping special characters.
  • page/inspector/Console.js:
  • page/inspector/inspector.js:
  • page/inspector/utilities.js:
3:39 PM Changeset in webkit [35615] by eric@webkit.org
  • 3 edits in trunk/WebCore

Reviewed by hyatt.

Fix a large animation leak found on the buildbot
(m_animations and m_transitions were never deleted)
Drag RenderStyle (kicking and screaming) into the 21st century
of memory management with a little application of OwnPtr.

  • rendering/style/RenderStyle.cpp: (WebCore::FillLayer::cullEmptyLayers): (WebCore::RenderStyle::diff): (WebCore::RenderStyle::contentDataEquivalent): (WebCore::RenderStyle::setContent): (WebCore::BindingURI::BindingURI): (WebCore::RenderStyle::setBoxShadow): (WebCore::ShadowData::ShadowData): (WebCore::RenderStyle::counterDirectives): (WebCore::RenderStyle::accessCounterDirectives): (WebCore::RenderStyle::adjustAnimations): (WebCore::RenderStyle::adjustTransitions): (WebCore::RenderStyle::accessAnimations): (WebCore::RenderStyle::accessTransitions):
  • rendering/style/RenderStyle.h: (WebCore::RenderStyle::clearAnimations): (WebCore::RenderStyle::clearTransitions):
3:17 PM Changeset in webkit [35614] by beidson@apple.com
  • 4 edits in trunk/WebCore

2008-08-06 Brady Eidson <beidson@apple.com>

Reviewed by MitzPettel

<rdar://problem/6078543> - Many LocalStorageThreads may be started

The original decision to key LocalStorage sets to PageGroups was a poor one.

It seems a much better solution is to key them to the actual path of on-disk storage,
which also solves all of the avenues that could lead to a boundless number of threads.

In this patch, I am not attempting to completely sever the attachment of LocalStorage
to PageGroups as that would require a more substantial re-working of the lifetime
between pages, groups, and local storage threads.

  • page/PageGroup.cpp: (WebCore::PageGroup::addPage):
  • storage/LocalStorage.cpp: (WebCore::localStorageMap): Map paths to LocalStorage objects here. (WebCore::LocalStorage::localStorage): Return the shared LocalStorage object for this path, or create a new one if it doesn't yet exist. (WebCore::LocalStorage::LocalStorage): (WebCore::LocalStorage::~LocalStorage): Add to remove the path->LocalStorage mapping
  • storage/LocalStorage.h:
3:12 PM Changeset in webkit [35613] by Beth Dakin
  • 2 edits in trunk/WebCore

2008-08-06 Beth Dakin <Beth Dakin>

Reviewed by Adele.

Fix for <rdar://problem/6130724>

Disconnected frames should not be allowed to resize the window.

  • page/DOMWindow.cpp: (WebCore::DOMWindow::resizeBy): (WebCore::DOMWindow::resizeTo):
2:27 PM Changeset in webkit [35612] by mitz@apple.com
  • 1 edit in trunk/WebKitTools/ChangeLog

Added bug URLs

2:15 PM Changeset in webkit [35611] by ddkilzer@apple.com
  • 5 edits
    2 adds in trunk

WebCore:

Bug 20038: REGRESSION (r35151): Can't post comments on flickr.com

<https://bugs.webkit.org/show_bug.cgi?id=20038>
<rdar://problem/6092270>

Reviewed by Eric Seidel.

Test: fast/forms/submit-to-url-fragment.html

The problem was that isFormSubmission was not being set to true in
FrameLoader::loadWithDocumentLoader() when we were actually
submitting a form, causing the page to scroll instead of the form to
be submitted.

The isFormSubmission variable wasn't set to true because a FormState
object was not being created in
FrameLoader::loadFrameRequestWithFormAndValues().

The FormState object was not being created because
HTMLFormElement::submit(Event*, bool activateSubmitButton) would
only set FrameLoader::m_formAboutToBeSubmitted to the current form
if there was a "text field" element in the form (type = text,
password, search or isindex).

Thus when FrameLoader::submitForm(const FrameLoadRequest&, Event*)
called FrameLoader::loadFrameRequestWithFormAndValues(), a null
HTMLFormElement pointer would be sent and cause the above failures.

  • html/HTMLFormElement.cpp: (WebCore::HTMLFormElement::submit): Call new FrameLoader::setFormAboutToBeSubmitted() method outside the for loop so we always set FrameLoader::m_formAboutToBeSubmitted exactly once for any form submission. The FrameLoader::recordFormValue() method is only called with the name/value pair of each text field.
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::setFormAboutToBeSubmitted): Added method that only sets m_formAboutToBeSubmitted. (WebCore::FrameLoader::recordFormValue): Removed PassRefPtr<HTMLFormElement> argument since this method only sets name/value pairs on m_formValuesAboutToBeSubmitted now. (WebCore::FrameLoader::loadFrameRequestWithFormAndValues): Create a FormState object as long as submitForm is not null so that other FrameLoader methods know when a form is being submitted.
  • loader/FrameLoader.h:

LayoutTests:

Bug 20038: REGRESSION (r35151): Can't post comments on flickr.com

<https://bugs.webkit.org/show_bug.cgi?id=20038>
<rdar://problem/6092270>

Reviewed by Eric Seidel.

  • fast/forms/submit-to-url-fragment-expected.txt: Added.
  • fast/forms/submit-to-url-fragment.html: Added.
1:26 PM Changeset in webkit [35610] by ddkilzer@apple.com
  • 4 edits in trunk/WebCore

BUILD FIX: Add HTMLPlugInImageElement.cpp to all other build systems

  • GNUmakefile.am:
  • WebCore.pro:
  • WebCoreSources.bkl:
1:22 PM Changeset in webkit [35609] by mitz@apple.com
  • 2 edits in trunk/WebCore
  • Windows build fix
  • WebCore.vcproj/WebCore.vcproj: Added PlugInImageElement.*
1:01 PM Changeset in webkit [35608] by eric@webkit.org
  • 6 edits in trunk/WebKitTools

Reviewed by darin.

Fix leaks seen on build-bot by fixing memory management of AccessibilityUIElement.

I've made AccessibilityUIElement a stack object for now. It could be
made RefCounted, but I figured that might be overkill for the moment.
Essentially it's just a RefPtr itself. :)

I also fixed a few typos, such as AccessibilityUIElement() instead of ~AccessibilityUIElement()
which was causing additional leaks. :)

I added toAXElement to remove a bunch of copy/paste code.

  • DumpRenderTree/AccessibilityController.h:
  • DumpRenderTree/AccessibilityUIElement.cpp: (toAXElement): (allAttributesCallback): (attributesOfLinkedUIElementsCallback): (attributesOfChildrenCallback): (lineForIndexCallback): (boundsForRangeCallback): (childAtIndexCallback): (getRoleCallback): (getTitleCallback): (getDescriptionCallback): (getWidthCallback): (getHeightCallback): (getIntValueCallback): (getMinValueCallback): (getMaxValueCallback): (getInsertionPointLineNumberCallback): (finalize):
  • DumpRenderTree/AccessibilityUIElement.h: (AccessibilityUIElement::platformUIElement):
  • DumpRenderTree/mac/AccessibilityControllerMac.mm: (AccessibilityController::focusedElement):
  • DumpRenderTree/mac/AccessibilityUIElementMac.mm: (AccessibilityUIElement::AccessibilityUIElement): (AccessibilityUIElement::getLinkedUIElements): (AccessibilityUIElement::getChildren): (AccessibilityUIElement::getChildAtIndex): (AccessibilityUIElement::attributesOfLinkedUIElements): (AccessibilityUIElement::attributesOfChildren):
11:47 AM Changeset in webkit [35607] by kmccullough@apple.com
  • 4 edits in trunk/WebCore

2008-08-06 Kevin McCullough <kmccullough@apple.com>

Reviewed by Tim and Darin.

<rdar://problem/6126160> URLs in the console should be clickable.

  • Implemented and now use linkifyString() to parse console messages for URLs.
  • page/inspector/Console.js:
  • page/inspector/inspector.js:
  • page/inspector/utilities.js:
10:23 AM Changeset in webkit [35606] by ddkilzer@apple.com
  • 8 edits
    10 adds in trunk

WebCore:

Images using QT plugin do not display correctly

<https://bugs.webkit.org/show_bug.cgi?id=9214>
<rdar://problem/5683413>

Reviewed by Darin.

Tests: fast/images/embed-image.html

fast/images/object-image.html

Use RenderImage instead of RenderPartObject for a renderer when
the <embed> tag references an image. Extract common code from
HTMLEmbedElement and HTMLObjectElement into HTMLPlugInImageElement.

  • WebCore.xcodeproj/project.pbxproj: Added new source files.
  • html/HTMLEmbedElement.cpp: Cleaned up #include statements. (WebCore::HTMLEmbedElement::HTMLEmbedElement): Extend HTMLPlugInImageElement which extends HTMLPlugInElement. (WebCore::HTMLEmbedElement::parseMappedAttribute): Split codeAttr and srcAttr cases. Create or clear HTMLImageLoader as needed for srcAttr. (WebCore::HTMLEmbedElement::rendererIsNeeded): Use HTMLPlugInElement::rendererIsNeeded() for images. (WebCore::HTMLEmbedElement::createRenderer): Create a RenderImage instead of a RenderPartObject for images. (WebCore::HTMLEmbedElement::attach): Create HTMLImageLoader, load image and populate RenderImage for images. (WebCore::HTMLEmbedElement::updateWidget): Don't call RenderPartObject::updateWidget() for images. (WebCore::HTMLEmbedElement::imageSourceAttributeName): Added.
  • html/HTMLEmbedElement.h: Removed unneeded namespace and class declarations. Extend HTMLPlugInImageElement which extends HTMLPlugInElement. (WebCore::HTMLEmbedElement::imageSourceAttributeName): Added. (WebCore::HTMLEmbedElement::serviceType): Moved to HTMLPlugInImageElement. (WebCore::HTMLEmbedElement::url): Ditto.
  • html/HTMLObjectElement.cpp: Cleaned up #include statements. (WebCore::HTMLObjectElement::attach): Removed local variable. (WebCore::HTMLObjectElement::isImageType): Moved to HTMLPlugInImageElement.
  • html/HTMLObjectElement.h: Extend HTMLPlugInImageElement which extends HTMLPlugInElement. (WebCore::HTMLObjectElement::isImageType): Moved to HTMLPlugInImageElement. (WebCore::HTMLObjectElement::serviceType): Ditto. (WebCore::HTMLObjectElement::url): Ditto.
  • html/HTMLPlugInImageElement.cpp: Added. (WebCore::HTMLPlugInImageElement::HTMLPlugInImageElement): Added. (WebCore::HTMLPlugInImageElement::~HTMLPlugInImageElement): Added. (WebCore::HTMLPlugInImageElement::isImageType): Moved from HTMLObjectElement.
  • html/HTMLPlugInImageElement.h: Added. (WebCore::HTMLPlugInImageElement::isImageType): Added from HTMLObjectElement. (WebCore::HTMLPlugInImageElement::serviceType): Ditto. (WebCore::HTMLPlugInImageElement::url): Ditto.
  • rendering/HitTestResult.cpp: (WebCore::HitTestResult::absoluteImageURL): Return urlString for embed elements with images.

LayoutTests:

Images using QT plugin do not display correctly

<https://bugs.webkit.org/show_bug.cgi?id=9214>
<rdar://problem/5683413>

Reviewed by Darin.

Test images rendered by both embed and object elements.

  • fast/images/embed-image.html: Added.
  • fast/images/object-image.html: Added.
  • platform/mac/fast/images/embed-image-expected.checksum: Added.
  • platform/mac/fast/images/embed-image-expected.png: Added.
  • platform/mac/fast/images/embed-image-expected.txt: Added.
  • platform/mac/fast/images/object-image-expected.checksum: Added.
  • platform/mac/fast/images/object-image-expected.png: Added.
  • platform/mac/fast/images/object-image-expected.txt: Added.
9:33 AM Changeset in webkit [35605] by Chris Fleizach
  • 4 edits in trunk

Fix an accessibility layout test that was failing on different platforms

7:50 AM Changeset in webkit [35604] by Adam Roben
  • 2 edits in trunk/JavaScriptCore

Try to fix the Windows build bots

  • API/JSBase.cpp: Touch this to force JSC to rebuild and re-copy the WTF headers.
6:30 AM Changeset in webkit [35603] by Simon Hausmann
  • 2 edits in trunk/WebKit/qt

2008-08-06 Benjamin C Meyer <ben@meyerhome.net>

Reviewed by Simon.

During the drag operation only accept the event if the action is not ignore action.

6:02 AM Changeset in webkit [35602] by vestbo@webkit.org
  • 2 edits in trunk/JavaScriptCore

Revert change 35595.

5:46 AM Changeset in webkit [35601] by jmalonzo@webkit.org
  • 2 edits in trunk/WebCore

2008-08-06 Marco Barisione <marco.barisione@collabora.co.uk>

Reviewed by Eric Seidel.

http://bugs.webkit.org/show_bug.cgi?id=20295
RegularExpression::match should not crash when a null string is passed

RegularExpression::match used to pass null subject strings to
jsRegExpExecute causing an ASSERT failure and then a crash.
There is no need to fix also RegularExpression:search and
RegularExpression::searchRev as they just call
RegularExpression::match.

  • platform/text/RegularExpression.cpp: (WebCore::RegularExpression::match): Return -1 if the string is null.
5:40 AM Changeset in webkit [35600] by jmalonzo@webkit.org
  • 2 edits in trunk

2008-08-06 Marco Barisione <marco.barisione@collabora.co.uk>

Reviewed by Eric Seidel.

http://bugs.webkit.org/show_bug.cgi?id=20276
[GTK] Makefile rules for the enum GTypes files depend on Makefile
instead of GNUmakefile

The patch committed in revision 34649 added the code needed to
generate GTypes for enums but the makefile rules for
stamp-webkitenumtypes.h and webkitenumtypes.cpp were depending on
Makefile (used by other ports) instead of GNUmakefile.

  • GNUmakefile.am: Depend on GNUmakefile instead of Makefile.
5:35 AM Changeset in webkit [35599] by jmalonzo@webkit.org
  • 3 edits in trunk/WebCore

2008-08-06 Jan Michael Alonzo <jmalonzo@webkit.org>

Gtk build fix only

  • bindings/js/ScriptControllerGtk.cpp:
  • plugins/gtk/PluginViewGtk.cpp: (WebCore::PluginView::getValue): (WebCore::PluginView::~PluginView):
5:11 AM Changeset in webkit [35598] by eric@webkit.org
  • 11 edits in trunk

Reviewed by Cameron Zwarich.

Fix warnings and errors seen on Qt bot.
Including one correctness fix for ThreadingQt.
According to the Qt 4.4 docs, we were always passing
seconds to "wait()" when we should have been passing
milliseconds.

Lots of float <-> int conversion warnings.

  • css/CSSCursorImageValue.cpp: (WebCore::CSSCursorImageValue::updateIfSVGCursorIsUsed):
  • dom/Element.cpp: (WebCore::adjustForLocalZoom): (WebCore::adjustForAbsoluteZoom): (WebCore::Element::setScrollLeft): (WebCore::Element::setScrollTop):
  • dom/EventTargetNode.cpp: (WebCore::EventTargetNode::dispatchMouseEvent):
  • loader/CachedImage.cpp: (WebCore::CachedImage::imageRect):
  • page/DOMWindow.cpp: (WebCore::DOMWindow::innerHeight): (WebCore::DOMWindow::innerWidth): (WebCore::DOMWindow::scrollX): (WebCore::DOMWindow::scrollY): (WebCore::DOMWindow::scrollTo):
  • page/EventHandler.cpp: (WebCore::scrollAndAcceptEvent):
  • plugins/npapi.cpp:
  • plugins/qt/PluginViewQt.cpp: (WebCore::PluginView::getValue): (WebCore::PluginView::~PluginView):
  • rendering/RenderReplaced.cpp: (WebCore::RenderReplaced::intrinsicSizeChanged):
4:46 AM Changeset in webkit [35597] by vestbo@webkit.org
  • 4 edits in trunk

2008-08-06 Ariya Hidayat <ariya.hidayat@trolltech.com>

Fix the Qt build due to recent ScriptController refactoring.

4:46 AM Changeset in webkit [35596] by vestbo@webkit.org
  • 1 edit in trunk/JavaScriptCore/ChangeLog

Fix non-Mac build.

4:17 AM Changeset in webkit [35595] by vestbo@webkit.org
  • 2 edits in trunk/JavaScriptCore

2008-08-06 Ariya Hidayat <ariya.hidayat@trolltech.com>

Fix non-Mac build.

3:50 AM Changeset in webkit [35594] by cwzwarich@webkit.org
  • 2 edits in trunk/JavaScriptCore

2008-08-06 Csaba Osztrogonac <oszi@inf.u-szeged.hu>

Reviewed by Darin. Landed by Cameron.

Bug 20272: typo in JavaScriptCore
<https://bugs.webkit.org/show_bug.cgi?id=20272>

Correct the documentation for op_not. (typo)
Fix #undef. (typo)

  • VM/Machine.cpp: (KJS::Machine::privateExecute):
3:37 AM Changeset in webkit [35593] by cwzwarich@webkit.org
  • 18 edits
    3 adds in trunk

2008-08-06 Cameron Zwarich <cwzwarich@webkit.org>

Reviewed by Maciej.

Bug 20286: Load constants all at once instead of using op_load
<https://bugs.webkit.org/show_bug.cgi?id=20286>

Load constants all at once into temporary registers instead of using
individual instances of op_load.

This is a 2.6% speedup on SunSpider.

JavaScriptCore:

  • JavaScriptCore.exp:
  • VM/CodeBlock.cpp: (KJS::CodeBlock::dump): (KJS::CodeBlock::mark):
  • VM/CodeBlock.h:
  • VM/CodeGenerator.cpp: (KJS::CodeGenerator::CodeGenerator): (KJS::CodeGenerator::newTemporary): (KJS::CodeGenerator::addConstant): (KJS::CodeGenerator::addUnexpectedConstant): (KJS::CodeGenerator::emitLoad): (KJS::CodeGenerator::emitUnexpectedLoad): (KJS::CodeGenerator::emitNewError):
  • VM/CodeGenerator.h:
  • VM/Machine.cpp: (KJS::slideRegisterWindowForCall): (KJS::Machine::unwindCallFrame): (KJS::Machine::throwException): (KJS::Machine::execute): (KJS::Machine::privateExecute):
  • VM/Machine.h:
  • VM/Opcode.h:
  • VM/RegisterID.h: (KJS::RegisterID::RegisterID): (KJS::RegisterID::makeConstant): (KJS::RegisterID::isTemporary):
  • kjs/NodeInfo.h:
  • kjs/Parser.cpp: (KJS::Parser::didFinishParsing):
  • kjs/Parser.h: (KJS::Parser::parse):
  • kjs/grammar.y:
  • kjs/nodes.cpp: (KJS::NullNode::emitCode): (KJS::BooleanNode::emitCode): (KJS::NumberNode::emitCode): (KJS::StringNode::emitCode): (KJS::ArrayNode::emitCode): (KJS::DeleteResolveNode::emitCode): (KJS::DeleteValueNode::emitCode): (KJS::VoidNode::emitCode): (KJS::ConstDeclNode::emitCodeSingle): (KJS::ReturnNode::emitCode): (KJS::ScopeNode::ScopeNode): (KJS::ProgramNode::ProgramNode): (KJS::ProgramNode::create): (KJS::EvalNode::EvalNode): (KJS::EvalNode::create): (KJS::FunctionBodyNode::FunctionBodyNode): (KJS::FunctionBodyNode::create): (KJS::FunctionBodyNode::emitCode):
  • kjs/nodes.h: (KJS::ScopeNode::neededConstants):

LayoutTests:

  • fast/js/constant-count-expected.txt: Added.
  • fast/js/constant-count.html: Added.
  • fast/js/deep-recursion-test.html:
  • fast/js/resources/constant-count.js: Added.
3:34 AM Changeset in webkit [35592] by eric@webkit.org
  • 6 edits in trunk/WebCore

Build fix only, no review.

Attempt to fix the Windows and Qt builds.

  • bindings/js/ScriptControllerWin.cpp: add runtime.h include
  • bindings/js/ScriptControllerQt.cpp: add QWidget include
  • page/win/FrameWin.cpp: remove bogus using namespace
  • plugins/PluginView.cpp: (WebCore::PluginView::bindingInstance): use script()
  • plugins/win/PluginViewWin.cpp: (WebCore::PluginView::getValue): use script() (WebCore::PluginView::~PluginView): use script()
3:17 AM Changeset in webkit [35591] by eric@webkit.org
  • 3 edits in trunk/WebCore

Build fix only, no review.

Attempt to fix the Linux and --no-svg builds.

  • bindings/js/ScriptController.h: wrap RetainPtr include in PLATFORM(MAC)
  • css/CSSComputedStyleDeclaration.cpp: (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue): add missing CSSProperties to switch
3:01 AM Changeset in webkit [35590] by eric@webkit.org
  • 35 edits
    3 copies
    1 add in trunk

Reviewed by Cameron Zwarich.

Move more methods from Frame into ScriptController
https://bugs.webkit.org/show_bug.cgi?id=20294

Finally finish more of the work which Darin started long ago
Move a few more Frame methods into ScriptController.
I also took this opportunity to clean up some of the plugin
code as well, and moved some of that into ScriptController.

I removed Frame::clearScriptController() and ScriptController::clear() is now clearWindowShell()

No functional changes, thus no test cases.

  • GNUmakefile.am:
  • WebCore.NPAPI.exp:
  • WebCore.base.exp:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/ScriptController.cpp: (WebCore::ScriptController::ScriptController): (WebCore::ScriptController::~ScriptController): (WebCore::ScriptController::bindingRootObject): (WebCore::ScriptController::createRootObject): (WebCore::ScriptController::windowScriptNPObject): (WebCore::ScriptController::createScriptObjectForPluginElement): New method. (WebCore::ScriptController::clearPlatformScriptObjects): (WebCore::ScriptController::disconnectPlatformScriptObjects): (WebCore::ScriptController::cleanupScriptObjectsForPlugin): (WebCore::ScriptController::clearScriptObjects):
  • bindings/js/ScriptController.h: (WebCore::ScriptController::sourceURL): (WebCore::ScriptController::clearFormerWindow):
  • bindings/js/ScriptControllerGtk.cpp: Copied from WebCore/page/gtk/FrameGtk.cpp. (WebCore::Frame::createScriptInstanceForWidget):
  • bindings/js/ScriptControllerMac.mm: Added. (WebCore::ScriptController::createScriptInstanceForWidget): (WebCore::ScriptController::windowScriptObject): (WebCore::ScriptController::clearPlatformScriptObjects): (WebCore::ScriptController::disconnectPlatformScriptObjects): (WebCore::updateRenderingForBindings): (WebCore::ScriptController::initJavaJSBindings):
  • bindings/js/ScriptControllerQt.cpp: Copied from WebCore/page/gtk/FrameGtk.cpp. (WebCore::ScriptController::createScriptInstanceForWidget):
  • bindings/js/ScriptControllerWin.cpp: Copied from WebCore/svg/graphics/cg/SVGResourceMaskerCg.cpp. (WebCore::ScriptController::createScriptInstanceForWidget):
  • bindings/objc/DOM.mm: (-[DOMNode KJS::Bindings::]):
  • bindings/objc/DOMInternal.mm: (-[WebScriptObject _initializeScriptDOMNodeImp]):
  • bridge/jni/jni_jsobject.mm: (createRootObject):
  • html/HTMLAppletElement.cpp: (WebCore::HTMLAppletElement::renderWidgetForJSBindings):
  • html/HTMLAppletElement.h:
  • html/HTMLEmbedElement.cpp: (WebCore::findWidgetRenderer): (WebCore::HTMLEmbedElement::renderWidgetForJSBindings):
  • html/HTMLEmbedElement.h:
  • html/HTMLObjectElement.cpp: (WebCore::HTMLObjectElement::~HTMLObjectElement): (WebCore::HTMLObjectElement::renderWidgetForJSBindings): (WebCore::HTMLObjectElement::detach):
  • html/HTMLObjectElement.h:
  • html/HTMLPlugInElement.cpp: (WebCore::HTMLPlugInElement::~HTMLPlugInElement): (WebCore::HTMLPlugInElement::detach): (WebCore::HTMLPlugInElement::getInstance): (WebCore::HTMLPlugInElement::parseMappedAttribute): (WebCore::HTMLPlugInElement::getNPObject):
  • html/HTMLPlugInElement.h:
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::clear):
  • page/Frame.cpp: (WebCore::Frame::Frame): (WebCore::Frame::~Frame): (WebCore::Frame::pageDestroyed): (WebCore::FramePrivate::FramePrivate):
  • page/Frame.h:
  • page/FramePrivate.h:
  • page/gtk/FrameGtk.cpp:
  • page/mac/FrameMac.mm:
  • page/qt/FrameQt.cpp:
  • page/win/FrameWin.cpp: (WebCore::computePageRectsForFrame):
2:06 AM Changeset in webkit [35589] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

Reviewed by Cameron Zwarich.

Speculative fix for an error I keep seeing in my Cygwin build

  • Scripts/webkitdirs.pm: make determineConfigurationProductDir() always call determineConfiguration() even on cygwin
12:09 AM Changeset in webkit [35588] by mitz@apple.com
  • 4 edits in trunk/WebKit/mac

Reviewed by Mark Rowe.

  • fix an assertion failure in Cache::setCapacities()
  • Misc/WebKitSystemBits.h: Changed the return type of WebMemorySize() to uint64_t.
  • Misc/WebKitSystemBits.m: (WebMemorySize):
  • WebView/WebView.mm: (+[WebView _setCacheModel:]):

Aug 5, 2008:

9:33 PM Changeset in webkit [35587] by sfalken@apple.com
  • 2 edits in trunk/WebCore

2008-08-05 Steve Falkenburg <sfalken@apple.com>

Fix last timer fix.
Add null check, clear timer using InterlockedExchange.


Reviewed by Alice Liu.

  • platform/win/SharedTimerWin.cpp: (WebCore::clearTimer): (WebCore::queueTimerProc): (WebCore::stopSharedTimer):
9:29 PM Changeset in webkit [35586] by mjs@apple.com
  • 2 edits in trunk/JavaScriptCore

2008-08-05 Maciej Stachowiak <mjs@apple.com>

Reviewed by Cameron.

  • add fast path for immediates to % operator, as we have for many other math ops


This fixes handling for a 0 divisor relative to the last patch. Only an 0.2% speedup on SunSpider but
still a 1.4x win on Oliver's prime test.


  • VM/Machine.cpp: (KJS::Machine::privateExecute):
5:57 PM Changeset in webkit [35585] by sfalken@apple.com
  • 2 edits in trunk/WebCore

2008-08-05 Steve Falkenburg <sfalken@apple.com>

Fix timer leak, flag.


Timers need to be deleted explicitly, even if they fire.
Also added WT_EXECUTEONLYONCE flag, since this is not a repeating timer.


Reviewed by Ada Chan.

  • platform/win/SharedTimerWin.cpp: (WebCore::queueTimerProc):
5:39 PM Changeset in webkit [35584] by cwzwarich@webkit.org
  • 5 edits in trunk

2008-08-05 Cameron Zwarich <cwzwarich@uwaterloo.ca>

Reviewed by Darin.

Bug 20293: Crash in JavaScript codegen for eval("const a;")
<https://bugs.webkit.org/show_bug.cgi?id=20293>

Correctly handle constant declarations in eval code with no initializer.

JavaScriptCore:

  • kjs/nodes.cpp: (KJS::ConstDeclNode::emitCodeSingle):

LayoutTests:

  • fast/js/const-expected.txt:
  • fast/js/resources/const.js:
5:36 PM Changeset in webkit [35583] by andersca@apple.com
  • 2 edits in trunk/WebKit/mac

2008-08-05 Anders Carlsson <andersca@apple.com>

Pass in the correct class here.


  • WebView/WebResource.mm: (-[WebResourcePrivate dealloc]):
5:19 PM Changeset in webkit [35582] by andersca@apple.com
  • 15 edits in trunk

WebCore:

2008-08-05 Anders Carlsson <andersca@apple.com>

Reviewed by Darin.

<rdar://problem/6037398>
ER: Deallocate WebKit objects on the main thread, even if released on secondary thread


Add calls to WebCoreObjCScheduleDeallocateOnMainThread in dealloc methods of objects we expose.


  • bindings/objc/DOMRGBColor.mm: (-[DOMRGBColor dealloc]):
  • bindings/objc/WebScriptObject.mm: (-[WebScriptObject dealloc]):
  • platform/mac/SharedBufferMac.mm: (-[WebCoreSharedBufferData dealloc]):

WebKit/mac:

2008-08-05 Anders Carlsson <andersca@apple.com>

Reviewed by Darin.

<rdar://problem/6037398>
ER: Deallocate WebKit objects on the main thread, even if released on secondary thread


Add calls to WebCoreObjCScheduleDeallocateOnMainThread in dealloc methods of objects we expose.

  • Carbon/CarbonWindowAdapter.m: (-[CarbonWindowAdapter dealloc]):
  • History/WebBackForwardList.mm: (-[WebBackForwardList dealloc]):
  • History/WebHistoryItem.mm: (-[WebHistoryItem dealloc]):
  • Misc/WebElementDictionary.mm: (+[WebElementDictionary initialize]): (-[WebElementDictionary dealloc]):
  • WebCoreSupport/WebEditorClient.mm: (-[WebEditCommand dealloc]):
  • WebCoreSupport/WebFrameLoaderClient.mm:
  • WebView/WebArchive.mm: (-[WebArchivePrivate dealloc]):
  • WebView/WebDataSource.mm: (-[WebDataSourcePrivate dealloc]):
  • WebView/WebHTMLView.mm: (-[WebHTMLViewPrivate dealloc]): (-[WebHTMLView dealloc]):
  • WebView/WebResource.mm: (-[WebResourcePrivate dealloc]):
4:43 PM Changeset in webkit [35581] by andersca@apple.com
  • 9 edits
    2 deletes in trunk

../../../../../../WebCore:

2008-08-05 Anders Carlsson <andersca@apple.com>

Reviewed by Darin.

Move the main thread deallocator here from WebKit.


  • WebCore.base.exp:
  • WebCore.xcodeproj/project.pbxproj:
  • platform/mac/WebCoreObjCExtras.c: (method_getImplementation): (deallocCallback): (WebCoreObjCScheduleDeallocateOnMainThread):
  • platform/mac/WebCoreObjCExtras.h:

../../../../../../WebKit:

2008-08-05 Anders Carlsson <andersca@apple.com>

Reviewed by Darin.

Remove MainThreadObjectDeallocator.{h|mm}.


  • WebKit.xcodeproj/project.pbxproj:

../../../../../../WebKit/mac:

2008-08-05 Anders Carlsson <andersca@apple.com>

Reviewed by Darin.

Remove MainThreadObjectDeallocator.{h|mm}.

  • WebView/MainThreadObjectDeallocator.h: Removed.
  • WebView/MainThreadObjectDeallocator.mm: Removed.


  • WebView/WebView.mm: (-[WebView dealloc]): Call WebCoreObjCScheduleDeallocateOnMainThread instead.
4:01 PM Changeset in webkit [35580] by dino@apple.com
  • 16 edits
    4 adds in trunk/WebCore

Parse CSS Animations, including the new
-webkit-animation properties and the @-webkit-keyframes rule.
Adds some new interfaces for the style rules.

https://bugs.webkit.org/show_bug.cgi?id=20088

Reviewed by David Hyatt.

  • GNUmakefile.am:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:

build config for new files

  • css/tokenizer.flex:
  • css/CSSGrammar.y:
  • css/CSSParser.cpp:
  • css/CSSParser.h:

new @-webkit-keyframes parsing

  • css/CSSPropertyNames.in:

-webkit-animation properties

  • css/CSSRule.h:
  • css/CSSStyleSelector.cpp:
  • css/CSSStyleSelector.h:
  • css/CSSValueKeywords.in:
  • css/StyleBase.h:
  • css/WebKitCSSKeyframeRule.cpp: Added.
  • css/WebKitCSSKeyframeRule.h: Added.
  • css/WebKitCSSKeyframesRule.cpp: Added.
  • css/WebKitCSSKeyframesRule.h: Added.
3:57 PM Changeset in webkit [35579] by andersca@apple.com
  • 4 edits in trunk/WebKit/mac

2008-08-05 Anders Carlsson <andersca@apple.com>

Reviewed by Darin.

Make the main thread object deallocator work with subclasses.


  • WebView/MainThreadObjectDeallocator.h:
  • WebView/MainThreadObjectDeallocator.mm: (deallocCallback): Call the correct dealloc method.


(scheduleDeallocateOnMainThread):
Store both the class and the instance, so we know which dealloc method to call.


  • WebView/WebView.mm: (-[WebViewPrivate dealloc]): Schedule deallocation on the main thread.
3:50 PM Changeset in webkit [35578] by adachan@apple.com
  • 4 edits in trunk/WebKit/win

Add a getter for CFURLRequestRef in WebMutableURLRequest.

Reviewed by Anders Carlsson.

  • Interfaces/IWebMutableURLRequestPrivate.idl:
  • WebMutableURLRequest.cpp: (WebMutableURLRequest::cfRequest):
  • WebMutableURLRequest.h:
3:35 PM Changeset in webkit [35577] by kmccullough@apple.com
  • 2 edits in trunk/WebCore

2008-08-05 Kevin McCullough <kmccullough@apple.com>

Reviewed by Tim.

  • LayoutTests fix. Log only to the Inspector because other listeners may not want XHR notifications, including DRT and the numerous tests that don't expect them.
  • xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::didFinishLoading):
3:34 PM Changeset in webkit [35576] by timothy@apple.com
  • 5 edits in trunk/WebCore

Makes DOM attribute editing start on double-click to match the styles
pane. Links in attributes are now triggered with a single click again.

https://bugs.webkit.org/show_bug.cgi?id=20281

Reviewed by Adam Roben.

  • English.lproj/localizedStrings.js: Removed some strings.
  • page/inspector/ElementsPanel.js: (WebInspector.DOMNodeTreeElement.prototype.onattach): Call _preventFollowingLinksOnDoubleClick. (WebInspector.DOMNodeTreeElement.prototype._preventFollowingLinksOnDoubleClick): Renamed from _makeURLsActivateOnModifiedClick. Simply sets the preventFollowOnDoubleClick property on all links. (WebInspector.DOMNodeTreeElement.prototype.onselect): Removed _selectedByCurrentMouseDown. (WebInspector.DOMNodeTreeElement.prototype.onmousedown): Moved editing code to ondblclick. (WebInspector.DOMNodeTreeElement.prototype.ondblclick): Calls _startEditing. Removed URL check. (WebInspector.DOMNodeTreeElement.prototype._updateTitle): Calls _preventFollowingLinksOnDoubleClick.
  • page/inspector/inspector.css: (.editing): Removed the text-decoration property. (.editing, .editing *): Added the text-decoration property here so it removes underlines on child elements.
  • page/inspector/inspector.js: (WebInspector.documentClick): Removed the followOnAltClick code. Added support for preventFollowOnDoubleClick, which sets a timeout on the first click where future clicks will cancel it (double click). If the timeout isn't canceled, the link is followed.
3:10 PM Changeset in webkit [35575] by Antti Koivisto
  • 3 edits in trunk/WebCore

2008-08-05 Antti Koivisto <Antti Koivisto>

Reviewed by Sam Weinig.

<rdar://problem/6112219> Video in standalone media document flashes when navigating away

When navigating out from a page, MediaPlayer was set invisible and then immediately
made visible again causing flicker in some cases.


The document is not yet marked being in page cache when willSaveToCache() is called.

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::willSaveToCache):
  • rendering/RenderVideo.cpp: (WebCore::RenderVideo::updatePlayer):
3:06 PM Changeset in webkit [35574] by mitz@apple.com
  • 2 edits in trunk/WebCore
  • fix typo
  • page/mac/FrameMac.mm:
3:04 PM Changeset in webkit [35573] by mitz@apple.com
  • 3 edits in trunk/WebKit/mac

Reviewed by Mark Rowe and Anders Carlsson.

  • fix WebMemorySize() reporting a value capped at 2GB
  • misc/WebKitSystemBits.m: (WebMemorySize): Changed to return the max_mem field, which, unlike memory_size, is not capped at 2GB.
  • WebView/WebView.mm: (+[WebView _setCacheModel:]): Made the cache sizes for over 2GB RAM the same as for 2GB, so that behavior on machines that have more than 2GB RAM is not affected by the fix to WebMemorySize().
3:03 PM Changeset in webkit [35572] by mitz@apple.com
  • 2 edits in trunk/WebCore
  • try to fix the 64-bit build
  • page/mac/FrameMac.mm:
2:41 PM Changeset in webkit [35571] by alp@webkit.org
  • 2 edits in trunk/WebCore

2008-08-05 Alp Toker <alp@nuanti.com>

Remove leftover mark pragma from r35568 to fix compiler warning spew.

  • rendering/style/RenderStyle.h:
2:33 PM Changeset in webkit [35570] by alp@webkit.org
  • 3 edits in trunk

2008-08-05 Marco Barisione <marco.barisione@collabora.co.uk>

Reviewed by Alp Toker.

http://bugs.webkit.org/show_bug.cgi?id=19045
[gtk] ./configure doesn't check for x toolkit availability

Check for the .pc file for Xt instead of hard coding -lXt in the
GNUmakefile.am.

Thanks to Dominik Röttsches for the initial patch.

  • GNUmakefile.am:
  • configure.ac:
12:15 PM Changeset in webkit [35569] by kmccullough@apple.com
  • 1 edit
    1 add in trunk/WebCore

2008-08-05 Kevin McCullough <kmccullough@apple.com>

  • Added a test that checks for several edge cases (see description in test.
  • manual-tests/inspector/profiler-test-start-but-dont-stop-profiling.html: Added.
12:08 PM Changeset in webkit [35568] by dino@apple.com
  • 6 edits in trunk/WebCore

Reviewed by Dave Hyatt

Add support for CSS Animation properties to RenderStyle
https://bugs.webkit.org/show_bug.cgi?id=20068

(also fix some minor whitespace issues and remove debugging code)

  • css/CSSComputedStyleDeclaration.cpp:
  • css/CSSStyleSelector.cpp:
  • page/AnimationController.cpp:
  • rendering/style/RenderStyle.cpp:
  • rendering/style/RenderStyle.h:
12:07 PM Changeset in webkit [35567] by kmccullough@apple.com
  • 2 edits in trunk/WebCore

2008-08-05 Kevin McCullough <kmccullough@apple.com>

  • Updated a manual test to be an accurate description of what it is actually testing.
  • manual-tests/inspector/profiler-test-stop-profiling-after-setTimeout.html:
12:06 PM Changeset in webkit [35566] by sfalken@apple.com
  • 2 edits in trunk/WebKit/win

2008-08-05 Steve Falkenburg <sfalken@apple.com>

Don't invalidate a WebView in the process of being closed when prefs change.


Reviewed by Ada Chan.

  • WebView.cpp: (WebView::notifyPreferencesChanged):
11:00 AM Changeset in webkit [35565] by kmccullough@apple.com
  • 2 edits in trunk/WebCore

2008-08-05 Kevin McCullough <kmccullough@apple.com>

Reviewed by Tim.

<rdar://problem/5732836> XMLHttpRequest: Inspector should show network
activity/XHR in Console (17233)

  • xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::didFinishLoading):
10:48 AM Changeset in webkit [35564] by dino@apple.com
  • 1 edit in trunk/WebCore/ChangeLog

Fix the ChangeLog that was corrupted by r34545.

9:47 AM Changeset in webkit [35563] by timothy@apple.com
  • 2 edits in trunk/WebCore

Makes the selection surround just the name or value that was
double clicked when editing CSS properties. This regressed when
the Inspector started using focus/blur events, since a focus on
a content editable region changes the selection.

https://bugs.webkit.org/show_bug.cgi?id=20280

Reviewed by Adam Roben.

  • page/inspector/StylesSidebarPane.js: (WebInspector.StylePropertyTreeElement.prototype.startEditing): Change the selection after calling WebInspector.startEditing, since WebInspector.startEditing will set an initial selection due to a focus event.
9:47 AM Changeset in webkit [35562] by timothy@apple.com
  • 2 edits in trunk/WebCore

Changes the editing behavior in the Inspector to commit changes
when focus is lost, instead of reverting the changes.

https://bugs.webkit.org/show_bug.cgi?id=20279

Reviewed by Adam Roben.

  • page/inspector/inspector.js: (WebInspector.startEditing): Call editingCommitted in blurEventListener instead of editingCancelled.
9:47 AM Changeset in webkit [35561] by timothy@apple.com
  • 4 edits in trunk/WebCore

Adds the ability to increment/decrement numbers in editable style
properties with the arrow and page keys.

Reviewed by Adam Roben.

https://bugs.webkit.org/show_bug.cgi?id=20273

  • page/inspector/StylesSidebarPane.js: (WebInspector.StylePropertyTreeElement.prototype.updateAll): Updates all properties in the current section, or all sections depending in the argument passed. (WebInspector.StylePropertyTreeElement.prototype.toggleEnabled): Call updateAll instead of having duplicate code. (WebInspector.StylePropertyTreeElement.prototype.startEditing): Assign the list element's handleKeyEvent to editingKeyDown. (WebInspector.StylePropertyTreeElement.prototype.editingKeyDown): Handles the Up and Down arrows keys. Finds the word based on the selection and extracts the number, prefix and suffix. Increments or decrements the number based on the key and modifiers. Replaces the original word with the new value and selects it. Also applies the style so the page reflects the changes live. (WebInspector.StylePropertyTreeElement.prototype.editingEnded): Delete the handleKeyEvent and originalCSSText properties. (WebInspector.StylePropertyTreeElement.prototype.editingCancelled): If originalCSSText exists, set the cssText to that value and call updateAll. Otherwise just call updateTitle like before. (WebInspector.StylePropertyTreeElement.prototype.editingCommitted): Factored out the style applying code to applyStyleText. (WebInspector.StylePropertyTreeElement.prototype.applyStyleText): Factored out from editingCommitted. Takes an argument that specifies if the interface should be updated when the style is applied.
  • page/inspector/inspector.js: (WebInspector.startEditing): Call the original key handler, if one existed before interrupting key events.
  • page/inspector/utilities.js: (getStyleTextWithShorthands): Helper to return CSS text that preserves shorthand values, since the cssText property only outputs longhands.
9:46 AM Changeset in webkit [35560] by timothy@apple.com
  • 4 edits in trunk/WebCore

Factors out the TextPrompt.scanBackwards function into a generic
helper function on the Node prototype, named rangeOfWord.

Reviewed by Adam Roben.

  • page/inspector/Console.js: (WebInspector.Console.prototype.completions): Use rangeOfWord.
  • page/inspector/TextPrompt.js: (WebInspector.TextPrompt.prototype.complete): Ditto. (WebInspector.TextPrompt.prototype.scanBackwards): Removed.
  • page/inspector/utilities.js: (Node.prototype.rangeOfWord): Added. Copied from TextPrompt.scanBackwards, and added a direction argument allowing scanning forward, backward or both directions from the offset.
9:46 AM Changeset in webkit [35559] by timothy@apple.com
  • 2 edits in trunk/WebCore

Fixes a bug where the property toggle button would not be hidden
during editing.

Reviewed by Adam Roben.

  • page/inspector/inspector.css: (li.editing .swatch, li.editing .enabled-button): Make the display !important, since a more specific rule for .enabledbutton overrides display.
6:40 AM Changeset in webkit [35558] by vestbo@webkit.org
  • 4 edits in trunk/WebKit/qt

2008-08-05 Tor Arne Vestbø <tavestbo@trolltech.com>

Reviewed by Simon.

Move event handling of the return-key from EditorClientQt to QWebPage.

https://bugs.webkit.org/show_bug.cgi?id=20191

This is a first step in refactoring the big switch block
in EditorClientQt::handleKeyboardEvent to using WebActions
instead.

The new logic uses two new StandardKeys from QKeySequence:

  • InsertParagraphSeparator
  • InsertLineSeparator

Which translate to the commands InsertNewline and InsertLineBreak
respectivly. On Windows/X11 pressing the shift modifier will invoke
the latter action. For Mac this is triggered by pressing the meta
modifier (Ctrl).

Initial patch by: Erik Bunce

5:51 AM S60CheckoutAndBuild edited by kvmayank@gmail.com
(diff)
4:00 AM Changeset in webkit [35557] by cwzwarich@webkit.org
  • 3 edits
    3 adds in trunk

2008-08-05 Cameron Zwarich <cwzwarich@uwaterloo.ca>

Reviewed by Oliver.

Roll out r35555 because of correctness issues.

JavaScriptCore:

  • VM/Machine.cpp: (KJS::Machine::privateExecute):

LayoutTests:

  • fast/js/mod-crash-expected.txt: Added.
  • fast/js/mod-crash.html: Added.
  • fast/js/resources/mod-crash.js: Added.
12:40 AM Changeset in webkit [35556] by mitz@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Geoffrey Garen.

  • remove redundant #imports from FrameMac.mm
  • page/mac/FrameMac.mm:
12:15 AM Changeset in webkit [35555] by mjs@apple.com
  • 2 edits in trunk/JavaScriptCore

2008-08-05 Maciej Stachowiak <mjs@apple.com>

Reviewed by Geoff.


  • add fast path for immediates to % operator, as we have for many other math ops


0.6% speedup on SunSpider. 1.4x speedup on a prime testing torture test that Oliver whipped up.

  • VM/Machine.cpp: (KJS::Machine::privateExecute):

Aug 4, 2008:

11:03 PM Changeset in webkit [35554] by mrowe@apple.com
  • 4 edits in tags/Safari-6528.0.2

Versioning.

11:03 PM Changeset in webkit [35553] by mrowe@apple.com
  • 7 edits
    1 add in tags/Safari-6528.0.2/WebKit

Merge r35546 and r35547.

10:13 PM Changeset in webkit [35552] by alice.liu@apple.com
  • 2 edits in trunk/WebKit/win

2008-08-04 Alice Liu <alice.liu@apple.com>

Reviewed by Steve Falkenburg.

Fix CFDictionary leak

  • CFDictionaryPropertyBag.cpp: (CFDictionaryPropertyBag::Write): m_dictionary is a RetainPtr and should adopt the CF type created
9:09 PM Changeset in webkit [35551] by mitz@apple.com
  • 5 edits in trunk

WebCore:

2008-08-04 Bernhard Kruepl <kruepl@dbai.tuwien.ac.at>

Reviewed by Dave Hyatt.

Fix for https://bugs.webkit.org/show_bug.cgi?id=19094
Corrected offsetTop for table cells by subtracting borderTopExtra

  • rendering/RenderObject.cpp: (WebCore::RenderObject::offsetTop):

LayoutTests:

2008-08-04 Bernhard Kruepl <kruepl@dbai.tuwien.ac.at>

Reviewed by Dave Hyatt.

Fix for https://bugs.webkit.org/show_bug.cgi?id=19094
Tests for offsetTop of table cells updated so that they don't include borderExtraTop

  • fast/dom/Element/offsetTop-table-cell-expected.txt:
  • fast/dom/Element/offsetTop-table-cell.html:
8:58 PM Changeset in webkit [35550] by weinig@apple.com
  • 3 edits in trunk/LayoutTests

2008-08-04 Sam Weinig <sam@webkit.org>

Rubber-stamped by Dan Bernstein.

Fix typo in Skipped lists.

  • platform/mac-tiger/Skipped:
  • platform/win/Skipped:
8:19 PM Changeset in webkit [35549] by mrowe@apple.com
  • 1 copy in tags/Safari-6528.0.2

New tag.

8:16 PM Changeset in webkit [35548] by mrowe@apple.com
  • 2 edits in trunk/WebKit/mac

Build fix.

7:02 PM Changeset in webkit [35547] by mrowe@apple.com
  • 4 edits in trunk/WebKit/mac

Adopt the formal protocols where necessary.
Final part of fix for <rdar://problem/5853147>.

Reviewed by Kevin Decker.

7:02 PM Changeset in webkit [35546] by mrowe@apple.com
  • 4 edits
    1 add in trunk/WebKit

Declare empty protocols when using versions of AppKit that do not use formal protocols for delegates and data sources.
Part one of fix for <rdar://problem/5853147>.

Reviewed by Darin Adler.

6:54 PM Changeset in webkit [35545] by dino@apple.com
  • 28 edits
    3 adds in trunk

2008-08-04 Dean Jackson <dino@apple.com>

Reviewed by Dave Hyatt.

Improve AnimationController
https://bugs.webkit.org/show_bug.cgi?id=19938

  • Transition class is now called Animation
  • new state-based AnimationController that can support CSS Animations
  • add support for -webkit-transition-delay
  • remove -webkit-transition-repeat-count (since it never existed officially)
  • updates the -webkit-transition shorthand to reflect removing repeat count
  • updates the Transition class so that properties can be shared with animations
  • adds a "now" keyword for -webkit-transition-delay
  • adds a new change type for style (changed by animation)
  • adds new event names (although they are not dispatched yet)
  • makes text stroke and text fill colors returned by RenderStyle const

Tests:

  • manual-tests/transition-left.html
  • manual-tests/transition-delay.html
  • manual-tests/transition-timing-functions.html
  • css/CSSComputedStyleDeclaration.cpp:
  • css/CSSParser.cpp:
  • css/CSSParser.h:
  • css/CSSPropertyNames.in:
  • css/CSSStyleSelector.cpp:
  • css/CSSStyleSelector.h:
  • css/CSSTimingFunctionValue.h:
  • css/CSSValue.h:
  • css/CSSValueKeywords.in:
  • css/WebKitCSSTransformValue.cpp:
  • dom/Document.cpp:
  • dom/Document.h:
  • dom/Element.cpp:
  • dom/EventNames.h:
  • dom/Node.cpp:
  • dom/Node.h:
  • history/CachedPage.cpp:
  • page/AnimationController.cpp:
  • page/AnimationController.h:
  • page/Frame.cpp:
  • rendering/RenderObject.cpp:
  • rendering/RenderWidget.cpp:
  • rendering/style/RenderStyle.cpp:
  • rendering/style/RenderStyle.h:
5:39 PM Changeset in webkit [35544] by eric@webkit.org
  • 4 edits in trunk/WebCore

Reviewed by Antti.

Currently we create two copies of script in the HTMLTokenizer.
One copy gets passed into the javascript engine, the other gets
stored in the DOM. Modify the HTMLParser so that it does not
chunk the string into it's normal 64K chunks for script code,
and modify the HTMLTokenizer to pass that same string into the
JS engine. On some sites (e.g. GMail), which have hundreds of
KB of inline JS, this saves more than a MB of RAM to run the page.
(Strings use 16bit strings, so 600KB of JS code == 1.2MB of memory)

  • html\HTMLParser.cpp
  • html\HTMLTokenizer.cpp
4:01 PM Changeset in webkit [35543] by eric@webkit.org
  • 2 edits in trunk/WebCore

Reviewed by Adam Roben.

Fix warning seen in MSVC converting ints to floats.

  • rendering/Length.h: (WebCore::Length::calcFloatValue):
2:24 PM Changeset in webkit [35542] by weinig@apple.com
  • 2 edits in trunk/WebCore

2008-08-04 Sam Weinig <sam@webkit.org>

Reviewed by Mark Rowe.

Remove extra braces left in from JSLock removal.

  • xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::dropProtection):
1:29 PM Changeset in webkit [35541] by weinig@apple.com
  • 3 edits in trunk/LayoutTests

2008-08-04 Sam Weinig <sam@webkit.org>

Reviewed by Dan Bernstein.

Update skipped lists for renamed tests.

  • platform/mac-tiger/Skipped:
  • platform/win/Skipped:
1:21 PM Changeset in webkit [35540] by weinig@apple.com
  • 6 edits
    14 adds in trunk

WebCore:

2008-08-02 Sam Weinig <sam@webkit.org>

Reviewed by Dan Bernstein.

Implement the Access-control for Cross Site requests
preflight cache.

Tests: http/tests/xmlhttprequest/access-control-basic-allow-preflight-cache-invalidation-by-header.html

http/tests/xmlhttprequest/access-control-basic-allow-preflight-cache-invalidation-by-method.html
http/tests/xmlhttprequest/access-control-basic-allow-preflight-cache-timeout.html
http/tests/xmlhttprequest/access-control-basic-allow-preflight-cache.html

  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • platform/KURL.h: (WTF::):
  • platform/KURLHash.h: Added. (WebCore::KURLHash::hash): (WebCore::KURLHash::equal): (WTF::):
  • xml/XMLHttpRequest.cpp: (WebCore::PreflightResultCacheItem::PreflightResultCacheItem): (WebCore::preflightResultCache): (WebCore::appendPreflightResultCacheEntry): (WebCore::canSkipPrelight): (WebCore::XMLHttpRequest::makeCrossSiteAccessRequestWithPreflight): (WebCore::parseAccessControlAllowList): (WebCore::parseAccessControlMaxAge): (WebCore::XMLHttpRequest::didReceiveResponsePreflight):

LayoutTests:

2008-08-04 Sam Weinig <sam@webkit.org>

Reviewed by Dan Bernstein.

Tests for the Access-control for Cross Site requests
preflight cache.

  • ChangeLog:
  • http/tests/xmlhttprequest/access-control-basic-allow-preflight-cache-expected.txt: Added.
  • http/tests/xmlhttprequest/access-control-basic-allow-preflight-cache-invalidation-by-header-expected.txt: Added.
  • http/tests/xmlhttprequest/access-control-basic-allow-preflight-cache-invalidation-by-header.html: Added.
  • http/tests/xmlhttprequest/access-control-basic-allow-preflight-cache-invalidation-by-method-expected.txt: Added.
  • http/tests/xmlhttprequest/access-control-basic-allow-preflight-cache-invalidation-by-method.html: Added.
  • http/tests/xmlhttprequest/access-control-basic-allow-preflight-cache-timeout-expected.txt: Added.
  • http/tests/xmlhttprequest/access-control-basic-allow-preflight-cache-timeout.html: Added.
  • http/tests/xmlhttprequest/access-control-basic-allow-preflight-cache.html: Added.
  • http/tests/xmlhttprequest/resources/access-control-basic-preflight-cache-invalidation.php: Added.
  • http/tests/xmlhttprequest/resources/access-control-basic-preflight-cache-timeout.php: Added.
  • http/tests/xmlhttprequest/resources/access-control-basic-preflight-cache.php: Added.
  • http/tests/xmlhttprequest/resources/portabilityLayer.php: Added.
  • http/tests/xmlhttprequest/resources/reset-temp-file.php: Added.
12:59 PM Changeset in webkit [35539] by eric@webkit.org
  • 2 edits in trunk/WebCore

Reviewed by hyatt.

Clean up Gradient constructors, make sure all members are initialized.

  • platform/graphics/Gradient.cpp: (WebCore::Gradient::Gradient):
12:53 PM Changeset in webkit [35538] by beidson@apple.com
  • 2 edits in trunk/WebKit/mac

2008-08-04 Brady Eidson <beidson@apple.com>

Reviewed by Mitz Pettel

Fix <rdar://problem/5820157> - Saving WebArchives of Mail attachments broken.

This broke in r31355 when we stopped returning nil WebResources when there was nil
resource data.

  • WebView/WebResource.mm: (-[WebResource _initWithCoreResource:]): Restore previous behavior of returning nil when the resource data is null.
9:02 AM Changeset in webkit [35537] by Simon Hausmann
  • 2 edits in trunk/WebKit/qt

2008-08-04 Erik Bunce <elbunce@thehive.com>

Reviewed by Simon.

https://bugs.webkit.org/show_bug.cgi?id=20221

Add updateAction() support to ToggleBold, ToggleItalic, and ToggleUnderline.
Add lookup table for mapping web actions to editor commands.

7:31 AM Changeset in webkit [35536] by Simon Hausmann
  • 2 edits in trunk/WebKit/qt

2008-08-03 Ariya Hidayat <ariya.hidayat@trolltech.com>

Reviewed by Simon.

A bunch of improvements to the QtLauncher.

Among others:

  • Unify and unclutter the toolbar, use menu for addition actions
  • Simplify URL edit, just use QLineEdit (no fancy close button etc)
  • Guess the URL from the command line so now we can run ./QtLauncher www.google.com
  • Shortcut keys for most actions
  • Simple autocomplete for the URL edit
  • Actions for zooming, New Window and Close Window
  • Show Format menu only when the content is set to editable
7:03 AM Changeset in webkit [35535] by Simon Hausmann
  • 2 edits in trunk/WebKit/qt

2008-08-04 Erik Bunce <elbunce@thehive.com>

Reviewed by Simon.

https://bugs.webkit.org/show_bug.cgi?id=20198

Allow Copy key sequence to work in non-editable areas.

Aug 3, 2008:

7:34 PM Changeset in webkit [35534] by ddkilzer@apple.com
  • 2 edits in trunk/LayoutTests

Future-proof fast/cookies/local-file-can-set-cookies.html

Reviewed by Anders Carlsson.

  • fast/cookies/local-file-can-set-cookies.html: Set cookie expiration date 60 seconds into the future instead of a hard-coded date.
2:58 AM Changeset in webkit [35533] by oliver@apple.com
  • 10 edits
    3 adds in trunk

Bug 19359: JavaScriptCore behaves differently from FF2/3 and IE when handling context in catch statement
<https://bugs.webkit.org/show_bug.cgi?id=19359>

Reviewed by Cameron Zwarich

Make our catch behave like Firefox and IE, we do this by using a StaticScopeObject
instead of a generic JSObject for the scope node. We still don't make use of the
fact that we have a static scope inside the catch block, so the internal performance
of the catch block is not improved, even though technically it would be possible to
do so.

1:34 AM Changeset in webkit [35532] by britto@apple.com
  • 13 edits
    8 adds in trunk

2008-08-02 Maxime Britto <britto@apple.com>

Reviewed by Eric Seidel.

<rdar://problem/6118969>
https://bugs.webkit.org/show_bug.cgi?id=19971
Modifiy the cursor interactively when on panning mode (middle click auto scroll)
When the panning is in progress the cursor is an arrow representing the direction of the current scroll

  • page/EventHandler.cpp: (WebCore::EventHandler::autoscrollTimerFired): Before each scroll we actualize the cursor to match the current direction (WebCore::EventHandler::setPanScrollCursor): determines the direction of the upcoming scroll. (WebCore::EventHandler::handleMouseMoveEvent):
  • page/EventHandler.h:
  • platform/Cursor.h: Add propotypes for the new cursor functions
  • platform/gtk/CursorGtk.cpp: to avoid build break (WebCore::middlePanningCursor): (WebCore::eastPanningCursor): (WebCore::northPanningCursor): (WebCore::northEastPanningCursor): (WebCore::northWestPanningCursor): (WebCore::southPanningCursor): (WebCore::southEastPanningCursor): (WebCore::southWestPanningCursor): (WebCore::westPanningCursor):
  • platform/mac/CursorMac.mm: to avoid build break (WebCore::middlePanningCursor): (WebCore::eastPanningCursor): (WebCore::northPanningCursor): (WebCore::northEastPanningCursor): (WebCore::northWestPanningCursor): (WebCore::southPanningCursor): (WebCore::southEastPanningCursor): (WebCore::southWestPanningCursor): (WebCore::westPanningCursor):
  • platform/qt/CursorQt.cpp: to avoid build break (WebCore::middlePanningCursor): (WebCore::eastPanningCursor): (WebCore::northPanningCursor): (WebCore::northEastPanningCursor): (WebCore::northWestPanningCursor): (WebCore::southPanningCursor): (WebCore::southEastPanningCursor): (WebCore::southWestPanningCursor): (WebCore::westPanningCursor):
  • platform/win/CursorWin.cpp: Add specific icons for the panning as windows resizing icons are both sides arrows (WebCore::middlePanningCursor): (WebCore::eastPanningCursor): (WebCore::northPanningCursor): (WebCore::northEastPanningCursor): (WebCore::northWestPanningCursor): (WebCore::southPanningCursor): (WebCore::southEastPanningCursor): (WebCore::southWestPanningCursor): (WebCore::westPanningCursor):
  • platform/wx/CursorWx.cpp: to avoid build break (WebCore::middlePanningCursor): (WebCore::eastPanningCursor): (WebCore::northPanningCursor): (WebCore::northEastPanningCursor): (WebCore::northWestPanningCursor): (WebCore::southPanningCursor): (WebCore::southEastPanningCursor): (WebCore::southWestPanningCursor): (WebCore::westPanningCursor):

2008-08-02 Maxime Britto <britto@apple.com>

Reviewed by Eric Seidel.


These icons are copies from the resizers in WebCore/Ressources

  • ChangeLog:
  • WebKit.vcproj/WebKit.rc:
  • WebKit.vcproj/panEastCursor.png: Added.
  • WebKit.vcproj/panNorthCursor.png: Added.
  • WebKit.vcproj/panNorthEastCursor.png: Added.
  • WebKit.vcproj/panNorthWestCursor.png: Added.
  • WebKit.vcproj/panSouthCursor.png: Added.
  • WebKit.vcproj/panSouthEastCursor.png: Added.
  • WebKit.vcproj/panSouthWestCursor.png: Added.
  • WebKit.vcproj/panWestCursor.png: Added.
  • WebKit.vcproj/resource.h:
  • WebKitDLL.cpp: (loadResourceIntoBuffer):
Note: See TracTimeline for information about the timeline view.