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

Unified Diff: ios/clean/chrome/browser/ui/omnibox/location_bar_mediator.mm

Issue 2761343002: [ios] Adds LocationBarCoordinator. (Closed)
Patch Set: Rebased. Created 3 years, 8 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/clean/chrome/browser/ui/omnibox/location_bar_mediator.mm
diff --git a/ios/clean/chrome/browser/ui/omnibox/location_bar_mediator.mm b/ios/clean/chrome/browser/ui/omnibox/location_bar_mediator.mm
new file mode 100644
index 0000000000000000000000000000000000000000..a95ce3e211a1ffa506818de2a5729be22da4ba50
--- /dev/null
+++ b/ios/clean/chrome/browser/ui/omnibox/location_bar_mediator.mm
@@ -0,0 +1,191 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#import "ios/clean/chrome/browser/ui/omnibox/location_bar_mediator.h"
+
+#include "base/memory/ptr_util.h"
+#include "base/strings/utf_string_conversions.h"
+#include "ios/chrome/browser/ui/toolbar/toolbar_model_delegate_ios.h"
+#include "ios/chrome/browser/ui/toolbar/toolbar_model_impl_ios.h"
+#include "ios/shared/chrome/browser/tabs/web_state_list.h"
+#include "ios/shared/chrome/browser/tabs/web_state_list_observer_bridge.h"
+#import "ios/shared/chrome/browser/ui/omnibox/location_bar_controller.h"
+#import "ios/web/public/navigation_manager.h"
+#import "ios/web/public/web_state/web_state.h"
+#import "ios/web/public/web_state/web_state_observer_bridge.h"
+
+#if !defined(__has_feature) || !__has_feature(objc_arc)
+#error "This file requires ARC support."
+#endif
+
+@interface LocationBarMediator ()<CRWWebStateObserver,
+ LocationBarDelegate,
+ WebStateListObserving>
+@property(nonatomic, readwrite, assign) WebStateList* webStateList;
+@end
+
+@implementation LocationBarMediator {
+ // Observes the WebStateList so that this mediator can update the UI when the
+ // active WebState changes.
+ std::unique_ptr<WebStateListObserverBridge> _webStateListObserver;
+
+ // Used to update the UI in response to WebState observer notifications. This
+ // observer is always observing the currently-active WebState and may be
+ // nullptr if no WebState is currently active.
+ std::unique_ptr<web::WebStateObserverBridge> _webStateObserver;
+
+ // The LocationBarController that wraps OmniboxViewIOS and
+ // OmniboxTextFieldIOS. This mediator updates the UI through |_locationBar|
+ // rather than through a consumer.
+ std::unique_ptr<LocationBarController> _locationBar;
+
+ // The ToolbarModelDelegate, backed by |_webStateList|.
+ std::unique_ptr<ToolbarModelDelegateIOS> _toolbarModelDelegate;
+
+ // The ToolbarModel that backs |_locationBar|.
+ std::unique_ptr<ToolbarModelImplIOS> _toolbarModel;
+}
+
+@synthesize webStateList = _webStateList;
+
+- (instancetype)initWithWebStateList:(WebStateList*)webStateList {
+ if ((self = [super init])) {
+ _webStateList = webStateList;
+
+ _webStateListObserver = base::MakeUnique<WebStateListObserverBridge>(self);
+ _webStateList->AddObserver(_webStateListObserver.get());
+ web::WebState* webState = _webStateList->GetActiveWebState();
+ if (webState) {
+ _webStateObserver =
+ base::MakeUnique<web::WebStateObserverBridge>(webState, self);
+ }
+
+ _toolbarModelDelegate =
+ base::MakeUnique<ToolbarModelDelegateIOS>(webStateList);
+ _toolbarModel =
+ base::MakeUnique<ToolbarModelImplIOS>(_toolbarModelDelegate.get());
+ }
+ return self;
+}
+
+- (void)dealloc {
+ _webStateList->RemoveObserver(_webStateListObserver.get());
+}
+
+- (void)setLocationBar:(std::unique_ptr<LocationBarController>)locationBar {
+ _locationBar = std::move(locationBar);
+ _locationBar->OnToolbarUpdated();
+}
+
+#pragma mark - LocationBarDelegate
+
+- (void)loadGURLFromLocationBar:(const GURL&)url
+ transition:(ui::PageTransition)transition {
+ web::WebState* webState = _webStateList->GetActiveWebState();
+ DCHECK(webState);
+
+ if (url.SchemeIs(url::kJavaScriptScheme)) {
+ // TODO(crbug.com/708341): Percent-unescape the url content first.
+ webState->ExecuteJavaScript(base::UTF8ToUTF16(url.GetContent()));
+ return;
+ } else {
+ // TODO(crbug.com/708341): Use WebState::OpenURL() here, because that method
+ // can handle different WindowOpenDispositions. The code below assumes
+ // CURRENT_TAB.
+
+ // When opening a URL, force the omnibox to resign first responder. This
+ // will also close the popup.
+ web::NavigationManager::WebLoadParams loadParams(url);
+ loadParams.transition_type = transition;
+ loadParams.is_renderer_initiated = false;
+ webState->GetNavigationManager()->LoadURLWithParams(loadParams);
+ }
+
+ _locationBar->HideKeyboardAndEndEditing();
+ _locationBar->OnToolbarUpdated();
+}
+
+- (void)locationBarHasBecomeFirstResponder {
+ // TODO(crbug.com/708341): Implement this method or edit this comment with an
+ // explanation of what this method needs to do.
+}
+
+- (void)locationBarHasResignedFirstResponder {
+ // TODO(crbug.com/708341): Implement this method or edit this comment with an
+ // explanation of what this method needs to do.
+}
+
+- (void)locationBarBeganEdit {
+ // TODO(crbug.com/708341): Implement this method or edit this comment with an
+ // explanation of what this method needs to do.
+}
+
+- (void)locationBarChanged {
+ // TODO(crbug.com/708341): Implement this method or edit this comment with an
+ // explanation of what this method needs to do.
+}
+
+- (web::WebState*)getWebState {
+ return _webStateList->GetActiveWebState();
+}
+
+- (ToolbarModel*)toolbarModel {
+ return _toolbarModel->GetToolbarModel();
+}
+
+#pragma mark - WebStateListObserver
+
+- (void)webStateList:(WebStateList*)webStateList
+ didChangeActiveWebState:(web::WebState*)newWebState
+ oldWebState:(web::WebState*)oldWebState
+ atIndex:(int)atIndex
+ userAction:(BOOL)userAction {
+ // If the omnibox is focused, force it to resign first responder when the
+ // active tab changes.
+ _locationBar->HideKeyboardAndEndEditing();
+
+ if (newWebState) {
+ _webStateObserver =
+ base::MakeUnique<web::WebStateObserverBridge>(newWebState, self);
+ } else {
+ _webStateObserver = nullptr;
+ }
+
+ _locationBar->OnToolbarUpdated();
+}
+
+#pragma mark - WebStateObserver
+
+// WebState navigation events that could potentially affect the contents of the
+// omnibox are caught below and used to drive updates to the UI.
+
+- (void)webState:(web::WebState*)webState
+ didStartProvisionalNavigationForURL:(const GURL&)URL {
+ _locationBar->OnToolbarUpdated();
+}
+
+- (void)webState:(web::WebState*)webState
+ didCommitNavigationWithDetails:
+ (const web::LoadCommittedDetails&)load_details {
+ _locationBar->OnToolbarUpdated();
+}
+
+- (void)webState:(web::WebState*)webState
+ didFinishNavigation:(web::NavigationContext*)navigation {
+ _locationBar->OnToolbarUpdated();
+}
+
+- (void)webState:(web::WebState*)webState didLoadPageWithSuccess:(BOOL)success {
+ _locationBar->OnToolbarUpdated();
+}
+
+- (void)webStateDidChangeVisibleSecurityState:(web::WebState*)webState {
+ _locationBar->OnToolbarUpdated();
+}
+
+- (void)webStateDestroyed:(web::WebState*)webState {
+ _webStateObserver = nullptr;
+}
+
+@end

Powered by Google App Engine
This is Rietveld 408576698