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

Side by Side Diff: chrome/common/extensions/permissions/permissions_data.cc

Issue 18854021: Making the extension permissions dialog scrollable, when needed (adding expandable sections for thi… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Polishing Created 7 years, 5 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) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 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/common/extensions/permissions/permissions_data.h" 5 #include "chrome/common/extensions/permissions/permissions_data.h"
6 6
7 #include "base/command_line.h" 7 #include "base/command_line.h"
8 #include "base/memory/scoped_ptr.h" 8 #include "base/memory/scoped_ptr.h"
9 #include "base/strings/string16.h" 9 #include "base/strings/string16.h"
10 #include "base/strings/string_number_conversions.h" 10 #include "base/strings/string_number_conversions.h"
(...skipping 403 matching lines...) Expand 10 before | Expand all | Expand 10 after
414 } 414 }
415 415
416 // static 416 // static
417 bool PermissionsData::HasEffectiveAccessToAllHosts(const Extension* extension) { 417 bool PermissionsData::HasEffectiveAccessToAllHosts(const Extension* extension) {
418 base::AutoLock auto_lock(extension->permissions_data()->runtime_lock_); 418 base::AutoLock auto_lock(extension->permissions_data()->runtime_lock_);
419 return GetActivePermissions(extension)->HasEffectiveAccessToAllHosts(); 419 return GetActivePermissions(extension)->HasEffectiveAccessToAllHosts();
420 } 420 }
421 421
422 // static 422 // static
423 PermissionMessages PermissionsData::GetPermissionMessages( 423 PermissionMessages PermissionsData::GetPermissionMessages(
424 const Extension* extension) { 424 const Extension* extension, bool details_only) {
425 base::AutoLock auto_lock(extension->permissions_data()->runtime_lock_); 425 base::AutoLock auto_lock(extension->permissions_data()->runtime_lock_);
426 if (ShouldSkipPermissionWarnings(extension)) { 426 if (ShouldSkipPermissionWarnings(extension)) {
427 return PermissionMessages(); 427 return PermissionMessages();
428 } else { 428 } else {
429 return GetActivePermissions(extension)->GetPermissionMessages( 429 return GetActivePermissions(extension)->GetPermissionMessages(
430 extension->GetType()); 430 extension->GetType(), details_only);
431 } 431 }
432 } 432 }
433 433
434 // static 434 // static
435 std::vector<string16> PermissionsData::GetPermissionMessageStrings( 435 std::vector<string16> PermissionsData::GetPermissionMessageStrings(
436 const Extension* extension) { 436 const Extension* extension, bool details_only) {
437 base::AutoLock auto_lock(extension->permissions_data()->runtime_lock_); 437 base::AutoLock auto_lock(extension->permissions_data()->runtime_lock_);
438 if (ShouldSkipPermissionWarnings(extension)) { 438 if (ShouldSkipPermissionWarnings(extension)) {
439 return std::vector<string16>(); 439 return std::vector<string16>();
440 } else { 440 } else {
441 return GetActivePermissions(extension)->GetWarningMessages( 441 return GetActivePermissions(extension)->GetWarningMessages(
442 extension->GetType()); 442 extension->GetType(), details_only);
443 } 443 }
444 } 444 }
445 445
446 // static 446 // static
447 bool PermissionsData::CanExecuteScriptOnPage(const Extension* extension, 447 bool PermissionsData::CanExecuteScriptOnPage(const Extension* extension,
448 const GURL& document_url, 448 const GURL& document_url,
449 const GURL& top_frame_url, 449 const GURL& top_frame_url,
450 int tab_id, 450 int tab_id,
451 const UserScript* script, 451 const UserScript* script,
452 int process_id, 452 int process_id,
(...skipping 168 matching lines...) Expand 10 before | Expand all | Expand 10 after
621 optional_permission_set_ = new PermissionSet( 621 optional_permission_set_ = new PermissionSet(
622 initial_optional_permissions_->api_permissions, 622 initial_optional_permissions_->api_permissions,
623 initial_optional_permissions_->host_permissions, 623 initial_optional_permissions_->host_permissions,
624 URLPatternSet()); 624 URLPatternSet());
625 625
626 initial_required_permissions_.reset(); 626 initial_required_permissions_.reset();
627 initial_optional_permissions_.reset(); 627 initial_optional_permissions_.reset();
628 } 628 }
629 629
630 } // namespace extensions 630 } // namespace extensions
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698