GitLab

Previously if the PDF plugin is disabled or click-to-play, the loading
indicator appeared over the click-to-play overlay. This hides the
loading indicator until a message is received from the plugin.

BUG=441132

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/451e822..912ed6e

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

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

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

BUG=417532

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

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

Currently, constrained web dialogs cannot autoresize. This change gives an option to pass in a minimum and maximum size, which enables the autoresizing functionality.

OSX has not yet been implemented.

BUG=217034

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

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

Add a null check instead of DCHECK() for web contents since it can be NULL
and cause a crash on debug builds, if the tab is closed while the plugin
settings bubble is visible and the setting was changed to Allow plugins.

BUG=440410

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

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

Remove dependency on the default constructor of StatsReport. This is in preparation of a WebRTC roll where this constructor has been removed.

TBR=perkj@chromium.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/ef1eaab..33cbe26

TBR=

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

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

TBR=machenbach@chromium.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/23bf4d4..ef1eaab

TBR=

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

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/2df2b3e..23bf4d4

TBR=

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

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

It makes the ownership clear and it is a good pratice when transfering
ownership from caller to callee. Passing a scoped_ptr makes it harder to
misuse.

Discussion in chromium-dev mailing-list:
https://groups.google.com/a/chromium.org/d/topic/chromium-dev/9rTsfEI1Ux0/discussion

BUG=None
TEST=unit_tests --gtest_filter=ConfirmBubble*
R=pkasting@chromium.org,avi@chromium.org

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

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

Do not use an action when naming an event that will be fired to the
delegate. Instead of saying what the delegate should do or will do, just
say what event happened. So in this case, say to the delegate that the
instructions import link on the Bookmarks Bar was clicked.

BUG=None
TEST=have zero bookmarks in your Bookmarks Bar, so you can see the link there,
click on it to show the import dialog. No regressions should be observed.
R=sky@chromium.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/c517219..2df2b3e

TBR=

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

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

Blink side here (depends on this CL):
https://codereview.chromium.org/782693003/

BUG=317991

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

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

BUG=428920

R=mfoltz@chromium.org

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

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

Revert of Ensure that the browser’s copy of page id is in sync with the renderer’s. (patchset #1 id:1 of https://codereview.chromium.org/795233002/)

Reason for revert:
Still crashing.

Original issue's description:
> Ensure that the browser’s copy of page id is in sync with the renderer’s.
>
> This might-maybe-hope-not crash, and so has debugging statements.
>
> BUG=407376
> TEST=no crashing, we hope
> TBR=rsesek@chromium.org
>
> Committed: https://crrev.com/0eba5c5e63e52330d3409193f5184aaffac45e90
> Cr-Commit-Position: refs/heads/master@{#308036}

TBR=rsesek@chromium.org,creis@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=407376

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

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

BUG=302083
R=thakis@chromium.org
R=kalman@chromium.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/e7e7876..c517219

TBR=

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/43893e4..451e822

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

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

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

Refactor URLRequestAdapter::Read to fix unbound buffer growth and data corruption in case of synchronous success.

BUG=433348

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

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

BUG=
TBR=robertphilips

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

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

__NR_open, __NR_access are not defined on arm64. So, I just blocked
the syscall to build properly on arm64.

BUG=None

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

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

These entries were found with the following command line:

$ git grep Newtab chrome/

BUG=None
R=sky@chromium.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/71085d0..e7e7876

TBR=

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

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

They don't seem to be necessary anymore.

R=thestig@chromium.org

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

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

chrome://settings and upon HotwordService creation.

Note: This also changes HotwordAudioHistoryHandler to maintain the
previous state of the pref if a successful connection could not be made
with the server. A separate CL will be created for reporting an error in
connection to the user.

Adds a new function to browser_options_handler to be used as a callback
to the call GetAudioHistoryEnabled.

Adds a task_runner to HotwordAudioHistoryHandler to enable testing.
Test added: HotwordServiceTest.AudioHistorySyncOccurs.

Changes to hotword_private_apitest reflect the fact that
hotword_audio_history_handler now takes a task_runner and that the
previous state of the pref is maintained.

Issue 436683 is included in this because the times which we sync (every
24 hrs, profile startup, opening chrome://settings) are considered
sufficient enough to maintain state and close enough to launching for
the purpose of opening the audio_verification_app.

BUG=438451,436683

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

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

BUG=439512

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

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

BUG=439512

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

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

BUG=439512

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/fbaace0..43893e4

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

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

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/cb219a0..71085d0

TBR=

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

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

Add OAuth2 authentication for speech recognition requests from the launcher, when audio-history is opted-into.

Audio history allows a user to see the speech recognition result at https://history.google.com/history/audio. This change will attach an auth (OAuth2) token to the recognition request, which is used to associate the request with the user. This only happens when the user opts-in to audio history, and only on speech recognition from the app launcher.

This change adds a helper class that handles retrieving and refreshing the auth token, and plumbs that token into the speech recognition request.

BUG=397019

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/502e869..cb219a0

TBR=

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/f2e542d..502e869

TBR=

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/b676431..f2e542d

TBR=

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/d8c8e9e..b676431

TBR=lushnikov@chromium.org,vsevik@chromium.org,dglazkov@chromium.org

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

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

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

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

Fix file naming issue causing page_run_end_to_end tests to never run (at least when run locally using run_tests script).
Fix a failing test from this file.

Found this while fixing unit test for max_failures patch
(https://codereview.chromium.org/790673005/).

BUG=440101

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

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