[project_settings] Add "workspace.uuid" field.

Adapt the implementation to the latest design changes, and require a
"uuid" in addition to the "root". Also change the validation of the
`devtools.json` to require "workspace" to be either absent or an object
that contains both "root" and "uuid" string properties.

Doc: https://goo.gle/devtools-json-design
Bug: 395037775, 395562934
Change-Id: Ia73568152abe7150db3823d210470e791c6a7b90
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6264208
Commit-Queue: Alex Rudenko <[email protected]>
Auto-Submit: Benedikt Meurer <[email protected]>
Reviewed-by: Alex Rudenko <[email protected]>

Author: bmeurer

front_end/models/project_settings/ProjectSettingsModel.test.ts

@@ -93,7 +93,7 @@ describe('ProjectSettingsModel', () => {
     sinon.stub(frame, 'id').get(() => frameId);
 
     pageResourceLoader.loadResource.withArgs(url, sinon.match({target, frameId, initiatorUrl}))
-        .returns(Promise.resolve({content: '{"workspace":{"root":"/home/foo"}}'}));
+        .returns(Promise.resolve({content: '{"workspace":{"root":"/home/foo","uuid":"foo"}}'}));
 
     const projectSettingsModel = ProjectSettingsModel.instance({
       forceNew: true,
@@ -103,7 +103,7 @@ describe('ProjectSettingsModel', () => {
     });
 
     const projectSettings = await projectSettingsModel.projectSettingsPromise;
-    assert.deepEqual(projectSettings, {workspace: {root: '/home/foo'}});
+    assert.deepEqual(projectSettings, {workspace: {root: '/home/foo', uuid: 'foo'}});
   });
 
   it('listens for navigations', () => {

front_end/models/project_settings/ProjectSettingsModel.ts

@@ -13,7 +13,7 @@ import * as SDK from '../../core/sdk/sdk.js';
  * @see https://goo.gle/devtools-json-design
  */
 export interface ProjectSettings {
-  readonly workspace?: {readonly root?: string};
+  readonly workspace?: {readonly root: string, readonly uuid: string};
 }
 
 const EMPTY_PROJECT_SETTINGS: ProjectSettings = Object.freeze({});
@@ -140,8 +140,11 @@ export class ProjectSettingsModel extends Common.ObjectWrapper.ObjectWrapper<Eve
       if (typeof workspace !== 'object' || workspace === null) {
         throw new Error('Invalid "workspace" field');
       }
-      if (!['string', 'undefined'].includes(typeof workspace.root)) {
-        throw new Error('Invalid "workspace.root" field');
+      if (typeof workspace.root !== 'string') {
+        throw new Error('Invalid or missing "workspace.root" field');
+      }
+      if (typeof workspace.uuid !== 'string') {
+        throw new Error('Invalid or missing "workspace.uuid" field');
       }
     }
     return Object.freeze(devtoolsJSON);