Opened 4 years ago

Last modified 4 years ago

#1937 new change

Clean WBPassthruSink

Reported by: sergz Assignee:
Priority: P4 Milestone:
Module: Adblock-Plus-for-Internet-Explorer Keywords: goodfirstbug
Cc: Blocked By:
Blocking: Platform: Internet Explorer
Ready: yes Confidential: no
Tester: Verified working:
Review URL(s):

Description

Background

  • There are WBPassthruSink::m_contentType and WBPassthruSink::m_boundDomain which we don't need to have as members anymore.
  • Methods WBPassthruSink::GetContentTypeFromMimeType, WBPassthruSink::GetContentTypeFromURL, and WBPassthruSink::GetContentType are referenced only from cpp file and they don't access any protected member of WBPassthruSink, thus we don't need to have them as members.

What to change

  • remove WBPassthruSink::m_contentType and WBPassthruSink::m_boundDomain and declare them in the function where they are used.
  • move WBPassthruSink::GetContentTypeFromMimeType, WBPassthruSink::GetContentTypeFromURL, and WBPassthruSink::GetContentType into anonymous namespace in cpp file.

Change History (1)

comment:1 Changed 4 years ago by oleksandr

  • Platform changed from Unknown to Internet Explorer
  • Priority changed from Unknown to P4
  • Ready set
Note: See TracTickets for help on using tickets.