LGTM


http://codereview.appspot.com/88149/diff/1/2
File src/com/google/caja/plugin/FileSystemEnvironment.java (right):

http://codereview.appspot.com/88149/diff/1/2#newcode70
Line 70: String uristr = self.relativize(ref.getUri()).toString();
Ok, I see now how you worked around the fact that the cajoler is
pre-absolutizing the fragment, and I now see how it should not do that.
Ziqing Mao also just pointed out to me that ExternalReference *already*
contains a pair: the source FilePosition and the target URI. Check. :)
So ok, we should just fix the cajoler attribute rewriting code.

http://codereview.appspot.com/88149

Reply via email to