[webkit-reviews] review requested: [Bug 173630] [Curl] ResourceHandleManager violate the class responsibility of ResourceHandle : [Attachment 314164] Refactoring

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jun 29 14:18:20 PDT 2017


Basuke Suzuki <Basuke.Suzuki at sony.com> has asked  for review:
Bug 173630: [Curl] ResourceHandleManager violate the class responsibility of
ResourceHandle
https://bugs.webkit.org/show_bug.cgi?id=173630

Attachment 314164: Refactoring

https://bugs.webkit.org/attachment.cgi?id=314164&action=review




--- Comment #1 from Basuke Suzuki <Basuke.Suzuki at sony.com> ---
Created attachment 314164

  --> https://bugs.webkit.org/attachment.cgi?id=314164&action=review

Refactoring

This patch is simply move methods and functions from ResourceHandleManager to
ResourceHandle if possible. ResourceHandleManager apparently violent the class
policy and it takes care of ResourceHandle too much which prevents to improve
its own responsibility, which is managing ResourceHandle well.


More information about the webkit-reviews mailing list