Notify the ThemeService on extension install and not extension load

Notify the ThemeService on extension install and not extension load

This is a speculative fix for the startup regression introduced by
169875.

Also fix a bug in GtkThemeService where use_gtk_ wasn't being set
to false if a custom theme was installed before the ThemeService
was created.

BUG=163706
TBR=estade@chromium.org

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

[akalin, 2012-12-01 00:29:05]

+asargent for extensions stuff
+estade for themes stuff

[akalin, 2012-12-01 00:29:41]

+simonjam fyi

[asargent_no_longer_on_chrome, 2012-12-01 00:54:56]

LGTM

https://codereview.chromium.org/11421191/diff/1/chrome/browser/extensions/ext...
File chrome/browser/extensions/extension_service.cc (right):

https://codereview.chromium.org/11421191/diff/1/chrome/browser/extensions/ext...
chrome/browser/extensions/extension_service.cc:2119: #endif
Perhaps this should go inside DoPostLoadTasks ?

[akalin, 2012-12-01 01:07:00]

I'll TBR estade so I can land this now and watch to see if it fixes the
regression over the weekend.

https://codereview.chromium.org/11421191/diff/1/chrome/browser/extensions/ext...
File chrome/browser/extensions/extension_service.cc (right):

https://codereview.chromium.org/11421191/diff/1/chrome/browser/extensions/ext...
chrome/browser/extensions/extension_service.cc:2119: #endif
On 2012/12/01 00:54:56, Antony Sargent wrote:
> Perhaps this should go inside DoPostLoadTasks ?

Per our chat, looks like I shouldn't do this!

[commit-bot: I haz the power, 2012-12-01 01:07:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/akalin@chromium.org/11421191/1

[commit-bot: I haz the power, 2012-12-01 01:07:20]

Presubmit check for 11421191-1 failed and returned exit status 1.


Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    chrome/browser/ui/gtk

[commit-bot: I haz the power, 2012-12-01 01:11:37]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/akalin@chromium.org/11421191/1

[commit-bot: I haz the power, 2012-12-01 03:08:22]

Change committed as 170640