Define PushMessagingApplicationId and replace ad-hoc code.

content/content_tests.gypi

 # Copyright (c) 2013 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 {
   'variables': {
     'layouttest_support_content_sources': [
       'public/test/layouttest_support.h',
       'public/test/nested_message_pump_android.cc',
       'public/test/nested_message_pump_android.h',
@@ skipping 510 matching lines @@
         'browser/media/capture/web_contents_audio_input_stream_unittest.cc',
         'browser/media/capture/web_contents_video_capture_device_unittest.cc',
         'browser/media/media_internals_unittest.cc',
         'browser/media/midi_host_unittest.cc',
         'browser/media/webrtc_identity_store_unittest.cc',
         'browser/net/sqlite_persistent_cookie_store_unittest.cc',
         'browser/notification_service_impl_unittest.cc',
         'browser/plugin_loader_posix_unittest.cc',
         'browser/power_monitor_message_broadcaster_unittest.cc',
         'browser/power_profiler/power_profiler_service_unittest.cc',
+        'browser/push_messaging_application_id_unittest.cc',

[fgorski, 2014/07/23 18:04:09]

your struct is in content/public/browser and tests are in content/browser
is that intended?

[Michael van Ouwerkerk, 2014/07/24 13:01:35]

Yes I did that intentionally. There aren't really any unit tests for the public
content API because they are supposed to contain no logic, no implementations.
So I was kind of bending the rules here, hoping for guidance from jam. I think I
got that guidance :-)

         'browser/quota/mock_quota_manager.cc',
         'browser/quota/mock_quota_manager.h',
         'browser/quota/mock_quota_manager_proxy.cc',
         'browser/quota/mock_quota_manager_proxy.h',
         'browser/quota/mock_quota_manager_unittest.cc',
         'browser/quota/quota_database_unittest.cc',
         'browser/quota/quota_backend_impl_unittest.cc',
         'browser/quota/quota_manager_unittest.cc',
         'browser/quota/quota_reservation_manager_unittest.cc',
         'browser/quota/quota_temporary_storage_evictor_unittest.cc',
@@ skipping 1307 matching lines @@
                   'files': ['<(PRODUCT_DIR)/libnpapi_test_plugin.so'],
                 },
               ],
             }],
           ],
         },
       ],
     }],
   ],
 }