From ba7711720b83c66b47d34a424d2ee504641ebabb Mon Sep 17 00:00:00 2001
From: Paul Rouget <paul@mozilla.com>
Date: Tue, 12 Mar 2013 18:46:16 -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 7abcde307a08..8e47eb1867b2 100644
--- a/browser/devtools/commandline/BuiltinCommands.jsm
+++ b/browser/devtools/commandline/BuiltinCommands.jsm
@@ -2080,10 +2080,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;
@@ -2104,10 +2109,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) {