Reland: Chrome side work for allowing a user to revert to their previous theme without closing ...

Reland: Chrome side work for allowing a user to revert to their previous theme without closing infobar

BUG=257724
TEST=ThemeServiceTest.*
R=yoz
TBR=ben (for ProfileResetterTest and InstantExtendedTest)

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=217652

[pkotwicz, 2013-08-09 19:43:21]

yoz@, can you please take another look? This CL is exactly the same as
https://codereview.chromium.org/19462009/

akuegel@ was nice enough to land a fix for the unittest crash in
https://codereview.chromium.org/22578004/
- I have also checked independantly that this CL will no longer cause a crash on
unittests for GTK because of the message loop

[Yoyo Zhou, 2013-08-09 20:12:40]

Still LGTM
On Aug 9, 2013 12:43 PM, <pkotwicz@chromium.org> wrote:

> Reviewers: Yoyo Zhou,
>
> Message:
> yoz@, can you please take another look? This CL is exactly the same as
>
https://codereview.chromium.**org/19462009/<https://codereview.chromium.org/1...
>
> akuegel@ was nice enough to land a fix for the unittest crash in
>
https://codereview.chromium.**org/22578004/<https://codereview.chromium.org/2...
> - I have also checked independantly that this CL will no longer cause a
> crash on
> unittests for GTK because of the message loop
>
> Description:
> Reland: Chrome side work for allowing a user to revert to their previous
> theme
> without closing infobar
>
> BUG=257724
> TEST=ThemeServiceTest.*
> R=yoz
> TBR=ben (for ProfileResetterTest and InstantExtendedTest)
>
> Please review this at
https://codereview.chromium.**org/22755003/<https://codereview.chromium.org/2...
>
> SVN Base:
svn://svn.chromium.org/chrome/**trunk/src<http://svn.chromium.org/chrome/trunk/src>
>
> Affected files:
>   M chrome/browser/extensions/**extension_install_ui_**browsertest.cc
>   M chrome/browser/extensions/**extension_service.cc
>   M chrome/browser/extensions/**extension_service_unittest.cc
>   M chrome/browser/profile_**resetter/profile_resetter_**unittest.cc
>   M chrome/browser/themes/theme_**service.h
>   M chrome/browser/themes/theme_**service.cc
>   M chrome/browser/themes/theme_**service_unittest.cc
>   M chrome/browser/themes/theme_**syncable_service.cc
>   M chrome/browser/ui/search/**instant_extended_interactive_**uitest.cc
>   A chrome/test/data/extensions/**theme_minimal/manifest.json
>
>
>

[commit-bot: I haz the power, 2013-08-14 18:20:31]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pkotwicz@chromium.org/22755003/4001

[commit-bot: I haz the power, 2013-08-14 21:33:35]

Change committed as 217652