Bug 1274057 (Part 1) - DeinterlacingFilter shouldn't duplicate rows when advancing from one pass to the next. r=njn, a=sylvestre
authorSeth Fowler <mark.seth.fowler@gmail.com>
Thu, 26 May 2016 00:19:04 -0700
changeset 333145 a58be6883471d213c256304e58b3633eb6543495
parent 333144 26182f061f00e55e30d7bbb4ff295720a206d5e9
child 333146 18b896da95a7c53a9416d3b37b69507181249f6e
push id6048
push userkmoir@mozilla.com
push dateMon, 06 Jun 2016 19:02:08 +0000
treeherdermozilla-beta@46d72a56c57d [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersnjn, sylvestre
bugs1274057
milestone48.0a2
Bug 1274057 (Part 1) - DeinterlacingFilter shouldn't duplicate rows when advancing from one pass to the next. r=njn, a=sylvestre
image/SurfaceFilters.h
--- a/image/SurfaceFilters.h
+++ b/image/SurfaceFilters.h
@@ -152,19 +152,16 @@ protected:
 
     // Determine which output row the next input row corresponds to.
     bool advancedPass = false;
     uint32_t stride = InterlaceStride(mPass);
     int32_t nextOutputRow = mOutputRow + stride;
     while (nextOutputRow >= InputSize().height) {
       // Copy any remaining rows from the buffer.
       if (!advancedPass) {
-        DuplicateRows(HaeberliOutputUntilRow(mPass, mProgressiveDisplay,
-                                             InputSize(), mOutputRow),
-                      InputSize().height);
         OutputRows(HaeberliOutputUntilRow(mPass, mProgressiveDisplay,
                                           InputSize(), mOutputRow),
                    InputSize().height);
       }
 
       // We finished the current pass; advance to the next one.
       mPass++;
       if (mPass >= 4) {