Skip to content

/ qa-resources

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add enabling extensions test case #42

Merged
merged 1 commit into from Jan 16, 2018
Merged

Add enabling extensions test case #42

merged 1 commit into from Jan 16, 2018

Conversation

@kjozwiak
Copy link
Member

kjozwiak commented Jan 4, 2018

Fixes #41
@kjozwiak
Add enabling extensions test case
6eb7f71 
Fix #41
@kjozwiak kjozwiak added the enhancement label Jan 4, 2018
@kjozwiak kjozwiak self-assigned this Jan 4, 2018
@kjozwiak kjozwiak requested review from srirambv and LaurenWags Jan 4, 2018
@LaurenWags
Copy link
Contributor

LaurenWags commented Jan 4, 2018

What should we do for the pre-enabled extensions? Turn those off and see if they cause unresponsiveness as well?
@srirambv
Copy link
Contributor

srirambv commented Jan 5, 2018

@LaurenWags Its only required if the browser shows the unresponsiveness on a clean profile which has those default ones enabled.
@srirambv
srirambv approved these changes Jan 5, 2018
View changes
Copy link
Contributor

srirambv left a comment

++
@kjozwiak kjozwiak merged commit 940e44a into master Jan 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@srirambv srirambv

@LaurenWags LaurenWags

Assignees

@kjozwiak kjozwiak

Labels
enhancement
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
@kjozwiak @LaurenWags @srirambv
You can’t perform that action at this time.