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

Unified Diff: Source/core/html/shadow/PluginPlaceholderElement.cpp

Issue 522783002: Add a custom element to own structure of shadow DOM plugin placeholders. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Blink-in-JS Created 6 years, 4 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: Source/core/html/shadow/PluginPlaceholderElement.cpp
diff --git a/Source/core/html/shadow/PluginPlaceholderElement.cpp b/Source/core/html/shadow/PluginPlaceholderElement.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..e52bec75f8b214b0462f3be6db133ab9187fe34b
--- /dev/null
+++ b/Source/core/html/shadow/PluginPlaceholderElement.cpp
@@ -0,0 +1,38 @@
+// Copyright 2014 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 "config.h"
+#include "core/html/shadow/PluginPlaceholderElement.h"
+
+#include "bindings/core/v8/V8PluginPlaceholderElement.h"
+#include "core/dom/Document.h"
+
+namespace blink {
+
+PluginPlaceholderElement::PluginPlaceholderElement(Document& document)
+ : HTMLDivElement(document)
+{
+ ScriptWrappable::init(this);
+}
+
+PassRefPtrWillBeRawPtr<PluginPlaceholderElement> PluginPlaceholderElement::create(Document& document)
+{
+ RefPtrWillBeRawPtr<PluginPlaceholderElement> element = adoptRefWillBeNoop(new PluginPlaceholderElement(document));
+ element->initializePlaceholderElements();
+ return element.release();
+}
+
+void PluginPlaceholderElement::setMessage(const String& message)
+{
+ bool success = V8PluginPlaceholderElement::PrivateScript::messageAttributeSetter(document().frame(), this, message);
+ ASSERT_UNUSED(success, success);
+}
+
+void PluginPlaceholderElement::initializePlaceholderElements()
haraken 2014/08/31 14:53:33 This helper function looks redundant.
jbroman 2014/08/31 16:53:49 Yes, this could be inlined into create, if that's
+{
+ bool success = V8PluginPlaceholderElement::PrivateScript::initializePlaceholderElementsMethod(document().frame(), this);
+ ASSERT_UNUSED(success, success);
+}
+
+} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698