Adds TODO with reference to bug

ui/views/mus/screen_mus.cc

 // 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.
 
+// This has to be before any other includes, else default is picked up.
+// See base/logging for details on this.
+#define NOTIMPLEMENTED_POLICY 5

[James Cook, 2016/08/24 23:15:37]

Could this live in the build file? Or do you only want it for this particular
file?

[sky, 2016/08/24 23:25:55]

Yes, if I add to build file it impacts all of views, which I don't think we
want.

+
 #include "ui/views/mus/screen_mus.h"
 
 #include "services/shell/public/cpp/connection.h"
 #include "services/shell/public/cpp/connector.h"
 #include "ui/aura/window.h"
 #include "ui/display/display_finder.h"
 #include "ui/display/display_observer.h"
 #include "ui/views/mus/screen_mus_delegate.h"
 #include "ui/views/mus/window_manager_frame_values.h"
 
-#ifdef NOTIMPLEMENTED
-#undef NOTIMPLEMENTED
-#define NOTIMPLEMENTED() DVLOG(1) << "notimplemented"
-#endif
-
 namespace mojo {
 
 template <>
 struct TypeConverter<views::WindowManagerFrameValues,
                      ui::mojom::FrameDecorationValuesPtr> {
   static views::WindowManagerFrameValues Convert(
       const ui::mojom::FrameDecorationValuesPtr& input) {
     views::WindowManagerFrameValues result;
     result.normal_insets = input->normal_client_area_insets;
     result.maximized_insets = input->maximized_client_area_insets;
@@ skipping 153 matching lines @@
         delegate_->OnWindowManagerFrameValuesChanged();
     }
   }
 }
 
 void ScreenMus::OnDisplayRemoved(int64_t id) {
   display_list_.RemoveDisplay(id);
 }
 
 }  // namespace views