[MacViews] Add NativeAppWindowViewsMac and port HideWithApp behavior from NativeAppWindowCocoa.

chrome/chrome_tests.gypi

 # Copyright 2013 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.
 {
   'variables': {
     'chrome_browser_extensions_test_support_sources': [
       # A list of sources which is shared between different browser tests.
       'browser/apps/app_browsertest_util.cc',
       'browser/apps/app_browsertest_util.h',
       'browser/extensions/browsertest_util.cc',
@@ skipping 2310 matching lines @@
         }],
         ['toolkit_views==1', {
           'sources': [ '<@(chrome_browser_tests_views_sources)' ],
           'dependencies': [
             '../ui/views/views.gyp:views',
           ],
         }],
         ['toolkit_views==1 and OS!="mac"', {
           'sources': [ '<@(chrome_browser_tests_views_non_mac_sources)' ],
         }],
+        ['toolkit_views==1 and OS=="mac" and mac_views_browser==1', {

[tapted, 2015/03/04 02:27:33]

nit: just check mac_views_browser?

[jackhou1, 2015/03/04 03:34:55]

Removed.

+          'sources': [
+            'browser/ui/views/apps/chrome_native_app_window_views_mac_browsertest.mm',

[tapted, 2015/03/04 02:27:33]

Hm. I think this needs a new variable, otherwise each change needs a
corresponding gn change as well (a variable will let us do just one gn change).
Since it will just be macviews stuff, I don't think an extra variable will make
things more confusing.

Then the variable can have a TODO like ~ "Move items from this list to
chrome_browser_tests_views_sources when they migrate from mac_views_browser to a
chrome://flag"

The non-test file list should have a variable too -- but just for the stuff
being added (i.e. a variable won't work for the `sources!` keys)

[jackhou1, 2015/03/04 03:34:55]

Done.

+          ],
+        }],
         ['OS!="android" and OS!="ios"', {
           'sources': [
             'browser/copresence/chrome_whispernet_client_browsertest.cc',
           ],
           'dependencies': [
             '../components/components.gyp:copresence',
             # build time dependency.
             '../v8/src/d8.gyp:d8#host',
           ],
         }],
@@ skipping 828 matching lines @@
             ['enable_webrtc==1', {
               'dependencies': [
                 '../third_party/libjingle/libjingle.gyp:libjingle_webrtc'
               ]
             }],
           ],
         }]
     }],
   ],  # 'conditions'
 }