Fix breakpoint condition wrongly becoming undefined

Review URL: https://chromiumcodereview.appspot.com/19947004

git-svn-id: svn://svn.chromium.org/blink/trunk@154681 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove Node::parentNodeForRenderingAndStyle and switch to parentRenderStyle

parentNodeForRenderingAndStyle() was a complicated way of doing NodeRenderingTraversal::parent.
This switches to that, but more importantly it makes us stop calling into the
ComposedShadowTreeWalker::traverseParent twice during recalcStyle which was crazy since that's
tons of code to do twice just for a ternary.

R=eseidel@chromium.org,ojan@chromium.org,eae@chromium.org

Review URL: https://chromiumcodereview.appspot.com/19962004

git-svn-id: svn://svn.chromium.org/blink/trunk@154682 bbb929c8-8fbe-4397-9dbb-9b2b20218538
RenderStyle no longer needs to be friends with StyleResolverState

StyleResolverState no longer needs to reach into the private methods
on RenderStyle and thus no longer needs to be declared as a friend.

BUG=237268
R=leviw@chromium.org

Review URL: https://chromiumcodereview.appspot.com/19771020

git-svn-id: svn://svn.chromium.org/blink/trunk@154683 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Simplify flakiness dashboard test list handling.

-Remove the GTEST_MODIFIERS stuff. The modifiers are now stripped
on the gtest side, so we don't need that here.
-Accept newlines and multiple spaces. This makes copy-paste
from other locations simpler.
-Don't use isCrossBuilderView in deciding whether to show the builder
or the text name. I saw this break once and couldn't reproduce it,
but the new code is also better because it doesn't rely on global state.

R=dpranke@chromium.org

Review URL: https://codereview.chromium.org/19888002

git-svn-id: svn://svn.chromium.org/blink/trunk@154684 bbb929c8-8fbe-4397-9dbb-9b2b20218538