Move FS related mojo methods to ArcFileSystemInstance.

components/arc/common/intent_helper.mojom

 // Copyright 2016 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.
 //
 // Next MinVersion: 16
 
 module arc.mojom;
 
 import "scale_factor.mojom";
 
@@ skipping 98 matching lines @@
 
 // Sends intents to ARC on behalf of Chrome.
 // Next method ID: 13
 interface IntentHelperInstance {
   // Sets the given package as a preferred package. The next time an ACTION_VIEW
   // intent is sent with a URL that requires disambiguation, instead of opening
   // the ResolverActivity, this package will be picked if it is on the list.
   // When multiple packages are set as preferred, the most recent setting wins.
   [MinVersion=7] AddPreferredPackage@8(string package_name);
 
-  // Asks the ContentResolver for the size of the file specified by the URL.
+  // DEPRECATED. Use FileSystemInstance.GetFileSize() instead.
   [MinVersion=15] GetFileSize@11(string url) => (int64 size);

[Luis Héctor Chávez, 2016/11/18 16:37:53]

Can you add the Deprecated suffix to this method please?

[Shuhei Takahashi, 2016/11/21 11:06:29]

Sure, done.

 
   // Passes an intent to an activity.
   [MinVersion=12] HandleIntent@10(IntentInfo intent, ActivityName activity);
 
   // Handles the URL by sending an ACTION_VIEW intent to the package. The
   // most suitable activity for the URL within the package will be started.
   [MinVersion=2] HandleUrl@2(string url, string package_name);
 
   // Handles the list of URLs by sending a specified intent to the handler.
   [MinVersion=5] HandleUrlList@7(array<UrlWithMimeType> urls,
                                  ActivityName activity,
                                  ActionType action_type);
 
   // Establishes full-duplex communication with the host.
   Init@0(IntentHelperHost host_ptr);
 
-  // Asks the ContentResolver to get an FD to read the file specified by the
-  // URL.
+  // DEPRECATED. Use FileSystemInstance.OpenFileToRead() instead.
   [MinVersion=15] OpenFileToRead@12(string url) => (handle? fd);

[Yusuke Sato, 2016/11/18 19:09:38]

same, OpenFileToReadDeprecated

[Shuhei Takahashi, 2016/11/21 11:06:29]

Done.

 
   // Requests 48dp * 48dp icons of the |activities| suitable for the
   // |scale_factor|. An array of icon data will be returned.
   [MinVersion=3] RequestActivityIcons@4(array<ActivityName> activities,
                                         ScaleFactor scale_factor)
       => (array<ActivityIcon> icons);
 
   // Requests a list of packages that can handle |intent|.
   [MinVersion=12] RequestIntentHandlerList@9(IntentInfo intent)
       => (array<IntentHandlerInfo> handlers);
@@ skipping 10 matching lines @@
   // specified.  Data can be sent as extras by including a JSON map string which
   // will be automatically converted to a bundle accessible by the receiver.
   //
   // Note: Broadcasts can only be sent to whitelisted packages.  Packages can be
   // added to the whitelist in ArcBridgeService.java in the Android source.
   [MinVersion=1] SendBroadcast@1(string action,
                                  string package_name,
                                  string cls,
                                  string extras);
 };