Skip to content
This repository has been archived by the owner on Apr 3, 2020. It is now read-only.

[Printing] Disable print preview to enable basic printing #374

Closed
wants to merge 1 commit into from
Closed

Conversation

mrunalk
Copy link
Contributor

@mrunalk mrunalk commented Aug 16, 2016

This is related to crosswalk-project/crosswalk#3851
to enable window.print() functionality.

BUG=XWALK-5680

This is related to crosswalk-project/crosswalk#3851
to enable window.print() functionality.

BUG=XWALK-5680
@crosswalk-trybot
Copy link

crosswalk-trybot commented Aug 16, 2016

Testing patch series with mrunalk/chromium-crosswalk@1cadc70 as its head.

Bot Status
Content Shell Android-x86 [SUCCESS 💚](https://build.crosswalk-project.org/try/builders/Content Shell Android-x86/builds/326)
Content Shell Linux [SUCCESS 💚](https://build.crosswalk-project.org/try/builders/Content Shell Linux/builds/324)

@rakuco
Copy link
Member

rakuco commented Aug 17, 2016

Why not change the default in Crosswalk's build/common.gypi? It'd also be good to explain why the print code is broken when this flag is on.

@mrunalk
Copy link
Contributor Author

mrunalk commented Aug 17, 2016

Ahh I didn't know crosswalk has it's own build/common.gypi too. Thanks. I will close this PR and add this change to the other PR with additional notes as to why we need this change.

@mrunalk mrunalk closed this Aug 17, 2016
@mrunalk mrunalk deleted the printing branch August 26, 2016 22:18
imreotto pushed a commit to tenta-browser/chromium-crosswalk that referenced this pull request Nov 2, 2017
Bug: 763036
Change-Id: I6820caddfc64889bb35eb00788285a629f8fd623
Reviewed-on: https://chromium-review.googlesource.com/657637
Commit-Queue: Justin Cohen <[email protected]>
Reviewed-by: Justin Cohen <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#500606}(cherry picked from commit 77f0bf1)
Reviewed-on: https://chromium-review.googlesource.com/677343
Reviewed-by: Olivier Robin <[email protected]>
Cr-Commit-Position: refs/branch-heads/3202@{crosswalk-project#374}
Cr-Branched-From: fa6a5d8-refs/heads/master@{#499098}
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants