Remove MOJO_STATIC_CONST_MEMBER_DEFINITION.

mojo/public/cpp/system/tests/macros_unittest.cc

Index: mojo/public/cpp/system/tests/macros_unittest.cc
diff --git a/mojo/public/cpp/system/tests/macros_unittest.cc b/mojo/public/cpp/system/tests/macros_unittest.cc
index ab4ca80c121276699dc516349bd525a1094edc34..8ee849038a0f5815dfb7346c5e668f06a65e8334 100644
--- a/mojo/public/cpp/system/tests/macros_unittest.cc
+++ b/mojo/public/cpp/system/tests/macros_unittest.cc
@@ -19,19 +19,6 @@
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace mojo {
-
-// The test for |MOJO_STATIC_CONST_MEMBER_DEFINITION| is really a compile/link
-// test. To test it fully would really require a header file and multiple .cc
-// files, but we'll just cursorily verify it.
-//
-// This is defined outside of an anonymous namespace because
-// MOJO_STATIC_CONST_MEMBER_DEFINITION may not be used on internal symbols.
-struct StructWithStaticConstMember {
-  static const int kStaticConstMember = 123;
-};
-MOJO_STATIC_CONST_MEMBER_DEFINITION
-const int StructWithStaticConstMember::kStaticConstMember;
-
 namespace {
 
 // Note: MSVS is very strict (and arguably buggy) about warnings for classes
@@ -140,12 +127,6 @@ TEST(MacrosCppTest, MoveOnlyType) {
   EXPECT_EQ(123, z.value());
 }
 
-// Use it, to make sure things get linked in and to avoid any warnings about
-// unused things.
-TEST(MacrosCppTest, StaticConstMemberDefinition) {
-  EXPECT_EQ(123, StructWithStaticConstMember::kStaticConstMember);
-}
-
 // The test for |ignore_result()| is also just a compilation test. (Note that
 // |MOJO_WARN_UNUSED_RESULT| can only be used in the prototype.
 int ReturnsIntYouMustUse() MOJO_WARN_UNUSED_RESULT;