DevTools: ConsoleViewMessage builders should explicitly add classes

DevTools: ConsoleViewMessage builders should explicitly add classes

We're no longer doing this right now.  Searching the codebase for 'object-value-" + ' reveals that lots of other places depend on object value classes to work this way.  At this time, moving the classes into builders won't be easy without breaking current styles.

[luoe, 2016-10-14 18:48:37]

The CQ bit was checked by luoe@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-14 18:49:18]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[luoe, 2016-10-14 18:49:27]

luoe@chromium.org changed reviewers:
+ lushnikov@chromium.org

[luoe, 2016-10-14 18:49:27]

This was split out from patches 6, 7 in
https://codereview.chromium.org/2397933005

[commit-bot: I haz the power, 2016-10-14 20:23:41]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-14 20:23:42]

Dry run: This issue passed the CQ dry run.

[lushnikov, 2016-10-18 23:15:19]

https://chromiumcodereview.appspot.com/2424443003/diff/1/third_party/WebKit/S...
File third_party/WebKit/Source/devtools/front_end/console/ConsoleViewMessage.js
(right):

https://chromiumcodereview.appspot.com/2424443003/diff/1/third_party/WebKit/S...
third_party/WebKit/Source/devtools/front_end/console/ConsoleViewMessage.js:526:
element.classList.add("object-value-regexp");
object-value-string?

[luoe, 2016-10-20 21:15:47]

Description was changed from

==========
DevTools: ConsoleViewMessage builders should explicitly add classes
==========

to

==========
DevTools: ConsoleViewMessage builders should explicitly add classes

We're no longer doing this right now.  Searching the codebase for
'object-value-" + ' reveals that lots of other places depend on object value
classes to work this way.  At this time, moving the classes into builders won't
be easy without breaking current styles.
==========