Refactoring: Update OfflinePageModel::GetPagesByURL and OfflinePageUtils::SelectPagesForURL to minimize namespace and tab ID handling |
||
Issue descriptionUpdate OfflinePageModel::GetPagesByURL and OfflinePageUtils::SelectPagesForURL to minimize namespace and tab ID handling Because the former is used almost exclusively by the latter we can simply a lot. This may require a bit work around duplicate handling, but overall the code could be much simplified.
,
Feb 3 2018
there might not be a huge difference, I have a patch and will upload later.
,
Feb 3 2018
,
Feb 8 2018
This doesn't seem like a simplification, so closing as won't fix. This might be aligned with the exposing of tasks. |
||
►
Sign in to add a comment |
||
Comment 1 by dim...@chromium.org
, Jan 31 2018Owner: romax@chromium.org
Status: Assigned (was: Untriaged)