Rename PermissionBubbleManager to PermissionRequestManager

chrome/chrome_tests_unit.gypi

Index: chrome/chrome_tests_unit.gypi
diff --git a/chrome/chrome_tests_unit.gypi b/chrome/chrome_tests_unit.gypi
index 008d404e505236b8b7adc7baeb00358a760fc7d1..6535d357178003696ffa7aa8b43e782cc0be6958 100644
--- a/chrome/chrome_tests_unit.gypi
+++ b/chrome/chrome_tests_unit.gypi
@@ -279,6 +279,10 @@
       'browser/ui/sync/sync_promo_ui_unittest.cc',
       'browser/ui/tests/ui_gfx_image_unittest.cc',
       'browser/ui/tests/ui_gfx_image_unittest.mm',
+      'browser/ui/website_settings/mock_permission_bubble_factory.cc',
+      'browser/ui/website_settings/mock_permission_bubble_factory.h',
+      'browser/ui/website_settings/mock_permission_bubble_view.cc',
+      'browser/ui/website_settings/mock_permission_bubble_view.h',
       'browser/ui/website_settings/website_settings_unittest.cc',
       'browser/ui/webui/browsing_history_handler_unittest.cc',
       'browser/ui/webui/fileicon_source_unittest.cc',
@@ -361,10 +365,6 @@
       'browser/ui/autofill/country_combobox_model_unittest.cc',
       'browser/ui/autofill/save_card_bubble_controller_impl_unittest.cc',
       'browser/ui/passwords/manage_passwords_ui_controller_unittest.cc',
-      'browser/ui/website_settings/mock_permission_bubble_factory.cc',
-      'browser/ui/website_settings/mock_permission_bubble_factory.h',
-      'browser/ui/website_settings/mock_permission_bubble_view.cc',
-      'browser/ui/website_settings/mock_permission_bubble_view.h',
     ],
     'chrome_unit_tests_spellchecker_sources': [
       'browser/spellchecker/feedback_sender_unittest.cc',
@@ -1501,6 +1501,7 @@
       'browser/memory/tab_manager_unittest.cc',
       'browser/memory/tab_manager_web_contents_data_unittest.cc',
       'browser/net/firefox_proxy_settings_unittest.cc',
+      'browser/permissions/permission_request_manager_unittest.cc',
       'browser/platform_util_unittest.cc',
       'browser/power/process_power_collector_unittest.cc',
       'browser/process_singleton_posix_unittest.cc',
@@ -1568,7 +1569,6 @@
       'browser/ui/toolbar/toolbar_actions_bar_unittest.h',
       'browser/ui/toolbar/toolbar_actions_model_unittest.cc',
       'browser/ui/toolbar/toolbar_model_unittest.cc',
-      'browser/ui/website_settings/permission_bubble_manager_unittest.cc',
       'browser/ui/website_settings/permission_menu_model_unittest.cc',
       'browser/ui/webui/help/version_updater_chromeos_unittest.cc',
       'browser/ui/webui/md_downloads/downloads_list_tracker_unittest.cc',
@@ -1916,6 +1916,7 @@
             # Exclude everything but iOS-specific files.
             ['exclude', '\\.(cc|mm)$'],
             ['include', '_ios\\.(cc|mm)$'],
+            ['exclude', '^browser/ui/website_settings/mock_permission_bubble_request\\.(cc|h)$'],

[Lei Zhang, 2016/07/13 01:53:14]

Is there a "non-ios" list to put the files in instead? If iOS magically switches
to GN tomorrow, they aren't going to get this for free.

[Lei Zhang, 2016/07/13 02:04:24]

Actually, is this even needed? Doesn't line 1917 start off excluding everything?

[tsergeant, 2016/07/13 04:20:53]

Hmm, I think you're right.

The iOS trybots seem fine without it.

             ['include', '(^|/)ios/'],
             # TODO(ios): Add files here as they are updated to compile on iOS.
             ['include', '^test/base/chrome_test_suite\\.cc$'],