<html>
    <head>
      <base href="https://bugs.webkit.org/" />
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [GTK] Enable IndexedDB in workers"
   href="https://bugs.webkit.org/show_bug.cgi?id=158475#c2">Comment # 2</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [GTK] Enable IndexedDB in workers"
   href="https://bugs.webkit.org/show_bug.cgi?id=158475">bug 158475</a>
              from <span class="vcard"><a class="email" href="mailto:mcatanzaro&#64;igalia.com" title="Michael Catanzaro &lt;mcatanzaro&#64;igalia.com&gt;"> <span class="fn">Michael Catanzaro</span></a>
</span></b>
        <pre>Comment on <span class=""><a href="attachment.cgi?id=280693&amp;action=diff" name="attach_280693" title="Patch">attachment 280693</a> <a href="attachment.cgi?id=280693&amp;action=edit" title="Patch">[details]</a></span>
Patch

View in context: <a href="https://bugs.webkit.org/attachment.cgi?id=280693&amp;action=review">https://bugs.webkit.org/attachment.cgi?id=280693&amp;action=review</a>

<span class="quote">&gt; Source/cmake/OptionsGTK.cmake:164
&gt; +WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_INDEXED_DATABASE_IN_WORKERS PRIVATE ON)</span >

I'll just note that this setting should not have been listed here previously, as ENABLE_INDEXED_DATABASE_IN_WORKERS is OFF by default, and this is a list of settings overrides.

I'm suspicious that it really makes sense for IndexedDB to be OFF by default for most ports, though, since most ports surely want it. In such cases, it'd be better to fix the default and disable it just for those ports that can't support it yet.

I'm also suspicious as to why ENABLE_INDEXED_DATABASE_IN_WORKERS is a separate option... why would any port want IndexedDB, but not in workers?</pre>
        </div>
      </p>
      <hr>
      <span>You are receiving this mail because:</span>
      
      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>