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

Unified Diff: content/renderer/pepper/pepper_file_chooser_host_unittest.cc

Issue 13409003: Hide ContentClient getters from embedders so that they they don't reuse content's embedder API. The… (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: sync Created 7 years, 8 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/pepper/pepper_file_chooser_host_unittest.cc
===================================================================
--- content/renderer/pepper/pepper_file_chooser_host_unittest.cc (revision 192622)
+++ content/renderer/pepper/pepper_file_chooser_host_unittest.cc (working copy)
@@ -31,11 +31,9 @@
class PepperFileChooserHostTest : public RenderViewTest {
public:
PepperFileChooserHostTest()
- : pp_instance_(123456),
- old_content_client_(NULL) {}
+ : pp_instance_(123456) {}
virtual void SetUp() {
- old_content_client_ = GetContentClient();
SetContentClient(&client_);
RenderViewTest::SetUp();
@@ -45,7 +43,6 @@
globals_.GetResourceTracker()->DidDeleteInstance(pp_instance_);
RenderViewTest::TearDown();
- SetContentClient(old_content_client_);
}
PP_Instance pp_instance() const { return pp_instance_; }
@@ -55,9 +52,6 @@
ppapi::TestGlobals globals_;
TestContentClient client_;
-
- // Original value for the content client.
- ContentClient* old_content_client_;
};
// For testing to convert our hardcoded file paths to 8-bit.
« no previous file with comments | « content/renderer/browser_plugin/browser_plugin_browsertest.cc ('k') | content/shell/android/shell_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698