C69 #642

bridiver · 2018-08-09T23:02:56Z

No description provided.

https://crrev.com/24feb5bfd766

…back http://crbug.com/807724

https://crrev.com/b6742ce84583 https://crrev.com/3db8a18444e5

https://crrev.com/385b5a513973

https://crrev.com/318708b8a2e5

https://crrev.com/71dba344e19b

https://crrev.com/5f734e3228cd

https://crrev.com/55944cb91cc2

…ating() https://crbug.com/714018

https://crrev.com/dbbd2c2c8630

https://crrev.com/014dec79d523

…losure https://crrev.com/52b9372f38d4

https://crrev.com/c35bf396d6b5 https://crrev.com/ce74950625ba https://crrev.com/e75e328a0268

…util https://crrev.com/2013965fad65 https://crrev.com/737e22b4d5c2

https://crrev.com/eea1b7a4b7e6

https://crrev.com/84c29aa09728

Also updated the code to reflect upstream Chromium changes. https://crrev.com/5f173cb15baf

https://crrev.com/b458879b37cb

https://crrev.com/019a2dc45de0

https://crrev.com/29ed9114f31f

https://crrev.com/5162124a8b00

https://crrev.com/0d67fc7cdf40

https://crrev.com/694a6830e3c5

https://crrev.com/4dfb375cee68 https://crrev.com/b63ff0d946a5 https://crrev.com/b67c0a8eaa84 https://crrev.com/f34bba7c3875

https://crrev.com/d5ecd79b3f85

bridiver · 2018-08-30T19:51:33Z

patches/master_patch.patch

-   // Stop intercepting keyboard events.
-   virtual void UnlockKeyboard() = 0;
-
-+#if !defined(MUON_CHROMIUM_BUILD)


I'm not sure I understand what's going on with this change

I've taken care of it in 18115e7

@bridiver

e6bf1d7#diff-fb76da0c9cc2defc5c9fa23dd04d5327 https://github.com/brave/muon/pull/642/files#diff-85ff0d997b1a2f089546f7b2823b0ad7L2067 Auditor: @bridiver

@bridiver

https://chromium.googlesource.com/chromium/src/+/3e0fccc527819cb8e09682bc280fb35d4b3fafdd Auditor: @bridiver

@bridiver

450fd34#diff-868023d36d447ab80b508fc483ead7ef makes it only gc once Auditor: @bridiver, @ltilve

@bridiver

cdb0144#diff-df5c427fd693f9ebb2e5952cf0e9c8c7 Auditor: @bridiver

Auditor: @briiver

@bridiver

Auditors: @bridiver, @bbondy

Add API method: electron.app.getCountryName

including "//content/public/browser" deps in order to access blink mojo bindings

With https://chromium.googlesource.com/chromium/src/+/f623bafe9c5b8cbd1d63c4d7c9b69de172552df5, the values were changed from an exclusion (ex: which values to NOT put in `OverlayUserPrefStore`) to an inclusion (ex: which values to PERSIST, even in an incognito profile) Fixes brave/browser-laptop#15191

Update incognito session initialization

@bsclifton

auditors @bsclifton @darkdh

@bsclifton

auditors @bsclifton @darkdh

@darkdh

auditors @darkdh @diracdeltas

… other Allow* methods in the observer

Issue: 15232 AllowScript should use atom::ContentSettingsManager like other Allow* methods in the observer

bridiver added the PR/work-in-progress ⚒ label Aug 9, 2018

bridiver assigned hferreiro and ltilve Aug 9, 2018

hferreiro and others added 27 commits August 15, 2018 13:16

Update patches to 69.0.3472.3

0da8407

Rename //net:net_browser_services to //net/dns:mojo_service

9828071

https://crrev.com/24feb5bfd766

TODO: Use net::CompletionOnceCallback in place of net::CompletionCall…

c5188ec

…back http://crbug.com/807724

Remove PlatformNotificationService::CheckPermissionOn{UI/IO}Thread

44d39c6

https://crrev.com/b6742ce84583 https://crrev.com/3db8a18444e5

DeleteCookiePredicate was moved to network::SessionCleanupCookieStore

f58494a

https://crrev.com/385b5a513973

Add resource_coordinator:mojo_bindings dependency

b09cd6b

Change CreateWebUIControllerForURL() to return a unique_ptr

5b4904a

https://crrev.com/318708b8a2e5

REVIEW: DispatchNonPerisistentClickEvent() parameters

a89c3ba

https://crrev.com/71dba344e19b

CreateLoginDelegate() parameters

2d47fed

https://crrev.com/5f734e3228cd

Rename WebFrameClient to WebLocalFrameClient

cd3ab8d

https://crrev.com/55944cb91cc2

TODO: Migrate from base::Bind() to base::BindOnce() or base::BindRepe…

bd53398

…ating() https://crbug.com/714018

Add missing GetAppGuid() to configurator

a357eae

https://crrev.com/dbbd2c2c8630

Rename WebContentsDelegate methods

2db4659

https://crrev.com/014dec79d523

Refactor RenderWidgetHost RendererUnresponsive() into generic base::C…

10f27db

…losure https://crrev.com/52b9372f38d4

Moved and renamed some Speech Recognition mojom files to Blink

80cadb7

https://crrev.com/c35bf396d6b5 https://crrev.com/ce74950625ba https://crrev.com/e75e328a0268

Remove WebContents::WasRecentlyAudible and fix use in extensions_tab_…

fcd1b85

…util https://crrev.com/2013965fad65 https://crrev.com/737e22b4d5c2

Move ExtensionService to extensions:: namespace

aa08fe0

https://crrev.com/eea1b7a4b7e6

UserDataKey() return type changed to const void*

b3c4113

https://crrev.com/84c29aa09728

component_updater::ComponentUpdateServiceFactory() changed params

7bbf5bc

Also updated the code to reflect upstream Chromium changes. https://crrev.com/5f173cb15baf

NetworkService: Add ClearHostCache() method

cdb0144

https://crrev.com/b458879b37cb

Capture Gaia password hashes used in content area

582d2a0

https://crrev.com/019a2dc45de0

Add missing resource_coordinator:site_characteristics_proto dep

9f616ce

https://crrev.com/29ed9114f31f

Update GetCurrentPrintPreviewStatus params

5d0fe83

https://crrev.com/5162124a8b00

TODO: Fixed reporting for iframe ugprade-insecure-requests upgrades

c45beac

https://crrev.com/0d67fc7cdf40

Remove unused RunMicrotasksObserver

a2554de

https://crrev.com/694a6830e3c5

C69 updates to io_thread.cc

b656050

https://crrev.com/4dfb375cee68 https://crrev.com/b63ff0d946a5 https://crrev.com/b67c0a8eaa84 https://crrev.com/f34bba7c3875

Rename PasswordSyncState to SyncState

e01e46c

https://crrev.com/d5ecd79b3f85

bridiver commented Aug 30, 2018

View reviewed changes

darkdh and others added 6 commits August 30, 2018 13:15

Fix broken geo location caused by

26d811a

e6bf1d7#diff-fb76da0c9cc2defc5c9fa23dd04d5327 https://github.com/brave/muon/pull/642/files#diff-85ff0d997b1a2f089546f7b2823b0ad7L2067 Auditor: @bridiver

Force cocoa to send non-cmd key event to menu

0396bc3

https://chromium.googlesource.com/chromium/src/+/3e0fccc527819cb8e09682bc280fb35d4b3fafdd Auditor: @bridiver

base::allocator::ReleaseFreeMemory needs to be called every minutes

7050961

450fd34#diff-868023d36d447ab80b508fc483ead7ef makes it only gc once Auditor: @bridiver, @ltilve

ClearHostCacheCallback should be in IsOffTheRecord block

232da3e

cdb0144#diff-df5c427fd693f9ebb2e5952cf0e9c8c7 Auditor: @bridiver

Turn IsSyncAccountEmail to default false

1247459

Auditor: @briiver

Bump v8.1.1+69.0.3497.57

5cfec9e

darkdh force-pushed the C69 branch from e6a322e to 5cfec9e Compare August 31, 2018 17:57

darkdh and others added 4 commits September 2, 2018 00:19

Emit WebGestureEvent attributes for scroll-touch-edge event

b9cec60

Auditors: @bridiver, @bbondy

Bump v8.1.2+69.0.3497.57

973fc2e

Merge pull request #647 from brave/get-country-name

86b6c02

Add API method: electron.app.getCountryName

Bump v8.1.3+69.0.3497.81

811ba76

bridiver force-pushed the C69 branch from 919fdc3 to 811ba76 Compare September 4, 2018 18:50

bridiver removed the PR/work-in-progress ⚒ label Sep 4, 2018

bridiver and others added 9 commits September 4, 2018 12:52

fix build error from last chromium .81 bump

c8ec091

Bump v8.1.4+69.0.3497.92

a4b05ce

Fix Windows blob.mojom.h not found error by

d9620b3

including "//content/public/browser" deps in order to access blink mojo bindings

Merge pull request #648 from brave/fix-15191

a712524

Update incognito session initialization

Bump v8.1.5+69.0.3497.92

2f20a5c

disable mac views browser

6c2f235

auditors @bsclifton @darkdh

fix broken build from disabling mac_views_browser

ff90369

auditors @bsclifton @darkdh

Bump v8.1.6+69.0.3497.100

e755b44

darkdh force-pushed the C69 branch from 695da63 to e755b44 Compare September 17, 2018 21:30

bsclifton mentioned this pull request Sep 18, 2018

Question: what's the future of Muon with the upcoming move to Chromium frontend? #554

Open

bridiver and others added 5 commits September 27, 2018 14:41

remove unused and dangerous apis

487188c

auditors @darkdh @diracdeltas

Bump v8.1.7+69.0.3497.100

6ad02d0

Issue: 15232 AllowScript should use atom::ContentSettingsManager like…

c18663a

… other Allow* methods in the observer

Merge pull request #651 from brave/noscript_issue

b67288b

Issue: 15232 AllowScript should use atom::ContentSettingsManager like other Allow* methods in the observer

Bump v8.1.8+69.0.3497.100

e9c4a09

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

C69 #642

C69 #642

bridiver commented Aug 9, 2018

bridiver Aug 30, 2018

darkdh Aug 30, 2018

C69 #642

Are you sure you want to change the base?

C69 #642

Conversation

bridiver commented Aug 9, 2018

bridiver Aug 30, 2018

Choose a reason for hiding this comment

darkdh Aug 30, 2018

Choose a reason for hiding this comment