Attachment 341070 Details for Bug 185877 – Patch

[patch] Patch

bug-185877-20180522215444.patch (text/plain), 21.32 KB, created by Brent Fulgham on 2018-05-22 21:54:44 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Brent Fulgham

Created: 2018-05-22 21:54:44 PDT

Size: 21.32 KB

patch

obsolete

>Subversion Revision: 232094
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index c95c6c50eafe14eedc09ae161d12f4742d9c854e..573ddb1f8c9770f808f050083965122b25537436 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,49 @@
>+2018-05-22  Brent Fulgham  <bfulgham@apple.com>
>+
>+        Avoid keeping FormState alive longer than necessary
>+        https://bugs.webkit.org/show_bug.cgi?id=185877
>+        <rdar://problem/39329219>
>+
>+        Reviewed by Ryosuke Niwa.
>+
>+        A number of crash fixes were done to prevent FormState objects from being
>+        accessed after their relevant Frames had been destroyed. Unfortunately, this
>+        could cause the FormState to persist after the owning Frame had been
>+        destroyed, resulting in nullptr dereferences.
>+
>+        This patch does the following:
>+
>+        1. Changes to use WeakPtr's for FormState objects passed to completion handlers,
>+           rather than RefPtr, since those completion handlers might fire as part of
>+           the clean-up process during Frame destruction. This allows us to use the FormState
>+           if they are still valid, but gracefully handle cases where a form submission
>+           is cancelled in-flight.
>+        2. Removes some extraneous WTFMove() calls being made on bare FormState pointers.
>+        3. Changes the trap from Bug 183704 so that it only fires if the FormState object
>+           is being retained more than once.
>+
>+        * loader/DocumentLoader.cpp:
>+        (WebCore::DocumentLoader::willSendRequest): Update for new CompletionHandler
>+        signature.
>+        * loader/FormState.cpp:
>+        (WebCore::FormState::willDetachPage): Revise trap to check for retain counts
>+        above one.
>+        * loader/FormState.h:
>+        (WebCore::FormState::weakPtrFactory const): Added.
>+        * loader/FrameLoader.cpp:
>+        (WebCore::FrameLoader::loadFrameRequest): Revise to use WeakPtr for FormState
>+        passed to the completion handler.
>+        (WebCore::FrameLoader::loadURL): Update for new CompletionHandler signature.
>+        (WebCore::FrameLoader::load): Ditto.
>+        (WebCore::FrameLoader::loadWithDocumentLoader): Ditto.
>+        (WebCore::FrameLoader::loadPostRequest): Ditto.
>+        * loader/PolicyChecker.cpp:
>+        (WebCore::PolicyChecker::checkNavigationPolicy): Revise to use WeakPtr for
>+        FormState passed to the completion handler. Remove some extraneous WTFMove()
>+        calls on bare pointers.
>+        (WebCore::PolicyChecker::checkNewWindowPolicy): Ditto.
>+        * loader/PolicyChecker.h:
>+
> 2018-05-22  Dean Jackson  <dino@apple.com>
> 
>         Optimized path zoom animation needs a valid UIImage and CGRect
>diff --git a/Source/WebCore/loader/DocumentLoader.cpp b/Source/WebCore/loader/DocumentLoader.cpp
>index 9e09bd35e0d34fc74a4907b92692ce023264d20d..7ed4f64c921181e48af9bd603259eed55d1b883a 100644
>--- a/Source/WebCore/loader/DocumentLoader.cpp
>+++ b/Source/WebCore/loader/DocumentLoader.cpp
>@@ -1,5 +1,5 @@
> /*
>- * Copyright (C) 2006-2017 Apple Inc. All rights reserved.
>+ * Copyright (C) 2006-2018 Apple Inc. All rights reserved.
>  * Copyright (C) 2011 Google Inc. All rights reserved.
>  *
>  * Redistribution and use in source and binary forms, with or without
>@@ -640,7 +640,7 @@ void DocumentLoader::willSendRequest(ResourceRequest&& newRequest, const Resourc
>     if (!didReceiveRedirectResponse && shouldContinue != ShouldContinue::ForSuspension)
>         return completionHandler(WTFMove(newRequest));
> 
>-    auto navigationPolicyCompletionHandler = [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)] (ResourceRequest&& request, FormState*, ShouldContinue shouldContinue) mutable {
>+    auto navigationPolicyCompletionHandler = [this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)] (ResourceRequest&& request, WeakPtr<FormState>&&, ShouldContinue shouldContinue) mutable {
>         m_waitingForNavigationPolicy = false;
>         switch (shouldContinue) {
>         case ShouldContinue::ForSuspension:
>diff --git a/Source/WebCore/loader/FormState.cpp b/Source/WebCore/loader/FormState.cpp
>index 9fa8bdd3e1cc36ab11ccde5f3019652894bfd05b..f8700b1ad3831a3d43163081a8f0c062b6ca06f4 100644
>--- a/Source/WebCore/loader/FormState.cpp
>+++ b/Source/WebCore/loader/FormState.cpp
>@@ -1,5 +1,5 @@
> /*
>- * Copyright (C) 2006-2017 Apple Inc. All rights reserved.
>+ * Copyright (C) 2006-2018 Apple Inc. All rights reserved.
>  *
>  * Redistribution and use in source and binary forms, with or without
>  * modification, are permitted provided that the following conditions
>@@ -52,7 +52,7 @@ Ref<FormState> FormState::create(HTMLFormElement& form, StringPairVector&& textF
> void FormState::willDetachPage()
> {
>     // Beartrap for <rdar://problem/37579354>
>-    RELEASE_ASSERT_NOT_REACHED();
>+    RELEASE_ASSERT(hasOneRef());
> }
> 
> }
>diff --git a/Source/WebCore/loader/FormState.h b/Source/WebCore/loader/FormState.h
>index 2634a97b1236c976234e50da7d5c20397cdcc2fb..77584329af16424650c00db8a793d9d955f5050f 100644
>--- a/Source/WebCore/loader/FormState.h
>+++ b/Source/WebCore/loader/FormState.h
>@@ -1,5 +1,5 @@
> /*
>- * Copyright (C) 2006-2017 Apple Inc. All rights reserved.
>+ * Copyright (C) 2006-2018 Apple Inc. All rights reserved.
>  *
>  * Redistribution and use in source and binary forms, with or without
>  * modification, are permitted provided that the following conditions
>@@ -29,6 +29,7 @@
> #pragma once
> 
> #include "FrameDestructionObserver.h"
>+#include <wtf/WeakPtr.h>
> #include <wtf/text/WTFString.h>
> 
> namespace WebCore {
>@@ -49,6 +50,8 @@ public:
>     Document& sourceDocument() const { return m_sourceDocument; }
>     FormSubmissionTrigger formSubmissionTrigger() const { return m_formSubmissionTrigger; }
> 
>+    auto& weakPtrFactory() const { return m_weakFactory; }
>+
> private:
>     FormState(HTMLFormElement&, StringPairVector&& textFieldValues, Document&, FormSubmissionTrigger);
>     void willDetachPage() override;
>@@ -57,6 +60,7 @@ private:
>     StringPairVector m_textFieldValues;
>     Ref<Document> m_sourceDocument;
>     FormSubmissionTrigger m_formSubmissionTrigger;
>+    WeakPtrFactory<FormState> m_weakFactory;
> };
> 
> } // namespace WebCore
>diff --git a/Source/WebCore/loader/FrameLoader.cpp b/Source/WebCore/loader/FrameLoader.cpp
>index 30d00253cde8a74837b0d475c0f17351f5aa8af8..c89e6ce8254a4b6f8df9e0e96e12165f701bd880 100644
>--- a/Source/WebCore/loader/FrameLoader.cpp
>+++ b/Source/WebCore/loader/FrameLoader.cpp
>@@ -1,5 +1,5 @@
> /*
>- * Copyright (C) 2006-2016 Apple Inc. All rights reserved.
>+ * Copyright (C) 2006-2018 Apple Inc. All rights reserved.
>  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
>  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
>  * Copyright (C) 2008 Alp Toker <alp@atoker.com>
>@@ -1228,7 +1228,7 @@ void FrameLoader::loadFrameRequest(FrameLoadRequest&& request, Event* event, For
>     else
>         loadType = FrameLoadType::Standard;
> 
>-    auto completionHandler = [this, protectedFrame = makeRef(m_frame), formState = makeRefPtr(formState), frameName = request.frameName()] {
>+    auto completionHandler = [this, protectedFrame = makeRef(m_frame), formState = makeWeakPtr(formState), frameName = request.frameName()] {
>         // FIXME: It's possible this targetFrame will not be the same frame that was targeted by the actual
>         // load if frame names have changed.
>         Frame* sourceFrame = formState ? formState->sourceDocument().frame() : &m_frame;
>@@ -1338,8 +1338,8 @@ void FrameLoader::loadURL(FrameLoadRequest&& frameLoadRequest, const String& ref
> 
>     if (!targetFrame && !frameName.isEmpty()) {
>         action = action.copyWithShouldOpenExternalURLsPolicy(shouldOpenExternalURLsPolicyToApply(m_frame, frameLoadRequest));
>-        policyChecker().checkNewWindowPolicy(WTFMove(action), WTFMove(request), formState, frameName, [this, allowNavigationToInvalidURL, openerPolicy, completionHandler = completionHandlerCaller.release()] (const ResourceRequest& request, FormState* formState, const String& frameName, const NavigationAction& action, ShouldContinue shouldContinue) {
>-            continueLoadAfterNewWindowPolicy(request, formState, frameName, action, shouldContinue, allowNavigationToInvalidURL, openerPolicy);
>+        policyChecker().checkNewWindowPolicy(WTFMove(action), WTFMove(request), formState, frameName, [this, allowNavigationToInvalidURL, openerPolicy, completionHandler = completionHandlerCaller.release()] (const ResourceRequest& request, WeakPtr<FormState>&& formState, const String& frameName, const NavigationAction& action, ShouldContinue shouldContinue) {
>+            continueLoadAfterNewWindowPolicy(request, formState.get(), frameName, action, shouldContinue, allowNavigationToInvalidURL, openerPolicy);
>             completionHandler();
>         });
>         return;
>@@ -1358,7 +1358,7 @@ void FrameLoader::loadURL(FrameLoadRequest&& frameLoadRequest, const String& ref
>         oldDocumentLoader->setLastCheckedRequest(ResourceRequest());
>         policyChecker().stopCheck();
>         policyChecker().setLoadType(newLoadType);
>-        policyChecker().checkNavigationPolicy(WTFMove(request), false /* didReceiveRedirectResponse */, oldDocumentLoader.get(), formState, [this, protectedFrame = makeRef(m_frame)] (const ResourceRequest& request, FormState*, ShouldContinue shouldContinue) {
>+        policyChecker().checkNavigationPolicy(WTFMove(request), false /* didReceiveRedirectResponse */, oldDocumentLoader.get(), formState, [this, protectedFrame = makeRef(m_frame)] (const ResourceRequest& request, WeakPtr<FormState>&&, ShouldContinue shouldContinue) {
>             continueFragmentScrollAfterNavigationPolicy(request, shouldContinue == ShouldContinue::Yes);
>         }, PolicyDecisionMode::Synchronous);
>         return;
>@@ -1419,8 +1419,8 @@ void FrameLoader::load(FrameLoadRequest&& request)
> 
>     if (request.shouldCheckNewWindowPolicy()) {
>         NavigationAction action { request.requester(), request.resourceRequest(), InitiatedByMainFrame::Unknown, NavigationType::Other, request.shouldOpenExternalURLsPolicy() };
>-        policyChecker().checkNewWindowPolicy(WTFMove(action), WTFMove(request.resourceRequest()), nullptr, request.frameName(), [this] (const ResourceRequest& request, FormState* formState, const String& frameName, const NavigationAction& action, ShouldContinue shouldContinue) {
>-            continueLoadAfterNewWindowPolicy(request, formState, frameName, action, shouldContinue, AllowNavigationToInvalidURL::Yes, NewFrameOpenerPolicy::Suppress);
>+        policyChecker().checkNewWindowPolicy(WTFMove(action), WTFMove(request.resourceRequest()), nullptr, request.frameName(), [this] (const ResourceRequest& request, WeakPtr<FormState>&& formState, const String& frameName, const NavigationAction& action, ShouldContinue shouldContinue) {
>+            continueLoadAfterNewWindowPolicy(request, formState.get(), frameName, action, shouldContinue, AllowNavigationToInvalidURL::Yes, NewFrameOpenerPolicy::Suppress);
>         });
> 
>         return;
>@@ -1533,7 +1533,7 @@ void FrameLoader::loadWithDocumentLoader(DocumentLoader* loader, FrameLoadType t
>         oldDocumentLoader->setTriggeringAction(action);
>         oldDocumentLoader->setLastCheckedRequest(ResourceRequest());
>         policyChecker().stopCheck();
>-        policyChecker().checkNavigationPolicy(ResourceRequest(loader->request()), false /* didReceiveRedirectResponse */, oldDocumentLoader.get(), formState, [this, protectedFrame = makeRef(m_frame)] (const ResourceRequest& request, FormState*, ShouldContinue shouldContinue) {
>+        policyChecker().checkNavigationPolicy(ResourceRequest(loader->request()), false /* didReceiveRedirectResponse */, oldDocumentLoader.get(), formState, [this, protectedFrame = makeRef(m_frame)] (const ResourceRequest& request, WeakPtr<FormState>&&, ShouldContinue shouldContinue) {
>             continueFragmentScrollAfterNavigationPolicy(request, shouldContinue == ShouldContinue::Yes);
>         }, PolicyDecisionMode::Synchronous);
>         return;
>@@ -1567,8 +1567,8 @@ void FrameLoader::loadWithDocumentLoader(DocumentLoader* loader, FrameLoadType t
>         return;
>     }
> 
>-    policyChecker().checkNavigationPolicy(ResourceRequest(loader->request()), false /* didReceiveRedirectResponse */, loader, formState, [this, protectedFrame = makeRef(m_frame), allowNavigationToInvalidURL, completionHandler = completionHandlerCaller.release()] (const ResourceRequest& request, FormState* formState, ShouldContinue shouldContinue) {
>-        continueLoadAfterNavigationPolicy(request, formState, shouldContinue, allowNavigationToInvalidURL);
>+    policyChecker().checkNavigationPolicy(ResourceRequest(loader->request()), false /* didReceiveRedirectResponse */, loader, formState, [this, protectedFrame = makeRef(m_frame), allowNavigationToInvalidURL, completionHandler = completionHandlerCaller.release()] (const ResourceRequest& request, WeakPtr<FormState>&& formState, ShouldContinue shouldContinue) {
>+        continueLoadAfterNavigationPolicy(request, formState.get(), shouldContinue, allowNavigationToInvalidURL);
>         completionHandler();
>     });
> }
>@@ -2861,13 +2861,13 @@ void FrameLoader::loadPostRequest(FrameLoadRequest&& request, const String& refe
> 
>     if (!frameName.isEmpty()) {
>         // The search for a target frame is done earlier in the case of form submission.
>-        if (Frame* targetFrame = formState ? 0 : findFrameForNavigation(frameName)) {
>-            targetFrame->loader().loadWithNavigationAction(workingResourceRequest, action, lockHistory, loadType, WTFMove(formState), allowNavigationToInvalidURL, WTFMove(completionHandler));
>+        if (auto* targetFrame = formState ? nullptr : findFrameForNavigation(frameName)) {
>+            targetFrame->loader().loadWithNavigationAction(workingResourceRequest, action, lockHistory, loadType, formState, allowNavigationToInvalidURL, WTFMove(completionHandler));
>             return;
>         }
> 
>-        policyChecker().checkNewWindowPolicy(WTFMove(action), WTFMove(workingResourceRequest), WTFMove(formState), frameName, [this, allowNavigationToInvalidURL, openerPolicy, completionHandler = WTFMove(completionHandler)] (const ResourceRequest& request, FormState* formState, const String& frameName, const NavigationAction& action, ShouldContinue shouldContinue) {
>-            continueLoadAfterNewWindowPolicy(request, formState, frameName, action, shouldContinue, allowNavigationToInvalidURL, openerPolicy);
>+        policyChecker().checkNewWindowPolicy(WTFMove(action), WTFMove(workingResourceRequest), formState, frameName, [this, allowNavigationToInvalidURL, openerPolicy, completionHandler = WTFMove(completionHandler)] (const ResourceRequest& request, WeakPtr<FormState>&& formState, const String& frameName, const NavigationAction& action, ShouldContinue shouldContinue) {
>+            continueLoadAfterNewWindowPolicy(request, formState.get(), frameName, action, shouldContinue, allowNavigationToInvalidURL, openerPolicy);
>             completionHandler();
>         });
>         return;
>@@ -2875,7 +2875,7 @@ void FrameLoader::loadPostRequest(FrameLoadRequest&& request, const String& refe
> 
>     // must grab this now, since this load may stop the previous load and clear this flag
>     bool isRedirect = m_quickRedirectComing;
>-    loadWithNavigationAction(workingResourceRequest, action, lockHistory, loadType, WTFMove(formState), allowNavigationToInvalidURL, [this, isRedirect, protectedFrame = makeRef(m_frame), completionHandler = WTFMove(completionHandler)] {
>+    loadWithNavigationAction(workingResourceRequest, action, lockHistory, loadType, formState, allowNavigationToInvalidURL, [this, isRedirect, protectedFrame = makeRef(m_frame), completionHandler = WTFMove(completionHandler)] {
>         if (isRedirect) {
>             m_quickRedirectComing = false;
>             if (m_provisionalDocumentLoader)
>@@ -3630,7 +3630,7 @@ void FrameLoader::loadDifferentDocumentItem(HistoryItem& item, FrameLoadType loa
> 
>     action.setTargetBackForwardItem(item);
> 
>-    loadWithNavigationAction(request, action, LockHistory::No, loadType, 0, AllowNavigationToInvalidURL::Yes, [] { });
>+    loadWithNavigationAction(request, action, LockHistory::No, loadType, nullptr, AllowNavigationToInvalidURL::Yes, [] { });
> }
> 
> // Loads content into this frame, as specified by history item
>diff --git a/Source/WebCore/loader/PolicyChecker.cpp b/Source/WebCore/loader/PolicyChecker.cpp
>index b6690ea9665a1515c97a63adc9407732b9819127..714e81dcf6dca96d09a4224f7be4a3abe365d554 100644
>--- a/Source/WebCore/loader/PolicyChecker.cpp
>+++ b/Source/WebCore/loader/PolicyChecker.cpp
>@@ -1,5 +1,5 @@
> /*
>- * Copyright (C) 2006-2016 Apple Inc. All rights reserved.
>+ * Copyright (C) 2006-2018 Apple Inc. All rights reserved.
>  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
>  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
>  *
>@@ -109,7 +109,7 @@ void PolicyChecker::checkNavigationPolicy(ResourceRequest&& request, bool didRec
>     // Don't ask more than once for the same request or if we are loading an empty URL.
>     // This avoids confusion on the part of the client.
>     if (equalIgnoringHeaderFields(request, loader->lastCheckedRequest()) || (!request.isNull() && request.url().isEmpty())) {
>-        function(ResourceRequest(request), nullptr, ShouldContinue::Yes);
>+        function(ResourceRequest(request), { }, ShouldContinue::Yes);
>         loader->setLastCheckedRequest(WTFMove(request));
>         return;
>     }
>@@ -124,7 +124,7 @@ void PolicyChecker::checkNavigationPolicy(ResourceRequest&& request, bool didRec
> #endif
>         if (isBackForwardLoadType(m_loadType))
>             m_loadType = FrameLoadType::Reload;
>-        function(WTFMove(request), nullptr, shouldContinue ? ShouldContinue::Yes : ShouldContinue::No);
>+        function(WTFMove(request), { }, shouldContinue ? ShouldContinue::Yes : ShouldContinue::No);
>         return;
>     }
> 
>@@ -134,7 +134,7 @@ void PolicyChecker::checkNavigationPolicy(ResourceRequest&& request, bool didRec
>             // reveal that the frame was blocked. This way, it looks like any other cross-origin page load.
>             m_frame.ownerElement()->dispatchEvent(Event::create(eventNames().loadEvent, false, false));
>         }
>-        function(WTFMove(request), nullptr, ShouldContinue::No);
>+        function(WTFMove(request), { }, ShouldContinue::No);
>         return;
>     }
> 
>@@ -147,7 +147,7 @@ void PolicyChecker::checkNavigationPolicy(ResourceRequest&& request, bool didRec
> #if USE(QUICK_LOOK)
>     // Always allow QuickLook-generated URLs based on the protocol scheme.
>     if (!request.isNull() && isQuickLookPreviewURL(request.url()))
>-        return function(WTFMove(request), formState, ShouldContinue::Yes);
>+        return function(WTFMove(request), makeWeakPtr(formState), ShouldContinue::Yes);
> #endif
> 
> #if ENABLE(CONTENT_FILTERING)
>@@ -183,9 +183,9 @@ void PolicyChecker::checkNavigationPolicy(ResourceRequest&& request, bool didRec
>         case PolicyAction::Use:
>             if (!m_frame.loader().client().canHandleRequest(request)) {
>                 handleUnimplementablePolicy(m_frame.loader().client().cannotShowURLError(request));
>-                return function({ }, nullptr, ShouldContinue::No);
>+                return function({ }, { }, ShouldContinue::No);
>             }
>-            return function(WTFMove(request), formState.get(), ShouldContinue::Yes);
>+            return function(WTFMove(request), makeWeakPtr(formState.get()), ShouldContinue::Yes);
>         }
>         ASSERT_NOT_REACHED();
>     });
>@@ -213,7 +213,7 @@ void PolicyChecker::checkNewWindowPolicy(NavigationAction&& navigationAction, Re
>             // It is invalid to get a "Suspend" policy for new windows, as the old document is not going away.
>             RELEASE_ASSERT_NOT_REACHED();
>         case PolicyAction::Use:
>-            function(request, formState.get(), frameName, navigationAction, ShouldContinue::Yes);
>+            function(request, makeWeakPtr(formState.get()), frameName, navigationAction, ShouldContinue::Yes);
>             return;
>         }
>         ASSERT_NOT_REACHED();
>diff --git a/Source/WebCore/loader/PolicyChecker.h b/Source/WebCore/loader/PolicyChecker.h
>index c329ec43b81e35e8b73abe6e601d13d4250852af..6f112119146c48b4eb4e9b02681dc0e9b2af56f0 100644
>--- a/Source/WebCore/loader/PolicyChecker.h
>+++ b/Source/WebCore/loader/PolicyChecker.h
>@@ -1,5 +1,5 @@
> /*
>- * Copyright (C) 2006-2016 Apple Inc. All rights reserved.
>+ * Copyright (C) 2006-2018 Apple Inc. All rights reserved.
>  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
>  *
>  * Redistribution and use in source and binary forms, with or without
>@@ -31,6 +31,7 @@
> 
> #include "FrameLoaderTypes.h"
> #include "ResourceRequest.h"
>+#include <wtf/WeakPtr.h>
> #include <wtf/text/WTFString.h>
> 
> #if ENABLE(CONTENT_FILTERING)
>@@ -59,8 +60,8 @@ enum class ShouldContinue {
> 
> enum class PolicyDecisionMode { Synchronous, Asynchronous };
> 
>-using NewWindowPolicyDecisionFunction = CompletionHandler<void(const ResourceRequest&, FormState*, const String& frameName, const NavigationAction&, ShouldContinue)>;
>-using NavigationPolicyDecisionFunction = CompletionHandler<void(ResourceRequest&&, FormState*, ShouldContinue)>;
>+using NewWindowPolicyDecisionFunction = CompletionHandler<void(const ResourceRequest&, WeakPtr<FormState>&&, const String& frameName, const NavigationAction&, ShouldContinue)>;
>+using NavigationPolicyDecisionFunction = CompletionHandler<void(ResourceRequest&&, WeakPtr<FormState>&&, ShouldContinue)>;
> 
> class PolicyChecker {
>     WTF_MAKE_NONCOPYABLE(PolicyChecker);

Actions: View | Formatted Diff | Diff

Attachments on bug 185877: 341006 | 341013 | 341019 | 341023 | 341070 | 341166 | 341170