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

Side by Side Diff: chrome/browser/android/service_tab_launcher.h

Issue 904453002: Feed back to the ServiceTabLauncher when the tab has been created. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@a-open-window
Patch Set: Created 5 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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_BROWSER_ANDROID_SERVICE_TAB_LAUNCHER_H_ 5 #ifndef CHROME_BROWSER_ANDROID_SERVICE_TAB_LAUNCHER_H_
6 #define CHROME_BROWSER_ANDROID_SERVICE_TAB_LAUNCHER_H_ 6 #define CHROME_BROWSER_ANDROID_SERVICE_TAB_LAUNCHER_H_
7 7
8 #include "base/android/jni_android.h" 8 #include "base/android/jni_android.h"
9 #include "base/callback_forward.h" 9 #include "base/callback_forward.h"
10 #include "base/id_map.h"
10 #include "base/memory/singleton.h" 11 #include "base/memory/singleton.h"
11 12
12 namespace content { 13 namespace content {
13 class BrowserContext; 14 class BrowserContext;
14 struct OpenURLParams; 15 struct OpenURLParams;
15 class WebContents; 16 class WebContents;
16 } 17 }
17 18
18 namespace chrome { 19 namespace chrome {
19 namespace android { 20 namespace android {
20 21
21 // Launcher for creating new tabs on Android from a background service, where 22 // Launcher for creating new tabs on Android from a background service, where
22 // there may not necessarily be an Activity or a tab model at all. 23 // there may not necessarily be an Activity or a tab model at all. When the
24 // tab has been launched, the user of this class will be informed with the
25 // content::WebContents instance associated with the tab.
23 class ServiceTabLauncher { 26 class ServiceTabLauncher {
27 using TabLaunchedCallback = base::Callback<void(content::WebContents*)>;
Maria 2015/02/19 22:50:50 is this C++ feature now allowed in chromium?
Peter Beverloo 2015/02/20 12:09:18 Yeah! :-) It's much nicer than typedefs. https://
28
24 public: 29 public:
25 // Returns the singleton instance of the service tab launcher. 30 // Returns the singleton instance of the service tab launcher.
26 static ServiceTabLauncher* GetInstance(); 31 static ServiceTabLauncher* GetInstance();
27 32
28 // Launches a new tab when we're in a Service rather than in an Activity. 33 // Launches a new tab when we're in a Service rather than in an Activity.
29 // |callback| will be invoked with the resulting content::WebContents* when 34 // |callback| will be invoked with the resulting content::WebContents* when
30 // the tab is avialable. This method must only be called from the UI thread. 35 // the tab is avialable. This method must only be called from the UI thread.
31 void LaunchTab(content::BrowserContext* browser_context, 36 void LaunchTab(content::BrowserContext* browser_context,
32 const content::OpenURLParams& params, 37 const content::OpenURLParams& params,
33 const base::Callback<void(content::WebContents*)>& callback); 38 const TabLaunchedCallback& callback);
39
40 // To be called when the tab for |request_id| has launched, with the
41 // associated |web_contents|. The WebContents must not yet have started
42 // the provisional load for the main frame of the navigation.
43 void OnTabLaunched(int request_id, content::WebContents* web_contents);
34 44
35 static bool RegisterServiceTabLauncher(JNIEnv* env); 45 static bool RegisterServiceTabLauncher(JNIEnv* env);
36 46
37 private: 47 private:
38 friend struct DefaultSingletonTraits<ServiceTabLauncher>; 48 friend struct DefaultSingletonTraits<ServiceTabLauncher>;
39 49
40 ServiceTabLauncher(); 50 ServiceTabLauncher();
41 ~ServiceTabLauncher(); 51 ~ServiceTabLauncher();
42 52
53 IDMap<TabLaunchedCallback> tab_launched_callbacks_;
54
43 base::android::ScopedJavaGlobalRef<jobject> java_object_; 55 base::android::ScopedJavaGlobalRef<jobject> java_object_;
44 56
45 DISALLOW_COPY_AND_ASSIGN(ServiceTabLauncher); 57 DISALLOW_COPY_AND_ASSIGN(ServiceTabLauncher);
46 }; 58 };
47 59
48 } // namespace android 60 } // namespace android
49 } // namespace chrome 61 } // namespace chrome
50 62
51 #endif // CHROME_BROWSER_ANDROID_SERVICE_TAB_LAUNCHER_H_ 63 #endif // CHROME_BROWSER_ANDROID_SERVICE_TAB_LAUNCHER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698