Adding decorations(icons) in right order to omnibox

chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.mm

Index: chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.mm
diff --git a/chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.mm b/chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.mm
index 441e8f02277ddfd74a9c3a88417bc646ecf3d262..87c9b7c1b848e727a98880b097a6333392130f4a 100644
--- a/chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.mm
+++ b/chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.mm
@@ -660,7 +660,9 @@ void LocationBarViewMac::Layout() {
   for (size_t i = 0; i < page_action_decorations_.size(); ++i) {
     [cell addRightDecoration:page_action_decorations_[i]];
   }
-  for (size_t i = 0; i < content_setting_decorations_.size(); ++i) {
+
+  // Reverse iterating through |content_setting_decoration_| as in aura

[Ilya Sherman, 2013/09/14 04:01:32]

nit: "Iterate through |content_setting_decorations_| in reverse order so that
the order in which the decorations are drawn matches the Views code."

[npentrel, 2013/09/17 11:13:22]

Done.

+  for (int i = content_setting_decorations_.size()-1; i >= 0; --i) {

[Ilya Sherman, 2013/09/14 04:01:32]

nit: Please use rbegin() and rend() for iterating through the vector.  Or, if
you really prefer to keep the index variable, add spaces around the minus sign
in "content_setting_decorations_.size()-1".

[npentrel, 2013/09/17 11:13:22]

Done.

     [cell addRightDecoration:content_setting_decorations_[i]];
   }