commit | e07d807c1d0ae31a9e5206f8220b2de5c100fdc7 | [log] [tgz] |
---|---|---|
author | Asanka Herath <asanka@chromium.org> | Wed Apr 11 13:20:20 2018 |
committer | Asanka Herath <asanka@chromium.org> | Wed Apr 11 13:20:20 2018 |
tree | c9a5364a5f84df20665fb7b53898858f1c741dab | |
parent | 8a6b5b2d7bd38cb2b49510a98cd4c38d26389b37 [diff] |
Remove usage of SEE_MASK_FLAG_NO_UI from ui::base::win The flag was originally added to suppress messages from popping up if the Windows shell failed to open the specified item. Specifically, the ui::base::win::OpenAnyViaShell() function handles the case where ShellExecute() returns ERROR_NO_ASSOCIATION when invoked with the "open" verb by attempting to invoke the same using the "openas" verb. Thus an intervening error message would not have been useful. However the SEE_MASK_FLAG_NO_UI flag may suppress other error messages and may also have side-effects beyond the display of failure messages. See bug for details. Bug: 819809 Change-Id: I8def155aa11b14f9a57a8f66add193dc281971cb Reviewed-on: https://chromium-review.googlesource.com/966963 Commit-Queue: Asanka Herath <asanka@chromium.org> Reviewed-by: Eric Lawrence <elawrence@chromium.org> Reviewed-by: Scott Violet <sky@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#548039}(cherry picked from commit 5bcdfd9c5595a61f67faece0a8305b9c91f4326c) Reviewed-on: https://chromium-review.googlesource.com/1007174 Reviewed-by: Asanka Herath <asanka@chromium.org> Cr-Commit-Position: refs/branch-heads/3359@{#679} Cr-Branched-From: 66afc5e5d10127546cc4b98b9117aff588b5e66b-refs/heads/master@{#540276}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .