NOT FOR REVIEW: Aura on Android

content/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/crypto.gni")
 import("//build/config/features.gni")
 import("//build/config/ui.gni")
 import("//content/browser/browser.gni")
 import("//media/media_options.gni")
 
@@ skipping 326 matching lines @@
     sources += rebase_path(content_browser_gypi_values.android_browser_sources,
                            ".",
                            "//content")
     sources -= [
       "browser_ipc_logging.cc",
       "device_sensors/data_fetcher_shared_memory_default.cc",
       "geolocation/network_location_provider.cc",
       "geolocation/network_location_provider.h",
       "geolocation/network_location_request.cc",
       "geolocation/network_location_request.h",
-      "renderer_host/begin_frame_observer_proxy.cc",
+#      "renderer_host/begin_frame_observer_proxy.cc",
       "tracing/tracing_ui.cc",
       "tracing/tracing_ui.h",
 
       # Android skips most, but not all, of the speech code.
       "speech/audio_buffer.cc",
       "speech/audio_buffer.h",
       "speech/audio_encoder.cc",
       "speech/audio_encoder.h",
       "speech/chunked_byte_buffer.cc",
       "speech/chunked_byte_buffer.h",
       "speech/endpointer/endpointer.cc",
       "speech/endpointer/endpointer.h",
       "speech/endpointer/energy_endpointer.cc",
       "speech/endpointer/energy_endpointer.h",
       "speech/endpointer/energy_endpointer_params.cc",
       "speech/endpointer/energy_endpointer_params.h",
       "speech/google_one_shot_remote_engine.cc",
       "speech/google_one_shot_remote_engine.h",
       "speech/google_streaming_remote_engine.cc",
       "speech/google_streaming_remote_engine.h",
       "speech/speech_recognition_engine.cc",
       "speech/speech_recognition_engine.h",
       "speech/speech_recognizer_impl.cc",
       "speech/speech_recognizer_impl.h",
     ]
+    
+    if (!use_aura) {
+      sources -= [
+        "renderer_host/begin_frame_observer_proxy.cc",

[no sievers, 2015/10/20 19:24:22]

let's consider moving this to 'compositor_browser_sources'  and into
content/browser/compositor/

+      ]
+    }
+    
     deps += [
       "//content/public/android:jni",
       "//media",
       "//mojo/android:libsystem_java",
       "//ui/android",
     ]
     libs += [ "jnigraphics" ]
   }
 
   if (is_mac) {
@@ skipping 22 matching lines @@
     ]
   }
 
   if (use_aura) {
     deps += [
       "//ui/aura",
       "//ui/aura_extra",
       "//ui/strings",
       "//ui/wm",
     ]
+
+    if (is_android) {
+      sources-= [
+        "accessibility/accessibility_tree_formatter_android.cc",
+        "accessibility/browser_accessibility_android.cc",
+        "accessibility/browser_accessibility_android.h",
+        "accessibility/browser_accessibility_manager_android.cc",
+        "accessibility/browser_accessibility_manager_android.h",

[no sievers, 2015/10/20 19:24:21]

let's pull out all accessibility-related changes into a separate cl, so they are
easier to revisit (probably also should have a TODO and bug)

+  
+        "android/animation_utils.h",

[no sievers, 2015/10/20 19:24:21]

android/animation_utils.h and touch handle and edge effect stuff from below we
could put into a 'android_browser_ui_effects' sort of source file list.

+        "android/background_sync_launcher_android.cc",
+        "android/background_sync_launcher_android.h",

[no sievers, 2015/10/20 19:24:22]

to-be-decoupled

+  #      "android/browser_jni_registrar.cc",
+  #      "android/browser_jni_registrar.h",
+  #      "android/browser_startup_controller.cc",
+  #      "android/browser_startup_controller.h",
+        "android/browser_surface_texture_manager.cc",
+        "android/browser_surface_texture_manager.h",

[no sievers, 2015/10/20 19:24:22]

try to keep this in

+        
+  #      "android/child_process_launcher_android.cc",
+  #      "android/child_process_launcher_android.h",
+        
+        "android/composited_touch_handle_drawable.cc",
+        "android/composited_touch_handle_drawable.h",
+        "android/content_protocol_handler_impl.cc",
+        "android/content_protocol_handler_impl.h",

[no sievers, 2015/10/20 19:24:22]

need to keep content_protocol_handler for file requests i think

+        "android/content_readback_handler.cc",
+        "android/content_readback_handler.h",
+  #      "android/content_startup_flags.cc",

[no sievers, 2015/10/20 19:24:22]

should keep content_startup_flags.cc

+  #      "android/content_startup_flags.h",
+        "android/content_video_view.cc",
+        "android/content_video_view.h",
+        "android/content_view_core_impl.cc",
+        "android/content_view_core_impl.h",
+        "android/content_view_render_view.cc",
+        "android/content_view_render_view.h",
+        "android/content_view_statics.cc",
+        "android/content_view_statics.h",
+        "android/date_time_chooser_android.cc",
+        "android/date_time_chooser_android.h",
+        "android/deferred_download_observer.cc",
+        "android/deferred_download_observer.h",
+  #      "android/devtools_auth.cc",
+        "android/download_controller_android_impl.cc",
+        "android/download_controller_android_impl.h",
+        "android/edge_effect.cc",
+        "android/edge_effect.h",
+        "android/edge_effect_l.cc",
+        "android/edge_effect_l.h",
+        "android/in_process/context_provider_in_process.cc",
+        "android/in_process/context_provider_in_process.h",
+        "android/in_process/synchronous_compositor_external_begin_frame_source.cc",
+        "android/in_process/synchronous_compositor_external_begin_frame_source.h",
+        "android/in_process/synchronous_compositor_factory_impl.cc",
+        "android/in_process/synchronous_compositor_factory_impl.h",
+        "android/in_process/synchronous_compositor_impl.cc",
+        "android/in_process/synchronous_compositor_impl.h",
+        "android/in_process/synchronous_compositor_output_surface.cc",
+        "android/in_process/synchronous_compositor_output_surface.h",
+        "android/in_process/synchronous_compositor_registry.cc",
+        "android/in_process/synchronous_compositor_registry.h",
+        "android/in_process/synchronous_input_event_filter.cc",
+        "android/in_process/synchronous_input_event_filter.h",
+        "android/in_process_surface_texture_manager.cc",
+        "android/in_process_surface_texture_manager.h",

[no sievers, 2015/10/20 19:24:21]

keep in_process_surface_texture_manager

+        "android/interstitial_page_delegate_android.cc",
+        "android/interstitial_page_delegate_android.h",
+        "android/load_url_params.cc",
+        "android/load_url_params.h",
+        "android/media_players_observer.cc",
+        "android/media_players_observer.h",

[no sievers, 2015/10/20 19:24:22]

this is gone, but keep all media_player stuff

+        "android/overscroll_controller_android.cc",
+        "android/overscroll_controller_android.h",
+        "android/overscroll_glow.cc",
+        "android/overscroll_glow.h",
+        "android/overscroll_refresh.cc",
+        "android/overscroll_refresh.h",
+        "android/popup_touch_handle_drawable.cc",
+        "android/popup_touch_handle_drawable.h",
+        "android/tracing_controller_android.cc",
+        "android/tracing_controller_android.h",

[no sievers, 2015/10/20 19:24:22]

keep tracing_controller

+        "android/url_request_content_job.cc",
+        "android/url_request_content_job.h",

[no sievers, 2015/10/20 19:24:22]

keep url_request_content_job

+        "android/web_contents_observer_proxy.cc",
+        "android/web_contents_observer_proxy.h",
+        
+        "media/android/browser_media_player_manager.h",
+        "media/android/browser_media_player_manager.cc",
+        "media/android/media_session.h",
+        "media/android/media_session.cc",

[no sievers, 2015/10/20 19:24:22]

keep these

+  
+        "renderer_host/compositor_impl_android.cc",
+        "renderer_host/ime_adapter_android.cc",
+        "renderer_host/ime_adapter_android.h",
+        "renderer_host/input/synthetic_gesture_target_android.h",
+        "renderer_host/input/synthetic_gesture_target_android.cc",
+        "renderer_host/native_web_keyboard_event_android.cc",
+        "renderer_host/render_widget_host_view_android.cc",
+        "renderer_host/render_widget_host_view_android.h",
+        "screen_orientation/screen_orientation_delegate_android.cc",
+        "screen_orientation/screen_orientation_delegate_android.h",
+        "screen_orientation/screen_orientation_message_filter_android.cc",
+        "screen_orientation/screen_orientation_message_filter_android.h",

[no sievers, 2015/10/20 19:24:22]

this we should also try to keep (for screen orientation web api), but looks like
there are some unfortunate ContentViewCore deps.

+        "web_contents/web_contents_android.cc",
+        "web_contents/web_contents_android.h",
+        "web_contents/web_contents_view_android.cc",
+        "web_contents/web_contents_view_android.h",
+        "power_save_blocker_android.cc",

[no sievers, 2015/10/20 19:24:21]

let's file a bug for implementing power save blocker. we just need to call
setKeepsScreenOn() on some View attached to the activity window, I think.

+      ]
+    }
   } else {  # Not aura.
     sources -= [
       "renderer_host/compositor_resize_lock_aura.cc",
       "renderer_host/compositor_resize_lock_aura.h",
       "renderer_host/input/synthetic_gesture_target_aura.cc",
       "renderer_host/input/synthetic_gesture_target_aura.h",
       "renderer_host/input/touch_selection_controller_client_aura.cc",
       "renderer_host/input/touch_selection_controller_client_aura.h",
       "renderer_host/native_web_keyboard_event_aura.cc",
       "renderer_host/render_widget_host_view_aura.cc",
@@ skipping 75 matching lines @@
   if (enable_webvr) {
     sources += [
       "vr/vr_device.cc",
       "vr/vr_device.h",
       "vr/vr_device_manager.cc",
       "vr/vr_device_manager.h",
       "vr/vr_device_provider.h",
     ]
   }
 }