Fix declaration vs. implementation ordering in NativeAppWindow{Views|Gtk}

Fix declaration vs. implementation ordering in NativeAppWindow{Views|Gtk}

This just reorders the functions in the NativeAppWindow interface. There are
other ordering problems but they aren't worth fixing.

Also move native_app_window.h into apps/ui/.

BUG=306535
TEST=compiles
TBR=benwells@chromium.org for file moves and trivial header path renames

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=228848

[James Cook, 2013-10-14 22:33:12]

Steven, PTAL

This CL depends on https://codereview.chromium.org/27029006/

This doesn't fix everything, but leaves these files better than they were
before.

[James Cook, 2013-10-14 22:35:22]

actually, let me do one more thing in this CL first

[James Cook, 2013-10-14 22:52:29]

Steven, now PTAL.

I figured I could safely lump these two changes into one "move things around"
CL, especially since I'm about to start another CL that depends on this one.  It
might be easier to review to look at patch set 1 first -- the second patch set
just moves the file.

[stevenjb, 2013-10-14 23:20:48]

lgtm

https://codereview.chromium.org/27207003/diff/6001/chrome/browser/ui/views/ap...
File chrome/browser/ui/views/apps/native_app_window_views.cc (right):

https://codereview.chromium.org/27207003/diff/6001/chrome/browser/ui/views/ap...
chrome/browser/ui/views/apps/native_app_window_views.cc:839: void
NativeAppWindowViews::HideWithApp() {}
WS

[commit-bot: I haz the power, 2013-10-15 04:59:39]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jamescook@chromium.org/27207003/6001

[commit-bot: I haz the power, 2013-10-15 04:59:52]

Failed to apply patch for
chrome/browser/ui/ash/launcher/app_shortcut_launcher_item_controller.cc:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file
chrome/browser/ui/ash/launcher/app_shortcut_launcher_item_controller.cc
  Hunk #1 FAILED at 4.
  1 out of 1 hunk FAILED -- saving rejects to file
chrome/browser/ui/ash/launcher/app_shortcut_launcher_item_controller.cc.rej

Patch:      
chrome/browser/ui/ash/launcher/app_shortcut_launcher_item_controller.cc
Index: chrome/browser/ui/ash/launcher/app_shortcut_launcher_item_controller.cc
diff --git
a/chrome/browser/ui/ash/launcher/app_shortcut_launcher_item_controller.cc
b/chrome/browser/ui/ash/launcher/app_shortcut_launcher_item_controller.cc
index
97f62bd4e6a7fe7f8a1bc3d9a5c0222aaf61f4b5..a19dcfd79460d1a96903a49e7ee5c8444ca28239
100644
--- a/chrome/browser/ui/ash/launcher/app_shortcut_launcher_item_controller.cc
+++ b/chrome/browser/ui/ash/launcher/app_shortcut_launcher_item_controller.cc
@@ -4,7 +4,7 @@
 
 #include
"chrome/browser/ui/ash/launcher/app_shortcut_launcher_item_controller.h"
 
-#include "apps/native_app_window.h"
+#include "apps/ui/native_app_window.h"
 #include "ash/wm/window_util.h"
 #include "chrome/browser/extensions/extension_process_manager.h"
 #include "chrome/browser/extensions/extension_system.h"

[commit-bot: I haz the power, 2013-10-15 17:30:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jamescook@chromium.org/27207003/15001

[commit-bot: I haz the power, 2013-10-15 17:41:29]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_clan...

[commit-bot: I haz the power, 2013-10-15 17:50:42]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jamescook@chromium.org/27207003/15001

[commit-bot: I haz the power, 2013-10-15 23:35:43]

Retried try job too often on win7_aura for step(s) browser_tests,
interactive_ui_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win7_aura&...

[commit-bot: I haz the power, 2013-10-15 23:56:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jamescook@chromium.org/27207003/41001

[commit-bot: I haz the power, 2013-10-16 03:09:14]

Change committed as 228848