[Webkit-unassigned] [Bug 193881] Use a load optimizer for some sites

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=193881

--- Comment #12 from Jiewen Tan <jiewen_tan at apple.com> ---
Comment on attachment 360281
  --> https://bugs.webkit.org/attachment.cgi?id=360281
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=360281&action=review

>>> Source/WebKit/UIProcess/Cocoa/NavigationState.mm:490
>>> +    completionHandler(false);
>> 
>> This could be written this way:
>> #if HAVE(LOAD_OPTIMIZER)
>>     page.websiteDataStore().loadOptimizer().tryLoading(navigationAction->request(), page, WTFMove(completionHandler));
>> #else
>>     completionHandler(false);
>> #endif
>> That way it is up to the loadOptimizer instance to know when it can or not optimize the load.
> 
> Good catch. I would keep that in mind. Since we have this lock step development process for things involving internal. I will address this later.

<rdar://problem/47582894>

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20190127/63ba2c5f/attachment-0001.html>