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

mojo/public/python/src/common.h

Index: mojo/public/python/src/common.h
diff --git a/mojo/public/python/src/common.h b/mojo/public/python/src/common.h
index 4eae72e51c4be1d7dd4768bd4f33d2f3cf6cc6ab..2ada2a411787bbea4d77d4c8571e195d23b101cc 100644
--- a/mojo/public/python/src/common.h
+++ b/mojo/public/python/src/common.h
@@ -37,18 +37,19 @@ 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);
 };