Remove the use of OwnedPtrDeleter in ImageDecoder.

Remove the use of OwnedPtrDeleter in ImageDecoder.

It's hard to mechanically replace specializations of OwnedPtrDeleter<T>
to std::unique_ptr equivalents. Therefore, they are converted to
std::unique_ptr manually.

New typedefs named XXXXUniquePtr is introduced so people can always use
std::unique_ptr with a correct deleter.

BUG=617504
Committed: https://crrev.com/b2ba5dac4355691372f643a0af03681769dae2c7
Cr-Commit-Position: refs/heads/master@{#398319}

[Yuta Kitamura, 2016-06-06 09:33:09]

The CQ bit was checked by yutak@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-06-06 09:33:25]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2037373002/1

[Yuta Kitamura, 2016-06-06 09:37:29]

yutak@chromium.org changed reviewers:
+ tzik@chromium.org, urvang@chromium.org

[Yuta Kitamura, 2016-06-06 09:37:29]

yutak@chromium.org changed required reviewers:
+ tzik@chromium.org

[Yuta Kitamura, 2016-06-06 09:37:30]

tzik & urvang: PTAL?
+urvang for platform/image-decoders/ approval.

[tzik, 2016-06-06 09:41:41]

lgtm

[commit-bot: I haz the power, 2016-06-06 10:39:56]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-06-06 10:39:57]

Dry run: This issue passed the CQ dry run.

[urvang, 2016-06-06 15:56:03]

https://codereview.chromium.org/2037373002/diff/1/third_party/WebKit/Source/p...
File third_party/WebKit/Source/platform/image-decoders/ImageDecoder.h (right):

https://codereview.chromium.org/2037373002/diff/1/third_party/WebKit/Source/p...
third_party/WebKit/Source/platform/image-decoders/ImageDecoder.h:64: struct
QCMSProfileDeleter {
64 to 72 can technically be moved to the cpp file, right?

[Yuta Kitamura, 2016-06-07 05:25:00]

The CQ bit was checked by yutak@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-06-07 05:25:09]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2037373002/20001

[Yuta Kitamura, 2016-06-07 05:25:40]

urvang: PTAL again?

https://codereview.chromium.org/2037373002/diff/1/third_party/WebKit/Source/p...
File third_party/WebKit/Source/platform/image-decoders/ImageDecoder.h (right):

https://codereview.chromium.org/2037373002/diff/1/third_party/WebKit/Source/p...
third_party/WebKit/Source/platform/image-decoders/ImageDecoder.h:64: struct
QCMSProfileDeleter {
On 2016/06/06 15:56:03, urvang wrote:
> 64 to 72 can technically be moved to the cpp file, right?

Yeah, fixed in PS2.

[commit-bot: I haz the power, 2016-06-07 09:02:03]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-06-07 09:02:04]

Dry run: This issue passed the CQ dry run.

[urvang, 2016-06-07 17:11:27]

The CQ bit was checked by urvang@chromium.org

[urvang, 2016-06-07 17:11:28]

lgtm

[urvang, 2016-06-07 17:11:28]

The patchset sent to the CQ was uploaded after l-g-t-m from tzik@chromium.org
Link to the patchset: https://codereview.chromium.org/2037373002/#ps20001
(title: "Move QCMSProfileDeleter to .cpp file.")

[commit-bot: I haz the power, 2016-06-07 17:12:01]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2037373002/20001

[commit-bot: I haz the power, 2016-06-07 17:16:30]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-07 17:17:48]

Description was changed from

==========
Remove the use of OwnedPtrDeleter in ImageDecoder.

It's hard to mechanically replace specializations of OwnedPtrDeleter<T>
to std::unique_ptr equivalents. Therefore, they are converted to
std::unique_ptr manually.

New typedefs named XXXXUniquePtr is introduced so people can always use
std::unique_ptr with a correct deleter.

BUG=617504
==========

to

==========
Remove the use of OwnedPtrDeleter in ImageDecoder.

It's hard to mechanically replace specializations of OwnedPtrDeleter<T>
to std::unique_ptr equivalents. Therefore, they are converted to
std::unique_ptr manually.

New typedefs named XXXXUniquePtr is introduced so people can always use
std::unique_ptr with a correct deleter.

BUG=617504

Committed: https://crrev.com/b2ba5dac4355691372f643a0af03681769dae2c7
Cr-Commit-Position: refs/heads/master@{#398319}
==========

[commit-bot: I haz the power, 2016-06-07 17:17:49]

Patchset 2 (id:??) landed as
https://crrev.com/b2ba5dac4355691372f643a0af03681769dae2c7
Cr-Commit-Position: refs/heads/master@{#398319}