Fix printing PDFs through right click on Mac and Linux. Looks like they both...

ui/base/models/simple_menu_model.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/base/models/simple_menu_model.h"
 
+#include "base/message_loop.h"
 #include "third_party/skia/include/core/SkBitmap.h"
 #include "ui/base/l10n/l10n_util.h"
 
 namespace ui {
 
 const int kSeparatorId = -1;
 
 struct SimpleMenuModel::Item {
   int command_id;
   string16 label;
@@ skipping 27 matching lines @@
 
 void SimpleMenuModel::Delegate::CommandIdHighlighted(int command_id) {
 }
 
 void SimpleMenuModel::Delegate::MenuClosed() {
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // SimpleMenuModel, public:
 
-SimpleMenuModel::SimpleMenuModel(Delegate* delegate) : delegate_(delegate) {
+SimpleMenuModel::SimpleMenuModel(Delegate* delegate)
+    : delegate_(delegate),
+      ALLOW_THIS_IN_INITIALIZER_LIST(method_factory_(this)) {
 }
 
 SimpleMenuModel::~SimpleMenuModel() {
 }
 
 void SimpleMenuModel::AddItem(int command_id, const string16& label) {
   Item item = { command_id, label, SkBitmap(), TYPE_COMMAND, -1, NULL, NULL };
   AppendItem(item);
 }
 
@@ skipping 216 matching lines @@
 void SimpleMenuModel::ActivatedAt(int index) {
   if (delegate_)
     delegate_->ExecuteCommand(GetCommandIdAt(index));
 }
 
 MenuModel* SimpleMenuModel::GetSubmenuModelAt(int index) const {
   return items_.at(FlipIndex(index)).submenu;
 }
 
 void SimpleMenuModel::MenuClosed() {
+  // Due to how menus work on the different platforms, ActivatedAt will be
+  // called after this.  It's more convenient for the delegate to be called
+  // afterwards though, so post a task.
+  MessageLoop::current()->PostTask(
+      FROM_HERE,
+      method_factory_.NewRunnableMethod(&SimpleMenuModel::OnMenuClosed));
+}
+
+void SimpleMenuModel::OnMenuClosed() {
   if (delegate_)
     delegate_->MenuClosed();
 }
 
 int SimpleMenuModel::FlipIndex(int index) const {
   return index;
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // SimpleMenuModel, Private:
@@ skipping 12 matching lines @@
 #ifndef NDEBUG
   if (item.type == TYPE_SEPARATOR) {
     DCHECK_EQ(item.command_id, kSeparatorId);
   } else {
     DCHECK_GE(item.command_id, 0);
   }
 #endif  // NDEBUG
 }
 
 }  // namespace ui