Skip to content
  • abarth@webkit.org's avatar
    Remove webkitPostMessage · 79bb5ebb
    abarth@webkit.org authored
    https://bugs.webkit.org/show_bug.cgi?id=96577
    
    Reviewed by Ojan Vafai.
    
    .:
    
    Add ENABLE_LEGACY_VENDOR_PREFIXES flag.
    
    * Source/cmake/WebKitFeatures.cmake:
    * Source/cmakeconfig.h.cmake:
    
    Source/JavaScriptCore:
    
    Add ENABLE_LEGACY_VENDOR_PREFIXES flag.
    
    * Configurations/FeatureDefines.xcconfig:
    
    Source/WebCore:
    
    webkitPostMessage works the same as postMessage. The spec and the
    implementation have been stable for a while. We should no longer be
    exposing this vendor-prefixed API.
    
    This patch places the webkitPostMessage API behind an ENABLE flag.
    We're going to try removing this API in the Chromium port. If we don't
    run into trouble, we'll remove it in all the other ports as well.
    
    This topic has been discussed on webkit-dev in
    http://lists.webkit.org/pipermail/webkit-dev/2012-April/020237.html and
    http://lists.webkit.org/pipermail/webkit-dev/2012-September/022189.html
    
    This patch introduces the ENABLE(LEGACY_VENDOR_PREFIXES) flag. Rather
    than having a flag for each vendor-prefixed API we're experimenting
    with removing, we'll add vendor prefixed APIs to this ENABLE when we
    want to try removing them. If we succeed, we'll just delete the APIs.
    If we fail, we'll remove them from the ENABLE. That way we avoid the
    churn of adding and removing many ENABLE macros.
    
    * Configurations/FeatureDefines.xcconfig:
    * GNUmakefile.am:
    * GNUmakefile.features.am:
    * bindings/js/JSDOMWindowCustom.cpp:
    (WebCore):
    * bindings/js/JSDedicatedWorkerContextCustom.cpp:
    (WebCore):
    * bindings/js/JSMessagePortCustom.cpp:
    (WebCore):
    * bindings/js/JSWorkerCustom.cpp:
    (WebCore):
    * bindings/v8/custom/V8DOMWindowCustom.cpp:
    (WebCore):
    * bindings/v8/custom/V8DedicatedWorkerContextCustom.cpp:
    (WebCore):
    * bindings/v8/custom/V8MessagePortCustom.cpp:
    (WebCore):
    * bindings/v8/custom/V8WorkerCustom.cpp:
    (WebCore):
    * dom/MessagePort.idl:
    * page/DOMWindow.idl:
    * workers/DedicatedWorkerContext.idl:
    * workers/Worker.idl:
    
    Source/WebKit/chromium:
    
    Add ENABLE_LEGACY_VENDOR_PREFIXES flag.
    
    * features.gypi:
    
    Source/WebKit/mac:
    
    Add ENABLE_LEGACY_VENDOR_PREFIXES flag.
    
    * Configurations/FeatureDefines.xcconfig:
    
    Source/WebKit2:
    
    Add ENABLE_LEGACY_VENDOR_PREFIXES flag.
    
    * Configurations/FeatureDefines.xcconfig:
    
    Tools:
    
    Add ENABLE_LEGACY_VENDOR_PREFIXES flag.
    
    * Scripts/webkitperl/FeatureList.pm:
    * qmake/mkspecs/features/features.pri:
    
    LayoutTests:
    
    Update tests to rely only upon the unprefixed API.
    
    * fast/canvas/webgl/script-tests/arraybuffer-transfer-of-control.js:
    (wrapSend):
    (wrapFailSend):
    * fast/dom/Window/post-message-crash.html:
    * fast/dom/Window/window-postmessage-args-expected.txt:
    * fast/dom/Window/window-postmessage-args.html:
    * fast/events/message-port.html:
    * fast/events/resources/message-port-multi.js:
    * fast/workers/resources/worker-call.js:
    * fast/workers/worker-call-expected.txt:
    
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@128658 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    79bb5ebb