PRESUBMIT: Split NATIVE_API_DIRS into two lists.

PRESUBMIT.py

 # Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
 #
 # Use of this source code is governed by a BSD-style license
 # that can be found in the LICENSE file in the root of the source
 # tree. An additional intellectual property rights grant can be found
 # in the file PATENTS.  All contributing project authors may
 # be found in the AUTHORS file in the root of the source tree.
 
 import json
 import os
@@ skipping 20 matching lines @@
   'webrtc/video',
 ]
 
 # List of directories of "supported" native APIs. That means changes to headers
 # will be done in a compatible way following this scheme:
 # 1. Non-breaking changes are made.
 # 2. The old APIs as marked as deprecated (with comments).
 # 3. Deprecation is announced to discuss-webrtc@googlegroups.com and
 #    webrtc-users@google.com (internal list).
 # 4. (later) The deprecated APIs are removed.
-# Directories marked as DEPRECATED should not be used. They're only present in
-# the list to support legacy downstream code.
 NATIVE_API_DIRS = (
+  'webrtc',
+  'webrtc/api',
+  'webrtc/media',
+  'webrtc/modules/audio_device/include',
+  'webrtc/pc',
+)
+# These directories should not be used but are maintained only to avoid breaking
+# some legacy downstream code.

[pthatcher1, 2016/05/12 03:43:13]

We should point out that if something in the NATIVE_API_DIRS folders exposes
something below through its methods, then those things exposed can be used.  For
example, scoped_ptr or sigslots, which are contained in webrtc/base.

[the sun, 2016/05/12 12:42:41]

I don't think we need to point that out at this time, it will be a natural
consequence of using the thing in NATIVE_API_DIRS, and will be sorted out once
we've move all de-facto external APIs into api/.

+LEGACY_API_DIRS = (
   'talk/app/webrtc',
-  'webrtc',
-  'webrtc/base',  # DEPRECATED.
-  'webrtc/common_audio/include',  # DEPRECATED.
+  'webrtc/base',
+  'webrtc/common_audio/include',
   'webrtc/modules/audio_coding/include',
-  'webrtc/modules/audio_conference_mixer/include',  # DEPRECATED.
-  'webrtc/modules/audio_device/include',
+  'webrtc/modules/audio_conference_mixer/include',
   'webrtc/modules/audio_processing/include',
   'webrtc/modules/bitrate_controller/include',
   'webrtc/modules/include',
   'webrtc/modules/remote_bitrate_estimator/include',
   'webrtc/modules/rtp_rtcp/include',
-  'webrtc/modules/rtp_rtcp/source',  # DEPRECATED.
+  'webrtc/modules/rtp_rtcp/source',
   'webrtc/modules/utility/include',
   'webrtc/modules/video_coding/codecs/h264/include',
   'webrtc/modules/video_coding/codecs/i420/include',
   'webrtc/modules/video_coding/codecs/vp8/include',
   'webrtc/modules/video_coding/codecs/vp9/include',
   'webrtc/modules/video_coding/include',
-  'webrtc/system_wrappers/include',  # DEPRECATED.
+  'webrtc/system_wrappers/include',
   'webrtc/voice_engine/include',
 )
+API_DIRS = NATIVE_API_DIRS[:] + LEGACY_API_DIRS[:]
 
 
 def _VerifyNativeApiHeadersListIsValid(input_api, output_api):
   """Ensures the list of native API header directories is up to date."""
   non_existing_paths = []
   native_api_full_paths = [
       input_api.os_path.join(input_api.PresubmitLocalPath(),
-                             *path.split('/')) for path in NATIVE_API_DIRS]
+                             *path.split('/')) for path in API_DIRS]
   for path in native_api_full_paths:
     if not os.path.isdir(path):
       non_existing_paths.append(path)
   if non_existing_paths:
     return [output_api.PresubmitError(
         'Directories to native API headers have changed which has made the '
         'list in PRESUBMIT.py outdated.\nPlease update it to the current '
         'location of our native APIs.',
         non_existing_paths)]
   return []
 
 
 def _CheckNativeApiHeaderChanges(input_api, output_api):
   """Checks to remind proper changing of native APIs."""
   files = []
   for f in input_api.AffectedSourceFiles(input_api.FilterSourceFile):
     if f.LocalPath().endswith('.h'):
-      for path in NATIVE_API_DIRS:
+      for path in API_DIRS:
         if os.path.dirname(f.LocalPath()) == path:
           files.append(f)
 
   if files:
     return [output_api.PresubmitNotifyResult(
         'You seem to be changing native API header files. Please make sure '
         'you:\n'
         '  1. Make compatible changes that don\'t break existing clients.\n'
         '  2. Mark the old APIs as deprecated.\n'
         '  3. Create a timeline and plan for when the deprecated method will '
@@ skipping 393 matching lines @@
     for builder in masters[master]:
       if 'presubmit' in builder:
         # Do not trigger presubmit builders, since they're likely to fail
         # (e.g. OWNERS checks before finished code review), and we're running
         # local presubmit anyway.
         pass
       else:
         try_config[master][builder] = ['defaulttests']
 
   return try_config