Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(225)

Unified Diff: visual_studio/NativeClientVSAddIn/UnitTests/TestUtilities.cs

Issue 11266051: Add PNaCl support for VS addin. (Closed) Base URL: http://nativeclient-sdk.googlecode.com/svn/trunk/src
Patch Set: fix nits and tests Created 8 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: visual_studio/NativeClientVSAddIn/UnitTests/TestUtilities.cs
diff --git a/visual_studio/NativeClientVSAddIn/UnitTests/TestUtilities.cs b/visual_studio/NativeClientVSAddIn/UnitTests/TestUtilities.cs
index 8a7eb00549282370199aef4485295c7d19d23f73..b3225ebc14c3e39734ff7e0a0a292586e157f17f 100644
--- a/visual_studio/NativeClientVSAddIn/UnitTests/TestUtilities.cs
+++ b/visual_studio/NativeClientVSAddIn/UnitTests/TestUtilities.cs
@@ -106,11 +106,13 @@ namespace UnitTests
Project proj = dte.Solution.Projects.Item(BlankNaClProjectUniqueName);
// Order matters if copying from the other Native Client type.
- if (pepperCopyFrom.Equals(NativeClientVSAddIn.Strings.NaCl64PlatformName))
+ if (pepperCopyFrom == NativeClientVSAddIn.Strings.NaCl64PlatformName)
{
proj.ConfigurationManager.AddPlatform(
NativeClientVSAddIn.Strings.NaCl64PlatformName, naclCopyFrom, true);
proj.ConfigurationManager.AddPlatform(
+ NativeClientVSAddIn.Strings.PNaClPlatformName, naclCopyFrom, true);
+ proj.ConfigurationManager.AddPlatform(
NativeClientVSAddIn.Strings.PepperPlatformName, pepperCopyFrom, true);
}
else
@@ -119,6 +121,8 @@ namespace UnitTests
NativeClientVSAddIn.Strings.PepperPlatformName, pepperCopyFrom, true);
proj.ConfigurationManager.AddPlatform(
NativeClientVSAddIn.Strings.NaCl64PlatformName, naclCopyFrom, true);
+ proj.ConfigurationManager.AddPlatform(
+ NativeClientVSAddIn.Strings.PNaClPlatformName, naclCopyFrom, true);
}
// Set the active solution configuration to Debug|NaCl64.

Powered by Google App Engine
This is Rietveld 408576698