Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(186)

Unified Diff: ios/web/navigation/navigation_manager_impl.mm

Issue 2705293014: Created web::UserAgentType. (Closed)
Patch Set: self review Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ios/web/navigation/navigation_manager_impl.mm
diff --git a/ios/web/navigation/navigation_manager_impl.mm b/ios/web/navigation/navigation_manager_impl.mm
index bb16cba464bb1a3689eeca3a77fd606ca20e5cfe..348ebe589a0163632298ae6a3c9f9d9a859f9cd4 100644
--- a/ios/web/navigation/navigation_manager_impl.mm
+++ b/ios/web/navigation/navigation_manager_impl.mm
@@ -16,6 +16,7 @@
#include "ios/web/navigation/navigation_manager_facade_delegate.h"
#include "ios/web/public/load_committed_details.h"
#import "ios/web/public/navigation_item.h"
+#import "ios/web/public/web_client.h"
#import "ios/web/public/web_state/web_state.h"
#include "ui/base/page_transition_types.h"
@@ -184,13 +185,26 @@ bool AreURLsInPageNavigation(const GURL& existing_url, const GURL& new_url) {
transition:navigation_type
initiationType:initiation_type];
- // Do nothing if pending item is the same as last committed item.
- if (GetPendingItem()) {
- bool use_desktop_user_agent =
- override_desktop_user_agent_for_next_pending_item_ ||
- (GetLastCommittedItem() &&
- GetLastCommittedItem()->IsOverridingUserAgent());
- GetPendingItem()->SetIsOverridingUserAgent(use_desktop_user_agent);
+ // Set the user agent type for web URLs.
+ NavigationItem* pending_item = GetPendingItem();
+ if (pending_item) {
liaoyuke 2017/02/25 01:28:19 Can we change this to: if (!pending_item) ret
kkhorimoto 2017/02/25 01:35:05 Done.
+ // |override_desktop_user_agent_for_next_pending_item_| must be false if
+ // |pending_item|'s UserAgentType is NONE, as requesting a desktop user
+ // agent should be disabled for app-specific URLs.
+ DCHECK(pending_item->GetUserAgentType() != UserAgentType::NONE ||
+ !override_desktop_user_agent_for_next_pending_item_);
+
+ if (pending_item->GetUserAgentType() != UserAgentType::NONE) {
+ NavigationItem* last_web_item = GetLastCommittedWebItem();
+ DCHECK(!last_web_item ||
+ last_web_item->GetUserAgentType() != UserAgentType::NONE);
+ bool use_desktop_user_agent =
+ override_desktop_user_agent_for_next_pending_item_ ||
+ (last_web_item &&
+ last_web_item->GetUserAgentType() == UserAgentType::DESKTOP);
+ if (use_desktop_user_agent)
+ pending_item->SetUserAgentType(UserAgentType::DESKTOP);
+ }
override_desktop_user_agent_for_next_pending_item_ = false;
}
}
@@ -408,10 +422,15 @@ bool AreURLsInPageNavigation(const GURL& existing_url, const GURL& new_url) {
}
void NavigationManagerImpl::OverrideDesktopUserAgentForNextPendingItem() {
- if (GetPendingItem())
- GetPendingItem()->SetIsOverridingUserAgent(true);
- else
+ NavigationItem* pending_item = GetPendingItem();
+ if (pending_item) {
+ // The desktop user agent cannot be used for a pending navigation to an
+ // app-specific URL.
+ DCHECK_NE(pending_item->GetUserAgentType(), UserAgentType::NONE);
+ pending_item->SetUserAgentType(UserAgentType::DESKTOP);
+ } else {
override_desktop_user_agent_for_next_pending_item_ = true;
+ }
}
bool NavigationManagerImpl::IsRedirectItemAtIndex(int index) const {
@@ -421,4 +440,18 @@ bool AreURLsInPageNavigation(const GURL& existing_url, const GURL& new_url) {
return transition & ui::PAGE_TRANSITION_IS_REDIRECT_MASK;
}
+NavigationItem* NavigationManagerImpl::GetLastCommittedWebItem() const {
+ int index = GetCurrentItemIndex();
+ NavigationItemList items = [session_controller_ items];
+ if (index < 0 || index >= static_cast<int>(items.size()))
+ return nullptr;
+ WebClient* client = GetWebClient();
+ while (index >= 0) {
+ NavigationItem* item = items[index];
+ if (!client->IsAppSpecificURL(item->GetVirtualURL()))
+ return item;
+ }
+ return nullptr;
+}
+
} // namespace web

Powered by Google App Engine
This is Rietveld 408576698