commit 7386530c8f79c69e6fa88f4f49691cb89597e60f
author Tom Sepez <tsepez@chromium.org> Thu Feb 13 19:27:56 2020 +0000
committer Chromium commit bot <commit-bot@chromium.org> Thu Feb 13 19:27:56 2020 +0000
tree ecde568ea83fbfeff04a3fdea3bd4ba4e68b5f81
parent 4d5c030d569d8dfba7c6cfdbdfbd125754da6180

No need to buffer CSS comments

Instead of looking for a slash and then looking behind for a star
in the output buffer, look for a star then look ahead for a slash.

Change-Id: I16ec56af8eeb27a972faebe08e34a11dad24d10f
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/66375
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>

core/fxcrt/css/cfx_csssyntaxparser.cpp

diff --git a/core/fxcrt/css/cfx_csssyntaxparser.cpp b/core/fxcrt/css/cfx_csssyntaxparser.cpp
index eb32297..8c92de4 100644
--- a/core/fxcrt/css/cfx_csssyntaxparser.cpp
+++ b/core/fxcrt/css/cfx_csssyntaxparser.cpp
@@ -159,11 +159,9 @@
           }
           break;
         case SyntaxMode::kComment:
-          if (wch == '/' && m_Output.GetLength() > 0 &&
-              m_Output.GetBuffer()[m_Output.GetLength() - 1] == '*') {
+          if (wch == '*' && m_Input.GetNextChar() == '/') {
             RestoreMode();
-          } else {
-            m_Output.AppendChar(wch);
+            m_Input.MoveNext();
           }
           m_Input.MoveNext();
           break;