This change has been discarded.

Describe the reason it's discarded (optional):

It is unlikely to achieve the in_tab scrren shot through javascript injection; Since different website has different DOM structure and the injection of elements may break it; futhurmore, some website does not allowed the script injections for security reasons.

Summary

[WIP] Screen shot should be taken in the website

Review Request #9765 — Created March 10, 2018 and discarded 7 years, 1 month ago — Latest diff uploaded 7 years, 1 month ago

Information

Owner

ridum*

Repository

rb-screenshot

Branch

master

Bugs

Depends On

Reviewers

Groups

rb-extension-pack, students

People

Description*

The screenshot cropping features are now implemented by first screen shooting the whole page than move it to a new tab then crop it.
The cropping features should occur at the targeting tab using script injection.

Testing Done

Diff Revision 1

This is not the most recent revision of the diff. The latest diff is revision 2. See what's changed.

orig

	README.md
	crx/manifest.json
	crx/js/background.js
	crx/js/capture.js

README.md

crx/manifest.json

crx/js/background.js

crx/js/capture.js