Docserver: Add support for viewing docs with a codereview patch applied

chrome/common/extensions/docs/server2/patcher.py

Index: chrome/common/extensions/docs/server2/patcher.py
===================================================================
--- chrome/common/extensions/docs/server2/patcher.py	(revision 0)
+++ chrome/common/extensions/docs/server2/patcher.py	(revision 0)
@@ -0,0 +1,21 @@
+# 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.
+
+class Patcher(object):
+  def GetPatchedFiles(self, version=None):
+    ''' Returns (added_files, deleted_files, modified_files).

[not at google - send to devlin, 2013/05/11 20:39:53]

Comment what version= means.

[方觉(Fang Jue), 2013/05/12 03:01:47]

Done.

+    '''
+    raise NotImplementedError()
+
+  def GetVersion(self):
+    ''' Returns patch version. Returns None when nothing is patched by the
+    patcher.
+    '''
+    raise NotImplementedError()
+
+  def Apply(self, paths, file_system, binary, version=None):

[not at google - send to devlin, 2013/05/11 20:39:53]

I can't see file_system being used anywhere

[方觉(Fang Jue), 2013/05/12 03:01:47]

No. It's not actually used because RietveldPatcher doesn't really do patching
against existing files. It just returns patched files' data.
However, other kind of patchers do need this.

+    ''' Apply the patch to added/modified files. Returns Future with patched
+    data. Throws FileNotFoundError if |paths| contains deleted files.

[not at google - send to devlin, 2013/05/11 20:39:53]

comment what version= means.

P.S. I was under the impression that codereview has issues with binary files?
Nevermind.

[方觉(Fang Jue), 2013/05/12 03:01:47]

Yes. And trying to fetch tarball for an issue with binary files results in a
500. But anyway, if it is fixed in the future, it should work with binary files
too.

+    '''
+    raise NotImplementedError()
Property changes on: chrome/common/extensions/docs/server2/patcher.py
___________________________________________________________________
Added: svn:eol-style
   + LF