Attachment 368812 Details for Bug 197523 – Patch

[patch] Patch

bug-197523-20190502131214.patch (text/plain), 4.63 KB, created by Timothy Hatcher on 2019-05-02 13:12:15 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Timothy Hatcher

Created: 2019-05-02 13:12:15 PDT

Size: 4.63 KB

patch

obsolete

>Subversion Revision: 244808
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index c1ca7a22d5a7c6189503ad8f29bb231dfbb5f54f..8561e62e4aef78b15eb5cc4dd3938f5a325b0264 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,14 @@
>+2019-05-02  Timothy Hatcher  <timothy@apple.com>
>+
>+        NSAttributedString conversion in a loop returns nil and WKUnknownError every other time.
>+        https://bugs.webkit.org/show_bug.cgi?id=197523
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * editing/cocoa/HTMLConverter.mm:
>+        (HTMLConverter::convert): Don't return early if m_dataSource is nil. This is already null
>+        checked later and only needed in specific cases, it shouldn't fail the whole conversion.
>+
> 2019-04-30  Commit Queue  <commit-queue@webkit.org>
> 
>         Unreviewed, rolling out r244773.
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index 1f6cb44e17f455c96cc0ba221c67d8f6bebc675c..ce98a77c2b2d23cd74031bdf7011e80d8c2ade95 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,22 @@
>+2019-05-02  Timothy Hatcher  <timothy@apple.com>
>+
>+        NSAttributedString conversion in a loop returns nil and WKUnknownError every other time.
>+        https://bugs.webkit.org/show_bug.cgi?id=197523
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Caching the WKWebView was loading about:blank to unload any previous content.
>+        This was causing subsequent rapid conversions to fail since the blank load
>+        would be confused with the real content loading. Ultimatly this wasn't needed
>+        since a process swap will likely happen, and cached views are closed quickly.
>+        It also avoids extra work and speeds up conversions a bit.
>+
>+        * UIProcess/API/Cocoa/NSAttributedString.mm:
>+        (+[_WKAttributedStringWebViewCache cacheWebView:]): Don't load about:blank when caching.
>+        * WebProcess/WebPage/Cocoa/WebPageCocoa.mm:
>+        (WebKit::WebPage::getContentsAsAttributedString): Use rangeOfContents() for a fail
>+        safe way to get the range needed.
>+
> 2019-04-30  Chris Dumez  <cdumez@apple.com>
> 
>         Regression(PSON) URL scheme handlers can no longer respond asynchronously
>diff --git a/Source/WebCore/editing/cocoa/HTMLConverter.mm b/Source/WebCore/editing/cocoa/HTMLConverter.mm
>index b7ab890bf22ade85e6488a8ed9dcacfe0482c5b7..1f38019f253f8822e7516bf4c688265b915c1f78 100644
>--- a/Source/WebCore/editing/cocoa/HTMLConverter.mm
>+++ b/Source/WebCore/editing/cocoa/HTMLConverter.mm
>@@ -407,8 +407,6 @@ NSAttributedString *HTMLConverter::convert(NSDictionary** documentAttributes)
>     ASSERT(commonAncestorContainer);
> 
>     m_dataSource = commonAncestorContainer->document().frame()->loader().documentLoader();
>-    if (!m_dataSource)
>-        return nil;
> 
>     Document& document = commonAncestorContainer->document();
>     if (auto* body = document.bodyOrFrameset()) {
>diff --git a/Source/WebKit/UIProcess/API/Cocoa/NSAttributedString.mm b/Source/WebKit/UIProcess/API/Cocoa/NSAttributedString.mm
>index 10a76952370fb3bbcc451f45ad7aa74fae07b6ac..78f5a3a0ba96fdc9cc8ee60ddd130fde1a9cbd2a 100644
>--- a/Source/WebKit/UIProcess/API/Cocoa/NSAttributedString.mm
>+++ b/Source/WebKit/UIProcess/API/Cocoa/NSAttributedString.mm
>@@ -170,9 +170,6 @@ + (void)cacheWebView:(WKWebView *)webView
>         return;
> 
>     [cache addObject:webView];
>-
>-    // Load a blank page to clear anything loaded while in the cache.
>-    [webView loadRequest:[NSURLRequest requestWithURL:[NSURL URLWithString:@"about:blank"]]];
> }
> 
> + (void)resetPurgeDelay
>diff --git a/Source/WebKit/WebProcess/WebPage/Cocoa/WebPageCocoa.mm b/Source/WebKit/WebProcess/WebPage/Cocoa/WebPageCocoa.mm
>index ac042460280abf10e249818963db19c5ff998511..88f55a2f61c05e2818f8b7cfa51b6e50c985431e 100644
>--- a/Source/WebKit/WebProcess/WebPage/Cocoa/WebPageCocoa.mm
>+++ b/Source/WebKit/WebProcess/WebPage/Cocoa/WebPageCocoa.mm
>@@ -205,18 +205,10 @@ WebPaymentCoordinator* WebPage::paymentCoordinator()
> 
> void WebPage::getContentsAsAttributedString(CompletionHandler<void(const AttributedString&)>&& completionHandler)
> {
>-    Frame& frame = m_page->mainFrame();
>-
>-    RefPtr<Range> range = TextIterator::rangeFromLocationAndLength(frame.document()->documentElement(), 0, INT_MAX);
>-    if (!range) {
>-        completionHandler({ });
>-        return;
>-    }
>-
>     NSDictionary* documentAttributes = nil;
> 
>     AttributedString result;
>-    result.string = attributedStringFromRange(*range, &documentAttributes);
>+    result.string = attributedStringFromRange(rangeOfContents(*m_page->mainFrame().document()->documentElement()), &documentAttributes);
>     result.documentAttributes = documentAttributes;
> 
>     completionHandler({ result });

Actions: View | Formatted Diff | Diff

Attachments on bug 197523: 368812 | 368830