GitLab

This time it should work properly. I checked all the images manually.

BUG=407781
TBR=newt@chromium.org

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

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

Native infobars are already capable of choosing between
a resource and a Bitmap for the infobar icon.  Pass this
information through to the Java side so the UI widgets
can also display the Bitmap version of the icon.

BUG=453170

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

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

There aren't many TODO(GYP) remaning and these are the ones I can do
since I don't have a Mac yet.

Tested with the following command line:

$ gn gen --args='os="android" cpu_arch="arm"' out_gn_android/Debug

BUG=None
TEST=see above
R=brettw@chromium.org

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

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

On OSX 10.10, using launch_msg() to remove a job does not guarantee that the
task is synchronously killed. The unit test needs to wait for the task to be
killed before proceeding.

BUG=390276

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

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

Add AlternateProtocolInfo member to HttpStreamFactoryImpl::Job.  Add
AlternateProtocolInfo argument to constructor, remove it from MarkAsAlternate.
This will be necessary in MaybeMarkAlternateProtocolBroken to pass it on to
SetBrokenAlternateProtocol().  See https://crrev.com/665083009 for context.

BUG=392575

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

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

Before the instrumentation, startup time wasn't reported in profiler data.

Profiling it will allow the genre of detailed analysis of the startup time via jankiness investigations.

This change requires temporarily initializing tracking earlier than usual. Thie will either removed once the investigations complete, or will be officially converted to permanent code.

BUG=453640

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

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

R=lcwu@chromium.org
BUG=451482

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

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

This patch removes additional sets in BuildRasterQueue and moves it
to a separate client function that is called by the tile manager.
Having this as a separate function explicitly allows us to control the
setting instead of implicitly relying on the fact that every time we
rebuild a queue this setting might change.

R=danakj
BUG=451147

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

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

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}