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

Unified Diff: ppapi/cpp/dev/var_resource_dev.cc

Issue 148213016: [PPAPI] Moving pp::VarResource_Dev API into pp::Var (now stable). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 10 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: ppapi/cpp/dev/var_resource_dev.cc
diff --git a/ppapi/cpp/dev/var_resource_dev.cc b/ppapi/cpp/dev/var_resource_dev.cc
deleted file mode 100644
index 69fb15b0898b8d662e867f6a259b4723b9db017e..0000000000000000000000000000000000000000
--- a/ppapi/cpp/dev/var_resource_dev.cc
+++ /dev/null
@@ -1,70 +0,0 @@
-// Copyright 2013 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 "ppapi/cpp/dev/var_resource_dev.h"
-
-#include "ppapi/c/dev/ppb_var_resource_dev.h"
-#include "ppapi/cpp/logging.h"
-#include "ppapi/cpp/module_impl.h"
-
-namespace pp {
-
-namespace {
-
-template <> const char* interface_name<PPB_VarResource_Dev_0_1>() {
- return PPB_VAR_RESOURCE_DEV_INTERFACE_0_1;
-}
-
-} // namespace
-
-VarResource_Dev::VarResource_Dev(const pp::Resource& resource) : Var(Null()) {
- if (!has_interface<PPB_VarResource_Dev_0_1>()) {
- PP_NOTREACHED();
- return;
- }
-
- // Note: Var(Null()) sets is_managed_ to true, so |var_| will be properly
- // released upon destruction.
- var_ = get_interface<PPB_VarResource_Dev_0_1>()->VarFromResource(
- resource.pp_resource());
-}
-
-VarResource_Dev::VarResource_Dev(const Var& var) : Var(var) {
- if (!var.is_resource()) {
- PP_NOTREACHED();
-
- // This takes care of releasing the reference that this object holds.
- Var::operator=(Var(Null()));
- }
-}
-
-VarResource_Dev::VarResource_Dev(const VarResource_Dev& other) : Var(other) {}
-
-VarResource_Dev::~VarResource_Dev() {}
-
-VarResource_Dev& VarResource_Dev::operator=(const VarResource_Dev& other) {
- Var::operator=(other);
- return *this;
-}
-
-Var& VarResource_Dev::operator=(const Var& other) {
- if (other.is_resource()) {
- Var::operator=(other);
- } else {
- PP_NOTREACHED();
- Var::operator=(Var(Null()));
- }
- return *this;
-}
-
-pp::Resource VarResource_Dev::AsResource() {
- if (!has_interface<PPB_VarResource_Dev_0_1>())
- return pp::Resource();
-
- return pp::Resource(
- pp::PASS_REF,
- get_interface<PPB_VarResource_Dev_0_1>()->VarToResource(var_));
-}
-
-} // namespace pp

Powered by Google App Engine
This is Rietveld 408576698