New issue
Advanced search Search tips

Issue 655281 link

Starred by 1 user
Status: Fixed
Owner:
erikc...@chromium.org
Closed: Oct 2016
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 547071



Sign in to add a comment

Remove deprecated methods from chrome/test/base/interactive_test_utils_mac.mm

Project Member Reported by erikc...@chromium.org, Oct 12 2016 
[40606/40640] OBJCXX obj/chrome/test/interactive_ui_tests/interactive_test_utils_mac.o
../../chrome/test/base/interactive_test_utils_mac.mm:24:3: warning: 'SetFrontProcess' is deprecated: first deprecated in macOS 10.9 [-Wdeprecated-declarations]
  SetFrontProcess(&psn);
  ^
/Users/erikchen/projects/osx_sdks/MacOSX10.10.sdk/System/Library/Frameworks/ApplicationServices.framework/Frameworks/HIServices.framework/Headers/Processes.h:603:1: note: 'SetFrontProcess' has been explicitly marked deprecated here
SetFrontProcess(const ProcessSerialNumber * PSN)              AVAILABLE_MAC_OS_X_VERSION_10_0_AND_LATER_BUT_DEPRECATED_IN_MAC_OS_X_VERSION_10_9;
^
1 warning generated.
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 12 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/aa6dec0f8ff61d3e46d2c3294b257d8523eaf433

commit aa6dec0f8ff61d3e46d2c3294b257d8523eaf433
Author: erikchen <erikchen@chromium.org>
Date: Wed Oct 12 21:25:46 2016

Fix a deprecated declaration warning in interactive_test_utils_mac.mm.

The method SetFrontProcess() can be replaced by -[NSApplication
activateIgnoringOtherApps:].

BUG= 655281 

Review-Url: https://codereview.chromium.org/2419673002
Cr-Commit-Position: refs/heads/master@{#424861}

[modify] https://crrev.com/aa6dec0f8ff61d3e46d2c3294b257d8523eaf433/chrome/test/base/interactive_test_utils_mac.mm

Comment 2 by erikc...@chromium.org, Oct 13 2016

Blocking: 547071
Status: Fixed (was: Assigned)
►

Sign in to add a comment