DevTools: Run devtools tests in release mode

chrome/browser/profiles/profile_io_data.cc

Index: chrome/browser/profiles/profile_io_data.cc
diff --git a/chrome/browser/profiles/profile_io_data.cc b/chrome/browser/profiles/profile_io_data.cc
index 11b38772b5b3193d09e97ee1b01c436ae722c135..f18a7335b904473414effeceb6bf93b696aff371 100644
--- a/chrome/browser/profiles/profile_io_data.cc
+++ b/chrome/browser/profiles/profile_io_data.cc
@@ -222,7 +222,8 @@ bool IsSupportedDevToolsURL(const GURL& url, base::FilePath* path) {
   if (inspector_dir.empty())
     return false;
 
-  *path = inspector_dir.AppendASCII(relative_path);
+  base::FilePath debug_path = inspector_dir.AppendASCII("debug");
+  *path = debug_path.AppendASCII(relative_path);

[mmenke, 2016/08/31 00:04:29]

Maybe *path = inspector_dir.AppendASCII("debug").AppendASCII(relative_path);?

One other FilePath variable with another name seems a little harder to follow to
me.

Also...Why is this needed?  The description does seem to say.  Maybe a comment? 
Does this need to match some other path generated elsewhere?  If so, should we
share code?

[mmenke, 2016/08/31 00:38:40]

Also, will this result in leaving junk in the old path?

[chenwilliam, 2016/08/31 00:53:06]

Done. I added a comment. This path needs to match with one in our BUILD.gn file
so I'm not sure if we can share the code. 

This shouldn't impact the old path.

   return true;
 }