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

Side by Side Diff: chrome/test/base/find_in_page_observer.h

Issue 653773004: Standardize usage of virtual/override/final in chrome/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_TEST_BASE_FIND_IN_PAGE_OBSERVER_H_ 5 #ifndef CHROME_TEST_BASE_FIND_IN_PAGE_OBSERVER_H_
6 #define CHROME_TEST_BASE_FIND_IN_PAGE_OBSERVER_H_ 6 #define CHROME_TEST_BASE_FIND_IN_PAGE_OBSERVER_H_
7 7
8 #include "base/memory/ref_counted.h" 8 #include "base/memory/ref_counted.h"
9 #include "content/public/browser/notification_observer.h" 9 #include "content/public/browser/notification_observer.h"
10 #include "content/public/browser/notification_registrar.h" 10 #include "content/public/browser/notification_registrar.h"
(...skipping 10 matching lines...) Expand all
21 // are available. Typical usage: 21 // are available. Typical usage:
22 // FindInPageWchar(); 22 // FindInPageWchar();
23 // FindInPageNotificationObserver observer(tab); 23 // FindInPageNotificationObserver observer(tab);
24 // observer.Wait(); 24 // observer.Wait();
25 25
26 // Always construct FindInPageNotificationObserver AFTER initiating the search. 26 // Always construct FindInPageNotificationObserver AFTER initiating the search.
27 // It captures the current search ID in constructor and waits for it only. 27 // It captures the current search ID in constructor and waits for it only.
28 class FindInPageNotificationObserver : public content::NotificationObserver { 28 class FindInPageNotificationObserver : public content::NotificationObserver {
29 public: 29 public:
30 explicit FindInPageNotificationObserver(content::WebContents* parent_tab); 30 explicit FindInPageNotificationObserver(content::WebContents* parent_tab);
31 virtual ~FindInPageNotificationObserver(); 31 ~FindInPageNotificationObserver() override;
32 32
33 void Wait(); 33 void Wait();
34 34
35 int active_match_ordinal() const { return active_match_ordinal_; } 35 int active_match_ordinal() const { return active_match_ordinal_; }
36 int number_of_matches() const { return number_of_matches_; } 36 int number_of_matches() const { return number_of_matches_; }
37 gfx::Rect selection_rect() const { return selection_rect_; } 37 gfx::Rect selection_rect() const { return selection_rect_; }
38 38
39 private: 39 private:
40 virtual void Observe(int type, 40 void Observe(int type,
41 const content::NotificationSource& source, 41 const content::NotificationSource& source,
42 const content::NotificationDetails& details) override; 42 const content::NotificationDetails& details) override;
43 43
44 content::NotificationRegistrar registrar_; 44 content::NotificationRegistrar registrar_;
45 // We will at some point (before final update) be notified of the ordinal and 45 // We will at some point (before final update) be notified of the ordinal and
46 // we need to preserve it so we can send it later. 46 // we need to preserve it so we can send it later.
47 int active_match_ordinal_; 47 int active_match_ordinal_;
48 int number_of_matches_; 48 int number_of_matches_;
49 gfx::Rect selection_rect_; 49 gfx::Rect selection_rect_;
50 // The id of the current find request, obtained from WebContents. Allows us 50 // The id of the current find request, obtained from WebContents. Allows us
51 // to monitor when the search completes. 51 // to monitor when the search completes.
52 int current_find_request_id_; 52 int current_find_request_id_;
53 scoped_refptr<content::MessageLoopRunner> message_loop_runner_; 53 scoped_refptr<content::MessageLoopRunner> message_loop_runner_;
54 54
55 bool seen_; // true after transition to expected state has been seen 55 bool seen_; // true after transition to expected state has been seen
56 bool running_; // indicates whether message loop is running 56 bool running_; // indicates whether message loop is running
57 57
58 DISALLOW_COPY_AND_ASSIGN(FindInPageNotificationObserver); 58 DISALLOW_COPY_AND_ASSIGN(FindInPageNotificationObserver);
59 }; 59 };
60 60
61 } // namespace ui_test_utils 61 } // namespace ui_test_utils
62 62
63 #endif // CHROME_TEST_BASE_FIND_IN_PAGE_OBSERVER_H_ 63 #endif // CHROME_TEST_BASE_FIND_IN_PAGE_OBSERVER_H_
OLDNEW
« no previous file with comments | « chrome/test/base/chrome_unit_test_suite.h ('k') | chrome/test/base/history_index_restore_observer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698