GitLab

Resetting destination store will clear all destinations.
If hasLoadedAllPrivetDestinations is not reset, privet destination
list will not get repopulated after reset_ (provided that the privet
destination loading had previously finished)

BUG=None
TEST=
On network with some privet printers setup:
1. Open print preview (Ctrl+P) and try to change selected destination
2. There should be some privet printers listed under "Local destinations"
3. When local destination loading is done (the spinner disappears)
add another account
4. Make sure "Local Destinations" list gets repopulated with
privet printers

Review URL: https://codereview.chromium.org/871853003

Cr-Commit-Position: refs/heads/master@{#314176}

This regressed by accident in crrev.com/6575fb41

BUG=365160

Review URL: https://codereview.chromium.org/888403002

Cr-Commit-Position: refs/heads/master@{#314175}

Revert of Add multiple device/os filtering and a config file. (patchset #9 id:160001 of https://codereview.chromium.org/879983002/)

Reason for revert:
Not working on bots.

Original issue's description:
> Add multiple device/os filtering and a config file.
>
> BUG=
>
> Committed: https://crrev.com/241e6870d1f5cb7e2f23a8b1edc33eb6db64ad24
> Cr-Commit-Position: refs/heads/master@{#314151}

TBR=jbudorick@chromium.org,rnephew@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=

Review URL: https://codereview.chromium.org/890143004

Cr-Commit-Position: refs/heads/master@{#314174}

That makes browser, foreground renderer, and GPU
have a similar priority. Otherwise the GPU process
might drop behind other (even unrelated bg) processes
and get killed too easily.

BUG=448549,453671
TBR=ppi@chromium.org

Review URL: https://codereview.chromium.org/889013002

Cr-Commit-Position: refs/heads/master@{#314173}

Always dispatch the extension activity logging IPCs to the UI thread
instead of checking first as the check will always be true.

Review URL: https://codereview.chromium.org/889033002

Cr-Commit-Position: refs/heads/master@{#314172}

Aligning tooltips to the left (or to the right according to the Locale
UI layout and the text directionality).

R=sky@chromium.org
BUG=334457
TEST=manual

Review URL: https://codereview.chromium.org/864223004

Cr-Commit-Position: refs/heads/master@{#314171}

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/7d3717c..bdb2d05

TBR=vsevik@chromium.org,japhet@chromium.org

Review URL: https://codereview.chromium.org/884843003

Cr-Commit-Position: refs/heads/master@{#314170}

As ChannelMojo is turned on by default, this CL do some clenaup
around the related switches:

 * It removes the about:flags entry. This no longer has to be
   user visible
 * It moves switches to mojo_channel_switches.h/cc as it does
   compute subtle decision. I make this public/ so that we can use
   it from ipc_fuzzer.
 * It adds --disable-channel-mojo for forcibly disable ChannelMojo usage.

We still leave --enable-renderer-channel-mojo because there is still a
small change that we turn ChannelMojo off again. Once ChannelMojo is
settled, we can safely remove it.

BUG=377980
R=nasko@chromium.org, sky@chromium.org

Review URL: https://codereview.chromium.org/886153003

Cr-Commit-Position: refs/heads/master@{#314169}

Also, take ownership for a related but un-owned histogram.

BUG=433130
TBR=sky@chromium.org

Review URL: https://codereview.chromium.org/862103002

Cr-Commit-Position: refs/heads/master@{#314168}

Remove and deprecate SEARCH_SUGGEST_ANSWER.  This type is no longer used in the UI code, since answers can be attached to suggestions of any type.

BUG=430513

Review URL: https://codereview.chromium.org/815273002

Cr-Commit-Position: refs/heads/master@{#314167}

NaCl now supports AVX2.

R=shyamsundarr@chromium.org
TBR=jmedley@chromium.org, shyamsundarr@chromium.org
BUG= https://code.google.com/p/nativeclient/issues/detail?id=3637

Review URL: https://codereview.chromium.org/888313002

Cr-Commit-Position: refs/heads/master@{#314166}

Now ChromotingHostContext disallows blocking operations on then
network thread. This will help to avoid issues such as the one
that was fixed in https://codereview.chromium.org/886913002/
(Network thread was used to load policies from disk).

Also made the following changes to avoid blocking operation on
the network thread:
 - Moved PolicyWatcher to the main thread in the Me2Me host.
   PolicyLoader uses blocking IO for initialization. Also this is
   more consistent with the It2Me host where PolicyWatcher also
   runs on the UI thread.
 - Moved NetworkChangeNotifier initialization out of
   HostSignalingManager. NetworkChangeNotifier creates a thread
   internally and therefore should be destroyed on the main thread.
 - Added a reference to URLRequestContextGetter in HostProcessMain
   to make sure it's destroyed on the main thread. This is necessary
   because BasicURLRequestContext owns threads and needs to be
   deleted on the main thread.

Review URL: https://codereview.chromium.org/891663005

Cr-Commit-Position: refs/heads/master@{#314165}

BUG=452257

Review URL: https://codereview.chromium.org/883693006

Cr-Commit-Position: refs/heads/master@{#314164}

This patch implements RevokePermission() method in the mojo
PermissionService. This method will probably be for internal
use only for now. The semantics of the method is that it
will reset the permission to its default value in chrome if
the permission is currently granted.

BUG=430238

Review URL: https://codereview.chromium.org/819463002

Cr-Commit-Position: refs/heads/master@{#314163}

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/3d91aad..ad49b3b

CQ_EXTRA_TRYBOTS=tryserver.blink:linux_blink_rel,linux_blink_dbg
TBR=stephana@google.com

Review URL: https://codereview.chromium.org/896443002

Cr-Commit-Position: refs/heads/master@{#314162}

Cr-Commit-Position: refs/heads/master@{#314161}

"by default" = whenever we build Chromoting.

This is a re-land of crrev.com/878793003, which failed on
the Official builds.

The build problem has been fixed in crrev.com/886543002.

BUG=

Review URL: https://codereview.chromium.org/875463004

Cr-Commit-Position: refs/heads/master@{#314160}

* Check size of scans.
* Create and cache selection scans (needed to check their size)....but only cache until the next selection change.
* Improve test coverage, covering both "while importing" and "insufficient size" cases.
* Improve tests to check for labels too (uncovering one bug in the test).
* Fix bug where we were no-longer updating import state/widget on selection changes.

BUG=420680
TEST=browser_test: FileManagerJsTest.*

Need a review from either mtomasz@ or hirono@, not both. First one to review, kindly remove the other.

Review URL: https://codereview.chromium.org/886093003

Cr-Commit-Position: refs/heads/master@{#314159}

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/c32ffd2..7d3717c

TBR=vsevik@chromium.org,japhet@chromium.org

Review URL: https://codereview.chromium.org/890143003

Cr-Commit-Position: refs/heads/master@{#314158}

BUG=452879

Review URL: https://codereview.chromium.org/891713002

Cr-Commit-Position: refs/heads/master@{#314157}

If the android build version pre-dates KitKat and the device
manufacturer is Samsung, skip the check for mmap exec support.
This avoids triggering a warning on these devices.
The version check is included because these devices do not show
the warning on later OS builds.

BUG=448084
TBR=rmcilroy@chromium.org

Review URL: https://codereview.chromium.org/869593002

Cr-Commit-Position: refs/heads/master@{#314156}

TBR=tbreisacher@chromium.org
BUG=none
TEST=http://build.chromium.org/p/chromium.fyi/builders/Closure%20Compilation%20Linux goes green

Review URL: https://codereview.chromium.org/895663002

Cr-Commit-Position: refs/heads/master@{#314155}

Currently, CredentialManagerDispatcher is a PasswordStoreObserver, and
handles responses from `request()` directly. We can't do that going
forward, as we also need to handle responses when dealing with (at
least) `notifySignedOut()`.

This patch splits the password store observer functionality out into
a new `PendingRequestTask` class, with the intent of introducing
`PendingSignedOutTask`, etc. classes in the future.

BUG=450581
R=vabr@chromium.org

Review URL: https://codereview.chromium.org/886793002

Cr-Commit-Position: refs/heads/master@{#314154}

'pngcrush -v' writes to stderr. Redirected stderr to stdout.

BUG=none
TEST=manually

Review URL: https://codereview.chromium.org/869433004

Cr-Commit-Position: refs/heads/master@{#314153}

Change HistoryService::ExpireLocalAndRemoteHistoryBetween() to
receive the history::WebHistoryService instance to use in order
to remove a dependency of HistoryService on Profile.

It is not possible to pass the WebHistoryService instance to the
HistoryService constructor and add a dependency between the two
factories due to http://crbug.com/171406 (as WebHistoryService
depends on a KeyedService calling Profile::GetRequestContext()).

BUG=453790
R=droger@chromium.org

Review URL: https://codereview.chromium.org/891763002

Cr-Commit-Position: refs/heads/master@{#314152}

BUG=

Review URL: https://codereview.chromium.org/879983002

Cr-Commit-Position: refs/heads/master@{#314151}

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/672177b..c32ffd2

TBR=vsevik@chromium.org,japhet@chromium.org

Review URL: https://codereview.chromium.org/891323003

Cr-Commit-Position: refs/heads/master@{#314150}

BUG=452710
TBR=glider@chromium.org
NOTRY=true

Review URL: https://codereview.chromium.org/894723002

Cr-Commit-Position: refs/heads/master@{#314149}

Now, NaCl supports to build x86-32 nonsfi binary even with target_arch=x64,
so this CL enables browser_tests for PPAPI.

BUG=358465
TEST=browser_tests --gtest_filter=*Transitional* locally. Run trybots.
CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:linux_rel_precise32,linux_arm_compile,linux_arm

Review URL: https://codereview.chromium.org/786273008

Cr-Commit-Position: refs/heads/master@{#314148}

At present, if a user changes zoom on OSX while the zoom bubble is
already being shown, the new zoom bubble (with the most up to date
information) is not shown. This CL changes this behaviour by allowing
a new zoom bubble to be created, and replace the existing one.

Example: load a PDF and repeatedly click the zoom +/- controls in the
lower-right corner of the viewer.

BUG=444995

Review URL: https://codereview.chromium.org/886883002

Cr-Commit-Position: refs/heads/master@{#314147}

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/c927918..3d91aad

CQ_EXTRA_TRYBOTS=tryserver.blink:linux_blink_rel,linux_blink_dbg
TBR=stephana@google.com

Review URL: https://codereview.chromium.org/883463004

Cr-Commit-Position: refs/heads/master@{#314146}

This [\\/] should account for both Windows \ and Posix /.

BUG=None
TEST=git cl presubmit -uv
R=danakj@chromium.org

Review URL: https://codereview.chromium.org/892743002

Cr-Commit-Position: refs/heads/master@{#314145}

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/3e5825c..672177b

TBR=vsevik@chromium.org,japhet@chromium.org

Review URL: https://codereview.chromium.org/890423002

Cr-Commit-Position: refs/heads/master@{#314144}

BUG=453982
TEST=tested manually
TEST=content_unittests:ServiceWorkerDispatcherHostTest.Register_FileSystem*

Review URL: https://codereview.chromium.org/889323002

Cr-Commit-Position: refs/heads/master@{#314143}

BUG=452950

Review URL: https://codereview.chromium.org/890463003

Cr-Commit-Position: refs/heads/master@{#314142}

A follow up to https://codereview.chromium.org/883773004/

BUG=None

Review URL: https://codereview.chromium.org/889913002

Cr-Commit-Position: refs/heads/master@{#314141}

This is a two part fix:
1. Fix get_sdk_extras_packages.py to correctly identify its own location.
2. Fix install-build-deps-android.sh to launch get_sdk_extras_packages.py
   when the current-working-directory isn't the location of the script.
This fixes the script when run in the way recommended by
https://code.google.com/p/chromium/wiki/AndroidBuildInstructions

Review URL: https://codereview.chromium.org/869633003

Cr-Commit-Position: refs/heads/master@{#314140}

BUG=454283

Review URL: https://codereview.chromium.org/886073003

Cr-Commit-Position: refs/heads/master@{#314139}

BUG=chromium:449920
TEST=apprtc loopback with resolution 640x354 on peach pit

Review URL: https://codereview.chromium.org/874083005

Cr-Commit-Position: refs/heads/master@{#314138}

This builds on the previous refactorings (linked in the bug), and unifies the
parsing of object files. It also removes the fuzzy matching of offsets done
that was previously also done in patch_orderfile.py.

It is also faster for three reasons:
- Elimination of an O(N^2) search
- Parallelization of object file parsing.
- No binary search

BUG=452879

Review URL: https://codereview.chromium.org/874683004

Cr-Commit-Position: refs/heads/master@{#314137}