Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Eth-wallet #14727

Closed
wants to merge 573 commits into from
Closed

Eth-wallet #14727

wants to merge 573 commits into from

Conversation

Slava
Copy link
Contributor

@Slava Slava commented Jul 13, 2018

Rebase of #13177 + updates

Submitter Checklist:

  • Submitted a ticket for my issue if one did not already exist.
  • Used Github auto-closing keywords in the commit message.
  • Added/updated tests for this change (for new code or code which already has tests).
  • Ran git rebase -i to squash commits (if needed).
  • Tagged reviewers and labelled the pull request as needed.
  • Request a security/privacy review as needed. (Ask a Brave employee to help if you cannot access this document.)

Test Plan:

Reviewer Checklist:

  • Request a security/privacy review as needed if one was not already requested.

Tests

  • Adequate test coverage exists to prevent regressions
  • Tests should be independent and work correctly when run individually or as a suite ref
  • New files have MPL2 license header

ryanml and others added 30 commits April 17, 2018 12:25
…ge when a wallet recovery is successful, and when it is cancelled or closed out of.
* Release notes for 0.22.22

* Update CHANGELOG.md
Opens add funds overlay when clicking on it in the notification
…ffer-window

Fix automated tests post-0.22.x
Resolves brave#13751

Auditors:

Test Plan:
Send hostname option to webtorrent to mitigate DNS rebinding
…cused

Fix brave#13859
Fix brave#13860
Fix brave#13865

Makes sure tabs are created in a shown window, or creates a new window, that the window is focused, and if the shown window is minimized, then that window is restored.
Fixes for the following scenarios...:
    - There are no windows open
    - There are only minimized windows open
...For the following actions:
    - Brave is default browser and link clicked in another application
    - macOS dock icon is clicked
    - 'Brave' menu actions such as 'New Tab' or 'About Brave'

Test plan:
Browser must be set as default. If testing from source, the best way is to create a build `CHANNEL=dev npm run build-package`, run that, and set it as default.
    - Click URL link in external application
      - Ensure that tab opens in an active window, and the window is focused with the following scenarios:
        - 1 window open
        - 2+ windows open
        - 0 windows open
        - 1 minimized window open
        - 1 minimized window open and 1 non-minimized window open
        - Application exited
      - macOS: click dock icon, ensure window is focused or created:
        - 1 window open
        - 2+ windows open
        - 0 windows open
        - 1 minimized window open
        - 1 minimized window open and 1 non-minimized

Even though the reported issues were macOS only, this test plan should be carried out on other platforms to ensure no regressions.
…-window

Ensure tabs are not created in hidden window and correct window is focused
Resolves brave#13721

Auditors:

Test Plan:
Fixes wallet recovery dialogue re-appearing on relaunch after a successful recovery.
Resolves brave#13227

Auditors:

Test Plan:
Handle auto-update case properly for quarantine
WIP -- unit test

Moved url stripping logic to urlutil. Implemented unit tests

Removed logging debug statements

shortenUrlToDomainTLD now more utility than ledger based.

corrected lint

corrected types and checks

merge

lint

wip -- refactor shortenUrlToDomainTLD

wip -- fixup unit tests

wip -- moved logic to own function to get publisher key

Removed file (wrong branch)

Updated url shortener and moved getter to separate getter.

linting

Corrected URL parsing logic and tests

Update verified publisher method and updated tests

corrected parsing for publisher keys

check for null parsedUrl.hostname

Renamed function to reflect params
Some verified publishers no longer remove green tick at sublevels - Fixes issue brave#10484
* muon unit test runner cleanup
* muon unit test runner able to do async
* muon unit test runner add before/affter

Auditors:

Test Plan:
Don't break up as URL for autocomplete if location bar text isn't a URL
…testing of STAGING ad-block definitions

Fixes brave#13909

Auditors: @bbondy
NejcZdovc and others added 28 commits May 21, 2018 17:05
Fixes disabled torrent viewer torrent download
Resolves brave#14141

Auditors:

Test Plan:
Add a session migration for uphold fingerprinting
Resolves brave#10767

Auditors:

Test Plan:
Referrer page loads quickly and code was previously calling createTabRequested well before window was ready.
This would cause the partner page to load first (before about:welcome).

Fixes brave#14220

Auditors: @NejcZdovc
Resolves brave#14190

Auditors:

Test Plan:
Only call createTabRequested if window is ready
fix brave#14231 (workaround)

Test Plan:
1. go to a secure URL ending in a hash such as https://github.com/brave/browser-laptop/wiki/Fingerprinting-Protection-Mode#technical-details
2. toggle shields or click 'reload' in the shields menu
3. the page should appear as still secure instead of changing to insecure
set reloadMatchingUrl to true for URLs ending in a hash
…e4-Fix

Fixed a mistake in the 0.22.x Release 4 notes
Resolves brave#14188

Auditors:

Test Plan:
Resolves brave#14229

Auditors:

Test Plan:
Resolves brave#8537

Auditors:

Test Plan:
…lude

Removes site settings for delete sites
@Slava
Copy link
Contributor Author

Slava commented Jul 13, 2018

Reopened in #14734

@Slava Slava closed this Jul 13, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.