Remove non-MD code from LabelTrayView

ash/common/system/chromeos/enterprise/tray_enterprise.cc

Index: ash/common/system/chromeos/enterprise/tray_enterprise.cc
diff --git a/ash/common/system/chromeos/enterprise/tray_enterprise.cc b/ash/common/system/chromeos/enterprise/tray_enterprise.cc
index 26f48caba17854093ab419d6b6cd83ee555fdcab..1b7bbb3f7c0a25b6407f467e87acc201e53f4af8 100644
--- a/ash/common/system/chromeos/enterprise/tray_enterprise.cc
+++ b/ash/common/system/chromeos/enterprise/tray_enterprise.cc
@@ -9,7 +9,7 @@
 #include "ash/common/system/tray/system_tray_delegate.h"
 #include "ash/common/system/tray/system_tray_notifier.h"
 #include "ash/common/wm_shell.h"
-#include "ash/resources/grit/ash_resources.h"
+#include "ash/resources/vector_icons/vector_icons.h"
 #include "base/logging.h"
 #include "base/strings/string16.h"
 
@@ -32,18 +32,18 @@ void TrayEnterprise::UpdateEnterpriseMessage() {
 }
 
 views::View* TrayEnterprise::CreateDefaultView(LoginStatus status) {
-  CHECK(tray_view_ == NULL);
+  CHECK(!tray_view_);

[Evan Stade, 2017/02/22 21:43:46]

nit: DCHECK

[tdanderson, 2017/03/30 14:51:39]

Done.

   // For public accounts, enterprise ownership is indicated in the user details
   // instead.
   if (status == LoginStatus::PUBLIC)
-    return NULL;
-  tray_view_ = new LabelTrayView(this, IDR_AURA_UBER_TRAY_ENTERPRISE);
+    return nullptr;
+  tray_view_ = new LabelTrayView(this, kSystemMenuBusinessIcon);
   UpdateEnterpriseMessage();
   return tray_view_;
 }
 
 void TrayEnterprise::DestroyDefaultView() {

[Evan Stade, 2017/02/22 21:43:46]

aside: it seems like this should be called DefaultViewDestroyed since you don't
actually own the default view. Alternately we could do
tray_view_->set_owned_by_client() and actually destroy it here so that this
function name makes sense, but I think changing the function name to match
behavior is better than vice versa.

I assume this pattern is the same everywhere we have a tray item overriding
Destroy*View().

[tdanderson, 2017/03/30 14:51:39]

Yes, agreed. Filed https://bugs.chromium.org/p/chromium/issues/detail?id=706821
to track that.
I believe it is, yes.

-  tray_view_ = NULL;
+  tray_view_ = nullptr;
 }
 
 void TrayEnterprise::OnEnterpriseDomainChanged() {