Bug 1564138 - Fix DevTools PropTypes errors when updating eslint. r=Ola.
authorNicolas Chevobbe <nchevobbe@mozilla.com>
Wed, 17 Jul 2019 10:35:44 +0000
changeset 483106 238a8eb35207e07a52db336b8440776f987fca8f
parent 483105 c8100315938bdd082faaf33a213addbc08d02f2a
child 483107 d0cf03130c937cb7a83662ea857286b8cc0adf12
push id36307
push useraciure@mozilla.com
push dateThu, 18 Jul 2019 03:45:56 +0000
treeherdermozilla-central@3d0e35ab6850 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersOla
bugs1564138
milestone70.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1564138 - Fix DevTools PropTypes errors when updating eslint. r=Ola. Differential Revision: https://phabricator.services.mozilla.com/D38093
devtools/client/application/src/components/Worker.js
devtools/client/shared/components/List.js
devtools/client/shared/components/tree/TreeRow.js
devtools/client/shared/components/tree/TreeView.js
devtools/client/webconsole/components/Output/Message.js
package-lock.json
package.json
--- a/devtools/client/application/src/components/Worker.js
+++ b/devtools/client/application/src/components/Worker.js
@@ -53,16 +53,18 @@ class Worker extends Component {
   static get propTypes() {
     return {
       client: PropTypes.instanceOf(DebuggerClient).isRequired,
       isDebugEnabled: PropTypes.bool.isRequired,
       worker: PropTypes.shape({
         active: PropTypes.bool,
         name: PropTypes.string.isRequired,
         scope: PropTypes.string.isRequired,
+        lastUpdateTime: PropTypes.number.isRequired,
+        url: PropTypes.string.isRequired,
         // registrationFront can be missing in e10s.
         registrationFront: PropTypes.object,
         workerTargetFront: PropTypes.object,
       }).isRequired,
     };
   }
 
   constructor(props) {
--- a/devtools/client/shared/components/List.js
+++ b/devtools/client/shared/components/List.js
@@ -37,16 +37,17 @@ loader.lazyRequireGetter(
 
 class ListItemClass extends Component {
   static get propTypes() {
     return {
       active: PropTypes.bool,
       current: PropTypes.bool,
       onClick: PropTypes.func,
       item: PropTypes.shape({
+        key: PropTypes.string,
         component: PropTypes.object,
         componentProps: PropTypes.object,
         className: PropTypes.string,
       }).isRequired,
     };
   }
 
   constructor(props) {
--- a/devtools/client/shared/components/tree/TreeRow.js
+++ b/devtools/client/shared/components/tree/TreeRow.js
@@ -54,16 +54,17 @@ define(function(require, exports, module
           level: PropTypes.number.isRequired,
           hasChildren: PropTypes.bool,
           value: PropTypes.any,
           open: PropTypes.bool.isRequired,
           path: PropTypes.string.isRequired,
           hidden: PropTypes.bool,
           selected: PropTypes.bool,
           active: PropTypes.bool,
+          loading: PropTypes.bool,
         }),
         decorator: PropTypes.object,
         renderCell: PropTypes.object,
         renderLabelCell: PropTypes.object,
         columns: PropTypes.array.isRequired,
         id: PropTypes.string.isRequired,
         provider: PropTypes.object.isRequired,
         onClick: PropTypes.func.isRequired,
--- a/devtools/client/shared/components/tree/TreeView.js
+++ b/devtools/client/shared/components/tree/TreeView.js
@@ -93,16 +93,17 @@ define(function(require, exports, module
         label: PropTypes.string,
         // Data provider (see also the interface above)
         provider: PropTypes.shape({
           getChildren: PropTypes.func,
           hasChildren: PropTypes.func,
           getLabel: PropTypes.func,
           getValue: PropTypes.func,
           getKey: PropTypes.func,
+          getLevel: PropTypes.func,
           getType: PropTypes.func,
         }).isRequired,
         // Tree decorator (see also the interface above)
         decorator: PropTypes.shape({
           getRowClass: PropTypes.func,
           getCellClass: PropTypes.func,
           getHeaderClass: PropTypes.func,
           renderValue: PropTypes.func,
--- a/devtools/client/webconsole/components/Output/Message.js
+++ b/devtools/client/webconsole/components/Output/Message.js
@@ -77,22 +77,26 @@ class Message extends Component {
       scrollToMessage: PropTypes.bool,
       exceptionDocURL: PropTypes.string,
       request: PropTypes.object,
       dispatch: PropTypes.func,
       timeStamp: PropTypes.number,
       timestampsVisible: PropTypes.bool.isRequired,
       serviceContainer: PropTypes.shape({
         emitNewMessage: PropTypes.func.isRequired,
+        onViewSource: PropTypes.func.isRequired,
         onViewSourceInDebugger: PropTypes.func,
         onViewSourceInScratchpad: PropTypes.func,
         onViewSourceInStyleEditor: PropTypes.func,
         openContextMenu: PropTypes.func.isRequired,
         openLink: PropTypes.func.isRequired,
         sourceMapService: PropTypes.any,
+        canRewind: PropTypes.func.isRequired,
+        jumpToExecutionPoint: PropTypes.func.isRequired,
+        onMessageHover: PropTypes.func.isRequired,
       }),
       notes: PropTypes.arrayOf(
         PropTypes.shape({
           messageBody: PropTypes.string.isRequired,
           frame: PropTypes.any,
         })
       ),
       isPaused: PropTypes.bool,
--- a/package-lock.json
+++ b/package-lock.json
@@ -690,28 +690,30 @@
       "resolved": "https://registry.npmjs.org/eslint-plugin-prettier/-/eslint-plugin-prettier-3.0.1.tgz",
       "integrity": "sha512-/PMttrarPAY78PLvV3xfWibMOdMDl57hmlQ2XqFeA37wd+CJ7WSxV7txqjVPHi/AAFKd2lX0ZqfsOc/i5yFCSQ==",
       "dev": true,
       "requires": {
         "prettier-linter-helpers": "^1.0.0"
       }
     },
     "eslint-plugin-react": {
-      "version": "7.12.4",
-      "resolved": "https://registry.npmjs.org/eslint-plugin-react/-/eslint-plugin-react-7.12.4.tgz",
-      "integrity": "sha512-1puHJkXJY+oS1t467MjbqjvX53uQ05HXwjqDgdbGBqf5j9eeydI54G3KwiJmWciQ0HTBacIKw2jgwSBSH3yfgQ==",
+      "version": "7.14.2",
+      "resolved": "https://registry.npmjs.org/eslint-plugin-react/-/eslint-plugin-react-7.14.2.tgz",
+      "integrity": "sha512-jZdnKe3ip7FQOdjxks9XPN0pjUKZYq48OggNMd16Sk+8VXx6JOvXmlElxROCgp7tiUsTsze3jd78s/9AFJP2mA==",
       "dev": true,
       "requires": {
         "array-includes": "^3.0.3",
         "doctrine": "^2.1.0",
         "has": "^1.0.3",
-        "jsx-ast-utils": "^2.0.1",
+        "jsx-ast-utils": "^2.1.0",
+        "object.entries": "^1.1.0",
         "object.fromentries": "^2.0.0",
-        "prop-types": "^15.6.2",
-        "resolve": "^1.9.0"
+        "object.values": "^1.1.0",
+        "prop-types": "^15.7.2",
+        "resolve": "^1.10.1"
       },
       "dependencies": {
         "doctrine": {
           "version": "2.1.0",
           "resolved": "https://registry.npmjs.org/doctrine/-/doctrine-2.1.0.tgz",
           "integrity": "sha512-35mSku4ZXK0vfCuHEDAwt55dg2jNajHZ1odvF+8SSr82EsZY4QmXfuWso8oEd8zRhVObSN18aM0CjSdoBX7zIw==",
           "dev": true,
           "requires": {
@@ -1299,28 +1301,52 @@
       "dev": true,
       "requires": {
         "define-properties": "^1.1.2",
         "function-bind": "^1.1.1",
         "has-symbols": "^1.0.0",
         "object-keys": "^1.0.11"
       }
     },
+    "object.entries": {
+      "version": "1.1.0",
+      "resolved": "https://registry.npmjs.org/object.entries/-/object.entries-1.1.0.tgz",
+      "integrity": "sha512-l+H6EQ8qzGRxbkHOd5I/aHRhHDKoQXQ8g0BYt4uSweQU1/J6dZUOyWh9a2Vky35YCKjzmgxOzta2hH6kf9HuXA==",
+      "dev": true,
+      "requires": {
+        "define-properties": "^1.1.3",
+        "es-abstract": "^1.12.0",
+        "function-bind": "^1.1.1",
+        "has": "^1.0.3"
+      }
+    },
     "object.fromentries": {
       "version": "2.0.0",
       "resolved": "https://registry.npmjs.org/object.fromentries/-/object.fromentries-2.0.0.tgz",
       "integrity": "sha512-9iLiI6H083uiqUuvzyY6qrlmc/Gz8hLQFOcb/Ri/0xXFkSNS3ctV+CbE6yM2+AnkYfOB3dGjdzC0wrMLIhQICA==",
       "dev": true,
       "requires": {
         "define-properties": "^1.1.2",
         "es-abstract": "^1.11.0",
         "function-bind": "^1.1.1",
         "has": "^1.0.1"
       }
     },
+    "object.values": {
+      "version": "1.1.0",
+      "resolved": "https://registry.npmjs.org/object.values/-/object.values-1.1.0.tgz",
+      "integrity": "sha512-8mf0nKLAoFX6VlNVdhGj31SVYpaNFtUnuoOXWyFEstsWRgU837AK+JYM0iAxwkSzGRbwn8cbFmgbyxj1j4VbXg==",
+      "dev": true,
+      "requires": {
+        "define-properties": "^1.1.3",
+        "es-abstract": "^1.12.0",
+        "function-bind": "^1.1.1",
+        "has": "^1.0.3"
+      }
+    },
     "once": {
       "version": "1.4.0",
       "resolved": "https://registry.npmjs.org/once/-/once-1.4.0.tgz",
       "integrity": "sha1-WDsap3WWHUsROsF9nFC6753Xa9E=",
       "dev": true,
       "requires": {
         "wrappy": "1"
       }
--- a/package.json
+++ b/package.json
@@ -6,25 +6,25 @@
   "dependencies": {
     "yarn": "^1.16.0"
   },
   "devDependencies": {
     "babel-eslint": "10.0.1",
     "eslint": "5.16.0",
     "eslint-config-prettier": "4.2.0",
     "eslint-plugin-babel": "5.3.0",
+    "eslint-plugin-fetch-options": "0.0.5",
     "eslint-plugin-file-header": "0.0.1",
     "eslint-plugin-flowtype": "3.6.0",
     "eslint-plugin-html": "5.0.3",
     "eslint-plugin-import": "2.16.0",
     "eslint-plugin-jest": "22.4.1",
     "eslint-plugin-jsx-a11y": "6.2.1",
     "eslint-plugin-mozilla": "file:tools/lint/eslint/eslint-plugin-mozilla",
     "eslint-plugin-no-unsanitized": "3.0.2",
     "eslint-plugin-prettier": "3.0.1",
-    "eslint-plugin-fetch-options": "0.0.5",
-    "eslint-plugin-react": "7.12.4",
+    "eslint-plugin-react": "7.14.2",
     "eslint-plugin-spidermonkey-js": "file:tools/lint/eslint/eslint-plugin-spidermonkey-js",
     "prettier": "1.17.0"
   },
   "notes(private)": "We don't want to publish to npm, so this is marked as private",
   "private": true
 }