Refactor ProfileInfoCacheObserver interface and usage thereof.

chrome/browser/ui/webui/options/manage_profile_handler.cc

Index: chrome/browser/ui/webui/options/manage_profile_handler.cc
===================================================================
--- chrome/browser/ui/webui/options/manage_profile_handler.cc	(revision 115334)
+++ chrome/browser/ui/webui/options/manage_profile_handler.cc	(working copy)
@@ -201,8 +201,8 @@
     ProfileMetrics::LogProfileAvatarSelection(new_icon_index);
     // Updating the profile preference will cause the cache to be updated for
     // this preference.
+    cache.SetIsUsingGAIAPictureOfProfileAtIndex(profile_index, false);
     pref_service->SetInteger(prefs::kProfileAvatarIndex, new_icon_index);

[SteveT, 2011/12/22 02:59:07]

I had to make this switch because SetInteger(kProfileAvatarIndex,...) sends a
notification that triggers updating the shortcut... however if we haven't set
IsUsingGAIAPicture to false before doing that, GetAvatarIcon returns the GAIA
pic rather than the default icon and the shortcut is incorrectly badged.

[sail, 2011/12/22 04:03:11]

But wouldn't it eventually become correct once
SetIsUsingGAIAPictureOfProfileAtIndex() is called?

Does the shortcut manager have trouble if consecutive notifications are being
sent?

[SteveT, 2011/12/22 04:36:24]

Hm... So SetIsUsingGAIAPictureOfProfileAtIndex currently only notifies observers
if it is currently being set to true. I suppose with our new logic, we can have
it notify every time, instead.

That way (assuming we re-swamp the two above lines), the SetInteger call sets
the avatar index in place... and then the SetIsUsingGAIAPicture call triggers
the actual OnProfileAvatarChanged notification.

How does that sound?

[sail, 2011/12/22 04:48:28]

Really? That's weird. Similar to the name functions we should notify when ever
an API call will make an actual change.

[sail, 2011/12/22 04:48:28]

Sounds good. I'm not worried about the actual order of the calls. I was just
worried about the shortcut manager code being very fragile.

-    cache.SetIsUsingGAIAPictureOfProfileAtIndex(profile_index, false);
   }
 
   ProfileMetrics::LogProfileUpdate(profile_file_path);