debugd printer configuration should take ppd files by contents, not file handle |
|||
Issue descriptionRight now we jump through a lot of hoops to pass around ppds as file handles. We should, instead, just pass the ppd contents directly to debugd over dbus -- the size of the largest ppds is comfortably under the limits of dbus, and this eliminates a lot of inter-process coordination about file lifetimes.
,
Jan 25 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/platform/system_api/+/83f092867190983307074773a93cbcabb88886c3 commit 83f092867190983307074773a93cbcabb88886c3 Author: Justin Carlson <justincarlson@chromium.org> Date: Thu Jan 05 21:33:08 2017 debugd: Add constants for new debugd dbus hooks. These are needed to invoke the new dbus api functions being added in https://chromium-review.googlesource.com/#/c/420974. BUG= chromium:674731 TEST=none Change-Id: I0238e06886b6f5df2b4daeb3bbcafc1d18aa46e1 Reviewed-on: https://chromium-review.googlesource.com/424848 Commit-Ready: Justin Carlson <justincarlson@chromium.org> Tested-by: Justin Carlson <justincarlson@chromium.org> Reviewed-by: Dan Erat <derat@chromium.org> Reviewed-by: Sean Kau <skau@chromium.org> [modify] https://crrev.com/83f092867190983307074773a93cbcabb88886c3/dbus/debugd/dbus-constants.h
,
Jan 26 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8af171f93dabf82b5fd0cc566ac3796a7a5ba782 commit 8af171f93dabf82b5fd0cc566ac3796a7a5ba782 Author: justincarlson <justincarlson@chromium.org> Date: Thu Jan 26 00:58:18 2017 Update chromiumos system_api DEP to pull in recent debugd changes. BUG= chromium:674731 Review-Url: https://codereview.chromium.org/2657583006 Cr-Commit-Position: refs/heads/master@{#446182} [modify] https://crrev.com/8af171f93dabf82b5fd0cc566ac3796a7a5ba782/DEPS
,
Jan 26 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ca066a6bb4e4b2ab4d0c2cd316f4044fe2b36d78 commit ca066a6bb4e4b2ab4d0c2cd316f4044fe2b36d78 Author: justincarlson <justincarlson@chromium.org> Date: Thu Jan 26 17:29:15 2017 Update debugd client. New dbusd API calls are being added to debugd in https://chromium-review.googlesource.com/#/c/420974. This updates the chromium debugd client to expose them. Note this will be held for submission until after the M57 branch (Jan 19) BUG= 674731 Review-Url: https://codereview.chromium.org/2618683003 Cr-Commit-Position: refs/heads/master@{#446361} [modify] https://crrev.com/ca066a6bb4e4b2ab4d0c2cd316f4044fe2b36d78/chromeos/dbus/debug_daemon_client.cc [modify] https://crrev.com/ca066a6bb4e4b2ab4d0c2cd316f4044fe2b36d78/chromeos/dbus/debug_daemon_client.h [modify] https://crrev.com/ca066a6bb4e4b2ab4d0c2cd316f4044fe2b36d78/chromeos/dbus/fake_debug_daemon_client.cc [modify] https://crrev.com/ca066a6bb4e4b2ab4d0c2cd316f4044fe2b36d78/chromeos/dbus/fake_debug_daemon_client.h
,
Jan 27 2017
,
Feb 1 2017
|
|||
►
Sign in to add a comment |
|||
Comment 1 by skau@chromium.org
, Jan 20 2017