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

Unified Diff: runtime/bin/filter.cc

Issue 130513003: [ZLIB] Add support for windowBits, memLevel, raw (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: integrate feedback Created 6 years, 10 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
« no previous file with comments | « runtime/bin/filter.h ('k') | runtime/bin/filter_patch.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/bin/filter.cc
diff --git a/runtime/bin/filter.cc b/runtime/bin/filter.cc
index 34c0ee2f1cb62248e7f5a10cb08b207314827151..1e7edf3ab8b86ff584f1e4e7f56ecf5ebb4db7e6 100644
--- a/runtime/bin/filter.cc
+++ b/runtime/bin/filter.cc
@@ -8,18 +8,15 @@
#include "include/dart_api.h"
-
namespace dart {
namespace bin {
-const int kZlibFlagMemUsage = 8;
-const int kZLibFlagWindowBits = 15;
const int kZLibFlagUseGZipHeader = 16;
const int kZLibFlagAcceptAnyHeader = 32;
static const int kFilterPointerNativeField = 0;
-Filter* GetFilter(Dart_Handle filter_obj) {
+static Filter* GetFilter(Dart_Handle filter_obj) {
Filter* filter;
Dart_Handle result = Filter::GetFilterPointerNativeField(filter_obj, &filter);
if (Dart_IsError(result)) {
@@ -31,15 +28,62 @@ Filter* GetFilter(Dart_Handle filter_obj) {
return filter;
}
-void EndFilter(Dart_Handle filter_obj, Filter* filter) {
+static void EndFilter(Dart_Handle filter_obj, Filter* filter) {
Filter::SetFilterPointerNativeField(filter_obj, NULL);
delete filter;
}
+static uint8_t* copyDictionary(Dart_Handle dictionary_obj) {
+ uint8_t* src = NULL;
+ intptr_t size;
+ Dart_TypedData_Type type;
+
+ if (Dart_IsError(Dart_ListLength(dictionary_obj, &size))) {
+ Dart_ThrowException(DartUtils::NewInternalError(
+ "Failed to get the zlib dictionary length"));
+ }
+
+ uint8_t* dictionary = new uint8_t[size];
+
+ if (dictionary == NULL) {
+ Dart_ThrowException(DartUtils::NewInternalError(
+ "Failed to allocate buffer for the zlib dictionary"));
+ }
+
+ Dart_Handle result = Dart_TypedDataAcquireData(
+ dictionary_obj, &type, reinterpret_cast<void**>(&src), &size);
+ if (!Dart_IsError(result)) {
+ memmove(dictionary, src, size);
+ Dart_TypedDataReleaseData(dictionary_obj);
+ } else {
+ if (Dart_IsError(Dart_ListGetAsBytes(dictionary_obj, 0, dictionary,
+ size))) {
+ Dart_ThrowException(DartUtils::NewInternalError(
+ "Failed to get the zlib dictionary"));
+ }
+ }
+
+ return dictionary;
+}
+
void FUNCTION_NAME(Filter_CreateZLibInflate)(Dart_NativeArguments args) {
Dart_Handle filter_obj = Dart_GetNativeArgument(args, 0);
- Filter* filter = new ZLibInflateFilter();
- if (filter == NULL || !filter->Init()) {
+ Dart_Handle window_bits_obj = Dart_GetNativeArgument(args, 1);
+ int64_t window_bits = DartUtils::GetIntegerValue(window_bits_obj);
+ Dart_Handle dict_obj = Dart_GetNativeArgument(args, 2);
+ uint8_t* dictionary = NULL;
+ if (!Dart_IsNull(dict_obj)) {
+ dictionary = copyDictionary(dict_obj);
+ }
+ Dart_Handle raw_obj = Dart_GetNativeArgument(args, 3);
+ bool raw;
+ if (Dart_IsError(Dart_BooleanValue(raw_obj, &raw))) {
+ Dart_ThrowException(DartUtils::NewInternalError(
+ "Failed to get 'raw' parameter"));
+ }
+ Filter* filter = new ZLibInflateFilter(static_cast<int32_t>(window_bits),
+ dictionary, raw);
+ if (!filter->Init()) {
delete filter;
Dart_ThrowException(DartUtils::NewInternalError(
"Failed to create ZLibInflateFilter"));
@@ -54,25 +98,34 @@ void FUNCTION_NAME(Filter_CreateZLibInflate)(Dart_NativeArguments args) {
void FUNCTION_NAME(Filter_CreateZLibDeflate)(Dart_NativeArguments args) {
Dart_Handle filter_obj = Dart_GetNativeArgument(args, 0);
Dart_Handle gzip_obj = Dart_GetNativeArgument(args, 1);
+ bool gzip = DartUtils::GetBooleanValue(gzip_obj);
Dart_Handle level_obj = Dart_GetNativeArgument(args, 2);
- bool gzip;
- if (Dart_IsError(Dart_BooleanValue(gzip_obj, &gzip))) {
- Dart_ThrowException(DartUtils::NewInternalError(
- "Failed to get 'gzip' parameter"));
+ int64_t level = DartUtils::GetInt64ValueCheckRange(level_obj, kMinInt32,
+ kMaxInt32);
+ Dart_Handle window_bits_obj = Dart_GetNativeArgument(args, 3);
+ int64_t window_bits = DartUtils::GetIntegerValue(window_bits_obj);
+ Dart_Handle mLevel_obj = Dart_GetNativeArgument(args, 4);
+ int64_t mem_level = DartUtils::GetIntegerValue(mLevel_obj);
+ Dart_Handle strategy_obj = Dart_GetNativeArgument(args, 5);
+ int64_t strategy = DartUtils::GetIntegerValue(strategy_obj);
+ Dart_Handle dict_obj = Dart_GetNativeArgument(args, 6);
+ uint8_t* dictionary = NULL;
+ if (!Dart_IsNull(dict_obj)) {
+ dictionary = copyDictionary(dict_obj);
}
- int64_t level = 0;
- Dart_Handle result = Dart_IntegerToInt64(level_obj, &level);
- if (Dart_IsError(result) || (level < kMinInt32) || (level > kMaxInt32)) {
- Dart_ThrowException(DartUtils::NewInternalError(
- "Failed to get 'level' parameter"));
- }
- Filter* filter = new ZLibDeflateFilter(gzip, static_cast<int32_t>(level));
- if (filter == NULL || !filter->Init()) {
+ Dart_Handle raw_obj = Dart_GetNativeArgument(args, 7);
+ bool raw = DartUtils::GetBooleanValue(raw_obj);
+ Filter* filter = new ZLibDeflateFilter(gzip, static_cast<int32_t>(level),
+ static_cast<int32_t>(window_bits),
+ static_cast<int32_t>(mem_level),
+ static_cast<int32_t>(strategy),
+ dictionary, raw);
+ if (!filter->Init()) {
delete filter;
Dart_ThrowException(DartUtils::NewInternalError(
"Failed to create ZLibDeflateFilter"));
}
- result = Filter::SetFilterPointerNativeField(filter_obj, filter);
+ Dart_Handle result = Filter::SetFilterPointerNativeField(filter_obj, filter);
if (Dart_IsError(result)) {
delete filter;
Dart_PropagateError(result);
@@ -185,27 +238,38 @@ Dart_Handle Filter::GetFilterPointerNativeField(Dart_Handle filter,
ZLibDeflateFilter::~ZLibDeflateFilter() {
+ delete[] dictionary_;
delete[] current_buffer_;
if (initialized()) deflateEnd(&stream_);
}
bool ZLibDeflateFilter::Init() {
+ int window_bits = window_bits_;
+ if (raw_) {
+ window_bits = -window_bits;
+ } else if (gzip_) {
+ window_bits += kZLibFlagUseGZipHeader;
+ }
+ stream_.next_in = Z_NULL;
stream_.zalloc = Z_NULL;
stream_.zfree = Z_NULL;
stream_.opaque = Z_NULL;
- int result = deflateInit2(
- &stream_,
- level_,
- Z_DEFLATED,
- kZLibFlagWindowBits | (gzip_ ? kZLibFlagUseGZipHeader : 0),
- kZlibFlagMemUsage,
- Z_DEFAULT_STRATEGY);
- if (result == Z_OK) {
- set_initialized(true);
- return true;
+ int result = deflateInit2(&stream_, level_, Z_DEFLATED, window_bits,
+ mem_level_, strategy_);
+ if (result != Z_OK) {
+ return false;
+ }
+ if (dictionary_ != NULL && !gzip_ && !raw_) {
+ result = deflateSetDictionary(&stream_, dictionary_, sizeof(dictionary_));
+ delete[] dictionary_;
+ dictionary_ = NULL;
+ if (result != Z_OK) {
+ return false;
+ }
}
- return false;
+ set_initialized(true);
+ return true;
}
@@ -222,6 +286,7 @@ intptr_t ZLibDeflateFilter::Processed(uint8_t* buffer,
bool end) {
stream_.avail_out = length;
stream_.next_out = buffer;
+ bool error = false;
switch (deflate(&stream_,
end ? Z_FINISH : flush ? Z_SYNC_FLUSH : Z_NO_FLUSH)) {
case Z_STREAM_END:
@@ -229,42 +294,46 @@ intptr_t ZLibDeflateFilter::Processed(uint8_t* buffer,
case Z_OK: {
intptr_t processed = length - stream_.avail_out;
if (processed == 0) {
- delete[] current_buffer_;
- current_buffer_ = NULL;
- return 0;
- } else {
- // We processed data, should be called again.
- return processed;
+ break;
}
+ return processed;
}
default:
case Z_STREAM_ERROR:
- // An error occoured.
- delete[] current_buffer_;
- current_buffer_ = NULL;
- return -1;
+ error = true;
}
+
+ delete[] current_buffer_;
+ current_buffer_ = NULL;
+ // Either 0 Byte processed or error
+ return error ? -1 : 0;
}
ZLibInflateFilter::~ZLibInflateFilter() {
+ delete[] dictionary_;
delete[] current_buffer_;
if (initialized()) inflateEnd(&stream_);
}
bool ZLibInflateFilter::Init() {
+ int window_bits = raw_ ?
+ -window_bits_ :
+ window_bits_ | kZLibFlagAcceptAnyHeader;
+
+ stream_.next_in = Z_NULL;
+ stream_.avail_in = 0;
stream_.zalloc = Z_NULL;
stream_.zfree = Z_NULL;
stream_.opaque = Z_NULL;
- int result = inflateInit2(&stream_,
- kZLibFlagWindowBits | kZLibFlagAcceptAnyHeader);
- if (result == Z_OK) {
- set_initialized(true);
- return true;
+ int result = inflateInit2(&stream_, window_bits);
+ if (result != Z_OK) {
+ return false;
}
- return false;
+ set_initialized(true);
+ return true;
}
@@ -282,6 +351,7 @@ intptr_t ZLibInflateFilter::Processed(uint8_t* buffer,
bool end) {
stream_.avail_out = length;
stream_.next_out = buffer;
+ bool error = false;
switch (inflate(&stream_,
end ? Z_FINISH : flush ? Z_SYNC_FLUSH : Z_NO_FLUSH)) {
case Z_STREAM_END:
@@ -289,25 +359,38 @@ intptr_t ZLibInflateFilter::Processed(uint8_t* buffer,
case Z_OK: {
intptr_t processed = length - stream_.avail_out;
if (processed == 0) {
- delete[] current_buffer_;
- current_buffer_ = NULL;
- return 0;
- } else {
- // We processed data, should be called again.
- return processed;
+ break;
}
+ return processed;
}
+ case Z_NEED_DICT:
+ if (dictionary_ == NULL) {
+ error = true;
+ } else {
+ int result = inflateSetDictionary(&stream_, dictionary_,
+ sizeof(dictionary_));
+ delete[] dictionary_;
+ dictionary_ = NULL;
+ error = result != Z_OK;
+ }
+ if (error) {
+ break;
+ } else {
+ return Processed(buffer, length, flush, end);
+ }
+
default:
case Z_MEM_ERROR:
- case Z_NEED_DICT:
case Z_DATA_ERROR:
case Z_STREAM_ERROR:
- // An error occoured.
- delete[] current_buffer_;
- current_buffer_ = NULL;
- return -1;
+ error = true;
}
+
+ delete[] current_buffer_;
+ current_buffer_ = NULL;
+ // Either 0 Byte processed or error
+ return error ? -1 : 0;
}
} // namespace bin
« no previous file with comments | « runtime/bin/filter.h ('k') | runtime/bin/filter_patch.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698