GitLab

This CL also reinstates the previous behaviour of MenuButton whereby a
second click on the button hides the menu, which is made possible by a
saner ownership model.

BUG=415410

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

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

R=tedchoc@chromium.org,lcwu@chromium.org
BUG=None

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

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

Fixes chrome://inspect?tracing

BUG=416578
TBR=dsinclair
CC=caseq

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

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

Last update to Pica pageset did not use vulcanizer, so load
time regressed by 75-100%. Now fixed.

TBR=ykyyip@chromium.org,tonyg@chromium.org
NOTRY=true

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

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

Add a null check on OriginPowerMap, since it is possible to get NULL from
OriginPowerMapFactory::GetForBrowserContext() in guest mode.

BUG=416374

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

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

BUG=None

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

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

Adds UMA logging to track if any batteries were present
at the start of Battery Status API.

BUG=

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

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

BUG=416130, 410550

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

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

BUG=416722
R=brettw@chromium.org

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

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

R=estade@chromium.org
BUG=416761

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

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

Adding missing field of transportId to
cast.streaming.udpTransport.setOptions() API. The
API is now working.

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

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

Experiments reservation mechanism moved to server-side therefore variation_ids.h is removed as it is not necessary anymore.

BUG=416881

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

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

BUG=416657

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

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

BUG=416941

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

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

1. Renamed FakeSocket to FakeStreamSocket and FakeUdpSocket to
   FakeDatagramSocket,
2. Added factories for both stream and datagram fake
   sockets,
3. Move fake sockets to separate files out of fake_session.h

BUG=402993

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/562c87e..b246733

TBR=pdr@chromium.org,wjmaclean@chromium.org,ager@chromium.org

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

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

Increase the #of pings from 9 to 36 for Dev (72 secs of
unresponsiveness).

Increased it to reduce the number of crash dump reports
we receive. Will revert back, when we can auto triage
jank on UI/IO threads.

BUG=352489
R=sky@chromium.org

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

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

This change prepares DeviceInfo class (and all other sync
classes that depend on DeviceInfo and implement DEVICE_INFO
syncable type) to be moved to components/sync_driver library
by removing any dependencies on chrome/browser.

To achieve this all chrome/browser specific code was moved to
LocalDeviceInfoProviderImpl which will stay in browser/sync/glue.
All other classes including DeviceInfo and LocalDeviceInfoProvider
should now depend only on base or components. I'll move
them to components/sync_driver in the next change.

Details:

1) In ui_thread_search_terms_data.cc DeviceInfo::GetLocalDeviceType()
was used just to find out if the device is an Android phone.
Since this code has nothing to do with Sync I replaced these
with direct calls to ui::GetDeviceFormFactor() - that exactly
what DeviceInfo::GetLocalDeviceType() does internally.

2) In sessions_sync_manager.cc - calling DeviceInfo::GetLocalDeviceType()
isn't necessary anymore because the local device type can be
accessed via the owned LocalDeviceInfoProvider instance.
This allowed me to move GetLocalDeviceType() to LocalDeviceInfoProviderImpl
and make it private.

3) DeviceInfo::MakeUserAgentForSyncApi() is called only
from the code that will stay in browser/sync/glue.
I moved the implemented to LocalDeviceInfoProviderImpl class.

BUG=396136

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

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

BUG=none
TBR=pkasting@chromium.org

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

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

, because ThreadSanitizer v2 doesn't need base::subtle to be annotated.
We can't drop the dependency on libdynamic_annotations yet, because some code calls RunningOnValgrind().
Also remove the unused dynamic_annotations.h headers from base/ and add one to content/test/, which uses RunningOnValgrind().

BUG=349861
R=ajwong@chromium.org,phajdan.jr@chromium.org

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

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

R=estade@chromium.org
BUG=none

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

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

Files moved and includes adjusted.

BUG=387072

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

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

BUG=416948

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

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

* Implement detachGLFunctor. The contract here is after detachGLFunctor
  is called, DrawGL is no longer called until the next requestDrawGL.
  Implement just with a mFunctorAttached variable.
* Only create one HardwareView. Each GLSurfaceView creates its own GL
  thread, but we only assume a single render thread. Using SurfaceView
  or hosting multiple AwContents in the same GLSurfaceView is too hard,
  so just do the simplest thing, and create the first view has hardware
  accelerated.

BUG=416981

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

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

This was being dropped on the floor.

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

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

BUG=371692

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

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

BUG=416721
R=brettw@chromium.org

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

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

R=yfriedman@chromium.org
BUG=None

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

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

Added checks for missing arguments in event_sender.cc. Also simplified the uses of x & y in EventSender::GestureEvent.

BUG=415970

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

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

This CL moves RulesRegistryService to //extensions. Since it was dependent on ContentRulesRegistry, we move the essential elements of that interface to //extensions as well, renaming what remains as ChromeContentRulesRegistry.

BUG=352293

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

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

This avoids the majority of the Chromium builders attempting to build
these isolates.

BUG=

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

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

This allows the cast extension to turn off wifi scan while
streaming is in progress, and also set the "media streaming mode"
further experimentation is need to see if these options are actually
useful.

BUG=410500

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

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

On the windows side (and likely chromeos for some devices) we can end
up in a situation where the pixel bounds is not an even multiple of the
scale factor. This means when TiledLayer calculates the scale it ends
up off.

BUG=416655,416775
TEST=added test coverage
R=danakj@chromium.org

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

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

Only whitelisted webUIs could embed webview tag, specifically, they have to be declared in _api_features.json and they have to relax CSP to allow internal browser plugins.

BUG=

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

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

BUG=414766

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

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

When an Android View is invalidated outside of frame dispatch
(animation/input/draw), a sync barrier may be inserted into the shared
UI thread message loop. This effectively blocks dispatch of all Chrome
tasks posted to the browser UI thread for an entire frame (or more if
the View is continually invalidated).

There are currently no easy or even automated ways to avoid this
untimely invalidation, and judicious coding can only take us so far. As
an intermediate assist in debugging the issue, use reflection and a
crude form of MessageQueue inspection to trace known cases where the
MessageQueue is blocked by a sync barrier.

Note that this detection is not perfect, neither is it exact, but it's a
solid proxy for informing traces about such pipeline stalls.

BUG=407133

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

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

ChannelMojo doesn't work on Windows with existing implementaion
and this CL fixes it.

On Windows, ChannelHandle isn't immediately usable: The handle has
to be activated through ConnectNamedPipe() windows API, which is done in
its own Connect() handlshaking phase. ChannelMojo didn't Connect()
underlying channel and took the ChannelHandle over so the handle
wasn't activated.

Instead of hijacking underlying ChannelHandle, this CL actually Connect()s
underlying channel, creates a pipe on the server side, send one side of the
pipe to the client process, and use the pipe for the MessagePipe
initialization.

These initialization task is encapsulated behind new MojoBootstrap class.
ChannelMojo creates MojoBootstrap class to get the PlatformHandle which
is already activated and usable.

BUG=377980
TEST=ipc_mojo_bootstrap_unittest.cc, ipc_channel_mojo_unittest.cc
R=viettrungluu@chromium.org, darin@chromium.org, yzshen@chromium.org

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

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

Revert of Removing browser.Start() and instead starts the browser at the point of creation. (patchset #6 id:120001 of https://codereview.chromium.org/565803002/)

Reason for revert:
investigate build error

Original issue's description:
> Removing browser.Start() and instead starts the browser at the point of creation.
>
> BUG=412126
>
> Committed: https://crrev.com/01c68e8765254d9971296b291ff718b4f6af3004
> Cr-Commit-Position: refs/heads/master@{#295591}
>
> Committed: https://crrev.com/74c6ef17a08754abadb6fb64da14249898285fa8
> Cr-Commit-Position: refs/heads/master@{#296107}

TBR=nednguyen@google.com,slamm@chromium.org,slamm@google.com
NOTREECHECKS=true
NOTRY=true
BUG=412126

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

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

Previous to this change, suffixing a "." to the URL from which a
dictionary was being suggested would allow servers to bypass the
check against a subdomain (e.g. evil.protected.good.com) setting a
dictionary to be used by a superdomain (.good.com).

BUG=389451
R=rsleevi@chromium.org

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

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

Properly initialize |drop_remaining_touches_in_sequence_| after
its addition in https://codereview.chromium.org/586553002/.

TBR=rbyers@chromium.org
NOTRY=true

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

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