Extensions-related fixups for scoped_refptr conversion operator removal.

extensions/common/permissions/permissions_data.h

Index: extensions/common/permissions/permissions_data.h
diff --git a/extensions/common/permissions/permissions_data.h b/extensions/common/permissions/permissions_data.h
index dd74d28dfa74c35e2dc0ee51747642124cd429ae..ba2f3c3aa54078bbca0e1843e7fc2ec9942e3e70 100644
--- a/extensions/common/permissions/permissions_data.h
+++ b/extensions/common/permissions/permissions_data.h
@@ -200,13 +200,14 @@ class PermissionsData {
   // page itself.
   bool CanCaptureVisiblePage(int tab_id, std::string* error) const;
 
-  scoped_refptr<const PermissionSet> active_permissions() const {
+  const scoped_refptr<const PermissionSet>& active_permissions() const {
+    // TODO(dcheng): What is the point of this lock?
     base::AutoLock auto_lock(runtime_lock_);
     return active_permissions_unsafe_;
   }
 
-  scoped_refptr<const PermissionSet> withheld_permissions() const {
-    base::AutoLock auto_lock(runtime_lock_);
+  const scoped_refptr<const PermissionSet>& withheld_permissions() const {
+    // TODO(dcheng): What is the point of this lock?
     return withheld_permissions_unsafe_;
   }