[Media Router] Clean up issues related code.

chrome/browser/ui/webui/media_router/media_router_webui_message_handler.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_UI_WEBUI_MEDIA_ROUTER_MEDIA_ROUTER_WEBUI_MESSAGE_HANDLER_H_
 #define CHROME_BROWSER_UI_WEBUI_MEDIA_ROUTER_MEDIA_ROUTER_WEBUI_MESSAGE_HANDLER_H_
 
 #include <vector>
 
 #include "base/macros.h"
@@ skipping 30 matching lines @@
   void UpdateRoutes(const std::vector<MediaRoute>& routes,
                     const std::vector<MediaRoute::Id>& joinable_route_ids,
                     const std::unordered_map<MediaRoute::Id, MediaCastMode>&
                         current_cast_modes);
   void UpdateCastModes(const CastModeSet& cast_modes,
                        const std::string& source_host);
   void OnCreateRouteResponseReceived(const MediaSink::Id& sink_id,
                                      const MediaRoute* route);
   void ReturnSearchResult(const std::string& sink_id);
 
-  // Does not take ownership of |issue|. Note that |issue| can be nullptr, when
-  // there are no more issues.
-  void UpdateIssue(const Issue* issue);
+  void UpdateIssue(const Issue& issue);
+  void ClearIssue();
 
   // Updates the maximum dialog height to allow the WebUI properly scale when
   // the browser window changes.
   void UpdateMaxDialogHeight(int height);
 
   void SetWebUIForTest(content::WebUI* webui);
   void set_incognito_for_test(bool incognito) { incognito_ = incognito; }
 
  private:
   // WebUIMessageHandler implementation.
@@ skipping 19 matching lines @@
   void OnReportSelectedCastMode(const base::ListValue* args);
   void OnReportSinkCount(const base::ListValue* args);
   void OnReportTimeToClickSink(const base::ListValue* args);
   void OnReportTimeToInitialActionClose(const base::ListValue* args);
   void OnSearchSinksAndCreateRoute(const base::ListValue* args);
   void OnInitialDataReceived(const base::ListValue* args);
 
   // Performs an action for an Issue of |type|.
   // |args| contains additional parameter that varies based on |type|.
   // Returns |true| if the action was successfully performed.
-  bool ActOnIssueType(const IssueAction::Type& type,
+  bool ActOnIssueType(IssueInfo::Action type,
                       const base::DictionaryValue* args);
 
   // May update the first run flow related properties in the WebUI. This is
   // called after the initial data is received to avoid unnecessary work when
   // initializing the WebUI.
   void MaybeUpdateFirstRunFlowData();
 
   // Returns the current cast mode for the route with ID |route_id| or -1 if the
   // route has no current cast mode.
   int CurrentCastModeForRouteId(
@@ skipping 21 matching lines @@
   bool dialog_closing_;
 
   MediaRouterUI* media_router_ui_;
 
   DISALLOW_COPY_AND_ASSIGN(MediaRouterWebUIMessageHandler);
 };
 
 }  // namespace media_router
 
 #endif  // CHROME_BROWSER_UI_WEBUI_MEDIA_ROUTER_MEDIA_ROUTER_WEBUI_MESSAGE_HANDLER_H_