Remove an unused method, make another private.

BUG=73098
R=sky@chromium.org

Review URL: https://codereview.chromium.org/427033002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286982 0039d316-1c4b-4281-b951-d872f2087c98

2 files changed, 4 insertions, 16 deletions

diff --git a/ui/base/win/shell.cc b/ui/base/win/shell.cc
index 7866e71..ee656ed 100644
--- a/ui/base/win/shell.cc
+++ b/ui/base/win/shell.cc
@@ -22,6 +22,8 @@
 namespace ui {
 namespace win {
 
+namespace {
+
 // Show the Windows "Open With" dialog box to ask the user to pick an app to
 // open the file with.
 bool OpenItemWithExternalApp(const base::string16& full_path) {
@@ -33,6 +35,8 @@ bool OpenItemWithExternalApp(const base::string16& full_path) {
   return (TRUE == ::ShellExecuteExW(&sei));
 }
 
+}  // namespace
+
 bool OpenAnyViaShell(const base::string16& full_path,
                      const base::string16& directory,
                      const base::string16& args,
@@ -57,11 +61,6 @@ bool OpenItemViaShell(const base::FilePath& full_path) {
                          base::string16(), 0);
 }
 
-bool OpenItemViaShellNoZoneCheck(const base::FilePath& full_path) {
-  return OpenAnyViaShell(full_path.value(), base::string16(), base::string16(),
-                         SEE_MASK_NOZONECHECKS | SEE_MASK_FLAG_DDEWAIT);
-}
-
 bool PreventWindowFromPinning(HWND hwnd) {
   // This functionality is only available on Win7+. It also doesn't make sense
   // to do this for Chrome Metro.
diff --git a/ui/base/win/shell.h b/ui/base/win/shell.h
index e732abf..3b64ce7 100644
--- a/ui/base/win/shell.h
+++ b/ui/base/win/shell.h
@@ -23,12 +23,6 @@ namespace win {
 // Returns 'true' on successful open, 'false' otherwise.
 UI_BASE_EXPORT bool OpenItemViaShell(const base::FilePath& full_path);
 
-// The download manager now writes the alternate data stream with the
-// zone on all downloads. This function is equivalent to OpenItemViaShell
-// without showing the zone warning dialog.
-UI_BASE_EXPORT bool OpenItemViaShellNoZoneCheck(
-    const base::FilePath& full_path);
-
 // Lower level function that allows opening of non-files like urls or GUIDs
 // don't use it if one of the above will do. |mask| is a valid combination
 // of SEE_MASK_FLAG_XXX as stated in msdn. If there is no default application
@@ -38,11 +32,6 @@ UI_BASE_EXPORT bool OpenAnyViaShell(const base::string16& full_path,
                                     const base::string16& args,
                                     DWORD mask);
 
-// Ask the user, via the Windows "Open With" dialog, for an application to use
-// to open the file specified by 'full_path'.
-// Returns 'true' on successful open, 'false' otherwise.
-bool OpenItemWithExternalApp(const base::string16& full_path);
-
 // Disables the ability of the specified window to be pinned to the taskbar or
 // the Start menu. This will remove "Pin this program to taskbar" from the
 // taskbar menu of the specified window.