Remove OffilnePageUtils::MaybeGetOfflineURLForOnlineURL |
||
Issue descriptionThis function is obsoleted, so it can be removed. Need to remove tests for this and add tests that use the async version (GetOfflineUrlForOnlineURL).
,
Jun 10 2016
,
Jun 15 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/61ebf56ef4cba2a43cf7a67091fc4f188f7716e4 commit 61ebf56ef4cba2a43cf7a67091fc4f188f7716e4 Author: weiran <weiran@google.com> Date: Fri Jun 10 19:53:21 2016 Remove obsoleted function and add test Remove OfflinePageUtils::MaybeGetOfflineURLForOnlineURL Add OfflinePageUtilsTest::GetOfflineURLForOnlineURL BUG= 618732 Review-Url: https://codereview.chromium.org/2059513002 Cr-Commit-Position: refs/heads/master@{#399255} [modify] https://crrev.com/61ebf56ef4cba2a43cf7a67091fc4f188f7716e4/chrome/browser/android/offline_pages/offline_page_utils.cc [modify] https://crrev.com/61ebf56ef4cba2a43cf7a67091fc4f188f7716e4/chrome/browser/android/offline_pages/offline_page_utils.h [modify] https://crrev.com/61ebf56ef4cba2a43cf7a67091fc4f188f7716e4/chrome/browser/android/offline_pages/offline_page_utils_unittest.cc |
||
►
Sign in to add a comment |
||
Comment 1 by bugdroid1@chromium.org
, Jun 10 2016