GitLab

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/d689ef2e..f47962ef

Please follow these instructions for assigning/CC'ing issues:
https://github.com/v8/v8/wiki/Triaging%20issues

Please close rolling in case of a roll revert:
https://v8-roll.appspot.com/
This only works with a Google account.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

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

Review-Url: https://codereview.chromium.org/2606973002
Cr-Commit-Position: refs/heads/master@{#440896}

BUG=b/33754182
TEST=Manually on device
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

TBR=oshima@chromium.org

Review-Url: https://codereview.chromium.org/2604523002
Cr-Commit-Position: refs/heads/master@{#440895}

R=dcheng@chromium.org
BUG=567770
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
NOTRY=true
NO_DEPENDENCY_CHECKS=true

Review-Url: https://codereview.chromium.org/2609453002
Cr-Commit-Position: refs/heads/master@{#440894}

(1) Add a boolean |restrict_to_one_preconnect_for_proxies| to network
session params. This allows us to run multiple experiments using a single
field trial. Next CL will add the variation param
|race_preconnects_to_proxies| to experiment with racing the alternative
and main jobs for proxy preconnects.

(2) Remove an extra parameter in
HttpStreamFactoryImpl::Job::Delegate::OnStreamReady() method.

BUG=667471,671291
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester

Review-Url: https://codereview.chromium.org/2600943002
Cr-Commit-Position: refs/heads/master@{#440893}

BUG=670843
R=qyearsley@chromium.org

Review-Url: https://codereview.chromium.org/2604853002
Cr-Commit-Position: refs/heads/master@{#440892}

BUG=648554, 648557, 673362

Review-Url: https://codereview.chromium.org/2576563002
Cr-Commit-Position: refs/heads/master@{#440891}

Creates an object to perform PersistedLogs metrics recording, so that
they can be overriden in the UKM implementation.

Also adds a missing histogram to histograms.xml

BUG=

Review-Url: https://codereview.chromium.org/2588873002
Cr-Commit-Position: refs/heads/master@{#440890}

BUG=677239
TEST=See failing tests in 677239 without this change.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2601963002
Cr-Commit-Position: refs/heads/master@{#440889}

https://pdfium.googlesource.com/pdfium.git/+log/04a407093c13..d0bbccdd3211

$ git log 04a407093..d0bbccdd3 --date=short --no-merges --format='%ad %ae %s'
2016-12-28 tsepez Make CFX_Observable copy constructable.

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

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls

TBR=dsinclair@chromium.org

Review-Url: https://codereview.chromium.org/2605913003
Cr-Commit-Position: refs/heads/master@{#440888}

This is a follow-up fix to a mistake in http://crrev.com/2590693002.

BUG=662584

Review-Url: https://codereview.chromium.org/2605933002
Cr-Commit-Position: refs/heads/master@{#440887}

Also fixes bugs exposed by those tests:
- when database does not exists it needs to be created.
- when database can't be opened null is passed to LevelDBWrapper.

BUG=586194

Review-Url: https://codereview.chromium.org/2597983003
Cr-Commit-Position: refs/heads/master@{#440886}

https://skia.googlesource.com/skia.git/+log/3f375d055a9b..7cf774573cc5

$ git log 3f375d055..7cf774573 --date=short --no-merges --format='%ad %ae %s'
2016-12-28 kjlubick update pixel and pixelxl to NMF26Q

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

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
TBR=liyuqian@google.com

Review-Url: https://codereview.chromium.org/2608523002
Cr-Commit-Position: refs/heads/master@{#440885}

Due to threading issues, we'd already changed several components
of the GL command buffer to dump their memory heirarchically through
a single registration point in ContextProviderCommandBuffer.

MappedMemoryManager was missed. Modifying it so it doesn't register
to dump directly, but is dumped through ContextProviderCommandBuffer >
GLES2Implementation > MappedMemoryManager.

R=ssid
BUG=675821
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2592803002
Cr-Commit-Position: refs/heads/master@{#440884}

InspectorBackendClass => Protocol.InspectorBackend
InspectorBackend => Protocol.inspectorBackend

BUG=none

Review-Url: https://codereview.chromium.org/2600323002
Cr-Commit-Position: refs/heads/master@{#440883}

Adds 6 new UMA metrics which log GL context memory. Memory is logged separately for WebGL and GLES contexts. Additionally, logging occurs at three points - periodically (every 1s), at shutdown, and when the GPU process gets a CRITICAL memory pressure signal. The new histograms are:

GPU.ContextMemory.WebGL.Periodic
GPU.ContextMemory.WebGL.Shutdown
GPU.ContextMemory.WebGL.Pressure
GPU.ContextMemory.GLES.Periodic
GPU.ContextMemory.GLES.Shutdown
GPU.ContextMemory.GLES.Pressure

BUG=667013
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2577843002
Cr-Commit-Position: refs/heads/master@{#440882}

The split will allow generation of edits on multiple configs (e.g. linux
vs windows OR rel vs dbg) and merging the edits before applying them once:
    $ tools/clang/scripts/run_tool.py rewrite_to_chrome_style \
        --generate-compdb --all out/rel >run_tool.linux.rel.out
    $ ...
    $ cat run_tool.*.out \
        | tools/clang/scripts/extract_edits.py \
        | tools/clang/scripts/apply_edits.py
        --generate-compdb --all out/rel >run_tool.linux.rel.out

Test steps:
- tools/clang/translation_unit/test_translation_unit.py
- tools/clang/scripts/test_tool.py rewrite_to_chrome_style
- manually running run_tool | extract_edits | apply_edits pipeline
  on WTF and verifying that it still builds after the rename

BUG=598138
TEST=See "Test steps" above.

Review-Url: https://codereview.chromium.org/2599193002
Cr-Commit-Position: refs/heads/master@{#440881}

BUG=672545
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2606963002
Cr-Commit-Position: refs/heads/master@{#440880}

BUG=672902

Review-Url: https://codereview.chromium.org/2601063002
Cr-Commit-Position: refs/heads/master@{#440879}

BUG=677166

Review-Url: https://codereview.chromium.org/2608443002
Cr-Commit-Position: refs/heads/master@{#440878}

This is a follow-up to https://crrev.com/2028473003, and lands
a missed part of server side change 126730342.

BUG=488484

Review-Url: https://codereview.chromium.org/2602883002
Cr-Commit-Position: refs/heads/master@{#440877}

https://skia.googlesource.com/skia.git/+log/e2d414167980..3f375d055a9b

$ git log e2d414167..3f375d055 --date=short --no-merges --format='%ad %ae %s'
2016-12-28 brianosman Add skia_enable_discrete_gpu argument to GN

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

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
TBR=liyuqian@google.com

Review-Url: https://codereview.chromium.org/2601073002
Cr-Commit-Position: refs/heads/master@{#440876}

This CL packages the fairly common code path where a gfx image is
extracted from the shared bundle and converted into a UIImage into a
single utility function. All of the applicable call sites in
ios/chrome/browser/ui/ are updated to use the function, and
includes that were only present to support this code are removed.

BUG=

Review-Url: https://codereview.chromium.org/2602903002
Cr-Commit-Position: refs/heads/master@{#440875}

- Introduced base class DevToolsDomainHandler.
- Agent host registers handlers in DevToolsSession.
- Passing DevToolsSession instance around.

This is to:
- reduce boilerplate;
- prepare for multi-client (there will be multiple sessions).

BUG=590878
TBR=pfeldman

Review-Url: https://codereview.chromium.org/2590293003
Cr-Commit-Position: refs/heads/master@{#440874}

BUG=579229

Review-Url: https://codereview.chromium.org/2609443002
Cr-Commit-Position: refs/heads/master@{#440873}

BUG=579229

Review-Url: https://codereview.chromium.org/2604913002
Cr-Commit-Position: refs/heads/master@{#440872}

* Add |http2_settings| field trial parameter to HTTP/2 trial group.
* Set max header table size, max concurrent pushed streams, and max
  receiving window size values in |http2_settings| in
  HttpNetworkSession.
* Pass |http2_settings| from HttpNetworkSession to SpdySessionPool to
  SpdySession.  Remove |stream_max_recv_window_size| from this path.
* Extract max header table size, max concurrent pushed streams, and max
  receiving window size values from |http2_settings| in SpdySession.
* Move max header table size and max concurrent pushed streams default
  value from SpdySession to HttpNetworkSession.
* Add |http2_settings| to SpdySessionDependencies; remove
  |stream_max_recv_window_size|.

BUG=622737

Review-Url: https://codereview.chromium.org/2600973002
Cr-Commit-Position: refs/heads/master@{#440871}

This lands server side change 130276296 by jamessynge.

BUG=488484

Review-Url: https://codereview.chromium.org/2600273002
Cr-Commit-Position: refs/heads/master@{#440870}

BUG=676746
TBR=dbeam@chromium.org

Review-Url: https://codereview.chromium.org/2606943002
Cr-Commit-Position: refs/heads/master@{#440869}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/ea41bdf0..d689ef2e

Please follow these instructions for assigning/CC'ing issues:
https://github.com/v8/v8/wiki/Triaging%20issues

Please close rolling in case of a roll revert:
https://v8-roll.appspot.com/
This only works with a Google account.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

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

Review-Url: https://codereview.chromium.org/2600423002
Cr-Commit-Position: refs/heads/master@{#440868}

List of import -> include
+#include "base/logging.h"
+#include "base/memory/ref_counted.h"
+#include "ios/testing/earl_grey/disabled_test_macros.h"
+#include "ios/web/history_state_util.h"
+#include "ios/web/navigation/navigation_manager_facade_delegate.h"
+#include "ios/web/public/active_state_manager.h"
+#include "ios/web/public/interstitials/web_interstitial.h"
+#include "ios/web/public/web_state/js/credential_util.h"
+#include "ios/web/public/web_state/url_verification_constants.h"
+#include "ios/web/public/web_state/web_state_delegate.h"
+#include "ios/web/public/web_state/web_state_observer.h"
+#include "ios/web/webui/web_ui_ios_controller_factory_registry.h"
+#include "ios/web/webui/web_ui_ios_impl.h"
+#include "mojo/public/cpp/system/watcher.h"

List of include -> import
+#import "base/ios/block_types.h"
+#import "base/ios/weak_nsobject.h"
+#import "base/mac/bind_objc_block.h"
+#import "base/mac/objc_property_releaser.h"
+#import "base/mac/scoped_nsobject.h"
+#import "base/test/ios/wait_util.h"
+#import "ios/net/clients/crn_forwarding_network_client.h"
+#import "ios/net/cookies/cookie_store_ios.h"
+#import "ios/net/request_tracker.h"
+#import "ios/testing/wait_util.h"
+#import "ios/web/interstitials/native_web_interstitial_impl.h"
+#import "ios/web/interstitials/web_interstitial_impl.h"
+#import "ios/web/navigation/crw_session_controller.h"
+#import "ios/web/navigation/crw_session_entry.h"
+#import "ios/web/navigation/navigation_item_impl.h"
+#import "ios/web/navigation/navigation_manager_delegate.h"
+#import "ios/web/navigation/navigation_manager_impl.h"
+#import "ios/web/navigation/nscoder_util.h"
+#import "ios/web/net/clients/crw_js_injection_network_client.h"
+#import "ios/web/net/clients/crw_redirect_network_client.h"
+#import "ios/web/net/clients/crw_redirect_network_client_factory.h"
+#import "ios/web/net/cookie_notification_bridge.h"
+#import "ios/web/net/crw_cert_verification_controller.h"
+#import "ios/web/net/request_group_util.h"
+#import "ios/web/net/request_tracker_factory_impl.h"
+#import "ios/web/net/request_tracker_impl.h"
+#import "ios/web/net/web_http_protocol_handler_delegate.h"
+#import "ios/web/public/block_types.h"
+#import "ios/web/public/image_fetcher/webp_decoder.h"
+#import "ios/web/public/interstitials/web_interstitial_delegate.h"
+#import "ios/web/public/navigation_item.h"
+#import "ios/web/public/test/response_providers/data_response_provider.h"
+#import "ios/web/public/test/response_providers/delayed_response_provider.h"
+#import "ios/web/public/test/response_providers/error_page_response_provider.h"
+#import "ios/web/public/test/response_providers/file_based_response_provider.h"
+#import "ios/web/public/test/response_providers/file_based_response_provider_impl.h"
+#import "ios/web/public/test/response_providers/html_response_provider.h"
+#import "ios/web/public/test/response_providers/html_response_provider_impl.h"
+#import "ios/web/public/test/response_providers/response_provider.h"
+#import "ios/web/public/test/response_providers/string_response_provider.h"
+#import "ios/web/public/test/test_native_content.h"
+#import "ios/web/public/test/test_native_content_provider.h"
+#import "ios/web/public/test/test_navigation_manager.h"
+#import "ios/web/public/test/test_web_state.h"
+#import "ios/web/public/test/test_web_view_content_view.h"
+#import "ios/web/public/web_client.h"
+#import "ios/web/public/web_state/context_menu_params.h"
+#import "ios/web/public/web_state/crw_web_view_proxy.h"
+#import "ios/web/public/web_state/ui/crw_content_view.h"
+#import "ios/web/public/web_state/ui/crw_web_view_content_view.h"
+#import "ios/web/public/web_state/web_state.h"
+#import "ios/web/public/web_state/web_state_policy_decider.h"
+#import "ios/web/public/web_state/web_state_user_data.h"
+#import "ios/web/public/web_view_creation_util.h"
+#import "ios/web/shell/shell_web_client.h"
+#import "ios/web/web_state/blocked_popup_info.h"
+#import "ios/web/web_state/web_state_impl.h"
+#import "ios/web/webui/crw_web_ui_page_builder.h"
+#import "ios/web/webui/url_fetcher_block_adapter.h"
+#import "testing/gtest_mac.h"
+#import "ui/base/test/ios/ui_view_test_utils.h"

BUG=None

Review-Url: https://codereview.chromium.org/2605913002
Cr-Commit-Position: refs/heads/master@{#440867}

Merge internal change 143035999.

BUG=

Review-Url: https://codereview.chromium.org/2603843002
Cr-Commit-Position: refs/heads/master@{#440866}

This CL includes:
 1.Creating, initializing and updating PasswordReuseDetector instance as a member of PasswordStore.
 2.Introducing all flows for checking password reuse with CheckReuse() of ChromePasswordManagerClient and receiving response.

ChromePasswordManagerClient.CheckReuse() is not called now, it will be implemented in next CLs.

BUG=657041

Review-Url: https://codereview.chromium.org/2585253002
Cr-Commit-Position: refs/heads/master@{#440865}

There are reports of random Chrome crash due to HW encode accelerator
on Android.
Possible reason may be: currently we'll reset client factory at
encoding error and notify client. But before client stops encoding
(calling Destroy() here), if there is any pending frame delivered to
encoder, it will fail to get the pointer of client factory.
Here we try to leave the client factory life managment to client and
only notify the encoding error to client in error handling.

BUG=676987
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2601003002
Cr-Commit-Position: refs/heads/master@{#440864}

https://skia.googlesource.com/skia.git/+log/29437eb5c782..e2d414167980

$ git log 29437eb5c..e2d414167 --date=short --no-merges --format='%ad %ae %s'
2016-12-28 kjlubick Update Nexus 5 to M4B30Z

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

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
TBR=liyuqian@google.com

Review-Url: https://codereview.chromium.org/2606883003
Cr-Commit-Position: refs/heads/master@{#440863}

Historically, we have dropped these when navigations are interleaved.

BUG=656919

Review-Url: https://codereview.chromium.org/2563423004
Cr-Commit-Position: refs/heads/master@{#440862}

Initially this callback is used to delay writing the schema version until data
actually gets committed. In follow-up CLs this will also be used to store
per-origin metadata.

BUG=586194

Review-Url: https://codereview.chromium.org/2597353002
Cr-Commit-Position: refs/heads/master@{#440861}

(And add a TODO for PPAPI download attribution)

BUG=676691

Review-Url: https://codereview.chromium.org/2606843002
Cr-Commit-Position: refs/heads/master@{#440860}

If the static HTML contains iframes, a call to
decidePolicyForNavigationAction is done with a navigationType other.
This should not trigger a navigation to URLLoader.

BUG=676575

Review-Url: https://codereview.chromium.org/2597133002
Cr-Commit-Position: refs/heads/master@{#440859}

BUG=None

Review-Url: https://codereview.chromium.org/2604843002
Cr-Commit-Position: refs/heads/master@{#440858}

Currently an invalid version just results in not using a backing store,
but in the future that will result in upgrading and/or recreating the
database.

BUG=586194

Review-Url: https://codereview.chromium.org/2607493003
Cr-Commit-Position: refs/heads/master@{#440857}