Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(48)

Unified Diff: content/renderer/renderer_webkitplatformsupport_impl.h

Issue 10449094: Fix client-side phishing detection test flakiness and ChromeOS failure. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Restore the previous sandbox-enabled state Created 8 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/renderer/renderer_webkitplatformsupport_impl.h
diff --git a/content/renderer/renderer_webkitplatformsupport_impl.h b/content/renderer/renderer_webkitplatformsupport_impl.h
index f125b0957326c6f10cdb63aee9b6994058bc7b45..413290928f5cb8f6ca7ed8446adfb51a0bb7d973 100644
--- a/content/renderer/renderer_webkitplatformsupport_impl.h
+++ b/content/renderer/renderer_webkitplatformsupport_impl.h
@@ -92,6 +92,15 @@ class CONTENT_EXPORT RendererWebKitPlatformSupportImpl
virtual WebKit::WebMediaStreamCenter* createMediaStreamCenter(
WebKit::WebMediaStreamCenterClient* client) OVERRIDE;
+ // Disables the WebSandboxSupport implementation for testing.
+ // Tests that do not set up a full sandbox environment should call
+ // SetSandboxEnabledForTesting(false) _before_ creating any instances
+ // of this class, to ensure that we don't attempt to use sandbox-related
+ // file descriptors or other resources.
+ //
+ // Returns the previous |enable| value.
+ static bool SetSandboxEnabledForTesting(bool enable);
+
protected:
virtual GpuChannelHostFactory* GetGpuChannelHostFactory() OVERRIDE;
« no previous file with comments | « content/public/test/render_view_fake_resources_test.h ('k') | content/renderer/renderer_webkitplatformsupport_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698