(TOBEDELETED)Select downloaded file in the folder.

chrome/browser/platform_util.cc

 // Copyright 2015 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.
 
 #include "chrome/browser/platform_util.h"
 
 #include "base/files/file.h"
 #include "base/files/file_util.h"
 #include "base/logging.h"
 #include "chrome/browser/platform_util_internal.h"
@@ skipping 10 matching lines @@
 void VerifyAndOpenItemOnBlockingThread(const base::FilePath& path,
                                        OpenItemType type,
                                        const OpenOperationCallback& callback) {
   base::File target_item(path, base::File::FLAG_OPEN | base::File::FLAG_READ);
   if (!base::PathExists(path)) {
     if (!callback.is_null())
       BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
                               base::Bind(callback, OPEN_FAILED_PATH_NOT_FOUND));
     return;
   }
-  if (base::DirectoryExists(path) != (type == OPEN_FOLDER)) {

[Evan Stade, 2016/04/07 16:58:58]

why this change?

[maksims (do not use this acc), 2016/04/14 08:30:33]

Because OpenItem() function used to send path.DirName(), but now it sends  full
path with the file name inside. 

OpenItem(profile, full_path.DirName(), OPEN_FOLDER,OpenOperationCallback()); ->
OpenItem(profile, full_path, OPEN_FOLDER,OpenOperationCallback());	

If there is (base::DirectoryExists(path) != (type == OPEN_FOLDER), then
OPEN_FAILED_INVALID_TYPE is called.

[Lei Zhang, 2016/04/15 01:39:24]

Sorry, but VerifyAndOpenItemOnBlockingThread(), basically OpenItem()'s
implementation, is used in many places. You can't just change it to suit your
own needs without considering the other callers.

Likely ShowItemInFolder() can't just simply reply on OpenItem() anymore.

I'd suggest ShowItemInFolder() first try to figure out if the file manager is
Nautilus. If it is not, then do the current behavior and ShowItemInFolder() for
the folder. For Nautilus, do something else.

+  if (base::DirectoryExists(path)) {
     if (!callback.is_null())
       BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
                               base::Bind(callback, OPEN_FAILED_INVALID_TYPE));
     return;
   }
 
   if (shell_operations_allowed)
     internal::PlatformOpenVerifiedItem(path, type);
   if (!callback.is_null())
     BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
@@ skipping 14 matching lines @@
               const base::FilePath& full_path,
               OpenItemType item_type,
               const OpenOperationCallback& callback) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   BrowserThread::PostBlockingPoolTask(
       FROM_HERE, base::Bind(&VerifyAndOpenItemOnBlockingThread, full_path,
                             item_type, callback));
 }
 
 }  // namespace platform_util