[mac] App shims

apps/app_shim/app_shim_host.h

+// Copyright 2013 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.
+
+#ifndef CHROME_BROWSER_WEB_APPLICATIONS_APP_SHIM_HOST_H_
+#define CHROME_BROWSER_WEB_APPLICATIONS_APP_SHIM_HOST_H_
+
+#include <string>
+
+#include "base/threading/non_thread_safe.h"
+#include "content/public/browser/notification_observer.h"
+#include "content/public/browser/notification_registrar.h"
+#include "ipc/ipc_listener.h"
+#include "ipc/ipc_sender.h"
+
+class Profile;
+
+namespace IPC {
+struct ChannelHandle;
+class ChannelProxy;
+class Message;
+}  // namespace IPC
+
+// This is the counterpart to AppShimController in
+// chrome/app/chrome_main_app_mode_mac.mm. The AppShimHost owns itself, and is
+// destroyed when the app it corresponds to is closed or when the channel
+// connected to the app shim is closed.

[koz (OOO until 15th September), 2013/03/14 23:51:05]

Perhaps add something explaining that this services IPCs from AppShimController.
(Site note, AppShimController <-> AppShimHost seems asymmetrical, don't we
normally name things X <-> XHost?)

+class AppShimHost : public IPC::Listener,
+                    public IPC::Sender,
+                    public content::NotificationObserver,
+                    public base::NonThreadSafe {
+ public:
+  AppShimHost();
+  virtual ~AppShimHost();
+
+  // Creates a new server-side IPC channel at |handle|, which should contain a
+  // file descriptor of a channel created by an IPC::ChannelFactory, and begins
+  // listening for messages on it.
+  void ServeChannel(const IPC::ChannelHandle& handle);
+
+  const std::string& app_id() { return app_id_; }

[palmer, 2013/03/14 19:05:04]

The methods themselves, as well as their return values, could be const?

[jeremya, 2013/03/14 23:33:01]

Done.

+  const Profile* profile() { return profile_; }
+
+ protected:
+  // Used internally; virtual so they can be mocked for testing.
+  virtual Profile* FetchProfileForDirectory(const std::string& profile_dir);
+  virtual bool LaunchApp(Profile* profile, const std::string& app_id);
+
+  // IPC::Listener implementation.
+  virtual bool OnMessageReceived(const IPC::Message& message) OVERRIDE;
+
+  // IPC::Sender implementation.
+  virtual bool Send(IPC::Message* message) OVERRIDE;
+
+ private:
+  // The app shim process is requesting that an app be launched. Once it has
+  // done so the profile and app id are stored, and all future messages from

[palmer, 2013/03/14 19:05:04]

NIT: I would say "|profile|" and "|app_id|".

[jeremya, 2013/03/14 23:33:01]

Done.

+  // the app shim relate to the app it launched. It is an error for the app
+  // shim to send multiple launch messages.
+  void OnLaunchApp(std::string profile, std::string app_id);
+
+  bool LaunchAppImpl(const std::string& profile_dir, const std::string& app_id);
+
+  // When the app closes, a signal is sent (by way of closing the channel) to

[palmer, 2013/03/14 19:05:04]

Not sure what this means.

1. Is the signal sent by the Observe method? The passive voice ("is sent")
obscures the actor of the sending.

2. "by way of closing the channel" — does that mean that closing the channel
*is* the signal?

[jeremya, 2013/03/14 23:33:01]

Made a better comment.

+  // the app shim to tell it to quit.
+  virtual void Observe(int type,
+                       const content::NotificationSource& source,
+                       const content::NotificationDetails& details) OVERRIDE;
+
+  // Closes the channel and destroys the AppShimHost.
+  void Close();
+
+  IPC::ChannelProxy* channel_;
+  std::string app_id_;
+  Profile* profile_;
+  content::NotificationRegistrar registrar_;
+};
+
+#endif  // CHROME_BROWSER_WEB_APPLICATIONS_APP_SHIM_HOST_H_