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

Unified Diff: chrome/browser/extensions/extension_tabs_module.cc

Issue 7714004: base: Add AsList() function to Value API. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: OVERRIDE Created 9 years, 4 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/extensions/extension_tabs_module.cc
diff --git a/chrome/browser/extensions/extension_tabs_module.cc b/chrome/browser/extensions/extension_tabs_module.cc
index a949b4df1de9d75d041314d9248999ed25cfe9b4..61c5eacf939b5e8a9a2e8255ec8bdcdc5d91aab2 100644
--- a/chrome/browser/extensions/extension_tabs_module.cc
+++ b/chrome/browser/extensions/extension_tabs_module.cc
@@ -417,7 +417,7 @@ bool CreateWindowFunction::RunImpl() {
std::string url_string;
url_value->GetAsString(&url_string);
url_strings.push_back(url_string);
- } else if (url_value->IsType(Value::TYPE_LIST)) {
+ } else if (url_value->AsList()) {
const ListValue* url_list = static_cast<const ListValue*>(url_value);
Evan Martin 2011/08/23 18:15:26 same transformation here (can set url_list in the
tfarina 2011/08/23 18:26:36 Done.
for (size_t i = 0; i < url_list->GetSize(); ++i) {
std::string url_string;

Powered by Google App Engine
This is Rietveld 408576698