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

Side by Side Diff: chrome/browser/extensions/extension_messages_apitest.cc

Issue 16844020: app_mode: Add runtime.onRestartRequired event. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: address comments in #1, add test and remove onBrowserUpdateAvailable Created 7 years, 6 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 #include "base/files/file_path.h" 5 #include "base/files/file_path.h"
6 #include "base/path_service.h" 6 #include "base/path_service.h"
7 #include "base/strings/string_number_conversions.h" 7 #include "base/strings/string_number_conversions.h"
8 #include "base/strings/stringprintf.h" 8 #include "base/strings/stringprintf.h"
9 #include "base/values.h" 9 #include "base/values.h"
10 #include "chrome/browser/extensions/event_router.h" 10 #include "chrome/browser/extensions/event_router.h"
(...skipping 149 matching lines...) Expand 10 before | Expand all | Expand 10 after
160 "getURL", 160 "getURL",
161 "reload", 161 "reload",
162 "requestUpdateCheck", 162 "requestUpdateCheck",
163 "connectNative", 163 "connectNative",
164 "sendNativeMessage", 164 "sendNativeMessage",
165 "onStartup", 165 "onStartup",
166 "onInstalled", 166 "onInstalled",
167 "onSuspend", 167 "onSuspend",
168 "onSuspendCanceled", 168 "onSuspendCanceled",
169 "onUpdateAvailable", 169 "onUpdateAvailable",
170 "onBrowserUpdateAvailable",
171 "onConnect", 170 "onConnect",
172 "onConnectExternal", 171 "onConnectExternal",
173 "onMessage", 172 "onMessage",
174 "onMessageExternal", 173 "onMessageExternal",
174 "onRestartRequired",
175 "id", 175 "id",
176 }; 176 };
177 return AreAnyRuntimePropertiesDefined(std::vector<std::string>( 177 return AreAnyRuntimePropertiesDefined(std::vector<std::string>(
178 non_messaging_apis, 178 non_messaging_apis,
179 non_messaging_apis + arraysize(non_messaging_apis))); 179 non_messaging_apis + arraysize(non_messaging_apis)));
180 } 180 }
181 181
182 GURL GetURLForPath(const std::string& host, const std::string& path) { 182 GURL GetURLForPath(const std::string& host, const std::string& path) {
183 std::string port = base::IntToString(embedded_test_server()->port()); 183 std::string port = base::IntToString(embedded_test_server()->port());
184 GURL::Replacements replacements; 184 GURL::Replacements replacements;
(...skipping 85 matching lines...) Expand 10 before | Expand all | Expand 10 after
270 ui_test_utils::NavigateToURL(browser(), kChromiumOrgUrl); 270 ui_test_utils::NavigateToURL(browser(), kChromiumOrgUrl);
271 EXPECT_EQ(COULD_NOT_ESTABLISH_CONNECTION_ERROR, 271 EXPECT_EQ(COULD_NOT_ESTABLISH_CONNECTION_ERROR,
272 CanConnectAndSendMessages(not_connectable->id())); 272 CanConnectAndSendMessages(not_connectable->id()));
273 EXPECT_FALSE(AreAnyNonWebApisDefined()); 273 EXPECT_FALSE(AreAnyNonWebApisDefined());
274 274
275 ui_test_utils::NavigateToURL(browser(), kGoogleComUrl); 275 ui_test_utils::NavigateToURL(browser(), kGoogleComUrl);
276 EXPECT_EQ(COULD_NOT_ESTABLISH_CONNECTION_ERROR, 276 EXPECT_EQ(COULD_NOT_ESTABLISH_CONNECTION_ERROR,
277 CanConnectAndSendMessages(not_connectable->id())); 277 CanConnectAndSendMessages(not_connectable->id()));
278 EXPECT_FALSE(AreAnyNonWebApisDefined()); 278 EXPECT_FALSE(AreAnyNonWebApisDefined());
279 } 279 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698