diff options
Diffstat (limited to 'extensions')
-rw-r--r-- | extensions/browser/sandboxed_unpacker.cc | 4 | ||||
-rw-r--r-- | extensions/browser/script_executor.cc | 2 | ||||
-rw-r--r-- | extensions/browser/user_script_loader.cc | 2 | ||||
-rw-r--r-- | extensions/common/api/sockets/sockets_manifest_permission_unittest.cc | 4 | ||||
-rw-r--r-- | extensions/common/extension_messages.cc | 41 | ||||
-rw-r--r-- | extensions/common/permissions/api_permission.cc | 2 | ||||
-rw-r--r-- | extensions/common/permissions/api_permission_set_unittest.cc | 2 | ||||
-rw-r--r-- | extensions/common/permissions/manifest_permission.cc | 3 | ||||
-rw-r--r-- | extensions/common/permissions/settings_override_permission.cc | 2 | ||||
-rw-r--r-- | extensions/common/permissions/socket_permission_unittest.cc | 4 | ||||
-rw-r--r-- | extensions/common/user_script.h | 3 | ||||
-rw-r--r-- | extensions/common/user_script_unittest.cc | 4 | ||||
-rw-r--r-- | extensions/renderer/user_script_set.cc | 13 |
13 files changed, 49 insertions, 37 deletions
diff --git a/extensions/browser/sandboxed_unpacker.cc b/extensions/browser/sandboxed_unpacker.cc index 38a7cd9a..8b17160 100644 --- a/extensions/browser/sandboxed_unpacker.cc +++ b/extensions/browser/sandboxed_unpacker.cc @@ -190,7 +190,7 @@ bool ReadImagesFromFile(const base::FilePath& extension_path, return false; IPC::Message pickle(file_str.data(), file_str.size()); - PickleIterator iter(pickle); + base::PickleIterator iter(pickle); return IPC::ReadParam(&pickle, &iter, images); } @@ -206,7 +206,7 @@ bool ReadMessageCatalogsFromFile(const base::FilePath& extension_path, return false; IPC::Message pickle(file_str.data(), file_str.size()); - PickleIterator iter(pickle); + base::PickleIterator iter(pickle); return IPC::ReadParam(&pickle, &iter, catalogs); } diff --git a/extensions/browser/script_executor.cc b/extensions/browser/script_executor.cc index 0502d89..6161f3c 100644 --- a/extensions/browser/script_executor.cc +++ b/extensions/browser/script_executor.cc @@ -73,7 +73,7 @@ class Handler : public content::WebContentsObserver { return false; int message_request_id; - PickleIterator iter(message); + base::PickleIterator iter(message); CHECK(iter.ReadInt(&message_request_id)); if (message_request_id != main_request_id_ && diff --git a/extensions/browser/user_script_loader.cc b/extensions/browser/user_script_loader.cc index 4407093..047080a 100644 --- a/extensions/browser/user_script_loader.cc +++ b/extensions/browser/user_script_loader.cc @@ -275,7 +275,7 @@ void UserScriptLoader::StartLoad() { // static scoped_ptr<base::SharedMemory> UserScriptLoader::Serialize( const UserScriptList& scripts) { - Pickle pickle; + base::Pickle pickle; pickle.WriteSizeT(scripts.size()); for (const UserScript& script : scripts) { // TODO(aa): This can be replaced by sending content script metadata to diff --git a/extensions/common/api/sockets/sockets_manifest_permission_unittest.cc b/extensions/common/api/sockets/sockets_manifest_permission_unittest.cc index 419ddd7..4b2dbcf 100644 --- a/extensions/common/api/sockets/sockets_manifest_permission_unittest.cc +++ b/extensions/common/api/sockets/sockets_manifest_permission_unittest.cc @@ -194,7 +194,7 @@ TEST(SocketsManifestPermissionTest, Empty) { IPC::Message m; ipc_perm->Write(&m); - PickleIterator iter(m); + base::PickleIterator iter(m); EXPECT_TRUE(ipc_perm2->Read(&m, &iter)); AssertEmptyPermission(ipc_perm2.get()); } @@ -399,7 +399,7 @@ TEST(SocketsManifestPermissionTest, IPC) { IPC::Message m; ipc_perm->Write(&m); - PickleIterator iter(m); + base::PickleIterator iter(m); EXPECT_TRUE(ipc_perm2->Read(&m, &iter)); EXPECT_TRUE(permission->Equal(ipc_perm2.get())); } diff --git a/extensions/common/extension_messages.cc b/extensions/common/extension_messages.cc index e7fdb51..42e25f4 100644 --- a/extensions/common/extension_messages.cc +++ b/extensions/common/extension_messages.cc @@ -95,7 +95,9 @@ struct ParamTraits<Manifest::Location> { int val = static_cast<int>(p); WriteParam(m, val); } - static bool Read(const Message* m, PickleIterator* iter, param_type* p) { + static bool Read(const Message* m, + base::PickleIterator* iter, + param_type* p) { int val = 0; if (!ReadParam(m, iter, &val) || val < Manifest::INVALID_LOCATION || @@ -114,7 +116,8 @@ void ParamTraits<URLPattern>::Write(Message* m, const param_type& p) { WriteParam(m, p.GetAsString()); } -bool ParamTraits<URLPattern>::Read(const Message* m, PickleIterator* iter, +bool ParamTraits<URLPattern>::Read(const Message* m, + base::PickleIterator* iter, param_type* p) { int valid_schemes; std::string spec; @@ -141,8 +144,9 @@ void ParamTraits<URLPatternSet>::Write(Message* m, const param_type& p) { WriteParam(m, p.patterns()); } -bool ParamTraits<URLPatternSet>::Read(const Message* m, PickleIterator* iter, - param_type* p) { +bool ParamTraits<URLPatternSet>::Read(const Message* m, + base::PickleIterator* iter, + param_type* p) { std::set<URLPattern> patterns; if (!ReadParam(m, iter, &patterns)) return false; @@ -162,8 +166,9 @@ void ParamTraits<APIPermission::ID>::Write( WriteParam(m, static_cast<int>(p)); } -bool ParamTraits<APIPermission::ID>::Read( - const Message* m, PickleIterator* iter, param_type* p) { +bool ParamTraits<APIPermission::ID>::Read(const Message* m, + base::PickleIterator* iter, + param_type* p) { int api_id = -2; if (!ReadParam(m, iter, &api_id)) return false; @@ -188,8 +193,9 @@ void ParamTraits<APIPermissionSet>::Write( } } -bool ParamTraits<APIPermissionSet>::Read( - const Message* m, PickleIterator* iter, param_type* r) { +bool ParamTraits<APIPermissionSet>::Read(const Message* m, + base::PickleIterator* iter, + param_type* r) { size_t size; if (!ReadParam(m, iter, &size)) return false; @@ -225,8 +231,9 @@ void ParamTraits<ManifestPermissionSet>::Write( } } -bool ParamTraits<ManifestPermissionSet>::Read( - const Message* m, PickleIterator* iter, param_type* r) { +bool ParamTraits<ManifestPermissionSet>::Read(const Message* m, + base::PickleIterator* iter, + param_type* r) { size_t size; if (!ReadParam(m, iter, &size)) return false; @@ -255,8 +262,9 @@ void ParamTraits<HostID>::Write( WriteParam(m, p.id()); } -bool ParamTraits<HostID>::Read( - const Message* m, PickleIterator* iter, param_type* r) { +bool ParamTraits<HostID>::Read(const Message* m, + base::PickleIterator* iter, + param_type* r) { HostID::HostType type; std::string id; if (!ReadParam(m, iter, &type)) @@ -281,9 +289,10 @@ void ParamTraits<ExtensionMsg_PermissionSetStruct>::Write(Message* m, WriteParam(m, p.scriptable_hosts); } -bool ParamTraits<ExtensionMsg_PermissionSetStruct>::Read(const Message* m, - PickleIterator* iter, - param_type* p) { +bool ParamTraits<ExtensionMsg_PermissionSetStruct>::Read( + const Message* m, + base::PickleIterator* iter, + param_type* p) { return ReadParam(m, iter, &p->apis) && ReadParam(m, iter, &p->manifest_permissions) && ReadParam(m, iter, &p->explicit_hosts) && @@ -309,7 +318,7 @@ void ParamTraits<ExtensionMsg_Loaded_Params>::Write(Message* m, } bool ParamTraits<ExtensionMsg_Loaded_Params>::Read(const Message* m, - PickleIterator* iter, + base::PickleIterator* iter, param_type* p) { p->manifest.reset(new base::DictionaryValue()); return ReadParam(m, iter, &p->location) && ReadParam(m, iter, &p->path) && diff --git a/extensions/common/permissions/api_permission.cc b/extensions/common/permissions/api_permission.cc index 77afe9d..7d8f04e 100644 --- a/extensions/common/permissions/api_permission.cc +++ b/extensions/common/permissions/api_permission.cc @@ -84,7 +84,7 @@ class SimpleAPIPermission : public APIPermission { void Write(IPC::Message* m) const override {} - bool Read(const IPC::Message* m, PickleIterator* iter) override { + bool Read(const IPC::Message* m, base::PickleIterator* iter) override { return true; } diff --git a/extensions/common/permissions/api_permission_set_unittest.cc b/extensions/common/permissions/api_permission_set_unittest.cc index 0574250..78a0161 100644 --- a/extensions/common/permissions/api_permission_set_unittest.cc +++ b/extensions/common/permissions/api_permission_set_unittest.cc @@ -280,7 +280,7 @@ TEST(APIPermissionSetTest, IPC) { IPC::Message m; WriteParam(&m, apis); - PickleIterator iter(m); + base::PickleIterator iter(m); CHECK(ReadParam(&m, &iter, &expected_apis)); EXPECT_EQ(apis, expected_apis); } diff --git a/extensions/common/permissions/manifest_permission.cc b/extensions/common/permissions/manifest_permission.cc index 6be6b18..a3ba98e 100644 --- a/extensions/common/permissions/manifest_permission.cc +++ b/extensions/common/permissions/manifest_permission.cc @@ -34,7 +34,8 @@ void ManifestPermission::Write(IPC::Message* m) const { IPC::WriteParam(m, singleton); } -bool ManifestPermission::Read(const IPC::Message* m, PickleIterator* iter) { +bool ManifestPermission::Read(const IPC::Message* m, + base::PickleIterator* iter) { base::ListValue singleton; if (!IPC::ReadParam(m, iter, &singleton)) return false; diff --git a/extensions/common/permissions/settings_override_permission.cc b/extensions/common/permissions/settings_override_permission.cc index 4514a3f..c45678c 100644 --- a/extensions/common/permissions/settings_override_permission.cc +++ b/extensions/common/permissions/settings_override_permission.cc @@ -109,7 +109,7 @@ APIPermission* SettingsOverrideAPIPermission::Intersect( void SettingsOverrideAPIPermission::Write(IPC::Message* m) const {} bool SettingsOverrideAPIPermission::Read(const IPC::Message* m, - PickleIterator* iter) { + base::PickleIterator* iter) { return true; } diff --git a/extensions/common/permissions/socket_permission_unittest.cc b/extensions/common/permissions/socket_permission_unittest.cc index 0b1fae0..1ed207e 100644 --- a/extensions/common/permissions/socket_permission_unittest.cc +++ b/extensions/common/permissions/socket_permission_unittest.cc @@ -275,7 +275,7 @@ TEST(SocketPermissionTest, IPC) { permission_info->CreateAPIPermission()); permission1->Write(&m); - PickleIterator iter(m); + base::PickleIterator iter(m); permission2->Read(&m, &iter); EXPECT_TRUE(permission1->Equal(permission2.get())); @@ -298,7 +298,7 @@ TEST(SocketPermissionTest, IPC) { EXPECT_FALSE(permission1->Equal(permission2.get())); permission1->Write(&m); - PickleIterator iter(m); + base::PickleIterator iter(m); permission2->Read(&m, &iter); EXPECT_TRUE(permission1->Equal(permission2.get())); } diff --git a/extensions/common/user_script.h b/extensions/common/user_script.h index 070679e..93ca14f 100644 --- a/extensions/common/user_script.h +++ b/extensions/common/user_script.h @@ -235,7 +235,8 @@ class UserScript { void Unpickle(const base::Pickle& pickle, base::PickleIterator* iter); private: - // Pickle helper functions used to pickle the individual types of components. + // base::Pickle helper functions used to pickle the individual types of + // components. void PickleGlobs(base::Pickle* pickle, const std::vector<std::string>& globs) const; void PickleHostID(base::Pickle* pickle, const HostID& host_id) const; diff --git a/extensions/common/user_script_unittest.cc b/extensions/common/user_script_unittest.cc index 4a964f7..1dae0a2 100644 --- a/extensions/common/user_script_unittest.cc +++ b/extensions/common/user_script_unittest.cc @@ -196,10 +196,10 @@ TEST(ExtensionUserScriptTest, Pickle) { HostID id(HostID::EXTENSIONS, kExtensionId); script1.set_host_id(id); - Pickle pickle; + base::Pickle pickle; script1.Pickle(&pickle); - PickleIterator iter(pickle); + base::PickleIterator iter(pickle); UserScript script2; script2.Unpickle(pickle, &iter); diff --git a/extensions/renderer/user_script_set.cc b/extensions/renderer/user_script_set.cc index 01e8a5a..6af8627 100644 --- a/extensions/renderer/user_script_set.cc +++ b/extensions/renderer/user_script_set.cc @@ -98,21 +98,22 @@ bool UserScriptSet::UpdateUserScripts(base::SharedMemoryHandle shared_memory, return false; // First get the size of the memory block. - if (!shared_memory_->Map(sizeof(Pickle::Header))) + if (!shared_memory_->Map(sizeof(base::Pickle::Header))) return false; - Pickle::Header* pickle_header = - reinterpret_cast<Pickle::Header*>(shared_memory_->memory()); + base::Pickle::Header* pickle_header = + reinterpret_cast<base::Pickle::Header*>(shared_memory_->memory()); // Now map in the rest of the block. - int pickle_size = sizeof(Pickle::Header) + pickle_header->payload_size; + int pickle_size = sizeof(base::Pickle::Header) + pickle_header->payload_size; shared_memory_->Unmap(); if (!shared_memory_->Map(pickle_size)) return false; // Unpickle scripts. size_t num_scripts = 0; - Pickle pickle(reinterpret_cast<char*>(shared_memory_->memory()), pickle_size); - PickleIterator iter(pickle); + base::Pickle pickle(reinterpret_cast<char*>(shared_memory_->memory()), + pickle_size); + base::PickleIterator iter(pickle); CHECK(iter.ReadSizeT(&num_scripts)); scripts_.clear(); |