Suffix Chrome's appid on user-level installs

chrome/browser/ui/browser.cc

Index: chrome/browser/ui/browser.cc
diff --git a/chrome/browser/ui/browser.cc b/chrome/browser/ui/browser.cc
index b92d4094b1ed9162f45703387239297573ed48db..50b536c2a8e2c00d841d11687182e5639beba158 100644
--- a/chrome/browser/ui/browser.cc
+++ b/chrome/browser/ui/browser.cc
@@ -534,7 +534,8 @@ void Browser::InitBrowserWindow() {
   // name.  See http://crbug.com/7028.
   ui::win::SetAppIdForWindow(

[grt (UTC plus 2), 2012/06/15 03:03:03]

the "developing a metro style enabled desktop browser" doc says "When running in
the Metro style experience, this AppUserModelID is immutable by the running
browser process." and "Metro style enabled desktop browsers should not set an
AppUserModelID at runtime when participating in the Metro style user experience"
Which makes me wonder if it's bad to ever do this when in metro mode.

[gab, 2012/06/15 19:01:04]

Would you be ok if I just log a bug for this for now? I'd rather not get
involved too far with this patch given the critical timing. (i.e. the fact that
its suffixed or not doesn't affect the comment made here)

[gab, 2012/06/15 20:28:24]

https://chromiumcodereview.appspot.com/10557017/

       is_app() && !is_type_panel() ?
-      ShellIntegration::GetAppId(UTF8ToWide(app_name_), profile_->GetPath()) :
+      ShellIntegration::GetProfileAppId(UTF8ToWide(app_name_),
+                                        profile_->GetPath()) :
       ShellIntegration::GetChromiumAppId(profile_->GetPath()),
       window()->GetNativeWindow());