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

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

Issue 164039: Add module-level permissions to extensions. (Closed)
Patch Set: final nits Created 11 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 unified diff | Download patch
« no previous file with comments | « chrome/browser/extensions/extension_host.cc ('k') | chrome/browser/extensions/extensions_ui.cc » ('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) 2006-2009 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2006-2009 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 <algorithm> 5 #include <algorithm>
6 #include <vector> 6 #include <vector>
7 7
8 #include "base/command_line.h" 8 #include "base/command_line.h"
9 #include "base/file_path.h" 9 #include "base/file_path.h"
10 #include "base/file_util.h" 10 #include "base/file_util.h"
(...skipping 502 matching lines...) Expand 10 before | Expand all | Expand 10 after
513 NormalizeSeperators(scripts[0].js_scripts()[1].path().value())); 513 NormalizeSeperators(scripts[0].js_scripts()[1].path().value()));
514 EXPECT_TRUE(extension->plugins().empty()); 514 EXPECT_TRUE(extension->plugins().empty());
515 EXPECT_EQ(1u, scripts[1].url_patterns().size()); 515 EXPECT_EQ(1u, scripts[1].url_patterns().size());
516 EXPECT_EQ("http://*.news.com/*", scripts[1].url_patterns()[0].GetAsString()); 516 EXPECT_EQ("http://*.news.com/*", scripts[1].url_patterns()[0].GetAsString());
517 EXPECT_EQ( 517 EXPECT_EQ(
518 NormalizeSeperators(extension->path() 518 NormalizeSeperators(extension->path()
519 .AppendASCII("js_files") 519 .AppendASCII("js_files")
520 .AppendASCII("script3.js") 520 .AppendASCII("script3.js")
521 .value()), 521 .value()),
522 NormalizeSeperators(scripts[1].js_scripts()[0].path().value())); 522 NormalizeSeperators(scripts[1].js_scripts()[0].path().value()));
523 const std::vector<URLPattern> permissions = extension->permissions(); 523 const std::vector<URLPattern> permissions = extension->host_permissions();
524 ASSERT_EQ(2u, permissions.size()); 524 ASSERT_EQ(2u, permissions.size());
525 EXPECT_EQ("http://*.google.com/*", permissions[0].GetAsString()); 525 EXPECT_EQ("http://*.google.com/*", permissions[0].GetAsString());
526 EXPECT_EQ("https://*.google.com/*", permissions[1].GetAsString()); 526 EXPECT_EQ("https://*.google.com/*", permissions[1].GetAsString());
527 ASSERT_EQ(2u, toolstrips.size()); 527 ASSERT_EQ(2u, toolstrips.size());
528 EXPECT_EQ(extension->GetResourceURL("toolstrip1.html"), 528 EXPECT_EQ(extension->GetResourceURL("toolstrip1.html"),
529 toolstrips[0].toolstrip); 529 toolstrips[0].toolstrip);
530 EXPECT_EQ(extension->GetResourceURL("lorem_ipsum.html"), 530 EXPECT_EQ(extension->GetResourceURL("lorem_ipsum.html"),
531 toolstrips[0].mole); 531 toolstrips[0].mole);
532 EXPECT_EQ(200, toolstrips[0].mole_height); 532 EXPECT_EQ(200, toolstrips[0].mole_height);
533 EXPECT_EQ(extension->GetResourceURL("toolstrip2.html"), 533 EXPECT_EQ(extension->GetResourceURL("toolstrip2.html"),
(...skipping 885 matching lines...) Expand 10 before | Expand all | Expand 10 after
1419 1419
1420 recorder.set_ready(false); 1420 recorder.set_ready(false);
1421 command_line.reset(new CommandLine(L"")); 1421 command_line.reset(new CommandLine(L""));
1422 service = new ExtensionsService(&profile, command_line.get(), 1422 service = new ExtensionsService(&profile, command_line.get(),
1423 profile.GetPrefs(), install_dir, &loop, &loop, false); 1423 profile.GetPrefs(), install_dir, &loop, &loop, false);
1424 EXPECT_TRUE(service->extensions_enabled()); 1424 EXPECT_TRUE(service->extensions_enabled());
1425 service->Init(); 1425 service->Init();
1426 loop.RunAllPending(); 1426 loop.RunAllPending();
1427 EXPECT_TRUE(recorder.ready()); 1427 EXPECT_TRUE(recorder.ready());
1428 } 1428 }
OLDNEW
« no previous file with comments | « chrome/browser/extensions/extension_host.cc ('k') | chrome/browser/extensions/extensions_ui.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698