[Ash] Remove pre-MD code paths from HoverHighlightView

ash/common/system/chromeos/network/network_info.h

 // Copyright 2014 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 ASH_COMMON_SYSTEM_CHROMEOS_NETWORK_NETWORK_INFO_H_
 #define ASH_COMMON_SYSTEM_CHROMEOS_NETWORK_NETWORK_INFO_H_
 
 #include <string>
 
 #include "base/strings/string16.h"
 #include "ui/gfx/image/image_skia.h"
 
 namespace gfx {
 class ImageSkia;
 }
 
 namespace ash {
 
 // Includes information necessary about a network for displaying the appropriate
 // UI to the user.
+// TODO(tdanderson): Remove the |highlight| member. See crbug.com/708190.

[Evan Stade, 2017/04/04 17:18:18]

why add a TODO instead of removing right away? Is there some complexity to
removing it?

[tdanderson, 2017/04/04 18:45:06]

It looked like it is still used in network ordering logic. Changing that seems
out of scope for this CL and has the potential to introduce an unrelated
regression since it's not 100% a matter of just deleting dead code. For those
reasons I think it is better to land in a separate CL.

 struct NetworkInfo {
   enum class Type { UNKNOWN, WIFI, TETHER, CELLULAR };
 
   NetworkInfo();
   NetworkInfo(const std::string& guid);
   ~NetworkInfo();
 
   bool operator==(const NetworkInfo& other) const;
   bool operator!=(const NetworkInfo& other) const { return !(*this == other); }
 
   std::string guid;
   base::string16 label;
   base::string16 tooltip;
   gfx::ImageSkia image;
   bool disable;
   bool highlight;
   bool connected;
   bool connecting;
   Type type;
 };
 
 }  // namespace ash
 
 #endif  // ASH_COMMON_SYSTEM_CHROMEOS_NETWORK_NETWORK_INFO_H_