Changeset 186046 in webkit


Ignore:
Timestamp:
Jun 28, 2015 9:05:35 AM (9 years ago)
Author:
mitz@apple.com
Message:

REGRESSION (r185915): Source-incompatible change in WKPageUIClient
​https://bugs.webkit.org/show_bug.cgi?id=146396

Reviewed by Anders Carlsson.

Source/WebKit2:

  • UIProcess/API/C/WKPageUIClient.h: Renamed

runJavaScript{Alert,Confirm,Prompt}_deprecatedForUseWithV0 back to runJavaScript{Alert,Confirm,Prompt}
in client versions prior to V5.

Tools:

  • TestWebKitAPI/Tests/WebKit2/DocumentStartUserScriptAlertCrash.cpp:

(TestWebKitAPI::runJavaScriptAlert): Removed _deprecatedForUseWithV0 suffix from the name.
(TestWebKitAPI::TEST): Updated for renames.

  • TestWebKitAPI/Tests/mac/FullscreenZoomInitialFrame.mm:

(runJavaScriptAlert): Removed suffix.
(TestWebKitAPI::FullscreenZoomInitialFrame::initializeView): Updated.

  • TestWebKitAPI/Tests/mac/PageVisibilityStateWithWindowChanges.mm:

(runJavaScriptAlert): Removed suffix.
(TestWebKitAPI::PageVisibilityStateWithWindowChanges::initializeView): Updated.

Location:
trunk
Files:
6 edited

Legend:

Unmodified
Added
Removed

  • trunk/Source/WebKit2/ChangeLog

    r186045 r186046  
     12015-06-28  Dan Bernstein  <mitz@apple.com>
     2
     3        REGRESSION (r185915): Source-incompatible change in WKPageUIClient
     4        https://bugs.webkit.org/show_bug.cgi?id=146396
     5
     6        Reviewed by Anders Carlsson.
     7
     8        * UIProcess/API/C/WKPageUIClient.h: Renamed
     9        runJavaScript{Alert,Confirm,Prompt}_deprecatedForUseWithV0 back to runJavaScript{Alert,Confirm,Prompt}
     10        in client versions prior to V5.
     11
    1122015-06-28  Tim Horton  <timothy_horton@apple.com>
    213

  • trunk/Source/WebKit2/UIProcess/API/C/WKPageUIClient.h

    r185915 r186046  
    117117    WKPageFocusCallback                                                 focus;
    118118    WKPageUnfocusCallback                                               unfocus;
    119     WKPageRunJavaScriptAlertCallback_deprecatedForUseWithV0             runJavaScriptAlert_deprecatedForUseWithV0;
    120     WKPageRunJavaScriptConfirmCallback_deprecatedForUseWithV0           runJavaScriptConfirm_deprecatedForUseWithV0;
    121     WKPageRunJavaScriptPromptCallback_deprecatedForUseWithV0            runJavaScriptPrompt_deprecatedForUseWithV0;
     119    WKPageRunJavaScriptAlertCallback_deprecatedForUseWithV0             runJavaScriptAlert;
     120    WKPageRunJavaScriptConfirmCallback_deprecatedForUseWithV0           runJavaScriptConfirm;
     121    WKPageRunJavaScriptPromptCallback_deprecatedForUseWithV0            runJavaScriptPrompt;
    122122    WKPageSetStatusTextCallback                                         setStatusText;
    123123    WKPageMouseDidMoveOverElementCallback_deprecatedForUseWithV0        mouseDidMoveOverElement_deprecatedForUseWithV0;
    … …  
    162162    WKPageFocusCallback                                                 focus;
    163163    WKPageUnfocusCallback                                               unfocus;
    164     WKPageRunJavaScriptAlertCallback_deprecatedForUseWithV0             runJavaScriptAlert_deprecatedForUseWithV0;
    165     WKPageRunJavaScriptConfirmCallback_deprecatedForUseWithV0           runJavaScriptConfirm_deprecatedForUseWithV0;
    166     WKPageRunJavaScriptPromptCallback_deprecatedForUseWithV0            runJavaScriptPrompt_deprecatedForUseWithV0;
     164    WKPageRunJavaScriptAlertCallback_deprecatedForUseWithV0             runJavaScriptAlert;
     165    WKPageRunJavaScriptConfirmCallback_deprecatedForUseWithV0           runJavaScriptConfirm;
     166    WKPageRunJavaScriptPromptCallback_deprecatedForUseWithV0            runJavaScriptPrompt;
    167167    WKPageSetStatusTextCallback                                         setStatusText;
    168168    WKPageMouseDidMoveOverElementCallback_deprecatedForUseWithV0        mouseDidMoveOverElement_deprecatedForUseWithV0;
    … …  
    213213    WKPageFocusCallback                                                 focus;
    214214    WKPageUnfocusCallback                                               unfocus;
    215     WKPageRunJavaScriptAlertCallback_deprecatedForUseWithV0             runJavaScriptAlert_deprecatedForUseWithV0;
    216     WKPageRunJavaScriptConfirmCallback_deprecatedForUseWithV0           runJavaScriptConfirm_deprecatedForUseWithV0;
    217     WKPageRunJavaScriptPromptCallback_deprecatedForUseWithV0            runJavaScriptPrompt_deprecatedForUseWithV0;
     215    WKPageRunJavaScriptAlertCallback_deprecatedForUseWithV0             runJavaScriptAlert;
     216    WKPageRunJavaScriptConfirmCallback_deprecatedForUseWithV0           runJavaScriptConfirm;
     217    WKPageRunJavaScriptPromptCallback_deprecatedForUseWithV0            runJavaScriptPrompt;
    218218    WKPageSetStatusTextCallback                                         setStatusText;
    219219    WKPageMouseDidMoveOverElementCallback_deprecatedForUseWithV0        mouseDidMoveOverElement_deprecatedForUseWithV0;
    … …  
    269269    WKPageFocusCallback                                                 focus;
    270270    WKPageUnfocusCallback                                               unfocus;
    271     WKPageRunJavaScriptAlertCallback_deprecatedForUseWithV0             runJavaScriptAlert_deprecatedForUseWithV0;
    272     WKPageRunJavaScriptConfirmCallback_deprecatedForUseWithV0           runJavaScriptConfirm_deprecatedForUseWithV0;
    273     WKPageRunJavaScriptPromptCallback_deprecatedForUseWithV0            runJavaScriptPrompt_deprecatedForUseWithV0;
     271    WKPageRunJavaScriptAlertCallback_deprecatedForUseWithV0             runJavaScriptAlert;
     272    WKPageRunJavaScriptConfirmCallback_deprecatedForUseWithV0           runJavaScriptConfirm;
     273    WKPageRunJavaScriptPromptCallback_deprecatedForUseWithV0            runJavaScriptPrompt;
    274274    WKPageSetStatusTextCallback                                         setStatusText;
    275275    WKPageMouseDidMoveOverElementCallback_deprecatedForUseWithV0        mouseDidMoveOverElement_deprecatedForUseWithV0;
    … …  
    328328    WKPageFocusCallback                                                 focus;
    329329    WKPageUnfocusCallback                                               unfocus;
    330     WKPageRunJavaScriptAlertCallback_deprecatedForUseWithV0             runJavaScriptAlert_deprecatedForUseWithV0;
    331     WKPageRunJavaScriptConfirmCallback_deprecatedForUseWithV0           runJavaScriptConfirm_deprecatedForUseWithV0;
    332     WKPageRunJavaScriptPromptCallback_deprecatedForUseWithV0            runJavaScriptPrompt_deprecatedForUseWithV0;
     330    WKPageRunJavaScriptAlertCallback_deprecatedForUseWithV0             runJavaScriptAlert;
     331    WKPageRunJavaScriptConfirmCallback_deprecatedForUseWithV0           runJavaScriptConfirm;
     332    WKPageRunJavaScriptPromptCallback_deprecatedForUseWithV0            runJavaScriptPrompt;
    333333    WKPageSetStatusTextCallback                                         setStatusText;
    334334    WKPageMouseDidMoveOverElementCallback_deprecatedForUseWithV0        mouseDidMoveOverElement_deprecatedForUseWithV0;
    … …  
    467467    WKPageFocusCallback                                                 focus;
    468468    WKPageUnfocusCallback                                               unfocus;
    469     WKPageRunJavaScriptAlertCallback_deprecatedForUseWithV0             runJavaScriptAlert_deprecatedForUseWithV0;
    470     WKPageRunJavaScriptConfirmCallback_deprecatedForUseWithV0           runJavaScriptConfirm_deprecatedForUseWithV0;
    471     WKPageRunJavaScriptPromptCallback_deprecatedForUseWithV0            runJavaScriptPrompt_deprecatedForUseWithV0;
     469    WKPageRunJavaScriptAlertCallback_deprecatedForUseWithV0             runJavaScriptAlert;
     470    WKPageRunJavaScriptConfirmCallback_deprecatedForUseWithV0           runJavaScriptConfirm;
     471    WKPageRunJavaScriptPromptCallback_deprecatedForUseWithV0            runJavaScriptPrompt;
    472472    WKPageSetStatusTextCallback                                         setStatusText;
    473473    WKPageMouseDidMoveOverElementCallback_deprecatedForUseWithV0        mouseDidMoveOverElement_deprecatedForUseWithV0;

  • trunk/Tools/ChangeLog

    r186023 r186046  
     12015-06-28  Dan Bernstein  <mitz@apple.com>
     2
     3        REGRESSION (r185915): Source-incompatible change in WKPageUIClient
     4        https://bugs.webkit.org/show_bug.cgi?id=146396
     5
     6        Reviewed by Anders Carlsson.
     7
     8        * TestWebKitAPI/Tests/WebKit2/DocumentStartUserScriptAlertCrash.cpp:
     9        (TestWebKitAPI::runJavaScriptAlert): Removed _deprecatedForUseWithV0 suffix from the name.
     10        (TestWebKitAPI::TEST): Updated for renames.
     11        * TestWebKitAPI/Tests/mac/FullscreenZoomInitialFrame.mm:
     12        (runJavaScriptAlert): Removed suffix.
     13        (TestWebKitAPI::FullscreenZoomInitialFrame::initializeView): Updated.
     14        * TestWebKitAPI/Tests/mac/PageVisibilityStateWithWindowChanges.mm:
     15        (runJavaScriptAlert): Removed suffix.
     16        (TestWebKitAPI::PageVisibilityStateWithWindowChanges::initializeView): Updated.
     17
    1182015-06-27  Carlos Garcia Campos  <cgarcia@igalia.com>
    219

  • trunk/Tools/TestWebKitAPI/Tests/WebKit2/DocumentStartUserScriptAlertCrash.cpp

    r185915 r186046  
    3737static bool done;
    3838
    39 static void runJavaScriptAlert_deprecatedForUseWithV0(WKPageRef page, WKStringRef alertText, WKFrameRef frame, const void* clientInfo)
     39static void runJavaScriptAlert(WKPageRef page, WKStringRef alertText, WKFrameRef frame, const void* clientInfo)
    4040{
    4141    ASSERT_NOT_NULL(frame);
    … …  
    5858
    5959    uiClient.base.version = 0;
    60     uiClient.runJavaScriptAlert_deprecatedForUseWithV0 = runJavaScriptAlert_deprecatedForUseWithV0;
     60    uiClient.runJavaScriptAlert = runJavaScriptAlert;
    6161
    6262    WKPageSetPageUIClient(webView.page(), &uiClient.base);

  • trunk/Tools/TestWebKitAPI/Tests/mac/FullscreenZoomInitialFrame.mm

    r185915 r186046  
    5858// WebKit2 WKPageUIClient
    5959
    60 static void runJavaScriptAlert_deprecatedForUseWithV0(WKPageRef page, WKStringRef alertText, WKFrameRef frame, const void* clientInfo)
     60static void runJavaScriptAlert(WKPageRef page, WKStringRef alertText, WKFrameRef frame, const void* clientInfo)
    6161{
    6262    EXPECT_TRUE(isWaitingForPageSignalToContinue);
    … …  
    113113
    114114    uiClient.base.version = 0;
    115     uiClient.runJavaScriptAlert_deprecatedForUseWithV0 = runJavaScriptAlert_deprecatedForUseWithV0;
     115    uiClient.runJavaScriptAlert = runJavaScriptAlert;
    116116
    117117    WKPageSetPageUIClient(wkView.pageRef, &uiClient.base);

  • trunk/Tools/TestWebKitAPI/Tests/mac/PageVisibilityStateWithWindowChanges.mm

    r185915 r186046  
    5353// WebKit2 WKPageUIClient
    5454
    55 static void runJavaScriptAlert_deprecatedForUseWithV0(WKPageRef page, WKStringRef alertText, WKFrameRef frame, const void* clientInfo)
     55static void runJavaScriptAlert(WKPageRef page, WKStringRef alertText, WKFrameRef frame, const void* clientInfo)
    5656{
    5757    EXPECT_TRUE(isWaitingForPageSignalToContinue);
    … …  
    100100
    101101    uiClient.base.version = 0;
    102     uiClient.runJavaScriptAlert_deprecatedForUseWithV0 = runJavaScriptAlert_deprecatedForUseWithV0;
     102    uiClient.runJavaScriptAlert = runJavaScriptAlert;
    103103
    104104    WKPageSetPageUIClient(wkView.pageRef, &uiClient.base);
Note: See TracChangeset for help on using the changeset viewer.