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

Unified Diff: chrome/browser/extensions/api/socket/socket_api.cc

Issue 14651017: Move RuntimeData and related permissions out of Extension class (Closed) Base URL: http://git.chromium.org/chromium/src.git@dc_unref_permissions
Patch Set: Latest master Created 7 years, 7 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/api/socket/socket_api.cc
diff --git a/chrome/browser/extensions/api/socket/socket_api.cc b/chrome/browser/extensions/api/socket/socket_api.cc
index 5b75fc9d1a48d9ad464fba784f46b365be7cf514..49699d26b932aafe61c2bec7ef0062f6bbd045b7 100644
--- a/chrome/browser/extensions/api/socket/socket_api.cc
+++ b/chrome/browser/extensions/api/socket/socket_api.cc
@@ -8,8 +8,6 @@
#include "base/bind.h"
#include "base/hash_tables.h"
-#include "chrome/common/extensions/extension.h"
-#include "chrome/common/extensions/permissions/socket_permission.h"
#include "chrome/browser/browser_process.h"
#include "chrome/browser/extensions/api/dns/host_resolver_wrapper.h"
#include "chrome/browser/extensions/api/socket/socket.h"
@@ -17,6 +15,9 @@
#include "chrome/browser/extensions/api/socket/udp_socket.h"
#include "chrome/browser/extensions/extension_system.h"
#include "chrome/browser/io_thread.h"
+#include "chrome/common/extensions/extension.h"
+#include "chrome/common/extensions/permissions/permissions_data.h"
+#include "chrome/common/extensions/permissions/socket_permission.h"
#include "net/base/host_port_pair.h"
#include "net/base/io_buffer.h"
#include "net/base/ip_endpoint.h"
@@ -205,8 +206,8 @@ void SocketConnectFunction::AsyncWorkStart() {
}
SocketPermission::CheckParam param(operation_type, hostname_, port_);
- if (!GetExtension()->CheckAPIPermissionWithParam(APIPermission::kSocket,
- &param)) {
+ if (!PermissionsData::CheckAPIPermissionWithParam(
+ GetExtension(), APIPermission::kSocket, &param)) {
error_ = kPermissionError;
SetResult(Value::CreateIntegerValue(-1));
AsyncWorkCompleted();
@@ -269,8 +270,10 @@ void SocketBindFunction::Work() {
if (socket->GetSocketType() == Socket::TYPE_UDP) {
SocketPermission::CheckParam param(
SocketPermissionRequest::UDP_BIND, address_, port_);
- if (!GetExtension()->CheckAPIPermissionWithParam(APIPermission::kSocket,
- &param)) {
+ if (!PermissionsData::CheckAPIPermissionWithParam(
+ GetExtension(),
+ APIPermission::kSocket,
+ &param)) {
error_ = kPermissionError;
SetResult(Value::CreateIntegerValue(result));
return;
@@ -304,8 +307,10 @@ void SocketListenFunction::Work() {
if (socket) {
SocketPermission::CheckParam param(
SocketPermissionRequest::TCP_LISTEN, params_->address, params_->port);
- if (!GetExtension()->CheckAPIPermissionWithParam(APIPermission::kSocket,
- &param)) {
+ if (!PermissionsData::CheckAPIPermissionWithParam(
+ GetExtension(),
+ APIPermission::kSocket,
+ &param)) {
error_ = kPermissionError;
SetResult(Value::CreateIntegerValue(result));
return;
@@ -515,8 +520,10 @@ void SocketSendToFunction::AsyncWorkStart() {
if (socket_->GetSocketType() == Socket::TYPE_UDP) {
SocketPermission::CheckParam param(SocketPermissionRequest::UDP_SEND_TO,
hostname_, port_);
- if (!GetExtension()->CheckAPIPermissionWithParam(APIPermission::kSocket,
- &param)) {
+ if (!PermissionsData::CheckAPIPermissionWithParam(
+ GetExtension(),
+ APIPermission::kSocket,
+ &param)) {
error_ = kPermissionError;
SetResult(Value::CreateIntegerValue(-1));
AsyncWorkCompleted();
@@ -721,8 +728,8 @@ void SocketJoinGroupFunction::Work() {
kWildcardAddress,
kWildcardPort);
- if (!GetExtension()->CheckAPIPermissionWithParam(APIPermission::kSocket,
- &param)) {
+ if (!PermissionsData::CheckAPIPermissionWithParam(
+ GetExtension(), APIPermission::kSocket, &param)) {
error_ = kPermissionError;
SetResult(Value::CreateIntegerValue(result));
return;
@@ -767,8 +774,9 @@ void SocketLeaveGroupFunction::Work() {
SocketPermissionRequest::UDP_MULTICAST_MEMBERSHIP,
kWildcardAddress,
kWildcardPort);
- if (!GetExtension()->CheckAPIPermissionWithParam(APIPermission::kSocket,
- &param)) {
+ if (!PermissionsData::CheckAPIPermissionWithParam(GetExtension(),
+ APIPermission::kSocket,
+ &param)) {
error_ = kPermissionError;
SetResult(Value::CreateIntegerValue(result));
return;
@@ -876,8 +884,10 @@ void SocketGetJoinedGroupsFunction::Work() {
SocketPermissionRequest::UDP_MULTICAST_MEMBERSHIP,
kWildcardAddress,
kWildcardPort);
- if (!GetExtension()->CheckAPIPermissionWithParam(APIPermission::kSocket,
- &param)) {
+ if (!PermissionsData::CheckAPIPermissionWithParam(
+ GetExtension(),
+ APIPermission::kSocket,
+ &param)) {
error_ = kPermissionError;
SetResult(Value::CreateIntegerValue(result));
return;
« no previous file with comments | « chrome/browser/extensions/api/media_galleries/media_galleries_api.cc ('k') | chrome/browser/extensions/api/tabs/tabs_api.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698