Add the Windows.IsPinnedToTaskbar metric.

chrome/utility/shell_handler_impl_win.h

+// Copyright 2016 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_UTILITY_SHELL_HANDLER_IMPL_WIN_H_
+#define CHROME_UTILITY_SHELL_HANDLER_IMPL_WIN_H_
+
+#include "base/macros.h"
+#include "chrome/common/shell_handler_win.mojom.h"
+#include "mojo/public/cpp/bindings/strong_binding.h"
+
+// Implements the ShellHandler mojo interface.
+class ShellHandlerImpl : public mojom::ShellHandler {

[gab, 2016/06/28 20:59:18]

How is this related to "class ShellHandler"? I'm not clear on the relationship
between the two (why does one use mojo and not the other?) and therefore find it
weird that this is named as the "Impl" of the other?

[Patrick Monette, 2016/06/28 23:22:25]

ShellHandler is the deprecated IPC version that needs to be migrated to Mojo. It
won't exist anymore once the migration is finished. I agree the names are
confusing for now.

The "Impl" is a just a convention for classes that implement Mojo interfaces.

[gab, 2016/06/30 15:28:11]

Got it, can you rename the old one then? IPCShellHandler or something with a
TODO to migrate it?

+ public:
+  static void Create(mojom::ShellHandlerRequest request);
+
+ private:
+  explicit ShellHandlerImpl(mojom::ShellHandlerRequest request);
+  ~ShellHandlerImpl() override;
+
+  // mojom::ShellHandler:
+  void IsPinnedToTaskbar(const IsPinnedToTaskbarCallback& callback) override;
+
+  mojo::StrongBinding<ShellHandler> binding_;
+
+  DISALLOW_COPY_AND_ASSIGN(ShellHandlerImpl);
+};
+
+#endif  // CHROME_UTILITY_SHELL_HANDLER_IMPL_WIN_H_