Clean up nits in settings_utils_linux.cc.

chrome/chrome_browser_ui.gypi

 # Copyright 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.
 
 {
   'variables': {
     # There are two sections of sources:
     # * "Platform"-based sources (OS, toolkit, etc.)
     # * Feature/directory-based sources (e.g., history, printing)
 
@@ skipping 1339 matching lines @@
       'browser/ui/cocoa/website_settings/website_settings_utils_cocoa.mm',
       'browser/ui/cocoa/window_size_autosaver.h',
       'browser/ui/cocoa/window_size_autosaver.mm',
       # TODO(estade): this class should be deleted in favor of a combobox model.
       # See crbug.com/590850
       'browser/ui/content_settings/content_setting_media_menu_model.cc',
       'browser/ui/content_settings/content_setting_media_menu_model.h',
       'browser/ui/proximity_auth/proximity_auth_error_bubble_stub.cc',
     ],
     # Files used only on desktop systems (not iOS, Android, ChromeOS).
     'chrome_browser_ui_desktop_sources': [

[tommycli, 2016/08/26 19:28:16]

And there's already a filter for _linux _mac and _win.cc files right?

[Lei Zhang, 2016/08/26 19:34:41]

Yep. It's only the linuxiness that's in question here.

       'browser/ui/avatar_button_error_controller.h',
       'browser/ui/avatar_button_error_controller.cc',
       'browser/ui/avatar_button_error_controller_delegate.h',
       'browser/ui/startup/default_browser_infobar_delegate.cc',
       'browser/ui/startup/default_browser_infobar_delegate.h',
       'browser/ui/startup/default_browser_prompt.cc',
       'browser/ui/startup/default_browser_prompt.h',
       'browser/ui/startup/default_browser_prompt_win.cc',
       'browser/ui/sync/one_click_signin_links_delegate.h',
       'browser/ui/sync/one_click_signin_links_delegate_impl.cc',
       'browser/ui/sync/one_click_signin_links_delegate_impl.h',
       'browser/ui/sync/one_click_signin_sync_observer.cc',
       'browser/ui/sync/one_click_signin_sync_observer.h',
       'browser/ui/sync/one_click_signin_sync_starter.cc',
       'browser/ui/sync/one_click_signin_sync_starter.h',
       'browser/ui/views/frame/avatar_button_manager.cc',
       'browser/ui/views/frame/avatar_button_manager.h',
       'browser/ui/views/profiles/avatar_button_delegate.h',
       'browser/ui/views/profiles/avatar_button_style.h',
       'browser/ui/views/profiles/new_avatar_button.cc',
       'browser/ui/views/profiles/new_avatar_button.h',
       'browser/ui/webui/app_launcher_page_ui.cc',
       'browser/ui/webui/app_launcher_page_ui.h',
       'browser/ui/webui/settings/settings_default_browser_handler.cc',
       'browser/ui/webui/settings/settings_default_browser_handler.h',
       'browser/ui/webui/settings/system_handler.cc',
       'browser/ui/webui/settings/system_handler.h',
+      'browser/ui/webui/settings_utils_linux.cc',
+      'browser/ui/webui/settings_utils_mac.mm',
+      'browser/ui/webui/settings_utils_win.cc',
       'browser/ui/webui/signin/inline_login_handler.cc',
       'browser/ui/webui/signin/inline_login_handler.h',
       'browser/ui/webui/signin/inline_login_handler_impl.cc',
       'browser/ui/webui/signin/inline_login_handler_impl.h',
       'browser/ui/webui/signin/inline_login_ui.cc',
       'browser/ui/webui/signin/inline_login_ui.h',
       'browser/ui/webui/signin/md_user_manager_ui.cc',
       'browser/ui/webui/signin/md_user_manager_ui.h',
       'browser/ui/webui/signin/signin_create_profile_handler.cc',
       'browser/ui/webui/signin/signin_create_profile_handler.h',
@@ skipping 662 matching lines @@
       'browser/ui/webui/settings/settings_media_devices_selection_handler.cc',
       'browser/ui/webui/settings/settings_media_devices_selection_handler.h',
       'browser/ui/webui/settings/settings_page_ui_handler.cc',
       'browser/ui/webui/settings/settings_page_ui_handler.h',
       'browser/ui/webui/settings/settings_startup_pages_handler.cc',
       'browser/ui/webui/settings/settings_startup_pages_handler.h',
       'browser/ui/webui/settings/site_settings_handler.cc',
       'browser/ui/webui/settings/site_settings_handler.h',
       'browser/ui/webui/settings_utils.cc',
       'browser/ui/webui/settings_utils.h',
-      'browser/ui/webui/settings_utils_linux.cc',
-      'browser/ui/webui/settings_utils_mac.mm',
-      'browser/ui/webui/settings_utils_win.cc',
       'browser/ui/webui/signin/get_auth_frame.cc',
       'browser/ui/webui/signin/get_auth_frame.h',
       'browser/ui/webui/signin/login_ui_service.cc',
       'browser/ui/webui/signin/login_ui_service.h',
       'browser/ui/webui/signin/login_ui_service_factory.cc',
       'browser/ui/webui/signin/login_ui_service_factory.h',
       'browser/ui/webui/signin/profile_signin_confirmation_dialog.cc',
       'browser/ui/webui/signin/profile_signin_confirmation_dialog.h',
       'browser/ui/webui/signin/profile_signin_confirmation_ui.cc',
       'browser/ui/webui/signin/profile_signin_confirmation_ui.h',
@@ skipping 1251 matching lines @@
           ],
           'sources': [
             '<@(chrome_browser_ui_non_mobile_sources)',
             '<@(chrome_browser_ui_omnibox_non_mobile_sources)',
           ],
         }],
       ],
     },
   ],
 }