Timeline


and

04/30/09:

21:15 Changeset [43111] by bdakin@apple.com
  • 2 edits in trunk/WebCore

2009-04-30 Beth Dakin <bdakin@apple.com>

Reviewed by Oliver Hunt.

Fix for <rdar://problem/6841106> REGRESSION: Crash drag selecting
To Do's in a Mail note (excessive recursion in mouse drag tracking)

Oliver's recent change to dragging seems to have revealed a long-
time bug in passSubframeEventToSubframe where we did not set
m_sendingEventToSubview to true before calling
handleMouseMoveEvent() in the NSMouseMoved case. This patch fixes
that and adds ASSERTions around all of the places where
m_sendingEventToSubview is set to true and then false to make sure
we are not trampling its state.

  • page/mac/EventHandlerMac.mm: (WebCore::EventHandler::eventLoopHandleMouseDragged): (WebCore::EventHandler::eventLoopHandleMouseUp): (WebCore::EventHandler::passSubframeEventToSubframe): (WebCore::EventHandler::passWheelEventToWidget):
21:10 Changeset [43110] by barraclough@apple.com
  • 3 edits
    3 adds in trunk

JavaScriptCore:

2009-04-30 Gavin Barraclough <barraclough@apple.com>

Reviewed by Geoff Garen.

Bug fix for rdar:/68455379. If a case-insensitive regex contains
a character class containing a range with an upper bound of \uFFFF
the parser will infinite-loop whist adding other-case characters
for characters in the range that do have another case.

  • yarr/RegexCompiler.cpp: (JSC::Yarr::CharacterClassConstructor::putRange):

LayoutTests:

2009-04-30 Gavin Barraclough <barraclough@apple.com>

Reviewed by Geoff Garen.

Add layout test for rdar:/68455379.

  • fast/js/regexp-range-bound-ffff-expected.txt: Added.
  • fast/js/regexp-range-bound-ffff.html: Added.
  • fast/js/resources/regexp-range-bound-ffff.js: Added.
20:55 Changeset [43109] by eric.carlson@apple.com
  • 5 edits
    3 adds in trunk

2009-04-30 Eric Carlson <eric.carlson@apple.com>

Reviewed by Simon Fraser.


MediaDocument falls back to plug-in unnecessarily
https://bugs.webkit.org/show_bug.cgi?id=25504
<rdar://problem/6844702>


Don't allow harmless media types to cause a MediaDocument to fall
back to PluginDocument.

Test: media/video-document-types.html

  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::disableUnsupportedTracks): add timecode, timecode 64, odsm, and sdsm to allowed track types.
  • platform/graphics/win/QTMovieWin.cpp: (QTMovieWin::disableUnsupportedTracks): Ditto.

2009-04-30 Eric Carlson <eric.carlson@apple.com>

Reviewed by Simon Fraser.

MediaDocument falls back to plug-in unnecessarily
https://bugs.webkit.org/show_bug.cgi?id=25504
<rdar://problem/6844702>

Test case for MediaDocument with previously unsupported track
types. Disabled on Windows for now until https://bugs.webkit.org/show_bug.cgi?id=25220
has been fixed.


  • media/content/counting.mp4: Added.
  • media/video-document-types-expected.txt: Added.
  • media/video-document-types.html: Added.
  • platform/win/Skipped:
20:48 Changeset [43108] by kevino@webkit.org
  • 5 edits in trunk

wx build fix, define ENABLE_JAVASCRIPT_DEBUGGER and remove JSCore dynamic library references.

18:57 Changeset [43107] by mrowe@apple.com
  • 4 edits
    3 deletes in trunk

Roll out r43106 as it introduced four regression test failures.

17:06 Changeset [43106] by darin@apple.com
  • 4 edits
    5 adds in trunk

WebCore:

2009-04-30 Darin Adler <darin@apple.com>

Reviewed by Brady Eidson.

Bug 25491: WebFrame leak when a subframe removes itself
https://bugs.webkit.org/show_bug.cgi?id=25491
rdar://problem/6833859

Test: fast/loading/subframe-removes-itself.html

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::checkLoadCompleteForThisFrame): When isStopping is true, treat load as complete, even if isLoadingInAPISense is still returning true.

LayoutTests:

2009-04-30 Darin Adler <darin@apple.com>

Reviewed by Brady Eidson.

Bug 25491: WebFrame leak when a subframe removes itself
https://bugs.webkit.org/show_bug.cgi?id=25491
rdar://problem/6833859

  • fast/loading: Added.
  • fast/loading/resources: Added.
  • fast/loading/resources/subframe-that-removes-itself.html: Added.
  • fast/loading/subframe-removes-itself-expected.txt: Added.
  • fast/loading/subframe-removes-itself.html: Added.
  • platform/gtk/Skipped: Skip loading tests.
16:55 Changeset [43105] by barraclough@apple.com
  • 3 edits in trunk/JavaScriptCore

2009-04-30 Gavin Barraclough <barraclough@apple.com>

Reviewed by Oliver Hunt.

OPCODE_SAMPLING without CODEBLOCK_SAMPLING is currently broken,
since SamplingTool::Sample::isNull() checks the m_codeBlock
member (which is always null without CODEBLOCK_SAMPLING).

Restructure the checks so make this work again.

  • bytecode/SamplingTool.cpp: (JSC::SamplingTool::doRun):
  • bytecode/SamplingTool.h: (JSC::SamplingTool::Sample::isNull):
16:36 Changeset [43104] by mjs@apple.com
  • 4 edits in trunk/JavaScriptCore

2009-04-30 Maciej Stachowiak <mjs@apple.com>

Reviewed by Gavin Barraclough.

  • Concatenate final three strings in simple replace case at one go

~0.2% SunSpider speedup

  • runtime/StringPrototype.cpp: (JSC::stringProtoFuncReplace): Use new replaceRange helper instead of taking substrings and concatenating three strings.
  • runtime/UString.cpp: (JSC::UString::replaceRange): New helper function.
  • runtime/UString.h:
15:58 Changeset [43103] by ggaren@apple.com
  • 10 edits in trunk/JavaScriptCore

2009-04-30 Geoffrey Garen <ggaren@apple.com>

Rubber Stamped by Gavin Barraclough.


Changed JSValueEncodedAsPtr* => EncodedJSValuePtr to support a non-pointer
encoding for JSValuePtrs.

  • API/APICast.h: (toJS):
  • bytecompiler/BytecodeGenerator.h: (JSC::BytecodeGenerator::JSValueHashTraits::constructDeletedValue): (JSC::BytecodeGenerator::JSValueHashTraits::isDeletedValue):
  • interpreter/Register.h: (JSC::Register::):
  • jit/JIT.cpp: (JSC::):
  • jit/JIT.h:
  • jit/JITCode.h: (JSC::):
  • jit/JITStubs.cpp: (JSC::JITStubs::cti_op_add): (JSC::JITStubs::cti_op_pre_inc): (JSC::JITStubs::cti_op_get_by_id_generic): (JSC::JITStubs::cti_op_get_by_id): (JSC::JITStubs::cti_op_get_by_id_second): (JSC::JITStubs::cti_op_get_by_id_self_fail): (JSC::JITStubs::cti_op_get_by_id_proto_list): (JSC::JITStubs::cti_op_get_by_id_proto_list_full): (JSC::JITStubs::cti_op_get_by_id_proto_fail): (JSC::JITStubs::cti_op_get_by_id_array_fail): (JSC::JITStubs::cti_op_get_by_id_string_fail): (JSC::JITStubs::cti_op_instanceof): (JSC::JITStubs::cti_op_del_by_id): (JSC::JITStubs::cti_op_mul): (JSC::JITStubs::cti_op_call_NotJSFunction): (JSC::JITStubs::cti_op_resolve): (JSC::JITStubs::cti_op_construct_NotJSConstruct): (JSC::JITStubs::cti_op_get_by_val): (JSC::JITStubs::cti_op_get_by_val_string): (JSC::JITStubs::cti_op_get_by_val_byte_array): (JSC::JITStubs::cti_op_sub): (JSC::JITStubs::cti_op_lesseq): (JSC::JITStubs::cti_op_negate): (JSC::JITStubs::cti_op_resolve_base): (JSC::JITStubs::cti_op_resolve_skip): (JSC::JITStubs::cti_op_resolve_global): (JSC::JITStubs::cti_op_div): (JSC::JITStubs::cti_op_pre_dec): (JSC::JITStubs::cti_op_not): (JSC::JITStubs::cti_op_eq): (JSC::JITStubs::cti_op_lshift): (JSC::JITStubs::cti_op_bitand): (JSC::JITStubs::cti_op_rshift): (JSC::JITStubs::cti_op_bitnot): (JSC::JITStubs::cti_op_mod): (JSC::JITStubs::cti_op_less): (JSC::JITStubs::cti_op_neq): (JSC::JITStubs::cti_op_urshift): (JSC::JITStubs::cti_op_bitxor): (JSC::JITStubs::cti_op_bitor): (JSC::JITStubs::cti_op_call_eval): (JSC::JITStubs::cti_op_throw): (JSC::JITStubs::cti_op_next_pname): (JSC::JITStubs::cti_op_typeof): (JSC::JITStubs::cti_op_is_undefined): (JSC::JITStubs::cti_op_is_boolean): (JSC::JITStubs::cti_op_is_number): (JSC::JITStubs::cti_op_is_string): (JSC::JITStubs::cti_op_is_object): (JSC::JITStubs::cti_op_is_function): (JSC::JITStubs::cti_op_stricteq): (JSC::JITStubs::cti_op_nstricteq): (JSC::JITStubs::cti_op_to_jsnumber): (JSC::JITStubs::cti_op_in): (JSC::JITStubs::cti_op_del_by_val): (JSC::JITStubs::cti_vm_throw):
  • jit/JITStubs.h:
  • runtime/JSValue.h: (JSC::JSValuePtr::encode): (JSC::JSValuePtr::decode):
15:37 Changeset [43102] by dglazkov@chromium.org
  • 2 edits in trunk/WebCore

2009-04-30 Dimitri Glazkov <dglazkov@chromium.org>

Reviewed by Eric Seidel.

Add CHROMIUM guard to unfork Document.cpp

  • dom/Document.cpp: (WebCore::Document::setFocusedNode): Added guard.
15:27 Changeset [43101] by ddkilzer@apple.com
  • 3 edits in trunk/WebCore

Use OwnPtr<HTMLParserQuirks> for m_parserQuirks

Reviewed by Geoff Garen.

  • html/HTMLParser.cpp: (WebCore::HTMLParser::~HTMLParser): Removed unneeded code.
  • html/HTMLParser.h: Made m_parserQuirks an OwnPtr.
14:51 Changeset [43100] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

2009-04-30 Gavin Barraclough <barraclough@apple.com>

Reviewed by Oliver "Abandon Ship!" Hunt.

Fix a leak in Yarr.

All Disjunctions should be recorded in RegexPattern::m_disjunctions,
so that they can be freed at the end of compilation - copyDisjunction
is failing to do so.

  • yarr/RegexCompiler.cpp: (JSC::Yarr::RegexPatternConstructor::copyDisjunction):
14:45 Changeset [43099] by ojan@chromium.org
  • 6 edits
    3 adds
    2 deletes in trunk/LayoutTests

2009-04-30 Ojan Vafai <ojan@chromium.org>

Reviewed by Eric Seidel.

Fix tests that depend on Mac-specific metrics now that text controls
are sized according to font metrics (r43007).

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

  • fast/forms/textarea-appearance-wrap-expected.txt:
  • fast/forms/textarea-appearance-wrap.html: This test only really cares that a line-wrap character is submitted with forms that have the appropriate wrap value and not with other forms. So checking that the submit contents have a line-wrap character, instead of looking for it in a specific place, is sufficient. This gets around that we wrap in a different place in some cases on Mac and Windows.
  • fast/forms/textarea-hard-linewrap-expected.txt: Removed.
  • fast/forms/textarea-hard-linewrap.html: Removed. This test is redundant with fast/forms/textarea-appearance-wrap.html.
  • fast/replaced/table-percent-height-expected.txt:
  • fast/replaced/table-percent-height-text-controls-expected.txt: Added.
  • fast/replaced/table-percent-height-text-controls.html: Added.
  • fast/replaced/table-percent-height.html: Pulled out the cases in table-percent-height.html that had text controls and moved them into their own test. The new test doesn't assert specific metrics, it just spits them out. This allows us to have different expected results for Windows and Mac without having "FAIL" messages in the expected results. It's not perfect, but it seems better than any alternatives I could come up with. We do still print "FAIL" if the results don't match any platform.
  • platform/win/Skipped:
  • platform/win/fast/replaced/table-percent-height-text-controls-expected.txt: Added.
14:39 Changeset [43098] by timothy@apple.com
  • 2 edits in tags/Safari-6530.8/WebCore

Merge r43040.

14:38 Changeset [43097] by timothy@apple.com
  • 7 edits in tags/Safari-6530.8

Merge r43039.

14:38 Changeset [43096] by oliver@apple.com
  • 3 edits in trunk/JavaScriptCore

Add function to CallFrame for dumping the current JS caller

Reviewed by Gavin Barraclough.

Added debug only method CallFrame::dumpCaller() that provide the call location
of the deepest currently executing JS function.

14:36 Changeset [43095] by timothy@apple.com
  • 3 edits in tags/Safari-6530.8/WebCore

Merge r43026.

14:34 Changeset [43094] by timothy@apple.com
  • 2 edits in tags/Safari-6530.8/WebCore

Merge r42945.

14:28 Changeset [43093] by dglazkov@chromium.org
  • 1 edit in trunk/WebCore/ChangeLog

Fixed ChangeLog entry, added comments.

14:26 Changeset [43092] by ddkilzer@apple.com
  • 16 edits
    1 add in trunk

Provide a mechanism to create a quirks delegate for HTMLParser

Reviewed by David Hyatt.

WebCore:

No tests since there is no change in behavior.

HTMLParserQuirks.h defines an abstract base class that may be
extended as needed. The ChromeClient::createHTMLParserQuirks()
factory method should be used to return an HTMLParserQuirks
subclassed object when needed.

  • WebCore.xcodeproj/project.pbxproj: Added HTMLParserQuirks.h.
  • html/HTMLParser.cpp: (WebCore::HTMLParser::HTMLParser): Initialize m_parserQuirks using ChromeClient::createHTMLParserQuirks(). (WebCore::HTMLParser::~HTMLParser): Delete m_parserQuirks if set. (WebCore::HTMLParser::reset): Call HTMLParserQuirks::reset() if m_parserQuirks is set. (WebCore::HTMLParser::insertNode): Call HTMLParserQuirks::shouldInsertNode() if m_parserQuirks is set, and return early if it returns false. (WebCore::HTMLParser::popBlock): Call HTMLParserQuirks::shouldPopBlock() if m_parserQuirks is set, and return early if it returns false.
  • html/HTMLParser.h: Added m_parserQuirks.
  • html/HTMLParserQuirks.h: Added. (WebCore::HTMLParserQuirks::HTMLParserQuirks): (WebCore::HTMLParserQuirks::~HTMLParserQuirks):
  • loader/EmptyClients.h: (WebCore::EmptyChromeClient::createHTMLParserQuirks): Added.
  • page/ChromeClient.h: (WebCore::ChromeClient::createHTMLParserQuirks): Added.

WebKit/gtk:

  • WebCoreSupport/ChromeClientGtk.h: (WebKit::ChromeClient::createHTMLParserQuirks): Added. The default implementation of this factory method returns 0.

WebKit/mac:

  • WebCoreSupport/WebChromeClient.h: (WebChromeClient::createHTMLParserQuirks): Added. The default implementation of this factory method returns 0.

WebKit/qt:

  • WebCoreSupport/ChromeClientQt.h: (WebCore::ChromeClientQt::createHTMLParserQuirks): Added. The default implementation of this factory method returns 0.

WebKit/win:

  • WebCoreSupport/WebChromeClient.h: (WebChromeClient::createHTMLParserQuirks): Added. The default implementation of this factory method returns 0.

WebKit/wx:

  • WebKitSupport/ChromeClientWx.h: (WebCore::ChromeClientWx::createHTMLParserQuirks): Added. The default implementation of this factory method returns 0.
14:20 Changeset [43091] by dglazkov@chromium.org
  • 3 edits in trunk/WebCore

2009-04-30 Dimitri Glazkov <dglazkov@chromium.org>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=25492
Add JSC guards around node wrapper cache calls. These are not used by V8.

  • dom/Document.cpp: (WebCore::Document::~Document): Added JSC guard.
  • dom/Node.cpp: (WebCore::Node::setDocument): Ditto and removed JSDOMBinding include.
14:18 Changeset [43090] by mjs@apple.com
  • 2 edits in trunk/JavaScriptCore

2009-04-30 Maciej Stachowiak <mjs@apple.com>

Reviewed by Geoff Garen.

  • make BaseStrings have themselves as a base, instead of nothing, to remove common branches

~0.7% SunSpider speedup

  • runtime/UString.h: (JSC::UString::Rep::Rep): For the constructor without a base, set self as base instead of null. (JSC::UString::Rep::baseString): Just read m_baseString - no more branching.
14:11 Changeset [43089] by timothy@apple.com
  • 4 edits in trunk

Versioning.

14:11 Changeset [43088] by dglazkov@chromium.org
  • 2 edits in trunk/WebCore

2009-04-30 Dimitri Glazkov <dglazkov@chromium.org>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=25493
Remove debugger/profiler custom method stubs, catching V8 bindings up to
http://trac.webkit.org/changeset/43072.

  • bindings/v8/custom/V8InspectorControllerCustom.cpp: Removed method stubs.
14:02 Changeset [43087] by barraclough@apple.com
  • 3 edits in trunk/JavaScriptCore

2009-04-30 Gavin Barraclough <barraclough@apple.com>

Reviewed by Oliver Hunt.

Two quick improvements to SamplingFlags mechanism.

SamplingFlags::ScopedFlag class to provide support for automagically
clearing a flag as it goes out of scope, and add a little more detail
to the output generated by the tool.

  • bytecode/SamplingTool.cpp: (JSC::SamplingFlags::stop):
  • bytecode/SamplingTool.h: (JSC::SamplingFlags::ScopedFlag::ScopedFlag): (JSC::SamplingFlags::ScopedFlag::~ScopedFlag):
13:59 Changeset [43086] by timothy@apple.com
  • 1 copy in tags/Safari-6530.8

New tag.

13:50 Changeset [43085] by simon.fraser@apple.com
  • 3 edits
    2 adds in trunk

2009-04-30 Simon Fraser <simon.fraser@apple.com>

Reviewed by Darin Adler

<rdar://problem/6839338> VoiceOver does not take into account transforms when drawing outlines

Use transform-aware quad methods when computing the boundingBoxRect for
an AccessibilityRenderObject. The code follows RenderObject::absoluteBoundingBoxRect().

Test: accessibility/transformed-element.html

  • page/AccessibilityRenderObject.cpp: (WebCore::AccessibilityRenderObject::boundingBoxRect):
13:47 Changeset [43084] by aroben@apple.com
  • 5 edits in trunk/JavaScriptCore

Restore build event steps that were truncated in r43082

Rubber-stamped by Steve Falkenburg.

  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCoreCommon.vsprops:
  • JavaScriptCore.vcproj/jsc/jscCommon.vsprops:
  • JavaScriptCore.vcproj/testapi/testapiCommon.vsprops: Re-copied the command lines for the build events from the pre-r43082 .vcproj files.
  • JavaScriptCore.vcproj/jsc/jsc.vcproj: Removed an unnecessary attribute.
13:38 Changeset [43083] by jmalonzo@webkit.org
  • 2 edits in trunk

2009-04-30 Jan Michael Alonzo <jmalonzo@webkit.org>

Unreviewed build GTK build fix

  • configure.ac: typo fix - javascript_debugger should be enable_javascript_debugger
13:24 Changeset [43082] by aroben@apple.com
  • 5 edits
    4 adds in trunk/JavaScriptCore

Move settings from .vcproj files to .vsprops files within the JavaScriptCore directory

Moving the settings to a .vsprops file means that we will only have to
change a single setting to affect all configurations, instead of one
setting per configuration.

Reviewed by Steve Falkenburg.

  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:
  • JavaScriptCore.vcproj/WTF/WTF.vcproj:
  • JavaScriptCore.vcproj/jsc/jsc.vcproj:
  • JavaScriptCore.vcproj/testapi/testapi.vcproj: Moved settings from these files to the new .vsprops files. Note that testapi.vcproj had a lot of overrides of default settings that were the same as the defaults, which I've removed.
13:18 Changeset [43081] by dglazkov@chromium.org
  • 3 edits in trunk/WebCore

2009-04-30 Dimitri Glazkov <dglazkov@chromium.org>

Unreviewed, Chromium build fix.

Catch up to ScriptObject changes from http://trac.webkit.org/changeset/42512.

  • bindings/v8/ScriptObject.cpp: (WebCore::ScriptGlobalObject::get): Renamed from getObject. (WebCore::ScriptGlobalObject::remove): Added.
  • bindings/v8/ScriptObject.h: Ditto.
13:02 Changeset [43080] by dglazkov@chromium.org
  • 2 edits in trunk/WebCore

2009-04-30 Dimitri Glazkov <dglazkov@chromium.org>

Unreviewed, build fix.

https://bugs.webkit.org/show_bug.cgi?id=25470
Extend the cover of ENABLE_JAVASCRIPT_DEBUGGER to profiler.

  • WebCore.pro: Fix copy-paste error.
12:55 Changeset [43079] by dglazkov@chromium.org
  • 3 edits in trunk/WebCore

2009-04-30 Dimitri Glazkov <dglazkov@chromium.org>

Unreviewed, build fix.

https://bugs.webkit.org/show_bug.cgi?id=25470
Extend the cover of ENABLE_JAVASCRIPT_DEBUGGER to profiler.

  • GNUmakefile.am: Add ENABLE_JAVASCRIPT_DEBUGGER definitions.
  • WebCore.pro: Ditto.
12:49 Changeset [43078] by ariya@webkit.org
  • 2 edits in trunk/WebKit/qt

2009-04-30 Ariya Hidayat <ariya.hidayat@nokia.com>

Unreviewed build fix after r43072.

  • Api/qwebframe.cpp: (qt_drt_setJavaScriptProfilingEnabled): Enclose with JAVASCRIPT_DEBUGGER.
12:49 Changeset [43077] by ariya@webkit.org
  • 3 edits in trunk/WebKit/qt

2009-04-30 Ariya Hidayat <ariya.hidayat@nokia.com>

Unreviewed build fix after r43063.

  • WebCoreSupport/ChromeClientQt.cpp: (WebCore::ChromeClientQt::addMessageToConsole):
  • WebCoreSupport/ChromeClientQt.h:
12:48 Changeset [43076] by darin@apple.com
  • 4 edits
    1 copy
    5 adds in trunk/LayoutTests

2009-04-30 Darin Adler <darin@apple.com>

Reviewed by Sam Weinig.

Add a first text iterator test. We'll add more later.
This test includes some known expected failures. At some point we'll fix those.

  • editing/text-iterator: Added.
  • editing/text-iterator/basic-iteration-expected.txt: Added.
  • editing/text-iterator/basic-iteration.html: Added.
  • editing/text-iterator/resources: Added.
  • editing/text-iterator/resources/TEMPLATE.html: Copied from editing/selection/resources/TEMPLATE.html.
  • editing/text-iterator/resources/basic-iteration.js: Added.
  • platform/gtk/Skipped: Skip editing/text-iterator since only Mac DumpRenderTree has this at the moment.
  • platform/qt/Skipped: Ditto.
  • platform/win/Skipped: Ditto.
12:40 Changeset [43075] by eric@webkit.org
  • 4 edits in trunk/WebCore

Reviewed by Darin Adler.

Change Position to pre-compute the AnchorType and store it.
Also store a "legacy editing mode" bit on the Position.
https://bugs.webkit.org/show_bug.cgi?id=24763

The old Postion(node, offset) constructor creates legacy positions
but there are new constructors to create explicitly anchored positions
which the next patch will use.

Once we fix all the "position-fixup" functions (like rangeCompliantEquivalent) to
only affect legacy positions, we will be able to distinguish
between [table, 1] and [table, after] in the code correctly!

  • WebCore.base.exp:
  • WebCore.xcodeproj/project.pbxproj:
  • dom/Position.cpp: (WebCore::Position::Position): (WebCore::Position::moveToPosition): (WebCore::Position::moveToOffset): (WebCore::Position::anchorTypeForLegacyEditingPosition): (WebCore::Position::element):
  • dom/Position.h: (WebCore::Position::): (WebCore::Position::Position): (WebCore::Position::anchorType): (WebCore::Position::deprecatedEditingOffset):
12:02 Changeset [43074] by dglazkov@chromium.org
  • 2 edits in trunk

2009-04-30 Dimitri Glazkov <dglazkov@chromium.org>

Unreviewed, build fix.

https://bugs.webkit.org/show_bug.cgi?id=25470
Extend the cover of ENABLE_JAVASCRIPT_DEBUGGER to profiler.

  • configure.ac: Add autoconfig options, missed in the first commit.
11:57 Changeset [43073] by weinig@apple.com
  • 5 edits
    2 adds in trunk

WebCore:

2009-04-30 Sam Weinig <sam@webkit.org>

Reviewed by Alexey Proskuryakov.

Fix bug where the WorkerLocation and WorkerNavigator wrappers would be
collected even if the WorkerContext is still alive.

Test: fast/workers/worker-context-gc.html

  • bindings/js/JSWorkerContextCustom.cpp: (WebCore::JSWorkerContext::mark):
  • workers/WorkerContext.cpp: (WebCore::WorkerContext::WorkerContext): (WebCore::WorkerContext::completeURL): (WebCore::WorkerContext::location):
  • workers/WorkerContext.h: (WebCore::WorkerContext::optionalNavigator): (WebCore::WorkerContext::optionalLocation):

LayoutTests:

2009-04-30 Sam Weinig <sam@webkit.org>

Reviewed by Alexey Proskuryakov.

Test for bug where the WorkerLocation and WorkerNavigator wrappers would be
collected even if the WorkerContext is still alive.

  • fast/workers/worker-context-gc-expected.txt: Added.
  • fast/workers/worker-context-gc.html: Added.
11:28 Changeset [43072] by dglazkov@chromium.org
  • 16 edits in trunk

JavaScriptCore:

2009-04-30 Dimitri Glazkov <dglazkov@chromium.org>

Reviewed by Timothy Hatcher.

https://bugs.webkit.org/show_bug.cgi?id=25470
Extend the cover of ENABLE_JAVASCRIPT_DEBUGGER to profiler.

  • Configurations/FeatureDefines.xcconfig: Added ENABLE_JAVASCRIPT_DEBUGGER define.

WebCore:

2009-04-30 Dimitri Glazkov <dglazkov@chromium.org>

Reviewed by Timothy Hatcher.

https://bugs.webkit.org/show_bug.cgi?id=25470
Extend the cover of ENABLE_JAVASCRIPT_DEBUGGER to profiler.

  • Configurations/FeatureDefines.xcconfig: Added ENABLE_JAVASCRIPT_DEBUGGER.
  • WebCore.vcproj/build-generated-files.sh: Ditto.
  • bindings/js/JSConsoleCustom.cpp: Added ENABLE(JAVASCRIPT_DEBUGGER) guard.
  • bindings/js/JSDOMWindowBase.cpp: Ditto.
  • bindings/js/JSInspectorControllerCustom.cpp: Ditto.
  • inspector/InspectorController.cpp: Moved profiler/debugger methods under

ENABLE(JAVASCRIPT_DEBUGGER) flag.

  • inspector/InspectorController.h: Ditto.
  • inspector/InspectorController.idl: Added ENABLE(JAVASCRIPT_DEBUGGER) guard.
  • page/Console.cpp: Replaced USE(JSC) with ENABLE(JAVASCRIPT_DEBUGGER) guard.

WebKit/mac:

2009-04-30 Dimitri Glazkov <dglazkov@chromium.org>

Reviewed by Timothy Hatcher.

https://bugs.webkit.org/show_bug.cgi?id=25470
Extend the cover of ENABLE_JAVASCRIPT_DEBUGGER to profiler.

  • Configurations/FeatureDefines.xcconfig: Added ENABLE_JAVASCRIPT_DEBUGGER define.

WebKitTools:

2009-04-30 Dimitri Glazkov <dglazkov@chromium.org>

Reviewed by Timothy Hatcher.

https://bugs.webkit.org/show_bug.cgi?id=25470
Extend the cover of ENABLE_JAVASCRIPT_DEBUGGER to profiler.

  • Scripts/build-webkit: Added ENABLE_JAVASCRIPT_DEBUGGER define.
11:28 Changeset [43071] by mrowe@apple.com
  • 2 edits in tags/Safari-6530.7.2/WebCore

Merge r43040.

11:27 Changeset [43070] by mrowe@apple.com
  • 3 edits in tags/Safari-6530.7.2/WebCore

Merge r43026.

11:09 Changeset [43069] by mrowe@apple.com
  • 4 edits in tags/Safari-6530.7.2

Versioning.

11:02 Changeset [43068] by mrowe@apple.com
  • 1 copy in tags/Safari-6530.7.2

New tag.

10:57 Changeset [43067] by kdecker@apple.com
  • 3 edits in trunk/WebKit/mac

Reviewed by Anders Carlson.

<rdar://problem/6823049>


Fix an issue where some plug-ins would cause the application icon to constantly bounce
up and down in the Dock.


  • Plugins/Hosted/NetscapePluginHostProxy.h: Change m_placeholderWindow ivar to be a subclass of NSWindow, WebPlaceholderModalWindow.
  • Plugins/Hosted/NetscapePluginHostProxy.mm: Added WebPlaceholderModalWindow NSWindow subclass. (-[WebPlaceholderModalWindow _wantsUserAttention]): Prevent NSApp from calling requestUserAttention: when the window is shown modally, even if the app is inactive. (WebKit::NetscapePluginHostProxy::beginModal): NSWindow -> WebPlaceholderModalWindow.
10:27 Changeset [43066] by dglazkov@chromium.org
  • 2 edits in trunk/WebCore

2009-04-30 Dimitri Glazkov <dglazkov@chromium.org>

Unreviewed, build fix.

https://bugs.webkit.org/show_bug.cgi?id=25275
This patch snuck in a re-definition of a local.

  • bindings/v8/custom/V8DOMWindowCustom.cpp: (WebCore::NAMED_PROPERTY_GETTER): Removed re-definition.
10:16 Changeset [43065] by weinig@apple.com
  • 2 edits in trunk/WebCore

2009-04-30 Sam Weinig <sam@webkit.org>

Reviewed by Mark Rowe.

Fix a leak introduced yesterday. Don't allocate a ScheduledAction
if the toString()ing throws an exception.

  • bindings/js/ScheduledAction.cpp: (WebCore::ScheduledAction::create):
09:55 Changeset [43064] by ap@webkit.org
  • 4 edits
    2 adds in trunk

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=25379
<rdar://problem/6809460> REGRESSION (r41772): Selecting a bank in American Express Pay Bill fails

Test: fast/forms/multiple-selected-options-innerHTML.html

This problem was caused by an inconsistency at when Node::instertedIntoTree() is called.
For normal HTML parsing, it is called immediately after an element is inserted, but for
innerHTML, it is only called after the whole subtree is inserted into a document.

It may make sense to harmonize these cases one day, but for now, I only made the minimal
changes necessary to fix the bug.

  • html/HTMLSelectElement.cpp: (WebCore::HTMLSelectElement::insertedIntoTree):
  • html/HTMLSelectElement.h: Recalculate list items when a SELECT element is inserted. OPTION elements cannot decide which one to keep selected themselves, because their logic assumes normal parsing, with insertedIntoTree() called after each element is inserted.
09:41 Changeset [43063] by dglazkov@chromium.org
  • 16 edits in trunk

WebCore:

2009-04-30 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=25342
Add MessageSource and MessageLevel parameters to the ChromeClient::addMessageToConsole.

Chromium async/out-of-process version of WebInspector is currently not based on
InspectorController. The reason was that we did not want to interfere with
the unforking effort, yet wanted to experiment. So we came up with these
agents concept that basically mimic InspectorController, but separating 'agent'
nature from the 'transport'. Now that InspectorController is unforked, I am
planning to bring these concepts into the WebKit land and use what we have in
Chromium as a proof of concept / experimental playground.

  • loader/EmptyClients.h: added MessageSource and MessageLevel parameters. (WebCore::EmptyChromeClient::addMessageToConsole): ditto
  • page/ChromeClient.h: ditto
  • page/Console.cpp: Used new method signature. (WebCore::Console::addMessage): ditto

WebKit/gtk:

2009-04-30 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=25342
Add MessageSource and MessageLevel parameters to the ChromeClient::addMessageToConsole.

  • WebCoreSupport/ChromeClientGtk.cpp: (WebKit::ChromeClient::addMessageToConsole):
  • WebCoreSupport/ChromeClientGtk.h:

WebKit/mac:

2009-04-30 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=25342
Add MessageSource and MessageLevel parameters to the ChromeClient::addMessageToConsole.

  • WebCoreSupport/WebChromeClient.h:
  • WebCoreSupport/WebChromeClient.mm: (WebChromeClient::addMessageToConsole):

WebKit/win:

2009-04-30 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=25342
Add MessageSource and MessageLevel parameters to the ChromeClient::addMessageToConsole.

  • WebCoreSupport/WebChromeClient.cpp: (WebChromeClient::addMessageToConsole):
  • WebCoreSupport/WebChromeClient.h:

WebKit/wx:

2009-04-30 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=25342
Add MessageSource and MessageLevel parameters to the ChromeClient::addMessageToConsole.

  • WebKitSupport/ChromeClientWx.cpp: (WebCore::ChromeClientWx::addMessageToConsole):
  • WebKitSupport/ChromeClientWx.h:
09:36 Changeset [43062] by mitz@apple.com
  • 9 edits in trunk

WebCore:

Reviewed by Simon Fraser.

Reverted r42987.

  • platform/graphics/cg/ImageCG.cpp: (WebCore::BitmapImage::draw):

LayoutTests:

Reviewed by Simon Fraser.

  • platform/mac/svg/W3C-SVG-1.1/render-groups-03-t-expected.checksum:
  • platform/mac/svg/W3C-SVG-1.1/render-groups-03-t-expected.png:
  • platform/mac/svg/custom/focus-ring-expected.checksum:
  • platform/mac/svg/custom/focus-ring-expected.png:
  • platform/mac/svg/custom/image-small-width-height-expected.checksum:
  • platform/mac/svg/custom/image-small-width-height-expected.png:
09:20 Changeset [43061] by beidson@apple.com
  • 8 edits
    2 adds in trunk

WebCore:

2009-04-30 Brady Eidson <beidson@apple.com>

Reviewed by Maciej Stachowiak

<rdar://problem/3785248> http://www.greekembassy.org/ gets in an infinite reload loop
https://bugs.webkit.org/show_bug.cgi?id=21193

Some websites use for attributes in script elements to specify events that the script
should be run for. One example is greekembassy.org which has the following in their <head>:

<script for=window event=onresize>

location.reload(false);

</script>

Since we don't support the full for attribute syntax, we would just blissfully ignore the
condition and execute the code unconditionally. This caused breakage on multiple real-world
sites besides greekembassy.org.

https://bugs.webkit.org/show_bug.cgi?id=16915 and <rdar://problem/4471751> track adding full
support for the for attribute in scripts. In the meantime it's best to not execute them
unconditionally.

Test: fast/dom/HTMLScriptElement/script-for-attribute-unexpected-execution.html

  • dom/ScriptElement.cpp: (WebCore::ScriptElementData::shouldExecuteAsJavaScript): After all other checks have passed, only return true if there is no for attribute in the script element.
  • dom/ScriptElement.h:
  • html/HTMLScriptElement.cpp: (WebCore::HTMLScriptElement::forAttributeValue): Return the attribute value, if any.
  • html/HTMLScriptElement.h:
  • svg/SVGScriptElement.cpp: (WebCore::SVGScriptElement::forAttributeValue): Return an empty string (like the other attribute getters do for SVGScriptElement)
  • svg/SVGScriptElement.h:

LayoutTests:

2009-04-30 Brady Eidson <beidson@apple.com>

Reviewed by Maciej Stachowiak

Test for <rdar://problem/3785248> http://www.greekembassy.org/ gets in an infinite reload loop
https://bugs.webkit.org/show_bug.cgi?id=21193

  • fast/dom/HTMLScriptElement/script-for-attribute-unexpected-execution-expected.txt: Added.
  • fast/dom/HTMLScriptElement/script-for-attribute-unexpected-execution.html: Added.
08:56 Changeset [43060] by kov@webkit.org
  • 2 edits in trunk/LayoutTests

2009-04-30 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Skip one more video test that sometimes fails.

  • platform/gtk/Skipped:
08:43 Changeset [43059] by aroben@apple.com
  • 2 edits in trunk/LayoutTests

Add a sometimes-timing-out test to the Windows skipped file

See Bug 25483: http/tests/navigation/redirect-cycle.html sometimes
times out on Windows
<https://bugs.webkit.org/show_bug.cgi?id=25483>

  • platform/win/Skipped: Added http/tests/navigation/redirect-cycle.html.
08:01 Changeset [43058] by hausmann@webkit.org
  • 4 edits in trunk/WebKit/qt

2009-04-30 Kenneth Rohde Christiansen <kenneth.christiansen@openbossa.org>

Reviewed by Simon Hausmann.

Implement encloseContentsWith and rename wrap to encloseWith.

Also in encloseWith, make sure that you cannot enclose with
non-enclosable elements.

07:56 Changeset [43057] by kov@webkit.org
  • 2 edits in trunk/LayoutTests

2009-04-30 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Skip two more tests that I had accidentaly generated results for,
locally.

  • platform/gtk/Skipped:
07:06 Changeset [43056] by kov@webkit.org
  • 2 edits in trunk/LayoutTests

2009-04-30 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Also disable new tests, and tests that are using stderr.

  • platform/gtk/Skipped:
06:12 Changeset [43055] by kov@webkit.org
  • 2 edits in trunk/LayoutTests

2009-04-30 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Disabling more video tests that seem to come and go.

  • platform/gtk/Skipped:
05:06 Changeset [43054] by kov@webkit.org
  • 2 edits in trunk/LayoutTests

2009-04-30 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Hopefully last try making the bot green.

  • platform/gtk/Skipped:
04:09 Changeset [43053] by ariya@webkit.org
  • 2 edits in trunk/LayoutTests

2009-04-30 Ariya Hidayat <ariya.hidayat@nokia.com>

Rubber-stamped by Simon Hausmann.

[Qt] Exclude canvas-putImageData.html from the skipped list since
Canvas's putImageData is implemented in r41355 already.

  • platform/qt/Skipped: Removed canvas-putImageData.html.
03:26 Changeset [43052] by hausmann@webkit.org
  • 8 edits
    3 adds in trunk

2009-04-30 Holger Hans Peter Freyther <zecke@selfish.org>

Reviewed by Darin Adler.

The Qt API exposes a global history patch CSSStyleSelector to make API consumers work again

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

The QtWebKit port made the mistake of exposing a global history. This broke
with the addition of PageGroup and LinkHash. This needs to be repaired
for Qt4.5.

Add a function to LinkHash.cpp that is resolving a URL. Use this
function from within CSSStyleSelector to forward the url to the
QWebHistoryInterface API.

It is sad that there is a path within visitedLinkHash which is now
doing a memcpy, it is sad to add a PLATFORM(QT) define to CSSStyleSelector
and using QtWebKit types within WebCore is a layering violation as well.

PageGroup::setShouldTrackVisitedLinks is currently not enabled. For
Qt4.6 a second version of the QWebHistoryInterface is going to be
added which will fix things up.

03:08 Changeset [43051] by hausmann@webkit.org
  • 3 edits in trunk/WebCore

Fix the Qt build.

03:04 Changeset [43050] by ariya@webkit.org
  • 2 edits in trunk/WebKit/qt

2009-04-30 Ariya Hidayat <ariya.hidayat@nokia.com>

Unreviewed build fix, after r43035.

Temporarily use Position::deprecatedEditingOffset().

  • Api/qwebpage.cpp: (QWebPage::inputMethodQuery):
03:00 Changeset [43049] by ariya@webkit.org
  • 2 edits in trunk/WebCore

2009-04-30 Ariya Hidayat <ariya.hidayat@nokia.com>

Unreview build fix after r43037.

Use MarkedArgumentBuffer instead of ArgList.

  • bridge/qt/qt_runtime.cpp: (JSC::Bindings::QtConnectionObject::execute):
02:15 Changeset [43048] by mjs@apple.com
  • 2 edits in trunk/JavaScriptCore

2009-04-30 Maciej Stachowiak <mjs@apple.com>

Reviewed by Alexey Proskuryakov.

  • speed up string concatenation by reorganizing some simple cases

0.7% SunSpider speedup

  • runtime/UString.cpp: (JSC::concatenate): Put fast case for appending a single character before the empty string special cases; streamline code a bit to delay computing values that are not needed in the fast path.
00:52 Changeset [43047] by barraclough@apple.com
  • 6 edits in trunk/JavaScriptCore

2009-04-30 Gavin Barraclough <barraclough@apple.com>

Reviewed by Maciej Stachowiak.

Add SamplingFlags mechanism.

This mechanism allows fine-grained JSC and JavaScript program aware
performance measurement. The mechanism provides a set of 32 flags,
numbered #1..#32. Flag #16 is initially set, and all other flags
are cleared. Flags may be set and cleared from within

Enable by setting ENABLE_SAMPLING_FLAGS to 1 in wtf/Platform.h.
Disabled by default, no performance impact. Flags may be modified
by calling SamplingFlags::setFlag() and SamplingFlags::clearFlag()
from within JSC implementation, or by calling setSamplingFlag() and
clearSamplingFlag() from JavaScript.

The flags are sampled with a frequency of 10000Hz, and the highest
set flag in recorded, allowing multiple events to be measured (with
the highest flag number representing the highest priority).

Disabled by default; no performance impact.

  • JavaScriptCore.exp:
  • bytecode/SamplingTool.cpp: (JSC::SamplingFlags::sample): (JSC::SamplingFlags::start): (JSC::SamplingFlags::stop): (JSC::SamplingThread::threadStartFunc): (JSC::SamplingThread::start): (JSC::SamplingThread::stop): (JSC::ScopeSampleRecord::sample): (JSC::SamplingTool::doRun): (JSC::SamplingTool::sample): (JSC::SamplingTool::start): (JSC::SamplingTool::stop):
  • bytecode/SamplingTool.h: (JSC::SamplingFlags::setFlag): (JSC::SamplingFlags::clearFlag): (JSC::SamplingTool::SamplingTool):
  • jsc.cpp: (GlobalObject::GlobalObject): (functionSetSamplingFlag): (functionClearSamplingFlag): (runWithScripts):
  • wtf/Platform.h:

04/29/09:

23:47 Changeset [43046] by weinig@apple.com
  • 6 edits
    2 adds in trunk

WebCore:

2009-04-29 Sam Weinig <sam@webkit.org>

Reviewed by Oliver Hunt.

Fix for https://bugs.webkit.org/show_bug.cgi?id=25440
setTimeout should stringify non-string/non-function first arguments

Test: fast/dom/Window/setTimeout-string-argument.html

  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::setTimeout): (WebCore::JSDOMWindow::setInterval):
  • bindings/js/JSWorkerContextCustom.cpp: (WebCore::JSWorkerContext::setTimeout): (WebCore::JSWorkerContext::setInterval):
  • bindings/js/ScheduledAction.cpp: (WebCore::ScheduledAction::create):
  • bindings/js/ScheduledAction.h:

LayoutTests:

2009-04-29 Sam Weinig <sam@webkit.org>

Reviewed by Oliver Hunt.

Test for https://bugs.webkit.org/show_bug.cgi?id=25440
setTimeout should stringify non-string/non-function first arguments

  • fast/dom/Window/setTimeout-string-argument-expected.txt: Added.
  • fast/dom/Window/setTimeout-string-argument.html: Added.
20:38 Changeset [43045] by mrowe@apple.com
  • 3 edits in trunk/WebKit/mac

More build fixing after r43037.

  • Plugins/Hosted/NetscapePluginInstanceProxy.h:
  • Plugins/Hosted/NetscapePluginInstanceProxy.mm:

(WebKit::NetscapePluginInstanceProxy::invoke):
(WebKit::NetscapePluginInstanceProxy::invokeDefault):
(WebKit::NetscapePluginInstanceProxy::construct):
(WebKit::NetscapePluginInstanceProxy::demarshalValues):

20:03 Changeset [43044] by mitz@apple.com
  • 13 edits
    2 adds in trunk

WebCore:

Reviewed by Simon Fraser.

  • WebCore part of <rdar://problem/6609509> Select All and then Delete should put Mail editing back into the same state as a new message

Test: editing/deleting/in-visibly-empty-root.html

  • WebCore.base.exp: Exported VisibleSelection::isAll(bool).
  • editing/SelectionController.h: (WebCore::SelectionController::isAll): Added. Calls through to VisibleSelection.
  • editing/TypingCommand.cpp: (WebCore::TypingCommand::makeEditableRootEmpty): Added. Removes all children of the root editable element the selection is in, other than a placeholder. Returns true iff it did anything. (WebCore::TypingCommand::deleteKeyPressed): When there is only a single visible position in the root editable element, but it has children other than a placeholder, remove those children.
  • editing/TypingCommand.h:
  • editing/VisiblePosition.h: Added a StayInEditableContent enum and a FIXME.
  • editing/VisibleSelection.cpp: (WebCore::VisibleSelection::isAll): Added. Returns whether the selection encompasses all visible positions, either in the document or in the editable root.
  • editing/VisibleSelection.h:

WebKit/mac:

Reviewed by Simon Fraser.

  • WebKit part of <rdar://problem/6609509> Select All and then Delete should put Mail editing back into the same state as a new message
  • WebView/WebView.mm: (-[WebView _selectionIsCaret]): Added. (-[WebView _selectionIsAll]): Added. Returns whether the selection encompasses the entire document.
  • WebView/WebViewPrivate.h:

LayoutTests:

Reviewed by Simon Fraser.

  • test and updated results for <rdar://problem/6609509> Select All and then Delete should put Mail editing back into the same state as a new message
  • editing/deleting/in-visibly-empty-root-expected.txt: Added.
  • editing/deleting/in-visibly-empty-root.html: Added.
  • platform/mac/editing/deleting/delete-at-start-or-end-expected.txt:
19:19 Changeset [43043] by weinig@apple.com
  • 3 edits in trunk/JavaScriptCore

2009-04-29 Sam Weinig <sam@webkit.org>

Another attempt to fix the windows build.

19:12 Changeset [43042] by weinig@apple.com
  • 2 edits in trunk/WebCore

2009-04-29 Sam Weinig <sam@webkit.org>

Fix style nit.

  • editing/visible_units.cpp: (WebCore::getLeafBoxesInLogicalOrder):
19:10 Changeset [43041] by weinig@apple.com
  • 3 edits in trunk/JavaScriptCore

2009-04-29 Sam Weinig <sam@webkit.org>

Try and fix the windows build.

19:02 Changeset [43040] by hyatt@apple.com
  • 2 edits in trunk/WebCore

2009-04-29 David Hyatt <hyatt@apple.com>

Reviewed by Dan Bernstein.

For now just drop the deferred layout on the floor, since it causes an infinite hang in mail. Even
trying to schedule a relayout for later leaves you in a state where you hit the needsLayout painting
assertions.

Basically what Mail is doing is crazy, and we can't support it other than to just drop the last layout
and not do it (which is basically what was happening before).

  • page/FrameView.cpp: (WebCore::FrameView::setContentsSize): (WebCore::FrameView::needsLayout):
18:35 Changeset [43039] by justin.garcia@apple.com
  • 7 edits in trunk

WebCore:

2009-04-29 Douglas Davidson <ddavidso@apple.com>

Reviewed by Justin Garcia.

<rdar://problem/6836921> Mail exhibits issues with text checking, e.g. menu items not always validated correctly


Updates to the text checking code to enable text checking even if spellchecking is turned off
and fix an off-by-one error in selection handling.

  • editing/Editor.cpp: (WebCore::Editor::markMisspellingsAfterTypingToPosition): (WebCore::Editor::markAllMisspellingsAndBadGrammarInRanges): (WebCore::Editor::markMisspellingsAndBadGrammar):
  • editing/Editor.h:
  • editing/TypingCommand.cpp: (WebCore::TypingCommand::markMisspellingsAfterTyping):

WebKit/mac:

2009-04-29 Douglas Davidson <ddavidso@apple.com>

Reviewed by Justin Garcia.


<rdar://problem/6836921> Mail exhibits issues with text checking, e.g. menu items not always validated correctly

  • WebView/WebHTMLView.mm: (-[WebHTMLView validateUserInterfaceItemWithoutDelegate:]): (-[WebHTMLView orderFrontSubstitutionsPanel:]):
  • WebView/WebView.mm: (-[WebView validateUserInterfaceItemWithoutDelegate:]):
18:34 Changeset [43038] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

2009-04-29 Gavin Barraclough <barraclough@apple.com>

Reviewed by Oliver "Peg-Leg" Hunt.

Coallesce input checking and reduce futzing with the index position
between alternatives and iterations of the main loop of a regex,
when run in YARR.

Consider the following regex: /foo|bar/


Prior to this patch, this will be implemented something like this pseudo-code description:


loop:

check_for_available_input(3) this increments the index by 3, for the first alterantive.

if (available) { test "foo" }

decrement_index(3)
check_for_available_input(3) this increments the index by 3, for the second alterantive.

if (available) { test "bar" }

decrement_index(3)
check_for_available_input(1) can we loop again?

if (available) { goto loop }

With these changes it will look more like this:

check_for_available_input(3) this increments the index by 3, for the first alterantive.
if (!available) { goto fail }

loop:

test "foo"
test "bar"
check_for_available_input(1) can we loop again?

if (available) { goto loop }

fail:

This gives about a 5% gain on v8-regex, no change on Sunspider.

  • yarr/RegexJIT.cpp: (JSC::Yarr::RegexGenerator::TermGenerationState::linkAlternativeBacktracksTo): (JSC::Yarr::RegexGenerator::generateDisjunction):
18:21 Changeset [43037] by oliver@apple.com
  • 46 edits in trunk

Clean up ArgList to be a trivial type

Reviewed by Gavin Barraclough

Separate out old ArgList logic to handle buffering and marking arguments
into a distinct MarkedArgumentBuffer type. ArgList becomes a trivial
struct of a pointer and length.

18:16 Changeset [43036] by eric@webkit.org
  • 2 edits in trunk/WebCore

No review, build fix only.

Fix m_offset uses added while I wasn't looking.

  • editing/visible_units.cpp: (WebCore::logicalStartPositionForLine): (WebCore::logicalEndPositionForLine):
18:09 Changeset [43035] by eric@webkit.org
  • 31 edits in trunk

Reviewed by Darin Adler.

Make m_offset private and change all callers to use deprecatedEditingOffset().
https://bugs.webkit.org/show_bug.cgi?id=25472

Per Darin's suggestion, this was just a simple search replace
of .m_offset with .deprecatedEditingOffset().

There was one change in InsertParagraphSeparatorCommand::doApply to use
Position::moveToOffset(0) instead of .m_offset = 0;

  • dom/Position.cpp: (WebCore::Position::rendersInDifferentPosition): (WebCore::Position::leadingWhitespacePosition):
  • dom/Position.h: (WebCore::Position::deprecatedEditingOffset): (WebCore::Position::moveToOffset): (WebCore::operator==):
  • dom/PositionIterator.h: (WebCore::PositionIterator::PositionIterator):
  • dom/Range.cpp: (WebCore::Range::create): (WebCore::Range::compareBoundaryPoints):
  • editing/ApplyStyleCommand.cpp: (WebCore::ApplyStyleCommand::applyRelativeFontStyleChange): (WebCore::ApplyStyleCommand::applyInlineStyleToRange): (WebCore::ApplyStyleCommand::removeInlineStyle): (WebCore::ApplyStyleCommand::nodeFullySelected): (WebCore::ApplyStyleCommand::nodeFullyUnselected): (WebCore::ApplyStyleCommand::splitTextAtStartIfNeeded): (WebCore::ApplyStyleCommand::splitTextAtEndIfNeeded): (WebCore::ApplyStyleCommand::splitTextElementAtStartIfNeeded): (WebCore::ApplyStyleCommand::splitTextElementAtEndIfNeeded): (WebCore::ApplyStyleCommand::mergeStartWithPreviousIfIdentical): (WebCore::ApplyStyleCommand::mergeEndWithNextIfIdentical): (WebCore::ApplyStyleCommand::joinChildTextNodes):
  • editing/BreakBlockquoteCommand.cpp: (WebCore::BreakBlockquoteCommand::doApply):
  • editing/CompositeEditCommand.cpp: (WebCore::CompositeEditCommand::insertNodeAt): (WebCore::CompositeEditCommand::positionOutsideTabSpan): (WebCore::CompositeEditCommand::rebalanceWhitespaceAt): (WebCore::CompositeEditCommand::prepareWhitespaceAtPositionForSplit): (WebCore::CompositeEditCommand::deleteInsignificantText): (WebCore::CompositeEditCommand::moveParagraphs): (WebCore::CompositeEditCommand::breakOutOfEmptyMailBlockquotedParagraph):
  • editing/DeleteSelectionCommand.cpp: (WebCore::updatePositionForNodeRemoval): (WebCore::updatePositionForTextRemoval): (WebCore::DeleteSelectionCommand::handleGeneralDelete): (WebCore::DeleteSelectionCommand::fixupWhitespace):
  • editing/Editor.cpp: (WebCore::Editor::setComposition): (WebCore::Editor::advanceToNextMisspelling): (WebCore::Editor::getCompositionSelection):
  • editing/InsertLineBreakCommand.cpp: (WebCore::InsertLineBreakCommand::doApply):
  • editing/InsertParagraphSeparatorCommand.cpp: (WebCore::InsertParagraphSeparatorCommand::doApply):
  • editing/InsertTextCommand.cpp: (WebCore::InsertTextCommand::performTrivialReplace): (WebCore::InsertTextCommand::input): (WebCore::InsertTextCommand::insertTab):
  • editing/MoveSelectionCommand.cpp: (WebCore::MoveSelectionCommand::doApply):
  • editing/ReplaceSelectionCommand.cpp: (WebCore::ReplaceSelectionCommand::doApply):
  • editing/SelectionController.cpp: (WebCore::SelectionController::debugRenderer):
  • editing/TextIterator.cpp: (WebCore::TextIterator::rangeFromLocationAndLength):
  • editing/TypingCommand.cpp: (WebCore::TypingCommand::deleteKeyPressed): (WebCore::TypingCommand::forwardDeleteKeyPressed):
  • editing/VisiblePosition.cpp: (WebCore::VisiblePosition::characterAfter): (WebCore::VisiblePosition::debugPosition): (WebCore::makeRange): (WebCore::setStart): (WebCore::setEnd):
  • editing/VisibleSelection.cpp: (WebCore::VisibleSelection::toNormalizedRange): (WebCore::makeSearchRange): (WebCore::VisibleSelection::debugPosition): (WebCore::VisibleSelection::showTreeForThis):
  • editing/htmlediting.cpp: (WebCore::comparePositions): (WebCore::rangeCompliantEquivalent):
  • editing/visible_units.cpp: (WebCore::previousBoundary): (WebCore::nextBoundary): (WebCore::startPositionForLine): (WebCore::startOfLine): (WebCore::endPositionForLine): (WebCore::nextLinePosition): (WebCore::startOfParagraph): (WebCore::endOfParagraph): (WebCore::logicalStartPositionForLine): (WebCore::logicalEndPositionForLine):
  • page/AccessibilityObject.cpp: (WebCore::updateAXLineStartForVisiblePosition):
  • page/AccessibilityRenderObject.cpp: (WebCore::AccessibilityRenderObject::indexForVisiblePosition):
  • page/DOMSelection.cpp: (WebCore::DOMSelection::anchorOffset): (WebCore::DOMSelection::focusOffset): (WebCore::DOMSelection::baseOffset): (WebCore::DOMSelection::extentOffset):
  • page/EventHandler.cpp: (WebCore::EventHandler::handleMousePressEventSingleClick):
  • page/Frame.cpp: (WebCore::Frame::selectionLayoutChanged):
  • page/mac/AccessibilityObjectWrapper.mm: (textMarkerForVisiblePosition): (visiblePositionForTextMarker):
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::indexForVisiblePosition):
  • rendering/RenderTreeAsText.cpp: (WebCore::writeSelection):
18:07 Changeset [43034] by eric@webkit.org
  • 2 edits
    1 add
    4 deletes in trunk/LayoutTests

Reviewed by Oliver Hunt.

Re-write image-clipped-hit.svg to be dumpAsText() (and clean up the code to be less insane)

  • platform/mac/svg/custom/image-clipped-hit-expected.checksum: Removed.
  • platform/mac/svg/custom/image-clipped-hit-expected.png: Removed.
  • platform/mac/svg/custom/image-clipped-hit-expected.txt: Removed.
  • platform/qt/svg/custom/image-clipped-hit-expected.txt: Removed.
  • svg/custom/image-clipped-hit.svg:
17:51 Changeset [43033] by kov@webkit.org
  • 2 edits in trunk/LayoutTests

2009-04-29 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Second try at making the bot green.

  • platform/gtk/Skipped:
17:48 Changeset [43032] by mitz@apple.com
  • 7 edits
    2 adds in trunk

WebCore:

2009-04-29 Xiaomei Ji <xji@chromium.org>

Reviewed by Dan Bernstein.

Fix https://bugs.webkit.org/show_bug.cgi?id=24168
RTL: Home/End key does not behave correctly in mixed bidi text in RTL document

Test: editing/selection/home-end.html

  • editing/SelectionController.cpp: (WebCore::SelectionController::modifyExtendingForward): Change calling endOfLine() to logicalEndOfLine() when granularity is LineBoundary. (WebCore::SelectionController::modifyMovingForward): Change calling endOfLine() to logicalEndOfLine() when granularity is LineBoundary (WebCore::SelectionController::modifyExtendingBackward): Change calling startOfLine() to logicalStartOfLine() when granularity is LineBoundary. (WebCore::SelectionController::modifyMovingBackward): Change calling startOfLine() to logicalStartOfLine() when granularity is LineBoundary.
  • editing/visible_units.cpp: (WebCore::getLeafBoxesInLogicalOrder): Added. Reconstruct leaf boxes in logical order. (WebCore::getLogicalStartBoxAndNode): Added. (WebCore::getLogicalEndBoxAndNode): Added. (WebCore::logicalStartPositionForLine): Added. Similar to startPositionForLine. (WebCore::logicalStartOfLine): Added. Similar to startOfLine. (WebCore::logicalEndPositionForLine): Added. Similar to endPositionForLine. (WebCore::inSameLogicalLine): Added. (WebCore::logicalEndOfLine): Added. Similar to endOfLine.
  • editing/visible_units.h:

LayoutTests:

2009-04-29 Xiaomei Ji <xji@chromium.org>

Reviewed by Dan Bernstein.

Fix https://bugs.webkit.org/show_bug.cgi?id=24168
RTL: Home/End key does not behave correctly in mixed bidi text in RTL document

  • editing/selection/extend-selection-expected.txt: Added result for extend in lineBoundary.
  • editing/selection/extend-selection.html: Added test cases and script for extend in lineBoundary.
  • editing/selection/home-end-expected.txt: Added.
  • editing/selection/home-end.html: Added.
17:24 Changeset [43031] by ojan@chromium.org
  • 5 edits in trunk/LayoutTests

2009-04-29 Ojan Vafai <ojan@chromium.org>

Reviewed by NOBODY (OOPS!).

Fix this test to only use a setTimeout in IE.
It doesn't need a timeout in WebKit and the
timeout requires using waitUntilDone/notifyDone,
which doesn't play nicely with JS tests.

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

  • fast/forms/resources/textarea-metrics.js: (assertTextareaMetrics):
  • fast/forms/textarea-metrics-expected.txt:
  • platform/win/fast/forms/textarea-metrics-expected.txt:
  • platform/win/Skipped:
16:52 Changeset [43030] by ojan@chromium.org
  • 1 edit
    1 move
    1 add in trunk/LayoutTests

Ojan Vafai <ojan@chromium.org>

Reviewed by Eric Seidel.

Put this file in the right place to fix the bot failure.

  • platform/mac-tiger/fast/forms/text-control-intrinsic-widths-expected.txt: Renamed from LayoutTests/platform/mac-tiger/text-control-intrinsic-widths-expected.txt.
16:50 Changeset [43029] by ojan@chromium.org
  • 2 edits in trunk/LayoutTests

2009-04-29 Ojan Vafai <ojan@chromium.org>

Reviewed by Eric Seidel.

Skip fast/forms/textarea-metrics.html. It's printing the TEST PASSED
before the test is finished. There's a race with any JS tests that
use setTimeout. Fortunately, this test doesn't need the setTimeout,
so it's easy to fix.

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

  • platform/win/Skipped:
16:34 Changeset [43028] by eric@webkit.org
  • 7 edits
    2 deletes in trunk

2009-04-29 Eric Seidel <eric@webkit.org>

No review, rolling out a patch.

Revert http://trac.webkit.org/changeset/43019

New tests failed because I removed the tabs from them
when landing (since we avoid tabs in WebKit files).
I couldn't tell if the new results were correct with
spaces or not.

  • editing/SelectionController.cpp: (WebCore::SelectionController::modifyExtendingForward): (WebCore::SelectionController::modifyMovingForward): (WebCore::SelectionController::modifyExtendingBackward): (WebCore::SelectionController::modifyMovingBackward):
  • editing/visible_units.cpp:
  • editing/visible_units.h:
16:27 Changeset [43027] by eric@webkit.org
  • 7 edits
    2 deletes in trunk

2009-04-29 Eric Seidel <eric@webkit.org>

No review, rolling out a patch.

Revert 43020 as it caused accessibility/lists.html to fail
and no one is around to tell me if it's a progression or not.

  • page/AccessibilityObject.h: (WebCore::):
  • page/AccessibilityRenderObject.cpp: (WebCore::AccessibilityRenderObject::roleValue): (WebCore::AccessibilityRenderObject::canSetFocusAttribute):
  • page/mac/AccessibilityObjectWrapper.mm: (RoleEntry::):
16:21 Changeset [43026] by hyatt@apple.com
  • 3 edits in trunk/WebCore

2009-04-29 David Hyatt <hyatt@apple.com>

Reviewed by Dan Bernstein.

Fix for <rdar://problem/6828164>, Mail hits the !root->needsLayout() assert because it re-marks the FrameView for layout
when the FrameView is resized. This bug was exposed by the elimination of the separate WebKit layout
flag on Mac.


FrameView now defers setNeedsLayouts during size changes. Once all of the size changes are finished (including re-entrant ones
from WebDynamicScrollbarsView and ScrollView::updateScrollbars) we then do a layout if necessary.

  • page/FrameView.cpp: (WebCore::FrameView::FrameView): (WebCore::FrameView::setContentsSize): (WebCore::FrameView::layout): (WebCore::FrameView::needsLayout): (WebCore::FrameView::setNeedsLayout):
  • page/FrameView.h:
16:20 Changeset [43025] by hyatt@apple.com
  • 2 edits in trunk/WebKit/mac

2009-04-29 David Hyatt <hyatt@apple.com>

Reviewed by Dan Bernstein.

Fix a bug in the bounds checking for setNeedsLayout dirtying when a WebView's size changes. The superview
of the WebView was being incorrectly checked instead of the WebView itself.

  • WebView/WebView.mm: (-[WebView _boundsChanged]):
16:09 Changeset [43024] by ojan@chromium.org
  • 1 edit in trunk/LayoutTests/ChangeLog

Fix ChangeLog description.

16:04 Changeset [43023] by ojan@chromium.org
  • 1 edit
    1 add in trunk/LayoutTests

2009-04-29 Ojan Vafai <ojan@chromium.org>

Reviewed by Eric Seidel.

Add expected results for failing tests.

15:48 Changeset [43022] by eric@webkit.org
  • 3 edits in trunk/WebCore

2009-04-29 Antony Sargent <asargent@chromium.org>

Reviewed by Dimitri Glazkov and Eric Seidel.

Use AtomicString to speed up window and document interceptors.
https://bugs.webkit.org/show_bug.cgi?id=25275

  • bindings/v8/custom/V8DOMWindowCustom.cpp: (WebCore::NAMED_PROPERTY_GETTER):
  • bindings/v8/custom/V8HTMLDocumentCustom.cpp: (WebCore::NAMED_PROPERTY_DELETER): (WebCore::NAMED_PROPERTY_GETTER):
15:28 Changeset [43021] by mrowe@apple.com
  • 2 edits in trunk/WebKitTools

Update build.webkit.org configuration for the addition of the new GTK buildbot.

  • BuildSlaveSupport/build.webkit.org-config/config.json:
15:22 Changeset [43020] by eric@webkit.org
  • 7 edits
    2 adds in trunk

2009-04-29 Sankar Aditya Tanguturi <sankaraditya@gmail.com>

Reviewed by Jon Honeycutt.

Add a new accessibility role for list items. Part of
http://bugs.webkit.org/show_id.cgi?id=20013

  • win/AccessibleBase.cpp: (MSAARole): Return ROLE_SYSTEM_LISTITEM for list item role.
15:16 Changeset [43019] by eric@webkit.org
  • 7 edits
    2 adds in trunk

2009-04-29 Xiaomei Ji <xji@chromium.org>

Reviewed by Dan Bernstein.

Fix https://bugs.webkit.org/show_bug.cgi?id=24168
RTL: Home/End key does not behave correctly in mixed bidi text in RTL document

Test: editing/selection/home-end.html

  • editing/SelectionController.cpp: (WebCore::SelectionController::modifyExtendingForward): Change calling endOfLine() to logicalEndOfLine() when granularity is LineBoundary. (WebCore::SelectionController::modifyMovingForward): Change calling endOfLine() to logicalEndOfLine() when granularity is LineBoundary (WebCore::SelectionController::modifyExtendingBackward): Change calling startOfLine() to logicalStartOfLine() when granularity is LineBoundary. (WebCore::SelectionController::modifyMovingBackward): Change calling startOfLine() to logicalStartOfLine() when granularity is LineBoundary.
  • editing/visible_units.cpp: (WebCore::getLeafBoxesInLogicalOrder): Added. Reconstruct leaf boxes in logical order. (WebCore::getLogicalStartBoxAndNode): Added. (WebCore::getLogicalEndBoxAndNode): Added. (WebCore::logicalStartPositionForLine): Added. Similar to startPositionForLine. (WebCore::logicalStartOfLine): Added. Similar to startOfLine. (WebCore::logicalEndPositionForLine): Added. Similar to endPositionForLine. (WebCore::inSameLogicalLine): Added. (WebCore::logicalEndOfLine): Added. Similar to endOfLine.
  • editing/visible_units.h:
15:14 Changeset [43018] by eric@webkit.org
  • 4 edits
    3 adds in trunk

2009-04-29 Sankar Aditya Tanguturi <sankaraditya@gmail.com>

Reviewed by Adele Peterson.

https://bugs.webkit.org/show_bug.cgi?id=23583
Populate role attribute for document objects correctly
on Windows platform.

  • AccessibleDocument.cpp: (AccessibleDocument::role): Return ROLE_SYSTEM_DOCUMENT for document objects on windows. (Matching Firefox).
  • AccessibleDocument.h: Add the role function prototype in header file.
15:11 Changeset [43017] by mrowe@apple.com
  • 2 edits in trunk/WebKitTools

Ignore the --use-remote-links-to-tests option for GTK as it leads to
perl errors being spewed at the end of run-webkit-tests.

  • Scripts/run-webkit-tests:
14:54 Changeset [43016] by mrowe@apple.com
  • 2 edits in trunk/WebKitTools

Implement test-result-archive for GTK.

  • BuildSlaveSupport/test-result-archive:
14:38 Changeset [43015] by eric@webkit.org
  • 4 edits
    4 adds in trunk/WebKit/qt

2009-03-27 Benjamin C Meyer <benjamin.meyer@torchmobile.com>

Reviewed by Simon Hausmann.

Update the page actions when a finishing loading even if the frame is
not the top frame such as when browsing inside of a website with frames.
https://bugs.webkit.org/show_bug.cgi?id=24890

  • WebCoreSupport/FrameLoaderClientQt.cpp: (WebCore::FrameLoaderClientQt::dispatchDidFinishLoad):
  • tests/qwebpage/frametest/frame_a.html: Added.
  • tests/qwebpage/frametest/index.html: Added.
  • tests/qwebpage/tst_qwebpage.cpp: (tst_QWebPage::backActionUpdate):
14:31 Changeset [43014] by eric@webkit.org
  • 6 edits in trunk

2009-04-29 Mike Belshe <mike@belshe.com>

Reviewed by Eric Seidel.

Update location while page is loading.
https://bugs.webkit.org/show_bug.cgi?id=21597

Tested by:

  • fast/dom/location-new-window-no-crash
  • fast/dom/Window/window-open-pending-url
  • page/Location.cpp:
14:28 Changeset [43013] by kov@webkit.org
  • 2 edits in trunk/LayoutTests

2009-04-29 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Rubber-stamped by Mark Rowe.

Disable failing tests to make the bot green for its new debut, so
that we can more easily track regressions.

  • platform/gtk/Skipped:
14:25 Changeset [43012] by eric@webkit.org
  • 1 edit
    3 adds in trunk/WebCore

2009-04-29 Feng Qian <feng@chromium.org>

Reviewed by Darin Adler.

Add Android port files to WebCore/platform.
https://bugs.webkit.org/show_bug.cgi?id=23296

  • platform/android: Added.
  • platform/android/ClipboardAndroid.cpp: Added.
  • platform/android/ClipboardAndroid.h: Added.
14:23 Changeset [43011] by eric@webkit.org
  • 1 edit
    4 adds in trunk/WebCore

2009-04-29 Feng Qian <feng@chromium.org>

Reviewed by Darin Adler.

Add Android-specific files to the WebCore/page directory.
https://bugs.webkit.org/show_bug.cgi?id=23295

  • page/android: Added.
  • page/android/DragControllerAndroid.cpp: Added.
  • page/android/EventHandlerAndroid.cpp: Added.
  • page/android/InspectorControllerAndroid.cpp: Added.
14:00 Changeset [43010] by eric@webkit.org
  • 7 edits
    2 adds in trunk/WebCore

2009-04-29 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Extract PageGroupLoadDeferrer from Chrome.cpp

  • GNUmakefile.am: added PageGroupLoadDeferrer
  • WebCore.pro: ditto
  • WebCore.scons: ditto
  • WebCore.vcproj/WebCore.vcproj: ditto
  • WebCore.xcodeproj/project.pbxproj: ditto
  • WebCoreSources.bkl: ditto
  • page/Chrome.cpp: PageGroupLoadDeferrer impl moved to a separate class.
  • page/PageGroupLoadDeferrer.cpp: Added. (WebCore::PageGroupLoadDeferrer::PageGroupLoadDeferrer): (WebCore::PageGroupLoadDeferrer::~PageGroupLoadDeferrer):
  • page/PageGroupLoadDeferrer.h: Added.
13:55 Changeset [43009] by kdecker@apple.com
  • 8 edits in trunk/WebKit/mac

Reviewed by Anders Carlson.

Allow WKN_CheckIfAllowedToLoadURL() to take an optional void* context parameter.


  • Plugins/WebNetscapeContainerCheckContextInfo.h:
  • Plugins/WebNetscapeContainerCheckContextInfo.mm: (-[WebNetscapeContainerCheckContextInfo initWithCheckRequestID:callbackFunc:context:]): (-[WebNetscapeContainerCheckContextInfo callback]): (-[WebNetscapeContainerCheckContextInfo context]):
  • Plugins/WebNetscapeContainerCheckPrivate.h:
  • Plugins/WebNetscapeContainerCheckPrivate.mm: (browserContainerCheckFuncs):
  • Plugins/WebNetscapePluginView.h:
  • Plugins/WebNetscapePluginView.mm: (-[WebNetscapePluginView checkIfAllowedToLoadURL:frame:callbackFunc:context:]): (-[WebNetscapePluginView _containerCheckResult:contextInfo:]):
  • Plugins/npapi.mm: (WKN_CheckIfAllowedToLoadURL):
13:43 Changeset [43008] by eric@webkit.org
  • 2 edits in trunk/JavaScriptCore

2009-04-29 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Sam Weinig.

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

Fix Qt build when ENABLE_JIT is explicitly set to 1
to overrule defaults.

13:31 Changeset [43007] by ojan@chromium.org
  • 558 edits
    1 copy
    191 adds in trunk

Reviewed by Dan Bernstein.

Make textarea and text input metrics more closely match IEs.

This involves:
-set text input width to size*avgCharWidth + maxCharWidth - avgCharWidth
-set textarea width to cols*avgCharWidth
-Make default CSS match IEs
-Correctly initializing m_avgCharWidth and m_maxCharWidth for each platform and SVG.

Those values for textarea and inputs were derived by doing a ton of manual
testing of IE's width values for various textareas and fonts. On Windows we match
IE exactly except for a couple fonts of the ~12 tested.

To get the average and max character width of a font, we do the following
for each platform:
-Win: TextMetrics expose avgCharWidth and maxCharWidth
-SVG: avgCharWidth = width of an '0', fallback on width of a space glyph, then m_xHeight

maxCharWidth = width of a 'W' for roman fonts, fallback on m_ascent

-Linux: avgCharWidth = width of an '0', fallback on m_xHeight

maxCharWidth = max of avgCharWidth and m_ascent

-Mac: Calculate the avgCharWidth and grab the maxCharWidth off the font.

If either one is non-positive, then calculate the value using the Linux approach.

Tests: fast/forms/text-control-intrinsic-widths.html

fast/forms/textarea-metrics.html
svg/custom/svg-fonts-in-text-controls.html

  • css/html4.css:
  • css/themeWin.css:
  • platform/graphics/SimpleFontData.cpp: (WebCore::SimpleFontData::SimpleFontData): (WebCore::SimpleFontData::initCharWidths):
  • platform/graphics/SimpleFontData.h: (WebCore::SimpleFontData::maxCharWidth): (WebCore::SimpleFontData::avgCharWidth):
  • platform/graphics/chromium/SimpleFontDataChromiumWin.cpp: (WebCore::SimpleFontData::platformCharWidthInit):
  • platform/graphics/chromium/SimpleFontDataLinux.cpp: (WebCore::SimpleFontData::platformCharWidthInit):
  • platform/graphics/gtk/SimpleFontDataGtk.cpp: (WebCore::SimpleFontData::platformCharWidthInit):
  • platform/graphics/gtk/SimpleFontDataPango.cpp: (WebCore::SimpleFontData::platformCharWidthInit):
  • platform/graphics/mac/SimpleFontDataMac.mm: (WebCore::SimpleFontData::platformCharWidthInit):
  • platform/graphics/qt/SimpleFontDataQt.cpp: (WebCore::SimpleFontData::platformCharWidthInit):
  • platform/graphics/win/SimpleFontDataCGWin.cpp: (WebCore::SimpleFontData::platformCharWidthInit):
  • platform/graphics/win/SimpleFontDataCairoWin.cpp: (WebCore::SimpleFontData::platformInit): (WebCore::SimpleFontData::platformCharWidthInit):
  • platform/graphics/win/SimpleFontDataWin.cpp: (WebCore::SimpleFontData::initGDIFont):
  • platform/graphics/wx/SimpleFontDataWx.cpp: (WebCore::SimpleFontData::platformCharWidthInit):
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::calcPrefWidths):
  • rendering/RenderTextControlMultiLine.cpp: (WebCore::RenderTextControlMultiLine::createInnerTextStyle):
  • rendering/RenderTextControlSingleLine.cpp: (WebCore::RenderTextControlSingleLine::preferredContentWidth):
13:25 Changeset [43006] by oliver@apple.com
  • 5 edits in trunk

Crash in profiler due to incorrect assuming displayName would be a string.

Reviewed by Steve Falkenburg

Fixed by adding a type guard.

13:23 Changeset [43005] by beidson@apple.com
  • 2 edits in trunk/WebCore

2009-04-29 Brady Eidson <beidson@apple.com>

Reviewed by Darin Adler.

  • bindings/objc/DOMCSS.mm: (kitClass): Added CSS_INITIAL to the switch statement, and changed to use the WebCore type and not the binding type.
13:21 Changeset [43004] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-04-29 John Abd-El-Malek <jam@google.com>

Reviewed by Dimitri Glazkov.

Reverted my previous change to add resetChromiumPluginCache().
https://bugs.webkit.org/show_bug.cgi?id=25318

  • plugins/chromium/PluginDataChromium.cpp:
13:21 Changeset [43003] by beidson@apple.com
  • 2 edits in trunk/WebCore

2009-04-29 Brady Eidson <beidson@apple.com>

Reviewed by Darin Adler

<rdar://problem/6573742> - Problem dragging standalone images from Safari to Mail

Roll out trac.webkit.org/changeset/35867 which fixed <rdar://problem/6153432>

Cloned <rdar://problem/6153432> to <rdar://problem/6839881> to find a better fix for that Tiger Mail bug.

  • loader/DocumentLoader.cpp: (WebCore::DocumentLoader::subresource): Remove the preload referenced check.
13:14 Changeset [43002] by eric@webkit.org
  • 3 edits in trunk/WebCore

2009-04-29 Alpha Lam <hclam@chromium.org>

Reviewed by Dimitri Glazkov.

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

Provide style sheet for UI controls of media elements for Chromium port.

  • rendering/RenderThemeChromiumWin.cpp: (WebCore::RenderThemeChromiumWin::extraMediaControlsStyleSheet):
13:12 Changeset [43001] by eric@webkit.org
  • 3 edits
    1 add in trunk/WebCore

2009-04-29 John Abd-El-Malek <jam@chromium.org>

Reviewed by Darin Adler.

Expose sudden termination to Chromium port.
https://bugs.webkit.org/show_bug.cgi?id=25457

  • platform/SuddenTermination.h:
  • platform/chromium/ChromiumBridge.h:
  • platform/chromium/SuddenTerminationChromium.cpp: Added. (WebCore::disableSuddenTermination): (WebCore::enableSuddenTermination):
13:00 Changeset [43000] by weinig@apple.com
  • 2 edits in trunk/WebCore

2009-04-29 Sam Weinig <sam@webkit.org>

Reviewed by Dimitri Glazkov.

Remove unused EventListeners.

  • page/DOMWindow.idl:
11:38 Changeset [42999] by hyatt@apple.com
  • 2 edits in trunk/WebKit/mac

2009-04-29 David Hyatt <hyatt@apple.com>

Reviewed by John Sullivan.

Fix for <rdar://problem/6835573>, Find Banner turns invisible when WebView is resized. Make sure
not to resize the interior views of a WebView in response to its bounds changing when not using
viewless WebKit. Auto-resizing rules were already in place to handle size adjustments for us.
Just mark as needing layout and do nothing else.

This does mean viewless WebKit is broken with the Find Banner, and that will likely require a
Safari change (using a new API that will enable clients to define the edges of the content area as offsets
from the sides of the WebView).

  • WebView/WebView.mm: (-[WebView _boundsChanged]):
10:41 Changeset [42998] by eric.carlson@apple.com
  • 2 edits in trunk/LayoutTests

2009-04-29 Eric Carlson <eric.carlson@apple.com>

Disable media/video-zoom-controls.html on Windows for now.

  • platform/win/Skipped: add media/video-zoom-controls.html.
10:38 Changeset [42997] by timothy@apple.com
  • 2 edits in trunk/WebCore

Reveal current execution line once SourceFrame content is loaded.

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

Reviewed by Timothy Hatcher.

  • inspector/front-end/SourceFrame.js: (WebInspector.SourceFrame.prototype._loaded):
10:27 Changeset [42996] by ariya@webkit.org
  • 4 edits in trunk

WebCore:

2009-04-29 Ariya Hidayat <ariya.hidayat@nokia.com>

Reviewed by Simon Fraser.

[Qt] Initialize GraphicsContext's and ImageBuffer's QPainter to match
the default values of canvas attributes.

  • platform/graphics/qt/ImageBufferQt.cpp: (WebCore::ImageBufferData::ImageBufferData):

LayoutTests:

2009-04-29 Ariya Hidayat <ariya.hidayat@nokia.com>

Reviewed by Simon Fraser.

Updated expected results after Qt's GraphicsContext fixes.

  • platform/qt/fast/canvas/set-colors-expected.txt:
07:56 QtWebKitTodo edited by ariya@webkit.org
(diff)
06:55 Changeset [42995] by ariya@webkit.org
  • 4 edits in trunk/WebKit/qt

2009-04-29 Ariya Hidayat <ariya.hidayat@nokia.com>

Reviewed by Simon Hausmann.

Implement QWebElement::evaluateScript.

  • Api/qwebelement.cpp: (setupScriptContext): (QWebElement::evaluateScript):
  • Api/qwebelement.h:
  • tests/qwebelement/tst_qwebelement.cpp: (tst_QWebElement::evaluateScript):
06:46 Changeset [42994] by hausmann@webkit.org
  • 7 edits in trunk/WebKit/qt

2009-04-29 Simon Hausmann <simon.hausmann@nokia.com>

Reviewed by Ariya Hidayat.

Replaced QWebElementCollection with QList<QWebElement>.

03:20 QtWebKitTodo edited by vestbo@webkit.org
(diff)
03:09 QtWebKitTodo edited by vestbo@webkit.org
(diff)

04/28/09:

23:49 QtWebKitContrib edited by hausmann@webkit.org
(diff)
23:38 Changeset [42993] by hausmann@webkit.org
  • 2 edits in trunk/WebCore

Fix the Qt build

23:31 Changeset [42992] by ap@webkit.org
  • 2 edits in trunk/LayoutTests

Fix the test for <https://bugs.webkit.org/show_bug.cgi?id=25420> - I broke it when
experimenting with other aspects of the issue, so it didn't check what it should have checked.

  • http/tests/xmlhttprequest/detaching-frame-2.html:
23:19 Changeset [42991] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptGlue

2009-04-28 Geoffrey Garen <ggaren@apple.com>

Rubber stamped by Beth Dakin.


Missed one.

  • UserObjectImp.cpp: (UserObjectImp::callAsFunction):
23:08 Changeset [42990] by ggaren@apple.com
  • 2 edits in trunk/WebCore

2009-04-28 Geoffrey Garen <ggaren@apple.com>

Rubber stamped by Beth Dakin.


Missed one.

  • bindings/scripts/CodeGeneratorJS.pm:
23:07 Changeset [42989] by ggaren@apple.com
  • 88 edits in trunk

JavaScriptCore:

2009-04-28 Geoffrey Garen <ggaren@apple.com>

Rubber stamped by Beth Dakin.


Removed scaffolding supporting dynamically converting between 32bit and
64bit value representations.

  • API/JSCallbackConstructor.cpp: (JSC::constructJSCallback):
  • API/JSCallbackFunction.cpp: (JSC::JSCallbackFunction::call):
  • API/JSCallbackObjectFunctions.h: (JSC::::construct): (JSC::::call):
  • bytecode/CodeBlock.cpp: (JSC::CodeBlock::dump):
  • bytecode/CodeBlock.h: (JSC::CodeBlock::getConstant):
  • bytecompiler/BytecodeGenerator.cpp: (JSC::BytecodeGenerator::emitEqualityOp):
  • interpreter/CallFrame.cpp: (JSC::CallFrame::thisValue):
  • interpreter/Interpreter.cpp: (JSC::Interpreter::callEval): (JSC::Interpreter::throwException): (JSC::Interpreter::createExceptionScope): (JSC::Interpreter::privateExecute): (JSC::Interpreter::retrieveArguments):
  • interpreter/Register.h: (JSC::Register::): (JSC::Register::Register): (JSC::Register::jsValue): (JSC::Register::marked): (JSC::Register::mark): (JSC::Register::i): (JSC::Register::activation): (JSC::Register::arguments): (JSC::Register::callFrame): (JSC::Register::codeBlock): (JSC::Register::function): (JSC::Register::propertyNameIterator): (JSC::Register::scopeChain): (JSC::Register::vPC):
  • jit/JITStubs.cpp: (JSC::JITStubs::cti_op_call_NotJSFunction): (JSC::JITStubs::cti_op_load_varargs): (JSC::JITStubs::cti_op_call_eval):
  • jsc.cpp: (functionPrint): (functionDebug): (functionRun): (functionLoad):
  • runtime/ArgList.h: (JSC::ArgList::at):
  • runtime/Arguments.cpp: (JSC::Arguments::copyToRegisters): (JSC::Arguments::fillArgList): (JSC::Arguments::getOwnPropertySlot):
  • runtime/ArrayConstructor.cpp: (JSC::constructArrayWithSizeQuirk):
  • runtime/ArrayPrototype.cpp: (JSC::arrayProtoFuncJoin): (JSC::arrayProtoFuncConcat): (JSC::arrayProtoFuncPush): (JSC::arrayProtoFuncSlice): (JSC::arrayProtoFuncSort): (JSC::arrayProtoFuncSplice): (JSC::arrayProtoFuncUnShift): (JSC::arrayProtoFuncFilter): (JSC::arrayProtoFuncMap): (JSC::arrayProtoFuncEvery): (JSC::arrayProtoFuncForEach): (JSC::arrayProtoFuncSome): (JSC::arrayProtoFuncReduce): (JSC::arrayProtoFuncReduceRight): (JSC::arrayProtoFuncIndexOf): (JSC::arrayProtoFuncLastIndexOf):
  • runtime/BooleanConstructor.cpp: (JSC::constructBoolean): (JSC::callBooleanConstructor):
  • runtime/DateConstructor.cpp: (JSC::constructDate): (JSC::dateParse): (JSC::dateUTC):
  • runtime/DatePrototype.cpp: (JSC::formatLocaleDate): (JSC::fillStructuresUsingTimeArgs): (JSC::fillStructuresUsingDateArgs): (JSC::dateProtoFuncSetTime): (JSC::dateProtoFuncSetYear):
  • runtime/ErrorConstructor.cpp: (JSC::constructError):
  • runtime/FunctionConstructor.cpp: (JSC::constructFunction):
  • runtime/FunctionPrototype.cpp: (JSC::functionProtoFuncApply): (JSC::functionProtoFuncCall):
  • runtime/JSArray.cpp: (JSC::JSArray::JSArray): (JSC::constructArray):
  • runtime/JSArray.h:
  • runtime/JSGlobalObjectFunctions.cpp: (JSC::encode): (JSC::decode): (JSC::globalFuncEval): (JSC::globalFuncParseInt): (JSC::globalFuncParseFloat): (JSC::globalFuncIsNaN): (JSC::globalFuncIsFinite): (JSC::globalFuncEscape): (JSC::globalFuncUnescape): (JSC::globalFuncJSCPrint):
  • runtime/MathObject.cpp: (JSC::mathProtoFuncAbs): (JSC::mathProtoFuncACos): (JSC::mathProtoFuncASin): (JSC::mathProtoFuncATan): (JSC::mathProtoFuncATan2): (JSC::mathProtoFuncCeil): (JSC::mathProtoFuncCos): (JSC::mathProtoFuncExp): (JSC::mathProtoFuncFloor): (JSC::mathProtoFuncLog): (JSC::mathProtoFuncMax): (JSC::mathProtoFuncMin): (JSC::mathProtoFuncPow): (JSC::mathProtoFuncRound): (JSC::mathProtoFuncSin): (JSC::mathProtoFuncSqrt): (JSC::mathProtoFuncTan):
  • runtime/NativeErrorConstructor.cpp: (JSC::NativeErrorConstructor::construct):
  • runtime/NumberConstructor.cpp: (JSC::constructWithNumberConstructor): (JSC::callNumberConstructor):
  • runtime/NumberPrototype.cpp: (JSC::numberProtoFuncToString): (JSC::numberProtoFuncToFixed): (JSC::numberProtoFuncToExponential): (JSC::numberProtoFuncToPrecision):
  • runtime/ObjectConstructor.cpp: (JSC::constructObject):
  • runtime/ObjectPrototype.cpp: (JSC::objectProtoFuncHasOwnProperty): (JSC::objectProtoFuncIsPrototypeOf): (JSC::objectProtoFuncDefineGetter): (JSC::objectProtoFuncDefineSetter): (JSC::objectProtoFuncLookupGetter): (JSC::objectProtoFuncLookupSetter): (JSC::objectProtoFuncPropertyIsEnumerable):
  • runtime/PropertySlot.h: (JSC::PropertySlot::getValue):
  • runtime/RegExpConstructor.cpp: (JSC::constructRegExp):
  • runtime/RegExpObject.cpp: (JSC::RegExpObject::match):
  • runtime/RegExpPrototype.cpp: (JSC::regExpProtoFuncCompile):
  • runtime/StringConstructor.cpp: (JSC::stringFromCharCodeSlowCase): (JSC::stringFromCharCode): (JSC::constructWithStringConstructor): (JSC::callStringConstructor):
  • runtime/StringPrototype.cpp: (JSC::stringProtoFuncReplace): (JSC::stringProtoFuncCharAt): (JSC::stringProtoFuncCharCodeAt): (JSC::stringProtoFuncConcat): (JSC::stringProtoFuncIndexOf): (JSC::stringProtoFuncLastIndexOf): (JSC::stringProtoFuncMatch): (JSC::stringProtoFuncSearch): (JSC::stringProtoFuncSlice): (JSC::stringProtoFuncSplit): (JSC::stringProtoFuncSubstr): (JSC::stringProtoFuncSubstring): (JSC::stringProtoFuncLocaleCompare): (JSC::stringProtoFuncFontcolor): (JSC::stringProtoFuncFontsize): (JSC::stringProtoFuncAnchor): (JSC::stringProtoFuncLink):

WebCore:

2009-04-28 Geoffrey Garen <ggaren@apple.com>

Rubber stamped by Beth Dakin.


Removed scaffolding supporting dynamically converting between 32bit and
64bit value representations.

  • bindings/js/JSAudioConstructor.cpp: (WebCore::constructAudio):
  • bindings/js/JSCanvasRenderingContext2DCustom.cpp: (WebCore::JSCanvasRenderingContext2D::setFillColor): (WebCore::JSCanvasRenderingContext2D::setStrokeColor): (WebCore::JSCanvasRenderingContext2D::strokeRect): (WebCore::JSCanvasRenderingContext2D::drawImage): (WebCore::JSCanvasRenderingContext2D::drawImageFromRect): (WebCore::JSCanvasRenderingContext2D::setShadow): (WebCore::JSCanvasRenderingContext2D::createPattern): (WebCore::JSCanvasRenderingContext2D::putImageData): (WebCore::JSCanvasRenderingContext2D::fillText): (WebCore::JSCanvasRenderingContext2D::strokeText):
  • bindings/js/JSClipboardCustom.cpp: (WebCore::JSClipboard::clearData): (WebCore::JSClipboard::getData): (WebCore::JSClipboard::setData): (WebCore::JSClipboard::setDragImage):
  • bindings/js/JSDOMApplicationCacheCustom.cpp: (WebCore::JSDOMApplicationCache::hasItem): (WebCore::JSDOMApplicationCache::add): (WebCore::JSDOMApplicationCache::remove): (WebCore::JSDOMApplicationCache::addEventListener): (WebCore::JSDOMApplicationCache::removeEventListener):
  • bindings/js/JSDOMStringListCustom.cpp: (WebCore::JSDOMStringList::item):
  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::open): (WebCore::JSDOMWindow::showModalDialog): (WebCore::JSDOMWindow::postMessage): (WebCore::JSDOMWindow::setTimeout): (WebCore::JSDOMWindow::setInterval): (WebCore::JSDOMWindow::atob): (WebCore::JSDOMWindow::btoa): (WebCore::JSDOMWindow::addEventListener): (WebCore::JSDOMWindow::removeEventListener):
  • bindings/js/JSDatabaseCustom.cpp: (WebCore::JSDatabase::changeVersion): (WebCore::JSDatabase::transaction):
  • bindings/js/JSElementCustom.cpp: (WebCore::JSElement::setAttribute): (WebCore::JSElement::setAttributeNode): (WebCore::JSElement::setAttributeNS): (WebCore::JSElement::setAttributeNodeNS):
  • bindings/js/JSGeolocationCustom.cpp: (WebCore::JSGeolocation::getCurrentPosition): (WebCore::JSGeolocation::watchPosition):
  • bindings/js/JSHTMLCollectionCustom.cpp: (WebCore::callHTMLCollection): (WebCore::JSHTMLCollection::item): (WebCore::JSHTMLCollection::namedItem):
  • bindings/js/JSHTMLDocumentCustom.cpp: (WebCore::documentWrite):
  • bindings/js/JSHTMLInputElementCustom.cpp: (WebCore::JSHTMLInputElement::setSelectionRange):
  • bindings/js/JSHTMLOptionsCollectionCustom.cpp: (WebCore::JSHTMLOptionsCollection::add):
  • bindings/js/JSHTMLSelectElementCustom.cpp: (WebCore::JSHTMLSelectElement::remove):
  • bindings/js/JSImageConstructor.cpp: (WebCore::constructImage):
  • bindings/js/JSInspectorControllerCustom.cpp: (WebCore::JSInspectorController::highlightDOMNode): (WebCore::JSInspectorController::addResourceSourceToFrame): (WebCore::JSInspectorController::addSourceToFrame): (WebCore::JSInspectorController::getResourceDocumentNode): (WebCore::JSInspectorController::search): (WebCore::JSInspectorController::databaseTableNames): (WebCore::JSInspectorController::setting): (WebCore::JSInspectorController::setSetting): (WebCore::JSInspectorController::wrapCallback):
  • bindings/js/JSJavaScriptCallFrameCustom.cpp: (WebCore::JSJavaScriptCallFrame::evaluate):
  • bindings/js/JSLocationCustom.cpp: (WebCore::JSLocation::replace): (WebCore::JSLocation::assign):
  • bindings/js/JSMessagePortCustom.cpp: (WebCore::JSMessagePort::startConversation): (WebCore::JSMessagePort::addEventListener): (WebCore::JSMessagePort::removeEventListener):
  • bindings/js/JSNodeCustom.cpp: (WebCore::JSNode::insertBefore): (WebCore::JSNode::replaceChild): (WebCore::JSNode::removeChild): (WebCore::JSNode::appendChild): (WebCore::JSNode::addEventListener): (WebCore::JSNode::removeEventListener):
  • bindings/js/JSNodeFilterCustom.cpp: (WebCore::JSNodeFilter::acceptNode):
  • bindings/js/JSNodeListCustom.cpp: (WebCore::callNodeList):
  • bindings/js/JSOptionConstructor.cpp: (WebCore::constructHTMLOptionElement):
  • bindings/js/JSQuarantinedObjectWrapper.cpp: (WebCore::JSQuarantinedObjectWrapper::construct): (WebCore::JSQuarantinedObjectWrapper::call):
  • bindings/js/JSSQLResultSetRowListCustom.cpp: (WebCore::JSSQLResultSetRowList::item):
  • bindings/js/JSSQLTransactionCustom.cpp: (WebCore::JSSQLTransaction::executeSql):
  • bindings/js/JSSVGElementInstanceCustom.cpp: (WebCore::JSSVGElementInstance::addEventListener): (WebCore::JSSVGElementInstance::removeEventListener):
  • bindings/js/JSSVGLengthCustom.cpp: (WebCore::JSSVGLength::convertToSpecifiedUnits):
  • bindings/js/JSSVGMatrixCustom.cpp: (WebCore::JSSVGMatrix::rotateFromVector):
  • bindings/js/JSSVGPathSegListCustom.cpp: (WebCore::JSSVGPathSegList::initialize): (WebCore::JSSVGPathSegList::getItem): (WebCore::JSSVGPathSegList::insertItemBefore): (WebCore::JSSVGPathSegList::replaceItem): (WebCore::JSSVGPathSegList::removeItem): (WebCore::JSSVGPathSegList::appendItem):
  • bindings/js/JSSVGPointListCustom.cpp: (WebCore::JSSVGPointList::initialize): (WebCore::JSSVGPointList::getItem): (WebCore::JSSVGPointList::insertItemBefore): (WebCore::JSSVGPointList::replaceItem): (WebCore::JSSVGPointList::removeItem): (WebCore::JSSVGPointList::appendItem):
  • bindings/js/JSSVGTransformListCustom.cpp: (WebCore::JSSVGTransformList::initialize): (WebCore::JSSVGTransformList::getItem): (WebCore::JSSVGTransformList::insertItemBefore): (WebCore::JSSVGTransformList::replaceItem): (WebCore::JSSVGTransformList::removeItem): (WebCore::JSSVGTransformList::appendItem):
  • bindings/js/JSWebKitCSSMatrixConstructor.cpp: (WebCore::constructWebKitCSSMatrix):
  • bindings/js/JSWebKitPointConstructor.cpp: (WebCore::constructWebKitPoint):
  • bindings/js/JSWorkerConstructor.cpp: (WebCore::constructWorker):
  • bindings/js/JSWorkerContextCustom.cpp: (WebCore::JSWorkerContext::importScripts): (WebCore::JSWorkerContext::addEventListener): (WebCore::JSWorkerContext::removeEventListener): (WebCore::JSWorkerContext::setTimeout): (WebCore::JSWorkerContext::setInterval):
  • bindings/js/JSWorkerCustom.cpp: (WebCore::JSWorker::addEventListener): (WebCore::JSWorker::removeEventListener):
  • bindings/js/JSXMLHttpRequestCustom.cpp: (WebCore::JSXMLHttpRequest::open): (WebCore::JSXMLHttpRequest::setRequestHeader): (WebCore::JSXMLHttpRequest::send): (WebCore::JSXMLHttpRequest::getResponseHeader): (WebCore::JSXMLHttpRequest::overrideMimeType): (WebCore::JSXMLHttpRequest::addEventListener): (WebCore::JSXMLHttpRequest::removeEventListener):
  • bindings/js/JSXMLHttpRequestUploadCustom.cpp: (WebCore::JSXMLHttpRequestUpload::addEventListener): (WebCore::JSXMLHttpRequestUpload::removeEventListener):
  • bindings/js/JSXSLTProcessorCustom.cpp: (WebCore::JSXSLTProcessor::importStylesheet): (WebCore::JSXSLTProcessor::transformToFragment): (WebCore::JSXSLTProcessor::transformToDocument): (WebCore::JSXSLTProcessor::setParameter): (WebCore::JSXSLTProcessor::getParameter): (WebCore::JSXSLTProcessor::removeParameter):
  • bindings/js/ScheduledAction.cpp: (WebCore::ScheduledAction::create): (WebCore::ScheduledAction::ScheduledAction):
  • bindings/js/ScheduledAction.h:
  • bindings/js/ScriptCallFrame.cpp: (WebCore::ScriptCallFrame::ScriptCallFrame):
  • bindings/js/ScriptCallFrame.h:
  • bindings/js/ScriptCallStack.cpp: (WebCore::ScriptCallStack::ScriptCallStack): (WebCore::ScriptCallStack::initialize):
  • bridge/c/c_instance.cpp: (JSC::Bindings::CInstance::invokeMethod): (JSC::Bindings::CInstance::invokeDefaultMethod): (JSC::Bindings::CInstance::invokeConstruct):
  • bridge/jni/jni_instance.cpp: (JavaInstance::invokeMethod):
  • bridge/objc/objc_instance.mm: (ObjcInstance::invokeMethod): (ObjcInstance::invokeDefaultMethod):

WebKit/mac:

2009-04-28 Geoffrey Garen <ggaren@apple.com>

Rubber stamped by Beth Dakin.


Removed scaffolding supporting dynamically converting between 32bit and
64bit value representations.

  • Plugins/Hosted/NetscapePluginInstanceProxy.mm: (WebKit::NetscapePluginInstanceProxy::marshalValues):
22:56 Changeset [42988] by ddkilzer@apple.com
  • 2 edits in trunk/JavaScriptCore

A little more hardening for UString

Reviewed by Maciej Stachowiak.

Revised fix for <rdar://problem/5861045> in r42644.

  • runtime/UString.cpp: (JSC::newCapacityWithOverflowCheck): Added. (JSC::concatenate): Used newCapacityWithOverflowCheck(). (JSC::UString::append): Ditto.
22:52 Changeset [42987] by ddkilzer@apple.com
  • 9 edits in trunk

2009-04-28 David Carson <dacarson@apple.com>

Reviewed by David Hyatt.

Restore alignment to device pixel boundaries.
https://bugs.webkit.org/show_bug.cgi?id=25458

Adjust the adjusted destination rect to be device pixel aligned.

  • platform/graphics/cg/ImageCG.cpp: (WebCore::BitmapImage::draw):

2009-04-28 David Carson <dacarson@apple.com>

Reviewed by David Kilzer.

Restore alignment to device pixel boundaries.
https://bugs.webkit.org/show_bug.cgi?id=25458

Updated test results with progressions.

  • platform/mac/svg/W3C-SVG-1.1/render-groups-03-t-expected.checksum:
  • platform/mac/svg/W3C-SVG-1.1/render-groups-03-t-expected.png: Yin-yang image shifted slightly to pixel boundaries.
  • platform/mac/svg/custom/focus-ring-expected.checksum:
  • platform/mac/svg/custom/focus-ring-expected.png: Focus rings drawn around rotated "focus text" and checkered images shifted slightly to pixel boundaries.
  • platform/mac/svg/custom/image-small-width-height-expected.checksum:
  • platform/mac/svg/custom/image-small-width-height-expected.png: Checkered images shifted slightly to pixel boundaries.
22:49 Changeset [42986] by ddkilzer@apple.com
  • 1 edit
    10 adds in trunk/LayoutTests

Added missing pixel test results.

  • platform/mac/fast/css-generated-content/beforeAfter-interdocument-expected.checksum: Added.
  • platform/mac/fast/css-generated-content/beforeAfter-interdocument-expected.png: Added.
  • platform/mac/fast/table/cell-coalescing-expected.checksum: Added.
  • platform/mac/fast/table/cell-coalescing-expected.png: Added.
  • platform/mac/webarchive/adopt-attribute-styled-body-webarchive-expected.checksum: Added.
  • platform/mac/webarchive/adopt-attribute-styled-body-webarchive-expected.png: Added.
  • platform/mac/webarchive/adopt-attribute-styled-node-webarchive-expected.checksum: Added.
  • platform/mac/webarchive/adopt-attribute-styled-node-webarchive-expected.png: Added.
  • platform/mac/webarchive/adopt-inline-styled-node-webarchive-expected.checksum: Added.
  • platform/mac/webarchive/adopt-inline-styled-node-webarchive-expected.png: Added.
22:39 Changeset [42985] by weinig@apple.com
  • 5 edits in trunk/WebCore

2009-04-28 Sam Weinig <sam@webkit.org>

Reviewed by Anders Carlsson.

Move unimplemented captureEvents and releaseEvents from JSDOMWindow
to DOMWindow.

  • bindings/js/JSDOMWindowCustom.cpp:
  • page/DOMWindow.cpp: (WebCore::DOMWindow::captureEvents): (WebCore::DOMWindow::releaseEvents):
  • page/DOMWindow.h:
  • page/DOMWindow.idl:
22:30 Changeset [42984] by ap@webkit.org
  • 2 edits in trunk

Correcting ChangeLogs - Sam talked to me about the patch, but it was Darin who reviewed it!

22:25 Changeset [42983] by ap@webkit.org
  • 9 edits
    3 adds in trunk

Reviewed by Sam Weinig.

https://bugs.webkit.org/show_bug.cgi?id=25420
<rdar://problem/6829570> REGRESSION: XMLHttpRequest allows loading from another origin

Test: http/tests/xmlhttprequest/detaching-frame-2.html

This was caused by faulty DOMWindow::document(), which could return a new document from
the window's frame after navigation.

  • bindings/js/JSDOMWindowCustom.h: (WebCore::JSDOMWindowBase::allowsAccessFromPrivate): Removed an obsolete check that allowed access when document was null. Contrary to what a comment said, that can happen for a window that is no longer in frame, not to one whose document is not constructed yet.
  • bindings/js/JSXMLHttpRequestConstructor.cpp: (WebCore::constructXMLHttpRequest): Bail out if context was not found. This currently happens due to a shortcoming in DOMWindow::document() - when it is fixed, the XMLHttpRequest object in included regression test will be constructed successfully, but won't be sent, because its context will be frameless.
  • page/DOMWindow.cpp: (WebCore::DOMWindow::document): Check that the window in frame hasn't been replaced yet. Added FIXME comments about how this may be better fixed in the future.
  • bindings/js/JSAudioConstructor.cpp: (WebCore::JSAudioConstructor::document): (WebCore::constructAudio):
  • bindings/js/JSImageConstructor.cpp: (WebCore::JSImageConstructor::document): (WebCore::constructImage):
  • bindings/js/JSMessageChannelConstructor.cpp: (WebCore::JSMessageChannelConstructor::construct):
  • bindings/js/JSOptionConstructor.cpp: (WebCore::JSOptionConstructor::document): (WebCore::constructHTMLOptionElement): Make matching changes to other constructors that hold a reference to global object.
21:54 Changeset [42982] by oliver@apple.com
  • 2 edits in trunk/JavaScriptCore

Bring back r42969, this time with correct codegen

Reviewed by Gavin Barraclough.

Add logic to the codegen for right shift to avoid jumping to a helper function
when shifting a small floating point value.

21:22 Changeset [42981] by kov@webkit.org
  • 1 copy in releases/WebKitGTK/webkit-1.1.6

Tagging 1.1.6.

21:02 Changeset [42980] by kevino@webkit.org
  • 6 edits in trunk

wxMSW build fix. Switch JSCore build back to static.

20:03 Changeset [42979] by ddkilzer@apple.com
  • 2 edits in trunk/WebKitTools

2009-04-28 Pierre d'Herbemont <pdherbemont@apple.com>

Reviewed by David Kilzer.

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

Make sure "--pixel --add-platform-exceptions" produces platform
specific pixel test results.

  • Scripts/run-webkit-tests: Support "--pixel --add-platform-exceptions"
18:24 Changeset [42978] by oliver@apple.com
  • 2 edits in trunk/JavaScriptCore

Roll out r42969, due to hangs in build bot.

18:00 Changeset [42977] by eric@webkit.org
  • 1 edit
    5 adds in trunk/LayoutTests

Reviewed by Simon Fraser.

Add a couple table hit tests
https://bugs.webkit.org/show_bug.cgi?id=21892

  • svg/hittest/svg-inside-display-table-expected.txt: Added.
  • svg/hittest/svg-inside-display-table.xhtml: Added.
  • svg/hittest/svg-inside-table-expected.txt: Added.
  • svg/hittest/svg-inside-table.xhtml: Added.
17:38 Changeset [42976] by oliver@apple.com
  • 3 edits in trunk/JavaScriptCore

Improve performance of string indexing

Reviewed by Geoff Garen

Add a cti_get_by_val_string function to specialise indexing into a string object.
This gives us a slight performance win on a number of string tests.

17:29 Changeset [42975] by eric.carlson@apple.com
  • 3 edits in trunk

2009-04-28 Pierre d'Herbemont <pdherbemont@apple.com>

Reviewed by Simon Fraser.

<rdar://problem/6834830>

Make sure we cover the two possible values reported by event.total that are playback engine
specific.

  • media/progress-event-total-expected.txt: Match the new test.
  • media/progress-event-total.html: Test the two values.
17:17 Changeset [42974] by timothy@apple.com
  • 7 edits in trunk/WebCore

Rename -[DOMRange lineBoxRects] to -[DOMRange textRects] and change how it
collects the individual rects, making sure to only include RenderText.

<rdar://problem/6810061>

Reviewed by Sam Weinig.

  • bindings/objc/DOM.mm: (-[DOMNode textRects]): Added. Make a Range and call textRects. (-[DOMNode lineBoxRects]): Call textRects method. (-[DOMRange textRects]): Call Range::textRects. (-[DOMRange lineBoxRects]): Call textRects method.
  • bindings/objc/DOMPrivate.h: Add the new methods and a comment about lineBoxRects being deprecated.
  • dom/Range.cpp: (WebCore::Range::boundingBox): Call textRects (renamed from addLineBoxRects). (WebCore::Range::textRects): Renamed from addLineBoxRects. Iterate over the nodes instead of the RenderObjects to correctly account for offsets. Also only call absoluteRectsForRange on RenderText.
  • dom/Range.h: Renamed addLineBoxRects to textRects.
  • page/Frame.cpp: (WebCore::Frame::selectionTextRects): Call textRects (renamed from addLineBoxRects).
  • rendering/RenderObject.cpp: (WebCore::RenderObject::absoluteRectsForRange): Remove if conditions that made this function very specific for the lineBoxRects API. These functions are still used by the InspectorController, and this change improves what the Inspector shows. (WebCore::RenderObject::absoluteQuadsForRange): Ditto.
17:11 Changeset [42973] by timothy@apple.com
  • 26 edits in trunk/WebCore

Remove the topLevel boolean argument from absolute{Rects,Quads}.
This argument was default to true and no one ever passed false.

Reviewed by Sam Weinig.

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::absoluteRects): (WebCore::RenderBlock::absoluteQuads):
  • rendering/RenderBlock.h:
  • rendering/RenderBox.cpp: (WebCore::RenderBox::absoluteRects): (WebCore::RenderBox::absoluteQuads):
  • rendering/RenderBox.h:
  • rendering/RenderInline.cpp: (WebCore::RenderInline::absoluteRects): (WebCore::RenderInline::absoluteQuads):
  • rendering/RenderInline.h:
  • rendering/RenderObject.h: (WebCore::RenderObject::absoluteRects): (WebCore::RenderObject::absoluteQuads):
  • rendering/RenderSVGHiddenContainer.cpp: (WebCore::RenderSVGHiddenContainer::absoluteRects): (WebCore::RenderSVGHiddenContainer::absoluteQuads):
  • rendering/RenderSVGHiddenContainer.h:
  • rendering/RenderSVGImage.cpp: (WebCore::RenderSVGImage::absoluteRects): (WebCore::RenderSVGImage::absoluteQuads):
  • rendering/RenderSVGImage.h:
  • rendering/RenderSVGInlineText.cpp: (WebCore::RenderSVGInlineText::absoluteRects): (WebCore::RenderSVGInlineText::absoluteQuads):
  • rendering/RenderSVGInlineText.h:
  • rendering/RenderSVGModelObject.cpp: (WebCore::RenderSVGModelObject::absoluteRects): (WebCore::RenderSVGModelObject::absoluteQuads):
  • rendering/RenderSVGModelObject.h:
  • rendering/RenderSVGTSpan.cpp: (WebCore::RenderSVGTSpan::absoluteRects): (WebCore::RenderSVGTSpan::absoluteQuads):
  • rendering/RenderSVGTSpan.h:
  • rendering/RenderSVGText.cpp: (WebCore::RenderSVGText::absoluteRects): (WebCore::RenderSVGText::absoluteQuads):
  • rendering/RenderSVGText.h:
  • rendering/RenderSVGTextPath.cpp: (WebCore::RenderSVGTextPath::absoluteQuads):
  • rendering/RenderSVGTextPath.h:
  • rendering/RenderText.cpp: (WebCore::RenderText::absoluteRects):
  • rendering/RenderText.h:
  • rendering/RenderView.cpp: (WebCore::RenderView::absoluteRects): (WebCore::RenderView::absoluteQuads):
  • rendering/RenderView.h:
16:50 Changeset [42972] by eric.carlson@apple.com
  • 5 edits
    2 adds in trunk/WebCore

2009-04-28 Eric Carlson <eric.carlson@apple.com>

Reviewed by Simon Fraser.

<rdar://problem/6794691> Media documents and <video controls> never show controls on Windows

RenderThemeWin doesn't implement the media element controller rendering methods, so media
controls stopped rendering when we switched from RenderThemeSafari. Rather than having copies
of the code in both places, the logic is in a new class that is called from both.

  • WebCore.vcproj/WebCore.vcproj: Add RenderMediaControls.cpp.
  • rendering/RenderMediaControls.cpp: Added. (WebCore::determineState): Translate renderer state to ThemeControlState. (WebCore::RenderMediaControls::adjustMediaSliderThumbSize): Adjust slider thumb. (WebCore::parentMediaElement): Return the HTMLMediaElement parent of a controller element. (WebCore::RenderMediaControls::paintMediaControlsPart): Paint a media controller part.
  • rendering/RenderMediaControls.h: Added.
  • rendering/RenderThemeSafari.cpp: (WebCore::RenderThemeSafari::paintSliderTrack): Remove the MediaSliderPart case, it is never called for the media slider. (WebCore::RenderThemeSafari::adjustSliderThumbSize): Call RenderMediaControls function. (WebCore::RenderThemeSafari::paintMediaFullscreenButton): Ditto. (WebCore::RenderThemeSafari::paintMediaMuteButton): Ditto. (WebCore::RenderThemeSafari::paintMediaPlayButton): Ditto. (WebCore::RenderThemeSafari::paintMediaSeekBackButton): Ditto. (WebCore::RenderThemeSafari::paintMediaSeekForwardButton): Ditto. (WebCore::RenderThemeSafari::paintMediaSliderTrack): Ditto. (WebCore::RenderThemeSafari::paintMediaSliderThumb): Ditto.
  • rendering/RenderThemeWin.cpp: (WebCore::RenderThemeWin::adjustSliderThumbSize): Ditto. (WebCore::RenderThemeWin::paintMediaFullscreenButton): Ditto. (WebCore::RenderThemeWin::paintMediaMuteButton): Ditto. (WebCore::RenderThemeWin::paintMediaPlayButton): Ditto. (WebCore::RenderThemeWin::paintMediaSeekBackButton): Ditto. (WebCore::RenderThemeWin::paintMediaSeekForwardButton): Ditto. (WebCore::RenderThemeWin::paintMediaSliderTrack): Ditto. (WebCore::RenderThemeWin::paintMediaSliderThumb): Ditto.
  • rendering/RenderThemeWin.h:
16:12 Changeset [42971] by bdakin@apple.com
  • 2 edits in trunk/WebCore

2009-04-28 Beth Dakin <bdakin@apple.com>

Reviewed by Sam Weinig.

Fix for <rdar://problem/6419515> REGRESSION (r31037): Positioned
images with % height are collapsed when printing

  • rendering/RenderBox.cpp: (WebCore::RenderBox::calcHeight): Make sure we don't ignore printing here!
15:36 Changeset [42970] by xan@webkit.org
  • 2 edits in trunk/JavaScriptCore

2009-04-28 Xan Lopez <xlopez@igalia.com>

Unreviewed: fix distcheck build, add (even more) missing files to list.

  • GNUmakefile.am:
15:22 Changeset [42969] by oliver@apple.com
  • 2 edits in trunk/JavaScriptCore

Improve performance of right shifts of large or otherwise floating point values.

Reviewed by Geoff Garen.

Add logic to the codegen for right shift to avoid jumping to a helper function
when shifting a small floating point value.

15:17 Changeset [42968] by adele@apple.com
  • 4 edits
    4 adds in trunk

WebCore:

2009-04-28 Adele Peterson <adele@apple.com>

Reviewed by Darin Adler.

Fix for <rdar://problem/6617298> Typing delete on an unquoted blank line unquotes the preceding, quoted blank line

Test: editing/deleting/type-delete-after-quote.html

When a selection is deleted that contains a whole number paragraphs plus a line break, we refrain from merging paragraphs after the delete,
since it is unclear to most users that such a selection actually ends at the start of the next paragraph. However, when a user hits delete
with a caret selection, they actually do expect the start of that paragraph to be merged into the paragraph before it. We can tell that
we're in this state because the TypingCommand creates the selection to delete but it doesn't change the endingSelection. So we can tell
that if we started with a caret selection, then we're not in this special case where we have to protect the user from unexpected behavior
from deleting a range they selected.

  • editing/DeleteSelectionCommand.cpp: (WebCore::DeleteSelectionCommand::initializePositionData):

LayoutTests:

2009-04-28 Adele Peterson <adele@apple.com>

Reviewed by Darin Adler.

Test for <rdar://problem/6617298> Typing delete on an unquoted blank line unquotes the preceding, quoted blank line

  • editing/deleting/type-delete-after-quote.html: Added.
  • platform/mac/editing/deleting/delete-4038408-fix-expected.txt: This expected result change reflects a change in how the editing is done, but the document structure remains the same, and there are no visible changes to the result.
  • platform/mac/editing/deleting/type-delete-after-quote-expected.checksum: Added.
  • platform/mac/editing/deleting/type-delete-after-quote-expected.png: Added.
  • platform/mac/editing/deleting/type-delete-after-quote-expected.txt: Added.
15:15 Changeset [42967] by xan@webkit.org
  • 2 edits in trunk/JavaScriptCore

2009-04-28 Xan Lopez <xlopez@igalia.com>

Unreviewed: fix distcheck build, add (more) missing files to list.

  • GNUmakefile.am:
15:12 Changeset [42966] by eric@webkit.org
  • 1 edit
    4 adds in trunk/LayoutTests

Reviewed by Simon Fraser.

Add text gradient test to cover
https://bugs.webkit.org/show_bug.cgi?id=25403

  • platform/mac/svg/text/text-gradient-positioning-expected.checksum: Added.
  • platform/mac/svg/text/text-gradient-positioning-expected.png: Added.
  • platform/mac/svg/text/text-gradient-positioning-expected.txt: Added.
  • svg/text/text-gradient-positioning.svg: Added.
15:09 Changeset [42965] by andersca@apple.com
  • 8 edits in trunk/WebKit/mac

2009-04-28 Anders Carlsson <andersca@apple.com>

Reviewed by Kevin Decker and Darin Adler.

Fix <rdar://problem/6836132>.


  • Plugins/Hosted/NetscapePluginHostProxy.mm: (WKPCCheckIfAllowedToLoadURL): Call the instance proxy.


(WKPCCancelCheckIfAllowedToLoadURL):
Ditto.


  • Plugins/Hosted/NetscapePluginInstanceProxy.h:
  • Plugins/Hosted/NetscapePluginInstanceProxy.mm: (WebKit::NetscapePluginInstanceProxy::NetscapePluginInstanceProxy): Initialize the check ID counter.


(WebKit::NetscapePluginInstanceProxy::checkIfAllowedToLoadURL):
Create a WebPluginContainerCheck, add it to the map, and start it.


(WebKit::NetscapePluginInstanceProxy::cancelCheckIfAllowedToLoadURL):
Remove the check from the map.


(WebKit::NetscapePluginInstanceProxy::checkIfAllowedToLoadURLResult):
Call the WKPH MIG callback.


  • Plugins/Hosted/WebHostedNetscapePluginView.h:
  • Plugins/Hosted/WebHostedNetscapePluginView.mm: (-[WebHostedNetscapePluginView _webPluginContainerCancelCheckIfAllowedToLoadRequest:]): Call the instance proxy.


(-[WebHostedNetscapePluginView _containerCheckResult:contextInfo:]):
Ditto.


  • Plugins/Hosted/WebKitPluginClient.defs:
  • Plugins/Hosted/WebKitPluginHost.defs: Add MIG declarations.
14:59 Changeset [42964] by eric@webkit.org
  • 1 edit
    4 adds in trunk/LayoutTests

Reviewed by David Hyatt.

Add text repaint test to cover
https://bugs.webkit.org/show_bug.cgi?id=16188

  • platform/mac/svg/text/text-repaint-rects-expected.checksum: Added.
  • platform/mac/svg/text/text-repaint-rects-expected.png: Added.
  • platform/mac/svg/text/text-repaint-rects-expected.txt: Added.
  • svg/text/text-repaint-rects.xhtml: Added.
14:38 Changeset [42963] by xan@webkit.org
  • 4 edits in trunk

WebCore:

2009-04-28 Xan Lopez <xlopez@igalia.com>

Unreviewed: fix distcheck build, add missing header to file list.

  • GNUmakefile.am:

JavaScriptCore:

2009-04-28 Xan Lopez <xlopez@igalia.com>

Unreviewed: fix distcheck build, add missing header to file list.

  • GNUmakefile.am:
14:27 Changeset [42962] by kov@webkit.org
  • 3 edits in trunk/WebKit/gtk

2009-04-28 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Unreviewed. Update gtk-doc control and base sgml files for 1.1.6.

  • docs/webkitgtk-docs.sgml:
  • docs/webkitgtk-sections.txt:
13:59 Changeset [42961] by eric@webkit.org
  • 19 edits in trunk

Reviewed by Sam Weinig.

Fix foreignObject transform order
https://bugs.webkit.org/show_bug.cgi?id=25433

Transforms were being applied in the wrong order.
When mapping from local to parent, first apply the viewport transform
then map from the viewport box to the parent box.

  • rendering/RenderForeignObject.cpp: (WebCore::RenderForeignObject::localToParentTransform):
13:56 Changeset [42960] by eric@webkit.org
  • 20 edits in trunk/WebCore

Reviewed by Simon Fraser.

Simplify nodeAtPoint for SVG
https://bugs.webkit.org/show_bug.cgi?id=25432

Added a new nodeAtFloatPoint method which takes a FloatPoint
instead of the *super confusing* x/y tx/ty pairs that HTML uses.
Mostly this is just lots of minus-lines as the new code is *much* simpler.

I made all the SVG renderers use the new nodeAtFloatPoint and ASSERT_NOT_REACHED
for the nodeAtPoint HTML/CSS int x/y version.

Eventually the rest of CSS/HTML render objects will follow suit
and move to nodeAtFloatPoint (possibly renamed back to nodeAtPoint), but changing them
over was well outside the scope of this change.

SVG hit testing is not actually floating point precise yet, since its still
passed in an integer x/y. Certain transform hit-test edge cases are likely fixed
by moving to FloatPoint here, but I didn't try to find one.

This should not cause functional changes for common-case hit testing, thus
no tests changed, nor should new tests be needed.

  • rendering/RenderForeignObject.cpp: (WebCore::RenderForeignObject::translationForAttributes): (WebCore::RenderForeignObject::paint): (WebCore::RenderForeignObject::computeRectForRepaint): (WebCore::RenderForeignObject::localToParentTransform): (WebCore::RenderForeignObject::layout): (WebCore::RenderForeignObject::nodeAtFloatPoint):
  • rendering/RenderForeignObject.h:
  • rendering/RenderObject.cpp: (WebCore::RenderObject::nodeAtFloatPoint):
  • rendering/RenderObject.h:
  • rendering/RenderPath.cpp: (WebCore::RenderPath::nodeAtFloatPoint):
  • rendering/RenderPath.h:
  • rendering/RenderSVGContainer.cpp: (WebCore::RenderSVGContainer::nodeAtFloatPoint):
  • rendering/RenderSVGContainer.h: (WebCore::RenderSVGContainer::pointIsInsideViewportClip):
  • rendering/RenderSVGHiddenContainer.cpp: (WebCore::RenderSVGHiddenContainer::nodeAtFloatPoint):
  • rendering/RenderSVGHiddenContainer.h:
  • rendering/RenderSVGImage.cpp: (WebCore::RenderSVGImage::paint): (WebCore::RenderSVGImage::nodeAtFloatPoint):
  • rendering/RenderSVGImage.h: (WebCore::RenderSVGImage::localToParentTransform): (WebCore::RenderSVGImage::localTransform):
  • rendering/RenderSVGModelObject.cpp: (WebCore::RenderSVGModelObject::nodeAtFloatPoint):
  • rendering/RenderSVGModelObject.h:
  • rendering/RenderSVGRoot.cpp: (WebCore::RenderSVGRoot::nodeAtFloatPoint):
  • rendering/RenderSVGText.cpp: (WebCore::RenderSVGText::nodeAtFloatPoint):
  • rendering/RenderSVGText.h:
  • rendering/RenderSVGViewportContainer.cpp: (WebCore::RenderSVGViewportContainer::pointIsInsideViewportClip):
  • rendering/RenderSVGViewportContainer.h:
13:42 Changeset [42959] by eric.carlson@apple.com
  • 2 edits in trunk/WebCore

2009-04-28 Eric Carlson <eric.carlson@apple.com>

  • fix the Tiger build
  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::cacheMovieScale):
13:30 Changeset [42958] by weinig@apple.com
  • 8 edits in trunk/WebCore

2009-04-28 Sam Weinig <sam@webkit.org>

Reviewed by Beth Dakin.

Consolidate ScheduleAction creation into ScheduledAction::create.
Autogenerate JSWorkerContext.clearTimeout and clearInterval.

  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::setTimeout): (WebCore::JSDOMWindow::setInterval):
  • bindings/js/JSWorkerContextCustom.cpp: (WebCore::JSWorkerContext::setTimeout): (WebCore::JSWorkerContext::setInterval):
  • bindings/js/ScheduledAction.cpp: (WebCore::ScheduledAction::create): (WebCore::ScheduledAction::ScheduledAction): (WebCore::ScheduledAction::execute):
  • bindings/js/ScheduledAction.h:
  • workers/WorkerContext.cpp: (WebCore::DOMWindow::setTimeout): (WebCore::DOMWindow::clearTimeout): (WebCore::DOMWindow::setInterval): (WebCore::DOMWindow::clearInterval):
  • workers/WorkerContext.h:
  • workers/WorkerContext.idl:
13:22 Changeset [42957] by sfalken@apple.com
  • 3 edits
    1 add in trunk/WebKitTools

Fix link warnings building TestNetscapePlugin on Windows due to mismatched
DLL name vs. LIBRARY directive name.


Reviewed by Jon Honeycutt.

  • DumpRenderTree/win/TestNetscapePlugin/TestNetscapePlugin.def:
  • DumpRenderTree/win/TestNetscapePlugin/TestNetscapePlugin.vcproj:
  • DumpRenderTree/win/TestNetscapePlugin/TestNetscapePlugin_debug.def: Added.
13:21 Changeset [42956] by kov@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-04-28 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

Updated with news for the 1.1.6 release.

  • NEWS:
13:14 Changeset [42955] by beidson@apple.com
  • 7 edits
    10 adds in trunk

WebCore:

2009-04-28 Brady Eidson <beidson@apple.com>

Reviewed by Darin Adler

<rdar://problem/6643219> Crashes after moving nodes between documents with Safari 4 Beta installed

When moving Nodes between documents we weren't properly updating style declarations. Certain operations, such
as creating webarchives, would tickle this bug.

Tests: webarchive/adopt-attribute-styled-body-webarchive.html

webarchive/adopt-attribute-styled-node-webarchive.html
webarchive/adopt-inline-styled-node-webarchive.html

  • dom/Node.cpp: Add a debug-only mechanism to insure that anyone who overrides (did/will)MoveToNewOwnerDocument calls their parent-class implementation after they've done their own work. (WebCore::setWillMoveToNewOwnerDocumentWasCalled): (WebCore::setDidMoveToNewOwnerDocumentWasCalled): (WebCore::Node::setDocument): (WebCore::Node::willMoveToNewOwnerDocument): (WebCore::Node::didMoveToNewOwnerDocument):
  • dom/StyledElement.cpp: (WebCore::StyledElement::attributeChanged): Add a comment explaining why we don't need to walk the nameAttrMap to update style declarations. (WebCore::StyledElement::didMoveToNewOwnerDocument): Update the parent pointer for the inline style declaration.
  • dom/StyledElement.h:
  • html/HTMLBodyElement.cpp: (WebCore::HTMLBodyElement::didMoveToNewOwnerDocument): Update the parent pointer for the link declaration.
  • html/HTMLBodyElement.h:

LayoutTests:

2009-04-28 Brady Eidson <beidson@apple.com>

Reviewed by Darin Adler

<rdar://problem/6643219> Crashes after moving nodes between documents with Safari 4 Beta installed

  • webarchive/adopt-attribute-styled-body-webarchive-expected.webarchive: Added.
  • webarchive/adopt-attribute-styled-body-webarchive.html: Added.
  • webarchive/resources/adopt-attribute-styled-body-iframe.html: Added.
  • webarchive/resources/adopt-attribute-styled-body-second-window.html: Added.
  • webarchive/adopt-attribute-styled-node-webarchive-expected.webarchive: Added.
  • webarchive/adopt-attribute-styled-node-webarchive.html: Added.
  • webarchive/resources/adopt-attribute-styled-node-second-window.html: Added.
  • webarchive/adopt-inline-styled-node-webarchive-expected.webarchive: Added.
  • webarchive/adopt-inline-styled-node-webarchive.html: Added.
  • webarchive/resources/adopt-inline-styled-node-second-window.html: Added.
13:12 Changeset [42954] by mrowe@apple.com
  • 2 edits in tags/Safari-6530.7.1/WebCore

Merge r42945.

12:48 Changeset [42953] by sfalken@apple.com
  • 3 edits in trunk/WebKitTools

Fix warnings about PRODUCTION and ARCHIVE_BUILD on Windows builds
of ImageDiff and DumpRenderTree.


Rubber stamped by Adam Roben.

  • DumpRenderTree/win/DumpRenderTree.vcproj:
  • DumpRenderTree/win/ImageDiff.vcproj:
12:46 Changeset [42952] by eric.carlson@apple.com
  • 2 edits in trunk/WebCore

2009-04-28 Eric Carlson <eric.carlson@apple.com>

Reviewed by Darin Adler.


<rdar://problem/6834876> Don't use BlockExceptions macros until QTKit supports
QTMoviePreferredTransformAttribute

  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::cacheMovieScale): use @try and @catch instead of BEGIN_BLOCK_OBJC_EXCEPTIONS and END_BLOCK_OBJC_EXCEPTIONS for builds of QTKit that throw an exception on QTMovieCurrentSizeAttribute but don't support QTMoviePreferredTransformAttribute
12:43 Changeset [42951] by darin@apple.com
  • 2 edits in trunk/WebCore

2009-04-28 Darin Adler <darin@apple.com>

Reviewed by Sam Weinig.

  • rendering/RenderText.cpp: (WebCore::RenderText::RenderText): Streamline code path to avoid a bit of reference count churn and remove a strange unneeeded PassRefPtr typecast. Also added a comment. (WebCore::RenderText::setTextInternal): Ditto.
12:37 Changeset [42950] by eric@webkit.org
  • 21 edits in trunk

Reviewed by Simon Fraser.

Simplify how SVG containers paint
https://bugs.webkit.org/show_bug.cgi?id=25431

Moved filterBoundingBox() from RenderSVGModelObject to SVGRenderSupport to share with other classes.
Gave RenderSVGText a modern clippedOverflowRectForRepaint and computeRectForRepaint.
RenderSVGText now applies transforms at paint() time instead of during chunk draw time
this should be much more efficient.

Fixed the order in which RenderSVGViewportContainer applied
transforms and clips. We now clip to the viewport first and apply
all transforms at once. Also since the viewport logic is now only
inside RenderSVGViewportContainer (instead of inside RenderSVGContainer)
we always know we have a viewport. We now use only viewportSize instead of
viewport() since RenderSVGViewportContainers can't have x/y offsets.

We now correctly transform the PaintInfo::rect (damage rect) when applying transforms.
This allowed us to apply the transform during text paint() instead of at chunk paint
time. Previously we had to apply the transform during chunk paint time because
RenderBlock (superclass of RenderSVGBlock) would check intersection with the damageRect
before telling any of the inlines to draw. Now that we adjust the damage rect correctly
we pass the intersection check correctly! (This probably fixed some <text> redraw bugs
but since I still have a bunch of those to fix, I've not tried to write additional tests
to cover these potential fixes yet.)

SVGRootInlineBox no longer needs to deal with transforms at chunk paint time, yay!

  • rendering/RenderPath.cpp: (WebCore::RenderPath::repaintRectInLocalCoordinates): (WebCore::RenderPath::paint):
  • rendering/RenderSVGContainer.cpp: (WebCore::RenderSVGContainer::paint): (WebCore::RenderSVGContainer::repaintRectInLocalCoordinates):
  • rendering/RenderSVGContainer.h: (WebCore::RenderSVGContainer::applyViewportClip):
  • rendering/RenderSVGImage.cpp: (WebCore::RenderSVGImage::layout): (WebCore::RenderSVGImage::repaintRectInLocalCoordinates): (WebCore::RenderSVGImage::clippedOverflowRectForRepaint): (WebCore::RenderSVGImage::computeRectForRepaint):
  • rendering/RenderSVGImage.h:
  • rendering/RenderSVGModelObject.cpp:
  • rendering/RenderSVGModelObject.h:
  • rendering/RenderSVGRoot.cpp: (WebCore::RenderSVGRoot::layout): (WebCore::RenderSVGRoot::selfWillPaint): (WebCore::RenderSVGRoot::paint): (WebCore::RenderSVGRoot::viewportSize): (WebCore::RenderSVGRoot::calcViewport): (WebCore::RenderSVGRoot::localToBorderBoxTransform): (WebCore::RenderSVGRoot::parentOriginToBorderBox): (WebCore::RenderSVGRoot::borderOriginToContentBox): (WebCore::RenderSVGRoot::localToRepaintContainerTransform): (WebCore::RenderSVGRoot::localToParentTransform): (WebCore::RenderSVGRoot::computeRectForRepaint): (WebCore::RenderSVGRoot::nodeAtPoint):
  • rendering/RenderSVGRoot.h:
  • rendering/RenderSVGText.cpp: (WebCore::RenderSVGText::clippedOverflowRectForRepaint): (WebCore::RenderSVGText::computeRectForRepaint): (WebCore::RenderSVGText::paint): (WebCore::RenderSVGText::repaintRectInLocalCoordinates):
  • rendering/RenderSVGText.h: (WebCore::RenderSVGText::localToParentTransform): (WebCore::RenderSVGText::localTransform):
  • rendering/RenderSVGViewportContainer.cpp: (WebCore::RenderSVGViewportContainer::applyViewportClip):
  • rendering/RenderSVGViewportContainer.h:
  • rendering/SVGRenderSupport.cpp: (WebCore::filterBoundingBoxForRenderer): (WebCore::applyTransformToPaintInfo):
  • rendering/SVGRenderSupport.h:
  • rendering/SVGRootInlineBox.cpp: (WebCore::SVGRootInlineBoxPaintWalker::chunkStartCallback): (WebCore::SVGRootInlineBox::paint):
12:37 Changeset [42949] by cfleizach@apple.com
  • 3 edits
    2 adds in trunk

Bug 25453: AX: fall back to PLACEHOLDER attr if form element is not labeled
https://bugs.webkit.org/show_bug.cgi?id=25453

12:33 Changeset [42948] by mrowe@apple.com
  • 4 edits in tags/Safari-6530.7.1

Versioning.

12:16 Changeset [42947] by mrowe@apple.com
  • 1 copy in tags/Safari-6530.7.1

New tag.

12:14 Changeset [42946] by cfleizach@apple.com
  • 9 edits
    2 adds in trunk

Bug 25452: AX: Don't create addition space AXStaticText element for every bold or link tag
https://bugs.webkit.org/show_bug.cgi?id=25452

11:57 Changeset [42945] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Work around <rdar://problem/6833240> by relying on static initialization to zero the entire struct.
This removes the need for us to explicitly initialize all of the members, which have a tendency
to change in meaning and number between versions of libxml2.

Reviewed by Sam Weinig.

  • dom/XMLTokenizerLibxml2.cpp:

(WebCore::):
(WebCore::sharedXHTMLEntity):
(WebCore::getXHTMLEntity):

11:56 Changeset [42944] by sfalken@apple.com
  • 2 edits in trunk/WebKitLibraries

Fix extraneous warning about AnalyzeWithLargeStack not being defined in Windows builds.
Indirected definition through a UserMacro.


Reviewed by Mark Rowe.

  • win/tools/vsprops/common.vsprops:
11:52 Changeset [42943] by sfalken@apple.com
  • 2 edits in trunk/WebCore

Fix linker warning by specifying /NODEFAULTLIB:LIBCMT for QTMovieWin.


Reviewed by Mark Rowe.

  • WebCore.vcproj/QTMovieWin.vcproj:
11:32 Changeset [42942] by cfleizach@apple.com
  • 3 edits
    2 adds in trunk

Bug 25449: AX: Respect the alt tag of ARIA button as AXDescription
https://bugs.webkit.org/show_bug.cgi?id=25449

11:29 Changeset [42941] by sfalken@apple.com
  • 2 edits in trunk/WebKitLibraries

Fix extraneous warning about PRODUCTION not being defined in Windows builds.
Indirected definition for PRODUCTION through a UserMacro.


Reviewed by Mark Rowe.

  • win/tools/vsprops/common.vsprops:
11:16 Changeset [42940] by mitz@apple.com
  • 2 edits in trunk/WebCore

Reviewed by John Sullivan.

  • formatting cleanup
  • rendering/RenderBox.cpp: (WebCore::RenderBox::calcHeight): Corrected indentation and added braces around a multi-line "else" clause.
11:05 Changeset [42939] by eric.carlson@apple.com
  • 4 edits in trunk

2009-04-28 Eric Carlson <eric.carlson@apple.com>

Reviewed by Adam Roben


<rdar://problem/6769968> media/video-size-intrinsic-scale.html fails on Windows
https://bugs.webkit.org/show_bug.cgi?id=25094


Display movies saved with a non-identity matrix at the correct size.

  • platform/graphics/win/QTMovieWin.cpp: (QTMovieWinPrivate::QTMovieWinPrivate): Initialize m_widthScaleFactor and m_heightScaleFactor. (QTMovieWinPrivate::cacheMovieScale): New, calculate difference between initial size and natural size so naturalSize() accounts for non-identity movie matrix. (QTMovieWinPrivate::task):Call cacheMovieScale when load state reaches kMovieLoadStateLoaded for the first time. kMovieLoadState -> QTMovieLoadState. (QTMovieWinPrivate::drawingComplete): kMovieLoadState -> QTMovieLoadState. (QTMovieWinPrivate::createGWorld): Don't bother creating gworld until load state reaches kMovieLoadStateLoaded because we do not try to draw it before that point. (QTMovieWinPrivate::setSize): Do not change movie box before reaching kMovieLoadStateLoaded because we need to get the movie's initial size in cacheMovieScale. (QTMovieWin::getNaturalSize): Multiply width and height by initial scale.

2009-04-28 Eric Carlson <eric.carlson@apple.com>

Reviewed by Adam Roben

<rdar://problem/6769968> media/video-size-intrinsic-scale.html fails on Windows
https://bugs.webkit.org/show_bug.cgi?id=25094

  • platform/win/Skipped: Re-enable video-size-intrinsic-scale.html.
10:43 Changeset [42938] by weinig@apple.com
  • 7 edits in trunk/WebCore

2009-04-28 Sam Weinig <sam@webkit.org>

Reviewed by Dan Bernstein.

Move timer code from JSDOMWindow to DOMWindow. clearTimeout and
clearInterval can now be autogenerated.

  • bindings/js/JSDOMWindowBase.cpp:
  • bindings/js/JSDOMWindowBase.h:
  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::createScheduledAction): (WebCore::JSDOMWindow::setTimeout): (WebCore::JSDOMWindow::setInterval):
  • page/DOMWindow.cpp: (WebCore::DOMWindow::setTimeout): (WebCore::DOMWindow::clearTimeout): (WebCore::DOMWindow::setInterval): (WebCore::DOMWindow::clearInterval):
  • page/DOMWindow.h:
  • page/DOMWindow.idl:
10:06 Changeset [42937] by kevino@webkit.org
  • 2 edits in trunk/WebCore

Reviewed by Kevin Ollivier.

Small optimization, don't get the widget's handle twice in one method.

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

09:46 Changeset [42936] by mitz@apple.com
  • 3 edits in trunk/WebKit/mac

Reviewed by John Sullivan.

  • fix <rdar://problem/6786360> Make PDF an insertable pasteboard type
  • WebCoreSupport/WebPasteboardHelper.mm: (WebPasteboardHelper::insertablePasteboardTypes):
  • WebView/WebHTMLView.mm: (-[WebHTMLView _documentFragmentFromPasteboard:inContext:allowPlainText:]): (+[WebHTMLView _insertablePasteboardTypes]): (-[WebHTMLView _documentFragmentFromPasteboard:forType:inContext:subresources:]):
09:32 QtWebKitJournal edited by hausmann@webkit.org
(diff)
09:31 Changeset [42935] by kevino@webkit.org
  • 2 edits in trunk/WebCore

Reviewed by Kevin Ollivier.

ScrollView copy and paste typo fix.

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

09:29 QtWebKitTodo edited by hausmann@webkit.org
(diff)
09:28 QtWebKitJournal edited by hausmann@webkit.org
(diff)
09:27 Changeset [42934] by hausmann@webkit.org
  • 6 edits in trunk/WebKit/qt

2009-04-28 Simon Hausmann <simon.hausmann@nokia.com>

Reviewed by Tor Arne Vestbø.

QWebElement API changes after another round of API review:

  • Fix argument names of findAll/findFirst
  • Split up toXml into innerXml and outerXml
  • Removed confusing toggleClass overload
  • Fixed casing of namespaceUri to follow QXmlStreamReader
  • Removed tagName from firstChild/nextSibling/etc.
  • Renamed append/prepend/insertAfter/insertBefore to [append|prepend][Inside|Outside]
  • Renamed wrapWith() back to wrap()
  • Made clone() const
  • Renamed remove() to takeFromDocument(), added removeFromDocument()
  • Renamed clear() to removeChildren()
  • Renamed scriptsFunctions/callScriptFunction to functions()/callFunction()
  • Renamed scriptProperty to scriptableProperty
09:25 QtWebKitTodo edited by hausmann@webkit.org
(diff)
09:09 Changeset [42933] by xan@webkit.org
  • 2 edits in trunk

2009-04-28 Xan Lopez <xlopez@igalia.com>

Unreviewed: bump versions in preparation for 1.1.6 release.

  • configure.ac:
09:06 Changeset [42932] by xan@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-04-28 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Leave a bit of space from the previous page when advacing a whole
page. This matches the behavior of our scrollbar scrolling code
and is just nicer in general.

  • webkit/webkitwebview.cpp:
08:55 Changeset [42931] by kevino@webkit.org
  • 5 edits in trunk/WebKit/wx

Reviewed by Kevin Ollivier.

Add a function for resetting the zoom factor to 1.0.

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

07:01 Changeset [42930] by sullivan@apple.com
  • 2 edits in trunk/WebCore

2009-04-28 John Sullivan <sullivan@apple.com>

<rdar://problem/6820221> REGRESSION: 2-byte character names of images are displayed as %-encoded ASCII

Reviewed by Adam Roben

  • loader/ImageDocument.cpp: (WebCore::ImageTokenizer::finish): decode the filename before processing it as part of the image name that's passed to the client
06:51 Changeset [42929] by hausmann@webkit.org
  • 5 edits in trunk/WebKit/qt

2009-04-28 Simon Hausmann <simon.hausmann@nokia.com>

Reviewed by Tor Arne Vestbø.

Fix some qdoc warnings.

06:36 QtWebKitTodo edited by hausmann@webkit.org
(diff)
06:35 QtWebKitTodo edited by hausmann@webkit.org
(diff)
06:18 api_changes.txt attached to QtWebKitTodo by hausmann@webkit.org
Diff of the suggested API changes after another round of review
02:59 Changeset [42928] by mjs@apple.com
  • 2 edits in trunk/WebCore

2009-04-28 Maciej Stachowiak <mjs@apple.com>

Not reviewed, build fix.

  • fix initialization order to match declaration order to fix release build
  • page/Settings.cpp: (WebCore::Settings::Settings):
01:55 Changeset [42927] by xan@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-04-28 Xan Lopez <xlopez@igalia.com>

Unreviewed: committed a few bits by mistake, remove them.

  • WebCoreSupport/EditorClientGtk.cpp: (WebKit::handleEditingKeyboardEvent):
01:45 Changeset [42926] by xan@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-04-28 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

https://bugs.webkit.org/show_bug.cgi?id=16135
[GTK] Support caret browsing

Refactor keyboard event handling to work properly under caret
browsing mode. Most of the code comes from the Windows port.

  • WebCoreSupport/EditorClientGtk.cpp: (WebKit::): (WebKit::interpretKeyEvent): (WebKit::handleEditingKeyboardEvent): (WebKit::EditorClient::handleKeyboardEvent):
01:44 Changeset [42925] by xan@webkit.org
  • 3 edits in trunk/WebKit/gtk

2009-04-28 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

https://bugs.webkit.org/show_bug.cgi?id=16135
[GTK] Support caret browsing

Based on a patch by Alp Toker.

Add enable-caret-browsing setting to WebKitWebView.

  • webkit/webkitwebsettings.cpp: (_WebKitWebSettingsPrivate::): (_WebKitWebSettingsPrivate::webkit_web_settings_class_init): (_WebKitWebSettingsPrivate::webkit_web_settings_set_property): (_WebKitWebSettingsPrivate::webkit_web_settings_get_property): (_WebKitWebSettingsPrivate::webkit_web_settings_copy):
  • webkit/webkitwebview.cpp:
01:44 Changeset [42924] by xan@webkit.org
  • 7 edits in trunk/WebCore

2009-04-28 Xan Lopez <xlopez@igalia.com>

Reviewed by Justin Garcia.

https://bugs.webkit.org/show_bug.cgi?id=16135
[GTK] Support caret browsing

Based on a patch by Alp Toker.

Implement basic support for caret browsing, active only when the
corresponding setting is enabled.

  • page/EventHandler.cpp: (WebCore::EventHandler::handleMouseReleaseEvent):
  • page/FocusController.cpp: (WebCore::FocusController::advanceFocus): (WebCore::clearSelectionIfNeeded): (WebCore::FocusController::setFocusedNode):
  • page/Frame.cpp: (WebCore::Frame::setFocusedNodeIfNeeded): (WebCore::Frame::setSelectionFromNone): (WebCore::Frame::respondToChangedSelection):
  • page/Settings.cpp: (WebCore::Settings::Settings):
  • page/Settings.h:
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::paintCaret):
01:08 Changeset [42923] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

2009-04-28 Gavin Barraclough <barraclough@apple.com>

Rubber stamped by Maciej "Henry Morgan" Stachowiak.

Enable YARR.
(Again.)

  • wtf/Platform.h:
00:10 Changeset [42922] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

2009-04-27 Gavin Barraclough <barraclough@apple.com>

Reviewed by Maciej Stachowiak.

Tweak a loop condition to keep GCC happy,
some GCCs seem to be having issues with this. :-/

  • bytecompiler/BytecodeGenerator.cpp: (JSC::BytecodeGenerator::breakTarget):
  • wtf/Platform.h:

04/27/09:

23:05 Changeset [42921] by weinig@apple.com
  • 1 edit
    1 move in trunk/LayoutTests

2009-04-27 Sam Weinig <sam@webkit.org>

Reviewed by Mark Rowe.

Disable test intermittent failing test.

  • svg/custom/filter-source-alpha.svg: Removed.
  • svg/custom/filter-source-alpha.svg-disabled: Copied from svg/custom/filter-source-alpha.svg.
22:14 Changeset [42920] by simon.fraser@apple.com
  • 3 edits
    5 adds in trunk/LayoutTests

2009-04-27 Pierre d'Herbemont <pdherbemont@apple.com>

Reviewed by Simon Fraser

Fix video-played.html layout test.

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

We implement playForMillisecs() which makes sure the video
will have been playing for a minimum time.

Note, this does not ensure a strict synchronization between the time
asked and the media playback. But given that media playback is the
slow part we just need to compensate for the delay involved by playback.
(And not the other way around.)

Moreover, we split the test in 3 in order to be more gentle which
slower machines that may tend to timeout more easily on those tests.

  • media/video-played-collapse-expected.txt: Added.
  • media/video-played-collapse.html: Added.
  • media/video-played-expected.txt:
  • media/video-played-reset-expected.txt: Added.
  • media/video-played-reset.html: Added.
  • media/video-played.html:
  • Split tests in multiple files.
  • Makes sure we always play as much as requested by using playForMillisecs();
  • media/video-played.js: Added. (testRanges): (nextTest): (pause): (canplay): (willCreateNewRange): (startPlayingInNewRange): (startPlaying): (secToMilli): (milliToSecs): (playForMillisecs.callPauseIfTimeIsReached): (playForMillisecs): (videoPlayedMain):
20:39 Changeset [42919] by dglazkov@chromium.org
  • 2 edits in trunk/WebCore

2009-04-27 Eric Roman <eroman@chromium.org>

Reviewed by Darin Fisher.

Fix a compile breakage.
<https://bugs.webkit.org/show_bug.cgi?id=25384>

  • bindings/v8/custom/V8DOMWindowCustom.cpp: (WebCore::CALLBACK_FUNC_DECL):
18:01 Changeset [42918] by eric.carlson@apple.com
  • 3 edits in trunk/WebCore

2009-04-27 Eric Carlson <eric.carlson@apple.com>

Reviewed by Darin Adler.

<rdar://problem/6822344> Use of QTMovieCurrentSizeAttribute generates exception and will crash

QTMovieCurrentSizeAttribute generates an exception with some versions of QTKit, so calculate a
multiplier to scale from natural size to current size when a movie is opened and use that to
return the correct value from the naturalSize() method.

  • platform/graphics/mac/MediaPlayerPrivateQTKit.h:
  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::MediaPlayerPrivate): Initialize m_scaleFactor. (WebCore::MediaPlayerPrivate::naturalSize): Return naturalSize transformed by initial scale. (WebCore::MediaPlayerPrivate::cacheMovieScale): New, calculate difference between initial size and natural size so naturalSize() accounts for non-identity movie matrix. (WebCore::MediaPlayerPrivate::updateStates): Call cacheMovieScale when load state reaches QTMovieLoadStateLoaded for the first time.
17:54 Changeset [42917] by bdakin@apple.com
  • 3 edits
    4 adds in trunk

WebCore:

2009-04-27 Beth Dakin <bdakin@apple.com>

Reviewed by Dave Hyatt.

Fix for <rdar://problem/6709057> REGRESSION (4PB-TOT): http://www.winway.com/main3/support/faq.aspx selector doesn't look right or work

The problem here is that we had a self-painting layer nested inside
a non-self-painting layer. We ended up ignoring the self-painting
layer both during painting and hit-testing. This patch corrects
that.

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::paintLayer): (WebCore::RenderLayer::hitTestLayer):

LayoutTests:

2009-04-27 Beth Dakin <bdakin@apple.com>

Reviewed by Dave Hyatt.

Test for <rdar://problem/6709057> REGRESSION (4PB-TOT): http://www.winway.com/main3/support/faq.aspx selector doesn't look right or work

  • fast/css/nested-layers-with-hover.html: Added.
  • platform/mac/fast/css/nested-layers-with-hover-expected.checksum: Added.
  • platform/mac/fast/css/nested-layers-with-hover-expected.png: Added.
  • platform/mac/fast/css/nested-layers-with-hover-expected.txt: Added.
17:23 Changeset [42916] by adachan@apple.com
  • 2 edits in trunk/WebCore

If it has been set to allow any HTTPS certificates for this host, set
kCFStreamSSLAllowsExpiredCertificates to true to allow expired
certificate, and set kCFStreamSSLValidatesCertificateChain false to skip
certificate chain validation. (<rdar://problem/6382059>)

Reviewed by John Sullivan.

  • platform/network/cf/ResourceHandleCFNet.cpp: (WebCore::makeFinalRequest):
17:12 Changeset [42915] by simon.fraser@apple.com
  • 1 edit
    4 adds in trunk/LayoutTests

2009-04-23 Simon Fraser <simon.fraser@apple.com>

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

Add pixel test for zoomed video with controls, with and without a transform.

  • media/video-zoom-controls.html: Added.
  • platform/mac/media/video-zoom-controls-expected.checksum: Added.
  • platform/mac/media/video-zoom-controls-expected.png: Added.
  • platform/mac/media/video-zoom-controls-expected.txt: Added.
17:08 Changeset [42914] by simon.fraser@apple.com
  • 4 edits in trunk

2009-04-27 Simon Fraser <simon.fraser@apple.com>

Reviewed by Adele Peterson

<rdar://problem/6822225> Full page zooming of <video> element in the browser window doesn't work well

Fix a 'size changed' notification thrash that caused standalone video elements
to continually resize after full page zooming, by using the movie's natual size (which is independent
of zooming), rather than its current size (which is not).

Note that this regresses <https://bugs.webkit.org/show_bug.cgi?id=25029>, so we have to
disable the media/video-size-intrinsic-scale.html test. However, we have to stop using
QTMovieCurrentSizeAttribute anyway; this will be addressed, and the test re-enabled via
<rdar://problem/6822344>.

  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::naturalSize):
16:58 Changeset [42913] by simon.fraser@apple.com
  • 3 edits in trunk/WebCore

2009-04-27 Simon Fraser <simon.fraser@apple.com>

Reviewed by Darin Adler

<rdar://problem/6642221> REGRESSION: With the Movie widget, movie trailer fails to
load if movie had been previously viewed

Fix the Dashboard quirk for display:none plug-ins by moving the code from HTMLObjectElement
to HTMLEmebedElement. It has to be in HTMLEmbedElement because the content we care about uses <embed>.

  • html/HTMLEmbedElement.cpp: (WebCore::HTMLEmbedElement::rendererIsNeeded):
  • html/HTMLObjectElement.cpp: (WebCore::HTMLObjectElement::rendererIsNeeded):
16:33 Changeset [42912] by weinig@apple.com
  • 14 edits in trunk/WebCore

2009-04-27 Sam Weinig <sam@webkit.org>

Reviewed by Geoffrey Garen.

Move more window event related code into DOMWindow.

  • dom/Document.cpp: (WebCore::Document::implicitClose): (WebCore::Document::setWindowAttributeEventListener): (WebCore::Document::dispatchWindowEvent): (WebCore::Document::dispatchLoadEvent):
  • dom/Document.h:
  • dom/Element.cpp:
  • dom/Element.h:
  • dom/Node.cpp:
  • dom/Node.h:
  • html/HTMLBodyElement.cpp: (WebCore::HTMLBodyElement::parseMappedAttribute):
  • html/HTMLFrameSetElement.cpp: (WebCore::HTMLFrameSetElement::parseMappedAttribute):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::stopLoading):
  • page/DOMWindow.cpp: (WebCore::DOMWindow::dispatchAllPendingUnloadEvents): (WebCore::DOMWindow::postMessageTimerFired): (WebCore::DOMWindow::dispatchEvent): (WebCore::DOMWindow::dispatchEventWithDocumentAsTarget): (WebCore::DOMWindow::dispatchLoadEvent): (WebCore::DOMWindow::dispatchUnloadEvent): (WebCore::DOMWindow::dispatchBeforeUnloadEvent):
  • page/DOMWindow.h:
  • page/Frame.cpp: (WebCore::Frame::shouldClose):
  • svg/SVGSVGElement.cpp: (WebCore::SVGSVGElement::parseMappedAttribute):
16:14 Changeset [42911] by adele@apple.com
  • 25 edits in trunk

WebCore:

2009-04-27 Douglas R. Davidson <ddavidso@apple.com>

Implement the editing mechanisms and add context menu items for
<rdar://problem/6724106> WebViews need to implement text checking

Reviewed by Justin Garcia.

  • editing/Editor.cpp:
  • editing/Editor.h:
  • editing/TypingCommand.cpp:
  • loader/EmptyClients.h:
  • page/ContextMenuController.cpp:
  • page/EditorClient.h:
  • page/mac/WebCoreViewFactory.h:
  • platform/ContextMenu.cpp:
  • platform/ContextMenuItem.h:
  • platform/LocalizedStrings.h:
  • platform/mac/LocalizedStringsMac.mm:

WebKit:

2009-04-27 Douglas R. Davidson <ddavidso@apple.com>

Add localizable strings for
<rdar://problem/6724106> WebViews need to implement text checking

Reviewed by Justin Garcia.

  • English.lproj/Localizable.strings:

WebKit/mac:

2009-04-27 Douglas R. Davidson <ddavidso@apple.com>

Add the various switches and context menu items needed for
<rdar://problem/6724106> WebViews need to implement text checking
and adopt updatePanels in place of old SPI _updateGrammar.

Reviewed by Justin Garcia.

  • WebCoreSupport/WebContextMenuClient.mm: (fixMenusReceivedFromOldClients):
  • WebCoreSupport/WebEditorClient.h:
  • WebCoreSupport/WebEditorClient.mm: (WebEditorClient::uppercaseWord): (WebEditorClient::lowercaseWord): (WebEditorClient::capitalizeWord): (WebEditorClient::showSubstitutionsPanel): (WebEditorClient::substitutionsPanelIsShowing): (WebEditorClient::toggleSmartInsertDelete): (WebEditorClient::isAutomaticQuoteSubstitutionEnabled): (WebEditorClient::toggleAutomaticQuoteSubstitution): (WebEditorClient::isAutomaticLinkDetectionEnabled): (WebEditorClient::toggleAutomaticLinkDetection): (WebEditorClient::isAutomaticDashSubstitutionEnabled): (WebEditorClient::toggleAutomaticDashSubstitution): (WebEditorClient::isAutomaticTextReplacementEnabled): (WebEditorClient::toggleAutomaticTextReplacement): (WebEditorClient::isAutomaticSpellingCorrectionEnabled): (WebEditorClient::toggleAutomaticSpellingCorrection): (WebEditorClient::checkTextOfParagraph):
  • WebCoreSupport/WebViewFactory.mm: (-[WebViewFactory contextMenuItemTagShowColors]): (-[WebViewFactory contextMenuItemTagCorrectSpellingAutomatically]): (-[WebViewFactory contextMenuItemTagSubstitutionsMenu]): (-[WebViewFactory contextMenuItemTagShowSubstitutions:]): (-[WebViewFactory contextMenuItemTagSmartCopyPaste]): (-[WebViewFactory contextMenuItemTagSmartQuotes]): (-[WebViewFactory contextMenuItemTagSmartDashes]): (-[WebViewFactory contextMenuItemTagSmartLinks]): (-[WebViewFactory contextMenuItemTagTextReplacement]): (-[WebViewFactory contextMenuItemTagTransformationsMenu]): (-[WebViewFactory contextMenuItemTagMakeUpperCase]): (-[WebViewFactory contextMenuItemTagMakeLowerCase]): (-[WebViewFactory contextMenuItemTagCapitalize]):
  • WebView/WebHTMLView.mm: (-[WebHTMLView validateUserInterfaceItemWithoutDelegate:]): (-[WebHTMLView isAutomaticQuoteSubstitutionEnabled]): (-[WebHTMLView setAutomaticQuoteSubstitutionEnabled:]): (-[WebHTMLView toggleAutomaticQuoteSubstitution:]): (-[WebHTMLView isAutomaticLinkDetectionEnabled]): (-[WebHTMLView setAutomaticLinkDetectionEnabled:]): (-[WebHTMLView toggleAutomaticLinkDetection:]): (-[WebHTMLView isAutomaticDashSubstitutionEnabled]): (-[WebHTMLView setAutomaticDashSubstitutionEnabled:]): (-[WebHTMLView toggleAutomaticDashSubstitution:]): (-[WebHTMLView isAutomaticTextReplacementEnabled]): (-[WebHTMLView setAutomaticTextReplacementEnabled:]): (-[WebHTMLView toggleAutomaticTextReplacement:]): (-[WebHTMLView isAutomaticSpellingCorrectionEnabled]): (-[WebHTMLView setAutomaticSpellingCorrectionEnabled:]): (-[WebHTMLView toggleAutomaticSpellingCorrection:]):
  • WebView/WebHTMLViewInternal.h:
  • WebView/WebPreferenceKeysPrivate.h:
  • WebView/WebUIDelegatePrivate.h:
  • WebView/WebView.mm: (-[WebViewPrivate init]): (-[WebView validateUserInterfaceItemWithoutDelegate:]): (-[WebView setGrammarCheckingEnabled:]): (-[WebView isAutomaticQuoteSubstitutionEnabled]): (-[WebView isAutomaticLinkDetectionEnabled]): (-[WebView isAutomaticDashSubstitutionEnabled]): (-[WebView isAutomaticTextReplacementEnabled]): (-[WebView isAutomaticSpellingCorrectionEnabled]): (-[WebView setAutomaticQuoteSubstitutionEnabled:]): (-[WebView toggleAutomaticQuoteSubstitution:]): (-[WebView setAutomaticLinkDetectionEnabled:]): (-[WebView toggleAutomaticLinkDetection:]): (-[WebView setAutomaticDashSubstitutionEnabled:]): (-[WebView toggleAutomaticDashSubstitution:]): (-[WebView setAutomaticTextReplacementEnabled:]): (-[WebView toggleAutomaticTextReplacement:]): (-[WebView setAutomaticSpellingCorrectionEnabled:]): (-[WebView toggleAutomaticSpellingCorrection:]):
  • WebView/WebViewPrivate.h:
16:12 Changeset [42910] by weinig@apple.com
  • 2 edits in trunk/WebCore

2009-04-27 Sam Weinig <sam@webkit.org>

Reviewed by Geoffrey Garen.

Fix for <rdar://problem/6772944> REGRESSION (r42223): PLT 2% slower, i-Bench 3% slower

Be slightly less eager in collecting after page close/navigation by increasing
the time before collection from 0 seconds to .5 seconds.

3% speedup on the PLT.

  • bindings/js/GCController.cpp: (WebCore::GCController::garbageCollectSoon):
16:06 Changeset [42909] by ddkilzer@apple.com
  • 10 edits
    2 adds in trunk

Consolidate runtime application checks for Apple Mail and Safari

Reviewed by Mark Rowe and Darin Adler.

WebCore:

  • WebCore.base.exp: Added exports.
  • WebCore.xcodeproj/project.pbxproj: Added RuntimeApplicationChecks.{h|mm} source files to the project.
  • page/mac/EventHandlerMac.mm: (WebCore::EventHandler::needsKeyboardEventDisambiguationQuirks): Removed local checkedSafari and isSafari variables and switched to use applicationIsSafari().
  • platform/mac/RuntimeApplicationChecks.h: Added.
  • platform/mac/RuntimeApplicationChecks.mm: Added. (WebCore::applicationIsAppleMail): Runtime application check for Apple Mail. (WebCore::applicationIsSafari): Runtime application check for Safari.

WebKit:

  • WebCoreSupport/WebContextMenuClient.mm: (isAppleMail): Removed. (fixMenusToSendToOldClients): Switched to use applicationIsAppleMail().
  • WebView/WebFrame.mm: (-[WebFrame reload]): Switched to use applicationIsSafari().
  • WebView/WebPDFView.mm: (-[WebPDFView menuForEvent:]): Ditto.
  • WebView/WebResource.mm: (+[WebResource _needMailThreadWorkaroundIfCalledOffMainThread]): Switched to use applicationIsAppleMail().
  • WebView/WebView.mm: (runningLeopardMail): Ditto. (runningTigerMail): Ditto. (-[WebView _needsKeyboardEventDisambiguationQuirks]): Switched to use applicationIsSafari().
15:34 Changeset [42908] by cfleizach@apple.com
  • 2 edits in trunk/WebCore

Bug 25374: AX: No AXValue change sent when text is auto-inserted
https://bugs.webkit.org/show_bug.cgi?id=25374

Updated to fix regression test failures

14:56 Changeset [42907] by xan@webkit.org
  • 2 edits in trunk/WebCore

2009-04-16 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

https://bugs.webkit.org/show_bug.cgi?id=21546
[GTK] ATK accessibility enhancements

Based on a patch by Alp Toker.

Cover more WebCore role -> ATK role conversions.

  • page/gtk/AccessibilityObjectWrapperAtk.cpp:
14:40 Changeset [42906] by xan@webkit.org
  • 2 edits in trunk/WebCore

2009-04-18 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

https://bugs.webkit.org/show_bug.cgi?id=21546
[GTK] ATK accessibility enhancements

Do not call ATK setters in the getters, just return the
value. Among other things calling the setter will emit the notify
signal, which is wrong.

  • page/gtk/AccessibilityObjectWrapperAtk.cpp:
14:33 Changeset [42905] by xan@webkit.org
  • 2 edits in trunk/WebCore

2009-04-16 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

https://bugs.webkit.org/show_bug.cgi?id=21546
[GTK] ATK accessibility enhancements

Based on a patch by Alp Toker.

Implement AtkImage interface.

  • page/gtk/AccessibilityObjectWrapperAtk.cpp:
14:25 Changeset [42904] by kov@webkit.org
  • 2 edits in trunk/WebCore

2009-04-27 Gustavo Noronha Silva <gns@gnome.org>

Unreviewed debug build fix.

  • platform/gtk/gtk2drawing.c: (moz_gtk_toggle_paint):
13:12 Changeset [42903] by dglazkov@chromium.org
  • 3 edits in trunk/WebCore

2009-04-27 Dimitri Glazkov <dglazkov@chromium.org>

Build fix.

https://bugs.webkit.org/show_bug.cgi?id=25382
Remove direct reference to CallFrame.h include to pacify Win build.

  • bindings/js/JSDOMBinding.h: Removed ScriptState.h include.
  • bindings/js/ScriptState.h: Replaced CallFrame.h with JSDOMBinding.h
12:49 Changeset [42902] by cfleizach@apple.com
  • 2 edits in trunk/WebCore

Bug 25374: AX: No AXValue change sent when text is auto-inserted
https://bugs.webkit.org/show_bug.cgi?id=25374

12:46 Changeset [42901] by dglazkov@chromium.org
  • 10 edits in trunk/WebCore

2009-04-27 Dimitri Glazkov <dglazkov@chromium.org>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=25382
Move scriptStateFromNode to ScriptState.

  • bindings/js/JSDOMBinding.cpp: Moved scriptStateFromNode to ScriptState.
  • bindings/js/JSDOMBinding.h: Ditto.
  • bindings/js/ScriptState.cpp: (WebCore::scriptStateFromNode): Added.
  • bindings/js/ScriptState.h: Ditto.
  • bindings/v8/ScriptState.cpp: (WebCore::scriptStateFromNode): Added.
  • bindings/v8/ScriptState.h: Ditto.
  • dom/NodeFilter.h: Removed JSDOMBinding header include.
  • dom/NodeIterator.h: Ditto.
  • dom/TreeWalker.h: Ditto.
12:44 Changeset [42900] by cfleizach@apple.com
  • 4 edits in trunk

Bug 25428: 5 crashes in DumpRenderTree at com.apple.WebCore • -[AccessibilityObjectWrapper accessibilityAttributeValue:] + 830
https://bugs.webkit.org/show_bug.cgi?id=25428

12:33 Changeset [42899] by andersca@apple.com
  • 2 edits in trunk/WebKitExamplePlugins

2009-04-27 Anders Carlsson <andersca@apple.com>

Reviewed by Adam Roben.

Update the example plug-in for new changes to the Cocoa event model.


  • NetscapeCocoaPlugin/main.m: (handleDraw): (handleMouseEvent): (NPP_HandleEvent):
11:54 Changeset [42898] by kdecker@apple.com
  • 2 edits in trunk/WebKit/mac

Fix the Tiger build.


  • Plugins/WebNetscapePluginView.mm: The base class of WebNetscapePluginView.mm, WebBaseNetscapePluginView, already implemented the two methods below. But the Tiger compiler didn't know that. (-[WebNetscapePluginView webView]): (-[WebNetscapePluginView webFrame]):
10:50 Changeset [42897] by kdecker@apple.com
  • 10 edits
    2 adds in trunk/WebKit

Reviewed by Anders Carlsson.

<rdar://problem/6352982>


  • Plugins/WebBaseNetscapePluginView.mm: Removed checkIfAllowedToLoadURL:frame:callbackFunc, cancelCheckIfAllowedToLoadURL, and _webPluginContainerCancelCheckIfAllowedToLoadRequest from the base class. These methods now exist in the subclass WebNetscapePluginView.


Added WebNetscapeContainerCheckContextInfo, which is used as a "contextInfo" object in
-[WebNetscapePluginView checkIfAllowedToLoadURL:frame:callbackFunc:]

  • Plugins/WebNetscapeContainerCheckContextInfo.h: Added.
  • Plugins/WebNetscapeContainerCheckContextInfo.mm: Added. (-[WebNetscapeContainerCheckContextInfo initWithCheckRequestID:callbackFunc:]): Added desiginated initializer. (-[WebNetscapeContainerCheckContextInfo checkRequestID]): Added. Returns the checkRequestID. (-[WebNetscapeContainerCheckContextInfo callback]): Added. Returns the callback.


  • Plugins/WebNetscapePluginView.h: Added two new ivars: _containerChecksInProgress and _currentContainerCheckRequestID.
  • Plugins/WebNetscapePluginView.mm: #import both WebPluginContainerCheck.h and WebNetscapeContainerCheckContextInfo.h


(-[WebNetscapePluginView checkIfAllowedToLoadURL:frame:callbackFunc:]): Added. This is the implementation of WKN_CheckIfAllowedToLoadURL.
Here, we increment the request ID and start the container check.


(-[WebNetscapePluginView _containerCheckResult:contextInfo:]): Added. This is a callback method for WebPluginContainerCheck.
It's where we actually call into the plug-in and provide the allow-or-deny result.


(-[WebNetscapePluginView cancelCheckIfAllowedToLoadURL:]): Added. This is the implementation of WKN_CancelCheckIfAllowedToLoadURL.
Here we lookup the check, cancel it, and remove it from _containerChecksInProgress.


(-[WebNetscapePluginView _webPluginContainerCancelCheckIfAllowedToLoadRequest:]): Added. WebPluginContainerCheck automatically calls
this method after invoking our _containerCheckResult: selector. It works this way because calling -[WebPluginContainerCheck cancel] allows
it to do it's teardown process.


(-[WebNetscapePluginView fini]): Release _containerChecksInProgress ivar.


  • Plugins/WebPluginContainerCheck.h: Removed initWithRequest: method from header; no client was using this method directly.
  • Plugins/WebPluginContainerCheck.mm: (+[WebPluginContainerCheck checkWithRequest:target:resultObject:selector:controller:contextInfo:]): Added optional contextInfo parameter. (-[WebPluginContainerCheck _continueWithPolicy:]): If there's a contextInfo object, pass it as a parameter to resultSelector. (-[WebPluginContainerCheck cancel]): Release _contextInfo ivar. (-[WebPluginContainerCheck contextInfo]): Added new method. Returns the contextInfo object, if one so exists.
  • Plugins/WebPluginController.mm: (-[WebPluginController _webPluginContainerCheckIfAllowedToLoadRequest:inFrame:resultObject:selector:]): Pass nil for contextInfo. WebKit-style plug-ins do not need additional context information.
09:57 Changeset [42896] by ap@webkit.org
  • 6 edits in trunk/WebCore

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=25399
<rdar://problem/6633943> REGRESSION: Many crashes reported accessing Lexis/Nexis database,
beneath WebCore::Cache::evict

The crash happened because a cached resource handle was removed from a document's cached
resources map twice recursively, so a destructor was called for a value in a deleted bucket.
The first call was from Cache::evict, and when destroying CachedResourceHandle destroyed
CachedResource, DocLoader::removeCachedResource() was called again, with HashMap being in
an inconsistent state.

I couldn't fully reconstruct the loading sequence to make a test.

  • loader/Cache.cpp: (WebCore::Cache::revalidateResource): Assert that the resource being revalidated is in cache (it makes no sense to revalidate one that isn't). (WebCore::Cache::evict): Don't remove the resource from document's map. Removing a resource from the cache in no way implies that documents no longer use the old version. This fixes the crash, and also fixes many cases of resource content being unavailable in Web Inspector.
  • loader/CachedResource.h: (WebCore::CachedResource::setInCache): When bringing a revalidated resource back to cache, reset m_isBeingRevalidated to maintain the invariant of resources being revalidated never being in cache. This fixes another assertion I saw on LexisNexis search: in rare cases, switchClientsToRevalidatedResource() results in the same resource being requested again, but we were only enforcing CachedResource invariants after calling this function. (WebCore::CachedResource::unregisterHandle): Assert that the counter doesn't underflow.
  • loader/DocLoader.cpp: (WebCore::DocLoader::removeCachedResource): Assert that the passed resource is removed, not some other resource that happens to have the same URL (this used to fail on LexisNexis search before this patch).
  • loader/ImageDocument.cpp: (WebCore::ImageTokenizer::write): Replaced ASSERT_NOT_REACHED with notImplemented(). This method can be legally called via document.write(), and should work. LexisNexis takes this code path, but apparently has a fallback for Safari, so it doesn't affect site functionality.
  • loader/CachedResource.cpp: (WebCore::CachedResource::clearResourceToRevalidate): Don't assert that m_resourceToRevalidate is being revalidated - this may no longer be true, because we now reset this member in CachedResource::setInCache().
09:24 Changeset [42895] by mitz@apple.com
  • 6 edits in trunk/WebCore

Reviewed by Sam Weinig.

  • make BidiContext a RefCounted class, starting with a refcount of 1, and share the root BidiContexts.
  • platform/graphics/GraphicsContext.cpp: (WebCore::GraphicsContext::drawBidiText): Use BidiContext::create().
  • platform/text/BidiContext.cpp: (WebCore::BidiContext::create): Added. For the four "root" contexts, returns a reference to a shared static BidiContext.
  • platform/text/BidiContext.h: (WebCore::BidiContext::BidiContext):
  • platform/text/BidiResolver.h: (WebCore::::commitExplicitEmbedding): Use BidiContext::create().
  • rendering/bidi.cpp: (WebCore::RenderBlock::determineStartPosition): Ditto.
08:23 Changeset [42894] by jmalonzo@webkit.org
  • 3 edits in trunk/WebCore

2009-04-27 Jan Michael Alonzo <jmalonzo@webkit.org>

Rubber-stamped by Gustavo Noronha.

Build fix for r42893: change GDK_exclaim to GDK_exclam

  • platform/chromium/KeyCodeConversionGtk.cpp: (WebCore::windowsKeyCodeForKeyEvent):
  • platform/gtk/KeyEventGtk.cpp: (WebCore::windowsKeyCodeForKeyEvent):
08:08 Changeset [42893] by jmalonzo@webkit.org
  • 3 edits in trunk/WebCore

2009-04-27 Fumitoshi Ukai <ukai@google.com>

Reviewed by Xan Lopez.

Fix wrong key code conversion.

Missing conversion for GDK_exlaim ('!').
Parens are opposite: GDK_parenleft is open paren '('
and GDK_parenright is close paren ')'.
https://bugs.webkit.org/show_bug.cgi?id=25367

  • platform/chromium/KeyCodeConversionGtk.cpp: (WebCore::windowsKeyCodeForKeyEvent):
  • platform/gtk/KeyEventGtk.cpp: (WebCore::windowsKeyCodeForKeyEvent):
08:08 Changeset [42892] by jmalonzo@webkit.org
  • 3 edits in trunk/WebCore

2009-04-27 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Gustavo Noronha.

[Gtk] gtkdrawing update
https://bugs.webkit.org/show_bug.cgi?id=25408

Update to the 25 Apr 2009 version of gtk2drawing.c and
gtkdrawing.h from Mozilla (http://hg.mozilla.org/mozilla-central)

  • platform/gtk/gtk2drawing.c: (setup_widget_prototype): (moz_gtk_get_combo_box_entry_arrow): (ensure_toolbar_widget): (ensure_tree_header_cell_widget): (moz_gtk_init): (moz_gtk_button_get_inner_border): (moz_gtk_toggle_paint): (moz_gtk_caret_paint): (moz_gtk_entry_paint): (moz_gtk_tree_header_cell_paint): (moz_gtk_combo_box_paint): (moz_gtk_toolbar_separator_paint): (moz_gtk_menu_separator_paint): (moz_gtk_get_widget_border): (moz_gtk_get_tab_scroll_arrow_size): (moz_gtk_get_toolbar_separator_width): (moz_gtk_get_menu_separator_height): (moz_gtk_widget_paint): (moz_gtk_shutdown):
  • platform/gtk/gtkdrawing.h:
08:08 Changeset [42891] by jmalonzo@webkit.org
  • 2 edits in trunk/WebKitTools

2009-04-27 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Gustavo Noronha.

[GTK] Make layout tests / jsc tests usable without build-webkit
https://bugs.webkit.org/show_bug.cgi?id=21784

This requires WEBKITOUTPUTDIR to be set to the build directory.
Set the location of the product dir to the location pointed to by
WEBKITOUTPUTDIR.

  • Scripts/webkitdirs.pm:
07:54 Changeset [42890] by kov@webkit.org
  • 2 edits in trunk/WebKitTools

2009-04-27 Gustavo Noronha Silva <gns@gnome.org>

Reviewed by Adam Roben.

Use libsoup's facilities to resolve relative URLs, instead, since
it is a more robust method.

  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (LayoutTestController::queueLoad):
07:53 Changeset [42889] by kov@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-04-27 Gustavo Noronha Silva <gns@gnome.org>

Reviewed by Xan Lopez.

https://bugs.webkit.org/show_bug.cgi?id=25409
[GTK] incorrectly fallsback on cancellation

Check whether we should fallback also on didFailLoad, so that we
don't get error pages showing up in parts of the content of some
sites because of, for instance, location changes.

  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::dispatchDidFailLoad):
07:53 Changeset [42888] by kov@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-04-27 Gustavo Noronha Silva <gns@gnome.org>

Reviewed by Xan Lopez.

https://bugs.webkit.org/show_bug.cgi?id=25409
[GTK] incorrectly fallsback on cancellation

Correctly handle cancellation and frame loading interruption
caused by policy change. This also fixes
fast/frames/location-change.html.

  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::shouldFallBack):
06:55 Changeset [42887] by ariya@webkit.org
  • 2 edits in trunk/WebCore

2009-04-27 Ariya Hidayat <ariya.hidayat@nokia.com>

Reviewed by Tor Arne Vestbø.

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

[Qt] Widget painting should follow the layout direction (LTR, RTL)
of the element style, not the application layout direction.

  • platform/qt/RenderThemeQt.cpp: (WebCore::RenderThemeQt::applyTheme):
06:27 Changeset [42886] by aroben@apple.com
  • 3 edits in trunk/JavaScriptCore

Windows Debug build fix

Not sure why the buildbots weren't affected by this problem.

  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj: Let VS re-order the file list, and added JavaScriptCore[_debug].def to the project. This was not necessary for the fix, but made making the fix easier.
  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore_debug.def: Removed a function that no longer exists.

04/26/09:

22:58 Changeset [42885] by xan@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-04-26 Xan Lopez <xlopez@igalia.com>

Reviewed by Mark Rowe.

Plug memory leak.

  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::dispatchDidFailLoad):
22:17 Changeset [42884] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

2009-04-26 Gavin Barraclough <barraclough@apple.com>

Reviewed by Weinig Sam.

Fix for https://bugs.webkit.org/show_bug.cgi?id=25416
"Cached prototype accesses unsafely hoist property storage load above structure checks."


Do not hoist the load of the pointer to the property storage array.

No performance impact.

  • jit/JITPropertyAccess.cpp: (JSC::JIT::privateCompileGetByIdProto): (JSC::JIT::privateCompileGetByIdProtoList):
21:32 Changeset [42883] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

2009-04-26 Gavin Barraclough <barraclough@apple.com>

Reviewed by Geoffrey "Gaffe or energy?" Garen.

Randomize address requested by ExecutableAllocatorFixedVMPool.

  • jit/ExecutableAllocatorFixedVMPool.cpp: (JSC::FixedVMPoolAllocator::FixedVMPoolAllocator):
17:03 Changeset [42882] by weinig@apple.com
  • 10 edits
    3 adds in trunk

WebCore:

2009-04-26 Sam Weinig <sam@webkit.org>

Reviewed by Dan Bernstein.

Fix for https://bugs.webkit.org/show_bug.cgi?id=25412
The Window object should be an EventTarget

Test: fast/dom/Window/dispatchEvent.html

  • bindings/js/JSEventTarget.cpp: (WebCore::toJS): Add DOMWindow case. (WebCore::toEventTarget): Ditto.
  • dom/EventTarget.cpp: (WebCore::EventTarget::toDOMWindow): Added.
  • dom/EventTarget.h:
  • page/DOMWindow.cpp: (WebCore::DOMWindow::scriptExecutionContext): Added. Returns the document. (WebCore::DOMWindow::dispatchEvent): Added.
  • page/DOMWindow.h: (WebCore::DOMWindow::toDOMWindow): (WebCore::DOMWindow::refEventTarget): (WebCore::DOMWindow::derefEventTarget):
  • page/DOMWindow.idl:

LayoutTests:

2009-04-26 Sam Weinig <sam@webkit.org>

Reviewed by Dan Bernstein.

Tests for https://bugs.webkit.org/show_bug.cgi?id=25412
The Window object should be an EventTarget

  • fast/dom/Window/dispatchEvent-expected.txt: Added.
  • fast/dom/Window/dispatchEvent.html: Added.
  • fast/dom/Window/resources/dispatchEvent.js: Added.
  • fast/dom/Window/window-properties-expected.txt
  • platform/mac-leopard/fast/dom/Window/window-properties-expected.txt:
14:09 Changeset [42881] by weinig@apple.com
  • 2 edits
    2 deletes in trunk

JavaScriptCore:

2009-04-26 Sam Weinig <sam@webkit.org>

Reviewed by Eric Seidel.

Remove scons-based build system.

WebCore:

2009-04-26 Sam Weinig <sam@webkit.org>

Reviewed by Eric Seidel.

Remove scons-based build system.

  • WebCore.scons: Removed.
11:11 Changeset [42880] by kov@webkit.org
  • 2 edits in trunk/WebKitTools

2009-04-26 Gustavo Noronha Silva <gns@gnome.org>

Reviewed by Xan Lopez.

Resolve relative URLs when queueing loads. This fixes
post-goback-same-url.html timing out.

  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (LayoutTestController::queueLoad):
10:23 Changeset [42879] by xan@webkit.org
  • 5 edits in trunk/WebKit/gtk

2009-04-26 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

https://bugs.webkit.org/show_bug.cgi?id=25391
G_SIGNAL_ACTION flag in WebKitWebView signals

Most of the signals in WebKitGTK+ have the G_SIGNAL_ACTION flag,
but I don't think this makes sense. The flag is meant for those
signals that can be emitted by the user to trigger some action in
the view, as if it were a normal API, which is not the case for
most of them.

  • webkit/webkitdownload.cpp: (_WebKitDownloadPrivate::webkit_download_class_init):
  • webkit/webkitwebframe.cpp:
  • webkit/webkitwebinspector.cpp: (_WebKitWebInspectorPrivate::webkit_web_inspector_class_init):
  • webkit/webkitwebview.cpp:
10:11 Changeset [42878] by xan@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-04-26 Xan Lopez <xlopez@igalia.com>

Unreviewed, actually apply the right patch.

  • webkit/webkitwebview.cpp: (webkit_web_view_move_cursor):
10:05 Changeset [42877] by xan@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-04-26 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Pass a variable to hold the return value of the 'move-cursor'
signal, otherwise a crash can happen.

  • webkit/webkitwebview.cpp: (webkit_web_view_move_cursor):
07:25 Changeset [42876] by darin@apple.com
  • 2 edits in trunk/WebCore

2009-04-26 Darin Adler <darin@apple.com>

Suggested by Darin Fisher.

Improve the fix for bug 25355: Crash when Creating New Tab or New Window when set to open Same Page
https://bugs.webkit.org/show_bug.cgi?id=25355
rdar://problem/6823543

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::loadItem): Allow shouldScroll to be true even if m_currentHistoryItem is 0. Not sure if when this case can really arise in practice, but it's good to match the original logic more closely.
00:02 QtWebKitTodo edited by hausmann@webkit.org
(diff)
00:01 Changeset [42875] by krit@webkit.org
  • 2 edits in trunk/WebCore

2009-04-25 Dirk Schulze <krit@webkit.org>

Reviewed by Oliver Hunt.

A width or height of zero for the destination or source rect of an
image causes a not invertible pattern matrix.
This crahes newer versions of Cairo and give some graphic gliches in
Canvas.
With this patch we check if there is something to draw and return if not.

  • platform/graphics/cairo/ImageCairo.cpp: (WebCore::BitmapImage::draw):
Note: See TracTimeline for information about the timeline view.