Add more unit tests for Blink loader code |
|||||
Issue descriptionThe classes of Blink's loader such as Resource/ResourceLoader/ResourceFetcher/MemoryCache have many corner cases that are not tested by unit tests (or by even any tests). Adding more tests will help clarifying the assumption/behavior of the loader code as well as finding existing bugs and catching regressions.
,
Jul 15 2016
I'm working on TextResourceTest now.
,
Jul 27 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/de7903c84701c01184c6ef2593c274f4d5de2bb0 commit de7903c84701c01184c6ef2593c274f4d5de2bb0 Author: hyungwook.lee <hyungwook.lee@navercorp.com> Date: Wed Jul 27 02:55:22 2016 Add FragmentIdentifier(), MakeLiveAndDead(), RemoveURLFromCache() unit tests for MemoryCache. BUG=625597 R=morrita@chromium.org Review-Url: https://codereview.chromium.org/2140993002 Cr-Commit-Position: refs/heads/master@{#408026} [modify] https://crrev.com/de7903c84701c01184c6ef2593c274f4d5de2bb0/third_party/WebKit/Source/core/fetch/MemoryCacheTest.cpp
,
Jul 27 2017
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 31 2017
,
Jul 31
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Aug 17
loading triage: hiroshige-san, do you think we need more works for this bug? |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by bugdroid1@chromium.org
, Jul 4 2016