New issue
Advanced search Search tips

Issue 616244 link

Starred by 4 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 3
Type: Task

Blocked on: View detail
issue 620147
issue 620438
issue 620932
issue 779224
issue 831683
issue 616247
issue 616646
issue 619076
issue 619617
issue 619659
issue 619687
issue 619783
issue 619984
issue 619990
issue 619991
issue 619992
issue 620045
issue 620083
issue 620089
issue 620134
issue 620139
issue 620143
issue 620443
issue 620465
issue 620480
issue 620489
issue 620884
issue 620887
issue 620907
issue 620939
issue 621250
issue 621692
issue 622072
issue 622485
issue 622967
issue 623125
issue 674991
issue 675001
issue 675005
issue 732525

Blocking:
issue 616288



Sign in to add a comment

☂ Use only public web// API in ios/chrome

Project Member Reported by eugene...@chromium.org, May 31 2016

Issue description

ios_internal/ must not use ios/web, only ios/web/public
 
Showing comments 5 - 104 of 104 Older
Project Member

Comment 5 by bugdroid1@chromium.org, Jun 2 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/3c796864f9d02fcc9aec1684fc0bd9b61869ef06

commit 3c796864f9d02fcc9aec1684fc0bd9b61869ef06
Author: eugenebut <eugenebut@google.com>
Date: Thu Jun 02 05:53:53 2016

Project Member

Comment 6 by bugdroid1@chromium.org, Jun 3 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2924a66000830f14055b5c087c0861a532c6a7fb

commit 2924a66000830f14055b5c087c0861a532c6a7fb
Author: eugenebut <eugenebut@chromium.org>
Date: Fri Jun 03 16:25:08 2016

[ios] Added WebTestWithWebController::web_state.

Subclasses of WebTestWithWebController should use WebState where
possible.

BUG=616244

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

[modify] https://crrev.com/2924a66000830f14055b5c087c0861a532c6a7fb/ios/web/test/web_test.h
[modify] https://crrev.com/2924a66000830f14055b5c087c0861a532c6a7fb/ios/web/test/web_test.mm

Project Member

Comment 7 by bugdroid1@chromium.org, Jun 6 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/32d389bb6e315b381e52154a6ed139c391afc717

commit 32d389bb6e315b381e52154a6ed139c391afc717
Author: eugenebut <eugenebut@google.com>
Date: Mon Jun 06 21:11:57 2016

Project Member

Comment 8 by bugdroid1@chromium.org, Jun 9 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/0f8ef15b23f8514a415928737b01770803a5218d

commit 0f8ef15b23f8514a415928737b01770803a5218d
Author: eugenebut <eugenebut@google.com>
Date: Thu Jun 09 20:05:03 2016

Project Member

Comment 9 by bugdroid1@chromium.org, Jun 10 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/ca68bf09e8f52d618c7092a55745193d571f0526

commit ca68bf09e8f52d618c7092a55745193d571f0526
Author: eugenebut <eugenebut@google.com>
Date: Fri Jun 10 00:20:50 2016

Project Member

Comment 10 by bugdroid1@chromium.org, Jun 10 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/8d484ac4eb2d934a7f92180353316c4669b84000

commit 8d484ac4eb2d934a7f92180353316c4669b84000
Author: eugenebut <eugenebut@google.com>
Date: Fri Jun 10 00:21:15 2016

Project Member

Comment 11 by bugdroid1@chromium.org, Jun 10 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/092f834e531307d4836b0cb5293ddd94a9565386

commit 092f834e531307d4836b0cb5293ddd94a9565386
Author: eugenebut <eugenebut@google.com>
Date: Fri Jun 10 07:35:57 2016

Project Member

Comment 12 by bugdroid1@chromium.org, Jun 10 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/ef310e5333ee523920d6a7988d374842d1c9d68b

commit ef310e5333ee523920d6a7988d374842d1c9d68b
Author: eugenebut <eugenebut@google.com>
Date: Fri Jun 10 14:15:02 2016

Project Member

Comment 13 by bugdroid1@chromium.org, Jun 10 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/592a7be060916424269e55c96799b9ebedc3aa71

commit 592a7be060916424269e55c96799b9ebedc3aa71
Author: gambard <gambard@google.com>
Date: Fri Jun 10 14:57:16 2016

Project Member

Comment 14 by bugdroid1@chromium.org, Jun 10 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/58dbcfbacb4bf35ec1746bf7d6ed507433a240f7

commit 58dbcfbacb4bf35ec1746bf7d6ed507433a240f7
Author: eugenebut <eugenebut@google.com>
Date: Fri Jun 10 15:33:56 2016

Blockedon: 619076
Project Member

Comment 16 by bugdroid1@chromium.org, Jun 13 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/0cda8182ac979592051f0ed5c458fa107b9344f7

commit 0cda8182ac979592051f0ed5c458fa107b9344f7
Author: eugenebut <eugenebut@google.com>
Date: Mon Jun 13 15:45:13 2016

Blockedon: 619617
Blockedon: 619783
Project Member

Comment 19 by bugdroid1@chromium.org, Jun 14 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/5acecbd23d74c22c66d3d09a3b0c44252bbd133c

commit 5acecbd23d74c22c66d3d09a3b0c44252bbd133c
Author: eugenebut <eugenebut@google.com>
Date: Tue Jun 14 05:01:36 2016

Project Member

Comment 20 by bugdroid1@chromium.org, Jun 14 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/b2b5c833d620a6dfea0dac1cc95cde6ec49e5973

commit b2b5c833d620a6dfea0dac1cc95cde6ec49e5973
Author: eugenebut <eugenebut@google.com>
Date: Tue Jun 14 05:10:56 2016

Project Member

Comment 21 by bugdroid1@chromium.org, Jun 14 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/ac64d7d68d62b5a3c68840b16d9ea92b957fe707

commit ac64d7d68d62b5a3c68840b16d9ea92b957fe707
Author: eugenebut <eugenebut@google.com>
Date: Tue Jun 14 05:12:55 2016

Blockedon: 619971
Blockedon: 619984
Blockedon: 619990
Blockedon: 619991
Blockedon: 619992
Blocking: 616288
Blockedon: -616288
Blockedon: 620045
Blockedon: 620083
Blockedon: 620089
Blockedon: 616646
Blockedon: 620134
Blockedon: 620139
Blockedon: 620143
Blockedon: 620147
Project Member

Comment 37 by bugdroid1@chromium.org, Jun 15 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/f3d39f09882b09d6b675c9751777854639a647c6

commit f3d39f09882b09d6b675c9751777854639a647c6
Author: eugenebut <eugenebut@google.com>
Date: Wed Jun 15 00:28:48 2016

Project Member

Comment 38 by bugdroid1@chromium.org, Jun 15 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/46e3b65ae902918aaac2e8bb2bcdc7881ce851d2

commit 46e3b65ae902918aaac2e8bb2bcdc7881ce851d2
Author: eugenebut <eugenebut@google.com>
Date: Wed Jun 15 00:28:58 2016

Project Member

Comment 39 by bugdroid1@chromium.org, Jun 15 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/2d9405cc6373f2f6221555903bded097d22388a8

commit 2d9405cc6373f2f6221555903bded097d22388a8
Author: eugenebut <eugenebut@google.com>
Date: Wed Jun 15 00:30:55 2016

Project Member

Comment 40 by bugdroid1@chromium.org, Jun 15 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/2912fc975e8b613e2de402d5a5fad4a5a2ce92b6

commit 2912fc975e8b613e2de402d5a5fad4a5a2ce92b6
Author: eugenebut <eugenebut@google.com>
Date: Wed Jun 15 00:31:00 2016

Project Member

Comment 41 by bugdroid1@chromium.org, Jun 15 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/dbd35c592dcaf2aa31dae9012cb6ad9c9f54d84b

commit dbd35c592dcaf2aa31dae9012cb6ad9c9f54d84b
Author: eugenebut <eugenebut@google.com>
Date: Wed Jun 15 18:19:20 2016

Blockedon: 620438
Blockedon: 620443
Blockedon: 620465
Blockedon: 620480
Blockedon: 620489
Project Member

Comment 47 by bugdroid1@chromium.org, Jun 16 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/f862daca4d5f12ff8cecea53970f1391b3ee52d1

commit f862daca4d5f12ff8cecea53970f1391b3ee52d1
Author: eugenebut <eugenebut@google.com>
Date: Thu Jun 16 16:32:08 2016

Blockedon: 620884
Blockedon: 620887
Blockedon: 620907
Blockedon: 620932
Blockedon: 620939
Project Member

Comment 53 by bugdroid1@chromium.org, Jun 17 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/aadc80df7cd4c4c8a74821d5ac3ecf589f532ef9

commit aadc80df7cd4c4c8a74821d5ac3ecf589f532ef9
Author: eugenebut <eugenebut@google.com>
Date: Fri Jun 17 02:41:27 2016

Project Member

Comment 54 by bugdroid1@chromium.org, Jun 17 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/d5a21798dd897232043608a3ad08c88c3e2fe729

commit d5a21798dd897232043608a3ad08c88c3e2fe729
Author: eugenebut <eugenebut@google.com>
Date: Fri Jun 17 07:57:10 2016

Project Member

Comment 55 by bugdroid1@chromium.org, Jun 17 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/a8cdcf8af2d3024238a9e09242317022bdaf0abd

commit a8cdcf8af2d3024238a9e09242317022bdaf0abd
Author: eugenebut <eugenebut@google.com>
Date: Fri Jun 17 18:47:10 2016

Blockedon: 621250
Project Member

Comment 57 by bugdroid1@chromium.org, Jun 20 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/49ac136d75d619662a2d8743e6270ae6d4d69733

commit 49ac136d75d619662a2d8743e6270ae6d4d69733
Author: eugenebut <eugenebut@google.com>
Date: Mon Jun 20 08:14:34 2016

Blockedon: 621692
Project Member

Comment 59 by bugdroid1@chromium.org, Jun 21 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/05d2128f7ba0e732ad68d89d94019dd640de1131

commit 05d2128f7ba0e732ad68d89d94019dd640de1131
Author: eugenebut <eugenebut@google.com>
Date: Tue Jun 21 04:38:06 2016

Project Member

Comment 60 by bugdroid1@chromium.org, Jun 21 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/8265cc87bf69b17b3e098f33873f6d9a3664369d

commit 8265cc87bf69b17b3e098f33873f6d9a3664369d
Author: eugenebut <eugenebut@google.com>
Date: Tue Jun 21 19:32:29 2016

Blockedon: 622072
Project Member

Comment 62 by bugdroid1@chromium.org, Jun 22 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/d007c9ceac3d4c5a77b8c2da80d9ea045112f7dc

commit d007c9ceac3d4c5a77b8c2da80d9ea045112f7dc
Author: eugenebut <eugenebut@google.com>
Date: Wed Jun 22 19:47:45 2016

Blockedon: 622485
Project Member

Comment 64 by bugdroid1@chromium.org, Jun 23 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/c13fe098921aa8f0d59fd05814023826e115b016

commit c13fe098921aa8f0d59fd05814023826e115b016
Author: eugenebut <eugenebut@google.com>
Date: Thu Jun 23 01:09:38 2016

Project Member

Comment 65 by bugdroid1@chromium.org, Jun 23 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/e773debc33b66964918c26aaea68ad6132599cff

commit e773debc33b66964918c26aaea68ad6132599cff
Author: eugenebut <eugenebut@google.com>
Date: Thu Jun 23 15:21:15 2016

Blockedon: 622967
Blockedon: 623125
Project Member

Comment 69 by bugdroid1@chromium.org, Jun 27 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/2be56dd4dec1cf8db29993a2c4628c485ccb9cc1

commit 2be56dd4dec1cf8db29993a2c4628c485ccb9cc1
Author: eugenebut <eugenebut@google.com>
Date: Mon Jun 27 18:55:49 2016

Project Member

Comment 72 by bugdroid1@chromium.org, Jun 29 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/14cd12c4aec0bbc13ae060f6a86cc7a33ab91664

commit 14cd12c4aec0bbc13ae060f6a86cc7a33ab91664
Author: eugenebut <eugenebut@google.com>
Date: Wed Jun 29 15:56:40 2016

Cc: kkhorimoto@chromium.org eugene...@chromium.org
Owner: ----
Status: Available (was: Assigned)
Labels: -Type-Bug Type-Feature
Blockedon: 619659
Blockedon: 619687
Cc: -eugene...@chromium.org
Owner: eugene...@chromium.org
Status: Assigned (was: Available)

Comment 78 Deleted

Project Member

Comment 79 by bugdroid1@chromium.org, Dec 13 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c64e082dd44826a935ec54019cfd2ea794808233

commit c64e082dd44826a935ec54019cfd2ea794808233
Author: eugenebut <eugenebut@chromium.org>
Date: Tue Dec 13 23:19:25 2016

[ios] Removed unused CRWWebControllerObserver methods.

Removed |handleCommand:webController:userIsInteracting:originURL:|
and |commandPrefix| methods.

BUG=616244

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

[modify] https://crrev.com/c64e082dd44826a935ec54019cfd2ea794808233/ios/web/BUILD.gn
[modify] https://crrev.com/c64e082dd44826a935ec54019cfd2ea794808233/ios/web/public/web_state/crw_web_controller_observer.h
[modify] https://crrev.com/c64e082dd44826a935ec54019cfd2ea794808233/ios/web/test/crw_fake_web_controller_observer.h
[modify] https://crrev.com/c64e082dd44826a935ec54019cfd2ea794808233/ios/web/test/crw_fake_web_controller_observer.mm
[delete] https://crrev.com/2f0ed9de3071a21b3bcf55f3c74adb09bd0c7036/ios/web/test/crw_fake_web_controller_observer_unittest.mm
[modify] https://crrev.com/c64e082dd44826a935ec54019cfd2ea794808233/ios/web/web_state/ui/crw_web_controller_observer_unittest.mm
[modify] https://crrev.com/c64e082dd44826a935ec54019cfd2ea794808233/ios/web/web_state/ui/crw_web_controller_unittest.mm
[modify] https://crrev.com/c64e082dd44826a935ec54019cfd2ea794808233/ios/web/web_state/web_controller_observer_bridge.h
[modify] https://crrev.com/c64e082dd44826a935ec54019cfd2ea794808233/ios/web/web_state/web_controller_observer_bridge.mm

Blocking: 674991
Blocking: -674991
Blockedon: 674991
Blockedon: 675001
Blockedon: 675005
Blockedon: 454984
Summary: ☂ Use only public web// API in iOS internal (was: Use only public web// API in iOS internal)
Summary: ☂ Use only public web// API in ios/chrome (was: ☂ Use only public web// API in iOS internal)
Cc: eugene...@chromium.org stuartmorgan@chromium.org droger@chromium.org shrey...@chromium.org
 Issue 583685  has been merged into this issue.
Labels: -Type-Feature Type-Task
Blockedon: 732525
Cc: -shrey...@chromium.org -stuartmorgan@chromium.org
Blockedon: -454984
Project Member

Comment 94 by bugdroid1@chromium.org, Jul 11 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/da4f211de7a88b443033d238891d4b9bb92b10df

commit da4f211de7a88b443033d238891d4b9bb92b10df
Author: eugenebut <eugenebut@chromium.org>
Date: Tue Jul 11 16:21:36 2017

Cleaned up webWillAddPendingURL:transition: callback.

Disable fullscreen inside webState:didStartNavigation: instead of
webWillAddPendingURL:transition:

BUG=616244

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

[modify] https://crrev.com/da4f211de7a88b443033d238891d4b9bb92b10df/ios/chrome/browser/tabs/tab.mm

Project Member

Comment 95 by bugdroid1@chromium.org, Aug 3 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/61ec9b3ef20c9e7f2132a033ae6f2bfad2043e10

commit 61ec9b3ef20c9e7f2132a033ae6f2bfad2043e10
Author: Eugene But <eugenebut@chromium.org>
Date: Thu Aug 03 15:15:47 2017

Added WasShown/WasHidden WebState and WebStateObserver API.

WebState's WasShown/WasHidden is called from Tab, so WebStateObserver
callbacks are actually work.
WebStateObserver's WasShown/WasHidden callbacks will allow to move
code from Tab to TabHelpers.

Bug: 616244
Change-Id: I2d8f3eb0f5854dd7204b9aec6db2aac593309638
Reviewed-on: https://chromium-review.googlesource.com/596783
Reviewed-by: Kurt Horimoto <kkhorimoto@chromium.org>
Commit-Queue: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#491739}
[modify] https://crrev.com/61ec9b3ef20c9e7f2132a033ae6f2bfad2043e10/ios/chrome/browser/tabs/tab.mm
[modify] https://crrev.com/61ec9b3ef20c9e7f2132a033ae6f2bfad2043e10/ios/web/public/test/fakes/test_web_state.h
[modify] https://crrev.com/61ec9b3ef20c9e7f2132a033ae6f2bfad2043e10/ios/web/public/test/fakes/test_web_state.mm
[modify] https://crrev.com/61ec9b3ef20c9e7f2132a033ae6f2bfad2043e10/ios/web/public/test/fakes/test_web_state_observer.h
[modify] https://crrev.com/61ec9b3ef20c9e7f2132a033ae6f2bfad2043e10/ios/web/public/test/fakes/test_web_state_observer.mm
[modify] https://crrev.com/61ec9b3ef20c9e7f2132a033ae6f2bfad2043e10/ios/web/public/test/fakes/test_web_state_observer_util.h
[modify] https://crrev.com/61ec9b3ef20c9e7f2132a033ae6f2bfad2043e10/ios/web/public/web_state/web_state.h
[modify] https://crrev.com/61ec9b3ef20c9e7f2132a033ae6f2bfad2043e10/ios/web/public/web_state/web_state_observer.h
[modify] https://crrev.com/61ec9b3ef20c9e7f2132a033ae6f2bfad2043e10/ios/web/web_state/web_state_impl.h
[modify] https://crrev.com/61ec9b3ef20c9e7f2132a033ae6f2bfad2043e10/ios/web/web_state/web_state_impl.mm
[modify] https://crrev.com/61ec9b3ef20c9e7f2132a033ae6f2bfad2043e10/ios/web/web_state/web_state_impl_unittest.mm

Project Member

Comment 96 by bugdroid1@chromium.org, Aug 4 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/0ffb61b2499a9f65399042eddbf25f4e4926f847

commit 0ffb61b2499a9f65399042eddbf25f4e4926f847
Author: Eugene But <eugenebut@google.com>
Date: Fri Aug 04 19:30:35 2017

PagePlaceholderTabHelper.

ios/web allows displaying Placeholder Overlay view using these APIs:
 - CRWWebController.usePlaceholderOverlay
 - CRWWebController.canUseViewForGeneratingOverlayPlaceholderView
 - [CRWWebController setOverlayPreviewMode:]
 - [CRWWebDelegate webController:retrievePlaceholderOverlayImage:]

There are 2 problems with this approach:
1.) Placeholder Overlay is pure UI feature which allows to display
    the the grey/colored snapshot of the web page during the page load
    (e.g. on Tab restoration after cold launch).
2.) ios/web already provides enough callbacks to support Placeholder
    Overlay by embedder and there is no need for another 4 methods just
    to support one minor feature.

This CL creates PagePlaceholderTabHelper which will replace all
existing Placeholder Overlay APIs and can be used in the new architecture.

Bug: 616244
Change-Id: I307460b8ac06e2f87f3c966c7fbb4c96591ce266
Reviewed-on: https://chromium-review.googlesource.com/585221
Commit-Queue: Eugene But <eugenebut@chromium.org>
Reviewed-by: Mark Cogan <marq@chromium.org>
Cr-Commit-Position: refs/heads/master@{#492100}
[modify] https://crrev.com/0ffb61b2499a9f65399042eddbf25f4e4926f847/ios/chrome/browser/tabs/BUILD.gn
[modify] https://crrev.com/0ffb61b2499a9f65399042eddbf25f4e4926f847/ios/chrome/browser/tabs/tab.h
[modify] https://crrev.com/0ffb61b2499a9f65399042eddbf25f4e4926f847/ios/chrome/browser/tabs/tab.mm
[modify] https://crrev.com/0ffb61b2499a9f65399042eddbf25f4e4926f847/ios/chrome/browser/tabs/tab_helper_util.mm
[modify] https://crrev.com/0ffb61b2499a9f65399042eddbf25f4e4926f847/ios/chrome/browser/web/BUILD.gn
[add] https://crrev.com/0ffb61b2499a9f65399042eddbf25f4e4926f847/ios/chrome/browser/web/page_placeholder_tab_helper.h
[add] https://crrev.com/0ffb61b2499a9f65399042eddbf25f4e4926f847/ios/chrome/browser/web/page_placeholder_tab_helper.mm
[add] https://crrev.com/0ffb61b2499a9f65399042eddbf25f4e4926f847/ios/chrome/browser/web/page_placeholder_tab_helper_delegate.h
[add] https://crrev.com/0ffb61b2499a9f65399042eddbf25f4e4926f847/ios/chrome/browser/web/page_placeholder_tab_helper_unittest.mm

Blockedon: -619971
Blockedon: 779224
Project Member

Comment 99 by bugdroid1@chromium.org, Dec 27 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/44835133e5dae4bbc458a7f75a0caa453791342a

commit 44835133e5dae4bbc458a7f75a0caa453791342a
Author: Sylvain Defresne <sdefresne@chromium.org>
Date: Wed Dec 27 12:27:10 2017

Use PagePlaceholderTabHelper to present overlay.

USe PagePlaceholderTabHelper to present the grey image overlay instead
of the CRWWebController implementation.

Bug: 616244,  620939 
Change-Id: I60589825448892ff25edd37b6cdf681b3c253892
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Reviewed-on: https://chromium-review.googlesource.com/836989
Commit-Queue: Sylvain Defresne <sdefresne@chromium.org>
Reviewed-by: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526219}
[modify] https://crrev.com/44835133e5dae4bbc458a7f75a0caa453791342a/ios/chrome/browser/tabs/tab.mm
[modify] https://crrev.com/44835133e5dae4bbc458a7f75a0caa453791342a/ios/chrome/browser/ui/browser_view_controller.mm
[modify] https://crrev.com/44835133e5dae4bbc458a7f75a0caa453791342a/ios/chrome/browser/ui/side_swipe/DEPS
[modify] https://crrev.com/44835133e5dae4bbc458a7f75a0caa453791342a/ios/chrome/browser/ui/side_swipe/side_swipe_controller.mm
[modify] https://crrev.com/44835133e5dae4bbc458a7f75a0caa453791342a/ios/chrome/browser/ui/tab_switcher/BUILD.gn
[modify] https://crrev.com/44835133e5dae4bbc458a7f75a0caa453791342a/ios/chrome/browser/ui/tab_switcher/tab_switcher_cache.mm
[modify] https://crrev.com/44835133e5dae4bbc458a7f75a0caa453791342a/ios/chrome/browser/web/page_placeholder_tab_helper.h
[modify] https://crrev.com/44835133e5dae4bbc458a7f75a0caa453791342a/ios/chrome/browser/web/page_placeholder_tab_helper.mm
[modify] https://crrev.com/44835133e5dae4bbc458a7f75a0caa453791342a/ios/chrome/browser/web/page_placeholder_tab_helper_unittest.mm
[modify] https://crrev.com/44835133e5dae4bbc458a7f75a0caa453791342a/ios/web/public/web_state/ui/crw_web_delegate.h

Blockedon: 831683
Project Member

Comment 101 by bugdroid1@chromium.org, Jun 5 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5964f7a0213d68a9ebb91fcbc80ce925fe785e6a

commit 5964f7a0213d68a9ebb91fcbc80ce925fe785e6a
Author: Eugene But <eugenebut@google.com>
Date: Tue Jun 05 15:09:54 2018

Removed extra public_deps from //ios/web:web target.

Bug: 616244
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: I3ca25359d999e8d44ec2c8e7994a5e0240c6d19a
Reviewed-on: https://chromium-review.googlesource.com/1073681
Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Commit-Queue: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#564503}
[modify] https://crrev.com/5964f7a0213d68a9ebb91fcbc80ce925fe785e6a/ios/web/BUILD.gn
[modify] https://crrev.com/5964f7a0213d68a9ebb91fcbc80ce925fe785e6a/ios/web/public/test/BUILD.gn

Components: Mobile>iOSWeb
Components: -Mobile>WebView>Glue
Components: -Mobile>iOSWeb Mobile>iOSWeb>PublicAPI
Showing comments 5 - 104 of 104 Older

Sign in to add a comment