Bug 223810 - Remove ENABLE_INDEXED_DATABASE & ENABLE_INDEXED_DATABASE_IN_WORKERS, it seems like it is on for all ports
Summary: Remove ENABLE_INDEXED_DATABASE & ENABLE_INDEXED_DATABASE_IN_WORKERS, it seems...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Sam Weinig
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-03-26 12:12 PDT by Sam Weinig
Modified: 2021-03-28 12:15 PDT (History)
28 users (show)

See Also:


Attachments
Patch (182.09 KB, patch)
2021-03-26 17:13 PDT, Sam Weinig
no flags Details | Formatted Diff | Diff
Patch (182.28 KB, patch)
2021-03-26 17:20 PDT, Sam Weinig
ews-feeder: commit-queue-
Details | Formatted Diff | Diff
Patch (184.64 KB, patch)
2021-03-26 19:27 PDT, Sam Weinig
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Sam Weinig 2021-03-26 12:12:11 PDT
As far as I can tell, ENABLE_INDEXED_DATABASE_IN_WORKERS is enabled for all ports. It is on for the cocoa ports via PlatformEnableCocoa.h. It is on for the non-cocoa ports via WebKitFeatures.cmake.
Comment 1 Sam Weinig 2021-03-26 14:36:23 PDT
Seems like this is also true for ENABLE_INDEXED_DATABASE. Let's do both.
Comment 2 Sam Weinig 2021-03-26 17:13:51 PDT Comment hidden (obsolete)
Comment 3 Sam Weinig 2021-03-26 17:20:31 PDT Comment hidden (obsolete)
Comment 4 EWS Watchlist 2021-03-26 17:21:53 PDT Comment hidden (obsolete)
Comment 5 Sam Weinig 2021-03-26 19:27:24 PDT
Created attachment 424432 [details]
Patch
Comment 6 Sam Weinig 2021-03-27 11:37:04 PDT
Any inspector folk know what's going on with the Windows's build?

C:\cygwin\home\buildbot\worker\Windows-EWS\build\Source\WebCore\inspector\agents\InspectorIndexedDBAgent.h(46,84): error C2039: 'IndexedDBBackendDispatcherHandler': is not a member of 'Inspector' (compiling source file C:\cygwin\home\buildbot\worker\Windows-EWS\build\WebKitBuild\Release\DerivedSources\WebCore\unified-sources\UnifiedSource-84c9f43f-2.cpp) [C:\cygwin\home\buildbot\worker\Windows-EWS\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]
C:\cygwin\home\buildbot\worker\Windows-EWS\build\Source\WebCore\inspector\agents\InspectorIndexedDBAgent.h(38): message : see declaration of 'Inspector' (compiling source file C:\cygwin\home\buildbot\worker\Windows-EWS\build\WebKitBuild\Release\DerivedSources\WebCore\unified-sources\UnifiedSource-84c9f43f-2.cpp) [C:\cygwin\home\buildbot\worker\Windows-EWS\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]
C:\cygwin\home\buildbot\worker\Windows-EWS\build\Source\WebCore\inspector\agents\InspectorIndexedDBAgent.h(46,118): error C2504: 'IndexedDBBackendDispatcherHandler': base class undefined (compiling source file C:\cygwin\home\buildbot\worker\Windows-EWS\build\WebKitBuild\Release\DerivedSources\WebCore\unified-sources\UnifiedSource-84c9f43f-2.cpp) [C:\cygwin\home\buildbot\worker\Windows-EWS\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]
C:\cygwin\home\buildbot\worker\Windows-EWS\build\Source\WebCore\inspector\agents\InspectorIndexedDBAgent.h(60,65): error C2065: 'RequestDatabaseNamesCallback': undeclared identifier (compiling source file C:\cygwin\home\buildbot\worker\Windows-EWS\build\WebKitBuild\Release\DerivedSources\WebCore\unified-sources\UnifiedSource-84c9f43f-2.cpp) [C:\cygwin\home\buildbot\worker\Windows-EWS\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]
C:\cygwin\home\buildbot\worker\Windows-EWS\build\Source\WebCore\inspector\agents\InspectorIndexedDBAgent.h(60,61): error C2923: 'WTF::Ref': 'RequestDatabaseNamesCallback' is not a valid template type argument for parameter 'T' (compiling source file C:\cygwin\home\buildbot\worker\Windows-EWS\build\WebKitBuild\Release\DerivedSources\WebCore\unified-sources\UnifiedSource-84c9f43f-2.cpp) [C:\cygwin\home\buildbot\worker\Windows-EWS\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]
Comment 7 Sam Weinig 2021-03-27 18:55:23 PDT
(In reply to Sam Weinig from comment #6)
> Any inspector folk know what's going on with the Windows's build?
> 
> C:\cygwin\home\buildbot\worker\Windows-
> EWS\build\Source\WebCore\inspector\agents\InspectorIndexedDBAgent.h(46,84):
> error C2039: 'IndexedDBBackendDispatcherHandler': is not a member of
> 'Inspector' (compiling source file
> C:\cygwin\home\buildbot\worker\Windows-
> EWS\build\WebKitBuild\Release\DerivedSources\WebCore\unified-
> sources\UnifiedSource-84c9f43f-2.cpp)
> [C:\cygwin\home\buildbot\worker\Windows-
> EWS\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]
> C:\cygwin\home\buildbot\worker\Windows-
> EWS\build\Source\WebCore\inspector\agents\InspectorIndexedDBAgent.h(38):
> message : see declaration of 'Inspector' (compiling source file
> C:\cygwin\home\buildbot\worker\Windows-
> EWS\build\WebKitBuild\Release\DerivedSources\WebCore\unified-
> sources\UnifiedSource-84c9f43f-2.cpp)
> [C:\cygwin\home\buildbot\worker\Windows-
> EWS\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]
> C:\cygwin\home\buildbot\worker\Windows-
> EWS\build\Source\WebCore\inspector\agents\InspectorIndexedDBAgent.h(46,118):
> error C2504: 'IndexedDBBackendDispatcherHandler': base class undefined
> (compiling source file
> C:\cygwin\home\buildbot\worker\Windows-
> EWS\build\WebKitBuild\Release\DerivedSources\WebCore\unified-
> sources\UnifiedSource-84c9f43f-2.cpp)
> [C:\cygwin\home\buildbot\worker\Windows-
> EWS\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]
> C:\cygwin\home\buildbot\worker\Windows-
> EWS\build\Source\WebCore\inspector\agents\InspectorIndexedDBAgent.h(60,65):
> error C2065: 'RequestDatabaseNamesCallback': undeclared identifier
> (compiling source file
> C:\cygwin\home\buildbot\worker\Windows-
> EWS\build\WebKitBuild\Release\DerivedSources\WebCore\unified-
> sources\UnifiedSource-84c9f43f-2.cpp)
> [C:\cygwin\home\buildbot\worker\Windows-
> EWS\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]
> C:\cygwin\home\buildbot\worker\Windows-
> EWS\build\Source\WebCore\inspector\agents\InspectorIndexedDBAgent.h(60,61):
> error C2923: 'WTF::Ref': 'RequestDatabaseNamesCallback' is not a valid
> template type argument for parameter 'T' (compiling source file
> C:\cygwin\home\buildbot\worker\Windows-
> EWS\build\WebKitBuild\Release\DerivedSources\WebCore\unified-
> sources\UnifiedSource-84c9f43f-2.cpp)
> [C:\cygwin\home\buildbot\worker\Windows-
> EWS\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]

I have been told this is just to be expected from the Windows port. So please ignore.
Comment 8 EWS 2021-03-28 12:14:27 PDT
Committed r275151: <https://commits.webkit.org/r275151>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 424432 [details].
Comment 9 Radar WebKit Bug Importer 2021-03-28 12:15:16 PDT
<rdar://problem/75934037>