forked from organicmaps/organicmaps
[android] Fixed review notes
https://github.com/mapsme/omim/pull/11137#discussion_r299845592
This commit is contained in:
parent
70cb6e471e
commit
cf6fb5d27d
1 changed files with 3 additions and 9 deletions
|
@ -271,14 +271,8 @@ public class BookmarksCatalogFragment extends BaseWebViewMwmFragment
|
|||
if (fragment == null)
|
||||
return false;
|
||||
|
||||
return fragment.downloadBookmark(url);
|
||||
}
|
||||
boolean result = fragment.downloadBookmark(url);
|
||||
|
||||
@Nullable
|
||||
@Override
|
||||
public WebResourceResponse shouldInterceptRequest(WebView view, String url)
|
||||
{
|
||||
WebResourceResponse webResourceResponse = super.shouldInterceptRequest(view, url);
|
||||
Uri uri = Uri.parse(url);
|
||||
List<String> pathSegments = uri.getPathSegments();
|
||||
for (String each : pathSegments)
|
||||
|
@ -286,10 +280,10 @@ public class BookmarksCatalogFragment extends BaseWebViewMwmFragment
|
|||
if (TextUtils.equals(each, SUBSCRIBE_PATH_SEGMENT))
|
||||
{
|
||||
openSubscriptionScreen();
|
||||
return webResourceResponse;
|
||||
return true;
|
||||
}
|
||||
}
|
||||
return webResourceResponse;
|
||||
return result;
|
||||
}
|
||||
|
||||
private void openSubscriptionScreen()
|
||||
|
|
Loading…
Add table
Reference in a new issue