Changeset 206038 in webkit

Timestamp:

Sep 16, 2016 12:32:20 PM (8 years ago)

Author:

Wenson Hsieh

Message:

RequiresUserActionForPlayback TestWebKitAPI tests are broken after r206033.
​https://bugs.webkit.org/show_bug.cgi?id=162080

Reviewed by Jer Noble.

I renamed "playingHandler" to the more generalized "testHandler", but did not realize that the
RequiresUserActionForPlayback tests also used "playingHandler". Renames the handler to account for this change.

• TestWebKitAPI/Tests/WebKit2Cocoa/RequiresUserActionForPlayback.mm:

(RequiresUserActionForPlaybackTest::SetUp):

Location:

trunk/Tools

Files:

• ChangeLog (modified) (1 diff)
• TestWebKitAPI/Tests/WebKit2Cocoa/RequiresUserActionForPlayback.mm (modified) (1 diff)

trunk/Tools/ChangeLog

@@ +1 @@
+2016-09-16  Wenson Hsieh  <wenson_hsieh@apple.com>
+
+        RequiresUserActionForPlayback TestWebKitAPI tests are broken after r206033.
+        https://bugs.webkit.org/show_bug.cgi?id=162080
+
+        Reviewed by Jer Noble.
+
+        I renamed "playingHandler" to the more generalized "testHandler", but did not realize that the
+        RequiresUserActionForPlayback tests also used "playingHandler". Renames the handler to account for this change.
+
+        * TestWebKitAPI/Tests/WebKit2Cocoa/RequiresUserActionForPlayback.mm:
+        (RequiresUserActionForPlaybackTest::SetUp):
+
 2016-09-16  Alex Christensen  <achristensen@webkit.org>
 

trunk/Tools/TestWebKitAPI/Tests/WebKit2Cocoa/RequiresUserActionForPlayback.mm

@@ -62 +62 @@
         handler = adoptNS([[RequiresUserActionForPlaybackMessageHandler alloc] init]);
         configuration = adoptNS([[WKWebViewConfiguration alloc] init]);
-        [[configuration userContentController] addScriptMessageHandler:handler.get() name:@"playingHandler"];
+        [[configuration userContentController] addScriptMessageHandler:handler.get() name:@"testHandler"];
         configuration.get()._mediaDataLoadsAutomatically = YES;
 #if TARGET_OS_IPHONE