Last modified: 2014-11-14 11:58:23 UTC
Not sure why cite ids are not identical anymore when double rt-diff is applied on a page .. This has been causing spurious rt diffs in the last 2 (at least) rounds of rt-testing.
similar to this? 7b003c9b1ed1290bf71759ecd16be699c9fa05a7
Doesn't seem to be. roundtrip-test.js uses processTopLevelDoc both times.
Check: http://parsoid.wmflabs.org:8001/result/51c71eb04fd3111e04702c3729118fd3cfd611a9/hiwiki/%E0%A4%AE%E0%A4%BE%E0%A4%A4%E0%A5%83%E0%A4%B5%E0%A4%82%E0%A4%B6_%E0%A4%B8%E0%A4%AE%E0%A5%82%E0%A4%B9_%E0%A4%B5%E0%A4%BE%E0%A4%8F So, the 2nd parse is showing ref id 40. Now, check this: http://parsoid.wmflabs.org/_rt/hiwiki/%E0%A4%AE%E0%A4%BE%E0%A4%A4%E0%A5%83%E0%A4%B5%E0%A4%82%E0%A4%B6_%E0%A4%B8%E0%A4%AE%E0%A5%82%E0%A4%B9_%E0%A4%B5%E0%A4%BE%E0%A4%8F There are only 5 refs. So, it shouldn't be getting an id of 40 at all. So, that is prettttty odd!
With recent cite fixes, haven't seen this much .. But leaving this open just in case it pops up its head again.
This seems to have been fixed by the fix for bug 67237
We are seeing a lot of these spurious diffs again in recent roundtrip test runs.