Autofill - get rid of one more RenderViewObserver method.

chrome/browser/ui/autofill/chrome_autofill_client.cc

 // Copyright 2014 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.
 
 #include "chrome/browser/ui/autofill/chrome_autofill_client.h"
 
 #include "base/logging.h"
 #include "base/prefs/pref_service.h"
 #include "chrome/browser/autofill/autofill_cc_infobar_delegate.h"
 #include "chrome/browser/autofill/personal_data_manager_factory.h"
@@ skipping 23 matching lines @@
 #include "chrome/browser/ui/android/autofill/autofill_logger_android.h"
 #else
 #include "components/ui/zoom/zoom_controller.h"
 #endif
 
 DEFINE_WEB_CONTENTS_USER_DATA_KEY(autofill::ChromeAutofillClient);
 
 namespace autofill {
 
 ChromeAutofillClient::ChromeAutofillClient(content::WebContents* web_contents)
-    : content::WebContentsObserver(web_contents), weak_pointer_factory_(this) {
+    : content::WebContentsObserver(web_contents),
+      weak_pointer_factory_(this),
+      screen_observer_(this) {
   DCHECK(web_contents);
 
 #if !defined(OS_ANDROID)
   // Since ZoomController is also a WebContentsObserver, we need to be careful
   // about disconnecting from it since the relative order of destruction of
   // WebContentsObservers is not guaranteed. ZoomController silently clears
   // its ZoomObserver list during WebContentsDestroyed() so there's no need
   // to explicitly remove ourselves on destruction.
   ui_zoom::ZoomController* zoom_controller =
       ui_zoom::ZoomController::FromWebContents(web_contents);
   // There may not always be a ZoomController, e.g. in tests.
   if (zoom_controller)
     zoom_controller->AddObserver(this);
 #endif
 
 #if defined(OS_MACOSX) && !defined(OS_IOS)
   RegisterForKeystoneNotifications();
 #endif  // defined(OS_MACOSX) && !defined(OS_IOS)
+
+  screen_observer_.Add(
+      gfx::Screen::GetScreenFor(web_contents->GetNativeView()));
 }
 
 ChromeAutofillClient::~ChromeAutofillClient() {
   // NOTE: It is too late to clean up the autofill popup; that cleanup process
   // requires that the WebContents instance still be valid and it is not at
   // this point (in particular, the WebContentsImpl destructor has already
   // finished running and we are now in the base class destructor).
   DCHECK(!popup_controller_);
 #if defined(OS_MACOSX) && !defined(OS_IOS)
   UnregisterFromKeystoneNotifications();
@@ skipping 133 matching lines @@
 
 void ChromeAutofillClient::WebContentsDestroyed() {
   HideAutofillPopup();
 }
 
 void ChromeAutofillClient::OnZoomChanged(
     const ui_zoom::ZoomController::ZoomChangedEventData& data) {
   HideAutofillPopup();
 }
 
+void ChromeAutofillClient::OnDisplayAdded(const gfx::Display& new_display) {
+}
+
+void ChromeAutofillClient::OnDisplayRemoved(const gfx::Display& old_display) {
+}
+
+void ChromeAutofillClient::OnDisplayMetricsChanged(const gfx::Display& display,
+                                                   uint32_t metrics) {
+  if (metrics | DisplayObserver::DISPLAY_METRIC_ROTATION)
+    HideAutofillPopup();
+}
+
 void ChromeAutofillClient::DetectAccountCreationForms(
     content::RenderFrameHost* rfh,
     const std::vector<autofill::FormStructure*>& forms) {
   password_manager::ContentPasswordManagerDriver* driver =
       password_manager::ContentPasswordManagerDriver::GetForRenderFrameHost(
           rfh);
   if (driver)
     driver->GetPasswordGenerationManager()->DetectAccountCreationForms(forms);
 }
 
 void ChromeAutofillClient::DidFillOrPreviewField(
     const base::string16& autofilled_value,
     const base::string16& profile_full_name) {
 #if defined(OS_ANDROID)
   AutofillLoggerAndroid::DidFillOrPreviewField(autofilled_value,
                                                profile_full_name);
 #endif  // defined(OS_ANDROID)
 }
 
 void ChromeAutofillClient::OnFirstUserGestureObserved() {
   web_contents()->SendToAllFrames(
       new AutofillMsg_FirstUserGestureObservedInTab(routing_id()));
 }
 
 }  // namespace autofill