Reland: [qcms] Fix build_output_lut to return correct data for parametric curves

third_party/qcms/src/transform_util.c

 //  qcms
 //  Copyright (C) 2009 Mozilla Foundation
 //
 // Permission is hereby granted, free of charge, to any person obtaining
 // a copy of this software and associated documentation files (the "Software"),
 // to deal in the Software without restriction, including without limitation
 // the rights to use, copy, modify, merge, publish, distribute, sublicense,
 // and/or sell copies of the Software, and to permit persons to whom the Software
 // is furnished to do so, subject to the following conditions:
 //
@@ skipping 582 matching lines @@
         return output;
 }
 
 void build_output_lut(struct curveType *trc,
                 uint16_t **output_gamma_lut, size_t *output_gamma_lut_length)
 {
         if (trc->type == PARAMETRIC_CURVE_TYPE) {
                 float gamma_table[256];
                 uint16_t i;
                 uint16_t *output = malloc(sizeof(uint16_t)*256);
+                uint16_t *inverted;
+                int inverted_size = 256;
 
                 if (!output) {
                         *output_gamma_lut = NULL;
                         return;
                 }
 
                 compute_curve_gamma_table_type_parametric(gamma_table, trc->parameter, trc->count);
-                *output_gamma_lut_length = 256;
+
                 for(i = 0; i < 256; i++) {
                         output[i] = (uint16_t)(gamma_table[i] * 65535);
                 }
-                *output_gamma_lut = output;
+
+                //XXX: the choice of a minimum of 256 here is not backed by any theory,
+                //     measurement or data, however it is what lcms uses.
+                //     the maximum number we would need is 65535 because that's the
+                //     accuracy used for computing the pre cache table
+                if (inverted_size < 256)
+                        inverted_size = 256;
+
+                inverted = invert_lut(output, 256, inverted_size);
+                if (!inverted)
+                        return;
+                *output_gamma_lut = inverted;
+                *output_gamma_lut_length = inverted_size;
+                free(output);
         } else {
                 if (trc->count == 0) {
                         *output_gamma_lut = build_linear_table(4096);
                         *output_gamma_lut_length = 4096;
                 } else if (trc->count == 1) {
                         float gamma = 1./u8Fixed8Number_to_float(trc->data[0]);
                         *output_gamma_lut = build_pow_table(gamma, 4096);
                         *output_gamma_lut_length = 4096;
                 } else {
                         //XXX: the choice of a minimum of 256 here is not backed by any theory, 
                         //     measurement or data, however it is what lcms uses.
                         *output_gamma_lut_length = trc->count;
                         if (*output_gamma_lut_length < 256)
                                 *output_gamma_lut_length = 256;
 
                         *output_gamma_lut = invert_lut(trc->data, trc->count, *output_gamma_lut_length);
                 }
         }
 
 }