Make sure that direct call to SendStack() doesn't send a stack trace for non googler

breakpad.py

 # Copyright (c) 2011 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.
 
 """Breakpad for Python.
 
 Sends a notification when a process stops on an exception.
 
 It is only enabled when all these conditions are met:
   1. hostname finishes with '.google.com' or 'chromium.org'
@@ skipping 15 matching lines @@
 # Configure these values.
 DEFAULT_URL = 'https://chromium-status.appspot.com'
 
 # Global variable to prevent double registration.
 _REGISTERED = False
 
 _TIME_STARTED = time.time()
 
 _HOST_NAME = socket.getfqdn()
 
+# Skip unit tests and we don't want anything from non-googler.
+IS_ENABLED = (
+    not 'test' in getattr(sys.modules['__main__'], '__file__', '') and
+    not 'NO_BREAKPAD' in os.environ and
+    _HOST_NAME.endswith(('.google.com', '.chromium.org')))
+
 
 def post(url, params):
   """HTTP POST with timeout when it's supported."""
   kwargs = {}
   if (sys.version_info[0] * 10 + sys.version_info[1]) >= 26:
     kwargs['timeout'] = 4
   request = urllib2.urlopen(url, urllib.urlencode(params), **kwargs)
   out = request.read()
   request.close()
   return out
@@ skipping 22 matching lines @@
         out += '\nread(): %s' % e.read()
       if hasattr(e, 'info') and callable(e.info):
         out += '\ninfo(): %s' % e.info()
   except Exception:
     pass
   return out
 
 
 def SendStack(last_tb, stack, url=None, maxlen=50):
   """Sends the stack trace to the breakpad server."""
+  if not IS_ENABLED:
+    # Make sure to not send anything for non googler.
+    return
   if not url:
     url = DEFAULT_URL + '/breakpad'
   print 'Sending crash report ...'
   try:
     params = {
         'args': sys.argv,
         'stack': stack[0:4096],
         'user': getpass.getuser(),
         'exception': FormatException(last_tb),
         'host': _HOST_NAME,
@@ skipping 38 matching lines @@
 
 def Register():
   """Registers the callback at exit. Calling it multiple times is no-op."""
   global _REGISTERED
   if _REGISTERED:
     return
   _REGISTERED = True
   atexit.register(CheckForException)
 
 
-# Skip unit tests and we don't want anything from non-googler.
-if (not 'test' in getattr(sys.modules['__main__'], '__file__', '') and
-    not 'NO_BREAKPAD' in os.environ and
-    (_HOST_NAME.endswith('.google.com') or
-     _HOST_NAME.endswith('.chromium.org'))):
+if IS_ENABLED:
   Register()
 
 # Uncomment this line if you want to test it out.
 #Register()