Commit aeee8c33 authored by Kouhei Ueno's avatar Kouhei Ueno Committed by Commit Bot

Remove unused flag BackgroundHTMLParser::should_coalesce_chunks_

This CL removes unused flag BackgroundHTMLParser::should_coalesce_chunks_.
The flag is always false and never true.

Bug: None
Change-Id: I46fbb259dfe9cf7b814935037c026d2db9e33beb
Reviewed-on: https://chromium-review.googlesource.com/1026332Reviewed-by: default avatarKent Tamura <tkent@chromium.org>
Reviewed-by: default avatarKentaro Hara <haraken@chromium.org>
Commit-Queue: Kouhei Ueno <kouhei@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553445}
parent 9f0eda63
...@@ -108,8 +108,7 @@ void BackgroundHTMLParser::Init( ...@@ -108,8 +108,7 @@ void BackgroundHTMLParser::Init(
BackgroundHTMLParser::Configuration::Configuration() BackgroundHTMLParser::Configuration::Configuration()
: outstanding_token_limit(kDefaultOutstandingTokenLimit), : outstanding_token_limit(kDefaultOutstandingTokenLimit),
pending_token_limit(kDefaultPendingTokenLimit), pending_token_limit(kDefaultPendingTokenLimit) {}
should_coalesce_chunks(false) {}
BackgroundHTMLParser::BackgroundHTMLParser( BackgroundHTMLParser::BackgroundHTMLParser(
std::unique_ptr<Configuration> config, std::unique_ptr<Configuration> config,
...@@ -129,8 +128,7 @@ BackgroundHTMLParser::BackgroundHTMLParser( ...@@ -129,8 +128,7 @@ BackgroundHTMLParser::BackgroundHTMLParser(
tokenized_chunk_queue_(std::move(config->tokenized_chunk_queue)), tokenized_chunk_queue_(std::move(config->tokenized_chunk_queue)),
pending_csp_meta_token_index_( pending_csp_meta_token_index_(
HTMLDocumentParser::TokenizedChunk::kNoPendingToken), HTMLDocumentParser::TokenizedChunk::kNoPendingToken),
starting_script_(false), starting_script_(false) {
should_coalesce_chunks_(config->should_coalesce_chunks) {
DCHECK_GT(outstanding_token_limit_, 0u); DCHECK_GT(outstanding_token_limit_, 0u);
DCHECK_GT(pending_token_limit_, 0u); DCHECK_GT(pending_token_limit_, 0u);
DCHECK_GE(outstanding_token_limit_, pending_token_limit_); DCHECK_GE(outstanding_token_limit_, pending_token_limit_);
...@@ -292,7 +290,7 @@ void BackgroundHTMLParser::PumpTokenizer() { ...@@ -292,7 +290,7 @@ void BackgroundHTMLParser::PumpTokenizer() {
break; break;
} }
if (!should_coalesce_chunks_ && should_notify_main_thread) { if (should_notify_main_thread) {
RunOnMainThread(&HTMLDocumentParser::NotifyPendingTokenizedChunks, RunOnMainThread(&HTMLDocumentParser::NotifyPendingTokenizedChunks,
parser_); parser_);
should_notify_main_thread = false; should_notify_main_thread = false;
......
...@@ -65,7 +65,6 @@ class BackgroundHTMLParser { ...@@ -65,7 +65,6 @@ class BackgroundHTMLParser {
// pendingTokenLimit // pendingTokenLimit
size_t outstanding_token_limit; size_t outstanding_token_limit;
size_t pending_token_limit; size_t pending_token_limit;
bool should_coalesce_chunks;
}; };
// The returned BackgroundHTMLParser should only be used on the parser // The returned BackgroundHTMLParser should only be used on the parser
...@@ -147,7 +146,6 @@ class BackgroundHTMLParser { ...@@ -147,7 +146,6 @@ class BackgroundHTMLParser {
int pending_csp_meta_token_index_; int pending_csp_meta_token_index_;
bool starting_script_; bool starting_script_;
bool should_coalesce_chunks_;
DISALLOW_COPY_AND_ASSIGN(BackgroundHTMLParser); DISALLOW_COPY_AND_ASSIGN(BackgroundHTMLParser);
}; };
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment