WebKit-https.git
11 years ago2009-02-18 Dimitri Glazkov <dglazkov@chromium.org>
dglazkov@chromium.org [Fri, 20 Feb 2009 20:36:21 +0000 (20:36 +0000)]
2009-02-18  Dimitri Glazkov  <dglazkov@chromium.org>

        Reviewed by Antti Koivisto.

        https://bugs.webkit.org/show_bug.cgi?id=23999
        Split off MIME type parsing into its own class.

        * GNUmakefile.am: Added ContentType sources.
        * WebCore.pro: Ditto.
        * WebCore.scons: Ditto.
        * WebCore.vcproj/WebCore.vcproj: Ditto.
        * WebCore.xcodeproj/project.pbxproj: Ditto.
        * WebCoreSources.bkl: Ditto.
        * html/HTMLMediaElement.cpp: Changed to use ContentType.
        (WebCore::HTMLMediaElement::selectMediaURL):
        * platform/ContentType.cpp: Added.
        (WebCore::ContentType::ContentType):
        (WebCore::ContentType::parameter):
        (WebCore::ContentType::type):
        * platform/ContentType.h: Added.
        (WebCore::ContentType::raw):
        * platform/MIMETypeRegistry.cpp:
        * platform/MIMETypeRegistry.h:
        * platform/graphics/MediaPlayer.cpp:
        (WebCore::MediaPlayer::load):
        * rendering/style/ContentData.h: Renamed ContentType to StyleContentType.
        * rendering/style/RenderStyleConstants.h: Ditto.
        (WebCore::):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41113 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-20 Holger Hans Peter Freyther <zecke@selfish.org>
zecke@webkit.org [Fri, 20 Feb 2009 20:32:43 +0000 (20:32 +0000)]
2009-02-20  Holger Hans Peter Freyther  <zecke@selfish.org>

        Unreviewed build fix.

        Use CString() instead of 0.

        * platform/gtk/FileSystemGtk.cpp:
        (WebCore::openTemporaryFile):
        * platform/qt/FileSystemQt.cpp:
        (WebCore::openTemporaryFile):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41112 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-20 Gustavo Noronha Silva <gns@gnome.org>
kov@webkit.org [Fri, 20 Feb 2009 20:10:44 +0000 (20:10 +0000)]
2009-02-20  Gustavo Noronha Silva  <gns@gnome.org>

        Reviewed by Holger Freyther.

        When dealing with local files, use a path instead of an URI. GFile
        has problems decoding URIs with percent signs on them.

        * platform/network/soup/ResourceHandleSoup.cpp:
        (WebCore::ResourceHandle::startGio):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41111 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-20 Gustavo Noronha Silva <gns@gnome.org>
kov@webkit.org [Fri, 20 Feb 2009 19:41:24 +0000 (19:41 +0000)]
2009-02-20  Gustavo Noronha Silva  <gns@gnome.org>

        Reviewed by Holger Freyther.

        Protect the ResourceHandle instance from being destroyed by
        didReceiveData inside the GIO readCallback call, so that
        cancelling caused by scripts is handled correctly.

        * platform/network/soup/ResourceHandleSoup.cpp:
        (WebCore::readCallback):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41110 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago Make IconDatabaseNone.cpp compile with -Wunused and pass check-for-exit-time...
ddkilzer@apple.com [Fri, 20 Feb 2009 19:27:13 +0000 (19:27 +0000)]
    Make IconDatabaseNone.cpp compile with -Wunused and pass check-for-exit-time-destructors

        Reviewed by Sam Weinig.

        * loader/icon/IconDatabaseNone.cpp:
        (WebCore::IconDatabase::defaultDatabaseFilename): Use DEFINE_STATIC_LOCAL().
        (WebCore::IconDatabase::open): Commented out unused parameter.
        (WebCore::IconDatabase::setPrivateBrowsingEnabled): Ditto.
        (WebCore::IconDatabase::iconForPageURL): Ditto.
        (WebCore::IconDatabase::iconURLForPageURL): Ditto.
        (WebCore::IconDatabase::defaultIcon): Ditto.
        (WebCore::IconDatabase::retainIconForPageURL): Ditto.
        (WebCore::IconDatabase::releaseIconForPageURL): Ditto.
        (WebCore::IconDatabase::setIconDataForIconURL): Ditto.
        (WebCore::IconDatabase::setIconURLForPageURL): Ditto.
        (WebCore::IconDatabase::setEnabled): Ditto.
        (WebCore::IconDatabase::pageURLMappingCount): Added stub.
        (WebCore::IconDatabase::retainedPageURLCount): Ditto.
        (WebCore::IconDatabase::iconRecordCount): Ditto.
        (WebCore::IconDatabase::iconRecordCountWithData): Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41109 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago Bug 23983: Remove illegal HTML tags from Sunspider web pages
ddkilzer@apple.com [Fri, 20 Feb 2009 16:27:11 +0000 (16:27 +0000)]
    Bug 23983: Remove illegal HTML tags from Sunspider web pages

        <https://bugs.webkit.org/show_bug.cgi?id=23983>

        Reviewed by Alexey Proskuryakov.

        * perf/sunspider-0.9/3d-cube.html: Removed </link> tag.
        * perf/sunspider-0.9/3d-morph.html: Ditto.
        * perf/sunspider-0.9/3d-raytrace.html: Ditto.
        * perf/sunspider-0.9/access-binary-trees.html: Ditto.
        * perf/sunspider-0.9/access-fannkuch.html: Ditto.
        * perf/sunspider-0.9/access-nbody.html: Ditto.
        * perf/sunspider-0.9/access-nsieve.html: Ditto.
        * perf/sunspider-0.9/bitops-3bit-bits-in-byte.html: Ditto.
        * perf/sunspider-0.9/bitops-bits-in-byte.html: Ditto.
        * perf/sunspider-0.9/bitops-bitwise-and.html: Ditto.
        * perf/sunspider-0.9/bitops-nsieve-bits.html: Ditto.
        * perf/sunspider-0.9/controlflow-recursive.html: Ditto.
        * perf/sunspider-0.9/crypto-aes.html: Ditto.
        * perf/sunspider-0.9/crypto-md5.html: Ditto.
        * perf/sunspider-0.9/crypto-sha1.html: Ditto.
        * perf/sunspider-0.9/date-format-tofte.html: Ditto.
        * perf/sunspider-0.9/date-format-xparb.html: Ditto.
        * perf/sunspider-0.9/math-cordic.html: Ditto.
        * perf/sunspider-0.9/math-partial-sums.html: Ditto.
        * perf/sunspider-0.9/math-spectral-norm.html: Ditto.
        * perf/sunspider-0.9/regexp-dna.html: Ditto.
        * perf/sunspider-0.9/string-base64.html: Ditto.
        * perf/sunspider-0.9/string-fasta.html: Ditto.
        * perf/sunspider-0.9/string-tagcloud.html: Ditto.
        * perf/sunspider-0.9/string-unpack-code.html: Ditto.
        * perf/sunspider-0.9/string-validate-input.html: Ditto.
        * perf/sunspider-0.9/sunspider-driver.html: Ditto.
        * perf/sunspider-0.9/sunspider-results.html: Removed </link> and </input> tags.
        * perf/sunspider-0.9/sunspider.html: Removed </link> tag.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41108 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago Add comment to generated code when shadowing a built-in object
ddkilzer@apple.com [Fri, 20 Feb 2009 16:23:35 +0000 (16:23 +0000)]
    Add comment to generated code when shadowing a built-in object

        Reviewed by Sam Weinig.

        * bindings/scripts/CodeGeneratorJS.pm: Added comment to
        generated code output.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41107 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-20 Avi Drissman <avi@chromium.org>
darin@chromium.org [Fri, 20 Feb 2009 15:34:42 +0000 (15:34 +0000)]
2009-02-20  Avi Drissman  <avi@chromium.org>

        Reviewed by Eric Seidel.

        https://bugs.webkit.org/show_bug.cgi?id=24036
        Keyboard events need disambiguation on the Linux platform too.

        * platform/chromium/PlatformKeyboardEventChromium.cpp:
        (WebCore::PlatformKeyboardEvent::disambiguateKeyDownEvent):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41106 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-19 Darin Fisher <darin@chromium.org>
darin@chromium.org [Fri, 20 Feb 2009 15:31:35 +0000 (15:31 +0000)]
2009-02-19  Darin Fisher  <darin@chromium.org>

        Reviewed by Eric Seidel.

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

        Several improvements to CString:
        1- Make it possible to initialize a CString from a CStringBuffer
        2- Make it possible to get a CStringBuffer from a CString
        3- Change CStringBuffer::data() to return a const pointer to ward off mutation
        4- Remove unused releaseBuffer() methods.
        5- Make CStringBuffer::create() private to force consumers to get a CStringBuffer from a CString.

        * platform/text/CString.cpp:
        (WebCore::CString::init):
        (WebCore::CString::mutableData):
        (WebCore::CString::newUninitialized):
        (WebCore::CString::copyBufferIfNeeded):
        * platform/text/CString.h:
        (WebCore::CStringBuffer::data):
        (WebCore::CStringBuffer::length):
        (WebCore::CStringBuffer::create):
        (WebCore::CStringBuffer::mutableData):
        (WebCore::CString::CString):
        (WebCore::CString::buffer):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41105 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-20 Gustavo Noronha Silva <gns@gnome.org>
kov@webkit.org [Fri, 20 Feb 2009 13:25:01 +0000 (13:25 +0000)]
2009-02-20  Gustavo Noronha Silva  <gns@gnome.org>

        Unreviewed build fix after r41100.

        * GNUmakefile.am:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41104 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago<rdar://problem/6606660> 2==null returns true in 64bit jit
oliver@apple.com [Fri, 20 Feb 2009 13:14:28 +0000 (13:14 +0000)]
<rdar://problem/6606660> 2==null returns true in 64bit jit

Reviewed by Mark Rowe

Code for op_eq_null and op_neq_null was incorrectly performing
a 32bit compare, which truncated the type tag from an integer
immediate, leading to incorrect behaviour.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41103 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-19 Dmitry Titov <dimich@chromium.org>
dimich@chromium.org [Fri, 20 Feb 2009 07:23:00 +0000 (07:23 +0000)]
2009-02-19  Dmitry Titov  <dimich@chromium.org>

        Reviewed by Alexey Proskuryakov.

        https://bugs.webkit.org/show_bug.cgi?id=24017
        Remove some usage of Document in Worker.

        * dom/WorkerContext.h:
        (WebCore::WorkerContext::userAgent):
        * dom/Document.cpp:
        (WebCore::Document::userAgent):
        * dom/Document.h:
        * dom/ScriptExecutionContext.h:
        Added virtual ScriptExecutionContext::userAgent(const KURL&).
        Document implementation uses FrameLoader::userAgent and
        WorkerContext receives the string on creation and stores it in a member.

        * dom/Worker.cpp:
        (WebCore::Worker::Worker):
        (WebCore::Worker::notifyFinished):
        * dom/Worker.h:
        (WebCore::Worker::create):
        Instead of Document the Worker constructor now gets a ScriptExecutionContext.
        Start using some methods on SEC (like completeURL() and userAgent()).
        For others, explicitly case to Document and add a FIXME.
        Remove Worker::document() too.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41102 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-20 Gustavo Noronha Silva <gns@gnome.org>
kov@webkit.org [Fri, 20 Feb 2009 06:20:38 +0000 (06:20 +0000)]
2009-02-20  Gustavo Noronha Silva  <gns@gnome.org>

        Rubber-stamped by Holger Freyther.

        Do not set httpStatus to SOUP_STATUS_OK when serving local files
        to match other ports' behavior, fixing xmlhttprequest test
        regressions.

        * platform/network/soup/ResourceHandleSoup.cpp:
        (WebCore::queryInfoCallback):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41101 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-19 Geoffrey Garen <ggaren@apple.com>
ggaren@apple.com [Fri, 20 Feb 2009 06:04:21 +0000 (06:04 +0000)]
2009-02-19  Geoffrey Garen  <ggaren@apple.com>

        Reviewed by Gavin Barraclough.

        First step in splitting JIT functionality out of the Interpreter class:
        Created JITStubs.h/.cpp, and moved Interpreter::cti_* into JITStubs.cpp.

        Functions that the Interpreter and JITStubs share moved to Operations.h/.cpp.

        * GNUmakefile.am:
        * JavaScriptCore.pri:
        * JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:
        * JavaScriptCore.xcodeproj/project.pbxproj:
        * interpreter/Interpreter.cpp:
        (JSC::Interpreter::resolveBase):
        (JSC::Interpreter::checkTimeout):
        (JSC::Interpreter::privateExecute):
        * interpreter/Interpreter.h:
        * jit/JITStubs.cpp: Copied from interpreter/Interpreter.cpp.
        (JSC::Interpreter::cti_op_resolve_base):
        * jit/JITStubs.h: Copied from interpreter/Interpreter.h.
        * runtime/Operations.cpp:
        (JSC::jsAddSlowCase):
        (JSC::jsTypeStringForValue):
        (JSC::jsIsObjectType):
        (JSC::jsIsFunctionType):
        * runtime/Operations.h:
        (JSC::jsLess):
        (JSC::jsLessEq):
        (JSC::jsAdd):
        (JSC::cachePrototypeChain):
        (JSC::countPrototypeChainEntriesAndCheckForProxies):
        (JSC::resolveBase):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41100 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years agoWebCore:
mitz@apple.com [Fri, 20 Feb 2009 01:41:55 +0000 (01:41 +0000)]
WebCore:

        Reviewed by Sam Weinig.

        - WebCore part of fixing https://bugs.webkit.org/show_bug.cgi?id=24027
          Do not send loader callbacks during CSS styling

        Undo the iChat-specific quirk added in
        <http://trac.webkit.org/changeset/41071>. Instead, always suspend memory
        cache client callbacks during attach() and recalcStyle().

        * WebCore.base.exp: Removed
        Settings::setNeedsIChatMemoryCacheCallsQuirk().
        * dom/ContainerNode.cpp:
        (WebCore::ContainerNode::suspendPostAttachCallbacks): Disable memory
        cache client callbacks and remember to enable them afterwards if needed.
        (WebCore::ContainerNode::resumePostAttachCallbacks): Re-enable memory
        cache client callbacks if they were disabled in
        suspendPostAttachCallbacks().
        (WebCore::ContainerNode::attach): Use suspendPostAttachCallbacks() and
        resumePostAttachCallbacks().
        * dom/ContainerNode.h: Made suspendPostAttachCallbacks()
        and resumePostAttachCallbacks() non-static.
        * dom/Document.cpp:
        (WebCore::Document::dispatchImageLoadEventsNow): Reverted iChat-specific
        workaround.
        * page/Settings.cpp: Removed m_needsIChatMemoryCacheCallsQuirk and
        related code.
        (WebCore::Settings::Settings):
        * page/Settings.h:

WebKit/mac:

        Reviewed by Sam Weinig.

        - WebKit part of fixing https://bugs.webkit.org/show_bug.cgi?id=24027
          Do not send loader callbacks during CSS styling

        * WebView/WebView.mm:
        (-[WebView _preferencesChangedNotification:]): Reverted the
        iChat-specific quirk added in <http://trac.webkit.org/changeset/41071>.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41096 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-19 Holger Hans Peter Freyther <zecke@selfish.org>
zecke@webkit.org [Fri, 20 Feb 2009 01:38:24 +0000 (01:38 +0000)]
2009-02-19  Holger Hans Peter Freyther  <zecke@selfish.org>

        Unreviewed build fix.

        Build fix after r41092. Make the memoryUsage method
        public. It will be shadowed by cf/mac and for curl/soup/qt
        the implementation from ResourceResponseBase will be used.

        * platform/network/ResourceResponseBase.h:
        (WebCore::ResourceResponseBase::memoryUsage):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41095 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-19 Beth Dakin <bdakin@apple.com>
bdakin@apple.com [Fri, 20 Feb 2009 01:23:58 +0000 (01:23 +0000)]
2009-02-19  Beth Dakin  <bdakin@apple.com>

        Reviewed by Dave Hyatt.

        Fix for <rdar://problem/6077775> Should be able to specify
        inactive ::selection color

        This patch makes the ::selction pseudo-element work with
        the :window-inactive pseudo type. This was, a user can specify a
        different ::selection style when a window is inactive.

        * css/CSSStyleSelector.cpp:
        (WebCore::CSSStyleSelector::SelectorChecker::checkSelector):
        (WebCore::CSSStyleSelector::SelectorChecker::checkOneSelector):
        * rendering/RenderObject.cpp:
        (WebCore::RenderObject::selectionBackgroundColor):
        (WebCore::RenderObject::selectionForegroundColor):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41094 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years agoWebCore:
weinig@apple.com [Fri, 20 Feb 2009 00:38:17 +0000 (00:38 +0000)]
WebCore:

2009-02-19  Sam Weinig  <sam@webkit.org>

        Reviewed by Beth Dakin.

        Patch for https://bugs.webkit.org/show_bug.cgi?id=24044
        Update querySelector/querySelectorAll to match the latest spec

        Update querySelector and querySelectorAll to match the latest version
        of the Selectors API spec.  We now stringify null and undefined to "null"
        and "undefined" respectively instead of to "".

        Test: fast/dom/SelectorAPI/undefined-null-stringify.html

        * dom/Document.idl:
        * dom/DocumentFragment.idl:
        * dom/Element.idl:

LayoutTests:

2009-02-19  Sam Weinig  <sam@webkit.org>

        Reviewed by Beth Dakin.

        Tests for https://bugs.webkit.org/show_bug.cgi?id=24044
        Update querySelector/querySelectorAll to match the latest spec

        * fast/dom/SelectorAPI/dumpNodeList-almost-strict-expected.txt:
        * fast/dom/SelectorAPI/dumpNodeList-expected.txt:
        * fast/dom/SelectorAPI/resig-SelectorsAPI-test-expected.txt:
        * fast/dom/SelectorAPI/resig-SelectorsAPI-test.xhtml: Updated.  We now fail 17 tests
        due to not throwing an exception when no arguments are passed to querySelector
        or querySelectorAll.
        * fast/dom/SelectorAPI/resources/undefined-null-stringify.js: Added.
        * fast/dom/SelectorAPI/undefined-null-stringify-expected.txt: Added.
        * fast/dom/SelectorAPI/undefined-null-stringify.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41093 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-19 Dimitri Glazkov <dglazkov@chromium.org>
dglazkov@chromium.org [Fri, 20 Feb 2009 00:31:48 +0000 (00:31 +0000)]
2009-02-19  Dimitri Glazkov  <dglazkov@chromium.org>

        Reviewed by Geoffrey Garen.

        https://bugs.webkit.org/show_bug.cgi?id=23732
        Rework CachedResource overhead accounting to allow platforms to diverge
        in numbers.

        * loader/CachedResource.cpp:
        (WebCore::CachedResource::overheadSize): Changed to ask ResourceResponse
        for its size and to use actual URL size.
        * platform/network/ResourceResponseBase.h:
        (WebCore::ResourceResponseBase::size): Added default size method.
        * platform/network/cf/ResourceResponse.h:
        (WebCore::ResourceResponse::size): Added Win/CF size method
        * platform/network/mac/ResourceResponse.h:
        (WebCore::ResourceResponse::size): Added Mac size method.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41092 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-19 Anders Carlsson <andersca@apple.com>
andersca@apple.com [Thu, 19 Feb 2009 23:53:14 +0000 (23:53 +0000)]
2009-02-19  Anders Carlsson  <andersca@apple.com>

        Reviewed by Kevin Decker.

        - Fix for <rdar://problem/6604968>

        On Tiger, create an autorelease pool before creating the NSGraphicsContext and drain it immediately
        after the call to -[NSView displayRectIgnoringOpacity:inContext:].

        * platform/graphics/mac/MediaPlayerPrivateQTKit.mm:
        (WebCore::MediaPlayerPrivate::paint):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41091 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years agoWebCore:
hyatt@apple.com [Thu, 19 Feb 2009 22:52:37 +0000 (22:52 +0000)]
WebCore:

2009-02-19  David Hyatt  <hyatt@apple.com>

        Fix a bug where reflections didn't work properly if opacity was < 1.  Make sure that replaced elements
        also consider reflections to be part of their visual overflow.  This had already been done for blocks
        and lines, but it wasn't being done yet for replaced elements.

        Also make sure that when the object being reflected has opacity < 1 that we don't end up popping the outer
        transparency layer early.  Since the reflected object paints twice, we don't want to end the transparency
        layer it pushed until we're done painting the real object (rather than the reflection).

        Reviewed by Dan Bernstein

        Added fast/reflections/reflection-masks-opacity.html

        * rendering/RenderLayer.cpp:
        (WebCore::RenderLayer::paintLayer):
        * rendering/RenderPartObject.cpp:
        (WebCore::RenderPartObject::layout):
        * rendering/RenderReplaced.cpp:
        (WebCore::RenderReplaced::layout):
        (WebCore::RenderReplaced::adjustOverflowForBoxShadowAndReflect):
        * rendering/RenderReplaced.h:

LayoutTests:

2009-02-19  David Hyatt  <hyatt@apple.com>

        Fix a bug where reflections didn't work properly if opacity was < 1.  Make sure that replaced elements
        also consider reflections to be part of their visual overflow.  This had already been done for blocks
        and lines, but it wasn't being done yet for replaced elements.

        Also make sure that when the object being reflected has opacity < 1 that we don't end up popping the outer
        transparency layer early.  Since the reflected object paints twice, we don't want to end the transparency
        layer it pushed until we're done painting the real object (rather than the reflection).

        Reviewed by Dan Bernstein

        * fast/reflections/reflection-masks-opacity.html: Added.
        * platform/mac/fast/reflections/reflection-direction-expected.txt:
        * platform/mac/fast/reflections/reflection-masks-expected.txt:
        * platform/mac/fast/reflections/reflection-masks-opacity-expected.checksum: Added.
        * platform/mac/fast/reflections/reflection-masks-opacity-expected.png: Added.
        * platform/mac/fast/reflections/reflection-masks-opacity-expected.txt: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41090 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-19 Gavin Barraclough <barraclough@apple.com>
barraclough@apple.com [Thu, 19 Feb 2009 22:51:40 +0000 (22:51 +0000)]
2009-02-19  Gavin Barraclough  <barraclough@apple.com>

        Reviewed by Oliver Hunt.

        Fix for x86-64. Where the JavaScriptCore text segment lies outside
        a 2gb range of the heap containing JIT generated code, callbacks
        from JIT code to the stub functions in Interpreter will be incorrectly
        linked.

        No performance impact on Sunspider, 1% regression on v8-tests,
        due to a 3% regression on richards.

        * assembler/AbstractMacroAssembler.h:
        (JSC::AbstractMacroAssembler::Call::Call):
        (JSC::AbstractMacroAssembler::Jump::link):
        (JSC::AbstractMacroAssembler::Jump::linkTo):
        (JSC::AbstractMacroAssembler::CodeLocationJump::relink):
        (JSC::AbstractMacroAssembler::CodeLocationCall::relink):
        (JSC::AbstractMacroAssembler::ProcessorReturnAddress::relinkCallerToFunction):
        (JSC::AbstractMacroAssembler::PatchBuffer::link):
        (JSC::AbstractMacroAssembler::PatchBuffer::linkTailRecursive):
        (JSC::AbstractMacroAssembler::differenceBetween):
        * assembler/MacroAssembler.h:
        (JSC::MacroAssembler::tailRecursiveCall):
        (JSC::MacroAssembler::makeTailRecursiveCall):
        * assembler/MacroAssemblerX86.h:
        (JSC::MacroAssemblerX86::call):
        * assembler/MacroAssemblerX86Common.h:
        * assembler/MacroAssemblerX86_64.h:
        (JSC::MacroAssemblerX86_64::call):
        (JSC::MacroAssemblerX86_64::moveWithPatch):
        (JSC::MacroAssemblerX86_64::branchPtrWithPatch):
        (JSC::MacroAssemblerX86_64::storePtrWithPatch):
        * assembler/X86Assembler.h:
        (JSC::X86Assembler::jmp_r):
        (JSC::X86Assembler::linkJump):
        (JSC::X86Assembler::patchJump):
        (JSC::X86Assembler::patchCall):
        (JSC::X86Assembler::linkCall):
        (JSC::X86Assembler::patchAddress):
        * interpreter/Interpreter.cpp:
        (JSC::Interpreter::tryCTICachePutByID):
        * jit/JIT.cpp:
        (JSC::JIT::privateCompile):
        (JSC::JIT::privateCompileCTIMachineTrampolines):
        * jit/JIT.h:
        * jit/JITArithmetic.cpp:
        (JSC::JIT::putDoubleResultToJSNumberCellOrJSImmediate):
        (JSC::JIT::compileBinaryArithOp):
        * jit/JITPropertyAccess.cpp:
        (JSC::JIT::privateCompilePutByIdTransition):
        (JSC::JIT::privateCompileGetByIdSelf):
        (JSC::JIT::privateCompilePutByIdReplace):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41089 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-19 David Levin <levin@chromium.org>
levin@chromium.org [Thu, 19 Feb 2009 22:35:40 +0000 (22:35 +0000)]
2009-02-19  David Levin  <levin@chromium.org>

        Reviewed by Alexey Proskuryakov.

        Bug 23980: WorkerRunLoop needs a way to run in a given mode similar to CFRunLoopInMode.
        <https://bugs.webkit.org/show_bug.cgi?id=23980>

        WorkerRunLoop has the ability to run in a mode which filters the tasks to be run.
        * When WorkerRunLoop::runInMode is called, only task for that mode will run.
        * When WorkerRunLoop::run is called (or the default mode is used), then all tasks
          will run regardless of their posted mode.

        Here's a demonstration of the api:

            RefPtr<NameResolution> nameResolution = NameResolution::create(workerRunLoop);

            //  Internally nameResolution will do workerRunLoop.postTaskForMode(task, "MyCoolMode")
            //  for any tasks that need to be run during this loop.
            nameResolution->setTaskMode("MyCoolMode");

            nameResolution->start();
            while (!nameResolution->done()) {
                // Only tasks which are posted for "MyCoolMode" will run.
                workerRunLoop.runInMode(context, "MyCoolMode");
            }

        No observable change in behavior, so no test.

        * dom/WorkerRunLoop.cpp:
        (WebCore::ModePredicate::ModePredicate):
        (WebCore::ModePredicate::operator()):
        (WebCore::WorkerRunLoop::WorkerRunLoop):
        (WebCore::WorkerRunLoop::~WorkerRunLoop):
        (WebCore::WorkerRunLoop::setSharedTimer):
        (WebCore::WorkerRunLoop::resetSharedTimer):
        (WebCore::WorkerRunLoop::run):
        (WebCore::WorkerRunLoop::runInMode):
        (WebCore::WorkerRunLoop::postTask):
        (WebCore::WorkerRunLoop::postTaskForMode):
        * dom/WorkerRunLoop.h:
        (WebCore::WorkerRunLoop::Task::create):
        (WebCore::WorkerRunLoop::Task::mode):
        (WebCore::WorkerRunLoop::Task::performTask):
        (WebCore::WorkerRunLoop::Task::Task):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41088 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-19 Jay Campan <jcampan@google.com>
dimich@chromium.org [Thu, 19 Feb 2009 22:13:28 +0000 (22:13 +0000)]
2009-02-19  Jay Campan  <jcampan@google.com>

        Reviewed by Alexey Proskuryakov.

        https://bugs.webkit.org/show_bug.cgi?id=23959
        LayoutTests/fast/dom/navigator-detached-no-crash.html may fail with other JS engines.

        * fast/dom/navigator-detached-no-crash-expected.txt:
        * fast/dom/navigator-detached-no-crash.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41087 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years agoAdd GLIB_LIBS to unit test library flags.
christian@webkit.org [Thu, 19 Feb 2009 21:41:25 +0000 (21:41 +0000)]
Add GLIB_LIBS to unit test library flags.

Rubber-stamped by Holger Freyther.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41086 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-19 Dimitri Glazkov <dglazkov@chromium.org>
dglazkov@chromium.org [Thu, 19 Feb 2009 21:36:06 +0000 (21:36 +0000)]
2009-02-19  Dimitri Glazkov  <dglazkov@chromium.org>

        Reviewed by Eric Seidel.

        https://bugs.webkit.org/show_bug.cgi?id=24034
        Fix up Selection->VisibleSelection change.

        * page/chromium/EventHandlerChromium.cpp:
        (WebCore::EventHandler::passMousePressEventToSubframe): Renamed Selection
        to VisibleSelection.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41085 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-19 Dimitri Glazkov <dglazkov@chromium.org>
dglazkov@chromium.org [Thu, 19 Feb 2009 21:33:03 +0000 (21:33 +0000)]
2009-02-19  Dimitri Glazkov  <dglazkov@chromium.org>

        Reviewed by Eric Seidel.

        https://bugs.webkit.org/show_bug.cgi?id=24041
        Correct Skia type conversion issues, fix Chromium Linux build.

        * platform/graphics/chromium/FontLinux.cpp:
        (WebCore::Font::drawGlyphs): changed parameters to RGBA32, not WebCore::Color.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41084 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-18 Geoffrey Garen <ggaren@apple.com>
ggaren@apple.com [Thu, 19 Feb 2009 20:35:55 +0000 (20:35 +0000)]
2009-02-18  Geoffrey Garen  <ggaren@apple.com>

        Reviewed by Oliver Hunt.

        Simplified .call and .apply in preparation for optimizing them. Also,
        a little cleanup.

        * runtime/FunctionPrototype.cpp:
        (JSC::functionProtoFuncApply):
        (JSC::functionProtoFuncCall): No need to do any specific conversion on
        'this' -- op_convert_this will do it if necessary.

        * runtime/JSImmediate.cpp:
        (JSC::JSImmediate::toThisObject): Slightly relaxed the rules on
        toThisObject to allow for 'undefined', which can be passed through
        .call and .apply.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41083 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-19 Jungshik Shin <jshin@chromium.org>
dglazkov@chromium.org [Thu, 19 Feb 2009 20:12:58 +0000 (20:12 +0000)]
2009-02-19  Jungshik Shin <jshin@chromium.org>

        Reviewed by Eric Seidel.

        https://bugs.webkit.org/show_bug.cgi?id=20531
        Chromium-part follow-up to the patch landed in r40636
        for bug 20531.
        Remove a static member function alternateFamilyName
        from Chromium's Win/Linux ports of FontCache. In r40636,
        alternateFamilyName was customized using #ifdef PLATFORM(WIN_OS).
        So, there's no more need for the static member function per
        platform.

        * platform/graphics/chromium/FontCacheChromiumWin.cpp:
        * platform/graphics/chromium/FontCacheLinux.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41082 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-19 David Levin <levin@chromium.org>
levin@chromium.org [Thu, 19 Feb 2009 19:40:07 +0000 (19:40 +0000)]
2009-02-19  David Levin  <levin@chromium.org>

        Reviewed by Alexey Proskuryakov.

        Bug 23976: MessageQueue needs a way to wait for a message that satisfies an arbitrary criteria.
        <https://bugs.webkit.org/show_bug.cgi?id=23976>

        * wtf/Deque.h:
        (WTF::Deque<T>::findIf):
        * wtf/MessageQueue.h:
        (WTF::MessageQueue<T>::waitForMessageFiltered):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41081 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-19 Simon Fraser <simon.fraser@apple.com>
simon.fraser@apple.com [Thu, 19 Feb 2009 19:20:33 +0000 (19:20 +0000)]
2009-02-19  Simon Fraser  <simon.fraser@apple.com>

        Reviewed by Adele Peterson.

        Call documentWillBecomeInactive() from Document::detach to ensure that
        media elements are shut down, because they may be kept alive by references
        from JS past document teardown.

        documentWillBecomeInactive() calls  renderView()->willMoveOffscreen(), so no
        need to do that in detach() as well.

        We no longer need to call documentWillBecomeInactive() from ~Page() (which
        was added for <https://bugs.webkit.org/show_bug.cgi?id=21116>), since this
        supersedes that change.

        * dom/Document.cpp:
        (WebCore::Document::detach):
        * page/Page.cpp:
        (WebCore::Page::~Page):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41080 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-19 Dimitri Glazkov <dglazkov@chromium.org>
dglazkov@chromium.org [Thu, 19 Feb 2009 19:19:46 +0000 (19:19 +0000)]
2009-02-19  Dimitri Glazkov  <dglazkov@chromium.org>

        Reviewed by Dave Hyatt.

        https://bugs.webkit.org/show_bug.cgi?id=24012
        Clarify objective of the test and clean up.

        * fast/forms/inline-ignored-on-legend-expected.txt:
        * fast/forms/inline-ignored-on-legend.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41079 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-19 Dimitri Glazkov <dglazkov@chromium.org>
dglazkov@chromium.org [Thu, 19 Feb 2009 19:14:58 +0000 (19:14 +0000)]
2009-02-19  Dimitri Glazkov  <dglazkov@chromium.org>

        Reviewed by Eric Seidel.

        https://bugs.webkit.org/show_bug.cgi?id=24028
        Fix up Skia path changes.

        * platform/graphics/skia/GraphicsContextSkia.cpp:
        (WebCore::GraphicsContext::strokePath): Removed illegal indirection.
        * platform/graphics/skia/PathSkia.cpp:
        (WebCore::boundingBoxForCurrentStroke): Changed call name.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41078 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-19 Gustavo Noronha Silva <gns@gnome.org>
kov@webkit.org [Thu, 19 Feb 2009 15:37:14 +0000 (15:37 +0000)]
2009-02-19  Gustavo Noronha Silva  <gns@gnome.org>

        Reviewed by Alexey Proskuryakov.

        Ignore ports on local URLs. This fixes a regression in
        fast/loader/file-URL-with-port-number.html

        * platform/network/soup/ResourceHandleSoup.cpp:
        (WebCore::ResourceHandle::startGio):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41077 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-19 Gustavo Noronha Silva <gns@gnome.org>
kov@webkit.org [Thu, 19 Feb 2009 15:18:33 +0000 (15:18 +0000)]
2009-02-19  Gustavo Noronha Silva  <gns@gnome.org>

        Reviewed by Alexey Proskuryakov.

        Make setPort remove port if 0 is given to it, as promised by the
        comment in KURL's header.

        * platform/KURL.cpp:
        (WebCore::KURL::setPort):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41076 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years agoWebCore:
mitz@apple.com [Thu, 19 Feb 2009 02:04:23 +0000 (02:04 +0000)]
WebCore:

        Reviewed by Brady Eidson.

        - WebCore part of fixing <rdar://problem/6507512> Crash in iChat at CSSStyleSelector::adjustRenderStyle

        The crash results from re-entry into
        CSSMutableStyleDeclaration::setCssText, which in turn is caused by
        the first style change causing a cached image to load from the memory
        cache, causing load delegate dispatch, and iChat's delegate method
        calling back into WebKit.

        The workaround is to use defer delegate callbacks for memory cache. In
        this case, deferring callbacks during image load event dispatch was
        found to be sufficient.

        The crash is a regression. See also the discussion in
        <https://bugs.webkit.org/show_bug.cgi?id=22521>.

        * WebCore.base.exp: Added
        Settings::setNeedsIChatMemoryCacheCallsQuirk().
        * dom/Document.cpp:
        (WebCore::Document::dispatchImageLoadEventsNow): If the quirk is
        enabled, defer memory cache callbacks during image load event dispatch.
        * page/Settings.cpp:
        (WebCore::Settings::Settings): Initialize
        m_needsIChatMemoryCacheCallsQuirk.
        (WebCore::Settings::setNeedsIChatMemoryCacheCallsQuirk): Added this
        setter.
        * page/Settings.h:
        (WebCore::Settings::needsIChatMemoryCacheCallsQuirk): Added this getter.

WebKit/mac:

        Reviewed by Brady Eidson.

        - WebKit part of fixing <rdar://problem/6507512> Crash in iChat at CSSStyleSelector::adjustRenderStyle

        * WebView/WebView.mm:
        (-[WebView _preferencesChangedNotification:]): Activate the WebCore
        workaround for this crash in iChat.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41071 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-18 Anders Carlsson <andersca@apple.com>
andersca@apple.com [Thu, 19 Feb 2009 01:46:58 +0000 (01:46 +0000)]
2009-02-18  Anders Carlsson  <andersca@apple.com>

        Reviewed by Sam Weinig.

        Fix for <rdar://problem/6542390>

        There's no need to call setDefersLoading here - we already defer anything a plug-in can do that
        would cause a load to begin.

        * Plugins/WebNetscapePluginView.mm:
        (-[WebNetscapePluginView sendEvent:isDrawRect:]):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41070 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-18 Dimitri Glazkov <dglazkov@chromium.org>
dglazkov@chromium.org [Thu, 19 Feb 2009 00:13:03 +0000 (00:13 +0000)]
2009-02-18  Dimitri Glazkov  <dglazkov@chromium.org>

        Reviewed by Eric Seidel.

        https://bugs.webkit.org/show_bug.cgi?id=24012
        Convert inline-ignored-on-legend to text-only test.

        * fast/forms/inline-ignored-on-legend-expected.txt: Added.
        * fast/forms/inline-ignored-on-legend.html: Changed to text-only.
        * platform/mac/fast/forms/inline-ignored-on-legend-expected.checksum: Removed.
        * platform/mac/fast/forms/inline-ignored-on-legend-expected.png: Removed.
        * platform/mac/fast/forms/inline-ignored-on-legend-expected.txt: Removed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41069 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-18 David Levin <levin@chromium.org>
levin@chromium.org [Wed, 18 Feb 2009 23:26:35 +0000 (23:26 +0000)]
2009-02-18  David Levin  <levin@chromium.org>

        Reviewed by Alexey Proskuryakov.

        Bug 23974: Deque::Remove would be a useful method.
        <https://bugs.webkit.org/show_bug.cgi?id=23974>

        Add Deque::remove and DequeIteratorBase<T>::operator=.

        Why was operator= added? Every concrete iterator (DequeIterator..DequeConstReverseIterator)
        was calling DequeIteratorBase::assign(), which called Base::operator=(). Base::operator=()
        was not implemented. This went unnoticed because the iterator copy code has been unused.

        * wtf/Deque.h:
        (WTF::Deque<T>::remove):
        (WTF::DequeIteratorBase<T>::removeFromIteratorsList):
        (WTF::DequeIteratorBase<T>::operator=):
        (WTF::DequeIteratorBase<T>::~DequeIteratorBase):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41068 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years agoAdd SPI to get WebKit's custom pointing-hand cursor
aroben@apple.com [Wed, 18 Feb 2009 22:59:34 +0000 (22:59 +0000)]
Add SPI to get WebKit's custom pointing-hand cursor

WebCore:

        Export WebCore::handCursor and Cursor.h

        Reviewed by John Sullivan.

        * WebCore.base.exp: Added WebCore::handCursor.
        * WebCore.xcodeproj/project.pbxproj: Made Cursor.h private.

WebKit/mac:

        Add SPI to get WebKit's custom pointing-hand cursor

        Reviewed by John Sullivan.

        * WebView/WebView.mm:
        (+[WebView _pointingHandCursor]): Added. Returns the custom
        pointing-hand cursor that WebKit uses.
        * WebView/WebViewPrivate.h: Added +_pointingHandCursor.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41067 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-18 Ojan Vafai <ojan@chromium.org>
darin@chromium.org [Wed, 18 Feb 2009 22:12:02 +0000 (22:12 +0000)]
2009-02-18  Ojan Vafai  <ojan@chromium.org>

        Reviewed by Alexey Proskuryakov.

        https://bugs.webkit.org/show_bug.cgi?id=23992
        REGRESSION: crash on windows loading http://www.stickam.com/liveStreams.do

        Unable to reduce to a layout test.

        * page/Frame.cpp:
        (WebCore::Frame::contentRenderer):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41066 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-18 Evan Stade <estade@chromium.org>
darin@chromium.org [Wed, 18 Feb 2009 22:07:10 +0000 (22:07 +0000)]
2009-02-18  Evan Stade  <estade@chromium.org>

        Reviewed by Eric Seidel.

        https://bugs.webkit.org/show_bug.cgi?id=23861
        Stroke font outlines on chromium linux

        TEST=LayoutTests/svg/custom/pointer-events-text.svg

        * platform/graphics/chromium/FontLinux.cpp:
        (WebCore::Font::drawGlyphs):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41065 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-18 Evan Stade <estade@chromium.org>
darin@chromium.org [Wed, 18 Feb 2009 22:05:25 +0000 (22:05 +0000)]
2009-02-18  Evan Stade  <estade@chromium.org>

        Reviewed by Eric Seidel.

        https://bugs.webkit.org/show_bug.cgi?id=23860
        Resync some graphics/skia files with their chromium counterparts

        This comes from chromium patches <http://codereview.chromium.org/17633>
        and <http://codereview.chromium.org/17454>

        * platform/graphics/skia/GraphicsContextSkia.cpp:
        (WebCore::GraphicsContext::clipPath):
        (WebCore::GraphicsContext::fillPath):
        * platform/graphics/skia/PlatformContextSkia.cpp:
        (PlatformContextSkia::addPath):
        (PlatformContextSkia::currentPath):
        * platform/graphics/skia/PlatformContextSkia.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41064 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-18 Gustavo Noronha Silva <gns@gnome.org>
kov@webkit.org [Wed, 18 Feb 2009 21:56:51 +0000 (21:56 +0000)]
2009-02-18  Gustavo Noronha Silva  <gns@gnome.org>

        Unreviewed.

        Build fix after r41060.

        * GNUmakefile.am:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41063 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago Rubber-stamped by Alexey Proskuryakov.
eric@webkit.org [Wed, 18 Feb 2009 20:19:21 +0000 (20:19 +0000)]
    Rubber-stamped by Alexey Proskuryakov.

        Add a few ignores to make-js-test-wrappers.

        * Scripts/make-js-test-wrappers:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41062 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-18 Dimitri Glazkov <dglazkov@chromium.org>
dglazkov@chromium.org [Wed, 18 Feb 2009 19:20:08 +0000 (19:20 +0000)]
2009-02-18  Dimitri Glazkov  <dglazkov@chromium.org>

        Reviewed by Eric Seidel.

        https://bugs.webkit.org/show_bug.cgi?id=24005
        Add an include to fix Chromium build.

        * page/animation/AnimationController.cpp: Add UnusedParam.h include.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41061 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-18 Gustavo Noronha Silva <gns@gnome.org>
kov@webkit.org [Wed, 18 Feb 2009 19:18:28 +0000 (19:18 +0000)]
2009-02-18  Gustavo Noronha Silva  <gns@gnome.org>

        Reviewed by Holger Freyther.

        Fix symbols.filter location, and add other missing files to the
        autotools build, so that make dist works.

        * GNUmakefile.am:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41060 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-18 Zan Dobersek <zandobersek@gmail.com>
kov@webkit.org [Wed, 18 Feb 2009 16:45:35 +0000 (16:45 +0000)]
2009-02-18  Zan Dobersek  <zandobersek@gmail.com>

        Rubber-stamped by Holger Hans Peter Freyther.

        Allow POST method for local requests.

        * platform/network/soup/ResourceHandleSoup.cpp:
        (WebCore::ResourceHandle::startGio):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41059 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-18 Gustavo Noronha Silva <gns@gnome.org>
kov@webkit.org [Wed, 18 Feb 2009 16:36:28 +0000 (16:36 +0000)]
2009-02-18  Gustavo Noronha Silva  <gns@gnome.org>

        Reviewed by Holger Hans Peter Freyther.

        Use KURL in startGio instead of passing a string with the URL, so
        that we can handle removing refs and queries more elegantly. This
        is fixing more regressions that came from the curl->soup switch.

        Original work by Zan Dobersek.

        * platform/network/ResourceHandle.h:
        * platform/network/soup/ResourceHandleSoup.cpp:
        (WebCore::ResourceHandle::start):
        (WebCore::ResourceHandle::startGio):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41058 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years agoImplement ResourceHandle::loadResourceSynchronously in ResourceHandleSoup.cpp
zecke@webkit.org [Wed, 18 Feb 2009 15:02:24 +0000 (15:02 +0000)]
Implement ResourceHandle::loadResourceSynchronously in ResourceHandleSoup.cpp

The implementation is needed to have synchronous loading, e.g. for
JavaScript interaction. This is fixing various regressions that
came from the curl->soup switch.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41057 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-17 Holger Hans Peter Freyther <zecke@selfish.org>
zecke@webkit.org [Wed, 18 Feb 2009 14:58:42 +0000 (14:58 +0000)]
2009-02-17  Holger Hans Peter Freyther  <zecke@selfish.org>

        Reviewed by Alexey Proskuryakov.

        Prepend file:// to the test result filename to make
        GtkLauncher display the result page.

        * Scripts/run-webkit-tests:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41056 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-18 Xan Lopez <xan@gnome.org>
kov@webkit.org [Wed, 18 Feb 2009 12:17:11 +0000 (12:17 +0000)]
2009-02-18  Xan Lopez  <xan@gnome.org>

        Reviewed by Mark Rowe.

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

        Based on a patch by Bo Yang  <techrazy.yang@gmail.com>

        Make the cursor cache global, that's all we really need and
        otherwise we can miss cursor transitions in some situations (see
        the bug for one testcase). Also remove some now useless code.

        * platform/Widget.h:
        * platform/gtk/WidgetGtk.cpp:
        (WebCore::Widget::Widget):
        (WebCore::Widget::~Widget):
        (WebCore::Widget::setCursor):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41055 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago Windows build fix
aroben@apple.com [Wed, 18 Feb 2009 05:48:20 +0000 (05:48 +0000)]
    Windows build fix

        * loader/FrameLoader.cpp:
        (WebCore::toPlugInElement): Don't rely on #if being processed before
        ASSERT.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41054 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-17 Geoffrey Garen <ggaren@apple.com>
ggaren@apple.com [Wed, 18 Feb 2009 04:17:47 +0000 (04:17 +0000)]
2009-02-17  Geoffrey Garen  <ggaren@apple.com>

        Reviewed by Sam Weinig.

        Fixed failure in js1_5/Regress/regress-168347.js, as seen on the Oliver
        bot.

        Technically, both behaviors are OK, but we might as well keep this test
        passing.

        * runtime/FunctionPrototype.cpp:
        (JSC::insertSemicolonIfNeeded): No need to add a trailing semicolon
        after a trailing '}', since '}' ends a block, indicating the end of a
        statement.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41053 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-17 Geoffrey Garen <ggaren@apple.com>
ggaren@apple.com [Wed, 18 Feb 2009 04:03:23 +0000 (04:03 +0000)]
2009-02-17  Geoffrey Garen  <ggaren@apple.com>

        Build fix.

        * runtime/FunctionPrototype.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41052 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years agoAdd assertion to guard against oversized pc relative calls.
oliver@apple.com [Wed, 18 Feb 2009 03:36:03 +0000 (03:36 +0000)]
Add assertion to guard against oversized pc relative calls.

Reviewed by Geoff Garen

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41051 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-17 David Levin <levin@chromium.org>
levin@chromium.org [Wed, 18 Feb 2009 00:50:04 +0000 (00:50 +0000)]
2009-02-17  David Levin  <levin@chromium.org>

        Reviewed by Alexey Proskuryakov.

        Bug 23977: Unnecessary timer related headers in files.
        <https://bugs.webkit.org/show_bug.cgi?id=23977>

        No observable change in behavior, so no test.

        * dom/Document.cpp:
        * dom/WorkerRunLoop.cpp:
        * dom/WorkerRunLoop.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41047 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years agoWebCore:
dimich@chromium.org [Wed, 18 Feb 2009 00:21:21 +0000 (00:21 +0000)]
WebCore:
2009-02-17  Peter Abrahamsen  <rainhead@gmail.com>

        Reviewed by Sam Weinig.

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

        Updated XMLHttpRequest with new header names from the latest Access
        Control draft: http://www.w3.org/TR/access-control/
        - Access-Control-Origin becomes Access-Control-Allow-Origin
        - Access-Control-Credentials becomes Access-Control-Allow-Credentials

        * xml/XMLHttpRequest.cpp:
        (WebCore::XMLHttpRequest::accessControlCheck):

LayoutTests:
2009-02-17  Peter Abrahamsen  <rainhead@gmail.com>

        Reviewed by Sam Weinig.

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

        Updated XMLHttpRequests tests to match the newest Access Control draft.

        * http/tests/security/originHeader/resources/print-origin.cgi:
        * http/tests/xmlhttprequest/resources/access-control-basic-allow-access-control-origin-header.cgi:
        * http/tests/xmlhttprequest/resources/access-control-basic-allow-print-headers.cgi:
        * http/tests/xmlhttprequest/resources/access-control-basic-allow-star.cgi:
        * http/tests/xmlhttprequest/resources/access-control-basic-allow.cgi:
        * http/tests/xmlhttprequest/resources/access-control-basic-get-fail-non-simple.cgi:
        * http/tests/xmlhttprequest/resources/access-control-basic-non-get-allow.cgi:
        * http/tests/xmlhttprequest/resources/access-control-basic-preflight-cache-invalidation.php:
        * http/tests/xmlhttprequest/resources/access-control-basic-preflight-cache-timeout.php:
        * http/tests/xmlhttprequest/resources/access-control-basic-preflight-cache.php:
        * http/tests/xmlhttprequest/resources/access-control-basic-whitelist-response-headers.cgi:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41046 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years agoJavaScriptCore:
ggaren@apple.com [Wed, 18 Feb 2009 00:14:30 +0000 (00:14 +0000)]
JavaScriptCore:

2009-02-17  Geoffrey Garen  <ggaren@apple.com>

        Reviewed by Sam Weinig.

        Fixed <rdar://problem/6595040> REGRESSION: http://www.amnestyusa.org/
        fails to load.

        amnestyusa.org uses the Optimist JavaScript library, which adds event
        listeners by concatenating string-ified functions. This is only sure to
        be syntactically valid if the string-ified functions end in semicolons.

        * parser/Lexer.cpp:
        (JSC::Lexer::isWhiteSpace):
        * parser/Lexer.h:
        (JSC::Lexer::isWhiteSpace):
        (JSC::Lexer::isLineTerminator): Added some helper functions for examining
        whitespace.

        * runtime/FunctionPrototype.cpp:
        (JSC::appendSemicolonIfNeeded):
        (JSC::functionProtoFuncToString): When string-ifying a function, insert
        a semicolon in the last non-whitespace position, if one doesn't already exist.

LayoutTests:

2009-02-17  Geoffrey Garen  <ggaren@apple.com>

        Reviewed by Sam Weinig.

        Test for <rdar://problem/6595040> REGRESSION: http://www.amnestyusa.org/
        fails to load.

        * fast/js/function-toString-semicolon-insertion-expected.txt: Added.
        * fast/js/function-toString-semicolon-insertion.html: Added.
        * fast/js/resources/function-toString-semicolon-insertion.js: Added.
        (compileAndSerialize):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41045 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years agowx build fix. Add missing constructor used for empty values.
kevino@webkit.org [Tue, 17 Feb 2009 22:59:50 +0000 (22:59 +0000)]
wx build fix. Add missing constructor used for empty values.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41044 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-17 Antti Koivisto <antti@apple.com>
antti@apple.com [Tue, 17 Feb 2009 22:16:18 +0000 (22:16 +0000)]
2009-02-17  Antti Koivisto  <antti@apple.com>

        Reviewed by Dave Kilzer.

        <rdar://problem/6592446> dynamically updating page doesn't seem to draw when updated

        Need update after callback.

        * bindings/js/JSCustomPositionCallback.cpp:
        (WebCore::JSCustomPositionCallback::handleEvent):
        * bindings/js/JSCustomPositionErrorCallback.cpp:
        (WebCore::JSCustomPositionErrorCallback::handleEvent):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41043 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-17 Eric Carlson <eric.carlson@apple.com>
eric.carlson@apple.com [Tue, 17 Feb 2009 20:43:06 +0000 (20:43 +0000)]
2009-02-17  Eric Carlson  <eric.carlson@apple.com>

        Reviewed by Antti Koivisto.

        https://bugs.webkit.org/show_bug.cgi?id=23917
        Allow a WebKit plug-in to act as a proxy for the <audio> and <video>
        element.

        * DerivedSources.make: add media element proxy exports to .exp file when feature is defined.

        * WebCore.VideoProxy.exp: New, define the informal protocol exported by a media element proxy.

        * WebCore.xcodeproj/project.pbxproj: Add MediaPlayerProxy.h.

        * html/HTMLMediaElement.cpp:
        (WebCore::HTMLMediaElement::HTMLMediaElement): Initialize m_needWidgetUpdate.
        (WebCore::HTMLMediaElement::attributeChanged): Don't detach+attach when PLUGIN_PROXY_FOR_VIDEO, the
        proxy plug-in handles the poster frame.
        (WebCore::HTMLMediaElement::rendererIsNeeded): New logic for PLUGIN_PROXY_FOR_VIDEO.
        (WebCore::HTMLMediaElement::createRenderer): Create RenderPartObject when PLUGIN_PROXY_FOR_VIDEO.
        (WebCore::HTMLMediaElement::attach): Set m_needWidgetUpdate when PLUGIN_PROXY_FOR_VIDEO
        (WebCore::HTMLMediaElement::load): Don't reallocate MediaPlayer when PLUGIN_PROXY_FOR_VIDEO, we keep the
        same plug-in for the life of the element.
        (WebCore::HTMLMediaElement::mediaPlayerNetworkStateChanged): update m_networkState when media player
        network state changes to EMPTY, otherwise we can get out of sync with engine.
        (WebCore::HTMLMediaElement::defaultEventHandler): pass event to widget when PLUGIN_PROXY_FOR_VIDEO
        (WebCore::HTMLMediaElement::deliverNotification): New, deliver notification from proxy plug-in to
        media player.
        (WebCore::HTMLMediaElement::setMediaPlayerProxy): New, pass proxy object to media player.
        (WebCore::HTMLMediaElement::initialURL): New, return the url from the "src" attr or the appropriate
        <source> element to be used as the initial url for the proxy.
        (WebCore::HTMLMediaElement::finishParsingChildren): New, allocate MediaPlayer and update widget.
        * html/HTMLMediaElement.h: Declare new methods for proxy, add m_needWidgetUpdate.
        (WebCore::HTMLMediaElement::setNeedWidgetUpdate):

        * html/HTMLVideoElement.cpp:
        (WebCore::HTMLVideoElement::attach): Poster image is handled by proxy when PLUGIN_PROXY_FOR_VIDEO.
        (WebCore::HTMLVideoElement::parseMappedAttribute): Ditto.
        * html/HTMLVideoElement.h:

        * loader/FrameLoader.cpp:
        (WebCore::toPlugInElement): Allow cast if element is <video> or <audio>

        * platform/graphics/MediaPlayer.cpp:
        (WebCore::MediaPlayer::MediaPlayer): Remove white space.
        (WebCore::MediaPlayer::setPoster): New, forward call to private player.
        (WebCore::MediaPlayer::deliverNotification): Ditto.
        (WebCore::MediaPlayer::setMediaPlayerProxy): Ditto.
        * platform/graphics/MediaPlayer.h:
        (WebCore::MediaPlayer::mediaPlayerClient):

        * platform/graphics/mac/MediaPlayerProxy.h: New, defines media player proxy interface.

        * rendering/RenderPart.cpp:
        (WebCore::RenderPart::RenderPart): Change constructor to take Element* instead of Node* as a
        non-element node doesn't need a renderer
        * rendering/RenderPart.h: Ditto.

        * rendering/RenderPartObject.cpp:
        (WebCore::RenderPartObject::RenderPartObject): Ditto.
        (WebCore::RenderPartObject::updateWidget): Package params for proxy plug-in when element is
        <video> or <audio>.
        * rendering/RenderPartObject.h:

2009-02-17  Eric Carlson  <eric.carlson@apple.com>

        Reviewed by Antti Koivisto.

        https://bugs.webkit.org/show_bug.cgi?id=23917
        Allow a WebKit plug-in to act as a proxy for the <audio> and <video>
        element.

        * Plugins/WebPluginContainerPrivate.h:
        * Plugins/WebPluginController.mm:
        (mediaProxyClient): New, cast to HTMLMediaElement if it is a video or audio element
        (-[WebPluginController _setMediaPlayerProxy:forElement:]): New, pass proxy to HTMLMediaElement
        (-[WebPluginController _postMediaPlayerNotification:forElement:]): New, deliver event to HTMLMediaElement

        * WebCoreSupport/WebFrameLoaderClient.mm:
        (WebFrameLoaderClient::createPlugin): Don't allow a media player proxy plug-in to be chosen by
        file extension, only want a match for the new MIME type proxy plug-ins should have.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41042 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years agoWebCore:
hyatt@apple.com [Tue, 17 Feb 2009 19:33:16 +0000 (19:33 +0000)]
WebCore:

2009-02-17  David Hyatt  <hyatt@apple.com>

        Reviewed by Eric Seidel

        Fix for https://bugs.webkit.org/show_bug.cgi?id=23985

        Don't allow legends to be anything but display:block.

        Added fast/forms/inline-ignored-on-legend.html

        * GNUmakefile.am:
        * WebCore.pro:
        * WebCore.vcproj/WebCore.vcproj:
        * WebCore.xcodeproj/project.pbxproj:
        * WebCoreSources.bkl:
        * css/html4.css:
        * html/HTMLLegendElement.cpp:
        * html/HTMLLegendElement.h:
        * rendering/RenderLegend.cpp: Removed.
        * rendering/RenderLegend.h: Removed.
        * wml/WMLInsertedLegendElement.cpp:
        * wml/WMLInsertedLegendElement.h:

LayoutTests:

2009-02-17  David Hyatt  <hyatt@apple.com>

        Reviewed by Eric Seidel

        Fix for https://bugs.webkit.org/show_bug.cgi?id=23985

        Don't allow legends to be anything but display:block.

        * fast/forms/inline-ignored-on-legend.html: Added.
        * platform/mac/fast/block/basic/fieldset-stretch-to-legend-expected.txt:
        * platform/mac/fast/borders/fieldsetBorderRadius-expected.txt:
        * platform/mac/fast/forms/006-expected.txt:
        * platform/mac/fast/forms/007-expected.txt:
        * platform/mac/fast/forms/fieldset-align-expected.txt:
        * platform/mac/fast/forms/float-before-fieldset-expected.txt:
        * platform/mac/fast/forms/inline-ignored-on-legend-expected.checksum: Added.
        * platform/mac/fast/forms/inline-ignored-on-legend-expected.png: Added.
        * platform/mac/fast/forms/inline-ignored-on-legend-expected.txt: Added.
        * platform/mac/fast/invalid/residual-style-expected.txt:
        * platform/mac/svg/custom/inline-svg-in-xhtml-expected.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41041 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years agoReviewed by Simon Fraser.
brettw@chromium.org [Tue, 17 Feb 2009 18:14:35 +0000 (18:14 +0000)]
Reviewed by Simon Fraser.

https://bugs.webkit.org/attachment.cgi?id=27666
Fix Chromium build build: forgotten include in RenderObject, sync
RenderTheme to the recent changes in RenderObject.

* rendering/RenderObject.cpp:
* rendering/RenderThemeChromiumMac.mm:
  (WebCore::RenderThemeChromiumMac::updatePressedState):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41040 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-17 Xan Lopez <xan@gnome.org>
kov@webkit.org [Tue, 17 Feb 2009 16:52:17 +0000 (16:52 +0000)]
2009-02-17  Xan Lopez  <xan@gnome.org>

        Rubber-stamped by Alexey Proskuryakov.

        Restoring change landed in r40715, which was accidentally undone
        by r40918.

        * webkit/webkitwebframe.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41039 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-17 Gustavo Noronha Silva <gns@gnome.org>
kov@webkit.org [Tue, 17 Feb 2009 12:43:58 +0000 (12:43 +0000)]
2009-02-17  Gustavo Noronha Silva  <gns@gnome.org>

        Reviewed by Alexey Proskuryakov.

        https://bugs.webkit.org/show_bug.cgi?id=23939
        Release build being built with debugging symbols

        * configure.ac: Revert change done in revision 40790, since we
        already have a AC_PROG_CXX macro call as part of WEBKIT_INIT

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41038 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-17 Gustavo Noronha Silva <gns@gnome.org>
kov@webkit.org [Tue, 17 Feb 2009 12:32:10 +0000 (12:32 +0000)]
2009-02-17  Gustavo Noronha Silva  <gns@gnome.org>

        Reviewed by Darin Adler.

        https://bugs.webkit.org/show_bug.cgi?id=22966
        crash when destroying a webview that opened a page containing <script>
        tags

        Fix m_group being set to 0 instead of to m_singlePageGroup when
        GroupName is set to empty

        * page/Page.cpp:
        (WebCore::Page::setGroupName):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41037 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years agoRoll out r41022 as it breaks qt and gtk builds
oliver@apple.com [Tue, 17 Feb 2009 03:38:42 +0000 (03:38 +0000)]
Roll out r41022 as it breaks qt and gtk builds

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41036 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-16 Beth Dakin <bdakin@apple.com>
bdakin@apple.com [Tue, 17 Feb 2009 03:38:33 +0000 (03:38 +0000)]
2009-02-16  Beth Dakin  <bdakin@apple.com>

        Reviewed by Sam Weinig.

        Fix for <rdar://problem/6386623>

        I made m_numParsedProperties and m_maxParsedProperties unsigned
        instead of int, and then added an early return from addPropery() if
        m_maxParsedProperties exceeds the max value.

        * css/CSSParser.cpp:
        (WebCore::CSSParser::addProperty):
        (WebCore::CSSParser::rollbackLastProperties):
        (WebCore::CSSParser::clearProperties):
        (WebCore::CSSParser::createFontFaceRule):
        (WebCore::CSSParser::deleteFontFaceOnlyValues):
        * css/CSSParser.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41035 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years agoWebCore:
justin.garcia@apple.com [Tue, 17 Feb 2009 03:36:16 +0000 (03:36 +0000)]
WebCore:

2009-02-16  Justin Garcia  <justin.garcia@apple.com>

        Reviewed by Mark Rowe.

        https://bugs.webkit.org/show_bug.cgi?id=16309
        HTML5: The third execCommand argument for insert{un}orderedlist should be ignored

        When we originally implemented this command, MSDN documentation said that IE set the
        id attribute of inserted lists to the third argument to execCommand, but IE doesn't
        do this nor do any other browsers.

        * editing/EditorCommand.cpp:
        (WebCore::executeInsertOrderedList):
        (WebCore::executeInsertUnorderedList):
        * editing/IndentOutdentCommand.cpp:
        (WebCore::IndentOutdentCommand::outdentParagraph):
        * editing/InsertListCommand.cpp:
        (WebCore::InsertListCommand::insertList):
        (WebCore::InsertListCommand::InsertListCommand):
        (WebCore::InsertListCommand::doApply):
        * editing/InsertListCommand.h:
        (WebCore::InsertListCommand::create):

LayoutTests:

2009-02-16  Justin Garcia  <justin.garcia@apple.com>

        Reviewed by Mark Rowe.

        https://bugs.webkit.org/show_bug.cgi?id=16309
        HTML5: The third execCommand argument for insert{un}orderedlist should be ignored

        * editing/execCommand/insert-list-with-id-expected.txt: Added.
        * editing/execCommand/insert-list-with-id.html:
        * platform/mac/editing/execCommand/insert-list-with-id-expected.checksum: Removed.
        * platform/mac/editing/execCommand/insert-list-with-id-expected.png: Removed.
        * platform/mac/editing/execCommand/insert-list-with-id-expected.txt: Removed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41034 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-16 Gustavo Noronha Silva <gns@gnome.org>
kov@webkit.org [Tue, 17 Feb 2009 02:59:29 +0000 (02:59 +0000)]
2009-02-16  Gustavo Noronha Silva  <gns@gnome.org>

        Reviewed by Adam Roben

        Actually use the installation prefix defined in WebKitInstallationPrefix,
        in autotools builds.

        * Scripts/webkitdirs.pm:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41033 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years agoJavaScriptCore:
weinig@apple.com [Tue, 17 Feb 2009 00:40:39 +0000 (00:40 +0000)]
JavaScriptCore:

2009-02-16  Sam Weinig  <sam@webkit.org>

        Reviewed by Geoffrey Garen.

        Fix for <rdar://problem/6468156>
        REGRESSION (r36779): Adding link, images, flash in TinyMCE blocks entire page (21382)

        No performance regression.

        * runtime/Arguments.cpp:
        (JSC::Arguments::fillArgList): Add codepath for when the "length" property has been
        overridden.

LayoutTests:

2009-02-16  Sam Weinig  <sam@webkit.org>

        Reviewed by Geoffrey Garen.

        Add tests for <rdar://problem/6468156>
        REGRESSION (r36779): Adding link, images, flash in TinyMCE blocks entire page (21382)

        * fast/js/function-apply-expected.txt:
        * fast/js/resources/function-apply.js: Add cases covering setting arugments.length and
        Array.length explicitly or implicitly using Array.prototype.unshift.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41028 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years agoWebCore:
mitz@apple.com [Mon, 16 Feb 2009 23:56:23 +0000 (23:56 +0000)]
WebCore:

        Reviewed by Geoffrey Garen.

        - fix <rdar://problem/6561077> REGRESSION (r39634): Unable to select an item in stocks widget with the mouse

        Test: platform/mac/fast/events/pointer-events-dashboard.html

        * css/CSSStyleSelector.cpp:
        (WebCore::CSSStyleSelector::applyProperty): Do not apply the
        pointer-events property in Dashboard backwards compatibility mode,
        in order to work around misuse of that property by the Stocks widget.

LayoutTests:

        Reviewed by Geoffrey Garen.

        - test for <rdar://problem/6561077> REGRESSION (r39634): Unable to select an item in stocks widget with the mouse

        * platform/mac/fast/events/pointer-events-dashboard-expected.checksum: Added.
        * platform/mac/fast/events/pointer-events-dashboard-expected.png: Added.
        * platform/mac/fast/events/pointer-events-dashboard-expected.txt: Added.
        * platform/mac/fast/events/pointer-events-dashboard.html: Copied from LayoutTests/fast/events/pointer-events.html.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41027 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years agoWebCore:
justin.garcia@apple.com [Mon, 16 Feb 2009 23:33:42 +0000 (23:33 +0000)]
WebCore:

2009-02-16  Justin Garcia  <justin.garcia@apple.com>

        Reviewed by Eric Seidel.

        https://bugs.webkit.org/show_bug.cgi?id=15381
        execCommand justify modifies the contentEditable node

        * editing/CompositeEditCommand.cpp:
        (WebCore::CompositeEditCommand::moveParagraphContentsToNewBlockIfNecessary): Avoid modifying not just
        the body element, but any root editable element.

LayoutTests:

2009-02-16  Justin Garcia  <justin.garcia@apple.com>

        Reviewed by Eric Seidel.

        https://bugs.webkit.org/show_bug.cgi?id=15381
        execCommand justify modifies the contentEditable node

        * editing/execCommand/15381-expected.txt: Added.
        * editing/execCommand/15381.html: Added.

        Made this test cross platform and changed it to reflect the fix:
        * editing/execCommand/5062376-expected.txt: Added.
        * editing/execCommand/5062376.html:
        * platform/mac/editing/execCommand/5062376-expected.checksum: Removed.
        * platform/mac/editing/execCommand/5062376-expected.png: Removed.
        * platform/mac/editing/execCommand/5062376-expected.txt: Removed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41026 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years agoBug 23979: AX: alt tag not returned for <input type="image">
cfleizach@apple.com [Mon, 16 Feb 2009 23:19:43 +0000 (23:19 +0000)]
Bug 23979: AX: alt tag not returned for <input type="image">
https://bugs.webkit.org/show_bug.cgi?id=23979

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41025 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years agoBuild fix.
mrowe@apple.com [Mon, 16 Feb 2009 22:23:03 +0000 (22:23 +0000)]
Build fix.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41023 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years agoBug 23787: Allow JIT to generate SSE2 code if using GCC
oliver@apple.com [Mon, 16 Feb 2009 22:15:40 +0000 (22:15 +0000)]
Bug 23787: Allow JIT to generate SSE2 code if using GCC
<https://bugs.webkit.org/show_bug.cgi?id=23787>

Provided by Csaba Osztrogonac
Reviewed by Oliver Hunt

GCC version of the cpuid check.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41022 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years agoReviewed by Kevin Ollivier.
kevino@webkit.org [Mon, 16 Feb 2009 22:06:47 +0000 (22:06 +0000)]
Reviewed by Kevin Ollivier.

The ref data is not properly set unless we use the CreateMatrix function.

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

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41021 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago Reviewed by John Sullivan.
mitz@apple.com [Mon, 16 Feb 2009 21:57:22 +0000 (21:57 +0000)]
    Reviewed by John Sullivan.

        - fix <rdar://problem/6516829> FontPlatformData constructor passes NULL to CTFontCopyGraphicsFont and CTFontGetPlatformFont

        * platform/graphics/FontCache.cpp:
        (WebCore::FontDataCacheKeyTraits::emptyValue): Changed to use the
        FontPlatformData(float, bool, bool) constructor.
        * platform/graphics/mac/FontCacheMac.mm:
        (WebCore::FontCache::createFontPlatformData): Changed to pass a valid
        NSFont to the FontPlatformData constructor, instead of implicitly
        passing 0 and then calling setFont().
        * platform/graphics/mac/FontPlatformData.h: Made the NSFont parameter
        of the constructor mandatory.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41020 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-16 Simon Hausmann <simon.hausmann@nokia.com>
hausmann@webkit.org [Mon, 16 Feb 2009 10:57:46 +0000 (10:57 +0000)]
2009-02-16  Simon Hausmann  <simon.hausmann@nokia.com>

        Reviewed by Tor Arne Vestbø.

        Avoid loading plugins when they're disabled in the settings.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41019 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago Move duplicate caretWidth constants to RenderObject.h
ddkilzer@apple.com [Sun, 15 Feb 2009 21:39:47 +0000 (21:39 +0000)]
    Move duplicate caretWidth constants to RenderObject.h

        Reviewed by Dan Bernstein.

        No test since no change in behavior.

        * rendering/RenderBlock.cpp:
        (WebCore::RenderBlock::localCaretRect): Removed caretWidth.
        * rendering/RenderBox.cpp:
        (WebCore::RenderBox::localCaretRect): Ditto.
        * rendering/RenderObject.h: Added caretWidth definition.
        * rendering/RenderText.cpp:
        (WebCore::RenderText::localCaretRect): Removed caretWidth.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41018 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-14 Dimitri Glazkov <dglazkov@chromium.org>
dglazkov@chromium.org [Sat, 14 Feb 2009 16:39:31 +0000 (16:39 +0000)]
2009-02-14  Dimitri Glazkov  <dglazkov@chromium.org>

        Reviewed by Eric Seidel.

        https://bugs.webkit.org/show_bug.cgi?id=23955
        V8HTMLPlugInCustom returns undefined, which fools interceptors.

        * bindings/v8/custom/V8HTMLPlugInElementCustom.cpp:
        (WebCore::NAMED_PROPERTY_GETTER): Use deferToInterceptor() return value.
        (WebCore::NAMED_PROPERTY_SETTER): Ditto.
        (WebCore::INDEXED_PROPERTY_GETTER): Ditto.
        (WebCore::INDEXED_PROPERTY_SETTER): Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41017 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-13 Dimitri Glazkov <dglazkov@chromium.org>
dglazkov@chromium.org [Sat, 14 Feb 2009 02:17:10 +0000 (02:17 +0000)]
2009-02-13  Dimitri Glazkov  <dglazkov@chromium.org>

        Reviewed by Eric Seidel.

        https://bugs.webkit.org/show_bug.cgi?id=23954
        Add V8 custom bindings for CanvasRenderingContext2D.

        * bindings/v8/custom/V8CanvasRenderingContext2DCustom.cpp: Added.
        (WebCore::toV8):
        (WebCore::toCanvasStyle):
        (WebCore::ACCESSOR_GETTER):
        (WebCore::ACCESSOR_SETTER):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41016 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-13 Adam Treat <adam.treat@torchmobile.com>
treat@webkit.org [Sat, 14 Feb 2009 01:48:09 +0000 (01:48 +0000)]
2009-02-13  Adam Treat  <adam.treat@torchmobile.com>

        Reviewed by George Staikos.

        https://bugs.webkit.org/show_bug.cgi?id=23960
        Crash Fix.

        Don't depend on 'initializeThreading()' to come before a call to 'isMainThread()'
        as QtWebKit only calls 'initializeThreading()' during QWebPage construction.

        A client app may well make a call to QWebSettings::iconForUrl() for instance
        before creating a QWebPage and that call to QWebSettings triggers an
        ASSERT(isMainThread()) deep within WebCore.

        * wtf/ThreadingQt.cpp:
        (WTF::isMainThread):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41013 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-13 Gavin Barraclough <barraclough@apple.com>
barraclough@apple.com [Sat, 14 Feb 2009 01:00:58 +0000 (01:00 +0000)]
2009-02-13  Gavin Barraclough  <barraclough@apple.com>

        Reviewed by Darin Adler.

        Some data in the instruction stream is potentially uninitialized - fix this.

        Change the OperandTypes constructor so that uninitialized memory in the int
        is zeroed, and modify the Instruction constructor taking an Opcode so that
        if !HAVE(COMPUTED_GOTO) (i.e. when Opcode is an enum, and is potentially only
        a byte) it zeros the Instruction first before writing the opcode.

        * bytecode/Instruction.h:
        (JSC::Instruction::Instruction):
        * parser/ResultType.h:
        (JSC::OperandTypes::OperandTypes):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41000 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years agoFix failing layout pixel test so bdash doesn't come to my house in the middle of...
hyatt@apple.com [Sat, 14 Feb 2009 00:46:11 +0000 (00:46 +0000)]
Fix failing layout pixel test so bdash doesn't come to my house in the middle of the night and beat me up.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@40999 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years agoFix ChangeLog.
andersca@apple.com [Sat, 14 Feb 2009 00:41:01 +0000 (00:41 +0000)]
Fix ChangeLog.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@40998 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-13 Anders Carlsson <andersca@apple.com>
andersca@apple.com [Sat, 14 Feb 2009 00:35:45 +0000 (00:35 +0000)]
2009-02-13  Anders Carlsson  <andersca@apple.com>

        Reviewed by Darin Adler.

        <rdar://problem/6583187>

        * page/NavigatorBase.cpp:
        Handle PPC64 and X86_64 as well.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@40997 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-13 Anders Carlsson <andersca@apple.com>
andersca@apple.com [Sat, 14 Feb 2009 00:04:32 +0000 (00:04 +0000)]
2009-02-13  Anders Carlsson  <andersca@apple.com>

        Reviewed by Kevin Decker.

        <rdar://problem/6584834> ESPN radio live stream link hangs Safari

        When a plug-in invokes JavaScript code that will destroy the plug-in, we need to
        defer destruction until we're done executing the script.

        * Plugins/Hosted/NetscapePluginHostProxy.mm:
        (WebKit::PluginDestroyDeferrer::PluginDestroyDeferrer):
        (WebKit::PluginDestroyDeferrer::~PluginDestroyDeferrer):
        Add a simple RAII object for deferring destruction of the plug-in instance.

        (WKPCEvaluate):
        (WKPCInvoke):
        (WKPCInvokeDefault):
        (WKPCConstruct):
        (WKPCGetProperty):
        (WKPCSetProperty):
        (WKPCRemoveProperty):
        (WKPCHasProperty):
        (WKPCHasMethod):
        Use the PluginDestroyDeferrer.

        * Plugins/Hosted/NetscapePluginInstanceProxy.h:
        (WebKit::NetscapePluginInstanceProxy::pluginID):
        Assert that the plug-in ID is not 0 here.

        * Plugins/Hosted/NetscapePluginInstanceProxy.mm:
        (WebKit::NetscapePluginInstanceProxy::NetscapePluginInstanceProxy):
        Initialize the call depth.

        (WebKit::NetscapePluginInstanceProxy::~NetscapePluginInstanceProxy):
        Set the plug-in ID to 0 to aid debugging.

        (WebKit::NetscapePluginInstanceProxy::willCallPluginFunction):
        Increment the call depth.

        (WebKit::NetscapePluginInstanceProxy::didCallPluginFunction):
        Decrement the call depth, if it's 0 and we should stop the plug-in, do so.

        (WebKit::NetscapePluginInstanceProxy::shouldStop):
        If we're called this with a non-zero call depth, set shouldStopSoon to true.

        * Plugins/Hosted/WebHostedNetscapePluginView.mm:
        (-[WebHostedNetscapePluginView shouldStop]):
        Call the proxy.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@40996 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-13 Geoffrey Garen <ggaren@apple.com>
ggaren@apple.com [Sat, 14 Feb 2009 00:01:27 +0000 (00:01 +0000)]
2009-02-13  Geoffrey Garen  <ggaren@apple.com>

        Build fix for non_JIT platforms.

        * bytecode/CodeBlock.h:
        (JSC::CodeBlock::setIsNumericCompareFunction):
        (JSC::CodeBlock::isNumericCompareFunction):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@40995 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-13 Darin Adler <darin@apple.com>
darin@apple.com [Fri, 13 Feb 2009 23:56:43 +0000 (23:56 +0000)]
2009-02-13  Darin Adler  <darin@apple.com>

        Reviewed by Mark Rowe.

        Fix broken tokenizer regression test that reflected a lifetime bug
        in the document.write optimization. The test failure was trivial to
        reproduce in COLLECT_ON_EVERY_ALLOCATION mode.

        * bindings/js/JSHTMLDocumentCustom.cpp:
        (WebCore::documentWrite): Convert strings to String rather than passing
        the pointer and length to SegmentedString. The optimization is thus
        mostly gone. However, there are two ways to bring it back: 1) Apply
        the patch that makes UString and String share the same buffers.
        2) Add a UString feature to SegmentedString; simple to do but might
        risk slowing down normal document parsing.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@40994 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years agoJavaScriptCore:
ggaren@apple.com [Fri, 13 Feb 2009 23:28:04 +0000 (23:28 +0000)]
JavaScriptCore:

2009-02-13  Geoffrey Garen  <ggaren@apple.com>

        Reviewed by Darin Adler.

        Fixed <rdar://problem/6584057> Optimize sort by JS numeric comparison
        function not to run the comparison function

        * bytecode/CodeBlock.cpp:
        (JSC::CodeBlock::CodeBlock):
        * bytecode/CodeBlock.h:
        (JSC::CodeBlock::setIsNumericCompareFunction):
        (JSC::CodeBlock::isNumericCompareFunction): Added the ability to track
        whether a CodeBlock performs a sort-like numeric comparison.

        * bytecompiler/BytecodeGenerator.cpp:
        (JSC::BytecodeGenerator::generate): Set the isNumericCompareFunction bit
        after compiling.

        * parser/Nodes.cpp:
        (JSC::FunctionBodyNode::emitBytecode): Fixed a bug that caused us to
        codegen an extra return at the end of all functions (eek!), since this
        made it harder / weirder to detect the numeric comparison pattern in
        bytecode.

        * runtime/ArrayPrototype.cpp:
        (JSC::arrayProtoFuncSort): Use the isNumericCompareFunction bit to do
        a faster sort if we can.

        * runtime/FunctionConstructor.cpp:
        (JSC::extractFunctionBody):
        (JSC::constructFunction):
        * runtime/FunctionConstructor.h: Renamed and exported extractFunctionBody for
        use in initializing lazyNumericCompareFunction.

        * runtime/JSArray.cpp:
        (JSC::compareNumbersForQSort):
        (JSC::compareByStringPairForQSort):
        (JSC::JSArray::sortNumeric):
        (JSC::JSArray::sort):
        * runtime/JSArray.h: Added a fast numeric sort. Renamed ArrayQSortPair
        to be more specific since we do different kinds of qsort now.

        * runtime/JSGlobalData.cpp:
        (JSC::JSGlobalData::JSGlobalData):
        (JSC::JSGlobalData::numericCompareFunction):
        (JSC::JSGlobalData::ClientData::~ClientData):
        * runtime/JSGlobalData.h: Added helper data for computing the
        isNumericCompareFunction bit.

LayoutTests:

2009-02-13  Geoffrey Garen  <ggaren@apple.com>

        Reviewed by Sam Weinig.

        Added a test for an edge case in <rdar://problem/6584057>.

        * fast/js/resources/sort-non-numbers.js: Added.
        * fast/js/sort-non-numbers.html: Added.
        * fast/js/sort-non-numbers-expected.txt: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@40993 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-13 Adam Treat <adam.treat@torchmobile.com>
treat@webkit.org [Fri, 13 Feb 2009 22:49:43 +0000 (22:49 +0000)]
2009-02-13  Adam Treat  <adam.treat@torchmobile.com>

        Caught by Darin Adler.

        No reason to check m_tokenizer twice for 0 in a row.

        * dom/Document.cpp:
        (WebCore::Document::write):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@40992 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-13 Lyon Chen <lyon.chen@torchmobile.com>
treat@webkit.org [Fri, 13 Feb 2009 21:49:04 +0000 (21:49 +0000)]
2009-02-13  Lyon Chen  <lyon.chen@torchmobile.com>

        Reviewed by Dave Hyatt.

        Take into account y() + height() when calculating getLowerRightCorner.

        * dom/ContainerNode.cpp:
        (WebCore::ContainerNode::getLowerRightCorner):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@40991 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-13 Antti Koivisto <antti@apple.com>
antti@apple.com [Fri, 13 Feb 2009 21:17:46 +0000 (21:17 +0000)]
2009-02-13  Antti Koivisto  <antti@apple.com>

        Reviewed by Dave Hyatt.

        Most of this code was in RenderBox, restore the copyright lines.

        * rendering/RenderBoxModelObject.cpp:
        * rendering/RenderBoxModelObject.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@40990 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-13 David Hyatt <hyatt@apple.com>
hyatt@apple.com [Fri, 13 Feb 2009 21:07:08 +0000 (21:07 +0000)]
2009-02-13  David Hyatt  <hyatt@apple.com>

        Get rid of printBoxDecorations, since it is dead code.

        Reviewed by Dan Bernstein

        * rendering/RenderObject.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@40989 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-13 Adam Treat <adam.treat@torchmobile.com>
treat@webkit.org [Fri, 13 Feb 2009 21:01:31 +0000 (21:01 +0000)]
2009-02-13  Adam Treat  <adam.treat@torchmobile.com>

        Reviewed by Darin Adler.

        https://bugs.webkit.org/show_bug.cgi?id=23952
        Add Document::write overload which takes a SegmentedString since this
        is what the tokenizer expects anyway.  Modified JSHTMLDocument so it once
        again calls Document::write instead of injecting the string directly into
        the tokenizer.  This ensures that all document.write's are funneled through
        one method again, but should still be just as fast for JSHTMLDocumentCustom.

        * bindings/js/JSHTMLDocumentCustom.cpp:
        (WebCore::documentWrite):
        * dom/Document.cpp:
        (WebCore::Document::write):
        * dom/Document.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@40988 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-13 David Hyatt <hyatt@apple.com>
hyatt@apple.com [Fri, 13 Feb 2009 20:38:21 +0000 (20:38 +0000)]
2009-02-13  David Hyatt  <hyatt@apple.com>

        Reviewed by Simon Fraser

        Remove updateWidgetPosition on RenderObject.  Change RenderView's set of widgets
        to have a tighter type: RenderWidget instead of RenderObject.  Devirtualize
        updateWidgetPosition, since all RenderWidgets (except for applets) are now
        RenderParts also.

        * rendering/RenderObject.cpp:
        * rendering/RenderObject.h:
        * rendering/RenderPart.cpp:
        * rendering/RenderPart.h:
        * rendering/RenderView.cpp:
        (WebCore::RenderView::updateWidgetPositions):
        (WebCore::RenderView::addWidget):
        (WebCore::RenderView::removeWidget):
        * rendering/RenderView.h:
        * rendering/RenderWidget.cpp:
        (WebCore::RenderWidget::updateWidgetPosition):
        * rendering/RenderWidget.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@40987 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-13 Darin Adler <darin@apple.com>
darin@apple.com [Fri, 13 Feb 2009 20:11:09 +0000 (20:11 +0000)]
2009-02-13  Darin Adler  <darin@apple.com>

        * StringsNotToBeLocalized.txt: Updated for recent changes.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@40986 268f45cc-cd09-0410-ab3c-d52691b4dbfc

11 years ago2009-02-13 David Hyatt <hyatt@apple.com>
hyatt@apple.com [Fri, 13 Feb 2009 20:04:11 +0000 (20:04 +0000)]
2009-02-13  David Hyatt  <hyatt@apple.com>

        Reviewed by Eric Seidel

        Remove isEditable() on RenderObject.  It is dead code.

        * rendering/RenderObject.cpp:
        * rendering/RenderObject.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@40985 268f45cc-cd09-0410-ab3c-d52691b4dbfc