Unreviewed, rolling out r167962 and r167975.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 30 Apr 2014 05:42:57 +0000 (05:42 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 30 Apr 2014 05:42:57 +0000 (05:42 +0000)
https://bugs.webkit.org/show_bug.cgi?id=132376

Mysteriously broke many tests (Requested by ap on #webkit).

Reverted changesets:

"[Mac, iOS] Support caption activation via JS
webkitHasClosedCaptions method"
https://bugs.webkit.org/show_bug.cgi?id=132320
http://trac.webkit.org/changeset/167962

"Unreviewed test fix after r167962."
http://trac.webkit.org/changeset/167975

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

Source/WebCore/ChangeLog
Source/WebCore/Modules/mediacontrols/mediaControlsApple.css
Source/WebCore/html/HTMLMediaElement.cpp
Source/WebCore/html/HTMLMediaElement.h
Source/WebCore/html/shadow/MediaControlElements.cpp
Source/WebCore/html/shadow/MediaControlElements.h

index 0a0a224..af05e7a 100644 (file)
@@ -1,3 +1,20 @@
+2014-04-29  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r167962 and r167975.
+        https://bugs.webkit.org/show_bug.cgi?id=132376
+
+        Mysteriously broke many tests (Requested by ap on #webkit).
+
+        Reverted changesets:
+
+        "[Mac, iOS] Support caption activation via JS
+        webkitHasClosedCaptions method"
+        https://bugs.webkit.org/show_bug.cgi?id=132320
+        http://trac.webkit.org/changeset/167962
+
+        "Unreviewed test fix after r167962."
+        http://trac.webkit.org/changeset/167975
+
 2014-04-29  David Hyatt  <hyatt@apple.com>
 
         [New Multicolumn] Multiple tests assert in RenderGeometryMap
index f4db032..c781c19 100644 (file)
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 
  */
 
-.hidden {
-    display: none;
-}
-
 audio {
     width: 200px;
     height: 25px;
index 0c3821e..9bd4c4e 100644 (file)
@@ -3695,64 +3695,9 @@ void HTMLMediaElement::configureTextTrackGroup(const TrackGroup& group)
             m_webkitLegacyClosedCaptionOverride = true;
     }
 
-    updateCaptionContainer();
-
     m_processingPreferenceChange = false;
 }
 
-void HTMLMediaElement::updateCaptionContainer()
-{
-    LOG(Media, "HTMLMediaElement::updateCaptionContainer");
-#if ENABLE(MEDIA_CONTROLS_SCRIPT)
-    Page* page = document().page();
-    if (!page)
-        return;
-
-    DOMWrapperWorld& world = ensureIsolatedWorld();
-
-    if (!ensureMediaControlsInjectedScript())
-        return;
-
-    ensureUserAgentShadowRoot();
-
-    ASSERT(m_mediaControlsHost);
-
-    ScriptController& scriptController = page->mainFrame().script();
-    JSDOMGlobalObject* globalObject = JSC::jsCast<JSDOMGlobalObject*>(scriptController.globalObject(world));
-    JSC::ExecState* exec = globalObject->globalExec();
-    JSC::JSLockHolder lock(exec);
-
-    JSC::JSValue controllerValue = m_mediaControlsHost->controllerJSValue();
-    if (controllerValue.isUndefinedOrNull() || !controllerValue.isObject())
-        return;
-
-    JSC::JSObject* controllerObject = controllerValue.toObject(exec);
-
-    // The media controls script must provide a method on the Controller object with the following details.
-    // Name: updateCaptionContainer
-    // Parameters:
-    //     None
-    // Return value:
-    //     None
-    JSC::JSValue methodValue = controllerObject->get(exec, JSC::Identifier(exec, "updateCaptionContainer"));
-    if (methodValue.isUndefinedOrNull() || !methodValue.isObject())
-        return;
-
-    JSC::JSObject* methodObject = methodValue.toObject(exec);
-
-    JSC::CallData callData;
-    JSC::CallType callType = methodObject->methodTable()->getCallData(methodObject, callData);
-    if (callType == JSC::CallTypeNone)
-        return;
-
-    JSC::MarkedArgumentBuffer noArguments;
-    JSC::call(exec, methodObject, callType, callData, controllerObject, noArguments);
-
-    if (exec->hadException())
-        exec->clearException();
-#endif
-}
-    
 void HTMLMediaElement::setSelectedTextTrack(TextTrack* trackToSelect)
 {
     TextTrackList* trackList = textTracks();
index 32956c3..d0f2be4 100644 (file)
@@ -721,8 +721,6 @@ private:
     void registerWithDocument(Document&);
     void unregisterWithDocument(Document&);
 
-    void updateCaptionContainer();
-
     Timer<HTMLMediaElement> m_loadTimer;
     Timer<HTMLMediaElement> m_progressEventTimer;
     Timer<HTMLMediaElement> m_playbackProgressTimer;
index 594bf0d..c833649 100644 (file)
@@ -1500,17 +1500,6 @@ void MediaControlTextTrackContainerElement::textTrackRepresentationBoundsChanged
     updateSizes();
 }
 
-void MediaControlTextTrackContainerElement::hide()
-{
-    classList()->add("hidden", IGNORE_EXCEPTION);
-}
-
-void MediaControlTextTrackContainerElement::show()
-{
-    classList()->remove("hidden", IGNORE_EXCEPTION);
-}
-    
-
 #endif // ENABLE(VIDEO_TRACK)
 
 // ----------------------------
index 69cd2e5..b8a7f40 100644 (file)
@@ -476,9 +476,6 @@ public:
     void exitedFullscreen();
     static const AtomicString& textTrackContainerElementShadowPseudoId();
 
-    virtual void show() override;
-    virtual void hide() override;
-
 private:
     void updateTimerFired(Timer<MediaControlTextTrackContainerElement>&);