Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 0 additions & 6 deletions src/jsc/bindings/bindings.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -3438,12 +3438,6 @@ JSC::EncodedJSValue ZigString__toRangeErrorInstance(const ZigString* str, JSC::J
return JSC::JSValue::encode(Zig::getRangeErrorInstance(str, globalObject));
}

static JSC::EncodedJSValue resolverFunctionCallback(JSC::JSGlobalObject* globalObject,
JSC::CallFrame* callFrame)
{
return JSC::JSValue::encode(JSC::jsUndefined());
}

JSC::JSPromise*
JSC__JSModuleLoader__loadAndEvaluateModule(JSC::JSGlobalObject* globalObject,
const BunString* arg1)
Expand Down
12 changes: 0 additions & 12 deletions src/jsc/bindings/webcore/HTTPParsers.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -169,18 +169,6 @@ bool isValidAcceptHeaderValue(const StringView& value)
return true;
}

static bool containsCORSUnsafeRequestHeaderBytes(const String& value)
{
for (unsigned i = 0; i < value.length(); ++i) {
char16_t c = value[i];
// https://fetch.spec.whatwg.org/#cors-unsafe-request-header-byte
if ((c < 0x20 && c != '\t') || (c == '"' || c == '(' || c == ')' || c == ':' || c == '<' || c == '>' || c == '?' || c == '@' || c == '[' || c == '\\' || c == ']' || c == 0x7B || c == '{' || c == '}' || c == 0x7F))
return true;
}

return false;
}

// See RFC 7231, Section 5.3.5 and 3.1.3.2.
// https://fetch.spec.whatwg.org/#cors-safelisted-request-header
bool isValidLanguageHeaderValue(const StringView& value)
Expand Down
123 changes: 0 additions & 123 deletions src/jsc/bindings/webcore/HeaderFieldTokenizer.cpp

This file was deleted.

64 changes: 0 additions & 64 deletions src/jsc/bindings/webcore/HeaderFieldTokenizer.h

This file was deleted.

1 change: 0 additions & 1 deletion src/jsc/bindings/webcore/ResourceTiming.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,7 @@
// #include "OriginAccessPatterns.h"
#include "PerformanceServerTiming.h"
// #include "SecurityOrigin.h"
#include "ResourceLoadTiming.h"

Check warning on line 34 in src/jsc/bindings/webcore/ResourceTiming.cpp

View check run for this annotation

Claude / Claude Code Review

ServerTiming::setParameter and single-arg ctor left dead after removing their only caller

Deleting `ServerTimingParser::parseServerTiming()` also stranded its callee-side helpers: `ServerTiming::setParameter()` (ServerTiming.cpp:31-47 / ServerTiming.h:44) and the single-arg `ServerTiming(String&&)` ctor (ServerTiming.h:40, 49-52) now have zero call sites. Since this PR already followed the transitive-deletion rule for `HeaderFieldTokenizer`, these should go too — and be named in the description alongside the other removals.
Comment thread
robobun marked this conversation as resolved.
#include "ServerTimingParser.h"
#include <wtf/CrossThreadCopier.h>

#include "NetworkLoadMetrics.h"
Expand Down
77 changes: 0 additions & 77 deletions src/jsc/bindings/webcore/ServerTimingParser.cpp

This file was deleted.

40 changes: 0 additions & 40 deletions src/jsc/bindings/webcore/ServerTimingParser.h

This file was deleted.

Loading