PageRule should not serialize @page in selectorText

third_party/WebKit/Source/core/css/CSSPageRuleTest.cpp

+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "core/css/CSSPageRule.h"
+
+#include "core/css/CSSRuleList.h"
+#include "core/css/CSSTestHelper.h"
+
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace blink {
+
+TEST(CSSPageRule, Serializing)
+{
+    CSSTestHelper helper;
+
+    const char* cssRule = "@page :left { size: auto; }";
+    helper.addCSSRules(cssRule);
+    if (helper.cssRules()) {

[Timothy Loh, 2016/01/31 23:54:24]

If this ends up null for some reason, then the test will still pass. You can
ASSERT it isn't null to safely check this (ASSERTs will stop the test if they
fail, while EXPECTs will keep going), although I don't think anyone really pays
much attention to whether people correctly use ASSERT or EXPECT.

I'm not sure the purpose of the if check below either, it seems like you're
already assuming the rule is non-null by dereferencing it before.

+        EXPECT_EQ(1u, helper.cssRules()->length());
+        EXPECT_EQ(String(cssRule), helper.cssRules()->item(0)->cssText());
+        EXPECT_EQ(CSSRule::PAGE_RULE, helper.cssRules()->item(0)->type());
+        if (CSSPageRule* pageRule = toCSSPageRule(helper.cssRules()->item(0)))
+            EXPECT_EQ(":left", pageRule->selectorText());
+    }
+}
+
+} // namespace blink