GitLab

TEST=TaskSwitchMetricsRecorderTest.VerifyTaskSwitchesForDesktopAreRecorded
TEST=DesktopTaskSwitchMetricRecorderTest.ActivatePositionableWindowWhenNullWindowWasActivatedLast
TEST=DesktopTaskSwitchMetricRecorderTest.ActivatePositionableWindowWhenADifferentPositionableWindowWasActivatedLast
TEST=DesktopTaskSwitchMetricRecorderTest.ActivatePositionableWindowWhenTheSamePositionableWindowWasActivatedLast
TEST=DesktopTaskSwitchMetricRecorderTest.ActivatePositionableWindowWhenANonPositionableWindowWasActivatedLast
TEST=DesktopTaskSwitchMetricRecorderTest.ActivateNonPositionableWindowBetweenTwoPositionableWindowActivations
TEST=DesktopTaskSwitchMetricRecorderTest.ActivateNullWindow
TEST=DesktopTaskSwitchMetricRecorderTest.ActivateNonPositionableWindow
TEST=DesktopTaskSwitchMetricRecorderTest.ActivatePositionableWindowWithNonInputEventReason
TEST=DesktopTaskSwitchMetricRecorderWithShellIntegrationTest.ActivatePositionableWindowWithInputEvent
TEST=DesktopTaskSwitchMetricRecorderWithShellIntegrationTest.ActivatePositionableWindowWithNonInputEvent

BUG=489813, 489814

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

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

R=khorimoto@chromium.org
BUG=

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

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

The right branch of the ternary was an rvalue, so the entire ternary operator made a copy of the child to parent map. This is expensive and is unnecessary.

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

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

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

We always update the model refreshSelectedTab, but that should not
end up making any visual updates. In onPrimaryColorChanged we should
be checking the model for isUsingBrandColor.

BUG=497241

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

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

When navigating cross-process, we use swapped-out RenderFrameHost as placeholders. The goal is to replace this with RenderFrameProxyHost, which is already the case for all subframes, but not the main frame. The code is far along that this can be done now for the main frame as well.

This CL starts the path to removing the swapped-out RenderFrameHost concept by deleting the RenderFrameHost for the main frame on cross-process navigations and only keeping a RenderFrameProxyHost around. It is doing this for --site-per-process only, so all details can be fleshed out. Subsequent CL will remove the --site-per-process restriction and enable it in all modes of Chrome.

BUG=357747

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

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

BUG=426500

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

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

"base/basictypes.h" in net/base/mime_sniffer*.

R=asanka@chromium.org
BUG=488550
TEST=none

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

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

The stack doesn't make sense, so it could be a DrMemory bug or surprising
function merging.

TBR=bruening@chromium.org
BUG=487500

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

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

R=bnc@chromium.org,cbentzel@chromium.org
BUG=382120
TEST=none

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

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

This CL takes care of moving unittest file to cc/base. The
ListContainer class now gets tested using dummy classes of following
types -

1. DerivedElement - This has classes derived from it. This tests
classes created using ListContainer(size_t max_size_for_derived_class)
constructor.

2. NonDerivedElement - This does not have any classes derived from it.
This tests classes created using ListContainer() constructor.

BUG=492952
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

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

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

Pulls methods from the Impl into the Interface so that
ChromeActivity can take in a generic TabCreator instead of
a ChromeTabCreator implementation.

This is prep work for making it so that document mode's TabDelegate is also a TabCreator, which will let it be
usable in webapp mode for handling window.open calls.

BUG=451453

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

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

This test becomes invalid with the upcoming angle changes. This test
expects webgle_dot_emu to be always there in translated shader source
but with the constant folding implemented in ANGLE for built-in
'dot', 'dot' function call will be replaced by actual result when all
the arguments are constant expressions and webgl_dot_emu will no
longer be there in translated shader source.

R=zmo

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

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

This found a bug in saturated_cast that was covered by ClampToInt tests
so it fixes that.

If you have a floating point value of MAX_INT, then comparing an
integer against it for equality with promote it to a float and
it will compare true. However if you cast the float to an int, its
actually outside the bounds of integer, so you can end up with a
negative int as a result. Added unittests to check this for
saturated_cast.

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

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

This is a port of https://codereview.chromium.org/475273002 to gn.

Also add a comment that explains that the skia_opts target in the GN build
isn't correct.

BUG=491209,82385,496512

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

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

We need a hook to customise how MediaPipelineDevice is created,
orthogonal to the vendor-specific aspect of creation.  Decided to
keep existing factory method for the vendor customisation so there's
no need to change vendor implementations for now (although this
will all get moved into libcast_media before long).  New hook is
in CastContentBrowserClient and can be substituted as the other
Platform* functions already are.

BUG=

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

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

This way apps can make assumptions about orders of destruction of message loop, delegate, applicationimpl and it works the same across all apps.

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

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

SSLConfigService::GetSSLConfig is not thread-safe. This should also resolve the
test flakiness; the policy change applies on a thread hop, so the value must be
queried on a subsequently scheduled hop.

Add a DCHECK to SSLConfigServicePref to catch this in future.

BUG=496872

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/cb67414..b500ffa

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
TBR=caryclark@google.com

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

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

This reverts commit 422456f9.

TBR=zea@chromium.org
BUG=465354
NOTRY=true
NOTREECHECKS=true

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

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

This CL fixes a bug where we were incorrectly building the
chrome_template_resources target with manifest_package = "org.chromium.chrome" for
internal Chrome builds.

This CL makes chrome_public_apk (which is not shared with the internal build)
depend on chrome_template_resources instead of chrome_staging_java (which is
shared with the internal build). This allows the internal build to depend on a
chrome_template_resources-like target which uses the correct package name.

BUG=495428
TEST=None

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

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

Blink currently does this operation on its display item lists to remove
superfluous begin/end pairs. To port Blink's current display item list
operations, such an operation is required.

This is not possible when in "cached picture" mode, but Blink currently only
does this on the unmerged list (in blink::DisplayItemList::add), so this should
be sufficient (if unfortunately asymmetric).

BUG=484943
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

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

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

Revert of ios: Remove use of MessageLoopProxy and deprecated MessageLoop APIs (patchset #3 id:40001 of https://codereview.chromium.org/1160403005/)

Reason for revert:
Broke IOS_Device compile.
https://build.chromium.org/p/chromium.mac/builders/iOS_Device/builds/8802

Original issue's description:
> ios: Remove use of MessageLoopProxy and deprecated MessageLoop APIs
>
> BUG=465354
>
> Committed: https://crrev.com/9c8e5d7259661cda32b698ee88254491e6984900
> Cr-Commit-Position: refs/heads/master@{#333080}

TBR=stuartmorgan@chromium.org,skyostil@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=465354

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

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

This change addresses some concerns raised post-submit in my
previous CL: http://crrev.com/1151833003

BUG=480604

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

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

This patch was mostly autogenerated with
https://codereview.chromium.org/1010073002/.

BUG=465354
TBR=nick@chromium.org

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

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

BUG=465354

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

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

- Split the tests into their own file.
- Add the deleted on server case.
- Factor out many common subroutines.

BUG=480604

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

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

BUG=489317

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

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

Add skeleton UI for sync page. This CL creates the new page, adds its strings, and brings in new Polymer dependencies on the paper-radio-* elements.

This CL does not add functionality or proper styling to the page. This will be done in follow-up CLs.

R=jhawkins@chromium.org

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

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

BUG=496438
NOTRY=True

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

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

This landed in https://chromium.googlesource.com/external/mojo/+/a500628dfc647cc520d999a972b80d9ff62b0b26, but I want to use it now in the Chromium repo.
BUG=478251

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

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

Cleaning up some tests to use helper functions and remove dangling
pointers. It also fixes the DisabledManagerWorksAfterBrowserRestart
test which wasn't initializing after creating a new manager.

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

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

Querying is done asynchronously, with a browser-side cache of responses.
GPU will try out the requested overlay configuration and report if it was
successful. Response delay is typically 1 frame but could be more in
rare situations. A maximum cache size is imposed so that a video that's being
moved/resized won't cause memory consumption to explode

TBR=danakj (trivial enum change)

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

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

With the fix of 411038 in (https://codereview.chromium.org/1162083003).
Re-enable the failing test on Android Nexus 5.

BUG=412145
TEST=build/android/test_runner.py gtest --debug -s webkit_unit_tests \
     --num_retries=0 --gtest-filter=TouchActionTest.Pan

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

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

BUG=496936

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

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

TBR=sievers (already reviewed)

Reason for reland:
Fix build for ozone_demo.

> Revert of ozone: Remove singleton pattern for SurfaceFactoryOzone. (patchset #1 id:1 of https://codereview.chromium.org/1159493010/)
>
> Reason for revert:
> Broke Linux ChromiumOS compile.
> http://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Ozone%20Builder/builds/17764/steps/compile/logs/stdio#error1
>
> Original issue's description:
> > ozone: Remove singleton pattern for SurfaceFactoryOzone.
> >
> > OzonePlatform is sigleton already, so it's not necessary for
> > SurfaceFactoryOzone to be singleton.
> >
> > Committed: https://crrev.com/48b0a7165bd6c9d889fc6d6959a6e59cf7ef9dc6
> > Cr-Commit-Position: refs/heads/master@{#332889}
>
> TBR=spang@chromium.org,sievers@chromium.org,alexst@chromium.org,dongseong.hwang@intel.com
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
>
> Review URL: https://codereview.chromium.org/1167533007
>
> Cr-Commit-Position: refs/heads/master@{#332899}

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

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

This CL picks up the value serialization changes from crrev.com/1157793003 as
well as the refactor of story_runner_unittest so that it no longer depends on
BuildbotOutputFormatter.

As a result, there are no remaining dependencies on BuildbotOutputFormatter in
Telemetry, save for results_options.py, which allows for consumers to continue
to use BuildbotOutputFormatter for the time being.

BUG=477056
R=nednguyen

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

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

R=jlklein@chromium.org, jlklein
BUG=none

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

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

See issue for details.

BUG=493438

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

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

Minor update to the docserver. This is a multi-faceted patch which takes
care of some issues with our server:

 - No more AppEngine cron. The front-end is no longer responsible for
   updating Datastore entities, it just serves up the data that's there.

 - No more Gitiles or Subversion filesystems. Github filesystems are
   deprecated but not fully removed.

 - New LocalGitFileSystem can retrieve arbitrary repo history data
   directly from a local copy of the repo using commandline git.

 - Refactor appengine_wrappers; unused concepts deleted, datastore and
   URL fetching moved to their own first-class concepts, added
   environment_wrappers for more generic cross-platform factories.
   Memcache still lives in appengine_wrappers for now.

 - Clean up some of the mess caused by Gitiles support: no more refresh
   tasks, so no more refresh "paths" in data sources. Most of the
   foo_data_source.py changes here are just reverting that crap.
   FileSystem versioning is no longer necessary and no longer used, but
   not fully removed in this patch.

 - Added update_cache.py, a utility which can scrape a local chromium
   repo and populate Datastore with the data needed on the front-end.
   This tool can also be used to load and store on-disk caches of said
   Datastore in order to speed up incremental updates from the same
   host.

BUG=
R=kalman@chromium.org

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

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

The main motivation is to enforce correct layering and dependencies

BUG=478250
R=jam@chromium.org

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

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