[Hotword, Extensions] Removing EXTERNAL_COMPONENT from IsPolicyLocation, adding a function to check…

chrome/browser/background/background_contents_service.cc

Index: chrome/browser/background/background_contents_service.cc
diff --git a/chrome/browser/background/background_contents_service.cc b/chrome/browser/background/background_contents_service.cc
index dd03caac4c2fb6400b3178ac84ef290d08af3722..89cb69dee4d05bef266089b4303c6041cf7bd26a 100644
--- a/chrome/browser/background/background_contents_service.cc
+++ b/chrome/browser/background/background_contents_service.cc
@@ -433,6 +433,7 @@ void BackgroundContentsService::Observe(
         break;
 
       const bool force_installed =
+          extension->location() == extensions::Manifest::EXTERNAL_COMPONENT ||

[Marijn Kruisselbrink, 2014/02/06 23:37:09]

I'm not entirely sure if this should be included here, and if it should be than
I can't think of a reason why regular COMPONENT extensions shouldn't also
auto-restart...

[rpetterson, 2014/02/07 00:23:51]

Seems reasonable. I've updated it to include both types of component extensions.

           extensions::Manifest::IsPolicyLocation(extension->location());
       if (!force_installed) {
         ShowBalloon(extension, profile);