From 1ea8d26488f1b1f09aadb6ca7ccad91da8ae0243 Mon Sep 17 00:00:00 2001
From: Paul Rouget <paul@mozilla.com>
Date: Tue, 12 Mar 2013 18:43:00 -0700
Subject: [PATCH] Bug 723281 - Add the ability to enable/disable paint flashing
 as a [gcli] command. r=jwalker

---
 .../devtools/commandline/BuiltinCommands.jsm  | 26 +++++++++++++------
 1 file changed, 18 insertions(+), 8 deletions(-)

diff --git a/browser/devtools/commandline/BuiltinCommands.jsm b/browser/devtools/commandline/BuiltinCommands.jsm
index 67ad4fa4cb08..03d30f03bdca 100644
--- a/browser/devtools/commandline/BuiltinCommands.jsm
+++ b/browser/devtools/commandline/BuiltinCommands.jsm
@@ -1705,10 +1705,15 @@ XPCOMUtils.defineLazyModuleGetter(this, "TargetFactory",
     description: gcli.lookup('paintflashingOnDesc'),
     manual: gcli.lookup('paintflashingManual'),
     params: [{
-      type: "boolean",
-      name: "chrome",
-      get hidden() gcli.hiddenByChromePref(),
-      description: gcli.lookup("paintflashingChromeDesc"),
+      group: "options",
+      params: [
+        {
+          type: "boolean",
+          name: "chrome",
+          get hidden() gcli.hiddenByChromePref(),
+          description: gcli.lookup("paintflashingChromeDesc"),
+        }
+      ]
     }],
     exec: function(args, context) {
       var window;
@@ -1729,10 +1734,15 @@ XPCOMUtils.defineLazyModuleGetter(this, "TargetFactory",
     description: gcli.lookup('paintflashingOffDesc'),
     manual: gcli.lookup('paintflashingManual'),
     params: [{
-      type: "boolean",
-      name: "chrome",
-      get hidden() gcli.hiddenByChromePref(),
-      description: gcli.lookup("paintflashingChromeDesc"),
+      group: "options",
+      params: [
+        {
+          type: "boolean",
+          name: "chrome",
+          get hidden() gcli.hiddenByChromePref(),
+          description: gcli.lookup("paintflashingChromeDesc"),
+        }
+      ]
     }],
     exec: function(args, context) {
       if (args.chrome) {