Use brackets for explicit GPU config params in tools

tools/flags/SkCommonFlagsConfig.cpp

 /*
  * Copyright 2015 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkCommonFlagsConfig.h"
 #include "SkImageInfo.h"
 
@@ skipping 78 matching lines @@
     " hwui"
 #endif
     ;
 
 static const char* config_help_fn() {
     static SkString helpString;
     helpString.set(configHelp);
     for (const auto& config : gPredefinedConfigs) {
         helpString.appendf(" %s", config.predefinedConfig);
     }
-    helpString.append(" or use extended form 'backend(option=value,...)'.\n");
+    helpString.append(" or use extended form 'backend[option=value,...]'.\n");
     return helpString.c_str();
 }
 
 static const char configExtendedHelp[] =
     "Extended form: 'backend(option=value,...)'\n\n"
     "Possible backends and options:\n"
 #if SK_SUPPORT_GPU
     "\n"
-    "gpu(api=string,color=string,dit=bool,nvpr=bool,inst=bool,samples=int)\n"
+    "gpu[api=string,color=string,dit=bool,nvpr=bool,inst=bool,samples=int]\n"
     "\tapi\ttype: string\tdefault: native.\n"
     "\t    Select graphics API to use with gpu backend.\n"
     "\t    Options:\n"
     "\t\tnative\t\t\tUse platform default OpenGL or OpenGL ES backend.\n"
     "\t\tgl    \t\t\tUse OpenGL.\n"
     "\t\tgles  \t\t\tUse OpenGL ES.\n"
     "\t\tdebug \t\t\tUse debug OpenGL.\n"
     "\t\tnull  \t\t\tUse null OpenGL.\n"
 #if SK_ANGLE
 #ifdef SK_BUILD_FOR_WIN
@@ skipping 256 matching lines @@
                   SkCommandLineConfigArray* outResult) {
     outResult->reset();
     for (int i = 0; i < configs.count(); ++i) {
         SkString extendedBackend;
         SkString extendedOptions;
         SkString simpleBackend;
         SkTArray<SkString> vias;
 
         SkString tag(configs[i]);
         SkTArray<SkString> parts;
-        SkStrSplit(tag.c_str(), "(", kStrict_SkStrSplitMode, &parts);
+        SkStrSplit(tag.c_str(), "[", kStrict_SkStrSplitMode, &parts);
         if (parts.count() == 2) {
             SkTArray<SkString> parts2;
-            SkStrSplit(parts[1].c_str(), ")", kStrict_SkStrSplitMode, &parts2);
+            SkStrSplit(parts[1].c_str(), "]", kStrict_SkStrSplitMode, &parts2);
             if (parts2.count() == 2 && parts2[1].isEmpty()) {
                 SkStrSplit(parts[0].c_str(), "-", kStrict_SkStrSplitMode, &vias);
                 if (vias.count()) {
                     extendedBackend = vias[vias.count() - 1];
                     vias.pop_back();
                 } else {
                     extendedBackend = parts[0];
                 }
                 extendedOptions = parts2[0];
-                simpleBackend.printf("%s(%s)", extendedBackend.c_str(), extendedOptions.c_str());
+                simpleBackend.printf("%s[%s]", extendedBackend.c_str(), extendedOptions.c_str());
             }
         }
 
         if (extendedBackend.isEmpty()) {
             simpleBackend = tag;
             SkStrSplit(tag.c_str(), "-", kStrict_SkStrSplitMode, &vias);
             if (vias.count()) {
                 simpleBackend = vias[vias.count() - 1];
                 vias.pop_back();
             }
@@ skipping 17 matching lines @@
         if (extendedBackend.equals("gpu")) {
             parsedConfig = parse_command_line_config_gpu(tag, vias, extendedOptions);
         }
 #endif
         if (!parsedConfig) {
             parsedConfig = new SkCommandLineConfig(tag, simpleBackend, vias);
         }
         outResult->emplace_back(parsedConfig);
     }
 }