GitLab

https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/4599cc694fd8..7cdf69dbbf43

git log 4599cc694fd8..7cdf69dbbf43 --date=short --first-parent --format='%ad %ae %s'
2020-03-10 szuend@chromium.org [utilities] Remove call-site of Array#remove from CSSModel
2020-03-10 tvanderlippe@chromium.org Add custom ESLint rule to check ES import statements
2020-03-10 tvanderlippe@chromium.org Fix incorrect imports caught by ESLint rule
2020-03-10 tvanderlippe@chromium.org Add eslint-plugin-rulesdir to node_modules
2020-03-10 tvanderlippe@chromium.org Add testonly support for ts_library

Created with:
  gclient setdep -r src/third_party/devtools-frontend/src@7cdf69dbbf43

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/devtools-frontend-chromium
Please CC devtools-waterfall-sheriff-onduty@grotations.appspotmail.com on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

Bug: chromium:1011811,chromium:1050549,chromium:1060003,chromium:1060123
Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com
Change-Id: I125989ce042d820f71fb5071470e84739e19c584
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2097045Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#748870}

Bug: 1034648
Change-Id: I622b68ad3d410b2eb4e528ef705acdc7b5b1d8fc
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2090730Reviewed-by: Eric Foo <efoo@chromium.org>
Commit-Queue: John Budorick <jbudorick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748869}

If the navigation state of the active state changes such in a way that
indicates the URL has changed, we should update the URL bar.

Also adds a browser test to verify the expected behavior.

Bug: 1025607

Change-Id: I58431ac15fbb04d516f4a2637d4717431346fb25
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2094134
Commit-Queue: Mugdha Lakhani <nator@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: Charlie Reis <creis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748868}

Requested by internal client.
Note the API is added in 82, so there is currently no backward
compatibility requirements.

Bug: 1035469
Change-Id: I471da6a83a2aa156aa8441e5d033478d8416840f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095877
Commit-Queue: Bo <boliu@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748867}

BUG=b/149874690
TEST=None

Change-Id: I8390efa1df473a1c3a79726c01b75498aeaab989
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2087958
Commit-Queue: Muhammad Hasan Khan <mhasank@chromium.org>
Reviewed-by: Yury Khmel <khmel@chromium.org>
Reviewed-by: Hidehiko Abe <hidehiko@chromium.org>
Auto-Submit: Muhammad Hasan Khan <mhasank@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748866}

This API is unused since the launch of Cast Harmony.

Bug: 1058528
Change-Id: I6ac2df2e21a5a3bdf68a4ae0bcc70ec907305763
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2088230Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Takumi Fujimoto <takumif@chromium.org>
Commit-Queue: mark a. foltz <mfoltz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748865}

Tab properties will be used for upcoming Performance Manager session
restore policy.

Bug: 1059341
Change-Id: I81214143b55d747cfbde02d6a6959f5c83c36323
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2091653Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
Reviewed-by: Sébastien Marchand <sebmarchand@chromium.org>
Reviewed-by: François Doray <fdoray@chromium.org>
Commit-Queue: Hailey Wang <haileywang@google.com>
Cr-Commit-Position: refs/heads/master@{#748864}

flattenhtml is sub-optimal for a few reasons:

1) Makes the HTML code being evaluated larger than necessary.
   For example when both light and dark mode images are inlined,
   they increase the HTML code size, even though only one of the
   two is used at a given moment.
2) flattenhtml does not work with Polymer3 (polymer_modulizer).

Instead, images are registered with the Settings data source, so
that they can be served by dedicated URLs for example from
chrome://settings/images/cookies_banner.svg

Bug: 1026426
Change-Id: I0c77b3bad2f27e9c810833aa153faab884954775
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095555
Commit-Queue: dpapad <dpapad@chromium.org>
Reviewed-by: Rebekah Potter <rbpotter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748863}

This will be enabled by a finch experiment to reduce
timeouts to 6 seconds for a specific group of users.

Bug: 1059017
Change-Id: Ie2a22c1b7f39856c57133df0992238419439f1c2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2096901Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Ahmed Mehfooz <amehfooz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748862}

In the previous CL,
https://chromium-review.googlesource.com/c/chromium/src/+/2063733, I
added mode change for Canvas, such that it discard the content of canvas
from the previous frame if canvas needs a full repaint in the current
frame. In this Cl, I added the mode change feature for OffScreenCanvas.

Bug: 1054666

Change-Id: I2b685641c40f2a31b50e04ee739f3ef26423297c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2088021
Commit-Queue: Yi Xu <yiyix@chromium.org>
Reviewed-by: Khushal <khushalsagar@chromium.org>
Reviewed-by: Fernando Serboncini <fserb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748861}

This is some more "pre cleanup" of code that the linter complains about
when moving code into storage service.  Doing this ahead of time to
make the future gigantic move CL easier to reason about.

Bug: 1015214
Change-Id: I7dcd977b7906710a7516412a11d83d64661d19fa
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2093051
Auto-Submit: enne <enne@chromium.org>
Reviewed-by: Daniel Murphy <dmurph@chromium.org>
Commit-Queue: enne <enne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748860}

Bug: 1046181
Change-Id: Ide50b7bf6332e2c162a2aba1e255498ab986bf68
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2067721Reviewed-by: Yusuf Ozuysal <yusufo@chromium.org>
Reviewed-by: Gang Wu <gangwu@chromium.org>
Commit-Queue: bttk <bttk@chromium.org>
Auto-Submit: bttk <bttk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748859}

This is how the code was earlier, but I changed it recently in CL
2071387 to do a Create instead, which is likely causing the side-effect
of leaving a stale key out there.

Bug: 1059314
Change-Id: I48f07e30621b61a7db2b60495c3b0b286323b0a3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095919
Commit-Queue: S. Ganesh <ganesh@chromium.org>
Auto-Submit: S. Ganesh <ganesh@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748858}

We are field trialing SkiaRenderer on Windows and Android. We want to
keep some smoke test coverage of GLRenderer during the field trial
in case we need to stop the SkiaRenderer field trial and fallback to
GLRenderer.

Bots with updated test suites:

testing/buildbot/chromium.android.json
android-pie-arm64-rel

testing/buildbot/chromium.win.json
Win10 Tests x64

Bug: 1019787, 809722
Change-Id: I8ee553426539ed43d1a98cde74d4d458bc273fdd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2083391
Commit-Queue: Jonathan Backer <backer@chromium.org>
Reviewed-by: Yuly Novikov <ynovikov@chromium.org>
Reviewed-by: Jonathan Ross <jonross@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748857}

This change moves the initializing of the crash reporter sooner, before
the command line modes are being handled.

There is still some code that executes outside of the crash handler, such
as command line parsing and logging initialization.

Bug: 1060326
Change-Id: I2631d7ba1be18a33ca4984a127dd3b4f56f47003
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2096902Reviewed-by: Joshua Pawlicki <waffles@chromium.org>
Commit-Queue: Sorin Jianu <sorin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748856}

When the browser is in kiosk mode, the user is unlikely to be able to
act upon or dismiss infobar notifications, so they should be suppressed
when the browser is started in this mode. The only exception is the
Automation info bar, still shown for anti-abuse reasons.

Bug: 445256
Change-Id: Ie49572e5a548e70ce83cd4defafb76510fe384b8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095714
Commit-Queue: Eric Lawrence [MSFT] <ericlaw@microsoft.com>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748855}

This is a follow up of [1], where the last actual override
of this hook, does it no more: web_contents_view_mac.mm|h.

[1] https://crrev.com/c/2092096

BUG=1051155
R=avi@chromium.org

Change-Id: Ieeb5862e9c467343d5844c156a1be40e5a205103
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2093535Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: Antonio Gomes <tonikitoo@igalia.com>
Cr-Commit-Position: refs/heads/master@{#748854}

The handling of iframe load events is finicky as indicated in the open
WHATWG issue:

https://github.com/whatwg/html/issues/490

This patch switches the wait mechanism to one that is well behaved
across browsers.  The new process arms the listener before inserting the
iframe into the document.

Bug: 1059963
Change-Id: I372d4fdae3b270699aedf2c5a4de4429e62f42a5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2096782Reviewed-by: Yi Gu <yigu@chromium.org>
Commit-Queue: Kevin Ellis <kevers@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748853}

This returns the TaskSignal that was passed to scheduler.postTask()
when the current task was scheduled in the general case. When outside
of a posted task or when a custom priority was provided to postTask(),
it creates an implicit TaskSignal encoding the relevant state for the
posted task.

Bug: 979017
Change-Id: I9f960fb76b2500dbc851971e2d9bd2c0f9c51567
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2079386
Commit-Queue: Nate Chapin <japhet@chromium.org>
Auto-Submit: Nate Chapin <japhet@chromium.org>
Reviewed-by: Scott Haseley <shaseley@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748852}

Bug: 1044139
Change-Id: I2a9e1c0fa87ff6525437a44a30bf0d1c7cf1a8e4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095469
Commit-Queue: Dan H <harringtond@chromium.org>
Reviewed-by: Ian Wells <iwells@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748851}

The variable here is an `int64_t`. On arm, this should be `%lld`. PRId64
solves these things.

Bug: 1058218
Change-Id: Ic31d842d345e5fae667dbedc739bbd132707bc58
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095941Reviewed-by: Ahmed Fakhry <afakhry@chromium.org>
Commit-Queue: George Burgess <gbiv@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748850}

This is required by win-asan which has the following GN config:
dcheck_is_configurable = true
is_chrome_branded = true
is_debug = false
is_official_build = true
target_cpu = "x86"

R=thakis@chromium.org

Bug: 1057995
Change-Id: I337e214f2194d3533ea329c22fd240dddae50e5a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2090136
Commit-Queue: Gabriel Charette <gab@chromium.org>
Auto-Submit: Gabriel Charette <gab@chromium.org>
Reviewed-by: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748849}

Roll ChromeOS orderfile from 82-4044.31-1583145468-benchmark-82.0.4077.0-r1 to 82-4044.31-1583145468-benchmark-82.0.4079.0-r1

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/cros-orderfile-chromium
Please CC c-compiler-chrome@google.com on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

Tbr: c-compiler-chrome@google.com
Change-Id: I0bfabf7dc538704c9fe007ae0ec71bf2190a338d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2094736Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Reviewed-by: Bob Haarman <inglorion@chromium.org>
Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#748848}

Bug: 1050101
Change-Id: Iee1a006708469c2f28402711a7e22cb4ad7932f9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2096102Reviewed-by: Tarun Bansal <tbansal@chromium.org>
Commit-Queue: Sophie Chang <sophiechang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748847}

TBR=govind@chromium.org

Change-Id: I4744ef8c8547c926dce14b749bfb8d21a2acb45d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2097017Reviewed-by: Ben Mason <benmason@chromium.org>
Reviewed-by: Krishna Govind <govind@chromium.org>
Commit-Queue: Krishna Govind <govind@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748846}

The crash in gpu::ImageReaderGLOwner::UpdateTexImage() is because
ImageReaderGLOwner only allows one image ref, SharedImageVideo doesn't
release the image ref when read access is finished. It releases the ref
when the SharedImageVideo is destroyed. Fix the problem by releasing
the image ref in EndReadAccess().

Bug: 1059461
Change-Id: I4ccc509bd48dfd6bb576212df32bd411f521e281
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2096777Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Reviewed-by: Khushal <khushalsagar@chromium.org>
Commit-Queue: Peng Huang <penghuang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748845}

Flaky on the KitKat Phone Tester (dbg) bot.
Disabling only for KitKat (i.e. less than L).

BUG=1060244
TBR=arbesser@chromium.org,hluca@chromium.org

Change-Id: I3ba42fc53e92ef33aac352275331f148b64dff8e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2096955
Commit-Queue: Tim Volodine <timvolodine@chromium.org>
Reviewed-by: Tim Volodine <timvolodine@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748844}

https://quiche.googlesource.com/quiche.git/+log/a66f68433ddd..ee0079aeee37

$ git log a66f68433..ee0079aee --date=short --no-merges --format='%ad %ae %s'
2020-03-10 bnc gfe-relnote: Remove DUPLICATE_PUSH frame.  Protected by gfe2_reloadable_flag_quic_enable_version_draft_25_v3 and gfe2_reloadable_flag_quic_enable_version_draft_27.
2020-03-09 wub gfe-relnote: In QUIC batch writers, always drop buffered packets on write errors. Protected by --gfe2_restart_flag_quic_batch_writer_always_drop_packets_on_error.
2020-03-09 renjietang Allow STOP_SENDING frame to be the first frame of a stream.
2020-03-09 fayang gfe-relnote: In QUIC, skip packet threshold loss detection if largest acked is a runt. Protected by gfe2_reloadable_flag_quic_skip_packet_threshold_loss_detection_with_runt.
2020-03-09 fayang gfe-relnote: In QUIC, truncate IETF ACK ranges if ACK frame does not fit. Protected by existing gfe2_reloadable_flag_quic_enable_version_draft_25_v3.
2020-03-09 dschinazi Make QuicToyServer support all IETF drafts
2020-03-09 ianswett gfe-relnote: Deprecate gfe2_reloadable_flag_quic_one_immediate_ack.
2020-03-06 wub gfe-relnote: In QUIC BBRv2, add ack height into PROBE_UP's queuing_threshold. Protected by --gfe2_reloadable_flag_quic_bbr2_add_ack_height_to_queueing_threshold.

Created with:
  roll-dep src/net/third_party/quiche/src src/third_party/quic_trace/src

R=bnc@chromium.org

Change-Id: I7ca6e18e2f3546d2bdd767988c4b5417df4df7df
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2097015Reviewed-by: Bence Béky <bnc@chromium.org>
Commit-Queue: Ian Swett <ianswett@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748843}

We want to treat file: and content: schemes as "null" for
onPostMessage(), this would inconsistent with window.origin result
if setAllowFileAccessFileUrls() set to true, but that is a more general
problem in Blink.

Bug: 918065
Change-Id: If639e3a8be42384258b45e787403f456dca6b3fd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2090920Reviewed-by: Changwan Ryu <changwan@chromium.org>
Reviewed-by: Richard Coles <torne@chromium.org>
Commit-Queue: Shimi Zhang <ctzsm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748842}

This CL was reverted due to TSAN failures. These were fixed by enabling
our Finch flags earlier.

Original CL description:
For users who have enhanced protection enabled, and are not incognito,
attach an OAuth access token to download protection pings to improve
the quality of verdicts.

Bug: 1051679
Change-Id: I5db16b335730063e0b00662b014e9dbfeaf2b8a5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2094871
Auto-Submit: Daniel Rubery <drubery@chromium.org>
Reviewed-by: Xinghui Lu <xinghuilu@chromium.org>
Commit-Queue: Xinghui Lu <xinghuilu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748841}

Implement a two-up view toggle button in PDF Viewer's zoom toolbar.
This button is available when feature "pdf-two-up-view" is enabled.

Bug: 51472
Change-Id: Icf6867e2897f1090b6b9fd8e9494006f648ffdd8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2012837Reviewed-by: Lei Zhang <thestig@chromium.org>
Reviewed-by: Karan Bhatia <karandeepb@chromium.org>
Reviewed-by: Rebekah Potter <rbpotter@chromium.org>
Commit-Queue: Hui Yingst <nigi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748840}

This class no longer has any affinity with LayoutSVGResourceFilter, so
move to another location (near the referencing class
SVGElementResourceClient).

Bug: 109224
Change-Id: I3a18d8df92e1b49b9f643309101515460ac0f7c9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2096742Reviewed-by: Stephen Chenney <schenney@chromium.org>
Commit-Queue: Fredrik Söderquist <fs@opera.com>
Cr-Commit-Position: refs/heads/master@{#748839}

This is a follow-up fix to address comments from https://crrev.com/c/2042302

Bug: 1051687
Change-Id: Id6bc1ed057b85d8ed66b42378825a47ce7775170
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2071360Reviewed-by: Peter Marshall <petermarshall@chromium.org>
Commit-Queue: Yury Semikhatsky <yurys@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748838}

Bug: 1046181
Change-Id: I79b993192750684fb24516bf53cb3bf79d0c8b4c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2021196
Auto-Submit: bttk <bttk@chromium.org>
Commit-Queue: Alexei Svitkine <asvitkine@chromium.org>
Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
Reviewed-by: Gang Wu <gangwu@chromium.org>
Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748837}

This CL adds testcases for border image, mask image, and image replaced
content.

Bug: 1059427
Change-Id: Ib46ee51df7024273d1c3cd548ae08bc92038eafa
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095916
Commit-Queue: Nicolás Peña Moreno <npm@chromium.org>
Reviewed-by: Annie Sullivan <sullivan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748836}

Bug: 1031293
Change-Id: I4858a8aa5eb37c3ce5268752efd87c80fcad43cb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095846Reviewed-by: Sorin Jianu <sorin@chromium.org>
Reviewed-by: Joshua Pawlicki <waffles@chromium.org>
Commit-Queue: Michael Chang <donchan@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#748835}

Bug: 1034046
Change-Id: Iaca83f31e2b91f928faa4454843710bcb419fba5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095610
Auto-Submit: Oliver Li <olivierli@google.com>
Commit-Queue: Oliver Li <olivierli@google.com>
Commit-Queue: François Doray <fdoray@chromium.org>
Reviewed-by: François Doray <fdoray@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748834}

This includes showing attributions and applying styles changes such as
text shadows.

Bug: 1032328
Change-Id: If28b2d98887baa802ae88bde68746100fabe2b44
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2090729
Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org>
Reviewed-by: Alex Gough <ajgo@chromium.org>
Reviewed-by: Esmael Elmoslimany <aee@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748833}

Adds a new private class which holds all the members for the tab helper
that need to be reset on every page load.
The encapsulating class is always initialized to keep the diff and logic
change to a minimum.

Bug: 1023483
Change-Id: I1c91ce7eb3d65937e35fcbd949c8d81907adc8f8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2097035
Commit-Queue: Robert Ogden <robertogden@chromium.org>
Reviewed-by: Ryan Sturm <ryansturm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748832}

Add tests stable on Fuchsia FYI bots to more Fuchsia bots.

Bug: 1054545,1054240
Change-Id: Ic74b7f340b36eed84ef1298b06b22b89cdecbae4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095848Reviewed-by: John Budorick <jbudorick@chromium.org>
Commit-Queue: Chong Gu <chonggu@google.com>
Cr-Commit-Position: refs/heads/master@{#748831}