(TOBEDELETED)Select downloaded file in the folder.

chrome/browser/platform_util.h

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_BROWSER_PLATFORM_UTIL_H_
 #define CHROME_BROWSER_PLATFORM_UTIL_H_
 
 #include <string>
 
 #include "base/callback_forward.h"
@@ skipping 19 matching lines @@
                              // was expected. I.e. OpenFile was called on a
                              // folder or OpenFolder called on a file.
   OPEN_FAILED_NO_HANLDER_FOR_FILE_TYPE,  // There was no file handler capable of
                                          // opening file. Only returned on
                                          // ChromeOS.
   OPEN_FAILED_FILE_ERROR,  // Open operation failed due to some other file
                            // error.
 };
 
 // Type of item that is the target of the OpenItem() call.
-enum OpenItemType { OPEN_FILE, OPEN_FOLDER };
+enum OpenItemType { OPEN_FILE, OPEN_FOLDER, SHOW_ITEM_IN_FOLDER };

[Lei Zhang, 2016/04/19 21:51:25]

You may want to make SHOW_ITEM_IN_FOLDER Linux-only. As is, it's not used on
other platforms, but compilers may notice switch statements that handle an
OpenItemType are forgetting to handle this enum value, and complain.

[maksims (do not use this acc), 2016/04/20 12:02:45]

Done.

[Lei Zhang, 2016/04/20 20:45:57]

Consider if the enum had many different values. Would you repeat them like you
did here?

How about:

enum OpenItemType {
  OPEN_FILE,
  OPEN_FOLDER,
#if defined(OS_LINUX)
  SHOW_ITEM_IN_FOLDER,
#endif
};

 
 // Callback used with OpenFile and OpenFolder.
 typedef base::Callback<void(OpenOperationResult)> OpenOperationCallback;
 
 // Opens the item specified by |full_path|, which is expected to be the type
 // indicated by |item_type| in the desktop's default manner.
 // |callback| will be invoked on the UI thread with the result of the open
 // operation.
 //
 // It is an error if the object at |full_path| does not match the intended type
@@ skipping 48 matching lines @@
 // the feature is supported and enabled, and false otherwise.
 bool IsSwipeTrackingFromScrollEventsEnabled();
 #endif
 
 #if BUILDFLAG(ANDROID_JAVA_UI)
 bool RegisterPlatformUtil(JNIEnv* env);
 #endif
 }  // namespace platform_util
 
 #endif  // CHROME_BROWSER_PLATFORM_UTIL_H_