Adding "send to mobile" and "add bookmark" to the action box menu and wiring to their respective ac…

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 ff0579130307dba0c2df59c78101475f616c1c82..53e7351cbffd796609464bda477b03e261c33afa 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
@@ -120,7 +120,8 @@ LocationBarViewMac::LocationBarViewMac(
   }
 
   if (extensions::switch_utils::IsActionBoxEnabled()) {
-    plus_decoration_.reset(new PlusDecoration(this, command_updater, browser_));
+    plus_decoration_.reset(new PlusDecoration(this, command_updater, browser_,
+                                              profile_));

[Scott Hess - ex-Googler, 2012/08/13 18:54:43]

Make this call consistent WRT use of parameter values versus member variables. 
I'm not sure if we have a style guideline, but I just noticed that it was
inconsistent.

Also ... would browser_->profile() work for PlusDecoration instead?

[beaudoin, 2012/08/17 20:04:21]

Good question, I did not know of browser_->profile(). Any idea why
LocationBarViewMac takes both Browser and Profile in this case?

[Scott Hess - ex-Googler, 2012/08/17 20:39:15]

LocationBarView* dates from deepest darkest times.  IOW, I don't know.  It just
looks like lots of stuff would break if browser_->profile() doesn't work.

   }
 
   for (size_t i = 0; i < CONTENT_SETTINGS_NUM_TYPES; ++i) {