GitLab

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

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

Apart from the places which expect a SyncLoadResult, it's only read in
RedirectToFileResourceHandler. In that case, it's always net::OK;
ResourceLoader checks request_->status().is_success() first in
OnResponseStarted before calling CompleteResponseStarted().

BUG=none

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

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

Two main cases:
1. <label> not wrapping input + span
2. for accessibility -- for=<selector-id> should instead use aria-labelledby

BUG=409938

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

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

This matches the logic in nss_ssl_util.cc.

Note: this CL has a test which only affects Windows XP. We
have no try coverage on XP, so if the XP bots break, please
revert.

BUG=405630

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

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

This is a re-work of palmer's patch at https://codereview.chromium.org/515023002/ which has more context, but comes down to stricter bounds checking.

We also correct an arithmetic bug when copying the image behind a control that is positioned before the origin of the image.

BUG=398384

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

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

R=dmazzoni@chromium.org
BUG=408405
TEST=keyboard navigate to bookmark star, press space: bookmark deletes and focus still exists.

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

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

Trying to initialize it causes content_shell --enable-browser-side-navigation
to crash on startup.

It corresponds to ResourceRequestInfo::GetVisibilityState, but that's only used
for categorizing requests made from prerender which needs a different bit
(WebPageVisiblityStatePrerender vs. others), so we'll need to revisit it
anyway.

BUG=376014

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

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

BUG=

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

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

unset GYP_DEFINES
python build/gyp_chromium
ninja -C out/Release -t commands base >gyp.txt

gn gen out/gn --args="is_debug=false is_component_build=false"
ninja -C out/gn -t commands base >gn.txt

python tools/gn/bin/gyp_flag_compare.py gyp.txt gn.txt >flags.txt

Outputs this at the moment: https://gist.github.com/sgraham/93e159eba1601a9e3668

(only work on Linux because of lack of handling @rsp files)

NOTRY=true
R=ajwong@chromium.org, brettw@chromium.org

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

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

Remove unused QUIC_VERSION_20.

We are already speaking QUIC_VERSION_21 in the wild, and QUIC_VERSION_20
was never intended to be used. Not much code removed in this CL, but
should make end to end tests a bit faster.

Merge internal change: 74594489
https://codereview.chromium.org/532093002/

Change GetLeastUnacked() to return the next packet that could be sent
when there are no packets in flight instead of 0.

This should help solve the problem we are seeing in trace logs from
internal servers where the ack line drops to 0.

Merge internal change: 74396185
https://codereview.chromium.org/530343002/

Change IsRetransmittable to not check transmission type but instead let
ShouldDiscardPacket check it directly.

Merge internal change: 74359363
https://codereview.chromium.org/536743002/

Change the QUIC CongestionMap to be based on a vector.

Saves ~0.75% CPU when profiling.

Merge internal change: 74321352
https://codereview.chromium.org/528083004/

Some improvements to quic_client_bin.
  - user can supply specific version
  - added a number of comments
  - added a number of descriptive ERROR logs on failure
  - removed unneeded flow control options (SetDefaults covers this)

Merge internal change: 74309904
https://codereview.chromium.org/532073002/

R=rch@chromium.org

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

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

This callback has the reentrance issue as calling MediaPlayerBridge::Release() will call into function.
The worst case happens when BrowserMediaPlayerManager dtor is called. In that case, this function will be called on each player, while BMPM is in a partially destroyed state.
Since BMPM controls the lifetime of the players, it can call OnMediaResourcesReleased() when it calls Release() on a player.
For MSE case, we also simplified the situation here since we can check MSP::IsSurfaceInUse(). This CL also fixes that call.

BUG=410096

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

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

These were failing on Android Builder (dbg) FYI bot.

BUG=409152
TBR=qinmin@chromium.org

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

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

Adds a DebugInfo message to the DistilledPageProto. The data for this message
is copied from the DomDistillerResult proto DebugInfo.

For now, only a field for the distiller log exists.

Depends on https://codereview.chromium.org/493853006/

BUG=409274

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

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

The CL https://codereview.chromium.org/415343002/ accidentally
introduced new files in //chrome/chrome_browser.gypi without
adding a newline.

This CL just cleans up the style of the .gypi.

BUG=383630

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

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

This keeps the "Visit Website" link from showing up in chrome://extensions,
which makes sense because shared modules are automatically installed.

BUG=408927

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

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

The notable changes:
. Do not allow horizontal dragging of the split windows.
. Do not show the 'Split' button when dragging a non-split window. Instead,
  highlight the split windows as drop targets.
. When a window is dropped on top of a left/right split window, switch to that
  window in that part of the split.

BUG=404119
R=oshima@chromium.org

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

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

TBR=xians@chromium.org
BUG=409485
NOTRY=true

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

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

This dialog explains the implications of accepting remote
assistance and provides a way for the user to decline
assistance.

This ensures that even if the caller (Hangouts) is compromised,
it won't be able to control the user's desktop without user
interaction.

BUG=405139

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

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

Some checkouts have the build directory mounted, which causes the script
to throw an exception when it attempts to remove the directory itself.

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

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

Initial ExtensionManagement class implementation, with legacy preference parsing and interface for obtaining basic structure of extension management settings.

BUG=177351
TEST=new unit test

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

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

This CL removes the passed parameter of device_scale_factor and in BrowserPlugin::UpdateDeviceScaleFactor since it's not used.

This CL also removes GetSizeParams because it's only used once.

Repaints are now set whenever the device scale factor changes.

Also remove all calls to .get() on scoped_ptrs which are unnecessary.

BUG=none
TBR=creis@chromium.org for trivial render_view_impl change.

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

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

When in overview mode, if a window is closed, then the rest of the windows
that behind the closed window in the stack are moved on step forward. However,
if there are no windows behind the closed window, then move the windows in
front of the closed window backward one step.

BUG=408280
R=oshima@chromium.org

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

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

We are removing the corresponding runtime flag in Blink, as it currently
has no effect. This patch removes its usage in Chromium side.

Blink side: https://codereview.chromium.org/535463002/

BUG=None

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

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

The gmock log spam was annoying.

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

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

The iOS6-style APIs have been deprecated, but the replacements are
more verbose and brittle so this category wraps the replacements in
methods similar to the iOS6-style APIs.

This category lives in a new target and directory: src/ui/ios, so
this CL sets up all the gyp/DEPS stuff and only adds a single
method. There will be more methods added in subsequent CLs.

BUG=364419

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

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

DVLOG(0) is way too spammy for such a low-level function.

BUG=376450
TBR=dalecurtis

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

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

This CL makes sure that the renderer is initialized before being used to handle
a navigation that has just committed.

BUG=376082

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

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

https://skia.googlesource.com/skia/+log/937c9c7eb4e06d4d3bc495e129c7b8103a5d6c0f..74364c97740eb48c86d6e7b42520c77130eb1486

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

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

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

https://chromium.googlesource.com/chromium/blink/+log/104ecc5a02c328914490c6c17ccc8a17d8e40fca..9d34a87dee82cbdcd5876669aa1cd1a430289e09
TBR=tkent@chromium.org,arv@chromium.org

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

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

The underlying issue has been fixed.

BUG=273986
TBR=andrewhayden@chromium.org

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

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

Revert of Changes exception for gyp files to only gyp files in build/ (patchset #1 id:1 of https://codereview.chromium.org/512693004/)

Reason for revert:
*SIGH* Encountered another problem. See
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...
. Reverting again.

Original issue's description:
> Changes exception for gyp files to only gyp files in build/
>
> I changed analyzer such that if a gyp file (or an included gypi file)
> changes all targets in the gyp file are marked as changed. This means
> I should no longer have to special case gyp files. I'm leaving the
> exception in place for any gyp files in build as any change to those
> really needs to recompile the world.
>
> BUG=109173
> TEST=none
> R=thakis@chromium.org
>
> Committed to pending queue: https://chromium.googlesource.com/chromium/src/+/5110e8b
>
> Committed: https://chromium.googlesource.com/chromium/src/+/e53e29fc161f96871160f6db39919752a0376333

TBR=thakis@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=109173

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

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

Last commit (#293147) got the boolean logic wrong.

BUG=410327
TBR=tkent,rvargas

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

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

This reverts commit 6087f972.

I reverted this on trunk instead of on branch-heads/2125. >.<

BUG=405056
TBR=noms@chromium.org

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

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

The outputs of a target were not previously considered when doing header checks.

Also, adds the new check_includes, testonlg, and allow_circular_includes_from variables to the "desc" command.

R=jamesr@chromium.org

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

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

BUG=None
R=oshima@chromium.org
TEST=buid chrome with use_athena=1
NOTRY=TRUE

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

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

Server side bug: b/17140979

BUG=none
TEST=none

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

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

Browser should go back to video page(having inline video) through
escape key of BT keyboard instead of goig back to main page.

BUG=396996

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

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

BUG=408962
TEST=SplitSplitViewControllerTest.LandscapeOnly

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

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

This requirement only exists on this implementation of PasswordStore.

BUG=408783

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

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

BUG=410202
TEST=Use Kinect camera on Linux. Go to page https://src.chromium.org/svn/trunk/src/chrome/test/data/webrtc/manual/peerconnection.html. Press button "Get devices". An entry "Microsoft Kinect Camera" should show up in the "Video source" list. Press button "Request GetUserMedia". You should get a stream with size 640x480 in the preview.

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

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