GitLab

4327: Flow.ifChanged(): Added variant that takes a mapping function. r=csadilek a=pocmo

This pattern emerged while migrating components. Often I want listen to value changes, but if the value changes then I need more than only the changed value.

One example is the downloads feature. I want to listen to download changes. But if that changes then I need the whole `SessionState` object.

Previously:

```kotlin
scope = store.flowScoped { flow ->
    flow.mapNotNull { state -> state.findCustomTabOrSelectedTab(customTabId) }
           .map { it.content.download }
           .ifChanged()
          .collect { download ->
              // Okay, the DownloadState object now. But I also want the enclosing SessionState.
          }
}
```

Now:

```kotlin
scope = store.flowScoped { flow ->
    flow.mapNotNull { state -> state.findCustomTabOrSelectedTab(customTabId) }
        .ifChanged { it.content.download }
        .collect { state ->
            // Yay, I have the whole session state and can get
            // to the download by using state.content.download
        }
}
```
Co-authored-by: Sebastian Kaspari <s.kaspari@gmail.com>

4331: Make BrowserAction names consistent r=pocmo a=csadilek

Very minor refactoring, but since I just noticed. We could also decide to remove `Action` from all names, as they are usually prefixed with their type e.g. `TabListAction.AddTab`. Either way, I think we should keep them consistent.
Co-authored-by: Christian Sadilek <christian.sadilek@gmail.com>

4324: Closes #4320: Add api to indicate if strict social tracking protection r=csadilek a=Amejia481
Co-authored-by: Arturo Mejia <arturomejiamarmol@gmail.com>

should be activated.

4316: Closes #4165: Add more logging for push r=csadilek a=jonalmeida

This adds different levels of logging to the AutoPushFeature since we're not always able to tell from a release build in an app that a consumer has initialized their `PushService` successfully; that's a bit of a blackbox for us when it comes to testing in release.
Co-authored-by: Jonathan Almeida <jalmeida@mozilla.com>

4106: fixes #1444 exposed title in HitResult and set image title as fragment title  r=jonalmeida a=mcarare

related Fenix issues : mozilla-mobile/fenix#4684 mozilla-mobile/fenix#580
added title in class Image & added title param in all usages. made it optional, with default null value, to keep WebView-based implementation of EngineView unchanged (SystemEngineView.kt)
added title param in handleLongClick from 'element.title', as it is exposed by element information from gecko
in ContextMenuCandidate added case for IMAGE. returning src string when title is null, blank or starting with http
Co-authored-by: mihai-adrian <mihai.carare.dev@gmail.com>
Co-authored-by: Mihai Adrian <48995920+mcarare@users.noreply.github.com>

4321:  Public Suffix List update (20190905-141413) r=pocmo a=MickeyMoz
Co-authored-by: MickeyMoz <sebastian@mozilla.com>

4322:  GeckoView update (beta) (20190905-141154) r=pocmo a=MickeyMoz



4323:  GeckoView update (nightly) (20190905-141410) r=pocmo a=MickeyMoz
Co-authored-by: MickeyMoz <sebastian@mozilla.com>

4319: Split changelog. r=jonalmeida a=pocmo

🤷

‍♂ 
Co-authored-by: Sebastian Kaspari <s.kaspari@gmail.com>

4318:  Docs update (20190905-121255) [ci skip] r=pocmo a=MickeyMoz
Co-authored-by: MickeyMoz <sebastian@mozilla.com>

4315: For #4284, 4285: HitResult and Download consumables for browser-state r=pocmo a=csadilek

@pocmo As discussed, there are 3 parts/commits here:

- Support for adding listeners to Consumables
- Add HitResult to browser-state (and sync from browser-session)
- Sync downloads between browser-session and browser-state
Co-authored-by: Christian Sadilek <christian.sadilek@gmail.com>

Co-authored-by: Sebastian Kaspari <s.kaspari@gmail.com>

4314: Bug 1578859 - Update lib-crash README.md [ci skip] r=jonalmeida a=travis79

Removing a note from the README.md. With the addition of better pre-init handling, this no longer requires Glean to be initialized first.
Co-authored-by: Travis Long <tlong@mozilla.com>

Removing a note from the README.md since the addition of better pre-init handling, this no longer requires Glean to be initialized first.

4232: Initialize sentry client lazily #3445 r=rocketsroger a=KKS161994
Co-authored-by: Kartikey Srivastava <kartikeysrivastava@kartikeys-macbook-pro.local>

4221: FxA webchannel support and account manager integration r=csadilek a=grigoryk

## Part 1:
Preparatory work for allowing users to choose what to sync. We are about to
tie SyncConfig with FxA's "capabilities" webchannel API, which takes a set of
"engines" as strings. To simplify our integration work, this patch replaces an
untyped set of "stores" with a typed set of "engines" at the API level.

## Part 2:
(should have been split up further for easier reviewing, but things got a bit tangled)
This patch does a few things:
- adds a webextension for communicating with fxa web content
- provides a backend for said webextension, which understands webchannel communication protocol
and knows how to relay messages to the account manager, and query its state
- expands account manager a bit to support the webchannel authentication protocol
- adds a (rather detailed!) AuthType parameter to AccountObserver:onAuthenticated, instead of the very vague 'newAccount' flag
  - Closes #4238 
Co-authored-by: Grisha Kruglov <gkruglov@mozilla.com>

This patch does a few things:
- adds a webextension for communicating with fxa web content
- provides a backend for said webextension, which understands webchannel communication protocol
and knows how to relay messages to the account manager, and query its state
- expands account manager a bit to support the webchannel authentication protocol
- adds a (rather detailed!) AuthType parameter to AccountObserver:onAuthenticated, instead of the very vague 'newAccount' flag
Co-authored-by: Arturo Mejia <arturomejiamarmol@gmail.com>
Co-authored-by: vladikoff <vlad.filippov@gmail.com>

Preparatory work for allowing users to choose what to sync. We are about to
tie SyncConfig with FxA's "capabilities" webchannel API, which takes a set of
"engines" as strings. To simplify our integration work, this patch replaces an
untyped set of "stores" with a typed set of "engines" at the API level.

4308: Fat geckoview everywhere! No more architecture-specific build flavors. r=Amejia481 a=pocmo
Co-authored-by: Sebastian Kaspari <s.kaspari@gmail.com>

4310:  Public Suffix List update (20190904-141244) r=pocmo a=MickeyMoz
Co-authored-by: MickeyMoz <sebastian@mozilla.com>

4303: Allow products to specify a custom path for Glean-generated docs r=Dexterp37 a=Dexterp37

This is required by Fenix, since their docs are in `$rootDir/docs` not in `$projectDir/docs`
Co-authored-by: Alessio Placitelli <alessio.placitelli@gmail.com>

4263: Closes #4311: Remove Top and Bottom Padding Causing Issues with Text Alignment and Clipping r=jonalmeida a=rocketsroger

…Alignment and Clipping
Co-authored-by: Roger Yang <royang@mozilla.com>