Bug 151421

Summary: Modern IDB: Populate indexes created in object stores that already have records
Product: WebKit Reporter: Brady Eidson <beidson>
Component: WebCore Misc.Assignee: Brady Eidson <beidson>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, alecflett, buildbot, commit-queue, jsbell, rniwa
Priority: P2    
Version: Safari 9   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 149117    
Attachments:
Description Flags
Patch v1
buildbot: commit-queue-
Archive of layout-test-results from ews103 for mac-yosemite
none
Archive of layout-test-results from ews116 for mac-yosemite
none
Patch again none

Description Brady Eidson 2015-11-18 16:10:11 PST
Modern IDB: Populate indexes created in object stores that already have records
Comment 1 Brady Eidson 2015-11-18 17:21:17 PST
Created attachment 265815 [details]
Patch v1
Comment 2 Build Bot 2015-11-18 18:05:46 PST
Comment on attachment 265815 [details]
Patch v1

Attachment 265815 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/447777

New failing tests:
storage/indexeddb/modern/index-4.html
Comment 3 Build Bot 2015-11-18 18:05:50 PST
Created attachment 265818 [details]
Archive of layout-test-results from ews103 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews103  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 4 Build Bot 2015-11-18 18:17:45 PST
Comment on attachment 265815 [details]
Patch v1

Attachment 265815 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/447796

New failing tests:
storage/indexeddb/modern/index-4.html
Comment 5 Build Bot 2015-11-18 18:17:50 PST
Created attachment 265820 [details]
Archive of layout-test-results from ews116 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews116  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 6 Brady Eidson 2015-11-18 18:44:28 PST
Layout test failures are because the index cursors patch hasn't landed yet.  Doh!
Comment 7 Brady Eidson 2015-11-18 19:53:25 PST
(In reply to comment #6)
> Layout test failures are because the index cursors patch hasn't landed yet. 
> Doh!

The bug/patch in question is https://bugs.webkit.org/show_bug.cgi?id=151278

The failing layout tests here relies on 151278 landing.

Will resubmit after CQ finally lands it.
Comment 8 Brady Eidson 2015-11-18 20:50:11 PST
Created attachment 265843 [details]
Patch again
Comment 9 WebKit Commit Bot 2015-11-19 00:20:01 PST
Comment on attachment 265843 [details]
Patch again

Clearing flags on attachment: 265843

Committed r192618: <http://trac.webkit.org/changeset/192618>
Comment 10 WebKit Commit Bot 2015-11-19 00:20:05 PST
All reviewed patches have been landed.  Closing bug.