ARC: Add a link to "Manage supported links" in the Chrome app info dialog.

chrome/browser/ui/views/apps/app_info_dialog/arc_app_info_links_panel.h

Index: chrome/browser/ui/views/apps/app_info_dialog/arc_app_info_links_panel.h
diff --git a/chrome/browser/ui/views/apps/app_info_dialog/arc_app_info_links_panel.h b/chrome/browser/ui/views/apps/app_info_dialog/arc_app_info_links_panel.h
new file mode 100644
index 0000000000000000000000000000000000000000..0cc831c02d469fca1bf69170495d0456906404ae
--- /dev/null
+++ b/chrome/browser/ui/views/apps/app_info_dialog/arc_app_info_links_panel.h
@@ -0,0 +1,53 @@
+// 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_BROWSER_UI_VIEWS_APPS_APP_INFO_DIALOG_ARC_APP_INFO_LINKS_PANEL_H_
+#define CHROME_BROWSER_UI_VIEWS_APPS_APP_INFO_DIALOG_ARC_APP_INFO_LINKS_PANEL_H_
+
+#include <stdint.h>
+
+#include <string>
+
+#include "base/macros.h"
+#include "chrome/browser/ui/app_list/arc/arc_app_list_prefs.h"
+#include "chrome/browser/ui/views/apps/app_info_dialog/app_info_panel.h"
+#include "ui/views/controls/link_listener.h"
+
+class Profile;
+
+namespace extensions {
+class Extension;
+}  // namespace extensions

[Peter Kasting, 2016/06/23 07:07:20]

Nit: Personally I find these // namespace comments clutter rather than helpful
when used on a forward-decl list like this as the list is so short no one is
going to confuse which namespace these are in.

[mtomasz, 2016/06/23 08:51:56]

No strong feelings. Removed. Done.

+
+namespace views {
+class Link;
+}  // namespace views
+
+// Shows a link to get to managing supported links activity on ARC side.
+class ArcAppInfoLinksPanel : public AppInfoPanel,
+                             public views::LinkListener,
+                             public ArcAppListPrefs::Observer {
+ public:
+  ArcAppInfoLinksPanel(Profile* profile, const extensions::Extension* app);
+  ~ArcAppInfoLinksPanel() override;
+
+ private:
+  // views::LinkListener overrides.

[Peter Kasting, 2016/06/23 07:07:20]

Nit: Just "views::LinkListener:" is sufficient (views code can't really agree on
what convention to use for this so I just encourage the shortest one)

[mtomasz, 2016/06/23 08:51:56]

Done.

+  void LinkClicked(views::Link* source, int event_flags) override;
+
+  // ArcAppListPrefs::Observer overrides.
+  void OnAppReadyChanged(const std::string& app_id, bool ready) override;
+  void OnAppRemoved(const std::string& app_id) override;
+  void OnAppRegistered(const std::string& app_id,
+                       const ArcAppListPrefs::AppInfo& app_info) override;
+
+  // Updates the link view.

[Peter Kasting, 2016/06/23 07:07:20]

Nit: This comment doesn't really add anything to the function name, so I'd
remove it, unless you want to expand it with more about what the function does
or when it should be called.

[mtomasz, 2016/06/23 08:51:56]

Removed. Done.

+  void UpdateLink(bool enabled);
+
+  views::Link* manage_link_;
+
+  DISALLOW_COPY_AND_ASSIGN(ArcAppInfoLinksPanel);
+};
+
+#endif  // CHROME_BROWSER_UI_VIEWS_APPS_APP_INFO_DIALOG_ARC_APP_INFO_LINKS_PANEL_H_