DescriptionRevert of Set user agent type of transient item the same as pending item. (patchset #9 id:280001 of https://codereview.chromium.org/2779263002/ )
Reason for revert:
Revert because this CL is breaking downstream tests.
Original issue's description:
> Set user agent type of transient item the same as pending item.
>
> In current code, when deciding which user agent type to use for
> pre-loading next item, the user agent of transient item is also taken
> into consideration, and because the user agent type of transient item
> is always MOBILE, this results in the following behavior:
>
> Goto www.facebook.com -> Request Desktop Site ->
> Goto https://expired.badssl.com -> Goto www.amazon.com.
>
> Now, instead of visiting the desktop version of www.amazon.com, it's
> visiting the mobile version.
>
> This CL fixes the issue by accurately set the user agent type of
> transient item as the user agent type of corresponding pending item.
>
> BUG=706446
>
> Review-Url: https://codereview.chromium.org/2779263002
> Cr-Commit-Position: refs/heads/master@{#461856}
> Committed: https://chromium.googlesource.com/chromium/src/+/0c1d4036c5f4aafb0530816aafce288cc56a691b
TBR=eugenebut@chromium.org,kkhorimoto@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=706446
Review-Url: https://codereview.chromium.org/2798813002
Cr-Commit-Position: refs/heads/master@{#461986}
Committed: https://chromium.googlesource.com/chromium/src/+/2b3e817a6ccd91bc68501777d88659d429ca50cd
Patch Set 1 #
Messages
Total messages: 8 (4 generated)
|