See also Bug 128928 but for now I'll only restore the basic functionality without making too many changes.
Created attachment 241286 [details] Patch
Comment on attachment 241286 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=241286&action=review LGTM > Tools/Scripts/webkitpy/port/linux_get_crash_log.py:44 > - cmd = ['gdb', '-ex', 'thread apply all bt 1024', '--batch', str(self._path_to_driver()), coredump_path] > + process_name = os.path.join(os.path.dirname(str(self._path_to_driver())), self.name) > + cmd = ['gdb', '-ex', 'thread apply all bt 1024', '--batch', process_name, coredump_path] why do you need str()? the path to driver should be a str already, otherwise it would be None, but in that case gdb will fail anyway. Wouldn't it be easier to pass bin_dir directly to GDBCrashLogGenerator instead of passing the driver only path to get its dirname?
Committed r175867: <http://trac.webkit.org/changeset/175867>
(In reply to comment #2) > Comment on attachment 241286 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=241286&action=review > > LGTM > > > Tools/Scripts/webkitpy/port/linux_get_crash_log.py:44 > > - cmd = ['gdb', '-ex', 'thread apply all bt 1024', '--batch', str(self._path_to_driver()), coredump_path] > > + process_name = os.path.join(os.path.dirname(str(self._path_to_driver())), self.name) > > + cmd = ['gdb', '-ex', 'thread apply all bt 1024', '--batch', process_name, coredump_path] > > why do you need str()? the path to driver should be a str already, otherwise > it would be None, but in that case gdb will fail anyway. Wouldn't it be > easier to pass bin_dir directly to GDBCrashLogGenerator instead of passing > the driver only path to get its dirname? Might be for unit-tests purposes. Anyway I didn't want to make too many changes in this patch, perhaps this could be fixed along in bug 128928.