Move Show and Hide impls from PlatformWindowMus to NativeWidgetMus

ui/views/mus/platform_window_mus.cc

 // 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.
 
 #include "ui/views/mus/platform_window_mus.h"
 
 #include "base/message_loop/message_loop.h"
 #include "build/build_config.h"
 #include "components/bitmap_uploader/bitmap_uploader.h"
 #include "components/mus/public/cpp/property_type_converters.h"
@@ skipping 87 matching lines @@
 
 PlatformWindowMus::~PlatformWindowMus() {
   if (!mus_window_)
     return;
   mus_window_->RemoveObserver(this);
   mus_window_->set_input_event_handler(nullptr);
   if (!mus_window_destroyed_)
     mus_window_->Destroy();
 }
 
-void PlatformWindowMus::Activate() {
-  mus_window_->SetFocus();
-}
-
 void PlatformWindowMus::SetCursorById(mus::mojom::Cursor cursor) {
   if (last_cursor_ != cursor) {
     // The ui::PlatformWindow interface uses ui::PlatformCursor at this level,
     // instead of ui::Cursor. All of the cursor abstractions in ui right now are
     // sort of leaky; despite being nominally cross platform, they all drop down
     // to platform types almost immediately, which makes them unusable as
     // transport types.
     mus_window_->SetPredefinedCursor(cursor);
   }
 }
 
 void PlatformWindowMus::Show() {
-  mus_window_->SetVisible(true);
+  NOTIMPLEMENTED();
 }
 
 void PlatformWindowMus::Hide() {
-  mus_window_->SetVisible(false);
+  NOTIMPLEMENTED();
 }
 
 void PlatformWindowMus::Close() {
   NOTIMPLEMENTED();
 }
 
 void PlatformWindowMus::SetBounds(const gfx::Rect& bounds) {
-  mus_window_->SetBounds(bounds);
+  NOTIMPLEMENTED();

[sadrul, 2016/05/09 16:38:26]

Add a class-level comment that PlatformWindowMus is going to be removed, and
remove the NOTIMPLEMENTED()s from here.

[Mark Dittmer, 2016/05/09 19:04:44]

Addressed in https://codereview.chromium.org/1953293004/ and here.

 }
 
 gfx::Rect PlatformWindowMus::GetBounds() {
   return mus_window_->bounds();
 }
 
 void PlatformWindowMus::SetTitle(const base::string16& title) {
   NOTIMPLEMENTED();
 }
 
@@ skipping 126 matching lines @@
 
   std::unique_ptr<ui::Event> event = ui::Event::Clone(event_in);
   delegate_->DispatchEvent(event.get());
   // NOTE: |this| may be deleted.
 
   ack_handler.set_handled(event->handled());
   // |ack_handler| acks the event on destruction if necessary.
 }
 
 }  // namespace views