Some things, like CSRF protection (#361), benefit from the ability to uniformly call helpers before and after a request is processed by the view. In the case of CSRF, we want to confirm that the token matches before processing the view, and make sure the token is set before delivering the response. Unlike WSGI middleware, this sort of wrapping is an integral part of the application.

As I don't have any simple way to test cloud storage, I'm filing this as a bug.

1. StorageObjectWrapper should have a close() method
2. process_media/__init__.py should use the real closing() function from contextlib instead of the current dummy one.

I would also suggest to rename StorageObjectWrapper to include something about CloudFiles.

Not sure when this started happening:

...........................E..................
======================================================================
ERROR: mediagoblin.tests.test_submission.TestSubmission.test_delete
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/cwebber/devel/mediagoblin/eggs/nose-1.1.2-py2.6.egg/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/home/cwebber/devel/mediagoblin/mediagoblin/tests/test_submission.py", line 188, in test_delete
    media = request.db.MediaEntry.find({'title': 'Balanced Goblin'})[0]
  File "build/bdist.linux-i686/egg/pymongo/cursor.py", line 388, in __getitem__
    raise IndexError("no such item for Cursor instance")
IndexError: no such item for Cursor instance