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

Unified Diff: chrome/browser/android/vr_shell/vr_omnibox.h

Issue 2647053002: Add initial VR interactive omnibox. (Closed)
Patch Set: Created 3 years, 11 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: chrome/browser/android/vr_shell/vr_omnibox.h
diff --git a/chrome/browser/android/vr_shell/vr_omnibox.h b/chrome/browser/android/vr_shell/vr_omnibox.h
new file mode 100644
index 0000000000000000000000000000000000000000..1989a1eb207890473802f457c83ac83e8c3e7435
--- /dev/null
+++ b/chrome/browser/android/vr_shell/vr_omnibox.h
@@ -0,0 +1,45 @@
+// 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.
+
+#ifndef CHROME_BROWSER_ANDROID_VR_SHELL_VR_OMNIBOX_H_
+#define CHROME_BROWSER_ANDROID_VR_SHELL_VR_OMNIBOX_H_
+
+#include "base/macros.h"
+#include "base/strings/string16.h"
mthiesse 2017/01/20 18:48:35 Looks like a lot of these includes don't need to b
cjgrant 2017/01/24 18:46:12 Scrubbed here and elsewhere (referred to lint outp
+#include "base/values.h"
+#include "chrome/browser/android/vr_shell/ui_interface.h"
+#include "chrome/browser/autocomplete/chrome_autocomplete_provider_client.h"
+#include "chrome/browser/profiles/profile_manager.h"
+#include "components/omnibox/browser/autocomplete_controller_delegate.h"
+
+class AutocompleteController;
+class AutocompleteInput;
+class Profile;
+
+namespace vr_shell {
+
+class UiInterface;
+
+class VrOmnibox : public AutocompleteControllerDelegate {
+ public:
+ explicit VrOmnibox(UiInterface* ui);
+ ~VrOmnibox() override;
+
+ void HandleInput(const base::DictionaryValue& dict);
+
+ private:
+ void OnResultChanged(bool default_match_changed) override;
+
+ UiInterface* ui_;
+
+ AutocompleteInput input_;
+ std::unique_ptr<Profile> profile_;
bshe 2017/01/23 16:24:37 do you need to own profile here? It looks like pro
cjgrant 2017/01/24 18:46:13 The profile handling looks a bit more interesting.
+ std::unique_ptr<AutocompleteController> autocomplete_controller_;
+
+ DISALLOW_COPY_AND_ASSIGN(VrOmnibox);
+};
+
+} // namespace vr_shell
+
+#endif // CHROME_BROWSER_ANDROID_VR_SHELL_VR_OMNIBOX_H_

Powered by Google App Engine
This is Rietveld 408576698