Comment 6 for bug 1300865

Revision history for this message
Mark Harmer (drivehappy) wrote :

@jazzynico
Thanks, there seems to be an issue when loading these in Windows, particularly in ColorProfile::set. This code looked complex, as I don't know the URI stuff, so my patch only dealt with the worst case if the file could not be loaded. As a local hack to get it to load in Windows I had added the following after fullpath was initialized in ColorProfile::set:

 if (fullpath.size() == 0) {
  fullpath = this->href;
 }

But I don't feel this is a true fix since I don't know what the URI handling code is really doing. As it begins to complicate the reproduce and verification of the above patch, I understand if it's not acceptable. This needs to be fixed eventually to actually load profiles, but maybe it's better to be tracked as another issue?

Even after loading the profile, the color selectors act strangely, I believe that behavior is the same as bug 632023.