Attachment 372574 Details for Bug 198902 – Patch

[patch] Patch

bug-198902-20190620114202.patch (text/plain), 19.04 KB, created by youenn fablet on 2019-06-20 11:42:02 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: youenn fablet

Created: 2019-06-20 11:42:02 PDT

Size: 19.04 KB

patch

obsolete

>Subversion Revision: 246549
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index f4d8755efb0797419557e265e45e96a870b31523..db06edf7fe021c0d8d5bebdcb443f4916b18481b 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,39 @@
>+2019-06-20  Youenn Fablet  <youenn@apple.com>
>+
>+        WebPageProxy should use the right path for sandbox extension
>+        https://bugs.webkit.org/show_bug.cgi?id=198902
>+        <rdar://problem/50772810>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Store the sandbox path, if any, in UIProcess for each page in its PageLoadState.
>+        This allows proper sandbox handling for reload after crash and process swap cases.
>+        Store the sandbox path, if any, in the b/w list so that the sandbox path can be properly computed
>+        during b/f navigation works.
>+
>+        Add SPI for test purposes to check what is the current sandbox path.
>+
>+        * Shared/WebBackForwardListItem.h:
>+        (WebKit::WebBackForwardListItem::resourceDirectoryURL const):
>+        (WebKit::WebBackForwardListItem::setResourceDirectoryURL):
>+        * UIProcess/API/Cocoa/WKWebView.mm:
>+        (-[WKWebView _resourceDirectoryURL]):
>+        * UIProcess/API/Cocoa/WKWebViewPrivate.h:
>+        * UIProcess/PageLoadState.cpp:
>+        (WebKit::PageLoadState::resourceDirectoryURL const):
>+        (WebKit::PageLoadState::setPendingAPIRequestURL):
>+        * UIProcess/PageLoadState.h:
>+        (WebKit::PageLoadState::setPendingAPIRequestURL):
>+        * UIProcess/WebPageProxy.cpp:
>+        (WebKit::WebPageProxy::maybeInitializeSandboxExtensionHandle):
>+        (WebKit::WebPageProxy::loadRequestWithNavigationShared):
>+        (WebKit::WebPageProxy::loadFile):
>+        (WebKit::WebPageProxy::reload):
>+        (WebKit::WebPageProxy::backForwardAddItem):
>+        (WebKit::WebPageProxy::backForwardGoToItemShared):
>+        (WebKit::WebPageProxy::currentResourceDirectoryURL const):
>+        * UIProcess/WebPageProxy.h:
>+
> 2019-06-18  Adrian Perez de Castro  <aperez@igalia.com>
> 
>         Unreviewed. Update OptionsWPE.cmake and NEWS for the 2.25.1 release
>diff --git a/Source/WebKit/Shared/WebBackForwardListItem.h b/Source/WebKit/Shared/WebBackForwardListItem.h
>index 5b2d2e033afd81684beef42c486ba87a8bb842f1..3b9b0bfd9b1e48b756329c55c6a442da9de19969 100644
>--- a/Source/WebKit/Shared/WebBackForwardListItem.h
>+++ b/Source/WebKit/Shared/WebBackForwardListItem.h
>@@ -67,6 +67,9 @@ public:
>     const String& url() const { return m_itemState.pageState.mainFrameState.urlString; }
>     const String& title() const { return m_itemState.pageState.title; }
> 
>+    const URL& resourceDirectoryURL() const { return m_resourceDirectoryURL; }
>+    void setResourceDirectoryURL(URL&& url) { m_resourceDirectoryURL = WTFMove(url); }
>+
>     bool itemIsInSameDocument(const WebBackForwardListItem&) const;
>     bool itemIsClone(const WebBackForwardListItem&);
> 
>@@ -87,6 +90,7 @@ private:
>     void removeSuspendedPageFromProcessPool();
> 
>     BackForwardListItemState m_itemState;
>+    URL m_resourceDirectoryURL;
>     WebCore::PageIdentifier m_pageID;
>     WebCore::ProcessIdentifier m_lastProcessIdentifier;
>     WeakPtr<SuspendedPageProxy> m_suspendedPage;
>diff --git a/Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm b/Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm
>index d11b57a76386e595f3aa6a1d2aafd6e4859d500f..bbeaef4865e1da30bdf1ac443adee07108f86397 100644
>--- a/Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm
>+++ b/Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm
>@@ -982,6 +982,11 @@ - (NSURL *)URL
>     return [NSURL _web_URLWithWTFString:_page->pageLoadState().activeURL()];
> }
> 
>+- (NSURL *)_resourceDirectoryURL
>+{
>+    return _page->currentResourceDirectoryURL();
>+}
>+
> - (BOOL)isLoading
> {
>     return _page->pageLoadState().isLoading();
>diff --git a/Source/WebKit/UIProcess/API/Cocoa/WKWebViewPrivate.h b/Source/WebKit/UIProcess/API/Cocoa/WKWebViewPrivate.h
>index c675bfff266b5748a566d2740bd6d731ede64909..723a34d507d2209fd4eb48f91f641beedafff881 100644
>--- a/Source/WebKit/UIProcess/API/Cocoa/WKWebViewPrivate.h
>+++ b/Source/WebKit/UIProcess/API/Cocoa/WKWebViewPrivate.h
>@@ -176,6 +176,8 @@ typedef NS_OPTIONS(NSUInteger, _WKRectEdge) {
> 
> @property (nonatomic, readonly, getter=_isShowingNavigationGestureSnapshot) BOOL _showingNavigationGestureSnapshot;
> 
>+@property (nonatomic, readonly) NSURL *_resourceDirectoryURL WK_API_AVAILABLE(macos(WK_MAC_TBA), ios(WK_IOS_TBA));
>+
> - (void)_close;
> 
> - (void)_updateWebsitePolicies:(_WKWebsitePolicies *)websitePolicies WK_API_AVAILABLE(macos(10.13), ios(11.0));
>diff --git a/Source/WebKit/UIProcess/PageLoadState.cpp b/Source/WebKit/UIProcess/PageLoadState.cpp
>index 10185a467764902d004f80c83767a1069510e3a5..0cf6a0de5b51579c76d6e6b0e4a532fcf11eaeb9 100644
>--- a/Source/WebKit/UIProcess/PageLoadState.cpp
>+++ b/Source/WebKit/UIProcess/PageLoadState.cpp
>@@ -239,10 +239,16 @@ const String& PageLoadState::pendingAPIRequestURL() const
>     return m_committedState.pendingAPIRequestURL;
> }
> 
>-void PageLoadState::setPendingAPIRequestURL(const Transaction::Token& token, const String& pendingAPIRequestURL)
>+const URL& PageLoadState::resourceDirectoryURL() const
>+{
>+    return m_committedState.resourceDirectoryURL;
>+}
>+
>+void PageLoadState::setPendingAPIRequestURL(const Transaction::Token& token, const String& pendingAPIRequestURL, const URL& resourceDirectoryURL)
> {
>     ASSERT_UNUSED(token, &token.m_pageLoadState == this);
>     m_uncommittedState.pendingAPIRequestURL = pendingAPIRequestURL;
>+    m_uncommittedState.resourceDirectoryURL = resourceDirectoryURL;
> }
> 
> void PageLoadState::clearPendingAPIRequestURL(const Transaction::Token& token)
>diff --git a/Source/WebKit/UIProcess/PageLoadState.h b/Source/WebKit/UIProcess/PageLoadState.h
>index dbeaa9a1744a2cd90b58c457f816ae5989d47f7f..20769180b01f9fec6b40619ba96944b228b63dd7 100644
>--- a/Source/WebKit/UIProcess/PageLoadState.h
>+++ b/Source/WebKit/UIProcess/PageLoadState.h
>@@ -27,6 +27,7 @@
> #define PageLoadState_h
> 
> #include "WebCertificateInfo.h"
>+#include <wtf/URL.h>
> #include <wtf/text/WTFString.h>
> 
> namespace WebKit {
>@@ -140,8 +141,10 @@ public:
> 
>     WebCertificateInfo* certificateInfo() const { return m_committedState.certificateInfo.get(); }
> 
>+    const URL& resourceDirectoryURL() const;
>+
>     const String& pendingAPIRequestURL() const;
>-    void setPendingAPIRequestURL(const Transaction::Token&, const String&);
>+    void setPendingAPIRequestURL(const Transaction::Token&, const String& pendingAPIRequestURL, const URL& resourceDirectoryPath = { });
>     void clearPendingAPIRequestURL(const Transaction::Token&);
> 
>     void didStartProvisionalLoad(const Transaction::Token&, const String& url, const String& unreachableURL);
>@@ -213,6 +216,8 @@ private:
> 
>         String title;
> 
>+        URL resourceDirectoryURL;
>+
>         bool canGoBack;
>         bool canGoForward;
> 
>diff --git a/Source/WebKit/UIProcess/WebBackForwardList.cpp b/Source/WebKit/UIProcess/WebBackForwardList.cpp
>index a65261869c96462ecf4230ae02ba5bee1d594684..8275cf6ec952fd0eb1952f0bac0f463184fa27ee 100644
>--- a/Source/WebKit/UIProcess/WebBackForwardList.cpp
>+++ b/Source/WebKit/UIProcess/WebBackForwardList.cpp
>@@ -434,6 +434,7 @@ void WebBackForwardList::restoreFromState(BackForwardListState backForwardListSt
>     Vector<Ref<WebBackForwardListItem>> items;
>     items.reserveInitialCapacity(backForwardListState.items.size());
> 
>+    // FIXME: Enable restoring resourceDirectoryURL.
>     for (auto& backForwardListItemState : backForwardListState.items) {
>         backForwardListItemState.identifier = { Process::identifier(), ObjectIdentifier<BackForwardItemIdentifier::ItemIdentifierType>::generate() };
>         items.uncheckedAppend(WebBackForwardListItem::create(WTFMove(backForwardListItemState), m_page->pageID()));
>diff --git a/Source/WebKit/UIProcess/WebPageProxy.cpp b/Source/WebKit/UIProcess/WebPageProxy.cpp
>index 1302a84e107fa6d380730da1b86939ff29bfc2dc..deb2ada1a22218230226f01d2a0b5dc38f23ed37 100644
>--- a/Source/WebKit/UIProcess/WebPageProxy.cpp
>+++ b/Source/WebKit/UIProcess/WebPageProxy.cpp
>@@ -1048,19 +1048,29 @@ bool WebPageProxy::tryClose()
>     return false;
> }
> 
>-bool WebPageProxy::maybeInitializeSandboxExtensionHandle(WebProcessProxy& process, const URL& url, SandboxExtension::Handle& sandboxExtensionHandle)
>+void WebPageProxy::maybeInitializeSandboxExtensionHandle(WebProcessProxy& process, const URL& url, const URL& resourceDirectoryURL, SandboxExtension::Handle& sandboxExtensionHandle)
> {
>     if (!url.isLocalFile())
>-        return false;
>+        return;
>+
>+    if (!resourceDirectoryURL.isEmpty()) {
>+        if (process.hasAssumedReadAccessToURL(resourceDirectoryURL))
>+            return;
>+
>+        if (SandboxExtension::createHandle(resourceDirectoryURL.fileSystemPath(), SandboxExtension::Type::ReadOnly, sandboxExtensionHandle)) {
>+            m_process->assumeReadAccessToBaseURL(*this, resourceDirectoryURL);
>+            return;
>+        }
>+    }
> 
>     if (process.hasAssumedReadAccessToURL(url))
>-        return false;
>+        return;
> 
>     // Inspector resources are in a directory with assumed access.
>     ASSERT_WITH_SECURITY_IMPLICATION(!WebKit::isInspectorPage(*this));
> 
>-    SandboxExtension::createHandle("/", SandboxExtension::Type::ReadOnly, sandboxExtensionHandle);
>-    return true;
>+    if (SandboxExtension::createHandle("/", SandboxExtension::Type::ReadOnly, sandboxExtensionHandle))
>+        willAcquireUniversalFileReadSandboxExtension(process);
> }
> 
> #if !PLATFORM(COCOA)
>@@ -1114,9 +1124,8 @@ void WebPageProxy::loadRequestWithNavigationShared(Ref<WebProcessProxy>&& proces
>     loadParameters.lockHistory = navigation.lockHistory();
>     loadParameters.lockBackForwardList = navigation.lockBackForwardList();
>     loadParameters.clientRedirectSourceForHistory = navigation.clientRedirectSourceForHistory();
>-    bool createdExtension = maybeInitializeSandboxExtensionHandle(process, url, loadParameters.sandboxExtensionHandle);
>-    if (createdExtension)
>-        willAcquireUniversalFileReadSandboxExtension(process);
>+    maybeInitializeSandboxExtensionHandle(process, url, m_pageLoadState.resourceDirectoryURL(), loadParameters.sandboxExtensionHandle);
>+
>     addPlatformLoadParameters(loadParameters);
> 
>     process->send(Messages::WebPage::LoadRequest(loadParameters), m_pageID);
>@@ -1156,7 +1165,7 @@ RefPtr<API::Navigation> WebPageProxy::loadFile(const String& fileURLString, cons
> 
>     auto transaction = m_pageLoadState.transaction();
> 
>-    m_pageLoadState.setPendingAPIRequestURL(transaction, fileURLString);
>+    m_pageLoadState.setPendingAPIRequestURL(transaction, fileURLString, resourceDirectoryURL);
> 
>     String resourceDirectoryPath = resourceDirectoryURL.fileSystemPath();
> 
>@@ -1336,9 +1345,7 @@ RefPtr<API::Navigation> WebPageProxy::reload(OptionSet<WebCore::ReloadOption> op
>         m_pageLoadState.setPendingAPIRequestURL(transaction, url);
> 
>         // We may not have an extension yet if back/forward list was reinstated after a WebProcess crash or a browser relaunch
>-        bool createdExtension = maybeInitializeSandboxExtensionHandle(m_process, URL(URL(), url), sandboxExtensionHandle);
>-        if (createdExtension)
>-            willAcquireUniversalFileReadSandboxExtension(m_process);
>+        maybeInitializeSandboxExtensionHandle(m_process, URL(URL(), url), currentResourceDirectoryURL(), sandboxExtensionHandle);
>     }
> 
>     if (!hasRunningProcess())
>@@ -5714,6 +5721,7 @@ void WebPageProxy::requestDOMPasteAccess(const WebCore::IntRect& elementRect, co
> void WebPageProxy::backForwardAddItem(BackForwardListItemState&& itemState)
> {
>     auto item = WebBackForwardListItem::create(WTFMove(itemState), pageID());
>+    item->setResourceDirectoryURL(currentResourceDirectoryURL());
>     m_backForwardList->addItem(WTFMove(item));
> }
> 
>@@ -5736,9 +5744,7 @@ void WebPageProxy::backForwardGoToItemShared(Ref<WebProcessProxy>&& process, con
>         return completionHandler({ });
> 
>     SandboxExtension::Handle sandboxExtensionHandle;
>-    bool createdExtension = maybeInitializeSandboxExtensionHandle(process, URL(URL(), item->url()), sandboxExtensionHandle);
>-    if (createdExtension)
>-        willAcquireUniversalFileReadSandboxExtension(process);
>+    maybeInitializeSandboxExtensionHandle(process, URL(URL(), item->url()), item->resourceDirectoryURL(),  sandboxExtensionHandle);
>     m_backForwardList->goToItem(*item);
>     completionHandler(WTFMove(sandboxExtensionHandle));
> }
>@@ -6789,6 +6795,16 @@ String WebPageProxy::currentURL() const
>     return url;
> }
> 
>+URL WebPageProxy::currentResourceDirectoryURL() const
>+{
>+    auto resourceDirectoryURL = m_pageLoadState.resourceDirectoryURL();
>+    if (!resourceDirectoryURL.isEmpty())
>+        return resourceDirectoryURL;
>+    if (auto* item = m_backForwardList->currentItem())
>+        return item->resourceDirectoryURL();
>+    return { };
>+}
>+
> void WebPageProxy::processDidTerminate(ProcessTerminationReason reason)
> {
>     if (reason != ProcessTerminationReason::NavigationSwap)
>diff --git a/Source/WebKit/UIProcess/WebPageProxy.h b/Source/WebKit/UIProcess/WebPageProxy.h
>index b7736b6a7d483bc34020f75960feade237cff726..1ada1df9ca85121d4413c625e2742bc0672b080e 100644
>--- a/Source/WebKit/UIProcess/WebPageProxy.h
>+++ b/Source/WebKit/UIProcess/WebPageProxy.h
>@@ -1559,6 +1559,8 @@ public:
> 
>     void requestStorageSpace(uint64_t frameID, const String& originIdentifier, const String& databaseName, const String& displayName, uint64_t currentQuota, uint64_t currentOriginUsage, uint64_t currentDatabaseUsage, uint64_t expectedUsage, WTF::CompletionHandler<void(uint64_t)>&&);
> 
>+    URL currentResourceDirectoryURL() const;
>+
> private:
>     WebPageProxy(PageClient&, WebProcessProxy&, WebCore::PageIdentifier, Ref<API::PageConfiguration>&&);
>     void platformInitialize();
>@@ -1895,7 +1897,7 @@ private:
>     void setPluginComplexTextInputState(uint64_t pluginComplexTextInputIdentifier, uint64_t complexTextInputState);
> #endif
> 
>-    bool maybeInitializeSandboxExtensionHandle(WebProcessProxy&, const URL&, SandboxExtension::Handle&);
>+    void maybeInitializeSandboxExtensionHandle(WebProcessProxy&, const URL& url, const URL& resourceDirectoryURL, SandboxExtension::Handle&);
> 
> #if USE(AUTOMATIC_TEXT_REPLACEMENT)
>     void toggleSmartInsertDelete();
>diff --git a/Tools/ChangeLog b/Tools/ChangeLog
>index 46999cceae7f439eb9380eda97e8be0247e25937..2573bffa82b1dbf88a97c73f99edf84dbf9396ad 100644
>--- a/Tools/ChangeLog
>+++ b/Tools/ChangeLog
>@@ -1,3 +1,15 @@
>+2019-06-20  Youenn Fablet  <youenn@apple.com>
>+
>+        WebPageProxy should use the right path for sandbox extension
>+        https://bugs.webkit.org/show_bug.cgi?id=198902
>+        <rdar://problem/50772810>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * TestWebKitAPI/Tests/WebKitCocoa/LoadFileURL.mm:
>+        (TEST):
>+        * TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm:
>+
> 2019-06-18  David Quesada  <david_quesada@apple.com>
> 
>         REGRESSION: _WKDownload.OriginatingWebView and _WKDownload.CrashAfterDownloadDidFinishWhenDownloadProxyHoldsTheLastRefOnWebProcessPool failing
>diff --git a/Tools/TestWebKitAPI/Tests/WebKitCocoa/LoadFileURL.mm b/Tools/TestWebKitAPI/Tests/WebKitCocoa/LoadFileURL.mm
>index 24f229bf1772ee340428f8d61def75242d06635d..dd1fdc3e01d30672522a6057964d3ced491744d5 100644
>--- a/Tools/TestWebKitAPI/Tests/WebKitCocoa/LoadFileURL.mm
>+++ b/Tools/TestWebKitAPI/Tests/WebKitCocoa/LoadFileURL.mm
>@@ -29,6 +29,7 @@
> #import "Test.h"
> #import "TestNavigationDelegate.h"
> #import <WebKit/WebKit.h>
>+#import <WebKit/WebViewPrivate.h>
> #import <wtf/RetainPtr.h>
> 
> 
>@@ -42,6 +43,7 @@ TEST(WKWebView, LoadTwoFiles)
>     NSURL *file = [[NSBundle mainBundle] URLForResource:@"simple" withExtension:@"html" subdirectory:@"TestWebKitAPI.resources"];
>     [webView loadFileURL:file allowingReadAccessToURL:file.URLByDeletingLastPathComponent];
>     [delegate waitForDidFinishNavigation];
>+    EXPECT_WK_STREQ(webView.get()._resourceDirectoryURL.path, file.URLByDeletingLastPathComponent.path);
> 
>     // Load a second file: resource that is in a completely different directory from the above
>     file = [[NSBundle mainBundle] URLForResource:@"simple2" withExtension:@"html" subdirectory:@"TestWebKitAPI.resources"];
>@@ -49,8 +51,18 @@ TEST(WKWebView, LoadTwoFiles)
>     [[NSFileManager defaultManager] createDirectoryAtURL:targetURL.URLByDeletingLastPathComponent withIntermediateDirectories:YES attributes:nil error:nil];
>     [[NSFileManager defaultManager] removeItemAtURL:targetURL error:nil];
>     [[NSFileManager defaultManager] copyItemAtURL:file toURL:targetURL error:nil];
>-    
>+
>     // If this second load succeeds (e.g. doesn't timeout due to a sandbox violation) the test passes
>     [webView loadFileURL:targetURL allowingReadAccessToURL:targetURL.URLByDeletingLastPathComponent];
>     [delegate waitForDidFinishNavigation];
>+    EXPECT_WK_STREQ(webView.get()._resourceDirectoryURL.path, targetURL.URLByDeletingLastPathComponent.path);
>+
>+    [webView _killWebContentProcessAndResetState];
>+    [webView reload];
>+    [delegate waitForDidFinishNavigation];
>+    EXPECT_WK_STREQ(webView.get()._resourceDirectoryURL.path, targetURL.URLByDeletingLastPathComponent.path);
>+
>+    [webView goBack];
>+    [delegate waitForDidFinishNavigation];
>+    EXPECT_WK_STREQ(webView.get()._resourceDirectoryURL.path, file.URLByDeletingLastPathComponent.path);
> }
>diff --git a/Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm b/Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm
>index 6217c9898a56fc3720b9120f04dfc90cb52ac17a..b532a6d3aa5f006bd3e4df1ccd7bc8c2de9d2b46 100644
>--- a/Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm
>+++ b/Tools/TestWebKitAPI/Tests/WebKitCocoa/ProcessSwapOnNavigation.mm
>@@ -6295,3 +6295,33 @@ TEST(ProcessSwap, SuspendAllMediaPlayback)
>     [webView synchronouslyLoadTestPageNamed:@"video-with-audio"];
>     TestWebKitAPI::Util::run(&notPlaying);
> }
>+
>+TEST(ProcessSwap, PassSandboxExtension)
>+{
>+    auto processPoolConfiguration = psonProcessPoolConfiguration();
>+    auto processPool = adoptNS([[WKProcessPool alloc] _initWithConfiguration:processPoolConfiguration.get()]);
>+
>+    auto configuration = adoptNS([[WKWebViewConfiguration alloc] init]);
>+    configuration.get().mediaTypesRequiringUserActionForPlayback = WKAudiovisualMediaTypeNone;
>+#if TARGET_OS_IPHONE
>+    configuration.get().allowsInlineMediaPlayback = YES;
>+#endif
>+    [configuration setProcessPool:processPool.get()];
>+    auto handler = adoptNS([[PSONScheme alloc] init]);
>+    [configuration setURLSchemeHandler:handler.get() forURLScheme:@"PSON"];
>+
>+    auto webView = adoptNS([[TestWKWebView alloc] initWithFrame:NSMakeRect(0, 0, 800, 600) configuration:configuration.get()]);
>+
>+    auto navigationDelegate = adoptNS([[PSONNavigationDelegate alloc] init]);
>+    [webView setNavigationDelegate:navigationDelegate.get()];
>+
>+    [webView loadRequest:[NSURLRequest requestWithURL:[NSURL URLWithString:@"pson://www.webkit.org/main.html"]]];
>+    TestWebKitAPI::Util::run(&done);
>+    done = false;
>+
>+    NSURL *file = [[NSBundle mainBundle] URLForResource:@"autoplay-with-controls" withExtension:@"html" subdirectory:@"TestWebKitAPI.resources"];
>+    [webView loadFileURL:file allowingReadAccessToURL:file.URLByDeletingLastPathComponent];
>+    [webView waitForMessage:@"loaded"];
>+
>+    EXPECT_WK_STREQ(webView.get()._resourceDirectoryURL.path, file.URLByDeletingLastPathComponent.path);
>+}

Actions: View | Formatted Diff | Diff

Attachments on bug 198902: 372216 | 372218 | 372219 | 372275 | 372340 | 372350 | 372513 | 372556 | 372574