(TOBEDELETED)Select downloaded file in the folder.

chrome/browser/platform_util.h

Index: chrome/browser/platform_util.h
diff --git a/chrome/browser/platform_util.h b/chrome/browser/platform_util.h
index 81cb4b8cf0f966aa5fbae381ee140f84f9766beb..336feda121d160ebc1ab38dac04f4848adcdb067 100644
--- a/chrome/browser/platform_util.h
+++ b/chrome/browser/platform_util.h
@@ -37,7 +37,7 @@ enum OpenOperationResult {
 };
 
 // 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;