Changeset 149198 in webkit
- Timestamp:
- Apr 26, 2013 11:24:08 AM (11 years ago)
- Location:
- trunk/Source/WebKit2
- Files:
-
- 3 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/Source/WebKit2/ChangeLog
r149196 r149198 1 2013-04-26 Andy Estes <aestes@apple.com> 2 3 [WebKit2] CustomProtocolManager's m_registeredSchemes HashSet can be accessed by multiple threads 4 https://bugs.webkit.org/show_bug.cgi?id=115267 5 6 Reviewed by Alexey Proskuryakov. 7 8 * Shared/Network/CustomProtocols/CustomProtocolManager.h: Added a Mutex to protect m_registeredSchemes. 9 * Shared/Network/CustomProtocols/mac/CustomProtocolManagerMac.mm: 10 (WebKit::CustomProtocolManager::registerScheme): Acquired the mutex before accessing m_registeredSchemes. 11 (WebKit::CustomProtocolManager::unregisterScheme): Ditto. 12 (WebKit::CustomProtocolManager::supportsScheme): Ditto. 13 1 14 2013-04-26 Alexey Proskuryakov <ap@apple.com> 2 15 -
trunk/Source/WebKit2/Shared/Network/CustomProtocols/CustomProtocolManager.h
r149194 r149198 97 97 98 98 HashSet<String> m_registeredSchemes; 99 Mutex m_registeredSchemesMutex; 99 100 ChildProcess* m_childProcess; 100 101 RefPtr<WorkQueue> m_messageQueue; -
trunk/Source/WebKit2/Shared/Network/CustomProtocols/mac/CustomProtocolManagerMac.mm
r149194 r149198 171 171 void CustomProtocolManager::registerScheme(const String& scheme) 172 172 { 173 MutexLocker locker(m_registeredSchemesMutex); 173 174 m_registeredSchemes.add(scheme); 174 175 } … … 176 177 void CustomProtocolManager::unregisterScheme(const String& scheme) 177 178 { 179 MutexLocker locker(m_registeredSchemesMutex); 178 180 m_registeredSchemes.remove(scheme); 179 181 } … … 181 183 bool CustomProtocolManager::supportsScheme(const String& scheme) 182 184 { 185 MutexLocker locker(m_registeredSchemesMutex); 183 186 return m_registeredSchemes.contains(scheme); 184 187 }
Note: See TracChangeset
for help on using the changeset viewer.