Remove --disable-app-shims.

apps/app_shim/app_shim_mac.cc

 // 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.
 
 #include "apps/app_shim/app_shim_mac.h"
 
 #include "base/command_line.h"
 #include "chrome/common/chrome_switches.h"
 
 namespace apps {
 
 bool IsAppShimsEnabled() {
   // Disable app shims in tests because shims created in ~/Applications will not
   // be cleaned up.

[tapted, 2014/05/22 04:11:11]

Is this the only reason? That is, can we move this function to an anonymous
namespace in web_app_mac.mm and achieve the same?

[jackhou1, 2014/05/22 04:59:58]

Hmm, when I was considering this, I thought it was better as-is because the
check happens earlier on the UI thread. The entry points in web_app_mac are all
on the FILE thread. On looking at it again, that seems a minor issue. Moving it
to web_app_mac would also simplify AppShortcutsManager::OnceOffCreateShortcuts.

-  return !(CommandLine::ForCurrentProcess()->HasSwitch(switches::kTestType) ||
-      CommandLine::ForCurrentProcess()->HasSwitch(switches::kDisableAppShims));
+  return !CommandLine::ForCurrentProcess()->HasSwitch(switches::kTestType);
 }
 
 }  // namespace apps