New issue
Advanced search Search tips

Issue 885915 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Create testing for USB printing

Project Member Reported by valleau@chromium.org, Sep 19

Issue description

Create a framework to test USB printing in order to ensure stability between releases of Chrome OS.
 
Labels: M-71
Project Member

Comment 2 by bugdroid1@chromium.org, Oct 25

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/virtual-usb-printer/+/e20a913c29a358e366d7103bdda2a2f270016d43

commit e20a913c29a358e366d7103bdda2a2f270016d43
Author: David Valleau <valleau@chromium.org>
Date: Thu Oct 25 03:07:17 2018

virtual_usb_printer: Deleting original code and retaining license

BUG=chromium:885915
TEST=None

Change-Id: Id5832227c1a93752891ef27e8bee5e93c589c68c
Reviewed-on: https://chromium-review.googlesource.com/1296970
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Tested-by: David Valleau <valleau@chromium.org>
Reviewed-by: Sean Kau <skau@chromium.org>

[delete] https://crrev.com/e9a56869415a3b35f589e5fc1aaeac65ba5ed575/c/Makefile
[delete] https://crrev.com/e9a56869415a3b35f589e5fc1aaeac65ba5ed575/c/hid-keyboard.c
[rename] https://crrev.com/e20a913c29a358e366d7103bdda2a2f270016d43/LICENSE
[delete] https://crrev.com/e9a56869415a3b35f589e5fc1aaeac65ba5ed575/c/Makefile.cross
[delete] https://crrev.com/e9a56869415a3b35f589e5fc1aaeac65ba5ed575/c/cdc-acm.c
[delete] https://crrev.com/e9a56869415a3b35f589e5fc1aaeac65ba5ed575/c/hid-mouse.c
[delete] https://crrev.com/e9a56869415a3b35f589e5fc1aaeac65ba5ed575/python/LICENSE
[delete] https://crrev.com/e9a56869415a3b35f589e5fc1aaeac65ba5ed575/python/hid-keyboard.py
[delete] https://crrev.com/e9a56869415a3b35f589e5fc1aaeac65ba5ed575/python/USBIP.py
[delete] https://crrev.com/e9a56869415a3b35f589e5fc1aaeac65ba5ed575/python/hid-mouse.py
[delete] https://crrev.com/e9a56869415a3b35f589e5fc1aaeac65ba5ed575/c/usbip.c
[delete] https://crrev.com/e9a56869415a3b35f589e5fc1aaeac65ba5ed575/doc/usbip_protocol.txt
[delete] https://crrev.com/e9a56869415a3b35f589e5fc1aaeac65ba5ed575/c/usbip.h
[delete] https://crrev.com/e9a56869415a3b35f589e5fc1aaeac65ba5ed575/README.md

Project Member

Comment 3 by bugdroid1@chromium.org, Nov 1

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/virtual-usb-printer/+/6712557a0822b646980ff07a4cf2fe8caba1514c

commit 6712557a0822b646980ff07a4cf2fe8caba1514c
Author: David Valleau <valleau@chromium.org>
Date: Thu Nov 01 09:17:56 2018

Adding headers which define USB structs and constants

BUG=chromium:885915
TEST=None

Change-Id: I655a219c1d749ae069ec47199bf3b9831364aa2f
Reviewed-on: https://chromium-review.googlesource.com/1296971
Commit-Ready: David Valleau <valleau@chromium.org>
Tested-by: David Valleau <valleau@chromium.org>
Reviewed-by: Sean Kau <skau@chromium.org>

[add] https://crrev.com/6712557a0822b646980ff07a4cf2fe8caba1514c/usbip_constants.h
[add] https://crrev.com/6712557a0822b646980ff07a4cf2fe8caba1514c/device_descriptors.cc
[add] https://crrev.com/6712557a0822b646980ff07a4cf2fe8caba1514c/device_descriptors.h

Labels: -M-71 Bolton-Testing M-72
Project Member

Comment 5 by bugdroid1@chromium.org, Nov 16

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/virtual-usb-printer/+/a5be3c0b13f3703f8bf3a1131b9ca7d855766c86

commit a5be3c0b13f3703f8bf3a1131b9ca7d855766c86
Author: David Valleau <valleau@chromium.org>
Date: Fri Nov 16 13:02:03 2018

Adding constructors and equality operators for device descriptors

BUG=chromium:885915
TEST=None

Change-Id: I3f3f471f1613e89cb9fde3d06e317aa8f5495035
Reviewed-on: https://chromium-review.googlesource.com/1331648
Commit-Ready: Sean Kau <skau@chromium.org>
Tested-by: David Valleau <valleau@chromium.org>
Reviewed-by: Sean Kau <skau@chromium.org>

[modify] https://crrev.com/a5be3c0b13f3703f8bf3a1131b9ca7d855766c86/device_descriptors.cc
[modify] https://crrev.com/a5be3c0b13f3703f8bf3a1131b9ca7d855766c86/device_descriptors.h

Project Member

Comment 6 by bugdroid1@chromium.org, Dec 4

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/virtual-usb-printer/+/64c8149180ce76b0396c1453d1f67d390bb3d6a8

commit 64c8149180ce76b0396c1453d1f67d390bb3d6a8
Author: David Valleau <valleau@chromium.org>
Date: Tue Dec 04 04:32:01 2018

Adding usbip library which defines USB/IP messages and helper functions

BUG=chromium:885915
TEST=None

Change-Id: I90537bc17d084a7e6e0e5e67fcb0fb1a141b6b99
Reviewed-on: https://chromium-review.googlesource.com/1331649
Commit-Ready: David Valleau <valleau@chromium.org>
Tested-by: David Valleau <valleau@chromium.org>
Reviewed-by: David Valleau <valleau@chromium.org>

[add] https://crrev.com/64c8149180ce76b0396c1453d1f67d390bb3d6a8/usbip.h
[add] https://crrev.com/64c8149180ce76b0396c1453d1f67d390bb3d6a8/op_commands.cc
[add] https://crrev.com/64c8149180ce76b0396c1453d1f67d390bb3d6a8/op_commands.h
[add] https://crrev.com/64c8149180ce76b0396c1453d1f67d390bb3d6a8/usbip.cc

Project Member

Comment 7 by bugdroid1@chromium.org, Dec 4

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/virtual-usb-printer/+/d0e68fafc3565f67e30541a0502480d53cc060d2

commit d0e68fafc3565f67e30541a0502480d53cc060d2
Author: David Valleau <valleau@chromium.org>
Date: Tue Dec 04 04:32:12 2018

Adding a script which converts encoded IPP messages to JSON

BUG=chromium:885915
TEST=Manually tested output

Change-Id: Iaaa81a8e1af2cfba1f7190531fe36a1459b33a99
Reviewed-on: https://chromium-review.googlesource.com/1333944
Commit-Ready: David Valleau <valleau@chromium.org>
Tested-by: David Valleau <valleau@chromium.org>
Reviewed-by: David Valleau <valleau@chromium.org>

[add] https://crrev.com/d0e68fafc3565f67e30541a0502480d53cc060d2/tools/generate_ipp.py

Project Member

Comment 8 by bugdroid1@chromium.org, Dec 11

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/virtual-usb-printer/+/d6206031dfc5a412c4ebb344cff56e658bcaa38e

commit d6206031dfc5a412c4ebb344cff56e658bcaa38e
Author: David Valleau <valleau@chromium.org>
Date: Tue Dec 11 21:00:51 2018

Adding UsbPrinter class which defines a single USB printer device

This printer class supports basic USB printing. It understands the basic
USB control requests and receives BULK OUT transfers during a print job.

BUG=chromium:885915
TEST=None

Change-Id: Ib59f3212562de6d81ee9b46a9bce5e4c4629bb25
Reviewed-on: https://chromium-review.googlesource.com/1296972
Commit-Ready: Sean Kau <skau@chromium.org>
Tested-by: David Valleau <valleau@chromium.org>
Reviewed-by: Sean Kau <skau@chromium.org>

[add] https://crrev.com/d6206031dfc5a412c4ebb344cff56e658bcaa38e/usb_printer.h
[add] https://crrev.com/d6206031dfc5a412c4ebb344cff56e658bcaa38e/usb_printer.cc

Project Member

Comment 9 by bugdroid1@chromium.org, Dec 15

Labels: -M-72 M-73
Project Member

Comment 11 by bugdroid1@chromium.org, Dec 18

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/virtual-usb-printer/+/4822bacc399b07926ee963887a820a6a8fbb509a

commit 4822bacc399b07926ee963887a820a6a8fbb509a
Author: David Valleau <valleau@chromium.org>
Date: Tue Dec 18 04:47:11 2018

Adding value_util library for common base::Value and JSON functions

BUG=chromium:885915
TEST=None

Change-Id: Ibd1803f9e73f2f2d162873aa9c4974b2658b56a4
Reviewed-on: https://chromium-review.googlesource.com/1374373
Commit-Ready: David Valleau <valleau@chromium.org>
Tested-by: David Valleau <valleau@chromium.org>
Reviewed-by: Sean Kau <skau@chromium.org>

[add] https://crrev.com/4822bacc399b07926ee963887a820a6a8fbb509a/value_util.h
[add] https://crrev.com/4822bacc399b07926ee963887a820a6a8fbb509a/value_util.cc

Project Member

Comment 13 by bugdroid1@chromium.org, Dec 18

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/virtual-usb-printer/+/46c3f413e72f46dbf2adb62ff7d61e02ec8f9f5b

commit 46c3f413e72f46dbf2adb62ff7d61e02ec8f9f5b
Author: David Valleau <valleau@chromium.org>
Date: Tue Dec 18 08:42:59 2018

Adding SmartBuffer class used for packing socket messages

BUG=chromium:885915
TEST=Added unit tests for SmartBuffer class

Change-Id: I2953ad84da30c47f5d439115fe08c595c931a3b7
Reviewed-on: https://chromium-review.googlesource.com/1374371
Commit-Ready: David Valleau <valleau@chromium.org>
Tested-by: David Valleau <valleau@chromium.org>
Reviewed-by: Sean Kau <skau@chromium.org>

[add] https://crrev.com/46c3f413e72f46dbf2adb62ff7d61e02ec8f9f5b/smart_buffer_test.cc
[modify] https://crrev.com/46c3f413e72f46dbf2adb62ff7d61e02ec8f9f5b/usbip.cc
[modify] https://crrev.com/46c3f413e72f46dbf2adb62ff7d61e02ec8f9f5b/usbip_constants.h
[modify] https://crrev.com/46c3f413e72f46dbf2adb62ff7d61e02ec8f9f5b/op_commands.cc
[modify] https://crrev.com/46c3f413e72f46dbf2adb62ff7d61e02ec8f9f5b/op_commands.h
[modify] https://crrev.com/46c3f413e72f46dbf2adb62ff7d61e02ec8f9f5b/usb_printer.cc
[modify] https://crrev.com/46c3f413e72f46dbf2adb62ff7d61e02ec8f9f5b/device_descriptors.h
[add] https://crrev.com/46c3f413e72f46dbf2adb62ff7d61e02ec8f9f5b/smart_buffer.h
[add] https://crrev.com/46c3f413e72f46dbf2adb62ff7d61e02ec8f9f5b/smart_buffer.cc
[modify] https://crrev.com/46c3f413e72f46dbf2adb62ff7d61e02ec8f9f5b/server.cc
[modify] https://crrev.com/46c3f413e72f46dbf2adb62ff7d61e02ec8f9f5b/server.h

Project Member

Comment 14 by bugdroid1@chromium.org, Dec 22

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/695e30be8815bc47b164efa95c006953de177c5b

commit 695e30be8815bc47b164efa95c006953de177c5b
Author: David Valleau <valleau@chromium.org>
Date: Sat Dec 22 02:11:57 2018

Adding ebuild for usbip

This package will be used in tandem with a virtual USB printer in
order to create new automated tests of USB printing.

BUG=chromium:885915
TEST=Able to successfully emerge to an asuka device

Change-Id: I286afa8457f5e88cd4383657f43e5108820fd600
Reviewed-on: https://chromium-review.googlesource.com/1374991
Commit-Ready: David Valleau <valleau@chromium.org>
Tested-by: David Valleau <valleau@chromium.org>
Reviewed-by: Chirantan Ekbote <chirantan@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[add] https://crrev.com/695e30be8815bc47b164efa95c006953de177c5b/net-misc/usbip/usbip-4.13.ebuild
[add] https://crrev.com/695e30be8815bc47b164efa95c006953de177c5b/net-misc/usbip/Manifest

Project Member

Comment 15 by bugdroid1@chromium.org, Dec 22

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/portage-stable/+/3f4b596d5e079edd8da9527e91956ec4a5832625

commit 3f4b596d5e079edd8da9527e91956ec4a5832625
Author: David Valleau <valleau@chromium.org>
Date: Sat Dec 22 02:11:57 2018

Adding dev-manager ebuild from portage upstream

BUG=chromium:885915,chromium:623441
TEST=Emerged and manually tested on an asuka device

Change-Id: I84a10f74a04012166b164419020636fc146bc39b
Reviewed-on: https://chromium-review.googlesource.com/1387225
Commit-Ready: David Valleau <valleau@chromium.org>
Tested-by: David Valleau <valleau@chromium.org>
Reviewed-by: Chirantan Ekbote <chirantan@chromium.org>

[add] https://crrev.com/3f4b596d5e079edd8da9527e91956ec4a5832625/virtual/dev-manager/dev-manager-0-r1.ebuild

Project Member

Comment 17 by bugdroid1@chromium.org, Jan 17 (5 days ago)

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/virtual-usb-printer/+/928c153a1d7ed30a7ddac817dec201d48e425716

commit 928c153a1d7ed30a7ddac817dec201d48e425716
Author: David Valleau <valleau@chromium.org>
Date: Thu Jan 17 13:16:38 2019

Adding UsbDescriptors class to group descriptors and simplify UsbPrinter

BUG=chromium:885915
TEST=Tested virtual printer setup using usbprinter.AddBasic autotest

Change-Id: I28b14cfc9481705df2969d5bd2fa7aefd39ddf64
Reviewed-on: https://chromium-review.googlesource.com/1377589
Commit-Ready: Sean Kau <skau@chromium.org>
Tested-by: David Valleau <valleau@chromium.org>
Reviewed-by: Sean Kau <skau@chromium.org>

[modify] https://crrev.com/928c153a1d7ed30a7ddac817dec201d48e425716/usb_printer.h
[modify] https://crrev.com/928c153a1d7ed30a7ddac817dec201d48e425716/virtual_usb_printer.cc
[modify] https://crrev.com/928c153a1d7ed30a7ddac817dec201d48e425716/server.cc
[modify] https://crrev.com/928c153a1d7ed30a7ddac817dec201d48e425716/usb_printer.cc

Project Member

Comment 18 by bugdroid1@chromium.org, Jan 19 (3 days ago)

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/virtual-usb-printer/+/65c04c8522d03cf848065af1b862cfa83b085594

commit 65c04c8522d03cf848065af1b862cfa83b085594
Author: David Valleau <valleau@chromium.org>
Date: Sat Jan 19 12:20:25 2019

virtual-usb-printer: Adding ability to write received files

This change adds the "record_document_path" option to
virtual-usb-printer. This path is used to record the documents received
during a print job so that tests can verify that the job completed
successfully.

BUG=chromium:885915
TEST=Tested using basic usb print autotest in tast

Change-Id: I5d55f084c18b2ce5c9f38ea9db54cf26119d7958
Reviewed-on: https://chromium-review.googlesource.com/1413292
Commit-Ready: David Valleau <valleau@chromium.org>
Tested-by: David Valleau <valleau@chromium.org>
Reviewed-by: Sean Kau <skau@chromium.org>

[modify] https://crrev.com/65c04c8522d03cf848065af1b862cfa83b085594/usb_printer.h
[modify] https://crrev.com/65c04c8522d03cf848065af1b862cfa83b085594/virtual_usb_printer.cc
[modify] https://crrev.com/65c04c8522d03cf848065af1b862cfa83b085594/usb_printer.cc

Project Member

Comment 19 by bugdroid, Today (46 minutes ago)

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/virtual-usb-printer/+/f086c65c0cbdc0e5b72ec6e7cb763768b1ed8b51

commit f086c65c0cbdc0e5b72ec6e7cb763768b1ed8b51
Author: David Valleau <valleau@chromium.org>
Date: Wed Jan 23 08:20:31 2019

virtual-usb-printer: Adding a README file

BUG=chromium:885915
TEST=None

Change-Id: I13ecb4299f7f79d27b3d0e0a81fd776e0cbcca39
Reviewed-on: https://chromium-review.googlesource.com/1427349
Commit-Ready: David Valleau <valleau@chromium.org>
Tested-by: David Valleau <valleau@chromium.org>
Reviewed-by: Sean Kau <skau@chromium.org>

[add] https://crrev.com/f086c65c0cbdc0e5b72ec6e7cb763768b1ed8b51/README.md

Project Member

Comment 20 by bugdroid, Today (45 minutes ago)

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/dae0aceea710cd47f2e1f5eefb54014eb5218248

commit dae0aceea710cd47f2e1f5eefb54014eb5218248
Author: David Valleau <valleau@chromium.org>
Date: Wed Jan 23 08:20:30 2019

Adding ebuild for virtual_usb_printer

BUG=chromium:885915
TEST=None

Change-Id: I4a340b6a4abb61869fcf028a2ea679f2b9168b9c
Reviewed-on: https://chromium-review.googlesource.com/1374550
Commit-Ready: David Valleau <valleau@chromium.org>
Tested-by: David Valleau <valleau@chromium.org>
Reviewed-by: Chirantan Ekbote <chirantan@chromium.org>

[add] https://crrev.com/dae0aceea710cd47f2e1f5eefb54014eb5218248/chromeos-base/virtual-usb-printer/virtual-usb-printer-9999.ebuild

Project Member

Comment 21 by bugdroid, Today (25 minutes ago)

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/virtual-usb-printer/+/f086c65c0cbdc0e5b72ec6e7cb763768b1ed8b51

commit f086c65c0cbdc0e5b72ec6e7cb763768b1ed8b51
Author: David Valleau <valleau@chromium.org>
Date: Wed Jan 23 08:20:31 2019

virtual-usb-printer: Adding a README file

BUG=chromium:885915
TEST=None

Change-Id: I13ecb4299f7f79d27b3d0e0a81fd776e0cbcca39
Reviewed-on: https://chromium-review.googlesource.com/1427349
Commit-Ready: David Valleau <valleau@chromium.org>
Tested-by: David Valleau <valleau@chromium.org>
Reviewed-by: Sean Kau <skau@chromium.org>

[add] https://crrev.com/f086c65c0cbdc0e5b72ec6e7cb763768b1ed8b51/README.md

Project Member

Comment 22 by bugdroid, Today (24 minutes ago)

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/dae0aceea710cd47f2e1f5eefb54014eb5218248

commit dae0aceea710cd47f2e1f5eefb54014eb5218248
Author: David Valleau <valleau@chromium.org>
Date: Wed Jan 23 08:20:30 2019

Adding ebuild for virtual_usb_printer

BUG=chromium:885915
TEST=None

Change-Id: I4a340b6a4abb61869fcf028a2ea679f2b9168b9c
Reviewed-on: https://chromium-review.googlesource.com/1374550
Commit-Ready: David Valleau <valleau@chromium.org>
Tested-by: David Valleau <valleau@chromium.org>
Reviewed-by: Chirantan Ekbote <chirantan@chromium.org>

[add] https://crrev.com/dae0aceea710cd47f2e1f5eefb54014eb5218248/chromeos-base/virtual-usb-printer/virtual-usb-printer-9999.ebuild

Sign in to add a comment