Prepare fullscreen_chromeos.cc to be used for ozone builds.

chrome/browser/BUILD.gn

 # Copyright 2014 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.
 
 import("//build/config/chrome_build.gni")
 import("//build/config/crypto.gni")
 import("//build/config/features.gni")
 import("//build/config/ui.gni")
 import("//build/split_static_library.gni")
 import("//chrome/common/features.gni")
@@ skipping 363 matching lines @@
     "favicon/favicon_service_factory.cc",
     "favicon/favicon_service_factory.h",
     "favicon/favicon_utils.cc",
     "favicon/favicon_utils.h",
     "favicon/large_icon_service_factory.cc",
     "favicon/large_icon_service_factory.h",
     "file_select_helper.cc",
     "file_select_helper.h",
     "file_select_helper_mac.mm",
     "fullscreen.h",
-    "fullscreen_chromeos.cc",
     "fullscreen_mac.mm",
     "fullscreen_win.cc",
     "geolocation/chrome_access_token_store.cc",
     "geolocation/chrome_access_token_store.h",
     "geolocation/geolocation_permission_context.cc",
     "geolocation/geolocation_permission_context.h",
     "geolocation/geolocation_permission_context_android.cc",
     "geolocation/geolocation_permission_context_android.h",
     "geolocation/geolocation_permission_context_extensions.cc",
     "geolocation/geolocation_permission_context_extensions.h",
@@ skipping 3104 matching lines @@
     if (!use_ozone) {
       sources += [
         "password_manager/native_backend_libsecret.cc",
         "password_manager/native_backend_libsecret.h",
       ]
       defines += [ "USE_LIBSECRET" ]
       deps += [ "//third_party/libsecret" ]
     }
   }
 
+  if (is_chromeos || use_ozone) {
+    sources += [ "fullscreen_chromeos_or_ozone.cc" ]

[rjkroege, 2016/10/26 18:50:16]

this is really weird naming. I'd like there be another way.

[fwang, 2016/10/31 11:38:49]

I'd like to understand what we want here. We have three possibilities:
chromeos=0 use_ozone=1, chromeos=1 use_ozone=0 and chromeos=1 use_ozone=1. Which
fullscreen_* version will be used in each case?
I would have think something like fullscreen_ozone when use_ozone=1 and
fullscreen_chromeos in the remaining case but then Robert suggested to merge
them into a single file, which seems confusing to me. Do we want some kind of
fullscreen_generic or something?

+  }
+
   if (android_java_ui) {
     deps += [ "//third_party/WebKit/public:android_mojo_bindings" ]
   }
 
   if (enable_plugin_installation) {
     sources += [
       "plugins/plugin_installer.cc",
       "plugins/plugin_installer.h",
       "plugins/plugin_installer_observer.cc",
       "plugins/plugin_installer_observer.h",
@@ skipping 808 matching lines @@
   # linking all of the test support.
   static_library("pepper_cdm_test_constants") {
     testonly = true
     visibility = [ "//chrome/*" ]
     sources = [
       "media/pepper_cdm_test_constants.cc",
       "media/pepper_cdm_test_constants.h",
     ]
   }
 }