Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(278)

Side by Side Diff: third_party/WebKit/Source/core/html/parser/HTMLTreeBuilder.cpp

Issue 2819773002: Change <menuitem> parsing rules to match spec (Closed)
Patch Set: Fix affected layout test Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2010 Google, Inc. All Rights Reserved. 2 * Copyright (C) 2010 Google, Inc. All Rights Reserved.
3 * Copyright (C) 2011, 2014 Apple Inc. All rights reserved. 3 * Copyright (C) 2011, 2014 Apple Inc. All rights reserved.
4 * 4 *
5 * Redistribution and use in source and binary forms, with or without 5 * Redistribution and use in source and binary forms, with or without
6 * modification, are permitted provided that the following conditions 6 * modification, are permitted provided that the following conditions
7 * are met: 7 * are met:
8 * 1. Redistributions of source code must retain the above copyright 8 * 1. Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * 2. Redistributions in binary form must reproduce the above copyright 10 * 2. Redistributions in binary form must reproduce the above copyright
(...skipping 714 matching lines...) Expand 10 before | Expand all | Expand 10 after
725 !type_attribute || 725 !type_attribute ||
726 !DeprecatedEqualIgnoringCase(type_attribute->Value(), "hidden"); 726 !DeprecatedEqualIgnoringCase(type_attribute->Value(), "hidden");
727 727
728 tree_.ReconstructTheActiveFormattingElements(); 728 tree_.ReconstructTheActiveFormattingElements();
729 tree_.InsertSelfClosingHTMLElementDestroyingToken(token); 729 tree_.InsertSelfClosingHTMLElementDestroyingToken(token);
730 730
731 if (disable_frameset) 731 if (disable_frameset)
732 frameset_ok_ = false; 732 frameset_ok_ = false;
733 return; 733 return;
734 } 734 }
735 if ((RuntimeEnabledFeatures::contextMenuEnabled() && 735 if (token->GetName() == paramTag || token->GetName() == sourceTag ||
736 token->GetName() == menuitemTag) ||
737 token->GetName() == paramTag || token->GetName() == sourceTag ||
738 token->GetName() == trackTag) { 736 token->GetName() == trackTag) {
739 tree_.InsertSelfClosingHTMLElementDestroyingToken(token); 737 tree_.InsertSelfClosingHTMLElementDestroyingToken(token);
740 return; 738 return;
741 } 739 }
742 if (token->GetName() == hrTag) { 740 if (token->GetName() == hrTag) {
743 ProcessFakePEndTagIfPInButtonScope(); 741 ProcessFakePEndTagIfPInButtonScope();
744 tree_.InsertSelfClosingHTMLElementDestroyingToken(token); 742 tree_.InsertSelfClosingHTMLElementDestroyingToken(token);
745 frameset_ok_ = false; 743 frameset_ok_ = false;
746 return; 744 return;
747 } 745 }
(...skipping 2023 matching lines...) Expand 10 before | Expand all | Expand 10 after
2771 DEFINE_STRINGIFY(kAfterFramesetMode) 2769 DEFINE_STRINGIFY(kAfterFramesetMode)
2772 DEFINE_STRINGIFY(kAfterAfterBodyMode) 2770 DEFINE_STRINGIFY(kAfterAfterBodyMode)
2773 DEFINE_STRINGIFY(kAfterAfterFramesetMode) 2771 DEFINE_STRINGIFY(kAfterAfterFramesetMode)
2774 #undef DEFINE_STRINGIFY 2772 #undef DEFINE_STRINGIFY
2775 } 2773 }
2776 return "<unknown>"; 2774 return "<unknown>";
2777 } 2775 }
2778 #endif 2776 #endif
2779 2777
2780 } // namespace blink 2778 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698