Drop unnecessary override of OnWindowBoundsChanged in NativeWidgetMus::MusWindowObserver

Drop unnecessary override of OnWindowBoundsChanged in NativeWidgetMus::MusWindowObserver

R=sky@chromium.org
BUG=617315
Committed: https://crrev.com/72925754efce72c58d353db9d7c2b85855e6dcb7
Cr-Commit-Position: refs/heads/master@{#398125}

[Mark Dittmer, 2016-06-05 18:14:56]

The CQ bit was checked by markdittmer@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-06-05 18:15:13]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2040783002/1

[commit-bot: I haz the power, 2016-06-05 19:02:39]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-06-05 19:02:40]

Dry run: Try jobs failed on following builders:
  win8_chromium_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_ng/...)

[Mark Dittmer, 2016-06-06 14:57:04]

Set delegate in StubWindow to enable normal flow for notifying OnBoundsChanged

[Mark Dittmer, 2016-06-06 14:57:25]

The CQ bit was checked by markdittmer@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-06-06 14:57:43]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2040783002/20001

[Mark Dittmer, 2016-06-06 14:59:34]

sky@chromium.org: PTAL:

Enabled use of delegate for mus + StubWindow case with optional StubWindow ctor
parameter. Since mus was the only case with no delegate (and now uses a
delegate), checks for StubWindow::delegate_ were also removed.

[Mark Dittmer, 2016-06-06 15:21:30]

Include base/logging.h for DCHECK

[Mark Dittmer, 2016-06-06 15:21:38]

The CQ bit was checked by markdittmer@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-06-06 15:21:57]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2040783002/40001

[commit-bot: I haz the power, 2016-06-06 16:05:16]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-06-06 16:05:17]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_compile_dbg_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Mark Dittmer, 2016-06-06 16:31:05]

Add build deps for base

[Mark Dittmer, 2016-06-06 16:31:18]

The CQ bit was checked by markdittmer@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-06-06 16:31:58]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2040783002/60001

[sky, 2016-06-06 16:35:26]

LGTM

https://codereview.chromium.org/2040783002/diff/20001/ui/platform_window/stub...
File ui/platform_window/stub/stub_window.h (right):

https://codereview.chromium.org/2040783002/diff/20001/ui/platform_window/stub...
ui/platform_window/stub/stub_window.h:19: bool use_default_accelerated_widget =
true);
It isn't obvious what use_default_accelerated_widget means. Please add a
description.

[commit-bot: I haz the power, 2016-06-06 17:58:43]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-06-06 17:58:44]

Dry run: This issue passed the CQ dry run.

[Mark Dittmer, 2016-06-06 18:21:00]

The CQ bit was checked by markdittmer@chromium.org

[commit-bot: I haz the power, 2016-06-06 18:21:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2040783002/60001

[commit-bot: I haz the power, 2016-06-06 18:28:04]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-06-06 18:28:05]

Failed to apply patch for ui/views/mus/window_tree_host_mus.cc:
While running git apply --index -3 -p1;
  error: patch failed: ui/views/mus/window_tree_host_mus.cc:53
  Falling back to three-way merge...
  Applied patch to 'ui/views/mus/window_tree_host_mus.cc' with conflicts.
  U ui/views/mus/window_tree_host_mus.cc

Patch:       ui/views/mus/window_tree_host_mus.cc
Index: ui/views/mus/window_tree_host_mus.cc
diff --git a/ui/views/mus/window_tree_host_mus.cc
b/ui/views/mus/window_tree_host_mus.cc
index
dcf8b81f683289fe398bd23644f79efdf7e19422..2105f56bd7a53eb658d0f985817733c844751f68
100644
--- a/ui/views/mus/window_tree_host_mus.cc
+++ b/ui/views/mus/window_tree_host_mus.cc
@@ -53,7 +53,10 @@ WindowTreeHostMus::WindowTreeHostMus(shell::Connector*
connector,
                          bitmap_uploader_.get()));
   }
 
-  SetPlatformWindow(base::WrapUnique(new ui::StubWindow(nullptr)));
+  SetPlatformWindow(base::WrapUnique(new ui::StubWindow(
+      this,
+      false)));  // Do not advertise accelerated widget; already set manually.
+
   // Initialize the stub platform window bounds to those of the mus::Window.
   platform_window()->SetBounds(window->bounds());

[commit-bot: I haz the power, 2016-06-06 18:28:26]

The CQ bit was unchecked by commit-bot@chromium.org

[Mark Dittmer, 2016-06-06 18:35:20]

rebase

[Mark Dittmer, 2016-06-06 18:35:56]

The CQ bit was checked by markdittmer@chromium.org

[Mark Dittmer, 2016-06-06 18:35:56]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/2040783002/#ps80001
(title: "rebase")

[commit-bot: I haz the power, 2016-06-06 18:36:25]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2040783002/80001

[commit-bot: I haz the power, 2016-06-06 19:08:20]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-06-06 19:08:22]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Mark Dittmer, 2016-06-06 19:55:54]

The CQ bit was checked by markdittmer@chromium.org

[commit-bot: I haz the power, 2016-06-06 19:56:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2040783002/80001

[commit-bot: I haz the power, 2016-06-06 21:18:55]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-06-06 21:20:12]

Description was changed from

==========
Drop unnecessary override of OnWindowBoundsChanged in
NativeWidgetMus::MusWindowObserver

R=sky@chromium.org
BUG=617315
==========

to

==========
Drop unnecessary override of OnWindowBoundsChanged in
NativeWidgetMus::MusWindowObserver

R=sky@chromium.org
BUG=617315

Committed: https://crrev.com/72925754efce72c58d353db9d7c2b85855e6dcb7
Cr-Commit-Position: refs/heads/master@{#398125}
==========

[commit-bot: I haz the power, 2016-06-06 21:20:13]

Patchset 5 (id:??) landed as
https://crrev.com/72925754efce72c58d353db9d7c2b85855e6dcb7
Cr-Commit-Position: refs/heads/master@{#398125}