Changeset 166574 in webkit


Ignore:
Timestamp:
Apr 1, 2014 2:07:25 AM (10 years ago)
Author:
zandobersek@gmail.com
Message:

[GTK] Don't copy the ResourceResponse object in webkitWebViewDecidePolicy
https://bugs.webkit.org/show_bug.cgi?id=131015

Reviewed by Carlos Garcia Campos.

  • UIProcess/API/gtk/WebKitWebView.cpp:

(webkitWebViewDecidePolicy): Avoid copying the ResourceResponce object returned by webkitURIResponseGetResourceResponse.
That function already returns a reference, so the return value should be stored accordingly.

Location:
trunk/Source/WebKit2
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebKit2/ChangeLog

    r166572 r166574  
     12014-04-01  Zan Dobersek  <zdobersek@igalia.com>
     2
     3        [GTK] Don't copy the ResourceResponse object in webkitWebViewDecidePolicy
     4        https://bugs.webkit.org/show_bug.cgi?id=131015
     5
     6        Reviewed by Carlos Garcia Campos.
     7
     8        * UIProcess/API/gtk/WebKitWebView.cpp:
     9        (webkitWebViewDecidePolicy): Avoid copying the ResourceResponce object returned by webkitURIResponseGetResourceResponse.
     10        That function already returns a reference, so the return value should be stored accordingly.
     11
    1122014-04-01  Eunmi Lee  <eunmi15.lee@samsung.com>
    213
  • trunk/Source/WebKit2/UIProcess/API/gtk/WebKitWebView.cpp

    r166410 r166574  
    274274
    275275    WebKitURIResponse* response = webkit_response_policy_decision_get_response(WEBKIT_RESPONSE_POLICY_DECISION(decision));
    276     const ResourceResponse resourceResponse = webkitURIResponseGetResourceResponse(response);
     276    const ResourceResponse& resourceResponse = webkitURIResponseGetResourceResponse(response);
    277277    if (resourceResponse.isAttachment()) {
    278278        webkit_policy_decision_download(decision);
Note: See TracChangeset for help on using the changeset viewer.