WebApps: unity-chromium-extensions

Code review comment for lp://qastaging/~justinmcp/unity-chromium-extension/native-messaging-2

  1. native-messaging-2
  2. Merge into staging
Revision history for this message
Alberto Mardegan (mardy) wrote :

Overall looks good (just a couple of inline comments).
Can you please add a unit test for the createApplicationDesktopName() and maybe for the pong method?

review: Needs Fixing

« Back to merge proposal