Only update symbols on current thread when breaking or stepping in GDB.

mojo/devtools/common/android_gdb/session.py

Index: mojo/devtools/common/android_gdb/session.py
diff --git a/mojo/devtools/common/android_gdb/session.py b/mojo/devtools/common/android_gdb/session.py
index bc05c291b0e8a58b9625225504298d7945328aa3..08aebccb2749f02f7f457642e967a711e08feb57 100644
--- a/mojo/devtools/common/android_gdb/session.py
+++ b/mojo/devtools/common/android_gdb/session.py
@@ -190,13 +190,31 @@ class DebugSession(object):
       return True
     return False
 
-  def _update_symbols(self):
+  def _find_mappings_current_thread(self, mapped_files):

[ppi, 2015/07/23 23:08:31]

this method is called "find", but doesn't return anything? please add a comment
explaining what it does.

Maybe this should be called _map_symbols_in_current_thread?

[etiennej, 2015/07/23 23:21:03]

Changed the method name.

+      frame = gdb.newest_frame()
+      while frame and frame.is_valid():
+        if frame.name() is None:
+          m = self._find_mapping_for_address(mapped_files, frame.pc())
+          if m is not None and self._try_to_map(m):
+            # Force gdb to recompute its frames.
+            _gdb_execute("info threads")
+            frame = gdb.newest_frame()
+            assert frame.is_valid()
+        if (frame.older() is not None and
+            frame.older().is_valid() and
+            frame.older().pc() != frame.pc()):
+          frame = frame.older()
+        else:
+          frame = None
+
+  def update_symbols(self, current_thread_only):
     """Updates the mapping between symbols as seen from GDB and local library
-    files."""
+    files.
+
+    If current_thread_only is True, only update symbols for the current thread.
+    """
     logging.info("Updating symbols")
     mapped_files = _get_mapped_files()
-    _gdb_execute("info threads")
-    nb_threads = len(_gdb_execute("info threads").split("\n")) - 2
     # Map all symbols from native libraries packages with the APK.
     for file_mappings in mapped_files:
       filename = file_mappings[0].filename
@@ -204,28 +222,22 @@ class DebugSession(object):
            filename.startswith('/data/app')) and
           not filename.endswith('.apk') and
           not filename.endswith('.dex')):
-        logging.info('Pre-mapping: %s' % file_mappings[0].filename)
-        self._try_to_map(file_mappings)
-    for i in xrange(nb_threads):
-      try:
-        _gdb_execute("thread %d" % (i + 1))
-        frame = gdb.newest_frame()
-        while frame and frame.is_valid():
-          if frame.name() is None:
-            m = self._find_mapping_for_address(mapped_files, frame.pc())
-            if m is not None and self._try_to_map(m):
-              # Force gdb to recompute its frames.
-              _gdb_execute("info threads")
-              frame = gdb.newest_frame()
-              assert frame.is_valid()
-          if (frame.older() is not None and
-              frame.older().is_valid() and
-              frame.older().pc() != frame.pc()):
-            frame = frame.older()
-          else:
-            frame = None
-      except gdb.error:
-        traceback.print_exc()
+          logging.info('Pre-mapping: %s' % file_mappings[0].filename)

[ppi, 2015/07/23 23:08:31]

Why this shift to the right? Not sure which one is right, but I think the old
alingment was easier to read.

[etiennej, 2015/07/23 23:21:03]

Done.

+          self._try_to_map(file_mappings)
+
+    if current_thread_only:
+      self._find_mappings_current_thread(mapped_files)
+    else:
+      logging.info('Updating all threads\' symbols')
+      current_thread = gdb.selected_thread()
+      nb_threads = len(_gdb_execute("info threads").split("\n")) - 2
+      for i in xrange(nb_threads):
+        try:
+          _gdb_execute("thread %d" % (i + 1))
+          self._find_mappings_current_thread(mapped_files)
+        except gdb.error:
+          traceback.print_exc()
+      current_thread.switch()
 
   def _get_device_application_pid(self, application):
     """Gets the PID of an application running on a device."""
@@ -266,8 +278,38 @@ class DebugSession(object):
 
     _gdb_execute('target remote localhost:9999')
 
-    self._update_symbols()
+    self.update_symbols(False)

[ppi, 2015/07/23 23:08:31]

nit: (current_thread_only=True), similarly below

[ppi, 2015/07/23 23:09:32]

I meant (current_thread_only=False).

[etiennej, 2015/07/23 23:21:03]

Done.

     def on_stop(_):
-      self._update_symbols()
+      self.update_symbols(True)
     gdb.events.stop.connect(on_stop)
     gdb.events.exited.connect(self.stop)
+
+    # Register the update-symbols command.
+    UpdateSymbols(self)
+
+class UpdateSymbols(gdb.Command):
+  """Command to update symbols loaded into GDB.
+
+  GDB usage: update-symbols [all|current]
+  """
+  _UPDATE_COMMAND = "update-symbols"
+
+  def __init__(self, session):
+    super(UpdateSymbols, self).__init__(self._UPDATE_COMMAND, gdb.COMMAND_STACK)
+    self._session = session
+
+  def invoke(self, arg, _unused_from_tty):
+    if arg == 'current':
+      self._session.update_symbols(True)

[ppi, 2015/07/23 23:08:31]

nit: (current_thread_only=True), similarly below

[etiennej, 2015/07/23 23:21:03]

Done.

+    else:
+      self._session.update_symbols(False)
+
+  def complete(self, text, _unused_word):
+    if text == self._UPDATE_COMMAND:
+      return ('all', 'current')
+    elif text in self._UPDATE_COMMAND + ' all':
+      return ['all']
+    elif text in self._UPDATE_COMMAND + ' current':
+      return ['current']
+    else:
+      return []