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

Side by Side Diff: chrome/browser/plugins/plugin_finder_unittest.cc

Issue 10910168: Separate plugin_metadata from plugin_installer, thread-safe plugin_finder (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: .. Created 8 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
« no previous file with comments | « chrome/browser/plugins/plugin_finder.cc ('k') | chrome/browser/plugins/plugin_installer.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 "chrome/browser/plugins/plugin_finder.h" 5 #include "chrome/browser/plugins/plugin_finder.h"
6 6
7 #include "base/values.h" 7 #include "base/values.h"
8 #include "chrome/browser/plugins/plugin_installer.h" 8 #include "chrome/browser/plugins/plugin_metadata.h"
9 #include "testing/gtest/include/gtest/gtest.h" 9 #include "testing/gtest/include/gtest/gtest.h"
10 #include "webkit/plugins/npapi/plugin_list.h" 10 #include "webkit/plugins/npapi/plugin_list.h"
11 11
12 using base::DictionaryValue; 12 using base::DictionaryValue;
13 using webkit::npapi::PluginGroup;
14 using webkit::npapi::PluginList; 13 using webkit::npapi::PluginList;
15 14
16 TEST(PluginFinderTest, JsonSyntax) { 15 TEST(PluginFinderTest, JsonSyntax) {
17 scoped_ptr<DictionaryValue> plugin_list(PluginFinder::LoadPluginList()); 16 scoped_ptr<DictionaryValue> plugin_list(PluginFinder::LoadPluginList());
18 ASSERT_TRUE(plugin_list.get()); 17 ASSERT_TRUE(plugin_list.get());
19 for (DictionaryValue::Iterator plugin_it(*plugin_list); 18 for (DictionaryValue::Iterator plugin_it(*plugin_list);
20 plugin_it.HasNext(); plugin_it.Advance()) { 19 plugin_it.HasNext(); plugin_it.Advance()) {
21 const DictionaryValue* plugin = NULL; 20 const DictionaryValue* plugin = NULL;
22 ASSERT_TRUE(plugin_it.value().GetAsDictionary(&plugin)); 21 ASSERT_TRUE(plugin_it.value().GetAsDictionary(&plugin));
23 std::string dummy_str; 22 std::string dummy_str;
(...skipping 19 matching lines...) Expand all
43 if (!plugin->GetList("versions", &versions)) 42 if (!plugin->GetList("versions", &versions))
44 continue; 43 continue;
45 44
46 for (ListValue::const_iterator it = versions->begin(); 45 for (ListValue::const_iterator it = versions->begin();
47 it != versions->end(); ++it) { 46 it != versions->end(); ++it) {
48 DictionaryValue* version_dict = NULL; 47 DictionaryValue* version_dict = NULL;
49 ASSERT_TRUE((*it)->GetAsDictionary(&version_dict)); 48 ASSERT_TRUE((*it)->GetAsDictionary(&version_dict));
50 EXPECT_TRUE(version_dict->GetString("version", &dummy_str)); 49 EXPECT_TRUE(version_dict->GetString("version", &dummy_str));
51 std::string status_str; 50 std::string status_str;
52 EXPECT_TRUE(version_dict->GetString("status", &status_str)); 51 EXPECT_TRUE(version_dict->GetString("status", &status_str));
53 PluginInstaller::SecurityStatus status = 52 PluginMetadata::SecurityStatus status =
54 PluginInstaller::SECURITY_STATUS_UP_TO_DATE; 53 PluginMetadata::SECURITY_STATUS_UP_TO_DATE;
55 EXPECT_TRUE(PluginInstaller::ParseSecurityStatus(status_str, &status)) 54 EXPECT_TRUE(PluginMetadata::ParseSecurityStatus(status_str, &status))
56 << "Invalid security status \"" << status_str << "\""; 55 << "Invalid security status \"" << status_str << "\"";
57 } 56 }
58 } 57 }
59 } 58 }
OLDNEW
« no previous file with comments | « chrome/browser/plugins/plugin_finder.cc ('k') | chrome/browser/plugins/plugin_installer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698