Python handler: Pass a python object to content handled application.

mojo/public/python/src/common.h

 // 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.
 
 #ifndef MOJO_PUBLIC_PYTHON_SRC_COMMON_H_
 #define MOJO_PUBLIC_PYTHON_SRC_COMMON_H_
 
 #include <Python.h>
 
 #include <map>
@@ skipping 19 matching lines @@
 };
 
 enum ScopedPyRefAcquire {
   kAcquire,
 };
 
 class ScopedPyRef {
  public:
   explicit ScopedPyRef(PyObject* object);
   ScopedPyRef(PyObject* object, ScopedPyRefAcquire);
+  ScopedPyRef(const ScopedPyRef& other);

[etiennej, 2015/02/24 13:30:23]

Then, unless I am missing something, you don't need that anymore. Same thing for
operator=.

[qsr, 2015/02/24 13:32:14]

 As discussed offline, I do, as the return value for my method is a ScopedPyRef,
and the default constructor do not take a reference.

 Also, those 2 methods should have been there since the beginning, since this is
a scoped object over a refcounted python object, so it makes sense to be able to
copy it.

 
   ~ScopedPyRef();
 
   // Releases ownership of the python object contained by this instance.
   PyObject* Release();
 
-  operator PyObject*() const;
+  operator PyObject*() const { return object_; }
+
+  ScopedPyRef& operator=(const ScopedPyRef& other);
 
  private:
   PyObject* object_;
-
-  MOJO_DISALLOW_COPY_AND_ASSIGN(ScopedPyRef);
 };
 
 
 class PythonClosure : public mojo::Closure::Runnable {
  public:
   PythonClosure(PyObject* callable, const mojo::Closure& quit);
   ~PythonClosure();
 
   void Run() const override;
 
@@ skipping 35 matching lines @@
   const mojo::Closure quit_;
 
   MOJO_DISALLOW_COPY_AND_ASSIGN(PythonAsyncWaiter);
 };
 
 }  // namespace python
 }  // namespace mojo
 
 #endif  // MOJO_PUBLIC_PYTHON_SRC_COMMON_H_