Fix disappearing page actions bug.

chrome/browser/ui/views/location_bar/location_bar_view.h

Index: chrome/browser/ui/views/location_bar/location_bar_view.h
diff --git a/chrome/browser/ui/views/location_bar/location_bar_view.h b/chrome/browser/ui/views/location_bar/location_bar_view.h
index c3eff2c908100644ca2ed3ed230781f0d179494c..120dff7bdd223e0a823943ba8a9f3dfb1f2da99a 100644
--- a/chrome/browser/ui/views/location_bar/location_bar_view.h
+++ b/chrome/browser/ui/views/location_bar/location_bar_view.h
@@ -301,7 +301,8 @@ class LocationBarView : public LocationBar,
   // of at least one of the views in |content_setting_views_| changed.
   bool RefreshContentSettingViews();
 
-  // Deletes all page action views that we have created.
+  // Clears |page_action_views_| and removes the elements from the view
+  // hierarchy.
   void DeletePageActionViews();
 
   // Updates the views for the Page Actions, to reflect state changes for
@@ -309,6 +310,10 @@ class LocationBarView : public LocationBar,
   // changed, or PageActionWithBadgeView were created/destroyed.
   bool RefreshPageActionViews();
 
+  // Whether the page actions represented by |page_action_views_| differ
+  // in ordering or value from |page_actions|.
+  bool PageActionsDiffer(const std::vector<ExtensionAction*>& page_actions);

[Peter Kasting, 2014/12/23 00:20:05]

Nit: Is there a typedef for this type?

[erikchen, 2014/12/23 00:55:03]

Yes. Changed to use the typedef.

+
   // Updates the view for the zoom icon based on the current tab's zoom. Returns
   // true if the visibility of the view changed.
   bool RefreshZoomView();
@@ -473,9 +478,6 @@ class LocationBarView : public LocationBar,
   // The manage passwords icon.
   ManagePasswordsIconView* manage_passwords_icon_view_;
 
-  // The current page actions.
-  PageActions page_actions_;
-
   // The page action icon views.
   PageActionViews page_action_views_;