Upstream the Media Router's Issue class.

chrome/browser/media/router/issue_manager.h

+// Copyright 2015 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_MEDIA_ROUTER_ISSUE_MANAGER_H_
+#define CHROME_BROWSER_MEDIA_ROUTER_ISSUE_MANAGER_H_
+
+#include <list>
+
+#include "base/containers/hash_tables.h"
+#include "base/observer_list.h"
+#include "base/threading/thread_checker.h"
+#include "chrome/browser/media/router/issue.h"
+#include "chrome/browser/media/router/issues_observer.h"
+
+namespace media_router {
+
+// IssueManager keeps track of current issues related to casting
+// connectivity and quality.
+// TODO(apacible): Determine what other issues will be handled here.
+class IssueManager {
+ public:
+  IssueManager();
+  ~IssueManager();
+
+  // Adds an issue.
+  // |issue|: Issue to be added. Must have unique ID.
+  void AddIssue(const Issue& issue);
+
+  // Removes an issue when user has noted it is resolved.
+  // |issue_id|: Issue::IssueId of the issue to be removed.
+  void ClearIssue(const Issue::IssueId& issue_id);
+
+  // Gets the number of unresolved issues.
+  size_t GetIssueCount() const;
+
+  // Removes all unresolved issues.
+  void ClearAllIssues();
+
+  // Removes all unresolved global issues.
+  void ClearGlobalIssues();
+
+  // Removes all unresolved issues with RouteId.
+  // |route_id|: MediaRouteId of the issues to be removed.
+  void ClearIssuesWithRouteId(const MediaRouteId& route_id);
+
+  // Registers an issue observer |observer|. The observer will be triggered
+  // when the highest priority issue changes.
+  // If there is already an observer registered with this instance, do nothing.
+  // Does not assume ownership of |observer|.
+  // |observer|: IssuesObserver to be registered.
+  void RegisterObserver(IssuesObserver* observer);
+
+  // Unregisters |observer| from |issues_observers_|.
+  // |observer|: IssuesObserver to be unregistered.
+  void UnregisterObserver(IssuesObserver* observer);
+
+ private:
+  // Checks if the current top issue has changed. Updates |top_issue_|.
+  // If |top_issue_| has changed, issues in |issues_observers_| will be
+  // notified of the new top issue.
+  void MaybeUpdateTopIssue();
+
+  std::list<Issue> issues_;

[DaleCurtis, 2015/05/02 18:13:32]

If you only have 10 elements, using a deque / vector may even be faster than a
list. You can just swap elements to the end for erasing w/o cost.

[Kevin M, 2015/05/04 18:45:48]

OK. I switched top_issue_ from an Issue* to an IssueId to support vector element
erasure. I think it's fine to continue using vector::erase() for clarity, as
there will only be a small number of issues active at any given time.

+
+  // Not owned by IssueManager.
+  ObserverList<IssuesObserver> issues_observers_;
+
+  // A pointer to the previously selected "top issue".
+  const Issue* top_issue_;
+
+  base::ThreadChecker thread_checker_;
+
+  DISALLOW_COPY_AND_ASSIGN(IssueManager);
+};
+
+}  // namespace media_router
+
+#endif  // CHROME_BROWSER_MEDIA_ROUTER_ISSUE_MANAGER_H_