Skip to content
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

Update Android test sheet #123

Merged
merged 5 commits into from Jun 10, 2019
Merged
Changes from 1 commit
Commits
File filter...
Filter file types
Jump to…
Jump to file
Failed to load files.

Always

Just for now

Update test sections

  • Loading branch information
srirambv committed Jun 7, 2019
commit 1a60f51ee898787daad10ceb642c2118e712592b
@@ -6,7 +6,9 @@

## Visual look

- [ ] Make sure there is no words of Chromium after any merge
- [ ] Make sure there after any merge
This conversation was marked as resolved by srirambv

This comment has been minimized.

Copy link
@kjozwiak

kjozwiak Jun 7, 2019

Member

nit: Make sure that after very merge:

- [ ] No Chromium words appear on normal or private tabs
This conversation was marked as resolved by srirambv

This comment has been minimized.

Copy link
@kjozwiak

kjozwiak Jun 7, 2019

Member

nit: Both Chromium/Chrome words are not being used in normal or private tabs

- [ ] No Chromium icons are shown in normal or private tabs
This conversation was marked as resolved by srirambv

This comment has been minimized.

Copy link
@kjozwiak

kjozwiak Jun 7, 2019

Member

nit: No Chromium/Chrome icons are being used in normal or private tabs


## Data
Pre-Requisite: Put previous build shortcut on home screen. Also have several sites 'Added to home screen' (from 3 dots menu) and then upgrade to new build
@@ -52,9 +54,10 @@ Pre-Requisite: Put previous build shortcut on home screen. Also have several sit

- [ ] Verify https://www.twitch.tv/adobe sub-page loads a video and you can play it

## Settings
## Settings and Bottom bar

- [ ] Verify changing default settings are retained and doesn't cause the browser to crash
- [ ] Verify bottom bar buttons(Home/Bookmark/Search/Tabs) work as expected
This conversation was marked as resolved by srirambv

This comment has been minimized.

Copy link
@kjozwiak

kjozwiak Jun 7, 2019

Member

nit: space between "buttons(Home/Bookmark/Search/Tabs)"


## Downloads

ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.