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

Unified Diff: components/plugins/renderer/test_plugin_placeholder.cc

Issue 1158063002: Add plugins::TestPluginPlaceholder class and allow its use in Blink layout tests. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@pluginfix
Patch Set: Created 5 years, 7 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: components/plugins/renderer/test_plugin_placeholder.cc
diff --git a/components/plugins/renderer/test_plugin_placeholder.cc b/components/plugins/renderer/test_plugin_placeholder.cc
new file mode 100644
index 0000000000000000000000000000000000000000..73abf3b43406a096476fb54c9d5644e99bc0901b
--- /dev/null
+++ b/components/plugins/renderer/test_plugin_placeholder.cc
@@ -0,0 +1,14 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "components/plugins/renderer/test_plugin_placeholder.h"
+
+namespace plugins {
+
+TestPluginPlaceholder::TestPluginPlaceholder(
+ blink::WebLocalFrame* frame, const blink::WebPluginParams& params)
+ : PluginPlaceholder(nullptr, frame, params, "<div>Test content</div>")
tommycli 2015/05/26 20:08:18 Can we just eliminate this class entirely? Why not
chrishtr 2015/05/26 20:52:36 The reason is that someone has to include the stri
tommycli 2015/05/26 21:18:02 Perhaps I am misunderstanding something. Can "<div
+{ }
+
+} // namespace plugins

Powered by Google App Engine
This is Rietveld 408576698