Revert of Media Galleries Partial Deprecation: Remove image metadata & libexif.

Revert of Media Galleries Partial Deprecation: Remove image metadata & libexif. (patchset #6 id:100001 of https://codereview.chromium.org/1773233002/ )

Reason for revert:
Causes build errors on official waterfall.

https://bugs.chromium.org/p/chromium/issues/detail?id=593580

Original issue's description:
> Media Galleries Partial Deprecation: Remove image metadata & libexif.
>
> Removing libexif is a great thing.
>
> It will remove a ~200KB DLL from Windows, Mac, and ChromeOS. It will remove bugs and security issues.
>
> BUG=542912, 577272
>
> Committed: https://crrev.com/f3890c75101ff9b07f48aed992285cc45bf5b4d5
> Cr-Commit-Position: refs/heads/master@{#380306}

TBR=phajdan.jr@chromium.org,rdevlin.cronin@chromium.org,thakis@chromium.org,tommycli@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=542912, 577272
Committed: https://crrev.com/0978e4ebde4a1c07f4e8da88cc9daa7732a63b1a
Cr-Commit-Position: refs/heads/master@{#380852}

[erikchen, 2016-03-11 23:00:42]

Created Revert of Media Galleries Partial Deprecation: Remove image metadata &
libexif.

[commit-bot: I haz the power, 2016-03-11 23:01:08]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1791623002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1791623002/1

[commit-bot: I haz the power, 2016-03-11 23:01:48]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1791623002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1791623002/1

[Nico, 2016-03-11 23:02:23]

please link to bot and paste build error

[Nico, 2016-03-11 23:02:43]

(in the revert cl description, that is)

On Fri, Mar 11, 2016 at 6:02 PM, <thakis@chromium.org> wrote:

> please link to bot and paste build error
>
> https://codereview.chromium.org/1791623002/
>

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[Nico, 2016-03-11 23:02:55]

(next time, i mean)

On Fri, Mar 11, 2016 at 6:02 PM, Nico Weber <thakis@chromium.org> wrote:

> (in the revert cl description, that is)
>
> On Fri, Mar 11, 2016 at 6:02 PM, <thakis@chromium.org> wrote:
>
>> please link to bot and paste build error
>>
>> https://codereview.chromium.org/1791623002/
>>
>
>

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[erikchen, 2016-03-11 23:04:30]

On 2016/03/11 23:02:55, Nico wrote:
> (next time, i mean)
> 
> On Fri, Mar 11, 2016 at 6:02 PM, Nico Weber <mailto:thakis@chromium.org>
wrote:
> 
> > (in the revert cl description, that is)
> >
> > On Fri, Mar 11, 2016 at 6:02 PM, <mailto:thakis@chromium.org> wrote:
> >
> >> please link to bot and paste build error
> >>
> >> https://codereview.chromium.org/1791623002/
> >>
> >
> >
> 
> -- 
> You received this message because you are subscribed to the Google Groups
> "Chromium-reviews" group.
> To unsubscribe from this group and stop receiving emails from it, send an
email
> to mailto:chromium-reviews+unsubscribe@chromium.org.

I linked a crbug whose opening comment is what you asked for. This avoids
spewing compile errors into the git commit message.

[Nico, 2016-03-11 23:06:58]

Oh, indeed. My brain filters out all the URLs in commit descriptions apparently
(since most of them tend to point to rietveld I guess). Carry on, then, sorry
:-)

[commit-bot: I haz the power, 2016-03-11 23:32:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1791623002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1791623002/1

[commit-bot: I haz the power, 2016-03-12 00:02:29]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1791623002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1791623002/1

[commit-bot: I haz the power, 2016-03-12 00:31:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1791623002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1791623002/1

[commit-bot: I haz the power, 2016-03-12 00:57:31]

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

[commit-bot: I haz the power, 2016-03-12 00:57:32]

Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[erikchen, 2016-03-12 02:47:18]

The CQ bit was checked by erikchen@chromium.org

[commit-bot: I haz the power, 2016-03-12 02:47:45]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1791623002/300001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1791623002/300001

[commit-bot: I haz the power, 2016-03-12 04:49:53]

Description was changed from

==========
Revert of Media Galleries Partial Deprecation: Remove image metadata & libexif.
(patchset #6 id:100001 of https://codereview.chromium.org/1773233002/ )

Reason for revert:
Causes build errors on official waterfall.

https://bugs.chromium.org/p/chromium/issues/detail?id=593580

Original issue's description:
> Media Galleries Partial Deprecation: Remove image metadata & libexif.
> 
> Removing libexif is a great thing.
> 
> It will remove a ~200KB DLL from Windows, Mac, and ChromeOS. It will remove
bugs and security issues.
> 
> BUG=542912,577272
> 
> Committed: https://crrev.com/f3890c75101ff9b07f48aed992285cc45bf5b4d5
> Cr-Commit-Position: refs/heads/master@{#380306}

TBR=phajdan.jr@chromium.org,rdevlin.cronin@chromium.org,thakis@chromium.org,t...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=542912,577272
==========

to

==========
Revert of Media Galleries Partial Deprecation: Remove image metadata & libexif.
(patchset #6 id:100001 of https://codereview.chromium.org/1773233002/ )

Reason for revert:
Causes build errors on official waterfall.

https://bugs.chromium.org/p/chromium/issues/detail?id=593580

Original issue's description:
> Media Galleries Partial Deprecation: Remove image metadata & libexif.
> 
> Removing libexif is a great thing.
> 
> It will remove a ~200KB DLL from Windows, Mac, and ChromeOS. It will remove
bugs and security issues.
> 
> BUG=542912,577272
> 
> Committed: https://crrev.com/f3890c75101ff9b07f48aed992285cc45bf5b4d5
> Cr-Commit-Position: refs/heads/master@{#380306}

TBR=phajdan.jr@chromium.org,rdevlin.cronin@chromium.org,thakis@chromium.org,t...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=542912,577272
==========

[commit-bot: I haz the power, 2016-03-12 04:49:54]

Committed patchset #2 (id:300001)

[commit-bot: I haz the power, 2016-03-12 04:51:06]

Description was changed from

==========
Revert of Media Galleries Partial Deprecation: Remove image metadata & libexif.
(patchset #6 id:100001 of https://codereview.chromium.org/1773233002/ )

Reason for revert:
Causes build errors on official waterfall.

https://bugs.chromium.org/p/chromium/issues/detail?id=593580

Original issue's description:
> Media Galleries Partial Deprecation: Remove image metadata & libexif.
> 
> Removing libexif is a great thing.
> 
> It will remove a ~200KB DLL from Windows, Mac, and ChromeOS. It will remove
bugs and security issues.
> 
> BUG=542912,577272
> 
> Committed: https://crrev.com/f3890c75101ff9b07f48aed992285cc45bf5b4d5
> Cr-Commit-Position: refs/heads/master@{#380306}

TBR=phajdan.jr@chromium.org,rdevlin.cronin@chromium.org,thakis@chromium.org,t...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=542912,577272
==========

to

==========
Revert of Media Galleries Partial Deprecation: Remove image metadata & libexif.
(patchset #6 id:100001 of https://codereview.chromium.org/1773233002/ )

Reason for revert:
Causes build errors on official waterfall.

https://bugs.chromium.org/p/chromium/issues/detail?id=593580

Original issue's description:
> Media Galleries Partial Deprecation: Remove image metadata & libexif.
>
> Removing libexif is a great thing.
>
> It will remove a ~200KB DLL from Windows, Mac, and ChromeOS. It will remove
bugs and security issues.
>
> BUG=542912,577272
>
> Committed: https://crrev.com/f3890c75101ff9b07f48aed992285cc45bf5b4d5
> Cr-Commit-Position: refs/heads/master@{#380306}

TBR=phajdan.jr@chromium.org,rdevlin.cronin@chromium.org,thakis@chromium.org,t...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=542912,577272

Committed: https://crrev.com/0978e4ebde4a1c07f4e8da88cc9daa7732a63b1a
Cr-Commit-Position: refs/heads/master@{#380852}
==========

[commit-bot: I haz the power, 2016-03-12 04:51:08]

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