ash: fix regression where ctrl+n put new window on wrong desktop

chrome/browser/ui/browser.h

Index: chrome/browser/ui/browser.h
diff --git a/chrome/browser/ui/browser.h b/chrome/browser/ui/browser.h
index dce38b644a549d05d094ef33e73bc8a20a5bad41..63cfd4db7e97f34d660975fd4807aeb88d4c906e 100644
--- a/chrome/browser/ui/browser.h
+++ b/chrome/browser/ui/browser.h
@@ -186,6 +186,9 @@ class Browser : public TabStripModelObserver,
 
     bool is_session_restore;
 
+    // Whether this browser was created by a user gesture.

[Mr4D (OOO till 08-26), 2017/02/13 23:57:39]

Please add a comment towards why this is&needed - not a repetition of the name.

[Elliot Glaysher, 2017/02/15 22:15:02]

Done.

+    bool user_gesture;

[Mr4D (OOO till 08-26), 2017/02/13 23:57:39]

The name "user_gesture" suggests that there must have been touch used. That is
rather confusing.

How about "user_action" - or "user_activation"?

[Elliot Glaysher, 2017/02/15 22:15:02]

I disagree. The name "user_gesture" predates touch support entirely, and has
always referred to an event caused by user input, whether keyboard or mouse (or
when we added touch support, touch).

$ git grep user_gesture | wc -l
1078

+
     // Supply a custom BrowserWindow implementation, to be used instead of the
     // default. Intended for testing.
     BrowserWindow* window;