GitLab

We keep a list of all pending frames insize the GLSurface. Each frame contains
a list of overlays. In the future it can contain overlay sync point info too.
When ready, we set the ready state and submit all completed frames in order

BUG=470185
TEST=link and rockchip

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

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

BUG=421063

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

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

BUG=474410

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

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

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/888a881c..c817f698

Please follow these instructions for assigning/CC'ing issues:
https://code.google.com/p/v8-wiki/wiki/TriagingIssues

TBR=hablich@chromium.org,machenbach@chromium.org,yangguo@chromium.org,vogelheim@chromium.org

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

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

Remove unused constant and dead code in these files.

BUG=473606

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

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

BUG=471242

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

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

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

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

Revert of Update OTS to revision 6d2e08b (patchset #1 id:1 of https://codereview.chromium.org/1062093002/)

Reason for revert:
This CL caused many tests fail on uninit read errors reported by MSan

BUG=474597

Original issue's description:
> Update OTS to revision 6d2e08b
>
> This brings in a bugfix of checksum calculation that caused assertion
> failure on debug builds.
>
> BUG=464780
>
> Committed: https://crrev.com/3a672fe050438bc787af3f9bb652048291558c6c
> Cr-Commit-Position: refs/heads/master@{#324023}

TBR=bashi@chromium.org,ksakamoto@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=464780

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

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

Applied to the old NTP and the fast NTP. Otherwise server-injected favicon
that's bigger than 16x16 will overflow.

BUG=474102

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

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

SetBufferAvailableState was from called DataFetcherSharedMemory::Stop for all
cases including TYPE_LIGHT, but for light, we are not using it as
we are using SetLightBuffer(light_buffer_, -1);

BUG=474586

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

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

BUG=470844

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

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

This test passes locally, but fails on the buildbot. Not sure of the reason,
disabling the test for now.

TBR=mef@chromium.org

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

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

BUG=465815

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

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

This CL adds a method to list available volumes. It's available only to kiosk
apps running in kiosk session and to some whitelited component apps/extensions.

TBR=fukino@chromium.org  # chrome_extensions.js reviewed at cr/90486340.
TEST=browser_tests: *KioskTest*GetVolumeList*,
     *FileSystemApiTestForRequestFileSystem*GetVolumeList*
BUG=440674

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/b679ca8..6cad1da

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

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

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

The page uses a big parsing blocking script and is thus interesting for the
BigJs benchmark.

R=skyostil@chromium.org
BUG=

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

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

For better flow control, call AccumulateMidiBytesSent() after
send operation finished.

BUG=467442

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

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

Use one umbrella command line switch.

BUG=469984,462445
R=nkostylev@chromium.org

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

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/ff271c2..b679ca8

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

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

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

These refactor these two functions into a single one to ease the
transition of https://codereview.chromium.org/920443003.

Tested on Android (with Nexus 5 device) with the following command lines:

$ ./build/gyp_chromium -DOS=android -Goutput_dir=out_android
$ ninja -C out_android/Debug content_unittests content_unittests_apk
$ export CHROMIUM_OUT_DIR=out_android
$ build/android/test_runner.py gtest -s content_unittests --debug -vvv

ContentShell tests:
$ ninja -C out_android/Debug content_shell_apk
$ ninja -C out_android/Debug content_shell_test_apk
$ build/android/adb_install_apk.py --apk=ContentShell.apk --debug
$ build/android/test_runner.py instrumentation --test-apk=ContentShellTest --isolate-file-path content/content_shell_test_apk.isolate --debug -vv

AndroidWebView tests:
$ ninja -C out_android/Debug android_webview_apk
$ ninja -C out_android/Debug android_webview_test_apk
$ build/android/adb_install_apk.py --apk=AndroidWebView.apk --debug
$ build/android/test_runner.py instrumentation --test-apk=AndroidWebViewTest --test_data webview:android_webview/test/data/device_files --debug -vvv

BUG=338338
TEST=see above
R=boliu@chromium.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/d073ccb0..888a881c

Please follow these instructions for assigning/CC'ing issues:
https://code.google.com/p/v8-wiki/wiki/TriagingIssues

TBR=hablich@chromium.org,machenbach@chromium.org,yangguo@chromium.org,vogelheim@chromium.org

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

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

Current implementation monitors DBT_DEVICEARRIVAL mesage, but that
is not a right message to monitor device changes since actual device
list that OS holds will be updated a little later. Instead, monitor
DBT_DEVNODES_CHANGES that is delivered after the list being updated.

In Chrome, SystemMessageWindowWin invokes base::SystemMonitor's observer
with DEVTYPE_UNKNOWN for DBT_DEVNODES_CHANGED, and the SystemMonitor
provides a way to observe the event on the registered thread.
Using the monitor must be the right implementation.

BUG=472980

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

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

WebRTC 8926:8931
Changes: https://chromium.googlesource.com/external/webrtc/trunk/webrtc.git/+log/5727038..6f9fcb2

Libjingle 8923:8930
Changes: https://chromium.googlesource.com/external/webrtc/trunk/talk.git/+log/b2811ad..e9dbaf8

TBR=tommi

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

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

BUG=474451

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

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

Fixes downstream gn bots affected by https://codereview.chromium.org/1063873002

TBR=brettw@chromium.org
BUG=474506

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

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

These changes to RenderFrameHostManager make it so that during a
navigation a new SiteInstance won't be created for a URL if a previously
created one still exists for that same URL. This also follows more
closely how the current navigation implementation works.

BUG=376094
TEST=WebContentsImplTest.ActiveContentsCountChangeBrowsingInstance

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

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

BUG=455409

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

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

BUG=371290
TBR=piman

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

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

This CL notifies the hotwording extension of a microphone state change
by adding a hotwordPrivate.onMicrophoneStateChanged event which triggers
an update of the hotwording state.

It also moves the media device observation from HotwordServiceFactory to
HotwordService.

BUG=461787
TBR=estade@chromium.org

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

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

This brings in a bugfix of checksum calculation that caused assertion
failure on debug builds.

BUG=464780

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/aeab2ed..44fc33d

TBR=falken@chromium.org

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

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

Also added myself to AUTHORS

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

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

Bookmark apps no longer toggle whether they support the location bar
via the browser features system, which didn't seem correct. Instead they
support the location bar feature always but change how the location
bar's height to bring it into and out of view.

This change also introduces a BookmarkAppBrowserController class to
encapsulate all the logic for bookmark apps to modify the browser UI.

BUG=463405

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/55028d3e..aeab2ed

TBR=jsbell@chromium.org,falken@chromium.org

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

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

This CL adds a pref that tracks when the app list is last launched and
doesn't warm up the app list on Windows unless the app list has been
launched in the last 28 days.

BUG=440224

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

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

* When we changed to OmniboxViewViews, we broke the behavior that the omnibox
could handle the escape key before it was processed as a "stop" accelerator.
Instead, Chrome trunk current processes escape as stop first (and exclusively),
then if/when there is no load to stop passes it through to the omnibox.  Restore
the other order, though not precisely with the old behavior.  Other than when
arrowing through the popup (in which cases escape just resets to the top entry):
** Escape when not editing (including when zerosuggest is open) will always both
reset the omnibox and stop any underlying load.
** Otherwise, escape will reset the omnibox but not stop any load.

* As part of the above fix, we now ensure that escape will always close any open
zerosuggest window regardless of which entry in it might be selected or what the
omnibox caret/selection currently is.

* When unfocusing the omnibox causes a zerosuggest popup to close, we also force
the omnibox to reset.  This ensures that, if zerosuggest caused us to
temporarily avoid displaying updates to the underlying page URL, they will now
be shown.  When combined with the changes above, this should mean that it's
impossible to leave zerosuggest mode but still be stuck seeing an out-of-date
URL.

* When copying the text of any item in the zerosuggest popup, including the
first one, and regardless of whether the visible URL is up-to-date, ensure that
the clipboard actually gets the correct text.  Previously, select-all + copy on
any zerosuggest item would copy whatever the current underlying URL of the page
was.

BUG=158913,423181,466565
TEST=See bugs

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

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

Revert of Fix scoped_ptr free to use delete [] instead of delete. (patchset #1 id:1 of https://codereview.chromium.org/1055433003/)

Reason for revert:
It looks like this broke the following test on XP Tests (1):
sbox_unittests WinUtils.SameObject

Example output:
WinUtils.SameObject (run #1):
[ RUN      ] WinUtils.SameObject
c:\b\build\slave\win_builder\build\src\sandbox\win\src\win_utils_unittest.cc(78): error: Value of: SameObject(file.Get(), file_name_nt1.c_str())
  Actual: false
Expected: true
c:\b\build\slave\win_builder\build\src\sandbox\win\src\win_utils_unittest.cc(79): error: Value of: SameObject(file.Get(), file_name_nt2.c_str())
  Actual: false
Expected: true
[  FAILED  ] WinUtils.SameObject (16 ms)

Original issue's description:
> Fix scoped_ptr free to use delete [] instead of delete.
>
> BUG=101717
>
> Committed: https://crrev.com/7804b679be3af7e24578394af0a85ba1249344a9
> Cr-Commit-Position: refs/heads/master@{#324013}

TBR=cpu@chromium.org,wfh@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=101717

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/61c4468..55028d3e

TBR=jsbell@chromium.org,falken@chromium.org

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

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

The icon appears at the right of the top row of icons.

BUG=469996

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

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