Bug 174783

Summary: WebDriver: rename m_browsingContext as m_currentBrowsingContext in Session
Product: WebKit Reporter: Carlos Garcia Campos <cgarcia>
Component: WebKit Misc.Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: bburg, buildbot, commit-queue
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Archive of layout-test-results from ews122 for ios-simulator-wk2 none

Carlos Garcia Campos
Reported 2017-07-24 03:41:41 PDT
We have m_toplevelBrowsingContext and m_browsingContext, which is confusing. m_browsingContext is actually the current browsing context, and the spec also uses current, so better use m_currentBrowsingContext.
Attachments
Patch (15.82 KB, patch)
2017-07-24 03:43 PDT, Carlos Garcia Campos
no flags
Archive of layout-test-results from ews122 for ios-simulator-wk2 (982.35 KB, application/zip)
2017-07-24 07:15 PDT, Build Bot
no flags
Carlos Garcia Campos
Comment 1 2017-07-24 03:43:44 PDT
Build Bot
Comment 2 2017-07-24 07:15:29 PDT
Comment on attachment 316279 [details] Patch Attachment 316279 [details] did not pass ios-sim-ews (ios-simulator-wk2): Output: http://webkit-queues.webkit.org/results/4178546 New failing tests: imported/w3c/IndexedDB-private-browsing/idbfactory_open12.html
Build Bot
Comment 3 2017-07-24 07:15:30 PDT
Created attachment 316289 [details] Archive of layout-test-results from ews122 for ios-simulator-wk2 The attached test failures were seen while running run-webkit-tests on the ios-sim-ews. Bot: ews122 Port: ios-simulator-wk2 Platform: Mac OS X 10.12.5
Blaze Burg
Comment 4 2017-07-24 11:48:15 PDT
Comment on attachment 316279 [details] Patch rs=me
WebKit Commit Bot
Comment 5 2017-07-24 12:16:59 PDT
Comment on attachment 316279 [details] Patch Clearing flags on attachment: 316279 Committed r219833: <http://trac.webkit.org/changeset/219833>
WebKit Commit Bot
Comment 6 2017-07-24 12:17:00 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.