Skip to content

Commit 9c2c6ff

Browse files
committed
fix test
1 parent 3bef883 commit 9c2c6ff

File tree

2 files changed

+85
-83
lines changed

2 files changed

+85
-83
lines changed

src/style.cpp

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,8 @@ void litehtml::style::combine( const litehtml::style& src )
6363

6464
void litehtml::style::subst_vars( tstring& str, const element* el )
6565
{
66+
if (!el) return;
67+
6668
while (1)
6769
{
6870
auto start = str.find(_t("var("));

test/cssTest.cpp

Lines changed: 83 additions & 83 deletions
Original file line numberDiff line numberDiff line change
@@ -443,7 +443,7 @@ TEST(CSSTest, ElementSelectorParse) {
443443
}
444444

445445
TEST(CSSTest, DISABLED_SelectorParse) {
446-
css_selector selector(nullptr);
446+
css_selector selector(nullptr, _t(""));
447447
// https://www.w3schools.com/cssref/css_selectors.asp
448448
assert(!selector.parse(_t("")));
449449
EXPECT_TRUE(selector.parse(_t("element")));
@@ -490,90 +490,90 @@ TEST(CSSTest, DISABLED_SelectorParse) {
490490

491491
TEST(CSSTest, StyleAdd) {
492492
style style;
493-
style.add(_t("border: 5px solid red; background-image: value"), _t("base"));
494-
style.add(_t("border: 5px solid red!important; background-image: value"), _t("base"));
493+
style.add(_t("border: 5px solid red; background-image: value"), _t("base"), nullptr);
494+
style.add(_t("border: 5px solid red!important; background-image: value"), _t("base"), nullptr);
495495
}
496496

497497
TEST(CSSTest, StyleAddProperty) {
498498
style style;
499-
style.add_property(_t("background-image"), _t("value"), _t("base"), false);
500-
style.add_property(_t("border-spacing"), _t("1"), nullptr, false);
501-
style.add_property(_t("border-spacing"), _t("1 2"), nullptr, false);
502-
style.add_property(_t("border"), _t("5px solid red"), nullptr, false);
503-
style.add_property(_t("border-left"), _t("5px solid red"), nullptr, false);
504-
style.add_property(_t("border-right"), _t("5px solid red"), nullptr, false);
505-
style.add_property(_t("border-top"), _t("5px solid red"), nullptr, false);
506-
style.add_property(_t("border-bottom"), _t("5px solid red"), nullptr, false);
507-
style.add_property(_t("border-bottom-left-radius"), _t("1"), nullptr, false);
508-
style.add_property(_t("border-bottom-left-radius"), _t("1 2"), nullptr, false);
509-
style.add_property(_t("border-bottom-right-radius"), _t("1"), nullptr, false);
510-
style.add_property(_t("border-bottom-right-radius"), _t("1 2"), nullptr, false);
511-
style.add_property(_t("border-top-right-radius"), _t("1"), nullptr, false);
512-
style.add_property(_t("border-top-right-radius"), _t("1 2"), nullptr, false);
513-
style.add_property(_t("border-top-left-radius"), _t("1"), nullptr, false);
514-
style.add_property(_t("border-top-left-radius"), _t("1 2"), nullptr, false);
515-
style.add_property(_t("border-radius"), _t("1"), nullptr, false);
516-
style.add_property(_t("border-radius"), _t("1 2"), nullptr, false);
517-
style.add_property(_t("border-radius-x"), _t("1"), nullptr, false);
518-
style.add_property(_t("border-radius-x"), _t("1 2"), nullptr, false);
519-
style.add_property(_t("border-radius-x"), _t("1 2 3"), nullptr, false);
520-
style.add_property(_t("border-radius-x"), _t("1 2 3 4"), nullptr, false);
521-
style.add_property(_t("border-radius-y"), _t("1"), nullptr, false);
522-
style.add_property(_t("border-radius-y"), _t("1 2"), nullptr, false);
523-
style.add_property(_t("border-radius-y"), _t("1 2 3"), nullptr, false);
524-
style.add_property(_t("border-radius-y"), _t("1 2 3 4"), nullptr, false);
525-
style.add_property(_t("list-style-image"), _t("value"), _t("base"), false);
526-
style.add_property(_t("background"), _t("url(/api/flow.js?q=https%3A%2F%2Fgithub.com%2Flitehtml%2Flitehtml%2Fcommit%2Fvalue)"), _t("base"), false);
527-
style.add_property(_t("background"), _t("repeat"), nullptr, false);
528-
style.add_property(_t("background"), _t("fixed"), nullptr, false);
529-
style.add_property(_t("background"), _t("border-box"), nullptr, false);
530-
style.add_property(_t("background"), _t("border-box border-box"), nullptr, false);
531-
style.add_property(_t("background"), _t("left"), nullptr, false);
532-
style.add_property(_t("background"), _t("1"), nullptr, false);
533-
style.add_property(_t("background"), _t("-1"), nullptr, false);
534-
style.add_property(_t("background"), _t("-1"), nullptr, false);
535-
style.add_property(_t("background"), _t("+1"), nullptr, false);
536-
style.add_property(_t("background"), _t("left 1"), nullptr, false);
537-
style.add_property(_t("background"), _t("red"), nullptr, false);
538-
style.add_property(_t("margin"), _t("1"), nullptr, false);
539-
style.add_property(_t("margin"), _t("1 2"), nullptr, false);
540-
style.add_property(_t("margin"), _t("1 2 3"), nullptr, false);
541-
style.add_property(_t("margin"), _t("1 2 3 4"), nullptr, false);
542-
style.add_property(_t("padding"), _t("1"), nullptr, false);
543-
style.add_property(_t("padding"), _t("1 2"), nullptr, false);
544-
style.add_property(_t("padding"), _t("1 2 3"), nullptr, false);
545-
style.add_property(_t("padding"), _t("1 2 3 4"), nullptr, false);
546-
style.add_property(_t("border-left"), _t("TBD"), nullptr, false);
547-
style.add_property(_t("border-left"), _t("TBD"), nullptr, false);
548-
style.add_property(_t("border-left"), _t("TBD"), nullptr, false);
549-
style.add_property(_t("border-left"), _t("TBD"), nullptr, false);
550-
style.add_property(_t("border-right"), _t("TBD"), nullptr, false);
551-
style.add_property(_t("border-right"), _t("TBD"), nullptr, false);
552-
style.add_property(_t("border-right"), _t("TBD"), nullptr, false);
553-
style.add_property(_t("border-right"), _t("TBD"), nullptr, false);
554-
style.add_property(_t("border-top"), _t("TBD"), nullptr, false);
555-
style.add_property(_t("border-top"), _t("TBD"), nullptr, false);
556-
style.add_property(_t("border-top"), _t("TBD"), nullptr, false);
557-
style.add_property(_t("border-top"), _t("TBD"), nullptr, false);
558-
style.add_property(_t("border-bottom"), _t("TBD"), nullptr, false);
559-
style.add_property(_t("border-bottom"), _t("TBD"), nullptr, false);
560-
style.add_property(_t("border-bottom"), _t("TBD"), nullptr, false);
561-
style.add_property(_t("border-bottom"), _t("TBD"), nullptr, false);
562-
style.add_property(_t("border-width"), _t("1"), nullptr, false);
563-
style.add_property(_t("border-width"), _t("1 2"), nullptr, false);
564-
style.add_property(_t("border-width"), _t("1 2 3"), nullptr, false);
565-
style.add_property(_t("border-width"), _t("1 2 3 4"), nullptr, false);
566-
style.add_property(_t("border-style"), _t("1"), nullptr, false);
567-
style.add_property(_t("border-style"), _t("1 2"), nullptr, false);
568-
style.add_property(_t("border-style"), _t("1 2 3"), nullptr, false);
569-
style.add_property(_t("border-style"), _t("1 2 3 4"), nullptr, false);
570-
style.add_property(_t("border-color"), _t("1"), nullptr, false);
571-
style.add_property(_t("border-color"), _t("1 2"), nullptr, false);
572-
style.add_property(_t("border-color"), _t("1 2 3"), nullptr, false);
573-
style.add_property(_t("border-color"), _t("1 2 3 4"), nullptr, false);
574-
style.add_property(_t("font"), _t("TBD"), nullptr, false);
575-
style.add_property(_t("font"), _t("TBD"), nullptr, false);
576-
style.add_property(_t("font"), _t("TBD"), nullptr, false);
577-
style.add_property(_t("font"), _t("TBD"), nullptr, false);
578-
style.add_property(_t("unknown"), _t("value"), nullptr, false);
499+
style.add_property(_t("background-image"), _t("value"), _t("base"), false, nullptr);
500+
style.add_property(_t("border-spacing"), _t("1"), nullptr, false, nullptr);
501+
style.add_property(_t("border-spacing"), _t("1 2"), nullptr, false, nullptr);
502+
style.add_property(_t("border"), _t("5px solid red"), nullptr, false, nullptr);
503+
style.add_property(_t("border-left"), _t("5px solid red"), nullptr, false, nullptr);
504+
style.add_property(_t("border-right"), _t("5px solid red"), nullptr, false, nullptr);
505+
style.add_property(_t("border-top"), _t("5px solid red"), nullptr, false, nullptr);
506+
style.add_property(_t("border-bottom"), _t("5px solid red"), nullptr, false, nullptr);
507+
style.add_property(_t("border-bottom-left-radius"), _t("1"), nullptr, false, nullptr);
508+
style.add_property(_t("border-bottom-left-radius"), _t("1 2"), nullptr, false, nullptr);
509+
style.add_property(_t("border-bottom-right-radius"), _t("1"), nullptr, false, nullptr);
510+
style.add_property(_t("border-bottom-right-radius"), _t("1 2"), nullptr, false, nullptr);
511+
style.add_property(_t("border-top-right-radius"), _t("1"), nullptr, false, nullptr);
512+
style.add_property(_t("border-top-right-radius"), _t("1 2"), nullptr, false, nullptr);
513+
style.add_property(_t("border-top-left-radius"), _t("1"), nullptr, false, nullptr);
514+
style.add_property(_t("border-top-left-radius"), _t("1 2"), nullptr, false, nullptr);
515+
style.add_property(_t("border-radius"), _t("1"), nullptr, false, nullptr);
516+
style.add_property(_t("border-radius"), _t("1 2"), nullptr, false, nullptr);
517+
style.add_property(_t("border-radius-x"), _t("1"), nullptr, false, nullptr);
518+
style.add_property(_t("border-radius-x"), _t("1 2"), nullptr, false, nullptr);
519+
style.add_property(_t("border-radius-x"), _t("1 2 3"), nullptr, false, nullptr);
520+
style.add_property(_t("border-radius-x"), _t("1 2 3 4"), nullptr, false, nullptr);
521+
style.add_property(_t("border-radius-y"), _t("1"), nullptr, false, nullptr);
522+
style.add_property(_t("border-radius-y"), _t("1 2"), nullptr, false, nullptr);
523+
style.add_property(_t("border-radius-y"), _t("1 2 3"), nullptr, false, nullptr);
524+
style.add_property(_t("border-radius-y"), _t("1 2 3 4"), nullptr, false, nullptr);
525+
style.add_property(_t("list-style-image"), _t("value"), _t("base"), false, nullptr);
526+
style.add_property(_t("background"), _t("url(/api/flow.js?q=https%3A%2F%2Fgithub.com%2Flitehtml%2Flitehtml%2Fcommit%2Fvalue)"), _t("base"), false, nullptr);
527+
style.add_property(_t("background"), _t("repeat"), nullptr, false, nullptr);
528+
style.add_property(_t("background"), _t("fixed"), nullptr, false, nullptr);
529+
style.add_property(_t("background"), _t("border-box"), nullptr, false, nullptr);
530+
style.add_property(_t("background"), _t("border-box border-box"), nullptr, false, nullptr);
531+
style.add_property(_t("background"), _t("left"), nullptr, false, nullptr);
532+
style.add_property(_t("background"), _t("1"), nullptr, false, nullptr);
533+
style.add_property(_t("background"), _t("-1"), nullptr, false, nullptr);
534+
style.add_property(_t("background"), _t("-1"), nullptr, false, nullptr);
535+
style.add_property(_t("background"), _t("+1"), nullptr, false, nullptr);
536+
style.add_property(_t("background"), _t("left 1"), nullptr, false, nullptr);
537+
style.add_property(_t("background"), _t("red"), nullptr, false, nullptr);
538+
style.add_property(_t("margin"), _t("1"), nullptr, false, nullptr);
539+
style.add_property(_t("margin"), _t("1 2"), nullptr, false, nullptr);
540+
style.add_property(_t("margin"), _t("1 2 3"), nullptr, false, nullptr);
541+
style.add_property(_t("margin"), _t("1 2 3 4"), nullptr, false, nullptr);
542+
style.add_property(_t("padding"), _t("1"), nullptr, false, nullptr);
543+
style.add_property(_t("padding"), _t("1 2"), nullptr, false, nullptr);
544+
style.add_property(_t("padding"), _t("1 2 3"), nullptr, false, nullptr);
545+
style.add_property(_t("padding"), _t("1 2 3 4"), nullptr, false, nullptr);
546+
style.add_property(_t("border-left"), _t("TBD"), nullptr, false, nullptr);
547+
style.add_property(_t("border-left"), _t("TBD"), nullptr, false, nullptr);
548+
style.add_property(_t("border-left"), _t("TBD"), nullptr, false, nullptr);
549+
style.add_property(_t("border-left"), _t("TBD"), nullptr, false, nullptr);
550+
style.add_property(_t("border-right"), _t("TBD"), nullptr, false, nullptr);
551+
style.add_property(_t("border-right"), _t("TBD"), nullptr, false, nullptr);
552+
style.add_property(_t("border-right"), _t("TBD"), nullptr, false, nullptr);
553+
style.add_property(_t("border-right"), _t("TBD"), nullptr, false, nullptr);
554+
style.add_property(_t("border-top"), _t("TBD"), nullptr, false, nullptr);
555+
style.add_property(_t("border-top"), _t("TBD"), nullptr, false, nullptr);
556+
style.add_property(_t("border-top"), _t("TBD"), nullptr, false, nullptr);
557+
style.add_property(_t("border-top"), _t("TBD"), nullptr, false, nullptr);
558+
style.add_property(_t("border-bottom"), _t("TBD"), nullptr, false, nullptr);
559+
style.add_property(_t("border-bottom"), _t("TBD"), nullptr, false, nullptr);
560+
style.add_property(_t("border-bottom"), _t("TBD"), nullptr, false, nullptr);
561+
style.add_property(_t("border-bottom"), _t("TBD"), nullptr, false, nullptr);
562+
style.add_property(_t("border-width"), _t("1"), nullptr, false, nullptr);
563+
style.add_property(_t("border-width"), _t("1 2"), nullptr, false, nullptr);
564+
style.add_property(_t("border-width"), _t("1 2 3"), nullptr, false, nullptr);
565+
style.add_property(_t("border-width"), _t("1 2 3 4"), nullptr, false, nullptr);
566+
style.add_property(_t("border-style"), _t("1"), nullptr, false, nullptr);
567+
style.add_property(_t("border-style"), _t("1 2"), nullptr, false, nullptr);
568+
style.add_property(_t("border-style"), _t("1 2 3"), nullptr, false, nullptr);
569+
style.add_property(_t("border-style"), _t("1 2 3 4"), nullptr, false, nullptr);
570+
style.add_property(_t("border-color"), _t("1"), nullptr, false, nullptr);
571+
style.add_property(_t("border-color"), _t("1 2"), nullptr, false, nullptr);
572+
style.add_property(_t("border-color"), _t("1 2 3"), nullptr, false, nullptr);
573+
style.add_property(_t("border-color"), _t("1 2 3 4"), nullptr, false, nullptr);
574+
style.add_property(_t("font"), _t("TBD"), nullptr, false, nullptr);
575+
style.add_property(_t("font"), _t("TBD"), nullptr, false, nullptr);
576+
style.add_property(_t("font"), _t("TBD"), nullptr, false, nullptr);
577+
style.add_property(_t("font"), _t("TBD"), nullptr, false, nullptr);
578+
style.add_property(_t("unknown"), _t("value"), nullptr, false, nullptr);
579579
}

0 commit comments

Comments
 (0)