Since some folks pinged me, this is about removing the Drive app API not the Drive app icon.

This is about removing the connected Drive apps of a user in the launcher, not changing search drive or the "Google Drive" app itself.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e81d6f1bbb48bc2bddb37d955662d34cd9de1823

commit e81d6f1bbb48bc2bddb37d955662d34cd9de1823
Author: Xiyuan Xia <xiyuan@chromium.org>
Date: Fri Dec 15 21:31:38 2017

app_list: Remove Drive app in launcher support

- Remove code that shows connected Drive apps in launcher;
- Update AppListSyncableService to remove sync data for those apps;

Bug:  794724 
Change-Id: I4e16df1d1e7dc216114174eae21cfac1b0eec189
Reviewed-on: https://chromium-review.googlesource.com/828026
Reviewed-by: Toni Barzic <tbarzic@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Reviewed-by: Yury Khmel <khmel@chromium.org>
Commit-Queue: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#524468}
[delete] https://crrev.com/c4dd10518c291c20a9881e6720d8b9a9505c2010/chrome/browser/apps/drive/DEPS
[delete] https://crrev.com/c4dd10518c291c20a9881e6720d8b9a9505c2010/chrome/browser/apps/drive/OWNERS
[delete] https://crrev.com/c4dd10518c291c20a9881e6720d8b9a9505c2010/chrome/browser/apps/drive/drive_app_converter.cc
[delete] https://crrev.com/c4dd10518c291c20a9881e6720d8b9a9505c2010/chrome/browser/apps/drive/drive_app_converter.h
[delete] https://crrev.com/c4dd10518c291c20a9881e6720d8b9a9505c2010/chrome/browser/apps/drive/drive_app_converter_browsertest.cc
[delete] https://crrev.com/c4dd10518c291c20a9881e6720d8b9a9505c2010/chrome/browser/apps/drive/drive_app_mapping.cc
[delete] https://crrev.com/c4dd10518c291c20a9881e6720d8b9a9505c2010/chrome/browser/apps/drive/drive_app_mapping.h
[delete] https://crrev.com/c4dd10518c291c20a9881e6720d8b9a9505c2010/chrome/browser/apps/drive/drive_app_mapping_unittest.cc
[delete] https://crrev.com/c4dd10518c291c20a9881e6720d8b9a9505c2010/chrome/browser/apps/drive/drive_app_provider.cc
[delete] https://crrev.com/c4dd10518c291c20a9881e6720d8b9a9505c2010/chrome/browser/apps/drive/drive_app_provider.h
[delete] https://crrev.com/c4dd10518c291c20a9881e6720d8b9a9505c2010/chrome/browser/apps/drive/drive_app_provider_browsertest.cc
[delete] https://crrev.com/c4dd10518c291c20a9881e6720d8b9a9505c2010/chrome/browser/apps/drive/drive_app_uninstall_sync_service.h
[delete] https://crrev.com/c4dd10518c291c20a9881e6720d8b9a9505c2010/chrome/browser/apps/drive/drive_service_bridge.cc
[delete] https://crrev.com/c4dd10518c291c20a9881e6720d8b9a9505c2010/chrome/browser/apps/drive/drive_service_bridge.h
[modify] https://crrev.com/e81d6f1bbb48bc2bddb37d955662d34cd9de1823/chrome/browser/extensions/BUILD.gn
[modify] https://crrev.com/e81d6f1bbb48bc2bddb37d955662d34cd9de1823/chrome/browser/prefs/browser_prefs.cc
[modify] https://crrev.com/e81d6f1bbb48bc2bddb37d955662d34cd9de1823/chrome/browser/ui/app_list/app_list_syncable_service.cc
[modify] https://crrev.com/e81d6f1bbb48bc2bddb37d955662d34cd9de1823/chrome/browser/ui/app_list/app_list_syncable_service.h
[modify] https://crrev.com/e81d6f1bbb48bc2bddb37d955662d34cd9de1823/chrome/browser/ui/app_list/app_list_syncable_service_factory.cc
[modify] https://crrev.com/e81d6f1bbb48bc2bddb37d955662d34cd9de1823/chrome/browser/ui/app_list/app_list_syncable_service_unittest.cc
[modify] https://crrev.com/e81d6f1bbb48bc2bddb37d955662d34cd9de1823/chrome/test/BUILD.gn
[modify] https://crrev.com/e81d6f1bbb48bc2bddb37d955662d34cd9de1823/ui/app_list/app_list_switches.cc
[modify] https://crrev.com/e81d6f1bbb48bc2bddb37d955662d34cd9de1823/ui/app_list/app_list_switches.h

Has this change rolled out? The Drive API is still seeing 40K Apps.List QPS coming from ChromeOS: http://shortn/_oT4Sisi6gb

CL in #5 is included in 65.0.3296.0. Chrome M65 is pushed to stable 100% this week.

There seems to be another code path in DriveIntegrationService using Drive Apps API,
e.g.
https://cs.chromium.org/chromium/src/chrome/browser/chromeos/drive/drive_integration_service.cc?rcl=2e4b433f9dac35b965e797247b8d15f287cad05c&l=409

Maybe we should also remove DriveAppRegistry from DriveIntegrationService too.

satorux@, what do you think?

DriveAppRegistry is used for Files app to open a file on Drive with a Drive app [1], so I think it should not be removed.

[1] https://cs.chromium.org/chromium/src/chrome/browser/chromeos/file_manager/file_tasks.h?type=cs

As part of fixing b/66499506, the Drive API no longer returns any Apps to ChromeOS for the Apps.List request (https://developers.google.com/drive/v2/reference/apps/list). The code that is calling Apps.List from ChromeOS should be removed since its not serving any purpose except generating empty load on Apiary and our server.

Ah ok. Filed a bug  issue 826090  for cleaning up the code

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ea971cb0900a0707294c9c761f8863696a87f292

commit ea971cb0900a0707294c9c761f8863696a87f292
Author: Xiyuan Xia <xiyuan@chromium.org>
Date: Fri Sep 21 20:15:46 2018

app_list: Remove Drive app clean up code

Bug:  794724 
Change-Id: Iba80e83e191f616af9331144a372e281dba49fa2
Reviewed-on: https://chromium-review.googlesource.com/1238441
Commit-Queue: Xiyuan Xia <xiyuan@chromium.org>
Reviewed-by: Yury Khmel <khmel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#593315}
[modify] https://crrev.com/ea971cb0900a0707294c9c761f8863696a87f292/chrome/browser/ui/app_list/app_list_syncable_service.cc
[modify] https://crrev.com/ea971cb0900a0707294c9c761f8863696a87f292/chrome/browser/ui/app_list/app_list_syncable_service.h
[modify] https://crrev.com/ea971cb0900a0707294c9c761f8863696a87f292/chrome/browser/ui/app_list/app_list_syncable_service_unittest.cc