Revert of Use consistent types for ICC profiles

Revert of Use consistent types for ICC profiles (patchset #5 id:80001 of https://codereview.chromium.org/2605743002/ )

Reason for revert:
Rebaselining not possible due to discrepancy between test outputs produced by continuous builders vs. try jobs. See crbug.com/677822 for details.

Speculatively reverting this CL as it has to do with ICC profiles and I am mostly seeing differences of color shades in pixel diffs.

Original issue's description:
> Use consistent types for ICC profiles
>
> In Blink we convert from a gfx::ICCProfile to a raw vector of data.
> Stop doing this, and, instead, pass the gfx::ICCProfile (or its gfx::ColorSpace
> or SkColorSpace, as appropriate).
>
> Merge several locations where we store ICC profiles for testing
> into ui/gfx/test/icc_profiles.
>
> BUG=634102
> TBR=esprehn
>
> Committed: https://crrev.com/0b283e725b6489c7be426f1958419d19e8b348b7
> Cr-Commit-Position: refs/heads/master@{#441004}

TBR=chrishtr@chromium.org,avi@chromium.org,esprehn@chromium.org,ccameron@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=634102, 677822
Committed: https://crrev.com/8d0ed9b51268dc2265f3fdf5c9a3b330d9a8a529
Cr-Commit-Position: refs/heads/master@{#441078}

[engedy, 2017-01-02 16:59:57]

The CQ bit was checked by engedy@chromium.org

[engedy, 2017-01-02 16:59:58]

Created Revert of Use consistent types for ICC profiles

[commit-bot: I haz the power, 2017-01-02 17:00:05]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[engedy, 2017-01-02 17:00:25]

Description was changed from

==========
Revert of Use consistent types for ICC profiles (patchset #5 id:80001 of
https://codereview.chromium.org/2605743002/ )

Reason for revert:
Rebaselining not possible due to discrepancy between test outputs produced by
continuous builders vs. try jobs. See crbug.com/677822 for details.

Speculatively reverting this CL as it has to do with ICC profiles and I am
mostly seeing differences of color shades in pixel diffs.



Original issue's description:
> Use consistent types for ICC profiles
> 
> In Blink we convert from a gfx::ICCProfile to a raw vector of data.
> Stop doing this, and, instead, pass the gfx::ICCProfile (or its
gfx::ColorSpace
> or SkColorSpace, as appropriate).
> 
> Merge several locations where we store ICC profiles for testing
> into ui/gfx/test/icc_profiles.
> 
> BUG=634102
> TBR=esprehn
> 
> Committed: https://crrev.com/0b283e725b6489c7be426f1958419d19e8b348b7
> Cr-Commit-Position: refs/heads/master@{#441004}

TBR=chrishtr@chromium.org,avi@chromium.org,esprehn@chromium.org,ccameron@chro...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=634102
==========

to

==========
Revert of Use consistent types for ICC profiles (patchset #5 id:80001 of
https://codereview.chromium.org/2605743002/ )

Reason for revert:
Rebaselining not possible due to discrepancy between test outputs produced by
continuous builders vs. try jobs. See crbug.com/677822 for details.

Speculatively reverting this CL as it has to do with ICC profiles and I am
mostly seeing differences of color shades in pixel diffs.



Original issue's description:
> Use consistent types for ICC profiles
> 
> In Blink we convert from a gfx::ICCProfile to a raw vector of data.
> Stop doing this, and, instead, pass the gfx::ICCProfile (or its
gfx::ColorSpace
> or SkColorSpace, as appropriate).
> 
> Merge several locations where we store ICC profiles for testing
> into ui/gfx/test/icc_profiles.
> 
> BUG=634102
> TBR=esprehn
> 
> Committed: https://crrev.com/0b283e725b6489c7be426f1958419d19e8b348b7
> Cr-Commit-Position: refs/heads/master@{#441004}

TBR=chrishtr@chromium.org,avi@chromium.org,esprehn@chromium.org,ccameron@chro...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=634102
BUG=677822
==========

[engedy, 2017-01-02 17:00:38]

The CQ bit was unchecked by engedy@chromium.org

[engedy, 2017-01-02 17:00:40]

The CQ bit was checked by engedy@chromium.org

[commit-bot: I haz the power, 2017-01-02 17:00:45]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[engedy, 2017-01-02 17:50:36]

The CQ bit was unchecked by engedy@chromium.org

[engedy, 2017-01-02 17:50:44]

Description was changed from

==========
Revert of Use consistent types for ICC profiles (patchset #5 id:80001 of
https://codereview.chromium.org/2605743002/ )

Reason for revert:
Rebaselining not possible due to discrepancy between test outputs produced by
continuous builders vs. try jobs. See crbug.com/677822 for details.

Speculatively reverting this CL as it has to do with ICC profiles and I am
mostly seeing differences of color shades in pixel diffs.



Original issue's description:
> Use consistent types for ICC profiles
> 
> In Blink we convert from a gfx::ICCProfile to a raw vector of data.
> Stop doing this, and, instead, pass the gfx::ICCProfile (or its
gfx::ColorSpace
> or SkColorSpace, as appropriate).
> 
> Merge several locations where we store ICC profiles for testing
> into ui/gfx/test/icc_profiles.
> 
> BUG=634102
> TBR=esprehn
> 
> Committed: https://crrev.com/0b283e725b6489c7be426f1958419d19e8b348b7
> Cr-Commit-Position: refs/heads/master@{#441004}

TBR=chrishtr@chromium.org,avi@chromium.org,esprehn@chromium.org,ccameron@chro...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=634102
BUG=677822
==========

to

==========
Revert of Use consistent types for ICC profiles (patchset #5 id:80001 of
https://codereview.chromium.org/2605743002/ )

Reason for revert:
Rebaselining not possible due to discrepancy between test outputs produced by
continuous builders vs. try jobs. See crbug.com/677822 for details.

Speculatively reverting this CL as it has to do with ICC profiles and I am
mostly seeing differences of color shades in pixel diffs.



Original issue's description:
> Use consistent types for ICC profiles
> 
> In Blink we convert from a gfx::ICCProfile to a raw vector of data.
> Stop doing this, and, instead, pass the gfx::ICCProfile (or its
gfx::ColorSpace
> or SkColorSpace, as appropriate).
> 
> Merge several locations where we store ICC profiles for testing
> into ui/gfx/test/icc_profiles.
> 
> BUG=634102
> TBR=esprehn
> 
> Committed: https://crrev.com/0b283e725b6489c7be426f1958419d19e8b348b7
> Cr-Commit-Position: refs/heads/master@{#441004}

TBR=chrishtr@chromium.org,avi@chromium.org,esprehn@chromium.org,ccameron@chro...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=634102,677822
==========

[engedy, 2017-01-02 17:50:52]

The CQ bit was checked by engedy@chromium.org

[commit-bot: I haz the power, 2017-01-02 17:51:06]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-02 20:05:51]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1483379452934370, "parent_rev":
"de339d883973b7e334acce4c3cee3f8a21d463e8", "commit_rev":
"b03f3626c4e750aeed5323617472e34b2209abf1"}

[commit-bot: I haz the power, 2017-01-02 20:06:23]

Description was changed from

==========
Revert of Use consistent types for ICC profiles (patchset #5 id:80001 of
https://codereview.chromium.org/2605743002/ )

Reason for revert:
Rebaselining not possible due to discrepancy between test outputs produced by
continuous builders vs. try jobs. See crbug.com/677822 for details.

Speculatively reverting this CL as it has to do with ICC profiles and I am
mostly seeing differences of color shades in pixel diffs.



Original issue's description:
> Use consistent types for ICC profiles
> 
> In Blink we convert from a gfx::ICCProfile to a raw vector of data.
> Stop doing this, and, instead, pass the gfx::ICCProfile (or its
gfx::ColorSpace
> or SkColorSpace, as appropriate).
> 
> Merge several locations where we store ICC profiles for testing
> into ui/gfx/test/icc_profiles.
> 
> BUG=634102
> TBR=esprehn
> 
> Committed: https://crrev.com/0b283e725b6489c7be426f1958419d19e8b348b7
> Cr-Commit-Position: refs/heads/master@{#441004}

TBR=chrishtr@chromium.org,avi@chromium.org,esprehn@chromium.org,ccameron@chro...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=634102,677822
==========

to

==========
Revert of Use consistent types for ICC profiles (patchset #5 id:80001 of
https://codereview.chromium.org/2605743002/ )

Reason for revert:
Rebaselining not possible due to discrepancy between test outputs produced by
continuous builders vs. try jobs. See crbug.com/677822 for details.

Speculatively reverting this CL as it has to do with ICC profiles and I am
mostly seeing differences of color shades in pixel diffs.



Original issue's description:
> Use consistent types for ICC profiles
> 
> In Blink we convert from a gfx::ICCProfile to a raw vector of data.
> Stop doing this, and, instead, pass the gfx::ICCProfile (or its
gfx::ColorSpace
> or SkColorSpace, as appropriate).
> 
> Merge several locations where we store ICC profiles for testing
> into ui/gfx/test/icc_profiles.
> 
> BUG=634102
> TBR=esprehn
> 
> Committed: https://crrev.com/0b283e725b6489c7be426f1958419d19e8b348b7
> Cr-Commit-Position: refs/heads/master@{#441004}

TBR=chrishtr@chromium.org,avi@chromium.org,esprehn@chromium.org,ccameron@chro...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=634102,677822

Review-Url: https://codereview.chromium.org/2609093002
==========

[commit-bot: I haz the power, 2017-01-02 20:06:24]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2017-01-02 20:08:55]

Description was changed from

==========
Revert of Use consistent types for ICC profiles (patchset #5 id:80001 of
https://codereview.chromium.org/2605743002/ )

Reason for revert:
Rebaselining not possible due to discrepancy between test outputs produced by
continuous builders vs. try jobs. See crbug.com/677822 for details.

Speculatively reverting this CL as it has to do with ICC profiles and I am
mostly seeing differences of color shades in pixel diffs.



Original issue's description:
> Use consistent types for ICC profiles
> 
> In Blink we convert from a gfx::ICCProfile to a raw vector of data.
> Stop doing this, and, instead, pass the gfx::ICCProfile (or its
gfx::ColorSpace
> or SkColorSpace, as appropriate).
> 
> Merge several locations where we store ICC profiles for testing
> into ui/gfx/test/icc_profiles.
> 
> BUG=634102
> TBR=esprehn
> 
> Committed: https://crrev.com/0b283e725b6489c7be426f1958419d19e8b348b7
> Cr-Commit-Position: refs/heads/master@{#441004}

TBR=chrishtr@chromium.org,avi@chromium.org,esprehn@chromium.org,ccameron@chro...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=634102,677822

Review-Url: https://codereview.chromium.org/2609093002
==========

to

==========
Revert of Use consistent types for ICC profiles (patchset #5 id:80001 of
https://codereview.chromium.org/2605743002/ )

Reason for revert:
Rebaselining not possible due to discrepancy between test outputs produced by
continuous builders vs. try jobs. See crbug.com/677822 for details.

Speculatively reverting this CL as it has to do with ICC profiles and I am
mostly seeing differences of color shades in pixel diffs.

Original issue's description:
> Use consistent types for ICC profiles
>
> In Blink we convert from a gfx::ICCProfile to a raw vector of data.
> Stop doing this, and, instead, pass the gfx::ICCProfile (or its
gfx::ColorSpace
> or SkColorSpace, as appropriate).
>
> Merge several locations where we store ICC profiles for testing
> into ui/gfx/test/icc_profiles.
>
> BUG=634102
> TBR=esprehn
>
> Committed: https://crrev.com/0b283e725b6489c7be426f1958419d19e8b348b7
> Cr-Commit-Position: refs/heads/master@{#441004}

TBR=chrishtr@chromium.org,avi@chromium.org,esprehn@chromium.org,ccameron@chro...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=634102,677822

Committed: https://crrev.com/8d0ed9b51268dc2265f3fdf5c9a3b330d9a8a529
Cr-Commit-Position: refs/heads/master@{#441078}
==========

[commit-bot: I haz the power, 2017-01-02 20:08:56]

Patchset 1 (id:??) landed as
https://crrev.com/8d0ed9b51268dc2265f3fdf5c9a3b330d9a8a529
Cr-Commit-Position: refs/heads/master@{#441078}