Timeline



Mar 22, 2009:

11:01 PM Changeset in webkit [41905] by oliver@apple.com
  • 4 edits in trunk/JavaScriptCore

Fix exception handling in API

Reviewed by Cameron Zwarich.

We can't just use the ExecState exception slot for returning exceptions
from class introspection functions provided through the API as many JSC
functions will explicitly clear the ExecState exception when returning.

9:31 PM Changeset in webkit [41904] by oliver@apple.com
  • 1 edit in trunk/JavaScriptCore/jit/JITStubs.cpp

Fix instanceof implementation to handle API object overriding hasInstance

Reviewed by Cameron Zwarich.

We can only take the fast paths for instanceof if the object doesn't override
hasInstance, so we have to check for that.

8:46 PM Changeset in webkit [41903] by mitz@apple.com
  • 2 edits in trunk/WebKit/mac

Reviewed by John Sullivan.

  • fix <rdar://problem/6640741> Messages not displaying after the Safari 4 beta was installed

Mail assumes that if -[WebArchive subresources] is not nil, then it
contains at least one object.

  • WebView/WebArchive.mm: (-[WebArchive subresources]): Preserve the behavior of returning nil if there are no subresources.
2:12 PM Companies and Organizations that have contributed to WebKit edited by alp@atoker.com
Add Nuanti (diff)
12:50 PM Changeset in webkit [41902] by Simon Fraser
  • 2 edits in trunk/WebCore

2009-03-22 Simon Fraser <Simon Fraser>

Reviewed by Dan Bernstein

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

Image-map code in RenderImage could result in RenderImage::nodeAtPoint()
setting HitTestResult::innerNode(), but returning false, which violates
hit testing rules. Use a temporary HitTestResult so that we only fill in
result when we know we've hit.

  • rendering/RenderImage.cpp: (WebCore::RenderImage::nodeAtPoint):
12:45 PM Changeset in webkit [41901] by kevino@webkit.org
  • 2 edits in trunk/WebCore

wxGTK build fix. Add missing header.

11:43 AM Changeset in webkit [41900] by Simon Fraser
  • 3 edits
    2 adds in trunk

2009-03-22 Simon Fraser <Simon Fraser>

Reviewed by Dan Bernstein

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

Fix hit testing regression from r41840. We need to pass the temporary
HitTestResult when testing sublayers, then only copy to 'result' when
the layer is known to have been hit.

Test: fast/layers/zindex-hit-test.html

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::hitTestLayer):
10:35 AM Changeset in webkit [41899] by Simon Fraser
  • 11 edits in trunk/WebCore

2009-03-20 Simon Fraser <Simon Fraser>

Reviewed by Darin Adler

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

Fix hit testing on video controls after full page zoom by fixing wider issue
with event->pageX(), pageY() with zooming. pageX and pageY are "fixed" to be
invariant under zooming (for JavaScript), so we keep an actual page point around
in MouseEvent::absoluteLocation() to avoid the need to factor in zooming everywhere.

  • dom/MouseRelatedEvent.cpp: (WebCore::MouseRelatedEvent::initCoordinates): (WebCore::MouseRelatedEvent::computePageLocation):
  • dom/MouseRelatedEvent.h: (WebCore::MouseRelatedEvent::absoluteLocation): (WebCore::MouseRelatedEvent::setAbsoluteLocation): Member var, and getter and setter for absoluteLocation. New method, computePageLocation(), to compute the actual page point, and call it when creating and initting mouse-related events.
  • dom/Node.cpp: (WebCore::Node::dispatchMouseEvent): (WebCore::Node::dispatchWheelEvent): Keep non-adjusted pageX and pageY around, and call setAbsoluteLocation() on the event to replace a potentially rounded point.
  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::defaultEventHandler): Clean up slider handling code.
  • html/HTMLSelectElement.cpp: (WebCore::HTMLSelectElement::listBoxDefaultEventHandler): Add FIXME comment for use of offsetX/offsetY.
  • page/ContextMenuController.cpp: (WebCore::ContextMenuController::handleContextMenuEvent): Use absoluteLocation() when hit testing for context menus.
  • rendering/RenderFrameSet.cpp: (WebCore::RenderFrameSet::userResize): Use absoluteLocation() when resizing frames.
  • rendering/RenderMedia.cpp: (WebCore::RenderMedia::forwardEvent): Use absoluteLocation() when hit testing media controls.
  • rendering/RenderSlider.cpp: (WebCore::HTMLSliderThumbElement::defaultEventHandler): (WebCore::RenderSlider::mouseEventIsInThumb): Use absoluteLocation() when handling slider events.

(WebCore::RenderSlider::forwardEvent):
Factor some code out of HTMLInputElement::defaultEventHandler().

  • rendering/RenderTextControlSingleLine.cpp: (WebCore::RenderTextControlSingleLine::forwardEvent): Use absoluteLocation() when hit testing search field buttons, which fixees bugs in the search field with zooming.
12:34 AM Changeset in webkit [41898] by levin@chromium.org
  • 1 edit
    5 adds in trunk/WebCore

2009-03-21 David Levin <levin@chromium.org>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=24727
Add V8XMLHttpRequest*.

  • bindings/v8/V8XMLHttpRequestUtilities.cpp: Added.
  • bindings/v8/V8XMLHttpRequestUtilities.h: Added.
  • bindings/v8/custom/V8XMLHttpRequestConstructor.cpp: Added.
  • bindings/v8/custom/V8XMLHttpRequestCustom.cpp: Added.
  • bindings/v8/custom/V8XMLHttpRequestUploadCustom.cpp: Added.
12:34 AM Changeset in webkit [41897] by levin@chromium.org
  • 1 edit
    2 adds in trunk/WebCore

2009-03-21 David Levin <levin@chromium.org>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=24725
Add V8NodeFilter.

  • bindings/v8/V8NodeFilter.cpp: Added.
  • bindings/v8/V8NodeFilter.h: Added.

Mar 21, 2009:

8:01 PM Changeset in webkit [41896] by cwzwarich@webkit.org
  • 2 edits in trunk/JavaScriptCore

2009-03-21 Cameron Zwarich <cwzwarich@uwaterloo.ca>

Reviewed by Oliver Hunt.

Bug 20049: testapi failure: MyObject - 0 should be NaN but instead is 1.
<https://bugs.webkit.org/show_bug.cgi?id=20049>
<rdar://problem/6079127>

In this case, the test is wrong. According to the ECMA spec, subtraction
uses ToNumber, not ToPrimitive. Change the test to match the spec.

  • API/tests/testapi.js:
7:52 PM Changeset in webkit [41895] by oliver@apple.com
  • 3 edits in trunk/JavaScriptCore

Ensure that JSObjectMakeFunction doesn't produce incorrect line numbers.

Reviewed by Cameron Zwarich.

Also make test api correctly propagate failures.

6:40 PM Changeset in webkit [41894] by oliver@apple.com
  • 2 edits in trunk/WebKitTools

Make build-jsc build testapi and minidom in addition to jsc itself.

Reviewed by Mark Rowe.

6:40 PM Changeset in webkit [41893] by oliver@apple.com
  • 3 edits in trunk/JavaScriptCore

Improve testapi by making it report failures in a way we can pick up
from our test scripts.

Reviewed by Mark Rowe.

11:34 AM Changeset in webkit [41892] by mitz@apple.com
  • 2 edits in trunk/WebCore

Rubber-stamped by Kevin Decker

  • remove some redundant #include statements
  • bindings/js/JSDOMWindowBase.cpp:
11:15 AM Changeset in webkit [41891] by xan@webkit.org
  • 2 edits in trunk/JavaScriptCore

Revert "2009-03-20 Mike Hommey <glandium@debian.org>"

This reverts commit 2ee472cbd180a196c08f3386b2b976d540a1d9e2.

Breaks JIT on x86 (at least linux) massively.

11:15 AM Changeset in webkit [41890] by xan@webkit.org
  • 4 edits in trunk

Revert "2009-03-20 Mike Hommey <glandium@debian.org>"

This reverts commit cb79d9c35d714a9e463a188a5cb295033715308e.

Breaks JIT on x86 (at least linux) massively.

Mar 20, 2009:

11:02 PM Changeset in webkit [41889] by mitz@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Mark Rowe.

  • fix <rdar://problem/6574185> REGRESSION (3.2.2-TOT): hang in text drawing code
  • platform/graphics/win/FontCGWin.cpp: (WebCore::drawGDIGlyphs): Changed glyph stroking to fill and stroke each glyph as a separate path, instead of all glyphs as a single path. This matches what CGContextShowGlyphsWithAdvances() does, and has comparable performance.
6:47 PM Changeset in webkit [41888] by dino@apple.com
  • 2 edits in trunk/WebCore

2009-03-20 Dean Jackson <dino@apple.com>

Reviewed by Simon Fraser

Build fix for ENABLE(3D_RENDERING)

  • rendering/RenderObject.h: (WebCore::makeMatrixRenderable):
6:29 PM Changeset in webkit [41887] by jmalonzo@webkit.org
  • 4 edits in trunk

2009-03-20 Mike Hommey <glandium@debian.org>

Reviewed by Gavin Barraclough. Landed by Jan Alonzo.

Enable JIT on x86-64
https://bugs.webkit.org/show_bug.cgi?id=24724

Enable JIT on x86-64 gtk+

6:18 PM Changeset in webkit [41886] by jmalonzo@webkit.org
  • 2 edits in trunk/JavaScriptCore

2009-03-20 Mike Hommey <glandium@debian.org>

Reviewed by Mark Rowe. Committed by Jan Alonzo.

Use the proper definition of RETURN_PAIR on !MSVC

The other version is actually a work around for a quirk in the windows
ABI, so checking whether we're under MSVC is more a appropriate check for
it. It also happens this workaround is not 64-bits safe, so we also error
out on x86-64 MSVC.

Thanks Gavin Barraclough.

  • jit/JITStubs.h:
5:57 PM Changeset in webkit [41885] by adele@apple.com
  • 2 edits in trunk/WebKit/mac

2009-03-20 Adele Peterson <adele@apple.com>

Build fix.

Reviewed by Darin Adler.

  • WebView/WebFrameView.mm: (-[WebFrameView initWithFrame:]):
5:15 PM Changeset in webkit [41884] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

2009-03-20 Geoffrey Garen <ggaren@apple.com>

Reviewed by Darin Adler.


Fixed a JavaScriptCore crash on the Windows buildbot.

  • bytecompiler/BytecodeGenerator.h: Reduced the AST recursion limit. Apparently, Windows has small stacks.
4:46 PM Changeset in webkit [41883] by levin@chromium.org
  • 2 edits
    1 add in trunk/LayoutTests

2009-03-20 Tony Chang <tony@chromium.org>

Reviewed by Alexey Proskuryakov.

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

Change the layout test from using a data: URL to using a file:///
URL for the iframe. If DRT's frame loader has
AllowLocalLoadsForLocalOnly, then it's not possible to load an image
from disk into the iframe. Since this test is about D&D, not
local load checks, switch to a local file that doesn't have this
problem.

This should fix the test on win DRT as well.

  • fast/events/resources/standalone-image-drag-to-editable-frame.html: Added.
  • fast/events/standalone-image-drag-to-editable.html:
4:46 PM Changeset in webkit [41882] by levin@chromium.org
  • 2 edits in trunk/WebCore

2009-03-20 Dave Moore <davemoore@google.com>

Reviewed by Dimitri Glazkov.

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

A bug in the V8 bindings is preventing chromium from setting the href on the
location object...any attempt throws a security error, not just for javascript
protocol

  • page/Location.idl:
4:46 PM Changeset in webkit [41881] by levin@chromium.org
  • 2 edits in trunk/WebCore

2009-03-20 Craig Schlenter <craig.schlenter@gmail.com>

Reviewed by Dimitri Glazkov.

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

Include stdio.h needed for printf on gcc 4.4.0

  • platform/KURLGoogle.cpp:
4:46 PM Changeset in webkit [41880] by levin@chromium.org
  • 2 edits in trunk/WebCore

2009-03-20 Mike Belshe <mike@belshe.com>

Reviewed by Darin Fisher.

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

Don't let comments at the end of an event handler
break the event handler.

No change in behavior, so no test.

  • bindings/v8/V8LazyEventListener.cpp: (WebCore::V8LazyEventListener::getWrappedListenerFunction):
4:45 PM Changeset in webkit [41879] by levin@chromium.org
  • 18 edits in trunk

JavaScriptCore:

2009-03-20 Norbert Leser <norbert.leser@nokia.com>

Reviewed by Darin Adler.

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

Fixes missing line terminator character (;) after macro call.
It is common practice to add the trailing ";" where macros are substituted
and not where they are defined with #define.
This change is consistent with other macro declarations across webkit,
and it also solves compilation failure with symbian compilers.

  • runtime/UString.cpp:
  • wtf/Assertions.h:

WebCore:

2009-03-20 Norbert Leser <norbert.leser@nokia.com>

Reviewed by Darin Adler.

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

Fixes missing line terminator character (;) after macro call.
It is common practice to add the trailing ";" where macros are substituted
and not where they are defined with #define.
This change is consistent with other macro declarations across webkit,
and it also solves compilation failure with symbian compilers.

No change in behavior, so no test.

  • bindings/js/JSDOMWindowShell.cpp:
  • bindings/js/JSEventListener.cpp:
  • bindings/js/JSImageConstructor.cpp:
  • bindings/js/JSInspectedObjectWrapper.cpp:
  • bindings/js/JSInspectorCallbackWrapper.cpp:
  • bindings/js/JSNamedNodesCollection.cpp:
  • bindings/js/JSNodeFilterCondition.cpp:
  • bindings/js/JSOptionConstructor.cpp:
  • bindings/js/JSQuarantinedObjectWrapper.cpp:
  • bindings/js/JSRGBColor.cpp:
  • bindings/js/JSWorkerContextBase.cpp:
  • bindings/js/JSXMLHttpRequestConstructor.cpp:
  • bindings/js/JSXSLTProcessorConstructor.cpp:
  • bindings/scripts/CodeGeneratorJS.pm:
4:45 PM Changeset in webkit [41878] by levin@chromium.org
  • 3 edits in trunk/WebCore

2009-03-20 Mike Belshe <mike@belshe.com>

Reviewed by Dimitri Glazkov and Dave Hyatt.

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

Make the minimum timer configurable for different platforms.

  • page/DOMTimer.cpp: (WebCore::DOMTimer::DOMTimer): (WebCore::DOMTimer::fired):
  • page/DOMTimer.h: (WebCore::DOMTimer::minTimerInterval): (WebCore::DOMTimer::setMinTimerInterval):
4:45 PM Changeset in webkit [41877] by levin@chromium.org
  • 2 edits in trunk/WebCore

2009-03-20 Dean McNamee <deanm@chromium.org>

Reviewed by Darin Adler.

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

Make sure to consistently match new/delete and fastMalloc/fastFree.

  • css/CSSSelectorList.cpp: (WebCore::CSSSelectorList::adoptSelectorVector): (WebCore::CSSSelectorList::deleteSelectors):
4:41 PM Changeset in webkit [41876] by mitz@apple.com
  • 7 edits
    4 adds in trunk

WebCore:

Reviewed by Dave Hyatt.

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::percentHeightDescendants): Added this accessor.
  • rendering/RenderBlock.h:
  • rendering/RenderTableSection.cpp: (WebCore::RenderTableSection::layoutRows): Extended the check for children that flex to include other descendants with percent height which is relative to the cell.

LayoutTests:

Reviewed by Dave Hyatt.

  • fast/replaced/percent-height-in-anonymous-block-in-table.html: Added.
  • fast/replaced/table-percent-height-expected.txt:
  • fast/replaced/table-percent-height.html:
  • platform/mac/fast/replaced/percent-height-in-anonymous-block-in-table-expected.checksum: Added.
  • platform/mac/fast/replaced/percent-height-in-anonymous-block-in-table-expected.png: Added.
  • platform/mac/fast/replaced/percent-height-in-anonymous-block-in-table-expected.txt: Added.
4:29 PM Changeset in webkit [41875] by dimich@chromium.org
  • 1 edit
    1 add in trunk/LayoutTests

2009-03-20 Dmitry Titov <dimich@chromium.org>

Fix broken LayoutTest/http/tests/workers/text-encoding.html.
Forgot to add a new file with the previous patch.

  • http/tests/workers/resources/subworker-encoded.php: Added.
4:24 PM Changeset in webkit [41874] by darin@chromium.org
  • 2 edits
    1 add
    1 delete in trunk/LayoutTests

2009-03-20 Alpha Lam <hclam@chromium.org>

Reviewed by Darin Fisher.

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

The test in this change is just to verify a no crash. Full tree
dump is not necessary, so change to test to dumpAsText and update
the affected test results.

  • fast/block/float/crash-on-absolute-positioning-expected.txt: Added.
  • fast/block/float/crash-on-absolute-positioning.html:
  • platform/mac/fast/block/float/crash-on-absolute-positioning-expected.txt: Removed.
4:03 PM Changeset in webkit [41873] by jmalonzo@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-03-20 Jan Michael Alonzo <jmalonzo@gmail.com>

Not reviewed. Fix style issues with my previous commit (r41866) as
suggested by Holger in https://bugs.webkit.org/show_bug.cgi?id=24493

  • tests/testwebhistoryitem.c: (test_webkit_web_history_item_get_data): (test_webkit_web_history_item_alternate_title):
3:56 PM Changeset in webkit [41872] by ggaren@apple.com
  • 3 edits in trunk/JavaScriptCore

2009-03-20 Geoffrey Garen <ggaren@apple.com>

Reviewed by Oliver Hunt.


A little cleanup in the RegisterFile code.


Moved large inline functions out of the class declaration, to make it
more readable.


Switched over to using the roundUpAllocationSize function to avoid
duplicate code and subtle bugs.


Renamed m_maxCommitted to m_commitEnd, to match m_end.


Renamed allocationSize to commitSize because it's the chunk size for
committing memory, not allocating memory.


SunSpider reports no change.

  • interpreter/RegisterFile.h: (JSC::RegisterFile::RegisterFile): (JSC::RegisterFile::shrink): (JSC::RegisterFile::grow):
  • jit/ExecutableAllocator.h: (JSC::roundUpAllocationSize):
3:52 PM Changeset in webkit [41871] by dimich@chromium.org
  • 16 edits in trunk

WebCore:

2009-03-20 Dmitry Titov <dimich@chromium.org>

Reviewed by Alexey Proskuryakov.

https://bugs.webkit.org/show_bug.cgi?id=24706
Remove ScriptExecutionContext::encoding() since Workers do not need it.
WorkerContext::encoding() is simply removed, while Document::encoding()
made non-virtual and private. Workers use UTF-8 now except when instructed
otherwise by http header. Also updated test.

  • dom/Document.h: Made encoding() non-virtual and private.
  • dom/ScriptExecutionContext.h: removed encoding().
  • workers/Worker.cpp: (WebCore::Worker::Worker): (WebCore::Worker::notifyFinished):
  • workers/WorkerContext.cpp: removed encoding() implementation. (WebCore::WorkerContext::WorkerContext): (WebCore::WorkerContext::completeURL):
  • workers/WorkerContext.h: removed encoding() (WebCore::WorkerContext::create):
  • workers/WorkerContextProxy.h:
  • workers/WorkerImportScriptsClient.cpp: (WebCore::WorkerImportScriptsClient::didReceiveData):
  • workers/WorkerMessagingProxy.cpp: (WebCore::WorkerMessagingProxy::startWorkerContext):
  • workers/WorkerMessagingProxy.h:
  • workers/WorkerThread.cpp: (WebCore::WorkerThreadStartupData::create): (WebCore::WorkerThreadStartupData::WorkerThreadStartupData): (WebCore::WorkerThread::create): (WebCore::WorkerThread::WorkerThread): (WebCore::WorkerThread::workerThread):
  • workers/WorkerThread.h: In all these, removed storing encoding and 'inheriting' it from the parent. Instead, they are all using UTF-8 now.

LayoutTests:

2009-03-20 Dmitry Titov <dimich@chromium.org>

Reviewed by Alexey Proskuryakov.

https://bugs.webkit.org/show_bug.cgi?id=24706
Remove ScriptExecutionContext::encoding() since Workers do not need it.
WorkerContext::encoding() is simply removed, while Document::encoding()
made non-virtual and private. Workers use UTF-8 now except when instructed
otherwise by http header. Updated test.
Also, added test for encoding used by importScripts().

  • http/tests/workers/resources/subworker-encoded.php: Added.
  • http/tests/workers/resources/worker-encoded.php:
  • http/tests/workers/text-encoding-expected.txt:
  • http/tests/workers/text-encoding.html:
3:40 PM Changeset in webkit [41870] by timothy@apple.com
  • 15 edits in trunk

Change how threading exceptions are checked so they are reported by what round they were added.

WebCore:

2009-03-20 Timothy Hatcher <timothy@apple.com>

Change how threading exceptions are checked so they are reported
by what round they were added. That way WebKit can decided the
behavior per-round based on linked-on-or-after checks.

<rdar://problem/6626741&6648478&6635474&6674079>

Reviewed by Darin Adler.

  • WebCore.base.exp: Export the new symbols.
  • bindings/objc/DOMAbstractView.mm: Use the new WebCoreThreadViolationCheckRoundOne macro.
  • bindings/scripts/CodeGeneratorObjC.pm: Ditto.
  • platform/ThreadCheck.h:
  • platform/mac/ThreadCheck.mm: (WebCore::readThreadViolationBehaviorFromUserDefaults): Refactor how the default is read. (WebCore::setDefaultThreadViolationBehavior): Take a round argument. (WebCore::reportThreadViolation): Ditto. (WebCoreReportThreadViolation): Ditto.

WebKit:

2009-03-20 Timothy Hatcher <timothy@apple.com>

Change how threading exceptions are checked so they are reported
by what round they were added. That way WebKit can decided the
behavior per-round based on linked-on-or-after checks.

<rdar://problem/6626741&6648478&6635474&6674079>

Reviewed by Darin Adler.

  • History/WebBackForwardList.mm: Use the new WebCoreThreadViolationCheckRoundOne macro.
  • History/WebHistoryItem.mm: Ditto.
  • Misc/WebIconDatabase.mm: Ditto.
  • WebView/WebArchive.mm: Use the new WebCoreThreadViolationCheckRoundTwo macro.
  • WebView/WebResource.mm: Ditto. (+[WebResource _needMailThreadWorkaroundIfCalledOffMainThread]): Check Mail's bundle version to truly decide if it is an old Mail client.
  • WebView/WebView.mm: Ditto.
  • Misc/WebKitVersionChecks.h: Add a new linked-on-or-after version define.
  • WebView/WebFrameView.mm: (-[WebFrameView initWithFrame:]): Set the default thread violation behavior per-round based on difference version checks and the Mail workaround check.
3:25 PM Changeset in webkit [41869] by andersca@apple.com
  • 2 edits in trunk/WebKitTools

2009-03-20 Anders Carlsson <andersca@apple.com>

Reviewed by Darin Adler.

Only release the result NPVariant if the call to NPN_Invoke was successful.


  • DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.cpp: (testNPRuntime):
1:31 PM Changeset in webkit [41868] by ggaren@apple.com
  • 2 edits in trunk/WebCore

2009-03-20 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.

Fixed up an out-of-date comment.

  • bindings/js/JSDOMWindowCustom.h: (WebCore::JSDOMWindow::customPut):
1:25 PM Changeset in webkit [41867] by Darin Adler
  • 2 edits in trunk/WebKit/mac

2009-03-20 Darin Adler <Darin Adler>

Reviewed by Timothy Hatcher.

  • WebView/WebTextIterator.h: Improved comments to point out some of the pitfalls of this SPI.
1:23 PM Changeset in webkit [41866] by jmalonzo@webkit.org
  • 3 edits
    1 add in trunk

2009-03-20 Jan Michael Alonzo <jmalonzo@gmail.com>

Reviewed by Holger Freyther.

[GTK] Misc patches for WebKitWebHistoryItem
https://bugs.webkit.org/show_bug.cgi?id=24493

Added build support for build the WebKitWebHistoryItem unit test.

  • GNUmakefile.am:

2009-03-20 Jan Michael Alonzo <jmalonzo@gmail.com>

Reviewed by Holger Freyther.

[GTK] Misc patches for WebKitWebHistoryItem
https://bugs.webkit.org/show_bug.cgi?id=24493

Add unit test for WebKitWebHistoryItem

  • tests/testwebhistoryitem.c: Added. (web_history_item_fixture_setup): (web_history_item_fixture_teardown): (test_webkit_web_history_item_get_data): (test_webkit_web_history_item_alternate_title): (main):
12:56 PM Changeset in webkit [41865] by Beth Dakin
  • 4 edits
    4 adds in trunk

WebCore:

2009-03-20 Beth Dakin <Beth Dakin>

Reviewed by Dave Hyatt.

Fix for https://bugs.webkit.org/show_bug.cgi?id=20909 REGRESSION
(r35318): A press release at pfizer.com does not display correctly

  • and corresponding - <rdar://problem/6680073>
  • rendering/RenderObject.cpp: (WebCore::RenderObject::invalidateContainerPrefWidths):
  • rendering/RenderObject.h: (WebCore::RenderObject::markContainingBlocksForLayout):

LayoutTests:

2009-03-20 Beth Dakin <Beth Dakin>

Reviewed by Dave Hyatt.

Test for https://bugs.webkit.org/show_bug.cgi?id=20909 REGRESSION
(r35318): A press release at pfizer.com does not display correctly

  • and corresponding - <rdar://problem/6680073>
  • http/tests/misc/generated-content-inside-table.html: Added.
  • platform/mac/http/tests/misc/generated-content-inside-table-expected.checksum: Added.
  • platform/mac/http/tests/misc/generated-content-inside-table-expected.png: Added.
  • platform/mac/http/tests/misc/generated-content-inside-table-expected.txt: Added.
12:45 PM Changeset in webkit [41864] by jmalonzo@webkit.org
  • 3 edits
    1 copy
    1 move in trunk

2009-03-20 Jan Michael Alonzo <jmalonzo@gmail.com>

Reviewed by Holger Freyther.

Separate gtk unit tests
https://bugs.webkit.org/show_bug.cgi?id=24039

Build the unit tests accordingly.

  • GNUmakefile.am:

2009-03-20 Jan Michael Alonzo <jmalonzo@gmail.com>

Reviewed by Holger Freyther.

Separate gtk unit tests
https://bugs.webkit.org/show_bug.cgi?id=24039

Split the current single-file unit test to make it more
modularized and manageable in the future as more unit tests are
written.

  • tests/main.c: Removed.
  • tests/testwebbackforwardlist.c: Copied from WebKit/gtk/tests/main.c. (main):
  • tests/testwebframe.c: Copied from WebKit/gtk/tests/main.c. (main):
11:45 AM Changeset in webkit [41863] by eric@webkit.org
  • 19 edits in trunk/WebCore

Reviewed by Justin Garcia.

maxDeepOffset is confusing and should be removed
https://bugs.webkit.org/show_bug.cgi?id=24586

Abstract some hard-to-read (but shared) logic into a new renderedAsNonInlineTableOrHR function.
Add first/lastDeepEditingPositionForNode Position creation functions
and deploy them to places we used to call maxDeepOffset.

Rename Position::atStart and atEnd to atStartOfTree atEndOfTree
Add a new Position::atFirst/atLastEditingPositionForNode() and use these
to replace a few more callers for maxDeepOffset()

Rename maxDeepOffset to lastEditingOffsetForNode (so that we mere mortals have some clue what it does)

"Editing positions" are confusing because they have one
of two behaviors, depending on if the container node is ignored
by editing (if editingIgnoresContent(node) returns true) or not.
Positions referring to nodes ignored by editing are
neighbor-relative (they are before or after the node) where as
positions reffering to other nodes are container-relative
(they are between two child nodes of the container, identified
by the offset() member). I will be fixing this confusion in
future patches. These renames hopefully make the current behavior clearer.

  • dom/Position.cpp: (WebCore::Position::previous): (WebCore::Position::next): (WebCore::Position::atFirstEditingPositionForNode): (WebCore::Position::atLastEditingPositionForNode): (WebCore::Position::atStartOfTree): (WebCore::Position::atEndOfTree): (WebCore::Position::previousCharacterPosition): (WebCore::Position::nextCharacterPosition): (WebCore::Position::upstream): (WebCore::Position::isCandidate): (WebCore::firstDeepEditingPositionForNode): (WebCore::lastDeepEditingPositionForNode):
  • dom/Position.h:
  • dom/PositionIterator.cpp: (WebCore::PositionIterator::operator Position): (WebCore::PositionIterator::increment): (WebCore::PositionIterator::decrement): (WebCore::PositionIterator::atEnd): (WebCore::PositionIterator::atEndOfNode):
  • editing/CompositeEditCommand.cpp: (WebCore::CompositeEditCommand::positionAvoidingSpecialElementBoundary):
  • editing/DeleteSelectionCommand.cpp: (WebCore::isTableCellEmpty): (WebCore::DeleteSelectionCommand::removeNode): (WebCore::DeleteSelectionCommand::handleGeneralDelete):
  • editing/Editor.cpp: (WebCore::Editor::advanceToNextMisspelling):
  • editing/InsertLineBreakCommand.cpp: (WebCore::InsertLineBreakCommand::doApply):
  • editing/InsertListCommand.cpp: (WebCore::InsertListCommand::doApply):
  • editing/ReplaceSelectionCommand.cpp: (WebCore::ReplaceSelectionCommand::positionAtEndOfInsertedContent):
  • editing/TypingCommand.cpp: (WebCore::TypingCommand::forwardDeleteKeyPressed):
  • editing/VisiblePosition.cpp: (WebCore::VisiblePosition::previous): (WebCore::VisiblePosition::leftVisuallyDistinctCandidate): (WebCore::VisiblePosition::left): (WebCore::VisiblePosition::rightVisuallyDistinctCandidate): (WebCore::VisiblePosition::right):
  • editing/VisibleSelection.cpp: (WebCore::VisibleSelection::selectionFromContentsOfNode): (WebCore::VisibleSelection::adjustSelectionToAvoidCrossingEditingBoundaries):
  • editing/htmlediting.cpp: (WebCore::nextVisuallyDistinctCandidate): (WebCore::previousVisuallyDistinctCandidate): (WebCore::firstEditablePositionAfterPositionInRoot): (WebCore::lastEditablePositionBeforePositionInRoot): (WebCore::lastOffsetForEditing): (WebCore::isFirstPositionAfterTable): (WebCore::isLastPositionBeforeTable): (WebCore::positionBeforeNode): (WebCore::positionAfterNode): (WebCore::enclosingEmptyListItem): (WebCore::caretMaxOffset):
  • editing/htmlediting.h:
  • editing/visible_units.cpp: (WebCore::renderedAsNonInlineTableOrHR): (WebCore::startOfParagraph): (WebCore::endOfParagraph): (WebCore::startOfEditableContent): (WebCore::endOfEditableContent):
  • page/AccessibilityObject.cpp: (WebCore::endOfStyleRange):
  • page/AccessibilityRenderObject.cpp: (WebCore::AccessibilityRenderObject::visiblePositionRange):
  • rendering/RenderBox.cpp: (WebCore::RenderBox::positionForPoint):
10:32 AM Changeset in webkit [41862] by dimich@chromium.org
  • 3 edits
    2 copies
    3 adds in trunk/WebCore

2009-03-20 Dmitry Titov <dimich@chromium.org>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=24689
Add (upstream) V8 bindings for Workers. Mostly style cleaning.

  • bindings/v8/WorkerContextExecutionProxy.cpp: Added.
  • bindings/v8/WorkerContextExecutionProxy.h: Added.
  • bindings/v8/WorkerScriptController.cpp: Added.
  • bindings/v8/WorkerScriptController.h: Added.
  • bindings/v8/V8Index.h: Added. This is just a wrapper for v8_index.h, like V8Proxy.h
  • bindings/v8/V8Proxy.h: Added domObjectMap() function that wraps GetDOMObjectMap().
  • bindings/v8/V8WorkerContextEventListener.cpp: (WebCore::V8WorkerContextEventListener::callListenerFunction): TrackEvent() renamed trackEvent()
10:27 AM Changeset in webkit [41861] by ariya@webkit.org
  • 3 edits in trunk/WebKit/qt

2009-03-20 Erik L. Bunce <elbunce@xendom.com>

Reviewed by Tor Arne Vestbø.

Fix QWebPage::WebActions action states to more closely match when they are
actually applicable and remove erroneous documentation.

  • Most WebActions implemented using editor commands now use the Editor::Command::isEnabled() to control their availability.
  • SelectAll is always enabled (since it's editor command is).
  • SetTextDirection{} family of WebActions are available when canEdit() is true and not just canEditRichly().

Fix and clarify documentation about the availability of various web actions.

  • Api/qwebpage.cpp: (QWebPagePrivate::updateAction): (QWebPagePrivate::updateEditorActions):
  • tests/qwebpage/tst_qwebpage.cpp: (tst_QWebPage::textSelection):
9:39 AM Changeset in webkit [41860] by eric@webkit.org
  • 3 edits in trunk/WebCore

2009-03-20 Stephen White <senorblanco@chromium.org>

Reviewed by Eric Seidel.

Fix for LayoutTests/fast/canvas/canvas-text-alignment.html
on chromium/skia. The problem was that the gradient matrix
for text was being applied twice. Fixed by reverting some of
https://bugs.webkit.org/show_bug.cgi?id=23957, so that skiaDrawText
is no longer responsible for measuring the text and scaling up
the gradient matrix. Instead, the text bounding box is passed
in from SVGPaintServerGradient. I didn't make this change for CG,
since it uses a different method (the gradient is drawn using the
text as a pre-rendered mask).
https://bugs.webkit.org/show_bug.cgi?id=24687

  • platform/graphics/skia/SkiaFontWin.cpp: (WebCore::skiaDrawText):
  • svg/graphics/SVGPaintServerGradient.cpp: (WebCore::SVGPaintServerGradient::setup):
7:50 AM Changeset in webkit [41859] by Darin Adler
  • 2 edits in trunk/WebKit/mac

2009-03-20 Darin Adler <Darin Adler>

Reviewed by Adele Peterson.

Use a better technique to handle finding out if something responds to a selector
in WebHTMLView's doCommandBySelector method.

  • WebView/WebHTMLView.mm: (-[WebHTMLView doCommandBySelector:]): Removed unneeded check for 0 coreFrame; this is already handled by coreCommandBySelector: so doesn't need to be checked twice. Got rid of initial value for eventWasHandled boolean to make it more clear. Use WebResponderChainSink to find out if a command is handled rather than walking the responder chain explicitly. (-[WebResponderChainSink initWithResponderChain:]): Added. (-[WebResponderChainSink detach]): Added. (-[WebResponderChainSink receivedUnhandledCommand]): Added. (-[WebResponderChainSink noResponderFor:]): Added. (-[WebResponderChainSink doCommandBySelector:]): Added.
4:09 AM Changeset in webkit [41858] by timothy@apple.com
  • 1 edit in trunk/WebKit/mac/WebView/WebView.mm

Fix the Tiger build, round two.

3:54 AM Changeset in webkit [41857] by timothy@apple.com
  • 1 edit in trunk/WebKit/mac/WebView/WebView.mm

Fix the Tiger build.

3:07 AM Changeset in webkit [41856] by xan@webkit.org
  • 3 edits in trunk/WebCore

2009-03-20 Xan Lopez <xlopez@igalia.com>

Rubber-stamped by Holger Freyther.

There seems to be some rounding error in cairo (or in how we use
cairo) with some fonts, like DejaVu Sans Mono, which makes cairo
report a height smaller than ascent + descent, which is wrong and
confuses WebCore's layout system. Workaround this while we figure
out what's going on.

  • platform/graphics/gtk/SimpleFontDataGtk.cpp: (WebCore::SimpleFontData::platformInit):
  • platform/graphics/gtk/SimpleFontDataPango.cpp: (WebCore::SimpleFontData::platformInit):
1:16 AM Changeset in webkit [41855] by ap@webkit.org
  • 3 edits
    2 adds in trunk

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=24122
<rdar://problem/6674179>
REGRESSION: DOM Range extractContents/deleteContents failures seen on Moxiecode tests

Test: fast/dom/Range/deleted-range-endpoints.html

  • dom/Range.cpp: (WebCore::Range::processContents): Set the final range in accordance to the specification.

Mar 19, 2009:

10:57 PM Changeset in webkit [41854] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

2009-03-19 Geoffrey Garen <ggaren@apple.com>

Reviewed by Oliver Hunt.


Fixed <rdar://problem/6033712> -- a little bit of hardening in the Collector.


SunSpider reports no change. I also verified in the disassembly that
we end up with a single compare to constant.

  • runtime/Collector.cpp: (JSC::Heap::heapAllocate):
9:31 PM Changeset in webkit [41853] by timothy@apple.com
  • 2 edits in trunk/WebKit/mac

Remove #ifndef BUILDING_ON_TIGER around code that schedules runloop modes
for Page, so the new RunLoopTimer in WebCore always gets a default mode.
Fixes the layout test failures on the Tiger build bots.

Reviewed by Mark Rowe.

  • WebView/WebView.mm: (-[WebView _commonInitializationWithFrameName:groupName:usesDocumentViews:]): (-[WebView scheduleInRunLoop:forMode:]): (-[WebView unscheduleFromRunLoop:forMode:]):
9:28 PM Changeset in webkit [41852] by ggaren@apple.com
  • 4 edits
    1 move in trunk

JavaScriptCore:

2009-03-19 Geoffrey Garen <ggaren@apple.com>

Reviewed by Cameron Zwarich and Oliver Hunt.

Fixed <rdar://problem/6406045> REGRESSION: Stack overflow on PowerPC on
fast/workers/use-machine-stack.html (22531)

Dialed down the re-entry allowance to 64 (from 128).


On a 512K stack, this leaves about 64K for other code on the stack while
JavaScript is running. Not perfect, but it solves our crash on PPC.

Different platforms may want to dial this down even more.

Also, substantially shrunk BytecodeGenerator. Since we allocate one on
the stack in order to throw a stack overflow exception -- well, let's
just say the old code had an appreciation for irony.

SunSpider reports no change.

  • bytecompiler/BytecodeGenerator.h:
  • interpreter/Interpreter.h: (JSC::):

LayoutTests:

2009-03-19 Geoffrey Garen <ggaren@apple.com>

Reviewed by Cameron Zwarich and Oliver Hunt.

Re-enabled this test, since it no longer crashes when JavaScriptCore is
compiled in non-JIT mode.

  • fast/workers/use-machine-stack.html: Copied from fast/workers/use-machine-stack.html-disabled.
  • fast/workers/use-machine-stack.html-disabled: Removed.
7:04 PM Changeset in webkit [41851] by cwzwarich@webkit.org
  • 3 edits
    3 adds in trunk

2009-03-19 Cameron Zwarich <cwzwarich@uwaterloo.ca>

Reviewed by Oliver Hunt.

Bug 24350: REGRESSION: Safari 4 breaks SPAW wysiwyg editor multiple instances
<https://bugs.webkit.org/show_bug.cgi?id=24350>
<rdar://problem/6674182>

The SPAW editor's JavaScript assumes that toString() on a function
constructed with the Function constructor produces a function with
a newline after the opening brace.

JavaScriptCore:

  • runtime/FunctionConstructor.cpp: (JSC::constructFunction): Add a newline after the opening brace of the function's source code.

LayoutTests:

  • fast/js/function-constructor-newline-after-brace-expected.txt: Added.
  • fast/js/function-constructor-newline-after-brace.html: Added.
  • fast/js/resources/function-constructor-newline-after-brace.js: Added.
6:50 PM Changeset in webkit [41850] by mrowe@apple.com
  • 1 edit
    3 moves
    6 adds in trunk/LayoutTests

Fix fast/text/Geeza-Pro-vertical-metrics-adjustment.html on Leopard by moving the results in to the location
that run-webkit-tests currently looks for them.

Reviewed by Stephanie Lewis.

5:52 PM Changeset in webkit [41849] by ggaren@apple.com
  • 5 edits in trunk

JavaScriptCore:

2009-03-19 Cameron Zwarich <cwzwarich@uwaterloo.ca>

Reviewed by Geoff Garen.

Bug 23771: REGRESSION (r36016): JSObjectHasProperty freezes on global class without kJSClassAttributeNoAutomaticPrototype
<https://bugs.webkit.org/show_bug.cgi?id=23771>
<rdar://problem/6561016>

  • API/tests/testapi.c: (main): Add a test for this bug.
  • runtime/JSGlobalObject.cpp: (JSC::JSGlobalObject::resetPrototype): Don't set the prototype of the last object in the prototype chain to the object prototype when the object prototype is already the last object in the prototype chain.

LayoutTests:

2009-03-19 Geoffrey Garen <ggaren@apple.com>

Reviewed by Oliver Hunt.


Fixed <rdar://problem/6279213> Regular expression run-time complexity
limit too low for long inputs (21485)


Added a test for a complex regexp match that should succeed, and one
that should fail.

  • fast/js/regexp-overflow-expected.txt:
  • fast/js/resources/regexp-overflow.js:
5:44 PM Changeset in webkit [41848] by mrowe@apple.com
  • 3 edits
    1 copy
    1 add in trunk/LayoutTests

Update test results for changes in handling of CMYK.

Reviewed by Sam Weinig.

  • fast/canvas/resources/set-colors.js:
  • fast/canvas/set-colors-expected.txt:
  • platform/mac-leopard/fast/canvas/set-colors-expected.txt: Copied from LayoutTests/fast/canvas/set-colors-expected.txt.
5:16 PM Changeset in webkit [41847] by andersca@apple.com
  • 3 edits in trunk/WebCore

2009-03-19 Anders Carlsson <andersca@apple.com>

Reviewed by Dan Bernstein.

<rdar://problem/6682554> Flash content not being rendered (Shockwave Flash 10.0 r22)


If calling updateWidget for some reason resulted in another widget being added to m_widgetUpdateSet, then
that object would never be updated.


  • page/FrameView.cpp: (WebCore::FrameView::updateWidgets): Factor the widget updating code out into this method. Return true if the update set is empty.


(WebCore::FrameView::performPostLayoutTasks):
Loop over the update set multiple times until all widgets have been updated or until we reach the cap.


  • page/FrameView.h:
5:03 PM Changeset in webkit [41846] by cwzwarich@webkit.org
  • 3 edits in trunk/JavaScriptCore

2009-03-19 Cameron Zwarich <cwzwarich@uwaterloo.ca>

Reviewed by Geoff Garen.

Bug 23771: REGRESSION (r36016): JSObjectHasProperty freezes on global class without kJSClassAttributeNoAutomaticPrototype
<https://bugs.webkit.org/show_bug.cgi?id=23771>
<rdar://problem/6561016>

  • API/tests/testapi.c: (main): Add a test for this bug.
  • runtime/JSGlobalObject.cpp: (JSC::JSGlobalObject::resetPrototype): Don't set the prototype of the last object in the prototype chain to the object prototype when the object prototype is already the last object in the prototype chain.
4:45 PM Changeset in webkit [41845] by timothy@apple.com
  • 6 edits
    2 adds in trunk

Make the defered data loading timer honor the Page's scheduled runloop pairs.
Introduces a new RunLoopTimer class that has an API mimicking Timer but
allows it to be scheduled with one or more SchedulePairs.

<rdar://problem/6687342> -[WebView scheduleInRunLoop:forMode:] has no affect on timers

Reviewed by Darin Adler.

  • WebCore.xcodeproj/project.pbxproj: Adds the new RunLoopTimer.{cpp,h} files.
  • loader/MainResourceLoader.cpp: (WebCore::MainResourceLoader::handleDataLoadNow): Use the MainResourceLoaderTimer typedef. (WebCore::MainResourceLoader::startDataLoadTimer): Added. Start the timer and on Mac platforms also schedule with the Page's SchedulePairs. (WebCore::MainResourceLoader::handleDataLoadSoon): Call startDataLoadTimer(). (WebCore::MainResourceLoader::setDefersLoading): Ditto.
  • loader/MainResourceLoader.h:
  • platform/cf/RunLoopTimerCF.cpp: Added. (WebCore::RunLoopTimerBase::~RunLoopTimerBase): (WebCore::timerFired): (WebCore::RunLoopTimerBase::start): (WebCore::RunLoopTimerBase::schedule): (WebCore::RunLoopTimerBase::stop): (WebCore::RunLoopTimerBase::isActive):
  • platform/RunLoopTimer.h: Added. (WebCore::RunLoopTimerBase::RunLoopTimerBase): (WebCore::RunLoopTimerBase::startRepeating): (WebCore::RunLoopTimerBase::startOneShot): (WebCore::RunLoopTimer::RunLoopTimer): (WebCore::RunLoopTimer::fired):
2:08 PM Changeset in webkit [41844] by kov@webkit.org
  • 2 edits in trunk/WebKitTools

2009-03-19 Gustavo Noronha Silva <Gustavo Noronha Silva>

Reviewed by Mark Rowe.

Make determinePassedArchitecture always consume the --32-bit
option, also in non-AppleMacWebkit platforms. Solution pointed out
by Mark Rowe.

  • Scripts/webkitdirs.pm:
1:47 PM Changeset in webkit [41843] by hyatt@apple.com
  • 1 add in trunk/WebKitSite/blog-files/scrollbarpicture.png

Adding scrollbar images for blog post.

1:32 PM Changeset in webkit [41842] by ggaren@apple.com
  • 7 edits in trunk

JavaScriptCore:

2009-03-19 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.


Fixed <rdar://problem/6603562> REGRESSION (Safari 4): regular expression
pattern size limit lower than Safari 3.2, other browsers, breaks SAP (14873)


Bumped the pattern size limit to 1MB, and standardized it between PCRE
and WREC. (Empirical testing says that we can easily compile a 1MB regular
expression without risking a hang. Other browsers support bigger regular
expressions, but also hang.)


SunSpider reports no change.


I started with a patch posted to Bugzilla by Erik Corry (erikcorry@google.com).


  • pcre/pcre_internal.h: (put3ByteValue): (get3ByteValue): (put3ByteValueAndAdvance): (putLinkValueAllowZero): (getLinkValueAllowZero): Made PCRE's "LINK_SIZE" (the number of bytes used to record jumps between bytecodes) 3, to accomodate larger potential jumps. Bumped PCRE's "MAX_PATTERN_SIZE" to 1MB. (Technically, at this LINK_SIZE, we can support even larger patterns, but we risk a hang during compilation, and it's not clear that such large patterns are important on the web.)
  • wrec/WREC.cpp: (JSC::WREC::Generator::compileRegExp): Match PCRE's maximum pattern size, to avoid quirks between platforms.

LayoutTests:

2009-03-19 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.


Made two layout tests less agressive, to accomodate a change I made
for <rdar://problem/6603562> REGRESSION (Safari 4): regular expression
pattern size limit lower than Safari 3.2, other browsers, breaks SAP (14873)

  • fast/js/regexp-charclass-crash-expected.txt:
  • fast/js/regexp-charclass-crash.html: Explicitly limit the number of iterations in the test loop. Otherwise, regular expression engines supporting very long patterns take a very very very long time to run this test.
  • fast/js/resources/regexp-overflow.js: Made the "too big" regexp in this test even bigger, to match our new limit.
12:56 PM Changeset in webkit [41841] by Dimitri Glazkov
  • 3 edits in trunk/WebCore

2009-03-19 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Darin Fisher.

https://bugs.webkit.org/show_bug.cgi?id=24702
Upstream miscellaneous bindings changes.

  • bindings/v8/ScheduledAction.cpp: (WebCore::ScheduledAction::execute): Changed to call lower-case evaluate.
  • bindings/v8/ScriptCallStack.h: Added an extra include.
12:14 PM Changeset in webkit [41840] by Simon Fraser
  • 3 edits
    4 adds in trunk

2009-03-19 Simon Fraser <Simon Fraser>

Reviewed by Dave Hyatt

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

When hit testing a RenderLayer whose parent lives in a preserves-3D hierarchy,
we need to compare the computed z-offset with the depth-test z-offset before
deciding that such a RenderLayer was hit. This fixes an issue, tested by the
3d-point-mapping-overlapping.html test, where the child of a transformed element
is found by hit testing, even when some other element with greater Z overlaps
them both.

Improved the code by adding a utility method, isHitCandidate(), which computes and tests
z-depth when necessary.

Tests: transforms/3d/point-mapping/3d-point-mapping-coplanar.html

transforms/3d/point-mapping/3d-point-mapping-overlapping.html

  • rendering/RenderLayer.cpp: (WebCore::isHitCandidate): (WebCore::RenderLayer::hitTestLayer):
9:43 AM Changeset in webkit [41839] by Dimitri Glazkov
  • 2 edits
    1 add in trunk/WebCore

2009-03-19 Jeremy Moskovich <jeremy@chromium.org>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=24456
Split ColorChromium.cpp into Mac & Windows variants.
Remove Chromium Dependency on platform/graphics/mac/ColorMac.mm since we
ultimately need to take a different approach. For now, createCGColor()
is copied from ColorMac.mm.

No observable change in behavior, so no test.

  • platform/graphics/chromium/ColorChromium.cpp: (WebCore::focusRingColor):
  • platform/graphics/chromium/ColorChromiumMac.mm: Added.
9:36 AM Changeset in webkit [41838] by Dimitri Glazkov
  • 1 edit in trunk/WebCore/ChangeLog

Fix WebCore/ChangeLog entry, add missing reviewer line.

9:35 AM Changeset in webkit [41837] by Dimitri Glazkov
  • 2 edits in trunk/LayoutTests

2009-03-19 Scott Violet <sky@google.com>

Reviewed by Simon Fraser.

https://bugs.webkit.org/show_bug.cgi?id=24678
Transitions tests somewhat flakey on Chromium.

Modifies transitions/transition-end-event-helpers.js to only process
results once.

  • transitions/transition-end-event-helpers.js:
9:29 AM Changeset in webkit [41836] by Dimitri Glazkov
  • 1 edit
    1 add in trunk/WebCore

2009-03-19 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=24675
Unforking frontend: add custom InspectorController methods
implementation.

  • bindings/v8/custom/V8InspectorControllerCustom.cpp: Added.
9:26 AM Changeset in webkit [41835] by Dimitri Glazkov
  • 3 edits in trunk/WebCore

2009-03-19 Jay Campan <jcampan@google.com>

https://bugs.webkit.org/show_bug.cgi?id=24625
Adding an accessor to the currently selected index in the PopupMenuChromium.
This is required for implementing the deletion of an autocomplete entry in Chromium.

  • platform/chromium/PopupMenuChromium.cpp: (WebCore::PopupContainer::selectedIndex):
  • platform/chromium/PopupMenuChromium.h:
9:09 AM Changeset in webkit [41834] by darin@chromium.org
  • 2 edits in trunk/WebCore

2009-03-19 Evan Stade <estade@chromium.org>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=24526
Improve windows skia text stroking.

Test: LayoutTests/svg/custom/struct-use-09-b.svg

  • platform/graphics/skia/SkiaFontWin.cpp: Close the path representing each font glyph polygon, rather than only closing the path once per letter. This fixes stroking for letters with multiple polygons, such as 'A' or 'D'. (WebCore::getPathForGlyph): (WebCore::skiaDrawText):
9:06 AM Changeset in webkit [41833] by Simon Hausmann
  • 3 edits in trunk/WebKit/qt

Oops, bail out last patch, that was incorrect.

8:59 AM Changeset in webkit [41832] by Simon Hausmann
  • 3 edits in trunk/WebKit/qt

2009-03-19 Laurent Montel <montel@kde.org>

Reviewed by Simon Hausmann.

Another fix for pedantic compilation in QtWebKit

(patch from qt-copy)

8:16 AM Changeset in webkit [41831] by Simon Hausmann
  • 3 edits in trunk/WebKit/qt

2009-03-19 Ariya Hidayat <ariya.hidayat@trolltech.com>

Reviewed by Simon Hausmann.

Fixes pedantic compilation in QtWebKit.

There are no semi-colons after namespace declarations.

8:16 AM Changeset in webkit [41830] by Simon Hausmann
  • 2 edits in trunk/WebKit/qt

2009-03-19 David Boddie <dboddie@trolltech.com>

Reviewed by Simon Hausmann.

Doc: Removed obsolete warning about Flash and other plugins.

8:16 AM Changeset in webkit [41829] by Simon Hausmann
  • 2 edits in trunk/WebKit/qt

2009-03-19 Paul Olav Tvete <paul.tvete@nokia.com>

Reviewed by Simon Hausmann.

Properly escape tooltip text

ManualTest: http://xkcd.com/554/

8:08 AM Changeset in webkit [41828] by Simon Hausmann
  • 6 edits in trunk/WebCore

2009-03-19 Simon Hausmann <simon.hausmann@nokia.com>

Reviewed by Tor Arne Vestbø.

Fixed support for doing calls from JavaScript into NPAPI Plugins for the Qt port on Windows.

Removed dead code for distinguishing between Widget and PluginView in the Qt port.

1:06 AM Changeset in webkit [41827] by mrowe@apple.com
  • 3 edits in trunk/WebKitTools

Work around <rdar://problem/6698023> by activating fonts from disk.

Reviewed by Oliver Hunt.

  • DumpRenderTree/DumpRenderTree.xcodeproj/project.pbxproj: Copy fonts into DumpRenderTree.resources

in the built products directory.

  • DumpRenderTree/mac/DumpRenderTree.mm:

(activateFonts): Activate the fonts from disk.

12:55 AM Changeset in webkit [41826] by cwzwarich@webkit.org
  • 4 edits
    3 adds in trunk

2009-03-19 Cameron Zwarich <cwzwarich@uwaterloo.ca>

Reviewed by Oliver Hunt.

Bug 24596: ASSERT in JSC::PropertySlot::slotBase @ iGoogle homepage
<https://bugs.webkit.org/show_bug.cgi?id=24596>
<rdar://problem/6686493>

JSDOMWindow::customGetOwnPropertySlot() does an access check after calling
JSGlobalObject::getOwnPropertySlot(). This causes the PropertySlot to be
set twice, once to the value that is illegal to access, and then to undefined
This causes an assertion failure in property access caching code.

The fix is to do the access check before calling JSGlobalObject::getOwnPropertySlot().

WebCore:

  • bindings/js/JSDOMWindowCustom.h: (WebCore::JSDOMWindow::customGetOwnPropertySlot):

LayoutTests:

  • http/tests/security/cross-frame-access-get-custom-property-cached-expected.txt: Added.
  • http/tests/security/cross-frame-access-get-custom-property-cached.html: Added.
  • http/tests/security/resources/cross-frame-access-get-custom-property-cached-iframe.html: Added.
  • http/tests/security/resources/cross-frame-access.js: (shouldBeUndefined):
12:32 AM Changeset in webkit [41825] by ap@webkit.org
  • 7 edits
    3 adds in trunk

Reviewed by Sam Weinig.

https://bugs.webkit.org/show_bug.cgi?id=24676
Simple cross-origin requests shouldn't dispatch upload progress events

Test: http/tests/xmlhttprequest/simple-cross-origin-progress-events.html

  • loader/CrossOriginAccessControl.cpp: (WebCore::passesAccessControlCheck): Added a comment explaining the somewhat unexpected behavior of this function.
  • xml/XMLHttpRequestUpload.cpp: (WebCore::XMLHttpRequestUpload::hasListeners):
  • xml/XMLHttpRequestUpload.h: Report whether there are any event listeners registered.
  • xml/XMLHttpRequest.h: Added m_uploadEventsAllowed.
  • xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::createRequest): Set m_uploadEventsAllowed flag. (WebCore::XMLHttpRequest::makeSameOriginRequest): Ditto. (WebCore::XMLHttpRequest::makeCrossOriginAccessRequest): Ditto. (WebCore::XMLHttpRequest::makeSimpleCrossOriginAccessRequest): Set request body - it can be non-empty for POST requests. (WebCore::XMLHttpRequest::makeCrossOriginAccessRequestWithPreflight): Set m_uploadEventsAllowed flag. (WebCore::XMLHttpRequest::handleAsynchronousPreflightResult): Ditto. (WebCore::XMLHttpRequest::abort): Only dispatch upload progress events if allowed. (WebCore::XMLHttpRequest::networkError): Ditto. (WebCore::XMLHttpRequest::abortError): Ditto. (WebCore::XMLHttpRequest::didSendData): Ditto.

Mar 18, 2009:

11:18 PM Changeset in webkit [41824] by darin@chromium.org
  • 2 edits in trunk/WebCore

2009-03-18 Marc-Antoine Ruel <maruel@chromium.org>

Reviewed by Darin Fisher.

https://bugs.webkit.org/show_bug.cgi?id=24398
Fix a crash when loading a svg file in Chromium's test_shell and
then reloading the page.

  • history/BackForwardListChromium.cpp: (WebCore::BackForwardList::BackForwardList): (WebCore::BackForwardList::close):
11:04 PM BuildingGtk edited by arthur.webkid@gmail.com
WebKitGTK doesn't depend on curl anymore. (diff)
9:42 PM Changeset in webkit [41823] by adachan@apple.com
  • 2 edits in trunk/JavaScriptCore

Rolling out r41818 since it broke the windows build.
Error: ..\..\runtime\DatePrototype.cpp(30) : fatal error C1083: Cannot open include file: 'langinfo.h': No such file or directory

  • runtime/DatePrototype.cpp: (JSC::formatLocaleDate):
6:54 PM Changeset in webkit [41822] by mrowe@apple.com
  • 2 edits in trunk/LayoutTests

Skip plugins/netscape-throw-exception.html due to <rdar://problem/6530010>.

Reviewed by Sam Weinig.

  • platform/mac-snowleopard/Skipped:
6:29 PM Changeset in webkit [41821] by mrowe@apple.com
  • 1 edit
    1 move in trunk/LayoutTests

Disable pluginDocumentView-deallocated-dataSource.html as it has not tested
what it was designed to test for some time, and is failing on some machines.
Fixing the test is tracked by <https://bugs.webkit.org/show_bug.cgi?id=24690>.

Rubber-stamped by Dan Bernstein.

  • platform/mac/plugins/pluginDocumentView-deallocated-dataSource.html-disabled: Renamed from LayoutTests/platform/mac/plugins/pluginDocumentView-deallocated-dataSource.html.
5:57 PM Changeset in webkit [41820] by andersca@apple.com
  • 2 edits in trunk/WebKit/mac

2009-03-18 Anders Carlsson <andersca@apple.com>

Reviewed by Oliver Hunt.

WebKit side of <rdar://problem/6688244>.


Try reinitializing the vendor port if it's invalid.


  • Plugins/Hosted/NetscapePluginHostManager.mm: (WebKit::NetscapePluginHostManager::spawnPluginHost):
5:49 PM Changeset in webkit [41819] by mrowe@apple.com
  • 8 edits in trunk

<rdar://problem/6693300> Don't rely on printf from TestNetscapePlugIn appearing in test results

Reviewed by Dan Bernstein.

Switch from using printf to using the NPAPI to invoke console.log so that plug-in messages appear
in test results even when the plug-in's stdout differs from DumpRenderTree's stdout.

  • DumpRenderTree/TestNetscapePlugIn.subproj/main.cpp:

(log): Invoke console.log via the NPAPI.
(NPP_Destroy): Call log instead of printf.
(NPP_SetWindow): Ditto.
(handleEventCarbon): Ditto.
(handleEventCocoa): Ditto.
(NPP_HandleEvent): Pass the instance in to the event handler.

Update test results for change to address <rdar://problem/6693300>.

  • plugins/mouse-events-expected.txt:
  • plugins/netscape-destroy-plugin-script-objects-expected.txt:
  • plugins/netscape-plugin-setwindow-size-2-expected.txt:
  • plugins/netscape-plugin-setwindow-size-expected.txt:
  • plugins/open-and-close-window-with-plugin-expected.txt:
4:21 PM Changeset in webkit [41818] by kov@webkit.org
  • 2 edits in trunk/JavaScriptCore

2009-03-18 Gustavo Noronha Silva <Gustavo Noronha Silva> and Thadeu Lima de Souza Cascardo <cascardo@holoscopio.com>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=24674
Crashes in !PLATFORM(MAC)'s formatLocaleDate, in very specific situations

Make sure strftime never returns 2-digits years to avoid ambiguity and
a crash.

  • runtime/DatePrototype.cpp: (JSC::formatLocaleDate):
4:09 PM Changeset in webkit [41817] by sfalken@apple.com
  • 4 edits in trunk/WebKit/win

Add webViewClosing to private UI delegate.

Reviewed by Adam Roben.

  • Interfaces/IWebUIDelegatePrivate.idl:
  • Interfaces/WebKit.idl:
  • WebView.cpp: (WebView::close):
1:41 PM Changeset in webkit [41816] by eric.carlson@apple.com
  • 2 edits in trunk/WebCore

2009-03-18 Eric Carlson <eric.carlson@apple.com>

Reviewed by Simon Fraser.

Fix for <rdar://problem/6685235>
<video> element poster cannot be set dynamically if not originally set up in HTML


Allocate the media engine immediately so the plug-in proxy is available for
scripting right away.

  • platform/graphics/MediaPlayer.cpp: (WebCore::MediaPlayer::MediaPlayer):
1:33 PM Changeset in webkit [41815] by Beth Dakin
  • 3 edits in trunk/LayoutTests

2009-03-18 Beth Dakin <Beth Dakin>

Rubber-stamped by Dan Bernstein.

Updating test to reflect our joy.

  • fast/dom/offset-parent-positioned-and-inline-expected.txt:
  • fast/dom/offset-parent-positioned-and-inline.html:
1:30 PM Changeset in webkit [41814] by Beth Dakin
  • 3 edits
    2 adds in trunk

WebCore:

2009-03-18 Beth Dakin <Beth Dakin>

Reviewed by Dave Hyatt.

Fix for <rdar://problem/6636747> REGRESSION (Safari 4 PB-r41326):
Popup menu appears at the wrong location on page at http://
www.signonsandiego.com/

This was a regression from http://trac.webkit.org/changeset/40769,
which changed the base class of RenderInline to
RenderBoxModelObject rather than RenderBox.

  • rendering/RenderObject.cpp: (WebCore::RenderObject::offsetParent): offsetParent should return a RenderBoxModelObject rather than just a RenderBox, which is more restrictive.

LayoutTests:

2009-03-18 Beth Dakin <Beth Dakin>

Reviewed by Dave Hyatt.

Test for <rdar://problem/6636747> REGRESSION (Safari 4 PB-r41326):
Popup menu appears at the wrong location on page at http://
www.signonsandiego.com/

  • fast/dom/offset-parent-positioned-and-inline-expected.txt: Added.
  • fast/dom/offset-parent-positioned-and-inline.html: Added.
1:15 PM Changeset in webkit [41813] by andersca@apple.com
  • 4 edits in trunk

WebCore:

2009-03-18 Anders Carlsson <andersca@apple.com>

Reviewed by Darin Adler.

<rdar://problem/6504776>
CrashTracer: [USER] 188 crashes in Safari at com.apple.WebCore • WTF::HashTableIterator<WTF::RefPtr<WebCore::ResourceLoader>, ...

Return early in case calling the client ends up spinning the run loop and completing/cancelling the load.


  • loader/NetscapePlugInStreamLoader.cpp: (WebCore::NetscapePlugInStreamLoader::didCancel):

WebKit/mac:

2009-03-18 Anders Carlsson <andersca@apple.com>

Reviewed by Darin Adler.

<rdar://problem/6504776>
CrashTracer: [USER] 188 crashes in Safari at com.apple.WebCore • WTF::HashTableIterator<WTF::RefPtr<WebCore::ResourceLoader>, ...


If the m_pluginView member was zeroed out as a result of making a call into the plug-in, the pluginFunctionCallDepth would be off
causing the plug-in never to be stopped. Simplify the code by using a RAII object.

  • Plugins/WebBaseNetscapePluginStream.mm: (PluginStopDeferrer::PluginStopDeferrer): (PluginStopDeferrer::~PluginStopDeferrer): (WebNetscapePluginStream::startStream): (WebNetscapePluginStream::wantsAllStreams): (WebNetscapePluginStream::destroyStream):
1:08 PM Changeset in webkit [41812] by levin@chromium.org
  • 1 edit
    1 add in trunk/WebCore

2009-03-18 David Levin <levin@chromium.org>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=24664
Upstreaming v8 collection.h

No change in behavior, so no test.

  • bindings/v8/V8Collection.h: Added.
1:02 PM Changeset in webkit [41811] by weinig@apple.com
  • 5 edits in trunk

WebCore:

2009-03-18 Sam Weinig <sam@webkit.org>

Reviewed by Simon Fraser.

Fix for https://bugs.webkit.org/show_bug.cgi?id=23966
REGRESSION: Custom arrow navigation functionality doesn't work at dropular.net
<rdar://problem/6589657>

Match the CSSOM spec for getClientRects and getBoundingClientRect by
returning a 0x0 rect at the correct top/left position for empty inline
boxes.

  • rendering/RenderInline.cpp: (WebCore::RenderInline::absoluteRects): (WebCore::RenderInline::absoluteQuads):

LayoutTests:

2009-03-18 Sam Weinig <sam@webkit.org>

Reviewed by Simon Fraser.

Update test results for https://bugs.webkit.org/show_bug.cgi?id=23966
REGRESSION: Custom arrow navigation functionality doesn't work at dropular.net
<rdar://problem/6589657>

  • fast/dom/getBoundingClientRect-expected.txt:
  • fast/dom/getClientRects-expected.txt:
11:02 AM Changeset in webkit [41810] by ap@webkit.org
  • 1 edit
    20 adds in trunk/LayoutTests

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=24670
Add tests for XHR progress events during cross-origin requests.

  • http/tests/xmlhttprequest/redirect-cross-origin-2-expected.txt: Added.
  • http/tests/xmlhttprequest/redirect-cross-origin-2.html: Added.
  • http/tests/xmlhttprequest/redirect-cross-origin-expected.txt: Added.
  • http/tests/xmlhttprequest/redirect-cross-origin-post-expected.txt: Added.
  • http/tests/xmlhttprequest/redirect-cross-origin-post-sync-expected.txt: Added.
  • http/tests/xmlhttprequest/redirect-cross-origin-post-sync.html: Added.
  • http/tests/xmlhttprequest/redirect-cross-origin-post.html: Added.
  • http/tests/xmlhttprequest/redirect-cross-origin-sync-double-expected.txt: Added.
  • http/tests/xmlhttprequest/redirect-cross-origin-sync-double.html: Added.
  • http/tests/xmlhttprequest/redirect-cross-origin-sync-expected.txt: Added.
  • http/tests/xmlhttprequest/redirect-cross-origin-sync.html: Added.
  • http/tests/xmlhttprequest/redirect-cross-origin.html: Added.
  • http/tests/xmlhttprequest/simple-cross-origin-denied-events-expected.txt: Added.
  • http/tests/xmlhttprequest/simple-cross-origin-denied-events-post-expected.txt: Added.
  • http/tests/xmlhttprequest/simple-cross-origin-denied-events-post-sync-expected.txt: Added.
  • http/tests/xmlhttprequest/simple-cross-origin-denied-events-post-sync.html: Added.
  • http/tests/xmlhttprequest/simple-cross-origin-denied-events-post.html: Added.
  • http/tests/xmlhttprequest/simple-cross-origin-denied-events-sync-expected.txt: Added.
  • http/tests/xmlhttprequest/simple-cross-origin-denied-events-sync.html: Added.
  • http/tests/xmlhttprequest/simple-cross-origin-denied-events.html: Added.
10:41 AM Changeset in webkit [41809] by levin@chromium.org
  • 2 edits in trunk/WebCore

2009-03-18 David Levin <levin@chromium.org>

Reviewed by Eric Seidel.

Match the changes done for windows in the bug
https://bugs.webkit.org/show_bug.cgi?id=24530.

No change in behavior, so no test.

  • rendering/RenderThemeChromiumLinux.cpp: (WebCore::RenderThemeChromiumLinux::extraDefaultStyleSheet):
8:58 AM Changeset in webkit [41808] by krit@webkit.org
  • 2 edits in trunk/WebCore

2009-03-18 Dirk Schulze <krit@webkit.org>

Reviewed by Holger Freyther.

Added support for SVG's stroked texts to Qt.

[Qt] SVGs stroked text support missing
https://bugs.webkit.org/show_bug.cgi?id=24262

  • platform/graphics/qt/FontQt.cpp: (WebCore::Font::drawComplexText):
1:15 AM Changeset in webkit [41807] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Fix fast/encoding/char-decoding.html with recent ICU.

Reviewed by Alexey Proskuryakov.

  • platform/text/TextCodecICU.cpp:

(WebCore::TextCodecICU::decode): Look for gb18030 case-insensitively,
as newer versions of ICU use GB18030 as the canonical name.

12:24 AM Changeset in webkit [41806] by oliver@apple.com
  • 6 edits
    1 add in trunk

<rdar://problem/6692138> REGRESSION (Safari 4): Incorrect function return value when using IE "try ... finally" memory leak work-around (24654)
<https://bugs.webkit.org/show_bug.cgi?id=24654>

Reviewed by Cameron Zwarich.

If the return value for a function is in a local register we need
to copy it before executing any finalisers, otherwise it is possible
for the finaliser to clobber the result.

Note: See TracTimeline for information about the timeline view.