• creis's avatar
    Revert of Reland: [qcms] Fix build_output_lut to return correct data for... · 517ca646
    creis authored
    Revert of Reland: [qcms] Fix build_output_lut to return correct data for parametric curves (patchset #14 id:260001 of https://codereview.chromium.org/1862053002/ )
    
    Reason for revert:
    This broke the Windows compile (again).
    https://build.chromium.org/p/chromium/builders/Win/builds/42772
    
    Original issue's description:
    > [qcms] Fix build_output_lut to return correct data for parametric curves
    >
    > build_output_lut() does not invert a parametric gamma curve when
    > computing the output curve data. The effect has not been visible
    > since Chrome only uses the output gamma values from the precache
    > table (which is inverted).
    >
    > Make build_output_lut() return inverted data for parametric gamma
    > curves. Add a test to write the inverted data, and the output of
    > LCMS function DefaultEvalParametricFn, to a file for comparison.
    >
    > For now the size of input and output gamma table for parametric
    > curves is hard-coded to 256; assert this in the test code. See
    > compute_curve_gamma_table_type_parametric for details. Future
    > implementations might return an arbitrary-sized table.
    >
    > BUG=600338
    >
    > Committed: https://crrev.com/a89370950f514a3df77ed84240d4b8e3abbee801
    > Cr-Commit-Position: refs/heads/master@{#389754}
    >
    > Committed: https://crrev.com/3a4a913c1de915ef228a35b4f85603662719c49f
    > Cr-Commit-Position: refs/heads/master@{#389866}
    
    TBR=noel@chromium.org,radu.velea@intel.com
    # Skipping CQ checks because original CL landed less than 1 days ago.
    NOPRESUBMIT=true
    NOTREECHECKS=true
    NOTRY=true
    BUG=600338
    
    Review URL: https://codereview.chromium.org/1920253003
    
    Cr-Commit-Position: refs/heads/master@{#389877}
    517ca646
qcms.gyp 2.77 KB