From 2bc00efe7bb826d683257264471878c05757f2bb Mon Sep 17 00:00:00 2001 From: adeshkp Date: Sat, 4 Aug 2018 14:22:42 +0530 Subject: Improve nsHtml5AtomTable performance (#693) * cpp change * h change --- parser/html/nsHtml5AtomTable.cpp | 14 ++++++++++++++ 1 file changed, 14 insertions(+) (limited to 'parser/html/nsHtml5AtomTable.cpp') diff --git a/parser/html/nsHtml5AtomTable.cpp b/parser/html/nsHtml5AtomTable.cpp index d523f58b0..d9c18485d 100644 --- a/parser/html/nsHtml5AtomTable.cpp +++ b/parser/html/nsHtml5AtomTable.cpp @@ -31,7 +31,11 @@ nsHtml5AtomTable::nsHtml5AtomTable() } nsHtml5AtomTable::~nsHtml5AtomTable() + : mRecentlyUsedParserAtoms{} { +#ifdef DEBUG + NS_GetMainThread(getter_AddRefs(mPermittedLookupThread)); +#endif } nsIAtom* @@ -44,13 +48,23 @@ nsHtml5AtomTable::GetAtom(const nsAString& aKey) NS_ASSERTION(mPermittedLookupThread == currentThread, "Wrong thread!"); } #endif + + uint32_t index = mozilla::HashString(aKey) % RECENTLY_USED_PARSER_ATOMS_SIZE; + nsIAtom* cachedAtom = mRecentlyUsedParserAtoms[index]; + if (cachedAtom && cachedAtom->Equals(aKey)) { + return cachedAtom; + } + nsIAtom* atom = NS_GetStaticAtom(aKey); if (atom) { + mRecentlyUsedParserAtoms[index] = atom; return atom; } nsHtml5AtomEntry* entry = mTable.PutEntry(aKey); if (!entry) { return nullptr; } + + mRecentlyUsedParserAtoms[index] = entry->GetAtom(); return entry->GetAtom(); } -- cgit v1.2.3