Disable new bookmark apps on Mac.

extensions/browser/api/management/management_api.cc

 // Copyright (c) 2012 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 "extensions/browser/api/management/management_api.h"
 
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
@@ skipping 66 matching lines @@
   if (extension.is_platform_app()) {
     launch_type_list.push_back(management::LAUNCH_TYPE_OPEN_AS_WINDOW);
     return launch_type_list;
   }
 
   launch_type_list.push_back(management::LAUNCH_TYPE_OPEN_AS_REGULAR_TAB);
 
 #if !defined(OS_MACOSX)
   // Open as window is currently disabled on Mac.
   launch_type_list.push_back(management::LAUNCH_TYPE_OPEN_AS_WINDOW);
-
-  if (!delegate->IsNewBookmarkAppsEnabled()) {

[benwells, 2015/08/18 06:32:07]

Why is this code being deleted? It only affects non-chromeos when the flag is
disabled. I believe it should stay as it was.

[dominickn, 2015/08/18 06:53:00]

Done, had the mistaken thought that pinned and fullscreen functionality had been
removed.

-    launch_type_list.push_back(management::LAUNCH_TYPE_OPEN_AS_PINNED_TAB);
-    launch_type_list.push_back(management::LAUNCH_TYPE_OPEN_FULL_SCREEN);
-  }
 #endif
   return launch_type_list;
 }
 
 scoped_ptr<management::ExtensionInfo> CreateExtensionInfo(
     const Extension& extension,
     content::BrowserContext* context) {
   ExtensionSystem* system = ExtensionSystem::Get(context);
   ExtensionRegistry* registry = ExtensionRegistry::Get(context);
   const ManagementAPIDelegate* delegate =
@@ skipping 808 matching lines @@
 ManagementAPI::GetFactoryInstance() {
   return g_factory.Pointer();
 }
 
 void ManagementAPI::OnListenerAdded(const EventListenerInfo& details) {
   management_event_router_.reset(new ManagementEventRouter(browser_context_));
   EventRouter::Get(browser_context_)->UnregisterObserver(this);
 }
 
 }  // namespace extensions